Skip to content

Updated tprofile to teprofile and nprofile to neprofile.#3543

Merged
grmtrkngtn merged 4 commits intomainfrom
3463-tprofile-and-nprofile-should-really-be-renamed-teprofile-and-neprofile
Feb 17, 2025
Merged

Updated tprofile to teprofile and nprofile to neprofile.#3543
grmtrkngtn merged 4 commits intomainfrom
3463-tprofile-and-nprofile-should-really-be-renamed-teprofile-and-neprofile

Conversation

@grmtrkngtn
Copy link
Copy Markdown
Collaborator

Description

Renamed TProfile to TeProfile, to highlight that it is an electron profile. Slight update to docstring.

Likewise for NProfile.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 14, 2025

Codecov Report

Attention: Patch coverage is 85.71429% with 2 lines in your changes missing coverage. Please review.

Project coverage is 31.31%. Comparing base (b61a1c5) to head (23c71a4).

Files with missing lines Patch % Lines
process/io/plot_proc.py 50.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3543   +/-   ##
=======================================
  Coverage   31.31%   31.31%           
=======================================
  Files          81       81           
  Lines       19515    19515           
=======================================
  Hits         6111     6111           
  Misses      13404    13404           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Collaborator

@chris-ashe chris-ashe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just a few small points to look over for clarity

Comment thread process/io/plot_proc.py Outdated
Comment thread process/io/plot_proc.py Outdated
Comment thread process/impurity_radiation.py Outdated
Copy link
Copy Markdown
Collaborator

@chris-ashe chris-ashe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@grmtrkngtn grmtrkngtn merged commit 069a00b into main Feb 17, 2025
@chris-ashe chris-ashe deleted the 3463-tprofile-and-nprofile-should-really-be-renamed-teprofile-and-neprofile branch February 17, 2025 10:33
ajpearcey pushed a commit that referenced this pull request Feb 26, 2025
* Updated tprofile to teprofile and nprofile to neprofile.

* "Updated docs with teprofile and neprofile change"

* update profile names

* added explicit reference to electron
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.

TProfile and NProfile should really be renamed TeProfile and NeProfile

3 participants