-
Notifications
You must be signed in to change notification settings - Fork 68
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
vistrails module uvcdat_cdms missing #581
Comments
I'll need more info on this issue. I've been using uvcdat-next and I don't see any problems with loading variables. -- Tom From: Charles <notifications@github.commailto:notifications@github.com> Jim's UVCDAT GUI fils loading in a variable Going back commit after commit, we found that the breaking point was introduced between the follwing 2 commits. I guess @ThomasMaxwellhttps://github.com/ThomasMaxwell you need to look into this. commit 03624da
commit 5e6ac22be23c7bbf360b16676f8eb6da83d1ea30 — |
I now get this as well....
|
and going back to your sha1 does not solve the issue.... No idea what is going on here.... |
I am not able to re-create this problem on my end |
may be you want to send the .uvcdat folder to @dakoop |
I think I'm onto something:
essentially pkgs is never used. that's probably why.
then it complians there's no such function. Thanks for pointers. |
I believe this code is not there in uvcdat-master. Make sure you are using the current vistails uvcdat-master. |
@aashish24 I don't think it has to do with .uvcdat. I removed it and it still does that. BUT when launching uvcdatI don't see the bits that says init uvcdaT_cdms (I think i used to):
|
@aashish24 you're right this code is not there anymore. I was looking at the sha1 of the "working" (not) one in this thread. What happened to this code? Where did it go? |
fresh build this morning for jim made it disappear for him. Fresh build for me made it appear (both caes on macs 10.9.4) |
so does 174d1ea |
ok now switching back to uvcdat-master it works... That's like black magic. |
I have absolutely no idea what is going on here... Both Jim and I had this appear and disappear |
this code was not required and Remi removed it. |
by the way this code IS in uvcdat-next.... |
is there a branch that was merged inappropriately? |
doutriaux1@sofia:[/lgm/uvcdat/2014-09-02-devel/vistrails]:[uvcdat-next]:[1803]> git branch --no-merged |
@remram44 should I merge both 520 issues into uvcdat-next? or just one? which one? |
It's really difficult for me to follow your Git workflow. The only thing I can see right now is |
cool! I think that's all i needed to know will merge uvcdat-520-fix-loaded-pkgs into uvcdat-next now. Thanks again. Workflow is as follow: Hope this helps. |
thx @remram44 |
duplicate #595 |
that one appeared and disappeared magically for Jim and I. Going back and forth in commits seems to magically fix it.... |
Jim's UVCDAT GUI fils loading in a variable
Going back commit after commit, we found that the breaking point was introduced between the follwing 2 commits. I guess @ThomasMaxwell you need to look into this.
The text was updated successfully, but these errors were encountered: