Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR aims to correct the csvy files
csvy_full.csvy
andcsvy_full_rad.csvy
Description
Changes made to the code are:
model_isotope_time_0: 0 day
in .csvy filesMotivation and Context
When tardis model is simulated using the csvy format file examples provided in repository, it gives an error. In the tardis/io/schemas/csvy_model.yml , it's description states:
model_isotope_time_0:
type: quantity
description: Initial time for isotope decay. Set to 0 for no isotopes.
Also, various other csvy files in the repository like
tardis/tardis/model/tests/data/exponential_csvy.csvy
andtardis/tardis/model/tests/data/uniform_csvy.csvy
have these corrections.How Has This Been Tested?
I checked it by running unit tests on it with python setup.py test that gave the following output on my local machine.
Screenshots (if appropriate):
Before the changes:
After the changes:
Types of changes
Checklist: