conda env throws an error on OpenCV

ModuleNotFoundError                       Traceback (most recent call last)
...
...
----> 3 import opencv as cv2
4 from tensorflow import image
5 from tensorflow.image import combined_non_max_suppression
ModuleNotFoundError: No module named 'opencv'

@AvrahamOhana

the syntax for importing opencv is:
import cv2
the conda env works with open cv:

the error remains even after changing to: import cv2


Random change to download URLs without redirection

In order to build historic software for testing, we have to use specific ancient OpenSSL versions like 1.0.2. We fetch these from the source archive at https://www.openssl.org/source/, eg https://www.openssl.org/source/1.0.2/openssl-1.0.2u.tar.gz.
The download path to these ancient source releases has suddenly acquired /old/ in the path, but no redirect was set, thus breaking links to the source bundles for everyone who had used the original URL.
Obviously you can withdraw any download if you wish , and putting /old/ in the canonical URL seems fine and reasonable, but not setting a redirect is very unhelpful and breaks our attempt to maintain access to the desired archive with curl -L ....
Please don't do this again.

This is in the process of being fixed now - a few things were missed in the site change over that are being resolved as quickly as we can.

Thanks. At least you didn't blue-screen 8.5M Windows machines!
Is there an ETA, or should I just commit my work-around?

Actually, the /1.0.2/ is new too.

The issue has been fixed. In your case, you should use:
https://www.openssl.org/source/openssl-1.0.2u.tar.gz
https://www.openssl.org/source/old/1.0.2/openssl-1.0.2u.tar.gz
I think this was never working https://www.openssl.org/source/1.0.2/openssl-1.0.2u.tar.gz (according to directory structure)

Yes, that was a typo. Thanks for the very rapid response.

Thanks for the quick feedback on the issue!

And I confirm that our original workflow ran successfully after your update.


Blank tiles in related search (17 claim search results vs 20 tiles)

good example here - got a few different versions of this bug with refreshing.
One example had 17 results, but would try to who 20 tiles.
https://odysee.com/@Etheve01:3/Au-delà-des-nuages-Odysee:3

Basically recsys is returning spent claims, and the frontend is failing to hide them if not resolved (via claim search)


[WIP] Cleanup and improve GitHub Reporting

Goals:

Make code simpler to maintain
Add stores of issue comments and other missing pieces of data

$ http http://localhost:5000/ossbot-f0cad/us-central1/GetRepoReport?repo=firebase-functions
HTTP/1.1 200 OK
cache-control: no-cache, no-store, must-revalidate
connection: close
content-length: 137
content-type: text/html; charset=utf-8
date: Tue, 04 Dec 2018 21:35:05 GMT
etag: W/"89-MYJ/plSothv1OLlHuduHmtu70v8"
expires: 0
pragma: no-cache
x-powered-by: Express

{
    "forks": {
        "after": 67,
        "before": 67,
        "diff": 0
    },
    "open_issues": {
        "after": 27,
        "before": 41,
        "diff": -14
    },
    "stars": {
        "after": 444,
        "before": 442,
        "diff": 2
    }
}

Example email report for firebase-tools:


[Multi-Spec] Load markdown pages

We had a bit of a conversation about all of this here: https://github.com/cloud-annotations/docusaurus-openapi/issues/82
You can draft up an rfc if you want to get feedback on your approach before opening a PR

@bourdakos1 I was trying to take a look at this but it seems to me that we'd have to duplicate a lot of logic from docusaurus-plugin-content-docs. Do you think there is a way of importing that functionality, or making the two plugins "talk"?


Problem with project references on .NET Core 2.0

Ionide doesn't see library reference on .NET Core 2.0 solution with two projects.
Repro steps

Create solution with two projects: classlib and console

mkdir netcore2-vscode
cd  .\netcore2-vscode\
dotnet new sln
dotnet new classlib -lang f# -o .\lib\
dotnet new console -lang f# -o .\app\
dotnet add .\app\app.fsproj reference .\lib\lib.fsproj
dotnet sln add .\app\app.fsproj
dotnet sln add .\lib\lib.fsproj
dotnet restore

Put following code into .\app\Program.fs

open System
open lib

[<EntryPoint>]
let main argv =
    Say.hello "World"
    0 // return an integer exit code

Build and check that's everything is OK

dotnet build
dotnet run --project .\app\app.fsproj

Open solution in VSCode

Expected behavior
Everything is fine. No errors in VSCode.
Actual behavior
Errors in VSCode:

Problem reading assembly 'c:\Users\m1k3y\Documents\netcore2-vscode\lib\bin\Debug\netstandard2.0\lib.dll': Build was not evaluated, expected the results to be ready after 'Eval'.'
The namespace or module 'lib' is not defined.
The value, namespace, type or module 'Say' is not defined.

Sorry for translated version of VSCode.
Related information

Operating system: Windows 8.1
.NET Core 2.0
Visual Studio Code 1.15.0, Ionide 2.32.2

Please find attached repro: netcore2-vscode.zip
Thanks!

CC: @enricosada

Issue is netstandard2.0 intellisense doesnt work atm, netcoreapp2.0 works.
We need to upgrade FSC used by FSAC. ref https://github.com/fsharp/FsAutoComplete/issues/180

Closing this one since it's tracked on FSAC repo.

OK, thanks a lot!

I'm getting this issue on netcoreapp2.1 had to downgrade to netcoreapp2.0.

I had this issue just now after changing one project from netstandard to netcoreapp.
Doing a git clean -xdf and a full rebuild seems to have sorted it.


http_callback blacklisted with no way out

Hello,
After a network modification I broke the callback link between marathon and marathon-lb. Marathon detected the callback was unsuccessful and decided to blacklist it (well, honestly that doesn't look right to me at all but it's up to you).

May 25 08:47:21 pssu-host01 marathon[6023]: [2016-05-25 08:47:21,810] INFO Will not send event deployment_step_success to unresponsive hosts: http://proxy.domain:8181 (mesosphere.marathon.event.http.HttpEventActor:marathon-akka.actor.default-dispatcher-14)

However, after fixing the issue I found no way to get the callback working again. Triggering marathon-lb restart make it subscribe again but Marathon keep refusing to run the callback, thanks to the blacklist.
I guess the only way out is to restart marathon....
Best regards, Adam.

 Marathon will retry this callback after the back off time automatically.


unix: don't always unlink unix socket on bind error

See this mailing list post for context.  Looking at uv_pipe_bind(), I don't think it should indiscriminately unlink the socket on error; the socket may have been created (and be in use) by another process.

#109 landed, closing!


Warning messages

create functionality in contact.jsx that displays a warning text to the user if they have not fill out the contact form correctly. should display three different message depending on which field is not complete correctly.

DONE!


Migration to netstandard 1.4

Migrating CoreUtilites to netstandard1.4
Adding a platform abstraction layer, to implement platform specific requirements

Hi @mayankbansal018, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Mayank Bansal). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Remove ProjectGuid from PlatformAbstractions csproj.

@mayankbansal018 please ensure sign.proj includes the new assemblies. It will break TestPlatform.CI.Real otherwise.


typos: init at 1.1.9

Motivation for this change
package typos a source code spell checker.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Thank you for the suggestions @SuperSandro2000!


MTurk qualification list sync via S3

Description
This script allows users to sync their local Mephisto DB qualification lists with txt files stored on S3.  It assumes that S3 credentials are stored in the environment.  It also currently will only add qualified workers to both local and S3 lists; it does not assume that one is the source of truth over another and therefore will not remove workers.
I've tested this script in my own environment but would appreciate someone else running it as well.
Type of change
Please check the options that are relevant.

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] Proposes a change (non-breaking change that isn't necessarily a bug)
[ ] Refactor
[X] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[ ] Documentation only change
[ ] Datasets Release
[ ] Models Release

Type of requested review

[ ] I want a thorough review of the implementation.
[X] I want a high level review.
[ ] I want a deep design review.

Before and After
Previously qualification lists were not shared, or shared by hand and imported with one-off scripts.  This is a general framework for syncing that can expand to include more qualification lists as it becomes relevant.
Testing
I've tested this pretty well on my end, including some edge cases (quals and files not existing), and adding workers to both local and shared lists.
Checklist:

[X] I have performed manual end-to-end testing of the feature in my environment.
[X] I have added Docstrings and comments to the code.
[ ] I have made changes to existing documentation where needed.
[ ] I have added tests that show that the PR is functional.
[X] New and existing unit tests pass locally with my changes.
[X] I have added relevant collaborators to review the PR before merge.
[ ] [Polymetis only] I have verified that all scripts in tests/scripts runs on hardware.

Codecov Report

Merging #939 (84b9ddd) into main (9c0d326) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #939      +/-   ##
==========================================
- Coverage   32.27%   32.26%   -0.01%
==========================================
  Files         346      346
  Lines       34145    34145
==========================================
- Hits        11020    11017       -3
- Misses      23125    23128       +3

Impacted Files
Coverage Δ

agents/_agent.py
43.44% <0.00%> (-8.14%)
:arrow_down:

interpreter/craftassist/tasks.py
74.45% <0.00%> (-0.55%)
:arrow_down:

agents/droidlet_agent.py
51.57% <0.00%> (+8.13%)
:arrow_up:


Allow setting the host parameter

Hi, thanks a lot for this project!
This contribution adds the ability to set the host parameters, as requested in #1. I had the exact same issue, and decided to fix it.
Maybe obvious, but this fix is contributed under the same license terms as this project.
If this is not something you'd like to include, feel free to close it.

Great idea. Would you mind also allowing a default value to be set in the config file like the port? We could set it to localhost by default and include a comment.
https://github.com/sobjornstad/tzk/blob/35ca2b896e90e1570b155e6e18c801ae6ee3ce04/tzk/default_config.py#L33

Sure! I'm not too familiar with Python though, these two PRs came about with quite some effort :)
Is it just a question of adding a line like this?
# Documentation...
host = 'localhost'

The __getattr__() magic method of the ConfigurationManager (here) will make host available as an attribute of cm() if it's in the config file. But then you need to use that property as the default value for the command-line option, so it will get picked up if no value is specified on the command line. Look at how the --port option does this here:
https://github.com/sobjornstad/tzk/blob/35ca2b896e90e1570b155e6e18c801ae6ee3ce04/tzk/__main__.py#L90
It appears that in what you have right now, you already copied that line without understanding what it did. So I think all you need to do is change the name of the attribute you're looking for on cm() to host and add the host property to the default config.

Looks good, thanks!


Corrected the Norwegian language

Corrected spelling mistakes & matched letter cases.
Using source branch staging!

Conflicts :)


Added new cypress github action within .github/workflows/

Added new cypress github action YAML file to .github/workflows/
I am also testing the github pull request process.

Checks failing is ok - I'm just following the github actions guide from here https://docs.cypress.io/guides/continuous-integration/github-actions


Instructions on compiling rubyc yourself?

Ultimately my goal here is to see if I could cross compile from x86_64 to arm, all while having a single executable. I don't see instructions on building rubyc itself, which may be just a rubyc package of the bin/rubyc command.
If I wanted this to work with cross compile targets, do you have any hints?
Also, great work. This is a very cool project.

FWIW, rubyc doesn't compile on Debian 9 on ARM (without cross compiling).

@stahnma wondering if you got this working?  We could compile on our arm board directly if needed.

I added a Rakefile and some build instructions to the README ↑
Please open a new issue if you find problems.


Detailed documentation updates

[ ] Explain request cancellation behavior in useQuery / abort controller / etc
[x] Clearly explaining behavior of invalidation/provides and different possible scenarios - see https://github.com/rtk-incubator/rtk-query/pull/39#discussion_r525339990
[x] Show transformResponse usage - currently in graphQL example
[x] Add info about serialized values in the store, especially when using transformResponse
[ ] Offline support documentation?
[x] Auth

[x] Local Storage (skipping due to bad practice, but a user can see where to do it from the token example)
[ ] Cookies
[x] Token from store

[ ] Do a small writeup focusing on existing redux users and how they can leverage this

[ ] Hooks encapsulation to emulate common redux selectors patterns

[ ] Document timestamps fulfilledTimestamp and startedTimeStamp
[x] More baseQuery details with examples

[x] Axios custom base query?

I'm closing everything here now. If there's any need for further discussion, that should take place in a new issue over at https://github.com/reduxjs/redux-toolkit/


[Gluten-core] Validate Parquet compression codec for Velox backend

What changes were proposed in this pull request?
Based on file metadata, validate Parquet compression codec for Velox backend.
How was this patch tested?
Testing on CI.

@zzcclp looks a little change to CK.

@zzcclp looks a little change to CK.

Yeah, I will raise a another pr to fix this issue first.

@jinchengchenghh Would you like to review again? This PR only conducts a refactor now.


what am I doing wrong

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
he don't start
📝 Error Message(s) or Log

🐻 Uptime-Kuma Version
21.6.2
💻 Operating System and Arch
Termux
🌐 Browser
Firefox
🖥 Deployment Environment

Runtime:
Database:
Filesystem used to store the database on:
number of monitors:

21.6.2

We have not published such a version.
The error message pretty clearly states that android is not a supported platflorm. It is not clear what you are trying to do..


Cashbook transactions v2

New version of the cashbook transactions API we decided to re-design using:
https://github.com/ministryofjustice/money-to-prisoners-api/blob/Cashbook-transactions-API-design/Cashbook-transactions-API-design.md

@SteveMarshall @ibrechin code review? Please don't merge it just yet as the cashbook app is not ready

@SteveMarshall @ibrechin made the suggested changes, does it look OK now?

All fine with me.


test: automated cases for Assets page account page tc1484 tc1288

Added automation tests for tc1484 and tc1288

@pcheremu could you please review the PR?
Thanks

@abilevych could you please merge the PR in case all is OK?
Thanks

@abilevych could you please review again?
All the mentioned issues seems to be fixed in latest commit


Trying to write parquet file in parallel results in corrupt file

Describe the bug
(from the mailing list)
Apparently, you can make a program that appears to write a parquet file in parallel, but it will   currently produce corrupt parquet data.
To Reproduce
Description in the email says:

I was attempting to build a single Parquet from the batches in what I thought was a parallel manner using the ArrowWriter.  I tried to "parallelise" the following serial code.

            let cursor = InMemoryWriteableCursor::default();
            let mut writer = ArrowWriter::try_new(cursor.clone(), schema, None)?;
            for batch in batches {
                writer.write(batch)?;
            }
            writer.close()?;

I realised that although the compiler accepted my incorrect parallel version of this code, it in-fact was not sound which caused the corruption.

Expected behavior
The API should not allow corrupted data / produce a compiler error
Actually writing a parquet file in parallel is tracked in https://github.com/apache/arrow-rs/issues/1718
Additional context
Mailing list https://lists.apache.org/thread/rbhfwcpd6qfk52rtzm2t6mo3fhvdpc91

Note there is a proposed PR for this: https://github.com/apache/arrow-rs/pull/1719


v0.3 regression, full_finetune_distributed slower ?

The recipe full_finetune_distributed
Appear to be much slower in v0.3 than v0.2.1
Everything seems to work as usual, but my job that used to work in v0.2.1 time out in v0.3.0.
I don't have much detail yet, but maybe as you are more familiar with the code base you could have an idea already based on what changed recently!

Can you share a few more details around which models you're using, size of dataset, machine type?

I tried on

8xA100 and 8xH100,
model is torchtune.models.llama3_1.llama3_1_70b
dataset is torchtune.datasets.alpaca_dataset

Hey @Delaunay - I looked into this and was able to repro! Unfortunately, still digging into the root cause, but a quick fix is to upgrade your PyTorch version to the nightlies.
pip install --pre torch --index-url https://download.pytorch.org/whl/nightly/

After doing this, training should be fast again:

I think @felipemello1 added some warnings about this in #1733, seems like the fix is to run on PyTorch nightlies here. Given that we have a resolution for this I am gonna close the issue, but @Delaunay if you are not unblocked please feel free to reopen.

I downgraded to 0.2.1 while I wait for pytorch to release a new version, in my case I cannot use nightlies.
But if it OOM why did I not see the error being raised ?
Is it linked to the CPU offloading where things get moved out to CPU to avoid OOM and it gets super slow but eventually OOM ?

@Delaunay since you can't use the nightlies I'll reopen this. The main change is that between 0.2.1 and 0.3.0 we moved onto FSDP2. Since this is a relatively new feature, it's likely that there have been some optimizations made since 2.4 (I can't pinpoint anything offhand but can dig in further here). Out of curiosity, where is it slow? Is it during training, checkpoint load, or somewhere else? And do you see similar slowdowns when running on smaller models (i.e. ones where we aren't doing CPU offload)?


Peer to Peer Code Review

Hi there! Awesome project! We only suggest the following:

Kindly add some comments to your code explaining each part of it. It would be more documented.
You can follow AAA pattern in some sections of your code. For example, defining constants before the functions would be easier to read and it will stick to best practices.

Hello @BahirHakimy. @yosaddis ask us to leave the issue in this repository

Hello @BahirHakimy. @yosaddis ask us to leave the issue in this repository

Oh, Okay


Create in-browser Live Development infrastructure based on postMessage

Brackets recently started to evolve its LiveDevelopment feature to work in multiple browsers.  The new architecture allows the editor to talk to a browser over a transport layer (currently WebSocket based) and communicate bi-directionally between the preview and editor document(s).
Our goal with Mozilla's editor for Thimble is to leverage this new architecture in order to create an in-browser editor with an in-browser livedev preview which substitute an iframe-based browser, a postMessage-based transport layer, and a nohost-based Blob URL Object server.
  I've started to make a list below, and I'm CC'ing @busykai for additional thoughts.  This list is incomplete, but should get us started.

LiveDev needs to default to multi browser (i.e., the livedev.multibrowser pref should be true):

https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/main.js#L77-L80
We can probably use PreferencesManager.set() in an extension to flip this to true at startup.  We just need to make sure it happens early enough (i.e., before AppInit.appReady) so as to not miss the multibrowser init here:
https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/main.js#L303-L314

Launch/open a browser to display the preview need to be altered:

https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/LiveDevMultiBrowser.js#L517-L527
https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/MultiBrowserImpl/launchers/Launcher.js#L47
We need to change that to not use Node, but to start the iframe-based browser.  Maybe we should override the launcher, or maybe we should implement the various “open browser” bits of the appshell:
https://github.com/humphd/brackets/blob/bramble/src/thirdparty/browser-appshell.js#L74-L79

We need to create a LiveDevServer that implements enough of the LiveDevelopment/Servers/BaseServer base class. The FileServer is an example:

https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/Servers/FileServer.js
We can do this in an extension.  We then need to register this server with the LiveDevServerManager such that it has a high enough priority to always get used (e.g., 9001, which is higher than 0 or 99):
https://github.com/adobe/brackets/blob/14370aedfb6fdb7416a53cb0b7dc8d1a7bd7a5d4/src/LiveDevelopment/LiveDevelopment.js#L1482-L1484

We should be able to alter how URLs are created to our server, and instead create Blob URL Objects (i.e., override the pathToUrl method):

https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/Servers/BaseServer.js#L92-L112
One issue to ponder is that we'll need to do this async (i.e., read path from fs), and pathToUrl is sync.  The URL gets used with the launcher here:
https://github.com/adobe/brackets/blob/14370aedfb6fdb7416a53cb0b7dc8d1a7bd7a5d4/src/LiveDevelopment/LiveDevMultiBrowser.js#L537-L541
We can steal code from nohost to create the Blob URLs:
https://github.com/humphd/nohost/blob/master/src/handlers.js#L510-L530

We need to switch the MultBrowser LiveDev from a web socket transport to postMessage.  This will require changes to both the editor side and remote (i.e., preview) side:

Editor side - https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/MultiBrowserImpl/transports/NodeSocketTransport.js

Remote/Preview side - https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/MultiBrowserImpl/transports/remote/NodeSocketTransportRemote.js

@busykai if you have pointers or thoughts on how we should approach this, we'd appreciate any guidance.  Especially if our work can be generalized so it can be upstreamed to Brackets proper vs. just used in our editor fork.

One question I have is how best to abstract the transport and launcher, such that we can use postMessage and iframe versions instead of ws and default browser.
In https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/LiveDevMultiBrowser.js#L803-L817 the default transport is assumed to be ws-based.  With the protocol I can later call setTransport on the protocol and swap it out; but it would be nice if we could pass it in via the config options.
The launcher is more difficult, since we'll have no choice but to override it in code: https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/LiveDevMultiBrowser.js#L88.  It would be nice if this was configurable.

@humphd, thanks for cc'ing me. I'll need to read carefully through this, collect some context and comment in details later (soon), but, at a glance, we had very similar ideas on how it Live Development should evolve further, so upstreaming support for your use case is the right path.

CC: @sebaslv

Started in on this here https://github.com/humphd/brackets-browser-livedev.  Anything that doesn't need to live in Brackets proper can go in this extension, while we experiment.
I've got the basics of the Blob URL server done (untested, ported from my nohost work).  I'll do more this week to integrate the iframe browser, postMessage transport, etc.

@humphd, current version of Thimble seems to use plain CodeMirror, right? So are planning to move it to Brackets?
Anyways, I assumed a couple of things, here are my thoughts which I hope you find helpful.

LiveDev needs to default to multi browser (i.e., the livedev.multibrowser pref should be true):
We can probably use PreferencesManager.set() in an extension to flip this to true at startup. We just need to make sure it happens early enough (i.e., before AppInit.appReady) so as to not miss the multibrowser init here:

There's a AppInit.extensionsLoaded() which seems the right moment to flip it. Additionally, you could define it in the "session" scope of the PreferencesManager to make it is of highest priority.

Launch/open a browser to display the preview need to be altered:
We need to change that to not use Node, but to start the iframe-based browser. Maybe we should override the launcher, or maybe we should implement the various “open browser” bits of the appshell:

It seem like (from looking at Thimble) your LivePreview would always be there, i.e. the lifecycle of the editor is tied to the lifecycle of the LivePreview. If that assumption is correct, you may consider running LivePreview server always, right from the start-up. Then your Launcher would be a postMessage telling the iframe to open the LivePreview server end-point.

We need to create a LiveDevServer that implements enough of the LiveDevelopment/Servers/BaseServer base class. The FileServer is an example:
https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/Servers/FileServer.js
We can do this in an extension. We then need to register this server with the LiveDevServerManager such that it has a high enough priority to always get used (e.g., 9001, which is higher than 0 or 99):

I believe for your purposes StaticServer (in src/extensions/default/StaticServer) is a much better example and you could leverage great deal of it (replacing express with your nohost server). StaticServer is a better example because it serves the content of currently opened documented. It is fundamental to be able to apply Live Preview to html documents. You will need to serve in-memory (in-editor) representation of the document rather the from a file.
Notes on the transport:

you'll need to create your postMessage-based implementation for the remote transport as well. transport has a getRemoteScript() which should return plain remote script in the textual form to be injected in the main html document.

transport load could be done using RequireJS paths definitions in a the same way it is done for the filesystem implementation here. it does not seem that (at least so far) two transport implementation need to live together. If we'd load it with require-configured path, no further changes would be needed.

Other notes:

We have plans for decoupling launch a browser from launch LivePreview. At the moment, Live Preview is launched with the launch of the first browser and is closed when the last one disconnects. Another model is needed to properly support multiple browsers. Currently they are coupled and it is unfortunate. However, like I pointed out before, you could launch LivePreview right at the beginning and then your iframe-based LivePreview would be connected to it permanently (given I'm assuming correctly) for the entire duration of the editing session.

Your comments are duly noted and we will take them into account and plan on them. Especially, ability to specify the transport implementation by configuration.

Let me know if I missed any of your questions. We'd be happy to stay up-to-date on your advancements and keep the implementations in-sync. Hope it all makes sense.
CC: @sebaslv

@humphdhttps://github.com/humphd, current version of Thimble seems to use plain CodeMirror, right? So are planning to move it to Brackets?
Correct.  Our plan is to rip out the raw CodeMirror instance and separate preview iframe, and do it all via a single Brackets based iframe with this modified live dev stuff.  We already have Brackets working in the browser with a filesystem based on IndexedDB.
Anyways, I assumed a couple of things, here are my thoughts which I hope you find helpful.
Thanks for all your thoughts, they align with guesses we made as we planned things out.  Our current thinking is to try the following flow:

Flip the pref for MultiBrowser LiveDev once extensions are loaded (AppInit.extensionsLoaded()), changing AppInit.appInit() here - https://github.com/humphd/brackets-browser-livedev/blob/master/main.js#L23

Listen once for the statusChange event with a ”STATUS_INACTIVE" on the LiveDevMultiBrowser object in our extension, and start our work here: https://github.com/humphd/brackets/blob/bramble/src/LiveDevelopment/LiveDevMultiBrowser.js#L816  As soon as this happens, wire the transport layer using setTransport(). Probably this means showing the iframe browser now, so transport has a ref.

Wait for our document to be loaded form Thimble in an editor.  Maybe listen one-time for projectOpen on ProjectManager? https://github.com/humphd/brackets/blob/bramble/src/project/ProjectManager.js#L895.  At this point we can trigger LiveDevMultiBrowser.open(), which will start our server and launch the browser.

We have to hack Launcher.launch(url) to do something like this:

function launch(url) {
    // For this to work, you might need to expose some "globals" (e.g., server/nohost) on appshell
    // We have server and LiveDoc
    // Get LiveDoc contents, feed through nohost.serveHTML to get Blob URL object
    // Give iframe the new URL to display
}

This will connect the preview back to the live dev instance via postMessage based protocol messages.
Let me know if I missed any of your questions. We'd be happy to stay up-to-date on your advancements and keep the implementations in-sync. Hope it all makes sense.
For sure, we will.  My goal is for us to have a version of this working by the end of the week, which should help us figure out what we still don’t know, what could be done through better abstraction, etc.
I’ll report back when we have something.

@humphd We have this working now (huzzah!). Should we close?

Let's leave open while we track https://github.com/adobe/brackets/issues/10558 and https://github.com/adobe/brackets/issues/10634 upstream.

@humphd Since we aren't waiting on upstream anymore, I'm going to close this.


BLE and timer capture

Hi OpenWCH team,
After ADV event I start capture using timer. In case of very small (10..20us) pulse duration I have timer interrupt invoked when the capture is completed.
But if the signal is long (1ms) - I never got the interrupt.
I assume that BLE stack pushing CH583 to sleep mode before capture completed and this why I don't get timer interrupt.
To solve the issue, I can disable sleep before the measurement and enable after.
Please, advise is the a way how I can temporarly disable sleep mode?

https://github.com/openwch/ch583/blob/aaa5f9839cefc3794bc8391f0207fc1568ddeb18/EVT/EXAM/BLE/HAL/SLEEP.c#L26
This is a callback when TMOS is idle, you can return 2 if you dont want sleep


Allow skipping YouTube ads

Well this needs lots of work but is possible. My api uses youtube-dl to get this info. Eg:
https://api.videokit.io/v1/info?url=https%3A%2F%2Fwww.youtube.com%2Fwatch%3Fv%3DC0DPdy98e4c

url field should point to the direct mp4 without any adverts


SingularTable setting is forgotten after using Table().Updates()

Code:
db.SingularTable(true)
db.Table("account").Where("last_online IS NULL").Updates(Account{ LastOnline: now })

account := &Account{}
db.First(&account)

Result:
no such table 'accounts'
Adding db.SingularTable(true) after the line which calls Updates() works around the problem.

seems works now.


Create pull request template

What
Added markdown pull request template to the project. Followed these GitHub instructions.
Why
To ensure we capture the what/why of our pull requests. This will help provide context for changes to the codebase which can be really useful when reviewing code history.

This implementation is not working and GitHub has conflicting documentation on implementing templates:
"Creating pull request templates":

Templates are available to collaborators when they are merged into the repository's default branch.

vs.
"About issue and pull request templates":

You must create templates on the repository's default branch. Templates created in other branches are not available for collaborators to use.

I believe I will have to re-do this work on the default branch, not as a PR.


Adding netcgo build tag

LGTM!
No errors, no warnings. :shipit:

  Generated by :no_entry_sign: Danger

Codecov Report

Merging #750 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #750   +/-   ##
=======================================
  Coverage   55.77%   55.77%
=======================================
  Files         264      264
  Lines       15432    15432
=======================================
  Hits         8607     8607
  Misses       6100     6100
  Partials      725      725

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 58add07...68e309a. Read the comment docs.


Add local SSH key when renting a new instance on vast.

For some reason it adds ssh keys for me but not others. I can force it to just add the local key.

https://github.com/Metta-AI/metta/pull/19


Added environment, but no data

We think
this project is a great idea :)
Steps to reproduce
We have deployed the app, and are able to login.
We have added an environment, but we can only see a line with the description of the environment.
Shouldn't we see the data/dashboard after adding the environment?
Expected behavior
A dashboard with data
Actual behavior
A list of added environments

You use PowerBI to access Cosmos DB and create your desired dashboards that way. https://docs.microsoft.com/en-us/azure/cosmos-db/powerbi-visualize

How do you know if your environment settings are correct and currently downloading data?  I am able to connect via my PowerBI and see Documents in there only.

Same, I only see Documents

@lukeapathy thank you for opening this issue. I am in the process of retiring this project because it is being replaced by Security and Management Insights Connector. With this in mind, I would like to encourage you to review that project and open any feature request if there are additional items you are interested in seeing.


Memory leak on nodeJs strongloop strong supervisor

Our node supervisor is having memory leak and it's getting crashed. For below memory it takes around 2 weeks before we get OOM error.
// boot script
start() {
cd ${APPDIR}
mkdir -p ${LOGDIR}
killall node
sleep 5
rm -rf ${APPDIR}/.tmp
(slc run --no-profile -d --cluster <%= @clusters %> --log ${LOGDIR}/${APP_NAME}.log -p ${LOGDIR}/${APP_NAME}.pid ${APPDIR}/${APP} --port **** --max-old-space-size=8192)    RETVAL=$?
//strongloop
/user/lig/node_modules/strongloop/nod_modules/strong-supervisor/bin/sl-run.js --no-profile --no-detach--cluster cpus --log/log
it runs on AWS m3.large 7.5 GB ram. There are 3 instances in cluster
Any suggestions are appreciated

Hi! We can certainly help, but with no known memory leaks in Express and without any code, I'm not sure how we can help.
We need the following information in order to proceed:

Version of Express you are using
Version of Node.js you are using
An entire application we can run that reproduces the issue. Try to keep the number of requires as small as possible, as anything can introduce a memory leak. Ideally the only require statement in your entire app is for express.

Version of Express you are using : 4.13.3
Version of Node.js you are using  : v0.10.39
Not sure how to share that info

Hi @vinojoy , if you cannot paste the application in this field, then you hve too much going on. You should trim your application down to something small, only using Express, to demonstrate that the memory leak actually is coming from Express and not something in other dependency, your code, or the interaction between things.
For example, your issue here is that there is a memory leak in Express. I assume that you have done all the debugging, which has pointed to it specifically being in Express. Can you at least share your findings that point to the issue being Express?
Without being able to see the memory issue ourselves, we are in the dark and there is pretty much no action we can possibly do on this issue. I know you said "Any suggestions are appreciated", but I'm not sure what suggestions I can give. Are you looking for something in particular? I have never used the strong-supervisor module you are showing above and it is not even part of Express.

@dougwilson issue seems to  be on strong supervisor which is part of strongloop

i will be moving this thread to strong supervisor

Ok. I'm closing this as a question, but I can always re-open if you have additional details.

Thanks
Regards,
Vino Joy
On Tue, Dec 15, 2015 at 12:34 PM, Douglas Christopher Wilson <
<EMAIL_ADDRESS>wrote:

Closed #2826 https://github.com/strongloop/express/issues/2826.
—
Reply to this email directly or view it on GitHub
https://github.com/strongloop/express/issues/2826#event-492702941.


Add debugging steps for no variables defined

 You should sign our Contributor License Agreement in order to get your pull request merged.

Hello @danielcompton,
thanks for adding this debugging instructions to the docs. I've merged your additions as 974c4310fcdc74d9933499e730e9b7a9529e25d3 (v0.15.docs) and 153dccc0e8c939fcbbb71afbb268817dbe704faa (master).
Cheers,
Digitalcraftsman


Leverage the Hostname of the Server

Historic versions leveraged a format of "server_$UUID" to name of the VPN server certificate for X509 verification.  This seems very impersonal.  The new code pulls the existing hostname of the machines and appends the 16 character UUID.  The new format is $hostname_$UUID.
Example:
A machine named "Martian" with a UUID of<PHONE_NUMBER>123456 would change from server_1234567890123456 to Martian_1234567890123456

Looks good to me @orazioedoardo @Giraffe1966 @DevZer0
What do you think of this?

i'm not sure if it's a good idea to feed binary data to tr, as for my knowledge it's a text processing tool
i'm getting this on osx for example
$ cat /dev/urandom | strings | tr -dc 'a-zA-Z0-9' | fold -w 16 | head -n 1
tr: Illegal byte sequence

@4s3ti Looks good to me.

I like the uuidgen approach, makes the code easier to read and understand.
@IcedComputer would you mind to make the necessary changes to use uuidgen instead?
should also be added as a dependency here: https://github.com/pivpn/pivpn/blob/8e3a95152412a9d35b67d51bfc80379350815252/auto_install/install.sh#L24


Static analyzer checks

Select your issue type: (check at least one)

[ ] Bug
[ ] Question
[x] Suggestion
[ ] Other (please describe):

Describe your issue:

AutoBoolCounter should probably be not copyable link, + there are some extra brackets in the same file.
minor rounding issue link i.e. the integer division will never return non-integer megabytes.
suspicious casts link don't know specific reasoning for this one :smile:

Hi there is another bunch of issues from pvs studio analyzer

https://github.com/antony-jr/AppImageUpdaterBridge/blob/master/src/zsyncwriter_p.cc#L776
Expression '!(constructed = true)' is always false.
https://github.com/antony-jr/AppImageUpdaterBridge/blob/master/src/helpers_p.cc#L15 Expression is always false (QNetworkReply::UnknownNetworkError is 99)
The possible overflows due to the cast were again reported

pvs studio analyzer is amazing @technic

@probonopd
Yes, I am now trying to get nice reports from gitlab-ci

I'm a little busy right now, will look into it when I have more time.

Ok, take your time. In case your are interested there is a PVS analyzer log

We do have codacy for now(just a placeholder for PVS analyzer IMO).

@technic is there a way to get PVS license for free??

@antony-jr Yes, they provide licenses for open source projects for free. However, there might be some additional limits or conditions https://www.viva64.com/en/b/0600/. You could send them an email and ask for it.

@technic Okay I think I got the free license. I tried with my protonmail but never got a license in the past(tried like 5 times). I created a gmail account just for this and I got the license in just few hours. PVS seems to hate secure mails.
I will integrate with PVS on v2.x

@antony-jr Great! But this issue is not about PVS integration, rather it's about fixing those few points where the analyzer throws warnings.

@antony-jr Great! But this issue is not about PVS integration, rather it's about fixing those few points where the analyzer throws warnings.

Actually some of them are intentional as to show off C idioms in the code. But I think readability is important than language idioms.

I will try to fix all warnings and errors given by PVS when I have time. Now you can see the analysis on each commit from the documentation itself. Documentation Link


Move example code out of Publisher API Reference

The reference page should only contain reference information.  If we need examples to clarify, link out to examples.
Reasons:

Maintenance gets easier
Clarity goes up (i.e., each section gets shorter)
Having a uniform "reference-only" format for each section will make it much easier to search and scan

I actually disagree with this. I think examples are good in a reference. I'd leave examples there, I think a better move would be to break up the API Ref so it's not all on one page.


the notification voice should be optional

Describe the solution you'd like
I think it would be nice if you could add a feature to disable that notification voice.
Additional context
It's creep and won't help, anyway.

OK. I just noticed the sound icon.
my bad.


[AppService] az functionapp: Only show warning but not throw error when checking runtime

Related command
az functionapp config appsettings set
az functionapp deployment source config-zip
az functionapp config set
az functionapp config container set
Description
Resolves:
#28394
#28382
#28358
Currently, we throw an error if we are unable to detect the runtime. However, there are many reasons why a linux app might not have a "conventional" runtime and runtime version (e.g. custom apps, centauri apps, apps running a docker image). For that reason, we should not be blocking command execution but should show a warning for how to set linuxFxVersion if we are unable to detect it and should not even show a warning when it is not applicable (e.g. centauri apps, apps running a docker image).
Testing Guide

History Notes

[AppService] az functionapp: Not block execution of command when runtime cannot be detected, and omit showing warning for runtime when not applicable (e.g. centauri apps, apps running a docker image).

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

AppService


Update the sandbox example to use networking not links

Links are being deprecated with the implementation of the new networking material. We need to update the trust_sandbox.md example to use docker networking rather than links.
ping @endophage  @NathanMcCauley @diogomonica

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

was fixed


BIGTOP-2521: Problem with deprecated methods in puppet code with puppet option parser=future

The changes look good, but I feel they will break the BIGTOP-2524 patch and it'd needed to get rebased. I will be pushing this one shortly unless I hear otherwise.

Committed, please close the PR


TypeError: matches.sort is not a function

/node_modules/@npmcli/map-workspaces/lib/index.js:117
matches = matches.sort((a, b) => a.localeCompare(b, 'en'))
^
TypeError: matches.sort is not a function

What's the value of workspaces in the relevant package.json? I'm guessing that value is not an array.
Not sure if it's Knip's job to guard against this.

Closing this due to inactivity.


MINIFI-116 Initial Travis CI configuration for Make based build.

MINIFI-116 Initial Travis CI configuration for Make based build.

This relies on #15 as well.  Updated it to include that change.

@apiri LGTM

once this is merged it should be easier to expand with CMake and different environments. I am particularly keen to ensure we are able to generate both deb and rpm files

@trixpan Yep, sounds good.  Onboard on all counts with that.  Would you like to draw up issues for those packaging formats (maybe one for each for the sake of granularity)?  Definitely getting functional on Windows is a priority.
Also, did you mean to put these comments on #14?  They seem a little more tailored around CMake or maybe I'm just overlooking something.

no no. just thinking about getting travis-ci working asap. Once that is done, evolving the work on #14 - i.e. CMake, windows, etc - should be easier due to the ability of iterating code until we get it right. :-)
Lets get this PR rolling and use it to complete the work started on #14 ;-)

@trixpan Thanks for clarifying, let me do some merge magic and we can get rolling.  Glad to have you join in on the minifi-cpp party! 👍


How to detect entire scenes and not only shot changes?

Hi guys,
I've been experimenting with different scene detection tools, mainly PySceneDetect and Google Video Intelligence. I noticed that what these tools call scene detection is really shot detection. They detect different shots, not scene. A scene, technically, is a sequence of shots, constituted of a setting (a place, time period, environment...), characters, events, dialogue, purpose and so on.
I would like to detect entire scenes. Is this possible? If so, how can this be done?

I think this library is from the Angle of shot view, not from the Angle of scene understanding trigger to cut


Plans for a new release?

I was hoping to use the recursive get/put functionality that was implemented since version 0.2.1, and I was curious if there are plans for a release any time soon.
Funny(ish) story: I actually implemented this in hopes of being a gcsfs contributor then realized it was already in master, just not released! 😅 Not my brightest moment haha.
Thank you for all of the work on this useful library 🙂

https://pypi.org/project/gcsfs/0.2.2/#files and coming soon to conda-forge, probably by this evening


Update how-to-write-links.md

I will delete this PR soon. I am just working through the steps in Edit an article on Docs.

#hold-off


Update Excel.workbookquery.formula.md

The property is read/write.

Docs Build status updates of commit 2e6620b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Excel.workbookquery.formula.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thanks for clarifying! I'm sorry for the delay in merging to master.


[now-static-build] Add support buildCommand, devCommand and outputDirectory

Add support buildCommand, devCommand and outputDirectory.
It further adds node_modules/.bin to the PATH for node projects that have a package.json file.
Also verified that now dev works on windows.
PRODUCT-781
PRODUCT-783

Codecov Report

Merging #3422 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3422      +/-   ##
==========================================
+ Coverage   14.53%   14.56%   +0.03%
==========================================
  Files         267      267
  Lines       10742    10779      +37
  Branches     1480     1481       +1
==========================================
+ Hits         1561     1570       +9
- Misses       9015     9043      +28
  Partials      166      166

Impacted Files
Coverage Δ

src/util/metrics.ts
61.53% <0%> (-6.65%)
:arrow_down:

src/util/prefer-v2-deployment.ts
55.1% <0%> (-4.9%)
:arrow_down:

src/util/projects/get-project-by-id-or-name.ts
19.04% <0%> (-4.49%)
:arrow_down:

src/util/dev/yarn-installer.ts
79.68% <0%> (-3.65%)
:arrow_down:

src/util/output/create-output.ts
61.22% <0%> (-3.22%)
:arrow_down:

src/util/get-files.ts
89.61% <0%> (-0.89%)
:arrow_down:

src/util/get-update-command.ts
9.37% <0%> (-0.63%)
:arrow_down:

src/util/dev/builder.ts
65.29% <0%> (-0.6%)
:arrow_down:

src/util/dev/server.ts
59.77% <0%> (-0.06%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 68deab9...efae807. Read the comment docs.


Small improvement during review

Replaced by https://github.com/camunda-community-hub/spring-zeebe/pull/151


WIP - undeploy command

connects to #27

Coverage decreased (-3.4%) to 67.01% when pulling 07ba877f38e056118438c822ae512289407adf3f on lholmquist:27-clean-deployments-undeploy-command into 1a80634ac4591927b97080e1634bab30844a5efb on bucharest-gold:master.

Coverage increased (+0.2%) to 70.598% when pulling 1b28b6ff7f939317ac0b1658c9feaf22631b8ecc on lholmquist:27-clean-deployments-undeploy-command into 1a80634ac4591927b97080e1634bab30844a5efb on bucharest-gold:master.

Coverage increased (+1.7%) to 72.137% when pulling d2c9f147718c26fdebd73ffe82c92b70cbe3aa4a on lholmquist:27-clean-deployments-undeploy-command into 1a80634ac4591927b97080e1634bab30844a5efb on bucharest-gold:master.

Coverage increased (+3.06%) to 73.504% when pulling fe891fe63950b4ad6e986e97311196b5248815f5 on lholmquist:27-clean-deployments-undeploy-command into 1a80634ac4591927b97080e1634bab30844a5efb on bucharest-gold:master.

Coverage increased (+3.4%) to 73.846% when pulling b9f925f2551a309616a565b1c4f2116cb4ae1dc8 on lholmquist:27-clean-deployments-undeploy-command into 1a80634ac4591927b97080e1634bab30844a5efb on bucharest-gold:master.

Coverage increased (+3.4%) to 73.846% when pulling 5f04b3658f52821671701a2a0b8543701e51fdfa on lholmquist:27-clean-deployments-undeploy-command into 1a80634ac4591927b97080e1634bab30844a5efb on bucharest-gold:master.


Dependency issue: Cannot find module '@truffle/provisioner'

A fresh installation of Sabre 0.10.2 throws an error when running the 'analyze' command.
$ sabre analyze contracts/MetaCoin.sol
Error: Cannot find module '@truffle/provisioner'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:668:15)
    at Function.Module._load (internal/modules/cjs/loader.js:591:27)
    at Module.require (internal/modules/cjs/loader.js:723:19)
    at require (internal/modules/cjs/helpers.js:14:16)
    at Object.<anonymous> (/Users/bernhardmueller/.nvm/versions/node/v11.15.0/lib/node_modules/sabre-mythx/node_modules/@truffle/resolver/lib/resolver.ts:3:19)
    at Module._compile (internal/modules/cjs/loader.js:816:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:827:10)
    at Module.load (internal/modules/cjs/loader.js:685:32)
    at Function.Module._load (internal/modules/cjs/loader.js:620:12)
    at Module.require (internal/modules/cjs/loader.js:723:19)
    at require (internal/modules/cjs/helpers.js:14:16)
    at Object.<anonymous> (/Users/bernhardmueller/.nvm/versions/node/v11.15.0/lib/node_modules/sabre-mythx/node_modules/@truffle/resolver/lib/index.ts:3:1)
    at Module._compile (internal/modules/cjs/loader.js:816:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:827:10)
    at Module.load (internal/modules/cjs/loader.js:685:32)
    at Function.Module._load (internal/modules/cjs/loader.js:620:12)
    at Module.require (internal/modules/cjs/loader.js:723:19)
    at require (internal/modules/cjs/helpers.js:14:16)
    at Object.<anonymous> (/Users/bernhardmueller/.nvm/versions/node/v11.15.0/lib/node_modules/sabre-mythx/lib/controllers/analyze.js:7:18)
    at Module._compile (internal/modules/cjs/loader.js:816:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:827:10)
    at Module.load (internal/modules/cjs/loader.js:685:32)

** Expected behaviour
Should run normally.

Hi @b-mueller , seems like gitcoin is not posting the applications in GitHub, just in case you didn't see it.
I started to look at this and the problem is that the @truffle/provisioner packager isn't getting its files published and when installing @truffle/resolver, the @truffle/provisioner dependency is installed empty.
I've sent a PR to truffle https://github.com/trufflesuite/truffle/pull/3280 to fix it.

Hi @b-mueller, sent this PR with a workaround to fix the dependency issue in the meantime.

This is great, thanks! I'm getting TypeError: Profiler.resolveAllSources is not a function error now but anyway, I'll create a separate issue / bounty for that.

@yacare555 the issue doesn't seem to be updating on Gitcoin, I think you have to change the status to "submitted" somehow? Right now I don't see a way of paying the bounty 😢

Hi @b-mueller, sorry, I forgot the submit to gitcoin, it's done now.
Btw, if you create a bounty for the other issue, count with me to work on it.

Hey @yacare555, I created a new bounty for the issue:
https://gitcoin.co/issue/b-mueller/sabre/106/100023582

Hey @yacare555, I created a new bounty for the issue:
https://gitcoin.co/issue/b-mueller/sabre/106/100023582

Thanks!


Assign environment variable GST_PLUGIN_PATH in development mode

GST_PLUGIN_PATH environment variable must be assigned to /usr/share/kaldi/src/gst-plugin in order to Kaldi GStreamer works.
In standard installation we assign it just before initiating the actual script with this line. (so it's fine)
But in development mode (pip install -e .) none of the files under debian/ are functional. In development mode when you run dragonfire it immediately enters to dragonfire/__init__.py and runs initiate().
We need to find a way to assign that environment variable from inside the Python. I tried to put os.environ["GST_PLUGIN_PATH"] = "/usr/share/kaldi/src/gst-plugin" to several locations but it seems not working. Not sure why though... (I think it needs to be assigned in parent shell)

Fixed with https://github.com/DragonComputer/Dragonfire/commit/03f977a5a4a43c1d328f45cf8c6cb18c4ac83545


Getting google maps native instance

Issue description
Trying to get google map instance to build route and to do other things, which this library does not support yet.  but getting google map instance does not work using getNativeMap() method, other sibling methods works fine, except this one.
Steps to reproduce and a minimal demo of the problem
private gmaps: GoogleMapsAPIWrapper

// getNativeMap promise does not trigger at all
this.gmaps.getNativeMap().then((map) => map.setCenter({
      lat: pickup.latitude,
      lng: pickup.longitude
    }));

@iBasit see my message here (at 15:36): https://gitter.im/SebastianM/angular2-google-maps/archives/2016/09/02#model-id-57c980726bf4c26505c667ed
It's supported right now. For every map, a separate instance gets created.
Closing this for now. Feel free to reopen.

Still, can't manage to do it. I have shared code snippet in the private chat message. Please see if you can help on this issue.

@iBasit Check my solution here, on this thread.


Fix reminder functionality and options page cleanup + Firefox fixes

This should:

Fix the reminder functionality (which I've never seen working before)
Fix the options layout in Firefox when addon is in overflow menu (scroll bars were appearing before)
Improve the options UI slightly with clickable labels

@g60madman @deepfriedfilth Can you take a look?

@deepfriedfilth thanks for verifying...merged


Include unicode-range for all CSS files

Currently, the unicode-range property is only provided for the [weight].css files. I suggest we provide the property for all CSS files so the browser can dynamically filter out unused font subsets.

The only blocker for this is the lack of information on what unicode-range's certain languages cover. The current information we have is insufficient to determine the optimal ranges for each subset as CJK fonts are split into numbered subsets. So we will need to find a way to analyze the font files for the correct information. Would appreciate some suggestions for this.

This is doable if we follow Wakamai Fondue implements its font analysis tooling.
This a gold mine of a website and there's lots to learn from it! I'm sure we can implement a bunch of the features from there too.


 No…

… time to stand beneath the boughs And stare as long as sheep or cows.
First draft of the cellxgene census download notebook, any additions, deletions, substitutions, suggestions, or style items, are appreciated.

Looks very good!

modified some stuff, very little in the cellxene_census scripts.
Did not look at 3-clustering_metrics.qmd, let me know when its done and the html is ready!
added .gitignoreand set it to ignore useless files, let's try to keep it updates so we don't upload unnecessary files that can muddy the pulls (like .DS_Store)

I'll let you merge it @kylekimler !!

@MarcElosua Just added a draft of the clustering metrics notebook. I'm wondering if we should go into sub-clustering there? It looks like a good opportunity!
Thanks for the added .gitignore!
I've also added a bibliographies subfolder, which allows Quarto to use a Zotero .bib - check out cluster_metrics.qmd for examples


Update CountDownLatch2.java

What is the purpose of the change
typo.
Brief changelog
java.util.concurrent.CountDownLatch2 -> java.util.concurrent.CountDownLatch
Verifying this change
ok.

@ThomasLau Could you change your destination repo to develop branch? more details you could see from here.


sidebar update

Description

General Checklist

[ ] Tests added for this feature/bug
if it was a bug, test must cover it.
[ ] Conform by the style guides, by using formatter
[ ] Documentation updated
[ ] (optional) Please add a label to your PR

Pull Request Test Coverage Report for Build 957651764

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Totals

Change from base Build 950606991:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Docker hub documentation

Hi @andrewstevenson,
I have reviewed all links related to kafka-connect-mongodb, but there is no direct guideline of how to use from the images, specifically I need to run kafka-connect-mongodb image. Would you please add some docker-compose.yml file to instruct the unfamiliar users with this technology?
Thanks

@mostafaghadimi have you looked here? https://github.com/Landoop/stream-reactor-dockers
All the connector options need to be added as ENV vars with the prefix CONNECTOR_
https://docs.lenses.io/connectors/sink/mongo.html#connector-configurations


[WIP] Equations in Models

This PR is aimed at several interrelated issues, including: #479 #857 and #889
The idea is to make the definition of the mathematical relationships between variables, 'equations' logically separate from the operational code that checks conditions, solves problems, and runs simulations.
Currently, the mathematical aspects of the model are blended in with the operational code. We've been trying to get away from this for some time.
This PR has one solution and a partial demonstration of how it would work. The Model class takes an equations argument, which is a dictionary. The equations are then used as appropriate. I've demonstrated how this refactoring could work for the assignments of thorn and aNrm.
This is just one way to do it. Here are two considerations:

It wouldn't be too hard to write a formula resolver, so that resolve('thorn') would:

look up the equation for thorn
know from its arguments that it needs the parameters dictionary as input
pass in that dictionary, and
get return the value. (Or a dictionary, with thorn assigned to the value).

Ultimately, we want these equations to be composable and built in such a way that they can be compiled by JAX. Maybe a PyTenson implementation would be more efficient down the line. But it would be harder to refactor the current HARK code to do this without doing the more basic Python version first.

[ ] Tests for new functionality/models or Tests to reproduce the bug-fix in code.
[ ] Updated documentation of features that add new functionality.
[ ] Update CHANGELOG.md with major/minor changes.

Also for reference...
I different way to implement a representation of mathematical equations has already been committed in the frame module. In that work, the equations (called Frame in that code, but really I should  rename that to Equation or remove frames entirely in favor of whatever system replaces is) are objects that include a target value, the arguments to its equation, the equation itself (as a Python function), and/or a flag to say if the target value is a control variable (needing a decision rule to be fully specified):
https://github.com/econ-ark/HARK/blob/master/HARK/ConsumptionSaving/ConsPortfolioFrameModel.py#L114-L231
The equations get evaluated in simulation in the scope of the current state/shock/parameter values:
https://github.com/econ-ark/HARK/blob/master/HARK/frame.py#L591-L670
At the time I wrote that, I had no idea how to integrate equations like these into the solver.  #1283 provides a way to do that.
But also, there are pros and cons to the implementation in frame.py and the version currently in this PR is an alternative.

The failing tests in the ConsAggShockModels are due to the fact that the AggShockModel constructor does not call the constructor of its superclass (IndShockConsumerType), but rather the AgentType base class constructor.
https://github.com/econ-ark/HARK/blob/master/HARK/ConsumptionSaving/ConsAggShockModel.py#L135-L156
This is not how other model inheritance in HARK works, where the superclass is called.
Is this intentional? Or just an oversight in how the ConsAggShockModel works? if the latter, I can easily fix it in this PR.

That looks like an oversight, but there might be some initialization
routine for IndShockConsumerType that's not supposed to be run for an
AggShockConsumerType. Let's try changing the inheritance and see if it runs
properly.
On Wed, Jul 5, 2023 at 10:58 AM Sebastian Benthall @.***>
wrote:

@sbenthall https://github.com/sbenthall requested your review on: #1292
https://github.com/econ-ark/HARK/pull/1292 [WIP] Equations in Models.
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1292#event-9732924376, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKRAFOY5HTVAAWDCWQYCM3XOV6JNANCNFSM6AAAAAAZWHG3XU
.
You are receiving this because your review was requested.Message ID:
@.***>

Let's try changing the inheritance and see if it runs properly.

Ok, I changed it locally and it got tests passing. This change has been committed to this PR.

Tests are now passing. KinkedR had a similar issue with inheritance.
The substance of this PR is now in and ready for review. Requesting review from @mnwhite
I know I need to do tests, docs, and changelog, and will start on that next.

Not sure why this just popped up in my inbox, but it prompted me to look again.
I think thinking about all this is premature.  It will really need to be integrated with whatever structure gets created from Pablo's architecture.   At most, at this point we could perhaps fruitfully build on what is already in dolo or nolib.

It's unfortunate that you've been missing the regular HARK meetings @llorracc I think we're quite resolved that this is a good idea.
Essentially, no matter what Pablo's architecture is, the HARK models will need to be moved in this sort of direction in order to integrate with that. This is a necessary incremental improvement to HARK no matter what.

I'd like to revisit this PR, which is now almost a year old, and perhaps out of date with respect to recent HARK changes. But it was trying to accomplish something that we are still trying to move towards.
The main point of this PR was to try attaching dynamics to the existing ConsumptionSaving models, so that they could be rewritten with a single source of truth about their model equations.
These dynamics, in this PR, looked like this:
IndShockConsumerType_dynamics = {
    **PerfForesightConsumerType_dynamics,
    'G' : lambda gamma, psi : gamma * psi,
    'Rnrm' : lambda R, G : R / G,
    'bNrm' : lambda Rnrm, aNrm : Rnrm * aNrm,
    'mNrm' : lambda bNrm, theta : bNrm + theta,
    'cNrm' : Control(['mNrm']),
    'aNrm' : lambda mNrm, cNrm : mNrm - cNrm
}

This is now essentially the same thing as what's included in the 'block' based consumption model, which was recently merged in:
https://github.com/econ-ark/HARK/blob/master/HARK/models/consumer.py#L26-L44
Now that there's a compact format for model equations already in HARK, it would be a good idea to implement a solution to #1346 and put the equations onto the old models.

Not getting what you mean by adding dynamics.  In economists’ use of the
word these models already have dynamics. They are dynamic, stochastic,
optimization models.
I’m keen on making some basic structural decisions first.
Sent from Gmail Mobile
On Sat, Jun 1, 2024 at 4:43 PM Sebastian Benthall @.***>
wrote:

I'd like to revisit this PR, which is now almost a year old, and perhaps
out of date with respect to recent HARK changes. But it was trying to
accomplish something that we are still trying to move towards.
The main point of this PR was to try attaching dynamics to the existing
ConsumptionSaving models, so that they could be rewritten with a single
source of truth about their model equations.
These dynamics, in this PR, looked like this:
IndShockConsumerType_dynamics = {
**PerfForesightConsumerType_dynamics,
'G' : lambda gamma, psi : gamma * psi,
'Rnrm' : lambda R, G : R / G,
'bNrm' : lambda Rnrm, aNrm : Rnrm * aNrm,
'mNrm' : lambda bNrm, theta : bNrm + theta,
'cNrm' : Control(['mNrm']),
'aNrm' : lambda mNrm, cNrm : mNrm - cNrm
}
This is now essentially the same thing as what's included in the 'block'
based consumption model, which was recently merged in:
https://github.com/econ-ark/HARK/blob/master/HARK/models/consumer.py#L26-L44
Now that there's a compact format for model equations already in HARK, it
would be a good idea to implement a solution to #1346
https://github.com/econ-ark/HARK/issues/1346 and put the equations onto
the old models.
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1292#issuecomment-2143582078, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKCK74KLV3MWV7UU3MWW7LZFIW53AVCNFSM6AAAAAAZWHG3XWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGU4DEMBXHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Just to clarify what I meant: I meant having a representation of the
equations governing how the variables change, which other parts of system
can use.
On Sun, Jun 2, 2024, 3:04 AM Christopher Llorracc Carroll <
@.***> wrote:

Not getting what you mean by adding dynamics. In economists’ use of the
word these models already have dynamics. They are dynamic, stochastic,
optimization models.
I’m keen on making some basic structural decisions first.
Sent from Gmail Mobile
On Sat, Jun 1, 2024 at 4:43 PM Sebastian Benthall @.***>
wrote:

I'd like to revisit this PR, which is now almost a year old, and perhaps
out of date with respect to recent HARK changes. But it was trying to
accomplish something that we are still trying to move towards.
The main point of this PR was to try attaching dynamics to the existing
ConsumptionSaving models, so that they could be rewritten with a single
source of truth about their model equations.
These dynamics, in this PR, looked like this:
IndShockConsumerType_dynamics = {
**PerfForesightConsumerType_dynamics,
'G' : lambda gamma, psi : gamma * psi,
'Rnrm' : lambda R, G : R / G,
'bNrm' : lambda Rnrm, aNrm : Rnrm * aNrm,
'mNrm' : lambda bNrm, theta : bNrm + theta,
'cNrm' : Control(['mNrm']),
'aNrm' : lambda mNrm, cNrm : mNrm - cNrm
}
This is now essentially the same thing as what's included in the 'block'
based consumption model, which was recently merged in:

https://github.com/econ-ark/HARK/blob/master/HARK/models/consumer.py#L26-L44

Now that there's a compact format for model equations already in HARK,
it
would be a good idea to implement a solution to #1346
https://github.com/econ-ark/HARK/issues/1346 and put the equations
onto
the old models.
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1292#issuecomment-2143582078,
or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAKCK74KLV3MWV7UU3MWW7LZFIW53AVCNFSM6AAAAAAZWHG3XWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGU4DEMBXHA>

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1292#issuecomment-2143726176, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQZEHMEYOYVSPVHMJ3CO3ZFK7XRAVCNFSM6AAAAAAZWHG3XWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTG4ZDMMJXGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

A reason to lay out the equations for more models is to validate that any
basic structural designs generalize beyond the portfolio consumption
problem.
As is, we risk spending months designing a general solution that works for
one specific problem.
On Sun, Jun 2, 2024, 7:53 AM Sebastian Benthall @.***> wrote:

Just to clarify what I meant: I meant having a representation of the
equations governing how the variables change, which other parts of system
can use.
On Sun, Jun 2, 2024, 3:04 AM Christopher Llorracc Carroll <
@.***> wrote:

Not getting what you mean by adding dynamics. In economists’ use of the
word these models already have dynamics. They are dynamic, stochastic,
optimization models.
I’m keen on making some basic structural decisions first.
Sent from Gmail Mobile
On Sat, Jun 1, 2024 at 4:43 PM Sebastian Benthall @.***>
wrote:

I'd like to revisit this PR, which is now almost a year old, and
perhaps
out of date with respect to recent HARK changes. But it was trying to
accomplish something that we are still trying to move towards.
The main point of this PR was to try attaching dynamics to the existing
ConsumptionSaving models, so that they could be rewritten with a single
source of truth about their model equations.
These dynamics, in this PR, looked like this:
IndShockConsumerType_dynamics = {
**PerfForesightConsumerType_dynamics,
'G' : lambda gamma, psi : gamma * psi,
'Rnrm' : lambda R, G : R / G,
'bNrm' : lambda Rnrm, aNrm : Rnrm * aNrm,
'mNrm' : lambda bNrm, theta : bNrm + theta,
'cNrm' : Control(['mNrm']),
'aNrm' : lambda mNrm, cNrm : mNrm - cNrm
}
This is now essentially the same thing as what's included in the
'block'
based consumption model, which was recently merged in:

https://github.com/econ-ark/HARK/blob/master/HARK/models/consumer.py#L26-L44

Now that there's a compact format for model equations already in HARK,
it
would be a good idea to implement a solution to #1346
https://github.com/econ-ark/HARK/issues/1346 and put the equations
onto
the old models.
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1292#issuecomment-2143582078,
or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAKCK74KLV3MWV7UU3MWW7LZFIW53AVCNFSM6AAAAAAZWHG3XWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGU4DEMBXHA>

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1292#issuecomment-2143726176, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQZEHMEYOYVSPVHMJ3CO3ZFK7XRAVCNFSM6AAAAAAZWHG3XWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTG4ZDMMJXGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

I don't think it will be months.
And once we have a satisfactory working example solution of two periods of
one particular problem, I am in complete agreement that we should take the
lessons learned and try adapting to another model.  I'm thinking of the
EndogenousRetirement problem, because it is so different from ConsIndShk.
On Sun, Jun 2, 2024 at 10:14 AM Sebastian Benthall @.***>
wrote:

A reason to lay out the equations for more models is to validate that any
basic structural designs generalize beyond the portfolio consumption
problem.
As is, we risk spending months designing a general solution that works for
one specific problem.
On Sun, Jun 2, 2024, 7:53 AM Sebastian Benthall @.***> wrote:

Just to clarify what I meant: I meant having a representation of the
equations governing how the variables change, which other parts of
system
can use.
On Sun, Jun 2, 2024, 3:04 AM Christopher Llorracc Carroll <
@.***> wrote:

Not getting what you mean by adding dynamics. In economists’ use of the
word these models already have dynamics. They are dynamic, stochastic,
optimization models.
I’m keen on making some basic structural decisions first.
Sent from Gmail Mobile
On Sat, Jun 1, 2024 at 4:43 PM Sebastian Benthall @.***>
wrote:

I'd like to revisit this PR, which is now almost a year old, and
perhaps
out of date with respect to recent HARK changes. But it was trying to
accomplish something that we are still trying to move towards.
The main point of this PR was to try attaching dynamics to the
existing
ConsumptionSaving models, so that they could be rewritten with a
single
source of truth about their model equations.
These dynamics, in this PR, looked like this:
IndShockConsumerType_dynamics = {
**PerfForesightConsumerType_dynamics,
'G' : lambda gamma, psi : gamma * psi,
'Rnrm' : lambda R, G : R / G,
'bNrm' : lambda Rnrm, aNrm : Rnrm * aNrm,
'mNrm' : lambda bNrm, theta : bNrm + theta,
'cNrm' : Control(['mNrm']),
'aNrm' : lambda mNrm, cNrm : mNrm - cNrm
}
This is now essentially the same thing as what's included in the
'block'
based consumption model, which was recently merged in:

https://github.com/econ-ark/HARK/blob/master/HARK/models/consumer.py#L26-L44

Now that there's a compact format for model equations already in
HARK,
it
would be a good idea to implement a solution to #1346
https://github.com/econ-ark/HARK/issues/1346 and put the equations
onto
the old models.
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1292#issuecomment-2143582078,

or

unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AAKCK74KLV3MWV7UU3MWW7LZFIW53AVCNFSM6AAAAAAZWHG3XWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGU4DEMBXHA>

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1292#issuecomment-2143726176,
or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAAQZEHMEYOYVSPVHMJ3CO3ZFK7XRAVCNFSM6AAAAAAZWHG3XWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTG4ZDMMJXGY>

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1292#issuecomment-2143870837, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKCK77ENUQ2BBA5UK7NRE3ZFMSCXAVCNFSM6AAAAAAZWHG3XWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTHA3TAOBTG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

--

Chris Carroll

PR is out of date, and not needed for HARK 1.0


Feature/zero downtime ecs instance redeploy

We've since made logspout daemon, and learnt that we don't want web to be a daemon as we'd like to have many of them as scale dictates.


allow support for eslint quiet option in webpack configuration.

Could we add a configuration option to only output eslint errors so that the output is essentially the same as calling eslint --quiet?

PR for that is welcome :)


Add gRPC enrichment

Similar to the API Request Enrichment, but using gRPC:

http://www.grpc.io/
https://github.com/grpc/grpc-java

Migrated to https://github.com/snowplow/enrich/issues/117


[HTTP server] Should :authority be higher on the list than Host?

The current spec says:

HTTP server instrumentations SHOULD do the best effort when populating server.address and server.port attributes and SHOULD determine them by using the first of the following that applies:

The original host which may be passed by the reverse proxy in the [Forwarded#host][Forwarded], [X-Forwarded-Host][X-Forwarded-Host], or a similar header.
The [Host][Host header] header.
The [:authority][HTTP/2 authority] pseudo-header in case of HTTP/2 or HTTP/3

However, the RFC says something different:

The ":authority" pseudo-header field conveys the authority portion (Section 3.2 of [RFC3986]) of the target URI (Section 7.1 of [HTTP]). The recipient of an HTTP/2 request MUST NOT use the Host header field to determine the target URI if ":authority" is present.

To me it sounds that when an HTTP 2.0 server receives an incoming request, the :authority header has higher priority than Host.

I put it after Host because from RFC9113 perspective, they should be the same and the order does not matter.
A server SHOULD treat a request as malformed if it contains a Host header field that identifies an entity that differs from the entity in the ":authority" pseudo-header field.


OpenEye reoders atoms when writing to PDB

Describe the bug
I think when using openeye to write a molecule to PDB file through the toolkit it puts the molecule in canonical order meaning the ordering is not consistent with the Molecule object. This stops when using other files types like SDF.
To Reproduce
from openforcefield.utils.toolkits import OpenEyeToolkitWrapper, RDKitToolkitWrapper
from openforcefield.topology import Molecule
mol = Molecule.from_smiles("OCCO")
mol.generate_conformers()
mol
mol.to_file(file_path="openeye.pdb", file_format="pdb", toolkit_registry=OpenEyeToolkitWrapper())
mol.to_file(file_path="rdkit.pdb", file_format="pdb", toolkit_registry=RDKitToolkitWrapper())
open_mol = Molecule.from_file("openeye.pdb")
rdkit_mol = Molecule.from_file("rdkit.pdb")
iso, atom_map = Molecule.are_isomorphic(open_mol, rdkit_mol, True)
print(atom_map)
{0: 1, 1: 0, 2: 2, 3: 3, 4: 4, 5: 9, 6: 5, 7: 6, 8: 7, 9: 8}

Output
The full error message (may be large, that's fine. Better to paste too much than too little.)
Computing environment (please complete the following information):
Packages
openeye-toolkits          2020.0.4                 py37_0    openeye
openforcefield            0.7.1              pyh39e3cac_0    omnia
rdkit                     2020.03.4        py37h585bbb4_0    conda-forge
Additional context
Add any other context about the problem here.

The default PDB flavor includes OEOFlavor_PDB_OrderAtoms, which might be responsible for this behavior. If we instead feed the writer a specific flavor that omits this component of the favor, I think this might fix it. If not (or I'm misunderstanding this option), I bet we could get OpenEye to implement a flavor flag to preserve atom ordering.

Whoops -- Forgot to include this in Pavan's recent PR. This was closed by #715 and the fix will come out in the 0.8.0 release.


Support multiple negations in conjuncts in JOIN condition clause

Support multiple negations in conjuncts in JOIN condition clause

@kokosing please take a look

This is corresponding issue https://github.com/facebook/presto/issues/3888

looks good

Addressed commits and rebased.

@cberner, can you take a look as you introduced initial non-equi join support?

I think this is too specific to NOT and LIKE. Instead lets solve the general case, so that it works for all expressions. I'll comment on the parts I think need to change, although there are probably some details to work out. Let me know if you have questions

Let me know when you've updated!

Update.
In this patch I treat all the expressions in form of e1 OP e2, where symbols from left and right join side are not mixed in neither e1 nor e2, old way. E1 and e2 computed as part of output for LEFT and RIGHT relation.
Conjuncts which do not match above schema are pushed down as post-join-condition after rewriting to use symbols from LEFT and RIGHT relation.
@cberner, please take a look and let me know what do you think.

I addressed most of the comments. Yet I will take a look at #4250.

:+1:
replied to your question about NOT a.x = b.x handling. Let me know when it's ready for me to take another look

Not sure what happened to my comment, so posting it again:

What you write makes sense. Though if, I understand correctly changing the code the way you suggest, is total rebuild of how join is planned today. We would need to add original join condition into JoinNode (it is not there right now - only list of EquiJoinClauses). And then move most of the logic from RelationPlanner.visitJoin to one or more optimizers.
As currently all the detailed planning happens in RelationPlanner.visitJoin I cannot move the normalization into SimplifyExpressions as this one is executing after RelationPlanner.visitJoin.
I would rather not put this Join planning refactor as part of this PR. Honestly I am not sure if with my current knowledge I would be able to efficiently implement this at all :)

Ya, lets not make major changes to the planning of JOIN in this PR. It shouldn't require adding the original predicate to the JoinNode when we do though. It can be planned as a join followed by a filter, and rely on predicate push down.
I think the normalization of not not a.x = b.x can be done pretty easily in the optimizers, in the way I mentioned earlier

Not sure what happened to my comment, so posting it again:

What you write makes sense. Though if, I understand correctly changing the code the way you suggest, is total rebuild of how join is planned today. We would need to add original join condition into JoinNode (it is not there right now - only list of EquiJoinClauses). And then move most of the logic from RelationPlanner.visitJoin to one or more optimizers.
As currently all the detailed planning happens in RelationPlanner.visitJoin I cannot move the normalization into SimplifyExpressions as this one is executing after RelationPlanner.visitJoin.
I would rather not put this Join planning refactor as part of this PR. Honestly I am not sure if with my current knowledge I would be able to efficiently implement this at all :)

Ya, lets not make major changes to the planning of JOIN in this PR. It shouldn't require adding the original predicate to the JoinNode when we do though. It can be planned as a join followed by a filter, and rely on predicate push down.
I think the normalization of not not a.x = b.x can be done pretty easily in the optimizers, in the way I mentioned earlier

@cberner. I pushed another changeset after rebase.
The possible additional which would not bloat the scope are:

remove normalize. I can do this. But I do not feel like it is correct approach unless we reintroduce it in optimizers. Which not super trivial. Keeping normalize does not help much - but helps a bit. And does not pollute code very much.
Only compute expression on joined relation sides for = operator and put all other operator in postJoinConditions. But I would not say that it will make code much simpler. We will not need the list of operator types. Rest would be still needed. This change is trivial. I can do this if you insist - but here again I do not see a big benefit from this change.

Please take a look when you have time and let me know if you want some more changes to this PR.

Couple minor comments

@martint can you look at the one part I tagged you in?

Adressed comments and rebased onto master.

Can you rebase this on master?

Rebased - travis build in progress

Merged, thanks!


⚠ Beamable Inventory (Object) has degraded performance

In 1821c62, Beamable Inventory (Object) (https://api.beamable.com/object/inventory/platform/docs) experienced degraded performance:

HTTP code: 504
Response time: 10019 ms

Resolved: Beamable Inventory (Object) performance has improved in 3df8bee after 13 minutes.


Electronik #34 preview missing

In this generative collection, one NFT is not loading preview properly. Even though the art is there when you open it. NFT link: https://kodadot.xyz/ahp/gallery/121-2405633641
Missing preview

Updated, the thumbnails are captured before the fix on a blank canvas. If the error persists, you need to clear the browser cache. Or you can check it out on our koda.art domain

you need to clear the browser cache

will adjust in this PR, so the user don't need to clear the browser cache
https://github.com/kodadot/private-workers/pull/179


nickel: 0.1.0 -> 0.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/tweag/nickel/releases.
meta.description for nickel is: "Better configuration for less"
meta.homepage for nickel is: "https://nickel-lang.org/"
meta.changelog for nickel is: ""
Updates performed

Rust version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.0 with grep in /nix/store/qrdvf5gc7124v6k8ncyfrgnwmqqf1j6q-nickel-0.2.0
found 0.2.0 in filename of file in /nix/store/qrdvf5gc7124v6k8ncyfrgnwmqqf1j6q-nickel-0.2.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
nickel

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qrdvf5gc7124v6k8ncyfrgnwmqqf1j6q-nickel-0.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nickel https://github.com/r-ryantm/nixpkgs/archive/8736aa01e9e3fb823df88b8e7632d546525e40d2.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qrdvf5gc7124v6k8ncyfrgnwmqqf1j6q-nickel-0.2.0
ls -la /nix/store/qrdvf5gc7124v6k8ncyfrgnwmqqf1j6q-nickel-0.2.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

nickel

Maintainer pings
cc @AndersonTorres for testing.

Result of nixpkgs-review pr 184616 run on aarch64-linux 1

1 package built:

nickel


info plugin: Allow custom formatting and human-readable lengths

Work in progress for implementing the feature discussed on #1689, as the workflow proposed by @awesomer would be pretty much aligned with one of my long-term wishlist for my library (ie. a neat way to assist in adding obscure/missing albums to MusicBrainz without the need for extra tools), and hopefully encourage others contributing back to MusicBrainz.
These commits intend to implement the suggested initial changes on the info plugin:

A good first step, should anyone choose to accept it, would be to extend the info plugin so it can be configured to output using formatting, like the list command.

The command now accepts the standard formatting argument (via cmd.parser.add_format_option), relying on the Item.__format__ machinery for formatting. Modifications have been made so an Item (either directly retrieved from the library if using library_data_emitter, or a "fake" temporary one built from the media file is using tag_data_emitter) is available to print_data().

At the same time, it would be a good idea to format length as a time by default—nobody probably wants to see a raw number of seconds.

Makes much sense to me! Just in case the raw number of seconds is useful to someone, an argument (--raw-length) has been added to switch between raw length and human-readable length (human-readable being the default).
I'd love some feedback on the current work, and also some thoughts on the preferred way for implementing the next steps. My thoughts go along the line of extending the info plugin with a listener for the "MusicBrainz match not found" event, but I'm not sure if this functionality would be better fitted into a less generic plugin?

Great! The format flag is an excellent first step, and something that could be useful in other contexts too—not just for uploading to MusicBrainz.
If it's OK with you, let's merge the format flag now. But I have a suggestion for the time formatting: let's make this the default for all of beets, not just the info plugin. Specifically, in library.py, let's change the $length field from types.FLOAT to a new type (TimeType?) that has a smarter format method. That way, even beet ls -f '$length' will get the new behavior.
Of course, that makes it somewhat more complicated to revert to the old, raw number if you want that for some reason. This, however, is a broader issue also shared by other fields like $bitrate and even $track, which people occasionally wish they could print as raw values. The current workaround is to define an inline field that just echoes the underlying field. But maybe we can think of a better solution for that also.

Sure, I'd be happy to leave out the time formatting changes out in favor of implementing them later at a wider scope - as a matter of fact, just pushed some commits that remove that functionality. At a first glance, I'd prefer to leave that change to someone more experienced with the types system instead of working on it myself, but if no one steps up I'd be more than willing to give it a try.

Wonderful! This looks great, and I appreciate the simplification of the path handling too. Let's call this done!


Request for READ_CONTACTS permission on android

Why
We needed to request and prompt users for reading contacts for API23+ since it is not automatically granted.
Changes

For ReactMethod requestPermission, if permission not granted call ActivityCompat.requestPermissions( ... ) and statically keep a reference of callback for feedback when handling onRequestPermissionsResult
Add static methods for onRequestPermissionsResult in ReactNativeContacts.java and ContactsManager.java

Issue
Fixes #180

Any plan to merge this in master ?

@FrancisToth have you reviewed it and confirm it works?

Well, having some tests would be awesome :) Maybe then we could review this properly.

reading https://github.com/rt2zz/react-native-contacts/blob/master/CONTRIBUTING.md shows the test should be added to https://github.com/morenoh149/react-native-contacts-test @jebora needs tests

@morenoh149 So I tried making a test for this the other day, but it seems like it's always automatically granting it contacts permissions, even on API 23+. In our app, we're integrating RN in our existing native code base and created our own ReactFragment/Activity that uses a react view factory with a common ReactInstanceManager. Is using FB's ReactActivity automatically grants required permissions? This shouldn't be the case right?

I believe the test app still uses android 22 https://github.com/morenoh149/react-native-contacts-test/blob/master/android/app/src/main/AndroidManifest.xml#L11 please update it

requestPermission not working on recent Android versions is quite a major issue since it mease it simply does not work. any clue why this PR never made it?

@antoinerousseau it got held up by the testing requirement. Does this PR work for you?

@morenoh149 haven't tried it, I'm just manually requesting permissions but it should be handled by this package

Hi all, sorry I completely forgot about this Pr. I'll make time to write a test later today

Hello. Made an integration test here https://github.com/morenoh149/react-native-contacts-test/pull/6
Finally, sorry for delay 😅

no worries. This is all volunteer time.


onChange handler is not working for prop when passed as function definition

Example:
// the value is set to undefined
<input onChange={onChange('name')} />

Issue Solved! v1.1.1


P710: Test description is not updated

The first line in this page:
it('applies in priority order', function() {...

The description was the same as last test; it should updated with something like
it('applies in name order when priorities are the same', function() {...

Fixed, thanks!


14 sof-detect-keypoints: wrong upside_down label on rare case with wrong key point locations and low confidence

When two "Complete" labels were detected, the key points with the higher score are used to determine the image orientation.

Closes #14


Sleek won't show todos: TypeError: Cannot read properties of undefined (reading 'includes')

Bug Report
App Version: Sleek  2.0.13 (Flathub)
Platform: Linux: Fedora 39. Same bug also occurs on Solus
Installation Method: Flathub
Expected Behavior:
Sleek should start and load my todo file and display existing todos.
Actual Behavior:
Sleek will start, seems to load my todo file (sleek-todo.txt), but does not display any existing todos.
On bottom left appears error message: Cannot read properties of undefined (reading 'includes')
Steps to Reproduce:

start sleek

Screenshots:

This bug exists for several weeks now. So after running flatpak update, I always have to run sudo flatpak update --commit=6a1415b69edc360d533d3423323943849f6662470001fb4483fff3378fe7474f com.github.ransome1.sleek to return to last functioning Sleek version 2.0.11
running flatpak run com.github.ransome1.sleek in terminal gives following output:
Pastebin

@sebo505 can you please empty your filters by pressing CTRL + 0 and then try again?

Pressing CTRL + 0 actually solves this issue. Would have never guessed that myself.

Unfortunately this is due to a minor incompatiblity between the way attribute filters are structered, introduced in 2.0.12 if I am not mistaken. I am aiming to solve this with the migration function of sleek in the next release. Until then I will pin this report in case others have the same issue.


Feature/add breadcrumb to public components

Purpose
Closes #1249
Often, components from different projects have the same title (e.g. "Study Materials" or "Data"). They are listed in your public profile without providing the project title for context, so components with the same name are indistinguishable from each other.
Changes

Before

After

Side effects
I refactored get_public_components in website/profile/views.py so that when this method is called, the summary of node that is rendered always have the path underneath.

Closing - updated and replaced with #2977.