Koa performance

Description
Need to wait for both
@shopify/statsd https://github.com/Shopify/quilt/pull/1068
and
@shopify/browser https://github.com/Shopify/quilt/pull/1082
to merge
Type of change

[ ]  Patch: Bug/ Documentation fix (non-breaking change which fixes an issue or adds documentation)
[ ]  Minor: New feature (non-breaking change which adds functionality)
[ ]  Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[ ] I have added a changelog entry, prefixed by the type of change noted above

closing this PR and reopening it so I have an easier time managing it


Block device support for domains

My first pass at providing support for block devices. Attempts to address #662 and #418. (Disclaimer: I am new to go).

checkout travis is failing
+go test -v -covermode=count -coverprofile=profile.cov -timeout=1200s -test.failfast ./libvirt

# github.com/dmacvicar/terraform-provider-libvirt/libvirt [github.com/dmacvicar/terraform-provider-libvirt/libvirt.test]

libvirt/resource_libvirt_domain_test.go:1221:43: cmd.String undefined (type *exec.Cmd has no field or method String)

libvirt/resource_libvirt_domain_test.go:1240:43: cmd.String undefined (type *exec.Cmd has no field or method String)

FAIL	github.com/dmacvicar/terraform-provider-libvirt/libvirt [build failed]

Makefile:36: recipe for target 'testacc' failed

make: *** [testacc] Error 2```

I am running same build commands locally and not seeing the errors:
:~/go/src/github.com/muroj/terraform-provider-libvirt$ make build
go fmt ./libvirt .
go run golang.org/x/lint/golint -set_exit_status ./libvirt .
go vet ./libvirt

Is it because I am running go1.13.4 instead of go1.11.13?

@muroj  dunno I didn't tried but try it :grin:

@muroj thx for pr great job

While trying to do a release I realize this PR broke integration tests.
It requires both root access to test this, and on top, it assumes libvirtd is running on the same machine as the test, and assumes /usr/sbin is on $PATH (eg.  mkfs.ext4).

Managed to fix this calling executables with full path (sbin). I am still stuck in:
Jan 03 13:36:32 piscolita.suse.de libvirtd[5524]: internal error: process exited while connecting to monitor: 2020-01-03T12:36:32.234735Z qemu-system-x86_64: -blockdev {"driver":"host_device","filename":"/tmp/cgvuajorbpkmtkbuetximqhkpcetcwlby","node-name":"libvirt-1-storage","auto-read-only":true,"discard":"unmap"}: 'host_device' driver expects either a character or block device

Which I guess comes from:
var configBlockDevice = fmt.Sprintf(`

resource "libvirt_domain" "%s" {
	name = "%s"

	disk {
		block_device = "%s"
	}

}`, randomDomainName, randomDomainName, tmpfile)

Which makes total sense. tmpfile is not a device. The loop device is a device. I wonder how this works in the libvirt used in travis.
If I change the code to use the loop device, I fall into this:
Error: Error creating libvirt domain: virError(Code=1, Domain=10, Message='internal error: process exited while connecting to monitor: 2020-01-03T13:19:32.338303Z qemu-system-x86_64: -blockdev {"driver":"host_device","filename":"/dev/loop0","node-name":"libvirt-1-storage","auto-read-only":true,"discard":"unmap"}: The device is not writable: Permission denied')

Which makes sense, as losetup is called as:
// Mount the file to a loop device
cmd = exec.Command("sudo", "/sbin/losetup", "--read-only", strings.TrimRight(string(loopdev), "\n"), filename)
fmt.Printf("Executing command: %s\n", strings.Join(cmd.Args, " "))

Which means we are setting up a rw disk on a ro device. However, we haven't implemented readonly for disks.
Fixing losetup, I need to fix the check function to check that the source is the device.
Fixing that, I get an error that tmpfile is not used. Looks like the temporary file is not cleaned. I have fixed that as well.
Fixes being prepared here:
https://github.com/dmacvicar/terraform-provider-libvirt/tree/dmacvicar_fix_tests_locally

@dmacvicar thx and sorry for not having catching it up.
Let me know if I can help somehow. Thx!

I put all the fixes in master before doing the release! thanks!
The PR was passing testacc fine on Travis! so not a review problem. openSUSE/libvirt were slightly different.

@dmacvicar @MalloZup  Thanks both for your work (and patience) with me on this.


[BUG] Problems with default recipe and dependent projects using RAPIDS.cmake

Describe the bug
If a dependent project also uses RAPIDS.cmake, the default recipe will download and overwrite the file in the root build directory. This is especially problematic if the dependent project downloads an older version of rapids-cmake, pulling in an older versions.json.
Steps/Code to reproduce bug
Configure and build https://github.com/NVIDIA/cuCollections; it should work. Then re-run cmake . and observe that older versions of libcudacxx and Thrust are added (based on what https://github.com/NVIDIA/nvbench pulls in). The build will fail afterwards because cuCollections requires the newer versions.
Expected behavior
The build should be stable and work after reconfiguration.

One possibility would be to download RAPIDS.cmake (and everything else pulled in from there) into CMAKE_CURRENT_BINARY_DIR, but I don't know how that works out with building the dependencies and potentially having the same dependent project used transitively by multiple intermediate projects...

This isn't an issue with rapids-cmake. It is a bug in the cuCollections CMakeLists.txt.
In the root CMakeLists.txt cuCollections has:
if(NOT EXISTS ${CMAKE_BINARY_DIR}/RAPIDS.cmake)
  file(DOWNLOAD https://raw.githubusercontent.com/rapidsai/rapids-cmake/branch-22.08/RAPIDS.cmake
       ${CMAKE_BINARY_DIR}/RAPIDS.cmake)
endif()
include(${CMAKE_BINARY_DIR}/RAPIDS.cmake)

This guard around the file download causes the nvbench version of the RAPIDS.cmake to be the one that 'sticks' around. Instead you should always download the file so that when you reconfigure you have the cuCollection version.

This guard around the file download causes the nvbench version of the RAPIDS.cmake to be the one that 'sticks' around. Instead you should always download the file so that when you reconfigure you have the cuCollection version.

And then the dependency again downloads its version of RAPIDS.cmake - not very nice overall...

You can version the download if you so wish( download it to RAPIDS_22.08 ) which would also solve the issue.
I will update the rapids-cmake docs/examples to use CMAKE_CURRENT_BINARY_DIR. I didn't think this was a short term solution as it requires an update to nvbench to correct cuCollections.

https://github.com/rapidsai/rapids-cmake/pull/241 updates the docs / examples to show best practices so the downloaded file doesn't get clobbered.


Channels for welcome / goodbye can be set to a non-text channel

Expected Behavior
Only being able to use a text channel
Current Behavior
Any channel is configurable
Possible Solution
Check the channel type before accepting
Steps to Reproduce (for bugs)
Just configure welcome / goodbye with a non-text channel
Context

This has been fixed here: https://github.com/mkevenaar/NickNelson/commit/f8c8492d7272e2f3c67f31c761d7d34ee62e4655

:tada: This issue has been resolved in version 0.1.0 :tada:


KeyNotFoundException when using XBoxOne Pad triggers or A button on an Interactable

Overview
A blocking exception is raised at each frame when using a button from xboxone controller in editor, play mode, with default mixed reality profile.
Also happens on most demo scenes (check TooltipExamples, and press on the blue diamond on the right)
Expected Behavior
No Exception when using a Pad Key that is not mapped
Actual Behavior
Spamming error each frame:

KeyNotFoundException: The given key was not present in the dictionary.
System.Collections.Generic.Dictionary`2[TKey,TValue].get_Item (TKey key) (at :0)
Microsoft.MixedReality.Toolkit.Input.DefaultPointerMediator.UpdatePointers () (at Assets/MixedRealityToolkit.SDK/Features/UX/Scripts/Pointers/DefaultPointerMediator.cs:107)
Microsoft.MixedReality.Toolkit.Input.FocusProvider.UpdatePointers () (at Assets/MixedRealityToolkit.Services/InputSystem/FocusProvider.cs:681)
Microsoft.MixedReality.Toolkit.Input.FocusProvider.Update () (at Assets/MixedRealityToolkit.Services/InputSystem/FocusProvider.cs:443)
Microsoft.MixedReality.Toolkit.MixedRealityToolkit.UpdateAllServices () (at Assets/MixedRealityToolkit/Services/MixedRealityToolkit.cs:994)
Microsoft.MixedReality.Toolkit.MixedRealityToolkit.Update () (at Assets/MixedRealityToolkit/Services/MixedRealityToolkit.cs:777)

Steps to reproduce

Create a new scene
Menu Mixed Reality Toolkit > Add to scene and Configure... and choose default configuration profile
Add a Cube in (0,0,10)
on this cube add an Interactable Script
(optional) - set the cube as Target of the Interactable Profile

=> Click on Play in Unity Editor, and press A, LB, RB... on the joystick

Exception is raised and all control are blocked

Unity Editor Version
2018.3.11f1
Mixed Reality Toolkit Release Version
v2.0.0-RC1

I tried it again with a fresh project, and i don't get the error ...
I may have broken something in the MRTK coonfiguration, or during the beta to RC1 change ...

Going from Beta to RC1 involved a number of breaking changes from beta - the profiles and namespaces of things changed up enough that even merging those changes on our side was quite tricky (so I can't imagine how it would be to try to go from Beta -> RC1 as a drop-in replacement).
The other possibility is you can try copying a working configuration profile (i.e. using a fresh MRTK + project), and try to map your previous configuration to the new one.
@davidkline-ms might have other ideas.

on the 'fresh' project, i do have the exception in the "HandInteractionExamples", other examples are working fine.

looking at the code DefaultPointerMediator.cs:
foreach (IMixedRealityPointer otherPointer in pointerByInputSourceParent[pointer.InputSourceParent])
on line 107 and 137
seems it doesn't check if there is somevalue in the dictionnary
replacing with:
HashSet<IMixedRealityPointer> children;
if (pointerByInputSourceParent.TryGetValue(pointer.InputSourceParent, out children))
get rid of the exception

@Railboy

I have this issue as well and it happens both on new and old projects. For old projects I have tried updating the MRTK to latest code, recreated the library folder, cleared cache all without resolution.
@gilbdev's fix above works for me

Following up on this one, it looks like was actually addressed when I fixed another issue in this PR:
https://github.com/microsoft/MixedRealityToolkit-Unity/pull/3808
Finally got around to trying this out and was wondering why this wasn't reproing on latest, but yeah, this should be good now.


add support for command-line completion

We're already using Cobra and it can auto-generate the completions with a little work. Some folks might like this? Also seems like a straightforward issue with low risk for someone new to otel-cli to pick up.
Bonus points: add these to the OS packages
https://github.com/spf13/cobra/blob/master/shell_completions.md

Shell completions are always super nice! I don't know if homebrew packages support it in a clean way, so we might be stuck there. But we should provide them wherever we can! :)


Since the update, the bots are no longer working, but...

As I wrote in the title, since the update, none of the bots are working. There are no errors in the logs, which remain on "Connecting to database..."
I had the idea to do a test on Telegram and I got the message "Oops! Embedding not found".
It's true that I changed the API and I forgot to put an Embedding model.
I fixed that, created a new bot and it's working.
My problem is that for the existing bots, I can't change the model and they don't work.
How can I fix this?
Thank you

Hey, sorry about that. Which version did you update from to get to the latest one? Also, in Settings > Models > Embedding Models, is your 'dialoqbase_eb_text-embedding-ada-002' model disabled?

I put the OpenAI model, because I don't see your, since the update it has disappeared.
I'll show you the screenshots

Hey, please enable this model; it will work fine :)

I added the template and if I create a new bot, no problem it works.
But with the old bot it doesn't work, I always have the same problem.
That's because the model isn't there and you can't change it.
Why isn't the model there?

Hey, please enable this model; it will work fine :)

Hey, could you go to Models > Embedding Model and enable text-embedding-ada-002 please? It may fix the old bot issue

Hey, please enable this model; it will work fine :)

Hey, dialoqbase_eb_text-embedding-ada-002 is the internal reference ID for text-embedding-ada-002. After referring to your screenshot, I think it's disabled, which is causing the issue. If you enable it from the embedding table, it may resolve all the issues.

I had understood correctly, I activated it but it doesn't change anything.
But you can see from the image that it doesn't have the same name

That's weird. I will look into it. Can you check your bot to see if it's working now? It should have worked.
Unlike the chat model, you can't currently change embedding models due to dimensional issues. Doing so may completely break the bot, which is why this feature is disabled. I will look into a way to fix it

Don't worry, I've deleted the docker and I'll start all over again.
It's not a bad thing

In version 1.8.2, embedding models won't be disabled, but you can change them. Make sure you switch to the correct dimension model, or you'll need to retrain all the data sources

Hello,
I started everything from scratch on another server.
Until yesterday, I had no problem, but now I again have the sentence "There was an error processing your request."
Here are the logs:
dialoqbase           | clvjpwa9z000747auf7pxisw7 dialoqbase           | TypeError: query.join is not a function dialoqbase           |     at DialoqbaseVectorStore.similaritySearchVectorWithScore (/app/utils/store.js:67:34) dialoqbase           |     at DialoqbaseVectorStore.similaritySearch (/app/node_modules/@langchain/core/dist/vectorstores.cjs:108:36) dialoqbase           |     at process.processTicksAndRejections (node:internal/process/task_queues:95:5) dialoqbase           |     at async VectorStoreRetriever.getRelevantDocuments (/app/node_modules/@langchain/core/dist/retrievers.cjs:70:29) dialoqbase           |     at async VectorStoreRetriever._streamIterator (/app/node_modules/@langchain/core/dist/runnables/base.cjs:151:9) dialoqbase           |     at async VectorStoreRetriever.transform (/app/node_modules/@langchain/core/dist/runnables/base.cjs:344:9) dialoqbase           |     at async RunnableSequence._streamIterator (/app/node_modules/@langchain/core/dist/runnables/base.cjs:1084:30)

hey, did you change the embedding model ?

No, actually I created two bots that I use on Telegram, and when it didn't work, I didn't have a PC.
I think before the outage, I made a third bot based on a YouTube video, but there was a problem and I deleted the bot.
If necessary, I don't mind deleting the server again, it's quick.

That is strange. I will try to reproduce this issue, since it's caused by the embedding model. By the way, which embedding model are you currently using

I'm using text-embedding-3-large, but I have also tried text-embedding-3-small.
Since I'm using the program for two Telegram bots, I erased everything and started over again.
I don't want to touch it for now, and if the problem comes back, I'll give you full access and start over on another server.
Thank you for your help.


Keep project namespaces after deletion if they existed already

How to categorize this PR?

/area usability
/kind enhancement
/priority normal
What this PR does / why we need it:
When existing Namespaces are adopted for Projects then they will now be configured to remain even after the Project is being deleted later again. Earlier, such namespaces were also deleted together with the Project.
Which issue(s) this PR fixes:
Fixes #2978
Special notes for your reviewer:
Release note:

When existing `Namespace`s are adopted for `Project`s then they will now be configured to remain even after the `Project` is being deleted later again. Earlier, such namespaces were also deleted together with the `Project`. Please note that this only takes effect for newly adopted project namespaces.

@timebertt @timuthy I had to rebase, please check again.


Target analysis

Target analysis plot

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

49 of 49 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 91.073%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
1816

Relevant Lines:
1994

💛  - Coveralls

looking good to me
@idomic @neelasha23

@neelasha23 looking good, the only thing that's missing is documentation, did you add it besides the sample notebook? We need to add some details about this plot, when to use it, what parameters it takes, common pitfalls etc. into the right section (let's say metrics)

When to use and parameters are mentioned in the guide. I will look into common pitfalls if any
@idomic

@neelasha23 for some reason, github did not trigger the CI. can you try adding a new commit and push?
git commit --allow-empty -m 'trigger ci'

Done @edublancas

nice work! 🚀


Remove padding on team building container

Reducing paddingLeft on the user bar to align the leftmost profile picture with the search bar
Shows horizontal scrollbar when scrolling on desktop. Also, on macOS when System Preferences > General > Show Scrollbars: always is enabled, no scrollbar is shown under the user bubbles. In this case, the scrollbar is rendered below both.

Current Padding

New Padding

@keybase/react-hackers


Extract graph and filter code to src/common/

Describe your issue
A lot of graph logic currently lives in src/web/ and is specific to the front-end. Some of this would be nice to share with the back-end (e.g. creating basic quick views when a topic is created).
Solution you'd like
No response
Alternatives you've considered
No response
Additional context
No response
Technical ideas and questions

a lot of graph methods rely on the web/ Node rather than a common/ Node. should the front-end mostly use common/ Node and just us web/ Node (flow data node?) when laying out?

some notes:

web node needs all the same stuff but react flow puts most of it behind data

also needs to be able to add front-end-specific stuff, like newlyAdded

graph methods shouldn't need web-specific stuff, can they use the api node type?

but Diagram component uses web-specific version

if we convert web-specific version to match common version, with newlyAdded

then we should be able to extract easy
hmm PositionedNode is extracted already, which layout uses

but layout should use non-web graph methods...


Feature request/question: bipartite knn?

Hi there and thank you for the extremely useful libraries, both this one and https://github.com/nmslib/hnswlib.
Does either library support bipartite knn?
In other words, if I have two lists of vectors A and B, I want to give it a certain element in A (call it A_i) and ask what are the k nearest elements in B. Ideally, I'd like to have it for all elements in A (there might be some speed-up by batch-querying, I guess).
If not, are you aware of any library/paper where they do it? Or alternatively, do you think one could hack either of your libraries to work in that situation?
Thank you for your time,
Fabio

Hi @iosonofabio although it may be possible to do via some clever merging, it is not clear if you can do better than:

indexing set A
batch query A using B
do the procedure in the other direction
I do not think, it's going to be crucial to have this as a feature.

Thank you - will try that way. Closing.


Use REST API instead of xmlhttprequest

Recreating PR #26, which contained an unrelated commit that wouldn't go away.
Fixes #16

@vboctor I think I've addressed all your comments.


[Makefile:43: build] Error 127

Hi
I am getting this error when I follow the commands provided

Any idea what is wrong or how to fix this?

Just tested it, works well on my end.
I recommend looking at the Makefile and execute the commands in the section "build" manually in your terminal to find out which one of them failed. Perhaps your user does not have the permission to create folders or files in /mnt/c, or the zip command failed executing.

That worked. I was running of a new machine for WSL and zip was not installed.
Thank you for the help.


Query behavior difference when using Continuation Token and ResponseContinuationTokenLimitInKb

Background:
I am testing the query behavior when using Continuation token.
Seeing from this post: https://stackoverflow.com/questions/54216251/cosmos-db-continuation-token-size-influences-whether-query-returns-new-documents/, he never gets documents that were created after the first continuation token had been created.
But I got different behaviors. When I use the continuation token, I will see the newly added document after the first continuation token is created if I don’t set the token size limit, and I won’t see revised document. Is this behavior expected?
Also, since I noticed that the continuation token size can grow very large, I would like to restrict the size of token using FeedOptions.ResponseContinuationTokenLimitInKb. But it seems that the limitation on size will lead to potential re-evaluating of the index, and we may get different results.
What we want to confirm:

Suppose we restrict the size of a continuation token by setting FeedOptions.ResponseContinuationTokenLimitInKb, except for RU costing difference, will there be any other behavior difference for query(e.g. different query result)?
And if so, during the query execution with the continuation token, will I see duplicate results (appear multiple times) if the document is being revised concurrently with the FeedOptions.ResponseContinuationTokenLimitInKb being set?
Besides setting this token size limit, would it be a good idea to do string compression/decompression on our side to reduce token length?

Can anyone help provide some information? Any insight would be appreciated!

Hello @blankor1 , my apologies for the late response here.
Please see answers below:

Suppose we restrict the size of a continuation token by setting FeedOptions.ResponseContinuationTokenLimitInKb, except for RU costing difference, will there be any other behavior difference for query(e.g. different query result)?

You may see a slightly different query result if your document set has changed due to insertion between continuations, and we are forced to reevaluate query because the continuation size was small enough that we could not serialize the filter postings.

And if so, during the query execution with the continuation token, will I see duplicate results (appear multiple times) if the document is being revised concurrently with the FeedOptions.ResponseContinuationTokenLimitInKb being set?

You will not see duplicate results.

Besides setting this token size limit, would it be a good idea to do string compression/decompression on our side to reduce token length?

As long as the query api receives the continuation token in the exact same form that it yielded, we do not care how you store your token or if you compress it etc.


Confusion about the store of uttid

hi, I'm a little confused about the calculation and store of uttid. In make_mixture.py, when calculating the uttid, it multiples 100, like
uttid = '{}{}{:07d}_{:07d}'.format(spkid, recid, int(startpos / args.rate * 100),int(endpos / args.rate * 100)).But why not do sth like str(int(startpos / args.rate *1000)/1000) to store it more precisely?

We followed the kaldi-style. https://kaldi-asr.org/doc/data_prep.html
Multiply by 100 means the frame index when using 10 msec frame shift.
I think this resolution is sufficient for our tasks.

Thanks, I'll check the website.


Push Error

Trying to use the action, but getting this error, does any1 knows how to solve it?
Run<EMAIL_ADDRESS>/usr/bin/docker run --name e1cc5172726b56add94e218e34c05e664cb352_caa02e --label e1cc51 --workdir /github/workspace --rm -e INPUT_ACCESSTOKEN -e INPUT_IDTOKEN -e INPUT_REFRESHTOKEN -e INPUT_CLIENTID -e INPUT_CLIENTSECRET -e INPUT_SCRIPTID -e INPUT_COMMAND -e INPUT_ROOTDIR -e INPUT_DESCRIPTION -e HOME -e GITHUB_JOB -e GITHUB_REF -e GITHUB_SHA -e GITHUB_REPOSITORY -e GITHUB_REPOSITORY_OWNER -e
GITHUB_RUN_ID -e GITHUB_RUN_NUMBER -e GITHUB_RETENTION_DAYS -e GITHUB_RUN_ATTEMPT -e GITHUB_ACTOR -e GITHUB_WORKFLOW -e GITHUB_HEAD_REF -e GITHUB_BASE_REF -e GITHUB_EVENT_NAME -e GITHUB_SERVER_URL -
e GITHUB_API_URL -e GITHUB_GRAPHQL_URL -e GITHUB_WORKSPACE -e GITHUB_ACTION -e GITHUB_EVENT_PATH -e GITHUB_ACTION_REPOSITORY -e GITHUB_ACTION_REF -e GITHUB_PATH -e GITHUB_ENV -e RUNNER_OS -e RUNNER_NAME -e RUNNER_TOOL_CACHE -e RUNNER_TEMP -e RUNNER_WORKSPACE -e ACTIONS_RUNTIME_URL -e
ACTIONS_RUNTIME_TOKEN -e ACTIONS_CACHE_URL -e GITHUB_ACTIONS=true -e CI=true -v
"/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v
"/home/runner/work/_temp/_github_workflow":"/github/workflow" -v
"/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v
"/home/runner/work/slick/slick":"/github/workspace" e1cc51:72726b56add94e218e34c05e664cb352  "***" "***" "***"
"***" "***" "***" "" "push" ""
ENOENT: no such file or directory, open '/github/workspace/appsscript.json'

here's the action i have:
name: Apps Script CI

on:
  push:
    branches: [ main ]
  pull_request:
    branches: [ main ]
jobs:
  deploy-apps-Script:
    runs-on: ubuntu-latest
    steps:
      - uses<EMAIL_ADDRESS>        with:
          accessToken: ${{ secrets.ACCESS_TOKEN }}
          idToken: ${{ secrets.ID_TOKEN }}
          refreshToken: ${{ secrets.REFRESH_TOKEN }}
          clientId: ${{ secrets.CLIENT_ID }}
          clientSecret: ${{ secrets.CLIENT_SECRET }}
          scriptId: ${{ secrets.SCRIPT_ID }}
          command: 'push'

All files (appscript.json, code.js and .clasp.json) lives in the root folder, and the action is in .github/workflows

Hi, have you solved the problem ?

Did you add all the variables in secrets ?
Can you try current version 1.1.0 ?
Can you try adding this line just before - uses<EMAIL_ADDRESS>:
- uses: actions/checkout@v2

I had the same problem and solved by manually creating appsscript.json under the working directory.

@shmpwk
Thank you for your report. I double checked and there is no mention of appsscript.json in this project.
If possible, could you please add a note to the README?


[feat] - use etcd partition on control plane nodes

What type of PR is this?

/kind feature
What this PR does / why we need it:

This PR configures etcd to be on a separate disk from the root disk on control plane nodes. The root disk size is split in half and the other half is used for etcd

💬 Discussion: Should we have the etcd storage size be configurable as part of the spec or leave it at this halving logic for ease of use? Instance creation can fail if the size is set larger than the storage plan allows for the instance (e.g. 80GB for g6-standard-2)

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
TODOs:

[x] squashed commits
[x] includes documentation
[ ] adds unit tests
[ ] adds or updates e2e tests

@AshleyDumaine
We're going to need the etcd partition size to be configurable as a parameter. Different users are going to want different.
To hold backups, work space for tarring the files, working with snapshots,etc. some users are going to want up to 16 or 32 GiB for a partition.
We can safely have a nice default but allow it configurable.

@AshleyDumaine
We're going to need the etcd partition size to be configurable as a parameter. Different users are going to want different.
To hold backups, work space for tarring the files, working with snapshots,etc. some users are going to want up to 16 or 32 GiB for a partition.
We can safely have a nice default but allow it configurable.

Sounds good, I created https://github.com/linode/cluster-api-provider-linode/issues/152 to address this. I think we'll want to include validating admission webhooks for this to reject if the requested etcd size exceeds the instance plan for the control plane nodes. This should be added in another PR.


[BUG] user_followers is not up to date

I run user_followers every day, and even after 24 hours, the values reported are still incorrect (outdated). I log in with username, password, and mfa every time, so is this just a limitation of this library?

I had this problem too, but I fixed it by setting the use_cache parameter to False inside the user_followers function: cl.user_followers(user_id,False)


Implement escaping in V4 POST policies

Policy should be encoded as defined in
https://docs.google.com/document/d/1XYkqJ_pPazkg6NuhRUEl_cReUtzX1DbgvF_jbPzgzog/edit#heading=h.rze9ynrxgv37

@dopiera I believe this can be closed. Is that correct?

Yes.


Feature: back source when no available peers

Signed-off-by: Jim Ma<EMAIL_ADDRESS>
Description

Related Issue

Motivation and Context

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist:

[ ] Code compiles correctly.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Codecov Report

Merging #521 (22ee628) into main (83cd262) will increase coverage by 0.02%.
The diff coverage is 41.34%.

@@            Coverage Diff             @@
##             main     #521      +/-   ##
==========================================
+ Coverage   39.34%   39.37%   +0.02%
==========================================
  Files          85       86       +1
  Lines        6438     6507      +69
==========================================
+ Hits         2533     2562      +29
- Misses       3539     3576      +37
- Partials      366      369       +3

Flag
Coverage Δ

e2etests
?

unittests
39.37% <41.34%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

client/config/peerhost.go
35.48% <ø> (ø)

client/daemon/peer/peertask_dummy.go
0.00% <0.00%> (ø)

pkg/source/source_client.go
2.10% <3.57%> (+2.10%)
:arrow_up:

client/daemon/peer/peertask_base.go
33.47% <30.35%> (+0.53%)
:arrow_up:

client/daemon/peer/peertask_file_callback.go
62.96% <50.00%> (ø)

client/daemon/peer/peertask_stream_callback.go
64.55% <50.00%> (ø)

pkg/source/httpprotocol/http_source_client.go
66.26% <53.84%> (-3.60%)
:arrow_down:

client/daemon/peer/peertask_stream.go
57.39% <57.57%> (-0.22%)
:arrow_down:

client/daemon/peer/peertask_file.go
53.90% <65.51%> (+0.09%)
:arrow_up:

client/daemon/peer/peertask_manager.go
21.01% <100.00%> (+1.75%)
:arrow_up:

... and 4 more


Re-build prod example simulation runs and projects

[x] Deploy prod
[x] Run example simulations
[x] Publish example runs

:tada: This issue has been resolved in version 8.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Как сменить модель на GPT35Turbo?

You can specify the model in various ways.

If you are creating a Request object there is a property called Model on most reqeusts which you can set.
You can set the model when calling the extension methods. See OpenAIDefaults.TextCompletionModel
You can set the default model by changing OpenAIDefaults.TextCompletionModel.

chat = api.Chat.CreateConversation();
chat.RequestParameters.Model = "gpt-3.5-turbo-16k";


bugfix: expand in PS model

This bug affects non-square inputs.

Codecov Report

Merging #177 (718ac4c) into main (58869b8) will decrease coverage by 12.31%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main     #177       +/-   ##
===========================================
- Coverage   75.17%   62.86%   -12.32%
===========================================
  Files          34       34
  Lines        3098     3094        -4
  Branches      652      652
===========================================
- Hits         2329     1945      -384
- Misses        574      964      +390
+ Partials      195      185       -10

Impacted Files
Coverage Δ

plenoptic/simulate/models/portilla_simoncelli.py
9.75% <0.00%> (-58.33%)
:arrow_down:

plenoptic/simulate/models/frontend.py
54.20% <0.00%> (-39.26%)
:arrow_down:

...simulate/canonical_computations/non_linearities.py
54.90% <0.00%> (-39.22%)
:arrow_down:

...e/canonical_computations/steerable_pyramid_freq.py
57.70% <0.00%> (-28.75%)
:arrow_down:

...enoptic/simulate/canonical_computations/filters.py
70.27% <0.00%> (-27.03%)
:arrow_down:

plenoptic/simulate/models/naive.py
80.55% <0.00%> (-13.03%)
:arrow_down:

plenoptic/tools/optim.py
78.94% <0.00%> (-10.53%)
:arrow_down:

plenoptic/synthesize/metamer.py
74.75% <0.00%> (-1.49%)
:arrow_down:

plenoptic/synthesize/synthesis.py
77.46% <0.00%> (-0.32%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Sorry for the slow reply and thanks @theowoo!  This fix looks good to me.  As you might guess, we are using exclusively square stimuli right now so this was missed in the matlab --> python conversion.


"New session" doesn't work properly

We received feedback that the "New Session" button doesn't work properly - in version v20.07.6. Apparently you always have to close and re-open the app to create a new session.
We need to investigate exactly what the problem is

Closing as we were not able to reproduce it.


Added the special Scheduling algorithms [#1649]

In an operating system there are several special scheduling algorithms such as follows

First-Come, First-Served (FCFS) Scheduling
Shortest-Job-Next (SJN) Scheduling
Priority Scheduling
Shortest Remaining Time
Round Robin(RR) Scheduling
Multiple-Level Queues Scheduling

Special scheduling algorithms are added keeping in mind that most of them requires user inputs.

Issue no ?

Issue no ?

1649

Please let me know if anything doesn't seem to be useful, I am new to Open Source but have learnt many things and happy to be a participant of GSSoc 2021. I have made this contribution by adding most of the important algorithms required in operating systems and made a folder because I thought it would be more appropriate to combine all of them and make a single directory.
Do let me know if you think this is good.
Thanks


Users/abmodi/master

Bugs found in 85.3 bug bash. Fixing hem. See commit details for fixes

Signed off

Merging on behalf of abmodi


Add RSS endpoint for media items

JWPlatform supports directly rendering an (m)RSS feed for a given item by means of its platform delivery API. This API has a (third) method for authentication so, firstly, 0a2528f implements that method anf then 52b16cc uses it to add the endpoint. The endpoint simply redirects to the (signed) URL.

The code is correct but I don't think this closes the issue completely as we need to generate collection RSS feeds as well using the collection_id from the video tags, see https://sms.cam.ac.uk/collection/2448176 and http://rss.sms.cam.ac.uk/rss/collection/2448176

The example given in the issue was for media items only. I thought the story was scoped to them. If not, it needs resizing.

I have removed the Closes ... so it can be merged now without closing the ticket, although I let you decide if you want to merge or not given the feedback form codecov

#38 goal is to replace RSS feeds, all of them, and replace the endpoint with apache to use the new ones (like with the video iframe) otherwise we can't move to jwplayer as the RSS feeds will point to the old SMS media item and it won't work.

I've updated the description of #38. Does that cover all feeds?
Re coverage: I was a fool and didn't check in the tests(!)


Timesheet - 13/04/2021 - James Pang

Task 1
Fixed issues/bugs in website
Proof of Work

https://github.com/thenewboston-developers/Website/pull/1706
https://github.com/thenewboston-developers/Website/pull/1707

Time Spent
1.5 hour

Task 2
Worked on teams api, changes based on updates from API. Resolved quite a few merge conflicts.
Proof of Work

https://github.com/thenewboston-developers/Website/pull/1663

Time Spent
2 hours

Total Time Spent
3.5 hours
Account Number
c54c04774efaae20746fd3f29cdd619fea512e119bc1082b863572b2e8844104

Approved

Approved


Generic RGBCCW CB2L Bulb (Daybetter Brand)

Dumped a paired bulb. Can also dump a stock (unpaired) bulb if it would help. Note that the stock bulb works with the Tuya-Generic/E27-RGBCW-Smart-Life-WB2L_M1 profile, while the paired bulb does not (SSL errors).
Note that the neither the bulb, packaging, nor listing include any model number.
dump.zip

"while the paired bulb does not (SSL errors)." this is interesting, does it mean that the firmware is remotely updated to the one without the exploit?

That's what I'm fearing. Perhaps I have mixed up batches of devices, but I really do not believe I have.

Not necessarily "without the exploit". It's enough if it updates to a different build of the same firmware, and the profile stops working already. @openshwprojects

Turns out there were two variants of this bulb, most likely depending on if it's a 12 pack or lesser quantity.
For those that identify as V2.9.16 once paired, these use a WB2L, and the Phopollo SBE2606 profile works.
For those that identify as V1.2.16 once paired, these use a CB2L, and the Tuya-Generic E27-RGBCW-Smart-Life-WB2L_M1 profile works.
Thanks to @Cossid and @kuba2k2 for helping me figure this out!


Changing Action Space to match the Openai Gym environments.

Issue summary
I am trying to train Atari Breakout, but for my use case, I need to be able to work with Retro, while keeping the results comparable to the standard training structure.
However, the gym Atari has a Discrete(4) action space whereas the Retro version has a MultiBinary[11]. Of course, this means that they are wildly different tasks, difficulty wise.
What is the proper way to change the gym-retro action spaces? Couldn't find the necessary hooks in the JSON files.
Thank you!

retro.make(game=game, use_restricted_actions=retro.Actions.DISCRETE)
It won't be the same number of actions since ALE uses the minimal action set, which is defined on a per-game basis.  You can use something like https://gist.github.com/christopherhesse/8e5c63c3b0007f4c3e333c6a158872cf to turn a multibinary into a discrete with whatever actions you want.  If there was a json file with the per-game allowed combos for ALE, I would be willing to add it to gym-retro somewhere (probably as a wrapper).

Thank you!


Improve data pending states on dashboard [ch3118]

Story Details: https://app.clubhouse.io/levels/story/3118/improve-data-pending-states-on-dashboard
First part of this feature. Show both lines in the given points and display a text in the middle. Text, lines color and height can be changed through parameters.
Second part will include the icon and the dashed line along with the x axis. I must create a separate example to avoid conflicts with the original repo and be able to update from upstream without too much trouble later.

Are we switching to a completely new chart library?

No. This is the same library that the RN bridge is using under the hood for native iOS. None of what we're using now was offering the feature we need so we have to implement it natively, then in the bridge and finally in our app. Next up: Android native implementation

Closing in favour of https://github.com/levelshealth/Charts/pull/2

Reopening because the performance issues were not related to the last version of this library


Update complete code sample

The current complete sample will crash when tapping on "Take Picture"

I signed it!

I signed it!


Xbox App Icon Request

Xbox App
com.microsoft.xboxone.smartglass

Hello! Weʼre switching to a new system for icon requests in order to be able to better manage them. From now on, icon requests can be submitted here: https://forms.gle/Fx8vZAiWdW1Tyjo57. If your icon request hasnʼt yet been fulfiled, it has been copied to our new icon request database, meaning no action is required. Consequently, this issue will be closed.


Error facedetect !!!! only eye and nose

The face detection/alignment module failed in your test photo. I use the opencv face detector, and CLM tracker to detect the points of eyes, mouth, and nose. You can replace the detector/alignment module with the latest and more accurate stuff.

Thank u !

can u tell me how to replace it  !  for example dlib


[bug] MinOZW /dev/ttyDevice works. Zwave2Mqtt does not

Version
Dev
Build/Run method

[x] Manually built (git clone - npm install - npm run build )

Zwave2Mqtt version: 2.1.1
Openzwave Version: 1.6-1026
Describe the bug
Running "MinOZW /dev/ttyZWAVE"after installing OpenZwave works correctly. Running and configuring Zwave2Mqtt is not able to connect to the /dev/ttyZWAVE device and throws the following error:
z2m:Zwave Exception calling OpenZWave::Manager::GetDriverStatistics in /home/node/open-zwave/cpp/src/Manager.cpp(405): Invalid HomeId passed to GetDriver getDriverStatistics  +13s
Additional context
Zwave2Mqtt output:
> node bin/www

  z2m:Store scenes.json not found +0ms
  z2m:Store nodes.json not found +43ms
  z2m:Gateway No customDevices file found +0ms
  z2m:App Application path:/home/node/Zwave2Mqtt +0ms
  z2m:Mqtt MQTT is disabled +0ms
Initialising OpenZWave 1.6.1026 binary addon for Node.JS.
        OpenZWave Security API is ENABLED
        ZWave device db    : /usr/local/etc/openzwave
        User settings path : /home/node/Zwave2Mqtt/store
        Option Overrides : --Logging false --ConsoleOutput false --QueueLogLevel 6 --DriverMaxAttempts 9999 --SaveConfiguration true --PollInterval 90000 --AutoUpdateConfigFile true --NetworkKey 0x0E,0xE6,0xFA,0xCC,0x63,0x19,0xB6,0x70,0xA3,0xEB,0xEB,0xCF,0xDC,0x16,0x17,0x5B
  z2m:Zwave Connecting to /dev/ttyZWAVE +0ms
  z2m:Zwave Zwave connected, Openzwave version: 1.6.1026 +3s
  z2m:App Listening on port 8091 +0ms
  z2m:Zwave Notification from node 255: Notification - TimeOut (1) +1s
  z2m:App New connection Wr5EekgKqf3bhBxnAAAA +8s
GET / 304 236.025 ms - -
GET /static/css/app.6071a1385bbbf2e6bba673b992155bbf.css 304 10.612 ms - -
GET /static/js/manifest.2ae2e69a05c33dfc65f8.js 304 10.859 ms - -
GET /static/js/vendor.61eca3eb3f1460776dd3.js 304 12.197 ms - -
GET /static/js/app.2f898150cffed81c0ac8.js 304 12.322 ms - -
  z2m:App New connection pqwcNo0gw320aWDDAAAB +540ms
GET /static/logo.png 304 16.269 ms - -
GET /static/favicons/favicon-32x32.png 200 25.320 ms - 1831
  z2m:App Zwave api call: getDriverStatistics [] +8s
  z2m:Zwave Exception calling OpenZWave::Manager::GetDriverStatistics in /home/rait/open-zwave/cpp/src/Manager.cpp(405): Invalid HomeId passed to GetDriver getDriverStatistics  +13s
^C  z2m:App Closing... +4s
  z2m:Gateway Closing Gateway... +20s

MinOZW output:
Starting MinOZW with OpenZWave Version 1.6-1026-g42ebaa96
2020-02-05 14:16:02.061 Error, Cannot find a path to the configuration files at ../../../config/, Using /usr/local/etc/openzwave/ instead...
2020-02-05 14:16:02.063 Info, Reading /usr/local/etc/openzwave/options.xml for Options
2020-02-05 14:16:02.063 Warning, Failed to Parse options.xml: Failed to open file
2020-02-05 14:16:02.063 Info, Options:
2020-02-05 14:16:02.063 Info,   appendlogfile: false
2020-02-05 14:16:02.064 Info,   associate: true
2020-02-05 14:16:02.064 Info,   assumeawake: true
2020-02-05 14:16:02.064 Info,   autoupdateconfigfile: true
2020-02-05 14:16:02.064 Info,   configpath: /usr/local/etc/openzwave/
2020-02-05 14:16:02.064 Info,   consoleoutput: true
2020-02-05 14:16:02.064 Info,   customsecuredcc: 0x62,0x4c,0x63
2020-02-05 14:16:02.064 Info,   drivermaxattempts: 5
2020-02-05 14:16:02.064 Info,   dumptrigger: 4
2020-02-05 14:16:02.064 Info,   dumptriggerlevel: 1
2020-02-05 14:16:02.064 Info,   enablesis: true
2020-02-05 14:16:02.064 Info,   enforcesecurereception: true
2020-02-05 14:16:02.064 Info,   exclude:
2020-02-05 14:16:02.064 Info,   include:
2020-02-05 14:16:02.064 Info,   includeinstancelabel: true
2020-02-05 14:16:02.064 Info,   interface:
2020-02-05 14:16:02.064 Info,   intervalbetweenpolls: true
2020-02-05 14:16:02.064 Info,   language:
2020-02-05 14:16:02.064 Info,   logfilename: OZW_Log.txt
2020-02-05 14:16:02.064 Info,   logging: true
2020-02-05 14:16:02.065 Info,   networkkey:
2020-02-05 14:16:02.065 Info,   notifyondriverunload: false
2020-02-05 14:16:02.065 Info,   notifytransactions: false
2020-02-05 14:16:02.065 Info,   performreturnroutes: false
2020-02-05 14:16:02.065 Info,   pollinterval: 500
2020-02-05 14:16:02.065 Info,   queueloglevel: 9
2020-02-05 14:16:02.065 Info,   refreshallusercodes: false
2020-02-05 14:16:02.065 Info,   reloadafterupdate: AWAKE
2020-02-05 14:16:02.065 Info,   retrytimeout: 10000
2020-02-05 14:16:02.065 Info,   saveconfiguration: true
2020-02-05 14:16:02.065 Info,   saveloglevel: 8
2020-02-05 14:16:02.065 Info,   securitystrategy: SUPPORTED
2020-02-05 14:16:02.065 Info,   suppressvaluerefresh: false
2020-02-05 14:16:02.065 Info,   userpath:
2020-02-05 14:16:02.065 Info,   validatevaluechanges: true
2020-02-05 14:16:02.066 Always, OpenZwave Version 1.6-1026-g42ebaa96 Starting Up
2020-02-05 14:16:02.099 Info, Loading Localization File /usr/local/etc/openzwave/Localization.xml
2020-02-05 14:16:02.105 Info, Loaded /usr/local/etc/openzwave/Localization.xml With Revision 8
2020-02-05 14:16:02.109 Always, Using Language Localization
2020-02-05 14:16:02.119 Info, Loading NotificationCCTypes File /usr/local/etc/openzwave/NotificationCCTypes.xml
2020-02-05 14:16:02.120 Info, Loaded /usr/local/etc/openzwave/NotificationCCTypes.xml With Revision 9
2020-02-05 14:16:02.126 Info, Loading SensorMultiLevelCCTypes File /usr/local/etc/openzwave/SensorMultiLevelCCTypes.xml
2020-02-05 14:16:02.127 Info, Loaded /usr/local/etc/openzwave/SensorMultiLevelCCTypes.xml With Revision 3
2020-02-05 14:16:02.129 Info, Setting Up Provided Network Key for Secure Communications
2020-02-05 14:16:02.129 Warning, Failed - Network Key Not Set
2020-02-05 14:16:02.183 Info, Manufacturer_Specific.xml file Revision is 74
2020-02-05 14:16:03.312 Info, Product name collision: FGR223 Roller Shutter Controller 3 type 303 id 1000 manufacturerid 10f, collides with FGRM223 Roller Shutter Controller 3, type 303 id 1000 manufacturerid 10f
2020-02-05 14:16:04.038 Info, Product name collision: ED2.0 Meter Adapter type 0 id 0 manufacturerid 128, collides with ED2.0 Meter Adapter, type 0 id 0 manufacturerid 128
2020-02-05 14:16:04.038 Info, Product name collision: ED2.0 Display type 128 id 0 manufacturerid 128, collides with ED2.0 Display, type 128 id 0 manufacturerid 128
2020-02-05 14:16:04.396 Info, Product name collision: GR105 Auto Valve type 202 id 511 manufacturerid 152, collides with GR105 Auto Valve, type 202 id 511 manufacturerid 152
2020-02-05 14:16:04.396 Info, Product name collision: GR105n Auto Valve type 3 id 512 manufacturerid 152, collides with GR105n Auto Valve, type 3 id 512 manufacturerid 152
2020-02-05 14:16:04.396 Info, Product name collision: ZSE02 Motion Sensor type 500 id 3 manufacturerid 152, collides with ZSE02 Motion Sensor, type 500 id 3 manufacturerid 152
2020-02-05 14:16:04.447 Info, Product name collision: ZWN-BPC Ceiling Mounted PIR Motion Sensor type 601 id 901 manufacturerid 11a, collides with ZWN-BPC Ceiling Mounted PIR Motion Sensor, type 601 id 901 manufacturerid 11a
2020-02-05 14:16:04.447 Info, Product name collision: ZWN-SC7 7-Button Scene Controller type 801 id b03 manufacturerid 11a, collides with ZWN-SC7 7-Button Scene Controller, type 801 id b03 manufacturerid 11a
2020-02-05 14:16:04.447 Info, Product name collision: ZW15S 15A On/Off Switch type 101 id 102 manufacturerid 11a, collides with ZW15S 15A On/Off Switch, type 101 id 102 manufacturerid 11a
2020-02-05 14:16:04.447 Info, Product name collision: ZW20R 20A TR Duplex Receptacle type 101 id 603 manufacturerid 11a, collides with ZW20R 20A TR Duplex Receptacle, type 101 id 603 manufacturerid 11a
2020-02-05 14:16:04.447 Info, Product name collision: ZWN-RSM1 PLUS-Smart Single Relay Switch Module type 111 id 605 manufacturerid 11a, collides with ZWN-RSM1 PLUS—Smart Single Relay Switch Module, type 111 id 605 manufacturerid 11a
2020-02-05 14:16:04.447 Info, Product name collision: ZW20RM 20A TR Smart Meter Duplex Receptacle type 111 id 101 manufacturerid 11a, collides with ZW20RM 20A TR Smart Meter Duplex Receptacle, type 111 id 101 manufacturerid 11a
2020-02-05 14:16:04.448 Info, Product name collision: ZW500D 500W In-Wall Preset Dimmer Switch type 102 id 201 manufacturerid 11a, collides with ZW500D 500W In-Wall Preset Dimmer Switch, type 102 id 201 manufacturerid 11a
2020-02-05 14:16:04.448 Info, Product name collision: ZW15RM Plus 15A TR Smart Meter Duplex Receptacle type 111 id 105 manufacturerid 11a, collides with ZW15RM Plus 15A TR Smart Meter Duplex Receptacle, type 111 id 105 manufacturerid 11a
2020-02-05 14:16:04.579 Info, Queuing Lookup on mfs.db.openzwave.com for Node 0
2020-02-05 14:16:04.581 Info,   Opening controller /dev/ttyZWAVE
2020-02-05 14:16:04.581 Info, mgr,     Added driver for controller /dev/ttyZWAVE
2020-02-05 14:16:04.582 Info, Starting DNSThread
2020-02-05 14:16:04.582 Info, LookupTxT Checking mfs.db.openzwave.com
2020-02-05 14:16:04.582 Info, Timer: thread starting
2020-02-05 14:16:04.583 Info, Trying to open serial port /dev/ttyZWAVE (attempt 1)
2020-02-05 14:16:04.583 Info, Serial port /dev/ttyZWAVE opened (attempt 1)
2020-02-05 14:16:04.584 Detail, Timer: waiting with timeout -1 ms
2020-02-05 14:16:04.585 Detail, contrlr, Queuing (Command) FUNC_ID_ZW_GET_VERSION: 0x01, 0x03, 0x00, 0x15, 0xe9
2020-02-05 14:16:04.608 Info, Lookup for mfs.db.openzwave.com returned 74
2020-02-05 14:16:04.620 Info, ManufacturerSpecificDB Initialized
2020-02-05 14:16:04.620 Detail, Notification: ManufacturerSpecificDB Ready
Notification: ManufacturerSpecificDB Ready
2020-02-05 14:16:04.620 Detail,
2020-02-05 14:16:04.621 Info, contrlr, Sending (Command) message (Callback ID=0x00, Expected Reply=0x15) - FUNC_ID_ZW_GET_VERSION: 0x01, 0x03, 0x00, 0x15, 0xe9
2020-02-05 14:16:04.621 Info, contrlr, Encrypted Flag is 0
2020-02-05 14:16:05.621 Error, contrlr, ERROR: Dropping command, expected response not received after 1 attempt(s). Command: "FUNC_ID_ZW_GET_VERSION: 0x01, 0x03, 0x00, 0x15, 0xe9"
2020-02-05 14:16:05.621 Detail, contrlr, Removing current message
2020-02-05 14:16:05.621 Detail, contrlr, Notification: Notification - TimeOut
Notification: Notification - TimeOut

OpenZWave contributor here... We've seen similar things before, it is not caused by Z2M and also not by OZW.
Something is interfering with your serial port.

Running "MinOZW /dev/ttyZWAVE"after installing OpenZwave works correctly.

You may think so, but it does not run as expected, this error is a sign of trouble on that device...
2020-02-05 14:16:05.621 Error, contrlr, ERROR: Dropping command, expected response not received after 1 attempt(s). Command: "FUNC_ID_ZW_GET_VERSION: 0x01, 0x03, 0x00, 0x15, 0xe9"
And this is another sign of something interfering with Z2M:
z2m:Zwave Notification from node 255: Notification - TimeOut (1) +1s
Node 255 is a broadcast address, that cannot be a valid Node ID..
I copy pasted this advice from the HASS website (credit where credit due) and I am sure it is sane advice:
AFIAK the "modemmanager" package will interfere with any Z-Wave (or Zigbee) stick and should be removed or disabled in the host OS. Failure to do so will result in random failures of those components, e.g. dead or unreachable Z-Wave nodes, most notably right after Home Assistant restarts. Connect to your host OS via SSH, then you can disable with sudo systemctl disable ModemManager and remove with sudo apt-get purge modemmanager (commands are for Debian/Ubuntu).

I guess this issue belongs in the Openzwave repository in that case.

guess this issue belongs in the Openzwave repository in that case.

Nope, it is your system...


Adds some missing translations

I would suggest squashing this commit. Sorry for the mess, did it in the browser ...
@hed854 Could you check my french translations? even so I live very close to the border, I have no idea if that is correct xD I'll give you permissions on the repo, that will makes things easier I guess ;-)

@hed854 thank you very much!🎉

@hed854 thank you very much!🎉

Thanks!

Thanks!


endless visualization of trajectory

hi,
is it possible to add a feature to display endlessly the trajectory of the learned inference function ?
At the moment it seemed to be fixed to "20" , i think it is "seconds"  ?
thx

By endlessly do you mean loop?  Our next push of the javascript viewer will have a 'loop' checkbox in the animation control panel.
If you mean actually mean infinite length trajectories, this is on our roadmap but will not happen straight away.  We are refactoring the viewer to support interactive physics, at which point you will be able to stream physics steps/inference indefinitely.
And yes, 20 seconds trajectory is what you would expect if your episode has 1000 frames with .02s dt.

i mean infinite length trajectory , if its on the roadmap thats cool.


Live demonstrator for classification network

Hi,
This is not an issue but a request for clarification.
I would like to use the Graph Interface to build a simple network topology used for image classification.
I am wondering if exists a memory accessor useful to feed the InputLayer.
The examples show how to feed the InputLayer with jpeg,ppm, numpy and random data but in case of a live demonstrator I plan to use a camera.
Do I have to write my own feeder ?
Thanks
Davide

Helllo @ddenaro,
Yes, you have to write your own Accessor to feed data from a camera into the graph.

Ok
thanks


Unable to find any valuable Window provider

Versions

Python: 3.7.0
OS: Windows 10
Kivy: 1.10.1
Kivy installation method: I followed the exact steps found here https://kivy.org/doc/stable/installation/deps-cython.html
(doing pip install -U --force-reinstall Cython==0.27.3) and here https://kivy.org/doc/stable/installation/installation-windows.html
(doing:
python -m pip install --upgrade pip wheel setuptools
python -m pip install docutils pygments pypiwin32 kivy.deps.sdl2 kivy.deps.glew
python -m pip install kivy.deps.gstreamer
python -m pip install kivy.deps.angle
python -m pip install kivy
)

Description
Trying to install Kivy
Code and Logs
[INFO   ] Logger: Record log in C:\Users\CASTILLO\.kivy\logs\kivy_18-12-10_124.txt
[INFO   ] Kivy: v1.10.1
[INFO   ] Python: v3.7.0 (v3.7.0:1bf9cc5093, Jun 27 2018, 04:06:47) [MSC v.1914 32 bit (Intel)]
[INFO   ] Factory: 194 symbols loaded
[INFO   ] Image: Providers: img_tex, img_dds, img_gif (img_sdl2, img_pil, img_ffpyplayer ignored)
[INFO   ] Text: Provider: sdl2
[CRITICAL] Window: Unable to find any valuable Window provider.
sdl2 - ImportError: DLL load failed: The specified procedure could not be found.
  File "C:\Users\CASTILLO\AppData\Local\Programs\Python\Python37-32\lib\site-packages\kivy\core\__init__.py", line 59, in core_select_lib
    fromlist=[modulename], level=0)
  File "C:\Users\CASTILLO\AppData\Local\Programs\Python\Python37-32\lib\site-packages\kivy\core\window\window_sdl2.py", line 26, in <module>
    from kivy.core.window._window_sdl2 import _WindowSDL2Storage

[CRITICAL] App: Unable to get a Window, abort.

import kivy
kivy.require('1.10.1')

from kivy.app import App
from kivy.uix.label import Label

class MyApp(App):

    def build(self):
        return Label(text='Hello world')

if __name__ == '__main__':
    MyApp().run()

I got the same issue on windows 10 but I was able to solve it.
Versions
Python : 3.7.1
OS: Windows 10
Kivy: 1.10.1
Fix
I found that the problem are the dependencies used by Kivy, so I use pip tools to get a good set of dependencies.
Install pip-tools
pip-tools is a great tool to get your pacakges on place you could get more information here
pip install pip-tools

Generate requirements.in
From the kivy page I got the main libraries that I need and based on that I created my requirments.in
(.venv) PS D:\projects\game> Get-Content -path .\requirements.in
kivy
cython
docutils
pygments
pypiwin32
kivy.deps.sdl2
kivy.deps.glew
kivy.deps.gstreamer
(.venv) PS D:\projects\game>

Compile requirements.in
(.venv) PS D:\projects\game> pip-compile.exe .\requirements.in
#
# This file is autogenerated by pip-compile
# To update, run:
#
#    pip-compile --output-file .\requirements.txt .\requirements.in
#
certifi==2018.11.29       # via requests
chardet==3.0.4            # via requests
cython==0.29.2
docutils==0.14
idna==2.7                 # via requests
kivy-garden==0.1.4        # via kivy
kivy.deps.glew==0.1.10
kivy.deps.gstreamer==0.1.13
kivy.deps.sdl2==0.1.18
kivy==1.10.1
pygments==2.3.0
pypiwin32==223
pywin32==224              # via pypiwin32
requests==2.20.1          # via kivy-garden
urllib3==1.24.1           # via requests
(.venv) PS D:\projects\game>

Install depdencies
In this point you could use different commands but just to continue with pip-tools I use pip-sync:
(.venv) PS D:\projects\game> pip-sync.exe .\requirements.txt

Please use the requirments.txt to install the dependencies.
I hope this help you a little bit.

What is the problem with dependencies that you guys have in the first place?
@VictorCabello you didn't installed kivy.deps.angle while @Raigiku used it
Is that the issue here?

Sorry, I can not give you any more information since I deleted the project and replaced my computer.

Installing cython was what fixed it for me.

I had the same problem. I solved it by first uninstalling Kivy and its dependencies :
python -m pip uninstall kivy
python -m pip uninstall kivy.deps.sdl2
python -m pip uninstall kivy.deps.glew
python -m pip uninstall kivy.deps.gstreamer
Now reinstalling everything except gstreamer.
python -m pip install --upgrade pip wheel setuptools
python -m pip install docutils pygments pypiwin32 kivy.deps.sdl2 kivy.deps.glew --extra-index-url https://kivy.org/downloads/packages/simple/
the https://kivy.org/......etc is the end of the previous python -m pip install docutils....etc...
then install kivy again
python -m pip install kivy
It should work, it worked for me !!!

@supergitnessy thank you so much that worked for me!

I'm getting the same error with kivy 1.11.1 on win10, with both angle and glew installed. I've tried uninstalling and reinstalling a number of different ways. I don't have (or need) gstreamer installed, but most of the posts I've seen either don't mention it or, as above, explicitly note leaving it out of the reinstall process.
I've just gone through the steps as listed by supergitnessy, then reran my app.

C:\Users\krenshala\Desktop>python testing.py
[INFO   ] [Logger      ] Record log in C:\Users\krenshala\.kivy\logs\kivy_20-05-17_7.txt
[INFO   ] [deps        ] Successfully imported "kivy_deps.angle" 0.1.10
[INFO   ] [deps        ] Successfully imported "kivy_deps.glew" 0.2.0
[INFO   ] [deps        ] Successfully imported "kivy_deps.sdl2" 0.2.0
[INFO   ] [Kivy        ] v1.11.1
[INFO   ] [Kivy        ] Installed at "C:\Users\krenshala\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\kivy\__init__.py"
[INFO   ] [Python      ] v3.7.7 (tags/v3.7.7:d7c567b08f, Mar 10 2020, 11:52:54) [MSC v.1900 64 bit (AMD64)]
[INFO   ] [Python      ] Interpreter at "C:\Users\krenshala\AppData\Local\Microsoft\WindowsApps\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\python.exe"
[INFO   ] [Factory     ] 184 symbols loaded
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_pil, img_gif (img_sdl2, img_ffpyplayer ignored)
[CRITICAL] [Window      ] Unable to find any valuable Window provider. Please enable debug logging (e.g. add -d if running from the command line, or change the log level in the config) and re-run your app to identify potential causes
sdl2 - ImportError: DLL load failed: The specified module could not be found.
  File "C:\Users\krenshala\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\kivy\core\__init__.py", line 63, in core_select_lib
    fromlist=[modulename], level=0)
  File "C:\Users\krenshala\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\kivy\core\window\window_sdl2.py", line 27, in <module>
    from kivy.core.window._window_sdl2 import _WindowSDL2Storage

[CRITICAL] [App         ] Unable to get a Window, abort.

C:\Users\krenshala\Desktop>

And the code I tried to run:
from kivy.uix.widget import Widget

class Testing(Widget):
    pass

class TestingApp(App):
    def build(self):
        return Testing()

if __name__ == "__main__":
    TestingApp().run()

@krenshala I have the same problem and no idea how to fix it. I have tried everything suggested in this post without success. I just started studying Python in my free time and it's a real pain in the *** to install this Kivy.

For support using kivy, please ask on discord rather than commenting on old issues. These comments posted here are all about unrelated issues.


files are not displayed properly on github

for example: https://github.com/chipitsine/haproxy/security/code-scanning/23?query=ref%3Arefs%2Fheads%2Fmaster

I guess it might be because of "workspace" in path. shouldn't it be removed ?

another question - the above finding is documentation. I do not think it worth being scanned. How any bugs from documentation can affect us ?
shouldn't documentation (i.e. "*.txt" files) be skipped by default ?
or can I skip them manually ?

another question - the above finding is documentation. I do not think it worth being scanned. How any bugs from documentation can affect us ?
shouldn't documentation (i.e. "*.txt" files) be skipped by default ?

This rule does appear to be written to only apply to code (though code of any language). Offhand I don't think we have a distinction between code and documentation. Just code and comment, and if something isn't a comment it's considered code. Since .txt doesn't have defined comment extensions the snippet here wouldn't be commented and would thus be "code".
At the very least I'll add an option so you can provide some extensions to skip.

or can I skip them manually ?

You can manually mark the issue as non relevant.

another question - the above finding is documentation. I do not think it worth being scanned. How any bugs from documentation can affect us ?
shouldn't documentation (i.e. "*.txt" files) be skipped by default ?

This rule does appear to be written to only apply to code (though code of any language). Offhand I don't think we have a distinction between code and documentation. Just code and comment, and if something isn't a comment it's considered code. Since .txt doesn't have defined comment extensions the snippet here wouldn't be commented and would thus be "code".
At the very least I'll add an option so you can provide some extensions to skip.

or can I skip them manually ?

You can manually mark the issue as non relevant.

there are trillions of "TLS" in txt files. can I skip entire "doc" folder in some DevSkim settings ?
also, txt does not appear to be language of any kind, I still think it might be whitelisted entirely


SponsorBlock unavailable: program exits

When using "--sponsorblock-mark all"  if the following error occurs the program exits rather than proceed with download.
[youtube] xxxxxxxxxxxxxx: Downloading webpage
[youtube] xxxxxxxxxxxxxx: Downloading android player API JSON
ERROR: Error communicating with SponsorBlock API - HTTP Error 502: Bad Gateway

IGNORING THE ISSUE TEMPLATE WILL RESULT IN ISSUE CLOSED AS INCOMPLETE
I am not going to close this since I was able to identify the issue even with the lack of information. However, next time, follow the issue template

[debug] Command-line config: ['-v', '--no-config', '--sponsorblock-mark', 'all', 'https://www.youtube.com/watch?v=C0DPdy98e4c']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] yt-dlp version 2021.09.25
[debug] Python version 3.9.4 (CPython 64bit) - Windows-10-10.0.22000-SP0
[debug] exe versions: ffmpeg n4.4-80-gbf87bdd3f6-20210811, ffprobe n4.4-80-gbf87bdd3f6-20210811, phantomjs 2.1.1, rtmpdump 2.3
[debug] Optional libraries: Crypto, mutagen, sqlite, websockets
[debug] Proxy map: {}
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=C0DPdy98e4c
[youtube] C0DPdy98e4c: Downloading webpage
[youtube] C0DPdy98e4c: Downloading android player API JSON
[debug] Sort order given by extractor: quality, res, fps, source, codec:vp9.2, lang
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, source, vcodec:vp9.2(10), acodec, lang, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, id
[debug] Testing thumbnail 41
[debug] Unable to connect to thumbnail 41 URL "https://i.ytimg.com/vi_webp/C0DPdy98e4c/maxresdefault.webp" - HTTP Error 404: Not Found. Skipping...
[debug] Testing thumbnail 40
[debug] Unable to connect to thumbnail 40 URL "https://i.ytimg.com/vi/C0DPdy98e4c/maxresdefault.jpg" - HTTP Error 404: Not Found. Skipping...
[debug] Testing thumbnail 39
[debug] Unable to connect to thumbnail 39 URL "https://i.ytimg.com/vi_webp/C0DPdy98e4c/hq720.webp" - HTTP Error 404: Not Found. Skipping...
[debug] Testing thumbnail 38
[debug] Unable to connect to thumbnail 38 URL "https://i.ytimg.com/vi/C0DPdy98e4c/hq720.jpg" - HTTP Error 404: Not Found. Skipping...
[debug] Testing thumbnail 37
[debug] Unable to connect to thumbnail 37 URL "https://i.ytimg.com/vi_webp/C0DPdy98e4c/sddefault.webp" - HTTP Error 404: Not Found. Skipping...
[debug] Testing thumbnail 36
[debug] Unable to connect to thumbnail 36 URL "https://i.ytimg.com/vi/C0DPdy98e4c/sddefault.jpg" - HTTP Error 404: Not Found. Skipping...
[debug] Testing thumbnail 35
[debug] SponsorBlock query: https://sponsor.ajay.app/api/skipSegments/754e?service=YouTube&categories=%5B%22intro%22%2C+%22sponsor%22%2C+%22preview%22%2C+%22outro%22%2C+%22interaction%22%2C+%22music_offtopic%22%2C+%22selfpromo%22%5D
ERROR: Error communicating with SponsorBlock API - HTTP Error 502: Bad Gateway
Traceback (most recent call last):
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\postprocessor\sponsorblock.py", line 90, in _get_json
    rsp = self._downloader.urlopen(sanitized_Request(url))
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 3256, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\urllib\request.py", line 523, in open
    response = meth(req, response)
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\urllib\request.py", line 632, in http_response
    response = self.parent.error(
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\urllib\request.py", line 561, in error
    return self._call_chain(*args)
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\urllib\request.py", line 494, in _call_chain
    result = func(*args)
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\urllib\request.py", line 641, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 502: Bad Gateway

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 1227, in wrapper
    return func(self, *args, **kwargs)
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 1265, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 1315, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 2272, in process_video_result
    info_dict, _ = self.pre_process(info_dict)
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 3043, in pre_process
    info = self.run_pp(pp, info)
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 2996, in run_pp
    files_to_delete, infodict = pp.run(infodict)
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\postprocessor\sponsorblock.py", line 36, in run
    info['sponsorblock_chapters'] = self._get_sponsor_chapters(info, info['duration'])
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\postprocessor\sponsorblock.py", line 40, in _get_sponsor_chapters
    segments = self._get_sponsor_segments(info['id'], self.EXTRACTORS[info['extractor_key']])
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\postprocessor\sponsorblock.py", line 82, in _get_sponsor_segments
    for d in self._get_json(url):
  File "c:\users\hassaan\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\postprocessor\sponsorblock.py", line 94, in _get_json
    raise PostProcessingError(f'Error communicating with SponsorBlock API - {e}')
yt_dlp.utils.PostProcessingError: Error communicating with SponsorBlock API - HTTP Error 502: Bad Gateway


filters: supporting alternative characters for quote

The regexp syntax can have some nasty characters to handle quoting
correctly, in practice. In certain scenarios a double quote works well,
but it can affect readability for certain regexps. This introduces a
quoting mode that treats / and | as a double quote to make the
quoting in regular expressions more familiar and readable.
This change is introduced in a backwards compatible manner, so existing
regexp quoting is not affected.
Closes #1233
Signed-off-by: Stephen J Day<EMAIL_ADDRESS>

Codecov Report

Merging #1874 into master will decrease coverage by 0.53%.
The diff coverage is 24%.

@@            Coverage Diff             @@
##           master    #1874      +/-   ##
==========================================
- Coverage   49.07%   48.53%   -0.54%
==========================================
  Files          86       87       +1
  Lines        8263     8435     +172
==========================================
+ Hits         4055     4094      +39
- Misses       3538     3661     +123
- Partials      670      680      +10

Flag
Coverage Δ

#linux
51.83% <25.33%> (-0.62%)
:arrow_down:

#windows
43.74% <24%> (-0.51%)
:arrow_down:

Impacted Files
Coverage Δ

filters/scanner.go
77.52% <100%> (+0.12%)
:arrow_up:

filters/parser.go
79.48% <100%> (ø)
:arrow_up:

filters/quote.go
22.22% <22.22%> (ø)

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 4a662b2...3960d6a. Read the comment docs.

LGTM

LGTM


NERDTreeHijackNetrw ignored when using Default Key: e

Docs state:
Default key: e
Map option: NERDTreeMapOpenExpl
Applies to: files and directories.

|:edit|s the selected directory, or the selected file's directory. This could
result in a NERD tree or a netrw being opened, depending on
|'NERDTreeHijackNetrw'|.

But even with NERDTreeHijackNetrw set to 0, pressing 'e' on a directory still opens the directory in a 'secondary' NERDTree.

It is used here:
https://github.com/MarcoWorms/vimfiles/blob/06fc8a4438a361630c8a1d78f7e77ee06dc33991/init.vim#L138-L140

This bug has been confirmed.  Also, I'm noting that the documentation here is wrong.  The e mapping does not currently operate properly on file nodes.


Strip off one extra bit at start of read out

This fixes nonsense readings on the BBG.
I'm not really sure whether this causes troubles on the BBB, maybe a
separate platform for the BBG should be created, maybe it can be fixed
with some timing adjustments? Any way, this change seems to give me
sensible readings.
Before:
debian@beaglebone:~/Adafruit_Python_DHT-master$ sudo ./examples/AdafruitDHT.py 11 P9_22
Data: 0x93 0x80 0xd 0x4 0x24
Temp=13.0*  Humidity=147.0%

After:
debian@beaglebone:~/Adafruit_Python_DHT-master$ sudo ./examples/AdafruitDHT.py 11 P9_22
Data: 0x27 0x0 0x1a 0x7 0x48
Temp=26.0*  Humidity=39.0%

hi this repo is going to be deprecated soon, please try
https://github.com/adafruit/Adafruit_CircuitPython_DHT
which will replace this library

Well, that was the last time I fixed a bug for you. It goes one and a half year without even looking at it and then is closed, again without looking at it. :-1:


Invalid InterceptionId. (-32602)

What versions are you running?
v0.7.8
$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.7.8
$ google-chrome --version
98.0.4758.102
$ go version
go version go1.17.1 windows/amd64

What did you do? Include clear steps.
          chromedp.ListenTarget(ctx, func(ev interface{}) {
			go func() {
				switch ev := ev.(type) {
				case *fetch.EventRequestPaused:
					c := chromedp.FromContext(ctx)
					execCtx := cdp.WithExecutor(ctx, c.Target)
					err := fetch.FailRequest(ev.RequestID, network.ErrorReasonFailed).Do(execCtx)
					if err != nil {
						log.Print(err)
					}
				}
			}()
		})

		if err := chromedp.Run(ctx,
			fetch.Enable().WithPatterns([]*fetch.RequestPattern{
				{
					URLPattern: pattern,
					RequestStage: "Request"
				},
			}),
			chromedp.Navigate(url),
		); err != nil {
			log.Fatal(err)
		}

What did you expect to see?
Requests matching pattern are failed.
What did you see instead?
Invalid InterceptionId. (-32602) error. This is occurring randomly and started with no code changes. Same code had been working fine for weeks.

This looks like an issue of the website instead of your code. Here is a reported issue that has more information: https://groups.google.com/g/chrome-debugging-protocol/c/Ztez863JGh8?pli=1
You can log the url which causes the error and see what's wrong with that url.
	chromedp.ListenTarget(ctx, func(ev interface{}) {
		switch ev := ev.(type) {
		case *fetch.EventRequestPaused:
			// create a new goroutine only when it's necessary
			go func() {
				// use chromedp.Run directly to simplify the source code
				err := chromedp.Run(ctx, fetch.FailRequest(ev.RequestID, network.ErrorReasonFailed))
				if err != nil {
					// log the URL when there is an error
					log.Print(err, ev.Request.URL)
				}
			}()
		}
	})

@ZekeLu yes it is always the same url since the pattern is a static one. Looking at the linked post it is dealing with the response mime type not matching. In my case I have the fetch on the Request stage and fail it before even going out to the url.

I believe that when the error is reported, there is something wrong with the request and it's not sent. Since your purpose is to block the request, you can safely ignore the error.
Here is another case shows when it will result in Invalid InterceptionId. (-32602): https://github.com/chromium/chromium/blob/c4d3c31083a2e1481253ff2d24298a1dfe19c754/third_party/blink/web_tests/http/tests/inspector-protocol/network/response-interception-cancel-xhr-while-responding-error.js . I hope that it helps you understand what does Invalid InterceptionId. (-32602) mean.
I can not do anything more without more information. Thanks!

@ZekeLu so I can ignore it for when I need to fail; requests. However I am also in need to modify a request response. From the article you linked it happens if a request is cancelled. To the best of my knowledge that is not occurring here. Code for the request modification:
chromedp.ListenTarget(ctx, func(ev interface{}) {
		switch ev := ev.(type) {
		case *fetch.EventRequestPaused:
			go func() {
				c := chromedp.FromContext(ctx)
				e := cdp.WithExecutor(ctx, c.Target)
				rawJsonB, err := fetch.GetResponseBody(ev.RequestID).Do(e)
				if err != nil {
					log.Println(err) // fails here with the Invalid InterceptionId
				}
				rawJson := string(rawJsonB)

                               ...
			}()
		}
	})

if err := chromedp.Run(ctx,
		fetch.Enable().WithPatterns([]*fetch.RequestPattern{
			{
				URLPattern: pattern,
				RequestStage: "Response",
			},
		}),
		chromedp.Navigate(url),
	); err != nil {
		log.Fatal(err)
	}

The key is to find out why the browser thinks that the interception id is invalid. Like I said before, the current information is not enough to find out the reason. It will be very helpful if you can provide a self-contained minimal reproducer (including the real URL, but a fake HTTP server which can be used to reproduce the issue is better).


Improve error handling

Errors when apps_domain is not set
Better message when CONFIG_PATH is not set
Better message when fails to open CONFIG_PATH
Better message when CONFIG_PATH is invalid json

@kirederik why should apps_domain be mandatory now?

@dlresende That's not changing the existing behaviour. Apps domain is mandatory already

Hi @kirederik -- We needed to apply some of your changes manually because some files no longer exist. Thank you!
George and Alberto


Support ReadyToRun format

Confuserex on C# 6.0 ReadyToRun image, it appear error:
Assembly does not appear to be a .NET assembly: ".NET data directory RVA is 0".

If file doesn't have dotnet rva seems like this file compiled into native code, im sure there's should be a dll file instead of exe, use the dll one for the obfuscation


Option to Hide Errors in the Overview Ruler

Add a feature to hide errors in the Overview Ruler much like editor.hideCursorInOverviewRuler hides the cursor in the Overview Ruler.
This setting could be editor.hideErrorsInOverviewRuler

You can use transparent colors for them which results in them being hidden:
"workbench.colorCustomizations": {
	"editorOverviewRuler.warningForeground": "#00000000",
	"editorOverviewRuler.errorForeground": "#00000000",
}


A null string is returned from the argument

Overview

If you define a nullable StringArgument and set it to null, it will become a "null" string.
We want null to be returned instead of a string.

Fixed in Navigation library.
So close this issue.
https://developer.android.com/jetpack/androidx/releases/navigation#2.6.0-alpha04
https://issuetracker.google.com/issues/247637434
https://android-review.googlesource.com/c/platform/frameworks/support/+/2244017


strings: Builder.String should either reset the Builder or avoid using unsafe semantics

Affects all Go versions since ever.
In some situation it may be desirable to build several strings which happen to prefix each other. Examples of such uses are filesystem hierarchies, x509 OIDs and other similar objects.
Nothing in the strings.Builder documentation implies, that an enterprising user will not be able to keep adding fragments to the builder after calling the String method once or twice.
Let's consider the following example:
var b strings.Builder
b.Grow(20)
b.WriteString("aaa")
x := b.String()
fmt.Printf("%s %p\n", x, unsafe.StringData(x))
b.WriteString("bbb")
y := b.String()
fmt.Printf("%s %p\n", y, unsafe.StringData(y))

Outputs:

aaa 0xc0000b2000
aaabbb 0xc0000b2000

Basically, we are able to break a cornerstone Go invariant with some really basic usage of "beginner" level facilities without any warning.
As a side note, it is not obvious why Builder is explicitly made non-copyable. Nothing bad will happen if a similar structure is copied here and there, it keeps working just fine, unlike this here issue.

But it is safe to do so in go, strings are immutable. Right?

I don't see anything to do here. strings consist of more than just the data part, appending to the builder won't change the original string.

As others have said, I'm not sure I see what the problem here is. Strings are safely allowed to alias, and IIRC the only requirement is that their underlying data is immutable. You also can't re-slice a string to be larger like you can with a slice, because it doesn't have a capacity like slices do. Finally, strings.Builder provides no facility for mutating that data, only appending.

Basically, we are able to break a cornerstone Go invariant with some really basic usage of "beginner" level facilities without any warning.

Which invariant do you mean?

As a side note, it is not obvious why Builder is explicitly made non-copyable. Nothing bad will happen if a similar structure is copied here and there, it keeps working just fine, unlike this here issue.

This requirement is necessary to achieve the performance characteristics Builder strives for while maintaining string invariants. Namely, if you copy Builder and write to both copies, the buffer the internal implementation carries may have the same memory written to more than once. Making that safe for copying would require having String return a copy of the internal buffer, which the type is explicitly trying to avoid.
This restriction does make for some weird semantics, but there's precedent for no-copy data structures elsewhere (e.g. sync.Mutex). It doesn't seem unreasonable to me to require the same of another type, even if it is just for performance.


Scraping runs in robot protection v0.38

https://www.bundesregierung.de/breg-de/themen/coronavirus
works fine in v0.35 and runs into a robot protection with v0.38
I am running changedetection.io with docker, so it was easy to fall back and check.
thank you for sharing changedetection as open source with all us geeks out there ❤

sorry, try the javascript/chrome branch https://github.com/dgtlmoon/changedetection.io/pull/114


breadcrumbs

SlamSearch definitely will cover this issue. So, it's here only to remember that breadcrumbs must be there and to make basic layout with couple of signal. (Probably from Router)


Add an environemt contributor that adds environment variables from the file

This PR changes the approach we use to inject MAVEN_OPTS environment variable.
Instead of using EnvironmentVariablesNodeProperty, which is attached to the node, additional environment variables (only MAVEN_OPTS ATM) are written to the file, next to the other files needed for the build scan injection to work. Then a custom EnvironmentContributor is used to read the file and inject variables into the job run.
Using this approach we do not leave any traces in the node config. Since the env variables decision is postponed to the moment when the job runs, in the future we might add other conditions to the decision if the injection needs to happen or not. For instance, we know that if Maven Integration is used, then we don't need to set MAVEN_OPTS, because this plugin doesn't parse it correctly and we need to use different injection mechanism.
Some other examples of different environment contributors can be found here

Acceptance tests passed in my local environment:


support extern alias and add unit test

This PR will fix project compilation error using extern alias.
related on https://github.com/OmniSharp/omnisharp-vscode/issues/2342
Aliases description can be referred in the following links.

https://docs.microsoft.com/en-us/visualstudio/msbuild/common-msbuild-project-items?view=vs-2017
https://docs.microsoft.com/en-us/visualstudio/msbuild/csc-task?view=vs-2017

CI test was failed, I will fix it

I'm using the latest version of VS Code and make the "omnisharp.path": "latest".
But still get The extern alias 'xxxx' was not specified in a /reference option (CS0430)

you can see the test project file here https://github.com/OmniSharp/omnisharp-roslyn/pull/1409/files#diff-2d3349e50030a4adcbf62d5602dcf874
is your setup similar? and if not, can you share a sample csproj file or a full repro?

There is my sample -> https://github.com/seanmars/omnisharp-roslyn-alias-test
It's fine to build, but in vs code always got error information like below image.
https://github.com/seanmars/omnisharp-roslyn-alias-test/blob/master/20190807_145935.png


jsPdf not working with android device browser app

i am building pwa using jsPdf to htmltopdf and download.But it works on machine.
when trying to android device browser it  fail to download on device.
how to fix it? can we save on google drive?

having the same issue here

not reproducable


Is it possible to delete useless range-diff?

Hi, sometime we may have a big change from vN to vN+1, the range-diff output
is useless, so is here anyway to suppress its output by some option?
@dscho

There is not yet any option, but it is easy to see how this could be valuable.
One strategy (easy to implement) would be a footer in the first comment similar to Cc:.
Another strategy (harder to implement) would be to compare the size of the range-diff to the size of the overall patches, and at a certain threshold replace the range-diff with a message like "The range-diff is too huge to be helpful."
All it needs is someone with enough time to implement either strategy (or a strategy I have not thought of, yet).

Looking at the second strategy.  Using this diff as an example.
/^\s+[+-][\s\n]/ is the count of changes.
`^\s+[+-]+[\s\n]/ is the count of range changes.
In this example, there are 42 changes and 60 range changes.  The range changes are 27 ++ and 33 -+.
Possible rules (VERY open to suggestions and numbers):

if the ++ count is  x% greater than the -+ count, a lot of changes were added so the range-diff could still be useful
if the -+ count is  x% greater than the ++ count, a lot of changes were removed so the range-diff may not be needed
if the ++ and -+ counts are similar and are x% of the total changes (approximately 40% in the example), then the range-diff may not be needed

Given the possibilities of getting it wrong, the footer comment would still need to be supported but as a boolean: version-diff: (yes|no).
How about this message?
No range-diff provided.  Please review the new patches as a replacement.
I guess that means both strategies. 🤔

Using this diff as an example.

That is actually a good example of a useful range-diff that is still larger than the patch itself (I think, at least) because it touches every line.
To me, this implies that there is no good "relative" measure.
Maybe a better measure would simply be the number of characters in the range-diff. Like, if the range-diff is even "only" 5kB, I would wager a bet that it is not very useful because it would be too hard to review.

No range-diff provided. Please review the new patches as a replacement.

That would make it sound to me as if the contributor did not contribute a range-diff. Which they didn't, but that's not their fault. I would prefer something along these lines:
Range-diff too large. Please review it locally by calling:

  git fetch https://github.com/gitgitgadget/git <previous-tip> <current-tip>
  git range-diff <options> <previous-range> <current-range>

where the tips and the ranges are provided as short hashes.

where the tips and the ranges are provided as short hashes

Using rev-parse or just an arbitrary length? Just wondering if rev-parse will have all the files/hashes needed if a shallow repo is being used.

That's a good point. Maybe hard-coding a length isn't such a bad idea here.


Extracts attester in the "target" directory by default

attester javascript files are now extracted in the local maven "target" directory by default instead of being extracted in the global temporary directory (java.io.tmpdir).
(in order to fix #8)

LGTM


The custom icon for a Script Class is only shown in the node creation dialog before restarting the editor

Godot version:
3.2 stable
OS/device including version:
Windows 10 x64 version 1909
Issue description:
The icon defined in a Script Class is displayed in the node creation dialog but not in the scene tree dock. Reopening the project solves the issue.

Node creation dialog
Scene tree dock

Also, several errors that seem to be related to image handling occur when using a Script Class, providing or not the optional icon. I guess those could be related, see #35895
Steps to reproduce:

create a new project.
create a GDScript script and open it.
register it as a type with the Godot icon: class_name MyClass, "res://icon.png"
save the script.
open the node creation dialog and search for the Script Class, it will be using the custom icon.
add the node to the tree. Now, instead of the custom icon, the Script Class appears with the icon from the class that it extends from.
save the scene. The editor will output several erros.
Reopen the project and open the scene created in the last step. The Script Class will now be using the custom icon in both the scene tree dock and node creation dialog.

I could replicate the issue in Ubuntu 19.10 too, using an svg icon.

I also encounter this issue, but if I close the project and reopen it then the icons show properly.
I have only tested it with svg icons and I am using linux as well

For the reference, this regressed in 3.2.2-stable, as now after adding a new script class icon, it's not visible in the node creation dialog at all:

And it's still the same as of 3.2.4 RC 1.
In 3.2 and 3.2.1, it behaves as described in the OP.
In either case, a restart of the editor solves it.

For the reference, this regressed in 3.2.2-stable, as now after adding a new script class icon, it's not visible in the node creation dialog at all:

And it's still the same as of 3.2.4 RC 1.
In 3.2 and 3.2.1, it behaves as described in the OP.
In either case, a restart of the editor solves it.

I also tried to set a custom icon from an atlast texture and it doesn't work. (Godot v3.5.rc7.official [e71cf7c56] on Windows 10)
class_name Bush,"res://bush_icon.tres"

I just wanted to note that I've also encounter this issue(with both .png and .svg) in Godot 4 beta 3 on Windows 10, but as stated above, a restart of the editor solves it.

It seems that the scene dock will only load each node icon one time (when the node is added).
After changing your script, you need to force the scene dock to reload (reopening the scene is enough).
https://www.youtube.com/watch?v=BWKHVkSrbQs
Godot version:
4.0.2 stable
OS/device including version:
Fedora 38