[mgmt] hdinsightcontainers release

https://github.com/Azure/sdk-release-request/issues/5410

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/arm-hdinsightcontainers


Can not view application

I follow the instructions, starting the back end with bootRun, and using npm to install and start the dev server. I get directed to http://localhost:8082/, and the browser displays Cannot GET /

execute init.sql

Execute it where? Can the instruction specify this? What DB engine does this use?

https://github.com/myliang/fish-admin/tree/master/api/db

Yes, I see where the file is. However, it's not clear to me where to execute it, and it's not mentioned in your Readme's quickstart guide. See previous comment.

db: postgresql
create database fish;

https://github.com/myliang/fish-admin/blob/master/api/src/main/resources/application.properties


Prevent AnonymousKernelCommands from ending up inside serialized KernelEvents

AnonymousKernelCommands are not supposed to be serialized across process boundaries and should therefore not be included as part of KernelEvents.

anonymous command should not leave the current host and should be handled locally even by proxy kernel

@colombod Agreed. This is part of what Jon and I discussed, and I have included the larger problem in this follow up issue - #2884 which I plan to look into soon.
This PR addresses one known case where AnonymousKernelCommand ends up leaving the current host today (see the fix in SubmissionParser.cs line 127 in the current PR). The plan that @jonsequitur and I discussed was to merge this fix now and tackle the larger issue in subsequent PR. I was essentially waiting for him to take a second look and approve the PR today.
Note that I was planning to merge this change today since this is one of the issues that came up when testing the VS side integration. So would appreciate if you could remove the block on the PR :)

If @jonsequitur approves this should auto merge.

Let's find some time to discuss how we want to do this while removing rather than adding code. There's too much complexity here.


Website forces https! Result: lost data

Until today the website of remix forces me to use https. All my contracts are stored in the browser using the http version. Therefore my remix ide looks like a fresh installation. All my code is gone (yes, I have backups).
Even if I switch back to http manually it redirects instantly back to https.
Is there any way to get the data back I have on the http version?

fixed


pypy 5.3 is released

http://morepypy.blogspot.com/2016/06/pypy2-v53-released-major-c-extension.html

Its interesting that the post you linked does not list PPC arch as being supported by this release, but they have download links (which 404) and checksums for those on the downloads page.

Still working on how to support the new s390x architecture. I think I have most of it figured out, but /etc/issue on RHEL 7 just contains garbage:
\S
Kernel \r on an \m

Actual release info is in /etc/redhat-release:
Red Hat Enterprise Linux Server release 7.2 (Maipo)

Isn't there any s390x user...? 😜   Please help us~

Here's what I had found:
https://twitter.com/mattgillard/status/481387591099375616
zlinux proc]$ uname -a
Linux zlinux 2.6.18-371.el5 #1 SMP Thu Sep 5 21:23:53 EDT 2013 s390x s390x s390x GNU/Linux

so, you can probably just check if uname -m is s390x...


Nvidia Apex Installation

I tried installing NVIDIA Apex (https://github.com/NVIDIA/apex) with uv (version 0.4.8) using
"uv pip install -v --no-cache-dir --no-build-isolation --config-settings "--build-option=--cpp_ext" --config-settings "--build-option=--cuda_ext" ./".
However, this does not work: the error i am getting is 'error: unexpected argument '--build-option' found'.
Is there a way around this problem?
Thanks in advance.

Can you share the full output?

uv pip install -v --disable-pip-version-check --no-cache-dir --no-build-isolation --config-setting "--build-option=--c
pp_ext" --config-setting "--build-option=--cuda_ext" git+https://github.com/NVIDIA/apex
error: unexpected argument '--build-option' found

  tip: a similar argument exists: '--build-isolation'

Usage: uv pip install --verbose... --no-cache <PACKAGE|--requirement <REQUIREMENT>|--editable <EDITABLE>> <--disable-pip-version-check|--user>

For more information, try '--help'.

As a quick fix, you can fork apex and set these args in the setup.py (super hacky fork you enough to give you an idea: https://github.com/NVIDIA/apex/compare/master...BitPhinix:apex:master

When you provide --config-setting "--build-option=--cpp_ext" I think your shell will remove the quotes and we will parse it as a flag?
In my shell, the CLI doesn't complain if I do something like this uv pip install anyio --config-setting '"--build-option=--cpp_ext"'


onDateSet retuns (month - 1)

the month value that is returned is always -1 from what i would expect.
e.g
I pick 01.09.2015
i get: 1.8.2015

This is by design. See the newly added FAQ in the documentation (or the previous github issues on this subject) for more information.


Java 9 ea167 missing java.lang.reflect.Module and Layer

Looks like Java 9 ea167 is missing some new Java 9 classes like java.lang.reflect.Module and java.lang.reflect.Layer.
When I upgraded, my code that uses those classes no longer compiled. Setting the JDK in IntelliJ also shows those classes are missing.
Downgrading  to Java 9 ea163 makes the problem go away.
Please report to Oracle if the problem is actually with their distribution of Java 9EA167.

We don't really have any contact with Oracle, we merely distribute the binaries on their site.
Perhaps you could try feeding this issue back to them yourself on their JDK 9 forums?

I thought you might want to remove this version from the  repository as it is obviously broken... and I have tried to report the bug but they don't make it easy to do it at all...
I also noticed that the version printed with java- version is not the one reported by SDKMAN.
So, I am not sure I should file the bug with Oracle instead of SDKMAN:
sdk use java 9ea163

Using java version 9ea163 in this shell.
➜  test-modules git:(master) ✗ java -version
java version "9-ea"
Java(TM) SE Runtime Environment (build 9-ea+157)
Java HotSpot(TM) 64-Bit Server VM (build 9-ea+157, mixed mode)

Oh, my previous comment obviously refers to the previous java version in SDKMAN, 9ea163 (which reports itself to be 9ea157). The latest one, 9ea167 reports the expected version.

Very odd, I'm not really seeing that on my side:
$ sdk u java 9ea163
$ java -version
java version "9-ea"
Java(TM) SE Runtime Environment (build 9-ea+163)
Java HotSpot(TM) 64-Bit Server VM (build 9-ea+163, mixed mode)

Could there be something else wrong here?


Larva test hangs for several minutes on the write of an [error]

Describe the bug
A larva test scenario will hang for a few minutes on a write action of [error]
Reporter
Wouter van der Schoot
Category
Use following Tags to categorize your bug report

Larva
Performance

To Reproduce
Steps to reproduce the behavior:

Go to 'https://vcs.aws.insim.biz/ibissource/si/ibis4adn'
Clone all ibis4adn related projects
The test scenario in question is in ibis4adn_outbound - OutboundTrigger\scenario02-ERROR - OutboundTrigger ERROR, PRC_TYDST is not updated in OUTBOUNDTRIGGER table
Run this test scenario
The scenario will hang for several minutes on step7.sender.SendMFTMessage_Solera.write=common/MFTResponse_Error.txt
Eventually the scenario will complete, but step7 takes way to long

Expected behavior
I expect that each step would take a few seconds
Environment:

Frankrunner
Browser chrome
IAF Version 7.6-20201123.110830

Additional context
Add any other context about the problem here. (f.e. ladybug report / test adapter with larva test)
In the same repo (ibis4adn_outbound) is the following test scenario:
ProcessOutbound_PMB_PPR\scenario07-ErrorMFT
This scenario does exactly the same as the first scenario (with the same adapters) with the following difference:
The first scenario starts the adapter from another ibislocalsender, in other words there are more nested adapters..
The ProcessOutbound scenario does NOT hang whereas the ProcessOutboundTrigger scenario does.
Pipeline ProcessOutboundTrigger_DUURTTELANG.zip
Pipeline ProcessOutbound_GAATWELGOED.zip

Could you create a simple/anonymous example that we can simply copy into a random Ibis to lower the threshold to analyze this issue?
Maybe this example can afterwards be added to Ibis4TestIAF as a regression test.

My guess is you are waiting for a timeout multiple times
One thing I used to do is change the timeout setting of the testtool, but then the testtool was still a separate project
looked in the code and this seems to be the place,
/larva/src/main/java/nl/nn/adapterframework/testtool/TestTool.java
protected static final int DEFAULT_TIMEOUT = 30000;
adjust to
protected static final int DEFAULT_TIMEOUT = 5000;
After discussing with the core team we decided that there must be something wrong and changing the timeout is just a workaround. In case of an error,  this error should be handled and not wait 30 seconds for the timeout.
So the adjustment should be that when we use a stubbed sender we should not use a timeout and in places where we do not use the stubbing (by default in databasessenders , localsenders and more) the timout is still valid.

The attached file contains a "simple/anonymous example" for this issue
NestedError.zip

unpack the zip file
Move the folder NestedErrorConfig to your local frank-runner\examples\Frank2Example1\configurations folder
Move the folder NestedErrorTest to your local frank-runner\examples\Frank2Example1\tests\Example1
Start the frank-runner (without setting project.dir)
Go to http://localhost/
Open Testing/Larva
Run the NestedErrorTest\Scenario01-ERROR
Wait :P


Wrong transaction fees calculation

Machine Windows 7
Scenario
Use coin control and select 10 different addresses (it may not need 10 addresses exactly).
The wallet will display the amount you can send by subtracting the transaction fees.
Use this amount and send the money to a valid address.
Expected behaviour
The transaction is finalized
Current behaviour
The wallet say that there is not enough money for the transaction, 0.001 XPY is missing.

That's because the fee is an estimate you are looking at prior to the actual transaction creation is an estimate. That's not to say that's not insanely obnoxious, I'm not sure about a fix off the top of my head.

How the wallet get the real amount of the fees allowing to display that some fees are missing when trying to send the coins?

Cause it's actually creating the transaction and calculating the full fees during creation.
It returns the error saying the fees are missing if transaction creation fails.

This is a duplicate of #344 and as stated there this is just an estimate which by definition is not guaranteed to be accurate. It's doing the best it can to calculate the fee but without actually creating the transaction and performing the calculation you can't expect it to be correct.


Add my college LTTC

Hi! I want to add my college to the list of Universities/Educational institutions
This is official Ly Tu Trong College website URL
https://lttc.edu.vn/
Thank a lot

@dotkhanh I am sorry, the domain lttc.edu.vn is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Add new autoscale tests

This PR adds an autoscaling test for the collector.  For the moment it can only be run on OpenShift.  Future changes will add a test for the ingester, and make it possible to run the tests on Kubernetes.

@objectiser @jpkrohling @rubenvp8510 @pavolloffay  Please review.  I'm not sure how this decreases test coverage as I only added a test.  Maybe I need to rebase?

@objectiser @jpkrohling @rubenvp8510 @pavolloffay I rebased, but codecov is still failing.  Is there something I can do to fix that, or can one of you just merge this?

@jpkrohling Any thoughts on the codecov failure? Otherwise is it ok to merge?

Yes, a rebase would probably have helped, but this specific check is very annoying.

@jpkrohling Is it possible that it's treating code in test/e2e/utils.go as regular code rather than test code?
In any case, I rebased yesterday, and think I am up to date.

Is it possible that it's treating code in test/e2e/utils.go as regular code rather than test code?

I'm not quite sure and I would need to investigate further, but I think it's a bug in codecov's side: what I think is happening is that if there are new tests added to master which are not present in a PR, it might be counting as "this PR has fewer lines covered", even though the overall number of lines is also lower.

@jpkrohling @objectiser Unless there's anything else for me to do can one of you merge this?

Merging, as does not seem to be related to this PR.


sds: Add support to hot-reload CRL files

Watch CRL file changes when configured via SDS using local file path and reload the file when it has been updated on disk.
Fixes #27750
Additional Description:
When certificate validation context is used to configure CRL with local file path (DataSource.filename) Envoy will now watch for the file changes and reload the file when it is updated on disk. The change is then picked automatically when new SSL context is created (existing functionality).
This PR makes it unnecessary to restart Envoy to take the new CRL file into use.
Similar hot-reload is already implemented for trusted_ca in validation context. This PR adds the same for crl.
Risk Level: Low
Testing: Modified existing test case.
Docs Changes: Added info that file watch will apply to CRL as well. The added text is the same as existing description for ca_certificate.
Release Notes: tls: added support for hot-reloading CRL file when it changed on disk.

Looks good to me. Adding @ggreenway for a second pair of eyes.

I've added missing changelog entry.


[Feature Request]: Separate instance

Code of Conduct

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

Potential Enhancement
A checkbox that would install the modpack to a separate folder inside .minecraft
How it could be implemented
A checkbox in the GUI. Folder name could be Fabulously Optimized x.x.x for example, where x.x.x is the Minecraft version so it can be easily updated by subsequent installs.
Anything else?
No response

This is already supported by the file dialog to select a folder so you could select any folder anywhere - especially since from everything I've heard recently, you shouldn't nest instances inside .minecraft, you should use a different folder inside pretty much anywhere else.
We might still want to implement this, but not sure.

Well, if I select a new folder outside .minecraft, e.g. .minecraft-2, it gets stuck on setting profiles.

Well, if I select a new folder outside .minecraft, e.g. .minecraft-2, it gets stuck on setting profiles.

Well that's apparently a bug...you don't need to file a new ticket for it unless you want to, we have enough 😅. What's probably happening is that dir is being passed to the create profile function, which needs to be .minecraft since that's the only place that file exists.

Indeed, the installer needs to know where .minecraft is, so I wouldn't exactly consider that a bug if I don't tell it where it is :D
So separate profiles should be either inside or next to .minecraft, IMO letting users define the specific path/folder name is not necessary.

What's probably happening is that dir is being passed to the create profile function, which needs to be .minecraft since that's the only place that file exists.

We could hardcode the .minceraft folder for the create profile function since the only way for it to need another value is to change the launcher's working directory which can only be changed through command line options. at which point we can assume the user knows enough to not be troubled by this.

We could hardcode the .minceraft folder for the create profile function since the only way for it to need another value is to change the launcher's working directory which can only be changed through command line options. at which point we can assume the user knows enough to not be troubled by this.

I don't agree with that, Minecraft Launcher can also be used as portable, in a different hard drive or various methods in Linux.


dom-repeat causes mixins in custom-style not to be rewritten.

Description
A dom-repeat with no content in this particular place in this document causes CSS mixins in the document not to be rewritten into plain CSS variables.
Live Demo
http://plnkr.co/edit/py2cAnXP7sT0rrZlgK2o?p=preview
Steps to Reproduce
I don't know how to reproduce this other than with the above demo.
Expected Results
The CSS mixins are always rewritten.
Actual Results
The CSS mixins are not rewritten on the page in that demo with the dom-repeat.
Browsers Affected

[x] Chrome
[ ] Firefox
[X] Safari 10
(I didn't test others.)

Versions

Polymer: v2.0.0-rc.3
webcomponents: v1.0.0-rc.7

cc @azakus

Need to investigate if this belongs in Polymer or ShadyCSS

Looks like dom-repeat and dom-if can call into ShadyCSS before the dependencies for custom-style have loaded.
However, elements that don't have a template like dom-repeat don't need to try styling themselves, so the simplest fix is to just ensure that only elements with template call into ShadyCSS.


Getting stacktrace prefixed with .lambda_method

Hi,
I am using Nlog file target and layout is as follows-

When the i run my website locally, stacktrace works just fine but when I deploy my website on server and run website then each of the stacktrace is prefixed by
"<no type>.lambda_method" .
for example-
NT AUTHORITY\SYSTEM  ||||  2014-06-13 05:20:56.2777 ||||  Info  ||||  <no type>.lambda_method => ReportsController.ViewReports => Log.Info ||||  User has logged into the EPCS Reporting application.
Anyone have seen problem before.
Which setting am I missing here ?
Thanks,
Navjot

@ernavjotsingh2 if this is still a problem, please submit the requested info and re-open this issue.


Rules method in a dependency container

Can we use rules method in a dependency container? I placed rules method like picture but it's not working.

@alexwenzel

hi @tovantiep
I already used rules() within my own project. should work. Only i did not checked with recent updates, if it still works. I will check.

hi @tovantiep
i cannot reproduce :( this code works for me

Thank for your answer.  It's ok for now


Update coreml converter bench

@vfa-nghiach
Thanks, but two points.

This repository follows Github flow. So, please create a pull request to master.
I have already committed similar update to master. So, the pull will apply only https://github.com/akirasosa/mobile-semantic-segmentation/pull/4/files#diff-f54d1b71942ec42f18e9f8fc5460392aR55 .


[Bug]: npmi will not open anything with nodejs isntalled

Version

[X] Yes

General Question

[X] No

What happened?
Im having issues running the npmi option even when nodejs is installed properly
my terminal enter option says  PS D:\HARD DRIVE\create-10k-nft-collection-1.1.0>
and when i do this it says:
PS D:\HARD DRIVE\create-10k-nft-collection-1.1.0> npmi
npmi : The term 'npmi' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1

npmi

  + CategoryInfo          : ObjectNotFound: (npmi:String) [], CommandNotFoundException
  + FullyQualifiedErrorId : CommandNotFoundException

What have you tried?
I Googled and asked the community in the discord server but could not find a solution.
Relevant log output
PS D:\HARD DRIVE\create-10k-nft-collection-1.1.0> npmi
npmi : The term 'npmi' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1
+ npmi
+ ~~~~
    + CategoryInfo          : ObjectNotFound: (npmi:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

Protect your information

[X] I Agree

Did you try "npm i" ? npm[space]i)


Jekyll bug

When I loop over the pages on my jekyll site it picks up on one of your files that your created.
However I don't see that file in my directory.
{% assign sorted_pages = site.pages | sort:"order" %}
sorted_pages.url will output /assets/css/style.css in the loop. And at the top of the file has your name on it with your github url.

Sorry but there's nothing here that is going to be causing that issue

The easy workaround as mentioned over at jekyll #7478:
theme:null

It seems to be caused by github-pages (which apparently includes your normalize.css) and it still persists in the current version (204).


Decide phase

Resolves #2.
Implements the DECIDE phase in Granite.
As created, this PR is incomplete. The round numbers of the DECIDE messages should be consolidated. Currently, 0 is always output as the decision round (which is the most probable reason for the tests failing).

Continued in #30. This is deprecated.


Add Jenkins CI

We should add Jenkins CI to test our builds in a more representative environment.

I don't really see a need for this any more, Travis is working great and given the lag time for simple deployment, I don't want to think about how long getting Jenkins CI and associated permissions would take. Closing.


Transaction metadata gets dropped by Unicode NUL character

OS
Your OS: Gentoo Linux
Versions
The db-sync version (eg cardano-db-sync --version):
cardano-db-sync <IP_ADDRESS> - linux-x86_64 - ghc-8.10
git revision ec5b22b4364cca9a866b20de3564c27f2c751969
PostgreSQL version: 15.3
Build/Install Method
The method you use to build or install cardano-db-sync: nix build
Run method
The method you used to run cardano-db-sync (eg Nix/Docker/systemd/none): manually from CLI
Additional context
I traced this to a workaround made for #297 , now in https://github.com/input-output-hk/cardano-db-sync/blob/master/cardano-db-sync/src/Cardano/DbSync/Era/Util.hs line 50. Obviously the workaround has come as is to production.
TLDR; PostgreSQL can not record \0 in stringy types. JSON and Unicode encodings may contain \0. JSON formatted value is dropped altogether.
Problem Report
While running cardano-db-sync to initially populate the database, there are intermittent Warning log level messages:
[db-sync-node:Warning:75] [2023-05-31 11:02:29.12 UTC] prepareTxMetadata: dropped due to a Unicode NUL character.
The warning message is misleading. Tx metadata is not dropped, but simply the json column of the row is recorded as null. Metadata payload is still stored in CBOR format in the bytes column.
While this behaviour may not be critical, it can be a surprise to database consumer.  It might be a good idea to document in the schema.md that the json column has this limitation and that the bytes column records the object payload in CBOR encoding, if not producing some actual fix to this.

Indeed we can improve the logs for this.


ERROR: Could not find a version that satisfies the requirement opencv-python==4.5.1

Hello, trying to pip install requirements but getting this error:
!cd omnimatte && ls && pip3 install -r requirements.txt
Collecting torch==1.8.0
  Using cached torch-1.8.0-cp37-cp37m-manylinux1_x86_64.whl (735.5 MB)
Requirement already satisfied: torchvision in /usr/local/lib/python3.7/dist-packages (from -r requirements.txt (line 2)) (0.10.0+cu111)
Collecting dominate>=2.4.0
  Using cached dominate-2.6.0-py2.py3-none-any.whl (29 kB)
Collecting visdom>=0.1.8
  Using cached visdom-<IP_ADDRESS>.tar.gz (676 kB)
Requirement already satisfied: matplotlib>=3.2.1 in /usr/local/lib/python3.7/dist-packages (from -r requirements.txt (line 5)) (3.2.2)
ERROR: Could not find a version that satisfies the requirement opencv-python==4.5.1 (from versions: <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>)
ERROR: No matching distribution found for opencv-python==4.5.1 ```

I updated the requirements.txt file to change it from 4.5.1 to <IP_ADDRESS>. It worked, although not sure if it would lead to any downstream issue yet.
Were you able to resolve the issue by any other means, what did you do and did it work for you?


add option for seperate DoSocs path

while we can change the fossology path and options for the database, it'd also be nice if the user could set a path for DoSocs if it's already installed.  The way it's currently setup, if the user has already been using DoSocs for another purpose, you force them to set up a second copy for the Git-Scanner

Not sure about this, for are install process we assume you are not using doSOCS and it downloads and installs it for you so you literally have nothing to do. Already having doSOCS setup in another location is not really an issue either because doSOCs does not need to be installed to run. Since both can connect to the same MySQL database, you can point it to the existing instance.

Not correcting, as you can use a central SPDX database and doSOCS will be automatically configured.

but if a user is using multiple programs with doSOCS dependencies wouldn't it be nicer to have one copy of doSOCS than 6?


Move Constraints

Start working on advanced constraints like blocking and capturing.

I think the important part is the condition function. It makes sense to define a condition: Callable[[Square], bool] together with some data. Since legal_moves generates all possible moves from the selected square, it makes sense to check for

board edge
stops at enemy pieces, including the captured square
stops at friendly pieces, excluding the occupied square.

def list_moves(board: Board, selected_square: Square) -> set[tuple[Square, Piece]]:
    piece = board[selected_square]
    def condition(square: Square) -> bool:
        try:
            other_piece = board[square]
         except IndexError:
             return False
         return other_piece is None
    target_squares = set()
    for square in piece.legal_moves(selected_square, condition):
        other_piece = board[square]
        if other_piece is None or other_piece.is_black != piece.is_black:
            target_squares.add((square, other_piece))
     return target_squares

I think the important part is the condition function. It makes sense to define a condition: Callable[[Square], bool] together with some data. Since legal_moves generates all possible moves from the selected square, it makes sense to check for

This would require to modify the legal_moves method because, for Range Pieces, the pieces would keep going even if capturing an enemy piece.

Interesting, wit some adaptation this seems to be a working function. Just some minor tweeks:
def list_moves(board: Board, selected_square: Square) -> set[tuple[Square, Piece]]:
    piece = board[selected_square]
    def condition(square: Square) -> bool:
        other_piece = None
        try:
            other_piece = board[square]
        except IndexError:
            return False
        return other_piece is None or other_piece.is_black != piece.is_black
    target_squares = set()
    for square in piece.legal_moves(selected_square, condition):
        other_piece = board[square]
        if other_piece is None or other_piece.color != piece.color:
            target_squares.append((square, other_piece))
     return target_squares

Thos tuples are expected to be unique and tuples are immutable, so set is in compliance with our general logic of collecting legal moves. Otherwise, lets keep this around.

A few remarks about the latest commit. I tried to stuff the simplest constraint for legal_moves (generated squares within bounds of a board) by forcing squares to return a None object when invalid coordinates are given for their creation. This in practice provides a natural stopping condition when operating on squares with vectors.
I believe there is a better way to do this, but I'm just putting this out there for you to review. To be frank, I am trying to stretch by python brain cells once in a while, and this __new__ puzzle was a good exercise.

I think the important part is the condition function. It makes sense to define a condition: Callable[[Square], bool] together with some data. Since legal_moves generates all possible moves from the selected square, it makes sense to check for

board edge
stops at enemy pieces, including the captured square
stops at friendly pieces, excluding the occupied square.

def list_moves(board: Board, selected_square: Square) -> list[tuple[Square, Piece]]:
    piece = board[selected_square]
    def condition(square: Square) -> bool:
        other_piece = None
        try:
            other_piece = board[square]
        except IndexError:
            return False
        return other_piece is None or other_piece.is_black != piece.is_black
    target_squares = list()
    for square in piece.legal_moves(selected_square, condition):
        other_piece = board[square]
        if other_piece is None or other_piece.is_black != piece.is_black:
            target_squares.append((square, other_piece))
     return target_squares

This would require to modify the legal_moves method because, for Range Pieces, the pieces would keep going even if capturing an enemy piece.

Oh wait... I see now! Basically taking piece legal moves to generate a player's total legal moves, and performing the trimming at that level. Nice!

Since comparing weather two pieces are of the same subclass type and color is obsolete, I have overridden Piece == and != to designate color difference. This operation could be left over for the value attribute along with inequalities even, but value is never to be compared, just stored and added (added I said eh?), so for now I make this notation useful for both Piece and Board.
See tests for an example:
    def test_board_conditions(self):
        """Test the various logical conditions imposed at board level."""
        from src.board import Board
        board = Board()
        source = board["e2"]

        assert source is not None

        assert source != board["e2"] is False
        assert source != board["e4"] is False
        assert source != board["e7"] is True

        assert source == board["e2"] is True
        assert source == board["e4"] is False
        assert source == board["e7"] is False

Before I proceed with letting board handle conditioning, I will focus on how to compactify code logic and more specifically, get rid of all the None checks.

Before I proceed with letting board handle conditioning, I will focus on how to compactify code logic and more specifically, get rid of all the None checks.

One way to do this is to define a maybe function
from typing import TypeVar, Optional
from collections.abc import Callable

A = TypeVar("A")
B = TypeVar("B")

def maybe(a: Optional[A], f: Callable[[A], B], default=B) -> B:
    if a is None:
       return default
    return f(a)

@prototact I too prefer to sanitize input than to use a universal None-handler. I will follow your advice and try to trim the algebra as much as possible.

I went ahead with implementing movement blocking and capturing logic. Sanitizing None control is still outstanding, None sanity checks are all over the place. I should try to pull them down at a fundamental level and eliminate some cases, where wrong input should not be given and thus let an exception be thrown.

No success with None reduction. I only eliminated one late None check by incorporating it naturally into Board.__getitem__ where it belongs. I want to “pull down” as many of these None checks as possible. Like the famous Python adage says:

You (should) only have to say something once.

In other news, I have successfully drafted a method that uses two squares to form a move. I wish to refactor it to take a piece on the board and a target square.
Note that Board.moves keys all possible moves with the source square and not the source piece because Pieces are not immutable. But I think that in my current implementation I can safely throw away the Piece.square attribute and make them mutable again, with the only sacrifice being that Piece.moves methods will now assume a square argument. If that happens, I can key Board.moves with the board pieces instead (which is natural), which in turn will make Board.move naturally check amongst a player's pieces as well. Alternatively, check the commit coming right after for a shortcut that I might like.

And here it is.  (No color check is added, that's the last thing to implement)

Hmmm, even if I remove the Piece.square attribute and make Piece objects hashable, their default hash will be by type. I tried unsafe hashing, but it failed, pieces remained identical by type even with the default eq defined by the dataclass decorator, which would normally compare Piece objects by all their attributes. I believe unsafe hashing looks and compares only the immutable components of the corresponding dataclass. Which means a manual __hash__ may be required. This is not unreasonable, because if that happens, we will be able to create sets of pieces on the board (in particular two sets of pieces, one for black and one for white). This will enable us to hold player states as well (what pieces they have and where). Since Board.move swaps Piece objects on the board and does not destroy/create them, that makes sense.
I want to try one more time via the dataclass decorator by setting both eq=False and unsafe_hast=True before we move to manual hashing.

I will devise a custom chess position with various pieces all over the board, checking for several movement arrangements and conditions, for testing.

This is the position I used for testing movement constraints. It have various checks. To make this right we should probably make separate unit tests for each piece in specialized conditions that test said piece's specific moves.

I have reached the end of my abilities with the current code-base, to finish the moves I need to go unto implementing player and game logic (classes Player and Chess) respectively. This is because players will hold a collection of the pieces they have and the pieces they have captured, so perusing the available moves per player will be far easier on the board. The color attribute of Piece will become cosmetic. The alternative would be to keep tracking color everywhere, but I'd rather implement player logic, for the advanced features I may implement.
Do we merge this work, or do I go on and implement the whole program in one branch? Please review and merge, otherwise tell me what's wrong with it, fix it and merge it, so I can proceed. I may just branch out to proceed, but if your fixes are extensive then I will have to re-adjust all my future work, so I prefer to resolve this now.

Sorry, I could not wait. I proceeded in implementing player logic. However, this will be used from fundamental threat-detection required for king and king-influenced moves (like pins or defenders or castles), so such changes are still on topic. Have a look. I have not touched the None sanity checking yet, I would like to see this together live in a zoom session if possible @prototact. Also, lemme know how you like the game logic as it starts to manifest via the Player and Chess classes.

I am getting somewhere. I need to redefine deployable and capturable on two layers. The biggest challenge is implementing pins on the King. What are they? Moves that are nominally legal, but are eventually not, because if performed, they expose the king to a (discovered) check. Which means said pieces are pinned in all ways than the ones that still protect it. For example, if a white bishop defends his King from a black bishop (of the same color of course), the white bishop cannot move away, but it can still move in the direction of the attacking black bishop, so it is partially pinned.
This one will be the trickiest to implement of all, because it needs to check the situation after the desired move is made, so to make the checks, all legit moves must be performed in vitro! It is not too much of work, and I logically see no other way around it. I am close to finishing the implementation!

I did it! Full king safety is implemented! Not in the most efficient way, but in the most clean way. Code is largely self-documenting, and there are comments for the rest. Chess is indeed a very, very complex game. No wonder it stretches the mind so much. We should play more often!
All that remains is to finish implementing castling and en-passant.
Also in other news, I zapped a critter that was hiding in Board.__setitem__ for a very, very long time! Gotta love the debugger(s)!

Also, a favor: If you can devise more challenging tests for said constraints, please do. Thanks!

Apart from en-passant (not implemented at all yet) the code is in a very good state right now. If you wait one more day, you may review the whole set of constraints and make final remarks on the code. It will be fixing/improving mode only after I implement en-passant.

The en-passant rule is turn-sensitive so I had to complete some running parts of the game (for example turn cycling) and a bad, but working move-reader. In theory we can play games now. Work is needed on those rules, before merging this commit.
For now, en-passant successfully detects the ghost squares behind jumping pawns, but I have not implement getting rid of said ghosts if any, and on top (the hardest part) getting rid of the actual enemy pawn, if player takes his en-passant chance. In some sence ghost piece and respective pawn must be linked. Perhaps an internal reference? Not sure that this straightforward, but I have an idea that might work. More on this the coming days.

All done and tested.


Handle new Duplicate error from telescopes

This PR handles the new Duplicate exception from the telescope library. See RedPRL/sml-telescopes#10.
I also took the opportunity to do a little local refactoring of the elaborator's error reporting mechanism.

The build will fail until I can update the telescope dependency. (Can't use the usual trick since the PR is from a branch in a fork.)

looks good! I've just merged the other PR, and fixed the access issue.


RDBES: RS_ObservationActivityType update of codes

Please see attached spreadsheet for the 6 code updates.
RS_ObservationActivityType updates.xlsx

RS_ will be dropped


Tooltip does not show graph

On grafana 6.4.2 I updated the plugin to 0.5.0 release and I configured the rules to show the graph on tooltip but I have the tooltip like in the pictures without graph.
What can I do?
Thanks

Hi @digitecnodata,
whoo!!, can you verify in query instector if you have several metrics ?
What is the type of datasource ?
BR
Arnaud

Hi Arnaud and thanks for your reply.
I use influxdb as datasource and in this case I have several metrics but I tested with a query with only one metric and I have the same problem.

You have only on line, then graph displays only one point
Do not write your query with aggregation like count, min, max
write it like :
select cpu from measure where ... group by ...
and in rule, select your aggregation like current, last, ...
BR
Arnaud

There could also be the issue that in Drawio/Diagram.net, a shape is in front of the shape where we want the tooltip. In this situation the tooltip does not work!

ok, can you give me your xml definition ?

For me the issue was with the drawio diagram. I have remove the shape in front of the one I expected to show a tooltip. Now it works.


Add the method eth_accounts

the method eth_accounts
Description
Was trying to set up ENS on a chain built using polygon-edge, and was met with the error of
ProviderError: the method eth_accounts does not exist/is not available
Your environment

Windows Eleven
v0.4.1
dev

Steps to reproduce

Git clone https://github.com/candleplatforms/ens-contracts
run yarn
then run npx hardhat deploy --network candle

Expected behaviour
Should successfully compile and deploy contracts onto the polygon-edge chain
Actual behaviour
Returns an error stating that the method eth_accounts is not available (yet).
Logs
Compiled 113 Solidity files successfully
An unexpected error occurred:

ProviderError: the method eth_accounts does not exist/is not available
    at HttpProvider.request (C:\Users\samue\OneDrive\Documents\GitHub\ens-contracts\node_modules\hardhat\src\internal\core\providers\http.ts:78:19)
    at GanacheGasMultiplierProvider.request (C:\Users\samue\OneDrive\Documents\GitHub\ens-contracts\node_modules\hardhat\src\internal\core\providers\gas-providers.ts:312:34)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Proposed solution
Implement this functionality into the polygon-edge chain.

@samisbakedham Hello, Thank you for raising the issue.
I'm afraid Polygon Edge doesn't support eth_accounts JSON-RPC endpoint currently because Edge doesn't have local account management feature.
I think this error is thrown by hardhat. hardhat calls eth_accounts if its config doesn't have private keys. You can put your private keys in hardhat.config.js like this:
const config = {
  networks: {
    edge: {
      url: "...",
      accounts: [
        // Put your private keys here
        "0x...",
      ],
    },
  },
};

@samisbakedham Hello, Thank you for raising the issue.
I'm afraid Polygon Edge doesn't support eth_accounts JSON-RPC endpoint currently because Edge doesn't have local account management feature.
I think this error is thrown by hardhat. hardhat calls eth_accounts if its config doesn't have private keys. You can put your private keys in hardhat.config.js like this:

const config = {

  networks: {

    edge: {

      url: "...",

      accounts: [

        // Put your private keys here

        "0x...",

      ],

    },

  },

};

You're right, thanks. Fixed it. Could also be worth looking into adding local account management perhaps?

@samisbakedham Glad to hear that it is fixed! :) To reply to your question, we don’t plan on adding Account management to Polygon Edge, as that would be a security liability and the most common modern setups utilize client-side signing.
If you have any other questions, please let us know.


add user password authentication to schemagen

This PR adds user and password flags to schemagen. We needed this to generate table models for a Cassandra/Scylla that is password-protected. I manually tested it.

Thanks.


Improve cross-browser support

Based on some experimental browser features used by JBrowse 2, more work needs to be done in bundling to increase cross-browser support. Currently Safari and iOS are not supported.

Fixed with upgrade to 1.0.4


Slime rework ideas

Ideas:

Different healing mechanics: Needing to consume food in order to heal. Possibly lose heath over time. Health could also affect the size of the slime.
Transformation/Mimic mechanic: Difficult to implement, unlikely to do so.
Multiply and divide: Mechanically, I'm thinking about Tainted Eve from TBoI. Would be kinda tricky to implement.

All I have for now. Will update once I gather more ideas.

Some of these could be put into their own traits tbh.


Keyboard navigation enhancements with project dropdown

User Story
As an Automate User who is not able to use a mouse or other pointing device, I want to be able to more effectively use the keyboard.
Teams is the first resource where we have added a dropdown to the create modal.

Once you have tabbed to the dropdown, you can open it by pressing return or space. Pressing return or space again will close it. You can move around the items in the dropdown with tab or up/down arrows. Once you have moved off the dropdown "home" the only way to close the dropdown (with the keyboard) is to make your way back to that "home" position and then press return or space.
Would be useful if Escape could close the dropdown. Right now, Escape will close the whole modal. Should be able to add another Escape handler in the dropdown library component so that it will process the escape first rather than the whole modal.
Definition of Done
Can close an open dropdown in a modal with Escape and still keep the modal itself open.
Demo Script / Repro Steps
Detailed above.

there's already a card for the accessibility of this component, the original issue was not fully implemented. i think it may still be over on jira though. ill assign myself to get all the info transferred over.

this work has been deprioritized.


Adding repeat section causes date-time question to loose its formatting

Scenario:

Create a questionnaire with a repeat section along with a date-time question.
Enter a date-time answer
Add a repeat section (click on the +).
Observe the format of the date-time answer previously populated.

Actual:
The date and time part gets get updated to e.g "2015-03-07T14:45"
Expected:
The date and time part should retain its original format.

Thanks. Not able to reproduce this yet.
Can you reproduce this with http://localhost:8080/forms/index.html?xform=https://ona.io/enketo/forms/18900/form.xml (after clean clone of this repo, and running grunt server)
Maybe it's specific to one browser?

please re-open when you have more info that allows us to reproduce


Can't see any other users

I am using Xcode9 beta and iOS 11 beta, running the app will be fine.
But after I registered 2 users through my email.The avatar, Name, E-mail Label is empty in the leftBarButtonItem in Nav Controller, I don't know why.Also, the feather icon on the right isn't work.
For example, I logout my user A and login user B, when I click the feather icon it will show my user A by common sense, but it says Seems like you're alone in this world.
It is very strange, but after waiting for short time, it shows my user A.
I think that there will be some problem form my network connecting.
!!!The importing thing is that, even click in the user A, the app crashed when I tried to chat with user A
The report are shown as follows:

Quick Cut -> Model -> User.swift -> (Line 106)downloadAllUsers

let credentials = data["credentials"] as! [String: String]

Thread 1: EXC_BREAKPOINT (code=1, subcode=0x1010e0fe8)
fatal error: unexpectedly found nil while unwrapping an Optional value
2017-09-12 10:54:31.029320+0800 QuickChat[6848:2306995] fatal error: unexpectedly found nil while unwrapping an Optional value

There must be some bugs in the model.HELP Please migrate to the new Xcode version and fix these bugs.Thank you.😁

Hey @iWeslie I'm working on new version of chat which will support latest Xcode 9 and iOS 11. It should be ready by end of the month.

Nice dude.  Really support it man.

Work around - In firebase go to database and click "Try Firestore BETA", click "enable" the application should start working.

@elvincsci i did that still app not working..

guys I've updated the project, all issues are fixed


Feature request: Toast details section

Hey! During my commercial project, we faced the problem of showing some details information of the message shown in toast.  For example, our backend return some common error message as There was a problem with fetching user data, and related details message Mock response not found.
Because we like the design and usage of vue-toasted, I decided to fork this project and implement this option on my own.

It looks quite nice, it's simple next action which expanded the toast with a given details message.
If you also think it is a nice feature, I will be happy to prepare dedicated PR.
However firstly you have to know, that my implementation require some changes in html toast structure.

@shakee93 what about?

this can be achieved easily via custom toasts.
https://github.com/shakee93/vue-toasted#custom-toast-registration
take a look at the advanced example section
Vue.toasted.register('my_app_error',
    (payload) => {

        // if there is no message passed show default message
        if(!payload.message) {
    	    return `<div class="my-description"> my content ${payload.message} </div>`
        }

        // if there is a message show it with the message
        return "Oops.. ";
    },
    options
)

hope that helps

@shakee93 sorry for late respond. I've tried to use this feature at the beginning but it couldn't figured out how to do it properly. Now I've check it again and you are right it's possible but with some hacks let's say... https://jsfiddle.net/dimitor115/mvpd0y9z/
I still think that this is a great feature, and because it's optional behaviour I don't see a reason why just not implemented it?
I've ready implementation in my fork of you library inlined to my project, and I really would like to introduce it in PR.


Typo in ISSUE_TEMPLATE/feature-req.yml

there is a typo in the screenshot question in the above file. healps is the word.

Hello @joypoddar , thanks for the PR.
It would be great if you could get the issue assigned first and then start working on the changes.
I would recommend reading the contribution guidelines.


Dose Filer Stores  support etcd username and password

use helm chart install seaweedfs 3.43 in k8s cluster

  extraEnvironmentVars:
    WEED_ETCD_ENABLED: "true"
    WEED_ETCD_SERVERS: "http://etcd.sso:2379"
    WEED_ETCD_TIMEOUT: "3s"
    WEED_ETCD_USERNAME: "root"
    WEED_ETCD_PASSWORD:

run the command:  helm install seaweedfs -f values.yaml seaweedfs-3.43.tgz
see the error

{"level":"warn","ts":"2023-03-02T10:16:05.254Z","logger":"etcd-client","caller":"v3@v3.5.7/retry_interceptor.go:62","msg":"retrying of unary invoker failed","target":"etcd-endpoints://0xc00066c380/etcd.sso:2379","attempt":0,"error":"rpc error: code = InvalidArgument desc = etcdserver: user name is empty"}
F0302 10:16:05.254230 filer.go:133 read filer.store.id= : kv get: etcdserver: user name is empty
goroutine 1 [running]:

Dose Filer Stores support etcd username and password ?

https://github.com/seaweedfs/seaweedfs/blob/master/weed/filer/etcd/etcd_store.go#L34
No. But it should be very simple to add them. You are welcome to send a PR.

Hi @chrislusf
Firs time contributing to a go project, so forgive me if I have overlooked something.
The commit addresses the issue mensioned and seems to work ok in my environment.
Does it look ok to you?

Looks good. But you would need to create a PR for it.

Thank you for implementing this. The problem when setting just WEED_ETCD_USERNAME as root (with etcd is always root), the etcd client is throwing misleading error kv get: etcdserver: user name is empty.
Setting the password as well, works. Could be set in the helm chart values:
filer:
  ...
  extraEnvironmentVars:
    ...
    WEED_ETCD_USERNAME: root
    WEED_ETCD_PASSWORD:
      secretKeyRef:
        name: etcd
        key: etcd-root-password


Get rid of Qt3Support Module

In order to allow iSEG to switch to Qt5, we need to get rid of Qt3Support.
The main use of Qt3Support is the layout of widgets and possible in some legacy API functions, like isOn vs isChecked.
child of ITISFoundation/osparc-simcore#580

Hi all,
I wonder how far is this issue right now? We would like to install on windows but on Debian 11 but qt4 or lower are not supported anymore. So any idea when porting to qt5 will be ultimate? Thanks a lot in advance,
Matteo

I haven't had enough time to complete this. The remaining work is

porting the image viewer widget, because it uses legacy scroll bars from qt3.
fixing function calls to qt3support api

I would be happy to review/merge any pull requests though.

I'd love to port it, but without qt5, I do not get access to a working environment/version in our current OS. Hopefully, someone else may pick this up in the short term

on ubuntu 18.04 there is still a qt4 package. see the github workflow here for building iseg: https://github.com/ITISFoundation/osparc-iseg/blob/master/.github/workflows/build.yml
on ubuntu 20.04 you need to add a thirdparty ppa, e.g. (untested)
sudo add-apt-repository ppa:rock-core/qt4


RET-2840: Add Serving Notification values

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/RET-2840
Change description
When ET1 Serving sends out a notification, the URL in the email should go to the CitizenHub for that case rather https://www.employmenttribunals.service.gov.uk/
AC
As a claimant, when I receive the email for ET1 Serving, and I select the link to view the message, then I should be directed to login to my CitizenHub account
e.g.  https://www.claim-employment-tribunals.service.gov.uk/citizen-hub/1234567891234567
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x ] No

@ikirsanov you are passing through a new parameter but have not got code for it


Zig Zag Persistence Examples/Documentation

I am interested in using JavaPlex to compute zig zag persistent homology. An example or two would really help a lot. I didn't find any in the documentation. Could someone add one?

Hi Alex (and others!),
I've essentially never computed any zigzag persistent homology examples in JavaPlex. But if anybody has, please let me know, as I'd consider adding nice Matlab examples to the tutorial! Unfortunately I don't have the time right now to try to figure out the code myself.
Alex, you may be interested in the following files, which rely on zigzag persistent homology:
https://github.com/appliedtopology/javaplex/blob/master/src/matlab/experimental/witness_bootstrap/circle_pairwise.m
https://github.com/appliedtopology/javaplex/blob/master/src/matlab/experimental/witness_bootstrap/figure8_example.m
https://github.com/appliedtopology/javaplex/blob/master/src/matlab/experimental/witness_bootstrap/torus_example.m
I think some users in the past had trouble running those files, so I'm not sure what the current state of affairs is.
Best, Henry

Hello again. I was able to successfully run one of the zigzag examples that used the vietoris rips complex: https://github.com/appliedtopology/javaplex/blob/master/src/matlab/experimental/vietoris_rips_bootstrap/basic_sphere_example.m.
I ran into trouble with the witness complex examples, however. For example, when I ran
https://github.com/appliedtopology/javaplex/blob/master/src/matlab/experimental/witness_bootstrap/figure8_example.m, I got the following error:
Error using zigzagex (line 42)
Java exception occurred:
java.lang.UnsupportedOperationException at edu.stanford.math.plex4.streams.impl.WitnessStream.setPlex3Compatbility(Unknown Source) at edu.stanford.math.plex4.homology.zigzag.bootstrap.WitnessBootstrapper.performProjectionBootstrap(Unknown Source)
Just to try it out, I commented out all of the
Y_stream.setPlex3Compatbility(false); lines in WitnessBootstrapper.java, but that resulted in:
Error using zigzagex (line 42)
Java exception occurred:
java.lang.UnsupportedOperationException at edu.stanford.math.plex4.streams.impl.WitnessStream.getAssociatedSimplices(WitnessStream.java:157) at edu.stanford.math.plex4.streams.impl.WitnessBicomplex.constructComplex(WitnessBicomplex.java:32) at edu.stanford.math.plex4.streams.interfaces.PrimitiveStream.finalizeStream(PrimitiveStream.java:95)
at edu.stanford.math.plex4.homology.zigzag.bootstrap.WitnessBootstrapper.performProjectionBootstrap(WitnessBootstrapper.java:142)
Would this be relatively easy to fix? Henry, given your last answer it doesn't sound like you have the time to get to the bottom of this (Although I'd be very grateful if you could help me). I'll tag Andrew Tausz, who wrote the code in question. @ogdet, do you know what is going on here?
Thanks,
Alex

Hi Alex,
I don't know this part of the code well enough to have much to say. @ogdet or @michiexile may be able to give you some ideas, but @ogdet has largely moved on to other responsibilities.
Best, Henry


Writing to C/C++ standard output causes the server file worker to hang

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed.
Reduced the issue to a self-contained, reproducible test case.

Description
Lean server file workers hang eternally when code writes to the C/C++ standard output (i.e., not to the Lean standard output via IO methods). Restarting them in VS Code requires  reloading the window (restarting the server does not restart the work).
Steps to Reproduce

Pack the following C++ code:

extern.cpp:
#include <stdio.h>
#include <lean/io.h>

extern "C" uint32_t my_add(uint32_t a, uint32_t b) {
    return a + b;
}

extern "C" lean_obj_res my_hello(lean_obj_arg /* w */) {
  puts("Hello World");
  return lean_io_result_mk_ok(lean_box(0));
}

And the following Lean code:
Extern.lean:
@[extern "my_hello"] constant myHello : IO PUnit
@[extern "my_add"] constant myAdd : UInt32 → UInt32 → UInt32

into a plugin.

Run the following code with said plugin passed to the server:

Bug.lean:
import Extern
#eval myAdd 1 2 -- evaluates to 3, as expected
#eval myHello -- hangs eternally

Observe the bug. Note that running this file with Lean directly produces the following output:

Hello World
3

Expected behavior:
Ideally, the message would appear in the Infoview. Alternatively, the output could be printed to the 'Output' tab of VS Code (like messages written to standard error) if the former is not possible.
Actual behavior:
The file worker hangs eternally.
Reproduces how often:
Always.
Versions
OS: Windows 20H2
Lean (version 4.0.0-nightly-2021-08-20, commit 1624e42a5d6d, Release)
Additional Information
This is follow-up of #639. I did some more testing and discovered this bug, which explains that issue. Since the error message for missing external symbols is written directly to the C/C++ standard output (see this line), it hangs.

Redirecting stdout in a robust manner is not feasible, we've tried. We can have the watchdog look out for response lines that are not valid JSON, assume that streams were crossed, write out the line as an actual diagnostic, and restart the file worker.

This is the likely source of the hang:
https://github.com/leanprover/lean4/blob/9686910c72aaf1d39d28792265950b8f9f44880e/src/Lean/Server/Watchdog.lean#L212-L215
The watchdog thinks the file worker is crashing and wants to reap it, but the worker doesn't know that anything went wrong.

Yeah, you really shouldn't be writing to the system stdout stream. Lean stdio is redirected and should hopefullly just show up as a diagnostic, but we assume that system stdout is only used for LSP messages. I guess we could create a new pipe and use that, but even then if we cannot catch stdout, it will behave imprecisely (will not show up at the right place as a diagnostic).

@Vtec234 I think creating a new pipe would be a good idea. Then normal stdout output could just appear in the console (i.e., the VS Code 'Output' panel) (like stderr messages do).


The project is not compatible with the new version of EF Core 7

A project with this library cannot be run on the new version of EF Core 7. Can you suggest a solution to the problem?

Have you tried the 7.0 rc release?

I am working with the stable version 3.0.1 and did not know that a new one was available. When is the stable version 4.0.0 scheduled to be released?

Soon. Maybe you can Make it happen faster by trying out the rc release?

I tried version 4.0.0-rc.2 and it works. Thank you! Waiting for the release of the stable version.

because this project use https://github.com/dotMorten/Microsoft.SqlServer.Types
and this project use Microsoft.Data.SqlClient 4.00 that has conflict with EF7.0
I fix it and wait to accpet my PR on https://github.com/dotMorten/Microsoft.SqlServer.Types

The correct fix is for this library to depend on the latest Microsoft.SqlServer.Types


Progress reporting in between commits

Describe the feature
For more insight into how the node is progressing, we should add a way for stages and the tree to report progress in between commits. This progress will be rolled back if the node crashes or is stopped for any reason.
These things are important:

It is clear that the progress is not committed, and thus not final
That there is still a metric for committed progress (checkpoints), not only ephemeral progress

It would be nice if this worked with ETAs.
This will most likely be implemented in two steps:

[ ] Integrate in pipeline
[ ] Integrate in the tree

Additional context
No response

We do have this to certain degrees, and it is an ongoing effort to improve, so i think we can close this


NEXT-37995 - Fix customer can add review for product more than one ti…

…me via store api

1. Why is this change necessary?
2. What does this change do, exactly?
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Checklist

[ ] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[ ] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[ ] I have read the contribution requirements and fulfill them.

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


Application is crash when camera lost connection and reconnect

Can i recovery( application continue working) after this Is it okay after this happens (the 3d camera cable is loose or or unplug it and plug it back in) ?

i founded it hot_plug.py in examples folder


Electron 0.36.6: Context menu no longer works on OS X

Clicking on entries from the context menu does not work anymore. A regression with the context menu refactorings maybe?

@bpasero I added a console.log in the click handler of MenuItem in the source code of VS Code, I can see the message in devtools console when I clicked an menu item, so the context menu did work. It is probably because of something else caused the regression.
If this regression happens not only for v0.36.6 but for all >= v0.36.1, then it is probably caused by #3833.

I will investigate further tomorrow, just wanted to file this bug as a heads up if maybe it was something obvious.

@zcbenz so it looks like with the newer electron version, calling menu.popup() is no longer blocking the renderer process, so the V8 execution just continues without interrupt.
So was this changing? Then we need to adopt this new behavior.

@bpasero Before #3833, calling menu.popup in renderer process would still pump the message loop while the call itself is blocking, so when you click the menu item, its handler will called in renderer process before menu.popup ends. After the change the handler will be called after menu.popup ends.

Yup makes sense, I am ok with that and can probably easy adopt, so I am fine closing this.

Works nicely. The only down side from the previous solution is that you no longer have a way to find out if the context menu was canceled without picking an entry or not.


HIL (redis)

How to test?
First of all, spin up redis
$ docker run -p 6380:6379 -d redis

And then add the following entry to your .env
NEUROAGENT_HIL__REDIS_URI=redis://localhost:6380

I added a get-now tool that is the only tool that has the hil: True.
$ curl -X POST -H "content-type: application/json" -H "x-project-id: $PROJECT_ID" -H "x-virtual-lab-id: $VIRTUAL_LAB_ID" -H "Authorization: Bearer $TOKEN" -d '{"query": "Could you please call the get-now tool three times as the same time"}' 'localhost:8001/qa/run/' | jq

You can then use the approvals router and the endpoint to view and decline/approve outstanding requests.

Similar to Phantasm in terms of idea (besides the approval dashboard), however there is still the issue of the execution hanging. The hanging time is handled with the ttl, which for now is set to 10 mins by default. Now in my opinion issues will arise when we will start having more users. With more traffic we will have more and more concurrent hanging executions. To limit that we will have to reduce the ttl until a certain limit, which must have a lower bound to ensure that a user has enough time to read and modify his configuration.
Also thinking about the future, if we implement the queue mechanism to pick up requests it will block the queue for as long as the user wants, up to the ttl limit. This is both an advantage and a disadvantage: If your request is picked up you should be able to execute it until the end without going back to the queue but you also block other requests that could run while you do your approval. This is rather for larger simulations, since I assume that you will want to double check/do some research before clicking the "Yes I want to spend a ton of money" button. Speaking of which, having a larger ttl might also be necessary for bigger simulations since you want to quadruple check it before ruining your vlab.
On the plus side this solution is definitely cleaner than ours. I'm pretty sure we could refactor ours big time but it will remain some sort of mess regardless.
Also couple of side questions:

Is it okay for the frontend to wait for the answer of the user's request, while sending approval requests on the side ?
What will happen if the user clicks on another conversation while polling for approval then going back to the original one ?
How is the frontend knowing when there is something awaiting for approval ? Do they have to regularly query the GET all to know ?
What if the user wants to list the tools used to generate an AI message and one of the tool was modified by HIL ? The user will see the wrong inputs ? Also we have seen that this can be an issue for the model, e.g. ask to resolve Thalamus id -> HIL -> change Thalamus to auditory cortex -> Model says "I found the id of the thalamus" while showing the ID of the Auditory cortex. Modifying the inputs of the AI_TOOL message solved the problem apparently.
Aren't we hanging resources on the frontend as well ?

Anyway I tried to be super critic, but definitely a great solution I like it. Simple and efficient. Thank you for that !

Ah I see that modifying the input is not possible for now. I guess it's the patch endpoint that will handle it somehow ?

Now in my opinion issues will arise when we will start having more users. With more traffic we will have more and more concurrent hanging executions

Possibly. Honestly, I am not able to say at this point what is going to happen when there are a lot of requests. We can alternatively go for the pub sub model (redis supports it) but I guess there might be other downsides.

having a larger ttl might also be necessary for bigger simulations since you want to quadruple check it before ruining your vlab.
Yeh. Very likely. Agree.

Is it okay for the frontend to wait for the answer of the user's request, while sending approval requests on the side ?

Sure. Frontends do everything async so this isn't a problem.

What will happen if the user clicks on another conversation while polling for approval then going back to the original one ?

So currently the approvals are basically identified via approval_key=approval/<userid>/<approvalid>. So they are "global" for a given user and there is no concept of threads (=conversations). Of course this could be customized.

How is the frontend knowing when there is something awaiting for approval ? Do they have to regularly query the GET all to know ?

Exactly. The frontend would indeed have to be polling the approvals/ endpoint every X milliseconds. https://developer.mozilla.org/en-US/docs/Web/API/Window/setInterval

What if the user wants to list the tools used to generate an AI message and one of the tool was modified by HIL ? The user will see the wrong inputs ? Also we have seen that this can be an issue for the model, e.g. ask to resolve Thalamus id -> HIL -> change Thalamus to auditory cortex -> Model says "I found the id of the thalamus" while showing the ID of the Auditory cortex. Modifying the inputs of the AI_TOOL message solved the problem apparently.

This POC does not allow for modifications. I guess they are useful and phantasm does support them.

Aren't we hanging resources on the frontend as well ?

I mean the polling Frontend -> Backend polling is not a problem. Backend -> Redis might be more prone to issues but I am not sure.

Anyway I tried to be super critic, but definitely a great solution I like it. Simple and efficient. Thank you for that !

:)) Thanks. Again, it was just a suggestion. Happy to change it/not use it
@WonderPG


fix(floating-big-letter): calculate max width

Instead of calculating width based on 2nd index of a line, we calculate it based on the longest length instead. Without it, we have rendering issues on "wide" characters being forcibly wrapped.
This enables future support of custom fonts via figlet or toilet.
Fixes: #69
Before

After

Very nice change, but seem like plugins author is too busy


ImageBindModel.forward() got an unexpected keyword argument 'post_process'

Audio-conditioned image generation Error

I'm almost done, the last error I can't find a solution to online, hope someone can help me :(

hi, you can just remove the "post_process" argument.

嗨，您可以刪除“post_process”參數。

Thank you for your reply. I have already translated the image at the end but the RAM of the GPU is not enough. I would like to ask how many computing resources are needed to run such a code?

I am using the GPU V100 16GB provided by Google Colab pro
CPU 25GB but GPU RAM is still not enough


feat: Instant refresh views

What
Instantly update views on product edit

Steps

Go to the edit page of any product
Edit the basic details like name, brand name, etc ...
go back to the previous screen, then again to the same edit page
the changes have not been updated

This pr would fix those problems

would immediately change the product state in the edit screen
works even when offline
Immediate response compared to what was earlier after the offline pr merge

Screen Recording
https://user-images.githubusercontent.com/57723319/188330807-ce4ea172-ea3d-4af3-9699-6ccac6c2abb1.mp4
Part of

https://github.com/orgs/openfoodfacts/projects/33

@monsieurtanuki
I think this is what Aman's PR is about.
https://user-images.githubusercontent.com/33085535/188146635-f072e9c8-bf9d-4582-9fce-32183198f88b.mp4

@abughalib This issue does not happen on the other edit detail pages, like "edit categories".
That's why I suggest to apply a code similar to what is implemented in "edit categories" regarding saving the data (and refreshing it locally).
Which won't work on offline mode.
And then, once the whole product editions work in online mode, find a way to make it work in a downgraded offline mode.

One problem here is probably a mix between the offline mode (with its tasks) and the normal mode in the rest of the app (we call the server, the server saves the product, we refresh from the server, the consumer/provider is called, the product data is refreshed in the whole app).

Bonjour @monsieurtanuki
Yeah agreed, some things were working fine earlier as you're suggesting
But since the last merge for keeping simple tasks in background like these edits and uploading images, What's actually happening is we are doing the task of calling the API and updating the database in background, and
I can't access context there, so can't set the state there in the background.
My solution was to, as we already know what fields we change
https://github.com/openfoodfacts/smooth-app/blob/develop/packages/smooth_app/lib/pages/product/nutrition_page_loaded.dart
    final Product? product = await daoProduct.get(
      _product.barcode!,
    );
    // We go and chek in the local database if the product is
    // already in the database. If it is, we update the fields of the product.
    //And if it is not, we create a new product with the fields of the _product
    // and we insert it in the database. (Giving the user an immediate feedback)
    if (product != null) {
      product.servingSize = changedProduct.servingSize;
      product.nutriments = changedProduct.nutriments;
      await daoProduct.put(product);
    }
    localDatabase.notifyListeners();
    if (mounted) {
      ScaffoldMessenger.of(context).showSnackBar(
        SnackBar(
          content: Text(appLocalizations.product_task_background_schedule),
        ),
      );
    }
    return true;
  }
}

We can, we just update the product in the local database and the change the set/view with provider accordingly so in case if our queries take time to be done (and also I can't update state from background) it gives the user immediate results that at least their changes did happen, and it appears also.
I am just stuck on the lib\pages\product\simple_input_page.dart pages to implement something where I could just get the changes, and create a product which is merge of the changed product and the cached local database product. This is where I was stuck and that's why I left the pr as draft, any helps would be appreciated
I Hope this clears some confusion about what this pr was intended for,

Hi @AshAman999!
I think I understood most of the situation here.
My suggestion is still the same: first do something that works (cf. SimpleInputPage), and in a next PR do something that would work offline too. I can help you with that next PR, which would be something like an upgrade of  saveAndRefresh.

I can't access context there, so can't set the state there in the background.

So don't use setState then. Maybe notifyListeners inside the isolate? Maybe notifyListeners outside the isolate when it returns?
But first, do something that works online.

Codecov Report

Merging #2901 (d6756cd) into develop (d9c35d7) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

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

@@            Coverage Diff             @@
##           develop   #2901      +/-   ##
==========================================
- Coverage     6.71%   6.69%   -0.02%
==========================================
  Files          228     228
  Lines        11117   11149      +32
==========================================
  Hits           746     746
- Misses       10371   10403      +32

Impacted Files
Coverage Δ

..._app/lib/pages/product/add_basic_details_page.dart
0.00% <0.00%> (ø)

...h_app/lib/pages/product/edit_ingredients_page.dart
0.00% <0.00%> (ø)

...h_app/lib/pages/product/nutrition_page_loaded.dart
0.00% <0.00%> (ø)

...mooth_app/lib/pages/product/simple_input_page.dart
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Hello @monsieurtanuki
For something that actually listens to the change from the database on that screen, I thought of other ideas
Currently, you make a change, and as soon as you exit the page the background process for that task is running and when it finishes it fetches the fresh product and saves it into the database.
Since the database product is already updated, what probably could be done is running a periodic function of maybe a 5-second delay on edit_product_page.dart that would fetch the latest product from the database and update the view there.
Surely there can be better options than running periodic function, but  for now, this is the only solution I was able to come up with, I could create a pr for that if you say,
Meanwhile, this pr would solve some of the problems for now

Bonjour @monsieurtanuki
Phew 😓 , Finally was able to understand the code for nutrient fields,
I hope this would be okay enough
Thanks :)

Ok, it seems work is still ongoing on the PR. Waiting for a while before merging.

Ok, it seems work is still ongoing on the PR. Waiting for a while before merging.

It's done, and approved
We can merge it for sure , no problem in that ✅

I've already reviewed (many many times) this code. I approved it this morning.
Then I think @AshAman999 had an unfortunate action that pushed again exactly the same code. Am I right?
If so, please consider the code approved, and please @AshAman999 avoid this kind of incident, your code could smoothly have been merged.

Real Sorry about this @monsieurtanuki
I just rebased the pr on the latest changes, Better be careful about this next time.


Failed to create vpn credentials, error: invalid character 'S' looking for beginning of value

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the library and it is still present.

zscaler-sdk-go version
2.5.0
Go environment
NA
Expected output
Expected the vpn credential to be created
Actual output
Getting an error when creating vpn credentials using zia Create API
" invalid character 'S' looking for beginning of value "
Code demonstrating the issue
configuredVpnCreds, _, err := service.Create(ziaVpnCreds) 	if err != nil { 		return nil, err 	}
Steps to reproduce
Kept the server running for a while that is responsible for making Zscaler requests via zscaler-sdk
Upon restarting the server, the request go through just fine.
References
No response

@gauravv9 Please share the configuration for analysis or open a support case with Zscaler support and ask for an escalation.
We can't provide you an answer without debug logs or the actual configuration itself.

 {
    "type": "UFQDN",
    "fqdn"<EMAIL_ADDRESS>    "comments":"Test purpose only",
    "preSharedKey":"xyz"
  }

This is the configuration that was applied and on restarting the server it succeeded.

Seems to be getting SESSION_NOT_VALID as a response.
2024-05-16 22:09:05.593
msg="Zia: unmarshal error: (invalid character 'S' looking for beginning of value): ([83 69 83 83 73 79 78 95 78 79 84 95 86 65 76 73 68])" file="zia/client.go:160"```
Please correct me if I misunderstood, the session is refreshed in GenericRequest isn't it?

@gauravv9
Please download the latest version of the SDK ZIA package v2.61.2, which contains a fix regarding session refresh, and let us know if it fixed the problem you've reported.
Thanks.
Zscaler DevRel

@gauravv9 Please share the GO program code you're using containing the client instantiation for further testing.
Thanks


DefinitionUri supports a relative path

Clarified documentation that DefinitionUri can support a relative path which is less steps than needing uploading the swagger definition to S3 first.

Thanks for the PR.
So not quite. If you use the AWS CLI command aws cloudformation package it will do the uploading job for you. But if you are passing a SAM template directly to CloudFormation, DefinitionUri has to be a S3 URI. Technically, the spec accepts only S3 URI for this property.
Would it be better to just add a note below the table, explaining that if you use a relative path, you should use the CLI to package first?


ink! playground

Grant Application
This application is (select one):

Speculative (use this by default)

This application is (select one):

Public (fully)

Abstract
ink! playground is a browser IDE for Substrate's smart contract.
Checklist

[x] The grants document has been read and understood.
[ ] The Google Form will be completed accurately. Note that the Google Form requires the pull request URL.
[x] Abstract (above) is succinct and complete.
[x] The application is being included into the correct directory: either 'targeted' or 'speculative'.
[x] The application includes a project description.
[x] The application includes all names of team members.
[x] The application includes a description of the team's experience.
[x] The application includes all necessary links (e.g. GitHub and LinkedIn)
[ ] The "Development Roadmap" section in the application has a timeline of development ("milestones").
[ ] The "Development Roadmap" section in the application has an estimate of funds required.
[ ] The "Development Roadmap" section gives an indication of the team's long term plans.
[ ] The "Development Roadmap" section includes documentation as a deliverable for at least one milestone.

Hello! Was this application made through the Google form located here? That form needs to be filled out for it to be considered.
Thanks,
Dieter

Hi!
I've just send the google form now. Sorry for being late
but still the google docs description file is now empty, so i will fill in hurry.

Now I updated the google document of a grant application, which I send you on google form. Please confirm it.


Broken domain link customembroideredpatches.com

https://metasmoke.erwaysoftware.com/domains/40223 produces a traceback after I tried to link it to gs-jj.com via the "Link a domain" dialog

Fixed with 3895ae7 and 0ac260e.


FID in CrUX update

What's changed, or what was fixed?

added a new update for FID in CrUX

Fixes: N/A
Target Live Date: 2018-07-10

[x] This has been reviewed and approved by (@igrigorik)
[x] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[x] I've staged the site and manually verified that my content displays correctly.

CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/


Not processing text containing html tags properly

Adding an item containing html tags to the list, such as the text: Text escaping: <xyz>tag</xyz> will yield incorrect suggestions.
Setup
<input id="myinput" />

or
<input class="awesomplete" data-list="#mylist" data-minchars="1" />

Text escaping: <xyz>tag</xyz>

Results
Typing the letter 't' in the input box will give the following incorrect suggestion (missing the <xyz> and </xyz> that was surrounding the word 'tag'):

Typing the letter 'x' in the input box will give the following incorrect suggestion (notice the missing '<' after the word "tag" in the suggestion):

I have a similar issue. It looks like awesomplete interprets HTML within suggestions.
Is there a setting to prevent this behavior and keep the tags escaped properly?
Note that


include:add FNM_CASEFOLD to header file

Summary
add FNM_CASEFOLD to header file
Impact
nothing
Testing
vela

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


fix: remove extra / in make install and uninstall

Although $(DESTDIR)/$(LIBEXECPODMAN) will install normally, but the correct path is $(DESTDIR)$(LIBEXECPODMAN).

LGTM

/lgtm


Ferini/kotlin gradle dsl

:loudspeaker: Type of change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[x] Refactoring

:scroll: Description

:bulb: Motivation and Context

:green_heart: How did you test it?
:pencil: Checklist

[ ] I ran ./gradlew spotlessApply before submitting the PR
[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] All tests passing

:crystal_ball: Next steps
:camera_flash: Screenshots / GIFs

Hi, thanks for the PR. How is this different to #807?

Hi, thanks for the PR. How is this different to #807?

Hi, I did this as part of experimenting with kotlin DSL on a separate branch and wanted a way to share the diff of what was done. It was not intended to be merged in this state, that's why it is only draft. In the hindsight, I probably could just create a PR in my forked repo.
I saw the other one already exists and it is mostly similar to this one. I took some different approach in some areas (eg. I kept separate shared_dependencies.gradle.kts and test_dependencies.gradle.kts and moved them to buildSrc as precompiled plugins), but I am not sure if that makes this one better suitable than the other.
If you want to check it out and think this might be better candidate to merge, I am happy to clean it up, otherwise feel free to close it.

@pfmaggi FYI

As @ghus-raba wrote, the main difference is having the precompiled plugins for the dependencies while #807 keep them together in a module.
Most of the other things seems small differences.
#807 is still using AGP 3.6 as there's an issue with 4.0 when running the tests.

Thanks @ghus-raba for the PR, I'm updating #807 using some of the ideas you used here.


Cascading Menu is hiding Some Setups

Project Details are :
https://paste.apache.org/cpo8y
Test case is available on VDI.
Go to Inventory from Select in Menu
Go to PSSM -> from Setups in Menu
Cascading Menu is hiding some setups because in PSSM sub-menu there is many setup it is showing 8 to 9 setups only.
I am unable to add css for this Cascading Menu because its ID is
body > div > div:nth-child(4) On this type of Id having div:nth-child(4) is no accessible in default.css

now it is showing only last setup on the top of application :


is there a faster way if i want to go to a specific year

It is not an issue but an improvement: What I am trying to do is to set the date to Jan-10-1968. Is there a quick way to do it.

[calendar selectDate:.... scrollToDate:YES];


lab expectations assertions

lab, it turns out, doesn't actually assert that all the expects() are called unless you include -a code into the lab command. This adds that command to the npm test script, and fixes the various incomplete expects statements.

:+1: thanks!


Good use of t.plan(n) rule

Rule proposal: Check the good use of t.plan

Check that t.plan(n) is used with n existing being one integer parameter
Check that the number of assertions equals n
Ignore this rule when n is not a Literal number (impossible to know how many assertions there are)

Use case: When linting the code in the editor. Not very useful otherwise.
Name TBD
Bad
test(t => {
  t.plan();
  t.is(1, 1);
});

test(t => {
  t.plan(2); // Incorrect number of assertions
  t.is(1, 1);
});

test(t => {
  t.plan('2'); // Not an integer
  t.is(1, 1);
});

test(t => {
  t.plan(2.5); // Not an integer
  t.is(1, 1);
});

test(t => {
  t.plan(1); // Incorrect number of assertions in all cases
  if (SOME_VAL === 1) {
    t.is(1, 1);
  }
});

Good
test(t => {
  t.plan(1);
  t.is(1, 1);
});

test(t => {
  t.plan(1);
  if (SOME_VAL === 1) {
    t.is(1, 1);
  } else {
    t.is(2, 2);
  }
});

test(t => {
  t.plan(SOME_OTHER_VAL); // Ignore this case, as it's not possible (usually?) to find out
  t.is(1, 1);
});

Good idea, but it should align with https://github.com/sindresorhus/ava/blob/master/docs/recipes/when-to-use-plan.md. There's no reason to use t.plan() with sync tests.

Man I should really take a deeper look at the recipes. Will do that soon(-ish) and come back afterwards :d

Check that the number of assertions equals n

This breaks a case, when you run assertions in a loop:
t.plan(4);

arr.forEach(item => {
  t.true(item);
});

Linter will count it as one assertion, thus not matching t.plan().

Yeah, thought of that afterwards, don't think it is really possible...


So in the demos its bugged

https://github.com/liabru/matter-js/blob/master/examples/terraing.js
it doesent work you put a g in it(in the terrain demo)

(outsider comment for clarification)
It seems like you are referring to the source code link that shows up on the Demos web page. Correct me if I am wrong.
https://brm.io/matter-js/demo/#terrain, at the button that says Source (the {} button).
My browser shows me that the Source button would take me to .../terraing.js (with a g).

Thanks for reporting, looks like this is no longer an issue now the demo has been updated.


queryx: Added CAS exec functions (#98)

Added ExecCAS and ExecCASRelease functions suitable for INSERT ... IF
NOT EXISTS and UPDATE's containing IF statement.
Functions returns information wheter query was applied or not.
Fixes #98

Thanks!


Update so that controlled/suggested vocabularies are also grabbed from Earthref

Otherwise, we get crazy problems when updating.  Sigh.  Or maybe go back to using the cached version of the data model....

For the moment, only used the cached stuff.  We need to either do all from earthref or all cached.

Looks like tests are passing and this is working again, and more efficiently than before.


"HTML-CSS" in config fails to render the equation

The font of my blog using the following config code to support latex:
<script type="text/x-mathjax-config">
  MathJax.Hub.Config({
    TeX: {
        equationNumbers: {
          autoNumber: "all"
        }
    },
    tex2jax: {
      inlineMath:  [ ["$", "$"] ],
      displayMath: [ ["$$","$$"] ],
      processEscapes: true,
    },
    "HTML-CSS": {
        availableFonts: ["TeX", "STIX-Web", "Asana-Math", "Neo-Euler", "Gyre-Pagella", "Gyre-Termes", "Latin-Modern"],
        preferredFont: "Asana-Math",
        showMathMenu: false,
    }
  });
</script>
<script src='https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.4/latest.js?config=TeX-MML-AM_CHTML' async></script>

However I have tried to change preferredFont for several times, but "HTML-CSS" block always fails to render in right way whether for showMathMenu or preferredFont.
I hope to change my latex font from this

to this one as in many papers:

It seems that there is a little difference but it indeed makes a big influence..
I have checked the Chrome in inspect mode that the font-family is as follows:

Could you help me? Appreciate in advance!

Note that by requesting TeX-MML-AM_CHTML as your config file, you are requesting the CommonHTML output renderer, not the older HTML-CSS output renderer.  So you are configuring the wrong renderer.  You will not be able to change the font for the CommonHTML because it only has data for the MathJax TeX font (based on the Computer Modern fonts), not the other web fonts available in SVG and HTML-CSS output.
Also, the two images you provide both appear to use Computer Modern (or perhaps Latin Modern) fonts.  The second one certainly is not the Asana font that you seem to be trying to use.

Thanks for your reply.
The second screenshot is taken from the paper, while the first one is taken from my jekyll-based blog.
I indeed hope to change the font to the second, so is there any way that I can configure the correct renderer to solve this problem, like reading source code of my jekyll theme?

I indeed hope to change the font to the second,

You have not said what the font actually is.  The math characters appear to the Computer Modern or Latin Modern fonts, and the output from MathJax is already using the MathJax TeX fonts, which are derived from the Computer Modern fonts, so the math characters themselves are already in the font you are requesting (or as close to it as MathJax can provide).
The text font appears to be something based on the Times font.  MathJax can be configured to use the surrounding text font for the contents of \text{} commands, and if your surrounding font is Times, then you should get that for the \text{} in your expressions.  Use mtextFontInherit: true in your configuration as
<script type="text/x-mathjax-config">
  MathJax.Hub.Config({
    TeX: {
        equationNumbers: {
          autoNumber: "all"
        }
    },
    tex2jax: {
      inlineMath:  [ ["$", "$"] ],
      displayMath: [ ["$$","$$"] ],
      processEscapes: true,
    },
    CommonHTML: {
      mtextFontInherit: true
    }
  });
</script>
<script src='https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.5/latest.js?config=TeX-AMS_CHTML' async></script>

MathJax only has a fixed number of fonts that it can be configured to work with, so it does not handle arbitrary fonts, in general.  But the MathJax TeX fonts, together with inheriting the font from the surrounding text (provided it is Times or a derivative), is as close to your second image as MathJax will produce.
I do see that there are some spacing differences between the two images, and the spacing in your first image is not what MathJax would normally produce.  I wonder if there is CSS on your page that is interfering with MathJax's layout (e.g., CSS that adds padding to all <span> elements, for example)?

Appreciate your patience and help, I am a newbie of front-end techniques.
I am using Mathjax to render my homepage, and as you say, I guess there might be other CSS rendering my page, fork from here.
But now I don't care the font of the formula anymore, I think the above one is fine! I will study your reply carefully. Btw $\Phi is much different between two pictures.
Thanks again for your help!


Reduce memory usage

Codecov Report

Merging #248 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #248   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          12     12
  Lines        1603   1610    +7
=====================================
+ Hits         1603   1610    +7

Impacted Files
Coverage Δ

ldap2pg/script.py
100% <100%> (ø)
:arrow_up:

ldap2pg/privilege.py
100% <100%> (ø)
:arrow_up:

ldap2pg/role.py
100% <100%> (ø)
:arrow_up:

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 14a64d2...c9e1d25. Read the comment docs.


fix(getTokenData2): Adding token icon lookup

:tada: This PR is included in version 2.7.4 :tada:
Your semantic-release bot :package::rocket:


HTTPRoute wildcard hostname precedence doesn't match spec

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I've deployed two HTTPRoutes, one with a wildcard hostname (*.tplant.au) and one specific (myhost.tplant.au). The second route should take precedence per the Gateway API spec, and did when implemented with the https://github.com/nginxinc/nginx-kubernetes-gateway GatewayClass.
What did you see instead?
The wildcard route took precedence over the specific route, and traffic only flowed to it.
What version of Traefik are you using?
2.9.6 via the official Helm chart (20.8.0)
What is your environment & configuration?
https://github.com/pl4nty/lab-infra/blob/main/kubernetes/base/networking/traefik/traefik.yaml
If applicable, please paste the log output in DEBUG level
No response

Hello @pl4nty,
You are right about the precedence that should apply and this is indeed the default behavior in Traefik, but it applies only when evaluating two routes/routers against each other. After double-checking the code base, it seems that the current implementation satisfies the expected behavior from the Gateway API Spec.
In the referenced commit, it seems that the two hostnames were defined on the same route, which would have produced only one router in Traefik, with a host rule combining the two hostnames. Which could explain the behavior you have experienced.
As a side note, the hostnames in the configuration in the referenced commit do not match the example you provided (i.e.: *.tplant.au and myhost.tplant.au).
We are closing this issue, but feel free to ask to re-open it if you think we are missing something. In such a case, could you please share the HTTPRoute/s definition/s?

@rtribotte thanks for the reply. The referenced commit was my workaround by removing the wildcard route, since the wildcard route was overriding other routes that I wanted to use. The examples are:
Name:         home-assistant
Namespace:    default
Labels:       app.kubernetes.io/instance=home-assistant
              app.kubernetes.io/name=home-assistant
API Version:  gateway.networking.k8s.io/v1alpha2
Kind:         HTTPRoute
Spec:
  Hostnames:
    home-assistant.tplant.au
  Parent Refs:
    Group:      gateway.networking.k8s.io
    Kind:       Gateway
    Name:       traefik-gateway
    Namespace:  default
  Rules:
    Backend Refs:
      Group:
      Kind:       Service
      Name:       home-assistant
      Namespace:  default
      Port:       8123
      Weight:     1
    Matches:
      Path:
        Type:   PathPrefix
        Value:  /

Name:         well-known
Namespace:    default
Labels:       app.kubernetes.io/instance=well-known
              app.kubernetes.io/name=well-known
API Version:  gateway.networking.k8s.io/v1alpha2
Kind:         HTTPRoute
Spec:
  Hostnames:
    *.tplant.au
    tplant.au
  Parent Refs:
    Group:      gateway.networking.k8s.io
    Kind:       Gateway
    Name:       traefik-gateway
    Namespace:  default
  Rules:
    Backend Refs:
      Group:
      Kind:       Service
      Name:       well-known
      Namespace:  default
      Port:       80
      Weight:     1
    Matches:
      Path:
        Type:   PathPrefix
        Value:  /

In this instance, all traffic was routed to well-known, and home-assistant was unreachable.

Hello @pl4nty,
Thanks for sharing the HTTPRoute manifests.
Can you please also share the full debug logs of Traefik?

Happy to close this one, I stopped using Traefik due to another known issue - dropping all HTTPRoutes if 1 service was unhealthy


Action group compact

Closes https://jira.corp.adobe.com/browse/RSP-1808
✅ Pull Request Checklist:

[ ] Included link to corresponding Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Team:

@snowystinger - wanna merge latest from master to bring this branch up to speed?

When testing I click between the buttons and I'm able to get a focus ring around the entire group.
Chrome

Firefox

Outside of scope, but should docs cover density and disabledKeys? I don't think isJustified needs to be covered.


Use behavior api for eth protocol

builds on #1150
What was wrong?
We need to get rid of ETHPeer and LESPeer.  These both have functionality that can be replaced by behaviors.
How was it fixed?
Create LESAPI and ETHAPI and move the handling of NewBlock/Announce messages into these.
To-Do

[x] Clean up commit history

Cute Animal Picture

Well, I borked this branches history (will fix) but wanted to drop a note that I've been running this version of the branch for a few hours and nothing seems out of place/broken because of it.