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.

Sign up for GitHub

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

Jump to bottom

NativeCsvWriter -- need columns support #64

Closed
2 tasks done
nicolay-r opened this issue May 30, 2023 · 2 comments
Closed
2 tasks done

NativeCsvWriter -- need columns support #64

nicolay-r opened this issue May 30, 2023 · 2 comments
Assignees
@nicolay-r
Labels
critical

Comments

@nicolay-r
Copy link
Owner

nicolay-r commented May 30, 2023 •
edited
Loading

  • columns saving
  • extra columns provider (temporary)

ARElight/arelight/brat_backend.py

Lines 170 to 172 in d87f81b

for i, e_ind in enumerate(sent_data[const.ENTITIES]):
sentence_entity_values = sent_data[const.ENTITY_VALUES]
sentence_entity_types = sent_data[const.ENTITY_TYPES]

@nicolay-r nicolay-r self-assigned this May 30, 2023
@nicolay-r
Copy link
Owner Author

nicolay-r/AREkit#462

@nicolay-r
Copy link
Owner Author

Consider the case of adding extra columns for this native writer:

for i, e_ind in enumerate(sent_data[const.ENTITIES]):
TypeError: 'NoneType' object is not iterable
Iter documents data: 0it [00:00, ?it/s]

@nicolay-r nicolay-r reopened this Jun 1, 2023
nicolay-r added a commit that referenced this issue Jun 1, 2023
@nicolay-r
#64 added
c823857
nicolay-r added a commit that referenced this issue Jun 1, 2023
@nicolay-r
#64 fix
23a0223
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nicolay-r nicolay-r

Labels
critical
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nicolay-r