Skip to content

refactor: Deprecate FieldInfo. #4287

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.

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

Merged
merged 13 commits into from
Jul 19, 2025
Merged

refactor: Deprecate FieldInfo. #4287

merged 13 commits into from
Jul 19, 2025

Conversation

prmukherj
Copy link
Collaborator

@prmukherj prmukherj commented Jul 16, 2025

Deprecate Field Info.

  1. Delete from documentation.
  2. Update other documentation like FileSession.
  3. Updated tests as well

Please refer the updated docs and tests for reference.

# Earlier:
>>> field_info.get_scalar_fields_info
>>> field_info.get_scalar_field_range
>>> field_info.get_vector_fields_info
>>> field_info.get_surfaces_info

# Now:
>>> field_data.scalars()
>>> field_data.vectors()
>>> field_data.surfaces()
>>> field_data.surface_ids()

@github-actions github-actions bot added documentation Documentation related (improving, adding, etc) enhancement Improve any current implemented feature labels Jul 16, 2025
@prmukherj
Copy link
Collaborator Author

In the vector and scalar fields is it better to add a validator along with is_active check since we don't have field_info exposed to users anymore? The validator can suggest close matching names as well.

Base automatically changed from maint/update_field_data_allowed_values_interface to main July 17, 2025 02:50
@prmukherj prmukherj marked this pull request as ready for review July 17, 2025 06:06
@prmukherj prmukherj enabled auto-merge (squash) July 18, 2025 05:03
@hpohekar
Copy link
Collaborator

@prmukherj We need to update doc\deprecated_pyfluent_apis.py.

@prmukherj prmukherj merged commit f8ff1da into main Jul 19, 2025
89 of 109 checks passed
@prmukherj prmukherj deleted the maint/deprecate_field_info branch July 19, 2025 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation related (improving, adding, etc) enhancement Improve any current implemented feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants