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

RFA (Restricted Filament Annealing) #89

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

RFA (Restricted Filament Annealing) #89

wants to merge 4 commits into from

Conversation

hanjinliu
Copy link
Owner

Instead of running on a 2D graph, we can just apply the same method to 1D graph, i.e., filament.
This PR implements both the frontend and the backend of this method, here we call RFA.

Copy link

codecov bot commented Jun 30, 2024

Codecov Report

Attention: Patch coverage is 90.00000% with 8 lines in your changes missing coverage. Please review.

Project coverage is 83.60%. Comparing base (b43d5c9) to head (aff1a5d).

Files Patch % Lines
cylindra/components/landscape.py 83.78% 3 Missing and 3 partials ⚠️
cylindra/widgets/sta.py 94.87% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
+ Coverage   83.55%   83.60%   +0.04%     
==========================================
  Files         105      105              
  Lines       14912    14968      +56     
  Branches     3153     3168      +15     
==========================================
+ Hits        12460    12514      +54     
- Misses       1630     1632       +2     
  Partials      822      822              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

None yet

1 participant