Skip to content

remove editor mentions#44

Open
SFJohnson24 wants to merge 4 commits into
mainfrom
readme
Open

remove editor mentions#44
SFJohnson24 wants to merge 4 commits into
mainfrom
readme

Conversation

@SFJohnson24
Copy link
Copy Markdown
Collaborator

No description provided.

@SFJohnson24 SFJohnson24 linked an issue Jun 1, 2026 that may be closed by this pull request
Comment thread docs/faq.md

If you provide a dataset AE where both AESEV and AESHOSP have empty values in a record, this rule will only flag a single error for that record, but it will display the empty values for both variables.

</details>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This belongs to previous tag is removed. Should we restore it?

Comment thread docs/faq.md
My test data dates are being treated as numbers or the engine is returning errors indicating that my dates cannot be parsed.
</summary>

When using Excel for data, one of the tradeoffs is that it tries to autoformat. If Excel recognizes a date, it will convert it to a numeric representation of the date and display it formatted. When the Rule Editor loads the date cell, it loads the number instead of the original text string date. You can confirm this by viewing the loaded test data section in the “Load Datasets” panel of the Rule Editor and checking if the date values are numbers instead of character strings.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This FAQ is still valid as excel dataset is supported with rules engine. Maybe we can just remove the mention of editor instead of the whole FAQ?

Comment thread docs/dev/doc.yuml
[UI]-.->[Rule YAML Editor]
[UI]-.->[Rule Test]
[Rule YAML Editor]<->[cdisc-rules-engine-repo]
[Rule YAML Editor]<->[cdisc-open-rules-repo]
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Do you think the name in the bracket also change? as I guess it is not an editor anymore.

Comment thread docs/README.md
### Schema for Rule Schematic

Intellisense is informed by a schema in JSON format. The editor also uses it to provide suggestions.
Intellisense is informed by a schema in JSON format.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is the mention of intellisense still valid as their is no rules editor anymore? Please let me know if I am misunderstanding something?

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.

Linter issues

2 participants