Skip to content

add PatchView to application list#2559

Merged
j9ac9k merged 2 commits intopyqtgraph:masterfrom
ZeitgeberH:master
Dec 23, 2022
Merged

add PatchView to application list#2559
j9ac9k merged 2 commits intopyqtgraph:masterfrom
ZeitgeberH:master

Conversation

@ZeitgeberH
Copy link
Copy Markdown
Contributor

Edited README.md to add PatchView in the application list.

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Dec 23, 2022

Hi @ZeitgeberH

Just took a look at your documentation, those plots are 🔥

Would you mind positioning PatchView to be in alphabetical order with the other items?

Thanks for the PR, hopefully the library has been working out well for you!

@ZeitgeberH
Copy link
Copy Markdown
Contributor Author

Hi @j9ac9k
I repositioned PatchView in the list per your request.
Pyqtgraph works pretty well for my project. I also learned quite a bit from the code base itself.
Thanks very much!

@ZeitgeberH ZeitgeberH closed this Dec 23, 2022
@ZeitgeberH ZeitgeberH reopened this Dec 23, 2022
@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Dec 23, 2022

Hi @j9ac9k I repositioned PatchView in the list per your request. Pyqtgraph works pretty well for my project. I also learned quite a bit from the code base itself. Thanks very much!

Not going to lie, when I started using pyqtgraph I also fell back on reading the codebase a lot. I can't help but feel anytime someone does that, it's a failure of documentation.

If you're on discord, we (pyqtgraph) has a channel on the main python server, some of us maintainers and regular contributors are there so if you want to have a convo about new features or things we can do to make situation better for application developers, that's probably the best place to find us (outside of the issue tracker that is).

@j9ac9k j9ac9k merged commit 26ce697 into pyqtgraph:master Dec 23, 2022
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