Skip to content

Jomanw asteroids#173

Open
jcswong wants to merge 7 commits into
aburgasser:mainfrom
jcswong:jomanw-asteroids
Open

Jomanw asteroids#173
jcswong wants to merge 7 commits into
aburgasser:mainfrom
jcswong:jomanw-asteroids

Conversation

@jcswong

@jcswong jcswong commented Aug 1, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

overview:

  • adds asteroid.py in splat/splat/ with PCA capabilities
  • adds new object classes -- AsteroidSpectrum & AsteroidTemplate
  • adds functionality to digest "visnir" and MITHNEOS-type data structure to create an AsteroidSpectrum instance

file additions:

  • adds DeMeo 2009 IR PCA vectors in splat/resources/SpectralTemplates/DemeoAsteroids/pca/
  • adds asteroid_tutorial.ipynb in splat/tutorials/
  • creates a folder with 2 spectra for asteroid_tutorial.ipynb at splat/tutorials/tutorial_asteroids

jcswong and others added 7 commits July 30, 2023 22:14
@jcswong
added PCA vectors to a folder within asteroid templates
e17e0d9
@jcswong
asteroid.py modifications
403f693
@jcswong
asteroid_tutorial.ipynb
d797339 
and two asteroid spectra in the tutorial folder
@jcswong
MITHNEOS_to_Spectrum reading path convention fix
52fd436 
/ or \\ convention differs by OS and user, splitting by only "/" could cause issues. Using os.path.abspath('some/path') mitigates this issue as its output is OS dependent. Also, all files from MITHNEOS are named with the observation date, this is currently included with the AsteroidSpectrum name.
@jcswong
classifyAsteroid shallow copy/reference object changed to deepcopy
2158a4a
@jcswong
minor edits
ae23983 
removed recently added debugging code. Code was added to uncover an improperly labeled omitted data point in a MITHNEOS spectrum
@jcswong
reverted previous change of variable type in MITHNEOS_to_spectrum
64d11d7 
line 816, int to float. Previously changed from float to int, but, as it turns out, not all data in MITHNEOS have the same number of columns. Some 200-300 are missing the 4th column for number of frames used as an average to create the data point. Hence, the column with index -1 is the noise as a string, hence throws an exception when casted to int from base 10.
@aburgasser aburgasser self-assigned this Jul 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@aburgasser aburgasser

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jcswong @aburgasser