#67 / File Metadata Table by demariadaniel · Pull Request #242 · overture-stack/stage

demariadaniel · 2025-05-28T13:48:42Z

Summary

Adds File Metadata Table to Bam Visualizer page

Issues

67

Description of Changes

Adds File Metadata Table component
Moves BamTable components into separate folder
Page reorganization (Loading, Context changes)

Readiness Checklist

Self Review
- I have performed a self review of code
- I have run the application locally and manually tested the feature
- I have checked all updates to correct typos and misspellings
Formatting
- Code follows the project style guide
- Autmated code formatters (ie. Prettier) have been run
Local Testing
- Successfully built all packages locally
- Successfully ran all test suites, all unit and integration tests pass
Updated Tests
- Unit and integration tests have been added that describe the bug that was fixed or the features that were added
Documentation
- All new environment variables added to .env.schema file and documented in the README
- All changes to server HTTP endpoints have open-api documentation
- All new functions exported from their module have TSDoc comment documentation

joneubank · 2025-05-28T19:08:49Z

+const getTableData = (file: FileTableData) => {
+	const fileAccess = file?.file_access;
+	const fileDataType = file?.data_type;
+	const fileDonorId = file?.donors?.hits.edges[0].node.submitter_donor_id;


I am concerned that with the new song data model, it is very unlikely that each file has a donor... this is no longer a required property in song analyses. This looks like it will work as is (if not present, just returns undefined) but I do wonder if we should include this in our standard table data.

joneubank · 2025-05-28T19:09:54Z

+	const fileStudy = file?.analysis?.collaborator.hits.edges[0].node.name;
+	const fileStrategy = file?.analysis?.experiment.experimentalStrategy;


Same comment as with donors, collaborator and experiment are not part of the required analysis model now.
I think study still is, I don't know which data model is using colalborator instead of studyId.

Added .? to experiments & collaborators in 0256b61

joneubank · 2025-05-28T19:10:14Z

+	const fileSize =
+ numFileSize < 10 ** 3
+ ? `${numFileSize} bytes`
+ : numFileSize < 10 ** 6
+ ? `${(numFileSize / 10 ** 3).toFixed(2)} KB`
+ : numFileSize < 10 ** 9
+ ? `${(numFileSize / 10 ** 6).toFixed(2)} MB`
+ : numFileSize < 10 ** 12
+ ? `${(numFileSize / 10 ** 9).toFixed(2)} GB`
+ : `${(numFileSize / 10 ** 12).toFixed(2)} TB`;


Probably extract this to a reusable function.

Started a table utilities file w/ a function definition for this

joneubank · 2025-05-28T19:11:33Z

+export const StatsTable = ({ file, fileId = '' }: { file: FileTableData; fileId?: string }) => {
+	const [showTable, setShowTable] = useState(true);
+	const { selectedRows } = useTableContext({ callerName: 'File Metadata Table' });
+	const theme: typeof defaultTheme = useTheme();


Can we infer the type here instead of declaring it? This way it will correctly reflect the type returned by useTheme()

Updated in 0256b61

joneubank · 2025-05-28T19:24:13Z

 };

-export type FileTableData = { id: string; file_type?: string; file: { size: number } };
+export type FileTableData = {


This query will definitely break when the new song model is indexed, arranger won't accept the collaborator/experiment/donors properties. Happy to discuss this with you and Brandon to review the requirements to latest Overture data state.

Updated in 0256b61 and opened thread to discuss in Slack

demariadaniel · 2025-05-28T13:48:42Z

Summary

Adds File Metadata Table to Bam Visualizer page

Issues

67

Description of Changes

Adds File Metadata Table component
Moves BamTable components into separate folder
Page reorganization (Loading, Context changes)

Readiness Checklist

Self Review
- I have performed a self review of code
- I have run the application locally and manually tested the feature
- I have checked all updates to correct typos and misspellings
Formatting
- Code follows the project style guide
- Autmated code formatters (ie. Prettier) have been run
Local Testing
- Successfully built all packages locally
- Successfully ran all test suites, all unit and integration tests pass
Updated Tests
- Unit and integration tests have been added that describe the bug that was fixed or the features that were added
Documentation
- All new environment variables added to .env.schema file and documented in the README
- All changes to server HTTP endpoints have open-api documentation
- All new functions exported from their module have TSDoc comment documentation

joneubank · 2025-05-28T19:08:49Z

+const getTableData = (file: FileTableData) => {
+	const fileAccess = file?.file_access;
+	const fileDataType = file?.data_type;
+	const fileDonorId = file?.donors?.hits.edges[0].node.submitter_donor_id;


I am concerned that with the new song data model, it is very unlikely that each file has a donor... this is no longer a required property in song analyses. This looks like it will work as is (if not present, just returns undefined) but I do wonder if we should include this in our standard table data.

joneubank · 2025-05-28T19:09:54Z

+	const fileStudy = file?.analysis?.collaborator.hits.edges[0].node.name;
+	const fileStrategy = file?.analysis?.experiment.experimentalStrategy;


Same comment as with donors, collaborator and experiment are not part of the required analysis model now.
I think study still is, I don't know which data model is using colalborator instead of studyId.

Added .? to experiments & collaborators in 0256b61

joneubank · 2025-05-28T19:10:14Z

+	const fileSize =
+ numFileSize < 10 ** 3
+ ? `${numFileSize} bytes`
+ : numFileSize < 10 ** 6
+ ? `${(numFileSize / 10 ** 3).toFixed(2)} KB`
+ : numFileSize < 10 ** 9
+ ? `${(numFileSize / 10 ** 6).toFixed(2)} MB`
+ : numFileSize < 10 ** 12
+ ? `${(numFileSize / 10 ** 9).toFixed(2)} GB`
+ : `${(numFileSize / 10 ** 12).toFixed(2)} TB`;


Probably extract this to a reusable function.

Started a table utilities file w/ a function definition for this

joneubank · 2025-05-28T19:11:33Z

+export const StatsTable = ({ file, fileId = '' }: { file: FileTableData; fileId?: string }) => {
+	const [showTable, setShowTable] = useState(true);
+	const { selectedRows } = useTableContext({ callerName: 'File Metadata Table' });
+	const theme: typeof defaultTheme = useTheme();


Can we infer the type here instead of declaring it? This way it will correctly reflect the type returned by useTheme()

Updated in 0256b61

joneubank · 2025-05-28T19:24:13Z

 };

-export type FileTableData = { id: string; file_type?: string; file: { size: number } };
+export type FileTableData = {


This query will definitely break when the new song model is indexed, arranger won't accept the collaborator/experiment/donors properties. Happy to discuss this with you and Brandon to review the requirements to latest Overture data state.

Updated in 0256b61 and opened thread to discuss in Slack

demariadaniel · 2025-05-28T13:48:42Z

Summary

Adds File Metadata Table to Bam Visualizer page

Issues

67

Description of Changes

Adds File Metadata Table component
Moves BamTable components into separate folder
Page reorganization (Loading, Context changes)

Readiness Checklist

Self Review
- I have performed a self review of code
- I have run the application locally and manually tested the feature
- I have checked all updates to correct typos and misspellings
Formatting
- Code follows the project style guide
- Autmated code formatters (ie. Prettier) have been run
Local Testing
- Successfully built all packages locally
- Successfully ran all test suites, all unit and integration tests pass
Updated Tests
- Unit and integration tests have been added that describe the bug that was fixed or the features that were added
Documentation
- All new environment variables added to .env.schema file and documented in the README
- All changes to server HTTP endpoints have open-api documentation
- All new functions exported from their module have TSDoc comment documentation

joneubank · 2025-05-28T19:08:49Z

+const getTableData = (file: FileTableData) => {
+	const fileAccess = file?.file_access;
+	const fileDataType = file?.data_type;
+	const fileDonorId = file?.donors?.hits.edges[0].node.submitter_donor_id;


I am concerned that with the new song data model, it is very unlikely that each file has a donor... this is no longer a required property in song analyses. This looks like it will work as is (if not present, just returns undefined) but I do wonder if we should include this in our standard table data.

joneubank · 2025-05-28T19:09:54Z

+	const fileStudy = file?.analysis?.collaborator.hits.edges[0].node.name;
+	const fileStrategy = file?.analysis?.experiment.experimentalStrategy;


Same comment as with donors, collaborator and experiment are not part of the required analysis model now.
I think study still is, I don't know which data model is using colalborator instead of studyId.

Added .? to experiments & collaborators in 0256b61

joneubank · 2025-05-28T19:10:14Z

+	const fileSize =
+ numFileSize < 10 ** 3
+ ? `${numFileSize} bytes`
+ : numFileSize < 10 ** 6
+ ? `${(numFileSize / 10 ** 3).toFixed(2)} KB`
+ : numFileSize < 10 ** 9
+ ? `${(numFileSize / 10 ** 6).toFixed(2)} MB`
+ : numFileSize < 10 ** 12
+ ? `${(numFileSize / 10 ** 9).toFixed(2)} GB`
+ : `${(numFileSize / 10 ** 12).toFixed(2)} TB`;


Probably extract this to a reusable function.

Started a table utilities file w/ a function definition for this

joneubank · 2025-05-28T19:11:33Z

+export const StatsTable = ({ file, fileId = '' }: { file: FileTableData; fileId?: string }) => {
+	const [showTable, setShowTable] = useState(true);
+	const { selectedRows } = useTableContext({ callerName: 'File Metadata Table' });
+	const theme: typeof defaultTheme = useTheme();


Can we infer the type here instead of declaring it? This way it will correctly reflect the type returned by useTheme()

Updated in 0256b61

joneubank · 2025-05-28T19:24:13Z

 };

-export type FileTableData = { id: string; file_type?: string; file: { size: number } };
+export type FileTableData = {


This query will definitely break when the new song model is indexed, arranger won't accept the collaborator/experiment/donors properties. Happy to discuss this with you and Brandon to review the requirements to latest Overture data state.

Updated in 0256b61 and opened thread to discuss in Slack

demariadaniel · 2025-05-28T13:48:42Z

Summary

Adds File Metadata Table to Bam Visualizer page

Issues

67

Description of Changes

Adds File Metadata Table component
Moves BamTable components into separate folder
Page reorganization (Loading, Context changes)

Readiness Checklist

Self Review
- I have performed a self review of code
- I have run the application locally and manually tested the feature
- I have checked all updates to correct typos and misspellings
Formatting
- Code follows the project style guide
- Autmated code formatters (ie. Prettier) have been run
Local Testing
- Successfully built all packages locally
- Successfully ran all test suites, all unit and integration tests pass
Updated Tests
- Unit and integration tests have been added that describe the bug that was fixed or the features that were added
Documentation
- All new environment variables added to .env.schema file and documented in the README
- All changes to server HTTP endpoints have open-api documentation
- All new functions exported from their module have TSDoc comment documentation

joneubank · 2025-05-28T19:08:49Z

+const getTableData = (file: FileTableData) => {
+	const fileAccess = file?.file_access;
+	const fileDataType = file?.data_type;
+	const fileDonorId = file?.donors?.hits.edges[0].node.submitter_donor_id;


I am concerned that with the new song data model, it is very unlikely that each file has a donor... this is no longer a required property in song analyses. This looks like it will work as is (if not present, just returns undefined) but I do wonder if we should include this in our standard table data.

joneubank · 2025-05-28T19:09:54Z

+	const fileStudy = file?.analysis?.collaborator.hits.edges[0].node.name;
+	const fileStrategy = file?.analysis?.experiment.experimentalStrategy;


Same comment as with donors, collaborator and experiment are not part of the required analysis model now.
I think study still is, I don't know which data model is using colalborator instead of studyId.

Added .? to experiments & collaborators in 0256b61

joneubank · 2025-05-28T19:10:14Z

+	const fileSize =
+ numFileSize < 10 ** 3
+ ? `${numFileSize} bytes`
+ : numFileSize < 10 ** 6
+ ? `${(numFileSize / 10 ** 3).toFixed(2)} KB`
+ : numFileSize < 10 ** 9
+ ? `${(numFileSize / 10 ** 6).toFixed(2)} MB`
+ : numFileSize < 10 ** 12
+ ? `${(numFileSize / 10 ** 9).toFixed(2)} GB`
+ : `${(numFileSize / 10 ** 12).toFixed(2)} TB`;


Probably extract this to a reusable function.

Started a table utilities file w/ a function definition for this

joneubank · 2025-05-28T19:11:33Z

+export const StatsTable = ({ file, fileId = '' }: { file: FileTableData; fileId?: string }) => {
+	const [showTable, setShowTable] = useState(true);
+	const { selectedRows } = useTableContext({ callerName: 'File Metadata Table' });
+	const theme: typeof defaultTheme = useTheme();


Can we infer the type here instead of declaring it? This way it will correctly reflect the type returned by useTheme()

Updated in 0256b61

joneubank · 2025-05-28T19:24:13Z

 };

-export type FileTableData = { id: string; file_type?: string; file: { size: number } };
+export type FileTableData = {


This query will definitely break when the new song model is indexed, arranger won't accept the collaborator/experiment/donors properties. Happy to discuss this with you and Brandon to review the requirements to latest Overture data state.

Updated in 0256b61 and opened thread to discuss in Slack

demariadaniel · 2025-05-28T13:48:42Z

Summary

Adds File Metadata Table to Bam Visualizer page

Issues

67

Description of Changes

Adds File Metadata Table component
Moves BamTable components into separate folder
Page reorganization (Loading, Context changes)

Readiness Checklist

Self Review
- I have performed a self review of code
- I have run the application locally and manually tested the feature
- I have checked all updates to correct typos and misspellings
Formatting
- Code follows the project style guide
- Autmated code formatters (ie. Prettier) have been run
Local Testing
- Successfully built all packages locally
- Successfully ran all test suites, all unit and integration tests pass
Updated Tests
- Unit and integration tests have been added that describe the bug that was fixed or the features that were added
Documentation
- All new environment variables added to .env.schema file and documented in the README
- All changes to server HTTP endpoints have open-api documentation
- All new functions exported from their module have TSDoc comment documentation

joneubank · 2025-05-28T19:08:49Z

+const getTableData = (file: FileTableData) => {
+	const fileAccess = file?.file_access;
+	const fileDataType = file?.data_type;
+	const fileDonorId = file?.donors?.hits.edges[0].node.submitter_donor_id;


I am concerned that with the new song data model, it is very unlikely that each file has a donor... this is no longer a required property in song analyses. This looks like it will work as is (if not present, just returns undefined) but I do wonder if we should include this in our standard table data.

joneubank · 2025-05-28T19:09:54Z

+	const fileStudy = file?.analysis?.collaborator.hits.edges[0].node.name;
+	const fileStrategy = file?.analysis?.experiment.experimentalStrategy;


Same comment as with donors, collaborator and experiment are not part of the required analysis model now.
I think study still is, I don't know which data model is using colalborator instead of studyId.

Added .? to experiments & collaborators in 0256b61

joneubank · 2025-05-28T19:10:14Z

+	const fileSize =
+ numFileSize < 10 ** 3
+ ? `${numFileSize} bytes`
+ : numFileSize < 10 ** 6
+ ? `${(numFileSize / 10 ** 3).toFixed(2)} KB`
+ : numFileSize < 10 ** 9
+ ? `${(numFileSize / 10 ** 6).toFixed(2)} MB`
+ : numFileSize < 10 ** 12
+ ? `${(numFileSize / 10 ** 9).toFixed(2)} GB`
+ : `${(numFileSize / 10 ** 12).toFixed(2)} TB`;


Probably extract this to a reusable function.

Started a table utilities file w/ a function definition for this

joneubank · 2025-05-28T19:11:33Z

+export const StatsTable = ({ file, fileId = '' }: { file: FileTableData; fileId?: string }) => {
+	const [showTable, setShowTable] = useState(true);
+	const { selectedRows } = useTableContext({ callerName: 'File Metadata Table' });
+	const theme: typeof defaultTheme = useTheme();


Can we infer the type here instead of declaring it? This way it will correctly reflect the type returned by useTheme()

Updated in 0256b61

joneubank · 2025-05-28T19:24:13Z

 };

-export type FileTableData = { id: string; file_type?: string; file: { size: number } };
+export type FileTableData = {


This query will definitely break when the new song model is indexed, arranger won't accept the collaborator/experiment/donors properties. Happy to discuss this with you and Brandon to review the requirements to latest Overture data state.

Updated in 0256b61 and opened thread to discuss in Slack

demariadaniel · 2025-05-28T13:48:42Z

Summary

Adds File Metadata Table to Bam Visualizer page

Issues

67

Description of Changes

Adds File Metadata Table component
Moves BamTable components into separate folder
Page reorganization (Loading, Context changes)

Readiness Checklist

Self Review
- I have performed a self review of code
- I have run the application locally and manually tested the feature
- I have checked all updates to correct typos and misspellings
Formatting
- Code follows the project style guide
- Autmated code formatters (ie. Prettier) have been run
Local Testing
- Successfully built all packages locally
- Successfully ran all test suites, all unit and integration tests pass
Updated Tests
- Unit and integration tests have been added that describe the bug that was fixed or the features that were added
Documentation
- All new environment variables added to .env.schema file and documented in the README
- All changes to server HTTP endpoints have open-api documentation
- All new functions exported from their module have TSDoc comment documentation

joneubank · 2025-05-28T19:08:49Z

+const getTableData = (file: FileTableData) => {
+	const fileAccess = file?.file_access;
+	const fileDataType = file?.data_type;
+	const fileDonorId = file?.donors?.hits.edges[0].node.submitter_donor_id;


I am concerned that with the new song data model, it is very unlikely that each file has a donor... this is no longer a required property in song analyses. This looks like it will work as is (if not present, just returns undefined) but I do wonder if we should include this in our standard table data.

joneubank · 2025-05-28T19:09:54Z

+	const fileStudy = file?.analysis?.collaborator.hits.edges[0].node.name;
+	const fileStrategy = file?.analysis?.experiment.experimentalStrategy;


Same comment as with donors, collaborator and experiment are not part of the required analysis model now.
I think study still is, I don't know which data model is using colalborator instead of studyId.

Added .? to experiments & collaborators in 0256b61

joneubank · 2025-05-28T19:10:14Z

+	const fileSize =
+ numFileSize < 10 ** 3
+ ? `${numFileSize} bytes`
+ : numFileSize < 10 ** 6
+ ? `${(numFileSize / 10 ** 3).toFixed(2)} KB`
+ : numFileSize < 10 ** 9
+ ? `${(numFileSize / 10 ** 6).toFixed(2)} MB`
+ : numFileSize < 10 ** 12
+ ? `${(numFileSize / 10 ** 9).toFixed(2)} GB`
+ : `${(numFileSize / 10 ** 12).toFixed(2)} TB`;


Probably extract this to a reusable function.

Started a table utilities file w/ a function definition for this

joneubank · 2025-05-28T19:11:33Z

+export const StatsTable = ({ file, fileId = '' }: { file: FileTableData; fileId?: string }) => {
+	const [showTable, setShowTable] = useState(true);
+	const { selectedRows } = useTableContext({ callerName: 'File Metadata Table' });
+	const theme: typeof defaultTheme = useTheme();


Can we infer the type here instead of declaring it? This way it will correctly reflect the type returned by useTheme()

Updated in 0256b61

joneubank · 2025-05-28T19:24:13Z

 };

-export type FileTableData = { id: string; file_type?: string; file: { size: number } };
+export type FileTableData = {


This query will definitely break when the new song model is indexed, arranger won't accept the collaborator/experiment/donors properties. Happy to discuss this with you and Brandon to review the requirements to latest Overture data state.

Updated in 0256b61 and opened thread to discuss in Slack

demariadaniel · 2025-05-28T13:48:42Z

Summary

Adds File Metadata Table to Bam Visualizer page

Issues

67

Description of Changes

Adds File Metadata Table component
Moves BamTable components into separate folder
Page reorganization (Loading, Context changes)

Readiness Checklist

Self Review
- I have performed a self review of code
- I have run the application locally and manually tested the feature
- I have checked all updates to correct typos and misspellings
Formatting
- Code follows the project style guide
- Autmated code formatters (ie. Prettier) have been run
Local Testing
- Successfully built all packages locally
- Successfully ran all test suites, all unit and integration tests pass
Updated Tests
- Unit and integration tests have been added that describe the bug that was fixed or the features that were added
Documentation
- All new environment variables added to .env.schema file and documented in the README
- All changes to server HTTP endpoints have open-api documentation
- All new functions exported from their module have TSDoc comment documentation

joneubank · 2025-05-28T19:08:49Z

+const getTableData = (file: FileTableData) => {
+	const fileAccess = file?.file_access;
+	const fileDataType = file?.data_type;
+	const fileDonorId = file?.donors?.hits.edges[0].node.submitter_donor_id;


I am concerned that with the new song data model, it is very unlikely that each file has a donor... this is no longer a required property in song analyses. This looks like it will work as is (if not present, just returns undefined) but I do wonder if we should include this in our standard table data.

joneubank · 2025-05-28T19:09:54Z

+	const fileStudy = file?.analysis?.collaborator.hits.edges[0].node.name;
+	const fileStrategy = file?.analysis?.experiment.experimentalStrategy;


Same comment as with donors, collaborator and experiment are not part of the required analysis model now.
I think study still is, I don't know which data model is using colalborator instead of studyId.

Added .? to experiments & collaborators in 0256b61

joneubank · 2025-05-28T19:10:14Z

+	const fileSize =
+ numFileSize < 10 ** 3
+ ? `${numFileSize} bytes`
+ : numFileSize < 10 ** 6
+ ? `${(numFileSize / 10 ** 3).toFixed(2)} KB`
+ : numFileSize < 10 ** 9
+ ? `${(numFileSize / 10 ** 6).toFixed(2)} MB`
+ : numFileSize < 10 ** 12
+ ? `${(numFileSize / 10 ** 9).toFixed(2)} GB`
+ : `${(numFileSize / 10 ** 12).toFixed(2)} TB`;


Probably extract this to a reusable function.

Started a table utilities file w/ a function definition for this

joneubank · 2025-05-28T19:11:33Z

+export const StatsTable = ({ file, fileId = '' }: { file: FileTableData; fileId?: string }) => {
+	const [showTable, setShowTable] = useState(true);
+	const { selectedRows } = useTableContext({ callerName: 'File Metadata Table' });
+	const theme: typeof defaultTheme = useTheme();


Can we infer the type here instead of declaring it? This way it will correctly reflect the type returned by useTheme()

Updated in 0256b61

joneubank · 2025-05-28T19:24:13Z

 };

-export type FileTableData = { id: string; file_type?: string; file: { size: number } };
+export type FileTableData = {


This query will definitely break when the new song model is indexed, arranger won't accept the collaborator/experiment/donors properties. Happy to discuss this with you and Brandon to review the requirements to latest Overture data state.

Updated in 0256b61 and opened thread to discuss in Slack

demariadaniel · 2025-05-28T13:48:42Z

Summary

Adds File Metadata Table to Bam Visualizer page

Issues

67

Description of Changes

Adds File Metadata Table component
Moves BamTable components into separate folder
Page reorganization (Loading, Context changes)

Readiness Checklist

Self Review
- I have performed a self review of code
- I have run the application locally and manually tested the feature
- I have checked all updates to correct typos and misspellings
Formatting
- Code follows the project style guide
- Autmated code formatters (ie. Prettier) have been run
Local Testing
- Successfully built all packages locally
- Successfully ran all test suites, all unit and integration tests pass
Updated Tests
- Unit and integration tests have been added that describe the bug that was fixed or the features that were added
Documentation
- All new environment variables added to .env.schema file and documented in the README
- All changes to server HTTP endpoints have open-api documentation
- All new functions exported from their module have TSDoc comment documentation

joneubank · 2025-05-28T19:08:49Z

+const getTableData = (file: FileTableData) => {
+	const fileAccess = file?.file_access;
+	const fileDataType = file?.data_type;
+	const fileDonorId = file?.donors?.hits.edges[0].node.submitter_donor_id;


I am concerned that with the new song data model, it is very unlikely that each file has a donor... this is no longer a required property in song analyses. This looks like it will work as is (if not present, just returns undefined) but I do wonder if we should include this in our standard table data.

joneubank · 2025-05-28T19:09:54Z

+	const fileStudy = file?.analysis?.collaborator.hits.edges[0].node.name;
+	const fileStrategy = file?.analysis?.experiment.experimentalStrategy;


Same comment as with donors, collaborator and experiment are not part of the required analysis model now.
I think study still is, I don't know which data model is using colalborator instead of studyId.

Added .? to experiments & collaborators in 0256b61

joneubank · 2025-05-28T19:10:14Z

+	const fileSize =
+ numFileSize < 10 ** 3
+ ? `${numFileSize} bytes`
+ : numFileSize < 10 ** 6
+ ? `${(numFileSize / 10 ** 3).toFixed(2)} KB`
+ : numFileSize < 10 ** 9
+ ? `${(numFileSize / 10 ** 6).toFixed(2)} MB`
+ : numFileSize < 10 ** 12
+ ? `${(numFileSize / 10 ** 9).toFixed(2)} GB`
+ : `${(numFileSize / 10 ** 12).toFixed(2)} TB`;


Probably extract this to a reusable function.

Started a table utilities file w/ a function definition for this

joneubank · 2025-05-28T19:11:33Z

+export const StatsTable = ({ file, fileId = '' }: { file: FileTableData; fileId?: string }) => {
+	const [showTable, setShowTable] = useState(true);
+	const { selectedRows } = useTableContext({ callerName: 'File Metadata Table' });
+	const theme: typeof defaultTheme = useTheme();


Can we infer the type here instead of declaring it? This way it will correctly reflect the type returned by useTheme()

Updated in 0256b61

joneubank · 2025-05-28T19:24:13Z

 };

-export type FileTableData = { id: string; file_type?: string; file: { size: number } };
+export type FileTableData = {


This query will definitely break when the new song model is indexed, arranger won't accept the collaborator/experiment/donors properties. Happy to discuss this with you and Brandon to review the requirements to latest Overture data state.

Updated in 0256b61 and opened thread to discuss in Slack

demariadaniel added 6 commits May 26, 2025 16:57

Stub Stats Table, separate files
9ea1f75

Very Basic Table
2b4b11b

Updated file table properties
0efd619

Improved StatsTable CSS
88171c7

Reorg components, use theme colors, add selected count
eadea46

Show / Hide Button
35e343b

demariadaniel self-assigned this May 28, 2025

demariadaniel added 5 commits May 28, 2025 10:18

Use span -/+ icon
d00af7c

Unused DemoStats
5391547

1px adjustment, Remove nested span CSS
8f75641

Header padding
41fc4d2

File Size Label, get data function
84607d5

demariadaniel marked this pull request as ready for review May 28, 2025 17:32

demariadaniel requested review from ciaranschutte, joneubank, justincorrigible and leoraba May 28, 2025 17:32

demariadaniel changed the title ~~WIP: #67 / File Metadata Table~~#67 / File Metadata TableMay 28, 2025

Rename function
3adeed1

joneubank approved these changes May 28, 2025
View reviewed changes

PR feedback, tableUtil file
0256b61

demariadaniel merged commit 571a8a0 into iobioJun 2, 2025

demariadaniel deleted the 67/feat-file-metadata-table branch June 2, 2025 21:23

demariadaniel added 6 commits May 26, 2025 16:57

Stub Stats Table, separate files
9ea1f75

Very Basic Table
2b4b11b

Updated file table properties
0efd619

Improved StatsTable CSS
88171c7

Reorg components, use theme colors, add selected count
eadea46

Show / Hide Button
35e343b

demariadaniel self-assigned this May 28, 2025

demariadaniel added 5 commits May 28, 2025 10:18

Use span -/+ icon
d00af7c

Unused DemoStats
5391547

1px adjustment, Remove nested span CSS
8f75641

Header padding
41fc4d2

File Size Label, get data function
84607d5

demariadaniel marked this pull request as ready for review May 28, 2025 17:32

demariadaniel requested review from ciaranschutte, joneubank, justincorrigible and leoraba May 28, 2025 17:32

demariadaniel changed the title ~~WIP: #67 / File Metadata Table~~#67 / File Metadata TableMay 28, 2025

Rename function
3adeed1

joneubank approved these changes May 28, 2025
View reviewed changes

PR feedback, tableUtil file
0256b61

demariadaniel merged commit 571a8a0 into iobioJun 2, 2025

demariadaniel deleted the 67/feat-file-metadata-table branch June 2, 2025 21:23

demariadaniel added 6 commits May 26, 2025 16:57

Stub Stats Table, separate files
9ea1f75

Very Basic Table
2b4b11b

Updated file table properties
0efd619

Improved StatsTable CSS
88171c7

Reorg components, use theme colors, add selected count
eadea46

Show / Hide Button
35e343b

demariadaniel self-assigned this May 28, 2025

demariadaniel added 5 commits May 28, 2025 10:18

Use span -/+ icon
d00af7c

Unused DemoStats
5391547

1px adjustment, Remove nested span CSS
8f75641

Header padding
41fc4d2

File Size Label, get data function
84607d5

demariadaniel marked this pull request as ready for review May 28, 2025 17:32

demariadaniel requested review from ciaranschutte, joneubank, justincorrigible and leoraba May 28, 2025 17:32

demariadaniel changed the title ~~WIP: #67 / File Metadata Table~~#67 / File Metadata TableMay 28, 2025

Rename function
3adeed1

joneubank approved these changes May 28, 2025
View reviewed changes

PR feedback, tableUtil file
0256b61

demariadaniel merged commit 571a8a0 into iobioJun 2, 2025

demariadaniel deleted the 67/feat-file-metadata-table branch June 2, 2025 21:23

demariadaniel added 6 commits May 26, 2025 16:57

Stub Stats Table, separate files
9ea1f75

Very Basic Table
2b4b11b

Updated file table properties
0efd619

Improved StatsTable CSS
88171c7

Reorg components, use theme colors, add selected count
eadea46

Show / Hide Button
35e343b

demariadaniel self-assigned this May 28, 2025

demariadaniel added 5 commits May 28, 2025 10:18

Use span -/+ icon
d00af7c

Unused DemoStats
5391547

1px adjustment, Remove nested span CSS
8f75641

Header padding
41fc4d2

File Size Label, get data function
84607d5

demariadaniel marked this pull request as ready for review May 28, 2025 17:32

demariadaniel requested review from ciaranschutte, joneubank, justincorrigible and leoraba May 28, 2025 17:32

demariadaniel changed the title ~~WIP: #67 / File Metadata Table~~#67 / File Metadata TableMay 28, 2025

Rename function
3adeed1

joneubank approved these changes May 28, 2025
View reviewed changes

PR feedback, tableUtil file
0256b61

demariadaniel merged commit 571a8a0 into iobioJun 2, 2025

demariadaniel deleted the 67/feat-file-metadata-table branch June 2, 2025 21:23

demariadaniel added 6 commits May 26, 2025 16:57

Stub Stats Table, separate files
9ea1f75

Very Basic Table
2b4b11b

Updated file table properties
0efd619

Improved StatsTable CSS
88171c7

Reorg components, use theme colors, add selected count
eadea46

Show / Hide Button
35e343b

demariadaniel self-assigned this May 28, 2025

demariadaniel added 5 commits May 28, 2025 10:18

Use span -/+ icon
d00af7c

Unused DemoStats
5391547

1px adjustment, Remove nested span CSS
8f75641

Header padding
41fc4d2

File Size Label, get data function
84607d5

demariadaniel marked this pull request as ready for review May 28, 2025 17:32

demariadaniel requested review from ciaranschutte, joneubank, justincorrigible and leoraba May 28, 2025 17:32

demariadaniel changed the title ~~WIP: #67 / File Metadata Table~~#67 / File Metadata TableMay 28, 2025

Rename function
3adeed1

joneubank approved these changes May 28, 2025
View reviewed changes

PR feedback, tableUtil file
0256b61

demariadaniel merged commit 571a8a0 into iobioJun 2, 2025

demariadaniel deleted the 67/feat-file-metadata-table branch June 2, 2025 21:23

demariadaniel added 6 commits May 26, 2025 16:57

Stub Stats Table, separate files
9ea1f75

Very Basic Table
2b4b11b

Updated file table properties
0efd619

Improved StatsTable CSS
88171c7

Reorg components, use theme colors, add selected count
eadea46

Show / Hide Button
35e343b

demariadaniel self-assigned this May 28, 2025

demariadaniel added 5 commits May 28, 2025 10:18

Use span -/+ icon
d00af7c

Unused DemoStats
5391547

1px adjustment, Remove nested span CSS
8f75641

Header padding
41fc4d2

File Size Label, get data function
84607d5

demariadaniel marked this pull request as ready for review May 28, 2025 17:32

demariadaniel requested review from ciaranschutte, joneubank, justincorrigible and leoraba May 28, 2025 17:32

demariadaniel changed the title ~~WIP: #67 / File Metadata Table~~#67 / File Metadata TableMay 28, 2025

Rename function
3adeed1

joneubank approved these changes May 28, 2025
View reviewed changes

PR feedback, tableUtil file
0256b61

demariadaniel merged commit 571a8a0 into iobioJun 2, 2025

demariadaniel deleted the 67/feat-file-metadata-table branch June 2, 2025 21:23

demariadaniel added 6 commits May 26, 2025 16:57

Stub Stats Table, separate files
9ea1f75

Very Basic Table
2b4b11b

Updated file table properties
0efd619

Improved StatsTable CSS
88171c7

Reorg components, use theme colors, add selected count
eadea46

Show / Hide Button
35e343b

demariadaniel self-assigned this May 28, 2025

demariadaniel added 5 commits May 28, 2025 10:18

Use span -/+ icon
d00af7c

Unused DemoStats
5391547

1px adjustment, Remove nested span CSS
8f75641

Header padding
41fc4d2

File Size Label, get data function
84607d5

demariadaniel marked this pull request as ready for review May 28, 2025 17:32

demariadaniel requested review from ciaranschutte, joneubank, justincorrigible and leoraba May 28, 2025 17:32

demariadaniel changed the title ~~WIP: #67 / File Metadata Table~~#67 / File Metadata TableMay 28, 2025

Rename function
3adeed1

joneubank approved these changes May 28, 2025
View reviewed changes

PR feedback, tableUtil file
0256b61

demariadaniel merged commit 571a8a0 into iobioJun 2, 2025

demariadaniel deleted the 67/feat-file-metadata-table branch June 2, 2025 21:23

demariadaniel added 6 commits May 26, 2025 16:57

Stub Stats Table, separate files
9ea1f75

Very Basic Table
2b4b11b

Updated file table properties
0efd619

Improved StatsTable CSS
88171c7

Reorg components, use theme colors, add selected count
eadea46

Show / Hide Button
35e343b

demariadaniel self-assigned this May 28, 2025

demariadaniel added 5 commits May 28, 2025 10:18

Use span -/+ icon
d00af7c

Unused DemoStats
5391547

1px adjustment, Remove nested span CSS
8f75641

Header padding
41fc4d2

File Size Label, get data function
84607d5

demariadaniel marked this pull request as ready for review May 28, 2025 17:32

demariadaniel requested review from ciaranschutte, joneubank, justincorrigible and leoraba May 28, 2025 17:32

demariadaniel changed the title ~~WIP: #67 / File Metadata Table~~#67 / File Metadata TableMay 28, 2025

Rename function
3adeed1

joneubank approved these changes May 28, 2025
View reviewed changes

PR feedback, tableUtil file
0256b61

demariadaniel merged commit 571a8a0 into iobioJun 2, 2025

demariadaniel deleted the 67/feat-file-metadata-table branch June 2, 2025 21:23

		const fileStudy = file?.analysis?.collaborator.hits.edges[0].node.name;
		const fileStrategy = file?.analysis?.experiment.experimentalStrategy;

Uh oh!

Conversation

demariadaniel commented May 28, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Issues

Description of Changes

Readiness Checklist

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

demariadaniel commented May 28, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Issues

Description of Changes

Readiness Checklist

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

demariadaniel commented May 28, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Issues

Description of Changes

Readiness Checklist

Uh oh!

Choose a reason for hiding this comment

demariadaniel commented May 28, 2025•
edited
Loading

demariadaniel commented May 28, 2025•
edited
Loading

demariadaniel commented May 28, 2025•
edited
Loading