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.

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

ALPS on VisTrails 2.2 #1070

Closed
remram44 opened this issue Apr 30, 2015 · 10 comments
Closed

ALPS on VisTrails 2.2 #1070

remram44 opened this issue Apr 30, 2015 · 10 comments

Comments

@remram44
Copy link
Member

Subworkflows need to be opened, upgraded and saved back, no automatic upgrade happens

ALPS setup discussion is here, will open a separate ticket for clarity 😉

Follows up on #1067

@vistrails-dev
Copy link

(Matthias)

I know, and tried it. Unfortunately bumping the version number makes the package initialization fail, and I thus cannot open them. I’ll try to edit them manually.

@remram44
Copy link
Member Author

I should have done all the necessary changes on ALPS in our alps-mpl branch (I imported ALPS from your SVN server). Is there something I forgot?

@vistrails-dev
Copy link

(Matthias)

I needed the open and save the ShowMplPlot.xml file and everything worked as long as I left the version number at 2.2.0.

I now bumped the version number of ALPS from 2.2.0 to 2.2.1 and get this error even if I comment out the subworkflows:

Critical message:
Error changing version
InvalidPipeline: Pipeline has errors. Please see the detailed message for more information.
Missing version 2.2.0 of package org.comp-phys.alps
Upgrading workflow failed.
Output port "self" of module "MplXYPlot" exists, but signatures differ "(org.vistrails.vistrails.basic:Module)" != "(org.vistrails.vistrails.matplotlib:MplPlot)"

Time:
2015-04-30 21:26:09,015
Location:
/Applications/VisTrails/VisTrails.app/Contents/Resources/lib/python2.7/vistrails/gui/vistrail_controller.py, line 564

@remram44
Copy link
Member Author

Yes, MplXYPlot now uses the "value" output port. An upgrade is necessary for the rename, which I provided on branch alps-mpl (50e2c1b).

@vistrails-dev
Copy link

(Matthias)

It does not seem to work though, even using that.

@vistrails-dev
Copy link

Just to clarify: the problem happens when I bump the ALPS version number from 2.2.0 to 2.2.1

On 30 Apr 2015, at 21:35, Matthias Troyer troyer@phys.ethz.ch wrote:

It does not seem to work though, even using that.

Matthias

On 30 Apr 2015, at 21:31, Remi Rampin <notifications@github.com mailto:notifications@github.com> wrote:

Yes, MplXYPlot now uses the "value" output port. An upgrade is necessary for the rename, which I provided on branch alps-mpl (50e2c1b 50e2c1b).


Reply to this email directly or view it on GitHub #1070 (comment).


vistrails-dev mailing list
vistrails-dev@vistrails.org mailto:vistrails-dev@vistrails.org
http://lists.vistrails.org/mailman/listinfo/vistrails-dev


vistrails-dev mailing list
vistrails-dev@vistrails.org
http://lists.vistrails.org/mailman/listinfo/vistrails-dev

@remram44
Copy link
Member Author

I see. From the log you posted, it looks like it doesn't upgrade to 2.2.1; did you change the target version numbers from 2.2.0 from 2.2.1 in handle_module_upgrade_request(), in init.py?

@vistrails-dev
Copy link

(Matthias)

Thank you, that helped!

@vistrails-dev
Copy link

(Matthias)

ALPS now seems to work well on my Mac, and the latest package that works is uploaded to our server. I’ll start on the Windows builds now.

@remram44 remram44 changed the title Subworkflows don't use package upgrades ALPS on VisTrails 2.2 Apr 30, 2015
@vistrails-dev
Copy link

(Matthias)

The Win32 version also works but is not yet on the server. I’m struggling with memory corruption caused by Boost.Python on Win64. Knowing which compiler Python was built with might help resolve the issue.

@remram44 remram44 closed this as completed Jun 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants