Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Subplots are removed when selecting other items in the canvas #264

Open
wants to merge 55 commits into
base: fix_deleting_plot
Choose a base branch
from

Conversation

CvTooren
Copy link

I can create subplots, e.g. scatterplots based on different variables in one QGIS layer. However, as soon as I select another point or area of that layer the different subplots are removed and only the latest plot based on the latest settings is shown. How can I solve this, or is it a bug?

ghtmtt and others added 9 commits March 19, 2021 11:38
* allow to choose the gridaxis color of the plot

* fix test?

* Update DataPlotly/core/plot_types/plot_type.py

Co-authored-by: Simon Gröchenig <simon.groechenig@salzburgresearch.at>

* Apply suggestions from code review

Co-authored-by: Simon Gröchenig <simon.groechenig@salzburgresearch.at>
Co-authored-by: Nyall Dawson <nyall.dawson@gmail.com>

* use hex color strings and lowerize all of them. fix tests?

Co-authored-by: Simon Gröchenig <simon.groechenig@salzburgresearch.at>
Co-authored-by: Nyall Dawson <nyall.dawson@gmail.com>
To avoid empty space above layout option group
move layout properties to layout options group in widget
@ghtmtt
Copy link
Owner

ghtmtt commented Sep 17, 2021

@CvTooren I think you hit a bug. Can you please open an issue rather a pull request? Thanks!

@CvTooren
Copy link
Author

CvTooren commented Sep 22, 2021 via email

Gustry and others added 19 commits September 28, 2021 15:02
* Expose Dataplotly on QGIS Server for a GetPrint request

* Fix tests about lint
Float now requires explicit cast to int
* Add options to change title, xticks and yticks font sizes.

Insert options in graph parameters to change settings for all three parameters.

Could be refined to also change x_label and y_label font sizes (if such parameters
exist)

* Add options for x/y labels and put all box in a collapsible group

* Update UI parameters (title, collapsed state, default values)

* Update default font size (ui and python).

* Use QgsFontButton instead of size input only.

Still needs some work to keep in mind the parameters between two sessions.

* Separate parameters into family, size and color

* Test if font related settings are correctly stored and restored.

* Fix pylint issues

Co-authored-by: Matthieu Denoux <matthieu.denoux@vivantes.fr>
Co-authored-by: julien.monticolo <monticolo@unistra.fr>
* Fix "Build a generic plot" processing algorithm

* Update DataPlotly/processing/dataplotly_algorithms.py
…288)

* new custom function to get the colors from categories used

* update the expression to be more generic
ghtmtt and others added 27 commits August 21, 2022 16:00
* fixing tests [1]

* fixing tests

* fixing comment

* fix datetime test
* using plotly.express provider and scatterplot algorithm

* json output optional

* fix lint

* remove useless utils file

* fix lint again

* add faceting
* add icon and remove some useless methods and strings

* add Processing test for scatterplot

* update changelog

* update copy info (thanks Etienne :) )

* lint

* add pandas to the requirement testing packages

* fix test with self.maxDiff

* use json instead of html for test, too unpredictable

* remove useless function

* try another test approach
* add bar width with custom data and standard value

* update the attribute table of the test layer

* restore label and marker size when the scatter plot is chosen
* version control ignores vscode settings

* first implementation

* add dock_id argument to loadDocFromDic

* disconnect signals when unloading

* try to create plot immediatly after dock init

* docks state and geometry can be restored from project

* remove extra comma

* feat : add new dock

* dock_title can be restored too

* dock title with space can be restored now

* resize add_dock_dlg

* feat : implementation remove dock

* fix: main dataplotly panel read_from_project

* Update .gitignore

Co-authored-by: Matteo Ghetta <matteo.ghetta@gmail.com>

* Update DataPlotly/data_plotly.py

Co-authored-by: Matteo Ghetta <matteo.ghetta@gmail.com>

* Update DataPlotly/gui/remove_dock_dlg.py

Co-authored-by: Matteo Ghetta <matteo.ghetta@gmail.com>

* Update DataPlotly/data_plotly.py

Co-authored-by: Matteo Ghetta <matteo.ghetta@gmail.com>

* fix : remove the return value dlg

* pylint : start

 remove unnecessary import and define variable in init

* move utils functoin in core_utils

* move restore in core_utils

* style : pylint

* format: pylint

* format: pylint

* format: pylint

* format : pylint

* format: pylint

* test: add removeDockWidget to QgisInterface

* test : add override_iface to DataPlotlyDock const

* test : add first tests to dock_manager

* test : add other tests

* style: pylint use-dict-literal

* style: pylint remove superflous parens

* test: add test on utils xml function

* ignore .qgs~

* test : add test on DataPlotlyNewDockIdValidator

* Update .gitignore

* Update DataPlotly/test/test_dock_manager.py

* fix pycodestyle

---------

Co-authored-by: Jean-Daniel LOMENEDE <jean-daniel.lomenede@agriculture-equipement.gouv.fr>
Co-authored-by: Matteo Ghetta <matteo.ghetta@gmail.com>
* feat : call create_plot when project is readed

* feat: clear  plot view when project is cleared

* style: pylint
* feat: add hole option to pie chart

* style: pylint

* fix: limit pie_hole to 0.95
* set the initial value to Auto, clearValue to Auto and tooltip

* tweak the bar width value to pass to plotly

* add changelog info

* bump new release and minimum QGIS version

* no more 3.22 in docker
* Python cleanup, raise to 3.7 minimum due to QGIS 3.28

* Do not use fstring when the string is translated

* Update DataPlotly/gui/plot_settings_widget.py
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.

None yet

8 participants