Skip to content

Conversation

@Eric-Liu-SANDAG
Copy link
Contributor

@Eric-Liu-SANDAG Eric-Liu-SANDAG commented Jan 14, 2026

Describe this pull request. What changes are being made?

Optimize data insertion for ASE data. Instead of loading in all data, we just load non-zero data, then use a TVF to get the data with zeros added in

(also adds in Black which was missing)

What issues does this pull request address?

Additional context

N/A

@Eric-Liu-SANDAG Eric-Liu-SANDAG self-assigned this Jan 14, 2026
@Eric-Liu-SANDAG
Copy link
Contributor Author

@GregorSchroeder do you think TVFs should have some kind of name prefix? Like [outputs].[tvf_ase_with_zeros] instead of [outputs].[ase_with_zeros]

@GregorSchroeder
Copy link
Contributor

@GregorSchroeder do you think TVFs should have some kind of name prefix? Like [outputs].[tvf_ase_with_zeros] instead of [outputs].[ase_with_zeros]

usually, I just stay with the generic "fn_" prefix regardless of the type of function

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.

[FEATURE] Add black to uv configuration [FEATURE] Speed up the program by speeding up data insertion

3 participants