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

5121 eclipse ease python #5159

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from
Open

Conversation

saulhidalgoaular
Copy link
Collaborator

@saulhidalgoaular saulhidalgoaular commented Apr 30, 2022

Fix #5121

@auto-assign auto-assign bot requested a review from IanMayo April 30, 2022 01:38
@IanMayo
Copy link
Member

IanMayo commented May 4, 2022

Thanks @saulhidalgoaular , I've been playing with this branch in Windows 10. Opened Debrief, opened Scripting perspective, then changed scripting language to Python (Py4J). A few seconds later this dialog opened:

image

Do I need to manually install Python on the target machine first? If that's the case, we should write some instructions on what is required.

@saulhidalgoaular
Copy link
Collaborator Author

Good afternoon. Yes, Python is necessary and it needs to be added into the configuration of Debrief. I will create some instructions about it.

@IanMayo
Copy link
Member

IanMayo commented May 4, 2022

Good afternoon. Yes, Python is necessary and it needs to be added into the configuration of Debrief. I will create some instructions about it.

Thanks, could you write the instructions into here please?

@saulhidalgoaular
Copy link
Collaborator Author

Good evening @IanMayo .
I consider this PR is ready to review. I have also updated the documentation

@IanMayo
Copy link
Member

IanMayo commented Jan 22, 2023

Thanks for all this Saul. I found your installation content.

Would you mind moving it into section 5.9 of this tutorial please?
https://docs.google.com/document/d/1q8_h2C7OT0-n-Kt0idWqw3iMleUFZO5S3SYbT85PSPY/edit#heading=h.w4oix6r9vc7y

In the File changes part of this PR I'll highlight a couple of blocks of Python code that I think it would be good to add to the tutorial.

Aah, but... The tutorial in general does try to be "hands on". We do a "walked example" of writing a JS module. Could you copy that content, and re-doc it for Python please? Then, by the end of the tutorial, users will have actually created and run a Python module themselves.

# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
#

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these lines, which pull in support modules

loadModule('/Debrief/Core'); # for plot, duration, date



Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

get a plot, then get the layers object, then get the tracks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integrate EASE Python scripting
2 participants