CLI Tool for creating governance preimages

This tool creates preimages in which can be executed on the relay chain to send governance messages to BridgeHub. The tool also computes the correct weights for Transact and UnpaidExecution.
It relies on generated bindings for BridgeHub and the Relay chain, which ensures type safety.
Right now, as examples, it supports governance commands for setting the operational mode of the gateway, as well as upgrading the gateway.
$ target/debug/snowbridge-control
Usage: snowbridge-control [FORMAT] <COMMAND>

Commands:
  gateway-operating-mode  Change the gateway operating mode
  upgrade                 Upgrade the Gateway contract
  help                    Print this message or the help of the given subcommand(s)

Arguments:
  [FORMAT]  Output format of preimage [default: hex] [possible values: hex, binary]

Options:
  -h, --help     Print help
  -V, --version  Print version

Compilation
The tool needs to be compiled separately for each our environments:
# Rococo (default)
cargo build

# Kusama
cargo build --no-default-features --features kusama

# Polkadot
cargo build --no-default-features --features polkadot

Merging as I require this change.


fix: the stored last n headers is less than expected

Updates

When add proved blocks,

if the count of new last N headers is less than the expected count of the last N blocks, just copy some old last N headers to make sure we have enough stored last N headers.

if the count of new last N headers is greater than the expected count of the last N blocks, just remove some of them to avoid unlimited memory growth.

Refactor unit tests to make it easier to add an assert to several unit tests, not just copy the code several times.
And also add more unit tests to improve the coverage.

Resolve #117.

Codecov Report
Base: 78.42% // Head: 79.56% // Increases project coverage by +1.14% :tada:

Coverage data is based on head (b629b2d) compared to base (c39cce3).
Patch coverage: 97.59% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #121      +/-   ##
===========================================
+ Coverage    78.42%   79.56%   +1.14%
===========================================
  Files           23       23
  Lines         4792     4855      +63
===========================================
+ Hits          3758     3863     +105
+ Misses        1034      992      -42

Flag
Coverage Δ

unittests
79.56% <97.59%> (+1.14%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/protocols/light_client/peers.rs
85.34% <80.00%> (-0.59%)
:arrow_down:

...s/light_client/components/send_last_state_proof.rs
93.06% <98.27%> (+7.57%)
:arrow_up:

...otocols/light_client/components/send_last_state.rs
94.59% <100.00%> (+0.15%)
:arrow_up:

src/protocols/light_client/mod.rs
80.87% <100.00%> (+0.74%)
:arrow_up:

src/protocols/status.rs
84.09% <100.00%> (+1.16%)
:arrow_up:

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

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


How to configure different worlds leveldataoverride.lua

I have multiple worlds, each of which is not the same. How can i configure it?

You can use the environment-variable LEVELDATA_OVERRIDES to set the leveldataoverride.lua contents for each server/container separately.

@thasmo I mean in the same docker-compose.yml file, how to configure multiple servers

This should be possible by using variable substitution. You have to use a different variable for each server tho, but that should work.

@thasmo I'm sorry, I do not know what to do. I observe your example, start.sh and leveldataoverride.lua and modoverrides.lua. They are not bound to a world.
I try to add MODS_OVERRIDES = mymodoverrides.lua in the world.env file, but it does not take effect

f you do not add in start.sh, it will not be configured successfully. But start.sh can not distinguish between the world

I tried variable substitution， can not take effect.
Log output has
shell main         | [00:00:05]: ERROR: Failed to load modoverrides.lua

Sorry, I'm not familiar with shell scripting

Again sorry, I tried a variety of ways to replace variables for each server. Can you write a demo?

I will help you in about 9 hours. :)

https://github.com/dst-academy/docker-dontstarvetogether/tree/develop/docs/examples/multiple Here you have a demo. It doesn't touch the leveldataoverride.lua but you can get the general idea.

@thasmo Thank you very much, I have tried to run your example, but they can not load leveldataoverride.lua and modoverrides.lua.
My approach is: in your example, change the token for my own token, then sh start.sh. Then observe the generated leveldataoverride.lua and modoverrides.lua. Very frustrating that it does not work.
@BraisGabin This example is very simple. I need to build multiple worlds, each of which has different leveldataoverride.lua and modoverrides.lua.

I try to do this, remove MODS_OVERRIDES in the world.env file, add MODS_OVERRIDES = in the modoverrides.lua file header, change the contents of start.sh:
 #!/usr/bin/env bash
 cat  modoverrides.lua >> world.env
 docker-compose up -d
It can work

@turing-king, you need to adapt your docker-compose.yml file similar to this:
version: "2.2"
  services:
    world:
      environment:
        - LEVELDATA_OVERRIDES=${LEVELDATA_OVERRIDES_ONE}
        - MOD_OVERRIDES=${MOD_OVERRIDES_ONE}

And in your bootstrap script you need to use:
#!/usr/bin/env bash
export LEVELDATA_OVERRIDES_ONE=$(< leveldataoverride.lua)
export MOD_OVERRIDES_ONE=$(< modoverrides.lua)
docker-compose up -d

Untested, but I hope you get the basic idea. You have to define separate variables for every file and use them inside the docker-compose.yml file and pass them to the actual environment setting. You could also use env_file instead of using environment directly.

@thasmo Docker-compose does not support this approach。
I wrote a script that helped me complete the setup：
#!/usr/bin/env bash

# Configure the world resource configuration file. Parameters: target file, world configuration file
handleWorldResources()
{
	filterLineBreaks $2
        overrides="LEVELDATA_OVERRIDES=$filerText"
        grep -q "^LEVELDATA_OVERRIDES=.*" $1 &&
        {
                # If the resource configuration already exists
                replaceText $1 'LEVELDATA_OVERRIDES=' "$overrides"
        } ||
        {
                addText $1 "$overrides"
        }

}

# Configure the mods configuration file. Parameters: target file, mods configuration file
handleModsConfig()
{
	filterLineBreaks $2
	overrides="MODS_OVERRIDES=$filerText"
	grep -q "^MODS_OVERRIDES=.*" $1 &&
        {
                  # If the mods configuration already exists
                replaceText $1 'MODS_OVERRIDES=' "$overrides"
        } ||
        {
                addText $1 "$overrides"
        }

}

# Handle mods. Parameters: target file, mods configuration file
handleMods()
{
	mods=$(cat "$2" >> /dev/null && perl -lane "print $& if /\d{9}/" < "$2")
	mods="MODS=$(echo $mods | sed "s/[ ][ ]*/,/g")"
	grep -q "^MODS=.*" $1 &&
	{
		  # If the mods configuration already exists
		replaceText $1 'MODS=' $mods
	} ||
	{
		addText $1 $mods
	}

	handleModsConfig $1 $2
}

# Remove the text Line breaks
filterLineBreaks()
{
	filerText=$(cat $1 | tr -d '\n')

}

# Replace the text contents. Parameters: target file, replaces the target, replacement
replaceText()
{
	sed -i "s/^$2.*/$3/g" $1
}

# Add content to the text. Parameters: the target file, add content
addText()
{
	echo "$2" >> $1
}

handleMods './env/mainworld.env' './mods/mainmods.lua'
handleMods './env/mainsuperworld.env' './mods/mainsupermods.lua'
handleMods './env/caveworld.env' './mods/cavemods.lua'
handleMods './env/cavesuperworld.env' './mods/cavesupermods.lua'

handleWorldResources './env/mainworld.env' './level/mainlevel.lua'
handleWorldResources './env/mainsuperworld.env' './level/mainsuperlevel.lua'
handleWorldResources './env/caveworld.env' './level/cavelevel.lua'
handleWorldResources './env/cavesuperworld.env' './level/cavesuperlevel.lua'

Glad you found a way! Nevertheless it should also work by substituting variables in the docker-compose.yml file.


Add Journaling to netbgp-bgp objects

Describe the solution you'd like
Seems like most objects in netbox (and some other plugins) have a Journal option along with the Changelog. This would be very useful in the BGP plugin. Use cases are to add context for non-standard route policies or add ticket numbers to prefix list changes etc. Hoping this is something you could inherit from netbox.
Describe alternatives you've considered
It's possible to document in confluence etc but it's perfect to have this directly accessible in the plugin itself.
thanks!

To implement it, HTML templates needs to be cleaned (remove tabs block as inherited) + use include(get_model_urls()) to inherit automatically from Changelog + jounal views in urls.py
@cruse1977 If ok, I'll do it

Thanks would be appreciated, am keen to look to expand on this plugin - the current plan is - final 4.0 release to address the api issue, then 4.1 release then circle back to some of the issues + PR's

Resolved in 0.13.3 and 0.14.0


Add function to retrieve nested project settings

I was running into an issue where we had tests nested within describe blocks and the reporter was unable to retrieve the project settings due to the hardcoded parent paths present.
Added a function that takes in the testCase object and recursively looks through it to retrieve the project name and browser, defaults to empty strings if it reaches the end of the parent chain was unable to find the _projectConfig property.

I will create a release shortly


[1.20.1] Matter values not being added to items no matter what I do

I've tried reloading the world and game, but it doesn't fix the issue. Even making a new world doesn't fix it, and running /reload does nothing.
Downgrading doesn't fix it, and reinstalling all 3 mods didn't work either.
Replication Version:
1.0.5
Titanium Version:
3.8.32
Also running v1.0.7 of Aequivaleo. Tested it with v0.1.132 and that didn't fix the issue.
Crashlog If Applicable (Upload to pastebin/gist):
https://gist.github.com/vizthex123/ffb662747358e8cf348613e566b87058

Update: Seems like it's delayed while trying to sync matter values, but it doesn't send the notification like it used to.
I'd also suggest adding a message in chat so it's more obvious when the sync is happening.

Fixed in 1.21


Generic properties in C# sample

Hello,
is there a way to generate C# sample with generic property which type won't be escaped? Please find below an example:

As you can see <> were removed from type definition despite being a valid class name. Similar issue will occur when type is described by full name (with namespace). Valid dots separating words in namespace will be removed. Is there any setting that allows for disabling escaping those characters?

We cannot support that generic syntax in JSON Schema


Update README.md

Two static images used in the README were using http://. Changed them to https://. Both images load fine. This prevents the insecure padlock sign when the package page is opened in npm.
https://www.npmjs.com/package/react-helmet

 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.


cancellation handler

motivation: allow clients of SwiftPM to cancel background activities
changes:

introduce new "cancellator" utility that is a registry for cancellation handlers
use the "cancellator" instead of ProcessSet to manage the cancellation of active processes (eg git, tests) and build system
add cancel method to GitRepositoryProvider protocol so that it can be interuppted
add cancel methods to RepositoryManager, RegistryDownloadManager and HTTPClient so that it can be interrupted
change workspace initializer to take a terminator so that CLI and other consumer of libSwiftPM can interrupt
register interruption points mentioned above in workspace
adjust related call sites, ie CLI signal handler now uses the workspace terminator to interrupt
add tests

rdar://64900054
rdar://63723896

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@abertelrud this is ready for review
@compnerd could you help make sure this builds / works on Windows

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

This seemed to have dropped the Windows fixes and has broken the windows build: https://ci-external.swift.org/job/oss-swift-windows-toolchain-x86_64-vs2019/1130/console


chore: "next" to "main" for 1.2.0 re-release

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

:tada: This PR is included in version 1.2.0-next.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


Go imports not updated after migrating influxdb to influxdata

This is an issue for those of us that don't already have the old, now removed, github.com/influxdb/influxdb repository already present in our $GOPATH/src.
Seems like all influxdb subpackage imports need to be updated. For example:
https://github.com/influxdata/influxdb/blob/master/client/influxdb.go#L16
github.com/influxdb/influxdb/models --> github.com/influxdata/influxdb/models
Or am I missing something? Thanks!

I ran into this today, deleted the old github.com/influxdb/influxdb src and did a go get github.com/influxdata/influxdb.  The client package is still importing the old github.com/influxdb packages.

Closing in favor of #5388. We were waiting for the 0.10.0 release to resolve the import path issue, so this should be fixed soon.


Fix InfluxDB retention policy usage in read adapter

Fixes https://github.com/prometheus/prometheus/issues/2779

:+1:

But this is still the case, as shown in #3402 ?


pull changes from releases/4.13.4 into master

@DennisDenuto this PR is more or less an issue or reminder for you so that you update master to latest release

@strehle thanks. we will update the release docs.


Cannot build on Ubuntu 16.04

I'm following all the steps in the docs, but building fails: https://cloud.mbed.com/docs/current/connecting/connecting.html
-- Generating done
-- Build files have been written to: /home/janjon01/mbed-cloud-client-example/__x86_x64_NativeLinux_mbedtls
Scanning dependencies of target platformCommon
[  1%] Building C object mbed-cloud-client-example/source/platform/CMakeFiles/platformCommon.dir/Linux/common_setup.c.obj
/home/janjon01/mbed-cloud-client-example/source/platform/Linux/common_setup.c:28:17: fatal error: pal.h: No such file or directory
compilation terminated.
mbed-cloud-client-example/source/platform/CMakeFiles/platformCommon.dir/build.make:62: recipe for target 'mbed-cloud-client-example/source/platform/CMakeFiles/platformCommon.dir/Linux/common_setup.c.obj' failed
make[3]: *** [mbed-cloud-client-example/source/platform/CMakeFiles/platformCommon.dir/Linux/common_setup.c.obj] Error 1
CMakeFiles/Makefile2:424: recipe for target 'mbed-cloud-client-example/source/platform/CMakeFiles/platformCommon.dir/all' failed
make[2]: *** [mbed-cloud-client-example/source/platform/CMakeFiles/platformCommon.dir/all] Error 2
CMakeFiles/Makefile2:83: recipe for target 'CMakeFiles/mbedCloudClientExample.elf.dir/rule' failed
make[1]: *** [CMakeFiles/mbedCloudClientExample.elf.dir/rule] Error 2
Makefile:162: recipe for target 'mbedCloudClientExample.elf' failed
make: *** [mbedCloudClientExample.elf] Error 2
2018-05-16 08:32:29,411 - pal-platform - ERROR - ** build failed **
2018-05-16 08:32:29,411 - pal-platform - INFO -
Completed fullBuild running for target = x86_x64_NativeLinux_mbedtls
With toolchain = GCC.
Output directory: /home/janjon01/mbed-cloud-client-example/out

ARM Internal Ref: IOTCLT-2726

@teetak01 How would I deploy this? It's not in the instructions...

Instructions suggest to use mbed import, which performs mbed deploy automatically.
Alternatively, if you clone the application via git clone, then you need to run mbed deploy seperately.

@teetak01 Interesting, I missed the mbed import part. but this is also pulling in mbed OS and all the dependencies like storage-selector. I don't like that.

I do not see that as a real problem. Pure linux-application does not include those libs at all. This reference application supports both Linux and Mbed OS out-of-box so some compromise needs to be done. We really would not want to write yet-another tool to deploy components seperately for Linux.

Closing this issue as resolved.


[DOC] Documentation for new reranking feature

What do you want to do?

[ ] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
2.12 introduces a new feature for reranking. We need to add docs for this.
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

Main RFC
Original Issue
Model Support PR
Feature Intake Issue

@HenryL27 Can you please add some info on re-ranker feature:

@HenryL27 thank you for tracking this - Could you also help outline how best to contribute when adding support for other models? Eg in my case I would like to add Cohere reranking with Opensearch

Creating a rerank pipeline:
PUT /_search/pipeline/rerank_pipeline
{
  "response_processors": [
    {
      "rerank": {
        "ml_opensearch": {
          "model_id": id of TEXT_SIMILARITY model [required]
        },
        "context": {
          "document_fields": [ "title", "text_representation", ...]
        }
      }
    }
  ]
}

The rerank pipeline has two parameters: the context object and the rerank-type object.
The rerank-type object (keyed by name; here it's "ml-opensearch") provides the rerank processor static information needed across all reranking calls. For example, the id of the reranking model in ml-commons.
ml_opensearch rerank type parameters

field name
required?
description

model_id
required
unique id of a TEXT_SIMILARITY model (deployed via ml-commons)

Requires context.document_fields
The context object provides the rerank processor information necessary for generating reranking context at query time. For instance, "document_fields" specifies where to look in each search result for context to pass to the reranking model.
context parameters

field name
dependent rerank types
description

document_fields
ml_opensearch
nonempty list of document fields to rerank over

Searching with a rerank pipeline:
POST /_search?search_pipeline=rerank_pipeline
{
  "query": {
    "match": {
      "text_representation": "Where is Albuquerque?"
     }
  },
  "ext": {
    "rerank": {
      "query_context": {
        "query_text": "Where is Albuquerque?"
      }
    }
  }
}

Reranking queries are pretty similar to ordinary queries, except that they have this additional "ext.rerank" section.
The query_context object must have exactly one of two parameters (mutually exclusive):
ext.rerank.query_context params

field name
description

query_text
the (natural language) text of the question you want to rerank over

query_text_path
the json path to the text of the question you want to rerank over

when specifying query_text_path, use the fully specified path. For example, for the above query, you'd set
query_text_path = query.match.text_representation.query.

@tianjing-li model supporting is handled by the ml-commons plugin. If you have a model you'd like to add support for, please raise an issue and contribute it there. As I understand, this PR may have added support for the Cohere reranker already?

Hi @HenryL27 , thanks for submitting the doc issue. Can you please submit a documentation PR with this info? We need to have a PR up in order for it to be considered for the 2.12 release. Thanks!

@HenryL27 thank you!


adding more test to os_eventq_poll

It would be better to derive time quantities from OS_TICKS_PER_SEC when calling os_time_delay(), or other os functions which take the number of ticks as a parameter. At the moment the ticks per second is 1000 for simulator, but there are reasons why we might want to adjust this.
If it gets adjusted, all the time delays in the code will change. I.e. wait of half a second might turn into a wait of 5 seconds. And that kind of a wait will not be acceptable within unit tests.
That's something to keep in mind for future.


Change Survey Seeder file #9

Change Survey Seeder file #9

did this changed came from production environment or client request


feat: use new guzzle code sample

The pecl 1 and 2 extensions are not what people are using in PHP world, and tbh I've never seen them in the wild in over a decade writing PHP. Apparently they're onto version 3 and 4 now wanyway.
Just look at any "best HTTP clients" article and you'll see no mention of the pecl extensions.

https://www.twilio.com/blog/5-ways-to-make-http-requests-in-php
https://ourcodeworld.com/articles/read/674/top-7-best-php-http-client-libraries

Let's use Guzzle, and if any more of these popular HTTP clients pop up on httpsnippet we can add those too.

It's loading the curl example because the httpsnippet library has not released a new version since September 2021.

Let's see if they reply to my latest offer of help. https://github.com/Kong/httpsnippet/issues/185.  otherwise we'll have to update our fork and use that (which is bloody ironic as I was asking we get rid of more of our forks). https://github.com/stoplightio/httpsnippet


how do you get mypy working with paramspec... I cannot

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the Asyncer documentation, with the integrated search.
[X] I already searched in Google "How to X in Asyncer" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to Asyncer but to AnyIO.
[X] I already checked if it is not related to Asyncer but to Trio.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
n/a

Description
Hi tiangolo!
Looks like a nice lib. I have a lib called asyncify (https://github.com/dynamic-graphics-inc/dgpy-libs/tree/master/libs/asyncify) pip install asyncify that has an asyncify function. I cannot get my asyncify function to work with mypy and paramspec. do you have any suggestions.
Operating System
Linux, Windows, macOS
Operating System Details
No response
asyncer Version
n/a
Python Version
36+
Additional Context
This is a random question and I thought you might be interested. DM me if you are interested in taking over the asyncify name as it is imo a better name.

Thanks! You need the latest version of mypy, ParamSpec support is very recent, in 0.930.

@tiangolo thanks. I was using 920 :/


Add VAR parameter to requested Publisher  OnBeforeGetJobConsumptionValueEntry(JobJournalLine : Record "Job Journal Line"; VAR IsHandled : Boolean) in Codeunit 1012 Job Jnl.-Post Line

regarding this issue I requested https://github.com/microsoft/ALAppExtensions/issues/10395#issue-776390409.
We forgot a VAR parameter.
Existing definition:
OnBeforeGetJobConsumptionValueEntry(JobJournalLine : Record "Job Journal Line"; VAR IsHandled : Boolean)
Requested definition:
OnBeforeGetJobConsumptionValueEntry(VAR ValueEntry : Record "Value Entry"; JobJournalLine : Record "Job Journal Line"; VAR IsHandled : Boolean)
Code
LOCAL GetJobConsumptionValueEntry(VAR ValueEntry : Record "Value Entry";JobJournalLine : Record "Job Journal Line") : Boolean
>>> New Code
IsHandled := FALSE;
OnBeforeGetJobConsumptionValueEntry(ValueEntry, JobJournalLine, IsHandled);
IF IsHandled THEN
  EXIT(TRUE);

<<< New Code End
WITH JobJournalLine DO BEGIN
  ValueEntry.SETCURRENTKEY("Job No.","Job Task No.","Document No.");
  ValueEntry.SETRANGE("Item No.","No.");
  ValueEntry.SETRANGE("Job No.","Job No.");
  ValueEntry.SETRANGE("Job Task No.","Job Task No.");
  ValueEntry.SETRANGE("Document No.","Document No.");
  ValueEntry.SETRANGE("Item Ledger Entry Type",ValueEntry."Item Ledger Entry Type"::"Negative Adjmt.");
  ValueEntry.SETRANGE("Job Ledger Entry No.",0);
  OnGetJobConsumptionValueEntryFilter(ValueEntry,JobJnlLine);
END;

EXIT(ValueEntry.FINDSET);

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


This package doesn't work with a blank new project for RN 59.10

Steps to reproduce:
https://github.com/manishjain-hrbl/pushnotificationtest2
Run npx react-native init pushnotificationtest2 --version 0.59.10
Open above folder in code editor:
Add below to package.json
"@react-native-community/push-notification-ios": "1.0.2",
"react-native-push-notification": "3.1.3"

(Above step also tried with push 3.1.9)
Run npm i
Run react-native link
(I created a blank Podfile just in case but that didn't matter)
Modify AppDelegate.m as per the documentation
Run react-native run-ios
(I have also tried clean from xcode>clean and iOS/build delete)
Errors:
Linking pushnotificationtest2
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRNCPushNotificationIOS.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTBlob.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTAnimation.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTActionSheet.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTGeolocation.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTImage.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTLinking.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTNetwork.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTSettings.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTText.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTVibration.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTWebSocket.a'
▸ Linking pushnotificationtest2
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRNCPushNotificationIOS.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTBlob.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTAnimation.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTActionSheet.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTGeolocation.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTImage.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTLinking.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTNetwork.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTSettings.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTText.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTVibration.a'
❌  clang: error: no such file or directory: '/Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Products/Debug-iphonesimulator/libRCTWebSocket.a'
error Failed to build iOS project. We ran "xcodebuild" command but it exited with error code 65. To debug build logs further, consider building your app with Xcode.app, by opening pushnotificationtest2.xcworkspace
** BUILD FAILED **

Old errors that I managed to fix from XCode:
The following build commands failed:
Ld /Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Intermediates.noindex/pushnotificationtest2.build/Debug-iphonesimulator/pushnotificationtest2.build/Objects-normal/x86_64/Binary/pushnotificationtest2 normal x86_64
Ld /Users/manishjain/pushnotificationtest2/ios/build/pushnotificationtest2/Build/Intermediates.noindex/pushnotificationtest2.build/Debug-iphonesimulator/pushnotificationtest2.build/Objects-normal/i386/Binary/pushnotificationtest2 normal i386
(2 failures)
. Run CLI with --verbose flag for more details.
Linking pushnotificationtest2
❌  Undefined symbols for architecture x86_64

Symbol: OBJC_CLASS$_RCTPushNotificationManager
Referenced from: objc-class-ref in AppDelegate.o

❌  ld: symbol(s) not found for architecture x86_64
❌  clang: error: linker command failed with exit code 1 (use -v to see invocation)
error Failed to build iOS project. We ran "xcodebuild" command but it exited with error code 65. To debug build logs further, consider building your app with Xcode.app, by opening pushnotificationtest2.xcodeproj
--Confirmed that package works with latest version 0.61.5
To reproduce the issue, download repo: https://github.com/manishjain-hrbl/pushnotificationtest2

So, I was getting this error because I had both old and new PushNotification xcodeproj in my Xcode project. I removed the old one. Added new one again. FYI, I was on Legacy build system.
Changes checked in: https://github.com/manishjain-hrbl/pushnotificationtest2

Closing this issue for triaging purpose.


Analyzer references are dropped

<Analyzer> nodes within an itemgroup should be maintained.

Not an issue, these are included by default as they are nuget packages


x86 decompiler and emulator are missing test registers support

As said in the title gidhra doesn't handle test registers (https://en.wikipedia.org/wiki/Test_register and https://retrocomputing.stackexchange.com/questions/21963/how-were-the-test-registers-used-on-the-i386-and-the-i486 )

Actually the decompiler used to support  test registers but they were commented out here https://github.com/NationalSecurityAgency/ghidra/blob/master/Ghidra/Processors/x86/data/languages/ia.sinc#L3164


Add NMF initialization to API

Solved by https://github.com/alam-shahul/SpiceMixPlus/commit/c9719d4f1f0f590b6b62998e616ff9a32cd64f37


Add option to show content in default language as fallback

Short description
This PR adds an option to show the content for POIs and events in the default language if there is no translation.
Proposed changes

I added the checkbox to the Region model and form
I added the logic to the API

Resolved issues
Fixes: #1001

This push should fix the issues that you listed, but I'm not confident, that how I implemented it, is the best way. Maybe you have an idea for a more simple way to solve the issues.


[MAINT] switch bidsschematools to pyproject.toml

Use pyproject.toml for config and package metadata.

I also think it's okay to keep using setuptools, if that's easiest.

I also think it's okay to keep using setuptools, if that's easiest.

tried and it was a bit easier

Thanks https://github.com/bids-standard/bids-specification/pull/1948/commits/bc83dcbddd9b4d5a74074f6d24ebb7459da4a053 @effigies
would have taken me a while to find it

This is good to go for me


Getting "Couldn't establish language server connection" when allowExperimental is enabled

Description:
$title
output of Ballerina log
[Info  - 2:19:11 PM] Connection to server got closed. Server will restart.
Couldn't establish language server connection.
ballerina: unknown option: '--experimental'

This happens when we have set
"ballerina.allowExperimental": true

with Swan Lake Alpha distribution.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

@dulajdilshan Is it for the latest SDK version or 1.2.x branch?

@dulajdilshan Is it for the latest SDK version or 1.2.x branch?

It was with the latest SL Alpha.


Route updates don't work in when two routes have same value for one of the route fields

Describe the bug
When we change the routes (e.g. delete some of the routes) from the route table CR, the CR status goes to out of sync for the remaining routes.
Steps to reproduce
This happens in specific case where route comparison logic does not work. E.g. when we have two routes with say, same vpcPeeringConnectionID.

Create following route table CR. Note that there are 2 routes with different destinationCIDRBlock , but same vpcPeeringConnectionID.

apiVersion: ec2.services.k8s.aws/v1alpha1
kind: RouteTable
metadata:
  name: rt1
spec:
  routes:
  - destinationCIDRBlock: <IP_ADDRESS>/16
    gatewayID: igw-<xxx>
  - destinationCIDRBlock: <IP_ADDRESS>/16
    vpcPeeringConnectionID: pcx-05f31992c7ca70be5
  - destinationCIDRBlock: <IP_ADDRESS>/16
     vpcPeeringConnectionID: pcx-05f31992c7ca70be5
  tags:
  - key: Name
    value: abc
  vpcID: vpc-<xxxx>

All 3 routes would get created correctly in aws and CR status will be sync
Delete route <IP_ADDRESS>/16 (without changing two routes with same vpcPeeringConnectionID ). This triggers update and controller tries to find what needs to be deleted/added.
CR goes to out of sync state with error something like

RouteAlreadyExists: The route identified by <IP_ADDRESS>/16 already exists.\n\tstatus
      code: 400, request id: a471eeab-5aa3-43c7-a146-ceb380c6b50f

This suggests that update tried to create route <IP_ADDRESS>/16 and failed with RouteAlreadyExists error.
On prima facie, there appears to be two issues

getMatchingRoute logic has issues when there is one field (vpcPeeringConnectionID) having same value for two routes. Due to this same value, it thinks, the route is getting updated from <IP_ADDRESS>/16 to <IP_ADDRESS>/16. So, it tries add <IP_ADDRESS>/16 and fails with the error.

Ideally, this logic should not have determined any update for these two routes with vpcPeeringConnectionID.

When update determines that there is a route to be deleted and added, it creates toAdd and toDelete lists.
Ideally, delete should happen first and then add should happen later. But current logic is reverse.

Expected outcome
CR should not go into out of sync state.
Environment

Kubernetes version
Using EKS (yes/no), if so version?
AWS service targeted (S3, RDS, etc.)

In fact, this happens when any kind of update is performed on the CR.
e.g. If we add a new tag (keeping routes section untouched), that also triggers the update. and getMatchingRoute logic fails to understand routes have not been modified. Result is that CR goes out of sync with the above mentioned error

@nnbu I guess this implementation is not correct https://github.com/aws-controllers-k8s/ec2-controller/blob/main/pkg/resource/route_table/sdk.go#L613


Whitespace in brace not preserved in import

Description
When importing get from lodash using object deconstruction and es6 imports js-beautify will strip whitespace from the beginning of the braces.
Input
The code looked like this before beautification:
import { find } from 'lodash';
import { get } from 'lodash';

Expected Output
The code should have looked like this after beautification:
import { find } from 'lodash';
import { get } from 'lodash';

Actual Output
The  code actually looked like this after beautification:
import { find } from 'lodash';
import {get } from 'lodash';

Steps to Reproduce
Environment
OS: MacOS - VS Code
Settings
Example:
{
    "indent_size": 2,
    "indent_char": " ",
    "eol": "auto",
    "preserve_newlines": true,
    "break_chained_methods": false,
    "max_preserve_newlines": 0,
    "space_in_paren": false,
    "space_in_empty_paren": false,
    "jslint_happy": false,
    "space_after_anon_function": false,
    "keep_array_indentation": false,
    "space_before_conditional": true,
    "unescape_strings": false,
    "wrap_line_length": 0,
    "e4x": false,
    "end_with_newline": true,
    "comma_first": false,
    "brace_style": "collapse-preserve-inline"
}

Just spotted #888
Closing this issues as it is a duplicate


Update X Lossless Decoder to v20141129a

Thank you for the contribution. It was merged directly as b606670df05421142287845e776db9c519ab3e5c to keep commit history cleaner. Your contribution is still credited to you.


When following someone, import their older posts into the activity stream

Currently, when you follow someone, you have to wait until they actually post something until you see stuff in your stream. When you follow a new person, Pump.io should import historical posts into your activity stream, so you can immediately see their stuff. This is especially nice on new installations, where the stream is essentially empty.

Consider only fetching the recent X number of posts, not the full history, to avoid hammering the original server. As an example in the wild, this is what Mastodon does.

Duplicated by: https://framagit.org/compa/compa/issues/5

Currently pump.io has pagination so, no get all user history in one shot.

This issue is not related to the web UI, but to what the server does when you follow a remote user.

Ah, you mean if you just get the stream from a remote server, it will automatically deliver only one page of notes? Yeah, that requirement resolves itself then. :-)


SA1500 (Curly brackets for multi-line statements must not share a line) wrongly raised for extension method on a collection with a collection initializer

The following code gives SA1500 with the analyzer but not with legacy SC:
public void Fish()
{
    var someList = new List<int>
                   {
                       1,
                       2,
                       3
                   }.Select(x => x.ToString());
}

Grabbing this.


Minor hedgedoc / minutes-import disagreement on ```language

Describe the issue
On a second look at today's CBOR minutes I saw that the hedgedoc import was not 100% correct:
"Source code" in hedgedoc:

```cddl
envelope-content =
    leaf /
    elided /
    node /
    assertion /
    wrapped

leaf = #6.201(any)
```

Rendition in hedgedoc's view mode: pre-formatted, just the content (the cddl wasn't interpreted to really good syntax highlighting, but it was more for the benefit of the readers anyway)
Rendition in the imported notes: There is an extra line after the pre-formatted text:
{: .language-cddl}
This is a really small issue, might just as well be fine to just accept it until that markdown part is next updated for other reasons – thanks for already making that import super smooth!
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

One problem is that ``` is not standard markdown.
So the processor might be interpreting this as a code span (foo), not a code block (which would be
foo

).

The actual imported file looks like:
## Does a generic "none of the above" tag make sense?   {#does-a-generic-none-of-the-above-tag-make-sense}

dcbor asks for tag, uses it in draft-mcnally-envelope to disambiguate

    envelope-content =
        leaf /
        elided /
        node /
        assertion /
        wrapped

    leaf = #6.201(any)
{: .language-cddl}

CA: Had similar pattern in earlier iterations of CoRAL – generic thing,


Bugs exposed by use of randomized Q

Original issue: https://charm.cs.illinois.edu/redmine/issues/259

Using randomized queues from branch ramv/randomized-msgq, autobuild suffers from crash/hang in several places. Some of them, that I was able to note, are listed here (occurs once in a while).

multisectiontest in charm++/megatest: crash
multi multisectiontest in charm++/megatest: crash/hang
multi migration in charm++/megatest: hang
templates in charm++/mgeatest: crash using ./charmrun ./pgm +p10 ++local
after Started first test after all tests are run in charm++/megatest: crash
charm++/delegation/multicast: hang
charm++/sdag/migration: hang
all-at-once] in converse/megacon: crash
megaampi

I suspect there are more such issues, but given the enormity of task, I was not able to capture them all. The bugs may be in the example programs, or inside Charm.

Original date: 2013-08-05 22:26:34

Which build is producing these failures?

git pull --rebase
git checkout  remotes/origin/ramv/randomized-msgq
./build charm++ net-linux-x86_64 -j16 --suffix randq-debug -O0 -g

I get no megatest failures from that on intellect at +p4, +p8, +p10
Is this perhaps an -O2 only issue?

Original date: 2013-08-07 19:23:40

Eric, per Ram's commit message on that branch, you need to build with --enable-randomized-msgq. This apparently requires a priotype other than bitvec as well:

build: --enable-randomized-msgq should now aid debugging / correctness
This commit introduces support for randomizing the scheduler queue.
Message priorities will not be respected when picking the next charm message for execution
This is intended to facilitate application debugging and should help detect:

applications / protocol that erroneously assume a message ordering between any sender / receiver pair
applications which erroneously depend on priorities for correctness
race conditions in the application

Requirements:

Currently, a charm build which has the STL-based msg queue enabled.
This requires that charm be built --with-prio-type != bitvec
Supported prio types can be any datatype that can be used as a template parameter

Original date: 2013-08-07 20:00:00

A trick for the people working on the megatest failures: it comes with a command line flag -repeat that makes multiple runs very easy, since it will keep going until the process aborts or gets killed.

Original date: 2013-08-07 22:12:25

multisectiontest had a race condition in its setup which implicitly assumed an ordering between two reductions.  It now (commit 165d1f63ea6b3e4ad9c881910dac68554d3785e0) properly waits for both to complete before triggering that actual test once both parts of its setup process are complete.
Megatest is now crashing for me (after many runs) in varraystest, which must be a different bug.

Original date: 2013-08-08 19:03:08

The various var tests all have a logical flaw of assuming equivalence in floating point numbers.  This can create occasional false positive errors from round off.  Not really related to randq per se, but after fixing these comparisons I have megatest running thousands of tests w/repeat in the randomizeQ build without fault.  Megatest with -repeat does have a memory leak (probably many), which we should address at some point.

Original date: 2013-08-13 01:26:09

Can someone give the exact steps for building/running with the randomized queue?
It seems this is not sufficient:
./build  charm++ net-linux-x86_64 --with-prio-type=int --enable-randomized-msgq -j12 -O3 -g

Original date: 2013-08-28 19:02:56

Please retest after recent fixes, and open distinct child issues for distinct detected problems.

Original date: 2015-02-11 22:51:01

I am re-running tests, and breaking out individual bugs for failing tests. This branch has been merged into charm, so the process I'm using for most of the tests is the following:
checkout the latest charm branch
./build charm++ net-linux-x86_64 --with-prio-type=int --enable-randomized-msgq -j16 --suffix randq-debug -O3 -g
export TESTOPTS="++local" (For some tests I also added a +p option)
I'm then running each of the tests in the tests directory a few dozen times on panache. Below are tests that either crash or hang. A child bug will be made for each one. There may be a bug in randomized queues, or it could just be the fact that the test is not written to take randomized queues into account. I ignored test directories without test targets (ie: io, jacobi), as well as tests not included in the master Makefile (ie: hello_crosscoruption, ping). I also verified that all of these tests pass when not using randomized queues.
charm++/alignment: Crashes, failed assertion.
charm++/communication_overhead: Hangs
charm++/delegation/multicast: Hangs
charm++/queue: Hangs (despite being aware of randomized queueing via an #if)
charm++/sdag/migration: Hangs
All converse tests pass.
./build AMPI net-linux-x86_64 --with-prio-type=int --enable-randomized-msgq -j16 --suffix randq-debug -O3 -g
ampi/megampi: Crashes rarely due to a failed assertion: Broadcast integer from master> expected 123, got 4!


python cache files are included in the repository

python cache files are included in the repository, this will cause issues with people updating.

python cache files are included in the repository, this will cause issues with people updating.

thank you I fixed it

i think you missed https://github.com/oztrkoguz/ComfyUI_Kosmos2_BBox_Cutter/tree/main/nodes lol


Disappear metric on line chart, again

ref: https://github.com/chainer/chainerui/pull/233

fixed by #273


Fetch remote proofs, new classes for internal APIs

In working on this (WIP) PR, I pulled on a thread and started re-factoring all of the internal methods in IpfsIdentity class (well - moving things into new classes)

Ending this PR with having wired in all of the new modules that used to live in index.js


Open Source Node Explorer

We would like to collaborate with teams in order to bring about an open source node explorer for the Polkadot network.

I would like to work on this project along with my other teammates.
Background: Myself, Ankit is working in Blockchain ecosystem from last 2.5 years. I had worked with Red Hat on Distributed System (GlusterFS) and on RPC protocol in GSoC. Also, I was the grantee from Eth Foundation in wave1 to work on solidity.  Previously we have created some node explorer for quorum project.
We would love to work on this.
If you suggest we will share the few mock ups and design for node explorer for it in few days and start the discussion on riot.

That sounds awesome! :-) We can try to help you coordinate with the right people. Obviously you can find the devs in the usual watercooler chat, otherwise you can also try in Substrate Technical.
I'd say dive right in! Feel free to drop thoughts here, or you can mail them over.

Thanks @EdwardAThomson  I have been active on riot. Please , put me touch with right person.

Cool. Let me reach out to someone at Parity to find out who the best person is for you to speak to. If you don't hear anything from Parity by end of next week then drop me a message here or on riot (@edwardt:matrix.org).

@EdwardAThomson please explain what sets a 'node explorer' apart from a 'block explorer'. I find that I am having difficult making a hard distinctions and explain this to ppl. It may be a spectrum and a bit blurred... Appreciate the input!!

Something along the lines of https://www.ethernodes.org/network/1
Client types, client versions, locations and any other info. This will require some fancier node crawling to figure out whats going on.
A node explorer is looking at the network, while block explorers are looking at the data stored within the blocks of the chain itself.

This issue is a candidate for our grants program. For details on how to apply, see your grants page: https://grants.web3.foundation


daed coin?

Are we still working on it?

Yes.

Cool. I bullieve on you guys. I’ll buy back my kidney after we moon.


David-Jordan/Network-Connectivity

Closes #1322
thanks @NodeGuy for the help with the tests

what is this PR about ?

as part of an effort to communicate trust and security.
when we implement https://github.com/cosmos/voyager/issues/1222 we can show this info in the tooltip on the lock.

When would this show an open lock?

So the icon has no purpose right now? As with code, should we maybe avoid implementing things we don't need yet?

I don't have the feeling that this symbol shows a "connection" maybe a "secure connection". But I don't feel strongly about it.


@Aggregates({EventSource.SAFEPOINT}) will fall into dead loop

When I create an Aggregator with @Aggregates({EventSource.SAFEPOINT}). The program will loop forever.
I read the source code and I found that there is no LogFileParser with the outbox SAFEPOINT .
Therefore, when the program execute the code aggregatorVerticles.forEach(AggregatorVerticle::awaitCompletion); , the Aggregator with @Aggregates({EventSource.SAFEPOINT}) will not complete forever.

Is GCToolkitVertxParametersForPreUnifiedLogs.java line 140 AggregatorVerticle aggregatorVerticle = new AggregatorVerticle(SAFEPOINT_OUTBOX); change to AggregatorVerticle aggregatorVerticle = new AggregatorVerticle(GCToolkitVertx.JVM_EVENT_PARSER_OUTBOX);?
Because there is no AggregatorVerticle with the inbox JVM_EVENT.
However, I think we should add an EventSource enum value of JVM_EVENT will be better than the before one.

Nice find!
I think the issue here is that the SafePointParser is never created. I think part of the fix is to add this block of code to the initLogFileParsers method of GCToolkitVertxParametersForPreUnifiedLogs.
        if (diary.isApplicationStoppedTime() || diary.isApplicationRunningTime()) {
            logFileParsers.add(new LogFileParser(GCToolkitVertx.PARSER_INBOX, SAFEPOINT_OUTBOX, consumer -> new SafepointParser(diary, consumer)));
        }

The if (diary.isApplicationStoppedTime() || diary.isApplicationRunningTime()) test is probably needed around the code that calls getAggregators(EventSource.SAFEPOINT, registeredAggregations) down in initAggregatorVerticles in the same class.
It would be awesome, @ZhangShushu123, if you could work this issue!

By the way, I think adding code to remove aggregatorVerticles which inbox can not match with the logFileParser in aggregateDataSource() is a better way, because it will prevent the dead loop. Moreover, logging the removed aggregatorVerticle  in console with error level will tell us which aggregatorVerticle can not match and we should fix in the future.

The code which should be added in aggregateDataSource() will show below. Of course, the code can be written more gracefully.
 aggregatorVerticles.removeIf(aggregatorVerticle -> {
      boolean isMatch = logFileParsers.stream().anyMatch(logFileParser -> logFileParser.getOutbox() == aggregatorVerticle.getInbox());
      if(!isMatch){
            LOGGER.log(Level.SEVERE, String.format("%s %s",aggregatorVerticle.getInbox(),aggregatorVerticle));
      }
      return !isMatch;
});

@ZhangShushu123 - would you be willing to create a PR for these changes?

Of course. I will take a pull request as soon as possible.

Thank you very much for taking the time to merge code. And I look forward to your next refactoring work.


Integration of all components [WORKING!]

Integrating main components! Currently generating villages. Not fully aesthetically pleasing but definitely a settlement. 🚀
Fixes some bugs and adds functionality

This PR has stuff from the live session today aswell! all is testing and working!


type = "h" (and possibly others) don't work

Also fixed for types "c", "s" and "S".:
Example behaviour after https://github.com/grantmcdermott/plot2/commit/40a1aeec4040391d799f647e24da241ceeb75ebf
library(plot2)

plot2(0:10, type = "h")

plot2(mpg ~ wt | cyl, mtcars, type = "h")

plot2(Temp ~ Day | Month, airquality, type = "s")

One thing we might need to think about, however, is controlled jittering for grouped "h" plots. (You could say the same thing for other plot types with overlap.)

plot2(Temp ~ Day | Month, airquality, type = "h")


Under linux index out of bounds: the len is 56520 but the index is 57004

With the new version 0.8.6  i get following error with the linux precompiled build.
thread 'main' panicked at /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/theframework-0.1.18/src/thedraw2d.rs:656:40:
index out of bounds: the len is 56520 but the index is 57004
stack backtrace:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Hi, I cannot recreate the error, when does it happen, directly after startup or later on ?


"Scărișoara" is listed in the drop-down menu

Description:
"Scărișoara" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Scărișoara" in the search bar.

Expected results
The location is listed in the drop-down menu as "Scărișoara, Alba (Scărișoara)".


Comparator definition does not imply total ordering

In the comparable documentation (https://github.com/dart-lang/sdk/blob/master/sdk/lib/core/comparable.dart) it says:

A comparison function represents an ordering on a type of objects. A total ordering on a type means that for two values, either they are equal or one is greater than the other (and the latter must then be smaller than the former).
A Comparator function represents such a total ordering by returning

a negative integer if a is smaller than b,
zero if a is equal to b, and
a positive integer if a is greater than b.

This is insufficient to define a total ordering since transitivity must also be satisfied (a > b and b > c implies a > c).

Correct. We do say that it's an "ordering", but do not say what that means (transitive, anti-symmetric, usually anti-reflexive).
It's then a total ordering if it's an ordering and all pairs of elements are related.


Publish to AppSource Library App

Im trying to publish to appsource a App that haves a dependency from other repo. It fails saying cannot find the dependency app.

In the /.AL-Go/settings.json i set the property "generateDependencyArtifact": true

But it seems only the main app get publish!?!
Dependency to projects '*' in https://github.com/xxxxxxx/yyyy@main, version latest, release status release
Azure.Storage Module is available in version 2.1.0
Using Azure.Storage version 2.1.0
Attempting authentication to *** using clientCredentials...
Authenticated as app ***
AppSource MainAppFolder .\MainApp
Main App File:
-MainApp_<IP_ADDRESS>.app
Library App Files:

None

The whole dependency handling when publishing to AppSource is not super great yet. We are currently publishing all apps to AppSource.

If you set generateDependencyArtifact to true in .AL-Go\settings.json - then the build should generate a dependency artifact and that should also be delivered to AppSource (for the apps, where the publisher name is the same as the main app)
If this doesn't work, please let me know.

I did that, seems that identifies both artifacts but keeps showing
Library App Files:

None

Would it be possible for me to get access to the repo in order to have a look at this?

@freddydk Sorry was in holidays. I will give access to the repo.
I saw that when the app creates the request in the appsource removes the Library app from the live version to the draft version.

Is this the reason?
"Make sure that these dependencies are part of your submission or that they have already been uploaded to AppSource and that they are available for the targeted country/region and release"
Do we need to submit the library app before? Isnt possible to uploaded in the same request?

if you set generateDependencyArtifact to true in .AL-Go/settings.json - it should include dependencies in the submission - else you need to submit the library app another way or beforehand

@freddydk i do that, i can see them getting downloaded but the Library Apps shows "None"
Downloading artifact xxxxxxxxxxx-main-Apps-<IP_ADDRESS>
https://api.github.com/repos/SK-Consulting-S-A/xxxxxxxxxxx/actions/artifacts/922653111/zip
D:\a\xxxxxxxxxxx\xxxxxxxxxxx.artifacts\xxxxxxxxxxx-main-Apps-<IP_ADDRESS>.zip
Analyzing artifacts
https://api.github.com/repos/SK-Consulting-S-A/xxxxxxxxxxx/actions/artifacts?per_page=100&page=1
project 'xxxxxxxxxxx'
**Downloading artifact xxxxxxxxxxx-main-Dependencies-<IP_ADDRESS>
https://api.github.com/repos/SK-Consulting-S-A/xxxxxxxxxxx/actions/artifacts/922653116/zip
D:\a\xxxxxxxxxxx\xxxxxxxxxxx.artifacts\xxxxxxxxxxx-main-Dependencies-<IP_ADDRESS>.zip
Artifacts:

xxxxxxxxxxx-main-Apps-<IP_ADDRESS>
xxxxxxxxxxx-main-Dependencies-<IP_ADDRESS>**
Applying settings from D:\a\xxxxxxxxxxx\xxxxxxxxxxx.github\AL-Go-Settings.json
Applying settings from D:\a\xxxxxxxxxxx\xxxxxxxxxxx.AL-Go\settings.json
Analyzing repository
Checking project dependencies
Azure.Storage Module is available in version 2.1.0
Using Azure.Storage version 2.1.0
Attempting authentication to https://api.partner.microsoft.com/.default using clientCredentials...
Authenticated as app ***
AppSource MainAppFolder Fisqal - xxxxxxxxxxx
Main App File:
SK Consulting S.A._Fisqal - xxxxxxxxxxx_<IP_ADDRESS>.app
Library App Files:
None

Hi @freddydk Did you got opportunity to check the repo?
I look on the Delivery.ps1 action looks for some reason the counter of the libraries app, always gives 0 "Empty".

A fix for this is currently in preview.
Same as #1093 and #1110


feat: soc interface

Proof of concept for exposing the SOC interface on the Bee class as discussed in #71.
Currently I see it as a good tradeoffs between the alternatives, maybe with better names.

Implemented in https://github.com/ethersphere/bee-js/pull/147


Radium.getState - proposal for querying element state

It would be nice to be able to access Radium's internal knowledge about component interaction state. I propose the following simple function for controlled access to Radium's state, primarily after #101 is in:
Radium.getState(state, key, value)

// Example

Radium.getState(this.state, 'button', 'isHovering')

state - you'll usually pass this.state, but sometimes you may want to pass a previous state, like in shouldComponentUpdate, componentWillUpdate, and componentDidUpdate
key - if you used multiple elements, pass the same key="" or ref="". If you only have one element, you can leave it blank ('root' will be inferred)
value - one of the following: isHovering, isActive, isFocused - alternatively, we could change this to :hover, :active, and :focus if that makes more sense
returns true or false

This would primarily unlock #105, changing the style of one element when another has some interaction.

Love this. I think :hover, etc makes sense to keep it consistent with the naming used in style objects.

+1 This would also make testing components easier, to ensure that radium's knowledge of the component state is in sync with the components knowledge of its state.

Shouldn't this issue be closed out?


Unmet Peer Dependency for Angular 4.0.0

My packages are:
"@angular/animations": "^4.0.0",
 "@angular/common": "^4.0.0",
 "@angular/compiler": "^4.0.0",
 "@angular/core": "^4.0.0",
 "@angular/forms": "^4.0.0",
 "@angular/http": "^4.0.0",
 "@angular/platform-browser": "^4.0.0",
 "@angular/platform-browser-dynamic": "^4.0.0",
 "@angular/router": "^4.0.0",

When I install ng2-img-tools I get these unmet depedencies:

── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  ├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>    ├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>    ├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>

Hi @tasoskakour,
I have just fixed this in version 1.0.5.


Dependencies are added to the global scope

The code to require the dependencies looks like this
_.forEach(data._dependencies || {}, function(value, key) {
    if(global[key] !== undefined) {
        console.error('The variable \'' + key + '\' already exists.');
        return;
    }

    global[key] = require(value);
});

As you can see, they are attached to the global scope which might interfere with others. Any ideas on how the scope of the variables could be limited to the _seed function?

It is possible by using the vm package of NodeJS. https://nodejs.org/api/vm.html


Loader feature needed for better experience

Is your feature request related to a problem? Please describe.
Yes when there is an API call sometimes it may not get fetched hence a loader is needed for better experience
Describe the solution you'd like
Would like to add Loader for smooth user experience while fetching data from api
@ManishaSwain8 Can you assign me this issue

@MurtazaShikari this issue is already raised in issue #23 , I follow first come first try method ,eventually who have raised the issue first will be assigned , if it is not done in certain days will proceed with assigning the second person and so o.


Fix caching for bccm web

Seems like we are occasionally serving old versions of stuff.
We have Cache-Control: max-age=60
I've seen websites embedded in the app jump from old version -> new version -> old version. Which could mean that both the browser cache and the cdn is serving old versions (the second time).
I think the issue right now is that both the cdn and the browsers are allowed to use a stale (>60 second old) response.
We could probably solve this by putting cache-control: max-age=60, must-revalidate, as must-revalidate prevents browsers and cdns from using stale responses.

Updated the cache header on index.html


Annotation review gallery

Rapid annotation review gallery
Overview:
Feature demo: https://www.youtube.com/watch?v=1wvYV7i7OkQ
This workflow takes a girder folder contains annotated slides, including
special Region-of-Interest (ROI) annotations that enclose annotated regions.
It downloads visualizations of those ROIs and tiles them into a couple of
galleries to be reviewed rapidly by the pathologist. It also adds a link
that the pathologist can use to access the same ROI in the original slide
to pan around it. See the demo video above for details.
Where to look:
    |_histomicstk/
    |  |_annotations_and_masks/
    |     |_review_gallery.py
    |     |_tests/
    |        |_review_gallery_test.py
    |_ docs/
       |_examples/
          |_review_gallery.ipynb

@manthey All tests passed. Again coverage is artificially low but I have the tests.
This should be reviewed and merged after PR #816 . Thanks!

Will incorporate recent updates to master and make a new PR that's easier to review.


Update dependencies

This library is great and it would be great to keep it updated.
I updated Lodash and all the testing dependencies.
I also updated the code to use the new Lodash modules individually without including the entire library during runtime. You can make it even leaner disk-wise by installing the Lodash modules all separately (i.e. lodash.foreach, lodash.map, etc). If you want this I can do that for you.

Merged - thanks!
Yes, any more PRs you can raise will be appreciated 👍

Cool, thanks.

Actually I'm going to leave it as is. Are you going to release a new update to NPM?


[AVM Module Issue]: tflint tests are broken

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Bug
(Optional) Module Version
No response
(Optional) Correlation Id
No response
Description
The GitHub action was updated upstream and we are failing some tflint tests:
6 issue(s) found:

Warning: [Fixable] local.role_definition_resource_substring is declared but not used (terraform_unused_declarations)

  on locals.tf line 2:
   2:   role_definition_resource_substring = "/providers/Microsoft.Authorization/roleDefinitions"

Reference: https://github.com/terraform-linters/tflint-ruleset-terraform/blob/v0.5.0/docs/rules/terraform_unused_declarations.md

Error: The `source` property constraint should start with `Azure/` and contain `avm-` to only involve AVM Module (required_module_source_tffr1)

  on main.tf line 2:
   2:   source  = "Azure/regions/azurerm"

Reference: https://azure.github.io/Azure-Verified-Modules/specs/terraform/#id-tffr1---category-composition---cross-referencing-modules

Error: The `source` property constraint should start with `Azure/` and contain `avm-` to only involve AVM Module (required_module_source_tffr1)

  on main.tf line 271:
 271:   source  = "Azure/subnets/azurerm"

Reference: https://azure.github.io/Azure-Verified-Modules/specs/terraform/#id-tffr1---category-composition---cross-referencing-modules

Warning: [Fixable] variable "client_id" is declared but not used (terraform_unused_declarations)

  on variables.tf line 30:
  30: variable "client_id" {

Reference: https://github.com/terraform-linters/tflint-ruleset-terraform/blob/v0.5.0/docs/rules/terraform_unused_declarations.md

Warning: [Fixable] variable "client_secret" is declared but not used (terraform_unused_declarations)

  on variables.tf line 37:
  37: variable "client_secret" {

Reference: https://github.com/terraform-linters/tflint-ruleset-terraform/blob/v0.5.0/docs/rules/terraform_unused_declarations.md

Warning: [Fixable] variable "key_vault_secrets_provider_enabled" is declared but not used (terraform_unused_declarations)

  on variables.tf line 54:
  54: variable "key_vault_secrets_provider_enabled" {

Reference: https://github.com/terraform-linters/tflint-ruleset-terraform/blob/v0.5.0/docs/rules/terraform_unused_declarations.md

------------------------------------------------

The preceding files contain terraform blocks that does not complies with tflint requirements.

make: *** [avmmakefile:40: tflint] Error 1

Fixed in PR #75


linux: Disable bond0 creation by bonding driver

Motivation for this change
Linux defaults to creating a bond0 interface when modprobing the
bonding driver, because the max_bonds option defaults to 1¹.
This bond0 interface is created with the default bond mode
balance-rr, which is most often undesirable.
Networkd in turn is unable to change this mode, since it does not
implement netdev recreation on parameter mismatch².
If we instead default max_bonds=0 the driver will be loaded, but no
default interface will be created, leaving networkd in a better position
to set up the network configuration.
[1] https://www.kernel.org/doc/Documentation/networking/bonding.txt
[2] https://github.com/systemd/systemd/issues/9627
Alternative to #161238. I'm not sure how fond we are of carrying
patches that we very likely are not going to be able to upstream
but this is an annoying papercut in combination with systemd-networkd
and the original behaviour just isn't very useful.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofBorg test networking

At a minimum we should probably have a section on our kernel in the NixOS documentation. We could put there things like notable configuration choices and patches. Of course I'm not asking you to make a big documentation effort here: starting with the fact that we default to 0 bonds is sufficient, I think.


Thread local JNIEnv allocates, which is not allowed in a signal handler

This is a serious issue as I found when the signal handler trigger while in malloc resulting in the following stack trace:
#0  0x00000031f98f80ae in __lll_lock_wait_private () from /lib64/libc.so.6
#1  0x00000031f987d35b in _L_lock_9495 () from /lib64/libc.so.6
#2  0x00000031f987a946 in malloc () from /lib64/libc.so.6
#3  0x00000031f94110d0 in tls_get_addr_tail () from /lib64/ld-linux-x86-64.so.2
#4  0x00000031f94115a0 in __tls_get_addr () from /lib64/ld-linux-x86-64.so.2
#5  0x00007fed0ade8134 in CurrentJniEnv () at /root/git/honest-profiler/src/main/cpp/globals.h:162
#6  Profiler::handle (this=0x19ab8b0, signum=, info=, context=0x4400045fc400) at /root/git/honest-profiler/src/main/cpp/profiler.cpp:103
#7
#8  0x00000031f98e54f7 in mprotect () from /lib64/libc.so.6
#9  0x00000031f987a3ab in _int_malloc () from /lib64/libc.so.6
#10 0x00000031f987a951 in malloc () from /lib64/libc.so.6
This led to the JVM hanging in one run out of 10 or so randomly, depending on load on the box etc.
In the comments in globals.h we have:
  // This is very dangerous.  __thread is not async-safe when used in
  // a shared library, because it calls malloc the first time a given
  // thread accesses it.  This is unlikely to cause problems in
  // straightforward Java apps, but a real fix involves either a fix
  // to glibc or to the Java launcher, and casual users will have a
  // hard time with this.
  static __thread JNIEnv
  *
  env_;

This is from original J.Manson implementation.

Fixed up locally by following the method described here for gaining access to JNIEnv:
http://stackoverflow.com/questions/12900695/how-to-obtain-jni-interface-pointer-jnienv-for-asynchronous-calls
Tested the new build and it seems to work and not hang, joy for all! Will be sending a PR.

Thanks!

Fixed by PR #74


made Generic Prompt unselectable

Before

After

Maintainer merge checklist

[ ] Title is descriptive/clear for inclusion in release notes.
[ ] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[ ] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

@misl6 why is macos unit test failing ?

Was unrelated to your PR. I did a re-run, so we will have a green check-mark here quite soon :)


quotactl should be unsafe and some quotactl_* functions are unsound

The final argument to quotactl must conform to a strict specification depending on the command being run. quotactl does not perform any validation of the pointer argument passed to it, and so it should be unsafe to call. The caller must guarantee that the addr pointer follows the specification for quotactl.
Because many functions in src/sys/quota.rs call quotactl, they also need to be checked for soundness. A quick glance indicates that quotactl_on is incorrect because it passes an &P: NixPath in where it should be passing a C string (for example, using with_nix_path.

Right on the first point, but quotactl_on is using with_nix_path.  Could you look again?

Ah yep, my bad. I think I mixed up p and thought it was an &P. Sorry for the confusion!


dma: stm32: enable half transfer interrupt

Enable half transfer interrupt of the dma controller
in case the channel was enabled in circular mode.
Signed-off-by: Shlomi Vaknin<EMAIL_ADDRESS>I am not sure if this is the correct way to do this. Any opinions?
This mode can be quite useful when trying to continuously sampling
with adc controller.

@erwango

@erwango I am using half-transfer irq in #33846. I guess it will take a lot of time to this to be merged, but I it is
the best usage I could find for this. I hope it is enough.

@erwango Can you approve this?


Add check to ensure generics for sorts are correct

Closes (#820)

Checking works, but I'm not sure this error is particularly helpful...
Code:
use flux_rs::*;

#[refined_by(n: int)]
struct S {
    #[field(i32[n])]
    f: i32,
}

defs! {
    fn foo(s: S<int>) -> int {
        s.n
    }
}

Output:
error[E0999]: cannot instantiate base generic with opaque type or a type parameter of kind type
 --> ../sort_gen.rs:8:1
  |
8 | struct S {
  | ^^^^^^^^
-Ztrack-diagnostics: created at crates/flux-fhir-analysis/src/wf/sortck.rs:394:18

error: aborting due to 1 previous error

It would be nice if we could point to the flux def itself and say something like:
Argument int supplied but definition for S has no generics. Either remove the generic type supplied to sort S or add missing generic type to S

Example input:
#![allow(unused)]

use flux_rs::*;

#[refined_by(n: int)]
struct S {
    #[field(i32[n])]
    f: i32,
}

defs! {
    fn foo(s: S<int>) -> int {
        s.n
    }
}

Output:
error[E0999]: sorts associated with this struct should have no generic arguments but 1 generic argument supplied
  --> ../sorts.rs:12:15
   |
12 |     fn foo(s: S<int>) -> int {
   |               ^ expected no generic arguments on sort
-Ztrack-diagnostics: created at crates/flux-fhir-analysis/src/conv/mod.rs:2017:40

error: aborting due to 1 previous error

Another example:
#![allow(unused)]

use flux_rs::*;

#[refined_by(n: int)]
struct S {
    #[field(i32[n])]
    f: i32,
}

defs! {
    fn foo(x: int<bool>) -> int {
        1
    }
}

And output:
error[E0999]: primitive sort int expects no generics but found 1
  --> ../sorts.rs:16:15
   |
16 |     fn foo(x: int<bool>) -> int {
   |               ^^^ Incorrect generics on primitive sort
-Ztrack-diagnostics: created at crates/flux-fhir-analysis/src/conv/mod.rs:2043:17

error: aborting due to 1 previous error

@vrindisbacher I think you put the wrong #[diagnostic] attributes. Both the error structs for self and opaque have
#[diag(fhir_analysis_generics_on_type_parameter, code = E0999)]

doh! thanks... can't believe i didn't catch that...


Point action docs to 2022.8.7 not latest

Fixes a mention in the docs to @latest which doesn't actually refer to a valid tag, my bad for not spotting! 👍🏻 Thanks @henryiii

Small question: Is @main possible, instead of @latest? If not, I'm ok with merging this 👍

Ahh sorry, I didn't read #662 yet :/

@main is fine too, you can put any branch or tag name there.

Ok. I'm personally more comfortable with @main, but a tag is fine too (however, we'll have to update that tag frequently).

A specific tag is always preferable IMO, it means a user of the action is always getting an exact version rather than whatever is on the main branch (which may, although unlikely, break). What do we think?

If we need to keep it up to date with new versions there's things like bump2version that can find and replace, or we just leave it with a comment that says e.g "any tagged version will work here"

That would resolve my concerns of using a specific tag!


Paper copy edits

Yeaaah! So I just went through each section of the paper and cleaned up the spelling/grammar/style. I did also beef up the data section a little more, with a brief explanation on what BART studies are and what they supposedly measure. We have 8 pages, counting figures.

Read through additions that you made. Wasn't the limit for the paper 6 pages?

We talked to jarrod. the 6 pages isn't a hard limit. as long as our information is meaningful

Okay in that case, this is great! Thanks Jane

Coverage remained the same at 91.304% when pulling 25418d329386c775cce61549f52d9f23030c31cd on janewliang:paperreview into 14fe60cf1a1d39f5b836051cb0d163492d32783e on berkeley-stat159:master.


Upgraded webpack PostCSS plugin definition

Issue: #1061 No PostCSS config found
What I did
It seems like either the config is defined differently in Webpack 2 or we upgraded to a newer version of PostCSS. PostCSS can't read our current plugin definition so I upgraded it to the latest listed on the docs.
How to test

create a story, then create a css file for the story.
put some test styles in the css.
in the story, either import 'styles.css', or require('styles.css').

it should not throw an error during build and console log the style-loader contents.

First merge! 🍾


Fixes DynamicModelChoiceField empty placeholder

Closes: #3187
What's Changed
This fixes the behavior of the UI select box for DynamicModelChoiceField and DynamicModelMultipleChoiceField. With either of those fields, if an empty_label/null_option were specified, the resulting HTML/Javascript select box would not correctly select the empty option even if the object being edited had an empty/null value for the field.
This change corrects that behavior by converting the Python None value to the Javascript null value in the field's options.
Before:

After:

TODO

[X] Explanation of Change(s)
[X] Added change log fragment(s) (for more information see the documentation)
[ ] ~Attached Screenshots, Payload Example~
[X] Unit, Integration Tests
[ ] ~Documentation Updates (when adding/changing features)~
[ ] ~Example Plugin Updates (when adding/changing features)~
[ ] Outline Remaining Work, Constraints from Design

This looks good so far. I'm wondering why we use a null_option argument instead of leveraging some of the functionality of Django's empty_label argument? They appear to do the same thing unless I'm missing something.

This looks good so far. I'm wondering why we use a null_option argument instead of leveraging some of the functionality of Django's empty_label argument? They appear to do the same thing unless I'm missing something.

When I first started looking into this I wondered the same thing. I don't know the history behind the use of null_option and I certainly don't know what the impact would be in changing it.

The UX portion of this has been addressed in #3315. However the breaking change to no longer need the duplicate null_option and empty_label should be open as a separate issue and could revive a portion of this PR.


Pedantic: deny mutable type generalization

Search Terms
type generalization
type expanding
Suggestion
Currently we have a vulnerability in type safety of mutable variables. You can generalize the type of some variable and change its value by passing there something not compatible with the old type. Examples:
const a: Array<number> = [1, 2]

const b: Array<number | string> = a

b.push('a is not array of numbers anymore')

const a = { data: 0 }

function b (param: { data: string | number }) {
  param.data = 'a is broken now'
}

b(a)

So I suggest adding a new compiler option under 'pedantic' label which will deny such type conversions:
const a: Array<number> = [1, 2]

// Error: type 'Array<number>' is not assignable to 'Array<number | string>'
const b: Array<number | string> = a

const a: Array<number> = [1, 2]

// OK: 'ReadonlyArray' won't let us mutate the state of 'a'
const b: ReadonlyArray<number | string> = a

const a = { foo: 0, bar: 'string' }

function b (param: { readonly foo: number | string, bar: string }) {
  param.bar = String(param.foo)
}

// OK
// property 'foo: number' is assignable to 'readonly foo: number | string'
// property 'bar: string' is assignable to 'bar: string'
b(a)

function c (param: { readonly foo: number | string, bar: number | string }) {
  param.bar = String(param.foo)
}

// Error: property 'bar: string' is not assignable to 'bar: number | string'
c(a)

I'm pretty sure we have experimented with strictly applying variance checks for methods (and mutable properties), but lib.d.ts would no longer pass a type-check. @RyanCavanaugh might remember better.

The DOM wouldn't typechecking under strict method variance because of how the event listener methods on DOM objects are defined (flow types these as "Function" to get around this). This would really be the least of your problems though; there's a reason C++ has const as a type and member modifier -- without those primitives, every declaration file would really need two versions (if not more) of every type.

@RyanCavanaugh

The DOM wouldn't typecheck under strict method variance because of how the event listener methods on DOM objects are defined

Can you give an example of the code that will be broken after this change?


Cannot display session activity logs when username contains capital letters

Describe the bug
When username in AWS IAM Identity Center contains capital letters, the session activity logs aren't displayed.
To Reproduce
Steps to reproduce the behavior:

Create user in IAM Identity Center with name containing capital letters, for example "TEAMuser"
Activate session with TEAM
Do something in session
Check session activity logs and see "No logs/No logs to display"

Expected behavior
Logs should be visible
Additional context
Cause for this seems to be the way that Cognito userpools handles usernames with SAML 2.0 identity providers. According to the documentation, Cognito prepends the IdP name to the username provided by IdP, but it seems that in addition to this the whole combination of IdP name + username are made to lower case.
I verified this behaviour by modifying the query used to fetch the session activity logs by adding "lower" to the query, (lower(useridentity.principalId) LIKE '%:${username}%')
So I changed the https://github.com/aws-samples/iam-identity-center-team/blob/f50c29a18299f6a48c4cc203dbe3bc2374f7a43f/amplify/backend/function/teamgetLogs/src/index.js#L144 to be
const input = {
      QueryStatement: `SELECT eventID, eventName, eventSource, eventTime FROM ${EventDataStore} WHERE eventTime > '${startTime}' AND eventTime < '${endTime}' AND lower(useridentity.principalId) LIKE '%:${username}%' AND useridentity.sessionContext.sessionIssuer.arn LIKE '%${role}%' AND recipientAccountId='${accountId}'`,
    };

The other option would be creating a new attribute mapping in Identity Center for the application of (for example PreferredUsername - ${user:subject} - basic ) and using that instead of username from Cognito.

@jyrkiput good call out. I like the first option. Can you create a pull request for this change and i can test and merge it afterwards. Thanks


Improving UI of the Rules.

the rules section feels a little empty , i can refactor it to make it look more pleasing. Can you please assign me this issue.
Thank you!
@can you please assign me this task @iamparas0

Please assign it to me
Thanks.


[NEKO] Unconnected ports/registers

In the compute_unit_fpga.v file the "resultsReadyTag_in" output and the "gprCommand" register are left unconnected.

Those are placeholders.


How to access admin panel !

How do I access the backend admin panel from my domain name ?.

If you don't know, India is always here to help (old, but it didn't change). So, if you didn't change it during installation, it should be yourdoma.in/admin

After install you go to example.com/admin

I get this error

@OfficialMikeJ it may be your website issue can you share you website configuration to check

what does that even mean ? "Website issue". Be specific please ?.

what does that even mean ? "Website issue". Be specific please ?.

i meant web server issue - share your nginx configuration

@OfficialMikeJ  you can try with Apache server
seems you web server config is not correct

@peter-mw I haven't changed anything. all default settings.

@ajitbhandari I would never share any sensitive information about any kind of server. That's just not smart. I have fixed the issue thanks to @peter-mw.


Cartesia streaming (WebSocket) and word-level timestamps support

This PR changes the Cartesia service over to use the new bidirectional streaming. This allows for better tone consistency between sentences.
We also change the logic for sending TextFrames down the pipeline. TextFrames are used to manage the LLM context. With word-level timestamps, we can send individual TextFrames at the same time as we expect the raw audio to be sent over the transport layer. This allows us to get very close to matching up what a human participant hears with the LLM context.
The logic here is a little bit messy, because of how our pipelines work and what the base TTSService class is doing. Feedback is welcome.
Note that it's helpful to move the context aggregation in front of the transport, if we're able to use word-level timestamps to manage the context. The transport queues up TextFrames for processing, which creates a delay. If the context aggregation is after the transport output in the pipeline, this causes the last few TextFrames to be flushed incorrectly most of the time, when an interruption happens.

This looks great! Added a few super minor comments.
The only other comment is that this will mess up the LLMResponseStartFrame and LLMResponseEndFrame. These frames wrap every LLM chunk (i.e. every TextFrame). So, since now we are modifying the order of the TextFrames we send downstream, those frames don't make sense in this case. We use those frames in LLMAssistantContextAggregator. Which is something we should fix, it's a bit confusing having LLMAssistantContextAggregator and also LLMAssistantResponseAggregator. We should discuss this and review all this.
But anyways, this is good!

This looks great! Added a few super minor comments.
The only other comment is that this will mess up the LLMResponseStartFrame and LLMResponseEndFrame. These frames wrap every LLM chunk (i.e. every TextFrame). So, since now we are modifying the order of the TextFrames we send downstream, those frames don't make sense in this case. We use those frames in LLMAssistantContextAggregator. Which is something we should fix, it's a bit confusing having LLMAssistantContextAggregator and also LLMAssistantResponseAggregator. We should discuss this and review all this.

I was trying to remember if there was something newer than LLMAssistantResponseAggregator. Should we get rid of that class entirely, now, and only use LLMAssistantContextAggregator?

This looks great! Added a few super minor comments.
The only other comment is that this will mess up the LLMResponseStartFrame and LLMResponseEndFrame. These frames wrap every LLM chunk (i.e. every TextFrame). So, since now we are modifying the order of the TextFrames we send downstream, those frames don't make sense in this case. We use those frames in LLMAssistantContextAggregator. Which is something we should fix, it's a bit confusing having LLMAssistantContextAggregator and also LLMAssistantResponseAggregator. We should discuss this and review all this.

I was trying to remember if there was something newer than LLMAssistantResponseAggregator. Should we get rid of that class entirely, now, and only use LLMAssistantContextAggregator?

For now I will fix LLMAssistantContextAggregator. I'll try to send a PR tomorrow.


If I want to add other functions, what shoud I do?

for example, I want add another func sum (a int,b int)(int) , what should I do ?
I mimic your original QueryUser function, but get wrong responses.

I have finished it.


Monorepo support

It would be nice to be able to support multiple deployments from one repo.
Instead of having a single DeployPath we could have a DeployPath for each App Instance fro example.

Idea: Make the deploySubpath setting accept a string | { [key: string] : string }


[Question]: Why do we need to hard-code the value of max tokens

Describe your problem
refer code.
    if not llm:
        llm = TenantLLMService.query(tenant_id=dialog.tenant_id, llm_name=llm_id) if not fid else \
            TenantLLMService.query(tenant_id=dialog.tenant_id, llm_name=llm_id, llm_factory=fid)
        if not llm:
            raise LookupError("LLM(%s) not found" % dialog.llm_id)
        max_tokens = 8192
    else:
        max_tokens = llm[0].max_tokens
   ...
    if "max_tokens" in gen_conf:
        gen_conf["max_tokens"] = min(
            gen_conf["max_tokens"],
            max_tokens - used_token_count)

Can we just use max_tokens of dialog.llm_setting?

This is for the case we can't find max token length for assigned LLM.

Before reviewing the code, I was puzzled as to why the answers were still getting cut off even after setting a high value for max_tokens in dialog.llm_setting.
Why is there such a limit that can't be adjusted on the front end? @KevinHuSh

What kind of LLM did you use, let me check if there's a bug or something?

I use Xinference to add model qwen2.5. @KevinHuSh

RAGFlow does not know the context length of models added through XInference, which needs to be improved.

And I got the similar problem that I even modify the max_tokens setting logic.
I modify api/db/services/dialog_service.py like:
    ...

    # Avoid messages fit
    # used_token_count, msg = message_fit_in(msg, int(max_tokens * 0.97))
    assert len(msg) >= 2, f"message_fit_in has bug: {msg}"
    prompt = msg[0]["content"]

    if "max_tokens" in gen_conf:
        # Do NOT consider the input message tokens
        gen_conf["max_tokens"] = min(gen_conf["max_tokens"], max_tokens)

    ...

And I still get a cut off after setting a high value for max_tokens in dialog.llm_setting. I used deepseek-chat and openai interface didn't get a stop signal. It was weird and I was considering if it was a frontend problem.

It‘s definitely controled by context length of LLM.

The following piece of code has a defect: sometimes, even when the value of max_tokens is set to be greater than 8192 in the chat assistant, max_tokens still gets assigned a value of 8192.
    if not llm:         # Model name is provided by tenant, but not system built-in         llm = TenantLLMService.query(tenant_id=dialog.tenant_id, llm_name=llm_id) if not model_provider else \             TenantLLMService.query(tenant_id=dialog.tenant_id, llm_name=llm_id, llm_factory=model_provider)         if not llm:             raise LookupError("LLM(%s) not found" % dialog.llm_id)         max_tokens = 8192     else:         max_tokens = llm[0].max_tokens
I have modified it as follows:
max_tokens  = 8192     if not llm:         # Model name is provided by tenant, but not system built-in         llm = TenantLLMService.query(tenant_id=dialog.tenant_id, llm_name=llm_id) if not model_provider else \             TenantLLMService.query(tenant_id=dialog.tenant_id, llm_name=llm_id, llm_factory=model_provider)         if not llm:             raise LookupError("LLM(%s) not found" % dialog.llm_id)     if llm and llm[0] and hasattr(llm[0], 'max_tokens'):         max_tokens = llm[0].max_tokens


How to set lr and learning policy if I train libra faster rcnn without pretrain

I already know whether using Pretrain has little effect on the final result, but how can I set lr and learning policy and epoch to ensure that the model can converge？

See details in training from scratch.


SDK 31 Upgrade

Hello,
Recently we needed to upgrade the project's  target SDK number to 31. When we did so, we saw that FullscreenRecorderActivity doesn't have 'exported=true' attribute declared in the manifest. To be able to run the app, we needed ziggeo to upgrade it's activities (that have intent-filter) to have 'exported=true'. (Saw in merged manifest, that FullscreenRecorderActivity and UriParserActivity don't have it.)
Can you please add this? Thanks a lot in advance.

@objectorientedperson please, take a look at the v2.4.1

@3akat fixed in v2.4.1. Thank you.


feat(regexp-assemble): Merge data files for 930100

Generating the regular expression for 930100 has required two files up
to now. With the improved regexp-assemble machinery the expression can
now be generated from a a single file.

@theseion I'm having trouble testing this. I can't get generate to work:
$ ./regexp-assemble.py generate 930100
Traceback (most recent call last):
  File "/home/xanadu/.git/coreruleset/util/regexp-assemble/./regexp-assemble.py", line 229, in <module>
    namespace.func(namespace)
  File "/home/xanadu/.git/coreruleset/util/regexp-assemble/./regexp-assemble.py", line 127, in handle_generate
    context = create_context()
TypeError: create_context() missing 1 required positional argument: 'namespace'

Also, update is causing errors:
$ ./regexp-assemble.py update 930100
Traceback (most recent call last):
  File "/home/xanadu/.git/coreruleset/util/regexp-assemble/./regexp-assemble.py", line 229, in <module>
    namespace.func(namespace)
  File "/home/xanadu/.git/coreruleset/util/regexp-assemble/./regexp-assemble.py", line 146, in handle_update
    updater = Updater(create_context(namespace.rule_id))
  File "/home/xanadu/.git/coreruleset/util/regexp-assemble/./regexp-assemble.py", line 214, in create_context
    return Context(root_directory, namespace)
  File "/home/xanadu/.git/coreruleset/util/regexp-assemble/lib/context.py", line 17, in __init__
    self.single_rule_id = namespace.rule_id if namespace else None
AttributeError: 'str' object has no attribute 'rule_id'

But compare works, somehow?!
$ ./regexp-assemble.py compare 930100
Processing 930100, chain offset 0
Generating LALR tables
Regex of 930100 has changed!

===========
first difference
...

@RedXanadu Sorry, needed to rebase to get those fixes in. Thanks for reviewing!

That PR hasn't been merged: https://github.com/coreruleset/coreruleset/pull/2534. You'll have to wait or manually merge the two branches (of fix the error by hand ;) )

Oh! I understand. Okay, I'll manually merge the branches and re-test this evening.

@theseion I merged those branches together by hand and confirmed that it all works as intended.
The new preprocessor in regexp-assemble to make this work is very cool!
I'll leave it to you to merge this branch in whatever order you think is best, i.e. wait for https://github.com/coreruleset/coreruleset/pull/2534 to be merged first or not.

Thanks @RedXanadu!


No Quit Button in Notification

We should add a button that allows the user to quit the application from the notification

Closed via 13eb084d18923e6183b96d8fc2ad5d19bbb090aa


Link issue

Hello. In the latest version 1.7.20 of generouted, using thetag of ant design, the route can be updated after clicking, but the page is not updated. Hope it can be solved.thanks

tag

Link tag

Hey, was it working with you in v1.7.19?
In any case, could you please provide a repro using something like StackBlitz or CodeSandBox?


Overlay shape while transcoding video

I added this function in 3_transcoding.c

It's called to draw an square in every frame it only changes the Y of the YUV file, I made it just to see if I could overlay images this way

This is the fist frame of the video, screenshot from VLC^^

This is the last frame of the video, screenshot from VLC^^

This is the pgm output of the save_gray_frame function^^

This is another example of the save_gray_Frame function ^^
Is this the intended way to overlay images?, Are there other ways of doing it?
I imagine that the image is weird because of the compression

hi @LentilStew nice addition, usually the overlay is handled by the ffmpeg filtering (my intent is to do a next chapter on that).
But you can see filtering in action on github.

If anyone wants to overlay the image this way, without it being ugly, you have to copy the frame

and send the encoder a copy of the frame Remember to free the new frame

cool @LentilStew thanks for sharing!
Why do you need to copy format, width... even using the av_frame_copy_props? (I don't know this API, just curious, by its name I suppose it should copy all the properties, I'm guessing)

I copied the code from here stackoverflow
But the av_frame_get_buffer() function says that "format (pixel format for video, sample format for audio)- width and height for video- nb_samples and channel_layout for audio" must be set before calling the function
and the function av_frame_copy_props() says that it does not copy pts, sample rate (for audio) or sample aspect ratio (for video), but not width/height or channel layout. Side data is also copied.
the av_frame_copy_props() only copies "metadata" fields from src to dst


Markdown mode does not recognise list after HR

When using markdown mode, if I write an HR, followed by newline, then list, it works as expected, with the next bullet being suggested...
---

- this
- <~ automatically inserted

If I have list immediately following --- without newline, it does not auto insert next bullet
---
- this
<~ no auto insertion of bullet

Poking around in the code, it has something to do with prevLineIsList not being set correctly, but I got lost beyond that.

@bfrohs Want to take a look at this? (The report makes is framed in terms of continuelist, but the issue actually also shows in the highlighting -- a list after a HR isn't recognized as such.)

This broke in 0bfd44a40248fe5d47268f1226ecc561eee74887 (PR #3178)

Fixed via #3282


Fix jupyter notebook usage: Replace stty with shutil.get_terminal_size

In an ipykernel (e.g. when in a jupyter notebook), install_extras emits a spurious error to stderr:
$ jupyter console
>>> import prettyprinter; prettyprinter.install_extras(warn_on_error=False)
stty: 'standard input': Inappropriate ioctl for device

The problem appears to be the usage of the stty shell command:
>>> import os; os.popen('stty size', 'r').read()
stty: 'standard input': Inappropriate ioctl for device
''

Replacing uses of stty with the builtin shutil.get_terminal_size() appears to avoid this:
>>> import shutil; shutil.get_terminal_size()
os.terminal_size(columns=133, lines=75)

Sorry for the multiple commits, this is ready for review.

Thanks! LGTM. Sorry for the delay in merging this.

Released to PyPi in https://github.com/tommikaikkonen/prettyprinter/releases/tag/v0.14.0!