Skip to content

Commit

Permalink
Cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
equinor-ruaj committed Jan 22, 2025
1 parent 34be2f1 commit cc67cd7
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions src/fmu/sumo/sim2sumo/tables.py
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,6 @@ def generate_table_meta(datafile, obj, tagname, config):
return metadata


# TODO: If the split is inherent in this method it should probably be renamed
def convert_table_2_sumo_file(datafile, obj, tagname, config):
"""Convert table to Sumo File ready for shipping to sumo
Expand All @@ -131,10 +130,6 @@ def convert_table_2_sumo_file(datafile, obj, tagname, config):

metadata = generate_table_meta(datafile, obj, tagname, config)

print(f"GENERATED METADATA: {metadata}")
# TODO: Use metadata["data"]["table_index"]
# If table_index is not present, do not split

files = []
chunk_size = 500
columns = metadata["data"]["spec"]["columns"]
Expand Down Expand Up @@ -164,8 +159,6 @@ def convert_table_2_sumo_file(datafile, obj, tagname, config):
f"--{tagname}", f"--{tagname}:{idx:03d}"
)

# TODO: FileOnJob resets _sumo metadata
# TODO: Create FileOnJob first, THEN update sumo_file.metadata["_sumo"]
sumo_file = FileOnJob(bytestring, chunk_meta)
sumo_file.path = chunk_meta["file"]["relative_path"]
sumo_file.metadata_path = ""
Expand All @@ -189,7 +182,6 @@ def convert_table_2_sumo_file(datafile, obj, tagname, config):
sumo_file.size = len(sumo_file.byte_string)
files.append(sumo_file)

# return sumo_file
return files


Expand Down Expand Up @@ -305,8 +297,6 @@ def upload_tables_from_simulation_run(
config (dict): the fmu config with metadata
dispatcher (sim2sumo.common.Dispatcher)
"""
print(f"CONFIG: {config}")

logger = logging.getLogger(__name__ + ".upload_tables_from_simulation_run")
for submod, options in submod_and_options.items():
if submod == "grid3d":
Expand Down

0 comments on commit cc67cd7

Please sign in to comment.