EntityFramework operations fails if you create new project with "." in the title

If you create new project like x new react-spa MyCompany.MyApp, the resulting generated block of code in Configure.Db.cs looks like this:
// $ dotnet ef migrations add CreateIdentitySchema
// $ dotnet ef database update
services.AddDbContext<ApplicationDbContext>(options =>
    options.UseSqlite(connectionString, b => b.MigrationsAssembly(nameof(MyCompany.MyApp))));

Because of how nameof operator works, it only passes "MyApp" as assembly name, even though full assembly name is "MyCompany.MyApp".
This causes running any dotnet ef command fail with error Could not load file or assembly 'MyApp, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.
Changing the code to use string litteral with full assembly name or getting assembly name through reflection fixes the issue.
services.AddDbContext<ApplicationDbContext>(options =>
    options.UseSqlite(connectionString, b => b.MigrationsAssembly("MyCompany.MyApp")));

services.AddDbContext<ApplicationDbContext>(options =>
    options.UseSqlite(connectionString, b => b.MigrationsAssembly(Assembly.GetAssembly(typeof(ConfigureDb))?.FullName)));

https://github.com/NetCoreTemplates/react-spa/blob/fda9be1e7d1c2b85ba4a221647f42c492736caa5/MyApp/Configure.Db.cs#L23

The project name only expects a single name, e.g. MyCompany or MyApp not a name with a namespace.
I recommend recreating the template with just MyCompany to avoid other tempting issues.


Ansible update fixes

Epel has now released version 5.4 of Ansible within stable with this there are a few things that broke these are the fixes.

Selinux commands

Ansible no longer defaults to platform-python which is python3.6 which allows the python36-libselinux package to be used, instead it uses python3.8 and therefore the libselinux package cannot be used due to being a different version. The fix was to set the ansible python interpreter to platform-python as this defaults to python3.6 and is the recommended solution: https://bugzilla.redhat.com/show_bug.cgi?id=2091684

grafana_datasource module

The fix was to change url to ds_url as due to this version update url no longer aliases to ds_url and now is an alias to grafana_url, therefore making the url explicit with ds_url fixed this problem. An overview of the new format can be found here: https://docs.ansible.com/ansible/5/collections/community/grafana/grafana_datasource_module.html.

I've been annoyed that we're stuck on Ansible 2.9 for so long so the fact that 5 is now available in EPEL is very welcome news. Given how much has changed in Ansible in the meantime, I'm surprised that this is all the changes that are needed. To give some more confidence, would you be able to also edit the CI config file (.github/workflows/main.yml) in your branch to change the version of Ansible specified from:
'ansible<2.10,>=2.9' 'ansible-lint<6.0.0'

to perhaps
'ansible>=5,<6' 'ansible-lint'

I expect there are things that will not be compatible.
Furthermore, the mgmt node bootstrap from Terraform (https://github.com/clusterinthecloud/terraform/blob/master/common-files/bootstrap.sh.tpl) runs Ansible before having installed EPEL so it will currently still be getting Ansible 2.9 (whereas the compute nodes to install EPEL before Ansible so will get the new version - https://github.com/clusterinthecloud/ansible/blob/6/roles/packer/templates/prepare_ansible.sh.j2). We might also need to install ansible-collection-community-general to get all the core community roles.


grant database role

Is your feature request related to a problem? Please describe.
The current resources "snowflake_grant_privileges_to_role" and "snowflake_role_grants" does not appear to allow for a statement such as:
grant database role snowflake.usage_viewer to role test_role
which doesn't follow the privilege grants paradigm of
grant privileges on object_type object_name to role role_name
The objective here is to grant a database role to an account role
Describe the solution you'd like
We would either need a new resource or adapt the grant_privileges_to_role to behave differently for the "DATABASE ROLE" type privilege
Describe alternatives you've considered
I am currently using snowsql_exec as an alternative
resource "snowsql_exec" "snowflake_database_grants" {
  name = random_pet.snowsql_query_identifier.id
  create {
    statements = "GRANT DATABASE ROLE SNOWFLAKE.USAGE_VIEWER TO ROLE TEST_ROLE"
  }
  delete {
    statements = "REVOKE DATABASE ROLE SNOWFLAKE.USAGE_VIEWER FROM ROLE TEST_ROLE"
  }
  depends_on = [
    random_pet.snowsql_query_identifier,
   ]
}

Additional context
Add any other context or screenshots about the feature request here.

I think database grant was released with v0.70.0 with https://github.com/Snowflake-Labs/terraform-provider-snowflake/pull/2014, but there are no documentation yet on how we can use it. Also another related issue https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/2018 is available.
Currently, also using snowsql as a workaround similar like @eraliod

Thanks @abid1995 but I think the snowflake_database_role resource released in v0.70.0 is to create a database role, not to grant a database role to another role. I did +1 the issue you mentioned, as it does appear the documentation is missing for that resource.

#2014

Thanks @eraliod , 🤔 saw the PR https://github.com/Snowflake-Labs/terraform-provider-snowflake/pull/2014 released in v0.70.0 which references to the grant database role

Thanks @abid1995 but I think the snowflake_database_role resource released in v0.70.0 is to create a database role, not to grant a database role to another role. I did +1 the issue you mentioned, as it does appear the documentation is missing for that resource.

Thanks @eraliod , 🤔 saw the PR https://github.com/Snowflake-Labs/terraform-provider-snowflake/pull/2014 released in v0.70.0 which references to the grant database role

on second review, this is a duplicate from #2014 as mentioned in the comments above


Release LuckPerms for additional platforms on Modrinth

Description
LuckPerms for platforms other than Forge and Fabric are not available on Modrinth.
Proposed Behaviour
As Modrinth now supports platforms such as Bukkit, LuckPerms should provide those releases there as well.
Extra Details
No response

LuckPerms is on Modrinth with all platforms https://modrinth.com/plugin/luckperms
Thank you @Zoriot for letting us know this could be closed!


Bug 1910160: Create OperatorConditions for CSVs being installed

Problem: If OLM creates an OperatorConditions for an object before the
CSV is being installed it could update the existing deployment which is
not upgradeable, killing the pod.
Solution: Do not create the OperatorCondition until the CSV is being
installed.

/bugzilla refresh


加载clip的预训练模型

加载预训练模型的时候，clip的模型是pt后缀，为什么你用的是pth，想问下在哪下载的？
我修改了后缀名，加载clip的pt文件但是后面还是会报错。
_MODELS = {
"ViT-B/16": os.path.join(MODEL_PATH, "ViT-B-16.pth"),
"ViT-L/14": os.path.join(MODEL_PATH, "vit_l14.pth"),
"ViT-L/14_336": os.path.join(MODEL_PATH, "vit_l14_336.pth"),
}

就是去哪里下载clip的图像encoder权重

Please follow the step in INSTRUCTION. You have to use the code here to generate the weight.

Thanks for your response!  Thanks for your work!

Thanks for your try! As there is no more activity, I am closing the issue, don't hesitate to reopen it if necessary.

请问在运行extract.ipynb文件时出现以下错误需要怎么解决呢

conda环境中clip已经安装了


Wont build on windows 10

Trying to build after installing both "Microsoft Visual C++ Compiler for Python 2.7" and "iconv".
I installed inconv using the .exe, was I supposed to do it differently?
When I try to install zbar-py using pip I get this:
c:\python27\include\pyport.h(9) : fatal error C1083: Cannot open include file: 'inttypes.h': No such file or directory     error: command 'C:\\Users\\Christian\\AppData\\Local\\Programs\\Common\\Microsoft\\Visual C++ for Python\\9.0\\VC\\Bin\\cl.exe' failed with exit status 2
I tried downloading inttypes.h from here: https://code.google.com/archive/p/msinttypes/downloads and tried placing it in the respective \include\ and \bin\ directories. Neither is working.

+1 on this is anyone is looking :)

Hi irbigturtle,
There are two .h files on the link. stdint.h and inttypes.h. For VC++ for python2.7 you will have to copy it in C:\Users\rn5949\AppData\Local\Programs\Common\Microsoft\Visual C++ for Python\9.0\VC\include
It should work I have used this method to build other packages.
Thanks

Hi
I am trying to build it in windows, I have some other errors.
#8


POSIX shell support

It would be good to not have bash as a dependency and to use #!/usr/bin/env sh instead for cross platform support.
I've made some headway with this which I'll paste below but it's beyond my shell scripting skills.
#!/usr/bin/env sh

set -eu

## Functions/
usage() {
cat << EOF
SYNOPSIS

    gpm leverages the power of the go get command and the underlying version
    control systems used by it to set your Go dependencies to desired versions,
    thus allowing easily reproducible builds in your Go projects.

    A Godeps file in the root of your Go application is expected containing
    the import paths of your packages and a specific tag or commit hash
    from its version control system, an example Godeps file looks like this:

    $ cat Godeps
    # This is a comment
    github.com/nu7hatch/gotrail         v0.0.2
    github.com/replicon/fast-archiver   v1.02   #This is another comment!
    github.com/nu7hatch/gotrail         2eb79d1f03ab24bacbc32b15b75769880629a865

    gpm has a companion tool, called [gvp](https://github.com/pote/gvp) which
    provides vendoring functionalities, it alters your GOPATH so every project
    has its own isolated dependency directory, its usage is recommended.

USAGE
      $ gpm             # Same as 'install'.
      $ gpm install     # Parses the Godeps file, installs dependencies and sets
                        # them to the appropriate version.
      $ gpm version     # Outputs version information
      $ gpm help        # Prints this message
EOF
}

is_in_use() {
  [ -e "$1/.git/index.lock" -o -e "$1/.hg/store/lock"  -o -e "$1/.bzr/checkout/lock" ]
}

# Iterates over Godep file dependencies and sets
# the specified version on each of them.
set_dependencies() {
  local deps=$(sed 's/#.*//;/^\s*$/d' < $1) || echo ""

  echo "$deps" | while read package version; do
    (
      echo ">> Getting package "$package""
      go get -u -d "$package"
    ) &
  done
  wait

  echo "$deps" | while read package version; do
    (
      local pkg_path=$(echo "$package" | awk -F/ '{print $1"/"$2"/"$3}')
      local install_path="${GOPATH%%:*}/src/${pkg_path%%/...}"
      echo ">> Setting $package to version $version"
      cd $install_path
      is_in_use $install_path && wait

      [ -d .bzr ] && bzr revert   -q -r   "$version"
      [ -d .git ] && git checkout -q      "$version"
      [ -d .hg  ] && hg update    -q      "$version"
      [ -d .svn ] && svn update   -r      "$version"
    ) &
  done
  wait
  echo ">> All Done"
}
## /Functions

## Command Line Parsing
case "${1:-"install"}" in
  "version")
    echo ">> gpm v1.3.1"
    ;;
  "install")
    deps_file="${2:-"Godeps"}"
    [ -r "$deps_file" ] || (echo ">> $deps_file file does not exist." && exit 1)
    (which go > /dev/null) ||
      ( echo ">> Go is currently not installed or in your PATH" && exit 1)
    set_dependencies $deps_file
    ;;
  "help")
    usage
    ;;
  *)
    ## Support for Plugins: if command is unknown search for a gpm-command executable.
    if command -v "gpm-$1" > /dev/null
    then
      plugin=$1 &&
      shift     &&
      gpm-$plugin $@ &&
      exit
    else
      echo ">> No command 'gpm $1'"
      usage && exit 1
    fi
    ;;
esac

Let's take it step by step: I'll initially make the loop change when I find the time to and then we'll go over all other things that need changing for POSIX support, I'm leaving this issue open to keep track of it

Sorry for the delay in getting back to you. My undergraduate thesis (the project during which I became frustrated with the POSIX incomparability of GPM) is due today. I'll have more time over the coming weeks to contribute.

Don't worry about it! We're all volunteers in this and I'm also taking a long time to respond to things lately.
Good luck with your thesis! :dancer:


Extract import / export type data?

I am looking for a way to programmatically extract the complete list of imports and exports from a WASM module along with each im/export's full type signature. It looks like this library might be just what I need, but the documentation doesn't really make it obvious how that could be accomplished. Is there an easy way to do it, or do I need to keep looking elsewhere?

Check the examples/dump-section.js , you just need to handle more states. See also https://github.com/wasdk/wasmparser/blob/648fff52d2a186fb6bee97dc1f37b97640e26fbf/src/WasmParser.ts#L1346-L1357

Closing as anwered.


Name the layer after the source instead of "Downloaded Layer"

Oh yeah, I meant to put that on the list, closely related to #8 - had to cut off continual improvements to be able to get it out. But I'll try to get this in for the next release.

Closed with #31


First PR:  new best-practices devops instance which uses AzureML Service to run a unit test

Description
This PR runs one unit test to check the path from DevOps to Github to AzureML etc.
The Best-practices DevOps instance has a pipeline called unit_tests_staging which will be triggered by PRs to staging. I will update the trigger once the PR is accepted.
As this version is for testing, I've left some print statements in the code to make it easier to validate setup.  Those prints will show up in the output files of the DevOps pipeline and will be removed once I confirm functionality.
The pipeline is temporarily private while testing this new change. Once the change works, it will be made public.
Docs for this will be added once it's publicly visible.
Related Issues
#449
Checklist:

[x ] My code follows the code style of this project, as detailed in our contribution guidelines.
[ ] I have added tests.
[ ] I have updated the documentation accordingly.

@bethz this looks good, it would be nice to have instructions on TEST.md. Scott and I proposed some suggestions, after a couple iterations, I would love to try this

unit test staging is failing, I think because the old and new pipelines have the new name. Maybe for the time being we could have different names for each

about this:

@bethz this looks good, it would be nice to have instructions on TEST.md. Scott and I proposed some suggestions, after a couple iterations, I would love to try this

I think I've answered all the issues raised so far. I also started info for docs on the repo wiki. It would to talk about what else to include for docs.


Spring Boot Buildpacks with BP_JVM_JLINK_ENABLED fails in Gitlab CI

Hi,
I hope someone can shed some light on this, i'm a bit stumped on :)
Using a standard spring initializr project with build packs enabled runs in local mac and gitlab ci.
If BP_JVM_JLINK_ENABLED = true, it will build in local mac but the same process does not run in Gitlab CI.
Extract from CI console below. Just wondering if this feature works for anyone? Kind of stumped why  enabling BP_JVM_JLINK_ENABLED stop the build only in Gitlab CI.
Your help would be much appreciated.
Gradle config, nothing specaial
`
plugins {
id 'org.springframework.boot' version '2.7.18'
id 'io.spring.dependency-management' version '1.0.14.RELEASE'
id 'java'
id 'jacoco'
id 'maven-publish'
id 'com.diffplug.spotless' version '6.25.0'
}
jar {
enabled = false
}
apply plugin: 'jacoco'
apply plugin: 'com.diffplug.spotless'
group = 'com.example.playground'
sourceCompatibility = '17'
configurations {
compileOnly {
extendsFrom annotationProcessor
}
}
repositories {
mavenLocal()
google()
mavenCentral()
}
sourceSets {
main {
java.srcDir file("${buildDir}/generated-src")
}
integrationTest {
    java.srcDir file('src/integration-test/java')
    resources.srcDir file('src/integration-test/resources')

    compileClasspath += main.output + test.output
    runtimeClasspath += main.output + test.output
}

}
task integrationTest(type: Test) {
testClassesDirs = sourceSets.integrationTest.output.classesDirs
classpath = sourceSets.integrationTest.runtimeClasspath
outputs.upToDateWhen { false }
group("verification")

}
check.dependsOn integrationTest
integrationTest.mustRunAfter test
dependencies {
implementation 'org.springframework.boot:spring-boot-starter'
implementation 'org.springframework.kafka:spring-kafka:2.9.13'
testImplementation 'org.springframework.boot:spring-boot-starter-test'
testImplementation 'org.springframework.kafka:spring-kafka-test'

}
spotless {
java {
importOrder()
googleJavaFormat()
removeUnusedImports()
trimTrailingWhitespace()
}
}
test {
useJUnitPlatform()
finalizedBy jacocoTestReport
reports {
junitXml.getRequired().set(true)
}
}
jacoco {
toolVersion = "0.8.11"
reportsDirectory = file("build/jacoco")
}
jacocoTestReport {
dependsOn test
reports {
xml.enabled true
xml.destination file("build/reports/jacoco.xml")
sourceSets sourceSets.main
executionData fileTree(project.rootDir.absolutePath).include("**/build/jacoco/*.exec")
}
}
tasks.named("bootBuildImage") {
environment["BP_JVM_VERSION"] = "17"
environment["BP_JVM_JLINK_ENABLED"] = "true"
}
`
console output:
`
[32;1m$ ./gradlew build bootBuildImage --imageName=playground:abc
Welcome to Gradle 7.5.1!
Here are the highlights of this release:

Support for Java 18
Support for building with Groovy 4
Much more responsive continuous builds
Improved diagnostics for dependency resolution

For more details see https://docs.gradle.org/7.5.1/release-notes.html
Starting a Gradle Daemon (subsequent builds will be faster)

Task :compileJava
Task :processResources
Task :classes
Task :bootJarMainClassName
Task :bootJar
Task :jar SKIPPED
Task :assemble
Task :compileTestJava
Task :processTestResources NO-SOURCE
Task :testClasses
Task :test
Task :compileIntegrationTestJava NO-SOURCE
Task :processIntegrationTestResources NO-SOURCE
Task :integrationTestClasses UP-TO-DATE
Task :integrationTest NO-SOURCE
Task :spotlessInternalRegisterDependencies
Task :spotlessJava
Task :spotlessJavaCheck
Task :spotlessCheck

Task :jacocoTestReport
..
Task :check
Task :build

Task :bootBuildImage
Building image 'playground:abc'

Pulling builder image 'docker.io/paketobuildpacks/builder-jammy-base:latest' ..................................................
Pulled builder image 'paketobuildpacks/builder-jammy-base@sha256:ff47160c561329011180fcec8f3ae32c18c044ab9c57c3d981157309734693e9'
Pulling run image 'docker.io/paketobuildpacks/run-jammy-base:latest' ..................................................
Pulled run image 'paketobuildpacks/run-jammy-base@sha256:436a84a3ae3bdbc9f0534b7e6a80122d05f734048137691fcfec36f8132758b1'
Executing lifecycle version v0.19.3
Using build cache volume 'pack-cache-4bc4e9944d46.build'

Running creator
[creator]     ===> ANALYZING
[creator]     Image with name "playground:abc" not found
[creator]     ===> DETECTING
[creator]     6 of 26 buildpacks participating
[creator]     paketo-buildpacks/ca-certificates   3.6.8
[creator]     paketo-buildpacks/bellsoft-liberica 10.5.5
[creator]     paketo-buildpacks/syft              1.45.0
[creator]     paketo-buildpacks/executable-jar    6.8.5
[creator]     paketo-buildpacks/dist-zip          5.6.10
[creator]     paketo-buildpacks/spring-boot       5.27.11
[creator]     ===> RESTORING
[creator]     ===> BUILDING
[creator]
[creator]     Paketo Buildpack for CA Certificates 3.6.8
[creator]       https://github.com/paketo-buildpacks/ca-certificates
[creator]       Launch Helper: Contributing to layer
[creator]         Creating /layers/paketo-buildpacks_ca-certificates/helper/exec.d/ca-certificates-helper
[creator]
[creator]     Paketo Buildpack for BellSoft Liberica 10.5.5
[creator]       https://github.com/paketo-buildpacks/bellsoft-liberica
[creator]       Build Configuration:
[creator]         $BP_JVM_JLINK_ARGS           --no-man-pages --no-header-files --strip-debug --compress=1  configure custom link arguments (--output must be omitted)
[creator]         $BP_JVM_JLINK_ENABLED        true                                                         enables running jlink tool to generate custom JRE
[creator]         $BP_JVM_TYPE                 JRE                                                          the JVM type - JDK or JRE
[creator]         $BP_JVM_VERSION              17                                                           the Java version
[creator]       Launch Configuration:
[creator]         $BPL_DEBUG_ENABLED           false                                                        enables Java remote debugging support
[creator]         $BPL_DEBUG_PORT              8000                                                         configure the remote debugging port
[creator]         $BPL_DEBUG_SUSPEND           false                                                        configure whether to suspend execution until a debugger has attached
[creator]         $BPL_HEAP_DUMP_PATH                                                                       write heap dumps on error to this path
[creator]         $BPL_JAVA_NMT_ENABLED        true                                                         enables Java Native Memory Tracking (NMT)
[creator]         $BPL_JAVA_NMT_LEVEL          summary                                                      configure level of NMT, summary or detail
[creator]         $BPL_JFR_ARGS                                                                             configure custom Java Flight Recording (JFR) arguments
[creator]         $BPL_JFR_ENABLED             false                                                        enables Java Flight Recording (JFR)
[creator]         $BPL_JMX_ENABLED             false                                                        enables Java Management Extensions (JMX)
[creator]         $BPL_JMX_PORT                5000                                                         configure the JMX port
[creator]         $BPL_JVM_HEAD_ROOM           0                                                            the headroom in memory calculation
[creator]         $BPL_JVM_LOADED_CLASS_COUNT  35% of classes                                               the number of loaded classes in memory calculation
[creator]         $BPL_JVM_THREAD_COUNT        250                                                          the number of threads in memory calculation
[creator]         $JAVA_TOOL_OPTIONS                                                                        the JVM launch flags
[creator]         Using Java version 17 from BP_JVM_VERSION
[creator]       BellSoft Liberica JDK 17.0.10: Contributing to layer
[creator]         Downloading from https://github.com/bell-sw/Liberica/releases/download/17.0.10+13/bellsoft-jdk17.0.10+13-linux-amd64.tar.gz
[creator]         Verifying checksum
[creator]         Expanding to /layers/paketo-buildpacks_bellsoft-liberica/jdk
[creator]         Adding 137 container CA certificates to JVM truststore
[creator]         Writing env.build/JAVA_HOME.override
[creator]         Writing env.build/JDK_HOME.override
[creator]       JLink: Contributing to layer
[creator]     unable to invoke layer creator
[creator]     unable to retrieve list of JVM modules for jlink
[creator]     unable to list modules
[creator]     exit status 1
[creator]     ERROR: failed to build: exit status 1

Task :bootBuildImage FAILED

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':bootBuildImage'.

Builder lifecycle 'creator' failed with status code 51
`

https://stackoverflow.com/questions/78317955/spring-boot-buildpacks-bp-jvm-jlink-enabled-fails-to-build-in-gitlab-ci/78318755


Edit Redis and add MySQL to Troubleshooting.md

These parts have been edited based on my experiences. It is impossible for a server to have the same allocation port twice, this is an issue that blocks you from connecting servers to a proxy. So Pterodactyl by default allows containers to access your local network through <IP_ADDRESS> since <IP_ADDRESS> inside the container would be the container itself.

Thanks for improving the docs :)


Implement localizable strings

Two strings can't be localizable, this PR can implement them.

Thanks!


Improve goto definition for MBE

This PR improve the macro resolution for goto definition and expression macro invocation by using proper path resolution for external macros.

bors r+

bors retry


Now permitting use of pre-installed llama.cpp

Hello, and thank you for your great work!
I'm addressing a problem affecting Unsloth when deployed in an isolated network: saving in GGUF performs a git pull for llama.cpp, which is bound to fail in those environments. The implemented solution is to first check for the necessary programs in the system first, and only download/compile llama.cpp if required. NOTE: llama.cpp python scripts can be executables, given that they contain the appropriate shebang, and they are looked up as such.
Thanks again, have a great one
pepi

OOO I like this idea - very interesting


Workaround for current temp on Metric system

HA rounding makes temperatures weird, such as 26.1, 24.9
This workaround gives whole numbers instead by converting to metric, rounding and converting back.

I'm not sure which one needs to be merged - can you put all needed changes into one PR for me to review?  Additionally, can you target the "dev" branch?


Ronnia conflict with osu-np bot

The problem is if user use !np command the osu-np will do this in chat:
@!sender!, Now Playing: !artist! - !title! [!version!] Mapped by !creator!, Download: https://osu.ppy.sh/beatmapsets/!set-id!#osu/!id!
@kamythol, Now Playing: LiSA - Rock-Mode [mode: 180BPM] Mapped by Smokeman, Download: https://osu.ppy.sh/beatmapsets/240756#osu/556307
but since ronnia check chat for beatmap links it will send !np as Now playing as a request beatmap
LiSA - Rock-Mode [mode: 180BPM] - Request sent!
Either there need a condition that check if it's an !np command or even a request.
and then a option to change what the String output in chat would be. but that is optional, since it's your project and bot not mine :D

You can exclude users from the dashboard. If you add np-bot's nickname to the list, ronnia should ignore it.

By default, the bot should ignore messages from your own account. You should disable test mode. If that still doesn't work, you can try adding your own username to excluded users.

Are you using https://github.com/aticie/ronnia-selfhost ?

It's a bit different with the selfhost bot.
Just make sure to have:
listen-to-self-message = False
If that does not work, try adding your twitch username to:
excluded-users =

so right now, !np works if i use my own account, and since i disable ronnia for streamer my own account it won't work but !np and ronnia does not work if i do it from a new account idk why

The error text right here is informative.
For example, if you are the ronnia-selfhost bot and osu-np bot on the same twitch account called wei, any messages coming from wei will be ignored by ronnia. This is what you wanted, right?

Could you give an example of what's happening exactly? Please include screenshots of what you are doing, and what you want to achieve.

I can't help with np-bot. You could try raising the issue on their repository OsuNowPlaying ?
From what I see, you have listen-to-self-message = True in your settings. And ronnia is working as expected.

I try to send a beatmap link in chat on a different account called chuuflow not my main account

Ronnia should respond to the last message on here. If it doesn't, could you show what it outputs to the terminal after you send a beatmap link from chuuflow account?

Do you see any logs for any other user's (for example NightBot's) messages?

It seems like for some reason both Ronnia and the np bot doesn't see the messages on your channel. Please double-check your TMI token and twitch username.


Is there anyway to disable diagnostic for stylistic error?

Is there anyway to disable diagnostic for stylistic error only?

You can create a .lintr file in your workspace root. See "project configuration": https://github.com/jimhester/lintr

You can create a .lintr file in your workspace root. See "project configuration": https://github.com/jimhester/lintr
Edit: also, a neat extension for this is 'Error Lens'. you can fine-tune the display of diagnostics through the extension's settings

Thanks for the suggestion. Able to implement it through .lintr.

Great! Closing this for now. Feel free to re-open if you have any issues with getting this to work


[Autocomplete][Doc] Mention preload can be set to false

Q
A

Bug fix?
no

New feature?
no

Issues
N/A

License
MIT

Setting preload to false is useful when you only want to display search results; that way nothing is loaded on focus.

Thank you @MatTheCat.


Ability to delete row at position

Reading the delete rows code syntax, your library is taking in the database table name and the pojo class but the setup seems to be saying something different. Maybe an update to the setup might be in order to ensure people that wish to use the library in the near feature don't get confused.
Also is it possible to add removing the rows according to the position or can the library already do this? I achieved this with the default database by doing
Cursor cursor = sqliteDatabase.query(DraftContract.DraftEntry.TABLE_NAME,
    null, null, null, null, null, null);
cursor.moveToPosition(position);
int id = cursor.getInt(cursor.getColumnIndex("_id"));
String[] whereClauseArgument = new String[1];
whereClauseArgument[0] = Integer.toString(id);

sqliteDatabase.delete(DraftContract.DraftEntry.TABLE_NAME, "_id=?", whereClauseArgument);

with this library and trying something like this which throws errors
Inquiry.get()
    .deleteFrom(Draft_DB, GateHelper.class)
    .where("_id = ?", position)
    .run();

Updated your description with syntax highlighting.
So basically, you want to be able to delete rows at a specific position? The problem with this is that not every table has an _id row, so it would only work for some people.

Am aware. Thanks for considering it as an enhancement to this library.

Thanks for considering this as an enhancement to the library.


Package tests

Unless I got lost in the package tree, I don't see any tests other than those in nixos/tests, which are effectively for the NixOS modules.
I want to start adding package tests, primarily for all those package I regularly update/maintain.
a) Where should I put those
b) How would I bootstrap them? (I want them to be run as part of the channel updates, not unlike the nixos/tests files)
c) Do we have concerns with Hydra build times if/when people were to start adding lots of tests?
/cc @vcunat @domenkozar @edolstra @shlevy @aszlig @fpletz @globin @joachifm @copumpkin @grahamc  (just pinging people who usually have good opinions on these things, anybody is welcome to provide feedback)

I know that @Profpatsch has already put a lot of thought into package tests and has some code ready. Not sure where, though. :)

Yes! It’s nearly finished!
@NeQuissimus, if you want we can chat (Profpatsch on freenode<EMAIL_ADDRESS>on Jabber/XMPP) and I can show you what I got so far.

@Profpatsch Very good, I will wait for the PR. Ping me when you have something reviewable :)
A little while ago I added a test for ammonite, not realizing that tests need to be explicitly bootstrapped. So I believe my test does not run at all right now. I just want a simple solution that lets us test packages, since we do have a good amount of breakage on a regular basis...

That would be nice.
Modules have a lot of ad-hoc code. Using a nixos module in production often means forking it, removing some hacks and adding my own (on the second deployment - removing my old hacks), that implies writing new tests or leaving the module untested.
On contrary, packages are much more stable and reusable. This makes them good objects for the tests.

For people who already want to write package tests, I’ve written a short overview on how to do it until the implementation gets merged:
https://github.com/NixOS/nixpkgs/pull/27641#issuecomment-319235606

@Profpatsch PING :D Any status updates? :)

Not yet. I will be working on it some more over the next month.

Ping? :innocent:

#36842 essentially implements this right?

@timokau Parts of it. I should really merge my code.
Wouldn’t comment on this thread too much, since it mentions so many people.

@Profpatsch, @7c6f434c, @timokau what is the status of this effort?
Was the whole idea abandoned after https://github.com/NixOS/nixpkgs/pull/36842 was rejected?
I'm still waiting for a good way to annotate packages with simple tests. Do you agree that checkPhase inside the derivation is currently the best way to do it? Would it be useful if package maintainers add it to their packages? Especially in the context of https://github.com/ryantm/nixpkgs-update/issues/13 and possibly duplicated effort. @ryantm

@davidak IMHO checkPhase is not a good place since it cannot be run independently of the package build. Tweaking the test would mean rebuilding the whole package, and so would disabling flaky test.
Me and @worldofpeace are adding new nixos tests to passthru.tests (example). r-ryantm runs derivations listed in that attribute (https://github.com/ryantm/nixpkgs-update/pull/118) and ofborg might do that as well (https://github.com/NixOS/ofborg/pull/410).
I think using runCommand in there might also work nicely.

I think using runCommand in there might also work nicely.

Yes! For simple programs an entire VM test is overkill and a derivation running a few commands is enough. See for example: https://github.com/NixOS/nixpkgs/pull/72855/files#diff-7ff02ed41d58b0e67d73301d85d22a40R3

@davidak I join the recommendation to use passthru.tests over checkPhase for tests we add in Nixpkgs without relying on the upstream tests. I think we also have some support for upstream's installed-tests.
@JohnAZoidberg the worst part is that even for LibreOffice NixOS VM tests are a bit of an overkill (VM setup takes longer than the test itself, starting LibreOffice and clicking on stuff and checking menus actually appear via OCR), but there we need to refactor the necessary functionality, such as launching display server and D-Bus.

The downside of passthru.tests is that test failures won't fail the build and tests are not run automatically. Linking nixos tests has also a discoverability problem.

@timokau that's just a tooling problem. We have to teach our tools to do this. Hydra, r-ryantm, ofBorg and maybe nix build --with-tests.
I think it would even be useful to extract the checkPhase to its own derivation, so that changing the test doesn't cause a rebuild for the program's derivation.

I don't think tooling is a nice solution for this. As long as running tests is optional, many people won't run them when making changes. Also many people will install broken packages from hydra without knowing they are broken. I like @Profpatsch's approach much better (inverting the dependencies so that the packages depend  on their tests, IIRC).
I'm not saying that passthru.tests is bad, just that its not optimal in my view.

Unavoidable tests, though, create a pressure to reduce them (to make installation faster) — so realistically we could switch from opt-in to opt-out, but mandatory will lead to a different conflict of interest.

Yeah, the reverse-dependency approach would essentially be opt-out. Each package could provide an untested attribute that skips the tests. I think opt-in would lead to a lot of failing tests that are just accepted.

I guess we end up having an untested set like we have pkgsi686Linux, but yeah, not having it as a default might be a good idea.
I do not think that including the tests in the main build itself leads to a good balance of incentives, so I guess putting them into passthru.tests for now is an OK idea.
(I guess in the long run we could teach mkDerivation that sequentialisation trick by @Profpatsch and give it a list of functions from untested outputs to installed tests)


Create a section for AMP Ads (A4) showing how to make the actual ads

The current examples only show how to embed ads.

cc: @jasti

The current one doesn't use correct a4a boilerplate and doesn't show what can be added like ad analytics.   Also might add  a sponsored Article example and other native format examples.

//cc @carlosfuertes

We definitely should fix the current samples. Migrating the existing samples to the correct format requires a different preview though.

https://github.com/ampproject/amp-by-example/issues/635


Install problems

Hello I wanted to give a try to GIMP-ML but I can't see the menu. I get this error during install
    ERROR: Command errored out with exit status 1:
     command: /home/emrah/Code/GIMP-ML/gimp-plugins/gimpenv/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-54icbb_9/enum/setup.py'"'"'; __file__='"'"'/tmp/pip-install-54icbb_9/enum/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-install-54icbb_9/enum/pip-egg-info
         cwd: /tmp/pip-install-54icbb_9/enum/
    Complete output (11 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/home/emrah/Code/GIMP-ML/gimp-plugins/gimpenv/lib/python3.7/site-packages/setuptools/__init__.py", line 5, in <module>
        import distutils.core
      File "/home/emrah/anaconda3/lib/python3.7/distutils/core.py", line 16, in <module>
        from distutils.dist import Distribution
      File "/home/emrah/anaconda3/lib/python3.7/distutils/dist.py", line 9, in <module>
        import re
      File "/home/emrah/anaconda3/lib/python3.7/re.py", line 143, in <module>
        class RegexFlag(enum.IntFlag):
    AttributeError: module 'enum' has no attribute 'IntFlag'
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

Maybe it confuses due to other packages in my system. Thanks

follow INSTALLATION.md

Had this same error with python 3.8. Not with 2.7 though.

Hi! Love your work! @kritiksoman ! I'm facing some issue. Under the "Layer" , i dont see GIMP-ML... Wondering why this is so!
I also had the same error of :  AttributeError: module 'enum' has no attribute 'IntFlag' (This occured during installation)

It is not installed in Ubuntu 20.4.1, someone has some solution?  If I change in the script to Python2, the script runs with errors. I tried with the script and manually and can not install it. I think that some changes are needed to run at Ubuntu 20.04 and Python 3.8. I'm just a GIMP user. Does anyone have an idea?

You can edit installGimpML.sh and change the line
python -m virtualenv gimpenv
to
python -m virtualenv -p `which python2` gimpenv
This is because it's likely that the default of "python" now points to python3 and you specifically want python2. There is also a restriction on your version of pip but this might actually get you where you need to go.

If the author is waiting for GIMP3 to update to Python3, the installer probably needs to be updated for the latest versions of Ubuntu. If I get time I'll contribute, but I'm not running anything still running on Python 2 on my system at the moment.

Working on gimp 3 development now .

The problem is that gimp2.10 no longer supports python2. There is no python support in GIMP right now. You can downgrade to 2.08 or wait until GIMP supports python again.

https://github.com/kritiksoman/GIMP-ML/tree/GIMP3-ML

The problem is that gimp2.10 no longer supports python2. There is no python support in GIMP right now. You can downgrade to 2.08 or wait until GIMP supports python again.

GIMP 2.10 supports Python 2 just fine, but some distros have dropped Python 2 and/or some Python modules that are necessary for the GIMP Python modules.

You should use Gimp from AppStore, this supported Python2. It also nessesary for health repair...
https://appimage.github.io/

I tried install new copy of gimp 2.99.14 and got this when loading.  Also got an error during installing torch 1.8.1
(gimp-2.99.exe:9704): GLib-GIO-WARNING **: 21:02:01.956: win32 session dbus binary not found
set device 'System Aggregated Pointer' to mode: disabled
Skipping potential plug-in 'C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\plugin_utils.py': plug-ins must be installed in subdirectories.
Skipping potential plug-in 'C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins_init_.py': plug-ins must be installed in subdirectories.
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\superresolution\superresolution.py", line 283, in do_query_procedures
self.set_translation_domain(
AttributeError: 'SuperResolution' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\semseg\semseg.py", line 203, in do_query_procedures
self.set_translation_domain(
AttributeError: 'SemSeg' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\monodepth\monodepth.py", line 204, in do_query_procedures
self.set_translation_domain(
AttributeError: 'MonoDepth' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\matting\matting.py", line 223, in do_query_procedures
self.set_translation_domain(
AttributeError: 'Matting' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\kmeans\kmeans.py", line 235, in do_query_procedures
self.set_translation_domain(
AttributeError: 'Kmeans' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\interpolation\interpolation.py", line 279, in do_query_procedures
self.set_translation_domain(
AttributeError: 'Interpolation' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\inpainting\inpainting.py", line 308, in do_query_procedures
self.set_translation_domain(
AttributeError: 'InPainting' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\filterfolder\filterfolder.py", line 221, in do_query_procedures
self.set_translation_domain(
AttributeError: 'FilterFolder' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\faceparse\faceparse.py", line 204, in do_query_procedures
self.set_translation_domain(
AttributeError: 'FaceParse' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\enlighten\enlighten.py", line 203, in do_query_procedures
self.set_translation_domain(
AttributeError: 'Enlighten' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\detectobjects\detectobjects.py", line 196, in do_query_procedures
self.set_translation_domain(
AttributeError: 'DetectObjects' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\denoise\denoise.py", line 202, in do_query_procedures
self.set_translation_domain(
AttributeError: 'Denoise' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\dehaze\dehaze.py", line 204, in do_query_procedures
self.set_translation_domain(
AttributeError: 'Dehaze' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\deblur\deblur.py", line 205, in do_query_procedures
self.set_translation_domain(
AttributeError: 'Deblur' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\colorpalette\colorpalette.py", line 47, in do_query_procedures
self.set_translation_domain(
AttributeError: 'ColorPalette' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\coloring\coloring.py", line 253, in do_query_procedures
self.set_translation_domain(
AttributeError: 'Coloring' object has no attribute 'set_translation_domain'
Traceback (most recent call last):
File "C:\apps\GIMP-ML\gimpenv3\lib\site-packages\gimpml\plugins\canny\canny.py", line 238, in do_query_procedures
self.set_translation_domain(
AttributeError: 'Canny' object has no attribute 'set_translation_domain'

You can edit installGimpML.sh and change the line python -m virtualenv gimpenv to python -m virtualenv -p `which python2` gimpenv
This is because it's likely that the default of "python" now points to python3 and you specifically want python2. There is also a restriction on your version of pip but this might actually get you where you need to go.

Hi, In GIMP I am unable to see Openvino Plugins In layer.
Error : (gimp-2.99.exe:9872): GLib-GIO-WARNING **: 11:59:11.778: win32 session dbus binary not found
set device 'System Aggregated Pointer' to mode: disabled
What the solution can any one hep me in this ?


Fixed typographical error, changed availabe to available in README.

ArtSabintsev, I've corrected a typographical error in the documentation of the Harpy project. You should be able to merge this pull request automatically. However, if this was intentional or you enjoy living in linguistic squalor please let me know and create an issue on my home repository.

👍🏾

hah, nice catch! Merging!


Set the prompt for each tmux session to bash$

This will remove the bash version from the test fixtures.
Other changes typical for fixture generation on a different system, different python version and test name updates.
Related #1376

@cidrblock looking at the diff I realise that huge amount of stuff that would cause tests to fail in the future, even time, as we check the age of the containers, publishing a new creator-ee will break tests too.
I am going to merge it because it looks to work in good direction but there is still a lot more to change.


[devicelab] Add timestamps to logs

https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android flutter_gallery__back_button_memory/908/overview is hard to debug as we don't know what actions are taking the longest.

This does already exist:
[flutter_gallery__back_button_memory] [STDOUT] adb logcat: 11-08 23:21:57.695  1503  4563 I ActivityManager: Start proc 21953:com.google.process.gapps/u0a42 for content provider com.google.android.gsf/.gservices.GservicesProvider

It's the 4th column


Maybe use dirty schedulers

Investigate if dirty schedulers would benefit re2_nif and,  if so, how it can be used in a backwards compatible way. We could build two different libs and conditionally load re2_nif.so or re2_nif_ds.so.

Dirty schedulers are beneficial when you have NIF tasks that regularly run too long on a normal scheduler thread — where "too long" means longer than 1-2ms — and there's no natural or easy way to break the work into chunks. One way to determine if your NIF workloads run too long on a normal scheduler is to use erlang:system_monitor/2 with the {long_schedule, Time} option, where you set Time to 2 (milliseconds).
In Erlang 17 and 18 dirty schedulers are labeled as an experimental feature, which means they're off by default and so if you don't specifically configure and build your system with them enabled, they won't be available. We hope to remove the experimental label for Erlang 19, but until then, you can conditionally compile code using dirty schedulers under the ERL_NIF_DIRTY_SCHEDULER_SUPPORT preprocessor macro, which is defined only when the system supports dirty schedulers. To check at runtime whether dirty schedulers are enabled, call enif_system_info() and then verify that the smp_support and dirty_scheduler_support fields in the resulting ErlNifSysInfo struct are non-zero (and yes, you're right you shouldn't have to check both fields, but for now you do).
The dirty scheduler API is relatively simple, since all it requires is some preprocessor macros, ERL_NIF_DIRTY_JOB_CPU_BOUND and ERL_NIF_DIRTY_JOB_IO_BOUND, that developers use to tell the Erlang runtime where to run a NIF. The API has been stable for some time, ever since I introduced enif_schedule_nif() in 17.3. The enif_schedule_nif() function allows you to decide at runtime to schedule NIF work on a normal scheduler or on a dirty scheduler. These macros can also be specified in the flags field of an ErlNifFunc instance so that a particular NIF is always dispatched to a dirty scheduler.
The bottom line is that if your NIF tasks regularly run too long and cause long_schedule events to regularly occur, you have three options:

Break the NIF into multiple functions and orchestrate their calls from Erlang. The idea here is to hide NIF calls within a regular Erlang function, where each NIF call is short enough to avoid long_schedule problems and the Erlang code ties them all together. This keeps the normal Erlang schedulers happy because the NIFs return to Erlang frequently.

Break a long-running NIF into chunks of work that can be scheduled with enif_schedule_nif(). With this approach, a NIF runs for a time that avoids long_schedule problems and if it still has work to do, essentially reschedules itself on a normal scheduler by calling enif_schedule_nif(). This keeps the normal Erlang schedulers happy because the NIF yields the scheduler thread whenever it calls enif_schedule_nif(), but this approach requires a way to break the work up so it can be done in chunks.

Run the work on a dirty scheduler. If the work can't be easily broken up with either of the previous two approaches, then it can run on a dirty scheduler. The overhead of switching to and from a dirty scheduler is quite tiny and will almost certainly be in the noise compared to the work the task performs. One thing to keep in mind is that dirty scheduler thread pools, one meant for CPU-bound jobs and the other for I/O-bound jobs, are shared resources, so if lots of processes are using them, jobs will queue up for them.

Thanks for the detailed response, @vinoski. I'll add it to the repo with some extra ideas.


Don't allow users to withdraw 0 BPT

Description

Set bptInValid to only be true if bptIn amount is > 0 so that users can't start a withdraw for 0 tokens. This value was previously always true.

Fixes: https://balancer-labs.sentry.io/issues/4000665600/?project=5725878&query=is%3Aunresolved+level%3Afatal&referrer=issue-stream&statsPeriod=14d&stream_index=1 (can see in additional info that bptIn is 0.0.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Dependency changes
[ ] Code refactor / cleanup
[ ] Documentation or wording changes
[ ] Other

How should this be tested?

Try withdrawing nothing from a pool. The preview button should now be disabled when you set the slider to 0 BPT.

Checklist:

[x] I have performed a self-review of my own code
[x] I have requested at least 1 review (If the PR is significant enough, use best judgement here)
[x] I have commented my code where relevant, particularly in hard-to-understand areas
[x] If package-lock.json has changes, it was intentional.
[x] The base of this PR is master if hotfix, develop if not

@garethfuller Ah I see, didn't realize that's how it worked. Ok I've set it back to a ref and added the check in the validAmounts computable.


Improve dot ast

previous

now

@fabsx00 Please review this one.

This seems better. Currently the plot is a little bit too wide to view, for example:

@evilpan This is the primary target of this PR.

@Liyw979 Thanks for the PR! Can you add line wrap instead of the .map(StringUtils.abbreviate(_, 50))?


18 keys is completely flipped

Describe your bug here.
https://github.com/user-attachments/assets/06c86d38-0ccf-4f3b-b1bd-f918b98a2ade
I was using the latest github build and saw that 18 keys was completely flipped in terms of the keybinds and notes 😅
(I'm not sure if it is the same for the other keycounts)
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No, I was just using the windows build from the latest commit

The keybinds are QWERTYUIOP ASDF[space] HJKL

Partially fixed in commit cb3dc42c532694f68ee8f434e45e31a7cd911072


HubConnection _pendingCalls Change to ConcurrentDictionary

As the documentation says here:
"
In general, use a System.Collections.Concurrent.ConcurrentDictionary in any scenario where you are adding and updating keys or values concurrently from multiple threads. In scenarios that involve frequent updates and relatively few reads, the ConcurrentDictionary generally offers modest benefits. In scenarios that involve many reads and many updates, the ConcurrentDictionary generally is significantly faster on computers that have any number of cores.
"

 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.:x: vpaulino sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi there!
Glad to ear feedback! I will not be able to review this topics today ( I have limited internet access today ) but tomorrow I will answer all your questions! :)

@vpaulino Awesome!

We originally considered a ConcurrentDictionary but decided against it because this is a fairly write-heavy dictionary and as you cited:

In scenarios that involve frequent updates and relatively few reads, the ConcurrentDictionary generally offers modest benefits.

Do you have data showing a concurrency problem in your scenario?

Hi there,
So I have made some tests and it looks like lock is more less time consuming than ConcurrentDictionary.
I have implemented a small console application that prints to the output the time consuming per iteration that compares booth approaches:
https://github.com/vpaulino/Concurrency-Playground
My conclusions are based on the Stopwatch that measure each operations and reports that I generate with VS extension: Concurrency Visualizer
Looks like bigger are the number of records that we want to manipulate on the dictionary the more time it takes to execute the operations concurrently.
Fell free to comment the project or add some ideas of more use cases to test. if you find some logic error on that test some input is welcome ! :)
For me if you can reject this PR because it clearly degrades performance. Sorry about that.

For me if you can reject this PR because it clearly degrades performance. Sorry about that.

To be clear, you're good with withdrawing this PR? I'm going to close it since it seems like that's the case, apologies if I misinterpreted your comment.
Thanks for showing interest in improving the product! We're certainly happy to keep exploring PRs like this when we've got some data to show a benefit. If you need help collecting some data, feel free to open an issue to discuss a possible improvement, or just send me a direct email ([my github<EMAIL_ADDRESS>and I'm happy to help give some guidance on how to collect the necessary data and explore improvements.


Support for intel linux driver

fixes #204

r+ after the change!

Ping @tomaka

Is this change in 0.2.0? If not, could a 0.2.1 release be made?


fix: handle missing solutions correctly

Rather than creating an [[]] the parser now creates [] which isEmpty().
This makes the test suite check the next challenge for a solution.
In addition, the logic for testing solutions was fixed.
@ShaunSHamilton @nhcarrigan if run npm run test:full-output in the curriculum there aren't too many failing tests.  If you've got any ideas how to fix them, please go ahead and push commits.

@nhcarrigan I have done most of what I can.
The issues I have added comments you can search with @nhcarrigan, and they all pretty much seem to be steps that ask Campers to do something, but that change does not follow through to the rest of the steps. So, I am unsure if you want to delete the step or add the change to all the steps.

I will take a look ASAP. If they aren't blocking, we can go ahead and land this and I'll make a new PR.

If they aren't blocking, we can go ahead and land this and I'll make a new PR.

Well, CI will fail for all new PRs (or rebased PRs), because the tests will fail.

I think this can be closed now.


fix(types): correct typo

I met some typo wrong when using this with Typescript, and fix it. Please consider to merge it.

Sure thing!


Multi-select

Thanks for this great library!  The components look great and it's been really easy to use.
There is 1 component that I cannot (easily) port over to using react-toolbox, and I would like to get it added to react toolbox, it's a multi select dropdown.  I am currently using https://github.com/Khan/react-multi-select which works well, but looks out of place with the rest of the UI.
Assuming it was built, is this something that could get added?  Should this be a separate component or something built using special options on Select?  I am not sure I could build it anytime soon, but if I do get a chance, I'd like to find out where it could fit in.
Thanks!

Yes, I think that's a good way to describe it.  The display would be pretty different, and the behaviour would be different in some ways too.  Putting it into a dream list, I come up with:

Selecting an item doesn't close the dropdown menu, so if selected/deselecting multiple items, the menu doesn't have to be re-opened
There would be the addition of a "Select/Deselect All" option at the top of the dropdown
The selected values display in 1 line - if all values are selected, the display shows "All items selected".  If a couple items are selected, it displays each item.  When the items fill 1 line, it switches from showing values individually to displaying "N items selected".
The dropdown shows all the options and has some indication of which items are selected like a checkbox or using highlighting.  This is just personal preference I like how the filters in Google Sheets/Excel work where all the options are visible
When not being used, the display fits in 1 line.  When closed, only the selected values or special message ("N items selected") displays.  The text input search box shows up as part of the dropdown list.

I think every options except the last one could easily fit with autocomplete and be configured via flags.  The last one could fit in there too, but I'm guessing would require a pretty big overhaul


Error on the documentation and validation

On the Swagger documentation, on the Parameter Object section, it is mentioned that the maxLength and minLength properties are limited to the objects of type integer.
This is wrong, this attributes are available for object of type string - as explained on the JSON schema.
The online validator also throws an false warning «object instance has properties which are not allowed by the schema: ["maxLenght"]»

The documentation says that the type of it is integer. It does not say it is limited to anything (even though JSON Schema indeed defines it for strings).
The validator throws a warning because the name of the field is maxLength and not maxLenght.

Hi Ron,
Thank you very much for the fast reply and sorry to have post such false and basic bug.
Best regards,
André
Sent from my Windows Phone

From<EMAIL_ADDRESS>Sent: 29/04/2015 12:42
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: Re: [swagger-spec] Error on the documentation and validation (#347)
The documentation says that the type of it is integer. It does not say it is limited to anything (even though JSON Schema indeed defines it for strings).
The validator throws a warning because the name of the field is maxLength and not maxLenght.

Reply to this email directly or view it on GitHub:
https://github.com/swagger-api/swagger-spec/issues/347#issuecomment-97398238

No worries, better submit something that's not needed than not submitting something that is :)


[BUG] Type 'OAProjectItem'  is not marked as serializable in VS2022 on files generation

Select category

[x] LHQ Editor - Visual Studio Extension
[ ] ScaleHQ.WPF.LHQ - NuGet Package
[ ] ScaleHQ.AspNetCore.LHQ - NuGet Package
(place x inside square brackets to mark category or multiple categories)

Version
2022.1
Visual Studio Edition

[x] Visual Studio Community
[x] Visual Studio Pro
[x] Visual Studio Enterprise
(place x inside square brackets to mark edition or multiple editions)

Visual Studio Version

[ ] 2015
[ ] 2017
[ ] 2019
[x] 2022
(place x inside square brackets to mark version or multiple versions)

Describe Problem
Generating files in VS2022 throws error:
Type 'Microsoft.VisualStudio.ProjectSystem.VS.Implementation.Package.Automation.OAProjectItem' in Assembly 'Microsoft.VisualStudio.ProjectSystem.VS.Implementation, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' is not marked as serializable.'

Found forum with same error:
http://vfrdtyky.blogspot.com/2019/04/visual-studio-serialization-error-when.html
possible fix:
The EnvDTE assemblies are COM interop assemblies. Your error can be avoided by creating a Runtime Callable Wrapper, which marshals calls to the COM object based off information in the interop-assembly. Microsoft has provided an extension method within the
Microsoft.VisualStudio.TextTemplating namespace:

<#@ template hostspecific="true" language="C#" #>

<#@ assembly name="EnvDTE" #>

<#@ import namespace="Microsoft.VisualStudio.TextTemplating" #>

<#

  IServiceProvider serviceProvider = (IServiceProvider)this.Host;

  EnvDTE.DTE dte = (EnvDTE.DTE) serviceProvider.GetCOMService(typeof(EnvDTE.DTE));

 #>

The GetCOMService(this IServiceProvider, Type type) extension method is the correct way to obtain the DTE COM object.

T4 templates run in a separate AppDomain, and I believe that is the reason your code is working despite the exception. IServiceProvider.GetService(typeof(DTE)) returns a transparent Proxy Object. This exception is because the proxy requires objects crossing an app domain be decorated with the Serializable attribute. You can confirm the DTE object in your code is a "transparent proxy" like this:

bool isProxy = RemotingServices.IsTransparentProxy(dte);

fixed in Release v2022.2


why is it incompatible with lithosphere?

neoforge 1.21.1 .79
and would there be a way to fix this incompatibility?

Not officially. Both mods use the same data-based methods to do stuff. You can read the same thing on the Litosphere's page:
_`Most worldgen mods (like Biomes O' Plenty, Nature's Spirit, ...) should be perfectly compatible, but lithosphere changes the deafult noise_settings so worldgen datapacks that change the overworld will be incompatible.
Datapacks that change the noise_settings files can be made compatible by changing their noise_settings to refer to Lithosphere's final_density.`_
Soo. techniacally, there is an easy way to make them somehow compatible: remove "noise_settings" from the Woodlands mod at all. I just tested it and it worked (extremely laggy though). But I wouldn't do this officially, because it ruins Woodlands' original look.
If you want to do so, open the Woodlands .jar with 7zip and go to data/minecraft/worldgen and delete "noise_settings" folder. This will remove every ground-cover change Woodlands makes but the mods would work together. The biomes will not look exactly as intended.
And! The more important thing is: will the result be Litosphere landscape with Woodlands biomes? I'm not sure :-D Both mods change the overworld dimension as is, so the result of their partial compatibility will be very questionable. For example with some other data-based mods Woodlands work, but completely takes over their settings, or it could be the other way around.


feat(store): add founders editions to notebooksbilliger.de

Description
Notebooksbilliger seems to be the (official) retailer for german FE drops now.
Fixes #584

Thank you!


Como se crean estas direcciones.

<EMAIL_ADDRESS>Estoy buscando el modo de poder generar urls así, como debo configurar clappr.
http://photocall.tv/cl.php?vid=
Saludos

This page is a simple PHP script which generate the following html content :
(vid GET query parameter as source url)
<title>PHOTOCALL.TV - Streaming</title>
<meta name="viewport" content="width=device-width, initial-scale=1">
<link rel="shortcut icon" href="http://www.espai.jazztel.es/favicon2.ico" type="image/x-icon" />

PS: you should always post in english (as far as possible), so more people can help you.

and how can I do to generate it. is there any place to download it? I do not know the code

Here a "fast and dirty" example :
(please note this is out of scope of Clappr)
cl.php :
<title>PHOTOCALL.TV - Streaming</title>
<meta name="viewport" content="width=device-width, initial-scale=1">
<link rel="shortcut icon" href="http://www.espai.jazztel.es/favicon2.ico" type="image/x-icon" />

You should also json_encode($url) the URL before echoing it so that it if it contains '"' for example it is escaped properly and doesn't break the JavaScript
On 1 Oct 2017, at 12:20, kslimani <notifications<EMAIL_ADDRESS>wrote:
Here a "fast and dirty" example :
(please note this is out of scope of Clappr)
cl.php :
PHOTOCALL.TV - Streaming

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/clappr/clappr/issues/1513#issuecomment-333366951, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADG-WVLSsaxUafVMv8H0zx-BjsQcMNp6ks5sn2dtgaJpZM4PpriC.

You should also json_encode($url) the URL before echoing it so that it if it contains '"' for example it is escaped properly and doesn't break the JavaScript On 1 Oct 2017, at 12:20, kslimani <notifications<EMAIL_ADDRESS>wrote: Here a "fast and dirty" example : (please note this is out of scope of Clappr) cl.php : PHOTOCALL.TV - Streaming      — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#1513 (comment)>, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADG-WVLSsaxUafVMv8H0zx-BjsQcMNp6ks5sn2dtgaJpZM4PpriC.

how to do it properly. im doing it but page broke in php.


check-automated-repositories does not warn that an user has been manually removed

Bug description
My assumption is that the check-automated-repositories should be used as read-only action to validate that the collaborators defined in the yml files are in-sync with the actual users within the repos.
But when i use it, i have noticed that it does not give any indication if an user has been manually removed (by an admin) but is still mentioned in the yml
is this by design or a bug?
Steps to reproduce

create a repo
define the repo in the outside-collaborators with an outside user with write access.
commit to main
the outside-collaborators-handler will add the user.
user gets email and accepts it
i run check-automated-repositories workflow
`Processing automated repository "repo-sandbox/poc"...

"user" with permissions "write"
...done with "repo-sandbox/poc" ✔
`

as a repo admin, i manually remove the user from the repo.
i run check-automated-repositories workflow
`Processing automated repository "repo-sandbox/poc"...

"user" with permissions "write"
...done with "repo-sandbox/poc" ✔
`

Expected behavior
Step 8 should indicate that the user is not present in the repo, but defined in the script.
Example repository
Additional context
No response

Hi @globaltopmedia

is this by design or a bug?

This is actually intended.
This action is triggered by a pull request:
https://github.com/icub-tech-iit/outside-collaborators/blob/2b34d07e13ec3c4becea8f6dfc4331fe2fea8654/.github/workflows/check-automated-repositories.yml#L4
It only verifies if the proposed modifications are consistent regarding users' eligibility (i.e., it exists, it's not an org member, and its permissions are just fine).
To keep things simple, I didn't implement anything that brings up a warning if a user is manually removed and I wouldn't do so, honestly. Alternatively, one could rely on the main handler itself.
Hope this clarifies.

Yes, that explains. thanks. issue close


Does not collect

Does not collect all channels

Explain more

I enter the address and mac and it does not collect all channels

E.g. 300 out of 1000 channels

And I don't have the ones I want and they are on the list

@fairy-root You are

####It did not happen on my end though I tested on lists of over 26k
####Text me here, and if the issue is in the script, I'll update it

Telegram: https://t.me/FairyRoot

Fixed

Already fixed


clang and gcc

The patch disables openmp package and uses the default in the system. Otherwise, it requires the same clang version as the openmp (which is a problem to compile with gcc).
The other part of the patch is to enable c++17 in openroad (they use this in their builds). It does not build in the latest gcc/clang otherwise.
The patch allows gcc and clang to work (I tried gcc 11 and clang 13, but others should work too). To compile with clang:
CXX=clang++ CC=clang bazel build ....

I create a small repo to test this (it points to renau/bazel_rules_hdl not the hdl/bazel_rules_hdl)

So there's a couple of things going on here. By default all code should be compiling with c++17 https://github.com/hdl/bazel_rules_hdl/blob/main/.bazelrc#L15, perhaps the run commands need to added to the .bazelrc
The CI runs on a hermetic version of clang so removing the openmp dependency will make that toolchain fail. It would at least need a select based on toolchain.

OK, I think that I know the issue.
My testing repo does not have the .bazelrc. This means that I do not have an hermetic llvm (I let to use the local in my machine. This allows me to try gcc or clang).
let me try with the bazelrc


decrease the size of the logo in the navbar for mobile screen ONLY

i want to do this

Assigned @tejas342


Scaling Max freq not found

İdk why

Hello @buzaslan129 ,
Which device do you use? Which version of Android do you have?
The screenshot you've attached won't open.


Download speed is crippling - I can download 100x faster with another method

Hello. When I download a video with latest youtube-dl it is extremely slow. Like 50 KB per second
With another method I follow I am able to download with 5000 KB per second
I go to en.savefrom . net
It generates some video save-as-able links
Then I copy and paste that link into uGet with 16 connections
How can I achieve same speed with youtube-dl?
Because this process is annoying

Duplicate of #29326, where solutions are provided. Also duplicate of at least 15 other issues that OP might have found by following the Broken site support template.

Confirming the 'drop in' youtube.py is working like a beast:
71.2% of 1.99GiB at 13.80MiB/s ETA 00:42
Was sobbing over 56K speeds after checking main page for new release, found this ticket, and did the following:

git clone repo
rm'd extractor/youtube.py
wget copy from above in its place
apt-get pandoc
make clean
make

Hardest part was following the damn instructions instead of hacking the youtube-dl directly to replace the file. Was much easier to do what I would normally do at work & pull repo, replace file, build per instructions.   (8-P
Just leaving a note that performance is crazy good with 'drop in' and offer some encouragement to other lazy a$$es that were on the fence whether to wait for another release.
Thx to the team for this utility!!! It has literally changed my life after adding a bash front end. 20TB+ deep into saving the planet from the scourge of L1BT@RD censorhip.


Use macros instead of functions

I want to affect the scope from which CPMFindPackage is called to append the package's source directory to CMAKE_MODULE_PATH, as they contain my project's CMake modules. With FetchContent it is possible, as FetchContent_MakeAvailable (apparently a macro) eventually calls add_subdirectory.
The CMake modules repository CMakeLists.txt is
list(APPEND CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}")
set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} PARENT_SCOPE)

Before realizing this was possible, I just included its first line after the FetchContent_MakeAvailable call.

Good point, I see how that would be an expected outcome. However, I'm reluctant to change CPMFindPackage to a macro, as the current version defines a bunch of variables that I don't want leaking to the outer scope. Another issue is that the dependency can be added at any other point in the project (by a dependency or a dependant), in which case CPMFindPackage would not actually call add_subdirectory. This means that you would still never be sure if the current scope actually modified. Possible ways to resolve the issue for now are:

If you have access to the dependency's CMakeLists.txt, you could define a function updateModulePath() that can be called from the outer package.
Use the DOWNLOAD_ONLY YES flag and call add_subdirectory(<dependency>_SOURCE_DIR) manually after CPM.cmake has added the package.

Ah, you're right about its shortcomings. Guess there's no way around having one extra line without making assumptions. Thank you.
I think this can be closed, but I'm hesitant to do it as you labelled it "enhancement".

Ah yes it can be closed, I was wondering if it still makes sense to use macros instead of functions for leading to the add_subdirectory call, but for now I guess it's not worth the effort.


To remove the use of ":" in usernames, names and namespaces

I can create an account with "test:", it has a colon at the end. This becomes an issue when using to send or receive nxs with the username:account.  "test::default" is not recognised and this will create an issue going forward.

So I have been looking into this and the issue stems from they way we extract that name/namespace/account from the input.
Your example it is treating test as a namespace and default as a username as it sees the ::
There is not technical reason why we can not have the : in the username as if I manually run

            hashNamespace = TAO::Ledger::SignatureChain::Genesis(SecureString("test:"));
            fFound = TAO::Register::GetNameRegister(hashNamespace, "default", nameObject);
            hashNameObject = TAO::Register::Address("default", hashNamespace, TAO::Register::Address::NAME);

I will get the correct data.
This code comes from src/TAO/API/commands/names/names.cpp as part of the GetName method.
I am not sure the best way to go about this, how do you determine when the nexus separators start and end and where does the user's input start and end.
If you input test::default
is test: the username and default the account
or is test the username and :default the account
OR is test the namespace and default the username
that example shows 3 possible outcomes from the same text.
We could force the non-use of : but this would be a restriction based on understanding the input, not by an technical issue.

Issue is being retroactive to all possible name types, there are some that have a colon in them and making a direct rule to this can break the previous ones


PackagedProductDefinition.description - cardinality and language

Current specification allows only one description without specifying the language. For multilingual countries this might be a problem, as the standard translations might not be clear/fair enough.

EMA implementation uses extension to indicate the language.
				<description value="Üks vaginaalpehmekapsel on blistris, mis koosneb läbipaistvast kolmekordsest PVC/PVdC/PVC laminaatkilest ja on suletud alumiiniumfooliumist kattega. Blister ja polüpropüleenist aplikaator on pakendatud kartongkarpi. ">
					<extension url="https://ema.europa.eu/fhir/extension/language">
						<valueCoding>
							<system value="https://spor.ema.europa.eu/v1/lists/100000072057"/>
							<code value="100000072172"/>
							<display value="Estonian"/>
						</valueCoding>
					</extension>
				</description>

In MedicinalProductDefinition.name, FHIR spec presents a different approach with country and name clearly indicated.

Should be discussed whether the current solution satisfies users or if a more consistent approach should be considered.

Any resource may define the language in the metadata.
Any human readable element may use the translation extension to convey additional translations see https://build.fhir.org/languages.html#ext
Is this not enough ?

I don't know. Noticed by EE and SE, and we don't really need it anyway, because we only have one official language.
We just noticed that the product name has a special language attribute while package description does not, so EMA has created their own extension for it.
I guess for multilingual countries it might be a bit of a political issue to choose which one of your languages is the 'real' one and which are given as translations. But you're right, technically it's not much of a problem.
My next question is, is the EMA implementation (see xml example above) good? Using a self-defined extension to indicate the language of a text block? (EMA IG, ch 4.2.1)

My next question is, is the EMA implementation (see xml example above) good? Using a self-defined extension to indicate the language of a text block? (EMA IG, ch 4.2.1)

I don't know .. we should ask them...I suppose there is a reason if they did it
the only thing I can see is that translation uses a code for the language, may they need to have a CodeableConcept.. but I'm guessing...

https://jira.hl7.org/browse/FHIR-39128


PR for a feature described in issue #43

Closes #43
Allows user to set a variable "THOW_ERROR_IF_VERSION_EXISTS" to fail the action execution if user forgot to increment the version

Mind opening this PR to https://github.com/Elskom/publish-nuget/?


🧪 Add unit tests

Added a bunch of unit tests, I suggest that we add a few more coordinate tests... I tested that a few sets of random glyphs could convert between glyphs, galactic coordinates, xyz back and forth 😋

Could we just bump the version if it needs it using npm version patch?

The logic behind the bumping is that npm doesn't allow the same version to be uploaded twice. I agree that we don't necessarily need this for things like unit tests, Readme updates, etc., but I haven't yet found a way to check for this (I think GH Actions has a files option you can use to check for certain files, I need to look into that)

I can implement this later today.
If no relevant files changed, then no release should be made either, so we need to add this to all workflows.
https://stackoverflow.com/questions/70478320/github-actions-ignore-certain-files-inside-a-directory

That looks cool 💪. Then the publish-npm.yml, publish-github.yml and check-version.yml would have those ignore paths set?
Maybe it makes more sense to set the paths to include for those 3 workflows with src/**.ts. Then only code changes would require a version bump.
One issue would be that unit tests would be picked up, so then we would need an exclude rule like src/**.spec.ts.
That should cover everything right? 🤔
Then we never have to think about the version number 😝
I can play with it right now, unless you want to do it later 😋

Huh, why can we not use the paths property? Does it simply not work?

Nope, we can't use both at the same time 😂

As long as it's just "at the same time", it shouldn't be much of an issue IMO. We just have to define whether it would be easier to include or to exclude (I tend towards exclude), and then we can set up the paths accordingly

Yup, PR is there if you want to look at it 😋. I think I usually tend towards include 🤔

LGTM


sort_reverse doesn't work for my integer field

I've got a simple integer field, "page", that I want to sort in reverse by default (3, 2, 1). I have:
list do
  sort_by :page
  sort_reverse true
end

Regardless of whether I set it to true, false, or remove sort_reverse entirely, my records are still sorted like 1, 2, 3.

Please try:
list do
  sort_by :page
  field :page do
    sort_reverse true
  end
end


Ci

I know u said that you dont wanted PRs for now. but this is just CI.

I've made local conflicting changes to the makefiles so they use the standard ps2sdk toolchain, and don't create the .txt files (removed them from git also). But I'm still using a local makefile system, instead of the 'samples' makefiles that ps2sdk installs.
I'll use your CI as the basis with perhaps some changes, but I can't pull this pr directly if you don't mind. Please just leave it open for the time being. Thank you for the effort.

After many pushes and force pushes I got a simple CI working the way I like. Thank you for the example.


The private key doesn't show up until the end

Account Information
Version: 0.13.2
Maximizing the screen doesn't show the last two characters.

Presentation changed for same as for public key (copy button added).
Fixed in https://github.com/nemgrouplimited/symbol-desktop-wallet/releases/tag/v0.13.5


Review editorial on data availability

Read:
https://www.tandfonline.com/doi/full/10.1080/26939169.2022.2141001
https://www.tandfonline.com/action/authorSubmission?show=instructions&journalCode=ujse21
https://authorservices.taylorandfrancis.com/data-sharing/share-your-data/data-availability-statements/

And https://jasa-acs.github.io/repro-guide/

Read https://www.tandfonline.com/doi/full/10.1080/26939169.2022.2074582


Activate gil release/call macros

This is a follow up to the earlier pull request (#265) updating the gil releaser code. Have been testing that with scheduler executions in multiply spawned threads for a while now and all seems ok.
Tests still pass (including the gil exercise).

I'll follow up this pull request with some code that actually makes good use of this shortly :)

trusting you :)


paths schema check and readme update

This PR adds a github workflow to check paths.json files agains the paths.schema.json.
It also updates the readme to include a "Path" section.

For context:

The PR/Discussion around adding path metadata: https://github.com/cosmos/chain-registry/pull/206
Akash is the only chain that currently has a paths.json file.

Adding paths.json files for other chains is in the works. Hoping for help from community as well.

For context:

The PR/Discussion around adding path metadata: https://github.com/cosmos/chain-registry/pull/206
Akash is the only chain that currently has a paths.json file.

Adding paths.json files for other chains is in the works. Hoping for help from community as well.

Closing in favor of: https://github.com/cosmos/chain-registry/pull/319


License compatibility with MIT or Apache 2.0

Hello,
libcsp seems to be under LGPL 2.1.  FreeRTOS, OTOH, is under MIT and doesn't support dynamic linking.  IANAL, but this seems incompatible to me.  Does libcsp offer exception for static linking?
Of course it's perfectly fine for libcsp project to use LGPL.  There is nothing wrong with it.  I just wanted to make sure that we are aware of it.
I'd be very nice if libcsp project add an exception saying that users can link against free software.
https://www.gnu.org/licenses/gpl-faq.html#GPLIncompatibleLibs
https://en.wikipedia.org/wiki/GPL_linking_exception

Sure we can add that. To be frank I don't care what you use it for. You seem to be into this, maybe you can come up with a suggestion to a better licence. We just want the most open license that still maintains the original licence text.

Thanks! I'll prepare an PR, then.

Sure we can add that. To be frank I don't care what you use it for. You seem to be into this, maybe you can come up with a suggestion to a better licence. We just want the most open license that still maintains the original licence text.

I have similar question. I need to link against not open project to. v2.0 has MIT license but v1.6 branch, which is till most popular version, has LGPL.   Can I assume 1.6 is MIT too?


Upstart Service Provider doesn't parse status correctly.

Adding description. Please be patient.

Upon further investigation I think I'm wrong. Sorry for the noise.


Backward compatibility

[x] Define Fybrik release name strategy #1670
[x] Define Backward compatibility requirements #1675
[x] Check what which changes can be done in a CRD without changing the CRD schema (Support backward compatibility of CRDs with different schemas can be done with Webhooks. In some Fybrik deployments, Webhooks usage is prohibited.)
[x] Check taxonomy work for different CRDs with the same schema.
[ ] Define Connectors (REST API) versioning #1677
[x] Define versioning of IT config policies #1678
[x] Define versioning of IT attributes. #1679
[x] Define Fybrik upgrade procedure  #1379

Multi-cluster is a dependency (Razee). We don't define the interface.

Implementation of versioning of connectors REST APIs was discussed with our clients, but was put on-hold for now.
Therefore, I'm closing the epic as done.


Add custom payload size

Add configuration to freely input payload size for both primary and secondary adapters

Make sure to handle cases where payload used for some other usage.
For example, on secondary adapter for mongo, the payload is to indicate the collection being accessed.
Handle it by either disabling multi collection for mongo, or add other fields

Parse payload size and add it to adapter fields


Fixes printk.

Previously the printk macro lacks a const modifier to actually work, resulting in print messages being skipped.
This adds that needed const :)
Signed-off-by: Weilong Cui<EMAIL_ADDRESS>

Coverage decreased (-0.02%) to 37.069% when pulling de35bf8d7fe47732d65ffa142c93184924d9c7e5 on Weil0ng:printk into f5f80c42ec22d394aeea0fc3a0788ba6cd34cf76 on cilium:master.

test-me-please

Friendly ping @joestringer :)

I think we're good to go without Joe's review here :-)


issue fix

https://github.com/Azure/azure-sdk-for-js/issues/25857

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


achivement event

Add an event that detects the completion of an achivement so I can give a reward for completion

Sounds good, I'll add it to the planned features

If someone else needs an event to receive achievements, I implemented it in my fork of this mod (release 2.4.8) (this is written in more detail in the Readme).:


Can't see last day

Hi, i work on your code, and i can't see the last day when i read my tacho card.
When i use cardpeek i see my last work day, but not with your code

Hi, i work on your code, and i can't see the last day when i read my tacho card.
When i use cardpeek i see my last work day, but not with your code

it's ok i found the problem in the code. i can see the last day now.


fix: add missing required property in template schema

Hey, I just made a Pull Request!

add missing required property in template schema as required here

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Related Issue
Closes #20571

Thanks! :tada:


Unable to invoke transaction that interacts with a PDC using fabric-gateway

I have a simple chaincode setup which writes to a single PDC. The collections config is defined as follows
[
    {
      "name": "CollectionOne",
      "policy": "OR('Org1MSP.member')",
      "requiredPeerCount": 0,
      "maxPeerCount": 1,
      "blockToLive": 0,
      "memberOnlyRead": false
    }
]

And the chaincode method is
    async createPrivateAsset(ctx, uuid) {
        const collection = "CollectionOne"
        const privateAsset = {};
        const transientData = ctx.stub.getTransient();
        privateAsset.content = transientData.get('content').toString('utf8');
        await ctx.stub.putPrivateData(collection, uuid, Buffer.from(JSON.stringify(privateAsset)));
    }

using the 2.2 node sdk I can submit a transaction and it get's committed without issue. However when I perform the same using the fabric-gateway it performs the initial endorse without any problem but then returns the following error message
EndorseError: 9 FAILED_PRECONDITION: no combination of peers can be derived which satisfy the endorsement policy: no peer combination can satisfy the endorsement policy

turning on gateway debug doesn't provide any further detail in the peer logs
2022-04-13 10:07:25.361 UTC 009f DEBU [gateway] func1 -> Found local endorser pkiid=3a3ced39e536baae3c8f68dff67499073d918894a63ee36f8d2f18475976064c
2022-04-13 10:07:25.361 UTC 00a0 DEBU [gateway] planFromFirstEndorser -> Sending to first endorser: MSPID=Org1MSP endpoint=peer0.org1.example.com:7051
2022-04-13 10:07:25.368 UTC 00a1 INFO [endorser] callChaincode -> finished chaincode: fixed-asset duration: 16ms channel=mychannel txID=22c60fa0
2022-04-13 10:07:25.379 UTC 00a8 ERRO [gateway] endorsementPlan -> PeersForEndorsement failed. error="no peer combination can satisfy the endorsement policy" errorVerbose="no peer combination can satisfy the endorsement policy\ngithub.com/hyperledger/fabric/discovery/endorsement.(*endorsementAnalyzer).computeEndorsementResponse\n\t/go/src/github.com/hyperledger/fabric/discovery/endorsement/endorsement.go:174\ngithub.com/hyperledger/fabric/discovery/endorsement.(*endorsementAnalyzer).PeersForEndorsement\n\t/go/src/github.com/hyperledger/fabric/discovery/endorsement/endorsement.go:101\ngithub.com/hyperledger/fabric/internal/pkg/gateway.(*registry).endorsementPlan\n\t/go/src/github.com/hyperledger/fabric/internal/pkg/gateway/registry.go:55\ngithub.com/hyperledger/fabric/internal/pkg/gateway.(*Server).planFromFirstEndorser\n\t/go/src/github.com/hyperledger/fabric/internal/pkg/gateway/api.go:351\ngithub.com/hyperledger/fabric/internal/pkg/gateway.(*Server).Endorse\n\t/go/src/github.com/hyperledger/fabric/internal/pkg/gateway/api.go:172\ngithub.com/hyperledger/fabric-protos-go/gateway._Gateway_Endorse_Handler.func1\n\t/go/src/github.com/hyperledger/fabric/vendor/github.com/hyperledger/fabric-protos-go/gateway/gateway.pb.go:1110\ngithub.com/hyperledger/fabric/internal/peer/node.unaryGrpcLimiter.func1\n\t/go/src/github.com/hyperledger/fabric/internal/peer/node/grpc_limiters.go:49\ngithub.com/grpc-ecosystem/go-grpc-middleware.ChainUnaryServer.func1.1.1\n\t/go/src/github.com/hyperledger/fabric/vendor/github.com/grpc-ecosystem/go-grpc-middleware/chain.go:25\ngithub.com/hyperledger/fabric/common/grpclogging.UnaryServerInterceptor.func1\n\t/go/src/github.com/hyperledger/fabric/common/grpclogging/server.go:92\ngithub.com/grpc-ecosystem/go-grpc-middleware.ChainUnaryServer.func1.1.1\n\t/go/src/github.com/hyperledger/fabric/vendor/github.com/grpc-ecosystem/go-grpc-middleware/chain.go:25\ngithub.com/hyperledger/fabric/common/grpcmetrics.UnaryServerInterceptor.func1\n\t/go/src/github.com/hyperledger/fabric/common/grpcmetrics/interceptor.go:31\ngithub.com/grpc-ecosystem/go-grpc-middleware.ChainUnaryServer.func1.1.1\n\t/go/src/github.com/hyperledger/fabric/vendor/github.com/grpc-ecosystem/go-grpc-middleware/chain.go:25\ngithub.com/grpc-ecosystem/go-grpc-middleware.ChainUnaryServer.func1\n\t/go/src/github.com/hyperledger/fabric/vendor/github.com/grpc-ecosystem/go-grpc-middleware/chain.go:34\ngithub.com/hyperledger/fabric-protos-go/gateway._Gateway_Endorse_Handler\n\t/go/src/github.com/hyperledger/fabric/vendor/github.com/hyperledger/fabric-protos-go/gateway/gateway.pb.go:1112\ngoogle.golang.org/grpc.(*Server).processUnaryRPC\n\t/go/src/github.com/hyperledger/fabric/vendor/google.golang.org/grpc/server.go:1180\ngoogle.golang.org/grpc.(*Server).handleStream\n\t/go/src/github.com/hyperledger/fabric/vendor/google.golang.org/grpc/server.go:1503\ngoogle.golang.org/grpc.(*Server).serveStreams.func1.2\n\t/go/src/github.com/hyperledger/fabric/vendor/google.golang.org/grpc/server.go:843\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1581" channel=mychannel ChaincodeInterest="chaincodes: <\n  name: \"fixed-asset\"\n  collection_names: \"CollectionOne\"\n>\n"

A workaround is to explicitly set the endorsement policy in the collections-config file as follows
[
    {
      "name": "CollectionOne",
      "policy": "OR('Org1MSP.member')",
      "requiredPeerCount": 0,
      "maxPeerCount": 1,
      "blockToLive": 0,
      "memberOnlyRead": false,
      "endorsementPolicy": {
        "signaturePolicy": "OR('Org1MSP.member')"
      }
    }
]

endorsementPolicy is optional however so should work without having to specify

The gateway is doing the right thing here, it's the node sdk 2.2 use that wasn't. Because the endorsement policy was the default one and it was a 2 org network, the node sdk is by default sending to both orgs and thus able to satisfy the endorsement policy but means that it's sending potentially sensitive data to an org that shouldn't. The gateway is protecting against this which results in the message seen (although if possible it would be nice for the gateway to say why it couldn't satisfy, ie it won't send to an org that shouldn't see the info).
The above collections-config should be used and in fact suggests that collections-config should be explicit about the required endorsement policy otherwise the chaincode endorsement policy may not be what is required.


Example heading in itertools package should be h4

This PR changes the heading types for all Examples in the itertools/iter.pony file to be
 instead of

Previously, ## Example got translated to
Example
, which was problematic for two reasons:

The method itself has a lesser heading (
)

 is visible in the Table of Contents

 still has an anchor link, and is no longer visible in the ToC, see screenshot of local mkdocs instance after making the change:

Thanks!

Thanks!


"reactions_add" and "reactions_remove" not allowing some emoji names ending in "_tick" or custom emojis

Reproducible in
The Slack SDK version
name         : slack-sdk
version      : 3.33.3
description  : The Slack API Platform SDK for Python
Python runtime version
Python 3.12.7
OS info
#134~20.04.1-Ubuntu SMP Tue Oct 1 15:27:33 UTC 2024
Steps to reproduce:

Any emoji whose name ends with "_tick" will fail to be added as a message reaction.
Other emojis appear to work as expected.
Manually using these emojis in slack works as expected and all of those emojis are available to us.
I just added some custom emojis and these also fail so only internal emojies may work but I don't know why that would be the case.

    for name in [
        "white_tick",
        "heavy_tick",
        "ballot_box_with_tick",
        "negative_squared_tick",
    ]:
        response = client.reactions_add(
            channel=channel_id, timestamp=timestamp, name=name
        )
        time.sleep(2)
        response = client.reactions_remove(
            channel=channel_id, timestamp=timestamp, name=name
        )

Expected result:
{'ok': True}
The emoji is added as a reaction to the specified message
Actual result:
{'ok': False, 'error': 'invalid_name'}
Requirements
For general questions/issues about Slack API platform or its server-side, could you submit questions at https://my.slack.com/help/requests/new instead. :bow:
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Hi @yoyo-circleci, thanks for asking this! It seems the API server accepts the following ones instead:

"white_check_mark"
"heavy_check_mark"
"ballot_box_with_check"
"negative_squared_cross_mark"

I haven’t checked the "_tick" ones that used to be accepted on the API server in the past, but for now, please go ahead with these different names. If you need further clarification on the API server behavior, please contact our customer support agents for more details. Thanks again for writing in.