View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0013575ParaView(No Category)public2012-10-26 14:312014-08-29 10:59
ReporterAlan Scott 
Assigned ToCory Quammen 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Versiongit-master 
Target VersionFixed in Version4.2 
Summary0013575: Off screen rendering
DescriptionI 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
TagsSettings 2.0
ProjectSandia
Topic Name13575_PV_PLUGIN_DEBUG
Typefeature
Attached Files

 Relationships

  Notes
(0032886)
Cory Quammen (developer)
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 (developer)
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 (manager)
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 (manager)
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 (developer)
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 (developer)
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 (administrator)
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 (administrator)
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 (developer)
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 (developer)
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 (developer)
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 (administrator)
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 (manager)
2014-07-25 21:30

 Very, very nice.

Tested local server, Linux, master.

 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

Copyright © 2000 - 2018 MantisBT Team