Skip to content

Conversation

@d33bs
Copy link
Member

@d33bs d33bs commented Oct 9, 2025

No description provided.

Co-Authored-By: Faisal Alquaddoomi <faisal.alquaddoomi@cuanschutz.edu>
@github-actions
Copy link

github-actions bot commented Oct 9, 2025

PR Preview Action v1.4.7
🚀 Deployed preview to https://CU-DBMI.github.io/set-website/preview/pr-48/
on branch gh-pages at 2025-10-10 22:16 UTC

@d33bs d33bs marked this pull request as ready for review October 9, 2025 17:46
about/index.md Outdated

If you'd like to attribute work to the Software Engineering Team, please see below for an example of how this might look.

> This work was conducted in collaboration with the SET (Software Engineering Team), a core support team under the Department of Biomedical Informatics (DBMI) at CU Anschutz. The SET are dedicated software developers in the DBMI at CU Anschutz. SET supports DBMI labs, faculty, and staff, as well as external groups throughout the University. SET collaborates with research and administrative groups to translate ideas or prototypes into sustainable high-quality software. SET also provides software engineering education as a service to the institution in the form of code review, presentations, blog posts, interactive workshops, mentorships, and other forms of outreach.
Copy link

@the-mayer the-mayer Oct 9, 2025

Choose a reason for hiding this comment

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

Can we take this opportunity to streamline the "official language" found in the facilities resource generator? Thinking:

This work was conducted in collaboration with the Software Engineering Team (SET), a core support team of dedicated developers within the Department of Biomedical Informatics (DBMI) at CU Anschutz.... etc

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks - good thinking! I'm not sure about how this works exactly within the facilities resource generator, it seems to be all combined without breaks between different groups.
image
@falquaddoomi, would you have any opinions here?

Choose a reason for hiding this comment

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

Oh, I hadn't actually thought about contributing back to the generator... but that is a good idea, if we can. I was just advocating for re-arranging that first couple of sentences for our website, which seemed a bit "wordy."

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks! I better understand now, I've added an update here.

Choose a reason for hiding this comment

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

@d33bs Sorry for the delay! So, yes, the resource text for DBMI unfortunately appears to have multiple resources listed in it. The original text is from an internal spreadsheet that I wasn't involved in putting together and was used as-is for the initial population of the database. Right now the system is still a prototype and we don't yet have a formal process yet for creating/updating resources, but I can add/update things manually if you'd like.

That said, you did identify a bug on the resource report page, specifically that it wasn't showing markup or breaking on newlines in the original text. I've since fixed it and updated the site. Thanks for that!

Copy link
Member Author

Choose a reason for hiding this comment

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

No worries and thanks @falquaddoomi, appreciate the update!

Co-Authored-By: D Mayer <32186106+the-mayer@users.noreply.github.com>
@d33bs
Copy link
Member Author

d33bs commented Oct 27, 2025

Merging this in, thanks again @the-mayer!

@d33bs d33bs merged commit 2476f72 into CU-DBMI:main Oct 27, 2025
2 checks passed
@d33bs d33bs deleted the attributions branch October 27, 2025 11:17
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.

3 participants