HTTPS ServiceEntry TLS random failures

Bug description
In our Isito cluster we enable, REGISTRY_ONLY for outbound traffic, as such any outbound traffic requires a ServiceEntry.
We find connections to HTTPS endpoints randomly fail to initiate and get blackholed from our pods when specifying the protocol as HTTPS in ServiceEntry, if we specify "protocol: TCP" we do not experience this issue.
Below is the ServiceEntry used
kind: ServiceEntry
apiVersion: networking.istio.io/v1alpha3
metadata:
  name: git-nonprod
  namespace: nflynn
  selfLink: >-
    /apis/networking.istio.io/v1alpha3/namespaces/nflynn/serviceentries/git-nonprod
  uid: 53ea85d5-3cfb-11ea-8723-0afaefed8f04
  resourceVersion: '23230583'
  generation: 2
  creationTimestamp: '2020-01-22T09:41:13Z'
  labels:
    app.kubernetes.io/managed-by: Helm

spec:
  hosts:
    - git.nonprod.company.plc
  addresses: ~
  ports:
    - name: https
      number: 443
      protocol: HTTPS
  location: MESH_EXTERNAL
  resolution: DNS
  endpoints: ~
  exportTo: ~
  subjectAltNames: ~

Ran inside one of the pods
bash-4.2# for i in {0..10}; do curl -vs https://git.nonprod.company.plc/  2>&1 | grep ERROR; echo $i; done
0
1
2
3
* NSS error -5938 (PR_END_OF_FILE_ERROR)
4
5
6
7
8
9
10

Trace logs on the envoy-proxy shows the follow when it fails;
2020-01-24T09:26:57.569538058Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.569][32][debug][filter] [external/envoy/source/extensions/filters/listener/original_dst/original_dst.cc:18] original_dst: New connection accepted
2020-01-24T09:26:57.569569253Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.569][32][debug][filter] [external/envoy/source/extensions/filters/listener/tls_inspector/tls_inspector.cc:72] tls inspector: new connection accepted
2020-01-24T09:26:57.569587368Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.569][32][trace][filter] [external/envoy/source/extensions/filters/listener/tls_inspector/tls_inspector.cc:165] tls inspector: recv: -1
2020-01-24T09:26:57.669721879Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.669][32][debug][conn_handler] [external/envoy/source/server/connection_handler_impl.cc:179] listener filter times out after 100 ms
2020-01-24T09:26:57.669870353Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.669][32][debug][conn_handler] [external/envoy/source/server/connection_handler_impl.cc:182] fallback to default listener filter
2020-01-24T09:26:57.670011677Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.669][32][debug][filter] [src/envoy/tcp/mixer/filter.cc:30] Called tcp filter: Filter
2020-01-24T09:26:57.670071075Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.670][32][debug][filter] [src/envoy/tcp/mixer/filter.cc:40] Called tcp filter: initializeReadFilterCallbacks
2020-01-24T09:26:57.670134343Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.670][32][debug][filter] [external/envoy/source/common/tcp_proxy/tcp_proxy.cc:204] [C23] new tcp proxy session
2020-01-24T09:26:57.670207814Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.670][32][trace][connection] [external/envoy/source/common/network/connection_impl.cc:293] [C23] readDisable: enabled=true disable=true
2020-01-24T09:26:57.670269893Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.670][32][debug][filter] [src/envoy/tcp/mixer/filter.cc:133] [C23] Called tcp filter onNewConnection: remote <IP_ADDRESS>:48352, local <IP_ADDRESS>:3129
2020-01-24T09:26:57.67036408Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.670][32][debug][filter] [external/envoy/source/common/tcp_proxy/tcp_proxy.cc:347] [C23] Creating connection to cluster BlackHoleCluster
2020-01-24T09:26:57.670441004Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.670][32][debug][upstream] [external/envoy/source/common/upstream/cluster_manager_impl.cc:1268] no healthy host for TCP connection pool
2020-01-24T09:26:57.670498294Z [Envoy (Epoch 0)] [2020-01-24 09:26:57.670][32][debug][connection] [external/envoy/source/common/network/connection_impl.cc:104] [C23] closing data_to_write=0 type

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[x ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
The connection works every time.
Steps to reproduce the bug
Deploy the ServiceEntry above with REGISTRY_ONLY enabled. Loop through a HTTPS endpoint for connection failures.
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
client version: 1.4.3
citadel version: 1.4.3
galley version: 1.4.3
ingressgateway version: 1.4.3
nodeagent version:
nodeagent version:
nodeagent version:
nodeagent version:
nodeagent version:
nodeagent version:
pilot version: 1.4.3
policy version: 1.4.3
sidecar-injector version: 1.4.3
telemetry version: 1.4.3
data plane version: 1.4.3 (5 proxies)
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.5", GitCommit:"20c265fef0741dd71a66480e35bd69f18351daea", GitTreeState:"clean", BuildDate:"2019-10-15T19:16:51Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.9-eks-c0eccc", GitCommit:"c0eccca51d7500bb03b2f163dd8d534ffeb2f7a2", GitTreeState:"clean", BuildDate:"2019-12-22T23:14:11Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}
How was Istio installed?
Istioctl
Environment where bug was observed (cloud vendor, OS, etc)
AWS - EKS.

From you yaml configuration, you are NOT using protocol sniffing for outbound as you define the port name as "https" and "tcp".
The problem is
2020-01-27T11:10:33.18320854Z [Envoy (Epoch 0)] [2020-01-27 11:10:33.183][30][debug][conn_handler] [external/envoy/source/server/connection_handler_impl.cc:179] listener filter times out after 100 ms

The TLS inspector cannot finish the tls inspection within 100ms and fallback to a default filter chain. This PR https://github.com/istio/istio/pull/20248/files should fix the problem. Or you can disable the protocolDetectionTimeout for outbound by setting global.proxy.protocolDetectionTimeout=0s.
The reason why changing the port name to TCP works is that if the port name is defined as "tcp", there is no tls inspector and no TLS inspector invokes.

I agree with you @yxue but I've got the following config deployed;
---
apiVersion: install.istio.io/v1alpha2
kind: IstioControlPlane
spec:
  profile: sds
  values:
    global:
      proxy:
        protocolDetectionTimeout: 0s

I can see "protocolDetectionTimeout":"0s" in istioctl manifest generate -f dev.yaml . I do an apply, delete the pod but issue stays the same. I'll just wait for 1.5 to appear to fix it. :)

@eperdeme could you share the config dump with me? It would help to check the timeout.

Sure, The proxy-config for the pod is;
https://gist.github.com/eperdeme/8bab478f81311d4378617a1943b921a2
So the dev.yaml profile contains the above protocolDetectionTimeout: 0s".
I've then to be 100% sure
dev.yaml
---
apiVersion: install.istio.io/v1alpha2
kind: IstioControlPlane
spec:
  profile: sds
  telemetry:
    enabled: true
  values:
    global:
      proxy:
        protocolDetectionTimeout: 0s
      outboundTrafficPolicy:
        mode: REGISTRY_ONLY
    pilot:
      enableProtocolSniffingForOutbound: false
      enableProtocolSniffingForInbound: false
    kiali:
      createDemoSecret: true
      enabled: true
      hub: docker-registry.prod.mycompany.plc/mirrors/quay.io/kiali
    tracing:
      enabled: true
      provider: zipkin
    nodeagent:
      image: node-agent-k8s
      env:
        CA_PROVIDER: "Citadel"
        CA_ADDR: "istio-citadel:8060"
        VALID_TOKEN: true
    gateways:
      enabled: true
      istio-ingressgateway:
        enabled: true
        autoscaleMin: 2
        autoscaleMax: 6
        ports:
    # Note that AWS ELB will by default perform health checks on the first port
    # on this list. Setting this to the health check port will ensure that health
    # checks always work. https://github.com/istio/istio/issues/12503
        - port: 15020
          targetPort: 15020
          name: http-status-port
        - port: 80
          targetPort: 80
          name: http2
        - port: 443
          name: https
        # GRPC ingress
        - port: 31400
          name: grpc
          nodePort: 31400
        loadBalancerSourceRanges:
          - "<IP_ADDRESS>/8"
          - "<IP_ADDRESS>/16"
          - "<IP_ADDRESS>/23"
        serviceAnnotations:
          service.beta.kubernetes.io/aws-load-balancer-internal: <IP_ADDRESS>/0
    sidecarInjectorWebhook:
      enabled: true
      # If true, webhook or istioctl injector will rewrite PodSpec for liveness
      # health check to redirect request to sidecar. This makes liveness check work
      # even when mTLS is enabled.
      rewriteAppHTTPProbe: true

istioctl manifest apply -f dev.yaml
kubectl rollout restart deployment --namespace nflynn

Hmm, it's weird. The config dump shows the listener filter timeout is still 100ms.
https://gist.github.com/eperdeme/8bab478f81311d4378617a1943b921a2#file-gistfile1-txt-L2247-L2249

@yxue @eperdeme Was this issue fixed in 1.5?

I believe this is fixed by #24763


Button prompt icons?

Initially I wanted to just hide them all but on second thought that might not be a good idea. But need to make sure that they are consistent.

Icons that need to be changed:

X (blue circle)
A (green circle)
Y (yellow, or blue circle with "hold"?)
Lock on (should be same as X)
Autopilot (same as Y)
Roll mode?


Implement device-info-spec in app-netutil

Add support to app-netutil to implement the device-info-spec. Primarily, it consists of pull additional data from the DeviceInfo field in the NetworkStatus.
As part of supporting the device-info-spec, migrate app-netutil to use network-attachment-definition-client data structs. app-netutil redefined some of the data structs from Multus and Userspace CNI. Since network-attachment-definition-client is now the location for these data structs, migrate to app-netutil to use these instead of defining it's own.

/hold
Need to get https://github.com/k8snetworkplumbingwg/network-attachment-definition-client/pull/30 merged first.

/hold cancel

btw, is there any device plugin patch required to make this work?

btw, is there any device plugin patch required to make this work?

https://github.com/k8snetworkplumbingwg/sriov-network-device-plugin/pull/287

@zshi-redhat Sorry, had some comments "pending" that I thought I submitted early last week. Let me know if you have any more comments. I pushed a small change up last week to count the number of "unknown" interfaces and take that into consideration when applying the PCIDEVICE_ interfaces.

@zshi-redhat Sorry, had some comments "pending" that I thought I submitted early last week. Let me know if you have any more comments. I pushed a small change up last week to count the number of "unknown" interfaces and take that into consideration when applying the PCIDEVICE_ interfaces.

@Billy99, I don't have further comments, shall I merge it now or wait until DP/Multus PRs be merged?

@zshi-redhat Sorry, had some comments "pending" that I thought I submitted early last week. Let me know if you have any more comments. I pushed a small change up last week to count the number of "unknown" interfaces and take that into consideration when applying the PCIDEVICE_ interfaces.

@Billy99, I don't have further comments, shall I merge it now or wait until DP/Multus PRs be merged?

Yes. I have other changes coming for exposing hugepages via DownwardAPI. So let's get this in and build on it.


Saved file not in the specified folder

Expected Behavior
I ask the ImagePicker to save in : /storage/emulated/0/Android/media/fr.piscarius.piscarius/PISCARIUS/
    ImagePickerSavePath savePath = new ImagePickerSavePath(context.getExternalMediaDirs()[0] + File.separator + "PISCARIUS", false);
    config.setSavePath(savePath);

Actual Behavior
In the return of the ImagePicker, the URI is correct and the image is retrieve through that in the ImagePicker. If I checked the Image.getPath it is good also. But if I go through an explorer, no file is created on /storage/emulated/0/Android/media/fr.piscarius.piscarius/PISCARIUS/ but it is created in /storage/emulated/0/Android/Pictures.
Steps to Reproduce the Problem
see above
Specifications

Image Picker Version: 3.0.0-beta4
Android OS: 11 (API 30)
Phone: Emulator

android-image-picker 3.0.0
Android 11


adding an option to hide the button

The horizontal space (width) the input needs might extend after initialization causing the layout of the page being broken. If this option is set to true (default=false) the showOn option will always be set to "focus".

Thank you for your contributions.

I just realized that I forgot to add the new option to the README. So you should probably add your own description to the README.


Add doc links to the form overview and user pages

Closes #91.

Apologies, I was having trouble running this for some reason! Verified it and looks good.


Remove svc resource output suggestion in UI

Describe the bug

There is no service any more, do not suggestion svc when referring resource output
Steps to reproduce
Create a new resource , fill form with ${
Result

svc shows

Expected behavior

Do not show svc
Environment

Walrus version: main

Additional context
Add any other context about the problem here.

Environment

Walrus version: main f287a04a0e0e04715f6b161ba77407d0e03d563e
UI version: dev-b6cc5ea

Test Result: pass


Add support to Flutter style by Blazor engine

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
No
Describe the solution you'd like
A clear and concise description of what you want to happen.
make avalonia more reative style, like flutter, which really improve productivity

@inject CounterState CounterState

<ContentView>
    <StackLayout>

        <StackLayout Margin="new Thickness(20)">
            <Label Text="@($"You pressed {CounterState.CurrentCount} times")" FontSize="30" />
            <Button Text="Increment from native"<EMAIL_ADDRESS>Padding="10" />
        </StackLayout>

        <BlazorWebView ContentRoot="WebUI/wwwroot" VerticalOptions="LayoutOptions.FillAndExpand">
            <FirstBlazorHybridApp.WebUI.App />
        </BlazorWebView>

    </StackLayout>
</ContentView>

@code {
    // initialization code
   // blablabla
}

https://devblogs.microsoft.com/aspnet/hybrid-blazor-apps-in-mobile-blazor-bindings-july-update/
@danroth27 please
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
No
Additional context
Add any other context or screenshots about the feature request here.

MAUI the end of 2021 .net 6, MAUI with blazor style 2022 .net 7 or maybe 2023 .Net 8?
And then, .Net UI developer will be less and less then?
if mix avalonia with blazor, that maybe another story!
It’s really cool, will Microsoft give some help @nat @migueldeicaza

Please, keep discussion in already created issue for alternative markup syntax.

https://github.com/AvaloniaUI/Avalonia/issues/2502#issuecomment-631845909

Note that MAUI is going the MVU route with their own syntax and even changes to C# to support it.


feat(xenial_builds): Added systemd service configuration for Front50.

Add systemd service configuration for front50.

LGTM, needs a rebase.


Can not delete repository using gogs

Installed Gogs-Version: <IP_ADDRESS>4 Beta
I have created a private test repository on my gogs-Server, named "us-git". As I don't need it anymore, I want to delete it now. Here are the steps I did:

Clicked on "Delete repository" in the repository options
Typed the name of the repository and my password in the pop-up dialogue
Click the delete button
All I get is "Please make sure the password is correct." (See screenshot, it is in German)

I checked my password several times now, I'm sure that I have no typo here. The online-log
of Gogs shows no error when clicking the delete button, also in 'log/xorm.log' I can not find any hint to this problem. I want to say that I am logged in to my gogs-Server via LDAP, but this workes without a problem.
If I can give you any more info, please give me an advise.
Thanks,
Uwe

PS: Is it possible to "hard-delete" a repository physically from "git/gogs-repositories"? Or will this make trouble?

Hi @turboscholz , thanks for the feedback!
I've pushed code on master branch so that you can help us test again!

Hi @Unknwon
Great, now I can delete my test-repository. Perfect! Thanks allot!!! :)

@turboscholz very well! Thanks your feedback again!


Can add encryption or security packer based DesktopDeployR?

Hi there,
I successfully completed a Shiny desktop deployment using DesktopDeployR framework, but the raw code is't crypted and is readable for others. So is there methods to "protect" raw code after DesktopDeployR processed?
Thanks

@SEvisual  Did you find any way. I also needed this

<EMAIL_ADDRESS>


Add Simple initial design

In this PR I did the following tasks:

This is the first time I will use Gitflow. Keep that in mind!
Write plain HTML markup with minimum styling (just to make the layout),
Use ES6 modules, with import and export.

Thank you, my code reviewer ✔🎉


Unable to run docker commands

Hi,
I am trying to run docker command with your docker image using Azure container service.
But I always get the error as below

Please let me know if I have missed something..!!

I Am also facing the same issue? any help?


Add API route for user bid history

We need an API route that will help us fetch all songs a user has bid on. This will help me finish up #9, which is the user profile screen and shows all bid history.

Claiming this issue. Working on it now.

Reopening to discuss outstanding questions re: filtering bid history
Asking in relation to a broader question / data design decision we need to make (unless there's a better way to query that idk about which is totally possible).
when querying the Play collection, I'm using the userId to find the bids for that user. However, if I use .find() i get all the bids for that user, the associated data in the Play object AND the other bids that are not attached to that userId since I'm not filtering deeply into the array.
E.g. response from GET /api/bid/history?userId=clb9vyqmg0007ufyxb5049st0 using .find()
[{
    _id: ObjectId("6390c04a9388e587a9be7d3d"),
    songId: '5EWPGh7jbTNO2wakv8LjUI',
    status: 'queued',
    queueTimestamp: '1670430794061',
    bids: [
      {
        userId: 'clb9vyqmg0007ufyxb5049st0',
        bidAmount: 25,
        timestamp: '1672504884943',
        rHash: 'd451f5859c0b24ef1e90363510d0c02f5a3afcd61e3b46739c6f2726ef7b307f'
      },
      {
        userId: 'clb9u0o4v0000x98fflh5ftiq',
        bidAmount: 17,
        timestamp: '1672505562332',
        rHash: '8aacd7f5d2e2af163ed232bfb62aa13e0dbd98f959e5e033814c7c96caea186a'
      }
    ],
    runningTotal: 95,
    __v: 0,
    hostId: 'nonce33jfkasjkdf',
    playId: 'clccbkbgx0001uwd83xkl5c2v'
  },
  {
    _id: ObjectId("639155119dc367a1e2bc4b46"),
    songId: '01Lr5YepbgjXAWR9iOEyH1',
    status: 'queued',
    queueTimestamp: '1670468881012',
    bids: [
      {
        userId: 'clb9vyqmg0007ufyxb5049st0',
        bidAmount: 20,
        timestamp: '1670468881012',
        rHash: 'c18e7e3e5f9c0fc3f974dae001bf4fd75f63dc53660ebf29ac0ad64f1a3d164c'
      }
    ],
    runningTotal: 20,
    __v: 0,
    hostId: 'nonce33jfkasjkdf',
    playId: 'clccbkc360002uwd8033sfhjt'
  },
  {
    _id: ObjectId("63b06635c7c1be86e50aed68"),
    songId: '5uU1uuyaUIBOiiUSQG7Wl3',
    status: 'queued',
    queueTimestamp: '1672504884943',
    bids: [
      {
        userId: 'clb9w3wm40008ufyx9553dghn',
        bidAmount: 25,
        timestamp: '1672504884943',
        rHash: 'd451f5859c0b24ef1e90363510d0c02f5a3afcd61e3b46739c6f2726ef7b307f'
      },
      {
        userId: 'clb9vyqmg0007ufyxb5049st0',
        bidAmount: 17,
        timestamp: '1672505562332',
        rHash: '8aacd7f5d2e2af163ed232bfb62aa13e0dbd98f959e5e033814c7c96caea186a'
      }
    ],
    runningTotal: 42,
    __v: 0,
    hostId: 'nonce33jfkasjkdf',
    playId: 'clccbkcfb0003uwd8d38w0wl5'
  }]

In this design, I have to use nested .filter() methods to remove bids without that userId.
const plays = await Plays.find({
            bids: { $elemMatch: { userId: req.query.userId } }
        });

const bids = plays.filter((play: Play) =>
            play.bids.filter((bid: any) => {
                return bid.userId === userId;
            })
        );

return res.status(200).json({ success: true, new: false, message: bids });

On the flip, if I use .aggregate, I can filter out the bids in each Play that do have the userId and return only those leaving out extra bids on that play by other users. However, I ONLY get back an array of bids and no Play info.
E.g. response from GET /api/bid/history?userId=clb9vyqmg0007ufyxb5049st0 using .aggregate()
[{
            "_id": "6390c04a9388e587a9be7d3d",
            "bids": [
                {
                    "userId": "clb9vyqmg0007ufyxb5049st0",
                    "bidAmount": 25,
                    "timestamp": "1672504884943",
                    "rHash": "d451f5859c0b24ef1e90363510d0c02f5a3afcd61e3b46739c6f2726ef7b307f"
                }
            ]
        },
        {
            "_id": "639155119dc367a1e2bc4b46",
            "bids": [
                {
                    "userId": "clb9vyqmg0007ufyxb5049st0",
                    "bidAmount": 20,
                    "timestamp": "1670468881012",
                    "rHash": "c18e7e3e5f9c0fc3f974dae001bf4fd75f63dc53660ebf29ac0ad64f1a3d164c"
                }
            ]
        },
        {
            "_id": "63b06635c7c1be86e50aed68",
            "bids": [
                {
                    "userId": "clb9vyqmg0007ufyxb5049st0",
                    "bidAmount": 17,
                    "timestamp": "1672505562332",
                    "rHash": "8aacd7f5d2e2af163ed232bfb62aa13e0dbd98f959e5e033814c7c96caea186a"
                }
            ]
        }]

Use first design
User is first in list
Need User object, not userId

Closed with #64
Yay @bnonni 🎉


extend/os/mac: ensure writable file for codesign

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

This should fix #13903. I don't have an ARM machine at the moment so would appreciate some more eyes on this. Regarding test coverage, I'm not sure this is currently covered by existing tests. If we do want to add tests for this change, I'd be happy to add some if someone can point me to where/which file the tests should be added in.

Review period will end on 2022-10-05 at 03:05:21 UTC.

Makes sense to me, thanks @alebcay!

Review period skipped due to critical label.

It'll be caused by GitHub gradually migrating macos-latest to be macos-12.
If we want a quick fix, can make the CI use macos-11. I'll take a look at reproducing the macOS 12 issues later this week.

It'll be caused by GitHub gradually migrating macos-latest to be macos-12.
If we want a quick fix, can make the CI use macos-11. I'll take a look at reproducing the macOS 12 issues later this week, but it looks to be various issues such as the default git branch defaulting to main rather than master.

Works for me 👍🏻

#13979 will fix CI.

Thanks @alebcay!


Best way to draw that accounts for aspect ratio❓

Question
Hey hey,
I am trying to draw a pose with this repo. It works great for normal 16:9 iphones but when someone uses an phone pro max which is 19.5:9 then the pose doesn't grow with the screen.

This is my current functions for rendering the pose:
 const [frameWidth, setFrameWidth] = React.useState(Dimensions.get('screen').width);
  const [frameHeight, setFrameHeight] = React.useState(Dimensions.get('screen').height);

  const getFrameSize = (): number[] => {
    let width: number, height: number;
    if (Platform.OS === "android") {
      if (
        frameWidth > frameHeight &&
        Dimensions.get("window").width > Dimensions.get("window").height
      ) {
        width = frameWidth;
        height = frameHeight;
      } else {
        width = frameHeight;
        height = frameWidth;
      }
    } else {
      width = frameWidth;
      height = frameHeight;
    }
    return [width, height];
  };

  const getViewBox = () => {
    const frameSize = getFrameSize();
    const viewBox = "0 0 " + frameSize[0] + " " + frameSize[1];
    return viewBox;
  };

const renderPose = () => {
    if (poses != null && poses.length > 0) {
      const selectedJoints = [
        poses[0].LeftShoulder,
        poses[0].RightShoulder,
        poses[0].LeftHip,
        poses[0].RightHip,
        poses[0].RightKnee,
        poses[0].LeftKnee,
        poses[0].RightAnkle,
        poses[0].LeftAnkle,
        poses[0].RightShoulder,
        poses[0].LeftToe,
        poses[0].RightToe,
        poses[0].RightElbow,
        poses[0].LeftElbow,
        poses[0].RightThumb,
        poses[0].LeftThumb,
      ]
        .filter((k) => (k.inFrameLikelihood ?? 0) > 0.3)
        .map((k, index) => {
          // Flip horizontally on android or when using back camera on iOS.
          return (
            <Svg >
              <Circle
                key={`skeletonkp_${k.type}`}
                cx={k.position.x}
                cy={k.position.y}
                r={25}
                strokeWidth="2"
                fill="transparent"
                stroke={Theme.COLORS.WHITE}
              />

              {/* <Circle
                key={`skeletonkp_${k.type}${index}`}
                cx={k.position.x}
                cy={k.position.y}
                r={10}
                strokeWidth="0"
                fill="#FAF9F6"
                stroke="transparent"
              /> */}
            </Svg>
          );
        });
      return (
        <Svg
          viewBox={getViewBox()}
        >
          {selectedJoints}
        </Svg>
      );
    } else {
      return <View></View>;
    }
  };

### What I tried

I've tried adding in the output widths and preview widths as I had before with [tfjs](https://github.com/tensorflow/tfjs-examples/blob/master/react-native/pose-detection/App.tsx#L161) but that didn't show any points at all. When I was first using tfjs, the pose worked well with all screen sizes.

### VisionCamera Version

"^2.15.2"

### Additional information

- [x] I am using Expo
- [X] I have read the [Troubleshooting Guide](https://react-native-vision-camera.com/docs/guides/troubleshooting)
- [X] I agree to follow this project's [Code of Conduct](https://github.com/mrousavy/react-native-vision-camera/blob/main/CODE_OF_CONDUCT.md)
- [X] I searched for similar questions [in the issues page](https://github.com/mrousavy/react-native-vision-camera/issues?q=is%3Aopen+is%3Aissue+label%3A%22❓+question%22) as well as [in the discussions page](https://github.com/mrousavy/react-native-vision-camera/discussions) and found none.

This may actually be a bug. When I console log the frame height and width of the iphone pro max, the ratio is still 16:9. Looking through the formats, it doesn't appear there are any 19.5:9 options that even appear.

Hi, i have similar problem, did you resolve it ?


let%hw as stritem

It would be useful to be able to write
let%hw x = ...

as top level expressions, which I think means adding some mapper clauses to rewrite appropriate stritem bindings.

I am thinking to apply [%hw] recursively to all expressions contained in that let%hw structure. Is that what you also have in mind ? Or are you thinking about something less thorough ?

I think there are 2 separate issues here (but I might be wrong as my ppx foo is only so so).
let x =
  let%hw x = <expr> in
  ...

in this context, the rewrite should be (fully) recursive within <expr> and would look equivalent to
let x =
  let x = [%hw <expr>] in
  ...

In the following case I believe the let%hw won't be picked up by the expression mapper, as it is a stritem (ie a top-level let without an in part).  However, the logic applied to <expr> is exactly the same.
let%hw x = <expr>;;

or equivalently
let x = [%hw <expr>];;

FWIW - the let%hw as a stritem thing is mostly useful when fiddling around in utop and not hugely important - it's something I can patch in at a later date if you like.

In the following case I believe the let%hw won't be picked up by the expression mapper, as it is a stritem (ie a top-level let without an in part). However, the logic applied to  is exactly the same.

True. However, the current policy for mapping the let%hw expression is to remap the left-hand side of the in and not the right-hand side. For example:
let%hw v = expr1 in expr2;;

is mapped to:
let v = [%hw expr1] in expr2;;

For the structural let%hw construct, we could either follow that policy, or also remap the right-hand side expression. For example:
let%hw v =
  let w = expr1 in expr2;;

would be mapped to:
let v =
 let w = [%hw expr1] in [% expr2];;

My thinking is that the expectation of the scope of a structural let construct is inclusive, i.e. all expressions after the = should be considered as part of the [%hw] scope.

Agreed but with the slight clarification that I would view
let%hw v =
  let w = expr1 in expr2;;

as
let v =
 [%hw let w = expr1 in expr2];;

That is (let .. = .. in ..) is an expression itself.  This would also extend to cases like
let ... =
  let%hw x = ...
   let y = ... in
   y
  in
  ...

where y would be rewritten as it is an expression within x.
Anyway, I am pretty sure we are talking about the same thing!

where y would be rewritten as it is an expression within x

I see your point. However, I fear that rewriting all expressions in the scope of a let%hw rewritten variable may be too inclusive. Following that scheme, it would be easy to lose track of context in large functions, especially in the future if we also add support for rewriting match and if expressions.

Anyway, I am pretty sure we are talking about the same thing!

Most likely :) Let me write the fully recursive version for the structural let. We can resume the discussion on expression let from that point on.

The solution I came up with implements two extensions:

%hw which does only recurse on the expression of a let value binding
%hw' which does recurse on each expression

The structural let binding only supports [%hw] as extension but actually defaults to [%hw']. This solution leads to the following combinations (cf. unit test file):
A. Structural let, full recursion
let%hw combine_signals a b =
  let sub_a = a.[11,08]
  and sub_b = b.[15,12]
  in
  sub_a @ sub_b

let structural_let context =
  let s0 = const "16'h0F00" in
  let s1 = const "16'hF000" in
  assert_equal (combine_signals s0 s1) (const "8'hFF")

B. Expression let, full recursion
let ext_rec_let context =
  let s0.       = const "16'h0F00" in
  let s1        = const "16'hF000" in
  let%hw' sub_0 = s0.[11,08] in
  let sub_1     = s1.[15,12] in
  let res       = sub_0 @ sub_1 in
  assert_equal res (const "8'hFF")

C. Expression let, left-hand recursion
let ext_rec_let context =
  let s0       = const "16'h0F00" in
  let s1       = const "16'hF000" in
  let%hw sub_0 = s0.[11,08] in
  let%hw sub_1 = s1.[15,12] in
  let%hw res   = sub_0 @ sub_1 in
  assert_equal res (const "8'hFF")

Hmmm, I am going to have to have a think about this.  I don't understand the "B" case at all - if you thought I was advocating that the rewrite should extend beyond the in (as this appears to do) then sorry for the misunderstanding but I don't see that as sensible.
Also, playing with the latest code I can do
let foo =
  let%hw x =
    let x = a + b in
    let y = a - b in
    x,y
  in
  x;;

which I /was/ advocating for, but didn't think you agreed with.

Hmmm, I am going to have to have a think about this. I don't understand the "B" case at all - if you thought I was advocating that the rewrite should extend beyond the in (as this appears to do) then sorry for the misunderstanding but I don't see that as sensible

This is indeed what I thought you were advocating for :p I don't think that is sensible either :)
The B case can stay internal and not be advertised. It is still necessary for the A case and the case you mention. I can go back to using hw*, invalid syntax as far as the lexer is concerned but accepted by AST parser.

which I /was/ advocating for, but didn't think you agreed with

That hit me last night. If the rewriting applies to the expression of the value binding, it must apply recursively.

I came up with another method to achieve the same result without using another extension. The rewriter now simply carries around a recurse expression attribute that is used to decide whether or not the rewriter needs to recurse on all sub-expressions. As a result, the "B" is not available anymore.

I am still not exactly sure whether we mean the same thing when we say recursion or not, and especially what it means when you turn it off.
I'm having a good look through the code to see what it is doing.

I'm having a good look through the code to see what it is doing.

Ok. Hopefully that will clarify things :)

I'm having a good look through the code to see what it is doing.

Ok. Hopefully that will clarify things :)
EDIT - here are the rules that the rewriter basically follows:

In case of an expression that is not a let, rewrite if necessary (bin ops, String.get, etc..)
 In case of a let, that means all value-bind expressions as well as the expression after the in keyword.
As a result, when a value-bind expression is a let expression, as for x below:
let foo =
  let%hw x =
    let x = a + b in
    let y = a - b in
    x,y
  in
  x;;

then all sub-expressions of that expression will be rewritten, i.e. a + b and a - b and x, y.

That's very helpful, thanks.

https://gist.github.com/andrewray/2c8b2cda626fcd3bed8ef03c8ed3c5bb
This is my take.  To be discussed...

Playing around with your gist, I saw the following behavior:
let some a b =
  let%hw c = a + 1 in
  b + c

producing:
let some a b =
  let c = a +: 1 in
  b +: c

where both the value-bind expression and the core expression are rewritten. Is that the expected output ? I thought from a previous comment that you (as I) did not find that to be sensible.

Your right it shouldn't be like that.  I had a quick look at some ppx documentation and apparently
let%hw p = e0 in e1

is a shortcut for
[%hw let p = e0 in e1]

Its fixable by changing the expr mapper as follows
    expr = (fun mapper expr ->
      match expr with

      (* 'let%hw pat = <expr> in ...' *)
      | [%expr [%hw let [%p? var] = [%e? e0] in [%e? e1]]] ->
        [%expr let [%p default_mapper.pat mapper var] =
          [%e hwexpr {mapper with expr=hwexpr} e0] in
          [%e default_mapper.expr mapper e1]]

      (* [%hw expr] *)
      | [%expr [%hw [%e? e]]] ->
        [%expr [%e hwexpr {mapper with expr=hwexpr} e]]

      | _ -> default_mapper.expr mapper expr);

So now we are encountering the same conundrum as I had previously :) This works well except when:

we are rewriting a let structure item
the value-bind is itself made of let expressions

In both of these cases, the expectation is that all expressions (not just value bindings) are rewritten. To address that problem, I carry down a hwrec attribute that specify that such a behavior must be implemented.

For 1. it works the way I would expect:
let%hw some a b =
  let c = a+1h in  (* everything rewritten *)
  b+c;;

Similarly for 2.
et some a b =
  let%hw c =
    let d = a+1h in d (* rewritten as expected - its the expression inside %hw *)
  in
  b +: c;;  (* not rewritten - must use +: *)

This is the behaviour I expect, do you think this is wrong, or have a broken example?

My bad, You are perfectly right. It works as expected.

Actually, there is a snag with multiple value binds:
let some a b =
  let%hw c = a + 1 and d = b + 1 in
  c + d

let some a b =
  let c = a +: 1 and d = b +: 1  in
  c +: d

The initial let expression is not matched and it defaults to the next, more general one. I don't think it's possible to extract list of value binds using metaquot, so I can just bring in the solution I currently have for that.

There's another bug as well
let x =
  let%hw a = 3h in
  let%hw b = 2h in (* fails here *)
  a +: b;;

This is the fix
     (* 'let%hw pat = <expr> in ...' *)
      | [%expr [%hw let [%p? var] = [%e? e0] in [%e? e1]]] ->
        [%expr let [%p default_mapper.pat mapper var] =
          [%e hwexpr {mapper with expr=hwexpr} e0] in
          [%e (*here*) mapper.expr mapper e1]]

Hmmm, I note other issues like let rec ....  Are we fighting the system here?
I thought let%hw p = e0 in e1 made sense only as let p = [%hw e0] in e1 but ocaml has different rules.  Maybe we would be better just doing what the system expects?

Ok, so lwt.ppx seems to do something similar to what we want https://github.com/ocsigen/lwt/blob/master/src/ppx/ppx_lwt_ex.ml
I think it deals with let%hw, let%hw ... and ... and and disallows let%hw rec.

See #11, closing.


Specify a file as optional

Requested by @lmperry: it would be nice to have a gear specify that one or more input files as optional. There are some specific algorithms where this would be ideal, if not required.
I think we'd add
"optional": {
    "type": "boolean",
    "description": "If true, the input is marked as optional and may not be provided"
}

to the inputs property list. Then we update the gear spec to mention that, for optional inputs, the entire folder (such as /flywheel/v0/inputs/my-input) may be missing. The algorithm will be expected to check & handle that case.
Pinging @nagem for her thoughts, and @ryansanford for a head's up on when we'd want to do this.

Reconstruction gears, which may or may not have a B0 field map as input are immediately impacted by the lack of optional input files. This also impacts a number of alignment gears (some of which are being built now and others which are planned) which could use a provided file or a template included in the gear as an alignment target. There are many other gears that are in the planning stages that all follow that same principle and have optional input files. Consider fMRI processing that may or may not have physio data as input, etc.


chore: Bump for node 18.17.0 to support latest next.js 14 release

Need to support node 18.17.0 for latest next.js version https://nextjs.org/blog/next-14#other-changes
Just updated to the latest chrome version because it wasn't there either
Only saw the latest node version with browser chrome as cypress/browsers:node18.12.0-chrome107
Also it's cool if we go past 18.17.0 to 18.18.2 for hydrogen tag https://nodejs.org/en/download/releases

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Really appreciate the effort keeping these up to date. This is ready for review but can't seem to trigger that per the docs. cc @chrisbreiding @astone123

related to https://github.com/cypress-io/cypress/issues/28185

@JackHowa
Is Next.js 14 compatible with the existing cypress/browsers:node-20.9.0-chrome-118.0.5993.88-1-ff-118.0.2-edge-118.0.2088.46-1 if Next.js 14 specifies "Node.js 18.17 or later"?

@JackHowa
I don't speak for the Cypress team here as I'm a community contributor only, so I can't say in advance what their response to your PR will be.
Just in case you are not aware and in case it might meet any urgent need, you can generate your own Cypress Docker image using the Cypress Docker factory with a choice of your own version parameters.

@JackHowa Thank you for providing details in the PR description to describe your needs for this image. Like @MikeMcC399 mention, we offer the Cypress Docker factory to allow you to build custom images that can be more specific to your test needs. Wanting a specific Node version than what we offer is a great use-case for leveraging the Factory image. You can use this example as a base-line for defining your own with just Chrome.


Rate candidates

Users can add:

Classes of True, False, Unsure
Tags such as AGN, Stars, Pulsars, Unknown, etc.

Users should be able to add their own tags.

Completed in #29


Create codeql.yml

Summary
Impact
Testing

@TimoSairiala I found this old PR, no idea whether it does something sane...


Update spec to final VEX-WG document

This PR updates the OpenVEX spec to match the final minimal requirements for VEX as defined by the VEX Working group and published by CISA.
The summary of changes to the spec is as follows. Except for the document version changing from string to integer, all fields are new so should not break existing consumers.
Document

Added wording on the context location to make it clearer it is not versioned until 1.0 (ref https://github.com/openvex/spec/issues/20)
version is now an integer
Added last_updated to record changes to the doc
author is now recommended to be a machine-readable identifier
role is now optional :tada:

Statement

@id takes an IRI to identify the statement and make it referenceable
version an integer field, accounts for changes in the statement. Defaults to zero, and may be omitted when so.
last_updated added to the statement
supplier an identifier of the supplier of the document

Fixes #18
Signed-off-by: Adolfo García Veytia (Puerco<EMAIL_ADDRESS>

Please note that supplier disappeared from the document metadata and is now an optional field at the statement level. Its meaning is different now too, now denoting the supplier of the product (previously it specified the supplier of the document).

Thanks for your reviews everyone, I'm tagging v0.0.2 of the spec :tada:
Please note that these changes will not get a technical release. The minor adjustments in v0.0.2 will be rolled into the upcoming updates to the OpenVEX libraries and tools as part of the v0.1.0 revision.

Congratulations team!  To 0.0.2 and BEYOND!!!
Cheers,
CRob
Director of Security Communications
Intel Product Assurance and Security
From: Puerco @.>
Sent: Tuesday, July 18, 2023 7:57 PM
To: openvex/spec @.>
Cc: Robinson, Christopher @.>; Comment @.>
Subject: Re: [openvex/spec] Update spec to final VEX-WG document (PR #25)
Thanks for your reviews everyone, I'm tagging v0.0.2 of the spec 🎉
Please note that these changes will not get a technical release. The minor adjustments in v0.0.2 will be rolled into the upcoming updates to the OpenVEX libraries and tools as part of the v0.1.0 revision.
—
Reply to this email directly, view it on GitHubhttps://github.com/openvex/spec/pull/25#issuecomment-1641140485, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQRFDLEO7A56OWN5L6SOFULXQ4PGPANCNFSM6AAAAAAYTKRALM.
You are receiving this because you commented.Message ID<EMAIL_ADDRESS>


Left nav divider demo

This PR is (unfinished and) for demo purposes currently. I'll probably delete this branch when it is no longer needed. 🙂
...
Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6730e012ab98e4f7635f844e--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6745e204ca057a73caa899c8--harness-developer.netlify.app


Hiding xaxis labels hide grid lines

Bug report
Codepen
https://codepen.io/pvictor-r/pen/qBabJaG
Explanation
When hiding xaxis labels with xaxis: {labels: {show: false}}, grid lines on xaxis are hidden despite setting :
grid: {
  xaxis: {
    lines: {
      show: true
    }
  }
}

Would it be possible to keep grid lines when hiding labels ?
Thank you !

Sure, fixed it


Dependabot PRs aren't Distinguishable

Currently the Standard Library Dependabot PRs are not distinguishable from the other PRs by looking at the title. The PR title should be changed to [Dependabot] ___.
@TharindaDilshan

Have we looked into whether we can use the name "Dependabot"? This is referring to a specific product isn't it?

Have we looked into whether we can use the name "Dependabot"? This is referring to a specific product isn't it?
Do they have copyrights for the name?

Thanks for pointing this out. Let's use keep the [Automated] for now.

Have we looked into whether we can use the name "Dependabot"? This is referring to a specific product isn't it?
Do they have copyrights for the name?

Thanks for pointing this out. Let's use keep the [Automated] for now.


Update to FVTTv10 + Bug fixes + Feature overhaul

Features

Update to Foundry VTT v10
Add compatibility with PopOut! module windows
Show the names of users who reacted on hover
Bumped emoji picker library to Picmo: emoji search now also searches tags
Emojipleter: Suggest emoji completions when typing colon : in chat

Uses the same search as Picmo, so it also searches emoji tags
Works with custom emojis
Toggleable in settings

Compact emoji button (for entering emojis in chat) and compact reaction button to take up less space

Toggleable in settings
Compatible with DF Chat Enhancements module roll buttons and merge messages feature

Add settings to show/hide specific emoji categories
Add settings to show/hide the recently-used emojis category and the max number of recent emojis to display

Bug Fixes

#14: will now warn the user if they don't have permission to use File Browser (which prevents them from using custom emojis) instead of breaking emoji picker functionality
Fix rendering for keycap emojis
Fix rendering for emojis made up of multiple Unicode characters e.g. flags

Known Issues

The "eye in speech bubble" emoji still renders incorrectly due to a bug in twemoji-parser; it renders as a separate eye and speech bubble emoji

@Asinphi I am sure you have found out that the CDN used for the Emojis went down XD. Any chance you plan on patching that too? ❤

I just saw this in my inbox. For some reason I wasnt sent the actual PR email

Patched the maxcdn issue by using the SVG twemojis that the emoji picker has stored 👍


Optimize number of transactions sent in refresh

Do not do a blank transaction if the record hasn't changed, because it causes extra 2 queries, transaction BEGIN and END. And wrap updating&deleting in 1 big transaction. Doing transaction per     update is expensive, since we do BEGIN, update, END, so 3 queries per one updateand the same for delete. With remote DB, this will add a lot of processing time.
Example:
ca = CustomAttribute.first; ca1 = CustomAttribute.second

# This does 1 BEGIN and COMMIT query for all updates inside
ActiveRecord::Base.transaction { ca.update_attributes!(:section => "labels1"); ca1.update_attributes!(:section => "labels1") }

# This does 1 BEGIN and COMMIT for each update
ca.update_attributes!(:section => "labels1"); ca1.update_attributes!(:section => "labels1")

Partially fixes:
https://bugzilla.redhat.com/show_bug.cgi?id=1436176

The performance improvements are mainly visible with second+ refresh, since creating objects by  association.push(new_records) is already using 1 transaction for many records created. We are just optimizing that number of transaction also for nested objects created.
** stats are being measured with applied PRs https://github.com/ManageIQ/manageiq/issues/13594 https://github.com/ManageIQ/manageiq/issues/14542
Performance improvements, given we are saving 290k object to the DB during refresh:

refresh
number of queries done during refresh
time on a local db
time on a remote db with ~5ms ping

1st refresh before
421 146
~18min
~53min

1st refresh after
385 162
~18min
~50min

improvement
9.3% smaller
0% faster
6% faster

2nd+ refresh before
786 380
~14min
~80min

2nd+ refresh after
135 198
~7min
~18min

improvement
481% smaller
100% faster
344% faster

** the second refresh is measured when nothing is updated (the same VCR). But this PR brings a big speedup when we do update/delete records, since we do only 1 query per updated/deleted item, not 3 queries, like we did before this PR.

I own a commit in this PR, so assigning over to @Fryguy

@Ladas Can you show some statistic on improvement?  Matrix of Remote DB/Local DB x Initial refresh/second refresh

Overall looks good...couple of minor things.

@Fryguy I've fixed the minor things and added the table showing the performance improvement in %

@simaishi euwe backport is at https://github.com/ManageIQ/manageiq/pull/14674

@miq-bot add_labels performance

Fine backport details:
$ git log -1
commit 130b0a6a6bd2a53fa35f507fb95a95818f33a522
Author: Jason Frey<EMAIL_ADDRESS>Date:   Fri Apr 7 11:46:20 2017 -0400

    Merge pull request #14670 from Ladas/optimize_number_of_transactions_sent_in_refresh

    Optimize number of transactions sent in refresh
    (cherry picked from commit 73555f3fe14869fb35e274dbf3e6eb1a8f85d3d3)

Backported to Euwe via https://github.com/ManageIQ/manageiq/pull/14674


Player Data Reset Cmd

Requesting a command to reset/remove a players data from the db

Closed via #33 - you can delete user data backups


selenoid/android - Google Play service is not installed

Hi everyone,
I'm trying to play with Android emulators with solenoid and it works nice, but I have one big issue: all android emulators don't have installed Google Play service, and my test application is showing the error on start:
This app won't run without Google Play services, which are missing from your phone.

Is there an option to automatically install Google Play services on my emulator? Is there any options/workaround for the issue?
Similar issue with other android emulators in docker https://github.com/budtmo/docker-android/issues/130

@TimurNurlygayanov there is such option in automation script we are using to build these images. However with Google Play services installed images for some reason do not start automatically (they block on the screen requiring to confirm some permission).

@TimurNurlygayanov there is such option in automation script we are using to build these images. However with Google Play services installed images for some reason do not start automatically (they block on the screen requiring to confirm some permission).

@vania-pooh could you please describe which option do you mean or share some examples?
Thank you!

@vania-pooh could you please describe which option do you mean or share some examples?
Thank you!

@TimurNurlygayanov take a look at this script: https://github.com/aerokube/images/blob/master/selenium/automate_android.sh#L91-L101

@TimurNurlygayanov take a look at this script: https://github.com/aerokube/images/blob/master/selenium/automate_android.sh#L91-L101

Ok I will try )

It works! @vania-pooh Thank you!
Just create a short description of how to build these images https://github.com/TimurNurlygayanov/test-tasks-example/blob/master/selenoid-android/README.md


Get next date with events

Hi,
I've been wondering if it's possible to get and display the next date with events?
Ex:
"The next event will be on 2016-08-30"
Thanks.

Sorry @extravaganza47 you can't, but you are free to implement that in your template.
Also, for the future this issues list unfortunately is only for bugs :) https://github.com/kylestetz/CLNDR#submitting-issues


Configure a workflow to deploy packages to NPM

Create a workflow that deploys the packages to NPM.

Closed at #17


Does not run all Python code; also, runs scripts out of order

Issue Type: Bug
Problem:
VS Code fails to run the last line of Python code in the terminal. Additionally, VS Code seems to run Python code out of order.
Environment:
Conda environment, Python 3.6.5; also reproduced with standard non-conda Python environment. This occurs on both my Mac and my PC.
Reproduce:
while True:
    print('Who are you?')
    name = input()
    if name != 'Joe':
        continue
    print('Hello, Joe. What is the password?')
    password = input()
    if password == 'swordfish':
        break
print('Access granted.')

Expected Result:
Terminal will ask 'Who are you?' and prompt the user to enter input. If the user does not enter 'Joe,' then the terminal will loop 'Who are you?' until the condition is satisfied. When the user enters 'Joe,' the terminal will print 'Hello, Joe. What is the password?' and prompt the user to enter input. If the user does not enter 'swordfish,' they will be prompted to re-enter the password. If the user enters 'swordfish,' then terminal will print 'Access granted.'
Actual Result:
'Access granted'  is never printed even when the conditions are met. Similarly, VS Code terminal will show
Who are you?
print('Access granted.')
Who are you?
#prompt for input

instead of exclusively stating 'Who are you?' and requesting input. The input works. However, 'Access granted' never appears in the terminal.
I tested the code in Spyder and it ran as expected without issue.
I have the exact same problem with every single Python script I have in VS Code, not just this one. The issue is reproducible on my Mac, my home PC desktop, and my work PC desktop.
Any help is greatly appreciated.
VS Code version: Code 1.27.1 (5944e81f3c46a3938a82c701f96d7a59b074cfdc, 2018-09-06T09:14:32.385Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
16.00GB (5.48GB free)

Process Argv
/private/var/folders/b3/mjkb2pk94zzg9l3kckj6sg740000gn/T/AppTranslocation/8C67BF6E-CB78-4445-A6ED-56210A00CEC8/d/Visual Studio Code.app/Contents/MacOS/Electron -psn_0_2691729

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

vscode-django
bat
0.17.0

jupyter
don
1.1.4

python-extension-pack
don
1.4.0

MagicPython
mag
1.0.12

anaconda-extension-pack
ms-
1.0.0

python
ms-
2018.8.0

vscode-yaml
red
0.0.16

python
tht
0.2.3

vscodeintellicode
Vis
1.0.3

jinja
who
0.0.8

I appear to have resolved the matter by utilizing the Debug window, which might have been the intention all along. The terminal still reproduces that which I described in my post, but I can successfully execute the commands in the Debug window. I am still new to coding and VS Code, so this might have merely been a noob error.


feature request: Is there any way to create custom strategy?

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
Is there any way to create custom strategy? Use case: use custom command runner, for runnig command outside neovim (like https://github.com/lolpie244/simple-kitty-runner.nvim)
Provide background
No response
What is the significance of this feature?
nice to have
Additional details
No response

Sure, you just need to implement all the required methods. The type is defined here:
https://github.com/stevearc/overseer.nvim/blob/a6dc0600f675f10b8840c61d3f9d72fdf8cf970c/lua/overseer/strategy/init.lua#L6-L14
You can look at the other strategies in that dir to see how they do things. The test strategy is nice in that it shows you exactly what the responsibilities of a strategy are with minimal code.

@stevearc, should I somewhere add this class? Or just make fork and add it inside code?

The strategy is passed in as a name and is loaded on this line:
https://github.com/stevearc/overseer.nvim/blob/fdcd46ce738ef342bce38e5433df004ebdab3a1a/lua/overseer/strategy/init.lua#L44
This means that if your strategy is "mystrategy", we'll be calling require("overseer.strategy.mystrategy"). So you can put the code in the file <runtimepath>/lua/overseer/strategy/mystrategy.lua. See :help runtimepath for more info, and :=vim.fn.stdpath("config") to see the most likely path to use as the root.


ColorBox not rendering properly

Hello,
I'm running into a issue every once and a while when I launch colorbox for the first time on a browser, where the CSS is all messed and the inner iFrame does not properly get loaded.  It is very sporadic, and I can't seem to reproduce the issue, but I have a screen shot of what the error looks like.  If anyone has a fix or a way to resolve this bug please let me know.
Screenshot Error
Thank you!

Make sure you load the CSS before the script.
On Mon, Nov 21, 2011 at 11:45 AM, Steve Barbera
<EMAIL_ADDRESS>wrote:

Hello,
I'm running into a issue every once and a while when I launch colorbox for the first time on a browser, where the CSS is all messed and the inner iFrame does not properly get loaded.  It is very sporadic, and I can't seem to reproduce the issue, but I have a screen shot of what the error looks like.  If anyone has a fix or a way to resolve this bug please let me know.
Screenshot Error
Thank you!

Reply to this email directly or view it on GitHub:
https://github.com/jackmoore/colorbox/issues/139

--
Jack Moore
jacklmoore.com
twitter.com/jacklmoore

I'm having this problem with version 1.3.19 as well.

CSS is loading before JS
JS is the minified version of the latest 1.3.19 from github.

It seems like the middle float elements are being set to the full width and thus wrapping when they shouldn't.
Version 1.3.18 is working just fine in my case.

artsyca, could you provide a link or demo?

I'm having a related issue, changing any of the examples to use jQuery 1.3.2 causes a lot of rendering problems. :(

Ok, I see that happening as well.  I'll look into it.

Hi,
I m also seeing the same problem. I could run a demo tutorial and it renders fine. For my project, I m using

I m using SharePoint 2010, SPServices, jquery and json object carry the values of href. Those href values are wrapped around p tags. Any thoughts? Thanks.


iPad Pro 11-Inch (2018) Not Supported

The new iPad Pro 11-Inch has rounded corners and no physical home button similar to the new iPhones.
When using this safe area view, the status bar is accounted for as expected, but the bottom area with the virtual home button is NOT.

Doesn't seem to be working for me.

this should be resolved in version 1.0 which was released today, thanks for reporting!


ci(documentation-website): lighthouse check with nginx

Replace nodejs lighthouse test with nginx because of poor performance score.
Closes #953
The Pull Request is ready

[ ] fixes #
[ ] all actions are passing
[ ] only fixes a single issue

Overview

Review points

Documentation-Website

[ ] mobile view is usable
[ ] desktop view is usable
[ ] no a-tags are used directly (NavLink, MailLink, ExternalLink instead)
[ ] all new texts are added to the translation files (at least the english one)
[ ] tests have been added (if required)
[ ] shared code has been extracted in a different file

Notes

Let me know if more changes are needed, @Idrinth


Proposal List - Duplicate Content & not filtered by DAO

Reproduce:

Go to e.g. https://dao-dev.hypha.earth/testdao/proposals
Login as a member

Experienced behaviour:
I see a lot of porposals duplicated and also proposals that were created in other DAOs.
Expected behaviour:
I should see every proposal only once and only proposals that were created in the TEST DAO

@DaniPoot Filtering by DAO seems to work but it seems like I still see a lot of duplicate proposals:

@DaniPoot cool, I wasnt sure if they are the same. I will close the ticket


[Improvement] Usage of <stdint.h> as often as possible for better cross-platform compatibility (+ easier readability)

Type of Improvement

Refactoring code

Proposal
Instead of using int, long, unsigned longetc., we should use the types defined in<stdint.h>(e.g:int64_t, uint64_t, etc.):

better cross-platform compatibility.
easier readability, as we know how many bytes/bits is used by the variable.

BREAKING CHANGE

fix!: usage of stdint instead of int types for cross-platform compatibility


[Docker] Update the tag of the built and published docker images

Two main changes:

Always tag the latest image with latest
Use a shortened version of the sha

Where images are stored: https://github.com/buildwithgrove/path/pkgs/container/path


How do I download posts made in the previous 30 hours?

--post-filter="date_utc <= datetime(2018, 5, 31)"
I've got this, but I'm not sure how to change it in a way that would allow me to download only posts made in the previous 30 hours from the moment I run the command (so if the command is run on June 2nd at 16:00, I only want to download posts made after June 1st at 10:00.

Under Linux:
--post-filter="date_utc <= datetime($(date -d "@$(expr $(date +%s) - 108000)" "+%Y, %m, %d, %H, %m"))"


removed invalid excludes

Closes: https://issues.redhat.com/browse/COST-1347
remove now invalid values from i18n exclude list

Codecov Report

Merging #1961 (750576a) into master (d2520fb) will increase coverage by 0.44%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1961      +/-   ##
==========================================
+ Coverage   71.10%   71.54%   +0.44%
==========================================
  Files         375      375
  Lines        6261     6857     +596
  Branches     1122     1471     +349
==========================================
+ Hits         4452     4906     +454
- Misses       1691     1930     +239
+ Partials      118       21      -97

Impacted Files
Coverage Δ

...c/pages/costModels/createCostModelWizard/table.tsx
34.28% <0.00%> (-10.16%)
:arrow_down:

src/components/i18n/i18nProvider.tsx
57.14% <0.00%> (-9.53%)
:arrow_down:

src/pages/state/maintenance/maintenanceState.tsx
66.66% <0.00%> (-8.34%)
:arrow_down:

.../pages/costModels/createCostModelWizard/markup.tsx
27.27% <0.00%> (-8.03%)
:arrow_down:

...ges/costModels/costModelsDetails/utils/toolbar.tsx
77.50% <0.00%> (-7.80%)
:arrow_down:

...ostModels/createCostModelWizard/priceListTable.tsx
32.07% <0.00%> (-7.46%)
:arrow_down:

.../costModels/costModelsDetails/bottomPagination.tsx
82.60% <0.00%> (-7.40%)
:arrow_down:

src/store/common.ts
81.81% <0.00%> (-7.08%)
:arrow_down:

...ponents/reports/reportSummary/reportSummaryAlt.tsx
52.94% <0.00%> (-7.06%)
:arrow_down:

src/api/reports/reportUtils.ts
27.50% <0.00%> (-6.88%)
:arrow_down:

... and 123 more

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 d2520fb...750576a. Read the comment docs.

Thx

On Apr 28, 2021, at 11:32 AM, boaz0 @.***> wrote:
@boaz0 approved this pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/project-koku/koku-ui/pull/1961#pullrequestreview-647218909, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANWXEALMD6ILDXPVP3XBLUTTLAS7NANCNFSM43XKECZA.


Adjust sidebar design according to specs

Closes E-166

Cards height are a bit too big, they make the screen scroll when not needed
Some project images have the wrong size in the sidebar
The sidebar width changes when opening it, is it possible to fix it ?


tf.print makes a variable a constant?

In [1]: import tensorflow as tf

In [2]: # using print

In [3]: entcoeff =  tf.Variable([0], dtype=tf.float32, trainable=False)
   ...: entcoeff = tf.Print(entcoeff,[entcoeff,"printing"])

In [4]: tf.assign(entcoeff, [-1.])
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-4-dd57efca5923> in <module>()
----> 1 tf.assign(entcoeff, [-1.])

/nohome/jaan/abhishek/anaconda3/lib/python3.6/site-packages/tensorflow/python/ops/state_ops.py in assign(ref, value, validate_shape, use_locking, name)
    270         ref, value, use_locking=use_locking, name=name,
    271         validate_shape=validate_shape)
--> 272   return ref.assign(value)

AttributeError: 'Tensor' object has no attribute 'assign'

In [5]: # not using print

In [6]: entcoeff =  tf.Variable([0], dtype=tf.float32, trainable=False)

In [7]: tf.assign(entcoeff, [-1.])
Out[7]: <tf.Tensor 'Assign:0' shape=(1,) dtype=float32_ref>

Could you give a new name for return value of tf.Print?

@facaiy not sure what you mean.

The output of a operation is immutable Tensor by default, including tf.Print. What is the purpose of assigning value to an immutable Tensor?

You're renaming the Python object entcoeff from a tf.Variable to the return value of tf.Print before trying tf.assign, which is why it crashes.
I think @facaiy wanted something like this which should work:
x =  tf.Variable([0], dtype=tf.float32, trainable=False)
entcoeff = tf.Print(x, [x, "printing"])
tf.assign(x, [-1.])

Going by the documentation of tf.Print it seems like your example should work though. A little weird that it turns Variable into Tensor considering:

Returns:
Same tensor as input_.

Hi, I have created a PR to fix the issue. If accepted, tf.Prints will convert Variable to mutable Tensor.

Hi, @abhigenie92 . As @alextp mentioned in #14878 , forwarding mutable references to variables is weird and discouraged, hence the behavior, returning a constant tensor, might be intent. Wish @alextp could clarify it, thanks.
By the way, I find the behavior is similar with tf.identity, see:
In [19]: v = tf.Variable([1])
In [20]: v
Out[20]: <tf.Variable 'Variable_2:0' shape=(1,) dtype=int32_ref>

In [21]: v3 = tf.identity(v)
In [22]: v3
Out[22]: <tf.Tensor 'Identity_1:0' shape=(1,) dtype=int32>

tf.identity has a clearer docstring.

Returns:
A Tensor. Has the same type as input

Yes, print should have the same behavior as identity (and it does). I'll happily accept a PR to fix the docstring.
I can see how the docstring is confusing if you think a variable is a tensor (it's not). A variable is a thing which can be converted to a tensor containing its current value. It also can be assigned to, partitioned, etc, which tensors can't.

At first I think a variable is not a tensor until I see the official document:
Tensors - Programmers' Guide

Some types of tensors are special, and these will be covered in other units of the Programmer's guide. The main ones are:

 tf.Variable
tf.Constant
 tf.Placeholder
 tf.SparseTensor

If I understand it correctly, there are only one kind of Tensor in C++ side: it can be mutable or immutable which depends on dtype. However, it is not that simple in Python side:

tf.Constant return immutable tf.Tensor.
tf.Variable doesn't return tf.Tensor, however they are treated as same class in most cases, say tf.add. I think tf.Variable is a wrapper for mutable tf.Tensor, right?

It seems that doc of tf.Print is accurate as tf.identity, what confuse us is how to regard tf.Variable: is it a tensor or not?

tf.Print:

Returns:
Same tensor as input_.

tf.identity:

Returns:
A Tensor. Has the same type as input.

tf.Variable s are not tensors, and neither are SparseTensors
On Fri, Dec 1, 2017 at 4:21 PM, Yan Facai (颜发才<EMAIL_ADDRESS>wrote:

At first I think a variable is not a tensor until I see the official
document:
Tensors - Programmers' Guide
https://www.tensorflow.org/programmers_guide/tensors
Some types of tensors are special, and these will be covered in other
units of the Programmer's guide. The main ones are:
tf.Variable
tf.Constant
tf.Placeholder
tf.SparseTensor
If I understand it correctly, there are only one kind of Tensor in C++
side: it can be mutable or immutable which depends on dtype. However, it
is not that simple in Python side:

tf.Constant return immutable tf.Tensor.
tf.Variable doesn't return tf.Tensor, however they are treated as
same class in most cases, say tf.add. I think tf.Variable is a wrapper
for mutable tf.Tensor, right?

It seems that doc of tf.Print is accurate as tf.identity, what confuse us
is how to regard tf.Variable: is it a tensor or not?
tf.Print:
Returns:
Same tensor as input_.
tf.identity:
Returns:
A Tensor. Has the same type as input.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/14788#issuecomment-348649139,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAATxUhn3SLxpKB9OQzUil42J-O4TTwrks5s8JgmgaJpZM4QnEuP
.

--

Alex

Thanks for your explanation, @alextp.
I create two dissimilar PRs for the issue:

#15068 changed tf.Print's behavior, which returns Variable for Variable. If so, the behavior is not consistent with tf.identity.
#15069 is the opposite. It fixes the docstring and states tf.Print behaves like tf.identity.

Be free to approve either one, and close the other.

Fixed by #15069


feat: configure Cypress

feat: configure Cypress
feat: add ci flow

Checklist:

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)

Closes #XXXXX

opening new pr soon


Feature: recipe details page

In this milestone:

Create recipe details page
Add ingredient method
Add information about recipe
Add recipe foods
Add methods modify and remove for recipe foods
Add public button functionality

Author:
@Luiscarlosvd

Good job! is ready for merge.


How to Guide for yBribe

How to Guide for users on how to use yBribe
Assignee: Farrah
I will be working on copy and PR

this was done and merged already! probably we forgot to close here
https://docs.yearn.finance/getting-started/products/ybribe/guide


Support deploying to Deta Space

Hi
I wonder if there's a way to deploy to Deta Space
If there isn't, would it be possible for you to allow it and writing docs on how to do it?
Thanks in advance!

Hey @jahirfiquitiva!
It doesn't seem like Deta Space supports Docker containers

@DarthSim I wonder if doing something like this is possible
https://github.com/deta/starters/blob/main/go-app/Spacefile
Although imgproxy probably is so much more complex 😬

imgproxy depends on tons of C dependencies, so it's not that simple

@DarthSim understood... I hope it becomes possible some day 😅🙌


General: Make exercise editor read only and hide submit button

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines and ensured that the layout is responsive.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context
When tutors/instructors/admins access the route of an exercise (to which they have access), they would still see the Submit button enabled and the text-editor/modeling-editor wouldn't be read-only, as well as the Upload file button. This wasn't intented since the only the actual user who is participating in the exercise can edit/submit the participation.
Description
Makes the editors read-only and hides the Submit button for text, modeling and file-upload exercises.
Steps for Testing

Prerequisites:

1 Instructor/Tutor/Admin
1 Student

As student:

Log in to Artemis as as student
Do steps 3 and 4 for Text, Modeling and File Upload exercises
Go to the exercise participation route
Verify that the editors can be used (aren't read-only) and that the Submit button is visible

As Instructor/Tutor/Admin:

Log in to Artemis as an instructor/tutor/admin
Do steps 3 and 4 for Text, Modeling and File Upload exercises
Go to the exercise participation route
Verify that the editors are read only and that the Submit button isn't visible

Review Progress
Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[x] Test 1
[ ] Test 2

Test Coverage

Class/File
Branch
Line

file-upload-submission.component.ts
74%
86%

modeling-submission.component.ts
77%
85%

Screenshots
Modeling
Before

After

Text
Before

After

File upload
Before

After

Tested on TS2. Works as expected.


Add required jar to the sdk import to support IntelliJ 2019.2

https://github.com/bazelbuild/intellij/issues/731
Adds the jar dependency in order to compile intellij-2019.2 and intellij-ue-2019.2

There are already separate BUILD files in this directory for 2019.2 with essentially this change (and the WORKSPACE file points 2019.2 to those build files).

When trying to run the latest bazel plugin with IntelliJ 2019.2.4 i am still getting ongoing crashes, only resolved by doing a custom build of the plugin inclusive of this change https://github.com/bazelbuild/intellij/pull/1082/files
I am running IntelliJ 2019.2.4 on the latest OSX version


Incorrect img link of IDE in CONTRIBUTING.md

Hi developers,
Appreciate your efforts and cool projects.
BTW, The img link of IDE in CONTRIBUTING.md is incorrect as follows.

I will submit a PR to fix it. If ok, merge it to fix it.
Thank you.

@nathanielsimard
Thank you for merging, I close this issue.


Idea: allSatisfy for Maps

Provide MapAsserts.allSatisfy()
I believe currently this is missing for assertions like:
Employee emperor = new Employee(2L, new Name("Sheev", "Palpatine"), 26);
Map<String, Company> customerMap = Maps.newHashMap("imperium",
            new Company(Arrays.asList(emperor), "imperium", null));

    assertThat(customerMap)
            .allSatisfy((name, customer) -> {
              assertThat(name).isNotBlank();
              assertThat(customer)
                      .fieldSatisfies("employees", (List<Employee> employees) ->
                              assertThat(employees)
                                      .allSatisfy(e -> assertThat(e.getAge()).isBetween(20, 30)));
            });

IterableAsserts already has something similar.
The alternative using the entry set has the slight disadvantage of not showing the types / semantics of the map entries:
assertThat(customerMap.entrySet())
            .allSatisfy(entry -> {
              assertThat(entry.getKey()).isNotBlank();
              assertThat(entry.getValue())
                      .fieldSatisfies("employees", (List<Employee> employees) ->
                              assertThat(employees)
                                      .allSatisfy(e -> assertThat(e.getAge()).isBetween(20, 30)));
            });

I mean map.allSatisfy((name, customer) -> { assertThat(customer) is easier to read than map.entrySet().allSatisfy(entry -> {assertThat(entry.getValue())

I am definitely in favor of the first option where a pair of parameters is passed.

I'm not sure to get the fieldSatisfies part though.

sorry, the fieldSatisfies was just a personal experiment. It did not bring sufficient benefit though. I forgot it was in the examples. Removing it.


Allow to strip search params via config

Describe the feature or enhancement
It would be nice if we could strip search parameters via a config. This may improve privacy and fixes #2260 for me.
Example:
<script
  defer
  src="http://mywebsite.com/umami.js"
  data-website-id="94db1cb1-74f4-4a40-ad6c-962362670409"
  data-strip-params="true"
/>

We've implemented a different fix in v2.10.0 so that long params won't throw an error anymore.

What do you think about this as a configuration to improve privacy when needed? Search parameters often contain user-specific data.


Periodic Scraper

Description: Finalize periodic scraper for continuous collection of new recipes
Requirements:

Package scraper code
Set up lambda for periodic runs

Time Estimation: 4 hours

Running in prod


Typo in migration guide

Coverage decreased (-0.007%) to 97.126% when pulling 2b439c6df4a42d8f2f6b78e4c3172057764e9a01 on hroland:patch-1 into cd0b4597134d7507d777e37c343179bcb7ba5eea on Vincit:master.

Thanks!


Contradiction in documentation for matrix("space", float)

The call matrix("object", 1.0) is documented in two places.
Let's assume object space is defined as a translation of the origin to the point (10, 0, 0), and that the common space is world space. So the point (0, 0, 0) in object space corresponds to the point (10, 0, 0) in common space.
There are two sections documenting this call, and they are contradicting each other:
§5.5 Matrix

// Construct matrices relative to something other than "common"
matrix q = matrix ("shader", 1);

The first form creates the matrix that transforms points from "common" space to "shader"
space. Transforming points by this matrix is identical to calling transform("shader",...).

That implies matrix("object", 1) returns the common-to-object space matrix. This is the inverse of the object transform matrix, i.e. a translation by (-10, 0, 0).
§7.4 Matrix functions

matrix matrix (string fromspace, float f)
Constructs a matrix relative to the named space, multiplying it by the space-to-common
transformation matrix. If the coordinate system name is unknown, it will be assumed to
be the identity matrix.
Note that matrix (space, 1) returns the space-to-common transformation matrix. If
the coordinate system name is unknown, it will be assumed to be the identity matrix.

Which implies the returned matrix will be the object transform matrix, i.e. a translation of (+10, 0, 0).
One of these sections is wrong and should be fixed.

This was fixed a while back, thanks for the report.


[SUGGEST] Use one code-style about {} in js code.

There are 2 styles in pixi-filters
xxxxx {

}

and
xxxxx
{

}

Whatever use which one , just use one,  please.
BTW,  I could accept the shader's code style is different from js's .

Fixed in the filters/all PR


Landscape orientation Support for iPad

Describe the feature you'd like
iPad Support for the unc0ver app.
Landscape support for the app it’s anoying rotating the iPad every time you need to jailbreak

It's already in version 5.0.1

It's already in version 5.0.1

Doesn't work for me on iPad Air 3

What ist not working for you? Unc0ver or Landscape mode?

What ist not working for you? Unc0ver or Landscape mode?

Landscape mode that’s why I submitted this under enhancement

Closing this issue since unc0ver 5.2 fixes this


Update to Nx 14

Current Behavior
Uses Nx 13
Expected Behavior
Uses Nx 14
Related Issue(s)
The failing vite test fails at current master as well ^^'
Fixes #241

Thanks for the contribution @DominikPieper, I'll give this a review this week. I'll need to release one more v13 version and then we can move forward to 14.


[BUG] parse_json function doesn't work with an array of objects

This works
let str = r#"{
    "test": ["str"]
}"#;
let map = engine.parse_json(str, true);

This doesnt work
let str = r#"{
    "test": [{ "str": "str" }]
}"#;
let map = engine.parse_json(str, true);

Error
Err(
    ErrorParsing(
        BadInput(
            "Unexpected \'{\'",
        ),
        2:34,
    ),
)

I looked a bit at the source code and it looks like you trim the string and then prepend a #. This assumes that nested objects start with a # in JSON, which is not correct. I assume that this is a problem with nested objects overall and not only an array of objects.

Hhhmmm... you're right.  This is a bug.  Yes, you need to append # in front of any object hash inside the JSON.
For now, you can use serde to deserialize JSON into an object map.  I am not sure this will be easy to fix...
Maybe I need to write a note in the documentation saying: Warning nested objects.

Alternatively, if you have no { in strings, you can replace all { with #{, except for the first one (the # will be added).  That should cause it to parse correctly...

The latest version in master will have support for a leading #{ for Engine::parse_json.  This is convenient if you'd just replace all { with #{.
Closing this for now.


Minecraft coordinates are real numbers- not intergers.

Minecrafts coordinates are stored as 'real' numbers (as defined by the IEEE 754-2008) - not intergers so your claim "This MinecraftForge mod extends Minecraft height and depth. The only limitation is size of 32-bit integer." is wrong the actural limitation is the maximium float value of -Inifinty to +Infinity (or 1.175494351 E - 38 to 3.402823466 E + 38)

You are right but also wrong.
Minecraft entity coordinates are floating point numbers. Specifically, double precision floating point numbers.
Rendering is done with single-precision floating point numbers (with clever tricks to keep the numbers involved reasonably small at huge coordinates in most cases).
Block and Tile Entity positions are 32-bit integers. Chunk positions are also 32-bit integers. Since 32-bit integers have smaller value range than double precision floating point numbers, they are the limitation.


Add checkbox support to markdown

Fixes #2217 and #3458. Figured I'd see if I could do it and it seems to work. Probably still need to add specs.

Thanks for the PR! I personally think it's a good addition because it's quite useful, a common implementation uses it, and the character sequence is pretty unambiguous. Does this PR limit checkboxes to the beginning of list items though? I'm pretty sure github does that and it might be a good idea.

This was previously closed as not being Markdown, but a GitHub extension. See #2217.

I'm actually in favor of providing a complete markdown implementation in the standard library, together with extensions (that can be disabled). The current markdown parser is a bit ugly/hacky, I don't know if there's a nice/formal algorithm to parse markdown in its entirety. If someone wants to provide a complete implementation, go ahead :-)

I would say that extensions must be disabled by default —they're unexpected.

@asterite I like that idea a lot. Maybe I'll look into this and see if I can come up with something

Here is a WIP implementation of CommonMark spec and markdown-it extensions. It's written on Crystal but it is a carbon-copy of markdown-it JavaScript-flavored algorithms. I don't think the code size and quality is suitable for Crystal standard lib but it's worth mentioning. There are two reference CommonMark implementations but they are not extensible.
I'm putting it here mostly for you to realize how big CommonMark spec is.

I think we'll eventually use markd and remove the poor Markdown support we have now, so closing.


Assess hyperlinked notes in archives data for future pain-points

What is it and who's it for?
Collections researchers
Us
The archive collections contain many different types of links to material across Wellcome Collection/Library and beyond. Some of these will become dead or redundant once we shut down Encore and wellcomelibrary.org. Others we may just need to verify that redirection will/is happening.
This issue collects the examples:
API/Search
Donor searching
https://search.wellcomelibrary.org/iii/encore/search/C__S"Donor: Medact"__Orightresult__U?lang=eng&suite=cobalt
"For books, reports and other printed materials from the collection of Medact, please see the main Library catalogue." - Linked to a list of all non-archival materials donated with Medact material.
https://wellcomecollection.org/works/qq7nc5j9
http://catalogue.wellcomelibrary.org/search~S12/?searchtype=Y&searcharg="Donor%3A+FPA%2C+formerly+the+Family+Planning+Association"&searchscope=12&SORT=DZ&extended=0&SUBMIT=Search&searchlimits=&searchorigarg=Ydonor+fpa%26SORT%3DDZ
https://wellcomecollection.org/works/chm5t55w
General updates to pass on to CI

Outdated link to NLM for digitised material - Crick collection https://wellcomecollection.org/works/hz43r7re
References to Codebreakers on Wellcome Library, example from Franklin collection https://wellcomecollection.org/works/hyjxver8

Donor searching
JT confirmed 591 field, where donor info is (along with other info as it's a general note?) is in the API. Donor info is found in search but comes at the bottom of results as it's not boosted as much as other fields.

@pollecuttn @jtweed As a general note it isn't structured, but it looks like 591 |r is used specifically for Donors as in this MARC record:
591 00 |zCopy 1.|rDonor: FPA, formerly the Family Planning         Association.
By structuring the data we'd be able to bring together all of the materials from that donor which has benefit when the materials aren't strictly Archive collections. Granted, it's probably of more use to staff - and it may be that not all donor information is treated equally from a public-facing point of view.

Outdated link to NLM for digitised material - Crick collection
This has been updated to https://profiles.nlm.nih.gov/spotlight/sc


Factorio crash and mapshot does not exit

Factorio version 1.1.110, No mods
Mapshot version 0.0.22
Running in a ubuntu:22.04 container
Factorio crashes and mapshot hangs.
31.281 Script @mapshot/control.lua:132: Mapshot: all screenshots started, might take a while to render; location: mapshot/mysave/d-6f6cf85b/
253.261 Script @mapshot/control.lua:356: marking as done @11357086
I0922 02:19:38.675826     731 factorio.go:139] interrupt requested
254.125 Received SIGINT, shutting down
I0922 02:19:38.675593     731 render.go:199] done file "/root/factorio/script-output/mapshot-done-d577810c-fdf3-47b8-904d-3534d5b0df4f" now exists
I0922 02:19:38.676602     731 render.go:205] output at mapshot/mysave/d-6f6cf85b/
Output: /root/factorio/script-output/mapshot/mysave/d-6f6cf85b
I0922 02:19:38.677073     731 render.go:210] removed done-file "/root/factorio/script-output/mapshot-done-d577810c-fdf3-47b8-904d-3534d5b0df4f":
254.180 Quitting: signal.
254.529 Error CrashHandler.cpp:639: Received SIGSEGV
Factorio crashed. Generating symbolized stacktrace, please wait ...
/tmp/factorio-build-GWejqv/src/Util/Logger.cpp (336): Logger::writeStacktrace(FileWriteStream*, StackTraceInfo*) [clone .constprop.0]
/tmp/factorio-build-GWejqv/src/Util/CrashHandler.cpp (188): CrashHandler::writeStackTrace(CrashHandler::CrashReason)
/tmp/factorio-build-GWejqv/src/Util/CrashHandler.cpp (642): CrashHandler::commonSignalHandler(int)
/tmp/factorio-build-GWejqv/src/Util/CrashHandler.cpp (648): CrashHandler::SignalHandler(int)
0x7fdac1cee51f
/tmp/tmp.AizlpDmMs1/objdir/x86_64-pc-linux-gnu/libstdc++-v3/include/bits/shared_ptr_base.h (337): std::_Sp_counted_ptr<decltype(nullptr), (__gnu_cxx::_Lock_policy)2>::_M_dispose()
Stack trace logging done
254.868 Error Util.cpp:100: Unexpected error occurred. If you're running the latest version of the game you can help us solve the problem by posting the contents of the log file on the Factorio forums.
Please also include the save file(s), any mods you may be using, and any steps you know of to reproduce the crash.

Sorry, I won't be able to follow up for a while here. That being said, would you mind sharing the exact command line you've used?
Fwiw - the shutdown of Factorio from Mapshot binary never worked well, as it is tricky to get right. I suspect in container it makes it behaves also slightly differently. From what I see in the log above, that should not prevent Mapshot from working (the render has been done), it is just shutting down not so cleanly.

Thanks yes it renders. Command was xvfb-run ./mapshot-linux render mysave --tilemin 16 --logtostderr --factorio_verbose
I'll wrap it in a timeout for now.


A couple of things

Hello! Thanks for making this project, it seems to be really interesting.
I have a couple of tips for you:

For Windows you also need to ship two SSL dlls on 64bit, namely ssleay64.dll and libeay64.dll, you can get those from https://nim-lang.org/download/dlls.zip
Specifying -d:danger in https://github.com/its5Q/ducominer/blob/main/ducominer.nim.cfg does not actually apply it (because user config is evaluated after the global one, so -d:danger does not actually change anything)

You need to pass -d:release and/or -d:danger explicitly in the command line (or in Nimble tasks) for it to apply.

I also have some minor code improvements which I will probably do in a PR :)

I also have some minor code improvements which I will probably do in a PR :)

Also, do you happen to know why with ducominer my DuinoCoin wallet balance doesn't seem to change?

Also, do you happen to know why with ducominer my DuinoCoin wallet balance doesn't seem to change?

Hmm, it seems to work if I authenticate with duino coin's protocol, but just "JOB,username" doesn't seem to add balance to my own account :thinking:

Hmm, it seems to work if I authenticate with duino coin's protocol, but just "JOB,username" doesn't seem to add balance to my own account :thinking:

Hmm, that

Hmm, it seems to work if I authenticate with duino coin's protocol, but just "JOB,username" doesn't seem to add balance to my own account 🤔

Hmm, strange, it works perfectly fine with JOB,username for me.

Hmm, that

Hmm, it seems to work if I authenticate with duino coin's protocol, but just "JOB,username" doesn't seem to add balance to my own account 🤔

Hmm, strange, it works perfectly fine with JOB,username for me.

Also, thanks for the tips, I will make a release with the libssl included.

Also, thanks for the tips, I will make a release with the libssl included.

Hmm, it seems to work if I authenticate with duino coin's protocol, but just "JOB,username" doesn't seem to add balance to my own account 🤔

I've fixed the issue, it was in my donation system. Turns out that the server accepts all shares under the first username that was sent to the server. And because it determines when the share submits as my username, it uses modulo, so the first share was always sent under my username, and so would the rest of the shares.

Hmm, it seems to work if I authenticate with duino coin's protocol, but just "JOB,username" doesn't seem to add balance to my own account 🤔

I've fixed the issue, it was in my donation system. Turns out that the server accepts all shares under the first username that was sent to the server. And because it determines when the share submits as my username, it uses modulo, so the first share was always sent under my username, and so would the rest of the shares.


typeahead: Prevent blue styling from mouse hover on typeahead item in dark mode.

reported here https://chat.zulip.org/#narrow/stream/9-issues/topic/unexpected.20search.20typeahead.20highlighting.20.28dark.20theme.29/near/1951962

Does this also fix the blue hover in the dark theme in the send later dropdown in #31359?

Does this also fix the blue hover in the dark theme in the send later dropdown in #31359?

No, because I only changed it for the search typeahead. We probably can make it for all typeaheads? I just didn't feel as confident thinking about the consequences of that, but it's probably what we want.

Merged, thanks @evykassirer!


Learn Form Validation by Building a Calorie Counter - Step 69

Describe the Issue
hi!
document.querySelectorAll("#breakfast input[type='number']");
the above line of code is correct. the challenge step should be corrected. because it is not passing with inner qoute marks around type attribute value 'number'.
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-form-validation-by-building-a-calorie-counter/step-69
Your code

function calculateCalories(e) {
  e.preventDefault();
  isError = false;
  const breakfastNumberInputs = document.querySelectorAll("#breakfast input[type='number']");

}

Expected behavior
the challenge step should be passed with quote marks around the type attribute value number .
Screenshots
No response
System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
No response

Thank you for opening this Issue, it seems to be a duplicate of #56081, as such I will be closing this in favor of the other one


Replace NumPy prod() with infinite width integer prod()

Avoids integer overflows in size calculations
Closes #846, see there for more detail
Contributor Checklist:

[x] I have added or updated my entry in the creators.json file
[ ] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes
[N/A] I have added/updated test cases
[N/A] I have included the rebuilt production build of the client (only if changes were made to the GUI)

Reviewer Checklist:

[ ] /azp run libertem.libertem-data passed

/azp run libertem.libertem-data

Not sure if or how to document this. It should not have any user-facing effects, but it is relevant for contributors.

/azp run libertem.libertem-data

/azp run libertem.libertem-data

FYI: get_args needs to be imported from typing_extensions for compatibility with Py<3.8

/azp run libertem.libertem-data

Ok, so get_args needs PEP 560 support, which was added in Python 3.7, so we are out of luck there... hm.

Ok, so get_args needs PEP 560 support, which was added in Python 3.7, so we are out of luck there... hm.

Ok, so no detailed typing. I already tried to construct a Union from a tuple, but that doesn't seem to work.

Ok, so no detailed typing.

How about copy & paste for now, and we fix it to get_args once we drop Py 3.6?

/azp run libertem.libertem-data


Added Load more button and fetch pagination

Defaults to pulling in 21 projects
Added Load more button to bottom of project results
Clicking Load more fetches the next 21 projects
Load button disappears when on the last set of projects
Added isLoading logic
Submitting empty query in search bar resets and pulls in default projects again

The below demo was recorded with the "pageSize" set to 13 for demo purposes but this PR has it set to 21:
https://github.com/user-attachments/assets/0caad88f-85d2-4e03-be3d-6d092a02a109

Awesome PR! Thank you for the contribution


I cant import Starscream on tvos

I'm usim cocoa pods with podfile:
platform :tvos, '9.0'
target 'Websocket' do
  use_frameworks!
  pod 'Starscream'
end

and when I try to import Starscream or StarscreamTv xcode fails to find it.

Ops. it turned out that restarting xcode was the solution. =)


(maint) don't activate the node when it's already active

Before this commit we were updating the deactivated column of certnames to null
even if it was already null. This caused unnecessary contention on the certnames
table with each command submission, causing more frequent retries than needed.
Also corrects the corresponding test logic: if nothing happens, then no records
are updated, and the result of maybe-activate-node should not be positive.

this only affects things when running with command-processing-threads > 1. The easiest way to verify is to make the corresponding change in master and observe your retried commands. Postgres logs will also no longer show this kind of thing (example from master):
   9955 2015-05-27 13:05:56.746 PDT [db:puppetdb,sess:5566239e.bab3,pid:47795,vtid:9/3179,tid:7817153] ERROR:  could not serialize access due to concurrent update
   9956 2015-05-27 13:05:56.746 PDT [db:puppetdb,sess:5566239e.bab3,pid:47795,vtid:9/3179,tid:7817153] STATEMENT:  UPDATE certnames SET deactivated=$1, expired=$2 WHERE certname=$3 AND (deactivated<$4 OR deactivated IS NULL) AND (expired<$5 OR expired IS NULL)

(i.e that specific statement)

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1316/

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1319/

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1323/

:+1:

I'm :+1: on this in master, but we decided to skip it in stable.

I'll stick it on master if we still want it there. My assumption based on the conversation this morning is that this is going into stable.

Yeah sounded like stable is good from this morning

+1 will merge once test disk issues have been resolved


ripgrep#observe#add_observer() ignores two or more observers

ripgrep#observe#add_observer() doesn't add an observer if the first observer for the same event has already been added.
Reproduction steps are as below in Vim 9.1(patches: 1-163).
function! Observer1(_) abort
  call setqflist([{ 'text': 'Observer1' }], 'a')
endfunction

function! Observer2(_) abort
  call setqflist([{ 'text': 'Observer2' }], 'a')
endfunction

call ripgrep#observe#add_observer('match', 'Observer1')
call ripgrep#observe#add_observer('match', 'Observer2')

call ripgrep#search('ripgrep')
copen
" Only "Observer1" are listed in the quickfix window.

@a5ob7r It was a bug. I fixed it so could you try it?

I confirmed it works well. Thank you.


Good defaults

Default permissions configuration (users + groups) might want to be established? Is a default configuration even necessary? The previous one already didn't do much. And defaults are handled differently now -- there's a special default subject data

Ok, default configuration is to give players connecting from localhost all permissions. This should be considered complete.