Skip to content

Commit

Permalink
changes to tstep and timestamp shouldn't be implemented yet (#151)
Browse files Browse the repository at this point in the history
  • Loading branch information
jClugstor authored Dec 7, 2023
1 parent 48a4d22 commit ccda4ef
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions src/SimulationService.jl
Original file line number Diff line number Diff line change
Expand Up @@ -417,11 +417,6 @@ function get_dataset(obj::JSON3.Object)
s3_url = get_json(tds_url).url
df = CSV.read(download(s3_url), DataFrame)

# there should always be a mapping from the dataset timestamp column name to 'timestamp'
if !any(v -> v == "timestamp", values(obj.mappings))
@warn "Expected mapping from <dataset timestamp column> to 'timestamp' not provided"
end

for (k,v) in get(obj, :mappings, Dict())
@info "`get_dataset` (dataset id=$(repr(obj.id))) rename! $k => $v"
if k != "tstep"
Expand Down

0 comments on commit ccda4ef

Please sign in to comment.