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.

Sign up for GitHub

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

Jump to bottom

Bugzilla 1972: add_hist_coord - look into misleading overloading #2

Open
cacraigucar opened this issue Oct 2, 2019 · 0 comments
Open

Bugzilla 1972: add_hist_coord - look into misleading overloading #2

cacraigucar opened this issue Oct 2, 2019 · 0 comments
Assignees
@gold2718
Labels
bug Something isn't working correctly
Milestone
cam_history

Comments

@cacraigucar
Copy link
Collaborator

Opened: Cheryl Craig 2014-05-08 14:12:48 MDT
add_hist_coord has 3 overloaded specific routines.

The actual minimal user interface is (name, value, label). If a user neglects to specify 3 values and only gives two (name, value), one of the overloaded routines (add_hist_coord_regonly) is called and value is overwritten with index.

This is misleading and should probably look at whether add_hist_coord_regonly should be part of the add_hist_coord overloading or not.

Reviewed: Cheryl Craig 2016-05-04 14:12:07 MDT
This might go away with the refactor.
jimmielin pushed a commit to jimmielin/CAM that referenced this issue May 31, 2020
@fritzt
Merge pull request ESCOMP#2 from jimmielin/staging/cesm_gc_hemco
c4ad0c1 
Fix an infinite loop issue
jtruesdal added a commit that referenced this issue Aug 6, 2020
@jtruesdal
Merge pull request #2 from ESCOMP/cam_development
267944e 
Cam development merge latest cam6_2_037
@cacraigucar cacraigucar added the bug Something isn't working correctly label Nov 2, 2020
gold2718 pushed a commit to gold2718/CAM that referenced this issue Jan 27, 2021
@cacraigucar
Merge pull request ESCOMP#2 from gold2718/update_externals_to_cesm2_3…
ba3c91c 
…_alpha02a

Fix for missing no_fill argument
jimmielin pushed a commit to jimmielin/CAM that referenced this issue Feb 24, 2021
@fritzt
Merge pull request ESCOMP#2 from fritzt/dev
6019c86 
[CESM-GC] Add MAM4 coupling
@xhuang-ncar xhuang-ncar mentioned this issue Oct 5, 2021
Closed
@cacraigucar cacraigucar mentioned this issue Dec 3, 2021
Closed
fvitt added a commit that referenced this issue Jan 19, 2022
@fvitt
Merge pull request #2 from billsacks/ozone2surf
e45697f 
Rename flag values and fix xml issue
@gold2718 gold2718 added this to the cam_history milestone Mar 16, 2022
@cacraigucar cacraigucar mentioned this issue Aug 8, 2022
Merged
gold2718 pushed a commit to gold2718/CAM that referenced this issue Apr 24, 2023
@DirkOlivie
Merge pull request ESCOMP#2 from matsbn/feature-replace-MICOM
cde84e5 
Replaced MICOM with BLOM in bld/config_files/definition.xml.
gold2718 pushed a commit to gold2718/CAM that referenced this issue Apr 24, 2023
@MichaelSchulzMETNO
Merge pull request ESCOMP#2 from monsieuralok/cam_cesm2_1_rel_05-Nor
ed3f8ba 
initialisation and redundant code merge
@cacraigucar cacraigucar mentioned this issue Aug 11, 2023
Merged
gold2718 pushed a commit to gold2718/CAM that referenced this issue Nov 29, 2023
@mvertens
Merge pull request ESCOMP#2 from gold2718/untabify
f4bfd9e 
Untabify config_compsets.xml
@adamrher adamrher mentioned this issue Feb 22, 2024
Open
@lizziel lizziel mentioned this issue Mar 21, 2024
Closed
gold2718 pushed a commit to gold2718/CAM that referenced this issue Jun 24, 2024
@oyvindseland
Merge pull request ESCOMP#2 from gold2718/forces_fix
2cc8526 
Fix PE-based bugs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gold2718 gold2718

Labels
bug Something isn't working correctly
Projects
CAM Development
Status: To Do
Milestone
cam_history
Development

No branches or pull requests
2 participants
@gold2718 @cacraigucar