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

fix: dRICH sensor material should not be AirOptical #106

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

c-dilks
Copy link
Member

@c-dilks c-dilks commented Sep 8, 2022

Briefly, what does this PR introduce?

Close #18

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue #__)
  • Documentation update
  • Other: improve material property tables

Please check if this PR fulfills the following:

  • Tests for the changes have been added
  • Documentation has been added / updated
  • Changes have been communicated to collaborators

Does this PR introduce breaking changes? What changes might users need to make to their code?

No

Does this PR change default behavior?

No

@veprbl veprbl added the topic: PID Particle identification label Oct 1, 2022
@kkauder
Copy link
Contributor

kkauder commented Dec 12, 2023

@c-dilks @chchatte92 Is this issue solved? When this file moved, git didn't understand that and marked it as deleted, and the new one is treated as a different one, so you can't easily compare/merge. Also, the current one looks to my eye substantially different, so I can't easily identify which instance of AirOptical needs (or needed) to be changed.

@rahmans1
Copy link
Contributor

rahmans1 commented Feb 6, 2024

@chchatte92 We are tagging a new release for the February campaign soon. Is there anyone working on this PR? Is it expected to be ready for February release?

@chchatte92
Copy link
Member

@chchatte92 We are tagging a new release for the February campaign soon. Is there anyone working on this PR? Is it expected to be ready for February release?

Dear Sakib, I will get back to you soon (either later today or in the weekends)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: PID Particle identification
Projects
Development

Successfully merging this pull request may close these issues.

dRICH: sensor material should not be AirOptical
6 participants