Cannot login via Kubernetes Ingress

Hi there, I've been struggling with setting up Authelia via the Kubernetes Ingress for the latest docker image.
Error in Browser:

The page isn’t redirecting properly
An error occurred during a connection to login.test.com

Authelia Debug Logs:
debug: date='Mon Apr 15 2019 04:30:37 GMT+0000 (UTC)' method='GET', path='/api/verify' requestId='3a6f5913-b8ed-4e2f-a274-4d69967cb1ca' sessionId='uwOYjJ6pFYKVBQVPf4ji1iQ_qjC80isQ' ip='<IP_ADDRESS>' message='Authentication session uwOYjJ6pFYKVBQVPf4ji1iQ_qjC80isQ was undefined. Resetting... If it's unexpected, make sure you are visiting the expected domain.'
debug: date='Mon Apr 15 2019 04:30:37 GMT+0000 (UTC)' method='GET', path='/api/verify' requestId='3a6f5913-b8ed-4e2f-a274-4d69967cb1ca' sessionId='uwOYjJ6pFYKVBQVPf4ji1iQ_qjC80isQ' ip='<IP_ADDRESS>' message='Header x-original-url is set to https://login.test.com/'
debug: date='Mon Apr 15 2019 04:30:37 GMT+0000 (UTC)' method='GET', path='/api/verify' requestId='3a6f5913-b8ed-4e2f-a274-4d69967cb1ca' sessionId='uwOYjJ6pFYKVBQVPf4ji1iQ_qjC80isQ' ip='<IP_ADDRESS>' message='domain=login.test.com, path=/, user=unknown, groups=unknown, ip=<IP_ADDRESS>'
debug: date='Mon Apr 15 2019 04:30:37 GMT+0000 (UTC)' method='GET', path='/api/verify' requestId='3a6f5913-b8ed-4e2f-a274-4d69967cb1ca' sessionId='uwOYjJ6pFYKVBQVPf4ji1iQ_qjC80isQ' ip='<IP_ADDRESS>' message='Header x-original-url is set to https://login.test.com/'
error: date='Mon Apr 15 2019 04:30:37 GMT+0000 (UTC)' method='GET', path='/api/verify' requestId='3a6f5913-b8ed-4e2f-a274-4d69967cb1ca' sessionId='uwOYjJ6pFYKVBQVPf4ji1iQ_qjC80isQ' ip='<IP_ADDRESS>' message='Reply with error 401: User 'unknown' is not sufficiently authorized to access login.test.com/.'
debug: date='Mon Apr 15 2019 04:30:37 GMT+0000 (UTC)' method='GET', path='/api/verify' requestId='3a6f5913-b8ed-4e2f-a274-4d69967cb1ca' sessionId='uwOYjJ6pFYKVBQVPf4ji1iQ_qjC80isQ' ip='<IP_ADDRESS>' message='NotAuthenticatedError: User 'unknown' is not sufficiently authorized to access login.test.com/.
    at Object.default_1 [as default] (/usr/src/server/src/lib/routes/verify/CheckAuthorizations.js:31:15)
    at Object.<anonymous> (/usr/src/server/src/lib/routes/verify/GetSessionCookie.js:29:38)
    at Generator.next (<anonymous>)
    at /usr/src/server/src/lib/routes/verify/GetSessionCookie.js:7:71
    at new Promise (<anonymous>)
    at __awaiter (/usr/src/server/src/lib/routes/verify/GetSessionCookie.js:3:12)
    at Object.default_1 [as default] (/usr/src/server/src/lib/routes/verify/GetSessionCookie.js:17:12)
    at /usr/src/server/src/lib/routes/verify/Get.js:25:45
    at Generator.next (<anonymous>)
    at /usr/src/server/src/lib/routes/verify/Get.js:7:71
    at new Promise (<anonymous>)
    at __awaiter (/usr/src/server/src/lib/routes/verify/Get.js:3:12)
    at verifyWithSelectedMethod (/usr/src/server/src/lib/routes/verify/Get.js:20:12)
    at /usr/src/server/src/lib/routes/verify/Get.js:38:19
    at Generator.next (<anonymous>)
    at /usr/src/server/src/lib/routes/verify/Get.js:7:71'

Kubernetes Ingress Config:
{
  "kind": "Ingress",
  "apiVersion": "extensions/v1beta1",
  "metadata": {
    "name": "authelia-ingress",
    "namespace": "default",
    "selfLink": "/apis/extensions/v1beta1/namespaces/default/ingresses/authelia-ingress",
    "uid": "e8b70ae6-5f36-11e9-a6e6-0273f7fa591c",
    "resourceVersion": "1329640",
    "generation": 1,
    "creationTimestamp": "2019-04-15T04:28:24Z",
    "annotations": {
      "kubernetes.io/ingress.allow-http": "false",
      "kubernetes.io/ingress.class": "nginx",
      "nginx.ingress.kubernetes.io/auth-signin": "https://login.test.com/#/",
      "nginx.ingress.kubernetes.io/auth-url": "http://authelia-service.default.svc.cluster.local/api/verify",
      "nginx.ingress.kubernetes.io/force-ssl-redirect": "true"
    }
  },
  "spec": {
    "tls": [
      {
        "hosts": [
          "login.test.com",
          "app1.test.com"
        ],
        "secretName": "authelia-tls"
      }
    ],
    "rules": [
      {
        "host": "login.test.com",
        "http": {
          "paths": [
            {
              "path": "/",
              "backend": {
                "serviceName": "authelia-service",
                "servicePort": 80
              }
            }
          ]
        }
      },
      {
        "host": "app1.test.com",
        "http": {
          "paths": [
            {
              "path": "/",
              "backend": {
                "serviceName": "app1",
                "servicePort": 7070
              }
            }
          ]
        }
      }
    ]
  }

So when I head to app1.test.com the browser redirects to https://login.test.com/#/?rd=https://app1.test.com%2F and shows the message I mentioned above.
If you have any idea what I'm doing wrong I would appreciate your feedback. I saw your latest commit said [BREAKING] but I wasn't sure whether I was doing something wrong or Authelia doesn't work with k8s ingress as of the latest commit.
Regards

Hello, @darkquasar Authelia works for sure on Kube since there are tests covering it. I don't see anything weird at first sight in your config.
Can you please provide me with the Authelia Docker tag you're using and the version of nginx-ingress-controller you're using?

Hi Clement, so again as with all the issues I've raised here it was my mistake :) In the Ingress JSON that I posted here, you can see that the same ingress specifies an entry for login.test.com and app1.test.com. This was causing an infinite "redirection" loop! Because to get to login.test.com the ingress was redirecting me to login.test.com as per the "nginx.ingress.kubernetes.io/auth-signin" annotation.
I've created a Helm Chart for Authelia and thus I'm kind of in the middle of making it work following your repo's examples and other hints. This issue can be closed now, sorry for the annoyance ;)

Good to know @darkquasar. It's awesome that you are creating a Helm chart. I was currently learning how to do it. That's a very good news.
Let's close this issue for the time being and I will add a new one for creating the Helm chart.

There is actually a ticket about helm: https://github.com/clems4ever/authelia/issues/274.


Add .circleci/config.yml

Administrative Info
Monday Board ID: 851212620
Make sure your branch name conforms to: <feature/staging/hotfix/...>/[username]/[Monday Item ID]-[3-4 word description separated by dashes]. Otherwise, please rename your branch and create a new PR.
Changes
What changes did you make?
Added .circleci/config.yml to set up circle ci for repo
Testing
How did you confirm your changes worked?
npm test is run when a push is made to any branch in the repo
Confirmation of Change
Upload a screenshot, if possible. Otherwise, please provide instructions on how to see the change.
View config.yml file under .circleci directory in repo to see the file added

LGTM!


Error Accessing C#-based gRPC Server when hosted behind Ambassador

Describe the bug
Hello Everybody! I am trying to host a C#-based gRPC Server behind Ambassador. When following the tutorial everything works fine up to the point where I try to implement TLS termination in Ambassador (but communication between ambassador and the grpc service is unencrypted). I.e. when trying to connect via a channel created by
_channel = GrpcChannel.ForAddress("http://my.host.com:80");

everything works fine. As soon as I try to establish a channel via
_channel = GrpcChannel.ForAddress("https://my.host.com:443"); // omitting the port does not change anything

and do any call the following exception is thrown:
Grpc.Core.RpcException: 'Status(StatusCode="Internal", Detail="Error starting gRPC call. HttpRequestException: Requesting HTTP version 2.0 with version policy RequestVersionOrHigher while server offers only version fallback.", DebugException="System.Net.Http.HttpRequestException: Requesting HTTP version 2.0 with version policy RequestVersionOrHigher while server offers only version fallback.
   at System.Net.Http.HttpConnectionPool.GetHttp2ConnectionAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean async, Boolean doRequestAuth, CancellationToken cancellationToken)
   at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
   at Grpc.Net.Client.Internal.GrpcCall`2.RunCall(HttpRequestMessage request, Nullable`1 timeout)")'

The C# server within the pod is listening on port 5000 with http/2 without tls:
public static IHostBuilder CreateHostBuilder(string[] args) =>
            Host.CreateDefaultBuilder(args)
                .ConfigureWebHostDefaults(webBuilder =>
                {
                    webBuilder.ConfigureKestrel(options =>
                    {
                        // Setup a HTTP/2 endpoint without TLS.
                        options.ListenAnyIP(5000,
                            options =>
                            {
                                options.Protocols = HttpProtocols.Http2;

                            });
                    });
                    webBuilder.UseStartup<Startup>();
                });

Deployment yaml:
---
apiVersion: getambassador.io/v2
kind: Host
metadata:
  name: example-host
spec:
  hostname: my.host.com
  acmeProvider:
    authority: https://acme-v02.api.letsencrypt.org/directory # Optional: The CA you want to get your certificate from. Defaults to Let's Encrypt
    email<EMAIL_ADDRESS>  requestPolicy:
    insecure:
      action: Route
---
apiVersion: getambassador.io/v2
kind: TLSContext
metadata:
  name: tls
spec:
  secret: my.host.com # this secret exists in the k8s default namespace, tls works perfectly (tested for webapps)
  hosts:
  - "*"
  alpn_protocols: [ "h2, http/1.1" ] # i tried to add http/1.1 but this did not have any effect
---
apiVersion: getambassador.io/v2
kind: Mapping
metadata:
  name: grpc-dotnet
spec:
  grpc: true
  prefix: /greet.MyGreeterService/
  rewrite: /greet.MyGreeterService/
  service: grpcservice
  # allow_upgrade: # i also tried this
  # - h2
  # host: my.host.com # i also tried this
---
apiVersion: v1
kind: Service
metadata:
  labels:
    service: grpcservice
  name: grpcservice
spec:
  type: ClusterIP
  ports:
  - name: grpc-greet
    port: 80
    targetPort: grpc-api
  selector:
    service: grpcservice
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: grpcservice
spec:
  replicas: 1
  selector:
    matchLabels:
      service: grpcservice
  template:
    metadata:
      labels:
        service: grpcservice
    spec:
      containers:
      - name: grpcservice
        image: my.container.registry/grpcservice:latest
        ports:
        - name: grpc-api
          containerPort: 5000
      restartPolicy: Always
      imagePullSecrets:
      - name: mysecret

I also checked that the right alpn settings are applied in envoy.
To Reproduce
Steps to reproduce the behavior:

Follow the tutorial on https://www.getambassador.io/docs/edge-stack/latest/howtos/grpc/ , but use a C#-based gRPC Server
Add a TLS context based on an existing secret according to the tutorial
Try to connect via a C# gRPC Client via a secure channel
See error

Expected behavior
The C# gRPC client should connect without any problems when establishing a secure channel. I assume it has something to do with http/2 and http/1.1. It could also be related to bugs in the C# grpc client libraries, I will open an issue there as well and link it. I really hope somebody has an idea about the root cause since I really tried everything and run out of ideas...
Versions (please complete the following information):

Abassador version 1.13.3
Kubernetes environment: Azure Kubernetes Service
Kubernetes Version 1.18.14

Additional context

Grpc.AspNetCore nuget package version 2.37.0 on gRPC server and client

BTW it currently seems like the C# client is not the problem: https://github.com/grpc/grpc-dotnet/issues/1287

You are setting up a TLS context for the mapping, but you are not referencing any certificate to be trusted by your secure gRPC TLS ALPN connection.
_channel = GrpcChannel.ForAddress("https://my.host.com:443"); // omitting the port does not change anything
Where is the public certificate to be trusted for this connection ?
For our Go and Dart client, we do reference the certificates to be trusted.

Thanks for the feedback - but the lets encrypt root ca is trusted in the environment where I execute the code (both on my machine and containerized). I finally found the reason by trial and error, I had to turn off the ambassador diagnostics page:
---
apiVersion: getambassador.io/v2
kind:  Module
metadata:
  name:  ambassador
spec:
  config:
    diagnostics:
      enabled: false # this is required to achieve non-failing grpc calls

After setting this everything worked without any problem. It seems like the diagnostics page somehow inferences with http/2 clients.

Thanks very much for sharing your solution - that's great to know.  Closing issue as resolved.


Update de.json

German Translation modified.

Thanks for your help!!!
Can you do this into development-branch instead since this conflicts with translations/corrections already done there?


Restore zwj emoji once they can be gotten rendering in desktop app on macOS

See https://github.com/mymonero/mymonero-app-js/issues/194

See isEmojiUnicodeSupported() in https://gitlab.com/gitlab-org/gitlab-ce/commit/e6fc0207cb37666cdf606c03641f2afbb5646213
https://about.gitlab.com/2018/05/30/journey-in-native-unicode-emoji/
https://gitlab.com/gitlab-org/gitlab-ce/issues/26371

https://github.com/mymonero/mymonero-app-ios/commit/e243fccd1da2c2554eed20e0aab3c7c7ffa61415

Closed. See comments on https://github.com/mymonero/mymonero-app-js/issues/194


Add packages tab to tag details

Adds packages tab to tag details

What do you think about changing the chart? Right now we're showing total packages and coloring in vulnerabilities. This is a little confusing because the total number of packages might not match the number of vulnerabilities.
This is with alpine:3.13. There are 14 packages but busybox has multiple vulnerabilities of different types.

Maybe we should keep the total packages in the title, but the pie chart should show total number of vulnerabilities?

Not sure I understand. You are saying to have the graph show the number of vulnerabilities instead of vulnerable packages? How would this be different from the vulnerabilities graph we have on the Security report tab

Let's use the alpine pie chart as an example. If you add up all the vulnerabilities, there's 15. We add those to the pie chart but then in the center we say 14 (packages). We're drawing vulnerabilities but stating packages. There's nothing showing the sum of the vulnerabilities. The number inside the chart should be the total number of vulnerabilities since that's what's being depicted.

Are you saying we are double counting packages? I see a problem where we have a single package which has a High and a Medium vulnerability we will end up counting that package twice

I think the idea of this chart is to have an answer to "How many packages do I have which have X level vulnerabilitiles"
I've update the total count to reflect that now

I think this change will address what I'm talking about. I saw the chart as trying to answer a different question: "How many vulnerabilities do I have across x packages?". I thought this since the chart is being populated by vulnerability data. When we fill in the chart we include the double counts, so we show both the high and mid vulnerabilities in the same package. But we exclude the double counts from the total shown in the chart.

Yeah, the latest commit should fix that

Perfect, that does fix it. Looks good. I'll approve in just a sec. As a general comment, it would be nice to see some more comments, especially around functions. Most of the code is pretty self-evident, but they can still be helpful.


Update MD file on new branch 2017-28-09_16-11-04

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

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


Egeria docker image build failing

See https://jenkins.odpi.org/view/egeria-docker/job/egeria-maven-docker-stage-master/lastBuild/
13:15:36 [INFO] Removing intermediate container 1ab36459a9a1
13:15:36 [INFO]  ---> fd02b20f25b6
13:15:36 [INFO] Step 25/34 : RUN groupadd -r egeria -g 1000 && useradd --no-log-init -r -g egeria -u 1000 -d /opt/egeria egeria
13:15:36 [INFO]
13:15:36 [INFO]  ---> Running in a0371786230f
13:15:37 [INFO] Removing intermediate container a0371786230f
13:15:37 [INFO]  ---> ee20ad7a5d97
13:15:37 [INFO] Step 26/34 : COPY --from=build --chown=egeria:egeria /root/egeria/open-metadata-implementation/server-chassis/server-chassis-spring/target/server-chassis-spring-*.jar /opt/egeria/server-chassis-spring.jar
13:15:37 [INFO]
13:15:40 [ERROR] When using COPY with more than one source file, the destination must be a directory and end with a /
13:15:40 [INFO] ------------------------------------------------------------------------
13:15:40 [INFO] BUILD FAILURE
13:15:40 [INFO] ------------------------------------------------------------------------
13:15:40 [INFO] Total time: 27:35 min
13:15:40 [INFO] Finished at: 2019-08-14T12:15:40Z
13:15:40 [INFO] ------------------------------------------------------------------------
13:15:40 [ERROR] Failed to execute goal com.spotify:dockerfile-maven-plugin:1.4.10:build (build-and-tag-latest) on project open-metadata-docker-egeria: Could not build image: When using COPY with more than one source file, the destination must be a directory and end with a / -> [Help 1]
13:15:40 org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.spotify:dockerfile-maven-plugin:1.4.10:build (build-and-tag-latest) on project open-metadata-docker-egeria: Could not build image

Egeria itself built ok, but may be caused by recent image split work

Debugging/fixing ...

The cause here is that the build now contains multiple files that match the pattern:
root@792e6d9604ad:~/egeria/open-metadata-implementation/server-chassis/server-chassis-spring/target# ls
antrun	 generated-sources	 maven-status					 server-chassis-spring-1.1-SNAPSHOT-sources.jar
apidocs  javadoc-bundle-options  rat.txt					 server-chassis-spring-1.1-SNAPSHOT.jar
classes  maven-archiver		 server-chassis-spring-1.1-SNAPSHOT-javadoc.jar  server-chassis-spring-1.1-SNAPSHOT.jar.original

This was changed recently (by me!) to include more of the artifacts needed to push to maven central - and to build them as part of a regular build (better verification). This includes the sources archive...
Unfortunately the docker 'COPY' command's wildcard support is more limited than full regex's. See https://golang.org/pkg/path/filepath/#example_Match
Since we may have xxxx-1.1-SNAPSHOT.jar, or xxxx-1.1.jar -- and 1.1 may be 2.1, 2.1.3 etc, getting a suitable pattern is tricky.
Given the build is in a seperate image (from) the easiest fix is to

change target to the directory - don't rename
remove javadoc, sources afterwards
rename
for both UI and chassis

An alternative would have been to either

Allow the generation of javadoc/sources to be skipped in the first build stage
remove these files in the build stage

fixed


Create Set 10 of CORE rules in rule editor

Tenth set of rules:

Check_ID
Textual statement of rule
Suggested Updated Textual Statement of Rule
Final CORE Rule ID
JIRA Issue ID

CHK0153
A standard code alias is not expected to be equal to the standard code (e.g. no equal code and decode for the same coding system is expected).
A standard code alias is not expected to be equal to the standard code (e.g. no equal code for the same coding system version is expected).
DDF00063
CORERULES-9598

CHK0166
A scheduled decision instance is not expected to refer to a timeline exit.

DDF00064
CORERULES-9599

CHK0168
A scheduled decision instance is not expected to have a sub-timeline.

DDF00065
CORERULES-9600

CHK0172
A scheduled decision instance is expected to refer to a default condition.

DDF00066
CORERULES-9601

CHK0180
A study cell must refer to at least one element.

DDF00067
CORERULES-9602


zve64* and zve32* complicated the arch string parser logic

Currently we don't have any extension name contain digit, zve64* and zv32* are the first extension named with digit, that cause the current arch string parser logic, since we assume the any digit are version before*, parser logic will become complicate without this assumption.
Could you consider to rename the extension name for exclusive digits?

LLVM and GCC parser are using similar logic to parse arch string, go to parsing version mode once got digit.

The ISA Manual (I:25.6) says:

Standard extensions can also be named using a single “Z” followed by an alphabetical name and an optional version number.

My reading of this implies that extension names like Zve32 are illegal.

Easily solved with zvesixtyfour?
On Fri, Jun 18, 2021, 11:01 Nick Knight, @.***> wrote:

The ISA Manual (I:25.6) says:
Standard extensions can also be named using a single “Z” followed by an
alphabetical name and an optional version number.
My reading of this implies that extension names like Zve32 are illegal.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-v-spec/issues/697#issuecomment-864101182,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAWIKMIJUBP4UD4ZF62DZDTTNNWDANCNFSM465PPERQ
.

@nick-knight Hmmm, yeah, ISA spec seems not forbid extension name contain with digit, but with this constraint parser would be easier to implement.
@David-Horner zvesixtyfour good to me, there is no digit in the name :P but I know it's really verbose than zv64*...

@kito-cheng It's implicitly forbidden by the current spec, because the current spec says the number will parse as a version number (i.e. Zve64 will be unambiguously interpreted as Zve, version 64).

Some offline chat with @aswaterman :
Kito: I know zve64 is pretty ambiguous with version, so how do you think about zve64x?
Andrew: I think Krste needs to change the ISA strings to avoid numbers, changing parser doesn't makes sense to me, because it violates the spec
...
Kito: but I admit zve64*/zve32* is really expressive
Andrew: It's unfortunate that it's both expressive and invalid, maybe zved and zvew (doubleword / word)

PS.  Given that we require _ to separate multi-letter extensions, I agree that something like zve64x would parse unambiguously, because of the x.  zve64x2p0 would give a version number in this case.

On 2021-06-29 1:44 a.m., Andrew Waterman wrote:

PS. Given that we require |_| to separate multi-letter extensions, I
agree that something like |zve64x| would parse unambiguously, because
of the |x|.

|zve64x2p0| would give a version number in this case.

Is the version number 2p0 in this case? asking for clarification?
there is roman numerals: XXXII and LXIV .
Greek - like roman numerals but more characters if modern form used.
There are other numeral representation.
三十 二 and 六十 四 for 32 and 64 respectively in Mandarin/Cantonese.
. סד and ל ב Hebrew for 32 and 64 respectively
The strings are assumed to be unicode, right?
I prefer the chinese in that it is: very expressive and intuitive;
reasonably concise; and acknowledges  major efforts from Asian participants.
I am open to other Asian representations, but the alternate dialects are
many and represent fewer portions of the populations there.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-v-spec/issues/697#issuecomment-870254455,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGIFYPEQZIO5Q5S7EE4OAPLTVFMSBANCNFSM465PPERQ.

Created a PR for clarify the rule.
https://github.com/riscv/riscv-isa-manual/pull/718


zkSnark

This is done with zkSNARK technology. See articles and videos here: https://github.com/howardwu/libsnark-tutorial

Fundamental compilation approach vs. Python wrapper approach for now


Make it work better on mobile.

It fits on the screen and when you touch the keys they respond faster.
Stops double tap from zooming the screen.
Check the live version of the fork here: https://nickcolley.github.io/Swearing-Xylophone/

Thanks!


feat(table): deprecate SegmentSize on table

Issue: https://github.com/ratatui-org/ratatui/issues/835
This PR adds:
for table

Added new flex method with flex field
Changed segment_size method to use flex method
Deprecated segment_size method and removed segment_size field
Updated documentation
modified tests to use flex instead of segment_size except for one test.
-test fn underconstrained_segment_size() still uses segment_size to test if it still works.

Thank you for the PR!


[3,5pt] Preserve data: Author and Game classes - Team Member #3

All data should be preserved by saving collections in .json files.

Pull request for this task


[NETBEANS-1074] Module Review j2ee.earproject

Added license to l10n.list
Added licenseinfo.xml for graphic files

Thanks, merging.


Fix JSON.parse panic

Sometimes bittrex just return unfinished JSON string as result and JSON.parse will panic the process, this patch will workaround it.

Thanks!


SimpleReplay doesn't work with database usernames with multiple dashes

We have a Redshift username like this xxxxxxxx-xxxxxx-xxxxxx-xxxx  consisting letters and dashes. Running replay.py fails with this kind of error.
python3 replay.py replay.yaml
[ERROR] 2021-04-06 09:31:24 Config file missing value for "odbc_driver". Therefore, replay will not use ODBC, and will use psql instead. Please provide a value for "odbc_driver" if playback using ODBC instead of psql is necessary.
[INFO] 2021-04-06 09:31:24 Found 403 total connections, 0 don't match filters. Replaying 403.
[INFO] 2021-04-06 09:31:24 Loading transactions from my-workload/Extraction_2021-04-06T08:54:17.054408+00:00, this might take some time.
Traceback (most recent call last):
  File "replay.py", line 1583, in <module>
    main()
  File "replay.py", line 1411, in main
    possible_connections = connection_idx_by_key[connection_key]
KeyError: 'dbname_foo_bar'

This has been addressed in the release 2.1.4 Please, provide us with any feedback you have on the latest release.


How to bind "toggle folder open/close" to

Self-Diagnosis

[x] I have searched the issues for an answer to my question.
[x] I have reviewed the NERDTree documentation. :h NERDTree
[x] I have reviewed the Wiki.
[x] I have searched the web for an answer to my question.

State Your Question
As titled, I believe I'm missing something, but I can't seem to find the function to toggle a folder open/close (reveal/hide its underlying files). I'd like to bind it to <TAB>. The default binding for that is <ENTER>.
I've tried :h NERDTree and browsed the web for an answer, but in vain..
I'm rather new to vim/neovim, coming from an Emacs background. Any help would be much appreciated!

:h NERDTree-<CR> is the section you needed to read. Try this:
let NERDTreeMapCustomOpen = '<TAB>'

Thanks for the reply @PhilRunninger ! Unfortunately, let NERDTreeMapCustomOpen = '<TAB>' has no effect. I wish I were better at vim configuration syntax -- are there any other ways I can try?

Show me what you're doing in your vimrc file.

@PhilRunninger
Sure, here you go: https://github.com/ianpan870102/my-dot-files/blob/master/.config/nvim/init.vim

This works.
" Package configurations
let g:NERDTreeMapCustomOpen = '<Tab>'
let g:NERDTreeShowHidden=1
nnoremap <silent><C-S-e> :NERDTreeToggleVCS<CR>
autocmd BufEnter * if tabpagenr('$') == 1 && winnr('$') == 1 && exists('b:NERDTree') && b:NERDTree.isTabTree() |
    \ quit | endif

The only thing I can think of, based on your last comment, is that you forgot the let keyword. If it still doesn't work, run this command while your cursor is in the NERDTree window:
:verbose map <Tab>

It should report back the following text:
n  <Tab>       *@:call nerdtree#ui_glue#invokeKeyMap("<Tab>")<CR>
        Last set from ~/.config/nvim/pack/tmp/opt/nerdtree/lib/nerdtree/key_map.vim line 58

If not, it will either say No mapping found or some other plugin will have created the mapping. Let me know what that command outputs.

Thanks @PhilRunninger ! It works now. Turns out I didn't prepend the g: before NERDTreeMapCustomOpen.

@PhilRunninger I noticed that when I bind g:NERDTreeMapCustomOpen to TAB, I can't use ENTER to perform g:NERDTreeMapCustomOpen anymore.
Is there a way to "add" TAB as an additional binding of g:NERDTreeMapCustomOpen and not overwrite ENTER? In other words, I'd like to use both TAB and ENTER interchangeably for g:NERDTreeMapCustomOpen.
There must be a way for vimscript to achieve that, I'm just not that good with it yet...
p.s. Probably not helping, but in Emacs, that'd be something like
(define-key NERDTree-active-map (kbd "<tab>") #'g:NERDTreeMapCustomOpen)

which adds "TAB" as an additional mapping for g:NERDTreeMapCustomOpen and also keeps the default ENTER as is.

@ianpan870102 Enter no longer works because you've told NERDTree that you want Tab to do that function instead. To do what you're asking, use this statement instead.
autocmd FileType nerdtree nmap <Tab> <CR>

autocmd defines a command to be run when a buffer of type nerdtree is created. The command, nmap, makes the Tab do the same thing as the Enter key. See ':help :nmapfor more information. In this situation, don't usennoremap, which will revert to the original unmapped Enter` behavior instead of the remapping done in NERDTree.

autocmd defines a command to be run when a buffer of type nerdtree is created. The command, nmap, makes the Tab do the same thing as the Enter key. See ':help :nmap for more information. In this situation, don't use nnoremap, which will revert to the original unmapped Enter` behavior instead of the remapping done in NERDTree.

Thank you so so much for the detailed explanation. It makes total sense. I really appreciate it!


Display active save path in main menu somewhere

This one should be easy!
Shiny new PathManager tweaks mean saved files can end up in several different places depending on exact OS (and the optional -homedir command line parameter). As a quick bit of user help I think we should display said path where it makes sense, like the create / load world screens.
Contributor friendly :-)
Optional bonus: Small button to open (or even change?) said directory in the default directory browsing tool for the OS, if that's easily doable (somewhat have my doubts on this part ...)

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/486403-display-active-save-path-in-main-menu-somewhere?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github).

Closing in favour of #2566.


[BUG] Release workflow uses deprecated actions

Is there an existing issue for this?

[X] I have searched the existing issues

Kubernetes Version
N/A
Shipwright Version
N/A
Current Behavior
The release workflow uses deprecated actions or commands. Notably:

create-release uses an EOL version of NodeJS
set-output command is deprecated.

Expected Behavior
Release workflow uses up to date actions.
Steps To Reproduce
N/A
Anything else?

set-output deprecation: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
NodeJS force upgrade: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Hi, I'd like to work on this.

@jaydee029 great! The workflow in question that we need to replace is in the .github directory linked here. If you find a similar workflow that appears to be well-maintained and kept up to date, please submit a pull request with your recommendation.
Otherwise you can write a script that uses the GitHub CLI (gh), which is available on all GitHub Actions runners. I believe this has commands which will let us draft releases.


Well it doesn't work on a server

Sort of fixed in https://github.com/Scubakay/litematica-enderchest-materials/commit/6322266b334f30b3342303ecbc9d55b09a0ab91d


Add ls alias for find

After creating a note the first thing I tried to do was run notes ls to show all of my existing ones, and was a little confused as to what command I was supposed to run instead.
While this means you can search with it as well, which isn't technically fully correct, it seems like a harmless alias.

Thanks for all your contributions! Seeing people get involved like this is great.
Sadly though, I'm not going to merge this one. I do agree that an ls command will be useful and is probably a common expectation, but I don't want it to just be an alias of find (as you pointed out: it's a bit weird that ls does searches) and I'm hoping to use it as part of the route to making managing trees of notes better.
I'm imagining maybe that find would return every single matching note (probably all of them as it does now), while ls would return direct matches at the given level. E.g:
> notes find
note.md
more-notes/another-note.md
more-notes/third-note.md
more-notes/yet-another-note.md

> notes ls
note.md
more-notes/

> notes ls more-notes
another-note.md
third-note.md
yet-another-note.md

That makes ls an simple effective tool for browsing your notes, and leaves find as the go-to tool for searching them. Just spitballing here, but I think that could work nicely. Does that make sense to you?
I'm going to close this for now, but feel free to open a new PR pivoting this to match the above or try out some other approaches if you'd like.


error: could not create 'fairseq.egg-info': Permission denied

Hi,
I get "could not create 'fairseq.egg-info': Permission denied" error when running "python setup.py install". Any insights on how to fix this?
ttambe@acc35:/group/vlsiarch/ttambe/models/fseq/fairseq$ sudo python setup.py install
running install
running bdist_egg
running egg_info
creating fairseq.egg-info
error: could not create 'fairseq.egg-info': Permission denied
ttambe@acc35:/group/vlsiarch/ttambe/models/fseq/fairseq$
Thanks,

This seems like a permissions issue in your source directory. Please try something like this: https://stackoverflow.com/a/43898861


Updating Sql tasks help markdown and adding README.md file for azure sql

For releases/m88 branch

Looks good


Move cruntime to separate module

This moves cruntime to a separate module, which allows this framework to be built on Linux. Discussion: https://github.com/wickwirew/Runtime/issues/18#issuecomment-434533969 and https://github.com/noahemmet/Runtime/commit/8d685eb4ff7be23385bac54c320aaa71117b38e3

It looked like the last few commits included *.xcodeproj and *.xcworkspace files, but if you'd like, I'm happy to add them to the .gitignore.
Tests pass locally on my Mac but I'm not set it up test on Linux; is that something Travis will do automatically?

Thanks for everyone's help on this!

I was getting a error: /Users/travis/build/wickwirew/Runtime/cruntime has no manifest in Travis so I added a Package.swift in the cruntime directory. The MacOS tests now pass but Linux is giving me a error: cannot find -lcruntime.

Not entirely sure. I don't have much experience with SwiftPM. My guess is the Runtime package needs the CRuntime package as a dependency

It doesn't look like you can have 2 packages in the same repo as explained here. I can create a new repo for the package if need be

Ah would you mind? I can PR cruntime against it.
PS, would you prefer this be named cruntime or CRuntime? I wasn't sure about prior art here.

Would prefer CRuntime just to keep it consistent if you wouldn't mind updating it. And created it https://github.com/wickwirew/CRuntime

Hey, tests pass! I've updated the dependency to point to wickwirew/CRuntime.git, so feel free to re-run them after https://github.com/wickwirew/CRuntime/pull/1 is merged.
A few things:

I moved Runtime to Sources/Runtime (ditto for Tests). It was helpful for debugging SPM, but I don't think it's strictly necessary for this PR; I'm happy to revert it if you'd like.
The latest SPM did some nice test autogeneration (see https://github.com/wickwirew/Runtime/pull/31/files#diff-b7f3e3662900e97b99353ab438d29031), so I left that in there.

Thanks again to everyone who helped!

I've verified this works by pushing a GraphQL app to Vapor's Cloud service here: https://graphql-star-wars.v2.vapor.cloud/graphiql

Sorry for the late response! Slipped my mind. Thanks again for tackling this!


logo png added (#15)

Fixes #15  🔐
Are you a part of LGM SOC' 21 ❓

[x] Yes ✔

Describe the changes you've made 🏆
A clear and concise description of what you have done to successfully close your assigned issue. Have you added any new files? Any changes that you think are relevant to be added. Feel free to let us know! 😊

Issue #15 (A new logo)

Type of change 🖇

[x] Design changes

Logo

Create a unique branch with your name and create another PR.


Crashes.

It crashes the moment I start it up, with or without Warframe running.

Yeah... so the problem is this depends on VoidGlitch's repository to function to fetch things like a list of prime items and ducat values, and that repository no longer exists.
I'm not sure it's worth the rewrite to that part at this point. Rarities and the # Owned are shown in-game now, so the only thing it really does extra is tries to fetch prices from Warframe.Market (though that's buggy at times anyway).

Any idea how could we fix it?

And if it is possible to fix, maybe it could be repurposed to indicate which items/relics are vaulted? That's the information I need these days.

I got the fix, but unfortunately Tesseract can't read properbly brown/gold rarity primes.
The fix to atleast make it work, is just swaping a line at VoidRewardParser.exe.config
Check for the line that has MissionDecks.txt and swap the value for this one
https://raw.githubusercontent.com/RainingTerror7/WarframeData-master/master/MissionDecks.txt

You would need to find a new place to get prime data from (names of items, ducat counts, rarity) and change the https://github.com/Xeio/VoidRewardParser/blob/master/VoidRewardParser/Logic/PrimeData.cs file to correctly parse the new source of information.

So, we'd need to parse this: https://n8k6e2y6.ssl.hwcdn.net/repos/hnfvc0o3jnfvc873njb03enrf56.html

Anybody able to update the system that reads new parts?

Probably fixed now


Document best practices for improving automated testing performance

Developers can experience slow automated testing with CockroachDB often tied to schema performance. We should add best practice documentation on the cluster settings and strategies that can help improve performance.
Details to be filled out once issues/PRs are opened

[ ] It is better to scope tests to a database as opposed to user-defined schemas due to inefficient user-defined schema validation that will be improved in the future


CI/CD test parsing fixes. WIP

We need to discuss the existence of both PASH_FLAGS and -w @dkarnikis @angelhof

@nvasilakis  After the issue with PASH_FLAGS is resolved, I think we could merge the branch.

We need to discuss the existence of both PASH_FLAGS and -w @dkarnikis @angelhof

@angelhof: I am merging now to allow CI to run on the main branch, but when you have time please take a look!

The PR looks good to me. I don't think there is any particular reason to have both PASH_FLAGS and -w in run.par.sh. I think they can be combined.


Fix issue #13: https://github.com/opensearch-project/opensearch-oci-o…

…bject-storage/issues/13
Description
This commit addresses issue #13
See issue #13 for details regarding the encountered problem.
This commit also include sl4j-simple as a dependency so that we can get logging statements from the OCI SDK to show up in OpenSearch logs
Issues Resolved
#13
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @plemouel1 , thank you for contribution, I understand the need to disable the security manager to get things working until we can get the policy file to behave the way we would like.
Can you please make the DCO checks to work?

DCO should be fixed now.

@samuel-oci Looks like now there isn't anything more for me to do. Can you have a look? Thanks

@plemouel1 I am sad that we are disabling JSM but if thats something which gets us unblocked Im ok with it.
Could you open an issue/bug to restore JSM?

Will wait for @samuel-oci's ack to merge.


Fixing evaluate function where list has int instead of str

Seqeval's classification_report function requires lists of str instead of int. This change casts all the lists into strings
https://github.com/BrikerMan/Kashgari/issues/222

Code check is still failing, please update code to remove lint warnings.

Sorry about this, I'll keep watch and make sure the checks pass


Fix segfault in case of no write access to log dir

🦟 Bug fix
Fixes a Segfault when building Ign common3 with pbuilder
Summary
If there is no write access to the log directory, the logger will call itself recusively in an infinite loop and segfault.
Note, this bug is also present in the latest version of ign common.

Note, this bug is also present in the latest version of ign common.

once merged, this will be forward ported across versions.

ping ?


cigien: Blacklist cannaful[\W_]+valley[\W_]+cbd(?:[\W_]+(?:review|oil|buy|\d++|[\da-f]{5,}+)s?)

cigien requests the blacklist of the keyword cannaful[\W_]*+valley[\W_]*+cbd(?:[\W_]*+(?:review|oil|buy|\d++|[\da-f]{5,}+)s?)*. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

cannaful[\W_]*+valley[\W_]*+cbd(?:[\W_]*+(?:review|oil|buy|\d++|[\da-f]{5,}+)s?)* has been seen in 5 true positives, 0 false positives, and 0 NAAs.

cannaful[\W_]*+valley[\W_]*+cbd(?:[\W_]*+(?:review|oil|buy|\d++|[\da-f]{5,}+)s?)* has been seen in 5 true positives, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ

Approved by tripleee in Charcoal HQ


Update parameter from "_ event" to "_event"

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

#label:"aq-pr-triaged"

@andresilva-msft Thanks for your suggestion. I approved it for publishing. In future, as an MSFT please use the private repo to create PRs. You can use the SpineEdit browser extension to take you straight there from the page. Thx
#sign-off


Improved project and item templates

See #814

Removed unnecessary content that are basically throw away
Removed unnecessary spaces
ViewModel now implements INotifyPropertyChanged (At least that's useful than having a Int32 property)

If you are making the view model template implement INotifyPropertyChanged, then I would suggest making another template which adds a basic runtime class because I currently use the view model template for that, for lack of better option.

Unfortunately, we have nobody on staff to maintain or test the project templates so they are largely frozen except for critical fixes. Everything under nuget/vsix/natvis is unfortunately very brittle and requires manual testing. Feel free to fork them and provide superior project templates but these will have to stay as they are.


New Notebook shortcut does not work on Windows

Issue Type: Bug
For the "File->New Notebook" action, the shortcut listed is "Alt+Windows+N".

However, using this shortcut on Windows brings up another popup to take a note.

This popup is from Windows or OneNote. Not a feature in ADS.
Azure Data Studio version: azuredatastudio 1.16.0 (29031462c7de8d562a348276a27b616e14ffc103, 2020-03-11T16:51:08.075Z)
OS version: Windows_NT x64 10.0.18363

We've had similar reports of this but this is machine setup-specific.
@yualan what are your thoughts here?


NCBI Gene

NCBI Gene import branch

@krishnaswamypradeep please update the expected output files to reflect the actual expected output of the data. The tests designed to check expected output are currently failing


Image overlap Telescope and others windows | Looks like transparent

Hi,
this is exactly the error i have:
https://github.com/3rd/image.nvim/issues/126
Example:

I tried to turn window_overlap_clear_enabled to true, but it doesn't seems to change anythings.
I'm using

Mac OS(M1)
Nvim 0.11.0

Here is my plugin setup:

Did I do somethings wrong?

I found that I can do require("image").clear() to clear image before opening a "windows".
I got to do it for

The file tree
Telescope
Terminal

And other that I may not think about.
Not the ideal.
But it is a workaround for the moment.

Note:
In a markdown file if I add opts.integrations.markdown.only_render_image_at_cursor = true it seems to be fine.
But if I directly open a .png file, there is where I have problems.


chore(deps): update k8s to v0.31.2

This PR contains the following updates:

Package
Type
Update
Change

k8s.io/apimachinery
require
patch
v0.31.1 -> v0.31.2

k8s.io/code-generator/cmd/deepcopy-gen

patch
v0.31.1 -> v0.31.2

Release Notes

kubernetes/apimachinery (k8s.io/apimachinery)
v0.31.2
Compare Source

kubernetes/code-generator (k8s.io/code-generator/cmd/deepcopy-gen)
v0.31.2
Compare Source

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 63.26%

Totals

Change from base Build<PHONE_NUMBER>4:
0.2%

Covered Lines:
1238

Relevant Lines:
1957

💛  - Coveralls


Fix empty string PIPELINE_TIMESTAMP issue

Recent openjdk8 build timestamp is either empty or use the build timestamp itself. It should use the timestamp generated by upstream pipeline job, which is passed in the jdk build job.
Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

empty timestamp:
https://ci.adoptopenjdk.net/job/openjdk8_build_x86-64_linux/397/
jdk build timestamp:
https://ci.adoptopenjdk.net/job/openjdk8_build_x86-64_linux/399/

@smlambert  really tricky one, no any clue from jenkins output.

I will merge your PR, to keep it the same as other scripts, but I think the empty string problem happens when someone runs the build job directly (not triggered from pipeline), and if I failed to add a line to the top of the build script in Jenkins to account for the manual build case.  The Mac build was missing this line (which I have now added:
[ -z $PIPELINE_TIMESTAMP ] && PIPELINE_TIMESTAMP="$(date +'%Y%d%m%H%M')"

Yes, adding the zero check can void the empty string when triggered manually.
The hidden issue is if you see the jobs on ppcle and on linux have different timestamps(OpenJDK8_ppc64le_Linux_201825061558.tar.gz, OpenJDK8_x64_Linux_201825061600.tar.gz). while job on mac has empty string.
These three  jobs were all  triggered by same pipeline job . That is because no  PIPELINE_TIMESTAMP passed in. Jobs are either using its own timestamp if there is the line [ -z $PIPELINE_TIMESTAMP ] && PIPELINE_TIMESTAMP="$(date +'%Y%d%m%H %M')"   added or using empty string if no timestamp generated in shell script.


fix urllib.request imcompatible issue

fix #494

Hi @chrisxiao
Closing as it has been quite a while.
Feel free to update/reopen if this still poses an issue or you would like to merge -- we would love new functionality!
Thanks


sales tax - updating phrasing based on Zach/Bryan feedback

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


(refactor): cache physical execution plan

Add plan cache for portal in extended query mode so that the execution of portal can directly use the plan rather than generating plan every time execute it.
Because different kinds of statement has different execution method. For now, I prioritize implementing for query statement first.

Can we postpone this first? This feature is quite error prone and has no much benefits. For example, when we do scale in/scale out, the distribution of mv may change, and the physical plan will change.

Yes

Can we postpone this first? This feature is quite error prone and has no much benefits. For example, when we do scale in/scale out, the distribution of mv may change, and the physical plan will change.

Agreed. Postgresql is singleton so it is not so hard to do this as client-side cache.
Also I think it will not be the must-require for e2e_test.


When the code block contains too much content, the highlight request server directly issues 431 Request Header Fields Too Large

Some of my code blocks are over 300 lines long. Highlight rendering always fails when entering the page for the first time. Only after refreshing can it render normally.

When I increase the header limit in nginx it seems to have some effect.
But my program is multilingual. It can be highlighted normally only in Chinese and English modes, but in Japanese mode some code blocks are still in 413 status and cannot be rendered. It can only be highlighted normally when refreshing.

So I'm thinking about using Post instead of Get
rehype-nuxt.ts
return await $fetch("/api/_mdc/highlight", {
        params: {
          code,
          lang,
          theme: JSON.stringify(theme),
          options: JSON.stringify(options)
        },
      });

me too


[style]: add animation on dark mode button

What's the style idea?
We need to add animation on dark mode button .
https://github.com/shyamtawli/devFind/assets/102541488/ac2ba57e-ea02-42c3-93a3-cfe916f2353c
Add screenshots
Add screenshots
Code of Conduct

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

Assigned @ahsan-abc

if possible to add hacktoberfest tag


Remove-Item failure on AppVeyor

The build and deploy still works, but I get this message near the end. I'm not familiar with Powershell so I'm not really sure what's going on.
Remove-Item : The method or operation is not implemented.
At C:\projects\eson\ci\before_deploy.ps1:22 char:1
+ Remove-Item $STAGE
+ ~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Remove-Item], NotImplementedException
    + FullyQualifiedErrorId : System.NotImplementedException,Microsoft.PowerShell.Commands.RemoveItemCommand

I encountered it after changing the layout of $STAGE's contents in a way I think caused the earlier Remove-Item *.* to match no files. I'm not clear on exactly which part failed but I changed the relevant part to just
Set-Location ..
Remove-Item $STAGE -Force -Recurse

to recursively delete $STAGE and that works.


Automatic screen rotation causes screen to go black

The last version that automatic screen rotation worked correctly was v<IP_ADDRESS>.  On versions <IP_ADDRESS> and <IP_ADDRESS> when the device is rotated from portrait to landscape (or vice versa) the screen goes black and it is not possible to scan the bar-code.

Hi, I have the same problem. On my LG G4c (Android 5.0.2) it works, on Samsung SM-T800 (Android 5.0.2) I have the same problem.

I have downgraded to v<IP_ADDRESS> and this seems to work for now.  For some reason it was broken in later releases.  Can this be fixed?


binding ocaml built failed with gcc13

Message:
Caused by:
  process didn't exit successfully: `/home/xuanwo/Code/apache/incubator-opendal/target/debug/build/ocaml-boxroot-sys-3082bb6216a3a01c/build-script-build` (exit status: 1)
  --- stdout
  cargo:rerun-if-changed=vendor/boxroot/boxroot.c
  cargo:rerun-if-changed=vendor/boxroot/boxroot.h
  cargo:rerun-if-env-changed=OCAMLOPT
  cargo:rerun-if-env-changed=OCAML_WHERE_PATH
  TARGET = Some("x86_64-unknown-linux-gnu")
  OPT_LEVEL = Some("0")
  HOST = Some("x86_64-unknown-linux-gnu")
  cargo:rerun-if-env-changed=CC_x86_64-unknown-linux-gnu
  CC_x86_64-unknown-linux-gnu = None
  cargo:rerun-if-env-changed=CC_x86_64_unknown_linux_gnu
  CC_x86_64_unknown_linux_gnu = None
  cargo:rerun-if-env-changed=HOST_CC
  HOST_CC = None
  cargo:rerun-if-env-changed=CC
  CC = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64-unknown-linux-gnu
  CFLAGS_x86_64-unknown-linux-gnu = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64_unknown_linux_gnu
  CFLAGS_x86_64_unknown_linux_gnu = None
  cargo:rerun-if-env-changed=HOST_CFLAGS
  HOST_CFLAGS = None
  cargo:rerun-if-env-changed=CFLAGS
  CFLAGS = None
  cargo:rerun-if-env-changed=CRATE_CC_NO_DEFAULTS
  CRATE_CC_NO_DEFAULTS = None
  DEBUG = Some("true")
  CARGO_CFG_TARGET_FEATURE = Some("fxsr,sse,sse2")
  running: "cc" "-O0" "-ffunction-sections" "-fdata-sections" "-fPIC" "-gdwarf-4" "-fno-omit-frame-pointer" "-m64" "-I" "/usr/lib/ocaml" "-I" "vendor/boxroot/" "-Wall" "-Wextra" "-o" "/home/xuanwo/Code/apache/incubator-opendal/target/debug/build/ocaml-boxroot-sys-0cb0f83fda934b69/out/vendor/boxroot/boxroot.o" "-c" "vendor/boxroot/boxroot.c"
  cargo:warning=vendor/boxroot/boxroot.c: In function ‘pool_remove’:
  cargo:warning=vendor/boxroot/boxroot.c:360:31: warning: unused variable ‘cl’ [-Wunused-variable]
  cargo:warning=  360 |   FOREACH_GLOBAL_RING(global, cl, {
  cargo:warning=      |                               ^~
  cargo:warning=vendor/boxroot/boxroot.c:164:13: note: in definition of macro ‘FOREACH_GLOBAL_RING’
  cargo:warning=  164 |       class cl = global_ring_classes[b__i - b__st];                     \
  cargo:warning=      |             ^~
  cargo:warning=vendor/boxroot/boxroot.c: In function ‘free_all_chunks’:
  cargo:warning=vendor/boxroot/boxroot.c:375:31: warning: unused variable ‘cl’ [-Wunused-variable]
  cargo:warning=  375 |   FOREACH_GLOBAL_RING(global, cl, { *global = NULL; });
  cargo:warning=      |                               ^~
  cargo:warning=vendor/boxroot/boxroot.c:164:13: note: in definition of macro ‘FOREACH_GLOBAL_RING’
  cargo:warning=  164 |       class cl = global_ring_classes[b__i - b__st];                     \
  cargo:warning=      |             ^~
  cargo:warning=vendor/boxroot/boxroot.c: In function ‘promotion_occupancy’:
  cargo:warning=vendor/boxroot/boxroot.c:521:17: warning: comparison of integer expressions of different signedness: ‘int’ and ‘long unsigned int’ [-Wsign-compare]
  cargo:warning=  521 |   if (threshold <= LOW_COUNT_THRESHOLD) return LOW;
  cargo:warning=      |                 ^~
  cargo:warning=vendor/boxroot/boxroot.c:522:17: warning: comparison of integer expressions of different signedness: ‘int’ and ‘long unsigned int’ [-Wsign-compare]
  cargo:warning=  522 |   if (threshold <= HIGH_COUNT_THRESHOLD) return HIGH;
  cargo:warning=      |                 ^~
  cargo:warning=vendor/boxroot/boxroot.c: In function ‘pool_reclassify’:
  cargo:warning=vendor/boxroot/boxroot.c:544:3: warning: enumeration value ‘NO_CHANGE’ not handled in switch [-Wswitch]
  cargo:warning=  544 |   switch (occ) {
  cargo:warning=      |   ^~~~~~
  cargo:warning=vendor/boxroot/boxroot.c: In function ‘scan_pool’:
  cargo:warning=vendor/boxroot/boxroot.c:821:14: warning: passing argument 1 of ‘action’ makes pointer from integer without a cast [-Wint-conversion]
  cargo:warning=  821 |       action(v, (value *)current);
  cargo:warning=      |              ^
  cargo:warning=      |              |
  cargo:warning=      |              value {aka long int}
  cargo:warning=vendor/boxroot/boxroot.c:821:14: note: expected ‘void *’ but argument is of type ‘value’ {aka ‘long int’}
  cargo:warning=vendor/boxroot/boxroot.c:821:17: warning: passing argument 2 of ‘action’ makes integer from pointer without a cast [-Wint-conversion]
  cargo:warning=  821 |       action(v, (value *)current);
  cargo:warning=      |                 ^~~~~~~~~~~~~~~~
  cargo:warning=      |                 |
  cargo:warning=      |                 value * {aka long int *}
  cargo:warning=vendor/boxroot/boxroot.c:821:17: note: expected ‘value’ {aka ‘long int’} but argument is of type ‘value *’ {aka ‘long int *’}
  cargo:warning=vendor/boxroot/boxroot.c:821:7: error: too few arguments to function ‘action’
  cargo:warning=  821 |       action(v, (value *)current);
  cargo:warning=      |       ^~~~~~
  cargo:warning=vendor/boxroot/boxroot.c: In function ‘boxroot_setup’:
  cargo:warning=vendor/boxroot/boxroot.c:1043:31: warning: unused variable ‘cl’ [-Wunused-variable]
  cargo:warning= 1043 |   FOREACH_GLOBAL_RING(global, cl, { *global = NULL; });
  cargo:warning=      |                               ^~
  cargo:warning=vendor/boxroot/boxroot.c:164:13: note: in definition of macro ‘FOREACH_GLOBAL_RING’
  cargo:warning=  164 |       class cl = global_ring_classes[b__i - b__st];                     \
  cargo:warning=      |             ^~
  cargo:warning=vendor/boxroot/boxroot.c:1045:24: warning: assignment to ‘void (*)(void (*)(void *, value,  volatile value *))’ {aka ‘void (*)(void (*)(void *, long int,  volatile long int *))’} from incompatible pointer type ‘scan_roots_hook’ {aka ‘void (*)(void (*)(void *, long int,  volatile long int *), scanning_action_flags,  void *, caml_domain_state *)’} [-Wincompatible-pointer-types]
  cargo:warning= 1045 |   prev_scan_roots_hook = caml_scan_roots_hook;
  cargo:warning=      |                        ^
  cargo:warning=vendor/boxroot/boxroot.c:1049:24: warning: assignment to ‘scan_roots_hook’ {aka ‘void (*)(void (*)(void *, long int,  volatile long int *), scanning_action_flags,  void *, caml_domain_state *)’} from incompatible pointer type ‘void (*)(void (*)(void *, value,  volatile value *))’ {aka ‘void (*)(void (*)(void *, long int,  volatile long int *))’} [-Wincompatible-pointer-types]
  cargo:warning= 1049 |   caml_scan_roots_hook = scanning_callback;
  cargo:warning=      |                        ^
  cargo:warning=vendor/boxroot/boxroot.c: In function ‘boxroot_teardown’:
  cargo:warning=vendor/boxroot/boxroot.c:1061:24: warning: assignment to ‘scan_roots_hook’ {aka ‘void (*)(void (*)(void *, long int,  volatile long int *), scanning_action_flags,  void *, caml_domain_state *)’} from incompatible pointer type ‘void (*)(void (*)(void *, value,  volatile value *))’ {aka ‘void (*)(void (*)(void *, long int,  volatile long int *))’} [-Wincompatible-pointer-types]
  cargo:warning= 1061 |   caml_scan_roots_hook = prev_scan_roots_hook;
  cargo:warning=      |                        ^
  exit status: 1

  --- stderr

  error occurred: Command "cc" "-O0" "-ffunction-sections" "-fdata-sections" "-fPIC" "-gdwarf-4" "-fno-omit-frame-pointer" "-m64" "-I" "/usr/lib/ocaml" "-I" "vendor/boxroot/" "-Wall" "-Wextra" "-o" "/home/xuanwo/Code/apache/incubator-opendal/target/debug/build/ocaml-boxroot-sys-0cb0f83fda934b69/out/vendor/boxroot/boxroot.o" "-c" "vendor/boxroot/boxroot.c" with args "cc" did not execute successfully (status code exit status: 1).

warning: build failed, waiting for other jobs to finish...
cargo cca  13.80s user 1.43s system 649% cpu 2.343 total

Gcc version:
:( cc --version
cc (GCC) 13.2.1 20230730
Copyright (C) 2023 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Also reported to https://gitlab.com/ocaml-rust/ocaml-boxroot/-/issues/15

The upstream is no response after 3 days, can we take some actions to workaround them?

This doesn't seem to be reproducible in my environment,executing cargo build has no errors.
Are there any other compile options used?
Gcc version:
cc (GCC) 13.2.1 20230801
Copyright (C) 2023 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

This doesn't seem to be reproducible in my environment,executing cargo build has no errors.
Are there any other compile options used?

Oh, stange. I'm just using cargo build.
I have linker settings like:
[target.x86_64-unknown-linux-gnu]
linker = "clang"
rustflags = ["-C", "link-arg=-fuse-ld=/usr/bin/mold"]

But I think they are not related to this issue.

It may be because of your OCaml version.
we don't support OCaml 5.0 yet, because 5.0 brings huge changes, ocaml-rs doesn't support it yet.
It is recommended to use opam according to version 4.14 opam switch create your-custom-name ocaml-base-compiler.4.14.0

Thanks a lot! It works now. Would you like to add some docs about opam in binding docs? I'm guessing most people don't know about it.

I'll add a documentation later


Revamp HTTPServer constructor.

Current dependencies on/for this PR:

main

PR #340   👈

PR #338

This stack of pull requests is managed by Graphite.


Add files via upload

Add CONTRIBUTING.MD with contribution guidelines

Cloned the repository to my local machine.
Created a new file named CONTRIBUTING.MD.
Copied and pasted the provided text into the CONTRIBUTING.MD file.
Ready for review and merging into the main repository.

Great addition!


[v5: Slider] Support @material-ui/lab

Slider was merged into @material-ui/lab, in theory can that be supported?

I will look into this. MUImui has many sub packages for different components. I am open for pr

@alexey-pelykh Thanks, I'll be happy review and add that.


React Native App Causing Phone to heat up extensively

Description
I have a react native app that worked seamlessly fine on android and ios a few days ago (no lags or slowdowns)
Until recently, I started noticing that mostly ios devices complain that their phones heat up immediately they enter into my app
Has anyone experienced such an issue before?, and If so what to do to fix it, I am memorizing most components and am caching some images that will render in my app multiple times as optimization efforts but the stats remain the same
I wish I had more info on this but this issue only surfaced when I had gone far into building the app
Below are some of the stats for the app from Xcode
Memory Usage from xcode

Cpu usage from xcode

Cpu usage from profiler

Suspected Select Culprit

More info about suspected selected culprit

Linked StackOverflow Issue
https://stackoverflow.com/questions/72087926/react-native-app-causing-phone-to-heat-up-extensively?noredirect=1#comment127374016_72087926
Version
0.68.1
Output of npx react-native info
System:
OS: macOS 12.1
CPU: (8) arm64 Apple M1
Memory: 101.39 MB / 8.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 17.8.0 - /opt/homebrew/bin/node
Yarn: 1.22.18 - /opt/homebrew/bin/yarn
npm: 8.5.5 - /opt/homebrew/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.0.1, iOS 15.0, macOS 12.0, tvOS 15.0, watchOS 8.0
Android SDK: Not Found
IDEs:
Android Studio: 2021.1 AI-211.7628.21.2111.8309675
Xcode: 13.1/13A1030d - /usr/bin/xcodebuild
Languages:
Java: <IP_ADDRESS> - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: ^17.0.2 => 17.0.2
react-native: ^0.68.1 => 0.68.1
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
**
Snack, code example, screenshot, or link to a repository
No response

Update::
Just commented out all my useEffects in the codebase and it seems as if the app is stable and not heating up anymore
my bad.

I have the same issue. How commenting out all useEffect could resolve this ? Do you have any clue why this happens ?

Sorry @ddeschenes-lacapitale forgot to post an update here too
UPDATE: I solved this issue, there were a number of reasons why it started

I had an animation running in the background because a useEffect function cleanup did not stop the animation when navigating away from the screen

I had numerous useFocusEffect without a React.useCallback function so the useFocusEffect would have had a cleanup function through the React.useCallback

Some useEffect functions had no cleanup- During the time I was facing the issue, I changed ALL (about 420 in total) of my components and screens from normal components to memoized (React.memo) which means that all my components were being stored in memory as soon as they rendered - which is bad because it obviously causes high RAM usage, so we had to delete those - and only use them on about 2 components that rendered multiple times and had costly computations that didn't need to run every time they rendered

After doing all of that, my CPU and memory usage drastically reduced to what it is below

same issue
@Jaybee4real
can you explain it more?
is it possible to help me?

@jackimatin, sure, if you are still experiencing it, reach out to me via mail<EMAIL_ADDRESS>

thank a lot,@Jaybee4real
I did it and everything is great now.

@jackimatin can you share what were the fixes.. any snippets can help ..

Hi @Jaybee4real , we are facing the exact similar issue what you have posted here,  surprisingly this is happening on iOS 17 and above devices only with our react native application which is built using React Native 0.72.0, any help will be highly appreciated.
Thank you!
https://stackoverflow.com/questions/78049075/how-to-identify-and-fix-device-heat-in-ios-application-developed-using-react-nat


For CatalogItems, attributes  in IncludedData is not working.

Describe the bug
In CatalogItems the option  IncludedData type for attributes is showing error for search_catalog_items function,  there seems to be a mistake in core code for not including this option please take a look to fix it.
To Reproduce
from sp_api.api import CatalogItems
CatalogItems(credentials=credentials, marketplace=Marketplaces.IN).search_catalog_items(keywords=['B09C2NSYVT,B0B6RLSSH8'],marketplaceIds=['A21TJRUUN4KGV'], includedData=["attributes"]).payload
Expected behavior
'attributes': {'binding': [{'value': 'spiral_bound',
'marketplace_id': 'A21TJRUUN4KGV'}],
'item_package_weight': [{'unit': 'kilograms',
'value': 0.07,
'marketplace_id': 'A21TJRUUN4KGV'}],
'language': [{'type': 'published',
'value': 'english',
'marketplace_id': 'A21TJRUUN4KGV'}],
'manufacturer': [{'language_tag': 'en_IN',
'value': 'Rays Of Ink',
'marketplace_id': 'A21TJRUUN4KGV'}],
'pages': [{'value': 20, 'marketplace_id': 'A21TJRUUN4KGV'}],
'recommended_browse_nodes': [{'value': '1318060031',
'marketplace_id': 'A21TJRUUN4KGV'}],
'bullet_point': [{'language_tag': 'en_IN',
'value': '[Stress Relieving Designs that are Great for Relaxation] Each coloring page is designed to provide calmness and relaxation as you channelize your energies for creative expression.',
'marketplace_id': 'A21TJRUUN4KGV'},
{'language_tag': 'en_IN',
'value': '[Beautiful Artwork and Designs] Well-crafted illustrations and designs that lay the groundwork for you to create your own frame-worthy mandalas.',
'marketplace_id': 'A21TJRUUN4KGV'},
{'language_tag': 'en_IN',
'value': '[High Resolution Digital Printing] Each image is printed in high resolution to offer crisp, sharp designs that enable trouble free coloring and high quality display.',
'marketplace_id': 'A21TJRUUN4KGV'},
{'language_tag': 'en_IN',
'value': '[Single-sided Pages] Every image is printed on a single-sided page, so that you can use a broad variety of coloring choices without fearing bleed through. Moreover, single-side pages can be framed to display your masterpieces.',
'marketplace_id': 'A21TJRUUN4KGV'},
{'language_tag': 'en_IN',
'value': '[Suitable for Adults With Different Skill Levels] This coloring book offers a broad variety of designs suited for all skill levels - ranging from beginner to expert level.',
'marketplace_id': 'A21TJRUUN4KGV'}],
'publication_date': [{'value': '2021T', 'marketplace_id': 'A21TJRUUN4KGV'}],
'item_dimensions': [{'width': {'unit': 'centimeters', 'value': 18.0},
'length': {'unit': 'centimeters', 'value': 24.0},
'height': {'unit': 'centimeters', 'value': 1.0},
'marketplace_id': 'A21TJRUUN4KGV'}],
'subject_keyword': [{'language_tag': 'en_IN',
'value': 'ar',
'marketplace_id': 'A21TJRUUN4KGV'}],
'product_description': [{'language_tag': 'en_IN',
'value': 'This adult coloring book from Rays of Ink has 20 unique handmade mandala designs. It provides hours of fun, calm, relaxation and stress relief through creative expression. These beautiful designs are printed single-sided and range in complexity and detail from intermediate to expert-level.',
'marketplace_id': 'A21TJRUUN4KGV'}],
'brand': [{'language_tag': 'en_IN',
'value': 'Rays Of Ink',
'marketplace_id': 'A21TJRUUN4KGV'}],
'supplier_declared_dg_hz_regulation': [{'value': 'not_applicable',
'marketplace_id': 'A21TJRUUN4KGV'}],
'generic_keyword': [{'language_tag': 'en_IN',
'value': 'coloring books for adult mandala colouring books mandala drawing book mandala books for adults adult coloring books therapy mandala art book for drawing mandala art book for adult',
'marketplace_id': 'A21TJRUUN4KGV'}],
'item_name': [{'language_tag': 'en_IN',
'value': 'Rays Of Ink Mandala Coloring Book For Adults | Ideal For Stress Relieving, Relaxing & Meditation | 120 GSM Thick Paper',
'marketplace_id': 'A21TJRUUN4KGV'}],
'list_price': [{'value_with_tax': 299.0,
'currency': 'INR',
'marketplace_id': 'A21TJRUUN4KGV'}],
'batteries_required': [{'value': False, 'marketplace_id': 'A21TJRUUN4KGV'}],
'product_site_launch_date': [{'value': '2021-08-06T16:21:56.988Z',
'marketplace_id': 'A21TJRUUN4KGV'}],
'item_package_dimensions': [{'length': {'unit': 'centimeters', 'value': 1.4},
'width': {'unit': 'centimeters', 'value': 20.9},
'height': {'unit': 'centimeters', 'value': 30.0},
'marketplace_id': 'A21TJRUUN4KGV'}]}
A clear and concise description of what you expected to happen.
The output should come like above, instead it shows this error
SellingApiBadRequestException: [{'code': 'InvalidInput', 'message': "Invalid 'includedData' requested.", 'details': ''}]

OS: [e.g. Linux/Mac/Windows]
Ubuntu 22.04

 includedData="attributes"

Not all sellers can get all data.

We have access to attributes data and it is working fine in Postman, but once I make a call with the Python API it shows an error
.This asin is also from our own brand


Fix turret plates on wrong team

I realised we have no patch in the game timeline, so the only fix i could think of was checking if we can get an event_dto for the turret plate or not.
If there is event_dto: 12.14 or earlier
If not: 12.15 or newer
I also added a test when merging to check that the number of turret kill events in the timeline matches the number of turrets destroyed in data.
I imagine this change in Riots timeline wasnt intended, as the teamId still belongs to the "owner" of the turret for turret kill events, but not for turret plate kill events. So maybe it'll change again in the future, who knows.
These are the vods for the test games I added:
ESPORTSTMNT03_2961931: https://www.youtube.com/watch?v=i34uceAGw0I (Post 12.15)
[ESPORTSTMNT01_3096461: https://www.youtube.com/watch?v=K2BZKorXGi4 (Pre 12.15)

Looks good overall!
Could you add a small test to check that previous behaviour was unchanged? Validating the value of LolGameTeamBuildingKill for one game before 12.15 and for another after.
Thanks a lot for your help, once I've merged the PR I'll do some cleaning up on this repo too.

Thank you! I don't understand how to validate LoLGameTeamBuildingKill, though, im kind of new in this programming world, sorry hahah

Just write a test with one of the .json you added and verify that the resulting value is good!
Pretty much just call the function and check the return values.

So... i made the checks with ESPORTSTMNT03_2961931 (the 12.16 game) checking the output with the bayes rofl, but nothing makes sense regarding the turret plates, for example

This image shows that the first turret plate for blue team was in mid lane, however both in the riot_transmute output and in bayes timeline it shows that both teams got their first turret plates in bot lane
Game timeline straight from bayes (first turret_plate events for each team):

The timestamp makes sense for both teams, but the lanes are completely wrong

Please fix the merge conflicts so we can run the tests!

Please fix the merge conflicts so we can run the tests!

Fixed!
Regarding your question in the issue, in my opinion we should merge the pr so that it won't fail with games from 12.15 to 12.17


Fixed english spelling

Noticed this small English typo when doing TryRuby.

I wasn't sure if I should also commit the changes from source/try_ruby_en.json after running
bundle exec middleman server

so I followed previous commit conventions and left it out.

Thanks for the update. With or without json is both okay, I have to 'github publish' anyway.


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
f13e310b064a622414774c39b55daf47c7a228a4
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.


Add flag for latest prusaslicer build

Currently, I don't think there's a way to set a flag to pull the "latest" tagged build of prusaslicer. This means that when you need to run the latest version, even if it's a RC or beta, there's no way to do it automatically. Using "latest" on the container just pulls the latest version of this container, not prusaslicer itself.
It would be great to add a flag to pull the latest version of prusaslicer and/or to specify a particular version/build of prusaslicer to pull.

Good suggestion! Check out the fresh commit and container that updates to the latest Prusaslicer on every start - unless your specify OVERRIDE_SLICER_URL in env to stick to a given version.

This is great, and better than the PR I was going to suggest. Thanks much!


Run tests against a mysql 5.6 server

Most of the PHP7 tests with MySQL were failing , so I "fenced" them like in my last PR.

Nice!
One thing: Could you make the MySQL test optional, for example, in Main.hx,
if (mysqlConnectionInfo != null)
    emu.addTest(MySQLTest);

Such that it would be easier for people to run the test locally without setting up MySQL?

Good idea, it's done.


ascii encoding errors when a user's name includes a funky character

One of our employee's names has a ç in it.  That causes this error:
>>> sc.server.channels.find("C0B8PQGSY")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/weitzenfeld/.virtualenvs/qbit/lib/python2.7/site-packages/slackclient/_channel.py", line 21, in __repr__
    return self.__str__()
  File "/Users/weitzenfeld/.virtualenvs/qbit/lib/python2.7/site-packages/slackclient/_channel.py", line 17, in __str__
    data += "{} : {}\n".format(key, str(self.__dict__[key])[:40])
  File "/Users/weitzenfeld/.virtualenvs/qbit/lib/python2.7/site-packages/slackclient/_server.py", line 36, in __str__
    data += "{} : {}\n".format(key, str(self.__dict__[key])[:40])
  File "/Users/weitzenfeld/.virtualenvs/qbit/lib/python2.7/site-packages/slackclient/_user.py", line 25, in __repr__
    return self.__str__()
  File "/Users/weitzenfeld/.virtualenvs/qbit/lib/python2.7/site-packages/slackclient/_user.py", line 20, in __str__
    encoded_val = str(self.__dict__[key]).encode('ascii', errors='ignore')
UnicodeEncodeError: 'ascii' codec can't encode character u'\xe7' in position 10: ordinal not in range(128)

It looks like the offending line of code has been modified, so I'm going to close this issue for now. If you're still hitting this error, please re-open and we'll get it fixed.
Sorry for the slow response and thanks for reporting this!


Code to be deprecated

After 2DC is fully deprecated, remove the following files / lines:
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/historyEngine.go#L1045-L1054
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/historyEngine.go#L2265-L2275
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/historyReplicator.go
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/historyReplicator_test.go
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableState.go#L140
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableState.go#L212-L213
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L109
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L217-L232
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L259-L260
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L283
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L357-L359
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L3882
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L3960
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L4180-L4189
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L4221-L4223
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L451-L455
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L494-L496
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L507-L510
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L529-L531
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L548-L557
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L559-L582
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/stateBuilder.go#L111-L116
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/stateBuilder.go#L186-L189
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/workflowResetor.go
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/workflowResetor_mock.go
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/workflowResetor_test.go
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/workflowExecutionContext.go#L116-L129
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/workflowExecutionContext.go#L961-L1138
Change the following lines:
https://github.com/uber/cadence/blob/9ecec8e62d06306fcb2c11747fbe930547d4436d/service/history/mutableStateBuilder.go#L4240
return (e.GetReplicationState() != nil || e.GetVersionHistories() != nil) &&
		e.domainEntry.GetReplicationPolicy() == cache.ReplicationPolicyMultiCluster

to
return e.domainEntry.GetReplicationPolicy() == cache.ReplicationPolicyMultiCluster

Those are clean up


Sharing does not work from installed kmdr-cli

Describe the bug
A clear and concise description of what the bug is.
Sharing does not work from installed kmdr-cli
To Reproduce
Steps to reproduce the behavior:

Go to your terminal
Enter 'kmdr e', hit enter, type your command and hit enter again
Scroll down to the end of the explanation, use your arrow key to select 'Share this explain session' and hit enter again. You are then prompted to put in a name for the session to save. At this point it does not matter if you put in a name or not. After hitting enter for a last time, you are presented a link that is clickable to open in the browser. This runs into a timeout on Chrome and Firefox for me. It makes no difference if I click the link or copy&paste into the browser.
See error

Expected behavior
A clear and concise description of what you expected to happen.
I expect the link to open up in the browser.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Debian Sid
Browser  Firefox 73 and Chrome 79
kmdr-cli 0.4.3

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hi @siduxdevil , thanks for the report.
I couldn't reproduce the bug in version 0.4.3. I created two command explanations [0] using the latest version and the URLs did open on my browser. It could be that the deployment goes to sleep after some inactive time
Please try to open these urls and check if it works. Your explanation sessions should work too.
[0] - https://explain.kmdr.sh/s/c75be660-51e9-11ea-9d47-a942542b7054, https://explain.kmdr.sh/s/c75be660-51e9-11ea-9d47-a942542b7054


Add unfurl image to abi.ninja links

Description
WIP: will update ASAP!
Additional Information

[ ] I have read the contributing docs (if this is your first contribution)
[ ] This is not a duplicate of any existing pull request

Related Issues
Closes #{issue number}
Note: If your changes are small and straightforward, you may skip the creation of an issue beforehand and remove this section. However, for medium-to-large changes, it is recommended to have an open issue for discussion and approval prior to submitting a pull request.
Your ENS/address:

Closing this because of #90


garmbot stole my token

henlo garmbot stole my token and is now buying RAM from amazon also it needs to stop stealing ur ram thank you that is all

this is unacceptable please fix @dnaf


Skip admin override for Solidus >= 1.0

We no longer use this override in new versions, so we should skip it.
This should remove the warning:
Deface: 1 overrides found for 'spree/admin/shared/_header'
Deface: 'auth_admin_login_navigation_bar' matched 0 times with '[data-hook='admin_login_navigation_bar'], #admin_login_navigation_bar[data-hook]'

:+1:, thanks @jhawthorn.


Duplicate references to framework assemblies added

I think this is very similar to #932, if not actually exactly the same thing.
When adding a package to a project using Paket, a <Choose> section is added to the project file with references to all the assemblies in that package, as well as apparently any of the framework assemblies that are stated as dependencies in the package but not already referenced by the project. When the next package is added, those references to the framework assemblies are not found and added again in that package's <Choose> section.
An example:
FsXaml and FSharp.Desktop.UI reference the following framework assemblies:

PresentationCore
PresentationFramework
System.Xaml
System.Xml (FsXaml only)
WindowsBase

I now use Paket to add those to a project that already references System.Xaml, System.Xml and WindowsBase (added through the References dialog, for example).
Adding FSharp.Desktop.UI now inserts the following into the *.fsproj file:
<Choose>
  <When Condition="$(TargetFrameworkIdentifier) == '.NETFramework' And ($(TargetFrameworkVersion) == 'v4.5' Or $(TargetFrameworkVersion) == 'v4.5.1' Or $(  TargetFrameworkVersion) == 'v4.5.2' Or $(TargetFrameworkVersion) == 'v4.5.3' Or $(TargetFrameworkVersion) == 'v4.6' Or $(TargetFrameworkVersion) ==   'v4.6.1')">
    <ItemGroup>
      <Reference Include="FSharp.Desktop.UI">
        <HintPath>..\..\packages\FSharp.Desktop.UI\lib\net45\FSharp.Desktop.UI.dll</HintPath>
        <Private>True</Private>
        <Paket>True</Paket>
      </Reference>
      <Reference Include="PresentationCore">
        <Paket>True</Paket>
      </Reference>
      <Reference Include="PresentationFramework">
        <Paket>True</Paket>
      </Reference>
    </ItemGroup>
  </When>
</Choose>

Next I add FsXaml; System.Xaml, System.Xml and WindowsBase are detected correctly, but the other two framework assembly references are not "seen" and get included again for FsXaml:
<Choose>
  <When Condition="$(TargetFrameworkIdentifier) == '.NETFramework' And ($(TargetFrameworkVersion) == 'v4.5' Or $(TargetFrameworkVersion) == 'v4.5.1' Or $(  TargetFrameworkVersion) == 'v4.5.2' Or $(TargetFrameworkVersion) == 'v4.5.3' Or $(TargetFrameworkVersion) == 'v4.6' Or $(TargetFrameworkVersion) ==   'v4.6.1')">
    <ItemGroup>
      <Reference Include="FsXaml.Wpf.TypeProvider">
        <HintPath>..\..\packages\FsXaml.Wpf\lib\net45\FsXaml.Wpf.TypeProvider.dll</HintPath>
        <Private>True</Private>
        <Paket>True</Paket>
      </Reference>
      <Reference Include="FsXaml.Wpf">
        <HintPath>..\..\packages\FsXaml.Wpf\lib\net45\FsXaml.Wpf.dll</HintPath>
        <Private>True</Private>
        <Paket>True</Paket>
      </Reference>
      <Reference Include="PresentationCore">
        <Paket>True</Paket>
      </Reference>
      <Reference Include="PresentationFramework">
        <Paket>True</Paket>
      </Reference>
    </ItemGroup>
  </When>
</Choose>

The result is that the respective framework assemblies show up multiple times in Solution Explorer; however, as far as I could see, the project still compiles and runs correctly, probably because they are always references to the exact same thing and thus don't cause conflicts as seen in #932.
My naive first thought would of course be simply putting all references to framework assemblies in normal ItemGroups, but considering we're dealing with Visual Studio, MSBuild and NuGet, that sounds too easy.

I guess we still need Choose Nodes, but need to merge all the packages...

I wonder why it installed the references at all. Usually it should not add the reference if it already finds one which doesn't have a Paket flag.
(this would also lead to a workaround)

I've also already noticed this behavior, but since it leads to no problems during the build and the result assemblies also have no problems at runtime and it's only weird display in the Solution Explorer, I haven't reported this one ;)
Currently a Choose node is generated per package and contains all assemblies, contained in the package as well as framework ones. Only the references already present without Paket flag will be removed from the list of references to include.
To deduplicate the nodes, we would need to collect framework references together with the conditions when each of it should be inserted and provide separate Choose nodes for them.
When removing packages, we would need to consider which of those references should also be removed, as not referenced anymore by any of the remaining packages.

I have the same behavior. If you add to empty F# project nuget packages - FSharp.Data and WindowsAzure.Storage you will see System.Xml twice under References node:

Although I have 3 System.Xml entries and 2 System.Xml.Linq entries in our solution it doesn't affect build process.

can you please zip that sample project and attach it here? thanks

Yes, in attachment.
ConsoleApplication1.zip

Ok one System.Xml.Linq.dll is coming from  FSharp.Data and one from Microsoft.WindowsAzure.Storage
we could somehow store if we already emitted the reference for a given framework and only reference it once. So it would be ommited for Microsoft.WindowsAzure.Storage.
Is it worth it?

I believe this goes into the category of "should fix". It's (I believe) mostly harmless but it's something that can confuse users and is not expected.

so what do we do? removing the second reference?
Reordering choose nodes by grouping by target framework (see @mexx 's comment) is not what I want to do before we know how nuget is doing this in vNext.


WIP Advise packagers on requesting upload size limit increase

Failing a test because I am not properly plumbing the route call through Pyramid's URL mapping:
warehouse/forklift/legacy.py:1001: in file_upload
    'help', _anchor='file-size-limit'
../lib/python3.6/site-packages/pyramid/url.py:261: in route_url
    mapper = reg.getUtility(IRoutesMapper)

Looks like we should also fix https://github.com/pypa/warehouse/blob/master/warehouse/forklift/legacy.py#L833 to use URL mapping.
Going to fix after lunch, would welcome tips.

Yes, you'll need to mock out route_url in the test, something like this:
db_request.route_url = pretend.call_recorder(
    lambda route, **kw: "/the/help/url/"
)

@di Thanks for the tip. I have fixed the test for the filesize message, and will look at the other upload error messages that need URL substitution in a separate PR. I believe this is ready to merge.


[Question] Minor questions about this tutorial

Thank you very much for writing this series of tutorials. It's a great study material for me. In the process of learning I had some small questions:
In Part 1: STARK Overview, here's what got me wondering:

Interpolation and Polynomial IOPs: …… At this point the AIR constraints give rise to operations polynomials that send low-degree polynomials to low-degree polynomials only if the constraints are satisfied. The verifier simulates these operations and can thus derive new polynomials whose low degree certifies the satisfiability of the constraint system, and thus the integrity of the computation.

what do 'operations polynomials' stand for ? Or maybe I got something wrong with my sentence break...?
'send low-degree polynomials to low-degree polynomials' means what ? After having the AIR constraints, I think it's the time for getting 'quotients' , what do the two 'low-degree polynomials' here refer to respectively?

I would appreciate it if you could answer my questions above.

Excuse me, that should be: operations on polynomials.
The preimage of this map is a trace polynomial; the afterimage of this map is a quotient. The quotient is a low degree polynomial if and only if the trace polynomial is a low degree polynomial and satisfies the AIR constraint.


getRemoteAddress() in javalin 3.5

Hi again,
I use to use WsSession.getRemoteAddress() in Javalin 2.7 but, now that I am using javalin 3.5, I don't see that method defined for any of the WsContext classes.
Does this method's functionality exist somewhere?
Thanks again for the help.

Its probably available through ctx.session.getRemoteAddress() (?)