Forward/Back Buttons and Keyboard Shortcuts

It's so easy to jump between pages in this wonderful tool, but excruciatingly painful to go back to the page (note) you just came from.  Could you please add a History, together with Back and Forward buttons and corresponding keyboard shortcuts?
This is a feature request.  (Unless it already exists and I just missed it...?)

Navigation and History button is really needed. Let me know I will like to work on this


Guardian.Plug.sign_in -> "argument error"

When i try to use Guardian.Plug.sign_in while testing, terminal tell me this

** (UndefinedFunctionError) function Guardian.Plug.sign_in/2 is undefined or private. Did you mean one of:

       * sign_in/3
       * sign_in/4
       * sign_in/5

but when i write before, in the top of file

use Guardian.Plug

The sign_in/2 is available, and i see this

 ** (ArgumentError) argument error

Here is my Api.Auth.login

def login(conn, user) do
conn
|> sign_in(user)
|> assign(:current_user, user)
end

Here is config
  config :api, Api.Auth.Guardian,   issuer: "api",   secret_key: "HNinpKh9Ne3tr8BpjCpAEh0xzCqTIG3PWsfkR2AtzvUaRIpbs6oIQ9RcmjmGPekJ"
And guardian callbacks is:

defmodule Rumbl.Auth.Guardian do

     use Guardian, otp_app: :api

     def subject_for_token(user, _claims) do
          sub = to_string(user.id)
          {:ok, sub}
      end

      def resource_from_claims(claims) do
          id = claims["sub"]
          user = Api.Repo.get(Api.User, id)
          {:ok, user}
       end

end

I read guide step by step, but something went wrong...

Maybe I can help you out (I'm not sure if this is 100% correct). You don't have to put use Guardian.Plug on top if the file. Remove that line, add alias Rumble.Auth.Guardian and call sign_int with the command Guardian.Plug.sign_in(user).
That worked in my case in https://github.com/hlhr/phoenix_authentication/blob/master/lib/phoenix_authentication_web/controllers/auth_controller.ex .

This work !! thanks ) I will close issue


Create harbour-chatgpt-pl.ts

@likot180 What happened, why close it?
My mistake.

Also consider joining the Transifex project if you'd like to help translating in the future (though you can always make a PR like this one if you prefer).


Add inject_evdev application

inject_evdev.zip
Minimum build without sentry enabled, and including the required library.

Example usage: echo -e "EV_ABS ABS_MT_TRACKING_ID -1\nEV_SYN SYN_MT_REPORT\n EV_SYN SYN_REPORT" | inject_evdev event2

Doesn't that duplicate what's done in iago and lamp ? Or is it for testing the event handling code ?

Not at all, this allows injecting fake evdev events via any script.
iago is for drawing shapes, so while it does inject pen events, it first has a UI for you to position the shape before drawing. This serves a very specific purpose, which inject_evdev does not.
lamp is just for listening to touch events and running things based on gestures.


Improve document for BUILD-WINDOWS.md

Just following the document BUILD-WINDOWS.md cannot build lsquic. So,  I need to configure every header path and lib path manually as follows, I hope document can be improved or providing builded libs.
vcpkg install zlib:x64-windows-static
vcpkg install libevent:x64-windows-static
vcpkg install pcre:x64-windows-static

vcpkg integrate install

##BoringSSL Debug
cmake -DCMAKE_GENERATOR_PLATFORM=x64 --config Debug -DBUILD_SHARED_LIBS=OFF -DOPENSSL_NO_ASM=1 .
msbuild ALL_BUILD.vcxproj

##BoringSSL Release
cmake -DCMAKE_GENERATOR_PLATFORM=x64 --config Release -DBUILD_SHARED_LIBS=OFF -DOPENSSL_NO_ASM=1 .
msbuild ALL_BUILD.vcxproj

##Debug
set include=X:\QUIC\vcpkg\installed\x64-windows-static\include
set vclib=X:\QUIC\vcpkg\installed\x64-windows-static\debug\lib

cmake -DCMAKE_GENERATOR_PLATFORM=x64 -DBUILD_SHARED_LIBS=OFF   -DVCPKG_TARGET_TRIPLET=x64-windows-static -DCMAKE_BUILD_TYPE=Debug  -DPCRE_LIB=%vclib%\pcred.lib -DPCREPOSIX_LIB=%vclib%\pcreposixd.lib  -DZLIB_INCLUDE_DIR=%include%  -DZLIB_LIB=%vclib%\zlibd.lib -DEVENT_INCLUDE_DIR=%include% -DEVENT_LIB=%vclib%\event.lib -DCMAKE_TOOLCHAIN_FILE=X:/QUIC/vcpkg/scripts/buildsystems/vcpkg.cmake   -DBORINGSSL_LIB_decrepit=X:\QUIC\boringssl\decrepit\Debug\decrepit.lib -DBORINGSSL_LIB_ssl=X:\QUIC\boringssl\ssl\Debug\ssl.lib -DBORINGSSL_LIB_crypto=X:\QUIC\boringssl\crypto\Debug\crypto.lib .

msbuild ALL_BUILD.vcxproj

##Release
set include=X:\QUIC\vcpkg\installed\x64-windows-static\include
set vclib=X:\QUIC\vcpkg\installed\x64-windows-static\lib

cmake -DCMAKE_GENERATOR_PLATFORM=x64 -DBUILD_SHARED_LIBS=OFF   -DVCPKG_TARGET_TRIPLET=x64-windows-static -DCMAKE_BUILD_TYPE=Release  -DPCRE_LIB=%vclib%\pcre.lib -DPCREPOSIX_LIB=%vclib%\pcreposix.lib  -DZLIB_INCLUDE_DIR=%include%  -DZLIB_LIB=%vclib%\zlib.lib -DEVENT_INCLUDE_DIR=%include% -DEVENT_LIB=%vclib%\event.lib -DCMAKE_TOOLCHAIN_FILE=X:/QUIC/vcpkg/scripts/buildsystems/vcpkg.cmake   -DBORINGSSL_LIB_decrepit=X:\QUIC\boringssl\decrepit\Release\decrepit.lib -DBORINGSSL_LIB_ssl=X:\QUIC\boringssl\ssl\Release\ssl.lib -DBORINGSSL_LIB_crypto=X:\QUIC\boringssl\crypto\Release\crypto.lib .

using lsquic.sln to do ALL_BUILD

Thank you for the bug report and the suggested updates.  You are right: these instructions are out of date.  We will fix the document.
(To build on Windows, the authoritative steps are listed in appveyor.yml, which successfully builds on Windows.)

appveyor-windows.yml has been updated per #350
There are additional steps needed to build a shared library (dll) but CMake flags -DBUILD_SHARED_LIBS=ON -DLSQUIC_SHARED_LIB=ON -DVCPKG_TARGET_TRIPLET=x64-windows -DCMAKE_WINDOWS_EXPORT_ALL_SYMBOLS=ON is a good starting point.
I'd like to contribute an appveyor-windows-shared.yml to build it if possible @litespeedtech @dtikhonov


Add rows icon

https://lucide.dev/icons?search=columns

Icon

Moved to #1003…

This was closed automatically because of #1003


Feat errormsg jwtexpired

alteração do retorno "jwt expired" para uma mensagem personalizada no arquivo JWT.ts.

Codecov Report

Merging #237 (2f04e28) into develop (e36e3a6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #237   +/-   ##
========================================
  Coverage    95.57%   95.57%
========================================
  Files           44       44
  Lines         1310     1310
  Branches        77       77
========================================
  Hits          1252     1252
  Misses          44       44
  Partials        14       14

Flag
Coverage Δ

integrationtest
95.57% <ø> (ø)

unittests
95.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


CHROME Manifest file is missing or unreadable Could not load manifest.

the Chrome manifest does not load properly and I cannot install the extension on Chrome

Hi @lorenzoleandro, thanks for reaching out.
Notero is a plugin for Zotero rather than Chrome. You can find installation instructions in the README that describe how to install it in Zotero.
Let me know if you have any more questions!


Failed to check IAM role name

s3fs syslog messages (grep s3fs /var/log/syslog, journalctl | grep s3fs, or s3fs outputs)
2024-01-26T09:37:02.945Z [INF] s3fs.cpp:s3fs_init(3382): init v1.90(commit:unknown) with GnuTLS(gcrypt)
2024-01-26T09:37:02.949Z [CRT] s3fs.cpp:s3fs_check_service(3520): Failed to check IAM role name(my-role).
Details about issue
I have an ec2 in region us-east-1
The s3 bucket is in eu-east-2
I'm trying to run s3fs using IAM role, but it seems to failed on permissions.
I was trying to give different access without any luck.
Please advise
Additional Information
Version of s3fs being used (s3fs --version)
Amazon Simple Storage Service File System V1.90 (commit:unknown) with GnuTLS(gcrypt)
Copyright (C) 2010 Randy Rizun<EMAIL_ADDRESS>License GPL2: GNU GPL version 2 https://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Version of fuse being used (pkg-config --modversion fuse, rpm -qi fuse or dpkg -s fuse)
Kernel information (uname -r)
6.2.0-1017-aws
GNU/Linux Distribution, if applicable (cat /etc/os-release)
PRETTY_NAME="Ubuntu 22.04.3 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.3 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy
How to run s3fs, if applicable
[] command line
sudo s3fs :<bucket_path> <local_path> -o iam_role=my-role -o endpoint=us-east-1 -f
[] /etc/fstab

The problem related to the IMDSv2 that was enabled on my ec2 machine

This was for the linux systems app "s3fs", not the python package "s3fs" :)


Show a message on git commit in the terminal when git.terminalGitEditor is true

Something like this:

Info: Git has been automatically configured to use VS Code as the commit editor. To disable, switch git.terminalGitEditor to false.

If we can have escape sequences there we could style it like this to indicate the message is coming from VS Code:

Seems this setting somehow got turned on by default? It was a little confusing today when my message editor suddenly became vscode instead of vim. Feels like this should be opt-in rather than opt-out.

@jpcastberg we had that discussion internally as I was more on your side. While we want the terminal to behave like other terminals, we expect the experience to generally be better in the editor so with the message proposed I'm much more comfortable going out with this and collecting feedback, since it's very obvious why it happened and is easy to switch off.
We're also exploring contextual actions in the terminal which could make it even easier to turn off.

Yes it's only in Insiders right now, I think the plan is to have it on by default in Stable in the upcoming version, but with the message. Thanks for the feedback

After further discussions within the team we have decided to change the default value of the git.terminalGitEditor setting to false. Out of the box, users will still be able to use the full editor to enter a commit message when using the “Commit” action in the source control viewlet, and they will be able to enable the terminal integration if they toggle the git.terminalGitEditor setting. This change will be included in Monday's Insiders release (2022-06-27).


cant get armor mods to load.

This is how I have everything located on my steam deck except modengine.toml I could not find that at all. also I put chr, menu, parts in the mod folder as if modengine2 was working by its self. I am not sheer if this is how its suppose to work. realistically I wish I could just use modengione2 by itself I got it to work wonderfully on my pc but with my steam deck in I really wanted to get it to work on there. Any help would be greatly appreciated.
ELDEN RING
└── Game
|
├── mods
│ ├── lua.dll
│ ├── modengine2.dll
│ ├──chr
│ ├──menu
│ ├──parts
|
├── movie
├── sd
├── amd_ags_x64.dll
├── bink2w64.dll
├── dinput8.dll
├── eldenring.exe
├── eossdk-win64-shipping.dll
├── eossdk-win64-shipping.so
├── mod_loader_config.ini
├── modengine.toml
├── oo2core_6_win64.dll
├── regulation.bin
├── start_protected_game.exe
└── steam_api64.dll

I have sens edeted config_eldenring.toml to modengine.toml pointing it to randomizer folder I have also changed mod_loader_config timer to 2500 insted of 5000.
I am still unable to load mods that are in the sub folder under randomizer.
ELDEN RING
└── Game
├── SeamlessCoop
│   └── cooppassword.ini
├── mods
│   ├── lua.dll
│   ├── modengine2.dll
│   ├──chr
│   ├──menu
│   ├──parts
├── movie
├── randomizer
│     ├──chr
│     ├──menu
│     ├──parts
├── sd
├── amd_ags_x64.dll
├── bink2w64.dll
├── dinput8.dll
├── eldenring.exe
├── eossdk-win64-shipping.dll
├── eossdk-win64-shipping.so
├── mod_loader_config.ini
├── modengine.toml
├── oo2core_6_win64.dll
├── regulation.bin
├── start_protected_game.exe
└── steam_api64.dll

I got it to work Thank you for the code this is awesome.

can you explain how you got it to work ?
thanks


URLs of blogs changes to English even if I type the url in Hindi  or other languages

Issue Summary

I typed the url exactly as in the title of the blog. But it automatically converted to English.
I want the urls to be  exactly similar to the title.
Steps to Reproduce

Go to ghost admin
Try to write a new post with title in Hindi.
Paste the Hindi title in  the right side drawer for url creation.
You can see that it automatically converts that to English

Ghost Version
5.7
Node.js Version
18
How did you install Ghost?
cli
Database type
SQLite3
Browser & OS version
chrome
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

This has been a recurring request and the latest web standards support non-latin characters in URLs. It is aligned with Ghost's goal to offer the platform in multiple languages now. Here's what I posted in the forum about this:

It used to be that such characters were not allowed in URLs, according to RFC 3986.
But according to the newer WHATWG standard for URLS, Unicode is allowed in URLs if the code points are in the range U+00A0 to U+10FFFD.
It appears the code points you tried to use were these two:
Hangul Syllable Hel (U+D5EC)
Hangul Syllable Ro (U+B85C)
It seems to me like they would be in the valid range.
To me, this seems like be a bug in Ghost if Ghost is still creating URLs based on the older RFC 3986 standard for URL formats and not the WHATWG URL spec.
Ghost might be able to replace it’s current URL building library based on the old standard with one based on the new standard, like whatwg-url - npm 2
That's from:  https://forum.ghost.org/t/can-i-not-use-english-on-post-url/40348
See Also:

https://forum.ghost.org/t/to-turn-off-automatic-url-translation-into-latin/1817
https://forum.ghost.org/t/how-to-use-multi-language-urls/33576


Blender rig controls

Have you considered additional rig controls for the UMA base in blender?
I'd be happy to share some suggestions or a PR.

There is infinite room for a new UMA base. I've had a few swipes at it myself, but there are so many possible routes to take.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Frostebite @.>
Sent: Friday, June 18, 2021 10:37:30 PM
To: umasteeringgroup/content-pack @.>
Cc: Subscribed @.***>
Subject: [umasteeringgroup/content-pack] Blender rig controls (#5)
Have you considered additional rig controls for the UMA base in blender?
I'd be happy to share some suggestions or a PR.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fumasteeringgroup%2Fcontent-pack%2Fissues%2F5&data=04|01||81f6349852124bcbf6bd08d932a14709|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637596490523230387|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=0QQfu6HZHE%2BXQohoq4Bd60Yv115Q%2FARckZ%2BIDpdfsSQ%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACYN4PYXDQAHJFX5DZ2YCHLTTO4BVANCNFSM466LNNZA&data=04|01||81f6349852124bcbf6bd08d932a14709|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637596490523230387|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=M%2Fpw%2FQREH9I%2BXETmg%2FXp9WB8Ll2OxpxjtXdpFahyQs0%3D&reserved=0.

hey is there a way to export uma avatar from unity with textures and maybe import to blender?

Maybe best to have questions on Discord?
https://discord.gg/FN4rfUnU
But in short, the content pack has the base models: https://github.com/umasteeringgroup/content-pack/tree/master/ContentPack_<IP_ADDRESS>/Blender_28
To export to fbx requires either using the Unlogick FBX exporter (asset store),  or converting to a prefab using the prefab maker, and then using the Unity FBX Exporter (and fixing any issues).


Mqtt Changes from HA

Hi,
I have seen the the HA has a new Warning.

What is hier to do?
Regards and keep up the good work.
Adi

Are you using my integration or the auto-discovery feature?

As the addin/inegration entities seem to be fine (see closed report above) we might have a problem here.

This piece of code is not mine and I don't understand exactly the new naming conventions adopted.
But please, try this script:
mqtt_adv_homeassistant.sh.gz
Overwrite the current file in /tmp/sd/yi-hack/script/mqtt_advertise and let me know if it works.

Just checked my Home Assistant and the warning is gone now. I did nothing regarding your script yet. A recent HA update must have done something. @AditaNi do you still have the message?

Recent Beta passed by and nothing was showing up again. So I guess this one was fixed in a different way or will show up later.

Hi,
just to confirm the issue is still here (HA 2023.9). Here's the log line:
2023-09-07 16:16:08.734 WARNING (MainThread) [homeassistant.components.mqtt.mixins] MQTT entity name starts with the device name in your config {'availability_topic': 'yicam/status', 'payload_available': 'online', 'payload_not_available': 'offline', 'device': {'identifiers': ['yi-cam'], 'connections': [['mac', 'b4:c9:b9:9b:b7:9a']], 'manufacturer': 'YI', 'model': 'YI Hack', 'name': 'Yi Camera', 'sw_version': '0.2.9', 'configuration_url': 'http://<IP_ADDRESS>:8080'}, 'icon': 'mdi:network', 'state_topic': 'yicam/info_global', 'name': 'Yi Camera Hostname', 'unique_id': 'yi-cam-hostname', 'value_template': Template<template=({{ value_json.hostname }}) renders=0>, 'entity_category': <EntityCategory.DIAGNOSTIC: 'diagnostic'>, 'force_update': False, 'encoding': 'utf-8', 'availability_mode': 'latest', 'qos': 0, 'enabled_by_default': True}, this is not expected. Please correct your configuration. The device name prefix will be stripped off the entity name and becomes 'Hostname'
Want me to test some fixes?
Thanks

Hey, I've tested the script you attached and the warnings are gone!
The last warning I have is:
 Entity sensor.yi_camera_wlan_strengh (<class 'homeassistant.components.mqtt.sensor.MqttSensor'>) is using native unit of measurement '%' which is not a valid unit for the device class ('signal_strength') it is using; expected one of ['dB', 'dBm']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+mqtt%22
Thanks!

Found a solution for the problem above:
Change line 235
CONTENT='{"availability_topic":"'$MQTT_PREFIX'/'$TOPIC_BIRTH_WILL'","payload_available":"'$BIRTH_MSG'","payload_not_available":"'$WILL_MSG'","device":'$DEVICE_DETAILS','$QOS' '$RETAIN' "device_class":"signal_strength","icon":"mdi:wifi","state_topic":"'$MQTT_PREFIX'/'$MQTT_ADV_TELEMETRY_TOPIC'","name":"'$UNIQUE_NAME'","unique_id":"'$UNIQUE_ID'","value_template":"{{ ((value_json.wlan_strength|int) * 100 / 70 )|int }}","platform": "mqtt","unit_of_measurement":"%","entity_category": "diagnostic"}'
to this
CONTENT='{"availability_topic":"'$MQTT_PREFIX'/'$TOPIC_BIRTH_WILL'","payload_available":"'$BIRTH_MSG'","payload_not_available":"'$WILL_MSG'","device":'$DEVICE_DETAILS','$QOS' '$RETAIN' "icon":"mdi:wifi","state_topic":"'$MQTT_PREFIX'/'$MQTT_ADV_TELEMETRY_TOPIC'","name":"'$UNIQUE_NAME'","unique_id":"'$UNIQUE_ID'","value_template":"{{ ((value_json.wlan_strength|int) * 100 / 70 )|int }}","platform": "mqtt","unit_of_measurement":"%","entity_category": "diagnostic"}'

So they just hid the error/warning... I must say that I did not check the logs, only waited for the message to appear.

So they just hid the error/warning... I must say that I did not check the logs, only waited for the message to appear.

Nope, error was still there. After thr fixes I mentioned, it was gone.

I am talking about the notification and the message in the settings menu. Both are not there anymore for me. Anyways, your fix should get this fixed and everything is fine then. :D


Parameterized file reader

This updates cryptol-verifier to still work with the changes made in https://github.com/GaloisInc/cryptol/pull/793.

These changes were made based on the commit in the submodule in saw-script, and it seems to include things that are not in master here. How should I proceed?

It looks like @robdockins did that with GaloisInc/saw-script@680edad966fe67544a3eb54ffaa028c940ca5cc0. It looks like we need to merge the split-arith branch of cryptol-verifier into master. It looks like split-arith actually was merged into master at one point (with #17), but maybe Rob kept pushing to that branch afterward?

I think that all the relevant code changes eventually got rebased into master, although I guess it's possible I missed something along the way.  That rebase got pretty hairy.
I think you should be able to rebase this branch onto master without too much trouble, or maybe just cherry-pick your commit.

I was really confused looking at the saw-script git history, because I was trying to see which PR Rob's "update submodules" commit (GaloisInc/saw-script@680edad) was part of, and there wasn't any merge commit at all, even though it looks like all the other PRs have merge commits.
Helpfully, github did inform me after clicking on the hyperlink for the commit that it was part of GaloisInc/saw-script#732, so I was able to figure it out. But we should probably be consistent about using merge commits so that people like me can actually make sense of the git history.

@robdockins which branch in which repo are you talking about rebasing? You definitely should not rebase the cryptol-verifier split-arith branch, as this (b46aed94) is the commit that the saw-script repo points to as a submodule.

The saw-script module points to an intermediate code change that has since been superseded by 57d90db93e7fabf7d4e2ae3ba97835135e21efd5, which is on cryptol-verifier/master.  This parameterized-file-reader branch should be rebased onto of cryptol-verifier/master.
I was waiting a bit to see if Cryptol would quiesce before trying to get a saw-script PR through again, so the submodule lags a bit.

Commit 57d90db is on the split-cmp branch, and it has not been merged to master either.

Sorry, 2fc49ce9ccc3e259a8f20d258cb8c3078b607dde.  I usually prefer to rebase PRs into master, as it retains a linear history, but it does make situations like this a bit trickier to untangle.

Ok, here's my current thinking: First of all, it is absolutely imperative that b46aed9 should never be garbage-collected and disappear from the cryptol-verifier repo, as a master version of saw-script points to this very revision as a submodule, and it is vital that all master versions of saw-script should continue to compile and work correctly.
So what this means is that we need to merge b46aed9 into master, even if all the changes in that branch have already been rebased into master by some other means. I'll make a PR to do that, and then once that's done we can start fixing everything else.

Just to complicate things a little more, cryptol master was updated in a way that breaks cryptol-verifier master.  PR #22 is pending to fix cryptol-verifier for that cryptol change.

I think you can tag that commit instead of merging to master if you just want to make sure it doesn't get GCd.

It sounds to me like the right approach is for me to wait a bit while things settle down, then revisit this as a need for a release of the Python client approaches. Or am I misunderstanding?

I'll defer to @brianhuffman here, but I've been trying to land patches into cryptol-verifier in the same order they land in cryptol, so probably #22 should come first.

@david-christiansen I've finally straightened out all the cryptol-verifier patches, and applied them in the right order. The current master version of cryptol-verifier (f33f4f234da9f48664f61bd9492b09259c78b60f) descends from the revision you built on (b46aed9412cbdc3313e41095283dd627da28bb7e), and also now works with the current master version of cryptol (GaloisInc/cryptol@6de2283acefe6da562c017f73338d172c79f4f6e).
I also have a saw-script PR that makes saw-script work with the latest cryptol and cryptol-verifier (GaloisInc/saw-script#764).

Thanks!
I suppose the right thing for me to do is to wait for those to get merged in, and then rebase my PR collection on top of them.

@brianhuffman, I think everything is straightened out now, right? I think this should be good to merge.

Yeah, it should be good to go.

I'm a little unsure of the usual practices on this repo regarding who pushes the merge button. But @kquick pointed out that it's ready to go, so I'll take that as me being the right one now that there's an approval stamp :)


Option to use serialized names instead of Go names for fields

Sometimes we want documentation generated using the field names as present in the Go source, but other times we want documentation to use the serialization names that appear in YAML or JSON content.
This would not only impact on the identifiers included in tables, but on sort ordering too.

We do  currently support both json and yaml serialized names. What we lack is the option to use the Go Fieldnames instead.


Add support for Discover floodlight

Tested and verified on my own Discover floodlight.

Device added to the test version of the app, going live on next update.
Thanks for your contribution and effort.


Animations on first Home banners

https://github.com/sanoojes/better-bloom/assets/68474172/5a4af0de-20c3-41f9-b7c5-2fa5fd5b4c05

can you checkout the animations now


misc: minor Client code cleanup

Remove code duplication in the Client class.
Thank you for a great library. Very useful and simple to use.
It's my first pull request as a contributor to an open-source project. Please don't judge harshly. =)

Also, I think it makes more sense to move the _check_response method from the Client class to the ApiResponse class. It's not been used anywhere except for initializing ApiResponse, and though it means to be part of __init__ method for class.

Hi @saleweaver,
Is there something wrong with this PR? Should we discard it and close it? Or maybe update something?
Thank you


[C/PyTorch] Add THD support for cuDNN attention

Description
This PR adds THD support for fused attention (F16_arbitrary_seqlen backend). This feature allows users to run attention for two more cases:
case 1: no padding between sequences
    8 sequences are packed as abcddddeeeffgh
    total sequence length t = 14
    cumulative sequence lengths: cu_seqlens = [0, 1, 2, 3, 7, 10, 12, 13, 14]
    offset tensor that helps cuDNN step into qkv tensors: offset_tensor = [0, 1, 2, 3, 7, 10, 12, 13]

case 2: with padding between sequences
    8 sequences are packed as abc0ddddeee0ffgh
    total sequence length t = 16
    cumulative sequence lengths: cu_seqlens = [0, 1, 2, 3, 7, 10, 12, 13, 14]
    offset tensor that helps cuDNN step into qkv tensors: offset_tensor = [0, 1, 2, 4, 8, 12, 14, 15]

Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Changes

Added THD support to cuDNN attention on the C level and in PyTorch modules. Jax and Paddle are to be worked on.
Updated cudnn-frontend to v1.4.0.
Added tests for max_seqlen_q=1 and head_dim=256 inference cases for thd, bshd, and sbhd formats.

Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

This PR breaks the fused attention API. I'm wondering if we can change cu_seqlens arguments to actual_seqlens, so that we can avoid the kernel to convert actual_seqlens to cu_seqlens.
Actually, cu_seqlens as API makes

I had a commit to fix the JAX build issue with the new API.

This PR changes the fused attention API. I'm wondering if we can also change cu_seqlens arguments to actual_seqlens, so that we can avoid the kernel to convert actual_seqlens to cu_seqlens.
Actually, cu_seqlens as API arguments makes JAX do two additional kernels, which is mask -> actual_seqlens -> sharding (DP) -> kernel to convert actual_seqlens to cu_seqlens -> call nvte_fmha -> kernel to convert cu_seqlens to actual_seqlens.

@zlsh80826 If mask -> cu_seqlens instead of mask -> actual_seqlens, would there still be two extra kernels? @ptrendx suggested that we make the cu_seqlens/actual_seqlens related changes in a different PR in order to keep this PR focused on THD. Any burning needs to make the cu_seqlens/actual_seqlens changes?

/te-ci pytorch

@zlsh80826 If mask -> cu_seqlens instead of mask -> actual_seqlens, would there still be two extra kernels? @ptrendx suggested that we make the cu_seqlens/actual_seqlens related changes in a different PR in order to keep this PR focused on THD. Any burning needs to make the cu_seqlens/actual_seqlens changes?

If changes to mask->actual_seqlens then it will not have two extra kernels. I did profile the performance impact and the extra kenrels are less then 0.1% in the e2e training, so it is not hurry for now.

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

Hi @cyanguwa, the changes for common looks good for me. I will fix the JAX build this week.

/te-ci pytorch

/te-ci jax

/te-ci paddle

/te-ci jax

@cyanguwa I have a few questions when exposing the THD format on JAX. Could you take a look on cudnn slack channel before merging it? Thanks.


Validation of required values should allow empty values

Problem statement
In an object definition, some properties (integer, string, etc.) are defined as "required".
The zero values (0, "") are all valid values of them. However, the go-swagger validation package github.com/go-swagger/go-swagger/httpkit/validate rejects these zero values as invalid.
It's now introducing big problems for my application when my user wants to clear some properties (set to 0, or "", they are all valid values in the application's logic) in the data models, the server simply reject the request as they are invalid.
Swagger specification
swagger: '2.0'
definitions:
  Property:
    description: A property of a resource
    properties:
      propType:
        type: string
        description: discriminating type of the property
      computed:
        type: boolean
        description: is property computed
      intval:
        type: integer
        format: int64
      strval:
        type: string
    required:
      - propType
      - intval
      - strval

The generated code is
swagger:model Property
*/
type Property struct {

	/* is property computed
	 */
	Computed *bool `json:"computed,omitempty"`

	/* Intval intval

	Required: true
	*/
	Intval int64 `json:"intval,omitempty"`

	/* discriminating type of the property

	Required: true
	*/
	PropType string `json:"propType,omitempty"`

	/* Strval strval

	Required: true
	*/
	Strval string `json:"strval,omitempty"`
}

It's unreasonable to put omitempty for all required properties. In the json generated from the model, all these properties get lost with zero values which are valid values.

duplicate of #300 ?

Related, but I would not say "duplicated". Because #300 is the opposite (required fields are pointers). In my situation, the fields generated are not pointers. I feel #300 should be fixed, this bug is related to validation logic, not duplicate of #300

this was fixed in 42bbea36acba818fbc269e19f625316c1a0e15fc


Implemented doctype nodes

This PR implements a new node type named "doctype". It is used for the <!doctype>  tags which works a bit differently than regular tags.
The common tree structure would be:
|- <document node (0)>
   |- <doctype node (1)>
   |- <element node "html" (2)>
   |  |- <element node "head" (3)>
   |  |   |- <element node "title" (4)>
   |- <element node "body" (5)>

Note that the document node (0) is the root node. We use a separate document node type for this, but this isn't really used anywhere except for holding a html tree.
The "document" which we use and create with DocumentBuilder::new_document() is a the structure that holds the node arena and other things. This may be a bit confusing in naming, especially when we will get to the "DOM" document (which is an interface that interacts with our document-tree).
Anyway, for now.. the doctype works, and we are at 92% pass rate.

Would it make sense to repurpose the existing Document node type for this? That's actually what I originally thought it was, I didn't realize it was just a placeholder.
Maybe when creating a new document we have an empty root until we find a doctype token (or force insert a default if not found)
If it's more work than what it's worth, I don't really mind going for this, but just seems a little odd to have both a Document and DocType type

The initial setup was indeed to have a document as a first class node but it turns out we dont really use it. However, there will be some settings (like the quirks mode etc,) that we want to keep in the document because the DOM document interface exposes these properties. I can image that at that point, the document node we have becomes more important


FIX: Prevent misalignment of [Spectrogram] at start and end of sample.

Short description
For Short audio files the spectrogram is only aligned towards the beginning and end of a file. This becomes especially apparent when zooming into the audio/spectrogram.
Resolves https://github.com/katspaugh/wavesurfer.js/discussions/3535
Implementation details

Add 2 empty frequencies to the beginning of the spectrogram for alignment: This was experimental and I am unsure what the theoretical explanation of it working would be. Perhaps someone can explain or come up with an idea?
Add 0 padding to the end of spectrogram if some data was left out due to window and file length mismatch.

Recommended improvements:

Check why 2 seems to be the magic number for front alignment.
Somehow add calculations for those first 2 frequencies.
Optimize function calls to reduce duplicates.

How to test it

Take a short audio file (<20 sec, even better <10s) with pronounced beats / cuts in the the audio, e.g. this heartbeat: heartbeat.zip

Load it into the examples/audio folder

Change the spectrogram example to this code: spectrogram.js.txt

Zoom in to the ends of the spectrogram

So far, the beats do not align, with this pull request they do.

Screenshots
Before

Spectrogram aligns in center

Spectrogram is "late" towards the end of the file.

Spectrogram is "early" towards the beginning of the file.
After

Spectrogram is aligned in the end

Spectrogram is aligned in the beginning
Checklist

[ ] This PR is covered by e2e tests

The umd.cy.js tests fail, but do not seem to be affected by this code change. All others pass

[x] It introduces no breaking API changes

I can't seem to figure out, why the first two sample need to be added to the visualization.
I'd appreciate any input on why they seem to be missing.
This is all that's holding me back, since I don't want to push this and breaking the visualization.

Tbh I don’t know either. The plugin was contributed a long time ago by someone else and I barely touched it since.


Link manifst docs to version info in crates.io docs

Fixes #1707

r? @wycats
(rust_highfive has picked a reviewer for you, use r? to override)

r=me with a relative URL

@bors: r=alexcrichton

:pushpin: Commit 9ff7734 has been approved by alexcrichton

:hourglass: Testing commit 9ff7734 with merge 377feb9...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


samsung-UnifiedLinuxDriver: fix multi-output cups

Motivation for this change
multi-output cups broke dynamic linking of filters (e.g. lib/cups/filter/rastertosamsungspl) in samsung-UnifiedLinuxDriver throwing errors in cups´ log:
Samsung: error while loading shared libraries: libcups.so.2: cannot open shared object file: No such file or directory
Things done
We Adress the issue by having patchelf set the binaries rpath to contain ${cups.out}/lib instead of ${cups}/lib so libcups.so.2 can be found

[X] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)

Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
Breaks due to pkgs/servers/nosql/apache-jena/fuseki-binary.nix:
error: while evaluating ‘callPackageWith’ at lib/customisation.nix:93:35, called from pkgs/top-level/all-packages.nix:9875:24: while evaluating ‘makeOverridable’ at lib/customisation.nix:54:24, called from lib/customisation.nix:97:8: syntax error, unexpected $undefined, at pkgs/servers/nosql/apache-jena/fuseki-binary.nix:27:45

[ ] Tested execution of all binary files (usually in ./result/bin/)

[X] Tested execution of lib/cups/filter/rastertosamsungspl

[X] Fits CONTRIBUTING.md.

CC @vcunat and @mbakke regarding evaluation breakage (possibly due to 4a7d31a392d78a8afc4a2d78d73dc5c789a602e8).

I'm not sure what's going on there. The intended wrapper is as such:
export FUSEKI_HOME="${FUSEKI_HOME:-/nix/store/i2m96a59pwj5pdl0yr45l1f68h1x7xs3-apache-jena-fuseki-2.4.0}"
I.e. set FUSEKI_HOME if not already set. This seems to be evaluated somewhere, and since FUSEKI_HOME is not set during evaluation, nix errors out.
I've fixed evaluation simply by setting $FUSEKI_HOME to $out in https://github.com/mbakke/nixpkgs/commit/3961643e40cc9e7947e9e9f7b64b73615595f8e3. You can include that to make nix happy again, but I will investigate a bit more what's going on in this particular case as it's a legitimate use case.

I ran nox-review again after merging a3072d3. It builds happily now.


[RFE] Use gopogh to convert tests results from JSON to user-friendly HTML

Current situation
Try to use gopogh to improve the integration-tests results shown in the CI. Example of usage in Minikube here.
Impact
It will ease our life while analysing a failure on the integration tests.
cc @alban

@blanquicet do you think we can close this issue since we have logs in summary?  or we keep it open in case we still want gopogh or something similar?

@blanquicet do you think we can close this issue since we have logs in summary? or we keep it open in case we still want gopogh or something similar?

I think the summary is enough for now and we can close this issue. In case we find that we can still improve it, we can open another issue.


removed statistic is active condition

Description of PR
According to issue  #2342, removed the condition where statistics is active is being checked
Relevant Issues
Fixes #2342

Refer to https://github.com/covid19india/covid19india-react/issues/2105#issuecomment-641833409


CDN version?

Is there CDN version (iife) for this tools? Thanks...

I think we have it https://github.com/antfu/drauu/blob/0849bd85ba62bab987f1ba394ba6ae2246863ae2/packages/drauu/package.json#L7-L8


Added a C++ version of raylib's loading thread example.

This is fantastic! Thank you so much. Looks a lot cleaner with the std lib here. Merging!

Thanks Rob, C++ all the way!


WithBindMount: create source directory (podman)

One distinction between podman and docker for bind mounts is that docker will create the specified path if it does not exist while podman will fail if it does not exist.
It would be nice if the Aspire orchestrator would create the directory when the orchestrator is podman so this difference doesn't bubble up and requires the user to handle it in the AppHost.

@karolz-ms Where do you think this logic should go?

I agree with @tmds, having this logic in the orchestrator makes sense

This is fixed in the 0.4.0 version of DCP


Driver for ultrasonic range meters + example

Driver for ultrasonic range meters like HC-SR04, HY-SRF05 and so on.

That is great!
I actually had in mind a little project involving ultrasonic distance measurement and esp8266 but hadn't have time to start working on it.
I will definitely check out this PR today evening.
Thanks @UncleRus!

I have tried the PR with the module I have and it works OK.
It would be interesting to try it under heavy WiFi traffic. I suspect the reading might be inconsistent due to WiFi interrupts.
In my opinion the PR is good for merge.


Accessibility issue with flu jab question - live service

Bug Report

What is the issue?
On the flu jab question, it is impossible to select the "Yes" option by tapping the word "Yes".
If you tap yes, it selects no.

What steps are required to reproduce the issue?
Answer the questions in the service, tap the word "yes" (don't tap the radio button).

What was the environment where this issue occurred?

Operating System: Android 10
Browser: Firefox

Is there anything else you think would be useful in recreating the issue?

Thanks for reporting the bug @edent - I'll pass it on to the live service

This has now been fixed on live.

Nice one @mcheung-nhs 😎


Netfilter modules missing in version 18.09.05

Hi,
I'm running boot2docker 18.09.5 and I have issues with the iptables-save and iptables-restore commands used by Rancher to set up port forwarding when deploying stacks:
iptables-restore v1.6.0: iptables-restore: unable to initialize table 'raw'
[...]
iptables-restore v1.6.0: Kernel module xt_set is not loaded in.

As in #1374 I miss a lot of iptables-related modules (for instance xt_set.ko):
# ls /lib/modules/$(uname -r)/kernel/net/ipv4/netfilter/
ipt_MASQUERADE.ko          nf_log_arp.ko
ipt_REJECT.ko              nf_log_ipv4.ko
iptable_filter.ko          nf_nat_ipv4.ko
iptable_nat.ko             nf_nat_masquerade_ipv4.ko

Am I missing something?

I think the netfilter TCL extension is missing. I've tried to install it manually but the modules don't load and if I reboot the package is gone.

We don't use the kernel from Tiny Core Linux -- we build our own (especially since Docker is fairly dependent on kernel functionality).
The Netfilter bits we have currently come directly from Docker's requirements: https://github.com/boot2docker/boot2docker/blob/02c9101ce80345fb4ce3d7218f685dd1eafd5403/files/kernel-config.d/docker#L53-L56
If we ought to add more, there needs to be a reasonable case made for their usefulness and something like Debian's kernel configuration to give us a credible list of which ones are sane for us to do so with.

I have added a PR with the list of modules required by Rancher: https://github.com/boot2docker/boot2docker/pull/1391

Since the PR has been merged I can close this issue.

Since the PR has been merged I'll close the issue.


How can we identify row or file is invalid csv

I have two questions here.

I am using GenericParserAdapter but just want to know if extra separator in any row then can we have the option to know if this is the incorrect row? need line-by-line validation. see example
ABC|123|xyz
ABD||124|xyd   -this row has an extra separator so I need to consider it invalid line no also needed.
Could you please assist with this?

How can we know about field datatype, by default now showing string?

If you're looking for row-by-row analysis, you probably want to use the GenericParser base class and iterate over each row to check the parsed inputs.  While doing that, you can verify the number of columns is what you expect (or alternatively use the ExpectedColumnCount to validate the number of columns) and take appropriate action if it doesn't meant expectations.
The parser will only return back the parsed raw string from the columns.  It is up to you to convert the type to the expected data type.


Any model with Dynamic Pathmap set becomes read-only after generating IDL or descriptors

Issue and tracking information
This only occurs in the streams/v2.1.x-maintenance version of CX.
When generating IDL or descriptors on an open model that has a dynamic pathmap, the model becomes read-only immediately after the operation finishes. The only work around is to close and reopen the model each time and try to save all "generations" until you are done making all changes with the model.
To recreate, take any SNA example model like BasicPubSub and add a dynamic pathmap. Save the model. Then generate IDL. Afterwards, expand random packages and you will see they are grayed out. Additionally, trying to change the name of any package like "ports" or "components" will result in the operation being rolled back because the model is read only.
Developer's time Estimated effort to fix (hours):
Developer's Actual time spent on fix (hours)
Issue reporter to provide a detailed description of the issue in the space below

@j26151 Would you be able to verify this fix on your side? https://github.com/ZeligsoftDev/CX4CBDDS/pull/431

This looks to have fixed the issue. Thank you!

Closing...


fixes grammar on thank_you gift_next_body

Small fix. your to you're

This typo is in most of the other locale files, too.
And there's another typo on the same line - 'notifing' (also in the other locales).

@Two-Finger thank you for the hint! @DerangedMind do you want to fix those too before merging?

@Two-Finger thanks!

Perfect. Thanks @Two-Finger. I've made the changes. 😸


Fixing typo in kubernetes workload scaler documentation

This patch fixes a typo in the Kubernetes workload scaler documentation
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

Would you mind updating v2.5 docs as well please?


OJ-2612: Use correct verification score in VC (Approach 1)

Proposed changes
This is approach 1, passing the SessionService
What changed
The verification_score property within a VC is now fetched from the SessionItem in DynamoDB. If there is no evidence block in the session item then the default value of 2 will be used.
Why did it change
We now have multiple levels of confidence that has to be accurately reflected in the VC.
Note: This PR is not yet completed. This is because there's another piece of work to be able to fetch the evidence block from the session record that is being done in another ticket. Once that goes in, this PR needs to be rebased and the getEvidence() calls need to be updated to use the actual methods that have been created to return those results.
Issue tracking

OJ-2612

Closing in favour of https://github.com/govuk-one-login/ipv-cri-kbv-api/pull/349/files


Fix race condition in RoutingDaemon.stop()

Currently, a call to Tunnel.disconnect() shuts down all three helpers in
parallel, and the first one to close triggers another call to
disconnect() via Promise.race(exits).  Both disconnect() calls try
to use RoutingDaemon.socket.write().  If it has already closed, the second
call to write() will throw an exception that is surfaced to the user.

If you would prefer to trade speed for determinism, we can make the shutdown order more fully serialized, but that would be a bigger change.


Image block saves the full server URL

It would be great while doing that, make it backwards compatible, so we don't break the current blocks out there. And, even if we do not break it, provide a Plone script that fixes them in all the site.

I'm pretty sure this is no longer true, since the Image block is using resolveUIDs since some years ago.


Feat/915-wallet-not-running-help

closes #915

@pgriscti @dexturr Appreciate another review here.

@pgriscti @dexturr Appreciate another review here.

Hi @mattrussell36, I was working on this as well, I didn't know you were taking over. I went a bit deeper (maybe not necessary) into the vega-wallet-service and exposing setWalletUrl to update the local storage where the url was kept. But I've reverted the changes now.

Sorry @pgriscti I should have assigned are talked to you first.


eosio-cpp

eosio-cpp -o mydapp.wasm mydapp.cpp --abigen generates empty abi file for
code written in https://developers.eos.io/eosio-home/docs/data-persistence.
OS: macOS Hight Seirra 10.13.6 (17G2112), eosio.cdt version is 1.3.2

yes,i know these,That is very common and has nothing to do with you，That's because of the official mistake，so if you really need it，I suggest you write your own ABI file.

Sadly, any eta about fix?

@philipp-sorin The documentation above is written to be compiled with CDT.  eosio-cpp is deprecated and will not support that code.  If you run this example with CDT and have an issue, please create a new issue.

What tool should I use now?
V1.4 reffers me to eosio-cpp(instead of eosiocpp in v 1.3)

eosio-cpp is indeed the tool to use.  It was eosiocpp that was deprecated.
Can you paste your cpp file here?  So far as I know it should at least be generating the ABI for the actions.  The bugs have mostly been involved with correctly generating information about all data types in public structures.

@jgiszczak  as example you can take code https://developers.eos.io/eosio-home/docs/data-persistence from there :)

The issue is that you are naming your output .wasm something other than the contract name.  Example: eosio-cpp -abigen -o addressbook.wasm mydapp.cpp or eosio-cpp -abigen --contract=addressbook -o mydapp.wasm mydapp.cpp or you can change the name of the addressbook class to mydapp.

@larryk85 I think @philipp-sorin is bad at explaining the issue, I'm having a similar problem.
The eosio-cpp tool we're supposed to use according to the data-persistence tutorial is using EOS 1.3.x when it needs to be using 1.4.x. There's a mismatch in the constructors that can be seen here, it's a pretty big blocker for people new to the ecosystem:
eosio-cpp -o addressbook.wasm addressbook.cpp --abigen addressbook.cpp:6:9: warning: unknown attribute 'contract' ignored [-Wunknown-attributes] class [[eosio::contract]] addressbook : public eosio::contract {         ^ addressbook.cpp:11:71: error: no matching constructor for initialization of 'eosio::contract'   addressbook(name receiver, name code,  datastream<const char*> ds): contract(receiver, code, ds) {}

Any update on that?
Here what happens if I launch the command:
eosio-cpp -o addressbook.wasm addressbook.cpp --abigen
with the addressbook example:
class [[eosio::contract]] addressbook : public eosio::contract {
        ^
addressbook.cpp:11:71: error: no matching constructor for initialization of 'eosio::contract'
  addressbook(name receiver, name code,  datastream<const char*> ds): contract(receiver, code, ds) {}
                                                                      ^        ~~~~~~~~~~~~~~~~~~
/usr/local/eosio.cdt/bin/../include/eosiolib/contract.hpp:26:7: note: candidate constructor not viable: requires
      single argument 'n', but 3 arguments were provided
      contract( account_name n ):_self(n){}
      ^
/usr/local/eosio.cdt/bin/../include/eosiolib/contract.hpp:18:7: note: candidate constructor
      (the implicit copy constructor) not viable: requires 1 argument, but 3 were provided
class contract {
      ^
/usr/local/eosio.cdt/bin/../include/eosiolib/contract.hpp:18:7: note: candidate constructor
      (the implicit move constructor) not viable: requires 1 argument, but 3 were provided
addressbook.cpp:16:29: error: use of undeclared identifier '_code'
    address_index addresses(_code, _code.value);
                            ^
addressbook.cpp:16:36: error: use of undeclared identifier '_code'
    address_index addresses(_code, _code.value);
                                   ^
addressbook.cpp:45:36: error: use of undeclared identifier '_code'
    address_index addresses(_self, _code.value);
                                   ^
addressbook.cpp:66:39: error: unknown type name 'erase'
EOSIO_DISPATCH( addressbook, (upsert)(erase))
                                      ^
addressbook.cpp:66:31: error: C++ requires a type specifier for all declarations
EOSIO_DISPATCH( addressbook, (upsert)(erase))
                              ^
addressbook.cpp:66:46: error: expected function body after function declarator
EOSIO_DISPATCH( addressbook, (upsert)(erase))
                                             ^
1 warning and 7 errors generated.```

@riccardopersiani
Update - Uninstall and reinstall everything. There seems to be an issue (not sure if it's still there) with the Homebrew installation. Install from source when possible for the greatest likelihood of success :)
Hopefully this gets cleared up soon, sounds like there was latency between the most recent CDT and EOS releases and the updates pushed to the onboarding tutorials.

+1

Add
public:
         using contract::contract;

These are the errors I get when trying to compile my smart contract. I recently upgraded to eos.cdt v1.4.1 and eos v1.5.1.
To compile I use: eosio-cpp -o cardgame.wasm cardgame.cpp --abigen
The compiler complains:
cardgame.cpp:83:29: error: unknown type name 'startgame'; did you mean
'cardgame'?
EOSIO_ABI(cardgame, (login)(startgame)(endgame)(nextround));
^~~~~~~~~
cardgame
./cardgame.hpp:5:27: note: 'cardgame' declared here
class [[eosio::contract]] cardgame : public eosio::contract {
^
cardgame.cpp:83:40: error: unknown type name 'endgame'
EOSIO_ABI(cardgame, (login)(startgame)(endgame)(nextround));
^
cardgame.cpp:83:49: error: unknown type name 'nextround'
EOSIO_ABI(cardgame, (login)(startgame)(endgame)(nextround));
^
cardgame.cpp:83:22: error: C++ requires a type specifier for all declarations
EOSIO_ABI(cardgame, (login)(startgame)(endgame)(nextround));
^
cardgame.cpp:83:1: error: C++ requires a type specifier for all declarations
EOSIO_ABI(cardgame, (login)(startgame)(endgame)(nextround));

Solved! I change EOSIO_ABI for EOSIO_DISPATCH. A non-empty ABI file is generated now


When trying to do simple custom query input.  I get a undefined value in the text area.

Why I get an undefined in the textarea in the query input?  the example shows to user [(ngModel)] but it does not allow ngModel.
<query-builder #qbuilder class="margin30" [(value)]='query' [config]='config'>
<ng-container *queryInput="let rule; type: 'textarea'">
<textarea class="form-control" [(value)]="rule.value"
placeholder="Custom Textarea">

How would i get the value off the fileupload from queryinput?
public config: QueryBuilderConfig = {
fields: {
age: {name: 'Age', type: 'number'},
gender: {
name: 'Gender',
type: 'category',
options: [
{name: 'Male', value: 'm'},
{name: 'Female', value: 'f'}
]
},
name: {name: 'Name', type: 'string', entity: 'file'},
notes: {name: 'Notes', type: 'file', operators: ['=']},
educated: {name: 'College Degree?', type: 'boolean'},
birthday: {name: 'Birthday', type: 'date', operators: ['=', '<=', '>'],
defaultValue: (() => new Date())
},
school: {name: 'School', type: 'string', nullable: true},
occupation: {
name: 'Occupation',
type: 'category',
options: [
{name: 'Student', value: 'student'},
{name: 'Teacher', value: 'teacher'},
{name: 'Unemployed', value: 'unemployed'},
{name: 'Scientist', value: 'scientist'}
]
}
}
};
<query-builder #qbuilder class="margin30" [(value)]='query' [config]='config'>
<ng-container *queryInput="let rule; type: 'file'">
<input type="file" [(value)]="rule.value.target.filename" />

I believe these are Angular questions. A common mistake is not importing FormModule when trying to use [(ngModel)]: https://github.com/ng-select/ng-select/issues/187
Here is one solution for file uploads using (change) event handler. You can register the value of the input on the rule in the function handle: https://stackoverflow.com/questions/47936183/angular-5-file-upload
Good luck with what you're trying to do!

Is there a way to trigger the queryinput template based on what type of operator selected?  Let say I created a rule with type as string and operator as 'infile' which cause a template override the textbox with file upload box?
Much help is appreciated.

Please see this plunkr: https://plnkr.co/edit/WvU8Znd44W4iodEqUgn6?p=preview

Exactly what I am looking for.  Thanks a bunch!


Build with Linux instructions

Hi @jeromewu
Wish you a belated Happy New Year.
I Need an instructions for the complete compilation of ffmpeg js in linux machine.
You have given a docker compilation guide https://github.com/ffmpegjs/ffmpeg.js#tutorials. Kindly provide the compile instruction of linux without docker

To compile without docker, make sure you have emscripten 1.39.0 installed and do following commands:

Clone the repository with submodule

$ git submodule init
$ git submodule update --recursive

Execute build-js.sh: https://github.com/ffmpegjs/FFmpeg/blob/n4.1.4-js/build-js.sh

$ bash build-js.sh

I haven't tried it myself, but it should work if you have installed all the dependencies.

@jeromewu. Thanks for your reply

Is there a dev workflow ? the compilation takes like 10+minutes running that docker file.


Session ID is newly generated after each request

I'm creating an application for seat reservations. Therefor I'm saving the Session ID to the Model, in order to prevent duplicate reservations. Now I have the problem, that the returned Session ID from Session::getId(); is different on every request.
I'm using the following Session config:
SESSION_DRIVER=redis
SESSION_LIFETIME=120

Here is my Vue Method:
<script>
export default {
    props: ['seat'],
    methods: {
        addSeatToCart() {
            this.$inertia.post(this.$route('seat.cart', this.seat.id), null, {
                replace: false,
                preserveState: true,
                preserveScroll: true,
                only: [],
            })
        },
    },
}
</script>

And here is the respondig Contoller Action:
use Illuminate\Support\Facades\Session;

public function addSeatToCart(Seat $seat)
{
    return redirect()->back()->with(['status' => Session::getId()]);
}

Here I visualized it with Status Notifications
:
Many thanks for your support!

Fixed it. Here is the Solution:
https://laracasts.com/discuss/channels/laravel/session-id-is-different-in-constructor-and-action

👍


Showing birthdays on frontpage should be an optional preference for users

The list is shown on https://www.inter-actief.utwente.nl/. Showing your name here should be optional.
Members that have disabled this preference should still be shown on this page https://www.inter-actief.utwente.nl/members/birthdays, since this page is only accesible by the board.

Feel free to continue on the closed PR, please do incorporate the feedback


pool poolbinance changes to poolbi0ce

in the name of the pool binance there is an entry NAN, which changes the name to 0
PS D:\pyasic-0.60.10> py test.py
MinerData(ip='<IP_ADDRESS>', datetime='2024-10-29T18:21:57.138586+03:00', timestamp=1730215317, device_info=DeviceInfo(make=<MinerMake.WHATSMINER: 'WhatsMiner'>, model=<WhatsminerModels.M50VH10: 'M50 VH10'>, firmware=<MinerFirmware.STOCK: 'Stock'>, algo=SHA256Algo), make='WhatsMiner', model='M50 VH10', firmware='Stock', algo='SHA256', mac='CA:09:1C:00:47:3B', api_ver=None, fw_ver='20230207.17.REL', hostname='WhatsMiner', hashrate=117.25353012 TH/s, expected_hashrate=111.97 TH/s, expected_hashboards=3, expected_chips=258, expected_fans=2, percent_expected_chips=100, percent_expected_hashrate=105, percent_expected_wattage=99, temperature_avg=70, env_temp=19.5, wattage=3467, wattage_limit=3500, voltage=None, fans=[Fan(speed=3720), Fan(speed=3690)], fan_psu=6176, hashboards=[HashBoard(slot=0, hashrate=38.21151107 TH/s, temp=67, chip_temp=79, chips=86, expected_chips=86, serial_number='HHM32X56103208K43455', missing=False, tuned=None, active=None, voltage=None), HashBoard(slot=1, hashrate=39.31463205 TH/s, temp=70, chip_temp=81, chips=86, expected_chips=86, serial_number='HHM32X56103208K43467', missing=False, tuned=None, active=None, voltage=None), HashBoard(slot=2, hashrate=39.727387 TH/s, temp=72, chip_temp=85, chips=86, expected_chips=86, serial_number='HHM32X56103208K43345', missing=False, tuned=None, active=None, voltage=None)], total_chips=258, nominal=True, config=MinerConfig(pools=PoolConfig(groups=[PoolGroup(pools=[Pool(url='stratum+tcp://sha256.poolbi0ce.com:8888', user='KIV0906.M50-110-03A', password='x'), Pool(url='stratum+tcp://bs.poolbi0ce.com:3333', user='KIV0906.M50-110-03A', password='x'), Pool(url='stratum+tcp://sha256.poolbi0ce.com:443', user='KIV0906.M50-110-03A', password='x')], quota=1, name='IWAK7Y')]), fan_mode=FanModeNormal(mode='normal', minimum_fans=1, minimum_speed=0), temperature=TemperatureConfig(target=None, hot=None, danger=None), mining_mode=MiningModePowerTune(mode='power_tuning', power=3500, algo=StandardTuneAlgo(mode='standard'), scaling=None)), fault_light=False, errors=[], is_mining=True, uptime=735887, efficiency=30, pools=[PoolMetrics(url=PoolUrl(scheme=<Scheme.STRATUM_V1: 'stratum+tcp'>, host='sha256.poolbi0ce.com', port=8888, pubkey=None), accepted=50450, rejected=13, get_failures=7, remote_failures=0, active=True, alive=True, index=1, user='KIV0906.M50-110-03A', pool_rejected_percent=0.025761448982422764, pool_stale_percent=0.013871549452073797), PoolMetrics(url=PoolUrl(scheme=<Scheme.STRATUM_V1: 'stratum+tcp'>, host='bs.poolbi0ce.com', port=3333, pubkey=None), accepted=0, rejected=0, get_failures=0, remote_failures=0, active=False, alive=True, index=2, user='KIV0906.M50-110-03A', pool_rejected_percent=0, pool_stale_percent=0), PoolMetrics(url=PoolUrl(scheme=<Scheme.STRATUM_V1: 'stratum+tcp'>, host='sha256.poolbi0ce.com', port=443, pubkey=None), accepted=0, rejected=0, get_failures=0, remote_failures=0, active=False, alive=True, index=3, user='KIV0906.M50-110-03A', pool_rejected_percent=0, pool_stale_percent=0)])
117.25353012 TH/s
PS D:\pyasic-0.60.10>

Might be time to swap JSON parsing to something that supports inf and nan...

What do I need to do?

What do I need to do?

Naw, this is an internal issue.  I changed the way the parsing is handled, should fix it.

Is it possible to get the serial number of the device in addition to the serial numbers of the hashboards?
It is displayed in the web interface. For S19k pro, S21, T21

Is it possible to get the serial number of the device in addition to the serial numbers of the hashboards?
It is displayed in the web interface. For S19k pro, S21, T21

Possibly?  Technically that is the serial number of the control board, and I've considered it in the past, but it's pretty much only for stock antminers that would work, all other miners tend to not expose that...

Yes, it is only in new devices. Will it not be too difficult for you to get it for those devices that return it?

Yes, it is only in new devices. Will it not be too difficult for you to get it for those devices that return it?
This is convenient for identifying the device, since the same serial number is pasted on the back of the device.

Agreed, but if you want to identify devices, I would recommend you use the built in miner.fault_light_on, which will flash the red light on the miner for identification.
I'll see what I can do regarding the SN, but pyasic tries to support a lot of miners, so if something can't be done easily for most of them it can be a bit of a problem to implement.

I see, thanks for the answer

Is this working as expected now?  If you want the serial numbers, I'm open to a new issue for that feature, but want to make sure this was actually fixed.

Yes, it works correctly. Thank you!


Moving items between the groups

Is your feature request related to a problem? Please describe.
I need a possibility to move items from one layout (group) to another, like in https://sortablejs.github.io/Sortable/
I'm trying to do it on my own, but code is very complex.

Sorry, that's not the point of this lib. You have to implement that yourself


Awake and OnEnable running before Construct with DecoratorSceneContext

Hello and glad to have you back! It was starting to become scary. :)
The package in the attachment contains everything to reproduce the bug. Open and run the 'Decorator Scene'.
To quickly describe the problem, there are two scenes, Main and Decorator. Main has a SceneContext while the Decorator has a DecoratorSceneContext. Both of these scenes have MonoBehaviour classes in them (MainSceneMono and DecoratorSceneMono) which expect dependencies from the other scene. The DecoratorSceneMono also has an Awake method which runs the 'SayHello()' method of the Foo class, which is bound in the Main Scene. Upon running the scene, the Awake method of the DecoratorSceneMono executes before the [Inject] Construct() method. Tests with Start() and OnEnable() have shown this execution order:
Awake() -> OnEnable() -> [Inject] Construct() -> Start()
DecoratorContext.zip

The problem here is that Unity will always fully start the decorator scene before the main scene is started.  And zenject needs to wait for the install phase to fully complete (including the installers in the main scene) before triggering [Inject].  So I'm not sure there's much zenject can do here.
I can think of some potential workarounds though:

Use Scene Contract Parent Names instead of decorator scenes (though in your case this probably won't help because you can't have scenes reference dependencies inside each other this way like you can with decorators)
For objects in your decorator scene, do not use Start and Awake for initialization and instead use [Inject]
Set the objects in decorator scene to be in-active and then call SetActive once injection occurs

Alright, thank you for the answer. I don't like it when I have to do anything other than providing references inside an [Inject] method, but I guess it can't be helped in this instance.


Correct synthetic transaction production

This PR correctly implements synthetic production, according to @PaulSnow and @bunfield's comments. It works like this:

Block 1

Deliver

Transactions are processed and produce synthetic transactions

Commit

Synthetic transactions are written to the staging bucket, added to the BPT (hashed), and added to the anchor chain (hashed)

Block 2

BeginBlock

The leader scans the anchor chain, generates signatures for the previous block's synthetic transactions, and sends a SyntheticSignTransactions transaction with the signatures to its own BVN

Block 3

Deliver

The synthetic transaction signatures are validated

Commit

The signatures are written to the signatures bucket and added to the BPT (hashed)

Block 4

BeginBlock

Pending signed synthetic transactions are batched and transmitted to BVNs

Commit

Signatures for sent transactions are removed from the signatures bucket

Block 5 (all BVNs)

Synthetic transactions are processed

This is ready for review. Tests are definitely still broken because of the changes to when and how synthetic transactions get settled.

The code looked fine.  The question I have is if Block 2 is scraping the anchor chain, is that is on the DN, or is that the root for the BVN of the previous block?

It's the root of the BVN of the previous block. At the end of the block, synthetic transaction IDs are appended to the anchor chain (BVN root) after anchors from records that changed. In the next block, the BVN scans its own anchor chain for those synthetic transaction IDs.
Paul suggested creating a separate chain just for the synthetic transaction IDs, since most consumers of the anchor chain won't care about synthetic transactions. I want to do that and some other work on chains, but not in this PR.


[1.20.4] Updated to Minecraft 1.20.4

WIP, very broken

Whats broken

Closed, main branch will be the branch of the current mc version from now on

I love the work you getting done here, if you need support or anything hit me up on discord <3 : dkto


Fix InvalidOperationException in TestTimerRegistry.FireAll()

Fix an InvalidOperationException if new timers are created during FireAll() method call.

By the way, there is a typo in TestKitSilo. TimerRegistry is spelled TimerReistry but the property is public so it would be breaking to fix it.

Thanks, @srollinet! I opened a new PR to fix that typo, too. 😄


Discussion: Profiles support

EAT (https://ietf-rats-wg.github.io/eat/draft-ietf-rats-eat.html) allows the use of any claims defined by CWT or JWT, in addition to those expressly defined by the  EAT standard. All claims are considered optional by EAT.
It is impractical to try to evaluate all the possible claims (as the set is essentially unbounded). Also, specific applications may require certain claims to always be present or have additional constraints on their values. It is proposed that this additional functionality is handled via EAT token "profiles".
A profile will handle the following:

Interpreting and validating additional claims, not defined in the core claims in the EAT standard.
Define claims that are mandatory (in the context of that profile).
Enforcing additional constraints on (both, core and extended) claims' values. (For core claims such constraints would have to result in strictly narrower valid ranges).
Enforcing inter-claim constraints. E.g, if claims are mutually-exclusive or the values of one are constrained by the values of another.

This thread is do discuss how such profiles might be added to the existing Eat implemention.

A concrete example of a profile that would define additional claims and add mandatory and inter-claim constraints is Arm PSA token: https://datatracker.ietf.org/doc/html/draft-tschofenig-rats-psa-token-05

It looks like there are two obvious options:

Extend:

Group the additional claims needed by a given profile into a new struct type (say, type MyProfile struct {});
Embed MyProfile into the the top level Eat;
Provide a func MyProfileValidator(e Eat) error that can access the extended EAT token in its entirety and apply all the intra and inter-claim constraints needed to satisfy the profile.

Compose:

Wrap each EAT claim with its JSON and CBOR tags, and make it publicly exportable;
Create a type MyProfile struct {} which embeds the (cherry-picked) EAT claims, and adds its own;
Handle validation of MyProfile in a func (MyProfile) Validate() error method.

I'd personally prefer the (less intrusive, inherently distributed) "compose" approach , however, I think Submods of the "non-token type" type, which are recursively defined in terms of Eat, are problematic to handle in that context.

I second the the preference for composition. However I don't think claims should be contained within the profile. The profile should wrap the token as a whole, and then provide interpretation and validation of the underlying claims, but it should not contain any of the "data"; and therefore, there would be no need for JSON/CBOR tag for the profile -- the only thing that gets serialised is the Eat.
Somewhat orthogonal to the Profiles, is the need for the EAT token to contain arbitrary claims -- this is specified by the spec itself and is not an extension to the format (well, strictly speaking, any CWT or JWT claims, rather than completely arbitrary). However, as we cannon provide strong type definitions for such claims, they would have to be deserialised into something like a map[int]interface{} (can we define a recursive IntMap type that enforces the integer key requirement?)
The Profile would then allow interpreting those "generic" claims as strongly typed and named values. However, the "master" value for each claim should be contained within the Eat, and not the profile. E.g.  setting the ArmPsaProfile.ImplementationId value should result in writing to key -75003 in underlying Eat map.


Property visibility switching not active when using "Create A New Block" or "Quick Edit" from ContentAreas in Optimizely CMS 12

Hi, we are having an issue where it seems this plugin is not enabled when using the "Create A New Block" option from the ContentArea dropdown in Optimizely CMS 12 solutions, it only seems to be enabled when going into edit mode for the block.
Would it be trivial to implement support for these types of content creation & editing?
This is extension is very useful, but it will be hard to rely on when these editing options are very popular among editors.

This is still not working, has anyone found a solution to this?


Charts testing

Helm supports testing out of the box for the Charts.
It would cool to introduce such testing in our CI pipeline.
We could probably use our Observability API for checking that the chart works as expected.
The docs for the Helm testing framework it's available here

Do we really need this? In our current business-tests we are using the helm charts to deploy two connectors, it would fail if something is wrong with them. At least this how the business-tests are implemented by now, this might be changed if we want to refactor it.

If I got it now business runs only with the configuration postgfres and hashicorp vault. So for example we are not testing the azure variant right?
We could have a fine grained test case on each charts variant/configuration without the need to run business tests against all of them.

We only have one helm chart tractusx-connector and using always the same helm chart to deploy the different variants. We just provide different image name and of course same other variables depending on the variant. Testing the azure variant would mean only provide the certain image name for azure and the required variables for it.
If we split the current business-tests into different tests then probably we would need a separate test only for helm chart but as of now I don't think this is required.


Color3 problem

Here is the error:

Now the problem is I'm trying to set a Color3 to the BackgroundColor` of a Frame:

The BackgroundColor under List is receiving the error, Color3 should work so why it is not is an issue.

Never mind the issue was BackgroundColor3 totally my fault closing


Connection to ROS

Hello,
i see in the cmakelist file that the package needs ROS (roscpp) to be built. Is it planned to have a version compatible with ROS2 also?
And could ndt_omp be easilty detached from ros, do you think?
thanks

Is it planned to have a version compatible with ROS2 also?

I wouldn't port it to ROS2 soon. But, any PR is welcome.

And could ndt_omp be easilty detached from ros, do you think?

The library itself is independent of ROS, and it can easily be detached by removing some catkin-related commands in CMakeLists.txt

Ok, thanks for the feedback!


Some fixes

added validation check in test script
small fixes in prediction and train new

Coverage decreased (-1.6%) to 69.844% when pulling 6e120faa1c8c355fe687c42bdb02dd927d705fbd on dev into c0784e88e4adbc4dad8522c56f7427e0cd9aafc3 on master.


Bump govspeak from 5.5.0 to 6.0.0

sanatize 5.0 is less strict then 4.6.4 which prevented upgrading to govspeak 5.6. Before raising this PR all govspeak in HTML attachments and editions was validated with the govspeak 6.0 gem.

Woop woop 🎉

It's a shame the small amount of changes in this PR doesn't reflect the research behind this.


thread 'main' panicked at 'not implemented'

Hi!
I was trying to use your library to access a simple .mat file containing a single Matlab struct with 8 fields, and I got the error below after executing the following code:
use matio_rs::{MatFile, Mat};

fn main() {
    let file_path = "/foo/thematlabfile.mat";
    let var_name  = "the_matlab_struct";

    let mat_file_read = match MatFile::load(file_path){
        Ok(file_read) => file_read,
        Err(error) => panic!("Panic while opening file, details: {:?}", error),
    };

    println!("{:?}", mat_file_read.info());  // This works => Matlab var.: "the_matlab_struct" with dims: [1, 1]

    let mat_variable: Mat = match mat_file_read.var(var_name){
        Ok(variable) => variable,
        Err(error) => panic!("Panic while extracting variable, details: {:?}", error),
    };

}

Do you have any idea of what the problem may be?
Many thanks in advance for your help.
Cheers
J.
PS: In case it helps, the data types of the 8 fields of the Matlab struct are:

2D matrix of double
2D matrix of double
String
Double
Double
Double
String
String

Error
thread 'main' panicked at 'not implemented', /Users/foo/.cargo/registry/src/index.crates.io-6f17d22bba15001f/matio-rs-1.3.1/src/datatype.rs:101:1
stack backtrace:
   0: rust_begin_unwind
             at /rustc/90c541806f23a127002de5b4038be731ba1458ca/library/std/src/panicking.rs:578:5
   1: core::panicking::panic_fmt
             at /rustc/90c541806f23a127002de5b4038be731ba1458ca/library/core/src/panicking.rs:67:14
   2: core::panicking::panic
             at /rustc/90c541806f23a127002de5b4038be731ba1458ca/library/core/src/panicking.rs:117:5
   3: matio_rs::datatype::MatType::from_ptr
             at /Users/foo/.cargo/registry/src/index.crates.io-6f17d22bba15001f/matio-rs-1.3.1/src/datatype.rs:86:26
   4: matio_rs::mat::Mat::from_ptr
             at /Users/foo/.cargo/registry/src/index.crates.io-6f17d22bba15001f/matio-rs-1.3.1/src/mat.rs:136:12
   5: matio_rs::mat::Mat::from_ptr
             at /Users/foo/.cargo/registry/src/index.crates.io-6f17d22bba15001f/matio-rs-1.3.1/src/mat.rs:165:23
   6: matio_rs::mat::<impl matio_rs::matfile::MatFile>::read
             at /Users/foo/.cargo/registry/src/index.crates.io-6f17d22bba15001f/matio-rs-1.3.1/src/mat.rs:33:13
   7: matio_rs::mat::<impl matio_rs::matfile::MatFileRead>::var
             at /Users/foo/.cargo/registry/src/index.crates.io-6f17d22bba15001f/matio-rs-1.3.1/src/mat.rs:76:9
   8: some_code::main
             at ./src/main.rs:14:39
   9: core::ops::function::FnOnce::call_once
             at /rustc/90c541806f23a127002de5b4038be731ba1458ca/library/core/src/ops/function.rs:250:5

The problem comes from the String in the Matlab structure, matio-rs is not handling reading and writing Strings yet

Thanks for you help!


RDK-28250 : Adding argument 'age' for Warehouse.isClean

isClean should return 'true' status after warehouse reset with used
value of age argument in seconds. And this value should be more
then time, starting from warehouse reset till moment of testing.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: vijs:x: Maksym SavchenkoMaksym Savchenko seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


#if DEBUG ignored

VS correctly highlights debug vs release code when the output type is changed, but only release .JS code is generated.
#if DEBUG
// never generated
#else
// always generated
#endif

Hello, have you tested this on our current and latest github branch? We've just developed this feature on issue #375. It will be available on NuGet for the next 1.9 release.
Do you agree your request is a duplicate of #375?

#375 sounds like it should cover this problem.


XPath-like engine

While implementing parse tree mapping, we meet the fact, that we write quite complex xpath queries. We need something like xpath, but more suitable for our case.
As the first step, we need to replace xpath engine by our own, not pretending to replace syntax.
The possibility to write custom functions should remain or already written functions should be included like a base functions.
Also we want to make an engine that way to walk through the tree less time - combine common paths from queries.

We don't want to fill model in a declarative way - we will walk parse tree as we did for internal SQL Editor model


Added --history-limit option.

Old chart versions are deleted when a chart is added
Implements https://github.com/kubernetes-helm/chartmuseum/issues/134

@davidovich Please let me know if there are any changes required. Thanks!

hi @aleerizw we will try to get this merged this week, thanks for your patience

@aleerizw sorry for the silence, work was keeping me very busy last week.
I am a bit uneasy on the point of adding a deletion process to an add operation (uploadChartPackage).
Could we envision starting a goroutine at server startup time that collects older charts and does the house keeping? We could trigger/control the house keeping in a channel from the uploadChartPackage function.
What do you think ?

We do need some sort of background task management in general.

I agree, I was not sure about this myself as well but wanted to get something out quickly :) I will make the changes


Does NGT support fast index build and query on GPU?

Does NGT support fast index build and query on GPU, or a mixed CPU and GPU?
Thanks a lot.

NGT doesn't support GPUs.


Link to epidemic algorithms seems wrong

Hi - the link to the epidemic algorithms seems wrong. Currently it redirects to a paper on "The Load, Capacity and Availability of Quorum systems" (https://www.cs.utexas.edu/~lorenzo/corsi/cs395t/04S/notes/naor98load.pdf). This paper doesn't seem to be an explainer on epidemic algorithms. Can we please get this double checked to correct the link?
Thanks,
Nandan Mankad.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ccf129c8-f33f-3747-45f5-e3e0eae16b75
Version Independent ID: bb3bd0c9-fd71-1f95-96ef-bc2fad1a5b9b
Content: Global distribution with Azure Cosmos DB- under the hood
Content Source: articles/cosmos-db/global-dist-under-the-hood.md
Service: cosmos-db
GitHub Login: @SnehaGunda
Microsoft Alias: sngun

@mankadnandan Thank you for pointing this out . We are engaging the content team to review and update this article as needed.

Updated link in the above PR. Changes will be live later today. We will close this issue now.
#please-close


[v1.0][F12-B3] Media Socializer

@Nethergale @KevinChuangCH

@shadow2496 @KevinChuangCH  Please check your code styling. As mentioned it is advised not to submit PRs without passing travis.

@OuyangDanwen This pull request was sent before Travis was set up for our team repo.
The Travis for our team repo has being set up and we are working on correcting all the previous PR now.

@OuyangDanwen After checking through the failed Travis test, we realised that the test failed because of a commit that is not ours.
(Link to the commit: https://github.com/nus-cs2103-AY1718S2/addressbook-level4/commit/e953ce0f997f937c6c21fc486e07e7e83b9fef65)
Is this because we did not setup our team CI before editing?

(upstream dev here) @OuyangDanwen weirdly, I verified that their repo passes Travis-CI, but this PR fails the Travis-CI check. Maybe can try restarting the Travis-CI build?

Maybe can try restarting the Travis-CI build?

I've done that once, but still fails.

May be try closing this PR and opening another PR, in case it's Travis issue that is peculiar to this PR only?

@Zhiyuan-Amos, I think that is the reason. Travis tests the commit that is not ours.

@Zhiyuan-Amos, @damithc, I will open another PR. Thank you for your help!


Populate called twice on same container

I am building a library to enable multi-tenancy applications, with tenant isolation and optionally module isolation.
This means there is a main application container, then a child container (per tenant), then optionally you can have isolated modules (which are child containers from the tenant container), or have some modules that aren't isolated, and directly add configuration to the tenant container.
At the moment, the order of play is Application Container is configured, then Tenant Container is created (and populated). Then if using Modules, Modules are located, and either child container is created for the module (isolated), or the existing tenant container is found and the module populates the tenant container (non isolated)
In order to keep my API's all container agnostic, all container configuration is using IServiceCollection. I am plugging in structure map as the default container provider as it supports nesting and child containers.
At the moment, for the non-isolated modules, I get an exception saying Populate can only be called once on a container. This happens because:

Tenant Container is populated via IServiceCollection
Modules are located, and each non-isolated module calls Populate() on the tenants container.

I could possibly give all non-isolated modules a single IServiceCollection but this would still result in Populate being called twice on the Tenant Container (once at the Tenant configuration level, and once at the module level for all of non-isolated modules that are loaded).
I could try to redesign my API's such that during tenant configuration, all non isolated modules are resolved at that point, so they can add services prior to the single Populate call, but this involves a tonne of redesign on my part - as this is all configured with a fluent API and I have designed the Modules section of the fluent API to be an Add-On rather than in-built into the tenant configuration API.
Is there any way you can make the constraint (i.e populate only being allowed to be called once) overridable - so I can disable it if I know I will be incrementally populating the container in stages?

Thanks, much appreciated

:smile:


snakemake long benchmark name results in "Removing output files of failed job generate since they might be corrupted:"

Snakemake version
snakemake 7.25.4
Describe the bug
Hello snakemake team,
I noticed that when I have the benchmark option in the snakefile, if the length of the benchmark file is longer, the rule is executed but results in an error. So even though the job finished successfully, the output files are removed and the error is:
"Removing output files of failed job generate since they might be corrupted:"
Logs

Using shell: /usr/bin/bash
Provided cores: 1 (use --cores to define parallelism)
Rules claiming more threads will be scaled down.
Job stats:
job         count    min threads    max threads

all             1              1              1
generate        1              1              1
total           2              1              1
Select jobs to execute...
[Wed May 17 14:15:32 2023]
rule generate:
output: test_bug.txt
jobid: 1
benchmark: test_aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa.log
reason: Missing output files: test_bug.txt
resources: tmpdir=/tmp
[Wed May 17 14:15:32 2023]
Error in rule generate:
jobid: 1
output: test_bug.txt
shell:
    echo "snakemake" > test_bug.txt

    (one of the commands exited with non-zero exit code; note that snakemake uses bash strict mode!)

Removing output files of failed job generate since they might be corrupted:
test_bug.txt
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message
Complete log: .snakemake/log/2023-05-17T141530.724000.snakemake.log

Minimal example

this is the Snakefile
rule all:
input:
"test_bug.txt"
rule generate:
output:
notebook = "test_bug.txt"
benchmark: f"test_{'a' * 500}.log"
shell:
"""
echo "snakemake" > test_bug.txt
"""

You can run the snakefile by: snakemake -j 1
Additional context

This may be a limit of the file system rather than an issue with snakemake, see for example this wikipedia article or https://serverfault.com/questions/9546/filename-length-limits-on-linux. I guess the issue can be closed?

At least with output files you get an explicit error saying what's wrong. I agree that it is frustrating when snakemake tells you Look above for error message and there is no error to look at. It would be nice if this could be fixed in general, but I do know how easy it would be.
In your specific case, file names that long being used as benchmark files should be rare enough that I don't think the developers need to worry (again, considering that this is not a snakemake bug).


Extension with an old version is not removed by reloading VsCode after upgrading to a newer version

Does this issue occur when all extensions are disabled?: Yes/No

Describe the bug
I upgrade an installed extension to latest version from marketplace and then reload VsCode. The old extension is not removed until I manually quit and re-open VsCode.
To Reproduce
Steps to reproduce the behavior:

Go to marketplace and pick an extension with an old version
Update the version and reload VsCode
Go to .vscode/extensions and both versions persist

VS Code Version: 1.78.2
OS Version: macOS Ventura 13.2
Additional context

See this recording for more information.

@sandy081 Can you help us understand where this bug is on your roadmap? It causes quite a bit of issues for those who are publishing new versions of the extension.

having the same problem, VSCode\data\extensions this folder still contains removed and older version extensions folders

The old extension is not removed until I manually quit and re-open VsCode.

This is as designed that we clean up obsolete or outdated extensions on restart of VS Code.


TiKV OOM

TiKV 7.1.1在开始事务后缺失commit会导致内存膨胀最终OOM，限制memory-usage-limit、storage.block-cache.capacity均无法阻止内存扩张。
大量并发：
txnKVClient.begin();
txnKVClient.put(key, value);
// txnKVClient.commit(); // 遗忘提交事务

@overvenus Will the recent change in TiKV OOM mitigation solve this issue?

Machine:
CentOS 7.6
cpu_logical_number:40
memory:256G
TiKV conf:
memory-usage-limit = "120G"
[raftstore]
raft-base-tick-interval = "2s"
[resolved-ts]
advance-ts-interval = "20s"
scan-lock-pool-size = 8
[rocksdb]
wal-recovery-mode = 0
[rocksdb.defaultcf]
soft-pending-compaction-bytes-limit = "256G"
[server]
grpc-concurrency = 20
[storage]
api-version = 2
enable-ttl = true
[storage.block-cache]
capacity = "60G"

resolved-ts.enable设置为false后内存占用明显减少。
[resolved-ts]
enable = false

Resolved TS Worker CPU match with OOM。

@vzong Thank you for your feedback! We have fixed a similar OOM issue that is caused by resolved ts. Could you test v7.4.0 or the nightly release? Also, could you share your test client (many prewrite, no commit)?

https://github.com/tikv/tikv/issues/14864
https://docs.pingcap.com/tidb/dev/release-7.4.0

I believe it has been fixed. Feel free to reopen if it happens again.


Doesn't work on Python 3 since you did not redeploy the fix for version 0.2.6 to PyPI

Please redeploy as 0.2.7, thanks

any update?


Rigging validations

Goal
Minimize mistakes and to create a more universal standard.
Motivation
Rigs without colors on the the controls, controls without freezed transformation and with constrained deformers/joints not hidden, default lambert1 on models gets published to the animators to often,
Implementation
These are not forced and we should be able to un-tick if needed when we publish.

Freeze controls, Check of there is any controls that doesn't have default values, ignore custom values.
Animation Controls color, Check if controls within the controls_SET has 'no' color.
Unwanted nodes in the viewport, only geo and nurbs curves should be visible.
Lambert1/grey material on models, Check if meshes inside Mesh group dosen't have the default material.
Keys on animation controls, Check within controls_SET to see if there is any keys done by the rigger.
Default names on control curves, Check the animation control curves within controls_SET to make sure it doesn't have a default name eg. nurbsCircle1

This is fantastic Calle.

Freeze controls, Check of there is any controls that doesn't have default values, ignore custom values.

Technically, freeze isn't enough and sometimes inapplicable. For example, if the animator has previously animated on a controller with values already set, that animation would become offsetted by the difference between what it was and what it became once frozen.
I've dealt with this before and found the more efficient and scalable approach - that also takes into account non-transform values such as custom attributes - was to "save" values during rigging and "load" them when necessary. Once saved, as a hidden attribute for example, we could then validate against these values, and also enable the animator to "reset" a rig by loading in the default values set by the rigger.
It's relative straightforward to set-up, I can add a "Save Controller Values" and "Load Controller Values" to your Mindbender menu.

Animation Controls color, Check if controls within the controls_SET has 'no' color.

Great.

Unwanted nodes in the viewport, only geo and nurbs curves should be visible.

Agreed.

Lambert1/grey material on models, Check if meshes inside Mesh group has default material.

Perfect.

Keys on animation controls, Check within controls_SET to see if there is any keys done by the rigger.

This I think relates to (1). In my experience, controls may sometimes need a key on them for whatever rigging purposes. We could make it such that for these channels, the rigger must set the channel to un-editable? That way it'll appear grey and we can exclude this from validation.

Lambert1/grey material on models, Check if meshes inside Mesh group has default material.

Interesting; do you ever want to preview a character (that is rigged) with a preview shader/colors? If so, when do you create such a "preview shader" and when is it applied? Does the animator apply it?

This I think relates to (1). In my experience, controls may sometimes need a key on them for whatever rigging purposes. We could make it such that for these channels, the rigger must set the channel to un-editable? That way it'll appear grey and we can exclude this from validation.

Yes, I believe this would work. No keys on any keyable and unlocked channels for controls.

Technically, freeze isn't enough and sometimes inapplicable. For example, if the animator has previously animated on a controller with values already set, that animation would become offsetted by the difference between what it was and what it became once frozen.

yes this is exactly the problem, once i notice the issue an animators has in many cases already started animating and it can't be corrected. that's why the pipeline should tell the one doing the rigg that its not freezed before he publish to the animator the first time/all times.
Could you elaborate some more why this isn't a good thing to make sure the rigger has done?

It's relative straightforward to set-up, I can add a "Save Controller Values" and "Load Controller Values" to your Mindbender menu.

Sounds like a great feature regardless :) We do have "Set default" in the animation menu, not sure what it does though :)

This I think relates to (1). In my experience, controls may sometimes need a key on them for whatever rigging purposes. We could make it such that for these channels, the rigger must set the channel to un-editable? That way it'll appear grey and we can exclude this from validation.

Sure we could do that, but like i said, all of these validations are in my mind optional and just there to remind the rigger. If a control with an a keyed channel is needed and is the only control that gets an error the rigger can simply un-tick the validator and continue to publish.

Lambert1/grey material on models, Check if meshes inside Mesh group has default material.

Interesting; do you ever want to preview a character (that is rigged) with a preview shader/colors? If so, when do you create such a "preview shader" and when is it applied? Does the animator apply it?
I can see now that i wrote that poorly, what i meant was that i do not want lambert1 materials, not the opposite :) So yeah i want basic representation of color unless a AnimLookdev is provided with a lower res texture.

Updated so it made since with the lamber1 validator and also added a new validator for anim Control names

Could you elaborate some more why this isn't a good thing to make sure the rigger has done?

The rigger should ensure a particular set of default values. I'm agreeing with you.
What I wanted to point out is that the Maya Freeze Transforms isn't the best or even a good option. Freezing transforms means putting those values somewhere else, somewhere hidden (see Attribute Editor > Transform Attributes > Local Space), that make working with its actual transform one level more difficult.
What I would recommend is that you store offsets in a parent group, and let the child be what the animator interacts with. That way offsets are explicit and visible to the rigger and developer.
On top of having transforms frozen, there are likely other attributes that also need a default value. This is another case where Freeze Transforms isn't an option.
I'm thinking ahead. I think what we'll do is implement it as you say so we can stare the new problem straight in the eye before we fix it.

Sure I understand, apply the validator however you like :)


Unable to compile on Linux for Android

I have copied the repo and checked out the feature branch feature/498-qr_add_hotspot.
I did yarn install; yarn android and obtained the following:
hotspot-app/android/app/build/generated/rncli/src/main/java/com/facebook/react/PackageList.java:55: error: package org.libsodium.rn does not exist
import org.libsodium.rn.RCTSodiumPackage;
                       ^
hotspot-app/android/app/build/generated/rncli/src/main/java/com/facebook/react/PackageList.java:123: error: cannot find symbol
      new RCTSodiumPackage(),
          ^
  symbol:   class RCTSodiumPackage
  location: class PackageList

This fails on Linux (Elementary OS, Ubuntu 18). The same works fine on a MacBook.
This is possibly a client issue, but I'm not sure what I did wrong? All the SDKs are up to date in Android Studio (same config for both the Linux laptop and the Mac laptop).
Is there any troubleshooting command that could help with it?
I did notice that the Mac build invoked CMake at some point, I'm assuming to build libsodium. I did not see any CMake invocation on Linux. I'm guessing that this is the root cause.

What's the solution?

same for me, any idea?


[NVIDIA XLA:GPU] cuDNN fMHA support infrastructure

Closing this since https://github.com/tensorflow/tensorflow/pull/59832 has been opened that combines this and https://github.com/tensorflow/tensorflow/pull/59153