Skip to content

Conversation

@avdata99
Copy link
Contributor

Error with frquency table

Could not copy frequency data to database:
extra data after last expected column
CONTEXT: COPY 737a3aad-c1c8-4507-8411-2dc6ca176f84-druf-freq,
line 2: "country,Afghanistan,6,2.52101,1"

They are 5 fields and the freq table only have 4

cur_statsfreq.execute(
        sql.SQL(
            """
            DROP TABLE IF EXISTS {};
            CREATE TABLE {} (
                field TEXT,
                value TEXT,
                count INTEGER,
                percentage FLOAT,
                PRIMARY KEY (field, value, count)
            )
        """
        ).format(freq_table, freq_table)

Temporary we add a text field just to continue but probably there is another correct answer

@avdata99 avdata99 mentioned this pull request Oct 29, 2025
@tino097
Copy link
Member

tino097 commented Nov 5, 2025

hey @avdata99 looking in your case text is ok, but i think we will need rank as FLOAT
Would you like to update this PR ?

@avdata99
Copy link
Contributor Author

avdata99 commented Nov 6, 2025

@tino097 PR updated here 60d9d73

@tino097
Copy link
Member

tino097 commented Nov 6, 2025

can you please resolve the conflict too ?@avdata99

@avdata99
Copy link
Contributor Author

avdata99 commented Nov 7, 2025

@tino097 it looks like this PR is no longer valid for main. The base code no longer exist.
We are based on the 2.0.0 tag and we create PRs to main from changes in use in our custom branch based on 2.0.0.
If you release a new version closed to main, we can move to it and our changes will be close to main

@avdata99
Copy link
Contributor Author

avdata99 commented Nov 7, 2025

Please, feel free to close this if you consider this change is no longer valid @tino097

@jqnatividad
Copy link
Collaborator

Thanks for this @avdata99. Just wanted to let you know that we'll be making a 3.0.0 release this month as we generalize DP+ and make the DRUF (Data Resource Upload Flow) generally available.

The DRUF is the interactive FAIRification where metadata suggestions are offered to the User based on the qsv-powered analysis of the file when its first uploaded.

@avdata99
Copy link
Contributor Author

Thanks @jqnatividad
Will DRUF be an optional feature?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants