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

EgammaV/ConversionValidator DQM ouptut is unstable in workflow 11634 #28072

Closed
schneiml opened this issue Sep 26, 2019 · 8 comments
Closed

EgammaV/ConversionValidator DQM ouptut is unstable in workflow 11634 #28072

schneiml opened this issue Sep 26, 2019 · 8 comments

Comments

@schneiml
Copy link
Contributor

While validating #28042 I noticed that many plots in the EgammaV/ConversionValidator hierarchy are unstable in the PR tests. This seems to only affect workflow 11634, while the same plots seem stable in WF 11624 (which only exists in 10_6, while 11634 is only in 11?).

We could blacklist it, but this might point to a deeper problem somewhere else.

# voms-proxy-init required
WF=11634
ME=EgammaV/ConversionValidator/ConversionInfo/convEtaAss
curl -E /tmp/x509up_u$(id -u) https://cmsweb.cern.ch/dqm/dev/data/json/samples | python -m json.tool  | grep -o '"[^"]*'$WF'.*"' | tr -d '"' > datasets
echo "Got $(wc -l datasets)."
for d in $(cat datasets); do 
  echo -n $d ';'
  curl -E /tmp/x509up_u$(id -u) https://cmsweb.cern.ch/dqm/dev/jsonfairy/archive/1$d/$ME ; echo
done > mes
echo "ME did not change in (at least) $(grep -c unsupported mes) tests"
echo "ME did change in $(grep -c title mes) tests"

Says:

ME did not change in (at least) 703 tests
ME did change in 36 tests

for wf 11624, but

ME did not change in (at least) 101 tests
ME did change in 54 tests

for wf 11634.

@cmsbuild
Copy link
Contributor

A new Issue was created by @schneiml Marcel Schneider.

@davidlange6, @Dr15Jones, @smuzaffar, @fabiocos, @kpedro88 can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@Dr15Jones
Copy link
Contributor

assign dqm, reconstruction

@cmsbuild
Copy link
Contributor

New categories assigned: dqm,reconstruction

@jfernan2,@slava77,@andrius-k,@schneiml,@fioriNTU,@perrotta,@kmaeshima you have been requested to review this Pull request/Issue and eventually sign? Thanks

@fabiocos
Copy link
Contributor

I believe that the first appearance of the issue in a merged PR is for #28011 , so I would say that the problem is present already in 11_0_0_pre9

@jfernan2
Copy link
Contributor

Hi, I believe this issue has been solved

@slava77
Copy link
Contributor

slava77 commented May 1, 2020

Hi, I believe this issue has been solved

@jfernan2
do you happen to know the origin of the fix?

@slava77
Copy link
Contributor

slava77 commented May 1, 2020

+1

this is apparently resolved, likely not related to reco in the end

@smuzaffar
Copy link
Contributor

this has been resolved

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

No branches or pull requests

7 participants