Skip to content

NOISSUE- Update computation documentation#116

Merged
drasko merged 8 commits intoultravioletrs:mainfrom
dorcaslitunya:dorcaslitunya/update-computations
Nov 21, 2025
Merged

NOISSUE- Update computation documentation#116
drasko merged 8 commits intoultravioletrs:mainfrom
dorcaslitunya:dorcaslitunya/update-computations

Conversation

@dorcaslitunya
Copy link
Copy Markdown
Contributor

What type of PR is this?

This is a documentation update because it updates the following documentation: Computations and bulk computations

What does this do?

Updates computations with a more recent manifest reflection

Which issue(s) does this PR fix/relate to?

Have you included tests for your changes?

No, I have not included tests because its a docs change

Did you document any new/modified features?

Notes

@dorcaslitunya dorcaslitunya force-pushed the dorcaslitunya/update-computations branch from e48dc83 to 4117891 Compare November 11, 2025 10:20
@dorcaslitunya dorcaslitunya force-pushed the dorcaslitunya/update-computations branch from 4117891 to 01cd286 Compare November 11, 2025 10:30
Comment thread docs/computations.md Outdated
Comment thread docs/computations.md Outdated
Comment thread docs/getting-started.md
Copy link
Copy Markdown
Contributor

@SammyOina SammyOina left a comment

Choose a reason for hiding this comment

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

please go through the flow and make sure commands appear where needed. this is meant for first time users and can contain as much and exact details as in https://docs.cocos.ultraviolet.rs/getting-started. you can tailor the getting started doc to a specific example algo

Comment thread docs/getting-started.md Outdated
Comment thread docs/getting-started.md
@dorcaslitunya dorcaslitunya force-pushed the dorcaslitunya/update-computations branch from 9c52788 to e9088a3 Compare November 17, 2025 18:04
Copy link
Copy Markdown
Contributor

@SammyOina SammyOina left a comment

Choose a reason for hiding this comment

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

make sure getting started is linear as the user will be interating with it exactly. you should use a specific example algo we have for demonstration

Comment thread docs/getting-started.md Outdated
Comment thread docs/getting-started.md Outdated
Comment thread docs/getting-started.md Outdated
Comment thread docs/getting-started.md Outdated
Copy link
Copy Markdown
Contributor

@SammyOina SammyOina left a comment

Choose a reason for hiding this comment

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

make sure getting started is linear as the user will be interating with it exactly. you should use a specific example algo we have for demonstration

@dorcaslitunya dorcaslitunya force-pushed the dorcaslitunya/update-computations branch from 35770e3 to be1308e Compare November 19, 2025 16:08
Comment thread docs/getting-started.md Outdated
Comment thread docs/getting-started.md Outdated
Comment thread docs/getting-started.md Outdated
@fbugarski
Copy link
Copy Markdown
Contributor

image This is not aligned.

Comment thread docs/getting-started.md
Comment thread docs/getting-started.md
Comment thread docs/getting-started.md
2. Click **Link to Computation**
3. Search and select the target computation
4. Confirm the association
### Create Dataset Asset
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The dataset used in this section is incorrect: iris.csv cannot be used with the addition demo. This demo should use the iris algorithm together with the iris dataset, as the current ‘simple addition’ description no longer applies.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The dataset is iris and the algo is Lin_reg..unless I am missing sth?

Comment thread docs/getting-started.md Outdated
@drasko drasko merged commit 392bb69 into ultravioletrs:main Nov 21, 2025
1 check passed
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.

5 participants