Skip to content

Conversation

@paxtonfitzpatrick
Copy link
Member

@paxtonfitzpatrick paxtonfitzpatrick commented Jan 12, 2021

vpn ➜  clustertools git:(master) ✗ tree .
.
├── __init__.py
├── cluster.py
├── file_objects
│   ├── __init__.py
│   ├── configs
│   │   ├── __init__.py
│   │   ├── base_config.py
│   │   ├── config_helpers.py
│   │   ├── global_config.py
│   │   ├── project_config.py
│   │   └── tracked_attr_config.py
│   ├── project_script.py
│   └── synced_file.py
├── project
│   ├── __init__.py
│   ├── job.py
│   ├── project.py
│   └── project_handler.py
├── shared
│   ├── __init__.py
│   ├── environ.py
│   ├── exceptions.py
│   ├── helpers.py
│   ├── multistream_wrapper.py
│   ├── object_monitors.py
│   ├── remote_process.py
│   └── typing.py
├── shells
│   ├── __init__.py
│   ├── base_shell.py
│   ├── local_shell.py
│   └── ssh_shell.py
└── templates
    ├── __init__.py
    ├── global_config.ini
    ├── monitor.py
    ├── project_config.ini
    ├── submit_old.py
    └── submitter.py

7 directories, 38 files

paxtonfitzpatrick and others added 25 commits January 8, 2021 15:02
…h allows for hooks to ensure lists are converted to MonitoredLists, dict is converted to MonitoredEnviron, etc. Also massively simplified __setattr__ and update methods
reorganized config object modules, wrote hooks to validate and update dependent properties when fields are updated
… implement this by trapping errors in wrapper script
… fields accessible through Project.configure()
…ant and doesn't actually do anything since jobs *must* be run in login shells anyway (in order for 'module' commands to work)
@jeremymanning
Copy link
Member

@paxtonfitzpatrick do you want me to merge?

@paxtonfitzpatrick paxtonfitzpatrick merged commit 956221e into ContextLab:v0.1 Jan 12, 2021
@paxtonfitzpatrick
Copy link
Member Author

sorry! I went ahead and did it since it's not the master branch

@jeremymanning
Copy link
Member

ah-- i guess so!

@jeremymanning
Copy link
Member

no prob; didn't know if you had write access

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants