[3] Convert daily checks into dashboards

Is your feature request related to a problem? Please describe.
In the spirit of drinking our own Champagne and making our work more demonstrable, let's convert some of the daily checks into dashboards.
Describe the solution you'd like
Grafana dashboards that can be demonstrated by sales and are useful to ops. The ambition should be to eventually make daily scripts unnecessary. (They can still be around for troubleshooting.)
Additional context
For inspiration, see how the Backup dashboard allows us to prove to ourselves that backups are working.
Also, check how the Capacity dashboard allows us to prove that all nodes / Pods are up.
Definition of done:

Grafana dashboards to automate daily checks.
Documentation for these daily checks on compliantkubernetes.io.

I'm interested in picking this up, but I'm not sure where to start. Most of the daily checks are for things like backups and disk usage, which can already be seen in the backups, Nodes or Persistent Volumes dashboards. (We even have alerts for most of these things, but not in older versions.) What's left is basically the things we don't have metrics for: S3 usage and InfluxDB shard sizes.
Daily tasks:

Check InfluxDB backup This can be seen in the backup dashboard (but it only checks successful completion, not size in S3).
Check InfluxDB shard usage
Check S3 quota usage
Check nodes disk usage This can be seen in the Nodes dashboard.
(Check deletion of old logs from SC in S3) This is checked indirectly since we detect if the Job fails.
Check Elasticsearch disk usage This can be seen in the Persistent Volumes dashboard.
Check Elasticsearch snapshot This can be seen in the backup dashboard (successful completion of the Job).
(Verify that old Elasticsearch snapshot has been deleted) Same as above. If the Job fails we will notice. This can be seen in Kibana and there seems to be some metrics that we could use to show it also in Grafana.

I could easily improve the current backup dashboard to show some more details for Elasticsearch snapshots. But for anything else a new exporter will be needed.
InfluxDB doesn't expose metrics for shard size so we need to actually go into the Pod and check the size of the folder. This is basically something we would need to build and maintain.
There are a few exporters for S3 (not sure how well they work with S3 "compatible services") that we could try. They seem to provide generic information like number of objects and total size. This should be enough for keeping an eye on the quota and possibly a bit more (e.g. size and date of last modified object.
So what should we focus on? To me S3 and Elasticsearch makes most sense so I'll start looking into them. Happy for your thoughts. :slightly_smiling_face:

InfluxDB doesn't expose metrics for shard size so we need to actually go into the Pod and check the size of the folder. This is basically something we would need to build and maintain.

Isn't this already exposed?
https://github.com/elastisys/influxdb-helm-charts/blob/elastisys_extensions/charts/influxdb/values.yaml#L18-L27

@lentzi90 I feel you are a better authority than me on this issue. Can you think about the following:

Should we put these into a "daily check" folder or dashboard to collect them into a single place?
Please go ahead with the lowest hanging fruit you suggested.

@pettersv please chose how to prioritize this.


Remove SinbadCogs

The project has been archived upstream. Cc @mikeshardmind

Types aren't actively maintained on it anymore, so it's definitely worth removal. Some of what were the more complex parts of that repo still exist in some form in either https://github.com/mikeshardmind/async-utils or https://github.com/mikeshardmind/discord-scheduler if you're interested in keeping any specific bits of the async logic that was well typed in primer runs, but I'm not significantly concerned that anything in these is likely to break in a way other things in the primer runs won't catch with the number of projects now included.


[storage] Downgrade GRPC logs to trace

Summary
In our current setup, we log at debug level and the storage GRPC log messages are extremely noisy. We have noticed perf issues before when we were logging too much. Fewer logs also helps debugging.

:hourglass: Testing commit 0c3ec9baee9914fac5f8cfff0c9d534b1862a0a4 with merge 67b2c898c276c2ce4a48e49d9efab376bbc048f7...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: lightmark
Pushing 67b2c898c276c2ce4a48e49d9efab376bbc048f7 to master...


Updated to latest commit of RV-DV

Commit ID "aa327f3544278654f49d2c54b01d31f2ae6a13dd"
Changes:
Only a minor syntax change in "riscv_privileged_common_seq.sv"

The directed test 'cmark' is timing out, kindly resolve this issue.

The directed test 'cmark' is timing out, kindly resolve this issue.

The directed test 'cmark' is timing out, kindly resolve this issue.

@behzadmehmood-lm is it cmark.c or cmark_iccm.c?

The directed test 'cmark' is timing out, kindly resolve this issue.

@behzadmehmood-lm is it cmark.c or cmark_iccm.c?

The directed test 'cmark' is timing out, kindly resolve this issue.
@haroonshafique-lm  checked again, it's passing. (Issue resolved).

The directed test 'cmark' is timing out, kindly resolve this issue.
@haroonshafique-lm  checked again, it's passing. (Issue resolved).


Is there any best practice to debug builtin function

I'm trying to add some new builtin functions, but found it's very time costy to debug; I'm wondering if there's some suggestions on:

After mading some changes to the builtin function/executor code, what's the fatest way to re-compile/deploy those changes?
What's the best way to log out some debug informations in my builtin functions?

Thanks,
YC

I suggest to compile (make teaclave_unit_tests) and run (make run-unit-tests) unit tests. You can see other builtin function implementations.
You can use log::debug!(). Then run services or tests with RUST_LOG=teaclave=debug env set.

I'm closing this issue. Feel free to reopen or create a new one if you have further questions. Thanks.


Model generators use tab indentation

Looking at lib/rails/generators/neo4j/model/templates/model.erb this seems to be fixed.  Reopen if it isn't!


FI master -> xplat

Interesting bits where there were conflicts + ref source change + changes to ToolsetLocalReader to match 8da64be.

@dotnet-bot test this please

@dotnet-bot test this please

Verified by manual build on my Mac laptop.


Add conversions to/from stdlib sets and maps

An alternative to #10 which keeps the underlying set and map representation opaque.

This seems much better. Could you however rename the functions to {to,of}_stdlib_{map,set}. So that it's more obvious what is happening when we read the code, String.Set.{to,of}_set is a bit confusing.

Agreed.  I updated the branch to use the *_stdlib_* names.

Thanks the patch is in as 88dcf40d47d55af3ebbfa

Thanks!


New endpoint for registering plugins

See https://github.com/saturncloud/saturn/pull/1373 to test this concept out
fixes CU-e0vwea

bad idea

Task linked: CU-e0vwea Futures cannot be unpickled on dask workers


Add Octopress 3.0 support

Octopress 3 appears to be abandoned. No activity in the project on Github in 8 years.


New primitive topology revealed a rendering problem

Changing the primitive topology from Triangles to Lines caused the image to render incorrectly. Some of the lines appear to be missing and used to work fine with Triangles.
Lines

Triangles

After spending some time reading about available topologies, I think the best course of action would be to create individual Player representations for every topology as chosen topology will affect the number of required vertices. Having this done should also contribute to #10.
The incorrect rendering you see in the first image is caused by my misunderstanding of how Line Loops work.

I addressed the lines rendering problem by implementing different representations of the Player type. Switching between primitive topologies is not as easy as I initially thought. Since the number (and order) of vertices differs between representations, I decided to separate each representation as a method of type Representation. More in commit: 47d8bb205df8d860516b6030645bfb88efae4742.
Now the images render correctly:
Points
(look closely, the points are there!)

Lines

Triangles


Improve charts descriptions

For example, this chart shows data for the last 30 days only, but it's not said anywhere.

We might also want to have some text describing the "Pull Requests Workflow" chart.

Should @src-d/product take care of providing better titles and some text boxes with descriptions where needed?

@carlosms The answer is yes. I am moving the issue into the product backlog.

@rpau okay thanks! I'll leave the needs-product-input until we don't have the new copies 👍


Add support for MongoDB Atlas CLI, with environment variable based importing and provisioning

This PR adds support for the MongoDB Atlas CLI. This is different from the core MongoDB CLI.
Core features

The authentication setup currently is "Public Key" and "Private Key". These are obtained from the Altas organization > Access Manager > API Keys. (One part that's not clear to me yet is, does the key generated here apply to all the projects within the organization? I am failing to a project-specific API generator)
The shell plugin looks for the Atlas CLI's environment variables and imports into 1Password.
If there are no environment variables in the current shell session, 1Password prompts the user to manually enter them.
Provisions the shell plugin for every "atlas" command (except help and version commands) with the environment variables.

Main issues

Atlas CLI supports config file too, but we don't look for it and import the credentials just yet because it uses a different type of authentication: access token, and refresh token.
When there is no Atlas CLI config file, but if a MongoDB CLI config file exists, the Atlas commands show a prompt asking if the user would like to import the config file from MongoDB CLI path to the Atlas CLI path. This kind of hinders the 1Password shell plugin's authentication flow.

This is my first time contributing to Shell Plugins, feedback welcome. Thanks, @arunsathiya, for getting me started with 1Password and Shell Plugins.

I think supporting the environment variables - public/private key-pair - workflow for now is a good approach. We could introduce the config file auth flow further down the line.

I've addressed your changes, please take a look into it.


docs: Update README to account for other regions

Hello, I updated the README default region to somewhere other than us-east-1, which allows us to create a secondary provider pointing to the us-east-1 where the ECR registry is stored. Covering varied use-cases

Fixes #
Description
How was this change tested?

Does this change impact docs?

[X ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The getting started guide is just a getting started guide. We expect folks to modify it to meet their needs. The more complicated this guide is, the harder it is to test, and the more likely it is to break. Changing the region is something we expect users to do when customizing the guide to meet their use case.

Closing since we are consolidating these docs as part of #3017


Truncate operator implementation

As proposed in #3201. Tested with unit tests and interactive shell.

please follow the signature and doc of numpy
https://docs.scipy.org/doc/numpy/reference/generated/numpy.trunc.html

Both fixed according to numpy. Somehow I did not immediately realize exact mirror of numpy was the purpose here.

Please also include this sentence:
The truncated value of the scalar x is the nearest integer i which is closer to zero than x is. In short, the fractional part of the signed number x is discarded.

Added. I guess we should also consider improving docs for other similar math operators. For example, compare these docs:
http://mxnet.io/api/python/symbol.html#mxnet.symbol.floor
https://docs.scipy.org/doc/numpy/reference/generated/numpy.floor.html#numpy.floor
http://mxnet.io/api/python/symbol.html#mxnet.symbol.ceil
https://docs.scipy.org/doc/numpy/reference/generated/numpy.ceil.html#numpy.ceil

Sure

Turned out floor and ceil were the only ones that needed improvement, out of rounding operators at least


Add the liveness checker to the CA instance

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
c68b53fa93ad0fab264139bbf4c7cb95dfbc2150
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.

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

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
1afa71c9bc2e7ec0791415c8e77c0edfad81449a
link
/test istio-presubmit

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.

@mangchiandjjoe: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
1afa71c9bc2e7ec0791415c8e77c0edfad81449a
link
/test istio-presubmit

prow/istio-unit-tests.sh
82cb4a028285bc03b20f9bdb909af77b408497ba
link
/test istio-unit-tests

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.

@mangchiandjjoe: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
82cb4a028285bc03b20f9bdb909af77b408497ba
link
/test istio-unit-tests

prow/istio-presubmit.sh
82cb4a028285bc03b20f9bdb909af77b408497ba
link
/test istio-presubmit

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.

@mangchiandjjoe: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
82cb4a028285bc03b20f9bdb909af77b408497ba
link
/test istio-presubmit

prow/istio-unit-tests.sh
36fd4a6a5371a1965d7114542fb57ac53c6b05d2
link
/test istio-unit-tests

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.

/retest

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
36fd4a6a5371a1965d7114542fb57ac53c6b05d2
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.

/retest

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
36fd4a6a5371a1965d7114542fb57ac53c6b05d2
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.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
09ccdcedb3abab67ab836a6e0f0e7b5663ad670d
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.

/retest

@wattli @myidpt  Could you PTAL?

/lgtm, i think you can submit it now.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
09ccdcedb3abab67ab836a6e0f0e7b5663ad670d
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.

Sorry for the late reply. I will take a look tonight. Thanks for the work!

/retest

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

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
7a3e1185b19616bcbf31781c3a23d1e5d1f42a63
link
/test e2e-smoke

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.

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

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
f512379933f51740fffd4c09c46d353ce386895a
link
/test e2e-smoke

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.

@mangchiandjjoe: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
f512379933f51740fffd4c09c46d353ce386895a
link
/test e2e-smoke

prow/istio-pilot-e2e.sh
f512379933f51740fffd4c09c46d353ce386895a
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.

@mangchiandjjoe: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
f512379933f51740fffd4c09c46d353ce386895a
link
/test e2e-smoke

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

prow/istio-unit-tests.sh
7c1c1c55757a35255e80b955990ed0706258155b
link
/test istio-unit-tests

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.

@myidpt Could you please take a look? Thanks.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
de4acd17e2083b5162bb7cecaf136b99e4603d3b
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.


SHIRO-458 - Refining merge + fixing another location

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr-jdk11/13/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/184/


Don't emit 'error' for excptns caught in 'end' cbs

Currently if the parser 'end' event is used to receive the parsed object and continue execution, there appears to be a bug with the interaction between the 'end' event callback and the parsers 'error' event. If the callback throws for whatever reason, the error event is emitted from the parser (even though parsing has finished). This appears to happen because the try/catch block surrounding @saxParser.write also ends up encompassing all the 'end' event callbacks due to the sync nature of EventEmitter.
The simplest fix seemed to be adding an 'ended' flag which checks if parsing has ended before emitting the error event.

"xml2js": "^0.4.17"      <= same problem


Generated file name is obscure

Issue
Suppose I have a directory assets with file diagram.drawio in it.
I have to convert the diagram.drawio to .png in the same directory.
I used the following configuration:
- name: Render .drawio Files
  uses: rlespinasse/drawio-export-action@v2
  with:
      format: png
      path: assets
      output: assets
      action-mode: all

But apparently there is no such file as assets/diagram.png.
Solution
I would like the file assets/diagram.png to be generated as a result of described above example.
Alternatives
Finally I found another action, that didn't work either, but at least it could be patched:
- name: Render .drawio Files
  uses: docker://ghcr.io/racklet/render-drawio-action:v1
  with:
      args: '"--sub-dirs=docs/assets" "--formats=png"'

However this action afaik is outdated and uses deprecated lib. I would love to use your action.
If I only knew how...

There are few important things you should know this to become clear to you:

Path passed with output parameter is relative to the path! That's not stated in documentation, neither it is obvious.
Considering my example, all of my files were be named: "[FILENAME]-Page-N.png", where:

[FILENAME] is the original file name (without extension).
Page is literally the word page (OR its translation to some language, in my case it was sometimes english sometimes russian).
N is the page number. Regularily, for ordinary .drawio diagrams, it will be 1.

Finally I had to use the following steps combination in order to achieve my original goal:
- name: Render .drawio Files
  uses: rlespinasse/drawio-export-action@v2
  with:
      format: png
      path: assets
      output: .
      action-mode: all

# WARNING! This step implies all of the .drawio sources are ONE PAGE diagrams only!
- name: Rename Files
    run: for f in assets/*.png; do mv "$f" "$(echo "$f" | sed 's/-.\+-[0-9]\+//g')"; done

I would also ask the developer for a few improvements, if they will be considered valueable:

Update documentation to make clear output parameter is relative to path.
Set the default output value to "." (for simplicity reasons).
Add script, renaming output files, containing only one page and removing the -Page-\d+ name part.
I could've helped you providing pull request if you find any of these proposals useful!

Thanks for the report.

The documentation needs some rewrites to make clear some behaviors built on top of the export feature for the Drawio Desktop app.
The default value for output will remain export to avoid breaking changes in this version. Will consider updating the default value in the next major version. The next documentation should make clearer how this works.
An option already exists for that: https://github.com/rlespinasse/drawio-export-action#remove-page-suffix, which can be considered to be set to true in the next major version.

Oh, great news about the option! Thanks a lot!


Expansion: $ expansion

$ expansion should not happen when the next character is not a valid character identifier

1857c5eb38c83d1664e207bf8c5b9b15019e9f56


Improve input fields

Description

[x] Add 1x placeholder to leverage input.
[x] User inputted numbers now have an x appended to them.
[x] Add 0.0 placeholders in both amount input fields.
[x] New Add button to open amount position input.

Related issue
#649
How Has This Been Tested?

Visit the markets and check that all the placeholders are displayed.
Click on the max button, it should match with your sUSD total balance.

Following @koredefashokun advice, I could append the x character to the leverage user input. Unfurtunately, it doesn't work with the NumericInput component and I don't have enough time to reverse-engineer dydx's leverage input (in the coming week).\
@platschi Does it correspond to what was expected or does it need further improvement? :)

Hey! This already looks great. Thank you! A few notes:

Can you align the "Amount - Set size order" text to the left, similar as the "Leverage" text below? (see screenshot)

Clicking "Max" currently defaults the sUSD amount to the max amount of sUSD that I have in my wallet. This is wrong. The "Max" button should default to the max possible amount (maxLeverage) to open/modify a position in a given direction. In the example screenshot, that should be my current available margin with a leverage of 1.31x.

In the Leverage input field, can you use secondarygray as color for the x, to distinguish it from the input amount?

I fixed the position of the order sizing title (1) as well as the max button behaviour (2).
Regarding the third point, I need to implement the new NumericInput component (see #719) first. Then, I'll be able to update the look and feel of the leverage input field.

Quick note on the latest updates

Leverage and amout input fields placeholders are correctly displayed.

When clicking on max, it sets the input fields to the possible amount to open/modify a position. It also updates the leverage  value to the value of maxLeverage(here 10).

Leverage inputted numbers now have an xappended to them, in the same grey color as the placeholder text. This was by far the most difficult task of the issue. In order to make it work, I created a new custom input component called CustomNumericInputwich takes a suffix parameter, the string to append to the numbers. The input field in this component is a text input and not a number one since we need to see the difference between 1, 1.and 1.0. Of course, I make sure that users can only input numbers and dots (commas are replaced with dots). Another thing, the maximum value the user can input corresponds to the value of maxLeverage(here 10).


Fix #1213: [Sim, Aimodel] Nudge regr to have err->0 around current_price

Fixes #1213

Test results
Let's see curprice_xval in [-8, -5, -2, 2, 5, 8]
Do it on a regularized linear learning (pick ridge). Note that it's similar results for the other linear learning approaches.
Do it on a non-linear function to show how it's forced. Pick quadratic.
export SHOW_PLOT=true
pytest pdr_backend/aimodel/test/test_aimodel_factory.py::test_aimodel__regr_0error_on_cur_price[RegrLinearRidge]

Result at -8

Result at -5

Result at -2

Result at +2

Result at +5

Result at +8


Converting typedef aliases to using aliases

using aliases are pretty universally preferred over typedefs. Modern
editors complain about them which is distracting. This is a pass through
all typedefs, changing the non-c uses of them to using aliases.

[approve ci fedora]


feat: injective integration

📝 Description
What this PR does :

Integrate injective currency
Since it's an ethermint chain, I needed to use @zondax/ledger-cosmos-js lib as cosmjs use not updated yet (and they use zondax lib anyway, so let's use it directly)
I removed two duplicated methods : buildUnsignedPayloadTransaction and postBuildUnsignedPayloadTransaction as they do the same as buildTransaction (now txToMessages) and postBuildTransaction (now buildTransaction) with different/partial inputs (one case uses nano input and the other one node info without fees as that's the goal of the estimation)

❓ Context

Impacted projects: LLC,LLD,LLM
Linked resource(s): https://ledgerhq.atlassian.net/browse/LIVE-8106

✅ Checklist

[x] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

Another future improvement, from the bots results:

Spec injective:
There are not enough accounts to cover all mutations. Please increase the account target to at least 7 accounts


Fix Cosmos DB Encryption Samples Issue

Fix issue#37809
@sandeep-sen , @joshfree , @kyle-patterson , @KrzysztofCwalina and @aayush3011 to notification.

API change check
API changes are not detected in this pull request.

@kushagraThapar Could you help merge this PR?

@v-xuto - done, thanks!


[WIP]  *: System wide v2

This PR is a work in progress to attach one global perf event, instead of creating one per cgroup. This has several advantages, including:

Enabling host-wide profiling: Allowing the profiling of the whole machine, without excluding a single process! As a side-effect, we will be able to profile processes that aren't containerised, too (*)
Reducing resource usage. Both in userland, NewCgroupProfilers instances linearly use memory and CPU, as well as kernel space (fewer data structures that the kernel needs to create, fewer events that have to be delivered, etc)
Making the architecture more flexible. This design will allow us to incorporate new profilers more easily

Rough plan

[x] Attach to a single perf_event_open
[x] Generate profiles
[ ] Add the right labels for Kubernetes and Docker workloads
[ ] Ensure that parca-agent's web ui works

(*): In this PR, the goal is to keep the same behaviour we have right now but with a different architecture, that is, we will only show profiles for the cgroups or containers running under kubernetes that are passed via the CLI (note that this is not implemented yet). In a future PR, we will add another flag to either profile the whole machine (--system-wide) or even individual PIDs
Test plan
Showing profiles for the cgroup where the test program runs:

Fixes #260

Just wanted to give a quick update on the current status. I am currently running Parca Agent and Parca on Minikube via make dev/up, which uses Tilt, in order to fetch the metadata we need from Kubernetes, such as the pod CgroupIDs and so on.
The first thing that looked a bit off is that all profiles were tagged with cgroup_id=1, which is not what I expected or saw while running Parca Agent in my bare-metal host.
To test whether this is an error on my side, I wanted to run a bpftrace script to fetch the cgroupid of the process running in the host
$ sudo bpftrace -e 'profile:hz:99 { @[comm, cgroup]=count(); }'

Installing and running it in Minikube proved to be a bit challenging, as it's a custom Linux build:
$ cat /etc/os-release
NAME=Buildroot
VERSION=2021.02.12-1-g14f2929-dirty
ID=buildroot
VERSION_ID=2021.02.12
PRETTY_NAME="Buildroot 2021.02.12"

Thankfully, https://github.com/kinvolk/cloud-native-bpf-workshop exists!
$ kubectl trace run --imagename "docker.io/albanc/kubectl-trace-bpftrace:e896345e3d8f80aa968422c6199ac5180d688f65" --init-imagename "docker.io/albanc/kubectl-trace-init:e896345e3d8f80aa968422c6199ac5180d688f65" parca-agent  -e "profile:hz:99 { @[comm, cgroup]=count(); }"

$ kubectl trace get kubectl-trace-e3b76215-53ea-476e-b820-f39f5cb50872
NAMESPACE       NODE            NAME                                                    STATUS  AGE
default         parca-agent     kubectl-trace-e3b76215-53ea-476e-b820-f39f5cb50872      Running 0s

$ kubectl trace attach kubectl-trace-e3b76215-53ea-476e-b820-f39f5cb50872
^C
first SIGINT received, now if your program had maps and did not free them it should print them out

@[kworker/1:4, 1]: 1
@[runc:[2:INIT], 1]: 1
@[runc:[0:PARENT], 1]: 1
@[docker-init, 1]: 1
@[bpftrace, 1]: 1
@[rcu_sched, 1]: 1
@[kworker/0:1, 1]: 1
@[kube-proxy, 1]: 2
@[runc:[1:CHILD], 1]: 2
@[dlv, 1]: 3
@[grpc-health-pro, 1]: 3
@[exe, 1]: 3
@[systemd, 1]: 3
@[5, 1]: 4
@[registry, 1]: 4
@[storage-provisi, 1]: 5
@[containerd-shim, 1]: 10
@[kube-scheduler, 1]: 11
@[coredns, 1]: 12
@[containerd, 1]: 12
@[runc, 1]: 22
@[parca-agent, 1]: 25
@[etcd, 1]: 42
@[kube-controller, 1]: 46

That confirms what I saw reported by Parca Agent. BPF's cgroup helper returns 1 for every single process. Will look into the Kernel source code to understand what can be the source of the issue.
A hypothesis is that Minikube seems to be using Cgroups v1, while my bare-metal host uses Cgroups v2, and this might be the cause for these results.
On Minikube:
$ grep cgroup /proc/filesystems
nodev   cgroup
nodev   cgroup2

We have both cgroup systems, but it seems like we might be using Cgroups v1:
$ cat /sys/fs/cgroup/cgroup.controllers
cat: /sys/fs/cgroup/cgroup.controllers: No such file or directory
$ grep ^cgroup /etc/mtab
cgroup /sys/fs/cgroup/systemd cgroup rw,nosuid,nodev,noexec,relatime,xattr,release_agent=/usr/lib/systemd/systemd-cgroups-agent,name=systemd 0 0
cgroup /sys/fs/cgroup/cpu,cpuacct cgroup rw,nosuid,nodev,noexec,relatime,cpu,cpuacct 0 0
cgroup /sys/fs/cgroup/cpuset cgroup rw,nosuid,nodev,noexec,relatime,cpuset 0 0
cgroup /sys/fs/cgroup/blkio cgroup rw,nosuid,nodev,noexec,relatime,blkio 0 0
cgroup /sys/fs/cgroup/perf_event cgroup rw,nosuid,nodev,noexec,relatime,perf_event 0 0
cgroup /sys/fs/cgroup/net_cls,net_prio cgroup rw,nosuid,nodev,noexec,relatime,net_cls,net_prio 0 0
cgroup /sys/fs/cgroup/devices cgroup rw,nosuid,nodev,noexec,relatime,devices 0 0
cgroup /sys/fs/cgroup/memory cgroup rw,nosuid,nodev,noexec,relatime,memory 0 0
cgroup /sys/fs/cgroup/freezer cgroup rw,nosuid,nodev,noexec,relatime,freezer 0 0
cgroup /sys/fs/cgroup/hugetlb cgroup rw,nosuid,nodev,noexec,relatime,hugetlb 0 0
cgroup /sys/fs/cgroup/pids cgroup rw,nosuid,nodev,noexec,relatime,pids 0 0

Will keep this PR updated once I learn more details

Superseded by https://github.com/parca-dev/parca-agent/pull/627


chore: PEP 735 dev dependencies

PEP 735 specifies the syntax for listing development dependencies in pyproject.toml in a package manager-agnostic way.
uv 0.4.27 includes support for this, so we can now cleanup our pyproject.
drive-by: Use --frozen --python for uv sync in CI, to ensure we are using the right dep versions.

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

Project coverage is 91.57%. Comparing base (242fa44) to head (662dbdb).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #598      +/-   ##
==========================================
- Coverage   91.58%   91.57%   -0.02%
==========================================
  Files          59       59
  Lines        6311     6311
==========================================
- Hits         5780     5779       -1
- Misses        531      532       +1

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

This would break local workflows relying on uv below this version, update should be easy for most people but needs to be flagged.
For nix, it will be broken until unstable does a version bump...

Nix now has uv 0.4.28, so I updated the devenv lock.
Could you check if it works?


Kioskbuttonbranch

A button that returns a user from the edit-profile page, back to the main kiosk. This is redundant currently as we rewrite the members portal but I will recycle this button into the portal.
{ https://trello.com/c/aC7Q5xq4/25-edit-my-profile-doesnt-save }
Done Checklist

[ ] Task has been completed as per the Trello card
[ ] All acceptance criteria are met
[ ] Developer has tested this locally
[ ] Developer has reviewed their own PR
[ ] Design has been reviewed by design team
[ ] Tests have been added or updated
[ ] Views are responsive for mobile devices
[ ] Feature files have been written or updated
[ ] Changelog updated

@Apochilles can you please make the changes as requested, then we can merge this PR


Check if queue can be sorted into another queue using stack in C++

💥 Proposal
(A clear and concise description of what the proposal is.)
Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Yes

/assign

Closing this due to inactivity. Thank you for your valuable time.😊


Changing points in a spreadsheet does not get marked as an update

If you export the gradebook, and change the total points from say 100 to 110, then reimport the spreadsheet, the change is not reflected as an update. Should it be? Or should we add info saying that particular change is not supported. If we change the title it becomes an entirely new item.

Hey @steveswinsburg ,
So I think we should aim for parity with GB Classic on this one. If you do change an item's total point value in the csv file and import via the "Import Grades" tab (the one-stop-shop, import-everything approach), changes to the total points get reflected, and don't create new GB items:
Before - item total points in GB Classic:

Updates to spreadsheet:

"Import Grades" confirmation screen:

After - item total points:

So I'd say we should allow for total points to be updated. Would this currently be considered a change in the item's title?

The the name and points are different. IIRC it currently only checks the name, hence this inconsistency.

This is now picking up the new total points for the item. Nice! :punch::boom::punch:
I noticed that when the total points for an item are changed, the status is "Modified".

If I update both the total points and the individual scores for an item, the status is also "Modified".
At the top of the interface we say, "Note: Selecting "Update" items will override existing scores for that item."
To simplify, I'm wondering if we'd want to use "Update" in both cases and change that text to read, "Note: Selecting "Update" items will override existing values for that item."
Otherwise, folks might be wondering what the difference between Modified and Update is.
Thoughts?

That makes sense to me.  PR submitted.

'Modified' means the total point value for the assignment has been changed. 'Update' is for the actual data values.
I think instead we should change:
importExport.status.5 = Modified
to
importExport.status.5 = Update
... so that they appear the same in the UI. The difference between the two statuses is an implementation detail but under the hood they do the same thing.

This bulk of this ticket has been merged already so I changed the status on this ticket. But I think we need to figure out what gets displayed here.

Modify so that end sees "Update" for both.
Also update text to read:   Note: Selecting "Update" items will override existing values for that item.

Confirmed!


Adding Blog Post on Dynamic Kubelet Configuration

For 5 days of K8s 1.11

/hold
/assign @zacharysarah
/assign @natekartchner

Deploy preview for kubernetes-io-master-staging ready!
Built with commit 0c13c3791b3d9981c775da52539037c7e7c8d6f9
https://deploy-preview-9457--kubernetes-io-master-staging.netlify.com

Deploy preview for kubernetes-io-master-staging ready!
Built with commit 31f817233c99a199a871d5a16d68e35fd5a3d1e1
https://deploy-preview-9457--kubernetes-io-master-staging.netlify.com

cc // @mtaufen

LGTM

/hold cancel

/lgtm
/approve


Error after typing open paren "(" in editor

Description of your problem
Spyder displays an error when I type an open paren "(" in the editor
What steps will reproduce the problem?

Activate "display balloon tips" option in the editor preferences.
Type an open paren in the editor "("

What is the expected output? What do you see instead?
Spyder activates the internal console and displays this error message:
Traceback (most recent call last):
  File "~/anaconda/lib/python3.5/site-packages/spyderlib/utils/introspection/plugin_manager.py", line 356, in _introspection_complete
    self._handle_pending()
  File "~/anaconda/lib/python3.5/site-packages/spyderlib/utils/introspection/plugin_manager.py", line 312, in _handle_pending
    self._post_message('')
  File "~/anaconda/lib/python3.5/site-packages/spyderlib/utils/introspection/plugin_manager.py", line 432, in _post_message
    statusbar = self.editor_widget.window().statusBar()
AttributeError: 'SpyderDockWidget' object has no attribute 'statusBar'

Versions and main components

Spyder Version: 2.3.8
Python Version: 3.5.1
Operating system: OS X 10.10.4

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
IPython >=1.0      :  4.1.2 (OK)
jedi >=0.8.1;<0.9.0:  0.9.0 (NOK)
matplotlib >=1.0   :  1.5.1 (OK)
pandas >=0.13.1    :  0.18.0 (OK)
pep8 >=0.6         :  1.7.0 (OK)
psutil >=0.3       :  4.1.0 (OK)
pyflakes >=0.6.0   :  1.1.0 (OK)
pygments >=1.6     :  2.1.1 (OK)
pylint >=0.25      :  None (NOK)
qtconsole >=4.0    :  4.2.0 (OK)
rope >=0.9.2       :  0.9.4-1 (OK)
sphinx >=0.6.6     :  1.3.5 (OK)
sympy >=0.7.3      :  1.0 (OK)
zmq >=2.1.11       :  15.2.0 (OK)

Could you post what you were trying to complete when this error appeared?

It actually happens after typing any open paren in the editor. It can follow any text. Doesn't matter if it's a valid completion (e.g. "print") or not (e.g. "fffff"). Can even be on a blank line by itself.
I also see the same error after typing a period ".". In this case, it only happens when the period follows an invalid completion. E.g. error typing "x." if x hasn't been defined. No error if it has. Or error on "import foo." if there's no module foo. No error on "import theano.", which exists. Whereas the open paren error depended on the 'display balloon tips' setting being true, the error when typing a period seems to depend on the 'automatic code completion' setting being true.
I also see the same error when typing a tab following other text. E.g. typing "pri" followed by a tab autocompletes to "print" and generates the error. Also happens when pressing tab after statements that have no valid autocompletion, e.g. "ffff". The tab error seems to happen independently of any editor settings. The editor autocompletes and generates an error both happen even if I turn autocompletion off.

I see this issue as well when the editor pane is separated from the main spyder application window.

Thanks @immersinn, that's good to know. @dalthviz, could you work on this one, please?

Hi everyone. I followed the steps, in the current version of Spyder (3.0.1), and no error message appears. I suggest you to update Spyder to the current version. If you are using Anaconda, you can use the command conda update spyder or if you are using pip you can use pip install -U spyder. Also is important to take into account that the version 2.3.8 isn´t in maintainance, so it won´t recive more updates.
Additionally is important to know that for the proper function of the editor separated of the main Spyder application window, you have to select the gear in the right-top corner of the editor and then select the option New window:

Please follow @dalthviz advice :-)


minor json syntax fixes to config.md

Minor revision to documentation (JSON syntax fixes).
Also, an experiment to see whether this project accepts pull requests because I'm interested in adding support for AWS IMDSv2 (metadata token requirement).

Wow, I totally missed this PR! I do accept them, but, as you noticed... i might not see them for a while. Going to merge this in.

Hey, I've heard this story somewhere before...
;)

While adding in RSA signature support to get it working on golang >1.16, I also brought in the AWS IMDS client to do IMDSv2 inside of nt.


Two Factor Auth

Hi,
everything is working fine so far.
I created a new user.

welcome email sent

activation email sent

activation is done for the created user

Logged in without Two Factor

In the User Settings
** Filled First Name and Last Name
** Set my phone number ( schema: +491234567890
** Set Country to "Germany" --> DB: "de"

Saved Profile

enabled Two Factor via Google Authenticator

Checked the DB --> field "two_factor" is filled

Logout

Tried to login with the user credentials

Expected: View to enter the two factor code
Current: Login view with "You have to login first"
Based on what I can see via Debugbar

POST login was successful
tries to run "/verification/phone/verify"
redirect to "/login"

I have added some debug statements, the result for $result inside processLogin is null.
Is it not implemented (yet - like the social auth) or did I forgot something?

got it!
in config/rinvex.fort.php I have to activate both options in providers
        'providers' => [

            'totp',
            'phone',

        ],

@noxify I'm really sorry for lack of appropriate support for this package, the thing is we were not going to release it until it's ready, but we thought maybe we could develop it in public. Anyway we've wild plans for this package, and we're eager to make it Laravel's Authentication/Authorization flagship, it's just a matter of time 😄

Hi,
no problem - currently it works as expected ( with some limitations, that's ok for me ).
I have updated the /config/rinvex.fort.php with the following
        'authy' => [

        	'enabled'   => false,

            'mode' => env('AUTHY_MODE', 'live'),

            'keys' => [
                'live'    => env('AUTHY_KEYS_LIVE', ''),
                'sandbox' => env('AUTHY_KEYS_SANDBOX', ''),
            ],

        ],

the enabled flag will be used in the settings view to enable/disable the "phone verification" stuff.
But thanks for the feedback :)


vercel db pool can not access envs in astro

Hi everyone,
I'm trying to use Vercel Postgres with Drizzle and Astro. When I try to use the Vercel database pool with Drizzle, it looks like Vercel can't access the environment file and returns undefined. This causes the following error:
VercelPostgresError: 'missing_connection_string' - You did not supply a 'connectionString,' and no 'POSTGRES_URL' environment variable was found.

The code I'm using is:
import { drizzle } from "drizzle-orm/vercel-postgres";

import { sql } from "@vercel/postgres";

export const db = drizzle(sql);

I was able to solve the issue by changing how Vercel DB can access the environment. I did this by importing the loadEnv function from vite and using it to get the environment variables. Here is the updated code:
import { loadEnv } from "vite";
const env = loadEnv(process.env.NODE_ENV, process.cwd(), "");

// src/postgres-connection-string.ts
function postgresConnectionString(type = "pool") {
  let connectionString;
  switch (type) {
    case "pool": {
      connectionString = env.POSTGRES_URL;
      break;
    }
  }
  return connectionString;
}

I'm not sure if this is a Vite issue, an Astro issue, a Vercel Postgres issue, or something else. I'm also not sure if I'm doing something wrong or if there's something wrong with my computer. I would appreciate any help you can provide.
I'm using Astro 3.0.8 and my OS is Ubuntu 22.

Hello @alireza-constantin. Vite does not expose environment variables in process.env so they must be explicitly set in your config or you must manually create the KV client. See A note for Vite users on our README. Thank you for your report, hopefully other searchers will find this issue.


Improve: subscript and superscript should now work for all languages and characters.

Description
Improve: subscript and superscript should now work for all languages and characters.
Previous implementation required the use of special open type fonts - normal true type fonts did not show sub or superscript characters. The example program used the Apple SF-UI font that can only be used for apple devices and is not permitted on Windows, Android, Linux or web. The Apple font only implemented these attributes for numbers and lower case english letters.
Related Issues
Issue 1290 discussed this issue and gave original solution for Apple devices using 'SF-Pro-Display-Regular.otf'. It was extended for use on Android which is not permitted for Apple-only fonts according to the Apple fonts web page.
Additional notes
Previous implementation required the Apple 'SF-Pro-Display-Regular.otf' font which is only licensed/permitted for use on Apple devices.
I would recommend removing this font from the sample app to prevent users inadvertently using this font on non-Apple devices.
Checklist

[ X ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ X ] I titled the PR using Conventional Commits.
[ X ] I did not modify the CHANGELOG.md nor the package version in pubspec.yaml files.
[ X ] All existing and new tests are passing.
[ X ] I have run the commands in ./scripts/before_push.sh and it all passed successfully

Breaking Change
Does your PR require developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
[ X ] No, this is not a breaking change.

@ellet0 please take a look when you get a chance

Thank you for your contributions. Can you share a minimal example to reproduce the current issue?
Regarding the apple font in the example, I will update it soon.

But I put it in in case someone is relying on special fonts that have capabilities that I am not aware of. What do you think?

I'm not aware of all the use cases for the old behavior too, I would usually make it optional and stick to the current behavior and change this in the next major release by document this somewhere or in todos
Although it's completely okay to do it this way and it is usually more preferable, some developers might prefer not to change the behavior much for the working app.
I prefer simplicity over stability and additional complexity when it's not required.
Thank you for contributing once again.

This PR should also fix issue 1846

@AtlasAutocode are you planning to make more changes for this PR or you feel it is in good shape already?

Can we revert Justify min width change?
The key doesn't match the value. Besides, we added new keys instead of changing this in case we want to revert or a new widget that uses it.

No more changes to this PR planned - ready to go.
justifyWinWidth should give tooltip 'Justify' - look in word processors (such as Word) to see what they use. Please refer to Issue 1806. If you want key and tooltip to match, then change the key to 'justify'. I think the original naming was to confer: 'Justify over the whole window width' but the current abbreviation is too confusing.

justifyWinWidth should give tooltip 'Justify' - look in word processors (such as Word) to see what they use. Please refer to Issue 1806. If you want key and tooltip to match, then change the key to 'justify'. I think the original naming was to confer: 'Justify over the whole window width' but the current abbreviation is too confusing.

I see however it's not directly related and doesn't change the key for other translations and require changing it, I would recommend creating a key and reverting this change for now.
This way it will be missing in untranslated.json so contributors can know the missing and incomplete.
The current way will give the incorrect text for the other 39 translations and won't update this file.

OK. Do you want to do it or should I?
Is this the correct syntax for quill_en.arb?

"justify": "Justify",
"@justify": {
"description": "Justify the text over the full window width"
}

And update  toggle_style_button.dart line 82

      'justify' => (
         context.loc.justify,
         Icons.format_align_justify
      ),

Perhaps the key should be alignJustify since it is part of the group with alignLeft etc? Tooltip 'Align justify' - would that be clearer?

OK. Do you want to do it or should I?

It's up to you, I can do it myself if you want.

Done and submitted.
There are no other usages of this translation key.
There seems to be a conflict in the 'ko' locale which I don't understand - can you review it?

Done and submitted. There are no other usages of this translation key. There seems to be a conflict in the 'ko' locale which I don't understand - can you review it?

I will take a look at it soon, at the moment I'm moving rich_clipboard to be part of flutter_quill_extensions
Do you know what this function does?
Future<void> _update(void Function() listener) async {
    final reader = await SystemClipboard.instance?.read();
    if (reader != null) {
      final available = reader.platformFormats;
      if (_canPaste != available.isNotEmpty) {
        _canPaste = available.isNotEmpty;
        listener();
      }
    }
  }

In the file (https://github.com/singerdmx/flutter-quill/blob/master/lib/src/widgets/toolbar/buttons/clipboard_button.dart)
I'm outdated as I haven't looked into the source code for a while and I'm unfamiliar with ClipboardMonitorIn

#1843

_update is called periodically to check the SystemClipboard to see if there is anything that could be pasted into the editor.
It is needed in case, at some future time, something is put on the clipboard from a different program. In this case, the quill editor would have no knowledge that something can now be pasted.
Essentially, it enables visual feedback for paste is now possible. Copy-paste between programs is very common on desktops.
I considered just having the paste button always active. In which case, if nothing can be pasted - give no response (ugh - user wonders why it didn't work) or error message (ugh - translations needed and user must dismiss the message). Visual feedback is simple, intuitive and needs no explanation.

The problem is, that I'm trying to abstract usages of super_clipboard and provide some default or disable some features that are not supported:

/// An abstraction to make it easy to provide different implementations
@immutable
abstract class ClipboardService {
  Future<bool> canProvideHtmlText();
  Future<String?> getHtmlText();

  Future<bool> canProvidePlainText();
  Future<String?> getPlainText();

  Future<bool> canProvideImageFile();
  Future<Uint8List> getImageFileAsBytes();

  Future<bool> canProvideGifFile();
  Future<Uint8List> getGifFileAsBytes();
}

The default implementation should only depend on Flutter without any external plugins, throwing an exception or simply ignoring the call will cause unexpected behavior, do you have any suggestions on this?

There seems to be a conflict in the 'ko' locale which I don't understand - can you review it?

This seems to be a conflict with:

#1911

As it has been merged and your branch is outdated with the master branch, as you edited the translations and added alignJustify key, while Git was able to detect the changes in the source of the translations, it wasn't for the generated dart code, the above PR update justifyWinWidth to 좌우로 정렬 in Korean
There are multiple ways to fix this, the simplest is to choose the changes of the #1911 of justifyWinWidth above this, while for the b/src/l10n/untranslated.json it needs to be regenerated again after fetching the changes of the latest master branch from the upstream remote

Conflicts resolved

@ellet0 could you please take another look? Let's merge it if possible

Reverted

Reverted

Thanks.


Bratseth/dispatch summaries take 4

@baldersheim please review
I'll merge at an suitable time.

Merge at will


Add installation steps for rosdistro, ros2doctor dependency

Some of the ros2doctor plugins requires rosdistro, see https://github.com/ros2/ros2cli/pull/631.
I have only tried this on Windows, but I guess it works on macOS as well.

@Mergifyio backport dashing galactic foxy


Selectability doesn't affect appearance when you click a tab

Even tabs for unselectable items are highlighted by default, thus not having any effect at all.

This is because TabButtonCell.state is changed to NSOnState through the default tracking method. Maybe isSelected should then be based on something else.


'main': Don't dequote the word in command position before analyzing it.

Fixes #656.
Fixes #660.
Regression from commit e1ecf950e23471b235e318ba7b85e77395a6b146, "main: Do path expanstion after alias expansion" (sic).

LGTM

Thanks for the review.  Merging now to maximize testing time before 0.7.0.  (If anyone has further comments please raise them and I'll address them post merge.)

Turns out that you wrote the very same patch back in April:
https://github.com/zsh-users/zsh-syntax-highlighting/issues/572#issuecomment-485036695
And the bug in it still exists in current master.


Add ManifestStaticFilesStorage option to remove original (non-hashed) files after processing

I have added an option for the removal of original files after processing which happens after post_process. method either renames the file  to include a hash of their content for cache busting or adjusts the files which contains references to other files.
I have added some checks before the removal of the original file like checking the date of creation of the file and the last modification of the file. So that the non-modified or the non-hashed file could be passed for the removal.

Superseded by #17328.


[docs] add supertree in README

Hi LightGBM Team!
I've added in readme a new package for decision tree visualization - supertree. It works with LGBMClassifier and LGBMRegressor. The package can be used in notebook environment (Jupyter, Colab) or in Python script. I hope it will be useful for LightGBM users. Thank you!

It works with LGBMClassifier and LGBMRegressor.

Looks like this package can visualize Booster as well. 👍

https://github.com/mljar/supertree/blob/7a0b90e0414705a8570dc05eaf83861ead5e6729/supertree/supertree.py#L487-L492


Use Sentinel with authentication

closes #594

Please explain what issues is being resolved and how. Will this work with Sentinel without authentication?

Need to set authentification for Sentinels with different paaswords for redis itself and sentinels.
yes, it is working, if you use an empty password for sentinels and if a password for redis itself is set.
I only need to remove one line in file src/Connection/Aggregate/SentinelReplication.php
$parameters['password'] = null;
you have to configure an extra password for the sentinel as follow. The password for redis itself is set in options as before
   'sentinels' => [
    [
            'scheme' => 'tcp',
            'host' => '<IP_ADDRESS>',
            'port' => 26379,
            'password' => 'secret-sentinel-pw',
     ],
     ....
   ]
   'options' => [
        'replication' => 'sentinel',
        'service' => 'mymaster',
        'parameters' => [
            'password' => 'secret-redis-pw',
            'database' => 0,
      ],
   ]

I removed the line now.

This seems like a risky merge. Please add test cases to SentinelReplicationTest for both password and no-password configs.
See https://github.com/predis/predis/commit/16957f3b39679b42b08ed2100debf4af89170fa4

Feel free to re-submit to the 2.0 main branch.


Make a Sketch version

Times have changed. I'm no longer using Photoshop for this stuff. I should really make a Sketch version of this project.

What is this?
Sent from my iPhone

On Jun 2, 2016, at 8:43 AM, Joel Glovier<EMAIL_ADDRESS>wrote:
Times have changed. I'm no longer using Photoshop for this stuff. I should really make a Sketch version of this project.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Add optimized 4-bit fully connected operator + unit tests

Adds a version of the fully-connected operator which unpacks 8-bit into 4-bit weights during runtime to save memory usage.
Change-Id: I883240225f21f9bb1af4fdbfd0242f7a226e9934

Could you fix the warning about parameter name in arm_nn_vec_mat_mult_t_s4(c), packed_rhs vs rhs_packed?


[Question]Where to view func obfs4proxy.start

Hi, Where to view the obfs4proxy.start function (in the func startobfs4proxy)? I checked the yawning/obfs4 project source but did not find this function. Thank you :)

Admitted, it's not obvious, if one doesn't want to try hunting it down by actually reading about how to build this project oneself in the README.md.
But if so, that would lead one to build.sh and that would lead to obfs4.patch and that would answer this question. 😗


Join multicast groups for mDNS.

Problem
MinMDNS was not receiving mDNS multicast queries (unicast would work and so did tests on linux because avahi was auto-joining multicast groups, but esp32 was not working).
Summary of Changes
Join IPv4 and IPv6 multicast groups. Tested on ESP32 and mdns discovery works. IPv6 on esp32 returns 'Not supported'.

@woody-apple @bzbarsky-apple ?


Throwing an exception in case the sum of weights is over uint32_t max value

Verifying that the sum of weights in the configuration file isn't over uint32_t maximal value.
If the sum is overflowing, an exception is thrown.
Risk Level: Low
Testing: None
Fixes fuzz test:<PHONE_NUMBER>513472
Signed-off-by: Adi Suissa-Peleg<EMAIL_ADDRESS>

/cc @asraa @snowp
If you think of a better way to verify overflow, or if there's a way to validate it using proto validators, please let me know.
Also, couldn't find the unit-tests of the weights configuration, so if there is a file that you are aware of, I'll add new tests to it.

Added tests for the new exceptions

Also: @envoyproxy/api-shepherds is this something we'd want to be documenting on the protos? Or are we okay with this just failing validation if it happens?

Change the error messages according to:
https://github.com/envoyproxy/envoy/blob/b5a3405b2725e979a93e7410fce7c57a0c66d8db/api/envoy/api/v2/endpoint/endpoint_components.proto#L93-L101
and
https://github.com/envoyproxy/envoy/blob/b5a3405b2725e979a93e7410fce7c57a0c66d8db/api/envoy/api/v2/endpoint/endpoint_components.proto#L116-L126

@snowp it probably makes sense to document. We can't use PGV though, you'd need something like CEL to express cross-field constraints which we don't have today.
@adisuissa can you ad a comment to https://github.com/envoyproxy/envoy/blob/master/api/envoy/api/v2/endpoint/endpoint_components.proto#L93 and https://github.com/envoyproxy/envoy/blob/master/api/envoy/api/v2/endpoint/endpoint_components.proto#L121 to make this explicit in the API docs? Thanks.

@snowp it probably makes sense to document. We can't use PGV though, you'd need something like CEL to express cross-field constraints which we don't have today.
@adisuissa can you ad a comment to https://github.com/envoyproxy/envoy/blob/master/api/envoy/api/v2/endpoint/endpoint_components.proto#L93 and https://github.com/envoyproxy/envoy/blob/master/api/envoy/api/v2/endpoint/endpoint_components.proto#L121 to make this explicit in the API docs? Thanks.

Sure.
I'm assuming it also needs to be added to v3 (probably will be handled by proto_format), but if I'm wrong please correct me.

/lgtm api

/retest

Please merge master so the v4alpha changes propagate.

merged latest master.


batchInsert incorrectly quotes SQL query with values, containing [[xxx]] symbols

What steps will reproduce the problem?
$this->db->createCommand()->batchInsert('table', ['column1'], [["input [[123]]"]])->execute();
What is the expected result?
Value "input [[123]]" inserted into a table
What do you get instead?
"input `123`"
Raw SQL
INSERT INTO `table` (`column1`) VALUES ('input `123`')
Additional info

Q
A

Yii version
2.0.12

PHP version
7.0

Operating system
Windows 10 64bit

duplicate of #11242


Update azure-portal-safelist-urls.md

The portal's SSL certificate and certificates for the backend Azure services use DigiCert in the US Gov (Fairfax) cloud. Without this, the CRL check will fail, which impacts the portal as well as services like Azure AD Connect, MSAL libraries, and various PowerShell modules in locked down environments (e.g. DoD).

@bspender : Thanks for your contribution! The author(s) have been notified to review your proposed change.

After posting this, I found there may be other variants of the digicert CRL URLs as noted here:
https://docs.microsoft.com/en-us/azure/active-directory/hybrid/tshoot-connect-connectivity
*.crl3.digicert.com | HTTP/80 | Used to verify certificates.
*.crl4.digicert.com | HTTP/80 | Used to verify certificates.
*.ocsp.digicert.com | HTTP/80 | Used to verify certificates.

This could potentially mean allow a ton of other CA authority sites for different services. I believe the certificates used by Microsoft services are provided by standard CA and should not require special configuration to allow access to them unless customers have blocked access to these CAs. We have not heard any such complaints in the last 5 years for the Azure Government portal. As such this should not be a general guidance to add all these CA URLs to the default allow list.  If required, you can add a note that Azure portal uses the standard CAs and add the standard CA list to bypass list if blocked explicitly and provide a link to that list.

Thanks @santhoshsomayajula, @bspender - can you take a look at the reply to this suggested change?
CC: @JnHs

Feel free to hit me up internally and I can show you specific references to the guidance - but we now use digicert specifically as the issuer for Azure Services in the US Gov (Fairfax) cloud, which is where I recently had a colleague encounter this (and caused a deployment issue while they had to troubleshoot).

#assign @JnHs

Per discussion, closing this PR for now and will confirm/update in a separate PR.
#please-close


Double toolbar in search

using today's insiders
I have double titles and toolbars in search and SCM views (where I only have a single view)

To repro:

Start with activity bar position on top, focus on search view
Reload window
Move activity bar to side


DualCurrencyField: Use existing Input component instead of native input

Is your feature request related to a problem?
The component is using input:
https://github.com/getAlby/lightning-browser-extension/pull/1000/files#diff-e341c06a28185f14a891bdb91afc6aa3ecaffb6f2f78dea6afb5e4b7d891d2c7R35
Describe the solution you'd like
Should be done like here:
https://github.com/getAlby/lightning-browser-extension/blob/feat/597_currency-support/src/app/components/form/TextField.tsx#L36
Describe alternatives you've considered
none
Additional context
Related to: #1000
Are you working on this?
No

Is this still blocked?

No, go ahead :)

I was looking into this and wanted to check what the benefit of this change was? I was looking at the code and the Input component will return an input element the same as the input element that is currently in the DualCurrencyField component. We would still have to pass all the same props down and it would add an extra step of importing the component. Unless I am missing something?

Reason for opening this was a feeling of generalization.
Input is used for TextField so I thought it would be a benefit using it here as well. Looking closer now I noticed that we moved all functionality from Input into this one because we need to combine it with the fiat info. Which is displayed "inside of the input".
In this case we're good here I believe. Thanks for you feedback!
I guess we could move the hint from the TextField into the Input but that's another task.

Awesome thanks for taking a second look and letting me know what the plan is!


[proxy] Fully close TLS sockets, as they have no CloseWrite

Fixes #1182
Testing this behaviour involves either: allocating a pty for "docker run" and checking a timeout, or using a different client, so I'm not sure it's worth the time investment.

@2opremio could you review this, please?

LGTM, although I would split it in two commits. One for the "sync" refactoring and another one for the actual CloseWrite change.

This hasn't been merged from 1.0 into master yet. And there are non-trivial conflicts. Please get the merge done asap because the longer we wait the harder it gets.

This hasn't been merged from 1.0 into master yet.

done


patch: added two more patch types

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Clarify documentation for AnimationPlayer.queue

Existing queue documentation indicates the new animation is queued after the current animation, which reads ambiguously as either replacing any previously queued animation or inserting the new animation at the head of the queue, neither of which is true. This PR updates the documentation to be explicit about the fact the animation is added to the end of the queue.

Thanks! And congrats for your first merged Godot contribution :tada:


append() & prepend() should support cheerio objects as parameters

The following should work:
var $script = $('<script>').attr('src', 'jquery.js');
$('head').append($script)

Right now we have to:
var $script = $('<script>').attr('src', 'jquery.js');
$('head').append($script.html())

This is fixed in Release 0.4.0


Update step-1-prepare-corp-domain.md

Update Code Blocks Language Types

@jvheaton : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@jvheaton The update looks good; thanks. Per the contributor guide, the code fencing should be all lowercase, so I made that edit.
My update triggered a Contributor License Agreement request for you as a newer contributor. Could you sign the CLA? Then, we'll merge the pull request. Thanks.
-PR review team

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks @jvheaton!


[stable/airflow] version 7.6.0

This PR adds a bunch of features, see UPGRADE.md for more information.
The most significant change is a refactor of the templates/ folder, (I have grouped together the YAML from each component).
Checklist

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

@gsemet can you please review? (Sorry its a big one)

Thanks a lot for this pull request, nothing to add it looks great :)
/lgtm

We're hitting an error now in our CI but I think this is an issue with Helm itself...?
coalesce.go:199: warning: destination for connections is a table. Ignoring non-table value []
coalesce.go:199: warning: destination for connections is a table. Ignoring non-table value []
Error: UPGRADE FAILED: rendered manifests contain a new resource that already exists. Unable to continue with update: existing resource conflict: kind: Role, namespace: staging, name: airflow-chart


Soletta Machine Learning: Add code samples to document

Task Description
Page: https://github.com/solettaproject/soletta/wiki/Soletta-Machine-Learning
Add code snippets throughout the document to better illustrate what is being discussed.
Dependencies
None

Instead of adding code snippets we preferred to link the source code, so we're sure it won't be outdated and stop building or working.
All samples are linked to source code (and eventually tutorials)


Adding docker configuration to travis build.

This change includes a flag to static link libs into the compile.

Coverage remained the same at 62.136% when pulling 7d66dd3a19590249c629ac5b7b0f21ce3f3d7b29 on Docker into 0e5b827721fb6b09e846857467b78bf9d6ffb014 on master.


Turn JsonApiDocument into an enum for Success, Failure and Info

As suggested in #23, turn JsonApiDocument into an enum and see how that changes the flow of the code.

I'm implementing a slight variation of this library as a JSON:API wrapper in my own library and have done this. My "top-level document" type looks like:
pub enum Document {
    Meta {
        meta: MetaValue,
    },

    Error {
        errors: Vec<ApiError>,
        #[serde(skip_serializing_if = "Option::is_none")]
        links: Option<LinksValue>,
        #[serde(skip_serializing_if = "Option::is_none")]
        meta: Option<MetaValue>,
    },

    Data {
        data: ApiResource,
        #[serde(skip_serializing_if = "Option::is_none")]
        links: Option<LinksValue>,
        #[serde(skip_serializing_if = "Option::is_none")]
        meta: Option<MetaValue>,
        #[serde(skip_serializing_if = "Option::is_none")]
        included: Option<IncludedValue>,
    },
}

Haven't done many things with it yet, just a single simple response type, so I can't speak to the ergonomics pros and cons, but keeps you from having to do run-time validation on something that seems like the compiler can verify for us. I'd be interested in helping to implement this here, but it's definitely a breaking change.

@scotttrinh I think your concept is headed in the right direction. I have been exploring the tests in this repo and discovered that aspects of the current implementation are not compliant. Currently this crate does not respect some of the top-level document guidelines

A document MUST contain at least one of the following top-level members:
The members data and errors MUST NOT coexist in the same document.

I think the enum is the right concept, but may also lack the rules outlined by the spec. I'm going to explore this concept more - as I do agree that it would be valuable to let Rust's type system handle some of these validations

@scotttrinh thanks again for your suggestion - I went ahead and implemented this change in #75 which was just merged. We haven't cut a versioned release for this yet but we'll be pushing that out in the next few days.


[zeus] qtbase: Add vulkan to PACKAGECONFIGs

Signed-off-by: Tom Hochstein<EMAIL_ADDRESS>(cherry picked from commit 62bb2ca66bfc13f3216a207013a51c3b9a1b9f26)

Actually, this is not critical to me, I forgot we are getting this layer from master anyway. Feel free to accept it or decline it as you see fit.

merged, thanks


Reopen last open orca file

A lot of editors, when you exit and reopen them, will open the last file(s). I think this would be consistent with user expectations.

Good idea :) Will implement!

Done.

Click resume to open last active file :)

Thanks! This is great.


How to edit variable in realtime?

Hello, is there a way to edit variable in editor and see it affect the shader in real time?
Issue: When I save the shader (CTRL-S) the pinned variable dissapears

Solved :D It has to be "uniform"


lock/ops: try to stop_node before unlock

We should try to call stop_node before unlock and don't even try to do this for vms.

Does not regress with smithis: https://pulpito-ng.ceph.com/runs/zack-2024-08-07_19:22:49-teuthology:no-ceph-main-distro-default-smithi


Image2d_from_buffer_positive comparison fix

Inverted the comparisons as we expect the alignments reported from the
extension query to always be greater or equal to what is reported from
the device in pixels through
CL_DEVICE_IMAGE_PITCH_ALIGNMENT/CL_DEVICE_IMAGE_BASE_ADDRESS_ALIGNMENT.
Signed-off-by: John Kesapides<EMAIL_ADDRESS>

Qualcomm would like more time to review this.

We think this comparison is flawed and should be removed altogether. The root cause of this issue is that CL_DEVICE_IMAGE_BASE_ADDRESS_ALIGNMENT returns the alignment in pixels which is highly problematic. It should have returned the alignment in bytes which is how most implementations work. Since there is no information about the actual image format in this query implementations have to make very conservative assumptions.  These assumptions will not always match up with CL_IMAGE_REQUIREMENTS_BASE_ADDRESS_ALIGNMENT_EXT (after conversion to bytes) which is specific to the image format. It is probably too late to correct CL_DEVICE_IMAGE_BASE_ADDRESS_ALIGNMENT in the spec, but I think the solution going forward will be to add clarifications to the image_requirements_info_ext spec.
Thanks,
Balaji Calidas
Qualcomm

We think this comparison is flawed and should be removed altogether. The root cause of this issue is that CL_DEVICE_IMAGE_BASE_ADDRESS_ALIGNMENT returns the alignment in pixels which is highly problematic. It should have returned the alignment in bytes which is how most implementations work. Since there is no information about the actual image format in this query implementations have to make very conservative assumptions. These assumptions will not always match up with CL_IMAGE_REQUIREMENTS_BASE_ADDRESS_ALIGNMENT_EXT (after conversion to bytes) which is specific to the image format. It is probably too late to correct CL_DEVICE_IMAGE_BASE_ADDRESS_ALIGNMENT in the spec, but I think the solution going forward will be to add clarifications to the image_requirements_info_ext spec.
Thanks, Balaji Calidas Qualcomm
I don't disagree about the unnecessary restrictions imposed by the legacy queries being in pixels, but since we have to adhere to the restrictions the check the test is doing should hold (unless I misunderstood your point).  I recommend we merge the fix for now so the test works assuming the spec as is, and move the discussions higher into the workgroup.

@bashbaug looks like I don't have permissions to add the focused review label back on. Can we resume the review please?

why was the focused review label removed from this one?

LGTM, we should file an issue so we don't lose track of the non-power-of-two image element size issue. Thanks!
Issue raised https://github.com/KhronosGroup/OpenCL-CTS/issues/1832

@bashbaug Can we please merge this one?

Merging as discussed in the November 7th teleconference.


Log err and do not bubble when disconnecting dgb

Fixes https://github.com/microsoft/vscode-jupyter/issues/10596#issuecomment-1201937921

Codecov Report

Merging #11045 (0c93123) into main (fafa651) will decrease coverage by 0%.
The diff coverage is 100%.

@@          Coverage Diff           @@
##            main   #11045   +/-   ##
======================================
- Coverage     63%      63%   -1%
======================================
  Files        482      482
  Lines      33821    33835   +14
  Branches    5512     5513    +1
======================================
- Hits       21378    21354   -24
- Misses     10412    10435   +23
- Partials    2031     2046   +15

Impacted Files
Coverage Δ

src/notebooks/debugger/kernelDebugAdapterBase.ts
76% <100%> (-3%)
:arrow_down:

src/notebooks/debugger/debugger.ts
66% <0%> (-12%)
:arrow_down:

src/platform/common/cancellation.ts
59% <0%> (-10%)
:arrow_down:

src/kernels/execution/notebookUpdater.ts
88% <0%> (-8%)
:arrow_down:

src/notebooks/outputs/tracebackFormatter.ts
93% <0%> (-7%)
:arrow_down:

src/notebooks/debugger/debuggingManagerBase.ts
69% <0%> (-6%)
:arrow_down:

src/platform/common/utils/misc.ts
78% <0%> (-5%)
:arrow_down:

src/platform/common/process/baseDaemon.node.ts
51% <0%> (-3%)
:arrow_down:

src/kernels/jupyter/baseKernelConnectionWrapper.ts
35% <0%> (-3%)
:arrow_down:

... and 14 more


Add new exceptions and syntax from Python 3.5.

Thanks to Yuri Selivanov for the original proposal of this patch.

PR are not used in this repository. Please send an updated syntax file to Bram for distribution with Vim. Thanks.

Thanks! I'll close the PR.
I sent the diff to Bram and Cc to vim-dev.


Build failure on OS X from vecLib

On OS X 10.9.1, I'm getting a build error on SciPy from master, apparently related to missing vecLib headers in Xcode. I have the most recent Xcode as well as the command line tools, and typically build SciPy from master every 2 weeks or so. Have not seen this one before:
building 'arpack_scipy' library
compiling C sources
C compiler: gcc-4.2 -DNDEBUG -g -fwrapv -Os -Wall -Wstrict-prototypes -arch x86_64 -arch x86_64 -arch i386 -pipe

creating build/temp.macosx-10.9-intel-2.7/Users
creating build/temp.macosx-10.9-intel-2.7/Users/fonnescj
creating build/temp.macosx-10.9-intel-2.7/Users/fonnescj/Code
creating build/temp.macosx-10.9-intel-2.7/Users/fonnescj/Code/scipy
creating build/temp.macosx-10.9-intel-2.7/Users/fonnescj/Code/scipy/scipy
creating build/temp.macosx-10.9-intel-2.7/Users/fonnescj/Code/scipy/scipy/_build_utils
creating build/temp.macosx-10.9-intel-2.7/Users/fonnescj/Code/scipy/scipy/_build_utils/src
compile options: '-Iscipy/sparse/linalg/eigen/arpack/ARPACK/SRC -I/Library/Python/2.7/site-packages/numpy-1.9.0.dev_e770f92-py2.7-macosx-10.9-intel.egg/numpy/core/include -c'
gcc-4.2: /Users/fonnescj/Code/scipy/scipy/_build_utils/src/wrap_g77_abi_c.c
In file included from /System/Library/Frameworks/vecLib.framework/Headers/vecLib.h:31,
                 from /System/Library/Frameworks/Accelerate.framework/Headers/Accelerate.h:20,
                 from /Users/fonnescj/Code/scipy/scipy/_build_utils/src/wrap_g77_abi_c.c:1:
/System/Library/Frameworks/vecLib.framework/Headers/vBasicOps.h:153:23: error: immintri/System/Library/Frameworks/vecLib.framework/Headers/vBasicOps.h:153:23: error: immintri/System/Library/Frameworks/vecLib.framework/Headers/vBasicOps.h:153:23: error: immintri/System/Library/Frameworks/vecLib.framework/Headers/vBasicOps.h:153:23: error: immintri/System/Library/Frameworks/vecLib.framework/Headers/vBasicOps.h:153:23: error: immintrin.h: No such file or directory
In file included from /System/Library/Frameworks/vecLib.framework/Headers/vecLib.h:31,
                 from /System/Library/Frameworks/Accelerate.framework/Headers/Accelerate.h:20,
                 from /Users/fonnescj/Code/scipy/scipy/_build_utils/src/wrap_g77_abi_c.c:1:
/System/Library/Frameworks/vecLib.framework/Headers/vBasicOps.h:153:23: error: immintrin.h: No such file or directory
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/lipo: can't open input file: /var/folders/yc/sh2m97w17_1dsm2bzmcj1v4m0000gn/T//ccpeo1uc.out (No such file or directory)
In file included from /System/Library/Frameworks/vecLib.framework/Headers/vecLib.h:31,
                 from /System/Library/Frameworks/Accelerate.framework/Headers/Accelerate.h:20,
                 from /Users/fonnescj/Code/scipy/scipy/_build_utils/src/wrap_g77_abi_c.c:1:
/System/Library/Frameworks/vecLib.framework/Headers/vBasicOps.h:153:23: error: immintrin.h: No such file or directory
In file included from /System/Library/Frameworks/vecLib.framework/Headers/vecLib.h:31,
                 from /System/Library/Frameworks/Accelerate.framework/Headers/Accelerate.h:20,
                 from /Users/fonnescj/Code/scipy/scipy/_build_utils/src/wrap_g77_abi_c.c:1:
/System/Library/Frameworks/vecLib.framework/Headers/vBasicOps.h:153:23: error: immintrin.h: No such file or directory
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/lipo: can't open input file: /var/folders/yc/sh2m97w17_1dsm2bzmcj1v4m0000gn/T//ccpeo1uc.out (No such file or directory)
error: Command "gcc-4.2 -DNDEBUG -g -fwrapv -Os -Wall -Wstrict-prototypes -arch x86_64 -arch x86_64 -arch i386 -pipe -Iscipy/sparse/linalg/eigen/arpack/ARPACK/SRC -I/Library/Python/2.7/site-packages/numpy-1.9.0.dev_e770f92-py2.7-macosx-10.9-intel.egg/numpy/core/include -c /Users/fonnescj/Code/scipy/scipy/_build_utils/src/wrap_g77_abi_c.c -o build/temp.macosx-10.9-intel-2.7/Users/fonnescj/Code/scipy/scipy/_build_utils/src/wrap_g77_abi_c.o" failed with exit status 1

I am setting the following flags before building:
export MACOSX_DEPLOYMENT_TARGET=10.9
export CFLAGS="-arch x86_64"
export ARCHFLAGS="-arch x86_64"
export FFLAGS="-static -ff2c"
export LDFLAGS="-Wall -undefined dynamic_lookup -bundle -arch x86_64"
export PYTHONPATH="/Library/Python/2.7/site-packages/"
export CC=gcc-4.2
export CXX=g++-4.2

Fixed by gh-4396


installment-sale template monetary value

Is your feature request related to a problem? Please describe.
The current template for 'installment-sale 0.13.0' uses "total amount e[{INITIAL_DUE}]" or "remaining balance of E[{TOTAL_DUE_BEFORE_CLOSING}]".
Describe the solution you'd like
instead of E, it would be nice to have a currency (EUR, GBP) or a currency symbol.
Describe alternatives you've considered
It may be possible to manually amend the text of the template every time you use it...
Additional context

Completed in #255


[Update] Update the text in hero section

What would you like to share or ask?
Updating the text in the hero section

Hi, I would like to take this up and work on it. Thanks!

I want to work on this issue as a contributor.
Pleasure Assign so that I can show my skills under GSSoC'23.


Insert function is not working

I am not able to insert the CSV data into the existing table in click house. using clickhouse-java
I run the below code but it throws the errro:
static long insert(ClickHouseNode server, String table) throws ClickHouseException {
        try (ClickHouseClient client = ClickHouseClient.newInstance(server.getProtocol())) {
            ClickHouseRequest.Mutation request = client.connect(server).write().table(table)
                    .format(ClickHouseFormat.RowBinary);
            ClickHouseConfig config = request.getConfig();
            CompletableFuture<ClickHouseResponse> future;
            // back-pressuring is not supported, you can adjust the first two arguments
            try (ClickHousePipedOutputStream stream = ClickHouseDataStreamFactory.getInstance()
                    .createPipedOutputStream(config, (Runnable) null)) {
                // in async mode, which is default, execution happens in a worker thread
                future = request.data(stream.getInputStream()).execute();

                // writing happens in main thread
                for (int i = 0; i < 10_000; i++) {
                    BinaryStreamUtils.writeString(stream, String.valueOf(i % 16));
                    BinaryStreamUtils.writeNonNull(stream);
                    BinaryStreamUtils.writeString(stream, UUID.randomUUID().toString());
                }
            }

            // response should be always closed
            try (ClickHouseResponse response = future.get()) {
                ClickHouseResponseSummary summary = response.getSummary();
                return summary.getWrittenRows();
            }
        } catch (InterruptedException e) {
            Thread.currentThread().interrupt();
            throw ClickHouseException.forCancellation(e, server);
        } catch (ExecutionException | IOException e) {
            throw ClickHouseException.of(e, server);
        }
    }

Error
java.util.concurrent.ExecutionException: com.clickhouse.client.ClickHouseException: Code: 33. DB::Exception: Cannot read all data. Bytes read: 2. Bytes expected: 4.: (at row 14272)
: While executing BinaryRowInputFormat. (CANNOT_READ_ALL_DATA) (version <IP_ADDRESS> (official build))
, server ClickHouseNode [uri=http://localhost:8123/ShabirBhat_project]@-2130346309
	at java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:357)
	at java.util.concurrent.CompletableFuture.get(CompletableFuture.java:1908)
	at com.datoin.ch.CHclientTest.insert(CHclientTest.java:77)
	at com.datoin.ch.CHclientTest.main(CHclientTest.java:40)
Caused by: com.clickhouse.client.ClickHouseException: Code: 33. DB::Exception: Cannot read all data. Bytes read: 2. Bytes expected: 4.: (at row 14272)
: While executing BinaryRowInputFormat. (CANNOT_READ_ALL_DATA) (version <IP_ADDRESS> (official build))
, server ClickHouseNode [uri=http://localhost:8123/ShabirBhat_project]@-2130346309
	at com.clickhouse.client.ClickHouseException.of(ClickHouseException.java:168)
	at com.clickhouse.client.AbstractClient.lambda$execute$0(AbstractClient.java:275)
	at java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1604)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:750)
Caused by: java.io.IOException: Code: 33. DB::Exception: Cannot read all data. Bytes read: 2. Bytes expected: 4.: (at row 14272)
: While executing BinaryRowInputFormat. (CANNOT_READ_ALL_DATA) (version <IP_ADDRESS> (official build))

	at com.clickhouse.client.http.HttpUrlConnectionImpl.checkResponse(HttpUrlConnectionImpl.java:184)
	at com.clickhouse.client.http.HttpUrlConnectionImpl.post(HttpUrlConnectionImpl.java:227)
	at com.clickhouse.client.http.ClickHouseHttpClient.send(ClickHouseHttpClient.java:124)
	at com.clickhouse.client.AbstractClient.sendAsync(AbstractClient.java:161)
	at com.clickhouse.client.AbstractClient.lambda$execute$0(AbstractClient.java:273)
	... 4 more

Process finished with exit code 0

I tried to replace the below code with my csv data
 // writing happens in main thread
                for (int i = 0; i < 10_000; i++) {
                    BinaryStreamUtils.writeString(stream, String.valueOf(i % 16));
                    BinaryStreamUtils.writeNonNull(stream);
                    BinaryStreamUtils.writeString(stream, UUID.randomUUID().toString());
                }

Replace with :
BufferedReader reader = new BufferedReader(new FileReader(csvFilePath));
                String line;
                while ((line = reader.readLine()) != null) {
                    System.out.println(line);
                    BinaryStreamUtils.writeString(stream, line);
                }

Hi @shabir1, apologize for the confusing methods. BinaryStreamUtils.writeString only works for writing a String in RowBinary data format. It looks like this is duplicated with #1402? Let me know if you want to close this and carry on discussion there.

@shabir1 did you managed?

@mzitnik I used the below method to upload the local file on clickhouse.
static long insert(String tableName, String csvFilePath) throws Exception {
        ClickHouseNode server = getServer();
        try (BufferedInputStream bufferedInputStream = new BufferedInputStream(new FileInputStream(csvFilePath))) {
            ClickHousePassThruStream stream = ClickHousePassThruStream.of(
                    bufferedInputStream,
                    ClickHouseCompression.NONE,
                    0,
                    ClickHouseFormat.CSVWithNames
            );
            CompletableFuture<ClickHouseResponseSummary> load = ClickHouseClient.load(server, tableName, stream);
            ClickHouseResponseSummary clickHouseResponse = load.get();
            return clickHouseResponse.getWrittenRows();
        } catch (Exception e) {
            System.out.println("Failed to  load");
            e.printStackTrace();
            throw new Exception();
        }
    }

But on huge files, it failed with memory issues, Most failures occurred on the file that contained more than 300 columns.
ERROR [2023-07-27 06:20:36,451] com.datoin.library.dataset.sql.utility.commons.abstracts.sql.AbstractDBEngine: Failed to load dataset
java.util.concurrent.ExecutionException: com.clickhouse.client.ClickHouseException: Code: 241. DB::Exception: Memory limit (total) exceeded:
would use 3.36 GiB (attempt to allocate chunk of 4360384 bytes), maximum: 3.24 GiB. OvercommitTracker decision: Query was selected to stop by
 OvercommitTracker. (MEMORY_LIMIT_EXCEEDED) (version <IP_ADDRESS> (official build))
, server ClickHouseNode [uri=http://localhost:8123/cayleytest95814, options={connect_timeout=5000000,socket_timeout=5000000}]@75218225
        at java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:357)
        at java.util.concurrent.CompletableFuture.get(CompletableFuture.java:1895)

@shabir1 from here it looks like a ClickHouse server limit issue did you tried to set memory

@mzitnik How to set memory in the above method?
and if I set max_memory_usage_for_user = 10, Is it 10 Bytes/MB/GB ?

@shabir1  It is bytes

@mzitnik  How to set memory in the below method?
like max_memory_usage_for_user, max_memory_usage, etc
        ClickHouseNode server = getServer();
        try (BufferedInputStream bufferedInputStream = new BufferedInputStream(new FileInputStream(csvFilePath))) {
            ClickHousePassThruStream stream = ClickHousePassThruStream.of(
                    bufferedInputStream,
                    ClickHouseCompression.NONE,
                    0,
                    ClickHouseFormat.CSVWithNames
            );
            CompletableFuture<ClickHouseResponseSummary> load = ClickHouseClient.load(server, tableName, stream);
            ClickHouseResponseSummary clickHouseResponse = load.get();
            return clickHouseResponse.getWrittenRows();
        } catch (Exception e) {
            System.out.println("Failed to  load");
            e.printStackTrace();
            throw new Exception();
        }
    }

@mshustov @mzitnik
If the file size is 5GB but the number of columns is less (less than 10 columns).then the above method works fine.
But if the file size is just 200MB but the number of columns is huge (greater than 800 columns) then the above method failed with memory issues.


UI Issues with "Assistant tools" and 'Knowledge" options in thread view.

Electron build from d3033b74

Can the color of the highlighted tool/knowledege from "Assistant tools" and 'Thread Knowledge"  be  altered to a different color? It is Red color because of having the remove action associated with selecting the item?

Once we remove an entry from  "Assistant tools" /"Thread Knowledge" option , after the removal of tool/knowledge , the "Assistant tools" and 'Thread Knowledge" option lists continue to be presented.
The only way to close this option is to click on the "Assistant tools" /"Thread Knowledge" icon or any where else.

Should we auto close the option lists once an entry is selected for removal and removal is complete?

When there are no Thread's Tool or knowledge , Can the "No Tool"/"No Knowldge" be presented in a different font or color to make is obvious that it is not a tool name. Also when we hover over these option , they are presentable as actionable which should not be the case.

https://github.com/user-attachments/assets/b8f4d82d-f4e8-4599-b083-db8e87f6a53a

1 and 2 were explicitly designed and implemented this way.
I will address 3.

PR for 3: https://github.com/gptscript-ai/desktop/pull/292

Tested with latest build from 74f3564
Issue relating to #3 in bug is addressed now.


Cannot read property 'hasOwnProperty' of null

Issue Id: 79564b68-4f32-a44b-5039-a9757a63bddcVersions - 1.6.0-insider (9/23/2016 6:06:48 AM) and 1.6.0-insider (9/26/2016 6:06:37 AM)
Stack TypeError: Cannot read property 'hasOwnProperty' of null/vs/editor/common/model/textModelWithDecorations.ts#L496:25 (hasOwnProperty)/vs/editor/common/model/textModelWithDecorations.ts#L545:8 (_removeDecorationsImpl)/vs/editor/common/model/textModelWithDecorations.ts#L138:17 (_deltaDecorationsImpl)/vs/editor/common/model/textModelWithDecorations.ts#L161:25 (deltaDecorations)/vs/editor/common/model/textModelWithDecorations.ts#L143:13 (callback)/vs/editor/common/model/textModelWithDecorations.ts#L298:17 (callback)/vs/base/common/eventEmitter.ts#L295:9 (func)/vs/base/common/eventEmitter.ts#L225:20 (safeInvokeNoArg)/vs/editor/common/model/textModelWithDecorations.ts#L291:14 (deferredEmit)/vs/editor/common/model/textModelWithDecorations.ts#L123:14 (_withDeferredEvents)

Someone is changing decorations on a disposed model...


No PID get create when we run sasjs run with Git stored code

bash-3.2$ sasjs run https://github.xxxx.com/raw/xxx/codes/master/test.sas
ℹ No target was specified. Falling back to default target 'viya' from your local sasjsconfig.json file.                      18:37:36
ℹ Sending temp-1627996055039.sas to SAS server for execution.                                                                18:37:36
ℹ Polling session status...                                                                                18:37:42
ℹ Current session state is 'idle'                                                                                18:37:44
ℹ Polling session status...                                                                                18:37:47
ℹ Job has been submitted for 'exec-temp-1627996055039.sas'.                                                                  18:37:50
ℹ You can monitor the job progress at 'https://xxxxxxxxxx/compute/sessions/7a47c00b-5e5c-4ac4-9b29-30c667839d55-ses0000/jobs/A36B4CF3-B874-AE4A-B2EE-C23CB7CEE8DB/state'.
ℹ Fetching logs from line no: 1 to 159 of 159.                                                                               18:37:53
✔ Job execution completed!                                                                                18:37:55
ℹ Creating log file in /Users/vt7/Desktop/demo/sasjsresults .                                                                18:37:55
✔ Log file has been created at /Users/vt7/Desktop/demo/sasjsresults/sasjs-run-20210803130755.log .

To clarify - we can add the PID but we can only do that for executions on Viya (won't be supported on SAS 9)

In viya 4 the pid won't be so interesting - closing this one due it having limited benefit (unless a sponsor can be found)


Customer signup page

Enable the customers to signup/register under a city (#282).
@ehab-hassan Helped me with the header, footer and some styles.
@bart-at-mobicage and @RubenMattan helped me with colors and some styles.
@FastGeert suggested a wizard for the form.
Related (should be merged first)

rogerthat-platform/rogerthat-backend#125 (app main service is used to list main services).

Background is repeated

Button hover colour should be a little darker instead of gray

When the city wants to approve a signup show an extra popup with all the functionallity that service will receive (select functionallity chosen by signup).

Inbox message does not have an avatar

Auto select all modules of a static service + the one the user selected
Also i don't see 'When and where'


Heap Sort Readme looks not good

Remove the extra ## heading formats in the readme under HeapSort folder.
since @AdvikEshan you worked on this, I am assigning to you

Ohk sir  I'll do it by today evening !!
On Dec 8, 2017 1:24 PM, "Prateek Chanda"<EMAIL_ADDRESS>wrote:

Remove the extra ## heading formats in the readme under HeapSort folder.
since @AdvikEshan https://github.com/advikeshan you worked on this, I
am assigning to you
[image: screenshot 2017-12-08 13 22 39]
https://user-images.githubusercontent.com/16035442/33756341-0c949bc4-dc1b-11e7-9ba2-5cea198791f3.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeIIEST/Algorithms/issues/127, or mute the thread
https://github.com/notifications/unsubscribe-auth/AWea6dEMtnLfVfxsxcQqma5MiZAxDIx6ks5s-OsqgaJpZM4Q6uez
.

May I do this sir.


Bugfix: only assign direction for attack and move

In replay games, guarding doesn't have a target, I took this to mean
that only attack and move actions have a target and used that to
differentiate the types.
In a live game, the guard action has a target of your own square, which
resulted in an undefined direction of {0, 0} and a crash.

Its weird that guarding has a target......