Fixes some failing tests on windows

I took a stab at addressing some of the failing tests under python3+windows.

corrects failing tests in rez/tests/test_packages.py on windows
adds %SYSTEMROOT% envvar to Python(ActionInterpreter) so rez/tests/test_context.py tests pass on windows

Haven't tried your PR yet, but I think version and changelog are handled by @nerdvegas

oh! Sorry, I was trying to follow the directions in CONTRIBUTING.md . I can undo the commit though if it is a problem.

I think you might be right. It's just that this was recently updated and I misunderstood the change:
https://github.com/nerdvegas/rez/blob/master/RELEASE.md

Do you think you change can in future be represented by the mechanism proposed here:
https://github.com/nerdvegas/rez/issues/737
Not saying that we don't need the workaround for the time being, but just checking if the proposal
might miss a feature.

Oh that looks very nice! No sense hiding conditionals all over the place :)
If I have understood your proposal correctly:

The first commit's fixes would be addressed by your "modify_op": PlatformDependent({"windows": "windowspath"}) to format the path consistently. It would just need to be used on both ends, or have a special object that can be used to test equality.

The second commit is kind of weird, and specific python-3 on windows. I'm curious if it is even needed in later versions of python. A cleaner workaround within your proposal might define a whitelist of variables to inherit from the parent process unless they are assigned by the user?

Sorry, I got excited and wanted that green tests-passing badge across the board :)

Me, too. Other then your fix I am working towards fixing the other Windows tests in #773

Just two general points:

Yes the contributing guidelines were changed yesterday, but I only
mentioned on slack, nw.
Generally if we need to put in a fix that we also know will change at
some point in future (eg as Blazej mentions about #737), I'm fine with
that, but please please put a big fat comment there highlighting the fact.

Cheers!
A
On Sat, Oct 26, 2019 at 6:03 AM Blazej Floch<EMAIL_ADDRESS>wrote:

Me, too. Other then your fix I am working towards fixing the other Windows
tests in #773 https://github.com/nerdvegas/rez/issues/773
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/pull/775?email_source=notifications&email_token=AAMOUSXGMH4HPR2CQ4IHVWDQQM7JTA5CNFSM4JFFBTQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECJIVMI#issuecomment-546474673,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMOUSVPP7AICZATAVSWZ73QQM7JTANCNFSM4JFFBTQQ
.

Thank you very much for the code review!
I'm updating the code now, and I'll post again once I hve it fixed.

Firstly, thanks very much for the code-review! I really appreciate the help.
I updated the code:

new rez.utils.filesystem.canonical_path()
new test file rez.tests.test_utils.test_filesystem to test canonical_path
rez.tests.test_context uses canonical_path
rezplugins.package_repository.filesystem.FileSystemPackageRepository uses canonical_path

Unrelated to previously mentioned:

rez.utils.filesystem.to_nativepath() was broken (os.path.join requires an expanded list). I also adjusted it to use replace('\\', '/') .

Misc Questions:

I noticed from old imports that you previously had a rez.vendor.mock but removed it. None of the other tests use mocks, so I'm guessing that you made an intentional decision to use TestCase.skipTest() instead. I followed this pattern, but just wanted to confirm this is how I should be proceeding.

Out of curiosity, what is the goal ofrez.utils.filesystem.{to_nativepath, to_ntpath, to_posixpath} versus {os.path, ntpath, posixpath}.normpath ? I feel like the latter might accomplish more, but I didn't want to break anything you already have in place.

Finally:
Thanks for clearing up about CONTRIBUTING.md - In the future, I'll leave the version and changes files alone. :)

Together with this PR I get green tests on Windows via docker for all three python version.

Nice, will get this merged soon.
I've asked ASWF about their container hosting, haven't heard back yet.
A
On Tue, Oct 29, 2019 at 8:57 AM Blazej Floch<EMAIL_ADDRESS>wrote:

Together with this PR I get green tests on Windows via docker for all
three python version.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/pull/775?email_source=notifications&email_token=AAMOUSR64OVJ35NA7BXVHPLQQ5N3BA5CNFSM4JFFBTQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECOQXEI#issuecomment-547163025,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMOUSUJQBLG6R5I2263AGDQQ5N3BANCNFSM4JFFBTQQ
.

@willjp For the test file you added, instead of rez/tests/test_utils/test_filesystem.py, we will need a rez/tests/test_utils_filesystem.py. This is so that the test can be shown when using rez-selftest.

sure, fixed!
Actually while on the subject of tests - do you want this testfile? None of the other utils are tested directly, I wasn't sure if this was something you guys would want, or if this was unecessary.

Tests are always welcome. We are lagging behind in terms of tests, so yes multiple parts of rez are not really tested. In general, I think we have about 43% of coverage. Thanks for the changes and adding the new tests :)

Out of curiosity, what is the goal ofrez.utils.filesystem.{to_nativepath, to_ntpath, to_posixpath} versus {os.path, ntpath, posixpath}.normpath ? I feel like the latter might accomplish more, but I didn't want to break anything you already have in place.

Good question. I don't remember, it's worht investigating. Could be that py2.6 didn't have that support? Not sure.
Made a ticket: https://github.com/nerdvegas/rez/issues/783

@nerdvegas @willjp Wondering if we could make the test_utils_filesystem.py -> test_utils.py for now.
I have literally a single test from data_utils (HashableDict) and I did not want to open a new test suite for it.
Since you also have a couple of tests I wonder if we could share that namespace and rather diverge in future when we have more tests.
Thoughts?

Yeah I don't think it's unreasonable to have a general utils test, so go
ahead. Rule here should just be that stuff tested in utils is low level and
standalone.
Cheers
A
On Wed, Nov 6, 2019 at 8:50 AM Blazej Floch<EMAIL_ADDRESS>wrote:

@nerdvegas https://github.com/nerdvegas @willjp
https://github.com/willjp Wondering if we could make the
test_utils_filesystem.py -> test_utils.py for now.
I have literally a single test from data_utils (HashableDict) and I did
not want to open a new test suite for it.
Since you also have a couple of tests I wonder if we could share that
namespace and rather diverge in future when we have more tests.
Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/pull/775?email_source=notifications&email_token=AAMOUSUQJCKWC2YCMXH5FK3QSHTCBA5CNFSM4JFFBTQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDEOREY#issuecomment-550037651,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMOUSSRF6BLOILTJSWCC23QSHTCBANCNFSM4JFFBTQQ
.

yeah absolutely, that sounds good to me. Give me a few minutes to finish up at work and I'll push an update.


Consuming Events

[Enter feedback here]
It would be good to explain if you use the RealtimeLocal or RealtimeRPC settings, how you then consume or view the events. e.g. how can I connect to the live session.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c35bddf8-d6a0-f462-2f5d-a86d9ff35116
Version Independent ID: 0431a2ca-fbea-37af-d8eb-7854d2c57e51
Content: Set-NetEventSession (NetEventPacketCapture)
Content Source: docset/windows/neteventpacketcapture/Set-NetEventSession.md
Product: w10
Technology: ****
GitHub Login: @Kateyanne
Microsoft Alias: v-kaunu

Hello, this kind of instruction would be out of the scope of the article.
For example, you can do following:

create net event session
add a network Event Packet Capture Provider
start the session
capture the events using any soft that supports ETW (like Microsoft Message Analyzer or Wireshark with ETW plugin)
stop the session

Thank you


k8s: Bring back TTY to Exec

This is a partial revert of commit 30310b18 ("connectivity, k8s, internal/utils: drop ExecInPodWithTTY and CtrlCReader").
Unfortunately, using the same context as StreamWithContext() to indicate a cancellation of a program running via Exec doesn't work as expected. From quick experimentation, it only closes a connection which doesn't necessarily send SIGTERM to a remote program. As a result, the tcpdump kept dangling in the encryption tests.
Bring back the ctrlreader, and use a separate context for it.
Signed-off-by: Martynas Pumputis<EMAIL_ADDRESS>

Successful run - https://github.com/cilium/cilium/actions/runs/3693474477/jobs/6253507992.


[Actual PR] Added favicon support.

As the title says.

@barryclark I noticed that this favicon field is gone now. How do I add a favicon to the Jekyll Now template in its current state?


Create symbols and characters subtopic

As per advice in https://docs.google.com/document/d/1mnkOQCR-FhziM3-bWFRYJogjjzUBVGgzCbZu9YBT4cA/edit#.
AusDTO/gov-au-content-guide#136

@joolswood I think this is right to be approved now Jools?

Closing this until it comes up in sprint.


Selecting an author on a blog post

At the moment, the author field on a blog post is a text field, when the page is rendered the name in this field is either matched with the name in the Authors node or a user in the backoffice.
In order to reduce typing errors and make this easier, an autocomplete style data type can be created.
The configuration of the data type will include

Relative path or content picker for the authors node
Option whether auto complete will populate from the authors node or the users or both
If the users is used then specify a user type to list the users from

The data type should also allow a name to be saved which is not in either of these lists.

I'm curious about why there are Author nodes if they are not directly associated with posts?

They are! There's actually author pages and author page listings now in v3.0


[octavia-ingress-controller] timeout too low when creating LBs

/kind bug
What happened:
When creating an ingress it fails as LB is in PENDING_CREATE status, this times out after I think 154 seconds based on
19 attempts increasing wait by factor of 1.2 starting at 1 second. In our cloud a load balancer can take around 5 minutes sometimes to provision.
What you expected to happen:
The timeout needs to allow for envs that take longer for LBs to provision
How to reproduce it:
Create an ingress in an openstack env when LBs are slower to create
Anything else we need to know?:
Environment:
Using octavia Zed

@sorrison do you want to submit a PR for this?

waitLoadbalancerActiveSteps = 19
waitLoadbalancerActiveSteps = 23

seems this value need code change, @dulek do you think we make it a config variable in configmap will help in the future ?
e.g different user might have different env so this value may vary?

The other problem in this code is that it's blocking^1.
This ties up an entire thread just waiting for a single LB to be created. Ideally we would not wait here and instead return from the reconciler and wait to be called again. This would allow LBs to be created in parallel. This would also mean there would be no good reason we couldn't wait forever, as long as we have exponential backoff between retries. We'd probably still want a configurable hard ceiling on that, though.


Gracefully fallback for bit libraries

How can I gracefully fallback with bit libraries and luarocks?
I wrote some code that works with lua 5.3 or luabitop.
But luabitop won't compile with lua 5.3.
So it looks like I can't support all lua versions with the one rockspec?
(output from https://travis-ci.org/daurnimator/lua-http/jobs/54412346#L523 )

Using https://rocks.moonscript.org/luabitop-1.0.2-1.src.rock... switching to 'build' mode
gcc -O2 -fPIC -I/usr/local/include -c bit.c -o bit.o
bit.c: In function ‘barg’:
bit.c:83:2: error: #error "Unknown number type, check LUA_NUMBER_* in luaconf.h"
Error: Failed installing dependency: https://rocks.moonscript.org/luabitop-1.0.2-1.src.rock - Build error: Failed compiling object bit.o

Use two different rockspecs (one for Lua 5.3 without luabitop in the dependencies, and one for Lua 5.1/5.2 with luabitop), or use bit32 instead of luabitop. bit32 is available for Lua 5.1, 5.2, and 5.3.

looks like I can't support all lua versions with the one rockspec?

Use two different rockspecs (one for Lua 5.3 without luabitop in the dependencies, and one for Lua 5.1/5.2 with luabitop)

So: yep :p

use bit32 instead of luabitop. bit32 is available for Lua 5.1, 5.2, and 5.3.

ah ha! this might save me for now.


Bug fixes

Closing this one in favor of https://github.com/vinsol-spree-contrib/spree_item_returns/pull/5.


Fix 10s minimum time for cache restores

We saw that every cache restore seems to take a minimum of 10s even for small 1MB caches from a local network using Minio or Seaweedfs.
The cause seems to be the listObjects function. It starts a timeout of 10s that is not cleared when the Promise is resolved or rejected. This causes the action to hang for a minimum of 10s even when the restore was faster.
Before fix:

After fix:

Changes

Clear the timeout in listObjects when the Promise is resolved or rejected.

Fix Issues

Fixes #43

The change looks good to me. But I wonder why it worked before....

Thanks for the merge and release @jackieli-tes
Do you use this action with ARC or something else?


RangeError when stringify a string with ending \0

Reproduction:
json5Stringify('abc\0'); // throws RangeError

Reason:
This project is ported from javascript code to dart.
dart is more strict than javascript in range checking.

Fixed #9

Fixed #9


more plot legend improvements #1617

option to layout horizontally instead of stacking vertically: legendLayout property (LegendLayout.HORIZONTAL/VERTICAL) was added to API
option to omit checkboxes: omitCheckboxes property (true/false) was added to API
cursor should be hand for dragging not I-bar for editing

please find usage examples in https://www.dropbox.com/s/vlzhwr2ifgcuodv/1617.bkr?dl=0

:+1:


Replace references to deprecated Boston housing prices In ReadMe

boston housing prices in ReadMe has replaced by California housing Price. But in readme document, with explaining tree model, the sample also uses boston housing price. I think it may be confused for some greeners when use it.

Closing this as duplicate of #2322


Prefetching error when finetuning

Hello, trying to run the finetuning code on an existing model, but I'm getting this error.  I am somewhat new to caffe so any help is appreciated!!
Also, is there a way to run the finetuning without a GPU?
[E prefetch_op.h:110] Prefetching error std::bad_alloc
[E prefetch_op.h:83] Prefetching failed.
[E net_async_base.cc:377] Failed to execute an op,  op VideoInput
[E net_async_base.cc:135] Error encountered in the run of 'r2plus1d_train'
WARNING:caffe2.python.workspace:Original python traceback for operator 0 in network r2plus1d_train in exception above (most recent call last):
WARNING:caffe2.python.workspace:  File "tools/train_net.py", line 501, in
WARNING:caffe2.python.workspace:  File "tools/train_net.py", line 496, in main
WARNING:caffe2.python.workspace:  File "tools/train_net.py", line 280, in Train
WARNING:caffe2.python.workspace:  File "/home/surajm72/pytorch/build/caffe2/python/data_parallel_model.py", line 34, in Parallelize_GPU
WARNING:caffe2.python.workspace:  File "/home/surajm72/pytorch/build/caffe2/python/data_parallel_model.py", line 231, in Parallelize
WARNING:caffe2.python.workspace:  File "tools/train_net.py", line 268, in add_video_input
WARNING:caffe2.python.workspace:  File "/home/surajm72/VMZ/lib/utils/model_helper.py", line 131, in AddVideoInput
Traceback (most recent call last):
File "tools/train_net.py", line 501, in
main()
File "tools/train_net.py", line 496, in main
Train(args)
File "tools/train_net.py", line 388, in Train
explog
File "tools/train_net.py", line 123, in RunEpoch
workspace.RunNet(train_model.net.Proto().name)
File "/home/surajm72/pytorch/build/caffe2/python/workspace.py", line 250, in RunNet
StringifyNetName(name), num_iter, allow_fail,
File "/home/surajm72/pytorch/build/caffe2/python/workspace.py", line 211, in CallWithExceptionIntercept
return func(args, kwargs)
RuntimeError: [enforce fail at pybind_state.cc:1188] success. Error running net r2plus1d_train
frame #0: c10::ThrowEnforceNotMet(char const, int, char const, std::__cxx11::basic_string<char, std::char_traits, std::allocator > const&, void const) + 0x76 (0x7ff7a7a78316 in /home/surajm72/pytorch/build/lib/libc10.so)
frame #1:  + 0x46046 (0x7ff7a816b046 in /home/surajm72/pytorch/build/caffe2/python/caffe2_pybind11_state_gpu.so)
frame #2:  + 0x91360 (0x7ff7a81b6360 in /home/surajm72/pytorch/build/caffe2/python/caffe2_pybind11_state_gpu.so)

frame #15: python() [0x4eb69f]
frame #19: __libc_start_main + 0xf0 (0x7ff7aba5c830 in /lib/x86_64-linux-gnu/libc.so.6)

should be fixed by https://github.com/facebookresearch/VMZ/issues/69


reduce blank space to footer is visible

when logged in display footer

I think it's related to SimpleDApp and it was fixed
in Rair tech I didn't see this issue
@RAIReth @sarora180673


CLOUD-445 modification in request numbers

@keyki or @martonsereg pls check this fix

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/752/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/762/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/763/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/764/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/766/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/768/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/770/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/773/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/774/


Svelte: auto-rename doesn't work

Hey, thank's for your plugins, lovely <3
I just found that auto rename doesn't work on .svelte file.
It works as expected on html files though, so I guess my config is not the cause.

fix #17


kernelCTF: Add CVE-2024-41010 LTS

Your understanding is generally correct except for:

This is done by adding another msg_msg to the queue with the overwritten msgmsg_seg?

Here is very detailed explanation that hopefully will give you a very deep understanding of what is going on:
For context, msg_msg and msg_msgseg are allocated when sending ipc messages and are deallocated by receiving ipc messages. Also, skbuf->data is allocated by sending data on one end of a unix socket pair and is deallocated by receiving the data on the other end of the socket pair.
Getting to kmalloc-cg-512 in the first place
Basically, at that stage I receive from the mqueue that contains the overwritten msg_msgseg (let's call it mqueue A) only to free the skbuf->data object (residing in kmalloc-cg-512) whose pointer I discovered in the previous step (using if_inaddr). After that, you can forget about the previous steps and the corrupted msg_msgseg, because what follows is bug agnostic:
The heap chunk containing that skbuf->data will be put on top of the cg-512 freelist because I freed it by receiving the corrupted msg_msgseg (from mqueue A), and I can now reclaim that chunk by allocating msg_msg using a different mqueue (let's call it mqueue B). At this point we will have a skbuf->data object and msg_msg object overlapping with each other on kmalloc-cg-512. If we now receive the skbuf->data. Keep in mind that by receiving that skbuf->data from the socket I will be:

reading the msg_msg struct and leaking it's contents
deallocating the chunk and putting it back on top of the freelist
After that, if I send data on the socket again I will reclaim the same chunk again and overlap the msg_msg struct, so I can effectively view and edit the msg_msg object as I like.

Getting from kmalloc-512 to kmalloc-cg-1k
Now that we have a fully controllable msg_msg object (which belongs to mqueue B) in kmalloc-cg-512, in order to go from that to kmalloc-cg-1k I do the following things:

send another msg on mqueue B with a size such that it gets allocated in kmalloc-cg-1k, (this will populate the controllable msg_msg with a pointer) to this second msg, which lives in kmalloc-cg-1k.
spray some messages on different mqueues (let's all them mqueue N) such that their msg_msg objects also get allocated in kmalloc-cg1k. This allows us to get a layout such that (in kmalloc-cg-1k) there will be the 2nd msg_msg of mqueue B followed by the 1st msg_msg of some mqueue N.
I read the unix socket to read the 1st msg of mqueue B, which contains the pointer of the 2nd msg of mqueue B. I know for a fact that the next object following that pointer is a msg_msg belonging to a different mqueue N, so I add sizeof(kmalloc-cg-1k object) to the pointer that I just leaked, which should let me know the address of some msg_msg belonging to some mqueue N.
I edit the 1st msg_msg of mqueue B by writing to the unix socket, such that it now no longer points to the 2nd msg of mqueue B and now points to the 1st msg_msg of some mqueue N. The keypoint that you should have in mind here is that some msg_msg that belongs to mqueue N is now not only linked in the list of mqueue N but ALSO in the list of mqueue B and it's accessible from both queues (let's called that msg_msg X).
I free msg_msg X by receiving everything from mqueue B, but msg_msg X is still reacheable from mqueue N, so mqueue N references a heap chunk that is in the freelist of cg-1k.
I send some skbuf->data with a size such that it gets allocated on kmalloc-cg-1k, reclaiming the slot of msg_msg X from the freelist. So now mqueue N has a reference to that skbuf->data.
I free the same chunk again by receiveing from mqueue N, which will free the skbuf->data while it is still reacheable from the unix socket.

Going from that to root
At this point I have a similar scenario as I had on kmalloc-cg-512, which is a dangling skbuf->data object that can be overlapped with any struct I want and I will be able to edit that struct. I overlap it with pipe_buffer to control the pipe_buffer->page pointer and get arb r/w on kernel memory.

thanks

Hey 0xTen!
Thank you for your submission!
It looks well documented and easy to read.
One question: why did you choose to scan the kernel memory to leak kASLR & find the modprobe address instead of using a ROP chain with target-specific symbol addresses? Was this scanning technique stable as well?


Repository move announcement

On Monday, October 4th, the imodel-unity-example repo will be transferred to the iTwin Platform organization on Github.
All branches, pull requests, issues, and discussions will be transferred along with the repository, and all links to the old repository will redirect to the new repository which will be found at https://github.com/iTwin/imodel-unity-example.
Once the move is complete, we recommend that you update the git origin in your local copy of the repo to the new repository location using:
git remote set-url origin https://github.com/iTwin/imodel-unity-example.git

The imodel-unity-example repo has been successfully transferred to the iTwin Platform organization. Please use this thread if you experience any issues related to the transfer.

Closing since no issues have been reported in the previous 6 months.


Add more links to API documentation

Closes #173
I tried to make a line on each end of section to the API docs and also converted some \type to \href in the text.
Also removed line of text on first section, otherwise NoContent state does not seem to work. Now it nicely makes a list of everything.

The original idea of \type was to automatically insert API links so this wouldn't be necessary. The problem is that we also use it for types from the examples which obviously don't have an API page.


Erasing

There's this weird bug that when you try to erase the list it doesn't work on the first time

Fixed in 2.0.1


[DOC] add Japanese README

I created Japanese translated README.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@eltociear, thanks for your contribution.


Delta Diff - don't show the " You can use lakeFS to compare Delta Lake tables" note so much

We should be more frugal with when we display the You can use lakeFS to compare Delta Lake tables. Learn more. note.

It's going to get annoying for users in several cases:

They're not even using Delta
They are using Delta and they know already that they can diff

Ideas:

Add a X so that it can be dismissed
Once the user has used the functionality once, don't show the note again
(maybe not worth the LOE) parse the data and only show the note if it's Delta data and the user hasn't yet used the feature

The note also becomes redundant once the UI is already telling the user that a diff is available:

The same UI would be as effective:

@itaiad200 @ozkatz Coming back to this - I'm working with Iceberg data, and this note is really annoying in the UI.
How about my idea of add a close icon to it so that it can be dismissed (and not shown again)?
thanks

@Jonathan-Rosenberg what's the effort of this? Some kind of cookie to know that the user has disabled it?

Wonderful, thanks @nopcoder !


Cannot read property 'lat' of null

While using the custom options:
            new L.Control.GPlaceAutocomplete({                 position: "topleft",                 callback: function(location){                     map.panTo(location);                 }                 }).addTo(map);
I got an error:

Uncaught TypeError: Cannot read property 'lat' of null

However, it works great with no custom options:
new L.Control.GPlaceAutocomplete().addTo(map);
I want to use custom options for zoomTo(location), instead of panTo(location)
What could be the problem?
Thank you in advance

Here's how I fixed it on my local project:
  new L.Control.GPlaceAutocomplete({     position: "topright",     callback: function(location){         map.panTo([           location.geometry.location.lat(),           location.geometry.location.lng()           ]);         map.fitBounds([                [location.geometry.viewport.getSouthWest().lat(),                 location.geometry.viewport.getSouthWest().lng()],                [location.geometry.viewport.getNorthEast().lat(),                 location.geometry.viewport.getNorthEast().lng()]               ]);     }}).addTo(map);

@khelal thanks a lot. It works great


Signin with GitHub button is not showing

In VS code when I click on settings sync to login with my GitHub account, it doesn't showing sign in with GitHub button. Meanwhile it directly redirects to the Microsoft account but I want to signin with GitHub account. To enable the signin with GitHub button what have to do?

Mee too. Server Ubuntu, client Macos.
$ code-server --version
4.17.1 2eba7af117ea58d45a6c6449ee4fe63c8d4d53aa with Code 1.82.2

Hello,
Same problem


Updated grammar and term consistency

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@Picklebunker
Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍 We're happy to accept these changes and look forward to future contributions.


Feature-Request: Downloading the QR code via SSH

Hi,
Pretty straight forward question here. Trying to setup my node so I can access it remotely. All works well but when I generate a qr code using any app selection it just won't shrink when I push control and -. Any ideas are appreciated
Thanks

what do you mean by "using any app selection"?

What I meant is I have tried choosing Shango wallet and the zues wallet and couldn't shrink any of them. I'm running straight from the raspberry Pi itself, did not ssh in, couldn't seem to get that going. I'm not an expert on networking.

I'm running straight from the raspberry Pi itself

You mean you connected a keyboard to your Pi and used Alt-F2 to access the command line and Raspiblitz menu?

Seems it's related to the terminal/shell being used.  When I SSH into my node ctrl plus/minus works for zooming in and out.

From this issue: https://github.com/Stadicus/RaspiBolt/issues/354#issuecomment-459031989

I'm running a light desktop (raspbian stretch lite + pixel) and was able to install gnome-terminal in order to zoom out (raspbian native terminal does not zoom). I suppose this would work ssh'ing in via another linux machine too. However, I was never able to get a QR of proper, scanable size using windows, PuTTy. So I guess the work around is to use a linux computer to ssh, zoom out via that linux terminal, and scan the QR, but not sure why it was so huge in the first place or how to zoom out in windows.

Hi, thanks for the reply. Yes I am just plugging a keyboard into the pi itself. I was trying to use putty on windows to login but just couldn't connect. I'm more a beginner in the network side of things, I used advanced IP scanner to see my raspberry ip but it starts with a 10.0.... so I think that's not the correct ip. I will have to get ssh going to make this work.
Thanks again

Hi, thanks for the reply. Yes I am just plugging a keyboard into the pi itself. I was trying to use putty on windows to login but just couldn't connect. I'm more a beginner in the network side of things, I used advanced IP scanner to see my raspberry ip but it starts with a 10.0.... so I think that's not the correct ip. I will have to get ssh going to make this work.
Thanks again

10.0.... is fine. Routers usually hand out IP addresses like that, or starting with 192.168....
If you can use the connected keyboard and login to get a command line prompt, try running ifconfig to see what the full ip-address is.  Then from your Windows machine (with Putty installed) try to ping it...
ping 10.0.xx.xx
if you get ping replies then you should be able to connect via SSH like this...
ssh 10.0.xx.xx or whatever the equivalent Putty does to connect (I don't use Windows).
Quick google of Putty gave me this link: https://www.ssh.com/ssh/putty/windows/install#sec-Configuring-and-installing

v1.2 is displaying the QR code on the LCD (and just by option as fallback as ASCII-code in the terminal). So on a normal RaspiBlitz setup with LCD the need to shrink the QR code should not be a thing anymore and it recommended - it also helps on finding the IP of the RaspiBlitz.
Recommending this issue to close.

I also can’t zoom to windows putty! How can I download QR cod via ssh?

downloading the QR code via SSH is an interesting idea/work-around ... will add this to a future release - for the "I dont have a LCD" case. Everybody else is advised to scan the QR code from LCD which is already the default for v1.2

I don't have the small 20 dollar lcd but using a monitor connected via HDMI. Even in the new version I don't see the qr code come up. It just hangs there, then I push enter and it then loads the console qr instead. I can't seem to ssh into the machine, there's obviously some sort of network issue on my end that I need to figure out. I did however notice when choosing the zues wallet, it said downloaded qr.png. I thought maybe it was now stored on the SD card but when exploring it I couldn't find such a file.

Run the same lndconnect command in the terminal with the option --image added to generate a .png picture file which then can be downloaded with scp / WinSCP and opened on any desktop.

@dsal1985 when using HDMI and no LCD-Hat - you need to config the RaspiBlitz manually to switch HDMI back on - see FAQ: https://github.com/rootzoll/raspiblitz/blob/master/FAQ.md#can-i-run-the-raspiblitz-without-a-displaylcd

It looks like if qrencode is used with "-t ansiutf8"  it will scale better in Putty (since Putty doesn't seem to natively support using ctrl+- to resize text). Example I used for my test:
qrencode -t ansiutf8 < "filename"
Where filename was the text from lndconnect -j.

@dickieadams Thank you, much better now. But it almost fitted using putty full screen, very close.
Do you know how to make it a little bit smaller?

@dickieadams Thank you, much better now. But it almost fitted using putty full screen, very close.
Do you know how to make it a little bit smaller?

Check your putty display scaling. It should fit regardless if the window size.

Thanks @dickieadams ... I will check if I can add the -t ansiutf8 parameters and the lndconnect -j text above the QR code for v1.7 release.

With v1.7RC1 the lndconnect go tool got removed due to build problems and no further maintance. It got replaced with the shell script is now generating the connection infos itself --> #2099
Also with v1.8 it will be targeted to have a webUI that makes pairing even easier.
So I am closing this issue. The optimization on QR code displaying in terminal and on LCD can be further commented on #2099


about the installation of vs code

I have installed them, just wondering

Does they have compiler in there like Open Cobol IDE?
Who can show me a video, how to use vs code to edit and debug Cobol?

Appreciated.
Tom

Who has run the "Hello world" successfully?

I didn't. As I wrote earlier, I am stuck on chapter 4.6, same as many of us beginners.

Hello negovan13
What is your issue?
I have run Hello pgm successfully.
I can help you
Eddie


docs: Mention the docker image

I shamelessly went through all the of pagination of the docker images of banzaicloud, instead of just trying the name directly. Might as well mention there is a docker image, and it's URL.

It helped me, thanks. @leprechaun

thanks @leprechaun


Research what can be stored in the manifest and try and store the metadata there

filename
mime type
size

Anything you want can be stored in the manifest nodes as metadata.   I'm storing a "Last-Modified" timestamp in my mantaray-js-constructed manifest nodes.


This addon no longer works with ember-cli 2.13.0

I know this project is deprecated but it no longer works with ember cli >= 2.13.0 due to a dependency issue, once the downstream ember-binding-macros addon fix is merged (https://github.com/cowboyd/ember-binding-macros/pull/1) and released, update this addon to use new ember-binding-macros version

If you're depending on this project, we'd be happy to pass it over to you so that you don't need us to help you maintain it.

I also would like to say emberx-range-input is pretty similar to this addon & supports modern ember https://github.com/thefrontside/emberx-range-input


elixir-ls installer using a not-yet existent function when used with an older elixir version

Environment

Elixir & Erlang versions (elixir --version):
Erlang/OTP 24 [erts-<IP_ADDRESS>] [source] [64-bit] [smp:10:10] [ds:10:10:10] [async-threads:1]

Elixir 1.13.4 (compiled with Erlang/OTP 24) - installed with ASDF

Elixir Language Server version:
v0.24.1
Operating system:
Sonoma 14.6.1
Editor or IDE name (e.g. Emacs/VSCode):
nvim
Editor Plugin/LSP Client name and version:
elixirls installed with Mason

Current behavior
It looks like elixir-ls installer is trying to use functions from elixir from the version not installed on my machine. Jumping to definitions doesn't work and the lsp logs are as follows:
[ERROR][2024-12-13 10:55:16] .../vim/lsp/rpc.lua:734	"rpc"	"elixir-ls"	"stderr"	"Running /Users/dominikstanaszek/.local/share/lvim/mason/packages/elixir-ls/launch.sh\n"
[ERROR][2024-12-13 10:55:16] .../vim/lsp/rpc.lua:734	"rpc"	"elixir-ls"	"stderr"	"Preferred shell is bash, relaunching\n"
[ERROR][2024-12-13 10:55:16] .../vim/lsp/rpc.lua:734	"rpc"	"elixir-ls"	"stderr"	"Looking for ASDF install\n"
[ERROR][2024-12-13 10:55:16] .../vim/lsp/rpc.lua:734	"rpc"	"elixir-ls"	"stderr"	"ASDF install found in /opt/homebrew/opt/asdf/libexec/asdf.sh, sourcing\n"
[ERROR][2024-12-13 10:55:16] .../vim/lsp/rpc.lua:734	"rpc"	"elixir-ls"	"stderr"	"Installing ElixirLS release v0.24.1\n"
[ERROR][2024-12-13 10:55:16] .../vim/lsp/rpc.lua:734	"rpc"	"elixir-ls"	"stderr"	"Running in /Users/<<HOME>>/<<PROJECT PATH>>\n"
[ERROR][2024-12-13 10:55:16] .../vim/lsp/rpc.lua:734	"rpc"	"elixir-ls"	"stderr"	'Could not find Hex, which is needed to build dependency true\nShall I install Hex? (if running non-interactively, use "mix local.hex --force")\n'
[ERROR][2024-12-13 10:55:18] .../vim/lsp/rpc.lua:734	"rpc"	"elixir-ls"	"stderr"	"* creating /Users/<<HOME>>/.asdf/installs/elixir/1.13.4-otp-24/.mix/archives/hex-2.1.1\n"
[ERROR][2024-12-13 10:55:18] .../vim/lsp/rpc.lua:734	"rpc"	"elixir-ls"	"stderr"	"Install complete\n"
[ERROR][2024-12-13 10:55:18] .../vim/lsp/rpc.lua:734	"rpc"	"elixir-ls"	"stderr"	"warning: Mix.Dep.Lock.read/1 is undefined or private. Did you mean:\n\n      * read/0\n\nInvalid call found at 2 locations:\n  /Users/<<HOME>>/.local/share/lvim/mason/packages/elixir-ls/installer.exs:356: ElixirLS.Mix.install/2\n  /Users/<<HOME>>/.local/share/lvim/mason/packages/elixir-ls/installer.exs:357: ElixirLS.Mix.install/2\n\n"

Mix.Dep.Lock.read/1 has been added to elixir after the version I am using, I might be missing something but shouldn't the installer work in a backwards compatible manner? Are there different installer versions and in my case the wrong one has been chosen?
Expected behavior
The installation should be successful

You right, Mix.Dep.Lock.read/1 has been added in 1.14. It's strange that it was not noticed until now. The installer has been used for almost two years. Would you like to raise a PR? Elixir 1.13 is near EoL and support such an old version is not my top priority.

Note the code is only executed if there is something wrong with your mix install dir and lockfile MD5 diverges. https://github.com/elixir-lang/elixir/blob/7df646d960c3cc9a4590861a87245d9c44f7fbc4/lib/mix/lib/mix.ex#L768
Maybe you need to cleanup your mix install dir


Bandit returns :closed error as a string for chunk

I got a CSV export controller that in production occassionally gives a match error, because I'm expecting {:error, :closed} for Plug.Conn.chunk/2 as the example from the docs shows:
Enum.reduce_while(~w(each chunk as a word), conn, fn (chunk, conn) ->
  case Plug.Conn.chunk(conn, chunk) do
    {:ok, conn} ->
      {:cont, conn}
    {:error, :closed} ->
      {:halt, conn}
  end
end)

However, Bandit returns a string ({:error, "closed"}) on this line:
https://github.com/mtrudel/bandit/blob/84e0d5d85c122724056607e60fc37164ba9e4347/lib/bandit/adapter.ex#L204
Which ultimately is from this line for the HTTP/1 socket: https://github.com/mtrudel/bandit/blob/84e0d5d85c122724056607e60fc37164ba9e4347/lib/bandit/http1/socket.ex#L428-L430
When I tested with HTTP/2 it seems like the process just continues to completion even though I closed the connection client side? If I let it wait for a long time it just silently stops.
So I've got two questions:

Shouldn't the HTTP/1 adapter return the :closed atom? I think we've looked at this before with no specified return types, but the example does show that Plug.Conn.chunk/2 returns {:error, :closed} (so at least the example needs to be updated there to show both cowboy and bandit).

Is it correct behavior to let HTTP/2 continue to process the request even though the client cancelled the request? Maybe I was testing this wrong or I don't understand how the HTTP/2 stream is supposed to work, but I couldn't get it to stop processing the request though I cancelled the download and closed the connection in the browser.

Shouldn't the HTTP/1 adapter return the :closed atom? I think we've looked at this before with no specified return types, but the example does show that Plug.Conn.chunk/2 returns {:error, :closed} (so at least the example needs to be updated there to show both cowboy and bandit).

Yep, that was a shortcoming of how we pushed all errors through Bandit.HTTPError, which (like all exceptions) requires a binary message. I added Bandit.SocketError in #433 to better capture the semantic of socket level errors (since they're unrecoverable as well, so shouldn't be signalled back to the client).

Is it correct behavior to let HTTP/2 continue to process the request even though the client cancelled the request? Maybe I was testing this wrong or I don't understand how the HTTP/2 stream is supposed to work, but I couldn't get it to stop processing the request though I cancelled the download and closed the connection in the browser.

HTTP/2 has a much different process model; the browser will typically keep the underlying TCP session (the 'connection' in HTTP/2 lingo) open, and only close the specific stream. We should be doing this here, which should bubble up to the stream process terminating. Do you have a repro case I can look at?

Yup, here's a demo: https://github.com/danschultzer/bandit_chunk_demo
I've also set up https in that repo on https://localhost:4001 so you can see how HTTP/2 function with chunked response and cancelled request.

Yep, that was a bug introduced in the big HTTP/2 refactor earlier this year, where we weren't checking for any RST_STREAM messages from the client before writing responses. Fixed on this branch, along with more consistent :closed error codes for HTTP/1.
@danschultzer would you be able to see if this branch addresses your issues in the manner you'd hoped?

Yup, it works for both HTTP/2 and HTTP/1.1, thanks!

Great to hear! I'll get this out later this week

Fixed in 1.6.1, just released


Longer project title gets cut off

Ah, I wasn't envisioning this as a generated field, more as a self contained logo that could be manually replaced. If it is dynamic we can use a different approach for the title rather than including it in the SVG.

I see. Yep, https://github.com/cloudfour/drizzle/issues/44 introduces a generated title field that will live in the project.yaml. :)

As mentioned in my conversation earlier with @erikjung, I don't think this area should be dynamic. Originally it was text-based, but I think that's a bad idea now that there's a logo there. And it would be weird to have the Drizzle icon next to some other type.
I do think the project name should populate the <title> element in the head, though.

@tylersticka Ah, right! I forgot about that. Sorry! I'll create a new PR that removes this for the logo. 😉

No longer an issue, closing! 😁


MR github issue burndown #4

Docs Build status updates of commit 75c046c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/hologram-stability.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit f296307:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/hologram-stability.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@hferrone - Is this one ready to go?

@jessemcculloch Yes sir!


Unnecessary dependencies

In setup.cfg the following dependencies are listed
ipython>=6
nbformat>=4
nbconvert>=5
requests>=2

Isn't that a mistake?
I also see Js2Py being used in scxml.py but only listed in requirements_dev.txt. That seems wrong as well.
Let me know if any of this is intended and then I'll fix in a PR.

The setup.cfg dependencies look wrong to me, too. And yes, Js2Py should be included.


Lz

This change is

Reviewed 3 of 4 files at r1.
Review status: 3 of 4 files reviewed at latest revision, 1 unresolved discussion.

lz/ch00/ut.cpp, line 7 at r1 (raw file):

{
public:
    virtual ~T(){}

consider using
virtual ~T() = default;

Comments from Reviewable

Reviewed 1 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


chore: rename innerJoin to intersectionWith

innerJoin is a sql command, R.innerJoin only has inner but no join, so rename it to intersectionWith may be more accurate?

@customcommander , Yes, thanks for remind.

@CrossEye Any objection?

I agree with the change.
I guess ,,we need to decide whether 1.0 or 0.29 is next.  When we rename a function, we usually keep the old name around as an alias for a minor version.  If we're building 0.29 I would do so, but I think if we're headed right for 1.0, then we need to skip it this time.

@CrossEye: I wouldn't bother with 0.29 at this point and would rather keep the momentum for 1.0.0 and ship as many breaking changes in that one.


Only Countries does not work

Hi,
<vue-tel-input
                                                :onlyCountries="['AU', 'BR']"
                                                wrapperClasses="h-12 border-b-1 border-red-500 w-96"
                                                mode="international"
                                                autocomplete="username"
                                                inputClasses="w-full"
                                                name="tel"
                                                :value="{{json_encode($user->tel)}}"
                                            />

It shows all countries, regardless.
This is in laravel blade view.
Tried to set default settings in app.js like so:

import VueTelInput from 'vue-tel-input'

Vue.use(VueTelInput, [{
    onlyCountries: ['AU', 'BR']
}])

It does not work too, it shows all countries.

I have this issue too

Should be fixed in v4.4.0


Attachments in Hour Registration is not visible in Duett Økonomi

This will be fixed in a couple of weeks. The well-being exists and will appear when the defect is corrected

Case closed


Can have intermediary keyboard backlight between none and first level?

For seeing keys when there is no light but not have too strong light to not disturb someone sleeping for instance.
Same for #1 maybe.
Related to Benjamin-Loison/cinnamon/issues/67.
+33

Related to Benjamin_Loison/gnome-control-center/issues/4.

ectool help

...
  pwmgetkblight
      Prints current keyboard backlight percent
...
  pwmsetkblight <percent>
      Set keyboard backlight in percent
...

Levels seem to be: 0, 20, 50, 100.

Could set a shortcut as a better workaround.

I set ctrl + alt + b.

Related to Benjamin_Loison/Debian/issues/94.


Forum Creation

I am very interested in this topic and I have been exploring the library in the past months in my free time. I find it hard to find people knowledgeable in the topic. Maybe a forum for questions might be created.
I am asking anyway my question here in the meantime:
As far as I understand all particle filters rely on the assumption that we have a model (even vague) of the underlying process. Has it ever been tried to learn the model directly from data and then apply the particle filters? I have a very complicated process with many observations and I think particle filters are the answer. But creating a model is basically imposible.

hi,
there is a "discussions" thing on github, I could activate it if you want, that seems like a better place than "issues" (which is for tracking bugs).
Regarding your question : as far as I know, there have been few attempts in this direction; one way to go would be to design a partly (or totally) nonparametric state-space models based on DPMs (Dirichlet Process mixtures); see this for instance :
https://arxiv.org/abs/1810.09291
or this (sorry, this is not on arxiv):
https://www.sciencedirect.com/science/article/pii/S0165168416000633?casa_token=xjtRJnI6BC0AAAAA:n4GqkbvamIrV853NBxA1SLYjOCCJ3sJ6_R0fTzZxWgxt7DK47wHrCOQEqQcamZGpmSC7W_wP4A
I am a bit sceptical about such approaches, because the estimation of parametric state-space models is already quite challenging; i.e. even with lot of data, some of your parameters may be hardly identified. So trying to use non-parametric model looks too ambitious. But there might be practical scenarios where it actually makes sense.

Closing this since the OP did not reply, but happy to activate the discussions tab if anyone thinks it could be helpful.

learn the model directly from data and then apply the particle filters?

I am also interested in this problem.
This paper might be interesting to you: https://arxiv.org/abs/2307.09607. They treat it as model selection by particle filtering over a large model space.


Correct Data for California

Hi -
I am a Property Tax Agent and depend on exact locations of property addresses. I have a number of checks to verify an address is reasonable.  I use Mapquest and Google to check addresses and resolve any ambiguities.
In any event, I started with open address as my base and have corrected any errors in the data while adding any missing parcels.  I would like to give back and send my data set with APN, Street, Zipcode, Latitude, and Longitude.  By the way, I found my data city names do not correlate with the zipcode for the different counties we service.
What should I do to send the corrected data?
Mike

Hi Mike! Thanks for contacting us.
I'm curious to hear about what sort of discrepancies you're seeing. If you want to post your data somewhere (like Dropbox maybe?) and include a link here, that'd be really interesting.
Having said that, we don't actually apply any changes to the data we download from sources. If there's something wrong with our data, the best we can do right now is find the contact info for the source that has the problems and pass them along, hoping that they will make changes on their end.
Also, keep in mind that the data we distribute today is address points, not parcels. We're working on processing parcels (and some of our sources are indeed parcel centroids), but some of the discrepancies might come from that distinction.

Hi Ian,
I did some sanity checking on the data for each county.  As I have over
4 Million records, I do not know which ones have been corrected.
Essentially, I have the most northern, southern, eastern, and western
point for each county.  I compare these coordinates with the Geo data
provided.  If a property is outside these coordinates, then the data is
corrupt.
By the way, before I loaded the geo data, I cleaned up all the duplicate
entries (and there are quite a few).
The other reasonable check is looking for two residential homes (not
condos) that have the same geo location.  Obviously two homes (different
street addresses) cannot have the same geo point.
As I have more records for counties than the open source data, I had to
find the geocode for those homes not listed.
Here is a list of the 12 counties I am processing.  I have not finished
all the counties and chipping away at them one by one.
Alameda
Contra Costa
Los Angeles
Orange
Placer
Riverside
Sacramento
San Bernardino
San Diego
San Joaquin
San Mateo
Santa Clara
On 6/22/2016 5:36 AM, Ian Dees wrote:

Hi Mike! Thanks for contacting us.
I'm curious to hear about what sort of discrepancies you're seeing. If
you want to post your data somewhere (like Dropbox maybe?) and include
a link here, that'd be really interesting.
Having said that, we don't actually apply any changes to the data we
download from sources. If there's something wrong with our data, the
best we can do right now is find the contact info for the source that
has the problems and pass them along, hoping that they will make
changes on their end.
Also, keep in mind that the data we distribute today is address
points, not parcels. We're working on processing parcels (and some of
our sources are indeed parcel centroids), but some of the
discrepancies might come from that distinction.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openaddresses/openaddresses/issues/1755#issuecomment-227729533,
or mute the thread
https://github.com/notifications/unsubscribe/ASp8tzhzheeK6lByJKgHUKfpy2utw3N4ks5qOSxKgaJpZM4I7bZK.

@xtgMike, that sounds super interesting. We were thinking about a similar thing a while back, when we noticed a few flipped lat/lon sources: https://github.com/openaddresses/machine/issues/318. I’m happy that you went ahead and did this!
Do you happen to be able to map your changes back to original rows?

Unfortunately no.  I made the changes directly into our database and
have geocoded data from a few places to get all the data for a county.
I can give you the points for the counties I have done so far.
M
On 6/23/2016 4:52 PM, migurski wrote:

@xtgMike https://github.com/xtgMike, that sounds super interesting.
We were thinking about a similar thing a while back, when we noticed a
few flipped lat/lon sources: openaddresses/machine#318
https://github.com/openaddresses/machine/issues/318. I’m happy that
you went ahead and did this!
Do you happen to be able to map your changes back to original rows?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openaddresses/openaddresses/issues/1755#issuecomment-228219471,
or mute the thread
https://github.com/notifications/unsubscribe/ASp8t-VF4pQ9F_9MHjBVhSBs6HqUxXVGks5qOxxTgaJpZM4I7bZK.

I bet we could turn that into some kind of diff as an experiment, especially if we know the timestamp of the files you downloaded.

The data came from the county file - although a few counties needed to
add a city file as well.  Let me know what you want to do.
M
On 6/23/2016 5:00 PM, migurski wrote:

I bet we could turn that into some kind of diff as an experiment,
especially if we know the timestamp of the files you downloaded.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openaddresses/openaddresses/issues/1755#issuecomment-228220698,
or mute the thread
https://github.com/notifications/unsubscribe/ASp8t3JcLJfJRi4xASs_QztcZcpwxgQ_ks5qOx40gaJpZM4I7bZK.


Include a license

By not including a license with your code (usually in the form of a LICENSE file), people are legally not allowed to fork, contribute or otherwise use the code you've published in their own projects, kinda defeating the point of making it open source.

Cool, I added WTFPL license :)

Cool, I added WTFPL license :)


Chart/graph control

Ability to embed a live chart/graph from PowerBI into an Adaptive Card. The chart/graph live updates with fresh data, clicking it takes the user to the PowerBI portal to view more of the graph.

Link to feature: https://adaptivecards.productboard.com/feature-board/planning/features/2181257


add svg basics test and example

#448

Looks like there's a bunch of compilation errors: https://travis-ci.org/rustwasm/wasm-bindgen/jobs/420500433#L3876
Does this compile and run OK for you locally?

In fact locally my example doesn't build but neither the old one (dom, hello-word...etc)
I used the script ./build.sh or i tried manually with wasm-bindgen cli i keep getting this errors.
`svg git:feature_svg_basics_exaple_testing ❯ ./build.sh     ⏎ ✹

cargo +nightly build --target wasm32-unknown-unknown
error: failed to resolve patches for https://github.com/rust-lang/crates.io-index
Caused by:
patch for wasm-bindgen in https://github.com/rust-lang/crates.io-index did not resolve to any crates. If this is unexpected, you may wish to consult: https://github.com/rust-lang/cargo/issues/4678`
This why i push to see the build result in travis, sorry for that.

I get this errors: Module not found: Error: Can't resolve './svg' in '/home/travis/build/rustwasm/wasm-bindgen/examples/svg'
It seems to me that i followed exactly the same structure that the others examples, but i'm certainly missing something, as it is not working.

Have you tried a cargo update perhaps?

Thanks @fitzgen cargo update solved all issues !

What's the status for this?

I'm gonna close this as it's been a bit quiet and stale for some time now, but we can of course resubmit at any time!


chore(slices): use stdlib slices.Index/IndexFunc

Description
With the release of go1.21 the slices package has been added, and we can replace a few of our internal slice utils with it.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change

see CI passing.

Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

#10612  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dhaus67 and the rest of your teammates on  Graphite


rails s trinidad is broken

Hi,
I ran rails s trinidad and got the following exception:
rails s trinidad
Exiting
NameError: wrong constant name trinidad
  const_get at org/jruby/RubyModule.java:2562
        get at /Users/Owen/.rvm/gems/jruby-1.6.6/gems/rack-1.2.4/lib/rack/handler.rb:21
     server at /Users/Owen/.rvm/gems/jruby-1.6.6/gems/rack-1.2.4/lib/rack/server.rb:221
      start at /Users/Owen/.rvm/gems/jruby-1.6.6/gems/railties-3.0.9/lib/rails/commands/server.rb:54
     (root) at /Users/Owen/.rvm/gems/jruby-1.6.6/gems/railties-3.0.9/lib/rails/commands.rb:30
        tap at org/jruby/RubyKernel.java:1785
     (root) at /Users/Owen/.rvm/gems/jruby-1.6.6/gems/railties-3.0.9/lib/rails/commands.rb:27
    require at org/jruby/RubyKernel.java:1036
     (root) at script/rails:6

I have trinidad in my Gemfile and running on JRuby 1.6.6. The example is here: https://github.com/engineyard/todo
Thanks,
Owen

You seem to be running trinidad 1.2.3 you should update to the latest 1.3.2 version (rack handler has been improved). However you're  a bit "out of luck" on rails 3.0.x since it locks you to rack 1.2.x thus you'll need to capitalize trinidad: rails s Trinidad (double checked that rails s trinidad works fine with rack 1.3.x on rails 3.1+)


Improve pooling

This pull request re-implemented pooling operator to support 1/2/3-D pooling operations without using expression template to increase performance.
Pooling: This implementation.
PoolingV1: Currently existing operator in MXNet.
CuDNNPooling: cuDNN pooling operator.
Table 1: max pooling benchmark results. New implementation’s 1D max pooling is compared with the equivalent 2D case.

1D: data=(10, 3, 100000), kernel=(64,)2D: data=(10, 3, 100000, 1), kernel=(64, 1)
CPU (ms)
GPU (ms)

Pooling(1D)
251
7

Pooling(2D)
680
14

CuDNNPooling(2D)
N/A
12

PoolingV1(2D)
4428
36

Table 2: max pooling 2D benchmark results.

2D: data=(10, 3, 100, 100), kernel=(8, 8)
CPU (ms)
GPU (ms)

Pooling
144
4

CuDNNPooling
N/A
5

PoolingV1
639
9

Table 3: max pooling 3D benchmark results.

3D: data=(10, 3, 100, 100, 100), kernel=(8, 8, 8)
CPU (ms)
GPU (ms)

Pooling
2509
53

CuDNNPooling
N/A
42

Table 4: avg pooling benchmark results. New implementation’s 1D avg pooling is compared with the equivalent 2D case.

1D: data=(10, 3, 100000), kernel=(64,)2D: data=(10, 3, 100000, 1), kernel=(64, 1)
CPU (ms)
GPU (ms)

Pooling(1D)
227
13

Pooling(2D)
938
17

CuDNNPooling(2D)
N/A
10

PoolingV1(2D)
2998
26

Table 5: avg pooling 2D benchmark results.

2D: data=(10, 3, 100, 100), kernel=(8, 8)
CPU (ms)
GPU (ms)

Pooling
123
6

CuDNNPooling
N/A
7

PoolingV1
412
8

Table 6: avg pooling 3D benchmark results.

3D: data=(10, 3, 100, 100, 100), kernel=(8, 8, 8)
CPU (ms)
GPU (ms)

Pooling
2301
85

CuDNNPooling
N/A
60

Please submit another PR to move conv and pooling to src/operator/nn/ after this

Any idea why 2d is that much slower than 1d?

For GPU 1/2-D kernels, one possible reason might be this line and the following "atomicAdd" function call.
https://github.com/dmlc/mxnet/pull/5519/files#diff-b6c5be2d23abce9c69031e26504e7c4fR387
The "if" condition would cause thread divergence in the 2D cases and consequently, more runtime than 1D.
The unpool_sum has no such condition check for 2/3 d kernels, and the relative difference between 1D and 2D is not that big.
But if we could have a mask as an invisible output, the unpool_max should be faster and have to atomicAdd calls.

still failing. not sure if its mkl bug or the new pooling op

I'm debugging to find the root cause.

@glingyan @zhenlinluo Looks like there is a bug with mkldnn pooling op when pad > 0

will looking it today

We will disable mkl for pad > 0 and merge this first. please re-enable when you submit a fix

@piiswrong how do you do the test, unit test?

consistency tests agains cudnn

@glingyan You can see the test log here. Just run test_pooling_versions() in test_operator_gpu.py.
The failing case is
pool_type='avg'
pooling_convention='full',
data = (2, 3, 20, 20)
kernel = (4, 5)
pad=(2, 3)
stride=(2, 3)
http://ec2-52-25-96-65.us-west-2.compute.amazonaws.com/blue/organizations/jenkins/mxnet/detail/PR-5519/5/pipeline#step-246-log-362

@piiswrong @reminisce Do we have any idea to improve also the deconvolution?

@piiswrong @reminisce @sxjscience deconvolution could be implment with mkl convolution api ,

@reminisce @piiswrong  will try to fix pooling when your patch is merged

I find the current deconvolution does not support 3D Deconv (https://github.com/torch/nn/blob/master/doc/convolution.md#nn.VolumetricFullConvolution), which has been used for video generation.

@sxjscience I think we can do similar things as we did for convolution by adopting Caffe's algorithm. The key functions im2col and col2im have been implemented for convolution. We use them for deconv as well.

@reminisce Yes, if that's not in the plan I can ask Leo @leezu  to help do the job

@piiswrong What do you think about @sxjscience 's proposal? If it has higher priority, I can do it.

@reminisce Yes we should do it. Shouldn't be too hard

@sxjscience If you have resources to work on deconvolution, you can lead it. Otherwise, I will spare time on this in the next a couple of weeks.
BTW: The deconvolution implemented is actually a transposed convolution. Tensorflow has renamed it to conv2d_transpose. Should we take the chance to make the name correct?
Refs:
https://github.com/tensorflow/tensorflow/issues/256#issuecomment-162257789
https://github.com/vdumoulin/conv_arithmetic

I think we can add an alias to the operator.

@piiswrong @reminisce
this problem is because when param.pooling_convention == pool_enum::kFull, the mkl's algorithm is the same with CUDNN_POOLING_AVERAGE_COUNT_EXCLUDE_PADDING
so the correct way to by pass is
#if MXNET_USE_MKL2017 == 1
if (param.kernel.ndim() == 2
&& (param.pooling_convention == pool_enum::kValid)
&& ((param.pool_type == pool_enum::kMaxPooling)
|| (param.pool_type == pool_enum::kAvgPooling))) {
switch (dtype) {
case mshadow::kFloat32:
return new MKLPoolingOp<cpu, float>(param);
case mshadow::kFloat64:
return new MKLPoolingOp<cpu, double>(param);
default:
break;
}
}

@reminisce We decide to first add the CUDNN version of 3D Deconv.

@glingyan Thanks for finding the root cause. Could you submit a PR to fix it?

@glingyan to be clear, you mean its a feature unsupported by mkldnn and need to be bypassed?

@piiswrong, current mkl2017 algorithm for pool_enum::kFull is designed for EXCLUDE_PADDING, if change to INCLUDE_PADDING will have segment fault
already report to mkl team
I will test mkl-dnn these days

@sxjscience I have some time in the following weeks. I plant to port Caffe's algorithm of deconv to MXNet as a low priority task, which means no deadline promise yet. I can merge it with your cuDNN 3D deconv work after I finish the implementation. Let me know if there are any conflicts. Thanks.

@reminisce @sxjscience Please try to reuse code instead of copy pasting for deconv

@piiswrong @reminisce an initial cudnn 3D deconv implementation is in #5615. Currently only the algoreg is reused, but if you have further suggestions we can refactor the code.


[5.9] Consider request validation before authorization

Description:
A long, long time ago, a PR was merged that changed the order of operations for the ValidatesWhenResolvedTrait: https://github.com/laravel/framework/pull/6407
I'm creating this issue to see if there would be an opportunity to either:

reverse the change (potential large breaking change)
add an option on the trait which individual request classes could override

The reason I believe the order should be: validate input -> authorize is because there are often times where the authorization logic needs to reference the request input. If the input has not yet been validated (as is current functionality), there are two problems:

developer has to manually validate inside the authorize method
user making the request is returned a 403 instead of the more-correct 422 response

Taking a slightly modified example from the Laravel documentation:
public function authorize()
{
    $comment = Comment::find($this->input('comment_id'));

    return $comment && $this->user()->can('update', $comment);
}

public function rules()
{
    return [
        'comment_id' => 'required|int|exists:comments,id'
    ];
}

If the user making this request specifies a nonexistent comment_id, I think it can be argued that the correct response is 422.
In addition, if the request input validation happened prior to the authorization, the authorize method could be cleaned up, since we're 100% sure the comment exists:
public function authorize()
{
    return $this->user()->can('update', Comment::find($this->input('comment_id')));
}

I'm more than willing to submit the PR for this, but I'd like to know if there's a chance it will be accepted (and which direction would be preferred).

Proposals should be sent to https://github.com/laravel/ideas/issues since the framework Issues are meant for bug reports: https://laravel.com/docs/5.8/contributions#bug-reports
<?php

namespace App\Http\Requests;

trait AuthorizesAfterValidation
{
    public function authorize()
    {
        return true;
    }

    public function withValidator($validator)
    {
        $validator->after(function ($validator) {
            if (! $validator->failed() && ! $this->authorizeValidated()) {
                $this->failedAuthorization();
            }
        });
    }

    abstract public function authorizeValidated();
}

class UpdateCommentRequest extends FormRequest
{
    use AuthorizesAfterValidation;

    public function authorizeValidated()
    {
        return $this->user()->can('update', Comment::find($this->input('comment_id')));
    }

    public function rules()
    {
        return [
            'comment_id' => 'required|int|exists:comments,id', // runs a duplicate database query
        ];
    }
}

For your example, the most appropriate response is a HTTP 404 status code if the comment isn't found. The resource to authorize doesn't exist. Laravel handles this case by using route model binding, no FormRequest necessary:
class CommentsController
{
    /**
     * PUT /comments/{comment}
     */
    public function update(Comment $comment)
    {
        $this->authorize('update', $comment);

    }
}

Also have a look at the once() memoization helper ( https://github.com/spatie/once ) that can be used in FormRequest objects to fetch models that aren't cached using route model binding.
class UpdateCommentRequest extends FormRequest
{
    public function authorize()
    {
        return $this->user()->can('update', $this->comment());
    }

    public function rules()
    {
        return [
            // no exists rule needed
        ];
    }

    // $request->comment() in the controller won't hit the database again
    public function comment()
    {
        return once(function () {
            // ModelNotFoundException thrown
            return Comment::findOrFail($this->input('comment_id'));
        });
    }
}

Either way, further discussion should be sent to the "laravel/ideas" Issues.

Heya, thanks for submitting this. This seems like a feature request or an improvement. It's best to post these at the laravel/ideas repository to get support for your idea. After that you may send a PR to the framework. Please only use this issue tracker to report bugs and issues with the framework.

Thanks @derekmd - My example was a bit too simplistic. I have hard time imagining there'd be much support to change this anyways, so I'll stick to a trait-based approach as you recommended.
Thanks.


Add pre-commit config

Any other check that might come in handy?

Should add:

https://github.com/executablebooks/mdformat
https://github.com/teemtee/tmt (might need https://github.com/teemtee/tmt/pull/2007 later on)
https://github.com/python-jsonschema/check-jsonschema (will use for github action, but maybe we can consolidate packit's lint there as well)


Add ERC721 docs.

issueERC721Property
issueERC721Token
transferERC721Token
destroyERC721Token

@GitCash 512 SV bits for @cgcardona

Hey cgcardona, user SpendBCH tipped you 512 bits in Bitcoin Cash ( ~ $0.213 ).
Click here to claim it!
You can also add the "thumbs down" reaction to SpendBCH's comment above to prevent future tips.


Provide option "Enter outlining mode when files open"

TypeScript Version:  2.3 / Visual Studio 2017 Preview 2
Please provide the following option for TypeScript files, shown here for C#:

Since VS 2017 has structure guidelines, the outlining is redundant for those who do not normally collapse sections of code.

//cc @billti

Closing due to lack of additional feedback


Nest Integration - Could not set temperature: Blocked

**Home Assistant release with the issue: 100.2
**Last working Home Assistant release (if known): 0.98
**Operating environment (Hass.io/Docker/Windows/etc.): Hass.io on Intel NUC
**Integration: Nest
https://www.home-assistant.io/integrations/nest/
**Description of problem: I am not sure if this is a "bug" in HA 100.2 or something changed on the Nest side (hopefully they didn't close down the developer service) but since yesterday I get an error while setting a target temperature on the Nest Thermostat v3.
The logbook reads as follows:
2019-10-24 09:01:01 ERROR (SyncWorker_6) [homeassistant.components.nest.climate] An error occurred while setting temperature: blocked
Strange thing is, I don't get the error every time I try to change temperature, but most of the time I do get this error. The nest thermostats (3 pieces) all have a good wifi connection and changing the temperature wit the Nest app works as normal.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

For info: Upgraded to 100.3 and issue is the same.

Hi there all,
I see now the same issue also here in my log file:
 an error occurred while setting temperature: blocked
i didn't move to Google with my account. It was working always correct.

I have been monitoring the issue for a while now. My temperature is changed whenever someone leaves the house etc. It works ok for 99.9% of the time. So I think it might be an issue with the nest server whenever the error occurs.

Here have problems with control NEST from HA. I can set temperature on NEST itself and I see the change in HA. When I change the temperature in HA there is nothing happen on the NEST itself.
I get the error message and see also the line "shows up X times"

On the Domoticz forum they build something to use NEST and Domoticz with Google Account instead of the NEST account
https://www.domoticz.com/forum/viewtopic.php?f=34&t=29984#p229124 2
Maybe developer of NEST integration in HA can take a look to this?

@poudenes that was using unofficial API revert engineering from Nest Mobile App. The underlying lib we used in HA, python-nest, was moving away from it long time ago. I personally don't have interest to re-implemnt it.
I think a better way to support new Google account is to acquire a official partnership from Google. I believe @balloob is working on it.


Retaining the nested fields that are not defined in payload collection

Link to reproduction
No response
Describe the Bug
Issue Summary:
When we try to update any nested field on Payload UI its overwriting the fields that are not defined on the Payload schema
Background:
Our team is working on the migration from existing CMS (KeystoneJS v4) to Payload, as it is not happening in KeystoneJS
when we migrate our application to Payload this issue will impact our data and can cause other impacts on the application that consuming those datas.
To Reproduce
Steps to Reproduce:
Step 1:
Create a collection as below structure
const Sample: CollectionConfig = {
slug: 'sample',
fields: [
{
name: 'name',
type: 'text',
},
{
name: 'address',
type: 'group',
fields: [
{
name: 'field1',
type: 'text',
},
{
name: 'field2',
type: 'text',
},
],
},
],
}
Step 2:
Create a sample data as mentioned below in the collection.
Ex. Sample data in a collection
{
"name": "",
"age": 20,
"address": {
"field1": "YYY",
"field2": "TTTTT",
"field3": "RRRRR"
}
}
Step 3:
On payload UI try to update the field "address.field1" OR "address.field2" and observe the data updated into the collection.
Updating a collection record(Ex. Sample) is overwriting nested fields that present in that db but that are not defined in the payload schema.
Tried Solution
We have tried the flattening module to modify the object before update query gets executed.
This way it will not impact any data that we update.
Refer the below code snippet modified on the file https://github.com/payloadcms/payload/blob/main/packages/db-mongodb/src/updateOne.ts
try {
// TODO: Remove this
if (options.flatten) {
const flat = require('flat')
data = flat.flatten(data);
}
const flat = require('flat');
data = flat.flatten(data);
result = await Model.findOneAndUpdate(query, data, options);
} catch (error) {
}
Also, ‘Options.flatten’ key comes as parameter that we set in the payload config object like below,
db: mongooseAdapter({
url: process.env.MONGO_URI,
flatten: true
})
We would like to know if this solution which we tried is acceptable and has no impact. Any other solutions using a different approach will be helpful.
Feedbacks are appreciated.
Payload Version
^2.18.3
Adapters and Plugins
@payloadcms/db-mongodb @payloadcms/plugin-relationship-object-ids

Hi @adarsh279 this is pretty much how the admin panel works as it pushed up the entire form state of the collection when editing.
Is there a particular reason the other data can't also exist in the payload schema?
As a worst case scenario you could put all of your external/old data in a JSON field with admin.hidden set to true so that you can then store it properly and retrieve it from your payload API.
Let us know if you need further guidance on dealing with this


Docs.MS Release Updates for azure-sdk-template

Update docs metadata and targeting for release of azure-sdk-template

Docs Build status updates of commit 63606fc:
:clock10: Pending: waiting for processors (31 builds ahead of you)
:warning: Docs Build is busy, currently there are 31 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 63606fc:
:clock10: Full build: the average full build time is 257 min(s) 30 sec(s), this is based on the last 881 build(s) for this repository.

Docs Build status updates of commit 63606fc:
:clock10: Full build: the average full build time is 261 min(s) 23 sec(s), this is based on the last 887 build(s) for this repository.

Docs Build status updates of commit 63606fc:
:clock10: Full build: the average full build time is 261 min(s) 23 sec(s), this is based on the last 887 build(s) for this repository.

Docs Build status updates of commit 63606fc:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/sdk-template-readme-pre.md
:warning:Warning
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)
Details

package.json
:white_check_mark:Succeeded

docs-ref-services/sdk-template-readme-pre.md

Line 56: [Warning-InvalidBookmark] Invalid link: 'Introduction'. The file docs-ref-services/sdk-template-readme-pre.md doesn't contain a bookmark named 'introduction'.
Line 56: [Warning-InvalidBookmark] Invalid link: 'Introduction'. The file docs-ref-services/sdk-template-readme-pre.md doesn't contain a bookmark named 'introduction'.
Line 56: [Warning-InvalidBookmark] Invalid link: 'Introduction'. The file docs-ref-services/sdk-template-readme-pre.md doesn't contain a bookmark named 'introduction'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


docs: configure Semantic Pull Requests

Configuration file for https://probot.github.io/apps/semantic-pull-requests/ based on the current https://github.com/xspec/xspec/blob/master/CONTRIBUTING.md

Tested with the title of #904. Working fine.


Update Travis CI to use standard test script and go 1.9.x.

More annoying go vet errors
swarm_test.go:119: result of (*peerstream.Swarm).String call not used
swarm_test.go:120: result of (*peerstream.Swarm).String call not used

Also see: https://github.com/libp2p/go-reuseport/pull/32 https://github.com/libp2p/go-libp2p/pull/245

I fixed the go vet error.

Okay, I rebased to meet our requirement that very commit passes the tests.

Something is trying to build example twice...


Alexa Screen SDK on Raspberry Pi - TuneIn stopped working - streamingError

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
Since couple of days tuneIn stopped working on Alexa Screen SDK on raspberry pi.
What is the expected behavior?
Alexa Screen SDK on raspberry pi should play tuneIn radio station.
What behavior are you observing?
After telling Alexa to play radio station , she is answering : 'playing radio station from tuneIn' but nothing happens.
Provide the steps to reproduce the issue, if applicable:
Tell Alexa to play radio station
Tell us about your environment:

What version of the AVS Device SDK are you using?
  2.2.0

Tell us what hardware you're using:

[x] Desktop / Laptop
[x] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[x] Other - tell us more:
Raspbian Buster
Windows 64-bit

Have you tried the same use case with AVS Device SDK SampleApp?

[ ] The issue is not applicable to the AVS Device SDK SampleApp.
[ ] The issue is not reproducible on the AVS Device SDK SampleApp.
[x] The issue IS APPLICABLE to the AVS Device SDK SampleApp, see:
Alexa SDK on raspberry pi - tuneIn stopped working

closing in favor of https://github.com/alexa/avs-device-sdk/issues/1808


Data Update - Current release

hey bud!  Wanted to check to see if the data needed to be updated, etc, for current release/any updates to NWMP
Thanks for the tool!

 Not sure about any new items, last update was from 3.0, so has been a while.
I'll will check later this week if there are any changes.

Looks like I have to change my parser first.
First I could use a single CraftingRecipes.json containing all the data but looks like that is now split in multiple files for each category.
Can probably look into this in the weekend.

Thanks for the update!  Appreciate ya!

v1.12.10 is ready. Fixed an issue with nwmarketprices.com and updated all the item data.


Update config_es

Hi Florent!
This patch looks ok to me, though I don't speak any spanish, so maybe you want to check for yourself?
Cheers

Yes, but it doesn't seem to be based on the up-to-date master, so the merge won't be fast-forward, which is not very clean. I'm exchanging emails with Carmen about all this, don't worry. :-)

Okeydokey!

Hmm, apparently Carmen has fixed this? I must check. I still have a screenshot showing the first patch-2 branch was based on frougon's master HEAD^ commit (parent of HEAD). :-)

Well, it has the new multiplayer testing options...

What did I fix that I haven't realized? lol

If I understand correctly, frougon had the feeling, that your HEAD was not in sync with frougon's HEAD...
But I can see no evidence of that so far. (Then again, I'm no git guru...)

Yes, history seems good now, and the last corrections too. :-)

I'm glad it worked but I have no idea of what I've done :)

Great! Now we have an up-to-date spanish config! Thanks!
Do you think about translating the rest too?

Argh, this GitHub merge button is crap! That's what Linus said, but now I understand why.
Carmen's history was fine: linear, based on my latest commit (commit 619bf70fc7d62bf41ce837ebcd724950c9287985 from Carmen is not a merge commit: it has only one parent, which was the latest commit on my master branch just before I pressed the 'Merge request' button). But when I pressed this button, GitHub automatically created a useless merge commit on my behalf. It is completely useless, because it is absolutely identical to Carmen's one, it just clutters history.
It is not a big deal once in a while, and now I know what to expect, but the SourceForge "merge feature" works much better (that's what Edward d'Auvergne said on the FG list). Next time, I'll try on the command line, not with the GitHub button...
Translation of the rest: yes, we talked about it, I gave Carmen instructions...

That's good news! \o/

I didn't undertand a word. I'm new here. But no problem ;)
Next time I'll just send you the files and that's all.

No, all was fine! It just looked strange, but that's not your fault, but github's...

And the GitHub  thing is very vicious (sorry if wrong word: this is French more or less): when I updated my clone:
% git pull
remote: Counting objects: 7, done.
remote: Compressing objects: 100% (4/4), done.
remote: Total 7 (delta 3), reused 7 (delta 3), pack-reused 0
Unpacking objects: 100% (7/7), done.
From github.com:frougon/FFGo
c1055c5..72ceab9  master     -> origin/master
Updating c1055c5..72ceab9
Fast-forward
ffgo/data/config/config_es | 124 ++++++++++++++++++++++++++++++++++++++-------
1 file changed, 107 insertions(+), 17 deletions(-)
%
As you can see, it says "Fast-forward", which means: linear history, no ugly merge. Because the merge commit was created directly on the SF repo, not by myself. So my clone has been fast-forwarded to the latest commit on the SF repo, which made it advance by 2 commits instead of 1 as I expected (first one: carmen; second commit: useless merge commit).
Yes, Carmen, don't worry. When you read the Git docs I gave you the URL's of, you'll understand better.

I wasn't worrying hahaha.
So much to learn in so little time.
I'll catch up

Next time: either send the files, or maybe you can commit in your clone and I can try to merge manually on my side, without the GitHub button. I think this should be doable, because I carefully checked your history and your commit was a proper direct descendant of my last one: c1055c5a9abed9855c37ef8c2ede91afdeb29bef

Yes it takes a little while, to get going, but then it becomes rather easy...

OK, good night folks! @Carmen: keep it coming! Great job!


SEO: increase site visibility

Increase site visibility by providing a robots.txt file that points to a generated page sitemap.
In addition this commit adds a verification code to be able to analyze it via Google Search Console. This does not track users or collect data, but only analyzes how well the page can be indext and found.

Merging this non-technical PR without approval to be able to check the site performance.


Display the sensitive information authorization groups easily, readily and quickly

Report authors do not know which authorization groups are available to choose from.
One report author cycled through all 26 letters of the alphabet on the keyboard to find out all the authorization groups available.

This can be resolved in 2.1 by using the new autosuggest selection widget

Closed by #1602


KTX2 - Fixing context issue and vis tests

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14825/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14825/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14825/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14825/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14825/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14825/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14825/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14825/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14825/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
Important - these might fail sporadically. This is an optional test.
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14825/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14825/merge/testResults/webgl2playwright/index.html


[randomstr] New port

Describe the pull request

Simple randomstr function in a header-only C89 library

https://github.com/offscale/c89randomstr

What does your PR fix?

New port

Which triplets are supported/not supported? Have you updated the CI baseline?

All

Does your PR follow the maintainer guide?

Yes
If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/
Although… I do have this TODO:
/* TODO:
 * - `BCryptGenRandom` on new Windows
 * - `CryptGenRandom` on old Windows
 * - `getrandom(2)` on Linux
 * - `arc4random` on BSD, macOS, and SunOS
 * */

Thanks for the port :D


zstd: 1.3.4 -> 1.3.5

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/zstd/versions.
These checks were done:

built on NixOS
/nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/bin/zstdcat passed the binary check.
/nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/bin/unzstd passed the binary check.
/nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/bin/zstdmt passed the binary check.
/nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/bin/zstd passed the binary check.
/nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/bin/zstdless passed the binary check.
/nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/bin/zstdgrep passed the binary check.
6 of 6 passed binary check by having a zero exit code.
0 of 6 passed binary check by having the new version present in output.
found 1.3.5 with grep in /nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5
directory tree listing: https://gist.github.com/c9f5f3b1ec0cb8753eaf906c1ca70238
du listing: https://gist.github.com/8cbddaa36df0d9e2c76ebcecdf78cec0

meta.description for zstd is: '"Zstandard real-time compression algorithm"'.
cc @orivej for testing.
Experimental: this build is cached with Cachix. To use the cache follow these experimental instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout e0cb36ee6e6df686a77e3d7ac4f8f9a20630deca && nix-shell --pure -I nixpkgs=. -p zstd

Success on x86_64-darwin (full log)
Attempted: zstd
Partial log (click to expand)
make[1]: Leaving directory '/private/tmp/nix-build-zstd-1.3.5.drv-0/source/programs'
post-installation fixup
gzipping man pages under /nix/store/l62q7xhyc4alaxm7blkqbgh6syyj94q4-zstd-1.3.5/share/man/
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/l62q7xhyc4alaxm7blkqbgh6syyj94q4-zstd-1.3.5/lib  /nix/store/l62q7xhyc4alaxm7blkqbgh6syyj94q4-zstd-1.3.5/bin
patching script interpreter paths in /nix/store/l62q7xhyc4alaxm7blkqbgh6syyj94q4-zstd-1.3.5
/nix/store/l62q7xhyc4alaxm7blkqbgh6syyj94q4-zstd-1.3.5/bin/zstdgrep: interpreter directive changed from "/bin/sh" to "/nix/store/q2wqq1k20v8kc3vckapqf5nws30brnni-bash-4.4-p23/bin/sh"
/nix/store/l62q7xhyc4alaxm7blkqbgh6syyj94q4-zstd-1.3.5/bin/zstdless: interpreter directive changed from "/bin/sh" to "/nix/store/q2wqq1k20v8kc3vckapqf5nws30brnni-bash-4.4-p23/bin/sh"
/nix/store/l62q7xhyc4alaxm7blkqbgh6syyj94q4-zstd-1.3.5/lib/libzstd.1.3.5.dylib: fixing dylib
/nix/store/l62q7xhyc4alaxm7blkqbgh6syyj94q4-zstd-1.3.5

Success on x86_64-linux (full log)
Attempted: zstd
Partial log (click to expand)
shrinking /nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/lib/libzstd.so.1.3.5
shrinking /nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/bin/zstd
gzipping man pages under /nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/share/man/
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/lib  /nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/bin
patching script interpreter paths in /nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5
/nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/bin/zstdless: interpreter directive changed from "/bin/sh" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/sh"
/nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5/bin/zstdgrep: interpreter directive changed from "/bin/sh" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5...
/nix/store/mqycyp5dkhcim84pq99kfqqf8s9k6bhp-zstd-1.3.5

Success on aarch64-linux (full log)
Attempted: zstd
Partial log (click to expand)
shrinking /nix/store/vsqyqc300v387dx4wpml7n7swdc404ya-zstd-1.3.5/bin/zstd
shrinking /nix/store/vsqyqc300v387dx4wpml7n7swdc404ya-zstd-1.3.5/lib/libzstd.so.1.3.5
gzipping man pages under /nix/store/vsqyqc300v387dx4wpml7n7swdc404ya-zstd-1.3.5/share/man/
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/vsqyqc300v387dx4wpml7n7swdc404ya-zstd-1.3.5/lib  /nix/store/vsqyqc300v387dx4wpml7n7swdc404ya-zstd-1.3.5/bin
patching script interpreter paths in /nix/store/vsqyqc300v387dx4wpml7n7swdc404ya-zstd-1.3.5
/nix/store/vsqyqc300v387dx4wpml7n7swdc404ya-zstd-1.3.5/bin/zstdgrep: interpreter directive changed from "/bin/sh" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/sh"
/nix/store/vsqyqc300v387dx4wpml7n7swdc404ya-zstd-1.3.5/bin/zstdless: interpreter directive changed from "/bin/sh" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/vsqyqc300v387dx4wpml7n7swdc404ya-zstd-1.3.5...
/nix/store/vsqyqc300v387dx4wpml7n7swdc404ya-zstd-1.3.5


Migrate to RxJS 6

Plugin migrated to RxJS
rxjs-compat used for Angular 5 compatibility

Thx for the PR. I've never used rxjs-compat but using it guarantees backwards compatibility and we don't need to adapt our peerDeps, right?

That's correct. We wouldn't need rxjs-compat if we were using Angular 6, for example. Technically, rxjs and @angular/core should be peerDeps, but that is implicitly covered by ionic-angular so no, no new peerDeps due to this PR.

Ok thx, and since Ionic 3.x is using<EMAIL_ADDRESS>we can't make this module depending on Angular6 implicitly.


python310Packages.pymsteams: 0.1.16 -> 0.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.pymsteams is: "Python module to interact with Microsoft Teams"
meta.homepage for python310Packages.pymsteams is: "https://github.com/rveachkc/pymsteams"
meta.changelog for python310Packages.pymsteams is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.2.0 with grep in /nix/store/hsl1ydrharwd73dcj34q5fhi44zdczjb-python3.10-pymsteams-0.2.0

found 0.2.0 in filename of file in /nix/store/hsl1ydrharwd73dcj34q5fhi44zdczjb-python3.10-pymsteams-0.2.0

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

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
python310Packages.pymsteams
python39Packages.pymsteams

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hsl1ydrharwd73dcj34q5fhi44zdczjb-python3.10-pymsteams-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 python310Packages.pymsteams https://github.com/r-ryantm/nixpkgs/archive/226728e600264087ddb257c1e09bad7fb0efb924.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/hsl1ydrharwd73dcj34q5fhi44zdczjb-python3.10-pymsteams-0.2.0
ls -la /nix/store/hsl1ydrharwd73dcj34q5fhi44zdczjb-python3.10-pymsteams-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

2 packages built:

python310Packages.pymsteams
python39Packages.pymsteams

Maintainer pings
cc @fabaff for testing.

Result of nixpkgs-review pr 155820 run on x86_64-linux 1

2 packages built:

python310Packages.pymsteams
python39Packages.pymsteams


Fix synthesis bug

Use inv_spectrogram method instead of inv_spectrogram_tensorflow, and the synthesis would be steady for all sentences.

Thanks for the PR!
Are you saying that inv_spectrogram produces better output than inv_spectrogram_tensorflow? I didn't notice this before, but if you have examples that demonstrate this, I'd be happy to accept this change.
Also, inv_spectrogram runs on the CPU, so it will be slower than inv_spectrogram_tensorflow when a GPU is available.  Perhaps this could be controlled by a flag so that you the user can choose which one to use?

Sorry I was wrong. Both inv_spectrogram and inv_spectrogram_tensorflow have the same results. And inv_spectrogram_tensorflow does possess speed advantage. Closing it.
eval-84000-inv-spectrogram.zip
eval-84000-inv-spectrogram-tensorflow.zip