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

TRD: fixes in raw data QC #1573

Merged
merged 1 commit into from
Nov 23, 2022
Merged

Conversation

martenole
Copy link
Contributor

Hi @bazinski and @tdietel
this contains some improvements for the raw data QC. It fixes the wrong binning in parsing error graphs, makes the axis labels more consistent and adds a simple histogram for statistics (number of TFs, triggers, etc). There are some examples in the test QCG.
The link graphs require also AliceO2Group/AliceO2#10315 to be meaningful, but it also does at least not crash without that patch.
Cheers,
Ole

parsing-new

@Barthelemy
Copy link
Collaborator

@TimoWilken @ktf error on the mac :

Collecting PyYAML==5.1
  Using cached PyYAML-5.1.tar.gz (274 kB)
  Preparing metadata (setup.py): started
  Preparing metadata (setup.py): finished with status 'error'
  error: subprocess-exited-with-error
  
  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [1 lines of output]
      ERROR: Can not execute `setup.py` since setuptools is not available in the build environment.
      [end of output]
  
  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

@martenole
Copy link
Contributor Author

CI failure is unrelated. Could this be merged please @Barthelemy ?

@Barthelemy Barthelemy merged commit 4a1f071 into AliceO2Group:master Nov 23, 2022
@martenole martenole deleted the trdfixes branch November 23, 2022 15:05
@martenole
Copy link
Contributor Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants