ansys / ansys/pyadditive

Refactor simulation inputs to use pydantic

Open
#291 3 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
3
Forks
1
Avg merge
1d 14h
Merged PRs (30d)
14

Description

### 📝 Description of the feature

AdditiveMaterial includes a lot of code which is built in to the pydantic library. Dunder methods `__repr__` and `__eq__` should be removed.

### 💡 Steps for implementing the feature

_No response_

### 🔗 Useful links and references

https://docs.pydantic.dev/latest/
https://youtu.be/502XOB0u8OY?si=YGVcgp72C0NpW0iZ

Contributor guide

Open the contributing guide

Research direction

Locate the AdditiveMaterial class and compare its behavior with the pydantic documentation linked in the issue. Confirm which behavior pydantic already provides, remove the redundant __repr__ and __eq__ methods, and verify that the existing simulation-input behavior remains unchanged.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend-api-design
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.