MeiliSearch as PostgreSQL extension

Is your feature request related to a problem? Please describe.
For most of the projects i need the feature set MeiliSearch provides i have almost always a regular database as my main data store – and this is almost always prostgreSQL. And in those cases data is often duplicated or at least linked with ids. Say i have a movie database with a long description for each movie i want to do a text search in. Those two datasets needs to be synced between the two data stores and it is often the case that i need to do multiple network/database round trips across that two data stores.
Say i have MeiliSearch to search for "Wizzard, Magik" and it returns some ids (123, 432, 666) that i use to query data in postgres that returns the full movie entry (Harry Potter, Persey Jackson, The Sorcerer’s Apprentice) so i can query the costomers that have watched that movies the last month on my streaming platform. You get the idea.
There is some effort behind the synchronization process and in handling the data in code and doing round trips and in modeling the data itself.
Describe the solution you'd like
I would like to have a PostgreSQL extension that is able to handle most of those complications. To make things short and don't waste time with an unsatisfying explanation from my side the idea is to have something similar to ZomboDB. ZomboDB is "linking" Elasticsearch and PostgreSQL in such a way that i only have to interact with PostgreSQL alone as a proxy to Elasticsearch. As Elasticsearch is way to excessive for almost all my use cases (i don't have petabytes of data to search through) and MeiliSearch covers all of previous usecases and is way easier to setup, maintain, configure and use this would ease up the process to "just quickly" add a text search to your project by just.
CREATE TABLE movies (
    id BIGINT NOT NULL PRIMARY KEY,
    name text NOT NULL,
    keywords varchar(64)[],
    short_summary text,
    long_description meili.fulltext,
    price bigint,
    inventory_count integer,
    discontinued boolean default false,
    availability_date date
);

CREATE INDEX idx_movies
          ON movies
       USING meili ((movies.*))
        WITH (url='localhost:7700/');

SELECT *
  FROM movies
 WHERE movies ==> 'botman+robin&limit=2';

@curquiza Thanks for transferring the issue to the right place! Out of curiosity if have started a little experiment myself. I don't think it will reach a production ready state and acts more as a learning project for myself. If it is ok i can post some updates if there is anything worth of sharing, maybe it can be useful for others.
It is out of my time budget to get anywhere close to what ZomboDB is doing for Elasticsearch but for a little experimental approach i am planning to have something along the lines of:
CREATE TABLE movies (
    id BIGINT NOT NULL PRIMARY KEY,
    name text NOT NULL,
    short_summary text,
    long_description text,
    -- and more
);

-- creates an index and updates documents on every INSERT/UPDATE/DELETE an the movies table via REST API
SELECT meili.watch("http://localhost:7700/", movies);

SELECT *
  FROM movies
 WHERE id IN (meili.query("harry potter"));

Thanks for sharing your updates!! This is really helpful! 😁

+1

Hello @israrwz
Welcome to the MeiliSearch community 🙂 if you have the time, what is the minimal feature you would expect in a Postgres plug-in?
Same question for everyone wanting the integration here of course!

@curquiza My expectation is something like @zombodb Reason: Currently, syncing the data is a pain. Or, if this can be sorted through something else, it is OK too. For example, I'm following up https://github.com/apache/nifi/pull/4065#issuecomment-1002862247 too

@israrwz thanks for allowing me to revisit this topic. @curquiza i haven't finished a minimal test that i would show others back then. Project still exists on my hard drive i may find some time over the next days to release something however bad – its just a minimal test with the pgx crate (from the author of zombodb) and would look pretty roughly like what i proposed in https://github.com/meilisearch/integration-guides/issues/71#issuecomment-752432762

@pythoneer FWIW, I personally see a very good future for it, as a separate project. It may very well be piggybacked on the hype of Hasura too. Good luck!

Hello @rrjanbiah and @pythoneer!
Thanks for your feedback! And for your involvement!
The Postgres integration is definitely an integration we (the integration team) want to build and maintain. Our current human resources and the time we have made us delay the project many times, but be sure this is something we keep in mind :)
The integration team is currently growing which will help, and the Postgres integration is one of our priorities in 2022. We may work around or with what the community built first. So, thanks for this!

I would vote for this integration.
also I would suggest a more deep integration like pgroonga, which we could use it more sql-like, but it's a bit trouble on scaling out.

Hello @DeoLeung !
Could you upvote on the roadmap?

Hello @DeoLeung !
Could you upvote on the roadmap?

I just went and upvoted that issue @bidoubiwa but, FYI, that issue doesn't appear to be the same as this one. That issue talks about SQL integration. This issue is asking for meilisearch to be exposed as a Postgres extension (i.e. this issue is specific to Postgres).  The solution to this issue (I expect) would not be compatible with other SQL databases (unless they implemented the Postgres API as some do).

Hi.  Eric Ridge here.  Primary developer behind ZomboDB and pgx.
Up above in https://github.com/meilisearch/integration-guides/issues/71#issuecomment-1008656726, @curquiza said:

Our current human resources and the time we have made us delay the project many times...

I'm not here to sell anything, but it's possible my employer would be willing to develop a MeiliSearch Postgres extension.  It wouldn't be cheap, but it'd be yours to do with whatever you want.
For what it's worth, we have no intentions of doing this on our own.  We are, in fact, working on our own solution -- yet another Rust text search library, but with Postgres as the first library consumer.  But I don't see why that'd preclude us from doing a similar MeiliSearch integration, if a business arrangement could be made.
Anyways, good luck.  MeiliSearch is super cool and the Postgres world can definitely benefit from better text search solutions!  Feel free to reach out to me at eebbrr(at)gmail.com or DM me on twitter (https://twitter.com/zombodb).

Hello, everyone watching this issue!
I'm closing this issue since it's been a while since there is no news regarding this topic, and it is not in our plans to work on such a feature.
In any case, it would be nice if someone from the community could work on it.

@brunoocasali This is an important feature. I followed Meilisearch for a long time, but synchronizing Melisearch with PostgreSQL is not easy.
And it is the same for any search engine. If you can do it, I think you will win the market.

Hello @meotimdihia
Thanks for this feedback
To let you know, we are currently working on finding a way to sync the primary database of the users with our Cloud offer. As you can guess it's not easy because multiple use cases and DB exist.
We plan first to release guide to help people manage it, and we plan to provide a more adapted solution in the future.
Stay tuned! 🤟

For someone who find yet the way to sync MeiliSearch with DB, I suggest to try meilisync before any PostgreSQL extension released.
It use CDC method not extension method. but has advantage of easy support of variety DB.


python3Packages.torch: general cleanups and fixups

Description of changes

Moved things which had some sort of selection logic to the caller, which now has a recursive attribute set, allowing the selection logic to change in response to overrides.
Removed constraint that Magma and MPI use the same version of CUDA as Torch, as they shouldn't interfere with each other and are loaded in different processes.
Fixes a warning thrown when allowAliases is true caused by a with cudaPackages; shadowing the copy of autoAddDriverRunpath provided by callPackage (cc @mweinelt).
Construction of a list of CUDA build inputs (creatively named cudaBuildInputs).
Cleaned up and moved shell variables to env so they are provided as environment variables to the builder, since PyTorch's setup.py expects those (also ensures we catch variables which aren't set to strings).
Fixed deprecation warnings caused by usage of substituteInPlace's --replace argument.
Updated the list of supported CUDA capabilities to include Thor (that's Hopper with additional features, 9.0a).
Closes #294111.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

CC @NixOS/cuda-maintainers @NixOS/rocm-maintainers

Add a :+1: reaction to pull requests you find important.

I'm able to build and run nix-cuda-test on my 4090:
$ nix run -L --override-input nixpkgs "github:NixOS/nixpkgs/f8b21116e19a7787194630d38ea3c39d7ffc9625" .#nix-cuda-test
...
Seed set to 42
Using bfloat16 Automatic Mixed Precision (AMP)
GPU available: True (cuda), used: True
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
HPU available: False, using: 0 HPUs
Files already downloaded and verified
Files already downloaded and verified
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0]

  | Name      | Type             | Params
-----------------------------------------------
0 | criterion | CrossEntropyLoss | 0
1 | model     | ViT              | 86.3 M
-----------------------------------------------
86.3 M    Trainable params
0         Non-trainable params
86.3 M    Total params
345.317   Total estimated model params size (MB)
Epoch 0:  58%|████████████████████████████████████████████████████████████████████████████▏                                                       | 451/781 [00:52<00:38,  8.64it/s, v_num=7, train_loss=2.310]

Because PyTorch is still using FindCUDA, they need to vendor fixes like https://github.com/pytorch/pytorch/commit/7d10b06e1a7ff87eb38d39e01eb717bd1001714e
Gotta find out how to get theirs working with our CMake 🙃


open supportCustomizedId feature for tinkerpop test

Change-Id: I486f17abcd007964f82169518788de89dffdbf72

please update the title of the PR too


[Coral-Trino] Migrate cast transformation from RelNode layer to SqlNode layer

What changes are proposed in this pull request, and why are they necessary?

The goal of this PR is to migrate the logic of visitCast from RelNode layer (in Calcite2TrinoUDFConverter.convertRel which takes a calcite rel and transforms it to trino compatible rel)  to the SqlNode layer as part of Coral IR v2. Some important changes to make this happen:

New CastOperatorTransformer class which is a sqlNode transformer that handles Calcite to Trino cast operator transformation.
Since CastOperatorTransformer requires RelType derivation, which requires Calcite validation, which does not preserve laterals (known and chosen behavior), we had to override this behavior as some queries which had casts and lateral joins were having their laterals incorrectly dropped as an unwanted side effect of CastOperatorTransformer.
Calcite2TrinoUDFConverter deleted since there are no more transformations in the class left

How was this patch tested?
gradle clean build
regression tests for trino, avro and spark all passing

Looks like there is no direct relationship between this PR and LATERAL removal. Does it makes sense to add test cases for LATERAL in this PR or a separate PR?

The goal here is to have no test regressions caused by dropped LATERALs since we know that would be semantically incorrect. There's no need to add any additional tests in this PR because if the current tests pass, we know that the LATERAL was added back correctly by https://github.com/linkedin/linkedin-calcite/pull/98. Therefore we are already testing LATERAL in Coral.

Also, I recall we had Coral-side code that reintroduces the LATERA before https://github.com/linkedin/linkedin-calcite/pull/98. Do you know why it does not show in the LHS version of this PR?

We introduce LATERAL in RelToTrinoConverter.convertToSqlNode which happens before CastOperatorTransformer, where LATERAL is then dropped again due to type derivation.

we know that the LATERAL was added back correctly by https://github.com/linkedin/linkedin-calcite/pull/98

I was suggesting to add Coral-side tests for the same issue. Could be inspired by the failing regression integration tests.

Also, I recall we had Coral-side code that reintroduces the LATERAL before https://github.com/linkedin/linkedin-calcite/pull/98

I was referring to code that @aastha25 introduced to manually reinstate LATERAL after validation. I think it was in TypeValidationUtil if I recall correctly. The reason I am asking is that now that we appropriately handle LATERAL in calcite, this reinstating code is no longer needed and should appear as deleted in the LHS of the PR diff.

I was suggesting to add Coral-side tests for the same issue. Could be inspired by the failing regression integration tests.

We already have unit tests that capture this issue. As long as the input query as a CAST operator and a LATERAL, the missing LATERAL post-validation issue will be tested.

to manually reinstate LATERAL after validation. I think it was in TypeValidationUtil if I recall correctly

I believe you're talking about the post processor @aastha25 wrote a long time ago to manually add the LATERAL back. This code was never commited/merged in the first place since the patch was a WIP so it wouldn't need to be deleted (deleted in the LHS of the PR diff).

@wmoustafa Addressed the last 2 javadoc comments. Thanks for the thorough review :)

Could you also add in the PR description that this patch has been tested against the li-trino-hotfix branch?
rest LGTM.


Use full API response

These commits were extracted from #2819 for easier merging.

Thanks @borisyankov !
Flow finds errors in a couple of these.
Some changes made in #3243 also apply to each of these: jsdoc immediately on function, and spread rather than intersection types.
-  dispatch(initAlertWords(await getAlertWords(auth)));
+  const alertWordsResponse = await getAlertWords(auth);
+  dispatch(initAlertWords(alertWordsResponse.alert_words));

Here's what I wrote about this back in September:
"""
So this one and a bunch of others pose a bit of an interesting API design problem: the response contains just one meaningful field (one field other than {'result': 'success', 'msg': ''}), and that field's name just repeats part of the name of the API call. It feels like the calling code gets a little worse when it has to extract that out -- it's just a bit of extra ceremony and repetition that doesn't add any clarity.
For these cases, I wonder if it'd be better to keep the extraction, but just do it in more idiomatic style?  E.g.:
export default async (auth: Auth): Promise<string[]> =>
  apiGet(auth, 'users/me/alert_words').then((r: ApiResponseAlertWords) => r.alert_words);

The price is a bit of non-uniformity between different API functions -- some do an extraction, some don't. I think the non-uniformity would make the extraction not worth it, if we could only apply it in a few cases. But it looks like in fact the majority of API calls follow this pattern (which makes sense -- lots of questions naturally have an answer that's just one thing, or one uniform list of things), so I think keeping the extraction for those probably is worth it.
What do you think?
"""

I think the apiCall().then(result => result.field) is a better approach than an extractor function.
But the reason to remove that function was exactly for consistency of the return value and also to make sure adding a field to a previously single-value response not be a breaking change.
I expect this to happen more in the future, but previously getMessages was returning only messages and now it returns several other fields found_newest, found_oldest etc.
What makes it a bit prettier is the use of destructuring like:
const { messages } = getMessages(...);

I went and changed the code from the type
const messagesResult = getMessages(...);

Also made all other minor changes. Should be ready.

Hmm, the destructuring does mitigate it. Still feels kind of awkward to me, but with this I think it'll be OK and I don't mind trying it out.
Smaller comments:

The messagesResponse appears in an early commit, then gets fixed in a later one:

-  const [messagesResponse, streams] = await Promise.all([
+  const [{ messages }, { streams }] = await Promise.all([

The getUsers commit has this old comment apply. Also the commit message is a bit confusing by referring to "all params" when there's only one; made me wonder if there were more, and go check.

The getMessages call is now one that would especially benefit from a link to the API doc, with all those new return properties.

As usual, let's have the commit summaries start like api:  rather than refactor:.

About the API docs links, have a look at this PR consisting of a single commit:
#3250

About the API docs links, have a look at this PR consisting of a single commit:
#3250

Yep, but that doesn't include getMessages which is the one I mentioned :slightly_smiling_face:

I did:

rebase
made the change to getMessages earlier
added a link in getMessages to https://zulipchat.com/api/get-messages
getUsers have a better commit message

Thanks!
This comment still applies:

The getUsers commit has this old comment apply.

I think I can merge up to that point, though.

(And thanks for the list of changes you made!)

I did undo the change of adding client_gravatar as an argument to getUsers.

Merged -- thanks again @borisyankov !


add example of how to get the right-side-up width/height

See #3124

Coverage remained the same at 100.0% when pulling 9d10be1c2bc20f82a4f51ba7d367f2a15a212025 on mindplay-dk:patch-1 into 23033e2050d9ef099ecb02dde0f56f9f23e8d30d on lovell:main.

Tak Rasmus!

Selv tak 😄


[DO NOT MERGE] Dummy PR that merges dev into master, just to run CI

Codecov Report

Merging #119 (ee8c808) into master (77b4f58) will decrease coverage by 0.66%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #119      +/-   ##
==========================================
- Coverage   92.80%   92.13%   -0.67%
==========================================
  Files           7        7
  Lines         264      267       +3
==========================================
+ Hits          245      246       +1
- Misses         19       21       +2

Impacted Files
Coverage Δ

src/common.jl
91.30% <33.33%> (-8.70%)
:arrow_down:

Continue to review full report at Codecov.

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

All green!


not working php:8.3.4

php:8.3.4
gotenberg：8.3.0
Executing the following code always returns：code=405, message=Method Not Allowed
  $request = Gotenberg::libreOffice("http://gotenberg:3000")             ->convert(Stream::path(BASE_PATH . '/storage/pdf/2023.docx'));         Gotenberg::save($request, BASE_PATH . '/storage');
Docker logs
{"level":"error","ts":1712074302.3096998,"logger":"api","msg":"code=405, message=Method Not Allowed","trace":"86ea7b13-eeb4-4cb6-b208-b6eca9267db6","remote_ip":"<IP_ADDRESS>","host":"gotenberg:3000","uri":"/forms/libreoffice/convert","method":"PUT","path":"/forms/libreoffice/convert","referer":"","user_agent":"GuzzleHttp/7","status":405,"latency":58400,"latency_human":"58.4µs","bytes_in":0,"bytes_out":18}

Please reopen if you find anything 😄


data-attributes support

It will be useful to support some data-attributes for initialisation

Nice idea, I'll think probably implement it in next version

Version 1.1.3 has been released and it contains what you wanted ;)


WIP: nixos/vagrant-virtualbox-image: init

Introduce a new vagrant profile
Build vagrant boxes directly out of nixos configurations instead of using packer like in nix-community/nixbox project.
This is untested for now as I don't have a virtualbox setup at the moment.

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-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
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/building-openstack-images-and-development-vms-from-a-macos-host/9522/5

To test:
cd
git clone --depth=1 https://github.com/NixOS/nixpkgs
cd ~/nixpkgs
git fetch --depth=1 origin pull/76071/head:vagrant-virtualbox-image
git checkout vagrant-virtualbox-image
nix-env -f https://github.com/zimbatm/nixos-generators/archive/vagrant-virtualbox.tar.gz -i
cd
vim ex.nix

ex.nix:
{ config, lib, pkgs, modulesPath, ... }:
{
  documentation.man.enable = false;
  documentation.nixos.enable = false;
  sound.enable = false;
}

NIX_PATH=nixpkgs=~/nixpkgs/ nixos-generate -f vagrant-virtualbox -c ex.nix

Starting in Vagrant
$ vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Box './dwn9q0skkayiq8jmjfi8a5vdiyrs8v16-virtualbox-vagrant.box' could not be found. Attempting to find and install...
    default: Box Provider: virtualbox
    default: Box Version: >= 0
==> default: Box file was not detected as metadata. Adding it directly...
==> default: Adding box './dwn9q0skkayiq8jmjfi8a5vdiyrs8v16-virtualbox-vagrant.box' (v0) for provider: virtualbox
    default: Unpacking necessary files from: file:///Users/josephlong/devel/dap_infrastructure/apphost/dwn9q0skkayiq8jmjfi8a5vdiyrs8v16-virtualbox-vagrant.box
==> default: Successfully added box './dwn9q0skkayiq8jmjfi8a5vdiyrs8v16-virtualbox-vagrant.box' (v0) for 'virtualbox'!
==> default: Importing base box './dwn9q0skkayiq8jmjfi8a5vdiyrs8v16-virtualbox-vagrant.box'...
==> default: Matching MAC address for NAT networking...
==> default: Setting the name of the VM: apphost_default_1603052536171_30324
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
    default: Adapter 1: nat
==> default: Forwarding ports...
    default: 22 (guest) => 2222 (host) (adapter 1)
==> default: Booting VM...
There was an error while executing `VBoxManage`, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.

Command: ["startvm", "0ada89ef-4748-4756-b3b6-b449365884eb", "--type", "headless"]

Stderr: VBoxManage: error: Implementation of the USB 2.0 controller not found!
VBoxManage: error: Because the USB 2.0 controller state is part of the saved VM state, the VM cannot be started. To fix this problem, either install the 'Oracle VM VirtualBox Extension Pack' or disable USB 2.0 support in the VM settings.
VBoxManage: error: Note! This error could also mean that an incompatible version of the 'Oracle VM VirtualBox Extension Pack' is installed (VERR_NOT_FOUND)
VBoxManage: error: Details: code NS_ERROR_FAILURE (0x80004005), component ConsoleWrap, interface IConsole

Starts after VBoxManage modifyvm c9a1e597-123b-4fdc-998a-a6f4cca03c29 --usb off, but missing passwordless sudo.
Rebuilt with a config including security.sudo.wheelNeedsPassword = false;, applied the same modifyvm command, and successfully booted!
It's not clear to me how to modify https://github.com/NixOS/nixpkgs/blob/d6daf4509202fcfec5fdc0d0d19c1a3fccfdcd32/nixos/modules/virtualisation/virtualbox-image.nix#L77 to share code with this target without adding (imo) unnecessary USB and audio support

Awesome. Do you want to take this PR to the finish line?

My Nix-fu is weak, so I would appreciate some advice on how to make an acceptable modification to the existing virtualbox expression to make the usb and audio stuff optional. Some kind of map of options with defaults?
Alternatively, I could copy the guts of the virtualbox script into the vagrant virtualbox instead of including it. I’m worried it would be rejected for code duplication though 😄

It looks like https://github.com/NixOS/nixpkgs/commit/be255392dd85e8d649aa2b278b5a5227de1543d4 took care of that for me. I'll update my PR to match.

Replaced by #101120


[Github Actions] Installation fails on windows-latest

Recent CI/CD runs fail on windows when trying to install the dependencies.
This could be related to the recent change to Windows Server 2022 for all windows-latest github actions.
Investigating ...

seems to be caused by the pip installation of cookiecutter:
  Installing pip packages: cookiecutter
  'DOSKEY' is not recognized as an internal or external command,
  operable program or batch file.
  'chcp' is not recognized as an internal or external command,
  operable program or batch file.
  'chcp' is not recognized as an internal or external command,
  operable program or batch file.
  info     ****************** Backtrace Start ******************
  debug    Loading configuration
  trace    Compute configurable 'create_base'
  trace    Compute configurable 'no_env'
  trace    Compute configurable 'no_rc'
  trace    Compute configurable 'rc_files'
  trace    Compute configurable 'root_prefix'
  trace    Get RC files configuration from locations up to HomeDir
  critical Cannot determine HOME (checked USERPROFILE, HOMEDRIVE and HOMEPATH env vars)
  info     ****************** Backtrace End ********************
  critical pip failed to install packages
  Error: The process 'C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe' failed with exit code 1
  C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -command "if (!(Test-Path $profile))

The issue was the pip call from the version of micromamba-bin used by the action (0.21). Specifying the mamba version to 0.20 seems to solve the issue:
- name: Install mamba and create environment
  uses: mamba-org/provision-with-micromamba@main
  with:
    micromamba-version: 0.20.0     # This version works, 0.21 does not.
    environment-file: ci/ci_test_env.yml
    environment-name: test_environment
    extra-specs: python=${{ matrix.python-version }}

I will try to create a minimal example reproducing the error and open an issue in the mamba repository.

I found this issue on their repo: https://github.com/mamba-org/mamba/issues/1468 could that be it?
The fix was released with 0.21.1 few hours ago...


Usage of disable_introspection method.

I'm new to rust, but let's go to what matters:
How can i write something like that, if the disable_introspection function uses (mut self) parameter?
pub fn create_schema() -> GqlSchema {
  let schema = Schema::build(...);

  if my_condition {
    schema.disable_introspection();
  };

  schema.finish()
}

If the inner code is just self.registry.disable_introspection = true, could it be write as schema.instrospection(true) to enable some type of "external" configuration?

Do it like this:
 let mut schema = Schema::build(...);

  if my_condition {
    schema = schema.disable_introspection();
  };

  schema.finish()

thanks!


[dev inspect] Move call entry point

Add direct Move call entry point for dev inspect

But will leave it with you all if you want to update to add the digest generation in the authority.

But will leave it with you all if you want to update to add the digest generation in the authority.

I think we should do it for all the end points, but I'll do it just the dev_inspect_move_call for now

All as in all the dry run end points, or all as in every endpoint?

All as in all the dry run end points, or all as in every endpoint?

The dry run and dev inspect endpoints. The normal endpoints already have the  transaction digest hashed as a part of the signed certificate


Add support for OIDC

To meet my requirements, I need to configure multiple OIDC providers within the Identity Server. Each OIDC provider has a different authority URL, which we need to configure from the database. It would be great if you could provide examples on how to achieve this, or we can include this in the current code.

In the image, Authority URL is dynamic and in the database, scheme column is  primary key. Multiple OIDCs cannot be added with different authorities. In spite of removing the primary key from scheme column, it overrides the previous record added for oidc.
While TheldServer perfectly fits my requirements, I cannot use TheldServer because I need my own implementation of an identity server.

CallbackPath must be unique or it doesn't work because the authentication middleware find the provider with the CallbackPath. Scheme must be unique too, it's why it's the key.


eagle api

Can you give a run down tutorial how eagle api works,
how do i create folders
how do i "upload images"
how do i insert tags, annotations
can u just point me in the code where exactly u do that

Can you give a run down tutorial how eagle api works, how do i create folders how do i "upload images" how do i insert tags, annotations can u just point me in the code where exactly u do that

The best way to understand the basic usage of the Eagle API is to read this Eagle API Document. I think it will be more understandable.
Eagle API Document
The part of the code for creating folders is here:
https://github.com/shingo1228/ComfyUI-send-eagle-slim/blob/af5f044990c7c9e4916b2303242348878869f540/send_eagle.py#L93-L97
The part of the code for uploading images is here:
https://github.com/shingo1228/ComfyUI-send-eagle-slim/blob/af5f044990c7c9e4916b2303242348878869f540/send_eagle.py#L126
Tags and annotations are in this part of the code. They are added to an item and then sent to Eagle.
https://github.com/shingo1228/ComfyUI-send-eagle-slim/blob/af5f044990c7c9e4916b2303242348878869f540/send_eagle.py#L69-L70
https://github.com/shingo1228/ComfyUI-send-eagle-slim/blob/af5f044990c7c9e4916b2303242348878869f540/prompt_info_extractor.py#L210-L235

Also a suggestion , instead of tracking prompt etc , the send to eagle node could have input fields where we just connect the prompts

Thank you. I have previously created a version with input fields, but I decided against adopting it because it was less intuitive to understand how to combine the prompts, which are a hidden attribute, with the text of the input fields, which are a public attribute.

Also a suggestion , instead of tracking prompt etc , the send to eagle node could have input fields where we just connect the prompts

Thank you for your suggestion regarding the 'Send to Eagle' node.
After careful consideration, I've decided to revisit the idea of incorporating input fields for connecting the prompts. I'm pleased to announce that I've now developed a new version of the program, named 'Send Eagle with text.' This updated version includes the feature you suggested, allowing users to directly connect prompts via input fields.
I believe this enhancement will make the tool more user-friendly and efficient. I appreciate your feedback and hope this update aligns with your expectations. Feel free to check it out and let me know your thoughts!


Support Conv2dTranspose and GroupConv2dTranspose for IE

@fujunwei @mingmingtasd PTAL, thanks!
Test results:
[----------] Global test environment tear-down
[==========] 104 tests from 1 test suite ran. (340 ms total)
[  PASSED  ] 102 tests.
[  FAILED  ] 2 tests, listed below:
[  FAILED  ] Conv2dTests.Conv2dTransposeWithAutoPadSameUpperDefault
[  FAILED  ] Conv2dTests.Conv2dTransposeWithAutoPadSameLowerNhwcIhwo

Reported a issue to openvino upstream.

Let's merge it, thanks.


Add codeowners file.

Is your feature request related to a problem? Please describe.
NA
Describe the solution you'd like
This is to add codeowners for different branches and also assign default reviewers based on the branching rules.
Describe alternatives you've considered
Na
Additional context
NA

Hi @sumanabhi  can you close this issue if there's nothing pending to work on it?


Add hcm.edu.vn

Institution/School Name
Ngành Giáo Dục và Đào Tạo Thành Phố Hồ Chí Minh
Ho Chi Minh City - Department of Education and Training
Instiution/School Website
https://hcm.edu.vn/
Email Users
<EMAIL_ADDRESS>Please place an x between the square brackets if yes

[x] Students
[ ] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[x] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

@trietminh First stage of verification passed, official domain for Ngành Giáo Dục và Đào Tạo Thành Phố Hồ Chí Minh is hcm.edu.vn as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@trietminh Could you please clarify whether the domain hcm.edu.vn is dedicated to a specific school, or all schools in the city use it? Thank you.

Thank you @philipto for your review.
hcm.edu.vn is the domain provided for the entire schools/departments/colleges under the Department of Education and Training of Ho Chi Minh City (Viet Nam), which does not have its own domain. Each student and teacher of the schools/departments/colleges is provided with an email address ending in @hcm.edu.vn, including both IT-related and non-IT-related fields of study.

Thank you @philipto for your review.
hcm.edu.vn is the domain for the entire schools/departments/colleges under the Department of Education and Training of Ho Chi Minh City (Viet Nam). Each student and teacher of the schools/departments/colleges is provided with an email address ending in @hcm.edu.vn, including both IT-related and non-IT-related fields of study.

@trietminh Thank you for your answer. As you see, this domain represents an umbrella institution, not a specific single university or a college. Regrettably, the rules of this repository prohibits to add the umbrella institutions domains to it.
There are about 1.000.000 students in the schools of the city. While we would be very glad to serve such a big number of passionate users from Vietnam, we also understand that not all of the students actually need a free JetBrains subscription. Therefore, some of them (who do not need the subscription) may experience temptation to share their free subscription with a third party that need it. While sharing a subscription with a third party is prohibited, it's unquestionable that when the number of students is over a million, it's impossible to ensure that every one of them follows the license terms.
Therefore, please consider different approach. Please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
If you are a teacher, please send us an email to<EMAIL_ADDRESS>and we'll suggest other possible options for your school, if the above-mentioned approach does not work for your students.
I have to close the request without merging.
Thank you for your understanding!


API Authentication Layer

APIs should authenticate & authorise users before granting access to any API endpoinds.
We should have at least two roles - admin and user.
Users should be able to use read-only APIs (eg Tasks API, Payloads API, Workflow Instances API, etc). Admins should also be allowed to use write/update APIs such as the Workflows API.

It should be possible to configure an OAuth2 provider URL in the configuration file
It should be possible to configure the token claim that gives access to Admin APIs, and what claim gives access to the read APIs (mapping external roles/properties to our "admin"/"user" roles).

TODO:  add security section to requirements

https://dev-aide.answerdigital.io:8443/auth/realms/aide/.well-known/openid-configuration

@elleyouseewhy You provided a pointer to AnsDigital OpenID configuration details.  Do you mean to suggest OpenID compliant solution that can authenticate with AnDigital is recommended?

The solution should pluggable with any OpenID connect and OAuth2 compliant provider.  i.e. Keycloak is fine for E2E testing and reference implementation, but it should support other Auth providers compliant to the same standards.
I don't see any issues with using Keycloak as a reference.

@slbryson we were talking about integrating with an OpenID provider to allow validating tokens when authenticating API requests. Lucy has just put this here as an example for the team as to what this URL is on the AIDE test environment.

Should we make this a standalone library like the storage & messaging repositoiries so it can be used by both IG & WM?

From refinement: Implement Mock auth service


Added TimeoutRetry example

@rprouse As discussed in NUnit main project #3345 issue, I've added an example of timeout retry attribute to this repository

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: lukzas sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@rprouse would you approve changes? 😉


carla can't run on GPU

Hi, I installed carla in a conda environment on a remote server, installed openGL in a docker container, and used ssh to remotely connect to Mabaxterm and configure X11-forwarding for visualization.
Execute the following command to run carla: CUDA_VISIBLE_DEVICES=0 . /carla/CarlaUE4.sh -prefernvidia -opengl -carla-server -benchmark -fps=5 -nosound --world-port=15302

But carla runs without GPU, resulting in rendering no environment.

Can you please tell me how to solve this problem？

We recommend to use vulkan, have you tried with it?

We recommend to use vulkan, have you tried with it?

Thanks for your reply.
I downloaded vulkan, but it reports an error: Cannot find a compatible yulkan driver (ICD)

The driver version on the server is: NVIDIA-SMI 515.65.01 Driver Version: 515.65.01 CUDA Version: 11.7
This seems to be because there is a fixed version requirement for the vulkan 1.3 driver, but I don't have permission to change the nvidia driver version on the school server.
Is there any other way to resolve this issue please?

Hi @Morizb
It seems that the loader is not finding your Vulkan drivers. Could you try sharing with your docker container the icd JSON files:
--volume=/usr/share/vulkan/icd.d:/usr/share/vulkan/icd.d:ro

Closing the issue. Please, feel free to reopen the issue if needed.

Hi, I installed carla in a conda environment on a remote server, installed openGL in a docker container, and used ssh to remotely connect to Mabaxterm and configure X11-forwarding for visualization.
Execute the following command to run carla: CUDA_VISIBLE_DEVICES=0 . /carla/CarlaUE4.sh -prefernvidia -opengl -carla-server -benchmark -fps=5 -nosound --world-port=15302

But carla runs without GPU, resulting in rendering no environment.
Can you please tell me how to solve this problem？

15/5000
Hello, may I ask how you finally solved this problem？


use atomics for metrics

AFAIK metricsMu is not really performing any useful function over and above the likely more efficient atomic integer operations available on most modern CPUs. In the Metric() return function, the metricsMu served to ensure all the counters are returned as of an instant in time when no other counter is incremented relative to other counters, but since the write locks are only held for the duration of an increment of a single element of the structure, even that has no utility (i.e., there are never increments of two counters at once while holding the lock across both increments, which is the only thing the read lock would protect).

I think you are right and this might be a good change. Before we merge this, have you done any benchmarking? I am asking because of this bit:

likely more efficient atomic integer operations available on most modern CPUs

I think you are right and this might be a good change. Before we merge this, have you done any benchmarking here? I am asking because of this bit:

likely more efficient atomic integer operations available on most modern CPUs

name                     old time/op    new time/op    delta
CacheSetWithoutTTL-8        290ns ± 0%     288ns ± 0%   ~     (p=1.000 n=1+1)
CacheSetWithGlobalTTL-8     516ns ± 0%     463ns ± 0%   ~     (p=1.000 n=1+1)

name                     old alloc/op   new alloc/op   delta
CacheSetWithoutTTL-8        99.0B ± 0%     97.0B ± 0%   ~     (p=1.000 n=1+1)
CacheSetWithGlobalTTL-8      256B ± 0%      253B ± 0%   ~     (p=1.000 n=1+1)

name                     old allocs/op  new allocs/op  delta
CacheSetWithoutTTL-8         2.00 ± 0%      2.00 ± 0%   ~     (all equal)
CacheSetWithGlobalTTL-8      4.00 ± 0%      4.00 ± 0%   ~     (all equal)

I think you are right and this might be a good change. Before we merge this, have you done any benchmarking here? I am asking because of this bit:

likely more efficient atomic integer operations available on most modern CPUs

name                     old time/op    new time/op    delta
CacheSetWithoutTTL-8        290ns ± 0%     288ns ± 0%   ~     (p=1.000 n=1+1)
CacheSetWithGlobalTTL-8     516ns ± 0%     463ns ± 0%   ~     (p=1.000 n=1+1)

name                     old alloc/op   new alloc/op   delta
CacheSetWithoutTTL-8        99.0B ± 0%     97.0B ± 0%   ~     (p=1.000 n=1+1)
CacheSetWithGlobalTTL-8      256B ± 0%      253B ± 0%   ~     (p=1.000 n=1+1)

name                     old allocs/op  new allocs/op  delta
CacheSetWithoutTTL-8         2.00 ± 0%      2.00 ± 0%   ~     (all equal)
CacheSetWithGlobalTTL-8      4.00 ± 0%      4.00 ± 0%   ~     (all equal)

Having seen the benchmark results, I'm not entirely convinced this would be a good change. It doesn't look like an improvement at all. Perhaps the banchmark tests should be different to properly test this?


readBody() does not take into account application/x-www-form-urlencoded; charset=utf-8

If charset=utf-8 is added to the content type readBody() will attempt to parse the body as json and fail. See
https://github.com/unjs/h3/blob/d11f817f1b18d5da6e311bdd0a222ae661efa2c2/src/utils/body.ts#L107-L115.

I think we should support charset in the content type header by either extracting it first or using string.contains

Thanks for reporting issue. It will be resolved in next release by (4af66eb72a3b1ba0be311ac50a7668706a4e10df). I have added an internal comment that in next iterations, we might actually use the charset to handle encoding.


Updated project to reflect bug when custom source sets are used

Added different source sets for different versions
Updated kotlin to 1.2.71
Updated gradle jlink plugin to 2.1.0

Fixed in 2.1.1


Using the test reporter without using MUnit as test framework

Hi there,
I am trying to use the MUnit sbt plugin as described here in order to generate an HTML test report for my project. I am boldly assuming that I can do so even if I am not using MUnit as the project test framework (my project uses Scalatest instead, which should still generate JUnit compliant XML results).
Sbt changes
I have defined a subproject that depends on another project/module in my build:
lazy val `lenses-domain-core-docs` =
  project
    .in(file("project-x-docs"))
    .dependsOn(`project-x`)
    .settings(
      mdocVariables := Map(
        "VERSION" -> version.value,
      ),
      mdocIn := file("project-x-docs/docs"),
    )
    .enablePlugins(MdocPlugin, MUnitReportPlugin)

When I run sbt 'project-x-docs/mdoc' I can see Mdoc working as expected and compiling scala snippets in my markdown files. However, the HTML report generation fails with a warning:
warning: Unable to generate MUnit HTML test report.

Is there a way to troubleshoot this error, perhaps by making the plugin more verbose?
Google Cloud Storage setup
Notice that, I have exported the GOOGLE_APPLICATION_CREDENTIALS_JSON and GOOGLE_APPLICATION_CREDENTIALS env variable pointing to a service account that has admin access to Google storage on my Gcloud. However, I could not create a bucket named munit-test-reports as I believe these have to be unique across accounts.

Finally, this is the content of the property file generated by MUnit in resource_managed:
❮ cat ./project-x-docs/target/scala-2.12/resource_managed/main/munit.properties
#MUnit properties
#Fri Apr 03 17:59:17 BST 2020
munitBucketName=munit-test-reports

@afiore any update on this? The bucket name is configurable
munitBucket := Some("my-custom-bucket-name")

https://github.com/scalameta/munit/blob/f61429f66f8fd81ee71e6c10dc2fe212073b5b07/munit-sbt/src/main/scala/munit/sbtmunit/MUnitPlugin.scala#L16

Feel free to reopen to continue the discussion.


Select multiple Microcenter Locations

Im fortunate enough to have multiple Microcenter locations available to me, would it be possible to select multiple microcenter locations to monitor?

Shout out to Georgia! Sounds like a good idea.


Change Cocoapods to build with xcframework

This PR allows Cocoapods to build Realm with an XCFramework rather than the fat libs for each individual platform. This adds support for Catalyst and Watch arm64. This implementation currently needs a fix-me that should be addressed in the the next release of pods (1.10) as using an XCFramework containing static libs inside a Cocoa framework currently does not setup and link correctly. To get around this the linker flags & library search paths are set manually. The end user does not need to do anything new to use this new implementation.

I don't think it's worth trying to hack around CocoaPods 1.9 rather than just waiting for 1.10 to be released. The goal of using a xcframework is to simplify the build setup and support catalyst, and this just makes it more complicated and doesn't support both catalyst and normal macos.

It's good to point that this PR will require changes in the way xcframeworks are built, here is an error from Cocoapods 1.10 beta
 Unable to install vendored xcframework `realm-sync` for Pod `Realm-iOS` because it contains static libraries
with differing binary names: realm-maccatalyst, realm-iphone-device, realm-macosx, realm-watch-device, realm-watch-simulator, realm-appletv-simulator, realm-appletv-device, and realm-iphone-simulator.

Why you closed this PR? Something not working as expected?
Because only one library in my project can't be ported to the Mac OS using Mac Catalyst and this is Realm... I need this.

@AndrewKochulab https://github.com/realm/realm-cocoa/pull/6710 we are still on it :)

@AndrewKochulab #6710 we are still on it :)

Because I was already afraid that you would refuse support of Catalyst. Thank you a lot!


Fixed Mozilla Firefox bugs

Corresponding issue #933
Fixes #933
Work Done:

Added the custom scrollbar in Mozilla Firefox
Hid the SVG in navbar on laptop screen to fix the overlap between Sidebar and Navbar

Relevant Screenshots
Before

After

@Kirsten-Nancy  Congratulations on getting your PR merged.

Thank you @smaranjitghose , it's been awesome exploring and working on the project. This is my first time contributing to an open-source project and I look forward to making more contributions to the projects.


Server message to all users before the server shuts down

Hi,
Is there any way to send a message to all connected users before the onClose event in order to tell all browsers to try to reconnect after a certain time?
Thanks in advance.
Kind regards
Lou

I have found there is the "onclose" event on the browser side...


Just a little fix

Based on @josejulio's comment I've realized that one may simply do this:
docker run --link=hawkular-cassandra -e CASSANDRA_NODES=hawkular-cassandra \
 -e HAWKULAR_USER="" -v /foo:/client-secrets/hawkular-services.password \
 -e HAWKULAR_USE_SSL=true -p 8443:8443 `whoami`/hawkular-services

I.e. passing the password as the mounted file and empty username via the env property, this would cause the auto-generation of the username, but also the secret password would be printed.. No big deal, because if one can pass any env. variable to the container, the system can be considered as already compromised and something even more nasty can be done via docker inspect or docker exec ... bash, but still.

Thanks, this LGTM


Fixes Model fetchAll event documentation

The fetching:collection and fetched:collection event tags have the event name wrapped in quotes. This is causing an issue in JSDoc which is preventing these events from being rendered in the menu and in the event detail blocks. This fixes that by removing the quotes and updating the JSDoc theme to use a more accurate regular expression for simplifying event names (bookshelf/bookshelf-jsdoc-theme#3).

Hey, I added a comment over at bookshelf/bookshelf-jsdoc-theme#3. Sorry for the late response.


auto generate META-INF content for DataTypeTransformer

Hi, I have some doubts regarding this PR, in particular,

I had to hack camel-package-maven-plugin in order to support the DataTypeTransformer folder structure META-INF/services/org/apache/camel/datatype instead of META-INF/services/org/apache/camel/ I was wondering if it is worth to use the "default" folder structure META-INF/services/org/apache/camel/transformer.
I've implemented a sanitizer so that file names are equal to the original ones, but it can be removed if yo are fine with files with : or +.

Yeah okay yeah so @christophd created this by hand at first, and yeah we should migrate to make it standard and generated like everything else in camel-core project.
And since they are transformer then they should ideally use META-INF/services/org/apache/camel/transformer as the folder name. This is what the JdkService would do if you specify transformer as the name.
And then you need to change the DefaultTransformerResolver to use the correct location.
For Camel 4.2 we can then add in the release notes about this change (though no camel end users is using this)

After this and the release 4.1.0 of camel-kamelets, we can maybe move the transformers from camel-kamelets repo to camel repo and autogenerating

After this and the release 4.1.0 of camel-kamelets, we can maybe move the transformers from camel-kamelets repo to camel repo and autogenerating

Ah good idea, can we create a JIRA or github issue for this so we wont forget

There are some data type transformer implementations still living in the camel-kamelets-utils library.
Can we use the same SpiGeneratorMojo to auto generate there?

There are some data type transformer implementations still living in the camel-kamelets-utils library.
Can we use the same SpiGeneratorMojo to auto generate there?

Wouldn't make more sense to move them in camel repo?

@davsclaus @oscerd I did not see your comments before writing my last comment. Yes, it makes sense to move the transformers into camel repo and use auto generation there
@Croway +1 on the sanitizer BTW

I mean, we should try to avoid overloading the camel-kamelets library, for example we have also some specific bean there, we should maybe move those in the main repo? WDYT?

the action is failing due to missing files, in particular I have the following locally

Is it safe to push these files? @oscerd @davsclaus

Yes its fine to merge

Go ahead +1 for me


What quality is used for a playlist

Hi,
When downloading a single url, a whole list of quality options is available.
For a playlist only a simple dropdown is shown:

What download quality is used in this case?

Hi,
That's because every video has different set of qualities available.
This will download highest possible quality stream for selected format in case with webm or mp4. In case with mp3 or ogg, it will download highest bitrate audio and convert it.

It would be useful if some selection could be made, and then program will download that format or less. For example if 1080p is available and it's a large playlist, there could be a large difference in bandwidth  / computer space and user may only want 720/480. In a way for a playlist, selecting the desired format is more important :)

That's a good point. Perhaps the selection could instead be "Maximum quality".

Sounds good. Also, for some playlists e.g. from the same author, all videos may be of the same quality.


VMWare - mouse jumping out of window on click in a VM

Hyprland Version
Hyprland, built from branch  at commit 96d555e8e794627bfc561e294e148ab8a9961fcc dirty ().
Bug or Regression?
Bug
Description
When running a Windows 10 virtual machine in VMware Workstation 17.0.2 build-21581411, the mouse jumps out of the window to the top left corner on click.  I'm sure it's something with xwayland and the way hyprland uses mouse bindings, but the issue never occurred on previous hyprland versions.
How to reproduce
Open a windows 10 VM in VMware under hyprland, click on anything.
Crash reports, logs, images, videos
No response

Just to add, this happens when VMWare is set to full screen as well

was fixed in -git


Conform header field in HTTP GET

Loading of the same resources from onInit() controller function or after produce different field in http GET calls:
In the 1st case the GET call have Cache-Control: max-age=0 but in the 2nd not. http://stackoverflow.com/questions/32647522/what-to-insert-in-oninit-function
This behavior produce some caching problems http://stackoverflow.com/questions/32633947/spa-single-page-app-load-always-from-cache

Hi cadupper,
this sounds like a bug to me and i will contact the responsible developers to take a look at the caching issue. They will keep you updated.
It would greatly help if you have a running sample on JSBin or some other code sharing platform so that we can easily reproduce the issue. Can you please provide a code sample for us?
Internal incident id:<PHONE_NUMBER>
Best regards,
Janina

This a simple example: https://jsbin.com/nidawem/edit?js

Hi @cadupper ,
how do you test this behavior? Do you use F5 to reload your application?
I tried to reproduce this but cannot with the following example: http://veui5infra.dhcp.wdf.sap.corp:8080/snippix/
What you mean above is a max-age: 0 in the request headers of the get request, isn't it? The browser sends this header when using F5 to refresh the page. If you do a reload by clicking in the address bar and pressing enter a "reload" should happen and no "refresh" and you should not see that header in the request headers.
Best regards,
Peter

In my opinion this is the behavior: the first time (just after deleting browser cache) that I want call ChangeFragment (in onInit) an DisplayFragment(after) only GET call launched from onInit have in the header max-age: 0.
In this mode only ChangeFragments will be reloaded each time

When loading the app - did you try to click in the address bar and press enter? Then you should not see the "Cache-Control" header in the request. It is a request header your are talking about isn't it?

@akudev informed my that I pasted an internal link and I did not check it properly - I checked it with following example: https://jsbin.com/wuhemaqife/

@petermuessig This is the behavior when:

I press refreh nutton on the url address...

...and then when I press edit butt on my UI loading the change frg

I press enter on the url address...

...and then when I press edit butt on my UI loading the change frg

Hi @cadupper
this is exactly the behavior I was talking about. If you press the refresh button it will send the max-age: 0 header along with all request being sent out for the initial set of resources. All follow up requests will not be affected which will occur due to user interaction. This is the browser behavior and UI5 cannot influence it.
What exactly is your goal? Do you want to make sure that the fragments are loaded again from the backend?
Best regards,
Peter

Yes, my goal is load updated fragments when I change the files in the webserver.
Now only fragments loaded in the onInit function are always loaded NOT from the cache.
Alternatively I would like understand the timing of the cache and how force the refresh

An option if you can control the server would be to use ETags or LastModified headers to force the browser to do conditional requests (304). Would this be an option? We do this e.g. in case of running the server in development mode that all resources get a LastModified header to force the browser to ask the server again each time if a resource has been changed or not.

I could have access to the server, but I would like not configure it because I think that I should eventually config only ui5 framework, not the server.
Ism y problem  really a bug? or only depends form a wrong use of the framework ui5?

Hi Peter @petermuessig ,
I found out the incident is fixed, could you please reply to @cadupper and close the issue.
Best regards,
ilhan


[TS migration] Migrate 'BankAccounts.js' lib to TypeScript

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/24903
PROPOSAL: N/A
Tests

[x] Verify that no errors appear in the JS console

Make sure you are using the staging server (Settings -> Preferences -> Use staging server)

Open Settings -> Wallet
Click on Add payment method -> Bank Account
Press Continue inside the Plaid modal
Select the bank to link and progress with the necessary steps
Select the bank account to use -> bank account should be successfully added
Click on the added bank account -> press Delete -> make sure the account was deleted

Offline tests

N/A
QA Steps

[x] Verify that no errors appear in the JS console

Make sure you are using the staging server (Settings -> Preferences -> Use staging server)

Open Settings -> Wallet
Click on Add payment method -> Bank Account
Press Continue inside the Plaid modal
Select the bank to link and progress with the necessary steps
Select the bank account to use -> bank account should be successfully added
Click on the added bank account -> press Delete -> make sure the account was deleted

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/23176449/0882b590-684d-41ba-8cd3-db8c430e6d69

Mobile Web - Chrome

https://github.com/Expensify/App/assets/23176449/ee629ff7-42e7-42b7-8740-37ccdc53ee2b

Mobile Web - Safari

https://github.com/Expensify/App/assets/23176449/b884c90e-3f7c-4f8a-a5cf-d011f974073a

Desktop

https://github.com/Expensify/App/assets/23176449/c53f9495-035a-4b2c-bb56-2ae77bd78196

iOS

https://github.com/Expensify/App/assets/23176449/5110e486-2a55-45ef-b9ed-adc6b2b89b8f

Android

https://github.com/Expensify/App/assets/23176449/fe263e2c-6f26-469e-86ab-2ece5461fe9a

@blazejkustra The only blocker here is that this PR has OnyxData type declared and your API PR also has it, if it's ok then I can mark it as ready for cross review

Ah, sorry. There is a conflict now.

@tgolen I've resolved conflicts and fixed a couple of TS errors after that

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/tgolen in version: 1.3.85-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.85-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/tgolen in version: 1.3.86-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.86-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Please release new versions

Hi, there is any work to update this docker image to the current versions of n8n (0.233.1:stable)?
Thanks

Hi @germanbravouy
Thanks for reporting. The pipeline is currently broken, we will try to fix ASAP.

Hi thanks @naskio
Is there any update please? We need to upgrade our n8n.
Thanks a lot and great work

Hi! Please any updates on this?
Thanks a lot

Hi, any updates please?
Thanks

Hi @naskio, just to know, is this project gonna receive new releases?
Thanks


Cheqd Cosmos Subql Starter

Got this error: "ERROR Failed to init api service Error: Request failed with status code 502" @web3cdnservices

@seandotau
Rpc now active.
PS don't forget  about the last block.
https://rpc.cheqd.nodestake.top/status
All public nodes - not archive.


Fix issues with while loop and trailing slash due to using sh instead of bash

When run using Sh, while loop is not being interpreted correctly, also the trailing slash at line 49

Thank you very much! I test the new shell and it works.


Discussion: user disposal and plugins

We now keep track of tournaments in which users currently are

It seems like we should use some sort of more-general model for this...

That was not the sort of comment I was expecting but you are right.
We should then think of an API to expose to plugins so that they also get rid of inactive users.

I've actually been thinking of a general model for this.
Really just two-way tracking between users and rooms in which they're participating in things.
For battle rooms, that'd be rooms they're battling in. For chat rooms, that'd be tournaments or room script games (limit 1 at a time).

Yea. They need to be standardized though.

user -> plugin map would be the easiest I think. A plugin would just call methods like user.onJoinEvent(plugin_id, event_id) or user.onLeaveEvent(plugin_id, event_id), and we set/clear everything at users.js side.
plugin -> user map is gonna be an issue. We may call plugin.onDestroyUser(plugin_id, event_id)... but we need to set up such a method for every plugin in which is relevant <_> Anything simpler than that is not viable, as there are plugins that store users in plain objects, others in maps, or sets, and some even in multiple places at the same time. The last one is the case of tournaments actually, and I am not satisfied enough with just disqualifying them :|

Won't have time for this for a while. Definitively not this week...
(Why can't PRs just be transformed into issues with a click? :|)

I mean, I said two-way map. Every user has a roomid -> holdtype mapping, every room has a userid -> holdtype mapping.
I say "hold" because if a user has any holds, they should be prevented from changing their name. The battle puts a "hold" on their username.
holdtype can be battle, tournament, or the name of a plugin (maybe namespaced, so it'd be like #scavengers so plugins wouldn't conflict with possible future official hold types)

Actually, new idea: instead of the concept of "hold", we have the concept of "player".
room.players works like room.users except only tracking stuff.
And users would have roomPlayerCount in addition to roomCount for the reverse mapping.
We don't need holdtype, rather, we specifically have room.game specify exactly what type of player the room has. I don't think there's a need for a room to simultaneously run a tournament and a scripted plugin-game, anyway.

they should be prevented from changing their name
What about unrated battles? Also tournaments are specifically coded to support player renames...
I don't think there's a need for a room to simultaneously run a tournament and a scripted plugin-game, anyway.
Sure, there isn't a need. But making it impossible to hold both a tournament and a short game, or even a trivia doesn't sound reasonable; after all, some people may attend the tournament and other people may play the alternate game. Also, there is the issue that it's planned (tagging @kotarou3 to confirm) to support multiple tournaments in the same room -specifically the Tournaments room.
--
Also, I am not convinced that we should pledge to linking plugins to rooms. What about chat games running across multiple rooms? (I think that was the original idea of Scavengers?)

Re: Multiple tournaments in a single room
I talked with Zarel about it a while back, and he basically said scrap the idea of multiple tournaments per room.
(This doesn't mean the code can't be easily modified to support multiple tournaments per room. In fact, it was designed with that in mind, and only the glue between room and tournaments need to be changed)

But making it impossible to hold both a tournament and a short game, or even a trivia doesn't sound reasonable; after all, some people may attend the tournament and other people may play the alternate game.

Why? The Staff room has a "don't interrupt serious dicussion" rule precisely because multiple things going on in the same room is really annoying to follow for everyone involved. What we should probably do instead is have something like having scripted games create their own room for the game, like how battles work.

The Staff room has a "don't interrupt serious dicussion" rule precisely because multiple things going on in the same room is really annoying to follow for everyone involved
I thought it was to avoid trivialization of relevant matters? If it were otherwise, it could just be "discuss one thing at a time".
have something like having scripted games create their own room for the game
Sounds very overkill but it's an interesting alternative.

@Slayer95, use > to quote, like in e-mail...

If it were just to avoid multiple things at a time, why not just make it read "discuss one thing at a time"? :|

Well, it's usually unnecessary to be that harsh, but yes, it is annoying when people discuss multiple important things at a time, and people get confused a lot when that happens.
It's the same when it's serious talk and when it's games. Especially since scripted games/tournaments fill the chat log with lots of messages... It seriously doesn't make sense to have more than one going on at a time. Imagine more than one battle going on in a battle room, each battle adding its own "POKEMON lost 26% of its health" messages.

Ok. I have slept over it, and while I am not yet fully comfortable with the idea, I now agree due to scalability.

The proposal doesn't handle the possibility of plugins running across multiple or no rooms hmm....

Multiple rooms: One of the rooms should be designated as the main room.
No rooms: ...why?

I guess it makes more sense as "no specific room" = "server-wide".
Stick it to global room? o_o

What kind of game would be server-wide?

Some sort of hunt? It doesn't need to be a "game" though.
In general, I can imagine chat-enhancing plugins (say instant-messaging-client-like features) that may hold references to users, and need to get rid of them when the User instances are destroyed.

I'm not sure a server-wide game would use this game API. Is there any reason it needs to?
In any case, no one should hold references to users. That just leads to bugs.

Alternatively: You can just create a new room for a server-wide game. It's not like a single additional room is all that problematic.

Fiiine.


rshijack: init at 0.3.0

Motivation for this change
TCP connection hijacker, Rust rewrite of shijack

https://github.com/kpcyrd/rshijack

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] 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.

@marsam Thank you!!

Added a network sniffer for #81418 (cc @JoshuaFern)


clip-path not vendor prefixed

"styled-components": "^2.2.0",
clip-path property is not being properly vendor prefixed.
actual code:
export const TopBlock = styled.div`
  clip-path: polygon(0 0, 100% 0, 100% 99%, 0 73%);
  background: #50E3C2;
  height: 160px;
  margin-top: 0 !important;
`;

output:

You will see the css does not include webkit-clip-path. This is problematic because Safari only respects the webkit version of the property.

@thysultan MDN lists this as unsupported by safari and the prefix doesn’t seem to be respected in a lot of browsers. caniuse also doesn’t list this
https://developer.mozilla.org/en-US/docs/Web/CSS/clip-path
I’m going to leave the decision whether to prefix it up to you, but I’m a little confused on this one

The most compelling argument i can think of is the fact that autoprefixer adds it:
https://github.com/postcss/autoprefixer/wiki/support-list#mask
I reckon that MDN article is out of date. Because I'm using it actively in Safari with success.

/cc @thysultan we should add this based on caniuse:

All the other mask variants are vendor prefixed so it makes sense that we should handle this as well.
Patched in 3.2.20

Thanks @thysultan!


fix(runtime): handle async transforms of same module

Summary
The code currently ends up putting duplicate modules into the module cache, leading to different instances added to the cache (or rather, newer ones overriding old ones) which breaks referential identity.
We now have a sort of mutex on the cache key as well as asserting the cache does not have an entry for the cache key before we put the module into it.
Fixes #11214.

Test plan
Test added

Codecov Report

Merging #11220 (7af697a) into master (420bcb7) will decrease coverage by 0.06%.
The diff coverage is 4.76%.

@@            Coverage Diff             @@
##           master   #11220      +/-   ##
==========================================
- Coverage   64.30%   64.24%   -0.07%
==========================================
  Files         309      309
  Lines       13485    13500      +15
  Branches     3286     3288       +2
==========================================
+ Hits         8672     8673       +1
- Misses       4104     4118      +14
  Partials      709      709

Impacted Files
Coverage Δ

packages/jest-runtime/src/index.ts
53.76% <4.76%> (-1.09%)
:arrow_down:

Continue to review full report at Codecov.

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


add header to invoke for extra logging, this closes #146

Description
Add {X-OW-EXTRA-LOGGING: on} header to all action invoke calls from aio rt
Related Issue
See issue #146
Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

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

Have you tried this feature against an active deployment? I am seeing failures against standalone openwhisk related to authentication. I believe that when a header is specified, npm openwhisk is not adding the basic auth header and invokes fail.
 WSK_CONFIG_FILE=~/.local-wskprops ./bin/run runtime:action:invoke user/login
 ›   Error: failed to invoke the action: the resource requires authentication, which was not supplied with the request (401 Unauthorized)
 ›     specify --verbose flag for more information


Added javascript questions

Summary of changes
Added questions to javascript quiz
Checklist

[x] I have read and followed the contribution guidelines.
[x] I have read through the Code of Conduct and agree to abide by the rules.
[x] This PR is for one of the available issues and is not a PR for an issue already assigned to someone else.
[x] My PR title has a short descriptive name so the maintainers can get an idea of what the PR is about.
[x] I have provided a summary of my changes.

closes #XXXXX

Made the changes as per requested,
Thank you


Implement uv pip build

Expose our PEP 517 build frontend, as an alternative to pypa/build or pip wheel.

Closing in favor of #1510, which has more discussion.


[Last Match] Medals do not align properly

This is happening when some medal title takes up 2 lines of text.
Using inspector, I can force that state, here is the comparison:

This problem will happen with floating columns of various heights. Things like Isotope's 'fitRows' option address this issue. But that introduces a whole library to fix the problem... the simplest way to fix this would be to give those medal columns a class and a specify a height, maybe one that supports three lines of text and hide overflow.

Thank you for the update.

fixed in 99ff64d


Data preprocessing tools

WIP

Would this work on sentences like:
“Now a link has been observed between rising levels of atmospheric CO2 — and other greenhouse gases — and weakening vertical wind shear along the East Coast of the US. That will make it less likely that recent violent episodes of extreme weather will start to dissipate after making landfall, and may instead grow in strength.”
The word “that” is referring to ‘the link between rising levels of atmospheric CO2 and weakening vertical wind shear’...
So we would want the second sentence to have the anaphora replaced to say:
“weakening vertical wind shear will make it less likely that recent violent episodes of extreme weather will start to dissipate after making landfall, and may instead grow in strength.“
if it can’t really work on these sentences then it probably won’t be very useful and another approach will likely be needed where the previous sentence is parsed as a node with multiple slots filled and then that node concept is linked to the future sentences that refer to that whole concept (similar to the sentences above).

Thanks for the concrete example


Add reverse battery protection

Need a diode between 12V and the VXO7805 power module.
I missed that the power module doesn't have reverse battery protection.
That became apparent when I accidentaly hooked the power up backwards.
That killed the power module which put 12V on the 5V output and killed the CAN transceiver.
Everything else was protected by the 3.3V regulator.
Profets were protected according to their datasheet.

Included on v7.4 PCBs


Add flexibility for LB component names

Current module only allows for fixed Backend service, health check and fw rule names and isn't useful when using terraform import as terraform will try to re-create the resources.
This PR offers flexibility around the naming convention.

Thanks for the PR! 🚀
Unfortunately it looks like some of our CI checks failed. See the Contributing Guide for details.

⚠check_whitespace
Failed whitespace check. More details below.

Checking for trailing whitespace
Checking for missing newline at end of file
Error: No newline at end of file ./variables.tf


Vulnerability roundup 114: ncurses-6.3: 1 advisory [7.1]

search, files

[ ] CVE-2022-29458 CVSSv3=7.1 (nixos-unstable)

CVE details
CVE-2022-29458
ncurses 6.3 before patch 20220416 has an out-of-bounds read and segmentation violation in convert_strings in tinfo/read_entry.c in the terminfo library.

Scanned versions: nixos-unstable: f419dc5763c.

@trofi you have experience with ncurses. Could you take a look at this, please?

Sounds good. Will do.

Fix announce: https://lists.gnu.org/archive/html/bug-ncurses/2022-04/msg00016.html . ncurses snapshot from 20220416 should contain the fix: https://github.com/mirror/ncurses/blob/master/NEWS
20220416
	+ add a limit-check to guard against corrupt terminfo data
	  (report/testcase by NCNIPC of China).
...

Created an update to latest snapshot as https://github.com/NixOS/nixpkgs/pull/172734 against staging (still draft to test against reverse dependencies locally).


After Installing Plugin: Cannot build from CLI (xcworkspace works perfectly)

First and foremost, the library seems great and frankly I'm not sure if this is the plugin, some environment setup issue, or what, but after installing this plugin I am no longer able to build for iOS from the CLI. i.e., cordova run ios. If I open xcworkspace and run/build/archive it works perfectly.
I have done about 2 days of research into various things like Cocoapods, etc. in order to try and figure it out, but there's not much information on them problem at hand.
I found an SO post saying to modify the pods .m files to reference the relative path of the linked libraries, but that's just crazy. And all the other configuration such as setting the Header Search Paths seems correct... the error I'm getting at the moment is:
FIRNetwork.m fatal error GoogleToolboxForMac/GTMNSData+zlib.h not found
Again, your code/plugin is working great! I'm just hoping you may have had some insight into similar setup issues. If you don't, then feel free to close and ignore. I'll also be creating a SO post.

Hello!
Have been solved yet?
Thanks!

@LimitTech no it hasn't, but see https://github.com/apache/cordova-ios/issues/394
There are outstanding, known issues with using cocoapods we had to use cordova-plugin-firebase as our plugin of choice because it doesn't use cocoapods. Instead it opts to include the source files along with the package.
The issue with that option is that it makes it hard to integrate with other firebase plugins because the versions may not be compatible.

PR with the fix for cordova cli was merged, so I believe the issue will be resolved in the next release.


Issue with k8s.io/docs/tutorials/hello-minikube/

This is a Bug Report

Problem:
Following the hello-minikube tutorial with Katacoda, I was not able to run the following command:
$ minikube addons disable metrics-server
*
X disable failed: running callbacks: [addon apply: sudo KUBECONFIG=/var/lib/minikube/kubeconfig /var/lib/minikube/binaries/v1.17.3/kubectl delete -f /etc/kubernetes/addons/metrics-apiservice.yaml -f /etc/kubernetes/addons/metrics-server-deployment.yaml -f /etc/kubernetes/addons/metrics-server-service.yaml: exit status 1
stdout:

stderr:
the path "/etc/kubernetes/addons/metrics-apiservice.yaml" does not exist
the path "/etc/kubernetes/addons/metrics-server-deployment.yaml" does not exist
the path "/etc/kubernetes/addons/metrics-server-service.yaml" does not exist
]
*
* minikube is exiting due to an error. If the above message is not useful, open an issue:
  - https://github.com/kubernetes/minikube/issues/new/choose

I executed all the commands in the order presented on the page.
Proposed Solution: It looks like some files should be in /etc/kubernetes/addons to fix this.
Page to Update:
https://kubernetes.io/...

@BenHall this might be relevant to you

@bartier I executed for me successfully on KataKoda environment.
$ minikube addons enable metrics-server

The 'metrics-server' addon is enabled
$ minikube addons disable metrics-server
"The 'metrics-server' addon is disabled
$ minikube addons list
|-----------------------------|----------|--------------|
|         ADDON NAME          | PROFILE  |    STATUS    |
|-----------------------------|----------|--------------|
| dashboard                   | minikube | enabled ✅   |
| default-storageclass        | minikube | enabled ✅   |
| efk                         | minikube | disabled     |
| freshpod                    | minikube | disabled     |
| gvisor                      | minikube | disabled     |
| helm-tiller                 | minikube | disabled     |
| ingress                     | minikube | disabled     |
| ingress-dns                 | minikube | disabled     |
| istio                       | minikube | disabled     |
| istio-provisioner           | minikube | disabled     |
| logviewer                   | minikube | disabled     |
| metrics-server              | minikube | disabled     |
| nvidia-driver-installer     | minikube | disabled     |
| nvidia-gpu-device-plugin    | minikube | disabled     |
| registry                    | minikube | disabled     |
| registry-creds              | minikube | disabled     |
| storage-provisioner         | minikube | enabled ✅   |
| storage-provisioner-gluster | minikube | disabled     |
|-----------------------------|----------|--------------|

@bartier Here is additional information that would be helpful:

The exact minikube start command line used

The full output of the minikube start command, preferably with --alsologtostderr -v=4 for extra logging.

The full output of minikube logs

The full output of kubectl get po -A

Thank you for sharing your experience!

/triage needs-information

@RA489 I tested today again and got success. Guess They've updated the VM config or something else in this meantime.
I'll close this issue since the scenario is not reproduceble anymore.
Thanks!


PyBag/Android porting

See: https://github.com/pygame-web/pkg-porting-wasm/issues/1
I know this has also been an issue with android. the freetype submodule just doesn't work very well outside the core platforms. With pygame.font adding new harfbuzz features allowing support for new languages - moving back to pygame.font is a priority.

so   i think   to start with it would be a good idea to make a list of   related code (code that may need change to facilitate pygame-web)
I'll start
https://github.com/MyreMylar/pygame_gui/blob/823474c31191116fe13fd778e498c78f5774cadc/pygame_gui/core/ui_font_dictionary.py

Since porting is mostly a font issue for web and download font size too is also an issue for mobile connections.
i'd suggest to make a separate font loader and use "fonts packs" for the 4 letters code given to set_script
i think making a "pygame.i18n" module would be benefecial to everyone : and pygame_gui is the ideal sandbox for making that happen.
i18n would:
-download a font pack for a code [async]
-preload all localized string ( maybe download them too )  [async]
if success:
-set the script
-monkey patch font.render to use a bidi algo.
else:
stay on C locale.


Bazel

This is still very exploratory, but anyone else trying to use bazel with Error Prone will need this.


Auto Rename Variables

As a developer,
I need to be able to modify variable names with synonyms of the original variable name by traversing a concrete syntax tree,
So that a semantic clone can look structurally different than it's seed source code.
Details and Assumptions

CST must be set up for traversing the program node types
Program can be rebuilt with the new changes

Acceptance Criteria
Given that I want to generate clones from a seed Python file,
When I enter the program into the file accept box,
Then the output file contains changes in variable names that are closely related to the original variable names.

Closed as of most recent commits for main and python-notebook-tests branches. Variable renaming is fully covered with a backup animals wordbank if Gemini synonym requests fail.


Can not get Variables, I try to run HIDDEN PRIVATEKEY and HIDDEN DOT ONION, but there is no return value

Can not get Variables, I try to run HIDDEN PRIVATEKEY and HIDDEN DOT ONION, but there is no return value

As I mentioned in apg/hidden-polls#1, you can obtain these after you do a git push.

I have successfully git push,
But were unable to obtain these variables

You don't need to get these variables from a Heroku app. They are the same values you get when you run any hidden service. If you have tor setup on your machine, expose a hidden service, find the values that get created and use those.


supporting multiple nested translation files

It would be neat if you could support multiple files with same "namespaces" like this:
{
  "module": {
      "submodule1": {
           "translation": "lalala"
      }
}

{
  "module": {
      "submodule2": {
           "translation": "lalala"
      }
}

Right now these should be in same file like so:
{
  "module": {
      "submodule1": {
           "translation": "lalala"
      },
      "submodule2": {
           "translation": "lalala"
      }
}

But being able to split them into several files would be nice =)

Would be nice. For instance a good use case is having an "error" property for packages.
Example: 2 packages make calls to methods. Each would have their i18n json data:
"errors": {
  "package1": "Package #1 error"
}

"errors": {
  "package2": "Package #2 error"
}

Any news about this? This is strongly needed to keep big translations readable.


Could not execute GraphicsMagick/ImageMagick: identify "-ping" "-format" "%wx%h" "chang1.png" this most likely mean s the gm/convert binaries can't be found

Why is this happening? I have installed ImageMagick for windows.
Error: Could not execute GraphicsMagick/ImageMagick: identify "-ping" "-format" "%wx%h" "chang1.png" this most likely mean
s the gm/convert binaries can't be found
at ChildProcess. (C:\Users\admin\AppData\Roaming\npm\node_modules\generator-rn-toolbox\node_modules\gm\lib
command.js:232:12)
at emitOne (events.js:96:13)
at ChildProcess.emit (events.js:188:7)
at ChildProcess.cp.emit (C:\Users\admin\AppData\Roaming\npm\node_modules\generator-rn-toolbox\node_modules\cross-spawn
\lib\enoent.js:36:37)
at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
I also try  convert --version . It displays like this......
PS F:\ReactNativeProject\MyApp> convert --version
Invalid drive specification.

Hi @TonyGao625,
Thanks for posting an issue here!
I see that you're on Windows, can you try to reinstall image magick with the legacy tools, to see if it works?
You can see how to do it here.

Hi @TonyGao625, do you still have an issue with the generator, or did that fix your problem?

@Almouro I did it and worked for me ;)
Then had another problem, but I'll open a new Issue for it.
Thanks

Ok thanks! closing this one then :)

@Almouro having this same issue (already reinstalled with legacy as suggested above).

I have same issue too after installed gm and imagemagick.
err: Error: Could not execute GraphicsMagick/ImageMagick: identify "-ping" "-format" "%wx%h" "/var/folders/nv/1m164hpn5038mj9t4s4vfjgc0000gn/T/upload_2d1b64f289c31203f77b63e8ff1154bd" this most likely means the gm/convert binaries can't be found

@jeanru I meant to come back here and update - but I solved mine by running cmd.exe as an administrator.
Basically the executable is unable to find/communicate with ImageMagick/Graphicksmagick, so as long as one of the two are installed locally, then something else is hindering communication. It could be a permissions issue - thus running the command line as an administrator. Hope it's the same for you.

Yes, @joebjoe  has the right solution. Open the command promt or the gitbash as Administrator. It works. Thanks

I have tried run command promt as admin but not working same issue.

same issue, windows app not working


Add dependabot auto-merge workflow & Fix dependabot.yml

This PR fixes the working directory used by dependabot.yml and adds a new workflow that checks if a new PR was made by dependabot in which case it tries to automatically merge it.
Follow up from JoeStrout/miniscript#73
Since Farmtronics doesn't have any other CI to run tests or build the project automatically, I'm not sure if the auto-merge command will work correctly.
But if it doesn't you would only need to manually merge the dependabot PRs for miniscript to update the version here.

After creating this PR I realised it might be better to directly merge these PRs instead of using the auto-merge feature, so I changed that with my second commit.


node:internal/modules/cjs/loader:1228   throw err;   ^

Error: Cannot find module 'C:\Users\eplan\src\process\init-agent.js'
at Module._resolveFilename (node:internal/modules/cjs/loader:1225:15)
at Module._load (node:internal/modules/cjs/loader:1051:27)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:174:12)
at node:internal/main/run_main_module:28:49 {
code: 'MODULE_NOT_FOUND',
requireStack: []
}
Node.js v20.18.0
Agent process exited with code 1 and signal null
Agent process ./andy.json exited too quickly and will not be restarted.
All agent processes have ended. Exiting.

[ './andy.json' ]
Using chat settings: { model: 'gpt-4o-mini', api: 'openai' }
Using embedding settings: { api: 'openai' }
Error with embedding model, using word overlap instead.
Error with embedding model, using word overlap instead.
Logging in...
C:\Users\eplan\Downloads\mindcraft-main\mindcraft-main\node_modules\mineflayer\lib\loader.js:108
const version = require('minecraft-data')(bot._client.version).version
^
TypeError: Cannot read properties of null (reading 'version')
at next (C:\Users\eplan\Downloads\mindcraft-main\mindcraft-main\node_modules\mineflayer\lib\loader.js:108:67)
at createBot (C:\Users\eplan\Downloads\mindcraft-main\mindcraft-main\node_modules\mineflayer\lib\loader.js:105:34)
at initBot (file:///C:/Users/eplan/Downloads/mindcraft-main/mindcraft-main/src/utils/mcdata.js:57:15)
at Agent.start (file:///C:/Users/eplan/Downloads/mindcraft-main/mindcraft-main/src/agent/agent.js:27:20)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Node.js v20.18.0
Agent process exited with code 1 and signal null
Agent process ./andy.json exited too quickly and will not be restarted.
All agent processes have ended. Exiting.

same issue here, will update you if I fix it 👍

ok I fixed it you need to use the command "npm run postinstall" to apply the patches to mineflayer


error$ doesn't clear

Please provide us with the following information:

This issue is for a: (mark with an x)
- [] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [x] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

I have an entityService.
I make a call to the middleware on entityService.add() or update().
The middleware returns an error.  Ie.  Description is required. (error code 400)
I tried clearCache, a successful update, add but the error$ is persistant.
It should clear on a successfull action.
Any log messages given by the failure

Expected/desired behavior

To have the error$ not be persistent and clear.

Is it possible to have a clearError like clearCache.
OS and Version?

macOs High Sierra version 10.13.6

Versions
ngrx-data 6.0.2-beta.10
"rxjs": "^6.2.2",
"@ngrx/effects": "^6.1.0",
"@ngrx/entity": "^6.1.0",
"@ngrx/router-store": "^6.1.0",
"@ngrx/store": "^6.1.0",
"@ngrx/store-devtools": "^6.1.0",
"@angular/core": "^6.1.3",
Mention any other details that might be useful
I upgraded from
ngrx-data: 6.0.0-beta.6.
rxjs: 6.0.0,
@ngrx 6.0.1
"@angular/core": "^6.0.0",
Windows 10 Pro
and the functionality worked perfectly.

Thanks! We'll be in touch soon.

I'm confused. Ngrx-data doesn't have an error$ observable. At least I can't find it.
There is such an observable in the sample app in the villain-editor.component.ts.
What am I missing?

Thanks
To me it makes sense to just clear the error$ every time its loading.


Add support for displaying data column information if it exists

GeoPackage supports adding column information to tile and feature tables via DataColumns.  If these exist, we should show the user that information when they are viewing the table information.

Fixed by pull #7