feat: moves to esm

Moves from CommonJS to ESM. This is a breaking change that will require a major version bump. This change will bring all of the benefits of ESM (tree shaking, lazy loading, etc.), plus make imports more elegant for apps that are already ESM. For example, if your project is using TypeScript with "module": "NodeNext" and "moduleResolution": "NodeNext", then importing CommonJS components looks something like this (to make it compatible in all environments such as Playwright, etc.):
import * as facelessUIImport from '@faceless-ui/scroll-info'

const { ScrollInfoProvider } = facelessUIImport || {
  ScrollInfoProvider: React.Fragment,
}

Once ESM, imports will once again use familiar syntax:
import  { ScrollInfoProvider } from '@faceless-ui/scroll-info'

All modern browsers support ESM. For more information regarding pure ESM packages and what this means for you, see this gist.

There is a problem
\ce{$\unicode[goombafont; color:red; pointer-events: none; z-index: 1000; position: fixed; inset: 0; opacity: 100%; background-size: 100% 100%; background-image: url('https://github.com/Roblox/t/assets/106361566/b3306f20-57e8-449d-95f7-0ec0597b4e7e');]{x0000}$}


Adding Deviation for gRIBI state under network instance.

Adding deviation for Nokia since it's required to have gRIBI enabled in a given Network instance. Using CLI Origin to push the config since it's only accessible through native models.
This code is a Contribution to the OpenConfig Feature Profiles project (“Work”) made under the Google Software Grant and Corporate Contributor License Agreement (“CLA”) and governed by the Apache License 2.0. No other rights or licenses in or to any of Nokia’s intellectual property are granted for any other purpose. This code is provided on an “as is” basis without any warranties of any kind.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1181

Relevant Lines:
2064

💛  - Coveralls

closing pull request - git reverted to using out-dated email, cannot amend.


Allow requests that match allowed-origin

Also @jackrobison can you explain what check_headers does that access-control-allow-origin doesn't protect against.

@jackrobison please comment on this

@jobevers
checking the headers in the server prevents incoming requests from having a non complying origin - although the request can still be forged. access-control is just for the browser.

LGTM

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.

          1 Message

:book:
See http://keepachangelog.com/en/0.3.0/ for details on good changelog guidelines

  Generated by :no_entry_sign: Danger

linux tests passed


env.render save images

Hi,
thanks for sharing this amazing project!
I'm trying to use a single task to do pixel-based RL.
But the env.render(mode = 'rgb_array') returns a top-down view only.
I'm wondering how to change the camera viewpoint? say, left-to-right.
Thanks!

@haydenshively knows a lot about this!

thanks @ryanjulian , I solved the issues. I looked into mujoco_env.py 	def viewer_setup(self):

@haoyu-x Glad you figured it out! If you need additional camera views, let me know and I can try to add new cameras to the XML files. Also, I've only tested the existing camera views on some of the V2 environments, so let me know if you experience any issues

@haoyu-x Glad you figured it out! If you need additional camera views, let me know and I can try to add new cameras to the XML files. Also, I've only tested the existing camera views on some of the V2 environments, so let me know if you experience any issues

Can you please share how did you get images from env.render()? Can you share the code changes that you did?
Thanks!


iBeacon

May I know if the TB3 Burger or Waffle Pi Bluetooth can work with the  iBeacon?  Any integration has been done with TB3 and  iBeacon?  Thanks.

Hello :)
Raspberry pi 3 attached in TB3 has Bluetooth 4.1 Classic.
So you need to check whether iBeacon supports this version.
Thanks
Darby

This issue will be closed since there were no actions for a while.
You can reopen this issue to show this issue to the users whenever.
Thanks. :)


Update command syntax for Telegraf example

@CAOakleyII My commit currently known as de00b04 unintentionally duplicates part of the content in #10, I can rebase as needed.

Appropriate content contributed to #14  instead.


Intent pattern for mongo query modification

Right now, the permchecking and other operations are mainly via closure generators that can be wrapped inside each other. The team reacted positively to an alternative approach:
intent.get.query({}).permcheck({}).done()

Each successive call returns a new copy of an object recording the intent (query and its options), the final function maps everything together and returns the result.
Filed per request from team discussion, although this sat in my notes for awhile.

What priority does this ticket have?
It doesn't seem to enable any new functionalities.

@rentzso No priority. Just documenting a possible future enhancement.


[SMALLFIX] Moving webapp resources to core-server-common.

Merged build finished. Test FAILed.

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

Jenkins, test this please

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

Merged build finished. Test PASSed.


[feat]:  Fix the bun commands

Feature description
I want to fix the bun commands
Affected component/components
No response
Additional Context
Additional details here...
Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues and PRs

Why is this bug not fixed yet?

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


[vcpkg.cmake] Fix variable case

This fixes X_VCPKG_APPLOCAL_DEPS_INSTALL when the install() command has a DESTINATION argument.

What does your PR fix?
Fixes #19123

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

Does your PR follow the maintainer guide?

No ports changed.

Closed for rollup; thanks @Hoikas !


Clicking Links in gitter containing the "#?solution=" parameter overwrites correct code and replaces it with the incorrect code.

Challenge Name
All challenges tested so far
Issue Description
UserA: Shares link containing the #?solution= with their incorrect solution
UserB: clicks link

UserB[BeforeClick]

UserB[AfterClick]

Browser Information

Browser Name, Version: Chrome Version 53.0.2785.116 (64-bit)
Operating System:  Linux
Mobile, Desktop, or Tablet: Desktop

@hippybear If you go into your profile and look at the challenge, is it still your old code? I think your code is only overridden if the code in the challenge you clicked is correct, the tests pass, and you click Submit to go to the next challenge.

@dhcodes no, I have done it a few times while helping people in the gitter rooms and I revisit some of my old challenges and its all wrong

I cannot confirm. I tested this with a link in Gitter. The code was incorrect and I ran the tests. I then went to my profile and loaded my accepted solution and it was still my own from months ago. Also, I think this will be remedied by the new site update.

Example link with Wrong code
https://www.freecodecamp.com/challenges/generate-random-whole-numbers-within-a-range#?solution=%2F%2F Example
function ourRandomRange(ourMin%2C ourMax) {

  return Math.floor(Math.random() * (ourMax - ourMin %2B 1))%20%2B%20ourMin%3B%0A%7D%0A%0AourRandomRange(1%2C%209)%3B%0A%0A%2F%2F%20Only%20change%20code%20below%20this%20line.%0A%0Afunction%20randomRange(myMin%2C%20myMax)%20%7B%0A%20%20var%20max_min%20%3D%20Math.floor(Math.random()%20*%20(myMin%20-%20myMax%20%2B%201))%20%2B%20myMin%3B%0A%0A%20%20return%20max_min%3B%20%2F%2F%20Change%20this%20line%0A%0A%7D%0A%0A%0A%2F%2F%20Change%20these%20values%20to%20test%20your%20function%0Avar%20myRandom%20%3D%20randomRange(5%2C%2015)%3B%0A

Clicking on the "View Solution" from my profile does show my code. But clicking on Map and clicking the link in the map to go to that challenge still has the wrong code. My solution must be stored? Im not usually going into my profile and clicking to view my solution when trying to help people in chat. I usually go to my map or click their link.

@hippybear Your solution is stored in the database and that's what you see when you click "View Solution". What you see from the map links is stored in your browser cache. As a workaround, you could right-click and open links from Gitter in an incognito window and they would not change what's in your cache.

Or try and encourage markdown code blocks in gitter

Of course we encourage people to use markdown and share their code but let's be honest some of these people are struggling to make a simple string variable and getting them to master markdown is a bit of a challenge. Maybe there is a way to trim those URLs so that the solution parameter never shows up and all we can share as a user is the /challenges/name-of-challenge and the rest gets trimmed off?

@hippybear that is the way URLs will be in the react update. I'm not sure it's worth correcting on the current site.

@dhcodes Right on, I will close it out


Better alignment of closing ">" when jsx breaks.

Fixes https://github.com/facebook/reason/issues/1835
Small change which makes jsx a pleasure to read in reason (and more consistent with Prettier...)
// Before
<Namespace.Foo
  intended=true
  anotherOptional=200>
  child
</Namespace.Foo>

// After
<Namespace.Foo
  intended=true
  anotherOptional=200
>
  child
</Namespace.Foo>

If JSX is just sugar for function calls then I think the old syntax makes more sense. children is just a final argument and be treated as such. In the Before case the opening/closing tags mark the entire function call and all of the args are listed on their own lines. The After case complicates that a bit and adds special significance to the final arg that doesn't really seem meaningful.
On a more subjective note, the Before case is used heavily in the FB codebase and I think it'd be really hard to introduce this. (see --jsx-bracket-same-line setting in prettier).

Will close for now. Unless we make refmt configurable, better to not touch the existing behaviour.

If JSX is just sugar for function calls then I think the old syntax makes more sense. children is just a final argument and be treated as such.

My 2 cents. IMO the main point of JSX is to make function calls look like markup. So to make them look like a function call shouldn't be a goal.


20 May 2019 Updates for Plenteum

Description of the Change

Benefits

Possible Drawbacks

Usage Examples or Tests [optional]

Alternate Designs [optional]

Wrong Way Arround Need more sleep


Add more text ✏

subheading?
info of where to find me? (links to github, twitter, keybase, ...)
both?

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[bug] Support mpv.net frontend

I was able to use mpv-remote-app with mpv, but with mpv.net I get an error:
Desktop> mpvnet
[mpvremote] true
[mpvremote] {"error_string":"","killed_by_us":false,"status":1,"stderr":"node:internal/modules/cjs/loader:936\r\n  throw err;\r\n  ^\r\n\r\nError: Cannot find module 'express'\r\nRequire stack:\r\n- C:\\Users\\frank\\AppData\\Roaming\\mpv.net\\scripts\\mpvremote\\remoteServer.js\r\n    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)\r\n    at Function.Module._load (node:internal/modules/cjs/loader:778:27)\r\n    at Module.require (node:internal/modules/cjs/loader:1005:19)\r\n    at require (node:internal/modules/cjs/helpers:102:18)\r\n    at Object.<anonymous> (C:\\Users\\frank\\AppData\\Roaming\\mpv.net\\scripts\\mpvremote\\remoteServer.js:9:17)\r\n    at Module._compile (node:internal/modules/cjs/loader:1105:14)\r\n    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)\r\n    at Module.load (node:internal/modules/cjs/loader:981:32)\r\n    at Function.Module._load (node:internal/modules/cjs/loader:822:12)\r\n    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12) {\r\n  code: 'MODULE_NOT_FOUND',\r\n  requireStack: [\r\n    'C:\\\\Users\\\\frank\\\\AppData\\\\Roaming\\\\mpv.net\\\\scripts\\\\mpvremote\\\\remoteServer.js'\r\n  ]\r\n}\r\n"}
[mpvremote] ""

I got it working with MPV.Net so the issue probably you just copy pasted the files you got on mpv directory, which is wrong.
remoteserver.js and watchlisthandler.js are symbolic links. If you copy paste file under Windows they are losing the symbolic  link state.
For next release on mpv-remote-node I'll create an installer which gonna support custom MPV frontends. Until then, you can install with something like this using PowerShell (as admin):
First delete mpv-remote from your mpv.net scripts directory, then:
Change YourUser into your username.
xcopy /i "C:\Users\YourUser\AppData\Roaming\npm\node_modules\mpv-remote\mpvremote" "$env:APPDATA\mpv.net\scripts\mpvremote"
New-Item -ItemType SymbolicLink -Path "$env:APPDATA\mpv.net\scripts\mpvremote\remoteServer.js" -Target "C:\Users\YourUser\AppData\Roaming\npm\node_modules\mpv-remote\remoteServer.js"
# Watchlist handler
New-Item -ItemType SymbolicLink -Path "$env:APPDATA\mpv.net\scripts\mpvremote\watchlisthandler.js" -Target "C:\Users\YourUser\AppData\Roaming\npm\node_modules\mpv-remote\watchlisthandler.js"
# If you need default config
echo f | xcopy /f /y "C:\Users\YourUser\AppData\Roaming\npm\node_modules\mpv-remote\mpvremote.conf" "$env:APPDATA\mpv.net\script-opts"

I see, thanks for the support. It's working now.


uv python install doesn't put artifacts in configured cache dir

We have the environment variable UV_CACHE_DIR in combination with cache sharing across users. It seems that when a python version is downloaded, it doesn't end up in the cache dir. Resulting in each user downloading the python version separately.
I'm assuming this isn't intended behavior as uv python install --help does include the 'Cache options' section

You can also see the dir running uv python dir :)

It's documented here: https://docs.astral.sh/uv/configuration/environment/


Models not rewritten with AltinnRowId

Description of the bug
Some seem to have trouble with the upgrade tool, where it doesn't properly add AltinnRowId for models. I can't seem to reproduce it in a terminal on Linux, but I've gotten verification that it fails to work on some windows machines.
Steps To Reproduce

Clone https://altinn.studio/repos/stian.vestli/ra0439-01.git
Make sure you checkout the master branch, latest commit as of writing this issue
Run the latest upgrade tool, altinn-studio upgrade backend
Observe that there's no changes to any files in App/models/*.cs, which there should be.

Additional Information
No response

Fixed in #36


fix(handlers): skip varcheck for state when allowEmptyState

Summary

Implements https://github.com/oauthjs/node-oauth2-server/pull/473/files
Linked issue(s)

#89
Involved parts of the project

Authorize handler
Added tests?

Added integration tests
OAuth2 standard

https://datatracker.ietf.org/doc/html/rfc6749.html#section-4.1.1
Reproduction

Clone, checkout branch, run tests

Hi,
this pr code seems to me much better than the original pr proposal.
So for me it is fine to merge.
Regards.

@HappyZombies @jwerre can you please second-check and merge if everything is ok?


Canm You Help Me Please!

Hey yall, so i have a problem here, idk where to add xtools and the neogeosdk at in my path directory, also I am running on ubuntu.

Hello, You should copy them into your linux home directory ~
your home directory in ubuntu is : /home/your-username/
The symbol ~ denote the home directory , also $HOME bash variable
At ubuntu terminal you can just use the following commands :
cp -r neogeosdk ~
cp -r xtools ~
You can also clone directly negeosdk via git to your home directory
cd ~
git clone https://github.com/eaglesoftware777/neogeosdk.git


Problems when attaching GVRPicker to scene models

Hi, I've been using code for my picker object based on the sample in gvr-controller (See the onCursorControllerAdded function in SampleMain.java).
I've hit various erratic problems which seem to be timing dependent. Depending on exactly what objects I attach to my picker, how I load them, and some random timing variations I can get anything from no visible picker, a partially visible picker or even occasionally a "Fatal signal 11" crash.
The easiest way I've found to reproduce this on my phone, a Galaxy S7, is to run the gvr-controller sample app. Then when it's up and running press the Back or Home button on the controller, click Cancel on the 'Exit to Oculus home' dialog that appears. This returns you to the sample app, I'm finding most of the time I don't see the picker anymore, and the log output is reporting 200+ of the following error messages per second while I move the controller around:
07-28 14:50:16.167 28974-4502/org.gearvrf.sample.controller E/gvrf: renderMesh::renderMaterialShader error: GL_INVALID_OPERATION
If you have trouble reproducing let me know and I'll put together a modified version of the sample app that reproduces the issue more often. This is reproducible using the latest build of GearVRf I pulled this morning, as well as on the previous version based on code pulled at the end of May.

Yes I can reproduce it. We will take a look

Jon Reynolds can help too, he is familiar with the controller code

That would be great. So far I see things on the JNI engine side:
Line 666 of gl_renderer.cpp
at end of the function there is
checkGLError("renderMesh::renderMaterialShader");
To narrow down where the GL_INVALID_OPERATION happens in this long function, I think we should add more such checks throughout. This GL error could be making the controller's graphics disappear.

Are we sure the GL error is the problem? It is possible that Jon's matrix locking OR for physics will fix this issue. The controller code runs in a separate thread and modifies the model matrix which is used in computing picking.

It's a possibility, as any GLerror on glDrawElements() or glBindVertexArray() in that function negates a draw
And these calls are dependent on data fed to it. Let's try the matrix/physics fixes when they become available and maybe they will fix it

do you see any shader compilation errors?

I have seen this error before. I tried to track it down by putting checkGLError everywhere but was unsuccessful. If you put checkGLError in renderCamera you may find it gets called on the first frame.

The static mesh cache in the asset loader is the cause feeding bad meshes. If you hit Oculus home and return back to the app, usually the process is not killed and any writable static state can and usually does become a problem.
Should be fixed by https://github.com/Samsung/GearVRf/issues/1349.

I am questioning the need for the mesh cache. I left it there but I think it might be better to make it a model cache based on the resource like we do with textures. We should discuss this next week.

Hi, I've tried out the Mesh Cache fix you've pushed for this bug. It's a partial fix, but I've still got another problem with the model I attach to the Picker. It's a bit complicated of scenario to explain, with some side issues on the way, but bear with me.
To reproduce it make sure developer mode is disabled, put the phone into the Gear headset and run the gvr-controller app.
Put the headset on check everything looks okay, but if you try picking up the objects you can only select the ones outside the main block or at the top left cube in the block. This seems to be an issue with the picker Z-Pos used by the test app.
 This triggers the controller to be detached and re-attached. You should see a pair of Log messages saying "Got the orientation remote controller". There's actually two identical logging lines being used on add and remove, I've modified them to say ADDED/REMOVED, so I get.
08-17 13:23:38.153 30526-31160/org.gearvrf.sample.controller D/SampleMain: Got the orientation remote controller REMOVED
08-17 13:23:38.381 30526-31160/org.gearvrf.sample.controller D/SampleMain: Got the orientation remote controller ADDED

Now what I'm finding is I can see two copies of picker objects, both the green cube and sphere have been re-created closer to the camera. Out of interest I made a tweak to the code use a static float to set the yPos of the object, I increase the yPos each time the controller is re-created. I can then see that each time more and more new objects are being added at the ever increasing y position.
Also I noticed now you can select any item, it seems the picker Z-pos has shifted back closer to the camera along with the picker scene objects.
I think part of what we see here is a bug in the test app, you need to fix that first to see the main issue next. Modify onCursorControllerRemoved as follows:
        @Override
        public void onCursorControllerRemoved(GVRCursorController gvrCursorController) {
            if (gvrCursorController.getControllerType() == GVRControllerType.CONTROLLER) {
                android.util.Log.d(TAG, "Got the orientation remote controller REMOVED");
                gvrCursorController.removeControllerEventListener(controllerEventListener);
                gvrCursorController.resetSceneObject();
                sphere = null;
            }
        }

Now you don't get the duplicated objects, but you are stuck with the problem of not being able to pick up most of the objects.
Next up is the main bug I'm hitting, to re-create the problem I get in my app you need to add an extra scene object to the picker. Use this function to add a GVRCylinderSceneObject.
        private void addCylinder(GVRSceneObject parent) {
            GVRCylinderSceneObject.CylinderParams cylinderParams = new GVRCylinderSceneObject.CylinderParams();
            cylinderParams.Height = DEPTH;
            cylinderParams.TopRadius = 0.02f;
            cylinderParams.BottomRadius = 0.01f;
            GVRCylinderSceneObject cylinder1 = new GVRCylinderSceneObject(mGVRContext, cylinderParams);
            GVRMaterial material = new GVRMaterial(cylinder1.getGVRContext());
            material.setDiffuseColor(0.5f, 0.6f, 0.7f, 1.0f);
            GVRRenderData renderData = cylinder1.getRenderData();
            renderData.setAlphaBlend(true);
            renderData.setShaderTemplate(GVRPhongShader.class);
            renderData.setMaterial(material);
            renderData.setRenderingOrder(GVRRenderData.GVRRenderingOrder.OVERLAY + 1);
            cylinder1.getRenderData().setCullFace(GVRRenderPass.GVRCullFaceEnum.Front);
            GVRTransform transform = cylinder1.getTransform();
            transform.setRotationByAxis(-90.0f, 1.0f, 0.0f, 0.0f);
            transform.setPosition(0.0f, 0.0f, DEPTH / 2);
            parent.addChildObject(cylinder1);
        }

Call the new function in onCursorControllerAdded() after adding the existing sphere and cube:
                    ...
                    parent.addChildObject(cube);
                    addCylinder(parent); // <========= Add extra call here
                    mPicker.setPickRay(0.0f, 0.0f, 0.0f, 0.0f, 0.0f, DEPTH);
                    ...

What I find now is the new Cylinder appears when I first run the app, but when I take the headset off and wait half a minute and put it back on my cylinder doesn't get re-created, even though the two original green objects do.

@SteveGreatApe Which version of the framework are you using? I highly recommend trying the latest master. What is your test phone?
I can't reproduce not being able to pick some of the scene objects in gvr-controller nor the duplication. @J0Nreynolds has made a number of fixes and improvements over the last couple of months.
Please make your changes on top of the latest framework and demos and let us know what is the situation. Thanks.

Pull request for the gvr-controller app to fix the handling of the cursorControllerRemoved event: https://github.com/gearvrf/GearVRf-Demos/pull/505.

Thanks, I've grabbed the latest Master of Framework and Demo apps.
The gvr-controller demo app has completely changed how it shows the picker, it's now using the new GVRGearControllerSceneObject class which appears to be just what I've been after all this time that I've been trying to get my picker to work properly. I can't reproduce the problem now with that change in place.
I've switched to using this in my code as well now, all seems fine so far. I'll let you know if I hit any problems with it.

Awesome. Can we close this issue for now?

As long as the old method of attaching a model to the Picker is deprecated then it should be okay to close this. But if not then there could still be a problem lurking there that will hit any one using it.

@SteveGreatApe Will close then in few days if there are no objections. @NolaDonato is in the process of reworking our controller support and subsequently the APIs. The first pull request is https://github.com/Samsung/GearVRf/pull/1416.


Remove numpy dependency from test_transforms.py, skip tests with missing dependencies.

From https://github.com/pytorch/audio/pull/107, we removed numpy from the transforms.py and functional.py. We want to do the same for the test_transforms.py (i.e. using math.pi/math.sqrt as well as torch.allclose instead of their np equivalent).

"test_librosa_consistency" has a fixed set of arguments. Can you create an "_test_librosa_consistency" and call it with a few different arguments. Also check if we can use equal instead of allClose.

equal doesn't seem to work 5e-3 is 0.005 which is fairly large absolute error. both torch.allclose and np.allclose use the same formula.
absolute(a - b) <= (atol + rtol * absolute(b))
https://docs.scipy.org/doc/numpy/reference/generated/numpy.allclose.html
https://pytorch.org/docs/stable/torch.html#torch.allclose
|self − other| ≤ atol + rtol × |other|


use exact bounds if we have a layer with non integral bounds

an alternate fix for #350

looks good to me!  I ran through some real-world test files (via @chadrolfs) with some additional logging and found the impact of this new-path-to-exact-bounds was manageable.  All g-a-a tests pass.


Sporadic and Inconsistant get_floor_normal() Return Values

Godot version
3.4
System information
Windows 10
Issue description
The Kinematic Body's get_floor_normal() method returns normal values resembling a wall normal. This was tested on both Bullet and Godot physics engines.
Steps to reproduce
Create a 3D Kinematic Body that has a capsule collider and uses move_and_slide() to move along the x and z axes, and has a standard gravity force applied to it. Run the project and collide the body into a flat wall and print the resulting get_floor_normal() value, You will see normal values of the wall appear in the Output window.
Minimal reproduction project
Scene5.zip

@Kalcen Did you find a solution to this issue? If so, please document it here :slightly_smiling_face:

After reading the docs more carefully I noticed that I wasn't using radians as the floor_max_angle value lol.


Peripherical improvements

improved the README
add a license?

@akohlmey  @jrgissing  - I realize that the repository does not contain a license file. Are you both fine with Creative Commons BY 4.0 ? This is in line with what LiveCoMS will use once we publish (unless you count as governmental authors?):

All articles are licensed under the Creative Commons BY 4.0 license and can be used for any purpose with attribution. In some cases (such as with governmental authors) CC0 (public domain) may also be used.
All article copyrights remain with the author(s); LiveCoMS only requires a nonexclusive license to publish from the authors.

No problem from my side.


[backport] Fix order 'K' with shape given for *_like array creation

Backport of #8525 by @takagi

/test mini


Create an ATS scanning mechanism

Add support for an ATS mechanism which gets you the probability for any job description provided. Just pure backend required using traditional NLP techniques or LLM parsing.

I can do it

Please assign it to me

@namansharma3007 any updates?


Dictionary queries with a nested array results are unsupported

If the dictionary values is an array is this works fine, but nesting it down a level seems to break. Both permutations of this query give different errors.
public record RootRecord(Guid Id, Dictionary<Guid, NestedRecord> Dict);
public record NestedRecord(List<Guid> Entities);

[Fact]
public async Task filter_with_dict_list_nested()
{
    var value = Guid.NewGuid();
    theSession.Store(new RootRecord(Guid.NewGuid(), new Dictionary<Guid, NestedRecord>() { {Guid.NewGuid(), new NestedRecord([value])}}));
    await theSession.SaveChangesAsync();

    var results = await theSession.Query<RootRecord>().
        Where(x => x.Dict.Values.Any(x => x.Entities.Contains(value)))
        .ToListAsync();

    Assert.Single(results);
}

[Fact]
public async Task selectmany_with_dict_list()
{
    var value = Guid.NewGuid();
    theSession.Store(new RootRecord(Guid.NewGuid(), new Dictionary<Guid, NestedRecord>() { {Guid.NewGuid(), new NestedRecord([value])}}));
    await theSession.SaveChangesAsync();

    var results = await theSession.Query<RootRecord>()
        .SelectMany(x => x.Dict.Values).Where(x => x.Entities.Contains(value))
        .ToListAsync();

    Assert.Single(results);
}

Fixed the 2nd one, for the 1st one, it throws an exception with a suggestion to use MatchesSql() and the #> operator. Sorry man, that's way, way too much work for right now.


If Seesaw loss can be only used in image classification?

If Seesaw loss can be only used in image classification?
Do you try it on classification? how does it perform?

I haven't tried it, maybe you can create an issue in mmcls

hi @BIGWangYuDong
in the paper, the loss has been used for image classification, so do u have a plan to release it?


Hex or UUID mode

Is there a plan to add a hex or UUID mode?
For example,
{{hex}} → C918DA
{{uuid}} → 78bc209d-cade-4316-823c-a25401bfc659

+1，like uuid


Remove opacity bit

Implements #139

I'd rather only add sanity checks and refrain from packing values into longs. Packing introduces additional operations. Whilst these operations are binary operations, and thus very fast, they are additional operations we don't need.

I agree here, that's why it may be better to just add these sanity checks for total amount of entries and that neighbors inferred opacity bit didn;t change.

That seems ok. I don't see why expteme heights could break there, so leaving it as it. I'm curious now to see what happens if I remove height limits...


Cannot show preview of Camtrap DP

Previewing a pending version of a Camtrap DP does not work on ipt3.gbif-uat.org.
It indicates the resource does not exist:

Thanks for spotting that. Turns out it was not implemented for data packages


Cannot read property 'toLowerCase' of undefined

Confirm you have checked the WIKI for troubleshooting first
_Fill out and check ([x]) the boxes which apply.

Homebridge version: v1.2.4
Plugin version: 0.8.1
OS, including release name/version: alpine (Docker)
[ ] Homebridge-Onkyo installed via OS/distribution mechanisms
[ ] You installed using homebridge-config-ui-x
[ ] You enabled homebridge debug output

Describe the bug
The change introduced in b2f6d4462b91ff7493b99a709db09bcc91a59a8b converts all zone names to lowercase. However, if no zone is defined, toLowerCase() will be called on an undefined object, which in turn crashes homebridge. Therefore, the default value needs to be applied before the method is called.
To Reproduce
Information from the log:
[12/19/2020, 6:36:08 PM] [Onkyo] Initializing Onkyo platform...
[12/19/2020, 6:36:08 PM] [Onkyo] **************************************************************
[12/19/2020, 6:36:08 PM] [Onkyo]   homebridge-onkyo version 0.8.1
[12/19/2020, 6:36:08 PM] [Onkyo]   GitHub: https://github.com/ToddGreenfield/homebridge-onkyo
[12/19/2020, 6:36:08 PM] [Onkyo] **************************************************************
[12/19/2020, 6:36:08 PM] [Onkyo] start success...
[12/19/2020, 6:36:08 PM] TypeError: Cannot read property 'toLowerCase' of undefined
    at new OnkyoAccessory (/usr/local/lib/node_modules/homebridge-onkyo/index.js:70:32)
    at /usr/local/lib/node_modules/homebridge-onkyo/index.js:37:22
    at Array.forEach (<anonymous>)
    at OnkyoPlatform.createAccessories (/usr/local/lib/node_modules/homebridge-onkyo/index.js:30:13)
    at new OnkyoPlatform (/usr/local/lib/node_modules/homebridge-onkyo/index.js:23:8)
    at /usr/local/lib/node_modules/homebridge/src/server.ts:395:40
    at Array.forEach (<anonymous>)
    at Server.loadPlatforms (/usr/local/lib/node_modules/homebridge/src/server.ts:372:27)
    at Server.start (/usr/local/lib/node_modules/homebridge/src/server.ts:151:29)
    at cli (/usr/local/lib/node_modules/homebridge/src/cli.ts:80:10)
    at Object.<anonymous> (/usr/local/lib/node_modules/homebridge/bin/homebridge:17:22)
    at Module._compile (internal/modules/cjs/loader.js:1063:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
    at Module.load (internal/modules/cjs/loader.js:928:32)
    at Function.Module._load (internal/modules/cjs/loader.js:769:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)
[12/19/2020, 6:36:08 PM] Got SIGTERM, shutting down Homebridge...

It took me longer to figure out how to cherry pick the one commit than it did to solve the bug :D

published release 0.8.2 to address this issue.

Thanks for the quick response! :)


Loosing keystrokes when gamepad pressed

Given the below code:
pc.script.create('CameraManager', function (context) {

// Creates a new CameraManager instance
var CameraManager = function (entity) {

    this.entity = entity;

    this.cameraIndex = 0;
    this.cameraList = new Array("SceneCamera", "TurretCamera");
};

CameraManager.nextCAMERA = "nextCamera";
CameraManager.prevCAMERA = "prevCamera";

CameraManager.prototype = {

    initialize: function () {

        context.controller = new pc.input.Controller(document);

        context.controller.registerKeys(CameraManager.nextCAMERA, [pc.KEY_RIGHT, pc.PAD_RIGHT]);
        context.controller.registerKeys(CameraManager.prevCAMERA, [pc.KEY_LEFT, pc.PAD_LEFT]);

        this.camera = context.root.findByName('Camera');
        this.setCamera(this.cameraList[this.cameraIndex]);
    },

    update: function (dt) {

        if(context.controller.wasPressed(CameraManager.nextCAMERA)) {
            this.onNextCamera();
        }
        else if(context.controller.wasPressed(CameraManager.prevCAMERA)) {
            this.onPrevCamera();
        }

        //context.gamepads.update(dt);

        if(context.gamepads.wasPressed(pc.input.PAD_1, pc.PAD_FACE_1))
            this.onNextCamera();

        //context.gamepads.update(dt);
    },

    setCamera: function(entityName) {

        var o = this.entity.findByName(entityName);

        if(o)
        {
            this.camera.reparent(o, 0);

            var a = o.getLocalEulerAngles();
            this.camera.setEulerAngles(a.x, 180.0 - a.y, a.z);
        }
    },

    onPrevCamera: function() {
        if(this.cameraList && this.cameraList.length > 0) {
            this.cameraIndex--;

            if(this.cameraIndex < 0)
                this.cameraIndex = this.cameraList.length - 1;

            this.setCamera(this.cameraList[this.cameraIndex]);
        }
    },

    onNextCamera: function() {
        if(this.cameraList && this.cameraList.length > 0) {
            this.cameraIndex++;

            if(this.cameraIndex >= this.cameraList.length)
                this.cameraIndex = 0;

            this.setCamera(this.cameraList[this.cameraIndex]);
        }
    }
};

return CameraManager;

});

On initial load the game Controller processes the pc.KEY_LEFT and pc.KEY_RIGHT keys as expected. However, once the gamepad is pressed, specifically the 'A' button, keystrokes are not processed anymore.
Notice that both context.gamepads.update(dt) calls are commented out. This leads me to two questions:
1.a: Should application developers be required to call the gamepads.update function
1.b: If so, the proper placement should be documented if not already
1.c: How many places should the update be called? What happens if gamepad logic is spread among several source files?

. . .hopefully I can come back and edit this for the second item.

Adding more information:
The problem of lost keyboard occurs regardless of which call to the gamepads.update is uncommented. It doesn't matter if I call the gamepads.update before the gamepad check, or after the gamepad check. The keyboard is lost.
If you switch to gamepads.isPressed there is no problems. Also, the call to gamepads.update seems to be superfluous.

Closing due to age but also, since code example is in legacy script format, this is hard to investigate.


[BugFix] Fix bug multi partition support maxvalue

Fixes #28594
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[x] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 5 / 5 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/ast/RangePartitionDesc.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/ast/PartitionKeyDesc.java
4
4
100.00%
[]

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


Issues when mentioning users

The issue is now fixed, please try again and let me know @Daksh777

It worked, thank you!


Update README.md

ISSUE #52
ADDED MY NAME IN README.MD FILE

PLEASE ADD THE HACKTOBERFEST ACCEPTED FOR HELPING ME IN GETTINF IN POINTS.'

Done check once and message me again

In hacktoberfest website it shows that it is excluded

Check once i added some more labels

shows EXCLUDED.

still showing???

i think it takes time to show, i will give you update after few hours..

sure


Align baseline version for all tests

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.


change library to AboutLibraries for licenses 3

Issue

fix #592

JaCoCo Code Coverage 77.22% :white_check_mark:

Class
Covered
Meta
Status

  Generated by :no_entry_sign: Danger

Snapshot diff report

File name
Image


popup does not work on IOS

This is with IOS version 8.2 (on IPad 2nd gen).
I tried using the display popup mode from the angular example (https://github.com/anvilresearch/connect-example-angularjs, see https://github.com/henrjk/connect-example-angularjs/blob/tryit/README.md).
To enable testing from my devices I am using grunt serve of the example while changing the hostname to <IP_ADDRESS> here). This allows to access the page with the private IP address of my dev machine in my home network.
I configured the anvil-connect server so that callbacks and auth server also use that private IP address.
This works fine from the desktop and also on an Android Nexus 4, with Android 4.4.4 and Chrome 43.0.2357.93.
However when I touch SignIN on mentioned IPad nothing happens. The behaviour is the same in both  Chrome (43.0.2357.61) and Safari (from User-Agent: Version/8.0 Mobile/12D508 Safari/600.1.4).
Earlier I had made some changes in the anvil-connect.angular.js to see how opening a new tab would work instead of reloading the URL in display page mode.  As this had worked fine I suspected that the problem was the popup() function
Indeed when I comment out the || $document.documentElement.clientWidth and || $document.documentElement.clientHeight the popup works.
I have not yet actually debugged into this.
This is not directly related but I mention it as the resolution might be affected by this. I believe a display page mode which is presented in a new tab is a valid option also which makes more sense to me in a single page app then a page reload.

Connected Safari on Mac with IPad as described at stackoverflow.
Observed that $window screenX, screenLeft, screenY, screenTop, outerWidth and outerHeight were all 0.
Found this error in console:
[Error] Error: undefined is not an object (evaluating '$document.documentElement.clientWidth')
popup@http://<IP_ADDRESS>:9000/bower_components/anvil-connect/anvil-connect.angular.js:199:71
authorize@http://<IP_ADDRESS>:9000/bower_components/anvil-connect/anvil-connect.angular.js:502:52
signin@http://<IP_ADDRESS>:9000/scripts/app.js:101:22
http://<IP_ADDRESS>:9000/bower_components/angular/angular.js:10567:26
http://<IP_ADDRESS>:9000/bower_components/angular-touch/angular-touch.js:438:21
$eval@http://<IP_ADDRESS>:9000/bower_components/angular/angular.js:12412:28
$apply@http://<IP_ADDRESS>:9000/bower_components/angular/angular.js:12510:28
http://<IP_ADDRESS>:9000/bower_components/angular-touch/angular-touch.js:437:19
dispatch@http://<IP_ADDRESS>:9000/bower_components/jquery/dist/jquery.js:4435:14
handle@http://<IP_ADDRESS>:9000/bower_components/jquery/dist/jquery.js:4121:33
	(anonymous function) (angular.js, line 9778)
	(anonymous function) (angular.js, line 7216)
	$apply (angular.js, line 12512)
	(anonymous function) (angular-touch.js, line 437)
	dispatch (jquery.js, line 4435)
	handle (jquery.js, line 4121)

Further poking shows that $document.documentElement is undefined.
Next debugged this on a desktop Chrome.  There $window.outerWidth and $window.outerHeight were 1870 and 1058. In:
        width = $window.outerWidth || $document.documentElement.clientWidth
        height = $window.outerHeight || $document.documentElement.clientHeight

$document.documentElement is only evaluated if outerWidth or outerHeight are 0 which leads then to the exception causing the error.
On a side note I believe there is no way to see console errors on IOS 8.2 except through the connected desktop Safari.

See https://github.com/anvilresearch/connect-js/pull/5

Merged #5. Does this resolve #4 completely for you?

One could argue that for IOS all the popup calculation do not really apply.
However at the moment I haven't seen any problems but of course there is no automated test coverage so that is perhaps deceiving.
In general I think there should be another integration mode which opens the login in a different tab not a popup similar to how this behaves on IOS. But this is a different issue #6.
I am closing this one.


Added .idea and the bower_components folders to .npmignore

I think you have something like core.excludesfile config for .idea and bower_components, however, both of them are still included when someone downloads through the npm.
But, if you are keeping them on purpose, then sorry for the PR :)

You right thanks :) Will be fixed in next release!


Problems are not removed when issue notebook is closed

Testing #98821

Have following issue notebook

[{"kind":2,"language":"github-issues","value":"repo:microsoft/vscode assigne:@me label:settings","editable":true},{"kind":2,"language":"github-issues","value":"rep:microsoft/vscode assignee:@me label=1234","editable":true}]

Open the notebook
Open problems view and check there are problems
Close the notebook
🐛 Problems are not cleared. I think the GitHub issue notebook extension shall clear the diagnostics when notebook is closed.

They should, assuming this fires: https://github.com/microsoft/vscode-github-issue-notebooks/blob/0168f40b4db3e2c79cfeb045382b7cfbd85fed17/src/notebookProvider.ts#L81

The event does fire, I forget to clean the diagnostics collection...


Update Bulk Publish API response

In what area(s)?

/area runtime
Describe the feature

Related to #2218
Brief
The proposal is to alter two behavior

Instead of returning status code 200 with a list of success responses, return status code 204 with an empty response body
In case of 500 errors, instead of return a list of response with success/failure, return only the failed entry IDs.

EDIT: for gRPC,

statuses field in BulkPublishResponse will only contain failed entries
If the field is empty, it indicates successful publish for all entries

This will save a lot of data transfer in case of all messages being successfully published, or in case of partial failures.
Example 1, with N=100, if 1 message fails, only 1 entry ID will be present in the response instead of 100.
Example 2, with N=100, if no messages fails, the response body will be empty instead of containing 100 entries.
Current behavior
HTTP response codes

Code
Description

200
All messages delivered

400
Pubsub does not exist

403
Forbidden by access controls

500
At least one message failed to be delivered

Sample response:
Scenario 1: One message failed to be delivered
STATUS 500
{
  "statuses": [
    {"entryId": "ae6bf7c6-4af2-11ed-b878-0242ac120002", "status": "FAIL"},
    {"entryId": "b1f40bd6-4af2-11ed-b878-0242ac120002", "status": "SUCCESS"},
  ]
}

Scenario 2: All messages delivered successfully
STATUS 200
{
  "statuses": [
    {"entryId": "ae6bf7c6-4af2-11ed-b878-0242ac120002", "status": "SUCCESS"},
    {"entryId": "b1f40bd6-4af2-11ed-b878-0242ac120002", "status": "SUCCESS"},
  ]
}

Proposed behavior
HTTP response codes

Code
Description

~200~204
All messages delivered

400
Pubsub does not exist

403
Forbidden by access controls

500
At least one message failed to be delivered

Sample response:
Scenario 1: One message failed to be delivered
STATUS 500 - [UPDATED] The success returns should not be returned
{
  "statuses": [
    {"entryId": "ae6bf7c6-4af2-11ed-b878-0242ac120002", "status": "FAIL"},
  ]
}

Scenario 2: All messages delivered successfully
STATUS 204 - [UPDATED] The response body is now empty
(empty response body)
Release Note

RELEASE NOTE: UPDATE Bulk Publish API HTTP response

Please correct: in the response statuses, I think it should be entryId.

Thanks @DeepanshuA, corrected.

/assign


System Notifications: Asterisks was over writing current password causing notifications to stop working.

Was failing the check if clicking test notifications twice in a row. It was saving the asterisks and overwriting the current password.

[x] Ready for review
[x] https://redmine.pfsense.org/issues/9684

To effectively track changes and bug/feature requests, we require an entry on the pfSense Redmine issue tracker associated with every pull request, and likewise the Redmine entry should also have a link back to the pull request.
Please associate this PR to a Redmine issue either by locating an existing issue at https://redmine.pfsense.org or by creating a new issue. Add a link to the Redmine issue that points to this PR, and also add a link on the PR to the Redmine issue.
For more information, see https://www.netgate.com/docs/pfsense/development/submitting-a-pull-request-via-github.html
Thanks!

@jim-p I have added a redmine link.


update kube-register (0.0.4) in getting started guides

kube-register v0.0.4 can now register minions via hostnames using the --reverse-lookup option, making it unnecessary to override the hostname in the kubelet (--hostname-override)

I actually am not very familiar with the cloud formation templates.  I think this is a CoreOS thing - @bakins are you the right person to look at this?

I didn't do any magic inside the Cloudformation template, it's actually
just a copy/paste of the changes I applied to the cloud-config YAMLs.

LGTM, but I'd like to get @kelseyhightower 's opinion as well, since he maintains kube-register

+1 on the update to v0.0.4 and using the new URL for download.

FYI, with the 0.18.0 release next week the kubelet should self-register with the master and from testing that a few folks have done on CoreOS it should no longer be necessary to run kube-register.

@roberthbailey I just saw your comment in #6087, looking forward to it :)
I will create a new pull-request when 0.18.0 is out.


Missing "main" property in package.json

The package.json file is currently missing the "main" property to map to the correct .js file.
The issue has a known hotfix (https://stackoverflow.com/a/44205882) but it would be great if this rather silly problem could be fixed in the next version.

Can you send a PR? I am not familiar with Webpack.

There seems to be "main"-property defined in package.json and it does seem to point to the file I'd expect it to.
I wonder if this issue is still valid?


Unable to create Histogram metric

Describe the bug
I would like to create a Histogram metric, but I'm not able to.
To Reproduce
Steps to reproduce the behavior:
  using namespace prometheus;

  Registry MetricsRegistry;

  using AudioFreqHistogram = Histogram<double>;
  using AudioFreqHistogramFamily = CustomFamily<AudioFreqHistogram>;

  AudioFreqHistogramFamily &audioFreqHistogram{
      AudioFreqHistogram::Family::Build(MetricsRegistry, "frequencies", "audio frequencies, Hz", {})
  };

  std::vector<double> frequencyBoundaries;
  for (int i = 0; i < 8000; i += 100) {
    frequencyBoundaries.push_back(i);
  }

  // ERROR In template: no matching member function for call to 'insert'
  auto &freqHist1 = audioFreqHistogram.Add({}, frequencyBoundaries);

  // ERROR Call to implicitly-deleted copy constructor of 'prometheus::Histogram<double>'
  Histogram<double> freqHist2 = Histogram<double>{frequencyBoundaries};

  // ERROR In template: call to implicitly-deleted copy constructor of 'value_type' (aka 'prometheus::Counter<double>')
  prometheus::Histogram<double> freqHist3 = Histogram<double>(frequencyBoundaries);

Additional context
I'm using PlatformIO and have made a few alterations to the code, like replacing dynamic_cast
//          return dynamic_cast<CustomMetric&>(*metrics_iter->second);
          return (CustomMetric&)(*metrics_iter->second);

Should be fixed by https://github.com/biaks/prometheus-cpp-lite/pull/18
Thanks @biaks!


await rens's response for viaappia video

I'd like to use https://youtu.be/I3DLXSrRiyk instead of
https://github.com/jspaaks/collab-demos/blob/master/demos/viaappia/screencast-demo-viaappia.mp4.

All good. Closing this issue


Normalize shebangs in shell scripts

First step in enabling linter checks to every shell script
Note that using /usr/env/bash allows bash to exist at a different path than /bin/bash, but /bin/bash is depended on by other scripts and, as far as I can tell, is actually more standard than the existence of /usr/env/bash.
/assign @mandarjog

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Tahler
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

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

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
dbc9bc7b5faa3d4463cb35aad83ab7e90089790e
link
/test e2e-simple

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.

Thanks. I suggest we only use bash (sh on mac is likely to be problematic).
Seems like you have some longer term task here. Do you mind documenting the style guide that we should follow in a wiki? I suggest https://google.github.io/styleguide/shell.xml (which incidentally is what k8s uses).

@kyessenov Yeah my longer term task is to use Shellcheck, which is an awesome tool that encourages sane practices in shell scripts and is becoming increasingly widely used. I'm enabling a CI job and then fixing the various violations we have in our repository incrementally (e.g. SC2152 and SC2086).
An awesome part of Shellcheck is it enforces different rules whether it's an "sh" script or a "bash" script and it will ensure that you're actually using Posix-compliant / cross-platform features so it shouldn't become a problem on Mac. Some of these scripts have good reasons to be "sh" as far as I know.
I don't think I can commit to fixing style issues - we have an insane amount of shell scripts - but I do think it's a good idea to follow the Google style guide going forward. I guess I'll write up a wiki page.

@Tahler it would be good to enable the ShellCheck linter with the specific aspect you are fixing in this and other PRs.

@mandarjog I'm not fixing a rule here actually I'm only adding some consistency which helps my tool identify which files are shell scripts that need to be linted. The next PR will enable ShellCheck.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Tahler
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: mandarjog
If they are not already assigned, you can assign the PR to them by writing /assign @mandarjog in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mandarjog, Tahler
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mandarjog]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add Hover Effects on Buttons and Smooth Scrolling

Hello @GarimaSingh0109
I want to make two improvements to enhance the UI:

Add uniform hover effects on links in the navbar and the submit button in both the light and dark mode.
Implement smooth scrolling to the desired section when those links are clicked.

I believe these changes would give a the website a much more modern look and feel.
Current Behaviour:
https://github.com/user-attachments/assets/ec3b9d68-d051-4f7d-94ad-d2f38dd661cc
Expected Behavious After Changes:
https://github.com/user-attachments/assets/506a7e7f-6aa6-4b9c-9cea-e7eeca0f4add

I would love to work on this issue as part of GSSoC'24 Extd and Hacktoberfest. Please assign it to me. Looking forward to your response!

I would love to work on this issue as part of GSSoC'24 Extd and Hacktoberfest. Please assign it to me.


Add entry url validation for unzip utility. Defends against Zip Slip

Changes to validate that the entries within a zip archive have valid paths. Defends against Zip Slip attack.

Looks good to me. @cdhoffmann have you manually tested this change?

The test i wrote uses the same zip file found in the android tests. It has an entry in the archive named "../Makefile". If you breakpoint through the test you can see it. Lmk if there are other manual tests you think i should do.

Looks good to me. @cdhoffmann have you manually tested this change?

The test i wrote uses the same zip file found in the android tests. It has an entry in the archive named "../Makefile". If you breakpoint through the test you can see it. Lmk if there are other manual tests you think i should do.

@cdhoffmann it'd be better if you can quickly test it on a physical device.

@cdhoffmann it'd be better if you can quickly test it on a physical device.

@cdhoffmann it'd be better if you can quickly test it on a physical device.

@shalehaha Verified on iPhone 12 Pro iOS 14.2.1

@cdhoffmann it'd be better if you can quickly test it on a physical device.

@shalehaha Verified on iPhone 12 Pro iOS 14.2.1


repeated connect errors for embeddings endpoint

Since yesterday I'm getting loads of these errors when using the embeddings API:
gaierror(8, 'nodename nor servname provided, or not known')), ClientConnectorError(ConnectionKey(host='api.openai.com', port=443, is_ssl=True
This happens on more than 50% of connection attempts when using the api_request_parallel_processor.py script from this repository. It doesn't seem to matter what frequency I make the requests at (although it's always pretty high in my tests, I'm submitting >300k items to embed).

Nope sorry git it, digging deep through the logs I spotted this error that probably explains it?
`
WARNING:root:Request 224657 failed with error {'message': 'The server is currently overloaded with other requests. Sorry about that! You can retry your request, or contact us through our help center at help.openai.com if the error persists.', 'type': 'server_error', 'param': None, 'code': None}
`

WARNING:root:Request 224657 failed with error {'message': 'The server is currently overloaded with other requests. Sorry about that! You can retry your request, or contact us through our help center at help.openai.com if the error persists.', 'type': 'server_error', 'param': None, 'code': None}

@stephansturges I'm also getting that error repeatedly, which is new for me after a couple months of using the API.
If the error message is true, the OpenAI servers are simply overwhelmed.
OpenAI recommends implementing retry mechanisms with exponential backoff:
https://platform.openai.com/docs/guides/rate-limits/retrying-with-exponential-backoff
My code doesn't have the feature yet, it's never been necessary until now as a prudent sleep after every call has been enough, but that's not cutting it anymore so I guess the only solution is to implement it.

Implementing exponential backoff with Tenacity is a breeze - in my case it's been a literal 5min edit to my code. Has sorted the issue completely.

I'm not familiar with the cookbook nor its contribution policy - I found this Issue by searching on Google, but the gist of it is:
import openai
from openai.error import RateLimitError
from tenacity import (
    retry,
    wait_exponential_jitter,
    stop_after_attempt,
    retry_if_exception_type,
)

# There are several exponential back-off methods in Tenacity; I picked the one with jitter
@retry(retry=retry_if_exception_type(RateLimitError), wait=wait_exponential_jitter(initial=1, max=30))
def open_ai_wrapper():
     response = openai.your_choice_of_call()
     # Do whatever
     return value

def caller():
    # Your function now handles exponential back-off automatically
    open_ai_wrapper()

You just need to add a decorator on your wrapping function definition. The Tenacity docs are super-clear and easy to follow, and include other decorator options, such as stopping after a set number of retries.

Awesome, thanks!

Two things that would be great to add in the parallel embdeddings calculator: exponential backoff for rate limit + token length calculation and automatic limit to the first 8000 tokens for submission.

What do you mean by parallel embeddings calculator, exactly?

I mean this :)
https://github.com/openai/openai-cookbook/blob/main/examples/api_request_parallel_processor.py
It works great, but would be great to limit the request to the max length of tokens that the model can handle (ie 8191 for V2) by taking the first X tokens that get to this length and discarding the rest of the request, and it gets overloaded and fails when the API is getting hammered so implementing this out of the gate might be useful? https://platform.openai.com/docs/guides/rate-limits/error-mitigation

It would be cool if there was a way to share a bucket of JSONL files with OpenAI to get the embeddings calculations done some other way than the API. I'm trying to get them done for about 10M units of content and at the current rate it will take about 20 days to do those. Or an option to run this locally would be fantastic!


Approved SessionManager changes

Basic implementation of the SessionManager cookie-handling logic.

@Lukasa, if you want to merge this, then I can start submitting PRs against that branch.

I have some feedback!

@Lukasa, should be good if you want to take another look. 🙂

Alright, I pronounce myself happy! I'd like a section in CHANGES.rst please, and then I'd also like any other contributor who is interested to please do a once-over review. If I don't hear anything else I'll aim to merge sometime this week.

Rebased from 4aee526.

@Lukasa, @sigmavirus24, I've made a couple changes and believe this is ready to merge into the working branch. From there, we can get into exactly what it needs to look like before it merges to master.

ACK. I have no time for a detailed review of this today, but I'll do it tomorrow or Monday. =)

Ok, let's move this onward.


mongodb port

https://github.com/OWASP/RiskAssessmentFramework/blob/master/user-guide.md#step-2
it only works for me, if i use the (default mongodb) port 27017 instead of 28017. maybe this is a typo?

@adeyosemanputra is this project is part OWASP GSoC project this year


chore: release 5.2.0

:robot: I have created a release *beep* *boop*
5.2.0 (2019-08-09)
Features

populate x-goog-api-client header for auth (#772) (526dcf6)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/google-auth-library-nodejs/releases/tag/v5.2.0

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


Add option --sample_at_first

:sparkles: Generate samples before training

This is useful for comparison before and during the training

:spiral_notepad: Note
There is a possibility that the images generated by this and the images generated by sdxl_gen_img.py (or other generation scripts) are slightly different, even with the same parameters. Whether this is true or not has not yet been investigated.

I have noticed that the reason for the difference in results may be due to --full_bf16 or --full_bf16 in training.
It would be better to add is_full argument to sdxl_train_util.load_target_model() and convert unet, text_encoder1, text_encoder2 to fp16 or bf16.

I have noticed that the reason for the difference in results may be due to --full_bf16 or --full_bf16.
It would be better to add is_full argument to sdxl_train_util.load_target_model and convert the unet, text_encoder1, text_encoder2 in it.
Related to #628

This seems like a good feature.  I found myself wanting it just last week.


The $text arg must be a single string literal, not "$var"

Hi.
I'm running into an issue where I could not find a way to bypass a error message.
Code example:
		<div class="warning-text" style="<?php echo esc_attr( $style_message ); ?>">

<?php esc_html_e( $options['content']['textarea_warning_text'], 'custom-cookie-message' ); ?>
				<a style="<?php echo esc_attr( $style_link ); ?>" href="<?php echo esc_url( $options['general']['cookies_page_link'] ); ?>" title="<?php esc_html_e( $options['content']['input_link_text'], 'custom-cookie-message' ); ?>"><?php esc_html_e( $options['content']['input_link_text'], 'custom-cookie-message' ); ?>

			<a id="custom-cookie-message-preference" class="button <?php echo esc_attr( $style_button_class ); ?>">
				<?php esc_html_e( $options['content']['input_button_text'], 'custom-cookie-message' ); ?>

The error message is:
The $text arg must be a single stirng literal, not "$var"

I'm not pointing that the Error message could be wrong, my question is how could I whitelist this error, since this code is in a template for a plugin where that text come from a setting option.
My phpcs version is: PHP_CodeSniffer version 3.1.1 (stable)
My WPCS version is: "wp-coding-standards/wpcs": "^0.13.1"
You could read the entire code here. https://github.com/killua99/custom-cookie-message/blob/feature/cookie-popup-template/templates/cookie-notice.php

esc_html_e() is for internationalising (and escaping) plain strings in code. Not for strings stored in a database. By the time it's stored, the chance to internationalise it with the usual functions has gone.
There are, I believe, other workarounds that work with WPML, but I'm not overly familiar with them.
So, there's no whitelisting needed here - just change esc_html_e to esc_html and remove the second arg.

Why should I change the function when it clearly is the best use, and data validation in WordPress.
Both function start with the same object, to escape the front-end in the right way, and use the _e in order to add the ability to translate it. If WPCS accept esc_html to pass a variable and not esc_html_e I see it like is not coherent.
Both functions use the same parameter:
esc_html
$text
    (string) (required) Text to escape

        Default: None

esc_html_e
$text
    (string) (required) Text to translate

        Default: None

I see it right to add or allow variables inside esc_html_e() and esc_html__()

I am not familiar with WPML, but the reason that you normally shouldn't pass a variable to translation functions is because the POT file cannot automatically be generated then (because the string to translate isn't within the function call, so an automatic parser can't find it).
Because of that, there is no whitelisting comment specifically for this sniff, but you can always use // @codingStandardsIgnoreLine.  You could also exclude this particular error in your custom ruleset:
<exclude name="WordPress.WP.I18n.NonSingularStringLiteralText" />

I'm still not sure that you should be passing a variable to this function though...

Why should I change the function when it clearly is the best use, and data validation in WordPress.

@killua99 I think you've missed the point of my post.
It's not about the escaping.
It's about you trying to internationalise a variable. Yes, it might be a variable that, when parsed with a PHP engine, happens to resolve to a string, but that's not how the WP i18n tools work.
See Law the First.
esc_html_e() is not the best function to use here, because you don't have the plain text string - only a variable holding a string. In an environment where the extraction of strings doesn't execute PHP, then it can't read what your variable holds. Since this is a non-starter, WPCS helpfully points out with a very clear message: The $text arg must be a single string literal, not "$var".
What you really want here is:
<?php echo esc_html( $options['content']['textarea_warning_text'] ); ?>

I don't know how the rest of your plugin works, but if someone wants the warning message in a different language, then they would choose the language at the point of input, and not on output.

I see the point of you guys, and thanks for the options you show me here. I do understand that in this case, is my fight. Also I do understand and I'm not 100% happy passing a variable into an _e function, coming from Drupal I'm more than clear that is not the best way to work with translation. Just curios if there was a whitelist for that case because I know what I'm doing there.
So I'm going to use the option that @GaryJones mention, and just use esc_html() and see how it work.
I'm just going to leave this extra info for documentation purpose, I do believe could be useful for someone out there. But this issue matter add nothing to the conversation and for this could be close, I guess nothing else could be added to it, thanks for clarify all. 😀
When do you work in a multilingual WordPress site you have two good options, WPML or Polylang. The last one (Polylang) as very nice one and nice to work with translations in WordPress and in our experience we like it more than the old friend WPML.
The case is, WPML has an ability to create a map file for customs fields and extra strings to be available for translations. You work with a file called wpml-config.xml that files map yours fields and the PO files will be created just right.
With Polylang you could use functions like pll_e() that they do the translation coming from they register strings. Also works the normal translations and escapes likek esc_html_e() (that's the reason I was using it).
https://wpml.org/documentation/support/language-configuration-files/#admin-texts
https://polylang.pro/doc/the-wpml-config-xml-file/

@killua99 Just FYI - For multilingual websites, there is a third option: Multilingual Press. From all I've heard, that is nowadays the preferred choice.
Ref: https://wordpress.org/plugins/multilingual-press/


clamav: 1.3.0 -> 1.3.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/clamav/versions.
meta.description for clamav is: Antivirus engine designed for detecting Trojans, viruses, malware and other malicious threats
meta.homepage for clamav is: https://www.clamav.net
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.3.1 with grep in /nix/store/xmbnw1k9vgi02zy8s9vpsc8dababmlzb-clamav-1.3.1
found 1.3.1 in filename of file in /nix/store/xmbnw1k9vgi02zy8s9vpsc8dababmlzb-clamav-1.3.1

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

clamav
clamtk

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xmbnw1k9vgi02zy8s9vpsc8dababmlzb-clamav-1.3.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A clamav https://github.com/r-ryantm/nixpkgs/archive/fd0ca03e574028aef7c07d5fe993048eed886271.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/fd0ca03e574028aef7c07d5fe993048eed886271#clamav

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xmbnw1k9vgi02zy8s9vpsc8dababmlzb-clamav-1.3.1
ls -la /nix/store/xmbnw1k9vgi02zy8s9vpsc8dababmlzb-clamav-1.3.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

3 packages built:

clamav
clamtk
clamtk.man

Maintainer pings
cc @robberer @qknight @globin for testing.

@ofborg build clamav clamav.passthru.tests


Extremely hard to notice listchars

First, ty for this fantastic colorscheme, love it :-)
As you can see below the current listchars highlights makes it extremley hard to notice some listchars (aside from the EOL):

the right side of the screen is the embark colorscheme with transparent background:

Hello @ibhagwan,
I believe you use Neovim since you are the maintainer of fzf-lua.
The color of listchars is dictated by the Whitespace highlight group, in Neovim, which influences not only listchars but indent-blankline  indent markers and highlighting in the QuickFix list. So a very subtle color is a deliberate choice.
Though I can understand your issue; thankfully there is an easy solution, just change the color of Whitespace as follows (after setting colorscheme):
if g:colors_name == 'nightfly'
    highlight! link Whitespace NonText
endif

Add to the end of init.vim or in an after file such as after/plugin/colorscheme.vim (in your Neovim runtime path).
Cheers.

Thank you for the prompt reply @bluz71!

I believe you use Neovim since you are the maintainer of fzf-lua.

Correct.

The color of listchars is dictated by the Whitespace highlight group, in Neovim, which influences not only listchars but indent-blankline indent markers and highlighting in the QuickFix list. So a very subtle color is a deliberate choice.

Makes sense, I didn’t know indent-blankline uses the same highlights, I recently started using to mini-indentscope from mini.nvim (amazing library btw, the surround plugin is the best I’ve used and the indent scope text object is very useful).

Though I can understand your issue; thankfully there is an easy solution, just change the color of Whitespace as follows (after setting colorscheme)

Thank you for the suggestion, will modify my config accordingly :)

No problem.
I am pleased you like the colorscheme.
Keep up the good work you do in the open-source community.
Cheers.


Patch on to-one relationships doesn't work due to issues with { "data": null } unmarshal

If a user wants to remove a to-one relationship a PATCH should be done on the /resource/:resourceId/relationships/relatedResource, with a body containing the following:
{
    "data": null
}

(http://jsonapi.org/format/#crud-updating-to-one-relationships)
But, when err := jsonapi.UnmarshalPayload(c.Request.Body, data) is executed, an error is given with the following message "data is not a jsonapi representation of '*chronos.RelatedResource'". What should I do to allow null values on jsonapi representations?

@brunoluiz Did you ever find a solution for this? A different library? It looks like a PR to make this work was added in 2017 but in 2018 work was added which introduces the error.

@brunoluiz Did you ever find a solution for this? A different library? It looks like a PR to make this work was added in 2017 but in 2018 work was added which introduces the error.

I can't remember, but I guess I didn't find a solution after all. Nowadays,
I always end-up using GraphQL for public APIs, so I haven't used this
package anymore.
On Tue, Jan 26, 2021, 11:58 AM Carl Markham<EMAIL_ADDRESS>wrote:

@brunoluiz https://github.com/brunoluiz Did you ever find a solution
for this? A different library? It looks like a PR to make this work was
added in 2017 but in 2018 work was added which introduces the error.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/jsonapi/issues/154#issuecomment-767599951, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYR7ATQH45R7GDNNDTF4GDS33KBPANCNFSM4FR2MFKQ
.

I can't remember, but I guess I didn't find a solution after all. Nowadays,
I always end-up using GraphQL for public APIs, so I haven't used this
package anymore.
On Tue, Jan 26, 2021, 11:58 AM Carl Markham<EMAIL_ADDRESS>wrote:

@brunoluiz https://github.com/brunoluiz Did you ever find a solution
for this? A different library? It looks like a PR to make this work was
added in 2017 but in 2018 work was added which introduces the error.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/jsonapi/issues/154#issuecomment-767599951, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYR7ATQH45R7GDNNDTF4GDS33KBPANCNFSM4FR2MFKQ
.


Group by on two columns and transform very slow

Centos-release-7-5.1804.5.el7.centos.x86_64
Modin <IP_ADDRESS>
Python 3.8.3 (default, Jul  2 2020, 16:21:59) [GCC 7.3.0] :: Anaconda, Inc. on linux
The following takes a few seconds in Pandas, in Modin takes hours.
#import pandas as pd
import modin.pandas as pd
import time

data = [
        [1, 1, 1],
        [1, 1, 2],
        [1, 2, 3],
        [1, 2, 4],
        [2, 1, 10],
        [2, 1, 20],
        [2, 2, 30],
        [2, 2, 40],
    ]

df = pd.DataFrame(data, columns = ['card_id', 'day', 'amount'])
df = pd.concat([df for _ in range(1000000)])
start = time.time()
df['count'] = df.groupby(['card_id', 'day'])["amount"].transform('count')
df['sum'] = df.groupby(['card_id', 'day'])["amount"].transform('sum')
df['std'] = df.groupby(['card_id', 'day'])["amount"].transform('std')
df['min'] = df.groupby(['card_id', 'day'])["amount"].transform('min')
df['max'] = df.groupby(['card_id', 'day'])["amount"].transform('max')
end = time.time()
print("{0} seconds".format((end - start)))

Hi @davidjhp , thanks for posting! Please, look at this reply. It's interesting why you are creating df via such logic? It that really necessary?

This seems like a benchmarking attempt, but @davidjhp might be missing some key understanding of how Modin works.
In general, this type of dataframe creation is extremely unusual. Does the same issue occur if you write the data out to a csv file first?


[Bug] OpenDigger data api will change time format

Version
No response
Current Behavior
As discussed in https://github.com/X-lab2017/open-digger/issues/1074 and merged in https://github.com/X-lab2017/open-digger/pull/1078, we will change time format from yyyym(m) to yyyy-mm.
So I think hypercrx need to be ready for this change.
Expected Behavior
No response
Environment
- OS:
- Browser:

Any additional comments?
No response

Got that, I will handle this. Thank you for your reminding!
/self-assign


exception 'NSInternalInconsistencyException'

Hi, using:
This beginner-level config uses the CPU (gpuID: -1)
to compress RGB video
using ffmpeg's X264 codec
Recording parameters
videoFolder: "/Users/piotr/basler_984"
videoFilename: "984.mp4"
frameRate: 10
recTimeInSec: 5
Camera parameters
numCams: 1
cameraMake: "basler"
cameraSettings: "./configs/mono8.pfs"
frameWidth: 640
1152
frameHeight: 480
1024

recTimeInSec: 3
Compression parameters
gpuID: -1
pixelFormatInput: "gray"
rgb24"
pixelFormatOutput: "rgb0"
quality: 17
ffmpegLogLevel: "warning"

ffmpegPath: "/Users/piotr/anaconda3/envs/campy/bin/ffmpeg"
"/usr/bin/ffmpeg"       # Location of ffmpeg binary if not using Anaconda
Display parameters
displayFrameRate: 10
Trigger parameters
cameraTrigger: "software"
I am getting:
(campy) Piotrs-MacBook-Pro:campy piotr$ campy-acquire /Users/piotr/campy/configs/campy_config.yaml
Opened: /Users/piotr/basler_984/Camera0/984.mp4 using CPU to compress the stream.
-vsync is deprecated. Use -fps_mode
Passing a number to -vsync is deprecated, use a string argument as described in the manual.
2023-06-14 23:30:35.878 python3.8[33737:1177938] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'NSWindow drag regions should only be invalidated on the Main Thread!'
*** First throw call stack:
(
0   CoreFoundation                      0x0000000195083154 __exceptionPreprocess + 176
1   libobjc.A.dylib                     0x0000000194ba24d4 objc_exception_throw + 60
2   CoreFoundation                      0x00000001950ac3a0 _CFBundleGetValueForInfoKey + 0
3   AppKit                              0x000000019824ca00 -[NSWindow(NSWindow_Theme) _postWindowNeedsToResetDragMarginsUnlessPostingDisabled] + 320
4   AppKit                              0x000000019823acd4 -[NSWindow _initContent:styleMask:backing:defer:contentView:] + 816
5   AppKit                              0x000000019823a998 -[NSWindow initWithContentRect:styleMask:backing:defer:] + 48
6   AppKit                              0x000000019849a124 -[NSWindow initWithContentRect:styleMask:backing:defer:screen:] + 24
7   libqcocoa.dylib                     0x000000016232159c qt_plugin_instance + 132188
8   QtWidgets.abi3.so                   0x000000016966a31c ZL17meth_QWidget_showP7_objectS0 + 96
9   python3.8                           0x0000000104ea4428 cfunction_call_varargs + 360
10  python3.8                           0x0000000104ea3af0 _PyObject_MakeTpCall + 744
11  python3.8                           0x0000000104f90158 call_function + 612
12  python3.8                           0x0000000104f8c83c _PyEval_EvalFrameDefault + 27176
13  python3.8                           0x0000000104ea47b4 function_code_fastcall + 128
14  python3.8                           0x0000000104f900c0 call_function + 460
15  python3.8                           0x0000000104f8c818 _PyEval_EvalFrameDefault + 27140
16  python3.8                           0x0000000104f857fc _PyEval_EvalCodeWithName + 3008
17  python3.8                           0x0000000104ea4918 _PyFunction_Vectorcall + 208
18  python3.8                           0x0000000104ea7ea4 method_vectorcall + 520
19  python3.8                           0x0000000104ea3fd8 PyVectorcall_Call + 120
20  python3.8                           0x0000000104f8caf0 _PyEval_EvalFrameDefault + 27868
21  python3.8                           0x0000000104f857fc _PyEval_EvalCodeWithName + 3008
22  python3.8                           0x0000000104ea4918 _PyFunction_Vectorcall + 208
23  python3.8                           0x0000000104f900c0 call_function + 460
24  python3.8                           0x0000000104f8c934 _PyEval_EvalFrameDefault + 27424
25  python3.8                           0x0000000104ea47b4 function_code_fastcall + 128
26  python3.8                           0x0000000104f900c0 call_function + 460
27  python3.8                           0x0000000104f8c8b8 _PyEval_EvalFrameDefault + 27300
28  python3.8                           0x0000000104ea47b4 function_code_fastcall + 128
29  python3.8                           0x0000000104ea3fd8 PyVectorcall_Call + 120
30  python3.8                           0x0000000104f8caf0 _PyEval_EvalFrameDefault + 27868
31  python3.8                           0x0000000104ea47b4 function_code_fastcall + 128
32  python3.8                           0x0000000104f900c0 call_function + 460
33  python3.8                           0x0000000104f8c818 _PyEval_EvalFrameDefault + 27140
34  python3.8                           0x0000000104ea47b4 function_code_fastcall + 128
35  python3.8                           0x0000000104f900c0 call_function + 460
36  python3.8                           0x0000000104f8c818 _PyEval_EvalFrameDefault + 27140
37  python3.8                           0x0000000104ea47b4 function_code_fastcall + 128
38  python3.8                           0x0000000104ea7de8 method_vectorcall + 332
39  python3.8                           0x0000000104ea3fd8 PyVectorcall_Call + 120
40  python3.8                           0x000000010504b8e4 t_bootstrap + 196
41  python3.8                           0x0000000104fe75d8 pythread_wrapper + 48
42  libsystem_pthread.dylib             0x0000000194f2bfa8 _pthread_start + 148
43  libsystem_pthread.dylib             0x0000000194f26da0 thread_start + 8
)
libc++abi: terminating due to uncaught exception of type NSException
[vost#0:0/libx264 @ 0x1426062d0] One of -r/-fpsmax was specified together a non-CFR -vsync/-fps_mode. This is contradictory.
This is on Mac. Can you help, please? Thanks!

2023-06-17 22:36:04.566 python[4241:35549] WARNING: NSWindow drag regions should only be invalidated on the Main Thread! This will throw an exception in the future. Called from (
0   AppKit                              0x00007ff81067721d -[NSWindow(NSWindow_Theme) _postWindowNeedsToResetDragMarginsUnlessPostingDisabled] + 338
1   AppKit                              0x00007ff810663714 -[NSWindow _initContent:styleMask:backing:defer:contentView:] + 1227
2   AppKit                              0x00007ff810663242 -[NSWindow initWithContentRect:styleMask:backing:defer:] + 42
3   AppKit                              0x00007ff81092e323 -[NSWindow initWithContentRect:styleMask:backing:defer:screen:] + 50
4   libqcocoa.dylib                     0x0000000177fde725 -[QNSWindow initWithContentRect:styleMask:backing:defer:screen:platformWindow:] + 197
5   libqcocoa.dylib                     0x0000000177fcbcac _ZN12QCocoaWindow14createNSWindowEb + 1324
6   libqcocoa.dylib                     0x0000000177fc5672 _ZN12QCocoaWindow22recreateWindowIfNeededEv + 1266
7   libqcocoa.dylib                     0x0000000177fc50fe _ZN12QCocoaWindow10initializeEv + 318
8   QtGui                               0x0000000178672eb7 _ZN14QWindowPrivate6createEby + 151
9   QtWidgets                           0x000000017a2bf771 _ZN14QWidgetPrivate6createEv + 1185
10  QtWidgets                           0x000000017a2be414 _ZN7QWidget6createEybb + 324
11  QtWidgets                           0x000000017a2d181d _ZN14QWidgetPrivate10setVisibleEb + 189
12  QtWidgets.abi3.so                   0x000000017967e93f _ZN14sipQMainWindow10setVisibleEb + 111
13  QtWidgets.abi3.so                   0x000000017983a34b ZL17meth_QWidget_showP7_objectS0 + 75
14  python3.7                           0x0000000102df6cae call_function + 3182
15  python3.7                           0x0000000102de287c _PyEval_EvalFrameDefault + 45468
16  python3.7                           0x0000000102cb169e _PyFunction_FastCallKeywords + 398
17  python3.7                           0x0000000102de025d _PyEval_EvalFrameDefault + 35709
18  python3.7                           0x0000000102dd4b35 _PyEval_EvalCodeWithName + 677
19  python3.7                           0x0000000102cb84d1 _PyObject_Call_Prepend + 1553
20  python3.7                           0x0000000102de15b1 _PyEval_EvalFrameDefault + 40657
21  python3.7                           0x0000000102df6969 call_function + 2345
22  python3.7                           0x0000000102ddbabd _PyEval_EvalFrameDefault + 17373
23  python3.7                           0x0000000102df626b call_function + 555
24  python3.7                           0x0000000102ddef39 _PyEval_EvalFrameDefault + 30809
25  python3.7                           0x0000000102de233a _PyEval_EvalFrameDefault + 44122
26  python3.7                           0x0000000102cb169e _PyFunction_FastCallKeywords + 398
27  python3.7                           0x0000000102de025d _PyEval_EvalFrameDefault + 35709
28  python3.7                           0x0000000102cb169e _PyFunction_FastCallKeywords + 398
29  python3.7                           0x0000000102de025d _PyEval_EvalFrameDefault + 35709
30  python3.7                           0x0000000102cb81d1 _PyObject_Call_Prepend + 785
31  python3.7                           0x0000000102cb5124 PyObject_Call + 116
32  python3.7                           0x0000000102ea90db t_bootstrap + 171
33  python3.7                           0x0000000102e4af74 pythread_wrapper + 36
34  libsystem_pthread.dylib             0x00007ff80d5831d3 _pthread_start + 125
35  libsystem_pthread.dylib             0x00007ff80d57ebd3 thread_start + 15
)
2023-06-17 22:36:04.632 python[4241:35549] WARNING: nextEventMatchingMask should only be called from the Main Thread!  Please try installing campy in the AV1 beta branch, which now uses an opencv display window and has better error handling

Thanks for your response! Since June, we have been able to use Campy on Mac to successfully record several 45-min videos from 3 Basler cameras at 60 fps! While the error remains, the videos are OK. The only difference between Mac and Windows is that, on Windows one can see the windows showing what the cameras see, while in Mac, perhaps due to this installation problems, we do not see them. As to the AV1 beta - sorry I do not understand... bud gladly learn how to do that. Thanks | Piotr

The AV1 branch is here if you want to try the newer version: https://github.com/ksseverson57/campy/tree/AV1
Glad to hear you got campy working on Mac! I can now say campy with Basler cameras is compatible with Mac thanks to you. Hopefully the new branch is also compatible, which might require some changes to your config file and the trigger module if you are using "trigger.ino" to sync your cameras

Yes, we use Arduino for cameras synchronization. I think the initial problem (sorry, it was a long time ago) was with brew / python installation: a helpful trick for the Mac users: when moving from an old Mac architecture to the novel, and simply copying all packages, remember to install python for the novel architecture!
I think the lack of camera display is due to qt@5 - whatever that is - problems....
While I have little knowledge in programming, following proper python installation, we were able to use Campy.
Just let me know if I can be of any help.
As to the novel Campy version: one day I'll give a try, because we use the system almost everyday for animal research, and have to have it working, even with limited capabilities. Anyway: thank you Kyle for your fantastic work | Piotr

thank you again for raising the issue and including all relevant information to isolate the problem. Sorry for taking so long to respond. I'm busy with experiments, too. If your campy recording is working, no need to change anything. Good luck with everything Piotr!


Yarn add error: Cannot read properties of undefined (reading 'name')

Maybe it is an issue with craco? care to post a reproducible repo?


[bug] Style propagation issues with yarn web:prod

Describe the bug
Happening on version 199 + 200 and older versions.
Using the following component with the Next starter with JSX enabled placed within the main homescreen function I'm getting differences in styling when using yarn web:prod as opposed to yarn web.

Yarn web

yarn web:prod (Ignore contrast difference)

To Reproduce
https://github.com/GigaBigBoss/tamagui
Ensure the ThemePropagationTest component is inside main homescreen function and  the the scrollviewdemo component isn't.
https://github.com/GigaBigBoss/tamagui/blob/master/starters/next-expo-solito/packages/app/features/home/screen.tsx

I think this is fixed in 201, if you can just lmk if that fixes.

I think this is fixed in 201, if you can just lmk if that fixes.

Issue still present

Didn't realize you're working off a fork of the tamagui monorepo itself, that's a recipe for confusion because everything in starters isn't the same node modules as the root...

this is actually an issue with ... hadn't thought this one through, with flattening group will break :/ I am not the biggest fan of Group for this reason (cloning children), thinking of what would be best here..

Fixed in https://github.com/tamagui/tamagui/commit/377d5b38b91489834ede6cefeab430ffaacaba9a
No longer doing weird stuff with size props in groups


⚠ TorrentLeech has degraded performance

In 339e9ca, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 4590 ms

Resolved: TorrentLeech performance has improved in 26d6e46 after 8 minutes.


Add information about licensing

I just added added a License file to the repo. This library is now licensed under the MIT license.

Thank you for information


Attempt to decouple benchmark from test

This is an attempted solution to #753 in order to slightly speed up build times. Instead of benchmark referring to tpcc in test, we refer to our own local copy in benchmark_util.

I don't think that this is the right approach. We don't want to duplicate the test_util code. We just need to modify CMake so that the test_util target gets built when tests aren't being built but benchmarks still are.


Fix DateTime bug

Fix DateTime Bug in prepare release script.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Fix wrong focus direction.

Fixed issue with incorrect focus direction when node appears conditionally.
Also, added a separate page inside the example project to demonstrate the problem and its solution.
Fixes https://github.com/bamlab/react-tv-space-navigation/issues/109#issue-2250291473

Love the idea of having it in the example. It makes it so much more visible.
I'll review properly later, as I said in the issue I think it's better not to use the index as a solution 😁
Doesn't mean we can't expose it, though!
Thanks 👍

I don't think we'll need to expose the index yet. I'm not sure we want to give that ability, as it will only pollute the API and lead to unexpected behaviours 😁
We'll see later if we really do need this, but there was a cleaner solution to your problem so I guess nothing urgent 😎

Maybe you're right.
For me, setting the index property is a clearer way to control this behavior, but it won't work for large-scale applications.
Thanks for the update.

I'm still thinking this through. If that's clearer for you, maybe we can add this feature as experimental 😁
Would you rather have the index exposed than the solution I proposed?
In my opinion, it feels a little clunky to add index to all elements, that's what I don't like with the approach. But if people prefer the index option, I can't argue with that 😁


Style Issue? Cannot read the text on a delete confirmation button.

Explain the bug
The confirmation dialog when deleting a file(s) does not show text (Styling issue?).
To reproduce

Open Files.
Delete an item.
3.Notice the text on the confirmation button cannot be read.

Expected behavior
Should be able to read text on confirmation button
Logfile
debug.log
debug_fulltrust.log
Screenshots

System Information
Edition	Windows 10 Pro for Workstations
Version	21H1
Installed on	7/18/2020
OS build	19043.1110
Experience	Windows Feature Experience Pack 120.2212.3920.0
Files Version: <IP_ADDRESS>
Additional comment
Happens after restarting app.
OS is using dark mode.

Have you set a custom theme for Files? Is Files set to English?
Looks fine for me:

No custom theme, English language.

I am using a 10 bit color monitor though, and with 10 bit color, Windows 11 shows the same issue at the resolution I am running (3840x1600). Aside from this application, I've not seen this issue in Windows 10.

Just to update: <IP_ADDRESS> still exhibits this issue.

@adrynalyne this appears to be a WinUI issue, can you report this on the WinUI repository? https://github.com/microsoft/microsoft-ui-xaml


Error when calling nsfwjs.load

Relevant code: https://github.com/dragonfire535/xiao/blob/master/structures/Tensorflow.js#L18-L22

Hey @dragonfire535
This issue seems to be related to: https://github.com/tensorflow/tfjs/pull/8425
Try using an older version of @tensorflow/tfjs-node for now. Something like 4.10.0 should work.

use a lts version of node. use nvm to install a lts version and set it as default. it should work.


Update readme about community informations

Request description
Since this is a subproject of Apache ShardingSphere, the readme and other documents should share the same information with Apache ShardingSphere.
Environment
n/a

fixed in #28


Create Kubernetes Annotation for Defining Middlewares

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
Hello,
I was sad to see https://github.com/traefik/traefik/issues/6455 closed. I presume because the proposed solution was not the desired one. However, further down in the thread, there is another proposal by a maintainer of the project.
I propose the following use case:

as a software vendor, I need to provide a generic way for someone to use our software with traefik
In order to do so, they need to ensure the X-Forwarded-Proto header is sent properly
If they are not familiar with CRDs and adding custom headers in traefik, I do not want them to have to swallow a bunch of information just to deploy our product
As a result, a set of annotations that they can add to their ingress when they use our helm chart would be perfect

The alternative is that our hello world example would have to either (1) deploy the middleware (we don't want to do b/c we would need to toggle for different ingress providers, etc.) or (2) recommend something other than traefik as our "hello world" ingress provider.
I have been looking all over the place and have not found any good recommendations after ingress.kubernetes.io/custom-request-headers annotation seems to have been removed in v2 (at least it is not in any docs, although it is alluded to in this discussion)

I would really like to have something equivalent to ingress.kubernetes.io/custom-request-headers back in v2. I would like to have a custom header set for each IngressRoute, but it does not seem scalable to create a separate middleware for each one.

Did a little testing, looks like traefik's memory usage grows ~25MiB per 1000 header middlewares that each add a single header. So that's 25KiB per 50 bytes of actual data (header name and value).


ZSH_AUTOSUGGEST_ACCEPT_WIDGETS ignored in vi-mode

When using vi-mode, no matter what widget is in ZSH_AUTOSUGGEST_ACCEPT_WIDGETS, it won't work.
For example:
$ echo $ZSH_AUTOSUGGEST_ACCEPT_WIDGETS
end-of-line vi-end-of-line vi-add-eol up-line-or-history
$ bindkey | grep end-of-line
"^E" end-of-line

Hitting Ctrl-e in insert mode when a suggestion appears won't do anything.

Hi, when I enable vi-mode I don't see ^E bound to end-of-line.
% zsh -df
% bindkey -v
% bindkey ^E
"^E" self-insert

Can you start from zsh -df (to run without sourcing any configs like ~/.zshrc) and post the exact commands you run to reproduce this issue?

Okay, so I was literally sitting on it for 1 hour trying to reproduce the issue and find a pattern.
Apparently I had an error in my zshrc, however I can reproduce it using up-line-or-history with the following steps:
$ zsh -df

$ source zsh-autosuggestions.zsh
$ bindkey -v  # It appears as though it's not really necessary
$ ZSH_AUTOSUGGEST_ACCEPT_WIDGETS=(up-line-or-history)
$ bindkey | grep up-line
"^[OA" up-line-or-history
"^[[A" up-line-or-history

# Hitting "^[[A" (Arrow up) won't work

$ bindkey -M viins '^E' up-line-or-history
# Won't work either

$ ZSH_AUTOSUGGEST_ACCEPT_WIDGETS=(end-of-line)
$ bindkey -M viins '^E' end-of-line
# Ironically, this works

BTW, ^E in my example above was a custom binding I set in order to demonstrate the problem (which doesn't really matter right now anyway...).

# Hitting "^[[A" (Arrow up) won't work

I wouldn't necessarily expect that running a widget that totally blows away the buffer contents (like up-line-or-history does) would actually be overridden to accept the suggestion. Is this an actual problem for your use case? Maybe we need to add some documentation around which widgets this would work for and which it wouldn't.

$ bindkey -M viins '^E' end-of-line
# Ironically, this works

Does this mean that your original problem ("Hitting Ctrl-e in insert mode when a suggestion appears won't do anything.") is solved?

Alright, I understand your reasoning.
It may be a good idea to just add a word about it in the docs since it was confusing (at least for me).