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

adding MSRx2 interfacer #13

Open
wants to merge 3 commits into
base: beaglebone
Choose a base branch
from
Open

adding MSRx2 interfacer #13

wants to merge 3 commits into from

Conversation

Basazababo
Copy link

adding solarpak charger controllers interfacers.

Copy link

@jlark jlark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tidy up the config file and it's ok

com_baud = 38400
[[[runtimesettings]]]
pubchannels = ToEmonCMS,
subchannels = ToRFM12,

# datacode = B #(default:h)
# scale = 100 #(default:1)
group = 210 #(default:210)
group = 3421749817 #(default:210)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this a crazy number?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This number was kind of encryption, node only send and receive data to another node with this group number.

apikey = xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
# data_send_interval = 20 #seconds
# status_send_interval = 60
#url = http://api.gle.solar
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets remove api keys and urls form the source

site_id = 5


#[[BMV]]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these comments look odd? We shuold leave the interfacer as a tempalte commented out.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don` understand what you mean, BMV and this data logger use the same serial port, one of them need to be commented out and here BMV is commented

import emonhub_interfacer as ehi
from emonhub_interfacer import EmonHubInterfacerInitError

"""class EmonhubSerialInterfacer
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets add a comment that this is for the MSrX charge controler

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

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