-
Notifications
You must be signed in to change notification settings - Fork 19
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
The problem of local configurations #27
Comments
After further inspection, it appears that there is a way to access the configuration options before loading the local configuration. However, a predetermined path choosing for problem/contest destination still means a difference in how local .configurations.lua files are done. |
No, when you download a problem the local configuration is loaded from the directory you choose
What settings would you like to change for different judges?
Local configuration is often useful, so I won't drop it until we find a better system.
You can ignore local configurations using global setup options:
So you'd like to store all contests in a predetermined folder configured with |
Yes, I figured it would be better to offer a boolean configuration which switches between predetermined paths and prompting the user. About the judge, it was just a point addressing local configuration, however, now I think a switch system configured from |
Reviving a dead thread, but this is an issue while receiving problems. The way the readme is phrased makes it look like For example, if I have two templates:
My original config uses Either the readme should be fixed to clarify this, or the code should be updated to load the local directory config on init (I've done so in a fork, happy to send a PR if required) |
@neov5 I also had the same confusion until I saw this thread. I think it would make more sense if every folder in the subfolder is affected by Since I have templates separately for codeforces / atcoder and every judge, and I can't figure out a way to use separate templates when running "CompetiTest receive problem" |
nvm, I just copied that one line from your fork and it works but maybe the maintainer want to reword the readme if its intended |
Currently, any receive commands asks for directory and/or name of file.
This is not really useful when you are solving problems, and can be really annoying after some time.
The issue is, a configuration option can't be used as the configuration needs to be local and extended (this is how it is currently with
config.load_local_config_and_extend
).Also, if a new problem is downloaded from a problem-set, it won't have a specific local configuration, as the problem does not exist on system for it to have a local configuration.
And if a per judge configuration is required, this should be done in the main configuration, as per judge configuration is not local to the problem but rather specific to the judge (by definition).
And specific compiler commands should have another solution than having local config files per problem, maybe a keybinding.
I think dropping the local configuration would allow the project to have configuration options, like "contest_directory", which would enable functionality similar to how cpbooster.nvim works.
If this is fine with the maintainer @xeluxee, I would like to work on such functionality.
The text was updated successfully, but these errors were encountered: