Skip to content

Hp cfa datatops faq - #172

Open
HPSLU wants to merge 15 commits into
mainfrom
hp-cfa-datatops-faq
Open

HPSLU wants to merge 15 commits into
mainfrom
hp-cfa-datatops-faq

Conversation

@HPSLU

@HPSLU HPSLU commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Created a new cfa-dataops FAQ document

@ryanraaschCDC

Copy link
Copy Markdown
Collaborator

is this the final PR? If it is please close out the other PRs. In the future, there should only be one PR per issue and if you add/change files it goes to the same PR, not a new one. Thanks!

Comment thread docs/catalog_creation.md
The `dataops_catalog_init` command is automatically available after installing the `cfa.dataops` package:

```bash
pip install cfa.dataops

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this kind of pip install won't actually work because the package is only on github. It will be pip install git+https://github.com/cdcgov/cfa-dataops.git.

Comment thread docs/catalog_creation.md

or

uv add cfa.dataops

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

similar to the comment above. this will be uv add cfa-dataops @ git+<same link as above>

## Key directories for developers:

### `datasets/`
contains TOML files defining dataset ETL pipelines, metadata, validation rules, and staging behaviours.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

capitalize C in contains for consistency

contains TOML files defining dataset ETL pipelines, metadata, validation rules, and staging behaviours.

### `workflows/`
contains reusable Python modules or workflow scripts supporting ETL.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

capitalize C in contains for consistency

Load with a version filter

```python
df = datacat.my_project.my_dataset.load.get_dataframe(version=">2024.12.01,<2025.08")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

version should be version_spec

See which version would be chosen

```python
v = datacat.my_project.my_dataset.load.resolve_versions(version="latest")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

same version_spec here

Comment thread docs/faq.md
# CFA DataOps Frequently Asked Questions (FAQ)

## Purpose
This FAQ document addressess common questions about the **cfa-dataops** repository and serves as a supplementary reference to existing documentation-including the technical guides, user guides, and quick start materials-to support consistent and effective use of the repository.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

put spaces around each of the -

Comment thread docs/faq.md
You should have access to:
- CDC network
- CFA DataOps Github
- CDC.gov repositories

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this should be CDCgov not CDC.gov

Comment thread docs/faq.md

If something is missing, [email cfatools](mailto:cfatoolsteam@cdc.gov) for technical support.

**2. Where should I work day-to-day?**

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this #2 could probably be removed

Comment thread docs/faq.md

datacat.private.scenarios.covid19vax_trends.extract()

datacat.private.scenarios.covid19vax_trends.transform()

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm pretty sure transform() and load() need arguments (dataframes) passed in

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