Skip to content

Issue #359 tofu-custom allow customization of script parameters#360

Merged
lasofivec merged 1 commit intodevelfrom
Issue359_ScriptDefaultParameters
Mar 3, 2020
Merged

Issue #359 tofu-custom allow customization of script parameters#360
lasofivec merged 1 commit intodevelfrom
Issue359_ScriptDefaultParameters

Conversation

@Didou09
Copy link
Copy Markdown
Member

@Didou09 Didou09 commented Mar 3, 2020

Main changes:

  • User-customizable default parameters of tofuplot and tofucalc have been moved to a dedicated module tofu/scripts/_def.py
  • tofu-custom now copies this module into the local .tofu/ directory so the user can edit these parameters

Issues:

Fixes, in devel, issue #359

…ofu-custom now copies them into _scripts_def.py
@Didou09 Didou09 requested a review from lasofivec March 3, 2020 12:55
@Didou09 Didou09 self-assigned this Mar 3, 2020
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #360 into devel will decrease coverage by 0.06%.
The diff coverage is 1.72%.

Impacted file tree graph

@@            Coverage Diff            @@
##           devel     #360      +/-   ##
=========================================
- Coverage   39.3%   39.24%   -0.07%     
=========================================
  Files         84       85       +1     
  Lines      24587    24625      +38     
=========================================
  Hits        9663     9663              
- Misses     14924    14962      +38
Impacted Files Coverage Δ
tofu/scripts/tofucustom.py 0% <0%> (ø) ⬆️
tofu/scripts/_def.py 0% <0%> (ø)
tofu/scripts/tofuplot.py 0% <0%> (ø) ⬆️
tofu/scripts/tofucalc.py 0% <0%> (ø) ⬆️
tofu/version.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4784c5b...5a8d273. Read the comment docs.

@lasofivec lasofivec merged commit 16f6c0b into devel Mar 3, 2020
@lasofivec lasofivec deleted the Issue359_ScriptDefaultParameters branch March 3, 2020 15:23
@Didou09 Didou09 mentioned this pull request Mar 10, 2020
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