Muammar El Khatib

Results 121 comments of Muammar El Khatib

@fornellas thank you very much for your work here. I also look forward to knowing how to solve this issue.

@emontnemery thanks for your reply. That is what I recently implemented and reduced the delay to ~6 seconds.

I have tried this and it does not work. The `blocking=True` option seems to be for older pychromecast versions. The piece of code that I use is this: ``` media_controller.play_media('http://'+localip+':5000/stream',...

Is there any way to get the optimized cartesian coordinates instead of the initial?

> This pull request **introduces 5 alerts** when merging [97c63cc](https://github.com/MolSSI/QCEngine/commit/97c63ccb28d32bf725d97c5afe0e1d138b4e5ca1) into [f9fea80](https://github.com/MolSSI/QCEngine/commit/f9fea80e012ecebb51e9b44c37b270f070200526) - [view on LGTM.com](https://lgtm.com/projects/g/MolSSI/QCEngine/rev/pr-a1fa87ba11cadf81a17612e141e65b584b6bb064) > > **new alerts:** > > * 2 for Unused local variable > *...

@dgasmith would it be possible to have the first review to know what steps to take forward? The only check not passing right now is the CI. I would not...

Yes, something like changing this function https://github.com/MolSSI/QCFractal/blob/5ac50f9a7aad70579d23679ee76cb291c4679737/qcfractal/postgres_harness.py#L48 to check if `pg_ctlcluster` is also there and use it instead of `pg_ctl`.

> You seem to have a good handle on this and can test out your solution on Debian, would you mind making a PR? > I can do that. I...

> @muammar Gentle reminder here, we will likely be putting out a release sometime tomorrow. I was working on this yesterday, and just did a commit you can see here:...

At the point where I made the change, ``` diff --git a/qcfractal/postgres_harness.py b/qcfractal/postgres_harness.py index f47c400..c3eecf9 100644 --- a/qcfractal/postgres_harness.py +++ b/qcfractal/postgres_harness.py @@ -166,6 +166,7 @@ Alternatively, you can install a system...