Skip to content

Use Tables.jl for GDX record storage#10

Closed
MartinBonde wants to merge 1 commit into
mainfrom
fix-review-issues
Closed

Use Tables.jl for GDX record storage#10
MartinBonde wants to merge 1 commit into
mainfrom
fix-review-issues

Conversation

@MartinBonde

Copy link
Copy Markdown
Collaborator

Summary

  • Preserve table metadata descriptions when writing Tables.jl-compatible parameter pairs.
  • Restrict the Tables.jl interface to record-backed GDX symbols so aliases do not advertise table support.
  • Update README docs for the Tables.jl default sink and add regression coverage.

Test plan

  • julia --project=. -e 'using Pkg; Pkg.resolve()'
  • julia --project=. -e 'using Pkg; Pkg.test()'

Make GDX symbols store generic Tables.jl-compatible records, preserve table metadata descriptions when writing, and document the new default read behavior.
@MartinBonde MartinBonde force-pushed the fix-review-issues branch from f0f569f to d2e839b Compare May 5, 2026 10:44
@MartinBonde MartinBonde changed the title Fix Tables API review issues Use Tables.jl for GDX record storage May 5, 2026
@MartinBonde MartinBonde closed this May 5, 2026
@odow odow deleted the fix-review-issues branch June 15, 2026 01:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant