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

WIP: Ciric PolyhedralPrism #2769

Draft
wants to merge 54 commits into
base: develop
Choose a base branch
from

Conversation

CoronelBuendia
Copy link
Contributor

@CoronelBuendia CoronelBuendia commented Oct 30, 2023

Linked Issues

Closes #{ID}

UPDATE: Put on hold, because we have been unable to get a working implementation. This is still desirable (it is a more general form of an arbitrary current source).

Description

Interface Changes

Checklist

I confirm that I have completed the following checks:

  • Tests run locally and pass pytest tests --reactor
  • Code quality checks run locally and pass pre-commit run --from-ref develop --to-ref HEAD
  • Documentation built locally and checked sphinx-build -W documentation/source documentation/build

* working bottura

* get rid of some minuses

* kernel start

* docstrings

* ruff

* make tests better written

* more maths

* fix API documentation
* add baseclass

* arbitrary planar polyhedral

* arbitrary planar polyhedral proper

* add tests

* addplot test

* DRY

* remove unused zip

* rebase and fix circuit tests
@CoronelBuendia CoronelBuendia added the magnetostatics Tasks relating to the magnetostatics module label Oct 30, 2023
@CoronelBuendia CoronelBuendia requested a review from a team as a code owner October 30, 2023 10:59
@CoronelBuendia CoronelBuendia changed the base branch from develop to matti/polyhedral-prism-fabri October 30, 2023 11:00
Copy link

sonarcloud bot commented Nov 16, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 15 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@CoronelBuendia CoronelBuendia added the on hold Tasks that are paused label Dec 12, 2023
Base automatically changed from matti/polyhedral-prism-fabri to develop January 24, 2024 12:01
@je-cook je-cook marked this pull request as draft March 15, 2024 08:14
@je-cook je-cook force-pushed the develop branch 5 times, most recently from 5a50acb to 7574352 Compare May 15, 2024 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
magnetostatics Tasks relating to the magnetostatics module on hold Tasks that are paused
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants