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

New custom step: CV - Display Images #161

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

neelan29
Copy link
Contributor

@neelan29 neelan29 commented Aug 6, 2024

Please include answers to these questions as part of your pull request

In the GitHub webUI, use the Write tab to modify the Markdown text that is part of the pull request. For each question simply place an X inside the square brackets, [X], that represents your answer. Make sure there are no blanks inside the brackets, otherwise MarkDown doesn't render properly. Using the Preview tab while editing this form, you can see the formatted/rendered version of the message.

  • Q1: Confirm that you have the right to submit the code that is being contributed. Please consider the origin of your code and confirm you have the appropriate rights to make the submission subject to the Apache 2.0 license that applies to everything in this repository of custom steps. If so, follow the instructions for the Contributor Agreement (which is based on the industry-standard Developer Certificate of Origin (DCO)).
    • Yes, I have the right to submit the contributed code on behalf of myself, my company, or any other owner of the code. I have also attached my signed copy of the DCO to this message.
    • No
  • Q2: Confirm that your contribution does not include any personally identifiable information (PII), for example, in any examples used in your README file.
    • My contribution does NOT include PII data
    • My contribution includes PII data
  • Q3: Confirm your contribution does not include any encryption or other export-controlled technology.
    • My contribution does NOT contain encryption or other export-controlled technology
    • My contribution includes encryption or other export-controlled technology

Contributor_Agreement_NV.txt

@snlwih snlwih added the new custom step This pull request represents a new custom step label Aug 9, 2024
Both README.md and About tab did not display the release date using the "standard" format.
@snlwih
Copy link
Collaborator

snlwih commented Aug 20, 2024

Quickly updated README.md and About tab to use the standard DDMMMYYYY format for release date

@snlwih
Copy link
Collaborator

snlwih commented Aug 20, 2024

Started an offline discussion about using Python SWAT to create CAS session if it does not exists instead of using proc python to create the CAS session and then have SWAT connect to it.

The custom step already reuses a CAS session if one existed on the Compute session. However, when the CAS session does not exist the step creates the compute session from SWAT directly. Not sure why it does that instead of having "proc python" create the CAS session. When Compute knows about the CAS session and libnames are created that point to caslibs, the Studio UI can display a list of CAS tables and you can use those tables in Studio Flow. Here's a code snippet for creating a CAS session from proc python and then defining a libname that points to a caslib:

// Start CAS session and point SAS libname to a caslib from proc python
SAS.submit("cas")
SAS.submit("libname casdata cas caslib='...'")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new custom step This pull request represents a new custom step
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants