Unable to skip SSL authentication when connecting to Emulator running in WSL.

Describe the bug
I would like to connect to a cosmos emulator running in WSL. I don't want to use SSL so I overrode CosmosClientOptions.HttpClientFactory to skip the SSL validation.
To Reproduce
Run the emulator in WSL (Ubuntu 22.04 distro from my side) using:
$ ipaddr="`ifconfig | grep "inet " | grep -Fv <IP_ADDRESS> | awk '{print $2}' | head -n 1`"
$ docker run -p 8081:8081 -p 10250-10255:10250-10255 -e AZURE_COSMOS_EMULATOR_PARTITION_COUNT=1 -e AZURE_COSMOS_EMULATOR_IP_ADDRESS_OVERRIDE=$ipaddr -i -t mcr.microsoft.com/cosmosdb/linux/azure-cosmos-emulator

Console app:
const string AccountEndpoint = "https://localhost:8081/";
const string AccountKey = "C2y6yDjf5/R+ob0N8A7Cgv30VRDJIWEHLM+4QDU5DE2nQ9nDuVTqobD4b8mGGyPMbIZnqyMsEcaGQy67XIw/Jw==";

var cosmosClient = new CosmosClient(AccountEndpoint, AccountKey, new()
{
    ConnectionMode = ConnectionMode.Gateway,
    HttpClientFactory = () =>
    {
        var httpMessageHandler = new HttpClientHandler
        {
            ServerCertificateCustomValidationCallback = HttpClientHandler.DangerousAcceptAnyServerCertificateValidator
        };
        return new HttpClient(httpMessageHandler);
    }
});
await cosmosClient.CreateDatabaseIfNotExistsAsync("demo");

Expected behavior
The program runs successfully.
Actual behavior
The program hangs in cosmosClient.CreateDatabaseIfNotExistsAsync, never returning and never creating a database.
Environment summary
SDK Version: 3.32.3
OS Version: Windows 11
Additional context
The issue is similar to #2706 but this issue is closed and I am not able to find a solution.

IME a hang in CreateDatabase is generally a symptom of a connectivity issue - an actual certificate check failure will yield you a specific exception. For a 'hang' like this, you should eventually get a HTTP 408 timeout or similar; generally that's a matter of a port not being mapped etc (example notes I took along the way to figuring out a similar mystery, even if it may look like entirely different circumstances: https://stackoverflow.com/a/75879659/11635)

I don't think it's an network issue or port not being mapped as I can access to https://localhost:8081/_explorer/index.html from my machine (with a SSL issue of course but I can't bypass it in the browser).

Don't bank on it - the port config for the admin UI and Gateway mode connections may or may not coincide.
I note also that your -p args are for the Direct ports, but your code is requesting Gateway - in general you're probably best off sticking with Direct everywhere
The bottom line is that not being able to create a database is probably not the problem - it's just the first thing you happen to be doing programmatically. I'd also add that this repo is about the .NET SDK/packages, and things that can be fixed by changing things in there; there's no reason to believe this is the case, so this issue really does not belong here.

I would like to use the Gateway mode locally and disable the SSL validation on the HttpClient used with:
    HttpClientFactory = () =>
    {
        var httpMessageHandler = new HttpClientHandler
        {
            ServerCertificateCustomValidationCallback = HttpClientHandler.DangerousAcceptAnyServerCertificateValidator
        };
        return new HttpClient(httpMessageHandler);
    }

This is not a chat forum for diagnosing/requesting docker configs
This is a repo for the .NET CosmosDB SDK  - you're better off using stackoverflow.com if you're hoping to to/fro with someone; there's only me and 69 other watchers on this repo
I found the MS documentation for configuring the emulator on a Mac to be pretty OK, but it takes time and diligence to figure it out.
If you have a specific message that you think tells you there is a problem with how you are disabling cert validation, then by all means sit and wait for an answer. Until then, you've yet to disprove that this is a connectivity issue that comes BEFORE all of that.

I understand that this is not a place for diagnosing. This actually seems to be a bug as with SSL enabled (in gateway connection mode and with the emulator in WSL), it works correctly.

OK, I guess you'll figure it out then.
For me, the key thing is that the connection mode greatly influences the ports used, so it's key to get to a point where you know what those port numbers are, and then validate that they are wired up correctly. You absolutely want to know the connection mode you're talking about and then follow the docs. In my case, I know Direct can be made to work locally, I know I want to use it in prod, and I know that you have  -p arguments related to them; hence I'd be targeting that. But if you know better, make sure you know the connection mode and/or ports.
Changing the cert validation policy comes after that - if there's a discrepancy or problem, you'll get a clear exception. You're not that point - it's hanging. (And, for avoidance of doubt, changing the cert check config does not affect the endpoint and/or connection mode.)

I am using Gateway connection mode locally as I do not want to setup any certificate and want to be able to bypass the SSL validation.

because this thread does not contain any exception or data that we could help with.

Yes because there is no exception.

Yes because there is no exception.

 If it's hung, it means the thread is being blocked, if not, it is still working or doing something, it might be failing to connect on the HTTP protocol, and that takes longer to surface the failure because the HTTP stack might be attempting to retry, it doesn't mean there is no exception, just that because of the nature of the connectivity problem, it takes longer to surface.


Dataset JSON is incorrect

Describe the bug
The JSON file generated for https://github.com/huggingface/datasets/blob/573f3d35081cee239d1b962878206e9abe6cde91/datasets/journalists_questions/journalists_questions.py is https://github.com/huggingface/datasets/blob/573f3d35081cee239d1b962878206e9abe6cde91/datasets/journalists_questions/dataset_infos.json.
The only config should be plain_text, but the first key in the JSON is journalists_questions (the dataset id) instead.
{
  "journalists_questions": {
    "description": "The journalists_questions corpus (version 1.0) is a collection of 10K human-written Arabic\ntweets manually labeled for question identification over Arabic tweets posted by journalists.\n",
    ...

Steps to reproduce the bug
Look at the files.
Expected results
The first key should be plain_text:
{
  "plain_text": {
    "description": "The journalists_questions corpus (version 1.0) is a collection of 10K human-written Arabic\ntweets manually labeled for question identification over Arabic tweets posted by journalists.\n",
    ...

Actual results
{
  "journalists_questions": {
    "description": "The journalists_questions corpus (version 1.0) is a collection of 10K human-written Arabic\ntweets manually labeled for question identification over Arabic tweets posted by journalists.\n",
    ...

As discussed, the metadata JSON files must be regenerated because the keys were nor properly generated and they will not be read by the builder:

Indeed there is some problem/bug while reading the datasets_info.json file: there is a mismatch with the config.name keys in the file...
In the meanwhile, in order to be able to use the datasets_info.json file content, you can create the builder without passing the name :

In [25]: builder = datasets.load_dataset_builder("journalists_questions")
In [26]: builder.info.splits
Out[26]: {'train': SplitInfo(name='train', num_bytes=342296, num_examples=10077, dataset_name='journalists_questions')}

After regenerating the metadata JSON file for this dataset, I get the right key:
{"plain_text": {"description": "The journalists_questions corpus (

Thanks!


Los Altos RIPA data is spread across sheets of an Excel file

This dataset contains information spread across many files that contain data for only part of a year. The minimum time duration currently allowed for a dataset is a year so if a dataset for a single year is split across multiple datasets, it must be combined into one dataset.
https://www.losaltosca.gov/police/page/racial-and-identity-profiling-act-ripa-data

This issue was made possible with the closure of https://github.com/openpolicedata/openpolicedata/issues/78. A methodology for including datasets like this in the source table was added. Examples of added datasets can be seen in https://github.com/openpolicedata/opd-data/commit/963777a931e2bc48eb8451428f096dc19a979210. It may be (at the time of writing, data.milwaukee.gov is not working) that this dataset now only needs to be added to the source table and then tested.
The CombineDataset data loader was added to handle CSV and Excel files spread across multiple files for a given year.


Vulkan: Connecting a TextureUniform node to the Normal Map output crashes the editor.

Godot version:

 #45879
OS/device including version:

OS: Windows 10 - Version 10.0.19041 - Build 19041
GPU: RTX 2070S - Driver version 461.40
Issue description:

The editor crashes when you connect a TextureUniform node to the Normal Map output in visual shaders.
These errors are shown in the console on launch of the editor:
ERROR: Condition "!shape->is_configured()" is true. Returning: Variant()
   at: shape_get_data (servers/physics_3d/physics_server_3d_sw.cpp:125)
ERROR: Condition "!d.has("cell_size")" is true.
   at: set_data (servers/physics_3d/shape_3d_sw.cpp:1688)
ERROR: Condition "!body" is true.
   at: body_attach_object_instance_id (servers/physics_3d/physics_server_3d_sw.cpp:614)
ERROR: Invalid ID.
   at: (servers/physics_3d/physics_server_3d_sw.cpp:1322)

This error is shown briefly before the editor crashes:
WARNING: A Thread object has been destroyed without wait_to_finish() having been called on it. Please do so to ensure correct cleanup of the thread.
     at: ~Thread (core/os/thread.cpp:111)
libc++abi: Pure virtual function called!

Apologies if this is a dup, I searched for similar issues the best I could. It seems shaders in general have had some issues the past couple of weeks, so this very well may be a broader problem than this specific circumstance.
Steps to reproduce:

Open the reproduction project.
Navigate to: MeshInstance3D -> mesh -> material -> VisualShader
Connect the TextureUniform to the Normal Map output in the shader graph.
Click on a different node in the scene tree, and then click back to the MeshInstance3D.
Editor will crash after a few seconds.

Minimal reproduction project:

NormalMapIssue.zip

I was digging around this issue a bit more today. When I re-opened the minimum reproduction project, I got this output in the console (no changes):
ERROR: Condition "!uniform_set" is true.
   at: draw_list_bind_uniform_set (drivers/vulkan/rendering_device_vulkan.cpp:6454)
ERROR: Uniforms supplied for set (3):
Set: 3 Binding: 0 Type: UniformBuffer Length: 128
Set: 3 Binding: 1 Type: Texture Length: 1
Set: 3 Binding: 2 Type: Texture Length: 1
Set: 3 Binding: 3 Type: Texture Length: 1
are not the same format as required by the pipeline shader. Pipeline shader requires the following bindings:
Set: 0 Binding: 1 Type: Sampler Length: 12
Set: 0 Binding: 2 Type: Sampler Length: 1
Set: 0 Binding: 3 Type: StorageBuffer Length: 0
Set: 0 Binding: 4 Type: StorageBuffer Length: 0
Set: 0 Binding: 5 Type: StorageBuffer Length: 0
Set: 0 Binding: 6 Type: UniformBuffer Length: 4224
Set: 0 Binding: 7 Type: StorageBuffer Length: 0
Set: 0 Binding: 8 Type: StorageBuffer Length: 0
Set: 0 Binding: 9 Type: Texture Length: 1
Set: 0 Binding: 10 Type: Texture Length: 1
Set: 0 Binding: 11 Type: StorageBuffer Length: 0
Set: 0 Binding: 12 Type: StorageBuffer Length: 0
Set: 0 Binding: 13 Type: UniformBuffer Length: 368
Set: 1 Binding: 0 Type: UniformBuffer Length: 2672
Set: 1 Binding: 1 Type: StorageBuffer Length: 0
Set: 1 Binding: 2 Type: Texture Length: 1
Set: 1 Binding: 3 Type: Texture Length: 1
Set: 1 Binding: 4 Type: Texture Length: 1
Set: 1 Binding: 5 Type: Texture Length: 1
Set: 1 Binding: 6 Type: Texture Length: 8
Set: 1 Binding: 7 Type: Texture Length: 8
Set: 1 Binding: 8 Type: StorageBuffer Length: 0
Set: 1 Binding: 9 Type: Texture Length: 1
Set: 1 Binding: 10 Type: Texture Length: 1
Set: 1 Binding: 11 Type: Texture Length: 1
Set: 1 Binding: 12 Type: Texture Length: 1
Set: 1 Binding: 13 Type: Texture Length: 1
Set: 1 Binding: 14 Type: Texture Length: 1
Set: 1 Binding: 15 Type: Texture Length: 1
Set: 1 Binding: 16 Type: Texture Length: 1
Set: 1 Binding: 17 Type: UniformBuffer Length: 896
Set: 1 Binding: 18 Type: Texture Length: 1
Set: 2 Binding: 0 Type: StorageBuffer Length: 0
Set: 3 Binding: 0 Type: UniformBuffer Length: 16
Set: 3 Binding: 1 Type: Texture Length: 1
   at: (drivers/vulkan/rendering_device_vulkan.cpp:6612)

I started a new project with the same scene tree setup as the minimum reproduction project, but put a texture uniform in the vertex shader node graph. After the original crash denoted at the top of the issue, I re-opened it and got this shader code output, which I have saved in the .txt file below because it's incredibly long.
ShaderErrorOutput.txt

I've continued testing, this time with a written shader. It happens exactly the same as with the visual shader.
UniformIssue.zip
Just uncomment the code in the MeshInstance3D mesh -> material -> shader, change the uniform value in the editor, click on a different object in the scene tree, and click back to the MeshInstance3D. It will crash with the error output in my original

Good news! This was resolved in 9ca0d66a3b51084eb528a85e133d9037b21bdd09.

Fixed by #46984.


Request to Support other VertexAI's LLM model Support

Feature request
Right now only text-bison model is support by Google PaLM. When tried code-bison its throwing below error:
---------------------------------------------------------------------------
ValidationError                           Traceback (most recent call last)
Cell In[39], line 1
----> 1 llm = VertexAI(model_name='code-bison')

File /opt/conda/envs/python310/lib/python3.10/site-packages/pydantic/main.py:341, in pydantic.main.BaseModel.__init__()

ValidationError: 1 validation error for VertexAI
__root__
  Unknown model publishers/google/models/code-bison; {'gs://google-cloud-aiplatform/schema/predict/instance/text_generation_1.0.0.yaml': <class 'vertexai.language_models._language_models._PreviewTextGenerationModel'>} (type=value_error)

Can we get other model supports as well?
Motivation
Support for many other PaLM models
Your contribution
I can update and test the code for different other models

Believe this was implemented in #6354

Just update to the latest version of langchain and you should be able to use the Codey models.

codey is supported, but still chat-bison and codechat-bison are throwing error

Just update to the latest version of langchain and google-cloud-aiplatform and you should be able to use the Codey models.

Any example link for this?

Same here:
model = TextGenerationModel.from_pretrained("code-bison")  # also tried code-bison@001

Error:

ValueError: Unknown model publishers/google/models/code-bison; {'gs://google-cloud-aiplatform/schema/predict/instance/text_generation_1.0.0.yaml': <class 'vertexai.language_models._language_models._PreviewTextGenerationModel'>}

Using latest stable google-cloud-aiplatform==1.30.1 at time of writing.

Seems that the newer model from Google is not supported. Getting the same error for text-unicorn@001:
pydantic.v1.error_wrappers.ValidationError: 1 validation error for ChatVertexAI
__root__
  Unknown model publishers/google/models/text-unicorn; {'gs://google-cloud-aiplatform/schema/predict/instance/chat_generation_1.0.0.yaml': <class 'vertexai.language_models.ChatModel'>} (type=value_error)

root
Unknown model publishers/google/models/gemini-pro; {'gs://google-cloud-aiplatform/schema/predict/instance/text_generation_1.0.0.yaml': <class 'vertexai.preview.language_models._PreviewTextGenerationModel'>} (type=value_error)

root Unknown model publishers/google/models/gemini-pro; {'gs://google-cloud-aiplatform/schema/predict/instance/text_generation_1.0.0.yaml': <class 'vertexai.preview.language_models._PreviewTextGenerationModel'>} (type=value_error)

Can you share a snippet of your code?

i was trying to use
I was following this, to use google models:
 Retrieval Augmented Generation (RAG) with Codey API
it uses code-bison, but i needed to use "codechat-bison" and when i changed the model name it gave me the error you are facing.
i managed to solve it doing the following:

Install langchain community :  pip install langchain-community
Use ChatVertexAI from  langchain_community instead of VertexAI when calling the model

this is the code i used:
from langchain_community.chat_models import ChatVertexAI

code_llm = ChatVertexAI(
    model_name="codechat-bison",
    max_output_tokens=2048,
    temperature=0.1,
    verbose=False,
    )

replacing this:
from langchain.llms import VertexAI

code_llm = VertexAI(
    model_name="code-bison@002",
    max_output_tokens=2048,
    temperature=0.1,
    verbose=False,
)

restart kernel and run again.
then the code worked fine, hope this helps.
i found this  here

I am getting error with medlm-large
pydantic.v1.error_wrappers.ValidationError: 1 validation error for ChatVertexAI
root
Unknown model publishers/google/models/medlm-large; {'gs://google-cloud-aiplatform/schema/predict/instance/chat_generation_1.0.0.yaml': <class 'vertexai.language_models.ChatModel'>} (type=value_error)

I am getting error with medlm-large
pydantic.v1.error_wrappers.ValidationError: 1 validation error for ChatVertexAI root Unknown model publishers/google/models/medlm-large; {'gs://google-cloud-aiplatform/schema/predict/instance/chat_generation_1.0.0.yaml': <class 'vertexai.language_models.ChatModel'>} (type=value_error)

I am facing same issue iwth medLm models, If you got solution please post it here


Fractional Knapsack Problem

Thanks for your contribution. Add your name to the contributors.md file. Also, write a brief description while making a PR.


Create OONI run buttons of interesting measurements

OONI run should provide an UI way of adding highlighted or previous generated OONI run links that could be reused for people that would like to reproduce measurements of other entities. Some examples include OONI run link buttons of all Wikipedia domains or a list of Tor directory authorities web servers.
This issue is based upon discussions on sessions occurred during IFF.

We have added such OONI Run buttons to our website in the get-involved Run section: https://ooni.org/get-involved/run/


Links in code of a storybook story contains "random" links

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[] Bug report
[ ] Performance issue
[ ] Feature request
[X] Documentation issue or request
[ ] Other... Please describe:

Current behavior
In the code section of stories, there are links to other stories, which not really make sense to me. So if I inspect code and want to copy it, the links are kind of annoying.
Expected behavior
The code of a story should not link into other stories.
Minimal reproduction of the problem with instructions

Navigate to https://reaflow.dev/?path=/story/demos-nested--nested-edges
Checkout the code and see links into other stories

I wrote this story in the documentation myself and this behaviour seems to come from storybook - not from the actual story code.

Hmmm - I believe this is the story plugin showing all stories for that file. Can you log a bug on the storybook repo instead?


concise.css/src/_atgrid.scss [container]

James,
The previous "issue" I opened and you accepted on this topic has a problem. Sorry.
If at the end of a container there is something similar to a dropdown menu with hidden items that extend beyond the container, a double scroll bar is displayed. See attached screenshot. The screenshots show Firefox, Chrome and Safari under OS-X.
I stumbled across a very simple and Concise solution.

Remove the "overflow: auto" we added to _atgrid.scss
In concise.css/src/_base.scss
:root {
--  background-color: getColor(background, body);

++ body {
++   background-color: getColor(background, body);
++ }
I am uncertain why this fix works, it makes no sense to me, but it does.
I believe this change is safe for the current version of Concise CSS.

Hi @FredHall
I also noticed that changing the background color from :root to body solved the problem, but because your solution also worked for me, I used that one, I didn't know it also created another scrollbar.
So I’ll add this change, but I’m not quite sure if it’s safe for a patch version because some users might be doing unexpected things so we can’t assume. What I’m going to do, is to remove the overflow: auto and change the background rule in a next major version.
But don’t worry, that won’t take too long, I have some plans for that as I mentioned in the last email :)

Alright, I have added the change in the development branch. Thanks!


Stutterting every 30 seconds when in game mode

I have ChimeraOS installed with an AMD GPU and CPU. While in game mode, about every 30 seconds to a minute, the system will freeze for a quarter of a second. It happens in any game/app, and even on the home screen while just scrolling through games. It makes it very difficult to play games with the frequent stuttering.

What specific hardware are you using?

What specific hardware are you using?

CPU: AMD Ryzen 7 3700X
GPU: AMD Radeon RX 5600XT
Not the latest top of the line, but should be plenty fast enough to run the game scope session, I’m assuming.
I turned on the performance monitor, and I can see the frame times remain consistent at 16ms, except when they randomly spike to 200ms about once or twice every minute.

We've seen some serious hitching with the 3000 series specifically in ChimeraOS. I'm not yet sure what the cause might be. I don't have the same issue when running EndeavourOS on similar hardware. (3700XT, Radeon VII)


migrate chaos mesh to the knuu

Closes #451
it is in draft mode because the tests are flaky, need to find a way to assure the tests always.
The tests are passing when run one by one, but when run altogether some of them fail. The attempts done to mitigate this behavior:

Running in sequence
Running in individual namespaces (each test has its own knuu object)
Added a function to wait until the Chaos rule is applied
Added some delay time to make sure the Chaos traffic shape rule is applied & effective
Increased the size of the bigfile to download to make the elapsed time measurement more accurate

Even tried to run them separately in different terminals, they also fail when they are running at the same time.

Can you add it to the github workflow matrix so we can see the failures in CI?
https://github.com/celestiaorg/knuu/blob/8d1b00fcae71de633bf5236bbec903d4b859303d/.github/workflows/knuu_testing.yml#L14
Can you share what the failures are that you are seeing locally?

Can you add it to the github workflow matrix so we can see the failures in CI?

done

Can you share what the failures are that you are seeing locally?

Sure. Here is when I run it all together with isolated namespaces sequentially:
=== RUN   TestRunSuite
=== RUN   TestRunSuite/TestBandwidth
time="2024-06-26T10:41:51+02:00" level=info msg="The .env file does not exist, continuing without loading environment variables."
time="2024-06-26T10:41:51+02:00" level=info msg="LOG_LEVEL: info" file="log/logger.go:34"
    suite_setup_test.go:67: Scope: 20240626-104151-940
    chaosmesh_test.go:140: Waiting 10s for chaos mesh to settle
    chaosmesh_test.go:145: Time taken for wget before bandwidth: 27.345988762s 	 after bandwidth: 12.980991868s
    chaosmesh_test.go:147:
        	Error Trace:	/home/moji/celestia/knuu/e2e/chaosmesh/chaosmesh_test.go:147
        	Error:      	"12.980991868s" is not greater than "27.345988762s"
        	Test:       	TestRunSuite/TestBandwidth
        	Messages:   	Time after bandwidth is not longer than time before bandwidth
=== RUN   TestRunSuite/TestCorrupt
time="2024-06-26T10:42:56+02:00" level=info msg="The .env file does not exist, continuing without loading environment variables."
time="2024-06-26T10:42:56+02:00" level=info msg="LOG_LEVEL: info" file="log/logger.go:34"
    suite_setup_test.go:67: Scope: 20240626-104256-035
    chaosmesh_test.go:108: Waiting 10s for chaos mesh to settle
    chaosmesh_test.go:113: Time taken for wget before corrupt: 29.818175866s 	 after corrupt: 14.994630351s
    chaosmesh_test.go:115:
        	Error Trace:	/home/moji/celestia/knuu/e2e/chaosmesh/chaosmesh_test.go:115
        	Error:      	"14.994630351s" is not greater than "29.818175866s"
        	Test:       	TestRunSuite/TestCorrupt
        	Messages:   	Time after corrupt is not longer than time before corrupt
=== RUN   TestRunSuite/TestDelay
time="2024-06-26T10:44:04+02:00" level=info msg="The .env file does not exist, continuing without loading environment variables."
time="2024-06-26T10:44:04+02:00" level=info msg="LOG_LEVEL: info" file="log/logger.go:34"
    suite_setup_test.go:67: Scope: 20240626-104404-580
    chaosmesh_test.go:27: Waiting 10s for chaos mesh to settle
    chaosmesh_test.go:32: Time taken for wget before delay: 26.889980033s 	 after delay: 26.170361577s
    chaosmesh_test.go:34:
        	Error Trace:	/home/moji/celestia/knuu/e2e/chaosmesh/chaosmesh_test.go:34
        	Error:      	"26.170361577s" is not greater than "26.889980033s"
        	Test:       	TestRunSuite/TestDelay
        	Messages:   	Time after delay is not longer than time before delay
=== RUN   TestRunSuite/TestDuplicate
time="2024-06-26T10:45:19+02:00" level=info msg="The .env file does not exist, continuing without loading environment variables."
time="2024-06-26T10:45:19+02:00" level=info msg="LOG_LEVEL: info" file="log/logger.go:34"
    suite_setup_test.go:67: Scope: 20240626-104519-042
    chaosmesh_test.go:81: Waiting 10s for chaos mesh to settle
    chaosmesh_test.go:86: Time taken for wget before duplicate: 27.027887443s 	 after duplicate: 26.465900458s
    chaosmesh_test.go:88:
        	Error Trace:	/home/moji/celestia/knuu/e2e/chaosmesh/chaosmesh_test.go:88
        	Error:      	"26.465900458s" is not greater than "27.027887443s"
        	Test:       	TestRunSuite/TestDuplicate
        	Messages:   	Time after duplicate is not longer than time before duplicate
=== RUN   TestRunSuite/TestLoss
time="2024-06-26T10:46:42+02:00" level=info msg="The .env file does not exist, continuing without loading environment variables."
time="2024-06-26T10:46:42+02:00" level=info msg="LOG_LEVEL: info" file="log/logger.go:34"
    suite_setup_test.go:67: Scope: 20240626-104642-121
    chaosmesh_test.go:54: Waiting 10s for chaos mesh to settle
    chaosmesh_test.go:59: Time taken for wget before loss: 13.95433273s 	 after loss: 14.305350963s
--- FAIL: TestRunSuite (343.31s)
    --- FAIL: TestRunSuite/TestBandwidth (64.09s)
    --- FAIL: TestRunSuite/TestCorrupt (68.55s)
    --- FAIL: TestRunSuite/TestDelay (74.46s)
    --- FAIL: TestRunSuite/TestDuplicate (83.08s)
    --- PASS: TestRunSuite/TestLoss (53.13s)
FAIL
FAIL	github.com/celestiaorg/knuu/e2e/chaosmesh	343.448s
FAIL

and when we run them one by one, they usually pass:
go test -v ./e2e/chaosmesh/ -timeout 60m -count=1 --run TestRunSuite/TestBandwidth
go test -v ./e2e/chaosmesh/ -timeout 60m -count=1 --run TestRunSuite/TestCorrupt
go test -v ./e2e/chaosmesh/ -timeout 60m -count=1 --run TestRunSuite/TestDelay
go test -v ./e2e/chaosmesh/ -timeout 60m -count=1 --run TestRunSuite/TestDuplicate
go test -v ./e2e/chaosmesh/ -timeout 60m -count=1 --run TestRunSuite/TestLoss
=== RUN   TestRunSuite
=== RUN   TestRunSuite/TestBandwidth
time="2024-06-26T10:54:40+02:00" level=info msg="The .env file does not exist, continuing without loading environment variables."
time="2024-06-26T10:54:40+02:00" level=info msg="LOG_LEVEL: info" file="log/logger.go:34"
    suite_setup_test.go:67: Scope: 20240626-105440-946
    chaosmesh_test.go:140: Waiting 10s for chaos mesh to settle
    chaosmesh_test.go:145: Time taken for wget before bandwidth: 13.117046776s 	 after bandwidth: 27.103344713s
--- PASS: TestRunSuite (65.91s)
    --- PASS: TestRunSuite/TestBandwidth (65.91s)
PASS
ok  	github.com/celestiaorg/knuu/e2e/chaosmesh	66.045s
=== RUN   TestRunSuite
=== RUN   TestRunSuite/TestCorrupt
time="2024-06-26T10:55:49+02:00" level=info msg="The .env file does not exist, continuing without loading environment variables."
time="2024-06-26T10:55:49+02:00" level=info msg="LOG_LEVEL: info" file="log/logger.go:34"
    suite_setup_test.go:67: Scope: 20240626-105549-726
    chaosmesh_test.go:108: Waiting 10s for chaos mesh to settle
    chaosmesh_test.go:113: Time taken for wget before corrupt: 26.214538065s 	 after corrupt: 26.921705398s
--- PASS: TestRunSuite (77.70s)
    --- PASS: TestRunSuite/TestCorrupt (77.70s)
PASS
ok  	github.com/celestiaorg/knuu/e2e/chaosmesh	77.845s
=== RUN   TestRunSuite
=== RUN   TestRunSuite/TestDelay
time="2024-06-26T10:57:10+02:00" level=info msg="The .env file does not exist, continuing without loading environment variables."
time="2024-06-26T10:57:10+02:00" level=info msg="LOG_LEVEL: info" file="log/logger.go:34"
    suite_setup_test.go:67: Scope: 20240626-105710-157
    chaosmesh_test.go:27: Waiting 10s for chaos mesh to settle
    chaosmesh_test.go:32: Time taken for wget before delay: 28.403200834s 	 after delay: 29.653790723s
--- PASS: TestRunSuite (83.11s)
    --- PASS: TestRunSuite/TestDelay (83.11s)
PASS
ok  	github.com/celestiaorg/knuu/e2e/chaosmesh	83.239s
=== RUN   TestRunSuite
=== RUN   TestRunSuite/TestDuplicate
time="2024-06-26T10:58:36+02:00" level=info msg="The .env file does not exist, continuing without loading environment variables."
time="2024-06-26T10:58:36+02:00" level=info msg="LOG_LEVEL: info" file="log/logger.go:34"
    suite_setup_test.go:67: Scope: 20240626-105836-117
    chaosmesh_test.go:81: Waiting 10s for chaos mesh to settle
    chaosmesh_test.go:86: Time taken for wget before duplicate: 14.752317089s 	 after duplicate: 13.546388261s
    chaosmesh_test.go:88:
        	Error Trace:	/home/moji/celestia/knuu/e2e/chaosmesh/chaosmesh_test.go:88
        	Error:      	"13.546388261s" is not greater than "14.752317089s"
        	Test:       	TestRunSuite/TestDuplicate
        	Messages:   	Time after duplicate is not longer than time before duplicate
--- FAIL: TestRunSuite (52.02s)
    --- FAIL: TestRunSuite/TestDuplicate (52.02s)
FAIL
FAIL	github.com/celestiaorg/knuu/e2e/chaosmesh	52.155s
FAIL
=== RUN   TestRunSuite
=== RUN   TestRunSuite/TestLoss
time="2024-06-26T10:59:30+02:00" level=info msg="The .env file does not exist, continuing without loading environment variables."
time="2024-06-26T10:59:30+02:00" level=info msg="LOG_LEVEL: info" file="log/logger.go:34"
    suite_setup_test.go:67: Scope: 20240626-105930-866
    chaosmesh_test.go:54: Waiting 10s for chaos mesh to settle
    chaosmesh_test.go:59: Time taken for wget before loss: 14.224850495s 	 after loss: 26.397818292s
--- PASS: TestRunSuite (64.47s)
    --- PASS: TestRunSuite/TestLoss (64.47s)
PASS
ok  	github.com/celestiaorg/knuu/e2e/chaosmesh	64.594s

Can we close this PR?

Can we close this PR?
Well if we decide not to use Chaos mesh in knuu, then we can close it.

Can we close this PR?

Well if we decide not to use Chaos mesh in knuu, then we can close it.

The teams do not use knuu with network shaping and it seems they will not in the near future.


Support ./gradlew modernizer

https://issues.apache.org/jira/browse/SOLR-XXXXX
A long time ago I was looking at ways to improve the build process to detect out of date usages. This adds ./gradlew modernizer that will find things that can be replaced.
based on https://github.com/andygoossens/gradle-modernizer-plugin
I went through all the failures and added some exclusions to make things clean. Mostly simple stuff.

Yup @stillalex this is just finding MOSTLY charset related changes right now. There are some disabled - https://github.com/apache/solr/pull/2006/files#diff-9257ac5900027c3c54700c25c646e201f7fcec13c7f39ecedd288f99c0ecb5f2R30. I don't know if this is worth it. We have cleaned up a lot of older usages with errorprone already.

Interestingly errorprone https://github.com/google/error-prone/releases/tag/v2.23.0 might have just addressed the charset stuff so this PR might not be necessary.

interesting. if they add this then yeah, I agree this change is on longer useful.

Yea https://github.com/apache/solr/pull/2035 covers MOST of this.


Automate multijvm compiler tck

This PR Automates multijvm compiler tck :

Changes in mk:

Defined commands to generate jtb, start agent, start harness on remote node.
Broke down compiler tests to subgroups.

Changes in playlist.xml :

MultiJVM compiler test targets added with xxx_multijvm suffix, using dev level.

JavaTestUtil: Generates command jtb file and summary report.
agent-drive.sh: Aids in running harness and agent processes and kills agent once done.
Jenkinsfilebase: When running tcks, call runTest() with Jenkins credentials, as needed for remote node connection.

Related : backlog/issues/957.
Signed-off-by: Mesbah-Alam<EMAIL_ADDRESS>

Test wth dev level : Grinder_JCK/974.

Quick test on JDK8 x86-64_linux in TC_Grinder/3118


Show the degrees when using rotation locking

@lipis may I pick this up?

Actually, I'd maybe discuss this first.
This feature may be more suitable to an "architect" mode or something, when we later introduce grid/object snapping etc.
If we add this right now, people will also want to add things like x/y coord labels when moving elements, width/height` labels and similar, and soon drawing on excal won't feel like drawing on a whiteboard anymore.


API error causes skeleton to remain and Related section to only show titles

When an API error occurs during the use of the Tool call, the following issues arise:
The skeleton of the page remains visible, indicating that the content failed to load properly.
The Related section, which is supposed to display relevant information, only shows the titles without any accompanying content.

Related: #14


Some issues in code

Actually I just review forks.
That part of functionality is not critical for me, but maybe it makes sense too... Please review 2 last commits.
https://github.com/corkscrw/node-ternary-search-trie/commits/master
searchWithinXXXDistance callback value arg bug fix
and
callback gets distance as 3rd arg
Kindly make this library perfect :sparkles: ... after 5 years :raised_hands:

The 1st commit has already been merged long ago. Now added the new feature in the 2nd one. :tada: v5.5.0
Haven't seen a good library for ternary tries in five years? ;)

Thank you a lot. I don't like rewrite https://github.com/tomi/vscode-rf-language-server 😄
You made the world a better place.


please add copyright statement (including years) to source.

src/jquery.nearest.js is missing copyright statement. Unfortunately link to license is not sufficient... Thanks.

Out of interest, why is a link to the licence not sufficient? jQuery itself only contains a link to its licence, not the full text: http://code.jquery.com/jquery-2.1.3.js

Out of interest, why is a link to the licence not sufficient? jQuery itself only contains a link to its licence, not the full text: http://code.jquery.com/jquery-2.1.3.js

IMHO jQuery is not following best practices. For instance they include link to their own site (instead of the http://opensource.org/licenses/MIT ) so one should open the link to make sure it is really what they meant by "MIT" license;
Free Software Foundation considers "MIT License" ambiguous because it refers to different licenses:

https://en.wikipedia.org/wiki/MIT_License

In your case only text of the license contains copyright statement. You can include copyright statement to .js file header accompanied by link to the text of the license but the latter is short so it will take only little space. For software which is often embedded to other projects It is good to document licensing terms and copyright clearly.

Also there is an obvious reason to include license instead of a link: imagine situation when some time later project changed hosting or taken down and bundled copies of script links to dead URL... I've seen many such cases where it is hard to assess license compatibility or even to check whether re-distribution is allowed...

OK, I've added a LICENSE.txt file in the root of this repository (and therefore on the documentation site as well).
I've also updated the js file comment headers to include years of copyright and a link to the hosted version of LICENSE.txt
In order to keep with the "don't ever modify a release" rules of semver, I've released version 1.3.1 with the new comment headers.
Let me know if there are any problems with this new wording. :)
Cheers,
Gil

All good, thank you. :)


Use 'heavyhash' for any block header operation instead of plain hash

merge heavyhash

This commit is the reason Optical Bitcoin is so slow, cannot understand why it has been done.


Manual Installation

CocoaPods is an awesome tool and make our life really easier, but there are some devs who still don't know how to use them.
It would be cool to add the Manual installation guide in your README.md. You can take a look at my iOS Readme Template to see how you can do it.

Hello @lfarah thank you for comment. I will add more informations about the installation process :)
Thank you

You're welcome 😉


Mi 9T adb running can pull app list but not file manager no screen monitor

Superb idea. Good gui. When i tried using with my Mi 9t the device name is shown, I can pull liste of apps. I can see batteery info on the device info screen. But i cannot use file manager. Although su/root has been selected in the settings the file listing in the device side shows nothing. And I also cannot see screen monitoring.

More info:
I am using windows 10.
It seems this is caused by the device listing tab trying to list \sdcard\Downloads but it should be /sdcard/Downloads. When I try to edit this manually it automatically converts to "".


Warn Message for developer

mihome-vacuum.0
2020-10-03 18:30:28.426
warn
(28030) This object will not be created in future versions. Please report this to the developer.

mihome-vacuum.0
2020-10-03 18:30:28.418
warn
(28030) Object mihome-vacuum.0.info.dnd is invalid: obj.common.type has an invalid value (switch) but has to be one of number, string, boolean, array, object,

I have also received it

should be fixed


fix(storybook): fix storybook bug with connectedform story

Overview

Storybook fix for ConnectedFormGroup

PR Checklist
Docs only change :)
Testing Instructions
ConnectedFormGroup story should open 🔥

Don't make me tap the sign.
PR Links and Envs
Docs only change :)

📬Published Alpha Packages:@codecademy/gamut@55.14.5-alpha.aaf14b.0@codecademy/gamut-kit@0.6.393-alpha.aaf14b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://660d816a5528790cdce5218c--gamut-preview.netlify.appDeploy Logs


feat: Add several missing headers

Re-export SERVER from hyper::header::Server as ssdp::header::Server
Re-export LOCATION hyper::header::Location as ssdp::header::Location
Re-export CACHE-CONTROL hyper::header::CacheControl as ssdp::header::CacheControl
Re-export hyper::header::CacheDirective as ssdp::header::CacheDirective

Coverage decreased (-33.5%) to 57.662% when pulling f2aea899b7ed19e35a42b0098c86f01302309053 on icorderi:feature/missing-headers into ea3ce7bac7895125f48ab23c15ed5bd44c5acf43 on GGist:master.

Actually these headers are already in hyper http://hyper.rs/hyper/v0.9.3/hyper/header/index.html and users of our library can use any of the headers mentioned on that page with our messages.
However, if you wanted to re-export those headers from hyper in ssdp so that users dont have to import hyper in order to use them, that would probably be ok since those headers are commonly set in SSDP messages.

Coverage decreased (-35.7%) to 55.476% when pulling d1a783e40a4e431f3f56b38db49cc141f08ffe5e on icorderi:feature/missing-headers into ea3ce7bac7895125f48ab23c15ed5bd44c5acf43 on GGist:master.

@GGist, I got rid of the explicit unsafe impl Sync, and Send for the headers.
Types are automatically Send and Sync unless something in them is !Send or !Sync.

Coverage decreased (-36.4%) to 54.775% when pulling 770218468a0855a46a3fed0a7f11420d91180ec2 on icorderi:feature/missing-headers into ea3ce7bac7895125f48ab23c15ed5bd44c5acf43 on GGist:master.

Yeah I am not sure why I had those in there. This was one of the first crates I published so I think I misunderstood the auto derivation of Send and Sync back then.
On a side note, I wish the Rust developers made types #[derive()] those traits (or at least show it in the documentation) as I have found that for types that wrap things like UnsafeCell, there is no way to tell that they aren't Sync without trying to compile.
Anyways, these changes look good. I see that some of the changes from the other pull request are in this one. If those aren't necessary for the code in this pull request (in particular, feat: Add several missing headers) then you can remove those and I can merge this in now. You could then go back and rebase the other pull request as I left a comment on the other pull request that affects one of the commits.

@ggist, ok I rebased it on master instead of being on top of the feature/ipv6 branch.
I also moved the rustfmt.toml to this PR and added my IDE artifacts to the .gitignore.

Feel free to tweak the rustfmt.toml line size to your liking, the default 80chars is a bit too short for my taste. rustfmt --config-help will give you a lot of options.

Coverage decreased (-33.2%) to 57.969% when pulling decd3bdd831729d23b69b74ce2a315b4edc69390 on icorderi:feature/missing-headers into ea3ce7bac7895125f48ab23c15ed5bd44c5acf43 on GGist:master.

Ok thanks, i'll check that out.


Documentation - Volume Replication Recovery

Hi, I have been considering using seaweed in some project i`m starting (best alternative for S3 i found sofar :)), and i have  several questions regards how seaweed handles Volume server failure.

What happens if Volume Server fails ? can i still perform writes or "volumes" will be "read-only" ?
Is there any rebalancing mechanism in case of adding/removing/failing a node ? i.e. rebalancing the data across remaining nodes ?
What happens when failed nodes returns, how it "catches up" data it missed (assuming volumes are still writable during "failure time" ?

Also i looked at the code  especially at store_replicate.go at ReplicatedWrite method, and from what i understand if one of replicas failed, request is considered as failed correct ?

One more observation that i made is that if "Master volume" node looses communication with slave volume node is sets volume status to "unavailable" and avoids further writes to the volume
Is this correct or am i missing something ?

One more observation that i made is that if "Master volume" node looses communication with slave volume node is sets volume status to "unavailable" and avoids further writes to the volume
Is this correct or am i missing something ?

What happens if Volume Server fails ? can i still perform writes or "volumes" will be "read-only" ?
Is there any rebalancing mechanism in case of adding/removing/failing a node ? i.e. rebalancing the data across remaining nodes ?
What happens when failed nodes returns, how it "catches up" data it missed (assuming volumes are still writable during "failure time" ?

These are good questions and should be included in the wiki.
If the volume server fails, the write request should restart again, request a new file id from the master.
Rebalancing feature is in weed shell by volume.balance command.
Volumes are readonly if volume peers are down.

If one replica write fails, the whole write request fails. A new file id would usually go to a different set of volumes.
If the volume server disconnects from the master, no writes can go to the volume server.

What happens if Volume Server fails ? can i still perform writes or "volumes" will be "read-only" ?
Is there any rebalancing mechanism in case of adding/removing/failing a node ? i.e. rebalancing the data across remaining nodes ?
What happens when failed nodes returns, how it "catches up" data it missed (assuming volumes are still writable during "failure time" ?

These are good questions and should be included in the wiki.
If the volume server fails, the write request should restart again, request a new file id from the master.
Rebalancing feature is in weed shell by volume.balance command.
Volumes are readonly if volume peers are down.

So there is no automatic rebalancing in case of volume server failure ?

Usually you don't want to instantly rebalance as soon as one node is down.
You can schedule some periodic cron jobs for that.
On Sat, May 11, 2019 at 2:44 PM Alexander Solganik<EMAIL_ADDRESS>wrote:

What happens if Volume Server fails ? can i still perform writes or
"volumes" will be "read-only" ?
Is there any rebalancing mechanism in case of adding/removing/failing a
node ? i.e. rebalancing the data across remaining nodes ?
What happens when failed nodes returns, how it "catches up" data it missed
(assuming volumes are still writable during "failure time" ?
These are good questions and should be included in the wiki.
If the volume server fails, the write request should restart again,
request a new file id from the master.
Rebalancing feature is in weed shell by volume.balance command.
Volumes are readonly if volume peers are down.
So there is no automatic rebalancing in case of volume server failure ?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/chrislusf/seaweedfs/issues/956#issuecomment-491546093,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALYX37Y2TDB553KYHHF6PLPU4447ANCNFSM4HMI4U5Q
.

@sashas-anyvision You can checkout how the Docker container is setup to run the re-balance on a schedule: https://github.com/chrislusf/seaweedfs/pull/922/files.


Update space styling for Item-Displays in Clinical-Item-Collection

Issue Description
Fix line-height/margin/padding/vertical-aligment of Displays when presented in the table view of the Clinical Item Collection, so that all text “baselines” are aligned and icons and text all appear to look centered vertically when there is only one line of text content.
Issue Type

[ ] New Feature
[x] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
All Text and icons, when there is only a single line of content, in the table view should appear visually "centered" or aligned to the same baseline.
Current Behavior
Start and End Accessories (icons) correctly display vertically centered. The Clinical Item Displays (text) are correctly aligned top but when single line of text, they align too high visually (row height could be too tall) and icons, the basic displays, and the comments - all align to different visual baselines.
Required Enhancement
Will resolve fix comment added in PR #91

@neilpfeiffer PR https://github.com/cerner/terra-clinical/pull/104 should have fixed this issue. Can you check to see if this was resolved when you get a chance?

@emilyrohrbough Excellent. Looks good!
Closing this issue as being resolved after merging PR #104


k0s: upgrade Argo CD from 2.5.4 to 2.5.15

Release notes at:
https://github.com/argoproj/argo-cd/releases/tag/v2.5.15

bors r+


Revert "fix only check missing tablets when commit i…

…nfo is not empty (#29326)"
This reverts commit 4634c723d7fb86f434060c0e6fa6168d92630cb9.
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Update BASE_VERSION to 1.13-2022-05-17T19-01-07

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security_istio_release-1.13
154af81481e6f65b5bf9c1dfcde85a4d93fdc37c
link
true
/test integ-security_istio_release-1.13

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security_istio_release-1.13
154af81481e6f65b5bf9c1dfcde85a4d93fdc37c
link
true
/test integ-security_istio_release-1.13

integ-pilot-multicluster_istio_release-1.13
154af81481e6f65b5bf9c1dfcde85a4d93fdc37c
link
true
/test integ-pilot-multicluster_istio_release-1.13

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster_istio_release-1.13
154af81481e6f65b5bf9c1dfcde85a4d93fdc37c
link
true
/test integ-pilot-multicluster_istio_release-1.13

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Remove defaultvalue in swagger documentation

In gitlab by @sylvaingaudan on Jul 19, 2017, 17:11
In the swagger documentation of the REST services, some default values are provided. For instance, sort is based on the country field, which is probably unexisting from most collections. Those default/example values should be removed.

In gitlab by @sylvaingaudan on Jul 21, 2017, 12:17
changed milestone to %6

In gitlab by @sfalquier on Jul 28, 2017, 10:27
added ~2377174 label

In gitlab by @sylvaingaudan on Aug 10, 2017, 08:31
changed milestone to %10

In gitlab by @sfalquier on Aug 18, 2017, 10:02
added ~2424199 and removed ~2424198 labels

In gitlab by @sfalquier on Aug 18, 2017, 10:14
mentioned in commit 19d07206fbf0968e902cbff30b86a9ad8774de39

In gitlab by @sfalquier on Aug 18, 2017, 10:15
mentioned in merge request !80

In gitlab by @sfalquier on Aug 18, 2017, 10:15
assigned to @sfalquier

In gitlab by @sfalquier on Aug 18, 2017, 10:41
mentioned in commit 1be7a5ee7a015f167e0f43f61d395cfb723e4908

In gitlab by @MohamedHamouGisaia on Aug 18, 2017, 16:12
closed via merge request !80

In gitlab by @MohamedHamouGisaia on Aug 25, 2017, 11:48
mentioned in commit 999efa7cddb5a510bd9f1b342bc6e26ab316cdee


cleanup

WARNING: HEAVILY OPINIONATED PR WITH BREAKING CHANGES AHEAD
This PR cleans up and polishes the entire package, making only relatively few breaking changes.
Fixes #168
Breaking changes:

Removes native es6 module support (this should be done in the same manner as coffe-script etc.)
Node v6+ only
Drop support for sequelize v3 and below (should still work but removed tests)

Notable changes:

Use npm (it's actually decent now)
Updates EVERYTHING
Removed babel and gulp entirely

Using typescript and ts-node as build system instead

Removed all runtime dependencies except bluebird
Beefed up linting and applied:

"prefer-arrow-callback": "error",
"prefer-const": "error",
"arrow-body-style": ["error", "as-needed"],

Remove test cases for coffee-script

Rationale: This doesn't really add any value, anyone can use a custom resolver, this functionality is sufficiently tested

@PascalPflaum any comments?

Hi @SimonSchick. I agree with most of your changes and the way how you did them. However I will not merge this PR at it is right now. I will spread the several changes you made to include simple improvements (updating dependencies, npm instead yarn, updated node versions, etc.) in the next minor release and to recheck bigger changes for the next major release.
Thanks for your contribution, a bigger cleanup was way overdue and on my list since I took responsibility for this package.

That's fine with me, cherry-pick all you want.
I just wanted to contribute my personal fork so others can (potentially) benefit.

Closing this PR but not deleting my branch as this will never be merged :)


OpenAi API key required for the project to run

You need to insert your own API keys in a .env file if you are trying to setup the project locally.

You need to insert your own API keys in a .env file if you are trying to setup the project locally.

Okay, thanks for the help !


GC resistant subclasses

Extracted from #14
Requires #26
This (incomplete) PR allows having pointers to crystal objects in your gobject subclass. This is achieved by splitting the subclass into a pointer class and a data class (which does the actual work).
As previously the implementation do ... end macro has been criticised, it has now been removed.
With this (and the current state of #26), the syntax for defining a subclass is as follows:
class TestPaintable < GObject::Object
  include Gdk::Paintable

  @[GObject::PrivateDataClass]
  class TestPrivate < self
    @test = "hi"

    def do_snapshot(snapshot, width, height)
      puts "Snapshot: #{width}x#{height}, snapshot: #{snapshot}, test: #{@test}, self: #{self}"
    end
  end
end

There only needs to be one class inheriting from the pointer class, having the annotation PrivateDataClass.

I was thinking about the problem of garbage collected objects with Crystal vfuncs and alive in C and had a idea, what if:
obj = MyTakeAwayObj.new
some_api.add_obj(obj)
obj.keep_alive!

This GObject::Object#keep_alive! would:

Add the object to the closure manager, so it wont get GC'ed.
Store a GWeakRef for the object in ClosureManager.
The callback, called at object disposal, would just remove the pointer from the ClosureManager, letting the object be GCed.

Pros
Pretty simple, minimal overhead
Cons
The solution isn't fully transparent to the user, but the exceptions throw can teach it.

I was thinking about the problem of garbage collected objects with Crystal vfuncs and alive in C and had a idea, what if:
obj = MyTakeAwayObj.new
some_api.add_obj(obj)
obj.keep_alive!

This GObject::Object#keep_alive! would:
* Add the object to the closure manager, so it wont get GC'ed.

* Store a `GWeakRef` for the object in ClosureManager.

* The callback, called at object disposal, would just remove the pointer from the `ClosureManager`, letting the object be GCed.

Pros
Pretty simple, minimal overhead
Cons
The solution isn't fully transparent to the user, but the exceptions throw can teach it.

I had also experimented with weakrefs but couldn't get it working. It's neither a good, nor a bad solution but at least it's better than the current state.
Maybe it's worth looking at implementations of GObject in other garbage-collected languages.

Maybe it's worth looking at implementations of GObject in other garbage-collected languages.

Yes, always is. However IMO this isn't a high priority, since there's a bunch of important stuff missing yet.

I was thinking about the problem of garbage collected objects with Crystal vfuncs and alive in C and had a idea, what if:
obj = MyTakeAwayObj.new
some_api.add_obj(obj)
obj.keep_alive!

This GObject::Object#keep_alive! would:
* Add the object to the closure manager, so it wont get GC'ed.

* Store a `GWeakRef` for the object in ClosureManager.

* The callback, called at object disposal, would just remove the pointer from the `ClosureManager`, letting the object be GCed.

Pros
Pretty simple, minimal overhead
Cons
The solution isn't fully transparent to the user, but the exceptions throw can teach it.

I just thought about this a little more and I think:

this is a very suboptimal solution
It's probably not even possible
It's not "minimal overhead"

Why it is suboptimal:
If we need to have a keep_alive! call inside crystal, we cannot instantiate these subclasses from outside crystal.
One important use-case which requires this is instantiating widgets in .ui files, which I want to make use of to have cleaner code.
Why it probably won't work:
This is a perfect example of a retain cycle.
As long as the GObject ref_count is > 0, a reference to the object is set inside crystal.
This reference itself will keep the object alive, but the object itself is the thing increasing the reference count.
Why it has significant overhead:
If we remove references to our object from the c world, the pointer would be removed from the ClosureManager. This means that we need to redo the keep_alive! call, which could be complicated in some situations.

I think we must split the implementation into two objects - one "dumb" pointer class (just like all other GObject classes generated by gi-crystal) and one class storing the data.
I'm not sure if we can hide the second class somehow, but I don't think this is possible.

Yes, the use case of letting the Gtk::Builder instantiate the GObject is the most important one.
Maybe it's possible if we let the instance_init instantiate the Crystal object when the object is created in C world, maybe passing the Crystal instance on user_data pointer... then if the user_data pointer is nil, the instance_init creates the Crystal instance... taking care to avoid double init and infinite loops and probably needing to add the GWeakRef to ClosureManager.
But I think there are still a lot of other important things missing in the bindings, so despite of this being a very important topic, it isn't in the priority list yet, since there's workarounds possible, not always beautiful ones, but possible.
My main concern is: The end solution must still look like ordinary Crystal code and don't have undefined behavior if the user do this or that.


Robustness to driver pod taking time to create

Purpose of this PR
Improve reliability.
Closes: #2302
Proposed changes:

Add a grace period for the driver pod to be created.
Grace period is controlled by a new config option driver-pod-creation-grace-period. Default is 10 seconds.
Add 3 new tests around reconciling submitted status, including the new grace period functionality. This is the majority of the lines changed.
Expose the new config option in the helm chart and add a helm test

Change Category

[ ] Bugfix (non-breaking change which fixes an issue)
[x] Feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that could affect existing functionality)
[ ] Documentation update

Checklist

[x] I have conducted a self-review of my own code.
[x] I have updated documentation accordingly - I think its sufficient just to add a description on the new argument on the helm chart and regenerate the helm docs.
[x] I have added tests that prove my changes are effective or that my feature works.
[x] Existing unit tests pass locally with my changes.

Additional Notes
Some logs of a real example on our prod cluster where the spark application was saved by the grace period added in this PR.
Explore-logs-2024-11-11 12_09_13.txt

The CI failure looks unrelated
• [FAILED] [300.030 seconds]
Example SparkApplication spark-pi-python [It] Should complete successfully
/home/runner/work/spark-operator/spark-operator/test/e2e/sparkapplication_test.go:383

  [FAILED] Unexpected error:
      <*fmt.wrapError | 0xc001b63360>:
      client rate limiter Wait returned an error: context deadline exceeded
      {
          msg: "client rate limiter Wait returned an error: context deadline exceeded",
          err: <context.deadlineExceededError>{},
      }
  occurred
  In [It] at: /home/runner/work/spark-operator/spark-operator/test/e2e/sparkapplication_test.go:386 @ 12/04/24 12:25:47.015

I'll try rebasing on latest main

@Tom-Newton Thanks for improving the robustness of Spark operator.

@Tom-Newton Thanks for improving the robustness of Spark operator.
/lgtm

@Tom-Newton Thanks for improving the robustness of Spark operator.

Thanks for reviewing :slightly_smiling_face:


Switch to @conda-forge/spacy team

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.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ No valid build backend found for Python recipe for package confection using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.
ℹ PyPI default URL is now pypi.org, and not pypi.io. You may want to update the default source url.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12282457498. Examine the logs at this URL for more detail.


Support @bind-ing Tyne table columns

This would let a caller set a default/initial value for column selections

The setting of this would need to be picked up by OnParametersSetAsync as it changing should trigger a table reload (which is async).

Closed in favour of #73


bower package min file contains merge error tags

Hi,
There is broken angular-local-storage.min.js file in dist folder of last bower package. It contains git merge tags like this:
/**
 * An Angular module that gives you access to the browsers local storage
<<<<<<< HEAD
 * @version v0.5.2 - 2016-09-28
 * @link https://github.com/grevory/angular-local-storage
 * @author grevory<EMAIL_ADDRESS> * @license MIT License, http://www.opensource.org/licenses/MIT
 */

//# sourceMappingURL=angular-local-storage.min.js.map
=======
 * @version v0.1.5 - 2015-01-28
 * @link https://github.com/grevory/angular-local-storage
 * @author grevory<EMAIL_ADDRESS> * @license MIT License, http://www.opensource.org/licenses/MIT

>>>>>>> ec3e609916307f2da82c6714c37bee754e1d10b6

Strange, I don't see it. I rebuilt the dist files since they looked a version behind. Try now and let me know.

I have this problem with merge now. Problem is actual

I see it. Looks like a problem with when I added the tag. Try now.

Yes, thx. Problem is fixed, min file is ok

Sorry about the trouble.


Review and update translations for es - Spanish

Up for grabs: please review and update/complete the translations in the es.lproj/GMImagePicker.strings file for the language es (Spanish). Use the original labels in Base.lproj as a starting point.
Contributions are highly appreciated!

I take this; this week i'll be submitting a PR.
Thank you for your hard work!


Add conv_pb_leqb

For a stronger cumulSpec0_ind lemma
Not sure if this is the right notion of comparison; I went with "leqb means 'stricter than'"

I would prefer the definition
Definition conv_pb_leqb (pb1 pb2 : conv_pb) : bool
:= match pb1, pb2 with
| Cumul , Conv => false
| _ , _ => true
end.


[Feature]: Allow Subtitles (textTracks) with M3U8 Videos

Description
I think it would be great if we can have Subtitles (textTracks) working with M3U8 videos.
Why it is needed ?
Because many times the M3U8 server does not allow subtitles in the m3u8 file and we need to provide it separately.
Possible implementation
No response
Code sample
No response

Hi @nicolaosm, we invite you to add a comment to our roadmap. If you want to have this ASAP, please contact us for paid development

Thanks @moskalakamil !


Refactor autocomplete widgets

Resolves #2530
Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).

We agreed with @dominik-zeglen that this can be closed due to a lot of conflicts, we'll open a new PR for that.


Can't the server find the LeagueClientUx?

I can find LeagueClientUx in local. but If i distribute project, i can't find it.
Should LeagueClientUx always be found from the front(client)?
We are developing a website
client: react
server: nodejs

When I changed to deployment status locally, I was told that I couldn't find LeagueClientUx in my web browser
Is it true?


Fix Ghost Template domain generation

Ghost Template doesn't generate a domain. I believe it's because it's missing the addition of this PR.

Thank you for the PR!


[QUESTION/HELP]

Hi,
First off thank you so much for all you efforts with this project. It has been extremely helpful to me.
I have a question regarding adding a node to an existing cluster. It appears the new node is copied from the last node that was created when the cluster was created. So all the labels and taints are copied as well. I get an error trying to change the taint on the new node.
Here are the files I used to reproduce it easily (I'm hoping this is a syntax issue)

Create the cluster from the following config file

kind: Simple
apiVersion: k3d.io/v1alpha4
metadata:
  name: tst
servers: 1
agents: 1
kubeAPI:
  hostPort: "6443"
ports:
- port: 80:80
  nodeFilters:
  - loadbalancer
- port: 443:443
  nodeFilters:
  - loadbalancer
options:
  k3d:
    wait: true
    disableLoadbalancer: false
    disableImageVolume: false
    disableRollback: false
  k3s:
    extraArgs:
    - arg: --disable=traefik
      nodeFilters:
      - server:*
    - arg: --disable=local-storage
      nodeFilters:
      - server:*
    - arg: --kubelet-arg=max-pods=400
      nodeFilters:
      - server:*
      - agent:*
    - arg: --node-name=server-0
      nodeFilters:
      - server:0
    - arg: --node-name=node-0
      nodeFilters:
      - agent:0
    - arg: --node-taint=workload=stateful:NoSchedule
      nodeFilters:
      - agent:0
    nodeLabels:
    - label: workload=stateful
      nodeFilters:
      - agent:0
  kubeconfig:
    updateDefaultKubeconfig: true
    switchCurrentContext: true
  runtime: {}
registries:
  create:
    name: docker-io # name of the registry container
    proxy:
      remoteURL: https://registry-1.docker.io # proxy DockerHub
    volumes:
      - /tmp/reg:/var/lib/registry # persist data locally in /tmp/reg
  config: |
    mirrors:
      docker.io:
        endpoint:
          - http://docker-io:5000
    configs:
      docker.io:
        tls:
          insecure_skip_verify: true

Try to add another node and change the  "workload=stateful:NoSchedule" taint to "workload=stateless:NoSchedule"

k3d node create node1 -c tst --k3s-arg '--node-name=node-1' --k3s-arg '--node-taint=workload=stateless:NoSchedule' --k3s-node-label 'workload=stateless' --trace

I get the following error:
[root@server ~]# k3d node create node1 -c tst --k3s-arg '--node-name=node-1' --k3s-arg '--node-taint=workload=stateless:NoSchedule' --k3s-node-label 'workload=stateless' --verbose
DEBU[0000] DOCKER_SOCK=/var/run/docker.sock
DEBU[0000] Runtime Info:
&{Name:docker Endpoint:/var/run/docker.sock Version:20.10.12 OSType:linux OS:CentOS Linux 7 (Core) Arch:x86_64 CgroupVersion:1 CgroupDriver:cgroupfs Filesystem:xfs}
INFO[0000] Adding 1 node(s) to the runtime local cluster 'tst'...
DEBU[0000] no netlabel present on container /docker-io
DEBU[0000] failed to get IP for container /docker-io as we couldn't find the cluster network
DEBU[0000] DOCKER_SOCK=/var/run/docker.sock
INFO[0000] Using the k3d-tools node to gather environment information
INFO[0000] Starting new tools node...
DEBU[0000] DOCKER_SOCK=/var/run/docker.sock
DEBU[0000] DOCKER_SOCK=/var/run/docker.sock
DEBU[0000] DOCKER_SOCK=/var/run/docker.sock
DEBU[0000] Created container k3d-tst-tools (ID: efab975199bb6a94e2b0b02fd41c073f40e602d7dcf29890e8d90bfe85ecf6ea)
DEBU[0000] Node k3d-tst-tools Start Time: 2022-07-22 09:52:27.281282904 -0400 EDT m=+0.397163988
INFO[0000] Starting Node 'k3d-tst-tools'
DEBU[0000] Truncated 2022-07-22 13:52:27.800145799 +0000 UTC to 2022-07-22 13:52:27 +0000 UTC
DEBU[0000] Deleting node k3d-tst-tools ...
DEBU[0001] DOCKER_SOCK=/var/run/docker.sock
INFO[0001] HostIP: using network gateway <IP_ADDRESS> address
DEBU[0001] Adding node k3d-node1-0 to cluster tst based on existing (sanitized) node k3d-tst-agent-0
DEBU[0001] DOCKER_SOCK=/var/run/docker.sock
DEBU[0001] Created container k3d-node1-0 (ID: aeafb755f8c56e13e8d2277d74fbe698d843dc7a7ee141e764c07f9dd60fec99)
DEBU[0001] DOCKER_SOCK=/var/run/docker.sock
DEBU[0001] No fix enabled.
DEBU[0001] Node k3d-node1-0 Start Time: 2022-07-22 09:52:28.282815451 -0400 EDT m=+1.398696616
INFO[0001] Starting Node 'k3d-node1-0'
DEBU[0002] Truncated 2022-07-22 13:52:28.989827593 +0000 UTC to 2022-07-22 13:52:28 +0000 UTC
DEBU[0002] Waiting for node k3d-node1-0 to get ready (Log: 'Successfully registered node')
FATA[0003] failed to add 1 node(s) to the runtime local cluster 'tst': failed to add one or more nodes: failed to run node 'k3d-node1-0': failed to start node 'k3d-node1-0': Node k3d-node1-0 failed to get ready: error waiting for log line `Successfully registered node` from node 'k3d-node1-0': stopped returning log lines

Any help would be appreciated

Sorry I forgot to mention the version:
k3d version v5.4.3
k3s version v1.23.6-k3s1 (default)

Any thoughts on the issue/question I posted ? Any help is appreciated.


[AzureMonitorExporter] include AAD in the logger messages

updated all the transmit-related log messages to include a bool if AAD is enabled.

API change check
API changes are not detected in this pull request.


Links to Pluralsights always return 403s when checked by unbroken

Page url
CI
Problem Description
Our CI checks links in our docs using unbroken. For some reason, working Pluralsight links keep reporting failure.
Failing CI run here: https://github.com/microsoft/react-native-windows-samples/runs/3940931407?check_suite_focus=true
The same issue is tracked here in the RNW repo: https://github.com/microsoft/react-native-windows/issues/8873

pluralsight might be throttling
worst case scenario add an exclusion to the pluralsight url


How to build mimalloc-redirect.dll?

I would like to build mimalloc-redirect.dll myself, but I don't find the related VS2017 project that builds it. mimalloc-override just expect mimalloc-redirect.lib in the bin folder. What would I need to do to build mimalloc-redirect, if possible at all?

Ah, at this time the source of the redirect dll's is not yet public; the plan is do make it open source too though in the future.

I see, thank you anyway. I am looking forward to it ;)

working on it.. :-)

We are waiting for! )

I'm also interested in the source to this. Is there any update on when this may be released? Thanks!

@daanx , any news on making the source of mimalloc-redirect public ?

@daanx Hello? It's been almost two years, what's so difficult about open sourcing mimalloc-redirect?

@Khodyn looking at how the malloc redirection is done in TBB malloc, there might be some internals of C runtime involved. It seems that Intel did some reverse engineering, while mimalloc-redirect may have access to copyrighted sources:
Sample from onetbb/src/tbbmalloc_proxy/proxy.cpp
const char* known_bytecodes[] = {
#if _WIN64
//  "========================================================" - 56 symbols
    "4883EC284885C974",       // release free()
    "4883EC284885C975",       // release _msize()
    "4885C974375348",         // release free() 8.0.50727.42, 10.0
    "E907000000CCCC",         // release _aligned_msize(), _aligned_free() ucrtbase.dll
    "C7442410000000008B",     // release free() ucrtbase.dll 10.0.14393.33
    "E90B000000CCCC",         // release _msize() ucrtbase.dll 10.0.14393.33
    "48895C24085748",         // release _aligned_msize() ucrtbase.dll 10.0.14393.33
    "E903000000CCCC",         // release _aligned_msize() ucrtbase.dll 10.0.16299.522
    "48894C24084883EC28BA",   // debug prologue
    "4C894424184889542410",   // debug _aligned_msize() 10.0
    "48894C24084883EC2848",   // debug _aligned_free 10.0
    "488BD1488D0D#*******E9", // _o_free(), ucrtbase.dll
 #if __TBB_OVERLOAD_OLD_MSVCR
    "48895C2408574883EC3049", // release _aligned_msize 9.0
    "4883EC384885C975",       // release _msize() 9.0
    "4C8BC1488B0DA6E4040033", // an old win64 SDK
 #endif

It might be possible to adapt TBB malloc redirection code for mimalloc.

working on it.. :-)

We are waiting for! ) @daanx
any news on making the source of mimalloc-redirect public ?

still waiting...

me too :/

@daanx Three years have passed, when do we have to wait? :(

waiting

waiting...(2024)

waiting.

For anyone who's here, trying to learn how to redirect/detour their own malloc on Windows, but don't want to use mimalloc yet. here is my guide to you:
use minhook and hook malloc/free functions from c runtime (ucrtbase.dll/ucrtbased.dll), it is really easy to setup. (you can use any similar lib like Microsofts Detour which I haven't used)
Additionally you can see how rmem, an instrumentation tool for MTuner, is using this.
here is my gist on how you can detour your own malloc/free and detour your c runtime functions using minhook:
https://gist.github.com/Erfan-Ahmadi/93369b8b7fbce77b1adf8e244ea33849


Isn't this a built in game feature?

This is available in the game settings. Is the purpose of this plugin just to temporary hide it while in combat and then re-enable it?

Is the purpose of this plugin just to temporary hide it while in combat and then re-enable it?

Yes!
At the time I didn't know about the "Show Activity Adviser" setting, but I decided to publish it anyway since there might be people who enjoy the adviser but don't want it to grief them during combat.


Add src/duct_hierarchy.edn to :resource-paths

Problem
When building an uberjar all duct_hierarchy.edn files are merged to create a full hierarchy. This works fine until you create an uberjar. Your main project's duct_hierarchy.edn is not included it the merged hierarchy.
Solution
Add src/duct_hierarchy.edn to the uberjar's profile resource-paths. This will make sure it is included when generating the uberjar.

This seems like a bug in Leiningen if true. I'll see if I can reproduce this issue.

@weavejester My colleagues and I came across this issue again. I think it's because the default duct template has no :source-paths definition, and possibly defaults to ["src"] ? We added our own: :source-paths ["src/clj" "src/cljs"] and it broke. We fixed it after changing this to: ["src/clj" "src/cljs" "src"].


Post Message - onMessageChannelReady is not getting called

Read this First
If you have a Trusted Web Activity related question, the best place to ask it is on StackOverflow, on the
trusted-web-activity tag, which is also monitored by the team.
Describe the bug

CustomTabsCallback onMessageChannelReady is not getting called even if the validation result is true.
Why do we have !mValidated condition in onNavigationEvent if onRelationshipValidationResult gets triggered only when mSession.requestPostMessageChannel is called from the function.

            @Override
            public void onRelationshipValidationResult(int relation, @NonNull Uri requestedOrigin,
                boolean result, @Nullable Bundle extras) {
                // If this fails:
                // - Have you called warmup?
                // - Have you set up Digital Asset Links correctly?
                // - Double check what browser you're using.
                Log.d(TAG, "==========Relationship result============: " + result);
                mValidated = result;
            }

            // Listens for navigation, requests the postMessage channel when one completes.
            @Override
            public void onNavigationEvent(int navigationEvent, @Nullable Bundle extras) {
                if (navigationEvent != NAVIGATION_FINISHED) {
                    return;
                }

                if (!mValidated) {
                    Log.d(TAG, "Not starting PostMessage as validation didn't succeed.");
                }

                // If this fails:
                // - Have you included PostMessageService in your AndroidManifest.xml?
                boolean result = mSession.requestPostMessageChannel(SOURCE_ORIGIN, TARGET_ORIGIN,
                    new Bundle());
                Log.d(TAG, "Requested Post Message Channel: " + result);

            }

           @Override
            public void onMessageChannelReady(@Nullable Bundle extras) {
                Log.d(TAG, "Message channel ready.");

                int result = mSession.postMessage("First message", null);
                Log.d(TAG, "postMessage returned: " + result);
            }

@onkar-rentomojo have you found a solution? I am experiencing the same issue


Unable to resolve path to module

Unable to resolve path to module from every local file

https://antfu.me/posts/why-reproductions-are-required


display problem

Hello, the latest version has been fixed, please update the code, if there are problems, I will continue to follow up, thank you

@cuiy245 Unfortunately, this problem has not been completely repaired.


Flattening transitive dependencies of a library results in Heisenberg vendoring uncertainty.

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.7
What operating system and processor architecture are you using (go env)?
darwin amd64
What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
In a library (called maglev-appcontainer-go below) I have vendored 3rd party dependencies. In the application consuming that library, there are also vendored 3rd party dependencies. There is now a type conflict between Package A at the path of the vendor directory of the library.
cisco.com/sample/service
cisco.com/sample/service/service.go:31: cannot use "vendor/github.com/emicklei/go-restful".RouteFunction(api.Ping) (type "vendor/github.com/emicklei/go-restful".RouteFunction) as type "maglev-appcontainer-go/vendor/github.com/emicklei/go-restful".RouteFunction in argument to rest.RegisterSomeAPI
cisco.com/sample/service/service.go:35: cannot use "vendor/github.com/emicklei/go-restful".RouteFunction(api.CreateUser) (type "vendor/github.com/emicklei/go-restful".RouteFunction) as type "maglev-appcontainer-go/vendor/github.com/emicklei/go-restful".RouteFunction in argument to rest.RegisterSomeAPI
What did you expect to see?
The govendoring experiment being continued with a proposed solution for this. Instead it seems to be labeled "Mission accomplished", but it still results in unresolvable undecidability between packages.
What did you see instead?
A well-defined problem left hastily unsolved. The solution being semantic versioning of packages (perhaps some decoration in package declaration of the .go files belonging to a package?) and including a resolver at compilation that looks at them in go vendor as the final word in establishing the type. If you're looking at aliasing methods and types in 1.9, please consider this for 1.9 as well. A match between decorations for the package and the package name would unite the type despite different paths. People are expecting breaking changes in 2.0 right? It's not even a breaking change. You could slip it into 2.0 and get knighted.

The general problem of vendering the same package twice and expect them to
interoperate correctly is unsolvable.
I don't think this should be solved by the tool chain. The user should make
sure a given package is only vendored once.

Much vendoring work is currently underway by a number of people. I'm going to close this bug (which is frankly a bit ranty) in favor of more specific, actionable and calm bugs.

Semantic versioning of packages (perhaps some decoration in package declaration of the .go files belonging to a package?) and including a resolver at compilation that looks at them in go vendor as the final word in establishing the type solves this. A match between decorations for the package and the package name would take precedence over the path and resolve the symbol.

@poncele1 What you are describing is in the works. Brad was not dismissing you. He was saying this issue wasn't itself actionable.


WANDIO support required

This issue is observed in the past as mentioned below. I am creating a new one with new logs with latest git clone in Ubuntu 20.04
https://github.com/CAIDA/libbgpstream/issues/191
when running ./configure from cloned repo I am seeing the following error
error: wandio HTTP support required. Ensure you have a working Internet connection and that libcurl is installed before building wandio.
Looks like the link https://broker.bgpstream.caida.org/v1/ is not reachable
from config.log:

configure:14324: gcc -o conftest -g -O2 -pthread   conftest.c -lwandio >&5
configure:14324: $? = 0
configure:14324: ./conftest
configure:14324: $? = 1
configure: program exited with status 1
configure: failed program was:
| /* confdefs.h /
| #define PACKAGE_NAME "bgpstream"
| #define PACKAGE_TARNAME "bgpstream"
| #define PACKAGE_VERSION "1.2.3"
| #define PACKAGE_STRING "bgpstream 1.2.3"
| #define PACKAGE_BUGREPORT<EMAIL_ADDRESS>| #define PACKAGE_URL ""
| #define BGPSTREAM_MAJOR_VERSION 1
| #define BGPSTREAM_MID_VERSION 2
| #define BGPSTREAM_MINOR_VERSION 3
| #define LIBBGPSTREAM_MAJOR_VERSION 2
| #define LIBBGPSTREAM_MID_VERSION 0
| #define LIBBGPSTREAM_MINOR_VERSION 0
| #define STDC_HEADERS 1
| #define HAVE_SYS_TYPES_H 1
| #define HAVE_SYS_STAT_H 1
| #define HAVE_STDLIB_H 1
| #define HAVE_STRING_H 1
| #define HAVE_MEMORY_H 1
| #define HAVE_STRINGS_H 1
| #define HAVE_INTTYPES_H 1
| #define HAVE_STDINT_H 1
| #define HAVE_UNISTD_H 1
| #define HAVE_DLFCN_H 1
| #define LT_OBJDIR ".libs/"
| #define PACKAGE "bgpstream"
| #define VERSION "1.2.3"
| #define HAVE_GETTIMEOFDAY 1
| #define HAVE_MEMSET 1
| #define HAVE_STRDUP 1
| #define HAVE_STRSTR 1
| #define HAVE_STDLIB_H 1
| #define HAVE_MALLOC 1
| #define HAVE_STDLIB_H 1
| #define HAVE_INTTYPES_H 1
| #define HAVE_LIMITS_H 1
| #define HAVE_MATH_H 1
| #define HAVE_STDLIB_H 1
| #define HAVE_STRING_H 1
| #define HAVE_TIME_H 1
| #define HAVE_SYS_TIME_H 1
| #define PTHREAD_YIELD_FUNC pthread_yield
| #define HAVE_PTHREAD_PRIO_INHERIT 1
| #define HAVE_PTHREAD 1
| #define WITH_DATA_INTERFACE_BROKER 1
| #define WITH_DATA_INTERFACE_SINGLEFILE 1
| #define WITH_DATA_INTERFACE_CSVFILE 1
| / end confdefs.h.  */
|
|     #include <wandio.h>
|     int main() {
|       io_t *file = wandio_create("https://broker.bgpstream.caida.org/v1/");
|       return (file == NULL);
|     }
|
configure:14327: error: wandio HTTP support required. Ensure you have a working Internet connection and that libcurl is installed before building wandio.

configure script is auto-generated with following code which seems to be wrong.

cat confdefs.h - <<_ACEOF >conftest.$ac_ext
/* end confdefs.h.  */
#include <wandio.h>
int main() {
  io_t *file = wandio_create("https://broker.bgpstream.caida.org/v1/");
  return (file == NULL);
}

_ACEOF

According to below commit it should have been proper link
https://github.com/CAIDA/libbgpstream/pull/192/commits

If you're sure you have the latest version of the code checked out, you could try re-running ./autogen.sh to rebuild the configure script.

Issue is in the configure.ac file in bgpstream project

The same logic in libbgpstream seams to be fine as shown below

If you're sure you have the latest version of the code checked out, you could try re-running ./autogen.sh to rebuild the configure script.

I I did try a complete clean build as mentioned below and still see the same issue

git clone https://github.com/CAIDA/bgpstream.git
cd bgpstream/
./autogen.sh
./configure

But you've opened an issue against the libbgpstream project?
https://github.com/CAIDA/bgpstream/ is deprecated in favor of libbgpstream and basically unmaintained at this point.
If you want to submit a PR to that repo then I'm sure we can merge it, but I'd strongly encourage you to upgrade to BGPStream v2.

Thanks for the reply @alistairking .
My bad.
Where can i get the access to BGPStreamV2 ? is that a separate repo ?

That's this repo :)
You can follow the instructions here: https://bgpstream.caida.org/docs/install/bgpstream#ubuntu

No wonder I faced many issues compiling BGPSTREAM in Ubuntu 22.04 and I have reverted to Ubuntu 20.04 to try the same.
I guess README inside BGPSTREAM should be updated with a NOTE recommending v2.0. Let me try the instructions shared.

I understand this is not the issue of bgpstream but one minor inconvenience is WANDIO path mentioned in installation steps is unreachable
https://research.wand.net.nz/software/libwandio.php

It should probably be https://github.com/LibtraceTeam/wandio

Agree.
If you are planning to update the install script then below order of commands should work

git clone https://github.com/LibtraceTeam/wandio
cd wandio/
./bootstrap.sh
./configure
make
sudo make install
sudo ldconfig

Feel free to submit a PR for this

But this change is needed in the below website right ? Where can I submit pull request for that ?
https://bgpstream.caida.org/docs/install/bgpstream#ubuntu

Oh good point. It should be in here: https://github.com/caida/bgpstream-web

Thanks for the link. Let me search for the exact page.
Also the README file of libbgpstream points to downloading the code from BGPSTREAM repo which is no more maintained. Dont you think it should be corrected ?
https://github.com/CAIDA/libbgpstream
Ideally current name of the libbgpstream repo should be BGPStreamv2.0 instead of libbgpstream. As per my understanding libbgpstream is just a library where as BGPSTREAM is the framework containing libbgpstream library, applications like pybgpstream, bgpreader and so on.

Sure. Again, PR's are welcome.

As per my understanding libbgpstream is just a library where as BGPSTREAM is the framework containing libbgpstream library

This is exactly why this repo is called libbgpstream and not bgpstream. This repo only contains the bgpstream library (and bgpreader).
pybgpstream is here: https://github.com/caida/pybgpstream

Closing the issue as it is resolved in Ubuntu 20.04 using bgpstream v2.0


Configure dev environment to work with just devDependencies

Feature description
Based on the changes introduced in #1016 we wanna ensure that installing only devDependencies and omitting optionalDependencies with npm ci --omit=optional we can ensure that the application will work running a dev environment with npm run start:dev.

Closing this issue since it was addressed in #1004


Crash when using shaders, still happening.

I see that this is not yet been fixed, here is the report again, in hopes that it gets fixed if the bug is reported enough.
crash-2022-03-01_03.14.25-client.txt

Spamming the same bug reports will not make us fix them any faster.
Duplicate of #8


fix: allowing domain to be added including a scheme

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

Related Issue # (issue)
Closes # (issue)
Fixes # (issue)

Remove if not applicable

Screenshots

Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help
[ ] Separate ticket for tests # (issue/pr)

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Added to documentation?

[ ] 📜 README.md
[ ] 🙅 no documentation needed

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Aye. I had to think a lot to come up with a proper way of making sure localhost and port are still allowed but https:// is not. 🤣 since there is no negative validation. So I can't say DONT ALLOW if it starts with :// or something.

I just wish I could customize the error message
ka user_repository.yaml
The Repository "new-repository-3" is invalid: spec.domain: Invalid value: "https://github.com": spec.domain in body should match '^\w+(\.|:[0-9]).*$'

But I could only do that with XValidate which is weird. :D


Implement first-class functions

Problem
Noir currently has generics, but no way to abstract over operations to perform on these generics. With the absence of traits, another method to provide functionality is by manually passing in functions. This requires higher-order functions in noir, or in other words, for functions to accept other functions as parameters and return them as return values.
Solution
Add higher-order functions including function types and lambdas to Noir.
Function types:
fn()
fn(Field) -> Field
fn(i32, T) -> (T, i32)

// function takes a function and returns a function
fn(fn(T) -> U) -> fn(T) -> U

Lambdas:
|| {}
|x| x
|x, t| (x, t)
|f: fn(T) -> U| f

@jfecher can this be closed?

Yes, implemented in #468


[AutoRelease] t2-selfhelp-2023-11-17-52487(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/4770
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=6080

issue link:https://github.com/Azure/sdk-release-request/issues/4770


zq "rare surprise" when Zed query clashes with filename

At the time this issue is being filed, Zed is at commit 08c04de.
As divulged in the zq usage docs (emphasis added):

To determine whether the first argument is a query or an input, zq checks the local file system for the existence of a file by that name or whether the name is an URL. If no such file or URL exists, it attempts to parse the text as a Zed program. If both checks fail, then an error is reported and zq exits. This heuristic is convenient but can result in a rare surprise when a simple Zed query (like a keyword search) happens to correspond with a file of the same name in the local directory.

As described in #5052, at one time we envisioned a -query flag that could help here. It was described at one time as:

To avoid this [surprise], you can provide the query with the -query flag, which specifies the Zed program to run and forces all arguments to be interpreted as inputs.

Of course, there may be other ways to address this.
In any case, this issue can remain open as a reminder that we're currently accepting our users' exposure to this "rare surprise", so comments can be added here if users bump into it and help determine if/when we might want to revisit the topic.

We ended up addressing this in the tooling via #5346 when the -c option was added to precede query text, similar to the proposed "-query flag" described above. The changes in #5457 then removed mention of the "rare surprise" from the docs.


Support python3

Not sure what this will take, but it will probably be a nightmare.  To support all of sparts means twisted and tornado need to be py3 ready.

Tornado works with Py3.
There's also Py3 Thrift.
I can see an argument for dropping Twisted support.

Twisted is optional.  Also, if I recall correctly, twisted-13+ works with Python 3.3 and above.

@ryanrhee - does Apache thrift support Python 3 also?  Or is it just fbthrift?

oh it may just be fbthrift - not sure.


Aghost Now Can Drop Things Wherever They Want

About the PR
Aghosts can now drop things wherever they please
Why / Balance
Closes #19225, Admin QOL
Technical details
Adds a tag that can be added that ignores the range and obstruction check when something leaves a players hands
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

https://github.com/space-wizards/space-station-14/assets/66805063/f09ee58d-f219-4c68-9453-a1a62f741879
Breaking changes

Changelog

:cl:

fix: Fixed AGhosts not being able to drop items past the default range.

label: Holy Shit
(this has made me very angry across many hours of testing things in-game localhost)

Thanks for the fix


.\KubeCluster.ps1 -ConfigFile .\Kubeclustervxlan.json -install

When I execute this command, I am prompted with the following error
What Should I do ?

Please raise questions on kubernetes-users slack channel or on stackoverflow. This doesn't look like a documentation issue.
/close


Close the block store if we fail to build it

In the case that we hit an error while adding files, this could leak
file descriptors; We'd close any flushed blocks, but not the ones
open for writing.
r? @kitchen

seems reasonable!
:shipit:


SSH-Agent not running

Hi there!
As far as i can see, you are not really adding the custom ssh keys to the ssh agent or even starting the ssh agent properly. Could you just quickly point me into the right direction? I can see that you are symlinking ssh keys, but not where you load them exactly.
Best,
Jeremy

Hi @jeremyzahner,
Toran proxy cron need ssh keys to clone privates repositories. Cron is run by www-data user (https://github.com/cedvan/docker-toran-proxy/blob/master/bin/toran-proxy-cron#L10). www-data home directory is /var/www. So I add ssh keys to /var/www/.ssh/ (https://github.com/cedvan/docker-toran-proxy/blob/master/scripts/install/ssh.sh#L13).

All well. I got it working.


Scale in unregistered htex blocks

This PR will make the htex scale-in code consider blocks which have been launched through the provider mechanism but have not registered to the interchange. This is especially relevant at shutdown time, when previous blocks still waiting in the queue (and so, not registered) would be abandoned by parsl shutdown.
[first commit on this PR is a test that should break; when that happens, I'll push the rest; this is to get a demonstrated failure of test in CI]
Changed Behaviour
More blocks scaling in. Tidier shutdown.
Fixes
Fixes #2627
Type of change

Bug fix

Since I wrote this code a few weeks ago, there's been a change to the high throughput executor status method to introduce a "MISSING" state, and I've also gone over the general status code with @yadudoc a bit.
Most importantly, I'd like to check that this code isn't actually calling the provider status calls (eg. invoking sbatch) unnecessarily: the current code is quite unclear as to when asking for status returns some in-memory cache of blocks (and block status) vs calling out to the LRM.
There might also be opportunities for aligning this code better with the code that handles MISSING status.

This draft PR fixes the issue described here: #3235.

@yadudoc - I don't think it does? this PR doesn't fix any cache-based status update delays? I thought we tested a different (no-PR-number) patch?


Enigma or reddle

Shall be the My next graphical animation's série!!!!

Oh!!! It's so beutifull!!


Nixpacks NodeJS deployment failing due to unsupported Python version

Problem Description

We received an email from Railway with subject [ACTION REQUIRED] Upgrade your databases, where they asked us to migrate to v2 of their plugins & database using their migration tool.
I followed their migration guide & both Postgres & Redis plugins got migrated successfully.

Migrated Services

This migration re-trigered the build & it failed due to following error. You can see error logs here.

Error

#8 28.72 Updating Autotools / GNU config script to a newer upstream version: ./deps/cares/config.guess

#8 28.73 configuring

#8 28.82 fixing libtool script ./deps/cares/ltmain.sh

#8 28.84 ./deps/cares/configure

#8 28.94 configure flags: --prefix=/nix/store/mqrbjaaz590rl2fg527j7msg51igr6c2-nodejs-14.21.3 --shared-libuv --shared-libuv-libpath=/nix/store/hba5ja1cri7jd7gvw77dqxsvsccijhz8-libuv-1.46.0/lib --shared-openssl --shared-openssl-libpath=/nix/store/2fhqpp0hnv80hf66y1s72am8dahl1bq1-openssl-1.1.1w/lib --shared-zlib --shared-zlib-libpath=/nix/store/s5gzrzha72q79v92wqq61x9ir8xiwbxk-zlib-1.3/lib --with-intl=system-icu --without-dtrace

#8 28.97 Node.js configure: Found Python 3.11.5...

#8 28.97 Please use python3.10 or python3.9 or python3.8 or python3.7 or python3.6 or python3.5 or python2.7.

#8 29.50 error: builder for '/nix/store/hh8n2lf9hxvjrp2rgp6542acrcyg153i-nodejs-14.21.3.drv' failed with exit code 1

#8 29.53 error: 1 dependencies of derivation '/nix/store/hr6azazr0rnipm5vda93z8fql7d6q20a-bf744fe90419885eefced41b3e5ae442d732712d-env.drv' failed to build

#8 ERROR: process "/bin/bash -ol pipefail -c nix-env -if .nixpacks/nixpkgs-bf744fe90419885eefced41b3e5ae442d732712d.nix && nix-collect-garbage -d" did not complete successfully: exit code: 100

Solution

I tried to set Python version in environment variable NIXPACKS_PYTHON_VERSION with 3.10 value but it again failed the deployment with same error.
Feeling helpless, I posted a new question in Railway Forum - https://community.railway.app/discuss/thread/nixpacks-node-js-deployment-failing-due-t-cd32bc04
I searched for similar issues in the forum & found similar issue - https://community.railway.app/discuss/thread/deployment-failure-due-to-python-version-c2ea27b3#p-7
It said to either upgrade Node version or downgrade Nixpacks version via railway.toml file. As our CMS didn't support Node >14 versions, I downgraded nixpacksVersion to 1.17.0 in commit https://github.com/scriptified/scriptified-cms/commit/c741c8f2e579cf09f3029fb508c24e866ed00c7f
It seems to have solved the build issue but latest deployment failed with Build timed out error, see logs here.

As per this comment, we wouldn't need to add/update railway.toml file if https://github.com/railwayapp/nixpacks/pull/1012 was merged.

Phew, after many tries - the deployment finally passed 🚀
active deployment - https://railway.app/project/41728679-1c01-4f64-9e27-b142c7231485/service/a841e3a3-3230-49dd-a6f3-572bc8a5fc94?id=b654bb18-a5a0-4d5f-ad6e-e0e520ebb06f
cc: @prateek3255 go through the issue thread once, I've completed the migration activity & made the latest deployment active 🎉