sassoftware / sassoftware/python-sasctl
Older examples needs to be updated for JSONFiles API changes
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 52
- Forks
- 45
- Ø Merge
- 23 Std. 38 Min.
- Gemergte PRs (30 T.)
- 2
Beschreibung
Describe the issue
This example has some API calls that don't work with the updated version of sasctl - changes to JSONFiles:
https://github.com/sassoftware/python-sasctl/blob/master/examples/pzmmCompleteModelExampleMLFlowSklearn.ipynb
For example, writeModelPropertiesJSON is now write_model_properties_json and has many changes in the parameters.
Other examples:
writeFileMetadataJSON
writeVarJSON
I checked other examples and it seems any of them 6 months or older are now out of date with the newer sasctl API.
To Reproduce
Run this command using latest sasctl
J.writeVarJSON(inputsDict, isInput=True, jPath=zipFolder)
and you will get things like
AttributeError: 'JSONFiles' object has no attribute 'writeVarJSON'. Did you mean: 'write_var_json'?
Version
The SHA is 729e72c8785065109ccfd1ffdb9816d4bf1dce55
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit examples/pzmmCompleteModelExampleMLFlowSklearn.ipynb und vergleiche dessen JSONFiles-Aufrufe mit der aktuellen sasctl API. Durchsuche die anderen Beispiele nach writeModelPropertiesJSON, writeFileMetadataJSON und writeVarJSON und aktualisiere anschließend veraltete Aufrufe und Parameter. Erledigt ist die Aufgabe, wenn ältere Beispiele mit der neuesten sasctl-Version ohne den gemeldeten AttributeError ausgeführt werden können.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- documentation
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100