search

lanl / SEPIA

Simulation-Enabled Prediction, Inference, and Analysis: physics-informed statistical learning.
Other
29 stars 6 forks source link

Stop tracking binaries and ipynb_checkpoints #36

Open luiarthur opened 3 years ago

luiarthur commented 3 years ago

Currently, it looks like some stuff in .ipynb_checkpoints/ are being tracked by git.

Also, binaries like images and pickle files from demos are also being tracked. Those should ideally be removed, unless there's a reason to keep them, especially since they are already tracked in the notebooks.

Scripts converted from notebooks should also be removed, again unless there's a strong reason to track them.

@jgattiker, @granthutchings Is it OK if we stop tracking some of these files?

luiarthur commented 3 years ago

I see some of the pickle files are used to test if a model can be restored, those I think can stay.

luiarthur commented 3 years ago

Better yet, the pickle files could be created during the tests, instead of created beforehand. That way they don't need to be tracked.

jgattiker commented 3 years ago

Yeah, definitely. I'd like to have the core distribution be smaller, to more easily copy to non-networked systems. Any way to make these lighter is great, and maybe there's a better way to segment this from the operational part of the distribution?

jgattiker commented 3 years ago

Also, the code that validates results against matlab results are pretty uninteresting to the average user. Maybe these should be in a separate validation repo? I don't want to lose them, but I think they're pretty much mothballed now.

granthutchings commented 3 years ago

Yes, I think it’s right to stop tracking these.

On Aug 20, 2021, at 1:12 PM, Arthur Lui @.***> wrote:

Currently, it looks like some stuff in .ipynb_checkpoints/ are being tracked by git.

Also, binaries like images and pickle files from demos are also being tracked. Those should ideally be removed, unless there's a reason to keep them, especially since they are already tracked in the notebooks.

Scripts converted from notebooks should also be removed, again unless there's a strong reason to track them.

@jgattiker https://github.com/jgattiker, @granthutchings https://github.com/granthutchings Is it OK if we stop tracking some of these files?

See some of the binaries and checkpoints being tracked in the diffs: https://github.com/lanl/SEPIA/compare/dev https://github.com/lanl/SEPIA/compare/dev — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/lanl/SEPIA/issues/36, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKJE6QYCSLZTZOEM5NWBWHDT52ZLRANCNFSM5CQ6IYYQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.