Skip to content

Issue #359 Minor bug fix#361

Merged
lasofivec merged 4 commits intodevelfrom
Issue359_ScriptDefaultParameters
Mar 10, 2020
Merged

Issue #359 Minor bug fix#361
lasofivec merged 4 commits intodevelfrom
Issue359_ScriptDefaultParameters

Conversation

@Didou09
Copy link
Copy Markdown
Member

@Didou09 Didou09 commented Mar 3, 2020

Realized :

os.chdir(...)
import

was not robust enough, the system can chdir in a separate terminal and the import ay not see the proper module

Replaced by a more robust
1/ temporary modification of the sys.path
2/ use of importlib library

Option 2 seems safer after some reading.

Also @lasofivec , I pushed changes to ne-doc for the bash commands, but could not build the doc because I can't install a Sphinx dependency here, could you build and push to gh-pages ?

Also: added draft for release notes for 1.4.3

@Didou09 Didou09 requested a review from lasofivec March 3, 2020 17:17
@Didou09 Didou09 self-assigned this Mar 3, 2020
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #361 into devel will not change coverage.
The diff coverage is 7.69%.

Impacted file tree graph

@@           Coverage Diff           @@
##            devel     #361   +/-   ##
=======================================
  Coverage   39.24%   39.24%           
=======================================
  Files          85       85           
  Lines       24625    24625           
=======================================
  Hits         9663     9663           
  Misses      14962    14962
Impacted Files Coverage Δ
tofu/scripts/tofuplot.py 0% <0%> (ø) ⬆️
tofu/imas2tofu/_core.py 0.87% <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 5a8d273...44a9874. Read the comment docs.

@lasofivec lasofivec merged commit 3c221cd into devel Mar 10, 2020
@lasofivec lasofivec deleted the Issue359_ScriptDefaultParameters branch March 10, 2020 16:07
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