Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Mark Python measures as in error if using classic CLI #671

Closed
macumber opened this issue Jan 22, 2024 · 2 comments · Fixed by #733
Closed

Mark Python measures as in error if using classic CLI #671

macumber opened this issue Jan 22, 2024 · 2 comments · Fixed by #733
Labels
Enhancement Request New feature or request
Milestone
OpenStudio Applic...

Comments

@macumber
Copy link
Collaborator

macumber commented Jan 22, 2024 •
edited
Loading

Enhancement Request

Classic CLI cannot run Python measures. If Python measures are in the workflow, add error badge if using the Classic CLI along with a message saying "Cannot use Python measures with the Classic CLI".

Should also hide Python measures in library if the Measure Manager is launched with the Classic CLI.
@macumber macumber added the Enhancement Request New feature or request label Jan 22, 2024
@macumber macumber mentioned this issue Jan 22, 2024
Closed
@macumber macumber added this to the OpenStudio Application 1.8.0 milestone Jan 22, 2024
@jmarrec
Copy link
Collaborator

jmarrec commented Aug 29, 2024

Did #720 address this as well or not?

@macumber
Copy link
Collaborator Author

The classic CLI is no longer used for MeasureManager so "Should also hide Python measures in library if the Measure Manager is launched with the Classic CLI." is fixed

But the "If Python measures are in the workflow, add error badge if using the Classic CLI" is not fixed

jmarrec added a commit that referenced this issue Sep 3, 2024
@jmarrec
Fix #671 - Mark Python Measures as in Error if use classic CLI to run…
3cb5c6d 
… the simulation
@jmarrec jmarrec mentioned this issue Sep 3, 2024
Merged
@jmarrec jmarrec closed this as completed in 8d50667 Sep 3, 2024
jmarrec added a commit that referenced this issue Sep 3, 2024
@jmarrec
Merge pull request #733 from openstudiocoalition/671_MarkPythonErrorC…
f43dd7a 
…lassicCLI

Fix #671 - Mark Python Measures as in Error if use classic CLI to run the simulation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement Request New feature or request
Projects
None yet
Milestone
OpenStudio Application 1.8.0
2 participants
@macumber @jmarrec