Ticket #5138 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Create a migration script for moving v1 & v2 of Python API

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: critical Milestone: Release 2.1
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Stuart Campbell

Description


Change History

comment:1 Changed 8 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Martyn Gigg

Only aliases visible at mantid namespace level in Python. Refs #5138

Changeset: 1ddb61d3597c82f827ae80ae04244c95658159a8

comment:3 Changed 8 years ago by Martyn Gigg

First cut at migration script. Refs #5138

Works for scripts that only use simple API through positional args & keywords but does nothing with return values yet. GUI isnot yet functional either.

Changeset: 1392edd50b3e94383df4f7c5a59d69ffaa38e711

comment:4 Changed 8 years ago by Martyn Gigg

Run python tests with creating pyc files. Refs #5138

Changeset: 6df6ca1565f4906ddebfd173f562840324009d3a

comment:5 Changed 8 years ago by Martyn Gigg

Remove the VanillaPython*Test directory before creation. Refs #5138

If tests were removed then they will linger and become stale.

Changeset: 2bca095c16731200a646ddcd42c9397fb67f6595

comment:6 Changed 8 years ago by Martyn Gigg

Only aliases visible at mantid namespace level in Python. Refs #5138

Changeset: 1ddb61d3597c82f827ae80ae04244c95658159a8

comment:7 Changed 8 years ago by Martyn Gigg

First cut at migration script. Refs #5138

Works for scripts that only use simple API through positional args & keywords but does nothing with return values yet. GUI isnot yet functional either.

Changeset: 1392edd50b3e94383df4f7c5a59d69ffaa38e711

comment:8 Changed 8 years ago by Martyn Gigg

Run python tests with creating pyc files. Refs #5138

Changeset: 6df6ca1565f4906ddebfd173f562840324009d3a

comment:9 Changed 8 years ago by Martyn Gigg

Remove the VanillaPython*Test directory before creation. Refs #5138

If tests were removed then they will linger and become stale.

Changeset: 2bca095c16731200a646ddcd42c9397fb67f6595

comment:10 Changed 8 years ago by Martyn Gigg

Python Simple api keywords override lhs variables. Refs #5138

Changeset: 53fe85e958b1f81c0adc83b2732bd5b8b43d007e

comment:11 Changed 8 years ago by Martyn Gigg

Minor code shuffle in migration. Refs #5138

Separates rule application into different objects. Also added some tests.

Changeset: 6995bffdb264baa97a22a84e86edb79e22e17f7c

comment:12 Changed 8 years ago by Martyn Gigg

Checkpoint migration work. Refs #5138

Changeset: 726dac4d5ab14389c9c10821d89a05415cb69ac8

comment:13 Changed 8 years ago by Martyn Gigg

Python Simple api keywords override lhs variables. Refs #5138

Changeset: 53fe85e958b1f81c0adc83b2732bd5b8b43d007e

comment:14 Changed 8 years ago by Martyn Gigg

Minor code shuffle in migration. Refs #5138

Separates rule application into different objects. Also added some tests.

Changeset: 6995bffdb264baa97a22a84e86edb79e22e17f7c

comment:15 Changed 8 years ago by Martyn Gigg

Checkpoint migration work. Refs #5138

Changeset: 726dac4d5ab14389c9c10821d89a05415cb69ac8

comment:16 Changed 8 years ago by Martyn Gigg

Migration script works for no arg return calls. Refs #5138

Changeset: fd9003e77b50842a3655dcc461f78ed3e038e545

comment:17 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.1 to Release 2.2

Moved at end of release 2.1

comment:18 Changed 8 years ago by Martyn Gigg

  • Status changed from accepted to verify
  • Resolution set to fixed
  • Milestone changed from Release 2.2 to Release 2.1

Closing for testing.

Importantly it only handles no-arg returns from simple API calls and will not translate Python algorithms.

#5197 has been created to improve this.

comment:19 Changed 8 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:20 Changed 8 years ago by Stuart Campbell

Seems to work for some basic scripts, but as it's still under development and this ticket has just been closed for testing - just going to pass it.

Last edited 8 years ago by Stuart Campbell (previous) (diff)

comment:21 Changed 8 years ago by Stuart Campbell

  • Status changed from verifying to closed

comment:22 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5984

Note: See TracTickets for help on using tickets.