gnmi-1.15 gnmi_set_test.go

Hi,
As Per interface yang file , useful convention for interface ID is combination of  base interface name and subinterface index.
Yang ref:
https://github.com/openconfig/public/blob/master/release/models/interfaces/openconfig-interfaces.yang#L191
typedef interface-id {
type string;
description
"User-defined identifier for an interface, generally used to
name a interface reference.  The id can be arbitrary but a
useful convention is to use a combination of base interface
name and subinterface index.";
}
Also please check other references where we are using subinterface index in id.
https://github.com/openconfig/featureprofiles/blob/bc37772def113e682052073c87a1bb8c8094483c/internal/fptest/networkinstance.go#L53
Thanks,
Prabha

Pull Request Functional Test Report for #2853 / 5cc76ceaab3e58581fb91407d6c07fd0717c2c3f
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

gNMI-1.15: Set Requests

Cisco 8000E

gNMI-1.15: Set Requests

Cisco XRd

gNMI-1.15: Set Requests

Juniper ncPTX

gNMI-1.15: Set Requests

Nokia SR Linux

gNMI-1.15: Set Requests

Openconfig Lemming

gNMI-1.15: Set Requests

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

gNMI-1.15: Set Requests

Cisco 8808

gNMI-1.15: Set Requests

Juniper PTX10008

gNMI-1.15: Set Requests

Nokia 7250 IXR-10e

gNMI-1.15: Set Requests

Help

As Per interface yang file , interface id should be combination of base interface name and subinterface index.

No, that's not true. There's no SHOULD in the yang. It literally says that the id is an arbitrary string.
cc @dplore

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 54.484%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1847

Relevant Lines:
3390

💛  - Coveralls

As Per interface yang file , interface id should be combination of base interface name and subinterface index.

No, that's not true. There's no SHOULD in the yang. It literally says that the id is an arbitrary string. cc @dplore

@LimeHat  please check updated description where we have other references which shows id is combination of interface name and sub interface index.

As Per interface yang file , interface id should be combination of base interface name and subinterface index.

No, that's not true. There's no SHOULD in the yang. It literally says that the id is an arbitrary string. cc @dplore

@LimeHat  please check updated description where we have other references which shows id is combination of interface name and sub interface index.

As Per interface yang file , interface id should be combination of base interface name and subinterface index.

No, that's not true. There's no SHOULD in the yang. It literally says that the id is an arbitrary string. cc @dplore

@LimeHat  please check updated description where we have other references which shows id is combination of interface name and sub interface index.

I think the code is correct as is.  An interface name is permitted to be any arbitrary string which is system (vendor) defined.
Therefore, we do expect the format of the string to vary between implementations and devices.   I don't believe this test intends to validate the format of the string, rather just that the interface node (identified by said string) exists when added to a NETWORK_INSTANCE.

I notice that in metadata.textproto, no DUT is assigned the use of deviations.InterfaceRefInterfaceIDFormat for this particular test.  If it is truly not needed  then the deviation could be removed.
But I think in practice, I see at least Cisco IOS XR does need this in some other tests?   I assume this is because Cisco IOS XR explicitly appends ".0" to their interface names when added to a NETWORK_INSTANCE?
@aaronmillisor

I notice that in metadata.textproto, no DUT is assigned the use of deviations.InterfaceRefInterfaceIDFormat for this particular test. If it is truly not needed then the deviation could be removed.

Looks like it is needed by the OP, but instead of enabling it, this PR proposes to change the id for everyone :-)

@LimeHat please check updated description where we have other references which shows id is combination of interface name and sub interface index.

I don't doubt that there are references and this format can be used.
On the other hand, if an implementation requires this format, I don't see why this shouldn't be a deviation (like it is today)

But I think in practice, I see at least Cisco IOS XR does need this in some other tests? I assume this is because Cisco IOS XR explicitly appends ".0" to their interface names when added to a NETWORK_INSTANCE?
@aaronmillisor

@dplore Yeah, we are adding interface into network instance along with subinterface index.
    niface := ni.GetOrCreateInterface(name)
niface.Interface = ygot.String(name)
niface.Subinterface = ygot.Uint32(uint32(sub))

It is already unique in this case, isn't it?

It is already unique in this case, isn't it?

Yes, in this testcase. My question was generic. Shouldn't the format of id remain same for all cases while making it unique for the I case i mentioned earlier? Implementations that do not support the use of interface-name+subinterface-index format can use a deviation to pick a format of their choice. Implementations that already use interface-name+subinterface-index format shouldn't be expected to use a deviation.

In practice because OC doesn't have a standard interface string format, any interface name manipulation or parsing should be done via deviation.  The only "standard" / "non-deviated" solution we can use at the moment is an exact string match.

@r4huluk the point is that OC does not require to use any fixed format.
Any implementation that requires a fixed format deviates from the OC definition. Would you agree with that?
Your argument is "the current id format, used in this test, cannot be used to create the two subinterfaces on the same interface". That is true, but does that invalidate the previous point? Absolutely not.

Honestly I think things got a bit murky over time wrt. the interface-id typedef cc: @aashaikh as the original commit went in > 8 years ago on 03/30/2016
From history, the intention from what I recall was that the interface-id was to represent the fully qualified interface format of an "interface" + "subinterface" however the delimiter was left up in the air (ideally a .).  It's not until items like this across various implementations expose potential differences and interpretations much like an interop exercise.
What this was not meant to be is a user-defined alias which appears to be how it is implemented in SRL (for very good reason since the description stmt states this).  Such a case of this is inconsistent then across the system and could vary anytime interface-id is in use.
In addition is another similar node under the interfaces model "subinterfaces/subinterface/state" named name which carries the following description stmt:
  leaf name {
    type string;
    description
      "The system-assigned name for the sub-interface.  This MAY
      be a combination of the base interface name and the
      subinterface index, or some other convention used by the
      system.";
    oc-ext:telemetry-on-change;
  }

This is another example of a fully qualified name (that could be leafref'd and keyed elsewhere however state only) however the description stmt states "system-defined" here.
As a consumer in this case, I want to bind an interface to a network-instance, why is a user-defined alias necessary other than introducing inconsistency and complexity in config gen + state retrieval?

OpenConfig project historian perspective only.
As a YANG practicality: we have lists of interfaces that require a base interface name, but optionally require a subinterface -- in such lists, we must have a key that isn't "interface subinterface" since the YANG keys must all be specified.
There was a long discussion on this matter: https://github.com/openconfig/public/pull/802

What this was not meant to be is a user-defined alias which appears to be how it is implemented in SRL (for very good reason since the description stmt states this). Such a case of this is inconsistent then across the system and could vary anytime interface-id is in use.

SRL implementation is based on what was discussed/clarified in pull/802 referenced by Rob.
Specifically, one of the comments in that discussion stated the following:
It should be possible to name the key anything that one wants (there are no such constraints described in the model). It is OK to require a specific format if and only if the interface and subinterface leaves are used to resolve the interface that is being referenced.

It should be possible to name the key anything that one wants

This defeats the purpose of the typedef and the key should then just be an opaque user-defined string if we have too loose of rules around.
IMO - the following logic should apply:

If the interface-id is meant to be a user-defined alias then create a common structure in a subtree that defines this value that has child interface + subinterface leafrefs (The pattern we see repeated all over various models).  Then adjust all other models to point to this definition.  Essentially, we just created an abstract user-defined name (alias) to represent a mapping to an interface (or interface + subinterface)

But is the above necessary?  Seems like unwarranted additional overhead for both the consumer + implementor.  What does an "alias" mean and how far would a system need to go to embed into their implementation.  All such cases are likely to be alien and thus all have impacts when it comes to data correlation to underlying subsystems.
OR

Put a strict protocol around interface-id so that it has common meaning.  I don't understand why we can't state:

If you have an interface without any subinterfaces, encode directly as the interface name (e.g. et-0/1/0)
If you have an interface with >= 1 subinterfaces, use a . delimeter (e.g. et-0/1/0.0)

You wont necessarily validate within the language rules of YANG but rather this typedef is a soft contract that it relates to certain schema nodes when parsed (common tooling can be written against the typedef now).  Child leafrefs (interface/subinterface) can still exist to provide the redundant hard relationship.
OR

We don't use interface-id at all - this example seems provide no benefit of the typedef and its an ungated user-defined string.

Every instance in the model-set that defines this structure that uses an interface-id for a key today means the consumer can generate whatever they want across all definitions.  If you wanted to create foo here to map an interface to NI, you are free to do so, in the ACL domain, you can create bar and map to the same underlying constructs.
I question any case of user-defined objects if they provide no real meaning, especially if we are trying to drive similar behaviors across the system and allow for divergence.

3. Put a strict protocol around interface-id so that it has common meaning.  I don't understand why we can't state:

If you have an interface without any subinterfaces, encode directly as the interface name (e.g. et-0/1/0)
If you have an interface with >= 1 subinterfaces, use a . delimeter (e.g. et-0/1/0.0)

I opened a pull request to explore a new string type with regex pattern constraint at https://github.com/openconfig/public/pull/1092


Add Mediator.Net

CATEGORY/PROJECT_NAME - Mediator.Net
PROJECT_DESCRIPTION
A simple mediator for .Net for sending command, publishing event and request response with pipelines supported

Thoughts: Yes it's new. However, there aren't too many in this space. Hmm

is it anything that i can do or should i just wait?

@mayuanyang wait a few days. If nobody else comments I'll merge it in for you

same as IoC, we got many in the list. MediatR is good and they got pipeline in recent 3.0, Mediator.Net was designed to have pipelines when it was born, in fact it focus on the mediator's middleware ecosystem, it has got many middleware side projects which can be pluged into the pipelines, e.g serilog, unitofwork, eventstore and messagequeue etc, you might notice that having the all is useful for domain driven design and event sourcing. Thought that both frameworks are mediator pattern, but the design focus is different


Mira channel

Changes to make mira.channel work

there was a pretty nasty bug in getFrames in your last commit, which I fixed.
One question I'd like to discuss: mira.channel is handled on the patcher level. However, mira.channel kinda behaves more like a global router/channeling object right (as in multiple patchers receiving the same messages if listening to the same channel)? Does it make sense to have them nested in the patcher's obj list this way incl the retrieval on the name lookup etc?
I could also think of a more global approach with a list of channels in the state and events when these get added/removed in Max and additional console output if a channel is not know in Max (bc when then also get register events in Xebra.js). So basically channels would be a bit more like frames, patchers etc. and if we like to extend their functionality it would already be more isolated etc.  Maybe that's a little overkill and not really needed but deffo a thought that crossed my mind. any thoughts @starakaj? We could also have a quick chat/call about that if u like.

merged after discussion with @starakaj


Can't build camlistore

On a new fedora system i tried the following:
go run make.go

and I get that:
camlistore.org/pkg/rollsum
camlistore.org/pkg/fileembed/genfileembed
tmp/build-gopath-nosqlite/src/camlistore.org/pkg/blobserver/blobpacked/wholefetch.go:31:2: cannot find package "camlistore.org/pkg/conv" in any of:
	/usr/lib/golang/src/camlistore.org/pkg/conv (from $GOROOT)
	/home/schmitch/camlistore/tmp/build-gopath-nosqlite/src/camlistore.org/pkg/conv (from $GOPATH)
Error building main binaries: exit status 1
exit status 1

Edit: git clone https://camlistore.googlesource.com/camlistore works fine..


enable to login

@ChanchalKaushik128 If this is the beginning of any issue, please provide more information.


Google Geocode fails due to OpenSSL error in GM 2.9.3 with PHP 5.6

from the list:
https://lists.osgeo.org/pipermail/geomoose-users/2018-April/006385.html
I looked into this today for a while and at first it looks like the 2.9.x google_geocoder.php should work fine:

If the user has an API key specified, then a google geocode request is correctly generated and sent.
If you independently load the requested URL in a browser, the Google geocode service returns the expected XML response.

However, the simplexml_load_file($request_url) request in the google_geocoder.php script fails with "url not loading", which is somewhat mysterious.
Looking in the Apache error logs, I found:
AH01215: PHP Warning:  simplexml_load_file(): SSL operation failed with code 1. OpenSSL Error messages:: error:14090086:SSL routines:ssl3_get_server_certificate:certificate verify failed in google_geocoder.php on line 54

AH01215: PHP Warning:  simplexml_load_file(): Failed to enable crypto in google_geocoder.php on line 54

AH01215: PHP Warning:  simplexml_load_file(https://maps.googleapis.com/maps/api/geocode/xml?address=800+NE+Oregon+Street%2C+Portland%2C+OR&key=blahblahblah): failed to open stream: operation failed in google_geocoder.php on line 54

AH01215: PHP Warning:  simplexml_load_file(): I/O warning : failed to load external entity "https://maps.googleapis.com/maps/api/geocode/xml?address=800+NE+Oregon+Street%2C+Portland%2C+OR&key=blahblahblah" in google_geocoder.php on line 54

So the error is related to:
http://php.net/manual/en/migration56.openssl.php
Solution would be to correctly enable OpenSSL verification, (or if desperate, turn it off) :
https://stackoverflow.com/questions/26148701/file-get-contents-ssl-operation-failed-with-code-1-and-more

The solution posted here:
https://stackoverflow.com/questions/32621929/error-when-loading-external-xml-file-with-php-via-https-ssl3-get-server-certif
solved the problem  when inserted above the call to simplexml_load_file($request_url).
In the case of MS4W users, they can point to the certificate that is included with that server, so the insert to google_geocoder.php looks like:
$context = stream_context_create(array('ssl'=>array(
    'verify_peer' => true,
    'cafile' => '/ms4w/Apache/conf/ca-bundle/cacert.pem'
)));
libxml_set_streams_context($context);

But that's obviously specific to MS4W configuration, so I guess a generic fix would involve being able to tell GeoMoose where your certificate is in local_settings.ini if that were possible?

Is there a reason the Windows build of PHP doesn't know where the CA files are by default?  Is there a php.ini setting or maybe compile time setting for this?
IIRC the Linux builds of PHP just know where to look.

Yeah that would be a better solution. Looking at the php.ini file for MS4W, it just looks like it is not set / commented out, so I'll close here and link this issue to a MS4W ticket instead

For reference, here is the related MS4W ticket: https://ms4w.com/trac/ticket/171


Investigate CRDs behavior's during upgrade

after considering #776  we need to make tests:
Scenario 1
apply crd v1(storage) and v2 in same yaml
apply v1 resource sample
apply delete v1 crd and  add v2(storage) in same yaml
assert v2 resource sample

[ ] does conversion work?

Scenario 2
test same but having crd.yaml in a helm template and use helm install/upgrade

[ ] what happens if we try helm rollback?

Scenario 3 - happy path  😄
helm install (apply crd v1 and v2 (storage) in same yaml)
apply v1 resource sample
apply delete crd v1 and create identical v2(storage) in same yaml
assert v2 resource sample

[ ] what happens if we try helm rollback?

Scenario 4
apply crd v1
apply again adding one optional field, what about the default value? Is it present in crd?
does it fails?

results:
myth 1: busted


Tweaks from 2.x

Extracted from the 2.0 branch, commits that don't make API changes.

Rebase on and target 1.2_stable.


Update packages

Coverage remained the same at 100.0% when pulling a89078f90304d5ff48fcf417f07f41e4a95ab4ee on chore/upgrade-packages into dbe93fa4d684e691f32d33ca1ba62d02396dfba8 on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Pipelines & Release workflow

Fix the actions needed to re-establish a development and release workflow

Hi. I'm still working on it. Unit tests now are fine, integration tests still break on Ubuntu: https://github.com/dex4er/freelens/actions
I would like to introduce nightly builds and I think they should be built on separate repo to not confuse normal users who will be looking for normal releases.
Would you mind to create repo like freelens-nightly or something similar?

Integration test breaks on Ubuntu on rpmbuild command. This is not native environment for rpm but somehow lens was provided as rpm package too, then I'll rather investigate it than disable rpm as a electron build package.

freelens-nightly

https://github.com/freelensapp/freelens-nightly-builds

Hi @dex4er you have now write permissions the needed repo, so I think you are able to go on working on the release process.
Can you help us to complete it about this issue and also about this https://github.com/freelensapp/freelens-node-pod-menu/issues/32 even without signign?
Meanwhile I'll check out too that topic:
https://github.com/freelensapp/freelens/issues/13
It's also important to find other collaborators to work on other imporetant topics like fixing the security issues.


Bootstrap local leaf hub clusters with KinD

Signed-off-by: myan<EMAIL_ADDRESS>

Looks good overall. just a small suggestion.

/lgtm


Update travis.yml to use latest version of Xcode (8.3)

Fixes #59

Hey @nap-sam-dean if you're happy with these can you merge them -- I don't have permissions 💯

@kerrmarin You do now :) Not sure why/how those permissions got revoked. I assume because you ran off to Canada ;)

😂  that's what I get for jumping ship


Farhan Ahmad Nurzi / Task - Refactor containers components [Translation Key Extraction]

Changes:

...

Type of change

[ ] Bug fix
[ ] New feature
[ ] Update feature
[x] Refactor code
[ ] Translation to code
[ ] Translation to crowdin
[ ] Script configuration
[ ] Improve performance
[ ] Style only
[ ] Dependency update
[ ] Documentation update
[ ] Release

Please resolve the conflicts :pray:


Prevent out of memory with multiple requests in a test

This fixes #656. I added a test and got it to pass by restoring some code from the v3 branch.
Here is the previous commit when the check was removed.
I do not use Laravel Octane or know enough about it to know if this fix would be compatible with Octane.

Nice work finding a fix for this - we are running into the same issue which made the upgrade to v4 impossible.
If that conditional is incompatible with Octane, it could be possible to check if Octane is running with something similar to this: https://github.com/laravel/octane/issues/508#issuecomment-1096937174


[BUG] Storage. Multiple large streaming uploads stuck on AKS.

Describe the bug
We have java Spring Boot application with an api where users can upload files of any size and multiple files at a time, the files while uploading are streamed directly to the backend storage (Azure storage in that case). We provide so-called resumable uploads so we upload large files using multiple PUT requests with variable size (depends on user's bandwidth). So when one or more users upload several large files PUT requests get stuck until some timeout. I tried to do that via Block Blob and also Append Blob, the result is the same.
That's not our application issue because when I launch locally or other cloud provider(Google) it just uploads fine to Azure storage, it only happens with AKS. Maybe something weird happens while converting InputStream to Flux but only on AKS instances.
When it's stuck if i just restart kubernetes pod it may continue until stuck another time.
Exception or Stack Trace
Usually it ends with EOF becaouse of client timeout:
Caused by: java.io.EOFException: Unexpected EOF read on the socket
To Reproduce
It might be difficult to explain but any Spring Boot app when some Upload Controller which passes InputStream directly to Azure Storage. Upload about 10 files at a time on AKS instance.
Code Snippet
      blockBlobClient.stageBlockWithResponse(blockId,
          Utility.convertStreamToByteBuffer(eis, partSize, maxBufferSize(), false), partSize,
          StringUtils.isNotEmpty(md5) ? Base64.getDecoder().decode(md5) : null, null)
          .block();

(retryable/seekable can't be true here).
Expected behavior
Uploads finish fine.
Setup (please complete the following information):

OS: AKS
IDE : IntelliJ
Version of the Library used azure-spring-boot-starter-storage v3.4.0, tried azure-storage-blob v12.10, v12.11, v12.12 beta.

Additional context
I tried to use the same code on GKE (Google cloud) connecting to Azure storage and it works fine. Also we support local disk, S3 and GCS as storage provider and upload to GCS from AKS is also working fine. It's only Java -> AKS -> Azure storage problem.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

@stliu could you please follow up?
/cc @gapra-msft

@denis111 it's difficult to me that to do root cause analysis  based on the description , would you provide a sample project to help us reproduce this issue?

@stliu that's not so easy but I guess it's the only way. I'll try to prepare sample project then.


added rg

Biggest Computer science high school on Balkans. http://rg.edu.rs/
We need you tools for C#, C++, Databases, python and web development.

@devfoFikiCar Pull request merged. Thank you!


Remove import macro and improve build macro

Fixes #304
Fixes #268
Some highlights:

Create a new crate for resolving WinRT dependencies. It's unfortunate we need another crate, but we would otherwise need to duplicate quite a lot of code between cargo winrt and the winrt_macros crate.
Creates a integration_tests crate which is used for integration tests against the Windows SDK and test component
Removes the import! macro and the ability to depend on the OS for metadata.

Looks promising!

@kennykerr this is ready for a more in-depth review!

Will do.


Filter by embedded resource count

Hello everyone,
In the example for Resource embedding I would like to be able to do following:
GET /films?select=title,nominations{rank}&nominations=count.ge.2
The main goal is that currently if you select embedded resources PostgREST always returns the parent.
I would like to be able to filter out parents that do not contain children. This gets more serious when embedding multiple levels of resources -> you have to transfer and filter lots of unnecessary data.
Possible alternative implementation(maybe better) could be:
GET /films?select=title,count(nominations),nominations{rank}&count=ge.2
There is currently support for computed columns but it cannot accept the embedded resources.
If there would be possibility to pass the list of embedded resources, it might be possible to create global function - though I am not sure if this is even possible in PGSQL -> this is beyond my PGSQL skill
CREATE FUNCTION count_resources(SETOF resources) RETURNS integer AS $$
    WITH res as resources
    SELECT COUNT(*) FROM res;
)
$$ LANGUAGE SQL;

The last approach is to implement for every relation function like:
CREATE FUNCTION film_count_nominations(film) RETURNS integer AS $$
  SELECT COUNT(*) FROM nominations WHERE film_id=$1.id;
$$ LANGUAGE SQL;

But this will result in lots of functions for every relation in every table.
Is there any other way how to resolve this?
Thank you very much!

One way to do it would be (but i am not completely sure it's optimal)
first, do not expose the schema where your tables live, expose a schema api where you have only view definitions
then have the last function that you defined somewhere and a view like
create or replace view api.films as
select id, name,  data. film_count_nominations from data.films;

then add a index like this
create index on data.films (data.film_count_nominations(data.films.*));

hopefully this index will prevent running that count query on each request.
If the index trick does not work then change the definition of the api.films view to be a join between films and nominations so that you can calculate that count column

this thing is off the top of my head so take it with the grain of salt, but the main idea is correct, you have to bring up the "child property" up to the level of the parent so that you can filter based on it as if it's a property of the parent

@aranwe did the comments here help you with your issue? Any progress?

@begriffs sorry I have not replied :)
In the end we have a bit reworked the design - but it would have had solved the problem partially:
We needed to filter the counted relations at the same time (i.e. WHERE count(nominations) > 0 AND nomination.rank = 'XY').
There is a workaround where we would query some aggregation table and from there the "films" and "nominations", but it would look rather messy.
So the count feature would have been great anyway (but I do not need it at the moment).
Anyway adding the count to the view directly for filtering is great postgrest design pattern -> we should create a Wiki with postgrest design patterns such as this as I believe lots of people could benefit from this (I know some tips&tricks are on the webpage, but there is quite a lack of PgSQL specific design patterns, let alone those that work nicely with postgrest).
I could write something up (though I will be unavailable for the next 3weeks) :).
If you find / stumble upon some interesting pattern link it here, I can go through that later and aggregate sth :).

Thanks, I'd love if you wanted to contribute a new section in https://github.com/begriffs/postgrest-docs about design patterns. (That's the repo that generates https://postgrest.com)

The way to implement this seems to be answered here. A separate issue for documentations can be opened in the docs repo


(music): Support adding tracks to playlist

Is your feature request related to a current shortcoming? Please describe.
My tadd script does exactly that: Adding the current playlist to a playlist, either provided or default playlist.
Describe the solution you'd like
Implement the use case "Add current playing track to a provided playlist", see tadd for related code inspiration and especially for the working AppleScript part.

This is a duplicate of #3. Closing


Add provision support for Ubuntu 20.04(Focal).

Added provision support in Ubuntu 20.04(Focal).
Though we need to upgrade transifex-client to 0.13.7 if we are using the python provided by Ubuntu 20.04 (python3.8.2).
Testing Plan:
Tested on VirtualBox with Focal iso provided by ubuntu(daily builds).
Work Towards #13738
.

@hackerkid @timabbott please check this.

This is great, merged, thanks @arpit551!  A few follow-ups:

We should update the documentation to communicate Focal is supported in the development environment.  git grep Bionic docs/ is a good approach.
We should get CI running on Focal.  @hackerkid and @gnprice I think should be able to help, but I think we first need to use the tools under tools/circleci/ to generate a Docker base image for CircleCI.
We should open an issue on the Groonga launchpad for them to add Focal support; it should be quick for them and once it's done we can stop building from source for Focal.
It'd be great to make the interface for virtualenv dependencies just that new function, rather than only using the new function on Debian.
We can start working on the production installer side of this.  You'll want to use a similar commit structure to what we merged here; the bit that enables actually installing in scripts/lib/install will be the last commit we merge, but obviously you'll need that for testing.

Yup. For making circleci work you need to first generate a docker image and then publish it in dockerhub. Once that is done you can update the config.yml file to run the tests on the new image. Once everything is working fine as expected Greg can probably push the image to his dockerhub, which is where the images are currently stored and then update the config file to reflect the new image ID. Let me or Greg know if you have any further queries regarding this.


pester autodetection, fix

Type of Change

[x] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Fix a problem in autodetection. There are (and there will be additional) tests which are not using neither $script:instance1 nor $script:instance2, like InModule.Help.Tests.ps1

this looks close to  done but has a werid error about help missing from a command that doesnt exist?

gotcha, this is a byproduct of having the module dot-sourced.
I'll change the routine to test only functions within the "functions" folder rather than the current behaviour of checking everything except internals.

aaand, it correctly reports fails for two commands which do not have corresponding help params defined.

groan, better to revert this

@potatoqualitee can you merge this ? I'll fix the two commands help params missing in another PR ASAP.

The PR fixing those params is https://github.com/sqlcollaborative/dbatools/pull/2561 .

btw, @niphlod - i believe you are admin, feel free to  merge when required like stuart does.


Vyacheslavtroshin/ch961/refine lumeod api server protocol

Changed custom string delimited protocol to json structures
Added wrapper structure for req/rep and notification calls.
Enforced and documented WebRTC communication protocol.

Regarding commits: I see some rebasing is needed before merge :)


Move off of "ibmq_qasm_simulator" and towards using AerSimulator for qiskit tests

As of right now, many of our tests are still using the ibmq_qasm_simulator cloud-service backend that is going to be deprecated in 2 weeks on May 15th. In response to this we are migrating from "ibmq_qasm_simulator" to a local simulator, AerSimulator, which is what Qiskit recommends for the migration.

[sc-62210]

[SC-55725]

@lillian542 @obliviateandsurrender I think this looks ready. There is some additional work to be done for hardware support and for additional tests on that front (mocks, etc...). I have created 2 PRs related to this but probably need one more for the mocking/conftest changes.


Deploying to AWS Beanstalk

Is it possible to deploy to AWS Beanstalk?

My guess is the short answer is No. Maybe some else can explain otherwise.
Maybe if you deployed to a docker instance and then pushed that to docker hub (or something like that) then you could probably use beanstalk to reference that?
Here is the official way to build a custom AMI.
http://docs.aws.amazon.com/elasticbeanstalk/latest/dg/using-features.customenv.html


Workflow entity child and member access pt 184904864

Improves access to child entities in workflow allowing children to be accessed as ${entity.children[n]} in addition to the current syntax $brooklyn:child("id")

This looks good to me.  Thanks @frogfather .
It does look like there are a couple of tests that are failing intermittently.  I've copied these below and will see if I can tell what's going on.
java.lang.AssertionError: null: lists don't have the same size expected [8] but found [16]
at org.apache.brooklyn.core.workflow.WorkflowBasicTest.testWorkflowLogging(WorkflowBasicTest.java:381)

testSimpleErrorHandlerOnStep(org.apache.brooklyn.core.workflow.WorkflowPersistReplayErrorsTest)  Time elapsed: 0.276 sec  <<< FAILURE!
java.lang.AssertionError: List of test is different length to list of data to test expected [8] but found [112]
at org.apache.brooklyn.core.workflow.WorkflowPersistReplayErrorsTest.testSimpleErrorHandlerOnStep(WorkflowPersistReplayErrorsTest.java:599)

testSimpleErrorHandlerOnWorkflow(org.apache.brooklyn.core.workflow.WorkflowPersistReplayErrorsTest)  Time elapsed: 0.271 sec  <<< FAILURE!
java.lang.AssertionError: List of test is different length to list of data to test expected [9] but found [111]
at org.apache.brooklyn.core.workflow.WorkflowPersistReplayErrorsTest.testSimpleErrorHandlerOnWorkflow(WorkflowPersistReplayErrorsTest.java:630)

there is no need for children to be a field; made a minor code cleanup in<PHONE_NUMBER>880438b9cb03153472749a36dc1712
also:  members was not handled, and the proposed map-style lookup ${entity.children["id"]} was not handled.  these done in e22c7325a39bfbb8d1ca7872dbfb7915cdaad913.


assignment operators page has unescaped html

https://docs.microsoft.com/en-us/cpp/cpp/assignment-operators#comments-container
The |= operator's HTML is not properly escaped and shows as "& # 124 ; = ".

@mikeblome @ghogen Can you fix the issue?

@ghogen @mikeblome - can you please take a look?

Fixed. Thanks @binarycrusader !

How long before the change shows up on docs.microsoft.com?

It's up there now. Thanks again!

It’s now fixed, thanks.


python310Packages.ssdeep: 3.4 -> 3.4.1

Description of changes
https://github.com/DinoTools/python-ssdeep/blob/3.4.1/CHANGELOG.rst

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 201914 run on x86_64-linux 1

2 packages built:

python310Packages.ssdeep
python39Packages.ssdeep


Allow users to disable or customize checks.

Allow any check to be disabled
Expose configuration options for some checks that could use them

cc @cnelson @jcscottiii

WDYT @cppforlife ?

@jmcarp slept on it. im going to pull this in within next few days.

@cppforlife: let me know if this needs revisions.

Any chance we could see this merged this week?

@jmcarp oops deleted one of your comments accidentally... jumpy browser scrolling.
@cnelson pulling in now...


Editing side on basketTrading partially fails

Within the basket trading feature, whilst a basket is off market, side can be edited at Basket level and will update side on constituents.
Issue: when side is toggled at Basket Level constituents are updated but BasketTrading row is not updated.

UI side bug: hard coded to Buy
Server side: if you update basket trading side, it flip the sides of constituents without checking whether parent side has actually changed. So if parent side is BUY and you change it to BUY - parent end up BUY but constituents ends up SELL


docs(Outdoor Hotspot Guide): Update name of hotspot from Indoor to Outdoor

Thanks for the fix @mattstabeler !


Scratchpad

log files in JSON (easier parsing and future interop ?)
Trap standard error (AWS) and add full python traceback in output.txt
Fonctions lambda dans GH ?? (t3.large revenu!)
possible memory leak during images_to_samples.py  (with file ...; wrap in a Python lambda ?)
Log files renaming --> add the config file name in the log name

Log files renaming --> add the config file name in the log name

Active Learning
Active learning aims to facilitate the annotation process by letting the model choose candidates for new annotations. After a phase of training, the model runs inference on a stack of unlabeled images. We then calculate entropy mesure on the predicted sample and use samples with low entropy (or uncertainty) as new candidate for annotations. The annotator is put to contribution in order to accept/reject the proposed annotations. The accepted annotations are then incorporated into the training dataset and we start a new iteration of training.
How the annotations will be accepted/rejected is still to be determined...

Test the Intel Python distribution
Intel claims their Python distro is much faster for anything numeric. Since it can be installed with conda, can we give it a try ?

Federated learning
"In a typical machine learning system, an optimization algorithm like Stochastic Gradient Descent (SGD) runs on a large dataset partitioned homogeneously across servers in the cloud. Such highly iterative algorithms require low-latency, high-throughput connections to the training data. But in the Federated Learning setting, the data is distributed across millions of devices in a highly uneven fashion. In addition, these devices have significantly higher-latency, lower-throughput connections and are only intermittently available for training."
https://ai.googleblog.com/2017/04/federated-learning-collaborative.html
Could this be applied in a later stage, e.g. to optimize our model through time ?


autoUpdater error: Running NSIS installer from powershell has failed to start[bug]

Describe the bug
Some computers running Windows 10 encounter errors when automatic updates are required.
 Running NSIS installer from powershell has failed to start:  PermissionDenied

When I start it in administrator mode, it prompts the following error:
The application cannot start because the application's parallel configuration is incorrect.

Reproduction
No response
Expected behavior
No response
Full tauri info output
[✔] Environment
    - OS: Windows 10.0.19044 X64
    ✔ WebView2: 124.0.2478.80
    ✔ MSVC: Visual Studio ���ɹ��� 2022
    ✔ rustc: 1.76.0 (07dca489a 2024-02-04)
    ✔ cargo: 1.76.0 (c84b36747 2024-01-18)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: 1.76.0-x86_64-pc-windows-msvc (directory override for 'C:\Users\wangwei1240\mcs-platform')
    - node: 20.12.2
    - yarn: 1.22.22
    - npm: 10.5.0

[-] Packages
    - tauri [RUST]: 1.6.1
    - tauri-build [RUST]: 1.5.0
    - wry [RUST]: 0.24.7
    - tao [RUST]: 0.16.5
    - @tauri-apps/api [NPM]: 1.5.3
    - @tauri-apps/cli [NPM]: 1.5.10

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../dist
    - devPath: http://localhost:1420/
    - framework: React
    - bundler: Rollup
Done in 41.87s.

Stack trace
No response
Additional context
No response

related https://github.com/tauri-apps/tauri/discussions/9747


Live

Test PR

Docs Build status updates of commit 725bca0:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Test PR


Drop Py3.5 support in Dec 2019

Following https://github.com/numpy/numpy/pull/14086/files

Hm, based on downloads, I don't think this is a wise idea:

OTOH, my tests aren't even running against 3.5. The spirit of the proposal is for all python scientific libs to do this.
In the graph above, 3.7 is really taking off, so I think I will wait until late Dec 2019 to evaluate.


Connected peripherals don't work

I am working with a buttonless DFU configuration, so I only start the DFU process after I've connected, but the library starts out by trying to connect to the peripheral and never progresses.
It would be nice to have connected peripheral support added. For example, I've added a quick hack to my local copy of DFUExecutor's start function:
func start() {
        DispatchQueue.main.async(execute: {
            self.delegate?.didStateChangedTo(DFUState.connecting)
        })

        let centralManager = self.initiator.centralManager
        centralManager.delegate = self
        if peripheral.state == .connected {
            peripheral.delegate = self
            // Caller is responsible for having discovered the DFU service and characteristics
            for aService in peripheral.services! {
                initiator.logger?.logWith(.verbose, message: "Discovered Service \(aService.uuid) on peripheral \(peripheral)")
                if aService.uuid == SecureDFUService.UUID {
                    //First priority if SDFU
                    self.didDiscoverDFUService(true)
                    return
                }

                if aService.uuid == LegacyDFUService.UUID {
                    //Second priority is legacy DFU
                    self.didDiscoverDFUService(false)
                    return
                }
            }
            //No DFU found at this point, disconnect and report
            self.deviceNotSupported()

        } else {
            centralManager.connect(peripheral, options: nil)
        }
    }

@kklobe That's a good idea, thanks for reporting, we'll get this done in the next release!

Fix is on it's way now, available from version 2.1.6 and above !

Reopened after a issue #45 shows that this still doesn't work , probably was due to the fact that it fails due to services were already discovered and not the fact that peripheral was already connected.

I think it's a combination - this issue was originally because the connect call never succeeded (because it was already connected), in the same way #45 was failing to discover because the services were already discovered. I've modified DFUExecutor.swift to work with both connected and disconnected states - I'm happy to submit a pull request if you are interested!

Hi, the Version 3 is now published on our Version3 branch. You can edit your Profile and give the whole repo and branch path there. Could you try with this version?
Version 3 is now in beta phase and we will publish it when we get some feedback.
Thanks!

I'm closing the issue. Please, feel free to reopen the issue if you still have this problem.


Helm chart for Learn-Layer5

Yes right, let me craete a helm chart for this, so that we can version them too.
Originally posted by @kumarabd in https://github.com/layer5io/meshery-kuma/pull/71#discussion_r479001155

https://github.com/layer5io/learn-layer5/pull/51
Unable to link as this is in another repository.

Completed.


timeout() and socks() don't return Result

The Result was necessary in the past because they couldn't both be set,
this is not true anymore after 7493630ed88af5558b84a1f715890f67cfa4ec24

Needs a cargo fmt and then I will ACK this!

force pushed with cargo fmt


[Feature Request] Add copy operator to MXNet to CoreML convertor

As revealed in the issue #13965 currently only 12 operators are supported to convert from MXNet to CoreML.
    'FullyConnected' : _layers.convert_dense,
    'Activation'     : _layers.convert_activation,
    'SoftmaxOutput'  : _layers.convert_softmax,
    'Convolution'    : _layers.convert_convolution,
    'Pooling'        : _layers.convert_pooling,
    'Flatten'        : _layers.convert_flatten,
    'transpose'      : _layers.convert_transpose,
    'Concat'         : _layers.convert_concat,
    'BatchNorm'      : _layers.convert_batchnorm,
    'elemwise_add'   : _layers.convert_elementwise_add,
    'Reshape'        : _layers.convert_reshape,
    'Deconvolution'  : _layers.convert_deconvolution,

Based on the user request, we could add copy and similar commonly needed ops as well in this file
incubator-mxnet/tools/coreml/converter/_layers.py
@mxnet-label-bot add [Feature request]

Not sure how I add "+1" to this request, so I'm just adding this comment.  I'd like to convert a ResNet-50 model created by MXNet under the hood of SageMaker's Image Classification built-in algo.  Unfortunately, it uses "_copy" (and perhaps other unsupported layers).  Thus, the converter is unable to create a coreML model.

I have the exact same use case as @markproy above. Would be great to have this feature!

@markproy I have exactly the same issue with SageMaker Object Detection. Did you solve this problem in any way? (other converter etc.) ?

Same problem here!


Update LICENSE

Description
I changed the year of copyright from 2014 to 2019.

Thanks for the contribution. Updating the year of the license file is not necessary, though.


Update Composer dependencies (2022-02-25-00-06)

Visual regression test passed!
View the visual regression test report


Add delete command implementation details

Changes:

Added implementation details about delete commands in Developer Guide

Codecov Report

Merging #184 (4a8deb8) into master (28a801a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #184   +/-   ##
=========================================
  Coverage     71.89%   71.89%
  Complexity      787      787
=========================================
  Files           122      122
  Lines          2345     2345
  Branches        252      252
=========================================
  Hits           1686     1686
  Misses          574      574
  Partials         85       85

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 28a801a...4a8deb8. Read the comment docs.


[Metrics] Available Metrics: List Metrics

List available metrics in the Metrics Explorer page under the Available Metrics heading.

Fixed with #1211


Duplicated entries in alt_files of some Freesound audio

Description

I haven't confirmed if this is an ingestion issue, but it's probably just a matter of adding a validation step to the provider script, Freesound.
See for example:

https://api.openverse.engineering/v1/audio/f28de5b2-642b-49ac-8bb3-67869c38b58a/ and
https://api.openverse.engineering/v1/audio/2f9db0ec-6549-4e88-9950-bf6733f41fdb/

Additional context

Related to WordPress/openverse-frontend#1180
Resolution

[ ] 🙋 I would be interested in resolving this bug.

Nice find! I suspect there's some difference in the alt file metadata on fields we aren't tracking.

It looks like we're only ever setting 1 alt file: https://github.com/WordPress/openverse-catalog/blob/e8400cd418df6651f7ff242b02f8f28431ac252c/openverse_catalog/dags/providers/provider_api_scripts/freesound.py#L249-L257
I suspect this is an issue with the upsert logic:
openledger> select count(*), updated_on, created_on from audio where jsonb_array_length(alt_files) > 3 and updated_on != created_on group by 2, 3;
+-------+-------------------------------+-------------------------------+
| count | updated_on                    | created_on                    |
|-------+-------------------------------+-------------------------------|
| 7497  | 2022-03-01 01:52:41.358292+00 | 2022-01-20 14:35:10.795854+00 |
| 715   | 2022-04-01 02:53:42.330805+00 | 2022-01-20 14:35:10.795854+00 |
+-------+-------------------------------+-------------------------------+

All of the records where this occurred were originally added in our first run, and have only ever been updated 😞

Indeed, here's the upsert query that's running:
alt_files = COALESCE(
       jsonb_strip_nulls(old.alt_files)
         || jsonb_strip_nulls(EXCLUDED.alt_files),
       EXCLUDED.alt_files,
       old.alt_files
     ),

Here's an example where the two results are being ||ed
openledger> SELECT COALESCE(jsonb_strip_nulls('[{"a": 1}]'::jsonb) || jsonb_strip_nulls('[{"a": 1}]'::jsonb), '[{"a": 1}]'::jsonb, null);
+----------------------+
| coalesce             |
|----------------------|
| [{"a": 1}, {"a": 1}] |
+----------------------+
SELECT 1
Time: 0.090s

From the documentation (emphasis mine):

jsonb || jsonb → jsonb
Concatenates two jsonb values. Concatenating two arrays generates an array containing all the elements of each input. Concatenating two objects generates an object containing the union of their keys, taking the second object's value when there are duplicate keys. All other cases are treated by converting a non-array input into a single-element array, and then proceeding as for two arrays. Does not operate recursively: only the top-level array or object structure is merged.
'["a", "b"]'::jsonb || '["a", "d"]'::jsonb → ["a", "b", "a", "d"]

Looks like we'll need to find another way to manage that logic

WELL, it looks like we can make a function to accomplish this!
openledger> CREATE FUNCTION jsonb_array_distinct(jsonb) RETURNS jsonb AS $f$
   SELECT jsonb_agg(DISTINCT x) FROM jsonb_array_elements($1) t(x);
 $f$ LANGUAGE SQL IMMUTABLE;
CREATE FUNCTION
Time: 0.005s
openledger> SELECT jsonb_array_distinct('[{"a": 1}, {"a": 1}]'::jsonb);
+----------------------+
| jsonb_array_distinct |
|----------------------|
| [{"a": 1}]           |
+----------------------+
SELECT 1
Time: 0.019s
openledger> SELECT jsonb_array_distinct('[{"a": 1}]'::jsonb || '[{"a": 1}]'::jsonb);
+----------------------+
| jsonb_array_distinct |
|----------------------|
| [{"a": 1}]           |
+----------------------+
openledger> SELECT jsonb_array_distinct(COALESCE('[{"a": 1}]'::jsonb || '[{"a": 1}]'::jsonb, '[{"a": 1}]'::jsonb, NULL));
+----------------------+
| jsonb_array_distinct |
|----------------------|
| [{"a": 1}]           |
+----------------------+
SELECT 1
Time: 0.017s

YAAAAAAAAAY 🥳

I've asked a couple of different Postgres affinity groups and we'll see if they have any other suggestions. For next week, @WordPress/openverse-developers is a custom for this something we're comfortable with? We do already have precedent for other custom functions.

This looks like a good approach to me. 👍 The function seems wise since we definitely might need this elsewhere.

Great find @AetherUnbound! 💯 It seems quite reasonable to leave that job at the DB level since the upserting process already leverages many functions there 👍

Confirmed that this approach will also work to correct existing records (though we could always run it against them directly):
openledger> SELECT jsonb_array_distinct('[{"a": 1}, {"a": 1}, {"a": 1}]'::jsonb || '[{"a": 1}]'::jsonb);
+----------------------+
| jsonb_array_distinct |
|----------------------|
| [{"a": 1}]           |
+----------------------+
SELECT 1
Time: 0.028s

We...already have a function that does exactly that 😂 😭 🤦🏼♀
https://github.com/WordPress/openverse-catalog/blob/88322d2da852324f1147417ff8da332721a9002a/openverse_catalog/dags/common/storage/columns.py#L57-L65
So I guess this is indeed a bug - we just need to change whatever typing information the UPDATE code is using for alt_files.


[1496] Use site.save instead of save! when creating

Context
Adding new locations.
Changes proposed in this pull request
This stop the backend from blowing up and instead sending down the validation errors.
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 New training location form: validations

Are we missing specs that would have caught this?

@benilovj yeah, I deliberately didn't copy+paste from the update tests because I thought "oh, everything is the same, why test it twice..."

@benilovj added a spec


Static domain

Hi,
How can I get a static domain name instead of a constantly changing one with zrok? E. g. om18uulyhbaf.share.zrok.io every single time I start the service. If you try to compete with ngrok, please provide something that they don't.

Use a reserved share. You reserve the share name and it's yours forever.
https://docs.zrok.io/docs/concepts/sharing-reserved/

please provide something that they don't

zrok actually provides a number of things that ngrok does not. Investigate how private sharing works, for example... and how you can bind your shares into all kinds of environments and in ways you cannot with some of the competitors.

Sounds interesting. I will try to continue testing.


Create mpas section

What this PR does / why we need it:
If implemented this will add a mpas section that explains how to install and get started with mpas.
Which issue(s) this PR fixes:
Fixes #137
Special notes for your reviewer:
Release note:

all the TODOs have now corresponding issues so I removed them.


A question about netconf connection error mentioned in #1411

Hello, I also encountered the problem described in this issue.
And I have solved it depending on the conversations in the link.
But I'm curious about why this problem happens when we try to login from netconf client, rather than installing yang models on netconf server. Does the netconf client compare the version of yang model from netconf server with the local yang models?
Any information would be appreciated, thank you.

The NETCONF client needs access to the YANG models which are installed on the server, otherwise it won't understand the structure (and meaning) of the data. For that, the client retrieves the YANG models from the server (or it uses a local cache), and when there's an error in these YANG models, the client won't connect successfully. Fix the YANG models and install the fixed version on the NETCONF server.

So when there are multiple versions of YANG models installed in NETCONF server, it is necessary for us to specify a specific version when importing them, otherwise errors will inevitably occur, right?

So when there are multiple versions of YANG models installed in NETCONF server, it is necessary for us to specify a specific version when importing them, otherwise errors will inevitably occur, right?

No, in that case the latest revision is always imported. That may be a problem for modules that expect the older revision to be imported and those are the ones that must import by revision.

Thank you for your quick reply.
By the way, where is the local cache used by NETCONF client?
In this link you said delete ~/.netopeer2-cli/modules/, but I only have two files "config.xml" and "history" there. I'm using netopeer2-2.1.36

The default search directory used is a cmake variable called YANG_MODULE_DIR and it is /usr/local/share/yang/modules/netopeer2 by default. So if all the modules are found there or in the explicit search directory, no modules need to be additionally cached.

Well, based on your response, when we have two versions of ietf-crypto-types.yang exist in NETCONF server, ietf-keystore.yang should pick the newer one, so everything should be OK. But this problem happens, which means there are other possible reasons😂.
I'm just curious, any guess from you is enough.

There is another scenario possible when the first time the import is being resolved a module is imported by revision. Then, other imports without revision will use the previously imported module. The idea is for all the modules to import a single module revision, if possible.

OK, that makes sense.
Thank you for your patience.


Implement circle sort

http://www.geeksforgeeks.org/circle-sort/

Hi! Can I work on this?

https://github.com/Eckankar/PySortDemo/pull/21

@StegSchreck No problem. I’ve had a look at your code and fiddled around a bit with the markers.
I think the algorithm is working much faster than the display is redrawn. That’s why the markers appear in weird places, or not at all. Only a guess, though. I don’t have any prior experience with pygame.

The visualization doesn't work correctly due to the algorithm not being implemented as an iterator. :) See https://github.com/Eckankar/PySortDemo/blob/master/CONTRIBUTING.md for more information about structuring the algorithms.


SWA: Installing the Git binary should be added to the Prerequisites section.

Installing the Git binary should be added to the Prerequisites section.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 59b3b161-b559-f90b-1566-f18776a5f7da
Version Independent ID: 3e536234-16ad-2ed3-fb58-c128ba2faab2
Content: Quickstart: Building your first static site with the Azure Static Web Apps
Content Source: articles/static-web-apps/getting-started.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

Thanks for the PR! I have assigned the issue to the doc author to update appropriately.


Erro with makenew.sh on OSX

Hi,
When I'm run makenew.sh occurs this erro:
sed: 1: "13i My description": command i expects \ followed by text

How to solve?

It's due to the differences between sed in unix and GNU sed. This will affect makenew.sh in the other projects too.
I don't have a Mac, but I can use FreeBSD (which should have the same behaviour) to test and fix it.

@silasrm Can you test if it works for you now on the mac-sed branch?
If so I'll update master and tag a new version.
It's working on FreeBSD, but I need to be sure it works on Mac before I update all the other scripts.

Hi,
With the mac-sed branch, makenew.sh run without error 💃


Improve directory insertion & deletion (RIPD-1353, RIPD-1488)

This commit introduces the SortedDirectories amendment, which addresses two distinct issues:
First, it corrects a technical flaw that could, in some edge cases, prevent an empty intermediate page from being deleted.
Second, it sorts directory entries within a page (other than order book page entries, which remain strictly FIFO). This makes insert operations deterministic, instead of pseudo-random and reliant on temporal ordering.
Lastly, it removes the ability to perform a "soft delete" where the page number of the item to delete need not be known if the item is in the first 20 pages, and enforces a maximum limit to the number of pages that a directory can span.

Finally!

The non-unity builds are failing:
/home/travis/build/ripple/rippled/src/test/ledger/Directory_test.cpp: In member function ‘void ripple::test::Directory_test::testDirectoryOrdering()’:

/home/travis/build/ripple/rippled/src/test/ledger/Directory_test.cpp:161:48: error: ‘BookDirs’ was not declared in this scope

                 Book({xrpIssue(), USD.issue()}));

                                                ^

/home/travis/build/ripple/rippled/src/test/ledger/Directory_test.cpp:164:38: error: unable to deduce ‘auto&&’ from ‘book’

             for (auto const& offer : book)

Codecov Report

Merging #2165 into develop will decrease coverage by 0.06%.
The diff coverage is 81.85%.

@@             Coverage Diff             @@
##           develop    #2165      +/-   ##
===========================================
- Coverage    68.98%   68.92%   -0.07%
===========================================
  Files          684      685       +1
  Lines        50461    50581     +120
===========================================
+ Hits         34812    34863      +51
- Misses       15649    15718      +69

Impacted Files
Coverage Δ

src/ripple/app/main/Amendments.cpp
100% <ø> (ø)
:arrow_up:

src/ripple/ledger/View.h
100% <ø> (ø)
:arrow_up:

src/ripple/ledger/ApplyView.h
100% <ø> (ø)
:arrow_up:

src/ripple/app/tx/impl/CancelTicket.cpp
100% <100%> (ø)
:arrow_up:

src/ripple/protocol/STVector256.h
100% <100%> (ø)
:arrow_up:

src/ripple/protocol/impl/STTx.cpp
81.46% <100%> (ø)
:arrow_up:

src/ripple/protocol/impl/Feature.cpp
100% <100%> (ø)
:arrow_up:

src/ripple/app/tx/impl/Transactor.cpp
81.46% <33.33%> (+0.25%)
:arrow_up:

src/ripple/app/ledger/impl/ApplyView.cpp
78.15% <78.15%> (ø)

src/ripple/ledger/impl/View.cpp
86.09% <82.08%> (-2.67%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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

:+1:

👍

In 0.80.0-b4


Remove OpenMMEngineSettings

Description
As these settings are not used in this protocol, we should remove them.

Maybe we could instead have a GromacsEngineSettings class?


Inline translation functionality missing from what users can / cannot do table

Please add to the table of things users can do the translate inline messages functionality

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5f080a5c-4268-1db4-566f-f1edc27920b5
Version Independent ID: ac556a92-58af-3e05-f53b-0fc00afc96c0
Content: Use guest access and external access to collaborate with people outside your organization - Microsoft Teams
Content Source: Teams/communicate-with-users-from-other-organizations.md
Service: msteams
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@TristanBa thank you for reaching out and helping improve MS Docs. Just for clarity should that go under the "things your users can do" table and is the functionality available for external access users and guests?

Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


use safer intermediate filenames

Is your feature request related to a problem? Please describe.
currently, the directory _pickles is created as is a file named .all_contigs.fna. it is possible that these filenames will already exist in a user's working directory
Describe the solution you'd like
a randomized, unused name should be used to ensure that other intermediate files are not removed
Additional context
it is important that the pickles are still retrievable as that directory is crucial for resuming from a checkpoint. the .all_contigs.fna file is only used by isPcr and can be safely renamed during other runs. make sure that cleanup can still find these files and delete them.

fixed in v1.4.3


big commit

Here's the functionality as requested by Adam:

image, description, header, and link text to equity page is different between main page and state page
admin can edit main page link to equity page image, description, header, and link text in the main page editor
admin can edit state page link to equity page image, description, header, and link text in the state page editor

 Separate Equity section from main Equity page


load login page after successful registration

This is a possible solution for #181

1 second might be too short a time to see registration is successful. A button to login is a better solution

Sure, lemme add that with swal

Btw you don't get a session after registration right now...
... and hiding sweetalert's button during success also hides the button for registration

The "login" button after registration you added is awesome! Better than my solution - I'm closing this PR and deleting the branch


install script create systemd service with a break line

When installing latest create_ap package, it is created a /usr/lib/systemd/system/wifi.service
The resulting script contains a ExecStart= broken up in 2 lines. I have not tested this but I don't think systemd is gonna like that.
May be just remove the \ in the install script ?
I found the creation of that service questionnable because the install script attempts to guess things, and sometimes is wrong.
Moreover, it would be better named create_ap.service

You're talking about the AUR package right? I'm not the maintainer of that. Please comment on the AUR package.

oh sorry about the confusion


Add a new lxc config device override command

Hm, not a huge fan of another subcommand for this very special case.

Accidentally pressed close.

Well, this very special case hits me every time I demo LXD.
The most common cases where you hit it is if you want to set a key on eth0 like ipv4.address or ipv6.address which must be set on a per-container basis. The same is true if you want to set the size property on the root device.
Before this subcommand, your only option was to run lxc config show --expanded to see the entire device you want to override, then create an override device with the exact same option using lxc config device add and adding the one thing you want to change on top of that.
Now you can do it with just:
lxc config device override c1 eth0 ipv4.address <IP_ADDRESS>


Why enclosing Variable inside lambda instead of the variable only?

First of all thanks a lot for your amazing autodiff post.
For Nth order derivatives you wrote 2.To change local_gradients to contain functions, instead of local gradient values, I think that if you only store a Variable with the weight/local_gradient, it would be sufficient as the lambda function only multiplies the path value by a constant factormultiply_by_locgrad, so get_gradients would use path_value * local_gradient where both are of Variable type.
side-note:
I have combined that with a tape/array for storing variables to save memory, but the combinatorial explosion in the number of possible paths in the graph does too much allocation especially for nth-derivative.

Hey, nice to hear from you; glad the post was helpful.
The lambda is used because local derivatives are being computed using Variable operations. It adds laziness, which prevents infinite recursion. For example:
def mul(a, b):
    value = a.value * b.value
    local_gradients = (
        (a, lambda path_value: path_value * b),
        # ^  the * operator results in this function, `mul` being called
        # the lambda prevents infinite recursion -> stackoverflow
        (b, lambda path_value : path_value * a)
    )
    return Variable(value, local_gradients)

mul uses mul to compute its local gradient, which in turn uses mul to compute its local gradient, and so on. This would recurse until stack overflow.
You can always give it a try without the lambda, to see what happens.

Nice that you added a gradient tape! That is what the more low level libraries do. As for saving memory, variables in the article aren't duplicated, but referenced, so memory usage isn't too bad.
The combinatorial explosion is a shame :D. Originally I had Nth order derivatives in this repo (SmallPebble), but it made things too slow, so I removed it.

I mean using the same old definition with only changing the variable type:
def mul(a, b):
    value = a.value * b.value
    local_gradients = (
        (a, b),
        # Now (path_val * b).value is reserved
        (b, a)
    )
    return Variable(value, local_gradients)

def compute_gradients(variable, path_value):
        for child_variable, local_gradient in variable.local_gradients:
            # "Multiply the edges of a path":
            value_of_path_to_child = path_value * local_gradient
            # "Add together the different paths":
            gradients[child_variable] += value_of_path_to_child
            # recurse through graph:
            compute_gradients(child_variable, value_of_path_to_child)

I am searching for ways to factor paths that Factoring Paths mentions. I can think of optimizing x*x to x.val*x.val with one gradient x.grads = ((x,var(2)*x)) and the same for +, but I don't know any more general rule without symbolic manipulation.
Thanks for your helping.

Ah, I could have picked a better example than mul, you are correct that we don't need lambdas for mul to work.
Here's div:
def div(a, b):
    value = a.value / b.value
    local_gradients = (
        (a, lambda path_value : path_value * ONE/b),
        # ^ ONE/b calls div
        (b, lambda path_value : path_value * NEG_ONE * a/(b*b))
    )
    return Variable(value, local_gradients)

Here the weights are computed using div. So if we compute them immediately, we get infinite recursion.

Thanks for sharing the Factoring Paths link, that's a great post.
Note that we do bypass the combinatorial explosion of paths (discussed here in the post), simply by using reverse-mode autodiff.

{“forward-mode differentiation” and “reverse-mode differentiation” are} algorithms for efficiently computing the sum by factoring the paths. Instead of summing over all of the paths explicitly, they compute the same sum more efficiently by merging paths back together at every node.

What you are attempting is a step even further than this. On first glance, I'm not sure that you will see enormous benefits from doing it. You could for example, define a pow operator, to enable backprop of operations like a*a*a*a*a to be more efficient.

Alright I see that, but you could make new variable for any recursive case like this: (a, Variable(1/b.value , ()) and (b, Variable(-a.value / b.value / b.value, ()).

Aha, I got it. I was thinking of only one node.
Thanks

I think that creating new Variables will lead to errors in the higher order gradients, because these are computed using the local gradients of the weights.
If you are planning to do gradient based optimisation, one way you could keep things fast is to only compute first derivatives, and estimate higher order derivatives numerically (this is the intuition behind optimisation methods such as Adam).
Cool that you are implementing this in Lisp!

Thanks for your time. I would look further into these.


Bug: v5.15.0 Schema failing on manual Docker Image build

Description:
As discussed, we build the bloodhound docker image internally via proxies which previously has never caused any issues.
When building the v5.15.0 image, we are experiencing SQL schema errors where constraints are being violated.
Are you intending to fix this bug?
No
Component(s) Affected:

Neo4j
PostgreSQL

Steps to Reproduce:

Git Pull Repo
Switch to v5.15.0 tag branch
docker build -t bloodhound:latest -f dockerfiles/bloodhound.Dockerfile .
See detailed error below

Expected Behavior:
Bloodhound image is able to start.
Actual Behavior:
I can see in the logs the app-db runs through the schema queries here - https://github.com/SpecterOps/BloodHound/blob/v5.15.0/cmd/api/src/database/migration/migrations/v5.15.0.sql
But Bloodhound is not able to start
Screenshots/Code Snippets/Sample Files:
bloodhound-1  | {"level":"error","time":"2024-09-12T07:01:43.509870758Z","message":"Error during SQL database migration phase: could not execute migrations: failed to execute migrations for v5.15.0: ERROR: null value in column \"role_id\" of relation \"roles_permissions\" violates not-null constraint (SQLSTATE 23502)"}
bloodhound-1  | {"level":"fatal","time":"2024-09-12T07:01:43.509888041Z","message":"Failed starting the server: failed to start services: rdms migration error: could not execute migrations: failed to execute migrations for v5.15.0: ERROR: null value in column \"role_id\" of relation \"roles_permissions\" violates not-null constraint (SQLSTATE 23502)"}

Environment Information:
BloodHound: v5.15.0
OS: Ubuntu 22.04
Neo4j: v4.4
PostgreSQL: v13.2
Docker version 27.2.1, build 9e34c9b
Additional Information:
No differences in docker build, just changes in apt urls etc.
Potential Solution (optional):
If you have any ideas about what might be causing the issue or how it could be fixed, you can share them here.
Related Issues:
If you've found related issues in the project's issue tracker, mention them here.
Contributor Checklist:

[x] I have searched the issue tracker to ensure this bug hasn't been reported before or is not already being addressed.
[x] I have provided clear steps to reproduce the issue.
[x] I have included relevant environment information details.
[x] I have attached necessary supporting documents.
[x] I have checked that any JSON files I am attempting to upload to BloodHound are valid.

As a workaround, I removed the NOT NULL requirement on the role_id in the roles_permissions table.
https://github.com/SpecterOps/BloodHound/blob/ca6c0831a8d2c8f5b2e27fd2eb68f760876a3e46/cmd/api/src/database/migration/migrations/schema.sql#L393

Would it also be possible to provide the Read-Only role the ability to read saved queries?
https://github.com/SpecterOps/BloodHound/blob/ca6c0831a8d2c8f5b2e27fd2eb68f760876a3e46/cmd/api/src/database/migration/migrations/v5.15.0.sql#L53

We'll get that fixed up - also, thanks for the find that RO users didn't get visibility to globally shared queries. Fixing that here: https://github.com/SpecterOps/BloodHound/pull/858

Apologies again, this seems like it was an internal GitHub Sync issue. Closing


Raspberry Pi, testing?

Nice project! Let me know if you need testing on a Raspberry Pi!

We are having problems with the cross-compiler, it can be able to be compiled from scratch for x86 & x64 but not for ARM. Could you be able to review it? I didn't worked with ARM before, and following the instructions of clfs I can't be able to find the clue... :-/

Probably not, sorry. But if I get some time, I can try to compile for x86 to get more familiar with the process, and maybe then ... =)

Cool then! :-)

By the way, as you can see on the readme, just by executing npm install will create QEmu files for you current platform, probably 64 bits. If you want to compile them for x86 32 bits, you can use the CPU or PLATFORM environment variables.

Thanks!

You are welcome :-)

Will it happen ?

I'm too much busy lately to work on the project, but pull-requests are welcome :-)

If you could tell me what and where to help out, I would give it a try


Error when running python hydra agent

Hydra connection expects a string as a value but a dictionary is passed instead.

I don't think, there is any that type of error. Anybody else is getting the same error?  @PrajwalM2212  Can you provide me a full log(include input url)? And make sure you haven't changed anything in your local.

@sandeepsajan0 redis_connection.set("EntryPoint", entrypoint_properties) -- entrypoint_properties is a dictionary. Line-58, hydra_graph.py

I don't see why are we setting EntryPoint as key separately when we have an entrypoint node in the redisgraph. It's not used anywhere in the code. Moreover, the user is not going to get the properties of entrypoint node from redis-cli. I think we should just remove this line of code. Maybe we should create another query which fetches the details about entrypoint node (something like show entrypoint).

Yes @ShravanDoda, I think it'll be a better way. If this line is making any error we can remove it by an additional query also. Because now I can't see any role of this key and I don't remember why we have put it there.

fixed in #94


WIP: DO_NOT_MERGE: lets see if e2e-ibmcloud-csi can succeed

ci/prow/e2e-ibmcloud-csi fails again in again in PR#47. Can it succeed at all?

ci/prow/e2e-ibmcloud-csi is perma-failing. See my comment here: https://github.com/openshift/ibm-vpc-block-csi-driver/pull/47#issuecomment-1765274247


support  colors

in GitHub actions:

in VS Code with this extension:

Can you please attach an example as a text file?

Indeed, this is now fixed in 1.1.4, thanks @iliazeus!

test file, fyi: 11_Run make test-e2e-ci.txt


Consider adding LICENSE file to repo

The code makes it clear what that the license is MIT, but it would be nice if you included a LICENSE file in the repo so that it's easier to find.
Thanks!

@tcdowney thanks for this, and good suggestion - if you are happy to PR this, I am happy to merge in.


Add a free license

This repo seems to lack license information. I suggest AGPLv3 :)

Well, since he based it on the powershell script he'll need to look at that license.

Well, since he based it on the powershell script he'll need to look at that license.

It's not based on that script, but it surely gave me the aspiration so I thought it deserves credit. :)
I'll ask the authors of the merged pull requests to see if they are okay with MIT License.

Done!


Don't confuse open / close in strings

NOTE: I've not started working on this, so if this is a problem for you, please upvote this by adding a comment or an emoji.
The current algorithm for nested pairs (those where the open and close are different, like parens and braces) simply counts open and cloes as it finds them.  But it could be smarter by paying attention to the syntax of each character (ppss).
For example:
(message "finding (, huh?")
                  ---------

If the cursor is anywhere past the paren inside the string, it will incorrectly identify the underlined portion as the pair instead of matching with the first paren before "message".
For this particular case, it would work if we use syntax-ppss and note that we started outside of a string, so we should ignore the paren inside the string.  Would work for skipping over comments too.
On a similar note, we could figure out which direction to search for quotes.  If we are on a closing quote in a comment and the next quote is not in a comment, we should try the other direction.

does syntax-ppss work with TreeSitter?

Good question @ckruse.  I believe it does.
I considered using tree sitter originally, but started with this code since I also use matching inside of strings and quotes.
It might not hurt to use tree-sitter if the point is not in a comment or string, however.  It would be foolproof, then.  I'll look into that.


php74: Enable zip support

Motivation for this change
This fixes #75427
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Notify maintainers
cc @

cc @aanderse

Haven't tested, but it builds so that seems like a good indicator. Thanks @etu!


Is there a way to scan repositories on github?

I have many repos under github and I want to scan them all for dependency check. Is there a better way of doing it?
And also want to integrate dependency check for  all the repos during the CI/CD

yes, define the maven plugin in your projects and run your builds on your CI service


Rename contents to copyToRoot

Since the nix2container beginning, the nix2container contents behavior
differed from the dockerTools.buildImage behavior.
See https://github.com/NixOS/nixpkgs/pull/179801 for details.

Hm, maybe we should we keep the contents attribute and print a deprecation warning...

I finally added a deprecation warning when contents is used.


Typo remove extra

I refactored the project! I guess the problem is fixed in the latest release! Thanks for your contribution! 👍🏼


TextStyle of Text: when i set height: 1.1 for Text line gap, it is lower than nothing to set

Flutter version: 1.12.13+hotfix.8
Set the height: 1.1
Text("Didn't find something you like?",style: TextStyle(height: 1.1)

Set noting for height
Text("Didn't find something you like?")

Hi @a530148758
It's said in doc about this https://api.flutter.dev/flutter/painting/TextStyle-class.html
in the 'Line height' paragraph:
For most fonts, setting height to 1.0 is not the same as omitting or setting height to null.
Closing, as this isn't an issue,
if you disagree please write in the comments and I will reopen it
Thank you

I mean if set height: 1.1, there should be a small line spacing between each line, right? but no.  it is normal on flutter 1.7.8 and 1.9.1


Issue#829: class extends null should bind this

A class that extends null should have 'this' binding in its constructor same way as a base class. Add new function attribute 'BaseConstructorKind' to capture 'extends null' cases during class definition evaluation. Add new opcode for branching on 'extends null' scenario for 'this' binding.
Fixes #829

https://github.com/Microsoft/ChakraCore/issues/829

@boingoing @Microsoft/chakra-es @Microsoft/chakra-developers could you take a look please?

Looks good

@dotnet-bot test Windows ci_slow_x64_debug Windows 7 ci_dev12_x64_debug please

@dotnet-bot test Windows 7 ci_dev12_x64_debug please

@dotnet-bot
test Windows 7 ci_dev12_x64_debug please

@mmitche Do you know why the checks I added in a separate branch (release/1.4-ci) (see #2326) have been created for this branch? I was hoping to stage that change but not have it applied to all branches yet.
/cc @rajatd
I'll double check that I don't have a bug in my script.

@dilijev thanks. Good to know.

You can now resolve the CI issues as described here: https://github.com/Microsoft/ChakraCore/pull/2332#issuecomment-271029950

Merged.
@tcare @dilijev thanks!


Problem with composer update

I have:
Composer (v1.0.0)
Composer-asset-plugin (v1.1.3)

try to use:
composer update
and got the error:
[ReflectionException]
Class Fxp\Composer\AssetPlugin\Repository\NpmRepository does not exist

[ErrorException]
Declaration of Fxp\Composer\AssetPlugin\Repository\AbstractAssetsRepository::whatProvides() should be
compatible with Composer\Repository\ComposerRepository::whatProvides(Composer\DependencyResolver\Poo
l $pool, $name, $bypassFilters = false)

I found a solution:
composer global update fxp/composer-asset-plugin --no-plugins
it's works for me, but only in composer.json looks like:
{
"config": {
        "secure-http": false
    },
    "require": {
        "fxp/composer-asset-plugin": "dev-master"
    }
}

works fine for me with the following versions:
$ composer --version
Composer version 1.1-dev (89cfde051d70a02b9335fd79ce41bb7b3090a365) 2016-04-19 10:00:39
$ composer global show -i
fxp/composer-asset-plugin v1.1.3 NPM/Bower Dependency Manager for Composer

I got the following error message:
[ErrorException]                                                                Declaration of Fxp\Composer\AssetPlugin\Repository\AbstractAssetsRepository     ::whatProvides() should be compatible with Composer\Repository\ComposerRepo     sitory::whatProvides(Composer\DependencyResolver\Pool $pool, $name, $bypass     Filters = false)
$ composer --version
Composer version 1.0.2 2016-04-21 12:30:18
$ composer global show -i
fxp/composer-asset-plugin v1.1.4 NPM/Bower Dependency Manager for Composer

After removing composer.lock and vendor folder everything works fine.

Great!


NX-OS syntax for ICMPv6 is wrong

Using Aerleon v1.70.0.
If you have a policy file like this:
---
filters:
- header:
    comment:
    targets:
      cisconx: icmp_v6-out inet6
  terms:
  - name: accept-icmp
    protocol: icmp6
    action: accept

And you generate the ACL, the code looks like this:
! $Id:$
! $Date:$
! $Revision:$
no ipv6 access-list icmp_v6-out
ipv6 access-list icmp_v6-out
 remark "$Date:$"
 remark "$Revision:$"

 remark accept-icmp
 permit icmp6 any any

exit

However, icmp6 is not valid syntax for NX-OS (tested on version 10.1(1)).
Config Session started, Session ID is 1
Enter configuration commands, one per line. End with CNTL/Z.
router(config-s)# no ipv6 access-list icmp_v6-out
ERROR: ACL with given name and type does not exist
router(config-s)# ipv6 access-list icmp_v6-out
router(config-s-ipv6-acl)#  remark "$Date:$"
router(config-s-ipv6-acl)#  remark "$Revision:$"
router(config-s-ipv6-acl)#
router(config-s-ipv6-acl)#
router(config-s-ipv6-acl)#  remark accept-icmp
router(config-s-ipv6-acl)#  permit icmp6 any any
                                                          ^
% Invalid parameter detected at '^' marker.
router(config-s-ipv6-acl)# abort

It must be icmp, without the 6. But if I change the protocol in the policy file, it throws an error during compile:
I0408 17:02:10.246660<PHONE_NUMBER>22272 aclgen.py:447] rendering one file
I0408 17:02:10.260283<PHONE_NUMBER>22272 plugin_supervisor.py:253] 0 plugins active.
I0408 17:02:10.260336<PHONE_NUMBER>22272 plugin_supervisor.py:254] 30 generators registered.
W0408 17:02:10.260611<PHONE_NUMBER>22272 cisco.py:458] Term accept-icmp will not be rendered, as it has icmp match specified but the ACL is of inet6 address family.

Yea searching for documentation it seems we just need to output icmp as you said. This should be a simple fix we can get in.

Just wanted to get back to you @mf84 it looks like what cisco is looking for is protocol icmpv6
try
  terms:
  - name: accept-icmp
    protocol: icmpv6
    icmp-type: echo-reply destination-unreachable time-exceeded
    action: accept

and you will see it works
Then try replacing icmpv6 with icmp6` and you will see it errors out
error encountered in rendering process:
Error generating target ACL for policies/pol/foo.yaml:
icmp-types specified for non-icmp protocols in term:  accept-icmp

You can also put in a BS protocol
  terms:
  - name: accept-icmp
    protocol: asdmo1m
    action: accept

results in the following
! $Id:$
! $Date:$
! $Revision:$
no ipv6 access-list icmp_v6-out
ipv6 access-list icmp_v6-out
 remark "$Date:$"
 remark "$Revision:$"

 remark accept-icmp
 permit asdmo1m any any

exit

While the above is a workaround we are discussing the solutions to this as there is inconsistency in how policies are defined now.

I see thanks for the clarification. If I search for "icmpv6" in the aerleon docs, I only get one page:
https://aerleon.readthedocs.io/en/latest/reference/generators/
And there it is under the "Juniper EVO" section. It should be made clearer which protocols are available.


'GetAllLocations' changed to increase performance

@CodiumAI-Agent /describe

Title
'GetAllLocations' changed to increase performance

User description
@CodiumAI-Agent /describe

PR Type
enhancement

Description

Enhanced performance of GetAllLocations methods by changing IEnumerable<string> to IReadOnlyList<string> and optimizing loop structures.
Improved string concatenation in MiKo_2034_CodeFixProvider.cs by using ConcatenatedWith.
Added additional checks for text length in MiKo_2040_LangwordAnalyzer.cs to avoid unnecessary processing.

Changes walkthrough 📝
Relevant filesEnhancement

DocumentationAnalyzer.csOptimize GetAllLocations method for performance improvements

MiKo.Analyzer.Shared/Rules/Documentation/DocumentationAnalyzer.cs
Changed IEnumerable to IReadOnlyList for performance improvement.  Optimized loop structures and variable usage for better performance.  Added checks for empty or short text to avoid unnecessary processing.

+55/-14

MiKo_2034_CodeFixProvider.csImprove string concatenation method for clarity

MiKo.Analyzer.Shared/Rules/Documentation/MiKo_2034_CodeFixProvider.cs
Replaced string.Join with ConcatenatedWith for string concatenation.

+1/-1

MiKo_2040_LangwordAnalyzer.csAdd text length check in LangwordAnalyzer

MiKo.Analyzer.Shared/Rules/Documentation/MiKo_2040_LangwordAnalyzer.cs

Added length check before whitespace check for text tokens.

+2/-1

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information


Fixed small typo in agents notebook

What does this PR do?

Fixes #142.
Small typo fixed.
I'm learning a lot from these notebooks! 😄 Thanks for providing them to the community
Who can review?
@merveenoyan and @stevhliu.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


chore: Refactor Types out of kona-derive

Description
Refactors types out of kona-derive and into kona-primitives.
Leaves Batch types in a lifted batch module within kona-derive.
Updates all dependencies and bumps versions.
Also introduces the online feature for kona-primitives in order to support online sidecar functionality. Maybe we should rename this feature flag? It's a relic of being part of kona-derive's "online" providers feature.

#454  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @refcell and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.3%. Comparing base (d1b63f8) to head (185dc25).

Additional details and impacted files

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


Documentation improvement - Distributed tracing - Elastic APM

Hello,
I cam across an issue where Elastic APM was not able to link the calls between my BFF (using Yarp) and my backend
I tried to follow the suggestion here:
https://microsoft.github.io/reverse-proxy/articles/distributed-tracing.html#pass-through-proxy
but it did not help
But by following the suggestion from Elastic Team:
https://discuss.elastic.co/t/elastic-apm-yarp/347667/2
It worked!
Here is the code that I used in the end:
builder.Services.AddReverseProxy()
    .LoadFromConfig(builder.Configuration.GetSection("ReverseProxy"))
    .ConfigureHttpClient((context, handler) => handler.ActivityHeadersPropagator = DistributedContextPropagator.CreateDefaultPropagator());

You could add it to the documentation just in case of similar issue
Thank you

Thanks for the report. It appears Elastic's .NET integration is relying on the existing header being set on the request, and not on the ambient Activity. This is unlike other integrations we've tested against.
I don't believe there's much we can do by default that would work for both scenarios.
Documenting that changing ActivityHeadersPropagator back to default works for Elastic in the existing tracing doc makes sense.


Extra use line in component templates

Screenshot

For the next version, it's line 20 in website/docs/more/development-tips.mdx
For 0.19, it's line 20 in website/versioned_docs/version-0.19.0/more/development-tips.mdx
The page doesn't exist in the docs for 0.18.
Since the code block has ,ignore, it's safe the just remove the line

We should just replace that template with one for function component


[Bug] Ensure install-plugin.sh compatibility with sh on Debian #5630

Purpose of this pull request

This PR remedies the "Bad substitution" error encountered when executing install-plugin.sh under dash on Debian systems.
Fixes  #5630
Does this PR introduce any user-facing change?

No
How was this patch tested?

manually run sh bin/install-plugin.sh 2.3.3
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Cool! Could you open CI in your fork repository?

Cool! Could you open CI in your fork repository?

enabled, how can I trigger it?

Cool! Could you open CI in your fork repository?

enabled, how can I trigger it?

add some empty commit

Cool! Could you open CI in your fork repository?

enabled, how can I trigger it?

add some empty commit

done

cc @EricJoy2048


copr: Support Vector data type

What is changed and how it works?
Issue Number: ref #17290
Support reading table data contains Vector data type
This PR should be merged before TiDB. Otherwise TiDB cannot read out the table containing Vector data through TiKV Coprocessor.
What's Changed:
The VectorFloat32's Datum / Chunk / Memory (in TiDB) layouts are identical, as follows:

The VectorFloat32 is simply defined as follows in Rust:
pub struct VectorFloat32 {
    pub value: Vec<OrderedFloat<f32>>,
}

pub struct VectorFloat32Ref<'a> {
    // 引用值
    value: &'a [OrderedFloat<f32>],
}

See:  https://github.com/EricZequan/tikv/blob/vector-type/components/tidb_query_datatype/src/codec/mysql/vector.rs
Detailed tech design:  https://docs.pingcap.com/tidbcloud/vector-search-data-types
Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

/test all

/cc @breezewish

/ok-to-test

@EricZequan Please try your best to verify the compiling and unit test work locally, instead of using CI. It would be a huge afford if everyone use CI as a development environment.
To compile and test locally, simply:
make dev

/retest

/test idc-jenkins-ci/test

/run-all-tests


Settings ServiceInfo in order to enable the reuse of java-support in other JVM languages

Enables us to customize the information reported in ServiceInfo

@viktorklang approved ?

Thanks @sleipnir! :)

It's nothing!


Add to HACS default repositories so no custom repository adding is needed

Home Assistant version
Enhancement of installation, not a problem with project
SmartIR version
1.13.2
SmartIR configuration
Enhancement of installation, not a problem with project
Add configuration here
Enhancement of installation, not a problem with project
Describe the bug
Add to HACS default repositories so no custom repository adding is needed.
I would love to all new people to just see it in standard HACS store. Even better, add it to Home Assistant plugins.
Thank you for hard work
Debug log

Add your logs here.

Additional context
Add any other context about the problem here.

There are specific reasons why SmartIR is not added to HACS as default repository. Thanks


remove datacenter var as vm.power does not work with relative vm path…

Manually merging as the code coverage is complaining about log statements and the e2e test flow is verified

/approve


Icon does not show in new project

I'm trying to set a custom icon for our project template but no matter what I do visual studio still shows the default icon for my projects.
Is there anything else beside adding the .ico file inside zip that I need to do?
Do I need to provide multiple sizes and qualities of icons for this to work?
Thanks.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 397b0790-ed8b-822d-931a-98a06c594ec6
Version Independent ID: f1b1aa72-1bd7-c0b1-3d77-5e9b978a9b11
Content: Creating Custom Project and Item Templates - Visual Studio
Content Source: docs/extensibility/creating-custom-project-and-item-templates.md
Product: visual-studio-windows
Technology: vs-ide-sdk
GitHub Login: @gregvanl
Microsoft Alias: gregvanl

@madskristensen Could you help with this question?

The .ico or .png file should be in the zip file AND the icon must be references from within the .vstemplate file like this sample

@madskristensen thanks for reply, I tried that and checked the zip file to make sure it contains the icon file but VS completely ignores my icon and still shows the default custom template icon. Are there some sort of validations for image quality like 32/16 bit or size 16/32pix? I see some guidelines here but not sure if they are compulsory.
https://docs.microsoft.com/en-us/visualstudio/extensibility/ux-guidelines/images-and-icons-for-visual-studio?view=vs-2019
https://docs.microsoft.com/en-us/cpp/windows/creating-an-icon-or-other-image-image-editor-for-icons?view=vs-2019
Thanks.

@batizar If you continue to have a problem with the icon, it's probably best to file a report at Developer Community. Please use the Report a Problem tool. When you do this, your report becomes part of the work items that are tracked on Visual Studio Developer Community, which allows you to engage directly with the engineering team while they address and fix the problem.

It could be that there are restrictions on the .ico file. Try with a square .png and see if that makes a difference.

Thanks for the feedback. If this above discussion hasn't helped you solve this issue, please use the Report a Problem tool. When you do this, your report becomes part of the work items that are tracked on Visual Studio Developer Community, which allows you to engage directly with the engineering team while they address and fix the problem.
#please-close


Fix isomorphism between of_seed/of_string and awa_gen_key

Redo this PR with X509.Private_key.of_string (which requires x5<IP_ADDRESS>. This small patch brings an other type error in some way related to #36. However, we can merge it as is. I did not really test the isomorphism nor look deeper on X509.Private_key.of_string.

Thanks for your PR. I just pushed a commit to your branch, what do you think about the result? :) it attempts to modify less code, and have the same observational behaviour (not removing the log messages).

Good for me :+1:.