alteryx / alteryx/promote-python

Username bug when calling predict

Offen
#19 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
15
Forks
9
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Currently in the `predict` method the logic is as follows:

```
prediction_url = urllib.parse.urljoin(self.url, os.path.join(
self.username, 'models', modelName, 'predict'))
username = username if username else self.username
response = requests.post(
url=prediction_url,
...
```
The way this logic works is that it builds the prediction url with `self.username` always and never considers if you give the `predict` method a `username` along with the `model` and `data`. It should be switched to look like this:

```
username = username if username else self.username
prediction_url = urllib.parse.urljoin(self.url, os.path.join(
username, 'models', modelName, 'predict'))
```
Unfortunately the repo is read-only so I can not push the change to a remote branch and open a PR. In the short-term I have modified my source code to work as intended.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.