-
Notifications
You must be signed in to change notification settings - Fork 3
Angiogenesis #190
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
Draft
cainja
wants to merge
56
commits into
main
Choose a base branch
from
cainja/growth-component
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Angiogenesis #190
Changes from 22 commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
0cf2719
started to update some import issues
cainja 9915650
fixing compiler errors
cainja fb498be
fixed compiler errors for angiogenesis
cainja 1748fca
spotless
cainja 363d67f
updated core graph utilities
cainja 74c0da2
doc string
cainja 86ba444
bisection precision tests
cainja 3e15f8b
slight refactor
cainja e146543
renamed function
cainja e9d09f9
capitalization error
cainja 17f8d9c
java docs
cainja a9fd46f
placeholder docstrings
cainja 9a2bc91
spotless
cainja 946889c
added tests for register and scheduler
cainja 253aad3
added test for getPath
cainja 0ba65bf
cleaning up growth component
cainja 2c832e6
removing redundant variables
cainja 5dda2c9
updated some javadocs
cainja bd7ee7f
linter updates
cainja fdd8d07
updated to work with multiple veins
cainja 943281d
updated boolean argument
cainja 5197158
updated doc strings
cainja eb06923
updated for consistency
cainja 4d2fe52
added input argument into string
cainja a7572c1
removed some useless code
cainja 98ad1bf
added doc string
cainja bf1ca4c
cleaned up some error passing
cainja a19778e
i have a lot of logging stuff right now
cainja 28dfc5c
it's a different bug now
cainja cab2f02
updated some matrix maths, adding some log statements trying to figur…
cainja 9f169a0
spotless
cainja 6037755
updated default threshold
cainja 6006b41
added transport
cainja 3cce139
added fraction
cainja e48151f
fixed directional check based on level
cainja 2086199
edges are still being removed before establishing
cainja 8bb6ef6
still not establishing edges
cainja 2399cb4
still having pointer issues
cainja 79bc465
temp edge fix
kristaphommatha 5cf3045
adding more specific debug log info
kristaphommatha cfad7eb
debug logging
kristaphommatha 5e74cb3
more debug logs
kristaphommatha db39c38
fix connecting two angio nodes and null sproutDir
kristaphommatha b1d605e
simulation growth is behaving abnormally
cainja 317484a
remove node delay logic
cainja d931ac1
changed node verification logic
cainja df34e85
Fixed mergeGraph() node map problem, added more debug logs
kristaphommatha f94e0da
style fixes
kristaphommatha 7cec022
fixing undefined edge direction, debug logger for angiogenicNodeMap t…
kristaphommatha 45ad5b3
fixing keyNodesToRemove logic
kristaphommatha 7656e5c
fixed issue where two sproutNodes intersected on the same edgeList
kristaphommatha 36d6e6a
removing debug logs, add updateRadiusToRoot back
kristaphommatha 7ab6c1a
resolved conflict
cainja b3da84a
added back in node delay logic, removed superfluous logging statements
cainja 9632e82
removed unnecessary node list
cainja 06db7cb
changing path.isEmpty() logic
kristaphommatha File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
22 changes: 22 additions & 0 deletions
22
src/arcade/core/util/exceptions/IncompatibleFeatureException.java
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| package arcade.core.util.exceptions; | ||
|
|
||
| /** Exception thrown when incompatible features are given. */ | ||
| public class IncompatibleFeatureException extends RuntimeException { | ||
| /** | ||
| * Constructs an {@code IncompatibleFeatureException} with the specified detail message. | ||
| * | ||
| * @param feature the feature name | ||
| * @param given the given associated feature | ||
| * @param expected the expected associated feature | ||
| */ | ||
| public IncompatibleFeatureException(String feature, String given, String expected) { | ||
| super( | ||
| String.format( | ||
| feature | ||
| + "is incompatible with " | ||
| + given | ||
| + ", must be associated with " | ||
| + expected | ||
| + ".")); | ||
| } | ||
| } |
13 changes: 13 additions & 0 deletions
13
src/arcade/core/util/exceptions/MissingSpecificationException.java
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| package arcade.core.util.exceptions; | ||
|
|
||
| /** Exception thrown when a specification is missing. */ | ||
| public class MissingSpecificationException extends RuntimeException { | ||
| /** | ||
| * Constructs an {@code MissingSpecificationException} with the specified detail message. | ||
| * | ||
| * @param missing the expected specification | ||
| */ | ||
| public MissingSpecificationException(String missing) { | ||
| super("The input file must contain the following specification: "); | ||
cainja marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| } | ||
| } | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.