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

CLIC dumper, PFNetDense updates #69

Merged
merged 19 commits into from
Jul 9, 2021
Merged

CLIC dumper, PFNetDense updates #69

merged 19 commits into from
Jul 9, 2021

Conversation

jpata
Copy link
Owner

@jpata jpata commented Jun 22, 2021

  • add a dumper for CLIC .slcio files
  • minor fix in PFNetDense distance matrix masking
  • optional momentum layer separation in PFNetDense with separate_momentum: yes
  • CMS-specific input encoding that precomputes sin, cos of angles, log of energy and pt
  • additional metrics: per-class recall (confusion matrix diagonal items)
  • optionally compute the loss only on signal particles, masking the hits with no associated particle, enabled with sample_weights: signal_only
  • make sample_weights configurable for each loss component (e.g. train regression losses masking only the signal particles, while the classification loss using all particles)
  • add a --modifier retrain_energy option to modify the configuration to retrain only the energy regression layer (PFNetDense)
  • benchmark model training with cms-gnn-dense-focal and the commit dc0dae6
    • performance summary at 300 epochs: indico
    • weights at /eos/user/j/jpata/mlpf/models/cms/2021_07_02/cms-gnn-dense-focal-285ae825.gpu0.local

@jpata jpata changed the title CLIC dumper CLIC dumper, PFNetDense updates Jun 28, 2021
@jpata jpata merged commit 8358c8e into master Jul 9, 2021
@jpata jpata deleted the clic branch August 31, 2021 08:32
jpata added a commit that referenced this pull request Sep 15, 2023
CLIC dumper, PFNetDense updates

Former-commit-id: 8358c8e
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