Embedding values from this library not matching SentenceTransformers

Hi, I tried comparing the values from here with those from SentenceTransformers and they are not the same:
from sentence_transformers import SentenceTransformer

model2 = SentenceTransformer("TaylorAI/bge-micro-v2")

from mlx_embedding_models.embedding import EmbeddingModel
model = EmbeddingModel.from_registry("bge-micro")
texts = [
    "isn't it nice to be inside such a fancy computer",
    "the horse raced past the barn fell"
]
embs = model.encode(texts)
print(embs.shape)
embs2 = model2.encode(texts)
# check if the embeddings are the same
import numpy as np

print(np.allclose(embs, embs2, atol=1e-5))

Output is false. Increasing atol doesn't work.
np.dot(embs[1], embs2[1]) is 0.94

This seems to be specific to bge-micro (I get True with bge-small) which makes me think this might be a pooling or configuration bug specific to this model. Looking into it.

Confirmed this is a pooling bug. This model uses mean-pooling, but when it's pulled from HuggingFace with SentenceTransformers, it somehow ends up with a class-pool instead, even though the configuration says it should use mean pooling in 1_Pooling/config.json.

Fixing pooling didn't resolve this yet, gotta dig deepers. Sorry! In the meantime, confirmed bge-small works so use that if you can.

Thanks for looking into it!
FYI, I have integrated this library into Vector-io to reembed text data with different models on Mac.

https://github.com/AI-Northstar-Tech/vector-io/blob/main/src%2Fvdf_io%2Fscripts%2Freembed.py#L548-L560

a quick rerun gives this error:
{
	"name": "AttributeError",
	"message": "module 'mlx.core.metal' has no attribute 'clear_cache'",
	"stack": "---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[1], line 12
      7 model = EmbeddingModel.from_registry(\"bge-micro\")
      8 texts = [
      9     \"isn't it nice to be inside such a fancy computer\",
     10     \"the horse raced past the barn fell\"
     11 ]
---> 12 embs = model.encode(texts)
     13 print(embs.shape)
     14 embs2 = model2.encode(texts)

File ~/miniforge3/lib/python3.10/site-packages/mlx_embedding_models/embedding.py:259, in EmbeddingModel.encode(self, sentences, batch_size, show_progress, **kwargs)
    257         del batch
    258     # we're done with this seqlen, clear the cache
--> 259     mx.metal.clear_cache()
    261 # concatenate embeddings and reverse the sort
    262 output_embeddings = mx.concatenate(output_embeddings, axis=0)

AttributeError: module 'mlx.core.metal' has no attribute 'clear_cache'"
}

debugging further...

Did you upgrade mlx_embedding_models to the latest version (now 0.0.9)? Does it have significant error still? (interested in the allclose but also the cosine similarity

Yes, it's the latest

This library's outputs for the bge-large and bge-m3 models also don't match the corresponding outputs from the FlagEmbedding library.

Is it just me or does bge-m3 already support running on the GPU with Apple Silicon? The GPU history window in Activity Monitor also shows similar GPU usage whether I run using HuggingFace's sentence_transformers library directly or using this library.
import timeit
from sentence_transformers import SentenceTransformer
model = SentenceTransformer("BAAI/bge-m3")
timeit.timeit(lambda: model.encode(["hi" * 100] * 100), number=10)
# 22.953382124999735
model.encode(["hi"])
# array([[-0.0536464 ,  0.00515981, -0.03156308, ...,  0.01571097,
#         -0.03756364,  0.04886195]], dtype=float32)

import timeit
from mlx_embedding_models.embedding import EmbeddingModel
model = EmbeddingModel.from_registry("bge-m3")
timeit.timeit(lambda: model.encode(["hi" * 100] * 100), number=10)
# 25.65148633300032
model.encode(["hi"])
# 100%|██████████████████████████████| 1/1 [00:00<00:00, 11.03it/s, seq_len=16]
# array([[-0.04499852,  0.02141248, -0.02566153, ..., -0.0094963 ,, seq_len=16]
#         -0.01805216,  0.05820245]], dtype=float32)

The first output matches the Colab that uses the FlagEmbeddings library:
model.encode(["hi"])
# {'dense_vecs': array([[-0.05365,  0.0051 , -0.03156, ...,  0.0157 , -0.03754,  0.04886]],
#        dtype=float16),
#  'lexical_weights': None,
#  'colbert_vecs': None}

here's the notebook for my check: https://colab.research.google.com/drive/1-2bJvoMuYun61QtkBwgmqwwJ0g6TdANI?usp=sharing

@jyc is the mlx version faster or use less memory compared to SentenceTransformer using bgem3 ?
Thanks

@x4080 No; see the output of timeit that I posted above. I also ran into a pretty severe memory leak which went away when I used the transformers library directly.

@jyc  Thanks for quick answer, I tested using onnx and it seems your code is faster (both mlx and sentencetransformers) - so you suggest best using mlx ? with the memory leak in sentencetransformers ?

@x4080 Personally I would recommend using the HuggingFace transformers library directly, because ultimately this library and SentenceTransformers are both just wrappers. I don’t think you want to use this library because it does not compute the embeddings correctly, seeing as the outputs differ from the outputs computed using the reference BAAI/bge-m3 code on Colab.
Even if you didn’t care about the outputs matching the reference library, the fact that they don’t match means it’s likely that the properties you’re looking for in a text embedding are not preserved.
You can use something like this. I extracted this code from my system which has been running for a few weeks now. I haven’t tested the excerpt but any necessary tweaks should be small.
import torch
from transformers import AutoModel, AutoTokenizer

model = "BAAI/bge-m3"
device = "mps"
tokenizer = AutoTokenizer.from_pretrained(model, device_map=device)
model = AutoModel.from_pretrained(model, device_map=device)
model.eval()

# If you want to save to a local directory, run the following once and then
# replace `model` with `local_path`.
# local_path = "./models/bge-m3"
# model.save_pretrained(local_path)
# tokenizer.save_pretrained(local_path)

encoded = tokenizer(
    input, padding=True, truncation=True, return_tensors="pt"
).to(device)

with torch.no_grad():
    output = model(**encoded)
    embeddings = output[0][:, 0]
    embeddings = torch.nn.functional.normalize(embeddings, p=2, dim=1)

@jyc Thanks man, very kind of you, I'll test your code

@jyc
Thanks for this - have you benchmarked mps versus mlx for speed*? I may be out of date, but mps was very slow compared to CUDA last time I checked it.
I've managed to fix memory leaks for batch embeddings using a custom implementation of MPNet, but I also can't get the results to exactly match the ST/HF results, and the downstream differences are huge (e.g. on clustering tasks or similar)
*clearly if the outputs aren't comparable then speed is basically irrelevant, but we should be able to fix the outputs - or at least get them close enough ( as an aside - different GPUs output slightly different results for embeddings from my experience, whether using ST or HF - it's in the hardware - the question is what's a tolerable difference).


[Event Request] table 5901 "Service Item Line" - OnBeforeCheckFinishingTime

Hello, would it be possible to add OnBeforeCheckFinishingTime event to table 5901  "Service Item Line", like this:
        field(17; "Finishing Time"; Time)           trigger OnValidate()
...
               SkipResponseTimeHrsUpdate := true;
               TestField("Finishing Date");
               GetServHeader();
               if "Finishing Time" <> 0T then begin
                   IsHandled := false;
>                    OnBeforeCheckFinishingTime(Rec, IsHandled); //NewEvent
                   if not IsHandled then
                       if ("Finishing Date" = "Starting Date") and
                          ("Finishing Time" < "Starting Time")
                       then
                           Error(
                             Text022,
                             FieldCaption("Finishing Time"),
                             FieldCaption("Starting Time"));
                   UpdateStartFinishDateTime("Document Type", "Document No.", "Line No.", "Starting Date", "Starting Time",
                     "Finishing Date", "Finishing Time", false);
               end;

   [IntegrationEvent(false, false)]
   local procedure OnBeforeCheckFinishingTime(var ServiceItemLine: Record "Service Item Line"; var IsHandled: Boolean)
   begin
   end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10868.


Fix example in README.md

fixes the example for coffee 0.3.
It's correct in the docs but not in the readme.

I just hit this too @hecrj.

I guess this is a side-effect of trying to keep all the docs updated :sweat_smile:
Would it help if we move the warning about breaking changes right after the example?

Ah I see, I missed that. I think it was partly to do with my copying the version line (coffee = { version = "0.3", features = ["opengl"] }) and then grabbing the example code below.
Docs are hard! Thanks 🙏


WIP: Improve side navigation

Hey, I just made a Pull Request!
Solves #472
I'm creating a draft PR for now since this is quite the big change.
Subtasks

[x] Change transition from shifting to fading
[ ] Add Search component
[ ] Add nav links

[ ] Selected indicators

[ ] User info
[ ] Starred behaviour
[ ] Dismissable ephemeral flash cards thingies

:heavy_check_mark: Checklist

[ ] All tests are passing yarn test
[ ] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[ ] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes

@Koroeskohr Had a quick look at the current changes, and wondering why the change to fade animation instead of the current slide animation?

I just followed the specs that @katz95 formalized in this GIF in the issue ticket
https://user-images.githubusercontent.com/61153904/78457453-8e2c3080-76aa-11ea-9be0-2e377eb3fd8e.gif

@Koroeskohr ah ok, I think the animation in that mockup is just a limitation in figma, but not sure.
@katz95 what do you think about the animation of the sidebar, keep what we have right now or make it animate like in the mockup?

@Rugvip @Koroeskohr keep what we have right now!! thank you!! 🙏 i will get back to you with answers to your questions as well!! hold tight 😄

First draft with all 3 nav groups looks like this so far. Still missing user component. I'll try and make a story out of it so that it's browsable in the PR storybook preview

👏

Hey @Koroeskohr any updates on this PR? No stress, just checking in 💯

Hey @Koroeskohr any updates on this PR? No stress, just checking in 100

Yup, redid a few things, I just need to do a few more modifications because there were some misleading items in the spec, should not be too long

I wonder if a recent bump of react-router broke this:
src/layout/Sidebar/Items.tsx:142:9 - error TS2322: Type '<Params extends { [K in keyof Params]?: string | undefined; }>(match: match<Params> | null) => boolean | null' is not assignable to type '<Params extends { [K in keyof Params]?: string | undefined; }>(match: match<Params> | null, location: Location<PoorMansUnknown>) => boolean'.
  Type 'boolean | null' is not assignable to type 'boolean'.
    Type 'null' is not assignable to type 'boolean'.

142         isActive={(match) => match && !disableSelected}
            ~~~~~~~~

  ../../node_modules/@types/react-router-dom/index.d.ts:75:5
    75     isActive?<Params extends { [K in keyof Params]?: string }>(match: match<Params> | null, location: H.Location<S>): boolean;
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    The expected type comes from property 'isActive' which is declared here on type 'IntrinsicAttributes & NavLinkProps<PoorMansUnknown> & RefAttributes<HTMLAnchorElement>'
src/layout/Sidebar/Items.tsx:156:7 - error TS2322: Type '<Params extends { [K in keyof Params]?: string | undefined; }>(match: match<Params> | null) => boolean | null' is not assignable to type '<Params extends { [K in keyof Params]?: string | undefined; }>(match: match<Params> | null, location: Location<PoorMansUnknown>) => boolean'.
  Type 'boolean | null' is not assignable to type 'boolean'.
    Type 'null' is not assignable to type 'boolean'.

156       isActive={(match) => match && !disableSelected}

@Rugvip any suggestions?

Seems to be a flaky test in CI. Could not reproduce. MERGING NOW! 😄


Using data-uri with an image macro to display a PDF only shows a empty square

If I have :data-uri: at the beginning of the document then use the image macro to display a PDF file, it shows an empty square instead of the image. I'm using it like this:
image::MyImage.pdf[]
If I remove :data-uri: from the file then it displays the PDF.

I didn't mean to write this in in asciidoctor-pdf so I closed and opened a separate one in the asciidoctor repo.

I didn't mean to write this in in asciidoctor-pdf so I closed and opened a separate one in the asciidoctor repo.


Can I change the name of the schema once it has been created/used?

Issue in ./help/xdm/schema/composition.md
It would be good to add this info to the table of either allowed or breaking changes!  Someone from engineering added new schemas and datasets to the prod sandbox of my customer and they did not like the chosen naming and did not think they could change it themselves. I haven't tried it so I don't know if the system lets you make this change or not.

Thank you for logging this issue. The documentation will be reviewed and updated as soon as possible.
Internal tracking: https://jira.corp.adobe.com/browse/PLAT-98616


Requests fails with consul exception

We shouldn't fail with just a consul exception, we should raise a custom RequestsException instead...
sample fail:
Apr 15 13:41:30 alamo-worker2 alamo_worker_52[10343]: Traceback (most recent call last):   File "/srv/alamo_worker/bin/alamo-worker", line 10, in <module>     sys.exit(main())   File "/srv/alamo_worker/lib/p
ython3.4/site-packages/alamo_worker/cli.py", line 135, in main     management.execute()   File "/srv/alamo_worker/lib/python3.4/site-packages/alamo_worker/cli.py", line 124, in execute     result = self.mana
ger.dispatch(check)   File "/srv/alamo_worker/lib/python3.4/site-packages/alamo_worker/plugins/__init__.py", line 161, in dispatch     _plugin.execute(payload)   File "/srv/alamo_worker/lib/python3.4/site-pa
ckages/alamo_worker/plugins/http_check.py", line 89, in execute     response = call_method(**call_kwargs)   File "/srv/alamo_worker/lib/python3.4/site-packages/requests/sessions.py", line 480, in get     ret
urn self.request('GET', url, **kwargs)   File "/srv/alamo_worker/lib/python3.4/site-packages/requests/sessions.py", line 468, in request     resp = self.send(prep, **send_kwargs)   File "/srv/alamo_worker/li
b/python3.4/site-packages/requests/sessions.py", line 576, in send     r = adapter.send(request, **kwargs)   File "/srv/alamo_worker/lib/python3.4/site-packages/requests/adapters.py", line 343, in send     c
onn = self.get_connection(request.url, proxies)   File "/srv/alamo_worker/src/requests-consul/requests_consul/adapters/service.py", line 119, in get_connection     url = self._build_instance_url(parsed).getu
rl()   File "/srv/alamo_worker/src/requests-consul/requests_consul/adapters/service.py", line 81, in _build_instance_url     instances = self._fetch_instances(service_name)   File "/srv/alamo_worker/src/requ
ests-consul/requests_consul/adapters/service.py", line 73, in _fetch_instances     service_name, dc=dc)[1])   File "/srv/alamo_worker/lib/python3.4/site-packages/consul/base.py", line 1145, in service     '/
v1/catalog/service/%s' % service, params=params)   File "/srv/alamo_worker/lib/python3.4/site-packages/consul/std.py", line 34, in get     self.session.get(uri, verify=self.verify)))   File "/srv/alamo_worke
r/lib/python3.4/site-packages/consul/base.py", line 111, in cb     raise ConsulException("%d %s" % (response.code, response.body)) consul.base.ConsulException: 500 No cluster leader\

How about retry couple times?

Well, we could for the next release, question is how many times should we do it ?

Configurable, default: 3 ;)


Remove bfadmin from main buildfarm repo

Buildfarm Admin has been moved to its own repo under this org (https://github.com/buildfarm/bfadmin). We can now delete it from the main buildfarm repo.

Don't forget .bazelci/test_bfadmin.sh, too


Optimize the "Hash" and make it faster

github.com/seiflotfy/cuckoofilter first calculate the hash when it's in initialization.
And I try this to optimize this "better" version of cuckoofilter.

BenchmarkFilter_Insert
BenchmarkFilter_Insert-8   	38737519	        29.49 ns/op
BenchmarkFilter_Lookup
BenchmarkFilter_Lookup-8   	44468107	        27.07 ns/op
BenchmarkFilter_Reset
BenchmarkFilter_Reset-8    	   97950	     12242 ns/op
PASS

@panmari

Thanks for breaking up the evaluation!
I would still prefer merging/discussing this as separate pull requests. Could you please split them up? See also comments above.


[WIP][HUDI-797] Small performance improvement for rewriting records.

When adding HUDI metadata field to schema, the position of incoming schema fields is retained. This allows us to lookup fields in exiting record and assign to the new record using field positions (array index lookup). This is faster than looking up fields using name (HashMap based lookup).
What is the purpose of the pull request
Small performance improvement for rewriting records during the ingestion phase. HUDI-797 as the details on the usecase and improvements.
Brief change log

Modified HoodieAvroUtils.addMetadataFields to retain the field positions.
Added a new rewriting function HoodieAvroUtils.rewriteHoodieRecord which rewrites using field positions rather than field names.

Verify this pull request
This change is verified automatically by all the HUDI client tests which ingest data into HUDI.
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@lamber-ken could you please review this

@lamber-ken could you please review this

OK : )

HoodieParquetRealtimeInputFormat has hardcoded the positions of the HUDI specific metadata fields.
// These positions have to be deterministic across all tables
public static final int HOODIE_COMMIT_TIME_COL_POS = 0;
public static final int HOODIE_RECORD_KEY_COL_POS = 2;
public static final int HOODIE_PARTITION_PATH_COL_POS = 3;
public static final String HOODIE_READ_COLUMNS_PROP = "hoodie.read.columns.set";
This is incompatible with the changes required in this PR. Am looking into the reason for the above and whether I can work around it.

@prashantwason the reason for these is to additionally project out the _hoodie_record_key field needed to merge base and log files.. Hive will only hand the column numbers to be read for a given query.. Hope that helps

Meta fields have to come first.. and I think its too late to change that in hudi , as lots of data have been written already :)

@prashantwason : Looking at the comments, it looks like this PR is going to be abandoned. If so, can you please close this PR.

Since the Hoodie Metadata fields have to be added at fixed positions, the other fields of the schema will get an "offset" position. This offset cannot be guaranteed to be fixed (= num_hoodie_metadata_fields) as the existing schema may have a field named with hoodie metadata or for any other reason.
Hence, for sake of data integrity, we cannot refer to fields by position in the current design.


SW-5748 Added modified time to application payload

#2234  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tommylau523 and the rest of your teammates on  Graphite


Press page from manga to move to the next one

Hey,
is there a possibility to implement the option to move to the next page of manga by pressing the image? Scrolling is fine, since it loads all the images at once, but this could be more convenient for some, and I think it's quite easy to code.
Cheers!

Thanks for the suggestion! I appreciate the feedback.
When you mention clicking the images to scroll, are you thinking about this on mobile or desktop or both? I find scroll on mobile is already pretty convenient, but could see how something like this would make reading easier on desktop.


Fix when using keyword-only argument and default argument is not set

Fixes #33

A couple comments @hakancelikdev but overall the fix looks solid. Thanks!

When are you planning to release this? 😊

Hopefully today @feluelle!


Increased version to 1.2.335

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@dragz, thanks for the bump here. Would you like to be a recipe maintainer, so that you can merge these yourself? If so, please just add yourself to the maintainers section in the recipe.
I will merge this PR now to make sure there's no further delay. If you'd like to be a maintainer, please open a new PR.


Reverts last commit.

It's necessary to have at least two parameters in the returning function definition in order to play nice with react-redux's connect function.

Whoops. Thanks @volrath!


Update data-classification-content-explorer.md with latest

Onedrive and EXO SITs are no longer in preview.
Teams is released WW.

Completing the PR.

Completing PR


simulcast is not fully implemented

hi,
mediasoup has some goods for simulcast, but in your lib they not implemented:
consumer event layerschange
https://mediasoup.org/documentation/v3/mediasoup/api/#consumer-on-layerschange
and consumer methods and fields for layers
https://mediasoup.org/documentation/v3/mediasoup/api/#consumer-preferredLayers
I need to change layer to max quality for consumer when I open him fullscreen, but I can't
If I wrong can you provide some information?

simulcast is not supported.


Adds a "Question" link to the new issue selection

This gives users a hint to use the GitHub "Discussions" for questions about bevy, instead of filing an issue.
Objective

Users sometimes file unhelpful issues when asking questions about how to use Bevy.

We should provide a link to the Discussion board in the list of "new Issue" options. This hopefully allows users to better find this option and reduces the number of question-issues.

fixes #5150

Solution

add a small config.yml that configures the link

Looks like this (currently live on my local fork https://github.com/themasch/bevy/issues/new/choose):

Open questsions

I am unsure about the wording.

Thanks :) I've left a wording suggestion for you.

Missed Alice not giving an explicit approval yet, whoops.

Controversial label added due to high user-visible impact :)

Couldn't it directly link to the new discussion page, possibly with Q&A selected?

Clever idea. I'm split on that; the current design encourages searching first.

So I think the comment beside the button should guide people searching for an answer first.

So I could change the link to https://github.com/bevyengine/bevy/discussions/categories/q-a to pre-select the Q&A category, and/or change the comment to something like

Questions about how to use or contribute to Bevy belong in Github Discussions. You can use the search to check if someone already answered that question!

I don't know if it's possible, but could changing the button color make it stand out even more? That gray looks a little dull.

From what I understand right now, thats not possible. For those links you basically got three properties: name, url and about.
The offical github docs about that are a bit terse on the details, but that might be because there are no details. A code search for what other people are doing did not turn up anything either.  The schema file that ships with the IDEA community edition also only knows these three properties.

bors r+


chore(release): add libraries to release page

Tested on a fork, looks like this: https://github.com/jeqo/tiered-storage-for-apache-kafka/releases/tag/v0.0.1

@AnatolyPopov thanks! Yes, agree on the core libs naming. I have changed that to pick the libs from core/build instead. See the last commit and release sample: https://github.com/jeqo/tiered-storage-for-apache-kafka/releases/tag/v0.0.1


PANIC: index out of bounds: the len is 1 but the index is 1

Hi Prisma Team! My Prisma Client just crashed. This is the report:
Versions

Name
Version

Node
v15.0.1

OS
debian-openssl-1.1.x

Prisma Client
2.9.0

Logs
prisma-client {
  engineConfig: {
    cwd: '/usr/src/app',
    enableDebugLogs: false,
    enableEngineDebugMode: undefined,
    datamodelPath: '/usr/src/app/node_modules/.prisma/client/schema.prisma',
    prismaPath: undefined,
    engineEndpoint: undefined,
    generator: {
      name: 'client',
      provider: 'prisma-client-js',
      output: '/usr/src/app/node_modules/@prisma/client',
      binaryTargets: [],
      previewFeatures: [],
      config: {}
    },
    showColors: false,
    logLevel: undefined,
    logQueries: true,
    flags: [],
    clientVersion: '2.9.0',
    enableExperimental: [],
    useUds: undefined
  }
}
plusX Execution permissions of /usr/src/app/node_modules/.prisma/client/query-engine-debian-openssl-1.1.x are fine
plusX Execution permissions of /usr/src/app/node_modules/.prisma/client/query-engine-debian-openssl-1.1.x are fine
plusX Execution permissions of /usr/src/app/node_modules/.prisma/client/query-engine-debian-openssl-1.1.x are fine
plusX Execution permissions of /usr/src/app/node_modules/.prisma/client/query-engine-debian-openssl-1.1.x are fine
prisma-client Error: PANIC: index out of bounds: the len is 1 but the index is 1

This is a non-recoverable error which probably happens when the Prisma Query Engine has a panic.

https://github.com/prisma/prisma-client-js/issues/new?body=Hi+Prisma+Team%21+My+Prisma+Client+just+crashed.+This+is+the+report%3A%0A%23%23+Versions%0A%0A%7C+Name++++++++++++%7C+Version++++++++++++%7C%0A%7C-----------------%7C--------------------%7C%0A%7C+Node++++++++++++%7C+v15.0.1++++++++++++%7C+%0A%7C+OS++++++++++++++%7C+debian-openssl-1.1.x%7C%0A%7C+Prisma+Client+++%7C+2.9.0++++++++++++++%7C%0A%0A%0A%0A%23%23+Logs%0A%60%60%60%0Aprisma-client+%7B%0A++engineConfig%3A+%7B%0A++++cwd%3A+%27%2Fusr%2Fsrc%2Fapp%27%2C%0A++++enableDebugLogs%3A+false%2C%0A++++enableEngineDebugMode%3A+undefined%2C%0A++++datamodelPath%3A+%27%2Fusr%2Fsrc%2Fapp%2Fnode_modules%2F.prisma%2Fclient%2Fschema.prisma%27%2C%0A++++prismaPath%3A+undefined%2C%0A++++engineEndpoint%3A+undefined%2C%0A++++generator%3A+%7B%0A++++++name%3A+%27client%27%2C%0A++++++provider%3A+%27prisma-client-js%27%2C%0A++++++output%3A+%27%2Fusr%2Fsrc%2Fapp%2Fnode_modules%2F%40prisma%2Fclient%27%2C%0A++++++binaryTargets%3A+%5B%5D%2C%0A++++++previewFeatures%3A+%5B%5D%2C%0A++++++config%3A+%7B%7D%0A++++%7D%2C%0A++++showColors%3A+false%2C%0A++++logLevel%3A+undefined%2C%0A++++logQueries%3A+true%2C%0A++++flags%3A+%5B%5D%2C%0A++++clientVersion%3A+%272.9.0%27%2C%0A++++enableExperimental%3A+%5B%5D%2C%0A++++useUds%3A+undefined%0A++%7D%0A%7D%0AplusX+Execution+permissions+of+%2Fusr%2Fsrc%2Fapp%2Fnode_modules%2F.prisma%2Fclient%2Fquery-engine-debian-openssl-1.1.x+are+fine%0AplusX+Execution+permissions+of+%2Fusr%2Fsrc%2Fapp%2Fnode_modules%2F.prisma%2Fclient%2Fquery-engine-debian-openssl-1.1.x+are+fine%0AplusX+Execution+permissions+of+%2Fusr%2Fsrc%2Fapp%2Fnode_modules%2F.prisma%2Fclient%2Fquery-engine-debian-openssl-1.1.x+are+fine%0AplusX+Execution+permissions+of+%2Fusr%2Fsrc%2Fapp%2Fnode_modules%2F.prisma%2Fclient%2Fquery-engine-debian-openssl-1.1.x+are+fine%0A%60%60%60&title=PANIC%3A+index+out+of+bounds%3A+the+len+is+1+but+the+index+is+1&template=bug_report.md

If you want the Prisma team to look into it, please open the link above 🙏

    at yw.handleRequestError (/usr/src/app/node_modules/@prisma/client/runtime/index.js:92:645)
    at /usr/src/app/node_modules/@prisma/client/runtime/index.js:129:3186
    at processTicksAndRejections (node:internal/process/task_queues:93:5)
    at async $w.request (/usr/src/app/node_modules/@prisma/client/runtime/index.js:211:3203)
plusX Execution permissions of /usr/src/app/node_modules/.prisma/client/query-engine-debian-openssl-1.1.x are fine
plusX Execution permissions of /usr/src/app/node_modules/.prisma/client/query-engine-debian-openssl-1.1.x are fine

Hello! Can you please provide a minimal reproduction for the above crash? Unfortunately the automatically collected information in the issue does not give us much to go with here - we will need a bit more information. What were you doing? Can you share the code and schema that was active?

I am going to close this as its been 3 months now and there is no activity here.
Please reply if you want us to take another look.

I am going to close this as its been 3 months now and there is no activity here.
Please reply if you want us to take another look.


Tests fail due to undefined symbols: AO2MOnr_e2_drv

Hi,
I'm currently trying to bring pyscf to conda-forge and am getting the following two errors during testing:
AttributeError: /home/conda/staged-recipes/build_artifacts/pyscf_1638267790335/test_tmp/pyscf/lib/libcc.so: undefined symbol: AO2MOnr_e2_drv

and
AttributeError: /home/conda/staged-recipes/build_artifacts/pyscf_1638267790335/test_tmp/pyscf/lib/libmcscf.so: undefined symbol: AO2MOmmm_ket_nr_s2

Do I need to switch something explicitly on during compilation, or why would these be missing?
Thanks!

Could you please provide more information like how you complied the code?
The AO2MOnr_e2_drv function is defined in pyscf/lib/ao2mo/nr_ao2mo.c and libcc and libmcscf depend on libao2mo.

Of course, the code compilation is here: https://github.com/conda-forge/staged-recipes/pull/17080/files
It's just running python -m pip install . -vv with export CMAKE_CONFIGURE_ARGS="${CMAKE_ARGS} -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=$PREFIX -DCMAKE_INSTALL_LIBDIR=lib -DBUILD_SHARED_LIBS=ON -DBUILD_LIBCINT=OFF -DBUILD_LIBXC=OFF -DBUILD_XCFUN=OFF -LAH".

libcc and libmcscf depend on libao2mo.

From what I can see in the logs, libao2mo should be included: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=419921&view=logs&jobId=6f142865-96c3-535c-b7ea-873d86b887bd&j=6f142865-96c3-535c-b7ea-873d86b887bd&t=22b0682d-ab9e-55d7-9c79-49f3c3ba4823

From the log, it seems libcc is not linked with libao2mo
2021-11-30T10:25:51.4133265Z    INFO (pyscf,lib/python3.9/site-packages/pyscf/lib/libcc.so): lib/python3.9/site-packages/pyscf/lib/libcvhf.so found in this package
2021-11-30T10:25:51.4172361Z    INFO (pyscf,lib/python3.9/site-packages/pyscf/lib/libcc.so): Needed DSO lib/libblas.so.3 found in conda-forge::libblas-3.8.0-11_h86c2bf4_netlib
2021-11-30T10:25:51.4216648Z    INFO (pyscf,lib/python3.9/site-packages/pyscf/lib/libcc.so): lib/python3.9/site-packages/pyscf/lib/libnp_helper.so found in this package
2021-11-30T10:25:51.4254910Z    INFO (pyscf,lib/python3.9/site-packages/pyscf/lib/libcc.so): Needed DSO lib/libgomp.so.1 found in conda-forge::_openmp_mutex-4.5-1_gnu
2021-11-30T10:25:51.4297609Z    INFO (pyscf,lib/python3.9/site-packages/pyscf/lib/libcc.so): Needed DSO x86_64-conda-linux-gnu/sysroot/lib64/libc.so.6 found in CDT/compiler package

I'm not sure why this happens.

Fixed also by PR #1142

Awesome, thank you!

@sunqm I'm still running into the undefined symbol: AO2MOmmm_ket_nr_s2 error after applying your patch.
Logs: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=432593&view=logs&j=6f142865-96c3-535c-b7ea-873d86b887bd&t=22b0682d-ab9e-55d7-9c79-49f3c3ba4823&l=6298
Patch: https://github.com/conda-forge/staged-recipes/pull/17080/files (Which is #1142 turned into a patch)


chart config object just set a value

chartutil/load.go
chart.Config has two properties:Raw and values ,why just set a raw value?forget ?

Hi @zhangjianweibj
Raw holds the yaml from disk and that is used to generate Values later.

@jascott1 very thanks, i understand the value 'Raw'.but values not set is unreasonable.i used helmclient get chart information,and  find values is nil. That not a rational design.is user get vlaues through parsing  raw by self?

Hi @zhangjianweibj So sorry for the delay. Can you elaborate on where you are seeing nil and possible post some output? Thank you.

Closing due to inactivity. Please open a new ticket against Helm 3 if this issue persists. Thanks!


Problems happen when I click the fix_host_save.cmd

And it shows like these：
Found Python at C:\Users\jihe\AppData\Local\Microsoft\WindowsApps\python.exe
Python Version: Python 3.12.6
Executing fix_host_save.py using C:\Users\jihe\AppData\Local\Microsoft\WindowsApps\python.exe...
Save Path: D:\jihe\Desktop\PalworldSaveTools.v0.8.2\LocalWorldSave\Level.sav
Running: "C:\Users\jihe\AppData\Local\Microsoft\WindowsApps\python.exe" "fix_host_save.py"
Traceback (most recent call last):
File "D:\jihe\Desktop\PalworldSaveTools.v0.8.2\fix_host_save.py", line 1, in
from fix_save import *
File "D:\jihe\Desktop\PalworldSaveTools.v0.8.2\fix_save.py", line 7, in
from internal_libs.import_libs import *
File "D:\jihe\Desktop\PalworldSaveTools.v0.8.2\internal_libs\import_libs.py", line 59, in
from cityhash import CityHash64
ModuleNotFoundError: No module named 'cityhash'

Kindly try to use v0.8.3 version of my tool, and response back. Thank you.

v0.8.3

It works，thanks a lot。

Thanks for the confirmation. Much appreciated!


Hotfix for 3D touch crash

Implements suggestion in http://stackoverflow.com/a/34899938/1470725 to avoid app crash upon force touching a photo when selecting an avatar or Reportback image.
Resolves #925 for short term / v1.1.0 launch.  Nice to have for a rainy day would be to implement expected 3D touch preview things.

Don't expect issues... but it'd be great if we can test this on iOS8.


How to migrate?

The rename of zplugin to zinit is very confusing for users who are not following reddit or the issue on github.
Even worse, when you try to run zplg self-update you get a fatal error:
remote: Repository not found.
fatal: repository 'https://github.com/zdharma/zplugin.git/' not found

This leaves users in a dead-end, because using google does not bring up any news about the rename. My first, personal and shocked, reaction was that the project was buried!
To make the situation harder, the new repo does not mention that zinit is the "successor" of zplugin. And the readme does not give any tips on what to do when you are stuck with the previous zplugin installation.

I would suggest making the rename more public.
When renaming a repository, github normally provides a redirect to new repo. Is there an option to get into that state?
What steps do zplugin users have to apply to migrate their setup to zplugin? Is a simple change of the github origin url sufficient?

It used to have. But this repo has been deleted before and may have lost its relationship with the original repo.
I think it is enough. Run git remote set-url origin https://github.com/zdharma/zinit.git in $ZPLGM[BIN_DIR], then update normally.

I've contacted github and asked them to restore the redirection. Thus now there's nothing required to migrate from Zplugin to Zinit.


mysql_root_user is undefined

I don't know why mysql_root_user is not available in that scope, but as defined in the mariadb task from trellis the root user is always root so I've hardcoded that username to the create backup jobs task:
- name: Create backup jobs
  set_fact:
   [...]

    backup_mysql_user: root
    backup_mysql_pass: "{{ mysql_root_password }}"

@o1y If you try to run the backup role without running the mariadb role, mysql_root_user will be undefined. If you are adding the backup role to an already-configured Trellis site, make sure you run:
ansible-playbook server.yml -e env=production
…with both mariadb and backup enabled and in that order.


X509 store ctx set purpose ex

X509_STORE_CTX_set_purpose() and _set_trust() do not set these parameters if they have been set previously. This is due to these functions calling X509_STORE_CTX_purpose_inherit().
This pull request includes to elements: It adds documentation describing the behavior and it proides _ex() functions adding the option to override previous settings.
Use case is the verification of CAdES signature, where timestamps might be embedded. Hence the same X509 store might be used for both checking the original purpose (like code signing) and validating the time stamp.
Checklist

[x] documentation is added or updated
[x] tests are added or updated

Whitespace fixes.

Nice to see Lutz once again active in the project. No issues with the proposed code.

My only concern is whether we need all three functions. Perhaps we could do with just the X509_STORE_CTX_purpose_inherit_ex function and make those two other just macros.

Changing existing functions to macros breaks the ABI. The functions are cheap enough.  I am not a big fan of macros.

Changing existing functions to macros breaks the ABI. The functions are cheap enough. I am not a big fan of macros.

I am not talking about existing functions I am talking about two of the three newly added _ex functions which are implemented as simple calls of the remaining one. The existing functions would of course need to be kept as functions. Although adding functions is cheap, it is not free and making just a redirect call with a different name and parameters as a new function is an overkill IMO.

Hmm. If I get it right, we need to leave the existing functions as functions, but they re missing the "override" parameter. I don't see how to fix this situation with a macro!?

Hmm. After having them converted to macros, now "check_docs" fails, as they are listed in the man page but do not exist as functions!?

Please add them to https://github.com/openssl/openssl/blob/master/util/other.syms

Updated. Checks are now successful.

Fixed.

To my best understanding, X509_STORE_CTX_purpose_inherit_ex() will set the trust that is provided within the X509_PURPOSE, just as it would be if the X509_STORE would come without a purpose/trust already set. That would be the behavior I would like to achieven. Override previously set values like they were never set.

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This needs a rebase

I have rebased but left the fixups untouched as they resulted from review. If you wish to have them squashed as well, this can of course be done.

@vdukhovni would you formally approve this, please?

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 61 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 92 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 123 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 154 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 185 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 216 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 247 days ago

This PR is waiting for the creator to make requested changes but it has not been updated for 30 days.  If you have made changes or commented to the reviewer please make sure you re-request a review (see icon in the 'reviewers' section).

This PR is waiting for the creator to make requested changes but it has not been updated for 61 days.  If you have made changes or commented to the reviewer please make sure you re-request a review (see icon in the 'reviewers' section).

This PR has been closed.  It was waiting for the creator to make requested changes but it has not been updated for 90 days.


♻ Use suspense for items page

I've also added a very simple skeleton
https://github.com/tiangolo/full-stack-fastapi-template/assets/667029/391d693d-9a00-467d-9cb6-e53d249b4b96

Update to only suspend the table, it looks much better 😊
https://github.com/tiangolo/full-stack-fastapi-template/assets/667029/6a19d7a1-ab25-4e39-991b-58937650d8fc
The only issue is the heading widths changing, but maybe we can live with that? (or we do set a fixed size for them) 😊

Update to only suspend the table, it looks much better 😊
CleanShot.2024-04-15.at.13.55.01.mp4
The only issue is the heading widths changing, but maybe we can live with that? (or we do set a fixed size for them) 😊

Yes, it looks nice! I think we can set a fixed size for it :thinking:

Awesome, thanks! Looks great :rocket:


Eagerly initialize RapidsShuffleManager for SPARK-45762 [databricks]

FIxes #11107

Initialize RapidsShuffleManager on construction for Spark 4.0 and Databricks 14.3
Disable lazy initialization and conf validation in the case above

build


Validate the MIME type of files using FileAllowed

The flask_wtf.file.FileAllowed validator is very useful to validate the extension of an uploaded file, however, it is also important to validate the MIME type of the files as a user could easily change the extension of a file to violate this validator, thus breaking the integrity of the application. This is also a recommendation made by the HTML Standard:

Authors are encouraged to specify both any MIME types and any corresponding extensions when looking for data in a specific format.

In my opinion, the best place to specify the allowed MIME types is in the same list that receives this validator (just as the accept attribute of <input type="file"> works), for example:
FileAllowed(upload_set=["doc", "docx", "xml", "application/msword", "application/vnd.openxmlformats-officedocument.wordprocessingml.document"])

I was thinking about how to implement this and came to the following design decisions:

If upload_set is a list, the validator will check that the extension OR the MIME type of an uploaded file belongs to that list. This is the same way the accept attribute I talked about previously works, and is very feasible because if, for example, someone wanted only videos and PDF files, this would suffice:
FileAllowed(upload_set=["video/*", "pdf"])

Isn't this elegant? 😊

If upload_set is a dictionary, the validator will check the extension AND the MIME type of an uploaded file. For example, the extensions ["jpg", "jpeg", "jfif", "pjpeg", "pjp"] belong to the MIME type "image/jpeg", now suppose someone wants files with this MIME type but only with the extensions ["jpg", "jpeg"], then:
FileAllowed(upload_set={
    "image/jpeg": ["jpg", "jpeg"]
})

Continuing with this example, if someone wants files with said MIME type and any of its extensions, then:
FileAllowed(upload_set={
    "image/jpeg": None
})

In the latter case, None means "there is no preference for some extensions in particular", however, this does not mean that an uploaded file can have any extension, but rather that it must have one of the extensions belonging to that MIME type, therefore "image/jpeg": None is equivalent to "image/jpeg": ["jpg", "jpeg", "jfif", "pjpeg", "pjp"].

The disadvantage of this functionality is that it will likely require the use of an external library such as python-magic or filetype, since the built-in mimetypes module relies on filenames to determine the MIME type, rather than using the contents of the files.
Please let me know if you like the idea and I can do a PR!

Coming late to the discussion, but I think this is a great idea! In fact, I stumbled upon this page because I had exactly this problem myself. Hope one of the maintainers see this soon!

Closing this as it did not get enough attention in almost 1 year.


Add in-browser record option to UploadPrompt

I used this previously: https://github.com/mattdiamond/Recorderjs
Is there a more up-to-date library? Do I even need an outside library? Will it work on mobile?

Oh yeah I did this.


[UI] Rancher UI display Validation failed in API: StatefulSet.apps "test" is invalid: spec: Forbidden: updates to statefulset spec for fields other than 'replicas', 'template', and 'updateStrategy' are forbidden

What kind of request is this (question/bug/enhancement/feature request):
bug: after deploy a statefulset workload with pvc , If I want to edit some properties of this workload , such as env, then it will show Validation failed in API: StatefulSet.apps "test" is invalid: spec: Forbidden: updates to statefulset spec for fields other than 'replicas', 'template', and 'updateStrategy' are forbidden in the rancher UI, but the prompt indicates that these properties are modifiable, and it can be modified normally by editing YAML. See the screenshot below for details:

Steps to reproduce (least amount of steps as possible):

Create a statefulset workload with A PVC template and add env on the UI
Edit this workload env,then click save and the issue will accurrd

Result:
display Validation failed in API: StatefulSet.apps "test" is invalid: spec: Forbidden: updates to statefulset spec for fields other than 'replicas', 'template', and 'updateStrategy' are forbidden.
Other details that may be helpful:
same problem in rancher v2.2.0
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
Rancher version: 2.4.5
Installation option (single install/HA):
single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Imported
Machine type (cloud/VM/metal) and specifications (CPU/memory):
VM: 6
cpu: 8 core
memory: 12 G
Kubernetes version (use kubectl version):
v1.15.3
Docker version (use docker version):
18.09

@nikkelma same problem in v2.4.5


formulahendry.vscode-mysql, mtxr.sqltools settings panel is blank

OS/Web Information

Web Browser: Firefox 89.0 (32 bit)
Local OS: Windows 7
Remote OS: Cent OS 7 (docker)
Remote Architecture:

code-server: v3.10.2
VS Code: v1.56.1
Commit: 387b12ef4ca404ffd39d84834e1f0776e9e3c005
Date: 2021-05-21T23:13:00Z
Browser: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:89.0) Gecko/20100101 Firefox/89.0
Steps to Reproduce
I install mysql extension (formulahendry.vscode-mysql, mtxr.sqltools) and their settings panel are blank.
extension details page are blank too.
Screenshot

Notes

This issue can be reproduced in VS Code: Yes/No
I do not know

Is this using HTTPS? I can't remember if the padlock with a cross means HTTP or mixed content.
This might be because these panels use web views which only work in a secure context (HTTPS or localhost).

No. Just http,

Today I change code-server address http to https now panel show. Only setcap command failed (sudo: setcap: command not found), but code-server run over https.
Thanks everyone.
Replaces "cert: false" with "cert: true" in the code-server config.
sed -i.bak 's/cert: false/cert: true/' ~/.config/code-server/config.yaml
Replaces "bind-addr: <IP_ADDRESS>:8080" with "bind-addr: <IP_ADDRESS>:443" in the code-server config.
sed -i.bak 's/bind-addr: <IP_ADDRESS>:8080/bind-addr: <IP_ADDRESS>:443/' ~/.config/code-server/config.yaml
Allows code-server to listen on port 443.
sudo setcap cap_net_bind_service=+ep /usr/lib/code-server/lib/node

Great! I'm interpreting that as fixed so I'll close this issue.


[BACKLOG-36967] updating swt version to 3.122 to resolve windows11 issue

:white_check_mark: Build finished in 2m 16s
Build command:
mvn clean verify -B -e -Daudit -amd -pl assemblies/core/client

:exclamation: No tests found!
:information_source:  This is an automatic message


Splitting required into notBlank and notNull

Is your feature request related to a problem? Please describe.
When using the required validator on a FormField, that can have non String-type values, like many custom FormFields do (like a DatePicker), the current required validator throws an exception, since it tests on value.length > 0. Some FormFields may even return null.
One can imagine FormFields that allow an empty array as valid value, but not null. In this case the current required validator will not work as expected.
Describe the solution you'd like
Replacing required validator with adding notNull and notBlank. Or keeping required but stating it does only work on type String.
Describe alternatives you've considered
none
Additional context
See my pull request

I am happy to contribute with a pull request.


Init instruments without failing requests

If the instruments fail to init call otel.Handle(err) to avoid erroring all requests. This follows recommended behaviour to discard metrics under error conditions documented here.  See how grpc otel implements this here
Currently we wait for the first request/response to determine if on the client or server. I've kept the once behaviour and initialise either or with a once. Could remove once and error on creating the NewInterceptor.

Could remove once and error on creating the NewInterceptor.

@jhump yep I think it would be better to error on creation but requires an API change. otel.Handle(err) is how errors are handled in other packages so would be required to monitor it. I can improve the docs to clarify?
This fixes the MUST NOT cause the application to fail later at run time and delays solving the initialise issue.

otel.Handle(err) is how errors are handled in other packages so would be required to monitor it.

But what kind of errors? Surely not initialization errors. The signature of the error handler, even if you install your own, isn't useful. "Monitoring it" isn't helpful because when you observe an error, there will be no metrics from this process at all, and there's not much the error handler could actually do about it -- perhaps stop/kill the current process, which seems bad. And it's doubly-bad that this happens after the process has already started taking actual RPC traffic, which means even if you monitor it, since it's post-facto, your operational data is already missing data.

I think it would be better to error on creation but requires an API change

Then let's make the API change. To me, the current behavior is a serious issue and the only proper fix is the API change. What's in this branch may be an improvement, but IMO not a sufficient one and still far from a proper solution. This repo has not yet reached a v1 (all releases have been marked as "pre-release") so, especially given the nature of the issue (that a failure to initialize metrics will fail all RPCs), the API change is definitely warranted and acceptable.

Surely not initialization errors.

Yes, see the linked example from grpc: https://github.com/open-telemetry/opentelemetry-go-contrib/blob/5adc27110c6f8edff55e07c668aeb140a166dcfd/instrumentation/google.golang.org/grpc/otelgrpc/config.go#L75-L80
Will update the API. This means having to initialize both client and server instruments, not not sure what the downside of that are.

Yes, see the linked example from grpc:

Oh, okay. We could do the same thing then. That's better because they are creating the instruments during initialization instead of deferring to first use. That is strictly better than what's in this branch because it means that the error handler could prevent the server from starting.
Although, TBH, I think an API change is even better. I'd think the otel error handler would be better for runtime errors (like a trace collector that encounters I/O errors writing data, or a metrics handler that fails to push to a statsd gateway, etc), not for initialization errors. 🤷
But even if we didn't make the API change, we should at least eagerly initialize the instruments so that the error is reported during startup instead of when serving actual traffic.

This means having to initialize both client and server instruments

Sadly, a shortcoming in the connect interceptor API that is tries to use the same interface for both. I don't actually think it's much to worry about because under-the-hood, most metrics libraries don't really do anything with the instruments until they get an observation, since most of the in-memory storage needs at least one actual label value (like RPC method name) to really initialize.

That is strictly better than what's in this branch because it means that the error handler could prevent the server from starting

I don't think your understanding the point of the otel.Handle(). It's not to stop the server from starting or ensure all requests fail. Errors in config are meant to drop metrics in favour of uptime.
Changed the API to return an error on init.

It's not to stop the server from starting or ensure all requests fail.

It is of course not to ensure requests fail. The previous behavior was certainly the worst thing we could do, operationally speaking.
As far as the point not being to stop the server: sure, that's not the point, but it certainly needs to be an option.

Errors in config are meant to drop metrics in favour of uptime.

The best way to preserve up-time is often to not roll out a new version of a service if it has errors in config. The best way to prevent roll-out of a bad version is to prevent the server from starting up. Sane deployment systems will pause roll out if too many new containers are failing to start. And doing so early preserves uptime, by causing the misconfigured server to fail before it was actually accepting any traffic


choppy/glitchy on image load

Great library thanks for sharing, I am having a slight issue though. When I load an image it seems to jump quickly between a few transitions before settling on one and going through it.
Kind of hard to explain so I can upload a video if needed, but basically my image will jump around to different scale sizes before finally picking one and smoothly doing the Ken Burns effect.

Hi schwiz, thank you for your report. I'd be glad if you could provide some steps to reproduce it. Anything like a video and/or a simple project demonstrating the issue would help a lot.

Turned out to be pilot error. I was setting the image more than one time which would generate a new transition even though the previous had only just started.


chore: add unit tests for ActionScreen

Add unit tests for the ActionScreen component.

@markwylde-deriv you need to setup gpg and sign your commit. otherwise I'm not able to merge it 😞
https://webdevstudios.com/2020/05/26/retroactively-sign-git-commits/

Thanks @shayan-deriv. It should now be signed!


[RFE] Add support for setting a server setting once for all servers (tools/configure_server_settings.rb)

There are two options:

Store the setting changes at a level higher than a server:  miq_regions and zones

Pro: We already have the ability to have servers inherit settings from regions and zones.
Pro: New servers would automatically inherit this setting.
Con:  If a server already has a setting, an override from the region/zone would be ignored.
Con:  the existing configure_server_settings.rb is geared towards servers and would need some work to deal with reading/writing values to the region/zone instead of for a server.  It might need to be renamed or at least changed to accept an optional list of either region, zone, or server ids.

Allow a -s "all" option or an array of server ids, or something simliar to set the same value for multiple servers

Pro: It would completely replace the existing functionality of tools/server_settings_replicator/server_settings_replicator.rb ✂ 🗑 🚽
Pro:  We already have the code in configure_server_settings.rb to set values on servers
Pro:  Any setting is saved for each server so it doesn't have the problem of an existing server setting overriding a value coming from a zone/region.
Con: You would need to push out this server setting change for each new server added to an environment.

@miq-bot assign @d-m-u


state not updating

Hi bwssytems,
Since version 4.5.x nor state not bridgeupdate commands updates the state (no matter true or false):
e.g.:
curl -H 'Content-Type: application/json' -X PUT -d {"on":true} http://<IP_ADDRESS>/api/homebridge/lights/9/state
curl -H 'Content-Type: application/json' -X PUT -d {"on":true} http://<IP_ADDRESS>/api/homebridge/lights/9/bridgeupdatestate
Results in the same state (assuming the initially was false):
"9": {
"state": {
"on": false,
"bri": 0,
"hue": 0,
"sat": 0,
"effect": "none",
"ct": 0,
"alert": "none",
"reachable": true
},
The action is performed correctly, but by not having the state updated, the information in homekit is not displayed correctly. This behavior started with 4.x series of ha bridge. The 3.x series worked flawlessly.
Is there something that needs to be done differently?
Best
T

Will take a look.

Under 4.5.6 I am getting the state back correctly. What method are you using to get the state?

Hi bwssytems,
I'm using: curl -X GET http://<IP_ADDRESS>/api/homebridge/lights
Also tested using 4.5.6. The issue is that the status in homebridge (which connects to habridge) no longer gets updated, and it was updating when on 3.5.x. The culprit seems to be the commands issued with curl as any actions in homebridge, gets habridge executing some scripts containg curl commands to habridge.
Any ideas?
Best,
T

I cannot get it to fail. Using Curl or Postman.

So for you state updates correctly? :(
The curl executes ok for me, action is performed, but status is not
updated...
--
Theodor Carp
From: BWS Systems<EMAIL_ADDRESS><EMAIL_ADDRESS>Reply: bwssytems/ha-bridge
<EMAIL_ADDRESS><EMAIL_ADDRESS>Date: 7 July 2017 at 22:17:32
To: bwssytems/ha-bridge<EMAIL_ADDRESS><EMAIL_ADDRESS>CC: 99ttz<EMAIL_ADDRESS><EMAIL_ADDRESS>Author
<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject:  Re: [bwssytems/ha-bridge] state not updating (#686)
I cannot get it to fail. Using Curl or Postman.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bwssytems/ha-bridge/issues/686#issuecomment-313769803,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYyOgNXiQVAKGgX9fqVaV-n_JjU0W7c9ks5sLoRMgaJpZM4ON8g4
.

Have you tried it on a command line outside of a script or using postman to verify? Also, is the state call immediately after the state call? If so, each call is independent and the first call may not be completed yet.

Yes. All the troubleshooting i'm doing outside the scripts by using the command line. I also waited for 30 secs before checking the state.
The interesting part is that if i use the web interface, which calls the exact same scripts, it updates the state correctly...

I am at a loss here as I would need to see your complete setup. How you start the ha-bridge, how it is stopped. Is  there another process taking over your port, etc.


UnhandledPromiseRejectionWarning: Error: Request workspace/configuration failed with message: Cannot read property 'source.fixAll.eslint' of null

Got eslint broken for me since yesterday.
Tried to reinstall plugin, removed all plugins except eslint from vscode, have latest eslint in the project. reinstalled vscode
Output of eslint in vscode:
(node:17052) UnhandledPromiseRejectionWarning: Error: Request workspace/configuration failed with message: Cannot read property 'source.fixAll.eslint' of null
    at /Users/justfly/.vscode/extensions/dbaeumer.vscode-eslint-2.1.3/server/out/eslintServer.js:1:8961
    at /Users/justfly/.vscode/extensions/dbaeumer.vscode-eslint-2.1.3/server/out/eslintServer.js:1:9255
    at Immediate.<anonymous> (/Users/justfly/.vscode/extensions/dbaeumer.vscode-eslint-2.1.3/server/out/eslintServer.js:1:9616)
    at processImmediate (internal/timers.js:439:21)
(node:17052) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3522)
(node:17052) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3522)

I have my settings.json
{
  "eslint.validate": [
    "javascript",
    "javascriptreact",
    "typescript",
    "typescriptreact"
  ],
  "eslint.enable": true,
  "eslint.packageManager": "yarn",
  "eslint.options": {
    "configFile": "./.eslintrc.js"
  },
  "css.validate": false,
  "less.validate": false,
  "scss.validate": false,
  "stylelint.enable": true,
  "stylelint.autoFixOnSave": true,
  "typescript.preferences.importModuleSpecifier": "non-relative",
  "typescript.tsdk": "node_modules/typescript/lib",
  "editor.codeActionsOnSave": {
    "source.fixAll.eslint": true
  },
  "cSpell.enableFiletypes": [
    "json"
  ]
}

vscode
Version: 1.44.1
Commit: a9f8623ec050e5f0b44cc8ce8204a1455884749f
Date: 2020-04-11T01:47:00.296Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 19.4.0

My projects running on node 13.11.0
Why does vscode using 12.8.1 node? How can I run vscode with node 13.11.0 ?
I have node running with nvm, and default node is also 13.11.0
Had no issues for this setup for long time
PS VS Code runs extensions on the node version that is built into electron (on which VS Code is based). This cannot be changed. https://stackoverflow.com/questions/45062881/custom-node-version-to-run-vscode-extensions

crosslinking an issue in vscode repo: https://github.com/microsoft/vscode/issues/95519

I have a fix for this in master but to ensure it is working correctly I tied to reproduce this but I can't. Can you still reproduce this. If so can you provide me with a GitHub repository I can clone.

@dbaeumer  I'm not sure that it is about the repo itself, but I can provide you access to one of my private repo.

@dbaeumer https://github.com/JustFly1984/drivefromto-gatsby/invitations
please remove repo after you test fix.

@JustFly1984  It works for me with my latest preview version.
Can you please give it a try and let me know if it works for you as well. It is available here: https://github.com/microsoft/vscode-eslint/releases/tag/release%2F2.1.4-next.1 and you can install it using the Install Extension from VSIX command.
Thanks.

@dbaeumer You are awesome! 2.1.4 is working for me too! Thank you


Update landscape.yml

Moved logz.io from logging to monitoring since we now have multiple products beyond just logging.
Added Twitter.
Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~5 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
duplicated mapping key at line 4126, column -5:
                item:
                ^

Deploy preview for landscape failed.
Built with commit cde4417c7da67b195b3307c6ec12d3345f6e5d22
https://app.netlify.com/sites/landscape/deploys/5ecd9922663e59000625f6bf

Build failed because of:
duplicated mapping key at line 4127, column -5:
                item:
                ^

Deploy preview for landscape failed.
Built with commit 3d990a6f54f7886e3f949ec9117e43185772d62f
https://app.netlify.com/sites/landscape/deploys/5ecd997e592c910007f58f15

LGTM


SP-2359 - use saved ecr trivy db #patch

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

Project coverage is 73.86%. Comparing base (e032145) to head (df157c0).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #575   +/-   ##
=======================================
  Coverage   73.86%   73.86%
=======================================
  Files         115      115
  Lines        4018     4018
=======================================
  Hits         2968     2968
  Misses        736      736
  Partials      314      314

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


Added drop graph functionality to MongoDB and a corresponding unit test

Description

What Changed?

Replaced MongoDBRyaDAO empty implementation for the drop graph functionality with a functional one.
Tests

Coverage?

Added a testDrop test to org.apache.rya.mongodb.MongoDBRyaDAOIT.java

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Yes

There are secondary indexes build for a graph in MongoDB. Dropping a graph will impact these indexes. This needs to be considered in the implementation.


Update Presets v3.10 and Changed Buildblocker for All Buildings

Changed Anno 1800 BuildBlocker Process that will fix #60 and Part 1 #323
Update Presets to version 3.10:

Added Tourists DLC Icons
Added Tourists DLC Ornaments to the Presets Tree
Added Tourists DLC Buildings to Presets Tree
Added Tourists DLC Buildings to Color.json (v<IP_ADDRESS>)
Anno Designer (for next Version)
Added translation for Tourist DLC Presets Tree Menu's

I just noticed something is wrong on the presets, i will fix that first so PR is closed for now

Fixed the Palace that was somehow moved to the Ornamental Menu
Added he missing residence face icons of Arctic and New World

Test adjusted to 4 nodes.
Commented out the Console.ReadKey() so tests can be running without waiting/hanging for that key press


FSRepository: End-to-end testing

Move tests/test_complex_application.py from taipy-core to taipy-integration-testing repository
Check with the CS team use cases/projects that can be used to validate the changes on Children classes
Add previous use cases to the taipy-integration-testing repository

Some links the CS team provided for this ticket:
https://github.com/Avaiga/demo-covid-dashboard
https://github.com/Avaiga/demo-churn-classification
https://github.com/Avaiga/demo-production-planning

Some links the CS team provided for this ticket:
https://github.com/Avaiga/demo-covid-dashboard
https://github.com/Avaiga/demo-churn-classification
https://github.com/Avaiga/demo-production-planning


esp32s3/smp: Dual Core Scheduling Problem.

At first, thank you for reading this issue. I'm not quite sure if it occurred by NuttX scheduling. Perhaps applications cause this issue.

Here is the process information of my board. It is running PX4-Autopilot, which is based on NuttX. On CPU0, there is one task taking up all the resources. And no new task can be scheduled on CPU0 (if I run smp test at this moment). Howerver this application has no problem on single core. The process information below.

And what makes me puzzled that ALWAYS CPU0 happens this problem and it never happens on CPU1. And, the task stuck on the CPU0 is random. It could change after reset. But sometimes the app runs well and no task stuck on one core.

HI @w2016561536 !
I didn't get exactly what is your problem, but I saw no problems regarding your process scheduling in the images you shared. Could you please elaborate what is the problem?

In picture 1, wq:rate_ctrl take up all the cpu resources. But actually, it does not run, just stuck there. It look like cpu 0 is dead

In picture 1, wq:rate_ctrl take up all the cpu resources. But actually, it does not run, just stuck there. It look like cpu 0 is dead

The picture does not show that. It shows that wq:rate_ctrl takes 50.2% of the CPU time. It is expected it to be that high because this task takes the highest priority (perhaps you should lower it). About being stuck: you would need to analyze the app itself: try to attach the debugger and check where it is being stuck.

In picture 1, wq:rate_ctrl take up all the cpu resources. But actually, it does not run, just stuck there. It look like cpu 0 is dead

The picture does not show that. It shows that wq:rate_ctrl takes 50.2% of the CPU time. It is expected it to be that high because this task takes the highest priority (perhaps you should lower it). About being stuck: you would need to analyze the app itself: try to attach the debugger and check where it is being stuck.

In NuttX，when dual core is enabled and system has very low load, each Idle task takes 50% time. Thank you for your advice. I will use debugger to inspect what happens on Core 0 next day. Here is middle night. I need to go to bed. Have a nice day. @tmedicci

@masayuki2009 since you did many improvements over original SMP, maybe you can have some suggestion to help here.

@acassis

@masayuki2009 since you did many improvements over original SMP, maybe you can have some suggestion to help here.

I think @tmedicci's comments are correct.
However, we should clarify which version of the nuttx  @w2016561536 is using.
Also, we should clarify the defconfig (i.e. esp32s3-devkit:smp) that @w2016561536 modified.


TinyMCE upgrade (v. 4.1.7), retry.

Merged back from a working implementation in v. 3.1.8. There are still some things to check/fix: ssmacron + spellchecker.
It's a start. I might have missed all kinds off dependencies, but it's working quite nicely as far as I can tell.
PS. Some more styling was changed after running compass after adding some tinyMCE specific stuff.
PPS. Perhaps also applicable for the 3.1 branch (as I got it working for that branch)? I don't think I'm breaking anything, but the options available for HtmlEditorConfig will change of course.

Closed with https://github.com/silverstripe/silverstripe-framework/pull/5037


XmlMarkupFormatter: escape less-than (<) and ampersand (&) in attributes

This issue was mentioned in #128

Thanks for the PR!

Why not merge?

Sorry, my bad! I pressed close and didn't merge beforehand ; ).


[Bug]: only the X11 backend works

What happened?
$ env -u DISPLAY gimp
Cannot open display:

What should have happened?
App starts using the wayland backend
Output of snap info $snap_name
$ snap info gimp
name:      gimp
summary:   GNU Image Manipulation Program
publisher: Snapcrafters✪
store-url: https://snapcraft.io/gimp
contact:   https://github.com/snapcrafters/gimp/issues
license:   unset
description: |
  Whether you are a graphic designer, photographer, illustrator, or scientist, GIMP provides you
  with sophisticated tools to get your job done. You can further enhance your productivity with GIMP
  thanks to many customization options and 3rd party plugins.

  This snap is maintained by the Snapcrafters community, and is not necessarily endorsed or
  officially maintained by the upstream developers.

  Upstream Project: https://www.gimp.org/
  snapcraft.yaml Build Definition:
  https://github.com/snapcrafters/gimp/blob/master/snap/snapcraft.yaml
commands:
  - gimp
snap-id:      KDHYbyuzZukmLhiogKiUksByRhXD2gYV
tracking:     latest/stable
refresh-date: 2022-08-20
channels:
  latest/stable:     2.10.30 2022-08-19 (393) 546MB -
  latest/candidate:  ↑
  latest/beta:       ↑
  latest/edge:       2.10.32 2022-08-23 (399) 547MB -
  preview/stable:    –
  preview/candidate: –
  preview/beta:      –
  preview/edge:      2.99.10 2022-08-17 (397) 650MB -
installed:           2.10.30            (393) 546MB -

Output of snap connections $snap_name
$ snap connections gimp
Interface                 Plug                  Slot                              Notes
browser-support           gimp:browser-support  :browser-support                  -
content[gnome-3-38-2004]  gimp:gnome-3-38-2004  gnome-3-38-2004:gnome-3-38-2004   -
content[gtk-2-engines]    gimp:gtk-2-engines    gtk2-common-themes:gtk-2-engines  -
content[gtk-3-themes]     gimp:gtk-3-themes     gtk-common-themes:gtk-3-themes    -
content[icon-themes]      gimp:icon-themes      gtk-common-themes:icon-themes     -
content[sound-themes]     gimp:sound-themes     gtk-common-themes:sound-themes    -
cups-control              gimp:cups-control     -                                 -
dbus                      -                     gimp:dbus-gimp                    -
desktop                   gimp:desktop          :desktop                          -
desktop-legacy            gimp:desktop-legacy   :desktop-legacy                   -
gsettings                 gimp:gsettings        :gsettings                        -
home                      gimp:home             :home                             -
network                   gimp:network          :network                          -
opengl                    gimp:opengl           :opengl                           -
removable-media           gimp:removable-media  -                                 -
unity7                    gimp:unity7           :unity7                           -
wayland                   gimp:wayland          :wayland                          manual
x11                       gimp:x11              -                                 -

Output of snap version
$ snap version
snap    2.60.3
snapd   2.60.3
series  16
ubuntu  22.04
kernel  6.2.0-33-generic

Relevant log output
No response
Teminal output of app
No response

Gimp is GTK2, therefore no Wayland support.

I guess snap packaging can't do much about that!


Fix code formatting for Java PipeAsCode Build step

Closes: #72
Fix code formatting for Java code. I've just used the same markdown used by other languages C# and Python
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for the fix. We've moved to use SAM Pipelines as the mechanism to build the pipeline, so this change isn't relevant in the current version.


[cgal] update hash for 4.11.1

#2894 updated to 4.11.1 but it looks like there has been an update to the release binary since then.

Thanks!


Remove undocumented page

We're not keeping this page up to date, so it's only a small
portion of undocumented content. This is misleading because it
makes readers think that this is the only information not
documented.

Yay for linkchecker 😜 .  I was thinking the intro could just link to https://github.com/rust-lang-nursery/reference/issues.  I also noticed another stray link "big issue" at the bottom of the intro page that should be removed. I wish pulldown-cmark would warn about those.


Sluggish integrated terminal

Issue Type: Bug
As soon as the integrated terminal has enough text to start scrolling, it becomes sluggish and very unpleasant to use.
Happens with all extensions disabled. On macOS Moneterey and Big Sur.
Tried the fix suggested here several times. Not helping
VS Code version: Code 1.62.2 (3a6960b964327f0e3882ce18fcebd07ed191b316, 2021-11-11T20:59:05.913Z)
OS version: Darwin x64 21.1.0
Restricted Mode: No
Remote OS version: Linux x64 3.10.0-1160.36.2.el7.x86_64

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz (16 x 2300)

GPU Status
2d_canvas: unavailable_softwaregpu_compositing: disabled_softwaremetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: disabled_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: disabled_softwarewebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
3, 2, 2

Memory (System)
16.00GB (2.25GB free)

Process Argv
--disable-extensions --crash-reporter-id 255a4b22-d8e4-4854-9527-eb963404bbb6

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: csd3

OS
Linux x64 3.10.0-1160.36.2.el7.x86_64

CPUs
Intel(R) Xeon(R) Gold 6142 CPU @ 2.60GHz (32 x 1943)

Memory (System)
187.37GB (146.29GB free)

VM
0%

Extensions disabled
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30384386
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
vsccppwtct:30382698
bridge0708:30335490
dockerwalkthru:30377721
bridge0723:30353136
pythonrunftest32:30373476
pythonf5test824:30373475
javagetstartedt:30391933
pythonvspyt187:30373474
vsqsis300:30386381
vsaa593:30376534
vssld246cf:30386378

Please see https://github.com/Microsoft/vscode/wiki/Performance-Issues for creating actionable performance problem issues.
also please provide a gif

@meganrogge Sorry about the lack of information. Hope this helps:
https://user-images.githubusercontent.com/30958850/141979667-e9d505f3-4714-41c5-a610-0e7c2cb7bdb3.mp4
Profile-20211116T114235.zip

thanks for that. can you open dev tools and capture a performance profile while you are scrolling/seeing the sluggish behavior?

I did. See the zip attachment on my earlier comment.￼

The perf profile is a little weird, looks like there isn't a renderer JS problem:

Can you try changing the terminal.integrated.gpuAcceleration setting and seeing if that helps?

That helps! It was set to "terminal.integrated.gpuAcceleration": "auto". Changing it to "on" did nothing. But "off" and "canvas" make scrolling much smoother.

Ah ok, looks like your machine is one of the edge cases where the webgl isn't playing nicely with your graphics card/drivers for some reason.

MacBook Pro 16-inch from 2019. Shouldn't this be addressed in additional edge case detection? These machines are going to be around for quite some time to come.

I'm not sure there's a reliable way to detect this, plus your machine seems like a one off, it doesn't happen on the majority of macbooks or this would be reported more (from within the team and the public). Updating your OS might help if it's not up to date.

This occurred on multiple OS versions. Started under macOS 11 and stayed under 12.0.1 (running now).


bpf/pcap: Use CAPTURE{4,6}_RULES macros

The CAPTURE{4,6}_RULES macros are defined but not used in the code.
Fixes: https://github.com/cilium/cilium/pull/15376.

There are no end-to-end tests for this datapath code, so nothing to trigger. All initial tests are passing and the team review request is covered. Merging.


Draft: Adds k8s configuration files that match dockerfile

I gave you master access, let's just update that file :)


Package updates for 2024

TODO list of packages to add/update in 2024.
High priority:

[ ] :arrow_up: QGIS 3.28 Firenze to QGIS 3.34 Prizren
[ ] :arrow_up: JupyterLab v3 to v4
[ ] :arrow_up: jupyterlab_myst to v2.1.0
[ ] :arrow_up: add VScode
[ ] :heavy_plus_sign: https://github.com/silx-kit/jupyterlab-h5web

Low priority:

[ ] :arrow_up: Python 3.10 to 3.12

Feel free anyone to request package updates by commenting below, or opening a Pull Request directly following instructions at https://github.com/CryoInTheCloud/hub-image#updating-packages-in-this-repository

Previous package update issue is at #86.

Thanks @weiji14 for the bump on these! I added a MyST update and adding VScode.

Also adding the jupyter-ai extension

Hi @weiji14 and @tsnow03 I think we need to update earthaccess to v0.9.0 and install voila to notebooks in the form of widgets, should I open a PR to modify the environment file?

Hi @weiji14 and @tsnow03 I think we need to update earthaccess to v0.9.0 and install voila to serve notebooks like widgets, should I open a PR to modify the environment file?

Yes please, go ahead and open a PR to modify the line here: https://github.com/CryoInTheCloud/hub-image/blob/a0063519a32a280fc6ba1dd9a6a3b402e7f53299/environment.yml#L100


Autoregistration for VMs doesn't work (doesn't associate endpoints with services)

Using Istio 1.8, and enabling auto-registration, the WorkloadEntrys that get created by WorkloadGroup don't seem to have any labels set.
This happens when using the WorkloadGroup created by
istioctl x workload group create --name "${VM_APP}" --namespace "${VM_NAMESPACE}" --labels app="${VM_APP}" --serviceAccount "${SERVICE_ACCOUNT}" > workloadgroup.yaml

It also doesn't work if following the docs and manually creating the WorkloadGroup:
https://istio.io/latest/docs/reference/config/networking/workload-group/
In there it says:
The set of labels will be associated with each workload instance during the bootstrap process,

But this is not the case.
If the labels are not included correctly, then a Service or ServiceEntry cannot pick them up.
/cc @stevenctl including you as this is slightly related to https://github.com/istio/istio/issues/29125

Currently we use labels sent via node metadata, and the labels field on the workload entry template. Will roll a PR shortly to close this gap.

The manual workaround for now is to edit your workload group and copy the labels to the template section like this:
apiVersion: networking.istio.io/v1alpha3
kind: WorkloadGroup
metadata:
  name: cloud-vm
  namespace: vm-ns
spec:
  metadata:
    annotations: {}
    labels:
      app: cloud-vm
  template:
    ports: {}
    serviceAccount: vm-sa
+   labels:
+     app: cloud-vm


peer health check and verifier issue

add peer health check
auto reconnect all peers
try fix verifier issue when the geth process exits abnormally which caused current block index behind latest index

LGTM


⚠ Cioos Pacific CKAN has degraded performance

In f0e80f2, Cioos Pacific CKAN (https://catalogue.cioospacific.ca) experienced degraded performance:

HTTP code: 200
Response time: 11840 ms

Resolved: Cioos Pacific CKAN performance has improved in 13df45d after 15 minutes.


Queue Activating when 28-32 Players in Server (sv_maxclients 120)

After configuring, 28-32 players are able to join my RedM server even though the sv_maxclients is set to 120. I have tried to wrap my head around what is locking up the queue. My work around is that I have to restart the queue script about a minute after my server restarts and it goes of course higher than 28-32 players.
Any tips or maybe I'm something I'm missing?

Make sure the script is started AFTER sv_maxclients is set


Removed inputs still get validated

When using the watch option, removed inputs still get validated. In my case, I did the following:

Dynamically add inputs, leaving required fields blank
Submit the form. As it is invalid, validation errors for the new inputs appear.
Remove the invalid inputs.
Submit the form. A JS error appears: TypeError: input.form is null. The input in question is not part of the form anymore (input.isConnected is false).

I see the same, but in my case I did the following:

Load a page with a jQuery DataTable
Remove a row of that table
Try to submit the form,  With logging turned on I see TypeError: input.form is null.

Would checking if input.form is not null on line 1017 be too simplistic of a fix?
// Removing an error from one input should also remove it from others with the same name (i.e. for radio button and checkbox lists).
if (input.form) {
    var inputs = input.form.querySelectorAll("input[name=\"".concat(input.name, "\"]"));
    for (var i = 0; i < inputs.length; i++) {
        this.swapClasses(inputs[i], this.ValidationInputValidCssClassName, this.ValidationInputCssClassName);
    }
}

Worth a try. Does anyone have a super simple repro I can include in the demos app?


Fix the failing E2E tests

This is an attempt to fix our E2E tests to avoid failures that are there now.
Description

This updates the expected version of TrustyAI pip package to what is expected on the image.
See my comment for more info.
How Has This Been Tested?

Merge criteria:

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

Hello! :slightly_smiling_face:  I opened a follow up pull request for the e2e tests, could you take a look once you are available? It is here: https://github.com/opendatahub-io/notebooks/pull/517
I collected all the prior tests (minimal & data science) under trustyAi one


Interaction with RDKit?

Sorry, this is more like a basic question. Can MolML Read moleculules obtained as RDKit objects (or other molecular formats like Mol2 or Mol V3000)?

By default MolML accepts a few simple inputs tuples of elements, numbers, coordinates, connectivity or it can take filename inputs for .xyz, .out, or .mol2.
In principle, any format is acceptable for MolML as you can define arbitrary loading functions for data. A very simple example of this can be found here:
https://github.com/crcollins/molml/blob/master/examples/simple.py#L85
Basically, if you give input_type a function that takes in your object and returns a LazyValues object, then you can use anything.
from molml.utils import LazyValues
from molml.features import CoulombMatrix

def read_data(obj):
    return LazyValues(elements=obj.get_elements(), coords=obj.get_coords())

objs = [obj1, obj2, obj3]
feat = CoulombMatrix(input_type=read_data)
feat.fit_transform(objs)

Hope that helps.

Thank you very much!
I'll give it a try today.


Gmod tickrate and fix for many maps

Fixes https://github.com/dgibbs64/linuxgsm/issues/685
Tickrate needs to be lowered in order to be able to handle a large amount of players.
Default tickrate is 66.66, but the -tickrate commnand will round numbers to the lower integer. So i'll default at 66 and not -the more accurate- 67 to not worry anyone with a "non standard" value.
For the +r_hunkalloclightmaps 0, it fixes the hunk overflow crash on server startup when using maps with too much lights (and there are a lot of them). It changes the way lights are stored in RAM, the 0 mode (default one a few years ago, and still default one on clients) can handle a way larger amount of lights, but a theoretically a bit slower way. I noticed no difference, so it's a good thing to fix it by default.

Can't pull request to 2 branches at the same time, but would be cool to pull to the development branche afterward.


tikv-ctl: implement drop unapplied log for v2

What is changed and how it works?

Issue Number: Ref https://github.com/tikv/tikv/issues/14654
What's Changed:

implement drop unapplied log for v2

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test

Release note
None

/merge


[v2.10] add gke 1.6.0 support in rancher-cis-benchmark chart

gke 1.6.0 is available in upstream: https://github.com/aquasecurity/kube-bench/tree/main/cfg/gke-1.6.0
so the current gke 1.2.0 profile needs to be updated to 1.6.0 in rancher-cis-benchmark chart

Validated on rancher v2.10-head(v2.10-3bdb12bbea67b8ef7d9592ef82c5601bb2e03a64-head)

Created GKE v1.30.5-gke.1628000 and v1.31.1-gke.2008000 clusters.
Installed CIS chart 7.0.0-rc.2.
Ran GKE-1.6.0 profile scans.
Scans are running fine with failures.


More information needed on dev_se and dev_autocorr parameters

I've been working with @charliehinchliffe on understanding the impact of Q deviations as implemented by SSMSE. SSMSE inputs time-varying LnQ_base_..._DEVadd parameters in the .par file but these were getting rescaled by the LnQ_base_..._dev_se parameter, causing confusion.
The role of these two parameter is confusing for a few reasons. I'm happy to help revise this section, but any input on how to do that would be welcome:

The terms "dev_se" and "dev_autocorr" used in the parameter labels can't be found in the user manual because different notation is used (see text below). Adding those parameter label suffixes alongside the math notation will help find this line within the 14.1.3 Specification of Time-Varying Parameters: Short Parameter Lines

The description of the deviation link 14.1.2 Specification of Time-Varying Parameters: Long Parameter Lines doesn't make clear where the parameters are specified (some kind of link to the section below would be helpful here)

If there's an easy explanation it would be helpful to explain the reasons for this difference: "SS3 expects the estimated deviations to be normal in distribution and the deviation values are multiplied by the standard error parameter as they are used. This differs from recruitment deviations and from the approach in SS3 v.3.24."

I have shared a draft google doc with SS3 team so we can work on completing that 5 ear old essay on this topic.

Rick, are you thinking that the essay could be included as an additional vignette or that it would be incorporated into the user manual?

Elizabeth,  I think both.  Devs in SS3-ADMB are not nearly as sophisticated as the random effects in TMB, so their use needs sidebars.  That is intention of the essay, which will need close scrutiny given the time since it was drafted.  Then we need to be sure the manual is not inconsistent with it and includes a link to the essay.


[rfc] rollback urfave/cli to tagged release v1.20.0

I noticed we bumped to some commit on master in https://github.com/containerd/containerd/pull/1688, and couldn't find a reference if there's specific things we needed that were not in v1.20.0. Given that people might be using go mod, they'll likely get the last tagged version, so perhaps it's good to use it in this repository as well (hoping they'll tag a new release at some point; https://github.com/urfave/cli/issues/728)
Full diff:
https://github.com/urfave/cli/compare/v1.20.0...7bc6a0acffa589f415f88aca16cc1de5ffd66f9c
pushing as a separate PR, because I wasn't sure we wanted to do this, or if there's any changes in the diff that are desirable; so also happy to close if we don't want to do this 😅

NOT LGTM ;)
vndr works fine for now

Codecov Report

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

@@          Coverage Diff           @@
##           master   #3187   +/-   ##
======================================
  Coverage    45.2%   45.2%
======================================
  Files         111     111
  Lines       11982   11982
======================================
  Hits         5416    5416
  Misses       5732    5732
  Partials      834     834

Flag
Coverage Δ

#linux
49.23% <ø> (ø)
:arrow_up:

#windows
40.58% <ø> (-0.03%)
: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 591e52c...03520c5. Read the comment docs.

Codecov Report

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

@@          Coverage Diff           @@
##           master   #3187   +/-   ##
======================================
  Coverage    45.2%   45.2%
======================================
  Files         111     111
  Lines       11982   11982
======================================
  Hits         5416    5416
  Misses       5732    5732
  Partials      834     834

Flag
Coverage Δ

#linux
49.23% <ø> (ø)
:arrow_up:

#windows
40.58% <ø> (-0.03%)
: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 591e52c...03520c5. Read the comment docs.

I'm happy with vndr as well, but 🤷♂

Going to close this one, this is not something we are trying to support right now and not going to rollback without a regression. In the future though we should make a better effort to catch unnecessary hash vendors when there is a suitable tag.


Upgrade xerces version

Upgrade xerces to 2.12.0

The xerces-bundle-version also needs to be updated in parent/pom


[5.7][Diagnostics] Diagnose conflicting pattern variables

Cherry-pick of https://github.com/apple/swift/pull/59210

Diagnose situations where pattern variables with the same name
have conflicting types:
enum E {
case a(Int)
case b(String)
}

func fn(_: () -> Void) {}

func test(e: E) {
  fn {
    switch e {
    case .a(let x), .b(let x): ...
    }
  }
}

In this example x is bound to Int and String at the same
time which is incorrect.
(cherry picked from commit f15a0b16cb83fde2cfc3b437dc5cff6e8f7963c3)
(cherry picked from commit 62ba74950c4669bc0a63178acb10cce2700761b2)
(cherry picked from commit 9c8bebe4f8f93ea4f56f7f018070e216f67c027b)

@swift-ci please test


[Feature] Publish Snippets

Motivation
Like Postman, I hope Mqttx have a snippets to store a list of [topic and message]. Then I can mock device different kinds of message quickly.
Detailed design
In the left bar add a [Snippets] below [Scripts] to manage my snippets. I can add messages with topics in this section.
Then in the [Connection] tab, I can select a snippet to send if i don't want to edit message manually.

@apprentice1988 Hi, thanks! this is a good idea💡 We will consider whether to add this feature.

But the current historical Topic and Payload functions should help you temporarily. When you publish successfully, Payload and Topic will have history records, click the button in the lower right corner to view, avoid the trouble of manual input.

In the script page, the fixed payload value is returned in the function, which can also avoid this problem

I hope there is a way：

import SyncApi file to snippets and can transfer many messages
pick one message can mock data and send
I will subscribe the channel/topic in my server
I am not sure if it's a common use path.


Circular Dependencies in angular

Compiling the frontend gives a few circular dependency warnings

Possible Solutions
The above link  suggests:

(1) simply moving any of the above function to same file hence avoiding the import problem
(2) separating the function call logic in a.ts to a new file c.ts and import both functions there etc.

Which solution would be better?


feat(apply-context): Support MongoDB ObjectId as context variables

Add MongoDB ObjectId support as context variables, and hopefully other types of objects with toJSON methods returning strings.

Thank you @Hybrid-Force, just for curiosity, how are you using this package?

Thanks for this great lib! I am using it for API access control of a multi-tenant app that I am working on.


Postgresql shared cluster

This pull request enables testing PostgreSQL helm chart in shared cluster.
The imagestream installation is moved to test suite it self.

[test-nos2i]

[test-nos2i]


Removing parameters from services adding dictionaries

Feature Request
Removing parameters from services adding dictionaries

Q
A

New Feature
no

RFC
yes

BC Break
yes

Summary
I am thinking to remove all parameters from services file like:
pugx_badge_allin_badges
pugx_badge_badges

In my opinion, is better to have dictionary files to store this information and remove dependencies parameters into SnippetGenerator.php.
In that service, we can call statically those dictionary files instead of injecting parameters array from services file.
What do you think?
I can work into this pull request if it can be of value for this project.
I think it can be a BC because we are changing SnippetGenerator constructor.

It's a nice idea to me! It's an improvement also to DX side to the maintainability of the software.
Thanks @AlessandroMinoccheri


potential bug: a field to a scriptableobject with [Expandable]/[GetScriptableObject] lost its reference to that SO

It happened after I upgraded to latest saintsfields so it could be a problem

This might not be a problem of expandable, but GetScriptableObject, well, which at least happened to my project.
Kinda difficult to locate what caused it though, need some time to figure it out

This issue might be because of a value comparison bug which has been fixed in 3.4.0
I'll leave this issue open for another month. If no problem found then I'll close it as fixed. (But if the issue still occurs, I'll open it again)

Update about this issue:
if the target is a prefab but the prefab is put directly inside a scene, the value will be lost.

Ah ok that's a problem. I think it's how Unity handles reference. Could you maybe add a warning in such conditions? To help prevent hard to find errors.

Ah ok that's a problem. I think it's how Unity handles reference. Could you maybe add a warning in such conditions? To help prevent hard to find errors.

Sorry for the misleading... When I post that reply I indeed find a bug that makes it not working on prefab.
That bug has been fixed, but not quite sure it's that bug that lead all this. So this issue is still pending. I need some more time to investigate on it.

Hi,
3.4.9 fixes a bug which related to [Expandable] in IMGUI, that a SerializedObject did not get cached and lead to problem.
This issue might be related to that.
Please upgrade to 3.4.9 to check if the bug is gone.
Thanks

Thank you.


When adding ngDraggable to a resizable div with a containment, after moving the div the resize bounds are off (goes outside the containment).

@aviad123
Thank you for reporting this issue. Dragging div is implemented by CSS transform functions. Resize bounds remain there, but the directive doesn't check the transformed properties. Need to fix this issue.

The same situation for me. Containment doesn't work correct with ngDraggable and ngResizable.

Is there any timeline as to when this will be fixed?

@smaranh
I will double check this issue.

@xieziyu I finally figured out why the jump is happening. If you have the resizable or draggable container with position:absolute it is a problem since it conflicts with the draggable and resizable style which is position:relative. Here is the sample
https://stackblitz.com/edit/angular-cli-esri-map2-qyyzpu?file=src/app/app.component.html
Do you think I should close this issue and create a new one for this?

@smaranh
Thank you, it's very helpful. I will double check it.
It's OK to create a new issue, up to you.


Update 4-bullet-shooter.html

W8 Homework: Bullet Shooter game
Sky Zhang

@Sky83478  There are some interesting solutions here. I would look again at two things:

In your switch statement, case 3 has no break, so enemies[3] will never be set as output
x is NOT a good variable name because we don't know what kind of information it is. Also, that variable should be a boolean, not a string. Start by changing line 70 to var canFire = true, then go from there.

In the end, I'm rejecting this pull request because you're requesting I overwrite the original assignment with your solution. I would not want to do that, even if your solution was perfect.