Add ibm-derived resource manager gem

Description
Adds IBM-derived resource manager sdk https://cloud.ibm.com/apidocs/resource-controller/resource-manager. Used in VPC provision.
Priority
Required to add provisioning to IBM VPC plugin. Need to wait for VPC SDK to be published by IBM.
Implementation
Use IBM tools to generate Resource Manager API access from https://cloud.ibm.com/apidocs/resource-controller/resource-manager.json.
Not covered

Resource Controller https://cloud.ibm.com/apidocs/resource-controller/resource-controller API is not added .

Spec

No specs created.

Common files

No common files changed.

So we discussed in a PM but just to summarize:

We want new APIs to use the openapi-sdkgen tool (:heavy_check_mark: here)
We want to group APIs logically by their api group (in this case resource_controller has resource_controller_v2 and and resource_manager_v2)
This means we'll need to "upgrade" the current ibm_cloud_resource_controller to the new format, push a major gem version (breaking change), update the ibm-cloud-powervs provider to use the new format, then add the resource_manager_v2 to it

I'm fine with updating and combining gems. Hopefully, that should simplify our requirements in the provider. The question is when can this get done? I need a list of available resource groups for VPC provisioning. I can work on volumes for now, but those also need resource groups.

Okay lets replace the existing resource_controller gem (https://github.com/IBM-Cloud/ibm-cloud-sdk-ruby/tree/master/gems/ibm_cloud_resource_controller) with this one, do a major version bump, and add an openapi-sdkgen generated resource_controller/resource_controller_v2 api

Closing. No longer needed.


Update the email mailing list form to post to hubspot

Epic
No response
Description
### As a:
maintainer of our email marketing list
I want to:
Update the sign up form on our website to post signups to Hubspot (new CRM) rather than Buttondown (old CRM).
So that:
Now we have moved over to using Hubspot for emailing customers any new sign ups are posted to Hubspot and the UX for people who are signing up makes sense.
Acceptance Criteria

[ ] Update the form (https://flowforge.com/blog/#sign-up) to write sign ups to Hubspot's API - https://legacydocs.hubspot.com/docs/methods/forms/submit_form
[ ] Update the sign up UX so that the person who has signed up gets a confirmation that their email address has been saved. I'd suggest an inline confirmation rather than taking the user to a new page.

@robmarcer can you check buttondown again. Reading the code on this page it looks like it new names were still being written to Buttondown?

Hello, yes it currently writes the signups to Buttondown but also to Hubspot as Hubspot picks up interaction with any forms on the website via there javascript add-in. As soon as the form is updated I can get rid of our Buttondown account.


Fixed #24427 -- Stopped writing migration files in dry-run mode when merging

Avoid writing migration to file when dry_run=True
Write migration to stdout when verbosity=3

https://code.djangoproject.com/ticket/24427

buildbot, test this please.

Hi @johngian , thanks for the patch. A few comments right away before I have a look at the patch over the weekend. Could you please

adjust the commit message to something like Fixed #24427 -- Stopped writing migration files in dry-run mode when merging
add a release note to docs/releases/1.7.6.txt (something like the commit message but a bit more detailed)

Hey @MarkusH !
Thanks for the review. I updated the PR after your comments.

buildbot, add to whitelist.

buildbot, test this please.

merged in 8758a63ddbbf7a2626bd84d50cfe83b477e8de0a, thanks! Will work on the backport later today as it's a bit complicated due to merge conflicts.


Imports

For Rockstar to become a semi-usable language there have to be imports, I would suggest using the taking X from Y in Z for import syntax, where X is a list of symbols to import separated by the and keyword, or 'X' and from are omitted and the symbols can be accessed via their file name and Y and Z are a relative path or an absolute path based on some pre-defined variable. We could then use them in multiple ways:

taking modulus and fiboacci from math in utils and then modulus and fibonacci can be used as normal functions
taking math in utils with this modulus from math and fibonacci from math can be used as functions.

BTW love the idea, keep up the good work.

For syntax, what about something like "To the tune of"?

That could work, but could you give a complete example of an import statement and how the symbols would be used?

To the tune of <module>: similar to Python's import <module>
To the tune of <module>'s <item>: similar to Python's from <module> import <item>

So To the tune of math's modulus would be similar to from math import modulus. Although perhaps Inspired by lends a better syntax for more than one import? E.g.
Inspired by:
math's modulus
foo's bar
util

(blank line ends the "inspirations" block, similar to how other Rockstar blocks end)

Having imports in a block structure admittedly hardly makes sense, but then again we're building a language around rock'n'roll lyrics to confuse recruiters here.

Also, I've never seen lyrics of a song that starts with "hey, search for the lyrics of that song because we copied half of it." And it is a bit harder for a parser. Leaving that recursive stuff out is not that bad

I like the idea of using syntax like "Featuring {util} from {lib}!" or "Give a hand for {lib}!" as if it were a live concert.

@aquacash5 has a good suggestion over in #63
Using off {filename} play {thing to import}
This could be equivalent to python's from X import Y, or typescript's import X from Y, or node's require(X).Y
And in the case where you want to import the whole module, just play {thing to import}
I think naming-wise, this should work more like python or typescript than node.
Node allows you to name the imported object whatever you please (const coolVariable = require('fs')), while python and typescript require you to use the names given in the module (typescript has optional aliases, don't think we need those)
So for example if your other track (module) defines My Bleeding Heart, you could Off 'My Other Track' play 'My Bleeding Heart', and it would import My Bleeding Heart from My Other Track, thus when playing (importing) from another track (module), it's expected that you use the names defined in that other track.
Alternatively, of course, off could be replaced with from, which might work better.
Not sure if idiomatically this works better than @dkolkena's suggestion though.


Created Transaction.get3() to help with SLP indexer

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Purpose of divider?

Hello @ilyasipek
Some time ago I've added pr that adds horizontal arrangement:
https://github.com/composeuisuite/ohteepee/commit/661e3e38dede6add1b388d09c71100898de9da2a
Last week i've bumped the version of ohteepee and bang: we have issue on how we display the otp field in our project.
The reason is the Divider added in this commit
https://github.com/composeuisuite/ohteepee/commit/7e958a25aaea748e3e0c45a5637bb1a02a1ab6d0
And because of that: instead of 12 dp space betweeen cells, i had 20.
So I have few questions related to this:

should the divider be even considered? Is there a case where you would like to have some custom spacer with custom color or whatever between cells?
if so - shouldn't the default value be set to null? Right now if you use horizontal spacer you need explicitly always set the divider value to null so it will not be overrwritten with 8dp spacer
if we still want to keep the divider - then there is no reason to keep the horizontalArrangement I've added (i guess). It just adds more chaos to how to implement otp.

And btw minor question: is there a plan to migrate material2 to material3 maybe? :D

1,2 ok :D
3: Ahhh i forgot that the horizontal arrangement is not only the Arrangement.spacedBy() that i use :D So yea - we can leave it. I added this point because if we only consider Arrangement.spacedBy() then divider is just duplicate of the arrangement and does the same thing. I only think what is the best implementation. None arragement and divider with 12dp width, or divider set to null with arragement.spacedBy(12.dp). But that minor i think :D
"how can I fix that for you?" No need. I mean i only added this issue thread to make sure to not implement something that won't be merged into main branch :D Otherwise when i will see any issue i will create pr that fixes it :D

Thats great! Maybe if i find time i can try to migrate the project to material3. But if that happens i will try to contact you to make sure if you did not already start the work on it :D


New version: TopazLabs.TopazPhotoAI version 1.3.8

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run


rds: IsErrorAlreadyExists does not appear to check for all error messages

What happened?

Somehow my local Crossplane got into a weird state where it is not able to successfully reconcile a mysqlinstance. Here's an example of the status of the object:
  status:
    conditions:
    - lastTransitionTime: "2019-09-25T20:05:18Z"
      reason: Managed resource is being created
      status: "False"
      type: Ready
    - lastTransitionTime: "2019-09-25T21:49:28Z"
      message: "DBInstanceAlreadyExists: DB Instance already exists\n\tstatus code:
        400, request id: 910d5710-7374-4e4e-ae3c-37e75a950dbe"
      reason: Encountered an error during managed resource reconciliation
      status: "False"
      type: Synced

It looks as though there is an issue with the IsErrorAlreadyExists function where it is only checking one of several error codes that could be returned in the case that the resource already exists.
Here is a link to the specific error message that was being returned, and the full list of error messages as well. It looks like there is at least one other case.
When looking into this issue, it may make sense to also check for more instances of this class of issue (the class is that only one error code is handled, when several should be handled). On quick inspection, it looks as though the IsErrorNotFound function may also have a similar issue. I did not check the logic for other resource types.
How can we reproduce it?

The gist of reproducing this the way that I ran into it is to create an instance using the Crossplane Stacks Guide and the AWS option, and then to get the AWS Stack to try to create the instance again. I believe this could be done by running kubectl edit on the object and removing the status.InstanceName, if it has one.
What environment did it happen in?
Crossplane version: helm crossplane/alpha:
$ helm list crossplane
NAME            REVISION        UPDATED                         STATUS          CHART                   APP VERSION     NAMESPACE
crossplane      1               Tue Sep 24 16:27:50 2019        DEPLOYED        crossplane-0.3.0        0.3.0           crossplane-system

This is running on the Kubernetes from Docker for Mac.

@suskin I believe the only update that needs to be made here is to change the check for rds.ErrCodeDBClusterAlreadyExistsFault to instead check rds.ErrCodeDBInstanceAlreadyExistsFault. I don't believe additional error codes need to be checked.


Fork from upstream and move customization

As discussed we need to be connected with upstream repository, this PR aims to import the customization applied for our needs.
Old repo is still available for a while: https://github.com/teamdigitale/cla-assistant-old

http-s proxy support
example env for docker-compose
styling
refactor: internalized assets, dedicated grunt task to copy them
set up CI with Azure Pipelines

Aren't the files in client/assets/ generated at build? We should keep them out of the repo, if possible.

Just few of them are generated and they are already out of it.


Deprecate the raw entry API in favor of HashTable

It is now moved under the "raw-entry" Cargo feature.
For now the raw-entry feature is kept enabled by default because HashSet uses it internally, but this will be disabled by default once this is no longer the case.

@bors r+

:pushpin: Commit c4ebfaeb014e5c008b02951886a55ea948af1d1e has been approved by Amanieu
It is now in the queue for this repository.

:hourglass: Testing commit c4ebfaeb014e5c008b02951886a55ea948af1d1e with merge f01e2714966a11443070866580f05bdd20181b6b...

:sunny: Test successful - checks-actions
Approved by: Amanieu
Pushing f01e2714966a11443070866580f05bdd20181b6b to master...


Change wording of "review" to "approval" #2554

Fixes: #2554

Thank you for speedy updates 😃
Only migration is missing: you can use ./tools/migrate.sh and after that ./tools/black.sh

Thank you for speedy updates 😃
Only migration is missing: you can use ./tools/migrate.sh and after that ./tools/black.sh

Done. I'm pushing all my updates to single commit if that's not a problem.

@MizukiTemma Does this merge automatically? It says it requires 2 approving reviews.

@abastola0
You need one more approval from one of us to mearge this PR since we do double-check, and I'd like to recommend not to activate auto-merge to avoid any potential problem.
I'll ping you when it becomes ready to merge so you can enjoy clicking "merge" button🙂

Awesome, thanks! Just a tiny detail missing, then I think this PR is good to go! 🎉

done!


TypeError: pop expected at most 1 argument, got 2

Summary
Traceback (most recent call last):
    response = session.get("/dataservice/admin/resourcegroup")
  File "/home/user/env/lib/python3.8/site-packages/vmngclient/session.py", line 126, in get
    return vManageResponse(super().get(url, *args, **kwargs))
  File "/home/user/env/lib/python3.8/site-packages/requests/sessions.py", line 600, in get
    return self.request("GET", url, **kwargs)
  File "/home/user/env/lib/python3.8/site-packages/vmngclient/session.py", line 184, in request
    self.logger.debug(self.response_trace(response, None))
  File "/home/user/env/lib/python3.8/site-packages/vmngclient/__init__.py", line 22, in wrapper
    wrapped = method(*args, **kwargs)
  File "/home/user/env/lib/python3.8/site-packages/vmngclient/response.py", line 91, in response_history_debug
    response_debugs += [response_debug(response, request)]
  File "/home/user/env/lib/python3.8/site-packages/vmngclient/response.py", line 61, in response_debug
    json.pop("header", None)
TypeError: pop expected at most 1 argument, got 2

Logs and external information
Request was not logged in vmngclient.log due to the exception. The response json does not have a "header"/"data" key like other responses.
I added this print statement to log the response
# python3.8/site-packages/vmngclient/response.py

        try:
            json = response.json()
            print(request_debug["url"], json) <--------------------------------
            json.pop("header", None)
            response_debug.update({"json": json})
        except JSONDecodeError:
            if response.encoding is not None:
                if len(response.text) <= 1024:

/dataservice/admin/resourcegroup

[{'id': '0:RESOURCE_GROUPNode:1641915164240:2', 'name': 'global', 'desc': 'System defined group for global network resources managed by global netadmins and readonly to all users.', 'siteIds': [], 'siteIdsIncludeShared': [...

Other responses:
/dataservice/client/server

{'header': {'generatedOn':<PHONE_NUMBER>037}, 'data': {'server': 'vManage1', 'tenancyMode': 'SingleTenant', 'userMode': 'tenant', 'VSessionId': 'ZGVmYXVsdA==',....}

Reproduction script
    response = session.get("/dataservice/admin/resourcegroup")

On what platform(s) does it occur?
Linux
Issue Severity
Low: It annoys me

Cannot reproduce on v0.8.2, client is not crashing and i have header-less json logged. Please check on latest (v0.8.2) and let us know if problem still occuring.

Looks likes fixed already: https://github.com/CiscoDevNet/vManage-client/pull/215
Please re-open if problem persist.

Thank you, the issue is not seen after upgrading to vmngclient==0.8.2.


Add category and datatype to user attributes

To allow for arbitrary user data attributes, a datatype such as int and a category such as health should be added to user attributes. For reference, this would be the adapted user schema
User Document db/users
{
_id:int,
username:string,
email:string,
password_hash:string,
profile_image:binary,
roles:[
role1,
role2,
...
roleN
],
attributes[
{attribute_name: weight, datatype: float, category: health,
{attribute_name: running time, datatype:float, category: health},
...
custom attribute 1,
custom attribute 2
...
custom attribute n
],
settings[
...
],
disabled: bool,
...
}

This should also be done with the datapoint documents as well.
Example schema
data{
_id:int,
username:str,
data_type:str,
catagory:str
next_document:str,
attribute_name:str,
datapoints: [[index: int, timestamp: timestamp, datapoint: int, string, bool, float] ... [index n: int, timestamp: timestamp, datapoint: int, string, bool, float]],
startTimeDate: timestamp,
endTimeDate: timestamp
}


KNOX-2725 - Add HiveServer2 webui support in service definition.

What changes were proposed in this pull request?
HiveServer2 is a service that enables clients to execute queries against Hive. It supports multi-client concurrency and authentication. We can support it in knox service definition, so that knox can proivide proxying service for it.
How was this patch tested?
Deploy knox in a cluster with HiveServer2, we can access HiveServer2 web ui with knox server port 8443.

Hi @smolnar82 , I meet the problem that Java8 workflows cannot pass. Can you tell me how to fix this?

@Flamingo93 - I've just re-triggered the build 🤞


Undefined symbols for architecture arm64

Undefined symbols for architecture arm64:
"_$ss8DurationV10componentss5Int64V7seconds_AE11attosecondstvg", referenced from:
_$s9Alamofire16WebSocketRequestC18startAutomaticPing5everyys8DurationV_tF in libAlamofire.a(WebSocketRequest.o)
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

With no other details I'll assume you're using an old build product from some other Xcode or Alamofire version, or perhaps have an active architecture issue in your project. Either way I suggest doing a clean build. Since Alamofire doesn't directly offer prebuilt products, there's nothing we can really do to help with your local build issues.


Create a version using reification --- step 2

This issue addresses the first step to solve issue #5.
The first step was described in #29.
See #5 and #19 for context.
We need to reify an ast.Literal as an ast.Function as follows:

u(a) -> u(a)
u(-a) -> u(-a)
not u(a) -> not1(u(a))
not not u(a) -> not2(u(a))

Check the grammar described in https://potassco.org/clingo/python-api/current/clingo/ast.html togeter with the classes Literal and Function.
File tests/test_reification2.py in branch jorge/reificationStep2 contains some test that give more information.
I you have downloaded this branch before, please do it again because I merged step 1 (try to use rebase instead of merge when possible https://www.atlassian.com/git/tutorials/rewriting-history/git-rebase).

@jorgefandinno Do we want UnaryOperators to be also transformed into ASTType.Functions?
Because what the assert is expecting is a term of ASTType.UnaryOperator once is reified, those are the ones I have commented out, since I don't know if I should construct a Function from a Literal, like you said, making the test to fail. Or if I should construct an UnaryOperator


Theme: Navigation Sidebar not working when using Webmin Servers Index

Hello. I first want to thank you so much for this amazing theme and all your hard work you've put into it.
I have this issue with the theme:
When I use Webmin Servers Index, I register a server using the 'Login via Webmin with username' link type. With username and password filled.
Clicking on the icon with a different user, who doesn't have access to edit the Servers Index - results in a gray screen but not showing the default page of the Indexed server Webmin. And also the Navigation Sidebar stays the same as the previous Webmin, where the Server Index is, and not the actual linked server.
I tried BWTheme and in that theme the navigation bar changes correctly when switching to a different server.
I was wondering how this can be achieved with the amazing authentic-theme as well. Thank you.

Hi,

I first want to thank you so much for this amazing theme and all your hard work you've put into it.

Thank you for your kind words!

Clicking on the icon with a different user, who doesn't have access to edit the Servers Index - results in a gray screen but not showing the default page of the Indexed server Webmin.

Oh, you're right! I see the bug there too. I will fix it for upcoming Authentic Theme 19.76 release.
Meanwhile, the work-around would be is to hold Ctrl key while clicking on the needed server, to force it opened on the new window.

Thanks for the quick reply.
Yes I see with Ctrl it works.
That would be awesome if you could include it in the next update, thank you!

That would be awesome if you could include it in the next update, thank you!

Already fixed and included to 19.76-beta1. You can go to Theme Configuration: Theme updates and Update Theme to the latest development version.


Text widgets with the same name must share the same value

Attach (recommended) or Link to PDF file here:
multiple.pdf
Steps to reproduce the problem:

Write something in a field

What is the expected behavior? (add screenshot)
The 3 fields must have the same value
What went wrong? (add screenshot)
The 2 other fields are empty
The three fields share the same name Text3.
It works as expected in Adobe and Chrome.

For information, in the pdf in https://github.com/mozilla/pdf.js/issues/13269, there are such text annotations.

For future reference, this is the part of the spec that seems to describe this behavior: https://www.adobe.com/content/dam/acom/en/devnet/pdf/pdfs/PDF32000_2008.pdf#page=442&zoom=auto,-215,555


Support ICAP

Add initial support for ICAP.
Once we have this we can look into what else related to ICAP we need.

https://github.com/chills42/icaparse can serve as inspiration

I'm interested in this. Could you assign me?

I'm interested in this. Could you assign me? Just made an draft PR

Sure thing, all yours. I can see you already started with it, so it gives me hope.
Know though that this wasn't marked as mentor available. That doesn't mean I cannot guide, help or anything like that, but it more means that I assume you are pretty good at programming and can independently get most of the way there.
Some requirements for this crate are:

the ICAP Client/Server implementation needs to be usable within a rama service layer (meaning it needs to be tokio-async and work with a &self reference (not mutable, but shared)
ICAP client/server is typically used within a high-load traffic environment, as such this implementation needs to allocate as little as possible and be "fast"

There is a lot of similarities with HTTP/1 so can see if there is stuff/code/ideas that can be borrowed from rama-http-core h1 proto code.
The deliverables in order to close this ticket are:

a good enough first version of a client and server
example (in /examples
a integration test verifying this example works

E.g. the example can be
client <-----> proxy <---------> server
                    [icap client]
                         ^
                         ||
                        V
                  [icap server]

However in case your PR only implements the protocol layer with sufficient tests and some simple integration tests showcasing the server client then that is already good enough to start it as well.
The core protocol server and client are unrelated from a rama service/layer. They are just stuff on their own.
Next to this (in this PR or a follow up PR) a layer service will be added that wraps the client, so that one can easily send passing http traffic via an ICAP server given by the client in that layer.
This is a pretty big task though, so take it as far as you can, so for the sake of the quality of your work might not want to overdo the scope of your PR as to make sure you have the will, energy and motivation to deliver the stuff that you did do with pride and quality.
I'm here for you for feedback, guidance or anything else.
Given this work became a lot more priority for some people I might take over in a week or two should this not progress fast enough. That wouldn't say anything about you though, its just that I do need it at some point soon. Until then however I have plenty of other stuff that I can still do myself. So also don't stress on it.


Update job image

Related to k3s-io/helm-controller#54 and k3s-io/klipper-helm#22
Signed-off-by: Brad Davidson<EMAIL_ADDRESS>

Skipping this in favor of #86

Skipping this in favor of #86


protoc version for generating pb/crypto.pb.go

Hi,
I am adding a new signature scheme, so I edited pb/crypto.proto and have to manually generate pb/crypto.pb.go, could you let me know what protoc version should I use to build it?
I tried protoc 3.2.0 but have the following error:
$ cd pb
$ make
protoc --gogo_out=. crypto.proto
[libprotobuf WARNING google/protobuf/compiler/parser.cc:546] No syntax specified for the proto file: crypto.proto. Please use 'syntax = "proto2";' or 'syntax = "proto3";' to specify a syntax version. (Defaulted to proto2 syntax.)
protoc-gen-gogo: program not found or is not executable
--gogo_out: protoc-gen-gogo: Plugin failed with status code 1.
Makefile:7: recipe for target 'crypto.pb.go' failed
make: *** [crypto.pb.go] Error 1

We use the gogo protobuf implementation: https://github.com/gogo/protobuf

And if we don't answer issues/PRs within a week, please bug us on IRC. We have a ton of repos and issues can slip through the cracks.

This has been fixed (we now explicitly specify proto2).


Intellisense is not working very well

Intellisense only show snippets sometimes when i try programming on it, it doesn't show everything based on the letter and current situation like Visual Studio does, like when i try to type things such as "Console" it does not appear, the only things intellisense want to appear are things like "try-catch" or "for" (curly brackets gang) and other things that i already used, also the icons of classes, methods and fields don't show up (just like they don't appear)
Note: this might be duped from other issues, however it is still a recent issue, and i searched into other forums like stackoverflow but nothing fixed here, so i'll thank if someone help me.

VS Code Version: 1.55.0
OS Version: Windows 7 SP1 Professional

Steps to Reproduce:

idk if it is just for me, but just entering in vscode

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

What language is the file? Look at the indicator near the right end of the status bar.

is a .cs file (c#- C-Sharp language)

but the only extension i was with was the C# one every other i completely uninstalled

Sure, but the c# extension is tracked in another repo: https://github.com/OmniSharp/omnisharp-vscode/issues

oh ok


fix undefined: body in http consumer

Fixed "undefined: body" in consumer/http.go
GoWork/src/github.com/trivago/gollum$ go build
# github.com/trivago/gollum/consumer
consumer/http.go:170: undefined: body
consumer/http.go:170: undefined: err
consumer/http.go:171: undefined: err
consumer/http.go:173: undefined: err in err.Error
consumer/http.go:178: undefined: body

I already fixed that one in the dev branch (0.4.5) but must have missed it in the master 🙄
As Gollum 0.4.5 is going to be released next week I'll not merge this unless you have concerns about the fix in 0.4.5 (@mikechris).

0.4.5 has been merged


add file jnu domain

Chonnam Univercity change domain chonnam to jnu.

@Depapepe Pull request merged. Thank you!


unhandled error due to go 1.19

connection.go:380: level=info, msg=connected, connection=TestNewConnectionDisconnect-27
    toxiproxy_client_test.go:46: level=debug, msg=disabled rabbitmq connection,
    connection_test.go:65:
                Error Trace:    /home/behm015/Development/amqpx/pool/connection_test.go:65
                                                        /home/behm015/Development/amqpx/pool/asm_amd64.s:1594
                Error:          Received unexpected error:
                                connection failed: Exception (501) Reason: "EOF"
                Test:           TestNewConnectionDisconnect
    connection_test.go:65:
                Error Trace:    /home/behm015/Development/amqpx/pool/connection_test.go:65
                                                        /home/behm015/Development/amqpx/pool/asm_amd64.s:1594
                Error:          Received unexpected error:
                                connection failed: Exception (501) Reason: "EOF"
                Test:           TestNewConnectionDisconnect
[... the same ...]
    connection_test.go:65:
                Error Trace:    /home/behm015/Development/amqpx/pool/connection_test.go:65
                                                        /home/behm015/Development/amqpx/pool/asm_amd64.s:1594
                Error:          Received unexpected error:
                                connection failed: Exception (501) Reason: "EOF"
                Test:           TestNewConnectionDisconnect
    toxiproxy_client_test.go:53: level=debug, msg=enabled rabbitmq connection,
    connection.go:380: level=info, msg=recovering, connection=TestNewConnectionDisconnect-29
    connection.go:388: level=debug, msg=connecting..., connection=TestNewConnectionDisconnect-29
    connection.go:380: level=info, msg=recovering, connection=TestNewConnectionDisconnect-98

this error occurs when establishing a new connection

improved error detection of errors that can be recovered from (especially network errors)


Alt-Enter on Method Name -> Create New Test

This would be invaluable for code coverage, or really in general from a testing perspective.  I am basically creating tests around older/imported code, and I am building tests around them.  Of course they show up in VS/R# as unused, so they stick out like a sore thumb, but when I hit Alt-Enter, nothing is shown to provide/create a test around it.  It would be super cool to have the ability to do this.

I'm quite not sure what exactly you would like to have. If you have unused method, than it's better to remove it from code 😉 Could you provide a small demo with feature description and the desired behavior?
Thanks.

Hah!  That is a good point. It is unused until there is a test written around it, in this case. :)  Or it may be a method that was used at one point and is no longer used, but could be valuable in the future.  In any case, the method doesn't have to be unused.
Quite simply, the expected behavior is exactly what you get when you have the cursor in the test name, except instead of "Create Test File", it's "Create Test Method".  When selected, the test file is loaded (or created if not already), and a method is created in that test file that is the same name as the method in the subject class.
This is the basic workflow I go through now:
https://www.screencast.com/t/lteN5DtBA3
Even better if it adds AutoFixture attributes:
https://www.screencast.com/t/FNrpUnXrC2
👼
No worries if the value doesn't translate here.  I thought I would bring it up to see if it was worth exploring. :)

Thanks for sharing those samples and 👍 for the AutoFixture usage 😉
You described that in a way I approximately expected. Well, I have concerns regarding this feature:

you expect that a test for a particular type's method would be prefixed by the method name. In fact that convention is followed relatively rarely from my experience and personally I use it not so often.
the Alt-Enter actions list will be polluted by an additional item which might be not frequently used, however you will see it for each time.
personally I don't need/like that feature, but it will cost a lot to support it (as JetBrains changes their API in each update).

Given that I'd prefer to not include and support this feature as it's very specific and has limited applicability.

As for the demos you showed, I'd suggest to create a better R# templates, so it will go simpler 😄 For instance, I have a theoryAutoNData template which inserts the following snippet (with my custom AutoData attribute):
[Theory, AutoNData]
public void $TESTNAME$($parameters$)
{
  // arrange
  // act

  // assert
}

So it's quicker to create a test and typing of the method's name (when required) is not so boring 😅

Alright cool.  Sorry for the delay here, I was browsing for some more documentation and realized that I had this open still.  Thanks again for your consideration/discussion.


Bluetooth: Controller: Use NRF_CCM HEADERMASK for ISO PDU encryption

Add missing headermask binding for NRF_CCM peripheral and
define HAS_HW_NRF_CCM_HEADERMASK Kconfig.
Use NRF_CCM HEADERMASK register to use correct Additional
Authentication Data (AAD) from ISO PDU header.
Fixes #55263.

@anangl Need you early review on dts and Kconfig changes (while I am implementing other changes in the Controller).


Explorer double click is broken when Open Editors is visible

Seen on macOS

Double click a file in the explorer

Two files get opened!

cc @bpasero

Nevermind, this only happens because I have "workbench.editor.enablePreview": false, so double click is a no-op!


[SPARK-27142][SQL] Provide REST API for SQL information

What changes were proposed in this pull request?
Currently for Monitoring Spark application SQL information is not available from REST but only via UI. REST provides only applications,jobs,stages,environment. This Jira is targeted to provide a REST API so that SQL level information can be found
A single SQL query can result into multiple jobs. So for end user who is using STS or spark-sql, the intended highest level of probe is the SQL which he has executed. This information can be seen from SQL tab. Attaching a sample.

But same information he cannot access using the REST API exposed by spark and he always have to rely on jobs API which may be difficult. So i intend to expose the information seen in SQL tab in UI via REST API
Mainly:
Id :  Long - execution id of the sql
status : String - possible values COMPLETED/RUNNING/FAILED
description : String - executed SQL string
planDescription : String - Plan representation
metrics : Seq[Metrics] - Metrics contain metricName: String, metricValue: String
submissionTime : String - formatted Date time of SQL submission
duration : Long - total run time in milliseconds
runningJobIds : Seq[Int] - sequence of running job ids
failedJobIds : Seq[Int] - sequence of failed job ids
successJobIds : Seq[Int] - sequence of success job ids

To fetch sql executions: /sql?details=boolean&offset=integer&length=integer
To fetch single execution:  /sql/{executionID}?details=boolean

parameter
type
remarks

details
boolean
Optional. Set true to get plan description and metrics information, defaults to false

offset
integer
Optional. offset to fetch the executions, defaults to 0

length
integer
Optional. total number of executions to be fetched, defaults to 20

Why are the changes needed?
To support users query SQL information via REST API
Does this PR introduce any user-facing change?
Yes. It provides a new monitoring URL for SQL
How was this patch tested?
Tested manually

Can one of the admins verify this patch?

Can one of the admins verify this patch?

cc @srowen @cloud-fan @dongjoon-hyun @gengliangwang

Can one of the admins verify this patch?

@ajithme Thanks for the work.
To make it consistent with other API, I think we need to have at least two APIs
@Path("sql")
@Path("sql/{sqlExecutionId: \\d+}")

@ajithme Thanks for the work.
To make it consistent with other API, I think we need to have at least two APIs
@Path("sql")
@Path("sql/{sqlExecutionId: \\d+}")

@gengliangwang  So the full api looks like
http://localhost:4040/api/v1/applications/{applicationId}/sql
http://localhost:4040/api/v1/applications/{applicationId}/sql/{executionId}

is this the expectation.?

http://localhost:4040/api/v1/applications/{applicationId}/sql
http://localhost:4040/api/v1/applications/{applicationId}/sql/{executionId}

Yes, it is consistent with
http://localhost:4040/api/v1/applications/{applicationId}/jobs/
http://localhost:4040/api/v1/applications/{applicationId}/jobs/{jobId}

Ok i have updated the PR accordingly. Please review

ok to test

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8870/
Test PASSed.

Test build #103450 has started for PR 24076 at commit 8827301.

gentle ping @gengliangwang  @dongjoon-hyun @srowen

As I say I'm not sure this is something we should expose this way. See prior comments on JIRA

As I say I'm not sure this is something we should expose this way. See prior comments on JIRA

@srowen i have updated the use case in the JIRA already for prior comments. Please have a look

@srowen i have updated the use case in the JIRA already for prior comments. Please have a look

gentle ping @cloud-fan @srowen @dongjoon-hyun @gengliangwang Any further suggestions of this feature.?

@ajithme I am +1 for this. I have several comments for the current code. I will get back here soon.

@ajithme I am +1 for this.
Some comments are not addressed correctly(e.g. #24076 (comment)), so I create a PR(ajithme#1) in your repo. Please review it.

Thanks @gengliangwang I got confused between core/src/main/scala/org/apache/spark/status/api/v1/api.scala and sql/core/src/main/scala/org/apache/spark/status/api/v1/api.scala i will merge your comments shortly here.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9307/
Test PASSed.

Updated with latest comments fixed. Please review

Test build #103950 has started for PR 24076 at commit b1a8deb.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9308/
Test PASSed.

Test build #103950 has finished for PR 24076 at commit b1a8deb.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103950/
Test FAILed.

retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9322/
Test PASSed.

Test build #103965 has started for PR 24076 at commit b1a8deb.

Test build #103965 has finished for PR 24076 at commit b1a8deb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103965/
Test PASSed.

@vanzin

@vanzin @srowen @dongjoon-hyun gentle ping. Can we re-look into this.?

Can one of the admins verify this patch?

This seems to be missing the plan description and metrics. Any specific reason not to expose those?

@vanzin Wanted this API to be light weight for monitoring. Returning plan and metrics can significantly increase the payload. Please share your thoughts

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13949/
Test PASSed.

Returning plan and metrics can significantly increase the payload.

Yes, but it's also useful information and there would be no way to retrieve it from the API...
In our APIs we generally have a query parameter that tells you what level of detail you have in the response, so you can have a small summary or a full view of an object, for example. That sounds like something we could add here.
I even did something similar in the StagesResource.oneAttemptData method with the details parameter. You could add that parameter here with a default value of false, for example, so only those who really want the extra data would get it.

Returning plan and metrics can significantly increase the payload.

Yes, but it's also useful information and there would be no way to retrieve it from the API...
In our APIs we generally have a query parameter that tells you what level of detail you have in the response, so you can have a small summary or a full view of an object, for example. That sounds like something we could add here.
I even did something similar in the StagesResource.oneAttemptData method with the details parameter. You could add that parameter here with a default value of false, for example, so only those who really want the extra data would get it.

Thanks @vanzin for inputs. I agree with your opinion and will update the PR

Can one of the admins verify this patch?

Handled comments from @vanzin and included plan and metrics with a flag for details

Please review @vanzin @gengliangwang @dongjoon-hyun

After handling review comments

@vanzin Thank you for review and i apologise for indent issues. Now i have run ./dev/scalafmt and updated the PR with other review comments. Please review

@vanzin please review, updated as per the comments

Weird tests aren't running.

ok to test

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/21296/
Test PASSed.

Test build #116509 has started for PR 24076 at commit 4560df9.

Test build #116509 has finished for PR 24076 at commit 4560df9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116509/
Test PASSed.

@vanzin gentle ping

Merging to master.

@ajithme I think we have to revert this one in branch-3.0. See my comments in https://github.com/apache/spark/pull/28588


Verify Send and Sync are valid on all structures

Right now all structures are Send and Sync, since the types are opaque and Rust cannot know whether they hide pointers or not.
https://doc.rust-lang.org/std/marker/trait.Send.html
https://doc.rust-lang.org/std/marker/trait.Sync.html

Looking at the scotch v6.1 manual: strategy creation from strings use not-thread-safe libc parsing functions and thus must be ran behind a lock. Otherwise, there is no mention of thread safety.
The v7 manual talks about context objects to encapsulate otherwise-global variables, I don't know if those variables are thread-local in Scotch v6. In such case, Graph and Mapping might not be Send.


Fixed the problem when add .csp in a versioned folder (ISSUE #22)

I think I fixed this by a457c53f31fbeb14a641aa17a02b7766aaeba808
If both file and routine did not existed then timestamp for file was mistakenly
'1840-12-31 00:00:00'
and timestamp for routine was '1840-12-31' that made it looks like routine should be refreshed from disk


Release diagnostics libraries version 4.3.1

Changes in Google.Cloud.Diagnostics.AspNetCore version 4.3.1:
No API surface changes; just dependency updates.
Changes in Google.Cloud.Diagnostics.AspNetCore3 version 4.3.1:
No API surface changes; just dependency updates.
Changes in Google.Cloud.Diagnostics.Common version 4.3.1:

Commit b1318ac:

fix: Respects the service client set through error reporting options.
Adds missing tests for all options.
Closes issue 7448

Packages in this release:

Release Google.Cloud.Diagnostics.AspNetCore version 4.3.1
Release Google.Cloud.Diagnostics.AspNetCore3 version 4.3.1
Release Google.Cloud.Diagnostics.Common version 4.3.1

Created release for Google.Cloud.Diagnostics.AspNetCore-4.3.1
Created release for Google.Cloud.Diagnostics.AspNetCore3-4.3.1
Created release for Google.Cloud.Diagnostics.Common-4.3.1


Edit invalid accessibility patterns urls

Several urls about accessibility patterns were invalid in the documentation. I replaced these urls with the good ones!
This pull request:

[ ] Fixes a bug
[ ] Adds additional features/functionality
[x] Updates documentation or example code
[ ] Other

Closes #403

Hi @hugodessomme, we have a related open issue here: #403.
We need to update this globally for all primitives, as well as all the previous version files (not just the latest ones).
Would you like to continue with that task?

Hello @benoitgrelard, I'm ok to continue this task :) By "all primitives", which section of the docs are you refering to?

By "all primitives", which section of the docs are you refering to?

I just mean any of the mdx files that have a aria link in the front matter.

Superseded by #479


[SPIKE] Define how to display Attestation IDs once we display attestations from all networks at the same time

Proposition when displaying a shortened ID:
0x0001...58e0fb, in other words: Chain Prefix ... End of the ID
Proposition when displaying a full ID:
0x000100000000000000000000000000000000000000000000000000000058e0fb
Proposition when displaying a human-readable ID:
Linea-#5,824,763, in other words: Chain Name-#ID as number


Take in a version prefix to use with Release step. This will be useful in general for folks who want a version like v.1.2.3 as their tag, but also means that they could do<EMAIL_ADDRESS>or something.

Oops, dup of #151


Error run ios 7 : not found for architecture armv7

I tried run on real iphone ios version 7.0
If I run on simulator , It 's fine.
ld: warning: directory not found for option '-F/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS9.0.sdk/Developer/Library/Frameworks'
Undefined symbols for architecture armv7:
  "_RCTSetLogFunction", referenced from:
      -[FinanceTests testRendersWelcomeScreen] in FinanceTests.o
ld: symbol(s) not found for architecture armv7
clang: error: linker command failed with exit code 1 (use -v to see invocation)

What I do wrong here?

ping @7kfpun

This link should help. Tell me if it works, thanks!
https://github.com/facebook/react-native/issues/2685#issuecomment-142626306

It works.
Thank you

Updated to<EMAIL_ADDRESS>try again please.

Now I can run on emulator
592
 URL: http://localhost:8081/index.ios.bundle?platform=ios&dev=true
 line: 7572
 message: undefined is not an object (evaluating 'this._localModuleNameToMethodIDToName[moduleName]')"
2015-10-26 16:48:39.618 [error][tid:com.facebook.React.JavaScript] "#CLOWNTOWN (error while displaying error): undefined is not an object (evaluating 'RCTDataManager.queryData')"
2015-10-26 16:48:39.618 [error][tid:com.facebook.React.JavaScript] "#CLOWNTOWN (error while displaying error): undefined is not an object (evaluating 'RCTDataManager.queryData')"
2015-10-26 16:48:39.619 [error][tid:com.facebook.React.JavaScript] "#CLOWNTOWN (error while displaying error): undefined is not an object (evaluating 'RCTDataManager.queryData')"

SyntaxError Unexpected token ','
{
"line" : 57479,
"sourceURL" : "http://<IP_ADDRESS>:8081/index.ios.bundle?platform=ios&dev=true"
}

I have tried and it works (iPhone 5s with 9.1). Try and tell me the outcome, thanks!
http://stackoverflow.com/questions/29306888/react-native-on-ipad-error-could-not-connect-to-development-server#answer-29307009

I have tried and it works. Try it!
http://stackoverflow.com/questions/29306888/react-native-on-ipad-error-could-not-connect-to-development-server#answer-29307009

I wonder may be error occur on ios 7.1.
Have you tested on ios 7.1 ?

Sorry I have only one device. I think you could open an issue in React-native.

ok thanks for help
I find this https://github.com/facebook/react-native/issues/3261


output folder-qsirecon only have logs folder

Hi Matt,
When i used the qsiprep,i was confused with some questions. I sincerely hope you can solve my problems if it is convenient for you.
q1,With the default options, the pipeline only implemented preprocessing, and the reconstruction results were not automatically generated, Do I need to set any specfic options according to the dMRI data (e.g. --recon-spec argument)?
q2, The qsiprep have completed preprocessing pipeline, and generated qsiprep folders, but there wa no any reconstruction results . In the output dir, there was only log folder in the qsirecon folder. It reported erros in the process.
210821-17:50:56,732 nipype.workflow INFO:
[Node] Finished "qsiprep_wf.single_subject_001_wf.dwi_finalize_acq_diff_wf.ds_interactive_report".
210821-17:50:56,732 nipype.workflow INFO:
[Job 108] Completed (qsiprep_wf.single_subject_001_wf.dwi_finalize_acq_diff_wf.ds_interactive_report).
Sentry is attempting to send 1 pending error messages
Waiting up to 2 seconds
Press Ctrl-C to quit
q3,I wanna continue analysis, if i chose the" --recon-spec"  option, what  the output dir should be input, like this " --output_dir /where/my/reconstructed/data/goes" ,Does it refer to the "sub-XX"folder generated in the proprecessing pipeline output?
$ qsiprep-docker
--bids_dir /path/to/bids
--recon_input /output/from/qsiprep
--recon_spec gqi_scalar_export.json
--output_dir /where/my/reconstructed/data/goes
--analysis_level participant
--fs-license-file /path/to/license.txt

@mattcieslak

Yes, for qsirecon to run , --recon-spec must be specified
That error doesn't actually seem to be a problem, since it comes after the last step of preprocessing (that is, creation of the interactive reports)
the output dir does not need to change for running reconstruction. That is, you can use the same one as qsiprep (e.g. /bids/derivatives/). If you want to use your previously computed results, you can specify --recon-only and --recon-input, which should point to your qsiprep output folder (e.g. /bids/derivatives/qsiprep)


Moving in dihedral space

I am new to OPS but I love its modular design! Kudos to the architects! I would like to use OPS to move between two states by moving in the space of dihedrals i.e. each MC step changes the dihedral angles of the conformation. I am wondering if I need to create a new PathMover subclass for this. I will really appreciate some guidance on this! TIA

Thanks for the interest in OPS! If I understand correctly, you want to do path sampling where your "trajectories" will come from MC (based on moves in dihedral space) instead of MD. This should be perfectly feasible, but the thing you'll want to change will be the DynamicsEngine.
If you can use OpenMM to calculate energies, I think the easiest path would be:

Use OpenMMTools and create a custom MCMC Move for your dihedral move (see the MCDisplacementMove and MCRotationMove for analogous examples -- I haven't done this before, but it looks like you just need to write a _propose_positions method, plus helpers like __getstate__ which allow the MC move to be stored to disk)
Wrap that MC move in an OpenMM integrator (you may want to check with the OpenMMTools team for details on that -- I can imagine some problems here).
Once you have that integrator, it can be used directly in the existing OPS OpenMM engine -- just replace the VVVR integrator in our examples with your MC move integrator.

If you can't use OpenMM (or prefer to use an existing tool for the MC), or if it isn't possible to wrap the MC move in an OpenMM integrator, then what you'd want would be to create a subclass of DynamicsEngine. Docs are a bit incomplete (as always), but the section on writing custom engines has a decent start.

I've thought about this a little more, and I'm pretty sure using the OpenMM MCMC would require a new DynamicsEngine. Good news is, writing a new engine is pretty easy. So I did the core of it already. See #1041.

That awesome! I am still working on proposing a step in dihedral space. I also have plan to propose steps in normal mode space. The new DynamicsEngine should be useful for this as well. Thanks!


Resolving C++ Build issues to make code coverage and progress more visible

Current C++ build is done using a separate shell script process (named bld) inside setup.py. That not only limits our visibility for C++ error investigation but also making adding code coverage to C++ part of the code complicated.
As alternative I want to use sk.setup tool to bring cmake runs into setup.py as a first class citizen to address both goals.

Note that a) bld is just one of several ways to build and) we need (and have) a general cross-platform cross-language way not depending on setup.py as the R builds generally do not have or need a full Python installation.

@eddelbuettel thanks for the comment. Unfortunately the way that python is doing it is very rudimentary (having a bash script called inside setup.py!) and I think I need to fix that first. But I think at the same time I can start looking at adding codecov to R C++ builds as well.

We have codecov in R builds:
https://github.com/single-cell-data/TileDB-SOMA/blob/dbd10899f0a5c6a5a6b9026e11e2c38ca60c925f/.github/workflows/r-ci.yml#L95-L98

Does it makes sense that for the code cove part of the code, I simply use bld script itself (I mean the cmake commands there) without involving python at all?
That saves me changing the way we are building python. After all, improving our build system is really my goal for codecov and I can apply codecov the same way that you do in R so we'll be consistent across the repo


different number of open and close parentheses

When I call setDataFromText function using the following tree representation
((ROOT (S (NP (NNP KeLP)) (VP (VBZ is) (ADJP (JJ very amazing)))))
it works fine but a few extra spaces here and there(shown using asteriks *) e.g. as shown below
(ROOT (S (NP (NNP****** KeLP)) (VP (VBZ is) (ADJP (JJ very amazing)))))
gives the error that parentheses are not equal and following is printed
Error in analyzing: (ROOT(S(NP(NNP((((((KeLP))))(VP(VBZ(is))(ADJP(JJ(very(amazing))))))))))
Another example:
((ROOT (S (NP (NNP KeLP)) (VP******(VBZ   is) (ADJP (JJ very amazing)))))
Error in analyzing:  (ROOT(S(NP(NNP(KeLP)))(VP(((VBZ(((is))(ADJP(JJ(very(amazing)))))))))).
So extra spaces are replaced by arbitrary number of parenthesis which creates a problem during analysis. Since I have source code lines i.e. entire sentences instead of words I cannot control the spaces. Hence any  workaround or suggestions  welcome. thanks

We will evaluate this for sure. In the meanwhile, can you make me an example of one tree of yours?

So the following statement would raise an exception
t2.setDataFromText("(ROOT (S (NP (NNP       KeLP)) (VP (VBZ is) (ADJP (JJ very amazing)))))");
Error in analyzing: (ROOT(S(NP(NNP(((((((KeLP))))(VP(VBZ(is))(ADJP(JJ(very(amazing)))))))))).
The number of opened and close parentheses are different.


isValidPhoneNumber true for invalid numbers.

+9779861eeeeee this number seems sure invalid. However, if passed to ,
await PhoneNumberUtil.isValidPhoneNumber(
phoneNumber: phone,
isoCode: selectedCountryCode.code!,
);
    is return true because the length exactly matches. Thank you for the package though.

isValidPhoneNumber is from the lib phone number package i can't see that its in this flutter_libphonenumber package, however I'd appreciate it being moved into this package so we can migrate fully from that to this

@martipello you can do the same by calling parse and if you get a null result then it's not a valid number, if it is valid it would return the country info associated with it.

@martipello you can do the same by calling parse and if you get a null result then it's not a valid number, if it is valid it would return the country info associated with it.

Very cool I will do just that, many thanks

it cant return null do you mean if it throws?

sorry you're right, but that's correct. just check the error when you pass a bad number to it and catch that accordingly.


EEA: Only lock if entity instance exists in a transaction

Summary
This changes the logic to check that an entity exists as part of a transaction
before trying to lock it in the EEA. This way we'd avoid race conditions of trying
to lock an entity that gets deleted at the same time.
Change Type
Mark the type of change your PR introduces:

[x] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 53.669% (+0.02%) from 53.647%
when pulling d595513cb03a45fe3344011e844645353847dc74 on eea-entity-instance-get
into 1f9c1ef5208e797bed70be2cbc5610ca941998d7 on main.


Updating Slack integration to comply with new Slack webhook format

Slack has changed their web hook format, this update makes the Slack integration work again.

Looks fine to me.
What happened to the extra parameters (like channel)?
Any comments @elingg?

Slack changed their format - each web hook URL is specific to a channel and account, so the extra parameters (channel, token) are now merged into the URL.

Understood, thanks for clarifying.

:+1:  I had the same question, but this makes sense. Let's merge!


Library tests

Everything worked.

For some reason - noone of styles applied to character after sending commands.


How to run NET5 App on Android ?

I compiled a Hello App using Net SDK 5.0.100-rc.1 targeting Android-ARM 👍
dotnet publish --configuration Release --runtime android-arm
But how to run my app on my phone ?
There's any NET Runtime for Android-ARM yet ?

@davidortinau @jamesmontemagno can you help with this question?

Hi @SasukeK93, until .NET 6 you'll want to continue using .NET Framework/Mono to target Android. .NET 5 may "work" by virtue of work we have done in preparation for .NET 6, but it doesn't have support for AOT, linker, crypto, and other features needed for most mobile applications. That work is coming in .NET 6.


Add ca.crt to the mine_functions so it's published when the salt-minion starts

We cannot rely on the ca.crt being published during the orchestration, since
it's an async operation and we cannot reliably wait for the contents to be there
to continue with the orchestration with the current version of salt that we are
using.
For this reason, it's safer to publish the ca.crt contents with the mine_functions,
since the file will be there upon start, and it will be published when the ca
salt-minion is starting.
Fixes: bsc#1049137
Fixes: bsc#1048548

Unifying on https://github.com/kubic-project/salt/pull/179


MDFloatingActionButtonSpeedDial adding icon to main button results in Attribute error.

MDFloatingActionButtonSpeedDial adding icon to main button results in Attribute error.
documentation stated you can add icon to main class, when i do so it throws an attribute error.
Line 1653 in button.py ( Kivymd\uix\button.py )
self._get_count_widget(MDFloatingRootButton).icon = value
that normaly returns its children for the generated buttons but this icon is meant for self so i think it should be:
self.icon = value
changing the above seems to work perfectly for me.
Code and Logs
Traceback (most recent call last):
File "C:\Data Tim\Applicaties\Dashboard\venv\lib\site-packages\kivy\lang\builder.py", line 700, in _apply_rule
setattr(widget_set, key, value)
File "kivy\weakproxy.pyx", line 35, in kivy.weakproxy.WeakProxy.setattr
File "kivy\properties.pyx", line 497, in kivy.properties.Property.set
File "kivy\properties.pyx", line 544, in kivy.properties.Property.set
File "kivy\properties.pyx", line 599, in kivy.properties.Property.dispatch
File "kivy_event.pyx", line 1214, in kivy._event.EventObservers.dispatch
File "kivy_event.pyx", line 1120, in kivy._event.EventObservers._dispatch
File "C:\Data Tim\Applicaties\Dashboard\venv\lib\site-packages\kivymd\uix\button.py", line 1653, in on_icon
self._get_count_widget(MDFloatingRootButton).icon = value
AttributeError: 'NoneType' object has no attribute 'icon'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:/Data Tim/Applicaties/Dashboard/Dashboard.py", line 281, in
MainApp().run()
File "C:\Data Tim\Applicaties\Dashboard\venv\lib\site-packages\kivy\app.py", line 829, in run
root = self.build()
File "C:/Data Tim/Applicaties/Dashboard/Dashboard.py", line 277, in build
return Builder.load_file("Dashboard.kv")
File "C:\Data Tim\Applicaties\Dashboard\venv\lib\site-packages\kivy\lang\builder.py", line 301, in load_file
return self.load_string(data, **kwargs)
File "C:\Data Tim\Applicaties\Dashboard\venv\lib\site-packages\kivy\lang\builder.py", line 405, in load_string
rule_children=rule_children)
File "C:\Data Tim\Applicaties\Dashboard\venv\lib\site-packages\kivy\lang\builder.py", line 657, in _apply_rule
root=rctx['ids']['root'], rule_children=rule_children)
File "C:\Data Tim\Applicaties\Dashboard\venv\lib\site-packages\kivy\uix\widget.py", line 469, in apply_class_lang_rules
rule_children=rule_children)
File "C:\Data Tim\Applicaties\Dashboard\venv\lib\site-packages\kivy\lang\builder.py", line 538, in apply
rule_children=rule_children)
File "C:\Data Tim\Applicaties\Dashboard\venv\lib\site-packages\kivy\lang\builder.py", line 707, in _apply_rule
e), cause=tb)
kivy.lang.builder.BuilderException: Parser: File "C:\Data Tim\Applicaties\Dashboard\Dashboard.kv", line 316:
...
314:        MDFloatingActionButtonSpeedDial:
315:            id: fab

316:            icon: 'menu'
317:            data: app.data
318:            rotation_root_button: True
...
AttributeError: 'NoneType' object has no attribute 'icon'
File "C:\Data Tim\Applicaties\Dashboard\venv\lib\site-packages\kivy\lang\builder.py", line 700, in _apply_rule
setattr(widget_set, key, value)
File "kivy\weakproxy.pyx", line 35, in kivy.weakproxy.WeakProxy.setattr
File "kivy\properties.pyx", line 497, in kivy.properties.Property.set
File "kivy\properties.pyx", line 544, in kivy.properties.Property.set
File "kivy\properties.pyx", line 599, in kivy.properties.Property.dispatch
File "kivy_event.pyx", line 1214, in kivy._event.EventObservers.dispatch
File "kivy_event.pyx", line 1120, in kivy._event.EventObservers._dispatch
File "C:\Data Tim\Applicaties\Dashboard\venv\lib\site-packages\kivymd\uix\button.py", line 1653, in on_icon
self._get_count_widget(MDFloatingRootButton).icon = value

OS: Win 10
Python: 3.7
Kivy: 1.11.1
KivyMD: 0.104.1

We still need a sample code to reproduce the error.

i use this in kv file.
	MDFloatingActionButtonSpeedDial:
    	on_kv_post: self.icon = 'icon/menu.png'
     data: app.lottomenu
     callback: app.floatLottoMenu

The button lib is currently in a major modification #501 let me check the speed dial for that specific bug

Fix in the master branch.


Current kernels cannot take functions as input arguments

It would be ideal to pass function to the julia kernel functions. This would enable generalized functions like broadcast kernel.
For this to be possible computeBlock of WGPUTranspiler should filter arguments based on types. General arguments will be Function, Scalar and WgpuArray Types.

Other types like structs are also possible. Will think about it later.


Not all hackers are taken by the police

Yet.

Cmon guys. What we are doing here is serious development of high value cryptographic code! Let's not give the chance to hackers to ruin our work!!!
PLEASE MAKE SURE YOU REPORT ALL HACKERS TO THE POLICE ASAP!


Unable to Disabling Windows EFI

Sorry if this is the incorrect place, but I am having an error disabling the windows efi entry to have the deck boot into rEFInd.
I had the common error of SteamOS deleting its own EFI entry, and following the readme I managed to re-enable it and reinstall rEFInd. SteamOS and rEFInd both work if manually selected in the boot loader, but the deck boots into windows by default.  I believe the rEFInd script also is unable to disable the windows EFI. The Windows EFI file exists, and if I boot into rEFInd using the boot loader, I can launch windows from rEFInd. I tried disabling the windows entry in manually with the error below.
(1)(deck@steamdeck ~)$ sudo efibootmgr

[sudo] password for deck:

BootCurrent: 0001

Timeout: 0 seconds

BootOrder: 0002,0000,0001,2001,2002,2003

Boot0000* rEFInd HD(1,GPT,676c28fe-3c8e-e04f-8a04-bfc22731e99d,0x800,0x20000)/File(\efi\refind\refind_x64.efi)

Boot0001* SteamOS HD(1,GPT,676c28fe-3c8e-e04f-8a04-bfc22731e99d,0x800,0x20000)/File(\efi\steamos\steamcl.efi)

Boot0002* Windows Boot Manager HD(1,GPT,676c28fe-3c8e-e04f-8a04-bfc22731e99d,0x800,0x20000)/File(\EFI\Microsoft\Boot\bootmgfw.efi)57494e444f5753000100000088000000780000004200430044004f0042004a004500430054003d007b00390064006500610038003600320063002d0035006300640064002d0034006500370030002d0061006300630031002d006600330032006200330034003400640034003700390035007d00000000000100000010000000040000007fff0400

Boot2001* EFI USB Device RC

Boot2002* EFI DVD/CDROM RC

Boot2003* EFI Network RC

(deck@steamdeck ~)$ efibootmgr -b 0002 -A

efibootmgr: Boot entry 2 not found

Could not set active state for Boot0002: No such file or directory

Some things I have tried:

Changing the boot order. Just changes itself back when restarting
Reinstalling rEFInd

Any ideas how to disable the windows efi?

There is one more thing that you should have tried, reading the README.
Closing this as it's basically a duplicate of solved issue 12.
Disabling the Windows EFI entry now requires being done from the command line when booted from the SteamOS recovery image, or being done from Windows using something like EasyUEFI trial or a proper bcdedit command. This is addressed in the updated README file.


IDE Window Size

Make the IDE Window size bigger  and add a close button .

Same for the paraview window.


Bug: Open last workfile does not work with AVALON_SCENEDIR

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior:
OpenPype inherited the feature from Avalon to specify an AVALON_SCENEDIR which acts as the subfolder in which workfiles should be saved in the work directory for a shot. It's basically what this issue describes.
For example as can be seen here the workfiles directory then becomes: AVALON_WORKDIR/AVALON_SCENEDIR.
The "open last workfile on launch" feature does not work with AVALON_SCENEDIR subfolder for work files because it does not consider the subfolder at all. Any pointers on how this could best be fixed?
Expected Behavior:
With AVALON_SCENEDIR defined in an application's environment the subfolder should be used to look for existing workfiles.
An alternative would be to completely deprecate the AVALON_SCENEDIR behavior to begin with and instead implement #2567 in such a way that the same feature can still be used but in a different manner, e.g. by having a scenedir template/profile per application.
Version
3.15.11-nightly.1
What platform you are running OpenPype on?
Windows
Steps To Reproduce:

Enable launch last workfile (= enabled by default)
Add "AVALON_SCENEDIR": "scenes" to e.g. the houdini or maya environment
Save a workfile with the Workfiles tool.
Restart the application, it should launch the existing workfile directly but it does not. (because it doesn't check in the subfolder but the work root instead)

Are there any labels you wish to add?

[X] I have added the relevant labels to the bug report.

Relevant log output:
No response
Additional context:
Issues with the AVALON_SCENEDIR have come up before so I'm linking theses issues as related:

Differentiate workdir and workfiles directory
API: Query workfiles for an asset
Workfiles tool to show all files

[cuID:OP-6160]

Closing this since AVALON_SCENEDIR has no matching functionality in AYON and hence does not exist anymore,


Mappings of an imported ADM catalog can be lost

As @pleacu described, going through:

Import from /atlasmap/ui/test-resources/adm/mockdoc.adm
Export the ADM (Export Icon on toolbar)
Reset All (Panel Settings > Reset All)
Import from your Downloads folder the adm you exported in step 2

The resulting mapping file has no mappings inside.

I'm suffering the same issue with the react UI.  This clearly used to work - don't know if it was functioning prior to your big PR (no blame - just curious when it last worked).  It appears to be related to how exportMappingsCatalog() works (and sanitizeJSON).  I think it needs a JSON styringify but it appears it is getting one...

Actually i just saw the problem. No its backend related to how i am overwriting the current catalog.    So as soon as i push the fix it will resolve both angular and react.

Cool - look forward to it! :)


Can I build for x86_64 on an arm64 device?

Just a question, can I build for x86_64 on an arm64 device?
Originally posted by @TechStudent11 in https://github.com/joshiemoore/snakeware/issues/162#issuecomment-939354882

Cross-compilation should work fine, that's a big part of what buildroot does. Just as we can build images targeting the RPi on x86, you should be able to build for x86 from an ARM host.
I'm closing this because I just enabled the "Discussions" tab so people can ask questions there instead of opening issues.


Small Feature Map

Nice work. I am thinking to train this using lighter bacbone network like tiny darknet. The problem is the output feature map from tiny darknet is of size (128,15,15) for input (3,255,255) and size (128,7,7) for input (3,127,127). So now after DepthwiseRPN, i get feature map whose size is (10/20,9,9). As anchor generation depends on this feature map size, so total number of anchors here will be 5x9x9 i.e 405 which is relatively less compared to alexnet (5x17x17 , 1445). So will this impact the regression for bbox output or is there a way to use smaller feature map size so that lesser anchors can be used.

It really depends on your search region. If your search region is not too big, I believe the reduce of number of anchors should not have a very significant influence.

OK so i use search region of let's say 180, it will compensate for lesser anchors. I will give it a go.
Thank you for the replay.

OK so i use search region of let's say 180, it will compensate for lesser anchors. I will give it a go.
Thank you for the replay.

No worries, sorry for this late reply, and let us know the outcome~

One thing, if i try to reduce image search size, than the feature map size will further reduces hence making more less anchors. So   i think, its not going to help in that?
what you prefer @zyc-ai

One thing, if i try to reduce image search size, than the feature map size will further reduces hence making more less anchors. So i think, its not going to help in that?
what you prefer @zyc-ai

It highly depends on your task as it is actually a hypterparameter. So if your objects move fast, smaller image size and larger search region would have been better, vice versa.

Since no further question is asked, I'm closing this issue for now.
Please create a new issue should you have other questions.


Make a dynamic banner for the repo README

Feature Description-
I would like to design a banner for the repo README with dynamically generating stats of the repo. It would complement this amazing repo.
I would like to work on this under NWoC.

Feature Description-
I would like to design a banner for the repo README with dynamically generating stats of the repo. It would complement this amazing repo.
I would like to work on this under NWoC.

That would be really nice!

Feature Description-
I would like to design a banner for the repo README with dynamically generating stats of the repo. It would complement this amazing repo.
I would like to work on this under NWoC.

That would be really nice!

Feature Description-
I would like to design a banner for the repo README with dynamically generating stats of the repo. It would complement this amazing repo.
I would like to work on this under NWoC.

That would be really nice!

Can you add NWoC label? I'll  make the PR, Thanks :)

Feature Description-
I would like to design a banner for the repo README with dynamically generating stats of the repo. It would complement this amazing repo.
I would like to work on this under NWoC.

That would be really nice!

Can you add NWoC label? I'll  make the PR, Thanks :)

Can you update the Leaderboard please! Ty

Can you update the Leaderboard please! Ty

Can you update the NWoC Leaderboard please! Ty

Done

Can you update the NWoC Leaderboard please! Ty

Done


Unable to use videogular2 in angular2-meteor project (bounty!)

I'm unable to use videogular/videogular2 in a recent angular2-meteor project. (This was working in earlier versions of angular2/angular2-meteor/meteor).  I suspect there is a packaging issue with videogular2 which makes it incompatible with angular2-meteor but despite my attempts to debug this I cannot figure it out.  I'm offering a $100 bounty to anyone who can help me solve this.  I have a minimal reproduction of the issue here:
https://github.com/slapfive/todos-meteor-1.3
and here is the link to the bounty:
https://www.bountysource.com/issues/38027022-unable-to-use-videogular2-in-angular2-meteor-project
Thanks in advance for any help!

@langfors did you solve it?

@Urigo  no, I coudln't solve it but I believe that it's a duplicate of videogular/videogular2#249 which implicates Angular2


Wrong serialisation of point at infinity?

Hi,
I may be wrong but I think the serialisation of the point at infinity is wrong. I would expect the two high order bits to be set (compressed = true, infinity = true), but converting Z1 to a private key has only the compressed bit set:
800000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000
As opposed to:
C00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000
I thought I'd double check by deserialising again and comparing the points:
console.log(Z1);
console.log(publicKeyToG1(publicKeyFromG1(Z1)));

This produces the following:
Point {
  x: Fp { _value: 1n },
  y: Fp { _value: 1n },
  z: Fp { _value: 0n },
  C: [Function: Fp] {
    ORDER:<PHONE_NUMBER>221667393417789825735904156556882819939007885332058136124031650490837864442687629129015664037894272559787n
  }
}
Point {
  x: Fp { _value: 0n },
  y: Fp { _value: 2n },
  z: Fp { _value: 1n },
  C: [Function: Fp] {
    ORDER:<PHONE_NUMBER>221667393417789825735904156556882819939007885332058136124031650490837864442687629129015664037894272559787n
  }
}

Thanks
Mark

Hi Mark,
This seems to be the case. We need to fix it.

👍
I believe using projective coordinates the point at infinity is generally represented as (0, 1, 0). I'll try to take a look later today to see if I can provide a PR to fix this.

Hi,
Finally got around to working on this here. I've fixed G1 serialisation (it wasn't a lot of code in the end!), I just want to review G2 and the de-serialisation routines then I'll send over a PR.
Mark

Perfect!


[🚀site]: migrate menu example page to playground base

As an ESL site holder, I want to have the menu page to be migrated to the UIPlayground base.
The current page is not proposed to be saved as a draft page.

Additional scope:
It is proposed to create 2 types of Menu demos:

"Menu Light (on basic Toggleables)" - the current example (based on ESLPanel)
(new) "Menu" - same example visually but using ESLPopup

As an additional note:
Feel free to update demo styles for menu. E.g. replace blue color to something more neutral


Fixing Date Time issue as seen in v3.1.0

SUMMARY

Fix was Merged #25318, but issue is still reproducible on Superset version 3.0.0 & 3.1.0
:ERROR:superset.views.base:Excel does not support datetimes with timezones. Please ensure that datetimes are timezone unaware before writing to Excel. Traceback (most recent call last): File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1823, in full_dispatch_request rv = self.dispatch_request() File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1799, in dispatch_request return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args) File "/usr/local/lib/python3.9/site-packages/flask_appbuilder/security/decorators.py", line 95, in wraps return f(self, *args, **kwargs) File "/app/superset/views/base_api.py", line 127, in wraps raise ex File "/app/superset/views/base_api.py", line 121, in wraps duration, response = time_function(f, self, *args, **kwargs) File "/app/superset/utils/core.py", line 1454, in time_function response = func(*args, **kwargs) File "/app/superset/utils/log.py", line 255, in wrapper value = f(*args, **kwargs) File "/app/superset/charts/data/api.py", line 256, in data return self._get_data_response( File "/app/superset/charts/data/api.py", line 412, in _get_data_response result = command.run(force_cached=force_cached) File "/app/superset/commands/chart/data/get_data_command.py", line 45, in run payload = self._query_context.get_payload( File "/app/superset/common/query_context.py", line 101, in get_payload return self._processor.get_payload(cache_query_context, force_cached) File "/app/superset/common/query_context_processor.py", line 600, in get_payload query_results = [ File "/app/superset/common/query_context_processor.py", line 601, in <listcomp> get_query_results( File "/app/superset/common/query_actions.py", line 224, in get_query_results return result_func(query_context, query_obj, force_cached) File "/app/superset/common/query_actions.py", line 186, in _get_results payload = _get_full(query_context, query_obj, force_cached) File
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@betodealmeida could you review this PR given that you added the # timezones are not supported comment?

This is still awaiting input from @betodealmeida, but in the meantime it also needs a rebase, and whatever adjustments are needed to pass CI. I'll convert this to draft mode until the touchups are made. I assume this is still an issue in 4.x, but that might warrant reproducing as well. Sorry this slipped through the cracks so long, we're trying to get better about being more timely on these things :)


Knative-ingress-gateway changes are overwritten by operator

Describe the bug
I changed my configuration as described on https://knative.dev/docs/serving/using-a-tls-cert/
On next reconcile my changes are lost.
Expected behavior
The operator should not overwrite the settings.
Otherwise the documentation should extended how to make the changes persist using KnativeServing CRD
To Reproduce
Make some changes on Knative-ingress-gateway and wait for the next reconcile. The added -hosts-Section will disappear
Knative release version
v0.21

If you install knative serving with the operator, configure it with the operator.
In terms of Knative-ingress-gateway, you should configure it as the instructions here: https://knative.dev/docs/install/operator/configuring-serving-cr/#configuration-of-knative-ingress-gateway
If you want to change the default content of knative-ingress-gateway, operator reconciling loop will revert it, because it loads the default knative-ingress-gateway each time and apply in the loop.

It also reminds me something similar in the old archive:
https://github.com/knative/serving-operator/pull/258
https://github.com/knative/serving-operator/issues/302
Time to revive.


Router with a regexp doesn't handle request in the correct layer

In case we define a layer that matches path with a regexp not from the beginning of the path, the router doesn't handle it in the correct way.
Here is a minimum reproducible example:
var app = express();

app.use(/\/api.*/, function(req, res){
  res.end('api')
});

app.use(function(req, res){
  res.end('error')
});

request(app)
.get('/test/api/123')
.expect('api', done);

This test is gonna fail becuase instead of api response, we will get error
This is happening because in the trim_prefix function, the router has a small bug: https://github.com/expressjs/express/blob/e1b45ebd050b6f06aa38cda5aaf0c21708b0c71e/lib/router/index.js#L291
It always tests for the / or . character from the beginning of the path, even though in this example, the layerPath variable is set to /api/123.
The funny thing, this will work correctly if you have the same amount of characters after the last / symbol as in the prefix (everything before the matched layerPath). So if you try /test/api/1234 it will work correctly.
I am preparing a pull request to fix this behavior.

In general this is on purpose as app.use is prefix matching only, while app.all would be to match anywhere in the path.

In this case, if you make a request to /test/api/1234 it will match against /\/api.* and you will get "api" as a result.

Hmm. Can you add that to the test? Seems an issue if the req.url is not fqdn or starting with a forward slash with this new case.
Another thing to check is what this new behavior does when req.urlis assigned within your app.use.
Ideally all of these different app.use features should be working if this now allows requests in that it did not before.

Right, because for your example app.all is what you should be using and not app.use as app.use is only to match based on the start of the path and not anywhere in the path. Is there a reason you're not able to use app.all ?

Our intention of using app.use was to match based on the start of the path, but it doesn't work like that in the case I have described.
I will copy my comment from PR here and we can continue the discussion in this thread:
To add a bit more context to it, we faced this bug in prod in our app. We have defined two layers:
app.use(/(\/foo|\/api)/, handlerA)
and
app.use(/\/proxy/, handlerB)
and when we were making a request to /proxy/api/12345 it was processed by handlerA instead of habdlerB but only in case we have 5 characters after the last / symbol. In all other cases, it was processed by hanlderB as expected.
We have fixed it by adding ^ to both regexps (e.g. /(^\/foo|^\/api)/ and /^\/proxy/)
OK, if app.use should only match based on prefix, the proper fix would be to add ^ to the regexp if the user didn't specify it (I think the same logic is already implemented somewhere in the code, because if I specify app.use('/api'... internally it will be replaced with a regexp /^\/api\/?(?=\/|$)/i

When I was preparing my PR I didn't know app.use is supposed to match only by prefix. In this case, app.use should never match /proxy/api/whatever against /\/api/ regexp

Sorry for the confusion, we have our regexp defined like this: /\/api.*/ (I have updated my previous comments)

I have updated the PR (reverted the change and added a unit test to showcase the problem)


Add support for trailing logic after @file:Include

This allows for running logic after the file include annotation.
For example: kscript '@file:Include("file.kt"); myFunction()'.

I'm not yet in favor of this PR, because

it is not a generic enhancement for annotation processing and just applies to @file:Include. Since we don't properly parse scripts but rather use regexes to parse out configuration from annotations, I think its cleaner to assume config annotations to be present one at time/line.

Allowing for multiple statements opens the box of Pandora, e.g. what about @file:Include("foo.kt"); @file:Include("bar.kt")?

I simply can't imagine a use case for this new feature. Could you provide some examples?

I'm not yet in favor of this PR, because

it is not a generic enhancement for annotation processing and just applies to @file:Include. Since we don't properly parse scripts but rather use regexes to parse out configuration from annotations, I think its cleaner to assume config annotations to be present one at time/line.

Understand if that's the case.

Allowing for multiple statements opens the box of Pandora, e.g. what about @file:Include("foo.kt"); @file:Include("bar.kt")?

Yeah, not great.

I simply can't imagine a use case for this new feature. Could you provide some examples?

I can run what I'm doing as:
$ kscript '@file:Include("file.kt")
myFunction()'

But I wanted to not have the bash newline, hence this PR.
I'm intending to replace some bash scripts in AWS CodeBuild with Kotlin / kscript.  With CodeBuild you have phases and sequences, so for each phase and sequence, I just want to include my common file (that has other includes and stores common logic) and then call a method specifically written for that phase and sequence.  No main methods, just plain invocation.
Feel free to close if you think it's best not to follow this path. 😄

We will keep the PR open to allow other users to share their opinion as well. As detailed out above I'm not convinced that the feature increment outweighs the additional complexity in parsing/inclusion logic
For the meantime, a workaround might be to use a custom interpreter for your aws usecase. See https://github.com/holgerbrandl/kscript/blob/master/docs/user_guide.md#create-interpreters-for-custom-dsls for details and an example


out of date

please update so it is no longer out of date
love this little helper

issue resolved in the 5:30 1/17 update


Network.evolve() & Neat.evolve() not mutating network

Description
Running network.evolve() & Neat.evolve() both are producing silent failures or not evolving the the network.
To Reproduce
Using Node.js v10.15.3 and running the following code, produces the following error.
Expected behavior

none of the created networks to throw an error
the error value to change

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
There is a large possibility that a large part of this error is coming from the fact that because the output neurons are no longer mutable and the activation functions thereof default to sigmoid, regression output outside of the (0, 1) can create huge network training issue - like experienced here.

@luiscarbonell do you have a code snippet that will cause this error? There have been some recent changes to Neat in an experimental branch and I'd like to see if this bug was resolved.


Bug in CHANGELOG for covariant example (?)

I was confused by this snippet:
predator(new Seal()); // Cats can't eat seals!

From https://github.com/dart-lang/sdk/blob/master/CHANGELOG.md
Should that be predator.chaseAndEat(new Seal()) ?

cc @munificent

Cool, thanks for the fast turnaround!


CONTRIBUTING.md

@paul-gauthier, just trying to help organize so I thought I would add a couple items of documentation and places for you to fill in if you want.

Thanks for the nudge. I've been meaning to add a CONTRIBUTING.md file. I probably can't use the PRs you created, as they form a very particular opinion. But I'll try and make some progress on this.

I added a contributing file. I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.

Thanks it looks good to me!


Support multiple versions

What would you like to be added:
Docforge should have a support for documentation with multiple versions.
/kind enhancement
Subtask of this enhancement:

[ ] Instead of version param, docforge should support two different params: mainBranch and supportedNTags
The first param will be for the main branch of the documentation for example master.
The second param will be an integer which will say how many releases the docforge will also support for the
documentation: for example 3. This will mean that docforge will build 4 versions of the same documentation. One for
the main branch and 3 for the latest 3 releases of the documentation.
[ ] Implement a logic where docforge will get and sort the latest N major releases based on the supportedNTags param.
If the releases are: 1.1.0, 1.1.1, 1.2.0,1.2.1, 1.3.0,1.3.1,1.3.2, the latest 3 major releases should be: 1.3.2, 1.2.1, 1.1.1.

[ ] Implement such a logic in the Git Handler
[ ] Implement such a logic in the GitHubHandler

[ ] Docforge should combine the main branch and the supported 3 versions in one variable (for example versions) and to
pass this variable to the manifest file. This will make it easier to parse the versions in the manifest file.
The variable should look in the following way: versions=master,1.3.2,1.2.1,1.1.1.
Notice that the main branch stays on index 0. This will also help when the manifest arguments are parsed.

/assign @Kostov6


Alignment rules missing for rescue/ensure

Currently there seems to be no cop to complain about the following.
begin
  a
      rescue
        b
            ensure
              c
end

And if we start with:
begin
a
rescue
b
ensure
c
end

...and auto-correct, Style/IndentationWidth (alone) makes it to look like this (by default):
begin
  a
  rescue
    b
  ensure
    c
end

PRs welcome! :-)

Ok, I'll try to submit one soon as I happened to do some groundwork already.  :) I guess it could be a new Lint cop that aligns the rescue and ensure keywords with the corresponding end without introducing any configs. EndAlignment and DefEndAlignment already do pretty good job to get that end aligned correctly. @bbatsov, @jonas054 Any comments at this point?

Sounds like a plan to me. I've nothing further to add at this point.


Add first piece of code

generates dna single strand of user specified length and GC-content

Very nice initial work. Thanks @grmnezlkv !


[release-1.3] perf(releasefile): adjust resource quota

What this PR does / why we need it:
Add your description
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to https://github.com/caicloud/log-pilot/pull/94

Special notes for your reviewer:
/cc @lichuan0620

Release note:

NONE

/lgtm


keyboard manager bug

Microsoft PowerToys version
0.69.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce

after I set the manager, it doesn't work as expected. for example, in the screenshots, you can see I set ctrl+q to alt+F4 ,but the program
would not be shut when I press ALT+Q or CTRL +Q . I still need to press alt+f4 to shut my program
✔ Expected Behavior
I hope the keyborad manager could successully work when I press the shortcut keys as I set before
❌ Actual Behavior
only the alt to ctrl and the ctrl to alt successfully work , the bottom setting doesn't act
Other Software
No response

/dup #25281


Binding a parameter via the pipeline can break implicit parameter-set selection

@ili101 discovered that binding a parameter via the pipeline can cause PowerShell to select the wrong parameter set under certain circumstances.
The issue can be resolved by designating a default parameter set.
Arguably, PowerShell shouldn't pick a default parameter set at all in this case, but it does:

defensibly with direct arguments,
seemingly incorrectly with conceptually equivalent pipeline input

Note that it is legitimate use case to have an explicit parameter set that has no mandatory parameter (set A below): You may want to group multiple parameters as logically related, even if none of them is individually mandatory.
Steps to reproduce
function foo
{
  [CmdletBinding()] # DefaultParameterSetName='A' would make the issue go away
  Param(
    [Parameter(ParameterSetName = 'A')]            $A1,
    [Parameter(ParameterSetName = 'A')]            $A2,
    [Parameter(ParameterSetName = 'B', Mandatory)] $B1,
    [Parameter(ParameterSetName = 'B')]            $B2,
    [Parameter(ValueFromPipeline)]                 $InputObject
  )
  $PSCmdlet.ParameterSetName
}

# OK: Argument-less invocation selects 'A'
foo  | Should -Be A
# OK: Binding -InputObject by direct argument also selects 'A'
foo -InputObject 1 | Should -Be A
# FAILS: Binding -InputObject via the pipeline inexplicably selects 'B'
# Just press Enter to answer the prompt.
& { 1 | foo }      | Should -Be A

Expected behavior
All tests should pass.
Actual behavior
The 3rd test fails: It unexpectedly selects set B prompts for a value for -B1.
Expected strings to be the same, but they were different.
String lengths are both 1. Strings differ at index 0.
Expected: 'A' But was:  'B'

Environment data
PowerShell Core 7.0.0-preview.6

Still repros for me; following exactly what this user has written:
> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.3.9
PSEdition                      Core
GitCommitId                    7.3.9
OS                             Microsoft Windows 10.0.19045
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

function foo {
    [CmdletBinding()] # DefaultParameterSetName='A' would make the issue go away
    Param(
        [Parameter(ParameterSetName = 'A')]            $A1,
        [Parameter(ParameterSetName = 'A')]            $A2,
        [Parameter(ParameterSetName = 'B', Mandatory)] $B1,
        [Parameter(ParameterSetName = 'B')]            $B2,
        [Parameter(ValueFromPipeline)]                 $InputObject
    )
    $PSCmdlet.ParameterSetName
}

# OK: Argument-less invocation selects 'A'
foo
# OK: Binding -InputObject by direct argument also selects 'A'
foo -InputObject 1
# FAILS: Binding -InputObject via the pipeline inexplicably selects 'B'
# Just press Enter to answer the prompt.
& { 1 | foo }


Pre-built docker image?

Looks like it takes around 1 min for GitHub (example) to build the current docker image that we use, and even more importantly; it does it every time for every repo that uses this action - which is not ideal.
Would have expected GitHub to cache that if it is from the same version or revision of an action, maybe that is not implemented yet?
What we could do however is to build and publish a docker image from this repo and refer to that instead in the action. Requires a bit more set up however when versioning, but should be doable. Alternatively can have the cargo-deny repo itself build and publish the images and simply just use the right one here in the action, that is likely the cleanest

Build time with the rust:alpine image is down to 25 sec now, so  this would be nice but not as important anymore


DCOS-8978: Fixing bug that drops empty environment variables

The JSON.stringify function is dropping keys with undefined values. This pull request makes sure that no environment variable value is in this state.

Reviewer: @Poltergeist

Hey @wavesoft, welcome to the team and congratulations on your first DC/OS PR!
Can we have a corresponding unit test in ServiceUtil-test.js please?

Looks good, feel free to merge once tests pass.
Note: That ServiceUtil.js file is not understandable and the functions in there are abnormally large. I created a ticket to clean it up.

mesosphere-ci retest this please