spine-tools / spine-items

Project items for Spine Toolbox.
https://www.tools-for-energy-system-modelling.org/
GNU Lesser General Public License v3.0
4 stars 4 forks source link

Fix Exporter preview #147

Closed PiispaH closed 1 year ago

PiispaH commented 1 year ago

Exporter's preview didn't show anything first when turned on. Only after a change was made in the mapping, it updated. The table names for the mappings in the preview were also missing. At least partially caused by a bug in pyside: https://bugreports.qt.io/browse/PYSIDE-2406

Fixes spine-tools/Spine-Toolbox#2233

Checklist before merging

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 66.66% and no project coverage change.

Comparison is base (1444c7d) 55.10% compared to head (2cf418e) 55.10%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #147 +/- ## ======================================= Coverage 55.10% 55.10% ======================================= Files 192 192 Lines 17379 17379 Branches 2852 2852 ======================================= Hits 9577 9577 Misses 7307 7307 Partials 495 495 ``` | [Files Changed](https://app.codecov.io/gh/spine-tools/spine-items/pull/147?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=spine-tools) | Coverage Δ | | |---|---|---| | [spine\_items/exporter/widgets/preview\_updater.py](https://app.codecov.io/gh/spine-tools/spine-items/pull/147?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=spine-tools#diff-c3BpbmVfaXRlbXMvZXhwb3J0ZXIvd2lkZ2V0cy9wcmV2aWV3X3VwZGF0ZXIucHk=) | `40.93% <66.66%> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.