David Ormsbee

Results 38 comments of David Ormsbee

@GlugovGrGlib: Thank you! Do you have a broad sense of what the bottlenecks are for those really large courses?

@GlugovGrGlib: Just as a heads up, when I run this locally on my laptop with the large test course and try loading just this REST endpoint in isolation, I get...

Or maybe profiling was left on?

@NiedielnitsevIvan: Is that because the collect phase of block transformers takes time to run, so the version you get immediately after making a change is still the course blocks of...

@Agrendalath: Nothing else that I can think of. @jristau1984: Please note that while this change has been tested as best as it's practical to do, the support team should immediately...

@mphilbrick211: Thank you for the reminder. FWIW, I did start re-reading through this today, but it's taking me a while. Aiming to have a review submitted by end of day...

@UvgenGen: Is this ready for another review? (I just want to make sure you're not waiting on me.)

> We investigate possibility to mock draft course data, but in witch case CourseOverview can be regenerated if we removed all create/update methods for draft modulestore? I'm not entirely sure...

@mphilbrick211: Just wanted to merge the last of the BD-13 PRs before this one. @kaustavb12, @Agrendalath: @glugovgrglib mentioned that [new failures](https://github.com/openedx/edx-platform/actions/runs/4811655665/jobs/8565912731?pr=31134#step:7:98) occurred on this PR after https://github.com/openedx/edx-platform/pull/31472 was merged. Do...

@Agrendalath: I think it's more a question of whether that https://github.com/openedx/edx-platform/commit/581f07c47d0644b446c3e7751b7100992fbafb2c should have been necessary–i.e. why did the calls to `block.render[STUDENT_VIEW]` break after #31472 merged?