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

several fixes #73

Merged
merged 4 commits into from
Mar 10, 2023
Merged

several fixes #73

merged 4 commits into from
Mar 10, 2023

Conversation

sofia-calgaro
Copy link
Collaborator

@sofia-calgaro sofia-calgaro commented Mar 10, 2023

  • plotting.plot_per_string() is back to its original form (all strings in one figure) -> you can still obtain one string per figure starting from an external code, you just subselect the dataframe section corresponding to the string of interest instead of passing the full dataframe as input
  • fixed if statements for resampled option in plotting.py
  • removed "if pulser" in plotting.plot_per_string() (wrong copy+paste): the function perfectly works also for one channel only, as it is the pulser
  • fixed colours when selecting different resampled options
  • fixed data_analysis.data into data_analysis (dataframe object) in all functions of plotting.py but make_subsystem_plots() (left some freedom there) and inside status_plot.py

@sofia-calgaro sofia-calgaro merged commit 8ffdcb6 into legend-exp:dev Mar 10, 2023
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