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

Adding Plotting Functions for Re-extraction #7

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

cmccully
Copy link
Contributor

@cmccully cmccully commented Oct 4, 2024

This PR adds the frontend changes and callbacks necessary to trigger banzai-floyds to re-extract data. I had to repackage many of the plots to make the callbacks work. Dash only allows elements to be the target of a single callback complicating some of the page structure. The performance for updating graphs was horrifically slow if you had to pass the whole figure between the front and backend. To alleviate that, I've written python clientside callbacks that wrap some basic javascript to do the plot updates on the client end.

I've included a mock archive client to make the testing loop faster. This looks for banzai reduced data in the sub directory wherever you run mock_archive.py.

I've moved a lot of the data processing that was happening in the frontend to only happen in the main banzai-floyds pipeline. The current version of the pipeline that works with this pr is https://github.com/LCOGT/banzai-floyds/tree/reextract. I can send you preprocessed data if that is easier.

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