Would like to separate the WPILib specific code in AHRS C++ implementation

My intent is to wrap the C++ class with cython so we can use it with RobotPy. RobotPy has it's own version of WPILib, so using the C++ WPILib would most likely cause conflicts.
At the moment, I think the best approach would be to turn the existing code into AHRSBase, and remove the WPILib.h include and the inheritance from SensorBase et al. The RegisterIO interface might be a good boundary here, I haven't looked too carefully at it.
Then, a separate class would be called AHRS, which would have the WPILib specific pieces in it. This would retain backwards compatibility, and be good separation of concerns anyways.

Before embarking on this, there are a couple of design considerations to work out.

The C++ RoboRIO Library for navX-MXP supports four interfaces:  TTL UART, USB, I2C and SPI.  Thus, the AHRS class uses the WPI library for Serial IO (TTL UART, USB) and Register IO (I2C and SPI)

The AHRS class inherits from SensorBase, LiveWindowSendable and PIDSource, in order to integrate well with the DriveStation and the PIDController infrastructure.

So abstracting out the Register/Serial IO as well as the PIDController and LiveWindowSendable portions out of the AHRS class is a significant amount of work.  Not impossible, but not trivial either.
In the same way that the C++ and the Java classes are similar but different, my intuition is that a Python library should be different too.
Also, the design should take into account both Serial interfaces, not only because there are many teams using TTL UART and USB, but also because the USB interface is featured on the new navX-Micro sensor, which is compatible w/the navX-MXP interface.
I'm happy to help and work with you, and would like to see a python library for FRC, so let's continue the discussion and work out the best path forward.  Two things that would help me:

Please post some information about the RobotPy WPILib so that we could analyze the similarities and differences between that and the C++ WPI Library.
Please also post links to info on how to get started w/RobotPy - I need to understand who I might actually test what might be developed.

Hm.
RobotPy's WPILib is pure python, and talks to the hardware via the HAL layer using ctypes, and can be found here. It does not have a WPILib serial port implementation yet, and the SPI/I2C implementations are very lightly tested at the moment.
The primary reason I was thinking about co-opting the C++ implementation was because I thought I had seen some math-intensive code somewhere -- and given that it's an interpreted language, python isn't particularly good at that sort of thing so it would be better to try and wrap it with cython. However, I looked around a bit, and couldn't find the mathy bits I thought I had seen previously. If that isn't the case, then maybe it'll just be easier to translate it all into pure python and be done with it.
Someone had put together a hacky implementation that uses a serial port, but I'd like to do a little bit better than that.
Information about running RobotPy can be found on our documentation site, and there are a number of examples ported from WPILib on the github repo. The current release does not work on the 2016 image yet.

Based on how "fresh" the RobotPy HAL sounds, I'd recommend using the SPI interface.  I2C can get rather complex - more so than SPI - and SPI is about 20X faster (2Mhz for SPI vs. 100Khz for I2C)  - not to mention SPI is pretty easy to debug if we have to break out a logic analyzer.
A port of C++ (or perhaps Java?) to Python is what I'd recommend.  As far as the math operations you mentioned, w/the new v2.0 navX-MXP/navX-Micro firmware that's been released, all of the math intensive operations have been moved down into microcontroller, which is a very capable 100Mhz 32-Bit ARM processor.  That makes the library porting simpler, and uses less cycles on the host robot controller.
There's a bit of thread synchronization code and also time delay code that will need some trivial porting.
So I think that's the best path.
The sources you'll want to look at are four files:

the AHRS class (for the interface definition, mostly)
RegisterIOSPI.cpp (here's where you'd map things to the Python HAL SPI IO functions)
RegisterIO.cpp (for the register value decoding), and
IMURegisters.h (for the decode helper functions and CRC calculations)

I quickly reviewed the RobotPy WPILib, it seems at the interface level like identical to the C++ SPI class interface - but of course the devil's in the details.
Once you've got something you'd like me to review, let me know.  I'm interested in getting a RobotPy infrastructure setup here in our lab, let me start spending a few cycles looking into that....

Yes, we're trying to keep the python WPILib as close to the Java WPILib as possible, so it's pretty similar. :)
Since all the math stuff is down in the firmware, then a pure python port does seem like the way to go. I'm working on the WPILib 2016 migration for RobotPy first for a few days, then I'll move to the NavX stuff after that I think... we've found that translating Java APIs tends to make more sense. I've been looking forward to playing with the NavX for a few weeks now!
The current RobotPy should be able to co-locate with either C++ or Java programs on the same RoboRIO as long as you haven't filled up your disk. The 2015 releases will only work on a 2015 image of the RoboRIO, we haven't pushed builds for 2016's image yet.

On the note of porting, do you guys know if there are going to be any major code base changes for the NavX between now and kickoff? I also ported the NavX stuff to the C# WPILib I've been writing, and hope there are no major changes that would require another rewrite.
Also Dustin I have a NavX on my test bench as well, so if you need any help testing the NavX, or any of the 2016 code changes I can help, as mine has the 2016 image too. I can also test any of the Java or C++ changes with the 2016 image if you guys at Kauailabs would like.

Hi Thad,
The navX firmware for the 2016 Build Season is now released, as are the libraries (announcement just went out on ChiefDelphi).  We just released the 2.3 firmware which has some minor fixes related to the TTL UART/USB interfaces.  That's recommended for your testing.  The C++/Java libraries and samples have been unchanged for several months now; the recent activity just completed was the work w/Team 900 (Zebracorns) on their new V2 LabView Library.
Looking back over the past few months, the libraries did undergo a rewrite about 3-4 months ago and new samples were added.  Functionally, a few functions were moved down to the microcontroller (the zeroing of the yaw, and the velocity and displacement integration [that last bit is accurate to 1 meter every 15 seconds, so I'm not trying to get anyone's hopes up as to that it can be used for autonomous displacement tracking]).  But if you forked this code within the last few months, you should be fine.
As to testing, we've been doing a lot of it here lately, but I will absolutely never turn down any offer to help test.   And I would be happy to try out the C# library.  I'd like to work out some arrangement whereby KauaiLabs can provide a link and some info about these new Python and C# developments that are underway and/or complete....

I forked it about a week ago, and looking at the commits since then it doesn't look like anything has changed. I was a little disappointed that the FPGA MXP issue still hadn't gotten fixed, but that seemed like a FIRST issue, and knowing a simple workaround makes it not a problem for us.
The C# libraries can be found here
https://github.com/robotdotnet/WPILib
The NavX stuff is in the Extra project. Note everything has been upgraded to 2016 spec, so you would need a 2016 image to test.

I don't have a 2016 image yet (didn't make it onto the beta team, unfortunately) - that's the one area where the libraries might change a small bit once we finally do get that and the beta.
Took a quick look at the C# libraries, I'm guessing you used the Java  lib as a starting point.  I noticed that you had the most recent change (which was a reversing of the pitch/roll register values), so I'm confident you're in synch there.
Are you still seeing the FPGA MXP issue?  I have never got definitive word from NI as to whether they found anything.  As noted in a conversation w/you earlier, we did discover that there were cases where an internal I2C bus glitch could cause the navX-MXP to stop sending data out the communication interfaces, and the fix for that went in around the 2.1 timeframe.  We've tested extensively that there are no issues in this area, but would very much appreciate it if you could go over that area in your testing.

I will do a lot more testing in the next few weeks to see if I can reproduce it again. I thought I saw it when I was porting the code, but I think at the time I was testing with some wrong I2C code. I'll update to the newest firmware and do more testing over the next few weeks. Hopefully its gone, and it was just my testing code that made me think I saw it.
It looks like 330 submitted a pull request to you guys for the 2016 image, and the things changed there are the only things I remember changing anyway.
And yeah it was ported from Java, as it only took about 20 minutes to copy paste, and refactor the few differences needed to be made. That's why all the documentation is messed up. I'm working on fixing that today.

Are you using I2C?  I'd highly recommend SPI, it's much faster than I2C and if you're using the C++ or Java WPI Lib (which don't support large burst reads on I2C) I2C is way slower than SPI.  I am very happy w/the SPI performance and reliability, for C++ and Java I2C is just not where I want it to be - I'm hopeful that this has been resolved in the latest WPI libraries.  There's some more discussion on that on the new wiki (http://navx-mxp.kauailabs.com/guidance/selecting-an-interface/).  All of our samples are now using SPI, and that's what we're recommending to anyone who asks.
The pull request from Joe Ross was just the PIDSourceType change, I'm guessing you hit that while porting.
Do you mind if we put a link to your C# work online once you're ready?  I realize it doesn't make sense until the 2016 firmware/libraries are available, but I'm thinking if it was available at kickoff, it could help some others.

We used SPI last season and plan on using it again. We saw the issue last year on both SPI and I2C however. Maybe it was just an issue with the LabVIEW libraries. This year we are switching. I don't have access to the 2016 LabVIEW beta, so I won't be able to test them there. But we would be using SPI this year as well, and since we know the workaround we are fine using it.
Yeah, the PIDSourceType was the only change that affected the code I believe. The I2C interface has not changed at all, and its still limited to 8 bytes at a time from the looks of things.
We would be happy for you to put a link to the C# work. Because of the rules, I have to release the code before Kickoff, and I was going to do that probably the Thursday or Friday before kickoff. Its pretty much ready for release now, just want to verify it with the final image when we get it. I should be able to port the examples by then too, so if you want a copy of those I should be able to get them to you.

The I2C 8 byte restriction was fixed this summer. https://usfirst.collab.net/gerrit/gitweb?p=allwpilib.git;a=commit;h=c21c47f546e5f30b49cad58634591c86399c07fe
I did some testing with the NavX, but not enough that I felt comfortable taking out the 8 byte workaround with my pull request.

Good to hear, sciencewhiz, I'm looking forward to getting ahold of this WPILib update, because it's going to mean that I2C is much more usable for data-intensive sensors like navX-MXP.  I still think SPI's a superior interface for navX-MXP (due to the higher clockrate) but navX-MXP (and navX-Micro) definitely get better performance for I2C once this is publicly available.

Interesting. I had forgotten about that. I changed MAX_WPILIB_I2C_READ_BYTES to 100,000 on my copy of the code, and it seems to be  working fine. I'll leave it running for a few hours and make sure it stays running and report back. Is that the only number that should need to be changed?

@kauailabs if you send an email to Brad Miller, you might be able to get access to the Beta test image. They gave Thad and I access, and neither of us are on beta teams -- but we're doing things for the greater good.

Thanks @virtuald, I appreciate the advice, I'll give that a try.
@ThadHouse, from the navX-MXP perspective, if we can get a contiguous read of 111 (0x6F) bytes, we're good.   I think a logical maximum is 256 bytes (given the 8-bit I2C addressing).  It's interesting (perhaps) to note that sensors w/more than 256 bytes of register space (e.g., the Bosch BNO055) use a bank-switching technique when accessing registers, so I believe this is a safe maximum given the I2C protocol specification and current practice by well-known silicon vendors.

@kauailabs Right now its telling me its sending and reading a single 108 byte packet. If I set MAX_WPILIB_I2C_READ_BYTES back to 7, it sends multiple 7 byte packets, and then a 3 byte packet. So it definitely seems like large byte array readings are working. And its been stable for the past 20 minutes.

@ThadHouse That's great news.  What time is it in Oregon, must be about midnight? Anyways, hopefully I'll be able to replicate this myself soon.
Please note that the navX-MXP SPI protocol inclues a CRC, whereas the I2C protocol does not.  It was added to SPI protocol because we envisioned supporting the higher bitrates with it and were concerned about detecting data corruption if noise was encountered.   I wish we had done this for I2C (it's a breaking change to fix it now), but it's also less of an issue because of the lower I2C bitrate and it's less "natural" to do so, because I2C is really designed to be a "read from registers" protocol.   Just wanted to be clear that because of this, even if we get a single contiguous burst read, that SPI will still be theoretically superior both because of higher bitrate as well as because of the protocol error detection.   The RoboRIO libraries will discard any SPI read where there's a CRC failure, but will have invalid data w/an I2C read.  I'll take the blame for the design issue.  Just wanted to make sure you were aware of it.

It's just approaching 11 now, so its not too late.
We still plan on using SPI anyway, but having I2C even a little faster could be very helpful.

When I looked at what it would take to update for the beta 2016 C++ changes, it was more then just the PIDSourceType, because the Task class changed significantly. I played with it a little and couldn't get the task part working and gave up.

Thanks, @sciencewhiz.  I just received access to the 2016 WPI library beta and will start looking into it.

@kauailabs I think I'm mostly through my 2016 updates for RobotPy, so I'll most likely take a shot at porting this during the weekend. I'll let you know how it goes.

I'm going to close this, as it's not necessary. I've got a pure python port working most of the way, just have some offsets incorrect or something. Tracking my progress at https://github.com/robotpy/robotpy-wpilib-utilities/issues/9


rpc: Make sure we internally use uint32 for latestledger in getEvents

Followup to https://github.com/stellar/soroban-tools/pull/1097

If it's not too much work, please close this PR and reopen in soroban-rpc. We could also merge this PR here (after tests pass) and cherry pick over to soroban-rpc, but it would be easier to directly merge this change to soroban-rpc.


Unmet peer dependency for<EMAIL_ADDRESS>

It looks like<EMAIL_ADDRESS>is a required peer dependency for the package<EMAIL_ADDRESS>Is it appropriate to change the top-level package.json file to include "eslint": "^3.19.0" instead of "eslint": "3.15.0"?

Indeed, thanks again @reefloretto !


Bug: Multi GPU

Current, I have a bug about data parallel on this repo. Please do not use multi GPU when training.

close it at: #9


asciidoctor / adoc-foundation .toc-root font setting

I'm using asciidoctor.css together with adoc-foundation.css and I've added a custom font everywhere except for code blocks. However, I've noticed that the element .toc-root (the title of the document at the top of the table of contents) is defined nowhere in any of the css files and therefore has retained the original font (wherever that was defined then, since apparently it also didn't inherit this setting from anywhere that I've changed the settings).
Could a setting for .toc-root maybe be added to the stylesheets? Thanks! :)

Nevermind, this is due to asciidoctor-multipage


PSTGRSCROC-135

Codecov Report

Merging #430 into master will decrease coverage by 0.63%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##             master     #430      +/-   ##
============================================
- Coverage     73.96%   73.32%   -0.64%
  Complexity      212      212
============================================
  Files            45       45
  Lines           626      626
  Branches         61       60       -1
============================================
- Hits            463      459       -4
- Misses          118      122       +4
  Partials         45       45

Impacted Files
Coverage Δ
Complexity Δ

...sqlboot/model/resourcetype/wrappers/RestWrapper.kt
52.38% <83.33%> (-19.05%)
5 <4> (ø)

Continue to review full report at Codecov.

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

@rultor merge

@rultor merge

@mgramin OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@mgramin Done! FYI, the full log is here (took me 7min)


Make monitoring prow as deployment

Fixes: https://github.ibm.com/powercloud/container-dev/issues/1536

/cc @Rajalakshmi-Girish

/lgtm


Adjust AppendObject to utilize NameAttribute from CsvHelper

Hi Steve,
You are too fast!
I noticed an issue when comparing results more closely between AppendCsv and AppendObject
I'd not taken into account usage of the NameAttribute.
But the conversion of dates, numbers and bools, is much better this way I find.
Thanks.

@drgrieve just merged and updated the version to get published to 2.0.22.  Thanks for the contribution.


DOC: remove duplicate link

As per @scpeters comment in https://github.com/ignitionrobotics/sdformat/pull/666

Codecov Report

Merging #702 (5a2602d) into sdf11 (d25bad7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            sdf11     #702   +/-   ##
=======================================
  Coverage   88.08%   88.08%
=======================================
  Files          73       73
  Lines       11023    11023
=======================================
  Hits         9710     9710
  Misses       1313     1313

Continue to review full report at Codecov.

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

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-03-25-fortress-edifice-citadel/1343/1


decommission mongo.activity, work the mongo.events instead

nah

we'll work events on top of the activities instead


Swagger: Supply per-route description and summary for each action

Currently the Swagger description and summary of each action is used for all routes, e.g. given an Action named Foo with two routes (one GET and on POST for the usual collection interface) both routes will have the exact same description and summary (Foo->getDescription() and Foo->getSummary()) shown in Swagger.
While there may be some Actions where this makes a lot of sense, at least in the case of generic collections it does not.
My proposal would be to return maps from both these functions:
virtual const char* getDescription() const;
virtual const char* getSummary() const;

would then become
virtual const std::map<qttp::HttpPath, const char*> getDescription() const;
virtual const std::map<qttp::HttpPath, const char*> getSummary() const;

Swagger::initialize() would have to be changed accordingly.
I think this would not be such a big change, but really help with usability of the generated Swagger documentation. I'd be willing to implement this, if you don't see any roadblocks and would accept a pull request.

If this issue would go forward it would also make sense to incorporate it for #27.

I also updated the initial proposal with a cleaner API by using a std::map.

I created a commit is a separate branch that implements this. Please note that it is not yet ready for a pull-request. I think a bit more discussion about the API and the consequences are necessary before that.
Once that has happened, I'm happy to take a look at the tests and examples as these are currently not adapted to the new API in the mentioned branch.


Make SI work on older glibc (or static link)

What happened?
Current "si" is not compatible with Ubuntu 2.0.04 LTS.  glibc in this Ubuntu is too old.  Installation completed OK but "si" does not launch.
--- Installation of 'si' release 'bin/si/binary/20230819.004856.0-sha.ac833bf05' complete        /home/gconnor/.local/bin/si: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.32' not found (required by /home/gconnor/.local/bin/si)
/home/gconnor/.local/bin/si: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.33' not found (required by /home/gconnor/.local/bin/si)        /home/gconnor/.local/bin/si: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.34' not found (required by /home/gconnor/.local/bin/si)
!!! We're sorry, but it looks like something might have gone wrong  during installation.
`
How can we reproduce this?
Used normal install curl method on Ubuntu 20.04
curl -sSfL https://auth.systeminit.com/install.sh | sh
Operating system
Linux
Architecture
x86_64 / amd64
What browsers are you seeing the problem on?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Additional context
No response

Workaround (and related talk) meanwhile in PR https://github.com/systeminit/si/pull/2645

You can upgrade your distribution if that's an option. That's what I did. If you need help just let me know and I can talk you through the commands.

I can confirm the issue on my WSL :
% si start
si: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.38' not found (required by si)
si: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by si)
si: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by si)
si: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by si)

~
1 % cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.6 LTS"

I upgraded ubuntu release but did not solve the problem yet (system is fully up to date)
~
125 % cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=22.04
DISTRIB_CODENAME=jammy
DISTRIB_DESCRIPTION="Ubuntu 22.04.3 LTS"

~
% si
si: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.38' not found (required by si)

finally I found workaround. It turns out that building and replacing GLIBC is not easy. Solution based on #PR2645 with small tweak - use arch becase they keep things up to date:
docker run -it --rm --privileged -v /var/run/docker.sock:/var/run/docker.sock -v /root/.local/share/:/root/.local/share/ archlinux bash
pacman -Syu
curl -sSfL https://auth.systeminit.com/install.sh | sh
si start

Given that we do not have the si binary at present, this issue can be closed.
We're looking into more ways for you to be able to use System Initiative, but for now, check out our README on how to get started.


Usage of TypeInfo members is incorrectly reported as not supported when analyzing .NET Framework assemblies

A bunch of APIs on Type are not supported in .NET Core.  Running ApiPort on an assembly that uses these types will recommend adding a call to GetTypeInfo().  Doing so will indeed make the code portable to .NET Core.  However, running the portability analyzer on an assembly that does call GetTypeInfo() for these APIs will still report them as unsupported.
This is because for the .NET Framework, TypeInfo derives from Type and simply inherits those APIs.  So in the method table you never see TypeInfo.get_Assembly, just Type.get_Assembly even if you are accessing it via a TypeInfo object.
To report portability of these APIs correctly would probably require analyzing the IL at the call sites.
The APIs on Type are some of the most common ones encountered that aren't supported on .NET Core.  So this issue makes it hard to use ApiPort to track progress towards .NET Core compatibility.

I suspect the best way to actually fix this is to add the overrides in the full framework itself so that these show up as calls to TypeInfo. Baring that I don't see any clean way to do this directly in ApiPort itself that isn't a big hack.

Known limitation of the catalog.


Blank red screen macOS

Hi, I have this blank red screen when I try to render something on screen. Weirdly, the only things that are rendered properly are images (DrawImage)
This is a screenshot from the drawing example

I'm using
go version go1.14 darwin/amd64
macOS Catalina Version 10.15.4
Do you guys have any idea what might be a problem?

Hi, fortunately I have access to a Mac currently and have the same issue. It must be something about the OpenGL driver (this Mac has an Intel graphics chip), as it behaves differently to other drivers. The technical part is that if a vertex attrib array was not enabled, then nothing would get rendered for some reason.
I just published a new version, so you can try that and hopefully it will resolve the issue.


Windows: [TP3] Move daemon config directory

Signed-off-by: John Howard<EMAIL_ADDRESS>This moves the daemon config location to under %PROGRAMDATA%. And removes another TODO Windows from the codebase :)

LGTM

LGTM


[GHSA-9vjp-v76f-g363]  SnappyFrameDecoder doesn't restrict chunk length any may buffer skippable chunks in an unnecessary way

Updates

Affected products

Comments
Adding ranges to include netty 3.x which was distributed as a single artifact netty under a different groupid: https://central.sonatype.com/namespace/org.jboss.netty

Hi there @normanmaurer! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory


[Bug] When scrolling, the main extension canvas style is disordered

Before you submit this issue, have you checked the following

[X] Is this really a problem?
[X] I have searched the Github Issues for similar issues, but did not find anything.

Affected packages and versions
0.1.14
Reproduction link

merge some cells
apply main  extension canvas, I drew some slashes and rect, try to hide the value
scroll page
the extension canvas is disordered, and the value is shown again.

Please see the video
https://github.com/dream-num/univer/assets/51111411/c74d7720-61b9-4c71-ac97-a9353165f31c
Expected behavior
the extension canvas looks fine, and cover the value
Actual behavior
the extension canvas is disordered, and the value is shown again
System information
No response

Please update the video or provide a demo that reproduces the issue. In the meantime, we will close this issue.
However, feel free to reopen it at any time once you have prepared the requested demo.

https://github.com/dream-num/univer/assets/51111411/f4ba3747-1969-433d-b5dc-54ae8b8b57fc

https://github.com/dream-num/univer/assets/51111411/4b7b3056-86a4-4eb6-beeb-0478c3b71df2

@jikkai hi, please review the updated video


lib.systems: introduce hasSharedLibraries

Description of changes
This PR adds hasSharedLibraries to lib.systems, and causes gcc to use hasSharedLibraries instead of !isStatic as the default for enableShared.
Includes:

https://github.com/NixOS/nixpkgs/pull/240590

Motivation
See https://github.com/NixOS/nixpkgs/pull/238154 for a concrete example.
We need plat.hasSharedLibraries in order to know whether or not to expect gcc (and many other tools) to emit shared libraries (like libgcc_s.so).  Many of the GNU build scripts are smart enough that if you configure them with --enable-shared on a platform (such as arm-none-eabi) that doesn't support dynamic linking, they will simply skip the shared libraries instead of aborting the configurePhase.  Unfortunately the missing shared libraries in the final build product cause very hard-to-troubleshoot problems later on.  So we need something in lib.systems that knows whether or not to expect these shared libraries to appear (and, if they won't appear, to not get upset about that).
The alternative to introducing hasSharedLibraries would be to set isStatic in these situations.  However doing so causes make-derivation.nix to insert -static between the pname and hostPlatform suffix, which is undesirable.
If at some point in the future we eliminate the -static suffix, then hasSharedLibraries can be made equal to !isStatic.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

This PR causes a rebuild of arm-none-eabi because it adds the --disable-shared flag to the ./configure invocation for that platform.


This changes inline code blocks to be white like the larger
formatted blocks.

Automated message from Dropbox CLA bot
@brockwhittaker, it looks like you've already signed the Dropbox CLA. Thanks!

@brockwhittaker can you update this with the full set of changes needed?

Just pushed a new version.

Cool, merged with a few tweaks (e.g. deleting the comment about 12/14 that was no longer related to the font size).  Thanks @brockwhittaker!


Visualize the model/params structure

Problem you have encountered:
Currently it is not clear how to inspect a model structure. For example to see whether some imported model uses dropout, batch-norm,... Or to find out which weight we want to freeze vs fine tune.
Similarly the params returned by model.init is difficult to inspect, as extracting the structure require writing some custom code.
What you expected to happen:
It would be nice if __repr__ was displaying some human-readable structure, like in pytorch

For params, the shape/dtype, rather than 1000+ lines of weights values:

>>> params
FrozenDict({
    params: {
        Encoder_0: {
            Conv_0: {
                kernel: float32[3, 3, 1, 16],
                bias: float32[16],
            },
            Conv_1: {
                kernel: float32[3, 3, 16, 32],
                bias: float32[32],
            },
            Conv_2: {
                kernel: float32[7, 7, 32, 64],
                bias: float32[64],
            },
        },
        Decoder_0: {
            ConvTranspose_0: {
                kernel: float32[7, 7, 64, 32],
                bias: float32[32],
            },
            ConvTranspose_1: {
                kernel: float32[3, 3, 32, 16],
                bias: float32[16],
            },
            ConvTranspose_2: {
                kernel: float32[3, 3, 16, 1],
                bias: float32[1],
            },
        },
    },
})

For model, the modules names & submodules:

Torch for example display the model structure quite clearly, so it is easy to view which operations are used:
import torchvision.models.resnet as resnet

model = resnet.resnet18()
print(model)

ResNet(
  (conv1): Conv2d(3, 64, kernel_size=(7, 7), stride=(2, 2), padding=(3, 3), bias=False)
  (bn1): BatchNorm2d(64, eps=1e-05, momentum=0.1, affine=True, track_running_stats=True)
  (relu): ReLU(inplace=True)
  (maxpool): MaxPool2d(kernel_size=3, stride=2, padding=1, dilation=1, ceil_mode=False)
  (layer1): Sequential(
    (0): BasicBlock(
      (conv1): Conv2d(64, 64, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1), bias=False)
      (bn1): BatchNorm2d(64, eps=1e-05, momentum=0.1, affine=True, track_running_stats=True)
      (relu): ReLU(inplace=True)
      (conv2): Conv2d(64, 64, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1), bias=False)
      (bn2): BatchNorm2d(64, eps=1e-05, momentum=0.1, affine=True, track_running_stats=True)
    )
    (1): BasicBlock(
      (conv1): Conv2d(64, 64, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1), bias=False)
      (bn1): BatchNorm2d(64, eps=1e-05, momentum=0.1, affine=True, track_running_stats=True)
      (relu): ReLU(inplace=True)
      (conv2): Conv2d(64, 64, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1), bias=False)
      (bn2): BatchNorm2d(64, eps=1e-05, momentum=0.1, affine=True, track_running_stats=True)
    )
  )
  (layer2): Sequential(
    (0): BasicBlock(
      (conv1): Conv2d(64, 128, kernel_size=(3, 3), stride=(2, 2), padding=(1, 1), bias=False)
      (bn1): BatchNorm2d(128, eps=1e-05, momentum=0.1, affine=True, track_running_stats=True)
      (relu): ReLU(inplace=True)

I think it makes sense for the __repr__ to print the variable structure (name+shape+dtype). This shouldn't be very difficult to add

Haiku has excellent model summary functionality (documentation here). I think equivalents to haiku.experimental.tabulate and haiku.experimental.eval_summary would be very helpful.

You can use parameter_overview in clu for params visualization:

Commands
import jax
import numpy as np
from flax import linen as nn
from clu import parameter_overview

class CNN(nn.Module):
  @nn.compact
  def __call__(self, x):
    x = nn.Conv(features=32, kernel_size=(3, 3))(x)
    x = nn.relu(x)
    x = nn.avg_pool(x, window_shape=(2, 2), strides=(2, 2))
    x = nn.Conv(features=64, kernel_size=(3, 3))(x)
    x = nn.relu(x)
    x = nn.avg_pool(x, window_shape=(2, 2), strides=(2, 2))
    x = x.reshape((x.shape[0], -1))  # flatten
    x = nn.Dense(features=256)(x)
    x = nn.relu(x)
    x = nn.Dense(features=10)(x)
    x = nn.log_softmax(x)
    return x

key = jax.random.PRNGKey(0)
variables = CNN().init(key, np.random.randn(1, 32, 32, 3))
print(parameter_overview.get_parameter_overview(variables))

+-----------------------+----------------+-----------+-----------+--------+
| Name                  | Shape          | Size      | Mean      | Std    |
+-----------------------+----------------+-----------+-----------+--------+
| params/Conv_0/bias    | (32,)          | 32        | 0.0       | 0.0    |
| params/Conv_0/kernel  | (3, 3, 3, 32)  | 864       | 0.00277   | 0.2    |
| params/Conv_1/bias    | (64,)          | 64        | 0.0       | 0.0    |
| params/Conv_1/kernel  | (3, 3, 32, 64) | 18,432    | 0.000202  | 0.0591 |
| params/Dense_0/bias   | (256,)         | 256       | 0.0       | 0.0    |
| params/Dense_0/kernel | (4096, 256)    | 1,048,576 | -1.54e-05 | 0.0156 |
| params/Dense_1/bias   | (10,)          | 10        | 0.0       | 0.0    |
| params/Dense_1/kernel | (256, 10)      | 2,560     | -0.00159  | 0.0622 |
+-----------------------+----------------+-----------+-----------+--------+
Total: 1,070,794

I think something native equivalent to what @n2cholas showed from Haiku provides would be very nice!

Assigning this to @cgarciae since he is working on this currently.

+1

Closing this since @cgarciae has written Module.tabulate, please check it out!

is there an way to visualize the model like http://alexlenail.me/NN-SVG/AlexNet.html


Changed ff service to work with ros2.

CI tests will be fixed in another pr when we address issue https://github.com/nasa/astrobee/issues/630


Fix computed tag in Ascon-80pq

Resolves #101.

finalize in Ascon80pq uses the last 16 bytes of the key, which is something that might get overlooked

Just to clarify as I was kind of confused by this comment, you mean that the last 16 bytes of the key is xor'red against the last 16 bytes of the state to compute the tag. As the whole key is also xorr'ed against the state before the permutation in the finalization step.

But of course, this has to be fixed in the test vector generation code.

Is this code public?

finalize in Ascon80pq uses the last 16 bytes of the key, which is something that might get overlooked

Just to clarify as I was kind of confused by this comment, you mean that the last 16 bytes of the key is xor'red against the last 16 bytes of the state to compute the tag. As the whole key is also xorr'ed against the state before the permutation in the finalization step.

Yes, indeed. E.g., Ascon v.1.1 does not specify that the last 16 bytes are being used to compute the tag, because this version does not include Ascon80pq and hence does not have to specify which bytes to use. Hence taking an old implementation and carelessly extending it to Ascon80pq can lead to such problems.

But of course, this has to be fixed in the test vector generation code.

Is this code public?

No, it is not. Having access to it would solve this issue much faster, of course. For example it probably would document the version of the paper that was used for the implementation.

Thanks a lot for doing this. I have a new implementation of Ascon, which confirms the tags above.

Given this confirmation, I'll go ahead and get this merged.


Add first e2e test - NEWBIES page

Description
Currently, we have Playwright installed.
However, we need to begin incorporating tests.
Screenshots
No response
Additional information
No response

I will work on this, I will add a that that simulate the user flow of the NEWBIES page


Custom resources - undefined method `split' for nil:NilClass

I am using a custom resource in Chef V12.6, which seems to work fine.  My cookbook depends on "compat_resource", which I think should allow the same cookbook to run on Chef V12.0.  However, I get the following error:
============================================================================
       Recipe Compile Error in /tmp/kitchen/cache/cookbooks/blah/resources/installation.rb
============================================================================

       NoMethodError
       -------------
       undefined method `split' for nil:NilClass

       Cookbook Trace:
       ---------------
         /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/resource.rb:42:in `resource_name'
         /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/resource.rb:52:in `resource_name='
         /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/resource/lwrp_base.rb:41:
in `build_from_file'

       Relevant File Content:
       ----------------------
       /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/resource.rb:

        35:            # Set the resource_name and call provides
        36:            if name
        37:              name = name.to_sym
        38:              # If our class is not already providing this name, provide it.
        39:              # Commented out: use of resource_name and provides will need to be
        40:              # mutually exclusive in this world, generally.
        41:              # if !Chef::ResourceResolver.includes_handler?(name, self)
        42>>               provides name#, canonical: true
        43:              # end
        44:              @resource_name = name
        45:            else
        46:              @resource_name = nil
        47:            end
        48:          end
        49:          @resource_name
        50:        end
        51:        def self.resource_name=(name)

would need a lot more information such as a repeatable example to be able to solve this.
chef 12.0.0 is definitely not supported by this cookbook, the minimal version we test against is 12.0.3 -- 12.0.0 through 12.0.2 is known to be broken and will not be fixed.


Over branching creation due to specialisation choices

Hello there, I am new to xsimd, so sorry if already answered.
In the implementation I understand that XSIMD_IF_CONSTEXPR macro will allow the compiler to choose for a branching when using suitable compiler.
But often this macro is not used for type sign checking like (in xsimd_sse2.hpp)
`'        // ssub
    template <class A, class T, class = typename std::enable_if<std::is_integral<T>::value, void>::type>
    inline batch<T, A> ssub(batch<T, A> const& self, batch<T, A> const& other, requires_arch<sse2>) noexcept
    {
        if (std::is_signed<T>::value)
        {
            XSIMD_IF_CONSTEXPR(sizeof(T) == 1)
            {
                return _mm_subs_epi8(self, other);
            }
            else XSIMD_IF_CONSTEXPR(sizeof(T) == 2)
            {
                return _mm_subs_epi16(self, other);
            }
            else
            {
                return ssub(self, other, generic {});
            }
        }
        else
        {
            XSIMD_IF_CONSTEXPR(sizeof(T) == 1)
            {
                return _mm_subs_epu8(self, other);
            }
            else XSIMD_IF_CONSTEXPR(sizeof(T) == 2)
            {
                return _mm_subs_epu16(self, other);
            }
            else
            {
                return ssub(self, other, generic {});
            }
        }
    }

``
Here the if (std::is_signed<T>::value) won't use if constrexpr statement, does it mean that a branching will be created and evaluated each time we use ssub

Here the if (std::is_signed::value)`` won't use if constrexpr statement, does
it mean that a branching will be created and evaluated each time we use ssub

Generally not, because constant folding is a basic compiler optimization. We
could still enforce this with the macro you mention... Do you feel like
submitting a patch?

Ok thanks, no development planed on my side.


Incorrect User/Secret

Should provide a meaningful error message. Currently returns:
SyntaxError: Unexpected end of input
at Object.parse (native)
at IncomingMessage. (/Users/davidallen/Projects/ApigeeBundleReaper/src/package/reaper_all.js:223:25)
at emitNone (events.js:72:20)
at IncomingMessage.emit (events.js:166:7)
at endReadableNT (_stream_readable.js:903:12)
at doNTCallback2 (node.js:439:9)
at process._tickCallback (node.js:353:17)

Fixed it. Used the same logic used in apigee-perf-summary
When the http call fails, it throws:
401: Unauthorized with {"host":"api.enterprise.apigee.com","port":443,"path":"/v1/organizations/saisarantest/apis","method":"GET","headers":{"Accept":"application/json","Authorization":"Basic c2Fpc2FyYW4wMDdAZ21haWwuY29tOmZkZnNk"}}


The new apk still has problem.

didn't find class "com.xingress.xglyph.Xglyph" on path base.apk.

Thank you reply, that problem has been solved?

Yes, the problem has been fixed with the new version.

ok.have fun.
2015-08-27 12:54 GMT+08:00 Langs Chiang<EMAIL_ADDRESS>
Yes, the problem has been fixed with the new version.
—
Reply to this email directly or view it on GitHub
https://github.com/xfunforx/IngressXglyph/issues/1#issuecomment-135292092
.


Fix nightly slack notification

I've added a BRANCH_NAME=master environment variable in the nightly build config, so we should now receive slack notifications on failures.
This patch is just about using task_wrapper() on the main node instead of the inner one. This would send notifications for failures during dcos_launch too.

run all integration tests


Message drop down not responsive at change on the profile bar if name not long

Hi
I have been facing some challenges on the message drop down anytime i change the profile bar.
When the name is not long enough or the icon on the top is changed the message bar is no longer responsive
In the first image below you will see that i have changed where the name is supposed to be on the top bar and replaced it with an icon and because of that the message box is no longer responsive

And on the image below i have added the name and icon and counted the number of characters (8) to be placed there aside of the icon and it works perfectly. But the problem with this is that not everybody addition of first name or surname will give 8 characters in space or .. Max John (with space) gives 8 Character. But what about if somebody is called just Kirk or Akolastemanou ?

Please check out a demo of both pages online
Username : justfortest
Password : justfortest
The page that does not work correctly because i did not write any name but replaced it with just an icon
http://usefaith.voiedusucces.net/standard_a.php
When you log in, resize the page to the smallest one and click on message and you will see the drop down problem
The page that works well because i had to calculate the length of what i am adding and add it to the icon
http://usefaith.voiedusucces.net/standard_b.php
This one works well only because i did not remove the name and the name and icon are using some space witch enables then to look responsive.
Please your help will be much appreciated. Thanks

I have found a fix, a work around but i don't know how to paste a code here. When i paste a code it does not appear. I would like to move this to the commits . Is it possible ? and how do i paste the code so that others may benefit from this too ?

To add code check markdown https://guides.github.com/features/mastering-markdown/ click on code pills.
  `
your code here
  `
Just to show you to make it work you need to remove space between   `
your code here

The fix for that is to position the menus as static and add right: 5% and left: auto to their css rules. Needless to say, this must apply only to the appropriate screen widths. I'll push a fix once I get a chance. Probably later today.

I got some issues with the all dropdown, I don't use them for now.
I even got issue with the Bootstrap one, I need to investigate...
But I will need to add one for language switcher asap.

Code


Disable buttons on doing something that makes the UI busy

For #8716
This is the minimum bar for interrupt/restart/export. Prevent the buttons from being run when in the middle of an operation.

Codecov Report

Merging #9500 into master will decrease coverage by 0.02%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master    #9500      +/-   ##
==========================================
- Coverage   60.88%   60.86%   -0.03%
==========================================
  Files         530      530
  Lines       28648    28651       +3
  Branches     4345     4345
==========================================
- Hits        17441    17437       -4
- Misses      10250    10256       +6
- Partials      957      958       +1

Impacted Files
Coverage Δ

...atascience/interactive-window/interactiveWindow.ts
19.51% <16.66%> (+0.34%)
:arrow_up:

src/client/common/utils/platform.ts
64.7% <0%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.36% <0%> (-2.28%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0%> (-0.73%)
:arrow_down:

Continue to review full report at Codecov.

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


chore(kernels, readme): updated readme and dropped kernels lists that are now pushed to the kernels branch

Signed-off-by: Federico Di Pierro<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind documentation

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area crawler

/area ci

/area utils

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: f975d298f2030017f6588de04ee58e8ddc5e8f4a


add minimal support for Warp to work as a terminal

initial support Warp terminal app

@ghouston Still interested in merging this in?

I might close this out around the 25th given we haven't heard back from the author.

Closing this out.


Test mount - not for review

.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Elm input fields use different focus color

see issue #1813
solution was done in pr #2815 (we use a color that fits to our color scheme -> won-secondary-color now, this is probably not used in the focused elm input fields yet -> change needed)
requires #2815

nlr


Expandable notes and todo

Made notes and todo components expandable.

The expanded views need to be finished, gave up for now. Will come back


superset_config.py not being picked up for db upgrade

Make sure these boxes are checked before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if any
[ ] I have reproduced the issue with at least the latest released version of superset
[ ] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
0.25.6
Expected results
Database upgrade script completes
Actual results
It is upgrading the sqlite db and not our postgres db.
we get this error in the upgrade - /opt/superset-venv/local/lib64/python2.7/site-packages/sqlalchemy/dialects/sqlite/base.py:1425: SAWarning: WARNING: SQL-parsed foreign key constraint '(u'datasource_name', u'datasources', u'datasource_name')' could not be located in PRAGMA foreign_keys for table metrics
Steps to reproduce
create a superset_config.py and set the db to a different database other than the sqllite
Run superset db upgrade
you will see - INFO  [alembic.runtime.migration] Context impl SQLiteImpl.

Just make sure superset_config.py in PYTHONPATH environment variable.

Also make sure superset_config is "interpretable". To test that you can fire up a python shell and import superset_config from the same shell you'd be launching the application.

Yes superset runs fine with our config...its when we do the db upgrade this issue occurs

Is it an issue with lacking the proper supporting library like myqlclient for mysql?

I am facing the same issue. Migration is running on sqlite instead of mysql db. PYTHONPATH variable is set to superset_config.py. Using superset 0.25.6 version.

PYTHONPATH environment variable should point to the location of the config file. Don't specify the file name.

On Feb 4, 2019, at 11:07 AM, Kedar Gundlupet<EMAIL_ADDRESS>wrote:
I am facing the same issue. Migration is running on sqlite instead of mysql db. PYTHONPATH variable is set to superset_config.py. Using superset 0.25.6 version.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks @mmuru. My PYTHONPATH environment was set to the correct path. However, I was running the migration as sudo and the environment variables were not getting inherited. Running the migration with sudo -E fixed it.


Feat: undo button

Add deleted habits to a deleted object global variable. Then remove them from habitData and save changes
Update refreshHabitZone() to append all deleted habits to the end of the list
Replace Delete button with Undo button
Undo will add the habits back to habitData and save the changes
But if you don't undo and you refresh, then all data will be lost


Add database versioning

Currently, there isn't an upgrade mechanism in place within the database which makes the current schema rigid and upgradeable. To remedy this, a new bucket should be added to the database (metaBucket) which stores key that house meta-data related to the current/version state of the database.
The createChannelDB method should be modified to create this new bucket+key during initializing. In a similar vein, when opening the database, the dbVersionKey should be read from the metaBucket and compared against some compile-time constant which stores the latest known database version. If the two versions differ (after some future database update/migration), logic should be inserted to convert the old format to the newly defined format. Additionally, it may be advisable to fist make a complete copy of the current database using bolt db's WriteToMethod as a fail-safe in case of bugs or failures during the database migration.

I will grab this one

Closed by #64.


Filtering by types does not work

Describe the bug
I am trying to filter by schools using the GooglePlacesSearchQuery property but it does not have any affect.
Reproduction - (required - issue will be closed without this)

  <GooglePlacesAutocomplete
          placeholder='Search'
          currentLocation={true}
          currentLocationLabel='Current location'
          enablePoweredByContainer={false}
          fetchDetails={true}

          onPress={(data, details = null) => {
            // 'details' is provided when fetchDetails = true
            console.log(data, details);
          }}
          query={{
            key: '****',
            language: 'en',
          }}
          GooglePlacesSearchQuery={{
            type: 'primary_school|school|secondary_school'
          }}
        />

Additional context

Library Version: 2.1.2

React Native Version: 0.63.2

[ X] iOS

[ ] Android

[ ] Web

[ ] I am using expo

Thanks for the bug report. I can confirm that it doesn't work as expected.
PRs welcome!

Thanks for the bug report. I can confirm that it doesn't work as expected.
PRs welcome!

I had the same problem with filtering by type address.
<GooglePlacesAutocomplete
      placeholder="Vyhledat adresu"
      onPress={async (data, details) => {
        ...
      }}
      query={{
        key: "...",
        language: "cs",
        components: "country:cz",
        types: "address",
      }}
    />

Results included cities, companies, etc. instead of just addresses.
I found a typo in the library in GooglePlacesAutocomplete.js line 512(?& in params instead of ?):
`${url}/place/autocomplete/json?&input=` +

should be
`${url}/place/autocomplete/json?input=` +

This fixed my problem.
Will be opening a pull request with a fix.

i was able to get it working with cities by adding the fix @fedor-moiseev recommended and using query={{ types: '(cities)',}} in

I'm having the same issue with

query={{type: "restaurant"}}
any workarounds on this?

Some of these issues may be because of invalid place types, such as "restaurants". According to the Google Autocomplete API, you're limited to a subset of Place 'types' you can refer to: https://developers.google.com/maps/documentation/places/web-service/autocomplete#types


Prevent Panphlan from rejecting all mapped reads when the --th_mismat…

…ches parameter is set.

Many thanks Florian ;)

Hi @leonarDubois ,
Could you release a new version of PanPhlan fixing this bug?
Thanks,
Florian


Cannot read property 'subtle' of undefined

Describe the bug
I cannot run the example code, it throws Error: Cannot read property 'subtle' of undefined
To Reproduce
Just copy the example code in repo, replace with my credential, then run.
Expected behavior
It should login successfully
Environment (please complete the following information):

OS: Linux
Node version: v14.15.0
ts-messenger-api package version: latest

Note: This package works only on NodeJS v15 and higher!

try upgrading your node version


[Bug]:

Expected Behavior
Loading the store.
Actual Behavior
Store fails to load with:
TypeError: Cannot read properties of null (reading 'items')
Possible Solution
In AppHeader.vue, changing:
    const wishlistTotalItems = computed(() => {
      const count = wishlist.value.items;

      return count ? count.length : null;
    });

to
    const wishlistTotalItems = computed(() => {
      const count = wishlist?.value?.items;

      return count ? count.length : null;
    });

Steps to reproduce
Running a fresh clone of the current version of the repo.
What version of plentymarkets integration are you using?
0.1.0
What version of Node.js are you using?
v15.14.0
What browser (and version) are you using?
Firefox
What operating system (and version) are you using?
MacOS 13.2
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks 👍 This issue was solved in this pr: https://github.com/plentymarkets/plentyshop-pwa/pull/155


Rename package in tests

The change in package name from akka.cluster.pi to akkapi.cluster
in a previous PR was not applied for the test correct. This PR corrects
that

Closing - superseded by #154


Click multi commands and Bash completion

Expected Behavior
I followed the example in Click documentation for implementing custom multi commands and then I want to combine this with Bash completion support as described here. When using TAB to complete Bash command line I expect to see a list of commands I can use, i.e.
$ mycommand <press TAB twice>
<list of possible commands>
$ mycommand

Actual Behavior
$ mycommand <press TAB twice>
$ mycommand cli

Environment

Python version: 3.8
Click version: 7.1.1

Still an issue in the new autocomplete system (https://github.com/pallets/click/pull/1622).

This was fixed as part of #1622, specifically 00d5dcc9ede551dc04b20010ba0e65ca4fdbe1ff.


How to run

Hi,
I am interested in your script but completely new to Rust. I managed to clone the repository and ran make install. I wonder how I now can send commands to my HDMI-switch? In which folder do I need to be and do I add any arguments from the cli to do any switching (ie tv beamer to have the tv tuner on input1 and beamer on output1)?
Is there also an option to get the current config?
Thank you!

Used chatGPT. It works


Fix Format documentation

This PR fixes the documentation for Format.Builder.setTileCountHorizontal and Format.Builder.setTileCountVertical.

I think we still want to avoid touching the API checks in the session.legacy package for the same reason outlined here: https://github.com/androidx/media/pull/1549#issuecomment-2233286116

Oh, I forgot about that, sorry. I will revert that change

Done. Now the PR only fixed the documentation for a couple of Format.Builder methods.

I'm going to send this for internal review now. You may see some more commits being added as I make changes in response to review feedback. Please refrain from pushing any more substantive changes as it will complicate the internal review - thanks!


Getting Random Nan for Loss

Hi,
I am getting random nan value for loss and loss_bbox in faster_rcnn_r50_fpn.py. To keep track of whether it is caused by my custom dataset or not, I am using 1 worker, 1 sample size, and printing the input file name. nan value for loss and loss_bbox occurs for random files(for example in first epoch nothing gives nan, then in second epoch 1 file gives nan, in third epoch, another file gives nan and so on.)
Similarly, in retinanet_r50_fpn_1x_coco.py I am getting inf for loss and loss_bbox in random points in the training.
Am I doing something wrong?
My system info:
sys.platform: linux
Python: 3.7.7 (default, May  6 2020, 10:21:04) [GCC 7.3.0]
CUDA available: True
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 10.2, V10.2.89
GPU 0: GeForce GTX 1080 Ti
GCC: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
PyTorch: 1.5.0
PyTorch compiling details: PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) Math Kernel Library Version 2020.0.0 Product Build 20191122 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v0.21.1 (Git Hash 7d2fd500bc78936d1d648ca713b901012f470dbc)
OpenMP 201511 (a.k.a. OpenMP 4.5)
NNPACK is enabled
CPU capability usage: NO AVX
CUDA Runtime 10.2
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_37,code=compute_37
CuDNN 7.6.5
Magma 2.5.2
Build settings: BLAS=MKL, BUILD_TYPE=Release, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -fopenmp -DNDEBUG -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DUSE_INTERNAL_THREADPOOL_IMPL -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, USE_CUDA=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON, USE_STATIC_DISPATCH=OFF,

TorchVision: 0.6.0a0+82fd1c8
OpenCV: 4.2.0
MMCV: 0.5.1
MMDetection: 2.0.0+6603790
MMDetection Compiler: GCC 7.5
MMDetection CUDA Compiler: 10.2

Please use the Error Template. You might need to be more specific about the modification you have done to the codebase and the config, e.g, how many classes in your dataset, what else keys you modified in the config?

@fatihcankurnaz Hi, have you solved the problem? I got a same after updating new version.

Hi, sorry but I could not. Because of the urgency of my project, I rolled back to version 1.0.0rc(which was the version I was using before updating), and it is working fine for me.

Hi, sorry but I could not. Because of the urgency of my project, I rolled back to version 1.0.0rc(which was the version I was using before updating), and it is working fine for me.

So sad, thank you.

Hi @ObKsEm ,
If you meet the problem and do not know how to solve it, please raise a new issue following Error Template.


initialising offset in case of fixed size

#8

@nishiths23 it seems this fix breaks the slider. Describe in more detail what this commit fixes.


Enable customers to compare multiple runs over times

writing of metadata from ML GH actions to logs or writing them to Cosmos DB and allow comparisons of multiple such generated data.

What kind of solution are you envisioning?
Is this not something that should be added to a separate Action? From our point of view the user can write back the output to wherever they want (e.g. GitHub Fastpages).
We could probably provide a template that shows how you can write back information to a private Fastpages page, which can then be used to compare results.

@pulkitaggarwl

How do you envision the feature?
Where and how should the data be stored?


Unseen Hibernation

text: If an "action ends (uncusessfully)" this is ruled as äquivalent to "the action fails" therefore Ruling #1 does not apply here, the source actually states, if an action fails, modifiers whose conditions are satiesfoed, can still be played.
link: https://www.vekn.net/forum/rules-questions/79389-when-does-nra-apply-not-apply?start=6#103563

The ruling has to do with NRA, not with the ability of playing further modifiers or not. AFAIK the ruling on ended actions still stands. Feel free to ask a confirmation/infirmation on the forum.

My bad (cf. ANK 20221205 & ANK 20220118). Fixed now


New package: KrajowaIzbaRozliczeniowaS.A.InstalatorSzafir version 2.6.684

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/114330&drop=dogfoodAlpha

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree company="TPA Polska"
Z poważaniem / Best regards / Mit freundlichen Grüßen
Maciej Wojtan
IT Specialist
[https://mail.tpa-group.pl/pictures/TPA/TPA_logo_signature.png]
TPA Poland
ul. Przyokopowa 33, 01-208 Warszawa
T: +48 22 64 79 700
M: +48 663 664 288
<EMAIL_ADDRESS>http://www.tpa-group.pl | http://www.tpa-group.com
Albaniahttp://www.tpa-group.al | Austriahttp://www.tpa-group.at/en | Bulgariahttp://www.tpa-group.bg/en | Croatiahttp://www.tpa-group.hr/en | Czech Republichttp://www.tpa-group.cz/en | Hungaryhttp://www.tpa-group.hu/en | Montenegrohttps://www.tpa-group.com/en/locations/podgorica/ | Polandhttp://www.tpa-group.pl/en | Romaniahttp://www.tpa-group.ro/en | Serbiahttp://www.tpa-group.rs/en | Slovakiahttp://www.tpa-group.sk/en | Sloveniahttp://www.tpa-group.si/en
Polecamy | We recommend
[https://mail.tpa-group.pl/pictures/TPA/2_Wind.png]https://windenergy-tpa.pl/[https://mail.tpa-group.pl/pictures/TPA/3_TaxChck.png]https://www.tax-checker.com/[https://mail.tpa-group.pl/pictures/TPA/4_Tools.png]https://www.tpa-tools.pl/[https://mail.tpa-group.pl/pictures/TPA/1_Blog.png]https://blog-tpa.pl/
Subscribe to our newsletter PLhttps://www.tpa-group.pl/#newsletter-subskrypcja | ENhttps://www.tpa-group.pl/en/#newsletter-subscription | DEhttps://www.tpa-group.pl/de/#newsletter-abonnieren Visit our blog PLhttp://blog-tpa.pl
Baker Tilly TPA, TPA Poland and Baker Tilly Legal Poland are trade names of TPA Sp. z o.o. Sp.k. and Baker Tilly Gajda Legal Sp.k, respectively, both firms are a member of the global network of Baker Tilly International Ltd., the members of which are separate and independent legal entities.
The data co-controllers of your personal data are: TPA Sp. z o.o. Sp.k., or TPA Management Sp. z o.o., or Baker Tilly Gajda Legal Sp. k., or Baker Tilly TPA Sp. z o.o., depending on which of the above entities you have concluded a contract for the provision of services with or taken steps to conclude a contract. For more information on the processing of your personal data, please read the Privacy policy PLhttps://www.tpa-group.pl/polityka-prywatnosci/#Klauzula | ENhttps://www.tpa-group.pl/en/privacy-policy/#Clause | DEhttps://www.tpa-group.pl/de/privacy-policy/#Klausel
[https://mail.tpa-group.pl/pictures/TPA/logo_BTEA_EN_150px.png]
From: microsoft-github-policy-service[bot] @.>
Sent: Wednesday, August 2, 2023 1:50 PM
To: microsoft/winget-pkgs @.>
Cc: Maciej Wojtan @.>; Mention @.>
Subject: Re: [microsoft/winget-pkgs] New package: KrajowaIzbaRozliczeniowaS.A.InstalatorSzafir version 2.6.684 (PR #114330)
@MWojtan-TPAhttps://github.com/MWojtan-TPA please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]
Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"
Contributor License Agreement
Contribution License Agreement
This Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
and conveys certain license rights to Microsoft Corporation and its affiliates (“Microsoft”) for Your
contributions to Microsoft open source projects. This Agreement is effective as of the latest signature
date below.

Definitions.
“Code” means the computer software code, whether in human-readable or machine-executable form,
that is delivered by You to Microsoft under this Agreement.
“Project” means any of the projects owned or managed by Microsoft and offered under a license
approved by the Open Source Initiative (www.opensource.orghttp://www.opensource.org).
“Submit” is the act of uploading, submitting, transmitting, or distributing code or other content to any
Project, including but not limited to communication on electronic mailing lists, source code control
systems, and issue tracking systems that are managed by, or on behalf of, the Project for the purpose of
discussing and improving that Project, but excluding communication that is conspicuously marked or
otherwise designated in writing by You as “Not a Submission.”
“Submission” means the Code and any other copyrightable material Submitted by You, including any
associated comments and documentation.
Your Submission. You must agree to the terms of this Agreement before making a Submission to any
Project. This Agreement covers any and all Submissions that You, now or in the future (except as
described in Section 4 below), Submit to any Project.
Originality of Work. You represent that each of Your Submissions is entirely Your original work.
Should You wish to Submit materials that are not Your original work, You may Submit them separately
to the Project if You (a) retain all copyright and license information that was in the materials as You
received them, (b) in the description accompanying Your Submission, include the phrase “Submission
containing materials of a third party:” followed by the names of the third party and any licenses or other
restrictions of which You are aware, and (c) follow any other instructions in the Project’s written
guidelines concerning Submissions.
Your Employer. References to “employer” in this Agreement include Your employer or anyone else
for whom You are acting in making Your Submission, e.g. as a contractor, vendor, or agent. If Your
Submission is made in the course of Your work for an employer or Your employer has intellectual
property rights in Your Submission by contract or applicable law, You must secure permission from Your
employer to make the Submission before signing this Agreement. In that case, the term “You” in this
Agreement will refer to You and the employer collectively. If You change employers in the future and
desire to Submit additional Submissions for the new employer, then You agree to sign a new Agreement
and secure permission from the new employer before Submitting those Submissions.
Licenses.

Copyright License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license in the
Submission to reproduce, prepare derivative works of, publicly display, publicly perform, and distribute
the Submission and such derivative works, and to sublicense any or all of the foregoing rights to third
parties.
Patent License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license under
Your patent claims that are necessarily infringed by the Submission or the combination of the
Submission with the Project to which it was Submitted to make, have made, use, offer to sell, sell and
import or otherwise dispose of the Submission alone or with the Project.
Other Rights Reserved. Each party reserves all rights not expressly granted in this Agreement.
No additional licenses or rights whatsoever (including, without limitation, any implied licenses) are
granted by implication, exhaustion, estoppel or otherwise.

Representations and Warranties. You represent that You are legally entitled to grant the above
licenses. You represent that each of Your Submissions is entirely Your original work (except as You may
have disclosed under Section 3). You represent that You have secured permission from Your employer to
make the Submission in cases where Your Submission is made in the course of Your work for Your
employer or Your employer has intellectual property rights in Your Submission by contract or applicable
law. If You are signing this Agreement on behalf of Your employer, You represent and warrant that You
have the necessary authority to bind the listed employer to the obligations contained in this Agreement.
You are not expected to provide support for Your Submission, unless You choose to do so. UNLESS
REQUIRED BY APPLICABLE LAW OR AGREED TO IN WRITING, AND EXCEPT FOR THE WARRANTIES
EXPRESSLY STATED IN SECTIONS 3, 4, AND 6, THE SUBMISSION PROVIDED UNDER THIS AGREEMENT IS
PROVIDED WITHOUT WARRANTY OF ANY KIND, INCLUDING, BUT NOT LIMITED TO, ANY WARRANTY OF
NONINFRINGEMENT, MERCHANTABILITY, OR FITNESS FOR A PARTICULAR PURPOSE.
Notice to Microsoft. You agree to notify Microsoft in writing of any facts or circumstances of which
You later become aware that would make Your representations in this Agreement inaccurate in any
respect.
Information about Submissions. You agree that contributions to Projects and information about
contributions may be maintained indefinitely and disclosed publicly, including Your name and other
information that You submit with Your Submission.
Governing Law/Jurisdiction. This Agreement is governed by the laws of the State of Washington, and
the parties consent to exclusive jurisdiction and venue in the federal courts sitting in King County,
Washington, unless no federal subject matter jurisdiction exists, in which case the parties consent to
exclusive jurisdiction and venue in the Superior Court of King County, Washington. The parties waive all
defenses of lack of personal jurisdiction and forum non-conveniens.
Entire Agreement/Assignment. This Agreement is the entire agreement between the parties, and
supersedes any and all prior agreements, understandings or communications, written or oral, between
the parties relating to the subject matter hereof. This Agreement may be assigned by Microsoft.

—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/winget-pkgs/pull/114330#issuecomment-1662067363, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BBVRXQSUTAF27RC26JLBHPTXTI5HPANCNFSM6AAAAAA3BEWXIA.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


orientdb 3.0.0: Exception "Index with type UNIQUE and algorithm SBTREE does not exist." when creating unique index

OrientDB Version: 3.0.0
Java Version:  1.8.0_144
OS: openSUSE 42.3 (x86_64)
Expected behavior
We 're upgrading to OrientDB 3.0.0.
The following works in 2.2.16 and all previous version:
// OClass object = createClass();
object.createIndex("NameAndTypeIndex", OClass.INDEX_TYPE.UNIQUE,
"JOB_NAME", "JOB_TYPE");
Actual behavior
com.orientechnologies.orient.core.index.OIndexException: Index with type UNIQUE and algorithm SBTREE does not exist.
Steps to reproduce
Storage: plocal

Create a class with two properties

Create UNIQUE index on these two properties using
OClass.createIndex(String iName, INDEX_TYPE iType, String... fields);

Hi @BZoladek ,
I am trying to reproduce it with:
`  OrientDB orientDB = new OrientDB("memory:", OrientDBConfig.defaultConfig());
orientDB.create("testIndex", ODatabaseType.MEMORY);
ODatabaseDocument db = orientDB.open("testIndex", "admin", "admin");
OClass userClass = db.createVertexClass("User");
userClass.createProperty("MailAddress", OType.STRING);
userClass.createProperty("SomeProp", OType.STRING);
userClass.createIndex("testIndex", OClass.INDEX_TYPE.UNIQUE, "MailAddress", "SomeProp");
OVertex doc = db.newVertex(userClass);
doc.setProperty("MailAddress", "foo@bar.com");
doc.setProperty("SomeProp", "BlaBla");
db.save(doc);

db.close();
orientDB.drop("testIndex");
orientDB.close();`

but with no success. Can you provide code spinet which cause exception?

Hi,
It should be UNIQUE_HASH_INDEX. Sorry for the confusion. I'll try to build
an example code for you.
Thanks,
Barbara.
On Wed, Sep 12, 2018 at 5:10 AM Marko<EMAIL_ADDRESS>wrote:

Hi @BZoladek https://github.com/BZoladek ,
I am trying to reproduce it with:
` OrientDB orientDB = new OrientDB("memory:",
OrientDBConfig.defaultConfig());
orientDB.create("testIndex", ODatabaseType.MEMORY);
ODatabaseDocument db = orientDB.open("testIndex", "admin", "admin");
OClass userClass = db.createVertexClass("User");
userClass.createProperty("MailAddress", OType.STRING);
userClass.createProperty("SomeProp", OType.STRING);
userClass.createIndex("testIndex", OClass.INDEX_TYPE.UNIQUE,
"MailAddress", "SomeProp");
OVertex doc = db.newVertex(userClass);
doc.setProperty("MailAddress", "foo@bar.com");
doc.setProperty("SomeProp", "BlaBla");
db.save(doc);
db.close();
orientDB.drop("testIndex");
orientDB.close();`
but with no success. Can you provide code spinet which cause exception?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/orientechnologies/orientdb/issues/8473#issuecomment-420572146,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXrhYuLkPNG7Nc98tT_HLB63QrAMWZacks5uaM98gaJpZM4WCF2K
.

--
Barbara Zoladek
Software Developer
*Business Protection You Can Count On *
104 Schneider Road | Kanata, Ontario K2K 1Y2
https://www.google.ca/maps/place/104+Schneider+Rd,+Kanata,+ON+K2K+1Y2/@45.340043,-75.9036994,17z/data=!3m1!4b1!4m2!3m1!1s0x4cd1fffa70d4d3f3:0xddf444ae3573dcc5
tel<PHONE_NUMBER> | www.supernaeyeglass.com
https://www.facebook.com/SupernaNET https://twitter.com/SupernaNet
https://www.linkedin.com/company/superna-net
--
CONFIDENTIALITY NOTICE: The information contained in this email is
privileged and confidential and intended only for the use of the individual
or entity to whom it is addressed.   If you receive this message in error,
please notify the sender immediately at<PHONE_NUMBER> and destroy the
original message and all copies. Thank you.

Hi,
This issue can be closed. It turned out to be the build problem, not OrientDB problem.
Thanks,
Barbara.


snap: access to dotfiles

The strict confinement policy blocks access to the .bash_history and .zsh_history file in the $HOME directory. The home interface (plugs subsection) also does not help because it only guarantees access to non-hidden files.

@fszymanski Thank you for your suggestions, contributions, PRs and your continued support of the project - I really appreciate it!
I guess that you are Snap expert. Therefore I would like to use this opportunity to ask you for help.
As you may noticed, I consolidated sources and distribution names towards the name HSTR, because hh/hstr ambiguity caused problems e.g. project have to solve this before being accepted by major distribution(s) maintainers (BTW I agree with making the name consistent).
Therefore I would like to have snap registered for hstr, but I'm not able to fix snapcraft.yaml configuration to make it build. You also sent PR https://github.com/dvorka/hstr/pull/314 which probably fixes snap build, but returns back to hh name (this is why I didn't merged it yet). I have registered both hstr and hh on snapcraft.io, but I would like to go with hstr. The problem is that I'm not able to convince snapcraft to pick hstr (and ignore hh) in bin/ directory (while building). The problem might be, that automake-based build creates hstr in bin/ directory + symlink hh to hstr. Does it make sense? Can I choose which binary to pick? Can I delete symlink as a part of snapcraft build?
Thank you in advance for any suggestion!

I guess that you are Snap expert. Therefore I would like to use this opportunity to ask you for help.

I'm hardly an expert when it comes to the snaps, but I will try to help as much as I can.

The problem is that I'm not able to convince snapcraft to pick hstr (and ignore hh) in bin/ directory (while building). The tool keeps to pick hh regardles there is no hh in snapcraft.yaml. Perhaps the problem might be that automake-based build creates hstr in bin/ directory + symlink hh to hstr. Does it make sense? Can I choose which binary to pick? Can I delete symlink as a part of snapcraft build?

You can exclude the hh binary from the snap in the parts/hstr/prime subsection:
parts:
  hstr:
    ...
    prime:
      - -bin/hh

Note: Notice the minus sign in front of the bin/hh
The snapcraft.yaml file would look like this.
You can also have the hstr and hh binary file in the snap. In this case the apps subsection would look like this:
apps:
  hstr:
    command: bin/hstr
  hh:
    command: bin/hh

Note: Now you have access to two executables hstr and an alias hstr.hh (and the prime subsection is missing)
The snapcraft.yaml file would look like this.
In both cases the hstr is the primary executable.

@fszymanski These are great suggestions - thank you!


Create service client internally based on passed hostPort option

client.NewClient, client.NewDomainClient, and worker.New doesn't accept service wrapper anymore.
client.NewClient, client.NewDomainClient accept ClientOption (not *ClientOption) to be consistent with worker.New and simplify code.
client.NewClient, client.NewDomainClient, and worker.New returns 2 values: object and error which can be not nil in case of gRPC error.
HostPort is added to both ClientOptions and WorkerOptions. It is used to connect to Temporal cluster (localhost:7322) by default.
GRPCDialer is added to both ClientOptions and WorkerOptions. It allows to override connection mehanizm. Use TLS, custom interceptors or share single connection between worker and client (connection needs to be stored somewhere in customer code).
CloseConnection is added to Client, DomainClient, and Worker and needs to be called to close underlying connection. goleaks in integration tests catches leaked connections also.

Do you need to do something similar for the Worker?

Do you need to do something similar for the Worker?

I did.


14 25 sysfs permissions

Fixes #14 and #25 by adding file modes which better match sysfs file permissions
Fixes subsequent permission error when export is successful but sysfs tree buildout is not complete before attempting to use the new value FH. Slower systems may need a longer delay. Looping with os.path.exists() would likely be a better fix.

Okay I have added the filemode patches to #31 and https://github.com/vitiral/gpio/tree/python-2.7
In lieu of sleep or waiting on a specific gid, I think we should - as mentioned above - just retry with a delay, and fail after N retries. I think that's likely to work reasonably in all cases and avoid edge cases and regressions.


Tester potentially cutting spaces?

13 | echo "aspas -> ' "
mini output = (aspas -> ')
bash output = (aspas -> ' )
15 | echo 'aspas -> " '
mini output = (aspas -> ")
bash output = (aspas -> " )

fixed in a1a0340d2ae9efc90e7425ee4a5cee8cec7db453


Dependencies are not being installed

When I try to install this plugin, I get the following error message:
Plugin could not be installed due to error message "Could not install plugin, got 2 failure(s).
Required plugin/package "pragmarx/google2fa ^8.0" is missing or not installed and activated
Required plugin/package "endroid/qr-code ^3.9" is missing or not installed and activated"

Maybe it is simply something I am missing out in the installation process? If so, I would be happy to contribute and update the README for future users.
EDIT: I am installing it locally using the repo

Either run composer install in the plugin directory, or use a packages release from the releases (and not the 'download zip' feature of GitHub)
Releases: https://github.com/runelaenen/shopware6-two-factor-auth/releases

Either run composer install in the plugin directory, or use a packages release from the releases (and not the 'download zip' feature of GitHub)
Releases: https://github.com/runelaenen/shopware6-two-factor-auth/releases

Thank you for the answer! I was unaware that the vendor folder has to be distributed with the plugin to make it work out of the box.

Thank you for the answer! I was unaware that the vendor folder has to be distributed with the plugin to make it work out of the box.


Challenge creation fails if no parent identifier is provided

The challenge creation fails if no parent identifier is provided in the request JSON.
Request JSON:
{
"name": "Misnamed Double Digitized Ways",
"identifier": "misnameddoubledigitizedways",
"difficulty": 2,
"description": "",
"blurb": "Double digitized ways with inconsistent naming",
"instruction": "Fix the inconsistent naming of what seems to be a single double-digitized road.",
"help": ""
}
Response JSON:
{"status":"KO","message":{"obj.parent":[{"msg":["error.path.missing"],"args":[],responseCode:400}],responseCode:400},responseCode:400}
Expected behaviour:
The challenge is created into the default project (as described in the API).

Fixed per PR #132


frontend: Adds JSON / object / KubeObjectInterface options

Linked to PR: https://github.com/headlamp-k8s/headlamp/pull/1727
Description
This PR enhances the singleApiFactory function in apiProxy.ts to improve its flexibility. The post function within singleApiFactory has been updated to accept parameters of different types: JSON, object, or KubeObjectInterface. This change ensures that the function can handle various data formats more efficiently, making it more versatile and robust.
Changes

[x] Updated the post function in singleApiFactory to accept parameters of type JSON, object, or KubeObjectInterface.

object type is almost like any. It will accept anything that's object (for example a function is an object). so this change essentially removes type safety from this function. I'm not sure what was the context for this change but this is probably not the way to do it.

This was my suggestion, after verifying that we had KubeObjectInterface for this post function, but not for the one it ends up calling, which allows JSON | object | KubeObjectInterface . The alternative would be to make many fields in KubeObjectInterface optional, but that's not reflective of a k8s resource, only of the user-sent data.
I think the current PR is fine.

Hmm, I merged it but perhaps I should have waited for your response back, @sniok . Please feel free to propose an alternative that solves both concerns, on top of what we have now.


An error is thrown if plugins is not defined in .graphqlrc.yml file

With 0.14.0-dev2, when plugins key is not defined in the configuration file, I am seeing an error
TypeError: Cannot convert undefined or null to object
    at Function.keys (<anonymous>)

I believe it is coming from here https://github.com/aerogear/graphback/blob/8fa8dcdebab081313fa9daf2fa21ea52d3311efe/packages/graphback/src/loadPlugins.ts#L6 since the pluginConfig is undefined.

https://github.com/aerogear/graphback/issues/1068
Yeah I have seen this before. To be completely honest I don't remember the exact reason that I closed it.
Why do you not have any plugin define? Isn't SchemaCRUDPlugin needed?

#1068
Yeah I have seen this before. To be completely honest I don't remember the exact reason that I closed it.

Oh the issue is closed that's why it did not show up when I did a quick search.
So for me was when I did not have plugins setup that I was seeing this error.

Why do you not have any plugin define? Isn't SchemaCRUDPlugin needed?

I thought with stuffs being done at runtime, I could try to remove some plugins graphback-schema being one of them hence the error. Probably a user mistake and if so, we should at least have a better error message.

Runtime still uses plugins to transform schema:
https://github.com/aerogear/graphback/blob/83def270c729aa2e003996961640439742280495/packages/graphback/src/GraphbackRuntime.ts#L22

Probably a user mistake and if so, we should at least have a better error message.

Absolutely, we are going to move to a code-based plugin approach anyway (see https://github.com/aerogear/graphback/issues/1283) so this can be resolved there :)

So templates in graphback have examples. We still need plugins there. We just do not need to use cli to generate code anymore (unless client side queries)

Yep. Same here. Quick fix needed.

Where are you seeing this?

https://github.com/aerogear/graphback/pull/1567#discussion_r445446494

But since it is probably a quick fix, if you are in that area feel free to action it :D

I'll create a separate PR


Making the demo slow without lag

The demo should be slow and the current js hack is making the keyboard input laggy, also tried using Bret Johnson's SLOWDOWN utility with 100 cycles i.e.
Dos(canvas, { cycles: 100, onprogress: ()=>{} }).ready((fs, main) =>
    fs.extract("snake.zip").then(() =>
        main(["-c", "SLOWDOWN /Q/S:0/B:N snake"]).then(ci =>
            swipedetect(swipedir => swipedir && ci.simulateKeyPress(36 + swipedir))
        )
    )
);

But there is a speed bump whenever a key is entered on the keyboard

253ead0 kinda solves it


Update feed to load ports in parallel

This isn't perfect, there are still some errors being thrown in the console from ports that couldn't be loaded. But the feed is no longer blocked.

Fixes #8

It works really well! thank you


Mac compatibility

I cannot install on mac because of nsxiv.

Sorry, I can installed but I can use soil command.

Hi,
If you read the README you can find this:

Caveats
This plugin has been developed on and for Linux following open source philosophy

My entire repo is meant to be used on Linux. If you want to use it on Mac then fork it and make the appropriate modifications.
Anyway, to use other image viewer, change the setup:
require'soil'.setup{
    image = {
        execute_to_open = function(img)
            return "nsxiv -b " .. img
        end
    }
}

I don't own a Mac to test this plugin but I'd bet that the :Soil command would work.
If plantuml is installed in your Mac I don't see why it wouldn't work.
Otherwise you can test it downloading the plantuml.jar from Official PlantUML and set it up (also check your JVM compatibility):
require'soil'.setup{
    -- If you want to use Plant UML jar version instead of the install version
    puml_jar = "/path/to/plantuml.jar",
}

Have a nice day!

Unfortunately changing the image configuration still seems to trigger a check for nsxiv from the :Soil command.
The lazy (AstroNvim 4) config I used was:
{
    "javiorfo/nvim-soil",
    dependencies = {
      "AstroNvim/astrocore",
      -- Optional for puml syntax highlighting:
      "javiorfo/nvim-nyctophilia",
      -- lazy = true,
      filetype = "*.pu",
      opts = {
        options = {
          g = {
            puml_jar = "/Users/username/.local/share/plantuml/current.jar",
            image = {
              darkmode = true, -- Enable or disable darkmode
              format = "png", -- Choose between png or svg

              -- This is a default implementation of using nsxiv to open the resultant image
              -- Edit the string to use your preferred app to open the image
              -- Some examples:
              -- return "feh " .. img
              -- return "xdg-open " .. img
              -- execute_to_open = function(img) return "nsxiv -b " .. img end,
              execute_to_open = function(img) return "open " .. img end,
            },
          },
        },
      },
    },
  }

Perhaps there is a check for nsxiv regardless of which command is configured to open the file.

Unfortunately changing the image configuration still seems to trigger a check for nsxiv from the :Soil command.
The lazy (AstroNvim 4) config I used was:
{
    "javiorfo/nvim-soil",
    dependencies = {
      "AstroNvim/astrocore",
      -- Optional for puml syntax highlighting:
      "javiorfo/nvim-nyctophilia",
      -- lazy = true,
      filetype = "*.pu",
      opts = {
        options = {
          g = {
            puml_jar = "/Users/username/.local/share/plantuml/current.jar",
            image = {
              darkmode = true, -- Enable or disable darkmode
              format = "png", -- Choose between png or svg

              -- This is a default implementation of using nsxiv to open the resultant image
              -- Edit the string to use your preferred app to open the image
              -- Some examples:
              -- return "feh " .. img
              -- return "xdg-open " .. img
              -- execute_to_open = function(img) return "nsxiv -b " .. img end,
              execute_to_open = function(img) return ":exec !open " .. img end,
            },
          },
        },
      },
    },
  }

There is a check for nsxiv regardless of which command is configured to open the file. https://github.com/javiorfo/nvim-soil/blob/master/lua/soil/core.lua#L18-L21
Creating a mock nsxiv binary using a shell script with !/bin/sh, and making that file executable, e.g. chmod a+x nsxiv, will trick soil into thinking the nsxiv binary is installed and the image is generated.
Then a suitable command is required to open that image using an image viewer from the MacOSX.

Hello,
I will remove the nsxiv validation.
Thanks for the info!


[PG13] Update CI's for PG13 branch

Changed trigger branch to PG13.
Updated installcheck.yaml for PG13.

@MuhammadTahaNaveed Thank you!


Allow MXL format

Hi,
your library is nice but it was missing the MXL download option. This is to download the music sheet, which can then be imported into a music sheet software.
I'd also appreciate the PDF format, but I couldn't figure out the download URL. If you know how it's built, would you mind adding it to the available formats?
I also ran Black on the code, sorry for the unnecessary high lines count.

Interesting. I never really considered that the MusicXML file has the same URL as the mp3 score file, just that it has a .mxl extension. I wrote the musdl tool by analysing the HTML source of a Musescore webpage, and the MusicXML file link did not appear, so it was not an option initially. Thanks for the commit, anyways!

I will now upload the updated script to PyPI.


feat(acir_gen): Use predicates with MemoryOps

Description
Problem*
Resolves #2133
Summary*
The ACVM was missing a predicate as part of MemoryOps, thus we were pushing MemoryOps in ACIR gen without their predicate. This caused memory operations under dynamic predicates to be executed no matter what. This is a bug as we can error out with an index out of bounds for valid execution. The linked has a full example of this and I have expanded the dynamic_index_failure test:
fn main(mut x: [u32; 5], z: Field) {
    let idx = z + 10;

    if z == 20 {
        x[0] = x[4];
    } else {
        // We should not hit out of bounds here as we have a predicate
        // that should not be hit
        if idx as u32 < 3 {
            x[idx] = 10;
        }
    }
    // Error should occur here as an index out of bounds read occurs
    assert(x[idx] != 0);
}

Patches:
Backend branch: https://github.com/noir-lang/acvm-backend-barretenberg/pull/247
ACVM: https://github.com/noir-lang/noir/pull/2400
Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[X] I have tested the changes locally.
[X] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Waiting to review this until the other PR is merged

This PR was originally created to avoid the execution error in ACVM with mem ops and predicates. This should be superseded by https://github.com/noir-lang/noir/issues/2493. As per https://github.com/noir-lang/noir/issues/2493#issuecomment-1701326747 we should be able to then remove the inclusion of a predicate on the MemoryOp ACIR opcode as well.

Superseded by https://github.com/noir-lang/noir/pull/2553


Implement a language parser.

It only needs to be simple.

So all that needs to be done in this issue is a framework for commands.
Specific commands should be in following commits whenever a feature is needed.

The language parser should be called by the connection handler if set to PROCESSING in the FSM. The parser should directly connect into the underlying cache layer and perform operations tehre. There needn't be any commands directly relating to the bloom filter or any underlying components. Only commands to the DHT and the cache should exist.