Ticket #4741 (closed: wontfix)
Build against PV 3.12
| Reported by: | Owen Arnold | Owned by: | Michael Reuter |
|---|---|---|---|
| Priority: | major | Milestone: | Release 2.2 |
| Component: | VATES | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Owen Arnold |
Description
The latest version (3.12.0) should be patched with our changes. Owen is going to adapt MantidPlot to identify the PV location, so that the CPack tweak we've made is no longer needed.
Questions that should be answered with this ticket.
- Does the VSI work with the new PV libraries?
- Do the plugins work with the new PV libraries?
- Can we ditch the InstallPV.sh script if we move to this release?
- Do the windows binaries distributed with PV, require the same version of Qt that we ship as part of MantidPlot (assign to Owen if you wish?
We should also look to see what additional features we get out of version 3.12.0
Change History
comment:2 Changed 9 years ago by Owen Arnold
I've been keeping an eye on their development list, and I'm not convinced 3.14 is going to be particularly stable, given the flame wars that has already erupted :)
comment:3 Changed 9 years ago by Michael Reuter
Refs #4741. Only change to make everything compile.
Changeset: d9bf08502d9606323b7be72daba4ac3fe6c56d52
comment:5 Changed 9 years ago by Michael Reuter
Refs #4741. Putting stretch back to view.
Changeset: bb28315498ef3847563be7391a26fd776778a7e1
comment:7 Changed 9 years ago by Owen Arnold
refs #4741. Template for PV locating dialog
Changeset: e69a4b84f9d0e15aee1728662113cd9484498aaf
comment:8 Changed 9 years ago by Owen Arnold
refs #4741. First working implementation
Changeset: cfeec451febf8729caebaf41cbd48a759d4ad958
comment:9 Changed 9 years ago by Owen Arnold
refs #4741. Improve vates identification
Changeset: f5e8ac67cba00e4a041555503493bc9acfe5627d
comment:10 Changed 9 years ago by Owen Arnold
Changeset: fd76d35ce1eccb50ddf1f2641ff37a44602494c1
comment:11 Changed 9 years ago by Owen Arnold
Last 4 commits on wrong ticket. Should have gone to #4742. Sorry for the spam Michael.
comment:12 Changed 9 years ago by Michael Reuter
Refs #4741. Adding missing update pipeline call.
Changeset: 46299dd86afd7e69511967666a964183ee32170b
comment:13 Changed 9 years ago by Michael Reuter
Refs #4741. Fixing all representation call to new convention.
Changeset: 115f3253633d2835ba6849cee866ce0740c6b400
comment:14 Changed 9 years ago by Michael Reuter
Refs #4741. Adding more update calls.
Changeset: 2476581047e177e81f073ea6a894767e58a66844
comment:15 Changed 9 years ago by Michael Reuter
Refs #4741. Adding another pipeline update call.
Changeset: 1248eccb5db7e243bc5aa9b00c1a1b11ec0ca2f9
comment:16 Changed 9 years ago by Michael Reuter
Refs #4741. Fixing merge from master.
Changeset: b402f09e5ac307fd366b92135fefda726c93d95e
comment:17 Changed 9 years ago by Michael Reuter
Refs #4741. Removing inadverently added line.
Changeset: 25bf1af066dbc5048fb95c7bcd6aed7c52f27c82
comment:18 Changed 9 years ago by Michael Reuter
Refs #4741. Applying cut immediately.
Changeset: fb378faa414b8f04eae59b9b7c2e41a64861ace0
comment:20 Changed 8 years ago by Michael Reuter
- Status changed from accepted to verify
- Resolution set to wontfix
As ParaView has moved versions on the major OSes and we are awaiting work from Kitware, I've decided to abandon this ticket. I'll use the associated branch as a jumping off point for the future upgrade work.
comment:21 Changed 8 years ago by Owen Arnold
- Status changed from verify to verifying
- Tester set to Owen Arnold
comment:22 Changed 8 years ago by Owen Arnold
- Status changed from verifying to closed
Sounds sensible. I believe that the fixes will be present in the next release of PV, so no point moving to 3.12. We may want to open a new ticket to prototype against the development version of PV (which contains the fixes now). This might make the transition even easier.
As an aside point, we should also do something about recognising the version of Paraview present since some of our future users will already have paraview installed, and we want to give them sensible errors/warnings.
comment:23 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 5588
I think I'm going to expand this to also looking at future PV support. There is a new release (3.14) due at the end of the month that has a possible fix for the data/axis scaling issue.