use python3 compatible syntax in DQMOffline packages

[davidlange6]

PR description:

adapt to

• lack of xrange function in python 3
• keys() is now an iterator
• integer division is //

[davidlange6]

please test

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26364/9095

• This PR adds an extra 24KB to repository

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/33986/console Started: 2019/04/04 21:30

[cmsbuild]

A new Pull Request was created by @davidlange6 (David Lange) for master.

It involves the following packages:

DQMOffline/L1Trigger
DQMOffline/Trigger

@andrius-k, @kmaeshima, @schneiml, @cmsbuild, @rekovic, @jfernan2, @fioriNTU can you please review it and eventually sign? Thanks.
@kreczko, @battibass, @mtosi, @jhgoh, @calderona, @HuguesBrun, @thomreis, @trocino, @folguera, @rociovilar this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

+1
Tested at: da09b6a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-26364/33986/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-26364/33986/summary.html

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 32
• DQMHistoTests: Total histograms compared: 3140495
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3140297
• DQMHistoTests: Total skipped: 197
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 31 files compared)
• Checked 133 log files, 14 edm output root files, 32 DQM output files

[andrius-k]

+1

[kreczko]

BTW, just out of curiosity: Why this import instead of using six.moves.range (as you've used six before)?

[davidlange6]

hi- I guess it was the first solution suggested by google in this case. Both solutions provide the eventual python3 behavior.

[fabiocos]

@rekovic please check, it looks ok to me, I will move forward with the integration for next IB

[fabiocos]

+1

[fabiocos]

merge