Add ValidatesRunner task for local_job_service and Java SDK harness

This adds a ValidatesRunner suite for the Java SDK against the local Python ULR.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[n/a] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Once I get the suite running I can do some refactors to try to share with setupVirtualEnv and clear out the extraneous sleepwaiting and whatnot.

Found a lot of exclusions before I started getting my disks filled by the build & local runner.
@robertwb you may be interested in the failures where it seems empty side inputs don't work

PTAL. I have a final set of exclusions, categorized, with all other tests passing locally. The local run took about an hour. I have adjusted applyPythonNature to be compatible with applyJavaNature and reused the code despite my misgivings. I did not refactor the build support code.

I have curated the commits into orthogonal changes, if you wish to focus your review.

https://ci-beam.apache.org/job/beam_PreCommit_Python_Commit/13653/ appears to be a failure in unrelated code. It does not look like something that would flake. Perhaps master got broken.

Run Python PreCommit

Flakes in this run:

https://issues.apache.org/jira/browse/BEAM-10474 failure in py DisplayDataTest

And then I just introduced an actual compile error in my fixups.

On the portable precommit, 4 gradle scans succeeded and then the Gradle daemon crashed.

Run Portable_Python PreCommit

(it could have been interrupted, I didn't dig too deep)

Flakes this time in Java precommit:

https://issues.apache.org/jira/browse/BEAM-9968 RemoteExecutionTest
https://issues.apache.org/jira/browse/BEAM-10485 ElasticSearchIOTest (just filed from seeing it flake on another PR)
https://issues.apache.org/jira/browse/BEAM-8025 CassandraIOTest
https://issues.apache.org/jira/browse/BEAM-10460 SparkPortableExecutionTest (now resolved, I think)

Run Java PreCommit

OK this is known flake https://issues.apache.org/jira/browse/BEAM-10470 in JdbcDriverTest, totally unrelated to this and shouldn't even be running. At this point I feel comfortable merging this so things can move on despite flakes.


Added node plugin to grpc-tools npm package, updated build_package_node

This makes the gRPC_build_packages job build the grpc-tools npm package and the corresponding binary artifacts. This fixes #5982.

This is a successful build from this branch: https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_build_packages/621/platform=linux/

The changes look fine to me, but I looked at the contents of that platform-specific tarballs (such as  linux-x64.tar.gz) and they seem to contain both  grpc_node_plugin and grpc_node_plugin.exe. Was that really intended?

No, that wasn't intended, and I think that last change fixed it.

I think I got it right this time: https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_build_packages/626/

LGTM.


feature: add action button to messagebar

I couldn't add action to  BitMessageBar based on the previous HTML structure. so it almost redesigned.
please let me know if anything should be enhanced relate to this changes

Thank you very much for applying your first contribution to this project.
but because there is still no contribution guide written in the README file, I think we need to have a talk about the general conventions which are required to be applied in every component. but until then you can study other components to grasp the general idea behind our conventions.

Hello mr-rahimi and thank you for your contribution.
in this state of project we are trying to keep .razor files as simple as they can get.
as you have mentioned the previous HTML structure had some flaws so would you elaborate more on these changes you have made ?

Hi @khashayar-pakkhesal
sure,
first of all, there wasn't any flaw in the previous HTML code, in my opinion, it was not extendable to support the action button as the Official Fluent Design implemetation did. so I tried to follow the official HTML structure to avoid reinventing the wheel:

Furthermore, I found some issues with the BitMessageBar may help us:

the MessageBar styles don't use some common theme property like font-size and font-weight.
some Html element doesn't follow HTML Semantic Elements, for example, Span tag is used instead of the Button tag

generally, I think following the official HTML structure of fluent design as much as possible, saves our time in terms of re-invention and extendability.

I think there are too many issues in this PR to be fixed simply with new commits.
I'm going to close this PR and request a meeting in order to discuss the issues & conventions in this project.
could you please provide a way for us to contact you? @mr-rahimi


Access checkpoints

Hi @MXueguang, may I ask how to access these checkpoints?

Checkpoints:
sentence selection: gs://pongo-bucket/xueguang/vert5-repl/ss-train
label prediction: gs://pongo-bucket/xueguang/vert5-repl/lp-train
sentence selection (with dev): gs://pongo-bucket/xueguang/vert5erini/train/monot5-3B-LP-balanced-dev
label prediction (with dev): gs://pongo-bucket/xueguang/vert5erini/train/monot5-3B-SS-balanced-dev

Thank you!

Hi @Elfsong, could you retry? I just modify the permission

It works! Thank you so much for your rapid reply!

Btw, there is an outdated parameter that may need to update in "Convert T5 output to sentence selection result file (in jsonl format)":

parser.add_argument("--thres", type=float, required=True)

In the "Run T5 inference for sentence selection" section:
Should the gin file "operative_config.gin" be specified as "gs://t5-data/pretrained_models/3B/operative_config.gin"?


keyboard in KeyboardActivity

replaced editText with actual keyboard in KeyboardActivity (ASCII chars only)

closed because of merge errors.


Do you mean that bd36982 & 7195053 are duplicate commits?   If so, I will squash this to 1 commit

This includes commits which are also contained in #151.  So need to review & merge that first, then this one.

Rebased on updated.  This PR is for test only, setting the new GlobalReadVectorWidth=1 needs more performance testing.

New version : set default to match VW.  If this passes, let's merge.

Can we get this merged? Would be helpful for the changes I am making to allow for mixed precision.


lang chain Azure vector search not working neither on its direct fields nor on its metadata fields

Please help , I am blocked on it from many days .
I am trying to filter question of answer from pdf doc based on uploaded email filter . Only if email match happen answer the question from that document else just empty .
I tried following code its not working rather even in case of OTHER email id also it  give answer from my email uploaded document which is wrong , seems like filtering not working .
First i tried by putting email as metadata it did not work then i added as independent email field its not working there either .
I am instantiating vector store like below , it has email field in it . I am able to create and successfully able to upload pdf doc with all the fields  including email is populated . I checked created index and email field is filterable , searchable .

Creating azure instance as below :
 self.vector_store = AzureSearch(azure_search_endpoint=endpoint, azure_search_key=admin_key, index_name=index_name, embedding_function=embedding_function,fields=fields)`

After doc uploaded as shown below it has email in it .

<EMAIL_ADDRESS>"https://ops-documents.search.windows.net/indexes('index-new-5')/$metadata#docs(*)",
  "value": [
    {
<EMAIL_ADDRESS>1,
      "id": "NjQ1YWViNWQtNDJkNy00NTcxLTlkMTktMDIzZTc0NTZlNDhm",
      "content": "ICAO TRIP 2023 –  .",
"metadata": "{\"source\": \"7-pl\", \"page\": 7, \"file_id\": \"65612fd773a9aa51a0939c96\", \"upload_document_name\": \"SITA Lab Furhat Backoffice IMM officer Dialog - v1.pdf\", \"email\"<EMAIL_ADDRESS>\"company\": \"sita\"}",
     "email"<EMAIL_ADDRESS>

Using following way to search :

my_retriever = self.vector_store.as_retriever(search_kwargs={'filter': {  'email': email }})
qa = RetrievalQA.from_chain_type(llm=self.llm,chain_type="stuff",retriever=my_retriever,chain_type_kwargs={"prompt": self.PROMPT},
            return_source_documents=True,
        )
        results = qa({"query": question})

Installations using as :
**I am using OPENAI_API_VERSION="2023-10-01-preview" with gpt4 model**
azure-common==1.1.28
azure-core==1.29.5
azure-identity==1.15.0
azure-search==1.0.0b2
azure-search-documents==11.4.0b8
langchain==0.0.326

Kindly let me know if anything needed from me . Thanks so much for your help .

I tried this but no luck :
my_retriever = self.vector_store.as_retriever(
search_kwargs={'filter': ("email eq '%s'" % email)}
)
    print(f"my_retriever : {my_retriever}")

    qa = RetrievalQA.from_chain_type(
        llm=self.llm,
        chain_type="stuff",
        retriever=my_retriever,
        chain_type_kwargs={"prompt": self.PROMPT},
        return_source_documents=True,
    )
    results = qa({"query": question})

Log : my retriever logged as below :
my_retriever : tags=['AzureSearch'] vectorstore=<langchain.vectorstores.azuresearch.AzureSearch object at 0x7fa3ab4fdc60> search_kwargs={'filter': "email eq<EMAIL_ADDRESS>

There is a typo in your key. It should be 'filters' instead of 'filter'

The documentation clearly says "filter"
 search_kwargs (Optional[Dict]): Keyword arguments to pass to the
                search function. Can include things like:
                    k: Amount of documents to return (Default: 4)
                    score_threshold: Minimum relevance threshold
                        for similarity_score_threshold
                    fetch_k: Amount of documents to pass to MMR algorithm (Default: 20)
                    lambda_mult: Diversity of results returned by MMR;
                        1 for minimum diversity and 0 for maximum. (Default: 0.5)
                    filter: Filter by document metadata


hide a map if no records produced

Hi, it is possible not to generate a map if the query produces 0 records?
here is my example
thank you
SELECT
    'title' AS component,
    'map' AS id;

SELECT
    'map' AS component,
    1 AS zoom,
    '' AS attribution;
SELECT
    (meta_value->>'title')::text AS title,
    (meta_value->>'description')::text AS description_md,
    (meta_value->>'latitude')::numeric AS latitude,
    (meta_value->>'longitude')::numeric AS longitude,
    'users' AS icon,
    20 AS size
FROM
    post_meta
WHERE
    post_id = $id::int
    AND meta_key = 'geo'
    AND meta_value->>'latitude' IS NOT NULL
    AND meta_value->>'longitude' IS NOT NULL;

Yes! You can add a where condition to the first two SELECT statements

awesome! it works now
SELECT
    'title' AS component,
    'map' AS id
WHERE EXISTS (
    SELECT 1
    FROM post_meta
    WHERE post_id = $id::int
    AND meta_key = 'geo'
    AND meta_value->>'latitude' IS NOT NULL
    AND meta_value->>'longitude' IS NOT NULL
);

SELECT
    'map' AS component,
    1 AS zoom,
    '' AS attribution
WHERE EXISTS (
    SELECT 1
    FROM post_meta
    WHERE post_id = $id::int
    AND meta_key = 'geo'
    AND meta_value->>'latitude' IS NOT NULL
    AND meta_value->>'longitude' IS NOT NULL
);

SELECT
    (meta_value->>'title')::text AS title,
    (meta_value->>'description')::text AS description_md,
    (meta_value->>'latitude')::numeric AS latitude,
    (meta_value->>'longitude')::numeric AS longitude,
    'users' AS icon,
    20 AS size
FROM
    post_meta
WHERE
    post_id = $id::int
    AND meta_key = 'geo'
    AND meta_value->>'latitude' IS NOT NULL
    AND meta_value->>'longitude' IS NOT NULL;


Remove unused load_from_cache_file arg from Dataset.shard() docstring

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


CI: add devcontainer-rebuild workflow

replicates the flow of how openpilot setup is. Only difference is it doesn't contain a separate setup.yaml action. Since it's just for rebuilding the command itself can be ran within the workflow devcontainer-build-with-retry
Resolves #32563

marking as ready for review since tools/devcontainer test passed
https://github.com/commaai/openpilot/actions/runs/9275825302/job/25521430624?pr=32564


voprhub: Check non-main seed on main

The voprhub mostly runs against main, but can test non-main branches (when PR's are tagged with vopr).
When the voprhub discovers a broken seed on a non-main branch, before creating a github issue for that seed, it should test the seed on main. If the seed also fails on main, it should specify that instead of the non-main branch in the PR description.
(Inspired by https://github.com/tigerbeetle/tigerbeetle/issues/1484)

Great idea.
On Mon, Jan 29 2024 at 7:14 PM, djg @.***> wrote:

The voprhub mostly runs against main, but can test non-main branches
(when PR's are tagged with vopr).
When the voprhub discovers a broken seed on a non-main branch, before
creating a github issue for that seed, it should test the seed on main.
If the seed also fails on main, it should specify that instead of the
non-main branch in the PR description.
—
Reply to this email directly, view it on GitHub
https://github.com/tigerbeetle/tigerbeetle/issues/1492, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADE4FVGYDX3AQB2JT2YAUDYQ7KGBAVCNFSM6AAAAABCPZ4BU2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGEYDMMBSGMYDIMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Probably not worth extra complexity and the loss of robustness here. The actual problem is solved by not having failing seeds on main, which we are at 99% for!


Fix a typo, change . to ,

Found a typo when trying to use the gem, fixed it and was able to use the gem to get information out of a Dataverse!

@dl-maura sorry it took so long to review. I missed the notification apparently.
Thanks for spotting the typo.
There seems to be another one on the same line: includCached should be includeCached. Would you mind fixing that too?

Fixed that as well!

@dl-maura Thanks. I'll roll out a new gem version later today.


Any news on when Gen 2 will be GA?

It’s been about 6 six months now so I’m just wondering if we have a possible date lined up for when this may become GA so we can use it for production data?

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

ID: b975977b-abaa-8429-c902-30fd470c43ac
Version Independent ID: 289cf357-a6d6-7c00-d282-fe93f66a5c4e
Content: Azure Data Lake Storage Gen2 Preview Introduction
Content Source: articles/storage/data-lake-storage/introduction.md
Service: storage
GitHub Login: @jamesbak
Microsoft Alias: jamesbak

@hassanbabaie we currently don't have an exact date that's published, i'd recommend posting in: https://feedback.azure.com/forums/327234-data-lake We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Web cmdlets feature should allow for scoping where headers are sent

Summary of the new feature / enhancement
Using the new CookieContainer in dotNet, we should allow scoping of headers.
https://learn.microsoft.com/en-us/dotnet/api/system.net.http.httpclienthandler.allowautoredirect?view=net-6.0
Proposed technical implementation details (optional)
No response

Could you write a test to show how this feature works in curl or wget ?


Folding City Paranoma Animation[GSSOC'23]

Describe the project you want to add with tech stack

Folding City Paranoma Animation
Expected behavior

Hello Sir,
I want to add my new CSS animation project on Folding City Paranoma Animation where we can see a picture of a city is folding like a paper like a paranoma. I have made it using HTML & CSS.
Screenshots (optional)

Additional context (optional )

Please assign me this issue under GSSOC'23.

@MohitGupta121 @pranjay-poddar @TusharKesarwani please assign me this issue under GSSOC'23.


Containers appear to be crashing with fork/exec resource unavailable

What happened:
On one particular node that runs 2 rabbitmq containers connected to two different rabbitmq clusters, the rabbit containers appear to be crashing with the above error.
What you expected to happen:
The containers to run properly
How to reproduce it (as minimally and precisely as possible):
helm install -n rabbit-1 stable/rabbitmq-ha
helm install -n rabbit-2 stable/rabbitmq-ha

Connect some clients to both cluster that create 5-10 queues, ping/pong type commands should do.
Wait for a day or two.
kubectl get pods -o yaml -n demo demo-rabbitmq-0
[...]
        message: 'failed to start shim: fork/exec /usr/bin/docker-containerd-shim:
          resource temporarily unavailable: unknown'
        reason: ContainerCannotRun

Anything else we need to know?:
 journalctl -u docker | tail -n 200
[...]
Mar 31 19:15:32 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T19:15:32Z" level=info msg="shim docker-containerd-shim started" address="/containerd-shim/moby/f5537aadb449b48a2e3f3e75cc8e01d9b2b11139fe172bb8d30d0fec1cd58ed1/shim.sock" debug=false pid=17287
Mar 31 19:15:46 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T19:15:46Z" level=info msg="shim reaped" id=cc95a7934f7203251394599bb8800700b595e7457998b34e38b393f6c78d6e9b
Mar 31 19:15:46 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T19:15:46.837885110Z" level=info msg="ignoring event" module=libcontainerd namespace=moby topic=/tasks/delete type="*events.TaskDelete"
Mar 31 19:15:46 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T19:15:46.875961539Z" level=warning msg="Cannot kill container cc95a7934f7203251394599bb8800700b595e7457998b34e38b393f6c78d6e9b: unknown error after kill: fork/exec /usr/bin/docker-runc: resource temporarily unavailable: : unknown"
Mar 31 19:15:47 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T19:15:47Z" level=info msg="shim docker-containerd-shim started" address="/containerd-shim/moby/b48df07b3f3c9ade5a4ac8bb853b21edf3ae1b9f6a327479f8319c283cffaf1a/shim.sock" debug=false pid=18696
Mar 31 19:16:26 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T19:16:26Z" level=info msg="shim docker-containerd-shim started" address="/containerd-shim/moby/9ae4aff4045752cb655114caa38c086916dfc04f8af450a08e5086ce626d7245/shim.sock" debug=false pid=21630
Mar 31 21:27:21 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T21:27:21.408313532Z" level=error msg="stream copy error: reading from a closed fifo"
Mar 31 21:27:21 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T21:27:21.411989927Z" level=error msg="stream copy error: reading from a closed fifo"
Mar 31 21:27:21 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T21:27:21.425543549Z" level=error msg="Error running exec e9c4cc3e1d5cc42356399896c12a8887f8fc7b940c5b70e29d226a3dfbc8fc7b in container: OCI runtime exec failed: exec failed: container_linux.go:348: starting container process caused \"process_linux.go:90: adding pid 30275 to cgroups caused \\\"failed to write 30275 to cgroup.procs: write /sys/fs/cgroup/cpu,cpuacct/kubepods/burstable/poddd27e9d8-51b1-11e9-871b-0e31a129cd1a/9ae4aff4045752cb655114caa38c086916dfc04f8af450a08e5086ce626d7245/cgroup.procs: invalid argument\\\"\": unknown"
Mar 31 23:03:05 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T23:03:05.622015961Z" level=error msg="stream copy error: reading from a closed fifo"
Mar 31 23:03:05 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T23:03:05.622443440Z" level=error msg="stream copy error: reading from a closed fifo"
Mar 31 23:03:05 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T23:03:05.636902513Z" level=error msg="Error running exec 28ee970a77964be05eaf971008ef6ae10198f7f205cc5e752256968c773c32cd in container: OCI runtime exec failed: exec failed: container_linux.go:348: starting container process caused \"process_linux.go:90: adding pid 18681 to cgroups caused \\\"failed to write 18681 to cgroup.procs: write /sys/fs/cgroup/cpu,cpuacct/kubepods/burstable/podad780255-51b1-11e9-871b-0e31a129cd1a/b48df07b3f3c9ade5a4ac8bb853b21edf3ae1b9f6a327479f8319c283cffaf1a/cgroup.procs: invalid argument\\\"\": unknown"
Mar 31 23:33:44 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T23:33:44Z" level=info msg="shim reaped" id=642c774c1d4a6c656dd5da88e436aebc59bc65cd3fb2ae6a4b07589b04f655c1
Mar 31 23:33:44 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T23:33:44.672119161Z" level=info msg="ignoring event" module=libcontainerd namespace=moby topic=/tasks/delete type="*events.TaskDelete"
Mar 31 23:33:46 ip-10-128-95-215.ec2.internal dockerd[3601]: time="2019-03-31T23:33:46Z" level=info msg="shim docker-containerd-shim started" address="/containerd-shim/moby/9ffdd43b4b3d92982107f6a38369bef5c24ce0c659e666d6c571ea0237b8b48f/shim.sock" debug=false pid=5037

[root@ip-10-128-95-215 ~]# cat /lib/systemd/system/docker.service | grep -v "#"
[Unit]
Description=Docker Application Container Engine
Documentation=https://docs.docker.com
After=network-online.target firewalld.service
Wants=network-online.target

[Service]
Type=notify
EnvironmentFile=-/etc/sysconfig/docker
EnvironmentFile=-/etc/sysconfig/docker-storage
EnvironmentFile=-/run/docker/runtimes.env
ExecStartPre=/bin/mkdir -p /run/docker
ExecStartPre=/usr/libexec/docker/docker-setup-runtimes.sh
ExecStart=/usr/bin/dockerd $OPTIONS $DOCKER_STORAGE_OPTIONS $DOCKER_ADD_RUNTIMES
ExecReload=/bin/kill -s HUP $MAINPID
LimitNOFILE=infinity
LimitNPROC=infinity
LimitCORE=infinity
TimeoutStartSec=0
Delegate=yes
KillMode=process
Restart=on-failure
StartLimitBurst=3
StartLimitInterval=60s

[Install]
WantedBy=multi-user.target

[root@ip-10-128-95-215 ~]# cat /lib/systemd/system/docker.service | grep Tasks
# Uncomment TasksMax if your systemd version supports it.
#TasksMax=infinity
[root@ip-10-128-95-215 ~]#

[root@ip-10-128-95-215 ~]# systemctl status docker | grep Tasks
    Tasks: 734
[root@ip-10-128-95-215 ~]#

Environment:

AWS Region: us-east-1
Instance Type(s): c5.xlarge
EKS Platform version (use aws eks describe-cluster --name <name> --query cluster.platformVersion): 1.11 "eks.2"
Kubernetes version (use aws eks describe-cluster --name <name> --query cluster.version): 1.11 eks.2
AMI Version: amazon-eks-node-1.11-v20190327 (ami-05fe3f841ac4df3bb)
Kernel (e.g. uname -a):

[root@ip-10-128-95-215 ~]# uname -a
Linux ip-10-128-95-215.ec2.internal 4.14.104-95.84.amzn2.x86_64 #1 SMP Sat Mar 2 00:40:20 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
[root@ip-10-128-95-215 ~]#

Release information (run cat /tmp/release on a node):

[root@ip-10-128-95-215 ~]# cat /tmp/release
cat: /tmp/release: No such file or directory
[root@ip-10-128-95-215 ~]#

Experiencing a similar issue with Redis, same version of eks AMI and eks platform. It ends up crashing other services in my cluster that depend on it after approx 2 days.

Getting a similar issue in EKS 1.12 trying to exec into any pods:
kubectl exec -it {podName} -- /bin/sh
failed to create runc console socket: stat /tmp: no such file or directory: unknown
command terminated with exit code 126
us-east-1
ami-0abcb9f9190e867ab

Not sure if this will help your issue but I restarted the kubelet on each node and I was able to run an exec which fixed my problem.
sudo systemctl restart kubelet

Seeing this on any cluster I have which uses 1.12,
  State:          Waiting
      Reason:       RunContainerError
    Last State:     Terminated
      Reason:       ContainerCannotRun
      Message:      failed to start shim: fork/exec /usr/bin/docker-containerd-shim: resource temporarily unavailable: unknown
      Exit Code:    128
      Started:      Wed, 24 Apr 2019 09:08:36 -0700
      Finished:     Wed, 24 Apr 2019 09:08:36 -0700
    Ready:          False

Environment:
AWS Region: us-east-1
Instance Type(s): m5.2xlarge
EKS Platform version (use aws eks describe-cluster --name  --query cluster.platformVersion): 1.12 "eks.1"
Kubernetes version (use aws eks describe-cluster --name  --query cluster.version): 1.12 eks.1
AMI Version: amazon-eks-node-1.12-v20190329 (ami-0abcb9f9190e867ab)
Kernel (e.g. uname -a): Can't get this because nodes ssh process is dead due to issue

I'm getting this too after ~2 days of a node running. New containers fail to start and processes seem to have issues starting as well.
 Error: failed to start container "...": Error response from daemon: failed to start shim: fork/exec /usr/bin/docker-containerd-shim: resource temporarily unavailable: unknown
  Warning  BackOff    7s (x2 over 13s)   kubelet, ip-172-31-100-123.ec2.internal  Back-off

Running AMI: amazon-eks-node-1.12-v20190614 (ami-0200e65a38edfb7e1)

One possible cause is running out of process IDs. Check you don't have 40.000 defunct processes or similar on nodes with problems.

We experienced this on Thursday of last week, it definitely feels like its running out of process IDs, likely due to some fork-bomb style bug.
Unfortunately when it gets into this state we can't even SSH onto the node to find out what is going on.

The node in question for us happened to be running coredns and tiller-deploy (which don't run on every node in the cluster, so I wanted to point this out as a possible breadcrumb)
Did anyone else notice if there was specific things running on that node?

@whereisaaron Thanks for the tip! Our nodes were indeed being filled with defunct processes that were started by a Kubernetes health check command to check the status of our Celery workers. We've removed the check for now.

@shshe do you have more information on that?  What sort of checks were they?

I can confirm this issue is still happening from time to time on v1.13.11-eks-5876d6, docker://18.6.1

@shshe do you have more information on that? What sort of checks were they?

Sorry for the really late response. Our defunct processes were being spawned when we ran celery inspect ping as part of our liveness probe. Here's the related issue in the celery project:
https://github.com/celery/celery/issues/4079#event-2765337059

@dcherniv @shshe Is this still an issue, or can we close the issue?

@dcherniv @shshe Is this still an issue, or can we close the issue?

Yes, I believe the zombie processes were spawned by celery and unrelated to the AMI.


Add tmux to tools on control node

To allow students to protect their ssh session in the event of an unstable network

@andrewhatfield can you rebase and re-submit?

done
On Tue, Aug 14, 2018 at 1:07 AM Sean Cavanaugh<EMAIL_ADDRESS>wrote:

@andrewhatfield https://github.com/andrewhatfield can you rebase and
re-submit?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/network-automation/linklight/pull/83#issuecomment-412551528,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADNP1VuVHic-rmCcJ5JLvrNVrcTdg8Bkks5uQZYxgaJpZM4VfiOw
.


Dev/v0.1.1 beta release

Pull Request
Description
Beta bug fix release update
Features
(REPLACE ME WITH A LIST OF FEATURES/FUNCTIONALITY/BUG FIXES)

Minor version increment

weird build issues reverting


Updates to Employment Service

Jira link

See PROJ-XXXXXX
Change description

Add in ET (et_msg_handler) and ECM (ecm_consumer) microservice as part of a change in how stitching is used in Employment.
Additional change to be raised to remove et_cos and ethos_repl_service once live with the new change

Testing done

Tested locally. PR is to test in lower environments before adding change in cnp-flux-config for prod

Checklist

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)
[ ] Does this PR introduce a breaking change

Closing as work will be temporarily paused. Will reopen when needed

As agreed with @hsjhita , closing this for now. He will re-open this PR once work on this is resumed.


Issue 258 - Add links to commit show page

The accordion is still not accordionned when the page loads, but everything else in issue #258 has been completed.

Links seem to work. Just need to decide on default state of the accordion before this can be merged.


create a line of images to use as the default

this image should be used if one isn't provided and should try to mimic a GitHub Actions runner as closely as possible

see https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners#preinstalled-software for a starting point of what the images should consist of

pretty sure I don't want to maintain this; opted to let users select a "fallback" image that defaults to "node:12" so one could pick as lightweight an image as possible, only having exactly the tools they require


coverage not collected from sub-propcesses result from child_process APIs

Describe the bug
Consider a package which is a CLI tool.
Consider a test suite that does the testing-trophy: it runs e2e with few happy paths, generates coverage, and then write unit-tests to cover only the parts that are not covered by the e2e and component tests.
Now consider that the e2e test cases run the CLI using any of the APIs of child_process: exec,  spawn, fork, or their sync equivalents.
The tool fails to collect coverage from the e2e suites.
The reproduction does only exec, because it is minimal. however, tried all of them.
I'd be delighted if it's just some setting that I'm failing to pass...
reproduction
scenario

given a project (full structure below) that uses child_process to run the bin of the package in e2e test of CLI tools, and unit tests to cover code-paths that do not appear in the e2e coverage

Run vitest run --coverage

expected
100% coverage. for all files.
Found
All tests run successfully, but only coverage collected using unit-tests is observed.
 RUN  v2.1.8 /home/user/ws/opensource/reproduction/vitest/child_process-not-covered
      Coverage enabled with v8

 ✓ test/calc.unit.mjs (1)
   ✓ calc (1)
     ✓ sub (1)
       ✓ should substract positive integers
 ✓ test/cli.e2e.mjs (2)
   ✓ fxgen command (2)
     ✓ when called with valid parameters (2)
       ✓ should not fail
       ✓ should return the right answer

 Test Files  2 passed (2)
      Tests  3 passed (3)
   Start at  17:13:40
   Duration  291ms (transform 18ms, setup 0ms, collect 16ms, tests 32ms, environment 1ms, prepare 128ms)

 % Coverage report from v8
-----------|---------|----------|---------|---------|-------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
-----------|---------|----------|---------|---------|-------------------
All files  |   14.28 |    33.33 |      25 |   14.28 |
 bin       |       0 |        0 |       0 |       0 |
  cli.mjs  |       0 |        0 |       0 |       0 | 1-3
 lib       |   18.18 |       50 |   33.33 |   18.18 |
  calc.mjs |     100 |      100 |      50 |     100 |
  cli.mjs  |       0 |        0 |       0 |       0 | 1-9
-----------|---------|----------|---------|---------|-------------------

Project structure:

vitest.config.js
import { defineConfig } from 'vitest/config';
export default defineConfig({
  test: {
    globals: true,
    reporters: ['verbose'],
    include: [
      'test/**/*.unit.mjs',
      'test/**/*.e2e.mjs',
    ],
  },
});

package.json
{
  "name": "child_process-not-covered",
  "version": "1.0.0",
  "bin": {
    "calc": "bin/calc.mjs"
  },
  "scripts": {
    "test": "vitest run --coverage"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "description": "",
  "devDependencies": {
    "@vitest/coverage-v8": "^2.1.8",
    "vitest": "^2.1.8"
  }
}

bin/cli
#!/usr/bin/env node
import cli from '../lib/cli.mjs';
cli({ process, console });

lib/cli
import * as calc from './calc.mjs';
export default ({ process, console }) => {
  const { argv: [ ,, op, a, b ] } = process;
  const result = calc[op](Number(a), Number(b));
  console.log(result);
}

lib/calc
export const add = (a, b) => a + b;
export const sub = (a, b) => a - b;

test/calc.unit.mjs
import * as calc from '../lib/calc.mjs';

describe('calc', () => {
  describe('sub', () => {
    it('should substract positive integers', () => {
      expect(calc.sub(44,2)).toEqual(42);
    });
  });
});

test/cli.e2e.mjs
import { execSync } from 'node:child_process';

describe('fxgen command', () => {
  describe('when called with two integers', () => {
    const ctx = {};
    beforeAll(() => {
      try {
        ctx.result = execSync('node bin/cli.mjs add 20 22').toString().trim();
      } catch(err) {
        ctx.err = err;
      }
    });

    it('should not fail', () => expect(ctx.err).toBeFalsy());
    it('should return their sum', () => expect(ctx.result).toEqual('42'));
  });
});

System Info
$ vitest --version
vitest/2.1.8 linux-x64 node-v20.18.0

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

disclaimer
Feels like de j'avoux.
I encountered this before while trying to migrating a moch+nyc project, and that effort turned away from vitest.
If I opened such a ticket before - excuse me, for some reason I cannot find it :(
I'm now working with a new customer on a new codebase - new chance :)
Anyway, now I really put the effort to reproduce the but in isolation :)

Vite doesn't intercept node:child_process. Previous discussion is at https://github.com/vitest-dev/vitest/discussions/3851.


import_from_git with authentication specified?

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject
  import_from_git
 No
 no
Question Description

I currently have the following code in our iOS apps fastfiles linked in using import_from_git. We bundle together some Fastlane functions in sequence that only ask for parameters that are needed from the developers (ex bundle identifier)
import_from_git(url<EMAIL_ADDRESS>branch: 'master', path: 'FastfileCommon')

On our mac slave we have removed ssh keys and are switching to https, is there a way that I can pass credentials to the https path so that it can do the clone? Our github setup requires a token for https

@fastlane-bot im not sure my env would help in this situation, can someone answer the question perhaps?

@janpio can you shed some light?

Have a look at the code being executed from import_from_git: https://github.com/fastlane/fastlane/blob/0de811cec8bc50e266b2724d96a3a3852c845e6e/fastlane/lib/fastlane/fast_file.rb#L235-L287 This effectively just calls git clone (line 258)

@janpio i was using username:token instead of just token, my bad.

Can you share an example of what turned out to work for you? Just for future developer googling and landing here ;)

`import_from_git(url<EMAIL_ADDRESS>branch: 'someBranch', path: 'yourFile')


Darwin Hang: new volunteer via Mozilla Science Lab Collaborate

I'm a former geneticist (now web developer) looking to contribute to Open Source.This issue was created by @darwinhang via Mozilla Science Lab Collaborate

Welcome! My name is Madeleine, I'm one of the project leads. I'm a Master's student at the University of Toronto in the physics department.
Thanks for your input already (#49)! Please feel free to make any other suggestions or jump in on an existing issue that looks cool. I'll try my best to clarify things if you have questions - this project is evolving a lot as we go.

Awesome. Thanks for the warm welcome!


Add Extensions to Short Cut Composite Indicators

Composite indicators provide a nice way of chaining indicators but they can be confusing to use sometimes. A couple of well written extension methods would help cover 90% of use cases: specifically:
//This is current operation:
_adx = ADX(_symbol, 30);
_smaADX = new SimpleMovingAverage(_symbol + " SMA of ADX",10).Of(_adx);

//Could be re-written with extensions to be:
_adx = ADX(_symbol, 30).SMA(10);

Hi Jared, quick question.  It seems to me like the last line:
`_adx = ADX(_symbol, 30).SMA(10);`

should be returning an SMA object instead of an ADX object, because the end result of the first three lines seems to be an SMA of the ADX data.  Is my understanding correct?

Yes exactly! The resulting obect should be the SMA of the ADX. It was
intended to wrap the .Of operator into something more intuative.
On Apr 16, 2016 3:11 AM, "Mike"<EMAIL_ADDRESS>wrote:

Hi Jared, quick question. It seems to me like the last line:
_adx = ADX(_symbol, 30).SMA(10);
should be returning an SMA object instead of an ADX object, because the
end result of the first three lines seems to be an SMA of the ADX data. Is
my understanding correct?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/QuantConnect/Lean/issues/372#issuecomment-210754240

Great, thanks!

Completed by c47eb47


Add jsx adapter

Closes #51

Coverage increased (+0.18%) to 98.03% when pulling 86ac8d570476b64c5c6ce746034c4cd2d3b3a47a on jsx into fe64e52557223b04406c0e2b770735789797d4e6 on master.


question: how hard to port style support downstream?

It appears the upstream oksvg has added some support for styles and style classes. https://github.com/srwiley/oksvg/blob/master/testdata/TopLevelStyle.svg?short_path=5d5e486
How difficult would it be to port that capability to this fork?

I'm not sure. I think I've followed upstream oksvg closely enough so that a PR could be adapted.
I can review if you want to try.


Request for new release.

This works great. Thanks a lot! I wanted to request a new release with the latest. Would make CI deployment easier with the latest features.

Closing. Found a work around (caches).


Make image override by each image

Description
To avoid the merge conflict from konflux updates in the same line , we are making the argument specific to each image .
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/cc @raptorsun

/test bundle-e2e-4-15

/lgtm
/test bundle-e2e-4-15

/override "Red Hat Konflux / ols-enterprise-contract / bundle"

/approve

/test bundle-e2e-4-16


Panic on a simple Delete with Where

GORM Playground Link

https://github.com/go-gorm/playground/pull/244
Description

Delete(&Pet{})

Delete(&Pet{})

From the docs, this should work.
It was working in v1.20.8.
Something broke it.
Besides, a panic is bad. In my honest opinion, it'd be better to add a validation.

From the docs, this should work.
It was working in v1.20.8.
Something broke it.
Besides, a panic is bad. In my honest opinion, it'd be better to add a validation.

@jinzhu
Are you really not fixing this?
It is documented in this way, and it was working fine in v1.20.8.
For a documented behavior to just stop working is a really bad thing.
One of two things must be changed: the docs or the code.
Considering that it is documented in this way and that this is a panic, I really recommend you to add a validation to the code to prevent this problem.

@jinzhu
Are you really not fixing this?
It is documented in this way, and it was working fine in v1.20.8.
For a documented behavior to just stop working is a really bad thing.
One of two things must be changed: the docs or the code.
Considering that it is documented in this way and that this is a panic, I really recommend you to add a validation to the code to prevent this problem.


Implement the weaver-gke purge command.

This command will do its best to cleanup the GCP project of all resources created by Service Weaver, including clusters, certificate authorities, IP addresses etc.
Other changes:

Rewrite the Certificate Authority code to use Go APIs, so that we can manage its deletion and re-creation more easily.
Fix a few bugs in the GCP update code.

Super super impressive stuff! I don't know how you learn how to use all the various cloud APIs so quickly. 🧠
Love the branch name too 😆

Thanks for the thorough review. I went through the cloud patcher code again and made some RMW changes.


[iOS] Rendering Picker triggers recursive call leads to app crash

React Native: 0.30.0
iOS: 9.3
Code triggered crash:
<Picker
    style={styles.picker}
    selectedValue={this.state.selected1}
    onValueChange={this.onValueChange.bind(this, 'selected1')}>
      <Picker.Item label="hello" value="key0" />
      <Picker.Item label="world" value="key1" />
</Picker>
Crash stack trace:
#0	0x0000000106ca803d in (anonymous namespace)::AutoreleasePoolPage::pop(void*) ()
#1	0x0000000107c90736 in _CFAutoreleasePoolPop ()
#2	0x0000000107dbb17c in -[_CFXNotificationRegistrar find:object:observer:enumerator:] ()
#3	0x0000000107c6a679 in _CFXNotificationPost ()
#4	0x0000000104e51cd9 in -[NSNotificationCenter postNotificationName:object:userInfo:] ()
#5	0x0000000105680e6c in -[UIScrollView(UIScrollViewInternal) _setContentOffset:animated:animationCurve:animationAdjustsForContentOffsetDelta:] ()
#6	0x0000000105cccba3 in -[UIPickerTableView _scrollRowAtIndexPathToSelectionBar:animated:] ()
(#67954 ~ #67964 repeats)
#67954	0x00000001056832ca in -[UIScrollView(UIScrollViewInternal) _adjustContentOffsetIfNecessary] ()
#67955	0x000000010567f858 in -[UIScrollView(UIScrollViewInternal) _stopScrollingNotify:pin:tramplingDragFlags:] ()
#67956	0x000000010567f918 in -[UIScrollView(UIScrollViewInternal) _stopScrollingAndZoomingAnimationsPinningToContentViewport:tramplingDragFlags:] ()
#67957	0x0000000105cccb2f in -[UIPickerTableView _scrollRowAtIndexPathToSelectionBar:animated:] ()
#67958	0x0000000105ccd404 in -[UIPickerTableView _updateContentInsets] ()
#67959	0x0000000105ccd1d8 in -[UIPickerTableView _rectChangedWithNewSize:oldSize:] ()
#67960	0x00000001056ef79a in -[UITableView setBounds:] ()
#67961	0x0000000105cce700 in -[UIPickerTableView setBounds:] ()
#67962	0x000000010566974c in -[UIScrollView setContentOffset:] ()
#67963	0x00000001056f082e in -[UITableView setContentOffset:] ()
#67964	0x0000000105ccdade in -[UIPickerTableView _setContentOffset:notify:] ()
#67965	0x0000000105e4e523 in -[UIPickerColumnView _moveTableViewIfNecessary:toContentOffset:] ()
#67966	0x0000000105e4e63c in -[UIPickerColumnView _pickerTableViewDidChangeContentOffset:] ()
#67967	0x0000000105ccda6e in -[UIPickerTableView _notifyContentOffsetChange] ()
#67968	0x0000000105cccbbc in -[UIPickerTableView _scrollRowAtIndexPathToSelectionBar:animated:] ()
#67969	0x0000000105cccf33 in -[UIPickerTableView selectRow:animated:notify:] ()
#67970	0x0000000105584a6b in -[UIPickerView _resetSelectionOfTables] ()
#67971	0x00000001055848f7 in __30-[UIPickerView layoutSubviews]_block_invoke ()
#67972	0x0000000105650680 in +[UIView(Animation) performWithoutAnimation:] ()
#67973	0x0000000105582393 in -[UIPickerView layoutSubviews] ()
#67974	0x0000000105585b24 in -[UIPickerView _selectRow:inComponent:animated:notify:] ()
#67975	0x0000000103f18b8d in __30-[RCTPicker setSelectedIndex:]_block_invoke at /Users/bb/Sites/Anomaly/ToolbeamReact/node_modules/react-native/React/Views/RCTPicker.m:46
#67976	0x00000001093c8d9d in _dispatch_call_block_and_release ()
#67977	0x00000001093e93eb in _dispatch_client_callout ()
#67978	0x00000001093d11ef in _dispatch_main_queue_callback_4CF ()
#67979	0x0000000107d020f9 in CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE ()
#67980	0x0000000107cc3b99 in __CFRunLoopRun ()
#67981	0x0000000107cc30f8 in CFRunLoopRunSpecific ()
#67982	0x000000010a9f5ad2 in GSEventRunModal ()
#67983	0x00000001055a0f09 in UIApplicationMain ()
#67984	0x0000000103d9923f in main at /Users/bb/Sites/Anomaly/ToolbeamReact/ios/Toolbeam/main.m:16
#67985	0x000000010941d92d in start ()
It does not crash if no Picker.Items are included in the  tag. Please let me know if more detail is required.
Thanks,
Frank

I'm having the same issue as well. Just simply adding a Picker with an Item causes the app to crash with an EXC_BAD_ACCESS.

+1 Having the same with PickerIOS.

@fwang I tried to replicate Picker errors with Item components - and was not able to ( ie: the Picker element with Picker.Item elements) was working fine for me.
I used the below code:
import React, { Component, } from 'react'
import { View, Picker} from 'react-native'

class TestPicker extends Component {
  constructor(props) {
    super(props)
    this.state = {
      language: 'java'
    }
  }

  render() {
    return (
      <View>
        <Picker selectedValue={this.state.language}
                onValueChange={(lang) => this.setState({language: lang})}>
          <Picker.Item label="Java" value="java" />
          <Picker.Item label="JavaScript" value="js" />
        </Picker>

      </View>
    )
  }
}

export default TestPicker

I notice the code you reference also comes from the docs at https://facebook.github.io/react-native/docs/picker.html - it is a known issue they reference outdated versions of the API with methods such as const someClass = React.createClass({}) instead of class someClass extends Component and getInitialState instead of just setting the state in the class constructor.
I suggest using the Picker syntax I used above, it should work better :)
Happy to help otherwise - sharing the rest of your component / page might be helpful.

@cpruijsen Thank for your help. I have tried your code in my current project and had no luck. Weird. I will create a new project to try it out and let you know.
Thanks!

The issue ended up being a third party UIScrollView category we included that's capture the Picker setContentInset event.
Thanks again @cpruijsen for your help.
Closing this issue as it is not React related.


Can't read public dataset from HuggingFace

Describe the bug
I'm trying to read the following HuggingFace dataset into Daft: https://huggingface.co/datasets/HuggingFaceM4/DocumentVQA
To Reproduce
Following https://www.getdaft.io/projects/docs/en/stable/user_guide/integrations/huggingface.html#reading-public-datasets
import daft

df = daft.read_parquet("hf://HuggingFaceM4/DocumentVQA")
df

unfortunately yields:
DaftCoreException: DaftError::External Generic http error: Invalid path: hf://HuggingFaceM4/DocumentVQA

Desktop (please complete the following information):

OS: Linux Mint 22 (based on Ubuntu 24.04)
Version: getdaft v0.3.1

Additional context
Many thanks for any help, and this amazing lib! :)

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

Hello,
Seems like our docs might be slightly wrong - you may need to add /datasets/ into the URI, like hf://datasets/HuggingFaceM4/DocumentVQA.

Can confirm this works! Many thanks :)
import daft

df = daft.read_parquet("hf://datasets/HuggingFaceM4/DocumentVQA")


Netty should set request header host by default when http1.1 is used.

Expected behavior
Netty should set request header host by default when http 1.1 is used if customer does not set the host header.
Actual behavior
Get 400 response if do not set header host.
Steps to reproduce
Use examples in the project, and delete set host header action in examples.
Minimal yet complete reproducer code (or URL to code)
Netty version
4.0.24.Final
JVM version (e.g. java -version)
build 1.8.0_151-b12
OS version (e.g. uname -a)
MacOS Mojavae 10.14.5

I disagree.. the user should do it as it is not clear if a hostname / ip etc should be used.


0.2.18 Matching URL problem

Hey,
Since 0.2.18, the URL matching does not catch up some URL
In the following example, when LinkedIn API calls back a sequential URL, ui-router catches it and proceeds.
When Facebook API calls back a URL with a query string (2 arguments), it is never catched.
It works perfectly in 0.2.15 / 0.2.16 / 0.2.17 and the controller of the called view does the cookie context thing, it has to do.
My fallback state is called in 0.2.18, and the authentication is not completed then.
 state('callbackLinkedin', {
    url: '/callbackLK/:from/:token',
    templateUrl: function () {
        return 'partials/commons/pages/authentication/callback-linkedin.html';
    }
}).
state('callbackFacebook', {
    url: '/callbackFB?rp&code',
    templateUrl: function () {
        return 'partials/commons/pages/authentication/callback-facebook.html';
    }
})

I guess there is something new in the UrlMatcher 0.2.18 that does not handle the case the same way as before.
I rolled back to 0.2.17 to make my app work.
C.

@chenav  can you show me an example facebook callback url?

The callbackURL that does not work is like so;
https://mydev.env.io/callbackFB?rp=account&code=AQDkJKAIxFhNAzaFPWchT5ybty7EVKgSJ-r57OR1R0isVhL1nJfi8HUlXGDw3h5-Oes0z1LLQAYzwH_YrcEUo5YXktQYbklDaObmiIWlVORytq_DzB4yZRDCxDr4WrH_9rh651dvcD0xYcTsNSLujDZxBbF51LfI-gR1ekS5hF-20MyBtMu6sMyl8LslvGG4skn-l1VnbZlwT3yPD5PFXxLJ-_b03nIlz-ZtOHtjr8Z57FulyZuW7NFTR_rXNvGAsswTpKkWC88OXNZ2D0Ycf0B8vCPRCiF053BTfQ5-YKQNoc1hCUAf-xET59K6l9o5FQ0#_=_

So, this is OAUTH or something? Facebook redirects the browser to that url, your app loads from the server into the browser (html5 mode), then ui-router bootstraps and checks the URL, but the parameters don't match?

Yes, this is oauth and the URL is the callback URL.
And you are right about the sequence of calls.
ui-router 0.2.18 does not match the URL.
All the previous versions do match it.

This seems to work for me.
http://plnkr.co/edit/wy1i9jSOOsmb2rLUpeN3?p=preview

Thank you for your feedback, Christopher.
Actually, it gave me headache because it was still not working on my side.
I traced all my states and state params to see what was different with 0.2.18.
And I finally found it.
I have a lot of states but only 2 with URL+query strings to match at router level.
Here they are (note the order of the declarations) :
state('home', {
    url: '/:sitehash?appMode&refresh',
    templateUrl: function () {
            return 'template1.html';
    }
}).
state('callbackFacebook', {
    url: '/callbackFB?rp&code',
    templateUrl: function () {
        return 'template2.html';
    }
})

In that order, it works with all version of ui-router inferior to 0.2.18
With 0.2.18, the Facebook Callback we talked about is matched as state 'home' with the following params:
sitehash = 'callbackFB'
appMode = undefined
refresh = undefined
rp = account
code = the_very_long_fbcode

It is a very tricky case and I am not sure it is what we should expect, as the signatures of the URLs (and the query strings) are different.
But in order to correct the behaviour, I just had to inverse the order of my 'home' state and my 'callbackFacebook' state.
With the 'callbackFacebook' state in first position, it matched it right with 0.2.18
I do understand that one is matched dynamically when the other is not, but anyway, it was working right before that version.
Now, since 0.2.18, it seems that we have to be extra careful with the order of states declaration .
Is that done on purpose ?


fleet:fix cluster copy bug

What type of PR is this?
/kind bug
What this PR does / why we need it:
This bug causes：  when deleting clusters from the fleet，only the label of the last cluster can be deleted
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

This bug causes： when deleting clusters from the fleet，only the label of the last cluster can be deleted

Not sure i understand. Can you elaborate a little more

I get to understand the issue now. I think we donot need deepcopy each object. We can do a shallow copy.
for _, cluster := range clusterList.Items {
                tmpCluster := cluster
		labeledCluster = append(labeledCluster, &tmp)
}

Can you also check pkg/fleet-manager/application_helper.go in generateFleetCluster we also iterate on cluster list and append its address

I get to understand the issue now. I think we donot need deepcopy each object. We can do a shallow copy.
for _, cluster := range clusterList.Items {
                tmpCluster := cluster
		labeledCluster = append(labeledCluster, &tmp)
}

o'kok

/label tide/merge-method-squash

Can you also check pkg/fleet-manager/application_helper.go in generateFleetCluster we also iterate on cluster list and append its address

updated

/lgtm
/approve


Problem suggestor

I like your website.But I have a question.
In A2OJ Ahmed Aly gathered all problem in such a way that after solving almost any problem you will learn something new.
Of what basis your website suggest problems?Is it like A2OJ or its suggest randomly(means rating wise)?If answer is randomly then why I'll use an extra website when I can easily get it from codeforces?So try to change this strategy.Otherwise after some month your website will be useless and your hard work will go in vain.

Thanks for the suggestion. But, problem suggestion is not random. It is based on the category type and ratings of the problems you solved earlier.  In a few days, I will integrate the problem tags suggestion also along with it. Please let me know if you think I can add more to it.

Thanks for the latest update. :)
I think User solved and Rating

And there is an another problem.
Suppose my rating is x.Problem suggestor suggest me problem of rating x+10/20 to x+400.
Here lower bound is to low and also upper bound is two high .In my opinion lower bound should be x+100 and upper should be like x+200/300...400 is to high...
and also white color of this  website is too dazzling.Try to follow the white color of codeforces. Also font.


App crash on startup with cordova-admob-pro installed

Hi, I just made a fresh project to test this plugin.
I went and downloaded the latest Google Play Service from Android Studio, but the app crashes on startup. adb logcat reports that the issue is related to some firebase library, so maybe the plugin needs to be updated to reflect the changes that were introduced yesterday after Google I/O.
Here is the stack trace:
D/AndroidRuntime(19028): Shutting down VM
E/AndroidRuntime(19028): FATAL EXCEPTION: main
E/AndroidRuntime(19028): Process: com.test.fuck, PID: 19028
E/AndroidRuntime(19028): java.lang.RuntimeException: Unable to get provider
com.google.firebase.provider.FirebaseInitProvider: java.lang.IllegalStateException: Incorrect provider authority in manifest. Most likely due to a missing applicationId variable in application's build.gradle.

Ok, I got it to work by following this advice on SO, which worked like a charm.
Unfortunately I don't know gradle enough to sumbit a PR :(

I think this issue has no much relationship with this plugin, as it won't change build.gradle.


Is there anyone to upload the codes?

I'm so interested in this project,but it seems that someone has forgotten to upload the codes.
If you can see this issue,could you upload the codes plz?
Thanks!

Same here. Any chances they would get uploaded before the start of 2021 ?

Could you upload your codes. Thx

is there any chance to upload the code within 2021? thanks


After I started the project in my own linux, there has a problem. Can anyone tll me why?

"An error occurred in the Server Components render. The specific message is omitted in production builds to avoid leaking sensitive details. A digest property is included on this error instance which may provide additional details about the nature of the error."

thanks，I have been solving this problem.

From: @.>
Date:  Wed, Oct 25, 2023, 01:47
Subject:  Re: [openchatai/OpenCopilot] After I started the project in my own linux, there has a problem. Can anyone tll me why? (Issue #173)
To: @.>
Cc: @.>, @.>
Hey @huhy-uniplorehttps://github.com/huhy-uniplore
Have you managed to get it to work? if not can you run
make logs
on your terminal and share the output here?
—
Reply to this email directly, view it on GitHubhttps://github.com/openchatai/OpenCopilot/issues/173#issuecomment-1777729704, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A25HYSZF4VRYRORFKLQEHXDYA75KPAVCNFSM6AAAAAA6KY6LIOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZXG4ZDSNZQGQ.
You are receiving this because you were mentioned.[image: https://github.com/notifications/beacon/A25HYS42YEBKWYQHMETFEMLYA75KPA5CNFSM6AAAAAA6KY6LIOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTJ6YAKQ.gif]Message ID: @.***>


The following modules couldn't be hot updated: (Full reload needed)

hi, I am trying to use your plugin with my express server. the thing is that, the server runs, but I can't get it to reload my stuff.

EDIT
I added  '?reload=true' in the entry and now it reloads. is this the intended behavior? with this option actually the browser is refreshed but I lose state

I keep getting this warning message in the chrome devtools:
[HMR] The following modules couldn't be hot updated: (Full reload needed)
process-update.js?e13e:64 [HMR]  - ./client/index.js

Here is everything needed to simulate the error (hopefully) with the same code I am using.
package.json
{
  "name": "hot",
  "version": "1.0.0",
  "description": "",
  "dependencies": {
    "express": "^4.13.4",
    "webpack": "^1.12.12",
    "webpack-dev-middleware": "^1.5.1",
    "webpack-hot-middleware": "^2.6.2"
  },
  "devDependencies": {
    "babel-core": "^6.4.5",
    "babel-loader": "^6.2.1",
    "babel-preset-es2015": "^6.3.13",
    "nodemon": "^1.8.1"
  },
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "start": "./node_modules/.bin/nodemon server/app.js"
  },
  "author": "",
  "license": "MIT",
  "private": true
}

webpack.config.js
var webpack = require('webpack');

module.exports = {
  devtool: '#eval-source-map',
  entry: [
    'webpack-hot-middleware/client',
    __dirname + '/client/index.js'
  ],
  output: {
    path: '/',
    filename: 'bundle.js',
    publicPath: 'http://localhost:3000/scripts/'
  },
  module: {
    loaders: [{
      test: /\.js$/,
      loader: 'babel',
      exclude: /(node_modules)/,
      query: {
        presets: ['es2015']
      }
    }]
  },
  resolve: {
    root: process.cwd(),
    modulesDirectories: ['node_modules'],
    extensions: ['', '.js']
  },
  plugins: [
    new webpack.optimize.OccurenceOrderPlugin(),
    new webpack.HotModuleReplacementPlugin(),
    new webpack.NoErrorsPlugin()
  ],
  target: 'web'
};

server/app.js
var path = require('path');
var express = require('express');
var webpack = require('webpack');
var webpackDevMiddleware = require('webpack-dev-middleware');
var webpackHotMiddleware = require('webpack-hot-middleware');
var webpackConfig = require('../webpack.config');
var app = express();
var router = express.Router();
var compiler = webpack(webpackConfig);
var root = path.resolve(__dirname, '../client');
var index = path.join(root, 'index.html');

app.use(webpackDevMiddleware(compiler, {
  publicPath: webpackConfig.output.publicPath,
  stats: {
    colors: true
  }
}));

app.use(webpackHotMiddleware(compiler, {
  log: console.log
}));

router.get('/', (req, res) => res.sendFile(index))

app.use(router);
app.listen(3000, () => console.log('listening on 3000'))

client/index.html
<!doctype html>
<script src="/scripts/bundle.js"></script>

client/index.js
console.log('something');

nodemon.json
{
  "ignore": "client/**/*"
}

$ npm install
$ npm start

This example https://github.com/madole/webpack-dev-middleware-boilerplate which is very similar works correctly, however is using<EMAIL_ADDRESS>

This isn't a bug, but it is a common source of confusion, I'm going to make some changes to the error messaging to try and make this clearer.
This module provides the transportation mechanism between webpack and the browser to enable hot reloading, but this is only part of the puzzle. Your modules also need some code in them to handle the hot reload itself.
http://webpack.github.io/docs/hot-module-replacement-with-webpack.html has full details of how this work.
You can either add a code block into your module to tell it what to do when reloading, or use a babel/webpack plugin that can add this magically for you.

The warning message has been expanded in 52b1929 - will be released as 2.6.3.

great, now i understand how this works :) module.hot.accept() for most cases.. thanks


Restore issue

I get this when trying to restore:
Uncaught (in promise) Object {code: 403, parameter: "REST HTTP error", message: "Forbidden"}

I get the same error. In my case, code: 15 and message "Operation aborted".


Storage quota checker NPE for Dimension Tables

Fixes a null pointer exception on https://github.com/apache/pinot/blob/master/pinot-controller/src/main/java/org/apache/pinot/controller/validation/StorageQuotaChecker.java#L95 because the Helix Resource Manager isn't initialised. That then stops dimension tables from being uploaded.
Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes

Documentation

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e59730a). Click here to learn what that means.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #8132   +/-   ##
=========================================
  Coverage          ?   14.17%
  Complexity        ?       81
=========================================
  Files             ?     1579
  Lines             ?    82262
  Branches          ?    12393
=========================================
  Hits              ?    11660
  Misses            ?    69735
  Partials          ?      867

Flag
Coverage Δ

unittests2
14.17% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../pinot/controller/api/upload/SegmentValidator.java
62.50% <100.00%> (ø)

...not/controller/validation/StorageQuotaChecker.java
78.04% <100.00%> (ø)

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 e59730a...9204c70. Read the comment docs.


Successfully instead ROCm v3.9.x and above, but TF-ROCm does not use GPU

OS: Ubuntu 18.04.5 (5.4 kernel)
GPU: Radeon VII
After successfully installing ROCm v.3.9.x, 2 commands
/opt/rocm/bin/rocminfo
/opt/rocm/opencl/bin/clinfo

both show the same GPU.
However, after installing TF-ROCm, TF-ROCm uses CPU only (checked by monitoring /opt/rocm/bin/rocm-smi). When running TF codes, it shows that GPU is not used because it misses libMIOpen.so and other packages.
Python 3.6.9 (default, Oct  8 2020, 12:12:24)
[GCC 8.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import tensorflow as tf
>>> tf.constant(1)
2020-12-11 04:07:20.679475: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libamdhip64.so
2020-12-11 04:07:20.721488: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1734] Found device 0 with properties:
pciBusID: 0000:05:00.0 name: Vega 20     ROCm AMD GPU ISA: gfx906
coreClock: 1.801GHz coreCount: 60 deviceMemorySize: 15.98GiB deviceMemoryBandwidth: 953.67GiB/s
2020-12-11 04:07:20.724722: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library librocblas.so
2020-12-11 04:07:20.724926: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libMIOpen.so'; dlerror: libMIOpen.so: cannot open shared object file: No such file or directory
2020-12-11 04:07:20.725019: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'librocfft.so'; dlerror: librocfft.so: cannot open shared object file: No such file or directory
2020-12-11 04:07:20.725107: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'librocrand.so'; dlerror: librocrand.so: cannot open shared object file: No such file or directory
2020-12-11 04:07:20.725124: W tensorflow/core/common_runtime/gpu/gpu_device.cc:1753] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
Skipping registering GPU devices...
2020-12-11 04:07:20.725511: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN)to use the following CPU instructions in performance-critical operations:  SSE3 SSE4.1 SSE4.2 AVX
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2020-12-11 04:07:20.733407: I tensorflow/core/platform/profile_utils/cpu_utils.cc:104] CPU Frequency:<PHONE_NUMBER> Hz
2020-12-11 04:07:20.733922: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x4a2fb60 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-12-11 04:07:20.733945: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
2020-12-11 04:07:20.735540: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1257] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-12-11 04:07:20.735561: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1263]
<tf.Tensor: shape=(), dtype=int32, numpy=1>

Hence, after installing missing packages shown in the above output (also suggested by shawonashraf.), GPU is used.
Python 3.6.9 (default, Oct  8 2020, 12:12:24)
[GCC 8.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import tensorflow as tf
tf.cons>>> tf.constant(1)
2020-12-11 04:09:17.198383: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libamdhip64.so
2020-12-11 04:09:17.239802: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1734] Found device 0 with properties:
pciBusID: 0000:05:00.0 name: Vega 20     ROCm AMD GPU ISA: gfx906
coreClock: 1.801GHz coreCount: 60 deviceMemorySize: 15.98GiB deviceMemoryBandwidth: 953.67GiB/s
2020-12-11 04:09:17.243047: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library librocblas.so
2020-12-11 04:09:17.244565: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libMIOpen.so
2020-12-11 04:09:17.259323: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library librocfft.so
2020-12-11 04:09:17.259728: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library librocrand.so
2020-12-11 04:09:17.259858: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1858] Adding visible gpu devices: 0
2020-12-11 04:09:17.260304: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN)to use the following CPU instructions in performance-critical operations:  SSE3 SSE4.1 SSE4.2 AVX
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2020-12-11 04:09:17.268531: I tensorflow/core/platform/profile_utils/cpu_utils.cc:104] CPU Frequency:<PHONE_NUMBER> Hz
2020-12-11 04:09:17.269112: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x5918970 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-12-11 04:09:17.269150: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
2020-12-11 04:09:17.271162: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x5984340 initialized for platform ROCM (this does not guarantee that XLA will be used). Devices:
2020-12-11 04:09:17.271186: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Vega 20, AMDGPU ISA version: gfx906
2020-12-11 04:09:17.708504: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1734] Found device 0 with properties:
pciBusID: 0000:05:00.0 name: Vega 20     ROCm AMD GPU ISA: gfx906
coreClock: 1.801GHz coreCount: 60 deviceMemorySize: 15.98GiB deviceMemoryBandwidth: 953.67GiB/s
2020-12-11 04:09:17.708622: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library librocblas.so
2020-12-11 04:09:17.708660: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libMIOpen.so
2020-12-11 04:09:17.708693: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library librocfft.so
2020-12-11 04:09:17.708724: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library librocrand.so
2020-12-11 04:09:17.708818: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1858] Adding visible gpu devices: 0
2020-12-11 04:09:17.729743: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1257] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-12-11 04:09:17.729784: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1263]      0
2020-12-11 04:09:17.729803: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1276] 0:   N
2020-12-11 04:09:17.729998: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1402] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 15385 MB memory) -> physical GPU (device: 0, name: Vega 20, pci bus id: 0000:05:00.0)

I wonder if this is expected. If not expected, it would be great to put all necessary packages into one place when installing ROCm with Tensorflow.
Thanks

Thanks @quocdat32461997 for reaching out.
I agree and too have the similar doubt :)
Let me try to gather some information on this.

Hi @quocdat32461997
I got an update from developers that: we need to make sure to serve all dependencies for tf to work.
So we need to install rocm-dkms, rocm-libs and rccl before launching tf.
Hope this helps.
Thank you.


Unreadable month names with LC_ALL = bg_BG.UTF-8 and some suggestions

Hello,
I've just started testing LubeLogger and after successfully importing my data, I noticed that the visualisation on the dashboard shows gibberish where the months names should be:

I've installed the app using Docker compose and the .env file contains these settings:
LC_ALL=bg_BG.UTF-8
LANG=en_UK.UTF-8

Also, the fuel mileage chart contains some bizzare numbers, both on the axis and on the tooltips:

A couple of tiny suggestions (from a guy, who makes his living in analytics):

the legend on the bar chart is misleading: it's green, but the bars have variable colours; either remove the legend altogether, or make the bars a uniform colour;
the bar colours make no sense: why would the month with the lowest expenses be red? IMO, either change the scale from red (high expenses) to green (low expenses) or opt for a uniform colour.
typically, when laying out tables, measures (i.e. numbers) and the respective column headers would be aligned to the right, whereas dimensions (categories) - to the left.

Thank you for all your hard work on LubeLogger!

Can you try
LC_ALL=en_BG
LANG=en_BG
?
I think the problem stems from mixing and matching locale and languages. This seems to render currency in European format and BGN as the currency formatter, but it uses English month names.
And thanks for the barchart suggestions, I do agree that the colors are misleading, they actually aren't tied to the values but rather they are hardcoded in a gradient, I will get that fixed.

I have also fixed the bar chart, so now it displays highest cost is red and lowest cost is green. It's the opposite for the fuel mileage chart, highest is green and lowest is red:

Brilliant! The month names are shown in English now, with LC_ALL=en_BG, and the colours of the bar charts make sense.
Thank you, @hargata !


(WIP) Graph representation

This is the WIP for #235:

[ ] Which commits to split in other PR(s)?
[ ] Better naming structure (currently lang.opt.CFA#CFA.graph and GraphRepresentation.phi)?
[ ] Better graph representation (currently a bunch of Maps from lhs to rhs)?
[ ] How to write a spec to verify results?
[ ] Readable pretty-printing / visualization of the graph?

I will split this and move the graph-related commit to another branch.


Clamping Model to ground causes issues with Ellipsoids

Project Background: Application we are developing is consuming real-time data and passing cartographic data into Cesium. We're using SampledPositionProperty to make use of interpolation and are setting the timestamp to now() + 5 seconds (to ensure we always have enough data for accurate interpolation paths. To make the code more straight forward, we are passing in the cartographic data into Cesium (including altitude). Due to some slight issues with our external program we were using to generate paths, the altitudes were being miscalculated. We worked around this by using HeightReference.CLAMP_TO_GROUND for our entities with 3D models associated to them.
Problem: When HeightReference.CLAMP_TO_GROUND is being used for entities that have an altitude assigned to them, any ellipsoid being drawn around the object is it the Cartesian3 height reference instead of the height the entity is being rendered at. Sounds like an edge case, but I would assume the same height value used for drawing would be used across all attributes of an entity. This also causes the trackedEntity marker to be placed in-between the model and the displayed ellipsoid.
    var viewer = new Cesium.Viewer('cesiumContainer', {
        infoBox : true,
        selectionIndicator : true,
        shadows : true
    });

    function createModel(url, height) {
        viewer.entities.removeAll();

    var position = Cesium.Cartesian3.fromDegrees(-123.0744619, 44.0503706, height);
    var heading = Cesium.Math.toRadians(135);
    var pitch = 0;
    var roll = 0;
    var hpr = new Cesium.HeadingPitchRoll(heading, pitch, roll);
    var orientation = Cesium.Transforms.headingPitchRollQuaternion(position, hpr);

    var entity = viewer.entities.add({
        name : url,
        position : position,
        orientation : orientation,
        model : {
            uri : url,
            minimumPixelSize : 128,
            heightReference: Cesium.HeightReference.CLAMP_TO_GROUND,
            maximumScale : 20000
        },
        ellipsoid : {
            radii : new Cesium.Cartesian3(20, 20, 20),
            material : Cesium.Color.BLACK,
            outline : true,
            outlineColor : Cesium.Color.BLACK
        },

    });
    viewer.trackedEntity = entity;
}

var options = [{
    text : 'Ground vehicle',
    onselect : function() {
        createModel('../../SampleData/models/CesiumGround/Cesium_Ground.glb', 100);
    }
}];

Sandcastle.addToolbarMenu(options);

Hello @Zizekftw,
Some primitive types like the Ellipsoid do not have support for ground clamping.  If you need this behavior, I would recommend using a glTF model of an ellipsoid instead.
In the future, please post things like this on our forum: https://groups.google.com/forum/?hl=en#!forum/cesium-dev
Thanks!


Unwanted border around social icons in footer.

Describe the bug
There is an added border around the social icons which makes the UI look broken.
Expected behavior
Either border must not be there or icons must be aligned in center. ( I prefer it should be without border)
Screenshots

Are you working on this issue? (Yes/No) Yes

@arpandesai0 I figured out the solution for this. Please assign this to me.

@Trupti0406 I am already on it and have made pull request for the same.

#2898

#2898


[Custom request] Increase test coverage to cover boot sequence

Using @MicronautTest is not working as intended because the operator needs a kubernetes mock server to be setup before the application is spawned. This is default to achieve with @MicronautTest. An easy way to get around this is by utilising testContainers. However, it is an approch that would be a last resort due to the added complexity of setting up a seamless flow for this.
Best approach, enable a CRUD kubernetes mock  server to be initialised before application injection takes place.

possible venue to explore with micronaut 3.6.0 release: (TEST RESOURCES)

https://micronaut.io/2022/08/04/micronaut-framework-3-6-0-released/
https://micronaut-projects.github.io/micronaut-test-resources/snapshot/guide/#introduction


align wording

Preview is available here:http://developers-pr175.previews.kube001.services.stellar-ops.com


Report definition for expressions

How to create a report definition for an expression? I am trying the following:
session.solver.tui.solve.report_definitions.add(
"dp2",
"single-val-expression",
"AreaAve([StaticPressure, inlet])-AreaAve([StaticPressure, outlet])",
"quit"
)

@ehsansadeghi1, please use this command to define the report definition with an expression:
tui.solve.report_definitions.add('dp2','single-val-expression','define',""AreaAve(StaticPressure, ['inlet'])-AreaAve(StaticPressure, ['outlet'])"",'q')

We should add a note or example for this in our documentation.


Merge latest changes from my migrate-to-csproj fork

I'm not expecting CI to work because I expect Travis doesn't support Mono 4.8.0 yet. However, it works on my machine! :smile:

Getting closer! The OSX build succeeds now but the Linux build fails. It looks like the issue is that MSBuild isn't available in Mono 4.8.0 on Ubuntu.

OK. Sounds like there's an "msbuild" package that needs to be installed for Linux.

And... it looks like the msbuild package isn't recent enough to support the latest .NET Core projects. :-(

Xamarin has updated the msbuild Debian package. I think it might work now!

OK. I'm going to go ahead and pull this into the migrate-to-csproj branch, now that it's working.


PowerRename

Description of the new feature / enhancement
Batch rename with a suffix and a prefix
Scenario when this would be used?

Supporting information
No response

Hello and thanks for your message. PowerRename is very powerful, especially with Regular Expressions. Have a look at the documentation for a few examples.


Make dedicated documentation site

This can be hosted on https://scandeval.github.io, like the leaderboards. It should involve things like:

How to benchmark a model;
Description of the datasets that models can be benchmarked on;
How to contribute with new datasets.

The beginning of this has been initiated now, with the new mkdocs-material documentation backend. It's available at:
https://scandeval.com/ScandEval/
Currently only the readme and API documentation is there, however, but the other parts can be added by simply adding markdown documents. So the next step is to write these.


Switch to YAXArrays

As in title.
Replace all use of NamedDims/AxisKeys to YAXArrays.
Simplifies many things including access of Zarr/netCDFs.

@ConnectedSystems am I crazy or this should be closed?


migrate to AndroidX

Summary
Added support for AndroidX included in React Native 0.60.0

Test Plan

What's required for testing (prerequisites)?
What are the steps to reproduce (after prerequisites)?
Compatibility

OS
Implemented

iOS
✅❌

Android
✅❌

Checklist

[ ] I have tested this on a device and a simulator
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typed files (TS and Flow)
[ ] I added a sample use of the API in the example project (example/App.js)

hi! Thank you for your PR. I'd prefer people to use the approach mentioned in https://github.com/react-native-community/react-native-google-signin/issues/688#issuecomment-507370752 so I'll close this. Please let us know if that approach does not work for some reason. Thanks.

@vonovak The jetifier approach is a workaround, not a long term solution.

Please merge it asap. With the introduction of size optimization through hermes,  a lot of developers are moving towards rn 0.60.x

@thedeepanshujain thanks and sorry for taking so long. I first want to test the PR and ideally update the example app to 0.60 or 0.61 before making a new major. With jetifier nothing is keeping you from using the current version with 0.60.
If you or someone else could, based on this PR, update the example app (and contribute a PR) and confirm all works that would help to make the merge faster.

This merge is available on npm ? I have this error with 3.0.3

@andreslargo yes, this is available since 3.0.0

I still have the same error with 3.0.3


Sitemap.xml Response ContentType

Hello,
Based on a couple of threads found online, i.e. https://stackoverflow.com/a/3272572, it seems it's recommended to put "application/xml" instead of "text/xml" for the response content type of any sitemap.xml files. This way, the encoding is automatically determined based on the XML document defined charset.
Tools such as this one also reports similar issues: https://www.xml-sitemaps.com/validate-xml-sitemap.html.
Would it be possible to either change the default or offer an option to change the response content type? As I've found inside the source code, under the SitemapMiddleware.cs file, it's currently hardcoded with "text/xml".
Thanks!

Hi @ddprince-yaksa
Interesting... I had not expected such a difference between the two. I'll check on this and create a new version so that this can get fixed or at least changed with a config.

Hi @ddprince-yaksa
This issue should now be resolved in the 2.1.1 version of SeoToolkit. If you could try and verify that it also works for you, then that would be greatly appreciated 🙏

@patrickdemooij9 I just checked and it seems all good! Thanks again! :)


bugs: fixes toggle menu

Closes #5645

@ravjot07 you need to sign the Plone Contributor Agreement to merge this pull request.
Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

@ravjot07 you need to sign the Plone Contributor Agreement to merge this pull request.
Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

@ravjot07 you need to sign the Plone Contributor Agreement to merge this pull request.
Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

I have already signed Plone Contributor Agreement

@ravjot07 please read and follow Contributing to Plone and First-time contributors, specifically Sign and return the Plone Contributor Agreement for the time.

My PR closes this issue.

@ravjot07 from that link I provided.

After a member of the Plone Foundation reviews and accepts your signed agreement, your GitHub account will be added to a team in the Plone GitHub organization with appropriate access. This process may take a few business days.

When you receive that email, then you may add another comment to this pull request to notify others that this pull request is ready for review. However you will also need to follow this tip to update your pull request against the main branch before anyone will take a look.

@ravjot07 from that link I provided.

After a member of the Plone Foundation reviews and accepts your signed agreement, your GitHub account will be added to a team in the Plone GitHub organization with appropriate access. This process may take a few business days.

When you receive that email, then you may add another comment to this pull request to notify others that this pull request is ready for review. However you will also need to follow this tip to update your pull request against the main branch before anyone will take a look.

@stevepiercy Can u look at my agreement I have signed it more than 15 days ago and I already recieved mail from Paul Roeland from plone team and already a member of zope foundation

@stevepiercy Can u look at my agreement I have signed it more than 15 days ago and I already recieved mail from Paul Roeland from plone team and already a member of zope foundation

Plone and Zope are two different organizations with different contributor agreements. This pull request is in the Plone organization.
I suggest that you contact the same email address to which you sent your agreement, whichever one it was, as well as verify that your username and email address that you use in GitHub match those which you put on your agreement.

Acceptance Tests / Core Volto Slate (20.x) (pull_request) Failing
Sir how can i resolve this issue

@ravjot07 it's OK not to call me "sir". I was never knighted by the King of England. Maybe someday...
Until then, what do you see when you click the link forDetails of the failing test?
Sometimes tests in GitHub Workflows fail because of network issues. What happens if you re-run the failing test? I don't know if you have permission to do that.
Do you run tests locally before pushing commits to your pull request?
Did you update your branch with main?

@ravjot07 it's OK not to call me "sir". I was never knighted by the King of England. Maybe someday...
Until then, what do you see when you click the link forDetails of the failing test?
Sometimes tests in GitHub Workflows fail because of network issues. What happens if you re-run the failing test? I don't know if you have permission to do that.
Do you run tests locally before pushing commits to your pull request?
Did you update your branch with main?

Maybe someday...
All Test passed......

@ravjot07 we do use that utility function because of something... Removing them with custom code is not ok.
We do have another utility function, more complete one in the code. Try to use that one: https://github.com/plone/volto/blob/316cadc02acfd09c59cd8d0430b964ce5f934968/packages/volto/src/helpers/Utils/useDetectClickOutside.js

@ravjot07 stop! Read and follow item 3 in Things not to do. Repeat offenses may result in you getting blocked from the organization. Thank you for respecting our community.

@ravjot07 stop! Read and follow item 3 in Things not to do. Repeat offenses may result in you getting blocked from the organization. Thank you for respecting our community.

There was some kind of synchronisation issue from my end my commit were not showing that's why I was trying different approaches. Okiee Steve I will be more carefull next time

@stevepiercy I have deleted this fork so i have to crate a new PR from my new Fork for this isssue Is that ok? And can you guide me why my test are failing as i have made very little change so i can be careful next time working on the issue

As a contributor, you must read how to be a contributor to the project, and learn how to use git and GitHub. This is an open source software community, not a place to be mentored by an individual.