For Temurin jck jdk url use direct tarball url rather than forcing zip compression

For temurin jck trigger use the actual filename.tar.gz rather than *jdk*/*zip*/target.zip, as that causes nginx to use chunked compressed streaming which means curl does not know how much data is expected, and if a issue occurs, eg.a keep alive, then it doesn't know it has not received all the data. With this change the download can now retry as it gets an rc=18 from curl. eg.
20:18:46  + ./get.sh -s /home/jenkins/workspace/Test_openjdk19_hs_sanity.jck_s390x_linux/aqa-tests/.. -p s390x_linux -r customized -j 19 -i hotspot -c https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk19u/job/jdk19u-linux-s390x-temurin/38//artifact/workspace/target/OpenJDK19U-jdk_s390x_linux_hotspot_2022-12-16-19-24.tar.gz --openj9_repo https://github.com/eclipse-openj9/openj9.git --openj9_branch master --tkg_repo https://github.com/adoptium/TKG.git --tkg_branch master
20:18:46  TESTDIR: /home/jenkins/workspace/Test_openjdk19_hs_sanity.jck_s390x_linux/aqa-tests
20:18:46  get jdk binary...
20:18:46  _ENCODE_FILE_NEW=UNTAGGED curl -OLJSks --user ****:**** https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk19u/job/jdk19u-linux-s390x-temurin/38//artifact/workspace/target/OpenJDK19U-jdk_s390x_linux_hotspot_2022-12-16-19-24.tar.gz
20:19:56  curl: (18) transfer closed with 157335124 bytes remaining to read
20:19:56  error code: 18. Sleep 300 secs, then retry 1...
20:24:50  check for OpenJDK19U-jdk_s390x_linux_hotspot_2022-12-16-19-24.tar.gz. If found, the file will be removed.
20:24:50  remove OpenJDK19U-jdk_s390x_linux_hotspot_2022-12-16-19-24.tar.gz before retry...
20:24:50  _ENCODE_FILE_NEW=UNTAGGED curl -OLJSks --user ****:**** https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk19u/job/jdk19u-linux-s390x-temurin/38//artifact/workspace/target/OpenJDK19U-jdk_s390x_linux_hotspot_2022-12-16-19-24.tar.gz
20:27:44  Uncompressing file: OpenJDK19U-jdk_s390x_linux_hotspot_2022-12-16-19-24.tar.gz ...

Fixes: temurin-compliance issue 283
Signed-off-by: Andrew Leonard<EMAIL_ADDRESS>

I ran a 1000x unit test download with this and succeeded on all 1000, there were 29 curl rc=18 in those.
@sophia-guo If you could review and merge please, thanks

Thanks @sophia-guo - you were already on it!


Default valid date to last change request date

Description of change
Change the default application "valid" date from the documents_validated_at to the last updated change request if any change requests exist on the application.
Also take care of deprecation warning about ".html.erb" partials
Story Link
https://trello.com/c/6vC0codP/317-default-validation-date-based-on-last-change-request

The tests are good and useful, thanks :-)
However, in the implementation, this sets the documents_validated_at even when the application has not yet been validated. I can imagine this being confusing in the future. What we really want to be doing is only overriding the default value in the HTML form, and leaving the DB alone.
I notice that the form builder doesn't support overriding the default value, which is annoying, but one way around this would be to set on the object on the validate_documents_form action. In fact, we're already doing that in planning_applications_controller line 85, so that's what needs editing.


Got a long time to stop emacs daemon via systemd

I created the emacs.service like this (refer to Archlinux Wiki)

~/.config/systemd/user/emacs.service
[Unit]
Description=Emacs: the extensible, self-documenting text editor
[Service]
Type=forking
ExecStart=/usr/bin/emacs --daemon
ExecStop=/usr/bin/emacsclient --eval "(kill-emacs)"
Restart=always
[Install]
WantedBy=default.target

Thus I got a long wait when I shutdown my machine as well as when I use command below
systemctl stop emacs.service
systemctl restart emacs.service
How was it happening?
I guessed it was the warning:"Saving desktop? ( y or n )", but it seems not to be.
I've tried this command: emacsclient --eval "(kill-emacs)". It works well, needing no waiting.

Sorry, I don't know how to help with this. Perhaps Emacs was prompting you to save unsaved buffers.


operators [R] hazelcast-platform-operator (5.0.0 5.1.0 5.2.0)

This reverts commit b75afdda
Signed-off-by: aborschuk<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[ ] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/merge possible

/retest

@J0zi @mvalarh Hi guys, can you tell me what is wrong with the revert commit? How to re-run it?

/retest

/retest-required

/merge possible


CI pip dependency cache may be stale

CI build error
What's wrong
It looks like all recent Travis CI builds are failing in part due to the mypy-extensions package (0.4.2) failing a hash check. I noticed this in the build results for my recent patch request:
[pipenv.exceptions.InstallError]: ['Collecting mypy-extensions==0.4.2 (from -r /tmp/pipenv-b9jw2c5s-requirements/pipenv-bt98vvy2-requirement.txt (line 1))', '  Using cached https://files.pythonhosted.org/packages/55/aa/2df16f9c6215b10c472fd7564b5f2c5644c13ab9c26d83b1c9373c11d97d/mypy_extensions-0.4.2-py2.py3-none-any.whl']
[pipenv.exceptions.InstallError]: ['THESE PACKAGES DO NOT MATCH THE HASHES FROM THE REQUIREMENTS FILE. If you have updated the package versions, please update the hashes. Otherwise, examine the package contents carefully; someone may have tampered with them.', '    mypy-extensions==0.4.2 from https://files.pythonhosted.org/packages/55/aa/2df16f9c6215b10c472fd7564b5f2c5644c13ab9c26d83b1c9373c11d97d/mypy_extensions-0.4.2-py2.py3-none-any.whl#sha256=c77e34350c01523cd63b6dc30b36515107b2c3cbacc8392d51d33ea18d59f8f1 (from -r /tmp/pipenv-b9jw2c5s-requirements/pipenv-bt98vvy2-requirement.txt (line 1)):', '        Expected sha256 a161e3b917053de87dbe469987e173e49fb454eca10ef28b48b384538cc11458', '             Got        c77e34350c01523cd63b6dc30b36515107b2c3cbacc8392d51d33ea18d59f8f1']

(output trimmed for brevity and to focus on the point)
I have not investigated if this is due to mimesis CI builds using an incorrect strategy when storing relevant cache keys (i.e. not noticing an update to this package) or if it is a stale Pipfile.lock value, but I suspect it is one of those and/or that perhaps mypy-extensions republished a package without bumping their package version. It may be worth bumping the requirement to mypy-extensions~=0.4.3 if it was a republished package snafu.
As a first step, the cache key should be bumped to trigger caching the latest package contents from the Python package index. If nobody else has the bandwidth to look into that, I can make a separate pull request to address it.
How it should be
The CI builds should not fail due to a stale pip dependency cache.
System information
This occurs during the project's Travis CI builds.

@jwilges First of all, thank you! It would be really great if you can create PR. I'm too busy unfortunately and I can't work on it right now, but I can merge PR — it not takes much time.

Thanks, @lk-geimfari and @ceccoemi.
I'll take a look at the remaining build issue for Windows 10 as it's also likely a fairly quick fix.


Give visual feedback if a specified type probably does not exist

Right now you can type in
builtin.strabcef
and the editor will not complain, even though the correct type here would probably be
builtin.str

This is checked on update time.


Failed to initialize zoom sdk

Hello, I found error
Failed to initialize Zoom SDK
and after I initialize zoom, I try to print and i get a result just like picture below

same error I got when I passed my own secret access key and api key..

same error I got when I passed my own secret access key and api key..

Let me check if i can reproduce the same issue

@tffajari @cool0305 Which api key and secret key you are using JWT or SDK

Sorry my bad, I was using the JWT app key which is not recommended....thanks for the solution @iamgouravturka

@evilrat we should think about it Because I also face this issue we need to change the text API Key and Secret to SDK Key and Secret.

@evilrat I created a PR regarding this please have a look.

Already merged

@tffajari how did you solve this error As i got the same error
Failed to initialize Zoom SDK****


[wip] Enum support

Like pin-project, by passing an argument with the same name as the method to the attribute, you can name the projection type returned from the method. This allows you to use pattern matching on the projected types.
use pin_project_lite::pin_project;
use std::pin::Pin;

pin_project! {
    project = EnumProj;
    project_ref = EnumProjRef;
    enum Enum<T, U> {
        Struct {
            #[pin]
            pinned: T,
            unpinned: U,
        },
        Unit,
    }
}

impl<T, U> Enum<T, U> {
    fn method(self: Pin<&mut Self>) {
        match self.project() {
            EnumProj::Struct { pinned, unpinned } => {
                let _: Pin<&mut T> = pinned;
                let _: &mut U = unpinned;
            }
            EnumProj::Unit => {}
        }
    }
}

The current syntax (project = EnumProj;) is tentative. We need to think of a better syntax for this.
Also, this does not provide support to tuple variants for the same reason that it does not support tuple structs.

I'm wondering what syntax is preferable, so I would appreciate any feedback.

What if the attribute syntax was used, similar to the one in pin-project?
So perhaps something like #[project = EnumProj] above the enum definition.

Ah, that looks like a good idea. There may be a limitation like #3, but I'll try it.

It's okay if there are a few limitations with known workarounds. I think it's amazing that you managed to pull this off at all without procedural macros :)

I think the implementation is basically complete, but I'll look into limitations and minor bugs before the merge.

Filed #36 to track diagnostics issues. I'm going to merge this.
bors r+

Published in 0.2.0.


Upload Retengo

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[ ] Filename in the lowercase format
[ ] Image format: png
[ ] Image size: 256 by 256 px

@ElSierra I lowercased it for you and added here https://github.com/TrustWallet/tokens/commit/f7bf88e7296236006a55a8221b33d64e7b2a9358
Also, you can use these graphics to say that your ERC20 token fully compatible with Trust Wallet (https://trustwalletapp.com/logo-assets). Tokens will show up automatically once the user receives them.
Feel free to share this on social media and tag Trust Wallet:

Twitter: @trustwalletapp

Facebook: @trustwalletapp


added index.md in release

/retitle [hi] Add releases/_index.md

/retitle [hi] Add content/hi/releases/_index.md

Resolves #31273

@jayesh-srivastava thanks I missed that

/assign
/label tide/merge-method-squash

/lgtm
/approve

Thank you @AvineshTripathi :)


Signaturehelp on safari

monaco-editor npm version: 0.7.3
Browser: chrome/ safari
OS: OS X
We experience some odd styling differences for signature help between safari and chrome.
Safari:

Chrome:

try here : http://<IP_ADDRESS>/calcmonaco/

The following seems to fix your issue:
.monaco-editor .parameter-hints-widget {
flex-direction: row !important;
}

@svenefftinge Was this caused by some CSS collision on your page or is this something that fails with the editor out of the box?

This fails out of the box. Try this in the Monaco playground:
function abc(){};

// function abc(x){};

abc();

Uncomment the second abc overload and everything works fine.

Yes, seems fixed (either via Monaco or via Safari 12).


Missing Flat decoration (spirv direct)

I'm trying to pass the instanceIndex from Vertex to Pixel shader
// Vertex
output.stage.id = InstanceIndex();

// Pixel
uint particleID = stage.id;

It compiles, it works, but Vulkan complains
Fragment OpEntryPoint operand 35 with Input interfaces with integer or float type must have a Flat decoration for Entry Point id 165.
  %gl_InstanceID = OpVariable %_ptr_Input_uint Input

The id was decorated with nointerpolation
struct PSin
{
  nointerpolation uint id : DRAW_ID;
  float2 uv : TEXCOORD;
};

In attachment the shader and the SPIRV result.
raster.zip
Note: in the Spirv output, there are 52 repeated lines of:
               OpExecutionMode %fragmentMain DepthReplacing

#3558 should fix this. The problem is not with missing flat decoration, it is that gl_InstanceID / InstanceId is no longer a supported builtin on Vulkan. It should be gl_InstanceIndex / InstanceIndex.

InstanceIndex is for ray tracing stages only.
To get vertex instance id, you need to use SV_InstanceId semantic:
VertexOut vertexMain(int instanceId : SV_InstanceId)
{
}

This make no difference with the current issue.
The problem might be that the compilation uses: -emit-spirv-directly -fvk-use-entrypoint-name and both vertex and fragment are in the same file.
The validation error appears when the value of instanceID is transmitted to the fragment,
Here is how this looks like, the particleID need to be made available to the fragment shader. But if particleID == instanceId or InstanceIndex(), there will be an error.
// Per-vertex attributes to be assembled from bound vertex buffers.
struct VSin
{
  [[vk::location(0)]] float3 position : POSITION;
  [[vk::location(1)]] float2 uv : TEXCOORD;
};

// Output of the vertex shader, and input to the fragment shader.
struct PSin
{
  nointerpolation uint id : DRAW_ID;
  float2 uv : TEXCOORD;
};

// Output of the vertex shader
struct VSout
{
  PSin stage;
  float4 sv_position : SV_Position;
};

// Output of the fragment shader
struct PSout
{
  float4 color : SV_Target;
};

VSout vertexMain(VSin input, int instanceId: SV_InstanceId)
{
  uint particleID = instanceId;
  float2 particle = input.position.xy * frameInfo.radius + (particles[particleID].position * frameInfo.scale * 2);

  VSout output;
  output.sv_position = mul(frameInfo.proj, float4(particle, 0.0, 1.0));
  output.stage.uv = input.uv;
  output.stage.id = particleID;

  return output;
}

// Fragment Shader
[shader("pixel")]
PSout fragmentMain(PSin stage, float4 fragCoord : SV_Position)
{
  uint particleID = stage.id;
...
}

In GLSL, to transfer the information, we use the layout, like this:
raster.vert
-----------
layout(location = 0) out vec2 outUv;
layout(location = 1) out flat int outParticleID;

main()
{
...
outParticleID = gl_InstanceIndex;
...
}

raster.frag
-----------
layout(location = 0) in vec2 inUv;
layout(location = 1) in flat int inParticleID;
layout(location = 0) out vec4 outColor;

// inParticleID can be used anywhere

So, the Slang code still current emit the error
[VUID-StandaloneSpirv-Flat-04744] Fragment OpEntryPoint operand 35 with Input interfaces with integer or float type must have a Flat decoration for Entry Point id 165.
  %gl_InstanceIndex = OpVariable %_ptr_Input_int Input
```

Tested with SLANGC version:<PHONE_NUMBER>

The error is confusing since instanceId is never defined as a fragment shader parameter, but somehow the validation layer complains that it is a fragment shader input.

Can you attach the full apirv here?

I reverted my changes and back using uint particleID = InstanceIndex(); and the error is gone with SLANGC version:<PHONE_NUMBER>
The error is present in 2024.0.3

It is incorrect to use InstanceIndex by spirv and Vulkan spec. InstanceIndex map to the built in gl_InstanceID or the InstanceId spirv built in, which is removed from the glsl spec / Vulkan. If you update to latest Vulkan sdk you should see the validation error.

On my test machine, I can't reproduce the validation error you mentioned. Probably because we use different sdk version?

The sample can be found under https://github.com/nvpro-samples/vk_mini_samples/tree/main/samples/realtime_analysis
The shader is raster.slang.
The Vulkan SDK is 1.3.275
Here is the SPIRV output
raster_slang.spv.zip

Will take a look. Thanks!


Add randomized exponential backoff

So my bots were blocked. After emailing them...

 In general, if there is an error response (5XX status code, or 429) you should do an exponential backoff in order to reduce load on the site. I've removed the block from your IP.
Ricky

Hopefully I did this the right way...

@not-an-aardvark I set max_retry_attempts to 5 (I wrote the bot when reddit was throwing errors left and right), and my bot was making requests (finding new posts and comments) in a continuous loop, regardless of whether it got an error.
I'll (try to) add a test right now.

Thanks, this mostly looks good. The tests weren't working, but I pushed a commit to your branch to fix them.
One last question -- why is the backoff randomized? Is there a disadvantage of making it deterministic?

The randomization is to prevent a DDoS attack on Reddit when a large number of clients fail at the same time (although it could be made better).


Training with Multiple GPUs

Hi @ShunyuanZheng , thanks for sharing the code. Amazing work! Do you plan to release the code of training on multiple gpus because training on a single gpu will take a lot of time. I tried to add the data parallelism but bugs appear.
I would appreciate it if you could provide some comments or suggestions for faster training.

Hi, @Lewis-lv0,  currently I have no plan to release the multi-GPU training code. However, the overall training time on one 3090 GPU takes less than two days, which is acceptable I think : )

Thanks for your reply. I will close the issue.


Module Federation in production

I now encounter a problem: in the case of two applications referencing each other, if the publicPath is not set to the corresponding absolute path, the specific module under the corresponding sub-application will not be obtained, but if it is set to the absolute path, it will be in the address The server (cdn) address is exposed on the bar. If set to auto, the refresh will initiate a request from the current route to obtain the submodule after switching the route, which also fails to obtain. How to solve it please?
I didn't find an answer for the response in the documentation.Dynamic publicPath failed.

ok, I resolved it by dyn publicPath!

Glad you fixed it.


Kd/add bountysource

Add bountysource to sponsors and readme page.
Unfortunately, the "activity" badge for bountysource isn't a great metric (would be much better to have the $ amount of outstanding bounties), but unfortunately I couldn't find a way to get that. I believe this is better than nothing.

Codecov Report

Merging #14323 (ae27f34) into master (8db0372) will decrease coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #14323      +/-   ##
==========================================
- Coverage   41.97%   41.85%   -0.12%
==========================================
  Files         735      744       +9
  Lines       78933    79638     +705
==========================================
+ Hits        33130    33335     +205
- Misses      40347    40816     +469
- Partials     5456     5487      +31

Impacted Files
Coverage Δ

routers/init.go
45.79% <0.00%> (-4.61%)
:arrow_down:

models/repo_permission.go
74.31% <0.00%> (-2.76%)
:arrow_down:

modules/auth/auth.go
61.53% <0.00%> (-2.51%)
:arrow_down:

modules/git/repo_commit_nogogit.go
63.33% <0.00%> (-1.67%)
:arrow_down:

models/repo_list.go
77.87% <0.00%> (-0.89%)
:arrow_down:

modules/auth/sso/sso.go
32.14% <0.00%> (-0.59%)
:arrow_down:

models/user.go
53.85% <0.00%> (-0.56%)
:arrow_down:

models/issue_comment.go
52.28% <0.00%> (-0.44%)
:arrow_down:

routers/api/v1/admin/user.go
35.54% <0.00%> (-0.44%)
:arrow_down:

routers/user/setting/account.go
25.12% <0.00%> (-0.39%)
:arrow_down:

... and 47 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #14323 (ae27f34) into master (8db0372) will decrease coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #14323      +/-   ##
==========================================
- Coverage   41.97%   41.85%   -0.12%
==========================================
  Files         735      744       +9
  Lines       78933    79638     +705
==========================================
+ Hits        33130    33335     +205
- Misses      40347    40816     +469
- Partials     5456     5487      +31

Impacted Files
Coverage Δ

routers/init.go
45.79% <0.00%> (-4.61%)
:arrow_down:

models/repo_permission.go
74.31% <0.00%> (-2.76%)
:arrow_down:

modules/auth/auth.go
61.53% <0.00%> (-2.51%)
:arrow_down:

modules/git/repo_commit_nogogit.go
63.33% <0.00%> (-1.67%)
:arrow_down:

models/repo_list.go
77.87% <0.00%> (-0.89%)
:arrow_down:

modules/auth/sso/sso.go
32.14% <0.00%> (-0.59%)
:arrow_down:

models/user.go
53.85% <0.00%> (-0.56%)
:arrow_down:

models/issue_comment.go
52.28% <0.00%> (-0.44%)
:arrow_down:

routers/api/v1/admin/user.go
35.54% <0.00%> (-0.44%)
:arrow_down:

routers/user/setting/account.go
25.12% <0.00%> (-0.39%)
:arrow_down:

... and 47 more

Continue to review full report at Codecov.

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


Reverse Geolocation for changeset not good enough, translation missing

After uploading a changeset in Leibnitz, Steiermark (Styria, part of Austria) I see:

The location is Leibnitz (County of Styria and main city there) in the Austrian state of Styria:
https://www.openstreetmap.org/changeset/58159266#map=14/46.7436/15.5616

Koglbauer is the name of a farm and a short road near Leibnitz:
https://www.openstreetmap.org/search?query=koglbauer#map=15/46.7605/15.5298

So Styria is not translated to Steiermark which would be nice and Koglbauer is not apropriate.

The current code that puts together the location string is here:
https://github.com/openstreetmap/iD/blob/85ae00dd05d998a8fbf14aba1fb78491fec498b4/modules/modes/save.js#L500-L511
What the code does is ask Nominatim to reverse geocode the center of the map where the user is looking when they did the save (their edits might not necessarily be there, but this is the place that they are looking at).
We get a result from Nominatim and iD tries to piece it together into something like City, Region..
There are some things we can do to improve this.

Can you remember what you were looking at when you clicked save on that edit?  If you can give me the lat/long coordinates, I can do the Nominatim lookup again and see what data is returned and maybe pick a better subset of the results, or maybe issue the query better?    (BTW, the Location panel does the same lookup - so if you go to the same place and open that up, it should also say something about Koglbauer, Styria and a bunch more).

We could query based on the bounding box where the user's changes were, not necessarily the map where the user is looking at when they click Save.  I'm not sure whether this would be more confusing to a new user though.

I tried again and the map is contains nearly only "Leibnitz Bahnhof" (railstation):

See https://www.openstreetmap.org/edit#map=20/46.78047/15.54668

This is shown when i do "Objektabfrage" at the red cross:

Is this "Location Panel lookup"?

Is this "Location Panel lookup"?

"Location Panel" is this feature in iD  (for me it's Cmd+Shift+L to show):

Koglbauer is also an isolated_dwelling:

It would be better not to show isolated_dwellings unless they are very near to the map center.

It would be better not to show isolated_dwellings unless they are very near to the map center.

We get a result from Nominatim and iD tries to piece it together into something...

Doesn't that mean that this is ultimately a problem for Nominatim to solve? All iD is doing is spitting out what Nominatim provides it. If reverse geolocation is finding an isolated dwelling 2.5 miles from where you're editing, that's an issue with them.
It seems like a Really Bad Idea to apply exceptions to what iD says a location "is" as Nominatim gives it, when the appropriate solution is to tell Nominatim to make their RG better. (In fact, an argument can be made that telling iD to make exceptions against Nominatim's RG would actually complicate bug reporting to that project.) Look at it this way: this is bringing visibility to Nominatim's reverse geolocation?

To explain my position here: if you want to cut off what "near" means at a certain admin level, that's one thing. 'We will never show an admin_level lower than 8' in the output, because we'll just assume that 8 is close enough to be friendly to the user.'
It would be quite another to start throwing around 'Well, we'll use an isolated_dwelling with a lower admin level of 15, when you're really editing "close to it", EXCEPT if you're not.'
That second kind of "fix" would really make identifying Nominatim problems that come through that query difficult.

To explain my position here: if you want to cut off what "near" means at a certain admin level, that's one thing. 'We will never show an admin_level lower than 8' in the output, because we'll just assume that 8 is close enough to be friendly to the user.'

Yes, what you said here is what I would do to fix this issue.  I think we can construct our query to Nominatim in way that returns a more regional result.

Ok, I swapped out suburb for town - this will probably display a more meaningful result on the post-save screen.
For info, here is the query and result.  We are already requesting a result at zoom=13, which seems fine.
query:
https://nominatim.openstreetmap.org/reverse?addressdetails=1&format=json&lat=46.7804013&lon=15.5466538&zoom=13
result:
{"place_id":"42083160","licence":"Data © OpenStreetMap contributors, ODbL 1.0. https:\/\/osm.org\/copyright","osm_type":"node","osm_id":"3096751413","lat":"46.7706284","lon":"15.5160898","display_name":"Koglbauer, Leibnitz, Styria, 8430, Austria","address":{"suburb":"Koglbauer","town":"Leibnitz","county":"Leibnitz","state":"Styria","postcode":"8430","country":"Austria","country_code":"at"},"boundingbox":["46.7705284","46.7707284","15.5159898","15.5161898"]}


Integration tests with redux-loop?

I'm trying to run mock redux store from my unit tests according to:
https://github.com/reactjs/redux/blob/master/docs/recipes/WritingTests.md
It works well for redux-thunk as well as redux-saga, because they are 'middleware' that could be set inside 'mock' redux store, but i can't figure out how to do it with redux-loop with its 'install()'

It doesn't look like redux-mock-store is compatible with enhancers other than applyMiddleware, which it calls internally 😦
If you definitely need to use redux-mock-store I would suggest getting in touch with the author via a github issue on that repository to figure out how you would go about applying an enhancer like install().


Use standard electropherogram DNA colours

These are the base colours I'm most familiar with seeing:
A-green
C-blue
G-gold
T-red
https://en.wikipedia.org/wiki/Electropherogram [where gold is replaced with black for visibility purposes]

In version 0.4, I have now added a second DNA colormap with the colors you suggested. You should now be able to upgrade to this version using pip install dnacol --upgrade.
Once you have upgraded, you can set the colormap to use for DNA sequences using a config file in your home directory called .dnacol. To use the colors you suggested, you would add this line to the config file:
dna_colormap: gbyr


Elaine/keyimport

Addresses issues in #131:
show success/error, show address info, load balance, add to acct list
Currently, AccountShow is displayed as a card on the screen, below the "success" message. In the future, if we have a notifications UI, the "success" message can go in the notifications center and then redirect straight to AccountShow

Updates also address #142


Using sqlType won't generate correct migration file by PgsqlPlatform

Hi,
Using <column sqlType="..."> won't generate correct migration file.
For example, consider this simple schema:

    <column name="email" required="true" type="VARCHAR" size="255" />

After installing it, the expected column is created.
Now adding sqlType attribute to email column, won't generate correct migration file.

    <column name="email" required="true" type="VARCHAR" size="255" sqlType="citext" />

Current Result:
return array (
  'test' => '',
);

Expected Result:
return array (
  'test' => '
BEGIN;

ALTER TABLE "foo"

  ALTER COLUMN "email" TYPE citext USING NULL;

COMMIT;
',
);

I was able to trace the code which is responsible for generating the diff, and it's in PgsqlPlatform class on line 525.
Adding sqlType to the list of conditions, and running diff command again generates expected results.
if (isset($changedProperties['size']) || isset($changedProperties['type']) || isset($changedProperties['sqlType']) || isset($changedProperties['scale'])) {

I agree, it would be great if propel internally would recognize that change  in schema, and create the diff accordingly. :+1:

Can you create a pull request for that change?

PR is now open.


ListView inside Pull to Refresh scrollview is not scrolling for scroll down event

Description
Added a list view as child view to the Parent ScrollView which is having RefreshControl to support for pull to refresh.
I expected when user drag down the child ListView, the the listView should scrolldown.
But Actually Pull to Refresh is happing for drag down event only for Android. On iOS it is working fine.
Reproduction Steps and Sample Code
<View style={styles.containerStyle}>
 <ScrollView contentContainerStyle={styles.contentContainerStyle} refreshControl={
      <RefreshControl
        refreshing={this.state.refreshing}
        onRefresh={this._onRefresh.bind(this)}
      />
    }>
  <View> Some header </View>
  <MyComponent> </MyComponent>
  <ListView
    dataSource={dataSource}
    initialListSize={10}
    renderRow={this.renderRow}
    renderSeparator={this.renderSeparator}/>
 </ScrollView>
</View>

List all the steps required to reproduce the issue you're reporting. These steps should be clear and concise.

Add a ScrollView with RefreshControl  as a parent
Add ListView as a child
ListView should contain enough data so that It can be scrolled in Android.
Now Drag down in List view.

Expected: List view should scroll for this drag-down event
Actual :  Pull to Refresh is happening for drag down/Scroll down event (only for Android). drag up/Scroll up is actually scrolling the list in Android.
Additional Information

React Native version:
react-native-cli: 2.0.1
react-native: 0.40.0

Platform: Android only.

Development Operating System: Mac

Does this still reproduce using FlatList? ListView is deprecated.

@hramos yes it is reproducible for FlatList as well. On Android somehow, scrollview and listview/flatlist cannot decide which one should respond to touches and eventually both responds. or the respond switch from one to another is very easy.
@jittu-tutu it has been long time but, I had the similar issue, try using ViewPagerAndroid instead. That worked for me. However, I had to move RefreshControl to listviews.

has anyone fixed this issue?.I have the same problem.Using the scrollView contains the listView ,the listView has a refreshControl,the "onRefresh" not invoke when user drag down.

I have the same problem with FlatList inside ScrollView.
react-native-cli: 2.0.1
react-native: 0.49.5
platform: Android only.

Any solution on this issue ?

@Skarpovich, i have the same problem but only iOS platform.
(using 'create-react-native-app')
"expo": "^23.0.4"
"react": "16.0.0"
"react-native": "0.50.3"
"react-native-cli": "2.0.1"
Also my scenario is similar to #14756,
I'm trying make a HOC with "auto scroll when needed" for all my screen/components, but one of them has a FlatList with RefreshControl not working only iOS platform.
I don't think it is a bad pattern / design like @dmr07 said.
Any sugeestions?

Adding to what @Komalov said, I fixed my issue, by simply moving the entire refresh control prop
refreshControl={
  <RefreshControl
    refreshing={this.state.refreshing}
    onRefresh={this._onRefresh.bind(this)}
  />
}

off of my SectionList & FlatList, and onto the ScrollView.


Updated CHANGELOG.md and package.json`

Codecov Report

Merging #121 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #121   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          32     32
  Lines         677    677
  Branches       83     83
=====================================
  Hits          677    677

Flag
Coverage Δ

#builder
100% <ø> (ø)
:arrow_up:

#runtime
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fixed the position of playground title

issue #3 :
Fixed the position of playground title 🚀 .
What does this do?
It helps users easily navigate from different components back to the home page.
@pranshuchittora plz let  me know about this PR or any changes .

Good Job!
I think navigating to the home page will be easier and faster now, otherwise we were to scroll up the side bar every time.

thanks! 👍


Add PlatformGuard.implementation?

This guards checks for specific Ruby implementations, we probably shouldn't try to be compatible with these corner cases.
Currently, it's only used for specific load paths in MRI.

This converts two errors in the command-line specs into failures.


IE9 rendering issue when hovering over row with hScroll enabled

This issue is isolated to IE9 and related to the IE9 bug outlined in this blog post. It adds additional white space to the bottom of the inner table on each row hover.
Here is an existing jsfiddle  that demonstrates the issue (albeit not using DataTables). Again, the issue only exists in IE9.
I was able to workaround this issue by adding the following CSS to my own stylesheet:
.dataTables_scrollBody
{
min-height:0%;
}

I haven't been able to reproduce that issue in IE9 with my scrolling examples:

http://datatables.net/examples/basic_init/scroll_y.html
http://datatables.net/examples/basic_init/scroll_x.html

Does IE9 break for you in those examples?

I just realized that the blog post and jsfiddle mention what triggers the bug, but I left it out in my initial description. The bug is triggered by a hover style applied to an element inside the table.
This selector should do the trick:
table.table tbody tr:hover td {
background-color:#ecfaff;
}

My examples use a td class on hover (css).
Can you link me to an example page that uses DataTables that shows this problem please?

I had to do some tweaking to get this issue to reproduce in a jsfiddle.
I had to manually add the class that DataTables was generating for my table:
class="display table table-bordered table-striped dataTable no-footer"

I hate to say it, but that example appears to work okay for me in IE9. I am using IE11 and changing the browser mode using the dev tools. Does it have to be pure IE9 perhaps? What are you using?

I am using IE9 (v. 9.0.8112.16421) on Win7.  I work for an organization that uses a base Win7 image which includes IE9 and has Group Policy applied. I was able to recreate this issue on multiple machines in my office. However, I had a coworker (developer) who was not able to reproduce this issue on his machine (IE9), but was able to login to other machines (IE9) and reproduce it on those machines.
Seems like it might be a (machine level) IE9 confguration or GPO setting???
Do you feel the min-height workaround I am using is safe to use with DataTables for the time being?

Yes, I think the min-height hack is perfectly safe. I can't see any issues with it, although I am reluctant to include potentially redundant CSS (and a hack) in the core DataTables stylesheet.
It would be interesting to know what is causing this but if different IE9 machines are showing different behaviour... These are the times when being a web-dev makes one slightly crazy(er) :-).
I'm going to close this just now, but if I have any more reports of this problem I'll look into adding your patch into the core stylesheet.

Agreed. Why add the unnecessary.
Thanks for your time with this, and thanks for DataTables!


[KYUUBI #5900][FOLLOWUP] Get the engine service node to delete instead of delete engine space

:mag: Description
Issue References 🔗

Followup for patch #5901
We shall find the engine node first and then delete it instead of deleting the engine space.
This pull request fixes #
Describe Your Solution 🔧
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Types of changes :bookmark:

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests

Checklists
📝 Author Self Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] This patch was not authored or co-authored using Generative Tooling

📝 Committer Pre-Merge Checklist

[ ] Pull request title is okay.
[ ] No license issues.
[ ] Milestone correctly set?
[ ] Test coverage is ok
[ ] Assignees are selected.
[ ] Minimum number of approvals
[ ] No changes are requested

Be nice. Be informative.

cc @Karsonnel

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (fd397d6) 61.53% compared to head (f729629) 61.59%.
Report is 6 commits behind head on master.

Files
Patch %
Lines

...ain/scala/org/apache/kyuubi/engine/EngineRef.scala
57.14%
1 Missing and 2 partials :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5910      +/-   ##
============================================
+ Coverage     61.53%   61.59%   +0.06%
  Complexity       23       23
============================================
  Files           616      616
  Lines         36376    36386      +10
  Branches       4976     4978       +2
============================================
+ Hits          22384    22412      +28
+ Misses        11579    11557      -22
- Partials       2413     2417       +4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

thanks, merged to 1.8.1


Simplify rendering API

Why do we suggest calling Gon::Base.render_data({})
since we can call Gon::Base.render_data?

Thank you!

Maybe we can point out in README that render_data can be called with some args


Add vCHORD to oncoanalyser for HRD detection in targeted mode

In the targeted mode, CHORD is disabled and vCHORD is used instead for HRD detection for a panel of samples. However, it's not implemented in nf-core/oncoanalyser yet. It will be great to have it here too. Thanks.

Agreed, this will be implemented in oncoanalyser in due course! I'll leave the issue open until that piece of work is complete


possible bug w/ upsampling projects added w/ Add Data button

Added project only worked at resolution 0

Fixed in https://github.com/neurodata/ndviz/commit/4f460164c81a2a95bb6e9ca3626f099cb9a3b18b


Added barcodeA support

Disabled by default for simple profile but enabled for default profile. Closes #26.

Thanks!


"Find" hotkey in settings should focus search text box

When in settings view "Find" hotkey should focus the search/filter text box.

Completed some time ago.

Resolved in builds >= 3.0.3414


Setup QoS / QoE AWS test infrastructure

As a Pillarbox developer I would like to have a QoS / QoE test infrastructure to which I can send events collected by the platform player I am working on.
Acceptance criteria

Event dispatcher and data storage / visualization tools are available.
Clients send data to our test platform, and this data is visualized in the simplest possible way.
Monthly costs stay reasonable (< Snitch).
We have the access level needed to administer our platform.

Tasks

[x] Discuss QoS / QoE AWS infrastructure needs and cost structure.
[x] Estimate potential costs.
[x] Deploy existing dispatcher / visualization tools.

I asked the sysadmins how to deploy the service. It's in progress.

Commented the following task https://github.com/SRGSSR/pillarbox-event-dispatcher/issues/1#issuecomment-2338688093

We will monitor costs in the upcoming weeks / months.


Add journey jump links

this, https://github.cfpb.gov/OAH/wp-oah/pull/10, will have to be merged too.
ES content will need to be reindexed in order to see it working

Who should review this— @amymok ?

@stephanieosan she can

@fna How do I test this?

I used UnityBox to run Wordpress and loaded the latest data dump there, login into WP admin and edit existing step, add Collapse link field, then re-indexed ES from it WORDPRESS=http://localhost:8080 sheer index --reindex after run OaH in sheer and go to process/XXXX (depends on where the step you changed is located) and process/XXXX#created-collapse-link should take you to that section and expand it

Looks good!


Adding issues to consulo-vim is not possible

https://github.com/consulo/consulo-vim/issues

Me returned Issues - but for now me not able to fix issues om this plugin


how to use this with ionic 2

Hi @BenBBear ,
I really appreciate for this, it's a very nice library.
I am working on ionic 2, wanted to implement this in ionic 2.

I did same as you have mentioned it in installation section.

Also import that in www/lib

gave appropriate path in index.html

but It did not work for me, because issue is-
ionic use ionic native and typescript.
How to get this working in ionic 2?

Ionic 2 Support would be really nice!!:)


Keyd's virtual pointer emits wrong button codes

This is probably not a keyd issue, but nevertheless I'm recording it here: After having updated my laptop (running Archlinux), the buttons emitted by keyd's virtual pointer are suddenly 18, 17 and 16 (leftmouse, middlemouse, rightmouse) instead of 1, 2 and 3. I had to fix this via xinput set-button-map. Is there any explanation for this strange behaviour?

I suppose it is possible that the X/Wayland events corresponding to the generated evdev events have changed, but that seems unlikely. I will investigate when I have some time. If you still have access to another system where it works, you might try comparing the generated keycodes displayed by evtest.

This fixes the issue for me:
diff --git a/src/vkbd/uinput.c b/src/vkbd/uinput.c
index b98e585..d1a9fca 100644
--- a/src/vkbd/uinput.c
+++ b/src/vkbd/uinput.c
@@ -103,7 +103,6 @@ static int create_virtual_keyboard(const char *name)

 static int create_virtual_pointer(const char *name)
 {
-       uint16_t code;
        struct uinput_user_dev udev = {0};

        int fd = open("/dev/uinput", O_WRONLY | O_NONBLOCK | O_CLOEXEC);
@@ -126,11 +125,11 @@ static int create_virtual_pointer(const char *name)
        ioctl(fd, UI_SET_RELBIT, REL_Y);
        ioctl(fd, UI_SET_RELBIT, REL_Z);

-       for (code = BTN_LEFT; code <= BTN_TASK; code++)
-               ioctl(fd, UI_SET_KEYBIT, code);
-
-       for (code = BTN_0; code <= BTN_9; code++)
-               ioctl(fd, UI_SET_KEYBIT, code);
+       ioctl(fd, UI_SET_KEYBIT, BTN_LEFT);
+       ioctl(fd, UI_SET_KEYBIT, BTN_RIGHT);
+       ioctl(fd, UI_SET_KEYBIT, BTN_MIDDLE);
+       ioctl(fd, UI_SET_KEYBIT, BTN_SIDE);
+       ioctl(fd, UI_SET_KEYBIT, BTN_EXTRA);

        udev.id.bustype = BUS_USB;
        udev.id.vendor = 0x0FAC;

@rrveex Odd, that shouldn't make a difference. What distro/kernel are you using? Can you post your /usr/include/linux/input-event-codes.h?
@herrsimon  What is the output of evtest $(evtest 2>&1|awk -F: '/keyd virtual pointer/{print $1;exit}')  when trying to use these keys?

What distro/kernel are you using?

6.2.2-artix1-1

Can you post your /usr/include/linux/input-event-codes.h?

[...]
#define KEY_WWAN		246	/* Wireless WAN (LTE, UMTS, GSM, etc.) */
#define KEY_WIMAX		KEY_WWAN
#define KEY_RFKILL		247	/* Key that controls all radios */

#define KEY_MICMUTE		248	/* Mute / unmute the microphone */

/* Code 255 is reserved for special needs of AT keyboard driver */

#define BTN_MISC		0x100
#define BTN_0			0x100
#define BTN_1			0x101
#define BTN_2			0x102
#define BTN_3			0x103
#define BTN_4			0x104
#define BTN_5			0x105
#define BTN_6			0x106
#define BTN_7			0x107
#define BTN_8			0x108
#define BTN_9			0x109

#define BTN_MOUSE		0x110
#define BTN_LEFT		0x110
#define BTN_RIGHT		0x111
#define BTN_MIDDLE		0x112
#define BTN_SIDE		0x113
#define BTN_EXTRA		0x114
#define BTN_FORWARD		0x115
#define BTN_BACK		0x116
#define BTN_TASK		0x117

#define BTN_JOYSTICK		0x120
#define BTN_TRIGGER		0x120
#define BTN_THUMB		0x121
#define BTN_THUMB2		0x122
[...]

I just compared with https://github.com/wyatt8740/mouseemu (which still worked after distro upgrade) and copy/pasted.
Looks like the second loop (BTN_0..BTN_9) does "something" to the udev-mouse.

@rrveex

Looks like the second loop (BTN_0..BTN_9) does "something" to the uinput-mouse.

By which I assume you mean:

  for (code = BTN_0; code <= BTN_9; code++)

          ioctl(fd, UI_SET_KEYBIT, code);

alone solves the problem. If so, my suspicion is that some mouse detection mapping/code has changed in X/libinput, which would also explain why it happened after updating.
@herrsimon

I clicked left, middle and right. It seems as if the middle and right buttons are recognized correctly, but still the clicks are not recognized
(in the sense that graphical applications do not react) without using xinput set-button-map as described above

BTN_RIGHT failing to generate a right click suggests a change higher up the input stack consistent with my theory, though BTN_SIDE being generated by leftmouse (I assume this is what you are using in your keyd config) shouldn't happen (indeed, I get BTN_LEFT using 26e19358 in my own testing).

@herrsimon Can you post the output of xinput list?

@rvaiya

If so, my suspicion is that some mouse detection mapping/code has changed in X/libinput

Yes, that's what I suspect too. Something like "if your pointer device has 17 buttons, it means it's some new iThingy and has to behave differently".
libinput debug-events, press keyboard "shortcut" for middle-mouse-click:
--- original keyd (with both loops)
-event13  TABLET_PAD_BUTTON       +39.030s	12 pressed (mode 0)
 event13  TABLET_PAD_BUTTON       +39.157s	12 released (mode 0)

--- modified keyd (second loop commented out)
 event13  POINTER_BUTTON          +0.000s	BTN_MIDDLE (274) pressed, seat count: 1
 event13  POINTER_BUTTON          +0.128s	BTN_MIDDLE (274) released, seat count: 0

@herrsimon Thanks.
I did a bit of digging and was able to narrow it down this commit in systemd:
https://github.com/systemd/systemd/commit/0855ce6772#diff-e3c18b2fc4f50acbb9f2d66a1daf229fde99a5c8785f17aa6f69e6a9c41ae2c7R261
It seems devices which are capable of emitting BTN_0 (and wheel events) are now treated as 'tablet pads'. This seems odd to me, since I believe BTN_0 is just a generic button event that can be used by any mouse driver, but I couldn't find any documentation which corroborates this (perhaps @whot can shed some light). In any case, the events are currently unused in keyd, so I have removed them in 0fb86d3a9, which should solve the problem.

I believe BTN_0 is just a generic button event that can be used by any mouse driver

Mice and similar devices use the buttons starting with BTN_LEFT and up. BTN_0 is really a "generic numbered button" and only tablet pads use those (statistically anyway) - you shouldn't see real-world mice with those buttons. Mind you, there's this systemd PR which may also fix this issue here I just haven't had the time to look at that one yet.

@whot Thanks for the edification :). I'm going to mark this as closed. @herrsimon Feel free to reopen if the issue remains unresolved.

couldn't find any documentation which corroborates this

Documentation is deprecated in systemd. Documentation is readable by humans, we only do binary, therefore can't read nor write it. By not adhering to documentation, we are able to boot more quickly.
~~ Poettering

@herrsimon just a troll, ignore them, nothing to be gained by playing in their hands.


ci: add a bug report template for the repo

This is a first attempt at a bug report template using Github Forms
(currently in beta for public projects). See:
https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository
https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-githubs-form-schema
Resolves #6

:tada: This PR is included in version 1.0.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How could you support an OS that does not exist (yet)

In Readme you describe you also support Debian9 and Debian10.
In metadata.json is only Debian 7.
Please streamline what you plan to support.

It does look like we got a bit over-enthused with that portion of the README. I'll work on a match to fix that as well as the metadata section. In the meantime, you can check our documentation to see what is supported: https://docs.newrelic.com/docs/infrastructure/new-relic-infrastructure/getting-started/compatibility-requirements-new-relic-infrastructure#operating-systems
Thanks!

This was fixed with #3


bind to <IP_ADDRESS>

without binding to <IP_ADDRESS>, evercam-api is not accessible from the host machine.

Thanks :+1:


Update utils.py - fix issue 273 , make_index not working

make_index doesnt work because 'break' column is str which makes the sum(axis=1) fail. issue 273
this fix will only use the columns in the original ticker_weights
an alternate fix (not included here) would be to include 'breaks' column in drop line
index.drop(columns=["first_day", "breaks"], inplace=True)

this fix will avoid make_index to error out, however the function is returning wrong values. it's hugely overstating results, basically it's rebalancing on the re-balance date and overstating returns on non rebalance dates. which has the net result of applying a leverage on the index.

Since this library no longer seems to be maintained, we have created a duplicate of it that we plan to maintain and improve over time. If you'd like to use our version check it out here: https://github.com/Lumiwealth/quanstats_lumi
Please feel free to fork and send your pull request to this new library, we will be actively monitoring the library and approving PRs


Infinite loop on small images

Mac user here. It seems that when I use images smaller than 10px (10px width, 9px height, png), the application hangs and seems to go in an infinite loop. It says: Optimizing atlas in the bottom right corner. I have to force quit it to stop it. When I use an image of 10x10px, it's fine. I searched in the source code, but I am having a hard time to find out why it does that. There is a for and while loop after that statement that displays the message:
bool SpriteAtlas::packWithRect(const QVector<PackContent>& content) {

Hi, please send me examples of files which cause this mistake, and the file of the project with settings

json of the project file:
{
    "algorithm": "Rect",
    "dataFormat": "pixijs",
    "destPath": "../../assets",
    "encryptionKey": "",
    "epsilon": 15,
    "heuristicMask": false,
    "imageFormat": "*.png",
    "jpgQuality": 80,
    "pixelFormat": "ARGB8888",
    "pngOptLevel": 7,
    "pngOptMode": "None",
    "premultiplied": true,
    "prependSmartFolderName": true,
    "rotateSprites": false,
    "scalingVariants": [
        {
            "forceSquared": false,
            "maxTextureSize": 2048,
            "name": "",
            "pow2": false,
            "scale": 1
        }
    ],
    "spriteBorder": 0,
    "spriteSheetName": "spritesheet",
    "srcList": [
    ],
    "textureBorder": 0,
    "trimMode": "Rect",
    "trimSpriteNames": true,
    "trimThreshold": 0,
    "webpQuality": 80
}

I used this sprites:

and sprite sheet worked correctly

did you also try the sprite I provided?

i found mistake
int step = (w + h) / 20;
incorrect calculate steps
try fix this: int step = max( (w + h) / 20, 1);

ah, so It would resolve to 0, hanging somewhere without doing any steps?

yes, see last commit
Thanks for the help :)

Nice. Btw, the latest release is from 2017, will you create a new release soon?

Oh I did not do it so long ago:)

37 commits behind master ;)

crashed when i choose a smaller size of the sprite sheet


Rename String#unescape_html just in case

Plus, call a different Ruby class that is more appropriate for what this method does, and handles stuff like —.

Definitely not the problem.  I'm fixing the real problem in a separate PR.


Rename getInitialGCSummaryDetails -> getBaseGCDetails

The GC blob is moving from data stores to container runtime. So, data stores shouldn't use getInitialGCSummaryDetails anymore as they don't care where the GC details are coming from.

This is not needed anymore as the PR moving GC blob to container runtime was closed.


[Azure-IoT-Sdk*] Update to 2020-07-19

Azure-IoT-Sdk for C release 2020-07-19

Hi @ewertons
Thanks for this PR.
Have you tested these features?

Hi @NancyLi1013 , thanks for the review and suggested changes.
Yes, our packages have been tested. Please merge the PR as soon as possible, so we can proceed with our release process.
Thanks.

Thanks @NancyLi1013 , actually upon verification these two ports do not need to change. I updated the PR not to update azure-macro-utils-c and umock-c.

@ewertons
Thanks for your clarification.
LGTM now.

@NancyLi1013 , if the changes are good could we please get them merged? Our latest release depends on this update. Thanks.

@ewertons
Since there is something wrong with CI checks. I will try to rerun this PR.
We will merge this if all CI checks have passed.

/azp run

/azp run

/azp run


Update license and readme

Open soooooource all the things

This was already reviewed by @droazen in a previous branch that I scrapped and rebooted.


Update text-stroke.json

Firefox 48 will support -webkit-text-stroke according to https://developer.mozilla.org/en-US/docs/Web/CSS/-webkit-text-stroke and https://developer.mozilla.org/en-US/Firefox/Releases/48

Thanks!


📝 Update docstring generator to include target branch for push

Description
The docstring generator workflow successfully runs when merging branches originating from upstream-main. However, it fails to push docstrings during workflow execution for branches originating from fork-main due to detached branches.
This PR resolves the issue by specifying the target branch for pushing docstrings.
Thanks @simon1hofmann for helping to figure this out!
Checklist:

[ ] The pull request only contains commits that are related to it.
[ ] I have added appropriate tests and documentation.
[ ] I have added a changelog entry.
[ ] I have created/adjusted the Python bindings for any new or updated functionality.
[ ] I have made sure that all CI jobs on GitHub pass.
[ ] The pull request introduces no new warnings and follows the project's style guidelines.

Then the changes of #475 can probably be reverted right?
on: push should be enough.

I think it's not possible to push to the branch like that, also add-and-commit should take care of that automatically.

ah great! Thank you!

@Drewniok @simon1hofmann does this work as intended now?

@marcelwa Not yet

Alright, keep me posted


When emitting all files, emit the changed file first

Fixes #16888
Currently, when a global file is updated, we will emit all files in include order. This changes it so that we first emit the changed file, then emit all other files in include order.
Also adds infrastructure for testing the builder.
The builder API is still in flux, so these tests will have to be updated once an API is decided on.

Waiting for review from @sheetalkamat

When emitting all to a single file: does it change the order of text in the output, or just the order of execution of emit logic?
Ensuring the textual order is preserved is critical for class inheritance.

It will preserve the order of text since any thats controlled by emitter..


limitations of spanned text formatting

The formatting system we have now currently has clients provide text ranges to format in specific ways. This has a few flaws compared to markdown which should be addressed:

Harder to upgrade. Clients will need to update their text inputs to make it possible to use new formatting capabilities. Increases maintainer workload. With markdown, unsupported syntax could still be written with an outdated client and it would still work.

Impossible / Inconsistent on some platforms. Bridges from other services will face issues with formatting, because of how on most of them the formatting is fundamentally text-based.

clients can just display a format for unsupported formats no problem, which will have the same amount of meaning as unknown symbols to a user.
this is actually the easiest format for bridging: all the data is already in a convenient format, ready to be made into whatever format the service can work with, which isn't necessarily markdown. incompatible formats can be discarded or converted into other formats via simple manipulation of span objects instead of trying to manipulate a CST for markdown, or unparsed markdown.


Defence - Sandwich defence

Insert the user input in between two prompts.
https://learnprompting.org/docs/prompt_hacking/defensive_measures/sandwich_defense
Each defence should include the following:

A frontend component for the defensive measure on the left side bar.
Checkbox to toggle the defensive measure.
Some way to get a description of the defensive measure.
Pulsing the defensive measure component when it captures malicious content.

Reopening now that the prompt enclosure defence is being added.
Blocked until that's merged in.


Option for datastore other than Google Cloud

Do you specifically mean a local Google Cloud Datastore compatible store, like the gcloud emulator? That's definitely doable right now, just set the DATASTORE_EMULATOR_HOST env var.
If you mean a different datastore altogether, eg MySQL or PostgreSQL, that's a much bigger project. Related background in https://github.com/snarfed/bridgy/issues/1323 . I wouldn't plan to work on this myself, but I'm happy to help if you want to tackle it!

I'll assume the latter, eg a different pluggable datastore like an RDBMS or the filesystem. This issue can track that feature request.

oh sorry! yes, i did mean the latter c:


Intel CPU Power Tracking via Power Gadget on Mac/Windows

Tried following options from online literature to fetch CPU Power usage:

PyJoules - unable to pip install
PyRAPL - unable to pip install
reading RAPL files at /sys/class/powercap/intel-rapl/* on Linux, however, unable to find these files on Azure and AWS VMs running Intel Processors.

Installed Intel Power Gadget on Mac. Generates current CPU power consumption across GT, IA, and DRAM and net Processor details via command line and is logged into a file, from which the Processor Power_{i}(Watt) values are fetched, to serve as current total Power drawn in Watts.

Thanks a lot for this PR @kngoyal we're almost there!

How do you install power_gadget on Linux? I want to test.
Could you also paste the install errors for PyJoules and PyRAPL, they should be straightforward to install.

From the official website, it is only available for MacOS and Windows.

So the alternative is to use code from energy-usage. I can do that this afternoon @kngoyal

import os
import re
import time
from pathlib import Path

BASE = "/sys/class/powercap/"

class RAPLFile:
    def __init__(self, name, path):
        self.name = name
        self.path = path
        self.recent = 0

    def set_recent(val):
        self.recent = val

    def __repr__(self):
        return "RAPLFile({}, {}, {})".format(self.name, self.path, self.recent)

def reformat(name, multiple_cpus):
    """ Renames the RAPL files for better readability/understanding """
    if 'package' in name:
        if multiple_cpus:
            name = "CPU" + name[-1] # renaming it to CPU-x
        else:
            name = "Package"
    if name == 'core':
        name = "CPU"
    elif name == 'uncore':
        name = "GPU"
    elif name == 'dram':
        name = name.upper()
    return name

def get_files():
    """ Gets all the RAPL files with their names on the machine
        Returns:
            filenames (list): list of RAPLFiles
    """
    # Removing the intel-rapl folder that has no info
    if not Path(BASE).exists():
        return [], False
    files = list(filter(lambda x: ':' in x, os.listdir(BASE)))
    names = {}
    cpu_count = 0
    multiple_cpus = False
    for file in files:
        if (re.fullmatch("intel-rapl:.", file)):
            cpu_count += 1

    if cpu_count > 1:
        multiple_cpus = True

    for file in files:
        path = BASE + '/' + file + '/name'
        with open(path) as f:
            name = f.read()[:-1]
            renamed = reformat(name, multiple_cpus)
        names[renamed] = BASE + file + '/energy_uj'

    filenames = []
    for name, path in names.items():
        name = RAPLFile(name, path)
        filenames.append(name)

    return filenames, multiple_cpus

def read(file):
    """ Opens file and reads energy measurement """
    if file == "":
        return 0
    with open(file, 'r') as f:
        return to_joules(int(f.read()))

def to_joules(ujoules):
    """ Converts from microjoules to joules """
    return ujoules*10**(-6)

def start(raplfile):
    measurement = read(raplfile.path)
    raplfile.recent = measurement
    return raplfile

def end(raplfile, delay):
    measurement = read(raplfile.path)
    raplfile.recent = (measurement - raplfile.recent) / delay
    return raplfile

def joules_to_kwh(j):
    return j * 2.7778e-7

def measure_files(files, delay = 1):
    """ Measures the energy output of all packages which should give total power usage
    Parameters:
        files (list): list of RAPLFiles
        delay (int): RAPL file reading rate in ms
    Returns:
        files (list): list of RAPLfiles with updated measurements
    """

    files = list(map(start, files))
    time.sleep(delay)
    files = list(map(lambda x: end(x, delay), files)) # need lambda to pass in delay
    return sum(joules_to_kwh(f.recent) for f in files)

def compute_cpu_power(delay=1):
    rapl_files, _ = get_files()
    if not rapl_files:
        return -1
    return measure_files(rapl_files, delay=delay)

compute_cpu_power(delay=1) will return the cpu power measured on all available cpus through the RAPL interface. Returns -1 if RAPL not available
Is that fine to integrate @kngoyal ?
Adapted from energy-usage.utils

So the alternative is to use code from energy-usage. I can do that this afternoon @kngoyal

Thanks @vict0rsch

import os
import re
import time
from pathlib import Path

BASE = "/sys/class/powercap/"

class RAPLFile:
    def __init__(self, name, path):
        self.name = name
        self.path = path
        self.recent = 0

    def set_recent(val):
        self.recent = val

    def __repr__(self):
        return "RAPLFile({}, {}, {})".format(self.name, self.path, self.recent)

def reformat(name, multiple_cpus):
    """ Renames the RAPL files for better readability/understanding """
    if 'package' in name:
        if multiple_cpus:
            name = "CPU" + name[-1] # renaming it to CPU-x
        else:
            name = "Package"
    if name == 'core':
        name = "CPU"
    elif name == 'uncore':
        name = "GPU"
    elif name == 'dram':
        name = name.upper()
    return name

def get_files():
    """ Gets all the RAPL files with their names on the machine
        Returns:
            filenames (list): list of RAPLFiles
    """
    # Removing the intel-rapl folder that has no info
    if not Path(BASE).exists():
        return [], False
    files = list(filter(lambda x: ':' in x, os.listdir(BASE)))
    names = {}
    cpu_count = 0
    multiple_cpus = False
    for file in files:
        if (re.fullmatch("intel-rapl:.", file)):
            cpu_count += 1

    if cpu_count > 1:
        multiple_cpus = True

    for file in files:
        path = BASE + '/' + file + '/name'
        with open(path) as f:
            name = f.read()[:-1]
            renamed = reformat(name, multiple_cpus)
        names[renamed] = BASE + file + '/energy_uj'

    filenames = []
    for name, path in names.items():
        name = RAPLFile(name, path)
        filenames.append(name)

    return filenames, multiple_cpus

def read(file):
    """ Opens file and reads energy measurement """
    if file == "":
        return 0
    with open(file, 'r') as f:
        return to_joules(int(f.read()))

def to_joules(ujoules):
    """ Converts from microjoules to joules """
    return ujoules*10**(-6)

def start(raplfile):
    measurement = read(raplfile.path)
    raplfile.recent = measurement
    return raplfile

def end(raplfile, delay):
    measurement = read(raplfile.path)
    raplfile.recent = (measurement - raplfile.recent) / delay
    return raplfile

def joules_to_kwh(j):
	"""
	Converts joules to kwh
	"""
    return j * 2.7778e-7

def measure_files(files, delay = 1):
    """ Measures the energy output of all packages which should give total power usage
    Parameters:
        files (list): list of RAPLFiles
        delay (int): RAPL file reading rate in ms
    Returns:
        files (list): list of RAPLfiles with updated measurements
    """

    files = list(map(start, files))
    time.sleep(delay)
    files = list(map(lambda x: end(x, delay), files)) # need lambda to pass in delay
    return sum(joules_to_kwh(f.recent) for f in files)

def compute_cpu_power(delay=1):
    rapl_files, _ = get_files()
    if not rapl_files:
        return -1
    return measure_files(rapl_files, delay=delay)

compute_cpu_power(delay=1) will return the cpu power measured on all available cpus through the RAPL interface. Returns -1 if RAPL not available. Unit is kWh.
If files are computed beforehand in the __init__ then measure_files should be used instead
Is that fine to integrate @kngoyal ?
Adapted from energy-usage.utils

This should be done in another PR, as this is a disjoint module for RAPL interface.

I agree I just thought you'd know better than myself where to put it :)

I've tested this PR on Windows and I've identified several issues:

The PowerLog executable is named PowerLog3.0.exe.
It is not in the PATH by default.
If the user don't have permission to write the csv file in the given directory, PowerLog don't show an error or return with a different return code. I'm not sure if we check that the user have write permission to the output dir.

After fixing those issues locally, it seems to works as it should.

I've tested this PR on Windows and I've identified several issues:

The PowerLog executable is named PowerLog3.0.exe.
It is not in the PATH by default.
If the user don't have permission to write the csv file in the given directory, PowerLog don't show an error or return with a different return code. I'm not sure if we check that the user have write permission to the output dir.

After fixing those issues locally, it seems to works as it should.

Thanks @Lothiraldan ! Good to know that it works on Windows as well.
In terms of local changes, if there are code changes, can you please suggest them or push those changes.

As you described, I believe this would change, _windows_exec = "PowerLog.exe" to _windows_exec = "PowerLog3.0.exe".

What is the default path it resides at? We can add it to the documentation, to set the PATH accordingly.

We can add to the documentation that a user specifies the output_dir  to the one they have write permissions to.

Let me know if you have additional suggestions.

I've tested this PR on Windows and I've identified several issues:

The PowerLog executable is named PowerLog3.0.exe.
It is not in the PATH by default.
If the user don't have permission to write the csv file in the given directory, PowerLog don't show an error or return with a different return code. I'm not sure if we check that the user have write permission to the output dir.

After fixing those issues locally, it seems to works as it should.

Thanks @Lothiraldan ! Good to know that it works on Windows as well.
In terms of local changes, if there are code changes, can you please suggest them or push those changes.
* As you described, I believe this would change, `_windows_exec = "PowerLog.exe"` to `_windows_exec = "PowerLog3.0.exe"`.

Yes we can do that for now, but not sure if the executable name will stay the same in a next release for Window. I think it's ok for now.

* What is the default path it resides at? We can add it to the documentation, to set the PATH accordingly.

I will paste the default path and I think we can add it as a backup check (like what we do on Mac). Users can change where it's installed tough.

* We can add to the documentation that a user specifies the `output_dir`  to the one they have write permissions to.

If we don't have write permissions to the output_dir we won't be able to write the emission.csv, right? I think we can check if we have write access (by creating an empty file emission.csv for example) and fallback on a temporary directory instead.

Let me know if you have additional suggestions.

I updated the PR but I sill have to test it on Windows. @kngoyal I think the backup fallback on OS X is flawed, could you check it? I don't have access to a Mac OS machine.

I updated the PR but I sill have to test it on Windows. @kngoyal I think the backup fallback on OS X is flawed, could you check it? I don't have access to a Mac OS machine.

Looks about right on my machine.
shutil.which("/Applications/Intel Power Gadget/PowerLog")
'/Applications/Intel Power Gadget/PowerLog'

I updated the PR but I sill have to test it on Windows. @kngoyal I think the backup fallback on OS X is flawed, could you check it? I don't have access to a Mac OS machine.

Looks about right on my machine.
shutil.which("/Applications/Intel Power Gadget/PowerLog")
'/Applications/Intel Power Gadget/PowerLog'

OK I think I know why I was confused, is /Applications/Intel Power Gadget/PowerLog the full-path to the OSX executable or its folder? For Windows, I've added the folder as a backup: _windows_exec_backup = "C:\\Program Files\\Intel\\Power Gadget 3.5\\"

I updated the PR but I sill have to test it on Windows. @kngoyal I think the backup fallback on OS X is flawed, could you check it? I don't have access to a Mac OS machine.

Looks about right on my machine.
shutil.which("/Applications/Intel Power Gadget/PowerLog")
'/Applications/Intel Power Gadget/PowerLog'

OK I think I know why I was confused, is /Applications/Intel Power Gadget/PowerLog the full-path to the OSX executable or its folder? For Windows, I've added the folder as a backup: _windows_exec_backup = "C:\\Program Files\\Intel\\Power Gadget 3.5\\"

Yeah it's the executable path.


Description of problem:
After https://github.com/home-assistant/home-assistant/pull/25590 was merged, some 3rd party components like roku-card stopped working properly, as powered off states don't report any attributes.
I can look into it this afternoon (I did the roku component changes), but I'm filing this for issue tracking. I am also considering maybe media_players whose state is "off" aren't allowed to have attributes, so please let me know if this is working as intended.

@manutenfruits I think that's what is going on here; if a player is off there are no sources. I can add a check in the roku-card to overcome that. I'm actually in the middle of a rewrite and was going to remove that source list anyways :)
Another, and perhaps better option from @Villhellm was to use STATE_STANDBY instead of STATE_OFF.

Can confirm using STATE_STANDBY solves this issue. I've been running a custom roku component with these changes using standby instead of off for over a month


NVD caching beginner question

Hi depedency check team,
to test NVD caching on my machine I

used vulnz 5.1.0 to fill a cache directory
placed that directory in a local tomcat on my machine
and added
<nvdDatafeedUrl>http://localhost:8080/nvdcache/nvdcve-{0}.json.gz</nvdDatafeedUrl>
and
<autoUpdate>false</autoUpdate>
to my dependency check plugin config

In the maven build log for my sonar profiles I see
...
[DEBUG]   (f) nvdDatafeedUrl = http://localhost:8080/nvdcache/nvdcve-{0}.json.gz
...
[DEBUG] Setting: nvd.api.datafeed.url='http://localhost:8080/nvdcache/nvdcve-{0}.json.gz'
...

Am I right? Is it that simple to use NVD caching?
If yes we'll then maybe run a nightly vulnz cron for a mirror URL and add those params to different parent pom snapshots when updating them to dependency check 9.x
Regards,
Gunnar

Autoupdate should not be set to false. If it is false it will not use the configured data feed.

Thank you.
I assume the feed URL has to be SSL?
a.t.m. I'm just testing localhost with a self signed cert produce by keytool.
"bad request" for cache.properties without SSL and with SSL "SunCertPathBuilderException: unable to find valid certification path to requested target"
I'll ask an admin to provide a mirror using our valid organzation cert.

depending on your server - properties files may not be served automatically. Try upgrading to vulnz 5.1.1 and ODC 9.0.5 (or higher). recent versions of vulnz started producing meta files in addition to the cache.properties and ODC 9.0.5+ will look for the meta files if it can't get the cache.properties file.

Thank you again, I'll do so next week because here's weekend now - have a nice weekend :)

Just some final testing feedback:
I just had to allow "*localhost" in nonProxyHosts to make it work on my notebook with 9.6.0.
Files are downloaded from my "localhost tomcat mirror"
For testing purposes I for example also

added 0 to the plugin config
renamed /.../dependency-check-data/9.0 to .../9.0-backup to force downloads

Worked like a charm.

ODC 9.0.5+ will look for the meta files if it can't get the cache.properties file

Did this break in the meantime?
I tried with 11.1.0 against https://mirror.cveb.in/nvd/json/cve/1.1/ (has no cache.properties but *.meta files) and got the dreaded 404 again.
 mvn org.owasp:dependency-check-maven:11.1.0:aggregate -DfailBuildOnCVSS=7 -DcveStartYear=2020 -DskipProvidedScope=true -DnodeAuditSkipDevDependencies=true -DnodePackageSkipDevDependencies=true -DnvdDatafeedUrl="https://mirror.cveb.in/nvd/json/cve/1.1/nvdcve-{0}.json.gz"
...
[ERROR] Unable to download the data feed META files
org.owasp.dependencycheck.data.update.exception.UpdateException: Unable to download the data feed META files
    at org.owasp.dependencycheck.data.update.NvdApiDataSource.getRemoteCacheProperties (NvdApiDataSource.java:637)
...
caused by ResourceNotFoundException: https://mirror.cveb.in/nvd/json/cve/1.1/cache.properties - Server status: 404


hm_iter is difficult to use in a thread-safe manner or in single-threaded cases where reentry is possible

Adding an extra "info" parameter that hm_iter passes into the iterator callback makes it possible to use hm_iter in many more scenarios. Without this, the caller has to resort to passing data that the iterator callback might need via a thread-local variable or some other out of band mechanism.

This was leftover; I removed this in favor of just letting folks iterate the map's .items normally.
The declaration should be removed. The implementation is.


Translation of the doc in english

The site https://ecomfe.github.io/zrender-doc/public/ is in Chinese only.
A translation to English would help the project to be used internationally.
I don't speak Chinese, any help appreciated 🙏 .

Source for site is here: https://github.com/ecomfe/zrender-doc


Feature/restaurant categorie

Summary
This pull request adds several new features and improvements to the TableTop app. It includes the following changes:

Refactors functionality to add a facility per venue.
Adds functionality to add a cuisine to a restaurant.
Adds an active field to cuisine_per_restaurant and soft deletion to it.
Fixes a minor bug.

Details
This commit is an update to the TableTop app, as it introduces several new features and improvements. The commit includes a refactor of the functionality to add a facility per venue, which will improve the performance and maintainability of the app.
In addition, the commit adds the ability to add a cuisine to a restaurant, which is an important feature for managing the menu. The commit also adds an active field to cuisine_per_restaurant and soft deletion to it, which makes it easier to manage the cuisines associated with each restaurant.
The commit also includes a fix for a minor bug, which will improve the stability of the app.

Changes according to feedback


Roll Flutter Engine from bcb281cde579 to d132adad9869 (2 revisions)

https://github.com/flutter/engine/compare/bcb281cde579...d132adad9869
2024-11-07<EMAIL_ADDRESS>iOS: Eliminate ScopedBlock, ScopedTypeRef, ScopedPolicy (flutter/engine#56410)
2024-11-07<EMAIL_ADDRESS>[skwasm] Make sure to include the transfer list when using postMessage. (flutter/engine#56431)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jsimmons<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Nested braces-support

I figured out, that CSSComb doesn't like nested braces.
.foo {
    width: calc((2 * @bar));
}

The CLI prints the following stack:
stack:  assets/less/test.less
Please check the validity of the block starting from line #3

1 | .foo {
2 |     width: calc((2 * @bar));
3*| }

Gonzales PE version: 3.0.0-10
CSScomb Core version: 2.0.3

It's useful to make further calculations.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/5491581-nested-braces-support?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github).

It still does.
width: calc((100% - 75px) / 4);

+1

This seems to be fixed by the latest csscomb release 🎆


feat: Aggregate log events based on rum flag

Aggregate logging events according to the log flag received from rum call to BCS.

Overview

Part 2 of the auto-logging feature work. This work will enable the logging feature to be controlled via BCS, which is ultimately based on user settings from UI (on/off, sampling, log level).
Related Issue(s)
https://new-relic.atlassian.net/browse/NR-323597
Testing
Extended existing logging aggregate unit and e2e tests around handling the flag.  Includes tests for each logging mode to ensure log messages of equal or higher levels are collected.  For example, if the log flag = 3 (INFO), then collect console.error, console.warn, console.info, and console.log events.
Extended existing tests to ensure no harvests when rum flag = 0 and events are drained.

Mentioned in standup today (and got ack'd) that the default level was changed after the initial approval. We will re-review again when officially merging feat branch to main, so gonna go ahead and merge now.


CMake Error at /opt/ros/noetic/share/catkin/cmake/catkin_package.cmake

CMake Error at /opt/ros/noetic/share/catkin/cmake/catkin_package.cmake:183 (message):
catkin_package() CATKIN_DEPENDS on 'decomp_util', which has been found in
'/home/zwh/plan/MPCC/TMPCPlanner/devel/share/decomp_util/cmake/decomp_utilConfig.cmake',
but it is not a catkin package

Hi, I cannot reproduce the issue (I tried a fresh install to check, but it works for me). I think the issue is that decomp_util did not get compiled somehow, did you use the provided tasks to build?
I think running catkin build decomp_util separately will fix the problem. Please let me know.

Hello, I have also encountered the same issue. I have already run catkin build decomp_util in advance according to the README, but when I generate the solver for mpc_planner_rosnavigation and then compile the workspace, this error will occur.
Looking forward to your reply.

How are you building the planner, are you using mpc_planner_ws?
If not, is the name of the package DecompUtil instead of decomp_util? In that case it may somehow be related to the name of the folder and it would be best to rename to decomp_util.
If the error remains, please let me know how you are building so that I can try to reproduce it.

I have renamed DecompUtil to decomp_util, but the error still persists.
I followed the prompt in the README, completed the build, and ran roslaunch mpc_planner_jackalsimulator ros1_jackalsimulator. launch successfully. However, when I wanted to use mpc_planner_rosnavigation, I first ran catkin build decomp_util. Then, python mpc_planner_rosnavigation/scripts/generate_rosnavigation_stolver.py was run successfully in Conda. When I tried to compile the workspace again, the above error occurred. My workspace is named catkin_ws_t-mpc.

Hello friends in the community, may I ask if catkin build mpc_planner is running during compilation? I'm not sure what 'your_system' means in 'catkin build mpc_planner_<your_system>' in readme. When I run catkin build mpc_planner, there will be the following error message:
Starting  >>> guidance_planner
Starting  >>> mpc_planner_util
Finished  <<< guidance_planner                   [ 0.3 seconds ]
Finished  <<< mpc_planner_util                   [ 0.2 seconds ]
Starting  >>> mpc_planner_solver

Errors     << mpc_planner_solver:cmake /home/rancho/1hmy/mpc_planner_ws/logs/mpc_planner_solver/build.cmake.004.log
CMake Error at /home/rancho/1hmy/mpc_planner_ws/src/mpc_planner/mpc_planner_solver/CMakeLists.txt:14 (include):
include could not find load file:
solver.cmake
CMake Error at /opt/ros/noetic/share/catkin/cmake/catkin_package.cmake:367 (message):
catkin_package() could not find CFG_EXTRAS file.  Either
'cmake/mpc_planner_solver-extras.cmake.develspace.em',
'cmake/mpc_planner_solver-extras.cmake.em',
'cmake/mpc_planner_solver-extras.cmake.develspace.in',
'cmake/mpc_planner_solver-extras.cmake.in',
'cmake/mpc_planner_solver-extras.cmake' or a variant specific to the
installspace must exist.
Call Stack (most recent call first):
/opt/ros/noetic/share/catkin/cmake/catkin_package.cmake:102 (_catkin_package)
CMakeLists.txt:16 (catkin_package)
cd /home/rancho/1hmy/mpc_planner_ws/build/mpc_planner_solver;  catkin build --get-env mpc_planner_solver | catkin env -si  /usr/bin/cmake /home/rancho/1hmy/mpc_planner_ws/src/mpc_planner/mpc_planner_solver --no-warn-unused-cli -DCATKIN_DEVEL_PREFIX=/home/rancho/1hmy/mpc_planner_ws/devel/.private/mpc_planner_solver -DCMAKE_INSTALL_PREFIX=/home/rancho/1hmy/mpc_planner_ws/install -DCMAKE_BUILD_TYPE=Release;  cd -
..........................................................................................................................
Failed     << mpc_planner_solver:cmake           [ Exited with code 1 ]
Failed    <<< mpc_planner_solver                 [ 0.5 seconds ]
Abandoned <<< mpc_planner_modules                [ Unrelated job failed ]
Abandoned <<< mpc_planner                        [ Unrelated job failed ]
[build] Summary: 4 of 7 packages succeeded.
[build]    Ignored:   19 packages were skipped or are skiplisted.
[build]    Warnings:  None.
[build]    Abandoned: 2 packages were abandoned.
[build]    Failed:    1 packages failed.
[build] Runtime: 1.0 seconds total.                                                                                Among them, I installed Acados according to the process in readme, but did not install Forces Pro. I don't know if this will have an impact? I am very much looking forward to receiving your reply, sincere hope and gratitude.

Hello friends in the community, may I ask if catkin build mpc_planner is running during compilation? I'm not sure what 'your_system' means in 'catkin build mpc_planner_<your_system>' in readme. When I run catkin build mpc_planner, there will be the following error message:
Starting >>> guidance_planner
Starting >>> mpc_planner_util
Finished <<< guidance_planner [ 0.3 seconds ]
Finished <<< mpc_planner_util [ 0.2 seconds ]
Starting >>> mpc_planner_solver
Errors << mpc_planner_solver:cmake /home/rancho/1hmy/mpc_planner_ws/logs/mpc_planner_solver/build.cmake.004.log
CMake Error at /home/rancho/1hmy/mpc_planner_ws/src/mpc_planner/mpc_planner_solver/CMakeLists.txt:14 (include):
include could not find load file:
solver.cmake
CMake Error at /opt/ros/noetic/share/catkin/cmake/catkin_package.cmake:367 (message):
catkin_package() could not find CFG_EXTRAS file. Either
'cmake/mpc_planner_solver-extras.cmake.develspace.em',
'cmake/mpc_planner_solver-extras.cmake.em',
'cmake/mpc_planner_solver-extras.cmake.develspace.in',
'cmake/mpc_planner_solver-extras.cmake.in',
'cmake/mpc_planner_solver-extras.cmake' or a variant specific to the
installspace must exist.
Call Stack (most recent call first):
/opt/ros/noetic/share/catkin/cmake/catkin_package.cmake:102 (_catkin_package)
CMakeLists.txt:16 (catkin_package)
cd /home/rancho/1hmy/mpc_planner_ws/build/mpc_planner_solver; catkin build --get-env mpc_planner_solver | catkin env -si /usr/bin/cmake /home/rancho/1hmy/mpc_planner_ws/src/mpc_planner/mpc_planner_solver --no-warn-unused-cli -DCATKIN_DEVEL_PREFIX=/home/rancho/1hmy/mpc_planner_ws/devel/.private/mpc_planner_solver -DCMAKE_INSTALL_PREFIX=/home/rancho/1hmy/mpc_planner_ws/install -DCMAKE_BUILD_TYPE=Release; cd -
..........................................................................................................................
Failed << mpc_planner_solver:cmake [ Exited with code 1 ]
Failed <<< mpc_planner_solver [ 0.5 seconds ]
Abandoned <<< mpc_planner_modules [ Unrelated job failed ]
Abandoned <<< mpc_planner [ Unrelated job failed ]
[build] Summary: 4 of 7 packages succeeded.
[build] Ignored: 19 packages were skipped or are skiplisted.
[build] Warnings: None.
[build] Abandoned: 2 packages were abandoned.
[build] Failed: 1 packages failed.
[build] Runtime: 1.0 seconds total. Among them, I installed Acados according to the process in readme, but did not install Forces Pro. I don't know if this will have an impact? I am very much looking forward to receiving your reply, sincere hope and gratitude.

@Yihappyy Sorry, I still cannot reproduce the issue. But looking at the original message, it seems that decomp_util was somehow build with cmake? Do you also have a folder decomp_util/cmake? If so, best to remove it and possibly to remove decomp_utilConfig.cmake too. The provided CMakelists.txt should be sufficient. Let me know if the issue persists.

@hmyooo You do not have to install Forces Pro. In your case, it seems that the solver generation was not successful. Please generate a solver first per the instructions (with for example <your_system> as jackalsimulator) then build with catkin build mpc_planner_jackalsimulator.
For easier use of the planner I highly recommend the VSCode environment: https://github.com/tud-amr/mpc_planner_ws/tree/main where these steps are captured in tasks.

@hmyooo You do not have to install Forces Pro. In your case, it seems that the solver generation was not successful. Please generate a solver first per the instructions (with for example <your_system> as jackalsimulator) then build with catkin build mpc_planner_jackalsimulator.
For easier use of the planner I highly recommend the VSCode environment: https://github.com/tud-amr/mpc_planner_ws/tree/main where these steps are captured in tasks.

Thank you very much for your reply. I have tried this website https://github.com/tud-amr/mpc_planner_ws/tree/main However, when running Jackal: Build, there is still a problem with the following steps: Starting  >>> mpc_planner_solver

Errors     << mpc_planner_solver:cmake /workspace/logs/mpc_planner_solver/build.cmake.000.log
CMake Error at /workspace/src/mpc_planner/mpc_planner_solver/CMakeLists.txt:14 (include):
include could not find load file:
solver.cmake
CMake Error at /opt/ros/noetic/share/catkin/cmake/catkin_package.cmake:367 (message):
catkin_package() could not find CFG_EXTRAS file.  Either
'cmake/mpc_planner_solver-extras.cmake.develspace.em',
'cmake/mpc_planner_solver-extras.cmake.em',
'cmake/mpc_planner_solver-extras.cmake.develspace.in',
'cmake/mpc_planner_solver-extras.cmake.in',
'cmake/mpc_planner_solver-extras.cmake' or a variant specific to the
installspace must exist.
Call Stack (most recent call first):
/opt/ros/noetic/share/catkin/cmake/catkin_package.cmake:102 (_catkin_package)
CMakeLists.txt:16 (catkin_package)
cd /workspace/build/mpc_planner_solver;  catkin build --get-env mpc_planner_solver | catkin env -si  /usr/bin/cmake /workspace/src/mpc_planner/mpc_planner_solver --no-warn-unused-cli -DCATKIN_DEVEL_PREFIX=/workspace/devel/.private/mpc_planner_solver -DCMAKE_INSTALL_PREFIX=/workspace/install -DCMAKE_BUILD_TYPE=RelWithDebInfo -DPYTHON_VERSION=3;  cd -
...............................................................................................
Failed     << mpc_planner_solver:cmake             [ Exited with code 1 ]
Failed    <<< mpc_planner_solver                   [ 2.1 seconds ]
Abandoned <<< mpc_planner_modules                  [ Unrelated job failed ]
Abandoned <<< mpc_planner                          [ Unrelated job failed ]
Abandoned <<< mpc_planner_jackal                   [ Unrelated job failed ]

Warnings   << roadmap:make /workspace/logs/roadmap/build.make.000.log       ...............................................................................................
Finished  <<< roadmap                              [ 16.3 seconds ]
Finished  <<< guidance_planner                     [ 30.3 seconds ]
[build] Summary: 9 of 13 packages succeeded.
[build]   Ignored:   14 packages were skipped or are skiplisted.
[build]   Warnings:  1 packages succeeded with warnings.
[build]   Abandoned: 3 packages were abandoned.
[build]   Failed:    1 packages failed.
[build] Runtime: 39.4 seconds total.
[build] Note: Workspace packages have changed, please re-source setup files to use them.

@hmyooo You do not have to install Forces Pro. In your case, it seems that the solver generation was not successful. Please generate a solver first per the instructions (with for example <your_system> as jackalsimulator) then build with catkin build mpc_planner_jackalsimulator.
For easier use of the planner I highly recommend the VSCode environment: https://github.com/tud-amr/mpc_planner_ws/tree/main where these steps are captured in tasks.

Thank you very much for your reply. I have tried this website https://github.com/tud-amr/mpc_planner_ws/tree/main However, when running Jackal: Build, there is still a problem with the following steps: Starting >>> mpc_planner_solver
Errors << mpc_planner_solver:cmake /workspace/logs/mpc_planner_solver/build.cmake.000.log CMake Error at /workspace/src/mpc_planner/mpc_planner_solver/CMakeLists.txt:14 (include): include could not find load file:
solver.cmake
CMake Error at /opt/ros/noetic/share/catkin/cmake/catkin_package.cmake:367 (message): catkin_package() could not find CFG_EXTRAS file. Either 'cmake/mpc_planner_solver-extras.cmake.develspace.em', 'cmake/mpc_planner_solver-extras.cmake.em', 'cmake/mpc_planner_solver-extras.cmake.develspace.in', 'cmake/mpc_planner_solver-extras.cmake.in', 'cmake/mpc_planner_solver-extras.cmake' or a variant specific to the installspace must exist. Call Stack (most recent call first): /opt/ros/noetic/share/catkin/cmake/catkin_package.cmake:102 (_catkin_package) CMakeLists.txt:16 (catkin_package)
cd /workspace/build/mpc_planner_solver; catkin build --get-env mpc_planner_solver | catkin env -si /usr/bin/cmake /workspace/src/mpc_planner/mpc_planner_solver --no-warn-unused-cli -DCATKIN_DEVEL_PREFIX=/workspace/devel/.private/mpc_planner_solver -DCMAKE_INSTALL_PREFIX=/workspace/install -DCMAKE_BUILD_TYPE=RelWithDebInfo -DPYTHON_VERSION=3; cd -
............................................................................................... Failed << mpc_planner_solver:cmake [ Exited with code 1 ] Failed <<< mpc_planner_solver [ 2.1 seconds ] Abandoned <<< mpc_planner_modules [ Unrelated job failed ] Abandoned <<< mpc_planner [ Unrelated job failed ] Abandoned <<< mpc_planner_jackal [ Unrelated job failed ]
Warnings << roadmap:make /workspace/logs/roadmap/build.make.000.log ............................................................................................... Finished <<< roadmap [ 16.3 seconds ] Finished <<< guidance_planner [ 30.3 seconds ] [build] Summary: 9 of 13 packages succeeded. [build] Ignored: 14 packages were skipped or are skiplisted. [build] Warnings: 1 packages succeeded with warnings. [build] Abandoned: 3 packages were abandoned. [build] Failed: 1 packages failed. [build] Runtime: 39.4 seconds total. [build] Note: Workspace packages have changed, please re-source setup files to use them.

Sorry, the latest situation is that I used the jackal: generate slow and build command again without any errors, but ran JackalSimulator: Run Simulator command reports RLException error: [ros1_jackalsimulator.launch] is neither a launch file in package [mpc_planner_jackalsimulator] nor is [mpc_planner_jackalsimulator] a launch file name
The traceback for the exception was written to the log file

Please use Jackal Simulator: Generate Solver and Build instead of Jackal: Generate Solver and Build to run the simulator. Then Jackal Simulator: Run Simulator

Your suggestion is very useful, thank you very much！！！


Updated Java docs and Test cases for code authored by samzx

resolves #146

Pull Request Test Coverage Report for Build 239

48 of 49 (97.96%) changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 91.033%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/logic/commands/OrderCommand.java
14
15
93.33%

Totals

Change from base Build 234:
0.09%

Covered Lines:
2203

Relevant Lines:
2420

💛  - Coveralls


Prioritize starred IntelliSense suggestion

When I try to use one of the methods of pd (pandas), there is few starred suggestions there.

This is cool, so why not utilize this one further on? When I type "d", then there are no starred suggestions.

I suggest that the behavior in the first picture should be carried out in the second picture as well. Starred ones should be at top and their order should not be changed as well.
I don't know the name for starred suggestions, so there is that.

Intellicode is context aware, it's supposed to change. Depending on the context it suggests different functions.

This looks like a bug rather than enhancement. Most probably IC does not get chance to augment the list. This works in "Microsoft" LS

This should be fixed in the next release.

This issue has been fixed in version 2021.9.3, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/main/CHANGELOG.md#202193-22-september-2021


fix(types): improve typescript support for dynamic imports (#179)

:tada: This PR is included in version 3.6.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: