[Question] Plugin recreation informant

Question
Hello, Aieuo!
Recently, myself and my friend have been working on a recreation of your plugin for Vanilla Bedrock Edition. Before we continue, we would like to contact you for your permission and/or involvement in the project.
If that intrigues your interest and you would like to discuss this further, please contact me at explosionhm on Discord.
(Sorry for this method of contact; I did not find any contact information linked to your profile.)
Plugin Version
undefined
Additional Information
No response

Hello,
Thank you for using Mineflow and for reaching out.
I appreciate your project on Vanilla Bedrock Edition. Feel free to use our code under the NCSA license.
I was actually working on making Mineflow compatible with BDSX as well. If you are using TypeScript, I can provide you with the code (please note that only some parts have been converted, and it is not yet optimized for TypeScript/JavaScript).


Slow to laggy to show tooltip on client side

After running on some time, it becomes laggy and slow to show tooltip on client side.

This should be due to Creating/Releasing frames, which are consuming memory.
The frame should be fixed, only data should be updated.
The Gametooltip shouldn't read text from the UI label, it should be set to the item link while creating the data.


"Buciumeni" is listed in the drop-down menu

Description:
"Buciumeni" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Buciumeni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Buciumeni, DÂMBOVIȚA".


Sike proofs

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Can we add a Readme.md file to /tests/saw/sike that explains what the SAW Proofs are proving?

Codecov Report

Merging #1030 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1030      +/-   ##
==========================================
- Coverage   86.53%   86.52%   -0.01%
==========================================
  Files         166      166
  Lines       15679    15679
==========================================
- Hits        13568    13567       -1
- Misses       2111     2112       +1


Do not attempt to apply changes for unknown models

Prevents an error in the event that a record's ID parses as a relational_pouch ID, but refers to a model which the running application does not have.
E.g., this might happen if you have an application that replicates from a CouchDB instance. As the application evolves, new models may be introduced and show up in the remote CouchDB instance. Clients are likely to have a span of time when they can see records for the new models, but have not yet updated to the newest code — particularly if the application is an offline webapp or a Cordova-based mobile app.

Ah, this is clever - good catch!

eb4f5635377c48ab4666805cbcc069779914f341


build.xml: add target for building customization specs

Temporary solution to build documentation for customizations of MEI. Used specifically to generate docs for MEI basic, but can probably work for other ones.
Built and reviewed over zoom with Benni, seems to work.


'Edit Code' button selects not all relevant code

Cases:

has no child

has child
, correct behaviour
has child
, incorrect behaviour

Somtimes only the inner
 gets selected. I can't identify a pattern.
1: always correct

2: this time correct

3: but most of the time incorrect

This issue has been resolved in the latest version of Codux.
Please let us know if you're encountering any further issues.


please provide a basic configuration so we dont need to check what is wrong

I am triying with this one and it says me is incorrect because doesnt found the model

Fixed in 01.1


💡 [REQUEST] - Integrate Pydantic Model Generation within Database Class

Describe your use-case or problem
I would like to request a feature that integrates ultimate-notion with FastAPI, enabling the dynamic creation of APIs with automatic Pydantic schema generation based on a provided database ID. This would streamline the process of building APIs that interact with Notion databases by automating schema creation and validation.
Proposed solution / feature
Currently, to retrieve and use a database schema, we follow these steps:
db = session.get_db(database_id)
schema = db.schema

I propose adding functionality to the Database class to simplify this process. Specifically, we could generate a Pydantic model directly from the Database instance like so:
pydantic_model = db.pydantic_model

It would be easy to integrate it with FastAPI.
Additional context or examples
No response

Really interesting idea @sarvesh4396. Just to understand your intention, db.pydantic_model would return a model that checks the properties when a database page is created, basically what db.create_page does internally, but using pydantic instead. I think that would be a cool feature. Would you like to provide a PR?

@FlorianWilhelm, sorry for the late reply—I've been caught up. The project is very robust and promising, but some things have changed since I last checked. I had written a script, but I noticed some code has been refactored. I'll see what I can do whenever I get the chance.

Hi @sarvesh4396, sure, take your time and Ultimate Notion is still alpha, so with every release things might change a bit. Maybe wait a few more days before you start as the 0.6 release is about to happen quite soon.


CI: Use CrateDB 5.4.4

About
On CI, use a more recent CrateDB version for the regular runs.
NB
Download location is https://cdn.crate.io/downloads/releases/crate-5.4.4.tar.gz. Correct or outdated/legacy? /cc @seut

Download location is https://cdn.crate.io/downloads/releases/crate-5.4.4.tar.gz. Correct or outdated/legacy? /cc @seut

Yes this location is the legacy download url used when CrateDB did not bundle the JDK.
Would be great if we could switch to using os/arch related download url's in future.


Add support for grouping results by process ID

I would like to propose this pull request to add groupby support for the process-util.py script. This will allow you to use --groupby id to automatically group the results by PID.
It also adds the missing parent_name header.

Hi @keithmccammon, the merge seems to have only merged the additional missing parent_name header, but removed the group_by functionality. The current version of process_util.py seems to be missing those changes.


Update language docs to use explicit path to Document.

@dhermes Eliminates accidental conflict on common class name, 'Document'.

@mckoss were these references broken?
The current links should be ok since .Document should be referencing, I believe, the imported Document.

Not broken now - just future-proofing.

Thanks @mckoss!


Saleor on MS SQL

Hello everyone,
First of all, we love Saleor, thank you so much, great work!
Here's my issue:
We have a back-office system, developed by our team in .net/c# which runs on MS SQL database (2017)
And we want to use saleor as our e-commerce storefront.
We have some intersects like, customer table, accounts, products, purchase etc.
I was wondering, if someone must've faced same issue before?
What is the best road you suggest us?
How should i manage to synchronize my system & saleor database at the same time?

Good question, we usually solve situations like this by introducing explicit synchronization points between systems. Ideally a bridge would be an external application that uses APIs of both systems but until #3343 lands it may be easier to add such a script directly to Saleor's codebase and run it as a Celery beat task.

Thank you for the reply. Yeah, apparently that sounds like an option.
I really think, Saleor is an outstanding application. It would be amazing if it also run on Mssql.


Reduce smoltcp dependency features to a minimum

Since Dependencies are resolved with the union of all features enabled on them, it's good practice to remove unnecessary dependencies.
More features are added by the dev-dependency so that the examples build

bors r+


Bug fixes

I fixed two bugs.
The first one was in the makefile.in, due to the order of the flags when compiling zeta I was unable to use SDL at all on Linux. Now it should be solved (and I hope it's fine for macOS).
The second bug was described in issue #27 and it was due to a misuse of pointers in this function. Now it should work.

I just noticed that this PR also contains the changes from the other PR, so I'm closing this and reopening when that one is merged. (I'm really not sure how would I go about making two different PRs for different things, I'm sorry for the mess)


tic tac toe game using c++

I want to make tic tac toe game using c++

Pls assign me this task

Hey this is Ansh Charak, 4th year UG from IIT Bombay
Kindly assign this issue to me under hacktoberfest and hacktoberfest2022
I can implement this effectively.


tracepoint args with multiple attach points

Perhaps one, or both, of these should work:
# bpftrace -e 'tracepoint:syscalls:sys_exit_statfs,tracepoint:syscalls:sys_exit_statx { @[args->ret] = count(); }'
The args builtin only works for probes with 1 attach point

# bpftrace -e 'tracepoint:syscalls:sys_exit_stat* { @[args->ret] = count(); }'
No such file or directory: /sys/kernel/debug/tracing/events/syscalls/sys_exit_stat*/format
Struct/union of type '_tracepoint_syscalls_sys_exit_stat*' does not contain a field named 'ret'

(Ignore the "No such file or directory" warning on wildcards; I'll file a separate bug for that: #133 )
There's multiple workarounds: you can list the probes one by one with the same code block for each. Or, you can use ctx and its offset from the /sys/kernel/debug/tracing/events/.../format file, eg:
# bpftrace -e 'tracepoint:syscalls:sys_exit_stat* { @[*(ctx + 16)] = count(); }'
No such file or directory: /sys/kernel/debug/tracing/events/syscalls/sys_exit_stat*/format
Attaching 2 probes...
^C

@[-2]: 2

So... should args work with mulitple attach points (probes)? args->whatever could mean different things logically, and matching them all with a wildcard may not make sens. They could also originate from different offsets.
I'm not adding a bug tag because I'm not sure yet if this is a bug, or is working as intended. Although I suspect people will try matching multiple tracepoints and will expect args->ret to be the return value.

fixed by #370


qt6 lrelease not found

Describe the bug
Building mpc-qt (see #280943) fails because lrelease cannot be found. The build systems expects it in qtbase, but it is provided by qttools.
Steps To Reproduce
Steps to reproduce the behavior:

checkout #280943
build mpc-qt

Expected behavior
A successful build of mpc-qt.
Notify maintainers
@milahu
@NickCao
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.7.1, NixOS, 24.05 (Uakari), 24.05.git.56e9d11d2023`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

@romildo is this issue about building 23.12 ? Not against it, just a bit unusual since it hasn't been merged yet

@romildo is this issue about building 23.12 ? Not against it, just a bit unusual since it hasn't been merged yet

It is about updating mpc-qt to version 23.12, which migrated to qt6. The mentioned PR fails to build due to the issue reported here. Therefore this issue has to be fixed before mpc-qt can be updated in the nixpkgs repository.

At least it fails when building in my unstable system, and also failed on the repository when I made the PR. But it seems to build in the repository now.

23.02 builds for me on master, thats why I was checking

Then I suppose we can merge https://github.com/NixOS/nixpkgs/pull/280943 now.

23.02 builds for me on master, thats why I was checking

This version still uses qt5. The new one migrated to qt6.

@kirillrdy could you review https://github.com/NixOS/nixpkgs/pull/280943, please?

yes, will do, can you close this, as mpc-qt builds on master

yes, will do, can you close this, as mpc-qt builds on master

I will do that after https://github.com/NixOS/nixpkgs/pull/280943 is reviewed.

We've been having this issue for long: https://github.com/NixOS/nixpkgs/issues/214765. The current fix is just to patch individual packages and hope qmake fade out one day.

Duplicate of https://github.com/NixOS/nixpkgs/issues/214765


Grasp width in mm

Hello there,
actually it is not an issue.
Could you please give me a reference of the equation you used  you used to transform pixels into mm?
I am trying to get something to use in a UR5 arm with realsense d435 but I am a little bit lost.
I really appreciate you help ;)

G'day @caiobarrosv
The issue here is my terrible documentation ;-)
The first calculation, to mm, is effectively an approximation of the field of view of the camera.  The network outputs the grasp width in image coordiantes (i.e. pixels), and this uses the field of view of the camera, some trig and the measured depth of the grasp to convert this to a width in real world coordiates.  I wasn't 100% sure just looking at it either, so I've sketched it in the attached image... hopefully that helps.
The second line of that equation is specific to the kinova gripper, which only lets you set position in terms of joint encoder positions.
If you're trying to use a different robot, I would recommend having a loot at my other repo https://github.com/dougsm/mvp_grasp.  It is much more up to date, and much more nicely laid out.  You can then use the ggcnn packages independently from your robot, and there are some xamples of using the services with a franka emika panda robot.

First of all thank you for the fast response. I really appreciate it.
It is well documented. The code is clear and easy to understand. You did a great job.
Now I complete understand what you did and I totally agree with it.
You really did something different in the mvp repo (code)
in this repo:
g_width = 2 * ((CURR_Z + 0.07)) * np.tan(0.1 * grip_width / 2.0 / 180.0 * np.pi) * 1000
in mvp repo:
width_m = width_img / 300.0 * 2.0 * depth_crop * np.tan(self.cam_fov * self.img_crop_size/depth.shape[0] / 2.0 / 180.0 * np.pi)
Could you please clarify some other things? :)
1 - width_img / 300.0 is used only for resizing, right?
2 - In this case you generate an image and then select the position of the max pixel created from np.argmax(points_outs). By using depth_crop instead of CURR_Z + 0.07 you are considering the distance of the cam to the max_pixel instead of the height (as shown in this draw below), right?

3 - When the height of the end effector varies, the grasp width also varies. Shouldn't it be constant regardless of the height? I am running a simulation in gazebo (I also have the real arm available) with the realsense d435 plugin in order to generate depth data.

4 - Why has grip_width been replaced with self.img_crop_size?
Thank you in advance :)

Hmm... Interesting... I didn't realise they were different!
After some reverse engineering, I think I understand how I came to the second equation for width.  Both are approximations, but make some different assumptions.  Instead of treating one pixel as an angle in space, it maps the full field of view of the camera to a width in physical space, then computes a fraction of this based on the width output from ggcnn.  Fortunately they give practically identical results! (see graph below).
To answer your questions:

This should be relative to the size of the cropped depth image (which happens to be 300px), but should really be using the img_crop_size variable. I'll make a note to fix this.
Yes that's right, both represent the same value -- the distance from the grasp to the camera.  The first way used the position of the gripper + the offset to the camera (presumably because this was more accurate than the depth camera values), the second uses the depth camera values directly.
The GG-CNN outputs the width in terms of pixels, so as the camera gets closer to an object, it should (in theory) output a larger value for the width because the object appears larger. And this will then be counteracted by the formula which scales by the depth.  I say in theory because the width output of the GG-CNN is probably the least accurate, probably due to the noisy training data, so there's no guarantee that it will adjust the width linearly as the depth changes (but it should be pretty close).
See below.

I understood the explanation for questions 1, 2 and 4.
In the case of question 3, the grasp width in mm must be at least constant as you said (when the depth is considered).
I recorded a video and plotted 2 graphics. The plot above shows the grasp width in mm and the plot below shows the height of the end effector relative to the base.
When the robot approaches the object, the grasp width must be the same (right?), but varies from 4.3 mm to 8 mm. You can see in the graph that when the robot moves downwards (red curve), the range also changes, but the depth should take care of that, right?
(I am using the same equation you used in mvp repo)


Looking for help

I'm happy to say I'm looking for a new collaborator to help to support this list. Someone who has a fresh glance on maintenance, ready to collaborate with contributors and will have fun working on this...
Responsibilities:

search/add new awesome entries to the list periodically
review/merge/dismiss new pull requests
remove outdated and fix broken/moved entries

The one will need to spend up to 30 minutes per week. Notice, I'm still on the track, but my priorities are changing and I will look into this less often.
If you are interested, say about this in the comments or contact me privately. I will keep this open for a couple of days.

Nice. I'm glad it's moving on :+1:
Some points related to Awesome List:
First of all, the list has to be awesome, that means no bullshit. IMO all shards have to:

Be tagged with version tags
Have tests
Have a certain amount of usefulness, be reusable throughout a vast amount of possible projects

I also propose to introduce some kind of health meters based on dependencies status, open issues and crystal version compatibility. Maybe it worth developing a service which asserts shards against these criterias.
Crystal ANN
In addition, I don't like current implementation of Crystal ANN. It has to be curated, not "anyone can add their gist-shards". It has to have a e-mail newsletter (it may already, but I haven't noticed).
Me
I'm happy with Crystal. Again, it's the best thing in my programmer's life. I can't develop on Ruby anymore because of knowing that I could do it better with Crystal. It's my curse also, because there is a lack of needed libraries and I don't have so much time to implement them all.
About time. ATM I'm on my way for creating a next big project with passive income (built with Crystal ofc). When I have money - I have time. I'm a creative person with a passion for programming. But the reality tells me to put my time in profit projects. I wish I could let myself to work on Crystal much more.
I need time and in the nearest future I hope I could take a major part in all this Crystal Community stuff.

Could you add me to it?:heart:
Would like to work on it!

I would be glad to help too :smile:

@tbrand @hugoabonizio welcome on board!

Thanks @veelenga! :)

@veelenga i'd be happy to review some PR's and libraries too, although i'd probably forget to do periodic things such as cleanup lol

Thanks!

@RX14 would love to see your reviews too ;)


Updated edge.js to properly support setting the page load strategy

Currently if we set the pageLoadStrategy in JavaScript :
    this.driverOptions.setPageLoadStrategy("none");

We get an error that this is not a valid ms:edge Option :
    WebDriverError: invalid argument: cannot parse capability: ms:edgeOptions
    from invalid argument: unrecognized msedge option: pageLoadStrategy

Since this is a Chromium, I had to change the capability so it gets passed into as a chromium option like we do with logPrefs

Is this still considered to be merged in or abonded and pageLoadStrategy will only be available with selenium 4?

Is this still considered to be merged in or abonded and pageLoadStrategy will only be available with selenium 4?

We migrated to Selenium 4 and it solved our problem but if people are still facing this issue, would be good to merge this in.

We're in the process of deprecating so there won't be any official releases with this change, but I'm merging this PR in case it is useful to anyone building the project from source. Thank you for your contribution!


[BUG] The learning rate scheduler is being ignored in the first optimization step.

Describe the bug
It appears that no matter how the learning rate scheduler is configured, during the first training step a learning rate of 1e-3 is always used. My understanding is that this happens because lr_scheduler.step() is always called after optimizer.step() and there is no initialization for the initial step learning rate. I'm currently using optimizer.set_lr(0) explicitly to work around this issue. It can be quite detrimental to full fine-tuning of LLMs (as opposed to just LoRA/PEFT training) as it throws the model off completely in the first training step.
Is this intentional? If so, should this be fixed inside DeepSpeed or are we supposed to just use optimizer.set_lr(0) like I'm currently doing. In either case, I think this should be documented somewhere as it is somewhat surprising behavior.
To Reproduce
It's pretty easy to reproduce with any configuration you already have. I was using this when debugging and finding this out:
"optimizer": {
    "type": "AdamW",
    "params": {
        "betas": [0.9, 0.98],
        "eps": 1e-6,
        "weight_decay": 3e-6,
    },
},
"scheduler": {
    "type": "WarmupDecayLR",
    "params": {
        "warmup_min_lr": 0,
        "warmup_max_lr": 1e-4,
        "warmup_num_steps": 100,
        "total_num_steps": 1000,
    },
}

@eaplatanios try out the linked branch. Another workaround would be setting the optimizer lr in the ds config to the desired starting lr.


KubeCon 2017: glossary entry for Persistent Volume Claim

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.9 Features: set Milestone to 1.9 and Base Branch to release-1.9
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: Please check the “Allow edits from maintainers” box (see image below) to
allow reviewers to fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.
NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box. This is a temporary workaround to address a known issue with GitHub.

This change is

Netlify failed but it seems to be transient issue unrelated to the changes.
Push again to rerun Netlify.

Thank you for the change @erwbgy !
You might be wondering why the Netlify preview is breaking, so here's some context: the glossary_tooltip tag sets up a bit of a merge dependency, since the other glossary term you are referencing has to already exist. In other words, your current PR requires the merge of https://github.com/kubernetes/website/pull/6591 to build properly.
You can either:
(A) Wait for said PR to get revised and merged
(B) Remove the glossary_tooltip from this PR (which SIG docs maintainers will add back in after-the-fact), so that this can be merged right away.
We're fine with either approach. Let us know if you have any questions, and apologies for the confusion!


Upgrade to keepalived 1.4.2

This Pull Request only upgrade the version ok keepalived to version 1.4.2
It was tested in our environment on baremetal and AWS and work fine.

CLA Signed

/assign @bowei

/ok-to-test

Why is not the Docker image on k8s.gcr.io/kube-keepalived-vip updated with this change?


[Improve][Connector-V2] Improve Paimon source split enumerator

Purpose of this pull request
Improve Paimon source split enumerator
Does this PR introduce any user-facing change?
no
How was this patch tested?
exists test
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

cc @hailin0

Please add test case.

Please add test case.

Thanks for your review. I think this is an optimization of the existing PaimonSourceSplitEnumerator. Currently, pamion's test case can already cover it.

Could you implement stream read from paimon source ? I don't think this is necessary in batch read scenarios.

Could you implement stream read for paimon source ? I don't think this is necessary in batch read scenarios.

Ok, I'd be happy to do it.

@dailai


Prompt to open exported notebook in native editor

For #8078

Codecov Report

Merging #8315 into master will decrease coverage by 0.77%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##           master    #8315      +/-   ##
==========================================
- Coverage   59.32%   58.54%   -0.78%
==========================================
  Files         504      504
  Lines       23096    22737     -359
  Branches     3737     3736       -1
==========================================
- Hits        13701    13311     -390
- Misses       8501     8533      +32
+ Partials      894      893       -1

Impacted Files
Coverage Δ

...ractive-window/interactiveWindowCommandListener.ts
59.61% <44.44%> (-0.69%)
:arrow_down:

...nt/datascience/jupyter/invalidNotebookFileError.ts
0% <0%> (-75%)
:arrow_down:

.../client/datascience/jupyter/jupyterConnectError.ts
0% <0%> (-66.67%)
:arrow_down:

...science/jupyter/jupyterKernelPromiseFailedError.ts
0% <0%> (-50%)
:arrow_down:

...ent/datascience/jupyter/jupyterWaitForIdleError.ts
0% <0%> (-50%)
:arrow_down:

src/client/interpreter/activation/service.ts
48.27% <0%> (-41.38%)
:arrow_down:

...cience/jupyter/jupyterDebuggerNotInstalledError.ts
50% <0%> (-30%)
:arrow_down:

src/client/language/characters.ts
12% <0%> (-21.34%)
:arrow_down:

src/client/testing/codeLenses/main.ts
60% <0%> (-17.78%)
:arrow_down:

src/client/datascience/cellFactory.ts
72.34% <0%> (-17.32%)
:arrow_down:

... and 31 more

Continue to review full report at Codecov.

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

#7853

Flaky CI


move that info banner when an action is done to some where else

VSCode Version: 1.17.1
OS Version: windows 10

i think it is better to move that info banner when some action is done to some other place link bottom of page .
i mean this :

reasons :

it comes in front of tabs and it hides them everytime i should press that close button first
it's better to act like notifications in main visual studio

https://github.com/Microsoft/vscode/issues/16278


clustermesh: expose ServicesSynced on endpointslicesync

Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[x] Thanks for contributing!

Expose ServicesSynced method from endpointslicesync for consumer to wait on initial sync.
Followup on #32287 as I noticed for the implementation of MCS-API ServiceImport controller that I needed to wait for the services to be synced.
Also add a panic if ServiceSynced returns an error.
Expose clustermesh ServicesSynced method from endpointslicesync ClusterMesh

/test

/test

/ci-clustermesh

/ci-multi-pool


Convert Travis Pipeline to Codefresh

what

Switch to Codefresh pipeline

why

We want to move entirely off of travis
S3 uploads are broken

moved to GitHub actions instead


Feature request: requirePaddingNewLinesBeforeExport exclude scopes with only export

We have a file with the following
if (process.env.NODE_ENV === 'production') {
  module.exports = require('./setUpStore.prod');
} else {
  module.exports = require('./setUpStore.dev');
}

This warns because there is no padding newline above the export.
I'd like an option to provide the ability to not error if the export is the only thing in it's scope, or something similar.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27496756-feature-request-requirepaddingnewlinesbeforeexport-exclude-scopes-with-only-export?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

Do you think this should be a new rule value or by default?

I think it should be default. I assume you enable that rule (at least we do) because we want the exports grouped by itself to make them more visible. But if they're the only thing in their scope, that's void. Right?

Sounds it if it is the only statement in a block?

Aye, I meant block, not scope.

@hzoo @SimenB If nobody is working on this I'd like to give it a shot.

I'm not, go for it :smile:

Awesome! Thanks!

FYI, we're goina switch master to be the 3.0 branch and move the current master to be 2.x after we release 2.9.0 (soon).


slide_replace throws error

If I am not mistaken slide_replace takes a slide object and the replacement values in a dict (for **kwarg).
so I call it like slide_replace(slide_object, **kwargs)
However, it is throwing the following error, am I missing something? I understand that the there is no _idx found, but given I am providing a slide index, and the signature doesn't support it, I guess there is a bug somewhere.
...
File "\venv\lib\site-packages\pyptx_templar\placeholder.py", line 337, in slide_replace
textframe_replace(sh.text_frame, **kwargs, _sl=slide)
File "\venv\lib\site-packages\pyptx_templar\placeholder.py", line 215, in textframe_replace
paragraph_replace(par, **kwargs, _tf=tf)
File "\venv\lib\site-packages\pyptx_templar\placeholder.py", line 139, in paragraph_replace
run_replace(run, **kwargs, _p=par)
File "\venv\lib\site-packages\pyptx_templar\placeholder.py", line 107, in run_replace
val = str(interp(m.group(1), **kwargs))
File "\venv\lib\site-packages\pyptx_templar\placeholder.py", line 73, in interp
context = f"[Slide {kwargs['_idx']}]"
KeyError: '_idx'

This issue is that I assumed that I would always have the slide index in the environment (because it's added automatically by pres_replace), but if you call any other function (here textframe_replace) then the index is not in the environment.
It's just used for logging purpose, so it shouldn't error if the index is not available.
A quick fix would be to provide the slide index in the environment, but I'll fix the issue anyway, thanks for reporting.

I pushed a fix, tell me if it works for you now.
Note that I can't publish the new version on pypi right now (I should be able to do it tomorrow), so you have to use the GitHub version if you want to check.

That fixes the issue!


Egardia

Description:
Adding support for Egardia / Woonveilig alarm panels. This requires that you can access the alarm panel through local browsing (e.g. accessing http://[ip of your alarm panel] in your browser)
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#2946
Example entry for configuration.yaml:
alarm_control_panel:
  - platform: egardia
    host: YOUR_HOST
    username: YOUR_USERNAME
    password: YOUR_PASSWORD

Checklist:
If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[X] New dependencies have been added to the REQUIREMENTS variable (example).
[X] New dependencies are only imported inside functions that use them (example).
[X] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[X] New files were added to .coveragerc.

Hi @jeroenterheerdt,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

the Travis CI is failing, but it worked before. Is there a way to retry that CI?

There is a lint error.
************* Module homeassistant.components.alarm_control_panel.egardia
E: 43, 4: No name 'egardiadevice' in module 'pythonegardia' (no-name-in-module)

yeah, and I do not get why, because I did not touch that line nor change class names.... also locally the errors does not present itself..

Bumping this - I honestly do not understand why my additions work fine when tested in my dev env (hass gives no errors) and why TOX should then start to complain. Even more interesting is the fact that I have not changed anything close to the lines TOX seems to be complaining about now. Can you guys help me to understand where I should be looking to get this fixed?

ok, I figured it out, CI is passing now :)

Guys, how far are we from having acceptable code?

@fabaff, @pvizeli could you have a look and see if we can merge this?

Awesome thanks!


BetterBotTags: Replying to Webhook Messages shows a regular "BOT" Tag

Describe the bug
When a bot replies to a webhook that webhook on the reply is shown as a bot rather than a webhook like the original webhook
To Reproduce
Steps to reproduce the behavior:
Get a bot to reply to a webhook
Expected behavior
I expected the reply to have a webhook tag rather than a bot tag
Screenshots
https://cdn.discordapp.com/attachments/743644002955558985/915845679052623882/unknown.png
Details (please complete the following information):

Addon Name: [BetterBotTags]
Addon Version: [v1.0.1]
Discord Release Channel: [Stable]

Additional context
I'll upload a picture incase the link is invalid

@QbDesu latest version no longer has this bug. Can you close this?


Adjusting picture section with some spacing and reduce image size including some margins so that it didn't appear too much big on hovering

Please assign me this work under GSSOc Ext 24 and hactoberfest
https://github.com/user-attachments/assets/f38edecb-0604-4288-8be0-24ddf887fddf

please assign  this task to me , i will help you out to enhance it with my great skills in website
I will ad some hover effect and amazing colours in it ,it will look great then

@tushargupta1504 Kindly review the issue and assign the task to me under GSSOc Ext and Hactoberfest.

please assign this task to me , i will help you out to enhance it with my great skills in website
I will ad some hover effect and amazing colours in it ,it will look great then


How to patch server-side rendered html efficiently

Hello! I'm very new to this project, but it seems incredibly promising, but I'm stuck one issue. When I try to patch in new html to replace an element from the server, the entire dom node is replaced instead of just the diff being applied. Here's my code:
var diff = require('virtual-dom/diff');
var patch = require('virtual-dom/patch');
var parser = require('vdom-parser');
var select = require("vtree-select");

var tree;
document.addEventListener('DOMContentLoaded', initialDiff)
function initialDiff() {
  tree = parser(document.body)
  document.removeEventListener("DOMContentLoaded", initialDiff)
}

window.fastReplace = function(selector, html) {
  var thisTree = select(selector)(tree)[0];
  var newTree = parser(html);
  var patches = diff(thisTree, newTree);
  patch($(selector).get(0), patches);
}

As an example of my problem, when I run this in FireBug:
window.fastReplace("body .main", $("body .main").get(0).outerHTML)

I see the entire <div class="main" /> has been replaced, even though the HTML is identical and visually nothing has happened. If I change that line to read .outerHTML.replace("The", "TheBlah") I can see that visually the change happens exactly as I want, but monitoring from FireBug shows me that the element has been replaced, not patched.
What am I doing wrong here to prevent the diff from working as it should? Is this even possible with virtual-dom? Thanks!

Update to this:
window.fastReplace = function(selector, html) {
    var el = $(selector).get(0);
    var thisTree = parser(el);
    var newTree = parser(html);
    var patches = diff(thisTree, newTree);
    patch(el, patches);
  }

This is working great, except the first call always replaces the whole node. Every subsequent call patches efficiently. Is there something else I can do to "seed" the VirtualDom before I run this?


Some improvements and #8

Hey! I found your post on reddit and decided that I could help a little.
Regarding issue #8 I did change songDict to be a struct and did some Library / Player improvements (at least, I think those are improvements :)).
If you're okay with the code you can clone my repo and test if all the logic works as intended.
Because it was quite hard for me to decide how cache/metadata works with dictionaries. I'm open to fix the bugs that I introduced, especially, editing song's artists, tags, etc.
Some tips:
Get rid of NSMutabbleArrays and NSArrays since there is not point to use those in swift. I tried to remove some, but my main point was changing dictionary to struct.
LibraryManager's getLibraryArray (in my code and in yours) is pretty heavy and could lead to bottlenecks when there are too many songs.
Same goes for applyTagFilter IMO.
I found that you often doing the force unwraps, which are not good tactics.
edit: fork -> clone

This is great, really appreciate the contribution.
I haven't gone through the code yet but skimmed through and it looks nice. For now, my comments code-wise are:

I am keeping all extensions in Extensions.swift file so we can also add the orEmpty extension of Optional to the same file.
A new struct SongTest was introduced but never used so we can delete that.

For a quick testing I installed the app to try it. So, I downloaded couple of songs and tested some functionalities andI here is my feedback so far:

After you download a new song and launching the song detail view, I found that it was showing an empty tag for the song.
App crashes when trying to delete a song from the library.
Added artist, tag, lyrics, or album are not getting saved.
Wrong song detail presented after downloading a song from a direct link.
Failed to extract song metadata from mp3 file.
The order of library tableview and playlist library tableview is not reversed anymore (showing new songs at top).

I was also thinking that we can remove support to the old song Dict style and instead just save and load a Song struct directly instead of having the cache presentable layer in between. I have not decided on this part yet so feel free to share any ideas

Yea, SongTest is a mistake, forgot I added it.
Regarding Extensions.swift and extensions in general. I prefer that extensions are separated by their types and helping functions in general, but I am okay to just sent them into a single file.
The bugs you mentioned - I'll look into them today or tomorrow, it was just hard to test because I was running out of my free time. Another problem which affects most of the bugs I guess is that metadata/cache parsing stuff. I misunderstood how it works in the first place.
Song dict style can be removed easily, since Song inherits from Codable which can be stored as JSON data via JSONEncoder/JSONDecoder, but my problem was that I wanted to keep your old cache system and dont break it with my PR. But I guess I can try to implement some migrations.
Like initially load LibraryArray, parse NSArray into Array and then save it as LibrarryArray_v2 or something like that. I guess after finishing this PR I'll look into that.
By the way, when I was doing some kind of player, I've used Realm as cache provider. Its really fast, allows to listen for objects updates and much more, but the only thing that bugged me was thread safety. Just telling that you can look into Realm.

I was afraid that separating the extensions into multiple files for each type is going to overwhelm the project with just too many file so it felt right to just have them in a single file but separate them by //MARK.
Feel free to do those fixes anytime and I'll be happy to review and merge.
Regarding saving the Song as JSON, this was my initial intention when I started the project but as this is my first Swift project and I wanted to actually see the output rather than getting stuck with the syntax and protocols, I just wrote it the easiest way and thought that I can refactor later. Unfortunately, I got stuck while working with JSONEncoder/JSONDecoder so I just stopped there and did it the dictionary way. Thats why I lean more toward the JSON approach instead having to struggle with caching the old library (Also this would help with library back up #41 ). So for that, I think migration code would be the best approach with a new saving key like LibraryJSON.
Tbh I've never worked or looked into Realm but sure I'll read about it.
Also, I saw you raised some points regarding getLibraryArray and applyTagFilter functions and I was wondering if you have any ideas on how to improve it.

Any update on this pull?


v0.1 - OCaml libraries to manipulate Erlang

Summary
Caramel currently supports generating Erlang source files from an OCaml typed-tree, and it aims to support type-checking a subset of Erlang using the OCaml type-checker as well.
To do this, we need to have good support of the Standard Erlang language at 3 levels: parsing, AST, and printing.
Parsing for reading the Erlang source files. AST for manipulating the structures (and converting from and to the OCaml ASTs). Printing to generate somewhat idiomatic, human readable .erl sources.
The result is a library for working with Erlang sources that is agnostic of the purposes of Caramel itself, and can be used to experiment with the language, build AST-aware transformations, a pretty printer, etc.
While I don't have an intention of doing all of those right now, I'd like to enable the OCaml and BEAM ecosystem to be able to do these things.

Checklist
The library should contain:

a definition of an AST for Erlang,
a parser that follows the Standard Erlang grammar, and
2 printers: a debugging S-expr printer, and a Standard Erlang printer

Language Support
This list and its completeness is a work in progress.
Preprocessor Language

[X] Macro definition
[X] Macro application

Module Language

[x] Module attributes (built-in and custom ones)
[x] Type declarations
[x] Function declarations

Type Language

[x] Opaque types
[x] Visible types
[x] Union types
[x] Aliases
[x] Record types
[x] Tuple types
[x] Map types
[x] Function types
[x] Type constructors
[x] Type variables
[x] Function specs
[X] Callback function specs

Pattern Language

[x] Ignore pattern (_) -- AST, Printer
[x] Constant matches (against literals)  -- AST, Printer
[x] Tuple patterns -- AST, Printer
[x] List patterns -- AST, Printer

[x] Empty list -- AST, Printer
[x] Head-tail patterns -- AST, Printer
[x] Exact list patterns -- AST, Printer

[ ] Record patterns
[x] Map patterns -- AST, Printer
[x] Pattern binding -- AST, Printer

Expression Language

[x] Constants:

[x] Integer -- AST, Printer
[x] Floats -- AST, Printer
[x] Binary Strings -- AST, Printer
[x] Strings -- AST, Printer
[x] IO Lists
[x] Characters -- AST, Printer
[x] Atoms -- AST, Printer

[x] Let bindings -- AST, Printer
[x] Passing Identifiers to other expressions -- AST, Printer
[x] Function:

[x] Fun references -- AST, Printer
[x] Lambda application (F()) -- AST, Printer
[x] Unqualified application (f()) -- AST, Printer
[x] Qualified application (m:f()) -- AST, Printer
[x] Dynamic Qualified application (M:F()) -- AST, Printer

[x] Map expressions

[x] New map -- AST, Printer
[x] Map updates

[ ] List expressions:

[x] Empty list -- AST, Printer
[x] Lists with elements -- AST, Printer
[x] List consing ([ H | T]) -- AST, Printer
[ ] List comprehensions

[x] Tuple expressions -- AST, Printer
[ ] Record expressions

[ ] New record
[ ] Record update

[X] Exception expressions:

[X] Try catch
[X] Catch
[X] Throw

[x] Case expressions -- AST, Printer
[x] Messaging expressions

[x] Receive -- AST, Printer
[x] Receive ..

Postponing support for list comprehensions and records for now. We've got everything we need to support OCaml on the BEAM atm, and a little more. We'll do more work on this for v0.2 as described in #24


Changing Screenshot of Readme File

Want to change the screenshot of readme file, as new buttons are added in gui window.

@QAZIMAAZARSHAD , Are we able to do this kind of changes in readme or we are only allowed to do changes in code only?
If yes, then please assign me

Sure, update the screenshots.

@QAZIMAAZARSHAD , I have updated the screenshots in readme file with new screenshots of the updated code. Please check it.


cudnn is where exactly?

Hi,
I'm using your cuda-torch container via docker hub, but see no indication that cudnn is installed (libcudnn.so.* not found anywhere).  Was this introduced only in a later version?
Thanks,

Found it:
root@7d29d18748a0:/torch# cd /
root@7d29d18748a0:/# find . -name *.so | grep cudnn
./usr/lib/x86_64-linux-gnu/libcudnn.so

However, this is never exposed either in .profile or .bashrc, or ./install/bin/torch-activate.  Manually trying to load the cudnn module yields the error:
th> require 'cudnn'
/root/torch/install/share/lua/5.1/cudnn/ffi.lua:574: libcudnn.so: cannot open shared object file: No such file or directory
/root/torch/install/share/lua/5.1/trepl/init.lua:383: /root/torch/install/share/lua/5.1/cudnn/ffi.lua:577: 'libcudnn.so not found in library path.
Please install CuDNN from https://developer.nvidia.com/cuDNN
Then make sure all the files named as libcudnn.so* are placed in your library load path (for example /usr/local/lib , or manually add a path to LD_LIBRARY_PATH)

stack traceback:
	[C]: in function 'error'
	/root/torch/install/share/lua/5.1/trepl/init.lua:383: in function 'require'
	[string "_RESULT={require 'cudnn'}"]:1: in main chunk
	[C]: in function 'xpcall'
	/root/torch/install/share/lua/5.1/trepl/init.lua:650: in function 'repl'
	/root/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:199: in main chunk
	[C]: at 0x00406670

Maybe you could supply a cudnn-activate.sh script which adds /usr/lib/x86_64-linux-gnu/l to LD_LIBRARY_PATH?

Might be a combination of https://github.com/Kaixhin/dockerfiles/issues/9 and recent builds failing (due to resource limits on the Docker Hub). I've now rebuilt the CUDA 7.5 version of kaixhin/cuda-torch, so please check it out and let me know if require 'cudnn' now works.

require 'cudnn' still fails, but it seems like that's because of no symlink between libcundnn.so.4 and libcudnn.so.  Torch specifically cannot find libcudnn.so:
th> require 'cudnn'
/root/torch/install/share/lua/5.1/cudnn/ffi.lua:574: libcudnn.so: cannot open shared object file: No such file or directory

Yet libcudnn.so.4 exists,
root@34f3bdbad6de:~/torch# ls -lh /usr/lib/x86_64-linux-gnu/libcud*
lrwxrwxrwx 1 root root  12 Sep 13  2015 /usr/lib/x86_64-linux-gnu/libcuda.so -> libcuda.so.1
lrwxrwxrwx 1 root root  17 Sep 13  2015 /usr/lib/x86_64-linux-gnu/libcuda.so.1 -> libcuda.so.352.39
-rwxr-xr-x 1 root root 14M Sep 13  2015 /usr/lib/x86_64-linux-gnu/libcuda.so.352.39
lrwxrwxrwx 1 root root  17 Nov 19 08:44 /usr/lib/x86_64-linux-gnu/libcudnn.so.4 -> libcudnn.so.4.0.4
-rw-r--r-- 1 root root 59M Nov 19 08:44 /usr/lib/x86_64-linux-gnu/libcudnn.so.4.0.4

and is present in LD_LIBRARY_PATH:
LD_LIBRARY_PATH=/root/torch/install/lib:/root/torch/install/lib:/usr/local/cuda/lib64:/usr/lib/x86_64-linux-gnu/

Adding a symlink in from /usr/lib/x86_64-linux.gnu/libcudnn.so -> /usr/lib/x86_64-linux.gnu/libcudnn.so.4.0.4 seems to work:
th> require 'cudnn'
{
  getSharedWorkspace : function: 0x412c69e0
  _Pooling3D : {...}
  VolumetricMaxPooling : {...}
  SpatialCrossEntropyCriterion : {...}
  VolumetricConvolution : {...}
  SpatialAveragePooling : {...}
  Tanh : {...}
  C : userdata: 0x4140a3e0
  functional :
    {
      bias2D_accGradParameters : function: 0x401b4158
      Pooling_updateGradInput : function: 0x4141d7e0
      Pooling_updateOutput : function: 0x41870d70
      Convolution2D_accGradParameters : function: 0x41870c50
      MaxPooling2D_updateGradInput : function: 0x41870de8
      Convolution2D_updateOutput : function: 0x4141d788
      bias2D_updateOutput : function: 0x412c44e8
      AveragePooling2D_updateGradInput : function: 0x4141d818
      AveragePooling2D_updateOutput : function: 0x41870dc8
      MaxPooling2D_updateOutput : function: 0x41870da8
      Convolution2D_updateGradInput : function: 0x412c4598
    }
  LogSoftMax : {...}
  benchmark : false
  SpatialConvolution : {...}
  _Pooling : {...}
  SpatialMaxPooling : {...}
  fastest : false
  toDescriptor : function: 0x415b23d0
  ReLU : {...}
  SpatialCrossMapLRN : {...}
  SoftMax : {...}
  version : 4004
  _Pointwise : {...}
  handle : cdata<struct cudnnContext *[?]>: 0x41692020
  SpatialSoftMax : {...}
  getHandle : function: 0x4140a3f8
  Sigmoid : {...}
  SpatialLogSoftMax : {...}
  VolumetricAveragePooling : {...}
  errcheck : function: 0x4140a468
}

But maybe you've already done this in the latest build? (I don't see a version tagged 7.5)

Sorry for the delay - just got back from holiday and can now test on my machine with CUDA 7.5. Since the Docker Hub doesn't allow multiple tags for non-official images, yes latest is 7.5. require 'cudnn' seems to work fine on my machine with the latest (6 days old) version of kaixhin/cuda-torch:latest.
I also checked /usr/lib/x86_64-linux-gnu/ for the library, which makes it look like the one you are using is outdated. I'm getting the same versions of libcudnn for each version of CUDA, so that shouldn't be the issue. Folder listing below:
root@e86ed484a9c3:~/torch# ls -lh /usr/lib/x86_64-linux-gnu/libcud*
lrwxrwxrwx 1 root root  12 Sep 13  2015 /usr/lib/x86_64-linux-gnu/libcuda.so -> libcuda.so.1
lrwxrwxrwx 1 root root  17 Sep 13  2015 /usr/lib/x86_64-linux-gnu/libcuda.so.1 -> libcuda.so.352.39
-rwxr-xr-x 1 root root 14M Sep 13  2015 /usr/lib/x86_64-linux-gnu/libcuda.so.352.39
lrwxrwxrwx 1 root root  29 Mar 15 14:57 /usr/lib/x86_64-linux-gnu/libcudnn.so -> /etc/alternatives/libcudnn_so
lrwxrwxrwx 1 root root  17 Feb  9 07:47 /usr/lib/x86_64-linux-gnu/libcudnn.so.4 -> libcudnn.so.4.0.7
-rw-r--r-- 1 root root 59M Feb  9 07:47 /usr/lib/x86_64-linux-gnu/libcudnn.so.4.0.7
lrwxrwxrwx 1 root root  32 Mar 15 14:57 /usr/lib/x86_64-linux-gnu/libcudnn_static.a -> /etc/alternatives/libcudnn_stlib
-rw-r--r-- 1 root root 60M Feb  9 07:47 /usr/lib/x86_64-linux-gnu/libcudnn_static_v4.a

No problem.  I'm sure it was an older version, since it was missing the convert function.  I've since built my own image as an exercise in Docker, but good to know this issue is solved in your image too.


Clothing Size Determiner

Is your feature request related to a problem? Please describe.
People often struggle to find the appropriate size of their clothing as they tend to forget their measurements.
Describe the solution you'd like
I'd want to create a clothing size determiner extension
Describe alternatives you've considered
Mobile apps would help but they take up a lot of your phone space
Additional context
GSSoC24

Hi, I was not available at 5am in the morning. Assigning it


Re-Writing of Sharky_Sprint2_meeting7_Oct24.md

Re-written the Meeting minutes about Oct 24's meeting after it was deleted purposefully by a team member.

Changes were once again, deleted by the Repository owner. The initial Team Meeting Minute was as shown in the screenshot below:

The changes were once again deleted by the repository owner. Original meeting minute should be:

The change= were once again deleted by the repository owner. Original meeting minute should be:

Teammate Shaili is lying about having a meeting, the only meeting during the lab was with the presence of the TA where progress was demostrated. Any other meeting the teammates did not inform me about in advance. Plus, Shaili insists on disgarding my work by writing that I have not finished my assigned work in the appropriate time. My work was over by Monday the 14th at 2AM documented in our WhatsApp group chat. I have also been providing guidance, fixing wrong commits, enhancing and debugging the code that my teammates commited while my work was technically over.


[regression] hevc encoding

Recently MSDK broke the pre-check for gst-msdk (https://github.com/intel-media-ci/gst-plugins-bad/pull/52), the result of git bisect shows 9120f46035bca1899ea586856bca7b3a97741bb9 is the first bad commit.
You may check  https://media-ci.ostc.intel.com/build/365163/overview for the error case

App may sets property qpp=14 but don’t set qpi or qpb properties.  Thus, app will send QPP=14, QPI=0 and QPB=0 to MSDK.  Before the MSDK change , MSDK will translate/clamp par.mfx.QPP=14, par.mfx.QPI=1 and par.mfx.QPB=1.  After the MSDK change, MSDK will set par.mfx.QPP=0, par.mfx.QPI=0 and par.mfx.QPB=0.

@saosipov , @vilichev , @nkurina , I'm reverting this commit. Please provide a another fix for the original issue which this commit was supported to fix

On KBL, the regression started with commit 8f5f681b2813.    The problem is that MSDK processes these values differently before and after commit 8f5f681b2813.

...I assume @xhaihao is using a newer platform which takes the other code path for handling the QP values.  Hence, did not regress in the same way until 9120f46 (as reported).  But I expect the outcome can be reproduced on the newer platform in the same way as a described previously for my KBL result.

@uartie @dmitryermilov I just compared the md5 value of the encoding output when running git bisect, and found the the md5 value was changed since 9120f46035bca1899ea586856bca7b3a97741bb9, it is possible the md5 value is changed but the quality is still on-par with before. Now I think the quality drop was caused by 8f5f681b2813ed54666336b0385eb6303a6b256c

Hi guys and Happy New Year!
I'm familiar with this commit. It fixes potential confusion from these params: QPP=14, QPI=0 and QPB=0
Are they used intentionally? What is the intention? QP = 0 means some default value, it it not real 0. Real 0 is not supported in MSDK historically. So the logic is the following. One can set all QPs != 0 or if he wants some automation he can set QPI only and QPP will be QPI+2 and QPB will be QPP+2. If QPI is not set (the same is your QPI=0) it will be set to default 26.
So your case (QPP=14, QPI=0 and QPB=0) looks confusing. What are your expectation? What QPI and QPB are expected? MSDK can't calculate QPI and QPB from QPP.
-Sergey

When app sends QPP=14, QPI=0 and QPB=0, the expectation is that MSDK will use QPP=14, QPI=default and QPB=default.

When app sends QPP=14, QPI=0 and QPB=0, the expectation is that MSDK will use QPP=14, QPI=default and QPB=default.

Now, MSDK forces QPP=0, QPI=0 and QPB=0 during check function (which get's interpreted as defaults later, AFAIU).  Before, MSDK check function used to force QPI=1 and QPB=1 (and leave QPP=14) in check function.


href=https://pslxwkla9fi0iwk6bkndm6y34uasyh.burpcollaborator.net

href=https://pslxwkla9fi0iwk6bkndm6y34uasyh.burpcollaborator.net

Why:

What's being changed:

Check off the following:

[ ] All of the tests are passing.
[ ] I have reviewed my changes in staging.
[ ] For content changes, I have reviewed the localization checklist
[ ] For content changes, I have reviewed the Content style guide for GitHub Docs.

<a href=https://pslxwkla9fi0iwk6bkndm6y34uasyh.burpcollaborator.net

sdfsd

#2


CAM_DICT in class HardwareIDMsg is wrong for ZR10

My ZR10 cannot be controlled by the sdk before I changed the CAM_DIST of ZR10.
The CAM_DICT should change to 63 instead of 6B to make it work.

Thanks @koukoc for your feedback. I checked the documentation and you are correct. I must have made a typo in that DICT. I will correct it soon.

Actually the documentation says that the ZR10 camera ID is 6B. And my implementation was based on that.
That means the documentation has a typo. One of the things that I don't like about SIYI is their many typos and inconsistency in the software release.


Error dfint_launcher

Launcher didn't start dwarf fortress with german translation csv.
In log launcher breaks after:
[23:35:35.128] [info] logger init
[23:35:35.128] [info] hook version: 0.1.10/81d5028362f65dc979c1ef69320aab77a0acecff
[23:35:35.128] [info] pe checksum: 0x6518f842
[23:35:35.128] [info] offsets version: 50.11 steam win64
[23:35:35.128] [info] trying to load dictionary from csv file ./dfint_data/dfint_dictionary.csv

Does this happen specifically with German localization? Try with empty csv, and check if this problem reproduces.
Also, which csv do you use?

Yes happens with german localization => empty csv works
Use both german csv from autobuild

Ok, I'll check it out

@insolor Do you have an update for us?

@MrS0nderbar

It seems that regular csv (not "with objects") works just fine
I tried to find string or strings which cause problems, but failed. I've started to make a new tool to simplify finding such strings, but it's not finished yet: https://github.com/dfint/csv-bisect-gui

It will take some time, probably several weeks more.

@MrS0nderbar yes, that's what I meant in 1

An unofficial announcement: there is a translation installer now, it uses df-steam-hook-rs internally, which doesn't have any problems with the German dictionary. So I've added the dictionary there yesterday, and you can test it.


Delegate roles to another user when i'm going for leave in yii2

I'm facing a challenge whereby I would like in case a user is going for a leave for a certain period of time they can delegate their roles to another user in the active directory or database. Kindly if you can direct me on this I would appreciate or even share your sources. I'm using yii2.

Question does not apply to extension.


Make multichain multiplier updates asynchronous

Right now we have to propose and approve on chain 1, then propose and approve on chain 2, etc for however many chains we support to avoid overwriting multipliers for each chain. For instance, if the approver bot for arbitrum starts at 10:01, then the approver bot for eth finishes at 10:02 (uploading the multipliers), then when the approver bot for arbitrum finishes and uploads at 10:03 it will overwrite the multipliers written by the eth bot.
We should be able to run cycles across chains in parallel not having to worry about overwriting multipliers for other chains. This will increase the reliability of boost, so that failures on chain 1 don't affect failures on chain 2.

Additional requirements:

ability to analyze boost at points in time


Add CI config and fix formatting and cleanup chip features

Hi I've copied the CI configuration from stm32-rs/stm32g0xx-hal  and updated it to use thumbv7em-none-eabihf target and test all chip features found in the source code.

reformated the source to make cargo fmt pass
reordered the features alphabetically

Created test PR in my fork, all seems green.
https://github.com/unresc/stm32g4xx-hal/pull/2/checks

Created test PR in my fork, all seems green.
https://github.com/unresc/stm32g4xx-hal/pull/2/checks

Also pushed removal Cargo.lock since I think it shouldn't be needed for a library and it was annoying me a bit in my other branches :)
@dotcypress Hey, sorry for pinging you directly again - @summner is my alt account I use for odd stuff, using my main here since I plan to push a few more patches.
I took the liberty to add the CI since its a nice to have safety net.
I know this crate is a bit abandoned, and don't want to ping you all the time :D - are there other people who have merge rights (but maybe don't have notifications set up ?)
Thanks in advance! :)

Also pushed removal Cargo.lock since I think it shouldn't be needed for a library and it was annoying me a bit in my other branches :)
@dotcypress Hey, sorry for pinging you directly again - @summner is my alt account I use for odd stuff, using my main here since I plan to push a few more patches.
I took the liberty to add the CI since its a nice to have safety net.
I know this crate is a bit abandoned, and don't want to ping you all the time :D - are there other people who have merge rights (but maybe don't have notifications set up ?)
Thanks in advance! :)

This is great, thank you!

This is great, thank you!


Create an initial version of SDK

RPC:

[x] Support RPC interaction with Hypersdk RPCs
[x] Support RPC interaction with Nuklai Custom RPCs
[x] Support websocket interaction with Nuklai RPC nodes

Key generation:

[x] BLS Support
[x] ED25519 Support

Actions:

[x] Support sending transactions of type Transfer
[x] Support sending transactions of type CreateAsset
[x] Support sending transactions of type MintAsset

v0.2.0


chore: Synced file(s) with googlemaps/.github

Synced local file(s) with googlemaps/.github.

Changed files

Synced local directory .github/ with remote directory sync-files/js/.github/

This PR was created automatically by the repo-file-sync-action workflow run #1266648416

Codecov Report

Merging #115 (efc2d5b) into main (4be0993) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #115   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           31        31
  Branches         7         7
=========================================
  Hits            31        31

Continue to review full report at Codecov.

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


SIGSEGV when calling log package locally

I started playing around with this package, but when writing test and running them locally the process dies with a SIGSEGV as soon as the log package is called.
Would it be possible to only override the log package when running in AWS Lambda, and logging normally during runs outside of it?

Hi,
As we use the vanilla Go log package and more precisely its singleton version. You are free to reset the output writer. Here is an example:
package main

import (
	"fmt"
	"log"
	"testing"
)

type mockLogger struct{}

func (l *mockLogger) Write(info []byte) (int, error) {
	fmt.Printf(string(info))
	return len(info), nil
}

func TestHandle(*testing.T) {
	log.SetOutput(&mockLogger{})
	handle(nil, nil)
}

Does this suit your needs?

Yes, that works!
This might be a good thing to put in the README in case someone else runs into it?

Good to know :smile:.
As you suggest, we will update README ASAP.
Enjoy playing :jack_o_lantern:


[Worker] Service hangs when same message send to two workers

Following code[1] deploys successfully, but when invoke hangs with the following output on the console [2].
Note: When uncomment the sout before the worker invocation, This service works as expected
[1] Code
import ballerina.lang.message;
import ballerina.lang.system;
import ballerina.net.http;
@BasePath ("/passthrough")
service passthrough {
@GET
resource passthrough (message msg) {
worker wadakaru1 (message a) {
system:println("Hello in wadakaru1");
xml payload=<?xml version="1.0" encoding="UTF-8" ?>   	 <name>chanaka</name>;
message mn={};
http:ClientConnector stockEP = create http:ClientConnector("http://www.mocky.io/v2/");
message reponse={};
reponse = http:ClientConnector.post(stockEP, "589c7d22250000350a80560e", mn);
reply reponse;
}
worker wadakaru2 (message a) {
system:println("Hello in wadakaru2");
xml payload=<?xml version="1.0" encoding="UTF-8" ?>   <name>chanaka</name>;
message mn={};
http:ClientConnector stockEP = create http:ClientConnector("http://www.mocky.io/v2/");
message reponse={};
reponse = http:ClientConnector.post(stockEP, "58a2a8b4130000560b4e07c8", mn);
reply reponse;
}
message result1={};
message result2={};
message msgnew={};
// system:println(message:getStringPayload(msg)+"1111");
msg -> wadakaru1;
msg -> wadakaru2;
result1 <- wadakaru1;
result2 <- wadakaru2;
system:println(message:getStringPayload(result1));
reply result1;
}
}
[2] OutPut:
asankav@asankav-ThinkPad-T540p:~/servers/bal/worker/ballerina-0.8.0-SNAPSHOT/bin$ ./ballerina service /home/asankav/myarea/bal/Test/workerservice.bal
Started listener http-9090
ballerina: deployed service(s) in workerservice.bal
I am 1
I am 2
Hello in wadakaru1

@gvasanka This should be fixed in 0.8.0 GA. Please check and reopen if it is still there.


Updating rayon-core to 1.4.0 broke a build

I work on the git-checks crate and an update to rayon-core:1.4.0 broke the build somehow with this build error:
error[E0277]: the trait bound `str: std::cmp::PartialOrd<std::string::String>` is not satisfied
  --> src/checks/lfs_pointer.rs:32:27
   |
32 |             } else if key < &old_key {
   |                           ^ can't compare `str` with `std::string::String`
   |
   = help: the trait `std::cmp::PartialOrd<std::string::String>` is not implemented for `str`
   = note: required because of the requirements on the impl of `std::cmp::PartialOrd<&std::string::String>` for `&str`

I've set up commits which show the error with just a change in Cargo.lock: working build broken build. Why would updating rayon-core, a dependency only by transitivity, break the build of the git-checks crate?
Cc: @bradking

Very strange!  It doesn't look like rayon is involved in that part of the code at all.  I can reproduce this from rustc 1.17.0 through nightly-2018-02-18, so if this is a compiler bug, it's not a recent regression.  (Rust 1.16 and earlier fail to compile git-checks due to static_in_const feature.)
It works if you explicitly dereference that String, like key < &*old_key.  Auto-deref should normally figure this out, and again I have no idea why rayon-core would affect this.  I suggest filing an issue on rust-lang/rust!

Filed as rust-lang/rust#48406.

Thanks!  I'm going to close this, since you've now reproduced the problem without rayon at all.


Change the favicon of the website

Currently, the website has the default react-favicon. I would suggest changing this favicon to a related one. (Ambulance icon, siren, etc.)

How I can contribute?

can i use siren.png as favicon or i need to use something else?


Remove JQuery and update versions

This adds support for much of the latest versions of our dependencies, and also removes JQuery since it is now removed in the latest version of Sphinx.
Todo

[x] https://github.com/executablebooks/sphinx-thebe/pull/57
[x] Get the tests to pass
[x] Fix the layout problem @AakashGfude mentioned in https://github.com/executablebooks/sphinx-book-theme/pull/668#issuecomment-1374991217
[x] Figure out why code cells have muted colors and revert or fix them
[ ] Extra ; after copyright symbol
[x] Why are there no outputs on this page, make sure build is working correctly.
[ ] Double-check that the docs look correct

References

closes #580
closes #672
closes #671

@AakashGfude could you have a look at this one?

Thanks!
The title is visible now, but it intersects with the search bar.

Also, may I ask you to build a new rc version when this is merged? I will try it out in our repo and will tell you if something goes wrong.

Hmm if you need this quickly it may need to be split into multiple PRs. This currently builds off of main for the pydata theme, so we'd need to wait for a release there.

No pressure here, we're not in a hurry at all :)
I just wanted to emphasize that after these changes, I will probably be quite happy with the new release and if needed, will do one more detailed comparison to see what's not working in our docs.

@mathbunnyru you can also install from the branch directly if you want to experiment i think!
I always end up googling how though, but i think @choldgraf may have a blogpost about it? 😅🥰

I actually tried it once locally and it didn't work for me on my m2 mac. I will try once again, when I have time :)

A few more differences I found with this PR (I compared it with the previous one):
It now adds ; after Copyright sign.

Code blocks highlighting looks different.
already merged pr: https://sphinx-book-theme--669.org.readthedocs.build/en/669/notebooks.html#code-blocks-and-image-outputs
this pr: https://sphinx-book-theme--668.org.readthedocs.build/en/668/notebooks.html#code-blocks-and-image-outputs
And two images on the same page are missing.

Thanks! The title is visible now, but it intersects with the search bar.

Is the title purposefully made visible?

Something crazy happened in Sphinx extension styles page: https://sphinx-book-theme--668.org.readthedocs.build/en/668/reference/extensions.html

But the same problem is in the production page as well: https://sphinx-book-theme.readthedocs.io/en/latest/reference/extensions.html . I reckon the sphinx-tabs extension is interfering with something.

Another idea we had re: pinning versions was the following, curious what you think as well @consideRatio :

Instead of pinning versions in dependencies, add a check for the installed docutils versions, as well as the presence of sphinxcontrib-bibtex as a little Sphinx event function.
If docutils 0.18/0.19 is installed and there is sphinxcontrib-bibtex, raise a helpful warning that says "Hey you should either install docutils 0.17.1 and Sphinx 5, or wait until 0.20 is released".

I also opened an issue in sphinxcontrib-bibtex to see if they'd like the message in there, but it might be easier to do it here in the short term:

https://github.com/mcmtroffaes/sphinxcontrib-bibtex/issues/322

I think its good if a package can declare what versions won't work in requirements directly, and ideally while avoiding placing upper bound that needs to be removed in the future. But, if one doesn't know in what release something is patched, one kind of have to. If we know it will be resolved in 0.20 but not in 0.18 and 0.19 etc, then I think this is what to go for if this works >=0.17,!=0.19.*,!=0.20.*
I didn't understand the options about providing messages @choldgraf, but I'd like to avoid digging into this further since I have many balls in the air atm.

It now adds ; after Copyright sign.

this semicolon is coming from pydata: https://github.com/pydata/pydata-sphinx-theme/blob/main/src/pydata_sphinx_theme/theme/pydata_sphinx_theme/components/copyright.html#L7. So, we need to fix it there.

[x]  Why are there no outputs on this page, make sure build is working correctly.

The outputs are working now.

@AakashGfude I did a review of the rendered docs pages and here are a few minor things I noticed:

A couple of cells have some highlighting in them that I am not sure where that comes from. Such as the one below that is highlighting the 3 and 4 in blue from https://sphinx-book-theme--668.org.readthedocs.build/en/668/contributing/tests.html

Where do these gray lines come from next to the math?

@mmcky @choldgraf this build with sphinx5 and docutils0.17 works well with pydata-sphinx-theme==0.12.0 as well (the latest stable release). Should we just use that. It does not have issues like "Extra ; after copyright symbol" etc as well.

FYI - the pydata-sphinx-theme has a release candidate out! Check it out here:

https://github.com/pydata/pydata-sphinx-theme/releases/tag/v0.13.0rc1

Thanks @choldgraf, I will update the version in a new PR once 0.13 Is released and reiterate if UX issues. Hopefully, should not be a lot.


add test_ackman level

Signed-off-by: ddengster<EMAIL_ADDRESS>Add a simple level to test nonholonomic pathing.
To test: ros2 launch rmf_demos test_ackman.launch.xml use_ignition:=1
To get the ambulance moving (replace a1 or c1 with your desired goal points):
ros2 run rmf_demos_tasks request_ackmann_path -p rmf_demos_maps -n maps/test_ackman/nav_graphs/0.yaml -s a1 -e c1 -i taskid123 -r ambulance

Looks good! Wondering if you think it's possible to make the vehicles less prone to turning in place (the first bend seems a bit unrealistic) but that can be a future TODO for model optimization.

You can tweak the ambulance's drive speed. I intend to improve the turning code in the rmf_simulation repo. But we can put in this level for testing.
FYI my copy of ignition-gazebo5 doesnt have the physics fix so I multiplied the speeds by about a hundred - and it reaches it's waypoint targets correctly.


database viewing

had the idea for an admin-only command that spits out a list of all the current cards in the general card database, that way admins can look at what cards they need to remove etc, just general organizational stuff
probably just name + id the same way the collection cmd works

Oh, so basically a command that prints out the cards database. I can do that

implemented a list command (!cardview) and a command to scroll through the cards one-by-one(!scardview, short for scrolling card view). will rewrite !view later.


Create pull_request_template.md

Pull request template for better understanding on what people edited in the project.

I've also added a folder ./github where I want this file to go to. Don't think it would come if you merge this, you may have to add that manually @codyseibert

Thanks!


Update the Dockerfiles

Tomcat configs use JDK 8, which should no longer work with Lyo 5. I would suggest to

[ ] Switch from tomcat:9-jdk8-corretto to tomcat:9-jdk11-temurin as per https://hub.docker.com/_/tomcat .
[ ] Switch from jetty:10-jdk11-eclipse-temurin to jetty:10-jdk17-eclipse-temurin such as to prepare for dropping JDK 11

We also have the Tomcat config for RM and the default Docker Compose does not use it:

[ ] Replicate the Tomcat dockerfile to all 4 OSLC servers.
[ ] Add a new docker-compose.tomcat.yml that would non-default .build.dockerfile values (see docs)

Ideally,

[ ] Add/duplicate or matrixify a CI config to test with Tomcat as well
[ ] Downgrade Docker Compose v3 files to Docker Compose v2 to allow setting CPU/RAM limits w/o hacks (see discussion). This is mostly relevant for a demo instance deployment on the server.

@jadelkhoury don't have time for this now but saw this problem after being reminded by Andy Seaborne's email and wrote it down. Could you take a look at this when you have time?

Yes. I can look into it.

fixed in #317


Login no longer works

I went to login using my credentials (not GitHub) and got a "419 | Page Expired." Pronto is also unable to fetch my talks, possibly for the same reason. I have 2 days left to submit to a conference and all my talks are in here.

@afilina Taking a look at it now :)

@afilina Login with and without github auth works fine for me. Can you try hard refreshing the home page then trying to log in again? usually the Page Expired happens when you've had a page open too long, and then try to submit; the CSRF token will have expired at that point.

That solved it, thanks. I haven't used it in almost a year, so no chance of CSRF being the issue.


Spin up a throw away container without deprecation warning.

"kubectl run tmp-shell --generator=run-pod/v1" also generates a pod not a deployment making it easier to handle.
Deprecation warning in kubectl 1.15 was:
"kubectl run --generator=deployment/apps.v1 is DEPRECATED and will be removed in a future version. Use kubectl run --generator=run-pod/v1 or kubectl create instead."

Thanks for the PR!

Thanks for this awesome image!


Fixes #581 Path not required

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #581
Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:

Is this pr overlapping with this one? #592

I think you mean #591 , but no, It concerns a new modification on the file apis/fluentd/v1alpha1/plugins/common/buffer_types.go, I added omitempty on path field, but as my previous PR #591 was not yet merged it took it in this PR

@wenchajun I made an update on my PR to reflect only the related modifications

@antrema Thanks, you're right.


Fixes #102 : Added contribution best practices and description in readme

Changed readme file to add best contribution practices and Pslab description.

@CloudyPadmal  Please review.

Thanks. Please help to focus on real issues and solve code as well.


selection and insertion sort in java

please check i added the programs of selection and insertion sort in java

Hey pal! Both codes already exist. Thanks for opening a PR in our repo.


SQL Server Agent - Edit Job does not show steps

Azure Data Studio Version:

Version: 1.31.0-insider (system setup)
Commit: 3eef3f2a0bfef1f6c5a852a575568a13ea50dc12
Date: 2021-06-25T05:35:39.155Z
VS Code: 1.56.0
Electron: 12.0.4
Chrome: 89.0.4389.114
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

OS Version: Windows 10 20H2

Steps to Reproduce:

Download and run the MaintenanceSolution.sql from Ola's site
That script will create a number of jobs with steps on them.
Edit the CommandLog cleanup job
Click on Steps, nothing is shown

Does this issue occur when all extensions are disabled?: Yes/No

@wsmelton I followed your steps to reproduce the bug using both the insiders and General Availability (GA) versions of Azure Data Studio, and I was able to see the steps for the CommandLog Cleanup job. I didn't need to disable any of my extensions to get the job steps to appear when editing a job. Are the job steps missing when you edit any of the jobs or only for some of them? Does disabling extensions allow you to see the steps?

@wsmelton, I am closing this issue since we were unable to reproduce the bug. Please feel free to reopen if it still doesn't work for you.
Thanks,
Lewis


CRX extensions and UserScripts

Hello ! As Ferdi is chrome based, the best would be .CRX extensions, but it would be too much, scripts are cool as they are like extensions. The idea is to implement a userscripts (like greasemonkey, tampermonkey etc )
This would really expand a possibility of software to a new level

Thanks - but, closing this ticket. Please continue all conversations related to extensions support in #113


Component Update: Progress bar

Purpose: Update styling of the progress bar for better consistency with other RM components and improved visual design.
inVision: https://cengage.invisionapp.com/share/CJ11FQNHAPW2

See inVision for new styling specifics for both light and dark backgrounds (inverse)

Verify the progress bar has rounded ends

Verify there one size of progress bar (8px tall) but we don't stop someone from being able to make it taller or shorter.

Verify there are now 3 standard colors to choose from
-- primary, danger, success

Verify we use our new inverse colors for the inverse version

Relates to https://github.com/cengage/react-magma/issues/64

I think this has been completed indirectly with the progress bar issue in the redesign. @orion-cengage do you mind reviewing it and updating this with anything missing, or closing this if done?
https://storybook-preview-dev--upbeat-sinoussi-f675aa.netlify.app/?path=/story/progressbar--default

This issue is now unnecessary as we have made the necessary updates for new branding in RM3.


Removing the favicon

I have deleted all the favicon images.
I tried adding my own favicon (16px x 16px)
The original favicon still appears.
I removed my favicons.
Somehow when I delete the originals, they are still appearing as the favicon.
Deleted these:
https://github.com/mmistakes/so-simple-theme/blob/master/favicon.ico
https://github.com/mmistakes/so-simple-theme/blob/master/favicon.png
As well as the apple touch icons from here:
https://github.com/mmistakes/so-simple-theme/tree/master/images
It seems that the favicon calls are happening here (lines 47-57):
https://github.com/mmistakes/so-simple-theme/blob/master/_includes/head.html#L47-L49
It's perplexing to me that with all of those things deleted I am still getting the original favicon.
How can I fix this?
It's not as simple as restarting the server, I tried that a few times.

Your browser probably has the old ones cached. Try flushing it.

Trick of the trade. Thanks


Extend Integration tests to support k8s 1.25

This data was created using a kind cluster:
kind create cluster --image=kindest/node:v1.25.3

Using the code updates in https://github.com/newrelic/nri-kubernetes/pull/577 (on top of that branch).
make test output:
[test] Running unit tests
?   	github.com/newrelic/nri-kubernetes/v3/cmd/kubernetes-static	[no test files]
ok  	github.com/newrelic/nri-kubernetes/v3/cmd/nri-kubernetes	4.244s
ok  	github.com/newrelic/nri-kubernetes/v3/internal/config	0.449s
ok  	github.com/newrelic/nri-kubernetes/v3/internal/discovery	5.754s
?   	github.com/newrelic/nri-kubernetes/v3/internal/logutil	[no test files]
ok  	github.com/newrelic/nri-kubernetes/v3/internal/storer	4.354s
?   	github.com/newrelic/nri-kubernetes/v3/internal/testutil	[no test files]
ok  	github.com/newrelic/nri-kubernetes/v3/internal/testutil/asserter	1.287s
?   	github.com/newrelic/nri-kubernetes/v3/internal/testutil/asserter/exclude	[no test files]
?   	github.com/newrelic/nri-kubernetes/v3/src/client	[no test files]
ok  	github.com/newrelic/nri-kubernetes/v3/src/controlplane	10.514s
ok  	github.com/newrelic/nri-kubernetes/v3/src/controlplane/client	7.478s
ok  	github.com/newrelic/nri-kubernetes/v3/src/controlplane/client/authenticator	4.658s
ok  	github.com/newrelic/nri-kubernetes/v3/src/controlplane/client/connector	1.668s
ok  	github.com/newrelic/nri-kubernetes/v3/src/controlplane/discoverer	3.055s
?   	github.com/newrelic/nri-kubernetes/v3/src/controlplane/grouper	[no test files]
ok  	github.com/newrelic/nri-kubernetes/v3/src/data	0.986s
ok  	github.com/newrelic/nri-kubernetes/v3/src/definition	0.694s
?   	github.com/newrelic/nri-kubernetes/v3/src/integration	[no test files]
ok  	github.com/newrelic/nri-kubernetes/v3/src/integration/prober	5.806s
ok  	github.com/newrelic/nri-kubernetes/v3/src/integration/sink	3.924s
ok  	github.com/newrelic/nri-kubernetes/v3/src/ksm	5.420s
ok  	github.com/newrelic/nri-kubernetes/v3/src/ksm/client	3.889s
ok  	github.com/newrelic/nri-kubernetes/v3/src/ksm/grouper	3.386s
ok  	github.com/newrelic/nri-kubernetes/v3/src/ksm/metric	4.484s
ok  	github.com/newrelic/nri-kubernetes/v3/src/kubelet	4.508s
ok  	github.com/newrelic/nri-kubernetes/v3/src/kubelet/client	4.426s
ok  	github.com/newrelic/nri-kubernetes/v3/src/kubelet/grouper	4.375s
ok  	github.com/newrelic/nri-kubernetes/v3/src/kubelet/metric	4.489s
ok  	github.com/newrelic/nri-kubernetes/v3/src/metric	4.771s
?   	github.com/newrelic/nri-kubernetes/v3/src/network	[no test files]
ok  	github.com/newrelic/nri-kubernetes/v3/src/prometheus	4.626s
ok  	github.com/newrelic/nri-kubernetes/v3/src/scrape	7.567s

@marcsanmi Would make sense to add already @htroisi team to the maintainers so that e2e tests can run leveraging the secrets?

Yes, I wanted to talk about it with @davidgit first. Invoking him...

@marcsanmi Would make sense to add already @htroisi team to the maintainers so that e2e tests can run leveraging the secrets?

This PR (from my fork of the repo) has an issue with accessing the secrets needed to run the e2e tests. I created a new PR from a branch pushed to this repo directly to see if this resolves the missing secrets issue: https://github.com/newrelic/nri-kubernetes/pull/585


Slow performance with multiple records of forms.

I use redux-form for multiple records of forms.
The performance is terrible when the page has more than 30-40 records, since focusing on one single form element forces all redux-form-connected component to rerender. It's simply because 'connectReduxForm' connect the very root of the redux-form's state with the following connector.
connect(state => state.form)

Though I could write shouldComponentUpdate to the decorated component, and yes I did it, I can not use stateless component with it...
So, here is my hack with reselect.
https://gist.github.com/haradakunihiko/4d27dadd4aba951f6582
Maybe reduxForm should accept only its related form data?
I mean, getSubForm should be outside of reduxForm. How do you think about this?

This library grew organically from something designed to only handle one form, and I think it might need a bit of a rewrite to properly fill the multi-form role that people are now expecting from it.
This issue is very thought provoking. I think you've steered me down the right path. Thank you, @haradakunihiko.

Thank you for this ticket. In v3.0, I implemented it to only connect to the Redux state slice that the form needs. I recommend upgrading.


Support for iPhone Xs Max

The library doesn't properly check if the device has a notch, it only checks if the device has the screen resolution of iPhone X and iPhone Xs but not iPhone Xs Max and because of that, the toast looks weird when shown using the .top valign.

Do you run in simulator? Check in real device :)

No, I ran it on my device and it shows the toast underneath the notch.

I was released Ver.1.3.0.
iPhone Xs Max and XR are supported.
Please give it a try.

Looks good now. Thanks! ✌


[web]: Blank page on Safari and iPhone (everywhere)

Steps to Reproduce
So I deployed a webpage with webdev build and uploaded it to a normal managed server and connected it with a domain.
The output on web:
Firefox: Works perfect
Chrome: Works perfect
Safari: White Blank Page
The output for iPhone:
Firefox: White Blank Page
Chrome: White Blank Page
Safari: White Blank Page
The output for Android:
perfect everywhere
bearbro.de is the domain to test.

this is the safari console log

Logs

webdev build
[WARNING] Throwing away cached asset graph because the build phases have changed. This most commonly would happen as a result of adding a new dependency or updating your dependencies.
[INFO] Cleaning up outputs from previous builds. completed, took 130ms
[INFO] Building new asset graph completed, took 1.7s
[INFO] Checking for unexpected pre-existing outputs. completed, took 1ms
[INFO]build_web_compilers:entrypoint on web/main.dart: Running dart2js with --minify --packages=.package-8673faf6aa3d99e65d2b2214ea745689 -oweb/main.dart.js web/main.dart
[WARNING] No actions completed for 15.1s, waiting on:
  - build_web_compilers:entrypoint on web/main.dart

[INFO]build_web_compilers:entrypoint on web/main.dart: Dart2Js finished with:

Compiled 18,419,520 characters Dart to 1,105,925 characters JavaScript in 16.4 seconds
Dart file (web/main.dart) compiled to JavaScript: web/main.dart.js
[INFO] Running build completed, took 21.6s
[INFO] Caching finalized dependency graph completed, took 166ms
[INFO] Reading manifest at build/.build.manifest completed, took 1ms
[INFO] Deleting previous outputs in `build` completed, took 102ms
[INFO] Creating merged output dir `build` completed, took 907ms
[INFO] Writing asset manifest completed, took 11ms
[INFO] Succeeded after 22.8s with 474 outputs (3193 actions)

info • Unnecessary new keyword • web/main.dart:7:10 • unnecessary_new
   info • Unnecessary new keyword • web/main.dart:8:11 • unnecessary_new
   info • Unnecessary new keyword • web/pages/Service1.dart:6:12 • unnecessary_new
   info • Unnecessary new keyword • web/pages/Service1.dart:8:14 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:14:12 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:18:22 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:20:18 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:21:20 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:24:17 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:24:60 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:28:17 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:28:48 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:36:22 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:38:18 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:39:20 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:42:17 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:42:60 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:46:17 • unnecessary_new
   info • Unnecessary new keyword • web/pages/home.dart:46:51 • unnecessary_new
   info • Unused import: 'dart:async' • web/pages/landingpage.dart:3:8 • unused_import
   info • Unused import: 'dart:math' • web/pages/landingpage.dart:4:8 • unused_import
   info • Unnecessary new keyword • web/pages/landingpage.dart:12:26 • unnecessary_new
   info • Unnecessary new keyword • web/pages/landingpage.dart:25:42 • unnecessary_new
   info • Unnecessary new keyword • web/pages/landingpage.dart:29:14 • unnecessary_new
   info • Unnecessary new keyword • web/pages/landingpage.dart:30:16 • unnecessary_new
   info • Unnecessary new keyword • web/pages/landingpage.dart:34:18 • unnecessary_new
   info • Unnecessary new keyword • web/pages/landingpage.dart:38:15 • unnecessary_new
   info • Unnecessary new keyword • web/pages/landingpage.dart:39:15 • unnecessary_new

(Seems like I have to rethink the using of "new")

[✓] Flutter (Channel stable, v1.7.8+hotfix.3, on Mac OS X 10.14.3 18D109, locale de-DE)
    • Flutter version 1.7.8+hotfix.3 at /Users/bastianmeyer/Documents/flutter
    • Framework revision b712a172f9 (4 days ago), 2019-07-09 13:14:38 -0700
    • Engine revision 54ad777fd2
    • Dart version 2.4.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/bastianmeyer/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 10.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2, Build version 10E125
    • CocoaPods version 1.6.1

[✓] iOS tools - develop for iOS devices
    • ios-deploy 1.9.4

[✓] Android Studio (version 3.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 34.0.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[✓] VS Code (version 1.36.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.2.0

Solved it: the problem was that there were 2 main.dart files.
One has the location web/main.dart and the other one lib/main.dart.
And I made the mistake and edited web/main.dart instead of lib/main.dart to make my own project.


Type name demangler

There is already a demangler for type names, but I think it only works for gnu compilers. I found this on StackOverflow, which provides a more portable (supports gnu, clang and msvc compilers) and cleaner solution. Should we use this instead?
I found this while looking into the PZ issue #30 . Could this help in that regard?

@keceli I realize that this is an old issue now, but can you work on fixing it when you get a chance?


Connect timeout limit

Hey there,
I got a question. In rare situations it takes up to ten seconds to connect to a server socket and because of that it results in a timeout exception. Is there a way to increase the timeout limit?
Error: Timeout connecting to WIN-Q6JGERFITLF:7000
(tested the socket using a portscan. It just takes a while to connect to the system itself in rare cases)
Thank you!

Hi @0x100001 I'm assuming (hoping) you found the Settings.ConnectTimeoutSeconds property on the client :)  Cheers!

Hey @jchristn yes I found out. Somehow I didn't see it on first look. 👀
Thank you!


Cursor position is lost when resizing the window if not on the bottom line

Windows Terminal version (or Windows build number)
10.0.19042.1110 (windows terminal 1.9.1942.0)
Other Software
No response
Steps to reproduce
If the cursor position is not on the last line (of the input), making the screen more narrow will push the cursor down to the last line when lines wrap and the original position is lost; on a resize to make it wider again it stays on the last line.
This affects utilities like "readline" when editing multiple lines of input. Hope it can be fixed as it makes multiline editing not work at all.
To reproduce, run the following batch file:
@echo off
echo Resize the window to make it more narrow and wider again.
echo The cursor will lose its position and move down.
echo.
echo The cursor is first positioned at the start of this long line.
set /p "ANSWER=Press <enter> to stop .."
echo.
echo Thank you.

(note: I think the paste lost it but there should be an ESC character in front of the [ characters to make the escape sequence function; the ESC[1F moves the cursor one up so it is no longer on the last line.)
Expected Behavior
The cursor should always stay at the same point in the input regardless  of a window resize. The "old" cmd terminal is correct and the cursor stays at the same point..
Actual Behavior

Here you see the cursor in Windows Terminal (I made it red) move down a line on the resize and lose its position.

Thanks for the helpful repro case! I'm gonna link this up to #5800, because this might be a repro for a heisenbug I've been hunting

Okay my notes:
this repros when the line is close enough to the bottom line of the terminal window, that the Press <enter> line will get placed on exactly the bottom line. When the long line wraps (which will create a blank line between the long line and the <enter> line), then the cursor gets left at the start of the <enter> line, instead of the long one.
Curiously, in conhost, if the buffer size == viewport size, then the cursor gets left at the end of the long line. It moves down one row, to the created blank one, then back up to the end of the long line when you de-flow the long line.

This is because we'll end up circling the buffer during the resize, after we found the cursor position.
Say the window has 20 rows. That puts the cursor on row 18. When we resize down, and reflow the buffer, we'll look for the old cursor. We'll find it at (0,18) in the old buffer, when we're writing to (0,19) in the new buffer. This is because long line 1 already reflowed, creating an extra row in the buffer.
So we'll stash (0,19) as the expected cursor position in the new buffer. We'll reflow this whole line into the buffer, which will cause us to circle! The place the cursor should really be is line 17, not 19.
I'm putting this in the same bucket as #10130. I think this is gonna just go away with #8000, though we're probably "out of the frying pan, into the fire" with that one

I observed the same. Basically, the cursor is not guaranteed to point to the same character that it was originally pointing at when the window resizing happens. This happens often enough, and makes it difficult for PowerShell to calculate the position of the initial prompt after a resizing.

Same here... I can't seem to reproduce this issue in 1.17/1.18. Did we fix this?

(Sorry for the other comment I posted here - I meant to post it in #10868.)
I believe this issue as described is fixed, but #14448 gave me an idea for reproducing a similar, probably related issue.
Using this .bat file:
@echo off
set /p "ANSWER=Could you share maybe a screenshot/? What shell are you using, and/or what program is doing the outputting? I bet this is #3088 just based on the description, but I can't be sure."

and by resizing the window randomly it is possible to place the cursor in the wrong position. From what I can tell, the issue is due to a desync between conhost and Windows Terminal, because Windows Terminal resizes its buffer before conhost does. This most obvious when actively printing text of course, but it also happens when regularly resizing. I believe my PR #15701 exacerbates this issue, because the reflow is so much faster giving more time for a desync to happen.
But as I said, I believe that the two issues you filed, as they're described, are fixed, even if reflow in general is still buggy.


changed author of doomsday.py

Purpose of Pull Request:
Checks:

[x] Tested on simulation
[ ] Tested on hardware
[ ] Added docstrings to the code
[ ] Made necessary changes to dependencies
[ ] Added changes to installation documentation

New Features:

hoi


It would be ideal to have a docker compose solution for this with possible ollama example

What problem or use case are you trying to solve?
THis would be ideal to solve any installation issues.
Describe the UX of the solution you'd like
Just a working docker-compose.yaml file with options to edit. May be couple of docker-compose examples.
Do you have thoughts on the technical implementation?
Use Docker-COmpose

I am using something like
 openhands:
    image: ghcr.io/opendevin/opendevin
    container_name: openhands_container
    restart: unless-stopped
    pull_policy: always
    stdin_open: true
    tty: true
    environment:
      SANDBOX_USER_ID: 1000
      PERSIST_SANDBOX: "true"
      SSH_PASSWORD: passwd
      LLM_API_KEY: "NA"
      LLM_BASE_URL: "Ollama address:11434"
      WORKSPACE_MOUNT_PATH: /home/user/workspace
    volumes:
      - /home/user/workspace:/opt/workspace_base
      - /var/run/docker.sock:/var/run/docker.sock
    ports:
      - "3000:3000"
    extra_hosts:
      - "host.docker.internal:host-gateway"

version: '3.8'
services:
openhands:
image: ghcr.io/opendevin/opendevin
container_name: openhands_container
restart: unless-stopped
pull_policy: always
stdin_open: true
tty: true
environment:
- SANDBOX_USER_ID=1000
- PERSIST_SANDBOX=true
- SSH_PASSWORD=passwd
- LLM_API_KEY=NA
- LLM_BASE_URL=localhost:11434
- WORKSPACE_MOUNT_PATH=/home/user/workspace
volumes:
- /home/user/workspace:/opt/workspace_base
- /var/run/docker.sock:/var/run/docker.sock
ports:
- "3000:3000"
extra_hosts:
- "host.docker.internal:host-gateway"