Skip to content

publishing#18588

Merged
MichaelCzcz merged 4 commits intosap-tutorials:masterfrom
helenaaaaaaaaaa:master
Jul 5, 2022
Merged

publishing#18588
MichaelCzcz merged 4 commits intosap-tutorials:masterfrom
helenaaaaaaaaaa:master

Conversation

@helenaaaaaaaaaa
Copy link
Copy Markdown
Contributor

I have made some further changes and clarifications.

publishing
@helenaaaaaaaaaa helenaaaaaaaaaa requested a review from jmmargo July 5, 2022 09:21
@helenaaaaaaaaaa
Copy link
Copy Markdown
Contributor Author

@LunaticMaestro FYI


[OPTION BEGIN [Postman]]

Click through `AI Core` > `lm` > `metrics` > `Get metrics` and double check the `executionId`.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think that saying "Navigate to" sounds better than "Click through".

)
```

The parameter `value="housepricemodel"` refers to the artifact name, which references the model that will be stored in AWS S3. It is vital that the name of this parameter matches the name that you defined in your yaml workflow.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

YAML workflow

[OPTION BEGIN [SAP AI Launchpad]]

Click on the `Metrics Resource` tab of your execution.
Click through `ML Operations` > `Executions` > `Metrics Resource` tab of your execution.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think that saying "Navigate to" sounds better than "Click through".

@helenaaaaaaaaaa
Copy link
Copy Markdown
Contributor Author

I have made the requested changes

@MichaelCzcz MichaelCzcz merged commit 9c9f588 into sap-tutorials:master Jul 5, 2022
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.

2 participants