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

Add diode component #343

Merged
merged 5 commits into from
Nov 21, 2024
Merged

Add diode component #343

merged 5 commits into from
Nov 21, 2024

Conversation

langestefan
Copy link
Contributor

@langestefan langestefan commented Nov 18, 2024

Checklist

  • Appropriate tests were added
  • Any code changes were done in a way that does not break public API
  • All documentation related to code changes were updated
  • The new code follows the
    contributor guidelines, in particular the SciML Style Guide and
    COLPRAC.
  • Any new documentation only uses public API

Additional context

afbeelding

@langestefan
Copy link
Contributor Author

Anything I need to do to fix the failed checks? Doesn't look like they are related to my code

@ChrisRackauckas
Copy link
Member

Yeah diode passed, the other bits are due to some madness of the day don't worry about that.

@ChrisRackauckas ChrisRackauckas merged commit 1827d63 into SciML:main Nov 21, 2024
6 of 12 checks passed
@langestefan langestefan deleted the diode branch November 22, 2024 11:37
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.

3 participants