Skip to content

feat!: get_data now returns XArrayDataArray instead of the previous TplotVariable structure feat: add PySPEDASSchema for pyspedas CDF.VATT metadata#22

Merged
Beforerr merged 1 commit into
mainfrom
push-lqpxmvkkqopt
May 19, 2026
Merged

feat!: get_data now returns XArrayDataArray instead of the previous TplotVariable structure feat: add PySPEDASSchema for pyspedas CDF.VATT metadata#22
Beforerr merged 1 commit into
mainfrom
push-lqpxmvkkqopt

Conversation

@Beforerr
Copy link
Copy Markdown
Member

Tags get_data output with SchemaDict(PySPEDASSchema(), ...) so semantic
attributes (:desc, :name, :unit, :scale, :labels, :display_type) resolve
through the nested CDF→VATT structure pyspedas emits.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 19, 2026

Benchmark Results (Julia v1)

Time benchmarks
main 7b7f088... main / 7b7f088...
DimArray 2.71 ± 0.18 μs 21.1 ± 2.5 μs 0.129 ± 0.017
DimArray (direct) 0.0337 ± 0.0033 ms 28.4 ± 1.4 μs 1.19 ± 0.13
get_data 30.4 ± 3.8 μs 5.83 ± 0.32 μs 5.21 ± 0.71
get_data (collect) 0.0368 ± 0.0042 ms 9.43 ± 2.1 μs 3.9 ± 0.99
time_to_load 4.61 ± 0.023 s 4.48 ± 0.0071 s 1.03 ± 0.0054
Memory benchmarks
main 7b7f088... main / 7b7f088...
DimArray 14 allocs: 0.844 kB 0.052 k allocs: 2.28 kB 0.37
DimArray (direct) 0.122 k allocs: 4.22 kB 0.084 k allocs: 3.45 kB 1.22
get_data 0.108 k allocs: 3.38 kB 0.032 k allocs: 1.17 kB 2.88
get_data (collect) 0.111 k allocs: 9.07 kB 0.035 k allocs: 6.87 kB 1.32
time_to_load 0.143 k allocs: 10.6 kB 0.143 k allocs: 10.6 kB 1

  previous `TplotVariable` structure
feat: add PySPEDASSchema for pyspedas CDF.VATT metadata

Tags get_data output with SchemaDict(PySPEDASSchema(), ...) so semantic
attributes (:desc, :name, :unit, :scale, :labels, :display_type) resolve
through the nested CDF→VATT structure pyspedas emits.
@Beforerr Beforerr force-pushed the push-lqpxmvkkqopt branch from dfbb87a to 7b7f088 Compare May 19, 2026 17:09
@Beforerr Beforerr merged commit f62f765 into main May 19, 2026
7 checks passed
@Beforerr Beforerr deleted the push-lqpxmvkkqopt branch May 19, 2026 17:30
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.

1 participant