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.

Sign up for GitHub

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

Jump to bottom

add features from cellmap fork #29

Open
wants to merge 49 commits into
base: master
Choose a base branch
from
Open

add features from cellmap fork #29

wants to merge 49 commits into from

Conversation

d-v-b
Copy link

@d-v-b d-v-b commented Feb 13, 2024

@mzouink and @rhoadesScholar can yall summarize the changes here?

mzouink and others added 30 commits August 30, 2023 15:04
@mzouink
fix distance mask predictor
eb1309c
@rhoadesScholar
Update black.yaml - force formatting
5fe699a
@rhoadesScholar
Update black.yaml - automatic black
0098fa2
@rhoadesScholar
Update black.yaml - remove auto black
13ae1f0
@mzouink
Update black.yaml
d741626
@mzouink
remove black
9351c16
@mzouink
add black format check
bb8cab5
@mzouink
black format on pull request
c25cb79
@mzouink
🎨 Format Python code with psf/black
08f134d
@mzouink
Merge pull request #19 from janelia-cellmap/actions/black
e73afa2 
Format Python code with psf/black push
@mzouink
bug fixes and better logs
ff61f7c
@mzouink
Update train.py
1490440
@mzouink
Update train.py
3c5f2da
@rhoadesScholar
Merge pull request #20 from janelia-cellmap/zouinkhim_fixes
34e8253 
bug fixes and better logs
@rhoadesScholar
feat: 🚧 Incorporate simple change from rhoadesj/dev
33bbc8a
@rhoadesScholar
feat: 🚧 Incorporate simple change from rhoadesj/dev
fe23b5d
@mzouink
Merge pull request #25 from janelia-cellmap/rhoadesj_simple_changes
5f50f9b 
Rhoadesj simple changes
@rhoadesScholar
feat: ⚡️ Incorporate start related changes from rhoadesj/dev
812acc1
@rhoadesScholar
docs: 📝 Add authors and versioning.
ce5d272
@mzouink
starter partial weight load
4f1dfed
@mzouink
Merge pull request #35 from janelia-cellmap/fix_start_loader
910b2e0 
Fix only starter weight loader
@mzouink
🎨 Format Python code with psf/black
906dfd6
@mzouink
Merge pull request #37 from janelia-cellmap/actions/black
8cce986 
Format Python code with psf/black push
@mzouink
publish to pypi
f5e584a
@rhoadesScholar
logo
281a768
@rhoadesScholar
Update README.md
65482f4
@rhoadesScholar
Update README.md
52409f9
@mzouink
Update publish.yaml
b8e18b4
@rhoadesScholar
fix: 🐛 Fix broken dependencies for MacOS.
b4b2780
@davidackerman
include and use more biases during watershed post processing of affin…
55a3892 
…ities
davidackerman and others added 19 commits February 9, 2024 14:47
@davidackerman
include weighting argument for affinities+lsd loss
58c7abe
@davidackerman
make predictor node optional
ce71fb5
@davidackerman
🎨 Format Python code with psf/black
353b8cb
@mzouink
Merge pull request #42 from janelia-cellmap/actions/black
4a54e31 
Format Python code with psf/black push
@mzouink
styles fixes for mypy
f243c7c
@mzouink
update git action, fix doc and no more publish
cebc737
@mzouink
remove unfinished cli and apply from main
7feab6a
@mzouink
fix test action, pytest 8.0.0 working
5d77af0
@mzouink
🎨 Format Python code with psf/black
e46acf0
@mzouink
Merge pull request #45 from janelia-cellmap/actions/black
cea9c03 
Format Python code with psf/black push
@mzouink
🎨 Format Python code with psf/black
232047c
@mzouink
Merge pull request #46 from janelia-cellmap/actions/black
8b9d44a 
Format Python code with psf/black push
@mzouink
test only with python 3.10
62d6278
@mzouink
bug fix: loading starter weight, layer exist but mismatch shape
3c2f0fe
@mzouink
Merge pull request #60 from janelia-cellmap/fix_bug_starter_shape
84436e8 
bug fix: loading starter weight, layer exist but mismatch shape
@mzouink
update size checking
4a4bd94
@mzouink
🎨 Format Python code with psf/black
5855dd0
@mzouink
Merge pull request #63 from janelia-cellmap/actions/black
6afcacd 
Format Python code with psf/black push
@mzouink
Merge pull request #64 from janelia-cellmap/fix_bug_starter_shape
d0512f9 
update size checking for starter loader
@mzouink
Copy link

mzouink commented Feb 13, 2024

i added comments to the code

@d-v-b
Copy link
Author

d-v-b commented Feb 13, 2024

who will review this PR? @davidackerman @funkey @pattonw ?

@mzouink
Copy link

mzouink commented Feb 13, 2024

i don't think this a goal now to merge with funkelab repo. Janelia-CellMap/Dacapo will be the main repo for now.

@d-v-b
Copy link
Author

d-v-b commented Feb 13, 2024

Janelia-CellMap/Dacapo will be the main repo for now.

OK, so funkelab/dacapo will be archived? Or how else will users know which repo is the main repo?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@d-v-b @mzouink @rhoadesScholar @davidackerman