Skip to content

Conversation

@Grufoony
Copy link
Collaborator

  • Add StochasticStreet class
  • Add Counter class, an utility class for spire streets
  • Some code refactoring

@Grufoony Grufoony added priority::high High priority stuff check required More details required code refactoring labels Jan 22, 2025
@Grufoony Grufoony requested a review from sbaldu January 22, 2025 10:52
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

Cppcheck (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@Grufoony
Copy link
Collaborator Author

I don't really know what's the problem with apple clang.
From tests, it seems to miss an evolve call: calling evolve once more in that test makes it pass on apple but unpass on linux... weird. @sbaldu have you an idea?

@Grufoony Grufoony merged commit 1d51329 into main Jan 23, 2025
25 checks passed
@Grufoony Grufoony deleted the StochasticStreet branch January 23, 2025 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

check required More details required code refactoring priority::high High priority stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants