Skip to content

Conversation

germa89
Copy link
Collaborator

@germa89 germa89 commented Apr 29, 2022

this was failing:

mapdl.input('file.txt') # file.txt in os.getcwd()

Because we were passing to the /INPUT only the fname, not the full route. When the python working directory and MAPDL one are different, this will raise an error.

Now it should work.

@germa89 germa89 added the BUG label Apr 29, 2022
@germa89 germa89 added this to the v0.61.4 milestone Apr 29, 2022
@germa89 germa89 self-assigned this Apr 29, 2022
@codecov
Copy link

codecov bot commented Apr 29, 2022

Codecov Report

Merging #1082 (3a24acf) into main (d123100) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1082   +/-   ##
=======================================
  Coverage   73.01%   73.02%           
=======================================
  Files          43       43           
  Lines        6381     6383    +2     
=======================================
+ Hits         4659     4661    +2     
  Misses       1722     1722           

Copy link
Collaborator

@akaszynski akaszynski left a comment

Choose a reason for hiding this comment

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

LGTM.

Milestone should be updated to v0.61.6

@akaszynski akaszynski merged commit 29e4fe4 into main Apr 29, 2022
@akaszynski akaszynski deleted the fix/input-not-working-in-local-and-without-fullpath branch April 29, 2022 21:16
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.

2 participants