Skip to content

Conversation

@cssherman
Copy link
Collaborator

No description provided.

@cssherman cssherman requested a review from CusiniM March 12, 2024 17:42
docs/index.rst Outdated
These will attempt to install the required packages into the python distribution indicated via the `Python3_EXECUTABLE` cmake variable (also used by pygeosx).
If any package dependencies are missing, then the install script will attempt to fetch them from the internet using pip.
After installation, these packages will be available for import within the associated python distribution, and a set of console scripts will be available within the GEOSX build bin directory.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
After installation, these packages will be available for import within the associated python distribution, and a set of console scripts will be available within the GEOSX build bin directory.
After installation, these packages will be available for import within the associated python distribution, and a set of console scripts will be available within the GEOS build bin directory.

self.compareGroups( file, base_file )

else:
self.output.write( f"\nRank {rank} Failed to load target and/or baseline files \n" )
Copy link
Collaborator

Choose a reason for hiding this comment

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

maybe it is useful to know which of the two failed.

Suggested change
self.output.write( f"\nRank {rank} Failed to load target and/or baseline files \n" )
if ( file is None ):
self.output.write( f"\nRank {rank} Failed to load target files \n" )
if ( base_file is None ):
self.output.write( f"\nRank {rank} Failed to load baseline files \n" )

Copy link
Collaborator

Choose a reason for hiding this comment

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

And I wonder if we should add the path info.

@cssherman cssherman merged commit 7dc048f into main Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants