bug: Hubble crashes right after libp2p is started

[update: I think an old kernel, 5.10.160 Linux kernel, could be the culprit]
What is the bug?
Hubble crashes right after log saying Starting libp2p. Repeated 3 times locally.
How can it be reproduced? (optional)
Machine: Ubuntu 22.04, arm64, Linux Kernel 5.10.160
Start command:
podman run --name hubble -v /home/johns/data/hubble/.hub:/home/node/app/apps/hubble/.hub -v /home/johns/data/hubble/.rocks:/home/node/app/apps/hubble/.rocks -p 2282:2282 -p 2283:2283 docker.io/farcasterxyz/hubble:latest node --no-warnings build/cli.js start --ip <IP_ADDRESS> --gossip-port 2282 --rpc-port 2283 --eth-mainnet-rpc-url https://mainnet.infura.io/v3/<key> --l2-rpc-url https://optimism-mainnet.alchemy.io/v3/<key> --network 1 -b /dns/nemes.farcaster.xyz/tcp/2282 --hub-operator-fid 710
Logs:
{"level":30,"time":1695088646697,"pid":1,"hostname":"abe3c7e4f374","msg":"Started watching contract events at address 0x00000000fcaf86937e41ba038b4fa40baa4b780a"}
{"level":30,"time":1695088646697,"pid":1,"hostname":"abe3c7e4f374","msg":"Starting fname events provider from 20011 using signer: 0xBc5274eFc266311015793d89E9B591fa46294741"}
{"level":30,"time":1695088646697,"pid":1,"hostname":"abe3c7e4f374","msg":"Fetched 30 fname events upto 20041"}
{"level":30,"time":1695088646697,"pid":1,"hostname":"abe3c7e4f374","rootHash":"864848e6d83721bd05a4553ebd7c47bc23763a45","msg":"Merkle Trie loaded from DB"}
{"level":30,"time":1695088646697,"pid":1,"hostname":"abe3c7e4f374","rootHash":"864848e6d83721bd05a4553ebd7c47bc23763a45","msg":"Sync engine initialized"}
{"level":40,"time":1695088646697,"pid":1,"hostname":"abe3c7e4f374","deniedPeerIds":[],"function":"createNode","msg":"No PEER-ID RESTRICTIONS ENABLED. This node will accept connections from any peer"}
{"level":30,"time":1695088646697,"pid":1,"hostname":"abe3c7e4f374","identity":"12D3KooWFyGD6bHkLuWQdBKr6XYH4WoL8yd8FYkBeVLojdtL4GAo","addresses":["/ip4/<IP_ADDRESS>/tcp/2282/p2p/12D3KooWFyGD6bHkLuWQdBKr6XYH4WoL8yd8FYkBeVLojdtL4GAo"],"msg":"Starting libp2p"}
{"level":50,"time":1695088646697,"pid":1,"hostname":"abe3c7e4f374","reason":"Uncaught exception","msg":"shutting down hub"}
{"level":40,"time":1695088646697,"pid":1,"hostname":"abe3c7e4f374","msg":"signal 'uncaughtException' received"}
{"level":30,"time":1695088646698,"pid":1,"hostname":"abe3c7e4f374","component":"Hub","msg":"Stopping Hubble..."}
{"level":30,"time":1695088646698,"pid":1,"hostname":"abe3c7e4f374","component":"rpcServer","component":"gRPC Server","msg":"Force shutdown succeeded"}
{"level":30,"time":1695088646712,"pid":1,"hostname":"abe3c7e4f374","component":"GossipNode","identity":"12D3KooWFyGD6bHkLuWQdBKr6XYH4WoL8yd8FYkBeVLojdtL4GAo","msg":"Stopped libp2p..."}

Startup Checks:
 Hubble Startup Checks
------------------------
✅ | Farcaster: 2023.8.23 Hubble: 1.5.4
✅ | Directory /home/node/app/apps/hubble/.rocks is writable
✅ | Directory /home/node/app/apps/hubble/.hub is writable
✅ | Found PeerId 12D3KooWNTj7f2EXvbc643R6V6rF8fcWL2iZXuwxgVsVJDbq3Jyf
⚠  | Hubble Monitoring is disabled
   | Please see help at https://www.thehubble.xyz/intro/install.html#monitoring-hubble
✅ | Network is MAINNET(1)
✅ | Bootstrapping from 1 peers
✅ | Hub Operator FID is 710(johns)
✅ | Connected to L1 Ethereum node
✅ | Connected to L2 Optimism node

I suspect this is actually some kind of networking error. We've seen this before when libp2p runs into issues traversing NAT. Looks like our logging didn't actually capture the error.
Fixed the logging in https://github.com/farcasterxyz/hub-monorepo/pull/1458. Please update to 1.6.1 when it's out and try again. That should tell us more about what's happening here.

Ok, just tested on my NanoPC-T6 again with Hubble v1.6.2 and no error. I'll post here if I see anything again

Seems fixed! no errors


get-p-status: don't include max temp in response if inverter didn't return it

see https://github.com/gch1p/inverter-tools/issues/1#issuecomment-981192681

fixed


Fix search for pinned notebooks

Most of the fix to allow search in pinned notebooks is located in the NotebookExplorerViewlet.
I had to modified how we stored pinned notebooks in workspace config, because the parent node in the search result would be the directory name of the notebook instead of the book root.
Image before modifying the workspace config.

Image after modifying the workspace config.

This PR fixes #12459
Note: One of the functions that I removed from the BookPinManager had a duplicate in the utils class. So I decided to move everything related to modifying the workspace to utils, since it's used in other files, too.

Pull Request Test Coverage Report for Build 284795433

37 of 56 (66.07%)  changed or added relevant lines in 7 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.007%) to 39.721%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/contrib/notebook/browser/notebookExplorer/notebookSearch.ts
0
1
0.0%

extensions/notebook/src/book/bookTreeView.ts
4
8
50.0%

src/sql/workbench/contrib/notebook/browser/notebookExplorer/notebookExplorerViewlet.ts
0
14
0.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
74.52%

Totals

Change from base Build 284782908:
-0.007%

Covered Lines:
19978

Relevant Lines:
45968

💛  - Coveralls


Low accuracy on Kinetics (2D resnet is better than 3D resnet)

I trained a 2D resnet101 on kinetics dataset, and it has much better accuracy than your 3D resent. I used https://github.com/gurkirt/2D-kinectics to train 2D resnet. Of course, pretrained on imagenet.
I got the top1 accuracy of 70.3 on kinetics-400.
Do you think it is because of pretraining?
Have you tried  2D networks with your training procedure?

In the Non-local Neural Network paper, the top 1 acc of 2D-ResNet50 baseline is 71.8.
@gurkirt @kenshohara
I believe the performance gap does not come from the ImageNet pretraining as @kenshohara  said.

@kenshohara @gurkirt @bityangke
Did anyone find the reason for the difference?


Minimap DataPoint Selection Information

Please describe your request and why you are requesting it.
The minimap icons always show the same color for the data points instead of having a distinction between the ones that have no selection and the one that is selected. It would be good to have to have an even clearer point of reference.
Describe the solution you'd like
Use the already established colors of blue and orange to set the color of data points on the minimap / big map to convey the information of selection through that view too.

Added in #46


healthcheck: add regular chain backend and disk space checks

This PR wraps our existing logger in one which will request graceful shutdown when the Critical logging level is used. This wrapper is used to add two health checks to lnd which will shutdown if we do not succeed after sufficient backoff/retry:

Check that we have a connection to our chain backend
Check that we have some space available on disk

The regularity and strictness (ie, number of retries we allow) of this check are made configurable, so that users can turn them off/ make them less strict, or turn them off entirely if they cause issues (particularly thinking about neutrino flakes).
Windows is windows, so we need to have a special windows space check (_windows, TIL!).
Partially addresses #4306 (does not cover failed disk writes, so issue not entirely fixed)
Fixes #1557, #3109

Rebased and updated in separate pushes, diff of changes here. Changes since last review:

Update changes to signal package so that we can only call intercept once
Add timeout to health checks (because btcd GetBestBlock blocks when btcd shuts down)
Add disk check for solaris behind build tag, as done for windows

Is it possible to make integration tests out of these with the various backends and kill the backends to see that lnd does shutdown? I don't think it would be possible to really test the disk healthcheck.

That's possible, but the test set up would need to be slightly distinct as all the other tests assumes that the target chain backend will be around the entire time. Agreed about the disk check....one way to test it would be using a container w/ a very small volume (would basically be unable to start).

Haven't been able to test this out

I have run some local tests for disk check, and chain backend for btcd + bitcoind, so it depends on how much time we want to put into testing. I can wrangle together some bash scripts to put up and kill docker containers if needed.

btcd hang comment above

Right now btcd does hang, which is why the timeout was added for these checks. The unfortunate part is that the goroutine calling the best block endpoint will hang indefinitely, but there's not much that can be done unless we make a change to btcwallet. There is the option of using zmq/websockets but that comes with some copy-fu from btcwallet, and halves the number of lnd nodes bitcoind can serve (down to 2, which sucks for local dev).

Haven't been able to test this out

I have run some local tests for disk check, and chain backend for btcd + bitcoind, so it depends on how much time we want to put into testing. I can wrangle together some bash scripts to put up and kill docker containers if needed.

btcd hang comment above

Right now btcd does hang, which is why the timeout was added for these checks. The unfortunate part is that the goroutine calling the best block endpoint will hang indefinitely, but there's not much that can be done unless we make a change to btcwallet. There is the option of using zmq/websockets but that comes with some copy-fu from btcwallet, and halves the number of lnd nodes bitcoind can serve (down to 2, which sucks for local dev).

Would be super helpful to have some docker scripts, maybe could even be run as part of our tests?


A possible deadlock issue

It seems that lock rdev->caw_lock is not released before Line 1736. Should it be a bug?
https://github.com/open-iscsi/tcmu-runner/blob/b4d1656ecc5500b85416e1014669669434ddb291/tcmur_cmd_handler.c#L1731-L1736

No, please check the handle_caw_read_cbk/handle_caw_write_cbk, the lock will be released there later for aio.

@lxbsz Thank you for your reply!


Unexpected bahaviour with respect to "more than one" predicate

Take the following sCASP specification where g is saying roughly that  f holds for more than one value of its first argument in any context (value of the second argument of f):
f(a,c).
f(b,d).

equal(a,a).
equal(b,b).

g(A) :- f(A,_), f(B,_), not equal(A,B).

Unexpectedly sCASP happily finds a model for both g(a) and not g(a). Is this due to a misunderstanding on my side or is this considered a bug?
See https://swish.swi-prolog.org/p/strange.pl
Modifying the program to make f unary the behaviour is not observable.
f(a).
f(b).

equal(a,a).
equal(b,b).

g(A) :- f(A), f(B), not equal(A,B).

In the ciao version of sCASP the same behaviour can not be observed. So it sais that not g(a) is not provable in both versions.


[UI] Markers on the map moving in wrong position

Environment: macOS Big Sur 11.5.1; Chrome version 97.0.4692.71
Reproducible: always.
Preconditions

Follow link https://oos.dmytrominochkin.cloud/#/
Set "Київ" city
Click on Search button

Steps to reproduce

Click on the Map view.
Zoom out the map.
Take a look to the markers position.

Actual result
Kyiv Markers are located on the border with Belarus

Expected result
Markers are located in appropriate position
Labels to be added
"Bug", Priority ("pri: medium"), Severity ("severity: minor"), Type ("UI").

I think this bug is not relevant anymore and can be closed.

This bug is not relevant anymore.


[incubator/vault] support annotations for Vault service account

Signed-off-by: Brian Surber<EMAIL_ADDRESS>What this PR does / why we need it:
Add custom annotation support for the Vault pod's service account
Which issue this PR fixes
None created
Special notes for your reviewer:
Tested via helm template and my own installation
# Source: ha-vault/charts/vault/templates/serviceaccount.yaml
apiVersion: v1
kind: ServiceAccount
metadata:
  name: vault-vault
  labels:
    app: vault
    chart: "vault-0.18.15"
    release: "vault-vault"
    heritage: "Tiller"
  annotations:
    iam.gke.io/gcp-service-account: <redacted>

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

I don't see (can't grep) any trailing whitespaces in values.yaml, I'm not sure why that test failed


Fixed padding

Reference

This will be superseded by #14663 since we haven't heard back on our comment. Thanks for your contribution though!


martin: 0.9.1 → 0.10.0

Description of changes
Changelog
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

That's a very interesting tool. I didn't know about it. Thanks for packaging !

protoc binary needs to be patched or replaced with one from nixpkgs:
failed to spawn command `"/build/martin-0.10.0-vendor.tar.gz/protoc-bin-vendored-linux-aarch_64/bin/protoc"

@nixos/rust Could you advise a proper way to fix it?

Either patch martin to not use this crate, or symlink our binaries (from buildPackages) into the respective directories.

Either patch these binaries, patch martin to not use this crate, or symlink our binaries (from buildPackages) into the respective directories.

I've tried to patch the binary or symlink to nixpkgs protoc, but I've got:
martin> error: the listed checksum of `/build/martin-0.11.3-vendor.tar.gz/protoc-bin-vendored-linux-x86_64/bin/protoc` has changed:
martin> expected: 239cd7cc1888c8855bc2a7ed571d4af3cb9d255fde86b78c1fc1cdabf9248c14
martin> actual:   636f4f115e97302cda39133d3714414ea41cfd5c415d9da2c060e080fd3e8775
martin> directory sources are not intended to be edited, if modifications are required then it is recommended that `[patch]` is used with a forked copy of the source

cargoHash remained the same.

Is this PR still draft ?

Is this PR still a draft ?

protoc binary needs to be patched or replaced with one from nixpkgs:
failed to spawn command `"/build/martin-0.10.0-vendor.tar.gz/protoc-bin-vendored-linux-aarch_64/bin/protoc"

I still don't know how to do it.


Add LayerEnv::apply_to_empty

It's a common pattern to apply a layer env and pass the resulting env to another layer. like so:
ruby_layer.env.apply(Scope::Build, &Env::new())

However, they're mostly applied to an empty env which is a little bit clunky. As a QOL enhancement, add LayerEnv::apply_to_empty which removes the second argument:
ruby_layer.env.apply_to_empty(Scope::Build)

GUS-W-10423858

I think I'm fine with apply and apply_to_empty. I want to ensure users realise that a LaverEnv is just a delta, on its own it has no meaning, but an Env does. In my head, this operation reads like "apply delta to env" and "apply delta to empty env".

Yeah makes sense. Happy to stick with that naming then. Could you move the additional test usages over to it, per https://github.com/Malax/libcnb.rs/pull/267#pullrequestreview-850634197 ? :-)

@edmorley I will!

@edmorley added extra RustDoc. For LayerEnv::new(), in the doctest, the initial and resulting envs are compared to show that applying an empty delta does not change the input Env. We can't just remove the initial one and use apply_to_empty if we want to keep that explicit example.

Ah I missed that aspect in the new() rustdoc, ty.


last_testflight_build_number always assumes the build number is an int, truncates real value

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
$ fastlane --verbose stock_app_store_submit
[✔] 🚀
[15:41:26]: Get started using a Gemfile for fastlane https://docs.fastlane.tools/getting-started/ios/setup/#use-a-gemfile
Successfully loaded Appfile at path 'XXXXX'
- apple_id: 'XXXXX'
- team_name: 'XXXXX'
- app_identifier: 'XXXXX'
-------
DEBUG [2018-01-26 15:41:26.46]: Checking if there are any plugins that should be loaded...
INFO [2018-01-26 15:41:26.65]: ----------------------------------------
INFO [2018-01-26 15:41:26.65]: --- Step: Verifying fastlane version ---
INFO [2018-01-26 15:41:26.65]: ----------------------------------------
INFO [2018-01-26 15:41:26.65]: Your fastlane version 2.78.0 matches the minimum requirement of 2.73.0  ✅
INFO [2018-01-26 15:41:26.68]: ------------------------------
INFO [2018-01-26 15:41:26.68]: --- Step: default_platform ---
INFO [2018-01-26 15:41:26.68]: ------------------------------
INFO [2018-01-26 15:41:26.68]: Driving the lane 'ios stock_app_store_submit' 🚀
INFO [2018-01-26 15:41:26.68]: --------------------------------------
INFO [2018-01-26 15:41:26.68]: --- Step: mkdir -p ./fastlane_logs ---
INFO [2018-01-26 15:41:26.68]: --------------------------------------
INFO [2018-01-26 15:41:26.68]: $ mkdir -p ./fastlane_logs
INFO [2018-01-26 15:41:26.69]: ------------------------------------------------------
INFO [2018-01-26 15:41:26.69]: --- Step: Switch to ios get_marketing_version lane ---
INFO [2018-01-26 15:41:26.69]: ------------------------------------------------------
INFO [2018-01-26 15:41:26.69]: Cruising over to lane 'ios get_marketing_version' 🚖
INFO [2018-01-26 15:41:26.77]: gem 'Xcodeproj' is already installed
INFO [2018-01-26 15:41:26.77]: Cruising back to lane 'ios stock_app_store_submit' 🚘
INFO [2018-01-26 15:41:26.77]: We are about to submit to the App Store. This is serious. The release are are submitting for review is 1.1.28.
INFO [2018-01-26 15:41:26.81]: --------------------------------------------
INFO [2018-01-26 15:41:26.81]: --- Step: latest_testflight_build_number ---
INFO [2018-01-26 15:41:26.81]: --------------------------------------------
INFO [2018-01-26 15:41:26.87]: Login to iTunes Connect (XXXXX)
Loading session from 'XXXXX'
Looking for iTunes Connect Team with ID XXXXX
INFO [2018-01-26 15:41:28.88]: Login successful
INFO [2018-01-26 15:41:29.45]: Fetching the latest build number for version 1.1.28
INFO [2018-01-26 15:41:31.11]: Latest upload for version 1.1.28 is build: 180124
INFO [2018-01-26 15:41:31.11]: The build number we are using is 180124

Environment
✅ fastlane environment ✅
Stack

Key
Value

OS
10.12.6

Ruby
2.4.2

Bundler?
false

Git
git version 2.5.0

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.12.6 (16G29)

Ruby Lib Dir
/usr/local/Cellar/ruby/2.4.2_1/lib

OpenSSL Version
OpenSSL 1.0.2l  25 May 2017

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode-9.1.app/Contents/Developer/

Xcode Version
9.1

We are using build numbers of the form 99999.99999 based on dates, but builds like this in TestFlight return truncated int values for the version e.g. "99999".
This has shown up what appears to be a bug in the implementation of AppStoreBuildNumberAction. The code always coerces the version string to int in order to do the sorting.
Build versions are not exclusively ints, although some people use them like this. Amusingly if you look at the Apple docs for CFBundleVersion it actually expects three part versions...
The offending code is:
https://github.com/fastlane/fastlane/blob/1a79353a6d918eddca61274e97962a5f2770210e/fastlane/lib/fastlane/actions/app_store_build_number.rb#L43
I would like to suggest this is changed to not attempt to sort the values - presumably Apple supplies them sorted because you cannot reliably sort multi-part build numbers especially as CFBundleVersion permits alphanumerics.

@marcpalmer Got a PR out for a fix for ya ^


hotfix: validate resize disk when change config

这个 PR 实现什么功能/修复什么问题:
添加腾讯云严格磁盘参数校验
是否需要 backport 到之前的 release 分支:

release/2.11
release/2.10.0

/area region
/cc @swordqiu @yousong

Codecov Report

Merging #2001 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #2001      +/-   ##
=========================================
- Coverage    7.19%   7.18%   -0.02%
=========================================
  Files         445     445
  Lines       69285   69355      +70
=========================================
- Hits         4985    4982       -3
- Misses      63888   63960      +72
- Partials      412     413       +1

Flag
Coverage Δ

#aFlag
7.18% <0%> (-0.02%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/compute/models/guestdrivers.go
20% <ø> (ø)
:arrow_up:

pkg/compute/models/guests.go
0.68% <0%> (-0.01%)
:arrow_down:

pkg/compute/guestdrivers/base.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/guest_actions.go
0% <0%> (ø)
:arrow_up:

pkg/compute/guestdrivers/qcloud.go
3.75% <0%> (-1.56%)
:arrow_down:

pkg/util/rbacutils/rbac.go
73.37% <0%> (-0.93%)
:arrow_down:

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 ed2804d...5712dce. Read the comment docs.

/lgtm

/approve


az storage blob metadata update -> lease-id parameter removed in 2.38

This is autogenerated. Please review and update as needed.
Describe the bug
After acquiring a lease through "az storage blob lease acquire --proposed-lease-id " we would like to update the metadata on that blob. Untill 2.37 we could provide --lease-id to the command and then it should work.
However, in 2.38 this parameter is removed and if you remove it in the call then you get the response
There is currently a lease on the blob and no lease ID was specified in the request.
ErrorCode:LeaseIdMissing
Command Name
az storage blob metadata update
Errors:
unrecognized arguments: --lease-id 952eea37-83c3-4c34-a975-460eff59eae6

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

`az storage blob lease acquire --account-name {} --subscription {} --auth-mode {} --container-name {} --blob-name {} --lease-duration {} --proposed-lease-id {}
az storage blob metadata update --account-name {} --subscription {} --auth-mode {} --container-name {} --name {} --metadata {} {} --lease-id {} --output {} --verbose

Expected Behavior
The metadata of the blob is updated.
Environment Summary
Windows-10-10.0.22000-SP0
Python 3.10.5
Installer: MSI

azure-cli 2.38.0

Extensions:
azure-devops 0.25.0

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

Seeing the same.
Code that worked two days ago now produces error message:
ERROR: unrecognized arguments: --lease-id xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx
And the documentation page no longer lists the "--lease-id" parameter.
Previous page (courtesy of the Wayback Machine):

Any update on this regression issue?

@bebound Can you check if this is caused by track2 migration?


Fix rubocop offenses

Update rubocop, remove erroneous cops, ignore specific files
Fix rubocop offenses

Coverage remained the same at 99.846% when pulling 82ebb907e4a67bb11ad286d980369887a0d4d07f on bf-fix-rubocop-offenses into 4e7eab3871f5915d3846da9559b8db40b73c502c on master.

Coverage remained the same at 99.846% when pulling 7747cf4d5442732d10b515931b4e57b1f70ab492 on bf-fix-rubocop-offenses into 4e7eab3871f5915d3846da9559b8db40b73c502c on master.

Coverage remained the same at 99.846% when pulling 9ac75f0a874d83d11cd5df3b79e2fb442b283cea on bf-fix-rubocop-offenses into 4e7eab3871f5915d3846da9559b8db40b73c502c on master.

Coverage remained the same at 99.846% when pulling 8d4544d621b8068bf5eb02630c4fe4e74b17be74 on bf-fix-rubocop-offenses into 4e7eab3871f5915d3846da9559b8db40b73c502c on master.

Coverage remained the same at 99.846% when pulling 9cd248b65b603eaa58968cf288627205ccdf4be0 on bf-fix-rubocop-offenses into 4e7eab3871f5915d3846da9559b8db40b73c502c on master.

Coverage remained the same at 99.846% when pulling 9cd248b65b603eaa58968cf288627205ccdf4be0 on bf-fix-rubocop-offenses into 4e7eab3871f5915d3846da9559b8db40b73c502c on master.

Coverage remained the same at 99.846% when pulling 75da64d8d586986dadb6639d2d63fb9f14e98a0a on bf-fix-rubocop-offenses into 73251d77607864a5957eba482b1e0b59eff7675f on master.

Coverage remained the same at 99.846% when pulling dbd55bde00f33b6cef2fbb374f4421802bf0016e on bf-fix-rubocop-offenses into 7a3faeca13460972c11d8083f348197fe0f4d6e7 on master.


1.1.0 not on npm

Hey, the latest code isnt on npm which is rather confusing when you go and look at the github page expecting the latest release to be on main branch    :)

Hi @danjenkins thanks for the hint!
I provided an update for the 0.9.x branch and therefore the latest tag got messed up on npm. Should be fixed now :-)


Multiples arr instances

Possible to add support for multiples arr docker instances like hostname:radarr and hostname:radarr4k or sonarr and sonarr4k?
thanks

Traefik middleware can be used with multiple services at the same time. You can also use multiple middlewares for a service. With that, I am fairly certain what you are looking for already exists.


[Beta] fix useMemo API doc error.

Fix issue #5223

does this sentence occur anywhere else in the doc?

this sentence occur once in useMemo API doc,.

does this sentence occur anywhere else in the doc?

this sentence occur once in useMemo API doc,.

does this sentence occur anywhere else in the doc?

@gaearon yes, please refer this


Unexpected behavior for advanced dependOn usage

Bug report

What is the current behavior?
According to the documentation and examples, entry points provided to another entry's dependOn property must:

[...] be loaded before this entry point is loaded

Whilst this works correctly with a single value, it doesn't give the desired outcome (see below) when providing it with more advanced values (e.g. a "chained" dependOn structure).
I noticed that in the Advanced dependOn example, the depended-on entries are included after the entry depending on them.
If the current behavior is a bug, please provide the steps to reproduce.

Given the following config (a stripped down version of our project's config, where we've noticed this after recently migrating to v5):
const path = require("path");
const HtmlWebpackPlugin = require("html-webpack-plugin");

// Entries
const commonEntry = "common";
const globalEntry = "global";
const entryNames = [globalEntry, "a", "b", "c"];

const entries = { [commonEntry]: ["react", "react-dom"] };

entryNames.forEach(entry => {
  entries[entry] = {
    import: path.join(__dirname, "src/entries", entry, "entry.js"),
    dependOn: entry === globalEntry ? [commonEntry] : [commonEntry, globalEntry],
  };
});

// Generate markup partials
const entryHtmls = Object.keys(entries).map(entry => new HtmlWebpackPlugin({
  chunks: [entry],
  inject: false,
  template: path.join(__dirname, "webpack/templates/js.html"),
  filename: `${entry}/js.html`,
}));

module.exports = {
  entry: entries,
  output: {
    filename: "[name]/[name]-[contenthash]-bundle.js",
    path: path.resolve(__dirname, "dist"),
  },
  optimization: {
    splitChunks: { chunks: "all" },
    runtimeChunk: "single",
  },
  plugins: [commonHtml, ...entryHtmls],
};

I see that the assets are bundled in the following order:
assets by info 3.91 MiB [immutable]
  assets by chunk 3.87 MiB (id hint: vendors)
    asset common/common-cc008264e0fab30884bc-bundle.js 2.48 MiB [emitted] [immutable] (name: common) (id hint: vendors)
    asset vendors-node_modules_lodash_lodash_js/vendors-node_modules_lodash_lodash_js-73af31127d98264393e1-bundle.js 1.39 MiB [emitted] [immutable] (id hint: vendors)
  asset c/c-b09613e149a23b90d79b-bundle.js 9.46 KiB [emitted] [immutable] (name: c)
  asset b/b-0ef964a8fc4dbd8bcf95-bundle.js 9.4 KiB [emitted] [immutable] (name: b)
  asset a/a-93fd194d2175735d03fd-bundle.js 9.35 KiB [emitted] [immutable] (name: a)
  asset global/global-19fa78b68955244d2f44-bundle.js 7.68 KiB [emitted] [immutable] (name: global)
  asset runtime/runtime-d1973ae5538d57c1c468-bundle.js 7.49 KiB [emitted] [immutable] (name: runtime)

And the outputted templates via html-webpack-plugin look like:
<!-- common/js.html -->
<script src="../runtime/runtime-d1973ae5538d57c1c468-bundle.js"></script>
<script src="../common/common-cc008264e0fab30884bc-bundle.js"></script>

<!-- global/js.html -->
<script src="../vendors-node_modules_lodash_lodash_js/vendors-node_modules_lodash_lodash_js-73af31127d98264393e1-bundle.js"></script>
<script src="../global/global-19fa78b68955244d2f44-bundle.js"></script>

<!-- a/js.html -->
<script src="../vendors-node_modules_lodash_lodash_js/vendors-node_modules_lodash_lodash_js-73af31127d98264393e1-bundle.js"></script>
<script src="../a/a-93fd194d2175735d03fd-bundle.js"></script>

<!-- b/js.html -->
<script src="../vendors-node_modules_lodash_lodash_js/vendors-node_modules_lodash_lodash_js-73af31127d98264393e1-bundle.js"></script>
<script src="../b/b-0ef964a8fc4dbd8bcf95-bundle.js"></script>

<!-- c/js.html -->
<script src="../vendors-node_modules_lodash_lodash_js/vendors-node_modules_lodash_lodash_js-73af31127d98264393e1-bundle.js"></script>
<script src="../c/c-b09613e149a23b90d79b-bundle.js"></script>

If I adjust the order of entryNames to the following (placing our "second dependOn" last):
const entryNames = ["a", "b", "c", globalEntry];

The assets are now bundled in the following order:
assets by info 3.89 MiB [immutable]
  assets by chunk 3.87 MiB (id hint: vendors)
    asset common/common-cc008264e0fab30884bc-bundle.js 2.48 MiB [emitted] [immutable] (name: common) (id hint: vendors)
    asset vendors-node_modules_lodash_lodash_js/vendors-node_modules_lodash_lodash_js-73af31127d98264393e1-bundle.js 1.39 MiB [emitted] [immutable] (id hint: vendors)
  asset global/global-19fa78b68955244d2f44-bundle.js 7.68 KiB [emitted] [immutable] (name: global)
  asset runtime/runtime-d1973ae5538d57c1c468-bundle.js 7.49 KiB [emitted] [immutable] (name: runtime)
  asset c/c-73abcc6182827bdc0368-bundle.js 4.5 KiB [emitted] [immutable] (name: c)
  asset b/b-974777b752aa37254653-bundle.js 4.45 KiB [emitted] [immutable] (name: b)
  asset a/a-ddf20413318aed626714-bundle.js 4.4 KiB [emitted] [immutable] (name: a)

Resulting in the expected output for the template files:
<!-- common/js.html -->
<script src="../runtime/runtime-d1973ae5538d57c1c468-bundle.js"></script>
<script src="../common/common-cc008264e0fab30884bc-bundle.js"></script>

<!-- global/js.html -->
<script src="../vendors-node_modules_lodash_lodash_js/vendors-node_modules_lodash_lodash_js-73af31127d98264393e1-bundle.js"></script>
<script src="../global/global-19fa78b68955244d2f44-bundle.js"></script>

<!-- a/js.html -->
<script src="../a/a-ddf20413318aed626714-bundle.js"></script>

<!-- b/js.html -->
<script src="../b/b-974777b752aa37254653-bundle.js"></script>

<!-- c/js.html -->
<script src="../c/c-73abcc6182827bdc0368-bundle.js"></script>

What is the expected behavior?
If this is intended functionality, I think it should be stated in the docs. However I have a feeling it is intentional, since it doesn't seem logical to have to place entries in this reverse order.
If not intentional, when providing entries in the first order (globalEntry at start of array) I would expect to have the output as from the second order (globalEntry at end of array).

Other relevant information:
webpack version:  5.35.1
Node.js version: 12.18.0
Operating System: macOS 10.15.7
Additional tools: -

Some extra things

After writing this up, I'm now thinking that the "must be loaded before this entry point is loaded" text is actually referring to loading the bundles within your app/page - and not the order in the entry object. However there still seems to be some odd behaviour here from what I can see, so I shall proceed with opening the issue.
I have a repro ready to go, I thought that maybe the above info alone would suffice, but if a slightly fuller example is better I'll create a repository.
Whether the outcome of this be an adjustment to the documentation or something at code level, I'm happy and eager to participate - if someone would be kind enough to point me in the right direction 🙂.

Thanks!

I have a repro ready to go, I thought that maybe the above info alone would suffice, but if a slightly fuller example is better I'll create a repository.

Will be great to look at this, thanks

Hey @alexander-akait, here's the repository as mentioned: https://github.com/breadadams/webpack-dependon-example
I've included the dist directory, so you can make the change mentioned in the README and compare the output.

Looks like bug

Thanks @alexander-akait, from skimming over some of what I believe is the related source code for handling entries/chunks and more so the dependsOn property I had a feeling it may be a bug, although I must admit I don't fully understand all the inner workings.
I'll try and get my hands dirty in the coming days, however if someone can pitch in here it would be highly appreciated.
Thanks again 🙂

Whilst running some trials this with the examples/tests in the repo, I noticed I couldn't reproduce the bug whatsoever. That lead me to the changelog where I saw that @sokra resolved this in 027f9bd9299237198004ac7c85286969a65fe692 - less than 24h before I opened this issue 😅.
Therefore, this is solved as of 5.36.2, thanks a lot! ❤


Broken scoring issues

Hi @lsv .
Issues like the one caused in https://github.com/lsv/fifa-worldcup-2018/commit/1601d1d95f87819618edaf0d9c475acf35c46c49 and fixed in https://github.com/lsv/fifa-worldcup-2018/pull/83 are causing huge side effects for our application that picks up the results and applies irreversible logic like sending automated pushes to users with these scores.
I suggest:

These updates be made through pull requests.
Adding myself and maybe a few others as collaborators to help validate each PR?

Other than that, thanks for this awesome project.
Shai

Sorry that my live score API provider didnt had the results updated correctly

@lsv I did not understand what you mean. What do you think about my suggestions?

@shaialon sure I can do it - but then there will be another delay
And I do not think that someone wanna check 30 commits pr day - within a 4 hour timeframe

True. Maybe just for the "finished" games then ?

Then it would be 1-4 per day.

Thats true - I could do that - though I would need more contributors because I will not be able to merge them.
But sure I can easily create PR's when a game will have the end PR

Great. So can you please add me (to have merge permissions)?

done
Im not sure I can get it fixed for todays first matches, but hopefully for the "exiting" England-Belgium match

Any ideas on how to automatic make a PR from a new branch ?

We could create a bash script that diffs lsv/data.json with broeneatsdinner/data/world-cup-2018.json whenever a match's "finished == true", to create a pull request.
The broeneatsdinner repository doesn't rely on an external API provider, but rather "watches" the games in real time from various sources (google, yahoo, fifa) and knows the time elapsed in each game. That prevents a "finished == true" from turning up before a game is actually finished.

Doesnt matter - you can still not create a PR from command line, though I dont know how anyway


Automatically cancel prior jobs on CI

Currently, if two jobs start on CI around the same time, it's nondeterministic which one will "win". In fact, it's entirely possible for the earlier job to cause the later one to get canceled, which has happened before. Instead, as soon as a deploy starts it should cancel any previous ones. This can probably be handled straightforwardly with a kill command added to deploy-phase2.py.

I believe this will be solved by #42.

CircleCI provides this out of the box.

CircleCI provides this out of the box.


Travis CI: Python 2.7 and 3.7 with flake8 tests

Run flake8 . --select=E901,E999,F821,F822,F823 under both Python 2.7 and Python 3.7 to find syntax errors and undefined names.  This should help us catch Python incompatibilities in pull requests before they are reviewed.
E901,E999,F821,F822,F823 are the "showstopper" flake8 issues that can halt the runtime with a SyntaxError, NameError, etc.  Most other flake8 issues are merely "style violations" -- useful for readability but they do not effect runtime safety.

F821: undefined name name
F822: undefined name name in __all__
F823: local variable name referenced before assignment
E901: SyntaxError or IndentationError
E999: SyntaxError -- failed to compile a file into an Abstract Syntax Tree

Description

Motivation and Context

Testing

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[x] I have read the README document
[x] I have added tests to cover my changes
[x] All new and existing tests passed

License

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@cclauss Thanks for the PR. I was thinking maybe this can go in tox or as a linter test script instead (could potentially add pylint etc. along with this in future), so that developers can run it locally before submission and wait for travis output. What do you think?

@cclauss thanks for the update. I would suggest including both py36 and py37 environments for testing. This would also require the package metadata information to include py37. Do you mind adding them as well?

Is this what you were looking for or did I misunderstand?

Hey @cclauss, Thanks for adding back py36 env in tox and travis. I meant adding the classifier information for Py3.7 in all packages' setup.py files, thus making them discoverable as Py3.7 packages on PyPI search.


Adds JELLYFIN_USER_ID to docker compose file

For users like me, who doesn't read the full README :-)
Makes it clear that this needs to be set.

Thanks for the pull request - I must have missed that!


Encryption support

It would be very helpful if this included support for sending encrypted emails.

Everything is going through the Outlook REST API which uses HTTPS, but beyond that I'm not sure what can be improved. If you have any ideas, definitely share and I'll try to implement - pull requests also accepted :)


CustomKeyboardView don't resize if empty

Hi,
when I have custom keyboard rendered:

and then I call KeyboardUtils.dismiss(), height doesn't update to 0px and hold same size without rendered custom keyboard:

When I was debugging then this line was "null" (without component):
https://github.com/wix/react-native-keyboard-input/blob/31564eb82e7683b4a0ea89ae3289eba3eb8f63d5/src/CustomKeyboardView.js#L119
Could you please help me, how I can fix it. Maybe it will be only some styles to hold height of component inside.
Thank you.

+1

same problem

+1

Guys, did you resolve the problem?

I just found a solution.
To fix it remove android:windowSoftInputMode="adjustResize" in AndroidManifest.xml file.

Hi guys, the same problem here any updates?


Table prefix on foreign key

I am using table prefixes in my project and everything goes just fine until I decide to add a foreign key to table that already exist.
This code has been failed with SQLSTATE[HY000]: General error: 1215 Cannot add foreign key constraint:
$table = $this->table('documents');
$table->addForeignKey('version_id', 'documents_versions', 'id',
        ['delete'=> 'CASCADE', 'update'=> 'CASCADE'])
        ->update();`

Command SHOW ENGINE INNODB STATUS; show a problem:
FOREIGN KEY (`version_id`) REFERENCES `documents_versions` (`id`) ON DELETE CASCADE ON UPDATE CASCADE:
Cannot resolve table name close to:
 (`id`) ON DELETE CASCADE ON UPDATE CASCADE

So adding a prefix works fine for me (but obviously its a bug):
$table->addForeignKey('version_id', 'ema_documents_versions', 'id',
        ['delete'=> 'CASCADE', 'update'=> 'CASCADE'])
        ->update();

I am using version 0.8.1 installed with composer.

Are you sure about this? Mysql can only complaint when creating references to tables that do not exist, not the opposite

Yes, I am sure. I can reproduce this behavior at any time.
I did not fully understand what do you mean "Mysql can only complaint when creating references to tables that do not exist, not the opposite"? How is it possible to crate reference on table that does not exsisit?

Sorry, I have clicked a wrong bottom. So, the problem not with the reference itself, problem with the prefix.

Please, check out my description again. I am using prefix ema_. While I just add new tables and columns everything worked fine for me.
BUT, once I have deicide to add  new foreign key for TABLE THAT ALREADY EXIST:

This doesn't work: $table->addForeignKey('version_id', 'documents_versions', ...
But this one: $table->addForeignKey('version_id', 'ema_documents_versions', ...  worked fine.

Booth cases has same configuration, but in the second one I should manually add prefix.
This is not a big problem to me, but its a bug. That is why I am writing here.
PS. Sorry for my bad English.

oh, you are using the automatic table prefix feature? Is that the case?

Yes.

Thanks for the clarification

You are welcome

Are you able to make a PR to fix things? That would be the quickest solution to this issue.

Yes, I can do that.

I think this is working correctly in the new 0.10.0 version


add export in createapp.env

Which issue this PR addresses:

N/A
What this PR does / why we need it:

The doc says we need to run source clusterapp.env but it doesn't work because clusterapp.env doesn't have export with each env, and the env variables aren't passed to hack/cluster create.
Test plan for issue:

Test it locally
Is there any documentation that needs to be updated for this PR?

N/A
Change the code according to the doc.
How do you know this will function as expected in production?

/azp run ci


Add column "production_volume" in new datastructure of pd.Dataframe

Done with 2b6895b9c9f7da83d0bfc669220a323d8b27c15a.
But not sure whether we should have such column in the scenario sections as well.


Add data range text - second try

Purpose of this pull request
This pull request (PR) adds an explanation of "Date Range" to the "Filters" section of the Insight Trainer article:
https://help.sumologic.com/docs/cse/rules/insight-trainer/#filters
NOTE: THIS IS THE SECOND ATTEMPT TO MAKE THE UPDATE BECAUSE THE FIRST PULL REQUEST (#2326) FAILED BECAUSE GITHUB WENT DOWN (see production.yml workflow).
Issue number:
GitHub issue #2325
Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

@jpipkin1 , it says no files were changed in this PR so nothing was merged

Thanks for letting me know, @kimsauce. The PR is in progress in the deploy-to-production workflow now. Let's see what happens there:
https://github.com/SumoLogic/sumologic-documentation/actions/workflows/production.yml


fix(deps<EMAIL_ADDRESS>

CVE-2024-42461 / CVE-2024-42460 / CVE-2024-42459

any idea why didn't we get this from renovate?

renovate only looks at our direct dependencies as declared in package.json. In this case it's a transitive dependency only visible in the lockfile. dependabot would catch it, though.


feat(longpolling env variable): added SBL_LONGPOLLING_DELAY_SECONDS

NOTE
@shindigira This still needs related PRs in sbl-project and regtech-eks-bootstrap
closes #708
Changes

feat: add SBL_LONGPOLLING_DELAY_SECONDS as an environmental variable to control the longpoll delay externally

How to Test

Set the SBL_LONGPOLLING_DELAY_SECONDS either 'backoff', a number in seconds or random text (defaults to standard).

Looks good! There's a little wonkiness right now with the deploys to AWS, so I'll approve this in maybe a day or two if that's ok?


the longer timer runing, the higher of overhead

at start, the execution rate is 1400 sample per second with 88 threads,after 24 hours it's only 90 sample per second while with high overhead.

Hi! I'm merging this issue with https://github.com/googleprojectzero/fuzzilli/issues/174
But also note that the execution rate naturally decreases significantly over time as the samples become more complex, e.g. because they trigger JIT compilation.


As I send cookie WebSocketClient?

As I send cookie WebSocketClient ?, I'm having problems with nodejs to connect from Traccar.
var WebSocketClient = require('websocket').client;
var client = new WebSocketClient();
client.on('request', function (request) {
console.log("Websocket connection request recieved");
});
var session =  {
Cookie:"JSESSIONID=for6iklifwqjrlr7lf0qicup"
};
client.connect('ws://localhost:8082/api/socket', session);

Sorry, the above code does not make much sense. Check the API doc please.


App crashed when adding a plants for the first time

log:
A/libc: Fatal signal 11 (SIGSEGV), code 2, fault addr 0xa0543ff8 in tid 4015 (hwuiTask2)

Does it happen when you scroll?
Please explain the behavior.

Can you elaborate more on the crash?  What device did you use, what OS version, etc?

Closing issue - please reply with more information if the crash is still happening.  Thanks!

Closing issue - please reply with more information if the crash is still happening.  Thanks!

Closing issue - please reply with more information if the crash is still happening.  Thanks!


merge training and testing arguments

merge training and testing arguments so that testing can be followed by training

it doesn't matter to the experiments, so it's not a problem any more


Webm background (lua)

What is your question?
Is webm background possible with lua? (0.5.2h

I dont think video backgrounds are compatible yet in psych

Also WEBM dosent exist in psych engine

Also WEBM dosent exist in psych engine

yeah, there is only mp4 support


Version number is incompatible with frameworks in Xcode.

Hi - I'm integrating this project into a swift project using Cocoapods frameworks - and all is well when building etc.
However when uploading to the store, I have only just came across this issue (since Xcode 7.1).
Apparently they don't want you to use 4 decimal points in a version number, and the fact that this project does means I need to manually edit the info.plist before submitting.

Running into the same issue. But, the podspec is not officially maintained, no?

Same here

A sane workaround until someone updates this repo, is to add a local MTLManagedObjectAdapter.podspec.json in your project.
Then in your podfile, point to it like:
pod 'MTLManagedObjectAdapter', :podspec => 'MTLManagedObjectAdapter.podspec.json'
{
  "name": "MTLManagedObjectAdapter",
  "version": "1.0.1",
  "license": "MIT",
  "summary": "Model framework for Cocoa and Cocoa Touch.",
  "homepage": "https://github.com/Mantle/Mantle",
  "authors": {
    "GitHub"<EMAIL_ADDRESS>  },
  "source": {
    "git": "https://github.com/Overcoat/MTLManagedObjectAdapter.git",
    "tag": "<IP_ADDRESS>"
  },
  "requires_arc": true,
  "platforms": {
    "ios": "5.0",
    "osx": "10.7",
    "watchos": "2.0"
  },
  "source_files": "MTLManagedObjectAdapter",
  "dependencies": {
    "Mantle": [
      "~> 2.0"
    ]
  },
  "frameworks": [
    "Foundation",
    "CoreData"
  ],
  "prepare_command": "PREFIX=\"mtl_moa_\"\n# Add prefix to header imports\next_header_prefix_src() {\n  SOURCE_FILE=$1\n  EXT_HEADER_NAME=$2\n  sed -i.bak \"s/\"${EXT_HEADER_NAME}\"/\"${PREFIX}${EXT_HEADER_NAME}\"/g\" ${SOURCE_FILE} && rm ${SOURCE_FILE}.bak\n}\next_header_prefix_src MTLManagedObjectAdapter/MTLManagedObjectAdapter.m EXTRuntimeExtensions.h\next_header_prefix_src MTLManagedObjectAdapter/MTLManagedObjectAdapter.m EXTScope.h\next_header_prefix_src MTLManagedObjectAdapter/extobjc/EXTRuntimeExtensions.m EXTRuntimeExtensions.h\next_header_prefix_src MTLManagedObjectAdapter/extobjc/EXTScope.m EXTScope.h\n# Change header name\next_header_prefix_mv() {\n  SOURCE_FILE=$1\n  FILE_NAME=`basename ${SOURCE_FILE}`\n  DIR_NAME=`dirname ${SOURCE_FILE}`\n  mv ${SOURCE_FILE} `dirname ${SOURCE_FILE}`/${PREFIX}`basename ${SOURCE_FILE}`\n}\nexport -f ext_header_prefix_mv\nexport PREFIX=${PREFIX}\nfind MTLManagedObjectAdapter/extobjc -name \"*.h\" -exec bash -c 'ext_header_prefix_mv \"$0\"' {} \\;\nunset ext_header_prefix_mv\nunset PREFIX",
  "subspecs": [
    {
      "name": "extobjc",
      "source_files": "MTLManagedObjectAdapter/extobjc",
      "private_header_files": "MTLManagedObjectAdapter/extobjc/*.h"
    }
  ]
}

The issue has been fixed and can be closed.


Doesn't always detect correct prefix

Here's my tmux.conf
I override the prefix as C-s if it detects I'm running in an SSH session, but prefix-highlight still thinks it's C-a
Moving the if -F "$SSH_CONNECTION" "source-file $HOME/dotfiles/tmux.remote.conf" line to above all the plugin settings doesn't seem to fix it either..

It does seem to work correctly on more recent builds of tmux (though i can't seem to pinpoint exactly which commit)

Hi @DanielFGray. Is this still your tmux config?
Could you point me where are you setting the prefix to C-s?
Also, what is the result of this command (inside tmux): tmux show-option -gqv prefix?

That's my current config, yes. On L81 it sources this config if I'm running in an ssh session, and tmux-prefix will usually say my prefix is still C-a.


Is there any way to play ads in the player?

Not a built in way. You will need to implement it yourself.


Update simann_example.swift

Checklist

[x] I've looked at the contribution guidelines.
[x] This pull request is complete and ready for review.

Description

Example file was not functional as found.
Corrected recursive definition where shuffle() was invoking self.shuffle() instead of shuffling the point array self.tour, which left the example in an infinite loop of shuffle() invocation.

Thanks for the fix!


matrix-synapse: 1.33.2 -> 1.34.0

Motivation for this change
Backport #123355
ChangeLog: https://github.com/matrix-org/synapse/releases/tag/v1.34.0
(cherry picked from commit 0dae433cdcfdd7cd6afe323fda3e3b582ab2da43)
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test matrix-synapse

Tests are lgtm.


question

is there way to get it working on replit?

Currently testing this right now and it looks promising however if you run it for long periods of time you will get temporarily banned from bloxflip (on replit)

oh ok


Implement DDS support

Pictus is definitely the best image viewer there is - it's lightweight, lightning fast and doesn't contain any unnecessary bloat. However, one feature that it's missing is DDS support. Maybe that's something could be implemented eventually.

Yey! for DDS support ;) Maybe also with embedded Mip-Maps support :D

After unpacking a load of texture files I'd agree this would be nice.


Backport fix from django-wkhtmltopdf that includes request in context

Thanks for the port to puppeteer!
This PR backports a fix from the upstream project that ensures request is a passed to template.render, which is required to make use of Django's built-in request context processor
See django-wkhtmltopdf #145 for additional context.

@jacobwegner hei! Thanks for this pull request. I'll review it today and get this resolved and released.


Micro sometimes doesn't properly open

Description of the problem or steps to reproduce
Open micro. An empty terminal window is shown in which keybinds (e.g. to close the editor) don't work. Nothing unusual is logged to log.txt with "-debug" enabled. Resetting the config with "micro -clean" doesn't resolve this issue. The issue occurs regardless of whether a file or an empty buffer is opened, and only occurs sometimes, seemingly randomly.

log.txt
Specifications
Commit hash: 68d88b57
OS: Arch Linux
Terminal: Konsole

Instructions/questions for issues creator
Micro sometimes doesn't properly open. It is interesting. Does "sometimes" is when user from what micro was open can't open the bindings.json file or when it doesn't exist?
Please tell me what you see in ~/.config/micro. Do you see there bindings.json file? If yes, what is there?
Did you try to open new terminal and kill micro (not terminal with micro)? Please try it, if my guess is correct you will see there something like: Error reading bindings.json file: .......
Possible reason
It reminded me one my test with config file bindings.json loading. I can guess it is related to initialization of bindings. I guess it is because in some cases micro can exit before initializing default bindings (Ctrl+q for exit is one of them`
Point of exit:
https://github.com/zyedidia/micro/blob/d8e9d61a955edf6f58c1533012d8a25728ba65be/internal/action/bindings.go#L39-L44
Point of default bindings initialization:
https://github.com/zyedidia/micro/blob/d8e9d61a955edf6f58c1533012d8a25728ba65be/internal/action/bindings.go#L51-L58
P.S: If I am right and the problem is that InitBindings function exits before initializing default bindings, PR #3068 resolves the problem (and reduce code duplication)

In case this does actually turn out to be the issue, here is my bindings.json. Since I reset the config and don't remember ever changing it it should be the default.
bindings.json

As I wrote in the start of the comment seemingly the issue not in the initing of bindings. It looks like problem is before initing bindings. And if problem in initing bindings it would print it before opening micro. But if you for example create folder binding.json instead of file you will see something similar about bindings but window won't be empty

But if you for example create folder binding.json instead of file you will see something similar about bindings but window won't be empty. In your case it is empty so it is another problem

Yup. The next time the issue occurs I will try killing the process from somewhere else and see what it spits out.

this has been happening to me. Right now it's happening every time I try to use micro.
There's no stdout or stderr from micro if I micro &> microlog and kill it.
I did a strace -tt -f -o microlog.txt micro. See the log attached: microlog.txt
At clock time~1:14:00 I run it, and I kill it at ~1:15:00
Seems there is something happening every 8 seconds (1:14:00, 1:14:08, 1:14:16, 1:14:24, ... etc.)
1163222 01:14:00.999757 epoll_pwait(3,  <unfinished ...>
1163230 01:14:00.999768 futex(0xc000580548, FUTEX_WAIT_PRIVATE, 0, NULL <unfinished ...>
1163218 01:14:01.009780 <... nanosleep resumed>NULL) = 0
1163218 01:14:01.009861 futex(0x560b138398a0, FUTEX_WAIT_PRIVATE, 0, {tv_sec=7, tv_nsec=925544986}) = -1 ETIMEDOUT (Connection timed out)
1163218 01:14:08.935616 nanosleep({tv_sec=0, tv_nsec=10000000},  <unfinished ...>
1163222 01:14:08.942715 <... epoll_pwait resumed>[], 128, 7935, NULL, 0) = 0
1163222 01:14:08.942754 epoll_pwait(3, [], 128, 0, NULL, 0) = 0
1163222 01:14:08.942838 futex(0xc000580548, FUTEX_WAKE_PRIVATE, 1) = 1
1163230 01:14:08.942918 <... futex resumed>) = 0
1163222 01:14:08.942932 epoll_pwait(3, [], 128, 0, NULL, 0) = 0
1163222 01:14:08.942961 epoll_pwait(3,  <unfinished ...>
1163230 01:14:08.942974 futex(0xc000580548, FUTEX_WAIT_PRIVATE, 0, NULL <unfinished ...>
1163218 01:14:08.945790 <... nanosleep resumed>NULL) = 0
1163218 01:14:08.945882 futex(0x560b138398a0, FUTEX_WAIT_PRIVATE, 0, {tv_sec=7, tv_nsec=997034582}) = -1 ETIMEDOUT (Connection timed out)
1163218 01:14:16.943114 nanosleep({tv_sec=0, tv_nsec=10000000},  <unfinished ...>
1163222 01:14:16.946662 <... epoll_pwait resumed>[], 128, 7999, NULL, 0) = 0
1163222 01:14:16.946699 epoll_pwait(3, [], 128, 0, NULL, 0) = 0
1163222 01:14:16.946741 futex(0xc000580548, FUTEX_WAKE_PRIVATE, 1) = 1
1163222 01:14:16.946781 epoll_pwait(3,  <unfinished ...>
1163230 01:14:16.946796 <... futex resumed>) = 0
1163222 01:14:16.946809 <... epoll_pwait resumed>[], 128, 0, NULL, 0) = 0
1163222 01:14:16.946827 epoll_pwait(3,  <unfinished ...>
1163230 01:14:16.946856 futex(0xc000580548, FUTEX_WAIT_PRIVATE, 0, NULL <unfinished ...>
1163218 01:14:16.953211 <... nanosleep resumed>NULL) = 0
1163218 01:14:16.953250 futex(0x560b138398a0, FUTEX_WAIT_PRIVATE, 0, {tv_sec=7, tv_nsec=993531664}) = -1 ETIMEDOUT (Connection timed out)
1163218 01:14:24.946978 nanosleep({tv_sec=0, tv_nsec=10000000},  <unfinished ...>
1163222 01:14:24.953327 <... epoll_pwait resumed>[], 128, 7999, NULL, 0) = 0
1163222 01:14:24.953362 epoll_pwait(3, [], 128, 0, NULL, 0) = 0
1163222 01:14:24.953398 futex(0xc000580548, FUTEX_WAKE_PRIVATE, 1) = 1
1163222 01:14:24.953438 epoll_pwait(3,  <unfinished ...>
1163230 01:14:24.953449 <... futex resumed>) = 0
1163222 01:14:24.953466 <... epoll_pwait resumed>[], 128, 0, NULL, 0) = 0
1163222 01:14:24.953487 epoll_pwait(3,  <unfinished ...>
1163230 01:14:24.953505 futex(0xc000580548, FUTEX_WAIT_PRIVATE, 0, NULL <unfinished ...>
1163218 01:14:24.957072 <... nanosleep resumed>NULL) = 0
1163218 01:14:24.957114 futex(0x560b138398a0, FUTEX_WAIT_PRIVATE, 0, {tv_sec=7, tv_nsec=996326729}) = -1 ETIMEDOUT (Connection timed out)
1163218 01:14:32.953600 nanosleep({tv_sec=0, tv_nsec=10000000},  <unfinished ...>
1163222 01:14:32.960026 <... epoll_pwait resumed>[], 128, 7999, NULL, 0) = 0
1163222 01:14:32.960076 epoll_pwait(3, [], 128, 0, NULL, 0) = 0
1163222 01:14:32.960119 futex(0xc000580548, FUTEX_WAKE_PRIVATE, 1) = 1
1163230 01:14:32.960150 <... futex resumed>) = 0
1163222 01:14:32.960164 epoll_pwait(3,  <unfinished ...>
1163230 01:14:32.960175 epoll_pwait(3,  <unfinished ...>
1163222 01:14:32.960187 <... epoll_pwait resumed>[], 128, 0, NULL, 0) = 0
1163230 01:14:32.960199 <... epoll_pwait resumed>[], 128, 0, NULL, 0) = 0
1163222 01:14:32.960212 epoll_pwait(3,  <unfinished ...>
1163230 01:14:32.960223 futex(0xc000580548, FUTEX_WAIT_PRIVATE, 0, NULL <unfinished ...>
1163218 01:14:32.963728 <... nanosleep resumed>NULL) = 0
1163218 01:14:32.963796 futex(0x560b138398a0, FUTEX_WAIT_PRIVATE, 0, {tv_sec=7, tv_nsec=996388873}) = -1 ETIMEDOUT (Connection timed out)
1163218 01:14:40.960418 nanosleep({tv_sec=0, tv_nsec=10000000},  <unfinished ...>
1163222 01:14:40.966695 <... epoll_pwait resumed>[], 128, 7999, NULL, 0) = 0
1163222 01:14:40.966768 epoll_pwait(3, [], 128, 0, NULL, 0) = 0
1163222 01:14:40.966929 futex(0xc000580548, FUTEX_WAKE_PRIVATE, 1) = 1
1163230 01:14:40.967011 <... futex resumed>) = 0
1163222 01:14:40.967043 epoll_pwait(3,  <unfinished ...>
1163230 01:14:40.967068 epoll_pwait(3, [], 128, 0, NULL, 0) = 0

Lots of futex timeout stuff, maybe a deadlock?
I also tried moving my whole micro config (cd ~/.config; mv micro micro2) but still hangs

this has been happening to me. Right now it's happening every time I try to use micro.

Just to be sure. Your screen is also absolutely empty like OP have? You tried to also use -debug option? micro -debug
What OS, terminal emulator, micros commit do you use? Micro's commit can be accessed via micro -version
About strace and bunch of epoll_pwait3; futex; nanosleep. I launched strace with the same parameters and got the similar log file where I saw similar bunch of epoll_pwait3; futex; nanosleep but my micro started and worked normally so it looks like it is a part of normal execution process. About he reason of many futexes: I don't know but have a guess. IIUC micro draws in separate thread and use mutex for synchronization:
https://github.com/zyedidia/micro/blob/d8e9d61a955edf6f58c1533012d8a25728ba65be/internal/screen/screen.go#L14-L26
While I inspected your strace log I found there common paths micro paths ~/.config/micro; ~/.config/micro/help; ~/.config/micro/syntax; ~/.config/micro/init.lua; ~/.config/micro/plug; ~/.config/micro/settings.json ..... Were the attempts to open them successful? ¯\_(ツ)_/¯.
The most interesting part for me was finding there ~/.config/micro/bindings.json path that means that micro at least reached the function of initialization bindings. Maybe it initialized default bindings maybe it returned before it but it reached that function and it looks like it went further

Arch Linux, also was using Konsole. I just tried with kitty and it's the same. Completely blank terminal screen.
micro -debug &> debugout is an empty file - no stdout or stderr
Note the issue is the same if I remove ~/.config/micro

@digitalsignalperson as you have the problem at each attempt to open micro I want to ask you. Did you try another terminals? You and OP have the same OS and terminal so maybe it is a fault of them? ¯\_(ツ)_/¯

¯\_(ツ)_/¯ I found in micros history thing that looks similarly https://github.com/zyedidia/micro/issues/2662 In the issue described some techniques that can probably solve your issue

@digitalsignalperson as you have the problem at each attempt to open micro I want to ask you. Did you try another terminals? You and OP have the same OS and terminal so maybe it is a fault of them? ¯_(ツ)_/¯

@dustdfg I also tried kitty terminal and it's the same

¯_(ツ)_/¯ I found in micros history thing that looks similarly #2662 In the issue described some techniques that can probably solve your issue

Ah, k I think I narrowed things down.
Completely uninstalling wl-clipboard solve it yes. But it only fails to open with wl-clipboard if the XWayland clipboard is empty.

wayland: open a text editor, copy text and then close the editor. This clears the clipboard. micro still works

on kde we can confirm clipboard status with qdbus org.kde.KWin /KWin org.kde.KWin.showDebugConsole

xwayland: copy some text from an X program. do xsel -bc; xsel -pc; xsel -sc to clear all clipboards. micro doesn't work.

this also leaves a wl-paste --no-newline process, which does not get destroyed if you killall micro

@digitalsignalperson as you have the problem at each attempt to open micro I want to ask you. Did you try another terminals? You and OP have the same OS and terminal so maybe it is a fault of them? ¯_(ツ)_/¯

@dustdfg I also tried kitty terminal and it's the same

¯_(ツ)_/¯ I found in micros history thing that looks similarly #2662 In the issue described some techniques that can probably solve your issue

Ah, k I think I narrowed things down.
Completely uninstalling wl-clipboard solve it yes. But it only fails to open with wl-clipboard if the XWayland clipboard is empty.

wayland: open a text editor, copy text and then close the editor. This clears the clipboard. micro still works

on kde we can confirm clipboard status with qdbus org.kde.KWin /KWin org.kde.KWin.showDebugConsole

xwayland: copy some text from an X program. do xsel -bc; xsel -pc; xsel -sc to clear all clipboards. micro doesn't work.

this also leaves a wl-paste --no-newline process, which does not get destroyed if you killall micro

I can no longer reproduce the issue. Can it be closed?

@edwloef yes seems to be fixed! Can probably close #2662 too


mvp code for create method issue with Uuid v4

Hi @treydempsey ,
we are having a weird issue with create method adding an entry to an sql table with two Uuid.
I have prepared an mvp here:
MVP issue Uuid
In the readme there is a more detailed explanation.
Thanks for your support in advance!

Adding some comments about documentation...
If I try to assemble a small example based on this example it seems not working. Below here a code snippet:
use sqlx::FromRow;
use sqlx_crud::SqlxCrud;
use sqlx_crud::Crud;
use sqlx::SqlitePool;
use anyhow::Result;

#[derive(Debug, FromRow, SqlxCrud)]
pub struct User {
   pub user_id: i32,
   pub name: String,
}

#[tokio::main]
async fn main() -> Result<()> {
    let db_url = "sqlite://test.db";

    let pool = SqlitePool::connect(db_url).await?;

    let new_user = User { user_id: 1, name: "new_user".to_string() };
    new_user.create(&pool).await?;

    if let Some(user) = User::by_id(&pool, 1).await? {
        println!("User: {:?}", user);
    }

    Ok(())
}

In migrations folder I have this sql:
CREATE TABLE users (
    id INTEGER PRIMARY KEY NOT NULL,
    username TEXT NOT NULL
);

Basically your example in crate docs.
If I run with cargo run i get:
Error: error returned from database: (code: 1) no such column: users.user_id

Caused by:
    (code: 1) no such column: users.user_id

What did the magic for me is adding #[external_id] to the struct like that:
#[derive(Debug, FromRow, SqlxCrud)]
#[external_id]
pub struct User {
   pub user_id: i32,
   pub name: String,
}

I think some correction are needed anyway in the docs, but beside that is it ok that I use that attribute to make it work?
Thanks!


Add Getting Started section to README

README Update
Please add a getting started section to the README so folks know how to....

Added Getting Started section in README file.


Not rendering "Values" bar completely

Hi! Please have a look at the following output chart. It "Values" bar is being incompletely rendered. What could be the issue??

What file and version are you using?

I am using v3. I have taken sample code from example_v3.html...

Could you please show me the code? May be I can find the problem.

I am getting JSON response using AJAX and then using following code:
var chart = new PersonalitySunburstChart({'selector': '#sunburstChart', 'version': 'v3'});
chart.show(profile, 'profile_photo.jpg');

And following are the dependencies:
<script type="text/javascript" src="https://cdnjs.cloudflare.com/ajax/libs/d3/3.5.17/d3.min.js"></script>
<script type="text/javascript"<EMAIL_ADDRESS><script src="https://d3js.org/d3-color.v1.min.js"></script>
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0-alpha.6/css/bootstrap.min.css" crossorigin="anonymous">

I am using bootstrap 4 alpha instead of bootstrap 3. Could this be the issue??

The chart part of your code I am sure is correct. And your dependencies also seem to be; I tried your dependencies in my website code and it worked perfectly, so I don't think they are the problem.
I am not one of the developers of the library, so I don't know its inner working. It could be some other css you have. If that isn't the problem, I would contact one of the developers, @germanattanasio, @arypbatista and @aprilwebster.

What other CSS could cause the problem?? I am using simple grid layout with two columns, one of them containing sunburst chart. No fancy stuff.

If you don't have any other css, I don't know what would be the cause of the problem. I would contact the lead developers.

Is this just happening to you or you found this to be a problem in the library? I want to make sure I can reproduce your problem. Can you share the code you are using?

This issue has no reproducible steps. Please re-open this issue if it still occurs.
Check the CONTRIBUTING.md for an example.


Tweet HTML creates broken mentions

There seems to be an issue with the tweet html reconstruction, specifically with mentions. I'm getting results like:

@  previously mentioned in @ that too much capital was / have been deployed (or is trying to be deployed) in private markets. @ is just a public illustration of a deeper trend.

It seems to be identifying the mentions correctly, but usernames are missing from the replaced link URL/contents. No issues with finding other links or hashtags.

Fixed in #53


data set

hello
iam an algerian student
can you give me the data set please (images )
all my respect

Hello,
please note, that the Nikiforov dataset is publicly available (see https://journals.plos.org/plosone/article?id=10.1371/journal.pone.0257635). For the non publicly available Tharun and Thompson you can write me an e-mail to<EMAIL_ADDRESS>with a short project description about what you want to to with the data.


Fix Debug.DebugBreak

In CoreRT as currently implemented, any call to Environment.FailFast calls through to Debug.DebugBreak which does nothing that loop indefinitely. I would expect that if I call FailFast() the program would actually crash. Could this be changed?

Fixed by https://github.com/dotnet/corert/pull/1534


news api migration to v2

as I was getting interested in this project, I saw that there is not much documentation on newsapi.org, furthermore, they are now migrating to version 2 of the api, for which they do have some more documentation and examples.
If it is possible, I would like to take on the migration to v2 of newsapi. Let me know what you think.

I'm not @Griffintaur, but did you make any progress toward this? I made a fork on Python3 with a lot of cleanup, and switching the API to a newer version would likely be beneficial.

sure , will love to go for v2 of newsapi. @drusc0

Awesome, I have worked very slowly on this.  I will try to finish up this weekend :)

cool

Hi @Griffintaur, it appears that this issue was never resolved, so I looked over the News API docs and was able to quickly migrate to V2.


Add options to set headers on upstream http requests

This PR adds the command line option extra-http-header, which facilitates setting arbitrary http headers on the upstream prometheus request. This is required to support queries to prometheus compatible endpoints like Grafana Mimir, which expect an "X-Scope-OrgID:" header in the request to specify the tenant to query the data for.
Additionally, I have added a rewrite-host-header-to parameter, which supports rewriting the host to a different value than the source request.  This would be needed in the case where the upstream prometheus endpoint is an nginx proxy or simliar using host base proxying. This addresses the issue https://github.com/prometheus-community/prom-label-proxy/issues/135.

Does not seem to be alot of activity in this repo. @SuperQ could you have a look at this PR ?

@simonpasquier are you able to look this PR ?


UTF-8: updates UI parser to support UTF-8 characters

fixes- #13564
In this PR three children added to LabelMatcher in promql.grammar which are namely-

LegacyLabelMatcher which is the old label matcher.
NewLabelMatcher which is the label matcher with new syntax.
MetricNameLabelMatcher which is a matcher for metric name inside quotes.

An important note: In contrast to the text-based exposition formats, PromQL also allows single quotes and backticks as quoting characters in addition to double quotes. See https://prometheus.io/docs/prometheus/latest/querying/basics/#string-literals
For the new-style quoted metric names and label names, the same quoting rules should work as they already do for label values. So something like {'foo\'bar', `a\dos\path`="boo"} should also work.
@ywwg I guess that means we have to revisit #13271. I didn't remember to check when I reviewed it. Maybe it already works, but we should add tests.

LegacyLabelMatcher is probably OK, as things will stay "legacy" forever (while the "new" stuff will grow old eventually). But I would still prefer something slightly more descriptive like ClassicLabelMatcher or UnquotedLabelMatcher.

Yeah, It's more convenient to have QuotedLabelMatcher and UnquotedLabelMatcher instead of NewLabelMatcher and LegacyLabelMatcher. Since I couldn't come up with a good name I left it open for discussions. I'll follow these nomenclature and will update the PR soon.

For the new-style quoted metric names and label names, the same quoting rules should work as they already do for label values. So something like {'foo'bar', a\dos\path="boo"} should also work.

For the metric name and label names regex I referred #13271 and and I don't think it has rules for label names in single quote and backtick so I think we need to revisit to that also.

@ywwg I guess that means we have to revisit #13271. I didn't remember to check when I reviewed it. Maybe it already works, but we should add tests.

noted

@ywwg I guess that means we have to revisit #13271. I didn't remember to check when I reviewed it. Maybe it already works, but we should add tests.

created a PR to add tests -- looks like we're all good: https://github.com/prometheus/prometheus/pull/13656

@beorn7 I've added support of single quotes and backticks for LabelName.

Thanks @NeerajGartia21 . I'll try this out ASAP. @Nexucis could you check in the meantime if all your comments have been addressed?

@beorn7 this comment has not been addressed: https://github.com/prometheus/prometheus/pull/13590#discussion_r1512519825

@Nexucis I'm currently implementing this part. I'll commit the changes ASAP.

@Nexucis @ywwg @beorn7 , should we update the GroupingLabels to support the QuotedLabelNames ? Or is it out of scope for this PR ?

I don't understand enough about GroupingLabels in the UI to make a call. @Nexucis @ywwg WDYT?
In any case, you could address the remaining comment in the meantime. We'll see if @Nexucis is happy then.

Are GroupingLabels the list inside the parens in  "sum by (labels) (foo{})"?  Hmm yes that will have to support quoting as well

@Nexucis I've simplified the grammar. If you have time please review it.

@beorn7 @Nexucis @ywwg I have added support of QuotedLabelName to the GroupingLabels in the latest commit and I think it completes all the necessasy and requested changes.

@Nexucis could you have a look? I'm still completely lost when it comes to the frontend parts.

Thanks @Nexucis @ywwg @beorn7 for your reviews !
I think all the tests passed so we can merge it now.


Why is kaspanet.io down?

If this is a cloudflare thing, maybe Brave browser and ipfs?

This is not related to rusty-kaspa

A server issue. Will be addressed shortly.
This does not affect https//wallet.kaspanet.io


[STYLE CHANGE] Better UI and style change for Bhagwad Gita Quiz button

Description:
The constantly glitching button for Bhagwad Gita Quiz is not visually pleasing. Also Better Color for the button would look more attractive. I feel the current button styling doesn't go with the overall aesthetic of the page.
Reason for Change:
Smoother UX and UI for the user.
Current Styling:

The gliching button of Start Bhagwad Gita Quiz should be changed.
Current color of button must be changed in accordance to the overall theme.

Proposed Styling:

Smoother hover effect for the "Start Bhagwad Gita Quiz button"
Pleasing color for the button.

Screenshots or Design Mockups (optional):
https://github.com/akshitagupta15june/Moksh/assets/97681609/fa5f5764-67fb-4684-b802-a56164d2227e
Additional Information (optional):
Please assign me this issue and accept it under GSSOC'24.

Priority Level (optional):

[x] High

Timeline (optional):
[Specify any preferred or required timeline for implementing the style change.]
Within 3 days.

@akshitagupta15june I request you to please assign this issue to me under GSSOC'24. I also request you to please assign level label also. Thank you


Truncation behavior rather than floor() after multiplying

The following code:
https://godbolt.org/z/v9P9rn
Has a negative result of multiplying two signed 15.16 numbers (one positive, one negative). When the multiply result is reduced from the 62-bit intermediate representation to a signed 15.16 number, a signed floor() function is used. Is there a flag to pass to the type to make reductions do simple truncation (i.e. throw away the bottom bits).

Firstly, multiplying is unrelated to the rounding here. Multiplication is lossless (assuming the result is not too big) and therefore rounding doesn't affect it. The rounding in your example happens when converting to a type with fewer fractional digits. You see the same result with this simplified example without multiplication.
Secondly, no floor() function is used. The 64-bit value is simply scaled using integer division and the result is indeed truncated. Note that the result is actually greater than the original value. That is what happens when a negative number is truncated. There's a helpful graph on the std::floor() page of cppreference.com which should make it clear that when you floor a negative value, you get a lower value with a greater magnitude.
Hope that makes sense.

Thanks John. Yes I didn't mean the multiply itself is truncating. Simply that I'm arriving at the truncation behavior by multiplying first and then scaling down to a signed 15.16 number.
With regard to your second point, I know that no floor function is used but since you are scaling the signed integer by division, that does mean CNL does effectively floor, as per the spec of std::floor(). Am I correct in that assessment?
What I would ideally want is an unsigned truncation. Or rather, simply taking bits 47:16 of the underlying representation. I'm modeling hardware that wants to do this.
Moreover, I am doing this in a templated class where I will be swapping out float for cnl::fixed_point and vice versa. So I'd like the code to simply be:
template <typename T>
void my_func(T a, T b, T c, T& z) {
  T a, b, c, z;
   z = a * b + c;
}

So if I have to go through manual steps, it looks like I'd have to overload the conversion operator. Could you show me where that is?

Circling back on this. I think I can get away with nearest-rounding mode. That is, an elastic integer base (expands during multiply) that will convert to a 32-bit signed internal representation using nearest-rounding. And also saturates on overflow (can be 0x8000_0001).
I tried something like this:
https://godbolt.org/z/GfrvWc
But it seems to still round the same way (floor int type). Am I doing that right?

See v1_fixed_point_division branch. Yes I know it is badly named, but originally I had question about the fixed point division. I fixed some tests for the the "tie_to_pos_inf_rounding_tag" (which is essentially add "half" and truncate for the HW designers) and then implemented floor_rounding_tag (that is 2's complement truncate rounding mode). These are the two ones the signal processing HW designers tend to use. These seem to work correctly, but I have not heavily tested these.
I probably still need to rename that according to p0105r1 to from floor_rounding_tag to neg_inf_rounding tag to be on par with tie_to_pos_inf rounding tag.
If we see these to fit our needs I make the final changes and John can marge/rebase these to v1 branch.

Now renamed. Thus if you now would use neg_inf_rounding_tag, you get desired 2's complement truncation rounding mode, which rounds towards negative infinity.

I've talked to the hardware guy and he's implemented a full rounding mode. So I'll be modeling that. cnl::rounding integer (per my code above) works. But unfortunately, it also sees an error when I try to combine it with elastic and overflow integers....

Thanks hbe72; I'll keep that branch in my back pocket for when the hardware does 2's complement truncation rounding.

I'll hope John has time to look at the branch. I think is good enough to be integrated to v1 branch, and bump the version to v.1.0.1, if he finds why the compilation does not work without INT128 support. The same error is in current v.1.0.0 with tie_to_pos_inf rounding mode, but there are no tests to expose the error. All rounding modes would always benefit of more rigorous testing.

@imgod2u

It would seem that a combination of overflow, rounding and elastic integer would push the internal representation higher than 128 bits (max supported by CNL). Any advice on this?

Is this still an issue?

Hi John, it is still an issue. However I've worked around it and basically abandoned elastic_integer. I went ahead and overloaded the * operator for my specific type:
using s15_16_rep = int32_t;
using s15_16_scale = cnl::power<-16>;
using s15_16_ovfl = cnl::overflow_integer<s15_16_rep, cnl::saturated_overflow_tag>;
using s15_16_round = cnl::rounding_integer<s15_16_ovfl, cnl::nearest_rounding_tag>;
using fp = cnl::scaled_integer<s15_16_roundingl,s15_16_scale>;

namespace cnl {
fp operator * (const fp& x, const fp& y) {
// code that handles expanding to 64-bit intermediate and truncation
return my_fp;
}

To get both the expanding and truncation, overflow and rounding behavior that I was after.
Can you verify that 128-bit is the absolute limit? And that trying to combine elastic_integer, rounding_integer and overflow_integer like I tried violated this? We can then close this issue.

There is a wide_integer type but that is nowhere near reliable. I'll leave this issue open for now but am I right in assuming that the problem you currently have is no longer about rounding modes but about numbers getting too wide during multiplication?

Well, it would be nice to have configurable rounding the way that hbe72 described. For my particular use-case, I was able to verify that the algorithm doesn't need the type of rounding I originally requested.

@hbe72

Now renamed. Thus if you now would use neg_inf_rounding_tag, you get desired 2's complement truncation rounding mode, which rounds towards negative infinity.

Confused: isn't rounding towards -ve infinity different to truncation for -ve values?

Confused: isn't rounding towards -ve infinity different to truncation for -ve values?

In two's complement binary representation dropping away the bits is equivalent to rounding towards negative infinity.
0011.0100 is 3.25
1100.1100 is -3.25
Now let's truncate -3.25
1100.1 is -3.5

@hbe72 Of these two opposing things described as 'truncation', the former is likely to confuse other users who are not aware of -- or concerned with -- the underlying implementation and its peculiarities, just like it is confusing me!
I think most people would say that if you truncate -3.25, you get -3. I would suggest that we find a different word for the thing that you and @imgod2u are describing.
Suggestion: just as right shift comes in two flavours: bitwise and arithmetic, so truncation should be qualified as either arithmetic truncation and bitwise truncation. Just as with right shift, bitwise truncation is the same as arithmetic truncation for non-negative values and unsigned types. And unless specified, truncation should refer to arithmetic truncation so that the model is simpler for users to follow, more implementation-agnostic and more consistent with expectation WRT floating-point behaviour.

@johnmcfarlane Yes it is difficult to discuss of the terminology is not exact. I understand now that truncation is a term generally used for signed magnitude values. The terminology should be common to more SW oriented and HW oriented users of the library. Sometimes it is difficult if you are accustomed to using and understanding the terms differently in your respective domain. But yes binary truncation is very different than arithmetic truncation.
Unfortunately I did state in the comments of here, that "Arithmetic operations using this tag round by truncating 2's complement binary representation, thus round all values towards minus infinity". It would have been sufficient to say this implements "round towards neg infinity rouding" and say nothing about 2's complement. I would imagine the ones who are after 2's complement binary truncation should understand it to be equivalent to rounding towards negative infinity.

@hbe72 Yes, incorrect would be worse than terse so that's fine. Given all of that, is @imgod2u 's original mode now supported?

Yes

Thanks @hbe72. OK, I'm inclined to close this issue then. @imgod2u, try neg_inf_rounding_tag rounding mode if it turns out you need the originally-requested behaviour.
In the longer term, the type, static_number is a type that combines properties from many other types in the library including wide_integer which supports arbitrary width. It is a longer-term aim of the project to make these types reliable and the transition to C++20 should make that a lot easier because many of the algorithms related to wide integers are better expressed using loops, than with recursion. Recursion is required, however, due to limitations in C++11 constant expressions. Concepts will also help move this work along. Note, though, that these are long-term goals with no timeline currently for achieving them.
Please don't hesitate to open a new issue if there are any outstanding issues arising from this thread.


header overlaps id links

e.g. using the link to repro --pull, the --pull entry is hidden under the header banner and you need to scroll up to actually see it.

Hmm, it works fine for me. What is the browser / system @casperdcl ?

odd can't reproduce now.
It was firefox on ubuntu.


SAK-29355 Keep “bad” URL in error site.

This allows a tool running in the error site to know the URL that the user was having problems accessing, rather than have all the URLs just include the error site ID.
A simpler option would have been to use the Sakai session but this would have meant error handling wouldn't have with multiple windows and it would have bloated the size of the session.

Thanks for review, updated and rebased branch.

Okay, LGTM


Symbols auto generating now?

Are symbols autogenerating now? Every once in a while I notice my Macbook is getting super slow and when I check the process list "ctags-darwin" is always claiming all of my resources.. There does not seem to be any setting inside of the package to enable or disable this?

Correct, once you generated tags once for a project it symbols-gen will keep those tagfiles up to date with your source.

Well, it's killing my laptop (Macbook Pro Retina '14). Eventually, me and my colleagues removed the plugin because it was making our laptops almost freeze every x minutes.

You could create a .ctags (in project or globally in ~/) file with excludes for e.g. bower_components, build/target etc. to avoid creating unnecessary tags for dependencies/concat/minified files etc.
http://ctags.sourceforge.net/ctags.html#OPTIONS

I'll give that a try and will come back to this after some testing, thanks @baelter.


Unify the class registration code.

In #81 @karroffel added a way to register native classes without the godot_class macro. This PR refactored the whole thing so that both systems use @karroffel's ClassBuilder instead of having two separate implementations.
In the process I had to change ClassBuilder from using the builder pattern (methods that consume self) to a simple object that directly calls into godot's registrations APIs (it made it easier to add support for signals, and without that, making it work with godot_class was a mess).
As a bonus, it is now possible to register properties and signals manually with ClassBuilder, and a lot of awful macro code was removed/de-duplicated.

I added back the possibility to rename the endpoints (optionally) and removed the godot_init macro to avoid confusion around having several ways to do the same thing.


splunk WebUI for setting the configuration as an enhancement

[ ] Splunk pump
Splunk server address: [ ]
Splunk server keys: [ ]
Auto-scale splunk pump instances: [x]

https://github.com/dell/iDRAC-Telemetry-Reference-Tools/pull/100/files


Syntax error on IE11

The svelte NPM uses the arrow syntax, but this is not transpiled for older browser, giving syntax errors on IE11.
The error can be reproduced using the example repo at https://github.com/meteor-svelte/tracker-example
This is the place, where the error appears in modules.js, see the last line:
///////////////////////////////////////////////////////////////////////////////////////////////////////////////////
//                                                                                //
// node_modules/svelte/internal.js                                                                               //
//                                                                                //
///////////////////////////////////////////////////////////////////////////////////////////////////////////////////
                                                                                //
'use strict';

Object.defineProperty(exports, '__esModule', { value: true });

function noop() {}

const identity = x => x;

The recompile option found in the Meteor forum didn't help:
https://forums.meteor.com/t/can-anyone-confirm-meteor-1-10x-cannot-run-in-old-browsers-from-2015-eg-chrome-35-and-cannot-run-in-ios-9x/53262/13
  "meteor": {
    "mainModule": {
      "client": "client/main.js",
      "server": "server/main.js"
    },
    "nodeModules": {
      "recompile": {
        "svelte": true
      }
    }
  },

Actually, I've found a workaround, which is described in a blog article from 2018: https://blog.meteor.com/meteor-1-7-and-the-evergreen-dream-a8c1270b0901

Clone the package repository into your application’s  imports  directory (that is, if you’re still using an  imports  directory)
Make any modifications necessary to compile the package to your liking, such as adding/modifying a  .babelrc  file or altering the  "main"  field of  package.json  to point to  src/index.js  instead of  lib/index.js
Use  meteor npm install imports/the-package  to link the package into  node_modules

Actually, I moved node_modules/svelte to imports/svelte and executed meteor npm install imports/svelte afterwards.

I’ve found a more elegant workaround in https://forums.meteor.com/t/litelement-import-litelement-html/45042/8

Installation of the NPM as always
Adding a symbolic link to the module in the imports directory

$ meteor npm i -s<EMAIL_ADDRESS>$ cd imports/
$ ln -s ../node_modules/svelte/

As I haven't found a way to trigger this behaviour in the svelte:compiler package, it should be at least documented.

Thank you for investigating this issue (and for the pull request). Do you know why the recompile option in package.json isn't working in this case? I think it was added as a replacement for the symlink method.

Unfortunately, I've no idea why it was not working in my case. I didn't investigate deeper in this specific issue, I was just happy to find an easy solution for my problem.

I experienced the same as @fknipp. Thank you for your workaround. It got Cordova on iOS to work. :)
@klaussner I recommend adding this to the Readme until there's a better solution.


TSC.SubscriptionItem, Payload is either malformed or incomplete. (0x5CE10192 : Specifying subscription attachments not allowed.)

Hello, I'm trying to create a subscription for a user on a view. I put this piece of code,
schedule_id = '60c392fc-d902-44b0-92cf-cfa3b512630f'
user_id = df[0].values[i]
# Create the new SubscriptionItem object with variables from above.
new_sub = TSC.SubscriptionItem('EssaiGD', schedule_id, user_id, target)
    # (Optional) Set other fields. Any of these can be added or removed.
    new_sub.attach_image = True
    new_sub.attach_pdf = True
    new_sub.message = "You have an alert!"
    new_sub.page_orientation = TSC.PDFRequestOptions.Orientation.Portrait
    new_sub.page_size_option = TSC.PDFRequestOptions.PageType.A4
    new_sub.send_if_view_empty = True

    # Create the new subscription on the site you are logged in.
    new_sub = server.subscriptions.create(new_sub)

I launch it and when I have the error it gives me 0x5CE10192 : Specifying subscription attachments not allowed.
THanks
Version :
Product version: 2024.2.0
REST API version: 3.23
Build number: 20242.24.0711.1807
Python 3.10.6

From the REST API documentation:

If subscriptions to attachments are disabled in Tableau server or site settings, then making a request that sets attachImage to false will cause an error.

the parameters are true and not false in my script. So when I manually create the subscription or when I use a request it works. So I don't understand why it doesn't work

What happens if you set image to True, pdf to False, and remove the pdf specific options? Try the simplest case first.

the same

?? Help me :-)

@gdrau what API version are you using in your script? This error message will also be returned if you are specifying an API version below 3.5, when the feature was implemented.

I created a sample for subscriptions and verified that it succeeds with the current server version, and fails using 2.4. https://github.com/tableau/server-client-python/pull/1487


Support for mako templates?

Sometimes, there is a need for more complex input prompts, so it would be nice to have support for mako templates.
Instead of this
@ell.simple(model="llama-3.1-70b-versatile", temperature=0.1)
def intention(prompt: str):
    return (f"""What is the intention of the following text:
    ```
    {prompt}?
    ```

    You must answer with one word only, in English. No punctuation or capitalization is needed.
    Small caps only.
    """)

I would do something like that (Semantic Kernel has something similar):
from os import path
from mako.template import Template

def load_prompt(prompt_file: str) -> Template:
    return Template(filename=path.join(f"prompts/{prompt_file}.txt"))

@ell.simple(model="llama-3.1-70b-versatile", temperature=0.1)
def intention(prompt: str):
    return load_prompt("intentions/_default").render(prompt=prompt)

Unfortunately today, it doesn't work:
.venv/lib/python3.11/site-packages/ell/util/closure.py", line 311, in _raise_error
    raise Exception(error_msg)
Exception: Failed to capture the lexical closure of global or free variable load_prompt. Error: Failed to capture the lexical closure of global or free variable Template. Error: Failed to capture the lexical closure of global or free variable _compile_text. Error: Failed to capture the lexical closure of global or free variable _compile. Error: Failed to capture the lexical closure of codegen.compile. Error: Failed to capture the lexical closure of global or free variable _GenerateRenderMethod. Error: Failed to capture the lexical closure of global or free variable _Identifiers. Error: Failed to capture the lexical closure of parsetree.DefTag. Error: Failed to capture the lexical closure of ast.ArgumentList. Error: Failed to capture the lexical closure of global or free variable PythonCode. Error: Failed to capture the lexical closure of pyparser.FindIdentifiers. Error: invalid syntax (<unknown>, line 9)
Recursion stack: intention -> load_prompt -> Template -> _compile_text -> _compile -> compile -> _GenerateRenderMethod -> _Identifiers -> DefTag -> ArgumentList -> PythonCode
Recursion stack: intention -> load_prompt -> Template -> _compile_text -> _compile -> compile -> _GenerateRenderMethod -> _Identifiers -> DefTag -> ArgumentList
Recursion stack: intention -> load_prompt -> Template -> _compile_text -> _compile -> compile -> _GenerateRenderMethod -> _Identifiers -> DefTag
Recursion stack: intention -> load_prompt -> Template -> _compile_text -> _compile -> compile -> _GenerateRenderMethod -> _Identifiers
Recursion stack: intention -> load_prompt -> Template -> _compile_text -> _compile -> compile -> _GenerateRenderMethod
Recursion stack: intention -> load_prompt -> Template -> _compile_text -> _compile -> compile
Recursion stack: intention -> load_prompt -> Template -> _compile_text -> _compile
Recursion stack: intention -> load_prompt -> Template -> _compile_text
Recursion stack: intention -> load_prompt -> Template
Recursion stack: intention -> load_prompt
Recursion stack: intention

Can you send me a sample template for me to replicate?

For example this one:
What is the intention of the following text:

${prompt}

You must answer with one word only, in English. No punctuation or capitalization is needed.
Small caps only.