fix(deps): pin mkdirp due to issues

See https://github.com/isaacs/node-mkdirp/issues/42.

Coverage: 100.0%. Remained the same when pulling f2ba0663787ce853f43856f957a2cca05f1c2ffa on fix into 06030130a6705e84d8d41d63440a6c67f0825719 on main.

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Recursive archiving with a temporary account and issues with downloading WARCs on local deployment

Being unable to create an account locally #616 , I was trying to use temporary accounts and download all warcs before the session expires.
The URL I've tested it was this, where I right-clicked on the images at the side (except the videos) so that they archive on another tab.

The thing is, those archives, that I've right-clicked seem to come from another port :8092 which seems to be a newer version of webrecorder.

but when hit stop on all recordings, not all archived pages show up

On the :8092-side, they don't appear either.

While I'm able to download a part of the warc on :8089,

clicking the download button on :8092 seem to just refresh the page and no download came.
I've set up webrecorder locally, just as advised here https://github.com/webrecorder/webrecorder/issues/614 , where the webrecorder/pywb:develop-image seems to be functioning.
These are the docker logs from that test:
webrecorder_app_1-templog.txt
webrecorder_recorder_1-templog.txt
webrecorder_warcserver_1-templog.txt
webrecorder_redis_1-templog.txt
webrecorder_frontend_1-templog.txt
webrecorder_nginx_1-templog.txt
Does it have to do with the account logic #616 or just being on the development branch?

@ikreymer actually, the UI at :8092 looks newer. How is it considered old?

Should this issue be closed, or be left open for feedback until it's fully resolved?


feat: add C implementation for math/base/special/acosdf

Resolves #649.
Description

What is the purpose of this pull request?

This pull request:

Resolves a part of #649 by adding the C implementation for math/base/special/acosdf.

Related Issues

Does this pull request have any related issues?

This pull request:

resolves part of #649

Questions

Any questions for reviewers of this pull request?

Open to reviews!!
Other

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

No.
Checklist

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

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

@stdlib-js/reviewers

@gunjjoshi would love your inputs on this, is it okay if i had to increase the tolerance a bit to accommodate single-precision floating point numbers?

@gunjjoshi would love your inputs on this, is it okay if i had to increase the tolerance in the tests a bit to accommodate single-precision floating point numbers?

I think the tolerance levels that we have here, i.e., 2.8 * EPS should be fine, as compared to the 1.4 * EPS of the double-precision equivalent, as we can expect a slight precision loss when we transition from double to single-precision floating-point numbers.

that's what i thought while increasing the tolerance, do you think there's any other stuff that needs to be done in order to merge this @gunjjoshi , would love to work on this

I've made all the required changes from code review, thank you for your time @gunjjoshi , @kgryte .

added the changes @gunjjoshi

@gunjjoshi Would you mind doing one more pass over this PR?

Only a single change needed, looks good otherwise.

is there anything that is left to do here @gunjjoshi ? , would be happy to implement


generate.py now requires an extra variable that should not be required

Describe the bug
https://github.com/Azure/azure-cli-extensions/pull/6507/files
This pr added a variable, dependent file, which was not present before. Since this is a required variable, it is failing to run the script and breaking our builds.
I can see why this would be needed for some wheels and extensions but it is not needed for others. I can get around this by adding -d None to the command but this should not be necessary.
Related command
scripts\refdoc\generate.py -e path_to_wheel
Errors
generate.py: error: the following arguments are required: -d/--dependent-file
Issue script & Debug output
generate.py: error: the following arguments are required: -d/--dependent-file
Expected behavior
-d/--dependent-file should not be required.
Environment Summary
azure-cli                         2.50.0
core                              2.50.0
telemetry                          1.0.8
Extensions:
azure-iot                         0.21.4
Dependencies:
msal                              1.22.0
azure-mgmt-resource             23.1.0b2
Python location 'C:\repo\venv38\Scripts\python.exe'
Extensions directory 'C:\repo\extensions'
Python (Windows) 3.8.3 (tags/v3.8.3:6f8c832, May 13 2020, 22:37:02) [MSC v.1924 64 bit (AMD64)]
Legal docs and information: aka.ms/AzureCliLegal
Your CLI is up-to-date.
Additional context
No response

Thank you for opening this issue, we will look into it.

Hi @vilit1,
I didn't anticipate that other users would directly use our CI scripts.
I will create a pull request to fix it.


Provider Bug in v0.52.2 - GetDeviceKey invoked with null NodeId

Summary
We upgraded to the v0.52.2 provider (from v0.50.1) and encountered a Provider Bug when running our pipelines again for validation. Our codebase had no changes between upgrading the provider.
The complete error message is below. It has to do with the apstra_datacenter_device_allocation resource. Something is up with the device_key attribute.
In our code line device_key = each.value.device_key == "" ? null : each.value.device_key # optional, the each.value.device_key attributes have values and are not equal to "".
Please take a look in the provider for what might be triggering this error.
Details
Error Message
│ Error: Provider Bug. Please report this issue to the provider maintainers.
│
│   with apstra_datacenter_device_allocation.device_allocation["PF-432"],
│   on resource.tf line 83, in resource "apstra_datacenter_device_allocation" "device_allocation":
│   83: resource apstra_datacenter_device_allocation device_allocation {
│
│ GetDeviceKey invoked with null NodeId

Code
resource apstra_datacenter_device_allocation device_allocation {
    for_each                 = var.device_allocations
    blueprint_id             = data.apstra_datacenter_blueprint.blueprint.id
    node_name                = each.value.name
    device_key               = each.value.device_key == "" ? null : each.value.device_key # optional
    deploy_mode              = each.value.deploy_mode
    initial_interface_map_id = one([for imap in data.terraform_remote_state.common_cluster.outputs.interface_maps: imap.id if (each.value.initial_interface_map_name == imap.name)])
    depends_on   = [
        null_resource.configure_device_properties
    ]
}

Thanks, Michael.
We'll look into it.

I think I found the problem here: 0c1fac8fbc33684f7a30cc549f42998df36d6211
I renamed a computed-only attribute from node_id to id because I thought the terraform plugin framework acceptance tests required an id field (this resource was developed before we were using the formal acceptance testing system.
Couple of things about that:

Willy-nilly renaming of attributes (even ones the users don't directly interact with) isn't that great of an idea, especially if you've written the code to error when an expected field is missing.
The newest release of the testing framework doesn't require an id field anymore


Support for Nut

It would be great if this buildpack supported Nut, a pretty popular dependency manager for Go. I'm working on this on a fork, but I'm curious if anyone else is interested in having this added to the buildpack.

:+1:

@bithavoc I've gotten it to work somewhat, but I've pretty much broken the buildpack's normal functionality to do it. Can you try forking and seeing if you can make any progress?

Personally (not speaking for my position @ Heroku) I'd much rather spend time adding support for vendor-spec, which all of these tools should support, especially as Go 1.6 approaches. I am working towards that with godep atm (albeit slowly).
With that said (putting my Heroku hat on). Is there sufficient interest in supporting NUT? The author of which also works @ Heroku. If you want support for https://github.com/jingweno/nut please +1 this issue.

:+1:

@jingweno would you be interested in seeing support for this with Heroku?

I :+1: to this. But that may require changes to nut to declare Go version etc. that the buildpack needs.

I second @freeformz on this.

@bithavoc second on vendor support or Nut?

@jadengore IMO Nut + godep + buildpack supporting the vendor spec is the best outcome, but I understand the argument for interim support of Nut though, as long as nut provides all of the information necessary to compile the code reproducibly (like go version).

@freeformz I have Nut working specifically for the app I'm working on, but my fork has significantly diverged and I don't have too much shell scripting experience to know if this is ready for public release.
What I plan to do is iterate on my work and have a PR for discussion/amendments so that maybe we can get this merged in with Nut support.

@jadengore SGTM Having something to review is probably best.

Can Nut not support the vendor-spec instead?

We're not going to be supporting nut, in fact I'd like to reduce the number of supported tools. Hopefully dep and the other plans around future dependency management stuff in go 1.10/11/etc will make that a reality.


Features/dropqueuedpackets

This pull request adds sock_queue_timeout configuration options which stores the amount of seconds spend in the socket queue after which the packet is considered old and should be dropped. The reason to drop is because client won't be waiting for the reply anymore and there is no reason to process the query.
There are also 2 metrics added:

num.queries_timed_out to store the amount of dropped packets
query.queue_time_us.max to collect the max time that was found for packets to wait in the socket queue (this can be easily converted to histogram if needed)

Yep, sure, you can put statements, there is no new code in these files, it's just copy-paste and cleanup.
Let me rebase and update the PR with the suggestions applied

With the adjustments, I'll approve and merge this. Thank you for the additional feature and cleanup!
After that, I will likely make a couple of smaller changes; spelling error, documentation mentions, and Changelog entry, to adjust the code a little. Thanks for the header license change, that is what I wanted.


Add Settings

WIP

Thanks to @TurboMarshmello the settings stuff actually works! Styling, features and more!
All we need to do now is actually make the settings save...

Finally! It has been completed


Update chemfiles to 0.7

This release was hard to get out, but now I am back working on Lumol!

Really nice work! Looking forward to keep using chemfiles in my projects.

Really nice work! Looking forward to keep using chemfiles in my projects.

You should be able to update with conda now, and use the fixed PDB writter!


Added detail slider + sketch-prompt slider

Resolves #27 and #28
short Description:
Added working sliders, changing values in js script, so we can use it later to set values for the python script and backend
Changes:

added Slidebar for Details from 0% to 100%

if the slidebar value is changed, a value in the js script is changed (detailValue)

same for Slidebar for Sketch-Prompt weigths

changed value: spValue

changed slidebar style according to the current Application style (red-themed)

Please review and let me know, if you find any bugs :)

Tested your code locally and works fine. Could not find any issues.
One thing that could be done is a description for the sliders when hovering over them (like how we did with the brush sizes for example). But for now I think you can merge it

Thanks for your review! I added following changes:

added hover descriptions of each slider
added side description of each slider
added sValue and pValue instead of inly spValue (extra value for prompt and sketch, for py file compatibility)


Tap performance followup, fixes and specs

to: @majapw @lencioni
Bug in my previous isSameDay change. Moment's .day() unfortunately returns day of week, not day of month like I thought. I didn't realize the PR was going to be merged (was adding specs for the modifiers changes, starting to add specs for isSameDay when I realized the error).

Coverage remained the same at 87.322% when pulling 0d0c702c0add583e233b7d79103c87a43578250f on moonboots:jack-tap-perf-fixes-specs into fafa451f4e5bf2a3ebcc9f53852098616e67f4eb on airbnb:master.

Thanks, will do


IndexError when running Examples

I tried running two official examples without any changing, both of them show the IndexError, for example, cifar10_cnn:

IndexError                                Traceback (most recent call last)
<ipython-input-1-52a55e5c76bc> in <module>()
     67 # let's train the model using SGD + momentum (how original).
     68 sgd = SGD(lr=0.01, decay=1e-6, momentum=0.9, nesterov=True)
---> 69 model.compile(loss='categorical_crossentropy', optimizer=sgd)
     70
     71 X_train = X_train.astype("float32")

.......

/home/cics/Theano/theano/tensor/nnet/abstract_conv2d.pyc in get_conv_output_shape(image_shape, kernel_shape, border_mode, subsample)
     53         out_shp = tuple(get_conv_shape_1axis(
     54             imshp[i], kshp[i], border_mode, subsample[i])
---> 55             for i in range(len(subsample)))
     56     return (bsize, nkern) + out_shp
     57

/home/cics/Theano/theano/tensor/nnet/abstract_conv2d.pyc in <genexpr>((i,))
     53         out_shp = tuple(get_conv_shape_1axis(
     54             imshp[i], kshp[i], border_mode, subsample[i])
---> 55             for i in range(len(subsample)))
     56     return (bsize, nkern) + out_shp
     57

IndexError: list index out of range

Just tried the cifar10_cnn example and it worked. Do you have the latest Theano and keras versions?

@rpinsler Yes, I have reinstalled several times. It's still there.

I assume you use the Theano version from github, not PyPI (sudo pip install git+git://github.com/Theano/Theano.git). Then maybe it's a problem with one of the dependencies, e.g. numpy. For some features, Theano requires numpy 1.8.

@rpinsler Look at the last track,
/home/cics/git/Theano/theano/tensor/nnet/abstract_conv2d.py in <genexpr>((i,))
     54         out_shp = tuple(get_conv_shape_1axis(
     55             imshp[i], kshp[i], border_mode, subsample[i])
---> 56             for i in range(len(subsample)))
     57     return (bsize, nkern) + out_shp
     58

"i" has value 0 and 1, but "imshp" is a "TensorVariable" not given value, so "pimshp[0"] is invalid.
Do you know what could cause it?

I had the same problem. I reinstalled theano from commit f52b6e421b78a70c7d341358311268f0dc5a65a0 and now the example is working.

there was a bug with Theano that was causing this, it should be fixed at HEAD now.

I have the same problem, i try to upgrade keras, but didn't work???
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/nn_ops.py", line 630, in convolution
input_channels_dim = input.get_shape()[num_spatial_dims + 1]
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/tensor_shape.py", line 502, in getitem
return self._dims[key]
IndexError: list index out of range

I have the same problem when I used Keras. could anyone please help me?
File "/home/hitech/anaconda3/envs/dl_env/lib/python3.6/site-packages/tensorflow/python/framework/tensor_shape.py", line 524, in getitem
return self._dims[key]
IndexError: list index out of range


Swift 3.0 Preview 4 support

Update

C7 0.11
Swift 3.0 Preview 4

Seems like macOS fail is not related to the code, merging. Thanks!


[Feature Request] use upnp for jit enabling

I've found a program that uses the jitterbug pair file system and utilizes upnp to sign altstore to the idevice. I am wondering if this can be utilized for non-paid developers to do on-device debugger attaching via libimobiledevice library jitterbug uses.
the only problem with this is that upnp is not the same on all routers and some routers have it disabled. For example, mine allows upnp but only permanent upnp requests and this repo uses timed upnp. from what I hear tho is you can set the time limit to 0 and it would work.
repo in question that could be utilized maybe: https://github.com/NyaMisty/AltServer-Linux

The point of Jitterbug is if you don’t have access to a PC or Mac. If you do, you should use AltStore or AltJIT.

@osy I believe @Spidy123222 is suggesting using this technique so that Jitterbug only requires a UPNP enabled router and the iPhone is connected to it for it to enable AltJIT. Unless I'm misunderstanding something?

The point of Jitterbug is if you don’t have access to a PC or Mac. If you do, you should use AltStore or AltJIT.

Thing is this can be utilized in other networks not in your home network is what im saying. Also, not everyone has a pc on all the time and AltJIT is not reliable on windows at all at the moment (especially since Apple mobile device service dies within 10 minutes of activation or use). I've also encountered people where they use a friend's computer to install said jitterbug and then go home to use the app. From what i look up upnp doesn't require a paid developer account could be wrong though.

@osy I actually wrote a little bit about this back in July here on the AltStore Roadmap: https://trello.com/c/LrsvqUfo/183-feature-implement-upnp-support-directly-in-altstore-to-bypass-lockdownd-and-install-w-o-altserver

Is this what you're proposing?

Use UPnP to map port 62078 on your iOS device to the router (say it's <IP_ADDRESS>)
Now try to connect to your router's WAN (public) IP from your iOS device (e.x: <IP_ADDRESS> <-> <IP_ADDRESS>)
<IP_ADDRESS> reaches your router whose NAT entry for port 62078 is <IP_ADDRESS>
You've now "looped back" to your own device's lockdownd using your router's NAT.

This is only useful if you have UPnP enabled. It's my understanding the most routers have it disabled by default now so I'm not sure how many people will get use out of it.

Is this what you're proposing?

Use UPnP to map port 62078 on your iOS device to the router (say it's <IP_ADDRESS>)
Now try to connect to your router's WAN (public) IP from your iOS device (e.x: <IP_ADDRESS> <-> <IP_ADDRESS>)
<IP_ADDRESS> reaches your router whose NAT entry for port 62078 is <IP_ADDRESS>
You've now "looped back" to your own device's lockdownd using your router's NAT.

that is pretty much what i was saying. somenew routers have it disabled by default on some but pre-existing ones dont. It is really dependent on what the router settings are but is an option for non-paid developers if possible. UPnP port mapping is still enabled by default in new devices. Gaming consoles still use UPnP to host games if they rely on it that way but less often. For security reasons it is still recommended disabling upnp but isnt required.
https://www.howtogeek.com/122487/htg-explains-is-upnp-a-security-risk/
https://www.upguard.com/blog/what-is-upnp

using UPnP you would need to compensate for different types of UPnP routers use like permanent UPnP and would need to remove the port map when done. Temporary/timed is simple enough though from what I know since it removes itself.

This is only useful if you have UPnP enabled. It's my understanding the most routers have it disabled by default now so I'm not sure how many people will get use out of it.

@osy The most popular purchased router the Archer A7 has it on by default. But even if it's off by default, do you see any value in allowing your users to activate JIT with only a single device simply by toggling "UPNP On" in their router? Because it's proven, it works (as a PoC at least, you're one of the few with the skills to make it more than a "proof of concept" tho).


1117-setup_mocks_with_msw

Changes

Setting up MSW with Next.js
Replacing Cypress fixtures by MSW handlers

Context

Closes #1117
Tests

I've checked my work by:

[X] Adding new tests or adjusting existing tests
[X] Testing manually

Some docs changes.

Thanks! Suggestions applied.


Add MongodDB project

projection allows for specific fields to be included or omitted in the response. Adding projection will allow for users to get just the data that they will be using, reducing bandwidth necessary. To use it, project can be added to the contracts.find and contracts.findOne calls within the query as an optional parameter(as well as internal calls). The default behavior is to return all fields.
rpcConfig.allowArbitraryProject is added to the config, with a default value of false. If set to true, it allows project values to be whatever(can have calculations done on result before being sent back to client), but to not overwhelm public nodes, the default configuration just allows 1/0 values(ability to include/exclude fields).
Example call:
curl --location 'https://engine.rishipanthee.com' \
--header 'Content-Type: application/json' \
--data '{
  "jsonrpc": "2.0",
  "method": "contracts.find",
  "params": {"contract" : "nft", "table" : "STARinstances", "query" : {}, "project" : {"account": 1}},
  "id": 2
}'

How would you say is the best way to expose/test it to the SC api? Add projection to a contract and use it there is what I'm thinking, but if you have a better idea, I'll look in that direction.

How would you say is the best way to expose/test it to the SC api? Add projection to a contract and use it there is what I'm thinking, but if you have a better idea, I'll look in that direction.

This sounds like a reasonable approach.  What you could do is create a dummy test contract with one action that specifically utilizes this new feature and logs something on success.  Then have a unit test case that triggers the action and checks for the expected result in the contract logs.

@Rishi556 any updates on exposing this functionality to the smart contracts API as well as RPC?  I don't consider doing that very important, though, as we don't have any immediate planned smart contract uses for it.  So if you'd like to consider this task done for now, and hold off on smart contracts API support for another time, I'm fine with that.  Please confirm either way.

I tried writing a dummy contract with a test at one point, but it ended up being a bit messy and I haven't looked at it again since then. I think we can expose it to the smart contracts if its needed at another point, especially since nothing is using it as of now.

I tried writing a dummy contract with a test at one point, but it ended up being a bit messy and I haven't looked at it again since then. I think we can expose it to the smart contracts if its needed at another point, especially since nothing is using it as of now.

Okay sounds good.  I'm happy to merge this as-is then, and consider it done for now.


Flutter docset directory structure error?

I build the docset and import it into the Zeal, but got the following error.

In my case, I have to remove the files in doc directory to the up level directory.
mv C:\Users\Liu.D.H\AppData\Local\Zeal\Zeal\docsets\flutter.docset\Contents\Resources\Documents\doc\* C:\Users\Liu.D.H\AppData\Local\Zeal\Zeal\docsets\flutter.docset\Contents\Resources\Documents\
rm -rf C:\Users\Liu.D.H\AppData\Local\Zeal\Zeal\docsets\flutter.docset\Contents\Resources\Documents\doc

Steps to Reproduce

build the docset described on https://github.com/flutter/flutter/issues/51203
import the docset in Zeal

Hi @liudonghua123
Did I understand correctly that you solved the issue? Is this also the case on the latest master?
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Supporting microarchitecture-specific builds

In various cases, users would like to take advantage of various CPU-specific features or instructions for improved performance, e.g., AVX* or AES/crytography.
xref issues:

conda-forge archspec-enabled packages

cc: @ltalirz

Other relevant links:

x86_x64_vx in archspec: https://github.com/archspec/archspec-json/blob/2d97655b19366ba8e836c1190a3807b57888f57f/cpu/microarchitectures.json#L243

With __archspec you can create a meta-package like x86_64_feature_level that depend on specific __archspec values and downstreams will depend on x86_64_feature_level.
I think a CEP is needed for how to handle virtual packages in a conda-build sense. Currently, virtual packages are one of the main things from a build machine that affect compilation.
For eg: let's say A was built with AVX512f and B needs A built with AVX512f. Then B can be compiled only on a build machine with AVX512f. We can workaround by setting CONDA_OVERRIDE_ARCHSPEC, but then, BUILD_PREFIX also gets that architecture and if there is a build tool relying on AVX512f, that build tool cannot be run.
Therefore CONDA_OVERRIDE_ARCHSPEC should affect only host and not run. (Even host is problematic when considering python etc). This is also an issue with cross-compilation.

Two questions/thoughts:

What is the minimum microarchitecture of your current x86_64 CI build hardware (not the one you choose to build for)?

Would it be acceptable to do cross-microarchitecture-compiles, and let the feedstock maintainers who want this feature worry about making sure that they continue to compile any build tools with the lowest-common-denominator microarchitecture? It would unfortunately mean that one cannot run the tests for these builds, but it may still be better than people who want this feature running off into different directions.

Ivybridge

Python blurs the line with build tools. For eg: numpy needs to be importable to get numpy.get_include() which means numpy needs to be built with the lowest-common-denominator architecture.

Very interested in this for arrow, faiss, pillow, onednn, etc.
Numpy actually has a very elaborate dispatch mechanism (for most relevant functions) based on run-time detection of the available CPU features, so we likely wouldn't have to worry about numpy specifically - i.e. can keep building it for the lowest CPU feature level without losing much.
I guess for other feedstocks it should be based on an analysis whether the impact is worth the blow up in the build matrix, but it would be an amazing tool to have.
The one thing that this makes worse is overloading build string semantics (e.g. cpu vs. cuda, license family, openmp flavour, now arch spec, see conda/conda#11053), which is even more relevant due conda/conda#11612 still being open.

Numpy actually has a very elaborate dispatch mechanism (for most relevant functions) based on run-time detection of the available CPU features

Ah, thanks for pointing this out, I was not aware - so this is even before you hit backends like the MKL (which also do this), correct?
While adding this dispatch capability may not be achievable for some packages (due to amount of work/knowledge required), technologically it would be the best solution in most cases, and I suspect that maintainers of smaller packages may often not be aware of the possibilities here (I certainly am/was not). When documenting the feature we propose in this thread, it would probably be wise to also point maintainers to resources on methods for selecting the best code for the microarchitecture at runtime (e.g. here is an article from the Guix blog on function-multi-versioning that I found interesting).

I don't think the numpy dispatch mechanism is feasible for others to reimplement, it's a very elaborate piece of engineering by some domain experts (not sure how realistic it would be to library-ize it for wider use... 🤔).

so this is even before you hit backends like the MKL (which also do this), correct?

It cannot affect external calls like to LAPACK, it just works for numpy-internal functions that get precompiled for various CPU features and then selected at runtime based on availability.
So this won't help for optimising blas/LAPACK etc. I think that for such compute-heavy libraries, we should just build v2/v3/v4 by default (once the infrastructure is in place, of course).
My point about doing this on an as-needed (resp. as-justified) basis is that not every package will have double-digit performance gains for higher CPU feature levels, and so we (in conda-forge) should not blindly multiply our build matrix by a factor of 3 on every feedstock.

If Linux and glibc is all you are targeting, rpath token expansion is the easiest way to have x86_64 feature level specific binaries. For eg: https://github.com/conda-forge/gmp-feedstock/blob/main/recipe/build.sh#L28-L45 produces power8 and power9 binaries. It can be extended to x86_64 feature levels as well.

Longer term, if we want to support RISC-V then we will probably want to support extensions as well.


[WIP] Implement mongodb changefeed

resolves #927
This pull request implements the changefeed for mongodb. Some changes made:

Remove uniqueness constraints on secondary indexes. This was causing problems and should be handled with #993
Fixed some queries

Current coverage is 96.34% (diff: 100%)

Merging #994 into master will increase coverage by 0.37%

@@             master       #994   diff @@
==========================================
  Files            35         35
  Lines          1789       2135   +346
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1717       2057   +340
- Misses           72         78     +6
  Partials          0          0

Powered by Codecov. Last update 438958f...0fdcff9

Neat! :dancer:


Show Input File existence in field validation error

Problem
Currently we are showing an error if the user does not enter text in the Input File text field. This is a good first step, but there might be more validation needed to prevent user from entering malformed input.
Proposed Solution
We can instead check for the existence of the file instead of "emptiness" for a better form of validation.

Or maybe don't allow the user to edit the file? They already picked a valid file to get this form open...

The input file parameter is no longer editable (see #78, #123) so I don't see the value in validating this field anymore. Closing.


DOCS: Better clarify topic-configured min.isr vs cluster-configured min.isr

I think this better clarifies the misunderstanding in https://issues.apache.org/jira/browse/KAFKA-7474
cc
@joel-hamill (reported the issue)
@ciudilo (reported the issue)
@hachikuji @vahidhashemian (reviewed the previous https://github.com/apache/kafka/pull/3035)

Sounds reasonable to me. @edoardocomar, do you agree?

@joel-hamill I changed it similar to your revision suggestion. Can you give it a look again?

@stanislavkozlovski i provided updated text for the entire paragraph - did you want to add that as well?

@joel-hamill yes, I reworked the whole text a bit based on your revision. Do you think the current version works?

I agree. Probably not the best opposite.

@stanislavkozlovski my revised text provided suggestions for the entire paragraph, not just the lines you changed. i'd prefer the entire paragraph was reworked, but it's your call/PR. :)

@joel-hamill I think we should reword this slightly. The current suggestion is this:
    A message is considered committed when all in-sync replicas for that partition have applied the message
    to their logs. Because only committed messages are given out to the consumer, the consumer will not see a message that could be lost if the leader fails.
    Producers have the option of either waiting for the message to be committed or not, depending on their latency and durability settings.
    This behavior is controlled by the producer acks setting. Topics have a setting for the "minimum number" of in-sync replicas. This setting is
    checked when the producer requests acknowledgment that a message has been written to the full set of in-sync
    replicas. If a less stringent acknowledgement is requested by the producer, the message is acknowledged early
    but is not committed or consumed until all the in-sync replicas apply it to their log.

I find the last sentence a bit unclear.
The behavior is the following: if a topic has a min.isr setting of 2. a producer with acks=all will get an acknowledgment when the message is written to 2. But the message will be "committed" once all the ISRs have the message. So if we have 3 ISRs, the producer will receive an ack when 2 save it but the consumer will receive the message when all 3 save it.
(cc @ijuma this is correct, right?)
How can we make this more clear?
I think we should say something like this:
 If a less stringent acknowledgment is requested by the producer, the message is acknowledged early     but is not committed or consumed until all the in-sync replicas, regardless of the minimum number setting, apply it to their log.

@stanislavkozlovski  your suggestion sounds good to me.

LGTM

After reading this, why would one ever set min.isr to a value > 1? I think the subtlety of that is not covered here. The key thing is that if min.isr=1 and there's only one replica in the ISR, the producer will consider it successful with acks=all. So, if the remaining broker dies, that message is lost. If min.isr=2, the producer would not be able to write until a second broker joined the ISR. If the producer can apply backpressure and replay data (in case it dies), then this is helpful. If any of those is not true, then min.isr=2 is not particularly helpful as the producer might have to start dropping data once its buffer fills or it might lose the data if it dies for whatever reason.

@ijuma I agree, it would be useful to clarify the pros/cons of min.isr values. Here's an initial suggestion:
Old version:
This setting is checked when the producer requests acknowledgment that a message
    has been written to the full set of in-sync replicas. If a less stringent acknowledgment is requested by the producer, the message is
    acknowledged early but is not committed or consumed until all the in-sync replicas, regardless of the minimum number setting, apply it to their log.

New version:
This setting is checked when the producer requests acknowledgment that a message
    has been written to the full set of in-sync replicas and directly affects durability guarantees.
If a less stringent acknowledgment is requested by the producer, the message is
    acknowledged early but is not committed or consumed until all the in-sync replicas, regardless of the minimum number setting, apply it to their log. Note that such an early acknowledgment, while increasing throughput,  is more prone to data loss in the event of a failure happening right after the acknowledgment.

I was wondering if there was any reason we're avoiding mentioning the config directly (e.g acks=all, min.isr). I have a feeling it would be easier to explain (and comprehend) if we gave out more concrete examples


there is no elevation in this couponcard widget?

for this i have tried to give boxshadow in decoration, but its not going outside of the card.

Hi @CKokila, Thanks for opening the issue.
I actually never tested this out with a BoxShadow, I will take a look as soon as I can and see why it's not applying Shadow.
You'll be notified once I make new related changes to it 🙏🏼

Hi @CKokila,
This issue has been fixed in this commit #e09e390.
New property has been added in v0.1.0 named shadow which enables you to apply a shadow to the CouponCard widget like this:
CouponCard(
    shadow: BoxShadow(
        color: Colors.black56,
        blurRadius: 10,
    ),
),

I hope it does the job for you.
Enjoy!

Now it's working. @devmuaz  thank you for your support.


Run IIS using Docker on AWS

I am not able to Dockerize my ASP.NET core using my Mac OS machine. I am using the latest base image you provided:
FROM mcr.microsoft.com/windows/servercore:ltsc2022
OR
FROM mcr.microsoft.com/windows/servercore/iis:windowsservercore-ltsc2022
Error: error: failed to solve: failed to register layer: link /files/program files/common files/system/ado/en-us/msader15.dll.mui /files/program files (x86)/common files/system/ado/en-us/msader15.dll.mui: no such file or directory
Error: ERROR: failed to solve: mcr.microsoft.com/windows/servercore:ltsc2022: failed to resolve source metadata for mcr.microsoft.com/windows/servercore:ltsc2022: no match for platform in manifest: not found

As far as I know you can't use windows docker images on MacOS (unless you're running a windows VM I suppose), have you tried using:
mcr.microsoft.com/dotnet/aspnet:8.0 as base image?
See also:
https://learn.microsoft.com/en-us/aspnet/core/host-and-deploy/docker/building-net-docker-images?view=aspnetcore-8.0


Feature: Validate & access various clinical read codes

The initial list of read codes to support are as follows

Snomed
Read v2
CTV3 (Read v3)

Do you have an update on when this enhancement might be done?


style: configure ESLint plugin:ngrx/component-store-strict

Description

Configures the NgRx ESLint plugin specific for component-store.
References

#0000

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@timdeschryver see #161
How can we exclude .ts files in the tools folder and in apps/**/e2e-* folders?
I tried this:
    {
      "files": ["*.ts"],
      "excludedFiles": ["apps/**/e2e-*/**/*.ts", "tools/**/*"],
      "extends": ["plugin:ngrx/component-store-strict"]
    },

@LayZeeDK that should do it.
The problem here is that the linter has problems to initialize because it can't find a tsconfig file (which is set in the config of the NgRx Plugin). To fix this, you can provide your own config file by overriding the parserOptions.project property.
This is also how the angular eslint plugin configurations are configured, maybe there's a better way though.
For now, since the component-store rules don't rely on type informations (which is why the project is required), I will remove the setting from the config. This means that only effects will require the parser options, but I'll have to think/try on it a bit more to cover all cases.
There's a commit incoming for this.

Thank you for contributing to the Danish Green Energy Hub and Energy Origin, @timdeschryver 🙇♂

Yes, thank you for taking the time! 🙂

Thanks for testing out the NgRx Plugin within a NX workspace 🙂


Replace Direction Keyword array with a spacy matcher

We are currently identifying if a direction was mentioned based on an array of verbs and their different forms. This could be done more sophisticated using a spacy matcher.
This would improve real world performance and make the code more maintainable.

Hello,
I am interested on working on this. Could I please be assigned? Thank you.


Detach for text

Hi, I am quite confused about the loss computation. For computing the loss for learnable queries, I saw the text features are detached and thus will not be computing the gradient.
https://github.com/microsoft/X-Decoder/blob/fca01f69542690e42d1a702a08944ae1af7bd4e2/xdecoder/body/decoder/xdecoder.py#L230

Hi,
Text features are not detached for all the settings, they detach on output (per-layer) but attach on query_emb. This is an empirical design choice.

So, during the training on the tasks related to learnable queries (e.g., segmentation, grounding), the text features are always detached?

Nope, please go back to the code: https://github.com/microsoft/X-Decoder/blob/fca01f69542690e42d1a702a08944ae1af7bd4e2/xdecoder/body/decoder/xdecoder.py#L233
This is attached.


Redirect After Login to intent url

When You try to access a dashboard and you are not logged in already.
The system will pass you to the login page. (that's ok) but later after login, the system does not redirect you to the dashboard that you tried to access at the first, the system will just redirect you to the welcome screen.
The same happens for iframes.

Looks like a limitation with flask-application-builder (or the way we are using it). We need to pass explicitly the next url we want to be redirected after login

Notice: this issue has been closed because it has been inactive for 559 days. Feel free to comment and request for this issue to be reopened.

Any chance to get it reopened? It's quite basic scenario.
Especially useful when someone passes URL, other person opens and needs to login to view it

Implemented a workaround with Superset + Nginx using cookies.
Here is a sample nginx config
`server {
listen 80;
server_name localhost;
location / {
    add_header Set-Cookie "intended=$uri$is_args$args;Path=/login;Max-Age=120";
    proxy_buffers 16 4k;
    proxy_buffer_size 2k;
    proxy_pass http://<IP_ADDRESS>:8000;
}

}`
In superset_config.py,
`from flask import abort, request, redirect, session as web_session
from flask_appbuilder.baseviews import expose
from flask_appbuilder.security.manager import AuthRemoteUserView, AUTH_REMOTE_USER
from superset.security import SupersetSecurityManager
from flask_login import login_user
class CustomAuthRemoteUserView(AuthRemoteUserView):
@expose('/login/')
def login(self):
    if web_session and '_flashes' in web_session:
        web_session.pop('_flashes')

    sm = self.appbuilder.sm
    session = sm.get_session

    # Login user handler
    user = session.query(sm.user_model).filter_by(username=YOUR_USERNAME).first()
    login_user(user)

    intended = request.cookies.get('intended');
    return redirect(intended)

class CustomSecurityManager(SupersetSecurityManager):
authremoteuserview = CustomAuthRemoteUserView
AUTH_TYPE = AUTH_REMOTE_USER
CUSTOM_SECURITY_MANAGER = CustomSecurityManager`

Hi @Esthove, im new with superset. Can i know this issue can just changes on superset_config.py?
Because currently give me error, NameError: name 'YOUR_USERNAME' is not defined. im not sure on this part.
user = session.query(sm.user_model).filter_by(username=YOUR_USERNAME).first()
Thanks for the help.

Do we have a fix for this? Can anyone provide more details?


added pysafebrowsing.version

added pysafebrowsing.version, which allows users to check the version number of the package within python using the syntax:
import pysafebrowsing
pysafebrowsing.__version__

I have also updated the setup.py such that there isn't multiple sources of version IDs. Additionally, I have bumped the version number to 0.1.2.

Thanks !


Canvas keybinding controls

This is the one that I really was interested in (especially the zoom).
Maybe also addresses: #147 (?)
https://user-images.githubusercontent.com/542105/177951522-daef7bbb-9e5e-41f6-8d3e-03eb4dd07db8.mp4

Adds keybinding actions for panning the canvas and zooming. Default keybindings: Arrow keys and +/- for zooming
Refactored the way cursors update their position, making the cursors themselves responsible for updating their location. Removed cursor position updates from all tools.

Had to be done, because panning was causing the cursor location to shift and adding the fix to that to all tools seemed like an extreme amount of code duplication.

Btw, while your'e at at...the list of keybindings becomes pretty big. Can we maybe sort them alphabetically by name? This way all the "Pan" actions would be grouped together for example and stuff is easier to find in general.

Looks good! Not sure why Godot thinks my Plus key (for zooming in) on my German keyboard is an Equal though. But i can remap it, so no big deal.

That is weird... I am using a German keyboard, too, and tested both: Numpad Plus and "normal" Plus keys and both are recognized correctly. I assume that there is a OS difference (I am on Linux, you on Mac?). I was worried about that. Godot is attempting to distinguish between "physical keys" and "regaular keys". I think the first ones are meant to be keyboard agnostic. However, I am not sure which ones I am actually recognizing with the scancode and OS-functions I am using... they seem to be the "regular keys" but there might be still some sort of bug. I assume that the Godot editor is correctly recognizing the "Plus"-key?

I'm on Windows..but yeah might be an OS difference. I can test it on Linux tomorrow (it's a laptop with a differnt keyboard though).

I assume that the Godot editor is correctly recognizing the "Plus"-key?

Do you mean in the code editor for example? Yes, it works there. But in the InputMap dialog i get an Equal for the normal key and BraceRight (Physical) for the physical key. Really weird.


Linux install notes on github

link to "Linux Installation Notes" on the main page readme/wiki goes to an empty page

Thanks Joe.  Lou updated the guide and has fixed the link, which is: https://github.com/LearningRegistry/LearningRegistry/wiki/Linux-Installation-Guide.


order numbers are incremented even if no successful payment is triggered.

If a customer attempts to pay and then cancels the payment, the internal order number of the shop and checkout.postfinance will still be incremented. Although no order is written to the shop, there are still order numbers that do not actually exist and there are gaps in the number sequence.

You can see it above. 42065 was successfull, then one customer tried 7 times to pay, and each time it was not successfull, so the shop generated order numbers, without generating the order, what is what we want, but, next successfull order, that has come with order 42073, normally should be 42066.
Regards

hi @PAO1908CH , you see the number being incremented because from our perspective, each tentative payment is a transaction, hence increasing the order number.
Maybe you could provide us how is this impacting your operations ?

It is not directly impacting our operations, but there is a gap in the order numbering and it could be a problem if we need to debug other problems. That's the reason I think why in the old system, they had a completely other number. The problem there was that you were not able with a easy way to which order the payment correspond. So, it's not that important, let it on standby and let's wait what others say. If nobody else is complaining, you can leave it as it is.
Thanks a lot and kind regards

@PAO1908CH done? if so, please close the issue so we can all see the remaining tasks. Thank you!


Paper theme is now available from bower

When i install bower install bootswatch-scss the paper theme is not in the latest relase

Paper theme is still not downloaded, any news on this?

It looks like bootswatch has scss support now. I'm no longer going to update this package.
https://github.com/thomaspark/bootswatch/blob/gh-pages/paper/_bootswatch.scss


Video support

Do you plan to add support for picking video like in system picker?
I mean smth like this:
func fmPhotoPickerController(_ picker: FMPhotoPickerViewController, didFinishPickingVideoWith videos: [URL]) { }

@congnd
ok. It's nice. I hope so.
I think FMPhotoPicker is an awesome library. many users need support for FMPhotoPicker.
and I hope more growing up FMPhotoPicker.
I added you a collaborator.
plz, check it.
I am very happy if you are the maintainer of FMPhotoPicker.
thank you sooooooo much.
regards.

Hi @sayler8182
Sorry for the late reply.
Thank you for the great idea. In fact, video viewer function is already supported in the core of FMPhotoPicker. So I think supporting video picking is not so hard for now. I will seriously consider doing it and inform you when I have an update.

Is there anything new about that?

With the new release, getting selected videos now becomes possible. Please checkout v1.3.0
https://github.com/congnd/FMPhotoPicker/releases/tag/1.3.0


Removal of key customisation in pagination?

Hi
I just upgraded to the V3 of Saloon and noticed that customisation of keys has been removed in the new Pagination-plugin?
In the OffsetPaginator for instance it is now hardcoded that the keys are 'limit' and 'offset', where it previously was possible to customise.
OffsetPaginator V3
$request->query()->merge([
    'limit' => $this->perPageLimit,
    'offset' => $this->getOffset(),
]);

OffsetPaginator V2
$request->query()->merge([
    $this->getLimitKeyName() => $this->limit(),
    $this->getOffsetKeyName() => $this->currentOffset(),
]);

Of course I can just create my own Paginator or extend and override the class, but it seems a bit more tedious than to just do, what I did in V2
$paginator
    ->setLimitKeyName('batch')
    ->setOffsetKeyName('skip')
    ->setTotalKeyName('Result.Count');

Hey @carnevalle, welcome to v3! 🙌
The best way to overwrite this is from within the paginator you have created on your connector. You can extend the applyPagination method and just change the query parameters which are being sent.
protected function applyPagination(Request $request): Request
{
    $request->query()->merge([
        'top' => $this->perPageLimit,
        'skip' => $this->getOffset(),
    ]);

    return $request;
}

https://docs.saloon.dev/digging-deeper/pagination/limit-offset-pagination#assumptions-made-with-the-offsetpaginator

Hey @carnevalle I'm going to close this issue for now but feel free to re-open if this doesn't fix the issue ☺ Have a great day.


Multiple Request

helo how to get result without waiting all request.
example :
$loop = \React\EventLoop\Factory::create();
        $handler = new \WyriHaximus\React\GuzzlePsr7\HttpClientAdapter($loop);

        $client = new \GuzzleHttp\Client([
            'handler' => \GuzzleHttp\HandlerStack::create($handler),
        ]);

        $client->getAsync('http://Example4.com')->then(function ($response) {
            var_export($response);
        });

        $client->getAsync('http://Example1.com')->then(function ($response) {
            var_export($response);
        });

        $client->getAsync('http://Example2.com')->then(function ($response) {
            var_export($response);
        });

        $client->getAsync('http://Example3.com')->then(function ($response) {
            var_export($response);

        });

        $loop->run();

how to get result 'http://Example1.com' when sucess without waitting all;
thanks

Hey @richardus1992 the code from your example (I've done some quick styling fixes to it btw) will do exactly that. They will come in as the respective servers respond to the request.

Ya Server respone when I'am Hit all url .... but waiting all request done for get the result.
i want get the result without waiting the request ........all done.like AWS have.
thx @WyriHaximus  respone.

Correct, you have to wait until the entire response body is on before you have the full result. If you want to stream the response in you can do something like this:
        $this->loop->addPeriodicTimer(0.001, function (TimerInterface $timer) use ($stream, $response) {
            if ($stream->eof()) {
                $timer->cancel();
                $this->emit('end');
                return;
            }

            $size = $stream->getSize();
            if ($size === 0) {
                return;
            }

            $this->emit('data', [$stream->read($size)]);
        });

Closing this issue for no activity thus assuming it has been resolved.


Can't cancel old transaction - entire safe is stuck

We've created a transaction that we want to cancel, but rejecting this transaction creates a new Cancellation transaction every time. Even if we accept the Cancellation transaction, it won't get through because the cancelation is stuck until the original request gets dealt with. So it seems like the entire wallet is now deadlocked

Every time we try to rejct it, a new cancelation transaction gets created

Could you please attach the safe address for us to debug or email it to<EMAIL_ADDRESS>

Could you please attach the safe address for us to debug or email it to<EMAIL_ADDRESS>
Yes sorry, I've attached the address to the original post as well
0xc9F95caf3c6db747751576b09029FC1BAD5Ee103

Sorry for the delay @orangebutblue. We're looking into it.


cli: demo records don't get properly indexed

Records (and their drafts) created via the demo CLI, go through the BibliographicRecordService, which eventually sends the records to be indexed under the records index name. This basically prevents the before_record_index signal receivers from running, since they're usually subscribed on the index name records-record-v1.0.0. In the end keys like _communities and metadata extensions end up not being indexed and cause issues during REST API serialization.
For this to be solved either:

the before_record_index has to change to become flexible and allow e.g. indexes to be checked via wildcards (i.e. records*)
index naming has to be normalized across modules (i.e. always use the alias records)

@lnielsen can confirm, but the idea was to get rid of those signlas in the new backend, and provide a configurable list of actions to perform before/after index. Using some sort of command pattern.

This is blocking the August release, as nobody can load the demo record so we need as a minimum a temporary solution.

Quick solution would be to add the full index names (i.e. records-record-v1.0.0 and drafts-draft-v1.0.0) in the record_to_index function in https://github.com/inveniosoftware/invenio-drafts-resources/blob/3050402a80d02144af5cfcd5f259eca229820599/invenio_drafts_resources/services/search.py#L26-L28

August release update: invenio-cli demo works without any of these changes (at least I didn't implement them...) in so far as there is no connection between communites and records by default.
Part of @lnielsen and @slint effort is to settle on a long-term solution for this is my understanding.

Closing this since the workflow changed and other modules will need to be integrated (i.e. get out of the old records-rest way)


isNumber, put isNaN check on end

isNaN throws errors sometimes for objects
"Cannot convert object to primitive value"

fixed<EMAIL_ADDRESS>


Mouse position off by a few pixels on OSX when external mouse plugged in

Clicking on the feature gets hard in some circumstances. Looks like only on OSX (in both chrome and firefox) if you have an external mouse plugged in (saw problem with wireless mouse), you need to click slightly to the left of a feature to select it. You can workaround this by 1) resizing the window (even just opening the dev console) 2) unplugging the external mouse and reload the page as from there on you wont have the issue. The problem seems to be related to the vertical scroll bar handling on osx. Have seen the issue on several OSX machines.

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Closing due to lack of activity.


cargo install didkit-cli is failing

The error is
    Updating crates.io index
  Downloaded didkit-cli v0.1.0
  Downloaded 1 crate (29.0 KB) in 0.40s
  Installing didkit-cli v0.1.0
error: failed to compile `didkit-cli v0.1.0`, intermediate artifacts can be found at `/var/folders/wy/pzkn5xrs7338tvz2r51syqlw0000gn/T/cargo-installb5UtI5`

Caused by:
  failed to select a version for the requirement `crypto-mac = "^0.7"`
  candidate versions found which didn't match: 0.11.1, 0.11.0, 0.10.1, ...
  location searched: crates.io index
  required by package `hmac v0.7.0`
      ... which is depended on by `hmac-drbg v0.2.0`
      ... which is depended on by `libsecp256k1 v0.3.2`
      ... which is depended on by `did-ethr v0.0.1`
      ... which is depended on by `didkit v0.2.0`
      ... which is depended on by `didkit-cli v0.1.0`

I had this same build error happen to me on a project that used libsecp256k1.  Unfortunately, it's not as simple as just bumping the version on libsecp256k1, because the latest version (0.6.0) is not backwards compatible (e.g. it changes the crate's module name from secp256k1 to libsecp256k1, among other things).  This version incompatibility seemed to crop up at some time, as if some crate was changed backward-incompatibly after the fact.
I've tried this build on two other (clean) systems, and verified that the same error occurs.  Anyone else getting this?

Hi @vdods,
We've migrated away from the libsecp256k1 crate in https://github.com/spruceid/ssi/pull/180 and https://github.com/spruceid/ssi/pull/271, which should fix this, but have yet to publish new versions of the did-ethr and did-tz crates with those changes. We will do this probably either as part of ssi v0.3 + didkit v0.3 releases, or by backporting the fix to the affected crates.

crypto-mac 0.7 was un-yanked so this is now resolved. Apologies for the inconvenience. Discussion continues in #197 to ensure the latest stable release always works.


[Feature Request] Allow CuptertinoTabScaffold to have null CupertinoTabBar

Use case
I want to use a bottom sheet as the primary means to navigate between different sections of my app, analogously to a tab bar. I want each section to maintain it's own Navigator state, so I am using CupertinoTabScaffold. However, I do not need the CupertinoTabBar to be visible.
This would also make the API between CupertinoTabScaffold and TabBarView more consistent.
Proposal
Allow CupertinoTabScaffold to have a null CupertinoTabBar.

Cc @HansMuller


Correct output of permutations sub

Right. Also, one might argue that we should also include the parens, because they are present in the real output.

I agree, but as they were omitted originally for reasons unknown to me, I left them out.


How can I put text next to an accordion vertical line?

Is your feature request related to a problem? Please describe.
This is either a feature request or question about how to use an existing feature.  I use the accordion lines in graphs right now and find them very useful as a visual guide, but I would like to put labels next to the accordion lines, simple ones like 1, 2, 3, 4....  for some reason I thought this existed already but I'm not seeing it anywhere...
Describe the solution you'd like
see above
Describe alternatives you've considered
If there is a way to define putting text anywhere on the diagram I could figure out how to put it next to an accordion trace, but that would be harder...
Additional context

The accordion doesn't currently support text, I'll add that to the feature request backlog.
The workaround is to use the Annotations plugin which allows you to place text on the plot.

@kmun I have implemented this in https://github.com/spectriclabs/sigplot/commit/877b7b6397fd103f413a324454759a6cc630cfea


1.12.2 menu toggle

How to you make it so instead of holding down the MineMenu button, I can just tap it and the Menu stays open

You can´t unfortunately.
1.12.2 have not been supported for years either.


[DATA] Tasksource fine-tuning

Hi,
Thanks for the great work
I consider fine-tuning moisacbert on tasksource https://github.com/sileod/tasksource/
The deberta-tasksource version is quite popular
But do you plan to release updates on moisacbert  ? e.g. longer training

Hey,
I'm not sure I understand your request correctly -- this repo isn't MosaicBERT but a separate effort to continue modernising the architecture & train on "modern" volumes of data.
We're definitely planning on releasing our checkpoints once training is over 😄


Fix set-env.bat typo

Guess I'm not so super consistent with upper-casing Windows, but at least on the README it is. Thanks!

Haha, I didn't even notice that at first. I was looking at "Windos" instead of "Windows". I just did the other upper-casing because I was on it anyway.
You're welcome :)

Hah, and I overlooked the "Windos" typo in fact until you just mentioned it... :-)


Add support for matching PipelineRuns to Pull Request labels

Implement annotation-based PipelineRun label matching functionality:

Add pipelinesascode.tekton.dev/on-label annotation for label-based PipelineRun triggers
Support label matching on GitHub, Gitea, and GitLab providers
Implement immediate PipelineRun triggering when labels are added
Enable re-triggering of PipelineRuns on commit updates with existing labels
Provide access to Pull Request labels via {{ pull_request_labels }} dynamic variable

Supported providers:

GitHub
Gitea
GitLab

Limitations:

Not supported on Bitbucket Cloud and Bitbucket Server

Demo:
https://github.com/user-attachments/assets/b0c5e418-a980-4676-83d5-11cdb736295b
Changes

Submitter Checklist

[ ] 📝 Please ensure your commit message is clear and informative. For guidance on crafting effective commit messages, refer to the How to write a git commit message guide. We prefer the commit message to be included in the PR body itself rather than a link to an external website (ie: Jira ticket).

[ ] ♽ Before submitting a PR, run make test lint to avoid unnecessary CI processing. For an even more efficient workflow, consider installing pre-commit and running pre-commit install in the root of this repository.

[ ] ✨ We use linters to maintain clean and consistent code. Please ensure you've run make lint before submitting a PR. Some linters offer a --fix mode, which can be executed with the command make fix-linters (ensure markdownlint and golangci-lint tools are installed first).

[ ] 📖 If you're introducing a user-facing feature or changing existing behavior, please ensure it's properly documented.

[ ] 🧪 While 100% coverage isn't a requirement, we encourage unit tests for any code changes where possible.

[ ] 🎁 If feasible, please check if an end-to-end test can be added. See README for more details.

[ ] 🔎 If there's any flakiness in the CI tests, don't necessarily ignore it. It's better to address the issue before merging, or provide a valid reason to bypass it if fixing isn't possible (e.g., token rate limitations).

/test go-testing

@chmouel I've copied same PR that you're using in video example but in output I'm getting:
Labels on PR:
{{
labels
}}

Yeah I need to update the video😅 it's called pull_requests_labels now since can only apply there , see the doc and e2e

/retest

@chmouel now I see
Labels on PR:
bug

@chmouel No, just saying that I am able to print labels with pull_request_labels variabl.

only add the support_matrix documentation tag, merging it due of @zakisk previous review


Release binary linux-amd64 not working in alpine:3.15

Trying to build a docker image using the release binaries from github, I tried with  buildkite-agent-metrics-linux-amd64 but on alpine (3.15) when I try to run the file I get an error:
$ ./buildkite-agent-metrics
bash: ./buildkite-agent-metrics: No such file or directory

Quickly tried with an ubuntu image it works fine, am I missing something? I find this strange since the Dockerfile example builds for an alpine image.

Looks like the pipeline doesn't use the static flag, that's probably why:
https://github.com/buildkite/buildkite-agent-metrics/blob/master/.buildkite/pipeline.yml#L15


Google Maps Traffic Layer

Could I open traffic layer?

I don't think so.

Closing this issue, but feel free to provide a pull req for that :)


PDF image not shown and format issue

Hi I am using phantom-html-to-pdf v 0.5.6, the image is not shown and also facing format issue. I attached a screenshot for your reference.

Hi,
this package is just a wrapper for phantomjs.
We can't solve the rendering issues of phantomjs here.
You can try luck in the phantomjs repository to get some help.


[Bug]: Resources deleted in target namespace if malformed yaml file is provided

What version or versions you have tested?
v1.2.3
Which operating systems have you used?

[ ] macOS
[ ] Windows
[X] Linux

What did you expect to happen?
Malformed YAML file does not implies that all the already existing resources in the target kubernetes namespace are deleted
What happened instead?
The folder provider as --filename parameter to the deploy command included a malformed yaml file. When the commad was executed all the existing resources in the target kubernetes namespace were deleted.
Environment: Azure DevOps pipelines, with bash script
Executed command:
mlp deploy --ensure-namespace=false --server $KUBE_URL --certificate-authority /tmp/kube-ca.pem --token $KUBE_TOKEN --deploy-type ${DEPLOY_TYPE} --force-deploy-when-no-semver=${FORCE_DEPLOY_WHEN_NO_SEMVER} -f ${DESTINATION_PATH} -n ${KUBE_NAMESPACE}
Obtained log:
accumulating resources: accumulation err='accumulating resources from '../../configuration': '/home/vsts/work/1/s/configuration' must resolve to a file': recursed accumulation of path '/home/vsts/work/1/s/configuration': accumulating resources: accumulating resources from 'omitted.networkpolicy.yaml': MalformedYAMLError: yaml: line 20: could not find expected ':' in File: omitted.networkpolicy.yaml
WARN: can't read input file at path /home/vsts/work/1/s/interpolated-files/kustomize-output.yaml

Deleting: Service omitted
Deleting: Service omitted
Deleting: Service omitted
Deleting: Service omitted
Deleting: Service omitted
Deleting: Service omitted
Deleting: ConfigMap omitted
Deleting: ConfigMap omitted
Deleting: ConfigMap omitted
Deleting: ConfigMap omitted
Deleting: ConfigMap omitted
Deleting: ConfigMap omitted
Deleting: Deployment omitted
Deleting: Deployment omitted
Deleting: Deployment omitted
Deleting: Deployment omitted
Deleting: Deployment omitted
Deleting: Deployment omitted
Deleting: IngressRoute omitted
Deleting: Secret omitted
Deleting: Secret omitted
Deleting: Secret omitted

file (before interpolation):
kind: NetworkPolicy
apiVersion: networking.k8s.io/v1
metadata:
  name: gateway-networkpolicy-ingress
spec:
  podSelector:
    matchLabels:
      app: api-gateway
  ingress:
    - from:
        - namespaceSelector:
            matchLabels:
              project: {{OMITTED_NAMESPACE}}
      ports:
        - protocol: TCP
          port: 8080
        - protocol: TCP
          port: 8081
  policyTypes:
    - Ingress

Code of Conduct

[X] I agree to follow this project’s Code of Conduct

I'll add another strange behaviour to this issue: I had the same error reported, but in my case no resources has been deleted, nor created or updated. Instead, mlp has logged the error, but the pipeline still finished with an OK status. This behaviour is wrong, I should have received from the pipeline a KO.

Hi guys, this problem just happened again to me and @ivanpasquariello.
It's very bad behaviour to delete all k8s resources due to a configuration error.
Moreover, the Job finishes with the status "OK", so you don't know the pipeline has failed until you directly check on the cluster and, surprise, nothing is left on it.

Did you plan to fix it?


[DP-70] Sync JetStore repo

Change description

Description here

Type of change

[ ] Bug fix (fixes an issue)
[X] New feature (adds functionality)
[ ] Documentation (adds/updates documentation)
[ ] Refactor (code refactoring with no business impact)
[ ] Chore (housekeeping tasks)

:warning: Be sure to add a matching label for your change type!
Related task(s)

Link #1

Checklists
Development

[ ] Application changes have been tested thoroughly
[ ] Automated tests covering modified code pass

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Ticket: [DP-72] JetStore Add month period function to determine new and terminated eligibility


Search opens but cursor does not default to the text box

When the user clicks the search box, the cursor should jump right into the new window's search textbox, to prevent the user from having to click twice.

Kyle and I worked on it for more than two hours but without success.
But it may have been fixed in angular-mdc v0.40.2

Fixed on some view, but not others. If you go here
https://theorangealliance.org/teams
And click the search icon, the cursor does not default to the box
whereas if you go here
https://theorangealliance.org/events
It does


How to manage setup

Question
As a follow-up to https://github.com/ckipp01/nvim-metals/pull/12#issuecomment-628661919, I think the setup needs to be addressed sooner rather than later.
Currently everything needs to be defined. For example, if you want to enable metals/status, then you need to do two things:

Set statusBarProvider to true in your init_options
Add a callback for metals/status

Then for every other thing that we are adding, we are having to do the same thing.
When I originally started this plugin I envisioned not have a "do this one thing to get started" like there is in coc-metals, but rather instructions in how to tie everything together to hopefully utilize a bunch of other great plugins. However, I'm quickly realizing that we are going to end up creating quite a few callbacks, a lot of the init_option need to be set since the configuration in nvim-lsp is the very base just for Metals to work correctly for a user out of the box without nvim-metals. All that to say, I'm leaning more towards a simple way to add in nvim-metals, and have it do the necessary configuration without having a user add in 30 lines (apart from mappings of course). I think you also had a good point @landerlo about adding a new feature. Ideally a new feature can be added without the user having to adjust their setup.
Approaches

Just handle the entire installation of Metals, bootstrapping etc. This would rid the need of nvim-lsp at all. However, this would be the most work, and I'm not sure I want this. Just adding it here as an approach

`Maybe instead of having to using the lua << whole setup code, have something like setup_nvim_metals { myOverrides }

Something completely different?

Search terms
setup

For nvim-jdtls I ended up going with 1), although I initially was also opposed to doing that.
You can have a look at the change here
In the end it wasn't as much logic as I had thought.

Thanks for linking that @mfussenegger! That's really helpful. That's also the route I'm planning on taking as well now when I get some time.


Some apps does not show lighthouse score

I just read in other issue it just takes 1min to generate lighthouse score. These PWA's are added a day ago but still no lighthouse info generated.

https://pwa-directory.appspot.com/pwas/5676600884461568#t=Linklet App&bg=%233f51b5&c=%23ffffff
https://pwa-directory.appspot.com/pwas/6230263776411648#t=Nerfed News&bg=%233f51b5&c=%23ffffff
https://pwa-directory.appspot.com/pwas/5655274761224192#t=aiueO&bg=%23ffffff&c=%23000000

Just reporting if its some kind of bug mentioned in #330
And FYI the PWA that was added just now https://pwa-directory.appspot.com/pwas/4846515507953664#t=Mixcloud - Radio %26 DJ mixes&bg=%23ffffff&c=%23000000 got the score.

Hi @tezhub, thank you for reporting those. I had to run their tasks manually to fix them, I need to check why sometimes the first tasks fails.


LOGOS

I will create logos for new projects.

Please review these. @Jayshah6699

Please review these. @Jayshah6699


Merge sdf10 -> sdf11

➡ Forward port
Port sdf10 to sdf11. There were some conflicts that I resolved in https://github.com/ignitionrobotics/sdformat/commit/4ea05ef118d91869d0dd8888765fdc9a6c501ce3 while merging:

find version 6.8 of ignition-math6
find ignition-utils1

Branch comparison: https://github.com/ignitionrobotics/sdformat/compare/sdf11...sdf10
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

I resolved one extra merge conflict in https://github.com/ignitionrobotics/sdformat/pull/808/commits/40b54f0ea771c95bcddb7078e8a7c9e5bf4bdcea, which is an attempt to retain the functionality for compiling static libraries contributed in #394 by @joxoby with reviews from @traversaro and @madebr. I've also submitted https://github.com/ignitionrobotics/ign-cmake/pull/203 to add this functionality to ign-cmake.

Codecov Report

Merging #808 (f03ec70) into sdf11 (5c6e870) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##            sdf11     #808      +/-   ##
==========================================
- Coverage   88.89%   88.89%   -0.01%
==========================================
  Files          73       73
  Lines       11072    11070       -2
==========================================
- Hits         9843     9841       -2
  Misses       1229     1229

Impacted Files
Coverage Δ

src/Filesystem.cc
98.80% <ø> (ø)

src/FrameSemantics.cc
73.85% <ø> (-0.05%)
:arrow_down:

src/World.cc
96.23% <100.00%> (ø)

Continue to review full report at Codecov.

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

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-03-25-fortress-edifice-citadel/1343/1

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-04-13-fortress-edifice/1367/1


Add sliding nonce resource to LR

Motivation
We need SR in LR for AOS transactions.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Existing SR tests.
Other
I ran cargo run which has included some other markdown changes, which should be fine.

Nice!

@bors-libra r=dmitri-perelman

:pushpin: Commit 5d48be5 has been approved by dmitri-perelman

:hourglass: Testing commit 5d48be55a82e33393a46f93ba45f5a53df5c297a with merge ea115342bf6a841c0f9ff588b619217866d2e677...

Would you mind adding a test for this? A tx that checks exists<SlidingNonce>(0xa550c18) should be enough.

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: dmitri-perelman
Pushing ea115342bf6a841c0f9ff588b619217866d2e677 to master...

@bors-libra r-
Seems as though cancel doesn't work :(

Would you mind adding a test for this? A tx that checks exists<SlidingNonce>(0xa550c18) would be good. @tzakian has recently made it possible to call tx scripts from the functional tests, so we can also test that some of the scripts that require a sliding nonce can now be sent from LR.

Covered in https://github.com/libra/libra/pull/5284 and https://github.com/libra/libra/pull/5321


Assignment 04 -- 13 Mar 2024

Read slide-1 and slide-2.
Perform all exercises from the slides on a notebook.
Upload the notebook as a gist.
Share here link to the gist.

Here is my gist link for my work: https://gist.github.com/denikaryono/225a6dd37d691b75f2f8639dd439d0fe

Here is my solution to assignment 4: https://gist.github.com/halserach/d9cc244565502a536fbfde16a94ace07

Here is my solution of assignment 4: https://gist.github.com/gitaameliiaa/039fda1d0843b5ce22a5a9127cb674e0


feat: add File.name and Directory.Name

Fixes #6416.

As discussed briefly with @helderco, I've descoped this PR to just include the File.name changes, since that should be an easier approval (also there's some weirdness in the Directory.name implementation I threw together, that I don't fully have the time to work through atm)


feat: ContainerApp-Added additionalPortMappings logic

Description
Adds UDT and property for additional port mappings for ingress.
Closes #3062
Pipeline Reference

Pipeline

Type of Change

[ ] Update to CI Environment or utilities (Non-module affecting changes)
[x] Azure Verified Module updates:

[ ] Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[x] Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] Breaking changes and I have bumped the MAJOR version in version.json.
[ ] Update to documentation

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[x] I have run Set-AVMModule locally to generate the supporting module files.
[x] My corresponding pipelines / checks run clean and green without any errors or warnings

Thanks for the review @AlexanderSehr. The vnet test passed (the one I changed), but the max (didn't change) one didn't pass as I had deployed the keyvault to soon after and the previous test so it still considered the name to be in-use.

Thanks for the review @AlexanderSehr. The vnet test passed (the one I changed), but the max (didn't change) one didn't pass as I had deployed the keyvault to soon after and the previous test so it still considered the name to be in-use.

Makes sense. This comes down to an Azure Caching issue. I recommend to always wait some time after a previous deployment before trying again, especially for bigger tests where many resources are involved. The good news is that you can then just restart the failed job and don't have to re-run the entire pipeline again. But a bit ... annoying none the less.


Change ROAS numbers in the MMM Example Notebook

Numbers in the ROAS chart are a little high, potentially want to consider changing to more reasonable numbers (ballpark 1-8 ROAS) is common
https://www.pymc-marketing.io/en/latest/notebooks/mmm/mmm_example.html

While we are at it, is there anything else that could be tweaked to increase the realism of this notebook?  The spends for the 2 channels is described as being realistic for each of 2 types of channel (social vs. offline). What about parameter values?
The reason I ask is that I imagine that other stuff is going to be built on top of this notebook.  So might as well fix it up before any of those extensions are created.

I'll have a more in-depth look today :)


Py code and doc 2 GitHub CWE 191

Adding code and docu for CWE-191 as part of issue #531

Not sure how David Wheelers 1e35601 got picked up again. Might need to look into cleaning out one of the forks or clones.


STYLE: Replace magic number SpatialObject MaximumDepth = 9999999

MaximumDepth should just be "as high as possible" (or as deep as possible), in order for a function call like IsInsideInObjectSpace(point, MaximumDepth) to go as deep as possible.

This pull request replaces 9999999 with std::numeric_limits<unsigned int>::max(), which is typically<PHONE_NUMBER>  (assuming 32-bit int). That's obviously a lot more than 9999999, but in practice the depth won't go near either of those two numbers. So I consider this to be a style improvement, not an enhancement.

A little off-topic, interesting, the iterator doesn’t seem to move here. May be intention, not sure, a little bit unusual.
template <unsigned int TDimension>
void
SpatialObject<TDimension>::RemoveAllChildren(unsigned int depth)
{
  auto it = m_ChildrenList.begin();
  while (it != m_ChildrenList.end())
  {
    auto itPtr = *it;
    it = m_ChildrenList.erase(it);
    itPtr->SetParent(nullptr);
    if (depth > 0)
    {
      itPtr->RemoveAllChildren(depth - 1);
    }
  }

  this->Modified();
}

@issakomi - from documentation: "The std::list::erase(it) function returns an iterator to the element that follows the last element erased, which can be used to adjust other iterator and pointer values that point to the list."

Thank you.


parser: implement Restore for RenameTableStmt

implement Restore for RenameTableStmt
Issue: pingcap/tidb#8532

the pr: #109 was implemented the TableToTable,after that pr merged,i will merge this pr


Links for Readme.md not working.

In Readme.md, under the contents section, the following links do not redirect to the required sections

Authentication/Authorisation
Request/Response bodies
Limiting request/response data

@Obtin The common thread appears that forward slashes / no longer url slugify with a hyphen but are instead omitted entirely. Should be a simple fix.


ReplicatePackageStatisticsJob is swallowing exceptions and can silently fail without writing records to DB

The output from the job is all positive and showing successful messages. But actually no records is being written to warehouse DB.
One scenario that can repro this issue is:
creating a blank warehouse DB, with tables, stored procedures and functions created. But the fixed dimensions are not populated.

This code has been updated.


File find/replace dialog is pixelated

I searched other issues to see if I can find something similar but I did not come up with anything. Apologize if this is a duplicate. On my machine, the file find/replace dialog becomes pixelated when I use it. I haven't found any specific set of repro instructions, but it happens every time I start typing into the dialog (it typically does not start pixelated). Attached is an image of what it looks like. Please let me know if there is additional information that I can provide.

Version: 1.46.0
Commit: a5d1cc28bb5da32ec67e86cc50f84c67cc690321
Date: 2020-06-10T08:59:04.923Z
Electron: 7.3.1
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 5.3.0-59-generic
Steps to Reproduce:

Use file find/replace dialog
Notice that is becomes pixelated

Does this issue occur when all extensions are disabled?: Yes

@rebornix from the screenshot, it look like greyscale AA on the find widget.

@sbatten may I ask what's the suggested fix for it?

@mattklein123 I tested on Ubuntu 18 but didn't reproduce this issue with latest Stable, can you please help check if the issue is still there?

I have not seen this recently (on Ubuntu 20.04) so perhaps it has been fixed? I can reopen if it happens again.


Starlink G4-4 not in upcoming launches

The December 17 launch of Starlink Group 4-4 is not present in the upcoming launches returned by the API. But I see it listed on r/SpaceX's sidebar. I assumed that the sidebar was updated based on the API, but I guess not?

Was coming here to ping about this too. It appears that SpaceX added a Starlink mission into their manifest kind of last minute. It's within 24 hours, so would appreciate some intervention to get the DB updated. Paging the Deus ex Machina @jakewmeyer 🙏

Trying to find any details about this launch, not seeing anything on SpaceX social media. Subreddit created a launch thread, but then removed it too?

My source is Celestrak which has TLEs for the launch which are sourced from SpaceX official launch data posted (non-publicly) to space-track.org.

See https://twitter.com/TSKelso/status/1470940418523099139?s=20


Path to the CLA.md cannot be used as variable in custom custom-notsigned-prcomment

Describe the bug
It is not possible to link to the CLA document if you use a custom custom-notsigned-prcomment in cla.yml.
To Reproduce
Add this line to cla.yml:
custom-notsigned-prcomment: '
Thank you for your Pull Request, we really appreciate it!
We kindly ask that you to sign our [Contributor License Agreement](${input.getPathToDocument()}) before....  ... .... '

But obviously, getPathToDocument cannot be called within a custom message.
Expected behavior
I expect to see the Hyperlink to the documentation.
=> How can I use a custom custom-notsigned-prcomment and still insert the CLA link dynamically?

Hey,
we got this working by turning it into an environment variable.
The following works for us:

        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          [...]
          PATH_TO_CLA: 'https://github.com/jubiotech/detl/blob/main/CLA.md'
        with:
          path-to-signatures: [...]
          path-to-document: $PATH_TO_CLA
          [...]
          custom-notsigned-prcomment: >
              Like many open-source projects we ask that you sign our [Contributor License Agreement]($PATH_TO_CLA) before we can accept your contribution.
          [...]


Update advanced-hunting-overview.md

Added required admin roles

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/1704

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@lomayor - PR has been copy edited and is ready for final review. This is based on issue #1704.
Thank you.
cc: @kenwith

@mypil, this is actually documented here:
https://docs.microsoft.com/microsoft-365/security/mtp/mtp-permissions
I suggest adding a link to that in the note instead. That topic covers more information, including how data is scoped by RBAC.

@msbemba - please apply the recommended changes from @lomayor
Thank you!

Thanks, @msbemba & @mypil. Tweaked the text and will now merge.


[BEAM-14016] Fixed flaky postcommit test

Fixed SpannerWriteIntegrationTest.test_spanner_update by fixing the
metric exporter usage in spannerio.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @chamikaramj
PTAL, thx!
With the change:

Without the change:

LGTM. Thanks.


[Question] Some dependencies were not found when I started the front-end

**For better global communication, Please describe it in English. If you feel the description in English is not clear, then you can append description in Chinese(just for Mandarin(CN)), thx! **
Describe the question
A clear and concise description of what the question is.
I ran the command 'npm install', there was no problem.But it showed some messages when I started it,
Could someone help me and explain it？
Which version of DolphinScheduler:
-[dev]
Additional context
These dependencies were not found:

async-validator in ./node_modules/element-ui/lib/element-ui.common.js
deepmerge in ./node_modules/element-ui/lib/locale/index.js
throttle-debounce in ./node_modules/element-ui/lib/element-ui.common.js
throttle-debounce/debounce in ./node_modules/element-ui/lib/element-ui.common.js, ./node_modules/element-ui/lib/select.js and 1 other
throttle-debounce/throttle in ./node_modules/element-ui/lib/element-ui.common.js

To install them, you can run: npm install --save async-validator deepmerge throttle-debounce throttle-debounce/debounce throttle-debounce/throttle
Requirement or improvement

Please describe about your requirements or improvement suggestions.

I make element-ui updated to 2.14.1,and this problem doesn't show any more.


feat: Allow tagging subnets individually

Description

This adds 3 variables private_subnet_tags_per_subnets, public_subnet_tags_per_subnets, intra_subnet_tags_per_subnets to allow tagging of private, public, intra subnets individually.
I do not think there is a need for other types of subnets at the moment.
Motivation and Context

Closes #259
Breaking Changes

None
How Has This Been Tested?

[ ] I have tested and validated these changes using one or more of the provided examples/* projects

Tested on one of my existing VPC, here an example tagging a single private subnet and a single public subnet:
Sample input
  private_subnet_tags_per_subnet = {
    1 = { component = "test" }
  }
  public_subnet_tags_per_subnet = {
    1 = { component = "test" }
  }

Sample plan
Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  ~ update in-place
  - destroy

Terraform will perform the following actions:

  # module.vpc.aws_subnet.private[1] will be updated in-place
  ~ resource "aws_subnet" "private" {
        id                                             = "subnet-qwertyuiop"
      ~ tags                                           = {
          + "component"                                  = "test"
            # (10 unchanged elements hidden)
        }
      ~ tags_all                                       = {
          + "component"                                  = "test"
            # (10 unchanged elements hidden)
        }
        # (14 unchanged attributes hidden)
    }

  # module.vpc.aws_subnet.public[1] will be updated in-place
  ~ resource "aws_subnet" "public" {
        id                                             = "subnet-asdfghjkl"
      ~ tags                                           = {
          + "component"                                  = "test"
            # (10 unchanged elements hidden)
        }
      ~ tags_all                                       = {
          + "component"                                  = "test"
            # (10 unchanged elements hidden)
        }
        # (14 unchanged attributes hidden)
    }

Plan: 0 to add, 2 to change, 0 to destroy.

FYI - there is work underway to look at making this possible for users in the next version of the module. you can track it here clowdhaus/terraform-aws-vpc-v4 if interested


Delete .DS_Store

@gmlwhdtjd oh my, hahaha, i didn't write in .gitignore it's folder name. can you add it plz?


ci: update reference docs

This PR is auto-generated by a nightly GitHub action to update the reference docs from source code in withastro/astro.

Fixing at source, to correct URLs changed by #6620 !


Re-add new best height checks from v4 instead of nChainWork

What
V4 basically just puts the block height into the created nChainWork of a CBlockIndex instead of checking the work it actually did. For now, we'll leave it like this to enable miners in the first place to upgrade to V5 and in the next step #969 (VIP-1) we are going to overhaul and redesign all the chain work checks/algorithms.
Why
Well miners shall be able to upgrade their system first to be able to mine against older version without creating an instant hard fork

duplicated by #974


HSDS-178: Update RadioCard component

JIRA issue
Feature
Purpose of this PR is to update RadioCard component to the newest design. The use of it can be found here: https://www.figma.com/file/JwgxQmv2KyQTKwoiuyxHGq/Spec%2F-Messages-1.5
The demo implementation: https://codepen.io/team/helpscout/pen/5d458fb128e71e612ef1ad7202b178cd
The main changes are in "checked" state, the whole card gets larger. Additional there's some opacity added for "non-checked" cards.
Solution
The implementation isn't precisely as in codepen, as there are used images and fixed sizes to achieve the result. However it looks and behaves very similarly. The only thing not implemented as part of this story is change of the font color - that's because it collides with the existing usage of this component.
The solution is to use scale transform function to make the card larger. The card itself doesn't have a fixed size, so there's no easy way to know how many pixels the component should be larger with. Also, using scale makes the implementation much simpler, without the need to use absolute position and wrap into some additional HTML elements.
As part of this PR I've also added additional available Icon size, as it would be needed in the place where it would be used (Messages builder). Additionally, IconWrapper component has possiblity to adjust better to the size of Icon. This change doesn't break anything, but adds another possibility to the usage.
There's new story added to the storybook of this component that shows an example of usage with hsds-illos.
https://c.hlp.sc/YEuRgQmD

Pull Request Test Coverage Report for Build 3478

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

Totals

Change from base Build 3460:
0.0%

Covered Lines:
6346

Relevant Lines:
6571

💛  - Coveralls

@jakubjanczyk one thing I noticed is that the RadioCard active state overwrite the selected one so when the mouse is down on the card, it will animate even though it's already selected. I think it would look better without the animation in that case.

@plbabin I haven't noticed that case, thanks!

@tinkertrain thanks! Is there someone responsible for releasing as in other projects or should I do it?


Quantic mdm

This PR is part of #27.
It overrides MDM._predict_distances to be compatible with our convex distance.
It also provides a wrapper around MDM which inherits from QuanticClassifierBase.
This allows running the optimization on a quantum backend if the convex metric is selected and quantum=True.
@toncho FYI

Thanks! Good luck with the example :)


CVE-2019-1010263 Crypt::JWT

Importing from https://github.com/vti/cpan-audit/issues/14 by @reneeb
http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2019-1010263

"Perl Crypt::JWT prior to 0.023 is affected by: Incorrect Access Control. The impact is: allow attackers to bypass authentication by providing a token by crafting with hmac(). The component is: JWT.pm, line 614. The attack vector is: network connectivity. The fixed version is: after commit b98a59b42ded9f9e51b2560410106207c2152d6c."


device: Accept the PCIAddress in the BDF format for block devices

The kata-agent now only accepts the PCIAddress from kata-runtime in
the "bridgeAddr/deviceAddr" format, which is tied to the device
hotplug interface in qemu. This patch extends the kata-agent to accept
input PCIAddress in the standard BDF format, which should be more
generic to support different hypervisors, e.g. cloud-hypervisor.
Fixes: #823
Depends-on: github.com/kata-containers/runtime#2909
Signed-off-by: Bo Chen<EMAIL_ADDRESS>

/test-clh

/test-clh

/test

@likebreath, @dgibson - please can you summarise that status of this PR? Is it still required?

@jodh-intel Thanks for the reminder.
As @dgibson pointed out, the proposed approach here (accepting new format in B/D/F) is not the right way to extend runtime/agent protocol. Let's follow up on the discussions in https://github.com/kata-containers/agent/issues/823. Closing the PR now.


IdToken in place of AccessToken

What's the point in passing Idtoken as AccessToken for the LoginAccessTokens.  See https://github.com/aws/aws-sdk-cpp/issues/834
What platform/OS are you using?
Macos
What compiler are you using? what version?
gcc

Hi @Sunhick ,
Sorry for the late response, but I'm not sure what you mean by this question. In the issue, using the idToken as an Access Token is an implementation choice, not forced by the sdk.
For more information on the differences between them and their intended purpose on Cognito refer to:
https://docs.aws.amazon.com/cognito/latest/developerguide/amazon-cognito-user-pools-using-tokens-with-identity-providers.html

Spoke offline with @KaibaLopez. Needs documentation.


Bluetooth: BAP: Fix missing cfg data in stream->codec_cfg for BIS

The codec configuration data from the BIS in the BASE was not properly copied to the stream->codec_cfg, and thus it only contained the subgroup codec configuration values.
This commit fixes that by first copying the subgroup codec configuration to the stream, and then setting or overwriting any fields (if they exist).
This requires that each stream have their own codec configuration struct allocated, so this will slightly increase memory usage.

Converted to draft while I'm adding bsim tests for this


Send-MailMessage Excluded from Non-Windows systems

Not sure of the background on this one but if it's something that just needs re-written and tweaked for non-Win systems then I could probably take that on.  I can't even count the number of scripts I have that rely on this.

Looks like it depends primarily on System.Net.Mail.
Fortunately, it also looks like we'll get some semblance of Mail with .NET Standard 2.0

@joeyaiello I think this issue can be closed as the Send-MailMessage cmdlet was ported in PR #3869


 Use a macro to initialize the mutex directly rather than a function that returns the mutex.

@swift-ci please test


Progressbar can not be changed after apply

Changing the position will create a second progressbar...
Why are progressbar etc. applied at all?
Could they not be kept in it's own layer until the final rendering?
Then it would be possible to change or remove them.

They are all applied on top of the images because this app still don't support multi layers.
Multi layer support will be the next big thing to implement on this software.


Support of BDV XML/N5 on S3 needed?

@constantinpape @martinschorb @jhennies,
Do you know whether there are still projects that use BDV XML/N5 on S3 ?
If not, I would consider deprecating our support for this (unless this is supported upstream by bdv or n5 libraries).

Yes, the majority of our projects are still saved in BDV XML.N5 and also use it for S3.

Same here. Most Volume EM projects are on BDV-N5 (Render exports this and they are still implementing OME-Zarr support).

Also the same here. Nothing much to add but BDV-N5 is still alive

@martinschorb could you please give me the location of such an XML?

found one:
https://github.com/mobie/platybrowser-project/blob/main/data/1.0.1/images/remote/sbem-6dpf-1-whole-raw.xml

However, pasting any of those in the browser tells me "key does not exist"....
https://s3.embl.de/platybrowser/rawdata/sbem-6dpf-1-whole-raw.n5/
https://s3.embl.de/platybrowser/rawdata/sbem-6dpf-1-whole-raw.n5/.zgroup
https://s3.embl.de/platybrowser/rawdata/sbem-6dpf-1-whole-raw.n5/.zattrs

Also:
❯ aws s3 ls s3://embl.de/platybrowser/rawdata/sbem-6dpf-1-whole-raw.n5 --no-sign-request

An error occurred (NoSuchBucket) when calling the ListObjectsV2 operation: The specified bucket does not exist

https://s3.embl.de/platybrowser/rawdata/sbem-6dpf-1-whole-raw.xml works for me.

This one works for me and has n5 data on s3: https://github.com/mobie/environmental-dinoflagellate-vCLEM
I'm currently on mobie-viewer-fiji-4.2.3.jar

I figured out how to further support this.