-
Notifications
You must be signed in to change notification settings - Fork 22
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
Update ISSUE_TEMPLATE.md #111
base: master
Are you sure you want to change the base?
Conversation
.github/ISSUE_TEMPLATE.md
Outdated
@@ -5,7 +5,7 @@ | |||
- [ ] Python version: <!-- (e.g. 2.7, 3.3, 3.6) --> | |||
- [ ] Python architecture: <!-- (e.g. x86, x64) --> | |||
- [ ] Operating system and version: <!-- (e.g. Windows XP, Windows 7, Windows 8, OSX 11, Red Hat, Ubuntu, Centos 6, Centos 7) --> | |||
- [ ] OpenDSSDirect.py version number: <!-- (e.g. v0.1.0, v0.2.5, v0.3.0) --> | |||
- [ ] OpenDSSDirect.py version number: <!-- (e.g. v0.1.0, v0.2.5, v0.3.0) You can use `opendssdirect.Basic.Version()` to get this information --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's just the engine version though:
OpenDSSDirect.py/opendssdirect/Basic.py
Lines 67 to 69 in bb20ea7
def Version(): | |
"""(read-only) Get version string for the DSS.""" | |
return get_string(CheckForError(lib.DSS_Get_Version())) |
Shouldn't opendssdirect.__version__
be more appropriate?
I think that including opendssdirect.__version__
in opendssdirect.Basic.Version()
is a good idea to get all info in one place, but we can't address that for old releases.
Maybe we could also instruct users to post text instead of images. |
We should also probably create a separate template for support questions. GitHub now allows multiple templates for issues. |
No description provided.