Skip to content
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

Use latest release of Delft-FIAT (backend) #610

Open
panosatha opened this issue Dec 9, 2024 · 0 comments
Open

Use latest release of Delft-FIAT (backend) #610

panosatha opened this issue Dec 9, 2024 · 0 comments
Assignees
Labels
beta release Feature New feature request

Comments

@panosatha
Copy link
Contributor

Kind of request

Changing existing functionality

Enhancement Description

So far FloodAdapt uses Delft-FIAT version v0.1.0rc2 as an executable.
If we want to use the latest one (v0.2.1) we need ti update the naming of the exposure columns to the new standard that follows a snake case format.

In order to avoid this issue in the future we should change the references in the code to the exposure columns, from direct use of column name to the use of an api style object that includes all the attribute naming for specific FIAT versions. Something like this:

class FIAT

This should be done after the event refactoring and FIAT adapter refactoring are merged.

Task list

TBD

Use case

No response

Additional Context

No response

@panosatha panosatha added beta release Feature New feature request labels Dec 9, 2024
@panosatha panosatha changed the title Use latest release of Delft-FIAT Use latest release of Delft-FIAT (backend) Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta release Feature New feature request
Projects
None yet
Development

No branches or pull requests

2 participants