-
Notifications
You must be signed in to change notification settings - Fork 21
Added some UPS device files for Ippon and Talegent #51
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
Conversation
Signed-off-by: penghong <penghong20000102@163.com>
|
@penghong123 : somehow XML markup crawled into the data dumps (many files, not sure if all of them or not), e.g.: Can you please revise the PR and add a commit to fix this?
|
Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
NOTE: It seems all ~30 files are identical, not data dumps of each referenceed device model. Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
NOTE: It seems all ~30 files are identical, not real data dumps of each referenced device model. Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
|
@penghong123 : I've revised those files to remove the XML markup; it is still questionable if there were some NUT data points missed in place where buggy information was. The manual revisiting also seems to have been a waste of time - it seems that all ~30 files are identical, and (all but one?) are not real data dumps of each referenced device model individually. So instead of editing 30 files individually I could have edited one and copied to 30 places much faster :( But in practice this duplication is also almost useless: there may be nuances of each device model exposed in their data points (different capacity etc.) We need real data to compare devices and drivers over time, find some patterns when hunting for bugs, etc.
And we still need you to check if any data points got lost with the broken markup of initial posting? |
Sorry, I've been too busy lately to pay attention to the message. Then I delete the other files and only keep one file, because those machines are borrowed by me and have been returned. |
|
I see... and just in case, you don't have any real test results for other UPS models on your computer, which you would have just forgot to add to git? Which actual model is posted here? It is hard to see from |
At that time, I only focused on whether the NUT could be adapted to these machines, and forgot to record the data. When I tested it, I found that their results were all not very different, so I didn't pay attention to the differences between them. Here's the data from the Do I need to delete other files and resubmit? |
|
If you can, yes, please. Thank you! |
Merge upstream/master into local master. The previously submitted .dev file lacking test data has been deleted.
Signed-off-by: penghong <penghong20000102@163.com>
|
Thanks, at least the resulting state looks good now. The DCO check is optional for now, detailed here : https://github.com/networkupstools/nut/wiki/Code-contributions,-PRs,-PGP-and-DCO (in short, a legalese-encouraged sign-off that a human who had the intellectual-property right to the contribution does contribute it). |
Now, is there anything else I need to do? |
No, thanks - all good now; merged :) |


I tested both NUT2.7.4 and NUT2.8.3.64, and they can both read the UPS status normally
and control the UPS devices properly, such as shutting down.
Make a contribution.
See also: