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

Feature/Implement logic to construct charge graph #203

Closed
wants to merge 16 commits into from

Conversation

CodeBear801
Copy link

@CodeBear801 CodeBear801 commented Mar 7, 2020

This pull request is too big, I will create several small pull request for better understanding. Please just use this pull request for reference purpose.

Issue

  • Implement logic to construct charge graph.
  • Fix issue related with charge strategy
  • Fix issue in station graph while constructing graph
  • Fix issue in graph's cost calculation

Tasklist

// ChargingStrategyCreator defines interface related with creation of charging strategy
type ChargingStrategyCreator interface {

// CreateChargingStrategies creates charge strategies which could be used by other algorithm
CreateChargingStrategies() []ChargingStrategy

Copy link
Author

Choose a reason for hiding this comment

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

Create charging candidates

@@ -1,14 +1,22 @@
package chargingstrategy
Copy link
Author

Choose a reason for hiding this comment

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

This pull just for reference!

@CodeBear801 CodeBear801 removed the request for review from wangyoucao577 March 8, 2020 01:43
@CodeBear801 CodeBear801 deleted the feature/oasis-integration2 branch April 2, 2020 18:02
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.

1 participant