Rippled crash ripple::Keylet::check (Version: 1.6.0)

Issue Description
rippled v1.6.0 for some reason crashed on the mainnet. I do not see any OOM related messages in the dmesg.
The VM where rippled is run has 32 GB of RAM. It's running on NVME drives.
Configuration
rippled.cfg:
[server]
port_rpc_admin_local
port_peer
#port_ws_admin_local
port_rpc_external
port_ws_external
port_ws_public_admin
#ssl_key = /etc/ssl/private/server.key
#ssl_cert = /etc/ssl/certs/server.crt

[port_rpc_admin_local]
port = 5005
ip = <IP_ADDRESS>
admin = <IP_ADDRESS>
protocol = http

[port_peer]
port = 51235
ip = <IP_ADDRESS>
protocol = peer

[port_rpc_external]
port = 51234
ip = <IP_ADDRESS>
protocol = http
user = ripplerpc
password = xxx
admin = <IP_ADDRESS>
#admin_user = ripplerpc
#admin_password = xxx

[port_ws_external]
port = 51233
ip = <IP_ADDRESS>
protocol = http
user = ripplerpc
password = xxx
admin = <IP_ADDRESS>
#admin_user = ripplerpc
#admin_password = xxx
protocol = ws

[port_ws_public_admin]
port = 6006
ip = <IP_ADDRESS>
admin = <IP_ADDRESS>
protocol = ws

#[port_ws_public]
#port = 5005
#ip = <IP_ADDRESS>
#protocol = wss

#-------------------------------------------------------------------------------

[node_size]
#
#   Tunes the servers based on the expected load and available memory. Legal
#   sizes are "tiny", "small", "medium", "large", and "huge". We recommend
#   you start at the default and raise the setting if you have extra memory.
#   The default is "tiny".
huge

[ledger_history]
256

# This is primary persistent datastore for rippled.  This includes transaction
# metadata, account states, and ledger headers.  Helpful information can be
# found here: https://ripple.com/wiki/NodeBackEnd
# delete old ledgers while maintaining at least 2000. Do not require an
# external administrative command to initiate deletion.
[node_db]
type=RocksDB
path=/home/ripple/.ripple/db/rocksdb
open_files=512
filter_bits=12
cache_mb=1024
file_size_mb=64
file_size_mult=2
online_delete=2000
advisory_delete=0

#type=NuDB
#path=/home/ripple/.ripple/db/nudb
#online_delete=2000
#advisory_delete=0

[database_path]
/home/ripple/.ripple/db

#[shard_db]
#type=RocksDB
#path=/home/ripple/.ripple/db/shards/rocksdb
#max_size_gb=5

# This needs to be an absolute directory reference, not a relative one.
# Modify this value as required.
[debug_logfile]
/home/ripple/.ripple/debug.log

[sntp_servers]
time.windows.com
time.apple.com
time.nist.gov
pool.ntp.org

# Where to find some other servers speaking the Ripple protocol.
#
# Where to find some other servers speaking the Ripple protocol.
#
[ips]
r.ripple.com 51235

[validator_list_sites]
https://vl.ripple.com

[validator_list_keys]
ED2677ABFFD1B33AC6FBC3062B71F1E8397C1505E1C42C64D11AD1B28FF73F4734

# Turn down default logging to save disk space in the long run.
# Valid values here are trace, debug, info, warning, error, and fatal
[rpc_startup]
{ "command": "log_level", "severity": "warning" }

# If ssl_verify is 1, certificates will be validated.
# To allow the use of self-signed certificates for development or internal use,
# set to ssl_verify to 0.
[ssl_verify]
1

Expected Result

Actual Result

Environment

$ ./rippled --version
rippled version 1.6.0

Ubuntu 18.04.3 LTS
Supporting Files

Sep 12 12:57:01 ripple-ac rippled[12059]: 2020-Sep-12 09:57:01.770515608 UTC SHAMapStore:WRN rotating  validatedSeq 58134184 lastRotated 58132184 deleteInterval 2000 canDelete_<PHONE_NUMBER> state full age 0s
Sep 12 12:57:49 ripple-ac rippled[12059]: 2020-Sep-12 09:57:49.484066817 UTC LoadMonitor:WRN Job: acceptLedger run: 1005ms wait: 1ms
Sep 12 12:57:49 ripple-ac rippled[12059]: 2020-Sep-12 09:57:49.761838852 UTC LoadMonitor:WRN Job: sweep run: 1416ms wait: 0ms
Sep 12 12:58:28 ripple-ac rippled[12059]: 2020-Sep-12 09:58:28.396540286 UTC SHAMapStore:WRN finished rotation 58134184
Sep 12 12:58:33 ripple-ac rippled[12059]: 2020-Sep-12 09:58:33.457665417 UTC LoadMonitor:WRN Job: processLedgerData run: 2177ms wait: 0ms
Sep 12 12:59:51 ripple-ac rippled[12059]: 2020-Sep-12 09:59:51.186981022 UTC LoadMonitor:WRN Job: sweep run: 1425ms wait: 0ms
Sep 12 13:01:56 ripple-ac rippled[12059]: 2020-Sep-12 10:01:56.547085324 UTC LoadMonitor:WRN Job: sweep run: 5360ms wait: 0ms
Sep 12 13:39:33 ripple-ac rippled[12059]: rippled: /home/build/ripple/source/src/ripple/protocol/impl/Keylet.cpp:37: bool ripple::Keylet::check(const SLE&) const: Assertion `sle.getType() == type' failed.
Sep 12 13:39:33 ripple-ac systemd[1]: rippled.service: Main process exited, code=killed, status=6/ABRT
Sep 12 13:39:33 ripple-ac systemd[1]: rippled.service: Failed with result 'signal'.

I've got a crash on 3 separate rippled nodes simultaneously.
Both are v1.6.0. Is it a bug affecting all v1.6.0 versions?
Ping @mellery451 @ximinez @nbougalis @seelabs

It seems there is a fix in develop branch If I'm right?
https://github.com/ripple/rippled/commit/ab9f3fa42aed39477a8a9d27c46830ada9ab890a

@gituser Thanks for reporting this issue!
We take crashes very seriously. In this case, the problem is an assert is triggering. asserts are normally disabled in production code. They turned on only in debug builds or by explicitly turning them in in release builds. Since the vast majority of servers are safe from this error, I think it's OK to wait for the fix to land in the next release rather than a hot fix.
Thanks again for raising the issue!

@seelabs so I guess it's because I have in my compilation build script this:
cmake -DCMAKE_BUILD_TYPE=Release -Dstatic=True -Dassert=On ../..
Guess I need to remove -Dassert=On and recompile rippled for now.

@gituser Yes, removing that -Dassert=On will fix things in the meantime.


[REST Server] Handle launcher error when submitting a job

Coverage increased (+0.0001%) to 49.79% when pulling 176244577c5c245bc97cee35e12305cb16b6ad9a on hwuu/restServer_handleLauncherErrors into 05f90222e5275c33ef70d7b4bb42fa0e190dde71 on master.

Coverage increased (+0.0001%) to 49.79% when pulling 176244577c5c245bc97cee35e12305cb16b6ad9a on hwuu/restServer_handleLauncherErrors into 05f90222e5275c33ef70d7b4bb42fa0e190dde71 on master.

Coverage increased (+0.0001%) to 49.79% when pulling 176244577c5c245bc97cee35e12305cb16b6ad9a on hwuu/restServer_handleLauncherErrors into 05f90222e5275c33ef70d7b4bb42fa0e190dde71 on master.

Coverage increased (+0.0001%) to 49.79% when pulling 176244577c5c245bc97cee35e12305cb16b6ad9a on hwuu/restServer_handleLauncherErrors into 05f90222e5275c33ef70d7b4bb42fa0e190dde71 on master.


Use resolve for consistency.

Use resolve in all cases of path.join in the test/test-all.js for consistency.

@paulmillr this is invalid AFAICT


Add support for mocking requests, websockets

Mocker should be able to store/cache/maybe persist responses

Already there???


When use aws_for_fluentbit disable create cloudwatch log group, will let fluentbit config setting LogGroupName to ""

Description
Please provide a clear and concise description of the issue you are encountering, and a reproduction of your configuration (see the examples/* directory for references that you can copy+paste and tailor to match your configs if you are unable to copy your exact configuration). The reproduction MUST be executable by running terraform init && terraform apply without any further changes.
If your request is for a new feature, please use the Feature request template.

[X] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Module version [Required]: v1.16.2

Terraform version: v1.9.5

Provider version(s):

Terraform v1.9.5
on darwin_arm64
+ provider registry.terraform.io/alekc/kubectl v2.0.4
+ provider registry.terraform.io/hashicorp/aws v5.61.0
+ provider registry.terraform.io/hashicorp/cloudinit v2.3.4
+ provider registry.terraform.io/hashicorp/helm v2.15.0
+ provider registry.terraform.io/hashicorp/http v3.4.4
+ provider registry.terraform.io/hashicorp/kubernetes v2.27.0
+ provider registry.terraform.io/hashicorp/local v2.5.1
+ provider registry.terraform.io/hashicorp/null v3.2.2
+ provider registry.terraform.io/hashicorp/random v3.6.2
+ provider registry.terraform.io/hashicorp/time v0.12.0
+ provider registry.terraform.io/hashicorp/tls v4.0.5
+ provider registry.terraform.io/rancher/rancher2 v4.1.0

Reproduction Code [Required]
module "eks_blueprints_addon" {
  ...
  source  = "aws-ia/eks-blueprints-addons/aws"
  version = "1.16.2"
  ...
  enable_aws_for_fluentbit = true
  aws_for_fluentbit_cw_log_group = {
    create = false
    name   = local.cb_log_group_name
  }
  aws_for_fluentbit = {
    create_namespace = false
    namespace        = var.namespace
    chart_version    = "0.1.26"
    wait                    = false
    timeout              = 600
    values = [
      <<-EOT
      cloudWatchLogs:
        autoCreateGroup: false
        endpoint: https://logs.${var.region}.amazonaws.com.
        region: ${var.region}
      hostNetwork: true
      dnsPolicy: ClusterFirstWithHostNet
      tolerations:
        - operator: Exists
      affinity:
        nodeAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            nodeSelectorTerms:
            - matchExpressions:
              - key: kubernetes.io/os
                operator: In
                values:
                - linux
              - key: kubernetes.io/arch
                operator: In
                values:
                - amd64
                - arm64
              - key: eks.amazonaws.com/compute-type
                operator: NotIn
                values:
                - fargate
    EOT
    ]
  }
}

Steps to reproduce the behavior:
Expected  behaviour
Define Good format like that pic.

Actual behaviour
Because this condition let local.aws_for_fluentbit_cw_log_group_name define to "" to let configmap aws-for-fluent-bit to wrong format.

You haven't shared what value you are actually using - we can't see what your variable is
aws_for_fluentbit_cw_log_group = {
    create = false
    name   = local.cb_log_group_name # what is this value?
  }

local.cb_log_group_name
can been "eks-log-group". (It can be any value but not "".

Why do you not want Terraform to create the log group?

In our company, we use Karpenter to create machines. Karpenter runs on Fargate, and the logic for creating a new EKS Cluster is to create the EKS Cluster first, followed by creating a Fargate Profile for CoreDNS and Karpenter. We've configured Fargate to send logs to a LogGroup, so the LogGroup should be created first. However, in our current setup, the aws_for_fluentbit addon module depends on the completion of the Karpenter Helm install, meaning that the LogGroup won't be created until after Karpenter is installed.
We want to record Karpenter's logs to the specified LogGroup from the beginning (which was originally created by aws_for_fluentbit). Therefore, we need to pre-create the LogGroup so that the applications running on Fargate, as well as aws_for_fluentbit, can send logs to the pre-created LogGroup.


Misspelled file names in download links lead to internal server errors (500)

Package version (if known): 10.0.0
Context
We've noticed that our server sometimes produces 500 responses for file downloads, because the file names in the accessed download links are transformed to lower-case.
It seems that those requests are coming from bing bots.
Example
The following is an example of a valid file download link: https://test.researchdata.tuwien.ac.at/records/rardm-wb730/files/lurkin.jpg?download=1
If the file name part of the URL is misspelled however (e.g. by using upper-case), it becomes invalid: https://test.researchdata.tuwien.ac.at/records/rardm-wb730/files/LURKIN.JPG?download=1
Accessing the second link results in a "500 internal server error" page, and the following stack trace on the server:
Exception on /records/rardm-wb730/files/LURKIN.JPG [GET]
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 2525, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1822, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1820, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1796, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
  File "/usr/local/lib/python3.8/site-packages/invenio_app_rdm/records_ui/views/decorators.py", line 79, in view
    return f(**kwargs)
  File "/usr/local/lib/python3.8/site-packages/invenio_app_rdm/records_ui/views/decorators.py", line 159, in view
    return f(**kwargs)
  File "/usr/local/lib/python3.8/site-packages/invenio_app_rdm/records_ui/views/records.py", line 171, in record_file_download
    return file_item.send_file(as_attachment=download)
AttributeError: 'NoneType' object has no attribute 'send_file'

Expected behavior
I would expect one of the two statements to be correct:

The file name in download links is case-insensitive
A misspelled file name in download links leads to a 404 page

IMO, the second one would be more reasonable.
I couldn't find any spot where Invenio would generate misspelled links.
It's not Invenio's fault if clients perform any kind of text manipulation on links before accessing them.
Further, we should probably add a robots.txt to prevent search bots from accessing file download links as well.

The problem happens when the file_item object in the record_file_download view function is set to None.
The object is being passed as keyword argument by the pass_file_item decorator.
This can happen if the file name isn't found in the record's file list in the files_service.get_file_content().
Given that there's an error handler for KeyError defined in the blueprint, I could see two simple fixes:

Raise a KeyError in the decorator or view function if the service returns None as file
Have the service raise a KeyError if the file can't be found

The behavior for the service in the latter case could be made controllable by an argument to the method (e.g. raise_=True). But I have a feeling that this needs to be very well documented to not cause confusion.
We could wrap this in a custom exception, but I'm not sure if this is any better.
So I'm slightly leaning towards the first approach, probably handle it in the decorator because that's "closer" to the service call.
We should check if there's any other spots where files_service.get_file_content() is called but not checked for None.


Publish permission with facebbok

Hi,
I'm trying to request publish permission while login with facebook but I'm not getting any result. I've created a test app in facebbok and tryied this config:
Client:
Accounts.ui.config({
requestPermissions: {
facebook: ['publish_actions']
}
});
Server:
ServiceConfiguration.configurations.remove({
service: 'facebook'
});
ServiceConfiguration.configurations.insert({
service: 'facebook',
appId: '',
secret: '***'
});
The loginUrl that I can see in the browser console does not include the publish permission in the scope.
I'm I doing it right?

Solved with the info in
Add configurable options for 3rd party login services #179

Yeah, it's a bit of a mess right now, but this is already implemented in the current code for v2.0.
Hopefully we'll have an alpha release soon ;-)


resources: fix config update container creating missing share dirs

The configuration update container will create missing share directories if they don't exist (as well as setting up initial perms). However, this will not work if the container doesn't have access to the data volume. Give the container access to the data volume.

I 100% hit the merge button and github closed the PR on me. Two times! This one I carefully made sure I hit the correct button.... sure enough. blammo. closed pr!


Add 404 support to the server

hi @bep I want to try to resolve this issue. Can you tell me how to start?

This was recently added, I forgot to close this issue.

Okay. Cooll


Cron Subsystem should be exposed as a plugin

The title.
It has implication. A programmatic access enables Cowj system to act like programmatic scheduling system to enable them to trigger jobs.

This PR talks about it : https://github.com/nmondal/cowj/pull/106

This is now done.


Gradient min max

Is your feature request related to a problem? Please describe.
Not related to a problem, just a limitation.
Describe the solution you'd like
Is it possible to use gradient over a defined min max instead of the min/max of the line (radial bar/else).
In vertical gradient.
So, with a radient line of green -> red.
Actually the vertical higher of the line is red, and bottom is green.
The requested feature is to have the gradient set to predefined value.
As example, in multiple gradient line, bottom color of lines would be green/cyan.
And will get into orange if the value is superior to let's say 50.
So bottom of 50 is green/cyan and top is orange/red (colors are example ofc).
Is it already possible to achieve something like this ? Looked at the options of fill, but didn't there isn't min/max value.
I can provide some illustration if it is not clear.

I more clear explanation with context https://github.com/apexcharts/apexcharts.js/issues/568#issuecomment-626731175

I am looking for this feature as well.


Support Path completion in AutoComplete for custom types

Hello,
I have an @Option being a List<Repository> where Repository is a custom type injected through a TypeConverter.
I recently integrated the AutoComplete bash completions generation and since then, I do have command completions, but I lost the path completions for this option.
I suspect the bash completion to complete overrule the shell built-in path completion and that the solution for this is to integrate the path completion within the completion generated by AutoComplete.
Looking at the documentation I see support for java.io.File/java.nio.file.Path when it comes to path completion.
I wasn't however able to find a way to force it for custom types.
Looking at the code in picocli.AutoComplete generatePositionalParamsCases/generateOptionsCases I see the java.io.File/java.nio.file.Path cases are hardcoded.
Do you think it would be valuable to make this configurable?
If yes, what would be the recommended approach?

Through a new field in the @Option and @Parameters annotations?
Through an annotation/default method on the TypeConverter?
I don't know if this approach is compatible with the AutoComplete, but I assume, it could be possible to scan the classpath for type converters of a given @Option/@Parameters  without instantiating the command line itself.
Through an additional optional parameter picocli.CommandLine#registerConverter method?
This seems not really compatible with the AutoComplete command that does not instanciate the command line but only needs the main command and the factory.

From an API point of view, I like option 2 because you declare the behavior once and it applies everywhere it is used, but it might be more complex to implement/not matching the philosophy.
Regards,
PS: I might be able to find some time to work on this topic if you deem it valuable

I like the idea to make this configurable.
One additional way to accomplish this that I thought of was to have a command line option for the AutoComplete.App and AutoComplete.GenerateCompletion commands. Something like this:
@Option(names = "--fileTypes", split=",",
    description = "Comma-separated list of fully qualified custom types for which to delegate to built-in file name completion.")
List<Class<?>> fileTypes = new ArrayList<>();

Then the logic could change to this:
} else if (type.equals(File.class) || "java.nio.file.Path".equals(type.getName()) || fileTypes.contains(type)) { //<-- check custom types
    buff.append(format("%s    %s (( currIndex >= %d && currIndex <= %d )); then\n", indent, ifOrElif, min, max)); // as before
    buff.append(format("%s      compopt -o filenames\n", indent));                                    // as before
    buff.append(format("%s      positionals=$( compgen -f -- \"%s\" ) # files\n", indent, currWord)); // as before

We would have to test this, and there may be other ideas.
What do you think?
If you have time to work on this, that would be great!

So your proposal is to not have this information as part of the command spec, but to have it as a parameter when calling the AutoComplete command. Did I get this right?
Sounds way easier to implement than to actually make it part of the model but maybe not as declarative in the approach.
Still it can a be a fairly decent first step, I like it. Later on, if need be we could still add auto-determination from the model for that new option. But KISS first!
I'll give it a try this week.


shim: devDependencies instead of dependencies

I may have missed it in the docs, etc. but how do I make the shim (deno.ns) show up under devDependencies instead of under dependencies?
My package code doesn't rely on Deno but my tests are written using Deno.test. Do I still need the shim? If so, I don't want NPM to show that my package depends on deno.ns as it only depends on it for development so I want it to be added to devDependencies instead of dependencies.

It should happen automatically. If you look in the output directory under the src folder it should show where the shim is being used in the distributed code.
Right now, the shim gets overzealously injected. Probably #37 will help you here once it's implemented or you can follow the instructions outlined in the "preventing shimming" section of the readme: https://github.com/denoland/dnt#preventing-shimming

found it; thank you


Addittion to Notification Settings (sound)

While "Flash" (in RuneLite -plugin's config) has the option to "Flash until cancelled", I suggest that there would be a similar setting for the sound as well, meaning that you would hear the notification sound repeatedly until cancelled.

Any update?

Any update?


[Defect] Black background added for transparent images

Description
For allowsEditing set to true, every images picked from library with alpha channel will be black backgrounded:

When editing is disabled, images have white background as expected:

How to repeat issue and example
Here is my options passed to ImagePicker.showImagePicker() as first argument:
{
  allowsEditing: true,
  cameraType: 'front',
  maxHeight: 512,
  maxWidth: 512,
  mediaType: 'photo',
  storageOptions: { path: 'images', skipBackup: true },
  title: null,
}

Solution
No solutions proposed yet.
Additional Information

React Native version<EMAIL_ADDRESS>Platform: iOS (not tested on Android)
Development Operating System: macOS Sierra 10.12.5
Dev tools: Xcode 8

Did you find a workaround for this?

Have you figured out this issue? I cant seem to remove the white backgrounds from png image. It was black until I removed the editing - now its white...

Hi @nica0012 - did you manage to figure this out?

@TommyHarty I sure did, instead or using response.uri, simply use response.origURL.

@nica0012 - my app crashes when I use response.origURL

@TommyHarty if I remember correctly I think it may only work on IOS.

I have found the same issue here for ios 13 black theme. The image list ls also dark and transparent.

Problem still persists on Android.

I discovered the problem. When using "maxHeight" or "maxWidth", the image is converted from PNG to JPEG, causing the transparent area to be converted to black.
Is there any way to keep the image in PNG?

Any updates on this?


[BUG] EKS imported cluster shows intermittent blank page

SURE-7400
Rancher Server Setup

Rancher version: 2.7.6
Installation option: Helm Chart

If Helm Chart, Kubernetes Cluster and version RKE1

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: v1.26.11-eks-8cb36c9

Cluster Type: Downstream

If downstream, what type of cluster? Imported

Imported = Running kubectl apply onto an existing k8s cluster
Hosted = EKS,
Infrastructure Provider = AWS

User Information

What is the role of the user logged in? Admin

Describe the bug
Error shown in the screenshot is appearing intermittently when working with the EKS imported cluster
To Reproduce
It is is an intermitent error hapeen when working with the UI console
Result
Expected Result
A fix for not getting the cannot read properties error
Screenshots

@elenagarciam Thanks for creating this issue. Would you be able to provide a title and expand on the steps to reproduce? If clear steps are unknown, can you provide the page/s that you see the error on?

Linking some related issues

https://github.com/rancher/dashboard/issues/9665 (possibly related underlying issue)
https://github.com/rancher/dashboard/issues/9542 (do not show dev full blank page errors in prod)

Fixed in 2.8.0 and an upcoming 2.7.x via https://github.com/rancher/dashboard/issues/10062


Get total number of videos of a channel or playlist

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.10.13 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
I'd like to get the number of videos in total a channel or a playlist contains. I tried the following command:
yt-dlp --print "%(playlist_count)s" --flat-playlist https://www.youtube.com/@hubermanlab | jq .

However, it has some issues:

The channel I used above has 178 videos, and the command returns the number "178" 178 times in a row, I only need it to return that number once.
The more videos a channel has, the longer it takes for the command to finish.

The total number of videos a channel has is already displayed on the website, I wonder if yt-dlp can use that to make this process quicker? Thank you!
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

The channel I used above has 178 videos, and the command returns the number "178" 178 times in a row, I only need it to return that number once.

-I -1

The more videos a channel has, the longer it takes for the command to finish.

For channels, youtube doesn't show the number of items anywhere and so we have to parse through all the items to get the count, which will be slow. For playlists however, we can get it quickly
❯ yt-dlp https://www.youtube.com/playlist?list=UU2D2CMWXMOVWx7giW1n3LIg -I0 -O playlist:playlist_count
178

@pukkandan On mobile the video count is on the channel home page, on desktop it's in the new about popup but it's only a short number e.g. 3.4K, so it'll be inaccurate compared to counting the videos.


Bug Fixing

Death knigths and executioners are no longer suicidal #2753
Kultiras cannon has an icon now #2711
Drop the Anchor tooltip fixed #2527
Fixed the Priestess of the moon icons #2583
Hotkeys position for Dalaran main building should no longer conflict #2539
(spiked barricade no longer conflicts with navigation, so this must have been fixed in an earlier patch)

Bugfixing the bugfix

[ ] move the dalaran spell to Z


Crashing Sketch 54.1 on Import

Successfully exported text styles JSON file, but am crashing Sketch 54.1 when I try to import. I am running MacOS Mojave 10.14.6

Resolved by upgrading plugin to v 1.81.
Fyi, I got the original plugin here, so you should probably change this version to 1.81 as well


Reasoning about remainders

As part of exploring CSE in Kontrol, a few issues have come up. Specifically:

The backend exhibits spurious branching because it's not able to prove a large remainder (11 branches) UNSAT. An initial investigation with @geo2a revealed that the satisfiability of the remainder is not checked with respect to the current path constraint. The corresponding bug report is attached: it exhibits a 9-way branching when only an 8-way branching is expected).
The remainders are not minimised, in the sense that even though some branches have been cut as infeasible, we still get the negation of their constraints in the remainder (in the example, the branching is initially 11-way and then three branches are discarded).
The remainder is not propagated back using rule_predicate, which leads to a mischaracterisation of the branching by pyk, and causes an infinite loop in combination with 1).

From what I understand, the remainders are initially of the form #Not ( ... #Or ... ), which is then transformed into #Not (...) #And #Not (...). I believe that there is room for further simplification or minimisation at the #Or stage, perhaps even Z3 could be asked to minimise this before the transformation to #And.
nine-way-branch.tar.gz

With the georgy/booster-remainders branch, Booster can now apply the rules from the bug report, but it still produces a 9-way (instead of 8-way) branch. I.e. the behaviour is on-par with Kore. I have some thoughts on why this happens and also some wants about the rules. I start with the wants and then describe a possible transformation to the rule format that may help us to prune the remainder.
Format of CSE rules

Suggested changes to CSE rules

the rules must preserve definedness and should be marked as such, otherwise Booster will not be able to apply them
the rules need to have IDs. I do not think it matters much if they are not actually UniqueIDs, but it is very annoying to not have them, as it becomes very hard to disambiguate the rules in the backend logs

Experiment: shift information to the side condition

the rules have function symbols in the LHS, which makes them invalid rewrite rules and more akin to simplifications.

This actually may be the reason why the remainder condition is over-approximating. Once we've matched with the rule and checking the side condition (or checking the remainder after applying several rules), we have already lost the information. Perhaps if we had the matching information when computing the remainder, it would not over approximate.
With that in mind, we should consider transforming the rules in the following way:

bind the expressions in the LHS to a variable
add an equality (likely ==K should work) constrains

This transformation of the rule may just work and the rules apply without a remainder.
We can of course extend the matching code in Booster to allow (total) function symbols in LHS of rewrite rules (experiment in this PR), but this does not actually solve anything, as we'd not have enough information to prune the remainder.

CSE rules are specifically designed to have only symbolic variables on the LHS, with the exception of +Bytes and map concatenation. If you see any other ones, please let me know.
+Bytes are fine, we have an excellent array of rules that does unification on them, which is deterministic. Concatenation of maps is something to be implemented.
In addition, my experience tells me that we should never be introducing additional constraints into the path condition unless absolutely necessary, and this is not necessary for +Bytes and should not be necessary for map concatenation.

The remainder condition that we are dealing with is over-approximating because in the path constraints, some symbolic variables are initialised in some of the branches, causing the parameters of function symbols to be computed partially or even the function to be fully evaluated. This issue, I believe, is not related to what's in the structural part.


feat: 🎸 upgrade huggingface-hub

it fixes CreateCommitError for OpenOrca. We pin a specific commit to fix the issue without waiting for the release of 0.20. I disabled the pip-audit test because it's complex to get it with a pinned commit

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f70c43e) 83.44% compared to head (97117a2) 90.84%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2134      +/-   ##
==========================================
+ Coverage   83.44%   90.84%   +7.39%
==========================================
  Files          11       93      +82
  Lines         592     5961    +5369
==========================================
+ Hits          494     5415    +4921
- Misses         98      546     +448

Flag
Coverage Δ

jobs_mongodb_migration
86.69% <ø> (?)

libs_libcommon
92.02% <ø> (?)

services_api
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Added License and Project Admin

Description
Added License and Project admin in the readme file.
Please check it.
Thanks
Fixes #68
Type of Change:

Bugs
New Feature
Documentation

How Has This Been Tested?

Checklist:

[x] My PR follows the style guidelines of this project
[x] I have performed a self-review of my own code or materials
[x] I have commented my code or provided relevant documentation, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[ ] New and existing unit tests pass locally with my changes

Ok I'll do it

Ok I'll do it

tag me once you are done

@ritik307 I'm done with the changes.
Please check it.

Ok
Sorry I misunderstood.
I'll do it.

Done please check.

@felixfaisal Changes looks good to be. Approved the changes so kindly merge.


chore: move to cloudflare pages and remove plausible

Codecov Report

Merging #291 (c77c63f) into main (c2a338f) will not change coverage.
The diff coverage is n/a.

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

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #291   +/-   ##
=======================================
  Coverage   91.85%   91.85%
=======================================
  Files         117      117
  Lines        3658     3658
  Branches      471      471
=======================================
  Hits         3360     3360
  Misses        202      202
  Partials       96       96

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


Verifying a receipt in guest code doesn't work

Is there an existing issue?

[X] I have searched the existing issues

Experiencing problems? Have you tried our Discord first?

[X] This is not a support question.

Description of bug
I want to do some external receipt verification inside the guest code. When I do the verification in the host, the receipt gets verified. However, when I send the receipt to the guest and do the verification in the guest, the verification doesn't succeed and I'm receiveing a message: Guest panicked: (...): verification indicates proof is invalid
Steps to reproduce

Create a small example that uses the hello-world example:

host code:
pub fn verify_hello_world(a: u64, b: u64) -> Receipt {
    let hello_world_env = ExecutorEnv::builder()
        // Send a & b to the guest
        .write(&a)
        .unwrap()
        .write(&b)
        .unwrap()
        .build()
        .unwrap();
    let hello_world_receipt = default_prover().prove(hello_world_env, HELLO_ELF).unwrap().receipt;

    let env = ExecutorEnv::builder()
        // Send hello-world receipt to guest for verification
        .write(&hello_world_receipt)
        .unwrap()
        .build()
        .unwrap();

    default_prover().prove(env, THIS_ELF).unwrap().receipt;
}

guest code:
fn main() {
    let receipt: Receipt = env::read();
    receipt.verify(HELLO_ID).expect("Receipt should get verified (but it doesn't :( )");
}

Interesting... so we run something like this in our CI testing. @Pasifaee what do you get when you run cargo risczero --version?


Can't run carthage update build

carthage update --platform iOS --verbose
*** Fetching ExpandingMenu
*** Checking out ExpandingMenu at "0.3.1"
*** Skipped building ExpandingMenu due to the error:
Dependency "ExpandingMenu" has no shared framework schemes for any of the platforms: iOS

same for me, any idea why??

I fixed this issue 346d2ec53a328e89fd1289a025c6341558d748ee
You can install this library ver 0.4.0 by Carthage.


First Build of Conductor

The following features should be implemented:

[x] Assign a contract to a purchasing agent ("conductor")
[x] A purchasing agent can move a contract through its stages
[ ] Handle completed contracts.

[x] A completed contract is cloned
[x] The purchasing agent is sent to a new page to fill in metadata
[x] The cloned contract doesn't show up in the search on Wexplorer/Scout until all appropriate metadata is filled in (new Spec #, new controller #, new awarded company).
[x] When cloned, all followers get notified, and stars/followers move to the new contract.

[x] Cloned contracts should have self-referencing links
[ ] Stages should have boolean properties of different actions that can be taken on them (right now I can only think of sending email reminders and posting to Beacon. Am I missing any @shellicious @hamhands ?), these boolean should populate potential actions in the detail UI.
[x] Basic listing UI (improvements to be made following user testing)
[ ] Basic detail UI (improvements to be made following user testing)
[ ] Unit tests

Out of scope, but features needed for future:

Adding additional stages
Organizing stages into flows
Handling problems when flows are changed while contracts are in specific stages.
Metadata page allows for company creation/assignment to contracts.

For detail UI:

[ ] Gray background on current stage
[ ] Title for current stage displayed
[ ] Build to spec to be attached by @shellicious
[ ] Ability to move backwards (to a specific destination)
[ ] Ability change the enter/exit date manually
[ ] Listing UI -- move view buttons into last column with dropdown

Here's how this is starting to shape up


In sqlserver, an error will be reported when the field is of date type

Laravel Version
8.83.27
PHP Version
8.3.0
Database Driver & Version
SQLSERVER2014
Description
The birthday field in the database uses date type, but when I set protected $casts = ['birthday'=>'date:Y-m-d'] in the model, an error will be reported. The date stored in date is 1962-06-20. , I don’t know what causes,This was no problem before, but I don’t know if it has something to do with the PHP version.
Steps To Reproduce

After testing, it is a problem with php8.3. If you switch back to php8.2, it will disappear.


Game very slow

Hi, i downloaded your fix but now in Yakuza 4 the game is very slow
Not laggy but all is like at 50%
Thanks for your help !

Does this still happen with a newly released Build 2? In theory it is unlikely to, but I'd like to make sure.

I have not heard a single report about slowdown with the Build 2 yet, and on paper it should be impossible to happen (unlike with the Build 1), so I am going to go ahead and close this issue.
Please reopen if it's still the cause (unlikely).


OTLP exporters should set User-Agent header as per specification

Is your feature request related to a problem? Please describe.
Currently the User-Agent header is not set by our exporters as defined in the specification.
Describe the solution you'd like
The User-Agent Header should be added as OTel OTLP Exporter JavaScript/<exporter version> in all OTLP exporters.
Additional context
Specification: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/protocol/exporter.md#user-agent

not stale & working on this.

@pichlermarc See https://github.com/open-telemetry/opentelemetry-specification/pull/3052 for the corrected format.

@pichlermarc any update?

@pichlermarc See open-telemetry/opentelemetry-specification#3052 for the corrected format.

Thanks!

@pichlermarc any update?

@dyladan No, not yet. I did not have time to work on that for a while. Also, it's not a single line change unfortunately.
I'll unassign myself for now in case someone has cycles and wants to pick it up.

The header is set for all Node.js exporters as of #4397, but still missing for browser exporters. It would probably make sense to centralize the logic in the otlp-exporter-base package.

@raphael-theriault-swi yep - thanks for pointing that out, I completely forgot about this issue. We don't set the User-Agent it used to be prohibited, but it will still lead to error logs in some browsers so we opted not to do it.
I'm closing this issue as we can consider it done :)


Font doesn't translate to all menus

This is a known issue with card-mod and outside of my ability to fix. It is implicitly included in the Known issues portion of the readme.

Pshh.  Who actually reads the README?
j/k  Thanks

Pshh. Who actually reads the README?
j/k Thanks

I don't. :p


OAuth provider redirect broken in CAS 4.2.5 (and possibly 4.2.6)

Hi,
I am experiencing problems using CAS as an OAuth server. Tested with 4.2.5 on Jetty built via Maven overlay.
(Tried 4.2.6, but there it may fail already due to the fact that 4.2.6 is not able to find the (defined) metrics registry)
Discussion on cas-user: [cas-user] Oauth2.0 not redirecting back to app
Here is the list of services registered:
[STDOUT] 2016-10-04 13:29:54,461 DEBUG [org.jasig.cas.services.DefaultServicesManagerImpl] - <Adding registered service localhost:8443/cas/oauth2.0/callbackAuthorize>
[STDOUT] 2016-10-04 13:29:54,462 DEBUG [org.jasig.cas.services.DefaultServicesManagerImpl] - <Adding registered service ^https?://localhost:\d+/callback>
[STDOUT] 2016-10-04 13:29:54,462 DEBUG [org.jasig.cas.services.DefaultServicesManagerImpl] - <Adding registered service https?://localhost:5000/.*>
[STDOUT] 2016-10-04 13:29:54,462 DEBUG [org.jasig.cas.services.DefaultServicesManagerImpl] - <Adding registered service localhost:8443/cas/oauth2.0/callbackAuthorize>
[STDOUT] 2016-10-04 13:29:54,462 DEBUG [org.jasig.cas.services.DefaultServicesManagerImpl] - <Adding registered service https://localhost:8443/cas/oauth2.0/callbackAuthorize>
There are two service definitions for the OAuth2.0 callbackAuthorize servce.
The first one seems to be registered by default by the oauth plugin and has a serviceId w/o a protocol.
The second one in the list (at the end) is the manual service registration I posted here already:
{
"@class": "org.jasig.cas.support.oauth.services.OAuthCallbackAuthorizeService",
"serviceId": "https://localhost:8443/cas/oauth2.0/callbackAuthorize",
"name": "OAuth Callback url",
"id":<PHONE_NUMBER>5849,
"description": "OAuth Wrapper Callback Url",
"logoutType": "BACK_CHANNEL",
"accessStrategy": {
"@class": "org.jasig.cas.services.DefaultRegisteredServiceAccessStrategy",
"enabled": true,
"ssoEnabled": true,
"requireAllAttributes": true,
"caseInsensitive": false
}
}
First issue: I get this (broken) URL in the Location header in the 302 response returned by /oauth2.0/authorize:
localhost:8443/cas/login?service=https%3A%2F%2Flocalhost%3A8443%2Fcas%2Foauth2.0%2FcallbackAuthorize
When I copy the Location and add a proper protocol (https://...) I continue to the next step.
Without my own service definition I run into the error page that my application is not authorized.
When I add my custom service definition for the OAuth service, I am finally redirected to the calling application.
The main culprit seems to be the (automatically generated?) service for the OauthAuthorize callback service which produces a broken redirect URL.
I have included the following in my report:

[x] CAS server version
[x] Container/environment type/version (Tomcat, Jetty, Java, Linux/Windows, etc)
[] Steps to duplicate the problem
[x] Stacktraces and CAS logs where appropriate.
[] Configuration/Overlay pushed to https://github.com/cas-projects/cas-issues/

Why would you consider that url to be broken?

Ah, let me clarify: by broken you mean the actual host name. Not the service parameter which is what I was going for.
This is a problem with your configuration. You need specify the proper cas name and prefix.

@mmoayyed, @leleuj asked me to create an issue as the outcome of our discussion on cas-user. So just shrugging it off as pilot error does not fix the issue.
This is the manual one I built (in src/main/resources/services in the overlay project):
{
  "@class": "org.jasig.cas.support.oauth.services.OAuthCallbackAuthorizeService",
  "serviceId": "https://localhost:8443/cas/oauth2.0/callbackAuthorize",
  "name": "OAuth Callback url",
  "id":<PHONE_NUMBER>5849,
  "description": "OAuth Wrapper Callback Url",
  "logoutType": "BACK_CHANNEL",
  "accessStrategy": {
    "@class": "org.jasig.cas.services.DefaultRegisteredServiceAccessStrategy",
    "enabled": true,
    "ssoEnabled": true,
    "requireAllAttributes": true,
    "caseInsensitive": false
  }
}

I think this service definition should work.
BUT I also find in the target classes folder this additional service definition which I did not create and which seemingly appears out of nowhere:
  @class: org.jasig.cas.support.oauth.services.OAuthCallbackAuthorizeService
  serviceId: localhost:8443/cas/oauth2.0/callbackAuthorize
  name: OAuth Callback url
  id:<PHONE_NUMBER>79567
  description: OAuth Wrapper Callback Url
  logoutType: BACK_CHANNEL
  accessStrategy:
  {
    @class: org.jasig.cas.services.DefaultRegisteredServiceAccessStrategy
    enabled: true
    ssoEnabled: true
    requireAllAttributes: true
    caseInsensitive: false
  }
}

The service Id here is broken and I frankly have not found yet where it comes from.
Getting rid of this second definition would solve this problem - if I had the slightest idea where it came from.

It might only affect CAS if the service definitions are not read from files but from the classpath - but it is annoying nonetheless

I didn't shrug it off. I provided an explanation. Whether or not you like that explanation is up to you.
The service is created by CAS automatically based on your settings. Evaluate those and see where that might come from.

Easiest way to diagnose this, would to provide an overlay as the issue template suggested so we can try to duplicate the issue.

Which setting would then cause the missing protocol prefix to the generated ServiceId?
Because the rest (web application path, hostname and port) are generated correctly.

Could you paste a copy of your cas.properties file?

server.name=localhost:8443
server.prefix=${server.name}/cas

cas.securityContext.adminpages.ip=127\.0\.0\.1|0:0:0:0:0:0:0:1

host.name=localhost

ticketreg.database.ddl.auto=create-drop
ticketreg.database.dialect=org.hibernate.dialect.MySQLInnoDBDialect
ticketreg.database.batchSize=10
ticketreg.database.driverClass=org.mysql.jdbcDriver
ticketreg.database.url=jdbc:mysqldb:cas-ticket-registry
ticketreg.database.user=root
ticketreg.database.password=
ticketreg.database.pool.minSize=6
ticketreg.database.pool.maxSize=18
ticketreg.database.pool.maxWait=10000
ticketreg.database.pool.maxIdleTime=120
ticketreg.database.pool.acquireIncrement=6
ticketreg.database.pool.idleConnectionTestPeriod=30
ticketreg.database.pool.connectionHealthQuery=select 1
ticketreg.database.pool.acquireRetryAttempts=5
ticketreg.database.pool.acquireRetryDelay=2000
ticketreg.database.pool.connectionHealthQuery=select 1

svcreg.database.ddl.auto=create-drop
svcreg.database.hibernate.dialect=org.hibernate.dialect.MySQLInnoDBDialect
svcreg.database.hibernate.batchSize=10
svcreg.database.driverClass=org.mysql.jdbcDriver
svcreg.database.url=jdbc:mysql:cas-ticket-registry
svcreg.database.user=root
svcreg.database.password=
svcreg.database.pool.minSize=6
svcreg.database.pool.maxSize=18
svcreg.database.pool.maxWait=10000
svcreg.database.pool.maxIdleTime=120
svcreg.database.pool.acquireIncrement=6
svcreg.database.pool.idleConnectionTestPeriod=30
svcreg.database.pool.connectionHealthQuery=select 1
svcreg.database.pool.acquireRetryAttempts=5
svcreg.database.pool.acquireRetryDelay=2000
svcreg.database.pool.connectionHealthQuery=select 1

tgc.encryption.key=[...]
tgc.signing.key=[...]
okie should be created only under secure connections.
tgc.secure=true

webflow.encryption.key=[...]
webflow.signing.key=[...]

ldap.url=[...]
ldap.useStartTLS=[...]
ldap.use.ssl=[...]
ldap.rootDn=[...]
ldap.baseDn=[...]
ldap.connectTimeout=[...]
ldap.managerDn=[...]
ldap.managerPassword=[...]

ldap.pool.blockWaitTime=500
ldap.pool.idleTime=100
ldap.pool.maxSize=5
ldap.pool.minSize=2
ldap.pool.validateOnCheckout=true
ldap.pool.validatePeriodically=true
ldap.pool.validatePeriod=1000
ldap.pool.prunePeriod=3000

ldap.authn.searchFilter=SAMAccountName={user}

ldap.allowMultipleDns=false

ldap.subtree.search=true

accept.authn.users=casuser::Mellon

memcached.servers=localhost:11211
memcached.hashAlgorithm=FNV1_64_HASH
memcached.protocol=BINARY
memcached.locatorType=ARRAY_MOD
memcached.failureMode=Redistribute

cache.monitor.warn.free.threshold=10
cache.monitor.eviction.threshold=0

Added cas-issues PR: https://github.com/cas-projects/cas-issues/pull/19
I could reproduce the problem as well with the version deployed on tomcat (tomcat 7).
2016-10-13 12:56:24,734 DEBUG [org.jasig.cas.services.DefaultServicesManagerImpl] - Adding registered service sso.xing.hh:443/oauth2.0/callbackAuthorize
2016-10-13 12:56:24,734 INFO [org.jasig.cas.services.DefaultServicesManagerImpl] - Loaded 6 services from JsonServiceRegistryDao.
2016-10-13 12:56:26,133 DEBUG [org.jasig.cas.support.oauth.web.OAuth20WrapperController] - method : authorize
2016-10-13 12:56:26,134 DEBUG [org.jasig.cas.support.oauth.web.OAuth20AuthorizeController] - method : authorize
2016-10-13 12:56:26,134 DEBUG [org.jasig.cas.support.oauth.web.OAuth20AuthorizeController] - client_id : 30ceed1c0a94bf4fad35f53e6e0b0442
2016-10-13 12:56:26,135 DEBUG [org.jasig.cas.support.oauth.web.OAuth20AuthorizeController] - redirect_uri : https://localhost:3000/callback
2016-10-13 12:56:26,135 DEBUG [org.jasig.cas.support.oauth.web.OAuth20AuthorizeController] - state : 3(#0/!~
2016-10-13 12:56:26,136 DEBUG [org.jasig.cas.support.oauth.web.OAuth20AuthorizeController] - callbackAuthorize : https://sso.xing.hh/oauth2.0/callbackAuthorize
2016-10-13 12:56:26,137 DEBUG [org.jasig.cas.support.oauth.web.OAuth20AuthorizeController] - loginUrlWithService : sso.xing.hh:443/login?service=https%3A%2F%2Fsso.xing.hh%2Foauth2.0%2FcallbackAuthorize

@mmoayyed I had a look at the template cas.properties and found there that the server.name in cas.properties actually includes the protocol prefix. In mine it is missing, so I admit that it actually seems to be pilot error. My bad. Thanks for looking into it.

No worries. Glad you were able to figure this out and thanks for following up.


Mangohud displaying metrics values only when Nvidia Reflex is enabled

I'm having Mangohud from "custom-metric" branch installed and working.
If Nvidia Reflex is enabled in game, I can see the latency graph (the green line) and latency values (actual numbers) both updated in real time.
But if Nvidia Reflex is disabled, only the graph (green line) remains working real-time, while the latency numbers displayed are stuck to the last whatever values they had when NR was enabled.
Ideally I'd like to see the latency values still displayed and updated real-time even when NR is not enabled, so I could compare the latencies in both situations.
I'm not sure if this is a bug, or something not implemented.

This is working as intended. With Reflex disabled the sleep callback doesn't get called which means we don't really get to measure the latency.

Understood, thank you.


Support duplicate Symbols

There are a couple of duplicate symbols. Eg
        "id": "tokamak-network",
        "symbol": "ton",
        "name": "Tokamak Network",

        "id": "ton-crystal",
        "symbol": "ton",
        "name": "TON Crystal",

cointop should support this.  At the moment, due to recent change to hide "duplicate" coins #130 #138 , only the first/highest-rank coin is shown.
Related issue - duplicate symbol vs name, eg:
        "id": "dogecoin",
        "symbol": "doge",
        "name": "Dogecoin",

vs
        Buff Doge Coin (DOGECOIN)

With the fix in #222 cointop now supports duplicate (symbol) coins.
$ ./bin/cointop holdings
           name symbol     price        holdings        balance  24h%   %holdings
Tokamak Network TON         8.72              10           87.2 -6.59       92.48
TON Crystal     TON     0.354533              20           7.09 -0.81        7.52


[BUG] DIN sample refers to old version of NVTabular and produces error when running w/ 22.09 container

Describe the bug
The DIN sample in the repo (https://github.com/NVIDIA-Merlin/HugeCTR/tree/main/samples/din) seems to be using outdated NVTabular API and produces an error when running in the 22.09 container
To Reproduce
Follow the instructions shown in the sample page.
Expected behavior
The sample runs without an error
Additional context
Have not tested w/ 22.10 which was released recently, but expecting to not work either as the sample hasn't been updated for 4 months.

22.10 also has the same issue. The 4_nvt_process.py will run into an error. "No module named column_group in nvtabular" which is an old API from v0.6.1. After I change this line, it gives me another error.

Anyone can help with this?
Thanks!

@Spidey0918 The fixed script will be available in next release. For a quick fix, you can replace the line features = LABEL + ColumnGroup(CAT_COLUMNS) with features = LABEL + CAT_COLUMNS >> FillMissing() and add from nvtabular.ops import FillMissing on the top. Use this fix is not sufficient to remove all the warnings but should be enough to get the script working. Thanks and let me know if you have any other questions.

@jershi425 Thanks! It helps a lot. I can get the din model working.


WTD

WTD: Edits from geraldinewhitt in Portland at WTD conference. Apply template, copyedit, apply style guide.

This change is

Ping!  Can you sign the CLA?

I'm trying to help w/ the CLA

Thanks! :-)
Gen
  From: Andrew Chen<EMAIL_ADDRESS>
To: kubernetes/kubernetes.github.io<EMAIL_ADDRESS>Cc: geraldinewhitt<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Sent: Sunday, May 14, 2017 6:10 PM
Subject: Re: [kubernetes/kubernetes.github.io] WTD (#3757)
Merged #3757.—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Rename the CLI to kubectl cert-manager

What this PR does / why we need it:
Renames the CLI tool to the public name
Release note:

NONE

/lgtm

/kind cleanup


Fixed Issue #371

As per Issue #371 , I've done the appropriate modifications.

Please take a look at my reply in the issue. This is wrong usage of the test_size parameter.
Nevertheless, if you want to fix this, please take a look at the contributing guide here https://github.com/Qiskit/qiskit-machine-learning/blob/main/CONTRIBUTING.md and consider similar changes to other datasets.

Yes, I just saw your response to the issue and saw where I was going wrong. Really sorry for the inconvenience.

Yes, I just saw your response to the issue and saw where I was going wrong. Really sorry for the inconvenience.

@bopardikarsoham my apologies, you are right. In the wine and iris datasets the test_size is applied in the same way as you mentioned in the issue. So, the fix looks correct.

I also had a query about why test_size is not set to 0.3 in datasets like wine and iris, as shown at https://github.com/Qiskit/qiskit-machine-learning/blob/main/qiskit_machine_learning/datasets/iris.py#L36.

Yes, I just saw your response to the issue and saw where I was going wrong. Really sorry for the inconvenience.

@bopardikarsoham my apologies, you are right. In the wine and iris datasets the test_size is applied in the same way as you mentioned in the issue. So, the fix looks correct.

Yes, l saw that too. That's why I made the change.

I also had a query about why test_size is not set to 0.3 in datasets like wine and iris, as shown at https://github.com/Qiskit/qiskit-machine-learning/blob/main/qiskit_machine_learning/datasets/iris.py#L36.

I'd say either test_size parameter should be set to 0.3 or to the passed test_size value everywhere.

I am not sure if this is something we would want to do at this point. The dataset is initially split such that 0.3 of it is kept for test. Then out of the data that is split 0.7/0.3 ration,  training size and test_size amounts are taken. Maybe the ratio of 0.7 to 0.3 as per the dataset split is different than training_size/test_size split such that its not so great. Either way I think this will end up as breaking change and with the datasets being deprecated for removal I think it may be better to live with it. A user can directly use breast cancer etc if the behavior here does not work for them.

I close this PR as the datasets are deprecated.


Update the repositories reference

I found this tool on this link http://blogs.msdn.com/b/paolos/archive/2015/03/02/service-bus-explorer-2-6-now-available.aspx, but only this current link reference is working. Where I can to find the executable/installer ? Thanks.

There's no such ATM. You'll have to D/L the project, compile and grab the artifacts.
Should we look into something like https://github.com/Squirrel/Squirrel.Windows for a quick setup?

PS: squirrel will still require to open the project, but can generate an install in a fast way
/cc @paolosalvatori

Hi guys
I know I should use some tools to create an exe whenever a new version of the tool is available, but I'm really busy and I can't spend time on this. Sorry about that. :(
Ciao
Paolo

@paolosalvatori I can add squirrel to the solution. While it's not the most convenient way to generate an installer, it is a way to get it with the solution and doesn't require CI/build server. Would that work?

Hi Liam
yes please, it looks like a lightweight solution to generate an installer. Please, go ahead and keep me informed! Thanks!
Ciao
Paolo

@LiamFeldman was me on my sons account :blush:
I'll look into that.

Ah ah ah Thanks!

@paolosalvatori shouldn't this stay opened until done? :) or you'll be opening a new issue?


Include _solib_local for MKL-DNN libs

_solib_local/libmklml_intel.so is not getting included in the package.
build_pip_package.sh has been updated to copy *solib* instead of just _solib_k8, so just update setup.py to include it.
See https://github.com/tensorflow/tensorflow/issues/13711 for details.

Can one of the admins verify this patch?

Change looks good, but could you take care of the CLA?

u need to pass cla ~ i was there, now i passed , just follow me:
#14867

Hi, I tried your code and it didn't solve the 'libmklml_intel.so: cannot open...' issue. After having a look at the find_files(), I found out that this function's second argument don't support filename pattern matching like bash script do.  And I my case I just need to change '_solib_k8' to '_solib_local', haven't figured out a general way to fix it though.:confused:

@ic0n You are completely right. I fixed it locally, but forgot to update this PR.

I signed it!

@tensorflow-jenkins test this please


Improve support pre-JB for Activity detection

This uses an older API (available to ICS+) to track activities instead
of relying on an ugly reflection trick known to work only on JB+ (and of
course not future proof).  There's a slight overhead increase in
Stetho.initialize as a result, but the timings added indicate it is
still very small (1-2ms on my Nexus 5).
The mechanism for tracking activities is partially exposed via
ActivityTracker which has a manual add/remove method which allows
developers to add support for pre-ICS even by hooking their own
application onCreate/onDestroy events just like HierarchyViewer's
ViewServer used in the past.
Closes #123

Okay looks good to me, ship it (y)


[WIKI] Fix command formatting, broken links OSB Activites

Description:
Review and fix command formatting and broken links in /docs/src/content/docs/osb/Activites
Changes:
Fix command formatting aerial-fishing.md
Fix formatting, broken links fountain-of-rune.md
Fix typo spacing guardians-of-the-rift.md
Fix command formatting hallowed-sepulchre.md
Fix command formatting mage-training-arena.md
Fix command formatting mahogany-homes.md
Add command, fix broken links motherlode-mine.md
Fix command formatting shooting-stars.md
Fix command formatting, broken link volcanic-mine.md
Fix broken link wintertodt.md
Other checks:

[ x] I have tested all my changes thoroughly.

At this point just rename the misc folder so it's spelt correctly. If you do, change the zalcano link in k.ts

Can do but will wait until I have some time to fix lots of links after making that change.  Admin can merge this PR if they want or wait i'll leave that decision to them.


ser: cleanup tests

Removes the macro to follow a similar strategy to de.rs.

Codecov Report

Merging #6 (2954757) into master (566c870) will increase coverage by 1.37%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
+ Coverage   72.69%   74.06%   +1.37%
==========================================
  Files           4        4
  Lines         509      536      +27
==========================================
+ Hits          370      397      +27
  Misses        139      139

Impacted Files
Coverage Δ

src/ser.rs
81.15% <100.00%> (+2.82%)
:arrow_up:

Continue to review full report at Codecov.

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


Feat: JsonRcp provider, #82

This is the start of a proposal for the implementation of the JsonRcp provider.
Use the same client for test in VM as well as mocknet or testnet
Fixes #82

This PR would be a game changer in terms of DX, what's the latest @friedger?

@lgalabru Discussion with @agraebe is going on to clarify what the final goal is for #82. It was mentioned that there should be support for Connect, however, I don't really understand. Maybe we need to change the API of the Provider interface. Until the API is settled on, I do not plan to push this further.

@lgalabru agree with your assessment :) would love @hstove's input, even if it's just a "let's move forward and review later"


silence -Wstring-plus-int warnings throughout source, remove -Wno-string-plus-int

I get that this warning is dumb so I understand if it gets rejected, I just figured it might be preferable to not have to disable it (especially if it happens to eventually catch an actual issue).

If the warning fixes are fine but the removal of probecc() isn't because there's potential for use in the future I can change that commit to just remove the -Wno-string-plus-int addition.

No, "string"+offset is valid C, the same way any other pointer+offset is well-defined C: it advances by offset*sizeof(*pointer) which in this case is 1 (LP64 says so and C99 section <IP_ADDRESS>). If you add two strings together you should get an error (because pointer + pointer has no meaning). If you add two ints together you don't have a pointer and it warns about the implicit type conversion when you pass it as an argument.
What "actual issue" is this even theoretically supposed to catch? How is taking the address of an array offset supposed to be clearer? I can see if (assignment) instead of if (comparison) being a common enough typo to require the extra parentheses, but I don't understand what's supposed to be wrong with this one?
I'm not auditing these for what else they might have done, please resubmit the other changes separately.

I wasn't saying it wasn't valid, just wasn't sure if silencing the warning in the code instead of disabling it would be objected to, but I completely understand why. In regards to a theoretical issue, when I was looking for the origin of -Wno-string-plus-int in toybox I saw that enh said "as for the usefulness of this particular warning, we have seen it catch real bugs in OEM code :-(", not that it seems likely or even possible that said "real bug" would appear here. The only other changes I made were switching to FLAG() macros, should I make a new PR with those changes, and if so, should I go through the codebase and switch to FLAG() macros wherever relevant and include all of that in one big "switch to FLAG() macros" PR?

when I was looking for the origin of -Wno-string-plus-int in toybox I saw that enh said "as for the usefulness of this particular warning, we have seen it catch real bugs in OEM code :-(", not that it seems likely or even possible that said "real bug" would appear here.

yeah, i think the only risk is if you're expecting a sudden influx of Java programmers trying to  make toybox changes. i'm pretty sure they'll fall at earlier hurdles than this :-)

I'm happy to write a toybox programming HOWTO with an entire "how to program in C" section at some point, but that's a post-1.0 activity. (Let's be honest, that's a world with Basic Income where I have time to DO that sort of thing activity.)


Effort does not load related data after insert

Description
When a row is added to a table, it is common to want to refresh the object to pick up auto-generated field values, related objects specified by foreign keys and so on. Effort currently does not support this scenario. Methods of the DbEntityEntry object that is retrieved from DbContext.Entry(object) fail with an InvalidOperationException against an Effort context.
Project
Database context creation:
var connection = EntityConnectionFactory
    .CreateTransient(ConfigurationManager.ConnectionStrings["myEntities"].ConnectionString);
return new MyEntities(connection);

Repository method:
    public async Task<CourseDeclaration> AddCourseDeclaration(int categoryId,
        long userId,
        long managerId,
        int roleId,
        CourseDeclarationType declarationType,
        string comment)
    {
        var declaration = new CourseDeclaration
        {
            CategoryId = categoryId,
            UserId = userId,
            ManagerId = managerId,
            RoleId = roleId,
            DeclarationTypeId = (byte)declarationType,
            Comment = comment,
            Date = DateTime.Now
        };
        DbContext.CourseDeclaration.Add(declaration);
        await DbContext.SaveChangesAsync();

        var entry = DbContext.Entry(declaration);
        entry.Reload();
        return entry.Entity;
    }

Unit Test
    [Fact]
    public async void AddCourseDeclaration()
    {
        var courseRepository = BuildCourseRepository();
        var categoryId = 4;
        const long userId = 0;
        const long managerId = 22;
        const int roleId = 3;

        var result = await courseRepository.AddCourseDeclaration(categoryId, userId, managerId, roleId, CourseDeclarationType.Approved, "a comment");

        Assert.NotNull(result);
    }

Exception
The following exception is thrown when entry.Reload() is called in the repository method, above.
System.InvalidOperationException
Member 'Reload' cannot be called for the entity of type 'CourseDeclaration' because the entity does not exist in the context. To add an entity to the context call the Add or Attach method of DbSet<CourseDeclaration>.
   at System.Data.Entity.Internal.InternalEntityEntry.ValidateNotDetachedAndInitializeRelatedEnd(String method)
   at System.Data.Entity.Internal.InternalEntityEntry.ValidateStateToGetValues(String method, EntityState invalidState)
   at System.Data.Entity.Internal.InternalEntityEntry.Reload()
   at Tessello.Services.Core.Course.CourseRepository.<AddCourseDeclaration>d__7.MoveNext() in C:\BITBUCKET\tessello-services\Tessello.Services.Core\Course\CourseRepository.cs:line 78
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at Tessello.Services.Tests.Course.CourseRepositoryTests.<AddCourseDeclaration>d__15.MoveNext() in C:\BITBUCKET\tessello-services\Tessello.Services.Tests\Course\CourseRepositoryTests.cs:line 234
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Xunit.Sdk.AsyncTestSyncContext.<>c__DisplayClass7_0.<Post>b__1(Object _)

Further technical details

EF version: 6.2.0
EF Effort version: 2.1.1
Database Provider: SqlServer

Thank you @dotcom9  for reporting,
We will look if that's possible for us or not to support the method Reload.
Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Hello @dotcom9 ,
We tried your code asynchronously and without async and everything seems to work on our side.
using System;
using System.Collections.Generic;
using System.Data.Common;
using System.Data.Entity;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace Effort.Lab.EF6
{
	public class Request_Reload
	{
		public static void Execute()
		{
			var connection = Effort.DbConnectionFactory.CreateTransient();

			using (var context = new EntityContext(connection))
			{
				var entitySimple = new EntitySimple {ColumnInt = 1};
				context.EntitySimples.Add(entitySimple);
				context.EntitySimples.Add(new EntitySimple { ColumnInt = 2 });
				context.EntitySimples.Add(new EntitySimple { ColumnInt = 3 });
				context.SaveChanges();

				var entry = context.Entry(entitySimple);
				entry.Reload();
			}
		}

		public class EntityContext : DbContext
		{
			public EntityContext(DbConnection connection) : base(connection, true)
			{
			}

			public DbSet<EntitySimple> EntitySimples { get; set; }		}

		public class EntitySimple
		{
			public int ID { get; set; }
			public int ColumnInt { get; set; }
		}
	}
}

Could you provide a full example as we did with the issue?
Best Regards,
Jonathan


fail to convert datatables data for nested properties

bug located at io.springlets.data.web.datatables.ConvertedDatatablesData#getChildValue
should be :
if (parentValue.containsKey(baseProperty)) {
Object value = parentValue.get(baseProperty);

I changed nested property resolution as the method didn't work properly in some circumstances


LoginSocialGoogle not work on mobile device

Work fine on Desktop device but on mobile device it's gives client_id error,
check this:

Network call:

have you tried to open site demo on mobile? do it works?


Look publish stuck at Extract Look on OP 3.15.1

In GitLab by @support-bot on Feb 23, 2023, 16:15

asset cow task surfacing on TEST_OP_PUB, publish look

the publish never finish

Softwares concerned
Maya, Fix

@kekefreedog I can't check on the ws213 can you try?

Solve with the last rocky linux install.

so to test we must have that last install ?

Yes I tried on the @kekefreedog  workstation.

I tested it on my workstation with the last packages add IT team and it didn"t work. We have to test it on an asset with texture

I'll come Monday to check with you.

it's due to a missing library in rocky linux, it freezes maketx. @fixstudio-it

The maketx -h command works on Rocky8, I will change the code for that.

@OlivierOsotimehin It should work on your computer.

Can you try the maketx command in a terminal and give me the log.

maketx
bash: maketx: command not found

Solve with the compile OIIO from @fixstudio-it which are stored in /home/linuxbrew/.linuxbrew/bin

it works in centos&rocky with the same OIIO. Thanks everyone !


Extend Color Cfgu Type to support RGB/A and HSL colors

Description

Currently the Color type supports Hex values only.
We wish to extend it or add 2 more types to support also rgb, rgba and hsl color representations like rgb(0 100 255), rgba(90% 90% 90% .3), hsl(150 30% 60%), etc.
Motivation

Extending Configu's type system capabilities.
Additional context

https://developer.mozilla.org/en-US/docs/Web/CSS/color_value
https://github.com/validatorjs/validator.js
https://configu.com/docs/cfgu/
https://github.com/configu/configu/blob/main/types/Cfgu.ts

@rannn505 Hey, this seems to be already implemented

I will take a look here

I will take a look here

That's great the last 3 files linked in additional context are the files you need to make changes to. In case you haven't joined the discord channel here is the link https://discord.gg/K7rrskUB. You may post any queries you have over there.

thanks @Sadaf-A

thanks @Sadaf-A

https://github.com/configu/configu/pull/203 you can also refer to this PR

@Sadaf-A As this issue is still idle, I would like to take this up now and open a PR asap. Thanks.

@Sadaf-A As this issue is still idle, I would like to take this up now and open a PR asap. Thanks.

Hey, It's glad to see that you want to work on this issue. Thank you for your interest and consideration.
https://github.com/configu/configu/issues/138#issuecomment-1676367159
However, As mentioned here it seems like this issue is already resolved. I think you can still add RGBA and open a PR as HSL and RGB are already implemented. @RonConfigu please provide your inputs.
Furthermore, I would urge you to take a look at the newer issues and see if you can help out. You can also join the configu discord server for any queries that may arise
Thank You


⚠ Bofrost.de has degraded performance

In 67337e3, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1359 ms

Resolved: Bofrost.de performance has improved in 225c62b.


Add Eslint

This adds eslint with the airbnb base settings.
There are some notable exceptions:

Forcing await in async functions
No semicolons
No trailing comma

closes #8

Why no dangling comma? This is something I have always liked as it produces. Proper diffs when you have lits of arrays.

Why no dangling comma? This is something I have always liked as it produces. Proper diffs when you have lits of arrays.
But this is something we can take off line. This shouldnt be blocking

100% personal preference, if you open an Mr to re-add it, I would just merge it :)


Include BP in |request| for Frustration/Return

Context: https://www.smogon.com/forums/threads/return-frustration-base-power-tooltip.3650730/

Also, not a request for this specific PR (since apparently that's ambiguous whenever I say these things), but I'm going to register a plan to migrate |request| data to use names instead of IDs, to make it easier for server-side-only mods to have custom moves/etc that show up correctly.

I'm going to register a plan to migrate |request| data to use names instead of IDs, to make it easier for server-side-only mods to have custom moves/etc that show up correctly.

The TODOs in the client already indicated as much. How does it make things easier mods exactly? They'd still need to edit the data files, which then gets built into the client on the server, no?

The TODOs in the client already indicated as much. How does it make things easier mods exactly? They'd still need to edit the data files, which then gets built into the client on the server, no?

I'm thinking about things like Staff Bros, which don't modify any client data files.
Although no one's reporting any issues with Staff Bros so far so either no one cares that the custom moves are sent as IDs, or we already have some other system to make them appear correctly?

or we already have some other system to make them appear correctly?

When we build in the client it copies the data files from the server already, so when we do Dex.getMove on the client we get the move name from the ID.
I think the current model where we pass around IDs as much as possible and only Dex.getMove on display is preferable - it doesn't really matter (though ID is going to be marginally less bandwidth), but switching just breaks a lot of things/causes a lot of migration work. One argument I can see in favor of name > ID is that downstream developers then dont need to munge or clone PS data files to get display versions of things (its a lot easily to implement toID in another language than it is to build up the data display tables required), but I'd prefer solving that rethinking PS's data to be easily ingestable by other languages (something I have a forthcoming design for).
tl;dr: not strongly opposed to the idea of using names instead of IDs, but think its probably less impactful than would justify the work involved

When we build in the client it copies the data files from the server already, so when we do Dex.getMove on the client we get the move name from the ID.

It copies the data files from data/; it shouldn't touch mods except to build teambuilder tables, which explicitly don't include names.


Document /whoami

Document this in the api section and mention that you get a fresh token as a result
 Front conversations

Closed with #494


Flakey Test TestScheduleIBCUpgrade_Succeeds

Summary of Bug
I noticed the TestScheduleIBCUpgrade_Succeeds has become flakey.  example failure
I have seen the following error several times
=== RUN   TestClientTestSuite/TestScheduleIBCUpgrade_Succeeds/execute_proposal_for_MsgIBCSoftwareUpgrade
=== NAME  TestClientTestSuite/TestScheduleIBCUpgrade_Succeeds
    tx.go:63:
        	Error Trace:	/home/runner/work/ibc-go/ibc-go/e2e/testsuite/tx.go:63
        	            				/home/runner/work/ibc-go/ibc-go/e2e/testsuite/tx.go:161
        	            				/home/runner/work/ibc-go/ibc-go/e2e/tests/core/02-client/client_test.go:115
        	Error:      	Received unexpected error:
        	            	rpc error: code = NotFound desc = rpc error: code = NotFound desc = account cosmos1qsklq72jkfypur82f407nhhj886548jttcg399 not found: key not found

A re-run tends to solve the issue.

Expected Behaviour
Test passes consistently.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Same thing for TestClient_Update_Misbehaviour:
Error Trace:	/home/runner/work/ibc-go/ibc-go/e2e/testsuite/tx.go:63
[216](https://github.com/cosmos/ibc-go/actions/runs/6273050262/job/17035783233?pr=4658#step:5:217)
        	            				/home/runner/work/ibc-go/ibc-go/e2e/tests/core/02-client/client_test.go:312
[217](https://github.com/cosmos/ibc-go/actions/runs/6273050262/job/17035783233?pr=4658#step:5:218)
        	Error:      	Received unexpected error:
[218](https://github.com/cosmos/ibc-go/actions/runs/6273050262/job/17035783233?pr=4658#step:5:219)
        	            	rpc error: code = NotFound desc = rpc error: code = NotFound desc = account cosmos1ag9sfaa0m70m7pzdlw38cvmy37kpdu5axrn8x4 not found: key not found
[219](https://github.com/cosmos/ibc-go/actions/runs/6273050262/job/17035783233?pr=4658#step:5:220)
        	Test:       	TestClientTestSuite/TestClient_Update_Misbehaviour

A number of these are flaky. It's an error I remember seeing when adding hermes. Making hermes snooze for a bit longer is what usually did the trick.

Perhaps we could add a WaitUntil(addressFound) type functionality here instead of sleeping longer? Or add to the list of retry messages.

Actually bumped into Luca from Informal that works on hermes and mentioned this to him. Going to coordinate a bit with him on slack to see if this is something that can be fixed on hermes.

I've also actually seen this issue in the V8 upgrade test also with the same error (I think also just with hermes, but not 100% sure on that, something to keep an eye out for)

I think this has been resolved by #6689 if we see this error again, I think we can either open this issue again or create a new one with more recent examples.


Enable more tests on travis

Currently we're disabling the tests at src/server/pfs/fuse and src/server/pps/persist/server since they won't pass on Travis.

This has been fixed by the changes in issue #270. The fuse tests (all server tests) are now running succeeded on travis
Closing since this is now redundant


each-in sets string | number for the key's type

import Component from '@glint/environment-ember-loose/glimmer-component';

export default class Foo extends Component {
  public declare readonly c?: { readonly [key: string]: number };
}

{{#each-in this.c as |a|}}
  {{sum a 2}}
{{/each-in}}

And a generic sum helper:
import Helper from '@glint/environment-ember-loose/ember-component/helper';

export interface ISum {
  readonly PositionalArgs: number[];
  readonly Return: number;
}

export default class Sum extends Helper<ISum> {
  public compute(numbers: ISum['PositionalArgs']): ISum['Return'] {
    return numbers.reduce((a, b) => a + b, 0);
  }
}

declare module '@glint/environment-ember-loose/registry' {
  export default interface Registry {
    sum: typeof Sum;
  }
}

Leads to:
app/components/foo.hbs:2:9 - error TS2345: Argument of type 'string | number' is not assignable to parameter of type 'number'.
  Type 'string' is not assignable to type 'number'.

2   {{sum a 2}}
          ~

Ignore the specific error - the problem is that a is typed as string | number instead of what I've said it can be - key: string.

It looks like this is intentional behavior of keyof in TypeScript itself: https://github.com/microsoft/TypeScript/issues/29249
I want to read a bit more about their justification there, but I think for {{#each-in}} it probably makes sense for us to paper over that behavior and do something like keyof T & string.


WIP Geometric delay

Ignore the merge conflict - this is a result of removing the recent changes to polling delay in master.
WIP First cut at geometric polling...
The better you guess the better response you get, and if you are close
that's pretty good too.

Pull Request Test Coverage Report for Build 296

28 of 28 (100.0%)  changed or added relevant lines in 3 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage increased (+3.9%) to 85.961%

Files with Coverage Reduction
New Missed Lines
%

lib/util.js
1
83.33%

lib/poller.js
3
88.14%

Totals

Change from base Build 290:
3.9%

Covered Lines:
244

Relevant Lines:
281

💛  - Coveralls


Settings Window will not open

Microsoft PowerToys version
0.33.1
Running as admin

[ ] Yes

Area(s) with issue?
Settings
Steps to reproduce
Right click on PowerToys in the system tray and select "Settings".
PowerToysReport_2021-04-28-20-32-27.zip
✔ Expected Behavior
The Settings window will open.
❌ Actual Behavior
The settings window does not open.
Closing PowerToys and restarting it does not change the behaviour. Restarting the machine has no effect.
Other Software
No response

Version 0.35.0 is doing the same. Needs to kill the process to get it working again.

Seems related with #11718

We'll add more logs for this scenario

also, there's a stacktrace: for settings UI:
ERROR: 21:50:23.8113921
    [Method]: LoadProfile [Class]: KeyboardManagerViewModel
    Exception encountered when loading Keyboard Manager profile
Object reference not set to an instance of an object.
Inner exception:

Stack trace:
   at Microsoft.PowerToys.Settings.UI.Views.KeyboardManagerPage.CombineRemappings(List`1 remapKeysList, UInt32 leftKey, UInt32 rightKey, UInt32 combinedKey)
   at Microsoft.PowerToys.Settings.UI.Views.KeyboardManagerPage.FilterRemapKeysList(List`1 remapKeysList)
   at Microsoft.PowerToys.Settings.UI.Library.ViewModels.KeyboardManagerViewModel.LoadProfile()

@yuyoyuppe have logs been updated since your last comment? Should we request another Report Bug log?

After looking into the code, it looks we already do the logging there, and also now handle this case correctly.
@pgcrooks, @marison could you please try reproducing the issue with 0.43?


Add default ModelComponentFactory implementations for common model support classes

Default factories for repositories, record processors, readers, etc should be implemented to allow for full model support if no manual implementation of support components exists.  This will allow for easier data warehouse implementation and lower the bar to going from model to fully implementation.

With the move to Spring Data REST and a tighter coupling of models to their components, this is likely not necessary.


Add a demote everyone error message

Added demote everyone error... I have no idea what I'm doing please check my work.

Nice :D


A Virus Named TOM [PCSB00955]

Game summary

Game name: A Virus Named TOM
Game serial: PCSB00955

Vita3K summary

Version: 0.01
Build number: 328
Commit hash: https://github.com/vita3k/vita3k/commit/6525c1c

Test environment summary

Tested by: waterflame
OS: Windows 7
CPU: Intel i7-2600
GPU: AMD Radeon HD 6700 Series
RAM: 8 GB

Issues
Game starts up and starts to play sound as shader shits the bed. For a split second, you can see the Achievement window popup saying that it's been init. Sound is very glitchy/scratchy.
Screenshots

Log
vita3k.log
vita3k_withShaders.log
Recommended labels

Audio Bug
Softlock Bug
Graphics Bug

Vita3K summary

Version: 0.01
Build number: 404 not found
Commit hash: https://github.com/vita3k/vita3k/commit/d4cd595

#Issues

It seems sceKernelCreateLwCond was implemented(I recall talk of it in #devchat) so one umpimp line down! (Line 15 of build 347's log)

All offsets seem to have changed again, but I assume that's to be expected

Have we moved to a lower version of GL? GL Version for 347 is 4.5.13399 but 404 is 4.1.13399... then again there was also a switch between Compatibility Profile and Core Profile...

No longer is there a spam of Vertex shader failed to compile errors... but but they were released with Uniform parameter not found / unimplemented cals ... so it's gotten further. Seems to get to sceAppUtilInit, create the savedata, start loading maps. Very nice.

vita3k_404-d4cd595.log

Vita3K summary

Version: v0.1
Build number: 523-04b60ab
Commit hash:  https://github.com/Vita3K/Vita3K/pull/292

Issues
Still blasting music and reading a file into infinity. But! It no longer crashes! Though it wasn't noted here, it when from softlock to crash(newest master build 487-e5d80e9) to softlocking again.
Log
AVirusNamedTom-523.log

Vita3K summary

Version: v0.1
Build number: 571-70599d3
Commit hash:  https://github.com/Vita3K/Vita3K/commit/70599d3

Blasting music like a mother and constant try file reading.
AVirusNamedTom.log

Vita3K summary

Version: v0.1
Build number: 696
Commit hash: https://github.com/vita3k/vita3k/commit/b915fa1

Issue
Crash on load, DEADBEEF.
Log
AVirusNamedTom.log

Vita3K summary

Version: v0.1
Build number: 696
Commit hash: https://github.com/vita3k/vita3k/commit/4279f967

Issue
We now have a black screen instead of pink... Also, no more deadbeef crash.
Log
AVirusNamedTom.log

Vita3K summary

Version: v0.1
Build number: 984
Commit hash: fb0798e of pull/456 (I don't want to append every issue to the PR)

A Virus Named TOM - [PCSB00955].log


Introduce a concept of message handler

An abstract object type is required, which will handle log messages (save them to memory, to a table, ...);
A concept of log level has to be introduced (ERROR, WARNING, INFO, DEBUG, ...); The level must be a number basically. ERROR, WARNING etc. are actually constants for some predefined level values.
Initial idea is to have a log level per handler,  that can be increased/decreased when necessary. Only messages which have equal or greater level will be passed further to the handlers. For example, a global handler for the whole system may store the level in the "context", which is a global variable. This will allow to change the level immediately for all current users;
It must be possible to register multiple message handlers;
Create methods for logging a message in the LOG$ package.
5.1. Create a generic method, which accepts both message and the level;
5.2. Create shortcut methods LOG$.ERROR, LOG$.WARNING, LOG$INFO and LOG$DEBUG with the predefined error levels;

Implemented.


Add radio button reveal

Extend _selection-button with conditionally revealing content
We want to use the pattern defined in GovUK pattern library for
conditionally revealing content
(http://govuk-elements.herokuapp.com/form-elements/#form-toggle-content).
This allows the options for radio selection buttons to have an extra
attribute reveal which defines the text box to reveal.
Laurence has approved the validation styling for the revealed component.

Hmm, is this validation correct? I feel like we don't have the double line on followup questions 🤔

I've run it passed @laurenceberry and he's happy. The grey line is there permanently to show that it's a revealed thing. Follow up questions don't have the grey, do they?

This is how validation errors look for the G-Cloud questions with a reveal - the grey turns red, you don't get a new bar:

Have update the styling. Need to run it by @laurenceberry but it now looks like the below screenshots. It also adds a hint as we do elsewhere in case of no JS (included in screenshots).
No errors:

No errors, no JavaScript (field is never hidden):

Errors, without and without JavaScript (the same):

Nice,  that looks a lot cleaner I think.
The content is slightly weird for the non-js hint text but it's probably livable.

It's the same content we use for the other hidden inputs we have. I'll get @laurenceberry to take a look and if he's happy i'll get it in.

Yeah I know its the same input - but it refers to an 'earlier' question, whereas this one reveals a new question sort of inside the parent. So earlier feels a bit weirder in this case. :shrug: like I said its probably fine. Just noting it.

It’s good if consistent with gcloud.
Your first version was a good solution as it didn’t rely on colour to
highlight the error, however there will be a validation banner that links
to the field directly so it is clear for all users either way.
I don’t think you need to highlight the parent. I’ll have a look at what
other services do on Monday and let you know.
On Fri, 23 Feb 2018 at 13:46, Samuel Williams<EMAIL_ADDRESS>wrote:

Yeah I know its the same input - but it refers to an 'earlier' question,
whereas this one reveals a new question sort of inside the parent. So
earlier feels a bit weirder in this case. :shrug: like I said its probably
fine. Just noting it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alphagov/digitalmarketplace-frontend-toolkit/pull/409#issuecomment-368012889,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADn0EAbJL85l2B56ePtqa07KmqwDgfe6ks5tXsEogaJpZM4SOBgV
.

Looks good y'all.
The default non-js hint is more confusing than not having it at all in my opinion, could we drop it from this pattern?

It's gone, going to merge.


Are Task requests rare?

(I don't know this is the right place to ask this question. Please redirect to discourse if needed.)
The doc comment on Http.task says:

Just like request, but it creates a Task. This makes it possible to pair your HTTP request with Time.now if you need timestamps for some reason. This should be quite rare.

However, I don't get why using tasks should be rare. (Or, does "This" point the use of Time.now?) So, some notes about the thoughts behind it would help.
Context
We have many API requests in the codebase at work. Like this.
module Api.Request exposing (..)

getFoo : String -> Task Http.Error Foo
postFoo : Foo -> Task Http.Error Foo
...

The reason of having Task based API is that we often have to combine requests.

Task.map2 or Task3 to combine multiple requests to initiate each page

Ideally, this should be done in parallel.

Task.andThen to chain requests

For example, getFoo fooId |> Task.andThen (\foo -> getBar foo.barId)

All the functions return Task by default so we can avoid calling toTask so often (which happened when I used Request based API in 1.0.0).
I guess the answer of my question is something like:

HTTP requests should not be sent very frequently (chatty API is an anti pattern)
UI should show each data respectively (object-based UI design)
Server should give optimized API for each page

They might be true, but our development cannot always do things right. Seeing labels like "should not" or "rare" is a kind of fear when we are deeply depending on the feature, because it sometimes means "this feature will be removed soon".
Edit:
Another context: I'm writing an explanation of Task in my book and it says "Task is useful to chain HTTP requests" with some examples.

Okay, I asked for feedback in discourse. Very interested to see how it goes!
https://discourse.elm-lang.org/t/using-task-to-send-http-requests/2696


feat: define logic to autogenerate guardian password

Addresses #393
This logic generates a 16-character password that guardians can copy and backup and present during password check.
Question: At what point should password check happen?
Updated user flow:

Autogenerate password flow.webm

We should also add an acknowledgement checkbox that the user has copied down the secure password

We should also add an acknowledgement checkbox that the user has copied down the secure password

Updated the PR description with screenshot and short video of UI with most recent changes.
The idea is to not allow a guardian to proceed with federation configuration until they acknowledge that the password has been copied and securely backed up.

My observation is that the password is the same in a session for all guardians (leader, follower, and solo) during the Federation details phase of setup.
Would it be necessary to refresh the password after a certain amount of time (e.g. 10 / 20 seconds)?

My observation is that the password is the same in a session for all guardians (leader, follower, and solo) during the Federation details phase of setup.

What do you mean here? the password is supposed to be the same on the same fedimint guardian instance regardless of what role they choose in the setup ceremony?

My observation is that the password is the same in a session for all guardians (leader, follower, and solo) during the Federation details phase of setup.

What do you mean here? the password is supposed to be the same on the same fedimint guardian instance regardless of what role they choose in the setup ceremony?

Makes sense. It's just a concern I had, thought I should mention it.


Events: listener without context can be added twice

var m = new L.Marker();
var x = function() {console.log("event fired")};

/* register without context */
m.on("test", x);
m.on("test", x);

/* fires twice */
m.fire("test");

/* register with context */
context = {}
m.on("test2", x, context);
m.on("test2", x, context);

/* fires once*/
m.fire("test2");

https://playground-leaflet.rhcloud.com/wum/1/edit?html,js,output

OK, this is clearly inconsistent.
I'm thinking on adopting the behaviour of DOM events for this particular bit: if an event handler is attached twice, it fires once and can be removed with just one call: http://playground-leaflet.rhcloud.com/wuf/edit?js,console,output
Is this acceptable, or is there any case where we should keep track of how many repeated event handlers are there?

Addendum: If you supply the second listener with two different contexts it also fires twice (once for each context).
I would say a listener with the same name and function that fires twice is a bug. I would even go as far as to throw an error when it's registered a second time to make people fix their code.

Any news about this? I think the event-handling code is so important that issues should get settled fast.

Addendum: If you supply the second listener with two different contexts it also fires twice (once for each context).

We had a small chat about this, and it seems it's a feature, not a bug ;)
Basically, this is what you are doing when you do something like:
this._map.on('click', this._onClick, layer1)
this._map.on('click', this._onClick, layer2)

So what needs to be unique is the combination of listener + context.

So what needs to be unique is the combination of listener + context.

Listener as in function or as the combination of event and function?
So should
this._map.on('click', this._something, layer1)
this._map.on('move', this._something, layer1)

be allowed?

I definitely think so, and I would be surprised if we didn't already have code that does exactly that.

I think the most common scenario that illustrates why we need to allow same listener function with different contexts, is classes like this:
L.Layer.extend({
    onAdd: function(map) {
        map.on('moveend', this._onMoveEnd, this);
    },

    _onMoveEnd: function(e) { ... }
});

The _onMoveEnd is the same function for all instances, but the context (this) will differ. We definitely must allow this; otherwise just the first instance will get the event, which would be surprising to say the least.

I don't know what the best behaviour should be, but in any case I think it's viable to phase out or change event-related stuff for the following reasons:

Leaflet is pre 1.0. Better change behaviour now than after 1.0.
We could emit warnings in 1.0 like reactJS does and remove behaviour in 1.1.
AFAIK there's no formal specification of the event-system and even in 1.0 beta included (buggy?) behaviour, that code might still rely on: #4474 #4475 #4478.

In that light, I urge maintainers to take another look at my PR:  https://github.com/Leaflet/Leaflet/pull/2755
It's not necessary to merge it, but for stability's and simplicity's sake it would be a good idea to unite the two code-paths of manipulating listeners with/without context into one.
That means getting rid of that if construct: https://github.com/Leaflet/Leaflet/blob/master/src/core/Events.js#L97

This is fixed now, is it?


Refactor to class components

Reusing React functional components seem to break exhibition. Refactor them to class components

We chose to host every artwork on it's own subdomain rather than integrating them in the exhibition project (e.g. as submodule)


N-QUEEN PROBLEM USING BACKTRACKING

Add in RecursionBacktracking repo of java

Hi , I would like to work on this.