View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0015127ParaView(No Category)public2014-11-13 21:482015-07-09 22:02
ReporterAlan Scott 
Assigned ToDan Lipsa 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version4.2 
Target Version4.4Fixed in Version4.3 
Summary0015127: t_index variable not available, Mode Shapes
DescriptionMy user did a good enough job explaining this bug, I will just use his verbiage. Here it is:

Hello,

Attached is an exodus file that I'm having trouble with in ParaView 4.2.0, 64-bit, RHEL5.

The problem is that the "t_index" variable is not available when I check the "Has Mode Shapes" option. I am trying to add an annotation that say something like "Eigenvalue is : EigenFrequency[ index ]", but there I can't seem to find what variable refers to the current "Mode Index."

If I uncheck the "Has Mode Shapes" option, then the variable t_index works and I can just do "EigenFrequency[t_index]" to get the frequency to be annotated.

Steps to reproduce:

1) open "example_beam-eig.g", load all variables.

2) Create an "PythonAnnotation" filter.

3) Select "Field Data"

4) Enter "EigenFrequency[t_index]" in the "Expression" field.

5) Apply. Annotation should show up as "0.0"

6) Increase to time step 7, it should read "1985.697"

7) Go back to top of pipeline and select "Has Mode Shapes" check box.

8) ---> ERROR: name t_index is not defined


Ok, well that doesn't work. Let's try the "AnnotateGlobalData" filter.

9) Add a "AnnotateGlobalData" filter.

9a) "Select Arrays" --> EigenFrequency

9b) "Prefix" --> Value is:

9c) Apply


It will show the entire vector -- which is not useful.

10) Uncheck "Has Mode Shapes"

The AnnotateGlobalData filter will just show a single value -- which is useful.

It seems like what is missing is a "Mode Shape Index" variable. Does this exist? If so, it doesn't seem like it's being properly referenced in the annotation filters. But at least I might be able to use a Python Annotation filter to get around this if it does exist.

Additional InformationVTK commit
==========
commit 9c7135b9432d6f2cd2f4f3e03dd881528fb60b38
Author: Dan Lipsa <dan.lipsa@kitware.com>
Date: Thu Dec 4 16:04:26 2014 -0500

    Add data_index to Exodus driver
    
    data_index[0] represents the time step (t_index) if HasModeShapes=0
    or ModeShape otherwise. This commit also removes an unused variable.
    
    Change-Id: I08716d6c904aa18eb5b6163a6d394e3c47bcb138
ParaView commit
===============
commit ccbcf3de1d18d4077fd1c0fcc5a6902333e03898
Author: Dan Lipsa <dan.lipsa@kitware.com>
Date: Thu Dec 4 16:23:54 2014 -0500

    Update VTK to: Add data_index to Exodus driver
    
    Change-Id: I030d1a2b089a7331e79f868291d9fee947203f1b

TagsNo tags attached.
ProjectSandia
Topic Nameexodus-15127
Typeincorrect functionality
Attached Files? file icon example_beam-eig.g [^] (399,788 bytes) 2014-11-13 21:50

 Relationships

  Notes
(0033918)
Utkarsh Ayachit (administrator)
2014-12-10 07:58

 ---------------------------------------------
Topics merged into master (v4.2.0-234-g5082012):
        settings-15161

(BUG 0015127 fix seems to already have been merged in git/master)

---------------------------------------------
Topics reverted from next:
  python-calc-15125
  sorting-15130
(0033941)
Alan Scott (manager)
2014-12-11 17:51

 Dan,
Talked to Utkarsh, and what you have below works correctly. Two modifications, if you don't mind.

* Please change the variable name from data_index[] to mode_shapes[]. If this seems wrong, please discuss with Utkarsh, and I will gladly go with whatever you two decide.

* Please add documentation to the Exodus library, giving an example of using this array. In other words, how does a user find out the name of this array? Once again, ping Utkarsh to flesh out this request if needed.

Thanks! Alan
(0033987)
Dan Lipsa (developer)
2014-12-19 10:28
edited on: 2014-12-19 10:28

 VTK
===
commit 084b0bb13545fbd3e5c3f937d4a69416cf8492ef
Author: Dan Lipsa <dan.lipsa@kitware.com>
Date: Mon Dec 15 11:05:29 2014 -0500

    Rename data_index to mode_shape in Exodus reader.
    
    Field variable mode_shape[0] represents the time step (t_index)
    if HasModeShapes is not set or the ModeShape otherwise.
    
    Change-Id: Id25aab00dfe04ae142a477650cbc3a17c9b11254

ParaView
========
commit f44839d3931b98d54bca2c1365f993af147ab1d7
Author: Dan Lipsa <dan.lipsa@kitware.com>
Date: Mon Dec 15 11:13:39 2014 -0500

    Move VTK: Rename data_index to mode_shape in Exodus reader
    
    Change-Id: I431bf9862dde8495b0f30bb33be2f92486806821

commit f107463f3959405efa6f5b94d43fee731a21d684
Author: Dan Lipsa <dan.lipsa@kitware.com>
Date: Mon Dec 15 11:11:59 2014 -0500

    Add documentation about mode_shape to the Exodus reader.
    
    Change-Id: Iedbb901685be021f976ef17ed28d6f8fb31cb148
(0034000)
Utkarsh Ayachit (administrator)
2014-12-23 07:47

 SUMMARY
---------------------------------------------
Topics merged into master (v4.2.0-339-g3d6f539):
(VTK) 14539-can-crash
        15073_fix_ResetCamera
        6657_add_outside_range_colors
(VTK) KW00001499_ColorByFieldData
        add_cgns_plugin_test
        camera-9503
        cfapi-null-logic
(VTK) exodus-15127
        fix-dashboard-warning
        h5part-15215
        mangle-pvbatch-test-names
(VTK) opengl2_updates
        pvweb-support-multiple-data-directories
        revert-cfapi-deprecation-fixes
        support_custom_link_path_setting
        update-catalyst-20141215
        webvisualizer-add-tooltips
(0034731)
Alan Scott (manager)
2015-07-09 21:25

 This now works local server, but fails remote server. Local server works as follows when you input the string into the PythonAnnotation filter.

ModeNumber[] (Result is: (1. 2. 3. 4. 5. 6. 7. 8. 9. 10. ) )
ModeNumber[1] (Result is: 2.0)
ModeNumber[t_index] (Result is: 1.0 (for the first timestep))

Remote server (8 ranks)
ModeNumber[] (Result is: (error))
ModeNumber[1] (Result is: (error))
ModeNumber[t_index] (Result is: (error))
(0034735)
Alan Scott (manager)
2015-07-09 22:02

 My bad. I have a bad remote server (numpy is bad).

Tested local server, Linux, master.

Looks good. Closing.

 Issue History
Date Modified Username Field Change
2014-11-13 21:48 Alan Scott New Issue
2014-11-13 21:50 Alan Scott File Added: example_beam-eig.g
2014-11-13 21:50 Alan Scott Target Version => bug_fix
2014-11-14 11:14 Utkarsh Ayachit Assigned To => Dan Lipsa
2014-11-14 22:56 Utkarsh Ayachit Target Version bug_fix => 4.3
2014-12-02 16:15 Dan Lipsa Status backlog => active development
2014-12-04 16:22 Dan Lipsa Topic Name => exodus-15127
2014-12-04 16:22 Dan Lipsa Additional Information Updated
2014-12-04 16:26 Dan Lipsa Additional Information Updated
2014-12-08 09:29 Dan Lipsa Status active development => gatekeeper review
2014-12-10 07:58 Utkarsh Ayachit Fixed in Version => git-master
2014-12-10 07:58 Utkarsh Ayachit Status gatekeeper review => customer review
2014-12-10 07:58 Utkarsh Ayachit Note Added: 0033918
2014-12-11 17:51 Alan Scott Note Added: 0033941
2014-12-11 17:51 Alan Scott Status customer review => todo
2014-12-15 11:20 Dan Lipsa Additional Information Updated
2014-12-15 11:21 Dan Lipsa Status todo => active development
2014-12-16 09:30 Dan Lipsa Status active development => gatekeeper review
2014-12-19 09:57 Dan Lipsa Status gatekeeper review => active development
2014-12-19 10:28 Dan Lipsa Note Added: 0033987
2014-12-19 10:28 Dan Lipsa Additional Information Updated
2014-12-19 10:28 Dan Lipsa Note Edited: 0033987
2014-12-20 09:15 Dan Lipsa Status active development => gatekeeper review
2014-12-23 07:47 Utkarsh Ayachit Status gatekeeper review => customer review
2014-12-23 07:47 Utkarsh Ayachit Note Added: 0034000
2015-01-02 10:43 Utkarsh Ayachit Fixed in Version git-master => 4.3
2015-07-09 21:25 Alan Scott Note Added: 0034731
2015-07-09 21:25 Alan Scott Status customer review => todo
2015-07-09 21:25 Alan Scott Target Version 4.3 => 4.4
2015-07-09 22:02 Alan Scott Note Added: 0034735
2015-07-09 22:02 Alan Scott Status todo => closed
2015-07-09 22:02 Alan Scott Resolution open => fixed

Copyright © 2000 - 2018 MantisBT Team