search

ibpsa / project1-boptest-gym

Other
43 stars 20 forks source link

Upgrade requests format due to werkzeug #131

Closed Learner0x5a closed 1 year ago

Learner0x5a commented 1 year ago

According to this question, one needs to add the correct location of the input data in requests.

e.g. change requests.put(url, data={'step': self.step_period}) to requests.put(url, json={'step': self.step_period})

Otherwise, flask would raise error: "The browser (or proxy) sent a request that this server could not understand"

dhblum commented 1 year ago

Thanks @Learner0x5a. This is an update needed for support of using the recently released BOPTEST v0.5.0. To confirm, you are using v0.5.0?

Learner0x5a commented 1 year ago

Thanks @Learner0x5a. This is an update needed for support of using the recently released BOPTEST v0.5.0. To confirm, you are using v0.5.0?

Yes. I'm using v0.5.0.

javiarrobas commented 1 year ago

@Learner0x5a thank you very much for opening the PR and sorry it took me so long to answer. Since this relates to the update to BOPTEST v0.5.0 that involves some further edits I'm opening a new issue for that here. I'll let you know once ready.

javiarrobas commented 1 year ago

I'm closing this pull request since addressed in https://github.com/ibpsa/project1-boptest-gym/pull/136.