Integrate all Debugger error information into a convenient Tree widget.

Fixes #15234 by vertically splitting the Errors box and introducing a read-only TextEdit to display the information which was previously only available through the tooltip in the errors ItemList.
Also inserts missing " " after "Type:" in the detailed information, and sets stretch_ratio on a couple of the boxes to give more space to the Errors and the Details by default, but since everything is contained in a SplitContainer, the user is free to resize them as needed.

Can you fix the conflicts? If you need help, feel free to reach out to us on the development IRC (#godotengine-devel on Freenode)

This is a great improvement, sorry for the delay in re-reviewing it after the 3.0 feature freeze. Once conflicts are fixed I would definitely consider it for merging.

This one is pretty cool, if it can be salvaged, then great

Closing as the PR has conflicts and no update so it can't be merged for the time being. It's a nice feature though and a new rebased PR would be welcome, either from OP or from another contributor extending the work in this PR (which is what the salvageable label means).


fix: add port name to deployment and service

based on #673 but also removes the targetPort from the service.yaml and adds the port name to the deployment.yaml

/approve

Hey @alxndr13 :wave:   thanks for your contribution


Crash after inactivity or for other unknown reason

Eel version
0.16.0
Describe the bug
After some time of inactivity in the app it crashes (server stops) I thinks it needs something like tcpkeepalive or websocket switch or at least displaying a reason of why the app stopped
To Reproduce
Steps to reproduce the behavior:

Launch an App
Wait from 5 to 15 minutes
Try to refresh the page (F5)

I have the same issue currently. I created my own custom logging and added a javascript keep alive message that is sent to python and python adds it to my logging but I get no errors or any other debug or info logged when the webpage crashes.


[FEATURE] Use Rancher authentication for Harvester

It takes a lot of effort to create an authentication solution, so with the Rancher pre-installed with Harvester (#513), we should try to reuse the Rancher authentication instead of creating Harvester own.
It should be transparent to the user. They shouldn't need to configure Rancher directly to make it work.

after enabling the rancher auth provider, the existing imported downstream Harvester will encounter the following issue:
"groups \"github_org://9343010\" is forbidden: User \"system:serviceaccount:cattle-impersonation-system:cattle-impersonation-user-ppg9k\" cannot impersonate resource \"groups\" in API group \"\" at the cluster scope"

And the error logs from the Rancher Server:
2021/07/26 05:16:17 [ERROR] error syncing 'fleet-local': handler namespace-auth: Operation cannot be fulfilled on clusterroles.rbac.authorization.k8s.io "p-j2467-namespaces-readonly": StorageError: invalid object, Code: 4, Key: /registry/clusterroles/p-j2467-namespaces-readonly, ResourceVersion: 0, AdditionalErrorMsg: Precondition failed: UID in precondition: 9ffdae4a-4768-4607-8322-9ee339ca0df2, UID in object meta: , requeuing
2021/07/26 05:16:17 [ERROR] error syncing 'fleet-default': handler namespace-auth: Operation cannot be fulfilled on clusterroles.rbac.authorization.k8s.io "p-j2467-namespaces-readonly": StorageError: invalid object, Code: 4, Key: /registry/clusterroles/p-j2467-namespaces-readonly, ResourceVersion: 0, AdditionalErrorMsg: Precondition failed: UID in precondition: 2856f008-598f-4af6-9e9f-e565812bab0e, UID in object meta: , requeuing
2021/07/26 05:16:18 [ERROR] error syncing 'fleet-local': handler namespace-auth: Operation cannot be fulfilled on clusterroles.rbac.authorization.k8s.io "p-j2467-namespaces-edit": StorageError: invalid object, Code: 4, Key: /registry/clusterroles/p-j2467-namespaces-edit, ResourceVersion: 0, AdditionalErrorMsg: Precondition failed: UID in precondition: de4d73d4-e15c-44e0-a0ba-4bcc7fb98c1e, UID in object meta: , requeuing
2021/07/26 05:16:18 [ERROR] error syncing 'fleet-local': handler namespace-auth: Operation cannot be fulfilled on clusterroles.rbac.authorization.k8s.io "p-j2467-namespaces-readonly": StorageError: invalid object, Code: 4, Key: /registry/clusterroles/p-j2467-namespaces-readonly, ResourceVersion: 0, AdditionalErrorMsg: Precondition failed: UID in precondition: c7b469dc-4886-4280-afe6-3d088cb3cdd7, UID in object meta: , requeuing
2021/07/26 05:16:18 [ERROR] error syncing 'cattle-fleet-clusters-system': handler namespace-auth: clusterroles.rbac.authorization.k8s.io "p-j2467-namespaces-readonly" already exists, requeuing
2021/07/26 05:16:18 [ERROR] error syncing 'fleet-default': handler namespace-auth: clusterroles.rbac.authorization.k8s.io "p-j2467-namespaces-readonly" already exists, requeuing
2021/07/26 05:16:18 [ERROR] error syncing 'fleet-default': handler namespace-auth: Operation cannot be fulfilled on clusterroles.rbac.authorization.k8s.io "p-j2467-namespaces-readonly": StorageError: invalid object, Code: 4, Key: /registry/clusterroles/p-j2467-namespaces-readonly, ResourceVersion: 0, AdditionalErrorMsg: Precondition failed: UID in precondition: 140dd5d3-42c6-47ac-9611-3ceee5fb28d1, UID in object meta: , requeuing
2021/07/26 05:16:18 [ERROR] error syncing 'fleet-default': handler namespace-auth: Operation cannot be fulfilled on clusterroles.rbac.authorization.k8s.io "p-j2467-namespaces-readonly": StorageError: invalid object, Code: 4, Key: /registry/clusterroles/p-j2467-namespaces-readonly, ResourceVersion: 0, AdditionalErrorMsg: Precondition failed: UID in precondition: 140dd5d3-42c6-47ac-9611-3ceee5fb28d1, UID in object meta: , requeuing
2021/07/26 05:16:18 [ERROR] error syncing 'fleet-local': handler namespace-auth: clusterroles.rbac.authorization.k8s.io "p-j2467-namespaces-readonly" already exists, requeuing

@guangbochen It is an upstream issue: https://github.com/rancher/rancher/issues/33798

please help to verify using the latest rancher release and the newOS.

Test Info

Verified version:

Rancher: v2.6.1-rc10
Harvester: v0.3.0-rc1

Environment: rancher and harvester are running in its own vm respectively

Test case

start rancher using docker in a vm and start harvester in another
import harvester into rancher from "Virtualization Management" page
On rancher page, go to "User & Authentication" -> "Auth Provider" and choose Github
Follow the instructions on the page and eventually copy over client id and client secrets from github
Log out rancher and see a new option "Log in with Github". Click on it to log in.
Go to harvester cluster from "Virtualization Management" page, and double check the cluster is healthy and I am able to create a vm

This is a bit confusing. When you state Rancher, do you refer to the rancher that is embedded into Harvester, or a separate install of Rancher that is using harvester as it's Virtualization Management?
We are needing to have harvester login utilize Azure AD, is this just as simple as going into the Harvester embedded rancher UI and configuring Azure AD authentication provider?

This is a bit confusing. When you state Rancher, do you refer to the rancher that is embedded into Harvester, or a separate install of Rancher that is using harvester as it's Virtualization Management?

AFAIK it means a separate install of Rancher that has imported the Harvester cluster. There's some documentation about this at https://docs.harvesterhci.io/v1.3/rancher/virtualization-management#multi-tenancy. I haven't tried this myself, but AIUI, you'd hook that Rancher up to use Azure as its auth provider, then assign users/groups to the appropriate cluster and/or project roles.

Why would we not just enable the auth provider within the existing embedded rancher for harvester? I don't understand the need to import harvester's rancher as a downstream rancher of and existing rancher install.

Because harvester's use of its embedded rancher is a restricted use case; we don't want to expose all of its functionality. There's actually work in progress to simplify Harvester's dependencies (see https://github.com/harvester/harvester/issues/5278) which may involve removing or replacing parts of the embedded rancher.

But we should not have install Rancher just to be able to use auth providers for harvester. We should be able to enable the auth provider within harvester, which intern enables it for the embedded Rancher.

The in built rancher is only mean for bootstrapping harvester and subsequent upgrade cycles. It does have a few features disabled. We recommend against using it for auth and management of more downstream clusters. Among other things this rancher is tied to the harvester packaging and that makes out of band upgrades of rancher difficult as you could end up in unexpected scenarios. If you are really keen on running rancher without VM's on harvester, we do recommend using the rancher-vcluster addon.

I think there is some confusion. I am not referring to using the embedded rancher for anything downstream. I am asking about using an auth provider to login to the harvester UI, instead of having to create all admins as harvester users.

If you are really keen on running rancher without VM's on harvester, we do recommend using the rancher-vcluster addon.

I have no idea what you are trying to say here. How did we get to talking about running Rancher not in harvester? We have a few harvester clusters that are running nothing but ordinary VMs, no Rancher VMs installed in the cluster at all. Just simple VM cluster management.

What we're trying to say is that the way the Harvester project has chosen to support external authentication providers is via a separate Rancher instance that has imported the Harvester cluster, and that you cannot use the Rancher UI embedded inside Harvester to enable external authentication providers.
Here's a screenshot of the embedded Rancher UI:

Note that it's only showing cluster resources. There's no bar down the left where you'd ordinarily find the link to the Users & Authentication screen where you'd go to set up an external authentication provider in Rancher proper.
If you don't wish to run a Rancher instance elsewhere in order to configure external auth for a given Harvester cluster, it's possible to run a separate Rancher instance directly on the Harvester cluster itself using the rancher-vcluster addon as mentioned above. You'd import the Harvester cluster(s) into that rancher instance, and have that Rancher instance configured to talk to an external authentication provider. Your users would then login to that Rancher instance and access Harvester via the Virtualization Management interface.

@tserong correct, I am pointing out that it is a huge oversight to force an install of Rancher in order to get SSO within harvester. Especially when the underlining technology that harvester is running on top of, already supports such a functionality.

It's not an oversight, it was a deliberate choice.
Using a separate Rancher instance has other benefits, notably multi-cluster management. You mentioned earlier that you have multiple Harvester clusters. If you also ran one Rancher instance, you'd be able to hook that Rancher instance up to your authentication provider, then use it to provide SSO for all your Harvester clusters, via a single pane of glass.
Further discussion probably really belongs on https://github.com/harvester/harvester/issues/4024


Strings with length constraints / Custom scalars in mutation input

Hi,
I am trying to create a mutation input that would have strings with length constraints.
What would be the best way to implement this?
Is it possible to have constraints for fields like: String @max(length: 10)?
Currently I am trying to implement these length constraints with custom scalars, for example String10.
In generated schema and input params these fields are defined response::Value.
How do you get this custom scalar from these values?

I'm not sure how to build that constraint into a schema definition, but you could certainly define your own directive and interpret it as you see fit in the resolvers. That would mostly be useful for documenting the schema though, because there's no way to tell cppgraphqlgen what your directive means.
Generally if I were going to do something like this, I'd check the length in the resolver/apply method. If the limit is exceeded, I'd throw a std::runtime_error or std::invalid_argument exception from the resolver/apply method with an error message for the caller. The graphqlservice library should catch that and bubble it up as a field error in the GraphQL results.
If you want to use a scalar type, that's also kind of opaque to cppgraphqlgen, and to most tooling. If you are returning it, whatever response::Value you build will get converted to JSON and included in the GraphQL results for that field. If you pass it as an input parameter, I think it will convert whatever value you pass to an equivalent response::Value argument/member, and it will be up to you to interpret it. You could pass strings that way, but there aren't conversion hooks that you could use to centralize that limit. If you want to apply it in multiple places, your best bet would probably be to extract a function to do that check and then call it from any resolvers/apply methods which use that type.

Thanks, I think I'll create a function to check for specific lengths in the apply methods 🙂
Seems to work great:
"message": "Field error name: createTestObject unknown error: exampleField should be under 5 in length"

Great, I'll go ahead and close the issue then.


AttributeError: module 'tensorflow._api.v1.compat' has no attribute 'v2'

Hi i faced this issue while running "python train.py --logtostderr --train_dir=training/ --pipeline_config_path=training/faster_rcnn_inception_v2_pets.config"
Traceback (most recent call last):
File "train.py", line 52, in
from object_detection.builders import model_builder
File "C:\Users\User\Anaconda3\envs\tensorflow1\lib\site-packages\object_detection-0.1-py3.6.egg\object_detection\builders\model_builder.py", line 47, in
from object_detection.models.ssd_mobilenet_edgetpu_feature_extractor import SSDMobileNetEdgeTPUFeatureExtractor
File "C:\Users\User\Anaconda3\envs\tensorflow1\lib\site-packages\object_detection-0.1-py3.6.egg\object_detection\models\ssd_mobilenet_edgetpu_feature_extractor.py", line 19, in
from object_detection.models import ssd_mobilenet_v3_feature_extractor
File "C:\Users\User\Anaconda3\envs\tensorflow1\lib\site-packages\object_detection-0.1-py3.6.egg\object_detection\models\ssd_mobilenet_v3_feature_extractor.py", line 25, in
from nets.mobilenet import mobilenet
File "C:\Users\User\Anaconda3\envs\tensorflow1\lib\site-packages\object_detection-0.1-py3.6.egg\nets\mobilenet\mobilenet.py", line 399, in
def global_pool(input_tensor, pool_op=tf.compat.v2.nn.avg_pool2d):
AttributeError: module 'tensorflow._api.v1.compat' has no attribute 'v2'
I'm using CPU
Tensorflow 1.13.1

Hi, I faced the same problem, did you solve it?

Yes i solved the issue by cloning the models-1.13.0

Could you please tell me how you solve this problem? Thanks

Because of your using tensorflow version and tensorflow models version difference. You should download right version from tensorflow models repository from here.
You can select version from branch section.
Example: İf you use TensorFlow 1.5.0 you should download TensorFlow models r1.5.


godaddy

Expected content
Actual content
Link to page:

hi i have error when i try to register my dns in godaddy.
Add all of these records to GoDaddy's DNS section.
but i have error because they not seens like dns

Hello @jensysantana
We're sorry you're having this issue!
Make sure you're only entering the host name minus the domain at the end.
For example: url0007 and not url0007.domain.com or s1._domainkey and not s1._domainkey.domain.com.
Points to* - Enter the value as provided by SendGrid.
If this doesn't work for you, please log into https://support.sendgrid.com and file a ticket.
SendGrid Docs Team


remove quotes around blocks of IPs in CIDR notation in egress rules

@vadimeisenbergibm: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
b64fa8ad8457576ba71349af2a32a3f3ecba89f0
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e

@vadimeisenbergibm: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
b64fa8ad8457576ba71349af2a32a3f3ecba89f0
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e

/lgtm


Picker jump

The picker jump, there is a scrollbar for a few mseconds. The solution:
.picker__holder {overflow-y:hidden;}
hope this is right?!

Yep - it's a bit of a tough situation to get around because of the transitions. I'll keep this around if anyone can come up with an elegant solution to it.


MRG: Add CSD/PSD equiv test

Adds the test requested by @agramfort and removes a bunch of deprecation warning noise from tests. Also changes the default for half_nbw is None to match elsewhere in MNE.

These indented blocks are really just adding indentation (to catch a
warning)  and will be removed once 0.16 is released, so it's probably not
necessary to improve them

Okay, params unified

thx a lot @larsoner


Fix issue #8:  Use corect careers URL

What

Fix of #8
Replaced an incorrect careers URL with the correct one.

Updated Flow
Upon clicking the current open positions hyperlink, the user is now directed to the valid careers page.

Fixed it now ;)
Oops.


Datadog docker compose file format version 2 error

Hi! I got this error when I have tried to change the format of docker compose file to the second version. Do you have an example of datadog docker-compose.yml for newer format?
➜  docker-compose-example docker exec dockercomposeexample_datadog_1 cat /var/log/datadog/dogstatsd.log
2016-03-20 19:39:59 UTC | INFO | dd.dogstatsd | utils.pidfile(pidfile.py:31) | Pid file is: /opt/datadog-agent/run/dogstatsd.pid
2016-03-20 19:39:59 UTC | INFO | dd.dogstatsd | daemon(daemon.py:157) | Starting
2016-03-20 19:39:59 UTC | INFO | dd.dogstatsd | daemon(daemon.py:171) | Pidfile: /opt/datadog-agent/run/dogstatsd.pid
2016-03-20 19:39:59 UTC | INFO | dd.dogstatsd | dogstatsd(dogstatsd.py:329) | Listening on host & port: ('', 8125)
2016-03-20 19:39:59 UTC | INFO | dd.dogstatsd | dogstatsd(dogstatsd.py:160) | Reporting to http://localhost:17123 every 10s
2016-03-20 19:40:09 UTC | INFO | dd.dogstatsd | dogstatsd(dogstatsd.py:205) | Flush #1: flushed 0 metrics, 0 events, and 0 service check runs
2016-03-20 19:40:24 UTC | ERROR | dd.dogstatsd | dogstatsd(dogstatsd.py:269) | Unable to post payload.
Traceback (most recent call last):
  File "/opt/datadog-agent/agent/dogstatsd.py", line 259, in submit_http
    r = requests.post(url, data=data, timeout=5, headers=headers)
  File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py", line 107, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py", line 53, in request
    return session.request(method=method, url=url, **kwargs)
  File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py", line 468, in request
    resp = self.send(prep, **send_kwargs)
  File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py", line 576, in send
    r = adapter.send(request, **kwargs)
  File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/adapters.py", line 449, in send
    raise ReadTimeout(e, request=request)
ReadTimeout: HTTPConnectionPool(host='localhost', port=17123): Read timed out. (read timeout=5)
2016-03-20 19:40:24 UTC | INFO | dd.dogstatsd | dogstatsd(dogstatsd.py:205) | Flush #2: flushed 1 metric, 0 events, and 0 service check runs

docker-compose.yml example
version: "2"
services:
  web:
    build: web
    command: python app.py
    ports:
     - "5000:5000"
    volumes:
     - ./web:/code
    links:
     - redis
  redis:
    image: redis
  datadog:
    build: datadog
    links:
     - redis
    environment:
     - API_KEY=__your_datadog_api_key_here__
    volumes:
     - /var/run/docker.sock:/var/run/docker.sock
     - /proc/mounts:/host/proc/mounts:ro
     - /sys/fs/cgroup:/host/sys/fs/cgroup:ro

I solved it by adding network_mode: 'bridge' to the redis and datadog services but I am not sure that it is right solution.

Thanks @AndrewSamokhvalov. I just updated the README to compose v2 format. I wasn't able to replicate the issue (maybe it's been resolved in Compose), but I'm going to leave this issue open in case others encounter it.


feat: menu improvements

This pull request implements a few improvements to the menu/menu item component:

Fixes an issue where the menu items were incorrectly focused when hovering between them while using shouldFocus with the Menu component. See video below of the issue.
Introduces a new prop called closeOnClick to the MenuItem component. This prop makes it possible to avoid closing the menu (used with MenuButton) when a MenuItem is clicked.

https://github.com/sanity-io/ui/assets/15094168/d82364e0-a74a-44f2-aef2-e09542f036e7

@hermanwikner
To clarify – in what use case do we not want to close the menu when a selection is made?


Add reagent sources to the guidebook

About the PR

As we quickly approach a future where we can no longer rely on our dear chem dispenser to give us every reagent we need, it's important to better utilize our alternative collection methods. One main issue, however, is that people have no clue what these are.
This PR seeks to remedy this by adding a new "sources" tab for each entry in the chem guidebook. The currently supported sources are:

reactions marked with source: true (electrolysis and centrifuge reactions added in my prior pr)
juiceable entities
grindable entities
gases that can be condensed.

This covers most of the alternative methods of obtaining chemicals outside of cargo.
Minor unrelated changes:

fixed capitalization in mixing verb text
catalysts are now preserved in reaction displays.
extractablecomponent is now in shared

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking Changes
ExtractableComponent is now in shared.
Changelog

:cl:

add: The chemistry guidebook now includes sources for various chemicals! These are alternative methods of collection, including grinding, juicing, condensation, and various others.

gource

sorse 2

Me, a tider, watching the chemist steal all the cables from maints leaving the station unpowered:


Add support for arm64 trees

Yes, done: https://github.com/ClangBuiltLinux/ClangBuiltLinux.github.io/pull/21


[#2473] refactor: enable UnusedVariable error-prone

What changes were proposed in this pull request?

enable  UnusedVariable error-prone

Why are the changes needed?
Fix: #2473
Does this PR introduce any user-facing change?

no

How was this patch tested?

./gradlew build

@coolderli Could you fix the issue of CI?


Hana: create procedure fails with syntax error at END if there is a loop inside.

System information:

Win10
DBeaver 21.2.0

Connection specification:
Sap Hana
Describe the problem you're observing:
Simple procedure cannot be created due to supposed syntax errors.
Steps to reproduce, if exist:
Run the SQL Script:
create or replace procedure abc
      (in x SMALLINT,
       in y SMALLINT)
as
begin
    declare loop_x SMALLINT;
    declare loop_y SMALLINT;

    declare maxID BIGINT;

    for loop_y in 1..:y do
        for loop_x in 1..:x do
            maxID := maxID + 2;
        end for;
    end for;
end;

It seems the parser sends only the string
create or replace procedure abc
      (in x SMALLINT,
       in y SMALLINT)
as
begin
    declare loop_x SMALLINT;
    declare loop_y SMALLINT;

    declare maxID BIGINT;

    for loop_y in 1..:y do
        for loop_x in 1..:x do
            maxID := maxID + 2;
        end for

truncating the statement after the end-for's semicolon char.
Error message is
SQL Error [257] [HY000]: SAP DBTech JDBC: [257] (at 302): sql syntax error: line 14 col 13 (at pos 302)

and the create statement is obviously valid, it works in Hana Studio.

Hello @wernerdaehn
Thanks for the bug report!
You can try to highlight your procedure statement in SQL Editor and use Ctrl+Enter to avoid block parsing problems.
Now we, unfortunately, cannot test this case, because the connection to the database you gave us is no longer working. If you have the opportunity to back our access to the database - it would be very cool!

I have shutdown the server but can start it at any time. It is just that I do no longer need it 24x7. Please send me an email when and for how long I should start the server and I will do so.

Thanks for your help. Your database was very useful for us.
Anyway. I see a blank line in your statement.
Can you please tell me - the "Blank line is statement delimiter" is enabled or disabled in your preferences?

Yes, it was checked. It tried with and without (opening a new editor after the change to be safe).
I also removed the blank lines from the procedure code.
Behavior is always the same, the code sent to the database ends at the line end for;

Fixed in DBeaver 21.2.2

Veified


[tobi] Cannot see focused elements on Product Carousel

When tabbing through product carousel, we cannot see the products that are focused off the screen. The carousel should switch to the next/previous slide so that we can see the focused product.

during fixing visibility, we can add arrow keys functionality (to go through the products in carousel) note: add that to auxiliary keys tests also

currently @tobi-or-not-tobi is taking care of this fix. he mentioned that this was already implemented, but is not working. he will investigate

Issue seems unrelated to visible focus in a strict sense, since -as per screenshot- the elements that are visible do have visible focus, therefore moving to BACKLOG.

This will be fixed in another ticket (not yet created)


Update and canonicalize source-build champions list

It's been a while since we've updated the source-build champions list and there are a couple versions floating around.  We should update this for 7.0.

[Triage] With the increased importance of source-build, the champions concept has not been utilizes in the past two releases now.  CODEOWNERS is the preferred contact mechanism as it is not repo specific.  Any board communication or escalation should be utilizing tactics.


Toolbox in Display

I open this to discuss how the Toolbox display should work.

Before I can form my mind about this I would like to make some questions:

Why do we need to select zoom/plot, can't it be active all time? There is a historical comment about this not being the best way. What was the original problem that motivated going for this solution?
Shouldn't restoring default be a button in the same dialog used for modifying it, do we need a general restore?
I would like some examples of custom functionalities that would be added in this box. Depending on that I could suggest to put those in the main menu. Also if they are really complex, they could have a display of it own: reusing code is important, but when the cost of doing that is too big, we should not be afraid of coping, even things that look similar in the beginning can evolve to become really distinct.

I'll try to answer your questions the best I can.

The issue with Zoom/Pan was that the wheel scroll was being interpreted as a tilt or file change.  I think this had something to do with the way the original ComboBox menu was structured.  I was having similar problems (up-down arrows were going through the ToolBox ComboBox menu instead of being related to tilt) when I put it back in yesterday.  After changing to the drop down menu, this behavior went away.
A second reason is that if using a touch mouse, they can be quite sensitive and send a signal out when not intended.

I have no objections against a restore for each button.  I only put the general restore in to get back to default conditions if mucking around with various tools.  This is mostly due to my laziness.

There are a couple of examples that will hopefully be coming soon.  I'll post again here when I add one.  My feeling is the main menu is becoming less useful as modifications will be done on a per display basis.  (At least that's the way I would want to use it.).  The tool that I'll include is a masking function, so you could apply a mask on one display (say removing all reflectivity under 15 dBZ) and compare to the other non-modified display.

Thank you for the explanation @nguy, I will need to reflect on this a little.

I would go the other direction.  I would make the onPick (value at a point)  into a Class and add it to the toolsBox.  (Actually I have done this on my local copy.)  My reasoning is two-fold.  First I feel that we should keep the core program as simple as we can.  The core program being just display the data.  That would help to keep things modular.  Second, I am trying to add a "select a region" class and I needed to turn off the onPick to free up the mouse for my "select a region" code.  And thinking off in the future a ways, I would put the toolsBox Classes in a separate folder to make it easier for others to add their own special Classes to fulfill their needs.  That is my thinking.

I had not thought about the mouse.  That's a good point.
I am still having trouble with the arrow keys not responding after certain operations.
Maybe a short term solution would be to create a tools.py file that holds the tools and then we can move this over into a separate folder once everything is working properly?

@nguy - I am good with a tools.py file.  That would be a good separation.

I wouldn't go so fast in separating a file, this can deceiving, having separated files but interdependent code. I would suggest new tools to be added as naive as possible in separated branches, so we can analyze how it works and how it could interact with the rest of the program. @pfhein if you already have that "select a region" in some where, I would be interested in having a look.
There is no reason why arrow keys couldn't work with zoom/pan, is just a questions of correcting focus. Several widgets, as buttons, handle arrow key (this is poorly described in the Qt documentation) and as tilt are handle in the window layer, anything can get in the way. But as I said, this is just a question of finding what is the problem and fix it.
About the filter, I have talked in issue https://github.com/ARM-DOE/pyart/issues/244 about pseudo-variables. That was how I implement filters in my personal project and the discussion in the issue indicates how this can be done in pyart. But if we were to implement this way, we would need a new class (as tilt window but more complex) to govern it, adding and removing pseudo-variables as to avoid misuse.  Eventually this class could use the main window to display it self, avoiding a new floating window.

I implemented a different masking procedure in AWOT - another package I've been working on - but pretty much extended the existing PyArt to be more comprehensive.  I'll take another look at that issue thread.  I scanned it, but had not read it closely.
What I'll do the tools.py is create the file, but not link it.  That way we can see if it looks like something that might work.  I'll push another commit here shortly with a few fixes.

I decided to wait until the next push to include tools.py.

@gamaanderson - Here is my broken code with all its warts.  There are 2 versions there with the second version commented out.  Both versions are not working.
#from matplotlib.widgets import LassoSelector
from matplotlib.lines import Line2D
from matplotlib.path import Path

##########################
# Select Region of Interest Class Methods #
##########################
class ROI:
    '''
    Select a Region of Interest: The code modified from
<EMAIL_ADDRESS>    '''

    def __init__(self, ax, fig, tilt, display):
        self.previous_point = []
        self.start_point = []
        self.end_point = []
        self.line = None
        self.verts = []
        self.ind = []
        self.xys = []
        self.xpts = display.x[:,tilt]
        self.ypts = display.y[:,tilt]
        self.xys = np.array([self.xpts,self.ypts]).transpose()
        self.fig =  fig
        self.fig.canvas.draw()

    def motion_notify_callback(self, event):
        if event.inaxes:
            ax = event.inaxes
            x, y = event.xdata, event.ydata
            if event.button == None and self.line != None: # Move line around
                self.line.set_data([self.previous_point[0], x],
                                   [self.previous_point[1], y])
                self.fig.canvas.draw()
            elif event.button == 1: # Free Hand Drawing
                    line = Line2D([self.previous_point[0], x],
                                  [self.previous_point[1], y])
                    ax.add_line(line)
                    self.previous_point = [x, y]
                    self.verts.append([x, y])
                    self.fig.canvas.draw()

    def button_press_callback(self, event):
        if event.inaxes:
            x, y = event.xdata, event.ydata
            ax = event.inaxes
            if event.button == 1:  # If you press the right button
                    if self.line == None: # if there is no line, create a line
                        self.line = Line2D([x,  x],
                                           [y, y],
                                           marker = 'o')
                        self.start_point = [x,y]
                        self.previous_point =  self.start_point
                        self.verts.append([x, y])
                        ax.add_line(self.line)
                        self.fig.canvas.draw()
                    # add a segment
                    else: # if there is a line, create a segment
                        self.line = Line2D([self.previous_point[0], x],
                                           [self.previous_point[1], y],
                                           marker = 'o')
                        self.previous_point = [x,y]
                        self.verts.append([x, y])
                        event.inaxes.add_line(self.line)
                        self.fig.canvas.draw()

            elif event.button == 3 and self.line != None: # close the loop
                        self.line.set_data([self.previous_point[0],
self.start_point[0]],
                                           [self.previous_point[1],
self.start_point[1]])
                        ax.add_line(self.line)
                        self.fig.canvas.draw()
                        self.line = None
                        path = Path(self.verts)
                        self.ind = np.nonzero([path.contains_point(xy) for xy in self.xys])[0]
                        print self.ind
                        print "Closed Loop"
                        print self.verts

    def connect(self):
        self.motionID = self.fig.canvas.mpl_connect('motion_notify_event', self.motion_notify_callback)
        self.buttonID = self.fig.canvas.mpl_connect('button_press_event', self.button_press_callback)

    def disconnect(self):
        self.fig.canvas.mpl_disconnect(self.motionID)
        self.fig.canvas.mpl_disconnect(self.buttonID)

#class RegionSelect:
#    '''
#    Class for Selecting a region of points
#    '''
#    def __init__(self, ax, radar, tilt, display):
#        self.xpts = display.x[:,tilt]
#        self.ypts = display.y[:,tilt]
#        self.ax = ax
#        self.tilt = tilt
#        self.radar = radar
#        #Need to have a 2D array of r and 2D array of theta for each point
#        #arr1 = np.zeros(shape=(len(self.radar.range['data']),self.radar.sweep_end_ray_index['data'][self.tilt]-self.radar.sweep_start_ray_index['data'][self.tilt]+1,1))+1
#        #arr1 = np.ones(shape=(self.radar.sweep_end_ray_index['data'][self.tilt]-self.radar.sweep_start_ray_index['data'][self.tilt]+1,1))
#        #rng_pts = arr1 * self.radar.range['data']
#        #azarr = (np.zeros(shape=(1,self.radar.sweep_end_ray_index['data'][self.tilt]-self.radar.sweep_start_ray_index['data'][self.tilt]+1))+1) * self.radar.azimuth['data'][self.radar.sweep_start_ray_index['data'][self.tilt]:self.radar.sweep_end_ray_index['data'][self.tilt]+1]
#        #az_pts = arr1 * azarr
#        #xpts = rng_pts * np.sin(az_pts)
#        #ypts = rng_pts * np.cos(az_pts)
#
##  need matplotlib.collections.PathCollection object
#        self.lasso = LassoSelector(ax, onselect=self.onselect)
#
#    def onselect(verts):
#        print verts

Thank you @pfhein.
Is it possible that the only thing you need directly from Display is self.fig, if that is the case, it could really work independent of Display. The only question is if fig need be a V variable, I have the impression it does not. If you need a graphical place, it could be in the main menu, but we still have to make it receptive for external tools.
As you are connecting with fig, and onPick is connecting with canvas, they may conflict. But if we get them to connect to the same instance they could work together (no need to deactivate one to use the other), lets see how this turns out.

@pfhein and @gamaanderson  I put the above code into the tools.py file and got it working. See pull request #24
One thing about this method is that you select the area with a left click and close the region with a right click.  Not sure yet how to get this info to user.
The next step would be to figure out what to do with this region.  I'll probably open another issue for this.


Reenable development build, uses upstream libraries

It's been suggested that we reenable development builds, using some libraries from upstream. We'll try here to find a useful yet reliable way of testing those cases for everyday use.
@jakirkham @jcrist @mrocklin @hameerabbasi

[ ] Tests added / passed
[ ] Passes flake8 dask

@pentschev, I pushed a commit to try and resolve the merge conflict introduced by merging PR ( https://github.com/dask/dask/pull/4525 ). Hope that is ok.
Based on your comment in the aforementioned PR, I think we should just pin Numba to the last know working version (i.e. 0.41.0) for now to fix the CI.

Thanks @jakirkham, I think this is probably the best solution we can get for now!

If someone could get me a reproducer (with versions), I'd like to check once more whether this is an issue with sparse or Numba.

The CI seems to reproduce it, but am having trouble locally. It may be I'm not reproducing the same environment yet.
FWIW I noticed that numba in conda-forge is pinned to numpy version 1.15.*, which could be related (as the same version shows up on CI). This would mean we have a version of numpy that doesn't support the __array_function__ protocol.

Hmm. That should have no bearing on whether or not sparse works...

@hameerabbasi sorry, I don't think I understand your question, versions of what? What I observed from Travis' log seems to indicate the most likely cause is the new version of Numba.
This is a diff (only the relevant part, I omitted overhead of packages that are exactly the same) I compiled from the passing Python 3.7 build, and failing development builds:
--- pass        2019-04-12 22:53:53.505918282 +0200
+++ fail        2019-04-12 22:53:53.737914841 +0200
-arrow-cpp                 0.13.0           py37h73cd1b8_0    conda-forge
+arrow-cpp                 0.11.1           py37h5c3f529_0
-boost-cpp                 1.68.0            h11c811c_1000    conda-forge
-brotli                    1.0.7             hf484d3e_1000    conda-forge
-certifi                   2019.3.9                 py37_0    conda-forge
+certifi                   2019.3.9                 py37_0
-dask                      1.1.5                      py_0    conda-forge
+dask                      1.1.5                      py_0    conda-forge
-gflags                    2.2.2             hf484d3e_1001    conda-forge
+gflags                    2.2.2                he6710b0_0
-glog                      0.3.5             hf484d3e_1001    conda-forge
+glog                      0.3.5                hf484d3e_1
+libboost                  1.67.0               h46d08c1_4
+libevent                  2.1.8                h1ba5d50_0
-libprotobuf               3.7.1                h8b12597_0    conda-forge
-llvmlite                  0.26.0          py37hdbcaa40_1000    conda-forge
+llvmlite                  0.28.0           py37hdbcaa40_0    conda-forge
-numba                     0.41.0          py37h637b7d7_1000    conda-forge
+numba                     0.43.1           py37he1b5a44_0    conda-forge
-numpy                     1.16.2           py37h8b7e671_1    conda-forge
+numpy                     1.17.0.dev0+96d892a           <pip>
-pandas                    0.24.2           py37hf484d3e_0    conda-forge
+pandas                    0.25.0.dev0+390.ge02ec8f89           <pip>
-parquet-cpp               1.5.1                         2    conda-forge
+parquet-cpp               1.4.0+20.nightly               0    twosigma
-pyarrow                   0.13.0           py37h0978efd_0    conda-forge
+pyarrow                   0.11.1           py37he6710b0_0
-re2                       2019.04.01           he1b5a44_0    conda-forge
-snappy                    1.1.7             hf484d3e_1002    conda-forge
+snappy                    1.1.7                hbae5bb6_3
-sparse                    0.7.0                     <pip>
+sparse                    0.7.0+6.g948cee7           <pip>
-thrift-cpp                0.12.0            h0a07b25_1002    conda-forge
+thrift-cpp                0.11.0               h02b749d_3
-zict                      0.1.4                      py_0    conda-forge
+zict                      0.1.4                      py_0    conda-forge
-zstd                      1.3.3                         1    conda-forge
+zstd                      1.3.3                h84994c4_0

Hmm... This is weird. The only sparse dependency in that list is Numba, and I can't seem to reproduce the error with the exact same Numba version and sparse master...
The entire test suite passes for me, other than the distributed failures I already mentioned in another thread.

@hameerabbasi I've adapted a Dockerfile that I use sometimes to reproduce Dask Travis' builds locally for debugging, I was able to reproduce the errors we see in this PR. If you would like to try, please make a copy of https://gist.github.com/pentschev/dcbe78364640f3e143795f03c6faaed8 in an empty directory dask-devbuild or some other name that you'd prefer and run it as follows:
cd dask-devbuild
docker build -t dask-devbuild .
docker run -it dask-devbuild /bin/bash -c "cd /usr/src/dask; source /root/miniconda/etc/profile.d/conda.sh; conda activate test-environment; source ./continuous_integration/travis/run_tests.sh"

Let me know if that works for you.

To make the build an allowed failure, you need to update:
  allow_failures:
    - env: *py36_dev
    - os: osx

to
  allow_failures:
    - env: *py37_dev
    - os: osx

Now it looks like Travis completely ignored building this PR, but I don't see why that happens.

Looks good to me, good to merge?

Maybe a test commit to check it?

LGTM. I guess we can merge.

Let's just wait for this last build to pass, which seems like it will now (allowed failure properly flagged by Travis), after that, it can be merged.

LGTM. Merging. Thanks @pentschev!

Thanks everyone for the reviews!


Keptn API: Retrieve metadata of Keptn installation

Right now there is no way to retrieve meta-data information from Keptn via the API, such as:

Which namespace Keptn is installed in (e.g., passed as an environment-variable to the api)
(Future / not yet available) Name/Label of the Keptn installation - please define the variable and set it to "keptn" for now
Installed Keptn Version (e.g., api-service has an environment variable for it)
Installed Keptn Bridge Version (can be the image tag of the docker image) - This might require an RBAC rule
(Future / not yet available) List of Keptn-services + which version they are installed in

However, this information can be manually collected using a lot of kubectl commands.
Definition of Done

[ ] Create an authenticated endpoint in the Keptn api that returns the above information
[ ] Also add this endpoint as a check that is executed when calling  keptn generate support-archive command
[ ] Create an integration test for it (or edit an existing one)

I could take this one.

@agrimmer can you support with any RBAC issues here?

Sure! Will help.

Hey,
I've some questions regarding the API...
What I did so far:

Added a /metadata definition in the swagger.yaml
Added a response model in "response_model.yaml#/definitions/metadata"
Executed as described in the README.md swagger generate server -A api -P models.Principal -f ./swagger.yaml

After running the swagger generate command a lot of not related files are changed diff.txt
So my questions:
Was my approach to add the endpoint to the API Service wrong?
Which version of the swagger should I use (I've installed v0.19.0)
swagger.yaml
  /metadata:
    get:
      tags:
        - Metadata
      operationId: metadata
      summary: Get keptn installation metadata
      responses:
        200:
          description: Success
          schema:
            $ref: "response_model.yaml#/definitions/metadata"

response_model.yaml
  metadata:
    type: object
    required:
      - token
    properties:
      token:
        type: string
      namespace:
        type: string
      keptnversion:
        type: string
      bridgeversion:
        type: string

Christian

Hi @checkelmann,
thank you for implementing this issue and your question!
Your swagger.yaml looks good. Also the response_model.yaml looks good except for the token. Which token would you like to return here? I guess you can remove it.
I have installed swagger version v0.22.0.
It is normal behavior that swagger generates many changes. But can you please point me to your branch? Then I could have a closer look at the generated source code?
Thanks,
Andreas

Hi @agrimmer

thank you for implementing this issue and your question!
Your swagger.yaml looks good. Also the response_model.yaml looks good except for the token. Which token would you like to return here? I guess you can remove it.

Ah ok, then I misunderstood the required parameter, I thought I need the token for the Authentication.

I have installed swagger version v0.22.0.
It is normal behavior that swagger generates many changes. But can you please point me to your branch? Then I could have a closer look at the generated source code?

I will try with your swagger version and push it to my https://github.com/checkelmann/keptn repo later. When I've done it, I will ping you on Slack.
Cu Later...

Hello again,
what you defined in the response_model is the return of the API call. Furthermore, you automatically get an authenticated endpoint because of the securityDefinitions in https://github.com/keptn/keptn/blob/master/api/swagger.yaml#L14, i.e. without providing a correct x-token in the header, no metadata will be returned.
I will check out your code.
Cheers, Andreas

Hi,
as discussed in the community meeting we should talk about the purpose of this endpoint.
@christian-kreuzberger-dtx mentioned this endpoint should be used for the create support archive command, but when the API is not reachable, this support archive will not be able to capture all the data as it does right now using kubectl commands.
If it should be an endpoint for other purposes, it will be slow and more expensive when scanning all deployments within the keptn namespace from the API Service.
@christian-kreuzberger-dtx @agrimmer @bacherfl
What are your thoughts?

My opinion: I think we should still add it to the support-archive, even though the information is redundant.
In the future we might add new information to that endpoint, therefore this could contain non-redundant information. In addition, it helps in trouble-shooting as we could spot mismatches between what the API endpoint returns vs. what the CLI was able to detect locally.

I also think it would make sense to have both the information from the CLI and the API Endpoint to detect any discrepancies between those.


AKS OIDC issuer URL clarification request

Hi,
In Get the OIDC Issuer URL section there's the following text:

By default, the Issuer is set to use the base URL https://{region}.oic.prod-aks.azure.com/{uuid}, where the value for {region} matches the location the AKS cluster is deployed in. The value {uuid} represents the OIDC key.

It mentions that issuer URL is https://{region}.oic.prod-aks.azure.com/{uuid} while in reality it is https://{region}.oic.prod-aks.azure.com/{tenant_uuid}/{uuid} as in the example output (same section).
My question is what is the {uuid} in the example above.
The text states that it is a OIDC key - but what sort of key is it? Could it be a wording and actually it's a key ID instead? If so what ID is it?
Main concern I have is if it is in any way a value that shall remain secret (which I do hope it's not) since it's an issuer URL.
Another concern is uncertainty if this {guid} would change in any way when rotating the signing key.

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

ID: deab7eb6-9bbf-28ea-f8c3-4150e3c5d076
Version Independent ID: 0ae60dcc-0a65-20fc-e303-e4af9690e132
Content: Create an OpenID Connect provider for your Azure Kubernetes Service (AKS) cluster - Azure Kubernetes Service
Content Source: articles/aks/use-oidc-issuer.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@herbatnik
Thanks for your feedback! We will investigate and update as appropriate.

@herbatnik it is actually a randomly generated guid for each cluster that is immutable.

Awesome, many thanks for confirming it 😄
Please do update documentation with this information if possible

@herbatnik - I've created PR https://github.com/MicrosoftDocs/azure-docs-pr/pull/256348 to update the article with this information. I'm going to proceed with closing this issue. Thanks.
#please-close


performance degradation when Tree.tokens is called repeatedly

v0.20.0 represents a significant performance hit in comparison with v0.1.0-RC4. We need to check what's going on and fix it. My hypothesis is the malfunction of the newly introduced tokenization cache.

Fixed in https://github.com/scalameta/scalameta/pull/409.


feature: Disable which-key UI

Did you check the docs?

[X] I have read all the which-key.nvim docs

Is your feature request related to a problem? Please describe.
I do Youtube videos and in videos which-key popup looks like screen flicker when keymaps are executed fast. Such as dhf I use to delete inside function. When timeoutlen is set to a long time like 3000ms and press d it wait's 3000ms to show up but If I do dh popup shows up immediately.
vim.o.timeout = true
vim.o.timeoutlen = 3000

I obviously want all the keymaps to be added through which-key. However, I want to temporary disable the UI popup.
Describe the solution you'd like
A WhichKeyDisableUI WhichKeyEnableUI & WhichKeyToggleEnableUI commands to enable disable UI
Describe alternatives you've considered
Could not find a way to disable UI.
Additional context
This is how the keymaps has been registered through lazy.nvim
return {
  'nvim-treesitter/nvim-treesitter',
  opts = {
    textobjects = {
      select = {
        enable = true,
        lookahead = true,
        keymaps = {
          ['ak'] = { query =<EMAIL_ADDRESS>desc = 'around block' },
          ['hk'] = { query =<EMAIL_ADDRESS>desc = 'inside block' },
          ['ac'] = { query =<EMAIL_ADDRESS>desc = 'around class' },
          ['hc'] = { query =<EMAIL_ADDRESS>desc = 'inside class' },
          ['a?'] = {
            query =<EMAIL_ADDRESS>            desc = 'around conditional',
          },
          ['h?'] = {
            query =<EMAIL_ADDRESS>            desc = 'inside conditional',
          },
          ['af'] = {
            query =<EMAIL_ADDRESS>            desc = 'around function ',
          },
          ['hf'] = {
            query =<EMAIL_ADDRESS>            desc = 'inside function ',
          },
          ['al'] = { query =<EMAIL_ADDRESS>desc = 'around loop' },
          ['hl'] = { query =<EMAIL_ADDRESS>desc = 'inside loop' },
          ['aa'] = {
            query =<EMAIL_ADDRESS>            desc = 'around argument',
          },
          ['ha'] = {
            query =<EMAIL_ADDRESS>            desc = 'inside argument',
          },
        },
      },
    },
  },
}

+1
I just found this issue while trying to find a way to toggle the UI. The plugin is really helpful when I need it, but distracting when it pops up accidentally. I would like to explicitly toggle it on when I need it.
Thanks for a great plugin!

+1
I have also struggled with this. When i cannot remember a particular key combination this plugin is amazing, however most of the time i do not have the need for it to show up and i find that just setting up a bigger timeout delay is not the optimal solution.
Great plugin either way!


Use dotnetcli and markdown fixes

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/install-guide/update.md
87
66
12
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/install-guide/index.md
90
81
12
link
:white_check_mark:

articles/install-guide/update.md
87
66
12
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 0a367c5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/install-guide/index.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/install-guide/update.md
:white_check_mark:Succeeded
View (qsharp-preview)

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/how-to-guides/create-qsharp-project.md
92
72
7
link
:white_check_mark:

articles/install-guide/index.md
90
81
12
link
:white_check_mark:

articles/install-guide/update.md
87
66
12
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 8a49987:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/how-to-guides/create-qsharp-project.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/install-guide/index.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/install-guide/update.md
:white_check_mark:Succeeded
View (qsharp-preview)

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/how-to-guides/create-qsharp-project.md
92
72
7
link
:white_check_mark:

articles/install-guide/index.md
90
81
12
link
:white_check_mark:

articles/install-guide/update.md
87
66
12
link
:white_check_mark:

articles/libraries/numerics/installation.md
82
45
9
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 0822328:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/how-to-guides/create-qsharp-project.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/install-guide/index.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/install-guide/update.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/libraries/numerics/installation.md
:white_check_mark:Succeeded
View (qsharp-preview)

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/how-to-guides/create-qsharp-project.md
92
72
7
link
:white_check_mark:

articles/install-guide/index.md
90
81
12
link
:white_check_mark:

articles/install-guide/update.md
87
66
12
link
:white_check_mark:

articles/libraries/numerics/installation.md
82
45
9
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 908edf6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/how-to-guides/create-qsharp-project.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/install-guide/index.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/install-guide/update.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/libraries/numerics/installation.md
:white_check_mark:Succeeded
View (qsharp-preview)

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/how-to-guides/create-qsharp-project.md
92
72
7
link
:white_check_mark:

articles/install-guide/index.md
90
81
12
link
:white_check_mark:

articles/install-guide/update.md
87
66
12
link
:white_check_mark:

articles/libraries/chemistry/installation.md
74
47
47
link
:white_check_mark:

articles/libraries/numerics/installation.md
82
45
9
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit d82674a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/how-to-guides/create-qsharp-project.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/install-guide/index.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/install-guide/update.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/libraries/chemistry/installation.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/libraries/numerics/installation.md
:white_check_mark:Succeeded
View (qsharp-preview)

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


Handle exception when refresh table since table may be dropped or re-created (#3291)

What type of PR is this：

[x] bug

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

run starrocks_be_unittest


[Bug report]: SRM: The entire balance cannot be sent

Platform
Desktop
Version
2.0.1
Description
From my wallet I wanted to send the entire balance. This was rejected. I could only send the balance minus 1375 Glow.
Expected behaviour
It should be possible for a wallet to be emptied completely. It reminds me of XRP.
Actual behaviour

Can the issue reliably be reproduced?
No
Steps to reproduce the issue
I can't say if this is reproducible. There were over 100 transactions with different tokens on the wallet.
Operating System
Mac
Diagnostics
-

Developer Profile
No
Profile Type
Hardware (Ledger)
Staking
Never staked
Crash Status
None of the above
Error Messages
no crash
Error Logs
no error

Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

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

You cannot send all of it, because some is locked as storage deposit for your native tokens.


I found this bug when I install new laravel project

PS D:\Laravel Projects> composer create-project --prefer-dist laravel/laravel admin_panel
Oops, something went wrong.  Please report this bug with the details below.


Consider GKE Autopilot

Once GKE Autopilot clusters are supported in Terraform, we should consider switching our deployment. The advantage will be that we're only charged for pod resources, not nodes (which are often not fully utilized).
Unfortunately it's not possible to convert a cluster, so we'd have to create a new cluster from scratch.

This is blocked by https://github.com/hashicorp/terraform-provider-google/issues/8553.

We'll probably not get to this anytime soon, but the Hail team is considering this as an option.


.NET 6 Project couldn't load since not found Microsoft.Extensions.DependencyInjection.Abstractions <IP_ADDRESS>

I want to build a blazor sample project as below, but always build failed.

Open Visual Studio Code
Type dotnet new blazorwasm on commander and create the sample project.
Type dotnet build.
And issue occurred below.
Could not load file or assembly Microsoft.Extensions.DependencyInjection.Abstractions, Version=<IP_ADDRESS>.

Path: c:\Users\10904246.vscode\extensions\ms-dotnettools.csharp-1.24.3-win32-x64.omnisharp\1.38.3-beta.33\OmniSharp.exe
PID: 8708
    Starting OmniSharp on Windows 6.2.9200.0 (x64)

    Checking the 'DOTNET_ROOT' environment variable to find a .NET SDK

    Using the 'dotnet' on the PATH.

    DotNetPath set to dotnet

    Located 3 MSBuild instance(s)
        1: Visual Studio Professional 2019 16.11.32228.343 16.11.2 - "C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Current\Bin"
        2: Visual Studio Build Tools 2019 16.11.32228.343 16.11.2 - "C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\MSBuild\Current\Bin"
        3: OmniSharp MSBuild 17.1.1 - "C:\Users\10904246\.vscode\extensions\ms-dotnettools.csharp-1.24.3-win32-x64\.omnisharp\1.38.2\.msbuild\Current\Bin"

    Registered MSBuild instance: OmniSharp MSBuild 17.1.1 - "C:\Users\10904246\.vscode\extensions\ms-dotnettools.csharp-1.24.3-win32-x64\.omnisharp\1.38.2\.msbuild\Current\Bin"
        MSBuildExtensionsPath = C:\Users\10904246\.vscode\extensions\ms-dotnettools.csharp-1.24.3-win32-x64\.omnisharp\1.38.2\.msbuild

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpFormattingWorkspaceOptionsProvider, Order: 0

    Detecting Cake files in 'c:\Blazor\6.0\demo'.

    Did not find any Cake files

    No solution files found in 'c:\Blazor\6.0\demo'

    Queue project update for 'c:\Blazor\6.0\demo\demo.csproj'

    Detecting CSX files in 'c:\Blazor\6.0\demo'.

    Did not find any CSX files

    Configuration finished.

    Omnisharp server running using Stdio at location 'c:\Blazor\6.0\demo' on host 31632.

    Loading project: c:\Blazor\6.0\demo\demo.csproj

    Microsoft.Extensions.Configuration.FileExtensions 6.0.0 depends on System.Text.Json (>= 6.0.0), but System.Text.Json 6.0.0 was not found. An approximate best match of System.Text.Json 5.0.0 was resolved。

    Successfully loaded project file 'c:\Blazor\6.0\demo\demo.csproj'.

c:\Blazor\6.0\demo\demo.csproj
c:\Blazor\6.0\demo\demo.csproj(0,0): Error: Microsoft.Extensions.Configuration.FileExtensions 6.0.0 depends on System.Text.Json (>= 6.0.0), but System.Text.Json 6.0.0 was not found. An approximate best match of System.Text.Json 5.0.0 was resolved。
    Adding project 'c:\Blazor\6.0\demo\demo.csproj'

    Update project: demo

Received response for /v2/getcodeactions but could not find request.
Received response for /v2/getcodeactions but could not find request.
Steps to reproduce

Open Visual Studio Code
Type dotnet new blazorwasm on commander and create the sample project.
Type dotnet build.

Expected  behavior
Should enable the sample web.
Actual behavior
Show the error below:
The type 'IServiceCollection' is defined in an assembly that is not referenced. You must add a reference to assembly 'Microsoft.Extensions.DependencyInjection.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'

After clean nuget cache and install the package, issue solved.
dotnet nuget locals all --clear
dotnet add package Microsoft.Extensions.DependencyInjection.Abstractions -v 6.0.0


Increase the Number of Names Returned from 25 to 75 When Searching for a Person in a Search Box

When searching for a person in a search box 25 names are returned but not necessarily the name being searched for.  But if the starts with a letter that falls towards the end of the alphabet, the searched for name is not available for select.  That's because there are many common names and the names are returned alphabetically.
The name search function is found in:
Reports
Report approval chain
Assigning personnel to positions
Reference item 49

We agreed that increasing the number of results is not the best solution, as there may still be many results earlier in the alphabet. Better would be to order on full-text search rank (if there are no other sorting criteria), i.e. show the most relevant hits at the top.
{Note that there are additional sorting criteria; if the search ranks are equal, the results are sorted as before, so on name ascending.]


TASK [geerlingguy.php : Ensure the default pool exists.] *********************** fatal: [drupalvm]: FAILED! =>

Issue Type

Bug Report / Support Request

Your Environment

Vagrant 2.2.7
VirtualBox 6.1.18r142142

Your OS

macOS (10.14.6)

Full console output

console-output--vagrant-up.txt
Summary

For an update of an old Drupal project I want to install PHP 7.0. The installation of DrupalVM 4.9.2 fails with the following message:
TASK [geerlingguy.php : Ensure the default pool exists.] ***********************
fatal: [drupalvm]: FAILED! => {"changed": false, "checksum": "565ef20c1a260db1be9e8e0b1b057e84a1952ea1", "gid": 80, "group": "80", "mode": "0644", "msg": "chgrp failed", "owner": "502", "path": "{{ php_fpm_conf_path }}/pool.d/www.conf", "size": 226, "state": "file", "uid": 502}
There is a full console output in the attached file.
Is there a solution for this or is it possible to install PHP 7.0 on a current version of DrupalVM?

Unfortunately, PHP 7.0 is no longer supported, and while you may be able to get it working with a lot of configuration changes, I won't be able to spend time helping with that.


Change the logo

Change the logo and customize it

Done, direct changes were uploaded to the main.


Fix for #191

This pull request also does the following:

Fix the same issue with the countLines method, although it's a quite simple fix for that method.

For this I've added the getLineEnding method, which returns the used line separator of a file.

Add better tests.

please resolve conflicts

Please request an review from @Clueliss  when ready.

Conflicts have to be resolved and the documentation now needs checked and maybe updated now, because the behaviour of the FileSeparatorQuerySource has been changed.

[ ] update documentation
[ ] resolve conflicts

I had a short look at some of the failing tests. They seem to be due to leading/trailing whitespaces/new line characters in tests that were removed before but are kept now.


Add getCacheDir and getFilesDir to Activity.mojom

I'll use them in Fitness App in a follow-up patch.
@abarth

LGTM


Introduce basic k8s configs for the mixer, prometheus, and grafana

We add configs for running the mixer via k8s, as well as prometheus and grafana. These have been tests e2e together. We also add a README that describes running these configs and how to interact with the deployment.

This change is

Codecov Report

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

@@           Coverage Diff           @@
##           master     #274   +/-   ##
=======================================
  Coverage   83.68%   83.68%
=======================================
  Files          49       49
  Lines        2831     2831
=======================================
  Hits         2369     2369
  Misses        434      434
  Partials       28       28

Impacted Files
Coverage Δ

adapter/prometheus/server.go
78.94% <ø> (-10.53%)
:x:

pkg/config/manager.go
100% <ø> (+2.77%)
:white_check_mark:

Continue to review full report at Codecov.

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

Updated per comments. Open questions are:

Do we want to move the yamls to some other dir?
Do we want to separate the mixer yaml from the others?
Do we want to remove the local registry yaml from this PR?


Problems with modal

Good Morning!
I am using the django-bootstrap3 library.
I'm working with django 1.8.6. And with the following script call:
"// ajax.googleapis.com/ajax/libs/jquery/2.2.4/jquery.min.js"
I've been using the bootstrap library for quite some time and I've never had problems ...
But now I'm working with modal and I'm encountering some problems. The "hidden.bs.modal" function is not executed.
I think it's some bug with the library.
To better simulate the problem and in another way, I opened an online javascript simulator and made the calls as http://getbootstrap.com/getting-started/. The alert message worked when closing the modal.
Function used:
$ Function () {$ ('# modal_x'). On ('hidden.bs.modal', function () {$ ('# modal_x')) removeData ('bs.modal');
//window.location.reload(true);
Alert ("Closed.");
});
});
A more complete example found on the internet: http://jsfiddle.net/84yTH/4/
Thanks any help!

Era conflito com a biblioteca "dajax/jquery.dajax.core.js"

Resolved:
BOOTSTRAP3 = {
'jquery_url': '//code.jquery.com/jquery.min.js',
}


3752 CID opt-in fix

Resolves #3752.

Using o.preferences.prefers_community_taxon === false would cause it to mishandle cases where the value is null.

@jwidness, how about o.preferences.prefers_community_taxon !== true or o.preferences.prefers_community_taxon === false || o.preferences.prefers_community_taxon === null?

I implemented a different option in the commit, but if you would prefer yours, I can change it.
For reference, this is the table I'm working off of:


Updated code to make sure the exemption is kept after undoing a tab close action.

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4905  👈
#4903
#4899
#4898
develop

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

Join @anikiki and the rest of your teammates on  Graphite


Missing API: SharedTreeEditor.setValue

Feature
The SharedTreeEditor API does not expose a setValue methods. This is inconsistent with the rest of the API which purports to support all types of edits.
Describe the solution you'd like
Add a setValue(target: NodeId, payload: Payload): EditId function to the API.
Current workaround
const editId = tree.applyEdit(Change.setPayload(target, payload));

@yann-achard-MS yes, the Change API has been improved and should be the primary method for building changes. Those can be submitted either through the applyEdit API or the Checkout API.


Accept a default httpHandler at the construct level in FetchAuthToken

I've been trying to make this work:
$httpHandler = HttpHandlerFactory::build(new GuzzleHttp\Client([
    'timeout' => 1.0,
    'connect_timeout' => 0.3,
    'read_timeout' => 0.7
]);
$client = new SpannerClient([
    'authHttpHandler' => $httpHandler
]);

From a high level this doesn't work because in gax-php GrpcCredentialsHelper.php it does not pass in the authHttpHandler function and at a higher level, none of the options get passed into GrpcCredentialsHelper because all of those get set via getGaxConfig and include none of the higher level configuration.
This fix works by setting the httpHandler in the construction of FetchAuthTokenCache.
Related: https://github.com/GoogleCloudPlatform/google-cloud-php/pull/692

Closing, see comments on https://github.com/GoogleCloudPlatform/google-cloud-php/pull/692


[SPARK-49246][SQL] TableCatalog#loadTable should indicate if it's for writing

What changes were proposed in this pull request?

For custom catalogs that have access control, read and write permissions can be different. However, currently Spark always call TableCatalog#loadTable to look up the table, no matter it's for read or write.
This PR adds a variant of loadTable: loadTableForWrite, in TableCatalog. All the write commands will call this new method to look up tables instead. This new method has a default implementation that just calls loadTable, so there is no breaking change.
Why are the changes needed?

allow more fine-grained access control for custom catalogs.
Does this PR introduce any user-facing change?

No
How was this patch tested?

new tests
Was this patch authored or co-authored using generative AI tooling?

no

Speaking of table-level privileges, it's not simply rw actually,

privilege type
desc

INSERT
for insert rows.

DELETE
for deleting rows.

SELECT
for data retrieval.

UPDATE
for changing column values.

ALTER
table, column, partition meta etc

@yaooqinn good point! I'll change it to def loadTable(ident, operationType) while operationType is an enum.

operationType shall be a collection, these types are not always used in isolation, e.g., some operations might be upsert, both read and write against the same table

also cc @huaxingao

@cloud-fan Thanks for pinging me! The new TableCatalog#loadTable API looks good to me, and it also looks good from Iceberg's perspective. Also cc @aokolnychyi @szehon-ho

The last commit just resolves a trivial merge conflicts, I'm merging this to master/3.5, thanks for reviewing!


Add Angular Elements to the "ng g library" compilation's results

🚀 Feature request
Command (mark with an x)
- [x] generate

Description
We know that the "ng g library" helps us with the creation of reusable libraries. The components we create into ous libs can be compiled into multiple JS formats (ES2015, ES5, UMD, ...), which is awesome! Thank you CLI team! However, would it be possible to have them compiled into Web Components as well?
Describe the solution you'd like
 I know that Angular Elements assist with the creation of Web Components. However, I was not able to add Elements to the libs created to "ng g library". Nonetheless, rather than adding it manually... it would be amazing if the CLI team could add that capability to the "ng g library" directly... so that this cmd could insert the necessary configuration for the ng-packager to use and automatically build Web Components alongside the ES2105, UMD, and other formats.
Describe alternatives you've considered
 None yet.

This is gonna be a really useful feature.

I'm looking forward to see that feature working. It's gonna be so useful.

This is gonna be a really useful feature

Will be very useful in tthe future.

I wait the proposed solution. Thia would gelo a lot.

In this way, we could have our components inside the libary, like:
project/custom/src/lib/
project/custom/src/lib/comp1
project/custom/src/lib/comp2
project/custom/src/lib/comp3
project/custom/src/lib/comp4

And by mixing Angular Elements and "ng g library", through the compilation process, we would achieve a result similar to:
dist/
dist/custom/
dist/custom/bundle
dist/custom/esm5
dist/custom/esm2015
dist/custom/comp1_web_component_version
dist/custom/comp2_web_component_version
dist/custom/comp3_web_component_version
dist/custom/comp4_web_component_version

Will be very useful in the hereafter.

Really looking forward to this one !!

Very excited to see this happen!

This would be very helpful. I am running into a situation where I need to be able to import Angular-generated web components into other apps as ES6 modules. I've tried creating the custom elements in separate modules within a library project and using ng-packagr to build, but it seems that it won't compile the web component unless its through an application module.

+1

Hello,
Temporary Solution
is there any temporary solution/workaround or scaffolding stuff that can help for the time being?
Generate Web Component from existing Angular Component
Moreover will it be possible to take the existing angular component from a library and build them as web component as well? So it will be possible to use the same component as Angular Component in an Angular application and as Web Component elsewhere.
My temporary solution
I just generate a library and then I just build via ngx plus, but I have to duplicate the component, any idea to avoid this?
Really thanks a lot for the hard work and the answers 🙏

This feature is highly needed. Would be great if it's added as earliest possible.

This would be amazing!! I'm ready to contribute however needed.

My temporary solution
I just generate a library and then I just build via ngx plus, but I have to duplicate the component, any idea to avoid this?

Meanwhile waiting this feature.
@blackat Can I ask if you have time to share more detailed instructions about your temporary solution to build via ngx plus. I'm not familiar with ngx-build-plus so any help is much appreciated.
This feature in official angular-cli would be super helpfull!

@blackat I'd like to ask you to provide the instructions, too! Would be very helpful, thanks in advance

This feature is highly needed. Would be great if it's added as earliest possible. We are eagerly waiting for this feature.

The command "ng g library" can assist us in the creation of a library of UI components... let's say a custom accordion, carousel, and typeahead. We can then compile that library and publish as an NPM package to be used in different Angular apps. However, if the build process was also able to compile those same components into individual Web Components, that would be perfect. The keyword here is "individual"... so the final dist folder would have a regular JavaScript file for each Web Component (accordion, carousel, and typeahead).

I want my angular element to be installable via npm. so my angular element has to be generated as a library that is published to npm. Anyone has made it this way? thanks

This feature is highly needed. Would be great to microfrontends architecture on project i'm working.

This would be a great feature to have. Especially like @fabionolasco mentioned, "individual" javascript files for each Web Component.

Brought this up with the rest of the tooling team and we agree this is a useful feature to include. We don't have bandwidth to work on this in the near future as Angular Elements has not been prioritized right now, but we would likely accept a PR implementing this feature.

Any news on this? I assume it's still not a thing and I need an application if I want to build web components?


We should add method for 'schema'

Q
A

Bug report?
no

Feature request?
yes

BC Break report?
no

RFC?
yes

Library version
0.4.1

How should that work? According to the docs one can achieve that simply by adding ?schema to the request.


xclip: 0.12-svn-20140209 -> 0.13

Things done

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

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Merged as c59d3cd849f87bb9d86aadfc6bfdc997658e4883


lookup reference field creationView

reference field should lookup reference if a default value is provided

@pelim: can you elaborate? I don't understand the issue. :)

Closing as no more feedback. Feel free to reopen if you still encounter the issue. :)


Service broker API of version 2.16 is showing the stack trace instead of single error msg line which is revealing internal code details

What is the problem?
Upon use of PUT v2/service_instances/{instance_id} by passing no body to this method it is throwing complete stack trace
Who does this affect?
This might pay way for insider attackers
Do you have any proposed solutions?
Need to check if request body is sent or not and throw only Required request body is missing instead of complete stack trace
Additional context
Add any other context or screenshots can go here.

CreateServiceInstanceRequest This is the class that is throwing this stack trace

This seems like an issue with spring-cloud-open-service-broker rather than this spec. Please open an issue in that repo.


feat(2662): 2phase incoming and outgoing payments

Changes proposed in this pull request

Address issue: https://github.com/interledger/rafiki/issues/2662

Context

Checklist

[x] Related issues linked using fixes #number
[x] Tests added/updated
[x] Documentation added
[x] Make sure that all checks pass
[x] Bruno collection updated

@koekiebox we discussed this last week and we were wondering if we could make the 2-phase transfer optional on all withdrawals. What do you think?

@koekiebox we discussed this last week and we were wondering if we could make the 2-phase transfer optional on all withdrawals. What do you think?

@sabineschaller this should not be a challenge. The accountingService.createWithdrawal will create a 1 or 2 Phase transfer depending on the timeout parameter being > 0. So we can update each of the withdrawal mutations to also have a timeout field as part of the input? Which will also mean the caller would be able to control the timeout for a 2-phase transfer. What do you think?

@sabineschaller this should not be a challenge. The accountingService.createWithdrawal will create a 1 or 2 Phase transfer depending on the timeout parameter being > 0. So we can update each of the withdrawal mutations to also have a timeout field as part of the input? Which will also mean the caller would be able to control the timeout for a 2-phase transfer. What do you think?

@koekiebox Yes, that is exactly how I thought this would work 🚀


Support for listing versions

Cider supports printing out the changelog for a version with cider describe [<version>]
This is incredibly useful for CI.
However, I have a script in which I would like to generate a changelog file for each version.
To do so, I would need to know which versions are available.
Would it be possible to add a command that prints all versions in the changelog?

Not sure I completely understand the idea. Can you maybe give a short example?

(it would also be really really nice if describe allowed getting only the body, without the title and the link at the end. perhaps as an optional flag. but maybe thats better for another issue)

Hey @clragon wanna try the list command? https://pub.dev/packages/cider/versions/0.2.0-aplha.1

that works brilliantly!

Implemented in 0.2.


E (4963) intr_alloc: No free interrupt inputs for USB interrupt (flags 0x802)

I (3933) esp_netif_handlers: sta ip: <IP_ADDRESS>, mask: <IP_ADDRESS>, gw: <IP_ADDRESS>
I (3933) example: got ip:<IP_ADDRESS>
I (3933) example: connected to ap SSID:PDCN password:3.1415926535
I (3943) example: mdns hostname set to: [esp-cam]
I (3943) example: cparam0=[<IP_ADDRESS>]
I (3953) HTTP: Start task_parameter=<IP_ADDRESS>
I (3953) HTTP: Starting HTTP server on http://<IP_ADDRESS>:8080
E (4963) intr_alloc: No free interrupt inputs for USB interrupt (flags 0x802)
ESP_ERROR_CHECK failed: esp_err_t 0x105 (ESP_ERR_NOT_FOUND) at 0x4200391a
file: "src/main.c" line 366
func: app_main
expression: initialize_usb_host_lib()
abort() was called at PC 0x4037e383 on core 0

i don't know how this happen,sometime it work fine with using wifi share by my laptop,and i turn it off,take it to my room and connect to my router,then it will broke,that's weird

Will this work in the same environment?
https://github.com/espressif/esp-idf/tree/master/examples/wifi/getting_started/station


helm rollback command is broken when specific to a namespace: Error: release: not found

terratest v0.26.3
Helm v3.1.2+gd878d4d
Doesnt take the namespace into account therefore :
Error: release: not found
I suspect you need something similar to the install/upgrade functions where the namespace is added to the args for the helm command

Add this
	args = append(args, getNamespaceArgs(options)...)


pagination color

Introduce different pagination color.Currently pagination only support primary color.
Like
'.pagination .pagination-primary'

I'm pretty sure .pagination-primary doesn't exist, .pagination is just using the primary color as his color.
The problem of creating color variants of each component is that it increases CSS bundle size (and therefore impacts performances).

pagination is most common and most useful features in all the blog website.
Hope, we will see this class soon.
.pagination-colorName    ( colorName are : -primary, secondary, success, info, warning etc. )

i agree with @dev0mhm . adding this small feature wont effect CSS bundle size in a huge manner.

I agree with @dev0mhm, if do you interesting I can to do this pull request

aye

I finished of develop this feature [pagination-color](feature https://github.com/twbs/bootstrap/pull/31482)

aye

I finished of develop this feature https://github.com/twbs/bootstrap/pull/31482

adding this small feature wont effect CSS bundle size in a huge manner.

It's not but I don't know how the core team decides about adding (or not adding) that kind of things to library core.
It doesn't look like a popular demand/feature to me so I'm advocating a mixin (you can use to generate colored pagination yourself) without adding this to the core of the lib.

I have create a pull request with mixin and class with theme colors, If interested, I can just add the mixin

I have create a pull request with mixin and class with theme colors, If interested, I can just add the mixin @Deckluhm

Usually, the pagination should look the same throughout the site.
It would be better to be able to set the color you want in variables.

Pagination colors are customizable through Sass variables, both in v4 and v5.
This has not been requested much, and as @axzx said I'm not sure it should be part of the core.
FWIW, customization might be easier soon whilst v5 moves to CSS custom properties. Using custom properties for pagination colors would make adding variant much easier.

okasss

I agree with @ffoodd, I think it's a good idea to change the color of pagination with CSS variables. I think this is generally a good direction for Bootstrap to keep going in, as it is getting to be rather large.

If will implement this https://github.com/twbs/bootstrap/issues/29853 with new versions of SCSS we could add new variants with additional colors without Bootstrap code change like:
@use '~bootstrap/scss-modular/pagination' with (
  $color: green
);

or
@include meta.load-css('~bootstrap/scss-modular/pagination', (
  color: green
));

Thanks, but for now we'll pass on adding color schemes for this. Would rather implement CSS vars for configuring the colors and direct folks to Sass variables for more.


Allow override session_id in exec

Summary
Allow overriding session_id per query
https://github.com/ClickHouse/clickhouse-js/issues/271
Checklist
Delete items not relevant to your PR:

[x] Unit and integration tests covering the common scenarios were added
[x] A human-readable description of the changes was provided to include in CHANGELOG
[ ] For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

Thanks for your contribution! Nice catch.
A few notes:

the exec method is not intended to be used with CREATE TABLE statements; the relevant method is command. The main purpose of exec is to provide a workaround for "unusual" statements that will provide a useful output (select with a default format, select into a file) and something similar; if some of the docs/examples are misleading, let's update that as well.
that said, it is worth checking if we can do a similar override in the "main" three methods - query, command, insert; I am AFK now, so can't check it quickly.
If it is indeed the case that we cannot override the session id in these methods, let's add it there, too, and enhance the integration test to use all three methods (command to create the table, insert to insert the data, query to select). Maybe the same for exec.
Note: If you use exec to create the table, don't forget to destroy the returned stream. This is done internally in the command method (though the stream is not destroyed, but "drained" instead, to preserve the keep-alive socket).

I will recreate it as a new PR from another branch (where I have write access).


Create ZKEmail Recovery Extension

As a smart contract/ERC-4437 account owner
I would like to Allow a ZKEmail account to recover (rotate) a smart contract account
So I can Recovery my account if I lose my keys or they are compromised
Details
https://zkemail.gitbook.io/zk-email/email-wallet-extensions-sdk

Create ZKEmail extension that allows a ZKEmail account to recover/rotate the keys on another non-ZKEmail smart contract account
Likely can match existing Safe recovery plugin pattern
Try and stay within the ZKEmail extension spec to ensure maximum compatibility & minimal work

Closing in favor of direct recovery work


ciao-cli: Handle the case where a user does not belong to a tenant.

When a tenant is not specified in the cli or environment variables,
ciao-cli uses getTenant function to fetch the tenantName and
tenantID the user belongs to.
getTenant was not handling the case where a user exists but does not
belong to any tenant resulting in an index out of range panic.
This commits handles this scenario and fails properly.
Fixes: #433
Signed-off-by: Alberto Murillo<EMAIL_ADDRESS>

Coverage remained the same at 65.426% when pulling 0c421ecde39c6a8e3c56f89077e0ebc9519963ab on albertomurillo:issue433 into 981bb660aac26415bd7976100eb5f337eb5a8572 on 01org:master.

Coverage decreased (-0.007%) to 65.419% when pulling 0c421ecde39c6a8e3c56f89077e0ebc9519963ab on albertomurillo:issue433 into 981bb660aac26415bd7976100eb5f337eb5a8572 on 01org:master.


What is the difference deep assigning with JSON.parse?

let deepClone = JSON.parse(JSON.stringify(objectToBeCloned))

The speed maybe? About speed: http://stackoverflow.com/a/5344074/115363
Or JSON is just for simple objects?

Different concepts. The above is actually cloning the object. This module assigns properties of an object to a new or existing object.


How does Toil interact with Kubernetes?

@laserson I'm not sure what is being asked - we have an Apache Mesos batch system implementation but nothing involving Kubernetes . Feel free to reopen with clarification if that doesn't answer your question.

Apologies for not being clearer (though I am just learning some of these tools so I don't necessarily know how they all fit in together).  According to the docs, Toil uses CGCloud to provision clusters on the cloud.  Could it instead use Kubernetes for this?

I see - I'm not very familiar with Kubernetes so I'm not sure what its capabilities are. You might be able to set up Kubernetes to provision a Toil cluster but I would recommend using CGCloud since that set up is done for you. Alternatively the next release of Toil will include an autoscaling feature that will allow the use a 'native' Toil provisioner in addition to CGCloud.

Interesting.  I am interested in Kubernetes because it seems to be the new hotness for managing clusters and especially for coordinating containers.  It also works with all the major clouds, AFAIK.

I know this a closed issue. I just wanted to mention that the new version of Kubernetes 1.5 now supports batch jobs https://kubernetes.io/docs/user-guide/jobs/
I have no idea how difficult it would be to and Kubernetes support to toil, but I think it would be a neat feature :)
Thanks!

Apologies for commenting on such an old issue -- however, with the current dominance of kubernetes in the cloud orchestration world, are there are any plans to support it in toil?


Space API

Tests:

[x] list
[x] retrieve
[ ] create
[ ] update
[ ] partial update
[ ] delete

All API endpoints have been created.

Fix auth on GET space-list


Add terraform 0.13 support

what

Update the required Terraform version to add support for v0.13.x

why

Using this module with Terraform v0.13.x is not possible unless we update the required version

Disregard this please. I did not notice that this was already covered in v0.14.0. My apologies 😓


Unable to install the cucumber in Eclipse from help< new install software. Error is throwing while installation

👓 What did you see?
Unable to connect to the repository https://cucumber.github.io/cucumber-eclipse/update-site/content.xml
✅ What did you expect to see?
Cucumber plugins should be installed in my eclipse without any error.
📦 Which tool/library version are you using?

io.cucumber
cucumber-java
7.2.3

🔬 How could we reproduce it?
1.Create the maven project.
2. Add the dependency in the pom.xml file
3. . Add the selenium, Cucumber and cucumber Junit dependency.
4. Navigate to the help<new install software  in the eclipse.
5. Enter the https://cucumber.github.io/cucumber-eclipse/update-site/ in the work with text box and enter.
6. Cucumber plugins should be installed but it is throwing error.

📚 Any additional context?

This text was originally generated from a template, then edited by hand. You can modify the template here.

Could you please look into this asap. As it is blocking our client deliverables.

Hi,
Any update on my issue?


[BUG]

The menu drawer doesn’t close after successfully navigating to the path

Yes noticed.
Can you help fix that?

Sure


Can we get the text spoken when doing ApiAi.startListening?

Hi,
Thanks for the project, helps us get a great jumpstart.
We are developing a chat application using APIAI as the backend and were wondering if there is any hook available to get the text when doing ApiAI.startListening. Typically as a chat user, it will be good to show the result of what the user spoke before getting the response from ApiAI.
Our chat app (without this) was doing a separate Google Speech-to-text plugin to do the conversion first and then sending it to the backend engine, but now that the ApiAI.startlistening is available, it makes sense to just let the backend engine do the speech to text itself, but we are not able to get the text before the response is available.
I looked inside to see if there are any hooks available, but doesnt look like it. Are there any other ways to achieve this?
Thanks!

Hi, thanks for using our lib ;)
You can still use your Speech-To-Text plugin and just send the text to api.ai:
  ApiAi.requestQuery("Some text for your api-ai agent", result=>console.log(result), error=>console.log(error));

For iOS i do the same thing in background with react-native-speech-to-text-ios.
Api.ai has it's own speech-to-text-engine but it's deprecated and was removed on the javascript client a few days ago - I think android will follow soon. I recommend to use the device speech-to-text functionality or any other service.

Thanks for the quick response @spoeck . We will use the requestQuery to do the conversion and send the text request itself.. Thanks again!