Skip to content

Fronius Gen24 with Storage and Inverter Control#3755

Open
vision1727 wants to merge 3 commits into
OpenEMS:developfrom
vision1727:fronius_gen24
Open

Fronius Gen24 with Storage and Inverter Control#3755
vision1727 wants to merge 3 commits into
OpenEMS:developfrom
vision1727:fronius_gen24

Conversation

@vision1727

Copy link
Copy Markdown

Created another version of a controller for the Fronius Gen24.
it is running some time now and looks good.

for the Controller the base was a SMA Stpse.

I hope it is created the right way. :-)

The components gen24, charger and battery are new.

in the original of meter I added support for Sunspec int + SF.


public enum BatteryState implements OptionsEnum {
UNDEFINED(-1, "Undefiniert"),
OFF(1, "Aus"),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Names are generally in English language

}

@Override
public int getValue() { return this.value; }

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please apply Coding guidelines for standard formatting etc. https://openems.github.io/openems.io/openems/latest/development/coding-guidelines.html

import org.osgi.service.metatype.annotations.ObjectClassDefinition;

@ObjectClassDefinition(//
name = "ESS Fronius Gen24 Battery with int+SF ", //

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Name and webconsole_configurationFactory_nameHint have to correlate - see other implementations

.accessMode(AccessMode.WRITE_ONLY)//
.unit(Unit.NONE)), //

// SELF_CONSUMPTION_ACTIVATION(Doc.of(OpenemsType.INTEGER) //

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Remove unused comments

}
)
@GenerateTargetsFromReferences("Modbus")
public class BatteryInverterFroniusGen24Impl extends AbstractSunSpecBatteryInverter

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A Battery-Inverter in OpenEMS is meant to be used by GenericEss. This will unfortunately require a lot of refactoring.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thx for reviewing. Working on changes

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