DXF export produces strange lines

I tried to use your tool to generate a 6 tooth pinion using your online instance. I clicked chose DXF for format. It was named svg but the file content was clearly dxf.
Autodesk Inventor (2014) imported it like this:

You wrote that the idea was to use arc's only to ease machining. I'm appreciating this approach. The generated dxf contains many splines (the dotted lines lead to the anchors of the splines). IMHO splines are always the beginning of disaster, because there interpretation is highly software dependent. Do you use splines for the svg as well?
Thanks for clarification,
Tobias

Send me the link to your gear in gearcutter and I'll have a look.
The tool generates only circular arcs.  In DXF, though, these are represented by SPLINE objects   They're "rational b-splines" with the parameters set to make perfect circular arcs.  DXF has a separate primitive for circular arcs, but I don't use it, because the way that the arcs are specified causes disconnected outlines that break Fusion 360.  2014 was a long time ago – it might be that your version of inventor doesn't understand the format properly.
The misnaming of the downloaded file seems very strange.  What browser do you use?
Cheers,
Matt

From: Tobias @.>
Sent: January 14, 2024 4:30 PM
To: mtimmerm/gearcutter @.>
Cc: Subscribed @.***>
Subject: [mtimmerm/gearcutter] DXF export produces strange lines (Issue #1)
I tried to use your tool to generate a 6 tooth pinion using your online instance. I clicked chose DXF for format. It was named svg but the file content was clearly dxf.
Autodesk Inventor (2014) imported it like this:
image.png (view on web)https://github.com/mtimmerm/gearcutter/assets/72009731/6ae44668-07de-4313-acde-8a90fdb53cf7
You wrote that the idea was to use arc's only to ease machining. I'm appreciating this approach. The generated dxf contains many splines (the dotted lines lead to the anchors of the splines). IMHO splines are always the beginning of disaster, because there interpretation is highly software dependent. Do you use splines for the svg as well?
Thanks for clarification,
Tobias
—
Reply to this email directly, view it on GitHubhttps://github.com/mtimmerm/gearcutter/issues/1, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABXYLQRP4S3UJT2HVN74NDTYORE65AVCNFSM6AAAAABB2MBM76VHI2DSMVQWIX3LMV43ASLTON2WKOZSGA4DAOJQGQ4TMMQ.
You are receiving this because you are subscribed to this thread.

Oh, I forgot to answer your question about SVG:  No, in SVG I use the circular arc primitive, which doesn't have the problem that the DXF primitive does.  Because of the way SVG makes arcs, though, rounding errors in the SVG might create tiny line segments between arcs that should be connected, depending on how  your tool converts SVG arcs to its own internal format.

From: Tobias @.>
Sent: January 14, 2024 4:30 PM
To: mtimmerm/gearcutter @.>
Cc: Subscribed @.***>
Subject: [mtimmerm/gearcutter] DXF export produces strange lines (Issue #1)
I tried to use your tool to generate a 6 tooth pinion using your online instance. I clicked chose DXF for format. It was named svg but the file content was clearly dxf.
Autodesk Inventor (2014) imported it like this:
image.png (view on web)https://github.com/mtimmerm/gearcutter/assets/72009731/6ae44668-07de-4313-acde-8a90fdb53cf7
You wrote that the idea was to use arc's only to ease machining. I'm appreciating this approach. The generated dxf contains many splines (the dotted lines lead to the anchors of the splines). IMHO splines are always the beginning of disaster, because there interpretation is highly software dependent. Do you use splines for the svg as well?
Thanks for clarification,
Tobias
—
Reply to this email directly, view it on GitHubhttps://github.com/mtimmerm/gearcutter/issues/1, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABXYLQRP4S3UJT2HVN74NDTYORE65AVCNFSM6AAAAABB2MBM76VHI2DSMVQWIX3LMV43ASLTON2WKOZSGA4DAOJQGQ4TMMQ.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thank you for your quick response.
If you click on
https://mtimmerm.github.io/webStuff/gearcutter.html
or click on the 6 teeth gear link
https://mtimmerm.github.io/webStuff/gearcutter.html#cr=1.5&gt=30&pt=6&ps=40
it happens the same, the file has the ending .svg.
I'm using Firefox on OSX:

It is true that my Autodesk version is pretty old. I dont like their new licence model so 360 is not an option for me. Autodesk is the inventor of the dxf format, I would assume they still have people who know how to decode it.
You're right in not using the circle primitive. I think you have to use the polyline with the bulge feature. Nearly all machine interfaces use dxf - exactly for that reason: to have an uninterupted path consisting of lines and arcs. GCode M2 and M3 have the same logic like svg, use implicitly the current point and specify the endpoint of the next segment - even if it is an arc. Hopefully I have next week some time to look into it. I'll will report my findings.

Cambam displays your file correctly. I transformed it (using Cambam) into a polyline and exported it again as dxf. Now it is usable by autodesk. So I have a working solution :-)
Still I still think the polyline would be the "nicer" solution.
github doesnt like dxf's, I'm trying as zip:
pinion6pa20.zip
Kind regards
Tobias

Ah, thanks!  I didn't know about the polyline bulge feature in DXF.  That's actually how I represent the arcs internally, and I would like that a lot better than using NURBS.  I'll switch it when I get a chance, if it works just as well in Fusion 360.

From: Tobias @.>
Sent: Monday, January 15, 2024 1:35 PM
To: mtimmerm/gearcutter @.>
Cc: Matt Timmermans @.>; Comment @.>
Subject: Re: [mtimmerm/gearcutter] DXF export produces strange lines (Issue #1)
Cambam displays your file correctly. I transformed it (using Cambam) into a polyline and exported it again as dxf. Now it is usable by autodesk. So I have a working solution :-)
Still I still think the polyline would be the "nicer" solution.
github doesnt like dxf's, I'm trying as zip:
pinion6pa20.ziphttps://github.com/mtimmerm/gearcutter/files/13943171/pinion6pa20.zip
Kind regards
Tobias
—
Reply to this email directly, view it on GitHubhttps://github.com/mtimmerm/gearcutter/issues/1#issuecomment-1892788133, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABXYLQXWK5URQJSLIR3TUBDYOWOKJAVCNFSM6AAAAABB2MBM76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJSG44DQMJTGM.
You are receiving this because you commented.Message ID: @.***>

export extension fixed.  I was sending the SVG Content-Type with the DXF download, and firefox actually renames the file to match it.

Switched DXF export to a simpler version R12 format with POLYLINEs

Works as expected. Just One closed polyline.
Thank you!


Cannot find path 'Azure:/azuredeploy.json' -- workaround

re: Chapter "Deploy the template", p.4, Note with workaround description
There is one much simpler workaround:
Use "$HOME/azuredeploy.json" instead of "azuredeploy.json" in a command-line

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

ID: 30ef211a-c625-61cd-9ae9-ca63e32ff4d2
Version Independent ID: b60685c0-1d9c-ebcb-4393-e231d7b09c24
Content: Use Visual Studio Code to create Azure Resource Manager template
Content Source: articles/azure-resource-manager/resource-manager-quickstart-create-templates-use-visual-studio-code.md
Service: azure-resource-manager
GitHub Login: @mumian
Microsoft Alias: jgao

@Georg-M Thanks for the feedback!
@mumian Can this be considered as a doc-enhancement? Please confirm.

@Georg-M - thanks for the suggestion.  The changes will go live later today.
#please-close


Error in the traduction to portuguese

Describe the bug
A clear and concise description of what the bug is.
When I use the app in the portuguese appear some phrases in Arabic
To reproduce
Steps to reproduce the behavior:

Go to 'Settiings'
Click on 'Preferences and switch the language to portuguese'
Go to 'Settings again'
See error

Expected behavior
A clear and concise description of what you expected to happen.
It is expected that the translation will be done correctly
Log file
Please upload the log file here so that we can understand your issue better.

You can access it from Settings->About->Open log location.
or
%localappdata%\Packages\49306atecsolution.FilesUWP_et10x9a9vyk8t\LocalState

You should see a file in this directory called debug.txt or/and debug_fulltrust.txt.
https://drive.google.com/file/d/1GPQ7R9eYICqFb_HhcqCGcbUvrWu4P6R-/view?usp=sharing
Screenshots
If applicable, add screenshots to help explain your problem.
https://drive.google.com/file/d/1Vt6znqvnM7F0_kWpgy5p_kxR6YyaOS32/view?usp=sharing
System Information

OS Version: [e.g. Windows 10 20H2 18363.14440] (Press WIN + PAUSE/BREAK, use the copy button at the bottom)
App version: [e.g. v<IP_ADDRESS>] (In the app, go to Settings -> About, the version is shown at the top)

Additional context
Add any other context about the problem here. Does this problem occur again after restarting the app?
Everytime

This issue should be resolved with #4131.


Unexpected alter table in auto migration

GORM Playground Link

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

Same problem


Lakes and Cities

Attached are 150 BC places to support the initial location options based on the user picking their destination from a dropdown.
An additional 7000 water bodies names are provided to support any aquatic related locationing or autofilling.
https://app.zenhub.com/files/296689811/0a318189-8b89-41cf-a86e-eeb919e07565/download
https://app.zenhub.com/files/296689811/13da3a51-c35f-40bf-af3b-0016bce311d7/download

@micheal-w-wells  we should discuss this as I don't think it would be necessary anymore if we can take the names from the layers?


[Istio] k8s IP service is set to default IP which does not work for all environments

Rancher Server Setup

Rancher version: v2.6.2
Installation option (Docker install/Helm Chart): N/A

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details: N/A
Rancher-Istio Chart Version: Starting in 100.0.2+up1.10.4

Describe the bug
Recent changes to the istio-installer resulted in the kubeconfig using the default IP for the k8s service. This is causing issues if you are not using the default IP address for the k8s service because the istio-installer is not able to find it.
Expected Outcome
Do not set the server address to a static variable in the istio-installer - instead utilize the existing ENV var or allow users to set it

Available to test on v2.6.3-rc1

Verified on v2.6.3-rc2

Created a node driver rke1 AWS 3 worker, 1 etcd and 1 cp cluster
Added the following in the edit as yaml while creating the cluster

    kube-api:
      service_cluster_ip_range: <IP_ADDRESS>/16
    kube-controller:
      service_cluster_ip_range: <IP_ADDRESS>/16
    kubelet:
      cluster_dns_server: <IP_ADDRESS>

Verified the kubernetes cluster IP is set to <IP_ADDRESS>
Installed istio in the downstream cluster version 100.0.2+up1.10.4
Verified the following error is seen

unable to retrieve Pods: Get "https://<IP_ADDRESS>/api/v1/namespaces/istio-system/pods?fieldSelector=status.phase%3DRunning&labelSelector=app%3Distiod": dial tcp <IP_ADDRESS>:443: i/o timeout
--
Mon, Nov 29 2021 8:59:15 pm | 1.10.4
Mon, Nov 29 2021 9:01:45 pm | Unable to connect to the server: dial tcp <IP_ADDRESS>:443: i/o timeout
Mon, Nov 29 2021 9:02:45 pm | 2021-11-30T05:02:45.496357Z error analysis error getting mesh config from running kube source: Get "https://<IP_ADDRESS>/api/v1/namespaces/istio-system": dial tcp <IP_ADDRESS>:443: i/o timeout

echo $KUBERNETES_SERVICE_HOST
<IP_ADDRESS>

Uninstalled the istio version and re-installed the version 100.1.0+up1.11.4
Installation went through fine without any error but kiali crd is not installed https://github.com/rancher/rancher/issues/35686

Linked issue is only a docs issue. As the validations look god for this ticket, closing.

Verified on v2.5-head

Created a node driver rke1 AWS 2 worker, 1 etcd and 1 cp cluster
Added the following in the edit as yaml while creating the cluster

    kube-api:
      service_cluster_ip_range: <IP_ADDRESS>/16
    kube-controller:
      service_cluster_ip_range: <IP_ADDRESS>/16
    kubelet:
      cluster_dns_server: <IP_ADDRESS>

Verified the kubernetes cluster IP is set to <IP_ADDRESS>
Installed istio in the downstream cluster version 1.10.400.
Installation went through fine without any error.


Adjust click area for series title inline edit

Keep clicking accidentally on the hotspot, with is 80% of the width I think at the monent.
We should optimise this so that the hotspot for invoking the inline edit should only be the length of the title string.

Assigning to @bp323 for review

Coverage remained the same at 100.0% when pulling 42be7cd384bb7ac40b07b41649fa7706282758c1 on Coenego:issue-277-click-area into d242efbe424b55317b36e19c03e24e47419de7e4 on CUL-DigitalServices:master.

Reassigning to @Coenego for a fix

Follow-up done, reassigning to @bp323 for review

Coverage remained the same at 100.0% when pulling 9f02ac5a7931218b1ccf6e6419f5870b0ea4bef5 on Coenego:issue-277-click-area into d242efbe424b55317b36e19c03e24e47419de7e4 on CUL-DigitalServices:master.

Back to @Coenego for minor followup

Follow-up done, reassigning to @bp323

Coverage remained the same at 100.0% when pulling c81cdc5380f26b9aa6dd5786d1777286768634c9 on Coenego:issue-277-click-area into d242efbe424b55317b36e19c03e24e47419de7e4 on CUL-DigitalServices:master.

Verified and merged.


Create a UI Using Freestyle SAPUI5

Tutorials: https://developers.sap.com/tutorials/btp-app-create-ui-freestyle-sapui5.html
In the optional Step 4 (for Windows) the fourth instruction asks the user to add the ui5-middleware-simpleproxy and ui5-middleware-livereload lines
From my experience following this, if you add these without removing the existing fiori lines, then using ui5 serve later on will not be successful. The existing entries in this ui5.yaml file need to be removed first. It could be helpful to point this out

Correct, also the package.json looks a bit different.
My excercise/state of files can be found in following commit: btp-application-cap-e2e
In short, package.json:

add both ui5 middleware modules
add the ui5 dependency block

As mentioned by CJSwift, delete the fiori tools config in ui5.yalm and add the middleware configuration.


[dkr] Fix service account auth with gsutil

gsutil needs this config before it will look for a service account
from instance metadata :-(

/land

:lock: Merge Conflict

/land

:broken_heart: Test Failed - Build images and run cluster test

/land

:broken_heart: Test Failed - Build images and run cluster test

/land

:broken_heart: Test Failed - Build images and run cluster test


Use separate lifetimes to avoid false borrowing

The example is not possible unless the API is updated to be more general. Using the same lifetime variable means that you tie together the two borrows, which is wrong (it's not what you actually want).

:+1: Like it!

Updated to use lifetime elision. Maybe it's more confusing, but it's the idiomatic thing to do. It's also the least typing -- the default &mut self -> &mut does the right thing for us!

Thank you. I doubt I would have spotted that error myself. Merged! :)

great.

You'd probably run into it eventually.

I guess, yeah. Thank you anyways!

This was to help @est31, this crate came up in #rust.

Aha, I saw the logs now. @est31: I think you're the first one to use this (apart from me, that is). Feel free to open an issue if you feel something is odd or some feature is missing. I'd love any feedback!


A Linux distribution for docker-compose

As I know，the image of  apache/dolphinscheduler was base on the Alpine image。But when I install Python dependencies，It takes a lot of time，at least an hour。What's more，It will need more any other  dependencies，for example
apk update && apk add --no-cache gcc g++ python3-dev libffi-dev librdkafka librdkafka-dev mariadb-connector-c-dev musl-dev libxml2-utils libxslt  libxslt-dev py3-numpy py3-pandas
But in the end, I fail to install python dependencies，because of the pandas。
So can you build another version based on other Linux，such as slim。 constrast
Thank you very much ！

@Joder5 This is not a problem. As for a any bare linux,  none of the libraries you mentioned will exist.
In order to improve the update speed, you can set the mirror source of alpine as follows:
# 1. install command/library/software
# If install slowly, you can replcae alpine's mirror with aliyun's mirror, Example:
# RUN sed -i "s/dl-cdn.alpinelinux.org/mirrors.aliyun.com/g" /etc/apk/repositories
# RUN sed -i 's/dl-cdn.alpinelinux.org/mirror.tuna.tsinghua.edu.cn/g' /etc/apk/repositories

As for pip, you can also use the mirror source like https://pypi.tuna.tsinghua.edu.cn/simple
pip install --no-cache-dir -i https://pypi.tuna.tsinghua.edu.cn/simple

As for slim, we will consider it later.

I give you a version of Dockerfile based on debian:slim, which will be added to the warehouse after optimization in the future.
#
# Licensed to the Apache Software Foundation (ASF) under one or more
# contributor license agreements.  See the NOTICE file distributed with
# this work for additional information regarding copyright ownership.
# The ASF licenses this file to You under the Apache License, Version 2.0
# (the "License"); you may not use this file except in compliance with
# the License.  You may obtain a copy of the License at
#
#     http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#

FROM openjdk:8-jdk-slim

ARG VERSION
ARG DEBIAN_FRONTEND=noninteractive

ENV TZ Asia/Shanghai
ENV LANG C.UTF-8
ENV DOCKER true

# 1. install command/library/software
# If install slowly, you can replcae alpine's mirror with aliyun's mirror, Example:
RUN echo \
"deb http://mirrors.tuna.tsinghua.edu.cn/debian/ buster main contrib non-free\n\
deb http://mirrors.tuna.tsinghua.edu.cn/debian/ buster-updates main contrib non-free\n\
deb http://mirrors.tuna.tsinghua.edu.cn/debian/ buster-backports main contrib non-free\n\
deb http://mirrors.tuna.tsinghua.edu.cn/debian-security buster/updates main contrib non-free" > /etc/apt/sources.list

RUN apt-get update && \
    apt-get install -y tzdata dos2unix python python3 procps netcat sudo tini postgresql-client && \
    echo "Asia/Shanghai" > /etc/timezone && \
    rm -f /etc/localtime && \
    dpkg-reconfigure tzdata && \
    rm -rf /var/lib/apt/lists/* /tmp/*

# 2. add dolphinscheduler
ADD ./apache-dolphinscheduler-incubating-${VERSION}-dolphinscheduler-bin.tar.gz /opt/
RUN ln -s /opt/apache-dolphinscheduler-incubating-${VERSION}-dolphinscheduler-bin /opt/dolphinscheduler
ENV DOLPHINSCHEDULER_HOME /opt/dolphinscheduler

# 3. add configuration and modify permissions and set soft links
COPY ./checkpoint.sh /root/checkpoint.sh
COPY ./startup-init-conf.sh /root/startup-init-conf.sh
COPY ./startup.sh /root/startup.sh
COPY ./conf/dolphinscheduler/*.tpl /opt/dolphinscheduler/conf/
COPY ./conf/dolphinscheduler/logback/* /opt/dolphinscheduler/conf/
COPY ./conf/dolphinscheduler/env/dolphinscheduler_env.sh /opt/dolphinscheduler/conf/env/
RUN dos2unix /root/checkpoint.sh && \
    dos2unix /root/startup-init-conf.sh && \
    dos2unix /root/startup.sh && \
    dos2unix /opt/dolphinscheduler/conf/env/dolphinscheduler_env.sh && \
    dos2unix /opt/dolphinscheduler/script/*.sh && \
    dos2unix /opt/dolphinscheduler/bin/*.sh && \
    rm -rf /bin/sh && \
    ln -s /bin/bash /bin/sh && \
    mkdir -p /tmp/xls /usr/lib/jvm && \
    ln -sf /usr/local/openjdk-8 /usr/lib/jvm/java-1.8-openjdk && \
    echo "Set disable_coredump false" >> /etc/sudo.conf

# 4. expose port
EXPOSE 5678 1234 12345 50051

ENTRYPOINT ["/usr/bin/tini", "--", "/root/startup.sh"]
``

close by #5158


fixes a minor markdown error in filters.md

@codeblech, thank so much for this.

sorry i had to be that guy 💀

@codeblech, not at all. The docs are getting larger and larger, so I will inevitably make mistakes.


v1.10 backports 2021-07-01

#15650 -- daemon: Add option --bpf-lb-external-clusterip (@joamaki)
#16662 -- k8s: Fix External Workloads service access (@jrajahalme)

Once this PR is merged, you can update the PR labels via:
$ for pr in 15650 16662; do contrib/backporting/set-labels.py $pr done 1.10; done

@joamaki I had to conflicts in the following files to merge #15650. Could you check that the changes in this PR look correct?

Documentation/gettingstarted/kubeproxy-free.rst
install/kubernetes/cilium/README.md
pkg/option/config.go

Ah, I see that there's an existing backport PR in progress (#16686) that needs to be merged before this one can be. Reverting to draft for now.

I've undone the issue label changes, and will do a new backport PR once #16686 is in. Closing.


fixing django dependency in xmacheck-utils

removed django's dependency from xamcheck-utils requirement .

Closing this due to wrongdoing of replacing ValidationError with AttributeError


Add option to display search results for metabolites, RNA, etc. that have no associated data

Element as check box in the left column (beneath the TOC)

@yosefdroth @lzy7071 this doesn't seem critical to me. Do agree that we can leave this to the future?

@yosefdroth @lzy7071 this doesn't seem critical to me. Do agree that we can leave this to the future?

I agree.

I agree too


[bug] GPU instance launch failure on GCP

Describe the bug
This issue has been raised directly with a google but documenting here for posterity and to report the solution if we figure out a workaround on our side.
Basic description... the nvidia daemonset has been installed and the node group has the GPU accelerator set up but the pod failures during initialization. It turns out that the driver installation on the pod fails. Example log:

    I0618 05:24:08.127227    6025 main.go:21] Checking if this is the only cos_gpu_installer that is running.                                                                                │
│ I0618 05:24:08.128658    6025 install.go:74] Running on COS build id 13310.1209.12                                                                                │
│ I0618 05:24:08.128675    6025 installer.go:187] Getting the default GPU driver version                                                                                │
│ I0618 05:24:08.128733    6025 utils.go:72] Downloading gpu_default_version from https://storage.googleapis.com/cos-tools/13310.1209.12/gpu_default_version                                                      │
│ E0618 05:24:08.177340    6025 artifacts.go:106] Failed to download gpu_default_version from public GCS: failed to download gpu_default_version, status: 403 Forbidden                                           │
│ E0618 05:24:08.177938    6025 install.go:175] failed to get default driver version: failed to get default GPU driver version: failed to download artifact gpu_default_version: failed to download gpu_default_v

For some reason the storage is inaccessible (despite access privileges being set to admin). The extra detail that make this fascinating is that the url is accessible from other pods on the cluster. i will update when we have a solution...

This was debugged with google support. It seems that the "GPU issue has been fixed after applying the https://www.googleapis.com/auth/devstorage.read_only ". Peculiar... but we have a GPU pool that now launch successfully.


B1 - DevOps - auto deploy branch state for front purposes

Hello team,
Just merged my local branch with upstream/sprint-1 and after testing some CI/CD changes
build process with maven fails because checkstyle  tests fail.
Error:  /home/runner/work/Banka-1-Backend/Banka-1-Backend/user-service/src/main/java/rs/edu/raf/banka1/controller/UserController.java:1: File does not end with a newline. [NewlineAtEndOfFile] Error:  /home/runner/work/Banka-1-Backend/Banka-1-Backend/user-service/src/main/java/rs/edu/raf/banka1/controller/UserController.java:10:47: Using the '.*' form of import should be avoided - org.springframework.web.bind.annotation.*. [AvoidStarImport] Error:  /home/runner/work/Banka-1-Backend/Banka-1-Backend/user-service/src/main/java/rs/edu/raf/banka1/controller/UserController.java:58:6: '{' is not preceded with whitespace. [WhitespaceAround] Error:  /home/runner/work/Banka-1-Backend/Banka-1-Backend/user-service/src/main/java/rs/edu/raf/banka1/controller/UserController.java:59:61: ',' is not followed by whitespace.
Could you please contact authors of these .java files in order to resolve these issue.
Suggested remediation for this type of bug is potential soulition. Authors can also follow the stack trace and try to change the source code accordingly.
Best,
Vid

In the meantime, workaround was found:
mvn -B clean package -Dcheckstyle.skip=true -DskipTests=false
Still, code should be changed in order to pass checkstyle.
Best,
Vid


[Enhancement] Improving the Webpack plugin?

Problem
The Webpack plugin and Next JS plugin have a bunch of issues at the moment. Thanks to help from Tobias Koppers, I have some ideas on how to fix these problems.
#179, #197, #209, #288
Background
StyleX compilation is essentially two steps:

Transform JS files and collect metadata (an array of objects)
Take the metadata from all the JS files and convert them to a CSS file.

We haven't done this right within Webpack and caching doesn't work at all. But we now have some pointers on how to do this right.
Approach 1: Using CSS data imports
We can take the generated metadata from each JS file and convert that in the CSS data file import within the file itself.
Instead of generating:
["xabnbd", { "ltr": ".xabnbd{color: red}" }, 4001]

We would insert this import within the file itself:
import "data:text/css;@layer priority_4001{.xabnbd{color:red}}";

Then, the usual plugins will collect all the CSS from all the files combine them in a CSS file.
(We need to ensure that there is only ever 1 CSS file. Figure out how to avoid bundle splitting for CSS)
This generated CSS file will then need to be processed with a custom PostCSS or LightningCSS plugin to do all the work that is currently done while generating the CSS file. (sorting rules, renaming layers. Removing unused styles etc.)
Challenges

Need to ensure that no bundle splitting happens. We want a single CSS file
We are planning to add new metadata that are not styles but information about which styles are defined and which ones are actually used. We will need to abuse CSS syntax to encode this into CSS itself.
We will need some way to marking StyleX styles to distinguish them from non-stylex styles.
All of the HACKs need to survive post-processing already done.

Possible solutions:

We can use CSS layers with a special __stylex__ prefix and encode priority information in a suffix.

@layer __stylex__priority__3008 {...}

Within such layers, we can use special classNames to mean other things:

.__stylex__VAR_USED {--variable-name: 1}

We can configure Webpack to not split CSS with config that looks like this:

optimization: {
  splitChunks: {
    cacheGroups: {
      styles: {
        name: "styles",
        type: "css",
        chunks: "all",
        enforce: true,
      },
    },
  },
},

Approach 2: Write module metadata and combine it
This is the approach we use with the Rollup plugin today but we haven't done things in Webpack correctly.
The approach is similar to the first one, but here we can use a custom mime type to keep the metadata in JSON format.
We would insert an import into the source JS file that looks like this:
import "data:text/stylex,[\"xabnbd\", { \"ltr\": \".xabnbd{color: red}\" }, 4001]";

A webpack loader can be configured:
{mimetype: "text/stylex", use: "stylex-loader"}

This loader will need to read and attach the metadata to the module being processed:
this._module.buildInfo.stylex ??= [];
this._module.buildInfo.stylex.push(JSON.parse(content))

NOTE: the buildInfo is cached and needs to be serialisable.
Finally a Webpack plugin will need to iterate all the modules from compilation.modules to read the styles and combine them, generate a CSS file and use emitAsset to generate the CSS file.
In the case of HTML files being used, we would need to add the generated CSS to every entrypoint:

compilation.entrypoints / entrypoint.files.

NextJS challenges
Even after all of this, there is a challenge with NextJS generating separate server and client bundles. The current approach handles this in a hacky unreliable way that can't be cached. Both of these approaches may be stuck with separate server and client CSS files which is not what we want. There was a mention of a "manifest" file as a possible way to get around this issue.
Workaround: Pre-compile everything
A third and final solution might be to build a CLI that can take a folder or "source" files and transforms them all along with a CSS file to a "src" folder. This would happen while you program rather than a part of bundling.
In more detail:
A pre-app or pre-src folder with your actual source code. A babel-watch command will transform this folder into app or src, respectively. Which runs in parallel with a standard swc-based next dev command combined with a single PostCSS or LightningCSS plugin.
Initially this won't support importing styles from node_modules, but we will be able to fix that with some import re-writing and configuration as well.

#476 Add this to the list of issues. The Webpack plugin is completely useless at the moment because it interferes with Webpack's tree shaking/optimization.

@nmn FYI, here is currently my implementation (kinda like approach 1 and 2 combined)

stylex-loader collects style rules from jsx/tsx files and append noop css imports to them:

import '/path/to/stylex-noop.css?rules=[serialized and encoded stylex rules]'

Configure code splitting (cacheGroup) targets all stylex-noop.css and merge them into a single file.
Instead of using buildInfo, we can just use module identifier. Inside the processAssets hook (phase PROCESS_ASSETS_STAGE_PRE_PROCESS), we find the stylex chunk (it should contain many stylex-noop.css?rules= modules) and iterates through modules of the chunk, collect and deserialize stylex rules from modules' _identifier.

Actual implementation: stylex-webpack. And this works with Next.js (without disabling SWC).

@SukkaW If your implementation works more reliably than the hacky plugin we have currently, I would love a PR to make it the official implementation going forward.
In the meantime, I'm going to do some testing with it and see how you got around some of the problems I encountered while trying to use that approach.

If your implementation works more reliably than the hacky plugin we have currently, I would love a PR to make it the official implementation going forward.
In the meantime, I'm going to do some testing with it and see how you got around some of the problems I encountered while trying to use that approach.

Sure! IMHO we could do some experiments based on the stylex-webpack project before merging it back to stylex.

@SukkaW Found it now and it seems to work perfectly for us


TeamsUpgradePolicy

Pull Request (PR) description
New Resource TeamsUpgradePolicy
This Pull Request (PR) fixes the following issues
N/A

This change is

Putting this one on the back burner, since it appears that the New, Remove, and Set are only available for On-Premises migrations.

Codecov Report

Merging #273 into Dev will decrease coverage by 0%.
The diff coverage is 1%.

@@          Coverage Diff          @@
##             Dev    #273   +/-   ##
=====================================
- Coverage     90%     90%   -1%
=====================================
  Files        104     105    +1
  Lines      11214   11281   +67
  Branches      10      10
=====================================
+ Hits       10201   10202    +1
- Misses      1003    1069   +66
  Partials      10      10

@ykuijs can you review please? Thanks


[Bug] Merge module only accepts dicomweb sourceName.

Describe the Bug
I have multiple dcm4chee servers I am trying to retrieve data through merge module.
Here is my config file
dataSources: [
   {
     sourceName: 'merge',
     namespace<EMAIL_ADDRESS>     configuration: {
       name: 'merge',
       friendlyName: 'Merge dicomwebDCM and dicomweb data at series level',
       seriesMerge: {
         dataSourceNames: ['dicomweb', 'dicomwebDCM'],
         defaultDataSourceName: 'dicomwebDCM',
       },
     },
   },
   {
     sourceName: 'dicomwebDCM',
     namespace<EMAIL_ADDRESS>     configuration: {
       friendlyName: 'Primary server',
       name: 'server one',
       wadoUriRoot: 'http://example.com/dcm4chee-arc/aets/DCM4CHEE/wado',
       qidoRoot: 'http://example.com/dcm4chee-arc/aets/DCM4CHEE/rs',
       wadoRoot: 'http://example.com/dcm4chee-arc/aets/DCM4CHEE/rs',
      // Rest of the configuration
      },
   },
  {
     sourceName: 'dicomweb',
     namespace<EMAIL_ADDRESS>     configuration: {
       friendlyName: 'secondary server',
       name: 'server two',
       wadoUriRoot: 'http://another-example.com/dcm4chee-arc/aets/DCM4CHEE/wado',
       qidoRoot: 'http://another-example.com/dcm4chee-arc/aets/DCM4CHEE/rs',
       wadoRoot: 'http://another-example.com/dcm4chee-arc/aets/DCM4CHEE/rs',
       // Rest of the configuration
     },
   },

Now when I view UI, the data shown is from server 2 , because I named it dicomweb.
No other name is being accepted. Only the server with sourceName dicomweb is displaying.
And if I dont name any server as dicomeweb, no data is displayed
Steps to Reproduce
1.Pull latest or any other image
2.Start the container.
3.Configure the merge module
The current behavior
Displays data with sourceName=dicomweb
The expected behavior
Should display data from all data sources
OS
CentOs 8 Stream
Node version
Node v20.15.1
Browser
Chrome v126

@Sufian-Afridi What is your defaultDataSourceName? maybe you have used 'dicomweb' there but it should be 'merge'.


Fix find by date for tasks bug

Remove additonal check for date flag

Codecov Report

Merging #155 (87c71a7) into master (bd6046f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #155   +/-   ##
=========================================
  Coverage     45.49%   45.49%
  Complexity      491      491
=========================================
  Files           129      129
  Lines          2462     2462
  Branches        303      303
=========================================
  Hits           1120     1120
  Misses         1254     1254
  Partials         88       88

Impacted Files
Coverage Δ

...du/address/logic/parser/FindTaskCommandParser.java
0.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 bd6046f...87c71a7. Read the comment docs.


Change the red bar under data size that indicates data size is bigger than the download limit

Change the red bar that indicates data size is bigger than the download limit
Summary
In the datasize column there is a red/green bar that indicates if the data is larger or smaller than the 5GB size limit. Many datasets are much larger than the 5GB limit, and most institutes offer a different download mechanism to download this size, however to the user it appears that their data is in an errored state.
It would be nice to change the colour or the experience to indicate which download mechanism to use, rather than having an unexplained red bar. For our users it is important to know the size of data.
Extra Details

@LAShemilt I would remove it completely as it is misleading and does not provide additional information to the user.
It is misleading because the color red indicates that it is not downloadable through the zip-service provided the limit imposed in configuration.
It is not a relevant information for facilities that are not using the zip-service.
Also it does not mean that the dataset has problems.

Something like the image before

That is already possible today just by changing the config.json:
"fileColorEnabled": false

@stephan271 Thank you for the info.
This need to be documented.

I just checked the code. This functionality is implemented here.
Color and sizes are hard coded.
If we want to keep the functionality, we should make color and sizes configurable.
Also, we should investigate a better visualization of the size.

To elaborate further, currently there are the following two entries in the frontend configuration:
  "fileColorEnabled": false,
  "fileDownloadEnabled": true,

I propose to add the following configuration options:
  "fileColorNoIssuesColor": <color> //default to green
  "fileColorWarningLimit": <file_size_limit_in_bytes>
  "fileColorWarningColor": <color> //default to orange
  "fileColorErrorLimit": <file_size_limit_in_bytes>
  "fileColorErrorColor": <color> //default to red

This are colors do not indicate an error in the dataset itself, they just warn the user about transfer (download) time, which is dependent on the type of data, file size and facility IT infrastructure

Sounds reasonable to me


Allow empty functions to be elaborated during translation

Previously, the following code would error during translation:
class Alu(Model):
        def elaborate_logic(self):
            @self.combinational
            def comb():
                pass

This pull request fixes that.

Interesting! Can you maybe explain where this programming pattern is common?

I removed the body of a combinational block to debug something. I don't know if that's common, but it doesn't seem like something the compiler should give a cryptic "Unexpected error during VerilogTranslation!" on.


Split _TORCH_CUDA_ARCH_LIST by either whitespace or semicolon

🚀 The feature
As per title.
Motivation, pitch
In https://github.com/pytorch/audio/blob/e3734feff3df2fd4c81572ba84ca7b8d52c4fab2/tools/setup_helpers/extension.py#L104, _TORCH_CUDA_ARCH_LIST is split by a semicolon.
On the other hand, in https://pytorch.org/docs/stable/cpp_extension.html?highlight=cudaextension#torch.utils.cpp_extension.CUDAExtension and https://github.com/pytorch/pytorch/blob/9dd1824741ab13a13ae84aadf5a0226831a118ee/Dockerfile#L48, TORCH_CUDA_ARCH_LIST uses whitespace as its separator.
Alternatives
No response
Additional context
The following Dockerfile fails to be built.
FROM pytorch/pytorch:1.10.0-cuda11.3-cudnn8-devel

RUN apt update && \
    apt install -y --no-install-recommends git sox libsox-dev

RUN conda install -y cmake

RUN pip install --no-cache-dir sox kaldi_io

RUN git clone --depth 1 https://github.com/pytorch/audio && \
    cd audio && \
    git submodule update --init --recursive && \
    TORCH_CUDA_ARCH_LIST="7.0 7.5 8.0 8.6+PTX" USE_CUDA=1 python setup.py develop

Hi @crcrpar
Thanks for bringing this up.
@malfet I went ahead and made https://github.com/pytorch/audio/pull/2050. Could you take a look?


[Support] Compatibility with Python3.10

Description
Trying to add python3.10 support on eox-hooks fails due to openedx-events.
Versions

ENV: Python 3.10.6

Error
ERROR: test_receiver_called_after_cert_change (eox_hooks.tests.test_receivers.TestCohortEventsHandler)
Test that hooks_handler is called the correct information after sending
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/django/test/utils.py", line 387, in inner
    return func(*args, **kwargs)
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/mock/mock.py", line 1346, in patched
    return func(*newargs, **newkeywargs)
  File "/home/edunextdev/Documents/edunext/eox-hooks/eox_hooks/tests/test_receivers.py", line 481, in test_receiver_called_after_cert_change
    COHORT_MEMBERSHIP_CHANGED.send_event(
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/tooling.py", line 170, in send_event
    format_responses(responses, depth=2),
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/utils.py", line 90, in format_responses
    ).pformat(obj)
  File "/usr/lib/python3.10/pprint.py", line 157, in pformat
    self._format(object, sio, 0, 0, {}, 0)
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/utils.py", line 31, in _format
    if isinstance(obj, collections.Callable):
AttributeError: module 'collections' has no attribute 'Callable'

======================================================================
ERROR: test_receiver_called_after_enroll_change (eox_hooks.tests.test_receivers.TestEnrollmentEventsHandler)
Test that hooks_handler is called the correct information after sending
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/django/test/utils.py", line 387, in inner
    return func(*args, **kwargs)
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/mock/mock.py", line 1346, in patched
    return func(*newargs, **newkeywargs)
  File "/home/edunextdev/Documents/edunext/eox-hooks/eox_hooks/tests/test_receivers.py", line 263, in test_receiver_called_after_enroll_change
    COURSE_ENROLLMENT_CHANGED.send_event(
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/tooling.py", line 170, in send_event
    format_responses(responses, depth=2),
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/utils.py", line 90, in format_responses
    ).pformat(obj)
  File "/usr/lib/python3.10/pprint.py", line 157, in pformat
    self._format(object, sio, 0, 0, {}, 0)
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/utils.py", line 31, in _format
    if isinstance(obj, collections.Callable):
AttributeError: module 'collections' has no attribute 'Callable'

======================================================================
ERROR: test_receiver_called_after_enrollment (eox_hooks.tests.test_receivers.TestEnrollmentEventsHandler)
Test that hooks_handler is called the correct information after sending
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/django/test/utils.py", line 387, in inner
    return func(*args, **kwargs)
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/mock/mock.py", line 1346, in patched
    return func(*newargs, **newkeywargs)
  File "/home/edunextdev/Documents/edunext/eox-hooks/eox_hooks/tests/test_receivers.py", line 231, in test_receiver_called_after_enrollment
    COURSE_ENROLLMENT_CREATED.send_event(
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/tooling.py", line 170, in send_event
    format_responses(responses, depth=2),
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/utils.py", line 90, in format_responses
    ).pformat(obj)
  File "/usr/lib/python3.10/pprint.py", line 157, in pformat
    self._format(object, sio, 0, 0, {}, 0)
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/utils.py", line 31, in _format
    if isinstance(obj, collections.Callable):
AttributeError: module 'collections' has no attribute 'Callable'

======================================================================
ERROR: test_receiver_called_after_unenrollment (eox_hooks.tests.test_receivers.TestEnrollmentEventsHandler)
Test that hooks_handler is called the correct information after sending
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/django/test/utils.py", line 387, in inner
    return func(*args, **kwargs)
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/mock/mock.py", line 1346, in patched
    return func(*newargs, **newkeywargs)
  File "/home/edunextdev/Documents/edunext/eox-hooks/eox_hooks/tests/test_receivers.py", line 295, in test_receiver_called_after_unenrollment
    COURSE_UNENROLLMENT_COMPLETED.send_event(
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/tooling.py", line 170, in send_event
    format_responses(responses, depth=2),
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/utils.py", line 90, in format_responses
    ).pformat(obj)
  File "/usr/lib/python3.10/pprint.py", line 157, in pformat
    self._format(object, sio, 0, 0, {}, 0)
  File "/home/edunextdev/Documents/edunext/eox-hooks/env/lib/python3.10/site-packages/openedx_events/utils.py", line 31, in _format
    if isinstance(obj, collections.Callable):
AttributeError: module 'collections' has no attribute 'Callable'

I'll be looking into this issue this week, thanks for the report!

Hello again. I've been thinking about this, and I'm not convinced we're ready for it, at least for now.
Can you specify why we should add support for python 3.10?
Our latest release should work for the master branch of the services that use openedx-events, which are currently using python 3.8 still. We should focus on maintaining the library without breaking changes since the olive cut is getting closer, and I'm not sure when the community is migrating to Python 3.10., And our primary goal should be not to introduce any bugs when it's not necessary. At least, that's what I think.
@felipemontoya, what are your thoughts on this?
I'll be happy to add it to the CI, the issue that makes me not wanna do it is that we would need to do some refactoring for it to work.

I think that adding support for 3.10 is a nice to have. I also would not risk introducing any bugs before the cut is made. But right
after we are game.
It also looks safe to fix since it appears that the only issue is with the line "if isinstance(obj, collections.Callable)". Which lives in the esponsePrettyPrinter(PrettyPrinter). This import call has been deprecated since python 3.3.
~$ python3
Python 3.8.10 (default, Jun 22 2022, 20:18:18)
[GCC 9.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import collections
>>> collections.Callable
<stdin>:1: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3, and in 3.10 it will stop working


Is there a way to optimize model parameters?

For instance in OCSVM there is this gamma parameter, is there an option to give it a range and test all combinations at once in order to find the best parameters?

What about using Gridsearch in sklearn? http://scikit-learn.org/stable/modules/generated/sklearn.model_selection.GridSearchCV.html


[Request] Add a --silent or similar switch to CLI commands

When running CLI commands like project:deploy, I would like to have a --silent or similar flag which would reduce the command's log output such that only errors are displayed.
This increased control over log output content would be immensely useful in reducing the size and clutter of CLI command output, particularly when executing from a continuous integration environment.

This is a good addition Eric,  thanks for the suggestion 👍

As a workaround in the meantime, we pipe the stdout of our project:deploy commands to null:
suitecloud project:deploy | 1>/dev/null

As a workaround in the meantime, we pipe the stdout of our project:deploy commands to null:
suitecloud project:deploy | 1>/dev/null

We've found this output redirection actually does not work because any error messages printed by the suitecloud command are printed to STDOUT instead of STDERR, so they get muted by the redirection as well.


DRM-GBM backend binaries load Wayland libraries?

The DRM-GBM client works on RPI4, but it will get a segmentation fault on i.mx8m devices.
# ldd ./flutter-drm-gbm-backend
        linux-vdso.so.1 (0x0000ffffb9c4e000)
        libxkbcommon.so.0 => /usr/lib/libxkbcommon.so.0 (0x0000ffffb9bd1000)
        libEGL.so.1 => /usr/lib/libEGL.so.1 (0x0000ffffb9b6c000)
        libdrm.so.2 => /usr/lib/libdrm.so.2 (0x0000ffffb9b49000)
        libgbm.so.1 => /usr/lib/libgbm.so.1 (0x0000ffffb9b35000)
        libinput.so.10 => /usr/lib/libinput.so.10 (0x0000ffffb9ae2000)
        libudev.so.1 => /lib/libudev.so.1 (0x0000ffffb9aa2000)
        libsystemd.so.0 => /lib/libsystemd.so.0 (0x0000ffffb99db000)
        libflutter_engine.so => /usr/lib/libflutter_engine.so (0x0000ffffb8f09000)
        libstdc++.so.6 => /usr/lib/libstdc++.so.6 (0x0000ffffb8d2a000)
        libm.so.6 => /lib/libm.so.6 (0x0000ffffb8c80000)
        libgcc_s.so.1 => /lib/libgcc_s.so.1 (0x0000ffffb8c5b000)
        libpthread.so.0 => /lib/libpthread.so.0 (0x0000ffffb8c2b000)
        libc.so.6 => /lib/libc.so.6 (0x0000ffffb8abe000)
        /lib/ld-linux-aarch64.so.1 (0x0000ffffb9c20000)
        libGAL.so => /usr/lib/libGAL.so (0x0000ffffb88fb000)
        libwayland-server.so.0 => /usr/lib/libwayland-server.so.0 (0x0000ffffb88d6000)
        libwayland-client.so.0 => /usr/lib/libwayland-client.so.0 (0x0000ffffb88b7000)
        libgbm_viv.so => /usr/lib/libgbm_viv.so (0x0000ffffb88a2000)
        libmtdev.so.1 => /usr/lib/libmtdev.so.1 (0x0000ffffb888c000)
        libevdev.so.2 => /usr/lib/libevdev.so.2 (0x0000ffffb885e000)
        librt.so.1 => /lib/librt.so.1 (0x0000ffffb8846000)
        liblzma.so.5 => /usr/lib/liblzma.so.5 (0x0000ffffb8811000)
        libcap.so.2 => /lib/libcap.so.2 (0x0000ffffb87fb000)
        libdl.so.2 => /lib/libdl.so.2 (0x0000ffffb87e7000)
        libffi.so.7 => /usr/lib/libffi.so.7 (0x0000ffffb87cc000)

Thread 1 "flutter-drm-gbm" received signal SIGSEGV, Segmentation fault.
0x0000fffff6fe3eb4 in __GI___pthread_mutex_lock (mutex=0xe9) at pthread_mutex_lock.c:67
67      pthread_mutex_lock.c: No such file or directory.
(gdb) backtrace
#0  0x0000fffff6fe3eb4 in __GI___pthread_mutex_lock (mutex=0xe9) at pthread_mutex_lock.c:67
#1  0x0000fffff6c6c340 in wl_proxy_create_wrapper () from /usr/lib/libwayland-client.so.0
#2  0x0000fffff7f448e0 in ?? () from /usr/lib/libEGL.so.1
#3  0x0000fffff7f3721c in eglInitialize () from /usr/lib/libEGL.so.1
#4  0x0000000000484498 in flutter::EnvironmentEgl::InitializeEgl() const ()
#5  0x00000000004842c0 in flutter::EnvironmentEgl::EnvironmentEgl(gbm_device*) ()
#6  0x00000000004830a4 in std::_MakeUniq<flutter::EnvironmentEgl>::__single_object std::make_unique<flutter::EnvironmentEgl, gbm_device*&>(gbm_device*&) ()
#7  0x00000000004829cc in flutter::NativeWindowDrmGbm::CreateRenderSurface() ()
#8  0x00000000004157cc in flutter::LinuxesWindowDrm<flutter::NativeWindowDrmGbm, flutter::SurfaceGlDrm<flutter::ContextEgl> >::CreateRenderSurface(int, int) ()
#9  0x0000000000427b48 in flutter::FlutterLinuxesView::CreateRenderSurface() ()
#10 0x0000000000410fa4 in FlutterDesktopViewControllerCreate ()
#11 0x000000000040ff50 in flutter::FlutterViewController::FlutterViewController(flutter::FlutterViewController::ViewMode, int, int, bool, flutter::DartProject const&) ()
#12 0x000000000040bd34 in std::_MakeUniq<flutter::FlutterViewController>::__single_object std::make_unique<flutter::FlutterViewController, flutter::FlutterViewController::ViewMode, int&, int&, bool&, flutter::DartProject&>(flutter::FlutterViewController::ViewMode&&, int&, int&, bool&, flutter::DartProject&) ()
#13 0x000000000040b55c in main ()

cc: @Mai-Matsuura
Related issues

https://github.com/sony/flutter-embedded-linux/issues/4

I'm using the BSP built for Wayland, that may be the cause.

gbm.h needs to be included before context_egl.h is first included, but I feel like not following that. We also need to refactor code files.

@HidenoriMatsubayashi
Not the following statement from i.MX Yocto Project User's Guide regarding available distros:

• fsl-imx-wayland - Wayland weston graphics.
• fsl-imx-xwayland - Wayland graphics and X11. X11 applications using EGL are not supported.
• fsl-imx-fb - Frame Buffer graphics - no X11 or Wayland. Frame Buffer is not supported on i.MX 8.

I would understand if then only fsl-imx-wayland supports EGL or maybe none of them?!
Maybe it helps.

Thank you for your comment. Yes, I knew that. I'm using fsl-imx-wayland now. However, I haven't known the cause of the following error yet :cry: glmark2-es2-drm works on i.MX8M mini.
Thread 1 "flutter-drm-gbm" received signal SIGSEGV, Segmentation fault.
0x0000fffff6fe3eb4 in __GI___pthread_mutex_lock (mutex=0xe9) at pthread_mutex_lock.c:67
67      pthread_mutex_lock.c: No such file or directory.
(gdb) backtrace
#0  0x0000fffff6fe3eb4 in __GI___pthread_mutex_lock (mutex=0xe9) at pthread_mutex_lock.c:67
#1  0x0000fffff6c6c340 in wl_proxy_create_wrapper () from /usr/lib/libwayland-client.so.0
#2  0x0000fffff7f448e0 in ?? () from /usr/lib/libEGL.so.1
#3  0x0000fffff7f3721c in eglInitialize () from /usr/lib/libEGL.so.1

Do you build the drm-gbm-backend on the target or cross via yocto?
I just wonder how you made it built through without errors ( note my comment in #4 )
Maybe it is still something wrong on my side.

In conclusion, I'm afraid we can't run Flutter with drm-gbm backend on i.mx8m platforms. We need to use Wayland backend on i.mx8m platforms. See: https://github.com/sony/flutter-embedded-linux/issues/4#issuecomment-824419854


Some fonts may not be processed by convert_ttf_to_sfd_mp.py

Hi,  I'm encountering the problem in the title.
For example, if mp.cpu_count()=8 and you have 8 fonts under font_ttfs/test, one font will not be processed.
In this case, process_nums=6 and font_num_per_process=1.
Processes = [mp.Process(target=process, args=(pid, font_num_per_process)) for pid in range(process_nums + 1)].

will launch 7 processes, but they will each process one font, so only 7 fonts will be processed in total.
It's not a pretty solution, but it seems to work by having the process with the largest pid handle all the remaining fonts, as shown below:
--- a/data_utils/convert_ttf_to_sfd_mp.py
+++ b/data_utils/convert_ttf_to_sfd_mp.py
@@ -26,8 +26,8 @@ def convert_mp(opts):
     else:
         font_num_per_process = font_num // process_nums

-    def process(process_id, font_num_p_process):
-        for i in range(process_id * font_num_p_process, (process_id + 1) * font_num_p_process):
+    def process(process_id, font_num_p_process, last=False):
+        for i in range(process_id * font_num_p_process, (process_id + 1) * font_num_p_process if not last else font_num):
             if i >= font_num:
                 break

@@ -72,7 +72,7 @@ def convert_mp(opts):

             cur_font.close()

-    processes = [mp.Process(target=process, args=(pid, font_num_per_process)) for pid in range(process_nums + 1)]
+    processes = [mp.Process(target=process, args=(pid, font_num_per_process, pid==process_nums)) for pid in range(process_nums + 1)]

     for p in processes:
         p.start()

Hi, thank you for your feedback!
Yes, the partition only works when font_num_per_process > process_nums,  i.e., the number of fonts is very big.
It seems to be correct to use the following code:
Processes = [mp.Process(target=process, args=(pid, font_num_per_process + 1)) for pid in range(process_nums)].

I will update it when I have tested it.


[$500] Split bill - When delete Split Bill report from attendee DM & go back - Error page showing

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.3.83-5
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Open New Expensify app
Navigate to 1:1 DM with any of the Attendees you split bill with
Delete the IOU report
Tap Back button

Expected Result:
User should be landed on LHN or 1:1 DM
Actual Result:
"Hmm... It's not here" error page appears
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/78819774/453fd44b-1d2d-47f4-94f8-ad8521716797

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ac48abac656adba3
Upwork Job ID:<PHONE_NUMBER>494998528
Last Price Increase: 2023-10-16

Just a note @lanitochka17 that if it's reproducable already, feel free to add External when you create it as well.

Proposal
Please re-state the problem that we are trying to solve in this issue.
404like page loads after deleting the money request
What changes do you think we should make in order to solve the problem?
This, but the if should not be made on the chat report ID which may still exist, but on the whole chatReport Object, like in the code below this one:
if (shouldDeleteIOUReport) {
    /*in this if im checking the reportID, which may not be null, so it should check the whole chatReport Object
to see that if it has been deleted and if it is now null/undefined it should  go back to home screen / chatlist */
    if (iouReport.chatReportID == null || iouReport.chatReportID == undefined) {
        Navigation.goBack(ROUTES.HOME);
        Navigation.goBack(ROUTES.HOME);
        //it should go back to the home screen / ChatList screen without loading the deleted report
    }
    else{
        Navigation.goBack(ROUTES.HOME);
        Navigation.navigate(ROUTES.REPORT_WITH_ID.getRoute(iouReport.chatReportID));
    }
}

So the 2nd if should be something like:
    if(!chatReportExists(iouReport.chatReportID)){
          Navigation.goBack(ROUTES.HOME);
          Navigation.goBack(ROUTES.HOME);
          //it should go back to the home screen / ChatList screen without loading the deleted report
    } ...

function chatReportExists( chatReportID ){ ... }

@parasharrajat intrigued for your take here please.

Bump here @parasharrajat

Still waiting on proposals then I suppose.

Bump here @parasharrajat.

Can we assign @tienifr?

hmmmmm this seems like a duplicate of https://github.com/Expensify/App/issues/26569 which will be fixed by https://github.com/Expensify/App/pull/26498
@sobitneupane @adamgrzybowski @JmillsExpensify @mountiny can you confirm that indeed this is the same issue (looks like it to me) and that the PR will fix this?

Seems like a dupe

@s77rt has taken over as the C+ of the issue. He has more information about the issue.

@parasharrajat This is a dupe and https://github.com/Expensify/App/pull/26498 will fix it


CIColorExtension.swift 'characters' is deprecated: Please use String or Substring directly

Dear Alexander, how are you?
On swift 4 build IOS 11 I'm facing a warning that is relatively easy to solve:
Class: CIColorExtension.swift
Line 32:
if scanner.scanHexInt64(&hexValue) {
let length = rgba.characters.count
Warning Message: 'characters' is deprecated: Please use String or Substring directly
Thanks

use
let length = rgba.count


Context Instances

Refactor the scope into a class for the client and one for the server exposing the instance key to the context.
The application should be able to call a component by key and access its public methods like:
prepare({instances})  {
   instances.counter.count++;
}

This should add a new layer of freedom for plugin makers

Hey, for formalities, you can assign me this one too :grin:

This will be merged soon, Thanks 💖


specific output file name include special character need handle

like #28  ,  this case is for option -o
go run main.go -o "*c/!:\test1".mov https://www.youtube.com/watch?v=54e6lBE3BoQ

fixed by #43
Thanks to @ilmanzo


Have ordinal return 0th instead of 0

Both dictionary.com and wikipedia recognize 0th or zeroth, which makes this special case a bit weird to me

Thank you for the careful thought and consideration as well as your time and effort into this pull request. Just because a word is defined does not make it colloquial. As a native speaker I cannot think of a case where zeroth makes sense in conversation.
Anything in the place of zeroth, to my mind, spoken still seems like it should be "first" in terms of speech. Which we cannot garentee is correct. I'll continue looking for an example where this is correct until tomorrow. Unless we think of an example of a reasonable usecase I'm closing this. Thanks again for your thought and time.

No worries, I was using this to ordinalize percentiles from postgres' NTILE method, and after actually reading the documentation, NTILE returns an "integer ranging from 1 to the argument value", so the 0 case will never get hit.
0th or zeroth are mostly in use in CompSci and Mathematics, as far as I can tell, so I might be suffering from a 'slight' bias here ;). Some examples from those fields:

Zeroth-order logic is first-order logic without variables or quantifiers src
Under zero-based numbering, the initial element is sometimes termed the zeroth element, rather than the first element src
the zeroth power of a number src

The last one of those even says as much:

the word, which was coined by physicists over a hundred years ago, does often show up in scientific contexts

Anyhoo, I completely defer to you here, since I no longer have a horse in the race here.

Thanks! dI'l be happy to revisit and have a conversation it it covers up again.

I think this use case makes sense in those context. My suggestion would be to to extend .ordinal to take a second, optional parameter that allows for 0th if it's being used for e.g. scientific humanization. For example:
ordinal(value, options = {}) {
  const { scientific = false } = options; // Maybe a better name...
  const number = parseInt(value, 10);

  if (number ===0 && !scientific) {
    return value;
  }

  ...
}

+1

I added a zeroth option, not sure if that's better or worse than scientific though ;)


Switch from gitbook to vitepress

Todo:

[x] Setup VitePress
[x] Write new content
[x] Update readme's about new domain
[x] Add note on gitbook about new link
[x] Update templates to have new comments
[x] Get Search Working Apply for algolia docsearch

reminder: get search working

Closed by #133


Email notifier interval settings is not honored

Hi,
I can't get the interval working for email notifiers. I've set the interval to 300 seconds but it still sends out an email every 10 seconds. Could something else affect that setting? I've tried playing around with the broker-offsets but that didn't affect it. These are the settings I've been using minus the identifying information.
Thank you so much for any help I could get!
[general]
logconfig=./logging.cfg

[zookeeper]
hostname=some host
port=2181
timeout=6
lock-path=/burrow/notifier

[kafka "some cluster"]
broker=some broker
broker-port=9092
zookeeper=some zookeeper
zookeeper-port=2181
zookeeper-path=/some path
zookeeper-offsets=true
offsets-topic=__consumer_offsets

[tickers]
broker-offsets=300

[lagcheck]
intervals=10
expire-group=604800
min-distance=1
zookeeper-interval=60
zk-group-refresh=300

[httpserver]
server=on
port=8990

[smtp]
server=some server
port=587
auth-type=plain
username=some user
password="some password"
<EMAIL_ADDRESS>template=./email.tmpl

[emailnotifier "some email"]
group=clusterFoo,consumerBar
interval=300
enable=true

Found it.
Need to add a section like this that controls notification interval on a global level I would assume:
[notify]
interval=300

Removing the interval from the emailnotifier causes burrow to not start so not sure if you want to close the issue or not.

@joellundell do you remember what the interval is used for in the emailnotifier stanza by chance?

@daodennis it's been a while since I touched this and I've changed company since but if memory serves me right it's ignored. It only looks at the interval under the notify section per my comment above.


Made ui changes in the game section

added a three.js component in this section to make the UI good

Hi @vaibhav-gosavi , The game section will inclue the actual 2d side scrolling game. I am currently building the game engine. We can work on this part together if you want. I cannot approve this part because the three JS wireframe is not appropriate for game/page section.
We can discuss the game engine build as a different topic.

ohh ok understandable Thank you helping in this matter


I deployed four keys but no mock data shown in grafana dashboard

I deployed the four keys systems, using the documentation ( using terraform with modified example/ variables ) and successfully pushed github mocked data. My big queries tables  contain some rows (changes, deployments, etc ) however when I go to graphana dashboard I don't see any data/reports/metrics

Ok, I found DORA metrics dashboard, using search dashboard mechanism, why not put this on the main/default  dashboard page?

Try accessing '/dashboards', there will be a Four Keys dashboard in the General folder, or search for For Keys via Grafana's searc.

how can I add authentication to the dashboard, did not find anywhere in the docs ...

To enable authentication you can look here. About making it clearer how to access it, I think it's valid, but I think it's better to wait for a response from one of the most experienced members of the project.


Overlapping of creatures

Try to fix overlapping on feeding/reproduction paths on pipes.

An easy way of doing this would be to make sure that we don't assign the "proxy" creatures to the same waypoint at the same time


Support command "TTL"

Returns the remaining time to live of a key that has a timeout. This introspection capability allows a Redis client to check how many seconds a given key will continue to be part of the dataset.
In Redis 2.6 or older the command returns -1 if the key does not exist or if the key exist but has no associated expire.
Starting with Redis 2.8 the return value in case of error changed:
The command returns -2 if the key does not exist.
The command returns -1 if the key exists but has no associated expire.

Hey, wanted to help here. we the first step, I am thinking about adding storage::Storage trait a method called meta:
fn meta(&self, key: &[u8]) -> Option<meta::RedisMeta>

it should return the TTL and the type, and serve both the TYPE and the TTL commands. possibly, other internal implementations that need either expiry or type.
WDYT?

Can you propose a full implementation of meta?

there is already a partial implementation in storage/modes/meta, since both data_type and expiry are public, I think it should be enough for now. It may grow in the future.

If you are gonna do ttl, I think itd be better to change Expiry from how it's currrently implemented with an Instant to use timestamps. TTL is doable with Instant but other commands like expireat and pexpireat take unix timestamps. And it's not doable with Instant afaik

But afaik, there is no way to go from an Instant to a unix timestamp. I think it would be better to use chrono for this, in particular with this method timestamp_millis, since the redis specification wants expires to be precise to at least 1ms.

Expires and persistence
Keys expiring information is stored as absolute Unix timestamps (in milliseconds in case of Redis version 2.6 or greater).This means that the time is flowing even when the Redis instance is not active.

@clarity0 see above PR to set prepare for solving this issue.

going forward I am thinking about a proper way to extend the Storage trait.
first, add:
    fn meta(&self, key: &[u8])-> Option<&RedisMeta>; //can be also &mut self if we want to do GC same as done during read

this can support ttl, pttl, type...
another option is to add:
    fn expiry(&self, key: &[u8])-> Option<&Expiry>; //same comment about mutbility as above
    fn type(&self, key: &[u8])-> Option<RedisType>;

adding meta is currently more useful but only because of the way metadata bookkeeping is implemented in the InMemoryStorage. Other types of storage might split expiry from type as an example. Adding separate expiry and type methods is narrower, hence a bit more resilient for future implementations.

@clarity0 pushed a PR that goes with the first option. tell me what you think.


[Misc] add the "download-dir" option to the latency/throughput benchmarks

Previously, the latency and throughput benchmarks do not support the --download-dir option. This PR adds the option to the benchmarks scripts so that users can specify the model download directory.

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

Hi, I have fixed the format issue by pulling the latest commits from the main branch. The format issue isn't related to the files I modified during this PR and it has been fixed on the main branch.


Android Java: 2.1 Use of Material Designs in UI

In this, I will be describing on how we can use material designs to enhance our App UI and make it user-friendly.
The various material designs include:

FloatingActionButton
SnackBar
Bottom Sheets
Navigation Drawer
Bottom Navigation

We will also be describing on how to make simple XML Views like Edit Text and Button more user professional to view with the help of material designs.

@VikasPandey121 can I work on this issue?

@VikasPandey121 can I work on this issue?

Go ahead !

Please keep these steps in mind while creating a pull request:

use git fetch and git pull commands before pushing commit it to the remote repository.
Your Contributing file should be in this directory Android_Development/Java/ with the file name starting from 2.2 for instance 2.2 Android UI Widgets
Note:- If you have multiple files to commit then add a tag at the beginning of your file name like 2.2.1, 2.2.2 etc
Create a new branch and push your PR on Android_Development_With_Java branch.
Pull request should contain only one commit
Note: If you have multiple commits then squash your commit.

Please keep these steps in mind while creating a pull request:

use git fetch and git pull commands before pushing commit it to the remote repository.
Your Contributing file should be in this directory Android_Development/Java/ with the file name starting from 2.2 for instance 2.2 Android UI Widgets
Note:- If you have multiple files to commit then add a tag at the beginning of your file name like 2.2.1, 2.2.2 etc
Create a new branch and push your PR on Android_Development_With_Java branch.
Pull request should contain only one commit
Note: If you have multiple commits then squash your commit.

ok Sir

good job vikas

I am from Android Development with Java Batch-6
I want to contribute/work in this issue via documentation

@anisha-01-phoenix  can you please give a progress update about the issue and if you are facing any problem feel free to ask.

@anisha-01-phoenix  can you please give a progress update about the issue and if you are facing any problem feel free to ask.

Yes Sir, I have completed the documentation on Floating Action Bar and Snack Bar along with an introduction on Material Designs and how they are better than normal XML views such as edit text or button. I have written about the usage of floating action button and Snack bar along with the codes on how to implement them.  I was thinking of making a simple project based on the usage of all the material designs, and will include that code in my documentation. Presently I am working on Bottom Sheets.


feat: extend husky checks

I added some more checks to huskys config to because not addressing them leads to a bunch of unnecessary cycles for reviews and CI and the overhead for the developer seems rather small. Let me know what you think!

Let's give this a try! (We can use git commit --no-verify git push --no-verify to skip the hooks, which I've never had to do before, but might start doing occasionally with this change.)


Using complicated hotkeys (ctrl+shift+e+n+g) doesnt work properly

React-hotkeys version: 0.10.0
React version: 15.4.2
We're also using electron v1.7.9.
Using hotkeys such I mentioned (ctrl+shift+e+n+g) provides unexpected(imo) behavior. With working ctrl+shift+e+n+g combination there are also a hotkey that triggers via ctrl + shift + g, after some tries I finally figured out that in combinations like mine with characters as members of a hotkey using special keys + last character key also triggers that hotkey.
Use this hotkey to reproduce: "ctrl+shift+e+n+g" and then press ctrl + shift + g

Hello, @reetou. Let me see if I understand you correctly:

You have a hotkey action attached to the key combination ctrl+shift+e+n+g (ctrl, then shift, then 'e', then 'n', then 'g')
You also have a hotkey action attached to the key combination ctrl+shift+g (ctrl, then shift, then 'g')
When you press the key sequence to call handlers attached to the first action (ctrl+shift+e+n+g), the handlers for the second action are also called (ctrl+shift+g).

If that's correct, can you tell me:

Which handler gets called first?
What arrangement and nesting of HotKeys components are you using? (which HotKeys defines each of the actions and which one defines each of the handlers?)

Closing this for now as it doesn't seem to be moving and I am still unclear about the specific test case to reproduce and resolve this issue. Please comment if you are interested in continuing this and I'll try and help out as best I can.


use main v3 tag of the json-yaml-validate action

Testing out the latest release of this action
Update: It looks good, this can be merged ✅

.deploy


mitochondrial transciptome

For
https://www.pombase.org/reference/PMID:29954949
transcriptome data
SRA accession: SRP159233

I manually annotated these, so we probably don't really need a  trancsrip[tome data track?
I have a curation tracker ticket for this but I'm still waiting for confirmation for some of the boundaries from Ying.
https://github.com/pombase/curation/issues/2588


Add a guide for "how to work with meta"

Hi, i tried to update post meta, when i created posts, but nothing happened
It's possible with node-wpapi?
					wp.song().create({ 					    title: '1', 					    content: 'Your post content', 					    status: 'publish', 					    meta: {key: 'youtube', value: '12'}, 					}).then(function( response ) { 					    console.log( response.id ); 					})

On the WordPress side of things you'll need to set things up so that you can read and post meta. See http://v2.wp-api.org/extending/modifying/#read-and-write-a-post-meta-field-in-post-responses

A better option these days is to use first-party meta support, introduced in 4.6, detailed here in this API status update post and further explained in the README from the pull request for meta handling

@kadamwhite is there some change I have a meta field upvote for my custom post and I have registered using register_meta() , as suggested and when I use a get request , I am getting the meta values like..
"meta": {
"upvote": "4",
"downvote": "0"
}.
When I try to update it works via postman if I pass the body as
{"meta":
{
"upvote": "4"
}
}
But the node-wpapi does not take this, if I pass it as
{
meta:{
key:'upvote'
value: '4'
}}
It does not update the values.

Merging #286 into this issue; documenting how to work with custom meta is currently our top-priority docs issue

When you create/update post, you pass meta object, for example:
wpapi.posts().create( {
	title: 'Your Title',
	content: ' Your content',
	status: 'publish',
	meta: {
		custom_meta_key: 'something'
	}
} ).then( function( response ) {
	// whatever
} );

On the server side, you must use register_meta(). If you are using custom post type, the post type must have custom-fields support (see documentation for $args['supports'] of register_post_type(). That is documented for REST API.
But if you are using meta with custom role, your role should also have edit_post_meta (and delete_post_meta if you are deleting) capability. See code for WP_REST_Meta_Fields::update_meta_value() and WP_REST_Meta_Fields::delete_meta_value(). I haven't seen that this is documented anywhere so I figured it out during debugging.

@dimadin Sorry, but could You be more specific, where exactly on server side I should change something to get meta work?

Messed with the same problem, being total noob in wordpress infrastructure. Solved by putting
register_meta('post', 'duration', [
  'type' => 'integer',
  'description' => 'duration',
  'single' => true,
  'show_in_rest' => true,
]);

into functions.php of theme. This unlocked me:
meta: { duration: 123 }

@Calemb @dmdb In WordPress 5.0 note that this will change to register_post_meta( 'post', ... -- the snippet you have here will register "duration" for all custom post types. (You'd change 'post' to 'my_cpt' if you want duration to apply to a custom post type). We will be releasing a comprehensive update to our meta handling documentation on the REST API handbook in conjunction with 5.0.

Responding to some prior questions in this thread, @chattes you should be using the same structure in Postman as in this library, so I'm unclear on why you believed you needed to separate the key and value. If you have registered meta, you'd update it as,
.update( {
    meta: {
        upvote: 5,
    }
} )

@b02902032 What you have there looks like it works (and I assure you, even seasoned contributors get confused by the inconsistent property naming!) For a post, page, or other custom post type resource though I'd stick with register_post_meta over register_rest_field unless you need to derive the data from something other than post meta.

Here's how to add a boolean meta field to tags and posts.
function prefix_register_meta_fields()
{
    $term_types = ['post_tag'];
    $post_types = ['post'];

    $key = 'the_example_key';
    $args = [
        'type' => 'boolean',
        'description' =>
            'An the_example_key boolean value.',
        'single' => true,
        'show_in_rest' => true
    ];

    foreach ($post_types as $type) {
        register_post_meta($type, $key, $args);
    }
    foreach ($term_types as $type) {
        register_term_meta($type, $key, $args);
    }
}

add_action('rest_api_init', 'prefix_register_meta_fields');

And then update it on some post of your choosing.

wp.tags()
        .id(TAG_ID)
        .update({
            meta: {
               the_example_key : true
            }
        })

wp.posts()
        .id(POST_ID)
        .update({
            meta: {
               the_example_key : true
            }
        })

documented

How do you add multiple meta values with the same key? Using a single object only allows attaching of 1 meta key.

I did not find an example of what to do if you need to add several values for one key.
I try in wp
register_meta( 'post', 'reader', array(     'show_in_rest' => true,     'single'       => false,     'type'         => 'string', ));
i add meta in wp-api
wp.posts().id( 32477 ).update({ 	title: 'A Better Title', 	slug:"kukusik-100", 	status: 'draft', 	meta:{ 		reader:'Anton Petrov,Oleg Arno', 	} }).then(function( response ) { 	console.log( response.id ); })
And I get in the post meta fields the following
reader:Anton  reader:Petrov  reader:Oleg  reader:Arno
I would like to see there
reader:Anton Petrov  reader:Oleg Arno


⚠ Turnos 24x7 has degraded performance

In e0cf901, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 6755 ms

Resolved: Turnos 24x7 performance has improved in 00db3f9.


Update border size for Checkbox from 1 to 2px

Background
When Checkbox was completed the border for the checkbox had to be changed from 2px to 1px because triggering the error state was causing the checkbox group to "jog"
As a: Design system team
I want: The styling for the checkbox and radio controls to be the "same"
so that: So the Design system (esp the radio and checkbox controls) are visually consistent
Acceptance criteria

[x] Checkbox border in error state is updated from 1px to 2px (use variables if appropriate)
[x] Checkbox group will not "jog" when error state is triggered
[x] Passing unit tests
[x] Linting
[x] update storybook
[x] Push to Alpha
[x] Manual design review

Presentable outcomes

[x] I can show the ui-component in storybook and provide the link to service teams so that they can view the component for themselves

Sprint Ready Checklist

Acceptance criteria defined
team has defined steps to satisfy acceptance criteria
Acceptance criteria is verifiable / testable
External / 3rd Party dependencies identified

Related issues:
GitHub issue: Orignial Checkbox story #278
Resources:
Mockups: Checkbox design
Testing URL: [If applicable a URL to the testing branch]
Design review:

:tada: This issue has been resolved in version 3.4.0-alpha.57 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.0.0-alpha.51 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.0.0-alpha.6 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added a Resnet example to quick_start demo

From #299. Cleaned up my git tree and doing a new PR.
Added a residual LSTM example from the GNMT paper.
@pengli09 @qingqing01 @reyoung @lcy-seso Thanks for the help on this PR!

Coverage increased (+0.03%) to 62.945% when pulling cc5adfb8e5c3e4be35229e4c83512e26f4f9cb75 on alvations:develop into 805856aaf155b7a0a794cd380fd2bb68575b370f on baidu:develop.


short delay for MCMC 24 million

Just woke up and am getting errors with the VPN required to connect to my desktop: "AnyConnect was not able to establish a connection to the specified secure gateway. Please try connecting again.".
Luckily with light traffic at this hour, the office is only 10 minutes away so I'm going to hunt down the MCMC run in person and should be able to upload shortly.

OK, just uploaded long run as mcmc_24mil to 2018.40.30_fecundity_matrix2. I also renamed Aaron's shorter chain to mcmc_12mil.
So I think we all need to

Rename mcmc on our computers to mcmc_12mil

Download mcmc_24mil and rename to mcmc

Re-run build(T)

[ ] Ian

[ ] Aaron

[ ] Chris

[ ] Andy

Thanks Ian, don't forget to source all.r also: source("all.r");build(T)

Document built successfully, with all tables and figures for the 24 million run.

And delete 12-mil figs and tables (or just delete knitr-cache).

Ian - do you need to check the box? Close this?


Unideal error message when not having user_allow_other

When having not the option user_allow_other set in /etc/fuse.conf the following message appears:
fusermount: option allow_other only allowed if 'user_allow_other' is set in /etc/fuse.conf
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 0, kind: Other, message: "Success" }', examples/null.rs:11:68

Please note the code and message which is not ideal.
Is there something we could do to get a better error message?

The easy fix is to check for that flag in your application before mounting the filesystem. For example, like this: https://github.com/cberner/fuser/blob/e30eb1d06612f78cacc6207c25d61845562bfd09/examples/simple.rs#L1938-L1946

I get the same error when I pass fuser::MountOption::AutoUnmount. Why is that? something libfuse-specific?


mi_heap_visit_blocks callback is missing explicit calling convention

The callback mi_block_visit_fun is missing a calling convention specification (e.g. __cdecl or the like), which can cause linkage problems when mixing a mimalloc library compiled with one ABI but linked into a target that uses a different default ABI (e.g. using the /Gv switch that can be common in performance-oriented codebases).
Its standard practice in Windows code to always tag a calling convention on callback function declarations to avoid this issue. Windows headers use the CALLBACK macro for this purpose though specifying a new macro for mimalloc's purpose would suffice too so the code remains portable.
The idea is that this function can be declared in mimalloc.h using this specifier and then any user code that is defining a callback for mi_heap_visit_blocks would use the same specifier macro for the callback declaration, guaranteeing compatibility even if the default calling conventions differ.

Thanks for spotting this. I will add this.

Fixed in the latest dev branch


uninstall

Trying to remove this as it didn't work for me. Cannot connect to the network now. Following the instructions, it says to remove gg-struggle from certmgr.msc but I don't have any there, I have 3 entries of gst-game.guiltygear.com now though since I tried installing it 3 times. How do I remove this and go back to being able to play the game?

Try to use an IP from https://dns.google.com/query?name=ggst-game.guiltygear.com
If you ping while this is installed a ping will likely return <IP_ADDRESS>

You have 3 entries of gst-game.guiltygear.com in your host file ?

no the entries are in my certmgr one appeared each time I attempted an install

In hosts file, it was pointing the non "real" server entry to <IP_ADDRESS>, are you saying I should change the IP there?

In hosts file, it was pointing the non "real" server entry to <IP_ADDRESS>, are you saying I should change the IP there?

Not the dev, but i managed to fully unninstal by deleting the "<IP_ADDRESS> ggst-game.guiltygear.com" line entirely in the hosts file, and delete the three certificates "ggst-game.guiltygear.com" in your case

The image shows what's safe to delete

Running the installer and uninstaller in 1.5.0 will remove rogue host entries


Modernise integrity tests

This work has be outdate by the #478.
#478 still uses the AntlrParser for the grammar check of queries, but consumes the scenarios through the TCK API which parser parameters and result sets through the Fastparse parser.

This work has be outdate by the #478.
#478 still uses the AntlrParser for the grammar check of queries, but consumes the scenarios through the TCK API which parser parameters and result sets through the Fastparse parser.


Add a closeOnHover option.

Add a closeOnHover option. Defaults to true for backward compatibility. Can be set to false to disable the feature that closes a toast when the toast is hovered upon (technically after a mouseout event). This came about because I needed a toast with a countdown, but didn't want the hover events to clear it.
Example usage:
// create a toast that is not closed by hover event, but will close when time out finishes.
toastr.options = {
  timeOut: 5000,
  extendedTimeOut: 0,
  closeOnHover: false
}
toastr.info("Message", "Title");

I've accidentally committed to Master. I'll try again.
I'm going to Close and Comment this pull request. I'm not sure if that gets rid of it or not.


Force namespace-wide authn policy name to be "default"

This PR enforce the name for authentication policy that have no target selector rules (i.e namespace-wide policy) must be default. This is to avoid having conflicts namespace-wide policies, as well as be more consistent with mesh-wide policy name.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: diemtvu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ayj
Assign the PR to them by writing /assign @ayj in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Done

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

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
4762382e9197189e1325bf9f9f588c48c6b26a85
link
/test e2e-simple

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

/test e2e-simple


fix version tomcat:9.0.95-jre11 causes angular to break when run in chromium using taiko gauge testing framework

We recently started seeing failures in our testing framework when run our smoke testing pipelines.
The angular version being used is 17.3.1 and the error we get when testing using the latest chromuim headless browser is a failure to load the js files for Pollyfill.js and other js libraries.

This sounds like an issue with the content being hosted using tomcat and not tomcat itself

Thanks for replying. It is a very odd issue with how the container env variables were configured.


GH Actions: remove some incorrect docs

PHPUnit 10 - at this time - can run on PHP 8.1/8.2/8.3 without version restrictions related to PHP compatibility issues.

coverage: 95.833%. remained the same
when pulling 6077e94f5aae1b3a743a612979e7312e56172cf4 on feature/ghactions-workflow-docs
into 28477a46124f6310a33cd65f6789b83d219ef927 on 2.x.


upgrade to ubuntu 20.04

this percolates

Note host AMI's are still 18.04 -- that is okay - BaseDockerfile is from 20.04


Describe AIRs in terms of Table Elements

The polynomial presentation of AIR constraints is too dense. Best to explain them in words first, then formulaically in terms of table cells with variable indices, and only then in terms of polynomials.
(Originally issue number 4 in the internal issue tracker.)
(Originally opened by @aszepieniec.)
Add the following part of the AIR:

[x] DNF in words for U32 Table's constraints
[x] Polynomials for U32 Table's constraints
[ ] Polynomials for Hash Table's transition constraints

In the code we need to use:

[x] Polynomials for U32 Table's constraints
[ ] Polynomials for Hash Table's transition constraints
[ ] the transition constraints of instruction groups in the Processor Table

The following parts of the AIR are missing at this point:

[ ] DNF in words for U32 Table's constraints
[ ] Polynomials for U32 Table's constraints
[ ] Polynomials for Hash Table's transition constraints

In the code we need to use:

[ ] Polynomials for U32 Table's constraints
[ ] Polynomials for Hash Table's transition constraints
[ ] the transition constraints of instruction groups in the Processor Table

This issue is essentially tracked by #15 and #28.


RepeatMode.Off does not stop at the end of the track

setRepeatMode(RepeatMode.Off) appears to do nothing if there is another track in the queue. One would expect it to stop at the end of the track.
If this is the expected behavior, I'd like to request an additional RepeatMode option to stop the track when it reaches the end, even if there are more tracks in the queue.

Perhaps this will be useful for you: https://github.com/DoubleSymmetry/react-native-track-player/issues/869
(Spoiler: no solution, just workaround)

Thanks @AdamSheaHewett and @Ryabchikus
Are you able to investigate and find a fix or further info?
Cheers, Brad

@bradleyflood
I'm sorry but I just used the workaround and got on with my work

Thanks @Ryabchikus what version of RNTP does this affect?
@AdamSheaHewett and you too?

@bradleyflood, sorry for delay!
Now I still use 1.2.7 because of old shitty code
We have really old project and only 2 month ago we able to move from RN0.59 to latest. There are too many parts to refactor...


Player Score

Players should receive 10 points when they enter a correct number. They have 10 points remove a number, and 5 points removed if they enter an incorrect number

added


Playlist can't rename

In src/views/componetns/menus/playlist.js line 41,
missing import dialog module, but important thing is: prompt method is not supported.
So, we need to find other way to solve this.

A description with it will also be better @neighborhood999 :smile_cat:

@funilrys Yep!


Access-Control-Allow-Credentials: false isn't real

Omit it, don't render it.

Fixed in the CORS GHSA.


chore: add docs for 1.0.0-beta.69

What/Why/How?
changelog and docs for 1.0.0-beta.69
Reference
Testing
Screenshots (optional)
Check yourself

[ ] Code is linted
[ ] Tested with redoc/reference-docs/workflows
[ ] All new/updated code is covered with tests

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

@RomanHotsiy please check now


961

Check List:

[x] Tests pass (./test.sh <username> shows no differences between expected and actual outputs)
[x] All formatting changes by the build are committed
[x] Your launch script is named calculate_average_<username>.sh (make sure to match casing of your GH user name) and is executable
[x] Output matches that of calculate_average_baseline.sh
[x] For new entries, or after substantial changes: When implementing custom hash structures, please point to where you deal with hash collisions (line number)

Execution time:
Execution time of reference implementation:

Please run mvn verify and amend this PR with any formatting changes coming out of it.

Gonna close this for now as I haven't heard back and need to clean the slate a bit. Please feel free to reopen once everything looks good (formatting, tests pass, 1B row results match). Thx!


[Feature Request] Min and Max reference count in settings as a text field

Right now the entry for "Minimal required count to show counter" and "Maximum file references to show" are a small slider which makes it impossible to select values like "2", which I wanted to do for "minimal count". I'd suggest turning it into a text entry field.
Cheers and great plugin :)

I agree with you. This slider control in Obsidian isn't ideal. I will change this to a number input to make it easier.
Until I get to it, if you click once on the slider, then you can use your arrow keys to left and right to change the setting value

I wanted to submit an enhancement just like this. Maybe until you make the change put that tip about the arrow keys?