fix: change the encoding into utf-8

[MistyField]

No description provided.

[MistyField]

Update: in the line 250 of parser.py refs is a string so when it was sliced, only 10 references in maximum could be added, and even worse, in the wrong order. So it may be changed into references[refs] = line and it was tested fine.

[Robaina]

Thanks so much for the PR!

I'm planning to update BRENDApyrser to accommodate the newly available json format, see issue #9. I'll leave this PR on hold for now since most likely this issue with the references will be solved.

[MistyField]

Thanks so much for the PR!

I'm planning to update BRENDApyrser to accommodate the newly available json format, see issue #9. I'll leave this PR on hold for now since most likely this issue with the references will be solved.

Thank you for your reply! And I successfully made a database of enzyme kinetics using BRENDApyrser which is very powerful! For me, parsing the json is a good idea, it will save a lot of time in parsing the data, but the change in this commit is needed regardless of whether the data format is .txt or .json since it will impact the consistency of the content in the references.

Best regards,

[Robaina]

Glad to hear that! It's awesome to learn that you have found the package useful!

[Robaina]

Hi @MistyField ,

sorry, could you update your branch with the current master branch to resolve these conflicts prior to merge?

[MistyField]

Hi @MistyField ,

sorry, could you update your branch with the current master branch to resolve these conflicts prior to merge?

Hello, I've updated my forked branch with the master branch:)

[Robaina]

I don't want to merge this commit with the ".idea" directory from JetBrains? This is a local directory, shouldn't be in the repo. Please, could you remove it? We don't need to include ".idea" in .gitignore either.

Otherwise, I'm fine with the rest of the PR!

[MistyField]

I don't want to merge this commit with the ".idea" directory from JetBrains? This is a local directory, shouldn't be in the repo. Please, could you remove it? We don't need to include ".idea" in .gitignore either.

Otherwise, I'm fine with the rest of the PR!

Of course! Apologize for the inconvenience I caused by changing the .gitignore🙏

[Robaina]

Excellent! Thank you!

[Robaina]

I don't want to merge this commit with the ".idea" directory from JetBrains? This is a local directory, shouldn't be in the repo. Please, could you remove it? We don't need to include ".idea" in .gitignore either.
Otherwise, I'm fine with the rest of the PR!

Of course! Apologize for the inconvenience I caused by changing the .gitignore🙏

No worries ! Merging PR