Kitware / paraview-visualizer

Web frontend to ParaView based on trame
BSD 3-Clause "New" or "Revised" License
30 stars 12 forks source link

Changed _update_path() parameters and parsing dict inside the function #9

Closed anshakeel closed 1 year ago

anshakeel commented 1 year ago

HI, There was a Bug in this Trame based ParaView-Visualizer that it was not able to read any file from data directory. It was throwing the following error when we click to load any file from Data directory:

9feabe7af9d039a8b510e6f0771ffcd5f109fd71 b293a485668b97ae27f2fb62c4117d42ab3a0fbd

According to Error, it seems that _update_path() method was triggered with incomplete arguments because of this the type was considered a unexpected keyword argument. I simple changed the parameters of _update_path() method and passed only $event as arguments from where this method was triggered and it received a dict . In method body, parsed dict and assigned values to the type and value.

Now it is working, it tested with the following files:

Regards, Ans

jourdain commented 1 year ago

Hum this is strange as the code before should have worked... Otherwise that seems reasonable.