default redirect all subdomains  to https? why ?

Hi
have the same problem as https://github.com/JrCs/docker-letsencrypt-nginx-proxy-companion/issues/293
HTTPS_METHOD=nohttp    why do i  need it set all subdomains containers  ?
I set -e "LETSENCRYPT_HOST=main-domain.com,www.main-domain.com" only for one. container with main domain. But nginx-proxy set redirect to https for all containers with subdomains crm.main-domain.com and other. But i don't don't need default redirect to https for all subdomains ....
for  example jenkins.prod.main-domain.com, i haven't set https for it.
But nginx-proxy set redirect

jenkins.prod.main-domain.com
upstream jenkins.prod.main-domain.com {
## Can be connected with "bridge" network
# jenkins
server <IP_ADDRESS>:8080;
# Cannot connect to network of this container
server <IP_ADDRESS> down;
}
server {
server_name jenkins.prod.main-domain.com;
listen 80 ;
access_log /var/log/nginx/access.log vhost;
return 301 https://$host$request_uri;
}
server {
server_name jenkins.prod.main-domain.com;
listen 443 ssl http2 ;
access_log /var/log/nginx/access.log vhost;
ssl_protocols TLSv1 TLSv1.1 TLSv1.2 TLSv1.3;
ssl_ciphers 'ECDHE-ECDSA-CHACHA20-POLY1305:ECDHE-RSA-CHACHA20-POLY1305:ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES128-GCM-SHA256:ECDHE-ECDSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-GCM-SHA384:DHE-RSA-AES128-GCM-SHA256:DHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-AES128-SHA256:ECDHE-RSA-AES128-SHA256:ECDHE-ECDSA-AES128-SHA:ECDHE-RSA-AES256-SHA384:ECDHE-RSA-AES128-SHA:ECDHE-ECDSA-AES256-SHA384:ECDHE-ECDSA-AES256-SHA:ECDHE-RSA-AES256-SHA:DHE-RSA-AES128-SHA256:DHE-RSA-AES128-SHA:DHE-RSA-AES256-SHA256:DHE-RSA-AES256-SHA:AES128-GCM-SHA256:AES256-GCM-SHA384:AES128-SHA256:AES256-SHA256:AES128-SHA:AES256-SHA:!DSS';
ssl_prefer_server_ciphers on;
ssl_session_timeout 5m;
ssl_session_cache shared:SSL:50m;
ssl_session_tickets off;
ssl_certificate /etc/nginx/certs/main-domain.com.crt;
ssl_certificate_key /etc/nginx/certs/main-domain.com.key;
ssl_dhparam /etc/nginx/certs/main-domain.com.dhparam.pem;
ssl_stapling on;
ssl_stapling_verify on;
ssl_trusted_certificate /etc/nginx/certs/main-domain.com.chain.pem;
add_header Strict-Transport-Security "max-age=31536000" always;
include /etc/nginx/vhost.d/default;
location / {
proxy_pass http://jenkins.prod.main-domain.com;
}
}

You can see it uses ssl from main domain ssl_certificate /etc/nginx/certs/main-domain.com.crt;
P.S. Sorry for my english

Hi
Is it the same as this issue ? https://github.com/jwilder/nginx-proxy/issues/1164

yes

The is an nginx-proxy issue then, closing this one.


Toolchain for firmware development

Hi all,
first: Congrats on achieving your goal of providing a fully open source FPGA!
I ordered my Clear board in February 2022 and finally received it two weeks ago.
Thanks for your great work!
Now I'm wondering how to get firmware on the device.
The "firmware" section in the README is a good starting point but up to now does not provide any details on the build dependencies required to build the firmware.
I assume that the documentation here and it's references are sufficient or is there anything needed in addition?
If you can give me some initial directions, I'm happy to contribute the basic documentation after getting the examples to run.
Greetings from Germany!
Benjamin

I managed to compile blinking led example from here .
I'm on debian like system:
$ sudo apt install gcc-riscv64-unknown-elf
$ make
generating hex for [blink]
riscv64-unknown-elf-gcc -I../fpga_bitstreams/ -I../common/ -O0 -mabi=ilp32 -march=rv32i -D__vexriscv__ -Wl,-Bstatic,-T,../common/sections.lds,--strip-debug -ffreestanding -nostdlib -o blink.elf ../common/crt0_vex.S ../common/isr.c ../common/gpio_program.c blink.c
riscv64-unknown-elf-objdump -s  blink.elf > blink.lst
riscv64-unknown-elf-objcopy -O verilog blink.elf blink.hex
sed -ie 's/@10/@00/g' blink.hex
$ python3 -m pip install pyftdi
$ make flash
generating hex for [blink]
python3 ../util/ftdi_flash.py blink.hex
Success: Found one matching FTDI device at ftdi://ftdi:232h:1:1c/1

Resetting Flash...
status = 0x00

JEDEC = b'ef4016'
Erasing chip...
done
status = 0x0
setting address to 0x0
addr 0x0: flash page write successful
addr 0x100: flash page write successful
addr 0x200: flash page write successful
addr 0x300: flash page write successful
addr 0x400: flash page write successful
addr 0x500: flash page write successful
addr 0x600: flash page write successful
addr 0x700: flash page write successful
addr 0x800: flash page write successful
addr 0x900: flash page write successful
addr 0xa00: flash page write successful
addr 0xb00: flash page write successful
addr 0xc00: flash page write successful
addr 0xd00: flash page write successful
addr 0xe00: flash page write successful
addr 0xf00: flash page write successful
addr 0x1000: flash page write successful
addr 0x1100: flash page write successful
addr 0x1200: flash page write successful
addr 0x1300: flash page write successful
addr 0x1400: flash page write successful

total_bytes = 5376
status reg_1 = 0x0
status reg_2 = 0x2
************************************
verifying...
************************************
status reg_1 = 0x0
status reg_2 = 0x2
setting address to 0x0
addr 0x0: read compare successful
addr 0x100: read compare successful
addr 0x200: read compare successful
addr 0x300: read compare successful
addr 0x400: read compare successful
addr 0x500: read compare successful
addr 0x600: read compare successful
addr 0x700: read compare successful
addr 0x800: read compare successful
addr 0x900: read compare successful
addr 0xa00: read compare successful
addr 0xb00: read compare successful
addr 0xc00: read compare successful
addr 0xd00: read compare successful
addr 0xe00: read compare successful
addr 0xf00: read compare successful
addr 0x1000: read compare successful
addr 0x1100: read compare successful
addr 0x1200: read compare successful
addr 0x1300: read compare successful
addr 0x1400: read compare successful

total_bytes = 5376

The board flashing is chaotic and doesn't works all time. I managed to flash with target flash2 and flash3 too but don't know exactly why it's so random.

thanks! ´flash´ did not work at first, then i tried ´flash2´ which was successful. ´flash´ and ´flash3´ are not very reliable and sometimes fail, exatly as you describe. Also sometimes, the device is not rebooted after flashing or something leaving it in ´not blinking state´

I wrote an issue for this chaotic flash.


Install Ubuntu in HP Pavilion Gaming Laptop 16 with nVidia 2060 RTX

https://www.hipercor.es/informatica/A35760126-portatil-hp-pavilion-gaming-16-a0025ns-i7-16gb-512gb-ssd-geforce-rtx-2060-max-q-6gb-freedos-sin-sistema-operativo/
In this ticket I will record the install process of a new laptop which include a nVidia Ray Tracing card: nVidia 2060 RTX. It is the mobile version (Max-Q).
It seems that the Linux drivers for the card are available since January: https://www.nvidia.com/Download/driverResults.aspx/141847/en-us
So we need to:

Install Ubuntu in HP Pavilion laptop (It seems to be easy at the end of 2019 , but not always)
Install the drivers for the nVidia card
Test the performance is the expected one
Check Vulkan support
Check the Ray Tracing support in Linux based on the Vulkan 1.2 extension

Here we go ... first step install it
https://ubuntu.com/tutorials/create-a-usb-stick-on-ubuntu#1-overview

Driver for nVidia card

Interesting:
«Así que muy chulo todo, pero de momento se reserva como placer de unos pocos. Sin embargo, la curiosidad de este lanzamiento es que NVIDIA ha liberado la implementación del trazado de rayos RTX en Quake II bajo licencia GPLv2 (que no GPLv3). Está en GitHub.» https://github.com/NVIDIA/Q2RTX

Ok, all is working now. Closing this ticket!
Next step is to have something to test the nvidia card.

We can download a demo from: https://www.nvidia.com/es-es/geforce/campaigns/quake-II-rtx/
https://international.download.nvidia.com/downloads/quake-2-rtx/q2rtx-1.3.0.deb

Not too bad but I have expected something more spectacular. But it is great to have a real game to test ray tracing and start living the experience.


The retrieval clients service URL building logic should be improved for organization resource accesses

Describe the issue:
The existing retrieval clients service URLs are build with an organization qualifier. But the organization qualifier for IS resource access is restricted. Hence the internal API calls for meta data retrieval should follow tenant qualified paths.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [IS 7.0.0-beta2]
OS: [Mac]
Database: [H2]
Userstore: [JDBC]

What's the impact of above ?

What's the impact of above ?

For an example, if authentication portal is hosted outside, while the authentication flow, the IDP logo retrieval would be done by invoking the IDP metadata endpoint in the IS. In order to retrieve those metadata, the IDP GET API is called via a client authentication mechanism.  Currently those request URLs are build with an organization qualifier /o/<org-id>/api/server/v1/identity-provider/..
We have defined a mechanism for tenant/organization resource/runtime API access  at the moment.

Accessing resources and runtime APIs in tenant ->  /t/<tenant-domain>/api/server/v1/identity-provider  &  /t/<tenant-domain>/oauth2/token
Accessing resources in org. -> /t/<tenant-domain>/o/api/server/v1/identity-provider
Invoking runtime APIs -> /o/<org-id>/oauth2/token

In order to comply as much with above, the effort of this issue is to build the URL in the format of
/t/<tenant-domain>/api/server/v1/identity-provider where the tenant-domain is the tenant related for the organization(sub-org).
Ideally the above URL should have been build as /t/<tenant-domain>/o/api/server/v1/identity-provider and invoked with organization bound token. But the client authentication mechanism having at the moment is not match with that.
As a summary, the effort is for avoiding URLs build with prefix with /o/... The given solution is not the optimal way but a workaround, where tenant domain and org-id is assumed to be equal (at the moment it is equal for orgs), and replace /o/ with /t/. Also, These calls are back channel calls.


Fix run profile

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Fix IDEA run profile
Why
With docker was not running with docker
Known limitations
[TODO or N/A]

Reference Server Preview is available here:https://anchor-ref-pr837.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr837.previews.kube001.services.stellar-ops.com/


command 'csharp.coreclrAdapterExecutableCommand' not found

When trying to debug inside of Visual Code I get this error:
command 'csharp.coreclrAdapterExecutableCommand' not found

Runtime Environment:
OS Name:     Windows
OS Version:  10.0.17134
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\2.1.302\
Host (useful for support):
Version: 2.1.2
Commit:  811c3ce6c0
.NET Core SDKs installed:
2.1.103 [C:\Program Files\dotnet\sdk]
2.1.201 [C:\Program Files\dotnet\sdk]
2.1.302 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.0.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.0.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

So what this means is that VS Code is somehow failing to find the command from the C# extension.
If you open a .cs file first, do you still see it?
If you go to the command pallet, do you see other C# extension commands (ex: '.NET: Generate Assets for Build and Debug)
If you don't see any C# extension commands, you might try uninstalling and reinstalling the C# extension. We had one previous customer run into this problem, and that fixed it for them.

I also have this problem on my Antergos. I have tried to reinstall C# extension, but it still won't download the ominisharp and nearly every command is not found.
Then I reopened vscode with --inspect-extensions flag, and in the developer tools, I  found this error.
[Extension Host] Activating extension `ms-vscode.csharp` failed:  Cannot find module './symbol/rxSubscriber'
t.log @ /opt/visual-studio-code/resources/app/out/vs/workbench/workbench.main.js:270
/opt/visual-studio-code/resources/app/out/vs/workbench/workbench.main.js:270 [Extension Host] Here is the error stack:  Error: Cannot find module './symbol/rxSubscriber'
	at Function.Module._resolveFilename (module.js:470:15)
	at Function.Module._load (module.js:418:25)
	at Function.t._load (/opt/visual-studio-code/resources/app/out/vs/workbench/node/extensionHostProcess.js:814:682)
	at Function.o._load (/opt/visual-studio-code/resources/app/out/vs/workbench/node/extensionHostProcess.js:765:373)
	at Module.require (module.js:498:17)
	at Module.patchedRequire [as require] (/opt/visual-studio-code/resources/app/extensions/git/node_modules/diagnostic-channel/dist/src/patchRequire.js:14:46)

Btw, I am using Settings Sync extension. And all extensions and user settings are synced from another computer. I don't know whether it matters.

Experiencing  exactly  same issue.
command 'csharp.coreclrAdapterExecutableCommand' not found
here is output of
dotnet --info :
.NET Core SDK (reflecting any global.json):
Version:   2.1.302
Commit:<PHONE_NUMBER>
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.16299
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\2.1.302\
Host (useful for support):
Version: 2.1.2
Commit:  811c3ce6c0
.NET Core SDKs installed:
2.1.302 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download
I created  .NetCore project using:
dotnet new console
dotnet new restore
By pressing Ctrl+Shift+B it prompts "No build task to run found. Configure Build Task"
-> "create task.json from template ->.Net Core (Execute .Net Core build command) -> it creates task.json:
{
    // See https://go.microsoft.com/fwlink/?LinkId=733558
    // for the documentation about the tasks.json format
    "version": "2.0.0",
    "tasks": [
        {
            "label": "build",
            "command": "dotnet build",
            "type": "shell",
            "group": "build",
            "presentation": {
                "reveal": "silent"
            },
            "problemMatcher": "$msCompile"
        }
    ]
}

However -Debug option still does not work.
When I press F5  - no .Net Debugger available in the list 👎 .
I added launch.json file manually under the .vscode folder.
When I open launch.json file- it shows "Add Configuration" button that  does nothing.
How can I fix it?

@Excited-ccccly do you still install your extensions to the standard location (~/.vscode/extensions)?
Does ~/.vscode/extensions/ms-vscode.csharp-1.15.2/node_modules/rxjs/symbol/rxSubscriber.js exist on your computer?

@MaxIakovliev if you uninstall and reinstall the C# extension, does the problem still happen for you?

@gregg-miskelly No, it doesn't exsit.
➜  ~ ls ~/.vscode/extensions/ms-vscode.csharp-1.15.2/node_modules/rxjs/symbol
iterator.d.ts

Then I execute npm install in ~/.vscode/extensions/ms-vscode.csharp-1.15.2/node_modules/rxjs folder. After installation, rxSubscriber.js still doesn't exist.
I have also tried to init another npm project and install rxjs as a dependency. The rxSubscriber.js exsits in this newly created project.

@gregg-miskelly  I have delete ms-vscode.csharp-1.15.2 manually, and reinstall the c# extension. It works!

Reinstalling the c# extension from the UI fixed it for me..

I just got this error, pretty sure after VS Code updated itself.
After reading the comments here I tried to uninstall and then reinstall the C# extension (In my case it's version 1.16.1), but that didn't help. I then went on to check if I had the file ~/.vscode/extensions/ms-vscode.csharp-1.16.1/node_modules/rxjs/symbol/rxSubscriber.js, and notices that although the file exists, it is completely empty. Lastly, after trying reinstalling again, I noticed that my ~/.vscode/extensions/ms-vscode.csharp-1.16.1 directory wasn't actually being removed when I uninstalled the extension.
So finally, the solution that worked for me -

Uninstall C# extnesion
Manually delete extension directory from ~/.vscode/extensions/
Reinstall the extension

Hope this might help someone

@nimrodolev I think if this is not added as bug, then you should raise this.


Collision rewards cause skewd reward distribution

Hello, there. Thanks for the great work!
The issue is that in default "highway-env" setting, collision_reward are relatively low(-1) in constrast to other rewards like highspeed_reward(+0.1), which would leads to the reward shift after lmap. So the reward recieved doesn't differ so much at every timestamp.

This issue is likely to cause the adavantage function estimated bad in my experiements. Currently, I  compensate it by adding a epsilon survival reward each timestamp and zero out the collision reward. Would you consider adding this type of reward?
Your help is greatly appreciated.

Hi,
I'm not sure that I follow exactly what you are saying:

indeed by default the collision reward is -1 and speed/right-lane rewards are in a range [0.-0.5].
and indeed, a linear mapping is applied by default, which rescales the reward to the [0,1] range.
but that means that already, if the vehicle is not in a collision state, the reward=0 it receives is converted to reward = 0.66, which you can already see as a survival reward (as opposed to collisions giving a terminal r=0 reward).

Anyway, this should all be configurable directly: you can disable normalize_reward if you don't like it, and you can set the collision_reward etc. directly to whichever value you like.
Let me know if I am missing something or you have any question.

Thanks a lot,
But I'm wondering if you notice that the rewards are almost always greater than 0.66. And a tiny speed/right reward would make the l-mapped reward near 1, which in turn makes ALL_ACTIONS being rewarded roughly same value.
Under RL, the ego vehicle's best solution is to slow down and follow the front car.
I don't know if it was clarified a bit more, but I will upload some experiment result in the future.

If I understand correctly, you are saying that

the +1 reward for avoiding collisions is significantly larger than other reward terms -speed and right lane, which can only total at +0.5
thus, a "bad" conservative policy which stays safe by driving slowly has a value of 1H, is already near-optimal (1T is 66% of the optimal value of 1.5*T, where T is episode length), and there isnt much incentive for the agent to discover something better

(I ignored the reward normalisation here, since it does not affect the ranking of policies)
I guess you are right that 2/3 of the optimising effort is allocated on just "staying alive", but there is still 1/3 on efficiency which is not negligible, and in my experience RL agents have been able to take advantage this 1/3 signal.
But of course these weights are all configurable, so feel free to adjust them if you think a different weighting of safety vs efficiency is preferable :)


A webtorrent browser add-on

Would you guys consider making a browser add-on (webextension) for webtorrent? There are many reasons why this would be useful... For example I can just install the javascript once, and webtorrent will automatically work on any website that uses src="torrent://<magnet-url>", users don't have to download the same library for every website that uses webtorrent, developers don't have to code anything they just have to use URLs in src, users might not have javascript enabled, reduce chances of people tampering with webtorrent javascript and incurring in XSS attacks, easier to update (just update the webextension instead of updating all the libraries in each website using it), no privacy issues with CDNs and trackings, less javascript, ....

good idea, go ahead!!!

The issue is that no webpages are going to use src="torrent://<magnet-url>" because not all browsers support WebRTC. Instead what I'm working on is a way to make it so web designers don't put an src at all. Instead they'd put an href and my script (still figuring out the logistics of the project) would:

Check WebRTC support
If no support, move the hrefs to the srcs
If support, download the blob and make the blob the src

Also need to figure out how to cache blobs.
The size of WebTorrent is an issue that could be solved with a simple extension! I'll update you today

I think a webtorrent browser extension is a cool idea. There's already an issue about it here: #443 If I get a free moment, I might hack it up, but anyone else is welcome to tackle it too.
The advantage of a chrome extension / chrome app is that it should be possible to support normal (tcp/utp) peers, meaning that users with this extension are more likely to find peers they can connect to, since they can connect to more than just the webrtc ones.


failing local install

Hi,
I am trying to install nngraph locally using $ luarocks install --local nngraph. I was surprised to see that it said that I had missing dependencies for nngraph, even though I could seemly import them with require.
Missing dependencies for nngraph:
torch >= 7.0
nn
graph

After trying to build I end up with the following message.
Install the project...
-- Install configuration: "Release"
-- Installing: /usr/share/torch7-dist/install/share/cmake/torch/TorchExports.cmake
CMake Error at cmake_install.cmake:48 (FILE):
 file INSTALL cannot copy file
 "/tmp/luarocks_torch-scm-1-446/torch7/build/CMakeFiles/Export/share/cmake/torch/TorchExports.cmake"
 to "/usr/share/torch7-dist/install/share/cmake/torch/TorchExports.cmake".
make: *** [install] Error 1

Error: Failed installing dependency: https://raw.githubusercontent.com/torch/rocks/master/torch-scm-1.rockspec - Build error: Failed installing.

Would appreciate any help.

Is torch installed in ~/torch?  Can you try running the following first?:
export PATH=/home/user/torch/install/bin:$PATH
export LD_LIBRARY_PATH=/home/user/torch/install/lib:$LD_LIBRARY_PATH
export DYLD_LIBRARY_PATH=/home/user/torch/install/lib:$DYLD_LIBRARY_PATH

eval $(luarocks path --bin)

Hey Hugh,
I tried what you suggested and had no luck. It is installed in /usr/share/torch7-dist.

Normally it would be installed in ~/torch.  I think you might consider reinstalling from scratch, following Soumith's instructions at https://github.com/torch/distro

Alright thanks


Link inaccessible

[Enter feedback here]
I was following this :
: Transformation with Azure Databricks - Azure Data Factory | Microsoft Learn
Under the section " Import a notebook for Transformation " -> 2. Select Import from: URL. In the text box, enter https://adflabstaging1.blob.core.windows.net/share/Transformations.html.
The " https://adflabstaging1.blob.core.windows.net/share/Transformations.html " is not available
Do you know who from your team might still have access to the notebook - that would be great. I am trying to leverage this resource for an upcoming demo.

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

ID: 91402667-4be2-ddaf-6b7a-d24d478e5cac
Version Independent ID: f171700d-f8e1-8b79-779c-9e12c0a03fa8
Content: Transformation with Azure Databricks - Azure Data Factory
Content Source: articles/data-factory/solution-template-databricks-notebook.md
Service: data-factory
Sub-service: tutorials
GitHub Login: @nabhishek
Microsoft Alias: abnarain

@sragh12
Thanks for your feedback! We will investigate and update as appropriate.

@sragh12
Thank you for bringing this to our attention.
I've delegated this to content author @nabhishek, who will review it and offer their insightful opinions.

@nabhishek
Could you please review add comments on this, update as appropriate.

@Naveenommi-MSFT
We are looking into URL not working.

Hi @sragh12, and thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


Missing aria-label in nav

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
In any docusaurus application including the official docusaurus website (https://docusaurus.io/docs), there are two <nav> elements, that don’t have aria-label.
Reproducible demo
No response
Steps to reproduce

Go to https://docusaurus.io/docs
Open dev tools
Search for <nav> elements (there are 4 <nav> elements in total)
First and second <nav> elements don’t have aria-label

Expected behavior
All <nav> elements have aria-label so that assistive technologies like screen readers can identify unique navigation elements.
Actual behavior
Main navigation and sidebar navigation elements don’t have an aria-label.
Currently, this is causing accessibility issues: https://dequeuniversity.com/rules/axe/4.4/landmark-unique.
Your environment

Docusaurus version used: 2.2.0

Self-service

[X] I'd be willing to fix this bug myself.

Thanks for reporting a11y issues.
I'm not an a11y expert, what do you suggest as aria-label here?
Is there a way to detect such issues easily as part of our CI?

You're welcome.
For main navigation, it can be aria-label="Main", for sidebar aria-label="Sidebar".
You can use Axe to detect such issues, it is an accessibility testing engine for websites and other HTML-based user interfaces.

It might be better if it would be possible to set the aria-label by a prop, the same way how you set it for navbar items in the docusaurus.config.js, just to give users a bit more control over the text content. Would you be interested in implementing it yourself or should I do it?

Hi,
All aria labels should be translated and we provide default translations for them. If label is provided through config then it makes it harder for users to localized those, and I doubt many users will want to customize this anyway so it should probably not be an option.
The way to customize it would be:

to swizzle the component
to use the translation strings (you can customize the default english translations for example)

Hi,
Thank you for the translation tip, I didn’t know about it! I agree with you, then we can just add the aria-label for the main and sidebar navbars, and it would be possible to change the values by translation strings.

Hi,
Would you be interested in implementing it yourself or should I do it?

Hi
It's an easy to contribute issue so I keep it for anyone willing to send a PR ;)

Hi @slorber and @Alena5041.  Im a first time contributor who navigated to this bug and decided to give it a shot.  I have created a draft PR for the issue and had some questions regarding if I processed this correctly.
when looking into this issue, I realized in the comments that "aria labels should be translated".  Im not entirely sure I understand what translated means, but when looking through the repo other aria-label attributes are made using the translate function from @docusaurus/Translate.
Each instance of a translate function accepted an Id Parameter.  I tried to follow the naming convention similarities I noticed, but am unsure if they have been named appropriately.
I have created a draft PR but am unsure if I completed the fix appropriately

based on the work I did in the PR, did I attach the correct Id parameter?

I noticed tests for the translate function were established in the packages/docusaurus/src/client/exports/__tests__/Translate.test.tsx file so I didnt know if any additional testing would be required.  Should I have written tests somewhere to verify the nav element is rendered with an aria-label?

Thanks a bunch in advance for the response/feedback.

Thanks @r-lawrence 👍
@Alena5041 can you please help review the PR, added some questions/concerns about those aria labels.
Also wondering if we do the right thing on mobile? We don't use nav here for the docs mobile sidebar apparently so there's no reported missing aria-label. Do screen reader users also use Docusaurus with a mobile viewport?

Hi @slorber,
Sure, screen reader users also use a mobile viewport. The fact that you change for mobile <nav>  for the <div class="navbar-sidebar"> is not a problem in your case, because the main and sidebar navigations wrap into one main <nav>. But still, need to have aria-label on the main navigation.

Thanks @slorber and @r-lawrence for the quick solution to my issue


Github source CrashLoopBackOff

Expected Behavior
github source pod running state
Actual Behavior
NAME                          READY   STATUS             RESTARTS   AGE
github-controller-manager-0   0/1     CrashLoopBackOff   29         124m

k logs  -n knative-sources github-controller-manager-0
2019/08/08 16:11:21 Registering Components.
2019/08/08 16:11:21 Adding the GitHub Source controller.
2019/08/08 16:11:21 no matches for kind "Service" in version "serving.knative.dev/v1beta1"

Steps to Reproduce the Problem

versions > 0.7.0
check github source pod status

Additional Info
Appears to be related to this PR https://github.com/knative/eventing-contrib/commit/c25d3142bddbec38fbee364b8a5857d4a7f43cd7
The service was change to v1beta1 from v1alpha1
but in the CRD for serving it's still v1alpha1
https://github.com/knative/serving/releases/download/v0.8.0/serving.yaml
apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
  labels:
    knative.dev/crd-install: "true"
    serving.knative.dev/release: "v0.8.0"
  name: services.serving.knative.dev
spec:
  additionalPrinterColumns:
  - JSONPath: .status.url
    name: URL
    type: string
  - JSONPath: .status.latestCreatedRevisionName
    name: LatestCreated
    type: string
  - JSONPath: .status.latestReadyRevisionName
    name: LatestReady
    type: string
  - JSONPath: .status.conditions[?(@.type=='Ready')].status
    name: Ready
    type: string
  - JSONPath: .status.conditions[?(@.type=='Ready')].reason
    name: Reason
    type: string
  group: serving.knative.dev
  names:
    categories:
    - all
    - knative
    - serving
    kind: Service
    plural: services
    shortNames:
    - kservice
    - ksvc
    singular: service
  scope: Namespaced
  subresources:
    status: {}
  versions:
  - name: v1alpha1
    served: true
    storage: true

cc @dugliln

This could be related to the yamls, right now just using the serving.yaml from 080.
serving-beta-crds.yaml and serving-alpha-crds.yaml
serving-post-1.14.yaml and serving-pre-1.14.yaml
Not sure how to mix/match these to get a stable release, don't see them covered in the docs yet.

@mattmoor any guidance or pointers to docs to help?

We should roll back the change to use serving at v1beta1 version.

Have the same issue...

Found a work-around - https://github.com/tektoncd/experimental/issues/177

this is fixed.


Scripted Logo Generation

Something I've been working on in my spare time for fun (don't judge)... This PR proposes generating Iris' logos via a script, rather than holding fixed image files in the repository. I appreciate integrating this will be a lower priority given our current projects.
For an idea of what the script can generate: see this pptx file or run the script yourself 🙂
Advantages over existing method

No reliance on fixed files that are created using un-traceable methods (typically via a GUI graphics program)

Script is fully visible to version control

Image configuration easily adjustable (colours, transparency, gradients etc.)
Iris' logo is made from relatively few elements with mostly simple configurations

Title image includes parameterised text - particularly useful if we continue to modularise Iris and need an matching title logo for each package

SVG format renders crisply at any resolution
Also doesn't blur in PDF's, unlike rasters (even high def ones)

SVG format means very small generated file sizes (<30kB)

Pythonic, in line with the rest of Iris' code

Don't need to hold multiple versions of the same image - all generated by a single script

To do
I've got as far as I can solo. Should we proceed with this, here is what I would need help with:

[ ] Adjust image configuration to match existing logo.
I did my best to reverse engineer from Iris7_1_trim_full.png, but a few things are a bit off since I didn't have access to the original Inkscape file (see advantage number 1). I believe @abooton has the original file for the current logo?
[ ] Integrate generate_logo.py into the docs build as a Sphinx extension. I would need @tkknight's help to understand and achieve this.
[ ] If above integration is successful:

Adjust all existing logo references to point at generated logos in the built docs.
(Tephi's README is a good example of this)
Remove all logo image files from the docs src _static directory.

Examples

Rotating

This is now ready for review, whenever there is time. It should ideally have @abooton 's review for content, and @tkknight 's review for sphinx integration.
Proposal (left) compared to current logo (right)

Coastlines simplified (generate_logo.py#L103), but no longer smoothed.
This is due to the difficulty of smoothing/rounding using the script, but personally I like the blocky coastlines.
Globe is now fully round.
I'm told the current distortion is a mistake (rather than an attempt to simulate Earth's oblate-spheroidal shape!).
Globe is no longer tilted anti-clockwise.
I assumed this was a mistake, but happy to be corrected.

I suggest updating your branch as there have been a few documentation related changes merged to master since youre branch was created, including conf.py

I suggest updating your branch as there have been a few documentation related changes merged to master since youre branch was created, including conf.py

@abooton what are your feelings on me rebasing? There's a lot of useful content in this PR so I'd be reluctant to create a new one just to get the latest changes, especially since the new PR might also stay un-merged for a week or two (during which time master will change some more).

I suggest updating your branch as there have been a few documentation related changes merged to master since youre branch was created, including conf.py

@tkknight done

A "What's New" entry will be provided in a separate PR once this one is approved.

Closing, since this is non-essential and there is limited interest


Display SourceLink enabled status

Feature request

Today NPE displays repository information on a package. This is only some of the metadata that SourceLink adds. NPE does not show whether the DLL/PDB files have correct SourceLink information.
Note: this can be done today using a command line tool - https://github.com/ctaggart/SourceLink#test

Why do we need it?

NPE would provide package authors a visual way to verify that SourceLink is enabled.

Sounds great! Definitely a useful feature to test validity of contained packages

@JamesNK Would a content viewer that shows the source information suffice?
The code in the sourcelink repo appears to go further and download each source to validate the hashes. Are you looking for that as well?

I think showing information is enough. Validating against the source would have perf implications. The dotnet tool can be used if someone wants to validate.

Available in build 4.5.86 and higher. In the CI feed first, then others soon.


iterm2: 3.4.15 -> 3.4.17

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Hello there,
there is already a new version 3.4.18. Would it be possible to update the package please?

@Koleon Nixpkgs is driven by the community and it can happen that not every patch level update is included because is takes some effort to review the changes and merge it. As result, we may jump multiple levels as in this pr.
Feel free to open a new pull request for 3.4.18.


Motivations Part 2a

Which verifiers are authorized to request what types of verifiable presentations.

request what types

Michael D: I would like to make a second argument, why the limitation on verifiers is problematic.
Typically legislation like the GDPR regulates what data an entity (a verifier) may process and such regulation is enforced much more generally. It also applies if the entity just presents the user with a form and asks for the data.
If we build this into the protocol, we would either have to verify the legal framework of the verifier during registration, or the legal framework of the trust registry would simply be applied globally.
Darrell: Many ecosystems will be wide open but others will require limitation or bounding of Verifiers. It's a choice. You don't need to limit it. All this protocol does is allow the asking of "Is this Verifier Authorized to ask for PROOF___ under this governance framework?"

comments from v1 draft - closing


Updated getKeyInfo function with actual implementation

Hello @cjbarth, @LoneRifle, @yaronn,
I have added implementation for getKeyInfo function, please check if it looks good, then I'll proceed with unit tests.
I am getting confused with below line of codes,
For FileKeyInfo, I'm assuming it accepts file which will be public X509 certificate and this.getKeyInfo function will return the X509Data. But, what is the use of this.getKey()?? I'm assuming it will return public certificate. But, in some places it used as privateKey, as shown in below statements,
In node-saml->src->xml.ts->signXml, signingKey gets the privateKey value assigned, as shown below,

However, in xml-crypto->lib->signed-sml.js, signingKey gets the value from KeyInfoProvider.getKey(this.keyInfo) which looks to be public certificate..

Can you please help me to clear this..

@ganesha289 , I'm not sure what the confusion is. Signing should be done with a private key and verification of a signature is done with a public key. The code that you reference in signed-xml.js is part of the function called checkSignature, so it would make sense that it would use a public cert. As for node-saml, that looks like it is correctly using a private key to sign an XML document.

@ganesha289 I think we have a general idea of how we'd like to move forward. Since @LoneRifle and I are pretty much the only ones left doing anything that looks like maintenance on this project, I think the three of us can figure out the correct pattern here, implement it, and close this out. I know node-saml will benefit from it, and I expect others will too, especially if the pattern is clearly established.


[CI] Switch to using Crowdin GitHub Action

Fixes #7910

Thanks for implementing this. Let's see if it works!

@matmair more work required it looks like:

Ok, got it working by fixing the token permissions:

New problem - the translation completion status is now way down:

Each language now has an additional "source" file:

And also, the incoming PR created by Crowdin contains way too many changes - to all the files essentially. So we need to reassess the file filter used when making the new branch / commit.


Support for things like the "bounds=" option for the Google geocoding service

Hey Alex,
When you have a minute can you take a look at the last three commits on my fork:
https://github.com/rosscooperman/geocoder/commits/provider_options
I implemented an (optional) options parameter to Geocoder.search that will allow you to say something like:
Geocoder.search("Some Intersection", :bounds => [[40.0, -120.0], [39.0, -121.0]])

to bias search results in a given bounds.  I only added the functionality to pass the options through to the Google lookup object but I was planning to add viewport biasing to the other lookup providers that support it.  As implemented the feature could also be used to implement other provider options.  I wanted to run this by you before I take it any further to see if you would consider pulling something like this in.
Thanks!
~Ross

Thanks, yes, I would definitely consider pulling this. The only other things we need are:

Tests (in test/lookup_test.rb) to make sure the bounds are included in the query URL.
Deprecation warnings and continued support (with tests) for passing a boolean as the second argument to Lookup::Base.search. Nobody should really be using this method directly, but it is public so I want to be careful.
Documentation (describe briefly in the README).

Looks great so far. Thanks!

Excellent.  I did add a test to test/services_test.rb that does what you mentioned in #1 (https://github.com/rosscooperman/geocoder/blob/574cf0dd28b567c844dc7b420490f341cc3c0101/test/services_test.rb#L31).  Would you rather see that in lookup_test.rb?  I figured I would write a test for each service.  I'll also probably write some tests around the behavior of the Geocoder.search with various combinations of arguments.
I'll get #2 and #3 done before I send a pull request.  Thanks!

Sorry, you're right: test/services_test.rb is the place for it, but I don't see it. Did you forget to push a commit?
Sounds great.

It's possible you're looking at the master branch in my fork.  I'm working in the provider_options branch (I did push a couple of commits to master by accident though).  The name of the test is test_google_query_url_contains_bounds.

Ah, I see it now. Looks great.

Hmm, so I was going to add a deprecation warning (and note in the rdoc) to Lookup::Base#search but I went back to the unmodified source and it doesn't appear to take a second parameter (https://github.com/alexreisner/geocoder/blob/master/lib/geocoder/lookups/base.rb#L25).  It looks to me like the first call that takes a reverse argument is the Lookup::Base#results method (overridden in each provider).  Should I add a deprecation warning there?  I was kinda assuming that nobody is calling those methods directly but it is certainly possible.


Bad Request when Creating a Store

When I run client.stores.create() I am getting HTTPError: 400 Client Error: Bad Request for url: https://us1.api.mailchimp.com/3.0/ecommerce/stores

The issue here is that the site ID I was passing was an integer and it wanted a string.  As I stepped through the code and inspected the response it did in fact say Bad Request, but there was more detailed info in there that let me know exactly what was wrong.
It would be nice if this library could pass us back this extra error info.


Accounts-Manager to continue accounts fetching procedure in case if genesis ledger account entry does not provide the sk property.

this has also been a source of test flakes in o1js CI

Fixed, building now for different arch and will rebuild lightnet images.

https://github.com/shimkiv/mina-accounts-manager/releases/tag/v0.1.1
Will be published to DockerHub after the https://github.com/o1-labs/mina-lightnet-docker/actions/runs/8660174646


e2e: Easier to follow log

Additional log cleanups split from #1628:

remove the file:line since it make it harder to follow the log and does not add much value.

it could be nice to have it we could move it to the end of the log https://github.com/uber-go/zap/discussions/1471

Remove unhelpful noise from test name "Deploy", "busybox"

Summary from discussion with @raghavendra-talur:
We want to replace the current format:
{deployer}-Deploy-{rbd|cephfs}-busybox

With:
{deploy|statefulset|daemonset|vm}-{block|file}-{deployer}-{volrep|volsync}

(I'm not sure about the order)
So the tests are about:

The workload
The volume mode (different modes need different deployment script for busybox)
The way the workload is deployed
The way we protect the workload

And not about specific storage provider (rbd, cephfs) - Ramen is storage agnostic.
The config determines the particular storage to use for the test. The same test will work on drenv, openshift, k8s, with any compatible storage,


How Paste image to docx

I try paste image
@thumb = '<img src="http://www.fips.ru/rutmimage/0/200000/270000/270000/270041-s.JPG" border="0">'.to_html
as raw (this is HAML)
= raw @thumb
then i get error
Unrecognized unit of measure: . Unrecognized unit of measure: . Unrecognized unit of measure: . Unrecognized unit of measure: .
Haw can i paste image from URI fo example?
Help please!

I answer myself!
I remove from action 'create_docx'
  respond_to do |format|
    format.docx do
...

In routes I set route
get "create_docx" => "fips_tzs#create_docx", format: 'docx'
And I set mime
Mime::Type.register "application/vnd.openxmlformats-officedocument.wordprocessingml.document", :docx
Then I get doc from URI 'http://fips.glafir.ru/fips_tzs/:ID/create_docx'
Its work fine!
gem 'rails', '5.0.2'

Please send full code. Does Not work, for me


Add override option

This PR adds the sources[n].override option that enables to override custom source behavior.
For example, you can change is_available condition like this.
cmp.setup {

  sources = {
    {
      name = 'nvim_lsp',
      override = {
        is_available = function()
          return context.in_treesitter_capture('comment')
        end
      }
    }
  }

}

This PR has following breaking changes.

Remove config.sources[n].keyword_pattern option.
Remove config.sources[n].trigger_characters option.

just wondering
what else does this pr add?
because I can't imagine that all those 5.6k loc are for the override option only

https://github.com/hrsh7th/nvim-kit
This PR adds the nvim-kit.

because I can't imagine that all those 5.6k loc are for the override option only

+1, I think adding nvim-kit should be a separate commit/PR (actually I'm not sure this is good to be merged to nvim-cmp core). Why not as a separate library or dependencies? I see many duplicates with plenary.nvim. It also seems a few more refactoring would be needed. It'd be greatly appreciated if @hrsh7th can please explain more rationale and benefits of having that merged to core.

The reason for embedding nvim-kit in core is that avoiding breaking changes and new dependencies for users.

I already use nvim-kit in some of the my codebases.
https://github.com/hrsh7th/cmp-core-example
https://github.com/hrsh7th/nvim-linkedit
https://github.com/hrsh7th/nvim-gtd
https://github.com/hrsh7th/nvim-insx
As the neovim core provides higher level APIs, it becomes less necessary, but is still needed at this point.

@wookayin I don't think it's bad to add the nvim-kit to the nvim-cmp. Why do you think it's bad?

@wookayin ping

@wookayin Hi, Shouldn't I install nvim-kit in nvim-cmp?

yes. I can't take time at the moment, and my motivation has decreased, so I closed it.


Variable default values

Simple variable default values, as discussed in #144 & #148
Changes are backwards compatible, tested it with some types. Would be great to hear feedback from others if this works for them or if I'm missing something :)

@fritz-gerneth Thanks! I just added additional test for parser, merging.


ENH: Add contours to MRICoregRPT report

See https://github.com/poldracklab/fmriprep/issues/758.

@oesteban Look reasonable?


activate: discard output from pyenv-sh-deactivate --quiet

Without this the mktmpenv script I am using would fail, because of the eval false.

Could you fix broken tests?

Could you fix broken tests?

Sure, will do so later.

Tests should be fixed now.


Error in matrix operation

code:

def gumbel_sample(log_probs, temperature=1.0):
    """Gumbel sampling from a categorical distribution."""
    u = numpy.random.uniform(low=1e-6, high=1.0 - 1e-6, size=log_probs.shape)
    g = -np.log(-np.log(u))
    return np.argmax(log_probs + g * temperature, axis=-1)

def predict(num_chars, prefix):
    inp = [ord(c) for c in prefix]
    result = [c for c in prefix]
    max_len = len(prefix) + num_chars
    for _ in range(num_chars):
        cur_inp = np.array(inp + [0] * (max_len - len(inp)))
        outp = model(cur_inp[None, :])  # Add batch dim.
        next_char = gumbel_sample(outp[0, len(inp)])
        inp += [int(next_char)]

        if inp[-1] == 1:
            break  # EOS
        result.append(chr(int(next_char)))

    return "".join(result)

print(predict(32, ""))

Error:
LayerError                                Traceback (most recent call last)
<ipython-input-27-6f9f9afc30e6> in <module>
     22     return "".join(result)
     23
---> 24 print(predict(32, ""))

<ipython-input-27-6f9f9afc30e6> in predict(num_chars, prefix)
     12     for _ in range(num_chars):
     13         cur_inp = np.array(inp + [0] * (max_len - len(inp)))
---> 14         outp = model(cur_inp[None, :])  # Add batch dim.
     15         next_char = gumbel_sample(outp[0, len(inp)])
     16         inp += [int(next_char)]

/opt/conda/lib/python3.7/site-packages/trax/layers/base.py in __call__(self, x, weights, state, rng)
    165       self.state = state  # Needed if the model wasn't fully initialized.
    166     state = self.state
--> 167     outputs, new_state = self.pure_fn(x, weights, state, rng)
    168     self.state = new_state
    169     self.weights = weights

/opt/conda/lib/python3.7/site-packages/trax/layers/base.py in pure_fn(self, x, weights, state, rng, use_cache)
    448       name, trace = self._name, _short_traceback(skip=3)
    449       raise LayerError(name, 'pure_fn',
--> 450                        self._caller, signature(x), trace) from None
    451
    452   def output_signature(self, input_signature):

LayerError: Exception passing through layer Serial (in pure_fn):
  layer created in file [...]/<ipython-input-13-6f7ffe0c061e>, line 21
  layer input shapes: ShapeDtype{shape:(1, 32), dtype:int32}

  File [...]/trax/layers/combinators.py, line 88, in forward
    outputs, s = layer.pure_fn(inputs, w, s, rng, use_cache=True)

LayerError: Exception passing through layer Embedding_256_512 (in pure_fn):
  layer created in file [...]/<ipython-input-13-6f7ffe0c061e>, line 18
  layer input shapes: ShapeDtype{shape:(1, 32), dtype:int32}

  File [...]/trax/layers/core.py, line 150, in forward
    return jnp.take(self.weights, x, axis=0)

  File [...]/jax/numpy/lax_numpy.py, line 3298, in take
    slice_sizes=tuple(slice_sizes))

  File [...]/jax/lax/lax.py, line 835, in gather
    slice_sizes=canonicalize_shape(slice_sizes))

  File [...]/site-packages/jax/core.py, line 273, in bind
    return self.impl(*args, **kwargs)

  File [...]/jax/interpreters/xla.py, line 228, in apply_primitive
    compiled_fun = xla_primitive_callable(prim, *unsafe_map(arg_spec, args), **params)

  File [...]/jax/interpreters/xla.py, line 262, in xla_primitive_callable
    *avals, **params)

  File [...]/jax/interpreters/xla.py, line 320, in primitive_computation
    raise RuntimeError(msg) from e

RuntimeError: Invalid argument: Slice size at index 0 in gather op is out of range, must be within [0, 1), got 1.:
This is a bug in JAX's shape-checking rules; please report it!

This code isn't self-contained (I think you're using a Trax model), so we would need more information to be able to reproduce it. Do you have a Colab notebook? Alternatively, can you find the code for this line:
File [...]/trax/layers/core.py, line 150, in forward
    return jnp.take(self.weights, x, axis=0)

and insert print(f'weights: {self.weights.shape}, x: {x.shape}') before the return statement, then paste the resulting logs?

This is corresponding file For that above code
On Sun, 9 Aug 2020 at 09:56, James Bradbury<EMAIL_ADDRESS>wrote:

This code isn't self-contained (I think you're using a Trax model), so we
would need more information to be able to reproduce it. Do you have a Colab
notebook? Alternatively, can you find the code for this line:
File [...]/trax/layers/core.py, line 150, in forward
return jnp.take(self.weights, x, axis=0)
and insert print(f'weights: {self.weights.shape}, x: {x.shape}') before
the return statement, then paste the resulting logs?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/jax/issues/3995#issuecomment-671004645, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALFGQ3GSMSZIWB7NNY7SIOLR7YQQTANCNFSM4PYIL5MQ
.

hey My problem is solved now and i think this code error arisude due to my
incomplete code so I think it better to close that issue...
and sorry for the inconvenience
On Sun, 9 Aug 2020 at 14:40, Keshav Bajaj<EMAIL_ADDRESS>wrote:

This is corresponding file For that above code
On Sun, 9 Aug 2020 at 09:56, James Bradbury<EMAIL_ADDRESS>wrote:

This code isn't self-contained (I think you're using a Trax model), so we
would need more information to be able to reproduce it. Do you have a Colab
notebook? Alternatively, can you find the code for this line:
File [...]/trax/layers/core.py, line 150, in forward
return jnp.take(self.weights, x, axis=0)
and insert print(f'weights: {self.weights.shape}, x: {x.shape}') before
the return statement, then paste the resulting logs?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/jax/issues/3995#issuecomment-671004645, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALFGQ3GSMSZIWB7NNY7SIOLR7YQQTANCNFSM4PYIL5MQ
.

No worries, we always welcome issues, and also welcome them fixing themselves :)


Test run aborts instead of failing

Description
Test run Aborts instead of failing
Steps to reproduce
https://github.com/sbaid/samples/tree/master/NetCoreCrashTest
Expected behavior
test fails
Actual behavior
test run aborts
Environment
d15prerel latest build

@sbaid
The crash is bound to happen, I am not sure if TP can do anything to prevent it.
Debug.Assert failure in NetCore throws DebugAssertException and fails fast. https://github.com/dotnet/coreclr/blob/d8250b52fc596cc9c13d6abbc5aa85dd1440b0d9/src/mscorlib/src/System/Diagnostics/Debug.Windows.cs
This can be reproed via a console app or even directly writing this code in the test.

Closing this since this is a non issue.


feat: footer newsletter form

:tada: This PR is included in version 1.175.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


intermittent out of gas reverts with swapper

@woodenfurniture had some failures when testing swaps related to OOG errors, it was intermittent and the specific swaps did use significantly more gas than other's he completed (potentially different routes/paths).
https://polygonscan.com/tx/0xb4932512bedbcd006a628a3211c2dd2c76a267639f31d31edd6847fee7483c00
https://polygonscan.com/tx/0xdd2a01b1f48988e1934b127abfd55cf3598d02294d6337600ad13df5f8035638
Either way seems we need to consider how we are estimating gas and resolve the issue here.
These were executed using native wallet

should we consider using the 0x based gasLimit vs estimating against our own node?

Closed by https://github.com/shapeshift/web/pull/6516


chores: removing translate samples which already moved to java-translate

Samples migrated in googleapis/java-translate googleapis/java-translate#253
snippets updated in cl/338718243
once cl/338718243 is submitted, its ready to merge
Please merge this PR for me once it is approved.

do not merge until CL is submitted.


versions:update-properties doesn't update properties to release from snapshot-disabled repos

Given a settings.xml containing a setup generated by Artifactory:
  <profiles>
    <profile>
      <repositories>
        <repository>
          <snapshots>
            <enabled>false</enabled>
          </snapshots>
          <id>central</id>
          <name>libs-release</name>
          <url>http://artifactory.dlogics.com:8081/artifactory/libs-release</url>
        </repository>
        <repository>
          <snapshots />
          <id>snapshots</id>
          <name>libs-snapshot</name>
          <url>http://artifactory.dlogics.com:8081/artifactory/libs-snapshot</url>
        </repository>

the following command:
$ mvn -V -B -N -X versions:update-properties

says stuff like:
[DEBUG] Skipping update check for artifact com.example:product (/Users/kam/.m2/repository/com/example/product/maven-metadata-central.xml) from disabled repository central (http://artifactory.dlogics.com:8081/artifactory/libs-release)
...
[DEBUG] getNewestVersion(): includeSnapshots='false'
[DEBUG] Property ${product.version}: Set of valid available versions is []
[DEBUG] Property ${product.version}: Restricting results to [4.0.0,5.0.0-!)
[DEBUG] lowerBoundArtifactVersion: 4.9.1-SNAPSHOT
[DEBUG] Property ${product.version}: Current winner is: null
[INFO] Property ${product.version}: Leaving unchanged as 4.9.1-SNAPSHOT

I was expecting it to update the version to 4.10.0, which exists in the libs-release repo.
Basically, when not allowing snapshots, it ignores the repository that can't have snapshots in it.
But if I remove the XML that sets snapshots.disabled to false, it works as expected.
Maybe I'm deeply not understanding something, but shouldn't it be looking in the release repo for a release version number?

Someone seems to be having a similar problem as mentioned in this Stack Overflow question.

@datalogics-kam - I noticed the same problem. While I wasn't able to isolate it in the code base, I was able to figure out that it has something to do with where the property is defined. For instance, if the property is defined in a parent pom, "mvn versions:display-property-updates" will correctly look across all of my repositories for the release versions and thus suggest the latest release over the currently configured snapshot version. If, however, the property is defined inside of the same pom file that declares the dependency, I see the behavior you describe and thus it never examines my release repositories.

Phew - This is the same issue that baffled me for over a few days. Need to have a fix for this. Had to follow the workaround as mentioned in the Stack Overflow thread to get it to work for now

This is unfortunate as I wanted to use update-properties as a workaround for #336
Well I guess I’ll have to allow snapshots for the release repositories then…

Please reopen this issue - it is actual.


72 | Edit Distance | C++

Title -
What will change -
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code
[x] Improving Code
[x] Improving Documentation
[x] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[x] Python
[x] C++
[x] Java

Self Check

Ask for issue assignment before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/Ftbj4a
Happy Coding,

I want to work on it.

Hey Can I work On it

hey Can i work on this issue??

[x] Assigned to @ricky1812

i can solve this please assign it to me.


Adding new Scenario in the troubleshooting section

:x: Organization membership check failed
Members of the MicrosoftDocs organization should use the private repo. Please move your changes to https://github.com/MicrosoftDocs/Dynamics-365-Operations, and close this pull request.


⚠ ClamAV has degraded performance

In 6d47d25, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 466 ms

Resolved: ClamAV performance has improved in 92f285e.


Universal Thermal Comfort Index

A new variable of interest would be: Universal Thermal Comfort Index
https://cds.climate.copernicus.eu/cdsapp#!/dataset/derived-utci-historical?tab=overview
As it is, one must download it MANY stages because it is daily data and it exceeds the ECMWF API requests.
It is also an index of interest because it goes beyond the traditional physical variables (temperature, radiation, etc) and can enable interdisciplinary applications of ERA5 data.

Hey @christinenam, @wundersooner and I can incorporate this into our roadmap. For now, if you'd like to get ahead of us, I recommend you use weather-dl from https://github.com/google/weather-tools/ to ingest the data.
How to write config files: https://weather-tools.readthedocs.io/en/latest/Configuration.html
Check out this config for an example of how to ingest this.
https://github.com/google-research/arco-era5/blob/main/raw/era5_sfc_cape.cfg
Happy to advise further.


Add Faroese Keyboard Layout with Custom Characters

Language
Faroese
Description
Generally quite similar to the Danish layout but with some differences to support the Faroese characters
q w e r t y u i o p å
a s d f g h j k l æ ø
z x c v b n m

Long pressing y should give the option to write ý
Long pressing u should give the option to write ú
Long pressing i should give the option to write í
Long pressing o should give the option to write ó
Long pressing a should give the option to write á
Long pressing d should give the option to write ð (Note, the uppercase variant is: Ð, have seen many Android keyboards use the incorrect ð)
Long pressing o should give the option to write ö

Complexity
No, it should be very similar to the Danish layout, but with some additional characters like the ð.

What if we give all letters their own keys? Like this:
  Å Æ Ø Ð Ý Ú Í Ó
Q W E R T Y U I O P
A S D F G H J K L
Z X C V B N M

We can avoid the need for long presses entirely, making typing faster.

Gboard has this layout:
q w e r t y u i o p ð
a s d f g h j k l æ ø
z x c v b n m <blank>

The blank key updates to show a letter with an acute (a -> á) whenever a non-acute letter is pressed. The current layout of FUTO Keyboard does not allow this layout.
Windows has this layout: https://kbdlayout.info/kbdfo
1 2 3 4 5 6 7 8 9 0 + ´
q w e r t y u i o p å ð
a s d f g h j k l æ ø
z x c v b n m

It has an acute accent dead key which is roughly the same concept (but backwards) which is not possible right now in FUTO Keyboard either.
SwifKey doesn't have a Faroese layout, it just has the English QWERTY layout with Faroese letters as long press options.

The Keyboard built in to my GrapheneOS
This keyboard has tap and hold for ð and acute letters.

Simple Keybpard
https://f-droid.org/packages/rkr.simplekeyboard.inputmethod/
This keyboard has tap and hold for ð and acute letters.

iOS
iOS has tap and hold for acute letters, but did give ð a space of its own.

A normal physical Faroese keyboard layout would look something like this:


Add a Newtype instance for Rational

Fixes #23
Would this work for you @natefaubion?

This is fine for me, yes. It does mean that you can construct an arbitrary Rational that may not  be reduced though. Otherwise you'd need to just provide a monomorphic runRational :: Rational -> Ratio Int.

I'm fine with the monomorphic version if you want to keep the reduced invariant.

Good point, let's do that.


add code compatible with IETF language tag

"xx" or "xx-XX" is the IETF language tag standard format, while "xx_xx" is not.
When we detect language with javascript window.navigator.language, it always return IETF standard tag. Adding code compatible with IETF language tag, will avoid conversion every time.

Hi @softwarerero, we need this fix for Wekan, could you please release a new version of  softwarerero:accounts-t9n on atmosphere?

1.1.7 is out

Cool, thank you very much :)


Rough Draft Roadmap

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[X ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ X] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c185bf7b117312bf53cf94--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c18c1f9b858117bfce0798--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c18d560fd51b1b46d257c6--harness-developer.netlify.app


some w3c credential properties should support array or single value

The generic verifiable credential supports pre-defined properties, that comply with the W3C specification.
Some of these properties must support array or single value, but currently support only single value.
e.g.: proof and credentialSchema are such pre-defined properties.

I'm working on switching our DEQAR (generic & EBSI) VC issuer from using the Core API to the Signatory API and have an issue as a result of that, too.
In our case we actually need multi-subject VCs, i.e. where the credentialSubject is an array. As far as I can see, the W3C specification allows that, even if it might be a rare use case.
(In our case, these are joint programme accreditations, joint institutional audits or similar: a single accreditation covering several universities, hence each university would be one subject.)


Get-FileHash

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Get-FileHash
[-InputStream]
[[-Algorithm] ]
[]
Expected behavior
steps: - task: DownloadPipelineArtifact@2 inputs: source: specific project: PowerShellCore pipeline: '696' preferTriggeringPipeline: true runVersion: latestFromBranch runBranch: '$(Build.SourceBranch)' artifact: finalResults patterns: '**/*.nupkg' path: '$(Pipeline.Workspace)/releasePipeline/finalResults' - task: DownloadPipelineArtifact@2 inputs: source: specific project: PowerShellCore pipeline: '696' preferTriggeringPipeline: true runVersion: latestFromBranch runBranch: '$(Build.SourceBranch)' artifact: metadata path: '$(Pipeline.Workspace)/releasePipeline/metadata' - pwsh: | #Exclude all global tool packages. Their names start with 'PowerShell.' $null = New-Item -ItemType Directory -Path "$(Pipeline.Workspace)/release" Copy-Item "$ENV:PIPELINE_WORKSPACE/releasePipeline/finalResults/*.nupkg" -Destination "$(Pipeline.Workspace)/release" -Exclude "PowerShell.*.nupkg" -Force -Verbose $releaseVersion = Get-Content "$ENV:PIPELINE_WORKSPACE/releasePipeline/metadata/release.json" | ConvertFrom-Json | Select-Object -ExpandProperty 'ReleaseVersion' $globalToolPath = "$ENV:PIPELINE_WORKSPACE/releasePipeline/finalResults/PowerShell.$releaseVersion.nupkg" if ($releaseVersion -notlike '*-*') { # Copy the global tool package for stable releases Copy-Item $globalToolPath -Destination "$(Pipeline.Workspace)/release" } Get-ChildItem "$(Pipeline.Workspace)/release" -recurse displayName: Download and capture nupkgs - task: NuGetCommand@2 displayName: 'NuGet push' inputs: command: push packagesToPush: '$(Pipeline.Workspace)/release/*.nupkg' nuGetFeedType: external publishFeedCredentials: PowerShellNuGetOrgPush

Actual behavior
Get-FileHash /etc/apt/sources.list | Format-List

Algorithm : SHA256
Hash      : 3CBCFDDEC145E3382D592266BE193E5BE53443138EE6AB6CA09FF20DF609E268
Path      : /etc/apt/sources.list

Error details
Get-FileHash C:\Users\user1\Downloads\Contoso8_1_ENT.iso -Algorithm SHA384 | Format-List

Algorithm : SHA384
Hash      : 20AB1C2EE19FC96A7C66E33917D191A24E3CE9DAC99DB7C786ACCE31E559144FEAFC695C58E508E2EBBC9D3C96F21FA3
Path      : C:\Users\user1\Downloads\Contoso8_1_ENT.iso

Environment data
$wc = [System.Net.WebClient]::new()
$pkgurl = 'https://github.com/PowerShell/PowerShell/releases/download/v6.2.4/powershell_6.2.4-1.debian.9_amd64.deb'
$publishedHash = '8E28E54D601F0751922DE24632C1E716B4684876255CF82304A9B19E89A9CCAC'
$FileHash = Get-FileHash -InputStream ($wc.OpenRead($pkgurl))
$FileHash.Hash -eq $publishedHash

True

Visuals
$stringAsStream = [System.IO.MemoryStream]::new()
$writer = [System.IO.StreamWriter]::new($stringAsStream)
$writer.write("Hello world")
$writer.Flush()
$stringAsStream.Position = 0
Get-FileHash -InputStream $stringAsStream | Select-Object Hash
Hash
64EC88CA00B268E5BA1A35678A1B5316D212F4F366B2477232534A8AECA37F3C

@PRACH4  you have posted 3 duplicates of this.  But what you show here is Get-FileHash working as designed, and generating the expected hash for a file.
Please explain what the command is doing that you believe to be wrong, or what extra functionality you would like to see.


[5.5] Route pattern method doesn't handles regexes properly

Laravel Version: 5.5.22
PHP Version: PHP 7.1.1
Database Driver & Version: N/A

Description:
The pattern matcher method for global route constraints, as described here, doesn't handles every regex properly. For example, the regex from the example below to validate a proper UUID v4 (which can also be a route parameter) isn't being validated properly. The validation behind the pattern method is done by the Str::is method. See the example in steps to reproduce to reproduce.
Regex for UUID v4:
/[a-f0-9]{8}-?[a-f0-9]{4}-?4[a-f0-9]{3}-?[89ab][a-f0-9]{3}-?[a-f0-9]{12}/i
Valid UUID v4:
5d8977f1-8809-47b0-a01d-492373fd81c6
Steps To Reproduce:
https://3v4l.org/QXph1

Pretty sure Str::is isn't used in route matching. preg_quote within wouldn't allow this to work i believe.
Your regex works fine, use it without the forward slashes and flags.
Route::get('/{id}', function ($id) {
    return 'UUID: ' . $id;
})->where('id', '[a-f0-9]{8}-?[a-f0-9]{4}-?4[a-f0-9]{3}-?[89ab][a-f0-9]{3}-?[a-f0-9]{12}');

Route::pattern('id', '[a-f0-9]{8}-?[a-f0-9]{4}-?4[a-f0-9]{3}-?[89ab][a-f0-9]{3}-?[a-f0-9]{12}');

Both working as expected.

@arjasco hmm no they don't on my end:
>>> str_is('[a-f0-9]{8}-?[a-f0-9]{4}-?4[a-f0-9]{3}-?[89ab][a-f0-9]{3}-?[a-f0-9]{12}', '5d8977f1-8809-47b0-a01d-492373fd81c6')
=> false

What version of Laravel are you running?

I believe Str::is is not used for that case see here - https://laravel.com/docs/5.5/helpers#method-str-is
The preg_quote function within the implementation would escape the pattern.
Might be wrong but i think route matching is the responsibility of the Symfony Route component.

@arjasco hmm, you're right I believe. No idea why I thought it was used for it. Think I was confused with the route naming matching (it's used there and also uses the word patterns).
But unfortunately the problem still happens.
Route::pattern('id', '[a-f0-9]{8}-?[a-f0-9]{4}-?4[a-f0-9]{3}-?[89ab][a-f0-9]{3}-?[a-f0-9]{12}');
This doesn't let's the value 5d8977f1-8809-47b0-a01d-492373fd81c6 pass (while it should).

This works for me:
Route::get('/{id}', function ($id) {
    return 'UUID: ' . $id;
})->where('id', '[a-f0-9]{8}-?[a-f0-9]{4}-?4[a-f0-9]{3}-?[89ab][a-f0-9]{3}-?[a-f0-9]{12}');

Heading to http://laravelcurrent.dev/5d8977f1-8809-47b0-a01d-492373fd81c6 it parses it correctly.

I noticed that it doesn't work when you append flags to the regex.
/[a-f0-9]{8}-?[a-f0-9]{4}-?4[a-f0-9]{3}-?[89ab][a-f0-9]{3}-?[a-f0-9]{12}/i won't work.
[a-f0-9]{8}-?[a-f0-9]{4}-?4[a-f0-9]{3}-?[89ab][a-f0-9]{3}-?[a-f0-9]{12} does but then it doesn't take the flags into account.

This indeed seems to be a limitation for Symfony's Router since it does all the regex checks. So I'll close this off here.


Use relative imports

This is needed on py3k (but it's still compatible with 2.7)

This works perfect! Thank you


Aws4HashCalculator.buildCanonicalRequest uses incorrect payload hash

Aws4HashCalculator.buildCanonicalRequest uses the client x-amz-content-sha256 header instead of reading the request payload and calculating the hash.  This allows the client spoof the payload.

s3 ninja is a testing tool, not a production level data store. Feel free to provide a fix if that issue disturbes you, but keep in mind, that authentication can be disable / by passed (by design) easily.

Closing as no fix was provided :(


Libnd4j fails to compile on armv8 jetson nano

Issue Description
LIbnd4j fails to compile due to the attached screenshot

Please describe our issue, along with:

expected behavior
Compiles
encountered behavior
Does not compile

Version Information
Snapshots
Please indicate relevant versions, including, if relevant:

Deeplearning4j version
Snapshot
Platform information (OS, etc)
Jetson nano Jetpack 4.3.
CUDA version, if used
10.0
NVIDIA driver version, if in use

Additional Information
Where applicable, please also provide:

Full log or exception stack trace (ideally in a Gist: gist.github.com)
pom.xml file or similar (also in a Gist)

Contributing
If you'd like to help us fix the issue by contributing some code, but would
like guidance or help in doing so, please mention it!

Workaround merged.

Confirmed compiles.


FR: UI for editing file extensions

Note: I'm OK with editing configuration files, so I don't need this feature. But the issue tracking this feature disappeared, so I created a new one.

What do you mean with the title? Registering file extensions to open with ecode or something else?

If it's registering file extensions then it's a duplicate of this one. And if it's for registering custom file extensions, as I said here, I won't support it.

Someone doesn't want to edit config.cfg to set which language is used for a file extension, so he asked for a graphical frontend to do so right in ecode. But for unknown reasons, all of his issues disappeared, so I created a new one. This feature is useful for people who don't want to edit configuration files.

For example: someone picked SmallBASIC for .bas extension but latter decided that they want VisualBasic instead.

Oh, OK. It did not disappear, but it was a discussion, not an issue.

Oh, OK. It did not disappear, but it was a discussion, not an issue.

Not my thread. Someone else submitted a FR. I have no problems with editing configuration files.

You're telling me that iahung3 and iahung2 are not the same person? OMG! This is the closest one, I remember talking about it but not having an issue for it. I don't delete issues (unless it's spam or something like that) so if someone created it must be somewhere. Anyway, a solution is needed for what you're requesting so let's keep this FR.

You're telling me that iahung3 and iahung2 are not the same person? OMG! This is the closest one, I remember talking about it but not having an issue for it. I don't delete issues (unless it's spam or something like that) so if someone created it must be somewhere. Anyway, a solution is needed for what you're requesting so let's keep this FR.

Both are my account. But as I recall the original FR is posted by @superchick.


Nothing happens in Safari on iOS

When I try the demo in iOS on Safari, nothing happens. There's nothing in the error console either. It's ok on desktop Safari.

Yeah, that's because Safari haphazardly blocks the AudioContext from running unless it's created from an interaction event.

Right, playing a silent buffer in the first call to play() got things working. Thanks.

Hey hmoffatt, can you show exactly how this works please? I'm not up to speed with HTML5, etc, but would love to get this working on iOs myself. Thanks

@Win10Developer I've put my fix in https://github.com/hmoffatt/metronome

@cwilso right, thanks. I knew Safari muted the sound until it's created from an interaction event but in this case it looks like the AudioContext's currentTime doesn't even start. Playing a dummy buffer before starting the scheduling gets it going.


update do README to reflect current version

Hi,
the README should be updated to reflect the current state.
Cause the setup has changed:

with using of the cds plugin system - the setup in server.js ist not required
the option/setting for the launchpad have moved to cds.env...

regards
Norbert

Hi Norbert,
Documentation is now up to date.
Best regards,
Geert-Jan


feature (v0.2.5): add interface tests

Codecov Report

Merging #30 (6a96d54) into main (95a0e53) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #30   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines          219       219
  Branches        48        48
=========================================
  Hits           219       219

Continue to review full report at Codecov.

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


Less common example

Would be nice to see a way to add controllers at runtime.  As an example if I have a modular UI and I'm allowing internal developers to upload new controllers at runtime.

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

ID: c6e3baf6-6cba-aee0-9a8d-1e1fced2b690
Version Independent ID: 74b76258-36a3-17de-6648-292da0a9e400
Content: Create a web app with ASP.NET Core MVC
Content Source: aspnetcore/tutorials/first-mvc-app/index.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Thanks for contacting us.
We don’t have the resources to invest in this area, so we are closing the issue. Should your request generate enough  👍 responses, we’ll reconsider.


Repo sync for protected CLA branch

The pull request is created from main637922414833281596 to main to fix git push error for protected CLA branch

Microsoft authors of articles need to make updates through the private repository, so the updates can be staged and validated by current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.
It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
 Public repo URL: https://github.com/MicrosoftDocs/{docs-repo}
 Private repo URL: https://github.com/MicrosoftDocs/{docs-repo-pr}
We have closed this pull request. Please resubmit the updates through the private repo. For more information, see the contributor guide.

Docs Build status updates of commit 7034729:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch main637922414833281596 has been deleted or has been force pushed

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 the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 7034729:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch main637922414833281596 has been deleted or has been force pushed

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 the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add Spring Cloud Deployer support

To propagate the trace context via environment variables we would need to ensure that tracing is added to the classpath...
@cppwfs @ilayaperumalg

Done via https://github.com/spring-cloud/spring-cloud-sleuth/commit/0f23f9d52fd7bf453debd8ff2846bc19f845f3bc

Done via https://github.com/spring-cloud/spring-cloud-sleuth/commit/0f23f9d52fd7bf453debd8ff2846bc19f845f3bc


Skip button for update found

What feature or improvement do you think would benefit Files?
When a new update has been found it should be possible to skip the update

With weak or unstable data connection the app can not be used-
Requirements

App can be used, when an update is not possible or wanted

Files Version
<IP_ADDRESS>
Windows Version
Windows 11 23H2
Comments
No response

Thanks for the feedback, This is apart of App installer and not Files so we can not change the functionally there. What happens if you press no?

Than i have just the "Update" button and the close "X". So no way to get into the app.

This is a limitation with the sideload version, to work around this issue, we suggest using the Store version which doesn't have this problem.


Not issue - but a large file performance stat

I have compared
nawk with frawk on a macOS Darwin: i7 with 16GB RAM.
I used a 17GB  .csv file for test using:
frawk 'BEGIN{FS=","};{count[NF]++};END{for(i in count){print "With " i " fields - count is: " count[i]}}' aggregated.csv
results:
frawk - 2 min
nawk - 23 min
Impressive. Thank for the work on frawk.

I have compared nawk with frawk on a macOS Darwin: i7 with 16GB RAM.
I used a 17GB .csv file for test using: frawk 'BEGIN{FS=","};{count[NF]++};END{for(i in count){print "With " i " fields - count is: " count[i]}}' aggregated.csv
results: frawk - 2 min nawk - 23 min
Impressive. Thank for the work on frawk.

Could you also give gawk a try? What does it behave?

gawk - just over 4 mins.

If you compare with gawk (https://www.gnu.org/software/gawk/), test with recent versions (e.g.: 5.2.1).  gawk 4.x.x for example is quite a bit slower.
gawk -b or gawk --characters-as-bytes will also speedup gawk quite a bit, as it does not care about locale encoding.
'mawk` (https://invisible-island.net/mawk/) is in general also quite fast.
frawk still will win most likely.
In case you want to see different awk programs in actions (before some frawk performance bugs were fixed):
https://github.com/ezrosent/frawk/issues/37

Sorry I missed this when you initially posted it (I do try and keep track of issues here, but August was a busy time). Thanks for the kind words (and thanks for the others who've weighed in!)


Test JINJA auto-generated GUIs

Would approximate something like this. Much more friendly to Python developers, as we already use Go-template syntax inside Helm charts and Python. Using React brings problems

should this be closed in favor of the react setup (#536 ) ?
@callumforrester @keithralphs ?

Yes, thanks!


shims: adjust WaitOnAddress usage on Windows

We would previously unconditionally wait indefinitely on the address
rather than waiting for the specified duration.  Adjust the timeout
parameter to the WaitOnAddress to ensure that we get the correct
behaviour.

@swift-ci please test


Added missing angle to JoystickDirection::from_degrees

When going left, only the bottom half of the left works, the  upper half of the left was missing

good catch, thanks for the PR :)


Add whitespaces and erase redundant whitespaces.

There are missing whitespaces after comma in Optional.swift.
I think that this makes it more readable.

Please remove unrelated trailing whitespace changes from the patch.  Thanks.

@gribozavr I removed unrelated changes.

Thanks, could you squash these commits, since there is no useful history in them?

Thanks, could you squash these commits, since there is no useful history in them?

@gribozavr @SkateLiu is it OK now?

@ken0nek Merged, thanks!

:+1:


[BUG] apply-hvst-upgrade-ABCD-prepare-on-NODE-with-EFGH pod displays invalid URL in log

Describe the bug
The log has an extra forward slash within the retry block.
To Reproduce
Steps to reproduce the behavior:

Prepare an upgrade from one version of Harvester to the next
Use something like kubetail to funnel the logs of: kubetail -l upgrade.cattle.io/controller=system-upgrade-controller -n cattle-system
You'll see URL's listed in the logs with an extra forward slash

Expected behavior
The logs to display the correct URL during upgrade.
Environment

Harvester ISO version: v1.1.3-rc3->v1.2.1
Underlying Infrastructure: 3 HP DL160s

Additional context
Example:
[apply-hvst-upgrade-b5dck-prepare-on-v113rc3-node16-with-7-9mmjp] curl: (23) Failure writing output to destination
[apply-hvst-upgrade-b5dck-prepare-on-v113rc3-node16-with-7-9mmjp] + echo 'Failed to download the requested file from "http://upgrade-repo-hvst-upgrade-b5dck.harvester-system/harvester-iso/bundle//rancherd/images/rancherd-bootstrap-images-v1.2.1.txt" to "/host/var/lib/rancher/agent/images/rancherd-bootstrap-images-v1.2.1.txt" with error code: 23, retrying (93)...'
[apply-hvst-upgrade-b5dck-prepare-on-v113rc3-node16-with-7-9mmjp] Failed to download the requested file from "http://upgrade-repo-hvst-upgrade-b5dck.harvester-system/harvester-iso/bundle//rancherd/images/rancherd-bootstrap-images-v1.2.1.txt" to "/host/var/lib/rancher/agent/images/rancherd-bootstrap-images-v1.2.1.txt" with error code: 23, retrying (93)...
[apply-hvst-upgrade-b5dck-prepare-on-v113rc3-node16-with-7-9mmjp] + sleep 10
[apply-hvst-upgrade-b5dck-prepare-on-v113rc3-node16-with-7-9mmjp] + i=94
[apply-hvst-upgrade-b5dck-prepare-on-v113rc3-node16-with-7-9mmjp] + [[ 94 -lt 100 ]]
[apply-hvst-upgrade-b5dck-prepare-on-v113rc3-node16-with-7-9mmjp] + curl -sSfL http://upgrade-repo-hvst-upgrade-b5dck.harvester-system/harvester-iso/bundle//rancherd/images/rancherd-bootstrap-images-v1.2.1.txt -o /host/var/lib/rancher/agent/images/rancherd-bootstrap-images-v1.2.1.txt

The URL: http://upgrade-repo-hvst-upgrade-b5dck.harvester-system/harvester-iso/bundle//rancherd/images/rancherd-bootstrap-images-v1.2.1.txt has an extra "/" forward slash after "bundle/":
Possibly related to something in lib.sh script UPGRADE_REPO_BUNDLE_ROOT="$UPGRADE_REPO_URL/bundle"?

The fix for this was to create the images folder on the host. I found this out by manually running the erroring curl command. The bad url is likely a red herring (error 23 = failure writing output to destination). Error during upgrade from 1.1.3 to 1.2.0. No other versions installed prior.
mkdir /host/var/lib/rancher/agent/images

@tristan-morris any chance we could have the full log from the pod?

Sadly @ibrokethecloud, I think these are long gone unless the logs are stored in a secret place? I didn't have any centralised logging setup then.
The Harvester cluster was fresh / ~1 day old. I installed (accidently, as I grabbed the latest non beta release from Github) harvester-v1.1.3-amd64.iso via some fun ipxe netboot hax as the hosts are remote. I setup a total of three nodes with this image. I then noticed the upgrade button in the UX and realised my install version mistake. I checked everything seemed stable in the cluster, after noticing some comments about split-brain issues. Hosts had "promote complete" so I assume it was happy running with 3 masters. I tested a VM deployed as expected with full networks. Hit the button and saw the apply-hvst-upgrade pod/ "Preload Images" step failed on the first node. Did this twice. Failed twice. Popped a console into the upgrade pod, manually ran the failing curl command. Looked to see what was written in the directory, which didn't exist. Thought I'd shot in the dark and create the directory and it happily continued on. I noticed this failed on each node and just popped a console into the pod and mkdir.
Apart from the non-standard PXE boot, I think these nodes may have had patchy Internet due to some other issues. The non-standard boot was remote redirect of an iPXE iso, and then chainloading harvester.
This was chain loaded via iPXE with chain http://<IP_ADDRESS>:3000/boot.txt. This is from a newer node which got the 1.2.1 version.
#!ipxe
kernel http://<IP_ADDRESS>:3000/harvester-v1.2.1-vmlinuz-amd64 cdroot rd.live.dir=/ ip=dhcp net.ifnames=1 rd.cos.disable rd.noverifyssl console=tty1 root=live:http://<IP_ADDRESS>:3000/harvester-v1.2.1-rootfs-amd64.squashfs harvester.install.automatic=false harvester.install.iso_url=http://<IP_ADDRESS>:3000/harvester-v1.2.1-amd64.iso

initrd http://<IP_ADDRESS>:3000/harvester-v1.2.1-initrd-amd64
boot

Hope something here helps, I assume it's not just a missing directory creation command in the upgrade script.

the directory creation is during the install  and is where the images are loaded during first install and subsequently during upgrades. The directory should have existed during the upgrade as well.

Looking at the remote host, it seems a bit empty but not sure what's impacted by the immutable stuff.

I experienced the absence of /var/lib/rancher/agent/images on a freshly installed v1.3.1 cluster, while attempting the update to v1.3.2.
This resulted in the "Images preloading"-phase hanging indefinitely until creating the directory manually on the first node that was trying to preload the images.
The subsequent cluster nodes did not show this issue.


Remove reply button when allow replies is false

When allow reply is false I feel like the reply button should not be shown.

Sorry for this one too, I didn't understand the doc properly. I thought that allowReplies needed to be on the server side when I read the doc. If you want, making it clearer could be interesting.


Unusual max_local_storage_nodes value

https://github.com/opensearch-project/opensearch-devops/blob/a2a2d8be7438a8dc5cbb300a8530f85a3784fd20/Helm/opensearch/values.yaml#L67-L68
Should these lines be present in the default values.yaml of the opensearch chart? According to the elastic documentation setting this to >1 is not safe, and not sure if it makes sense in a kubernetes statefulset.

@smlx The value is just a demo configuration as mentioned in the values.yaml and does not portray anything.

Ok, I'll send a PR to remove it then because as a demo setting it doesn't make sense for a cluster running as a kubernetes statefulset.


Attempt to smooth out the audio...

@raysan5 Here's a video of what the updated audio buffer code sounds like....
https://files.robloach.net/s/iafD95PLonBgJpL/download

@RobLoach I see, I had this same issue before... could you try increasing bufferSize instead of 60? I think samples are consumed before the next frame is called for refilling, so the choppy sound... but probably you should provide more audio frames for refilling, like twice the provided ones

Didn't help, unfortunately. Played around with a bunch of different sizes for the buffer, and it still resulted in choppy sounds.... I think the best would be an internal buffer, along with a way to make Raylib queue from the internal buffer. Not quite sure how to accomplish that though...
In SDL, it's SDL_QueueAudio().


Backend issue

1.even if i ma logged in buying anuthing asks me tolog in again and upon typing the correct password it says wrong passowrd.
2. the fabric filter of cotton linen velvet does not work
3. transition from one page to another might have minor issue please recheck

Log in issue is solved. Try now


[TACHYON-786] Login Mechanism: Add CUSTOM_LOGIN (by JAAS framework)

JIRA: TACHYON-786
In production use of Tachyon, we want different applications to have separate user identities, even if they are running under same operating system user. We should allow this by letting the application to specify their username, through Java configuration parameter. This JIRA enables that in the SIMPLE login mechanism. If tachyon.security.username is specified by the application and is not empty, the login modules uses tachyon.security.username as the username; otherwise, we fall back to OS_SPECIFIC_LOGIN.

@dongche @shenguoquan @apc999 Could you review this PR? Thanks.

Discussed with @apc999 , we will use System.getProperty() for now and move to TachyonConf after TachyonConf is refactored to Singleton.

@ooq LGTM. Thanks for figuring out this problem, and the solution is good.
Only one minor comments is also marking the System.setProperty in the test as TODO, in case missing updating this test part in the future work. Since we already have TODO in the code, this is not a big deal. Up to you.

@ooq , thanks for the patch. I left a few minor comments. It looks in general good.

@apc999 @dongche Thanks for the feedback and apologize for the delay.
I have addressed all the issues. For @apc999 's comment on checking the login user against the OS user, I am now using System.getProperty("user.name") to retrieve the latter, let me know if that would not work.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/4421/
Test PASSed.

LGTM +1. Thanks @ooq

@ooq Nice idea. I think it is ok to use that system property. The user.name property should have no chance to be changed in test execution.
Original getSimpleLoginUserTest could be also checked like this. Could you help to update it in this PR? Thanks.

@dongche Good thought on updating the simple login tests. But given these are code logic changes that are not related to this PR, we should probably update it in separate PR.

@ooq Sure, I will open another PR for this.
LGTM. Thanks.

+1 LGTM


Opening a user history/popup will remove notification for going live

Describe the bug
When clicking on a user to open chat history/user profile the twitch notification settings will disable.
ie. you follow Summit1g and have notifications for him going live enabled the notification settings will be disabled for his channel when you open said popup.
To reproduce
Follow a streamer and enable notifications.
Check https://www.twitch.tv/settings/notifications -> "Per Channel" if it is enabled
Open popup within Chatterino and go refresh twitch settings (F5) notifications are now disabled for said user
Screenshots

Chatterino version
Chatterino 2.2.2-fix (commit a82f57f94)
Operating system
Windows 10

Can reproduce on commit https://github.com/Chatterino/chatterino2/commit/a82f57f94387c8bfc7b8826216fad1253be7cb40 but not on master
Can you try a nightly build (https://github.com/Chatterino/chatterino2/releases/tag/nightly-build) and see if it still happens?

does not happen on nightlies in my experience

Can confirm, seems to be fixed with the nightly.

Just for short explanation, this was caused due to an old bug which caused re-following someone every time you open a usercard of a user you're already following (the tickbox updating triggered the follow event, which, by default, was a follow with no notifications). Got fixed along while migrating block methods in #2370

Since this issue is fixed with the next release, i'm closing this issue.
Thanks for the explanation @zneix


Remove defaultValue from Stream.{firstWhere, lastWhere}

Tracking issue for tasks associated with removing the defaultValue parameter from Stream.firstWhere and Stream.lastWhere.
TODO

[x] Deprecate
[x] Fixup package:async:

[x] Fixes: https://github.com/dart-lang/async/pull/54
[x] Roll to SDK
[x] Roll internal

[x] Fixup package:shuttlecock

[x] Roll internal

[x] Fix additional packages (add here)
[x] Roll packages (add here)
[x] Fix internal
[x] Land in SDK

[x] Follow up on breaking change
[x] Roll flutter
[x] Roll internal

cc @lrhn @keertip @srawlins @anders-sandholm @vsmenon @dgrove

Internal code is clean, @lrhn the change can be landed in the SDK as soon as you're ready.

https://dart-review.googlesource.com/c/sdk/+/43420 has landed, and has rolled to Flutter. Has not yet rolled to internal.

internal has landed as well.
@leafpetersen can this be closed?