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
Discussion: future of this project #292
Comments
Hey @Gekkio, I think you're doing a pretty amazing job developing and maintaining imgui-rs, and we're happily using it (almost as-is). You mentioned you didn't have a lot of time on your hands to look at bigger PR's which is what I responded to, and wanted to help out with.
This is along the lines I'm thinking tbh, the project itself looks like it won't need a lot of supervision (especially since Dear ImGui doesn't break back compat all that much), but if you're lacking time at the moment I'm happy to jump in and do some of that work with / for you. |
Cool! 😄
Out of curiosity, what kind of changes do you have on top of upstream imgui-rs? Any pain points that could be alleviated by upstream changes? |
@Gekkio I'd like to echo @Jasper-Bekkers comments, I was really responding to the exact same mention of not having time for reviewing larger PR's, as I'd like to also help keep this updated if needed. It is used by multiple projects in the amethyst org, as well as my amethyst-imgui crate, which is used by amethyst users. Since you have @Jasper-Bekkers as a collab now, and are at least willing to accept reviews from them, I'm happy with the resolution as well! I just wanted to make sure to be able to keep this project alive and moving forward. |
Just curious, is there any update on this? I'd like very much to experiment with the |
Same here, but i don't want to experiment with it but rather use it within engine. |
Nothing has really changed:
|
Hey guys. I'm pretty desperate for the docking branch and have the time (one week full-time), to pull it down and bring it across. Has anyone else made progress on this before I start looking into it? |
I don't believe so, no. Great that you're planning to work on it! |
okay great. I'll post progress updates, to the docking issue #249. |
Hey, I just want to comment on this further. The docking branch of dear imgui is super useful for people that want to use imgui as the main ui for their application. It does all the work of layout out your window correctly. I've showed a snap shot of a rust app, using it fine. I'm based off the master from this repo, with docking merged in. The thing is, to get this working, changes need to be made in imgui-rs. On top of that, there are multiple people interested in moving this forward, sorting out the safe api so that when this features goes to stable upstream, imgui-rs will be ready. It's a long lived feature that is in beta. It's not a short feature branch. In order to allow multiple people to collaborate on getting docking working in imgui-rs, with a rust like api, there needs to be a central place to push/pull. The options are:
In my view, currently, this is the canonical place to go to for rust bindings for imgui, it seems unnecessary for someone else to make a fork for this branch. It would fragment development. Please please please, say yes to adding a docking branch, and if you are super busy at the moment, please find someone who can be in charge of managing pull requests for this project. |
Is there a discord / telegram / gitter channel for imgui-rs? If not, it would be really great to have one! :) I think imgui-rs is the best option for complex GUIs in Rust right now, but a lot of things about it are non-obvious / require workarounds. So it would be really helpful to have a discord channel, also to increase adoption of imgui-rs by creating a community around it. |
I just found out there is already a discord for the C++ version of imgui, with over 1000 members: |
|
...discussion based on comments in #285
Can everybody please clarify what exactly is expected of me that is not being provided at the moment? I'll happily consider giving full access to more people, but I'm also getting very mixed signals here.
@rasky wants review of this branch, which I'm not capable of delivering because this pull request has very little information about the change, and reviewing the pull request requires a good understanding of both the Dear Imgui feature (which is still experimental), and how that maps to imgui-rs.
What about @Jasper-Bekkers, @jaynus? What needs do you have that are not being met at the moment but would be if I gave full admin access to some people (you?)? My criteria for a crate being "unmaintained" is usually that pull requests don't get merged, which doesn't seem to be the case here, because the problem seems to be lack of review and/or lack of active development. I'm not actively doing work on this crate, but it's not the same thing as not accepting any changes.
There's nothing preventing somebody else from reviewing this pull request right here. In fact, if docking was in dear imgui master and somebody from Embark had reviewed this pull request, I'd merge it without hesitation.
There's nothing preventing somebody else from implementing "critical changes that weren't finished". If reasonable pull requests were submitted, I'd merge them.
And just to be clear: in any case, forking won't be needed. I'd much rather completely give away this project to somebody else than let it rot. However, the discussion so far seems to be mostly about credentials, not about the actual effort that moves the project forward. I've already given contributor access to one other person, and will gladly do it again. But what exactly does it change in practice other than credentials?
TL;DR:
"I need X but Gekkio doesn't do/provide Y" <- please fill in X and Y
"Q is not possible right now, but if Gekkio gave admin access to <name of person>, it would be possible" <- please fill in Q
The text was updated successfully, but these errors were encountered: