net: TestVariousDeadlines failing on plan9/386 builder

TestVariousDeadlines is regularly failing on the plan9/386 builder.
--- FAIL: TestVariousDeadlines (6.21s)
    timeout_test.go:880: 1ns run 1/1
    timeout_test.go:905: for 1ns run 1/1, good client timeout after 81.296µs, reading 0 bytes
    timeout_test.go:915: for 1ns run 1/1, server in 3.50244ms wrote 458752: readfrom tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:62762: write tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:62762: write /net/tcp/166/data: Hangup
    timeout_test.go:880: 2ns run 1/1
    timeout_test.go:905: for 2ns run 1/1, good client timeout after 56.991µs, reading 0 bytes
    timeout_test.go:915: for 2ns run 1/1, server in 7.46919ms wrote 1048576: readfrom tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:38454: write tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:38454: write /net/tcp/167/data: Hangup
    timeout_test.go:880: 5ns run 1/1
    timeout_test.go:905: for 5ns run 1/1, good client timeout after 58.667µs, reading 0 bytes
    timeout_test.go:915: for 5ns run 1/1, server in 7.167473ms wrote 1048576: readfrom tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:59766: write tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:59766: write /net/tcp/168/data: Hangup
    timeout_test.go:880: 50ns run 1/1
    timeout_test.go:905: for 50ns run 1/1, good client timeout after 56.153µs, reading 0 bytes
    timeout_test.go:915: for 50ns run 1/1, server in 4.392508ms wrote 524288: readfrom tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:62830: write tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:62830: write /net/tcp/169/data: Hangup
    timeout_test.go:880: 100ns run 1/1
    timeout_test.go:905: for 100ns run 1/1, good client timeout after 56.153µs, reading 0 bytes
    timeout_test.go:915: for 100ns run 1/1, server in 4.517385ms wrote 622592: readfrom tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:50076: write tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:50076: write /net/tcp/170/data: Hangup
    timeout_test.go:880: 200ns run 1/1
    timeout_test.go:905: for 200ns run 1/1, good client timeout after 57.829µs, reading 0 bytes
    timeout_test.go:915: for 200ns run 1/1, server in 3.538478ms wrote 458752: readfrom tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:44015: write tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:44015: write /net/tcp/171/data: Hangup
    timeout_test.go:880: 500ns run 1/1
    timeout_test.go:905: for 500ns run 1/1, good client timeout after 64.534µs, reading 0 bytes
    timeout_test.go:915: for 500ns run 1/1, server in 3.929036ms wrote 524288: readfrom tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:50845: write tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:50845: write /net/tcp/172/data: Hangup
    timeout_test.go:880: 750ns run 1/1
    timeout_test.go:905: for 750ns run 1/1, good client timeout after 56.991µs, reading 0 bytes
    timeout_test.go:915: for 750ns run 1/1, server in 7.222787ms wrote 1048576: readfrom tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:54489: write tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:54489: write /net/tcp/173/data: Hangup
    timeout_test.go:880: 1µs run 1/1
    timeout_test.go:905: for 1µs run 1/1, good client timeout after 56.991µs, reading 0 bytes
    timeout_test.go:915: for 1µs run 1/1, server in 4.337192ms wrote 557056: readfrom tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:55578: write tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:55578: write /net/tcp/174/data: Hangup
    timeout_test.go:880: 5µs run 1/1
    timeout_test.go:905: for 5µs run 1/1, good client timeout after 59.506µs, reading 0 bytes
    timeout_test.go:915: for 5µs run 1/1, server in 3.603013ms wrote 458752: readfrom tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:35569: write tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:35569: write /net/tcp/175/data: Hangup
    timeout_test.go:880: 25µs run 1/1
    timeout_test.go:905: for 25µs run 1/1, good client timeout after 57.829µs, reading 0 bytes
    timeout_test.go:915: for 25µs run 1/1, server in 2.280483ms wrote 294912: readfrom tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:58149: write tcp <IP_ADDRESS>:43229-><IP_ADDRESS>:58149: write /net/tcp/176/data: Hangup
    timeout_test.go:880: 250µs run 1/1
    timeout_test.go:910: for 250µs run 1/1, timeout (5s) waiting for client to timeout (250µs) reading
FAIL
FAIL	net	30.849s

See https://build.golang.org/log/4b9709667343395dcd2d93c1447461e4fb807a33
This is related to issue #19519.
@millerresearch

Re-opening so we can confirm that the flakiness gets fixed after the change is deployed.

An update here. I've looked over the plan9-386 failures over the last 2 days:

https://build.golang.org/log/eb6fd6a9e522e232ede78ce0bd75fb114f84ecc3
https://build.golang.org/log/4535350c4d4ada456799d694b22e6b01caa465ee
https://build.golang.org/log/aad3df6203098849aee1d9ddd2c414a38c00c7b5
https://build.golang.org/log/1b0397c388f1b47a4989f5b8bec6ce5ab2f2eea0
https://build.golang.org/log/2403454e2cab9ba6454d5a9b96b9be4819c6a6d2
https://build.golang.org/log/7359f2eca19de70865cdd9a567cea7e7c1eaac78
https://build.golang.org/log/13695e48cb873b13cdcd1072e5715f53d71462fe
https://build.golang.org/log/91d65a04e89b93cb6c8285d93c75f7ec56709b9c
https://build.golang.org/log/edc7973e321a709f9c803f31c5682b673c9e90ec
https://build.golang.org/log/0be11115a5b3086dcde291d7afc2a8a32c6d0ee4
https://build.golang.org/log/a73d70779dbdcff7f12e1d3ff95354cb03a58649

Out of the 11 builds that failed, 5 were due to TestVariousDeadlines failing with the same "Hangup" errors. It seems this flakiness issue isn't completely resolved yet.


Referred var reagent.impl.batching/do-later does not exist

Im getting the following error when trying to run a very minimal re-frame app. Not sure where this error is coming from, why are we trying to find do-later in batching? There is no such def in batching...
clojure
Compiling "resources/public/js/compiled/hello_world.js" from ["src-cljs"]...
{:tag :cljs/analysis-error}
ANALYSIS ERROR: Referred var reagent.impl.batching/do-later does not exist on file null, line null, column null
clojure
Im using the following versions:
[reagent "0.5.1"]
[re-frame "0.6.0"]

You have two choices in terms of the dependencies you use (in project.clj)

[reagent "0.5.1"] goes with [re-frame "0.6.0"]
[reagent "0.6.0-alpha"]  goes with [re-frame "0.7.0-alpha"]

The first option is easiest, and probably preferred.
The second option is more complicated because [re-frame "0.7.0-alpha"] is not available in clojars yet.  You'll have to check out master from this repo and lein install it into your local maven repo.

Ok thanks. Works now


“Suppress“ JPG when DNG exists - why? Considered harmful.

Hello, I've upgraded to the latest version of GeoTag and immediately had to downgrade again. There is a new feature to “suppress“ working on a JPG file when a DNG with the same name exists. I believed this to be a bug, until I found out it's intentionally.
I don't see a use or reason in this? I kindly ask to make this configurable.
I am using an Android Phone with multiple camera apps. Everyone of those apps lets me decide which format I want to use, JPG or DNG or BOTH. I am using “BOTH“ everywhere, because for daily quick usage the JPGs are more handy, while having the DNGs is nice in some rare cases, for when I take a photo Elvis entering an UFO I'll happily sell the DNG to the newspaper, but usually I don't work with DNGs.
Additionally: Some Android Cameras do NOT use the same filename here. They use the “current“ timestamp for both files individually, so maybe one has 120000.jpg and 120001.dng , I have that alot in my collection.
Long story said short: This creates a mess for me, because I intentionally want both files, the JPG is even the usual “master“, plus additionally the mapping per filename is totally not reliable.
I think someone had the idea people would take photos as DNG only and then “develop“ them as JPGs later, but this seems not to be a realistic case. Nobody will i.e. attach a DNG to his WhatsApp message, so JPG is the daily driver here.
(It's not a real workaround to edit JPG and DNG separately, because this would make it painful to make the identical correction on both files)
Would be great if you could add a config option here.

OK, I still don't get what might be the advantage of that feature, for me it just breaks normal workflows.
However, maybe others see that as a problem also, so, this is what I do to work around it:

Get a tool like “NameChanger.app“ from MRR Software
rename all dng files with a schema like “Replace: .dng with .dng“, now your DNG files have an additional "" at ther end of the name
Work with GeoTag like before
rename all dng files the other way around, from _.dng to .dng

…and please, I would really like a config checkbox for this. Why would I hide half of my photos from tagging them?

Changing the way it current works is on the to do list.  Adding a check box is one of the possible changes.
An issue: if each representation of an image (dng and jpg) are treated separately then each can have a different location. In that situation at least one (and maybe both) images are incorrectly located.  Switching between two versions of the same image and having the map jump around drives those with a touch of OCD crazy.
My personal solution is to never shoot jpg.  I know that doesn’t work for everyone.  In your "attach to a WhatsApp message" scenario I'd crop and edit the DNG image to taste, export the edited image as a jpeg in the appropriate size, then attach that image.  No, it's not quick. And not something I do very often.  An image attached to a message is most likely to be a quick snapshots taken with my phone.
Anyway, next release (don't know when that will be, yet) will have changes in the way DNG/JPG images are handled; hopefully changes that meet your need.

Thanks a lot! That would help so much!

The branch GeoTag5 contains what will be the next release when I finish testing and updating the documentation.   It has a user option to ignore the jpeg part of raw/jpeg pairs.  When disabled the file is listed, but not selectable or editable.  I hope that meets your needs.


Added linters

Added linters for following code style of this project

Hi, @OneLiL05 The deployed preview looks a lot different from the current site. I am unable to figure out the reason can you help me to find the solution.

OMG, now I fix this problem

ok

Have you see my proposal about site counter hit counter?

@CoderSaty, I have solved all problems


why each host must have a default IPv6 route

Summary
I have an ipv6-only cluster running on a single node. Without default gw the calico-kube-controller is always in container creating state.
Current Behavior
sudo kubectl get svc
kubernetes   ClusterIP   fd20::1              443/TCP   43m
sudo kubectl get nodes -o wide
ubuntu161   Ready    master   62m   v1.17.2   2001:db8:0:f101::4           Ubuntu 16.04.2
LTS   4.4.0-62-generic   docker://18.9.7
The communication to my cluster IP from my node is  failing ( connection refused)
curl -6 https://[fd20::1]:443
But if I add dummy default route in the system as shown below, everything is working as expected
sudo /sbin/route -A inet6 add default gw 2001:0db8:0:f101::1
The calico document for "enbaling ipv6 support"   says, each host must have a default IPv6 route. Can some one make me understand, why we require default gw ?

Your Environment

<!--- Include as many relevant details about the environment you experienced the bug in
Calico version : v3.10.2
Orchestrator version (e.g. kubernetes, mesos, rkt): v 1.17.2
Operating System and version: ubuntu 16.04
Link to your project (optional):
</li>

@etavene Is the dummy gw pointing at an actual route or just "some" IP? I'm not sure this is a calico dependency or a lower level/Linux requirement for IPv6 networking. It may have something to do with the fact that cluster IPs are NAT'ed.

@rafaelvanoni  Thanks for your reply. It is just some IP.
The reason for asking above question is, the calico documentation for ipv6 support (https://docs.projectcalico.org/v3.10/networking/ipv6)  says that each host must have default ipv6 route. Would like to know why calico expects to have default route ?
Each Kubernetes host must have an IPv6 address that is reachable from the other hosts.
Each host must have the sysctl setting net.ipv6.conf.all.forwarding setting it to 1. This ensures both Kubernetes service traffic and Calico traffic is forwarded appropriately.
Each host must have a default IPv6 route.

@etavene I believe this is because it is a requirement for the host to do neighbor discovery proxying for the pod. If the host doesn't think it can route the traffic, it won't proxy neighbor discovery requests.

I found same issue in my pure ipv6 cluster, but Dualstack don't have such issue, error as below:
  Normal   Scheduled               52m                    default-scheduler        Successfully assigned kube-system/calico-kube-controllers-65f8bc95db-gd7lv to vm-0-67-ubuntu
  Warning  FailedCreatePodSandBox  52m                    kubelet, vm-0-67-ubuntu  Failed to create pod sandbox: rpc error: code = Unknown desc = [failed to set up sandbox container "0aa422d50bfd0039862b7888e9ceb2f89a6dc85c7bf020775a4c85ce63bbacc7" network for pod "calico-kube-controllers-65f8bc95db-gd7lv": networkPlugin cni failed to set up pod "calico-kube-controllers-65f8bc95db-gd7lv_kube-system" network: error getting ClusterInformation: Get https://[fd00:2::1:0:0:1]:443/apis/crd.projectcalico.org/v1/clusterinformations/default: dial tcp [fd00:2::1:0:0:1]:443: connect: connection refused, failed to clean up sandbox container "0aa422d50bfd0039862b7888e9ceb2f89a6dc85c7bf020775a4c85ce63bbacc7" network for pod "calico-kube-controllers-65f8bc95db-gd7lv": networkPlugin cni failed to teardown pod "calico-kube-controllers-65f8bc95db-gd7lv_kube-system" network: error getting ClusterInformation: Get https://[fd00:2::1:0:0:1]:443/apis/crd.projectcalico.org/v1/clusterinformations/default: dial tcp [fd00:2::1:0:0:1]:443: connect: connection refused]
  Normal   SandboxChanged

root@VM-0-67-ubuntu:~# curl -6 https://[fd00:2::1:0:0:1]:443
curl: (7) Failed to connect to fd00:2::1:0:0:1 port 443: Connection refused
root@VM-0-67-ubuntu:~#

``


cargo-deny: 0.14.2 -> 0.14.3

Description of changes
Changelog: https://github.com/EmbarkStudios/cargo-deny/releases/tag/0.14.3

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, 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

[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

cargo-deny


Revert "Add assert that the root widget has been attached."

Reverts flutter/flutter#32437

/tbr @gspencergoog


document luarocks dependency issue workaround

Related to #909

I would rather not document such setup-specific issues in the README. The first reason being this issue never surfaced before, so I believe it is a rare edge-case you encountered, and the second one being that people can search the GitHub issues if they face a problem and will stumble upon your fix.
The README is already quite long as it is to add a troubleshooting section as well.


Add Particle Finance token

[x] I will not ping the Discord about this pull request.

Token Address: B3Ggjjj3QargPkFTAJiR6BaD8CWKFUaWRXGcDQ1nyeeD
Token Name: PARTI
Token Symbol: PARTI
Logo: https://raw.githubusercontent.com/partifinance/particle-brand/main/particle-finance-logo.png
Link to the official homepage of token: https://parti.finance
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Super great ia project

Wonderful, keep healthy and spirit sir 🔥❤

The Excellent Project,i love and always support

lesgooo

Buy now bro, is real and excellent projects good, to the moon
#PARTI @ParticleFinance

Go $PARTI

Buy now Parti and feel like you have tokens with amazing projects

Lovely project

Not received token ,good luck and solid Community

This is amazing

Good project. Sufficient suppy. NeW ATH WILL COME.

I never lost hope on this project  I will hold  till lambo


[Accessibility]A11y_AzureDataStudio_TextTab__Keyboard :"Toggle More " menu item is not accessible through keyboard.

Check out Keros! - Identify accessibility bugs before check-in and make bug fixing faster and easier.
Environment Details
Application name: Azure Data Studio (Agent, Create External Data, Notebooks)
Version: 1.8.0-insider (user setup)
Electron: 3.1.8
Node.js: 10.2.0
V8: 6.6.346.32
OS: MacOS Mojave 10.14.5
Additional Detail:
Tool Name: Keyboard
MAS Reference: MAS2.1.1
Steps to Reproduce :

Install the app Azure Data Studio.
Connect to the Database->Go to file->New Notebook.
Select query->Click on the Run command->click on text tab
Try to navigate “Toggle More ” menu item using keyboard.

Actual:
“Toggle More ” menu item is not accessible through keyboard.
Similar issue is observed for the below pages:
1.NotebookQueryTable: Navigate to the connected server->Select the database->Navigate to the  file->New Notebook->Select any query->Click on the Run command
Expected:
“Toggle More ” menu item should be accessible through keyboard.
User Impact:
User with disabilities find it difficult to navigate to “Toggle More” item list.
Recommendation:
Please refer the following snippet link for fixes:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx
MAS Reference:
MAS2.1.1:
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={8492c4eb-c179-40ae-8777-cd044ed725a2}
Attachment for Reference:
5717_[Accessibility]A11y_AzureDataStudio_TextTab__Keyboard.pptx
Does this issue occur when all extensions are disabled?: Yes

This is a dupe of #5719.

Verified the bug in
Version: 1.8.0-insider (user setup)
Electron: 3.1.8
Node.js: 10.2.0
V8: 6.6.346.32
OS : MacOS Mojave 10.14.5
and still it is repro, hence reopened the bug.
5717_[Accessibility]A11y_AzureDataStudio_TextTab__Keyboard.pptx


FAPTOS meme - add to COIN REGISTRY

Hey there!
Dev of FAPTOS here.
I would like to request for a token registration of our meme that has been launched 2 days ago.
If you wanna confirm https://x.com/faptosonaptos1 you cross check here on our official X/Twitter page.
we also have website created here : https://www.faptosaptos.com/
Thank you.

conflicts

Conflicts resolved.

Merged here https://github.com/pontem-network/coins-registry/pull/145


Add analytics widget in profile page

There's a widget in lib/widgets/analytics.dart,  we need to add it in profile page.
For reference you can see screen 2 in link

I would like to work on it!

I would like to work on it!


Announce to several chats

I often want to announce to Trade, World, LFG, and General chat, but the addon can only announce to one chat at the time. It's not so bad when you get to the higher levels where everyone joined the world chat, but the first 30 levels people tend not to do that (it seems).

The chat posting might be disabled on blizzards end in a future build, they did so on retail, still waiting on confirmation on it.


✨ Update appVersion to 0.22.0 and eliminate server volume

This PR bumps the chart to use kcp v0.22.0, which felt like a minor version bump for the chart version, hence the bump to v0.5.0 as well.
In addition, this PR picks up on some goodies we added in the new kcp release:

Since generating the admin.kubeconfig and associated files is now hidden in the admin battery (which is not enabled in  the batteries of this helm chart), we can replace the PVC with an emptyDir. The kcp binary still complains if /etc/kcp/config does not exist, but it creates nothing in that directory. So I put a dummy volume with a size limit of 50Mi in there.
This unlocks scaling the server deployment past the single replica we have been stuck with (if you also enable leader election, which I plan to incorporate into the Helm chart as a follow up; also the current audit logging configuraton is problematic). So now we can make the replicas and deployment strategy configurable.

/lgtm
/approve


Character should accept only assigned unicode character

According to unicode standard, unassigned code point should can be used because the compatibility with future version of unicode.


UIINREACH-92: INN-Reach Staff Interface: INN-Reach Transaction Detail View Action Menu (Local hold) - Transfer hold to another item

Purpose
Transfer local hold to another item
Approach

Transfer local hold to another item has been added.
Added tests to handle the Transfer local hold to another item action.
Transfer hold and cancel hold have been minimized.

Ref
UIINREACH-92

yarn run v1.22.18
$ eslint ./ && stylelint "src/**/*.css"
/home/jenkins/workspace/folio-org_ui-inn-reach_PR-205/project/src/components/transaction/TransactionDetails/components/ActionMenu/components/LocalActions/LocalActions.js
6:8  error  Strings must use singlequote  quotes
✖ 1 problem (1 error, 0 warnings)
1 error and 0 warnings potentially fixable with the --fix option.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.


[PoC] JTAG support

Now the JTAG driver uses SPI for one specific case: long byte-wise transfers. This includes:

JTAG chain scans
ECP5 SRAM programming
ECP5 flash programming
ECP5 flash readout

In all the other cases the bitbang mode is used.
With ecpdap and --freq 50000 it uses 27MHz TCK frequency and SRAM programming speed is about 1MByte/s.


Updated vulkan physical and logical devices working.

Refactored and implemented physical and logical devices that work. There are still work that needs to be done with the swapchain. So here is working physical and logical devices implementations

Delete spdLog::log(criticals) not up to date for edge cases.
Good job!


Individual Availability Tests in the dashboard not possible (vs. Overall)?

Isn't it possible to display individual availability tests in the Dashboard, below the diagram?
I see Name and RunLocation properties are set, so why would the availability result be merged into the "Overall" and not displayed separately?

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

ID: 6be60b15-5e41-4a0f-41a2-a592afc4ff45
Version Independent ID: aa652576-5e64-2cbf-e556-ca9c7fd91b83
Content: Create and run custom availability tests using Azure Functions - Azure Monitor
Content Source: articles/azure-monitor/app/availability-azure-functions.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @morgangrobin
Microsoft Alias: mogrobin

@deyanp Thanks for the question! We're investigating this and will get back to you shortly.

@deyanp if you create the test via the portal UI you are able to scope the view to the individual test. Tests created via TrackAvailability() do not currently allow this functionality and thus all test results are displayed under overall. So what you are seeing is currently expected behavior.
@zakimaksyutov would be able to comment on if there are any plans to change this behavior in the future.

Also adding @cartersocha

@mrbullwinkle Thanks for the clarification! Will wait for @zakimaksyutov and @cartersocha to provide further updates, if any.

@deyanp I stand corrected, when last I had tested this functionality everything was grouped under overall, but the engineering team has since changed the behavior in the past three months. I just tested to confirm and the experience now looks as follows when you create a custom tests with TrackAvailability():

I've added an update to the doc to reflect the new UI experience. This should merge/go-live tomorrow between 10-11 AM (UTC-7).

@mrbullwinkle Thanks a lot for the updates!
@deyanp Hope this addresses your question. We will now proceed to close this thread as resolved, but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@mrbullwinkle @BhargaviAnnadevara-MSFT , thanks a lot!


Next release

I am swamped with the other work, but I have awesome stuff to share. Hope I will have time soon.
Also, I am waiting for new Formik, hopefully we will be able to leverage it.

And createFormErrors! Both static and dynamic! useMutation then will need only input and payload. Super easy!


3.0.1 release

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 8 committers have signed the CLA.:white_check_mark: PratheepV:white_check_mark: shabarish-testsigma:white_check_mark: rashinarasheeed:x: KR:x: vijayvulchi:x: vikram-chaitanya:x: praveenatest:x: rukmangadaKR seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[ZEPPELIN-2471] - NotebookTest fails on travis

What is this PR for?
The stacktrace for this issue gives the error:
06:31:15,263 ERROR org.apache.zeppelin.notebook.repo.VFSNotebookRepo:151 - Can't read note file:///tmp/ZeppelinLTest_1495261875233/notebook/2BQA35CJZ
com.google.gson.JsonSyntaxException: 2016-03-29T16:21:09-0700

This issue is related to notebooks failing to load with date ParseException and  was fixed in ZEPPELIN-1129. However, in ZEPPELIN-2395 the fix for old format is excluded and hence the issue occurs again. Added the same code to fix the old date format issue in the fromJSON method of Note.java
What type of PR is it?
Bug Fix
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2471
How should this be tested?
Tests should pass on CI
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

There's one test failure - could you take a look https://travis-ci.org/pravin-dsilva/zeppelin/jobs/235945509

The failure that i see is
test(org.apache.zeppelin.interpreter.InterpreterOutputChangeWatcherTest)  Time elapsed: 2.034 sec  <<< FAILURE!
java.lang.AssertionError: expected:<1> but was:<2>
	at org.junit.Assert.fail(Assert.java:88)
	at org.junit.Assert.failNotEquals(Assert.java:834)
	at org.junit.Assert.assertEquals(Assert.java:645)
	at org.junit.Assert.assertEquals(Assert.java:631)
	at org.apache.zeppelin.interpreter.InterpreterOutputChangeWatcherTest.test(InterpreterOutputChangeWatcherTest.java:95)

This doesnt seem related to my change, but could anyone confirm this?

Thanks @pravin-dsilva for the fix.
LGTM and merge to master if no further comment.


appear garbled characters when recorize chinese

IN ORDER TO ASSIST YOU, PLEASE PROVIDE THE FOLLOWING:

Speech SDK log taken from a run that exhibits the reported issue.
See instructions on how to take logs.

A stripped down, simplified version of your source code that exhibits the issue. Or, preferably, try to reproduce the problem with one of the public samples in this repository (or a minimally modified version of it), and share the code.

If relevant, a WAV file of your input audio.

Additional information as shown below

Describe the bug
A clear and concise description of what the bug is. If things are not working as you expect,
describe exactly what you are getting and why that is not what you expect.
For example, speech recognition "does not work" may mean you got a cancellation
event with a particular error message, or you did not get any recognition events,
or the recognition result you got contains text that does not match what was spoken.
To Reproduce
this->_recognizer->Recognized.Connect([](const SpeechRecognitionEventArgs& e) {
if (e.Result->Reason == ResultReason::RecognizedKeyword) {
Logger::info(" RecognizedKeyword:" + e.Result->Text);
} else if (e.Result->Reason == ResultReason::RecognizedSpeech) {
Logger::info(" RecognizedSpeech:" + e.Result->Text);
} else if (e.Result->Reason == ResultReason::NoMatch) {
Logger::info(" NoMatch");
}
});
Steps to reproduce the behavior:

... speck chinese

Expected behavior
A clear and concise description of what you expected to happen.
Version of the Cognitive Services Speech SDK
1.41.1
Which version of the SDK are you using.
Platform, Operating System, and Programming Language

OS: Windows
Hardware - x64
Programming language: C++

Additional context

Error messages, stack trace, ...
Any additional information.

another question: The recognition speed is extremely slow. It takes about 20 seconds.


Rails social media website completed

This PR includes:

Deploy application to Heroku
Styling  the friendship pages with Bulma
Final touches to the README.md file

closes #4, #7

Hello @stratospherique :wave:
Thanks for your review :pray:
We are going to merge this branch then into the development branch and then open a new pull request to merge it to master. :+1: :raised_hands:


broad try/except blocks makes it hard to handle exceptions

There is a wide-use of a Python exception handling anti-pattern in the code base. This anti-pattern looks like this:
try:
    logger.info("some info message")
    db.query.images.get(id=1)
except Exception:
    logger.exception("error encountered when querying")

This is problematic in many different ways:

The try part has usually more than one call and can have many lines. A logger.info will probably never cause an exception, and should not be caught. Catching exceptions should be done as targeted as possible.
There should be a good reason to catch every single exception possible, and in this case (as it is with most cases in the code base) there isn't one. Exception catching should be as granular as possible - not generic and broad. This allows flow control by callers by catching individual exceptions.
It is impossible to test this properly because the exception does not bubble up
In most cases where this pattern is found, parent callers of the function are also doing the same catching which doesn't make sense for the apparent goal of producing a message when any exception is raised.
It makes it extremely hard, if not impossible, to debug. It is becoming useful to remove whole try/except blocks when debugging because of this pattern.

Exceptions are there to allow behavior changes and make logical decisions on what to do next. Broad exceptions do not allow that and forces this pattern everywhere else to suppress errors.
Although this is technically a "tech debt" problem - it has been raised before, and the issue is significant, so I'm going to add this as a "bug" category/label as well. It is pressing to address this problem.

This can probably be broken down into a few different issues we can tackle as part of a milestone to cleanup.

Explicit code for the "fail safe" pattern where some code needs to execute and no "reasonable" (e.g. not a RuntimeException) should cause it to exit with an exception that it doesn't explicitly throw. This would help identify these cases and distinguish from lazy handling.
Create a better set of specific Exceptions for Engine. There are parts of the code that have their own but far too many cases just raise Exception types with a message. Without this change the handlers must necessarily be generic.
Consistent mappers in the API layers to handle and raise AnchoreApiError exceptions rather than returning specific response payloads. This is used in some services but not others, so we have a mix of catch-all and return using 'make_error_response()' and raising the more specific exceptions that the web stack will catch and has proper handlers for marshalling responses into HTTP.

I do not think, however, that this is something that can be done randomly or as part of other feature work/bug fixes. We'll need to divide up the code into sections and work thru it systematically since these changes will certainly cause potential regressions in control flow and user-facing error responses.

This can probably be broken down into a few different issues we can tackle as part of a milestone to cleanup.

Explicit code for the "fail safe" pattern where some code needs to execute and no "reasonable" (e.g. not a RuntimeException) should cause it to exit with an exception that it doesn't explicitly throw. This would help identify these cases and distinguish from lazy handling.
Create a better set of specific Exceptions for Engine. There are parts of the code that have their own but far too many cases just raise Exception types with a message. Without this change the handlers must necessarily be generic.
Consistent mappers in the API layers to handle and raise AnchoreApiError exceptions rather than returning specific response payloads. This is used in some services but not others, so we have a mix of catch-all and return using 'make_error_response()' and raising the more specific exceptions that the web stack will catch and has proper handlers for marshalling responses into HTTP.

I do not think, however, that this is something that can be done randomly or as part of other feature work/bug fixes. We'll need to divide up the code into sections and work thru it systematically since these changes will certainly cause potential regressions in control flow and user-facing error responses.


Add orographic drag

Purpose
We would like to add the orographic drag parameterization following Garner05.
Cost/Benefits/Risks

Benefits: When we are able to run simulations with topography, it adds proper momentum dampings due to mountain drags (and will largely improve the surface pressure results -- comments from Isaac Held).
Cost/Risks: Depending on how it is implemented, it might add more allocations and reduce performances. We will need help from software engineers (mostly likely via a separate issue).

People and Personnel
(@ the personnel required/requested to implement the software solution)

Lead: @jiahe23
Collaborators: @LenkaNovak @szy21
Reviewers: TBD

Components

[ ] Read topography information from files;
[ ] Compute base flux;
[ ] Compute flux saturation profile;
[ ] Compute (u,v) forcings;
[ ] Hook up to ClimaAtmos infrastructure;
[ ] Tests

[ ] figure 6 in the Garner paper;
[ ] reanalysis data (e.g., ERA5) which have orographic drag diagnostics - though only surface stress here.

Inputs

Theory from Garner05;
Implementation following GFDL.

Results and deliverables

[ ] Orographic drag parameterization that can be switch on for simulations with topography;
[ ] A simple test that generates a figure comparable to figure 6 of the paper.

Task Breakdown And Tentative Due Date

[ ] ClimaAtmos infrastructure for this parameterization -- Nov 18
[ ] Read topo data (and remap if needed) -- Nov 22
[ ] Implement the calculations -- Dec 2
[ ] Tests -- Dec 9

Proposed Delivery Date
Dec 9 (subject to modifications based on the progress).

LGTM. Please make sure that good documentation is included in the deliverables.

LGTM. Please make sure that good documentation is included in the deliverables.

Yep, I've added it into the tasks and deliverables.


Typo in New-SPSite.md

Remove duplicated 'the'.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/New-SPSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

Thank you for your contribution!


Production build with NextFederationPlugin is crashing on a large project

Hi @ScriptedAlchemy!
First of all, thanks for creating this plugin!
I'm in the process of adding it to a project, but I'm currently stuck due to a "Reached heap limit Allocation failed - JavaScript heap out of memory" error when creating a production build for one of the federated apps.
Package Versions:
next: 13.2.4
react: 18.2.0
@module-federation/nextjs-mf: 7.0.8
max_old_space_size: 16gb
Additional information:

Starting the dev server used to heap overflow, but no longer does after sharing all dependencies except next and module-federation/nextjs-mf as singleton.
This app has ~500 pages and builds fine without NextFederationPlugin, but heap overflows with it.
Changing the number of exposed components does not seem to fix the problem, but production builds do succeed while using NextFederationPlugin if I delete most of the pages.

try v8, its on the canary tag on npm.

Thanks for your response!
On 8.1.0-canary.5, I need to manually set process.env.NEXT_PRIVATE_LOCAL_WEBPACK = 'true', and after that, I see "Uncaught Error: Shared module is not available for eager consumption" for packages that are not direct dependencies of my app. This was not something I encountered on 7.0.8.
As for the heap out of memory issue, it seems to be correlated with importing large packages (ie ones that do not tree shake) in many pages. Two separate things I tried were lazy loading such packages, and disabling SSR in my _app.js using https://www.npmjs.com/package/react-no-ssr, and while First Load JS for each page dropped 75%, I'm still unable to successfully build the app without deleting a subset of pages.
Are there any steps you would recommend taking to debug the next build process, or NextFederationPlugin's impact on it?

Hi @ScriptedAlchemy, I was able to get 8.1.0-canary.1 to run, but production builds still cause a heap overflow.

@ScriptedAlchemy Repro: https://github.com/ChrisCantThinkOfAUsername/nextjs-v13-heap-issue-repro

Thanks I'll take a look and my pull in some of my ByteDance infra engineers who are well versed in low level runtime stuff.
This might solve it. We will see. But I'm very close to releasing v1.5 of federation where we have made significant upgrades and added new runtime APIs. Once it lands I'll rework next.js plugin into of these new APIs and see how much of next plugin is still needed. https://github.com/module-federation/universe/pull/1623

bump

Thanks @ScriptedAlchemy; I will retest once the next @module-federation/nextjs-mf canary build is published.

bump

@ChrisCantThinkOfAUsername you can try https://www.npmjs.com/package/@module-federation/nextjs-mf?activeTab=versions which is using the new v1.5 module-federation/enhanced plugin and federation/runtime packages

@ScriptedAlchemy Still seeing heap overflow on 0.0.0-feat-node-support-1703050615268 unfortunately. I'll just disable federation for the server build in the meantime.

It's probably the chunk correlation plugin which is quite intense. Can you increase memory limit of node? Max old space

Hello,
i am facing an issue with the Production environment. Currently i am using
Next.js version: 13.5.6
React version: 18
@module-federation/nextjs-mf: 7.0.8
webpack: 5.89.0
When i try with next dev it's working fine but when i create a build and try with next start. it will not work.
And getting this error:

Please let me know if have any solution

Hi,
Firstly, thank you for your efforts on the plugin – it has greatly enhanced the community's capabilities.
I find myself in a situation similar to @ChrisCantThinkOfAUsername, but we're working with a lower version of Next.js. Currently, we're undertaking the challenge of transitioning from a heavily bloated legacy Next.js application to multiple micro frontends. Unfortunately, due to certain constraints, we can't follow the conventional approach of rebuilding from the outside in (shell first and micro frontends under a new shell). Instead, we have to adopt an inside-out strategy, relying on the legacy Next.js application with new micro frontends loaded inside. During this process, we've encountered an issue when attempting to incrementally adopt module federated apps within the legacy application.
Here are our package versions:

next: 12.3.4
react: 18.2.0
@module-federation/nextjs-mf: 7.0.8 (we've also tried the latest next tags without success)

Our attempts to address the issue included increasing the node memory limit, which proved effective in a local environment. However, our CI environment imposes memory limitations that prevent us from implementing the necessary memory bump.
If you require additional information or if providing dumps would be helpful, please let me know. Your assistance is much appreciated!
Thanks!

Thanks for creating this plugin!
I have same problem. any update for this issue?
Package versions:
"next": "13.1.6",
"@module-federation/nextjs-mf": "^8.1.3"
I see most of the storage inside .next/cache/webpack. In my case, ~800MB without this plugin and ~7GB using it

This issue likely resides in the chunk correlation plugin which iterates over a large amount of stats data.
If it does away when not applied client side then its probs the stats stuff.
As a workaround, increase heap size of node. We will replace the stats plugin this quarter i believe

issue wont exist at runtime. The problem is related to build and that i have to loop over large amount of data in compiling to construct correlations. I may improve heap issue when switch over to #2108

Hi @ScriptedAlchemy, does this issue only appear with nextjs-mf: 7.0.8 or with nextjs-mf: 8 as well?

solved by setting webpack config.cache = false in next.config.js but I don't think it's a good solution

Issue exists since v1 of next mf.
Increase memory heap.

Hi, I was wondering if there's been any progress on resolving this issue since it was marked as stale? Has the latest version or any recent workaround helped improve the situation?

Refer to previous comment on other issue


Communication: Lecture and Exercise Links open in Browser

When clicking a Lecture or Exercise link in a chat, this link is opened in the browser instead of the native Android app.
While participation in exercises in not possible on mobile devices, the users should still be able to check the status of an exercise via the link within the app (see iOS).
Potentially Related
In the Google PlayConsole, under Grow > Deep links there are multiple errors, some also related to lectures and exercises.

I created a PR on the main Artemis repo that may fix the issue


[BULK] DocuTune - Rebranding

Global effort for the Microsoft Learn platform rebrand
The Microsoft team is applying changes to content for the Microsoft Learn platform rebrand. Changes include modifications to current Microsoft Learn and Microsoft Docs terminology as well as associated domain name and content URL path changes.
DO NOT MERGE THIS PULL REQUEST WHILE IN DRAFT STATUS! Some of the changes to links and other content are not valid until after go-live.
Please review this draft PR and provide any necessary comments as soon as possible. Because these fixes are tightly scoped, we have approval to automatically merge these changes to the default branch for publication. If you have any questions about the project or our approach, contact Donna Taylor.
This PR was generated via DocuTune. It includes only targeted fixes and minor formatting adjustments.
CorrelationId: d8c4c3ac-d49a-4c95-9ddc-9822ca297907
InitiativeId: docutune-rebranding-2022-09
PointOfContact: Alex Buck

#docutune-draft

#docutune-ready

@IEvangelist @Rick-Anderson
Please merge - thanks!
#docutune-request-approval


Found a bug with the shuffle in the music app

Describe the Issue
There is a bug on the music app, when you delete all the songs and click shuffle, the reset playlist button dissappear, it cause to dont have acces to do anything else with the app, the only solution is to refresh the web.
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-basic-string-and-array-methods-by-building-a-music-player/step-99
Steps to Reproduce

Delete all the songs.
Click on Shuffle.
There is no way to reset the list by using the app.

Expected behavior
I expected to still see the reset button in UI, not to delete it.
Screenshots

As you can see, step 99, and doing the bug the reset button dissappear.
System

Device: [Alls]
Browser: [Alls]

Additional context
Would be good to add another step by resolving this bug :D

You are right. The Reset Playlist button is added, to the songs list, by deleteSong when deleting the last song. When shuffle is used, songs list calls renderSongs which clears the button.
I think better place for adding Reset Playlist button would be renderSongs function. Alternatively shuffle could return early if there's no songs.

Hi @gikf
I am working on the fix for this issue.
The only thing is, I am not sure in which Step of the Curriculum the code snippet should be introduced.

We just have to write the resetButton logic separately and use it in both the shuffle and DeleteSong to make sure that resetButton is correctly rendered when there is no songs in the playlist.

hey can i work on this ?


Add TabPFN

Issue #, if available:
#2806
Description of changes:

Adds the TabPFN model to AutoGluon.

TODO:

[ ] Run AutoMLBenchmark to verify correctness & performance
[ ] Verify implementation with TabPFN authors

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hello @noahho and @SamuelGabriel, I am looking to add TabPFN to AutoGluon for the upcoming v0.8 release on June 13th!
If you had time, it would be great if you could give a quick look at the PR and mention any suggestions/changes you'd recommend.
In particular, I'd be curious your thoughts on the sample_rows value and how to best handle missing values.

Hey :) I have left two comments on your points. It looks good in general, I think, besides the 0 imputation strategy.
Generally I think it is important to know that the main costs of TabPFN are incurred with loading the model to (GPU) memory at the first construction of the TabPFNClassifer and in the fit function, where the full forward pass happens.
Best,
Sam

On 4. Jun 2023, at 05:47, Nick Erickson @.***> wrote:
Hello @noahho https://github.com/noahho and @SamuelGabriel https://github.com/SamuelGabriel, I am looking to add TabPFN to AutoGluon for the upcoming v0.8 release on June 13th!
If you had time, it would be great if you could give a quick look at the PR and mention any suggestions/changes you'd recommend.
In particular, I'd be curious your thoughts on the sample_rows value and how to best handle missing values.
—
Reply to this email directly, view it on GitHub https://github.com/autogluon/autogluon/pull/3270#issuecomment-1575363978, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACK7PSLIGCRM6UZ4QVS2TGTXJQALRANCNFSM6AAAAAAYZVQF3A.
You are receiving this because you were mentioned.


[Android] [Navigator] HorizonalSwipeJump animation overlap

Device: Nexus 5
OS: Android 6.0
React-native: 0.14.0-rc
Scenes overlap during HorizonalSwipeJump animation. Problem is not reproduced in both Android and iOS with react-native 0.12.x and iOS with react-native 0.14.0-rc
Example with 2 scenes and a Navigator with HorizontalSwipeJump

Found that the problem is in ReactAndroid. bf598647d2ff7aad1d29c7c5d28d816c01941c1c breaks it for me

Added example project to reproduce the bug at https://github.com/dgladkov/react-native-3773

cc @kmagiera


Can't import Bestiary - Rakshasa (d20pfsrd.com - no, aonprd.com - no)

Can't import d20pfsrd.com statblock:
RakshasaCR 10
XP 9,600
LE Medium outsider (native, shapechanger)
Init +9; Senses darkvision 60 ft.; Perception +14
DEFENSE
AC 25, touch 16, flat-footed 19 (+5 Dex, +1 dodge, +9 natural)
hp 115 (10d10+60)
Fort +9, Ref +12, Will +8
DR 15/good and piercing; SR 25
OFFENSE
Speed 40 ft.
Melee +1 kukri +16/+11 (1d4+4/15–20), claw +10 (1d4+1), bite +10 (1d6+1)
Special Attacks detect thoughts
Spells Known (CL 7th)
3rd (5/day)—lightning bolt (DC 16), suggestion (DC 16)
2nd (7/day)—acid arrow, invisibility, minor image
1st (7/day)—charm person (DC 14), mage armor, magic missile, shield, silent image
0th (at will)—dancing lights, detect magic, ghost sound (DC 13), mage hand, mending, message, prestidigitation
STATISTICS
Str 16, Dex 20, Con 22, Int 13, Wis 13, Cha 17
Base Atk +10; CMB +13; CMD 29
Feats Combat Expertise, Dodge, Improved Critical (kukri), Improved Initiative, Weapon Finesse
Skills Bluff +20, Diplomacy +16, Disguise +24, Perception +14, Perform +16, Sense Motive +14, Stealth +18; Racial Modifiers +4 Bluff, +8 Disguise
Languages Common, Infernal, Undercommon
SQ change shape (any humanoid, alter self)
SPECIAL ABILITIES
Detect Thoughts (Su)
A rakshasa can detect thoughts as per the spell of the same name (CL 18th). It can suppress or resume this ability as a free action. When a rakshasa uses this ability, it always functions as if it had spent three rounds concentrating and thus gains the maximum amount of information possible. A creature can resist this effect with a DC 18 Will save. The save DC is Charisma-based.
ECOLOGY
Environment any
Organization solitary, pair, or cult (3–12)
Treasure double (+1 kukri, other treasure)

Can't import aonprd.com statblock:
Rakshasa CR 10
Source Pathfinder RPG Bestiary pg. 231
XP 9,600
LE Medium outsider (native, shapechanger)
Init +9; Senses darkvision 60 ft.; Perception +14
Defense
AC 25, touch 16, flat-footed 19 (+5 Dex, +1 dodge, +9 natural)
hp 115 (10d10+60)
Fort +9, Ref +12, Will +8
DR 15/good and piercing; SR 25
Offense
Speed 40 ft.
Melee +1 kukri +16/+11 (1d4+4/15–20), claw +10 (1d4+1), bite +10 (1d6+1)
Special Attacks detect thoughts
Spells Known (CL 7th)
3rd (5/day)—lightning bolt (DC 16), suggestion (DC 16)
2nd (7/day)—acid arrow, invisibility, minor image
1st (7/day)—charm person (DC 14), mage armor, magic missile, shield, silent image
0—dancing lights, detect magic, ghost sound (DC 13), mage hand, mending, message, prestidigitation
Statistics
Str 16, Dex 20, Con 22, Int 13, Wis 13, Cha 17
Base Atk +10; CMB +13; CMD 29
Feats Combat Expertise, Dodge, Improved Critical (kukri), Improved Initiative, Weapon Finesse
Skills Bluff +20, Diplomacy +16, Disguise +24, Perception +14, Perform +16, Sense Motive +14, Stealth +18; Racial Modifiers +4 Bluff, +8 Disguise
Languages Common, Infernal, Undercommon
SQ change shape (any humanoid, alter self)
Ecology
Environment any
Organization solitary, pair, or cult (3–12)
Treasure double (+1 kukri, other treasure)
Special Abilities
Detect Thoughts (Su) A rakshasa can detect thoughts as per the spell of the same name (CL 18th). It can suppress or resume this ability as a free action. When a rakshasa uses this ability, it always functions as if it had spent three rounds concentrating and thus gains the maximum amount of information possible. A creature can resist this effect with a DC 18 Will save. The save DC is Charisma-based.
Description
The rakshasa is an evil spirit that cloaks itself in the guise of a humanoid creature that it might walk unseen among its prey. They embody what is taboo among most societies, and in the shape of those it seeks to defile, a rakshasa gorges itself on these hideous acts. Were they human, these acts of cannibalism, blasphemy, and worse would mark them as criminals condemned to the cruelest of hells.
When not disguised as a humanoid, the otherwise humanoid rakshasa has the head of an animal. Often, they possess the heads of great cats (such as a tiger or panther) or a snake (like a cobra or viper), yet other heads are not unknown—apes, jackals, vultures, elephants, mantises, lizards, rhinos, boars, and more are possible. In most cases, the type of head a rakshasa possesses speaks in some way to its personality—a tiger-headed rakshasa is stealthy and ravenous, while a boar-headed one might be gluttonous and crude. These changes rarely impact the rakshasa’s base statistics, although there are more powerful variants of the standard rakshasa that possess multiple heads, more potent spellcasting powers, and additional deadly and unusual special abilities.
Rakshasas scoff at religion—they understand the power of the divine, but see themselves as the only thing worthy of worship from the mortal races. Rakshasa clerics are thus quite rare. Although rakshasas are outsiders, they are also very much creatures of the Material Plane, and many believe the first rakshasas chose this exile over some other role offered them by a long-forgotten god. Although they usually work alone, it isn’t unheard of to find extended families of rakshasas working together to ruin a mortal civilization from the inside out over the course of many generations. A rakshasa is 6 feet tall and weighs 180 lbs.

Can't import Hero Lab statblock:
Rakshasa      CR 10
XP 9,600
Male rakshasa (Pathfinder RPG Bestiary 231)
LE Medium outsider (native, shapechanger)
Init +9; Senses darkvision 60 ft.; Perception +14
Defense
AC 25, touch 16, flat-footed 19 (+5 Dex, +1 dodge, +9 natural)
hp 115 (10d10+60)
Fort +9, Ref +12, Will +8
DR 15/good, 15/piercing; SR 25
Offense
Speed 40 ft.
Melee +1 kukri +16/+11 (1d4+4/15-20) or
bite +10 (1d6+1), 2 claws +10 (1d4+1)
Special Attacks detect thoughts (DC 18)
Sorcerer Spells Known (CL 7th; concentration +10)
3rd (5/day)—lightning bolt (DC 16), suggestion (DC 16)
2nd (7/day)—acid arrow, invisibility, minor image (DC 15)
1st (7/day)—charm person (DC 14), mage armor, magic missile, shield, silent image (DC 14)
0 (at will)—dancing lights, detect magic, ghost sound (DC 13), mage hand, mending, message, prestidigitation
Statistics
Str 16, Dex 20, Con 22, Int 13, Wis 13, Cha 17
Base Atk +10; CMB +13; CMD 29
Feats Combat Expertise, Dodge, Improved Critical (kukri), Improved Initiative, Weapon Finesse
Skills Acrobatics +5 (+9 to jump), Bluff +20, Diplomacy +16, Disguise +24, Perception +14, Perform (Enter Choice) +16, Sense Motive +14, Stealth +18; Racial Modifiers +4 Bluff, +8 Disguise
Languages Common, Infernal, Undercommon
SQ change shape (any humanoid; alter self)
Other Gear +1 kukri
Ecology
Environment any
Organization solitary, pair, or cult (3-12)
Treasure double (+1 kukri, other treasure)
Special Abilities
Change Shape (any humanoid; alter self) (Su) You can change your form.
Combat Expertise +/-3 Bonus to AC in exchange for an equal penalty to attack.
Damage Reduction (15/good) You have Damage Reduction against all except Good attacks.
Damage Reduction (15/piercing) You have Damage Reduction against all except Piercing attacks.
Darkvision (60 feet) You can see in the dark (black and white only).
Detect Thoughts (DC 18) (Su) Detect thoughts as spell, always functions as if spent three rounds concentrating. (Will neg.)
Spell Resistance (25) You have Spell Resistance.

The HeroLab one works, the others are missing the Perform Type, e.g. "Perform +16" needs to be "Perform (Sing) +16".


Correct copyright on http://atomiq.io

Current copyright on atomiq.io is incorrect (as Atomiq is not a proper legal entity)
Copyright © 2016-2017 Atomiq and contributors => Copyright © 2016-2017 Axway
I'm unable to file an issue against the atomiq.io website, so filing it here. Please move as appropriate.

Done.

I meant to indicate replacing one phrase with the other. Will correct in a PR.

I think #1076 resoles this issue.


fetch tiny llama if nothing passed

examples/llama.py fetches TinyLlama weights if nothing is passed.

@geohot do you want to test this since you asked for it?

Do you need a subdir? Again, same problem. The default hash naming is a lot better

The hash naming is definitely better. The issue was that the model load function used path suffix checks and the tokenizer was found by file name.
The load function should be redone. If this looks ok, I will do that with the data hashing in a separate PR.

Closing. Too much feedback and there's a random whitespace change in here


Refactor REST_API_Factory class

Feature Description
Add in https://github.com/google/web-stories-wp/pull/6906 the REST_API_Factory class is used to register REST APIs. These classes are different for a number of reason.

REST API should be registered on rest_api_init action.
These classes extend WP_REST_Controller, so the methods is defined upstream in WordPress core.
Many of these classes take a string as param and multiple instances of the classes are instantiated.

Let's find a way to use the class injector and service model we already use to keep things a little cleaner.
Alternatives Considered

Additional Context

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

I am CCing @schlessera on this one, as I think his input would be useful.


Fix doc example.

This closes #206
We should get our doc string testing working. The current form of this doc example won't actually run unless we instance a client and give it a uri that actually points to a server.

This PR seems rather straightforward considering worst case scenerio: its still wrong. If nobody objects, I'm going to merge it this evening.


Bump Checker dataflow to 3.16.0

This pulls in https://github.com/typetools/checker-framework/pull/4748 which should give us a perf win

Pull Request Test Coverage Report for Build #551

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

Totals

Change from base Build #550:
0.0%

Covered Lines:
3496

Relevant Lines:
3921

💛  - Coveralls

On our single jmh benchmark, before:
Benchmark                   Mode  Cnt  Score   Error  Units
CaffeineBenchmark.compile  thrpt   25  2.189 ± 0.011  ops/s

After:
Benchmark                   Mode  Cnt  Score   Error  Units
CaffeineBenchmark.compile  thrpt   25  2.276 ± 0.020  ops/s

That's a roughly 4% speedup in bottom-line compile time!  Of course, real-world wins will be much less, since this benchmark only runs NullAway and no other Error Prone checks, no annotation processors, etc.

I'll also note that the speedup comes from faster CFG construction for code containing lambdas.  So code bases without much use of lambdas won't see much difference.


doesn't output any files

I ran the following command:
./node_modules/.bin/flow-coverage-report \\
-f ./node_modules/.bin/flow \\
-i 'packages/wonder-blocks-*/**/*.js' \\
-x 'packages/wonder-blocks-*/dist/*.js' -x '**/generated-snapshot.test.js' \\
-o "./flow-coverage"

and it prints out the coverage but doesn't store the output anywhere.  The -o seems redundant since "./flow-coverage" is the default so I tried with and without that parameter with the same results.

Hi @kevinbarabash
The default report type is "text", which will produce the coverage report on the console, but do not generate the html report or any other file locally.
To generate the html report you have to explicitly ask for its report type on the command line (e.g. by running: flow-coverage-report -t text -t html ...) or from one of the supported configuration sources (e.g. from the package.json file as used in this repository: https://github.com/rpl/flow-coverage-report/blob/5a72d56bcc661d00ce698915845e6e55fd93ecf1/package.json#L164-L175).


Resolve: Opening index.html over file:// won't work.

Inside the file config/index.js altere o assetsPublicPath: '/' para assetsPublicPath: './' in build object

Imagine that you're using history mode, and ./ will always load resource from relative path, eg: at url  '/user/egoist' the browser will load '/user/egoist/bundle.js' which will lead to 404 error.
So my suggestion is always using absolute path in publicPath

To be able to use this with cordova, you have to be able to use relative paths, because 1) it uses file://  2) the root is unpredictable.  I agree with @guilhermefelipesilva that this should be fixed.

@asselin you can always change it in the config file, that's why it exists.

Looking further, this is a dup of #200


Incorrect display of lists in online version

If you enter a list such as [1; 2; 3], the closing bracket is missing. Things get weirder with nested lists. The problem seems to lie in the color formatting syntax of jQuery terminal.

For me nested lists are still not printed at all or is extremely strange in the online demo:
# let asd = with choose_all handle
 let x = choose_int 0 5 in
 let y = choose_int 1 x in
 [x;y]
val asd : int list list = [[1; 1]; [2; 1]; [2; 2]; [3; 1]; [3; 2]; [3; 3];
                           [4; 1]; [4; 2]; [4; 3]; [4; 4]; [5; 1]; [5; 2];
# with choose_all handle
 let x = choose_int 0 5 in
 let y = choose_int 1 x in
 [x;y]

<Nothing is printed>

# [[1]]

...
- : int list list =
- : int = 1
- : int = 1
- : int = 1
- : int = 1
- : int = 1
- : int list list =
- : int = 1
- : int = 1
- : int = 1
- : int = 1
- : int list list =
- : int = 1
- : int = 1
- : int = 1
- : int list list =
- : int = 1
- : int = 1
- : int list list =
- : int = 1
- : int list list =
#

This is indeed very weird.
Evaluating [[1]] actually crashes my browser. I'll take a look into it.


updating shapes.util to accomodate for n-dimensional broadcasting

Description
Please include a short summary of the change.
This PR aims to greatly simplify the current sum_shapes utils by taking advantage of the np.broadcast_shapes function.
This also generalizes the mul_shapes function to be able to adhere to n-dimensional broadcasting rules. Previously it required that all outer dimensions (i.e. any dimension other than the last-two) be the same for both operands.
However, NumPy allows its typical broadcasting rules on the outer dimensions. Ex: (2,2,2) @ (2,2) is allowed because the RHS is broadcast to (1,2,2).
TODO: add some tests using hypothesis and maybe improve some of the docstrings.
Issue link (if applicable):
Type of change

[x] New feature (backwards compatible)
[ ] New feature (breaking API changes)
[ ] Bug fix
[ ] Other (Documentation, CI, ...)

Contribution checklist

[x] Add our license to new files.
[x] Check that your code adheres to our coding style.
[x] Write unittests.
[x] Run the unittests and check that they’re passing.
[x] Run the benchmarks to make sure your change doesn’t introduce a regression.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.William Zijie Zhang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Ok, I noticed a slight issue with the changes in this PR.
np.broadcast_shapes is a relatively new API function (see image below) and hence fails the builds for Python 3.9 and 3.8.

What should we do about this?

Could we check the NumPy version and then have a fallback to the old code? Maybe just error if someone is trying to use ndarrays with an old NumPy version?
Alternatively, Python 3.8 is EOL in October, so we could just plan to not support it. This PR won't be backported to 1.4.x or 1.5.x. Python 3.9 is EOL in October '25 (so a while from now :( ), but the oldest supported NumPy is 1.19.3, so it doesn't feel crazy to me to just bump our minimum NumPy version to 1.20. and tell people to stay on 1.5.x when 1.6.x comes out.


[Bug]: The intercom pop is not displayed in the release-Frozen instance

Is there an existing issue for this?

[X] I have searched the existing issues

Description
The intercom pop is not displayed in the release-Frozen instance
Steps To Reproduce

take a pull from appsmith/appsmith-ce:release-frozen
run docker run -d --name appsmithAuditLogs -p 80:80 -p 9001:9001 -e APPSMITH_AUDITLOG_ENABLED=true -v "$PWD/stacks:/appsmith-stacks" appsmith/appsmith-ce:release-frozen
3.Open the localhost
3.Navigate to audit log page
4.Click on upgrade button
The intercom pop up doesnt show

Public Sample App
No response
Version
release-frozen

Not a blocker issue


Logs showing up in root

Need to fix:

Me thinks ${APACHE_LOG_DIR} is not a centos thing?

Looks like the path string should be: /var/log/httpd/, just going to replace the var with that.

Fixed.


CTRL+C does ot kill child process

I have a start script to run my express server:
"start": "env-cmd ./.env node ./dist/server.js"
When it runs my express app starts listening on some port
When I hit CTRL+C it exists but the node precess of express still alive
I use windows 10 but I think it's an issue of spawn, you spawn a process and dont send he signal to it when you recieve it...

I think it's more of npm issue https://github.com/npm/npm/issues/4603


Improve script search performance

There are now enough scripts that search is actually somewhat slow, especially with the character search feature. I put no effort into performance in the first implementation. It's now worth it to look into performance, perhaps by adding some full-text indexing of script titles (with a library) or by doing something less naive (like search a list of characters, then search an index of scripts by character). It might also be useful to cancel an existing search if possible.

Here's a very fast one that indexes all the scripts as a Webassembly module at build time: https://stork-search.net/

https://www.fusejs.io/ looks like a good drop-in replacement, I'll try this

Yeah that looks great - we'll have both fuzzy searching and character + title search, which is perfect. Looks like a little work is needed to generate an index at build time, but you should be able to integrate it into fetch-assets.


Hide data science walkthrough for the web

This is now done in the control tower. Once we bring the DS walkthrough back we just need to make sure to disable it for web users, like we do in the Python walkthrough (cc @minsa110).
To verify: make sure you don't see the Python for Data Science walkthrough tile in the getting started page anytime you open VS Code, (whether it's locally using insiders, stable, whether it's the Python insiders version or default, whether it's github.dev or Codespaces).


Helm 3:  set custom manifest config media type on chart push

This finalizes the custom mediatype for Helm charts stored in an OCI registry:
application/vnd.cncf.helm.chart.config.v1+json

A current manifest looks like the following:
{
  "schemaVersion": 2,
  "config": {
    "mediaType": "application/vnd.cncf.helm.chart.config.v1+json",
    "digest": "sha256:44136fa355b3678a1146ad16f7e8649e94fb4fc21fe77e8310c060f61caaff8a",
    "size": 2
  },
  "layers": [
    {
      "mediaType": "application/vnd.cncf.helm.chart.meta.layer.v1+json",
      "digest": "sha256:ca9588a9340fb83a62777cd177dae4ba5ab52061a1618ce2e21930b86c412d9e",
      "size": 19,
      "annotations": {
        "org.opencontainers.image.title": "chart-meta.json"
      }
    },
    {
      "mediaType": "application/vnd.cncf.helm.chart.content.layer.v1+tar",
      "digest": "sha256:a66666c6b35ee25aa8ecd7d0e871389b5a2a0576295d6c366aefe836001cb90d",
      "size": 459,
      "annotations": {
        "org.opencontainers.image.title": "chart-content.tgz",
        "sh.helm.chart.name": "mychart",
        "sh.helm.chart.version": "2.7.0"
      }
    }
  ]
}

More modifications coming soon to the "layers" section, but this top-level media type is important to decide on first. Before this, it was using the ORAS default (application/vnd.oci.image.config.v1+json).
This is important for registries that wish to whitelist package types that they wish to store.
Also in this PR: upgrade to ORAS 0.5.0 which gives us the oras.WithConfigMediaType method.
This PR based on top of #5717 (lets merge that first)

I just noticed the Helm Chart version annotation is placed on the layer. Wouldn't this be more appropriate on the overall object/manifest?
Either an annotation in the config, or actually use the config object to store this.
Having at the layer complicates parsing at the registry, and allows different layers to have different versions.
From the registry perspective, we'd like to basically ignore details on the layer. By parsing information at the manifest, we can surface information in listings (portal or CLI), and we can surface annotations on search results. I would like to avoid the perf and complications of parsing metadata at the layer level.

That not fully accurate @stevelasker. Layer annotations as in the manifest. So that is already being parsed. It’s not a concern for perf.
I would discuss the design aspect.

I'm still a bit concerned about the possible duplication. It just feels wrong to have annotations of a thing, at layers that can be reused. Although, if the chart is a reusable layer, I may be convincing myself the other way.
I'd like to start some depth conversations on when information should be in annotations, vs. a config object.

Hi, the final annotations and layers are still up for debate. This PR aims
to nail down the manifest media type only.
On Sat, May 11, 2019 at 12:24 PM Steve Lasker<EMAIL_ADDRESS>wrote:

I'm still a bit concerned about the possible duplication. It just feels
wrong to have annotations of a thing, at layers that can be reused.
Although, if the chart is a reusable layer, I may be convincing myself the
other way.
I'd like to start some depth conversations on when information should be
in annotations, vs. a config object.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/helm/helm/pull/5719#issuecomment-491524840, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADACFTK5QN35HE2SFZGITTPU3XMPANCNFSM4HMEZMBQ
.

I need @jdolitsky to comment on the intent here. The manifest.config.mediaType is used to determine the type of artifact. In this case, a Helm Chart. It could contain information about the object, just as the image-spec defines how a container image is run. But a config file is optional.
You're correct, the image-spec covers multiple mediaTypes. The manifest has a subset, which we're extending for new artifacts.
I also saw a layer that was a meta layer. If the thing @jdolitsky is trying to persist is a seperate file, used by helm, then, yes, this would be a meta.layer in the layers collection. However, we still need a way for registries and registry tools, like docker, helm, singularity, opa, to know what object types they're requesting. The manifest.config.mediaType is the piece of information used. If you have no artifact configuration information, just pass in a null pointer, or an empty config.json file. We should support null config files, so if this doesn't currently work, it's a bug.

The other key difference is the OCI Image Spec is for container images (docker). The OCI Artifact Registry design is to support additional artifacts. The distribution spec will outline the differences. Although, we're working hard to make them minimal as we expect a lot of common tooling.
Here's the beginning of the distribution spec, talking about the formatting of the manifest.config.mediaType: https://github.com/AzureCR/distribution-spec/blob/artifact-registry/artifacts.md

Sorry I am away and sending from email so pardon any lack of detail/clarity
here-
The diff is misleading. We are not renaming from meta to config. We simply
were not using a custom manifest config media type prior to this and were
using the ORAS default.
In terms of the other renames, they just seemed more readable. Keep in mind
this PR is not focused on the later media types, I still want to iterate on
this prior to 3.0. I’m fine with reverting this if necessary.
In terms of the KnownMediaTypes - there is essentially no change here. The
manifest config media type need not be here for ORAS-based pull.
I’ll have computer access later today if you want to request any changes
given this explanation.
On Tue, May 14, 2019 at 12:41 PM Matthew Fisher<EMAIL_ADDRESS>wrote:

@bacongobbler commented on this pull request.
In pkg/registry/constants.go
https://github.com/helm/helm/pull/5719#discussion_r283895734:

@@ -20,11 +20,14 @@ const (
// HelmChartDefaultTag is the default tag used when storing a chart reference with no tag
HelmChartDefaultTag = "latest"

// HelmChartMetaMediaType is the reserved media type for Helm chart metadata
HelmChartMetaMediaType = "application/vnd.cncf.helm.chart.meta.v1+json"

// HelmChartConfigMediaType is the reserved media type for the Helm chart manifest config
HelmChartConfigMediaType = "application/vnd.cncf.helm.chart.config.v1+json"

I don't understand why we're re-naming it to config, and from what I
read in that blog post I could not figure out the reason. Can you point out
why we need to change it to config? meta is the standard descriptor
we've been using for chart metadata so I'm not sure why we need to change
the convention to follow.
The image-spec documentation
https://github.com/opencontainers/image-spec/blob/master/media-types.md
doesn't appear to follow that convention laid out in the blog post either.
manifest, descriptor, index, header and others all don't follow that
convention, so I'm led to believe that this isn't a convention we need to
follow.
According to the image-spec, the OCI image config specification
https://github.com/opencontainers/image-spec/blob/master/config.md uses
the term config because the metadata stored in that object is the image
configuration - that is, it describes how the layers should be used by a
container runtime and execution tool and its relationship to filesystem
changesets, and therefore it's "config". In other words, the content and
how it's being used defines the schema, not the other way around.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/helm/helm/pull/5719?email_source=notifications&email_token=AADACFSIQIM3J5Z57QT5WW3PVLTSDA5CNFSM4HMEZMB2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOBYS7VRY#discussion_r283895734,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADACFX47YEI32X4VHJTAK3PVLTSDANCNFSM4HMEZMBQ
.

Ok, so this isn't renaming the meta layer, it's really just providing the info for registries and registry tooling to understand the artifact type. The ORAS default is simply OCI, which isn't really a type. It's like saying I have a file on disk, with no type.
So, we do need a type. The ORAS library leaves it to the artifact author to define their type.
Here's an example of how ACR knows the pushed artifact is a Helm Chart.

Notice the manifest.config.mediaType is set to helm. (this was the original mediaType). We've since changed it to: application/vnd.cncf.helm.chart.config.v1+json, based on the PR Josh sent.
We also have OPA as well:  https://github.com/Azure/acr/blob/master/docs/artifact-media-types.json

fixes https://github.com/helm/helm/issues/5286

Updated this PR to use config mediatype application/vnd.cncf.helm.chart.v1+json instead
cc @bacongobbler @SteveLasker


Client can be dropped while port is open

Currently it is possible (and may happen by accident) that a Client is dropped, while a port or virtual endpoint is still open.
If the port is an output, you can still send to the output without an error, but no message is actually sent. Similarly, if the port is an input, you just won't receive any messages.
I don't think that this is a memory safety issue, but it might be a source of surprising behavior. The fix would be to couple the lifetimes of ports to the Client, though this makes the API slightly more complicated.

Hi @Boddlnagg, thanks for the heads up. I have spent 10 minutes thinking on that, and I have to recognise that the solution with lifetimes, and the actual implications on the user side for this change, go beyond my current understanding of lifetimes and the borrow checker.
I haven't been able to spend much time on rust during the last year, but I'll keep it in mind the next time I revisit the corresponding chapters from the Rust book.


[BugFix] Fix in resetting SumReward and StepCounter transforms

Description
In the current version of TorchRL, when envs are reset, a single TensorDict with the bool tensor _reset is passed to the env. This causes problems in the SumReward and StepCounter transforms.
Summarising, for batched environments every time one environment reaches the end of the episode, these transforms reset their count for all envs because its gets the most recent count from the input TensorDict itseld. This PR proposes a possible solution.
Now, the transforms keep track of the latest information for each environment and only reset the count for the envs indicated in _reset.

We can keep transforms stateless as long as the whole current tensordict is passed on to the env.reset() method
https://github.com/pytorch/rl/blob/main/torchrl/collectors/collectors.py#L757
But doing raises errors in other transforms. Is there any design decision preventing us from passing the whole tensordict?

Can you give me a minimally reproducible example to work with? I think i can fix the collectors to make this work

Fixed by #1021


Linux support

Could you please also make a linux release? I use Kubuntu 22.04 if it's matter.

I'll try it.

Any updates on this?

I run into a deployment issue when implementing this and I currently don't know how to strike it... https://github.com/shrinktofit/FBX-glTF-conv/runs/8083554602?check_suite_focus=true

Is there any progress, I use debian 11.

@shrinktofit,
there is a later version. May be try with it? https://www.autodesk.com/developer-network/platform-technologies/fbx-sdk-2020-3
And I don't know PowerShell, but shouldn't be the line like this: Write-Output 'yes' | $fbxSdkInstallationProgram $fbxSdkHomeLocation?

After some tries, I built a ubuntu release here: https://github.com/cocos/FBX-glTF-conv/releases/tag/release-v1.0.0-test.41 . But there's a pitfall, it only works on Ubuntu 22.04(or in theory later) since it was built on 22.04. Especially, Ubuntu 20.04 is not able to use.

I'll close this PR for now.


Find commits on non master branches

When I tag a new release, I typically create a branch called release-1.2.3, then update the version field in Project.toml. with a commit on that branch, push that branch, trigger registrator on that commit on that branch.
So when things are merged in the registry, the commit that corresponds to the tag will not be on master yet. It seems that tagbot then doesn't find the commit? Is it only looking in the history of master for commits that might correspond to a new tag?
The process I described above is also what PkgDev.tag does.
I could see two solutions: 1) just check commits in all branches, or 2) also look for commits in branches that are named release-1.2.3 specifically.

This should be fixed now, let me know if it works or not for you please!

Yep, works, thanks for the quick turnaround!


Error when multiple directories in include_dirs for extension

I want to compile my c extension using the setup.py shown below. Because the c files are contained inside of the subfolders, I want to include multiple directories using include_dirs. I think I am using include_dirs correctly, but cython still couldn't find my files. Can someone please tell me where I might do wrong?
This is my code:
from setuptools import setup
from setuptools.extension import Extension
setup(
    name="mycodecpy",
    version="1.0",
    description="mycodecpy",
    setup_requires=[
        'setuptools >= 18.0',
        'Cython',
    ],
    ext_modules=[
     Extension("mycodecpy", ["mycodecpy.pyx", "cpu.c", "cpupitch.c", "decimation.c", "ecebank.c", "filterbank.c", "Hcmbank.c", "command.c", "filenames.c", "pario.c", "sigio.c", "IPEMAuditoryModel.c", "Audimod.c", "AudiProg.c"],
               include_dirs=[
                   'AuditoryModel/src/audiprog',
                   'AuditoryModel/src/library',
                   'AuditoryModel/src/',
               ],

               )
    ],
)

This is my directory tree:
├── _AuditoryModel
│   ├──__init__.py
│   └── _src
│       ├──__init__.py
│       ├── (some files need to be included here)
│       ├── _audiprog
│       │   ├──__init__.py
│       │   └── (some files need to be included here)
│       └── _library
│           ├──__init__.py
│           └── (some files need to be included here)
├── setup.py
├── mycodecpy.pyx
└── (some other files)

Please Note that in the mycodecpy.pyx, I do have something related to audiprog.h (as shown below). I really hope that include_dirs will help mycodecpy.pyx find the audiprog.h inside of the AuditoryModel/src/audiprog subfolder.
cdef extern from "audiprog.h":
 # something here

The error I got is:
"C:\Program Files\JetBrains\PyCharm 2018.2.3\bin\runnerw.exe" C:\Users\Andy\AppData\Local\Programs\Python\Python27\python.exe D:/GitHub/LEGION-Nengo/Cython_IPEM/setup.py build_ext --inplace --compiler=msvc
running build_ext
cythoning mycodecpy.pyx to mycodecpy.c
C:\Users\Andy\AppData\Local\Programs\Python\Python27\lib\site-packages\Cython\Compiler\Main.py:367: FutureWarning: Cython directive 'language_level' not set, using 2 for now (Py2). This will change in a later release! File: D:\GitHub\LEGION-Nengo\Cython_IPEM\mycodecpy.pyx
  tree = Parsing.p_module(s, pxd, full_module_name)
building 'mycodecpy' extension
C:\Users\Andy\AppData\Local\Programs\Common\Microsoft\Visual C++ for Python\9.0\VC\Bin\amd64\cl.exe /c /nologo /Ox /MD /W3 /GS- /DNDEBUG -I./AuditoryModel/src/audiprog -I./AuditoryModel/src/library -I./AuditoryModel/src -IC:\Users\Andy\AppData\Local\Programs\Python\Python27\include -IC:\Users\Andy\AppData\Local\Programs\Python\Python27\PC /Tcmycodecpy.c /Fobuild\temp.win-amd64-2.7\Release\mycodecpy.obj
mycodecpy.c
C:\Users\Andy\AppData\Local\Programs\Common\Microsoft\Visual C++ for Python\9.0\VC\Bin\amd64\cl.exe /c /nologo /Ox /MD /W3 /GS- /DNDEBUG -I./AuditoryModel/src/audiprog -I./AuditoryModel/src/library -I./AuditoryModel/src -IC:\Users\Andy\AppData\Local\Programs\Python\Python27\include -IC:\Users\Andy\AppData\Local\Programs\Python\Python27\PC /Tccpu.c /Fobuild\temp.win-amd64-2.7\Release\cpu.obj
cpu.c
c1 : fatal error C1083: Cannot open source file: 'cpu.c': No such file or directory
error: command 'C:\\Users\\Andy\\AppData\\Local\\Programs\\Common\\Microsoft\\Visual C++ for Python\\9.0\\VC\\Bin\\amd64\\cl.exe' failed with exit status 2

Process finished with exit code 1

As you can see, it can't find cpu.c, but I'm sure cpu.c is in the subfolder AuditoryModel/src/audiprog.
If I put all the files into one folder and the same level as setup.py, there would be no problem. I wonder if there is any better way to solve this issue?
I am using python 2.7, windows 10, and Microsoft Visual C++ Compiler for Python.

This is not related to Cython but to distutils/setuptools.
This is not a bug or feature ticket at all, but more of a help request, which is best asked on the mailing list.

You are compiling a module from multiple sources, but you are not providing the correct paths for the source files. Sources are not looked up in the "include" path, which is only for files that the C compiler includes (and that Cython need for the translation). Use the correct relative paths for the source files and it should work.


Changeset UnitTest TestTpchPlanOnly

This changeset contains the new test and the statistics data. It has been tested against the unit test suite and is functional.
This changeset contains a unit test (TestTpchPlanONnly) modeled after
TestTpcdsPlanOnly. The statistics file included to run this test is
tpch/statistics/sf1
and contains statistics data from a 1G run of the TPC-H benchmark.

I will merge this one. Next step is to generate 1g "expected" plan and compare "results" plan during unittest - same as we do for tpch0001.
Thanks,
Qingqing


iOS - Crash when requesting money or splitting bills and then logging into that account

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open New Expensify Beta app
Sign into a valid account (User A)
Split bill and include User B OR request money from User B
Sign out of User A
Sign in to User B

Expected Result:
User expects to log in normally
Actual Result:
The app crashes
EXTRA: This happens more often when using the "Split Bill" option
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] iOS app

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/208921428-3adc6a9c-4320-4c23-8877-83335779d3ae.mp4

Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

I could reproduce this on staging but I couldn't reproduce on dev.

Fixed with https://github.com/Expensify/App/pull/13758


Swaylock, after suspending PC, does not recognize password

Hello guys, I started testing Sway on Arch Linux and everything is awesome. I only have the following (weird) problem:
What I want to do is invoke swaylock after turning on the computer from a suspension (laptop is suspended, I press power button and swaylock appears on the screen). I'm basing it on two scripts I found, which work flawlessly on X11 using i3lock (the systemd setting https://gist.github.com/victorhaggqvist/603125bbc0f61a787d89 and the script to execute swaylock  https://gist.github.com/embik/eef65b532c88c101af3c8b058084e5bc).
The problem is, when I execute that systemd script as the user, and suspend the laptop, when I turn on, swaylock appears, but it seems not to recognize the password, even though I'm typing it right. I can say this is the only case where swaylock acts weird, because I had also configured swaylock to start when I press the lock key, and it works fine. It seems related to the systemd process that executes the script.
I don't know if there is a way to reproduce logs from that on swaylock because I didn't found any debug mode like on sway but I can give you more information. Thanks.

Just in case, I tested the script alone and it works without problems. So it seems it is related to the systemd process that executes the script.

Just in case, I found a way to start swaylocks using systemd. The gist is here https://gist.github.com/WolfangAukang/ffc3f481696c239c5583b5b3a0499e06
So the question is more related to why swaylock isn't recognizing the password

Are you running this as a system unit or a user one? I think you should
run it as the latter.
On Tue, May 9, 2017, at 23:09, Pedro Rodríguez de Oliveira wrote:

Just in case, I found a way to start swaylocks using systemd. The gist
is here
https://gist.github.com/WolfangAukang/ffc3f481696c239c5583b5b3a0499e06> So the question is more related to why swaylock isn't recognizing the
password> — You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub[1], or mute the
thread[2].>

--
Hugo Osvaldo Barrera
Links:

https://github.com/SirCmpwn/sway/issues/1214#issuecomment-300353592
https://github.com/notifications/unsubscribe-auth/AAsmu66MbQLROOXlshdKqnJ2DzTK92DLks5r4RxhgaJpZM4NTNmD

Right now, as a user one (and it's working). Before I made this issue, I executed it as a system unit

Hey @WolfangAukang could you maybe tell me in more detail how you made it work?
I think i tried the same, but I get the following error:
Nov 08 17:57:49 arch-tp systemd[1]: Starting Lock current sway session at device sleep...
Nov 08 17:57:49 arch-tp swaylock[13260]: error: XDG_RUNTIME_DIR not set in the environment.
Nov 08 17:57:49 arch-tp swaylock[13260]: E: 11/08/17 17:57:49 - [registry.c:256] Error opening display
Nov 08 17:57:56 arch-tp swaylock[13260]: sway socket not detected.
Nov 08 17:57:56 arch-tp swaylock[13260]: E: 11/08/17 17:57:56 - [main.c:550] Unable to connect to wayland compositor
Nov 08 17:57:56 arch-tp systemd[1]: lock-screen.service: Control process exited, code=exited status=1
Nov 08 17:57:56 arch-tp systemd[1]: lock-screen.service: Failed with result 'exit-code'.
Nov 08 17:57:56 arch-tp systemd[1]: Failed to start Lock current sway session at device sleep.

my lock-screen.service file is located in /etc/systemd/system/lock-screen.service and I enabled it. It looks like this:
[Unit]
Description=Lock current sway session at device sleep

[Service]
User=MYUSERNAME
Type=forking
Environment=DISPLAY=:0
ExecStart=/usr/bin/swaylock -i /home/MYUSERNAME/pics/MYPIC.jpg --scaling=fill

[Install]
WantedBy=suspend.target

Adding After=suspend.target or Before=suspend.target to the unit file does not change anything..

Try to run it as a user unit instead of system. [This is the service file I created] (https://gist.github.com/WolfangAukang/ffc3f481696c239c5583b5b3a0499e06)

I've been struggling to set this up too..., but now I have it "working"
I have a question though @WolfangAukang, I wonder how you made this work as a "user unit".
From what I see it should not be possible (and I couldn't make it work myself indeed)
From that Arch link you sent just before: systemd --user runs as a separate process from the systemd --system process. User units can not reference or depend on system units.
Which AFAICS means that user unit should not be triggered by the system suspending
Could you please post again the unit file you're using, along with its location on the filesystem?
I'm curious as well about that lock-screen.sh script you have (is it still the same as sent before?)
For me depending on the way swaylock is launched, I needed to remove Type=forking from the unit
Last question :) : when you come back from suspend, do you as well have that problem that first the screen with its applications is shown briefly (like 1sec) before the lock appears?
I tried to change orders with Before: but it doesn't seem to have any effect on that behavior...
Thanks for any pointers
I'm trying to have this run a bit smoother than it is now

Hey @tardypad, it has been a long time I haven't been using Sway with the Arch setup, but as I can recall, if I used it as a system service, I needed to use the root password to unlock the computer. Using it as a user unit fixed that issue for me (although, if running as system worked for you, that's great). I can't give you a good technical explanation of why it worked that way, though.
The lock-screen script is based on this one here (https://gist.github.com/embik/eef65b532c88c101af3c8b058084e5bc), and the service file is the one above your comment. It specifies the user, environment as it is managed differently on Wayland and the forking path is just what I've dragged from another script, without making proper tests of what it does.
I wish I could help you more, but it has been a long time without using my personal laptop with Arch :(

Thanks for the extra info anyway :)
For now I removed this "feature" :/
There were too many minor issues that I didn't feel like spending time on at the moment:

problem when screen is already locked manually before the suspend
screen displayed before the lock on resume
fragile looking scripting


cleanup: simplify Bazel dependency loads

Take advantage of the maybe() utility to load dependencies only if they are defined.  I appeased buildifier by adding an (unused) name argument to the google_cloud_cpp_deps() macro, and documenting why it is there.

This change is

Hmm... I think this may be throwing renovate-bot

Hmm... I think this may be throwing renovate-bot

Say more words... Why?  What do you think is broken?  I am guessing you are referring to the latest gRPC update, which is missing the changes to gRPC?  If so, maybe (he he) I figured it out.  Renovate seems to want the rules to be written without repo_rule=:
https://github.com/renovatebot/renovate/blob/e41681db3d2b4fc2ee47eaa2c6ad5b6a5d82b189/lib/modules/manager/bazel/parser.ts#L264-L266
I will send a CL so we can try that, and then we can rollback if it does not work.

Ack. Thanks for fixing.


Android 8, 8.1 not working

Hi, i using react native onesignal for push notification but it's not woking on Android 8, 8.1 Please help me!

Closing due to a total lack of details. We have an issue template for this reason.

@Nightsd01 I could say his issue is legit. Since I am facing the same problem too. Onesignal in my project works well with other version of android but with 8.0 and 8.1 are not working.
Given the context, I am using "react-native-onesignal": "3.0.9" then I try to use "react-native-onesignal": "3.2.5" and I debugged the error with Android Studio and it showed java.lang.NoClassDefFoundError: Failed resolution of: Lcom/google/android/gms/common/internal/zzbq;
That's all the context. If you need any more information please ask away, I need to solve this problem!
Thank you sir!
@tranvannhat Did you solve it?


diy: add support for TTGO T-Display S3

I expected the T-Display S3 to have identical USB Vendor ID and Product ID as the T-Display, but I discovered this isn't the case.

Thanks for this - will include it asap.

Merged, many thanks.