-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Is this library still maintained? #849
Comments
Same question. |
I made a fork that is a WIP but will be adding additional features and support and can be installed using |
my models from a while ago all die at the utils section now. Was moving to torchgeo but now torchgeo is still having issues with this repo. I guess I'll look at torchseg. |
TorchGeo may switch to TorchSeg if smp is no longer maintained. |
Well analyzed @adamjstewart ! Would be great to hear the verdict on this. |
This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
The irony is palpable |
@qubvel great to see significant activity on this repo over the last week! However, my question still remains. TorchGeo is still deciding between depending on (and contributing to) SMP or TorchSeg. We don't really want to maintain TorchSeg if we don't have to, but SMP is missing several things I think we would need for long-term use:
If these terms seem reasonable to you, please reach out to us and let us know. We would love to combine forces and either merge the improvements to TorchSeg back into this repo or vice versa. |
@adamjstewart thanks for reaching out! First of all, I apologize a lot for the low activity and for ignoring the library for the last year.. lots of things have happen. But since I am back to open source, I am going to maintain the library again! Regarding your questions:
More than that, I already have a list of ideas I would like to be implemented! |
Awesome, let us know how we can help! I'll close this issue once there are more maintainers, that's probably the biggest threat to maintenance. |
@qubvel are there any updates on this? We would like to decide whether to switch TorchGeo to torchseg or stick with SMP in the near future, but I haven't seen any progress on the above points. |
|
|
|
Awesome, thanks for adding us! I'll let @isaaccorley submit PRs for some of the changes he's already made to TorchSeg and try to review them when I get a chance. I briefly spoke to him about the encoder stuff and I think his decision was made to keep TorchSeg as simple as possible (which is extremely important for maintainability), but at the cost of backwards compatibility (which is also important to avoid upsetting users). We can probably discuss this on GitHub and come up with a good solution together. Do you have a preferred platform (slack, discord, email) for discussing maintainer questions (e.g., merge policies, release schedules, etc.)? My email is on GitHub and I think @isaaccorley's is also easy to find, but slack might be easier for long-term discussions. That's what we primarily use for TorchGeo. |
@adamjstewart @isaaccorley are you using Discord? I've created a server there, here is a join link https://discord.gg/jWrNZwpB |
Hey @qubvel @adamjstewart |
Hi @patrontheo, we are going to merge |
It's been almost a year since the last release, and most commits since then have been limited to auto-generated dependabot PRs. The outdated version of timm required to use smp now makes it incompatible with the latest release of lightly: microsoft/torchgeo#1824. PRs to update the version of timm have been ignored (#839), and requests to unpin the timm dependency have been rejected (#620). Our own contributions have been closed, and requests to reopen them have been ignored as well (#776).
Which begs the question: is this library still maintained?
If yes, then it would be incredibly helpful to unpin the timm dependency.
If no, then would you be willing to pass the torch (heh) on to someone else so this incredibly useful library does not become abandoned? Alternatively, does anyone know any alternatives that offer the same functionality and compatibility with modern timm releases as smp?
The text was updated successfully, but these errors were encountered: