Giter VIP home page Giter VIP logo

Comments (7)

lizziel avatar lizziel commented on June 1, 2024

Let's set the values to NaN so it doesn't appear that the version outputs the variables with all zeros. The diff and frac diff plots can be grayed out to be clear that the comparison is not applicable.

from gcpy.

yantosca avatar yantosca commented on June 1, 2024

I have pushed commit d872148, which will now print emissions totals for all variables that are present in at least one of the Ref and Dev datasets. Missing values will now show up as "nan" in the emissions totals.

We will also make similar edits to the plotting routines so that missing values will be shown with NaNs. That will take a bit longer.

from gcpy.

yantosca avatar yantosca commented on June 1, 2024

I have pushed commit 7df6df1 to master. This will now plot variables that are in Ref but not Dev (or vice-versa) as all grey in surface or zonal mean plots. This commit should resolve this issue (but I'll leave the issue open in case further tweaks are needed).

from gcpy.

yantosca avatar yantosca commented on June 1, 2024

I have pushed commit 7df6df1 to master. This will now plot variables that are in Ref but not Dev (or vice-versa) as all grey in surface or zonal mean plots. This commit should resolve this issue (but I'll leave the issue open in case further tweaks are needed).

from gcpy.

yantosca avatar yantosca commented on June 1, 2024

I have pushed commit 6900abe to master. This now fixes some issues in the benchmark plots caused incorrect tests for arrays that are all zero and arrays that are all NaN. We now use more robust tests.

This now produces plots that look like this for GC-Classic:

(1) Emissions plot for CH4 anthro, which is zero in both Ref and Dev:

gcpy_fix_sfc

(2) Concentration plot (single level) for a species in "Ref" that is not present in "Dev":

gcpy_fix_1

(3) Zonal mean plot for a species in Ref that is not in Dev:

gcpy_fix_zm

from gcpy.

yantosca avatar yantosca commented on June 1, 2024

I also discovered that some plots that have a missing species in one dataset can cause the color range for the non-missing species to be incorrectly plotted:

In this plot, EmisDST1_Anthro is missing in Ref, but present in Dev. The Ref panel, as well as all of the absolute difference panels should be plotted as all gray, but are not:

nan1

Upon further review, we had to rewrite the logic in how the min and max values for the data ranges (i.e. vmin0, vmax0, vmin1, vmax1, vmin, vmax) were computed in order to better account for NaN values. This was added to commit 8d8282c

This commit now fixes the issue:

nan2

from gcpy.

yantosca avatar yantosca commented on June 1, 2024

Added 90fed1a to fix an typo that caused single-level plots with a missing species in Ref to be plotted incorrectly when using the RdBu color scale. This commit now produces the correct behavior:

nan3

from gcpy.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.