Horizontal tab: The app crashes when trying to open a new tab

Edit: This happens if "Multitasking flyout" is disabled
Log:
2020-08-03 22:21:50.1164|ERROR|Files.App|Object reference not set to an instance of an object.|System.NullReferenceException: Object reference not set to an instance of an object.
at Files.Controls.InteractionViewModel.set_TabStripSelectedIndex(Int32 value)
at Files.UserControls.HorizontalMultitaskingControl.horizontalTabView_TabItemsChanged(TabView sender, IVectorChangedEventArgs args)
at System.Collections.Specialized.NotifyCollectionChangedEventHandler.Invoke(Object sender, NotifyCollectionChangedEventArgs e)
at System.Collections.ObjectModel.ObservableCollection1.OnCollectionChanged(NotifyCollectionChangedEventArgs e)    at System.Collections.ObjectModel.ObservableCollection1.InsertItem(Int32 index, T item)
at System.Collections.ObjectModel.Collection`1.Insert(Int32 index, T item)
at Files.Views.MainPage.AddNewTab(Type t, String path, Int32 atIndex)
at Files.UserControls.HorizontalMultitaskingControl.TabView_AddTabButtonClick(TabView sender, Object args)
at System.Runtime.CompilerServices.AsyncMethodBuilderCore.<>c.b__7_0(Object state)
at System.Threading.WinRTSynchronizationContextBase.Invoker.InvokeCore()

I have been facing same issue, but in my case, I only disabled this multitasking flyout option on settings, then I tried to open a new tab, and app crashes.

I confirm that it crashes after disabling "Multitasking Flyout"

Thank you for creating this feedback item. This issue was resolved in our stable branch and should be available from the Microsoft Store.


Not working in Firefox

Hi
Tweets are not showing in firefox showing wrong json format please the screenshots of chrome and firefox.

Thanks

Thanks for the feedback, I will need to fix this.

Issue was with innerText vs. textContent. This is now fixed.


Port billing and usage updates from demo branch.

Ported demo branch changes:

Refactor billing functions for new config
Rename dimensions to billed_dimensions
Add empty metering based on reporting time
Add bill utils module and move all relevant functions.
Add test billing interval

Fix comments.

Add warning log for debugging local csp meter call

Use last bill time to calculate next bill time

Add unit test support for updated csp_cache module and the newly added
bill_utils module.
Renamed config_good.yaml to config_good_average.yaml, as it specifes
average as the usage_aggregation method.
Added a new config_good_maximum.yaml that specifies to use a maximum
usage_aggregation method.
Added a new config_test_mixed.yaml which has two metrics, one with
average as the usage_aggregation method, and the other with maximum
as the usage_aggregation method.
Updated conftest.py to reflect rename of default config file, and to
also return a Config object.
Also moved the registration of in-memory cache and csp_config hook
implementations into the conftest.py's cba_pm fixture.
Co-authored-by: smarlowucf<EMAIL_ADDRESS>Co-authored-by: rtamalin<EMAIL_ADDRESS>

With these changes I'm up to ~87.5% coverage:
---------- coverage: platform linux, python 3.10.6-final-0 -----------
Name                                       Stmts   Miss  Cover
--------------------------------------------------------------
csp_billing_adapter/__init__.py                6      0   100%
csp_billing_adapter/adapter.py                59     32    46%
csp_billing_adapter/bill_utils.py             61      4    93%
csp_billing_adapter/config.py                 28      0   100%
csp_billing_adapter/csp_cache.py              20      0   100%
csp_billing_adapter/csp_config.py              7      0   100%
csp_billing_adapter/csp_hookspecs.py          11      0   100%
csp_billing_adapter/hookspecs.py              10      1    90%
csp_billing_adapter/local_csp.py              19      0   100%
csp_billing_adapter/memory_cache.py           19      0   100%
csp_billing_adapter/memory_csp_config.py      19      0   100%
csp_billing_adapter/product_api.py             9      0   100%
csp_billing_adapter/storage_hookspecs.py      15      0   100%
csp_billing_adapter/utils.py                  30      2    93%
--------------------------------------------------------------
TOTAL                                        313     39    88%

FAIL Required test coverage of 90.0% not reached. Total coverage: 87.54%

With a couple of last minute tweaks to test_bill_utils.py and test_utils.py, the coverage is up to 88.82%.
% pytest --cov=csp_billing_adapter --cov-report term-missing
========================================================= test session starts ==========================================================
platform linux -- Python 3.10.6, pytest-7.3.1, pluggy-1.0.0
rootdir: /home/fergal/work/public-cloud-team/csp-adapter/github/csp-billing-adapter
configfile: setup.cfg
testpaths: tests
plugins: cov-4.0.0
collected 40 items

tests/unit/test_bill_utils.py ........                                                                                [ 20%]
tests/unit/test_config.py ...                                                                                [ 27%]
tests/unit/test_csp_cache.py ....                                                                                [ 37%]
tests/unit/test_csp_config.py .                                                                                [ 40%]
tests/unit/test_local_csp.py ....                                                                                [ 50%]
tests/unit/test_memory_cache.py ....                                                                                [ 60%]
tests/unit/test_memory_csp_config.py ....                                                                                [ 70%]
tests/unit/test_product_api.py .                                                                                [ 72%]
tests/unit/test_utils.py ...........                                                                                [100%]

---------- coverage: platform linux, python 3.10.6-final-0 -----------
Name                                       Stmts   Miss  Cover   Missing
------------------------------------------------------------------------
csp_billing_adapter/__init__.py                6      0   100%
csp_billing_adapter/adapter.py                59     32    46%   58-102, 106
csp_billing_adapter/bill_utils.py             61      2    97%   68, 95
csp_billing_adapter/config.py                 28      0   100%
csp_billing_adapter/csp_cache.py              20      0   100%
csp_billing_adapter/csp_config.py              7      0   100%
csp_billing_adapter/csp_hookspecs.py          11      0   100%
csp_billing_adapter/hookspecs.py              10      1    90%   45
csp_billing_adapter/local_csp.py              19      0   100%
csp_billing_adapter/memory_cache.py           19      0   100%
csp_billing_adapter/memory_csp_config.py      19      0   100%
csp_billing_adapter/product_api.py             9      0   100%
csp_billing_adapter/storage_hookspecs.py      15      0   100%
csp_billing_adapter/utils.py                  30      0   100%
------------------------------------------------------------------------
TOTAL                                        313     35    89%

FAIL Required test coverage of 90.0% not reached. Total coverage: 88.82%

========================================================== 40 passed in 0.40s ==========================================================


Analyzer becomes very slow for many joins

Hey there,
Been playing around with this library for a pet project of mine. Really enjoying it so far, so thanks for your work!
I ran into an issue with the analyzer. I haven't quite figured out exactly what's causing it, but for a certain number of joins the analyzer seems to run into an expensive loop somewhere. I created a small program to demonstrate the issue:
https://gist.github.com/ligustah/2f058e3dc6635bd113048301d1c227e7
Tried profiling/tracing to see where it gets stuck, but it didn't really come out conclusive for me. Maybe you can have a look or point me in the right direction?
(edit: worth noting, it doesn't seem to be an endless loop. in another test on my machine the query would complete after like 6 minutes. on a mysql server running in docker it completes within milliseconds)

Hi @ligustah,
This is almost certainly a result of inefficiencies in the join planning algorithm. The current algorithm to construct a join plan given a table ordering is pretty wasteful and duplicates a lot of work. We have tests of 5 tables, but apparently 7 pushes it over the edge. The algorithm is brand new and has not been optimized yet.
If you wanted to try to take a pass at making it faster, this is where you should look:
https://github.com/dolthub/go-mysql-server/blob/master/sql/analyzer/join_search.go#L332
I have ideas for making the complex tree building process used here unnecessary, but it involves larger changes to the analyzer as a whole. So for now, you should focus on pruning the search space in this algorithm.
More about the algorithm in this blog post:
https://www.dolthub.com/blog/2020-12-28-join-planning/
Let me know what else I can tell you,
Zach

Hi @ligustah,
This is almost certainly a result of inefficiencies in the join planning algorithm. The current algorithm to construct a join plan given a table ordering is pretty wasteful and duplicates a lot of work. We have tests of 5 tables, but apparently 7 pushes it over the edge. The algorithm is brand new and has not been optimized yet.
If you wanted to try to take a pass at making it faster, this is where you should look:
https://github.com/dolthub/go-mysql-server/blob/master/sql/analyzer/join_search.go#L332
I have ideas for making the complex tree building process used here unnecessary, but it involves larger changes to the analyzer as a whole. So for now, you should focus on pruning the search space in this algorithm.
More about the algorithm in this blog post:
https://www.dolthub.com/blog/2020-12-28-join-planning/
Let me know what else I can tell you,
Zach

Hey @zachmu,
thanks for getting back to me and pointing me in the right direction. I've been reading through the blog and also looking at some other material, so I'm starting to get a better understanding of the issue. I might take a stab at it and just see if I can make it any faster, but for now will content with trying to avoid having too many joins.
Do you already have a prospective timeline for when those bigger changes are going to happen?

Hey @zachmu,
thanks for getting back to me and pointing me in the right direction. I've been reading through the blog and also looking at some other material, so I'm starting to get a better understanding of the issue. I might take a stab at it and just see if I can make it any faster, but for now will content with trying to avoid having too many joins.
Do you already have a prospective timeline for when those bigger changes are going to happen?

No, I don't have an estimate for that yet.

No, I don't have an estimate for that yet.

@ligustah We've now rewritten the join order analyzer, and it works for 10 tables in under a second now. We haven't done a release of the package since then, so you'll need to get the version @master for this optimization. Let me know if you have any trouble with it.

Wow, that's amazing! I'll definitely check it out. Thank you 🙇


add the taxable area calculation on backend

Added a bunch of logic for calculating GST differently. The values I get from this updated code line up with the in-depth screenshot shared by Bal so this should be the correct calculation.

Your Pull Request code is being promoted!  Please follow the link below.
Main Merge Workflow

DEV deployments have completed successfully!
Backend available
Frontend available


Global service scale should not be allowed to edit/change through API

Rancher Versions:
Server:v1.4.0-rc6
Environment Type: (Cattle/Kubernetes/Swarm/Mesos): Cattle
Steps to Reproduce:

Create a global service
Edit the scale to value 6 through API.

The scale gets updated to the new value in the HTTP response.
Note: The service does not scale as the label "io.rancher.scheduler.global": "true" is honored
HTTP/1.1 200
Date: Wed, 01 Feb 2017 00:19:09 GMT
X-Api-User-Id: 1a1
Server: Jetty(9.2.11.v20150529)
X-Api-Account-Id: 1a5
Transfer-Encoding: chunked
Content-Type: application/json; charset=utf-8
X-Api-Schemas: http://<IP_ADDRESS>:8080/v2-beta/projects/1a5/schemas
X-Rancher-Version: v1.4.0-rc6
X-Api-Client-Ip: <IP_ADDRESS>
Expires: Thu, 01 Jan 1970 00:00:00 GMT

{
"id": "1s8",
"type": "service",
"links": {
"self": "…/v2-beta/projects/1a5/services/1s8",
"account": "…/v2-beta/projects/1a5/services/1s8/account",
"consumedbyservices": "…/v2-beta/projects/1a5/services/1s8/consumedbyservices",
"consumedservices": "…/v2-beta/projects/1a5/services/1s8/consumedservices",
"instances": "…/v2-beta/projects/1a5/services/1s8/instances",
"networkDrivers": "…/v2-beta/projects/1a5/services/1s8/networkdrivers",
"serviceExposeMaps": "…/v2-beta/projects/1a5/services/1s8/serviceexposemaps",
"serviceLogs": "…/v2-beta/projects/1a5/services/1s8/servicelogs",
"stack": "…/v2-beta/projects/1a5/services/1s8/stack",
"storageDrivers": "…/v2-beta/projects/1a5/services/1s8/storagedrivers",
"containerStats": "…/v2-beta/projects/1a5/services/1s8/containerstats"
},
"actions": {
"upgrade": "…/v2-beta/projects/1a5/services/1s8/?action=upgrade",
"update": "…/v2-beta/projects/1a5/services/1s8/?action=update",
"remove": "…/v2-beta/projects/1a5/services/1s8/?action=remove",
"deactivate": "…/v2-beta/projects/1a5/services/1s8/?action=deactivate",
"removeservicelink": "…/v2-beta/projects/1a5/services/1s8/?action=removeservicelink",
"addservicelink": "…/v2-beta/projects/1a5/services/1s8/?action=addservicelink",
"setservicelinks": "…/v2-beta/projects/1a5/services/1s8/?action=setservicelinks"
},
"baseType": "service",
"name": "globalservice",
"state": "updating-active",
"accountId": "1a5",
"assignServiceIpAddress": false,
"createIndex": 3,
"created": "2017-01-31T23:48:13Z",
"createdTS":<PHONE_NUMBER>000,
"currentScale": 3,
"description": null,
"externalId": null,
"fqdn": null,
"healthState": "healthy",
"instanceIds": [ 3 items
"1i34",
"1i35",
"1i36"
],
"kind": "service",
"launchConfig": {
"type": "launchConfig",
"capAdd": [ ],
"capDrop": [ ],
"dataVolumes": [ ],
"dataVolumesFrom": [ ],
"devices": [ ],
"dns": [ ],
"dnsSearch": [ ],
"imageUuid": "docker:sangeetha/testnewhostrouting",
"instanceTriggeredStop": "stop",
"kind": "container",
"labels": {
"io.rancher.scheduler.global": "true",
"io.rancher.container.pull_image": "always"
},
"logConfig": {
"type": "logConfig",
"config": { },
"driver": ""
},
"networkMode": "managed",
"ports": [ ],
"privileged": false,
"publishAllPorts": false,
"readOnly": false,
"secrets": [ ],
"startOnCreate": true,
"stdinOpen": true,
"system": false,
"tty": true,
"version": "0",
"dataVolumesFromLaunchConfigs": [ ],
"vcpu": 1
},
"lbConfig": null,
"linkedServices": null,
"metadata": null,
"publicEndpoints": [ ],
"removed": null,
"retainIp": null,
"scale": 6,
"scalePolicy": null,
"secondaryLaunchConfigs": [ ],
"selectorContainer": null,
"selectorLink": null,
"stackId": "1st5",
"startOnCreate": true,
"system": false,
"transitioning": "yes",
"transitioningMessage": "In Progress",
"transitioningProgress": null,
"upgrade": null,
"uuid": "7c81c80c-edbe-4c47-9d7d-f0e3babaf0a3",
"vip": null
}

Closing as we are planning to allow global-> scale and vice versa: https://github.com/rancher/rancher/issues/8159


Bump to Dart SDK 1.23.0-dev.10

allows us to understand flutter usage via telemetry
brings in @immutable

Fixes: #9042
@abarth @tvolkert @Hixie
FYI: @sethladd @kevmoo

Looks like appveyor is unhappy.
Anyone know how to get at flutter_01.log on an appveyor build?
@sethladd @tvolkert ?

Anyone know how to get at flutter_01.log on an appveyor build?

@goderbauer

brings in @immutable

How dire is our need to use this?  I ask because I'd like to minimize the time difference between when we start using new SDK features externally and when they're available internally within Google.  The longer that lag time, the more backed up we get when trying to bump Flutter within Google.  @keertip may also know the likelyhood of dev.10 landing internally soonish.

@tvolkert - @dgrove says dev.10 should be this week in Google3

This week is an extremely optimistic view! We are fixing google3 code in preparation, so should have a better idea of when dev.10 will be in by mid week.

I ask because I'd like to minimize the time difference between when we start using new SDK features externally and when they're available internally within Google.

Is this true for analyzer lints as well?  We could start using the @immutable annotations and old versions of the analyzer would just ignore it.

We could start using the @immutable annotations and old versions of the
analyzer would just ignore it.

Good point - in this case there's no harm in using this immediately.

Anyone know how to get at flutter_01.log on an appveyor build?

Not that I am aware of. However, it's probably crashing because the ansiSupported getter we use in [1] moved from platform to stdin in 1.23-dev.10. The new API is https://api.dartlang.org/dev/1.23.0-dev.10.0/dart-io/Stdin/supportsAnsiEscapes.html. My understanding is that the new API can replace the entire colorSupported logic in Flutter as it now supports detecting ANSI support on POSIX and Windows platforms.
If it is sill crashing after that change, I am happy to run it locally on my Windows machine and get you the log from there.
[1] https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/base/logger.dart#L253

Thanks all!
@danrubel (or anyone), I'm curious why the test script has two analysis passes.
bin/flutter analyze --flutter-repo

followed by:
bin/flutter analyze --flutter-repo --watch --benchmark

I notice that the first doesn't catch everything but shouldn't it?

I'm curious why the test script has two analysis passes.

We kept breaking the --watch version when we were only running the first one.  I would have thought they'd report the same issues.  /cc @Hixie

Thanks @abarth .

I just updated flutter analyze to call dartanalyzer, and at this point:

flutter analyze calls dartanalyzer
flutter analyze --flutter-repo uses the embedded analyzer library in a custom way
flutter analyze --watch (all forms) calls AnalysisServer

The first and third should always produce the same results. Now I need to enhance dartanalyzer before I can bring the second into line with the other two.

Thanks @goderbauer!  Am I right that really we need to update the platform package?  It looks like you started that a ways back (https://travis-ci.org/google/platform.dart/builds/211981851) actually.  Is that something we should pick up again?

@pq Yes, package:platform will have to be updated for this as well.

LGTM

@pq Where do I find the declaration for @immutable?  Maybe we need to publish a new copy of package:meta ?

Apparently, this didn't give us @immutable. For next time, to better understand how to tell if a feature is available, why didn't dev.10 give us this feature?
cc @pq

@sethladd give me the commit where this landed in the SDK and I can show you "one cool trick"

@immutable exists in package:meta, not the SDK.  Presumably, the SDK version is bundled with an analyzer version that respects @immutable...

https://github.com/dart-lang/sdk/commit/4f28d97ac23605a280b6ad26d78d83655018af0f
Look at SDK commits.

So this should be working. Assuming you mean in the analyzer...

Ah, I see the -dev.11 in that screenshot. OK, probably just some wires crossed. Thanks!


feat(prettier): print leading comments with newlines

Current dependencies on/for this PR:

main

PR #1434   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 19, 9:46 AM: @Boshen merged this pull request with Graphite.


Move transpiler tests

moving transpile tests from compiler/test_compiler.py to transpiler/test_transpile.py.
This PR goes on top of https://github.com/Qiskit/qiskit-terra/pull/2082

on hold, until #2082 gets merged.

Removing on hold since #2082 is merged.


Move to mirrored yq image

Builds on internal CI are failing, we're not allowed to pull the image anymore.

/lgtm


high latency on xaiver nx if align depth enabled

Required Info

Camera Model
D455 (FRAMOS D455e)

Firmware Version
<IP_ADDRESS>

Operating System & Version
Nvidia Jetpack R32.6.1 (based on Ubuntu 18.04)

Kernel Version (Linux Only)
4.9.253-tegra

Platform
Nvidia Jetson Xavier NX

SDK Version
2.45.10

Language
C++ & realsense2_camera (ROS) v2.3.0

Segment
Robot

Issue Description
The Problem we encounter currently is already perfectly described in its symptoms by the following issue: 9519
(Why I open a new issue, because the issue not document any solution for the problem)
If we just launch rs_camera.launch the RGB images are received with no latency and the system
is only under a moderate / expected load.
If we launch the node with rs_aligned_depth.launch (so align_depth parameter is set to true) the rgb and
aligned depth stream show a significant latency and the system is under high load (as described
in issue  9519).
Also as depcited in issue 9519 the effect is only present when the realsense-ros ros driver is
used. The realsense-viewer does not show the effect (is align_depth used here?).
To give you a little bit more context, here are the instructions which are executed for device bring-up:
#!/bin/bash

# install latest nvidia software updates
sudo apt-get update
sudo apt-get upgrade -y
sudo apt-get install -y nvidia-jetpack

# create symlink to fix the issue that
# nvidia-jetpack installs opencv 4 but
# ros melodic expects opencv 3
sudo ln -s /usr/include/opencv4/ /usr/include/opencv

#install ros noetic
ROS_DISTRO=melodic

#install required packages
sudo apt-get install -y --no-install-recommends \
    git \
    cmake \
    build-essential \
    curl \
    wget \
    gnupg2 \
    lsb-release \
    ca-certificates \

sudo apt install -y libglfw3-dev libglew-dev libgtk-3-dev libssl-dev libusb-1.0-0-dev pkg-config  \
                 libgl1-mesa-dev libglu1-mesa-dev dkms libboost-dev \
                 libjsoncpp-dev libtclap-dev libeigen3-dev

sudo rm -rf /var/lib/apt/lists/*

sudo sh -c 'echo "deb http://packages.ros.org/ros/ubuntu $(lsb_release -sc) main" > /etc/apt/sources.list.d/ros-latest.list'
sudo curl -s https://raw.githubusercontent.com/ros/rosdistro/master/ros.asc | sudo apt-key add -

sudo apt-get update

sudo apt-get install -y python3-catkin-tools
sudo apt-get install -y ros-${ROS_DISTRO}-desktop-full

echo "source /opt/ros/${ROS_DISTRO}/setup.bash" >> ~/.bashrc
source ~/.bashrc

# get driver for framos d455e camera
wget -N https://www.framos.com/framos3d/D400e/Software/Latest/FRAMOS_D400e_Software_Package_Linux64_ARM_latest.tar.gz

# unpack framos files
tar xpvf FRAMOS_D400e_Software_Package_Linux64_ARM_latest.tar.gz

# install FRAMOS camera suite and driver
sudo apt install -y ./FRAMOS_D400e_Software_Package/FRAMOS_CameraSuite*.deb
sudo apt install -y ./FRAMOS_D400e_Software_Package/FRAMOS-librealsense2*.deb

rm -rf FRAMOS_D400e_Software_Package*

cd ~/Desktop
mkdir -p catkin_ws/src
cd catkin_ws/src
git clone https://github.com/pal-robotics/ddynamic_reconfigure.git
git clone https://github.com/IntelRealSense/realsense-ros.git
cd realsense-ros
git checkout 2.3.0
cd ../../
catkin build
echo "source ~/Desktop/catkin_ws/devel/setup.bash" >> ~/.bashrc

sudo apt-get autoremove -y

reboot

I can also confirm, that this effect is still present after a clean installation
of Jetpack 4.6 and the execution of only the above listed steps.
Are there any other differences to issue: 9519?
Yes, my guess is that FRAMOS not compile the librealsense with the cuda compile flag.
Any advise how to analyse or solve the problem is highly appreciated!
@ProtossDragoon as autor of issue: 9519 could you perhaps provide additional information
on how you solve the problem?

Hi @MartyG-RealSense,
ok I have contacted FRAMOS now. In parallel I will try to investigate the problem with my D455 (Intel). Any advice on the best way to proceed?

Hi @manuelvolk,  I just reinstalled entire system several times with few changes (e.g. change build parameters or package/SDK installation order B->A instead of A->B ...) and it fixed the issue at a certain moment. 😭
I tried to figure out the reason of the problem, but I couldn't. I'm very sorry I can't help you.


Add logging output for chain reorgs

What is wrong?
Chain re-orgs happen and it would be nice to see some info about them when running trinity.
How can it be fixed
Whenever trinity is syncing and it encounters a chain reorg we should log something with depth and hashes.  Probably configurable to a certain minimum depth.
For lightchain, this should be relatively easy.
https://github.com/ethereum/py-evm/blob/master/p2p/lightchain.py#L214
Anywhere that headerdb.coro_persist_header is called should return an iterable of headers which are new to the canonical chain.  If the returned list contains more than 1 entry then the chain just re-organized.
Doing this for full sync is going to require updates to the Py-EVM API so that chain.import_block returns an iterable of blocks that are part of the new canonical chain.

registering this  for myself.

did not get time to work on this. If this is blocking anyone, I have no problem with anyone else picking this up!

is this still valid? It seems like the link provided in the title is outdated. #975 did some refactoring. Also, logging seems to have been added in #974.


ocamlPackages.dune-configurator: add minimumOCamlVersion

Motivation for this change
Source: https://github.com/ocaml/opam-repository/blob/master/packages/dune-configurator/dune-configurator.2.7.1/opam#L21
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions: Ubuntu

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build ocaml-ng.ocamlPackages_4_02.dune-configurator @GrahamcOfBorg build ocaml-ng.ocamlPackages_4_08.dune-configurator


Update plotly/raster.py to fix Raster for plotly

This should fix an issue with Raster for plotly backend which was reported here:
https://github.com/ioam/holoviews/issues/2821

LGTM, will merge when tests pass.

if ci would be so kind and report which of the notebooks are failing to be tested - I would probably take a look at it...
Situation is getting even more strange given that this test failure is Python3 sensitive. Python2 test is passed with no issues.

Don't worry about it, don't think it has anything to do with your change.

I'll merge and fix it up later if the issue persists.


Mapping Tab

Hi,
can you give me short hint how the "Mapping" tab works? Or do you have an example? I see the screenshot but I don't understand how to use it ;-)
Thank you

Off course
I've added rounded rectangle on your example and 3 texts (value 1, value2 and Value3).
Can i add this animated in docs for next release ?
Open image on tab or full screen to see detail.
PS : Sorry for my powerless english

Thank you very much, now i understand :-)
Sure, use it for your documentation.


Error: No NgModule metadata found for 'undefined' for typescript converted js file

Hi,
I am able to load module dynamically using import() method for angular modules in developing mode where I used typescript. i.e. when modules are written in typescript its working fine. Then I tried it in production mode and build all my typescript and generate js bundle and load it into web server. In that case it is not working and  I am getting the following error Error: No NgModule metadata found for 'undefined' to load the module.
The scenario where I am getting error is, I developed some plugins with component, module and service, then build the typescript files to get the JS files and import ...module.js file using import() method. In that case, it is not working. But whenever I am using typescript plugins with angular-cli then it works fine. How can I import external JS module by avoiding the mentioned error? Thanks.

@khaled-ansary That seems like it is an Angular specific build issue. I would check with the angular-cli team. This repo is to discuss the specification.

Indeed.


Reorganize example folders inside PrimeQA

Current example folders include run_* scripts, they should be part of PrimeQA core as entrypoints. The example folder should rather contain example scripts which have used these PrimeQA core modules for different tasks or papers.

This is complete. PR merged.


Fix markdown lists with codeblocks

/cc @eranco74
/cc @omertuc

I don't like that, why? It's nice to be able to specify the language syntax

I don't like that, why? It's nice to be able to specify the language syntax

When using a CLI tool to render markdown I discovered that not putting a blank line after the list item was not standard markdown. Then I noticed that numbered lists were not even being detected as lists:
Check and see that there are two items number 3
https://github.com/rh-ecosystem-edge/ib-orchestrate-vm/blob/master/README.ibi.md
So this PR makes that we comply with markdown syntax
I was also shocked that you cannot specify the type of syntax when doing it this way
Anyway, I don't have a strong opinion on either of the ways, if you prefer to leave the broken numbered lists in favor of better syntax highlighting, so be it :)

You're mentioning two issues (blank line after list item, numbered lists not being detected as lists) but you didn't mention what triple-back-tick vs indentation for codeblocking (which is what I care about) has to do with it

indentation codeblocks are the default in markdown
triple-back-tick code blocks are called fenced codeblocks
Markdown only supports fenced codeblocks at the root level, meaning they cannot be nested inside of a list
And so, when adding fenced codeblocks in the middle of a list, they "break" the list
In the case of numbered lists, it is obvious because the broken automatic numbering of items makes it visible
Indented codeblocks can be nested, so by changing to that codeblocks, the lists are not broken
I see three options here:

Use indented codeblocks when they need to be nested
Only use indented codeblocks when there is a numbered list. We will have syntax highlighting everywhere but in ordered lists
Dont use indented codeblocks, and write numbers manually the numbered lists

Since I don't have such a strong preference for any of those, I'd say we can go for options 2 or 3
You seem to have a stronger opinion on the issue, so tell me your preference and I'll amend this PR with that in mind

Lol my opinion is not strong, it's just markdown syntax :laughing:. I'm just trying to understand what's going on. People often mistake my questions for opinions.
Anyway, option 3 is my favorite

Lol my opinion is not strong, it's just markdown syntax 😆. I'm just trying to understand what's going on. People often mistake my questions for opinions.

Sorry for the misunderstanding :D :D

Anyway, option 3 is my favorite

I just updated the markdowns to do that, please check it out


How to deprecate a Dev Container Feature in a git repository? (and remove the Feature from the metadata?)

We had to deprecate some Dev Container Features.

eitsupi/devcontainer-features#30
guiyomh/features#17

I did this by creating a directory deprecated in the repository and moving the source code from the src directory.
It would be great if such a method could be documented and standardized.

Great timing! @samruddhikhandale has opened a new spec issue to add a deprecatedIds property to the devcontainer-feature.json to solve issues like this. Please let us know on that issue your thoughts.

Oh, I did not expect to be able to use deprecatedIds for this purpose. That sounds nice!
If we specify their own id as follows, this Feature would need to be removed from the repository's metadata and devcontainers index.
{
  "id": "foo",
  "deprecatedId": [
    "foo"
  ]
}

In the spec issue, I was trying to use deprecatedId for renaming an existing Feature 🤔
So that anyone referencing the old Feature will still be able to install that Feature without breaking their configs.
I believe you are simply trying to deprecate a Feature?

If we specify their own id as follows, this Feature would need to be removed from the repository's metadata and devcontainers index.

devcontainers index - Moving the source folder of the Feature out of the src folder (or even deleting the folder) will avoid adding the metadata to devcontainer-collection.json . We build the index daily, hence in max 24 hours, the Feature will vanish from the index which is then consumed by https://containers.dev/features and VS Code.
repository's metadata - We could simply delete the OCI artifact in this case?
Feature --> Package settings

My suggestion is that such a step be documented (the name of the directory to be moved to be defined)

Makes sense. I'll document this part, thanks 👍

or that deprecatedIds be made available for this use.

I somehow think that this might complicate the property and the source folder will continue to exist. So documenting on how to clean up should be the best move forward?

If the intention is that a user wants to abandon maintaining  a Feature, but not actually delete the current published artifacts, we might want to do additional work here
Perhaps actually introduce a deprecated flag that the action will append to the collection metadata if a Feature is removed from the ./src directory.  Supporting tools (our index, etc...) can surface helpful information with that property.
Right now the action just concatenates all the individual devcontainer-feature.json, although the published index file does have more properties (like majorVersion).

In the future, when the CLI automatically generates the top-level README for the repository, it would be great if it could also generate a section on deprecated Features.
To do this, we need to tell the CLI which features have been deprecated. (I was assuming it would be a specific subdirectory, e.g. /deprecated)

@eitsupi Based on the discussion in this issue, updated the spec to add a new isDeprecated property. Would appreciate your thoughts/feedbacks on the approach on that spec issue. Thanks!

I think #146 seems good, thank you for working on this.
Once this is implemented I will use this to deprecate some Features.

The new deprecated property is working well now. So I am closing this issue.
https://github.com/eitsupi/devcontainer-features/blob/af60bc8a47c03d97971a1effd9631ef7f1f3bd97/src/just/devcontainer-feature.json#L5
Thank you for working on this series of tasks.


Crash while loading jpeg

Crashed: com.contentful.Concorde
EXC_BAD_ACCESS KERN_INVALID_ADDRESS at 0x00000000
Thread : Crashed: com.contentful.Concorde
0  libobjc.A.dylib                0x31e2af78 lookUpImpOrForward + 47
1  libobjc.A.dylib                0x31e2af3f _class_lookupMethodAndLoadCache3 + 34
2  libobjc.A.dylib                0x31e2af3f _class_lookupMethodAndLoadCache3 + 34
3  libobjc.A.dylib                0x31e311f9 _objc_msgSend_uncached + 24
4  Concorde                       0x001f7a64 _TFFC8Concorde19CCBufferedImageView10connectionFS0_FTCSo15NSURLConnection14didReceiveDataCSo6NSData_T_U_FT_T_ + 372
5  libdispatch.dylib              0x3238540f _dispatch_client_callout + 22
6  libdispatch.dylib              0x3238e861 _dispatch_barrier_sync_f_invoke + 48
7  Concorde                       0x001f62a8 _TToFC8Concorde19CCBufferedImageView10connectionfS0_FTCSo15NSURLConnection14didReceiveDataCSo6NSData_T_ + 304
8  CFNetwork                      0x2410aeed __65-[NSURLConnectionInternal _withConnectionAndDelegate:onlyActive:]_block_invoke + 56
9  CFNetwork                      0x2410aea7 -[NSURLConnectionInternal _withConnectionAndDelegate:onlyActive:] + 182
10 CFNetwork                      0x2410af99 -[NSURLConnectionInternal _withActiveConnectionAndDelegate:] + 36
11 CFNetwork                      0x240f8a11 _NSURLConnectionDidReceiveData(_CFURLConnection*, __CFData const*, long, void const*) + 80
12 CFNetwork                      0x2401deab ___ZN27URLConnectionClient_Classic29_delegate_didReceiveDataArrayEv_block_invoke + 158
13 CFNetwork                      0x240bc76d ___ZN27URLConnectionClient_Classic18_withDelegateAsyncEPKcU13block_pointerFvP16_CFURLConnectionPK33CFURLConnectionClientCurrent_VMaxE_block_invoke_2 + 60
14 CFNetwork                      0x24010d71 RunloopBlockContext::_invoke_block(void const*, void*) + 60
15 CoreFoundation                 0x244c35a1 CFArrayApplyFunction + 36
16 CFNetwork                      0x24010c2b RunloopBlockContext::perform() + 182
17 CFNetwork                      0x24010af1 MultiplexerSource::perform() + 216
18 CFNetwork                      0x2401098d MultiplexerSource::_perform(void*) + 48
19 CoreFoundation                 0x24578fd7 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 14
20 CoreFoundation                 0x245783eb __CFRunLoopDoSources0 + 222
21 CoreFoundation                 0x24576a69 __CFRunLoopRun + 768
22 CoreFoundation                 0x244c3b31 CFRunLoopRunSpecific + 476
23 CoreFoundation                 0x244c3943 CFRunLoopRunInMode + 106
24 GraphicsServices               0x2b8a1051 GSEventRunModal + 136
25 UIKit                          0x27ab5f21 UIApplicationMain + 1440
26 Chata de Galocha               0x00043e34 main (AppDelegate.swift:19)

Can you share a bit more information about the crash, e.g. the header of the crash log to see OS version and device it happens on?

Hi, sure:

I also have the jpeg the user was trying to open:
http://chatadegalocha.com/fotos_app/65-2661f4fbc22de22593bb6d1d535e5132.jpg

Thanks, will have a look at it.

Just found out that this image:
http://www.chatadegalocha.com/fotos_app/63-970a2b130a2ebf8a75dff03f473ee9a1.jpg
Always crash for that iPhone 4s user. It works fine with my 5c. It's working fine now that I replaced it with a progressive jpeg.

I keep getting these errors on another images, always on iPhone 4s. Can you give me some directions on workarounds to this error? Not showing the image would be more acceptable then crashing the application.
Thanks!

Sorry for dropping the ball on this. Unfortunately, I can't reproduce it so far, don't have an iPhone 4S at my disposal. Doesn't happen on other 32-bit devices for me, like an iPod touch.

Closing because I was never able to reproduce this.


[NEW ICON] SQLite

Skill Name
SQLite3
Why?
SQLite is one of the most widespread DBs! It's baked into a lot of systems that run our world and is used by default in a lot of languages.
Reference Image
https://upload.wikimedia.org/wikipedia/commons/9/97/Sqlite-square-icon.svg

SQLite is already in the project, you can use it by using this https://skillicons.dev/icons?i=sqlite

[!NOTE]
You may want to use the new rolling release project which features a lot more icons and is community driven here since there is no real sign of life of the maintainer...


[Op] Add Concat+Cast fusion

Issue: https://github.com/changqi1/DeepRec/issues/24

@shanzhou2186 @changqi1 @Duyi-Wang  I ran the benchmark 3 times on AliCloud. You can see the results in the table below. The performance numbers shows the throughput (in MB/s) (higher is better).

If Fused
Name
Performance no. 1 [MB/s]
Performance no. 2 [MB/s]
Performance no. 3 [MB/s]
Average [MB/s]
Performance boost comparing to nonfused version

fused
concat_cast_2_1_fp32_bfloat16_512x16
770.00
776.3
771.2
772.50
105%

nonfused
concat_cast_2_1_fp32_bfloat16_512x16
738.1
738.9
727.1
734.70

fused
concat_cast_13_-1_fp32_bfloat16_512x1
221.4
223.5
225.2
223.37
120%

nonfused
concat_cast_13_-1_fp32_bfloat16_512x1
184.6
186.5
185.3
185.47

fused
concat_cast_13_-1_fp32_bfloat16_128x1
76.4
75.6
76
76.00
112%

nonfused
concat_cast_13_-1_fp32_bfloat16_128x1
68.1
68
67.9
68.00

fused
concat_cast_16_2_fp32_bfloat16_512x32x16
3985.2
3943.5
3913.4
3947.37
130%

nonfused
concat_cast_16_2_fp32_bfloat16_512x32x16
3059
3036.2
3029.1
3041.43

fused
concat_cast_64_1_bfloat16_fp32_400x60
3676.5
3815.2
3596.2
3695.97
142%

nonfused
concat_cast_64_1_bfloat16_fp32_400x60
2356.2
2607.7
2835
2599.63

More data is available on Jira. Is there anything else you need?

@aalbersk  This is a real good job. Please tell us how to use grappler to find and replace concat+cast pattern and its kernel logic by meeting?

@aalbersk This is a real good job. Please tell us how to use grappler to find and replace concat+cast pattern and its kernel logic by meeting?

Sure, I can explain it. How would you like to receive the explanation? Is a presentation enough?

Yes, a presentation is enough.

@changqi1 Shared presentation in Jira. Let me know if you need any more changes in the code.

@changqi1 Shared presentation in Jira. Let me know if you need any more changes in the code.

Good job.

LGTM


CHAOS-6351 Add FAQs

Add FAQs

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


Speaker Diarilization

I want to perform speech to text on a wav file and get Multi speaker transcript with Speaker diarisation (who spoke what and when)
But, I am confused is this possible with Deep speech?

This repo is only for ASR, Speaker diarisation is another  task.


B 18058 UI - Create new step for AOA confirmation for customer

B-18058
Summary
This PR adds a confirmation step for the Customer PPM shipments that have requested an Advance. The contents of the UI step on the Customer shipment page will change depending on several conditions. See the screenshots at the bottom of the summary for a visual representation of what the component should look like in each of the following cases:

Customer just submitted an Advance Request but the request has yet to be approved by a Counselor. In this case a 5th Step is displayed stating "Advance request submitted"
Customer requests and Advance for 1 PPM shipment and the Counselor approves it. The 5th Step displays with "Advance request reviewed" and the PPM 1 shipment has a link to download the AOA packet. NOTE: The actual process of downloading the PDF will be done in a follow on story.
Customer requests an Advance for 1 PPM shipment and the Counselor rejects it. The 5th Step displays with "Advance request denied"
Customer requests an Advance for multiple PPM shipments and the Counselor has some and rejected some advances. The 5th Step displays with "Advance request reviewed" and the accepted advances have links to download the AOA packet and the rejected ones display text stating "Advance request denied" with no links.

Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test
Verify Client Tests pass:  make client_test
For this test we will use 2 new customers. With one of them we will create just 1 PPM shipment and request an Advance. With the other we will create at least 2 PPM shipments and request an Advance for each. The UI will update for approved and rejected Advances as soon as the Counselor approves or rejects them. The Counselor doesn't need to approve the shipment so we can use this to our advantage and just toggle the various approve/reject statuses for shipments to test the different scenarios.

Login as a new customer and create at least 1 PPM shipment and select Yes when asked if you want an Advance.
After you create the PPM shipment and submit it, you should now see the "Advance request submitted" as noted in case 1 in the summary above.
Login to the Office App as a Services Counselor for the same GBLOC as the shipment you created.
Access the shipment as a counselor and edit the shipment and Approve the shipment and then select Save.
Go back to the customer app and refresh the page, should see case 2 in summary above.
Go back to counselor page, edit shipment, now reject the advance and save.
Go back to customer app, refresh, should see "Advance request denied" as in case 3 above
For final test we need to create a new customer again
Create multiple PPM shipments (at least 2) and request an advance for each.
Login as Services Counselor for that GBLOC, find the shipment
Edit at least one shipment to accept the advance and save
Edit at least one shipment to reject the advance and save
Go back to Customer and refresh page. You should see a link for the approved shipments and no link for rejected shipments

Optional but also important tests, throw in an HHG shipment and make sure the UI doesn't display for HHG shipments only for PPM shipments.
Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Screenshots

 Everything looks good, but when I have one AOA Approved, one AOA Denied, and another still pending, it makes things look a little funky. Could we maybe add "Pending Review" to the still pending AOA, so that the link is pushed down to the next line and looks a little more clear for the user? So in this case, PPM1 will say Pending Review, PPM2 will have the link, and PPM3 will still say denied?

I added a Pending label to those that haven't been approved or rejected yet but has others in the move that have been approved or rejected.

 Everything looks good, but when I have one AOA Approved, one AOA Denied, and another still pending, it makes things look a little funky. Could we maybe add "Pending Review" to the still pending AOA, so that the link is pushed down to the next line and looks a little more clear for the user? So in this case, PPM1 will say Pending Review, PPM2 will have the link, and PPM3 will still say denied?

I added a Pending label to those that haven't been approved or rejected yet but has others in the move that have been approved or rejected.

Looks good!


Child UITableViewController's scrollview can not scroll to bottom

Thanks for such a great library!
I had a problem that when I use library with UITableViewController embedded with UINavigationBar controller, first of all I had the same issue as #252
Then I adjust the frame of buttonBarView as follows
let originFrame = buttonBarView.frame
var offset: CGFloat = 0
let statusBarHeight = UIApplication.sharedApplication().statusBarFrame.height
offset += statusBarHeight
if let navBarHeight = self.navigationController?.navigationBar.frame.size.height {
    offset += navBarHeight
}
buttonBarView.frame = CGRectMake(0, offset, originFrame.width, originFrame.height)

The button bar is visible but the bottom part of UITableView can never be reached with scrolling.
I found the frame of containerView seems exceeds beyond the border that the scrollbar is going beyond the lower part of screen.
Hope someone can help me to solve this problem.
Thanks!

Hey @imadeit ! It's been some time but are you connecting the buttonBarView outlet ?
Regarding the scrolling issue, you didn't said if you are using the framework with storyboard or programmatically so can you provide a sample project to reproduce the issue ?

Closing due to lack of response

@santiagofm Hi, I am facing same issue container scroll view is not moving up. please suggest how can I make container view scroll vertically as well  so that I can see my all subviews of child controller by scrolling please suggest.

I too have same issue, I am using through storyboard.


Measurements map is not initialized for MetricTelemetry and AggregateMetricTelemetry

When NewMetricTelemetry() or NewAggregateMetricTelemetry() is called, the resulting telemetry struct has its Measurements field left un-initialized (nil).

You cannot submit Measurements against metric telemetry, which is why it is nil.  I figured that was better leaving a map that doesn't get used, misleading a user into thinking they submitted telemetry when they did not.
But really, the issue is that Measurements comes from the "base struct", mostly to keep all the telemetry items looking alike and reduce boilerplate, since it implements most of the Telemetry interface.  That is an ugly shortcut, and I should probably fix that.  (Telemetry.GetMeasurements would still return nil for metrics).

Well, yes, that would prevent me from barking up the wrong tree and implementing a bunch of code to translate the metric information my service is getting into something that turns out to be unsupported in the current version of the library :-/
But also note that although the .NET SDK base does not expose the ability to send multiple measurements as a single metric record, it seems to be a SDK limitation only and the underlying data contract does have this capability https://github.com/Microsoft/ApplicationInsights-dotnet/blob/7321d2bf7b9c4733fb43b6c186e381863931eb1a/src/Microsoft.ApplicationInsights/DataContracts/MetricTelemetry.cs#L35
This capability is used by the data source I am dealing with, namely, the library that is producing the metrics from a service process is doing a lot of pre-aggregations. For example, for "request_time" metric, it is giving me not only sum, stddev, min, max and #of sambles for a given period, but also p75, p90 and p99 percentiles. I'd really like to be able to pass this info along into AI
@SergeyKanzhelev any thoughts on that?

Sorry about that.
The last I heard on the subject is that multiple data points in a metric telemetry are not supported, contrary to what the data contracts say.  I don't think the data collector rejects them, but instead it's one of those undefined behavior scenarios.  Sergey might correct me here.
It sounds like p75, p90, and p99 can be single value metrics in their own right.
When my team has needed to send multiple metrics that are grouped together -- an example is detailed memory usage information, in which we need to be able to find all the values from the same sample during analysis -- we attach multiple measurements to an Event and access that data via Log Analytics queries rather than the portal.

All right, thanks JJ. I will refrain from sending these values for now.
Feel free to close this issue as necessary, but I am curious what Sergey thinks about it.


safe-transaction-service connect port 8545 and exited with code 1

docker reported safe-infrastructure-txs-worker-notifications-webhooks-1 and safe-infrastructure-txs-worker-contracts-tokens-1 exited with code 1. I found both of them try to connect http://localhost:8545, but no service is listening on it obviously. It seems my config has some error.

Below are error messages:
safe-infrastructure-txs-worker-notifications-webhooks-1  | requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=8545): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f720f5af5b0>: Failed to establish a new connection: [Errno 111] Connection refused'))
safe-infrastructure-txs-worker-notifications-webhooks-1 exited with code 1

safe-infrastructure-txs-worker-contracts-tokens-1        | requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=8545): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fbf2a9e4100>: Failed to establish a new connection: [Errno 111] Connection refused'))
safe-infrastructure-txs-worker-contracts-tokens-1 exited with code 1

file .env:
REVERSE_PROXY_PORT=8000
REVERSE_PROXY_UI_PORT=8080

CFG_VERSION=latest
CGW_VERSION=latest
TXS_VERSION=latest
UI_VERSION=latest

RPC_NODE_URL=https://erpc.apothem.network/

All latest images support XDC chain now.

file container_env_files/txs.env
PYTHONPATH=/app/
DJANGO_SETTINGS_MODULE=config.settings.production
DJANGO_SECRET_KEY='Very-secure-secret-string'
DEBUG=0
DATABASE_URL=psql://postgres:postgres@txs-db:5432/postgres
ETH_L2_NETWORK=1
REDIS_URL=redis://txs-redis:6379/0
CELERY_BROKER_URL=amqp://guest:guest@txs-rabbitmq/
DJANGO_ALLOWED_HOSTS="*"
FORCE_SCRIPT_NAME=/txs/
# CSRF_TRUSTED_ORIGINS="http://localhost:8000"
CSRF_TRUSTED_ORIGINS="http://localhost:8000,http://<IP_ADDRESS>:8000"

Only CSRF_TRUSTED_ORIGINS is modified.

chain config:
{
  "count": 1,
  "next": null,
  "previous": null,
  "results": [
    {
      "chainId": "51",
      "chainName": "apothem",
      "shortName": "txdc",
      "description": "Xinfin Testnet Apothem",
      "l2": true,
      "rpcUri": {
        "authentication": "NO_AUTHENTICATION",
        "value": "https://erpc.apothem.network/"
      },
      "safeAppsRpcUri": {
        "authentication": "NO_AUTHENTICATION",
        "value": "https://erpc.apothem.network/"
      },
      "publicRpcUri": {
        "authentication": "NO_AUTHENTICATION",
        "value": "https://erpc.apothem.network/"
      },
      "blockExplorerUriTemplate": {
        "address": "https://explorer.apothem.network/address/{{address}}",
        "txHash": "https://explorer.apothem.network/txs/{{txHash}}",
        "api": "https://api-apothem.blocksscan.io/api?module={{module}}&action={{action}}&address={{address}}&apiKey={{apiKey}}"
      },
      "nativeCurrency": {
        "name": "TXDC",
        "symbol": "TXDC",
        "decimals": 18,
        "logoUri": "/media/chains/51/currency_logo.png"
      },
      "transactionService": "http://nginx:8000/txs",
      "vpcTransactionService": "http://nginx:8000/txs",
      "theme": {
        "textColor": "#ffffff",
        "backgroundColor": "#000000"
      },
      "gasPrice": [
        {
          "type": "fixed",
          "weiValue": "250000000"
        }
      ],
      "ensRegistryAddress": null,
      "recommendedMasterCopyVersion": "1.3.0",
      "disabledWallets": [],
      "features": [
        "CONTRACT_INTERACTION",
        "ERC1155",
        "ERC721",
        "SAFE_APPS",
        "SPENDING_LIMIT"
      ]
    }
  ]
}

This problem is solved by set enviroment ETHEREUM_NODE_URL for service txs-worker-contracts-tokens and txs-worker-notifications-webhooks in file docker-compose.yml.


New version: Mozilla.Thunderbird version 115.1.0

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/115733&drop=dogfoodAlpha

@vedantmgoyal2009,
Like in https://github.com/microsoft/winget-pkgs/pull/115186#issuecomment-1681114793, this package also looks like it is supposed to have numerous localization installers, but only the English installers are used.


Any method for converting to gguf?

Is there any method to convert Griffin models to gguf?
I want to quantize this model to q4_K type
Any kind of help is appreciated
Thanks

Our understanding is that the gguf format is used by llama.cpp quantize tool and our model isn't suitable in llama.cpp so I don't think this format will be useful to you for quantizing.

Thank you for your response, I know that it is not possible cause I tried it once, but I might look into the model let you know if I was able to quantize it. Because I want to use this model with ollama, that's why I was asking for gguf format
Anyways thank you sir

Sure. Note as per the discussion in https://huggingface.co/google/recurrentgemma-2b-it/discussions/6, we expect that the linear layers can be quantised but the recurrence might require some care.
We have seen that someone has produced a quantised version of the model but this wasn't done by us and we haven't evaluated their model or their approach but in case useful to you as a pointer - https://huggingface.co/PrunaAI/recurrentgemma-2b-it-bnb-4bit-smashed.


fix scrollToLocation not consistent between platforms

Summary
first, I edited this post so that the comment below are NOT relevant any more
problem: scrollToLocation behavior in SectionList on master is inconsistent between platforms.
Recently, there have been two PRs changing this function: https://github.com/facebook/react-native/pull/21577 and https://github.com/facebook/react-native/pull/24034
let me just quote the docs here

Scrolls to the item at the specified sectionIndex and itemIndex (within the section)

old behavior
originally, the code was let index = params.itemIndex + 1;
so if we call
    node.scrollToLocation({
      animated: false,
      itemIndex: 0,
      sectionIndex: 0,
      viewPosition: 0,
      viewOffset: 0,
    });

inside scrollToLocation, index is incremented and that results into the list scrolling to the first item (the first section heading is not visible - see the gif). If I specify itemIndex = -1, it will scroll all the way up. I was expecting itemIndex = 0 to bring me all the way up, so that the first section heading is visible. Not sure which way this should work but at least this behavior is in line with the docs.
new behavior
next, there was https://github.com/facebook/react-native/pull/21577 that changed it to
let index = Platform.OS === 'ios' ? params.itemIndex: params.itemIndex - 1;
so that https://github.com/facebook/react-native/issues/18098 is fixed. The PR had a bug and so next, there was https://github.com/facebook/react-native/pull/24034 that changed it to
let index = Platform.OS === 'ios' ? params.itemIndex : params.itemIndex + 1;
the reasoning was

Note however, how the sign for non iOS changed from + to - causing a crash on Android when trying to scroll to index 0 as that will be evaluated to -1 instead of 1 and thus be out of bounds.

with this change, the list does not crash. However, the behavior is not consistent between platforms (see gifs):
this PR
So, this PR works under the assumption that
    node.scrollToLocation({
      animated: false,
      itemIndex: 0,
      sectionIndex: 0,
      viewPosition: 0,
      viewOffset: 0,
    });

should scroll all the way up, so that even the first section heading is visible. This should keep this bug https://github.com/facebook/react-native/issues/18098 fixed. However, this change means different behavior from how it used to be and from how it is documented, which should be pointed out in release notes and docs need to be updated.
Finally, if we agree this is the way to go, it'd be nice to have some automated tests.
cc @melina7890 @danilobuerger
Changelog
[Android][Fixed] - fix scrollToLocation not consistent between platforms
Test plan
I used https://gist.github.com/vonovak/d85a95ac6bb189a8b5d1ef61c58e0406 (just copy-paste it to RNTesterApp.android.js/ios.js in the RNTester app)
gifs of master branch behavior:

gifs of this branch behavior (both behave the same, so just one gif)

However, it seems that the author failed to notice (?) that the list does not scroll to the very top, but to the second item in the list.

The intention of PR #21577 was, as far as I can tell, to change the behaviour for iOS, not modify it for Android. That is why my PR (#24034) reverted the behaviour on non-iOS to what it was before. I considered it to be out of scope for my PR to judge wether the previous behaviour was indeed correct or not. It was about reverting a obvious mistake (+ 1  vs - 1).

Just tested, I can confirm that this works as advertised.
However, unrelated to this specific change, I noticed during testing that when having stickySectionHeadersEnabled = true, the scrollToLocation will not account for the sticky header. I will open another PR with a bugfix for that.

@danilobuerger thanks for fats response

Just tested, I can confirm that this works as advertised.

just to be sure - you're talking about this PR fixing the issue of scrolling to top on android?

I noticed during testing that when having stickySectionHeadersEnabled = true, the scrollToLocation will not account for the sticky header

I think this is the intended behavior, if you take a look at https://facebook.github.io/react-native/docs/sectionlist#scrolltolocation the docs says

'viewOffset' (number) - A fixed number of pixels to offset the final target position, e.g. to compensate for sticky headers.

just to be sure - you're talking about this PR fixing the issue of scrolling to top on android?

Yes.

I think this is the intended behavior

However, viewOffset is not listed in the typing of scrollToLocation. Also, why do it manually when it can be accounted for automatically. But it was just a sidenote, I am going to open a PR for it, no point of discussing it here.

@vonovak you should edit the description based on what @pull-bot suggested so that someone can actually review and merge it.

the PR is ready, I rewrote the description so please keep in mind the above comments are not relevant any more
cc @cpojer since you merged the two PRs that changed this function recently


using PTVS to organize all python environments

Since PTVS has the ability to scan through my computer and find all of my python environments.  Would it be possible to make it generate an output that shows all of my environments, where they are stored.  What packages are installed and generate a report of this?  Even if it was a powershell script that would be great.  Especially, if I could specify multiple network drives to read through, the packages, and the windows environment that the python environment is identified as.
The reality is that for those of us doing python development in PTVS, we are also doing python development for windows deployment.  Being able to administrate, different programs, and evaluate the existing python libraries+Packages on a multiple computers is really valuable.

I would suggest looking at PEP 514, which defines the registration for Python interpreters. Currently the only ones we detect that are not registered like this are Canopy and IronPython - I believe Canopy has its own management tools, and IronPython is simply a different registry key right now. We don't do any disk searching or other analysis beyond looking in the registry ("In the face of ambiguity, resist the temptation to guess").
Once you've got the executable paths, the packages can be obtained using pip. All we do in Visual Studio is run pip list and parse the output.
I agree that it's valuable, but we haven't really looked at addressing the needs of system administrators on this level. Right now it's unlikely that we'll get to it any time soon, so if you'd benefit from it, I'd say dive in and don't wait for us :)

FWIW, I started work on a Python package pep514tools that can do this. It doesn't have all the functionality yet, but I'd much rather provide the tools as a Python library first.

Marking this wontfix - hopefully the external tools are sufficient (the py.exe launcher is also gaining a -0 option that will do this, so there's very little motivation to add it to VS).


Allow users to type in f(x) = x^2

Catch  the function notation... g(x) h(x) y(x) c(x) p(x)... don't catch x(x).
Maybe down the line detect the variable too h(t)  f(u) and so on.

This is definitely related to the y = issue and affects calculus.  We need a better design to work with the userFunction whether or not = is typed.    No equal userFunction, y= userFunction or h(x) = userFunction.
Maybe a method getUserFunction(currEntry) that takes in an Entry and returns a nice well formed userFunction... default to some sentinel (null or something) when unable to resolve.  To be used for graphing and evaluating.  I suppose it could return an object:
{
name: f,
var: x,
userFunction: function(){...}
}

I implemented getUserFunction in the mathConvert.js file and added the ability to graph using function notation and evaluating too.  This needs thorough testing ... but it is a start.


fix(api): add default values to optional fields of QUADRANT configuration

Closes RQA-2992
Overview
The QuadrantNozzleLayoutConfiguration has two optional fields- frontRightNozzle and backLeftNozzle, but they didn't have a default value assigned. So when the app tries to fetch the run's data, if the RunStore reads the following valid command from the database-
{
  "id": "44c2ebda-4e81-42f4-bbd7-91710ca30242",
  "createdAt": "2024-08-13T20:00:19.662654+00:00",
  "commandType": "configureNozzleLayout",
  "key": "2a57aa0e5f1e286b479c11d95e2cce35",
  "status": "succeeded",
  "params": {
    "pipetteId": "984f1433-acfd-48e7-b213-3c6fdfd85321",
    "configurationParams": {
      "style": "QUADRANT",
      "primaryNozzle": "H1",
      "backLeftNozzle": "B1"
    }
  },
  "result": {},
  "startedAt": "2024-08-13T20:00:19.664360+00:00",
  "completedAt": "2024-08-13T20:00:19.676488+00:00",
  "notes": []
}

..then pydantic runs into a validation error since it doesn't know how to parse configurationParams with a missing frontRightNozzle.
This was resulting in the 'Download Run Log' option to give a 500 error.
Once default values are set for the fields, pydantic is able to parse the params correctly and run log downloads without any issues.
Test Plan and Hands on Testing

[ ] Run the below protocol on a robot
[ ] Close the run (this is important)
[ ] Download run log from app. This should succeed

Protocol for testing:
from opentrons.protocol_api import PARTIAL_COLUMN

metadata = {
    "protocolName": "8 Channel SINGLE Happy Path A1 or H1",
    "description": "Tip Rack South Clearance for the 8 Channel pipette and a SINGLE partial tip configuration.",
}

requirements = {
    "robotType": "Flex",
    "apiLevel": "2.20",
}

def run(protocol):
    trash = protocol.load_trash_bin("A3")  # must load trash bin
    thermocycler = protocol.load_module("thermocycler module gen2")
    partial_tip_rack = protocol.load_labware(
        load_name="opentrons_flex_96_tiprack_50ul",
        label="Partial Tip Rack",
        location="B3",
    )
    pipette = protocol.load_instrument(instrument_name="flex_8channel_50", mount="right")
    # mount on the right and you will get an error.
    pipette.configure_nozzle_layout(
        style=PARTIAL_COLUMN,
        start="H1",
        end="B1",  # 2 Tips
        tip_racks=[partial_tip_rack],
    )
    source_labware_B2 = protocol.load_labware(
        load_name="nest_96_wellplate_100ul_pcr_full_skirt",
        label="B2 Source Labware",
        location="B2",
    )
    destination_labware_C2 = protocol.load_labware(
        load_name="nest_96_wellplate_100ul_pcr_full_skirt",
        label="C2 Destination Labware",
        location="C2",
    )
    volume = 10  # Default volume for actions that require it
    pipette.transfer(volume, source_labware_B2["A6"], destination_labware_C2["A6"])

Review requests
It's been a while since I looked at the partial tip code so want to make sure that parts of the code that consume quadrant configuration know to verify the presence of at least one of frontRightNozzle or backLeftNozzle and to handle the case where neither is specified.
The alternative to this approach would be to make the two fields non-optional and make the python API extract the info of the missing field before calling the engine command.
Risk assessment
None. Bug fix.

The alternative to this approach would be to make the two fields non-optional and make the python API extract the info of the missing field before calling the engine command.

Talked with @CaseyBatten and it's looking like this is the better approach. Going to check the work involved and make the changes so hold off on reviewing.

Opening a new PR since the fix approach changed


getting number of iterations from lgb.cv

I am running 200 trials of a hyperpram search using Optuna with lightgbm.  The call within Objective looks like this
    def f1_eval(preds,dtrain):
      preds = preds.reshape(self.nclass,-1).T
      f_score = f1_score(dtrain.get_label(),preds.argmax(axis=1),average="micro")
      return "f1_score",f_score,True

    early_stop = lgb.early_stopping(20, first_metric_only=False, verbose=False)
    eval_hist = lgb.cv(lgb_prm, dtrain, seed=12345, num_boost_round=200,
                            feval=f1_eval, callbacks=[early_stop])
    return eval_hist["f1_score-mean"][-1]

Later based on the "best" set of params I am retraining one final model as shown below.
    model_init = lgb.train(lgb_prm, dtrain, num_boost_round=50, feval=None)
    early_stop = lgb.early_stopping(20, first_metric_only=False, verbose=False)
    model = lgb.train(lgb_prm, dtrain, valid_sets=[dvalid], num_boost_round=150,
                          init_model=model_init, feval=f1_eval, callbacks=[early_stop])

By default, lgb.cv using 5-way x-val and my train/valid split has similar sizes.  I am requiring my final training run to go at least 50 iterations before early_stop can be invoked!
Is there a way to get the average number of iterations (based on the early stop) for the 5 x-val runs so that I can use that instead of the 50?

Looks like I can get this info by setting return_cvbooster=True and this to record as part of the trial trial.set_user_attr("niter", np.sort(eval_hist["cvbooster"].num_trees())[3]).  I am finding it odd that niter is coming out to be 293 even though num_boost_round=200.

Hi @ironv, thank you for your interest in LightGBM. After you set return_cvbooster=True you can access the best iteration as eval_hist["cvbooster"].best_iteration, because the num_trees() method will return the total number of iterations instead of the best one, i.e. will not take into account the early stopping.
Please let us know if this helps.


[prototype] sidebar rewrite in react

Not for review, just posting here to have something to link other devs to
This will be broken down, polished and submitted in small chunks

All separated out into separate PRs now


Pressing enter in MS Edge works as if backspace was pressed

When editing text, inserting a line-break using the return key on the keyboard does not work. Instead, the last typed character is deleted (just like as if backspace key was pressed).

Released new version v1 🎉


added myetherwallet-kr.com

Reported via FB message

no response. Either already down, or user submitted with a typo. https://urlscan.io/result/8fe10257-ed3e-424a-9814-99ce67cec133#summary
🤷


ERROR:   Failed to compile.

Failed to compile.
[copy-webpack-plugin] unable to locate 'C:\laragon\www\vuejs\vuex-master\static' at 'C:\laragon\www\vuejs\vuex-master\static'

Have you tried creating a 'static' folder? It worked for me.


Add transform to make population an integer

Refs: https://github.com/mapzen/vector-datasource/issues/230
Connects with: https://github.com/mapzen/vector-datasource/pull/416
@zerebubuth: could you review please?

:+1: looks good to me!


how to read runtime property? ex, port

I want to send port info to logstash, how can i do it?

See the Custom Fields section of the readme for information about how to add fields to the JSON output for a log event.

the port in the above, is not my Mservice port; can I overidde it ?


feature(android): adding android app shell!

The type of this PR is: Feature

This PR resolves [869]
Description

[X] A yarn command to launch the app.
yarn android, you could also run yarn start before that for having the two running separately.
[X] A yarn command to clear the android build cache.
yarn clean, yarn clean-ios, yarn clean-android.
[X] Adding the cache clear command to the flip_table command in the Makefile.
[X] A custom artsy app icon.
[X] The ability to load config from our .env.shared file.
[X] The ability to render a view defined in TypeScript in our src/ directory.
AndroidApp.tsx
[X] It should be possible to launch the development build of the app both in a simulator and on an android device.

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

Nice! thanks!

Nice! thanks!

Nice! thanks!

Nice! thanks!

🎉 🎉 🎉 We should probably update the README with Android instructions, right?

🎉 🎉 🎉 We should probably update the README with Android instructions, right?

yup. after the build setup is done, so we know what is needed exactly 👍.

yup. after the build setup is done, so we know what is needed exactly 👍.


Need to abandon middleware in sub mux

@zenazn
So while I was messing with a logging middleware, I was printing out middleware.Pattern(ctx) in the log as the url that was hit, but when this is set on the root mux and you have a sub-mux, you'll never get the right pattern.
example:
root := goji.NewMux()
users := goji.SubMux()

//setup logging at the root mux
root.UseC(NewLoggerMiddleware())

root.HandleFunc(pat.Get("/my-app"), IndexHandler)
root.Handle(pat.Get("/my-app/*"), subRouter)

subRouter.HandleFuncC(pat.Get("/some-route"),SomeHandler)

// logging middleware does something like this:
fmt.Println(middleware.Pattern(ctx)

When you call /my-app, the log prints "/my-app". that's fine
But when you hit /my-app/some-route" the log will always print "/my-app/*". This is because the logger was added on the root mux and that's the pattern that was matched on the root mux. That makes sense.
Unfortunately, If I also add the logger to the sub-mux, I'll get 2 log entries for every request, which is not what I want.
I should be able to do:
subRouter.Abandon(root.NewLoggerMiddleware)
subRouter.UseC(NewLoggerMiddleware)

So, some of the time I have middleware that's truly global and I don't care if it runs on the root context, but other times, I want to have a "sepecial" instance of the middleware for a sub-mux that overrides the instance on the root mux

Actually, I was able to work around my issue by using a couple more muxs.

I know you've found a workaround, but a brief comment on something you mentioned:
subRouter.Abandon(root.NewLoggerMiddleware)
subRouter.UseC(NewLoggerMiddleware)

Unfortunately, this can't possibly work: root.NewLoggerMiddleware has already happened by the time subRouter gets a chance to look at the request. The rough mental model here is that sub Muxes are not special at all—they are just Handlers like everything else—and that middleware always happens immediately before Handlers execute.

@jdoklovic, can you describe your workaround? We are having the same issue.

@troylelandshields The work around was to expand everything into their own routers and then apply logging middleware to each sub router. The root router doesn't have logging middleware at all and also has no actual routes other than mapping patterns to sub routers


Add config option to generate minify static pages

It would be good if when generating the static pages we could pass the --minify flag to it in order to optimize the generated files.

Thanks for the suggestion @gbrlmrllo , I have now added this feature in v1.4.0 👍

ohh that's was quick, thanks!!!


Start rke2-server for the first time with an empty config.yaml

What type of PR is this?

[X ] bug

What this PR does / why we need it:
When starting the rke2-server service for the first time, and --profile=cis-1.5 or --profile=cis-1.6 is specified, the auto-generated TLS certificates will have startBefore timestamps in the future which prevents the rke2-server service from starting. For more information, see https://github.com/rancher/rke2/issues/1702
As a workaround to this issue, the rke2-server service is started before the rke2_config values are persisted to config.yaml. Once the TLS certificates are initialized with an empty config.yaml, the rke2_config values are persisted and a second restart is applied.
Which issue(s) this PR fixes:
No issue has been created
Testing
Manually tested with the following configuration
rke2_config:
  profile: cis-1.6

Release Notes
Start rke2-server for the first time with an empty config.yaml to ensure valid startBefore timestamps in the auto-generated TLS certificates

I found the root cause: the hardware clock was 2 hours in the future, and the first ntpd sync after reboot occurred after starting rke2-server for the first time. The solution was to synchronize the hwclock.


Update Docker container for Identity tests

Updates the Docker container for Identity tests.
Note: Some of the nightly tests are failing, but they are orthogonal to these changes. The relevant pipeline for this change is here. Prior to this container update, the Build & Test step failed just trying to run dotnet commands.

@weshaggard @benbp - Are there any outstanding concerns that I should address before you are comfortable approving?

/azp run net - identity - tests

/azp run net - identity - tests

/azp run net - identity - tests

/azp run net - identity - tests


Vte based terminals fail to load the image if it didn't exist before the script was run.

If the image isn't in the temp folder then vte based terminals fail to display it. The image is generated successfully but isn't displayed until you run the script a second time.
I don't know what's causing this as it also happens when text is disabled entirely.

I'm no longer able to reproduce this. If you're still having this problem, open another issue.


Is verkey expansion implemented correctly?

If I resolve did:indy:sovrin:WRfXPg8dantKVubE3HX8pw (see Universal Resolver), then I get this verification method in the DID document:
  "verificationMethod": [
    {
      "controller": "did:indy:sovrin:WRfXPg8dantKVubE3HX8pw",
      "id": "did:indy:sovrin:WRfXPg8dantKVubE3HX8pw#verkey",
      "publicKeyBase58": "WRfXPg8dantKVubE3HX8pwP7F3BNs5VmQ6eVpwkNKJ5D",
      "type": "Ed25519VerificationKey2018"
    }
  ]

It seems the expanded verkey is simply the "id" string value "WRfXPg8dantKVubE3HX8pw" plus the "verkey" string value "P7F3BNs5VmQ6eVpwkNKJ5D". I think this is implemented here:
https://github.com/IDunion/indy-did-resolver/blob/b26f76501c609762202865affae04639ac01b62b/indy-didresolver/src/did_document.rs#L64
But I'm not sure the implementation is correct. I don't think the base58-encoded strings themselves should be concatenated. Instead, the "id" and "verkey" strings should be base58-decoded, then the byte arrays should be concatenated, and then the combined byte array should be base58-encoded again. The result is different.

Afaik that is the correct implementation for the current (indy-node 1.12) implemenation - i do believe this changes with did:indy:
indyscan shows (https://indyscan.io/tx/SOVRIN_MAINNET/domain/18) the same verky WRfXPg8dantKVubE3HX8pwP7F3BNs5VmQ6eVpwkNKJ5D:
{
  "txn": {
    "data": {
      "dest": "WRfXPg8dantKVubE3HX8pw",
      "verkey": "~P7F3BNs5VmQ6eVpwkNKJ5D",
      "verkeyFull": "WRfXPg8dantKVubE3HX8pwP7F3BNs5VmQ6eVpwkNKJ5D"
    },
    "metadata": {
      "digest": "618ed1f21281f13cb769fe17308fd75279b22c41e8004280fb97170a4eb6282a",
      "from": "BrYDA5NubejDVHkCYBbpY5",
      "reqId":<PHONE_NUMBER>982387
    },
    "type": "1",
    "typeName": "NYM"
  },
  "txnMetadata": {
    "seqNo": 18,
    "txnId": "d8f845175979a23b35235c097ed20d564dc8ad6a5ef7cc7f19b53e838386012d",
    "txnTime": "2017-07-31T17:38:53.000Z"
  }
}

Both indyscan and this indy-did-resolver driver here are incorrect. The expanded verkey is the concatenation of the DID bytes and the shortened verkey bytes, not the concatenation of the Base58 encoded strings.
Take for example the well-known "000000000000000000000000Trustee1" seed that is a default value in some sample deployments. With this seed you get the following, all in Base58 encoding:
did: V4SGRU86Z58d6TV7PBUe6f
private key: GJ1SzoWzavQYfNL9XkaJdrQejfztN4XqdsiV4ct3LXKL
verkey: ~CoRER63DVYnWZtK8uAzNbx
wrong expanded verkey: V4SGRU86Z58d6TV7PBUe6fCoRER63DVYnWZtK8uAzNbx
correct expanded verkey: GJ1SzoWzavQYfNL9XkaJdrQejfztN4XqdsiV4ct3LXKL
Now try any arbitrary Ed25519 implementation, try to sign a string "foo" with the private key above, and check which expanded verkey can be used to verify the signature. You will find it's the latter.

Apparently this has already been raised in indyscan: https://github.com/Patrik-Stas/indyscan/issues/52

Hmm interesting, should be automatically fixed once the indy-vdr did:indy branch is released and we switch to that (or rather indy-vdr-proxy) for the driver - I will check the implementation there but that should be correct. I will create a small fix for this repository for the time being.
Good catch!

There's a very good chance that I just used this implementation here for indy-vdr, since we started with this project and then moved stuff to indy-vdr.

There's a very good chance that I just used this implementation here for indy-vdr, since we started with this project and then moved stuff to indy-vdr.

Yep, that part also exists in the feature branch of indy-vdr. I will create a fix here and create a PR to the did-indy branch on indy-vdr.
I am currently looking at the code in indy-sdk and trying to understand where this comes from: https://github.com/hyperledger/indy-sdk/blob/main/libindy/src/utils/crypto/verkey_builder.rs#L8-L13
I have never seen a verkey that carries some form of crypto identifier with it. Does anyone of you have any idea if this is currently used anywhere / still relevant? I guess it doesn't hurt to also include this but I was kind of puzzled where this comes from.

I have never seen a verkey that carries some form of crypto identifier with it. Does anyone of you have any idea if this is currently used anywhere / still relevant? I guess it doesn't hurt to also include this but I was kind of puzzled where this comes from.

Same here, I've never seen anything like this, but would also agree that it probably wouldn't hurt to include it.

The new version should work correctly and I created a new release / docker image. Does https://dev.uniresolver.io/ pull latest from time to time or do we have to upgrade this somehow @peacekeeper ?

@c2bo Thanks for fixing! The new version is now deployed at https://dev.uniresolver.io/, looks good.


Enable leafrefs in ietf-alarms

When YANG leafref datatype was not supported, all leafrefs in
ietf-alarms were commented and used string as type. The patch undoes
that change, resetting ietf-alarms to its original state.

To test it I informally run a lwaftr.conf that raises one alarm and query it with alarms get-state.
...
  alarm-list {
    alarm {
      alarm-type-id arp-resolution;
      alarm-type-qualifier '';
      resource 32684;
      alarm-text "Make sure you can resolve external-interface.next-hop.ip address manually.  If it cannot be resolved, consider setting the MAC address of the next-hop directly.  To do it so, set external-interface.next-hop.mac to the value of the MAC address.";
      is-cleared false;
      last-changed 2018-06-13T13:55:49Z;
      perceived-severity critical;
      status-change {
        time 2018-06-13T13:55:49Z;
        alarm-text "Make sure you can resolve external-interface.next-hop.ip address manually.  If it cannot be resolved, consider setting the MAC address of the next-hop directly.  To do it so, set external-interface.next-hop.mac to the value of the MAC address.";
        perceived-severity critical;
      }
      time-created 2018-06-13T13:55:49Z;
    }
    last-changed 2018-06-13T13:55:49Z;
    number-of-alarms 1;
  }
  summary {
    alarm-summary {
      cleared 0;
      cleared-closed 0;
      cleared-not-closed 0;
      not-cleared-closed 0;
      not-cleared-not-closed 1;
      total 1;
    }
  }


change connector schedule_type if operator manual parameter conflicts

We were confused when testing the FivetranOperator because the manual param defaults to True... we weren't expecting it to remove the sync settings from the Fivetran UI and had to dig around in the code to find the API call to patch the connector schedule type.
The idea of this PR is to make the Operator flip the schedule type if the manual parameter differs from the current connector setting.

@iamepps I can merge and make these changes too if that would be easier, thanks again for the pr!

Hey Nick. I was just looking at this and remembered we had a discussion about it in our team... we tried to keep the manual param as a bool to avoid breaking changes to the Operator.
If you're okay with that then I've pushed the change above with some docstring updates.
If we wanted to avoid potentially breaking changes we could keep the manual parameter and have a ternary statement that converts it into the relevant string. It does pin you down to either manual or something else though, so not ideal.

... also realised it's worth validating the input if using changing the param to a string.
It's late here (UK) and I've not had a chance to test - I will do tomorrow if you don't.

@iamepps this is working great for me! It'll be up on pypi soon for airflow-provider-fivetran 1.0.4, I'll let you know when that is out


new Data from Luxembourg is available

Thank you for the data update! I somehow missed the mail notification for your PR, so this slipped under my radar.


Job log generation crashes, no job log output in joblog.json "TypeError: 'NoneType' object is not iterable"

I am getting a python type error when running builds in our corporate dev environment.  At the end of the build there is nothing in joblog.json, and the bottom of output.log has the bellow errors.
Environment:  local windows development with vscode, deploying to ifs and building with Bob.

                    * * * * *   E N D  O F  C R E A T E  S E R V I C E  P R O G R A M  L I S T I N G   * * * * *
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

Traceback (most recent call last):
  File "/QOpenSys/pkgs/lib/bob/scripts/getJobLog", line 38, in <module>
    cli()
  File "/QOpenSys/pkgs/lib/bob/scripts/getJobLog", line 34, in cli
    save_joblog_json(args.cmd, args.cmdtime, args.jobid, args.f)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 98, in save_joblog_json
    records = get_joblog_for_job(jobid)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 93, in get_joblog_for_job
    joblog_dict = query_job._dump_results_to_dict(results)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 63, in _dump_results_to_dict
    records, column_names = results
TypeError: 'NoneType' object is not iterable
/QOpenSys/pkgs/lib/bob/scripts/launch: line 21: mktemp: command not found
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
CPC2198: Current library changed to RDGJWEIRIC.
ACMSLIB/ACMSLIBL REL(NELNET/GNF/GNFCORE) ENV(DVP) DVPNAME(JWEIRIC9)
CPD000D: Command ACMSLIB/ACMSLIBL not safe for a multithreaded job.
/QOpenSys/pkgs/lib/bob/scripts/launch: line 42: : No such file or directory
/QOpenSys/pkgs/lib/bob/scripts/launch: line 43: : No such file or directory
>> executing command: DLTOBJ OBJ(RDGJWEIRIC/BNDSTM2) OBJTYPE(*BNDDIR)
sed: The label :a;N;$!ba;s/\n/ /g is greater than eight characters.
>> liblist:
>> executing time: 2022-11-27-15.18.17.%N

<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

Traceback (most recent call last):
  File "/QOpenSys/pkgs/lib/bob/scripts/getJobLog", line 38, in <module>
    cli()
  File "/QOpenSys/pkgs/lib/bob/scripts/getJobLog", line 34, in cli
    save_joblog_json(args.cmd, args.cmdtime, args.jobid, args.f)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 98, in save_joblog_json
    records = get_joblog_for_job(jobid)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 93, in get_joblog_for_job
    joblog_dict = query_job._dump_results_to_dict(results)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 63, in _dump_results_to_dict
    records, column_names = results
TypeError: 'NoneType' object is not iterable
/QOpenSys/pkgs/lib/bob/scripts/launch: line 21: mktemp: command not found
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
CPC2198: Current library changed to RDGJWEIRIC.
ACMSLIB/ACMSLIBL REL(NELNET/GNF/GNFCORE) ENV(DVP) DVPNAME(JWEIRIC9)
CPD000D: Command ACMSLIB/ACMSLIBL not safe for a multithreaded job.
/QOpenSys/pkgs/lib/bob/scripts/launch: line 42: : No such file or directory
/QOpenSys/pkgs/lib/bob/scripts/launch: line 43: : No such file or directory
>> executing command: CRTBNDDIR BNDDIR(RDGJWEIRIC/BNDSTM2)
sed: The label :a;N;$!ba;s/\n/ /g is greater than eight characters.
>> liblist:
>> executing time: 2022-11-27-15.18.19.%N

CPD000D: Command *LIBL/CRTBNDDIR not safe for a multithreaded job.
CPC5D02: Binding directory BNDSTM2 created in library RDGJWEIRIC.
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

Traceback (most recent call last):
  File "/QOpenSys/pkgs/lib/bob/scripts/getJobLog", line 38, in <module>
    cli()
  File "/QOpenSys/pkgs/lib/bob/scripts/getJobLog", line 34, in cli
    save_joblog_json(args.cmd, args.cmdtime, args.jobid, args.f)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 98, in save_joblog_json
    records = get_joblog_for_job(jobid)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 93, in get_joblog_for_job
    joblog_dict = query_job._dump_results_to_dict(results)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 63, in _dump_results_to_dict
    records, column_names = results
TypeError: 'NoneType' object is not iterable
/QOpenSys/pkgs/lib/bob/scripts/launch: line 21: mktemp: command not found
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
CPC2198: Current library changed to RDGJWEIRIC.
ACMSLIB/ACMSLIBL REL(NELNET/GNF/GNFCORE) ENV(DVP) DVPNAME(JWEIRIC9)
CPD000D: Command ACMSLIB/ACMSLIBL not safe for a multithreaded job.
/QOpenSys/pkgs/lib/bob/scripts/launch: line 42: : No such file or directory
/QOpenSys/pkgs/lib/bob/scripts/launch: line 43: : No such file or directory
>> executing command:  ADDBNDDIRE BNDDIR(RDGJWEIRIC/BNDSTM2)  OBJ((*LIBL/SLA6UPR *SRVPGM) (*LIBL/SLA1UPR *SRVPGM) (*LIBL/SLA3UPR *SRVPGM)  (*LIBL/SLA4UPR *SRVPGM) (*LIBL/SLA5UPR *SRVPGM) (*LIBL/SLB2SRV *SRVPGM)  (*LIBL/SLA7UPR *SRVPGM) (*LIBL/SLA8UPR *SRVPGM) (*LIBL/SLJ1UPR *SRVPGM)  (*LIBL/SLA9UPR *SRVPGM) (*LIBL/UPD_SRV1 *SRVPGM) (*LIBL/SLJ2UPR *SRVPGM)  (*LIBL/SLJ4UPR *SRVPGM) (*LIBL/SLJ5UPR *SRVPGM) (*LIBL/SLA2UPR *SRVPGM) (*LIBL/SLJ8UPR *SRVPGM) (*LIBL/SLJ9UPR *SRVPGM) (*LIBL/SLP2UPR *SRVPGM))
sed: The label :a;N;$!ba;s/\n/ /g is greater than eight characters.
>> liblist:
>> executing time: 2022-11-27-15.18.22.%N

CPD000D: Command *LIBL/ADDBNDDIRE not safe for a multithreaded job.
CPC5D05: 18 entries added, 0 entries not added to Binding directory BNDSTM2 in library RDGJWEIRIC.
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

Traceback (most recent call last):
  File "/QOpenSys/pkgs/lib/bob/scripts/getJobLog", line 38, in <module>
    cli()
  File "/QOpenSys/pkgs/lib/bob/scripts/getJobLog", line 34, in cli
    save_joblog_json(args.cmd, args.cmdtime, args.jobid, args.f)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 98, in save_joblog_json
    records = get_joblog_for_job(jobid)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 93, in get_joblog_for_job
    joblog_dict = query_job._dump_results_to_dict(results)
  File "/QOpenSys/pkgs/lib/bob/makei/ibm_job.py", line 63, in _dump_results_to_dict
    records, column_names = results
TypeError: 'NoneType' object is not iterable``

Note that this happens when I invoke Bob from VSCODE, developing locally.  When I ran 'makei build' in an ssh session, I did not get this error, and the joblog was created.

contents of my .profile:

Not sure if VSCODE is using this environmental variable or not.  Let me dig into the docs a bit.

I'm thinking it must not be using my path - When I start a PASE shell from within VSCODE, it has no problem finding the command.

@JDubbTX Correct. When Actions are run, it is run in a non-interactive shell.
https://www.vanimpe.eu/2014/01/18/different-shell-types-interactive-non-interactive-login/

Ok, then I'm not sure how to set a path in the non interactive shell command.

@JDubbTX if bash is your default shell, look into using a .bashrc to setup the path.

I assume this is now resolved