drop dependency on iproute2 binary and libraries

Fixes https://github.com/istio/istio/issues/35356
Currently, distroless images do not have iproute2 binary. This means
ipv6 or tproxy are 100% broken on distroless.

One option is to go add the libraries to distroless. However, they are
*massive*, and sort of defeat the purpose of distroless. The
`github.com/vishvananda/netlink` library is already a common library
used by our dependencies (so we are moving from indirect -> direct
dependency) that configures the same underlying NETLINK configurations
that the `ip` command does, but with pure go code in a small dependency.

This updates all of our usages of `ip` to use this library. With this,
tproxy and ipv6 both work with distroless images. While we currently do
not have CI testing of this, I am working on it (this is a prereq).

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

This is WIP just until https://github.com/istio/istio/pull/35359 merges to split up the PR a bit

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
b1859fac09c87cfebc9000e7b1b5fa73b0e4acf5
link
true
/test lint_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
b1859fac09c87cfebc9000e7b1b5fa73b0e4acf5
link
true
/test lint_istio

gencheck_istio
b1859fac09c87cfebc9000e7b1b5fa73b0e4acf5
link
true
/test gencheck_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
b1859fac09c87cfebc9000e7b1b5fa73b0e4acf5
link
true
/test lint_istio

gencheck_istio
b1859fac09c87cfebc9000e7b1b5fa73b0e4acf5
link
true
/test gencheck_istio

integ-cni-k8s-tests_istio
b1859fac09c87cfebc9000e7b1b5fa73b0e4acf5
link
true
/test integ-cni-k8s-tests_istio

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

@howardjohn: PR needs rebase.

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

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni-k8s-tests_istio
61e241650890909202639475c0f44be3b5962777
link
true
/test integ-cni-k8s-tests_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-mc-k8s-tests_istio
92d586d75b746d3dcfb5c36bb158b1eaf604a2bd
link
true
/test integ-telemetry-mc-k8s-tests_istio

unit-tests_istio
92d586d75b746d3dcfb5c36bb158b1eaf604a2bd
link
true
/test unit-tests_istio

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

/retest

Generally LGTM.
The missing piece is the fwmark for ipv6. See below. The ipv4 and ipv6 iptables are independent
Feel free to ignore if the missing ipv6 is not a regression.
$ sudo ip rule add fwmark 123 lookup 123
$ sudo ip rule
0:      from all lookup local
32765:  from all fwmark 0x7b lookup 123
32766:  from all lookup main
32767:  from all lookup default
$ sudo ip -6 rule
0:      from all lookup local
32766:  from all lookup main

Not a regression. Will fix anyways


Added new integration - Sumo Logic Automation Tools

Purpose of this pull request
This pull request...
Select the type of change

[ ] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions, updating sections
[x] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

@rishav-sumo-dev, this PR is a draft. When it's ready for review, click the Ready for review button.


ci: macos-latest runner don't have python 3.6 and 3.7

Se don't bother, stop testing that. We continue testing on Ubuntu.

Including in #155


Failed to load the datafile

When running the 1st example from the Genie docs -- demo1_harness_simple_job.py  -- Im getting an error about loading a datafile.
(base) root@9d5025edcdf5:/opt/examples/libraries/harness_simple# pyats run job demo1_harness_simple_job.py --testbed-file eveng-lab-001.yml
.
.
2020-02-26T22:18:18: %EASYPY-ERROR: pyats.aetest.exceptions.DatafileError: Failed to load the datafile '/opt/conda/lib/python3.7/site-packages/genie/libs/sdk/genie_yamls/nxos/trigger_datafile_nxos.yaml'

However I can confirm this file is present.
ls -l /opt/conda/lib/python3.7/site-packages/genie/libs/sdk/genie_yamls/nxos/trigger_datafile_nxos.yaml

Am i missing something simple?
Version Info:
 python
Python 3.7.4 (default, Aug 13 2019, 20:35:49)
[GCC 7.3.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>>
(base) root@9d5025edcdf5:/opt/examples/libraries/harness_simple# pip freeze | grep -E "genie|pyats"
genie==20.2
genie.libs.conf==20.2
genie.libs.filetransferutils==20.2
genie.libs.ops==20.2
genie.libs.parser==20.2
genie.libs.robot==20.2
genie.libs.sdk==20.2
genie.telemetry==20.2
genie.trafficgen==20.2
pyats==20.2
pyats.aereport==20.2
pyats.aetest==20.2
pyats.async==20.2
pyats.connections==20.2
pyats.datastructures==20.2
pyats.easypy==20.2
pyats.kleenex==20.2
pyats.log==20.2
pyats.reporter==20.2
pyats.results==20.2
pyats.robot==20.2
pyats.tcl==20.2
pyats.topology==20.2
pyats.utils==20.2

Can you make sure you have a device with alias uut.
Can you also try
import yaml
with open('/opt/conda/lib/python3.7/site-packages/genie/libs/sdk/genie_yamls/nxos/trigger_datafile_nxos.yaml') as f:
    data = yaml.safe_load(f)

Does this work

Yep the devices have an alias of uut. This also creates the same error if you use the example testbed file and the mock device(s).
As for the code snippet, Ive run this and it runs fine.

this might be related to conda environment - how is this env created?

Discussed with @rickdonato  - One issue in non venv that was fixed, and confirmed it works fine with Docker / venv.


Company Page - Contract View - Per Project Billing - Design Change

Problem -
Billing Mode name change Required
Solution
Change to "Billing Event"

initiated by @vinpahilwani
verified by @mitulpanch

"No Service Found" to "No Service Attached"

Yes


[BUG]: Video pauses when alarm goes off, doesn't resume when alarm is dismissed.

Version
6.0.0-rc.1 - 6.4.2
What platforms are you having the problem on?
iOS
System Version
17.5.1
On what device are you experiencing the issue?
Real device
Architecture
Old architecture
What happened?
Playing a video when a timer or alarm goes off, or a phone call is received, the video pauses (expected), but does not resume when the timer/alarm is dismissed.
Reproduction
Step to reproduce this bug are:

Set a timer to go off in ~20 seconds
Play video
Timer goes off, with an IOS provided modal on top of the app. (Video pauses as expected)
Dismiss the modal, and the video does not resume.

Alternatively

Play Video
Receive phone call
Either send to voice mail or accept, but stay in the app playing the video (caller must hang up if you accepted the call)
Video does not resume

Additionally

This can reproduced on 6.0.0.rc1, 6.1.2, 6.3.0, and 6.4.2 (other versions I haven't tried).
Changing playInBackground, playWhenInactive, and ignoreSilentSwitch have no effect on the bug.
With a listener on onPlaybackStateChanged, an event with isPlaying: false is sent when the alarm goes off, but there is no event sent when the timer is dismissed
There are no events sent via onAudioFocuseChanged

Hi @ericschoettle, thanks for posting Issue! We'll try to fix it as soon as possible!


Chat screen is near impossible to click top navigation buttons.

Expected behavior
When viewing the chat page, I should be able to scroll up and down and continue to navigate using the buttons at the top of the page. Viewport should not be pushed down because of new messages loading.
Actual behavior
Once I open the page and scroll up, the backlog loads - this pushes the view down slightly so navigation buttons are off screen. I scroll up to see them, more backlog loads and it gets pushed away - over and over. On a low performance machine its really difficult to scroll and click before the viewport is moved.
How to reproduce
View chat page, scroll down, scroll up to the top of the page - https://busy.org/messages/general

Browser: Firefox 50.1.0
Operating system: Fedora 23 template inside QubesOS

The chat has been depreciated in new-design i'm closing this issue.


image: add image.ListFormats

This allows access to the list of currently registered format names.
This is useful for example when an image is available in multiple
formats to not have to open each format in order to find out which one
we are able to decode.

The len() check is there to prevent allocating an empty slice (make([]string, 0)) and instead return a nil slice, which makes more sense in case there is nothing to return.


BUG in ICA function find_bads_ecg

@agramfort  @dengemann
1.) When using the find_bads_ecg() function in ica.py with the arguments "start" and "stop" set to some ,  values and the "method" argument set to <method='CTPS'> the "start", "stop" arguments are not used at all, which in turn results in (too many) ecg events from the entire raw file
2.) in ctps_.py the estimated p value in function _prob_kuiper() get's easily close to zero (or 1 after normalisation) if high synchronization was detected AND a large number of trials (epochs) are used. To prevent for an under- or overflow (i.e., unrealistic perfect synchronization measurement with p=0.0 or pk=1.0) the np.exp() function in line 154 should use "float128". This is tolerable as the array is small. For this, the line should read np.exp(-2. * fact * j2 * l2, dtype='float128').
If you want us to do the changes, we are happy to do so.

Hi Jürgen!
Thanks for reporting. Your contribution is most welcomed and we’re happy to
help getting the changes in. It also sounds as if it wasn’t so difficult.
Denis
On Tue 21 Nov 2017 at 16:29, Juergen Dammers<EMAIL_ADDRESS>wrote:

@agramfort https://github.com/agramfort @dengemann
https://github.com/dengemann
1.) When using the find_bads_ecg() function in ica.py with the arguments
"start" and "stop" set to some , values and the "method" argument set to
<method='CTPS'> the "start", "stop" arguments are not used at all, which in
turn results in (too many) ecg events from the entire raw file
2.) in ctps_.py the estimated p value in function _prob_kuiper() get's
easily close to zero (or 1 after normalisation) if high synchronization was
detected AND a large number of trials (epochs) are used. To prevent for an
under- or overflow (i.e., unrealistic perfect synchronization measurement
with p=0.0 or pk=1.0) the np.exp() function in line 154 should use
"float128". This is tolerable as the array is small. For this, the line
should read np.exp(-2. * fact * j2 * l2, dtype='float128').
If you want us to do the changes, we are happy to do so.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/4768, or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fismtC0zFSbFEodv3xtx2OWK2SV4-ks5s4uxegaJpZM4QmBYh
.

yes please send us a PR.
thanks

OK, PR is coming very soon.


Why is this not in core?

Describe the solution you'd like
Is there any plan for this custom integration to make it into core one day?
Describe alternatives you've considered
I've looked through issues and discussion section to see if this has been asked before. I'm sure it must have been but I can't find it.
Additional context
Add any other context or screenshots about the feature request here.

Not at this moment 😃

It's all a bit finicky, so it will be hard to provide a stable UX.
It would require moving large parts of the code to a new package, as direct IO is not allowed in the integration code in HA.

Ok 👍

@HarvsG Unfortunately the finicky bits will continue as long as this uses an unofficial API in a way that was never intended

True that, thanks @DurgNomis-drol for breaking it. Also the initial reason to have it as a separate integration was to allow the team to iterate through releases faster :)


Else in StackRouter.getStateForParam is never going to be used

In the following code:
https://github.com/react-community/react-navigation/blob/9cda155dfba2023b26140668fbb42e1aa00122da/src/routers/StackRouter.js#L151-L170
The else block is never going to be executed, since the parent if requires the childRouter to be defined, or am I wrong?

Hi @JCMais! It looks like this is not a bug of the library nor a feature request, so I'll close it: I suggest next time you use Stack Overflow which is better suited for answering these types of questions. I also encourage you to join the Reactiflux community on Discord. There are React Native and React Navigation channels with helpful people who might be able to answer your question. I would also like to mention that there is a collection of helpful links and tutorials for React Navigation in the README which might be helpful for you.
If you believe that you are experiencing a bug or have a feature request, please open a new Issue following the dedicated template or submit a PR.

This is not a question, this is a possible bug with the library.


Price description for manage_buy_offer should be updated

I think the price description for manage_buy_offer and manage_sell_offer should be different.
For both it is currently Price of 1 unit of asset sold in terms of asset bought.
Shouldn't the the description for manage_buy_offer be Price of 1 unit of asset bought in terms of asset sold?

@poliha is correct. From https://www.stellar.org/developers/guides/concepts/list-of-operations.html#manage-buy-offer:

Price
{numerator, denominator}
Price of 1 unit of buying in terms of selling. For example, if you wanted to buy 30 XLM and sell 5 BTC, the price would be {5,30}.

This is effectively how the protocol is working.

@poliha want to make a PR to update that? Copy changes are pretty straightforward.

That change needs to be made here in GenericOffer

Fixed in #402

Thanks @vcarl for fixing this. Sorry was away and couldn't get to this.

Credit to @jeesunikim! No worries, just needed some time before we could get to it ourselves.


Save GHA job results as artifacts, for 20 days

Signed-off-by: Víctor Cuadrado Juan<EMAIL_ADDRESS>

Example of artifacts: https://github.com/viccuad/hypper/actions/runs/646741460.


Doubt

Good afternoon,
Sorry if I posted in the wrong place
I have a doubt..
What is the difference between the 2 unbound below?
https://hub.docker.com/r/klutchell/unbound
https://hub.docker.com/r/klutchell/unbound-dnscrypt

Great question! This is the current deploy location: https://hub.docker.com/r/klutchell/unbound
https://hub.docker.com/r/klutchell/unbound-dnscrypt is the old repo and is no longer updated. I could probably change my deploy tools to push to both but I would rather deprecate unbound-dnscrypt I think.


UI issues in host details page

Version - v1.0.1-rc7
-[ ] Drop-down no longer resizes to name of host

[ ] Container restart/menu buttons on top of each other instead of next to each other at bottom of page.

resolved in https://github.com/rancher/ui/pull/598

master 4-19
Not fixed

I'm not able to recreate stacked action menu issue

https://github.com/rancher/ui/pull/763

Version - master 7/8
Verified fixed


Won't load grunt tasks from included files.

All of my grunt tasks are kept in separate files (in the grunt directory), but the plugin won't acknowledge any of these tasks. I just get 'Grunt file parsed, found 0 tasks'

Does it work from the command line directly? Could you post your Gruntfile?

yeah works from the command line.
gruntfile.js
module.exports = function(grunt) {
        // measures the time each task takes
        require('time-grunt')(grunt);

            // load configs
        require('load-grunt-config')(grunt);
};

Can confirm the same, appears not to work with the load-grunt-config way of creating Grunt tasks, my grunt file is pretty similar:

Bizarre. Grunt-runner uses your local installation of grunt to parse tasks and reads directly from it, so I wouldn't think requiring within the function would matter unless there's something asynchronous going on. I'll play around a bit with it this weekend to try to see what's up.

This is still an issue. Any news?

Still an issue and somehow might be related to this one #90.


[WIP] widgets: Polish poll widget.

I added this PR mainly to familiarize myself with the widget structure, so that I can work on a TODO widget. This is still WIP.
Tasks:

[x]  Add basic styling for poll widget
[x] Add question in the widget itself

commit 1:
Before and After styling for poll widget:

commit 2
Add question in the widget itself

Looks good so far, can you rebase to get the build passing?  I think I fixed the test flake.

@showell Updated this!

The first commit looks great!
For asking the question, do we want to restrict this feature to the person that created the poll?
Also, I think it will be useful to say "/poll What do you want for dinner?" so that the question can be created from the compose box.

Hmm, now that I think about it restricting doesn't seem right. Also, /poll What do you want for dinner? sounds good. Maybe we can have both: If the question is not mentioned, we prompt using the input box, and if it is mentioned, we should it directly. What are your thoughts?

I provided a bit of feedback on chat:

structure code more to clearly divide the is-my-poll and is-not-my-poll logic
add comments about why we show disabled input boxes
squash two commits adding poll

The build seems to be failing here.  It may be a test flake.

You can probably still squash the questions changes a bit more, so that this PR just has one commit related to the UI side of adding question (so basically squash the "Restrict poll question..." into the prior commit).  I'd still leave the other commit by itself (the one that parses what you typed in the compose box).

@showell the build is passing now.

I tried testing this out, and I got some errors.  Sorry, I don't have the errors handy, but basically extra_data was null and it confused some JS module.  I think you can reproduce this as follows:

go to master
make some widgets (poll, tictactoe, etc.)
go to your branch
run the app

I'd also re-work the last commit a bit.  I think you can take that function I gave you and extend it to return both widget_type and extra_data as a tuple, so that all the parsing more or less happens up front.  And it will return (None, None) if it's not a slash command.  I think this will be slightly cleaner and may fix the bug.

@showell Thanks for pointing that bug out. It was basically being caused by no question being set in from the extra_data of the polls started in the master. I have added these lines to fix that. Also refracted the last commit.

Merged, thanks @rheaparekh!  I really like the experience here for the "other" people.  For the person creating the poll, I could see us getting some feedback, like maybe we should leave the question in the input box next to "Edit question" to make it easy to edit typos or something.  I'm looking forward to the next deploy!

Hi,
Any pointers to make it (widgets/poll) work on the mobile app?
Thanks,
Mahaveer
On Mon, Jul 2, 2018, 5:27 AM showell<EMAIL_ADDRESS>wrote:

Merged, thanks @rheaparekh https://github.com/rheaparekh! I really like
the experience here for the "other" people. For the person creating the
poll, I could see us getting some feedback, like maybe we should leave the
question in the input box next to "Edit question" to make it easy to edit
typos or something. I'm looking forward to the next deploy!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/pull/9833#issuecomment-401642017, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMhhrtx-KM5r1HaL3X6_IGEnO8ZwWgolks5uCWICgaJpZM4U3v_q
.


Update the Navtabs with Hover effect

Reduce The overlapping between Content.
Add hover effect on every image when the user hover the image it increase their size little bit by transition effect. Also Update the navtabs with better design.

@arghadipmanna101
I Issue the PR, kindly check #511


Set log-level immediately, before rootless setup

If we don't do this, we print WARN level messages that we should not be printing by default.
Up one WARN message to ERROR so it still shows up by default.
Fixes: #4115
Fixes: #4012

@giuseppe PTAL. I think this is safe to do before rootless setup happens.

Tentative LGTM! I hand-built this on my f30 laptop, copied the binary over to an f31 vm, played around with it in nonroot mode. Warning messages are gone! Lots of cgroups failures, but I'm guessing that's because my build didn't have the right tags. Thanks, @mheon!

yeah lgtm here

LGTM, nice catch/fix @mheon

Happy green test buttons here too

/lgtm


Animate elements outside the component which is scrolled.

I am not able to animate those elements which are outside the element which is been scrolled.
I have three parallel vertical divs. One div has element which can be scrolled and I want to animate an SVG which is in the other div which cant be scrolled, and not able to do so currently.

Can you post your example online somewhere?
On Thu, 9 May 2019, 22:40 Manas Ladha<EMAIL_ADDRESS>wrote:

I am not able to animate those eliments which are outside the element
which is been scrolled.
I have three parallel vertical divs. One div has element which can be
scroled and I want to animate an SVG which is in the other div which cant
be scrolled, and not able to do so currently.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alexfoxy/laxxx/issues/50, or mute the thread
https://github.com/notifications/unsubscribe-auth/AABFBXB3QZHOQMAKLCOWN2TPUSK5VANCNFSM4HL6HC3A
.

test123test123HelloWorld


comand failed ninja;-v

-- Configuring x86-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:56 (message):
Command failed: ninja;-v
Working Directory: C:/vcpkg/buildtrees/opencv/x86-windows-rel/vcpkg-parallel-configure
See logs for more information:
C:\vcpkg\buildtrees\opencv\config-x86-windows-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_cmake.cmake:256 (vcpkg_execute_required_process)
ports/opencv/portfile.cmake:238 (vcpkg_configure_cmake)
scripts/ports.cmake:71 (include)
Error: Building package opencv:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: opencv:x86-windows
Vcpkg version: 2018.11.23-nohash
Additionally, attach any relevant sections from the log files above.
PS C:\vcpkg> ./vcpkg update
Using local portfile versions. To update the local portfiles, use git pull.
No packages need updating.

Solution: trying to build in x86. Correct is x64. Forgot to close Environment Variables after adding VCPKG_DEFAULT_TRIPLET and value x64-windows...

@ChrisAmm
See logs for more information:
C:\vcpkg\buildtrees\opencv\config-x86-windows-out.log

@ChrisAmm, thank you for posting this issue, does this issue still occur? could you attach
C:\vcpkg\buildtrees\opencv\config-x86-windows-out.log?
I can't repro the issue
PS F:\VCPKG\src> ./vcpkg install opencv:x86-windows
Your feedback is important to improve Vcpkg! Please take 3 minutes to complete our survey by running: vcpkg contact --survey
The following packages will be built and installed:
opencv[core,eigen,flann,jpeg,opengl,png,tiff]:x86-windows
Starting package 1/1: opencv:x86-windows
Building package opencv[core,eigen,flann,jpeg,opengl,png,tiff]:x86-windows...
-- Using cached F:/VCPKG/src/downloads/opencv-opencv-3.4.3.tar.gz
-- Using source at F:/VCPKG/src/buildtrees/opencv/src/3.4.3-701a6234df
-- Configuring x86-windows
-- Building x86-windows-dbg
-- Building x86-windows-rel
-- Performing post-build validation
-- Performing post-build validation done
Building package opencv[core,eigen,flann,jpeg,opengl,png,tiff]:x86-windows... done
Installing package opencv[core,eigen,flann,jpeg,opengl,png,tiff]:x86-windows...
Installing package opencv[core,eigen,flann,jpeg,opengl,png,tiff]:x86-windows... done
Elapsed time for package opencv:x86-windows: 11.44 min
Total elapsed time: 11.44 min
The package opencv provides CMake integration:
find_package(OpenCV REQUIRED)
target_include_directories(main PRIVATE ${OpenCV_INCLUDE_DIRS})
target_link_libraries(main PRIVATE ${OpenCV_LIBS})

Thanks for your feedback. I installed opencv successfully with VS 2017 15.9.11, Please try to use latest vcpkg version and reopen a new issue if it is still a problem for you!


Initial state of nodes in a network

Is the number of source nodes determines the number of attractors?. For example, I have three source nodes in the network (13 nodes) and I have 2 ^ 3 = 8 attractors. In the state transition graph (STG), I could only see the state of the source nodes and state of other nodes are not shown. Is this because of the large number of nodes and hard to create a STG  with initial state of all nodes?
Thanks
Priya

It seems to be working, but your code does not display the plots or save the STGs. The code below worked for me:
import pystablemotifs as sm
import pyboolnet
import pystablemotifs.export as ex
import networkx as nx
import matplotlib.pyplot as plt
rules='''DCAF5*= not (miR335 or miR124 or miR4507 or miR653 or miR3154 or miR5191 or X1) and (BCL6 or MYC or KDM5A or KDM2B or TCF4)
BCL6*= not (miR124 or miR335)
MYC*= not (miR335)
KDM5A*= not (miR335)
KDM2B*= KDM2B
TCF4*= TCF4
miR335*= BCL6 or MYC or KDM5A
miR4507*= BCL6 or TCF4
miR653*= MYC or KDM5A
miR3154*= MYC
miR5191*= MYC or KDM5A or TCF4
X1*= KDM2B
miR124*= miR124'''

primes = sm.format.create_primes(rules)

max_simulate_size=20
ar=sm.AttractorRepertoire.from_primes(primes, max_simulate_size=max_simulate_size)
ar.summary()

df=ex.attractor_dataframe(ar)
df

for a in ar.attractors:
    if a.n_unfixed == 0: continue # skip steady states
    print(a.attractor_dict)
    nx.draw(a.stg,with_labels=True)
    plt.show()

This is the same as the code you posted, but I've added import matplotlib.pyplot as plt to the top and plt.show() to the for loop. You might also want to consider saving the graphs to files using the exporting tools in networkx.
It should be noted that the STGs being plotted are subgraphs of the full STG. Specifically, these are the terminal strongly connected components (i.e., attractors) of the full STG.

Thank you, it worked


Type to use with the tidy function

I'm trying the performance tutorial with typescript, any ideas how to fix:
TS2345: Argument of type '(keep: any) => void' is not assignable to parameter of type 'string | ScopeFn'.
Type '(keep: any) => void' is not assignable to type 'ScopeFn'.

Instead of using keep as the argument in tidy, you can call dl.keep() directly.

Yes, since 0.5.0 dl.tidy() takes a function with no arguments. that is
dl.tidy(() => {
  dl.keep(something);
});

Thx for clarification


Enhancement - generate src/package.json from root/package.json

Please consider an enhancement in the generator library to maintain the src/package.json.
During the deployment of umd build, the developer need to edit the src/package.json peer dependencies to ensure the require 3rd party packages are referenced correctly, in order not to cause problems in the imported application projects.
Although it is not very hard to maintain src/package.json, adding extra automation (if possible) will significantly add to the usability of this generator library, throughout the application life cycle of npm component development.
The maintenance of src/package.json is particularly important for components that is developed with peer dependencies that are based in non-Angular dependencies.
Please consider this request as one of the possible enhancment.
Typical import problems when the npm component does not have the right peer dependencies will be a NPM module error (Unexpected value '[object Object]) as follows:
Uncaught Error: Unexpected value '[object Object]' imported by the module 'AppModule'. Please add a @NgModule annotation.
    at syntaxError (compiler.es5.js:1689)
    at compiler.es5.js:15373
    at Array.forEach (<anonymous>)
    at<EMAIL_ADDRESS>(compiler.es5.js:15356)
    at<EMAIL_ADDRESS>(compiler.es5.js:26679)
    at<EMAIL_ADDRESS>(compiler.es5.js:26652)
    at<EMAIL_ADDRESS>(compiler.es5.js:26581)
    at<EMAIL_ADDRESS>(core.es5.js:4595)
    at<EMAIL_ADDRESS>(core.es5.js:4581)
    at Object.../../../../../src/main.ts (main.ts:11)

I don't think automating would help a lot here as one should only add one line to src/package.json.
However, as suggested in #147, it do believe it would be great if we had a step-by-step guide on how to include a third party library in a library.
I will close this issue for now, but if you are interested in writing a guide, please don't hesitate to re-open this issue and contact me.
Thanks again!


Fix bundle export error

Fix #8048
@Eschon can you please check?

Just tried it out and the error is gone. Looks good to me.
Thanks for the quick fix!


[BugFix] Fix complex window + agg prune bug

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Like SQL:
select v2
from (
   select v2, x3
   from (select v2, sum(v1) over (partition by v3) as x3 from t0) as tt0
   group by v2, x3
) ttt0
group by v2;

First, Aggregate-GroupBy(v2, x3) and Aggregate-GroupBy(v2) will be merge to Aggregate-GroupBy(v2)
Then, prune column will prune x3 column, window function will be empty
The bug is here, we should prune empty window node

Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test


Update MIP10c9: add newline formatting

Was walking a partner through MIP10c9 and noticed the requirements section markdown isn't registering the new lines.

LGTM.


Tables: allow un-numbered tables

eLife allows un-numbered inline tables and a Key Resources un-numbered table - there is currently no way to enter these in Texture.
Inline table:

The following unnamed table is an example of an inline table that has no heading.

pY	Experiment	Concentration (μM)

                                    IGF1R-fl + IGF1	+	<italic>K</italic><italic>m</italic> ATP	500, 400, 300, 250, 125, 62.5, 31.3, 15.6, 7.8

                                    IGF1R-fl + IGF1	+	<italic>K</italic><italic>m</italic> Peptide	600, 300, 150, 75, 37.5, 18.8, 9.4

                            <fn>

This is an unmarked footnote for an anchored/inline table

                            </fn>

Key resources table:

                Key resources table

                            Reagent type <break/>(species) <break/>or resource
                            Designation
                            Source or <break/>reference
                            Identifiers
                            Additional information

                            Gene <break/>(<italic>Nicotiana</italic>
                                <break/><italic>attenuata</italic>)

                                <italic>NaMaT1</italic>

                            PRJNA355166
                            XM_019403695.1

                            Gene <break/>(<italic>N. attenuata</italic>)

                                <italic>NaMaT2</italic>

                            PRJNA355166
                            XR_002066055.1

@michael just to let you know this is on our to-do list, but not yet reached the top. It's on our internal trello board as requiring input from eLife

Thank you @Melissa37, take your time!


chore(deps): update common-dev-assets digest to c0b9fc9

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
dda6d1f -> c0b9fc9

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

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


Inconsistent data between articles

Data is not consistent with this article https://learn.microsoft.com/en-us/training/modules/design-data-storage-solution-for-non-relational-data/6-design-for-azure-files
For example, Maximum Share/Volume Throughput for premium Azure Files listed here as 10 GiB/s, while in other article it's only 5.
Latency is listed as Single-millisecond while in other article it's listed as Double-digit for Standard Tier.
Please correct the discrepancies.
Thank you.

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

ID: 27132a89-4dee-aa95-1d0a-acb454b342c7
Version Independent ID: dc63897c-8280-d17b-7fdc-dd1f859e2574
Content: Azure Files and Azure NetApp Files Comparison
Content Source: articles/storage/files/storage-files-netapp-comparison.md
Service: storage
Sub-service: files
GitHub Login: @khdownie
Microsoft Alias: kendownie

@Tsubasanut
Thanks for your feedback! We will investigate and update as appropriate.

@Tsubasanut
I apologize for the confusion. The information in the article you mentioned is correct. The maximum share/volume throughput for premium Azure Files is 5 GiB/s, and the latency for the standard tier is double-digit milliseconds. The information in the article you linked to may be outdated. I would recommend referring to the official Microsoft documentation for the most up-to-date information on Azure Files performance. You can find the official documentation at the following link:
https://learn.microsoft.com/en-us/azure/storage/files/storage-files-scale-targets
if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Is this just the bottom-up attention or does it include the full VQA model?

If it's just the attention, do you plan on releasing the VQA model?

We have now released the code for the captioning model: https://github.com/peteanderson80/Up-Down-Captioner. The VQA model is the work of Damien Teney from the University of Adelaide, he may consider releasing it if you contact him.
thanks


Different behavior regarding resolved inputs when listing transactions with and without query parameters

Context

Reported by an exchange on Slack.

Quote:
"We need to check input, output values in a transaction.  However, if we use start, end parameters in /transactions api, input values don't appear in the response. If we fetch all the transactions without start and end parameters, they appears. But we don't want to fetch all the transactions belonging the wallet neither due to the performance concerns."

Information
-

Version
Unknown

Platform
Unknown

Installation
Unknown

Steps to Reproduce

see #1685
Expected behavior

Responses are coherent with and without start / end
Actual behavior

Responses are different.

Resolution

Resolve transaction inputs in the database lookup, when fetching inputs and outputs ==> #1685

QA

Added a regression test to capture the scenario described below in comment as TRANS_REG_1670

Going quickly through the code, one of the reason might be:
    -- This relies on DB.readTxHistory returning all necessary transactions
    -- to assemble coin selection information for outgoing payments.
    assemble

So, we the history returned is used both as the result, but also as an index for resolving transaction inputs. Therefore, if a transaction input refers to a known transaction that is discarded by the range parameters, then it won't be present in the return history and will fail the lookup.
Making assemble fetch transactions from the database would probably fix the issue, but at what cost? This would require at most 1 request per input (so possibly up to 80/90 requests per transactions) although it'd still be better than fetching the entire history in memory every time.
Another alternative could be to list transactions as already resolved from the database with an inner join; haven't quite thought about it but it seems doable from the top of my head and should not induce that much overhead.
cc @piotr-iohk : if this intuition is correct :point_up: , we should be able to reproduce this by making two transactions, the second one using at least one output of the first one, and then, by listing all transactions  and listing all transaction with a range only including the second transaction.

Here is an example case from my test wallet

Indeed I can reproduce:
I have 3 outgoing txs on my wallet:
id1 -> inserted at: 2020-05-18T14:15:36Z
id2 -> inserted at: 2020-05-18T14:26:56Z
id3 -> inserted at: 2020-05-18T14:31:56Z
TC0: Listing all transactions. => The amount is shown for every input for each tx.
cardano-wallet-byron transaction list 05bb19daeb9e3aaca992856fb21f64ded1632aec | jq '.[] | .inputs'

[
  {
    "amount": {
      "quantity": 657187,
      "unit": "lovelace"
    },
    "address": "37btjrVyb4KEWxuxXEVFoj7HqjS6Q6nVkAHVFa33QR8QuqScKxD4p9Aw3PYfXQj724wWNPBGnJLWEiW9SYR5c4EwB8rCRRysab3pYamT8Wwun45WGd",
    "id": "bf6f9b0f2067314e2c507750f47744bf82bd0f1a7562000747f90d16aee619ea",
    "index": 0
  }
]
[
  {
    "amount": {
      "quantity": 828665,
      "unit": "lovelace"
    },
    "address": "37btjrVyb4KCckzCvuVxt7MizbjCDc2N9LbppckfeMDJffPae6ngk3Ms551hesD2fr5wbyFCmHSAFLbvTwTMdieKjT8D6GsEZ4ZZixTfzYrGXuvF7P",
    "id": "89c680afc1e3ca4cad2f442d99f9a00a2b921283c7a64624f1b575e60c1f2ad2",
    "index": 0
  }
]
[
  {
    "amount": {
      "quantity": 1000000,
      "unit": "lovelace"
    },
    "address": "37btjrVyb4KDBr7TwHm5HivAt8r8Jk4eXDERDyj6G8s9Rw6Prmr5RrB9dnqmk1cmmXQCWURyLLfa4BNKfrJDpTsaxSvWwviHXH4bRnLJuxm16Fzvca",
    "id": "0b089cb29fddb79307ee36dcd5ce3728d3fea794782901c7d2550bd7fc8548ec",
    "index": 0
  }
]

TC1: List txs with --start 2020-05-18T14:15:36Z ( start before tx id1) => Amount for tx id1 is missing.
cardano-wallet-byron transaction list 05bb19daeb9e3aaca992856fb21f64ded1632aec --start 2020-05-18T14:15:36Z | jq '.[] | .inputs'
[
  {
    "amount": {
      "quantity": 657187,
      "unit": "lovelace"
    },
    "address": "37btjrVyb4KEWxuxXEVFoj7HqjS6Q6nVkAHVFa33QR8QuqScKxD4p9Aw3PYfXQj724wWNPBGnJLWEiW9SYR5c4EwB8rCRRysab3pYamT8Wwun45WGd",
    "id": "bf6f9b0f2067314e2c507750f47744bf82bd0f1a7562000747f90d16aee619ea",
    "index": 0
  }
]
[
  {
    "amount": {
      "quantity": 828665,
      "unit": "lovelace"
    },
    "address": "37btjrVyb4KCckzCvuVxt7MizbjCDc2N9LbppckfeMDJffPae6ngk3Ms551hesD2fr5wbyFCmHSAFLbvTwTMdieKjT8D6GsEZ4ZZixTfzYrGXuvF7P",
    "id": "89c680afc1e3ca4cad2f442d99f9a00a2b921283c7a64624f1b575e60c1f2ad2",
    "index": 0
  }
]
[
  {
    "id": "0b089cb29fddb79307ee36dcd5ce3728d3fea794782901c7d2550bd7fc8548ec",
    "index": 0
  }
]

TC2: List txs with --start 2020-05-18T14:20:00Z (before tx id2) => Amount for tx id2 is missing.
cardano-wallet-byron transaction list 05bb19daeb9e3aaca992856fb21f64ded1632aec --start 2020-05-18T14:20:00Z | jq '.[] | .inputs'
Ok.
[
  {
    "amount": {
      "quantity": 657187,
      "unit": "lovelace"
    },
    "address": "37btjrVyb4KEWxuxXEVFoj7HqjS6Q6nVkAHVFa33QR8QuqScKxD4p9Aw3PYfXQj724wWNPBGnJLWEiW9SYR5c4EwB8rCRRysab3pYamT8Wwun45WGd",
    "id": "bf6f9b0f2067314e2c507750f47744bf82bd0f1a7562000747f90d16aee619ea",
    "index": 0
  }
]
[
  {
    "id": "89c680afc1e3ca4cad2f442d99f9a00a2b921283c7a64624f1b575e60c1f2ad2",
    "index": 0
  }
]

TC3: List txs with --start 2020-05-18T14:30:00Z (before tx id3) => Amount for tx id3 is missing.
cardano-wallet-byron transaction list 05bb19daeb9e3aaca992856fb21f64ded1632aec --start 2020-05-18T14:30:00Z | jq '.[] | .inputs'
Ok.
[
  {
    "id": "bf6f9b0f2067314e2c507750f47744bf82bd0f1a7562000747f90d16aee619ea",
    "index": 0
  }
]

Conclusion: amount is not displayed for the first outgoing transaction that is shown after --start parameter.
Can be observed for wallet on byron testnet: onion game wish tuna smooth category clock language hour subway mirror soccer.

Similar issue, but for incoming transactions: #573.

Looking at my previous example https://github.com/input-output-hk/cardano-wallet/issues/1670#issuecomment-630233182 the original issue is no longer the case. I.e. the amount is being shown when one uses start to narrow listed transactions.
I noticed however that there is some discrepancy on transaction input amounts before and after the fix. I mean transactions on the wallet that is restored on recent master are different from tx on the wallet restored on revision before the fix (https://github.com/input-output-hk/cardano-wallet/commit/031835950a38384b289f3adc43c671a00ef9000b)
Looking at the transaction from my previous example 7ddde107f4691e0db21462324a41c7d1d719979b4c4861e2486af1937ac0ac97:
Before the fix:
input amount = 657187
output1 = 155
output2 =  485654
After the fix:
input = 100 (which doesn't seem right)
output1 = 155
output2 =  485654

@piotr-iohk , should be fixed by #1697 now :ok_hand:

lgtm.


[WIP Don't Merge] Refactor search dialog to a searchbar like in browsers and text editors

Refactor search dialog to a searchbar, like in browsers and text editors
Add Key bindings: Enter for navigating to the next match, Shift+Enter for navigating to the previous match, and ESC to exit searchbar.
If the user selected a note before opening the search dialog, use the lyric of the selection as the search word.
Show both the count of matches and the index of current match.

TODO:

Full word matching
search within phonemes
Case insensitive mode


Switch from GTK to a Rust native GUI

It would be nice to switch to a Rust native GUI at some point, which would simplify the build and package process quite a bit. Potential crates are egui, Dioxus, and Iced.
The only reason we can't switch now is because they all rely on on the same base window crate, and that crate doesn't yet have the ability to be a drag and drop source. We wouldn't be able to drag images out of StableKeepr into, say, a web browser or discord - a pretty common task.
I'm watching relevant issues and pull requests on the base window crate, and will look into the feasibility of switching when that feature is completed.

Trying to use GTK was an exercise in frustration, and pretty much stalled the project. According to this, winit is getting closer to having drag and drop support, which should enable it in the GUI libraries mentioned above. As such, I'm pivoting from GTK to Iced.


[Discuss] New issue tracker tags

Hey everyone! We are trying a new set of tags for the issue tracker, as you might have probably already noticed. We plan on gathering feedback for a few weeks and test how it fares, and adjust accordingly.
Please do share your thoughts here!

When would status:implemented be used?

When a issue is closed there might be lack of information whether it was effectively solved or not. Issue closed+status:implemented can be read as done. Other closing reason like duplicate or invalid exhibit that there where no action taken in the source code.
When a PR is merged there is already evidence that it is implemented so there is no need to apply the tag there.

A few thoughts about the new labels and some more changes we might implement:

We should add a kind:refactor or more general kind:task, for those things that are not actually features (and certainly nor bugs neither questions). We know, it's not easy to classify things into buckets, and the balance between being inaccurate and having an unusable number of buckets is difficult to achieve.

An issue goes through different stages during its lifetime:
1- First, after being created, it must be accepted, meaning that we acknowledge the bug or accept that it is a feature we want to have in the language.
2- Afterwards, if the issue is not trivial, we need to agree on a design, on how to achieve that feature, or which API it would expose. This design might have been provided as part of the issue originally, in case of drafts.
3- Once we have a design we like, the issue is ready to be implemented, and the process follows in a PR.

We want for people arriving to the issue tracker to easily know how they can contribute. As such, a generic label community:help-wanted is unclear as what is needed in that particular issue without having to go through it. We might split this label into:
1- community:to-research, for issues where we need help investigating something
2- community:to-design, for issues already accepted where we need to define how it should be implemented
3- community:to-implement, for issues where a PR with the implementation is needed

A good practice would be to batch certain related issues, until we reach a point where we want to tackle all of them together; these issues could be marked as deferred, delayed, batched, or whichever better name we come up with. We still got to decide what artifact we want to use for batching:
1- Another issue marked as an epic or similar, like #2665
2- A label, by refining the topics we currently have
3- A milestone, though we might want to use them later once we clean up the roadmap
4- A github project, where we can choose which issues of a topic will be tackled in the next, though it might be overkill

Thoughts?

@spalladino I think either "epic" issues or github projects are what I would use here. Although github projects may get a little messy unless we decide they must be structured in a specific way.

Fixed in #3984


Remove redundant spaces

What this PR does / why we need it:

There are two spaces in the comment
Some of the commented variables are not consistently aligned with others.

ci check this


Closing viewer programmatically

🚀 Feature
I can't find a way to close the window programmatically, but it'd be a useful feature to have.
Motivation
I'm writing a small python package and in it I need to create and close napari viewers as needed. To make things easier for the user I'd like to close the viewer programmatically when an operation is completed, rather than requiring the user to press on the 'x' button. This makes the flow of the application smoother and faster and reduces the risk of user errors.
Pitch
add a .close() method that closes the viewer as needed.
Alternatives
Not doing and leaving just the 'x' button.
Additional context
I've tried using this:
     viewer.window._qt_window.close()

but this doesn't work, it leaves the viewer open and unresponsive to button presses.
Minimal code to reproduce:
import napari

with napari.gui_qt():
    v1 = napari.Viewer()

    v1.window._qt_window.close()

with napari.gui_qt():
    v2 = napari.Viewer()

v2 is never displayed because v1 doesn't close properly.
This was with napari v 0.2.12 on macOS.

Hi @FedeClaudi,
We revamped the shutdown process and added a viewer.close() method in #1014.   Try the current master branch at let us know if it works for you.

I'll also add that part of what you're seeing here might have to do with using multiple context managers like that in a script.  I don't have a full explanation... but there's obviously some race conditions that happen when spinning up and shutting down a Qt event loop.  So you might run into weirdness when using two napari.gui_qt() calls like that in sequence in a script.
Even on the master branch, creating a viewer and immediately closing it like this has issues:
import napari

with napari.gui_qt():
    v1 = napari.Viewer(title='v1')
    v1.close()

but this works fine and shows viewer 2:
import napari

with napari.gui_qt():
    v1 = napari.Viewer(title='1')
    v1.close()
    v2. = napari.Viewer(title='v2')

part of this is probably stuff we can fix (though it's proven very hard to debug), but the other aspect here is just using Qt a little differently, and not over-taxing the spin-up/shut-down proces.

Hi,
Thanks for getting back to me so quickly.
I've installed from the master branch and tried it.
There is a close method but it doesn't seem to work.
This is the result:

As far as I can tell, viewer.close() just calls self.window.close() so it's the same as what I was trying earlier.

sorry, see my previous comment... viewer.close() does call viewer.window.close(). but #1014 significantly changed viewer.window.close() as well.  So the master branch is not the same as v0.2.12 .... however, the bigger issue here is the immediate opening and closing of a viewer in a context manager.

Hi,
This solution:
import napari

with napari.gui_qt():
    v1 = napari.Viewer(title='1')
    v1.close()
    v2. = napari.Viewer(title='v2')

works, but I can't use it in my application.
The two viewers are created by different functions at different times, so I need to call with napari.gui_qt(): twice.

p.s. I've installed from the master branch with:
pip install git+https://github.com/napari/napari

is there something else I should try?

napari.gui_qt() is only a convenience for spinning up a quick Qt event loop.  If you need a more persistent Qt application (with which to run multiple different functions), then you should be starting it elsewhere. (for instance, using ipython --gui=qt, using the %gui qt IPython magic, or calling QtWidgets.QApplication() directly in your program before using napari).  the most appropriate choice will depend on the specifics of your application.
If you can direct me to the "different functions at different times" in your package, I might be able to provide a suggestion

Awesome thanks!
So the users need to first click on a set of points on the reference image used for image registration.
This is done using get_fixed_points in imgreg2D.points.py (https://github.com/BrancoLab/imgreg2D/blob/master/imgreg2D/points.py)
After that the user needs to click on the same points on the image being aligned, this is done with get_registering_points, in the same script.
The reason why I want them to be separate, is that the user might have pre-defined fixed_points that he/she wants to use for the registration, so I want to give the option to skip the get_fixed_points step. Alternatively, one might just want to get the fixed_points without  doing the registration, so again it's useful to have the two steps separate.
I'd rather not use ipython/jupyter if possible, but I'm not very familiar with Qt application workflow. Any help would be appreciated, thank you!

Hi,
So,
import napari

with napari.gui_qt():
    v1 = napari.Viewer()

    v1.window._qt_window.close()

with napari.gui_qt():
    v2 = napari.Viewer()

Doesn't work.
But I decided to try viewer.close() in my code anyways, and there it works!
Not sure if it's becaues of the delay between the two calls to with napari.gui_qt():  or something else, but at least in my application viewr.close() seems to be working perfectly.
Thank you very much for the help!

glad to hear it.
still happy to show you an alternative to using gui_qt() throughout your code.
It's a bit hard from your readme to figure out exactly how you're calling all your functions (and the GIF isn't high enough resolution).  How do you actually launch your program?  and does it involve multiple "executions", or is the full "run" contained in a single session?
It looks like you're using the VScode launch button on test.py ... but test.py doesn't have a if __main__ statement.  what do you have in your launch.json?

a general pattern you might consider is to try to design your program such that you only need a single with napari.gui_qt call... so, for instance, your test.py might look like this instead:
from imgreg2D.register import register
from imgreg2D.points import get_fixed_points

def main(reference, registering):
    # Get fixed points
    points = get_fixed_points(reference)
    # Register
    register(reference, registering, fixed_points=points)

if __name__ == "__main__":
    import napari

    # could grab these from a command line call with sys.argv
    reference = "media/original.jpg"
    registering = "media/rotated.jpg"

    with napari.gui_qt():
        main(reference, registering)

then you would remove all of the gui_qt calls inside your code.
Another thing you should try to avoid is accessing napari layers after your context managers are closed and the napari viewer is closed:  For instance, in this line, you access points_layer on a viewer that has presumably already been closed...  I would instead design that whole context manager as an independent function that you can call, and have that function return the points_layer.data (which seems to be all you're using from the napari layer anyway), rather than the whole layer object itself(which is now orphaned from its viewer).
If you change all those context managers to functions, then you will only need to start up one Qt context per "run", and that should be done at the very highest level, when you start your program (e.g. test.py)

Hi,
awesome, thank you so much for taking the time to look into this. I'll see if I can restructure the code to start the Qt context only once, but for the time being the master branch version of napari's viewer.close() seems to be working very well.
Thank you again,
Federico

ok, I'm gonna close this issue, since the original request was added in a recent PR, but feel free to leave more comments or open a new issue with any additional thoughts


Missing Firewall Rule For Commercial Cloud

A deployment of virtual machines for the pool will fail if you do not also add HTTPS outbound to wvdportalstorageblob.blob.core.windows.net. The DSC execution will fail without access to that URI.

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

ID: 406f8b6d-6471-c983-c102-9d60a57964cd
Version Independent ID: 98fa8587-36e1-690c-12c5-b865955ada0b
Content: Azure Virtual Desktop required URL list - Azure
Content Source: articles/virtual-desktop/safe-url-list.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@aidanfinn Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate. We will update you accordingly.

@Heidilohr Can you please check and add your comments on this doc update request as applicable


TS map in healpix

Allow user to pass appropriate nside parameter. Maybe get default from response nside?

Done for DC2


Support compound column names to query nested types

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Follow up for https://github.com/apache/arrow-datafusion/pull/55#discussion_r619815966.
We want to support queries like  SELECT t1.customer.address.zip, ...
Describe the solution you'd like
Extend the data structure we use to represent columns to handle compound column names.

https://github.com/apache/arrow-datafusion/pull/628 already implemented the core logic, anyone who is interested in this work can pick up that PR and finish it.


feat(STONEINTG-755): add rego for checking clamav output

ClamAV output can be processed by rego and tested by standard conftest.
We don't need multiple steps for parsing resutls, by using EC-cli it can be done in single step.
This is prerequisite for next steps

Requires https://github.com/redhat-appstudio/hacbs-test/pull/201

Changes looks fine for me, will provide review once its moved rom draft.


Elsevier web scrapping is not working in my case!!

I was trying to web scrap using the code provided in the repository, but not able to generate any output data using the code in web scrap folder (like elsevier.py file). Can you share the process or highlight the correct way of performing the task.

What's the error in your case? Have you provided the correct API key?


Updating image

Updating image based on feedback that the previous one didn't look good.

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

Hi @kelvinxia-MSFT - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Would you submit your image update in the private repo?
Note that in your new PR in the private repo, you must add an image file to the repo (in the same PR) in order to reference it in an MD file. Referencing an image that's hosted outside the repo isn't supported.
We'll close this PR . . . Thanks!


Remove double empty line

Remove double empty line due to code convention changes

  /// <summary>

Might be good to correctly split up these properties as you're making these changes

Refers to: src/CodeGenerators/TimedScopeGen/TimedScopeClassTemplate.cs:333 in b5f9de1. [](commit_id = b5f9de14df6c3398c49e0eda85b20c19072d3122, deletion_comment = False)

  /// <summary>

Might be good to correctly split up these properties as you're making these changes
Refers to: src/CodeGenerators/TimedScopeGen/TimedScopeClassTemplate.cs:333 in b5f9de1. [](commit_id = b5f9de1, deletion_comment = False)

Next PR that I was planning is to delete of TimedScopeGen :) so I don't think it worth investing in it


Documentation link is invalid

Hi,
When reviewing your template, we noticed that the documentation link is invalid. https://www.data-marketing-school.com/documentation
Please fix that link. Otherwise, looks good. Thanks.

Hello Andy, you already approved this template and I removed the link to the 404 documentation in this commit 697abb4

Sorry for the mix up. It looks like it was an issue with our tooling. Thanks.

Okay no worries, while you are on my github, I submitted 2 new matomo tag templates (this one and that one). :)


modul mip on ESP32

Hello
During the installation I encounter problems with the command:
install mip
the ESP32 is connected to the Wifi.
Question: is there a "mip" module for the ESP32?

Hi,
mip module is a standard module, but the right command for loading a module is 'import'. Did you use 'import' or 'install'

hi
yeah, import mip don't work.  import time as an example works.

import mip
Traceback (most recent call last):
File "", line 1, in
ImportError: no module named 'mip'
import time

I assume you are using an older version of Micropython. Unfortunately, at the moment the documentation is updated more quickly than the 'stable' versions. In the June version, a number of functions do not yet exist, but can be found in the documentation. mip is the successor to upip - see Nightly builds


windows slash appended to svn info by bower

Hi
I'm moving a project from a win2008 to a win2012 server.
The only difference is svn version 1.7.10(2012) and 1.7.5(2008)
bower update
:
bower buypass-grunt#0.1.8                                   ECMDERR Failed to execute "svn list https://svnserver.intra.buypass.no/svn/bpw/bower/buypass-grunt\tags --verbose", exit code of #1 svn: warning: W160013: Unable to connect to a repository at URL 'https://svnserver.intra.buypass.no/svn/bpw/bower/buypass-grunt\tags' svn: E200009: Could not list all targets because some targets don't exist
This should be
"svn list https://svnserver.intra.buypass.no/svn/bpw/bower/buypass-grunt/tags --verbose"
$ cat bower.json
{
  "name": "minsideclient",
  "version": "1.4.2",
  "date": "2015-03-04 07:44:32 +00:00",
  "description": "Buypass min side client",
  "authors": [
    "Buypass AS"
  ],
  "homepage": "http://buypass.no",
  "private": true,
  "ignore": [
    "**/.*",
    "node_modules",
    "bower_components",
    "test",
    "tests",
    "war",
    "dist",
    "coverage"
  ],
  "license": "Private",
  "dependencies": {
    "angular": "1.2.16",
    "angular-i18n": "1.2.5",
    "angular-sanitize": "1.2.16",
    "angular-route": "1.2.16",
    "angular-resource": "1.2.16",
    "angular-animate": "1.2.16",
    "jquery": "1.11.0",
    "angular-bootstrap": "0.10.0",
    "angular-ui-select2": "0.0.5",
    "jquery-ui": "1.10.3",
    "select2": "3.4.6",
    "fancybox": "2.1.5",
    "bootstrap": "3.1.1",
    "es5-shim": "2.3.0",
    "respond": "1.4.2",
    "html5shiv": "3.7.0",
    "buypass-bootstrap": "svn+https://svnserver.intra.buypass.no/svn/bpw/bower/buypass-bootstrap#~0.4.0",
    "buypass-grunt": "svn+https://svnserver.intra.buypass.no/svn/bpw/bower/buypass-grunt#0.1.8",
    "buypass-angular-utils": "svn+https://svnserver.intra.buypass.no/svn/bpw/bower/buypass-angular-utils#*",
    "angular-translate": "2.4.2",
    "angular-translate-loader-static-files": "~2.4.2"
  },
  "devDependencies": {
    "angular-mocks": "1.2.16"
  }
}

$ bower --version
1.3.12
$ node --version
v0.10.25
$ svn --version
svn, version 1.7.10 (r1485443)
compiled Jun  1 2013, 09:36:13
I see the same if I run in cygwin, dos cli or jenkins. Any idea what it is.
All other projects on the server seem to work fine.
Thanks
Morten Bjoernsvik, Buypass, Oslo, Norway

Any change you use bower version from master branch instead actual 1.3.12?

Hi thanks for the reply. There was no changed with the latest from git master. But I found the error:
Administrator@vmbld05it /cygdrive/c/jenkins/jobs/Min Side Client
$ npm install
/cygdrive/c/nodejs/nodejsv0.10.25/npm: line 2: $'\r': command not found
/cygdrive/c/nodejs/nodejsv0.10.25/npm: line 4: $'\r': command not found
/cygdrive/c/nodejs/nodejsv0.10.25/npm: line 5: syntax error near unexpected token `$'in\r''
'cygdrive/c/nodejs/nodejsv0.10.25/npm: line 5: `case `uname` in

Administrator@vmbld05it /cygdrive/c/jenkins/jobs/Min Side Client
$ dos2unix /cygdrive/c/nodejs/nodejsv0.10.25/npm
dos2unix: converting file /cygdrive/c/nodejs/nodejsv0.10.25/npm to Unix format...

when that file is in proper format it continues, probably if this file is in windows crlf mode it will give out '/'
Thanks again

I don't understand. Is your issue resolved? The second error you posted seems like a different issue.


Add support for Babel

SystemJS now supports Babel as an alternative transpiler.
Change the plugin to optionally load Babel as an include instead of Traceur.

Added Babel support: 0db0e6496471f911588adb68b356d291f23aed22


Update Koin to 4.0.1 and refactor configuration

#1  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


LAU-622 lau-idam-backend performance improvement

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/LAU-622
Change description
These changes are to improve performance for lau-idam-backend query
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Can you please remove the following configs from application.yaml

minimumIdle: 2
idleTimeout: 10000

Add the datasource type in application.yaml

type: com.zaxxer.hikari.HikariDataSource


Incorrect translation status after machine translation (XLIFF-exported pages)

Describe the Bug

Translation status of "currently in translation" pages does not transit correctly after machine translation.
Steps to Reproduce

Go to a page.
Export the page for translation in a language for which machine translation is allowed
Machine translate it by bulk action.
See the status remains "currently in translation" with clock icon.

Expected Behavior

After machine translation the status changes to "machine translation" (and the robot icon).
Actual Behavior

Aftter machne translation the status remains "currently in translation" (with the clock icon).
Additional Information

Related issue #2319

Traceback

It seems the problem is more generall: when Redis is activated, the icon doesn not transit qucikly (sometimes transits after a while/some other operations, sometimes remains unchanged) although the change is actually saved in the database.

It seems the problem is more generall: when Redis is activated, the icon doesn not transit qucikly (sometimes transits after a while/some other operations, sometimes remains unchanged) although the change is actually saved in the database.

Thanks for investigating this issue. In this case, I'd say we re-open it and make sure to invalidate the cache correctly.


[Bug]: Typescript performance issue

Package
lucia
Describe the bug
I noticed significant performance issues with TypeScript when using Lucia., which are causing the IDE to become extremely slow.
Steps to reproduce:

Clone repo

git clone https://github.com/SikandarJODD/Auth-Lucia

comment DATABASE_URL import inside Auth-Lucia/src/lib/server/db.ts
Generate performance tracing

npx tsc --noEmit --incremental false --generateTrace tracing
npx @typescript/analyze-trace tracing/

This was caused by using DrizzleSQLiteAdapter from @lucia-auth/adapter-drizzle with drizzle-orm/libsql
I switched to LibSQLAdapter from @lucia-auth/adapter-sqlite and that fixed my issue.


feat: add sas 9 remote (via IOM bridge) support

Summary
This adds sas 9 IOM bridge support. There are a few things to note here:

This uses ITC to connect to SAS 9 via the IOM bridge protocol. Our current local sas 9 connection uses the same underlying connection. Therefore, instead of recreating the wheel, our current com session was modified.
All of the connection files were moved from com -> itc to better match connection info
fetching file results no longer assumes the file is on your local machine. Instead, it uses the file service to grab the results from the last run

Testing

[x] Test running multiple sas programs via SAS 9.4 (remote - IOM)
[x] Test running multiple sas programs via SAS 9.4 (local)

Create a new profile with SAS 9.4 (Remote-IOM). The property description is not consistent with the connection type.  Put mouse over 'host', 'port' and 'username'.

It failed to run the code with error below.
Error detail:
There was an error executing the SAS Program. See console log for more details.

Create a new profile with SAS 9.4 (Remote-IOM). The property description is not consistent with the connection type. Put mouse over 'host', 'port' and 'username'.

I tested the connection details you sent me and was unable to test the connection using SAS Integration Technologies Configuration Wizard. That leads me to believe this is an issue with the server configuration.

It failed to run the code with error below. Error detail: There was an error executing the SAS Program. See console log for more details.

I tested the connection details you sent me and was unable to make a successful connection using SAS Integration Technologies Configuration Wizard. That leads me to believe this is an issue with the server configuration.

Create a new profile with SAS 9.4 (Remote-IOM). The property description is not consistent with the connection type. Put mouse over 'host', 'port' and 'username'.

Resolved in https://github.com/sassoftware/vscode-sas-extension/pull/592/commits/3947797b37504d3cf02238896f99149d63f76bbb

Some unreadable Chinese characters are shown on the result page with Local94 connections (COM).
Please update the sasv9.cfg as below at C:\Program Files\SASHome\SASFoundation\9.4 to start a Chinese SAS Session to reproduce.
-config "C:\Program Files\SASHome\SASFoundation\9.4\nls\zh\sasv9.cfg"

sampleCode.txt

Suppose having a .sas file and the file name contains Chinese characters(Test测试数据.sas).  It will fail to run on a specific remote94 server.
Error:
There was an error executing the SAS Program. See console log for more details.

Hey @Zhirong2022 . Do you mind testing the latest two test cases on sas 9 using COM (on the main branch)? I've been looking into the chinese character in the filename issue you've mentioned, and it looks like that issue was introduced by https://github.com/sassoftware/vscode-sas-extension/pull/524

@scottdover , I tested the two cases on Main Branch.  It works well.

VScode help menu => Toggle Developer Tools to find the console error message.
It might be related to https://github.com/sassoftware/vscode-sas-extension/issues/583. The coding of powershell and SAS should be the same. In my case, SAS session is n utf-8 encoding, however powershell is in zh_CH (936?) code which make issue 583. So I put chcp 65001 to powershell's $PROFILE and solve the coding issue.

VScode help menu => Toggle Developer Tools to find the console error message.
It might be related to #583. The coding of powershell and SAS should be the same. In my case, SAS session is n utf-8 encoding, however powershell is in zh_CH (936?) code which make issue 583. So I put chcp 65001 to powershell's $PROFILE and solve the coding issue.

How did you accomplish this? I updated the code to spawn a child process using chcp 65001 >NUL & powershell.exe ..., but that didn't seem to work for me

echo "chcp 65001" >> $PROFILE

For current user and current host, Windows - $PROFILE is  $HOME\Documents\PowerShell\Microsoft.PowerShell_profile.ps1
or %USERPROFILE%\Documents\PowerShell\Microsoft.PowerShell_profile.ps1

Unreadable characters in log output. Please run the code below to have a try.
proc print data=sashelp.cars (obs=1); run;

Hey @smorrisj @clangsmith @scnwwu .
I know we continue to have some issues around encoding that @smorrisj is helping me look into, and @snlwih has mentioned expanding IOM support while we get through these encoding issues. Given the majority of the code works as expected, I'm wondering if we can/should...

Create a new branch based on main, called sas9-iom-integration (or something similar)
Merge these changes into sas9-iom-integration

That would allow us to more easily review a more focused PR for things like:

handling the encoding issues
expanding iom support for libraries/etc

That said, it does have the added overhead of first merging into sas9-iom-integration, then merging those changes into main, but it's pretty straightforward

Unreadable characters in log output. Please run the code below to have a try.
proc print data=sashelp.cars (obs=1); run;

Could you retest with changes from https://github.com/sassoftware/vscode-sas-extension/pull/592/commits/138ad73095605f3ad0368edca158ff7bf63b615c?

This looks good @smorrisj....just looks like tests need to be patched

@scottdover @smorrisj  Thank you for your update.  I retested the raised issues against the new package.
New findings below:
Issue1: Specify a remote-IOM connection profile and try to run SAS Code, it keeps showing 'SAS code running' initially
Issue2: Cancel sas code running, Ran man icon will be disabled
Issue3: Unreadable Chinese characters on result page.
Issue4: Errors or something unexpectedly happened if .sas file name contains Chinese characters.
Issue5: Errors or log issue with code having NLS characters.
Issue6: Unreadable Chinese characters in output log
Issue3 to issue6 , please check the attached doc file.  I listed different scenarios which had errors or something improperly.

See issue https://github.com/sassoftware/vscode-sas-extension/issues/583 . It is quite important to match the encoding of powershell and SAS session (UTF-8 SAS can't read zh_CN Powershell string, vice versa), and then the SAS local with powershell display language. It is impossible to change the encoding of SAS during the session but only at the startup (e.g. sasv9.cfg). So that it seems not workable to force using UTF-8 in powershell. It might be better to leave it to user decided. See issue https://github.com/sassoftware/vscode-sas-extension/issues/594

Issue3: Unreadable Chinese characters on result page.
If possible,  check the original html file in the SAS temp to see whether it is good or not.

It looks that the unreadable Chinese characters appear at a fixed position.  It is encoded as "EF BF BD". Something associated with boundary.

Create a new profile with SAS 9.4 (Remote-IOM). The property description is not consistent with the connection type. Put mouse over 'host', 'port' and 'username'.

I noticed this (incorrectly references "SSH") in the new connection prompts as well (even though I selected IOM in this case, not SSH)...

I assume this still needs to be resolved?

I can successfully run code with MBCS/DBCS characters as long as the SAS server session encoding is set to one that can handle those characters (which one would expect, and how other clients have always behaved). The ODS results also properly render the characters. However, I have not been able to get the DBCS/MBCS characters to render in the log in VS Code in any scenario, even when the VS Code display language matches the SAS session locale and encoding and regardless of whether I explicitly set the Powershell encoding to Simplified Chinese (936) or utf8 (65001) (in the Powershell $PROFILE). So, there still appears to be a problem with that scenario (rendering DBCS/MBCS characters in the log).

Create a new profile with SAS 9.4 (Remote-IOM). The property description is not consistent with the connection type. Put mouse over 'host', 'port' and 'username'.

I noticed this (incorrectly references "SSH") in the new connection prompts as well (even though I selected IOM in this case, not SSH)...
I assume this still needs to be resolved?

Good catch. That's been resolved in https://github.com/sassoftware/vscode-sas-extension/pull/592/commits/1ffda86cece6cf5b78f7bceb69740be46e578b70 . I basically updated the text to say "SAS 9 server" instead of "SAS 9 SSH server." Let me know if we'd prefer to keep the strings more specific (i.e. "SAS 9 IOM server" and "SAS 9 SSH server")

I can successfully run code with MBCS/DBCS characters as long as the SAS server session encoding is set to one that can handle those characters (which one would expect, and how other clients have always behaved). The ODS results also properly render the characters. However, I have not been able to get the DBCS/MBCS characters to render in the log in VS Code in any scenario, even when the VS Code display language matches the SAS session locale and encoding and regardless of whether I explicitly set the Powershell encoding to Simplified Chinese (936) or utf8 (65001) (in the Powershell $PROFILE). So, there still appears to be a problem with that scenario (rendering DBCS/MBCS characters in the log).

This has been resolved for me with @scottdover's latest changes, thanks! The DBCS characters are now rendering correctly in the log as well. Thus, DBCS characters are now rendering correctly everywhere I'd expect when an appropriate SAS server session encoding is set -- in the log and ODS results.

I did find an issue though...
The "Close Session" menu is no longer available after making changes to the active profile, even though I clearly have an open session.
Steps to reproduce:

Define an IOM profile (ex. uitsrv05, port 8591)
Submit any code (ex. "%put hello;")
Hover over the profile on the status bar at the bottom of VS Code and confirm the "Close Session" menu is present, as expected (but do not click it).
Open the settings.json file and change the port to another server context on that server (ex. port 8596, still on uitsrv05).
Save the settings.json file.
Hover over the profile again on the status bar at the bottom of VS Code and confirm the "Close Session" menu is NO LONGER present.
Submit the same program again.
Hover over the profile again on the status bar at the bottom of VS Code.

Problem: Notice the "Close Session" menu is still not there. I expected it to re-appear after running with the new port, since I thought that would have resulted in a new session.
If you explicitly close the session (in step #3), then the problem does not occur.
You can recover the "Close Session" menu by explicitly running the command from command palette to close the SAS session, then make a new session.
I just realized, it looks like I'm not getting a new session when I only change the port (and confirmed same even if I change the host), since I was not re-prompted for password. So, it looks like changing any active profile properties doesn't close the active session, which is ok, but the "Close Session" menu should not have gone away.

I did find an issue though... The "Close Session" menu is no longer available after any changes to the settings.json file are saved, even though I clearly still have an open session.
Steps to reproduce:

Define an IOM profile (ex. uitsrv05, port 8591)
Submit any code (ex. "%put hello;")
Hover over the profile on the status bar at the bottom of VS Code and confirm the "Close Session" menu is present, as expected (but do not click it).
Open the settings.json file and change the port to another server context on that server (ex. port 8596, still on uitsrv05).
Save the settings.json file.
Hover over the profile again on the status bar at the bottom of VS Code and confirm the "Close Session" menu is NO LONGER present.
Submit the same program again.
Hover over the profile again on the status bar at the bottom of VS Code.

Problem: Notice the "Close Session" menu is still not there. I expected it to re-appear after running with the new port, since I thought that would have resulted in a new session.
If you explicitly close the session (in step #3), then the problem does not occur.
You can recover the "Close Session" menu by explicitly running the command from command palette to close the SAS session, then make a new session.
I just realized, it looks like I'm not getting a new session when I only change the port (and confirmed same even if I change the host), since I was not re-prompted for password. So, it looks like changing any active profile properties doesn't close the active session, which is ok, but the "Close Session" menu should not have gone away.

It's not specific to this PR and can be reproduced in main branch with Viya profile too. It can be addressed separately.

I did find an issue though... The "Close Session" menu is no longer available after any changes to the settings.json file are saved, even though I clearly still have an open session.
Steps to reproduce:

Define an IOM profile (ex. uitsrv05, port 8591)
Submit any code (ex. "%put hello;")
Hover over the profile on the status bar at the bottom of VS Code and confirm the "Close Session" menu is present, as expected (but do not click it).
Open the settings.json file and change the port to another server context on that server (ex. port 8596, still on uitsrv05).
Save the settings.json file.
Hover over the profile again on the status bar at the bottom of VS Code and confirm the "Close Session" menu is NO LONGER present.
Submit the same program again.
Hover over the profile again on the status bar at the bottom of VS Code.

Problem: Notice the "Close Session" menu is still not there. I expected it to re-appear after running with the new port, since I thought that would have resulted in a new session.
If you explicitly close the session (in step #3), then the problem does not occur.
You can recover the "Close Session" menu by explicitly running the command from command palette to close the SAS session, then make a new session.
I just realized, it looks like I'm not getting a new session when I only change the port (and confirmed same even if I change the host), since I was not re-prompted for password. So, it looks like changing any active profile properties doesn't close the active session, which is ok, but the "Close Session" menu should not have gone away.

It's not specific to this PR and can be reproduced in main branch with Viya profile too. It can be addressed separately.

Thank @scnwwu. I agree. I'll open a separate bug.

I tested against a server configured with encrypted communications (specifically, Triple DES algorithm). I submitted very simple code (ex. "proc print data=sashelp.class; run;"). The code appears to run correctly, but the Results tab does not appear and the ODS results are not displayed no matter what I submit. @scottdover, I'll send you the connection information for that server, so you can test it out.

Hey @scnwwu / @Zhirong2022 . Do you both mind taking one more look at this one you get a chance? I've been working with Joe and Casey to test/resolve all the things, but just want to get your final blessing before merging things in

I went through all the issues raised in PR.
And now only two left for verification:

Unreadable Chinese characters, the root cause is that I use SAS Dark theme to render the specific SAS code. So, it is nothing to do with OS locale, VSC language configure or locale94/IOM connection.
As @clangsmith  mentioned the server configured with encrypted communications (specifically, Triple DES algorithm). It cannot connect to the server. I have no idea how to do verification.

As @clangsmith mentioned the server configured with encrypted communications (specifically, Triple DES algorithm). It cannot connect to the server. I have no idea how to do verification.

Fwiw, we believe this one is now resolved. @clangsmith tested this yesterday I believe

As @clangsmith mentioned the server configured with encrypted communications (specifically, Triple DES algorithm). It cannot connect to the server. I have no idea how to do verification.

Fwiw, we believe this one is now resolved. @clangsmith tested this yesterday I believe

Correct. I have tested it and it is now (after Scott's fix) working as expected for me -- the ODS results are now appearing. I played around with that server using our extension for awhile and did not encounter any other issues.

The issue that Chinese characters were not rendered correctly has been fixed in the latest package. However I found a performance issue with coding running.
1.Suppose I ran the same sample code against old package and new package.
old package: 5-8 seconds to show result page
new package 50-1miniute30seconds  to show result page
2. If try to run the given code below to show a big table with same IOM connection.
proc print data=maps.africa;

old package: 1minute50seconds  to show result table
new package: sometimes it will keep showing 'SAS code running' without any response.
At such scenario, try to cancel the running.  The running man icon will be disabled even witching to a new profile.

The issue that Chinese characters were not rendered correctly has been fixed in the latest package. However I found a performance issue with coding running. 1.Suppose I ran the same sample code against old package and new package. old package: 5-8 seconds to show result page new package 50seconds-1miniute30seconds to show result page 2. If try to run the given code below to show a big table with same IOM connection.
proc print data=maps.africa;

old package: 1minute50seconds to show result table new package: sometimes it will keep showing 'SAS code running' without any response. At such scenario, try to cancel the running. The running man icon will be disabled even switching to a new profile.

This was one of my concerns in making a change to "collect all the bytes" first (in https://github.com/sassoftware/vscode-sas-extension/pull/592/commits/ad48cd15c82de5327db9f50411a6e69992178f93). Before this commit, we were reading bytes and writing them directly to a file. Now, we are storing all of the bytes in memory (which we'll eventually exhaust) and then attempting to encode all of them at once (which again, could be a problem for a huge number of bytes).
I'm not sure the best approach to take here. It seems the options are:

Write to the file system as often as possible, keeping a lower memory overhead but ending up with some encoding issues, or...
Write to the file system once, with a much higher memory footprint but no encoding issues

@clangsmith @smorrisj @scnwwu Thoughts on how to proceed, here?

The issue that Chinese characters were not rendered correctly has been fixed in the latest package. However I found a performance issue with coding running. 1.Suppose I ran the same sample code against old package and new package. old package: 5-8 seconds to show result page new package 50seconds-1miniute30seconds to show result page 2. If try to run the given code below to show a big table with same IOM connection.
proc print data=maps.africa;

old package: 1minute50seconds to show result table new package: sometimes it will keep showing 'SAS code running' without any response. At such scenario, try to cancel the running. The running man icon will be disabled even switching to a new profile.

This was one of my concerns in making a change to "collect all the bytes" first (in ad48cd1). Before this commit, we were reading bytes and writing them directly to a file. Now, we are storing all of the bytes in memory (which we'll eventually exhaust) and then attempting to encode all of them at once (which again, could be a problem for a huge number of bytes).
I'm not sure the best approach to take here. It seems the options are:

Write to the file system as often as possible, keeping a lower memory overhead but ending up with some encoding issues, or...
Write to the file system once, with a much higher memory footprint and potential for failed runs, but no encoding issues

@clangsmith @smorrisj @scnwwu Thoughts on how to proceed, here?

Some thoughts:

How about simply read files with Nodejs for local connection (same as main branch)? Currently it already checks this._config.protocol === ITCProtocol.COM to resolve file path. I don't know if it will be faster than the IOM approach. If we're not able to make a perfect solution for remote IOM, at least it's not a regression compared to v1.5.
How about replace System.IO.StreamWriter with System.IO.FileStream? So we read bytes and write bytes thus can be safely sliced. I guess this is a desired way if it works, as it supports text and non-text files uniformly.
If the above doesn't work, how about replace $objFile.OpenBinaryStream with $objFile.OpenTextStream? It looks like designed for characters.

The issue that Chinese characters were not rendered correctly has been fixed in the latest package. However I found a performance issue with coding running. 1.Suppose I ran the same sample code against old package and new package. old package: 5-8 seconds to show result page new package 50seconds-1miniute30seconds to show result page 2. If try to run the given code below to show a big table with same IOM connection.
proc print data=maps.africa;

old package: 1minute50seconds to show result table new package: sometimes it will keep showing 'SAS code running' without any response. At such scenario, try to cancel the running. The running man icon will be disabled even switching to a new profile.

This was one of my concerns in making a change to "collect all the bytes" first (in ad48cd1). Before this commit, we were reading bytes and writing them directly to a file. Now, we are storing all of the bytes in memory (which we'll eventually exhaust) and then attempting to encode all of them at once (which again, could be a problem for a huge number of bytes).
I'm not sure the best approach to take here. It seems the options are:

Write to the file system as often as possible, keeping a lower memory overhead but ending up with some encoding issues, or...
Write to the file system once, with a much higher memory footprint and potential for failed runs, but no encoding issues

@clangsmith @smorrisj @scnwwu Thoughts on how to proceed, here?

Some thoughts:

How about simply read files with Nodejs for local connection (same as main branch)? Currently it already checks this._config.protocol === ITCProtocol.COM to resolve file path. I don't know if it will be faster than the IOM approach. If we're not able to make a perfect solution for remote IOM, at least it's not a regression compared to v1.5.
How about replace System.IO.StreamWriter with System.IO.FileStream? So we read bytes and write bytes thus can be safely sliced. I guess this is a desired way if it works, as it supports text and non-text files uniformly.
If the above doesn't work, how about replace $objFile.OpenBinaryStream with $objFile.OpenTextStream? It looks like designed for characters.

@scottdover and I stepped through the EG code and we are now doing essentially the same thing that EG is (pseudo-code):
binaryStream = SAS.IFileref.OpenBinaryStream(Mode.StreamOpenModeForReading)
while (true)
{
	binaryStream.Read(CommonDownloadHelper.ChunkSize, out data); // EG ChunkSize is 8192
	if (data == null || data.Length == 0)
		break;
	fileStream.Write((byte[])data, 0, data.Length);
}

The previous issue is that we were encoding the byte chunks as we received them, which would break up DBCS characters if they fell on the border.
The testing I have done since the last fix appears to have resolved the issue. I also feel good knowing this is now the same way that EG handles it, considering EG has been doing it for many years (field-proven experience).
(Btw, given that printing maps.africa generates a 20MB results file, I consider that to be an edge-case scenario. EG doesn't even attempt to open results files larger than 5MB by default. I still tried this scenario in the extension with the latest changes, and it was successful, but took about a minute total before I saw the results. This is not unreasonable in my opinion.)
@scnwwu, @Zhirong2022, when you all return next week, please take another look, and if you feel this is reasonably addressed, don't encounter any other serious issues, or have any other concerns, please feel free to go ahead and merge this in.

All issues have been fixed properly.

Thanks, @scnwwu and @Zhirong2022 🎉


docs typo in PauliList.evolve

Environment

Qiskit Terra version: Online documentation
Python version:
Operating system:

What is happening?
https://qiskit.org/documentation/stubs/qiskit.quantum_info.PauliList.evolve.html
I think the value returned with frame = 'h' should be different from the value returned with frame='s'. Seems like one or two daggers need to move:

If we trust the docs for Pauli.evolve, then the error on PauliList.evolve docs is for the default behavior (frame = 'h'), which should instead be C^dag P C :

How can we reproduce the issue?
https://qiskit.org/documentation/stubs/qiskit.quantum_info.PauliList.evolve.html
What should happen?
See above
Any suggestions?
No response

You are correct. the documentation for PauliList should be the same as Pauli.

Is this issue open to work? I see a pull request already being made but some error in reviewing and issue not yet been closed. If its open, I would like to work on it!

Hi @Ak-ash22, It seems like @thspreetham98 is actively working on it. Thanks for the interest anyway! There are more good first issues in https://qisk.it/good-first-issues

@thspreetham98, do you mind leave a comment in this issue so we can assign it to you?

Please assign this to me.

Please review https://github.com/Qiskit/qiskit-terra/pull/10334


add the ability to visualize and select connected devices

fixed by https://github.com/dart-atom/dartlang/pull/756


exitonfailure conflicts with retries

The option --exitonfailure conflicts with the retries. It should only exit after the max number of retries.
Currently with that option the retry will fail

Any news about it ?


Questions about javadoc generation

Hello, could you please share your scripts about pre-processing the java comment? I  tried to do that myself but it seems that the sentence looks different from yours.

I would encourage you to use Java Language processor implemented here using Tree Sitter. You can modify the code to get different types of comments ({"comment", "line_comment", "block_comment"}).


Canaloplasty with suprachoroidal drainage (CSD)

137760 GLAUCOMA, PRIMARY OPEN ANGLE; POAG
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC9585494/
Canaloplasty with suprachoroidal drainage (CSD) is a surgical technique used to treat glaucoma, a condition that can damage the optic nerve and lead to vision loss. This procedure involves accessing the drainage canal (Schlemm's canal) in the eye and creating a small opening in the suprachoroidal space, the space between the choroid layer and the sclera layer of the eye.
During CSD, a small incision is made in the eye to access Schlemm's canal, and a microcatheter is used to dilate and expand the canal. A small opening is then created in the suprachoroidal space, which allows for increased drainage of aqueous humor, the fluid that circulates inside the eye. This increased drainage can help to lower intraocular pressure in the eye, which is a key factor in glaucoma.
CSD is a newer variation of the traditional canaloplasty technique, and it may have some advantages over traditional canaloplasty, including potentially greater intraocular pressure reduction and a lower risk of complications such as bleeding and infection.

added term. Will be availablle in next release.


What is the right mechanism for fixed attributes? (CON-1103)

Some comments from Boris makes me question how fixed attributes are handled in esp-matter.
Right now they are put into the attribute partition using mfg tool. But then what about OTA? Say you OTA in a different image supporting new locales. OTA can't update the attribute partition.
That leads me to conclude we are using the wrong mechanism to store fixed attributes. An alternative scheme could work like this....

User app defines the fixed attributes as an array of constants.
When the cluster is created, pass the array into esp-matter
esp-matter internally uses attribute overrides to turn the constants into fixed attributes.
Now the attributes update with OTA.

You'll still use mfg_tool for the certificates, it is just the fixed strings which would move into the code.
This brings up some interesting problems with rebranding. Is there a generic OTA image for all brands? or does each branch need an OTA image? I'd probably lean to each brand needing an image (the OEM vendor would produce an OTA image for each licensed brand).
The "Device instance information options" should use the existing mechanism.
The "Device instance options" need to use code constants. (this also solves the problem of how to internationalize Fixed Labels)
Device instance information options:
  -v VENDOR_ID, --vendor-id VENDOR_ID             Vendor id
  --vendor-name VENDOR_NAME                       Vendor name
  -p PRODUCT_ID, --product-id PRODUCT_ID          Product id
  --product-name PRODUCT_NAME                     Product name
  --hw-ver HW_VER                                 Hardware version
  --hw-ver-str HW_VER_STR                         Hardware version string
  --mfg-date MFG_DATE                             Manufacturing date in format YYYY-MM-DD
  --serial-num SERIAL_NUM                         Serial number
  --enable-rotating-device-id                     Enable Rotating device id in the generated binaries
  --rd-id-uid RD_ID_UID                           128-bit unique identifier for generating rotating
                                                  device identifier, provide 32-byte hex string, e.g.
                                                  "1234567890abcdef1234567890abcdef"

Device instance options:
  --calendar-types CALENDAR_TYPES [CALENDAR_TYPES ...]
                                                  List of supported calendar types. Supported Calendar
                                                  Types: Buddhist, Chinese, Coptic, Ethiopian,
                                                  Gregorian, Hebrew, Indian, Islamic, Japanese, Korean,
                                                  Persian, Taiwanese
  --locales LOCALES [LOCALES ...]                 List of supported locales, Language Tag as defined by
                                                  BCP47, eg. en-US en-GB
  --fixed-labels FIXED_LABELS [FIXED_LABELS ...]  List of fixed labels, eg: "0/orientation/up"
                                                  "1/orientation/down" "2/orientation/down"
  --supported-modes SUPPORTED_MODES [SUPPORTED_MODES ...]
                                                  List of supported modes, eg:
                                                  mode1/label1/ep/"tagValue1\mfgCode, tagValue2\mfgCode"
                                                  mode2/label2/ep/"tagValue1\mfgCode, tagValue2\mfgCode"
                                                  mode3/label3/ep/"tagValue1\mfgCode, tagValue2\mfgCode"

@jonsmirl Agreed on the fixed label part, they has to be the part of code and OTA-able. Will see how this can be sorted.

I have raised a PR https://github.com/project-chip/connectedhomeip/pull/32893/files which adds few APIs to set that fixed data.
In esp-matter, we'll hide the one level complexity through cluster::create() and take these strings as input and then the cluster create will call the appropriate APIs.

Hi @jonsmirl, here's how we can update the fixed type attributes in device info provider, https://github.com/project-chip/connectedhomeip/blob/master/docs/guides/esp32/providers.md#example.
I'm checking how this can be handled/consumed when creating the esp-matter data model.


icebram equivalent?

Is there any way to replace initialisation data for BSRAM in gw1nr devices like you can on ice40 with icebram?

AFAIK bsram currently is not supported

gowin_pack now recognizes INIT for memory primitives and creates a corresponding chunk in the generated image.


Can this be built for Raspberry pi 2b (intend to use with external wifi/bluetooth dongle)

Any one tried to build this for Raspberry pi 2b?  Appreciate steps to do that if different..
Many thanks

As far as I know, RPi does not have USB OTG capability. It is required for us to connect with the car.