Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated FeatureClassifier: partial update, still integrate to 1e4 #40

Closed
wants to merge 133 commits into from

Conversation

Ethadhani
Copy link
Contributor

Made multiple changes that only affect FeatureClassifier

Integration is still to 1e4 orbits of inner most planet

  • Changed implementation of how features are generated
  • Ensured model compatibility with new version of REBOUND
  • Re-trained model with cleaned data and tested impact of duplications/mislabeling in original dataset
  • Removed systems that go unstable in short integration from testing dataset
  • Updated saved model

Change comparisons, tests, and backwards compatibility

In paper_plots

  • testUnstable.ipynb demonstrates change in AUC calculation to not include systems that go unstable in short integration
  • cleanVnonClean.ipynb compares model trained with clean data verses not clean data.
  • oldReboundNewRebound.ipynb compares old rebound verses new rebound

commit 2a79a42 contains additional info and comparisons of:

  • Old REBOUND to new REBOUND with training/testing
  • Model trained with clean verses dirty data
  • Initial Tsec tests

commit 9468319 contains comparisons for old feature generation implementation verses new implementation.

commit 086be72 contains all updates except for Tsec related integration and features, this version is avalable for backwards compatibility.

Data filtration/clean data generation is done in accordance to, and with the data from, 10.5281/zenodo.14327471

Ethadhani added 30 commits July 10, 2024 22:25
…arison. This new method allows more flexibility for future advancements/new features without impacting the deep regressor, while decreasing computation time.
…dentical features, proper documentation to come
…e and full examples that the old feature generation method and new feature generation method yield the exact same results and are compatible to each other
…od, also isolated featureclassifier from other models such that featureclassifier.py, ClassifierSeries.py, and features.py operate together and independently from the rest of the spock system to allow changes to be made
… make a copy of simulation if using new rebound, this is because old rebound does not support sim.copy()
…n. Was supposed to state, fixed pool function to generate training data and removed redundant file, made it so that the training data pool runs through jupyter notebook
…fically accounted for rebound.Collision and rebound.Ejected
Ethadhani and others added 28 commits November 27, 2024 15:56
fixed depreciating warning by updating used XGBoost version, this also decreases training time and increases performance by marginal amount. (was using 1.7, now 2.1)
Copy link

gitnotebooks bot commented Dec 23, 2024

Found 11 changed notebooks. Review the changes at https://app.gitnotebooks.com/dtamayo/spock/pull/40

@Ethadhani Ethadhani closed this Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant