MIT-PSFC / disruption-py

An open-source physics-based Scientific Framework for Disruption Analysis of Fusion Plasmas for AI/ML applications
https://mit-psfc.github.io/disruption-py/
MIT License
11 stars 0 forks source link

Remove populate param #307

Closed amdecker closed 2 months ago

amdecker commented 2 months ago

Problem

The parameter populate=True has been hardcoded in decorator.py after the populate argument was removed in PR 189

Proposed Solution

Remove all the logic associated with populate. Otherwise, if we want to supporting decorated methods that that should not be populated, lmk and we should add that option back in.