Static typing of metric values

https://github.com/prometheus/prometheus/blob/master/retrieval/format/processor0_0_1.go#L56
Is a context sensitive union type that can produce runtime errors when the "type" key does not match the expected type of the value of the "value" key.  Consider using a key name for the type rather than a "type" key with the metric type as a value.  Like:
[
  {"metric": {"labels":[], "count": 1} },
  {"metric": {"labels":[], "histogram": { "0.01": 15, "0.1": 20 }
]

This means that collectors ignore metrics they don't know how to join.  If you want to be explicit, keep the "type" key as the tag for the union of possible metric types.  Then the schema can use static typing for possible metric values.
TL;DR: Do not use interface{} types inside in the schema definition for significant fields.

JSON format 0.0.1, I think we are over it.


[11.x] Fixes trust proxy REMOTE_ADDR not working in Swoole

When using REMOTE_ADDR set the trust proxy in Swoole, like this:
      $middleware->trustProxies(
            at: [
                'REMOTE_ADDR',
            ],
            headers: Request::HEADER_X_FORWARDED_FOR
        );

Is nothing set to request->trustedProxies  because $_SERVER['REMOTE_ADDR'] does not exist in Swoole. But Laravel internally converts the server information in Swoole into the request->server attribute, which includes REMOTE_ADDR, and then request()->server->get('REMOTE_ADDR') can get the corresponding value normally.
This leads to some logical ambiguity and increases the cognitive burden and learning cost.
So we need to unify the logic to avoid inconsistency. I can't change this in the Request class because of the method attribute restrictions, so the solution I've come up with so far is this, it works for me, and I hope there's another better way to solve it.

Can you add a test then mark as ready for review?

hi @taylorotwell  I added one test case. the checks were unsuccessful, but not caused by this change, it seems some other network is wrong. Can you recheck this?


scale update not correct if entering scale value by typing rather than increment

I see this problem in my sim plots. If I type a value for scale into the spinbox the scale might change to smaller when I actually increased the scale. No problem with scale change when using the increment/decrement of the spinbox.

This is truly bizarre.  I have traced through and it is definitely setting the new scale and calling the same Update method correctly in both cases.  I can't see any diff.  If you just twiddle something else in the config it displays properly -- seems like a second pass of updating is happening with the incr / decr but I can't find it..

should be fixed now.


Fixed Object.create polyfill issue when called through Babel's Object's create wrapper

Opening this in favor of #706 to run CI
Original comment by @roland-vachter:
"arguments can be 2 even if descriptors is explicitly undefined, so it's good to check if it's not undefined. Tested, solves the issue."

@JakeChampion is on holiday but he gave the ok before leaving

Looks OK to me.


⚠ Processors has degraded performance

In f36a330, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 84 ms

Resolved: Processors performance has improved in f0c6527.


Eliminate special case for extracting from a binary operator chain

It assumed left-associativity and was, therefore, wrong for (e.g.) exponentiation.
Arguably, if a user selects a + |b + c|, they want to extract b + c, not a + b + c.  Not being able to do so is surprising (and we may eventually want to allow it), but so is having the rest of the least-common subtree extracted.

Fixes #18268

Why not continue allowing it for associative operators like + and *?

@DanielRosenwasser I think it would be great to support that, but I'd want it to extract the range I specified - not an expanded range.  Do you want to file a suggestion to that effect?

Offline, @DanielRosenwasser and I agreed that implementing the desired behavior is a separate issue.


Suggestion (III) - Reduce the yellow bar in the main menu ever so slightly

Could you please do this so when using the MAP mod the exit game command does not sit on top of this bar?
https://i.imgur.com/Nl4xdE6.jpeg

This bar is a texture and a part of the background, so no.

OK, I was wrong, this bar is actually drawn as part of the menus:

However, this is still not for SilentPatch to do. The Pause Menu mod should have handled that, it's not for SP to fix things the other mods did not account for. I recommend opening an issue in https://github.com/gennariarmando/menu-map instead.


added issue template #896

Added issue template
Related issue: https://github.com/quran/quran.com-frontend/issues/896
Checklist

[x ] Manually tested
[x ] Prettier & ESLint were run

Deployed to: http://staging.quran.com:32880


Add more code examples to Readme

Hopefully this makes sense!

Very helpful modification!

Very helpful modification!


Let IsolatedProject delegate equals and hashCode

Follow-up to #28011.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle merge

WARN: This is a feature pull request but no changes to release note found.

The merge queue build has started. Click here to see all failures if any.


Vitest nodejs tests do not run - CAC error

Describe the bug
Looks like it's having trouble with another library.  Debug console says "Please start a debug session to evaluate expressions". There doesn't seem to be any discussion about this anywhere.
/home/user/.nvm/versions/node/v18.19.1/bin/node ./node_modules/vitest/vitest.mjs run vitest --silent=false --disable-console-intercept path/__test__/test.mjs -t Test Controller Test testUnit --api.port 36015 --api.host <IP_ADDRESS>
Uncaught CACError CACError: Unknown option `--disableConsoleIntercept`
    at checkUnknownOptions (path/node_modules/cac/dist/index.mjs:400:17)
    at runMatchedCommand (path/node_modules/cac/dist/index.mjs:598:13)
    at parse (path/node_modules/cac/dist/index.mjs:537:12)
    at <anonymous> (path/node_modules/vitest/dist/cli.js:72:7)
    at run (internal/modules/esm/module_job:195:25)
No debugger available, can not send 'variables'
Process exited with code 1

Reproduction
Run any test from within vscode
System Info
Ubuntu 23.10, vscode 1.87.2, vitest 0.4.1

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

The latest version is 0.5.6 (pre-release).

The latest version is 0.5.6 (pre-release).

Happens in this version as well.  It looks like it's happening somewhere in the vitest code. There is no string --disableConsoleIntercept anywhere that I can see, though.  So I don't know why it's complaining about this option.

Turned out to be an issue with conflicting module versions


Make "Export with Debug" more visible in the Export dialog

This swaps out the CheckButton with a CheckBox, which has two benefits:

The checkbox icon appears to the left of the text, which moves it closer from the text. This makes it more easily noticeable, as it also appears below the "File:" text now.
It follows the UI convention of using checkboxes for options that do not bear an immediate effect, unlike CheckButtons which are expected to have an immediate effect when toggled.

This closes #25170.
Preview

The colored text is still not easy to see, but that's a theme issue which can be seen in other places.

Now that Godot 3.1 has been released, we don't plan to cherry-pick new features and enhancements to the 3.0 branch, unless there is very strong support in favor of it. Removing cherrypick label for 3.0.


bundlerEnv: Allow overriding bundler

Motivation for this change
Prior to this it doesn't seem to be possible to customize the version of
bundler used in a bundlerEnv app. This change allows something like the
following:
let bundler = pkgs.buildRubyGem rec {
  gemName="bundler";
  name="bundler-2.2.11";
  version="2.2.11";
  source.sha256="1izx6wsjdm6mnbxazgz1z5qbhwrrisbq0np2nmx4ij6lrqjy18jf";
};
in pkgs.bundlerEnv { inherit bundler; }{
  name="test";
  gemdir=./.;
}

to use bundler 2.2.11 rather than the 2.1.5 default.
(As a one-liner test from the nixpkgs repo:
nix-shell -I nixpkgs=./ -p 'let bundler = pkgs.buildRubyGem rec {gemName="bundler"; name="bundler-2.2.11"; version="2.2.11"; source.sha256="1izx6wsjdm6mnbxazgz1z5qbhwrrisbq0np2nmx4ij6lrqjy18jf";}; in pkgs.bundlerEnv.override {inherit bundler;} {name="test"; gemdir=./pkgs/development/tools/sass; }' --pure --run "bundler --version"

Things done

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

[ ] NixOS
[x] 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.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/overriding-bundler-version-in-bundlerenv-with-a-possible-fix/11616/5

ping @alyssais

@ofborg eval

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/overriding-bundler-version-in-bundlerenv-with-a-possible-fix/11616/8


Boss import

Added Boss import feature and button for it.
Also changed a bit ExportUtilTests to use new function.

Please fix conflicts and I don't like that ExportUtils class contains code for importing things... Its nonsense.

Conflicts fixed. Also ExportUtils renamed to JsonUtils as it contains functions both for export into json and import from json files.


import module

module “cpp_wrappers.cpp_subsampling.grid_subsampling”has no attribute "subsample" when I run train_S3DIS

Hello, have you solved this problem? I also encountered this problem.


wz stats by @UsernameDiscord tag

Might be useful to check the stats of the people registered.
Possible con is pinging people around so maybe we could just use that persons nametag, whoever, if they have a different name than their discord username, this might be a problem.


V4 fix bson issue

Fix for #75
Users may want to pass on mongo Object ids as attributes.
These ObjectIds are BSON objects. asyncPopulate thinks of them
plain son objects and copies them deeply which messes up the ObjectId

@gunar, @jonatanklosko thanks for the feedback. I have updated the PR with your suggestions!


chore: stop subscription goroutines if node is stopped

Fixes #388
We should also add a screen to display when the node is offline, which can be used by backends other than LDK, because currently Alby Hub goes blank if the node is offline with no message. But we can open a separate issue for that(?)

@im-adithya @bumi  what about we shut down the LNClient if we detect LND is not running, and also do not allow Alby Hub to start if LND is not running. Is this possible?
The current change only stops the subscriptions, and leaves the LNClient in a weird state

This is how the Hub looks once the node goes down. I tried starting it again and after a few seconds (roughly a minute), it was working perfectly as it used to 👌


Wrong youtube thumbnail loaded

I am using this library to load youtube player in a recycler view. On scrolling up and down, wrong thumbnail of the video is loaded. Sometimes wrong thumbnail is load ever after the correct thumbnail loading is done, i.e, the correct thumbnail replaces by the wrong thumbnail within a fraction of a second. I am using this in native android project. I have tried Glide and Picasso libraries to load image inside ImageLoader interface

This looks like a recycling issue. Can you paste the gist of the code to help us debug ?
Also the demo app uses recycler view to load 5/6 youtube views. Are you able to reproduce the same issue in the demo app ?


Docs: Add search feature

What this PR does / why we need it
This adds documentation search to /docs using DocSearch v3 from Algolia. The index configuration can be found at algolia/docsearch-configs
Details for the Release Notes (PLEASE PROVIDE)

Which issue(s) this PR fixes
Fixes: #1243
Describe testing done for PR

hugo sever
Navigate to /docs/
Note: results will resolve to tanzucommunityedition.io and not localhost

Special notes for your reviewer
This is migrates DocSearch from v2 to v3.alpha. The v3 interaction is different as it uses a button/overlay to invoke a search interface. We could stick to the v2 input if this doesn't present well.

Thanks for the note! Updated package.json and package-lock.json with the most recent docsearch v3 alpha build. I think having an explicit version/build will make dependency tracking a little easier.

will defer to @jpmcb for validation

@jpmcb can you validate, thanks


Validate the input and check that it is in the correct format

At the moment the code assumes that it is in the following format...
NicknameWinner NicknameLoser MatchLength
Modi Jesse 11
Make a check that the reported match is according to the following pattern...
Word Word Number
If it is not correct an empty string should be returned.

Probably, not needed after changing to report with Actions because the input is stricter controlled.


Enhancement: show duplicate version properties

When I run mvn versions:display-property-updates including the parent versions, it doesn't show any information about where the property is found.
It would be nice for me to see if I have manually specified any version property which is the same (same name and value) as a property in any of the ancestors (parent, grandparent etc). In that case, I could delete my version property and clean up my pom a little bit.

Generally it looks like rule for enforcer plugin.
Which can check any override properties.

Not planned in this plugin.


How to increase speech to text speed when using whisper cpp?

@nyadla-sys  Currently I am using whisper_tiny.en.tflite for my android project. And the inference time for speech to text is quite long.
https://github.com/usefulsensors/openai-whisper/tree/main/android_app/Whisper-TFLIte-Android-Example
I/flutter ( 3449): Model name: /assets/whisper-tiny.en.tflite
I/flutter ( 3449): Load success
I/flutter ( 3449): Load model time 0:00:03.274847
I/htotext_exampl( 3449): ProcessProfilingInfo new_methods=1340 is saved saved_to_disk=1 resolve_classes_delay=8000
I/tflite ( 3449): Initialized TensorFlow Lite runtime.
W/1.ui ( 3449): type=1400 audit(0.0:15783): avc: denied { read } for name="u:object_r:vendor_default_prop:s0" dev="tmpfs" ino=14533 scontext=u:r:untrusted_app:s0:c127,c257,c512,c768 tcontext=u:object_r:vendor_default_prop:s0 tclass=file permissive=0
E/libc ( 3449): Access denied finding property "ro.hardware.chipname"
I/tflite ( 3449): Created TensorFlow Lite XNNPACK delegate for CPU.
I/flutter ( 3449): Result process audio file: I love you.
I/flutter ( 3449): Processing time 0:00:11.894501
Is there any way to shorten the time, please help me?
Thanks!

Are you running on Android

Are you running on Android

Yes, i am running it on android
I have customized the C source into separate functions for each function. Like load model and process audio. And run in flutter.
I'm not sure if the cause is inference or not. However, when performing call func process audio to text conversion takes up to 11 seconds. Model loading is done when initializing the application.
And I also tried running your original Android source again, but it also took the same amount of time. Is it because of the model?
Can you give me a solution on how to reduce this time?

@vilassn pls help this query

Meanwhile try below whisper  app on your android phone.let me know how my much time does it take to display text.
https://play.google.com/store/apps/details?id=com.whisper.android.tflitecpp

Meanwhile try below whisper app on your android phone.let me know how my much time does it take to display text. https://play.google.com/store/apps/details?id=com.whisper.android.tflitecpp

I just downloaded the application and tested it on Samsung Galaxy A04 and the time fluctuated around 10 seconds

Try on high end android phone like pixel 7 or latest samsung galaxy series

Old phones may not have better compute cycles available

Old phones may not have better compute cycles available

Sorry for bothering you. I currently do not have a newer Android device. Can you tell me the maximum speed you have used of the app?
If so, the speed depends on the device's computing configuration. Is there any way to speed up calculation?

Is the source code of the app on CHPlay this source?
https://github.com/usefulsensors/openai-whisper/blob/main/android_app/android_whisper_app.zip

Yes


negative age

Hi，shamangary：
In the age prediction process，the age is negative，is there a way to avoid this？
Thanks。

Hello @zhly0, in our funcion: https://github.com/shamangary/SSR-Net/blob/master/training_and_testing/SSRNET_model.py#L144
, the final prediction is 'age = (a+b+c)*V'
Considering the index is also controlled by the dynamic values 'index = (i+lambda_local*x[6][:,i])',
it is possible to cause negative values which is not supposed to happen. The solution could be:
(1) set lambda_local = 0 (2) change the initial index 'https://github.com/shamangary/SSR-Net/blob/master/training_and_testing/SSRNET_model.py#L301': from 0 to 1 and do the same in the three for loop.

Thanks，I will try！


Make cross-platform portable

This pull request addresses two bugs exclusive to Windows:

Breakage during installation caused by oniguruma v7.2.0 addressed by the "Lock first-mate subdependency to v7.3.0" commit
Path-related errors addressed by the "Update path segment separators to be cross-platform" commit

Details are provided in their respective commit descriptions. Tested on Windows 10 and Debian 9.

@googlebot I signed it!

Hi @sebastianbenz - just a ping on whether you want to accept or reject this PR so we can close it out. Thanks!

Let's close this for now, as we're going to deprecate the old template build process anyway. The should make sure though that the new version works cross platform.


Update build-constraints.yaml

added network-multicast

@snoyberg I think Travis failed fetching the sources on this one. Meanwhile I've uploaded the latest version in upstream: https://hackage.haskell.org/package/network-multicast

I checked on my local system before merging

@snoyberg Thank you


install_requires in setup.py

It would be nice if you could provide install_requires in the setup.py file to specify the requirements. There are tools (like pip/pip3 and Debian's dh-python) that use the dependency information from this entry. For more information on this, have a look at the discussion section of the Python Packaging User Guide.

I've been considering that, however the tool is designed in a way to run fine and provide the core functionality without any additional modules/libraries. If you need features like GeoIP or fuzzy hashes (ssdeep), then you need to install the requirements. So I decided to put everything in requirements.txt file and let the user install it on demand. Does it sound reasonable?

I see. That is of course a reasonable decision. What about making that decision explicit by adding
install_requires = [],
extras_require = {
    'geoip': ['GeoIP>=1.3.2'],
    [...]
}

(along the lines of what is described here)?

If I'm not mistaken I need to specify names of extras to be installed. What is the advantage over pip3 install -r requirements.txt?

I think there are two advantages of using install_requires/extras_require:

It expresses the dependency relation in more detail (what are compulsory dependencies, what is optional).
It allows for more flexibility/granularity in using/specifying optional dependencies.

Finally I followed your suggestion and added extras_require in commit dee6008. Now it should be possible to install bare minimum with pip install dnstwist or along with all extra packages with pip install dnstwist[full].


(master) Fixed bug with using Redis for core-command in Docker.

Fixes #2108 in master. See #2111 for discussion and verification steps.
Signed-off-by: Brandon Forster<EMAIL_ADDRESS>

Codecov Report

Merging #2112 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2112   +/-   ##
=======================================
  Coverage   41.76%   41.76%
=======================================
  Files         184      184
  Lines       12151    12151
=======================================
  Hits         5075     5075
  Misses       6779     6779
  Partials      297      297

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 0916a3a...b00b29d. Read the comment docs.


Fix some incorrect options used

There are incorrect options being used in lock script which results in errors when executed, fix it by referring man page of i3-lock.

i3 lock, and i3 lock color may have some differences, also the version may be a cause see to it if i3 lock color works with that config.


ImportError: libgfortran.so.3: cannot open shared object file

Hi
I'm trying to update a biconda recipe 'stream' but got some unexpected errors. Björn suggested to me that it might be related to rpy2. https://github.com/bioconda/bioconda-recipes/issues/14550. When you have a minute, I would be really grateful if you could help take a look .
Thanks a lot!

Please try the latest build and reopen if this is still an issue


Does this work on the latest React-native (0.24.1) version?

Can confirm it does work without any floors! 👍


After update to lnd 0.5-beta and bitcoin-core 0.17.0 cannot unlock wallet on a testnet

Hi,
after following your instructions for updating lightning client and bitcoin core to newest version,
I end up with following error when I try to unlock freshly created wallet as admin:

[lncli] unable to read macaroon path (check the network setting!): open /home/admin/.lnd/data/chain/bitcoin/mainnet/admin.macaroon: no such file or directory

I cannot figure out why is it looking for mainnet admin.macaroon as I created wallet only on the testnet,
and tried to unlock it as admin
lncli --network=testnet unlock
Any idea what could be forcing lncli to look for the mainnet macaroon?
I have checked both bitcoin.conf and lnd.conf, both are set for the testnet.
Which i also see if i run lnd as bitcoin user

2018-09-25
01:23:05.536 [INF] LTND: Version 0.5.0-beta commit=3b2c807288b1b7f40d609533c1e96a510ac5fa6d
2018-09-25 01:23:05.536 [INF] LTND: Active chain: Bitcoin (network=testnet)
2018-09-25 01:23:05.538 [INF] CHDB: Checking for schema update: latest_version=6, db_version=6
2018-09-25 01:23:05.549 [INF] RPCS: password RPC server listening on [::]:10009
2018-09-25 01:23:05.549 [INF] RPCS: password gRPC proxy started at <IP_ADDRESS>:8080
2018-09-25 01:23:05.550 [INF] LTND: Waiting for wallet encryption password. Use lncli create to create a wallet, lncli unlock to unlock an existing wallet, or lncli changepassword to change the password of an existing wallet and unlock it.

I had the same issue. I ended up keeping the default paths for macaroons files (not specifying any in lnd.conf), and --network=testnet is mandatory when on testnet.
Useful trick, add alias tlncli='lncli --network=testnet' to /home/bitcoin/.bashrc
Have a look at https://github.com/lightningnetwork/lnd/issues/533

I just updated the whole guide from beginning to testnet, building a new RaspiBold from scratch. Can you check out if your settings match the ones I just validated?

Thanks for the updated guides, unfortunately it still does not work (although settings match). I will just wipe everything and start from scratch.


Reset dirty property after submit

Hello
I'm trying to reset the dirty property after my form is submitted.
I'm using it to know if users made changes in the form and alert them when they want to leave the page.
First question : is it the good property tu use for that ?
If yes, how could I reset the boolean to false, once the form is submitted ?

Something like resetForm({ values: currentValues }) (where currentValues is the values that are now not "dirty") should work, that will set the initial values and compute the dirty property to false

documented in https://github.com/formium/formik/pull/2743

add enableReinitialize to Formik

@inurlan

add enableReinitialize to Formik

When I use enableReinitialize the submitCount resets to 0 for some reason and I need this to set a "saved" state on the form. Same happens when I use resetForm({ values: {...} }). I'm setting submitCount to 1 in resetForm as a workaround.

Ref: https://stackoverflow.com/a/55803798/6613333
This worked for me
onSubmit={(values, { resetForm }) => {

      // do your stuff
      resetForm();

}}


Requesting config and lights every 5 seconds

Hi!
I installed this plugin to access my original Philips Hue. It works except for a request every 5 seconds to get the config and lights
7/21/2017, 12:13:24 PM] [Hue] Club-Robbo: hue bridge request #41: get /config
[7/21/2017, 12:13:25 PM] [Hue] Club-Robbo: hue bridge request #41: ok
[7/21/2017, 12:13:25 PM] [Hue] Club-Robbo: hue bridge request #42: get /lights
[7/21/2017, 12:13:25 PM] [Hue] Club-Robbo: hue bridge request #42: ok
Is there a config parameter to stop this?
Also as a side note I get lamps that will intermittently turn off when they shouldn't. Is that a Homebridge issue or a result of one of the above messages timing out?

No, this is by design.  As the Hue bridge offers no notifications, homebridge-hue, like any Hue app, needs to poll the bridge, retrieving it's state every five seconds.  You can set the polling interval using the heartrate parameter in config.json, or using the custom characteristic in the bridge service.  You'd need another HomeKit app for that, as the iOS built-in Home app doens't support custom characteristics.
These debug messages actually only show when homebridge is started with -D.

Thanks for the quick reply. I know the messages are only on debug as I was setting up the home bridge system I wanted to learn what was going on. I'll look into the heartrate option.
Thanks once again.


Stop Retrying contract calls with failing requires?

When a require inside a contract call fails, should we even retry it? Otherwise we may end up in an infinite loop, see
b395fbc5667913050eadf2dd725ac5cc299620e0 and issue #87

We should do that. Here's another case of an infinite loop:
May 26 09:10:40 shutter-demo-0 shuttermint[454]: runenv.go:210: Retrying action id=1585, => keyper slasher: appeal for half step 6744; err=failed to estimate gas needed: execution reverted: KeyperSlasher: wrong signer

These problems are caused by other bugs in our code. In a bug-free version of our software we wouldn't try call contracts with bad arguments causing requires to revert the call.

quickfixes for the demo system in:  30ad92efa4303b6f37ab9, 53d2133140db7bd85fcfaacc, b395fbc5667913050eadf2dd725ac


chore: support parameters api for mysql

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

Project coverage is 0.00%. Comparing base (e97d07c) to head (dba8957).

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1315   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files         69      69
  Lines       6685    6685
=====================================
  Misses      6685    6685

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


Mob spawn probabilities broken

Minecraft Version
1.20.x
Mod Loader Version
0.14.21
Mod Version
1.0.1
Describe the Issue
After setting the probability of all difficulty mob spawns to 1.0, none of the mobs spawn.
Relevant code snippet from the config:
    "minecraft:green_wool": {
      "spawns": [
        "minecraft:skeleton"
      ],
      "chance": {
        "easy": 1.0,
        "normal": 1.0,
        "hard": 1.0
      },
      "replace": "minecraft:air"
    }

latest.log
latest.log
Do you use any other mods except the required ones?
No response

Thanks for reporting, fixed with https://github.com/Globox1997/DungeonZ/commit/e6e74654211072430364e10d519778d354ec7d99


Write deployment scripts

Ansible seems to be nice solution. Doesn't require any kind of agent on the remote machine. Pure ssh.

https://github.com/data-streams/data-streams/commit/bea3a272a61e10f17a17607b82db62ceb5d85081


refactor: changed the way messages holds files

This PR changes the way that messages hold files. Rather than there being 3 vectors of strings for: name, content and mimetypes, there is now a vector of message_file_data structs.
This doesn't break the way you add files, but rather breaks the way you read files you've added to a message.
Code change checklist

[ ] I have ensured that all methods and functions are fully documented using doxygen style comments.
[x] My code follows the coding style guide.
[ ] I tested that my change works before raising the PR.
[ ] I have ensured that I did not break any existing API calls.
[x] I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

This is proof of it still working!

Reverted to draft as more changes are required.

No longer in draft, ready for review!


UIMenuController with copy action

This pull request provides the ability to copy the value from the selected token.
Full description:
When the user long presses the token a UIMenuController is shown with one action - copy. You can specify what exactly would be copied using the new method from the VENTokenFieldDataSource protocol. It is useful, for example, when your tokens represent contacts email-addresses, but the actual text is the contact's username.
Besides it, as we added the UILongPressGestureRecognizer to the VENToken, I decided to give the user opportunity to add its own longPressBlock, if for some reasons he doesn't want to use the menu controller functionality.
And, of course, I've updated the README file.

@ayanonagon, Hi, can you have a look at this PR? :)

Thank you for the PR! This is cool. Do you think the delegate method might be more clearly named as tokenField:copyStringForTokenAtIndex:? Also if the delegate does not implement that method, a sensible default might be the title of the token. Let me know what you think. :octocat:

@ayanonagon Hi! Thanks for watching my PR. I've already tried to name this method as copyString:, but there was a problem with VENToken interface - it cannot have a copyString property, because copy a reserved system prefix.
And yes, it's a good idea to use the title as the default value - I'll add this functionality.

@ayanonagon I've added the default value for copying and renamed all methods and properties from underlyingString to copyString. Have a look please.

Hi, sorry about the delay. Finally getting to catch up on some PR’s. I just tried this out and got stuck in an infinite loop.
Thanks for the PR! :octocat:


pacemaker_ra/VirtualDomain: add support of sync_config_on_stop

This patch adds the support of the "sync_config_on_stop parameter" when "seapath" parameter is set to "true".
When the "sync_config_on_stop" parameter is set to "true", the domain configuration is saved to the rbd image metadata when the resource is stopped.
The sync config validates the XML configuration and requires libxml2-utils packages installed by this PR: https://github.com/seapath/build_debian_iso/pull/117.

I guess it does not hurt to make this option work with ceph rbd if someone really wants it, but I for one would never use this.
Dumpxml dumps a lot of data (all the hidden defaults, all the pci allocations, etc.) that I don't want to see in the VM xml. I like to have this xml clean and readable, easier to amend.
Seapath comes with the edit_metadata.sh tool where you can easily change the VM configuration and restart the VM to apply the change.


Kadira Successfully connected but no data

I don't get any data on apm server after connecting my application to the server.
The apm application page still display settings instructions even after I get the message "Kadira: Successfully connected" and  "Kadira: completed instrumenting the app" with the keyid and keysecret.
I don't have a firewall blocking connection.

Do you have any client side errors?

Please check if you have the nginx issue described in 2. https://github.com/lmachens/meteor-apm-server#configuration-details
If the port shows 80 this might be the issue.
Did you setup the server with mup?

Ok that's it, the MUP hooks may be failing because I see port 80. Yes the server has been setup with MUP. What should I try to resolve this issue?

Take a look at https://github.com/lmachens/meteor-apm-server/issues/14#issuecomment-360994087
Looks like the mup hook didn't work for you https://github.com/lmachens/meteor-apm-server/blob/master/mup-placeholder.js#L57
@jehartzog did you notice issues with your solution?

It works fine for me, but I freely admin that fix is hacky and probably won't work in all cases. For example, it relies on the apm-engine app being listed first in the proxy file, which may not always happen.
If it doesn't work @funkyremi you'll need to manually edit the port in the /etc/nginx/conf.d/default.conf file in the docker container.

Thank you it's working after manually editing the default.conf file and restarting the service.

I am also having trouble with this default.conf file. The "solution" with manually editing default.conf is somehow automatically reverted after a while (around 8 hours). How can I find what is causing this? I manually edit the file and then run npx mup deploy to make it work for a while. I also tried docker restart mup-nginx-proxy at my apm server but that would instantly revert my changes and use port 80 again.

If the server restarts then the proxy settings are reset, same with restarting the proxy. Sounds like your server is probably restarting every 8 hours for some reason.

I have an uptime that says otherwise...
Am I missing some detail in how docker container works? I make manual changes outside the container then copying it to a default2.conf using
docker cp default.conf mup-nginx-proxy:/etc/nginx/conf.d/default.conf2

followed by
docker exec -t mup-nginx-proxy cp /etc/nginx/conf.d/default.conf2 /etc/nginx/conf.d/default.conf

I don't see anything wrong with how you're doing it, but I just directly edited using vi or vim, which I had to install in the container.
Other things which will reset the proxy config are running a mup deploy again, even if you server is up and running, so if you are pushing code updates that could also be causing the reset.

Made a lazy attempt to install vim in the container but it didn't work first try so I did it like this. :)
After manually editing the file, should I just restart the container mup-nginx-proxy? That makes the most sense to me but it didn't really work.

I think we need a better solution here. If the nginx container restarts and overwrites the config, this error will happen again.
Maybe we can use the mup advanced proxy settings?
// This config can add additional upstream and server blocks if needed.
nginxConfig: './path/to/config',

Or use env vars for nginx proxy to change the nginx config?
// Environment variables for nginx proxy
env: {
 DEFAULT_HOST: 'foo.bar.com'
}

I think the issue is that we expose different ports for different domains.
There is a discussion on https://github.com/jwilder/nginx-proxy/issues/59.
Ideas/PRs are welcome :).

I spent a decent amount of time looking into this, and there was not a simple solution, at least many months ago. That was before MUP released it's nginx config features, so it may be doable now. I'll take another look into it, no promises though :D.

Hello there,
I just installed successfully the APM, but got this issue.
I fixed manually using these steps :

docker ps to list dockers
docker exec -ti CONTAINER_ID /bin/bash to enter the docker instance named mup-nginx-proxy
manually edit /etc/nginx/conf.d/default.conf to change port 80 by port 11011 (I also had to install vi on the docker)
nginx -t to test config, then service nginx reload
I am pretty sure this fix won't stay after a new deployment using mup...

@vparpoil see discussion in https://github.com/lmachens/meteor-apm-server/issues/14


xdstp: avoiding encoding of : in the resource name's path

Commit Message: xdstp: avoiding encoding of ":" in the resource name's path
Additional Description:
xDS-TP resource names should be encoded according to are encoded to RFC-3986.
There's a current bug where ":" is being encoded to "%3A", but it shouldn't.
This PR fixes the issue.
Risk Level: low - control-plane only change
Testing: Added unit-tests.
Docs Changes: N/A
Release Notes: Added
Platform Specific Features: N/A
Optional Runtime guard: Added runtime guard envoy.reloadable_features.xdstp_path_avoid_colon_encoding to temporarily disable this fix.

https://www.rfc-editor.org/rfc/rfc3986#section-2.2
      reserved    = gen-delims / sub-delims

      gen-delims  = ":" / "/" / "?" / "#" / "[" / "]" / "@"

      sub-delims  = "!" / "$" / "&" / "'" / "(" / ")"
                  / "*" / "+" / "," / ";" / "="

Seems that here are some other reserved characters, like '@', should we also added it to the reserved chars list?

https://www.rfc-editor.org/rfc/rfc3986#section-2.2
      reserved    = gen-delims / sub-delims

      gen-delims  = ":" / "/" / "?" / "#" / "[" / "]" / "@"

      sub-delims  = "!" / "$" / "&" / "'" / "(" / ")"
                  / "*" / "+" / "," / ";" / "="

Seems that here are some other reserved characters, like '@', should we also added it to the reserved chars list?

It's a bit more complicated than that (https://www.rfc-editor.org/rfc/rfc3986#section-3.3):
pchar         = unreserved / pct-encoded / sub-delims / ":" / "@"

This PR is to fix the current bug, but I think we may want to eventually use some generic code that validates this and we trust (maybe shared with UHV).


Make intersect and cover into methods

r? @CAD97

With this PR applied, I indeed find text-size perfect and 1.0.0 worthy :)

bors r+


Segmentation fault when try to start a node

Describe the bug
I use the official build_chain.sh file to construct a group of 4 nodes. The shell download the binary file of fisco bcos from: https://github.com/FISCO-BCOS/FISCO-BCOS/releases/download/v3.0.0-rc1/fisco-bcos-linux-x86_64.tar.gz
I first ran the start_all.sh file to start this group and produced some data. After that, I use the latest code (commit hash: 1f0dd3e9de3984db26eee37d4183bedfff8e408b) to compile a new binary. However, the third node cannot start successfully.

To Reproduce
Steps to reproduce the behavior:

Use 3.0-rc1 binary to start the chain
replace the binary with the rc2 one.
restart the group's nodes
See error

Expected behavior
The node should start successfully
Screenshots
Screenshots when using gdb to debug:

Environment (please complete the following information):

OS: Ubuntu 20.04
FISCO BCOS release-3.0.0-rc2 (commit hash: 1f0dd3e9de3984db26eee37d4183bedfff8e408b)

The previous implementation method was to construct a temporary lock first, then std::move the temporary lock as a shared_ptr lock, and pass the lock into the lamda expression.
When the lock is reconstructed, it will be thread insecure, causing the same block to enter the execution twice.
In the process, it is now changed to directly construct the lock of shared_ptr

This problem is finally located when the scheduler is initialized, and the executor in the executorManager is empty. If the synchronization and consensus modules execute the block before the executorManager addExecutor, the block will not be executed while the block has already been put into the cache,  and the corresponding callback will not be called.
When the block is executed next time, it will hit the cache and return an empty execution result, causing the node to coredump,


Bug Report:

1. Current bug behavior
when sending Local from Flutter App for example (en_US) to open an account
We get error
✅ Body:
{"error":"Validation failed: Locale is not included in the list","details":{"locale":[{"error":"ERR_INCLUSION","description":"is not included in the list"}]}}
2. Expected behavior
Should open the account, we have 'en' as default in Mastodon server we tried to use 'en_US' in env file, still getting the same error.
3. Steps to reproduce
Call create account with any local en, en_US de, de_De ...   - still getting the error
4. Flutter doctor output
Flutter outpu
I/flutter (15297): Country.unitedStates
I/flutter (15297): Language.english
I/flutter (15297): en_US
[log] An error occurred: MastodonException: Required parameter is missing or improperly formatted.
✅ Status Code:
422
✅ Request:
POST https://mastodon-dev.connectto.com/api/v1/accounts?
✅ Headers:
{connection: keep-alive, cache-control: private, no-store, transfer-encoding: chunked, date: Thu, 26 Oct 2023 21:40:26 GMT, x-ratelimit-limit: 5, x-ratelimit-reset: 2023-10-26T22:00:00.122426Z, vary: Authorization, Origin, strict-transport-security: max-age=63072000; includeSubDomains, referrer-policy: same-origin, x-ratelimit-remaining: 4, content-type: application/json; charset=utf-8, x-xss-protection: 0, server: Mastodon, x-request-id: cfde03da-4b9b-4bf5-9a98-86781f3d3bb1, x-runtime: 0.288634, x-frame-options: DENY, x-content-type-options: nosniff, content-security-policy: default-src 'none'; frame-ancestors 'none'; form-action 'none'}
✅ Body:
{"error":"Validation failed: Locale is not included in the list","details":{"locale":[{"error":"ERR_INCLUSION","description":"is not included in the list"}]}}
Please create an Issue if you have a question or suggestion for this exception.
https://github.com/mastodon-dart/mastodon-api/issues
Output of: flutter doctor -v

√] Flutter (Channel beta, 3.16.0-0.3.pre, on Microsoft Windows [Version 10.0.19041.630], locale en-US)
    • Flutter version 3.16.0-0.3.pre on channel beta at C:\xxx\projects\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 476aa717cd (7 days ago), 2023-10-19 20:10:52 -0500
    • Engine revision 91cde06da0
    • Dart version 3.2.0 (build 3.2.0-210.3.beta)
    • DevTools version 2.28.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\xxx\AppData\Local\Android\Sdk
    • Platform android-33, build-tools 34.0.0
    • ANDROID_HOME = C:\Users\xxx\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio4\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.4)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.7.34031.279
    • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2021.3)
    • Android Studio at C:\Program Files\Android\Android Studio2
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[√] Android Studio (version 2022.3)
    • Android Studio at C:\Program Files\Android\Android Studio4
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[√] IntelliJ IDEA Ultimate Edition (version 2018.2)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA 2018.2
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart

[√] IntelliJ IDEA Ultimate Edition (version 2021.3)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA 2021.3.2
    • Flutter plugin version 63.2.4
    • Dart plugin version 213.5744.122

[√] VS Code (version 1.75.1)
    • VS Code at C:\Users\xxxx\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.64.0

[√] Connected device (4 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 8.0.0 (API 26) (emulator)
    • Windows (desktop)                  • windows       • windows-x64    • Microsoft Windows [Version 10.0.19041.630]
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 118.0.5993.90
    • Edge (web)                         • edge          • web-javascript • Microsoft Edge 118.0.2088.46

[√] Network resources
    • All expected network resources are available.

• No issues found!
PS C:\Users\xxx>

5. More environment information

6. Log information
I/flutter (15297): Country.unitedStates
I/flutter (15297): Language.english
I/flutter (15297): en_US
[log] An error occurred: MastodonException: Required parameter is missing or improperly formatted.
✅ Status Code:
422
✅ Request:
POST https://mastodon-dev.connectto.com/api/v1/accounts?
✅ Headers:
{connection: keep-alive, cache-control: private, no-store, transfer-encoding: chunked, date: Thu, 26 Oct 2023 21:40:26 GMT, x-ratelimit-limit: 5, x-ratelimit-reset: 2023-10-26T22:00:00.122426Z, vary: Authorization, Origin, strict-transport-security: max-age=63072000; includeSubDomains, referrer-policy: same-origin, x-ratelimit-remaining: 4, content-type: application/json; charset=utf-8, x-xss-protection: 0, server: Mastodon, x-request-id: cfde03da-4b9b-4bf5-9a98-86781f3d3bb1, x-runtime: 0.288634, x-frame-options: DENY, x-content-type-options: nosniff, content-security-policy: default-src 'none'; frame-ancestors 'none'; form-action 'none'}
✅ Body:
{"error":"Validation failed: Locale is not included in the list","details":{"locale":[{"error":"ERR_INCLUSION","description":"is not included in the list"}]}}
Please create an Issue if you have a question or suggestion for this exception.
https://github.com/mastodon-dart/mastodon-api/issues

# 7. More information

We have DEFAULT_LOCALE en in env file  - we tried other languages, but getting errors, we even tried en_US same error

This is how the Locale is created
Locale locale = const Locale(lang: Language.english, country: Country.unitedStates);
This is the function
Future createMastodonAccount({
required String username,
required String email,
required String password,
required bool agreement,
required Locale locale,
String? reason,
}) async {
try {
final response = await mastodon!.v1.accounts.createAccount(
username: username,
email: email,
password: password,
agreement: agreement,
locale: locale,
reason: reason,
);
if (response.status.code == 200) {
  if (kDebugMode) {
    print("Account created successfully");
    log("Token Data: ${response.data}");
    print("Failed to create account");
  }
}

} catch (e) {
if (kDebugMode) {
print(locale.country.toString());
  print(locale.lang.toString());

  print(locale.toString());

  log("An error occurred: $e");
}

}
}
And this is how it is called
await createMastodonAccount(username: 'xxxxr', email<EMAIL_ADDRESS>agreement: true, password: 'xxxx',locale: locale, reason: 'xxxxx');


add domain hust.edu.vn for HUST

a

@4l0ne4ever I am sorry, the domain sis.hust.edu.vn is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


It is better to use the repository field in Cargo.toml

To allow Crates.io , lib.rs and the Rust Digger to link to it. See the manifest for the explanation.

Thanks for spotting it. Could you please update it also on tuono_lib and tuono_lib_macro crates?

done


Allow user to choose port number for publish/preview.

Is your feature request related to a problem? Please describe.
I have a webserver running yunohost.org and it uses the 8080 port, so publish site will not work for me.
This is reproducible to some extent
0. have a server using port 8080

fresh install of vscodium, no addons
installed dendron
downloaded test vault

Describe the solution you'd like
Publish should skip to an open port like Hugo does, or we should be able to pick a port for it.
Describe alternatives you've considered
N/A (there aren't any other options in my case).

I saw that environment variable ELEV_PORT is used some place, but 8080 is hardcoded at other places.
I tried setting ELEV_PORTand running dev server usingdendron-cli` but it didn't work (At least the message on the terminal says 8080)

Yeah, this is why this is in the feature request stage. We need to be consistent about using that everywhere at which point we can close this ticket 😅

this is now possible in 0.38.0 :)
https://wiki.dendron.so/notes/f2ed8639-a604-4a9d-b76c-41e205fb8713.html#previewport


linux.go missing '0x'

I think the code:
ctx.questionableRe = regexp.MustCompile((?:\[\<[0-9a-f]+\>\])? \? +[a-zA-Z0-9_.]+\+0x[0-9a-f]+/[0-9a-f]+)
should be:
ctx.questionableRe = regexp.MustCompile((?:\[\<[0-9a-f]+\>\])? \? +[a-zA-Z0-9_.]+\+0x[0-9a-f]+/0x[0-9a-f]+)

Please provide a test that fails now.

I can't provider a test, but it's very obvious.
Error Log file seems like:
[ 1457.954795] Call Trace:^M
[ 1457.955561]  dump_stack+0x83/0xb5^M
[ 1457.965551]  ? do_get_mempolicy+0x853/0xc10^M
[ 1457.994055]  print_address_description+0x6b/0x280^M
[ 1457.996476]  ? do_get_mempolicy+0x853/0xc10^M
[ 1458.053833]  kasan_report+0x260/0x340^M
[ 1458.054882]  __asan_report_load2_noabort+0x14/0x20^M
[ 1458.056587]  do_get_mempolicy+0x853/0xc10^M
[ 1458.057913]  ? sp_free+0x60/0x60^M
[ 1458.058961]  SyS_get_mempolicy+0xcf/0x180^M
[ 1458.060259]  ? SyS_migrate_pages+0x500/0x500^M
[ 1458.061519]  ? SyS_mbind+0xd0/0x150^M
[ 1458.078516]  ? compat_SyS_mbind+0x220/0x220^M
[ 1458.080152]  ? fput+0x20/0x150^M
[ 1458.081198]  ? SyS_ioctl+0x66/0xc0^M
[ 1458.082702]  entry_SYSCALL_64_fastpath+0x1a/0xa5^M
you want to exclude the line start with '?'，but the regular expression is
((?:\[\<[0-9a-f]+\>\])? \? +[a-zA-Z0-9_.]+\+0x[0-9a-f]+/[0-9a-f]+), and you did exclude
the line with '?' because ? SyS_ioctl+0x66/0xc0 meets the regular expression.

Why did you close this? Is there still a problem or not?

Eh..., I think it's not a big deal so I close this.


Pre-commit hook should run in --check mode.

I don't agree with this. Why?

Fair enough. I was just trying to make it work like the other pre-commit hooks I've got, but I can do that in my local config files instead :-)


Create default ASCII arts

When we reach the maximum of requests per hour no ASCII is displayed, only a default error message.
Add default ASCIIs to show if they match a string

The best way would be through the API, but we could start with simple constants in the code


calling update-deps.sh fails

get the following error:
warning: ignoring symlink /home/ylifshit/go/src/knative.dev/eventing-contrib/test/test_images/kafka_performance/kodata/refs
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>unrecognized import path "vbom.ml/util": https fetch: Get "https://vbom.ml/util?go-get=1": dial tcp: lookup vbom.ml on <IP_ADDRESS>:53: no such host

It seems like a dns problem.

It seems like a dns problem.

do you mean it is a temporary problem?

Check you dns config. <IP_ADDRESS> is not a public dns server.

I've set the IP statically (changing DNS didn't seem to help), but now I get a cert issue:
<EMAIL_ADDRESS>unrecognized import path "vbom.ml/util": https fetch: Get "https://vbom.ml/util?go-get=1": x509: certificate has expired or is not yet valid: current time 2020-09-16T10:49:51+03:00 is after 2020-01-03T23:59:59Z

I've set the dns address to a public one, but didn't seem to help:
<EMAIL_ADDRESS>unrecognized import path "vbom.ml/util": https fetch: Get "https://vbom.ml/util?go-get=1": dial tcp: lookup vbom.ml on <IP_ADDRESS>:53: no such host

@yuvalif Which Go version are you using?
I cannot reproduce this with go 1.15.2 even if I drop all related go mod cache.
Note that vbom.ml/util has been moved to github.com/fvbommel/util(See the issue 7 in that repo).

go version go1.14.6 linux/amd64

I've just tried on Fedora 32 with the same Go version and it's working.
Can you share the output of the command go env?

$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/ylifshit/.cache/go-build"
GOENV="/home/ylifshit/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/ylifshit/go"
GOPRIVATE=""
GOPROXY="direct"
GOROOT="/usr/lib/golang"
GOSUMDB="off"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/golang/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/ylifshit/go/src/knative.dev/eventing-contrib/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build446643634=/tmp/go-build -gno-record-gcc-switches"

It seems ok.
Are you on the default branch?
/priority awaiting-more-evidence

It seems ok.
Are you on the default branch?

yes, on "master"

/priority awaiting-more-evidence

fwliw, I've hit this as well on the test-infra repo

For those coming here via search, considering this is a dns issue (instead of a eventing-contrib issue) and I've verified others users are hitting this well, I've manually worked around the issue by adding a replace line in go.mod to avoid using the vanity url, ie:
vbom.ml/util => github.com/fvbommel/util v0.0.2
Closing this issue out for now


Gekko is not running on cloud

bug report
I deployed gekko on google cloud today ,however it didn't run on google cloud .giving me the below error

Gekko is running fine, that's a warning. The warning comes from you having set headless to false in the UIConfig. Please see the documentation here: https://gekko.wizb.it/docs/installation/installing_gekko_on_a_server.html
:)


Add result_type as option for google lookup

Hey guys.
Just wondering if it would be possible have the option to return result_type from the google geocoder reverse lookup as seen here.
https://developers.google.com/maps/documentation/geocoding/intro#ReverseGeocoding

Looks good to me. Thanks!


Split out ibc-clients crate

Part of: #965
Description

PR author checklist:

[ ] Added changelog entry, using unclog.
[ ] Added tests.
[ ] Linked to GitHub issue.
[ ] Updated code comments and documentation (e.g., docs/).
[ ] Tagged one reviewer who will be the one responsible for shepherding this PR.

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

Two things before this PR is fully ready for review and merging:

The ibc_core::extra::tendermint::validate_self_client module complains about not being to find the latest_height method on the ClientState type (imported from ibc_client_tendermint_types::client_state::ClientState). This is because the latest_height method is implemented for ClientState as part of the ClientStateCommon trait: this trait implementation exists in the handlers part of the ics07-tendermint crate, not the ics07-tendermint/typessub-crate. How should we go about resolving this? Should theClientStateCommontrait implementation be moved to the types sub-crate? We could also opt to move thelatest_heightfunction out of theClientStateCommontrait and have it exist standalone in thetypes` sub-crate. What's the preferred way to address this?
The ibc-testkit crate's import paths still need to be updated.

Two things before this PR is fully ready for review and merging

Big thanks @seanchen1991 for taking care of the clients migration! 🙌
I made a few more tweaks to the ibc-core PR. Some code shuffled around, and I ditched ibc-core-extra crate, moving its codebase to the other right crates. I've updated your PR as well to sync up with these changes and also tackled issues you pointed out earlier.

fyi: to avoid confusion arising from the varied use of ClientState in different types and implementations, I've updated the naming around. The newly defined newtype ClientState is now named ClientStateWrapper, while the original Tendermint ClientState struct has been preserved as TmClientState wherever it is imported. The term ClientState still refers to the trait, ensuring consistency in naming with the ConsensusState as well. Hope it works.

Merged this PR to have a smoother review of changes within a consolidated branch.


Implement 'SHOW DATABASES' statement

What would you like to be added:
The show databases statement should only list those databases in the same tenant.
Please see the official manual, https://dev.mysql.com/doc/refman/5.7/en/show-databases.html
Why is this needed:
The database in ARANA means 'logical database' instead of 'physical database', we need to handle it.

assignees me,me try learn

Defined as row to return to the caller, database prefix 1 bytes from there take or generate ?

@jjeffcaii for help


subscribe topic lost

server run expire some days,I find conncetion keepalive(look at admin site dashboard),but subscribe topic lost,why?

Invalid issue, closed.


[Bug] extra divs are generated around custom components for node

Describe the bug
I heve use custom component for link node with react:
https://github.com/Saul-Mirone/milkdown/blob/main/gh-pages/pages/integrations/react/index.md#custom-component-for-node
const TSLink: React.FC = ({ children }) => {
    const { node } = useNodeCtx();

    const clickLink = (evt: any) => {
      evt.preventDefault();

      window.parent.postMessage(
        JSON.stringify({
          command: 'openLinkExternally',
          link: node.attrs.href,
        }),
        '*'
      );
    };
    return (

        {children}

    );
  };

const editor = useEditor(
    (root, renderReact) => {
      const nodes = commonmark
        .configure(paragraph, { view: renderReact(TSParagraph) })
        .configure(link, { view: renderReact(TSLink) })

      return (
        Editor.make()
          .config((ctx) => {
            context = ctx;
            ctx.set(rootCtx, root);
          })
          .use(nord)
          .use(nodes)
          .use(commonmark)
      );
    }
  );

but I see in DOM its have extra
 around link:
My markdown is: [github](http://github.com) after link inline text

github

 after link inline text

as the result " after link inline text" is displayed on the next line..
I have try to fix this and override paragraph component with inline-flex style
 const TSParagraph: React.FC = ({ children }) => (

{children}

  );

but there is extra
 generated around paragraph node too:

github

 after link inline text

I think additional divs can't avoid due to the limitation of prosemirror's node view feature, however, I certainly can do some optimize to wrap the inline nodes with span instead of div and add some class name to make it easier to add style.

Hey, I'm facing the same issue, and also when I'm trying to configure a custom link with
.configure(link, { view: renderReact(CustomLink) })

I get this errror:
Type 'NodeViewFactory' is not assignable to type 'MarkViewFactory'

Do you know how I can fix it properly? :)
Thanks for your help!

Hey, I'm facing the same issue, and also when I'm trying to configure a custom link with
.configure(link, { view: renderReact(CustomLink) })

I get this errror:
Type 'NodeViewFactory' is not assignable to type 'MarkViewFactory'

Do you know how I can fix it properly? :) Thanks for your help!

It's a bug and I've already fixed it in main branch, will be released in this week.

That's great, thanks for your quick answer! 🙏
Also, do you know when the fix for generated divs around custom components will be available?

That's great, thanks for your quick answer! 🙏
Also, do you know when the fix for generated divs around custom components will be available?

They are all done in main branch now. Div are still there, however, it will be optimized as mentioned above.

Released in v4.13


upgrade Go from 1.18 to 1.20

Description
Quick checks:

[ ] I have followed the Code Guidelines.
[ ] There is no other pull request for the same update/change.
[ ] I have written unit tests.
[ ] I have made sure that the PR is of reasonable size and can be easily reviewed.

Done in #41.


Vanilla Extract files built in dev, but error for prod build

Describe the bug
A component library that consumes VE styles builds correctly in dev, but errors in dev.
Example repo: https://github.com/brandonpittman/ve-mitosis-library
The repo has vars and sprinkles for VE. The components are written with Builder's Mitosis library and compiled to Qwik and React components. The repo has a docs page that can be built with npm run dev. When building the docs page for prod with npm run build, I get the css.ts context error.
Reproduction
https://github.com/brandonpittman/ve-mitosis-library
System Info
System:
    OS: macOS 14.0
    CPU: (10) arm64 Apple M1 Pro
    Memory: 114.67 MB / 16.00 GB
    Shell: 3.6.1 - /opt/homebrew/bin/fish
  Binaries:
    Node: 18.18.0 - ~/Library/pnpm/node
    Yarn: 1.22.19 - ~/Library/pnpm/yarn
    npm: 9.8.1 - ~/Library/pnpm/npm
    pnpm: 7.9.5 - ~/Library/pnpm/pnpm
  Browsers:
    Chrome: 119.0.6045.123
    Firefox Nightly: 107.0a1
    Safari: 17.0

Used Package Manager
npm
Logs
error during build:
Error: Styles were unable to be assigned to a file. This is generally caused by one of the following:

- You may have created styles outside of a '.css.ts' context
- You may have incorrect configuration. See https://vanilla-extract.style/documentation/getting-started
    at Object.getFileScope (/Users/blp/Developer/pneuma-share/node_modules/@vanilla-extract/css/fileScope/dist/vanilla-extract-css-fileScope.cjs.prod.js:35:11)
    at createGlobalTheme (/Users/blp/Developer/pneuma-share/node_modules/@vanilla-extract/css/dist/vanilla-extract-css.cjs.prod.js:319:48)
    at file:///Users/blp/Developer/pneuma-share/packages/docs/server/assets/entry.ssr-7f894bdb.js:262:22084
    at ModuleJob.run (node:internal/modules/esm/module_job:194:25)

Validations

[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

Getting some errors when trying to run npm run dev in the docs package. I think you need to declare some more dependencies on your scoped packages.

Getting some errors when trying to run npm run dev in the docs package. I think you need to declare some more dependencies on your scoped packages.

@askoufis What's the error? I'm not seeing any.

I ran:

npm install
cd packages/docs
nr dev

Seeing this:
Error:   Failed to scan for dependencies from entries:
  /Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/packages/docs/src/entry.ssr.tsx

  ✘ [ERROR] Failed to resolve entry for package "@scoped-example/qwik". The package may have incorrect main/module/exports specified in its package.json. [plugin vite:dep-scan]

    ../../node_modules/vite/node_modules/esbuild/lib/main.js:1373:21:
      1373 │         let result = await callback({
           ╵                      ^

    at packageEntryFailure (file:///Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:28725:11)
    at resolvePackageEntry (file:///Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:28722:5)
    at tryNodeResolve (file:///Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:28453:20)
    at Context.resolveId (file:///Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:28212:28)
    at Object.resolveId (file:///Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:44276:64)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async resolve (file:///Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:44581:26)
    at async file:///Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:44758:34
    at async requestCallbacks.on-resolve (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:1373:22)
    at async handleRequest (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:729:13)

  This error came from the "onResolve" callback registered here:

    ../../node_modules/vite/node_modules/esbuild/lib/main.js:1292:20:
      1292 │       let promise = setup({
           ╵                     ^

    at setup (file:///Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:44748:19)
    at handlePlugins (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:1292:21)
    at buildOrContextImpl (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:978:5)
    at Object.buildOrContext (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:786:5)
    at /Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:2186:68
    at new Promise (<anonymous>)
    at Object.context (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:2186:27)
    at Object.context (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:2026:58)
    at prepareEsbuildScanner (file:///Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:44531:26)

  The plugin "vite:dep-scan" was triggered by this import

    src/root.tsx:8:23:
      8 │ import { Button } from "@scoped-example/qwik";
        ╵                        ~~~~~~~~~~~~~~~~~~~~~~

    at failureErrorWithLog (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:1649:15)
    at /Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:1058:25
    at runOnEndCallbacks (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:1484:45)
    at buildResponseToResult (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:1056:7)
    at /Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:1068:9
    at new Promise (<anonymous>)
    at requestCallbacks.on-end (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:1067:54)
    at handleRequest (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:729:19)
    at handleIncomingPacket (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:755:7)
    at Socket.readFromStdout (/Users/askoufis/code/vanilla-extract-css/issues/1226/ve-mitosis-library/node_modules/vite/node_modules/esbuild/lib/main.js:679:7)

@askoufis
Maybe it was because I didn't have the patch-package postinstall script in there. Just pushed a change.
VE doesn't recognize qwik.mjs files, which Qwik requires. I patched the dev CSS filter for VE to accept css.qwik.mjs files. It's in the repo now. Should build.

That doesn't seem to have fixed it. Vite is complaining about not being able to scan for dependencies because your docs packages hasn't declared a dependency on @scoped-example/qwik yet it uses it in src/root.tsx.

@askoufis Can you tell me how to declare a dependency for a workspace inside a project? I Googled for something like that, but didn't find it. Thought it would just work since it's a workspace declared in the root.
I'm not seeing that error when I run npm run dev either. 😢

You can just add the package as a dependency. NPM should just figure out that you're referencing a workspace package and prefer that instead of fetching it from the registry.

@askoufis Pushed a change. No difference in my output.

The dev server at least starts now, but getting a new error:
Failed to resolve entry for package "@scoped-example/vanilla-extract". The package may have incorrect main/module/exports specified in its package.json.

@askoufis Added explicit exports declaration for the ve package.
I wonder why this all just worked for me (without explicit declaration) but not for you. 🤔

Still getting the same error. Really struggling to understand why it's working for you. Do you have some packages installed globally? Have you started with a clean node_modules? Could you try deleting a re-cloning the repo?

@askoufis
Deleted the repo, cloned it, and the same errors you saw started. Will fix and push. Thanks for your patience.

@askoufis
Ah! The exports key is referencing the dist build files, so without an initial build, the exports files aren't there.

After some debugging, it looks like the issue is those .qwik. files. Even though you've patched VE, there's probably some more work needed to support those files. That being said, I'm not sure if it should be on Vanilla Extract to support those files in the first place, but instead something that mitosis needs to handle. Qwik itself supports vanilla extract, but I guess this component meta-framework doesn't.
Dev mode likely works because it's not depending on built files, instead using source code directly.

I don't see how it can be Mitosis. Mitosis just spits out JSX files and it runs as a CLI command and not part of Vite's build process.

I don't see how it can be Mitosis. Mitosis just spits out JSX files and it runs as a CLI command and not part of Vite's build process.

But your docs app references the Button component from your ui package, which means it loads atoms.css.ts from your vanilla-extract package, finally resulting in a atoms.css.qwik.mjs file, which eventually calls globalStyle, which throws the error.

I don't see how it can be Mitosis. Mitosis just spits out JSX files and it runs as a CLI command and not part of Vite's build process.

But your docs app references the Button component from your ui package, which means it loads atoms.css.ts from your vanilla-extract package, finally resulting in a atoms.css.qwik.mjs file, which eventually calls globalStyle, which throws the error.

@askoufis and because I patched that, it doesn’t throw in dev? That makes sense.
When it errors in the prod build, it happens in a entry.server-xxx.js file with a different pattern. I patched that pattern into the prod filter but it had no effect.

@askoufis
Maybe Qwik is just weird. If I could find a React component library built on VE and Vite I think I could figure a way to make Qwik work.

@askoufis Maybe I have a fundamental misunderstanding of what's possible…

My hope was that I could export components that called functions that the consuming app's VE would pick up. I'm running into all kinds of problems.
If I build the Qwik library using the Vite VE plugin, I get components with baked-in class names and a bundle CSS file that can be imported in a consuming app.

Is #1 just not a possibility and #2 is the only way to build a component library with VE?

1 should be possible. The idea being that you don't do anything to your css.ts files other than convert them to css.js/cjs/mjs files. Mitosis is adding its own extension, which is IMO the cause of the issue.
2 is also viable, though ideally you'd enable something like preserveModules so that each component ends up with its own CSS file that it imports.
I've seen both methods done before. The difference here seems to be mitosis. Since you're generating both qwik and react components, my guess is that the react components will work just fine.

@askoufis Thanks for all your time. We seem to be going in circles so I'm going to close this issue.

So…by making @scoped-example/vanilla-extract external in the Qwik vite.config.ts, all the Qwik stuff works—and I was able to remove the patch!
Still having trouble importing the components into a Remix project though. Getting a .css.ts error on vars.css.ts?used in the dev server for Remix. Will debug a little more, but all the Qwik stuff is working. 😄

If I re-export my sprinkles function from a local sprinkles.css.ts file, things work, but importing directly from the package creates the above error as well.

Working in non-Remix React app. Must be something funny about Remix.


Added "grpc" to pkg-config --libs

Added "grpc" to the list of pkg-config libraries needed
Simplified part of the makefile

@markdroth

You'll need to sign the CLA before we can look at this.

I went in to see the CLA and it only provides one option: "Select this if
you are contributing code as an employee", which is not the case.
-brian
On Wed, Mar 25, 2020 at 2:46 PM Mark D. Roth<EMAIL_ADDRESS>wrote:

You'll need to sign the CLA before we can look at this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/22478#issuecomment-604078131, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACG4TLR3UBNPRYR4ORMXRVDRJJUQZANCNFSM4LTXUBWQ
.

@Phantal  Hi I'm Heather with The Linux Foundation. I wasn't able to find an icla on file through the EasyCLA system. For this pull request to be authorized, you'll need to click the link in the communitybridge-easycla bot comment that says "Please click here to be authorized". You'll choose "individual" and then you'll be taken to the document to sign via Docusign. Once that is complete, your pr will be authorized.

@WillsonHG When I go there it only offers an option to sign as a corporate contributor, not as an individual contributor.

@Phantal  that sounds like a UI issue that is not allowing you to scroll down to see the individual option beneath the corporate one. Are you able to try in a different browser?

@WillsonHG Looks like that's sort of right.  In chrome I cannot see it at all; on another computer I pulled it up in Edge and the individual option is partially visible, covered by the blue banner at the bottom of the page.  I'll get it sorted.

@Phantal I'm glad you were able to get it working. We'll get that layout fixed, thanks!

I forgot to explain why the change was needed.  When trying to link the application it fails with an undefined symbol reference to gpr_log, which is satisfied by libgrpc.so

Nico, can you please review to make sure these changes seem sensible?  Thanks!

So where are we with this pull request?


[Swift 4.2] Update All-Pairs Shortest Paths to Swift 4.2

Checklist

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

Description

Update All-Pairs Shortest Paths to Swift 4.2

Thanks for the update 💯


LimeSuiteNG on arm does not recognize LimeSDR Mini

Using a Raspberry Pi 4 and the latest (fully updated) version of Bookworm Lite.   I know that many changes have been implemented over this period!
After this LimeSuiteNG update, LimeSDR Minis (version 1 or version 2) are no longer recognised by LimeSuiteNG, but are recognised by LimeSuite.
pi@raspberrypi:~ $ lsusb
Bus 002 Device 002: ID 0403:601f Future Technology Devices International, Ltd FT601 32-bit FIFO IC
Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
Bus 001 Device 002: ID 2109:3431 VIA Labs, Inc. Hub
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
pi@raspberrypi:~ $ limeDevice --full
No devices found
pi@raspberrypi:~ $ LimeUtil --make
Make device
  Device name: LimeSDR-Mini
  Expansion name: UNSUPPORTED
  Firmware version: 6
  Hardware version: 2
  Protocol version: 1
  Gateware version: 1
  Gateware revision: 30
  Gateware target: LimeSDR-Mini
  Serial number: 0x1d4c2cae8c28a4
  Free connection... OK

LimeSDR Minis were recognised by commit https://github.com/myriadrf/LimeSuiteNG/commit/626a781f2f1d20e25b3e731f3fe8fb350a7adc97.
Of course, calls to LMS_GetDeviceList(NULL) return 0 devices making LimeSuiteNG unusable with a LimeSDR Mini.
Thanks
Dave, G8GKQ

Works fine for me on RPi4, was libusb-1.0 dev installed when you compiled LimeSuiteNG?

Thanks Ricardas
Yes - all looks OK:
-- Found Threads: TRUE
-- Found libusb-1.0:
--  - LIBUSB_1_INCLUDE_DIRS: /usr/include/libusb-1.0
--  - LIBUSB_1_LIBRARIES: /usr/lib/aarch64-linux-gnu/libusb-1.0.so
-- Could NOT find Gnuplot (missing: GNUPLOT_EXECUTABLE)
-- Found wxWidgets: -L/usr/lib/aarch64-linux-gnu;-pthread;;;-lwx_gtk3u_richtext-3.2;-lwx_gtk3u_core-3.2;-lwx_baseu-3.2;-lwx_gtk3u_gl-3.2 (found version "3.2.2")
-- wxWidgets_DEFINITIONS: _FILE_OFFSET_BITS=64;WXUSINGDLL;__WXGTK__
-- wxWidgets_INCLUDE_DIRS: /usr/lib/aarch64-linux-gnu/wx/include/gtk3-unicode-3.2;/usr/include/wx-3.2
-- wxWidgets_LIBRARIES: -L/usr/lib/aarch64-linux-gnu;-pthread;;;-lwx_gtk3u_richtext-3.2;-lwx_gtk3u_core-3.2;-lwx_baseu-3.2;-lwx_gtk3u_gl-3.2
-- Found OpenGL: /usr/lib/aarch64-linux-gnu/libOpenGL.so
-- OPENGL_INCLUDE_DIR: /usr/include
-- OPENGL_LIBRARIES: /usr/lib/aarch64-linux-gnu/libOpenGL.so;/usr/lib/aarch64-linux-gnu/libGLX.so;/usr/lib/aarch64-linux-gnu/libGLU.so
-- Enabling native optimizations
-- Found Git: /usr/bin/git (found version "2.39.2")
-- Module Soapy_limesuiteng configured with version: 0.2.0-22f2b2e8
-- Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE)
--
-- ######################################################
-- Packages found:
 * Threads
 * libusb-1.0
   Adds USB communication support
 * wxWidgets
 * OpenGL
 * SoapySDR
   Adds LimeSuiteNG bindings for SoapySDR
 * Git

-- ######################################################
-- Missing required packages:
 * LinuxKernelHeaders

-- ######################################################
-- Missing recommended packages:

-- ######################################################
-- Missing optional packages:
 * Gnuplot
 * Doxygen

-- ######################################################
-- ## LimeSuiteNG enabled features
-- ######################################################
--
 * HEADERS, The limesuiteng headers
 * LIBRARY, The limesuiteng library
 * USB_FTDI, USB support for FTDI
 * USB_FX3, USB support for Cypress FX3
 * LIMEPCIE, PCIe support
 * LIMEPCIE_KERNEL_MODULE, Build Linux LimePCIe kernel module
 * EXAMPLES, LimeSuite library API examples
 * CLI, LimeSuite command line interface tools
 * GUI, GUI Application for LimeSuite
 * AMARISOFT_PLUGIN, LimeSuite Amarisoft integration plugin
 * DESKTOP, LimeSuiteNG freedesktop integration
 * SOAPYSDR, SoapySDR bindings for LimeSuiteNG
 * UDEV_RULES, Install Linux udev rules

-- ######################################################
-- ## LimeSuiteNG disabled features
-- ######################################################
--
 * DKMS, DKMS support for lime PCIe kernel module
 * GNURADIO, GNU Radio 3.9+ Plug-in
 * DOXYGEN, Build Doxygen documentation

-- Install prefix: /usr/local
-- Build timestamp: 2024-08-06
-- Lime Suite version: 0.2.0-g22f2b2e8
-- ABI/so version: 0.2-1
-- Configuring done
-- Generating done
-- Build files have been written to: /home/pi/LimeSuiteNG/build

Although I did get this warning:
-- Enabling native optimizations
CMake Warning (dev) at /usr/share/cmake-3.25/Modules/FetchContent.cmake:1279 (message):
  The DOWNLOAD_EXTRACT_TIMESTAMP option was not given and policy CMP0135 is
  not set.  The policy's OLD behavior will be used.  When using a URL
  download, the timestamps of extracted files should preferably be that of
  the time of extraction, otherwise code that depends on the extracted
  contents might not be rebuilt if the URL changes.  The OLD behavior
  preserves the timestamps from the archive instead, but this is usually not
  what you want.  Update your project to the NEW behavior or specify the
  DOWNLOAD_EXTRACT_TIMESTAMP option with a value of true to avoid this
  robustness issue.
Call Stack (most recent call first):
  external/CMakeLists.txt:11 (fetchcontent_declare)
This warning is for project developers.  Use -Wno-dev to suppress it.

Dave

That warning is due to CMake version, you can ignore it.
Just to make sure try to run limeDevice from the build/bin directory, as the global installed binary might be from different version.

This issue is reproducable.
Build a fresh install of raspios bookworm arm64 lite, and install the LimeSuiteNG dependencies.  Then git clone the latest version of LimeSuiteNG, cmake, make and install.  Connect a LimeSDR Mini; limeDevice --full will then return "No devices found".
If you then delete the LimeSuiteNG folder, git clone the latest version of LimeSuite and then checkout an older build (I used commit 5afb2a3 ) and cmake, make and install.  limeDevice --full then sees the LimeSDR Mini.
If you delete the LimeSuiteNG folder again, git clone the latest version of LimeSuiteNG, cmake, make and install.  Connect a LimeSDR Mini; limeDevice --full will then return "No devices found".
Next, again delete the LimeSuiteNG folder, git clone the latest version of LimeSuite and then checkout the older build (5afb2a3) and cmake, make and install.  limeDevice --full then sees the LimeSDR Mini.
Next, DO NOT delete the LimeSuiteNG folder, but just checkout the latest commit (I used 770847c) and then cmake, make and install.   limeDevice --full then sees the LimeSDR Mini.
So it appears that there is something in the older builds of LimeSuiteNG that is required to still be in place for the newer builds of LimeSuiteNG to work correctly.  I hope that gives you enough to reproduce the issue and track it down.
I had been deleting old builds of LimeSuiteNG on each update, which is why I get the issue and you had not seen it.
Dave


julia-client boot script fails because Juno.jl does no precompile

Atom 1.10.2, latest releases of uber-juno, julia 0.5.0.
Upon startup I've got
INFO: Precompiling module Atom.
WARNING: Module Juno with uuid<PHONE_NUMBER>263 is missing from the cache.
This may mean module Juno does not support precompilation but is imported by a module that does.
ERROR: LoadError: Declaring __precompile__(false) is not allowed in files that are being precompiled.
 in macro expansion; at ./none:2 [inlined]
 in anonymous at ./<missing>:?
while loading /home/astukalov/.julia/v0.5/Atom/src/Atom.jl, in expression starting on line 5
ERROR: LoadError: Failed to precompile Atom to /home/astukalov/.julia/lib/v0.5/Atom.ji.
 in require(::Symbol) at /home/astukalov/.julia/v0.5/Requires/src/require.jl:12
while loading /home/astukalov/.atom/packages/julia-client/script/boot.jl, in expression starting on line 36

But when I've added __precompile__() to src/Juno.jl, the installation succeeded.

Same issue as https://github.com/JunoLab/uber-juno/issues/15.
I don't think we actually want to add __precompile__() to Juno.jl, since other not-precompilable packages might want to require it.

Does it mean that the solution is to disable Atom.jl precompilation? Would it affect Juno startup times?

No, the problem here is spurious and might be related to our startup script, don't know yet.
Doing using Atom once in the REPL seems to fix it for good though.

This is hopefully fixed now.


Able to see categories from another clusters catalog

Version: master 2.2 (1/2/19)
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Have two clusters set up. Add a custom catalog to one of them such as https://github.com/tfiduccia/charts (the catalog must have categories set)
On the second cluster (with no custom catalogs set up) create a project and under that project launch the catalog apps.

Result:
You can't see the catalog apps, BUT you can see the categories from that catalog from the first cluster.
Screenshot showing these two categories we should not be able to see: from the Categories drop-down:

You cannot see the catalog listed under the Catalogs drop-down, so that is good. This appears to only be an issue with categories.
Other details that may be helpful:
This may be related to #17165 and PR #2535

Version: master 2.2 (1/4/19)
Confirmed fix.
I can no longer see catalogs from another cluster where I set scope to just that cluster.
I tried to break this and could not. Halo tested and validated HTTP.


Override address question?

Is it possible when using the override to display the original email intended address that is being overridden? This would make testing considerably easier if the original address was somehow listed but not sent.

@uhuntgx00 I was looking to obtain the same thing. Were you able to display the original recipient?


support for linux aarch64

I have followed the example for Windows and MacOS here and used ~x64
in the version specifier, but we might want to change that to ~aarch64
instead.

I'm updating this branch right now, will push soon.

@simeonschaub I updated the branch and to me it all looks good now. Could you also take another look whether this makes sense to you?

And then once you can confirm that this works, we need to add it to our Github Actions release pipeline as well, of course!

Ok, I can confirm that it still builds just fine on my android phone. GitHub doesn't allow me to approve my own PRs, but feel free to merge whenever.

And then once you can confirm that this works, we need to add it to our Github Actions release pipeline as well, of course!

Have you ever looked into cross compilation with Rust? I must admit I don't have any experience in that regard. I almost wonder whether it might be easiest to just take advantage of the BinaryBuilder infrastructure, I believe it does support Rust as well.

Have you ever looked into cross compilation with Rust?

Yes, and it is very easy and all fully integrated into the existing GitHub Actions. I'll update the workflow.

@simeonschaub could you try the binaries that are now build as part of the release process https://github.com/JuliaLang/juliaup/releases/tag/v1.2.11 ? I also added M1 Apple builds while I was at it.

I noticed that the binaries didn't have the executable bit set, otherwise everything worked great though.


Best practise for multiple apllications

Hello,
we want to use this library in multiple applications. We are using the Pusher subscriber to send our messages to the echo server. This works great. My questions are:
How can we access the channels for an specific app_id?
Can we set an authEndpint for each client?
Is there a best practise to handle multiple applications (host one echo server per apllication for example)

Laravel-echo-server uses 1 application. Namespaces is not (yet) implemented.

Is there a preferred API? I would try to implement it

Not sure, namespaces are part of the SocketIO server; https://github.com/socketio/socket.io/blob/master/docs/API.md#namespace
But this would also need seperation at the API level, which is possible with the Http API, but not with redis so easily I think.

i think you have three options: prefix the Redis key, map a redis databse to an app_id or map a redis instance to an app_id.
What is the current use case for multiple clients? Is it only the api access?
Is an client (appi_id) equal a names or can a client access multiple namespaces?

The current use case is only multiple api clients, but same channels/connections etc. (To map pusher api).
For multiple apps, current approach would be running multiple socketIO instances on different ports/configs.

I did this for staging and production on the same server.
Used different default queue per environment and prefixed each broadcast channel with environment name.
Also used two different echo-servers on two different ports, however I think it's not a must, because users subscribed to different channels and both server get all broadcasted channels anyway.

Same here. I'm running test and production environment at same webservicer, in diferent subdomains. I've tried run 2 instances of laravel-echo-server on port 6001 and 6002, but the events are firing for both.

@rafwell : do you use the same redis server?

@catalinux yes, same server.

Make sure you use a different redis database for each instance.

To be clear, you should probably change the Redis database on your Laravel broadcasting app. Both your echo-servers are listening to the same broadcasting database.

I think you are right. I've stopped my websocket of test env and continue receiving messages at production. you can help me how configure different database for environments? Whiches files i need change?

In my laravel app ive modified the file config/database:
'redis' => [

        'cluster' => false,

        'default' => [
            'host' => env('REDIS_HOST', '<IP_ADDRESS>'),
            'password' => env('REDIS_PASSWORD', ''),
            'port' => env('REDIS_PORT', 6379),
            'database' => env('REDIS_DB_NUM', 0),
        ],

    ],

in my .env ive set a REDIS_DB_NUM for production and other for test env. But the problem persist.
Also, can I use letters like redis database name or just numbers?

Hey guys, I don't think Redis broadcasts to individual databases, as per their docs. I was curious about this as well, as I just started developing a Laravel app with Laravel-Echo-Server. I currently have my dev and prod environments on the same server.
https://redis.io/topics/pubsub
Database & Scoping
Pub/Sub has no relation to the key space. It was made to not interfere with it on any level, including database numbers.
Publishing on db 10, will be heard by a subscriber on db 1.
If you need scoping of some kind, prefix the channels with the name of the environment (test, staging, production, ...).
I think my approach will be to prefix channels using env(environment) in the event's broadcastOn() method. If you guys have both environments working, try that and let me know if that works out. Note, you will need to pass the environment down to your clients when instantiating a WS connection via Laravel Echo, too.

@cdr95985 i think this will work, but is poor security solution, a bad guy will can hear events of another's environment. Think in a shared host with many applications.
I will change my code soon and send the result here.

@rafwell You're right, it doesn't seem like a very good security practice, however, I think that if we use private channels and channel authentication (channels.php) really well, then you might be okay... From face-value, I think the only way a bad guy could get into another environment channel is if their user (or the user they are impersonating) has rights to that channel.
Perhaps you could lock down channels on an environment-level as well, ie) disallow anyone but users with X roles can access Y,Z,A environments. You could wrap all of the channels.php in their own validation system in that regard.
I'm just rambling though. Idk how we could make it more secure.

@cdr95985 after change my code to use channels test.chat.{userID} and prod.chat.{userId} the problem of hear events from another environment has solved. About the security, i will change my code to use private channels.
@catalinux  someone who has a link:port can hear messages from all users on channels (chat.1, chate.2, etc. the number is the id of my user), but now my channel is public, i need change  that to private to see if is possible hear events from other domain who use the same redis server.

Here is my solution,I use defined prefix to solve it.
1st,in dot env file
MIX_BROADCAST_PREFIX=

2nd,echo.js
let prefix = process.env.MIX_BROADCAST_PREFIX || ''
window.Echo.private(`${prefix}channel-private`)

3rd,channels.php
Broadcast::channel(config('site.echo_server_prefix').'channel-private',function($user){
 ......
});

finnaly,in Job file
return new PrivateChannel(config('site.echo_server_prefix').'channel-private');

every thing is ok


feat: support subindex on ExactNNSearch

Currently, the subindex feature is not available on exact in-memory NNIndexer. This PR aims to support nested documents for ExactNNIndexer

This PR introduces support for sub-indexing in the exactNNSearch by enabling nested documents. Here's a summary of the changes:

Abstract class methods are utilized to create a parent_id / initialization for each nested document.
The backend stores everything in the new_schema type, including the parent_id, but user need raw type. To handle this, the PR includes helper methods to convert the raw type back into the original type that the user inputted to the Indexer.
To ensure that the data type internally stored in the backend is not changed during casting, a copy method is included. The built-in document copy method does not support PyTorch tensors, so a new shallow copy method was created and tested.
The PR also includes test code, which is similar to the tests for other backends.
These changes aim to enhance the functionality of the exactNNSearch by supporting nested documents.

I'll take a look on that. All doc contents should be persist. If the load_binary function call the initialization, it should reconstruct subindices. I'll try some examples

And test it

This new commit supports the persist storage on subindex document and includes the relevant test.


awesome plugin!! i can help you simplify it!

this is a really cool usage of our plugin api!
we now have made it so that query and setQuery are internal details of a plugin. you can access current query and setQuery callback internally! see the changelog for graphiql explorer plugin. this prevents cursor jumping bugs and extra re-rendering. setting query and handler of the GraphiQL component leads to a semi controlled state that is not always ideal.

@cimdalli here is the preferred pattern:
https://github.com/graphql/graphiql/blob/main/packages/graphiql-plugin-explorer/CHANGELOG.md#minor-changes
your implementation no longer needs to require the user creating a volatile useState() implementation

@acao This is awesome, we have jumping cursor issue with the explorer plugin but seems it's solved already. I will apply your suggestion asap.


File Extension Error

variable for a zip code 'ng-model="vm.course.zip"' got preprended with a cdn url throwing angular sytanx error
Error: [$parse:syntax]
Changed to 'ng-model="vm.course.zipcode"' to fix the issue

Interesting. Most likely because of the zip suffix

Yeah, it thinks it's a zip file...

I wonder what the best way to approach this would be... Check the nature of the enclosing tag and emit any that have ng (or other directives from different frameworks)? Provide a way for the user to manually escape it from the cdnex? This is kind of a weird use case, I wonder how common it is to do something like this.

This also happened on the new HQ site with writing an email in quotes in a paragraph of text, for reference.
We didn’t want a<EMAIL_ADDRESS>email.

Closed with new code at https://github.com/jsonmaur/cdnex


bump msrv

Thank you!
Thank you for your contribution.
Please make sure that your submission includes the following:
Must

[ ] The code compiles without errors or warnings.
[ ] All examples work.
[ ] cargo fmt was run.
[ ] Your changes were added to the CHANGELOG.md in the proper section.
[ ] You updated existing examples or added examples (if applicable).
[ ] Added examples are checked in CI

Nice to have

[ ] You add a description of your work to this PR.
[ ] You added proper docs for your newly added features and code.

What requires a point release over 1.66.0? 1.66.1 didn't affect anything language-related, I think.