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.

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

Allow writing console output to separate files while also using checkpoint output from another file #29806

Closed
wants to merge 1 commit into from

Conversation

GiudGiud
Copy link
Contributor

@GiudGiud GiudGiud commented Feb 4, 2025

refs #29805

I m not sure why we dont allow it?
@loganharbour

using checkpoint output from another file
refs idaholab#29805
@GiudGiud GiudGiud self-assigned this Feb 4, 2025
@moosebuild
Copy link
Contributor

Job Documentation, step Docs: sync website on e0ef113 wanted to post the following:

View the site here

This comment will be updated on new commits.

@moosebuild
Copy link
Contributor

Job Coverage, step Generate coverage on e0ef113 wanted to post the following:

Framework coverage

Coverage did not change

Modules coverage

Inconsistent report tags were found between the head and base reports.
This can happen when reports are missing from either the head or the base.

Inconsistent tags:
module-chemical_reactions
module-chemical_reactions-parallel
module-chemical_reactions-recover
module-combined
module-contact
module-contact-parallel
module-electromagnetics
module-electromagnetics-parallel
module-electromagnetics-recover
module-external_petsc_solver
module-external_petsc_solver-parallel
module-external_petsc_solver-recover
module-fluid_properties
module-fluid_properties-parallel
module-fluid_properties-recover
module-fsi
module-fsi-parallel
module-fsi-recover
module-functional_expansion_tools
module-functional_expansion_tools-parallel
module-functional_expansion_tools-recover
module-geochemistry
module-geochemistry-parallel
module-heat_transfer
module-heat_transfer-parallel
module-heat_transfer-recover
module-level_set
module-level_set-parallel
module-misc
module-misc-parallel
module-misc-recover
module-navier_stokes
module-navier_stokes-parallel
module-navier_stokes-recover
module-optimization
module-optimization-parallel
module-optimization-recover
module-peridynamics
module-peridynamics-parallel
module-phase_field
module-phase_field-parallel
module-porous_flow
module-porous_flow-parallel
module-ray_tracing
module-ray_tracing-parallel
module-ray_tracing-recover
module-rdg
module-rdg-parallel
module-reactor
module-reactor-parallel
module-richards
module-scalar_transport
module-scalar_transport-parallel
module-scalar_transport-recover
module-solid_mechanics
module-solid_mechanics-parallel
module-solid_properties
module-solid_properties-parallel
module-solid_properties-recover
module-stochastic_tools
module-stochastic_tools-parallel
module-thermal_hydraulics
module-thermal_hydraulics-parallel
module-thermal_hydraulics-recover
module-xfem
module-xfem-parallel

Full coverage reports

Reports

This comment will be updated on new commits.

@GiudGiud GiudGiud marked this pull request as ready for review February 4, 2025 16:42
@GiudGiud
Copy link
Contributor Author

@loganharbour

@loganharbour
Copy link
Member

@milljm we need to talk about this tomorrow

@GiudGiud
Copy link
Contributor Author

please do I need this or another solution for the VTB

@loganharbour
Copy link
Member

We can do this if we enable working directory to not be out of the directory. Is that something you're willing to support for the VTB?

@GiudGiud
Copy link
Contributor Author

Yup that's how I am setting up the tests here
idaholab/virtual_test_bed#518
and how I plan to do this moving forward

@GiudGiud
Copy link
Contributor Author

GiudGiud commented Feb 17, 2025

any chance this could get in? It is needed for VTB HPC testing

@GiudGiud
Copy link
Contributor Author

What exactly are we waiting for here?

@loganharbour
Copy link
Member

What exactly are we waiting for here?

I thought of a quicker solution. I'll enable --sep-files testing with working_directory iff the working directory doesn't resolve out of directory

@loganharbour
Copy link
Member

#29928 should resolve the immediate issue

@GiudGiud GiudGiud closed this Feb 23, 2025
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.

3 participants