insightsengineering / insightsengineering/teal.goshawk

"Select a Biomarker" menu item construction.

Open
#239 2 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
R
Stars
3
Forks
3
Avg merge
2d 6h
Merged PRs (30d)
11

Description

### Feature description

1. seems that there is a validation that the analysis variable has to be PARAMCD but PARAM would work equally well. Can PARAM be added to the validation so that it too is acceptable as this argument? param_var = "PARAM"?
2. If above is acceptable then we need to deal with the menu item sorting. currently the sort looks like a regular case sensitive sort by ASCII collating sequence so upper case sorts first then lower case. but what we want is a case insensitive sort like in the following example
```
# Sample vector
my_vector <- c("apple", "Banana", "cherry", "Date")

# Perform case-insensitive sorting
sorted_vector <- my_vector[order(tolower(my_vector))]

# Display the sorted vector
print(sorted_vector)

```
### Code of Conduct

- [X] I agree to follow this project's Code of Conduct.

### Contribution Guidelines

- [X] I agree to follow this project's Contribution Guidelines.

### Security Policy

- [X] I agree to follow this project's Security Policy.

Contributor guide

Open the contributing guide

Research direction

Locate the code that constructs the "Select a Biomarker" menu and validates the analysis variable. Check how PARAMCD is currently accepted, then verify that PARAM is also valid and that menu entries sort case-insensitively. Done means both variable names work and the menu ordering matches the requested behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
frontend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.