Empty entry in XML confuses the generated parser (tamplate bug)

Hi,
I discoverer following bug in generated parser:
I modified "tests/xmls/MyPantry.xsd" to add "Comment" entry, and after that following change in "tests/xmls/MyPantry.xml" breaks generated parser for ObjC:
<?xml version="1.0" encoding="UTF-8"?>
<p:Pantry
	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
	xmlns:p="http://www.example.org/Pantry"
	xsi:schemaLocation="http://www.example.org/Pantry Pantry.xsd">
	<p:Shelf>
		<p:Food>
			<p:Name>Apple</p:Name>
			<p:FoodGroup>fruit</p:FoodGroup>
			<p:Comment></p:Comment><!-- here parser is confused -->
			<p:Cost>$0.50</p:Cost>
		</p:Food>
	</p:Shelf>
	<p:Shelf>
		<p:Food>
			<p:Name>Chocolate</p:Name>
			<p:FoodGroup>sweetFatOil</p:FoodGroup>
			<p:Cost>$0.50</p:Cost>
		</p:Food>
	</p:Shelf>
</p:Pantry>

Modification done to this xsd and xml are available at my branch: https://github.com/MarekR22/xsd2cocoa/tree/test/emptyEntryBug
I will try to fix it my self, but this template file is not very easy to understand.

Fix has been provided.

merged


Fetched Results Controller improvements

Addresses #56 and #57.

Looks really good. I'm happy with it other than the final declaration.

Fixed


GUI: Test Virtual Media page

Test Virtual Media page in GUI
Dev Story - https://github.com/ibm-openbmc/dev/issues/176

[ ] Download ISO image and start virtual media session from BMC web GUI
[ ] Mount ISO and boot host, and view the Install at petitboot
[ ] Mount ISO and boot to Host
[ ] Install using petitboot after mount via virtual media
[ ] Mount and confirm at petitboot and then unmount and confirm it auto unmounted
[ ] Mount and start install and then un mount when install is in progress

Is this working for IMG image? I mean HDD image redirection will success?
I can able to redirect HDD (.img or .ima) image, but can not mount properly. Getting some  error like "can't read superblock".

@edineshkumar What happens if you try to mount your image using a loop device on the host?  This would isolate issues with your image from issues like block simze or unimplemented commands.

@mdmillerii  I have raised issue on virtual media redirection, kindly help me out to resolving the same.
https://github.com/openbmc/openbmc/issues/3669


Allow creation of vhosts & exchanges

These are order specific to create.

Vhosts
Users (depends on vhosts)
Exchanges (depends on users)

@tizz98 thanks for your contribution :+1:


Filament incompatible with sanctioned glb models

As mentioned on khronos site, the kenney models "tower defense kit" fail to load due to filament error, e.g. for towerRound_sampleF.glb:
 E/Filament: Unable to parse glb file.

with throwable CompletionException:
java.lang.IllegalStateException: Failed to load gltf

I can reproduce the issue. Would you mind filing an issue here please: https://github.com/google/filament/issues? The root cause seems to be that cgltf (the glTF parser we use) itself is having issues. We've made contributions to cgltf before so we can fix this ourselves.
CC @prideout

This asset is non-conformant because the scene is defined by a non-root node.  According to the spec:
All nodes listed in scene.nodes array must be root nodes

cgltf checks for this violation and bails. (it checks for this incgltf_fixup_pointers)

Done: https://github.com/google/filament/issues/2443


[FEATURE] Speaker Dashboard

Description
We need to make the Speaker-Dashboard.

[ ] Speaker-Dashboard

Screenshots

Additional information
No response

Assign it to me @AliReza1083

How is it going?

Assigning this issue to Ali as i am busy and not able to complete it on time sorry for that.


Normalize path in compiler before sending it to twig->load()

This PR addresses the issue https://github.com/rcrowe/TwigBridge/issues/438.
While compiling the twig templates, the view cache generated had the full path as the template name instead of the actual template name. This was a problem specially when using view composers. As we define view composers with template names, they were not being triggered due to the wrong name in cache files.
This fix adds code to normalize the path before sending it to twig->load() method, which expects the name of the template and not the path. Normalizing the path would convert it into the template name.
I'm not sure if this is the proper solution for this so please suggest alternatives if required.
Thanks!

This would not work in all the cases, for example with themes. I think we need a way to pass in the template name instead of normalizing the path but I'm not exactly sure how to solve this problem.


I suggest changing the source also to my repo where I keep the updated list from iso.org

I prepared a repo where you can use the CLI to fetch the newest ISO 3166 list or use the countries.json file. https://github.com/detrin/download-iso3166-list

I guess you want to update it manually. Would you at least comment on it? @alcohol
ISO 3166-1 updates from time to time. I suggest having at least a github action that would check your data against the iso.org list
See https://github.com/thephpleague/iso3166/issues/7 which is not 100% related, but the idea is similar


VNET enabled runtime IP Restrictions

Is the data factory service tag not supported when the managed VNET setting is enabled?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 33fc045d-cec5-9c08-0eee-8af574b9e76f
Version Independent ID: 94e41877-46e8-f5fa-f1f7-14714db3a447
Content: Azure Integration Runtime IP addresses - Azure Data Factory
Content Source: articles/data-factory/azure-integration-runtime-ip-addresses.md
Service: data-factory
Sub-service: integration-runtime
GitHub Login: @lrtoyou1223
Microsoft Alias: lle

@johnayoub we are looking in to this feedback. We will share updates soon.

@johnayoub  Data flows and Azure Integration Runtime which enable Managed Virtual Network don't support the use of fixed IP ranges.
Please Note, GitHub forum is dedicated for docs related issues. For any technical queries or clarifications, we encourage to utilize Microsoft Q & A platform. Kindly raise your query on Microsoft Q&A Platform.


Modify src/scene/scene.js to verify if this.defaultMaterial is valid before destroying

modify src/scene/scene.js to verify if this.defaultMaterial is valid before destroying (to solve the problem that deleting the last app in multiple apps will cause an exception)
Fixes #
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

The only place that this.defaultMaterial is set to null is in that destroy function which implies that Scene.destroy is being called twice. 🤔

I agree with @yaustar - the null check should not be needed here, as it simply indicates the issue is elsewhere.

perhaps it'd be good to put some marker on that default material when it gets constructed, for example
this.defaultMaterial.hackId = Math.random();
console.log("created: " + this.defaultMaterial.hackId);
`;

and log it out as well from material's destroy function when it gets destroyed.
This could show us where it's getting destroyed from multiple times.

This is no longer applicable due to refactoring - the scene no longer owns nor destroys the defaultMaterial


OpenAI forwarding API configuration

OpenAI forwarding API configuration error, use the official website tutorial to configure
(
setx OPENAI_API_KEY "sk-xxx"
setx OPENAI_API_BASE "https://api.wlai.vip"
aider --model openai/gpt-4o
)

There is no output

Thanks for trying aider and filing this issue.
This is most likely a problem with your api server. It's not responding, or not responding in an OpenAI compatible way.

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


feat(Card): pass props to onClick

Fixes #949

Current coverage is 100% (diff: 100%)

No coverage report found for master at f450f67.

Powered by Codecov. Last update f450f67...aca603f

@levithomason - updated 👍


pycubedmini library updates

Changes:

Initializing sun sensors and h bridges in separate try-except statements. This allows us to pinpoint what exactly is failing
Added burnwire as an entry in the hardware dictionary (init function in Satellite class)
General documentation and style (local variable names, spacing, comments). I didn’t change global variable names because I’m not sure where else these are accessed yet. I think it would be best to refactor once we have all the code ready, and this isn’t a priority.
Reinit function:

Line 216: dev.lower() -> dev.strip().lower(). This change strips the string dev of whitespace and then makes it all lowercase, which helps us avoid small errors (like extra spaces in commands)
Line 220: self.radio2… -> self.radio… This is because we only have one radio on Pycubedmini (I suspect that it says radio2 because it wasn’t updated from PyCubed)

Burn function: Current version used variable names that are never initialized in pycubedmini (but do exist in PyCubed). Cleaned up this function to support the current variables and hardware.

Necessary Updates:

General: Add the camera board as an entry in the hardware dictionary (init function)
storage_stats function: Return the storage statistics for the Nicla board as well, as it has its own file system

Possible Updates:

acceleration, magnetic, gyro, temperature functions. 2 options:

return the reading when this function is called (this is currently the case)
store readings and return readings with matching timestamps

Burn function: We currently have to pass a string or an array of strings as burn_num: is there anything stopping us from passing an integer?

@yashikabatra15 please run this on the mainboard before merging - we definitely want some level of testing for this.

I've reviewed / implemented all changes and linted the file! Please let me know if there's anything else y'all see before we merge.


Create .replit config schema

It's strange but autocomplete doesn't work until I rename my test .replit file to .replit.toml.

I don't know why nothing work for .replit but everything is fine for .replit.toml. Here is my suggestion for .Repl.it maintainers - allow using .replit.toml.

I found the solution to not to add .toml extension: change language mode to TOML. But I am not sure whether it's the best solution. @tamasfe

Thanks for the PR!
I'd probably leave the "additionalProperties": false parts out as it will cause false errors if the schema is not updated or there are any unknown/undocumented keys. Just a thought, it's not necessary and I don't know a good solution for this. Anyways, LGTM.

I found the solution to not to add .toml extension: change language mode to TOML.

Yes that's the only way right now, maybe we should auto-detect TOML files in the extension, but that requires the LSP to be running all the time.


getting odd LLM results, advice for newbies?

Hi,
Great job on the repo, I am trying the Collab notebook code like this below which pops up a chat interface:
from ctransformers import AutoModelForCausalLM
from IPython.display import display, clear_output
import ipywidgets as widgets

# Output widget for AI response
output = widgets.Output()
display(output)

llm = AutoModelForCausalLM.from_pretrained("TheBloke/LLaMa-7B-GGML",
                                           temperature=0,
                                           gpu_layers=50)

while True:
    user_input = input("Human: ")
    if output:
      print(f"AI: {llm(user_input)})")

I asked the LLM 3 questions, denoted by Human: below and the LLM response is AI:. One question to make a hello world Flask app, another question to make an LLM rest API server, and then 200 words on what sick building syndrome is. It seemed to do well in the sick building response but notice in the code creation a Fetching files, just curious what this is? Just curious about any tips to get any better results! Great job on the GPU support!

This is an exercise in learning how to use the Python programming language in the context of the Flask framework for building web applications.
The following are the steps you will need to take:

Install the latest version of the Python development environment (the one that comes with your operating system).
Create a directory, and then cd into it.
Run the following command in the terminal:
python -m flask runserver 8000
This will create an application at http://localhost:8000/.
Open up a web browser (e.g., Chrome) and go to http://localhost:8000/
You should see something like the following:
Now, type in your name, and press enter.
Press Ctrl + D to exit.
Run the following command in the terminal:
This will create an application at http://localhost:8000/
Open up a web browser (e.g., Chrome) and)
Human: write a flask app that is a REST API server for an LLM. The app will take in text data on a POST, process the data through the LLM, and return the LLM's response as JSON
AI: .
The project should be 100% written in Python using Flask. I can provide the source code of the LLM if needed.
Hi, I am an experienced python developer with 2 years experience in python development. I have already done many projects like this. Please contact me for more details.)
Human: in 200 words write what sick building syndrome is
AI:
Sick Building Syndrome is a condition that occurs in the workplace. It is caused by the air quality, temperature, humidity, lighting, ventilation, and other factors. The symptoms are headaches, dizziness, fatigue, nausea, cough, sore throat, dry mouth, watery eyes, nosebleeds, nasal congestion, sinusitis, asthma, breathlessness, chest tightness, phlegm, and other respiratory problems.
The cause of Sick Building Syndrome is the air quality in the workplace. The symptoms are headaches, dizziness, fatigue, nausea, cough, sore throat, dry mouth, watery eyes, nosebleeds, nasal congestion, sinusitis, asthma, breathlessness, chest tightness, phlegm, and other respiratory problems.
The cause of Sick Building Syndrome is the air quality in the workplace. The symptoms are headaches, dizziness, fatigue, nausea, cough, sore throat, dry mouth, watery eyes, noseble)

may be you can look into my repo
https://github.com/talhaanwarch/streamlit-llama/blob/main/app.py

I didn't realize that the Bloke had so many models. Would you have any
advice for more advanced model logic and reasoning type of questions I am
asking the LLM?
https://huggingface.co/TheBloke
On H20 GPT, my questions are answered just fine on full blown 70b Llama and
others but the compressed models I definitely have some issues.
This is really cool IMHO for testing a lot of model results at the same
time.
https://gpt.h2o.ai/
Thanks!
On Mon, Aug 7, 2023, 3:18 PM Ravindra Marella @.***>
wrote:

Hi, the Fetching files message is from
AutoModelForCausalLM.from_pretrained() call. It fetches model files from
hugging face repo and caches it locally, so when it is called it checks if
it is using the latest version of model.
When using in a notebook, you should keep model loading code and text
generation code in separate cells as loading model is expensive. See
example
https://colab.research.google.com/drive/1Ihn7iPCYiqlTotpkqa1tOhUIpJBrJ1Tp
.
For getting better results, you can use larger models like 13B or other
quantization formats like q4_0. By default it downloads the smallest model
file (q2_K format) which runs fast but has lower quality. For example, in
TheBloke/LLaMa-7B-GGML repo it uses the llama-7b.ggmlv3.q2_K.bin by
default which can be changed using AutoModelForCausalLM.from_pretrained("TheBloke/LLaMa-7B-GGML",
model_file="llama-7b.ggmlv3.q4_0.bin", ...). Also try out the latest
llama 2 models.
—
Reply to this email directly, view it on GitHub
https://github.com/marella/ctransformers/issues/74#issuecomment-1668514997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHC4BHOHMRAR77TCTT7ZPNTXUFEPVANCNFSM6AAAAAA3FL2KTY
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Paket.Restore.targets: use fixed version ([1.0])

fixes https://github.com/fsprojects/Paket/issues/2653
When the version is specified without the braces, then it will choose any version equal or higher, with preference for the lowest.

I don't think the previous behavior would have caused any issues, but safe is safe ...
The only case I can think of is if the nuget cache for some reason didn't contain the exact version (paket restore should have put it there), but a higher one. Then nuget would have silently chosen the higher one.

Aaand github is down again?

thx!

damn, sorry, can you please revert this?
This changes the behavior of dotnet pack.
Before it would generate a constraint >=, now it generates a constraint ==.
I don't think it is possible to tell nuget to use (==) for resolution, but use (>=) for the nuspec.
Since paket restore puts the exact version into the cache anyway, I don't think there is any realistic chance for something to go wrong.


Navigation Router

I'm getting in touch to thank you for your kind comment about my Navigation router and to suggest a way to shorten your navigation.js code. You can merge the code in your separate navigated functions into a single onNavigate handler.
The states in the Navigation router configuration accept custom properties. You can add your component and display properties onto them:
var stateNavigator = new StateNavigator([
    {key: 'todos', route: 'todos', component: <Todos store={todoStore} />, display: 'block' },
    {key: 'welcome', route: '', component: <Welcome />, display: 'block' }
]);

Then you can add an onNavigate handler that extracts the custom properties from the current state and calls your routeAdd function. The handler's called every time a state is navigated to.
stateNavigator.onNavigate((oldState, state, data) => {
    RouteActions.routeAdd({
        id: state.key,
        component: () => state.component,
        display: state.display
    });
});

hello
I did the modifications
Thanks very much

Modification done


Blazor API Review: Components programming model

PR for #11610
This is the API review changes and feedback for the components programming surface.
Draft while I get through all of the changes and discussion.

@SteveSandersonMS @pranavkm @javiercn - making this non-draft. There's some straightfoward things here that we have consensus on already.


powershell 6.2 of powerbi need update

1 ps version （pie chart）:
add 6.2,6.3 preview
2 Monthly usage by version,monthly usage by OS（Histogram）.I think it is better from 1st to 31st of each month.
3More your idea, welcome to discuss

@chuanjiao10 There's already a pie chart for the different versions being used
There is also monthly usage by OS and version already in bar chart form.

yes i see.6.2 too less, causing me not to see it before.
but "monthly usage" right?
look:

Yeah, need to fix that rendering problem.  AppInsights by default keeps 3 months of data, so we only show 3 months worth.  Those columns each represent a month.  I'll see about fixing this.

@chuanjiao10 that should be fixed now

thanks for fix !
is it only rendering problem？
the date is from 1 to 31，not 24 this month to 10 next month？
if “24 this month to 10 next month”，then Inaccurate data.

@chuanjiao10 it was only a rendering problem, the data is inclusive for a specific month.

good，thanks again~


Slow operation in couch_proc_manager

We've noticed a couple couch_proc_manager backups when deploying the latest version. Inspecting code I ended up stumbling over the new config code. Its theoretically quite a bit slower than the old code since its doing OS env checking but its also doing things quite inefficiently beyond that.
https://github.com/apache/couchdb/blob/master/src/couch/src/couch_proc_manager.erl#L375-L383
The three big things to notice here are that we're joining the variable names inside the loop which is a bit wasteful. Given the size of the strings I'm not sure how bad that is though. I'd wager it likely depends on how many environment variables there are to know whether its making a difference. The tokens call with a pattern match also seems like an expensive thing to be doing. However the killer bit is probably that we're looping over os:getenv() every time we create a new process which is likely what's caused the backups.
I'd propose two changes. The first simpler of the two is to just create our expected environment variable and use os:getenv/1.
Something like:
get_env_for_spec(Spec, Target) ->
    SpecStr = Spec ++ Target,
    os:getenv(SpecStr, undefined).

We're still going to be hitting os:getenv/1 every time we create a new process though so I'd also recommend caching succcessful finds in an ets table or similar. That would allow us to have similar speed to before the patch while still maintaining the OS env config approach.

Also a bit more background I realize I didn't share. We had a couple nodes (not even entire clusters) that experienced a backup on the couch_proc_manager process to the tune of 6M+ message in the mailbox. What likely happened was that when the node was booted there was a sudden massive surge of view indexing triggered. The slightly slower couch_proc_manager was then not able to keep up processing the new process requests which leads to a positive feedback cycle. Lots of processes trying to use get_proc piling into the message queue, and the larger that message queue gets the slower couch_proc_manager gets because its garbage collecting a lot due to that new loop over the os:getenv() output with the string creations and so on.
Its fairly similar to how couch_server can get stuck with the same issues. The big mailbox causes garbage collection to dominate the CPU time of that process preventing it from being able to catch up and clear out its mailbox.

I wrote that super quickly. Don't @ me.

@janl I considered something like that but for some reason discounted the idea thinking we didn't know all of the possible languages at boot time. Though you're quire right that we can just look for the prefix on the environment variables and then take everything after as the language name after lower casing the suffix. I like that approach quite a bit better.

@davisp 👌

@iilyak self-assigned on IRC :)


[fix][broker] Optimize /metrics, fix unbounded request queue issue and fix race conditions in metricsBufferResponse mode

Fixes #22477
Motivation
There are multiple problems in the /metrics endpoint:

requests are handled one-by-one and added into a queue

when requests have timed out, they get processed regardless. time outs are detected in the later phases of the processing. Instead, timed out requests should be short-circuited in the beginning of the processing.

processing is single threaded and therefore the throughput is low
the metricsBufferResponse mode added in #14453 improves things, but it contains some race conditions where a buffer can get released when it is in use.

Modifications

fix the request timeout issue
enable multithreading. currently fixed to 4 threads since higher concurrency requires more memory and leads to OOM issues.

by default, combine concurrent requests to reuse the same result. this improves throughput even when metricsBufferResponse mode isn't enabled

remove the previous metricsBufferResponse solution since the TimeWindow and WindowWrap classes arean't needed at all. The concurrent request combining solution can also be used for metricsBufferResponse solution.
optimize some details of stats generation which were allocating a lot of objects.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Performance test results with the changes.
Reproducing the results:
gh pr checkout 22494
mvn -Pcore-modules,-main -T 1C clean install -DskipTests -Dspotbugs.skip=true
rm -rf data
PULSAR_MEM="-Xms2g -Xmx4g -XX:MaxDirectMemorySize=6g" PULSAR_GC="-XX:+UseG1GC -XX:+PerfDisableSharedMem -XX:+AlwaysPreTouch" PULSAR_EXTRA_OPTS="-Djute.maxbuffer=20000000" PULSAR_STANDALONE_USE_ZOOKEEPER=1 bin/pulsar standalone -nss -nfw 2>&1 | tee standalone.log

git clone https://github.com/lhotari/pulsar-playground
cd pulsar-playground
./gradlew shadowJar
# create the topics
java -cp build/libs/pulsar-playground-all.jar com.github.lhotari.pulsar.playground.TestScenarioCreateLongNamedTopics

git clone https://github.com/lhotari/pulsar-playground
cd pulsar-playground/experiments/metrics-performance

❯ k6 run load_test.js

          /\      |‾‾| /‾‾/   /‾‾/
     /\  /  \     |  |/  /   /  /
    /  \/    \    |     (   /   ‾‾\
   /          \   |  |\  \ |  (‾)  |
  / __________ \  |__| \__\ \_____/ .io

     execution: local
        script: load_test.js
        output: -

     scenarios: (100.00%) 1 scenario, 100 max VUs, 10m30s max duration (incl. graceful stop):
              * default: 10000 iterations shared among 100 VUs (maxDuration: 10m0s, gracefulStop: 30s)

     data_received..................: 3.8 TB 8.1 GB/s
     data_sent......................: 880 kB 1.9 kB/s
     http_req_blocked...............: avg=35.04µs  min=1µs      med=3µs     max=3.92ms   p(90)=6µs      p(95)=8µs
     http_req_connecting............: avg=25.17µs  min=0s       med=0s      max=3.02ms   p(90)=0s       p(95)=0s
     http_req_duration..............: avg=4.7s     min=457.09ms med=4.68s   max=9.71s    p(90)=8.15s    p(95)=8.58s
       { expected_response:true }...: avg=4.7s     min=457.09ms med=4.68s   max=9.71s    p(90)=8.15s    p(95)=8.58s
     http_req_failed................: 0.00%  ✓ 0         ✗ 10000
     http_req_receiving.............: avg=185.35ms min=86.9ms   med=183.9ms max=469.76ms p(90)=214.08ms p(95)=224.08ms
     http_req_sending...............: avg=58.44µs  min=3µs      med=15µs    max=41.26ms  p(90)=28µs     p(95)=38µs
     http_req_tls_handshaking.......: avg=0s       min=0s       med=0s      max=0s       p(90)=0s       p(95)=0s
     http_req_waiting...............: avg=4.51s    min=302.02ms med=4.5s    max=9.56s    p(90)=7.96s    p(95)=8.4s
     http_reqs......................: 10000  21.179615/s
     iteration_duration.............: avg=4.7s     min=457.16ms med=4.69s   max=9.71s    p(90)=8.15s    p(95)=8.58s
     iterations.....................: 10000  21.179615/s
     vus............................: 5      min=5       max=100
     vus_max........................: 100    min=100     max=100

running (07m52.2s), 000/100 VUs, 10000 complete and 0 interrupted iterations
default ✓ [======================================] 100 VUs  07m52.2s/10m0s  10000/10000 shared iters

One interesting detail is that in the load test, the system couldn't keep up with the load without making the optimization to direct buffer allocation in commit eb53342.

Most recent test run without turning on the caching:
❯ k6 run load_test.js

          /\      |‾‾| /‾‾/   /‾‾/
     /\  /  \     |  |/  /   /  /
    /  \/    \    |     (   /   ‾‾\
   /          \   |  |\  \ |  (‾)  |
  / __________ \  |__| \__\ \_____/ .io

     execution: local
        script: load_test.js
        output: -

     scenarios: (100.00%) 1 scenario, 100 max VUs, 10m30s max duration (incl. graceful stop):
              * default: 10000 iterations shared among 100 VUs (maxDuration: 10m0s, gracefulStop: 30s)

     data_received..................: 3.8 TB 8.5 GB/s
     data_sent......................: 880 kB 2.0 kB/s
     http_req_blocked...............: avg=42.58µs  min=1µs      med=3µs      max=4.93ms   p(90)=5µs      p(95)=7µs
     http_req_connecting............: avg=33.7µs   min=0s       med=0s       max=4ms      p(90)=0s       p(95)=0s
     http_req_duration..............: avg=4.46s    min=298.14ms med=4.44s    max=8.7s     p(90)=7.8s     p(95)=8.2s
       { expected_response:true }...: avg=4.46s    min=298.14ms med=4.44s    max=8.7s     p(90)=7.8s     p(95)=8.2s
     http_req_failed................: 0.00%  ✓ 0         ✗ 10000
     http_req_receiving.............: avg=178.21ms min=75.75ms  med=176.97ms max=359.87ms p(90)=204.48ms p(95)=213.44ms
     http_req_sending...............: avg=37.42µs  min=4µs      med=13µs     max=25.6ms   p(90)=25µs     p(95)=33µs
     http_req_tls_handshaking.......: avg=0s       min=0s       med=0s       max=0s       p(90)=0s       p(95)=0s
     http_req_waiting...............: avg=4.29s    min=143.93ms med=4.26s    max=8.56s    p(90)=7.62s    p(95)=8.03s
     http_reqs......................: 10000  22.281335/s
     iteration_duration.............: avg=4.46s    min=298.19ms med=4.44s    max=8.7s     p(90)=7.8s     p(95)=8.2s
     iterations.....................: 10000  22.281335/s
     vus............................: 22     min=22      max=100
     vus_max........................: 100    min=100     max=100

running (07m28.8s), 000/100 VUs, 10000 complete and 0 interrupted iterations
default ✓ [======================================] 100 VUs  07m28.8s/10m0s  10000/10000 shared iters

I did allocation profiling with Async Profiler. Used similar commands as in https://github.com/apache/pulsar/pull/22494#issuecomment-2052043146 , but with async profiler activated with OPTS.
OPTS="-agentpath:$HOME/tools/async-profiler/lib/libasyncProfiler.so=start,event=cpu,alloc=2m,lock=10ms,file=$PWD/profile.jfr" PULSAR_MEM="-Xms2g -Xmx4g -XX:MaxDirectMemorySize=6g" PULSAR_GC="-XX:+UseG1GC -XX:+PerfDisableSharedMem -XX:+AlwaysPreTouch" PULSAR_EXTRA_OPTS="-Djute.maxbuffer=20000000" PULSAR_STANDALONE_USE_ZOOKEEPER=1 bin/pulsar standalone -nss -nfw 2>&1 | tee standalone.log

With OPTS="-agentpath:$HOME/tools/async-profiler/lib/libasyncProfiler.so=start,event=cpu,alloc=2m,lock=10ms,file=$PWD/profile.jfr", it's possible to do CPU, allocation and lock profiling all at once. I run async-profiler on a Linux box to get best accuracy.
After profiling, I then use this shell script function to generate multiple flamegraph htmls out of the JFR file:
https://github.com/lhotari/pulsar-contributor-toolbox/blob/c150c3d9afc23d4865c2e3283c087e1c1261b4ee/functions/pulsar-contributor-toolbox-functions.sh#L1438-L1458

One interesting detail is that in the load test, the system couldn't keep up with the load without making the optimization to direct buffer allocation in commit eb53342.

I guess it is related to CompositeByteBuf#consolidateIfNeeded, once the number of components exceeds MAX_COMPONENT, it will combine all the components to one component, memory copy happens.

One interesting detail is that in the load test, the system couldn't keep up with the load without making the optimization to direct buffer allocation in commit eb53342.

I guess it is related to CompositeByteBuf#consolidateIfNeeded, once the number of components exceeds MAX_COMPONENT, it will combine all the components to one component, memory copy happens.

Memory copying isn't the biggest problem. A bigger problem is direct memory OOM that seems to happen when the memory space is so fragmented that there isn't free space for the allocation requests. The Netty pool has a maximum chunk size (8MB default) and all larger allocations are huge allocations that aren't pooled. That's why I think the added logic helps since it pre-allocates up to 8MB chunks.
Memory copying will happen, but that seems to be fine.

With 'Accept-Encoding: gzip header in a k6 test, the performance isn't great, however the system is able to stay available and about 25% of the requests timed out under load.
It would be able to optimize the solution further to cache the Gzipped response, but that seems premature optimization.


auto-shutdown failing sometimes on some users

I have seen several cases where for a particular Sagemaker user, the extension seems to not work and idle instances are left running for several days until we catch them through other means.  I would appreciate if you can provide any guidance for debugging.
We are running the server lifecycle script with no modifications, attached to our Sagemaker domain and the timeout set to 2 hours:
#!/bin/bash
# This script installs the idle notebook auto-checker server extension to SageMaker Studio
# The original extension has a lab extension part where users can set the idle timeout via a Jupyter Lab widget.
# In this version the script installs the server side of the extension only. The idle timeout
# can be set via a command-line script which will be also created by this create and places into the
# user's home folder
#
# Installing the server side extension does not require Internet connection (as all the dependencies are stored in the
# install tarball) and can be done via VPCOnly mode.

set -eux

# timeout in minutes
export TIMEOUT_IN_MINS=120

# Should already be running in user home directory, but just to check:
cd /home/sagemaker-user

# By working in a directory starting with ".", we won't clutter up users' Jupyter file tree views
mkdir -p .auto-shutdown

# Create the command-line script for setting the idle timeout
cat > .auto-shutdown/set-time-interval.sh << EOF
#!/opt/conda/bin/python
import json
import requests
TIMEOUT=${TIMEOUT_IN_MINS}
session = requests.Session()
# Getting the xsrf token first from Jupyter Server
response = session.get("http://localhost:8888/jupyter/default/tree")
# calls the idle_checker extension's interface to set the timeout value
response = session.post("http://localhost:8888/jupyter/default/sagemaker-studio-autoshutdown/idle_checker",
            json={"idle_time": TIMEOUT, "keep_terminals": False},
            params={"_xsrf": response.headers['Set-Cookie'].split(";")[0].split("=")[1]})
if response.status_code == 200:
    print("Succeeded, idle timeout set to {} minutes".format(TIMEOUT))
else:
    print("Error!")
    print(response.status_code)
EOF
chmod +x .auto-shutdown/set-time-interval.sh

# "wget" is not part of the base Jupyter Server image, you need to install it first if needed to download the tarball
sudo yum install -y wget
# You can download the tarball from GitHub or alternatively, if you're using VPCOnly mode, you can host on S3
wget -O .auto-shutdown/extension.tar.gz https://github.com/aws-samples/sagemaker-studio-auto-shutdown-extension/raw/main/sagemaker_studio_autoshutdown-0.1.5.tar.gz

# Or instead, could serve the tarball from an S3 bucket in which case "wget" would not be needed:
# aws s3 --endpoint-url [S3 Interface Endpoint] cp s3://[tarball location] .auto-shutdown/extension.tar.gz

# Installs the extension
cd .auto-shutdown
tar xzf extension.tar.gz
cd sagemaker_studio_autoshutdown-0.1.5

# Activate studio environment just for installing extension
export AWS_SAGEMAKER_JUPYTERSERVER_IMAGE="${AWS_SAGEMAKER_JUPYTERSERVER_IMAGE:-'jupyter-server'}"
if [ "$AWS_SAGEMAKER_JUPYTERSERVER_IMAGE" = "jupyter-server-3" ] ; then
    eval "$(conda shell.bash hook)"
    conda activate studio
fi;
pip install --no-dependencies --no-build-isolation -e .
jupyter serverextension enable --py sagemaker_studio_autoshutdown
if [ "$AWS_SAGEMAKER_JUPYTERSERVER_IMAGE" = "jupyter-server-3" ] ; then
    conda deactivate
fi;

# Restarts the jupyter server
nohup supervisorctl -c /etc/supervisor/conf.d/supervisord.conf restart jupyterlabserver

# Waiting for 30 seconds to make sure the Jupyter Server is up and running
sleep 30

# Calling the script to set the idle-timeout and active the extension
/home/sagemaker-user/.auto-shutdown/set-time-interval.sh

I know for several cases what user, what time, and what KernelGateway apps this happened for, but I have not been able to find corresponding logs in Cloudwatch that include the messages that the lifecycle extension prints out, even for cases where I assume the extension has been working.   Should I have log streams under the group /aws/sagemaker/studio for each user's lifecycle scripts?  Or are there other places I should be looking?

I haven't added any other extensions in the LifecycleConfig, although its possible users have tried installing plugins within a session.
Good to know about the potential issues with timing, I will have to dig into that more.  I don't have any log streams below the level of /aws/sagemaker/studio/<domain>/<user>/JupyterServer/default/, which seems to suggest the Lifecycle script isn't actually running.

@durgasury

If your users restart the jupyter server, it's possible that the idle checker will stop polling for apps (see the readme here).

Does this mean that if me or someone else restarts the JupyterServer, auto-shutdown capabilities are gone?

Found the issue—either through user error or one of the Sagemaker updates in the last year or so, the lifecycle was not set as a default for all users.  I see the LifeCycleConfig log streams now.
Thanks for your help.


Spelling

I'm not sure what your goals are for this repository.
Since you have a historical branch, typos can safely be left there...
I'm also not sure which functions are APIs and thus can't be renamed.
I've dropped one prototype since afaict it's unused.
There's at least one end-user facing string change, which I'd hope you'd want to take.

I'm really impressed that folks care so much about such details.  I'm good with that!

#48 for Trie


bugfix: fix daemon panic when meta snapshotter is nil

fix #1139

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

Codecov Report

Merging #1140 into master will increase coverage by 0.85%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1140      +/-   ##
==========================================
+ Coverage   14.75%   15.61%   +0.85%
==========================================
  Files         172      172
  Lines        9628     9688      +60
==========================================
+ Hits         1421     1513      +92
+ Misses       8102     8067      -35
- Partials      105      108       +3

Impacted Files
Coverage Δ

daemon/mgr/container.go
4.59% <0%> (-0.01%)
:arrow_down:

pkg/utils/utils.go
78.15% <0%> (-1.85%)
:arrow_down:

cli/command.go
0% <0%> (ø)
:arrow_up:

cli/cli.go
0% <0%> (ø)
:arrow_up:

client/client.go
46.66% <0%> (+10.3%)
:arrow_up:

LGTM

Thanks a lot for your contribution. This is exactly what we need. While we encourage that provide more details about the pull request description to illustrate what you did. @oiooj
Maybe next time, more info is better. Thanks again.


Documentation

In our documentation we have a configuration section that is not clear, we should mention Hub and Satellite but in config we do not have any such config instead we have Hub, Cluster, and Watcher.
We should have Watcher reference also in our architecture.

so what is the prolem that needs to be solved?

so what is the prolem that needs to be solved?

Documentation, we need to clearly define satellite config, hub config and cluster config correlated with our Initial diagram, where we have not mentioned anything about clusters.
Similarly, in configuration we have not defined anything about satellites or maybe we have defined but have not mentioned the same terminology.

i can do this for you only if you allow me to work with you

i can do this for you only if you allow me to work with you

Sure

alright i am new to git how can i contribute to this? @anonfeed

okay i can do it and send it to you within the period of two days


[MINOR] if sync mode is glue, fix the sync tool class

Change Logs
While writing the Hudi table in spark, I want to sync the table to Glue.
However, the sync mode: "hoodie.datasource.hive_sync.mode": "glue" is actually not applicable in spark.
It instead works using this setting: "hoodie.meta.sync.client.tool.class": "org.apache.hudi.aws.sync.AwsGlueCatalogSyncTool".
I believe that the former config offers a better developer experience and is rather cheap to implement. Raising the PR for same as well.
Impact
Some spark code might spurious hoodie option of "hoodie.datasource.hive_sync.mode": "glue" but is actually writing to some other hive metastore. The behavior might start breaking.
Risk level (write none, low medium or high below)
na
Documentation Update
"none"
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@prabodh1194 Thanks for the contribution, another choice is we fix the HoodieSyncConfig to set up the sync class automatically as a default config option.

@danny0405 , agreed. I however felt that modifying the default at that level might be a breaking change. Adding the AWS Glue catalog here and pairing it with a default class will make this change backward compatible.

If you the sync tool class is the only officially provided tool, I think we are fine to go like this to just set up a default when there is no class explicitly declared.

sounds good. i'll update my PR in that case.

also, @danny0405 , I am not part of the apache JIRA yet, so couldn't raise a ticket against this PR.


pull request for e2e test at 12/03/2016 12:15:04

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Annotation Position and Font

Hi all,
Is there any way to change the position and font of the annotations?
Thanks in advance.

The annotation is added by sending calls into the MMAL camera software and these do not provide for controlling the start position or the font.
You can control size, text color and background color.
As the core software centres the whole text then some very limited control of postioning can be achieved by putting spaces into the annotation string to shift to the right or putting spaces at the end to shift to the left.

@roberttidey Thanks a ton for your reply. I do hope that in the future these options will be added,


Random bogus error in relocate/failover dialog: "No DRPolicy found"

4.13 BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2235423

/test odf-console-e2e-aws

/cherry-pick release-4.14

/cherry-pick release-4.14-compatibility

/retest-required

/test odf-console-e2e-aws

/retest-required

/retest

/override ci/prow/odf-console-e2e-aws
our CI only have checks for ODF and this PR introduces changes in MCO UI, therefore overridding the CI to merge this PR (CI is blocked right now due to issue under investigation)


chore(actions): add cache to NPM dependencies on GitHub actions

This uses bahmutov/npm-install to install NPM dependencies with caching without any configuration.
Also uses cycjimmy/semantic-release-action as a wrapper for Semantic Release.

Nice! Could you add this to react-table and react-charts, and react-virtual too? 😉

Nice! Could you add this to react-table and react-charts, and react-virtual too? 😉

Sure!

@tannerlinsley one note regarding the NPM_TOKEN you have configured on the project.
The action is throwing the following error:

You can follow this steps to fix the issue.

I haven’t changed anything And It was working before. You sure it’s Pulling from the right env variable?
Tanner Linsley
On Aug 19, 2020, 11:42 PM -0600, Rui Saraiva<EMAIL_ADDRESS>wrote:

@tannerlinsley one note regarding the NPM_TOKEN you have configured on the project.
The action is throwing the following error:
You can follow this steps to fix the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I haven’t changed anything And It was working before. You sure it’s Pulling from the right env variable? Tanner Linsley

I'm using the same variables you have before. I'm going to revert to the previously working semantic release step code. #895 895

:tada: This PR is included in version 2.7.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Nice! Could you add this to react-table and react-charts, and react-virtual too? 😉

Here they are:
react-table: https://github.com/tannerlinsley/react-table/pull/2652
react-charts: https://github.com/tannerlinsley/react-charts/pull/115
react-virtual: https://github.com/tannerlinsley/react-virtual/pull/50


Add methods to get type information

Thank you for your contribution! Would you mind adding a unit test for this change?

Added tests and confirmed it passed in local

Sorry, I put some comments to https://github.com/estie-inc/snowflake-connector-rs/pull/25. Could you take a look into it? Thanks!

Thank you for reviewing.  This PR is now ready to be merged.

Fixed and squash commits

v0.2.1 is now published! Thank you for your contribution!


don't merge saving this

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Should this be a Draft PR, or at least have a hold label? Or should we close this? ¯\_(ツ)_/¯

BTW @hughhhh we've now re-enabled all CI checks for draft mode.


pytorch nndct pytorch version

Hi, I'm trying quantization my network model.
However, I was getting error with pytorch version.
In terminal, pytorch_nndct can run only pytorch v1.4, however, I have to use pytorch 1.8 over because of my graphic card(RTX 3080Ti).
Anyone else know about this problem?

You can use pytorch 1.8 to train and save model with torch.save(..., _use_new_zipfile_serialization=False)
Then the checkpoint can be opened and quantized in pytorch1.4 with pytorch_nndct

Pytorch 1.7 version has been supported in Vitis-AI 1.4, and pytorch 1.8, 1.9 version have been supported in Vitis-AI 2.0.
Further more, solution to change pytorch versions among the supported version range is released, please refer to the related part about script replace_pytorch.sh in user guide.

Hi @reddittoob ,
From https://github.com/Xilinx/Vitis-AI/tree/master/tools/Vitis-AI-Quantizer/vai_q_pytorch#quick-start-in-docker-environment , have you tried using /opt/vitis_ai/scripts/replace_pytorch.sh to use PyTorch 1.8?

@hanxue  hello,dear hanxue,I followed the steps in replace_pytooch.sh.
1.I launched the VITIS AI 1.4.
2.conda create -n $1 --clone vitis-ai-pytorch
3.pip uninstall -y torch torchvision
4.pip uninstall  -y pytorch_nndct
5.pip install -r requirements.txt
6.pip install -r requirements.txt
7.cd pytorch_binding
python setup.py bdist_wheel -d ./
pip install ./pytorch_nndct-.whl
I got a pytorch version I wanted .It also works fine.
But after I close it.I found it's missing.
So have I miss some steps?
Does
"""  mamba install -y python=3.6 --force-reinstall
sudo rm -rf /scratch/     """"
mean save the pytorch environement I create?
Or how can I get a stable vai_q_pytorch  with pytorch version I wanted


[Bug]: auto install error: attach failed com.sun.tools.attach.AttachNotSupportedException: Unable to open socket file: target process not responding or HotSpot VM not loaded

Preflight Checklist

[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.
[X] I am not looking for support or already pursued the available support channels without success.

Version
1.0.1
Installation Type
Official Docker Compose
Service Name
Deploy(DongTai-deploy)
Describe the details of the bug and the steps to reproduce it
#lsb_release -a
LSB Version:    :core-4.1-amd64:core-4.1-noarch
Distributor ID: CentOS
Description:    CentOS Linux release 7.6.1810 (Core)
Release:        7.6.1810
Codename:       Core
#java -version
java version "1.8.0_261"
Java(TM) SE Runtime Environment (build 1.8.0_261-b12)
Java HotSpot(TM) 64-Bit Server VM (build 25.261-b12, mixed mode)
tomcat 9.0
iast-agent 1.0.1
error logs
[cn.huoxian.dongtai.iast] trying attach dongtai to process 19263, agent address is /home/webeye/tmp/agent.jar
[cn.huoxian.dongtai.iast] attach failed
com.sun.tools.attach.AttachNotSupportedException: Unable to open socket file: target process not responding or HotSpot VM not loaded
        at sun.tools.attach.LinuxVirtualMachine.<init>(LinuxVirtualMachine.java:106)
        at sun.tools.attach.LinuxAttachProvider.attachVirtualMachine(LinuxAttachProvider.java:63)
        at com.sun.tools.attach.VirtualMachine.attach(VirtualMachine.java:208)
        at com.secnium.iast.agent.AttachLauncher.attach(AttachLauncher.java:20)
        at com.secnium.iast.agent.Agent.main(Agent.java:41)
engine install success for pid: 19263

Additional Information
No response
Logs
No response

Thanks for the feedback, verification test has been scheduled

fixed in v1.0.4


Attempt to load compiled library from lib and ext to handle rubygems …

…moving the library
This seems to be how other gems with native extensions handle it

I see this is fixes issue when upgrading to Ruby 3.2.
Could this be merged?

We are upgrading our Ruby version and ran into the underlying issue too. Hoping it can be merged soon.

Thank you for this! Are the maintainers aware/planning to merge this issue?
@dbalatero? :eyes:

Could this be merged please?

+1, would be great to have this fix incorporated

another bump for a merge 🙏

Any update on this fix being merged?


Getting 401 errors

I have followed the 401 practice as mentioned above. But somehow when I am hitting the controller using an Azure Function it is throwing 401 unauth error. But when I hit the same controller with POSTMAN things are working fine. This has blocked us, please assist.

NOTES:

Our BOT API is hosted in multi region with Traffic Manager in Front
Also in one of our (staging) environment, things are working fine. But in this specific environment it never works.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 521a9f9a-025c-c55b-c939-d9d4efc5c6d0
Version Independent ID: 3bcafb54-f7eb-54f5-1d78-5f2087e2e2dc
Content: Send proactive notifications to users - Bot Service
Content Source: articles/v4sdk/bot-builder-howto-proactive-message.md
Service: bot-service
GitHub Login: @JonathanFingold
Microsoft Alias: kamrani

For troubleshooting help, post your question on Stackoverflow.


Update remove-former-employee-step-4.md

Added details on how to grant access to another user on former employees onedrive

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/5087

@JohanFreelancer9 Please copy review

@msbemba

@kwekuako Please merge this PR. Thanks


Change Operating System course in Core Systems

See reasoning here:
https://github.com/ossu/computer-science/issues/433

Well-formed PR, thank you!


Build the in-memory storage

As part of the v2, we need to implement the in-memory storage exporter based on our current in-memory storage.

Closing as this is a low-level issue related to Jaeger v2 which might take a different path


AFL timeout

This closes #107, and depends on rust-fuzz/afl.rs#162.
I also made the current target display in AFL's output, and dialed the default time limit up to 30 seconds. (ten seconds is sometimes barely enough for AFL to get through its preparation when resuming)

rust-fuzz/afl.rs#162 has baked for a while and been released a couple times; going to merge this now.


feat: cellfirst not none

Closes #799
Summary of Changes
Added static method to Cell, which returns a Cell containing the polars expression of the first Cell in the list that is not None.
Works on a list of Cells, regardless of Cell type (numeric, string, date, etc.).
Returns None if given an empty list.

Do I need to resolve the conversations or will they be resolved by the reviewers?
Because all the requested changes should be implemented in the latest commit.

Yes, please resolve conversations you've handled yourself.

:tada: This PR is included in version 0.27.0 :tada:
The release is available on:

v0.27.0
GitHub release

Your semantic-release bot :package::rocket:


Crash while playing

Mod Version:
1.2.2-<IP_ADDRESS>
Forge Version:
<IP_ADDRESS>55
Link to client log:
https://pastebin.com/5p34dLtm
Link to crash log:
https://pastebin.com/SgGvNpcZ
Description:
Played with my friends on LAN and after 1-2 hours play, light blue screen came to the Minecraft screen(only for me, maybe because I was the host) then crashed and because of VanillaFix mod it showed the problem was Dynamic Surroundings mod.
Then I tried go back to the world, but I can't, the light blue screen came again and crashed.
Tried to delete the mod and config too, but when I did that the game wasn't started.

Did you just add the mod to a pack?
No.

Did it just start crashing, or has it always crashed?

I using this mod 1 years ago and never happened same.

Did you update any other mods in your pack and it started crashing?
No, just added some new one.

Is the modpack you are using available online somewhere?  If the crash is due to mod interaction having the pack available makes things much easier.  It is much harder to work these types of issues with a private kitchen sink pack containing 250 mods.
No, it isn't.

Did you update Animania recently?

Did you update Animania recently?

Did you update Animania recently?

No, we started to play with Animania maybe 2-3 days ago, but we didn't interact with animals when the crash happened(if it matters).
If Animania is a possible problem, we can delete it, isn't an important mod for us.

Did you update Animania recently?

No, we started to play with Animania maybe 2-3 days ago, but we didn't interact with animals when the crash happened(if it matters).
If Animania is a possible problem, we can delete it, isn't an important mod for us.

When Animania updated to 2.x it broke DS.  The latest BETA versions of DS have the changes to be compatible again.

When Animania updated to 2.x it broke DS.  The latest BETA versions of DS have the changes to be compatible again.

When Animania updated to 2.x it broke DS. The latest BETA versions of DS have the changes to be compatible again.

Okey, I'll try for sure, but if I delete or try to change the file to another version, the game(loading screen too) don't show up, dont happen anything.

When Animania updated to 2.x it broke DS. The latest BETA versions of DS have the changes to be compatible again.

Okey, I'll try for sure, but if I delete or try to change the file to another version, the game(loading screen too) don't show up, dont happen anything.

You will also need to install Mixin Bootstrap.  The BETA versions do not include Mixin within the package.

You will also need to install Mixin Bootstrap.  The BETA versions do not include Mixin within the package.

You will also need to install Mixin Bootstrap. The BETA versions do not include Mixin within the package.

Okey, I'll try it.
Thank you for the fast reply.

You will also need to install Mixin Bootstrap. The BETA versions do not include Mixin within the package.

Okey, I'll try it.
Thank you for the fast reply.

@Worest0 I was wondering you were able to get this working?

@Worest0 I was wondering you were able to get this working?

@Worest0 I was wondering you were able to get this working?

Oh, sorry, I forgot to inform you about the solvation.
Yes, its worked, thanks a lot for fast help, and the question too! :)

@Worest0 I was wondering you were able to get this working?

Oh, sorry, I forgot to inform you about the solvation.
Yes, its worked, thanks a lot for fast help, and the question too! :)

Cool.  I will close out.  Let me know if you need anything else.

Cool.  I will close out.  Let me know if you need anything else.


Game Closes After Replay

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
Windows 10 x86_64
What version of the game are you using? Look in the bottom left corner of the main menu.
The Development Build 1.6 From https://ci.appveyor.com/project/KadeDev/kade-engine-windows/branch/master/artifacts
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
When I Play This Replay Of Me FCing Milf At The The Game Closes.
Please describe your issue. Provide extensive detail and images if possible.
There Will Be A DxDiag File And The Replay In .txt Format. To Make The Replay Playable Just Make .txt Into .kadeReplay.
DxDiag.txt
Milf(FC).txt

still happening :(


Fix README.md mistaske

We did not add the dir to PATH, so adding the prefix ./.

Thanks @SmallHanley for contributing!


Add some initial development scaffolding

Add a run script to perform some common dev tasks
Add stuff for pyenv and poetry
Add some packages we'll probably use, black, isort, gunicorn
Create a very simple Flask app
Add a basic route at /system/ping
Add a test for the ping route
Update the README

Everything seems to work as intended. I'm more or less able to follow everything that's going on and the packages you've added all seem reasonable.
I had some trouble getting pyenv working with just build-essentials on Bookworm. This guy's setup guide did the trick, though. We may want to incorporate his apt install line into our documentation. Carl Pearson | Setting up Python with Pyenv and Poetry on Debian

@yorkshirelandscape I added a bit into the README for it. I'm going to go ahead and merge this and if we want to make further adjustments we can open a new PR (or include in a future PR, I'm sure we'll have plenty)


Copy/Paste not possible

It would be easy to allow pasting into a host file.
Manually writing IP addresses can be challenging.

Work on the latest release


how to remove sideberry border

Description
i update the code , but now my sideberry has a thin border around it. I am using rounded corner tweak
Include screenshot / video recording if possible.

Configuration

Firefox Version: 128.0 (64-bit)
OS: Nixos Unstable
Tweaks enabled:

The uc.tweak.theme.sidebery option enables the borderless Sidebery appearance. If you want to remove the shadow around the Sidebery window without using the Sidebery theme, you can disable it by removing or commenting this line in userContent.css:
@import url("tweaks/sidebery.css") (-moz-bool-pref: "uc.tweak.theme.sidebery");


Bicycle simulation stops when velocity is zero

If we are running the simulation and stop pedaling, the velocity of the bicycle goes to zero. This also stops the state of the bicycle from changing as seen here:
    if (m_dt == static_cast<real_t>(0)) { // discrete time state does not change
        m_Ad.setIdentity();
        m_Bd.setZero();
    } else {

https://github.com/oliverlee/bicycle/blob/0ffe6f2ba4de25b1c01fb78ef64168aaae1deadf/src/bicycle/bicycle.cc#L276-L279
We should allow the simulation to advance in time for zero velocity and determine how to handle special cases (e.g. start of the simulation before the rider starts pedaling).

The above code stops the simulation from running if dt = 0, not v = 0. In fact, it is more likely that we limit the state when simulating the bicycle here:
    { // limit allowed bicycle state
        state_t x = m_observer.state();

        auto limit_state_element = [&x](model::Bicycle::state_index_t index, real_t limit) {
            const real_t value = model::Bicycle::get_state_element(x, index);
            if (std::abs(value) > limit) {
                model::Bicycle::set_state_element(x, index, std::copysign(limit, value));
            }
        };

        real_t roll_angle = model_t::get_state_element(m_observer.state(),
                model_t::state_index_t::roll_angle);
        if (std::abs(roll_angle) > constants::pi) {
            // state normalization limits angles to the range [-2*pi, 2*pi]
            roll_angle += std::copysign(constants::two_pi, -1*roll_angle);
        }

        limit_state_element(model_t::state_index_t::roll_rate, roll_rate_limit);
        limit_state_element(model_t::state_index_t::steer_rate, steer_rate_limit);

        m_observer.set_state(x);
    }

https://github.com/oliverlee/phobos/blob/99f9db1afbec528f7189ef97663b91626f360698/projects/src/simbicycle.hh#L93-L114

@mickvangelderen Can you check if the bike starts falling over with v=0? I think it should fall until the roll angle limit is reached, then you would see the state freeze.

This will also need to be handled for the steer-by-wire bicycle which may not have the same solution.

The LQR controller does a good job at solving this issue. See #224.

#224 Hasn't been merged it. I'd like to leave this open until it has been merged (hopefully later this afternoon).


Move User.is_active check from database to Python

https://github.com/sunscrapers/djoser/issues/223
Also renamed active_users_kwargs to filter_users_kwargs, and active_users to filtered_users, because the "active" word is no longer applicable. Feel free to change these variable names to something else if necessary.

Coverage remained the same at 96.937% when pulling f5f2d50b196bcbb2e86e923b7b9dd56f36a252c3 on davidtgq:issue/223 into 22fffb8873627006eec5d1957b0741550856a66e on sunscrapers:master.

Coverage remained the same at 96.937% when pulling f5f2d50b196bcbb2e86e923b7b9dd56f36a252c3 on davidtgq:issue/223 into 22fffb8873627006eec5d1957b0741550856a66e on sunscrapers:master.

Coverage remained the same at 96.937% when pulling f5f2d50b196bcbb2e86e923b7b9dd56f36a252c3 on davidtgq:issue/223 into 22fffb8873627006eec5d1957b0741550856a66e on sunscrapers:master.

Thanks for the feedback, I've made the changes

Coverage decreased (-0.007%) to 96.93% when pulling 4a92af56c3949bf1be804e9f4e7f02993d31032b on davidtgq:issue/223 into 22fffb8873627006eec5d1957b0741550856a66e on sunscrapers:master.

Awesome! Everything is great, so I'm merging your PR into master. Again, thank you very much for the contribution.


Difference between face_recognition.load_image_file(path) and cv2.imread(path)

face_recognition version: face-recognition (1.0.0), face-recognition-models (0.3.0)
Python version: 3.5.2
Operating System: Ubuntu-16.04-LTS x86_64

Description
First of all, thank you for your excellent work!
I'm just confused about this:
What does the method face_recognition.load_image_file(path) do when loading a file? I'm asking this because I found that this method returns a slightly different result compared to cv2.imread(path) when loading the same .jpg image. This means, if I do these:
In [1]: import cv2, face_recognition, numpy as np

In [2]: img_fr = face_recognition.load_image_file('/home/miles/三十.jpg')[:,:,::-1]
          # use [:,:,::-1] to convert RGB -> BGR
          # "三十.jpg" is a photo containing exactly containing one face.

In [3]: img_cv = cv2.imread('/home/miles/三十.jpg')

In [4]: img_fr.shape
Out[4]: (424, 301, 3)

In [5]: img_cv.shape
Out[5]: (424, 301, 3)

In [6]: difcount = 424*301*3 - np.sum(img_fr==img_cv)

In [7]: difcount
Out[7]: 56072

There will be 56072 different pixels. Moreover, if I compute the distance:
In [1]: import cv2, face_recognition, numpy as np

In [2]: img_fr = face_recognition.load_image_file('/home/miles/三十.jpg')[:,:,::-1]

In [3]: img_cv = cv2.imread('/home/miles/三十.jpg')

In [4]: location_fr = face_recognition.face_locations(img_fr)

In [5]: location_cv = face_recognition.face_locations(img_cv)

In [6]: encodings_fr = face_recognition.face_encodings(img_fr, location_fr)

In [7]: encodings_cv = face_recognition.face_encodings(img_cv, location_cv)

In [8]: dis = face_recognition.face_distance(encodings_fr, encodings_cv[0])

In [9]: dis
Out[9]: array([ 0.03327288])

It won't be ZERO while it should be, because they are actually the same image. This took me 2 hours to find out as I was implementing a bit-of-complicated app. I'm curious about how the difference comes. Is it caused by floating number error when decoding the .jpg file?

Good question! The library is just calling scipy.misc.imread(path) with a mode parameter, but it's possible that function is doing some scaling on the values. I'll have to investigate. In the meantime, feel free to use cv2.imread(path) instead to load your images.

face_recognition loads the image using the RGB ordering convention. The method cv2.imread() does this using the inverse ordering, BGR convention. Consider the trick img_rgb = img_bgr[...,::-1].


Spree 3.0 3.1 support

master isn't supporting, obviously

   spree_reports was resolved to 0.1.0, which depends on
      spree_core (~> 2.4)

That's correct. There are currently no plans to support spree >= 3.0 from our side, as we are not using it.


docs: easy issues aren't easy, they're complexity/easy

https://flynn.io/docs/development links to github issues which are
labelled easy, but the label is now complexity/easy so the link
results in no entries.
Signed-off-by: Simon Amor<EMAIL_ADDRESS>

Thanks!


P2P call sample does not work when build iOS ipa (release mode)

Specify the sample to which the issue belongs (use [x]):
[ ] Chat sample
[x] P2P Call sample
[ ] Conference Call sample
Platform (use [x])
[ ] Android
[x] iOS
Env
Flutter 1.22.6
Device info
Manufacture: Apple
Model: iPhone 11 + iPad
OS version: 13.3.1
Describe the bug:
Admin side:

I uploaded APNS Production certificate (p12).
Add 2 users.

App (iOS):

Change our profiles ... to build app.
Change configs.dart with our app info + users.

When I execute the sample project in debug mode (from flutter or hit run in XCode), app works well (I can make audio/video call between 2 devices).
Now I use XCode to export app to ipa file, remove debug app in devices, install the new ipa, login, make call, the other device does not receive the call.
I remove app on 1 device, run app in debug mode on it (device A). I keep app in relese mode on other device (device B). When i make call from device A, device B does not receive. But when I make call from device B, device A still receive the call.
I tried to upload all type of certificates to Admin panel (APNS Sandbox certificate, VOiP certificate) but nothing's solved.
So what's differences between debug & release mode? Did I missing something?
Thanh you.

First of all, check the subscription type there, then check the environment for event there. Try to uncomment commented code it should resolve your issue. The main thing on debug build you have to use the DEVELOPMENT environment and on release build - PRODUCTION. On Android you can use any environment for both types of build.

I got it.
Thank you!


Since upgrading to mac monterey receiving "invalid header"

What version of the package or command are you using?
3.5.1
What are you trying to do?
unarchive tar.gz
What steps did you take?
called  archiver.Unarchive returned err == invalid header
What did you expect to happen, and what actually happened instead?
err == nil
How do you think this should be fixed?
Yes
Please link to any related issues, pull requests, and/or discussion

Bonus: What do you use archiver for, and do you find it useful?
It's been working with same archive file for last 5 yrs without issue, upgraded MAC os to Monterey and stopped working

Can you share exact steps to reproduce the issue and a sample file it reproduces with? I'll try to find a copy of macos Monterrey... (I dev on Linux)

Closing due to inactivity.
(Would also be curious if this happens with Archiver v4.)


add description about file option for add and import

This PR add description about file option for add and import

Thanks @vyloy !


help install

Can you give information about the installation?

I'm waiting for your respond please contact with me on Discord or Telegram
Discord : krypton3881
Telegram : https://t.me/multihunterr

we are working on guide for install


Right buttons in BloomStyleActivity are off screen first time menu opens

First time FAB is clicked and menu is open in the BloomStyleActivity, right buttons are off screen (see attached image). Closing and opening again the menu, buttons are correctly located.

I will fixed, thanks~

Hello. I have the same problem when you first open the menu ((

if I use SpringFloatingActionMenu.ANIMATION_TYPE_BLOOM

Same issue using ANIMATION_TYPE_BLOOM on android 7.0


When Haxelib selfupdate fails (no sudo), it will definitively no more work (OSX).

Under OSX, when trying to selfupdate haxelib without sudo, it fails.
Then subsequent try to selfupdate it (with sudo) has no effect (considered as up to date).
Then if I try to install git projects, their dependencies resolution do not work.

This is now "fixed" as selfupdate doesn't try to change global haxelib executable file (#172)


always get "Transfer err:failed to get account state: adnl request timeout, node <IP_ADDRESS>:4701",why this happen?

here is my code
var messages []*wallet.Message
messages = append(messages, &wallet.Message{
    Mode: 1,
    InternalMessage: &tlb.InternalMessage{
    Bounce:  false,
    DstAddr: addr,
    Amount:  amount,
    Body:    comment,
  },
})
// send tx and wait for it
tx, block, err := w.SendManyWaitTransaction(context.Background(), messages)
if err != nil {
	global.LOG.Sugar().Error("Transfer err:", err.Error())
	return err
}

Hi, looks like liteserver you are using is down, try to use diff one, or initialize pool using config url, like in example

Hi, looks like liteserver you are using is down, try to use diff one, or initialize pool using config url, like in example

hey, thanks for reply,but problem still exists :
client := liteclient.NewConnectionPool()
	configUrl := "https://ton-blockchain.github.io/global.config.json"
	err = client.AddConnectionsFromConfigUrl(context.Background(), configUrl)
	if err != nil {
		global.LOG.Sugar().Error("add connections err:", err.Error())
		return "",err
	}

Hi, looks like liteserver you are using is down, try to use diff one, or initialize pool using config url, like in example

hey, thanks for reply,but problem still exists :
client := liteclient.NewConnectionPool()
	configUrl := "https://ton-blockchain.github.io/global.config.json"
	err = client.AddConnectionsFromConfigUrl(context.Background(), configUrl)
	if err != nil {
		global.LOG.Sugar().Error("add connections err:", err.Error())
		return "",err
	}

Try to change config url to https://ton.org/global.config.json, maybe yours is outdated and some nodes from it are down. Also to skip bad nodes you may add retry and context timeout


Include points interpolated from TIGER edges?

What do y'all think about adding code to machine that would create interpolated points from linear address range features? A primary example of this would be the TIGER address ranges attached to their road "edges", but there are plenty of other sources we've run past that only distribute road centerlines with address ranges attached to them.
Is the increased coverage with lower accuracy acceptable? If we include TIGER, we would surely end up with duplicate address points: who should do the de-duplication? Would including a de-duplication step in OA's build system be a pre-requisite for this to be useful?

This makes me a bit uncomfortable. I’d feel better including ranges as a first-class object in some form, rather than generating fake points.

ref: https://github.com/pelias/pelias/wiki/Interpolation:-existing-standards

@iandees can I close this or leave this open? Thanks!

Let's close it for now.


Existing update commands should expose generic update fields

For consistency across commands, and to fill any gaps that may occur when a service exposes new functionality before a CLI rev, we should expose the generic update parameters (--add --set --remove) on each of our existing update commands.
The generic update is currently assembled from a setter and a getter, with common code to handle the generic parameters. We should be able to relatively easily insert our existing custom commands into that flow.
The recommended execution order would be:

Call the getter to retrieve the object.
Pass the object into the custom command (this will require removing the getter and setter statements from the existing custom command and adding a parameter for the object)
Pass the object on to the standard generic update code.
Pass the object to the setter.

PR #735 merged.


Fehler 403: access_denied

Hello,
I have an issue with the google  authorization (browser link)
Always this error:

I tried to recreate plexdrive and credentials with no luck..
Anyone an idea?
Cheers
Stephan

Try to create the oauth credentials using the same google account your logged in with in the browser or logout first and log into the right account before opening the link from plexdrive.
Felix Bünemann

Am 11.08.2021 um 23:48 schrieb Stephan @.***>:

Hello,
I have an issue with the google authorization (browser link)
Always this error:
I tried to recreate plexdrive and credentials with no luck..
Anyone an idea?
Cheers
Stephan
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

You should hit publish on the oauth consent screen as mentioned in the rclone docs https://rclone.org/drive/#making-your-own-client-id


[REVIEW] Voting System and Leaderboard

Description
A simple voting system, with a live leaderboard
Project Link:

Repo - https://github.com/hrahul2605/jv-client
Live at  - https://jv.hrahul2605.tech/
You can also join the discord community here
Feel free to check out other cool repositories of EddieHub Community here

Hey, @hrahul2605 Thanks for submitting. I would love to give you some suggestion

Add  Code of conduct & Contributing MD's
Add issue and pr template
Add more screenshots of your working project in the readme
Add actions to your project
Add project board to plan new features and add milestones to them
Add more and more issues for your upcoming plans so that the contributors can see what you are planning for the future of the project.
Add manual or automated releases
Add customized issue tags

Nice one with the mobile view lol 😂

@schmelto Haha! you are right! I just checked the mobile view :rofl:

@kaiwalyakoparkar Hey, thanks for the review, I'll do the needful.
lol yes I plan to do the mobile view soon enough :grin:

@hrahul2605 Can't wait to see that!

This aint to be stale. @nhcarrigan maybe we use a older version of the actin, but supply the arguments for the newer versions. We are on actions/stale@v1 we should be on actions/stale@v3?

Reviewed on live stream https://www.youtube.com/watch?v=XT3_O4jULRE

looks good
deployed link is great
no releases or tags
suggest adding a code of conduct
suggest hiding packages and wiki if not used
screenshot in the readme would be useful (extra points for animated gif)
suggest using issues even if just you working on it

Thank you for the feedback eddie! Much appreciated


Blue Lake Milling emission factor

SA1 BLULAKE1 currently 0.657 feels low

it's a bioenergy plant:
https://www.smartcompany.com.au/industries/agribusiness/food-manufacturer-turns-oat-husks-into-electricity-in-bordertown/
in theory bioenergy plants should have emissions factors close to zero.
what units is 0.657 in?

There is (was?) a small diesel power reciprocating engine there well before the biogas plant. That's what this would refer to - and the emissions intensity would be consistent with that.

what units is 0.657 in?

t/MWh


[WIP] Issue 184 xauth bundle

close #184

[ ] correct docs
[ ] change name of bundle

Implemented already.


Does the obs-gamecapture command work on created flatimages?

This is more of a question rather than an issue, but could I input the obs-gamecapturecommand on a flatimage? I use the obs-vkcaoture plugin for OBS.

Hello! Yes, it should work without a problem. You can use it with gamescope to help avoid issues with fullscreen and OBS like this gamescope -w 1920 -h 1080 -- ./game.flatimage --select-index 0, assuming there is only one game in the image, if there is more you can discover their indices with ./game.flatimage --select-list.


[WIP] Example: Add a new Kafka Broker filter

Signed-off-by: Nguyen Phan<EMAIL_ADDRESS>
Commit Message: Add a new example for the Kafka Broker filter
Additional Description:
Risk Level: Low
Testing: README.md guideline
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

unknown type: envoy.extensions.filters.network.kafka_broker.v3.KafkaBroker for type Any

https://dev.azure.com/cncf/envoy/_build/results?buildId=112228&view=logs&j=8c169225-0ae8-53bd-947f-07cb81846cb5&t=6b0ace90-28b2-51d9-2951-b1fd35e67dec
Does the envoyproxy/envoy-build-ubuntu in the pipeline support KafkaBroker?
My local machine works well with envoyproxy/envoy-contrib-dev:latest
@phlax Do you have any comments?

Does the envoyproxy/envoy-build-ubuntu in the pipeline support KafkaBroker?

its not really related to the build image - more the release binary that is being tested - i think that we are not testing against the binary with contribs included
if you check examples/BUILD there are listed exclusions there, you can add kafka

if you check examples/BUILD there are listed exclusions there, you can add kafka

Hi @phlax, the pipeline is good now, please help me take a look at the MR.
Thank you

@nguyenptk apologies for slow review - as mentioned offline i have been busy with release so will review once that is out of the way
/wait-any

@nguyenptk overall we need to simplify and cleanup the verify script
As discussed offline im happy to add docs for this but it needs to be super-clear step-by-step what is going on for me to do it

perhaps you could tell me what the "steps" are that you expect in the documentation, i think that will make it easier to clariify what the tests should be in the verify script

here is the "steps"

couple of points

from an end users perspective it is not related to verifying - the verification is that the docs do what they say
starting/stopping the containers is not the responsibility of the verify script

hi @adamkotwasinski while working on this PR we noticed that when you bring up the docker composition it churns out endless logs with:
proxy           | [2022-08-16 08:18:06.482][1][debug][dns] [source/extensions/network/dns_resolver/cares/dns_impl.cc:354] dns resolution for <IP_ADDRESS> started
proxy           | [2022-08-16 08:18:06.631][1][debug][dns] [source/extensions/network/dns_resolver/cares/dns_impl.cc:275] dns resolution for <IP_ADDRESS> completed with status 0
proxy           | [2022-08-16 08:18:06.631][1][debug][upstream] [source/common/upstream/upstream_impl.cc:275] transport socket match, socket default selected for host with address <IP_ADDRESS>:9092
proxy           | [2022-08-16 08:18:06.631][1][debug][upstream] [source/common/upstream/strict_dns_cluster.cc:180] DNS refresh rate reset for <IP_ADDRESS>, refresh rate 5000 ms
proxy           | [2022-08-16 08:18:10.487][1][debug][main] [source/server/server.cc:251] flushing stats
zookeeper       | [2022-08-16 08:18:11,622] WARN Unexpected exception (org.apache.zookeeper.server.NIOServerCnxn)
zookeeper       | EndOfStreamException: Unable to read additional data from client, it probably closed the socket: address = /<IP_ADDRESS>:60574, session = 0x0
zookeeper       | 	at org.apache.zookeeper.server.NIOServerCnxn.handleFailedRead(NIOServerCnxn.java:163)
zookeeper       | 	at org.apache.zookeeper.server.NIOServerCnxn.doIO(NIOServerCnxn.java:326)
zookeeper       | 	at org.apache.zookeeper.server.NIOServerCnxnFactory$IOWorkRequest.doWork(NIOServerCnxnFactory.java:522)
zookeeper       | 	at org.apache.zookeeper.server.WorkerService$ScheduledWorkRequest.run(WorkerService.java:154)
zookeeper       | 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
zookeeper       | 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
zookeeper       | 	at java.base/java.lang.Thread.run(Thread.java:829)

tracking the zookeeper issue it seems like despite warning about an Unexpected exception (not sure if that should be a warning or unexpected etc) that this is "normal" behaviour when a client disconnects
apart from the issue of log hogging and make any real errors hard to track it would fill up a disk/logrotate pretty quickly
it seems as though the error/warning is triggered in response to Envoy flushing stats
so, we are wondering if there is a way to prevent this either by Envoy not flushing stats (so frequently) or suppressing this warning/error - and i guess whether there is something incorrect in either the config here or Envoy's interaction with kafka and/or zookeeper

Hey, I'm not too experienced in Kafka's usage of Zookeeper, but it should be a fairly simple one (it only keeps some topology metadata). However I think I've seen something similar happening in the past in case of network failure - so maybe that's a docker thing? (you never know)
I'm on holidays until end-of-week, but if I were to debug, I'd check the behaviour without Envoy in the stack just to remove the potential culprit (so only 9092 instead of 10000).
The other ideas that come to my mind would be:

changing logging of org.apache.zookeeper.server.NIOServerCnxn in zk's log4j.properties
(exploratory) getting rid of ZK in the stack and adopting KRaft

so we hit a bigger problem with this sandbox - we realized that it wasnt actually using the proxy
probably best to wait until you get back - but could do with a bit of input into what exactly should go through the proxy
the tests/examples we were adding were

add/check a topic
send a message
receive the message

when updating the config to actually use the proxy it kinda falls apart - so not sure how it should work when containerized - ie not using localhost

Apart from the two comments above (first one is important) everything in this PR feels okay - it is okay to use Envoy for every Kafka-related operation (just like in the commands to create-topic / list-topics / send / receive message).
There's some small manual for configuring Envoy/Kafka - https://adam-kotwasinski.medium.com/deploying-envoy-and-kafka-8aa7513ec0a0 (esp. first part - Routing all traffic through Envoy (including replication))
I do not think that hostname change from localhost to something else is going to hurt, as the docker compose AFAIU should make everything resolve-able.
If that doesn't help, I'll try to take a look next week when I'm back!

I think we solved this problem here with
KAFKA_LISTENER: INTERNAL://kafka-server:9092,EXTERNAL://proxy:10000
# Advertised listener value needs to be equal to Envoy's listener
# (will make clients discovering this broker talk to it through Envoy).
KAFKA_ADVERTISED_LISTENERS: INTERNAL://kafka-server:9092,EXTERNAL://proxy:10000
KAFKA_LISTENER_SECURITY_PROTOCOL_MAP: INTERNAL:PLAINTEXT,EXTERNAL:PLAINTEXT
KAFKA_INTER_BROKER_LISTENER_NAME: INTERNAL
KAFKA_OFFSETS_TOPIC_REPLICATION_FACTOR: 1

I updated the health check script of the Zookeeper and the error of
EndOfStreamException: Unable to read additional data from client, it probably closed the socket:

was gone.

This metric is increased by kafka-topics --create invocations. If you point
the script at the proxy it should go through envoy and increase the metric.
You can try that with an already-setup demo.
niedz., 28 sie 2022, 09:38 użytkownik Nguyên @.***>
napisał:

hi @adamkotwasinski https://github.com/adamkotwasinski, I have a
question about the kafka.kafka_broker.request.create_topics_request
stats. I cannot receive the kafka.kafka_broker.request.create_topics_request:
1 on my local machine (I saw it for the first time when I grepped the
stats but now). Currently, I only have
kafka.kafka_broker.request.create_topics_request: 0
Could you point me more information about that?
Note: other stats look good. For examples:
kafka.kafka_broker.request.api_versions_request: 4
kafka.kafka_broker.request.find_coordinator_request: 1
kafka.kafka_broker.request.metadata_request: 4
kafka.kafka_broker.response.api_versions_response: 4
kafka.kafka_broker.response.find_coordinator_response: 1
kafka.kafka_broker.response.metadata_response: 4
cluster.kafka_service.max_host_weight: 1
cluster.kafka_service.membership_healthy: 1
cluster.kafka_service.membership_total: 1
—
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/pull/22031#issuecomment-1229399423,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEJDEL62ISDZM6DFSVU5ZXLV3MJPFANCNFSM52YEP6JQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

/docs