Skip to content

update imports in the broken example demos#2087

Merged
j9ac9k merged 3 commits intopyqtgraph:masterfrom
Wubbzi:exampleImports
Nov 14, 2021
Merged

update imports in the broken example demos#2087
j9ac9k merged 3 commits intopyqtgraph:masterfrom
Wubbzi:exampleImports

Conversation

@Wubbzi
Copy link
Copy Markdown
Contributor

@Wubbzi Wubbzi commented Nov 14, 2021

No description provided.

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Nov 14, 2021

Hi @Wubbzi

Thanks for the PR! I know I suggested this fix on the discord, but after taking a closer look, I believe there is a better fix. Specifically in this line:

https://github.com/pyqtgraph/pyqtgraph/blob/master/pyqtgraph/examples/ExampleApp.py#L484

We should replace cwd=path to cwd=example_path.

Would you mind undoing the changes in this PR, and modify that subprocess call instead?

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Nov 14, 2021

Thank you @Wubbzi !

@j9ac9k j9ac9k merged commit a9bf3f8 into pyqtgraph:master Nov 14, 2021
ntjess pushed a commit to ntjess/pyqtgraph that referenced this pull request Dec 10, 2021
change path of the current working directory that is passed to the subprocess.Popen call when an edited file is run.
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