Ndt7: Add a way to measure latency

Hi everyone,
Is there a way to measure latency on ndt7 servers? So far I can only get results like this one:
    {
      "machine": "mlab1-bom02.mlab-oti.measurement-lab.org",
      "location": {
        "city": "Mumbai",
        "country": "IN"
      },
      "urls": {
        "ws:///ndt/v7/download": "ws://ndt-mlab1-bom02.mlab-oti.measurement-lab.org/ndt/v7/download?access_token=eyJhbGciOiJFZERTQSIsImtpZCI6ImxvY2F0ZV8yMDIwMDQwOSJ9.eyJhdWQiOlsibWxhYjEtYm9tMDIubWxhYi1vdGkubWVhc3VyZW1lbnQtbGFiLm9yZyIsIm1sYWIxLmJvbTAyLm1lYXN1cmVtZW50LWxhYi5vcmciXSwiZXhwIjoxNjAxMzU4OTMwLCJpc3MiOiJsb2NhdGUiLCJzdWIiOiJuZHQifQ.-3QOaLvbNaZ3bUD4jDNGqP17FpR5BQp8DuC5mtQ3SZnFyE2HcYbpZSuTh3N-a1Ff9XLOoBECYgE4heZFu5T1BQ",
        "ws:///ndt/v7/upload": "ws://ndt-mlab1-bom02.mlab-oti.measurement-lab.org/ndt/v7/upload?access_token=eyJhbGciOiJFZERTQSIsImtpZCI6ImxvY2F0ZV8yMDIwMDQwOSJ9.eyJhdWQiOlsibWxhYjEtYm9tMDIubWxhYi1vdGkubWVhc3VyZW1lbnQtbGFiLm9yZyIsIm1sYWIxLmJvbTAyLm1lYXN1cmVtZW50LWxhYi5vcmciXSwiZXhwIjoxNjAxMzU4OTMwLCJpc3MiOiJsb2NhdGUiLCJzdWIiOiJuZHQifQ.-3QOaLvbNaZ3bUD4jDNGqP17FpR5BQp8DuC5mtQ3SZnFyE2HcYbpZSuTh3N-a1Ff9XLOoBECYgE4heZFu5T1BQ",
        "wss:///ndt/v7/download": "wss://ndt-mlab1-bom02.mlab-oti.measurement-lab.org/ndt/v7/download?access_token=eyJhbGciOiJFZERTQSIsImtpZCI6ImxvY2F0ZV8yMDIwMDQwOSJ9.eyJhdWQiOlsibWxhYjEtYm9tMDIubWxhYi1vdGkubWVhc3VyZW1lbnQtbGFiLm9yZyIsIm1sYWIxLmJvbTAyLm1lYXN1cmVtZW50LWxhYi5vcmciXSwiZXhwIjoxNjAxMzU4OTMwLCJpc3MiOiJsb2NhdGUiLCJzdWIiOiJuZHQifQ.-3QOaLvbNaZ3bUD4jDNGqP17FpR5BQp8DuC5mtQ3SZnFyE2HcYbpZSuTh3N-a1Ff9XLOoBECYgE4heZFu5T1BQ",
        "wss:///ndt/v7/upload": "wss://ndt-mlab1-bom02.mlab-oti.measurement-lab.org/ndt/v7/upload?access_token=eyJhbGciOiJFZERTQSIsImtpZCI6ImxvY2F0ZV8yMDIwMDQwOSJ9.eyJhdWQiOlsibWxhYjEtYm9tMDIubWxhYi1vdGkubWVhc3VyZW1lbnQtbGFiLm9yZyIsIm1sYWIxLmJvbTAyLm1lYXN1cmVtZW50LWxhYi5vcmciXSwiZXhwIjoxNjAxMzU4OTMwLCJpc3MiOiJsb2NhdGUiLCJzdWIiOiJuZHQifQ.-3QOaLvbNaZ3bUD4jDNGqP17FpR5BQp8DuC5mtQ3SZnFyE2HcYbpZSuTh3N-a1Ff9XLOoBECYgE4heZFu5T1BQ"
      }
    }

and while the ICMP protocol works on mlab1-bom02.mlab-oti.measurement-lab.org it's not something that I can use in a Javascript client where only HTTP/HTTPS/Websocket is accepted. It'd be great if it can be measured through the WebSocket on the client side since that would allow the client to also measure the jitter. So far I came up with a hacky solution of using the wss:///ndt/v7/upload to upload 1 bit of data, wait for the response and measure the time between the two, then divide by 2. It's working "okay" relative to what I get with a ping command but sometimes it's completely not accurate.
Thanks!

@stephen-soltesz

I have started working on this and have implemented ~50% of it here: https://github.com/bassosimone/ndt-server/tree/feature/roundtrip. It should take 1-2 more weeks for me to finish it 🤞

Awesome news! Thank you so much! And how does it work when it comes to deployment exactly once the feature is released? Is it automatically deployed to all your NDT7 servers?

@EKami if the PR is accepted upstream, I guess it should be released in a few weeks.

@EKami the method you described for the "hacky solution" using upload will not work the way you want. The messages sent by the server are randomly timed. So, there is no helpful correspondence between upload start time, time of first message received from the server, and the system to system ping times.
@EKami what is the use-case for this type of latency measurement? Application level latency measurements (as @bassosimone is proposing) may be inherently larger than the equivalent ping measurement from the same systems.

@stephen-soltesz
yeah, I knew there was something fishy about my implementation lol. The use case is simply to be able to run this with a js client since we can't use ICMP. That would also allow me to calculate the jitter which in turn, can tell if the connection is stable or has some micro-disconnections/ is unstable.

I have written a prototype and am now evaluating it (see the above commits referencing this issue). It will take a little more time to see whether this works in reasonable way in JavaScript. If I see convincing data, then I will turn this prototype into a spec and submit it to the M-Lab core team for evaluation.

Awesome news, thanks a lot @bassosimone :D

ping! (excuse the pun)
is there any prospect of this feature being implemented? @bassosimone did anything come from your implementatin/evaluation?
I'm also interested in having this feature (application level latency-only test) available.

FYI: @mattmathis in case you have thoughts.


Tube/plate/well counter examples succeeded validation

it seems like the rules aren't consistently working

Counter example src/data/invalid/Database-biosamples-rna-in-plate-no-well-val.yaml succeeded validation
Counter example src/data/invalid/Database-biosamples-rna-in-tube-with-well-value.yaml succeeded validation
Counter example src/data/invalid/Database-biosamples-dna-in-tube-with-well-value.yaml succeeded validation
Counter example src/data/invalid/Database-biosamples-dna-in-plate-no-well-val.yaml succeeded validation

This still needs work. Start a new branch.

@turbomam is this still needed? I checked at your request in https://github.com/microbiomedata/nmdc-schema/issues/1243 and I tested your rules and they appear to work and not pass validation. It looks to me like these files can be moved into the invalid folder and this can be closed?

thanks for looking. there have been changes to the rules engine since I last tried. I'll double check now.


Feature: Save tcp and udp request inside database

Is your feature request related to a problem? Please describe.
It's already done for http request, we should do it aswell for tcp and udp stream.
This is a daemon change you can see how i did it for inside bin/nanocld/src/utils/proxy.rs
You basically have to create a database model with diesel then create the data when you receive it.
The receive process is already done. The ncdproxy should print a line starting with #STREAM when a tcp / udp request is fired.

Hi @leon3s. I would like to work on this issue. I have experience working with diesel.

Hi @Narayanbhat166 sure go ahead! Let me know if you have problem or need additional information!

Thankyou @leon3s. Yes, that would be helpful.
I have looked at the file proxy.rs
and found the part where TCP/UDP stream logs should be parsed and saved to the database.
https://github.com/nxthat/nanocl/blob/4cbf3155434e81e0de3828f91ac63832da4610b9/bin/nanocld/src/utils/proxy.rs#L76
I have setup the docker locally and want to test out a tcp request so that I can know the structure of it. How can I achieve this? Is there any way I can get this structure without running the application and testing out a tcp request through the proxy?

Yeah you can see how it's structured there:
https://github.com/nxthat/nanocl/blob/4cbf3155434e81e0de3828f91ac63832da4610b9/bin/nproxy/nginx.conf#L132
and we have a commented structure that you could probably reuse there:
https://github.com/nxthat/nanocl/blob/4cbf3155434e81e0de3828f91ac63832da4610b9/bin/nanocld/src/models/http_metric.rs#L208
To know the structure without testing, but you will have to test at some point!
To trigger the log you can deploy this file:
Kind: Deployment
ApiVersion: v0.8

Namespace: global

# See all options:
# https://docs.next-hat.com/references/nanocl/resource
Resources:
  - Name: deploy-example.com
    Kind: ProxyRule
    Version: v0.5
    Config:
      Watch:
        - tcp-example.global.c
      Rules:
        - Network: Public
          Protocol: Tcp
          Port: 5555
          Target:
            Key: tcp-example.global.c
            Port: 9000

# See all options:
# https://docs.next-hat.com/references/nanocl/cargo
Cargoes:
  - Name: tcp-example
    Container:
      Image: nexthat/nanocl-get-started:latest

Then you can test by going on http://localhost:5555


Repository home page show error file last commit message.

Compare github's home page
https://github.com/go-gitea/gitea/tree/release/v1.0

vs https://try.gitea.io/gitea/gitea/src/branch/release/v1.0

Possibly related to https://github.com/go-gitea/git/pull/91. I will investigate as soon as I get the chance.

Repo.cache will be invoked in multiple go routines.

Maybe add a test some commit tree the same as github.

Confirmed release/v1.3 has no this issue.

resolved by #3240


fix typos in README.md

Thanks @kungfoome, we'll keep the same process as Zarf and have 2 approvers for external PRs. @bdw617, pending your approval.


(Bug) #750 Receive: main two buttons are without margins between them

Description
Set the strict margin for the share button in the Receive component.
About #750
How Has This Been Tested?
Register/login to the app.
Open receive screen (click to receive button next to the claim button)
Also, the Claim screen countdown element margin fixed in the scope of this PR.
To test open claim screen. The claim button and countdown element should be with margin between them always.
Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[x] I have added screenshots related to my pull request ( for frontend tasks)
[ ] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

Screenshots:

Claim
Receive

@AnastasiiaOdnoshevna this can be tested in preview branch

@yaroslav-fedyshyn-nordwhale this is also an issue in the claim screen, when user is not verified and he sees both counter+button

Yeah, right. Forgot about that. I found the issue with the margins. Fixed in a new commit.


[CONNECT] Limit the nesting level when truncating a protobuf message

What changes were proposed in this pull request?

Abbreviating a protobuf message is ineffective if the message is very deeply nested, causing a serious performance problem when converting the message into a string. This PR adds a new parameter, maxLevel, to ProtoUtils.abbreviate which limits the maximum nesting level to abbreviate and truncate any data that is below the level. As a result, ProtoUtils.abbreviate may return a partial protobuf message in case the message is deeply nested and a small value of maxLevel is supplied.
Why are the changes needed?

Described above.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

"truncate nested message" added to AbbreviateSuite.scala.
Was this patch authored or co-authored using generative AI tooling?

No.

cc @zhengruifeng . Let's file a JIRA and add it into the PR title

Merging

@changgyoopark-db we also have a similar truncation in python client
https://github.com/apache/spark/blob/19fbdaa54cd783aec988ff480bc190e983a48d1b/python/pyspark/sql/connect/client/core.py#L1009-L1052
would you mind help fixing it too?

@zhengruifeng thanks, wasn't aware of it - I'll address this under https://issues.apache.org/jira/browse/SPARK-49357


Coverage.

When i am trying to run bin/kahlan --config="kahlan-config.travis.php" --coverage="kahlan\plugin\Call".
It outputs:
Coverage Summary
----------------

Unexisting namespace: `kahlan\plugin\Call`, coverage can't be generated.

It is a quite strange.

Weird, it's working for me on a clean install.

I am reproducing it with a clean install. Any suggestions how to trace this.

Also tried to running it with HHVM and it works. Try to clean up caches rm -rf /tmp/kahlan and run it again.
What php version are you using ?
Does bin/kahlan --config="kahlan-config.travis.php" --coverage="kahlan\plugin" spit out the kahlan\plugin\Call.php file ?

PHP 5.5.20-1+deb.sury.org~precise+1 (cli) (built: Dec 21 2014 19:54:33)
Copyright (c) 1997-2014 The PHP Group
Zend Engine v2.5.0, Copyright (c) 1998-2014 Zend Technologies
    with Zend OPcache v7.0.4-dev, Copyright (c) 1999-2014, by Zend Technologies
    with Xdebug v2.2.5, Copyright (c) 2002-2014, by Derick Rethans

Removed cache and still the same.

Yes bin/kahlan --config="kahlan-config.travis.php" --coverage="kahlan\plugin" echo a coverage on plugin\Call.php

Are you running it inside a vagrant box or something ?

The displayed error is this one: https://github.com/crysalead/kahlan/blob/master/src/reporter/Coverage.php#L266
you should be able to do a:
print_r(array_keys($this->metrics()->get("kahlan\plugin")));

And you should see all classes inside plugin.
but if 'plugin\Call.php' has been spitted out, there's no reason:
print_r(array_keys($this->metrics()->get("kahlan\plugin\Call")));

can't display the correct information.

Yes, i am running inside a vagrant.

Still not able to reproduce on travis (https://travis-ci.org/crysalead/kahlan/builds/49312153). Feel free to reopen if you find the root cause.

Find a bug. Fix it on https://github.com/crysalead/jit/pull/1


Adding DNF Sort in Sorting Methods

As we know DNF sort is the special sorting way in which we can sort the array having 0's, 1's, and 2's in O( n ) and without taking extra space which makes one of the efficient ways to sort a set type of array.

I would like to work on this issue.
As a GSSOC'22 Contributor.
And Assure You that this will add value to your repo.

DNF sorting Algorithm already present not want to create ambiguity.


chore(spin.rb): Spin v2.2.0 bumps

(To merge in conjunction with upcoming cloud plugin 0.7.0 release/availability)

Thanks @vdice. As you mention, lets merge this after https://github.com/fermyon/spin-plugins/pull/50 merges


Authorize endpoint: start on validation, error codes, etc.

Start testing authorization endpoint, fixing a few bugs as I go.

Is mostly the good. Just little naggy things/things requiring some discussion


How fast is the descriptor extraction?

This is hard for me to test as I don't have Matlab and didn't find an answer in the paper. Could anyone help? I'm essentially wondering what kind of throughput this can achieve on a GPU in terms of images/s.

Hi, I'll check this for you tonight as I don't have Matlab at hand right now.
Btw, there is an independent tensorflow implementation of NetVLAD (though they didn't use it for place recognition but for winning the Youtube-8M challenge):
https://github.com/antoine77340/LOUPE
There's also an independent caffe implementation used for place recognition:
https://github.com/hyojinie/crn

I checked some old logs, as right now I don't have access to a MATLAB+GPU, and as far as I can see, for a batch of 4 images that are 640x480 each (so much larger than the standard 224x224), including all overhead of loading the image and saving descriptors on disk (nothing smart done here, no pre-fetching or reading images in parallel as is generally standard in deep learning):
VGG-16 with Max on conv5_3: 1.2s
VGG-16 with NetVLAD on conv5_3: 1.75s
AlexNet with Max on conv5: 0.15s
AlexNet with NetVLAD on conv5: 0.45s
I'm not sure, but I think this was done on a K20. I did not use cuDNN, which should bring all these numbers down.


implement new import scripts to update Brazilian data

The Brazilian dataset is quite outdated. Especially as a recent organizational reform changed a lot of Ministries.
The source dataset has changed a lot too. Used to be a big XML dump (this is what the import scripts used to read), but now it's a RESTful API using JSON. The data model and fields available have also changed. So the scripts need to be completely re-done.

@augusto-herrmann are you interested in taking this or should I move forward with this script?

Sure, but @todrobbins before I work in this, I'd like to confirm whether or not you actually did any work in it already, considering a couple of months have passed since then.

@augusto-herrmann i don't believe @todrobbins has done any specific work here so please go ahead!

@rufuspollock is correct as I have not done any work in this area. Carry on!

I am having difficulty with this as I am unsure about how to handle the changes in organizational structures and how to map identifiers between versions of the organizational structure. I think we need to solve issue #68 before properly handling this.
Alternatively, I could just naïvely run the scripts to obtain the latest version without caring about keeping the ids consistent between the version in use in publicbodies.org (which is quite old) and the current organizational structure of the Brazilian government. Of course, if there is something that currently depends on those ids, it would most likely be broken by this approach.

@augusto-herrmann i've commented in #68. do you want to try something like that here?

@augusto-herrmann i've commented in #68. We could trial out the solution of #68 here and see how it goes ...

I did not handle or track in time the changes to the structure of public bodies in this PR. There are too many changes over so many years, that it would take an "epic" effort to be able to do that. I don't even think that the structure at a certain point in time is available as open data, so that would be difficult to obtain if not outright impossible.
For now, we just replace the file with the current structure of public bodies in Brazil. Not even the ids are necessarily consistent with the old data, as a different algorithm was used now to generate the slugs (we now use the python-slugify package, while it was a custom slug making code before – I don't think even this lib existed at the time).


Cards' contents overflow

If a project has a long name without spaces, project card is broken and project item in sidebar, same applies to experiment (and all cards in general, as well as the sidebar entries).

Fixed in #97


Emit suggestions and an explanation when CBMC runs out of memory

Improves the error reported to the user when the system kills CBMC to reclaim memory.
Resolves #2715
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

That's great! Thanks.
As a future improvement we should create a wiki with different ways of improving the memory consumption and link it here.

I was thinking the same thing.

@celinval Since this is a benign change it's probably all right if I don't wait for a second review?

@celinval Since this is a benign change it's probably all right if I don't wait for a second review?

You don't need to wait for a second review. I'll merge it for you this time 😛


[Feature] Add new Backgroud and loader

Description
I wanted to add new background and loader to the beautify web
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[x] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Please assign me this issue so that I can add new things to beautiify

Do you want to add 2 new components @Coding-Charvi

yeah sure @Rakesh9100

Which background and loader, pls specify their names clearly with any screenshots if you have @Coding-Charvi

https://pngtree.com/freebackground/blue-stereo-light-and-shadow-texture-box-background_1595689.html I want to add this kind of background and want to add rainbow syled loader @Rakesh9100

Go ahead @Coding-Charvi

Pls provide your PR @Coding-Charvi


Error parsing spread syntax with lang="pug"

Before You File a Bug Report Please Confirm You Have Done The Following...

[X] I'm using eslint-plugin-svelte. (*.svelte file linting does not work with the parser alone. You should also use eslint-plugin-svelte with it.)
[X] I'm sure the problem is a parser problem. (If you are not sure, search for the issue in eslint-plugin-svelte repo and open the issue in eslint-plugin-svelte repo if there is no solution.
[X] I have tried restarting my IDE and the issue persists.
[X] I have updated to the latest version of the packages.

What version of ESLint are you using?
8.25.0
What version of eslint-plugin-svelte and  svelte-eslint-parser are you using?

<EMAIL_ADDRESS><EMAIL_ADDRESS>
What did you do?

Configuration
.eslintrc.yaml
env:
  browser: true
  es6: true
extends:
  - "eslint:recommended"
  - "plugin:svelte/recommended"
parserOptions:
  ecmaVersion: 2020
  sourceType: module

Test.svelte
<script>
  let objs = [{a: 1, b: 2}, {a: 2, b: 3}]
</script>

<template lang="pug">
  +each("objs as obj")
    p("{...obj}") {obj.a}
</template>

What did you expect to happen?
Successful parse.
What actually happened?
$ eslint Test.svelte
/home/projects/node-zm1om6/Test.svelte
7:8  error  Parsing error: Unexpected token
✖ 1 problem (1 error, 0 warnings)
Link to Minimal Reproducible Example
https://stackblitz.com/edit/node-zm1om6?file=Test.svelte
Additional comments
With ("{...obj}") dropped linter issues no-unused-vars and no-undef. It seems eslint-plugin-svelte processes markup as plaintext.
With svelte-preprocess in place Svelte compiler won't issue a warning; however a genuine error would likely result in incorrect line number. Apparently preprocessor does not map original pug to resulting html (same issue with sass). Not good in compiler, it's probably a no-go for linter.
That sounds like a load of work and wontfix. However if you think it's feasible please outline the solution, I'd like to dig in.

Thank you for posting issue.
I don't use pug so I'm not sure if it's possible.
At least it can't be used as an eslint parser if it can't be mapped to the correct location. Also, eslint's parser needs to be synchronous, so we can't even include svelte-preprocess in this parser.

It looks like it is possible to pull out the same trick svelte-preprocess does synchronously. There are two objections to it however:

we'll still have cursor positions mangled; it's really bad but handles no-undef and such; personally I could live with it
more importantly, current AST seems to be html-specific; expanding it to Pug and rewriting eslint-svelte-plugin rules to handle both Html and Pug (which would fix cursor positions) sounds tedious enough, but what if someone wants to handle another markup? another style processor or scripting language?

What do you think? Do you plan to support the whole gamut of languages supported by svelte-preprocess in this parser/plugin, and if yes then what's your vision for it (AST / rules)? Would you accept Pug parsing support without a plan to fix cursor position?

Would you accept Pug parsing support without a plan to fix cursor position?

If it doesn't return the correct position of the node, ESLint's auto-fix will break the source code and is unacceptable.

Do you plan to support the whole gamut of languages supported by svelte-preprocess in this parser/plugin.

The <style> tag does not contain scripts, so I think it can be partially supported, but <template> contains various directives and fragmented scripts, so I think it is difficult.
And, I don't know Pug syntax or AST. So I'm not sure how to implement Pug support.
If someone is willing to work on Pug support and even maintain it,  I would consider adding Pug support to svelte-eslint-parser and eslint-plugin-svelte.
However, those who worked on eslint-plugin-vue's experimental Pug support quickly stopped maintaining them.
So I'm afraid I won't be able to find someone to do the maintenance.


Add optimum-neuron TGI 0.0.19

Description of changes:
This bumps the NeuronX TGI version to 0.0.19.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jinyoung-lim I successfully built the image locally, but the CI is failing, and I don't have access to the logs: can you tell me what went wrong and how I can fix it ?

Lets close this in favor of https://github.com/awslabs/llm-hosting-container/pull/60

Lets close this in favor of #60

The reason why I left it was because otherwise 0.0.19 won't ever be built (the latest version is only rebuilt when merging the pull-request).

Lets close this in favor of #60

What do you mean? Isn't this #60?


Shoe vs Sandal vs Boot Image Classification #466

Pull Request for DL-Simplified 💡
Issue Title : Shoe vs Sandal vs Boot Image Classification #466

Info about the related issue (Aim of the project) :  The aim of this project is to classify the the footwear based on the image .
Name: Dipayan Majumder
GitHub ID: dipayan22
Email ID<EMAIL_ADDRESS>Idenitfy yourself: (Mention in which program you are contributing in. Eg. For a JWOC 2022 participant it's, JWOC Participant) DWOC 24

Closes: #issue number that will be closed through this PR
Describe the add-ons or changes you've made 📃
Give a clear description of what have you added or modifications made
Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[c] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested? ⚙
Describe how it has been tested
Describe how have you verified the changes made
Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] I have added things that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

I have mistakenly given two files (Show vs Sandal vs Boot Classification and Spam vs Ham classification) in one PR. What should I do now?
Please guide me . @abhisheks008

Remove a project folder, then create a PR. Wait for it to be merged after that push the another PR.
@dipayan22

@abhisheks008 file updated . Waiting for your early response.


remove dataset route and add/verify delete endpoints

closes #386

Note: I changed the order of endpoints in api/v1/project.py, so that now they are sorted based on their address. This messes up the PR diff (sorry!), but it brings a lot of clarity when just glancing through the file. Let's do it also on other api files (in a different PR).


Create zh_cn.json

Thanks for the translations! These'll be in for the next update (probably 1.21).


Port TCP Client as s-c-s processor

From @sabbyanandan on November 14, 2015 0:14
As a developer, I'd like to move tcp-client module from XD to s-c-s repo, so I can use it as processor to build streaming pipeline.
Acceptance:

tcp-client as source module is available in spring-cloud-stream-modules repo
maven repo includes tcp-client module with respective group/artifact/version detail

Copied from original issue: spring-cloud/spring-cloud-stream-modules#73

From @ericbottard on January 20, 2016 11:20
Indeed I would write this as a processor today. Back then, it kind of integrated its own source of messages to be transformed. With a trigger-on-steroids that could e.g. generate increasing numbers, this would definitely simplify the setup, with the tcp-client having less responsibility

From @garyrussell on January 19, 2016 19:9
Seems to me that this should be a processor, not a source; similar to http-client.
Not sure why this was made a source in XD.
@ericbottard ??


rawLocalTimestamp time zone is not updated on time zone change

From SiarheiKuchukBelitsoft:
Hello
I have an issue with extracting time zone from rawLocalTimestamp for Node BotFramework SDK 4.9.3
For example I'm in Minsk.
That code
(context.activity).rawLocalTimestamp
shows
...19:00+03:00
If i change the time zone in Windows Clock to UTC+01
and then send message to Bot, it will get from TurnContext
...19:01+03:00
But if i restart Teams Desktop app, it works fine, i.e. it will show
...19:01+01:00
Can you please adjust time zone on each message in Teams Desktop app.
Another big issue with rawLocalTimestamp - that after update Bot to SDK with that feature, users have to relogin to Teams so bot will start getting on backend that field. Please fix it,because noone from customers is going to relogin on bot update!

We could able to reproduce the issue on our side raised a bug.

Restart of Teams is required after timezone is changed. Please ensure Teams was restarted. This is by design

We are closing this issue for now, please feel free to reopen it, if required.


Help with re-creating http "on upgrade" within express-ws

Hi there.
I've successfully been able to relay inbound MQTT websocket requests to another server using the following invocation given a direct reference to the http server, like this (with irrelevant application-details about html pages removed)...
    var httpServer = http.createServer();

    var app = express();

    httpServer.on("request", app);

    var proxy = httpProxy.createProxyServer({
        ws:true,
        target:"ws://localhost:3000",
    });

     httpServer.on('upgrade', function (req, socket, head) {
         proxy.ws(req, socket, head);
     });

However, I'm struggling to find how to satisfy the same constraints within an expressWs route. I've been attempting something like...
    var httpServer = http.createServer();

    var app = express();

    httpServer.on("request", app);

    var proxy = httpProxy.createProxyServer({
        ws:true,
        target:"ws://localhost:3000",
    });

    var expressWsConfig = expressWs(app, httpServer);

    //handle requests for websocket upgrade - in the future, route according to cookies
    app.ws("*", function(ws, req){
        proxy.ws(req, req.connection);
    });

The reason I'm trying to use expressWs is that I need to use the same authentication middleware when serving the page and proxying the connection (also through port 80).
In particular I need to proxy to different backing servers depending on the authenticated identity and session, so ideally I would be able to e.g. inspect Passport's req.isAuthenticated() and benefit from previous calls to req.login() to employ user records to route the proxied websocket request ( http://passportjs.org/docs/login ).
The example client code I am deploying for testing is like this...
var mqtt =  require("mqtt"),
    $ =     require("jquery-browserify");

var client = mqtt.connect("ws://localhost:8080/mqtt");

$(function(){
    setInterval(function(){
        client.publish("hello", "world" + Date.now());
    }, 1000);
})

...and it is successfully able to connect to MQTT and dispatch messages with the first example server code I shared (the on-upgrade event), but when I attempt it using the second version (the express-ws route) it results in the following in-browser error every second...
WebSocket connection to 'ws://localhost:8080/mqtt' failed: Invalid frame header
Any suggestions for how to properly replicate the proxy behaviour when embedded inside an express-ws route?

Anyone solved this?
I can't figure out how to implement socket upgrade with express-ws library. I have website behind ngingx reverse proxy, which takes care of secure trafic. But my backend app is unsecured. I'm using this atm:
app.ws('/ws', function (ws, req) {
  console.log('Client connected');
  ws.on('upgrade', (request, socket, head) => {
    console.log('Upgrade connection');
    ws.handleUpgrade(request, socket, head, (websocket) => {
      console.log('Connection upgraded');
      ws.emit("connection", websocket, request);
    });
  });

  ws.on('close', function close() {
    console.log('Client disconnected...');
  });
});

I never see upgrade triggered :(


Is it possible to dynamically show items?

The main idea is to show number of items in row depending on it's sizes in @media.
Here you can see what am I talking about http://www.owlgraphic.com/owlcarousel/demos/custom.html
Any ideas are highly appreciated because I love fotorama so much that don't want to use anything else.

We are dynamically showing items. check this out and let me know if it is what you are talking about...
We do not list the images in the html, we are pulling data from a database and using django code to pull them out of the db, and putting them plus the data we need from the database into the html.
http://www.arkansasonline.com/galleries (select a gallery when you get there)


Fix auto login

Fix the automatic enter of the only oauth providers, which makes it unable to logout

@dokterbob I did a new PR from the good branch this time 😉

I could not help myself from digging a little deeper and it seems that OAuth2 has a prompt=consent option to the auth request. This way, a user is not logged out from their OAuth provider (we don't want users to log out from GH if they use GH to login to Chainlit!), but it will require users to confirm their login (and allow them to change to a different user).
To implement this, we could add prompt=consent on all outgoing auth requests -- which AFAIK (we need to test this) only happens when a user as logged out, requiring chainlit to send the request in the first place.
That seems like a much simpler and elegant solution!

Hello, this method is interesting, I'm going to dig that.

@dokterbob I restored the previous behaviour and added prompt=consent to the authorization params for each one of the 9 chainlit external providers, as refered here

here

Great! Could you please indicate to what extent you manually tested it? It's essential for this we do extensive manual testing!

Sure, I only tested it on github oauth, but since it is defined in the OpenID connect core 1.0 spec, it should be sure that it work with the other one. If you want to test them, it would be great, I personnaly do not have the possibility to do so...

hello @dokterbob, for now we are just going to follow your temporary solution, prompt=login everywhere except for GH. Can you commit that or do you want me to do so ? Dan wants me to close this issue ASAP


Non-existent domain

Hi, I've just tried this docker image and it works good, except one issue:
nslookup clien.net says it's Non-existent domain.
This is unacceptable, although I like the ad-block option of the docker image, because clien.net is one of the Web-sites that I visit daily.
Thanks in advance.

Try setting it with the UNBLOCK environment variable? it's probably blocked due to the block lists. Let me know if that doesn't work.

Wow, that solved the problem. Thanks!
But what if the site is really attacked? Trust my web browser and firewall?

Oh it just came from one of the block lists my program gathers (qdm12/updated).
I have no clue why but that's usually because some malicious activity was detected from that website.
If you've blocked ads too, then it's likely because it got classified as a source of ads.
You can dig through the files in qdm12/files (another repository) if you are curious, but I also happily live with a bunch of domains in UNBLOCK without knowing the exact reason of their blocking 😄


"cannot access AbstractSafeParcelable"

Hey there,
When building the app with "ionic build android" i get the error above...

Platform: Android
Device manufacturer and model: Nexus 5x
Cordova version (cordova -v): 6.5.0
Cordova platform version (cordova platform ls):  android 6.1.2

Context
let config = {
desiredAccuracy: 0,
stationaryRadius: 20,
distanceFilter: 10,
debug: true,
interval: 2000
};
BackgroundGeolocation.configure((location) => {

  console.log('BackgroundGeolocation:  ' + location.latitude + ',' + location.longitude);

  // Run update inside of Angular's zone
  this.zone.run(() => {
    this.lat = location.latitude;
    this.lng = location.longitude;
  });

}, (err) => {

  console.log(err);

}, config);

// Turn ON the background-geolocation system.
BackgroundGeolocation.start();

I had the same problem. Just update your android platform project.properties
from
cordova.system.library.3=com.google.android.gms:play-services-location:9.8.0
to
cordova.system.library.3=com.google.android.gms:play-services-location:+
Anyone knows hoy can i define anywhere before the platform generation? i need to automate this.
Thank you very much

Notice: this issue has been closed because it has been reported for branch 2.x or later. It could be already fixed in v3 or is not relevant anymore. You may reopen this issue if it has been closed in error.


Explicitly use 64bit integers when converting from BQ types

On 64-bit windows 10 the int values are cast into 32-bit integers for reasons given here
https://stackoverflow.com/questions/36278590/numpy-array-dtype-is-coming-as-int32-by-default-in-a-windows-10-64-bit-machine
This leads to unhanded exceptions when handling values beyond 32-bit signed range
In Bigquery, int is always 64bit, so the solution is to specify the data type explicitly

This is to solve the issue #119

Codecov Report

Merging #121 into master will decrease coverage by 44.71%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           master     #121       +/-   ##
===========================================
- Coverage    75.7%   30.99%   -44.72%
===========================================
  Files           8        8
  Lines        1626     1626
===========================================
- Hits         1231      504      -727
- Misses        395     1122      +727

Impacted Files
Coverage Δ

pandas_gbq/gbq.py
20.56% <0%> (-55.8%)
:arrow_down:

pandas_gbq/tests/test_gbq.py
26.88% <0%> (-56.76%)
:arrow_down:

pandas_gbq/_load.py
62.5% <0%> (-35%)
:arrow_down:

Continue to review full report at Codecov.

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

I think that looks good! Thanks!
I don't think there's a helpful test we could add, given we don't test on Windows - @tswast ?


Reverse a queue using stack || ds || C

Have you read the Contributing Guidelines on Pull Requests?
yes
Description
fixes #6138
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
issue #6138

@HarshCasper  @ankitaggarwal23  can you please review my pr

@ankitaggarwal23  I have made the changes in the code . can you please review it


chore: Upgrade go-jinja2

Description
This upgrades go-jinja2 to the latest commit, which uses a go-embed-python version that can be used by Nix maintainers to patch the python binaries properly before packaging.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

@netthier fyi. When this is released, you should be good to go.


feat: allow factory to destroy forwarders

Guys, take a look please at this commit e982984
cc @birchmd @raventid


Responsive design

The responsiveness of some pages needs some work.
Catalogue browsing pages
The "browse store" dropdown doesn't seem very touch-screen friendly (as it uses hover effects)

The rendering of a product could do with being improved. Seems like some things need centre justification.

Basket page
The basket line rendering is a little odd in narrow viewports

closing all issues that are feature requests that don't have open PR's attached.


Corrected small typo in the connection string

I think the suffix of the host should be mgmt and not man as the shown mesos example clarifies.

Close/reopen to get CLA validation to run.

Hi @ciberado, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@rgardler - Ross, could you review this update in the public repo from @ciberado?

#sign-off

@ciberado - I'll merge via the private repo - thanks for the fix.


JavaScript mixin properties not checked

TypeScript Version:  master (as of #26283)
Code
/**
 * @template {new (...args: any[]) => any} T
 * @param {T} SuperClass
 */
function myMixin(SuperClass) {
    return class extends SuperClass {
        method() { return 0; }
    }
};

class Base {}
const Mixed = myMixin(Base);

const instance = new Mixed();
instance.method(); // Correctly typed in signature help
instance.meshod(); // But not checked

Run with allowJs, checkJs, and strict.
Expected behavior:
Error at meshod.
Actual behavior:
No error.

This is true of Typescript as well:
function mix<T extends new (...args: any[]) => any>(S: T) {
    return class extends S {
        f() { return 0; }
    }
};

class Base {}
const Mixed = mix(Base);

export const m = new Mixed();
m.f(); // Correctly typed in signature help
m.j(); // should error here, but doesn't

Looks like it has to do with the type T extends returning any -- though I think the actual T ought to (but isn't) used instead of its constraint when outside of mix?

Well, the reason that you see the apparent type of T is that the any comes from class extends S. S has type T, whose apparent type has a constructor with return type any, which is equivalent to extending any. That’s what allows you to access any property on m.

I think the problem then is that we force the user to write any in that position rather than unknown, which is an error (Base constructor return type 'unknown' is not a class or interface type.)

I though about using InstanceType in the return, that actually makes sense, although the code does throw when expected, it also throws extending S
function mix<T extends new (...args: any[]) => InstanceType<T>>(S: T) {
    return class extends S { // Throws: Base constructor return type 'InstanceType<T>' is not a class or interface type.
        f() { return 0; }
    }
};

class Base {}
const Mixed = mix(Base);

export const m = new Mixed();
m.f(); // Correctly typed in signature help
m.j(); // Throws: Property 'j' does not exist on type 'mix<typeof Base>.(Anonymous class) & Base'.

Maybe I'm using InstanceType poorly, but I think this should be the way to go.


Uncaught TypeError: Cannot read property 'componentDidLeave' of undefined

I found this problem when I tried to use EasyTransition with a custom Loading component.
The component mount and unmount a children component depending of a flag.
When the component is unmount it fires this error:
Uncaught TypeError: Cannot read property 'componentDidLeave' of undefined

Here is the example:
import React, { Component } from 'react';
import { Panel } from 'react-bootstrap';
import EasyTransition from 'react-easy-transition';
import WLoading from '../widgets/WLoading';

export class WPanel extends Component {

	render() {
		const properties = {
			path: location.pathname,
			initialStyle: { opacity: 0 },
			transition: 'all 0.3s ease-in',
			finalStyle: { opacity: 1 }
		};

		const { children, loading, title } = this.props;

		return (
			<EasyTransition {...properties}>
				<Panel header={title} bsStyle="success">
					{loading ? <WLoading /> : children}
				</Panel>
			</EasyTransition>
		);
	}
}

export default WPanel;

The value that controls the loading / unloading of the children should be the "path" prop. Your flag should be fed into the "path" prop of EasyTransition component.

Actually my loading flag is a boolean that changes from false when the app is requesting data to the api, to true when the data is fetched correctly.
The path property only accepts string values, but converting loading to a string makes that the animation/transition runs twice, when loading changes its value.
By setting the path prop with any string, this solves my problem, for now.
const properties = {
    path: 'foo',
    ...
};

thanks for providing a solution. not sure if a default value for path in the component could solve that for all users


Added min and max validation to dateTime directive.

Note: only validation was added. The user can still choose an invalid date from the picker.
This was tested on chrome.

the file you want to change is this https://github.com/g00fy-/angular-datepicker/blob/master/app/scripts/datePicker.js the files in the dist/ folder are files that are being compiled by grunt task's. Make your changes on the app/ folder files build with grunt and send the PR.
Thanks.

Ok, I'll do that as soon as I make peace with my node installation :D

@eralha I opened #103 with the things you've requested.


attribute "resp" in critical apparatus

In the file https://dharman.in/texts/DHARMA_CritEdBhuvanakosa, strings such as
<l n="c"><app><lem type="emn" resp="Sanderson">daśāṇām api nāḍīnām</lem><rdg wit="#La #Ba #Bb">darśśaṇam api nāpīnāṁ</rdg></app>|</l>
produce the error
attribute "resp" not allowed here; expected attribute "cert", "hand", "rend", "varSeq", "wit" or "xml:id"
The encoding guide for critical editions does not contain instruction on how to encode the specific case of attributing responsibility for an emendation.
A way to encode the attribution of responsibility of an emendation or conjecture in the critical apparatus needs to be implemented, or some changes have to be made in the EGC and in the code for file validation and for html display.

@michaelnmmeyer — have you proposed a change in the EGC? if not, this issue should remain open, shouldn't it?

@arlogriffiths Noted.
Looking at the EGC, it turns out we had an ongoing discussion about the addition of @resp and @source to <lem>, which I forgot about. You proposed there a slightly different use of @resp.
I suggest we add both @resp and @source, but restrict their use somewhat, as follows:

Iff lem/@type is one of "emn" or "conj", an additional @resp or @source attribute might be supplied to indicate the emendation/conjecture's origin.
@resp and @source shall not be used at the same time.
@resp shall be used to indicate a person's name (in free-text form, or with the form part:X to indicate a member of the project), and only for this purpose.
@source shall be used to indicate a bibliographic reference (of the form bib:X), and only for this purpose.

I'm not sure what to do with @source, further input from you and @danbalogh would be useful. Are page numbers, etc. (as in em. Sanderson XXXX p. YY) useful here? How should we display the information?
Best,
Michaël

Hello, I'm not familiar with the details and the preceding discussion, so sorry if I'm missing something obvious. I basically agree with what Michael is proposing here, with the following observations:

I think @resp should only be used with part:X and not with free text, and should be used in lemmas (or readings) only to indicate project members. I'm not even sure whether free text is permitted in this attribute; according to the TEI documentation, its value should be of the teidata.pointer type. But even if free text is technically OK, I am averse to doing that. Continuing this, I'm not sure we even need this. Readings and emendations by the DHARMA editor (the author of the XML document) should not need to be indicated explicitly. Still, it may be good to have the facility to indicate input from other DHARMA members in this way.
@source should be used with bib:X IDs to point to publications in the same way as done in the EGD (§9.1.3, 9.1.4). I definitely think adding page numbers or any additional details here would be unnecessary.
I'm not sure if @arlogriffiths or anyone else anticipates a need to credit unpublished suggestions (personal communication) in the markup. I suggest, as above, that suggestions by other DHARMA members could be handled through @resp. Suggestions by outsiders should not, in my opinion, be encoded, but rather mentioned in a note to that apparatus entry (assuming that the EGC allows apparatus notes). Perhaps, if you feel the need to be more rigorous, we could create a part:external or suchlike (if one doesn't already exist), and use that as a value of @resp in addition to the note saying "personal communication with Alexis Sanderson, 26 June 2023" etc.
if in a particular case, a page number reference is necessary, then that could likewise be added informally in a note, in addition to the source attribute. We definitely don't need machine actionable page numbers on apparatus entries.

@michaelnmmeyer —

which section of the EGC are you alluding to for that discussion and my slightly different proposal?
Without having seen again what I had proposed, I should say that your proposal above suits me well, but it needs to be modified because Dan is right: @resp cannot take free text but values need to be defined in the teiHeader and pointed to with #. See [https://www.tei-c.org/release/doc/tei-p5-doc/en/html/ref-att.global.responsibility.html](https://www.tei-c.org/release/doc/tei-p5-doc/en/html/ref-att.global.responsibility.html.
Could we follow the TEI Guidelines and make it mandatory for a <respStmt> to be present for @resp to refer to? But see my reaction to Dan's alternative suggestion below.
Provided that a unique xml:id "Sanderson" can be defined somewhere in our system without risk of conflicting definitions, Andrea's example thus has to become <l n="c"><app><lem type="emn" resp="#Sanderson">daśāṇām api nāḍīnām</lem><rdg wit="#La #Ba #Bb">darśśaṇam api nāpīnāṁ</rdg></app>|</l> and the teiHeader needs consequently to be modified.
But if we adopt Dan's suggestion, then it would have to be <l n="c"><app><lem type="emn" resp="part-ext:Sanderson">daśāṇām api nāḍīnām</lem><rdg wit="#La #Ba #Bb">darśśaṇam api nāpīnāṁ</rdg></app>|</l>
Once this discussion is over, can you make the necessary suggestions for modifying the gdoc for EGC? It will need to be modified in more than one place.
Whether we make our @resp refer to an external authority list or to an xml:id defined in the xml file itself, the desired display is the value of <surname> (or, in the absence of the element, of <name>) of the relevant person in parentheses after the display of the indication "em." / "conj." / "norm." corresponding to the value of @type, with a space between the two.
Andrea's example should thus be roughly like this: daśāṇām api nāḍīnām] em. (Sanderson), darśśaṇam api nāpīnāṁ La Ba Bb.
Please let us know when this is implemented for html display (and conversion to LaTeX).

@danbalogh —

Thanks for your valuable input!
I think we need both: the encoding of the style @resp="#Sanderson" for people whose input often needs to be referred to, and encoding with <note> in <app> (which is certainly allowed by EGC) for cases where more details needs to be furnished (a date, a page reference).
I am not sure what the pros/cons of your suggestion to introduce a special prefix part-ext: or something like that
would be. One pro is presumbly the avoidance of the problem of conflicting definitions. But if that problem can be avoided otherwise, then referring to an entry in <respStmt> may be nicer as it makes it possible to describe the relevant person's contribution in the specific edition in question, in a manner that your alternative of referring to the file DHARMA_idListMembers_v01.xml with a prefix does not. Or perhaps both methods can be combined?
If you have time, your careful reading of and commenting on the entire EGC in gdoc will be extremely welcome.

@aso2101 — any feedback?

A quick clarification first. My suggestion was not part-ext: (part-ext:Sanderson etc.), but part:ext (or whatever) as an umbrella for anything and everything that is neither a Dharma project member nor a referable publication. It would always require clarification in a note, but would dispense with the need to declare e.g. #Sanderson in the header. It could be displayed as e.g. daśāṇām api nāḍīnām] em. (external) or daśāṇām api nāḍīnām] em. (see note).

I don't have much to add except that I'm also OK with declaring frequently cited outsider contributors in the respStmt of the XML document.

@arlogriffiths I'm referring to the notes under EGC §5.3.2.
Concerning @resp, if I understand well, we would have:

@resp shall be used to indicate a person's name, and only for this purpose. The value of @resp can be of two forms: either part:X, to indicate a member of the project, or #X, where X is the @xml:id of a <respStmt> that gives the person name and that is declared in the same file.

Is that OK ?
Concerning the HTML and LaTeX display, I need to write a proper parser, which will take a while.

This is, I believe, still an unresolved issue that I am therefore reopening once again. See screenshots:


WRP-27429 : Update to apply google analytics 4 to sampler

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
Need to apply ga4(google analytics 4) and cookie banner to enactjs.com/sampler
Resolution
Added process of applying ga4 and inserting cookie banner when building sampler in storybook
Additional Considerations
Links
WRP-27429
Comments
Enact-DCO-1.0-Signed-off-by: SJ RO<EMAIL_ADDRESS>

Checked codecov and Travis. There's no problem, only github page issue.


Participants Unable to join

Type: Bug
Open VsCode
Start a live share session
Ask participant to join
They recieve the error "Joining collaboration session: Unable to connect to the host. Verify both you and the host are online.

I am logged in and I have tried this with 4 different users. I have uninstalled vs code and restarted my computer. I have turned off my VPN.

Extension version: 1.0.5733
VS Code version: Code 1.71.2 (Universal) (74b1f979648cc44d385a2286793c226e611f59e7, 2022-09-14T21:05:37.721Z)
OS version: Darwin x64 21.6.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-1038NG7 CPU @ 2.00GHz (8 x 2000)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
4, 4, 4

Memory (System)
16.00GB (0.03GB free)

Process Argv
--crash-reporter-id 7be7d533-2a03-4467-a746-c0e1b7224bce

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
i497e931:30553904
pyindex848:30577860
fc301958:30573243

Myself and another member of my team is running into this issue as well.

Myself and another member of my team are running into this issue as well. It previously worked until yesterday.

@JoseyKinnaman we were able to "resolve" the issue on our side. It turns out it was related to proxy settings. Not sure why this issue is new but once I changed these settings:
"http.proxyStrictSSL": false,  "http.proxySupport": "off",
It partially fixed the issue.

Apologies for the delay here. As is, this issue doesn’t have enough information to be reproduced. Please reopen if the issue persists with any additional details and after reviewing https://learn.microsoft.com/en-us/visualstudio/liveshare/reference/connectivity. Thank you.


proposed updates to new router, re: issue #125

captures the bug fix and terminology suggestion from my comment: https://github.com/Day8/re-frame/issues/125#issuecomment-153569317

Thanks!


Export the list of documents

It would be great if we can export the list of documents pulled by this tool

Thanks. Will add to backlog for future release


LogoutRequest wrong nameId?

Hi there,
I'm new to saml so bar with me ;)
<samlp:LogoutRequest xmlns:samlp="urn:oasis:names:tc:SAML:2.0:protocol" xmlns:saml="urn:oasis:names:tc:SAML:2.0:assertion" ID="ONELOGIN_1eba4c60-5f35-49a2-98cf-d29e64f295c3" Version="2.0" IssueInstant="2017-09-20T15:20:49Z" Destination="https://idp.ssocircle.com:443/sso/IDPSloRedirect/metaAlias/publicidp" >

<saml:Issuer>superFancySp2</saml:Issuer>

<saml:NameID Format="urn:oasis:names:tc:SAML:2.0:nameid-format:entity" SPNameQualifier="superFancySp2">https://idp.ssocircle.com</saml:NameID>

</samlp:LogoutRequest>

Isn't the value of saml:NameID wrong here? Shouldn't it be something sp-related?

You are sending a LogoutRequest to the IdP SLS endpoint https://idp.ssocircle.com:443/sso/IDPSloRedirect/metaAlias/publicidp
To the logout method, you can provide nameId and nameidFormat values (that you may extract from the previous SAMLResponse sent by the IdP).
If you don't provide those values, then a saml:NameID of the format entity will be set (as the one you shared).

@pitbulk than I would say the sample application is some kinda broken because the
<saml:NameID Format="urn:oasis:names:tc:SAML:2.0:nameid-format:entity" SPNameQualifier="superFancySp2">https://idp.ssocircle.com</saml:NameID>

always results from when a new LoginResponse is constrcted, or am I missing something?

The "sample app" is just an example, you are free to modify it with your needs.
If you want to provide a PR extending it you are welcome, you may need to store on session the nameid format and retrieve on the do_logout view the nameid and nameid_format from the session and provide them to the logout method.


How do I make nginx-proxy redirect to www. and enforce https/ssl?

Not a single hint?

I think you are more likely to get a response if you include something in the body :)
There is a section on enforcing HTTPS/SSL in the README - nginx-proxy does this by default by forwarding the user to HTTPS and sending them an HSTS response that prevents the browser from even trying HTTP again for a year.
I'll assume you are running another nginx container with VIRTUAL_HOST=www.foo.com and you want to redirect the foo.com requests to it.  In this scenario, you should use VIRTUAL_HOST=www.foo.com,foo.com on that container, then add something like this to your foo.com site's nginx container:
server {
    listen 80;
    server_name foo.com;
    return 308 https://www.foo.com$request_uri;
}

Now requests for foo.com will go through nginx-proxy to the foo.com nginx container, which will send them back and HTTP 308 Permanent Redirect with Location: https://www.foo.com/<original_path>; then the browser will make a second request to the correct location.

Thanks, actually thats what I did eventually, but on the nginx-proxy, by including a vhost file. However 1. I think this should be done upfront at the proxy, so that the request is redirected right there and does not hast to go all the way through ...
2. I since using www. is good partice, I think this should be a simple option of the nginx-proxy.
Thanks though

Redirecting to www is definitely a common thing - I do it as well.  Maybe there is a graceful way for us to do it in nginx-proxy,  I'll need to think about it.

For the https rewrite from example.com to www.example.com, here I how I got it done:

in my service container docker-compose.yml:
environment:
VIRTUAL_HOST: www.example.com
LETSENCRYPT_HOST: example.com,www.example.com
Note, I intentionally removed example.com from the VIRTUAL_HOST because I don't want to have the server example.com block to be generated from the template. We still need the example.com TLS certificate though in the LETSENCRYPT_HOST.

Put an extra.conf in /etc/nginx/conf.d on the nginx container so it can redirect:
server {
listen 80;
server_name example.com;
return 308 https://www.example.com$request_uri;
}

server {
listen 443 ssl;
server_name example.com;
ssl_certificate /etc/nginx/certs/example.com.crt;
ssl_certificate_key /etc/nginx/certs/example.com.key;
ssl_dhparam /etc/nginx/certs/example.com.dhparam.pem;
return 308 https://www.example.com$request_uri;
}
It is working, however, this is still a workaround as I have domain hardcoding in the extra.conf.
With this workaround, all the 3 below requests ended up on https://www.example.com:
http://www.example.com
http://example.com
https://example.com

@xczheng is this config still working for you? Or you found something better?
With the current versions of nginx-proxy and docker-letsencrypt-nginx-proxy-companion, I'm doing something similar but the le-companion clashes with the custom configuration: https://github.com/JrCs/docker-letsencrypt-nginx-proxy-companion/issues/521
I've been messing a long time with this. User @witten suggests to use a separate redirect container. Seems like overkill, but at least it works. https://hub.docker.com/r/morbz/docker-web-redirect/

I am running nginx-proxy for internal services. I wanted to redirect all short names to their corresponding fully qualified names. e.g. http://git/ -> https://git.internal.example.com/
I borrowed from @xczheng's idea, but instead of adding a new config file I modified the server_name '_'; section of the nginx.tmpl file:
--- a/nginx.tmpl
+++ b/nginx.tmpl
@@ -96,7 +96,9 @@ server {
        listen [::]:80;
        {{ end }}
        access_log /var/log/nginx/access.log vhost;
-       return 503;
+
+        # short name redirection.
+        return 308 https://$host.internal.example.com$request_uri;

Then I removed all the short names from the docker-compose.yml VIRTUAL_HOST.

Thanks @xczheng your instructions worked perfect for my nonwww to www redirect, and the rest of my subdomains route as expected too.

I'm very confused about how this works across containers:
location /old/URL {
rewrite ^.* https://newURL.need.to/visit permanent;
return 307;
}
The wrong container is answering. When /old/URL is visited, it is via the domain need.to/old/URL, so the nginx-proxy talks to need.to's container, but the newURL.need.to needs to be the container nginx-proxy talks to... This nginx config was placed in the correct vhost.d/VIRTUAL_HOST filename. Do I need to use internal URLs or some docker-magic?
Thanks a lot... Stuck!

OK, I figured it out using the following:
location /old/URL {
proxy_pass http://newURL.need.to/visit;
proxy_set_header host http://newURL.need.to/visit;
}
However... I also needed to use "HTTPS_METHOD=noredirect" in the container's env... And this has ended up causing other issues... Is there no way in this configuration to have no SSL redirect for a single regex / specific URL? This one proxy_pass has caused the website to required the noredirect setting for its entire domain!
Thanks again for your help!

This seems like the solution:
https://docs.nginx.com/nginx/admin-guide/security-controls/securing-http-traffic-upstream/
Anyone know which .pems to specify, though? And which files in nginx-proxy's certs mount to use as the .key file (as specify in the above nginx doc)?

I've consolidated & clarified my question here: https://github.com/jwilder/nginx-proxy/issues/1340

This is how I'm doing it for now; not wanting to add configuration to the nginx-proxy and keeping VIRTUAL_HOST, LETSENCRYPT_HOST as-is.
VIRTUAL_HOST: example.com,www.example.com
LETSENCRYPT_HOST: example.com,www.example.com

In the nginx container below the reverse proxy define one of the below or a variant:
non-www -> www
server {
  server_name "~^(?!www\.)(.*)$" ;
  return 301 $scheme://www.$1$request_uri ;
}

www -> non-www
server {
  server_name "~^www\.(.*)$" ;
  return 301 $scheme://$1$request_uri ;
}

Hopefully this should work.

@propheth were you able to get it working? I've been attempting to solve this issue without success

@dannycarrera yes. Did you try the above? Just add one of the server block to your nginx container configuration.

@propheth great to hear that it works. I'm using https://github.com/evertramos/docker-compose-letsencrypt-nginx-proxy-companion which seems to be adding some more complexity. I will try again tomorrow from scratch. Thanks!

@propheth I think your solution only works when redirecting from an http site. According to nginx docs, when no listen directive is provided within a server block, it defaults to listen on 80. I would ensure it's working for you with browser caching disabled.
Also, I submitted #1369 to address this issue. If you want to take a look and use it before it gets merged, all you need is the updated nginx.tmpl. See #1369 for usage details.
Cheers, hope it helps!

@dannycarrera the solution should work for both.
The standard jrcs/letsencrypt-nginx-proxy-companion handles the redirection to https on 443 and by default communicates upstream via http 80 unless you set some other env vars.
Before applying the solution, your setup should already be redirecting from http://example.com -> https://example.com and http://www.example.com -> https://www.example.com.
The upstream nginx will check for a non-www and do a redirect to the www version whatever the schema/protocol be and vice-versa.

@propheth so it seems you are you using 2 nginx containers? I think that's the part I missed from your solution.
With your approach you'll still run into the double redirect issue (#1204).


Slave host auto-initiate stalls if master does not support extended reject indications, and procedure collision occurs

Problem
If an LLCP collision occurs during auto-initiate, e.g. with PHY_UPDATE, and master sends a REJECT_IND, auto-initatiate is not continued, as no notification is sent to host from LL.
Impact
Other host- or app initiated LLCP requests are never sent.
Additional context
In function ctrl_rx (ull_conn.c), PDU_DATA_LLCTRL_TYPE_REJECT_IND is not handled properly. The case needs to handle the packet similar to reject_ind_phy_upd_recv in reject_ext_ind_recv.
NOTE: This was found on the 2.2.1 branch, but as far as I can tell, it hasn't been fixed on master.

@mtpr-ot will a PR be upstream soon? I will be available to review/test the same.

@mtpr-ot will a PR be upstream soon? I will be available to review/test the same.

I'm not sure I can provide the full functionality - I have only fixed locally for PHY_UPDATE. Perhaps I can have Thomas (@thoh-ot) make the full implementation.

The following patch implements REJECT_IND handling for PHY collision only:
phy_reject_ind_fix.zip


32-bit release?

Is it possible to make a 32-bit release?

@shadiakiki1986 we do not have any 32-bit Linux machines in our build farm, but we have made building Teleconsole from source significantly easier.
See this:
https://github.com/gravitational/teleconsole/blob/master/BUILDING.md


Automatic export when source file changes

Hello.
There are many small things to improve in this wonderful program...

Output in G21 (mm)
Use . as decimal separator instead of ,
offset output so that lowest left corner is at 0,0
set default S value
stop putting "M62 P0" - bCNC does not like it..

So I made a C# program that reads SVG-to-GCode generated file and corrects all these little things, so I would no bother you!
However, there is one thing I can´t do it with my program...
A checkbox that enables "Automatic Convert"
My C# program has one! Here´s how it works:
When the checkbox is checked, my C# program will

read the source file´s "last written date"
start a 1second timer

When the checkbox is unchecked, my C# program will stop the timer.
At the timer function, it reads the source file´s "last written date" and compares.
If the file has changed, it rebuilds.
This helps to make a workflow:
When I save a file in Inkscape...
SVG-to-GCode will notice file has changed and automatically reconvert.
My C# program will notice file has changed, "fix" it, and save to another file.
bCNC will notice file has changed and prompt to read again (it does!)
A few last words:
I did a timer because I´m lazy. I should use a notification... shame on me!
In the Export dialog (where there is the "Export Now" button) there could be an option of Rotation (0, 90, 180, 270) so that rotation will also be done automatically, and not because user pressed the "Rotate 90" button.
It would also be nice to have Xmirror and Ymirror..
Thank you

Hi, and thanks for your interest!
I don't want to add an 'automatic export when source file changes' because it would lose any changes you made in SVG to GCode, such as scaling, tiling, removing sections, etc.
I also don't really use this application anymore and currently don't have the time to maintain it - feel free to make the changes needed!
At some point I may convert this to a C# application, but that is a lot of work.
Cheers!


Make test case recorder

Goal: to make it super lightweight to make a new test case so that you don’t think twice about it, while keeping them robust enough not to rot if we rework internals

New command cursorless.recordTestCase, which asks for a test case name / maybe directory

Waits for next cursorless command

On next cursorless command, takes a snapshot of current file, including cursor locations, visible lines, file type, and relevant decorations

Takes snapshot after cursorless command, including cursor and visible lines

Saves before, command, context, and after as a test case

Then add code that can parse this type of fixture and run it as a test

I'd argue that cursorless command should be captured pre-inference, but don't feel strongly

This technique can be used for testing transformations, inside / outside, actions, etc

For example, to check whether the delimiter / inside outside stuff is working for eg "type" transformation, can run the command with "take" and "Chuck". This makes it robust to changes in internal representations, at the expense of being slightly less granular

Also save highlights to make sure those don't break?

could then also use these test cases to automatically record GIFs to add to the docs. Could do this either by having talon send the full command text and save it with the test case, or build something to generate talon command from json

For the GIF generation, we could easily automatically find all test cases that use a particular action to put into the doc for that action.  Same with transformations, etc

Ideas for making a good GIF:

making text larger,
making command text show a good delay before action,
having one command per GIF so you can see the same one over and over,
slowing down highlight
put something in the gif where you can clearly see when the loop point is (thanks @timo!).  Can possibly use https://github.com/nwtgck/gif-progress
have 1 sec delay at start of gif (thanks @timo)
use longer decorationSleepTimeout than usual
somehow call out the decoration that is referenced in the command, possibly by making it bigger?  Could either do this from the start of GIF, or right after they say the command before it executes

[ ] let's use this one to capture https://github.com/pokey/cursorless-vscode/pull/32#issuecomment-872564310 as a test case

Extra credit is:

[ ] capture clipboard before and after
[ ] capture visible lines before and after
[ ] test ability to work across splits
[ ] capture highlights

Note that we'll probably want to finish #69 before we record any test cases as that will change the wire format from talon

Use this to test:

extracting name from exported classes and functions in Typescript
extracting name from decorated classes and functions in Python

To test:

Delimiters get removed from source of "move" and "cut"
"that" gets set properly
Behaviour of "wrap" wrt current selection:

Target selected
Cursor at beginning of target
Cursor at end of target
Cursor bigger than target tot left / right / both
Multiple cursors? (Eg one cursor at end and one at beginning, etc)

space delimited sequence support when combined with punctuation, eg for "This is, a test**!**":

"chuck sit"
"move sit after each"
"move air after each"
"move each before air"
"bring air before bang"

[ ] Make sure leading delimiter with "chuck type" gets deleted

[ ] Test #12 and various overlapping ranges

To test

Excluding positions in ranges. take air until bat take air tween bat

More to add:

[ ] "move sit to before file"
[ ] "chuck last char air"

[ ] Make sure basic things work in unsupported languages

[ ] Multi-target inference rules, eg "swap funk air with bat", "swap air with funk bat", etc

[ ] select a token, then wrap it and make sure selection in right place

[ ] "take token" / "take first word" when cursor is at beginning, middle and end of token


What XML source do we currently use - what XML source should we use, or not?

@Madoshakalaka @aradu12 @eddieantonio This is a revival of #165: Looking at the current new itwêwina content, we appear to be using giella/art/dicts/crk/engcrk_cw_md.xml, since I am seeing MD prefixes as dictionary entries that have not been orthographically standardized and which have not been matched with CW dictionary entries. The only source XML which contains those MD dictionary entries is engcrk_cw_md.xml. The problem with that source is that it does not contain e.g. stems (and inflectional categories), and due to indexing English-to-Cree contains the Cree headwords multiple times under each English index word. Also, the English definitions have been truncated for verbs, to start with an infinitive.
In order to read in the Cree dictionary entries, it would seem to me that giella/art/dicts/crk/crkeng_cw_md.xml contains all the content that we need, i.e. the stems. Of course, implementing English-to-Cree indexing would require some processing, but due to the morphological simplicity of English that should be relatively straight-forward to implement - I think.
The most important thing to note of either Cree-to-English or English-to-Cree XML sources used by the old itwêwina (and the new) is that they are by no means anywhere close to perfect representations of the aggregation of the dictionary source. The XML sources are representations of what could achieved by AWK scripting from the original sources, and maybe be missing some information and is difficult to impossible to reconstruct from the XML alone.
In this respect, on the longer term, we would want to operate directly from the CW Toolbox and MD TSV sources, since those sources are being constantly updated. Also, in order to keep properly track of changes to both sources, using a relational database would seem to be the best practical solution. When working with an aggregate dictionary, the additional task that needs to be done is the harmonization and integration of their content in terms of orthography, linguistic information (word-classes, inflectional categories and stems), and (English) translations. For CW-v.2014 and MD-v.2014, we already have an assessment by Katie in the gloss comparison file, that can be used for matching dictionary entries from  both sources, as they were several years ago. CW entries that do not have a match with MD should generally be fine as they are, though AEW is continuing to create new dictionary entries. However, there are a non-insignificant number of MD entries that did not match with CW, which need to be orthographically standardized and provided with word-class and inflectional category information. Also, we need to scrutinize every new dictionary entry in CW as to whether it matches with an MD entry.
In my view, we should not put any more effort into working with the last XML sources than is absolutely necessary for making new itwêwina work as close to specifications as possible. One should not put in much effort in trying to automatically do comparisons that in then end would best be done by a linguist, and if those are done properly once with a proper lexical database, need never to be done again by a linguist.
In this respect, the main thrust of our energies should be put into implementing the specifications described in the following GDoc our Tech team has access to:
https://docs.google.com/document/d/1uBJtG8WxRbUIBSdeNBynksKQoRImXJoCbEVFcW9HIJw/edit
Importantly, the efficient implementation of such a lexical database relies on a proper understanding of the appropriate dictionary data model that we have been discussing over the last weeks.

The immediate solution to this is in issue #258, when still working with XML format source. The longer term solution to this is the development of the Dictionary Database, which now has its own project: https://github.com/UAlbertaALTLab/dictionary-database-model


Can't select a color after creating a new Neo block

Describe the bug
We have a Neo field with a Button Box - Colours field inside. On an entry, when a new block is created, the dropdown of the Colours field does not updated the selected value when clicked. If we save the entry first, we can then update the field value.
Steps to reproduce

Add Button Box - Colours field to a Neo field
Add Neo block with the Colour field in the Entry edit screen
Try to change a Colour

Craft CMS version
<IP_ADDRESS>
Plugin version
4.2.3
Multi-site?
No
Additional context
No response

Thank you for the quick fix!

Fixed in 4.2.4


add support for OpenAPI spec v3 annotation

Hello, great work with the Swagger generator
So the appointer/swaggervel is archived by the owner. It is now read-only.
I opted to use https://github.com/DarkaOnLine/L5-Swagger as the generator ,
Now "darkaonline/l5-swagger" latest version that is 8 is compatible with OpenAPI Spec  version 3 only,
This package has only "@SWG (SWAGGER annotations) support." stubs it would be nice to have OpenAPI spec v3 annotation (@OA)

Hello  @mitulgolakiya 👀

Hi, we are already working on this and almost done. We will be pushing Open API Specification support by next week probably.

Hi, we are already working on this and almost done. We will be pushing Open API Specification support by next week probably.

Thanks in advance, really looking forward to it.

Thanks in advance, really looking forward to it.


no selection char count on bottom ~ yes, the 'Editor Selection' is checked

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

VS Code Version:
OS Version:

Version: 1.76.2 (Universal)
Commit: ee2b180d582a7f601fa6ecfdad8d9fd269ab1884
Date: 2023-03-14T17:54:09.061Z (1 wk ago)
Electron: 19.1.11
Chromium: 102.0.5005.196
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.3.0
Sandboxed: Yes
Steps to Reproduce:

no selection char count on bottom
yes, the 'Editor Selection' is checked

What do you mean editor selection is checked?


Unable to page.setRequestInterception(true) if connecting to chrome

For the browser instance puppeteer.launch it can do somethings like
await page.setRequestInterception(true);
page.on('request', request => {
    if (request.resourceType() === 'image') {
        request.abort();
    }
    else {
        request.continue();
    }
});

But it can not setRequestInterception(true) for a chrome instance ( I do puppeteer connect with chromelaucher or create a chrome with remote debug port manually)

I'm glad it's not just me. I kept checking my code wondering why it was different to the docs. In my case I use the following launch option since I'm running in WSL:
executablePath: 'chrome.exe'
Everything seems to work, except the requests are not intercepted.

@xiaohangsu @jazoom are you guys sure you use Puppeteer version that matches your chrome version? The compat table is in the very beginning of our api documentation. Also, check out our FAQ for the explanation on why it is important.

@aslushnikov ah thanks. I was wondering where the compatibility table was. I have been using those API docs for the last 4 hours getting everything set up but I never noticed that table because it looks like it's part of the table of contents. I just skipped over the table of contents and did a lot of ctrl+F. Perhaps it would be better placed down with the actual content? Or alternatively, give it some styling so it doesn't blend in with the table of contents?
I'm using Puppeteer 1.9.0 (automatically downloaded by Yarn), which isn't even in that compatibility table. I will downgrade to 1.7.0 to match the version of Chrome and see if it works...
Nope... interception still doesn't work with Chrome 70 and Puppeteer 1.7.0.

@jazoom can you please post your script?

@jazoom just curious, does the interception work for you in headless mode (I can get it to work in headless mode, but not with the browser actually being shown)?

  let browser;
  try {
    browser = await puppeteer.launch({
      headless: false,
      executablePath: 'chrome.exe',
      args: [`--proxy-server=${options.proxy.ip}:${options.proxy.port}`]
    });

    const page = await browser.newPage();
    await page.setViewport({ width: 800, height: 600 });

    await page.setRequestInterception(true);
    page.on('request', interceptedRequest => {
      console.log('YOU WILL NEVER SEE THIS LOGGED');
      if (['image', 'media', 'font', 'stylesheet'].includes(interceptedRequest.resourceType())) {
        return interceptedRequest.abort();
      }
      interceptedRequest.continue();
    });

    const response = await page.goto(options.url);

    const source = await response.text();

    const content = await page.content();

  } catch (error) {
  }

  browser && await browser.close();

@aslushnikov  Since you're already interpreting that code, can you see why response would be null? I got it from your comment here (https://github.com/GoogleChrome/puppeteer/issues/749#issuecomment-328600349). Unfortunately, response is always null so trying to get text() from it causes a TypeError.

@mor9anz I cannot run it in headless mode due to this issue: https://github.com/GoogleChrome/puppeteer/issues/1837#issuecomment-413697840

Thanks @aslushnikov I gonna close this one.

So why exactly was this closed? The issue remains.

@jazoom it's closed because Puppeteer works only with a matching Chromium version. Please see this explanation
If you are sure you're using correct Puppeteer-core version against your stable Chrome version, please file a separate issue with enough details for us to reproduce locally.

Why not use this issue? I've already posted my code here months ago and confirmed I was using matching stable versions. For some reason it was closed after I confirmed that. The new issue would be the same as this one.

Why not use this issue?

@jazoom Keeping unrelated discussions separate is a good hygiene.

it helps for future users to search across issues and liberates them from reading through long unrelated discussions.
it doesn't send e-mails to those who are on this thread and are not interested in our discussion.

I've already posted my code here months ago and confirmed I was using matching stable versions.

You code is not runnable locally for me - there's no URL, and there's no PROXY. Can you simplify your script and provide with the exact URL so that I can run it locally?
I'd appreciate if you file a separate issue for this, and make sure we can run your code and reproduce the issue.

Why is this discussion unrelated? I can't see any mention here from OP or from me that the issue was related to a version mismatch. Where did you get that assumption? What am I missing here?


(auto) does not go away after I explicitly select the already picked language

Testing #129436

Paste some code so we auto detect it as Typescript for example
Click on the language picker and choose Typescript
I would expect the (Auto) to go away, since the user confirmed that the choice is good

Confirmed. Gonna save this for the next milestone since we might get rid of the (Auto) text.

We removed (Auto) text.


BUG:Scroll of the Review section not working

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The scrollbar of the Reviews are not working
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I want to work on this issue please assign me this issue.


SUGGESTED FEATURES Hint Button

SUGGESTED FEATURES

Hints that are revealed when the user clicks a button. (I know, it’s a quiz, so maybe we should not help, but sharing the idea anyway in case it might inspire related features.) For example: total number of tags, number of tags that start with a certain letter, which letters of the alphabet are NOT used to start a tag, etc.

Originally posted by @CorinaMurg in https://github.com/YurisCodingClub/html-tag-quiz/discussions/20#discussioncomment-7233911

i would like to contribute

As the label points out this ticket still needs some updates.

Can I work on it?

Sure @Tashuuuu, please go ahead

Hi @Tashuuuu how are you doing? Two weeks have passed with no PR from you. Are you still working on this issue? Otherwise, the issue will be unassigned, as written in the contribution guideline.

Is the issue open for contribution? If so can I work on this?

Hi @ShivangiRai1310 . The issue is already assigned.
@CorinaMurg Coming to think of it, is this issue still relevant for our current state of the app? 🤔

I would like to contribute. Seems like this issue is a bit stale and you've had bad luck with other offers.


Remove bottom margin

Was there any reason for a big margin at the bottom? Pages look cleaner this way in my opinion.

I also agree

Ah, but you see when you scroll down to the bottom, with that extra space, the last text of the screen... is centre screen. This is good aesthetics.

Wasting half of the screen real estate after jumping to the bottom of the page is NOT good aesthetics. From a usability perspective this is horrible, it violates multiple design principles. The scroll bar indicates to the user that there is more content, whilst there isn't.


chore: revert "feat: add secrets and env files support"

Reverts cdklabs/cdk-ecs-service-extensions#86
The build was failing for this PR due to the tests added. This should not have been merged.

@Mergifyio refresh

@Mergifyio requeue


logseq: 0.9.8 -> 0.9.9

Description of changes
https://github.com/logseq/logseq/releases/tag/0.9.9
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

I am closing this pull request because the changes have been duplicated in #237805, and this one has already been merged.


change the act.name for LinearActivation() to "linear" so that it won't fail in hl_activetype; also fix the hasinputsset in submodel

Coverage increased (+0.2%) to 62.649% when pulling 45f6e1abee60f0bb14c84b455ffe47a903c3c806 on yu239:hl_activetype into 8d4c453bbb7980491d539bbe574d5e729c90a097 on baidu:develop.


Update description field in .github/settings.yml

@xcorail
We will be using the description listed in the .github/settings.yml for each project to create a summary list for the OpenSSF website getting started page and an upcoming press release. Please edit the description field to include a short (1-2 sentence) description. Due Date - Thursday 10/8/2020
See example here:
https://github.com/ossf/gb-planning-committee/blob/master/.github/settings.yml

@kaywilliams proposal in this PR: #27


500.00$ worth without destination tag.

Sent 500.00$ worth of stellar and never reached destination wallet without destination tag.. how long will I get my stellar back ??? This is nuts

Every transaction to a custodial exchange requires a valid memo for every deposit. It's impossible to change the memo once the transaction is submitted.
You need to contact the destination wallet support team and provide a link to your transaction. They can process your deposit manually.


Adds SpatiaLite as export format

Closes #226

Adds the SQLite-based SpatiaLite file format as supported export format, in order to improve integration with QGIS.
@rochoa @javisantana @andrewxhill

Nice approach!! :smile: Try to see if we can avoid doing the out_format === 'SQLite' thing.


Enable loading translated strings from razor TS code

Enable loading translated strings from razor TS code
This change wraps the many user displayed strings in the razor portion of the project wth l10n.t(''), using the format specifier {0} style where necessary.  reportIssueCreator.ts is probably the single most interesting file.
Conversion from ${abc} to {0} syntax is the most likely area for issues.

I guess we might want to merge this after next Monday after the release?

I guess we might want to merge this after next Monday after the release?

That makes sense to me.


Need SidebarHeading React.createClass

To render inside the SideBar module ./client/src/sidebar.coffee

done with commit 426cfd673f


Support for returning data within a time range

Would be great to introduce the ability to return data within a time range.
This functionality isn't available in the New Relic API Explorer, but works if you're calling it manually via CURL as per this doc https://docs.newrelic.com/docs/apis/rest-api-v2/requirements/extracting-metric-data
For example, the below would return the metrics between 05/09/2014 - 11/09/2015 in 1 hour periods;
-d'names[]=from=2015-09-05T10:00:00+00:00&to=2015-09-11T10:00:00+00:00&period=3600'

Python isn't my primary language, but I'lltry and submit a pull request myself unless someone thinks they can do it before me.

Sounds great! I'd be happy to add this

Perfect, thanks @micahhausler :+1:


Additional XForm validation

Right now, if a user uploads an XLSForm, pyxform will validate it. One thing that pyxform will do is pass the resulting XForm to ODK Validate. However, if a user uploads an XForm, Central will complete only limited validation. The backend will use htmlparser2 to parse the XML, but htmlparser2 is a relatively forgiving parser that will tolerate some errors. The backend won't pass the XML to ODK Validate. There would be benefits to additional validation of XForms:

If there is a form error that ODK Validate would catch, the user would discover it sooner, before encountering an issue on the client
It's hard to predict how a client would respond to the presence of XML errors. Collect and Enketo may respond differently to an XML error. It'd be better to catch any error at the time of form creation.
We may be interested in using the Postgres xml data type in more cases. However, Postgres seems to be strict about XML validation, so that data type is only available for forms whose XML is well formed. If there was a guarantee that all new forms have well formed XML, it might feel safer to use the xml data type in more cases.

Probably what would be most useful would be to pass all XForms through ODK Validate. However, even just a basic check that the XML is well formed would be useful.
Whatever the new validation is, it should ensure continuity for servers that have already uploaded forms with XML that is not well formed. It should be possible for those servers to continue to use any form that has already been uploaded, to upgrade to new versions of Central, and to restore database backups.

We've also had the thought that it'd be helpful to check that submission XML is well formed. Validating submission XML would probably be simpler than form XML, because ODK Validate wouldn't be involved. However, there would also probably be fewer benefits.

Here are a couple of examples of XML errors that Backend currently accepts but that Postgres does not consider well formed:

The closing tag does not match the opening tag, for example, if a <select1> tag is closed by </select> instead of </select1>.
An attribute is enclosed by curly double quotes, for example, version=“53”. Postgres outputs: AttValue: " or ' expected.
The same attribute is specified twice, for example, if readonly="true()" is specified twice for the same element.


Do not emit a file if there are no documented members

Currently outputting files that have just a header.

Also should probably delete files that no longer have content.


Account Configuration Deployment Issue because Multi Location is not enabled.

Select the SuiteCloud tool where you encountered the issue
SuiteCloud CLI
Version of the SuiteCloud Tool
1.8.0
Operating system
macOS Ventura 13.5
Node Version / VSCode Version
Node.js: 16.17.1/VSCode Version: 1.81.1
JAVA Version
java 17.0.5+9-LTS-191
Bug description
I get an error when deploying the Accounting Configuration into a fresh instance.

An error occurred during configuration validation.
Details: Configuration of INVENTORY(Inventory) feature failed with error: Turning on this feature (Inventory) requires enabling the following features.
Multi-Location Inventory

The error says I need to enable the Multi-Location Inventory to enable INVENTORY, but I have the "MULTILOCINVT" feature set to be enabled in the configuration file.

Steps To Reproduce

When running suitecloud project:deploy
In an account where when trying to enable INVENTORY and MULTILOCINVT with the same configuration file.

Actual Output
No response
Expected Output
No response
Anything else?
No response

@purnagadde I was analysing the issue, and there is nothing to be done from the tools perspective.
Please contact NetSuite Support with this, and they will be able to help you.


Bug: Wallet should sanitize mint URLs

Wallet addas mint URLs with trailing / if they are added manually. The URL should be sanitized before treading it as a mint URL.
NUT reference: https://github.com/cashubtc/nuts/pull/151

Is this still open? Or has it been solved by https://github.com/cashubtc/nutshell/pull/605/ ?

better fix in https://github.com/cashubtc/nutshell/pull/606 still pending


Create symlink for rm2fb-server to fix exe name in /proc//stat

The current patch just fix the name of rm2fb-server in /proc/<id>/cmdline but not the executable name in /proc/<id>/stat. Therefore in some process list still the server is shown as xochitl. This patch creates a symlink /opt/bin/rm2fb-server and use this as exec in the systemd unit.

While this works, a cleaner solution to me would be to use prctl(PR_SET_NAME, "rm2fb-server"); to set the process name in rm2fb. I tested it and it does change the value that appears in /proc/*/stat.

Do you have a preference? @raisjn

prctl sounds better, thank you (and thanks for finding this, jan)

I added the prctl call to upstream. I’ll close this PR and open a new one to update the package to the latest upstream release. Thanks for your help @JanLuca!


Energy Systems Catapult Data

Dataset Name: Energy Systems Catapult Data
Dataset Description: Various but includes data from our living lab trials of >100 homes using smart heating controls
Dataset Owner: Energy Systems Catapult
License: Bespoke Open (to comply with funding conditions) https://usmart-static.s3-eu-west-1.amazonaws.com/Data+Sharing+Licence+08102019+v3.0.pdf
Link: https://usmart.io/org/esc/

Hey @richard-ESC, just took a closer look at this.
From the metadata, it looks like your data descriptor is conforming to data.json non-federal-v1.1.
When I run it through a validator I get two errors though. See here.
This means that our harvester breaks. Is there a way for you to fix the metadata?
Otherwise, I'll see if I can convert that manually into something that works.
Thanks!

Hi Flo,
I've asked someone to look into it - I'll see if they can fix our end before you do anything too manual.
Thanks,
Richard
Dr. Richard Dobson
Practice Manager - Data Systems
Mob: +44 (0)7970 689889
Switchboard: 0121 203 3700
From: Flo<EMAIL_ADDRESS>Sent: 24 November 2020 17:30
To: openclimatefix/ckanext-openenergy<EMAIL_ADDRESS>Cc: Dr. Richard Dobson<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [openclimatefix/ckanext-openenergy] Energy Systems Catapult Data (#16)
Hey @richard-ESChttps://github.com/richard-ESC, just took a closer look at this.
From the metadata, it looks like your data descriptor is conforming to data.json non-federal-v1.1.
When I run it through a validator I get two errors though. See herehttps://labs.data.gov/dashboard/validate?schema=non-federal-v1.1&output=browser&datajson_url=https%3A%2F%2Fdiscovery.usmart.io%2Forg%2F92610836-6c2b-4a26-a0a0-b903bde0dc46%2Fdcat%2Fdata.json&qa=true&as_sfid=AAAAAAUM-YFRiCOFDvJOX58v_8EKFM56qbhNbk2z0GwFhkjnMZEbxr86M7N3ot0uIvqjikok6alqppWF0PLk80ubUC5UA_xvwrYK3bVHRXZyAgyXBlTjRcueBOKjq4-townjZdQ%3D&as_fid=63af75cfd657aa40cc1bdb6ecbd99a482c495848.
This means that our harvester breaks. Is there a way for you to fix the metadata?
Otherwise, I'll see if I can convert that manually into something that works.
Thanks!

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/openclimatefix/ckanext-openenergy/issues/16#issuecomment-733126696, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARQQM4BJPZQFK64W2PVBJLDSRPURHANCNFSM4S7DNCQQ.

This document is marked as confidential testing


@Inject CamelContext on RouteBuilder seems not working

@ApplicationScoped
public class MyCamel extends RouteBuilder {

    @Inject
    private CamelContext camel;

    @Override
    public void configure() throws Exception {
        from("timer:foo").routeId("myRoute").
                process(exchange -> System.out.println(camel));
    }
}

This prints null

this is probably because it is loaded by classpath scanning thus it bay pass arc. We should have a way to distinguish between auto discovered routes from classpath and ‘runtime routes’.

remember the discussion about adding a custom annotation or leveraging BindToRegistry

Argh yeah this rings a bell.

See PR
https://github.com/apache/camel-quarkus/pull/358

Yes, there are issues with @Inject in RouteBuilder. But if you only need the CamelContext, you can call getContext() from configure().

See ticket #136


Remember previous url to return to after successful login

https://stackblitz.com/edit/ngx-web-storage

https://stackoverflow.com/questions/50675401/angular-5-redirect-the-previous-page-after-sign-in

https://www.baeldung.com/spring-security-5-oauth2-login#2-custom-authentication-success-and-failure-behavior

https://stackoverflow.com/questions/20349594/adding-additional-details-to-principal-object-stored-in-spring-security-context

implemented with ngx-web-storage