-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ldo scenarios #132
base: main
Are you sure you want to change the base?
Ldo scenarios #132
Conversation
Remarks: | ||
Maximum upload is 2 Gb | ||
Dem is aggregated to 5x5 results created at 0.5x0.5m | ||
Etc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The idea was that you would put relevant remarks here, this is just the example I provided, why leave 'etc.'?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I add a bit more explanation. I think is clearer. Also a bit long but clearer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added TODO's in the code.
Changed structure a bit and create LDO_API class.
def full_structure(self): | ||
return print(FOLDER_STRUCTURE) | ||
|
||
def to_file_dict(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you need this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I delete lines 116 and 117. With the previous function (lines 98 to 105) is more than enough
mamba wasnt found
Mamba to conda
add oracledb
I did some comments on the code.
No description provided.