MantisBT - ParaView |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0013575 | ParaView | (No Category) | public | 2012-10-26 14:31 | 2014-08-29 10:59 |
|
| Reporter | Alan Scott | |
| Assigned To | Cory Quammen | |
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | |
| Platform | | OS | | OS Version | |
| Product Version | git-master | |
| Target Version | | Fixed in Version | 4.2 | |
| Project | Sandia |
| Topic Name | 13575_PV_PLUGIN_DEBUG |
| Type | feature |
|
| Summary | 0013575: Off screen rendering |
| Description | I would like to add a site specific way to create a default off screen rendering setting - either do so, or don't do so. If this site specific off screen rendering setting isn't set, default to current setting.
Lets target release AFTER 4.0
|
| Steps To Reproduce | |
| Additional Information | |
| Tags | Settings 2.0 |
| Relationships | |
| Attached Files | |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2012-10-26 14:31 | Alan Scott | New Issue | |
| 2014-02-24 10:45 | Utkarsh Ayachit | Assigned To | => Cory Quammen |
| 2014-02-24 10:46 | Utkarsh Ayachit | Tag Attached: Settings 2.0 | |
| 2014-03-17 11:53 | Cory Quammen | Status | backlog => active development |
| 2014-06-17 15:37 | Cory Quammen | Topic Name | => 13575_default_for_offscreen_rendering |
| 2014-06-17 15:37 | Cory Quammen | Note Added: 0032886 | |
| 2014-06-17 15:37 | Cory Quammen | Status | active development => gatekeeper review |
| 2014-06-17 15:37 | Cory Quammen | Fixed in Version | => git-next |
| 2014-06-17 15:37 | Cory Quammen | Resolution | open => fixed |
| 2014-06-17 15:42 | Cory Quammen | Note Added: 0032887 | |
| 2014-06-24 17:50 | Sebastien Jourdain | Status | gatekeeper review => customer review |
| 2014-06-24 17:50 | Sebastien Jourdain | Note Added: 0032914 | |
| 2014-06-26 18:41 | Alan Scott | Note Added: 0032922 | |
| 2014-07-05 21:59 | Cory Quammen | Topic Name | 13575_default_for_offscreen_rendering => 13575_site_settings_search_paths |
| 2014-07-05 21:59 | Cory Quammen | Note Added: 0032970 | |
| 2014-07-05 21:59 | Cory Quammen | Status | customer review => gatekeeper review |
| 2014-07-07 11:52 | Cory Quammen | Note Added: 0032973 | |
| 2014-07-11 09:54 | Utkarsh Ayachit | Fixed in Version | git-next => git-master |
| 2014-07-11 09:54 | Utkarsh Ayachit | Status | gatekeeper review => backlog |
| 2014-07-11 09:54 | Utkarsh Ayachit | Note Added: 0033012 | |
| 2014-07-16 16:50 | Utkarsh Ayachit | Status | backlog => gatekeeper review |
| 2014-07-16 16:50 | Utkarsh Ayachit | Status | gatekeeper review => customer review |
| 2014-07-21 15:53 | Utkarsh Ayachit | Note Added: 0033048 | |
| 2014-07-21 15:53 | Utkarsh Ayachit | Status | customer review => todo |
| 2014-07-23 23:57 | Cory Quammen | Topic Name | 13575_site_settings_search_paths => 13575_PV_PLUGIN_DEBUG |
| 2014-07-23 23:57 | Cory Quammen | Note Added: 0033073 | |
| 2014-07-23 23:57 | Cory Quammen | Status | todo => gatekeeper review |
| 2014-07-23 23:57 | Cory Quammen | Fixed in Version | git-master => git-next |
| 2014-07-24 00:16 | Cory Quammen | Note Added: 0033074 | |
| 2014-07-24 00:17 | Cory Quammen | Note Added: 0033075 | |
| 2014-07-25 09:34 | Utkarsh Ayachit | Fixed in Version | git-next => git-master |
| 2014-07-25 09:35 | Utkarsh Ayachit | Status | gatekeeper review => backlog |
| 2014-07-25 09:35 | Utkarsh Ayachit | Note Added: 0033088 | |
| 2014-07-25 09:38 | Utkarsh Ayachit | Status | backlog => customer review |
| 2014-07-25 21:30 | Alan Scott | Note Added: 0033100 | |
| 2014-07-25 21:30 | Alan Scott | Status | customer review => closed |
| 2014-08-29 10:59 | Utkarsh Ayachit | Fixed in Version | git-master => 4.2 |
|
Notes |
|
|
(0032886)
|
|
Cory Quammen
|
|
2014-06-17 15:37
|
|
commit 44f0e02f3dd4fded5412577339b94db45e228f17
Author: Cory Quammen <cory.quammen@kitware.com>
Date: Tue Jun 17 15:33:07 2014 -0400
Prepended application name to SiteSettings.json settings file
This matches what is done with the UserSettings.json settings file.
Change-Id: Ic4b7be7d9ef7b7570455df2003809574edb7458e |
|
|
|
(0032887)
|
|
Cory Quammen
|
|
2014-06-17 15:42
|
|
To test:
- Create a text file named ParaView-Settings.json in the same directory as the ParaView executable
- Set the contents of this file to
{
"settings" : {
"RenderViewSettings" : {
"UseOffscreenRenderingForScreenshots" : 1
}
}
}
- Launch ParaView
- Choose Edit menu -> Settings
- Enable advanced view (gear icon)
- Search for "off" and click on Render View tab
- The settings "Set whether offscreen rendering should be used when saving screenshots of animations" should be checked.
- Quit ParaView
- Now modify the file above to change the setting value for UseOffscreenRenderingForScreenshots to 0
- Launch ParaView and look at the settings again. The setting in question should be unchecked. |
|
|
|
(0032914)
|
|
Sebastien Jourdain
|
|
2014-06-24 17:50
|
|
SUMMARY
---------------------------------------------
Topics merged into master:
13575_default_for_offscreen_rendering
14705_hide_time_marker
14806_fix_protobuf
add_pythonalgorithm_dependency
fix-msvc-iterator-warning
fix_install_rules_visitbridge
fix_lic_on_structured_grids
fixing_markers_crashes
(VTK) pvweb-fix-lookup-table-usage
(VTK) pvweb-replace-missing-server-import
pvweb-update-install-guides
remove-dead-smsettings-code
safer-catalyze
pvweb-update-WAMPv2
---------------------------------------------
Topics reverted from next:
14788-pqTextEdit
14790_save_restore_settings_buttons
pvweb-update-to-WAMP-v2
stage-push-data |
|
|
|
(0032922)
|
|
Alan Scott
|
|
2014-06-26 18:41
|
|
|
Couldn't replicate fix. Requested feedback from Utkarsh and Cory in a personal e-mail. Also asked about docs. |
|
|
|
(0032970)
|
|
Cory Quammen
|
|
2014-07-05 21:59
|
|
commit f2cde10d959a183d1c982942b1a4b10592065313
Author: Cory Quammen <cory.quammen@kitware.com>
Date: Fri Jun 27 12:10:00 2014 -0400
Fixed search path for site settings file in linux installs
ParaView will now look under <install_directory>/share/paraview-4.1
for site-wide custom settings.
Change-Id: I5a7226551090ecc15dd468995276d1da6b7d723c |
|
|
|
(0032973)
|
|
Cory Quammen
|
|
2014-07-07 11:52
|
|
Added another commit to fix dashboard failures.
commit d26fafe028fbb7cfe8c63cfa5a01a26e54862eaf
Author: Cory Quammen <cory.quammen@kitware.com>
Date: Mon Jul 7 11:44:29 2014 -0400
Added check for path length
Added check for sufficient path string length prior
to searching for "/bin" at the end of the path. This
solves exceptions thrown by a number of tests.
Change-Id: Ibefe7356a9f5b0353822258155bba706f1e26f2e |
|
|
|
(0033012)
|
|
Utkarsh Ayachit
|
|
2014-07-11 09:54
|
|
SUMMARY
---------------------------------------------
Topics merged into master:
0014846-update-benchmarks
13575_site_settings_search_paths
14830_initial_opacity_function_display
api_cleanup
avi-compression
dashboard-warnings
tree_widget_height
volume-rendering-catalyst
(VTK) vtkweb-arg-for-custom-ws-endpoint
|
|
|
|
(0033048)
|
|
Utkarsh Ayachit
|
|
2014-07-21 15:53
|
|
I tried a clean install with git/master and then ensured that I removed by $HOME/.config/ParaView.
Then I put a file ParaView-Settings.json under <install>/lib/paraview-4.1 as suggested. I don't see the setting respected.
BTW, you should add some environment variable that users can set to print which json files are being read like we do for PV_PLUGIN_DEBUG. |
|
|
|
(0033073)
|
|
Cory Quammen
|
|
2014-07-23 23:57
|
|
commit 8d4ed01b742c54ab7b16de56af40fb83edd0f8a9
Author: Cory Quammen <cory.quammen@kitware.com>
Date: Wed Jul 23 16:27:28 2014 -0400
If no user settings file is found, create a user settings collection
If this isn't done, then the site settings collection, if it exists,
will be treated incorrectly as the user settings collection, setting
changes will be saved there, and that collection will be written to
the user settings file.
Change-Id: I3581efc6ecc2bb2ae43f41854c7f8580776632e9 |
|
|
|
(0033074)
|
|
Cory Quammen
|
|
2014-07-24 00:16
|
|
There was a subtle error in the loading logic that would have led to no site settings file being picked up from <install>/lib/paraview-4.1 as Utkarsh and Alan observed. That has been fixed in the commit cited above, and the search for the site settings file now looks in <install>/share/paraview-4.1 instead of the lib directory.
New testing steps:
- Install ParaView in some directory $INSTALL
- Delete $HOME/.config/ParaView directory
- Launch paraview (local server)
- Edit -> Settings...
- Click on gear icon to enable advanced settings
- Click on Render View tab
- Enter search text "Off". See that offscreen rendering is not enabled
- Quit ParaView
- Create a file $INSTALL/share/paraview4.1/ParaView-SiteSettings.json with the content
{
"settings" : {
"RenderViewSettings" : {
"UseOffscreenRenderingForScreenshots" : 1
}
}
}
- Launch paraview
- Edit -> Settings...
- Click on Render View tab
- Enter search text "Off". See that offscreen rendering is now enabled
- Remove search text
- Click on General tab
- Enable "Automatically convert data arrays as needed..."
- Click OK in settings dialog box
- Quit ParaView
- View file $HOME/.config/ParaView/ParaView-UserSettings.json
- It should have the contents
{
"settings" : {
"GeneralSettings" : {
"AutoConvertProperties" : 1
},
"RenderViewSettings" : {
"UseOffscreenRenderingForScreenshots" : 1
}
}
} |
|
|
|
(0033075)
|
|
Cory Quammen
|
|
2014-07-24 00:17
|
|
|
Note that you can also define the environment variable PV_SETTINGS_DEBUG to view the directories where the settings are being searched. |
|
|
|
(0033088)
|
|
Utkarsh Ayachit
|
|
2014-07-25 09:35
|
|
SUMMARY
---------------------------------------------
Topics merged into master:
13575_PV_PLUGIN_DEBUG
14771_fix_global_annotation_filter
14885_python_view_crash
fix_apply_pqProxyWidgetDialog
fix_get_proxies
fix-int64_t
(VTK) remove_duplicate_tiff_readers
remove_matplotlib_utilities
|
|
|
|
(0033100)
|
|
Alan Scott
|
|
2014-07-25 21:30
|
|
Very, very nice.
Tested local server, Linux, master. |
|