Chrome logs "Intervention" error when dragging with touch

Repro Steps:
-Use simple example  (https://bertrandg.github.io/angular-split/#/examples/simple-split)
-Use responsive emulator in Chrome and drag with emulated touch (or use device with touch)
-Chrome logs the following error in the console: "[Intervention] Unable to preventDefault inside passive event listener due to target being treated as passive. See https://www.chromestatus.com/features/5093566007214080"

Some solutions for that? Having the same problem with nested split.

I have the same problem... any solution?

I had the same problem. After reading this explanation https://developers.google.com/web/updates/2017/01/scrolling-intervention, where it states "We've found that a large majority of impacted pages are fixed relatively easily by applying the touch-action CSS property whenever possible.",  I was able to suppress the error by adding
as-split {
touch-action: pan-y;
}
to my CSS (for direction="vertical", you might have to adjust to your case).


when run fist_train.sh ,return non-zero exit status 2

error:
(open-mmlab) liuguangcan@ubuntu:~/mmdetection$ ./tools/dist_train.sh configs/htc/htc_r50_fpn_1x_coco.py  2

Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.

usage: train.py [-h] [--config CONFIG] [--work-dir WORK_DIR]
[--resume-from RESUME_FROM] [--no-validate]
[--gpus GPUS | --gpu-ids GPU_IDS [GPU_IDS ...]] [--seed SEED]
[--deterministic] [--options OPTIONS [OPTIONS ...]]
[--cfg-options CFG_OPTIONS [CFG_OPTIONS ...]]
[--launcher {none,pytorch,slurm,mpi}]
[--local_rank LOCAL_RANK]
train.py: error: unrecognized arguments: configs/htc/htc_r50_fpn_1x_coco.py
usage: train.py [-h] [--config CONFIG] [--work-dir WORK_DIR]
[--resume-from RESUME_FROM] [--no-validate]
[--gpus GPUS | --gpu-ids GPU_IDS [GPU_IDS ...]] [--seed SEED]
[--deterministic] [--options OPTIONS [OPTIONS ...]]
[--cfg-options CFG_OPTIONS [CFG_OPTIONS ...]]
[--launcher {none,pytorch,slurm,mpi}]
[--local_rank LOCAL_RANK]
train.py: error: unrecognized arguments: configs/htc/htc_r50_fpn_1x_coco.py
Traceback (most recent call last):
File "/home/liuguangcan/anaconda3/envs/open-mmlab/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/liuguangcan/anaconda3/envs/open-mmlab/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/liuguangcan/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/distributed/launch.py", line 260, in
main()
File "/home/liuguangcan/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/distributed/launch.py", line 256, in main
cmd=cmd)
subprocess.CalledProcessError: Command '['/home/liuguangcan/anaconda3/envs/open-mmlab/bin/python', '-u', './tools/train.py', '--local_rank=1', 'configs/htc/htc_r50_fpn_1x_coco.py', '--launcher', 'pytorch']' returned non-zero exit status 2.
please tall me where i with wrong?
think you vary much

Did you modify tools/train.py? The original train.py has no '--' before 'config' in L23:parser.add_argument('config', help='train config file path')

Have you solved your problem?
You can reopen this issue if the problem still exists, we can discuss it.

This problem occurred because I modified the config，I have solved this problem，think you

How did you solve it in the end？I met the same problem, please help!

Generally, such errors are caused by the wrong modification of the config file or the arg parameter. My problem is that I modified the arg parameter in train.py. If you do not have the same error as mine, can you change Send me your error information in detail so that it is easier to determine where the error is.

Generally, such errors are caused by the wrong modification of the config file or the arg parameter. My problem is that I modified the arg parameter in train.py. If you do not have the same error as mine, can you change Send me your error information in detail so that it is easier to determine where the error is.

请问最后是怎么解决的？我也遇到了同样的问题，请帮忙！

Generally, such errors are caused by the wrong modification of the config file or the arg parameter. My problem is that I modified the arg parameter in train.py. If you do not have the same error as mine, can you change Send me your error information in detail so that it is easier to determine where the error is.


Any instrucctions to run with PM2 or similar?

First of all, kudos on this amazing machine learning example.
We are working on localizing it for LATAM, it's an invaluable resource that we want to use it to show what machine learning is capable of doing.
Since we are expecting some traffic, budo doesn't seem like the right choice, so we are trying to serve it using PM2, with no success.
Do you have any suggestions/alternatives?

Thanks!
The project is made for hosting on AppEngine. You can create a project there, and deploy to it using the gcloud app deploy command line tools for Google Cloud.

Easy peasy, runs like a charm!
https://teachable-machine.appspot.com/
We've just finished translating the wizard and most of the UI, just missing a couple of texts and the custom domain and we are good to go, thank you so much!

perfect!


how to dapr init by using a exist redis ?

i have a redis now ,
i want run dapr init to use my exist redis,is there any idea?
or how to change the redis port by run dapr init?

Bind for <IP_ADDRESS>:6379 failed: port is already allocated.

@bulolo As of right now you can run dapr init --slim to install only the binaries and basic configuration. Please make note that this also skip running zipkin and redis ... https://docs.dapr.io/operations/hosting/self-hosted/self-hosted-no-docker/#initialize-dapr-without-containers


Problem with automation using state changed to:

Dear Gunkut,
using your AxPro integration in home assistant.
Arming and disarming goes flawless. Many thanks for your work.
I run into a problem with automation.
I want to perform actions on disarm and arm.
The actions run fine when I manually trigger them.
When the AxPro is disarmed, the trigger is automatically fired and the command is executed.
However, for arming, it's another story.
In the drop down, I can select Armed away and Armed home as values for the change to:.
However, in the eventlog, it shows that AxPro has changed to 'Armed'. (Not Armed away or Armed Home)
When I select Armed away, or Armed home from the selection list, the trigger is not fired....
When I just type Armed as the chaned to:, the trigger is not fired either.
Am I doing something wrong, or have a stumbled upon a small bug?
Hope to hear from you,
Kind regards,
Erik

I solved it for now, by using the 'changed from' Disarmed in stead of the 'changed to:' Armed

same problem as me, i need to have different status...


Box dimension sample project

I'm trying to locate the sample project mentioned here under the Sensor SDK that measures box dimension.
https://azure.microsoft.com/en-us/services/kinect-dk/
Thanks

When do you plan to share it?

Also asked for this sample.
Or any samples for  world coordinate transform with depth values?


Contrary to help message dump-default-config does not default to stdout

$ rustfmt --dump-default-config
Argument to option 'dump-default-config' missing

$ rustfmt --dump-default-config ''
No such file or directory (os error 2)

$ rustfmt --dump-default-config=
No such file or directory (os error 2)

see #1988
see cbd3608c30cfdee4954820d8ee1f940a4f1abf08 which apparently broke this

I'd like to work on this one.

I worked on this too, solving the second and third example also.

Yeah that is fine by me, I only tried those other ways because the first didn't work. Thanks for the quick fix!


[webapp] Container update command doesn't validate image name

I just noticed that the new Docker Container pane for App Service on Linux instances provides image name validation, which is a really handy feature for preventing subtle config errors:

It would be great if the az appservice web config container update command provided the same validation, so that regardless whether you were managing your app from the CLI or the portal, you receive the same experience. Definitely not a blocker, but I prefer the CLI-driven workflow, and therefore, would love to see parity with the portal as applicable.

As another point of discussion, I just spoke with someone who was running into issues deploying a container to App Service, and the problem was a mis-typed container name, that the CLI didn't validate. This feature request would definitely help guide folks towards success :)

Additionally, the CLI doesn't validate the server URL when pulling an image from a private registry. It requires you to include the scheme in the URL, but you can update with foo.com as opposed to https://foo.com and it will happily accept it. The portal, however, provides a helpful error to fix this.

It is bit more coding involved, that we need to check it is a v1 or v2 registry and use appropriate API to detect.

It would be nice to have an api from the backend to validate the image

CC: @btardif to follow up on the API support for full validation.

This WI if blocked on API Support for both Windows and Linux container scenarios.

@btardif do you know if we have an ETA on the API availability here?

@panchagnula  Backend is targeting ANT85 so moving this out to a more appropriate sprint

we will assign a dev to work on this once the backend work to support this is complete.

Need to move this since we are waiting for a backend API work to be completed for this.

@btardif  do you know if this now available?

@btardif please assign this back to me when the API is ready for use. Thanks!

@btardif just checking if this change is ready and deployed for consumption? Thank you.

@btardif let me know when this is ready.

Closing this issue as won't fix.
Validating a container image in most cases will require permissions on the container repository that under secure environments would only be available to the runtime and not the operator.


Failure to resolve Service

Hello,
I am refactoring my project to modularize it and while I could make things work properly for the project part, now I get the following failure when running my Unit Tests :
Swinject: Resolution failed. Expected registration:
{ Service: MoreOptionsManagerProtocol, Factory: Resolver -> MoreOptionsManagerProtocol }
Available registrations:
{ Service: MoreOptionsManagerProtocol, Factory: Resolver -> MoreOptionsManagerProtocol, ObjectScope: container }
Fatal error: Unexpectedly found nil while unwrapping an Optional value: file
Anyone could help to tell me what is going on ?

This occurs because in Contianer.swift, getEntry fails to return the entry, but I dont understand why. See screenshot.

@taonhannguyen have you solved the problem? I am facing the same

@bartguminiak @taonhannguyen I don't know if it's still relevant for you, but I solved the issue by changing the object scope of the Container to .container:
let container = Container(defaultObjectScope: ObjectScope.container)

I solved the problem by gluing all dependencies in the main app module.
I may be wrong but the defaultObjectScope you mentioned sounds like something else, not related with cross module dependencies resolving

Any updates on this? I'm facing the same issue.

I was having the same issue but just got it resolved. Turns out I was trying to inject AVRouteDetector which is an AirPlay related class that when created on a real device is created successfully, but when running tests in the simulator (which doesn't support AirPlay), the AVRouteDetector class resolved to nil (even though it was a non-optional type).
So for anyone in the future: this error might be that your Container registration is producing a nil object, even though the registration is with a non-nil type.


uninstall? undo the changes ?

is there any way to uninstall or undo the changed made by this tool ? or any documentation on what is doing to know what to roll back

nope, this is a prerelease tool. I'd recommend taking a look at the source scripts for the util(s) that you ran and try to revert that way

@icf20
deleting the tool directly from home dir. deleting the link to .bashrc file  and reinstalling .bashrc.bak file.  did undo most if not all of the changes. iam still testing tho.

did undo most if not all of the changes.

This is true only if you run the bash prompt configuration or any other tools that make modifications to ~/.bashrc, and that's only true if the configuration was run only once. This advice will otherwise not be helpful and potentially confusing.
The most generalized advice for users trying to revert changes would be to examine the shell scripts of the tools that were run to understand and restore the original configuration, if possible.

Yes, reverting is possible via taking a look at the scripts contents and undoing what the script did. As of now me and @adamperkowski and @lj3954 tried implementing this but found out that adding this would break a lot of systems via uninstalling certain deps. So this feature is not intended for now.

/close


Unable to save error in Fossil Sample New Resource

Error in Fossil Sample when trying to save new entry - image of error emailed

This was caused if there was no Geological Age tile. Checking for tile existence first.


Support Angular 2.0.0-rc.0 (2016-05-02)

Good news everyone!
Angular is now in RC0 and it would be awesome if this module could support it.

I have an open PR #481

We're looking forward for support to RC. Is there any forecast ?

Thank you for fast delivery!


add storyshots config

adds a coverage:visual-regression script to package.json, and a storyshots/index.js file to ensure storyshots run
also includes slight modifications to the jest config to ensure the stories work better with jest, and a custom jest config to ensure that coverage thresholds can be loosened a bit
support for window.matchMedia for jsdom added to scripts/setupTests to allow storyshots to work properly
Checklist:

[ ] Documentation N/A
[ ] Tests N/A
[x] Ready to be merged

Codecov Report

Merging #902 (405ae76) into main (8e81743) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #902   +/-   ##
=======================================
  Coverage   99.39%   99.39%
=======================================
  Files         114      114
  Lines         825      825
  Branches      219      219
=======================================
  Hits          820      820
  Misses          5        5

Impacted Files
Coverage Δ

components/footer/src/index.tsx
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

yeah - I did spot the linting issue - my fix to the footer component triggers the rule - doh!
my instant thought was to disable the rule, possibly globally, as I'm not convinced the rule is useful.  haven't managed to give it much thought yet tho - blanket disabling the rule feels like it might be a bit drastic

new comment was added to the storybook thread https://github.com/storybookjs/storybook/issues/15378#issuecomment-884668183, so have updated this accordingly, restoring the defaultProps and changing the default story definition to fix the previous error


Unitrack on window10?

Hello, can I run the UniTrack demo file on window10 if I set the environment like ubuntu?

Hi, I think it's possible to run on windows if you are using WSL (Windows Subsystem for Linux).

Thank you! I will try it.


Support for Hugo i18n translation files or a top level list?

Discussed in https://github.com/tinacms/tinacms/discussions/4197

Originally posted by major-mayer September  9, 2023
I recently tried to migrate my Hugo Website from Forestry to TinaCMS and have a few problems.
One of it is that the document sections is still not supported as it was in Forestry: https://github.com/tinacms/tinacms/issues/1982
I especially need it for the translation files of which I have a few of.
They are looking like this:
---
- id: nav_button
  translation: Kontakt
- id: service_subtitle
  translation: Entdecken Sie
- id: service_title
  translation: Unsere Angebote

I tried manually creating a schema, but as this is a top-level list and you can not specify fields of type list, i found no way how to create a schema.
I tried adding a "translations" object on the top level and putting the list below as a workaround:
fields: [
          {
            type: "object",
            name: "translations",
            label: "Übersetzung",
            list: true,
            fields: [
            {
              type : "string",
              name : "id",
              label  :"ID"
            },
             {
              type : "string",
              name : "translation",
              label : "Translation"
             }
            ],
          },
        ],

But when I add "translations" as the top-level object, Hugo's i18n tooling cannot find the translations anymore.
Any idea how to solve this problem?

Seems like I am not the only one affected by this.
Any workaround for us until we can have a proper solution?

@major-mayer currently no work around. TinaCMS would need to support "Top level lists"

Okay thanks for the confirmation. Then I would like to make this a feature request.

I'm going to close this issue, and re-open: https://github.com/tinacms/tinacms/discussions/4243
(We try and keep feature requests inside GitHub Discussions)


Wrong count of emails

It looks like https://github.com/ProtonMail/WebClient/issues/79 but may be a different issue.
Describe the bug
The number of unread email / conversations is not always right.
To Reproduce
Unknown sadly.
Expected behavior
I expect the counter, between parenthesis, to actually reflect the number of either email or conversations in the said folder.
Screenshots
In this screenshot, it may be a bit hard to tell, but on the right side there's my inbox with a few emails. Two of them are unread (white, the 7th and 9th message if first is one). But the count tells (1):

In this specific case, if I mark as read one of them, the count drops to 0. I tried on both. One of them is a conversation, the other is simply an email.
Desktop:

OS: Debian bullseye/sid
Browser Firefox 68.0.1
Plugins Privacy Badger, Passbolt.
Resolution 1920×1080
URL mail.protonmail.com

Additional context
I use my session for several days, so as the time of writing my uptime is 25 days, my Firefox process was started 9 days ago, I did not reloaded the Protonmail tab during those 9 days.

Happened again today, right after an "Empty spam", left column show I have 5 spams and right column displays a single one (Which is probably also another bug, as after clicking "Empty spam" I expect to see 0, not 1 (yes, it disappeared like 10 or 20 seconds later by itself and the count went to 0 (dissapearing))).


Online-Status changes on mouse move

My Setup

Operating System: Windows 10 + Mac OS
App Version: 2.10.5

[x] I have tested with the latest version
[] I can simulate the issue easily -> we can't bring the bug to work on a different system with different or the same accounts

Description

Current Behavior

The Bug is related to the App. In our working environment 3 guys have these bug. It appears on Windows 10 and Mac.
The user is always 'away' instead of 'online', except he moves his cursor in the Rocket.Chat App
Expected Behavior

The User should stay 'online' :)

A reinstall doesn't help.
The only thing that helped was:

uninstalling rocketchat
delete all files in C:\Users\username\AppData that are found with "Rocket"

I think that isn't a good fix for this bug

In our configuration we had setted the Idle Time Limit to 300 seconds
Now with the update of the server it had the value "0.3 ms"
So we changed it to 300000 ms and now this bug is gone.
We dont know what the problem was exactly, but it is gone...

From my understanding this Bug an #1147 are different
This one is talking about that the time value is handled incorrectly but the option is set to "true". So going to the away status is wanted
On ##1147 the option is set to "false" and its switching to away but its not wanted


JAMES-3516 Plug threadIdGuessingAlgorithm to MessageManager

I have some questions:

Am I supporting threadId for other protocols beside JMAP also? I am interacting with how James store threadId generally.
Should we have CassandraThreadIdGuessingAlgorithm and InMemoryThreadIdGuessingAlgorithm... separately in the future?

It's failing somewhere else related to JPA, still working on this.
Look like we are missing threadId at JPAMailboxMessage...

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.22.2:test (default-test) on project apache-james-mailbox-cassandra: There are test failures.

java.lang.NullPointerException: threadId is required (252 occurence...)

The related JPA tests fail is real. But these cassandra tests fail seems weird... they are green on local.

Pro-tip: git rebase master & rerun ;-)

Rebased on master + squash fixups


Gastby SSR construct

Requests:

truecolin https://discord.com/channels/983865673656705025/983866416832864350/1074475380758544524

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Bug Report: crashing when Snap app window

Environment
Windows build number: 10.0.18362.175
Windows Terminal version (if applicable): 0.2.1715.0

Any other software?

Steps to reproduce
Open Windows Terminal, snap window to side or corner of desktop, attempt to interact with Terminal app with no response, then app crashes and closes unexpectedly.
Expected behavior
Snapping Windows Terminal window to corner or side of desktop, then interact with app as usual.
Actual behavior
After snapping app window, app becomes unresponsive, then crashes and closes unexpectedly.

I can confirm this issue. Is happening to me too. I've got same build

Strange. I don't have the issue, and I am running the same Windows Terminal Version. Does anyone have a dump from msvsmon.exe when the crash occurs?

I only have Event Viewer application errors logs.
Errors seen grouped by WindowsTerminal.exe app crash then pwsh.exe .NET Runtime crash. Have PowerShell core as the default profile.
Since first reported, I've had the "2019-05 Cumulative Update for .NET Framework 3.5 and 4.8" update installed and a reboot and now no longer experiencing the issue.

tried for all terminals within the treminal app, couldn't reproduce the behavior. as u mentioned this probably is causing due to the version of the .Net framework installed in your system machine.
If anyone is having the same problem try Updating .Net Framework your system machine to the latest It should resolve this issue.

I'm going to need a crash dump here to go further. I tried to reproduce it locally and could not.
If you have the event viewer log of the crash in the Application log with event ID 1001 (Windows Error Reporting) and can give me that, I might be able to find the crash ID in our reporting system and figure this out from that.

@miniksa I've attached the application logs when the Terminal app was crashing, as requested. There seemed to be a couple of different WER events for the app so they are included. Let me know if you need any thing else.
AppLog-WinTerminalCrash.zip

So there are mainly 2 problems from the logs I see. PowerShell.exe and pwsh.exe are both crashing due to the console input being redirected somehow. And WindowsTerminal.exe is crashing due to access violations. The module is TerminalControl.dll. Can someone use just in time debugging when the crash occurs and drop the VS debugger output?

See this guy? The hashed bucket number? I looked in the one with WindowsTerminal.exe crashing.

I can find the crash analysis on the back-end with that.
Crash analysis tells me that this is the stack of the failure...
0c (Inline Function) --------`-------- TerminalControl!Microsoft::WRL::ComPtr<ID2D1RenderTarget>::operator->+0x7 [c:\program files (x86)\windows kits\10\include\10.0.17763.0\winrt\wrl\client.h @ 316]
0d 0000004e`0edff7e0 00007ff8`04d8f892 TerminalControl!Microsoft::Console::Render::DxEngine::StartPaint+0x198 [c:\ba\2009\s\src\renderer\dx\dxrenderer.cpp @ 689]
0e 0000004e`0edff830 00007ff8`04d8f7da TerminalControl!Microsoft::Console::Render::Renderer::_PaintFrameForEngine+0x72 [c:\ba\2009\s\src\renderer\base\renderer.cpp @ 84]
0f 0000004e`0edff920 00007ff8`04d8f552 TerminalControl!Microsoft::Console::Render::Renderer::PaintFrame+0x6a [c:\ba\2009\s\src\renderer\base\renderer.cpp @ 65]
10 (Inline Function) --------`-------- TerminalControl!Microsoft::Console::Render::RenderThread::_ThreadProc+0x40 [c:\ba\2009\s\src\renderer\base\thread.cpp @ 165]
11 0000004e`0edff950 00007ff8`3db77bd4 TerminalControl!Microsoft::Console::Render::RenderThread::s_ThreadProc+0x52 [c:\ba\2009\s\src\renderer\base\thread.cpp @ 148]

... checking the DxEngine object in the crash is showing me that _d2dRenderTarget is null at this point.
The line blamed is DxRenderer.cpp @ 689. Which doesn't make sense for a nullptr deref on master, but it does make sense as of this commit time: https://github.com/microsoft/terminal/blob/9b92986b49bed8cc41fde4d6ef080921c41e6d9e/src/renderer/dx/DxRenderer.cpp
Now reasoning this out.... There are three paths above to calling this code.

If we don't _haveDeviceResources we called _CreateDeviceResources(true). Looking inside that method, it would have to successfully create and store a _d2dRenderTarget in order to return successfully and then for StartPaint to get down to 689... so it wasn't this.
If clientSize changed, we reset the _d2dRenderTarget and then tried to remake it with _PrepareRenderTarget (which is the same method called inside _CreateDeviceResources(true).) At which point, we should have a valid _d2dRenderTarget if we got this far or it would have returned a failure code and we would have bailed.
The third route, the else case, which isn't set. We supposedly had device resources and didn't change the size, but the _d2dRenderTarget is gone anyway! How the heck did that happen?

OK. Let's find all places where the contents of _d2dRenderTarget is modified.

Line 263, creation of the render target in _PrepareRenderTarget
Line 308, reset of the smart pointer when _ReleaseDeviceResources is called
Line 683, reset of the smart pointer when ResizeBuffers-ing the _dxgiSwapChain during StartPaint.

For Line 263, _PrepareRenderTarget appears to be always wrapped in a result code checker and so are the methods that use it. So I don't think it's that one.
For line 308, _ReleaseDeviceResources first operation is to set _haveDeviceResources to false. But if it got to this one and _haveDeviceResources was false, then we wouldn't have hit the third route above. We would have created them first and errored if they weren't created. _haveDeviceResources isn't set to true until the bottom of _CreateDeviceResources after all the early return errors. So the target should have existed.
For line 683, this is among the newest things going on. I added this code more recently than the other bits to attempt to resize without unspooling the entire device resources pipeline and recreating it because it was expensive (and was probably causing some sort of flickering or performance nightmare. I forget exactly, but it had to be something like that.)
So I'm going to suspect line 683 here or the nearby code.
Reasoning this out... I'm first concerned because the ->ResizeBuffers call is returning an HRESULT and it's not being checked in any way. This means if it fails, we're not reporting it.
Secondly, I'm detaching the _d2dRenderTarget on 683 then attempting to resize and recreate it while leaving the state of _haveDeviceResources as true. If line 685 threw an error and returned (because it couldn't create the new _d2dRenderTarget... then we could end up in a torn state. The _d2dRenderTarget was reset but the boolean still seems to think it exists.
That sounds very plausible.
So what I'm going to do here is go on the assumption that this is the problem. I don't want to "throw out the baby with the bathwater" by undoing this change which must have been for good reason. I'm just going to try to make it more robust.
I'll set up a situation where:

We check ->ResizeBuffers for a failure because it probably should be and RETURN_IF_FAILED if it is bad.
If ResizeBuffers or _PrepareRenderTarget fails while we're in this somewhat torn state (where I only released part of the device resources but haven't set _haveDeviceResources to false yet), I'll call _ReleaseDeviceResources on the way out to make sure everything gets cleaned up and is recreated from new the next time around.

I'm not certain exactly what the problem is while creating the render target, but this proposal should stop the crash from happening by eliminating the torn state.
After that, if there's still weirdness when snapping the window, we should be able to reproduce it under the debugger and see what error codes are being thrown by DirectX (because the RETURN_IF and LOG_IF macros should be reporting those failures when attached to a debugger in the output window) and go from there.

Thanks again for the report! This was just submitted to the store with the v0.2.1831.0 servicing release. It may take some time for the store to process it.

@DHowett-MSFT thanks for the update, that's great!


feat(frontend/expr): extract common factor from boolean expression

What's changed and what's your intention?
For expressions like (A & B) | (B & C), we can extract its common factor and turn it into (A | C) & B. And the conjunction form will become [(A | C), B]. By applying this kind of rewrite rule, each item in the conjunction form would have fewer InputRefs.
This pr adds support for common factor extraction.

Describe any limitations of the current code

Since that Subquery can't be Hashed or Eqed, this pr only applies common factor extraction for expressions that don't have Subquery.
HashMap does not preserve the order of the element, so the order of the expressions might be changed. But this does not affect the correctness.
Checklist

[x] I have written the necessary docs and comments
[x] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)
This pr close #1509.

Seems that the order of the expressions changes across each run, so the CI would not pass.:rofl:  I will try to make it stable.


Roblox Crashes Randomly

(using 64 bit byfron version of roblox using bloxstrap) Game randomly crashes at some random point, it freezes then crashes. Ill list an example.
https://user-images.githubusercontent.com/119818604/236091947-eb52f6be-41a7-4b7f-94cb-7725838a0bb9.mp4
It freezes, then later Roblox closes. Im not sure if this is a bloxstrap issue, because I am using custom fflags different from bloxstraps built  in fflags.

I've heard of other people experiencing this same crash with the new 64-bit client. Uninstall Bloxstrap to revert back to the original bootstrapper and see if it's still happening.

I've heard of other people experiencing this same crash with the new 64-bit client. Uninstall Bloxstrap to revert back to the original bootstrapper and see if it's still happening.

im using the original bootstrapper and i still experience crashes

I've heard of other people experiencing this same crash with the new 64-bit client. Uninstall Bloxstrap to revert back to the original bootstrapper and see if it's still happening.

im

using the original bootstrapper and i still experience crashes

same issue

use zcanary channel its a temp fix

use zcanary channel its a temp fix

doesn't work

didn't even let me play this time

it makes me crash randomly without message

set it back to 32 bit

Closing this since there's not much I can do about this, and it's a few weeks old at this point. Though, since then, I have made this Wiki page:
https://github.com/pizzaboxer/bloxstrap/wiki/Information-on-Roblox-crashes

similar issue when i minimize my roblox tab it freezes and only the sounds could be heard very annoying when i need to close the tab to do something

Stop using Vulkan

send fflags


Revert "Remove the test plan packages"

This reverts commit 844b142346166369e50a106c4bbd3a12487a0860.

+1 for adding this back in. I would like to live without a ppa but we need another release before we can do that.


Switch desired thrust to WrenchStamped

The desired thrust is today sent from controllers to the thrust allocation by a non-stamped wrench message. This is a problem for later data analysis since the messages do not contain a timestamp. By switching from a Wrench message to a WrenchStamped message there will be a timestamp included with each message.

since rosbag adds its own timestamp, this is no longer needed


Remove TagSearchable class

Supersedes #2436, with differences / additions as follows:

get_indexed_objects was dropped in #2498 and no longer needs to be moved
popular_tags is now the helper function wagtail.wagtailadmin.utils.popular_tags_for_model instead of a class method
the deprecation warning now uses the correct stacklevel parameter to point back to the calling module
references to TagSearchable in migrations have been removed (and a release note added telling users to do the same for custom image/document models)

Rebased and merged in 367c71dbaa070e18012d2c9d431ee17cc500c572 +parents. Some minor coding style tweaks made in 7d490f7a6fa5fa127b396584389c0f70db167b8d
Cheers! :beers:


Could not establish SSL connection in docker container

Hi,
Recently I started using Azure AppConfiguration, and I like it.
I created a very simple demo project (dotnet core console app) just to write out the value
Code:
var builder = new ConfigurationBuilder();
            builder.AddAzureAppConfiguration(options =>
            {
                options.Connect("ConnectionString")
                    .Select("test:*", "test")
                    .TrimKeyPrefix("test:");
            });
            var config = builder.Build();
            Console.WriteLine(config["test"]);

When I run the app on my local machine, everything works just fine, the app itself downloads all the configuration from Azure and work in order.
But when I try to package app into a docker image and run it in the container I get following error:
(The SSL connection could not be established, see inner exception.)
 ---> Azure.RequestFailedException: The SSL connection could not be established, see inner exception.
 ---> System.Net.Http.HttpRequestException: The SSL connection could not be established, see inner exception.
 ---> System.Security.Authentication.AuthenticationException: The remote certificate is invalid according to the validation procedure.

A container always crash with this message.
I publish app with command:
dotnet publish -c Release -o out
Dockerfile:
 FROM mcr.microsoft.com/dotnet/core/aspnet:3.0
 WORKDIR /app
 COPY /out ./
 ENTRYPOINT dotnet name-of-app.dll

I tried using both Windows and Linux containers, but I always end up with the same error.
Do you ever encounter this issue?

@zhenlan we're getting this same error when not running in a docker container.  Some dev workstations work without issue, others get the SSL connection error referenced above when calling builder.AddAzureAppConfiguration

The issue, in my case, was the company firewall. After a call with our network administrator, everything works as it should.
Double check your networking, I lost lots of time on such a simple problem with an easy solution :)


[WIP] attempt to strip down redundant build processes & build fix

WIP!
Description
Here's an attempt to strip down all unnecessary/redundant build processes. The main thing I've done here is to remove the command executor and replaced it with @nrwl/js:tsc using the build command instead of _buildTsc
Time reduced to ✨  Done in 30.57s.

yarn build
DEBUG=true yarn test:e2e:node --filter=test-lit-action-1-sig.mjs

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

 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.1 out of 2 committers have signed the CLA.:white_check_mark: Ansonhkg:x: GarandorYou have signed the CLA already but the status is still pending? Let us recheck it.

Closed for a better commit history here https://github.com/LIT-Protocol/js-sdk/pull/387


page turning problem

Hi~
It's a nice program [DLSiteDumperGUI] !!
Thank you very much.
But I have a problem with using...
[ left page (<-) ] There is no problem.
[ right page (->) ] Only the first page... ,
The program will show that it has successfully stopped ..
How can I solve this problem?
Because the issues cannot be opened..
So I can only post here..

Ahh sorry I never noticed someone posted an issue for DLSiteDumperGUI.
And it never send me an email. >_<
Since it is 1y old now, I hope you already got what you want now. Sorry again!


Test GitHub Actions failure

Just a test to see that our GitHub Actions workflow is broken for no reason.

OK, found the fix. Reopening as #10791.


[map] Map tooltip overflows on narrow windows

need better way to handle the tooltip for the map because it sometimes overflows on narrow windows
https://screenshot.googleplex.com/7KnFK3mCVuddWTm

this is fixed


FlGridData vertical line not showing

Hello,
Thank you for the great library.
I found a problem while using this library. Vertical Gridline is not showing using this code:
gridData: FlGridData(                       show: true,                       getDrawingHorizontalGridLine: (value) {                         return FlLine(                             color: ColorRes.button_border, strokeWidth: 1);                       },                       checkToShowHorizontalGrid: (value) {                         final val = _maxY / 100;                         if (value % (val.floor() * 50 / val.floor()) == 0) {                           return true;                         }                         return false;                       },                       getDrawingVerticalGridLine: (value) {                         return FlLine(                             color: ColorRes.button_border, strokeWidth: 5);                       },                       checkToShowVerticalGrid: (value) => true,                       drawHorizontalGrid: true,                       drawVerticalGrid: true),
This is the result. Only horizontal grid is showing.

Hi, since 0.3.4  version, we swapped the horizontal and vertical functionalities, maybe this is your problem, check it out.

Any answer?

Heloo?

@imaNNeoFighT I tried switching to  0.3.3. drawVerticalGrid affects Horizontal line. But drawVerticalGrid=true doesn't show the vertical grid.

if drawVerticalGrid=true it should draw the vertical line.(green)
and for drawHorizontalGrid=true it should draw the horizontal line.(blue)

You are correct, but it is not important to made breaking changes.
I will fix them at a proper time.
Thank you!

Thanks @imaNNeoFighT

You're welcome!

Hi there, It has been renamed in 0.6.0.
Thanks for reporting :)

I have the same problem. Version 0.36.1

@Serty32 What is your problem exactly?

@imaNNeoFighT Vertical lines are not visible.

Okay got it. we decided to remove the horizontal value concept from the BarChart. That's why we don't show anything.
Feel free to make a new issue to follow up on this decision. I need to understand the use case (Because I think in the barChart only yAxis is important).

Hey please can i get the bar chat code you used to create this?


Tool adaptation and support

For all of those looking at this repository, @CybercentreCanada has "forked" this tool into its JsJaws service for Assemblyline and will be providing support and maintenance for their version of the tool: https://github.com/CybercentreCanada/assemblyline-service-jsjaws/tree/main/tools/malwarejail/env/wscript
Thank you @mrpapercut  for your work on this tool!
🇨🇦

Hi Kevin,
This makes me happy, I'm glad you've found a use for this tool. As you may have noticed, I haven't maintained this repo since 2017. Soon after I finished it, my situation changed and I no longer had much use for it. I hope the code is clear enough to aid you in modernizing and maintaining. All the best!
Kind regards,
Mischa Rodermond


Angular Playground fails to compile

Hi Dan
I was interested to see if you are able to compile and run Angular Playground after upgrading to Angular 10. I am seeing an issue in my projects so was interested to see if anyone else was experiencing the same problems. When I get your latest code and run 'npm run playground' I'm seeing the following:
ERROR in The target entry-point "angular-playground" has missing dependencies:

C:/Users/jdutton/Desktop/Angular-JumpStart-master/Angular-JumpStart-master/src/app/about/about.component.sandbox
C:/Users/jdutton/Desktop/Angular-JumpStart-master/Angular-JumpStart-master/src/app/customer/customer-details/customer-details.component.sandbox
C:/Users/jdutton/Desktop/Angular-JumpStart-master/Angular-JumpStart-master/src/app/customer/customer-orders/customer-orders.component.sandbox
C:/Users/jdutton/Desktop/Angular-JumpStart-master/Angular-JumpStart-master/src/app/customers/customers-card/customers-card.component.sandbox
C:/Users/jdutton/Desktop/Angular-JumpStart-master/Angular-JumpStart-master/src/app/customers/customers-grid/customers-grid.component.sandbox
C:/Users/jdutton/Desktop/Angular-JumpStart-master/Angular-JumpStart-master/src/app/customers/customers.component.sandbox

When I browse to http://<IP_ADDRESS>:4201/ I see an error.
I am also seeing this in all of my projects (different dependencies obviously).
Are you seeing the same problem when you compile and run locally?
Thanks very much.
James

I'm seeing the same error. Sounds like we're not the only ones from what I'm seeing in their repo.

Looks like there's an easy fix. See https://github.com/SoCreate/angular-playground/issues/262#issuecomment-668990093.


Enabled saving images after scanning

Merged changes from different branches to allow image saving and viewing QR codes scanned by other users

LGTM!


add postcss-will-change-transition plugin

Hi!
I think i have problem with yarn run add. Npm gave same result :(
I can change authors list manually if you will accept PR then

The authors list is actually currently broken, and I am trying to fix it soon, so don't worry about it right now :)


opts argument in select() is not passed to selectCollection()

The opts object in select() https://github.com/devvmh/redux-crud-store/blob/master/src/selectors.js#L43 is currently not being passed to selectCollection() https://github.com/devvmh/redux-crud-store/blob/master/src/selectors.js#L51.
This means select(..., { interval: 1000 }) has no effect for selecting collection.

Thanks for the detailed report!
I've pushed 0552ed0d58f601a470301e2f8fa3134c47a0a4cf which should fix this. I'm publishing 5.3.0 now. Please try this release and let me know if it works

I'll close this assuming 5.3.0 worked for you. If you still have issues please re-open this 😃


[4.2] FileHandle.truncateFile should reposition on truncate

Accepting truncating to any position, not just 0.
Reposition to the truncate offset.
Darwin throws exceptions on error so we use fatalError() instead of silently failing.

(cherry picked from commit fb1e71bbe58329267b799695ff8ea1d307a752d2)

@swift-ci test 4.2


Print....Line 526..

Hi, thanks for sharing this with us....
I think I see none of the "print" using "()", which prompts errors on our end,
Also, after adding all parentheses ..still seeing the line 532 error...
else:
SyntaxError: invalid Syntax
Using python 3.6, 3.7, running into same errors.
Not sure what your logic is, but I bet the nested if is the problem...
Thanks again

@tonynotears this script was developed for Python 2.x which print statements don't use brackets. You should run the script on Python 2.7

There is a pending pull request that will add Python 3 support thanks to @juanvelascogomez

I just merged @juanvelascogomez code which adds Python3 support


Docker improvement for proxied network

Chore
Describe the chore
The dockerfile in its current state fails at multiple states on VPN

curl calls needed -k
python pip needs CERT changes
git needed skip SSL checks

Additional context
So, I have modified the dockerfile, it is makes sense I can make a PR.
FROM postgres:13

RUN apt-get update
RUN apt-get install build-essential git cmake curl -y
RUN apt-get install postgresql-server-dev-13 -y
# Required by libgraphql
RUN apt-get install python2 -y
RUN apt-get install python3-pip -y # using this to install pip
RUN pip install --trusted-host pypi.org --trusted-host pypi.python.org --trusted-host files.pythonhosted.org ctypesgen # trusted hosts

RUN git config --global http.sslVerify false # setting SSL verify to false

RUN git clone https://github.com/graphql/libgraphqlparser.git \
    && cd libgraphqlparser \
    && cmake . \
    && make install

ENV LD_LIBRARY_PATH="/usr/local/lib:${PATH}"

COPY . pg_graphql
WORKDIR pg_graphql
RUN make install

@hemanth based on Div's feedback I'm going to mark this as wontfix and close but please follow up if we're misunderstanding something

MITM'ing SSL traffic might be an edge case, sure, thanks.


Renames. Delete HandlerConfig, Codegen SetXXXX mtds

Next step is to use the new SetXXXX mtds from the Mixer's hand written code (HandlerFactory.go and config/handler.go)
Release note:

release-note-none

This change is

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
2799c6952e840c2046ef9bef059d28891159c847
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/retest

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
2799c6952e840c2046ef9bef059d28891159c847
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
3850656ab394e7176afbd014d0561ebc7eeae60a
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
3850656ab394e7176afbd014d0561ebc7eeae60a
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
3850656ab394e7176afbd014d0561ebc7eeae60a
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
fb1db742f4708c52eb0df29ebb2001d9464331f0
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/test all

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
fb1db742f4708c52eb0df29ebb2001d9464331f0
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
fb1db742f4708c52eb0df29ebb2001d9464331f0
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
9bda9583e428cf2e702a169ec34a3da732c9a456
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
9bda9583e428cf2e702a169ec34a3da732c9a456
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
9bda9583e428cf2e702a169ec34a3da732c9a456
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
d594a3962509e3e83cd48b6db66a6f2224f08ad2
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
a6e8b94c9217394139f41acb5c9bfdc19426690d
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
a6e8b94c9217394139f41acb5c9bfdc19426690d
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

e2e test result:
./e2e.sh  -logtostderr -test.v -v 4 --mixer_hub=gcr.io/istio-testing --mixer_tag=a6e8b94c9217394139f41acb5c9bfdc19426690d
PASSED: //tests/e2e/tests/mixer:go_default_test
PASSED: //tests/e2e/tests/bookinfo:go_default_test

Adding do not merge since this PR is blocked on https://github.com/istio/istio/pull/679/files

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
02cb3243a6f745d372c11290c6b637d21587a227
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mandarjog can you please force merge, once your run of this PR finishes successfully.
e2e test is passing
./e2e.sh  -logtostderr -test.v -v 4 --mixer_hub=gcr.io/istio-testing --mixer_tag=02cb3243a6f745d372c11290c6b637d21587a227
Tests Summary
PASSED: //tests/e2e/tests/mixer:go_default_test
PASSED: //tests/e2e/tests/bookinfo:go_default_test

/lgtm

@guptasu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
02cb3243a6f745d372c11290c6b637d21587a227
link
/test mixer-e2e-smoketest

prow/mixer-presubmit.sh
92b5c9251c3efa8ad8867160874a4d6c24b4a7de
link
/test mixer-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

@mandarjog @douglas-reid
e2e passed
./e2e.sh  -logtostderr -test.v -v 4 --mixer_hub=gcr.io/istio-testing --mixer_tag=b03500989348b4e10fb9cff1fc201fa757a793d3
Tests Summary
PASSED: //tests/e2e/tests/mixer:go_default_test
PASSED: //tests/e2e/tests/bookinfo:go_default_test

@guptasu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
02cb3243a6f745d372c11290c6b637d21587a227
link
/test mixer-e2e-smoketest

prow/mixer-presubmit.sh
96c4389893c17ddbff6c1df848f242c31c0df768
link
/test mixer-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@guptasu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
02cb3243a6f745d372c11290c6b637d21587a227
link
/test mixer-e2e-smoketest

prow/mixer-presubmit.sh
96c4389893c17ddbff6c1df848f242c31c0df768
link
/test mixer-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
96c4389893c17ddbff6c1df848f242c31c0df768
link
/test mixer-e2e-smoketest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

e2e passed
./e2e.sh  -logtostderr -test.v -v 4 --mixer_hub=gcr.io/istio-testing --mixer_tag=96c4389893c17ddbff6c1df848f242c31c0df768
Tests Summary
PASSED: //tests/e2e/tests/mixer:go_default_test
PASSED: //tests/e2e/tests/bookinfo:go_default_test


API/ENH: support datetime.time in a Timedelta

So this as expected. Though I its pretty straightforward to add support for this directly to a Timedelta, after the notion is the same, though a time object can support a time-zone so not sure if this is odd (or maybe we should add an optional tz to Timedelta).
In [5]: from datetime import time

In [6]: time(secIn [8]: pd.Timedelta(time(second=1))
ValueError: Value must be Timedelta, string, integer, float, timedelta or convertible

In [9]: pd.Timedelta('1s')
Out[9]: Timedelta('0 days 00:00:01')

What is the best way to convert a column of times to a timedelta? As
pd.to_timedelta(hours=df['time'].dt.hour, minutes=df['time'].dt.minute, seconds=df['time'].dt.second)

or
df['time'].apply(lambda x: datetime.timedelta(hours=x.hour, minutes=x.minute, seconds=x.second))

seems not very concise

actually this is a dupe of  #8343

as I said, I think this type is useless. Those solns's are fine for the very limited amount this comes up. An in-between soln is to allow conversion directly to Timedelta which is not that hard.

@jreback I think it are two different things, how I understand it.
In this issue, it is proposed that you can provide a datetime.time object to pd.to_timedelta to convert it to a timedelta (just as you can also provide strings, ints, ..).
#8343 is about integrating datetime.time in the pandas internals as a timedelta (which will lead to automatic conversion of datetime.time to timedeltas), which is a much more profound question (and not something I am in favor of)


Search beyond workspace root when discovering configuration

Summary
Previously, we wouldn't respect configuration files in directories above a workspace root. But this is somewhat problematic, because any pyproject.toml will define a workspace root...
Instead, I think we should start the search at the workspace root, but go above it if necessary.
Closes: #5929.
See: https://github.com/astral-sh/uv/pull/4295.

I think if we search above a workspace, though, we may want to consider ignoring pyproject.toml and only looking at uv.toml.

I think if we search above a workspace, though, we may want to consider ignoring pyproject.toml and only looking at uv.toml.

I think that makes sense

I'll do it separately.


adding the EN_DEMAND value to the ENsetnodevalue function

Added the possibility to set Demands programmatically. Please confirm if it's right.
Regards.

closing as redundant. Thanks for getting involved! there are many issues under discussion right now that will need PRs, so feel free to browse and volunteer.


Update fabric.mod.json to support 1.21

The mod works as intended in 1.21 without any other changes.

I only meant for the commit to fabric.mod.json to be sent in

Thanks for that, i'll get working on it officially soon!

Here is the official 1.21 release!
1.21 Release


Special deployment considerations needed?

Thank you for creating a plugin for blitz-js, this makes my current project much easier!
What I did not understand from the docs, or the PRs, is how an application deployment with prisma-multi-tenant works.
For example, the docs for blitz feature Vercel and Render, however Render only gives you one "database" (which looks like the result of CREATE DATABASE) per instance of their paid offering.
What I'm wondering now is, how does PMT implement tenancy exactly regarding this?
Is each "database" in PMT an actual (Postgres) DB, meaning I need to deploy either multiple nodes, or services if managed. Or is it using some kind of schema modifications to implement tenancy inside one instance of a postgres DB?
Sorry if this has been asked before, especially given the existance of #31 and #40
Thanks for your hard work!

Hi @thuringia 👋
Thanks for the question. The current architecture of prisma-multi-tenant is indeed based on the "one tenant = one database" model, as well as another database for the "management" of your tenants. This means that you would indeed need to have multiple databases. (I don't know Render, but it does look like you can only have one database indeed)
However, as you pointed out, once #40 is implemented, you will be able to use prisma-multi-tenant with a "one database = multiple tenants" model, which would let you deploy on a single database. (I haven't had a lot of time this past couple month to work on pmt, I'll try to find more time in the next couple weeks 😄 )


modify readme example

let the example more clear.

Codecov Report

Merging #46 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
- Coverage   18.09%   18.08%   -0.01%
==========================================
  Files          54       54
  Lines        3167     3168       +1
==========================================
  Hits          573      573
- Misses       2594     2595       +1

Impacted Files
Coverage Δ

test/readme_example.py
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Unprivileged normalize API

Description
When using the current normalize API, we require elevated permissions. This is due to the usage of map_files.
Here is for instance the failing syscall from a strace:
4122778 openat(AT_FDCWD, "/proc/4122778/map_files/61bc0125c000-61bc0125e000", O_RDONLY|O_CLOEXEC) = -1 EPERM (Operation not permitted)

We could propose a normalization API that works from source.
The logic would be similar to what we propose here
This uses the mapping information (from proc/<PID>/maps) and the segment information from the file. Finding files associated to mappings can be a challenge. So perhaps it should be the responsibility of the user, to provide the matching elf file.
Motivation
This would allow us to use the API within a process (or a child process) to symbolize itself (or the parent).
Our use case is to annotate crashes or errors with associated symbols. From the associated stack trace, we could normalize and symbolize from source.

This is somewhat a different use case from the issues I previously submitted. Thanks again for being so fast in reacting and answering my issues.

Yeah, I can see it being useful to have a solution for cases where fewer permissions are available. We changed this behavior with 327f2884205d5f3e7b24dc8b06f9bd9a40d71f2f and effectively requiring root has certainly not been a great characteristic to have ever since.

We could propose a normalization API that works from source.

What are you referring to when you say "from source"?
We still keep what we call the symbolic path around, so we could introduce an option to use that instead of the maps_file. I can't think of a better API design off-hand.

@r1viollet I created https://github.com/libbpf/blazesym/pull/520, which I'd think should cater to your use case by setting the map_files argument to false in the blaze_symbolize_src_process object. Feel free to give it a try and let us know if it works for you.

Closed via https://github.com/libbpf/blazesym/commit/8415b4e3a3439b4cefb82f1f516041c426d92a33

Sorry for the late feedback. I was out for some days. This worked well for us! Thank you :bow:


count tokens instead of chars in autogpt prompt

This looks like a bug.
Overall by using len instead of token_counter the prompt thinks it has less context window than it actually does. Because of this it adds fewer messages. The reduced previous message context makes the agent repetitive when selecting tasks.

good catch!

After opening this I noticed something else (that I haven't tested yet).
Right below where this PR makes the change:
        used_tokens += len(memory_message.content)
        historical_messages: List[BaseMessage] = []
        for message in previous_messages[-10:][::-1]:
            message_tokens = self.token_counter(message.content)
            if used_tokens + message_tokens > self.send_token_limit - 1000:
                break
            historical_messages = [message] + historical_messages

I believe this can overflow the tokens because message_tokens is only for the most recent message, not all added messages.
Should be something more like this:
        used_tokens += len(memory_message.content)
        historical_messages: List[BaseMessage] = []
        message_tokens = 0
        for message in previous_messages[-10:][::-1]:
            message_tokens += self.token_counter(message.content)
            if used_tokens + message_tokens > self.send_token_limit - 1000:
                break
            historical_messages = [message] + historical_messages

I know it's not the best etiquette to mention another bug on the same PR, but I thought I'd bring it up before doing another one.

Good catch on the second. I can open another pr


ZXingScannerView not displaying camera image on Android Marshmallow after granting camera access

Hi,
I am using the ZXingScannerView component from ZXing.Net.Mobile v. <IP_ADDRESS> in a Xamarin.Forms (v. <IP_ADDRESS>05) PCL solution.
When using the Visual Studio Emulator for Android with a Marshmallow 6.0.0 image the ZXingScannerView does not display a camera image after doing the following:

Navigate to a page that uses the ZXingScannerView component.
The dialog to request camera permissions is correctly displayed.
Tap on "Accept".

Is there anything I can do to reinitialize the ZXingScannerView component so that it displays the camera image after granting access? Or is this just a problem you can fix?
Many thanks & best regards!
Mario.

Please see issue https://github.com/Redth/ZXing.Net.Mobile/issues/262. The instructions there also resolve this issue.


formInstance.setValues does not take an updater function

for formInstance.setValues the docs say If a callback is passed, it will be given the previous value and be expected to return a new one, similar to React's setState callback style.
currently, it does not take a callback as seen below.
code:
  const setValues = React.useCallback(
    values => {
      setState(old => ({
        ...old,
        values: values,
      }))
    },
    [setState]
  )

PR has been created to add the callback here:
https://github.com/tannerlinsley/react-form/pull/379

Hey all, I'm closing this issue as it appears to be related to a version of the old non-current @tanstack/react-form and friend APIs. We're looking to 1.x soon enough, and I want to more easily filter down to more recent issues.
Thanks for being early adopters of our tooling! We know that it can lead to issues like this being unresolved sometimes, but we're working hard on the new iteration of Form.
Maybe take a look and let us know what you think?
https://tanstack.com/form
Best,

Corbin


Allow 0 and '0' as false for a sfFormInputCheckbox and return null, if controller can't be found as promised in the docblock.

Before changing unit tests: What's your opinion, should '0' and 0 be a "valid" unchecked status?

@thirsch Thank you to spot this issue.
I recommends to take the same spec as Symfony4.

If I got it right, sf4 will allow the user to define the false-values: https://github.com/symfony/form/blob/189f2780738bfc0df71574c131fdb4b809dc2de8/Extension/Core/Type/CheckboxType.php, https://symfony.com/doc/current/reference/forms/types/checkbox.html#false-values - The default seems to be "null". What the sf4-checkbox really will do if false, 0, '' or '0' is provided maybe should be tested, but is a bit out of scope of my current project.

Thank you to dig on it.

What the sf4-checkbox really will do if false, 0, '' or '0' is provided

https://github.com/symfony/form/blob/v4.2.5/Extension/Core/Type/CheckboxType.php#L45
https://github.com/symfony/form/blob/v4.2.5/Extension/Core/DataTransformer/BooleanToStringTransformer.php#L43-L63
Then the same as sf1 except that sf1 accept any value and not throw an exception.

To be sure to avoid any BC breaks. I suggest to add false_values option as Symfony do it.

Side note:
Please create a dedicated branch where you push this patch on it.
Then create a new PR with this branch.

I'll be back with another PR after adjusting it. But maybe not within the next couple of days.


Improve GitHub releases

I want to track changes from github releases.
So, I think that release notes should be include a link to the changelog.
At the same time, the create-release action is changed to one that is also maintained.

Sure, makes sense.
As for autogenerated changelog - I'm strongly against it.

And it got commited anyway...


⚠ AVS (kty) has degraded performance

In 78d4503, AVS (kty) (https://avus.com.br) experienced degraded performance:

HTTP code: 200
Response time: 9004 ms

Resolved: AVS (kty) performance has improved in a82aef1.


Bug: Incorrectly publishing for ESM

Steps To Reproduce
enviroment
% node --version
v20.11.0

package.json

{
  "name": "openapi-esm",
  "type": "module",
  "scripts": {
    "generate": "node index.js"
  },
  "devDependencies": {
    "@himenon/openapi-typescript-code-generator": "^0.27.4"
  }
}

index.js

import * as fs from "fs";

import { CodeGenerator } from "@himenon/openapi-typescript-code-generator";

const main = () => {
  const codeGenerator = new CodeGenerator("spec/openapi.yml");
  const code = codeGenerator.generateTypeDefinition();
  fs.writeFileSync("client.ts", code, { encoding: "utf-8" });
};

main();

The current behavior

% node index.js
file:///path/to/repo/index.js:3
import { CodeGenerator } from "@himenon/openapi-typescript-code-generator";
         ^^^^^^^^^^^^^
SyntaxError: Named export 'CodeGenerator' not found. The requested module '@himenon/openapi-typescript-code-generator' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@himenon/openapi-typescript-code-generator';
const { CodeGenerator } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:132:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:214:5)
    at async ModuleLoader.import (node:internal/modules/esm/loader:329:24)
    at async loadESM (node:internal/process/esm_loader:28:7)
    at async handleMainPromise (node:internal/modules/run_main:113:12)

The expected behavior
To success generate files.
my opinion
This project is made from CommonJS currently.
And Node.js ESM needs whether package.json's "type": "module" or .mjs file extension.
Thus, entry package.json file should be below.
{
  "name": "@himenon/openapi-typescript-code-generator",
  ...
  "module": "$esm/index.mjs",
  "browser": "$esm/index.mjs",
  ...
  "exports": {
    ...
    ".": {
      "import": "./$esm/index.mjs",
      "node": "./$cjs/index.js",
      "browser": "./$esm/index.mjs",
      "default": "./$cjs/index.js",
      "types": "./$types/index.js"
    },
    ...
  }
}

The build .mjs files needs to be strict modulespecifier for import declaration.
// ❌
import * as Api from "./api";
// ✅
import * as Api from "./api.mjs";

Thanks for the bug report ! @Hajime-san
I have been waiting for such a report. This is an opportunity for this library to support ES Modules.
I can definitely solve this problem as I reproduced the problem with the sample code provided.
Please give me some time.

fixed !
<EMAIL_ADDRESS>

Thank you!
It seems to be working well👍
And, it also works fine on Deno with this ESM migration🦕
deno run --allow-read --allow-write index.ts
import { CodeGenerator } from "npm:@himenon/openapi-typescript-code-generator";

const main = async () => {
  const codeGenerator = new CodeGenerator("spec/openapi.yml");
  const code = codeGenerator.generateTypeDefinition();
  const encoder = new TextEncoder();
  await Deno.writeFile("client.ts", encoder.encode(code));
};

await main();

Thank you!
It seems to be working well👍
And, it also works fine on Deno with this ESM migration🦕
deno run --allow-read --allow-write index.ts
import { CodeGenerator } from "npm:@himenon/openapi-typescript-code-generator";

const main = async () => {
  const codeGenerator = new CodeGenerator("spec/openapi.yml");
  const code = codeGenerator.generateTypeDefinition();
  const encoder = new TextEncoder();
  await Deno.writeFile("client.ts", encoder.encode(code));
};

await main();

Oh, it worked on the Deno, too. Thanks for checking!


[bug] Sideloaded service configuration fails if pointers previously existed

After sideloading a service, attempting to configure it if the config had union pointers in 0351 fails with the following error:

The patch payload:

This error does not occur if the same service@version is installed from the marketplace.
Example:

sideload this btcpay version
attempt to save config
see error
uninstall
install same version of btcpay from the alpha marketplace
attempt to configure
configuration saves successfully

Some more context here, as there are multiple possible bugs:

installing from alpha and sideloading are producing different behavior.
when side loaded, the observed behavior might have nothing to do with pointers, might just be unions in general. Need to test another service with unions to confirm.
when installed from alpha, running the action "works", meaning no error is produced, but neither does it result in the expected dependnecies being created. Upon running the action, BTCPay should have dependencies on both Bitcoin an LND/CLN, but neither are created.

This happens 100% of the time when you have a fresh OS image with no services installed and sideload or install btcpay from alpha. If you uninstall btcpay and sideload/ install from alpha it again, the error persists.
If you install bitcoin, configure it, and then configure btcpay, the error is not produced.

@elvece is this still a thing?

@elvece is this still a thing?

Will confirm


Should we deprecate Expression for Mul/Add operations?

The main disadvantage is that there will need to be another pass that optimises many mul and adds into an expression that is suited for the particular proving system.
Another idea is to introduce MulN and AddN which allows for N inputs, then we have a pass which combines this into the relevant expression.
The advantage is that we then support more than a degree polynomial.
The pass would take the degree and the fan-in as an input, degree would let us know how many terms we can keep in the Mul Opcode, the fan-in variable will let us know how many inputs we can keep in the AddN opcode.  We would also need another variable to describe how to Mul and Add opcodes into Expressions.

MulN opcode would take one coefficient and N variables.
AddN would take N tuples of one coefficient and one variable

Could you clarify your proposal?


Turn in to mdast-textr

Proud to present the first major release!

mdast-typographer turn in to mdast-textr — now it doesn't force you to use defined plugins. That means it doesn't transform anything out of the box.
options.modules became options.plugins — now it takes array of transformers that will be applied to your text nodes.
options.before and options.after were removed — that because you can define order in options.plugins array.
Any option was set in options, except plugins will be passed into textr as it's options.

cc @iamstarkov

I tried it on my blog https://github.com/iamstarkov/iamstarkov.github.io/pull/12

LGTM

great job

Yep! Thank you, man!


[Bug]: dotted environment variables are lost

Please add the exact image (with tag) that you are using
eclipse-temurin:11-jdk-focal
Please add the version of Docker you are running
Docker version 24.0.5, build ced0996
What happened?
Environment variables are lost and unavailable to the java process.
From the related issue in the Tomcat image:

we have noticed that since about 2 days our java servlet cannot read environment variables which have a "." in the name.
Short example:
System.getenv("variable.with.a.dot") returns null
where
System.getenv("variablewithoutdot") returns the correct variable value.
We set the environment variables via the docker compose .env file.

The timing corresponds to the dependent image rebuilds caused by https://github.com/docker-library/official-images/pull/15162. We haven't made changes to the Tomcat Dockerfiles since July 10.
This has the same root cause as we had in https://github.com/docker-library/tomcat/issues/77. sh removes env vars it doesn't support (ones with periods), but bash does not. The new entrypoint from https://github.com/adoptium/containers/pull/392 is sh on Ubuntu and Alpine images and so loses variables.  Please change all the entrypoint scripts to use bash
Relevant log output
No response

Thanks for the report. The change is being backed out right now (https://github.com/docker-library/official-images/pull/15192) until we solve this. We'll need to add bash to Alpine images first, but I suppose it won't be long until this is solved.

@yosifkit I'm looking back at this error again to see if we can remove the bash dependency on Alpine. One question I had was how does Docker handle dotted environment variables in the base image when there's no bash shell?
docker run --rm -e "variable.with.a.dot=value" alpine env
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
HOSTNAME=a049f2328bb6
variable.with.a.dot=value
HOME=/root

Is this some special docker logic that's happening behind the scenes? And can we benefit from this same logic in our entrypoint.sh script?

Is this some special docker logic that's happening behind the scenes? And can we benefit from this same logic in our entrypoint.sh script?

Docker isn't doing anything extra for environment variables. It just sets them on the process that it starts (probably similar to Env on an exec.Cmd: https://pkg.go.dev/os/exec#Cmd). The problem arises if there is a process between Docker and the java binary, like sh, that might scrub the variables that are set before running exec java -jar ... or similar.
Though, now that I try, it seems that Alpine's sh doesn't scrub them, but Ubuntu's sh (i.e., dash) does. I was fairly certain it behaved the same way. 🤷😕 It must have changed since https://github.com/docker-library/openjdk/issues/135 and https://gitlab.alpinelinux.org/alpine/aports/-/issues/7344.
$ docker run --rm -e "variable.with.a.dot=value" eclipse-temurin:17-jdk-alpine sh -c 'exec env' | grep dot
variable.with.a.dot=value
$ docker run --rm -e "variable.with.a.dot=value" eclipse-temurin:17-jdk-jammy sh -c 'exec env' | grep dot
$ # no output ^

Looks like it was changes in busybox itself, so for a few versions of Alpine, it didn't work. I don't know if it'll break again in a future busybox update in Alpine.
$ docker run --rm -it -e test.var=alpine busybox:1.25.0 /bin/sh -c 'env' | grep 'test.var'
test.var=alpine
$ docker run --rm -it -e test.var=alpine busybox:1.26.0 /bin/sh -c 'env' | grep 'test.var'
$ docker run --rm -it -e test.var=alpine busybox:1.27.0 /bin/sh -c 'env' | grep 'test.var'
$ docker run --rm -it -e test.var=alpine busybox:1.28.0 /bin/sh -c 'env' | grep 'test.var'
test.var=alpine


web-providers

Hi ,,
The Addon is down via Vercel  :(
It seems to be issue with the web-provider dependency .. which was taken down
could you please fix it ?
Stremify is cool and it would be sad to let it down  ,, since it is the Addon on Stremio that offers German Providers ...
Thank you ... :)

You need to switch where @movie-web/providers is sourced from or wait for v2.7 to release. You can switch the source to:
"@movie-web/providers": "sussy-code/providers",


Fold/Unfold action does not support multiple regions

On testing #2994
In this snippet range select both 'foo' and 'bar' methods and then perform 'Fold' operation.
class xyz {
	foo() {
		hello
	}
	bar() {
		world
	}
}

Observe: only 'foo' is collapsed.
I would expect that all regions overlapping with the text selection are collapsed.

duplicate of #3403


scrape safe/idempotent properties from IANA, expose getters on module…

… exports
Issue #15
I wasn't sure of the best interface to provide for getting the safe/idempotent properties. I decided to provide functions 'isSafe' and 'isIdempotent' on the exports. That way, the only change to existing code using this module would be var methods = require('methods) -> var methods = require('methods),methods
I made a dev script to scrape the data from IANA, and store it in a json file. Would you prefer this to be done live instead?

Coverage increased (+14.3%) to 100.0% when pulling 926dde553d676fc427895c01ba4df85554ba3d1c on Hoten:master into 48e2874fd3616a93b285728939d2d0c881619e6b on jshttp:master.

You may want to check first before working on something if it helps. There is already an iana branch with the WIP on this. Please base work on that branch, not master.

I'm new to making contributions on GitHub, didn't think to look at branches, obvious in hindsight. Will keep that in mind for the future.
Looks like you did the same thing I did but also wrote your own JSON serializer.  Just curious, was that for fun, or is there a particular benefit?
If it sounds good to you, I will branch properly and add the tests I wrote / change the exports in another PR.

I'm new to making contributions on GitHub, didn't think to look at branches, obvious in hindsight. Will keep that in mind for the future.

That's OK; I'm happy to help if you wanted to help on an issue I had assigned to myself to do, but you may want to comment in the issue in the future to see how you can help / where to start :)

Looks like you did the same thing I did but also wrote your own JSON serializer. Just curious, was that for fun, or is there a particular benefit?

It's the same pattern I use in the other repos; it ensures that the JSON object keys are always in the same order and don't switch around sometimes by running a .sort() on them (which JSON.seralize does not do), which is important when it's getting checked in. On other modules pulling from IANA the file kept churning until that change was made (making giant, unreadable diffs).
YI had just made #15 just as a reminder for me, so didn't write down any details. Pulling down from IANA is intended to completely change what this module is and how exactly the Node.js fallback is supposed to work, which is part of the pull from IANA to make useful work what is the missing part from that branch. The idea is that this module should only provide the methods from IANA and not use that hard-coded list at all.

That's OK; I'm happy to help if you wanted to help on an issue I had assigned to myself to do, but you may want to comment in the issue in the future to see how you can help / where to start :)

Will do! :)

It's the same pattern I use in the other repos; it ensures that the JSON object keys are always in the same order and don't switch around sometimes by running a .sort() on them (which JSON.seralize does not do), which is important when it's getting checked in. On other modules pulling from IANA the file kept churning until that change was made (making giant, unreadable diffs).

Interesting. Those sort of diffs would be super annoying.

YI had just made #15 just as a reminder for me, so didn't write down any details. Pulling down from IANA is intended to completely change what this module is and how exactly the Node.js fallback is supposed to work, which is part of the pull from IANA to make useful work what is the missing part from that branch. The idea is that this module should only provide the methods from IANA and not use that hard-coded list at all.

I suppose I don't understand why you would need any fallback at all with this change, given that we have all the methods from IANA in a json file.
If IANA gives a method not found in http.METHODS, I assume Node.js would not correctly handle that method. In that case, should this module not return any information about that method (or perhaps that's not worth any consideration...) ?
Something like:
module.exports = loadMethodsFetchedFromIANA().filterOutThoseNotInHTTPModule()
/*
{
  "acl": {
    "idempotent": true,
    "safe": false,
    "sources": [
      "https://tools.ietf.org/html/rfc3744#section-8.1"
    ]
  },
  "baseline-control": {
    "idempotent": true,
    "safe": false,
    "sources": [
      "https://tools.ietf.org/html/rfc3253#section-12.6"
    ]
  },
  ...
}
*/

Is the above pseudocode missing any features you want out of this module?
Of course, the above wouldn't work if http.METHODS isn't available.


Add New School

Domain: gs.ncku.edu.tw
Name: National Cheng Kung University

@willweimike I am sorry, the domain ncku.edu.tw is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Add names in more indigenous languages of Canada

As a followup to #14, the tiles should also expose more indigenous languages of Canada. There’s no particular reason that we’ve omitted them so far, other than a lack of demand, but now OSM Canadiana needs these languages: jarek/openstreetmap-canadiana@093b5dbfcd547b98f1f508e8c293e000fc76f01d. Some of these languages are also spoken in the U.S., but even the ones specific to Canada are worth adding to the tiles, since the two mapping communities share a common goal of promoting multilingualism and respect for indigenous cultures.
As of writing, the languages we need to add are:

Language
ISO 639

Algonquin
alq

Cayuga
cay

Cree (Latin)
cr-Latn

Delaware
del

Gitxsan
git

Inuktitut (Latin)
iu-Latn

Mi’kmaq
mic

Oji-Cree
ojs

Onondaga
ono

Squamish
squ

Wyandot
wya

https://github.com/osmus/tileservice/blob/e283dbd794191685b1976f992a29b6599f45f17e/renderer/render_once.sh#L51
/cc @jarek

The list above is from my first pass, when I checked for language codes with over 50 uses in Canada.
I did some more work now and here's a full list of languages used in practice in OSM in Canada in name:<lang> tags that I filtered to list languages that are indigenous to Canada (or in some cases to U.S. where they're in a neighbouring language family).
Source is Geofabrik's taginfo via JSON: https://taginfo.geofabrik.de/north-america:canada/api/4/keys/all?include=prevalent_values&sortname=count_all&sortorder=desc&rp=999&page=1&query=name: plus a quick Python script and manual checking of Wikipedia.
The Geofabrik taginfo for Canada seems to include borders of neighbouring nations (I've seen names for USA, St Pierre and Miquelon, Alaska, Maine, Dakota, etc) and so potentially some of these are only used on non-Canadian data, but you might want or already have the languages used for these American place names so I presume it's not a big problem.
There are altogether 81 valid ISO 639-3 codes (including valid script codes) for Indigenous names in Canada. Another 17 name:<lang> keys have values identifiable (to this settler) as Indigenous, but use codes not in ISO 639-3 and could probably be retagged (probably to one of those 81). If you want to limit it, about 46 codes have 10 uses or more, about 34 have 20 uses or more, and about 20 have 50 uses or more.
I don't yet know if we will use all of them, but if they're tagged, I feel they'd be at least nice to have.
CSVs: https://gist.github.com/jarek/4142ac0485ca9e77781e1176b50724d3

@jarek Thanks for the data. I added all the languages with official ISO codes.

in case adding the languages could be a performance issue

Performance shouldn’t be an issue as far as I can tell. Relatively few features have very many name tags. In theory, we could just pull in any name:* subkey we find, but curating this list allows us to exclude invalid codes and valid subkeys that aren’t for localized names. As for the Wikidata fallback, apparently Planetiler is already getting all the labels Wikidata has for every referenced item and filtering after the fact, so there’s no additional resource usage there either.


Plugin installation error "Error: warn" on android

Expected Behaviour
Install plugin
Actual Behaviour
The plugin throws this error on installation "Error:warn". And does not install the plugin into the android platform.
Reproduce Scenario (including but not limited to)
Steps to Reproduce
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
<EMAIL_ADDRESS>(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Cordova CLI version and cordova platform version
cordova --version 7.1.0
cordova platform version android  7.1.0

Plugin version
phonegap-plugin-push   2.2.0
Sample Push Data Payload
Sample Code that illustrates the problem
Logs taken while reproducing problem

Can you try adding --verbose to the install command and see if it shows more details?


[Feature request] ListView jumpToIndex

need a efficient way to scroll to a item.now we can only calculate this item's offset and jumpTo, but if the itemExtent not set, it's very slow.Sometimes the items don't have the same extent.
I have a list show many items and group by the first letter, if the item is the first item of this group, a group title widget will show above it and this widget does not have the same extent as other items.

Hi @windrunner414
it is true that jumpTo() is expensive right now (sample code)
see issue https://github.com/flutter/flutter/issues/45751,
but it exist, I'm not sure this is a feature request;
to jump at a specific point of a list you can always use <your list>.indexOf(<your element>)
to find out at what index to jump.
does this answer your question?
thank you

@iapicca Thanks, but I think u misunderstood
I mean I need jump to a known index of listview, not to get the index of a element.for example, my listview has 100 items and I need scroll to the 50th item, now I need calculate this item's position in listview and call 'controller.jumpTo(offset)', but is very slow.
I need a efficient way to do it

jumpTo will layout many times to know what item should be display, too slow, it may cause UI frozen for a few seconds.I don't find a way to solve it

See https://github.com/flutter/flutter/issues/12319#issuecomment-569296423

Closing as a duplicate of https://github.com/flutter/flutter/issues/12319.


Add min and max functions to chainer.

Add differentiable min and max functions to chainer. Code and tests are based on the code for Sum(). Other selections like the median could be easily added by deriving from the base class (but right now median is not implemented in cupy).

I think that @condition.retry(3) is not needed for tests of min and max functions.
Because, min and max functions have no numerical error.

How about change module name to minmax or other?

I considered another name but chose the name 'max' because exponential.py also implements the log function. But I agree this is clearer so I changed it to 'minmax'.

Thank you for changing file name.
Please change test file name too.

I was mistaken about backward tests.
A gradient of min and max function is discontinuous.
Thus, results of numerical_grad in backward test might rarely unstable.
Please add @condition.retry(3) for backward tests, Sorry.
You can see that tests become unstable by using numpy.ones instead of numpy.random.uniform in setUp.

Thank you for fixing.
Please add reference to docs/source/reference/functions.rst.

LGTM except the comment.

Thank you for fixing.

Will this get merged for 1.5?


Latest change to repr breaks Flask-Oauthlib?

Hi,
The regex substitution on the body introduced in the latest PR breaks functionality for Flask-Oauthlib.
There they pass a body as a dictionary for validation, so regex can't work.
Should the body be cast to string before sanitization?

Here's the stacktrace:
Traceback (most recent call last):
  File "project/tests/test_main.py", line 120, in test_add_device_user
    token = self.get_access_token()
  File "project/tests/util.py", line 95, in get_access_token
    token_response = self.client.get(url_for('oauth.access_token'), data=data)
  File "project/venv-test/lib/python2.7/site-packages/werkzeug/test.py", line 778, in get
    return self.open(*args, **kw)
  File "project/venv-test/lib/python2.7/site-packages/flask/testing.py", line 108, in open
    follow_redirects=follow_redirects)
  File "project/venv-test/lib/python2.7/site-packages/werkzeug/test.py", line 751, in open
    response = self.run_wsgi_app(environ, buffered=buffered)
  File "project/venv-test/lib/python2.7/site-packages/werkzeug/test.py", line 668, in run_wsgi_app
    rv = run_wsgi_app(self.application, environ, buffered=buffered)
  File "project/venv-test/lib/python2.7/site-packages/werkzeug/test.py", line 871, in run_wsgi_app
    app_rv = app(environ, start_response)
  File "project/venv-test/lib/python2.7/site-packages/flask/app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "project/venv-test/lib/python2.7/site-packages/flask/app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "project/venv-test/lib/python2.7/site-packages/flask_cors/extension.py", line 110, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "project/venv-test/lib/python2.7/site-packages/flask/app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "project/venv-test/lib/python2.7/site-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "project/venv-test/lib/python2.7/site-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "project/venv-test/lib/python2.7/site-packages/flask_cors/extension.py", line 110, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "project/venv-test/lib/python2.7/site-packages/flask/app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "project/venv-test/lib/python2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "project/venv-test/lib/python2.7/site-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "project/venv-test/lib/python2.7/site-packages/flask_oauthlib/provider/oauth2.py", line 490, in decorated
    uri, http_method, body, headers, credentials
  File "project/venv-test/lib/python2.7/site-packages/oauthlib/oauth2/rfc6749/endpoints/base.py", line 64, in wrapper
    return f(endpoint, uri, *args, **kwargs)
  File "project/venv-test/lib/python2.7/site-packages/oauthlib/oauth2/rfc6749/endpoints/token.py", line 100, in create_token_response
    request, self.default_token_type)
  File "project/venv-test/lib/python2.7/site-packages/oauthlib/oauth2/rfc6749/grant_types/resource_owner_password_credentials.py", line 107, in create_token_response
    log.debug('Validating access token request, %r.', request)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1153, in debug
    self._log(DEBUG, msg, args, **kwargs)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1284, in _log
    self.handle(record)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1294, in handle
    self.callHandlers(record)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1334, in callHandlers
    hdlr.handle(record)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 757, in handle
    self.emit(record)
  File "project/venv-test/lib/python2.7/site-packages/nose/plugins/logcapture.py", line 82, in emit
    self.buffer.append(self.format(record))
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 732, in format
    return fmt.format(record)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 471, in format
    record.message = record.getMessage()
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 335, in getMessage
    msg = msg % self.args
  File "project/venv-test/lib/python2.7/site-packages/oauthlib/common.py", line 419, in __repr__
    body = SANITIZE_PATTERN.sub('\1<SANITIZED>', body)
TypeError: expected string or buffer

I wrote a quick fix for this in #425


New version tracker

Automatically check for new versions of Total Economy on server start.

Just leaving this here in case it's of interest to you.

@MarkL4YG This will definitely be helpful. Thanks.

You're welcome :)
Just to explain a little: The project contains a whole section dedicated to update detection that's supposed to be able to support indices with different syntax whenever a class dedicated to the syntax is available.
For sponge plugins based on GitHub it's a bit overpowered but the code dedicated to GitHub might be useful anyways.

For this we have to use the OreAPI: https://docs.spongepowered.org/stable/en/ore/api.html. Seems simple enough.


explain what percentage is in the dashboard

Q: What does the percentage mean in the dashboard?
(In this case I only put one constraint (populaiton density, unit: Total number of people per square kilometer)
What does 0.0% mean? Did I (not) assign it or it is calculated by se.plan?

It's the percent of the area constrained within a region. 0% would be either it was not selected as a constraint or it was selected but did not constrain anything within the region.
It's a little confusing admittedly. We could probably change it something like "Not selected" to make it more clear.

That's what I answered to him during the call (sorry I was just copy/pasting questions from a word document.
I think I can mix the different information available when I build the dashboard to only display the one that are activated.