+
Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: geoschem/gcpy
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: geoschem/gcpy
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: dev
Choose a head ref
  • 10 commits
  • 7 files changed
  • 2 contributors

Commits on Jun 25, 2025

  1. Add vtune_plot_hotspots.py, based on gprofng_functions.py

    gcpy/plot/core.py
    - Moved function "gprofng_text_to_data_units" (from module
      "gprofng_functions") to this module as "text_to_data_units".
      This will make it available as a general plotting function.
    
    gcpy/profile/gprofng_functions.py
    - Import "text_to_data_units" from "gcpy.plot.core"
    - Fixed incorrect error message string
    
    gcpy/profile/vtune_plot_hotspots
    - Added this function to create a bargraph from Intel VTune
      hotspot reports.
    
    CHANGELOG.md
    - Updated accordingly
    
    Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
    yantosca committed Jun 25, 2025
    Configuration menu
    Copy the full SHA
    53d2690 View commit details
    Browse the repository at this point in the history
  2. Add RTD doc for plotting hotspots from Intel VTune

    docs/source/Code-Profiling.rst
    - Added a section describing how to plot hotspots with
      gcpy.profile.vtune_plot_hotspots.
    
    CHANGELOG.md
    - Updated accordingly
    
    Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
    yantosca committed Jun 25, 2025
    Configuration menu
    Copy the full SHA
    8c37b18 View commit details
    Browse the repository at this point in the history
  3. Drop certain system calls from being plotted by vtune_plot_hotspots.py

    gcpy/profile/vtune_plot_hotspots.py
    - Skip plotting entries such as "wait.h", "mutex.c", etc, as
      these can have multiple entries in an Intel VTune report, which
      will confuse the bargraph plotting.  We mostly care about user
      routines that can be modified to save time, rather than system
      calls.
    
    Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
    yantosca committed Jun 25, 2025
    Configuration menu
    Copy the full SHA
    857bbe8 View commit details
    Browse the repository at this point in the history
  4. Prevent VTune output from spoofing benchmark_scrape_gcclassic_timers

    gcpy/benchmark/modules/benchmark_scrape_gcclassic_timers.py
    - Updated the marker that denotes when to stop reading from a
      GEOS-Chem Classic log file from "----------" to "-"*78 (that is,
      78 dashes in a row).  This is necessary in order to prevent text
      appended by Intel VTune to the end of a GEOS-Chem log file from
      interfering with the scraping of the timers.
    
    CHANGELOG.md
    - Updated accordingly
    
    Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
    yantosca committed Jun 25, 2025
    Configuration menu
    Copy the full SHA
    fed4082 View commit details
    Browse the repository at this point in the history

Commits on Jun 26, 2025

  1. Merge PR #365 (F )

    This merge brings PR # (Update the "end of search" criteria in
    benchmark_scrape_classic_timers.py to avoid being spoofed by
    output from Intel VTune, by @yantosca) into the GCPy development
    stream.
    
    In gcpy/benchmark/modules/benchmark_scrape_gcclassic.timers.py, the
    code starts reading the timers information backwards from the end of the
    file. The signal to stop reading was if a line contained "----------".
    However, when output from the Intel VTune profiler is appended to the
    end of a GEOS-Chem Classic log file, there is a line with at least
    as many "-" characters, which causes the parsing of the file to end before
    the timing information is reached.
    
    The solution is to change the criteria to look for a line with 78 "-"
    characters in a row (which occurs just before GEOS-Chem timer row. This
    will allow the parsing of the log file to proceed normally.
    
    Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
    yantosca committed Jun 26, 2025
    Configuration menu
    Copy the full SHA
    5deccdc View commit details
    Browse the repository at this point in the history
  2. Merge PR #364 (Add script to plot hotspots from Intel VTune reports)

    This merge brings PR #364 (Add script to plot hotspots from Intel VTune
    reports, by @yantosca) into the GCPy development stream.
    
    PR #364 does the following:
    
    1. Adds gcpy/profile/vtune_plot_hotspots.py, which can create
       bargraph plots (similar to gprofng_functions.py) from hotspot reports
       from Intel VTune. This tool accepts hotspot reports listed by function
       and by line.
    
    2. Moves the gprofng_text_to_data_units functions out of
       gcpy/profile/gprofng_functions.py to gcpy/plot/core.py, so that it can
       be used as a generic plotting function.
    
    3. Updates ReadTheDocs documentation in docs/source/Code-Profiling.rst
       accordingly.
    
    Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
    yantosca committed Jun 26, 2025
    Configuration menu
    Copy the full SHA
    4764def View commit details
    Browse the repository at this point in the history

Commits on Jul 14, 2025

  1. Fix grid area calculation

    1Dandan committed Jul 14, 2025
    Configuration menu
    Copy the full SHA
    4b98df7 View commit details
    Browse the repository at this point in the history
  2. TBD releasing

    1Dandan committed Jul 14, 2025
    Configuration menu
    Copy the full SHA
    9b2a596 View commit details
    Browse the repository at this point in the history
  3. Merge PR #367 (Fix grid area calculation)

    This merge brings PR #367 (Fix grid area calculation #367,
    by @1Dandan) into the GCPy 1.6.3 development stream.
    
    PR #367 fixes an issue with the computation of grid areas for
    GCHP cubed-sphere and stretched grids.
    
    Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
    yantosca committed Jul 14, 2025
    Configuration menu
    Copy the full SHA
    903c9cc View commit details
    Browse the repository at this point in the history

Commits on Jul 16, 2025

  1. Hotfix: Remove duplicate entries for GCPy 1.6.2 in CHANGELOG.md

    CHANGELOG.md
    - Removed duplicated entries for version 1.6.2.  These were
      probably a result of Git automatic merges.
    
    Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
    yantosca committed Jul 16, 2025
    Configuration menu
    Copy the full SHA
    b7212f1 View commit details
    Browse the repository at this point in the history
Loading
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载