bemoody

Also note the PAUSE_CREDENTIALING variable, which is somewhat related though intended for a different purpose. PAUSE_CREDENTIALING causes the link to the credentialing form to be hidden - it doesn't prevent...

Okay, so I had the thought that it might be better to add the behavior to `django.db.models.FileField` instead of to the views. So basically I did this ``` diff --git...

It seems to work the way I want if I wrap the body of edit_training in transaction.atomic, or if I wrap TrainingForm.save in transaction.atomic, or even if I just wrap...

A couple of small things: In training/test_views.py, we don't want to duplicate the TestMixin class. I don't think this is needed here at all - we can just remove lines...

The URLs are a little messy: ``` path('settings/platform-training//', views.take_training, name='platform_training'), path('settings/platform-training//module/-/', views.current_module_block, name='platform_training_module_block'), path('settings/platform-training//next/-/', views.next_module_block, name='next_module_block'), path('settings/platform-training//previous/-/', views.previous_module_block, name='previous_module_block'), ``` Using the `module_id` (primary key) in the URL is not...

* When possible, please use the same name for the function in views.py as the "name" used for the URL in urls.py. It is confusing to see `redirect('platform_training_module_block', ...)` which...

> A while back, Alistair uploaded a copy of the MIMIC-IV demo to: https://huggingface.co/datasets/physionet/mimic-iv-demo **Please, please, please don't use unversioned URLs** :(

Looks good, thanks!

As far as I know, we don't have a local database of who has permission to access restricted Google Cloud buckets. The only way to know who has access is...

Sorry, I replied in a hurry and should have been more clear. > Of course, we expect a different message with a gsutil command to be shown after the user...