-
Notifications
You must be signed in to change notification settings - Fork 1
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
Data viewing fixes #4
Conversation
Hi @Unaiperezdiaz, i like the idea of making the HDF5 handler as a single class, but the other parts of the code would need to be corrected to handle that code before any those updates can be published. Also, can I get a copy of your test file than you're using to get that error with BivitattusView? I will check it out and see if i can trace it. |
This file is from the same module "BDB_io.py". |
it looks like you ran the output file from |
I used the example file |
Would you like to continue working on it, or should I give it a shot? |
I'm working on it, but I have found a strange file output executing ex_auto_commut.py |
That's the encrypted database file. I found out on other projects that converting the encrypted text to hexadecimal decreases the risk of corruption or mis-understood directive when it comes to read/sending files. The key should be the password you set when initializing the |
After the commit this is the output:
|
When you restructered BDB.io into a class, the other files can't find the functions to read and write correctly. You got to change the read/write/edit functions in the other files so they can read the right code. For example, in BDB_tb.py:
would need to be:
|
|
it looks like the data is never decompressed when reading the table:
|
umh... this is new when I create a new DB using the example code: |
Yeah, I expanded the first error: |
Why don't we check the JSON before dumping it? Because if this JSON has a problem, which we can't see because it is compressed and encrypted, we can't know if there will be more problems or not. |
What I think is happening is that when we reload the JSON when reading the data into a newly loaded table, it's not getting decoded and decompressed before being converted back to obj format. |
Sometimes it appears: |
Yeah, that's unhexlify trying to decode the encryption, but it's finding bytes instead of hexadecimal |
So, I found there was a clause to decode and decompress during the read process, and then we were trying to do it again afterwards, which threw an error. I fixed that in both the bdb_tb and bdb_metadata tables, but I've came across another error while reading a already created table, which I am working on currently. |
Ok... didn't mean to merge it... but hopefully you can find the error of why it's having problems reopening the file |
Also, I just want to let you know, I'm working on a C code that can replace the use of HDF5 files, so this issue may just solve itself. |
I have solved the problem to be able to display the |
what about |
With |
Yeah, I was getting the same error, I though I'd give it to you to try. Right now I have it committed with this error (by accident) so hopefully we can fix it. quick tip. you can expand the traceback by importing traceback and adding the line |
I have fixed this error with the modification I have made, because every time you displayed the treeview you got an error message that doesn't appear anymore. But now I don't know how to fix the following.
![Captura de pantalla 2024-06-24 124536](https://private-user-images.githubusercontent.com/49728036/342475810-a853c008-778d-4f95-ab87-63db82b39489.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk5OTA1NzMsIm5iZiI6MTcxOTk5MDI3MywicGF0aCI6Ii80OTcyODAzNi8zNDI0NzU4MTAtYTg1M2MwMDgtNzc4ZC00Zjk1LWFiODctNjNkYjgyYjM5NDg5LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDMlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzAzVDA3MDQzM1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTgwMzkwNGMzZGM1ZDRmMDllYmM2MDBlYTI5NWZmNTk3NDhiNzQxMzhkYjExZmI5NmI5OTViZGQ4NTg4MDE0YzImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.qOddmIwh-0Eic4xfUoUBdb7kwsrV6vHuBobn0jO2Meg)
![Captura de pantalla 2024-06-24 220113](https://private-user-images.githubusercontent.com/49728036/342476287-3f27e2da-19db-4985-b637-438d06a6ec86.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk5OTA1NzMsIm5iZiI6MTcxOTk5MDI3MywicGF0aCI6Ii80OTcyODAzNi8zNDI0NzYyODctM2YyN2UyZGEtMTlkYi00OTg1LWI2MzctNDM4ZDA2YTZlYzg2LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDMlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzAzVDA3MDQzM1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTQ5YTJkN2JiOTc3YWRkMjVkNmZkYWJmYmQyODFhNGE4ODZkOGRmMDJmNzVlZjkyY2Q0Mzc4NWU0ZWZmMGI1MzQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.FAcVI7oa1UwpAHqnGTi_TQ4d4aKdLi_11J73OeIDRCE)
(Using the test file):