Skip to content

refactor: Move the paraview plugin and create a VTK filter for CreateConstantAttributePerRegion #122

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 66 commits into
base: main
Choose a base branch
from
Open

refactor: Move the paraview plugin and create a VTK filter for CreateConstantAttributePerRegion #122

wants to merge 66 commits into from

Conversation

RomainBaville
Copy link
Contributor

@RomainBaville RomainBaville commented Aug 6, 2025 •
edited
Loading

In the context of the repository refactoring, this will consists in three main tasks:

  • Move the Paraview plugin to geos-pv and adapte it to use the vtk filter
  • Create a VTK filter independant of the paraview plugin in geos-mesh
  • Add the possibility to choose the number and the name of component and the value per component

close #106

RomainBaville and others added 30 commits June 24, 2025 16:48
@RomainBaville
add a function to get the type of a vtk array
0ca3fe5
@RomainBaville
uptade the function createAttribute to preserve the type of the vtk a…
a905450 
…rray
@RomainBaville
update the typing in the test of the function createAttribute
ec0302f
@RomainBaville
Update createAttribute and createConstantAttributeDataSet
6c501c4
@RomainBaville
Merge branch 'main' into RomainBaville/feature/update_geos-mesh_utils
510887d
@RomainBaville
update fillPartialAttribute and fillAllPartialAttributes
490135c
@RomainBaville
Add a function to get the vtk data type of an attribute of a multiblo…
15a67fa 
…ckdataset if it exist
@RomainBaville
Formating for the CI
5b17644
@RomainBaville
Uptade functions calling utils functions
bd63003
@RomainBaville
Fix the doc issue
19ffa8d
@RomainBaville
Create the two files one for the plugin, one for the filter
27b9c83
@RomainBaville
Create the vtk filter from the paraview plugin
69dca82
@RomainBaville
Fix build issue
27b6abf
@RomainBaville
Merge branch 'RomainBaville/feature/update_geos-mesh_utils' into test
dcadd7a
@RomainBaville @paloma-martinez
Apply suggestions from code review
bcdf4bd 
Co-authored-by: paloma-martinez <[email protected]>
@RomainBaville
Generalize error message of copyAttribute
e79f5ab
@RomainBaville
Add a raise assertion error in case of the mesh doen't have the attri…
b17e2e5 
…bute
@RomainBaville
Update the default value for uint case for fillpartialattribute
5941980
@RomainBaville
Cleen and add logger to manadge output messages
f46fde5
@RomainBaville
clear the tests and functions of arrayModifiers
614cafa
@RomainBaville
Clean fillpartialattribute and its test
1423482
@RomainBaville
clean the code and add a funtion to test if an attribute is partial.
0e2ded2
@RomainBaville
fix the test of isAttributeGlobal
68d6c3c
@RomainBaville
Clean the code
57c9bd2
@RomainBaville
Clean for ci
b4ff24e
@RomainBaville
Clean for the ci
7da8f9b
@RomainBaville
Clean doc
3c8f5d6
@RomainBaville
Clean for ci
1d16852
@RomainBaville
Clean For ci
b4e2084
@RomainBaville
Clean For ci
f052c14
@RomainBaville
Manadge the input region index
9eb7454
@RomainBaville
Remove the use of VTKPythonAlgorithmBase in the filter CreateConstant…
6956e97 
…AttributePerRegion
@RomainBaville
Update log to be more clear
1f99e6d
@RomainBaville
Add a Handler to count warnings messages
1e2007f
@RomainBaville
Add the function to get the vtk data type of a mesh (single or multi …
1932910 
…block)
@RomainBaville
update test for the filter
c24c004
@RomainBaville
Clean the log and the doc
c9ae028
@RomainBaville
Clean variables names
e00c918
@RomainBaville
Clean the doc and the variables names
2632df8
@RomainBaville
Clean variables name and typing
96f2237
@RomainBaville
Remove the AsDF function
2ac03bc
@RomainBaville
Change variables iter to iterator
a021fa7
@RomainBaville
Clean for ci
f100bb8
@RomainBaville
Remove the plugin from geos_prep and update the doc.rst file
0d65425
@RomainBaville
Fix doc
2eb0ebb
@RomainBaville
Merge branch 'RomainBaville/feature/update_geos-mesh_utils' into Roma…
4fcbc27 
…inBaville/feature/VTK_filter_and_PVCreateConstantAttributePerRegion

Update to last version of geos-mesh utils
@RomainBaville RomainBaville self-assigned this Aug 6, 2025
Base automatically changed from RomainBaville/feature/update_geos-mesh_utils to main August 6, 2025 18:24
@paloma-martinez paloma-martinez changed the title feat: Move the paraview plugin and create a VTK filter for CreateConstantAttributePerRegion refactor: Move the paraview plugin and create a VTK filter for CreateConstantAttributePerRegion Aug 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paloma-martinez paloma-martinez Awaiting requested review from paloma-martinez

@alexbenedicto alexbenedicto Awaiting requested review from alexbenedicto

At least 1 approving review is required to merge this pull request.

Assignees

@RomainBaville RomainBaville

Labels
flag: ready for review type: cleanup type: feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create vtk filter from PVCreateConstantAttributePerRegion in geos-mesh move PV plugin to geos-pv
1 participant
@RomainBaville