Add support for loading assets through the HTTP/2 server

Part of the fix for https://github.com/clockwork-dog/cogs/issues/1328
This change detects if we're in a situation where we can use a HTTP/2 server to load the assets and uses a different asset URL if we are.
On the client side this is determined by the environment we're connecting from (and the media cogs-av-box version, if applicable).
Server side is dictated by COGS sending a message to say the HTTP/2 server is supported.

Ready for another review 👍


Generator: Remove auroLabs question during generate

Describe the bug

To Reproduce
Steps to reproduce the behavior:

Run the generate script for a new component
Note the question regarding auroLabs

Expected behavior
Since we no longer support auroLabs we should remove this question

I wouldn't call that a bug, it's a feature update. It still works. Do we want it? No. But it's not broken.


Private Docker Registry - sidecar pull issue

Hello-
I have pushed all the istio required docker images to our private registry and successfully running all the pods in my eks cluster.
I have enabled the namespace with automatic istio-injection and deployed my application using helm. The istio proxy images is pointing to my private registry but the the proxyv2 is still pointing to the docker.io.
How do i change that to point to private registry?
Events:
Type     Reason     Age              From                                     Message

Normal   Scheduled  2m               default-scheduler                        Successfully assigned van-dit/microapi-field-config-856fc84d69-dh9bs to ip-10-10-120-120.ec2.internal
Normal   Pulling    2m               kubelet, ip-10-10-120-120.ec2.internal  pulling image "dtr.abc.qq.yy.xyz.com/asdf/istio_proxy_init:1.1.2"
Normal   Pulled     2m               kubelet, ip-10-10-120-120.ec2.internal  Successfully pulled image "dtr.abc.qq.yy.xyz.com/asdf/istio_proxy_init:1.1.2"
Normal   Created    2m               kubelet, ip-10-10-120-120.ec2.internal  Created container
Normal   Started    2m               kubelet, ip-10-10-120-120.ec2.internal  Started container
Normal   Pulling    2m               kubelet, ip-10-10-120-120.ec2.internal  pulling image "dtr.abc.qq.yy.xyz.com/asdf/my-microservice:5.1.0"
Normal   Pulled     2m               kubelet, ip-10-10-120-120.ec2.internal  Successfully pulled image "dtr.abc.qq.yy.xyz.com/asdf/my-microservice:5.1.0"
Normal   Started    2m               kubelet, ip-10-10-120-120.ec2.internal  Started container
Normal   Created    2m               kubelet, ip-10-10-120-120.ec2.internal  Created container
Warning  Failed     2m (x2 over 2m)  kubelet, ip-10-10-120-120.ec2.internal  Failed to pull image "docker.io/istio/proxyv2:1.1.2": rpc error: code = Unknown desc = Error response from daemon: Get https://registry-1.docker.io/v2/: x509: certificate signed by unknown authority
Warning  Failed     2m (x2 over 2m)  kubelet, ip-10-10-120-120.ec2.internal  Error: ErrImagePull
Normal   BackOff    1m (x3 over 2m)  kubelet, ip-10-10-120-120.ec2.internal  Back-off pulling image "docker.io/istio/proxyv2:1.1.2"
Warning  Failed     1m (x3 over 2m)  kubelet, ip-10-10-120-120.ec2.internal  Error: ImagePullBackOff
Warning  Unhealthy  1m (x4 over 2m)  kubelet, ip-10-10-120-120.ec2.internal  Readiness probe failed: Get http://<IP_ADDRESS>:8080/health: dial tcp <IP_ADDRESS>:8080: connect: connection refused
Normal   Pulling    1m (x3 over 2m)  kubelet, ip-10-10-120-120.ec2.internal  pulling image "docker.io/istio/proxyv2:1.1.2"
Thanks

I fixed the issue by editing the configmap.
kubectl edit configmap istio-sidecar-injector  -n istio-system


osd-list exception

Can one of the admins verify this patch?

This PR needs a rebase from the latest changes

later a new pr


Minecraft crash with the latest version of connector but not on the old version

Describe the bug
crash
Steps to reproduce

...
Logs
https://mclo.gs/OFT3Uwq
Additional context
It works with version 21, https://mclo.gs/aNYPHr5

Please provide information about your installed mods. The log doesn't show any errors

https://paste.atlauncher.com/view/e035e02a-7ace-4427-a0ea-802aafdb92bf. Does this help?

  Nothing in the log or debug output, no crash log.

that doesnt have a crash in it

Exactly, no clue what's causing it.  Could be related to https://github.com/Sinytra/Connector/issues/480 though.

bewitchment doesnt work

Forge 47.1.30 != Forge 47.1.3


Update to a more reliable way to get VM Status

Fix for https://github.com/Azure/azure-devtestlab/issues/495
Updating the code for getting the status of a VM to jump to the underlying Compute VM for the 'real' status.  In most cases, this is 1 additional call to Azure (Get-AzureRmVm -Status)

Also adding code to fix https://github.com/Azure/azure-devtestlab/issues/494 , we are going to block on older versions of Azure PowerShell.

Suggest putting code under '-ExtendedState' parameter for performance reason and add an -'ExtendedState' parameter to dtl-getvm as well


NLM denoising (HAL)

Non-local means denoising for CV_8UC3.
Implementation based on ftp://ftp.math.ucla.edu/pub/camreport/cam08-01.pdf
Note: Not ready to merge. Needs HAL acceleration.

Even without HAL methods, the new algorithm is quite fast.
Note that the current "fast" implementation is also multithreaded via parallel_for_.
photo_posix_x64_950e6ff_20150703-161150.xml

                       Name of Test                             Number of     Number of    Min       Median   Geometric mean    Mean    Standard deviation
                                                            collected samples outliers
fastNlMeansDenoisingColored::Size_HvalType::(1280x720, 10)         13             1     1988.86 ms 2036.33 ms   2052.64 ms   2053.24 ms      52.33 ms
fastNlMeansDenoisingColored::Size_HvalType::(1920x1080, 10)        10             0     4378.75 ms 4517.57 ms   4517.59 ms   4518.64 ms     103.32 ms
fastNlMeansDenoisingColored::Size_HvalType::(640x360, 10)          100            8     504.03 ms  598.88 ms    646.60 ms    657.44 ms      122.65 ms
fastNlMeansDenoisingColored::Size_HvalType::(960x540, 10)          13             1     1101.74 ms 1129.82 ms   1144.90 ms   1145.37 ms      34.34 ms
halNlMeansDenoising::Size_HvalType::(1280x720, 10)                 10             0     1613.56 ms 1616.46 ms   1617.33 ms   1617.34 ms      3.72 ms
halNlMeansDenoising::Size_HvalType::(1920x1080, 10)                13             1     3635.22 ms 3677.91 ms   3699.68 ms   3700.76 ms      95.44 ms
halNlMeansDenoising::Size_HvalType::(640x360, 10)                  10             0     397.68 ms  401.46 ms    402.86 ms    402.88 ms       4.41 ms
halNlMeansDenoising::Size_HvalType::(960x540, 10)                  10             0     907.36 ms  912.16 ms    912.28 ms    912.28 ms       3.58 ms

Original image:

Current NLM:

New NLM:

ok, let's close it


[Feature][Dashboard][OpenAPI] Implement OpenAPI for dashboard

Implement all OpenAPI for the feature list in https://github.com/oam-dev/kubevela/issues/675

Closing it as It's duplicate to https://github.com/oam-dev/kubevela/issues/740

Closing it as It's duplicate to https://github.com/oam-dev/kubevela/issues/740


407 custom donation input 2

Issue
#407 Donation Parsing / Custom Donation Input:
Version 2 of PR #498
Currently, Amplify only allows donations of $1, $2, $20, and $50 (and soon $1.50). It would be nice to allow users to make a custom donation, as well as make it more clear what the default donation is.
Fix
This PR primarily tackles step 1 / part 1 of 4 subtasks, which is to (1) update UI to accept custom donations and clearly indicate a default value, but paves the way for (3) that donations are correctly being sent to Stripe.
Here are some changes, assumptions, and consideration:

[x] added a "Custom Amount" v-btn
[x] added a "Donation Amount" v-text-input
[x] added corresponding input validation

[x] provides feedback message to user for

No amount
Minimum amount ($1.50)
Maximum amount ($10,000)

[x] validated user input will always be a number with 2 decimal places
[x] if user tries to manually enter a number with 2+ decimal places, it'll be rounded up
[x] todo: backend will only need to multiply by 100 for Stripe Checkout

[x] applied consistent styles for all toggle buttons & text input
[x] default value is $2, and user can input $1.5 as minimum value. So, we might be able to say the $1.50 button is solved together with this custom amount feature?
[x] considered mobile responsiveness for different screen sizes
[x] improved code readability

Enjoyed working with @coderlore & @Alex-is-Gonzalez on this!
Feedback is always welcome!

@yoyoyojoe Hey Joey! How is the PR going? I'd also like to remind you to update the branch with the changes from main so that it remains consistent and we minimize conflicts and resolve them as soon as we get them. :)

Noting that with the failing test, I have some questions/observations:
(1) In the logs, the failing Integration Tests is recommending to update caniuse-lite. Should I go ahead and do that?

Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db
Why you should do it regularly: https://github.com/browserslist/browserslist#browsers-data-updating

(2) When I run the following test locally, it seems to run OK. Should I change something there?
npm test server/__tests__/integration/axios-cache-interceptor.test.js

(3) Is this test getting into the territory of testing axios-cache-interceptor?

The tests seemed to have passed after clicking on Update branch. 👍🏻

This branch is out-of-date with the base branch .
Merge the latest changes from main into this branch.

Noting that with the failing test...

Noting that with the failing test, I have some questions/observations:
(1) In the logs, the failing Integration Tests is recommending to update caniuse-lite. Should I go ahead and do that?

It's a very common message I've noticed in many open-source projects! @DietBepis1 Do you think we should do it?

The tests seemed to have passed after clicking on Update branch. 👍🏻

This branch is out-of-date with the base branch .
Merge the latest changes from main into this branch.

Noting that with the failing test...

Looks like you got it to pass now. :)


Remove error-prone caching step.

Description
It is error prone because if the curl request fails, the cache might have corrupt data.  Furthermore, while a aws s3 cp operation is running, the file is not in a consistent state.
Test plan
test in sandbox

I tested this:
curl <gisaid_url> | bunzip2 | zstd | aws s3 cp - <blah>
Killed curl, and even though aws s3 cp got the message that the pipe failed, it left a partial dump on s3.  This is really bad.

Literally, the only reason for this to exist is during development, and there's wdl caching to handle that.  No need to complicate production workflows!


ipp-nio builds fail

all builds on SPM seem to fail because of this target with a custom path
        .executableTarget(
            name: "PrintPDF",
            dependencies: [
                .target(name: "IppClient"),
            ],
            path: "Examples/PrintPDF",
            exclude: ["hi_mom.pdf"]
        ),

error: 'spi-builder-workspace': invalid custom path 'Examples/PrintPDF' for target 'PrintPDF'
it works just fine locally - any tips?
Originally posted by @sliemeobn in https://github.com/SwiftPackageIndex/PackageList/issues/6115#issuecomment-1879057462

I've just run the build locally and it works, so there must be something tripping up our build system.

I have just been looking into this, too, and I do get the error in every environment.
I tried it with our build environment tool, first and could reproduce the error, but I also get it from a simple checkout of the project and a swift build.

Ohh wait, that * is suspicious, and sure enough look what's happened.

We must be deleting those two files with the build tool somehow, maybe as we're trying to clean up misleading xcodeproj files, which I know we do.

Thanks so much for the report, @sliemeobn. We'll need to figure this out and get it fixed.

I mean what could possibly go wrong, right? 😆

https://github.com/SwiftPackageIndex/SwiftPackageIndex-Server/issues/2798#issuecomment-1867900493

haha, sometimes things don't work in mysterious way, but rather crudely ; )
thanks for jumping on this so quickly!
should I rename to folder for now, or is this something you are intent on fixing soon-ish?
PS: can't wait to hear about Sven joining the SSWG on the next podcast episode.

Nothing to change on your end! It's a simple fix that I'll push out tomorrow, no worries :)

Fixed here: https://gitlab.com/finestructure/swiftpackageindex-builder/-/merge_requests/246
Will re-trigger builds once deployed and close the issue once it's confirmed to work.

Deployed and re-triggered:

It's odd that the visionOS build succeeds while iOS fails but we've made some progress at least. I'll close this for now, please re-open if these still look wrong to you in some way.
You can find the complete build command properly shown also for failing builds when you click through on individual builds. That should help verify that everything is in order.
Thanks again for the report, @sliemeobn !


added slicer (4.5.0-1,461633)

https://www.slicer.org

Merged as 9d9ce919bc2d28cfab4b51e3709969c9ba08cb6a.


Feature request - Make config available for reading triggers

Currently when the jobs are started the config is not available. Due to this the triggers cannot be made a part of the config and needs to be either hard-coded or manually read from a config file.
It would be good if config is available when triggers are evaluated.

@anshbansal This fork repo is no longer being used.  Would you mind recreating this issue in the main repo (https://github.com/grails-plugins/grails-quartz)?

created
https://github.com/grails-plugins/grails-quartz/issues/33


Accommodation for pds4file, convert associated_abspaths

From rms-webtools created by juzen2003: SETI/rms-webtools#63

In Pds4File/pdsfile.py, convert associated_abspaths function to list all the associated absolute paths of a pdsfile instance.

Fixed by pull #17


Floating point FBO in SDL_gpu

I was wondering whether SDL_gpu supports floating point framebuffer objects.
I'm working on a remake of an older game in C/C++ and I used SDL_gpu to be able to utilize shaders. However, when I draw using the shader of my choice (ntsc-adaptive from libretro), everything seems to have a pink tint to it (left - shader off, right - shader on):

I learned that this might be a side effect of not utilizing a floating point FBO. Does SDL_gpu support this in some way? When calling GPU_CreateImage I can pick the format, but whatever SDL_gpu format I use, it all seems to be of internal format GL_RGBA/type GL_UNSIGNED_BYTE under the hood (and I'm assuming I need GL_RGBA32F/GL_FLOAT).

That is correct.  SDL_gpu assumes GL_UNSIGNED_BYTE.  Floating point framebuffers would probably be a good feature to expose, as lacking it limits things like HDR shaders.  I can't tell if that's your actual issue here, but it is an issue.

I see, thanks. SDL_gpu doesn't seem to enjoy more than 4 bytes per pixel under the hood too, so quickly adding in GL_RGBA32F/GL_FLOAT doesn't seem like it's going to work.
Great library, by the way. Really fun to use.

I can confirm the issue was indeed the lack of floating point framebuffer object support in SDL_gpu. I implemented that functionality (GL_RGBA32F support) myself and it's all working now.

I might create a pull request later.


[Improve] [doc] Display plugins in the form of a table

It would be better that we can display our plugins in the form of a table, which will be more intuitive to users.

Any comments are welcome :smile:

I would like to do it.


Treat expressions as statements

Description
This commit makes it so that all expressions are now treated as statements.
Some changes where made to MLS as well as it can now occur as a "statement string" or a multi-line comment,
where previously a standalone MLS was always a comment. To be a string it needed to be part of  an assignment.
If the MLS is at the top of a file it will be considered as a comment, otherwise as a Literal.
closes #3367
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

I feel like I am missing some tests but I don't now which.

About the tests, I don't immediately see what else you could add. This is a pretty big change to the parser but we can't go and verify that all possible parse paths are tested. I think just have to have confidence that they all have tests already. If not, I'm sure the advanced testing we've got planned for the end of the sprint will reveal any issues.

(in which case we can improve the testing)

Processing this pull request

Merged into branches master in 10cabbe94f788c729f23488d6892710eea9eb09a

Processing #3972.


cleaner Swift solutions with Surge

the swift solutions will get pretty messy if we have to keep writing all the parts of numpy that we are missing. A better solution is to use a module for matrices, like Surge.
Compare my pure solution for p004 (also a pull request) to the one here - about half of the number of lines of code, and considerably more white-space and comments!

I am noticing that large portions of the Surge library and not completed, including a large portion of matrix operations. I am withdrawing this PR.


#663 complete support for submissions

[x] add tests for scores adapter
[x] add tests for uncovered branches of UpdateScoresCommand
[x] add tests for auto-merging submissions
[x] add tests for data-integrity tokens from GameService.Adapt
[x] add tests for GameService.Delete -> Adapt
[x] add tests for GameService.Upsert -> Adapt
[ ] fix exception when logged in as an admin

Resolves #663

Exception raised when viewing partially submitted score card and logged in as an admin

After saving matching submission the controls are still editable

Switch between submissions does not switch the home man of the match.


Hapi-FHIR server is down

Server : http://fhirtest.uhn.ca/baseDstu2  ,   Is it the server or should I do something, because it was running last night and giving me errors today. If server is down then please fix that up asap.

How long will it take for it start up again?

The server is unreliable right now because the disk is almost full. Someone has uploaded a massive amount of data to it over the last few weeks.
I'm trying to get it under control right now.

the server is up again.


Add semantic tests

Adds init_test to initialise the test folder
Adds isoltest binary to generate inputs and outputs for semantic tests

Where are the sources for the isoltest executable?
Can we add this whole thing to CI?

Where are the sources for the isoltest executable? Can we add this whole thing to CI?

I have added the binary.
We want setup our own runners so that PRs aren't stalled by semantic tests, I'll start conversation with the devops team today/tomorrow? Is that acceptable for now?

Where are the sources for the isoltest executable? Can we add this whole thing to CI?

I have added the binary. We want setup our own runners so that PRs aren't stalled by semantic tests, I'll start conversation with the devops team today/tomorrow? Is that acceptable for now?

Yes. But how did you get that binary?) Where are the sources that you built it from?

Where are the sources for the isoltest executable? Can we add this whole thing to CI?

I have added the binary. We want setup our own runners so that PRs aren't stalled by semantic tests, I'll start conversation with the devops team today/tomorrow? Is that acceptable for now?

Yes. But how did you get that binary?) Where are the sources that you built it from?

Oh that patch is in nethersolc https://github.com/NethermindEth/nethersolc/pull/31


Add specific version for Realm in Podfile

I think for now Podfile is not specified Realm version, so when run pod install it will be installed Realm 0.93.0 and actually I've been occurred build error below.(Because Migration API was changed in 0.93.0)

So it needs to support for Realm 0.93.0 or describe version in Podfile or share Podfile.lock, I think.

Yeah that code just needs removed. I put it in there initially and it isn't used. Will fix!

This commit should fix this without needing to add specific Realm version:
https://github.com/Roobiq/RBQFetchedResultsController/commit/06819a2b8fbea5402d306219f34c84a33dd5c094

OK, got it! Thanks!
But maybe it should be specified the Realm version or be shared Podfile.lock?
Because same trouble will occur when Realm API will be changed. I think.

That's true it might be better to update the supported Realm as contributor versus leaving it open for developer just wanting to use library to possibly run into the issue themselves first.


How to save pointcloud as .pcd file in python ?

Required Info

Camera Model
{ D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10)

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC

SDK Version
{ legacy / 2.. }

Language
{python }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
I want to save pointcloud as a " .pcd  " file in python but i couldn't find any way. Is is possible in python ?
i just find this
pc = rs.pointcloud() points = rs.points() ... pc.map_to(color_frame) points = pc.calculate(depth_frame) points.export_to_ply("./ply_file.ply", color_frame)

I hope that the discussion in the link below will be useful to you.
https://github.com/IntelRealSense/librealsense/issues/2483
In that discussion, it makes reference to how a RealSense community member recommends a solution called python-pcl
https://github.com/strawlab/python-pcl

Thank you for your response.
This python-pcl did not work for me but i did with pclpy
My solution
import pclpy
from pclpy import pcl
...
pc = rs.pointcloud()
points = rs.points()

...

depth_intrinsics = rs.video_stream_profile(depth_frame.profile).get_intrinsics()
w, h = depth_intrinsics.width, depth_intrinsics.height
out = np.empty((h, w, 3), dtype=np.uint8)
out.fill(0)
points = pc.calculate(depth_frame)

v, t = points.get_vertices(), points.get_texture_coordinates()
verts = np.asanyarray(v).view(np.float32).reshape(-1, 3)  # xyz
texcoords = np.asanyarray(t).view(np.float32).reshape(-1, 2)  # uv

pc2 = pcl.PointCloud.PointXYZ(verts)
writer = pcl.io.PCDWriter()
writer.writeBinary("./PointCloud.pcd",pc2)

Thanks for sharing your solution!

PCD is a format closely related to PCL, so it makes sense to use one of the PCL wrappers to do it.
We are not planning to add this capability directly to librealsense.
Given a solution was found, I'm closing this one. @Gazi13 let us know if you need any more information.


pull request for e2e test at 01/16/2017 19:26:13

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


Spot lights not working ?

No, currently spotlights don't work. The lighting part is easy, but implementing shadows is a bit difficult. I plan to fix it soon.

Got it.

 fix spotlights


Refactor minmax API

This PR is part of https://github.com/stdlib-js/todo/issues/1435
Description

What is the purpose of this pull request?

This pull request:

Refactors minmax package to support assign when the result needs to overwrite a given array

Related Issues

Does this pull request have any related issues?

This pull request is part of https://github.com/stdlib-js/todo/issues/1435
Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

The typescript linter doesn't like that the optional arguments are defined in the middle of the assign signature.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

@kgryte the only thing that remains are the TypeScript declarations 😉


Discontinuities

I'm surprised but discontinuities like the attached figure seem to cause us problems still. I can copy my old heuristic code over for splitting the light curves on discontinuities but it might also be interesting to try to figure out what's going wrong...

P.S. this is KIC 10419108


installed lunchy,zsh error occurred

error message:
complete:13: command not found: compdef
lunchy version: Lunchy 0.10.4
zsh version: zsh 5.0.5 (x86_64-apple-darwin14.0)
in ~/.zshrc ,add the following code
LUNCHY_DIR=$(dirname gem which lunchy)/../extras
if [ -f $LUNCHY_DIR/lunchy-completion.zsh ]; then
. $LUNCHY_DIR/lunchy-completion.zsh
fi

Add -U to Autoload
https://github.com/eddiezane/lunchy/issues/66


Add header routing

Does not include routing for resources page as Matt is working on this page in parallel.

Chatted offline. router-link inherits the a formatting from homepage.css in a way that is not straightforward to override. We're going to use @click for now since it is functional and the formatting looks correct, modulo no cursor showing up on hover.

Chatted offline. router-link inherits the a formatting from homepage.css in a way that is not straightforward to override. We're going to use @click for now since it is functional and the formatting looks correct, modulo no cursor showing up on hover.

For posterity, you can force the pointer to show up on hover by adding cursor: pointer; to the header-links CSS class.


Recordset performance issue?

Here are the links to the data commons demo:

https://nih-commons.derivacloud.org/chaise/recordset/#1/public:KC7 Search Demo
https://dev.isrd.isi.edu/chaise/recordset/#44989/public:GTEx_lung_cancer_demo_data_v2-clean@sort(key)

Notice that the table is very wide but not very big. It has no fks, etc.. However, the browser will hang if we choose 100 or 200 items per page.
The workaround is to work on the visible columns. But maybe we can look at this page to give us some clue on our performance issue..

This issue is related to #1378

In the following I try to summarize how the process of investigating this issue went down. I will reference this document in my explanations. In there you can find more information about different experiments that I ran. The load times that are written there were based on the output that chrome developer tools would print out. They might not be accurate, but for the sake of comparison it was satisfactory.
In the default case (52 range facets, 26 choice facets, 84 columns, 25 rows), the page would take  around 11 seconds to load. In the first few experiments, I tried changing number of rows, columns, and facets. What I realized was that the number of facets in this scale (rows less than 25), was the biggest factor. Investigating more revealed that the main issue was the range facets and the fact that we're attaching all the heavy plotly elements to the DOM even when the facet is closed. Therefore I changed the range facet behavior to only attach the plotly when the facet is opened. This change reduced the load time by around 50%.
After this, we tried experimenting with the number of rows and columns in a higher scale. What we found out is that in those scales (rows * columns > 5000) it's the main table rows that take a long time to render. Even in higher scales this delay would result in browser not responding for around 20 seconds. This showed that we might need to change the way that we're showing the main result table.

I tried different methods for reducing the load time in the extreme cases, but none of the trivial methods worked. This requires more research on the methods that we can improve the performance for ng-repeat.


feat: increase maximum CPU count allouwed in machine_cpu_count valida…

…tion to 96

/gcbrun

Thank you for running the CI @imrannayer


Issue with demo in docs: invalid LambdaRequest variant

Hi there, I am following the setup on the getting started guide, and running into issues when invoking the lambda locally. Example repo here: https://github.com/kykyi/new-lambda-project
Steps to reproduce:
% cargo lambda watch

In a new shell
% cargo lambda invoke --data-ascii "{ \"command\": \"hi\" }"

The following errors appear:
% cargo lambda watch
 INFO invoke server listening on [::]:9000
 INFO starting lambda function function="_" manifest="Cargo.toml"
    Finished dev [unoptimized + debuginfo] target(s) in 0.15s
     Running `target/debug/new-lambda-project`
ERROR Lambda runtime invoke{requestId="ee7b6bc4-3b39-48a1-905a-62c9df1659c0" xrayTraceId="Root=1-6489b435-c73d8d35b6e6e17b4010966a;Parent=93efcd1dbfe8be77;Sampled=1"}: DeserializeError { inner: Error { path: Path { segments: [] }, original: Error("data did not match any variant of untagged enum LambdaRequest", line: 0, column: 0) } }

And
% cargo lambda invoke --data-ascii "{ \"command\": \"hi\" }"

Error: lambda_runtime::deserializer::DeserializeError

  × failed to deserialize the incoming data into the function's payload
  │ type: data did not match any variant of untagged enum LambdaRequest
  │

But if I visit in the browser at http://localhost:9000/lambda-url/new-lambda-project/ it works as expected.
Dependencies:
https://github.com/kykyi/new-lambda-project/blob/b6ffdb6f93e3be42e5a8cf12813d99d905b7e3f3/Cargo.toml#LL17C1-L22C87
Thanks!

And it is an HTTP function via the HTTP gateway
Is this function an HTTP function? (y/N)
yes
? Which service is this function receiving events from?
  I don't know yet
  Amazon Elastic Application Load Balancer (ALB)
  Amazon Api Gateway REST Api
  > Amazon Api Gateway HTTP Api
  Amazon Api Gateway Websockets
  AWS Lambda function URLs


chore: drop zerollup, upgrade ts, upgrade other dependencies and security updates on the lock file

TS bumped up to 4.9, jumping to 5 breaks due to tslint, but migrating to eslint can be done on its own job. After tslint is migrated to eslint then ts can be bumped to v5
Globals moved to the new syntax and the root walker into its own file.
Ze-rollup dropped in favour of tsc-alias
/mnt/linuxdrive/code/opensource/generate-it/src/lib/template/helpers/pathParamsToDomainParams.ts

minor change, checks if the 'this' object is present due to the unit test

tested and all looks good


Extend filter entries to allow 4 digits to compare against weights

1. Why is this change necessary?
Make sure you can compare to the same digit amount you can store the product weights.
2. What does this change do, exactly?
Set digits property to number-fields that are used for weights as the default falls back to 2.
3. Describe each step to reproduce the issue or behaviour.

Setup a weight rule when the cart is at least 1.234 kg heavy
Input field rounds to 1.23 kg

4. Please link to the relevant issues (if any).
Follow up to @tinect s #997
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-9993
Please use this issue to track the state of your pull request.

@JoshuaBehrens you set digits to 4 instead of 3🤔

Four is just used to lure us into the perception that we have more control but you are correct. Three is enough.
You activated my trap card! e.g. I have a shop selling lightweight collectables like stamps, trading card game cards or coins? You can't enter the prices and compare them properly.

How can shopware be the platform to sell drugs online (fast)™ without having the weight be precise as a milligram?

Thanks again @JoshuaBehrens
I've changed the number digits to 3 :)

Thanks for the merge :)
@pweyck Do you think increasing this to 6 is bad? For real we need it to be 6 and made it a core hack.

@JoshuaBehrens I think it should be configurable, but then you might run into the float point precision limit. But I would be fine with increasing this to 6 digits. That should fit into 32 bit floats.

The 32-bit float hint is a good one. I'll have a look at it.


Project selector should select account after sign in

start logged out of all google accounts
load up the new project selector and click to open a dialog
sign into your account

-> I would have expected the account I just signed into to be selected now. Instead, no account is selected  (it is in the dropdown, I just need to manually choose it):

+1
On Mon, Dec 18, 2017 at 4:13 PM, Etan Shaul<EMAIL_ADDRESS>wrote:

start logged out of all google accounts
load up the new project selector and click to open a dialog
sign into your account

-> I would have expected the account I just signed into to be selected
now. Instead, no account is selected (it is in the dropdown, I just
need to manually choose it):
[image: image]
https://user-images.githubusercontent.com/1735744/34128367-6eef16fa-e40e-11e7-86fa-2dd936251925.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-intellij/issues/1791,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HVwYCqdpIAeqrRm7K8N4Oju6nLUBks5tBtWPgaJpZM4RGFJ-
.


Get tqdm working in vscode

This come from https://github.com/iris-hep/opendata-higgs-discovery/issues/19 - see that for more details.
But our current implementation of tqdm means that you get a lot of "junk" when running in vscode's notebooks.

The bug is that we aren't closing a bar (or two)... To repro:

Create the bar
Don't let it quite finish (so it remains open)
Then create another bar

And it will start producing those extra lines, one after the other.

Ah - the issue is that you need to just have more than one bar open at a time in vscode:
bar1 = tqdm(total=1000, dynamic_ncols=True, leave=True, unit='file', desc="hi 1", bar_format='{l_bar}{bar}| {n_fmt}/{total_fmt} [{elapsed}]')
bar2 = tqdm(total=1000, dynamic_ncols=True, leave=True, unit='file', desc="hi 2", bar_format='{l_bar}{bar}| {n_fmt}/{total_fmt} [{elapsed}]')
for i in range(1000):
    bar1.update(int(1))
    bar2.update(int(1))
    bar1.refresh()
    bar2.refresh()
    sleep(0.001)

will do it every time.

See tqdm issue for more discussion from the tqdm side. Also, a stack overflow issue to try and track this down.


chore(RHIF-172):Remove App info|Optimization effort

RHIF-172-Optimization Efforts :boom:
As an ongoing effort to optimize apps, this PR replaces the app info from front-end components and just imports it locally.
There should be no change in behavior/ flow
Suggested formats:
Why do we need this change? :thought_balloon:
Just performance/optimization effort.
Security Checklist :lock:
Upon raising this PR please go through RedHatInsights/secure-coding-checklist
:guardsman: Checklist :dart:

[ ] Bugfix
[ ] New Feature
[x] Refactor
[ ] Unittests Added
[ ] DRY code
[ ] Dependency Added

Hey @PreetiW  just popping this back up

Hey @PreetiW  just popping this back up

Hello, thank you for raising the PR! Sorry I couldn't review it earlier. I will go through it today 💯

@mkholjuraev I've passed down the inventoryId in this new commit

Thank you @adonispuente, @mkholjuraev and @PreetiW!


[Question] How to handle player's connection lost during a game?

Scenario: a player cut his game unintentionally or on purpose, should the game stop immediately?
Situations:

getting a call
enter background
kill the app
...

#66
If a player dropped before game ended:

Remove the game group, so other players get kicked out.

When a player finishes the game

Add oneself to the finished array
When the array reaches 4 players, check if everyone passes

All passed, show result page
not all passed, kick back to the chatroom

I think now it is correctly handled, except killing the app.


replaced instances of DS_API_URL with baseURL

Description
I replaced all instances of DS_API_URL with baseURL so that it is consistent throughout the file.
Video Link
[Loom Video]
Trello Link
Fixes # BL-930
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have removed unnecessary comments/console logs from my code
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

Great job replacing any instance of DS_API_URL with baseURL to make the overall file more consistent and improving the readability.


Update checklist.md

Status
Ready/In Progress/In Hold (Reason for hold)
Related Issues
fixes: link to the issue
Description
A few sentences describing the overall goals of the pull request's commits.
Screenshots
Paste here any images that will help the reviewer

Was handled in another pr


"alias -g" not working

I've added a custom/file.zsh everything works except:
alias -g G='| grep -vi '
alias -g L='| less'
the other aliases (without -g option) work fine.
$ grep ' -g' *.zsh
file.zsh:alias -g G='| grep -vi '
file.zsh:alias -g L='| less'

$ alias | grep 'grep'
G='| grep -vi '

The alias command returns both normal aliases and globals. If you do alias -g do you still see them?

Yes :
$ alias -g
...=../..
....=../../..
.....=../../../..
......=../../../../..
CA='2>&1 | cat -A'
G='| grep -vi '
H='| head'
L='| less'
LL='2>&1 | less'
M='| most'
NE='2> /dev/null'
NUL='> /dev/null 2>&1'
P='2>&1| pygmentize -l pytb'
T='| tail'

Ok, so it does work. How are you using them?

$ alias | grep pol
Apol='apt-cache policy '

$ alias G pol
must return same line but return every aliases

alias G pol is not the same as alias | grep pol, but alias | grep -vi pol. It's different. The G alias you defined inverts the match, so it only shows what doesn't match pol.

thank, sorry, my mistake.

No harm no foul :wink:


Make vApp templates orderable

Purpose or Intent
The initial version of the vApp template parser set orderable property
to false rendering templates unavailable for service dialogs. This small
patch just marks all vApp templates orderable.
@miq-bot add_label providers/vmware/cloud, enhancement

@gberginc would it make sense to leave these as not orderable until we get the provisioning piece working?  That way people don't try to use something that isn't fully working yet

@agrare We can postpone this, sure. In this case we can close this PR and have it part of the one(s) that will have support for provisioning?


Custom Asset Libraries

Godot version:
v3.1.alpha.calinou.9c0cc0d
Issue description:
Over the past few days, I've been exploring the idea of making a custom asset library for Godot. Barring the work needed to make such an idea work (such as needing to figure out the data that the official asset library's api sends to the client), one issue I found is that I don't see a way to add custom asset libraries while in-editor.
Godot already provides an official asset library (served from https://godotengine.org/asset-library/api), and also allows you to access an asset library running on localhost (served from http://<IP_ADDRESS>/asset-library/api), but so far I don't know of any way to add your own without editing the engine's source.
It would be nice if we could add our own asset library URLs while in-editor.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


[assertions] consider functional coverpoints

It would be nice to add some functional coverpoints to the Ibex design
to augment the verification efforts, and potentially take advantage of
some formal methods. I'm just starting the conversation here, but we
could suggest some coverpoints as follow-up in this issue.
CC @udinator

@GregAC: This presumably ties in to the functional coverage work you've been doing. I think we can probably just close this issue, since we've started the work now. What do you think?

@GregAC: This presumably ties in to the functional coverage work you've been doing. I think we can probably just close this issue, since we've started the work now. What do you think?

Yup I think so, still plenty to do here but the foundation is in.

Yup I think so, still plenty to do here but the foundation is in.


documentation: Add CONTRIBUTING.md, closes #17

This PR adds a brief CONTRIBUTING.md file stating how potential contributors can help evolving this project.
The CONTRIBUTING.md file currently lacks information regarding code formatting. The IDSA Dataspace Connector project uses a slightly amended sun_checks.xml and PMD configuration which IMO can be applied to this project as well. This is going to be added by a subsequent issue / pull request.
 Denis Neuling<EMAIL_ADDRESS>Daimler TSS GmbH, legal info/Impressum

I am unsure about the license header to be included to each source code file or whether it would be fully sufficient to just include the correct SPDX-License-Identifier. Would anybody please help clarifying what contributors should include there?


Remove bounded and exact iterators

This reduces our iterator traits to just ParallelIterator and
IndexedParallelIterator.  The bounded and exact iterators had some
theoretical use, but in practice the only distinction we use today is
whether something can be indexed.
The upper_bound method is totally removed.  The rest of the methods in
the removed traits are now merged into IndexedParallelIterator.
Fixes #229.
Closes #259.

Let's do it.


PersonCard: make use of CommandExecutor to execute commands

This PR extracts CommandExecutor interface into its own file, and passes the MainWindow::executeCommand method which fits the interface down to individual UI components PersonListPanel and then PersonCard. This allows the custom UI methods to use the same machinery for executing commands as CommandBox (with the exception of not adding the command to storage, since that is only done in CommandBox). This removes a source of duplication of code, and avoids a direct dependency on MainWindow.
TODO: update the DG if this will be merged.

Closing since it requires quite a bit of changes to the DG diagrams too close to the deadline.


PSoC6/Cat1 add binary blob for Cortex-M0+ core

Origin
I would like to add binary blob (CM0P_SLEEP) for Cortex-M0+, which are using in PSoC 6 /Cat1 (cy8cproto_062_4343w).
The background for using pre-built image is documented in RFC (https://github.com/zephyrproject-rtos/zephyr/issues/42883)

In the PSoC 6-based integration, the family's Cortex-M0+ will be controlled by a pre-built image, rather than being open to the user. This aligns with our current PSoC 6 strategies.

Type

[ ] Precompiled library
[x] Firmware image

Module
The Zephyr module that this blob(s) will be referenced from:
boards::arm::cy8cproto_062_4343w for now.
Purpose
Prebuilt application images are executed on the Cortex M0+ core of the CAT1 dual-core MCU.
Note: Each application image has a variant based on the hardware die (e.g. psoc6_01, psoc6_02, psoc6_03, ...)
CAT1A Images
COMPONENT_CM0P_SLEEP
This image starts CM4 core  and puts CM0+ core into a deep sleep mode.
COMPONENT_CM0P_CRYPTO
This image starts crypto server on CM0+ core, starts CM4 core and puts CM0+ core into a deep sleep mode.
COMPONENT_CM0P_BLESS
This image starts BLE controller on CM0+ core, starts CM4 core and puts CM0+ core into a deep sleep mode.
COMPONENT_CM0P_SECURE
This image starts CM4 core at address corresponding to Secure Boot policy, sets required security settings, initializes and executes code of Protected Register Access driver, puts CM0+ core into a deep sleep mode.
In scope of this ticket, we add support only for CM0P_SLEEP image (This image starts CM4 core and puts CM0+ core into a deep sleep mode.).
Pull Request

https://github.com/zephyrproject-rtos/zephyr/pull/44211 (commit https://github.com/zephyrproject-rtos/zephyr/pull/44211/commits/eb665f5c71a93c28dd78cfc6ca1420a294d936b1)

https://github.com/zephyrproject-rtos/hal_infineon/pull/6/ - Update hal_infineon,  added blob support for cm0+ images.

License
Copyright 2020-2022, Cypress Semiconductor Corporation (an Infineon company)
SPDX-License-Identifier: Apache-2.0

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

    http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.

cc: @ifyall

(adding here since the discussion is supposed to be here)
IMO Zephyr should be the repository of its code. It is one thing to have a module for a library, but this is core SoC support. Worse, it is a binary blob!
This should be implemented in C within Zephyr. The DT files can be generated but then should be checked into Zephyr.
Zephyr already supports Cortex-M0. I heard mention of a problem with the size...how much bigger is it with Zephyr than without?

The pre-compiled Cortex-M0+ images accomplish two goals:

Providing the default image for the CM0+ for PSoC 6 variants where the CM0+ is not open as an application processor.
Providing an image for the CM0+ when the CM0+ is open as an application processor but the user doesn't want to use it. This might be because they are evaluating the solution and don't need the complexity of writing code for both cores or their final system doesn't require developing their own code for the Cortex-M0+. In scope of the current integration, the Cortex-M0+ will not be open. We will be revisiting this in the future as we complete more work and bring additional devices to Zephyr.

In either of these cases, the code for the CM0+ must be as small as possible. In the current solution, 8 kb of flash have been allocated to the CM0+ sleep image. If we compare that size to the size of the simple Zephyr blinky sample on PSoC 6 Cortex-M4, that comes to over 33 kb of flash.
Yes, Zephyr supports CM0+, but in this case Zephyr is either significant overkill for the goal (e.g. a bare-metal main() that sets one register and then calls a deep sleep routine in an infinite loop) and/or incompatible with the license of the device (e.g. a CM0+ that is not open for application code).

In either of these cases, the code for the CM0+ must be as small as possible. In the current solution, 8 kb of flash have been allocated to the CM0+ sleep image. If we compare that size to the size of the simple Zephyr blinky sample on PSoC 6 Cortex-M4, that comes to over 33 kb of flash.

33KB? Sounds like something very wrong with your HAL or libraries, seeing 19KB here for that on an M4 and 16KB for that on an M0.
This can be fixed with sysbuild anyway, espressif have a similar requirement for a bootloader being required - see discussion here https://discord.com/channels/720317445772017664/733037524498382881/1072154327977840660 and that can be done using sysbuild. In fact, sysbuild can be used to build, include or flash an image that is not generated in zephyr, so you could add a sysbuild target here which builds the firmware for the M0+ core from scratch without using the zephyr libraries (though same compiler and host tools I would assume) and then have it flashed when someone runs their application.

@nordicjm,
Thanks for the pointers to the sysbuild work. We will take a look at that when we bring in the full multi-core support for the devices.
Regarding the flash size, it turns out I gave numbers for the button sample, not blinky. However, your point stands. We will audit our implementation to understand if this can be optimized.
Thanks,
Ian

IMO this is an odd use of a binary blob...
a) the source is available but sometimes need to be secret sometimes
b) or maybe it will not be secret but it is too big when using Zephyr (or will it?)
c) we are unable to build something with two binaries (or are we?)
If this goes ahead, I hope it can be quickly removed once one of these is resolved

Approved by TSC, closing.


How to build win64 JNI?

Hi, I'm attempting to build the JNI library on win64, and cant seem to find any instructions on how to build.
Reading the GNUMakefile, I've managed to setup appropriate links to the win32 JDK JNI headers, but compilation is failing
Trying the cross-compile option on linux, I get the following failure:
/XXXX/git/jffi2/jni/jffi/Library.c: In function ‘Java_com_kenai_jffi_Foreign_dlopen’:
/XXXX/git/jffi2/jni/jffi/Library.c:91:19: error: implicit declaration of function ‘PathIsRelativeW’ [-Werror=implicit-function-declaration]
dwFlags = PathIsRelativeW(path) ? 0 : LOAD_WITH_ALTERED_SEARCH_PATH;
^~~~~~~~~~~~~~~
I've tried adding the following include, but that then gives a slew of other errors per the further below:
-I/usr/i686-w64-mingw32/include/
Any suggestions on how to resolve the above?
Thanks!
#if __INTRINSIC_PROLOG(__movsd)
^~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/i686-w64-mingw32/include/psdk_inc/intrin-impl.h:1932:1: error: this use of "defined" may not be portable [-Werror=expansion-to-defined]
/usr/i686-w64-mingw32/include/psdk_inc/intrin-impl.h:1932:1: error: this use of "defined" may not be portable [-Werror=expansion-to-defined]

Hey there, sorry for the delay in responding!
The native builds are run by Ant, based on the instructions in the readme. Did you try that first?
FWIW we do have an x86_64 build for Windows located in the archive dir. Is it not working for you?


View form

Create a method called current_user in your ApplicationController to make current user data available to all controllers.

It will return the first user from the database.

Don't worry about proper authentication at this time. You will add it in the upcoming projects.

Create forms to perform the following functions:

Creates a Post on behalf of the current_user (use the method that you created in your ApplicationController).

Create a comment on behalf of the current_user (use the method that you created in your ApplicationController).

Allow Users to add likes to Posts.

@Mihndim2020 thank you for reviewing my project


Language switch

Hey, can you make a feature, which can switch the language of leon.

Hi @Mr-MubelBubel, it is already the case :wink:

I'm waiting for German.


reversedims python to julia without copying

Currently the tests will fail. Need to confirm exactly how they fail.

The issue is definitely with FillArrays showing up for Δ in the pullback. Although PyReverseDims does the conversion, something is getting lost.


One-click to turn on/off READ-ONLY setting

Is your feature request related to a problem? Please describe.
I often need to turn this setting on/off, to prevent my clobbering a database by mistake.   Currently, to do this, I need to click several times to "Edit Connection", which is inconvenient.
Describe the solution you'd like
Make turning on/off of READ-ONLY into a simple one-click from the right-button pull-down menu.
Admittedly, this is only a minor issue, but I thought I'd ask to make this great tool even BETTER!    Thank you!
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context

in case the connection read-only setting is switched on/off when data viewer is opened and another connection is chosen at db navigator, focus on connection is lost.

verified


Add Back My Stack banner and update file page design

This PR follows up #209  adds Back My Stack banner and updates file page design based on the Figma design provided.
Desktop Screenshot

Mobile Screenshot

Codecov Report

Merging #211 into design/homepage will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           design/homepage     #211   +/-   ##
================================================
  Coverage            50.22%   50.22%
================================================
  Files                   14       14
  Lines                  227      227
  Branches                41       41
================================================
  Hits                   114      114
  Misses                  91       91
  Partials                22       22

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 5059e8f...af6aa59. Read the comment docs.

Oh! Thank you, added that now.


Make  a child of

<tbody> is only allowed inside of <table>, see http://www.w3.org/TR/html-markup/tbody.html.
There are no styling changes needed, though now these two rules apply. https://github.com/elm-lang/elm-lang.org/blob/fc1fe4462baeab5abf7369ce360f589ec6feb6e7/assets/style.css#L184-L189
Should the table cells have borders? If not those rules can probably get removed.

Thanks!


Make simple usage of Microsoft.OpenAPI trimmable

I'm trying to use this library in a console app. When trying to publish using trimming, I encountered the fact that this library is not trimmable for even the simplest of cases.
Is your feature request related to a problem? Please describe.
The library is not trimmable when only the following functionality is used:

Creating the model by hand through: new OpenApiDocument { ... }
Serializing the model using OpenApiJsonWriter or OpenApiYamlWriter

Since these 2 steps don't feel like they are using reflection, I would expect this to be trimmable, which it is not. AFAIK, this is due to EnumExtensions.GetDisplayName():
/_/src/Microsoft.OpenApi/Extensions/EnumExtensions.cs(27,13): Trim analysis warning IL2075: Microsoft.OpenApi.Extension
s.EnumExtensions.GetAttributeOfType<T>(Enum): 'this' argument does not satisfy 'DynamicallyAccessedMemberTypes.PublicCo
nstructors', 'DynamicallyAccessedMemberTypes.PublicMethods', 'DynamicallyAccessedMemberTypes.PublicFields', 'Dynamicall
yAccessedMemberTypes.PublicNestedTypes', 'DynamicallyAccessedMemberTypes.PublicProperties', 'DynamicallyAccessedMemberT
ypes.PublicEvents' in call to 'System.Type.GetMember(String)'. The return value of method 'System.Object.GetType()' doe
s not have matching annotations. The source value must declare at least the same requirements as those declared on the
target location it is assigned to.

Describe the solution you'd like
In short, make Microsoft.OpenAPI trimmable for the simple scenario as described before (creating the model and serializing it).
The code that triggers this is:
public static string GetDisplayName(this Enum enumValue)
{
    var attribute = enumValue.GetAttributeOfType<DisplayAttribute>();
    return attribute == null ? enumValue.ToString() : attribute.Name;
}

public static T GetAttributeOfType<T>(this Enum enumValue) where T : Attribute
{
    var type = enumValue.GetType();
    var memInfo = type.GetMember(enumValue.ToString()).First();
    var attributes = memInfo.GetCustomAttributes<T>(false);
    return attributes.FirstOrDefault();
}

This is only used to associate a string with an enum value. My proposed solution is:
Wrap this logic into an IEnumDisplayValueStrategy/DefaultEnumDisplayValueStrategy that we can supply on the OpenApiWriterSettings. End users can implement this themselves, allowing the compiler to trim the DefaultEnumDisplayValueStrategy, thus removing the reflection code.
Describe alternatives you've considered

Instead of using a [Display] attribute, create an extension method that maps Enum to string. This would look something like this:

public enum ParameterStyle
{
    [Display("matrix")] Matrix,
    [Display("label")] Label
}

public static class ParameterStyleExtensions
{
    public static string GetDisplayName(this ParameterStyle style)
    {
        switch (style)
        {
            case ParameterStyle.Matrix:
                return "matrix";
            case ParameterStyle.Label:
                return "label";
            default:
                return style.ToString();
        }
    }
}

Use source generation to generate these methods (similar to 1. but less maintenance). Something like https://github.com/andrewlock/NetEscapades.EnumGenerators would probably suffice.

The issue with both of these solutions is that they would modify the behavior of a public method. This would be a breaking change.

If this is something that's wanted, I'd be happy to pick this up.


[TritonMP] Fix border error issue and check missing liberty

[TritonMP] Fix border error issue and check missing liberty
I temporarily used the "exit" function after the "error" messages at this moment.

On checking for liberty, you should exit when there is a missing file but rather allow the program to continue. You will see a message printed to the screen that the sta internally creates a black box with no timing arcs. This is to allow partially completed designs to run until the designer is further along in the process. OpenROAD is full automation, no human in the loop, but the user writing the RTL and assembling the design components will do so in an iterative fashion, just like we write code. Allowing the tool to continue is very helpful.

I think its best to separate the two issues into two different PR's. The tapein flow for balckparrot is stuck here. Can you make a new PR with just the macroplace error to warning change with the implicit location calculation?

The missing liberty updates were reverted.
There were three updates in this new PR:

avoid PAD classes in Macros.  (without this patch, TritonMP will waste lots of runtimes -- GF14-BlackParrot had this issue.)
borderline assign issues.
missing liberty issues.

The 1) and 2) is on this PR to enable the BlackParrot flow, and 3) has been reverted.
Could you check and merge it?


[1.x] Slack - Stringify objects in attachments

Fixes https://github.com/Seldaek/monolog/issues/1192

@Seldaek the failing tests don't appear to be related to this PR, is there something I'm missing?

Nah seems like a random failure, I retried the build and it passed.

Issue already fixed in https://github.com/Seldaek/monolog/commit/07681ea4ba6fc0cb0077110000369f638863d8e8


Don't pass synthesized node to typeToTypeNode

Fixes #19395

Note that the asserts are in a separate commit in case they turn out to be too aggressive.


openimageio: 1.8.14 -> 1.8.15

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/openimageio/versions.
meta.description for openimageio is: '"A library and tools for reading and writing images"'.
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/jyr9w69i5b04harp1s5nmfcw13fb6qxb-openimageio-1.8.15-bin/bin/iconvert had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/jyr9w69i5b04harp1s5nmfcw13fb6qxb-openimageio-1.8.15-bin/bin/idiff had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/jyr9w69i5b04harp1s5nmfcw13fb6qxb-openimageio-1.8.15-bin/bin/igrep had a zero exit code or showed the expected version
/nix/store/jyr9w69i5b04harp1s5nmfcw13fb6qxb-openimageio-1.8.15-bin/bin/iinfo passed the binary check.
/nix/store/jyr9w69i5b04harp1s5nmfcw13fb6qxb-openimageio-1.8.15-bin/bin/maketx passed the binary check.
/nix/store/jyr9w69i5b04harp1s5nmfcw13fb6qxb-openimageio-1.8.15-bin/bin/oiiotool passed the binary check.
3 of 6 passed binary check by having a zero exit code.
0 of 6 passed binary check by having the new version present in output.
found 1.8.15 with grep in /nix/store/jyr9w69i5b04harp1s5nmfcw13fb6qxb-openimageio-1.8.15-bin
directory tree listing: https://gist.github.com/e4c0d27198ddd9d4bea98163845e3fb2
du listing: https://gist.github.com/9053f016d608629151f0d9d450b3136e

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:
nix-shell -p cachix --run 'cachix use r-ryantm'

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/f8f0a36c74c15b98302ce0f5eb09de22119cfea1.tar.gz -p openimageio
ls -la /nix/store/jyr9w69i5b04harp1s5nmfcw13fb6qxb-openimageio-1.8.15-bin

cc @cillianderoiste for testing.

Failure on x86_64-darwin (full log)
Attempted: openimageio
Partial log (click to expand)
  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:3 (project)

-- Configuring incomplete, errors occurred!
See also "/tmp/nix-build-openimageio-1.8.15.drv-0/source/build/CMakeFiles/CMakeOutput.log".
See also "/tmp/nix-build-openimageio-1.8.15.drv-0/source/build/CMakeFiles/CMakeError.log".
builder for '/nix/store/1cl9yp5l3yyvd1vcrpqp4w60b9vkv0xb-openimageio-1.8.15.drv' failed with exit code 1
error: build of '/nix/store/1cl9yp5l3yyvd1vcrpqp4w60b9vkv0xb-openimageio-1.8.15.drv' failed

Success on x86_64-linux (full log)
Attempted: openimageio
Partial log (click to expand)
checking for references to /build in /nix/store/vsg0w0rg0nh8kawwxif4gh7lkmis46pc-openimageio-1.8.15...
shrinking RPATHs of ELF executables and libraries in /nix/store/xbvyb3hwdv3m5pbk7cs1d5fslqajq4gs-openimageio-1.8.15-dev
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/xbvyb3hwdv3m5pbk7cs1d5fslqajq4gs-openimageio-1.8.15-dev
checking for references to /build in /nix/store/xbvyb3hwdv3m5pbk7cs1d5fslqajq4gs-openimageio-1.8.15-dev...
shrinking RPATHs of ELF executables and libraries in /nix/store/pghp8rdb64mws1vy5ywgsp9q4ngvx8dz-openimageio-1.8.15-doc
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/pghp8rdb64mws1vy5ywgsp9q4ngvx8dz-openimageio-1.8.15-doc
checking for references to /build in /nix/store/pghp8rdb64mws1vy5ywgsp9q4ngvx8dz-openimageio-1.8.15-doc...
/nix/store/jyr9w69i5b04harp1s5nmfcw13fb6qxb-openimageio-1.8.15-bin

Success on aarch64-linux (full log)
Attempted: openimageio
Partial log (click to expand)
checking for references to /build in /nix/store/ra06phr7bmml54g7bpizszz614gbdb4f-openimageio-1.8.15...
shrinking RPATHs of ELF executables and libraries in /nix/store/xh47alawgxaxrblfd8qhf615iwmprp03-openimageio-1.8.15-dev
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/xh47alawgxaxrblfd8qhf615iwmprp03-openimageio-1.8.15-dev
checking for references to /build in /nix/store/xh47alawgxaxrblfd8qhf615iwmprp03-openimageio-1.8.15-dev...
shrinking RPATHs of ELF executables and libraries in /nix/store/n81cv2c1785c7ghxkc8chxddm0ai68z6-openimageio-1.8.15-doc
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/n81cv2c1785c7ghxkc8chxddm0ai68z6-openimageio-1.8.15-doc
checking for references to /build in /nix/store/n81cv2c1785c7ghxkc8chxddm0ai68z6-openimageio-1.8.15-doc...
/nix/store/jzmx32079bc6qriwr29s20acyzajc5pl-openimageio-1.8.15-bin


How to get simple text message

How to get simple text message? such as text from custom button and etc.
They are not command.
I am using webhook method.

Check Commands/SystemCommands/GenericmessageCommand.php you can handle it there.

You can add a custom commands directory and copy the src/Commands/SystemCommands/GenericmessageCommand.php file into it.
Then you can handle each and every incoming message the way you want to!
The method you're interested is execute().

Thanks
Maybe It is better to write it on wiki pages.


polling mode doesn't work on OSX

because it uses inotify, I guess? :/

That's right. It shouldn't be too hard to add another file watcher backend, it's already abstracted in lib/fs.ml. A patch would be welcome :)

See https://github.com/samoht/irmin-watcher as an example of portable filesystem watcher.

Thanks @samoht. If it's truly universal, should Jenga just adopt it?

it is not universal, it is targeted to fit the Irmin watcher API. Could be possible to extract something more universal from it, I know @dsheets was interested about it.

We should start a package fsnotify which only depends on inotify or fsevents depending on the platform it's on and offers a lowest common denominator interface (to start).


Add individuals to index page

We should have a link to apage that shows all individuals found in the database

Yep, and this is where you would select a custom "case"
But it could probably fit on the same page as cases, perhaps using tabs?

I can do this too in the next server update

Super, we should also think of how we can add different ways of filter the individuals. We could start with HPO. So 'give me all individuals with HPO term x'.


Does a Brew Update restart the dnscrypt-proxy service on the Mac?

I have a brew update scripted to run nightly, Not sure if brew will do the restart automatically or if I need to code  the restart.
Thanks

brew services restart dnscrypt-proxy is the command that will do that for you.


It just hangs

On macOS with python and python3 installed, pipenv hangs with this Pipefile:
[packages]
awacs = "*"
awscli = "*"
aws-shell = "*"
botocore = "*"
jetstream = "*"
mycli = "*"
sops = "*"
troposphere = "*"
troposphere-cli = "*"

[requires]
python_version = "3.6"

Here's what it shows:
$ pipenv install --three
Creating a virtualenv for this project...
⠋Running virtualenv with interpreter /usr/local/bin/python3
Using base prefix '/usr/local/Cellar/python3/3.6.0_1/Frameworks/Python.framework/Versions/3.6'
New python executable in /Users/nikolay/.local/share/virtualenvs/aws-0MMXWS-b/bin/python3.6
Also creating executable in /Users/nikolay/.local/share/virtualenvs/aws-0MMXWS-b/bin/python
Installing setuptools, pip, wheel...done.

Virtualenv location: /Users/nikolay/.local/share/virtualenvs/aws-0MMXWS-b
No package provided, installing all dependencies.
Pipfile found at /Users/nikolay/Projects/DevOps/aws/Pipfile. Considering this to be the project home.
Pipfile.lock not found, creating...
Locking [dev-packages] dependencies...
⠸Locking [packages] dependencies...

Notice the cursor leftover here as well:
⠋Running virtualenv with interpreter /usr/local/bin/python3

Hey @nikolay, thanks for taking the time to open this. Unforunately, I'm currently unable to reproduce the problem locally. Could you specify which version of mac OS you're using, and the version of pipenv as well?
The locking procedure may take a while but shouldn't be significant (longer than a minute or two) on modern hardware. How long were you leaving this task running? Does this also happen when you run pipenv lock?

My macOS version is 10.12.3 (16D32).
I canceled it after more than 30 minutes. Same packages install in a couple of minutes with: pip install -U -r requirements.txt
It happens with pip lock, too, unfortunately.

So, without being able to reproduce this on my end, all I can do is speculate at this point. My guess is either something in do_download_dependencies or get_downloads_info. Are you comfortable cloning the pipenv repo and doing some debugging here to see where the code is hanging?
If not, I'm not sure if we can proceed any further here. My only immediate hunch is the proper_case call to https://pypi.org/simple/ not timing out. We should probably add a timeout here, but I'd like to verify where the failure is happening before we start making changes.

Hey @nikolay, since I'm unable to reproduce this, and we don't have any meaningful way to move forward here, I'm going to close this out. Please feel free to reopen this if you have further details or are able to pinpoint where things are hanging. Thanks!

I've noticed this happening as well. Here's how I reproduced and investigated it:
In cli.py I added the following:
import faulthandler
import os
faulthandler.register(14)  # 14 == SIGALARM
print("kill -s ALRM {}".format(os.getpid()))

Then I ran pipenv lock --verbose and the last few lines it printed before hanging indefinitely were these:
  numpy==1.13.3             requires numpy==1.13.3
  incremental==17.5.0       requires incremental==17.5.0
  werkzeug==0.12.2          requires Werkzeug==0.12.2
------------------------------------------------------------
Result of round 5: stable, done
^C
Aborted!

It seemed to have completed but that ^C is me having to CTRL-C the process and kill it.
I ran pipenv lock --verbose again and waited until it got to that point and then ran kill -s ALRM {the pid} and this printed:
------------------------------------------------------------
Result of round 5: stable, done
Thread 0x000070000ab73000 (most recent call first):
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/threading.py", line 293 in wait
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/queue.py", line 164 in get
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/concurrent/futures/thread.py", line 64 in _worker
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/threading.py", line 862 in run
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/threading.py", line 914 in _bootstrap_inner
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/threading.py", line 882 in _bootstrap

Thread 0x000070000a670000 (most recent call first):
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/threading.py", line 293 in wait
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/queue.py", line 164 in get
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/concurrent/futures/thread.py", line 64 in _worker
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/threading.py", line 862 in run
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/threading.py", line 914 in _bootstrap_inner
  File "/Users/jackdanger/.pyenv/versions/3.5.3/lib/python3.5/threading.py", line 882 in _bootstrap

Current thread 0x00007fffce4233c0 (most recent call first):
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/urllib3/util/connection.py", line 73 in create_connection
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/urllib3/connection.py", line 141 in _new_conn
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/urllib3/connection.py", line 284 in connect
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/urllib3/connectionpool.py", line 850 in _validate_conn
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/urllib3/connectionpool.py", line 346 in _make_request
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/urllib3/connectionpool.py", line 601 in urlopen
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/requests/adapters.py", line 440 in send
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/requests/sessions.py", line 618 in send
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/requests/sessions.py", line 508 in request
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/requests/sessions.py", line 521 in get
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/pipenv/utils.py", line 481 in resolve_deps
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/pipenv/cli.py", line 1083 in do_lock
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/pipenv/cli.py", line 1974 in lock
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/pipenv/vendor/click/core.py", line 535 in invoke
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/pipenv/vendor/click/core.py", line 895 in invoke
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/pipenv/vendor/click/core.py", line 1066 in invoke
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/pipenv/vendor/click/core.py", line 697 in main
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/lib/python3.5/site-packages/pipenv/vendor/click/core.py", line 722 in __call__
  File "/Users/jackdanger/.virtualenvs/tensorflow-1.0-DYoUwvq2/bin/pipenv", line 11 in <module>

The last line of pipenv before going into the requests library is utils.py:481 in the resolve_deps method which looks like this:
478            if 'python.org' in '|'.join([source['url'] for source in sources]):
479                try:
480                    # Grab the hashes from the new warehouse API.
481       ------>      r = requests.get('https://pypi.org/pypi/{0}/json'.format(name))
482                 api_releases = r.json()['releases']
483
484                 cleaned_releases = {}

That requests.get is likely hanging indefinitely so I changed it to requests.get('https://pypi.org/pypi/{0}/json'.format(name), timeout=5 and, while it definitely hung for a little while, the process did finish correctly the next time through:
------------------------------------------------------------
Result of round 5: stable, done
Updated Pipfile.lock (9a1095)!

I've opened #885 to fix this with a (more generous) 10-second timeout

FYI - in my case it was hanging because Pipfile.lock was read-only. Before blaming pipenv, check whether it's possible to write something to that file. Bad I realized that after hours spent trying to figure out what the heck is going on. Cheers 🍾


test: show error on inserting an inline embeddable as null

Fixed with https://github.com/mikro-orm/mikro-orm/commit/6e0bedf5610b6e423605d250ed2ac1a15a63381e


DOCS] Add icons for connectors in pmlist doc

Feature Description
https://hyperswitch.io/pm-list
Need to add icons for all connectors here in the pmlist doc,which have a default icon at the moment.
Possible Implementation
Need to add icons for all connectors here in the pmlist doc,which have a default icon at the moment.
Have you spent some time checking if this feature request has been raised before?

[X] I checked and didn't find a similar issue

Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to submit a PR?
No, but I'm happy to collaborate on a PR with someone else

It is approved, you can start working on it @Sakilmostak


Change: #959 ditch async_trait for the singlethreade feature

Issue #959.
This PR substitutes the existing async_trait macro usage with the async_fn_in_trait language feature which is generally available in Rust 1.75.0 if the singlethreaded feature is turned on.
With this change, in order to use the singlethreaded feature, the minimum supported Rust version is bumped to 1.75.0.
Checklist

[ ] Updated guide with pertinent info (may not always apply).
[ ] Squash down commits to one or two logical commits which clearly describe the work you've done.
[ ] Unittest is a friend:)

This change is

@schreter


How to wait for browser event?

Hi,
This is a question not an issue, so if it's not the right place to post this, sorry.
I'm trying out this piece of code, but I can't figure out how to "wait" for something from the browser.
For example:
I want to go to a url, load a page and wait for something in the console or for an event.
The url is opening, but the nodejs code is immediately closed(which makes sense).
Page.navigate({'url': 'http://mypage.com'});
// how to wait?

This is your sample code:
var Chrome = require('chrome-remote-interface');
Chrome(function (chrome) {
    with (chrome) {
        Network.requestWillBeSent(function (params) {
            console.log(params.request.url);
        });
        Page.loadEventFired(close);
        Network.enable();
        Page.enable();
        once('ready', function () {
            Page.navigate({'url': 'https://github.com'});
            //how to wait for a console message or event?
        });
    }
}).on('error', function () {
    console.error('Cannot connect to Chrome');
});

Thanks in advance!

NodeJS closes once chrome.close() is called. In the above example this happens when the load event is fired by Chrome due to this line:
Page.loadEventFired(close);

All you have to do is remove that line and call chrome.close() once your event is fired.
For example, say you want to exit once a certain word ("png") appears in a requested URL:
Network.requestWillBeSent(function (params) {
    console.log(params.request.url);
    if (params.request.url.match(/png/)) {
        close();
    }
});
// Page.loadEventFired(close);

Another example, here the program exit 5 seconds after the page load event is fired:
Page.loadEventFired(function () {
    setTimeout(function () {
        close();
    }, 5000);
});

If instead you just want to wait a specific time since the beginning:
// Page.loadEventFired(close);
once('ready', function () {
    Page.navigate({'url': 'https://github.com'});
    setTimeout(function () {
        close();
    }, 5000);
});

I'm showing just the relevant bits, but it should be clear nonetheless. Otherwise please do not hesitate to ask.

Thank you for your quick and comprehensive answer! I really missed this call:
Page.loadEventFired(close);

You're welcome!


Unclear what is the order of rules for write_relabel_configs

What did you do?
I tried limit number of metrics sent via remote_write. As per documentation of remote_write pointing to relabel_config one can define multiple rules e.g. drop and keep. Unfortunately it is unclear in which order the rules are executed. Given drop and keep rules that affect the same label it is unclear which rule wins. For example in the case below I would expect that first rule drops everything and next rules allow specific labels but it is not working and I don't know if this is due to the fact that that drop rule wins or due to the wrong order of rules.
  write_relabel_configs:
    - source_labels: ['__name__']
      regex: '.+'
      action: 'drop'
    - source_labels: ['__name__']
      regex: 'container_cpu_usage_seconds_total'
      action: 'keep'

What did you expect to see?
The documentation should be clear about what happens when there are multiple rules with different kinds and which wins. Additionally, I think the most common use case is that everything is blocked and then specific metrics are allowed and it would be nice to have such example in the documentation.
What did you see instead? Under which circumstances?
It is unclear what is the Prometheus behavior in case of multiple relabel rules affecting same target.
System information
No response
Prometheus version
version=2.31.1, branch=HEAD, revision=411021ada9ab41095923

Prometheus configuration file
No response
Alertmanager version
No response
Alertmanager configuration file
No response
Logs
No response

The rules are a list, I think it is clear that they are executed sequentially, from top to bottom. The kind of rule is not important and does not affect order. Only the position in the list matters.

@roidelapluie Thanks for clarification. Then, all metrics except container_cpu_usage_seconds_total should be dropped in my case, but the actual behavior is that everything is dropped anyway.

    - source_labels: ['__name__']
      regex: '.+'
      action: 'drop'

After this rule, everything is dropped.
You just need the second rule:
    - source_labels: ['__name__']
      regex: 'container_cpu_usage_seconds_total'
      action: 'keep'

Yeah, that's correct. Only the second rule is sufficient. Thanks.


⚠ Bofrost.lu has degraded performance

In 8f6ad76, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1376 ms

Resolved: Bofrost.lu performance has improved in 967e84d after 42 minutes.


fix: tests

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3928   👈

This comment was auto-generated by Graphite.


Add comments and format code

Left all functionality the same, added some comments and formatted the code.

Good job! Do you want me to merge this?
Also, what happened to the project.godot file. Why is it looking completely different from the original one?

I think that's a result of upgrading to 4.1, and those files auto-updated. It'll probably change again once upgraded to 4.2.

If it looks good to you, yes please merge it :)


May I exclude nuxt-swiper global css? #169

I don't want to add (swiper.css as global css) to my project since it overwrite my projects scoped style.

any update for that cuz I want to add css styles in my project manually not import all styles to all pages! this is seo loading page problem guys!

Swiper and SwiperSlide are global components. They add global styles. Disabling global components may solve the problem

Please try v2 which should solve this case -> https://github.com/cpreston321/nuxt-swiper/pull/109
Docs: https://github.com/cpreston321/nuxt-swiper/tree/feat/elements?tab=readme-ov-file#nuxt-swiper
Then within the the nuxt config:
swiper: {
 bundled: false
}

Thanks,
CP


Missing or incomplete author information from most classes

Is this the class docblocks?

Yeh.

Do you want to list all three of us on each class?

No.

That's the issue. We need to work out who's actually authored what.

I'm on it.

A simple git blame won't cut it because files have been moved about. :/

Then we'll have to do it going forward?

D'no. Maybe we should just delete any existing ones we have?

Just stay without?

We're going to start adding them going forward.


Kotlin swift upgrade

With the native portion of the plugin this small, the Swift/Kotlin maintenance is really overkill.
@YoussefKababe  If you're interested, I can prepare a Java/ObjC port in the future..

@YoussefKababe any update or opinion on the Java port/

Currently running into issues on Android related to Kotlin version. Any update on this?
@ened I completely agree. Plugins should stay on Java for sure with much Kotlin changes. ObjC also helps.


Inclusion of new paper

👍 Thank you for creating and maintaining such a great repository. I found that these works have not been included and hope they can be added.

Title
Link
Code
Venue
Classification
Model
Comment

Formalizing and Benchmarking Prompt Injection Attacks and Defenses
arxiv
github
USENIX'24
A7. Prompt Injection
LLM
Benchmark

Hi Junjie!
We have added this great paper. Thanks for your recommendation! 😄


Redmine 3.0.1 Container will not start after power failure

Created by :  docker run --name=xxxxredmine -h xxxxredmine -d -p <IP_ADDRESS>:80:80 -p <IP_ADDRESS>:443:443 -p <IP_ADDRESS>:22:22 -v /var/run/docker.sock:/run/docker.sock -v $(which docker):/bin/docker sameersbn/redmine:3.0.1
Upgraded from the previous 2.6 version.  Everything upgraded great and server was running well.
Then I accidentally powered off my Ubuntu Docker host (id10T).  Now the container will not start and exits within 1 second.  Starting this container does start the database.
root@xxxx:~# docker start -i 7e07bd263da2
Database connection details not specified.
Will try to spin up a new postgresql image with the name postgresql-redmine.
Please manually configure the database connection in production.
Using existing postgresql container...
Waiting for database server to accept connections
Migrating database. Please be patient, this could take a while...
/var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/resolver.rb:333:in block in verify_gemfile_dependencies_are_found!': Could not find gem 'rails (= 4.2.0) ruby' in the gems available on this machine. (Bundler::GemNotFound) 	from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/resolver.rb:307:in each'
from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/resolver.rb:307:in verify_gemfile_dependencies_are_found!' 	from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/resolver.rb:199:in start'
from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/resolver.rb:182:in resolve' 	from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/definition.rb:193:in resolve'
from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/definition.rb:132:in specs' 	from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/definition.rb:177:in specs_for'
from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/definition.rb:166:in requested_specs' 	from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/environment.rb:18:in requested_specs'
from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/runtime.rb:13:in setup' 	from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler.rb:122:in setup'
from /var/lib/gems/2.1.0/gems/bundler-1.9.1/lib/bundler/setup.rb:18:in <top (required)>' 	from /usr/lib/ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in require'
from /usr/lib/ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in `require'

Able to copy my data folder out of the stopped container
docker cp 7e07bd263da2:/home/redmine/data /root/redminedata
Going to reinstall.

@bignay2000 You have used the quickstart guide to start the container, which goes and start a postgresql container for you. Note this is only for testing purposes. You should manually (or using fig/docker-compose) configure the image and its requirements as specified in the README. This includes mounting data volumes for the redmine as well as the mysql or postgresql container.
Mounting volumes for your container ensures that there is no data loss when the container(s) is stopped/deleted/started because all the data is left intact for you to recreate the container.
The quickstart guide is only present to allow new users to quickly take a look at what redmine offers. It is not intended or tested to be used on production. Production installs require that you read the README beyond the quick start guide.
As far as the error above goes, I myself have never tried using the quickstart guide to start the container and then stop them and start again. Whatever the cause of the error, it would be difficult to give you instructions on how to recover from it.

@bignay2000 i just tested starting the redmine container as per the quick start guide. Then stopped the redmine and redmine-postgresql containers and started the redmine container again using docker start redmine and everything started up fine and with no errors.
Maybe you had some data loss/corruption when the host pc was powered off.

This issue was caused by me.  When I upgraded from Redmine 2.6, I moved the entire /home/redmine/data directory to the new Redmine 3.0.1 container's /home/redmine/data.  This ran great until Redmine was stopped and then tried to start with a data/tmp folder from the older (2.6) version.
Reinstalled 3.0.1 and only copied the /home/redmine/data/files directory to the new container (dotfiles was empty not sure what this directory is for and omitted it too).  Container can now be stopped and started.  Also confirmed attachments are working.
Awesome.   A BIG THANK YOU to sameersbn for responding quickly to this issue.


on the backend when filtering by content_text ILIKE  👉🏾  we should instead use duckdb for filtering

not doing it for now

Tested with apache solr, meilisearch and quickwit, not good results.


Specify the 'downloads' directory

I added a way to specify the folder to place downloaded files.
You can share a "downloads" folder with multiple windows installer build instances
without copying from one instance to another.
rake DOWNLOADS="c:\downloads" # specify the folder to place downloaded files, absolute/relative
In addition above, I added the ability to handle dependency files
more properly in multi-toolchain.
Download a set of dependency files from distinguished URL path
and store them under the "downloads" sub-directory separately.
config/compilers/mingw64.rb is just a sample.
You'll fail in downloading the dependency files, since there aren't such files.
:knap_path value is used for separation.

It looks promising!

I'm in the middle of getting some other stuff together, so give me a couple days and I'll look at it more closely. =)

may need updating the software versions.


Add company to lists like in optional-list.txt

In optional-list.txt you add the site company and that give a much better overview then in referral-sites.txt & whitelist.txt.
It would be nice if you can create the lists in the better format so we know which sites are for that domain

@beerisgood take a look at this PR


Encountering Strange Behavior when upgrading runn

Hello! Thanks for this awesome software !
This is a very rare case and we may not have to take care of it, but I would like to report that when I update runn, the behavior was different and the error message reported strange lines.
In versions up to v0.92, example.yml passes, but in v0.93 and later, an error occurs.
Perhaps the backquote on line 7 is recognized as an invalid string, but the error message says line 4 is invalid.
This discrepancy is probably caused by the three lines of comments.
example.yml
desc: Example
# comment
# comment
# comment
steps:
  hello:
    desc: '`echo` hello' # ⇦ Single quote followed by backquote, line 7
    dump: "'Hello world!'"

version 0.100 (error)
➜  runn git:(main) runn --version
runn version 0.100.0
➜  runn git:(main) runn run example.yml
Error: failed to load runbook example.yml: yaml: line 4: found character that cannot start any token

version0.92 (pass)
➜  runn git:(main) runn --version
runn version 0.92.0
➜  runn git:(main) ✗ runn run example.yml
Hello world!
.

1 scenario, 0 skipped, 0 failures

@okazaki-kk Thank you for your report!
I will see which is the correct behavior.

The behavior of v0.92.0 was found to be correct.
We are currently finding the cause and fixing it.

We will be able to fix this when https://github.com/goccy/go-yaml/pull/440 is merged and released.

@okazaki-kk
Fixed at v0.104.0

Fixed
https://github.com/goccy/go-yaml/releases/tag/v1.12.0


Replace std::result_of with std::invoke_result

C++17 deprecates std::result_of<T> and C++20 completely removes it from the spec in favor of std::invoke_result<T>, preventing compilation of g3log if using C++20-level features.
This is the simplest fix which merely replaces all instances of the former with the latter, but it will break compilation on C++14 or older. It would also be possible to make an alias of these templates and use macro compile flags to preserve compatibility with C++14, but I wasn't sure if that would be necessary.
I also took the liberty to use the *_t declarations which are much less verbose.

Thanks. Before I merge this I need to creat a branch and release for “last release with C++14 support”.
On your end, please update the CI configuration to work with your changes. Once the tests are working I’ll review this PR

Any idea when this can be merged?

Thanks for the reminder. I’ve planned to create the release w/tagging and review and this this PR this ongoing week

I have confirmed this fixed #337.


Pytest

Hi @LinxuanHu ,
Do we need to do pytest ?? Or only linting is enough?

I believe pytest is for unit test, right? in that case, yes, but you can confirm from instructor.

And, the raw codes are merged I guess, you can go ahead for your task 3. I still need to polish my code a bit but I guess the unit test would be fine at this stage. @LinxuanHu

Hi @LinxuanHu , Do we need to do pytest ?? Or only linting is enough?

We need to do both. They are different things.

Okay, thank you for clarification.

yeah, i did both, you can check that


Is there a change log somewhere for the project?

Hi, I'm wondering if there is a change log to the repository maintained somewhere? It would be helpful to me as I'm deciding on whether to update from 1.2.1 to 1.2.4 while having several projects developed within 1.2.1. Thanks.

see https://github.com/tensorflow/tensor2tensor/releases
and for more details #325, #318, #290 etc. (see closed PR).