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.

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

Fix empty scalefactors in model 125 #1

Closed
wants to merge 1 commit into from
Closed

Fix empty scalefactors in model 125 #1

wants to merge 1 commit into from

Conversation

cfstras
Copy link

@cfstras cfstras commented May 7, 2019

Fixes sunspec#32

@altendky
Copy link
Owner

altendky commented May 7, 2019

@cfstras, is this urgent for you? Or can we wait a few days to see where sunspec#39 goes? The change seems fine so if you follow my own workflow with an official PR (and mention me) I'll go ahead and merge it asap.

@altendky altendky changed the base branch from master to master_plus_pull_requests May 7, 2019 16:52
@altendky
Copy link
Owner

altendky commented May 7, 2019

Kick CI

@altendky altendky closed this May 7, 2019
@altendky altendky reopened this May 7, 2019
@cfstras
Copy link
Author

cfstras commented May 7, 2019

nope, not urgent.
I realized I had seen your workflow before about a minute after opening the PR, but the figured we wait to hear about sunspec#39, as you suggested.

@cfstras
Copy link
Author

cfstras commented May 8, 2019

moved to sunspec repo: sunspec#40

@cfstras cfstras closed this May 8, 2019
@cfstras cfstras deleted the patch-2 branch May 8, 2019 15:37
altendky pushed a commit that referenced this pull request Apr 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants