Unresolved compilation - Lambda expression for Java in unit tests

Hello,
I tried to follow up the project build instruction but the project is being detected as Java 7 by m2e. I'm getting compilation errors from Eclipse and mvnw regarding lambda expressions from the unit tests. I am actually surprised to see some Unit Test making use of lambda expression. Did I miss something on the project set up ?
Many thanks.
Regards,
Daniel

You have to turn on in your IDE the Ide profile. It will turn on java8. Mind you that we use java8 only for tests

thanks @marcingrzejszczak . Went too fast over the doc. Fixed my issue ;)

No problem :)


Allow retrieving a batch of tasks together via a batch ARN

Describe the feature you'd like
I would like a batch ID of some kind, analogous to a task ARN, to be assigned to a batch created by device.run_batch, so that I can use a single string to recall the data from a batch at a later time.
How would this feature be used? Please describe.
Let's say I've defined a rabi_sequence with a FreeParameter for length, and use it to make a list of pulse sequences, then execute them as a batch, as is done in the Bring Up example notebook. If I want to pull up the results again later, after my kernel has closed, I would like after submitting the batch to run:
batch_arn = batch.metadata()["quantumBatchArn"]

or maybe just something like
batch_arn = batch.arn

to retrieve a single ID string. I would then save that string and later use it to retrieve the batch:
from braket.aws import AwsQuantumTaskBatch
batch = AwsQuantumTaskBatch(batch_id=batch_arn)

Describe alternatives you've considered
Currently I have an upload notebook and a data notebook. Once I run something in the upload notebook, I copy-paste the relevant tasks into the data notebook. This allows me to keep track of the strings I will need to retrieve the batch data later, and keep them together, but a shared ID for the batch that allowed it to be recalled as a single object would be easier to keep track of and use than long lists of task ARN strings.

Hello,
You might be able to leverage resource tags in this case. When you are creating the tasks using device.run_batch(), you could pass in a tag value:
batch = device.run_batch(circuits, shots=100, tags={"my_batch_1": "foo"})

You can later use this tag value to retrieve all the tasks which were tagged with this specific tag value:
import boto3

client = boto3.client('resourcegroupstaggingapi')
response = client.get_resources(
    TagFilters=[
        {
            'Key': 'my_batch_1'
        },
    ],
)
tasks = [t["ResourceARN"] for t in response["ResourceTagMappingList"]]

Please let us know if this solves the problem @lillian542. Thanks!

@kshitijc Sorry for the delayed response - I was away for a couple of weeks. Thanks for the suggestion, I'll try it out and let you know once the tasks have run!

Unfortunately this didn't work; I'm not sure where I went wrong. I created the tasks like this:
import pennylane as qml
from braket.pulse import ConstantWaveform, PulseSequence
from braket.parametric import FreeParameter
import numpy as np

# create device
lucy = qml.device(
    "braket.aws.qubit",
    device_arn="arn:aws:braket:eu-west-2::device/qpu/oqc/Lucy",
    wires=8,
    max_parallel=50,
)

# define pulse sequence
def rabi_pulse(wire, amp):
    """creates a constant drive with amplitude 0.07 (value from AWS example notebook) and
    drive frequency on-resonance for the given wire"""

    frame_name = f"q{wire}_drive"
    readout_name = f"r{wire}_measure"
    phase = 0

    waveform = ConstantWaveform(FreeParameter("length"), amp)
    device = lucy._device

    drive_frame = device.frames[frame_name]
    readout_frame = device.frames[readout_name]

    rabi_sequence = (
        PulseSequence()
        .set_frequency(drive_frame, drive_frame.frequency)
        .play(drive_frame, waveform)
        .capture_v0(readout_frame)
    )

    return rabi_sequence

# create batch of pulse sequences
wire=7
start_length=10e-9  # minimum pulse duration is 8ns
end_length=80e-9
lengths = np.arange(start_length, end_length, 2e-9)

rabi_sequence = rabi_pulse(wire, amp=0.03)
pulse_sequences = [rabi_sequence(length=length) for length in lengths]

# run with tags
lucy._device.run_batch(pulse_sequences,
                       shots=1000,
                       tags={"rabi_on_q7": "foo", "amplitude": "0.03", "User": "Lillian"},
                       max_parallel=len(lengths))

I can retrieve them individually by their task ARN numbers, but when I try to return using tags like this:
import boto3

client = boto3.client('resourcegroupstaggingapi')
response = client.get_resources(
    TagFilters=[
        {
            'Key': 'rabi_on_q7'
        },
    ],
)
tasks = [t["ResourceARN"] for t in response["ResourceTagMappingList"]]

I get an empty list. Any pointers on where I went wrong?

I do see the tags on the task in the console

Thanks @kshitijc , that did work.

That's great! Closing the issue. Please feel free to reopen if you have any other questions. Thanks!


Double value rounds to integer, which should be double.

Problem
Double value rounds to integer, which should be double. Issue found after #90 fixed after #91
Demo
./AParser
>>> 11 + 1.3333
12
>>> 11.3333 + 1
12

Issue solve in #93.


Bjorncs/more junit5

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

🎖


ScatterplotVisOp does not work with unmatched schema

Similar to #1191, this is related to change #1146 where the tuple needs a matched schema to build. Currently, the front end does not render points. I changed the schema attribute names and it can fix the issue temporarily. We need a more robust fix (related to #1166) to use the same schema (reference equal) in Desc and Exec.

Assigning to myself as I've fixed it in #1166


Circuit Breaking doesn't work with TLS Origination

Bug description
I followed Circuit Breakin instructions and adapted it to our system but the circuit breaker configuration is ignored when using TLS Origination
Expected behavior
Circuit Breacing config works with TLS Origination same as without (HTTP standard)
Steps to reproduce the bug
Set outbound traffic policy mode to REGISTRY_ONLY

check if an outgoing call is now forbidden for not configured hosts (out of pod)
``curl -I http://httpstat.us/200?sleep=100` -> HTTP/1.1 502 Bad Gateway

Allow traffic for httpstat.us, create VirtualService with timeout and DestinationRule with Circuit Breaking config. Working example without TLS Origination

kubectl apply -f - <<EOF
---
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
    name: se-httpstat
spec:
    hosts:
        - httpstat.us
    ports:
        - number: 80
          name: http
          protocol: HTTP
    resolution: DNS
    location: MESH_EXTERNAL
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
    name: vs-httpstat
spec:
    hosts:
        - httpstat.us
    http:
        - name: "httpstat-route"
          route:
              - destination:
                    host: httpstat.us
                    subset: httpstat-origination
          timeout: 0.8s
---
apiVersion: networking.istio.io/v1alpha3
kind: DestinationRule
metadata:
  name: dr-httpstat
spec:
  host: httpstat.us
  subsets:
    - name: httpstat-origination
  trafficPolicy:
    outlierDetection:
      consecutiveErrors: 1
      interval: 1s
      baseEjectionTime: 30s
      maxEjectionPercent: 100
EOF

Check if an outgoing call is now allowed and works within timeout boundary (out of pod)
curl -v http://httpstat.us/200?sleep=100 -> HTTP/1.1 200 OK

Check timeout and Circuit Breaking works
curl -I http://httpstat.us/200?sleep=1000
First trial ends with Timeout from envoy

HTTP/1.1 504 Gateway Timeout
content-length: 24
content-type: text/plain
...
server: envoy

Next trial returns HTTP 503 from envoy due to Circuit Breaking is working
HTTP/1.1 503 Service Unavailable
content-length: 19
content-type: text/plain
...
server: envoy

Adapt these configuration to TLS Origination use. First check outgoing call is forbidden for not configured hosts (out of pod)
curl -I http://www.google.com -> HTTP/1.1 502 Bad Gateway

Allow traffic for www.google.com, create VirtualService with timeout and DestinationRule with Circuit Breaking config and TLS Origination

kubectl apply -f - <<EOF
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: se-google
spec:
  hosts:
  - www.google.com
  ports:
  - number: 443
    name: https-google
    protocol: HTTPS
  resolution: DNS
  location: MESH_EXTERNAL
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: vs-google
spec:
  hosts:
  - www.google.com
  http:
  - match:
    - port: 80
    route:
    - destination:
        host: www.google.com
        port:
          number: 443
    timeout: 0.001s # Even to short for google
---
apiVersion: networking.istio.io/v1alpha3
kind: DestinationRule
metadata:
  name: dr-google
spec:
  host: www.google.com
  trafficPolicy:
    outlierDetection:
      consecutiveErrors: 1
      interval: 1s
      baseEjectionTime: 30s
      maxEjectionPercent: 100
    portLevelSettings:
    - port:
        number: 443
      tls:
        mode: SIMPLE
EOF

Check if an outgoing call is now allowed and works within timeout (Set timeout to 0.5s) boundary (out of pod)
curl -I http://www.google.com -> HTTP/1.1 200 OK

Check Timeout and Circuit Breaking - Set timeout to 0.001s (out of pod)
curl -I http://www.google.com
First trial ends with Timeout from envoy - Timout is working correctly

HTTP/1.1 504 Gateway Timeout
content-length: 24
content-type: text/plain
...
server: envoy

Further trials also return only HTTP 504 instead of the expected HTTP 503
I used www.google.com as an example to hide the real HTTPS endpoint I want to call but the effect is the same.
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
Istio:
client version: 1.2.0
citadel version: 1.3.3
egressgateway version: 1.3.3
galley version: 1.3.3
ingressgateway version: 1.3.3
pilot version: 1.3.3
policy version: 1.3.3
sidecar-injector version: 1.3.3
telemetry version: 1.3.3
Kubernetes
Client Version: version.Info{Major:"1", Minor:"14",
Server Version: version.Info{Major:"1", Minor:"14+"
Helm
Client: &version.Version{SemVer:"v2.14.1"
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)
GCP

Is this the same as https://github.com/istio/istio/issues/19101?

Hello @howardjohn,
both tickets are from me but I think it's quite a difference in it.
I think it could be same code area but the occurrence is a little different

@hubermark sorry not sure why I thought they were the same, I understand now.
I can reproduce this, thanks for the config. I will take a look

I think this is actually caused by two misconfigurations
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: se-google
spec:
  hosts:
  - httpbin.org
  ports:
  - number: 443
    name: https-google
    protocol: HTTPS
  resolution: DNS
  location: MESH_EXTERNAL
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: vs-google
spec:
  hosts:
  - httpbin.org
  http:
  - match:
    - port: 80
    route:
    - destination:
        host: httpbin.org
        port:
          number: 443
---
apiVersion: networking.istio.io/v1alpha3
kind: DestinationRule
metadata:
  name: dr-google
spec:
  host: httpbin.org
  trafficPolicy:
    portLevelSettings:
    - port:
        number: 443
      tls:
        mode: SIMPLE
      outlierDetection:
        consecutiveErrors: 1
        interval: 1s
        baseEjectionTime: 30s
        maxEjectionPercent: 100

Then send curl httpbin.org/status/501 a few times and it will switch to 503.
The problem with httpstat is the subset doesn't actually select the service entry, so its empty, resulting in 503
The problem with the google.com one is the portLevel setting replaces, not merges, with the global settings. So you don't actually apply the outlier detection
This was pretty hard to find though, I am not sure how we can detect situations like this

I think this is user error that is fixed in Istio 1.6 by validating properly. Additional validation is tracked in https://github.com/istio/istio/issues/24860

@howardjohn This was the exact issue with the indentation of portLevelSettings and outlierDetection. Surprisingly, it is not mentioned in anywhere in the docs, so after spending 2 days debugging, stumble upon your comments 😅 and saved the day 😄 . But it would be helpful if it can be added in the docs itself with example. Thanks for this awesome product 😄

I think this is actually caused by two misconfigurations
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: se-google
spec:
  hosts:
  - httpbin.org
  ports:
  - number: 443
    name: https-google
    protocol: HTTPS
  resolution: DNS
  location: MESH_EXTERNAL
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: vs-google
spec:
  hosts:
  - httpbin.org
  http:
  - match:
    - port: 80
    route:
    - destination:
        host: httpbin.org
        port:
          number: 443
---
apiVersion: networking.istio.io/v1alpha3
kind: DestinationRule
metadata:
  name: dr-google
spec:
  host: httpbin.org
  trafficPolicy:
    portLevelSettings:
    - port:
        number: 443
      tls:
        mode: SIMPLE
      outlierDetection:
        consecutiveErrors: 1
        interval: 1s
        baseEjectionTime: 30s
        maxEjectionPercent: 100

Then send curl httpbin.org/status/501 a few times and it will switch to 503.
The problem with httpstat is the subset doesn't actually select the service entry, so its empty, resulting in 503 The problem with the google.com one is the portLevel setting replaces, not merges, with the global settings. So you don't actually apply the outlier detection
This was pretty hard to find though, I am not sure how we can detect situations like this


Cannot show window with non-visible parent

Describe the bug
When calling ShowDialog() on a Window, one cannot hide the window before displaying the new dialog. Culprit:
https://github.com/AvaloniaUI/Avalonia/blob/f9956e5ba96eedfe5e2872c84125e80f5ad14faa/src/Avalonia.Controls/Window.cs#L748-L751
To Reproduce
...
this.Hide();
await new Window().ShowDialog(this);
this.Show();
...

Expected behavior
There should be no reason for the initial window to be mandatorily visible while creating and displaying a new dialog (it is not enforced in WinForms/WPF). Example where this might be useful: hiding a login screen while displaying a modal main window, while awaiting for it to be closed, when the login screen is displayed again, without the need to create a new instance. This also prevents closing the entire application if the login window is the initial window displayed when the application is ran.

Commenting to bump this.
This is not what I would call a rare use case, so I (among many others) would be delighted to see this being worked on.

Hallo
Can anyone tell me how to handle this problem
Thanks


Toasts: .hide CSS can conflict with utilities

Close this toast https://twbs-bootstrap.netlify.app/docs/5.0/components/toasts/#color-schemes
Notice that its display is not set to none

Probably affects v4 too, I haven't checked. We should use !important I guess? - EDIT: might not be so simple, we need to consider the use cases.

/cc @MartijnCuppens for thoughts since I think he last updated toasts for positioning.

There doesn't seem to be a v4 version of this example: (https://getbootstrap.com/docs/4.5/components/toasts/#stacking).
We could just wrap the toast content in an additional <div> with the flex classes?

Well, it seems https://twbs-bootstrap.netlify.app/docs/5.0/components/toasts/#custom-content also has the same issue. its display is not set to none, when we close toast.


feat(popup): Accessing context variables from within popup template

I'm creating a site where a popup uses a template that needs access to data from scope (let's thing of it as a popup assigned for every item in a *ngFor).
How can I do that? Couldn't find it in the docs.

Have you tried defining the <ng-template> inside the *ngFor, like so:

    <ng-template #template>

{{ item.name }}

    </ng-template>
    <button class="ui button" suiPopup [popupTemplate]="template">Action</button>

Just made a plunkr demonstrating this - does that suit your use case?

Ok @edcarroll you've got this one. I was trying to use the template outside the element, using it inside the iteration did the trick, nice catch. Maybe add it to the docs? either way this issue will work as a heads up for anyone looking for a solution.
My exact use case is via sui-select and is written like this
<ng-template let-option #selectTemplate>
	<ng-template let-popup #popupTemplate>

{{ option.field }}

			{{ option.otherField }}

	</ng-template>

			{{ option.field }} ({{ option.id }})

</ng-template>

<sui-select [(ngModel)]="selected" [options]="options" [optionTemplate]="selectTemplate" labelField="field" [isSearchable]="true" #select>
	<sui-select-option *ngFor="let option of select.availableOptions" [value]="option">
	</sui-select-option>
</sui-select>

Glad that's sorted it, and yes good idea, will add a note to the docs

One idea just crash my mind... what if I wanna reuse templates? this workaround will not fix the underlying issue...
What about adding a let-thingie in order to pass variables to the popup? Maybe just like sui-select does with its own options via let-option.
Will result in a code like this
	<ng-template let-var #popupTemplate>

{{ var.field }}

			{{ var.otherField }}

	</ng-template>

			{{ option.field }} ({{ option.id }})

This isn't a bad idea actually... Though I've made a tonne of changes to the popup component in the datepicker, so will leave this for now until I've merged those changes, otherwise it's futile!

Ok then, let's wait for the next release! How can we follow this feature request?
pst: this issue is still with label investigating

@genuinefafa this will be in the next release. You can bind a context by setting popupTemplateContext, and you can access it in the template like so:
<ng-template let-popup #popupTemplate>
    {{ popup.context.myVariable }}
</ng-template>

This is now live in 0.9.5. Let me know if you have any further issues 😄

I see no example using context in docs. Can we add it there too?

I've just tried making it to work, with no luck. Here's the code I have:
...

    {{ person.id }}
    {{ person.name }}

                 Edit

            <button class="ui icon button" suiPopup [popupTemplate]="confirmDeleteTemplate" [popupTemplateContext]="person" popupTrigger="outsideClick" #popup="suiPopup">
                 Delete
            </button>

...

<ng-template let-popup #confirmDeleteTemplate>

Delete person

You are deleting this person, are you sure?

            <button class="ui positive button" (click)="deletePerson(popup.context.id)">Yes</button>
            <button class="ui negative button" (click)="popup.close()">No</button>

</ng-template>

I'm getting an error because context is undefined, so it seems that the context is not correctly set. Any idea to why that would be?
Thanks in advance.

are you using latest version of semantic-ui? can you provide an example
plunkr?
On Fri, Aug 25, 2017 at 5:47 AM, Xavier Hahn<EMAIL_ADDRESS>wrote:

I've just tried making it to work, with no luck. Here's the code I have:
...

{{ person.id }}
{{ person.name }}

 Edit

 Delete

...
<ng-template let-popup #confirmDeleteTemplate>
Delete person

You are deleting this person, are you sure?

<button class="ui positive button" (click)="deletePerson(popup.context.id)">Yes
<button class="ui negative button" (click)="popup.close()">No

I'm getting an error because context is undefined, so it seems that the
context is not correctly set. Any idea to why that would be?
Thanks in advance.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/edcarroll/ng2-semantic-ui/issues/127#issuecomment-324860363,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABeGXeHoiEcIZxQRXtNUqlBwtCHbf7ZCks5sboo-gaJpZM4OEDll
.

@genuinefafa Yes, I'm indeed using the latest version of semantic-ui.
Here's a plunkr that exhibits the issue.
http://plnkr.co/edit/l2T5T4GoriTWcigPNECj?p=preview
Thanks for your help.

@Gimly thanks for the plunkr. Hmmm, looks like a bug. Will reopen for now and assess properly when I get a chance. Sorry for the delay!

@edcarroll any news on this?

Any news on this?

@edcarroll - is this still an outstanding bug? I am seeing similar behavior in the current alpha build

I have checked its still not fixed. I am waiting for it too..

Just in case anyone is stumbling across this again. This is how it works:

  hover here

<ng-template let-context="context" #popupTemplate>
    {{ context.myVariable }}
</ng-template>

Just in case anyone is stumbling across this again. This is how it works:

  hover here

<ng-template let-context="context" #popupTemplate>
    {{ context.myVariable }}
</ng-template>

@janpapenbrock thank you man) you saved my day!

@janpapenbrock thank you man) you saved my day!


Does not use nice color in Jupyter

logzero version: 1.5.0
Python version: 3.6
Operating System: Linux

Description
Logzero is really neat and it would be great if one could make full use of it in Jupyter notebook.
Unfortunately it does not recognize the color capability and the output shows no color formatting.
Two things would solve this:

easy way to switch to STDOUT (without losing the formatter)
enforcing TTY mode since apparently Jupyter notebook/lab can show color but does not announce that

What I Did
Do in Jupyter notebook (conda install notebook; jupyter notebook) or Jupyter lab.
from logzero import logger
logger.error("TEXT")

Any progress on this?   I have the same problem.   When I run a python program in standalone mode, I get lovely color-coded output (different colors for warnings, errors, info, debugging).   But inside Jupyter, everything looks the same.
If someone can offer suggestions as to where to start, I'm happy to attempt to implement this, and submit patch.


Caching users in memory

For the Auth System V2 project.
In V2 of the authorization system we will most likely be fetching the user objects from the database more frequently than right now. As a result, fetching users from the database may mean that we cannot guarantee the SLA described in the v2 auth system design doc.
As part of this task we need to:

Set up and run performance tests on hs_auth to measure the impact of fetching the user object from the database (may be useful to do these tests on an EC2 instance that we would use in production). We should assume that an authorization request to hs_auth could require up to 3 user fetches so if a single request to the database takes more than 1.5 ms, the tests can be very simple (running the same request for a single user say 100 times). Otherwise, the tests should cover datasets of multiple sizes (e.g: 10, 100, 1000 users in the database) and the access pattern should be at least somewhat randomized.
Discuss the results with the team and decide if caching users will be required
If required, investigate options for caching users in memory, discuss them with the team and create and merge a PR updating the logic of fetching users to make use of the in-memory cache.

Definition of done:

Completed performance tests, collected the results and discussed them with the team.
Made a decision on whether or not in-memory caching for user objects is going to be necessary.
If required, implemented and merged in-memory caching for users. Also added performance benchmarks for fetching users to the integration tests suite.

Closing, as the MongoDB client already does caching which seems to be sufficient.


[TEST] Try adding a CI check

This PR adds a CI workflow that runs on each PR.

I think this has to be under .github/workflows/pr.yaml. Additionally the way CI is currently setup probably means you need other steps, for example a .github/workflows/build.yaml. Is the intent to make pynvjitlink a full conda-forge/wheels package?

I think this has to be under .github/workflows/pr.yaml. Additionally the way CI is currently setup probably means you need other steps, for example a .github/workflows/build.yaml. Is the intent to make pynvjitlink a full conda-forge/wheels package?

Yes, we should ultimately have both wheels and conda packages.

Ok, then we probably need to do the entire setup here. That means we will need to copy https://github.com/rapidsai/cudf/blob/branch-23.12/.github/copy-pr-bot.yaml and https://github.com/rapidsai/cudf/blob/branch-23.12/.github/ops-bot.yaml into this PR. We will then need to add conda recipes and wheels build scripts. I can probably help get started here but we will also need help from @bdice and @vyasr who are the experts in packaging.
For the conda recipe, the closest recipe to pynvjitlink I know of is perhaps UCX-Py, so what I would do is first copy its conda files and remove UCX dependencies, probably replace UCX with Numba. We will then need to add build and tests CI scripts, which I would do the same, copy from UCX-Py and replace relevant ucx-py with pynvjitlink. Wheels build and tests scripts are also in the same ci/ subdirectory and probably can get us a starting point, there are no recipes like in conda but we often require static linking, where @vyasr is definitely someone who we will need help from.
If you want, please ping me on Slack and I can try helping get moving here a bit more, but most definitely we will need more help from Bradley and Vyas later anyway.

@pentschev we've met once or twice and made some progress planning this out - will loop you in on slack.

@bdice do you know why the ctk installation fails on centos by any chance? details below.

--> Finished Dependency Resolution
Error: Package: nsight-systems-2023.3.3-2<IP_ADDRESS>_233333266658v0-0.x86_64 (cuda)
           Requires: libOpenGL.so.0()(64bit)
           Available: 1:libglvnd-opengl-1.0.1-0.6.git5baa1e5.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:libglvnd-opengl-1.0.1-0.8.git5baa1e5.el7.x86_64 (base)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-367.48-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-375.26-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-375.51-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-375.74-1.el7.x86_64 (cuda)
 You could try using --skip-broken to work around the problem
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-375.88-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-384.66-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-384.81-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-384.111-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-384.145-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-384.183-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-387.26-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-390.12-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-390.30-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-[390](https://github.com/rapidsai/pynvjitlink/actions/runs/6734331671/job/18305010392?pr=10#step:5:391).116-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-396.26-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-396.37-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-[396](https://github.com/rapidsai/pynvjitlink/actions/runs/6734331671/job/18305010392?pr=10#step:5:397).44-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-396.82-1.el7.x86_64 (cuda)
               libOpenGL.so.0()(64bit)
           Available: 1:xorg-x11-drv-nvidia-gl-346.46-1.el7.x86_64 (cuda)
               Not found
           Available: 1:xorg-x11-drv-nvidia-gl-352.39-1.el7.x86_64 (cuda)
               Not found
           Available: 1:xorg-x11-drv-nvidia-gl-352.55-1.el7.x86_64 (cuda)
               Not found
           Available: 1:xorg-x11-drv-nvidia-gl-352.63-1.el7.x86_64 (cuda)
               Not found
           Available: 1:xorg-x11-drv-nvidia-gl-352.68-1.el7.x86_64 (cuda)
               Not found
           Available: 1:xorg-x11-drv-nvidia-gl-352.79-1.el7.x86_64 (cuda)
               Not found
           Available: 1:xorg-x11-drv-nvidia-gl-352.93-1.el7.x86_64 (cuda)
               Not found
           Available: 1:xorg-x11-drv-nvidia-gl-352.99-1.el7.x86_64 (cuda)
               Not found
 You could try running: rpm -Va --nofiles --nodigest

@bdice do you know why the ctk installation fails on centos by any chance? details below.

Not sure. Is there a package with fewer dependencies? We don't need NSight for this package to build.

I think the build is presently failing because the default standard is C++03 in GCC 4.8.5, and there isn't an explicit -std=c++11 flag. Hopefully C++11 is sufficient, as it's the last C++ standard supported by GCC 4.8.

@bdice is there a container that would be suitable for testing that does not have cuda built in at all? It isn't needed at all on the testing machines because we statically link the two pieces of it that we need in the build step and it's actually getting in the way of testing due to https://nvbugswb.nvidia.com/NvBugs5/SWBug.aspx?bugid=4375843&cmtNo=.

(Unless you're not relying on gha-tools or other RAPIDS CI tooling and can use a plain base container like ubuntu.)

@gmarkall and I met to fix up tests.  I'll follow up in a separate PR to fix those tests.  For now it seems like PR is ready for review then merge

This adds CI to PRs -- but we will need a build.yaml to make this publish the wheels to a nightly index for testing. @brandon-b-miller and @quasiben, are we ready for that? I think so?

This adds CI to PRs -- but we will need a build.yaml to make this publish the wheels to a nightly index for testing (maybe triggered on tags?). @brandon-b-miller and @quasiben, are we ready for that? I think so?

This seems like a logical next step to me


Android: Live regions

Description

This seems to exist as a prop, but is not functioning properly according to the documentation.
It only seems to exist on  and only seems to announce changes to the View's accessibilityLabel, not its descendant elements.

React Native version:
v0.63

I can no longer reproduce the issue here, this property seems to be functioning correctly.
Snack example link - https://snack.expo.io/9nyaxiJSK
I think we can close this one out.


Improve perfomance

Hello,
I recently see a change at gatsby codebase where they modified the original mitt source code to use a slightly similar implementation but 25% faster:
https://github.com/gatsbyjs/gatsby/pull/23223/files#diff-57dc49b17585e7e09b4cabd11499e032R29
It could be interesting to implement these changes back to here!
Do would you accept a PR in that direction?

Yeah, that seems like a compelling case. There are a few options here fwiw. Using Set is actually technically an API change, because it doesn't allow duplicate handlers, which EventEmitter allows.

This should be fixed by #99 :)


resize_image_with_crop_or_pad() for a batch of images

resize_image_with_crop_or_pad() should work for multiple images (batch of images) similar to tf.image.resize_images()

Duplicate of #521.


Add neon.running_xacts_overflow_policy to make it possible for RO replica to startup without primary even in case running xacts overflow

Problem
Right now if there are too many running xacts to be restored from CLOG at replica startup,
then replica is not trying to restore them and wait for non-overflown running-xacs WAL record from primary.
But if primary is not active, then replica will not start at all.
Too many running xacts can be caused by transactions with large number of subtractions.
But right now it can be also cause by two reasons:

Lack of shutdown checkpoint which updates oldestRunningXid (because of immediate shutdown)
nextXid alignment on 1024 boundary (which cause loosing ~1k XIDs on each restart)

Both problems are somehow addressed now.
But we have existed customers with "sparse" CLOG and lack of checkpoints.
To be able to start RO replicas for such customers I suggest to add GUC which allows replica to start even in  case of subxacts overflow.
Summary of changes
Add neon.running_xacts_overflow_policy with the following values:

ignore: restore from CLOG last N XIDs and accept connections
skip: do not restore any XIDs from CXLOGbut still accept connections
wait: wait non-overflown running xacts record from primary node

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Hmm, what's the plan on how this would be used? Setting GUCs for individual endpoints is not very convenient.
Alternatively, you can briefly start and stop the primary, so that it writes the shutdown checkpoint. That seems more convenient.

LGTM

You are still not a member of @neondatabase/compute so your review is not  considered as review of code owner:)

You are still not a member of @neondatabase/compute so your review is not considered as review of code owner:)

Added myself to the group, so I think it's good to go, unless you want me to ping someone else for review :)


Need help adding LCT002 Philips light

I added this to devices.js:
    {
        zigbeeModel: ['LCT002'],
        model: '9290002579A',
        vendor: 'Philips',
        description: 'Hue white and color ambiance BR30',
        supports: generic.light_onoff_brightness_colortemp_colorxy().supports,
        fromZigbee: generic.light_onoff_brightness_colortemp_colorxy().fromZigbee,
        toZigbee: generic.light_onoff_brightness_colortemp_colorxy().toZigbee,
    },

zigbee2mqtt succesfully adds it, however it's not getting displayed in home-assistant and I don't see it published in MQTT.fx

For home assistant integration, it also needs to be added here: https://github.com/Koenkk/zigbee2mqtt/blob/dev/lib/extension/homeassistant.js#L457
Can you make a PR with the above change? Then I will make sure it will get added to homeassistant.js.

Wow, I'm sorry, made a mistake, it should be this:
    {
        zigbeeModel: ['LTW011'],
        model: '4374300',
        vendor: 'Philips',
        description: 'Hue With Ambiance BR30',
        supports: generic.light_onoff_brightness_colortemp().supports,
        fromZigbee: generic.light_onoff_brightness_colortemp().fromZigbee,
        toZigbee: generic.light_onoff_brightness_colortemp().toZigbee,
    },

In 'model' I used SKU found with Google.

Just did a PR. When and in what branch can I see the change in the docker version of zigbee2mqtt?

Supported in the dev branch (-dev docker images)


Confirm deletion of collection

after:

https://github.com/kodadot/nft-gallery/pull/7960

there is currently no step between you clicking the button of delete and signing burning your collection

I would also rename this to Delete collection
it will be more clear what its about

design added in handoff, small things https://www.figma.com/file/JXNtryaNYhY0XJNArzvPQx/Handoff?type=design&node-id=1287%3A18959&mode=design&t=eidhg2mWSkXNBA35-1

its the same component as in migration go to home confirmation

Deposit should be mentioned only on ahk/ahp

@JustLuuuu I mean..
there is a deposit even on rmrk

I would also rename this to Delete collection it will be more clear what its about

don't you think that's a bit repetitive? "delete collection", "report collection"
plus goal to this button in futur would be to delete all items inside & collection when clicked (ref #938, #6047)

I would also rename this to Delete collection it will be more clear what its about

don't you think that's a bit repetitive? "delete collection", "report collection"
plus goal to this button in futur would be to delete all items inside & collection when clicked (ref #938, #6047)

Yeah, maybe rename from report collection to only report?


When the pod starts, add host aliases to the /etc/hosts file

When trino needs to access multiple clusters, the hosts for external cluster interactions need to be added to the /etc/hosts file, otherwise the IP of domain name format may not be reachable. Therefore, I added this feature to add the specified host aliases to the /etc/hosts file when the pod starts.

This seems to have a very specific use case. Can you add a test for this, similar to https://github.com/trinodb/charts/blob/main/charts/trino/templates/tests/test-connection.yaml, and configure it in https://github.com/trinodb/charts/blob/main/test-values.yaml?

This looks abandoned.


[NFC] Removed SILValue::getOwnershipKind.

It has been deprecated for awhile.

@swift-ci please smoke test


Translate Tutorial Page

Hello guys! Finally manage to translate the tutorial page.
Kindly please review. Thank you
PS: I am not translating React terms like props and state and class names.

Awesome mas @martinock 🎉.
@regalius @arfianadam, let's review this.

Thanks for the review, already fixed typos and revamp weird translations in the latest commit.

@regalius is this ready to be merged in?


Workaround gradle/gradle#11466

see Codearte/gradle-nexus-staging-plugin#141

Thanks for your PR. Could you add Gradle 6 to the Gradle regression testing suite to verify it fails with @ToString and does not after your changes?

This PR and #144 both contain a test now, which fails in #144 but succeeds here.

I slightly reworked your tests to be notified by CI if/when it will be fixed in Gradle. Merged manually, thanks.


x/website:

What version of Go are you using (go version)?

$ go version

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?

What did you expect to see?
What did you see instead?

Hi! It looks like you submitted an empty template, so I am going to close this. If you were trying to report an issue, please try again and fill in the details.


Add ability to copy files

Add ability to copy files

@creocoder @tonydspaniard can you merge this one? This is needed in one of my projects.
Also don't forget to add version 0.3 Git tag.
Thanks guys! ;-)


⚠ ClamAV has degraded performance

In fa98980, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 38 ms

Resolved: ClamAV performance has improved in 7c6cb3b.


Check correct time index of update-predict when y is univariate, both and multivariate

Check correct time index of update-predict when y is univariate, both and multivariate

I am closing this pr because it has been merged to #1301


Enables Cypher highlighting to be embedded in some programming languages

Adds highlighting within embedded strings in all the languages we have an official driver for:

Java
Python
Typescript / Javascript
Go
.NET: C# and F#

It works with multiline strings which include a //cypher or /*cypher*/ comment and with single line strings with a //cypher comment:
String a = """// cypher

MATCH (n)
"""

String b = """/* cypher */

MATCH (n)
"""

String c = "//cypher MATCH (n)"

Also with templated strings in js/ts:
/*cypher*/`
MATCH (n)

This is similar approach to https://marketplace.visualstudio.com/items?itemName=qufiwefefwoyn.inline-sql-syntax

I think this change would warrant a changeset as well!

The changeset I'd prefer to introduce it in another pr because I'll have to rewrite the README, fix a detail in the release job, etc.

Looking slick!! 💯


Resource requests

As discussed on the 3/27 call:

resource are no longer expressed as minimums
on shared compute resources these may serve as hard limits on the task
on cloud systems these may be used to size the instance that is provisioned

"these may"
So an implementation is free to ignore?

Yes @geoffjentry I think it is best to let implementations decide on how best to use resource requests.

👍


Podman fails on RHEL 7 in cronjob with default PATH

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description

Steps to reproduce the issue:

add podman command to crontab -e

wait for job to run

see error output

Describe the results you received:
level=error msg="Error adding network: failed to locate iptables: exec: \"iptables\": executable file not found in $PATH"

Describe the results you expected:
Podman command runs successfully
Additional information you deem important (e.g. issue happens only occasionally):
It can be worked around by setting PATH explicitly in the job to the same as the root user where it works.
Output of podman version:
Version:            1.4.4
RemoteAPI Version:  1
Go Version:         go1.10.3
OS/Arch:            linux/amd64

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.10.3
  podman version: 1.4.4
host:
  BuildahVersion: 1.9.0
  Conmon:
    package: podman-1.4.4-4.el7.x86_64
    path: /usr/libexec/podman/conmon
    version: 'conmon version 0.3.0, commit: unknown'
  Distribution:
    distribution: '"rhel"'
    version: "7.7"
  MemFree:<PHONE_NUMBER>
  MemTotal:<PHONE_NUMBER>
  OCIRuntime:
    package: runc-1.0.0-65.rc8.el7.x86_64
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.1-dev'
  SwapFree: 0
  SwapTotal: 0
  arch: amd64
  cpus: 2
  hostname: r76
  kernel: 3.10.0-1062.1.1.el7.x86_64
  os: linux
  rootless: false
  uptime: 143h 38m 2.91s (Approximately 5.96 days)
registries:
  blocked: null
  insecure: null
  search:
  - registry.access.redhat.com
  - docker.io
  - registry.fedoraproject.org
  - quay.io
  - registry.centos.org
store:
  ConfigFile: /etc/containers/storage.conf
  ContainerStore:
    number: 1
  GraphDriverName: overlay
  GraphOptions: null
  GraphRoot: /var/lib/containers/storage
  GraphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "true"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 1
  RunRoot: /var/run/containers/storage
  VolumePath: /var/lib/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
podman-1.4.4-4.el7.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
OpenStack VM

I don't know if we can work around this easily, as this is part of CNI - I don't think we can manually give it a path, it always uses $PATH

do you think you can run a script with an enhanced path?  anyways, this should probably be filed downstream in bugzilla if n o work-around is present.  And as mheon points out, probably against cni

Please open a bugzilla on this.


Solved #103 Implement Dark Theme for Website

Closes #103 Implement Dark Theme for Website
Earlier , there was no option to opt for dark theme, so now one can use it.
<I have checked the live website .>
[✓] My code follows the style guidelines of this project
[✓] I have performed a self-review of my own code
[✓] I have commented my code, particularly in hard-to-understand areas
[✓] I have added demonstration in the form of GIF/video file
[✓] I am an Open Source Contributor
Screenshots/GIF📷

Why scroll to top button is changed. Don't make changes to the exisiting codes.
When we are on dark theme, the moon icon should change to sun @thatguywhocode


Revert "Update rollup-plugin-babel to the latest version 🚀"

Reverts tabler/tabler-react#300

:tada: This PR is included in version 1.20.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update the widget for switching debuggers to also show the current debugger like in the one for tools

The widget for switching debuggers should be like the one for multiple tools in the inspector


Modelling resource constraints: Compute and Memory Units

Hello,
I have a question about the general model and intention of the resource constrains in Sonata.
Assuming I have a 8 core and 32GB RAM machine, and I would like to model 8 data centres using the sonata emulator.  We define network functions to consume 512 MB and 1/8 of a core and each datacenter will have 1 core and memory of 3584 MB available.
This means that each datacentre will be able to host a maximum of seven network functions.
The flavor of a network function will be 1 compute units, 512 MB memory, a data centre will have 8 compute units to model a single core.
As of this moment, a  basic model of our code is composed as follows:
MAX_CU = 8 # max compute units per machine, equivalent to one core
MAX_MU = 3584 # max memory units per machine, can host 7 VNFS
MAX_CU_NET = 64 # max compute unit available on the host machine
MAX_MU_NET = 28672 # max memory to be made available on the host machine
net = DCNetwork(controller=RemoteController, monitor=True,
				dc_emulation_max_cpu=MAX_CU_NET, dc_emulation_max_mem=MAX_MU_NET,
				enable_learning=True)

# Add 8 "datacentres" (or servers) to the network
chain_server1 = net.addDatacenter('chain-server1')
chain_server2 = net.addDatacenter('chain-server2')
chain_server3 = net.addDatacenter('chain-server3')
chain_server4 = net.addDatacenter('chain-server4')
chain_server5 = net.addDatacenter('chain-server5')
chain_server6 = net.addDatacenter('chain-server6')
chain_server7 = net.addDatacenter('chain-server7')
chain_server8 = net.addDatacenter('chain-server8')

# Create one resource model per "dc"(or server)
rm1 = UpbSimpleCloudDcRM(MAX_CU, MAX_MU)
rm2 = UpbSimpleCloudDcRM(MAX_CU, MAX_MU)
rm3 = UpbSimpleCloudDcRM(MAX_CU, MAX_MU)
rm4 = UpbSimpleCloudDcRM(MAX_CU, MAX_MU)
rm5 = UpbSimpleCloudDcRM(MAX_CU, MAX_MU)
rm6 = UpbSimpleCloudDcRM(MAX_CU, MAX_MU)
rm7 = UpbSimpleCloudDcRM(MAX_CU, MAX_MU)
rm8 = UpbSimpleCloudDcRM(MAX_CU, MAX_MU)

# Assign each resource model to the corresponding server
chain_server1.assignResourceModel(rm1)
chain_server2.assignResourceModel(rm2)
chain_server3.assignResourceModel(rm3)
chain_server4.assignResourceModel(rm4)
chain_server5.assignResourceModel(rm5)
chain_server6.assignResourceModel(rm6)
chain_server7.assignResourceModel(rm7)
chain_server8.assignResourceModel(rm8)

It seems to work fine and does not fail. However, I am concerned about the correctness of this approach. Is this how the resource model of Sonata is meant to be used, or are we potentially misinterpreting some functionalities (e.g., modelling the network maximum as the total amount of resources across all dcs)?
Thank you,
fruffy

Good morning @FRuffy,
maybe I can clarify some things.
The first thing you need to understand is that the resource models only modify the memory limits and the CPU shares (available CPU time fraction) of a container. NOT the number of cores used per container or datacenter.
For the CPU model this means, you first need to set the overall CPU time fraction that should be dedicated to the emulated VNFs (the containers) which is done by your code with the parameter MAX_CU_NET. Important here is that this describes the fraction of CPU time so it should be a value between 0.0 and 1.0 (e.g. 0.7 to use max. 70% of CPU cycles of the host machine for all containers in your emulation). I assume this needs to be changed here since you use 64 (maybe 0.64 is a good option for you). Using 1.0 does normally not make sense since you need to leave some CPU resources to your host os etc. ...  Memory is different: Here the absolute value in MByte has to be given (this looks good in your code).
Then you can specify the abstract compute units per datacenter as you did with MAX_CU. Here you can really use any positive natural numbers since this is only an abstract unit. The selected resource model will then compute the fraction of CPU time of a single container based on the CU values assigned to the started container (e.g. MAX_CU_NET / sum(MAX_CU of each dc) = fraction of CPU time that corresponds to one CU requested by container instantiation). The model will reject the instantiation of a container when CU of already running containers + new request > MAX_CU for a datacenter (no oversubscription).
You see, the CPU model does not have any influence on the number of CPU cores used, only the CPU time! This means all available CPU cores are always used in this model.
However, you can dedicated cores to containers by setting the cpu_set parameter when they are instantiated. Which means you could also modify the used code to allocate all containers of one emulated DC to a single  CPU core. But this requires some (smaller) modifications of the code (maybe subclassing the resource model might be a good option to start here).
Does this help? Let me know if you have more questions.
Best,
Manuel

Hi Manuel,
thanks a lot, that was very helpful. I have read the paper (that is how we found Sonata!), but I was not quite sure about how units were related to actual computing power, especially the max compute of the data centre. Our intention was to "emulate" our cores in terms of computing power. We were not specifically concerned about the right amount of cores. So eight compute units basically expressed the power of one core in a data centre host.
We are going to remodel our code based on your suggestions!

Great. Let me know if you have any trouble.
I'll close this issue for now. If you have further questions feel free to re-open it or create a new one.


[feature] Jump to the line instead of only jumping to the page.

usecase
Nice plugin. Currently, it is capable of searching & jumping to the correct settings page.
But when dealing with really-long settings pages (namely, Excalidraw and Style settings), only jumping to the right page still leaves an awful lot of reading and scrolling to do.
proposed solution
When clicking the link from search result, jump to the exact line that contains your query, instead of only jumping to the page.

+1, it works well on iOS mobile for jumping to the right line in long setting pages like "style setting", but it jumps back unexpectedly to the main page on macOS Obsidian. A video can be uploaded if necessary.

Please make it jump to the found block and also highlight it for a second or two with a border or a padding, fading out.

This should be fixed now


[EOSi Frontend] #20 For the numbers, add a "count up...

For the numbers, add a "count up" style JS effect.
For example:
https://inorganik.github.io/countUp.js/
Source url: https://eosi-frontend.netlify.app/why-eosi.html
Reported by: Chris ()
Reported at: 07 Jul at 14:48 UTC
Type: Feedback
Location: US, Massachusetts, Buzzards Bay
Browser: Firefox 102.0
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Firefox/102.0
OS: macOS 10.15
Screen: 2560x1440
Viewport: 1341x726
Referrer: https://eosi-frontend.netlify.app/industry-template.html
Screenshot:

For more details please visit report page on Ybug:
https://ybug.io/dashboard/reports/detail/qyhgvhw08z9g1mw6d2qb

I tried to do this <3

I tried creating it this way, I'm having a problem, scrolling down doesn't work, just scrolling up works, I don't know how to reverse scroll
@webrgp
import { CountUp } from 'countup.js';

const init = ( selector: string ) => {
  const elements = document.querySelectorAll( selector );
  elements.forEach((el) => {
    const countUp = new CountUp(el.id, parseFloat(el.innerHTML), {suffix: "+", enableScrollSpy: true});
    !countUp.error ? countUp.start() : console.error(countUp.error);
    countUp.handleScroll(countUp);
  });
}

init('[data-count]')

I will take a look.

I decided to just straight up write our own function because the library was too generic and added too much bloat.
Also, it didn't format the numbers with commas.


HF Integration

Follows what OLMo does with their HF integration.
This allows us to work with HF without having to create new classes in the upstream transformers repo. We now directly read from this repo, so we also don't need to worry about the OpenLM codebase being updated in the future.
Usage is exactly the same as the standard HF usage, except with the additional import in the first line.
from open_lm_hf import *

from transformers import AutoTokenizer, AutoModelForCausalLM
tokenizer = AutoTokenizer.from_pretrained("tri-ml/openlm-7b-300b")
model = AutoModelForCausalLM.from_pretrained("tri-ml/openlm-7b-300b")
a = tokenizer("Hi, nice to meet you.", return_tensors='pt')
out = model.to("cuda").generate(a['input_ids'].to("cuda"))
print(tokenizer.decode(out[0]))

Some things not implemented yet:

some extra HF functions like ResizeTokenEmbeddings
The HF forward output tuple CausalLMOutputWithPast usually returns the full hidden states, but OpenLM's forward doesn't return that, so I left it as None for now.
There's also a chunk if labels is not None: that I just copied from OLMo and didn't test.

HF model repo config.json should look something like this
{
  "dim": 4096,
  "n_layers": 32,
  "n_heads": 32,
  "vocab_size": 50432,
  "norm_eps": 1e-5,
  "seq_len": 2048,
  "weight_tying": false,
  "apply_qk_norm": true,
  "norm_type": "gain_only_lp_layer_norm",
  "positional_embedding_type": "rotary",
  "ffn_type": "swiglu"
}

These correspond to attributes in the Params class.


tests: get rid of unused parameter

Would it be possible to get rid of these functions?
I remember they were duplicates of makeChainedTransactions.

Would it be possible to get rid of these functions?

That would be ideal. Someone should take care of these in the future.

Would it be possible to get rid of these functions?

That would be ideal. Someone should take care of these in the future.

Relates #1107

Also it's currently blocking my work on the execution engine. I just have this one failing test, but the test was written in such a terrible copy-paste way that it's hard to see what the test is actually trying to accomplish.


Kopernicus doesn't work in 1.0

Source
There are dev-versions out that fix the issue but nothing official yet.

Failing because Jenkins is in the future.


 Deployer behind drills so lava can be destroyed by placing blocks, then breaking them.
I dont see a log of the game being useful here.
Update: a workaround i found is you can place the deployers in FRONT of the drills, but in order for this to work properly you have to use blocks on the outside of your drill machine to allow the deployers to connect to the rest of the drill contraption. My assumption now is there is something wrong with the drills where they dont detect placed blocks fast enough and just dont mine them as a result, so it has to be two blocks in front first.
Fabric 1.20.1
Create version: 0.5.1-f/build.1417+mc1.20.1
Game Log
https://mclo.gs/gQQyR9F
Debug Information
No response

My guess is that this is caused by the deployer placing the block inside the drill sometimes depending on alignment. I can almost guarantee this happens on forge too


Add clarification about how to contribute to README

I wanted to make a (small) edit to https://beancount.github.io/docs/external_contributions.html (to add the full list of the existing importers of @tarioch https://github.com/tarioch/beancounttools), and figured this Git repo is the source of that site.
It's just not clear (to me) yet, neither from the README nor the CONTRIBUTING, how one may raise a PR or suggest an edit - in whatever form the maintainers of this (amazing) community prefer.
CONTRIBUTING says to check this thread which I have glanced over but still not understood how to.
Perhaps this could be clarified; I'm happy to raise a PR if when I learn how you like contribs.
@bradyt @xuhcc

@vorburger In the past contributing required write access to google documents. Some community members had it, but things could have changed.
I think it's better to ask about that in the maling list.


Add little delay to make sure the client gets the success message

It can happen that the sockets closes before the client got the success massage, this makes sure that this doesn't happen or at least makes it far less likely to happen.

delay for message


Invalid background size error: (-215) size() == mask.size() in function copyTo

Thank you for this fun project!
I ran into the background issue at runtime.
Just adding cv::resize(bg,bg,cv::Size(width,height)) after loading the background did the trick: https://github.com/floe/deepbacksub/blob/master/deepseg.cc#L94
Maybe just changing background.png to background_retro.png in the sample will avoid the ugly opencv exception only meaning the file does not exist
Regards

Adding that line fixed the problem I ran into.  Thanks!

Done and done. Thanks for the suggestion! I'll leave this open for the moment because it looks like cv::resize is messing with the color ordering, though (BGR <-> RGB)...


DistributionData and view.Rows in general do not get reset between Reporting Intervals

I have a program that records the latency of a request using the OCHTTP server middleware.
In my program, I am using the DataDog OpenCensus Exporter to report this metric.
Therefore, I wanted to simulate a latency spike by having my handler do the following:

The first 4 minutes, the HTTP Handler takes between 200-400ms to respond.

The second 6 minutes, the HTTP handler takes between 1-2seconds (latency spike)

The third 4 minutes, the HTTP handler takes between 200-400ms (latency goes back to normal)

The latency gets reported every 10 seconds
In other words, OpenCensus calls DataDog's ExportView every 10 seconds.
OpenCensus collects the Min/Max latencies and passes them as DistributionData to the DataDog exporter.
My expectation is that for every 10 seconds, OpenCensus calculates the Min/Max/Mean/etc and passes them to an Exporter, but then it should reset those numbers so that they don't obscure the next bucket of time. Is this assumption incorrect?
Even if this assumption is wrong, let's continue with how this behavior has obscured the latency spike significantly:
The fact that OpenCensus does not "reset" the Min every 10 seconds, means that the Min stays the same across the entire lifetime of the Go process.
Therefore, if we had a dashboard of the "min" value only, you'll see that the min never went up, it stays as ~200ms for the entirety of the 14 minutes. Even though for 6 minutes, the Min value was at least 1000ms.
On the other hand, if you have a "max" metric, you'll see that it went up to ~2s, but then it never went back down even though the traffic spike ended after 10 minutes.
Predictably, if I "kill" the process and restart it, suddenly the dashboard gets fixed and that's because we no longer hold the "min/max" that was previously preventing the dashboard to be accurate. See the following screen shot for demonstration:

What you see here is that the Purple line is the Max metric while the Blue line is the Min metric.
When 4 minutes passed, the Purple line went up, but the Blue line stayed down. This is because even though new incoming requests were clocking between 1-2 seconds, OpenCensus kept comparing those numbers to the Min that was initially recorded in the first 4 minutes.
However, once I kill the process and restart the server (resuming the traffic spike) you can see the Min gets updated. Similarly, when the spike finished, the Max never goes down until I forcefully reset the Max.
As far as I understand, and at the very least, restarting the process should not significantly affect what the chart looks like as seen in the screenshot above
My understanding is that OpenCensus creates a metric for a bunch of Measures per Reporting Interval (I can easily be wrong about this). Therefore, if I manually empty out the DistributionData after every interval, you can see the Dashboard now looks very correct:

Finally, I have created a reproduction Repository here: https://github.com/marwan-at-work/rcrepro
I included a Vendor folder that fmt.Println's the incoming DistributionData and you can notice that even when the traffic starts to go up to 1-2 seconds for 6 minutes, the Min value that DataDog receives stays the same.
AFAIK, there's no need to actually set up Datadog, you can just run the program and inspect the logs.
Also, I'm not familiar with the OpenCensus codebase (until this issue), but all I had to do to fix it was add this line: https://github.com/marwan-at-work/opencensus-go/commit/1380fae97d1e9d7e7d96593154699df45bfb1b7d#diff-2000ebb97830a1f0f1c5c4856a737f78R236
Not sure how thorough the unit tests are, but this didn't break any of them.

@odeke-em gentle ping to see if you or anyone be able to look at this. Thanks!

Thank you for the ping Marwan! @rghetia works on OpenCensus and
OpenTelemetry actively and might be able to help out, kindly pinging you
Rahul.
On Wed, Nov 6, 2019 at 9:55 PM Marwan Sulaiman<EMAIL_ADDRESS>wrote:

@odeke-em https://github.com/odeke-em gentle ping to see if you or
anyone be able to look at this. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/census-instrumentation/opencensus-go/issues/1181?email_source=notifications&email_token=ABFL3V3SY77SQ6MYSJ3OGQDQSOUUZA5CNFSM4JHACB2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDKRQOY#issuecomment-550836283,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFL3V6KC5TVL2ZGHFYMTCDQSOUUZANCNFSM4JHACB2A
.

@marwan-at-work I just commented on your PR.


Make shallow clone when fetching dependencies

What language and solver does this apply to?
CMake build
Describe the problem you are trying to solve.
I tried building OR-Tools with all dependencies on a machine with slow and unreliable internet access (very strict corporate proxy) and git clone couldn't fetch some of the largest dependencies (e. g. protobuf) because of their size. The problem went away when I forced CMake to reduce the size by making shallow clone.
Describe the solution you'd like
Add GIT_SHALLOW 1 to FetchContent_Declare for large dependencies in dependencies CMakeLists. It shouldn't break anything because CMake only needs the latest commit to build a dependency.

Done with https://github.com/google/or-tools/commit/f5d9ea77be10a8d33f1e1afbcd9201fcdc9828f3
note: GIT_SHALLOW do not work when using commit sha1


HIVE-25291: Update masking_mv_by_text_2.q.out

What changes were proposed in this pull request?

update masking_mv_by_text_2.q.out file
Why are the changes needed?

Stable build
Does this PR introduce any user-facing change?

No
How was this patch tested?
Qtests

+1


Output panel font is very hard to see

Hi, this might be very minor issue that isn’t worth posting on github but personally I think it’s really annoying: The dark theme makes the output panel unreadable. The light theme’s output panel is very clear but it’s too bright and I prefer dark theme.

Thanks for reaching out. This is not supposed to happen and should be a bug. Can you try to turn dark theme off and on in the app's setting menu?

I tried turning on/off the dark setting, to automatic and light and different themes and even exit the app and open again in new projects and files but still the same, if its dark mode then whichever themes will make output panel and only output panel invisible/ very hard to read font

Got it. As I cannot reproduce this issue on my device, the last thing you could try is to reboot your device. Please provide the app version (scroll to the bottom of the settings page), iPadOS version and device model so that I can try to reproduce this issue.

Rebooting didnt help.
App ver 1.2.7 Build 39
IpadOS 14.7.1
Ipad 8th gen 10.2 inch model A2270

Thank you! I will further investigate into this issue.

I really love the app, clean and truly vscode-like ui, thank you!

just updated to the latest ipadOS 15.2, panel looks nice now


Replace the select2 dropdown boxes

Create plan: edit field/combo box. Investigate a better input mechanism. Difficult for accessibility challenged users to use. Though does pass accessibility review.

going to replace with the jquery option recommended by @JEK-III
https://a11y.nicolas-hoffmann.net/autocomplet-list/

@JEK-III @sjDCC @stephaniesimms here is a screenshot of what the jquery dropdown looks like right now. Let me know if I should tweak any of the fonts, colors, etc.
Note that the search it uses does a 'contains' style search so I can type 'Tech' and get 'Tampere University of Technology'. Its also case insensitive 👍

@briri omg it looks so much better! i think the font style and size should be as close as possible to matching those on the rest of the page ("Select the primary research org..."), but maybe @JEK-III can confirm.

I agree with Stephanie in terms of consistency, and also that this is already a huge improvement.
I do have a some of specific suggestions about the layout, but nothing critical– if anything is going to be too time-consuming, don't sweat it.
I'd style the clear field [X] buttons a little differently– using the font awesome times-circle icon or something similar. And, the proportions of the button are weird (too narrow), but I think the best thing would be to get rid of the border and shadow altogether and just have the icon. Finally, to cut down on clutter, if would be nice to hide the button entirely when the text entry field is blank.
The suggestion text in the dropdown feels pretty cramped; it could use 10 to 20px of padding all around.  If possible, it would also be great to fix the width of the dropdown to be the same as the text entry field.

Ok. replaced their [X] clear button with the fontawesome icon. Added some padding around the suggestions in the dropdown and bumped the width of the suggestions so that it matches the width of the textbox.
I also set it up so that the clear button only shows if there is text in the box.

Really like the rework. Looks great!

Agreed– this does look great!

Also noted in #150 create plan workflow, but maybe better to address separately here: Is it possible and/or advisable to provide a full dropdown list in addition to the type ahead selector? Specifically, I'm concerned about users knowing what to type for the Funding Org and think they might also need/want to see a list of options in some cases. Maybe the Fundref integration solves this issue?
Here's a screenshot of one solution from a login wireframe, but I'll defer to @JEK-III for advice here...

The feasibility of putting a full list of funders in really depends on whether we're limiting to funders with templates in the system (feasible) or all of fundref (not feasible).

I agree that its a little unclear on what they should do with these new type-ahead boxes. We say 'select' in the instruction above the box but the user has to start typing something. If they don't know what to type though it can be confusing.
I was originally thinking it would be nice if they could hit the down arrow and just see the suggestions but the tool specifically prevents this (I assume to prevent having to show an enormous list). We could hack it to allow for that functionality but I'm reluctant to do that unless its something I could get them to include in their codebase. If they don't want it we'd then have to maintain our own version of their code.

@briri And I actually requested that you remove the language to "select" in the instruction boxes above because I think this will confuse users for the Org question in particular. I think it's more straightforward to just label the boxes "Primary research org" and "Funding org" with the primary question for each above.
But I expect usability testing would reveal that some users just don't know what to start typing in either box, especially in the funder box. Hacking is not desirable. Neither is an enormous dropdown list. Is it possible to filter Fundref to only display funders with templates in the system?

Perfect with the instruction to 'Begin typing to see a filtered list). These aren't showing on roadmap-staging yet so leaving for @stephaniesimms to close out.
One thing I noticed is that I naturally type the short form of the funder name i.e. NERC not natural environment research council, which means it doesn't come up. I assume we can just add the short forms in brackets at the end of the funder name though to fix this?

I just got some feedback about this very thing: someone looking for the "NASA" template. I agree we need to include short forms somehow and just wonder if that's supported by Fundref. I'll follow up with @briri and @JEK-III when they return next week. We can manually input the short forms for now at least.

The new combobox has an issue with strings which contain an ampersand, when you select an Org like "Cowmilk & Cheese College" the returned string is "Cowmilk & Cheese College" which doesn't match the Org's ID and therefore is not selectable - see https://github.com/DMPRoadmap/roadmap/issues/270#issuecomment-304676180

all good on Roadmap stage now. closing out.

@stephaniesimms should I create a separate issue for the ampersand bug?

i believe @sjDCC noted that she changed all & to "and," which should fix the problem. we don't use & in DMPTool.

We should still fix it though as other people using the codebase may have special characters in their data. I think its acceptable to handle in the MVP though instead of MVP0.3

created new issue for CDL MVP #396


prospect: Add version 0.4.0

Closes #9433

[x] I have read the Contributing Guide.

/verify

/verify

/verify


README example not correct

On line 21, it has facility: bsyslog.local0.
Should that be facility: bsyslog.facility.local0?

All of the facilities are available on both the module, and on the facility object. See here.


Roll Flutter Engine from 1d5e3cc55a75 to b097a62af5db (4 revisions)

https://github.com/flutter/engine/compare/1d5e3cc55a75...b097a62af5db
2024-06-26<EMAIL_ADDRESS>Roll Skia from 0a979d9f3606 to 173ee0af82c9 (2 revisions) (flutter/engine#53593)
2024-06-26<EMAIL_ADDRESS>Remove otherwise unused third_party/web_dependencies. (flutter/engine#53588)
2024-06-26<EMAIL_ADDRESS>Copy flutter/flutter/docs/engine to flutter/engine/docs as-is (no changes) (flutter/engine#53595)
2024-06-26<EMAIL_ADDRESS>Roll Dart SDK from 38bb74f63829 to c01f907d34d8 (5 revisions) (flutter/engine#53585)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Debian Jessie use /etc/default/snmptrapd for $trap_sysconfig

It seems that it changed between releases.
TRAPDRUN and TRAPDOPTS now need to be set in /etc/default/snmptrapd instead of snmpd.

I forgot to mention that it also has it's own package and it's not bundled with snmpd anymore.


Saving links with different lengths and playing with the config values

Screenshots (if appropriate)
Types of changes

Changes visible to users:

[x] Bug fix (prefix: fix - non-breaking change which fixes an issue)
[ ] New feature (prefix: feat - non-breaking change which adds functionality)
[ ] Breaking change (prefix: feat!! or fix!! - fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (prefix: docs - improvements to any documentation content)

Internal changes:

[ ] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
[ ] Tests (prefix: test - additions and improvements to unit tests and the smoke tests)
[ ] Infrastructure (prefix: chore - examples include GitHub Actions, issue templates)

Checklist

[x] My code follows the code style of this project and passes npm run lint.
[x] My change requires an update of README.md
[ ] I have updated the README.md

Thanks for the PR idea, but I close it in favor of #189.


Add locations to the nodes on the Network Graph [feat]

Is your feature request related to a problem? Please describe.
I've taken a more semantic approach to naming my z-wave nodes, e.g. /kitchen/light_switch, /playroom/light_switch (instead of /kitchen/kitchen_light_switch or /playroom/playroom_light_switch. So as I've got multiple entities with the same name (but different locations), it makes the network graph hard to read (without clicking on each node).
Describe the solution you'd like
I suspect not everyone will want to see the location, so I'd like to be able to see the location by a toggle above the graph. When enabled, the node names will appear in the format {location}/{node_name}.
Describe alternatives you've considered
The only alternative is to put the location name in the node name, or to click each node to see it's location.
Additional context

@djdd87 It could be done yes, would you mind to send a PR for that?

I'll happily take a look. Is there any way I can fake a set of nodes for the mesh? I only have one Z-Stick and it's running my Home Automation.
Does this UI change work for you?

@djdd87 Yes that is how I would made that :) To fake nodes just add a script that adds them, check Mesh.vue the convertNode function is the one that creates the nodes to add to the mesh graph :)

I'd like to do it! I saw that method, but I still can't work out how to get the nodes in. I'm a C# dev of nearly 20 years and I can't seem to get the Visual Studio Code debugger to attach/not error/do anything.
Where and how can I script the nodes in? I tried importing my nodes.json file, but that doesn't seem to do anything, and I can't work out how to replace the running store directory with a copy of my live store directory.

You could create a fake nodes array:
:
addFakeNodes(){
var nodes = [{//fakenode}, {//fakenode},{//fakenode}]
for (var i = 0; i < nodes.length; i++) {
        this.nodes.push(this.convertNode(nodes[i]))
}
}

And call this method in mounted()

@djdd87 Don't waste your time I have a PR ready for this


Automate the process of updating SQLite version

This PR creates a new workflow dedicated for the process of updating SQLite.
The thought behind this addition is that it seems much easier and cleaner in regards of both safety with binary files and wide testing. Just thought of creating this since the topic of safety was brought up by @JoshuaWise recently in a PR conversation.
The workflow is manually dispatched and presents you with the following prompt which are also the variables needed for the download script.

Notes:
🟢 This PR does not break the current manual update flow. In fact as seen in the diff, it's done without changing a single line of code in the download script.
🟢 Instead of directly pushing the changes, this workflow will do the changes on a new branch and an automated pull request will be created from that branch which will then be tested automatically just like any other PR.
🟢 The functionality of this workflow (with slight changes) has already been tested at better-sqlite3-multiple-ciphers.
🔴 The automatically created PR will not trigger tests/checks at the moment because of a limitation imposed by GitHub when using GITHUB_TOKEN. This can be easily fixed by using a Personal access token with repo privileges as the secret just in this step.
If you check this automated PR you can see that it triggered the tests and works as intended by using a PAT instead of GITHUB_TOKEN.
Hope this was worth the time and effort. ✌🏼

This is fantastic, thank you ❤


Allow AWS_XRAY_DAEMON_ADDRESS or DaemonAddr to use a hostname

The example ecs task definition on https://docs.aws.amazon.com/xray/latest/devguide/xray-daemon-ecs.html uses a hostname instead of an IP address for the AWS_XRAY_DAEMON_ADDRESS environment variable. This causes an error in daemoncfg/daemon_config.go:128 where the address is assumed to be an IP.
I've locally added net.LookupIP() to translate the hostname to IP address but this might be a good addition to the SDK as well.

Hi @dgiamanco-fortmason ,
Thanks for the feedback. We are working on the issue. I think net.ParseIp() : Link was unnecessary that prevented hostname parsing.
We can rely on underline UDP and TCP resolver, to resolve IP/Hostname and propagate errors, if any: Link
Best,
Yogi

That should work, thanks for looking into it.


Sensitive features tied to assessment data

Expected Behavior
When training/assessment dataset are passed, Lens creates versions of the evaluator for each dataset with the appropriate sensitive feature. It seems this functionality should exist in _cycle_add_through_ds_feat
Actual Behavior
Sensitive features are specified in relationship to the assessment dataset. This breaks down when only training is specified OR when assessment and training have different sensitive features.

@IanAtCredo is there a ticket for this?

What is the rational for using different sensitive features in training vs assessment?
What would be the expected behavior? -> Cycle separately for each of the dataset?


Move functionality into user control

Hi zetoken,
I like your editor. It is simple and clean!
But one thing: is it possible that you move the gui of the editor from JsonEditorMainForm into a user control? This would increase the the reusability so much!
Best regards from Germany
Sebastian

Hi,
You're right, I should have done that before.
Definitely on my "to do list".

The JsonTreeView control handles all json details (parsing) and treeview. The API is simple:

SetJsonSource: method to set the source of the control as a string or a stream.
SaveJson: method to save the loaded json to a stream.
AfterSelect: event called after a node is selected.

Thanks for the idea.


Revoke throws an error when trying to process the response

When I call revoke, I end up getting an error because it is trying to destructure the response JSON when the json is null. This happens even though the request to Intuit succeeds and the token is revoked. Also, it looks like the tests for revoke are also failing locally.

I'm seeing this as well.  The error  I get is "TypeError: Cannot destructure property 'request' of '(intermediate value)(intermediate value)(intermediate value)' as it is null."
It says it's occurring here https://github.com/intuit/oauth-jsclient/blob/master/src/OAuthClient.js#L319

Thanks @Matthew-John-Emerson and @bogeorget4 , could you please help with the steps to reproduce this error.

I run into this as well. I am able to see it when I connect to QBO, and then disconnect through the use of OAuthClient.revoke(); which responds with status 200 but there is no body on the JSON response. Thus, the disconnect is successful but it throws an error.

Hi, @rajeshgupta723
I have the same problem, I use intuit-oauth: 4.1.2.

I too am encountering this issue, on a DELETE request. 4.1.2

Hi, thank you all, will look into this. In the meantime, feel free to raise a PR for the fix.

Hi @amadeogallardo @vektorblake @stepanFederlian @Matthew-John-Emerson @crisebrough @bogeorget4  -- could you all please get the latest from the branch release-4.1.3 and test this out. Please confirm if the fix works for you. Thanks

Hi @rajeshgupta723 , I did try the release-4.1.3 and still got an issue.
The res returns an empty body and a null json. It seems like the validation of the json is to make sure we get a valid response from the server.
I have attached a screen shot.

I have changed line 319 from
const authResponse = res.json ? res : null;

to
const authResponse = res.hasOwnProperty('json') ? res : null;

and the error ran away.

I am also hitting this in 4.1.2. It was introduced in #165. /cc @ianberryman

Hi @m4xleb could you please get the latest from the branch named "Release-4.1.3" and retest the same. This has been fixed. Let me know if you still see any issues. Thanks

Hi @andrew-amplinks please see the latest release 4.1.3 here: https://github.com/intuit/oauth-jsclient/issues/170#issuecomment-2447243254


Segmentation fault when saving checkpoints with saveable Dataset Iterator

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): CentOS 7.6.1810

Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No

TensorFlow installed from (source or binary): Binary

TensorFlow version (use command below): 1.13.1

Python version: 3.6.8

Bazel version (if compiling from source): None

GCC/Compiler version (if compiling from source): None

CUDA/cuDNN version: None

GPU model and memory: None

tf.version: 'v1.13.1-0-g6612da8951' 1.13.1

Describe the current behavior
Segmentation fault in saving an initializable dataset iterator when entering the tf.train.MonitoredSession context manager.
Describe the expected behavior
The initializable iterator is saved and restored properly, behaving the same with the one shot iterator.
Code to reproduce the issue
"""Illustrate saveable dataset iterator
"""
import tensorflow as tf

DATASET_SIZE = 4
SAVE_STEPS = 2
TRAIN_STEP = 3
CHECKPOINT_DIR = '/tmp/tf_dataset_saveable'

def test_saveable():
    """test saveable"""
    graph = tf.Graph()
    with graph.as_default():
        dataset = tf.data.Dataset.range(DATASET_SIZE).repeat()
#        dataset_iterator = dataset.make_one_shot_iterator()
        dataset_iterator = dataset.make_initializable_iterator()
        dataset_init = dataset_iterator.initializer
        data = dataset_iterator.get_next()

        saveable = tf.contrib.data.make_saveable_from_iterator(dataset_iterator)
        tf.add_to_collection(tf.GraphKeys.SAVEABLE_OBJECTS, saveable)

        global_step = tf.train.get_or_create_global_step()
        inc_global_step = tf.assign_add(global_step, 1)  # critical

        saver = tf.train.Saver()
        checkpoint_dir = CHECKPOINT_DIR
        scaffold = tf.train.Scaffold(saver=saver)
        checkpoint_hook = tf.train.CheckpointSaverHook(
            checkpoint_dir=checkpoint_dir,
            save_steps=SAVE_STEPS, scaffold=scaffold)

        hooks = [checkpoint_hook]
        session_creator = tf.train.ChiefSessionCreator(
            scaffold=scaffold, checkpoint_dir=checkpoint_dir)
        with tf.train.MonitoredSession(
                session_creator=session_creator, hooks=hooks) as mon_sess:
            gstep = mon_sess.run(global_step)
            if not gstep:
                mon_sess.run(dataset_init)
            for _ in range(TRAIN_STEP):
                print(mon_sess.run([global_step, data]))
                mon_sess.run(inc_global_step)

if __name__ == '__main__':
    test_saveable()

Other info / logs
(tf-1.13-py3) [huwh1@huwh1-centos worksync]$ python tf_dataset_saveable.py
WARNING:tensorflow:From /home/huwh1/virtualenv/tf-1.13-py3/lib/python3.6/site-packages/tensorflow/python/data/ops/dataset_ops.py:1419: colocate_with (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Colocations handled automatically by placer.

WARNING: The TensorFlow contrib module will not be included in TensorFlow 2.0.
For more information, please see:
  * https://github.com/tensorflow/community/blob/master/rfcs/20180907-contrib-sunset.md
  * https://github.com/tensorflow/addons
If you depend on functionality not listed there, please file an issue.

WARNING:tensorflow:From tf_dataset_saveable.py:20: make_saveable_from_iterator (from tensorflow.contrib.data.python.ops.iterator_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use `tf.data.experimental.make_saveable_from_iterator(...)`.
2019-06-20 10:43:20.947675: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2019-06-20 10:43:20.951984: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2019-06-20 10:43:20.952497: I tensorflow/compiler/xla/service/service.cc:150] XLA service 0x3f10ca0 executing computations on platform Host. Devices:
2019-06-20 10:43:20.952539: I tensorflow/compiler/xla/service/service.cc:158]   StreamExecutor device (0): <undefined>, <undefined>
Segmentation fault (core dumped)
(tf-1.13-py3) [huwh1@huwh1-centos worksync]$

I tried on colab with Tensorflow 1.13.1. I am able to reproduce the issue.

allenl@ -- not sure if these APIs are expected to work all together. Can you advise as to whether there's a better way?

Oops. Actually tagging @allenlavoie

Is the issue that the iterator isn't initialized when the first checkpoint is written? You may need to get MonitoredTrainingSession to do the initialization so it happens before saving. Otherwise the APIs should go fine together AFAIK.
Either way, it probably shouldn't segfault. @saxenasaurabh may be more familiar with the serialization op itself.

@allenlavoie is correct, you need to run the dataset_iterator.initializer before you can save the iterator.
I am going to create a change that will produce an informative error message for this case (as opposed to segfaulting).

Many thanks to the fruitful discussions. Adding dataset initializer to the collection TABLE_INITIALIZERS fixes the segfaulting. Nevertheless, the iterator starts from the very beginning of the dataset every time after recovering from the checkpoint.
Is there any solution to the problem at present? Maybe something like init_fn in tf.train.Scaffold ...

That should not be the case. The whole point of saving the iterator is that you can checkpoint its state (and we have tests that verify that this functionality works).
Please create a new issue with instructions on how to reproduce your issue so that someone can investigate why is your program resetting the iterator state.

I think this is a missing feature. We added CheckpointInputPipelineHook to fix exactly this problem. However, it requires an estimator as an arg. It doesn't necessarily have to. We just did it this way for simplicity.
It should be fairly straightforward to extend  CheckpointInputPipelineHook to support non-estimator use-cases e.g. by explicitly passing the required args i.e. num_worker_replicas, task_type, task_id, model_dir , save_checkpoints_secs, save_checkpoints_steps. I would be happy to review if you want to send in a PR.
As a really hacky workaround you could just build a mock Estimator object that implements the expected fields. That may be prone to breakages though.

Thank you very much. It helps greatly! :-)


Error loading shared libraries: libopencv_core3.so.3.3

Hi,
I understand that this may not be the IFM error, but I am not able to use IFM camera because of this error-
ifm3d$ ifm3d ls
ifm3d: error while loading shared libraries: libopencv_core3.so.3.3: cannot open shared object file: No such file or directory
How can I fix this error?

I was able to fix it by following the below link -
https://github.com/cggos/dip_cvqt/issues/1


workflow for adding issues to bug board, trying a different PAT

Description
Using a new GitHub action (beta) to automatically add new issues into RN Bug Board for easier triage.

update: didn't work, needs org level read:write permissions. Will follow up offline regarding what token auto-merge uses.


Xml lower

mzml and mzxml must be lower case for wrapped tool to accept them correctly.

@bgruening Everything works; should we increment the version or just the build suffix?

Thanks @eschen42


Added MSVC build to workflow & Fixed MSVC build issues

Just some fixes for building with MSVC

Added MSVC C/C++ build to the windows workflow
Fixed assertion with range_(r)matches::iterator::operator== when comparing iterators from different containers in debug mode
Fixed incorrect /arch flags for MSVC when building stringzilla_test_cpp20_x86_*

Epic, @ashbob999! Thank you!

Ouch. I've rushed merging and now the Windows pipeline fails. @ashbob999, any chance you known why is that?

It's the iterator issue still, since you reverted it back to using begin/end and iterator debugging is enabled for release mode, it now errors.

:tada: This PR is included in version 3.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Blender plugin will happily export invalid attribute names

Name an armature bone using a character that isn't allowed in an attribute name and the exporter will happily export it (and then the file will be rejected by the library and tools).

Not fixing this. For one, I don't want to duplicate name handling in the Python code. I'd rather keep the Python code as trivial as possible as the language simply can't handle any level of complexity. Secondly, the output of the plugin is a humanly editable text file. If people really want massively long attribute names, they get to trim them down themselves.


small update for dockerfile

fun pull

@code-ga nah, i don't think that work.

you can test that


Add Bitbucket support

Add https://bitbucket.org/ support similar to how you support GitHub.

+1 I used to work an open source project on BitBucket and liked it a lot before other projects pulled me over to GitHub.

In fact, @david-a-wheeler and I have discussed offline abstracting the open source infrastructure support so that GitHub is just one implementation. That would pave the way for support of additional instances like BitBucket and SourceForge.

There's already a class to separate the repo details.  I expect we'll have a class representing access to version controlled-report, and another class representing access to the issue tracker (for example).  We've prioritized GitHub because it's so widely used, but we definitely want to add more.  Pull requests are especially welcome :-) !

FYI: Watched FLOSS Weekly this AM.

Pruning my old issues.


[Feature Request] add a option to add virtual display in scrcpy 3.0

in the new scrcpy there is a function called virtual display, please can you add it?

The virtual display function has been integrated into the Start APP. If you don’t want to start the APP, you can enable the virtual display function by clicking Home and cooperating with the Simulated Display option in the preferences.


iOS: blank screen when opening a PDF file

Hi! Here is some problem with opening pdf files, iOS shows a blank screen. If the user manages to click the "open in browser" button - the file is successfully opened in Safari, but this button disappears after a while.

Hi @sergeykorablin,
Thanks for the report! -- I am able to reproduce the bug.
Steps:

Try to click "loremipsum.pdf"
It will redirect to a blank page.
After 1 or 2 seconds, bottom-right corner icons (e.g. Open in Browser) disappear
(disabled state).
No way to display or download that pdf without clicking those icons.

Looking at the screenshot, could this be an instance of https://github.com/zulip/zulip-mobile/issues/2113?


Live Weather Location

Hey! I tried to enter my zip code(15036) and ISO code (PER/PE) to the live weather but it doesn't work  :(  It says that "An unknown error occurred" :'((
Please, can u fix this? I would like to listen at rainy music like the weather in here u.u
Thank you!!

@GineDBZ thanks for reporting this issue! We are already tracking this in #79 so we'll close this to prevent duplicates.


feat: create settings screen component

Added settingsscreen template.
Merged main into mad-screens.

This PR fails some checks. Let me know when those are resolved! Then I will start the review @Sandtory

If my code was a burning building and you were a firefighter, consider my code extinguished 🔥🚒

@Sandtory should we close this? Since Settings Screen already has been added?

yes was thinking the same, can be closed @tormodAase


AVIF Images Do Not Load - Dev Build 561

Steps to reproduce
The error occurs in the reader
Expected behavior
The image loads and reads correctly
Actual behavior
The reader does not display the image, only the black screen.
Crash logs
No response
TachiyomiSY version
TachiyomiSY preview r561
Android version
Android 8.1.0
Device
Samsung Galaxy J7
Other details
I installed the latest preview version of the App and noticed that the images in .avif format are not displaying correctly. The same does not happen in the latest stable version of the App, not even Mihon and Mihon Preview versions.
It's also not a network problem, I tested it on 3 different ones and in all of them the same happens(I also tested it on different source extensions) , the loading animation appears, it ends and the screen goes dark, no crash problems. The problem persists even when downloading the chapters.
The only constant variable seems to me to be a problem in avif or coil.
This may be the case for a different issue:
Taking advantage I would like to report that the extensions have an extremely slow performance in this version, all the extensions are slow, take at least twice as long to load and some of them seem that they won't even do it no matter how long you wait.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 1.10.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

After further investigation I found out that it is not an isolated problem of the fork, the same is true of the latest preview version of mihon, Also, after some testing, I was able to come to the conclusion that the problem only occurs in the scroll reading format (Long strip, Long strip with gaps and Webtoon reader mode) 📜: That is, I suggest those with the same problem (black screen in the chapter reader) to disable the function to detect Webtoon and change the reading mode to "paged (vertical)".
*The paged (vertical) reader mode can be a bit of a nuisance because at the end/beginning of each page(in the cap) there is a slight "dash", jump or skip, whatever you want to call it, boring experience.

If you have investigated that this is not an isolated problem with the fork, I would recommend closing this issue.


[client] add new ternoa CLI subcommands

binary renaming to ternoa-client
In addition to existing commands, prepare the following CLI. implementation of functionality is out of scope of this issue. just the CLI
encryption
ternoa-client encrypt <inputfile>
# will create inputfile.cyphertext and inputfile.aes256 with the symmetric key

NFT stuff
The NFT contains a filename of the capsule /cyphertext file. It is identified by a u32
ternoa-client nft create <owner> <filename>
# must return a u32 NFTId

# transfer ownership
ternoa-client nft transfer <from> <to> <NFTId>

# change filename
ternoa-client nft mutate <owner> <NFTId> <filename>

KeyVault stuff
ternoa-client keyvault list
# lists urls of registered enclaves, one per line

ternoa-client keyvault provision <NFTId> <urllist> <N>
# will read aes256 key, shamir-split shares, provision all keyvaults and verify
# N: number of shares needed to recover key (must be smaller than number of urls)

ternoa-client keyvault check <NFTId> <url>
# check if the key share for NFTId is stored in the keyvault with <url>. exit code 1 if negative

ternoa-client keyvault get <NFTId> <owner> <url>
# returns single key share

decryption
ternoa-client decrypt <inputfile>
# decrypts cyphertext using the aes256 key stored in inputfile.aes256. for debug only

ternoa-client decrypt <inputfile> <keysharesfile>
# reads key shares from second file, shamir-combines the shares into the original assuming the exact number of shares given that is needed

Just to be sure: All these commands will need to be implemented within the trusted section?

not sure what the difference between cli ind client is. do you mean client vs stf/cli?
encrypt and decrypt happens client side, that is for sure (no content is sent to the keyVault)
I'd suggest to do encryption and decryption in client/src/main.rs because it has nothing to do with the KeyVault directly

@brenzi is keyvault list really in the right place? It is necessary for keyvault interaction, but it does not talk with the keyvaults at all if I've understood correctly. Is it not a modified list-workers?

According to the comments in PR #16 and discussion in https://github.com/integritee-network/worker/issues/197 & looking at the main.rs file again I'm proposing the following new client structure:

to avoid restructuring the whole main.rs file (pretty messy), I'd leave it as is and add encrypt and decrypt functions. Just the clap arguments, inlcuding getting the argument from the matches. The functionality I'd implement in two dedicated, separate files:
client/decrypt.rs resp. client/encrypt.rs
to avoid duplicate code I suggest to keep the functions such as add_account_id_args for the time being but positioned in the respective clap file, resp. in the main file if it's used in more than one subcommand
for subcommands, such as keyvault nft, I'm thinking about placing them within a dedicated subcommand folder. (client/keyvault/keyvault_clap.rs). The functionalites will be implemented in separate files but the same folder (client/keyvault/keyvault_provision.rs)

Summarizing it could look like:
client
|-- main.rs (+ encrypt & decrypt clap function)
|-- decrypt.rs
|-- encrypt.rs
|-- keyvault
|    |-- clap.rs
|    |-- provision.rs
|    |-- ...
|-- nft
|    |-- clap.rs
|    |-- ...

This is not the optimal solution, but somewhat close to what was decided on in https://github.com/integritee-network/worker/issues/197 without touching main.rs too much (which is more than a PoC task..)
@brenzi does this sound like a sound solution to you?

Acutally, no... that's too much trying to hold on to the old structure which is outdated.
Different proposal: Add a new ternoa_commands.rs file, which contains all ternoa clap commands. It's handling all commands and subcommands of the newly added client calls.
The functionalities will be implemented in different files, somewhat similar to the structure above:
client
|-- main.rs
|-- ternoa_commands.rs
|-- ternoa_implementation
|    |-- encrypt.rs
|    |-- decrypt.rs
|    |-- keyvault
|    |    |-- provision.rs
|    |    |-- ...
|    |-- nft
|    |    |-- transfer.rs
|    |    |-- ...

LGTM.

Closing with merge of #16