You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This could remunerate the tokens (and adjust the heads)
I thought of doing this at the time of implementation, but:
it's more complex
sometimes you want some tokens to have the new token as head, so you'd have to adjust the head adjustment
it might be worth it, though! I'm still trying to balance the mode's complexity with its features. ideally I'd have the mode less complex — it currently reimplements lots of things that are in cl-conllu and other UD libraries — and delegate tasks like this one to these libraries. we can even make it abstract enough so that we can support all of them. for instance there might be a variable conllu-add-token-command that specifies a shell command to be run with the current sentence as input plus an id. when the user calls the conllu-insert-token-line command the command specified by the variable is run with the ID at point as argument + the current sentence, and the output is expected to be a new sentence with a new token added after the ID (and the new sentence might even come with readjusted heads or whatever). this can even be implemented outside the mode with the API I've just exposed, although if it's generally useful it should probably be added to the mode. we should get more UD people using the mode so that we have more feedback and contributions ;P
and add a token line with a number already filled?
if we were to add a new-multi-word-token and new-empty-node commands, then sure! it does not do that so that the token template that is inserted is equally useful for the three kinds of token.
PS: I changed the name of the issue because I think it's better to refer to the command name than to the keybinding (which can be changed by the user)
This could remunerate the tokens (and adjust the heads) and add a token line with a number already filled?
We also need commands to
The text was updated successfully, but these errors were encountered: