isFormed Not Defined Crashing the RTU Gateway

The RTU Gateway has been observed to sometimes crash at chunk load. This should not be occurring due to how the peripherals are protected, but it looks like now the isFormed function can no longer be guaranteed to be defined.
Environment: Minecraft 1.21, CC:Tweaked 1.113.1, Mekanism <IP_ADDRESS>

Asked around in the Mekanism Discord:

when the multiblock is formed, CC will receive a capability invalidation and it will need to re-acquire the peripheral
since capabilities changed, it may be to do with that

isFormed would not be able to return nil, so it is likely a peripheral issue: https://github.com/mekanism/Mekanism/blob/3fea841571108c95152dc413d53e5073da90d55a/src/main/java/mekanism/common/tile/prefab/TileEntityMultiblock.java#L408-L410
Logs
E9pGwusi.log
u2URr0er.log
ymL8a03K.log

I was able to reproduce this in ATM10, logs attached to this comment.
The fix I've made worked and the RTU Gateway continued operation and properly recovered.
2024-12-20 13:01:27 [INF] ========================================
2024-12-20 13:01:27 [INF] BOOTING rtu.startup v1.10.20
2024-12-20 13:01:27 [INF] ========================================
2024-12-20 13:01:27 [DBG] RUNTIME:          ComputerCraft 1.113.1 (Minecraft 1.21.1)
2024-12-20 13:01:27 [DBG] LUA VERSION:      Lua 5.2
2024-12-20 13:01:27 [DBG] APPLICATION:      rtu
2024-12-20 13:01:27 [DBG] FIRMWARE VERSION: v1.10.20
2024-12-20 13:01:27 [DBG] COMMS VERSION:    3.0.2
2024-12-20 13:01:27 [DBG] GRAPHICS VERSION: 2.4.7
2024-12-20 13:01:27 [DBG] LOCKBOX VERSION:  1.1
2024-12-20 13:01:27 [INF] PPM: found a mekanism:boiler_casing (bottom)
2024-12-20 13:01:27 [INF] PPM: found a modem (top)
2024-12-20 13:01:27 [INF] PPM: found a boilerValve (boilerValve_0)
2024-12-20 13:01:27 [INF] PPM: found a spsPort (spsPort_0)
2024-12-20 13:01:27 [INF] PPM: found a turbineValve (turbineValve_0)
2024-12-20 13:01:27 [INF] PPM: found a dynamicValve (dynamicValve_0)
2024-12-20 13:01:27 [INF] PPM: found a modem (right)
2024-12-20 13:01:27 [DBG] boot> running sys_config()
2024-12-20 13:01:27 [INF] sys_config> device 'turbineValve_0' is not formed
2024-12-20 13:01:27 [INF] sys_config> initialized RTU unit #1: turbineValve_0 (turbine_valve) [1] for reactor 1
2024-12-20 13:01:27 [INF] sys_config> device 'spsPort_0' is not formed
2024-12-20 13:01:27 [INF] sys_config> initialized RTU unit #2: spsPort_0 (sps) for the facility
2024-12-20 13:01:27 [INF] sys_config> initialized RTU unit #3: boilerValve_0 (boiler_valve) [1] for reactor 1
2024-12-20 13:01:27 [WRN] sys_config> failed to check if 'dynamicValve_0' is a formed dynamic tank multiblock
2024-12-20 13:01:27 [INF] sys_config> initialized RTU unit #4: dynamicValve_0 (dynamic_valve) [1] for the facility
2024-12-20 13:01:27 [DBG] startup> conn watchdog started
2024-12-20 13:01:27 [DBG] startup> comms init
2024-12-20 13:01:27 [INF] startup> completed
2024-12-20 13:01:27 [DBG] main thread start
2024-12-20 13:01:27 [DBG] comms thread start
2024-12-20 13:01:27 [DBG] rtu unit thread start -> turbine_valve (turbineValve_0)
2024-12-20 13:01:27 [DBG] rtu unit thread start -> sps (spsPort_0)
2024-12-20 13:01:27 [DBG] rtu unit thread start -> boiler_valve (boilerValve_0)
2024-12-20 13:01:27 [DBG] rtu unit thread start -> dynamic_valve (dynamicValve_0)
2024-12-20 13:01:27 [INF] turbine_valve (turbineValve_0) [1] for reactor 1 is now formed
2024-12-20 13:01:27 [INF] sps (spsPort_0) for the facility is now formed
2024-12-20 13:01:27 [INF] dynamic_valve (dynamicValve_0) [1] for the facility is now formed
2024-12-20 13:01:29 [INF] supervisor connection established
2024-12-20 13:01:35 [INF] supervisor connection established
2024-12-20 13:01:35 [INF] PPM: [@dynamicValve_0] initialized previously undefined field getLength()
2024-12-20 13:01:35 [INF] PPM: [@dynamicValve_0] initialized previously undefined field getWidth()
...

rtugw-574-testing.log


using kube-spawn from a release instead of sources is awkward

There are 4 binaries in the release:

cni-noop
cnispawn
kube-spawn-runc
kube-spawn

There should be a way to install them in system directories (/bin etc.) to make it work without requiring $CNI_PATH and $GOPATH. But at the moment, kube-spawn-runc has to be installed in $GOPATH/src/github.com/kinvolk/kube-spawn/kube-spawn-runc and the CNI bridge has to be installed in $CNI_PATH.
My full steps to install kube-spawn on Fedora 27:
sudo setenforce 0
sudo dnf install -y btrfs-progs git go iptables libselinux-utils polkit qemu-img systemd-container
mkdir go
export GOPATH=$HOME/go
go get -u github.com/containernetworking/plugins/plugins/...
export CNI_PATH=$GOPATH/bin
mkdir -p $GOPATH/src/github.com/kinvolk/kube-spawn
cp kube-spawn-runc $GOPATH/src/github.com/kinvolk/kube-spawn/kube-spawn-runc
sudo -E ./kube-spawn create --nodes=3
sudo -E ./kube-spawn start
export KUBECONFIG=/var/lib/kube-spawn/default/kubeconfig
KUBERNETES_VERSION=$(curl -s https://storage.googleapis.com/kubernetes-release/release/stable.txt)
sudo curl -Lo /usr/local/bin/kubectl https://storage.googleapis.com/kubernetes-release/release/${KUBERNETES_VERSION}/bin/linux/amd64/kubectl
sudo chmod +x /usr/local/bin/kubectl
kubectl get nodes # repeat until the nodes are "Ready"

I think we should also add the CNI bridge in the release, to be installed in /usr/libexec/kube-spawn-$VER/. And kube-spawn-runc could be installed there too.


documentation about relationships between key components of Cactus architecture

Is your feature request related to a problem? Please describe.
No. But we should clarify the relationship between key components of Cactus architecture for defining interface of libraries and servers.
Describe the solution you'd like
We should describe the common understanding on relationship between key components in whitepaper. Then, we can apply re-factoring on existing codes for alignment with new architecture.

Is this considered done? can we close this issue? @sfuji822

@sfuji822 +1 ^^

Closing as stale for now, feel free to re-open anytime of course!


[1.10.2] Rendering crash with Tinkers

Issue description: Rendering error cause client to crash
What happens: If a player inserts the Personal Chest (Mekanism) into a Tinkers Crafting Table (Tinkers Construct) and leaves the crafting table interface, Shortly after the game the players client (And anyone within 250 blocks of that client) will also crash.
Steps to reproduce:

Open tinkers crafting table GUI
Insert Personal Chest
Exit GUI and wait

Version:

Minecraft: 1.10.2
Forge: 2092
Mekanism: <IP_ADDRESS>2
TConstruct: 2.5.3

Does this issue occur on a server? yes
Crash Report (Client): http://pastebin.com/NW7dtY4T

Known issue.  Has been reported before.  Disable the tabletop render in Tinker's config until it is fixed, and remove Optifine (the vast majority of mods refuse to support it).


Middle Click Link Error

When someone middle-clicks on links within the app, it opens them up in an app window with incorrect formatting and also doesn't support the Shapeshift site.
But if clicked as normal with the left mouse button, they open in the default browser as normal.

https://github.com/electron/electron/issues/2770

Fixed in https://github.com/lbryio/lbry-desktop/issues/1844


[8.x] Factory needs to guess the model name, even though it's created from one

Laravel Version: 8.x
PHP Version: 8.x
Database Driver & Version: n/a

Description:
I noticed this seemingly unnecessary complexity when setting up factories for models in nested namespaces. To make it work I had to add or override decent amount of code, even though it seems not really needed in the first place.
How it works right now:

Factory instance is created by calling ModelName::factory(), which then calls Factory::factoryForModel().
The call is made with the model class name as an argument and it is used to determine the factory class name. So far so good.
The newly created factory instance has no ties back to the original model, so it guesses the model class name using its own class name. At this point the original namespace of the model is lost, even though we had it available at the beginning.

Unless I'm missing something obvious here, $factory::new() could keep the model class name, which could be optionally overridden by static::$modelNameResolver and the $model property just as it is now. The default resolver isn't particularly useful anyway, as it just searches for the class in two default namespaces (App\Models and App).
But before I invest time into preparing a PR I would like to get a confirmation whether that's actually a bug or there is a scenario that needs to have it done this way.
Steps To Reproduce:

As above

Please resubmit this with exact versions and exact steps to reproduce. Thanks


Pimp the tf.image to the tensorboard on validset in OnlineExport(callback)

Hi tensorpack,
I would like to ask this issue:
I am using image2image.py and training with 2 datasets ds_train and ds_valid declared in
def get_config():
    logger.auto_set_dir()
    # dataset = get_data()
    ds_train, ds_valid = get_data()
    ds_train.reset_state()
    ds_valid.reset_state()
    #print ds_train.size()
    return TrainConfig(
        dataflow=ds_train,
        callbacks=[
            PeriodicTrigger(ModelSaver(), every_k_epochs=100),
            ScheduledHyperParamSetter('learning_rate', [(200, 1e-4)]),
            PeriodicTrigger(InferenceRunner(ds_valid, [ScalarStats('L1_loss')]), every_k_epochs=1),
        ],
        model=Model(),
        steps_per_epoch=ds_train.size(),
        max_epoch=4000,
    )

The OnlineExport callback (http://tensorpack.readthedocs.io/en/latest/casestudies/colorize.html#callbacks) allows me to generate the input and predicted outputs as well as the related loss values but it doesn't show up similar tf.image in the tensorboard.
Is there anyway to pimp the results of prediction on ds_valid to the tensorboard via this callback?
Best regards,

There is a plan about it but not ongoing.
Basically the goal is to let the monitor support data types other than scalar.

Also a request would be if you could allow the callback to say save images of the validation set in a folder. That would be extremely useful is one of the main issues I have with the current repository. Is there a way to hack inference runner to do this currently?

If it is just a fixed validation set, why do you need to do it during training instead of before training?
And you can do that by writing an Inferencer. No need to hack anything.

The new tensorboard has a slider below each image-summary. If tf.summary.image is called on the same input, one could inspect the result directly in the browser from different epochs.

You can now use self.trainer.monitors.put_image(name, array) to put a numpy array to tensorboard.


Implement errors package in senml transformer, readers and writers

Signed-off-by: Ivan Milošević<EMAIL_ADDRESS>What does this do?
Use mainflux errors package in senml transformer, readers and writers
Which issue(s) does this PR fix/relate to?
Related to issues #538 and #870
List any changes that modify/break current functionality
Readers' API endpoints return error message in response body for errors (not just status response code in header)
Have you included tests for your changes?
Yes

Codecov Report

Merging #1108 into master will decrease coverage by 0.05%.
The diff coverage is 26.31%.

@@            Coverage Diff             @@
##           master    #1108      +/-   ##
==========================================
- Coverage   76.96%   76.90%   -0.06%
==========================================
  Files          95       95
  Lines        6823     6786      -37
==========================================
- Hits         5251     5219      -32
+ Misses       1231     1224       -7
- Partials      341      343       +2

Impacted Files
Coverage Δ

readers/api/responses.go
100.00% <ø> (ø)

readers/cassandra/messages.go
88.46% <0.00%> (ø)

readers/influxdb/messages.go
76.92% <0.00%> (ø)

readers/mongodb/messages.go
86.44% <0.00%> (ø)

readers/postgres/messages.go
90.76% <0.00%> (ø)

writers/cassandra/messages.go
86.66% <0.00%> (ø)

writers/postgres/messages.go
58.82% <14.28%> (-2.41%)
:arrow_down:

writers/influxdb/messages.go
73.33% <20.00%> (-3.42%)
:arrow_down:

writers/mongodb/messages.go
73.33% <33.33%> (-5.24%)
:arrow_down:

transformers/senml/transformer.go
93.10% <50.00%> (-1.19%)
:arrow_down:

... and 5 more

Continue to review full report at Codecov.

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


[bug] Cannot destroy launchpad from Getting Started Guide

Describe the bug
I was following along with the Getting Started tutorial without issue. I had destroyed the 'Networking Scenario' and the 'Foundations' landingzones, no issues either. I left it in that state for a day. After a day, I did a rover login again and wanted to cleanup the launchpad landingzone. I couldn't destroy it and I am seeing the error message below.
"Authentication failure. This may be caused by either invalid account key, connection string or sas token value provided for your storage account.
landing zone already deleted"
To Reproduce
Same as above. Following the Getting Started guide and leaving it open for a day before trying to destroy the Launchpad landingzone.
Expected behavior
I expected that I would be able to destroy the Launchpad landingzone even after a day.
Screenshots

Configuration (please complete the following information):

OS and version: [e.g. Windows 10 Enterprise 19042.804]
Version of the rover: 2102.0100
Version of the landing zone (How do I check this?)

Additional context
None.

hi @jshchn this is usually due to clock skew between container and laptop... can you try either rebooting the laptop of running sudo hwclock -s in your WSL distro.

Thanks @arnaudlh, restarting my laptop did fix it! 👍


.NET Core & Identity Server 4 & nginx

I'd like to add a compose file that spins up the .NET Core sample app (the weather forecast one) with Identity Server 4's provided sample. I am are using nginx as a reverse proxy to handle all the fun ssl stuff. Took me a bit to figure it all out, would this stack be useful?

Hi @LeishaBurford , my repo nearly fulfills your requirement. It's currently a lot for improvement. But the base structure is there for docker. You can check out at my repo NetCore


Receiving notifications after being logged out

I have already left my previous company and have been logged out from my company mail(outlook) and from teams.
The issue is that I am able to see the messages in teams through notifications, but on opening the app it says I've been logged out.

@ArjunMahesh97 Could you please share a screenshot of the issue you are facing?

@ArjunMahesh97 Could you please share a screenshot of the issue you are facing?

I actually found another issue here. As soon as I open the app I can see the messages for a second before it asks me to sign in. I have attached screenshot for the notifications issue. I have a screenrecording for the second issue but not able to upload here.

The compressed screen recording.
Inked19-09-26-14-34-42.zip

@ArjunMahesh97 Thanks for sharing the video. We are looking into this.

@ArjunMahesh97 After you pressed the signout button, did you wait for the signout to complete or you killed it and reopened the app?

As I mentioned earlier, I have logged out of my company mail(The mail attached to this Teams account). I have not directly logged out using teams.

@ArjunMahesh97 Thanks for reporting this. We have raised a bug for this and we are tracking it internally.

This is resolved now. Please check.


Service not accessible without router

Hi all,
I installed zwavejsui on a docker VM (Proxmox and Debian 11 OS with docker), I've got portainer too on docker, I don't know why when I disconnect my router I can't access to the web UI, it's like the container isn't present or up... MQTT didn't received any information from zwavejsui without router and works fine with. I tried different things and I don't know. When I disconnected my router I have access to portainer wich is in the same docker system...
If someone can help me please..
Thanks.

I don't understand what you mean here... What's your LAN setup? When you say "I can't access the web UI" you mean from external or from your local LAN? How is your local LAN configured?


views of .js.es6.jsx not hot-reloading

added an initializer and added file type to react_rails_hot_loader.rb but no joy :(
Any ideas?
[HotLoader] updating: intern_badges.js.es6.jsx
[HotLoader] update finished
but the content doesn't reload.. i have to manually refresh.

Loading consists of:

Executing the JavaScript in the updated file
Calling ReactRailsUJS.mountComponents() to re-mount components in views.

This only works if the global reference to the component (eg MyApp.MyComponent) was updated by executing the JS in that file.
Is that true in this case, does intern_batches.js update the global reference to the component? If you're not sure, feel free to share the contents of the file.

Sadly, I'm experiencing the same with a .js.jsx file with only one component.
The [HotLoader] logs are shown on the console but I have to manually refresh to see the updated content.
Where can I find the intern_batches.js file to see what's going on?
Thanks in advance.

intern_badges.js.es6.jsx is a file in travisdmathis's app. It's the component that isn't reloading properly.
@kevgathuku, if you'd like to share some details, I'd be happy to debug with you. Here is some useful info you could share:

The call to react_component(...) in your Rails view, showing the component name and props
The source code of the component which doesn't update
The log messages from [HotLoader]

Thanks for clarifying @rmosolgo
The log messages are shown below:

Here's a repo with the project in question:
https://github.com/kevgathuku/rails-react-demo
The call to react_component is in https://github.com/kevgathuku/rails-react-demo/blob/master/app/views/site/index.html.erb
The component itself is: https://github.com/kevgathuku/rails-react-demo/blob/master/app/assets/javascripts/components/main.js.jsx

Hmm..that seems interesting.
I currently don't have any idea how to approach such a problem given I am new to the Ruby and Rails ecosystem, but I am familiar with JavaScript.
Any idea how I could help?

i apologize, i missed this update, and I'm no longer working on this project, it has since been completed so don't have any access to work w/ this any further.. I'm also not using react-rails anymore so..

I also encountered the same problem. Anyone known how to fix it?


BRO

Adding BRO algorithm (https://arxiv.org/pdf/2405.16158)
Description

Motivation and Context

[ ] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)

Checklist:

[ ] I've read the CONTRIBUTION guide (required)
[ ] I have updated the changelog accordingly (required).
[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly (required for a bug fix or a new feature).
[ ] I have updated the documentation accordingly.
[ ] I have reformatted the code using make format (required)
[ ] I have checked the codestyle using make check-codestyle and make lint (required)
[ ] I have ensured make pytest and make type both pass. (required)
[ ] I have checked that the documentation builds using make doc (required)

Note: You can run most of the checks using make commit-checks.
Note: we are using a maximum length of 127 characters per line

closing in favor of https://github.com/araffin/sbx/pull/60


  I think loadout.ink would be (even) better if it somehow communicated these effects in the bar charts.
Per splatoonwiki.org, the effects are

Ability
Active period
Special Charge Up
Swim Speed Up
Run Speed Up
Ink Saver (Main)
Ink Saver (Sub)
Ink Recovery Up
Quick Respawn

Opening Gambit
First 0:30

30
30

Last-Ditch Effort
Last 0:30 + OT or enemy within 30 of victory

24
24
24
24

Comeback
First 0:20 after being splatted
12
12
12
12
12
12

(All numbers are effective added ability points where main = 10 and sub = 3.)
Perhaps the added benefit could be displayed as an extension to the existing bars that has a different color?  Or maybe the page has a checkbox that lets you toggle the effect?  Just throwing some ideas out there.
If you implement this, make sure to cap the ability points at their max value of ~60, or else you'd start seeing not only diminishing returns but negative returns past a certain point. ;)
Thanks for developing and maintaining the site!

This is a planned feature. The current limitation (and why it isn't already implemented) is figuring out the best way to convey this in the UI without making it too cluttered.
Definitely something I'd like to tackle once I get back to implementing new features!


deviceready not fired with angular 9

Bug Report
Problem
What is expected to happen?
when document.addEventListener("deviceready", onDeviceReady, false); is registered the success-method (onDeviceReady) should be called.
What does actually happen?
Sucess method never gets called. Also console-log "deviceready has not fired after 5 seconds." is printed.
Information
No code changes done with update angular 8 to angular 9. Cordova platform "android" has been removed and added again but without luck.
Environment, Platform, Device
Samsung Galaxy S7, Android 9
cordova-plugin-device: 2.0.3

Is this reproducible in a new, plain Cordova app? A minimal reproduction repository would really help to debug and later fix this issue. More information on how to create one: https://github.com/apache/cordova-contribute/blob/master/create-reproduction.md

Somehow after removing platform and multiple plugins and re-add them again it worked fine! Thank you for your time and support!


Separating IMU pipeline in Unity

Required Info

Camera Model
D455

Firmware Version
<IP_ADDRESS>

Operating System & Version
Windows 11

Kernel Version (Linux Only)

Platform
PC

SDK Version
2.51.4348

Language
Unity  2022.1.21f1

Segment

Hi Teams,
I reported IMU speed issue in #11111 , and get some successful result.
Because my final target is using D455 in Unity environment, I have tested in Unity Editor.

So, according to https://support.intelrealsense.com/hc/en-us/community/posts/360047049173-Community-contribution-for-using-multiple-cameras-in-RealSense-Unity-wrapper, I do followings.

Copy the first RSDevice to second one
For the first RSDevice, Remove Gyro & Accel elements in profile list
For the second RSDevice, Only remains Gyro & Accel elements in profile list
Enter the same device serial number for both RSDevice

But when I run the scean in Unity Editor, the second RSDevice never start properly.
Unity shows the following error.
ExternalException: rs2_pipeline_start_with_config(pipe:0000015D59593550, config:0000015D59593390)
Rethrow as Exception: Couldn't resolve requests
Intel.RealSense.ErrorMarshaler.MarshalNativeToManaged (System.IntPtr pNativeData) (at <0f0f03ac46654106a5e240dad4689795>:0)
Intel.RealSense.Pipeline.Start (Intel.RealSense.Config cfg) (at <0f0f03ac46654106a5e240dad4689795>:0)
RsDevice.OnEnable () (at Assets/RealSenseSDK2.0/Scripts/RsDevice.cs:71)

How can I properly set up one physical device with two separate pipeline ?

Hi @choe-hyun-ho  You could first test whether the problem is with two RsDevices trying to access the same camera by disabling the RsDevice that has the video stream so that only the remaining RsDevice with the IMU profile starts when the project is run.
The Couldn't resolve requests error usually occurs if a stream configuration has been requested that is not currently supported by the camera at the time that the project is run.  If you have tested the same IMU profile with one RsDevice and it works then this suggests that the configuration is correct if you are using the same frequencies for Accel and Gyro on the second RsDevice.
I would recommend double-checking the frequencies on the second RsDevice.  For example, on older D455 units the default frequencies for accel and gyro may be 63 / 200, but on a newer D455 unit the defaults may be 100 / 200 - as described at https://github.com/IntelRealSense/librealsense/issues/11089#issuecomment-1311940940
Also check that the formats on the second RsDevice are set to Motion Xyz 32f for accel and gyro like in the image below.

Thanks, MartyG.
The problem is caused by incorrect framerate for each IMU unit.
I changed values to Accel=100, Gyro=200, and now it works fine.
BTW, even I separated pipeline, I feel no difference to the single pipeline case.
There are lots of the same timestamp frame for both Accelo & Gyro.
I am using the following IMU script for gathering data. Any advice?
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using Intel.RealSense;
using Intel.RealSense.Math;
using System;
using System.Threading;
using UnityEngine.Events;
using System.Runtime.InteropServices;

public class IMU : MonoBehaviour
{
    private static TextureFormat Convert(Format lrsFormat)
    {
        switch (lrsFormat)
        {
            case Format.Z16: return TextureFormat.R16;
            case Format.Disparity16: return TextureFormat.R16;
            case Format.Rgb8: return TextureFormat.RGB24;
            case Format.Rgba8: return TextureFormat.RGBA32;
            case Format.Bgra8: return TextureFormat.BGRA32;
            case Format.Y8: return TextureFormat.Alpha8;
            case Format.Y16: return TextureFormat.R16;
            case Format.Raw16: return TextureFormat.R16;
            case Format.Raw8: return TextureFormat.Alpha8;
            case Format.Disparity32: return TextureFormat.RFloat;
            case Format.Yuyv:
            case Format.Bgr8:
            case Format.Raw10:
            case Format.Xyz32f:
            case Format.Uyvy:
            case Format.MotionRaw:
            case Format.MotionXyz32f:
            case Format.GpioRaw:
            case Format.Any:
            default:
                throw new ArgumentException(string.Format("librealsense format: {0}, is not supported by Unity", lrsFormat));
        }
    }

    public RsFrameProvider Source;

    public Stream _stream;
    public Format _format;
    public int _streamIndex;

    Predicate<Frame> matcher;

    public Vector ImuData;
    public double Timestamp;

    private void Start()
    {
        Source.OnStart += OnStartStreaming;
        Source.OnStop += OnStopStreaming;
    }

    protected void OnStopStreaming()
    {
        Source.OnNewSample -= OnNewSample;
    }
    void OnNewSample(Frame frame)
    {
        try
        {
            if (frame.IsComposite)
            {
                using (var fs = frame.As<FrameSet>())
                using (var f = fs.FirstOrDefault<MotionFrame>(matcher))
                {
                    if (f != null)
                    {
                        ImuData = f.MotionData;
                        Timestamp = f.Timestamp;
//Debug.Log("X=" + ImuData.x + " Y=" + ImuData.y + " Z=" + ImuData.z);
                        f.DisposeWith(fs);
                    }
                    return;
                }
            }
            if (!matcher(frame))
                return;
            ImuData = ((MotionFrame)frame).MotionData;
            Timestamp = frame.Timestamp;
//Debug.Log("X=" + ImuData.x + " Y=" + ImuData.y + " Z=" + ImuData.z);
        }
        catch (Exception e)
        {
            Debug.LogException(e);
            // throw;
        }
    }
    public void OnStartStreaming(PipelineProfile activeProfile)
    {
        matcher = new Predicate<Frame>(Matches);
        Source.OnNewSample += OnNewSample;
    }
    private bool Matches(Frame f)
    {
        using (var p = f.Profile)
            return p.Stream == _stream && p.Format == _format && p.Index == _streamIndex;
    }
}

In the past some users of the RealSense Unity wrapper have experienced long delays when stopping IMU streaming.  This is discussed in https://github.com/IntelRealSense/librealsense/issues/5993 and https://github.com/IntelRealSense/librealsense/issues/9278

I re-tested codes with measuring frame rate.
Before separating IMU pipeline, I got 30 fps for each IMU, and after separating, I got 190~200 fps per each.
Thanks for the support, MartyG.

That's great to hear that you were successful, @choe-hyun-ho - thanks very much for the update!


OAuth2 bad gateway error

I am trying to use the oauth2 authenticator but am running into an issue.  I dont know exactly what the issue is but there seems to be a few different things that are going wrong.
First being it seems that it is calling a GET on '/api/1/auth/oauth2' which results in it just returning 'Rerouting...'  I dont know if this is the underlying issue, but it seems like it should be calling a put here.  I have tried this from firefox/chrome/IE all of which i will see the 'Rerouting...' message pop up which tells me it is sending a GET instead of a POST.
Second thing i am seeing is we are actually getting the "Whoa There" error message on lemur which after looking into the requests in the network tab i can see it is actually getting a 502 return code for Bad Gateway on the '/api/1/auth/oauth2' call.
I am not seeing any logs in the lemur.logs, so that is not helping me much as to what exactly is happening.  But I am able to see some logs in the nginx logs which seems to have two different errors in it.
[error] 12034#0: *45 upstream prematurely closed connection while reading response header from upstream, client: <ip address>, server: , request: "POST /api/1/auth/oauth2 HTTP/2.0", upstream: "http://<ip address>:<port>/api/1/auth/oauth2", host: "<hostname>", referrer: "https://<hostname>/"
and
[error] 12034#0: *37 open() "<path>/lemur/static/dist/scripts/ng-table.min.js.map" failed (2: No such file or directory), client: <ip address>, server: , request: "GET /scripts/ng-table.min.js.map HTTP/2.0", host: "<hostname>"
Any ideas what could be causing my issue of not being able to authenticate with our oauth2 provider either due to code or setup?
Note: I am aware of the incorrect use of 'PING_JWKS_URL' in auth/views.py which i have accounted for by setting this variable.

Issue was being caused by some network settings that denied the request from being completed.  Once this was resolved we were able to continue to troubleshoot different errors.
Closing issue.


Initial work on direct emission of SPIR-V

This change adds a first vertical slice of support for emitting SPIR-V code directly from the Slang IR, instead of generating it indirectly via GLSL.
This work isn't usable for anything valuable right now; the goal is just to get something checked in that we can incrementally extend over time.
When invoking slangc, the -emit-spirv-directly option can be used to turn on the new code path.
I have not bothered to add an equivalent API option, because this flag is only intended to be used for testing in the immediate future.
The existing emitEntryPoint() function has become emitEntryPointSource() to more accurately reflect its role in a world where we can also emit entry points to a binary format.
Much of the logic that was inside emitEntryPoint() had to do with linking and then optimizing/transforming Slang IR code to get it ready for emission on a particular target.
This logic has been factored into a new linkAndOptimizeIR() function that can be shared between the path that emits source and the new one that emits SPIR-V.
The meat of the change is then the emitSPIRVFromIR() function in slang-emit-spirv.cpp, which is called after all the optimizations and transformations have been applied to the Slang IR to get it ready.
Rather than repeat myself here, I will try to make the comments in slang-emit-spirv.cpp usable as documentation of the approach being taken.
Smaller notes:

I've included a test case that compares slangc output directly to expected SPIR-V. This is perhaps not an ideal plan for how to test SPIR-V emission going forward, but it suffices for now.

The external/ directory needed to be added to the include dirs for the slang project so that the new code can depend on the SPIR-V header.

In slang-ir-link, the direct SPIR-V generation path means that we now link with a target of SPIR-V instead of GLSL. In principle this can be used to ensure that appropriate variants of intrinsics are selected based on the knowledge that we are emitting SPIR-V. In practice, that isn't being used at all.

If all instructions become ids (I haven't read spec, and reading this it's not 100% clear), having a list to map ids to instructions and then just dealing with ids (as opposed to pointers) might simplify some of this.

That was more or less how I started, but then there were cases where I need to emit something and get back a SpvInst so that I can add children to it, at which point I needed the SpvInst anyway.
There's definitely a cleaner design waiting to be uncovered here, but I reached the limit of how long I wanted to spend on cleanup for code that we can't really use any time soon.


Information on "Unknown error"

Progress:  93%|███████▏          | 27/29 [00:04<00:00,  5.32part/s]Unknown error for 571-440129-4 from mouser
Unknown error for 571-440129-4 from mouser
Progress:  93%|██████▏          | 27/29 [00:05<00:00,  5.32part/s]Part ['J3#4'] not found at mouser
Part ['J3#4'] not found at mouser
Progress:  93%|███████▏          | 27/29 [00:05<00:00,  5.32part/s]No Mouser part number found!
No Mouser part number found!
Progress:  93%|███████▏          | 27/29 [00:05<00:00,  5.32part/s]No Mouser part quantity found!
No Mouser part quantity found!
Progress:  93%|███████▏          | 27/29 [00:05<00:00,  5.32part/s]No Mouser pricing information found!
No Mouser pricing information found!
Progress:  97%|██████▌     | 28/29 [00:05<00:00,  4.96part/s]mouser ['X1#4']
mouser ['X1#4']
Progress:  97%|██████▌     | 28/29 [00:05<00:00,  4.96part/s]Unknown error for M20-7820646 from mouser
Unknown error for M20-7820646 from mouser

Hi,
How can I debug what is happening when get an Unknow error for part?
Thank you

Use the debug option.
kicost --help
Will appear the call parameter --debug. Use 0 (less) to 8 (more deep debug). If still have some problem with the software, create the issue related or re-open this one.

Could you attach the BoM file? I just tested the M20-7820646 as bellow and worked fine.
test.csv.zip
test.xlsx

I forgot to mention that the error shows up for all the parts and I just pasted a part of the output.
project.xml.zip

for no reason it worked now :s

We realize that site distributors block multiple consecutive access by the same IP.
For this, you could use --throttling_delay to give a time between access to the same distributor.


Improve logging of FSharp.Formatting

With this I plan to fix the mentioned lack of logging (in #325).
I removed the shared Log.fs files and added a new FSharp.Formatting.Common project to have a single point of configuration for logging (from the outside, as otherwise we have multiple classes at runtime, one in each project).
Another nice sideeffect of this patch is that all the Fharp.Formatting libraries will no longer write into the standard output stream by default (When not explicitly configured with a ConsoleTraceListener).
Finally I changed the type of some messages to no longer appear in the console (The warning when a documentation for a member could not be found). Those can now be found in the FSharp.Formatting.svclog file.
We should start a discussion of what logfiles we want to write and which messages should be printed by default. My current configuration (Setup via FSharp.Formatting.fsx) is like this:

Console: >= Information (Just the message)
Svclog: ALL (+ALL) -> Will become quite big (several MB), but contains every bit of information
Logfile: >= Warning (+DateTime)
And all files will be deleted on startup (so only the logfile of the last run is available).
Now thinking about it, the warning-only logfile is probably not that useful.

IMHO this is a nice way to cleanup the output a bit and at the same time have a way to improve debugging.
TODOs:

[x] Switch FSharp.Formatting to TraceSource
[x] Switch Yaaf.FSharp.Scripting to TraceSource
[x] Add Logging as suggested by #325 (should simplify debugging of #328 as well)

Edit:
It seems like mono doesn't support filtering on TraceSources, this means on mono all messages will be printed to the console, that's a bit unfortunate but not a real problem IMHO.
http://lists.ximian.com/pipermail/mono-list/2012-November/049106.html
http://stackoverflow.com/questions/22590799/mono-tracesource-switch-behaviour

@dsyme Regarding your comment

It uses a secondary in-process F# interactive instance to evaluate stuff during documentation generation, in the context of an existing fsi.exe F# interactive execution of the generate.fsx script. There's definitely potential for problems or gremlins here.

It's actually not that bad, because we can access the complete "fsi.exe" output, so when something fails this output tells us exactly what has happened.
The problem is that we don't have access to the same amount of information via "FSharpChecker". We often have the problem that it works intially, but as soon as we dig into the references (which are the assemblies we want to analyse) it crashes with some internal compiler error nobody is able to decrypt. Is there a trick to access something like the "fsi" stdout?

it would be very helpful if FSharp.Formatting printed out the exact command line option set being passed to create the FSharpChecker and also to FsiEvaluationSession.

All this information is now available with this path in a logfile called "FSharp.Formatting.svclog"

FSharpChecker: Watch out for entries called "Checker Arguments: "
FsiEvaluationSession: Watch out for entries called "Starting nested fsi.exe with args: "

Very nice work, thank you @matthid! This is going to be super useful for the future of the project (I added logging to R provider and it really helped tremendously debugging things!)
I was a bit worried about adding another dll, but looking at your changes - I agree that this was the right thing to do here. I'm just releasing a new version with this, so we'll see what happens when people start using it :-). Hopefully they reference it using the fsx file these days!

Hopefully they reference it using the fsx file these days!

Exactly, I couldn't figure out a good way to set the defaults from within the code (without rendering app.config unusable).
Another note is that the logfile is really only useful when opened on windows :). I don't know how good the XmlTraceListener is on the latest mono versions, but in the past I used my own implementation to get a proper logfile on mono (https://github.com/matthid/Yaaf.Logging/blob/develop/src/source/Yaaf.Logging/SimpleXmlWriterTraceListener.fs#L56). So, if we get unusable logfiles we can either switch to raw text or use my implementation.

Sounds good! In any case, getting a good log in 50% of cases is better than in 0% of cases :-)


OCPBUGS-29303: release-4.15: revocation controller backport

Updating ose-hypershift-container image to be consistent with ART
Reconciling with https://github.com/openshift/ocp-build-data/tree/3af369afe4b44f4d17fa432026e107be365944ef/images/hypershift.yml

Merge pull request #3247 from openshift-bot/art-consistency-openshift-4.15-hypershift
OCPBUGS-19271: Updating ose-hypershift-container image to be consistent with ART
ACM-8466: Add Kubernetes SCC V2 options to HO containers
Signed-off-by: Roke Jung<EMAIL_ADDRESS>
update app-sre saas_template.yaml
Signed-off-by: Roke Jung<EMAIL_ADDRESS>
Merge pull request #3311 from openshift-cherrypick-robot/cherry-pick-3284-to-release-4.15
[release-4.15] ACM-8466: Add Kubernetes SCC V2 options to HO containers
HOSTEDCP-1273: Added support for OLM Disable default sources on HC creation
The pull request (PR) embeds the OperatorHubSpec field into the hostedcluster.Spec.Configuration API field. This allows us to disable/enable default sources in Management or Guest placement only during creation.
This functionality is also incorporated into the CLIs through a flag called .
However, it's important to note that the OperatorHub capability to disable/enable individual default sources is not covered in this PR. As a result, that field will be disregarded in the CPO and HCCO reconciliation.
Signed-off-by: Juan Manuel Parrilla Madrid<EMAIL_ADDRESS>
Update generated client code

vendor: pull in dependencies
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
api,client: add the CertificateSigningRequestApproval type
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
control-plane-pki-operator: add a label
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
control-plane-pki-operator: add a CSR flow for break-glass creds
In order to acquire break-glass credentials, end users will not have
access to the management plane from which they can read certificates.
Instead, we expect them to create (by proxy) a CertificateSigningRequest
that we will sign, if it is approved.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
api: move csra to v1beta1
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
hypershift-operator: conditionally apply cppo RBAC
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
hypershift-operator: use struct literals for clarity
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
Merge pull request #3324 from openshift-cherrypick-robot/cherry-pick-3267-to-release-4.15
[release-4.15] HOSTEDCP-1257: control-plane-pki-operator: add a CSR flow for break-glass creds
control-plane-operator: reconcile the CCO
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
control-plane-operator: remove CCO podmonitor, metrics container
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
hcco: only do the right reconciliation for the cloudcredential
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
NO-JIRA: no-op empty commit publish HO 4.15 to MCE 2.5
Signed-off-by: Roke Jung<EMAIL_ADDRESS>
Merge pull request #3338 from rokej/4.15-noop-no-jira
NO-JIRA: no-op empty commit publish HO 4.15 to MCE 2.5
Merge pull request #3336 from openshift-cherrypick-robot/cherry-pick-2794-to-release-4.15
[release-4.15] CCO-511: control-plane-operator: reconcile the CCO
Merge pull request #3321 from openshift-cherrypick-robot/cherry-pick-3320-to-release-4.15
[release-4.15] OCPBUGS-25782: Added support for OLM Disable default sources on HC creation
hostedcontrolplane: don't start pki operator when disabled
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
Merge pull request #3376 from openshift-cherrypick-robot/cherry-pick-3368-to-release-4.15
[release-4.15] OCPBUGS-26223: hostedcontrolplane: don't start pki operator when disabled
OCPBUGS-22399: Disable UWM Telemetry writer when telemeter-client cm not exists
Signed-off-by: Juan Manuel Parrilla Madrid<EMAIL_ADDRESS>
OCPBUGS-22399: Documented to disable UWM telemetry writer in disconnected envs
Signed-off-by: Juan Manuel Parrilla Madrid<EMAIL_ADDRESS>
OCPBUGS-23362: Set status on SG deletion when AWS returns error
Signed-off-by: Juan Manuel Parrilla Madrid<EMAIL_ADDRESS>
Merge pull request #3379 from openshift-cherrypick-robot/cherry-pick-3348-to-release-4.15
[release-4.15] OCPBUGS-26410: Disable UWM Telemetry writer when telemeter-client cm not exists
Merge pull request #3380 from openshift-cherrypick-robot/cherry-pick-3332-to-release-4.15
[release-4.15] OCPBUGS-26410: Disable UWM Telemetry remote writer when MGMT cluster is disconnected
external OIDC: copy Authenitcation OIDC client secrets through to guest cluster

Merge pull request #3381 from openshift-cherrypick-robot/cherry-pick-3307-to-release-4.15
[release-4.15] OCPBUGS-26412: Set new condition on SG deletion.
set KAS runtime-config in alignment with feature gates

Merge pull request #3399 from sjenning/ocpbugs-26600
[release-4.15] OCPBUGS-26600: set KAS runtime-config in alignment with feature gates
OCPBUGS-25696: Apply Scheduling Configuration for kCCM
https://issues.redhat.com/browse/OCPBUGS-25696
kCCM deployment was not respecting the node selector configured at hcp.spec.nodeSelector that was set with --node-selector flag in the create cli command.
This PR fixes the issue, by applying 'SetDefaults()' to the deploymentConfig, which sets the node selector along with other required configurations (e.g. tolerations, affinity, replica count, etc.)

availability-prober: wait for infrastructure name to be set

hcco: use manual mode for CCO
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
OCPBUGS-26757: Change KAS bootstrap and local ignition provider image to cluster-config-api

OCPBUGS-26757: Update CPO to reconcile ignition  server with cluster-config-api image

OCPBUGS-26757: Change controlplane docker file to use rhel-9 with glibc 2.34

Merge pull request #3417 from openshift-cherrypick-robot/cherry-pick-3382-to-release-4.15
[release-4.15] OCPBUGS-27071: Apply Scheduling Configuration for kCCM
Merge pull request #3423 from Patryk-Stefanski/cherry-pick-3400-to-release-4.15
[release-4.15] OCPBUGS-27155: Change KAS bootstrap image to cluster-config-api
consider HCP upgradeable if CVO has no upgradable condition

Merge pull request #3419 from openshift-cherrypick-robot/cherry-pick-3409-to-release-4.15
[release-4.15] OCPBUGS-27076: availability-prober: wait for infrastructure name to be set
Add GC knobs for KAS
Exposes GOMEMLIMIT and GOGC for kube-apiserver via annotations in
HostedCluster.

indicate cluster profile to render the correct manifests
In order to get different schemas and featuregates for hypershift, it is
necessary to indicate that we're rendering for hypershift.

Merge pull request #3393 from openshift-cherrypick-robot/cherry-pick-3373-to-release-4.15
[release-4.15] HOSTEDCP-1374: external OIDC: copy Authentication OIDC client secrets through to guest
OCPBUGS-26977: Required RBAC for network-node-identity is not created when hosted cluster networkType is set to Other.
Signed-off-by: Mario Vazquez<EMAIL_ADDRESS>
Merge pull request #3444 from openshift-cherrypick-robot/cherry-pick-3239-to-release-4.15
[release-4.15] OCPBUGS-27432: consider HCP upgradeable if CVO has no upgradable condition
api: move CSRA to a new group
We can't get a more natural directory structure for this new group using
the generators as they are - but we do get the correct API paths, etc,
so it seems like we should deal with the structure for now and fix it
when or if upstream decides to support nested groups.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
Revert "Merge pull request #3448 from stevekuznetsov/skuznets/rename-cert-gv"
This reverts commit 6c84753aab99ce494c8817ef79d9365a755b153c, reversing
changes made to 9b08dcf2ddc2d2b3f76be2216d1ed98aa5a39d6c.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
Revert "Merge pull request #3460 from stbenjam/revert-3448-1706107197301"
This reverts commit d7b8d7596f1296ae0ab7f4fe430420674b490830, reversing
changes made to 6c84753aab99ce494c8817ef79d9365a755b153c.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
api: re-add the v1beta1 CSRA
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
Merge pull request #3467 from openshift-cherrypick-robot/cherry-pick-3403-to-release-4.15
[release-4.15] OCPBUGS-28235: Required RBAC for network-node-identity is not created when hosted cluster networkType is set to Other.
Merge pull request #3465 from openshift-cherrypick-robot/cherry-pick-3443-to-release-4.15
[release-4.15] OCPBUGS-27380: indicate cluster profile to render the correct manifests
node spread anti-affinity for HA HCP

kubevirt, e2e: Add additional network test
Add a e2e at kubevirt provider with a additional network nodepool that
will check if that additional network is configured at the VMI.
Signed-off-by: Enrique Llorente<EMAIL_ADDRESS>
Remove '--attach-default-network' from productized cli
Signed-off-by: David Vossel<EMAIL_ADDRESS>
HOSTEDCP-1272: Added CLI support to create DualStack clusters using default values
This feature is only supported for now in the Agent provider.
Signed-off-by: Juan Manuel Parrilla Madrid<EMAIL_ADDRESS>
Merge pull request #3500 from davidvossel/remove-default-pod-cli-arg-4.15
[release-4.15] CNV-37394: Remove '--attach-default-network' from productized cli
Merge pull request #3499 from qinqon/release-4.15-kubevirt-add-additional-net-e2e
[release-4.15] NO-JIRA: kubevirt, e2e Add additional network test
Merge pull request #3420 from openshift-cherrypick-robot/cherry-pick-3375-to-release-4.15
[release-4.15] OCPBUGS-27103: hcco: use manual mode for CCO
Merge pull request #3457 from openshift-cherrypick-robot/cherry-pick-3384-to-release-4.15
[release-4.15] OCPBUGS-27818: Add GC knobs for KAS
manifests: Skip pod anti affinity for nil labels
At kubevirt provider we pass "nil" labels to the deployment defaults
configurator, that should skip pod anti affinity, but a bit was missing
to do so.
Signed-off-by: Enrique Llorente<EMAIL_ADDRESS>
Merge pull request #3502 from openshift-cherrypick-robot/cherry-pick-3276-to-release-4.15
[release-4.15] HOSTEDCP-1272: Added CLI support to create DualStack clusters using default values
Bubble up issues with blocked progression to the HCP Available condition
Signed-off-by: David Vossel<EMAIL_ADDRESS>
Remove validation from webhook that has ording dependency on secrets
Signed-off-by: David Vossel<EMAIL_ADDRESS>
Merge pull request #3482 from stevekuznetsov/skuznets/backport-api-moves
OCPBUGS-28594: backport API moves
Merge pull request #3515 from davidvossel/better-release-validation-415
[release-4.15] OCPBUGS-28603: Remove webhook validations that introduce resource ordering
OCPBUGS-28659: Add ValidatingAdmissionPolicy to KAS config
This plugin is in base OpenShift KAS config, but missing in HyperShift

Merge pull request #3495 from openshift-cherrypick-robot/cherry-pick-3286-to-release-4.15
[release-4.15] OCPBUGS-28764: node spread anti-affinity for HA HCP
Merge pull request #3523 from openshift-cherrypick-robot/cherry-pick-3488-to-release-4.15
[release-4.15] OCPBUGS-29029: Add ValidatingAdmissionPolicy to KAS config
hypershift-operator: expose images to the CPO
There's no real reason these days to do the self-referential lookup
logic inside of the CPO for the images it needs to know about, since the
CPO itself is part of the payload and the HO knows every single image
that the CPO needs, and can simply provide them.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
control-plane-operator: reconcile PKI operator earlier
There's no reason we can't start the PKI operator as soon as possible,
which will mean we arrive at the correct PKI state faster. This is
likely not required right now but when the controller handles all of the
PKI for the control plane it will be.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
control-plane-operator: set the namespace for the PKI operator
This is something that the operator can look up itself, but if we're
trying to emulate the controller locally, we need the flag.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
hypershift-operator: allow injecting CPO image labels
When the control-plane-operator image is injected via annotation, it is
possible to have the system use an image that resolves at the node level
and runs, but is not query-able with the normal image metadata client.
In order to not require the image to be present in some real registry,
we can furthermore allow an annoation to provide the content that
normally would be determined by resolving the LABEL data from image
metadata.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
control-plane-operator: use a synthetic image for PKI operator
There's no such thing as an image for the PKI operator, but all of the
existing mechanisms for image overrides that the other parts of the
system only work if the components that we deploy act as if they have
custom images.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
client: update generator to use metav1 applyconfigs
Previously, we were not using the correct apply configurations for
upstream types in the metav1 group/version.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
api,client: add a CertificateRevocationRequest type
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
control-plane-pki-operator: add a revocation controller
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
test: add an integration suite
This suite uses a local kind cluster and quick image builds to allow
us to iterate on controller functionality in a rapid and reproducible
manner.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
test/e2e/nodpool_test: improve conditions messages
Try to be kind to the dev breaking this test - be clear in what is being
reported, report all the information for a condition, but never
report information more than once if it has not changed.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
test/e2e/util: improve conditions messages
In an ideal world with structural typing we can write this using
generics but we're not there yet.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
test/integration: update shell script for MacOS
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
certificate revocation: make loading errors best-effort
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>

I need to re-go the hack/tools bits with golang-1.20.

/approve

/retest

/jira cherrypick OCPBUGS-29088

/lgtm
/approve


fix Print, is_empty, read_file op' cn doc test=develop

![Print](https://user-images.githubusercontent.com/48898730/65371306-fbd7b880-dc94-11e9-90b4-2740c477fc84.png)

@Xreki @wangchaochaohu 可以帮忙review下is_empty么

保持和 #1198 统一

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

Can one of the admins verify this patch?


Demo Crash

There is exception when demo try to make route and this message apper
"graphHopperLocation is not specified. call init before"
graphHopperFolder is nil please help me

graphHopperFolder is nil please help me

The "graphHopperLocation is not specified. call init before" error is in the GraphHopperTest.java#L397 file, which is part of the test suite and the iOS project doesn't call that.

At what point do you get this error?
What method are you following to get started, the main instructions or the sample app?
If possible, please provide a stack trace with the error.

When i run the sample App
in this method [_hopper load__WithNSString:location];

(jboolean)load__WithNSString:(NSString *)graphHopperFolder ;

and the stack trace

thread #7: tid = 0x3e35c, 0x017aca6b libobjc.A.dylibobjc_exception_throw, name = 'main', queue = 'com.apple.root.default-qos', stop reason = breakpoint 1.1   frame #0: 0x017aca6b libobjc.A.dylibobjc_exception_throw

frame #1: 0x000fe917 graphhopper-ios-sample-[GraphHopper load__WithNSString:](self=0x7f8570d0, _cmd=0x005eea2e, graphHopperFolder=0x00000000) + 151 at GraphHopper.m:528 frame #2: 0x000d5be7 graphhopper-ios-sample-[Directions hopper](self=0x7a7e8e40, _cmd=0x005eea8b) + 599 at Directions.m:85
frame #3: 0x000d638d graphhopper-ios-sample__19-[Directions route]_block_invoke(.block_descriptor=<unavailable>) + 829 at Directions.m:109 frame #4: 0x032f230a libdispatch.dylib_dispatch_call_block_and_release + 15
frame #5: 0x03312e2f libdispatch.dylib_dispatch_client_callout + 14 frame #6: 0x032fb10f libdispatch.dylib_dispatch_root_queue_drain + 634
frame #7: 0x032fc84a libdispatch.dylib_dispatch_worker_thread3 + 115 frame #8: 0x0366e1da libsystem_pthread.dylib_pthread_wqthread + 724
frame #9: 0x0366be2e libsystem_pthread.dylib`start_wqthread + 30

This looks like a graph data import issue. If you imported the data as described in the sample app, make sure you have all the Requirements, especially Maven. Check if the import succeeded by running ls -l graphhopper/graph-data.osm-gh/ in the repository root, which should show something like this:
$ ls -l graphhopper/graph-data.osm-gh/
total 204912
-rw-r--r--  1 calin  staff    50M Apr 20 20:57 edges
-rw-r--r--  1 calin  staff    26M Apr 20 20:57 geometry
-rw-r--r--  1 calin  staff    12M Apr 20 20:56 location_index
-rw-r--r--  1 calin  staff   2.0M Apr 20 20:57 names
-rw-r--r--  1 calin  staff    10M Apr 20 20:57 nodes
-rw-r--r--  1 calin  staff    32K Apr 20 20:57 properties

If you don't have this then most probably your import failed and you need to reimport the data.
If everything checks out, try running everything again, from scratch. Below is the sequence of commands I just ran successfully on my local machine. They are exactly like in the docs but I wrote them here for convenience. Watch out for any errors.
$ git clone --recursive https://github.com/graphhopper/graphhopper-ios.git
$ cd graphhopper-ios
$ ./graphhopper-ios-sample/import-sample.sh
$ curl -L -o j2objc-<IP_ADDRESS>.zip "https://github.com/google/j2objc/releases/download/<IP_ADDRESS>/j2objc-<IP_ADDRESS>.zip" && unzip j2objc-<IP_ADDRESS>.zip && mv j2objc-<IP_ADDRESS> j2objc && rm j2objc-<IP_ADDRESS>.zip
$ open graphhopper-ios-sample/graphhopper-ios-sample.xcodeproj
# in Xcode, Build & Run on Simulator

graph-data.osm-gh my file is graph-data.osm.pbf

graph-data.osm.pbf is the raw data file from http://download.geofabrik.de/ and if you don't have the other files it means you didn't import the data into GraphHopper. Try running the import again using ./graphhopper-ios-sample/import-sample.sh and then check if the import succeeded like described above.

i have graph-data.osm.pbf how i can make it as graph-data.osm-gh

By running ./graphhopper-ios-sample/import-sample.sh.

when i run it
Downloading http://download.geofabrik.de/europe/romania-latest.osm.pbf

If the download starts it means you don't have it in the right place. However that's fine, after the download finishes the graph data will be created automatically and you can start playing with it in Xcode. There are instructions for downloading Germany as well in the sample app.

in xcode i will import romania-latest.osm.pbf file?

Yes, by default the import script imports romania-latest.osm.pbf, so you can only create routes within Romania. This is just an example, you can follow the instructions on the sample app to import any country you want.

i found the problem in marven ,did it take time when convert
Thanks

i still have problem but from marven Now
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] GraphHopper ........................................ FAILURE [03:54 min]
[INFO] GraphHopper Tools .................................. SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 04:36 min
[INFO] Finished at: 2015-04-21T22:59:56+02:00
[INFO] Final Memory: 15M/244M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-resources-plugin:2.6:resources (default-resources) on project graphhopper: Execution default-resources of goal org.apache.maven.plugins:maven-resources-plugin:2.6:resources failed: Plugin org.apache.maven.plugins:maven-resources-plugin:2.6 or one of its dependencies could not be resolved: The following artifacts could not be resolved: org.apache.maven.doxia:doxia-sink-api:jar:1.0-alpha-7, org.apache.maven:maven-error-diagnostics:jar:2.0.6: Could not transfer artifact org.apache.maven.doxia:doxia-sink-api:jar:1.0-alpha-7 from/to central (https://repo.maven.apache.org/maven2): Remote host closed connection during handshake: SSL peer shut down incorrectly -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginResolutionException

I'm not familiar with Maven, it is used by GraphHopper as its build system. Try google-ing the errors to see if you can find something useful. This popped up on a very quick search, not sure if it will be of any help.

http://stackoverflow.com/a/17802240/1104534

As this error is not related to graphhopper-ios but core graphhopper maybe @karussell would know why Maven is complaining.

thanks Again I try in it

It looks, like some HTTPS/host problems with maven central. Maybe you just try it again? If that does not help try what was suggested in @clns its link: rm -rf <home>/.m2/repository or even rm -rf <home>/.m2/

How was this fixed? @dimohamdy

I install Maven that use to convert  graph-data.osm.pbf to graph-data.osm-gh


ContextBridge not updating without calling root.render()

Without calling root.render(), the ContextBridge doesn't update.
In the example below, the context in the Canvas does not update, the number stays 42.
Do you know any workaround for this?
const NumberContext = createContext<number | undefined>(undefined);

function Number() {
  const number = useContext(NumberContext);

  return
number: {number ?? "none"}
;
}

function Canvas({ Component }: { Component: any }) {
  const [el, setEl] = useState<HTMLDivElement>();
  const [root, setRoot] = useState<Root>();
  const ContextBridge = useContextBridge();

  useEffect(() => {
    if (!el) return;
    const root = createRoot(el);
    setRoot(root);
    return () => root.unmount();
  }, [el]);

  useEffect(() => {
    if (!root) return;
    root.render(<ContextBridge><Component/></ContextBridge>);
  }, [root, Component]);

  return
 el && setEl(el)}>
;
}

function App() {
  const [number, setNumber] = useState(42);
  return (
    <NumberContext.Provider value={number}>
      <button onClick={() => setNumber((n) => n + 1)}>increment</button>
      <Number />
      <Canvas Component={Number}></Canvas>
    </NumberContext.Provider>
  );
}

Yes, you should have roots render alongside its parent component as a render-effect. This ensures that effect timings and context stays in sync.
function CustomCanvas({ children }) {
  // Init root from canvas
  const [canvas, setCanvas] = React.useState()
  const root = React.useMemo(() => canvas && createRoot(canvas, { events }), [canvas])
  // Render children as a render-effect
  root?.render(children)
  // Cleanup on unmount
  React.useEffect(() => () => root?.unmount(), [root])
  // Use callback-style ref to access canvas in render
  return <canvas ref={setCanvas} />
}

Alright, that makes sense. Due to the nature of my library (React in Angular in React), there's no direct render connection between the useContextBridge and the usage of ContextBridge, but I've solved it by passing the render calls through a side channel (render$).
const render$ = new Subject<void>();

function Canvas({ Component }: { Component: any }) {
  const [el, setEl] = useState<HTMLDivElement>();
  const [root, setRoot] = useState<Root>();
  const ContextBridge = useContextBridge();

  render$.next();

  useEffect(() => {
    if (!el) return;
    const root = createRoot(el);
    setRoot(root);
    return () => root.unmount();
  }, [el]);

  const render = useCallback(() => root?.render(<ContextBridge><Component/></ContextBridge>), [root, Component]);

  useEffect(() => {
    render();
  }, [render]);

  useEffect(() => {
     const subscription = render$.subscribe(render);
     return () => subscription.unsubscribe();
  });

  return
 el && setEl(el)}>
;
}

function App() {
  const [number, setNumber] = useState(42);
  return (
    <NumberContext.Provider value={number}>
      <button onClick={() => setNumber((n) => n + 1)}>increment</button>
      <Number />
      <Canvas Component={Number}></Canvas>
    </NumberContext.Provider>
  );
}

By using a render-effect, React shows the following warning:
Render methods should be a pure function of props and state; triggering nested component updates from render is not allowed.

I've solved this by wrapping it in a dependency-less useLayoutEffect, but I'm not entirely sure why this warning is showing.

For future reference, here's how I solved it:
Where the ContextBridge is created and the side channel is called:
https://github.com/bubblydoo/angular-react/blob/91ea6139b635ab76d9dd755c374f21c020c5fa3c/projects/angular-react/src/lib/angular-module-context/angular-module-context.tsx#L31-L53
Where the side channel calls render():
https://github.com/bubblydoo/angular-react/blob/91ea6139b635ab76d9dd755c374f21c020c5fa3c/projects/angular-react/src/lib/react-wrapper/react-wrapper.component.tsx#L35-L39
and
https://github.com/bubblydoo/angular-react/blob/91ea6139b635ab76d9dd755c374f21c020c5fa3c/projects/angular-react/src/lib/react-wrapper/react-wrapper.component.tsx#L71-L79
Final result: https://angular-react-fwkh0xlns-bubblydoo.vercel.app/?path=/story/basics-context-bridging--showcase

For future reference, in the end I ended up not using its-fine. Because I only have 1 renderer, I finally came up with a solution that uses portals and implements the same api as ReactDOM.createRoot (with render and unmount).
I called it useInTreeCreateRoot:
https://github.com/bubblydoo/angular-react/blob/e0f370a7ea3563b1d92530aad207575c086ba3ef/projects/angular-react/src/lib/use-in-tree-create-root/use-in-tree-create-root.ts


Make better links in CONTRIBUTING

Would like CONTRIBUTING on GitHub to link to ReadTheDocs where needed, but when included in docs/contributing.rst these should change to local links so that they go to the right version (or testing, or whatever) within rtd. (Otherwise they work as "external" links to the default rtd version.)

Dunno if there's a way to do substitution when including a file in reST (or especially when including a parsed Markdown file). Maybe able to check Myst for extensions?


Update personal-access-tokens.md

remove ">", which is accidentally inserted in "?>api-version=5.0".

@wangyoutian : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Error [ERR_REQUIRE_ESM] with Node.js 13

I created new project using ESM, since Node 13.2+ supports it without any flags.
Instead of using .mjs extension I added "type: module" to package.json.
With this setup "tap" throws an error, when I try to run "tap test/test.js":
Error [ERR_REQUIRE_ESM]: Must use import to load ES Module
require() of ES modules is not supported.
require() of /index.js from /test/test.js
even though I only use imports everywhere and in test.js import of index.js looks like this:
import {fastify} from '../index.js';
If I delete "type: module" & change extensions to ".mjs" everything works just fine.
Is it Node.js or "tap" issue?
Node version: 13.10.1
npm: 6.13.7
tap: 14.10.6

Argh, ok, that's known problem, discussed here:
https://github.com/standard-things/esm/issues/868

Will be fixed by #651
For now just manually disable ESM.

Note the PR you referenced will not be part of tap v14, that is going into tap v15 (it is a breaking change).  Does manually disabling user-land ESM solve the issue for you?

Yeah, I see that.
Adding --no-esm fixes the issue => it's an issue with "esm" package, probably it doesn't support "type: module" field...

I think it's that the esm module doesn't really support .js files the same way as it supports .mjs files.  Setting type: module but still using .mjs would work without disabling the esm package.
On a more important note based on what you said --no-esm is the right choice for your tests to run correctly.  You want your tests to load your code via the node.js native ES module loader, the esm package bypasses this.  So absolutely I recommend setting --no-esm for any module which provides type: module or has production code with .mjs extension.

I think I have a workaround, at least it seems to be working for me:
Just put a package.json to your test root, it can be an empty json document like {}.
I have "type": "module" set for the project, and I can use tap --coverage for testing. AFAIK esm traverses the project tree until first package.json is found. and this way it is getting tricked, as it finds the one without "type": "module" first.

I'm seeing this error with node-tap v15, running tests written in TypeScript. My test includes the following import:
// index.test.ts
import { Geometry } from 'three/examples/jsm/deprecated/Geometry';

... which is a .js file (with type definitions) annotated with {type: 'module'}. The error below is printed:
$ tap test/*.test.ts --ts | tap-spec
Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /Users/donmccurdy/Documents/Projects/three-to-cannon/node_modules/three/examples/jsm/deprecated/Geometry.js
require() of ES modules is not supported.
require() of /Users/donmccurdy/Documents/Projects/three-to-cannon/node_modules/three/examples/jsm/deprecated/Geometry.js from /Users/donmccurdy/Documents/Projects/three-to-cannon/test/index.test.ts is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename Geometry.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from /Users/donmccurdy/Documents/Projects/three-to-cannon/node_modules/three/examples/jsm/package.json.

Notably, I'm not explicitly writing require() anywhere, I guess that's output of the compiled TS test file? And the Geometry.js file is both outside my project and intended for use in browsers, so .cjs is not an option. It sounds like the --no-esm option no longer exists, is something else required for TS?

tap v15.0.9
node v14.15.4


switch to GA4 format

universal analytics are being depracated, plus it is much more similar to our other analytics
To Test:

[x] test in web + android
[x] user properties are set
[x] events are sent/received with all data
[ ] on user_id/user properties are handled correctly on GA4 check with @drptiago

Not all user properties are set: isLoggedIn is always undefined

We are handling the events / properties and user_id correctly.
Just one observation: In Good Dapp the info is passing through GTM.
In the Wallet the infos are sent directly to Google Analytics.  Maybe we want to send directly in Good Dapp too?

@drptiago what difference does it make?

No difference in the final data.
But maybe can be better to you guys keep the same process in both @sirpy


Add License file & License name in readme.md

Hi,
Your module had a license in composer.json
https://github.com/robertotru/ToInlineStyleEmailBundle/blob/master/composer.json
but it was not visible in the readme.
In order to avoid confusion, I added a license file and I mentioned the type of license chosen for your code in the readme
Regards,
Tomap

@Richtermeister lgtm


error 'google' is not defined no-undef

This doc: https://tomchentw.github.io/react-google-maps/basics/directions isn't working at all.
can't get around:  error 'google' is not defined no-undef
I did find a solution that is partly working, e.g. using
  return (
       <GoogleMapLoader
               containerElement={mapContainer}
               googleMapElement={
                   <GoogleMap
                       defaultZoom={12}
                       defaultCenter={this.props.center}
                       options={{ streetviewcontrol: false, mapTypeControl: true}}>
                   </GoogleMap>
               }

But I have no idea how to match this to the tutorial  with componentDidMount etc. to get it to work with the direction service. Any help is greatly appreciated.

I am one step further, see this question: https://github.com/tomchentw/react-google-maps/issues/435

@jwamsterdam - Take a look at #414

Please refer to Getting Help section in the README (or #469).

add this line to top of your file: /global google/ -> for disable ESLint

What about putting it in your tsconfig?


Fixes unused MTU settings at TcpSocket dispatch

This commit fixes a small bug, where the TcpSocket computed the maximum
size per payload in a tx_buffer without taking into account the MTU
defined by the underlying network device.

Thanks @mustermeiszer for the fix and @Dirbaio for the review!


Update Travis to only build master branch

What was a problem?
Travis will build all branches with toggle on, no branches including master if off. We want master and PR.
How this PR fixes the problem?
Limits branch building to master.
Check lists (check x in [ ] of list items)

[ ] Test passed
[X] Coding style (indentation, etc)

Additional Comments (if any)
Tests unknown, changed in github ui.

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   59.64%   59.64%
=======================================
  Files           2        2
  Lines         171      171
=======================================
  Hits          102      102
  Misses         59       59
  Partials       10       10

Continue to review full report at Codecov.

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


NTRs: new types of manual assertions

Hi,
As mentioned in https://github.com/evidenceontology/evidenceontology/issues/307, we are developing a new literature system for the Alliance of Genome Resources and as part of that work, we would like to be able to track which humans make manual assertions about entities and data types mentioned and studied in papers.
Do you think it is within scope of ECO to create child terms of manual assertion to capture roles such as author or biocurator?
For example:
assertion method (ECO:0000217)

manual assertion (ECO:0000218)

author assertion (ECO:new)

biocurator assertion (ECO:new)

Definitions of the new terms might be:
—author assertion (ECO:new)
Proposed definition: A type of manual assertion made by an author of a publication.
Synonym (RELATED): researcher assertion
—biocurator assertion (ECO:new)
Proposed definition: A type of manual assertion made by a biocurator.
Reference: https://en.wikipedia.org/wiki/Biocuration
Synonym (RELATED): curator assertion
While looking over ECO and the possible placement of terms using these assertion methods, we realized that the potential parent, 'author statement' (ECO:0000204), might cause truth path violations, since the definition of that term says:
"A type of documented statement evidence that is based on an assertion by the author of a paper, which is read by a curator."
but if an author flags a paper for entities and/or data types, then the 'read by a curator' part would not apply.
We realize this proposal may also have implications for groups like GO, who use child terms as evidence codes for annotation.
Let us know what you think!
Many thanks,
--Kimberly
@cerivs @valearna @azurebrd

Just checking to see if this has been discussed or if we need to add more information to the submission.

@cerivs
Please see https://github.com/evidenceontology/evidenceontology/issues/307#issuecomment-1615049588

Apologies on the delay for these.
Haven't forgotten you and will be addressing them soon.

Thanks @mgiglio99
Let us know if you'd like to set up a time to chat about them.
We'd be happy to explain more about what we're trying to model and why we'd like the new codes, and get your feedback.

@vanaukenk
If you get a chance please let us know your thoughts on Michelle's edits in her latest post. I am planning to include these in our release tomorrow.
Thanks,
Suvvi

Thanks for checking @nsuvarnaiari
We discussed this at our Tuesday Alliance literature meeting (8/29) and weren't sure we entirely understood the proposal, so I think it's best if we keep this ticket open and continue discussion before adding the terms above.
I will try to put together our questions as soon as possible to move this along.

Thanks for a quick response @vanaukenk. Sure, will keep this ticket open for further discussion.
Have a nice long weekend
Suvvi

@nsuvarnaiari @mgiglio99
I think that for our manual data type and entity flagging pipelines, we thought all flagging would be based on author statement evidence but we wanted to capture who made the manual assertion in our curation tools: initially, either the author or the curator.
Something like this:
-author statement used in manual assertion (GO:0000302)
--author statement used in manual assertion by author (GO:new)
--author statement used in manual assertion by curator (GO:new)
Does this make sense?  I am also wondering if there are broader implications for this proposal wrt GO evidence codes, though.
Perhaps we are overthinking it, too :-)
Let me know if you'd like to have another in-person meeting.  Thanks.


Increased compilation time

see https://www.devitoproject.org/devito-performance/#codegen.TTI.time_forward

fixed by #1487


fixes bug 1259146 - call it "All Contributors" instead

Coverage remained the same at 84.784% when pulling 272d3eea61d92b4e535bf23b03a8ec8f66a59012 on peterbe:bug-1259146-call-it-all-contributors-instead into 55842de0e9b34e095a1342ec85d8d62d92307922 on mozilla:master.


[SPARK-22252][SQL][2.2] FileFormatWriter should respect the input query schema

What changes were proposed in this pull request?
https://github.com/apache/spark/pull/18386 fixes SPARK-21165 but breaks SPARK-22252. This PR reverts https://github.com/apache/spark/pull/18386 and picks the patch from https://github.com/apache/spark/pull/19483 to fix SPARK-21165.
How was this patch tested?
new regression test

cc @gatorsmile

Test build #82684 has started for PR 19484 at commit 1a15cb2.

The PR description needs a correction.

Test build #82684 has finished for PR 19484 at commit 1a15cb2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82684/
Test FAILed.

Test build #82711 has started for PR 19484 at commit e527540.

Test build #82711 has finished for PR 19484 at commit e527540.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82711/
Test PASSed.

Thanks! Merged to 2.2. Could you close this?


Fix closeness tests in testOnComplexPlane for small inputs. Update expm1 inaccuracy regions.

As in the title.
Requires https://github.com/openxla/xla/pull/10376 that fixes all inaccuracy issues in jax.numpy.expm1:

before this PR the inaccuracies and mismatch rate was 17.7%
after this PR the inaccuracies rate is 0.1%

There are some failures in one of the CI jobs – looks like inf/nan mismatches.

There are some failures in one of the CI jobs – looks like inf/nan mismatches.

Yes, I am in a process of fixing these. These emerged because the closeness checking is changed in this PR and the mapping of inaccuracy lists requires updates re the tests that passed false-positively before.

Tests are failing on internal CPU.
Overall, I think the issue here is that these tests are too brittle: they make detailed assertions about the dark corners of hardware-dependent implementation details. Rather than add more exceptions for particular backends / package versions / devices / etc. I think it would be better to make the tests simpler and less brittle. What do you think?

xla_extension_version >= 244 needs to be updated as the current xla_extension_version value conflicts with the one in the main branch, hence the test failures.

... I think it would be better to make the tests simpler and less brittle.

I agree. I suggest replacing numpy with mpmath as a reference library.

Created https://github.com/google/jax/pull/20373 to resolve confusion in import/copybara.

Closing as https://github.com/google/jax/pull/20373 has landed.
The items in the description will be addressed in follow-up PRs.

@pearu, are the XLA PRs that address the inaccuracies ready for review? That is, are the following PRs ready for review?

https://github.com/openxla/xla/pull/8589
https://github.com/openxla/xla/pull/10376
https://github.com/openxla/xla/pull/10503
https://github.com/openxla/xla/pull/10525


[20.03] mailcather is broken since default ruby is 2.6

mailcather requires ruby-2.5 and it is broken since default pkgs.ruby bumped to 2.6 in nixpkgs-20.03
it seems tricky to fix because the ruby infrastructure (bundlerApp and friends) does not have a straightforward way to replace ruby version for a particular derivation.
I started to fix and it quickly evolved to big refactoring which is not acceptable to do when there is only few days before releasing nixpkgs-20.03

(in my branch I just reverted default ruby to 2.5)

How unfortunate... it seems like 20.03 is suffering from a number of ruby related issues ☹
@zarelit @nicknovitski any insight?
cc @manveru @thefloweringash @mawis

@aanderse @volth I just tested the build and the service with nixos-shell against release-20.03 and it looks like it works as expected and it is based off ruby 2.6 as well. I think it has been fixed in another issue

Interesting, here I have failed build with
these derivations will be built:
  /nix/store/vkffapci6ig30qiignhw3x2z1237zi9f-ruby2.6.5-mailcatcher-0.7.1.drv
  /nix/store/kkhi4vdrz6rmgdjwxnwwy9y1hdkh1ip8-mailcatcher-0.7.1.drv
  /nix/store/j4hdqy3gphnvad6w6jibb0bi2xyij1js-mailcatcher-0.7.1.drv
building '/nix/store/vkffapci6ig30qiignhw3x2z1237zi9f-ruby2.6.5-mailcatcher-0.7.1.drv'...
unpacking sources
patching sources
configuring
no configure script, doing nothing
installing
buildFlags:
WARNING:  You build with buildroot.
  Build root: /
  Bin dir: /nix/store/1fjlrfr3wc1ji2b0wc2dnlnhipzwmmyd-ruby2.6.5-mailcatcher-0.7.1/lib/ruby/gems/2.6.0/bin
  Gem home: /nix/store/1fjlrfr3wc1ji2b0wc2dnlnhipzwmmyd-ruby2.6.5-mailcatcher-0.7.1/lib/ruby/gems/2.6.0
Successfully installed mailcatcher-0.7.1
1 gem installed
/nix/store/1fjlrfr3wc1ji2b0wc2dnlnhipzwmmyd-ruby2.6.5-mailcatcher-0.7.1/lib/ruby/gems/2.6.0 /build
removed 'cache/mailcatcher-0.7.1.gem'
removed directory 'cache'
/build
cp: cannot create regular file '/nix/store/1fjlrfr3wc1ji2b0wc2dnlnhipzwmmyd-ruby2.6.5-mailcatcher-0.7.1/lib/ruby/gems/2.5.0/gems/mailcatcher-0.7.1/lib/mail_catcher/web/application.rb': No such file or directory
note: keeping build directory '/tmp/nix-build-ruby2.6.5-mailcatcher-0.7.1.drv-0'
builder for '/nix/store/vkffapci6ig30qiignhw3x2z1237zi9f-ruby2.6.5-mailcatcher-0.7.1.drv' failed with exit code 1
cannot build derivation '/nix/store/kkhi4vdrz6rmgdjwxnwwy9y1hdkh1ip8-mailcatcher-0.7.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/j4hdqy3gphnvad6w6jibb0bi2xyij1js-mailcatcher-0.7.1.drv': 1 dependencies couldn't be built
error: build of '/nix/store/j4hdqy3gphnvad6w6jibb0bi2xyij1js-mailcatcher-0.7.1.drv' failed


Error when compiling on WSL

Hello, here's the error:
LINK chronos
/usr/bin/ld: error: LLVM gold plugin has failed to create LTO module: Unknown attribute kind (70) (Producer: 'LLVM13.0.0-rust-1.57.0-stable' Reader: 'LLVM 10.0.0')
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [Makefile:35: chronos] Error 1

This sounds more like an issue with your build environment.
Can you give me information about what distro you are using under WSL.

Alright so I'm thinking -flto is causing problems for your specific machine. You should edit the first line of the Makefile and remove that flag, and optionally switch to using gcc instead of clang.

It was indeed the -flto option, works now even when compiling with clang.
Thanks :)


Track date last updated in compound groups

In objects and in specifications. Spreadsheet will be trustee source of dates.

Hey Akos,
I'm not sure if this is the right sort of date but it is part of the Compound record. It's called the 'release date' but not sure what that means. cid 2323 is just an example:
s=pcp.Compound.from_cid(2323)
for rec in s.to_dict()['record']['props']:
    print rec['urn']['release']

Code returns:
2011.04.04
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13

Very cool find. I agree that it's not obvious that what this means. It
would be good to check the (PUG REST API docs)[
https://pubchem.ncbi.nlm.nih.gov/pug_rest/PUG_REST.html] and if still
unclear, to email PubChem support to find out how to get the date a
Compound was created.
On Saturday, June 25, 2016, rashupee<EMAIL_ADDRESS>wrote:

Hey Akos,
I'm not sure if this is the right sort of date but it is part of the
Compound record. It's called the 'release date' but not sure what that
means. cid 2323 is just an example:
s=pcp.Compound.from_cid(2323)for rec in s.to_dict()['record']['props']:
print rec['urn']['release']
Code returns:
2011.04.04
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
2011.09.13
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/akokai/metacamel/issues/10#issuecomment-228553097,
or mute the thread
https://github.com/notifications/unsubscribe/ABx3pZJo9S3tyTj8N56yuiRB3eWdaoWNks5qPVMxgaJpZM4I-NIv
.

--
Akos Kokai http://kaios.net/
PhD candidate, Department of Environmental Science, Policy & Management
http://ourenvironment.berkeley.edu/
Fellow, Berkeley Center for Green Chemistry http://bcgc.berkeley.edu/
University of California, Berkeley

Hey, so I sent an email asking about the creation date associated with cids. No response yet.
For the same cid as above (2323) I used requests and hit PubChem with its rest client thing-er:
import requests
r = requests.get('http://pubchem.ncbi.nlm.nih.gov/rest/pug/compound/cid/2323/dates/JSON')
r.json()

returns
{u'InformationList': {u'Information': [{u'CID': 2323,
    u'CreationDate': {u'Day': 25, u'Month': 3, u'Year': 2005}}]}}

which is really different from the strange 'release' dates found in the Compound record for the same cid in the previous message:
2011.04.04
2011.09.13

The puchem docs describe that the above rest-style way of doing things is good for small batches of cids - like 5 per second. It also describes that for cids only the creation date is kept. I don't think they keep track of update or modification dates... but I asked them in my email. We'll see what they say.

Confirmed from the API docs [1] that you can use https://pubchem.ncbi.nlm.nih.gov/rest/pug/compound/cid/<CID>/dates/JSON and get r.json()['InformationList']['Information']['CreationDate'] to get the date "when a CID first appeared." Yay!
[1] https://pubchem.ncbi.nlm.nih.gov/pug_rest/PUG_REST.html#_Toc409516779


NativeBase Item floatingLabel doesntwork with redux form

Hi!
I'm try to use NativeBase Item floatingLabel with redux form, but this two together doesnt work.
Replacing floatingLabel by stackedLabel or View everything works fine.
React: 15.4.2
react-native: 0.42.3
native-base: "2.1.0
redux-form: 6.7.0
const validate = values => {

    let errors = {};

    errors.name            = nameValidator(values.name);

    return errors

};

const nameValidator = name=> {

     let error;
    switch (true) {

        case !name:
            error = 'Required';
            break;
        default:
            let names= name.split(' ');
            for (let i =0; i < names.length; i ++) {
                if (names[i].length < 3) {
                    return error = 'a message'
                }
            }

            if (names.length < 2) {
                error = 'some message'
            }
            break;

    }

    return error;

};

const FormRegister = (props) => {

    return (
        <ScrollView  keyboardShouldPersistTaps={'handled'}>
            <Field
                label="Na"
                name={'name'}
                icon={"contact"}
                // format={Formatters.nameFormatter}
                // parse={Parsers.nameParser}
                component={FormInputField}
            />

        </ScrollView>
    );
};

export default reduxForm({
    form: 'register',
    validate
})(FormRegister);

const FormInputField = (props) => {

    const {icon, input, keyboardType, label, meta, secureTextEntry, ...inputProps} = props;
    const isErro                                                                   = !meta.valid && meta.touched && !meta.active;
    console.log("isErro", isErro);

    return (
        <View >
            <View >
                {icon && <Icon active name={icon}/>}
                {/*<View style={styles.inputContainer}>*/}
                <Item floatingLabel >
                    {/*<Item stackedLabel >*/}
                    <Label >{label}</Label>
                    <TextInput {...inputProps}
                               onFocus={input.onFocus}
                               onBlur={input.onBlur}
                               style={styles.input}
                               secureTextEntry={secureTextEntry}
                               keyboardType={keyboardType}
                               onChangeText={input.onChange}
                               value={input.value}/>
                </Item>
                {/*</View>*/}
            </View>
            {isErro && <Text >{meta.error}</Text>}
        </View>
    );

};

#708

That seems to be a whole different issue altogether. How is this problem solved by the ticket you linked to?
The real issue is in this line here
https://github.com/GeekyAnts/NativeBase/blob/master/src/basic/Item.js#L129
that initializes a variable 'inputProps' - but only for <Input> child elements. Since redux-form doesn't use <Input> child components, but <Field> components, as helpfully pointed out in NativeBase's own documentation here, this variable isn't initialized. That means that any 'onChangeText' handler that might be necessary for redux-form to work doesn't fire, which in return causes floatingLabel to not work redux-form, as indicated in the title of this bug report. Reopen?!


Add PIN_APA102 and PIN_LED support to 2nd stage bootloader

Added support in the 2nd stage bootloader for the  APA102 indicator and conventional LED,  Both will illuminate during the detection time (looking for UF2 button)

r


Fixes #202

Replace Packet mapper and handlers with methods on InterledgerResponsePacket.
Signed-off-by: David Fuelling<EMAIL_ADDRESS>

Codecov Report

Merging #225 into master will increase coverage by 0.02%.
The diff coverage is 73.33%.

@@             Coverage Diff              @@
##             master     #225      +/-   ##
============================================
+ Coverage     74.29%   74.32%   +0.02%
+ Complexity      390      388       -2
============================================
  Files           112      110       -2
  Lines          1743     1741       -2
  Branches        154      154
============================================
- Hits           1295     1294       -1
+ Misses          380      379       -1
  Partials         68       68

Impacted Files
Coverage Δ
Complexity Δ

...rg/interledger/core/InterledgerResponsePacket.java
55% <73.33%> (+55%)
4 <4> (+4)
:arrow_up:

...org/interledger/core/InterledgerFulfillPacket.java
33.33% <0%> (-16.67%)
1% <0%> (ø)

Continue to review full report at Codecov.

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

I still need to make this adjustment in IldcpResponsePacketMapper and BtpPacketMapper

@sudheesh001 I think this is ready for final review.


OPENFED-20: Fix encoding on file entity

Fixes:

Apply patch to fix wrong encoding when using Download link formatter plugin (see https://www.drupal.org/project/file_entity/issues/3277484)

Improvements:

Move file_entity related configurations to new submodule openfed_file_entity
Remove file_entity dependency from openfed default configurations and openfed.info.yml
Re-adapt hook openfed_update_101204()

Test with a fresh installation of OpenFed 12:

Fresh install OpenFed 12
Make sure the file_entity module is not enabled by default
Try to install openfed_file_entity and make sure the file_entity module is installed along with the same configurations that were moved from the openfed config/install folder into the new submodule configurations openfed_file_entity/config/optional
Make sure the roles authenticated, anonymous have the "view files" permission by default
Make sure the role content_editor has the "create files" permission by default
Try to create a new media type with a file field and use the display formatter "Download link"
Make sure you create a media field in one of your content types and allow the media type created previously
Create a new page and upload a file with special characters in the filename in the media field
When viewing the page, make sure the download link for the file is encoded correctly

Test with an existing OpenFed site:

Run composer install
Run drush updb
Make sure the hook openfed_update_101204() was successfully executed (test issue #39 to make sure it's working)
Try to create a new media type with a file field and use the display formatter "Download link"
Make sure you create a media field in one of your content types and allow the media type created previously
Create a new page and upload a file with special characters in the filename in the media field
When viewing the page, make sure the download link for the file is encoded correctly

Good work, all working well


Is it possible to have a column always hidden?

Hi,
I understand that the attribute ' data-priority="-1" ' would hide the column and not show it on the display dropdown. However, if I click on display all, the column is shown.
I don't want to show the column.
Thanks in advance.

Fixed 👍 Thank you for reporting!


No way to access ccmr1_input

The type is defined, but the RegisterBlock only defines ccmr1_output and there is no obvious way to convert between the two.

On stm32f401 to be specific, but I don't see a way to access it on other CPUs.

This is really an svd2rust issue (https://github.com/rust-embedded/svd2rust/issues/16) which we can't solve here.
You can use the .bits() method on CCMR1_OUTPUT to directly read/write the equivalent CCMR1_Input fields though it's inelegant.


Display all golangci-lint errors sorted with filename and line number

Description
Jonathan reported golangci-lint messages did not display the line number in the Github Action.  This patch adds consistent output configuration options between "make pre-commit" and Github Actions
Fixes # (issue)
Type of change
What is it?

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[X] I run make pre-commit to check fmt/vet/lint/test-no-fdo

Codecov Report
Base: 36.89% // Head: 36.33% // Decreases project coverage by -0.56% :warning:

Coverage data is based on head (ac46a0e) compared to base (58b7295).
Patch has no changes to coverable lines.

:exclamation: Current head ac46a0e differs from pull request most recent head eb765e3. Consider uploading reports for the commit eb765e3 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1587      +/-   ##
==========================================
- Coverage   36.89%   36.33%   -0.57%
==========================================
  Files          67       66       -1
  Lines        9009     8818     -191
==========================================
- Hits         3324     3204     -120
+ Misses       5277     5221      -56
+ Partials      408      393      -15

Impacted Files
Coverage Δ

cmd/migrate/main.go
0.00% <0.00%> (ø)

cmd/migrate/orgmigration/orgmigration.go

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Extension issue

Issue Type: Bug
Extension Name: vscode-azureresourcegroups
Extension Version: 0.5.3
OS Version: Windows_NT x64 10.0.19043
VS Code version: 1.68.1

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Revert "trezor-suite: mark as broken"

This reverts commit 2beb7de23e38e81a0b2948a32243a1b958ca81dc.
Description of changes
Trezor Suite is working fine.
This is my configuration:
{ config, pkgs, ... }:

{
  config = {
    services = {
      trezord = {
        enable = true;
      };
      udev.packages = with pkgs; [ trezor-udev-rules ];
    };
    environment.systemPackages = with pkgs; [
      trezor-suite
      trezorctl
    ];
  };
}

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Would you please also post the config in #281975? Thanks a lot.

Also @sudoforge for testing.

Also @sudoforge for testing.

i can confirm that the previous issue appears to be resolved (at least as of 2726f127c15a4cc9810843b96cad73c7eb39e443).


multiple fixes for makefile and issues

This PR has 5 commits

add new makefile target env so that users can simply do git clone ... and then make env to install all dependencies.

fix #34 by adding makefile targets to build for individual platforms

fix #29 (partial fix)

updates electron-builder and electron (minor version updates)

I've tested and verified these changes on my Fedora 29 host.

thanks a lot @cyfrost
changes are great!