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

Using BTAX DataFrames for Dynamic Plotting #634

Closed
brittainhard opened this issue Aug 22, 2017 · 4 comments
Closed

Using BTAX DataFrames for Dynamic Plotting #634

brittainhard opened this issue Aug 22, 2017 · 4 comments
Assignees

Comments

@brittainhard
Copy link
Contributor

@hdoupe @MattHJensen @PeterDSteinberg

The central problem is that the data returned from BTAX to CCC cannot be used for plotting -- specifically the plots that Haylee and Dora have created.

BTAX serializes the original DataFrames for use in the HTML table, but it doesn't preserve the original structure of the many DataFrames that BTAX returns. They need to change the BTAX code to return serialized versions of those DataFrames alongside the JSON to be used for the HTML table.

If they can have access to the original DataFrames, then they can use them to create dynamic plots to be put onto the CCC results page alongside the HTML table.

They need to change the BTAX function to return the DataFrames, and we need to include that new version into our next release. Their deadline is the end of this week.

@jdebacker
Copy link
Contributor

@brittainhard I think this issue can be closed. Can you confirm?

@brittainhard
Copy link
Contributor Author

@jdebacker does it still apply to the bubble plot PR?

@jdebacker
Copy link
Contributor

@brittainhard I think the fix to B-Tax noted here was completed with B-Tax PR #171.

With that, I think Haylee should be able to complete Web-app Public PR #642.

@brittainhard
Copy link
Contributor Author

Ok, I'll close it.

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

No branches or pull requests

2 participants