Bugfix: Fix retrieval of navigate function

What this PR does / why we need it:
This PR fixes a tiny bug with the UI when a user is led to the landing page with a prompt to create new project settings for a project that has yet to be set up.
Upon clicking on the link in the prompt, the user would not get redirected correctly and nothing happens.

Which issue(s) this PR fixes:
This PR fixes the way in which the navigate function from the react-router-dom package gets retrieved.

Thanks for the quick approval! 🚀


Replace more js with jquery in server.js

Replace javascript code with more readable and clearer jquery
Add 3 static rows to tserver table instead of dynamically creating the rows

@ctubbsii I want to do the other tables on this page as well. If you want to hold off reviewing this, I can convert it to a draft and clean up the entire server.js.


Where are the Alert definition for the Alerts from the runbook

Hi,
in the current kubevirt operator only a subset of the Alerts documented in this repo are defined.
source: https://github.com/kubevirt/kubevirt/blob/main/pkg/virt-operator/resource/generate/components/prometheus.go
Where can I find the definition for the other alerts?

All alerts include a runbook. Perhaps you are referring to recording rules?

@ulikl Sorry for the delay. These alerts were removed since they were determined as too opinionated and may not fit all clusters. But it is possible to add them as custom alerts if you require them.
The runbooks were left by accident and I removed them now.


Recon script thats fit in github pd action?

any one working on this?

I don't think so, but if you're on it, let me know!


planning/control component is dependent on the packages outside of planning/control component

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I've agreed with the maintainers that I can plan this task.

Description
https://github.com/autowarefoundation/autoware/pull/5194 found that the planning/control component is dependent on the following packages outside of the planning/control component.

map/map_loader
map/autoware_map_projection_loader
sensing/autoware_pcl_extensions
sensing/autoware_pointcloud_preprocessor

Since the dependencies will no longer follow a parent-child relationship, the components become tightly coupled.
Purpose
Ensure that there are no dependency packages spanning across components. Connections between components should depend only on the core packages or the autoware.universe/common packages.
Possible approaches
Ensure that there are no dependency packages spanning across components. Connections between components should depend only on the core packages or the autoware.universe/common packages.
Definition of done
The planning/control packages will no longer directly depend on the following packages.

[ ] map/map_loader
[ ] map/autoware_map_projection_loader
[ ] sensing/autoware_pcl_extensions
[ ] sensing/autoware_pointcloud_preprocessor

@youtalk
Thank you for the issues, and sorry to be late.
Could you give me the command to show the list of the packages outside of the planning/control component which the planning/control component is dependent on?

@takayuki5168 I only checked by CI builds.
I made a reproducible PR https://github.com/autowarefoundation/autoware/pull/5331 and the latest result was https://github.com/autowarefoundation/autoware/actions/runs/11287273469/job/31392958946?pr=5331#step:5:988.
 > [rosdep-universe-planning-control-depend 3/4] RUN rosdep keys --ignore-src --from-paths src     | xargs rosdep resolve --rosdistro humble     | grep -v '^#'     | sed 's/ \+/\n/g'    | sort     > /rosdep-universe-planning-control-depend-packages.txt     && cat /rosdep-universe-planning-control-depend-packages.txt:
1.349 ERROR: no rosdep rule for 'autoware_map_projection_loader'
1.349 ERROR: no rosdep rule for 'autoware_pointcloud_preprocessor'
1.349 ERROR: no rosdep rule for 'map_loader'

If we are going to include tier4_control_launch and tier4_planning_launch, I had to add the following dependencies as well:

autoware_control_evaluator
autoware_evaluator_utils
autoware_planning_evaluator
autoware_external_cmd_converter
autoware_raw_vehicle_cmd_converter


Pin distroless base image to debian11

Per the comments on the distroless README:

Images are explicitly tagged with Debian version suffixes (e.g. -debian11). Specifying an image without the distribution will currently select -debian11 images, but that will change in the future to a newer version of Debian. It can be useful to reference the distribution explicitly, to prevent breaking builds when the next Debian version is released.


[ros2] Ignition Transport message duplication caused by bidirectional bridge

Environment

OS Version: Ubuntu 20.04
Ignition: Edifice, built from source
ROS 2: Foxy, binary install
ros_ign branch: ros2 (54e12e2653054fbb925829784ae9151c793288c3), built from source

Note: Tested inside Docker container. I would appreciate someone to reproduce this issue to ensure it is not caused by my setup.

Description

Expected behaviour: Running a bidirectional parameter_bridge with Ignition publisher (separate) should generate only ROS 2 messages.
Actual behaviour: Running a bidirectional parameter_bridge with Ignition publisher (separate) generates both ROS 2 messages and duplicate versions of ignition-transport messages.

What could happening?:

Bidirectional bridge creates Node A: Ignition subscriber + ROS 2 publisher, and Node B: ROS 2 subscriber + Ignition publisher.
Once an ignition-transport message is published, Node A receives it and generates ROS 2 message. Subsequently, Node B receives this message generated by Node A and generates a duplicate ignition-transport message. (problem with ROS 2 subscriber configuration?)
This is just a speculation, I have not investigated the code. I am sure the original developers thought of this, otherwise there would be an infinite recursion happening if this issue occurred in both directions/multiple times.

Workaround: Use unidirectional bridge (99% of the use cases) to prevent message duplication that both decreases performance and could potentially break something. Updating examples to use unidirectional bridge instead of bidirectional bridge might also be an idea.

Note: Issue does not seem to occur for ROS 2 messages, i.e. bridge does not duplicate any ROS 2 messages.
Note: I have not tested this with ROS 1, so I am not sure if it experiences the same problem.

Steps to reproduce
Shell 1:
ros2 run ros_ign_bridge parameter_bridge /chatter<EMAIL_ADDRESS>Shell 2:
ign topic -e -t /chatter
Shell 3:
ign topic -t /chatter -m ignition.msgs.StringMsg -p 'data:"This message should be received only once"'
Output
root@P5550:~# ign topic -e -t /chatter
data: "This message should be received only once"

data: "This message should be received only once"

Expected output (same as without running any bridge)
root@P5550:~# ign topic -e -t /chatter
data: "This message should be received only once"

Interesting, I thought this had been fixed by https://github.com/ignitionrobotics/ros_ign/pull/33. I see the code that's supposed to prevent it in the ros2 branch:
https://github.com/ignitionrobotics/ros_ign/blob/117e2b2efb2430b199779e82d40344386dd8a253/ros_ign_bridge/src/factory.hpp#L98-L100
Maybe the IntraProcess call is not doing the trick anymore.

I believe the issue originates in the ROS 2 subscriber as there is currently no mechanism preventing this to occur. Ignition IntraProcess seems to work fine (otherwise there would be the same issue in the opposite direction and ROS 2 messages would get duplicated).
I have proposed a solution in #146 that makes use of ignore_local_publications subscriber option. However, it does not fix the issue for all rmw implementations, e.g. Cyclone DDS works, but FastRTPS (ROS 2 Foxy default) still has the issue. I added more info into the PR itself.

This issue was resolved by https://github.com/gazebosim/ros_gz/pull/146 in this codebase. Furthermore, FastRTPS RMW also supports this functionality now (https://github.com/ros2/rmw_fastrtps/pull/536).
Closing...


appsre-ansible: support aarch64 machines

This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

Thank you! This looks good, and I don't think there's anything that can't be addressed in follow-ups.
This turned out to be quite hairy :)
I do have a general comment for future PRs, and this might be more a personal taste thing. But in general I don't think its desirable to have the history of a PR/the review cycles of a PR visible in the git log if there's no reason for it. Example are the tools/build-rpms.py: no more globals  and improve f-string commits, which really should be squashed into the commit that introduced tools/build-rpms.py (should be quite easy to do with rebase -i).

Thanks for the comments :)
What I had in mind was that the this whole branch will be squashed into one commit, but I still felt the separate commits will make things much easier for reviewers, so I did them anyway.


changed ActionBarActivity (deprecated) to AppCompatActivity

yeah, because ActionBarActivity just got deprecated. We should use AppCompatActivity instead.

Ping @neokree


Performance regression for strconv TestIsPrint and TestIsGraphic for strconv

go_goto branch (on personal computer)
=== RUN  TestIsPrint
-- PASS: TestIsPrint (49.5220510959625)

=== RUN  TestIsGraphic
-- PASS: TestIsGraphic (51.3606071472168)

master branch (on github actions):
-- PASS: TestIsPrint (12.3976559638977)

=== RUN  TestIsGraphic
-- PASS: TestIsPrint (12.3976559638977)

I suspect it will be even slower when ran on github actions with less powerful specs, but exactly the same.

Closing this, probably a false flag of having lots of prints for debugging on my computer.
Latest from master with goto changes merged:
-- PASS: TestIsGraphic (13.485641002655)
 -- PASS: TestIsPrint (12.5122849941254)

Good news!
On Tue, Nov 5, 2024 at 4:58 PM Joey @.***> wrote:

Closing this, probably a false flag of having lots of prints for debugging
on my computer.
Latest from master with goto changes merged:
-- PASS: TestIsGraphic (13.485641002655)
-- PASS: TestIsPrint (12.5122849941254)
—
Reply to this email directly, view it on GitHub
https://github.com/go2hx/go2hx/issues/180#issuecomment-2457707879, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AA646NTZZXHEXID4GRITCZLZ7D2MHAVCNFSM6AAAAABRES3YBSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJXG4YDOOBXHE
.
You are receiving this because you commented.Message ID:
@.***>


add lerna badge in README

create-react-app is using lerna that's why I think we should add lerna badge.

Thanks for the PR. However, we only use Lerna internally and we don't currently support using Create React App with Lerna. So I think adding this badge would be confusing for users.


Parry & Toughness

Would be nice to have a "Defense" item that showed Parry and Toughness. Even nicer if it gave the full breakdown like the character sheet does.

You could maybe use other modules to show it, e.g. Bar Brawl ?

Yeah, Bar Brawl works for what I want.
On Tue, Feb 6, 2024 at 7:56 AM Steve Culshaw @.***>
wrote:

You could maybe use other modules to show it, e.g. Bar Brawl ?
—
Reply to this email directly, view it on GitHub
https://github.com/mrcomac/token-action-hud-swade/issues/34#issuecomment-1929959931,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADZH6LK234Y7BBTPCGYRZTTYSJAC3AVCNFSM6AAAAABC3BRMB2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRZHE2TSOJTGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thank you, @steveculshaw. I'm hesitant to transform the HUD into a sheet where you can see those stats. The HUD is intended to be a shortcut for actions (Token Action HUD) rather than for displaying stats/info.
Maybe I'm wrong and we should have those info in the HUD.
There's always the possibility to add a configuration option for how the HUD behaves (display everything from the sheet or only the possible actions?).
btw, I'm working to make the Argo token hud compatible with savage worlds, that hud indeed will have that information.

I mean, it's all optional right? If you want only actions you can leave out
those items. If you want it, add them in
On Tue, Feb 6, 2024, 1:05 PM Marcos @.***> wrote:

Thank you, @steveculshaw https://github.com/steveculshaw. I'm hesitant
to transform the HUD into a sheet where you can see those stats. The HUD is
intended to be a shortcut for actions (Token Action HUD) rather than for
displaying stats/info.
Maybe I'm wrong and we should have those info in the HUD.
There's always the possibility to add a configuration option for how the
HUD behaves (display everything from the sheet or only the possible
actions?).
btw, I'm working to make the Argo token hud compatible with savage worlds,
that hud indeed will have that information.
—
Reply to this email directly, view it on GitHub
https://github.com/mrcomac/token-action-hud-swade/issues/34#issuecomment-1930666914,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADZH6LIRII5NUIPS6QCQ7LDYSKEITAVCNFSM6AAAAABC3BRMB2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZQGY3DMOJRGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

This request pertains to the addition of non-actionable items to the HUD. I am closing it in favor of proceeding with task https://github.com/mrcomac/token-action-hud-swade/issues/35


Ddpb 3442

Purpose
Briefly describe the purpose of the change, and/or link to the JIRA ticket for context
Fixes DDPB-####
Approach
Explain how your code addresses the purpose of the change
Learning
Any tips and tricks, blog posts or tools which helped you. Plus anything notable you've discovered about DigiDeps
Checklist

[ ] I have performed a self-review of my own code
[ ] I have updated documentation (Confluence/ADR/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have approved these changes

Frontend

[ ] I have run an in-browser accessibility test (e.g. WAVE, Lighthouse)
[ ] There are no deprecated CSS classes noted in the profiler
[ ] Translations are used and the profiler doesn't identify any missing

I left it as a separate task, most likely part of the POST ticket, just to keep these incremental releases a little smaller. Focus here was to write the service that orchestrates everything, but without too much implementation elsewhere. Not sure if that is the right approach but it probably won't matter either way as it won't get switched on until it's all done.

Yeah makes sense - I'm all for smaller incremental releases.

@acsauk-mojd Please can I get a reapproval. I have added extra scaffolding code to ensure no errors when the service is invoked whilst there is no API for documents/queued. These empty methods have temporarily decreased unit test coverage until we implement those methods.


Result list is too large

We need to add pagination for two reasons:

The result list is too large
We are currently only returning a fraction of the results


Enable logout redirect feature by default or provide a way to add your url to the redirect urls whitelist

Hi guys,
When using UAA as OAuth2 server it's really inconvenient that by default an application can't redirect to its page after UAA logout. By default logout redirect is disabled in UAA. Why did you disable redirects by default?
Also, it looks like there is no way to enable logout redirect in UAA without redeploying the server. It looks really inconvenient. It would be cool if an application can "register" itself in UAA to appear in its "whitelist".
Could you please clarify what is the correct way of enabling redirects in UAA for client applications? Does it really require server redeploy?
Any help would be really appreciated. Thanks!

Guys, any updates on it?


ImageAI on colab

Hi all,
I am trying to run the folwoing example from the wiki in Google colab. I initialize the environment as it is stated on the readme file and I get the error about tensorfolow.
My code:
`from imageai.Detection.Custom import DetectionModelTrainer
trainer = DetectionModelTrainer()
trainer.setModelTypeAsYOLOv3()
trainer.setDataDirectory(data_directory="hololens")
metrics = trainer.evaluateModel(model_path="hololens/models", json_path="hololens/json/detection_config.json", iou_threshold=0.5, object_threshold=0.3, nms_threshold=0.5)
print(metrics)`
Initialization
 !pip install tensorflow-gpu==1.13.1  !sudo apt-get purge nvidia*  !sudo wget https://developer.nvidia.com/compute/cuda/10.0/Prod/local_installers/cuda-repo-ubuntu1604-10-0-local-10.0.130-410.48_1.0-1_amd64  !sudo dpkg -i cuda-repo-ubuntu1604-10-0-local-10.0.130-410.48_1.0-1_amd64  !sudo apt-key add /var/cuda-repo-/7fa2af80.pub  !sudo apt-get update  !sudo apt-get install cuda=10.0.130-1
Error:
ImportError: cannot import name 'RegisterShape' from 'tensorflow.python.framework.ops' (/usr/local/lib/python3.7/dist-packages/tensorflow/python/framework/ops.py)
Alternative initialization
`!update-alternatives --install /usr/bin/python python /usr/bin/python3.7.6
!pip install tensorflow-gpu==2.4.0
!pip install keras==2.4.3 numpy==1.19.3 pillow==7.0.0 scipy==1.4.1 h5py==2.10.0 matplotlib==3.3.2 opencv-python keras-resnet==0.2.0
!pip install imageai --upgrade`
Error
`--> 177 from tensorflow.python.keras.layers.normalization import LayerNormalization
178 from tensorflow.python.keras.layers.normalization_v2 import SyncBatchNormalization
179
ImportError: cannot import name 'LayerNormalization' from 'tensorflow.python.keras.layers.normalization' (/usr/local/lib/python3.7/dist-packages/tensorflow/python/keras/layers/normalization/init.py)`
And the same error with the following initialization:
!pip install tensorflow-gpu==2.4.0 !pip install tensorflow-gpu==2.5.0rc0 !pip install folium==0.2.1 !pip install imgaug==0.2.6 !pip install bokeh==2.3.0 !pip install keras==2.5.0rc0 numpy==1.19.3 pillow==7.0.0 scipy==1.4.1 h5py==2.10.0 matplotlib==3.3.2 opencv-python keras-resnet==0.2.0 !pip install imageai --upgrade
Could someone please help me on the correct initialization for Colab?

can you share the link to your notebook you ran to get this thy

I hope this is accessible:
https://colab.research.google.com/drive/1y2eH2jz4PgfvZ5Ykl4F6Oobh4crfTz9t?usp=sharing

I needed to restart runtime in Colab and finally it was fixed with the following initialization
!pip install tensorflow-gpu==2.5.0rc0
!pip install folium==0.2.1
!pip install imgaug==0.2.6
!pip install bokeh==2.3.0
!pip install keras==2.5.0rc0 numpy==1.19.3 pillow==7.0.0 scipy==1.4.1 h5py==2.10.0 matplotlib==3.3.2 opencv-python keras-resnet==0.2.0
!pip install imageai --upgrade


URL parser does not handle [] in the link

I'm having trouble generating links if the URL has [] in it, which is used in Rails to pass through arrays and hashes. For example, this, an URL like this is not parsed:
https://shalazam.info/maps/1?x=1691.0&y=755.0&zoom=5.0&pin_loc[x]=3440&pin_loc[y]=3716
I can also repreoduce this on the bbob playground with an example as simple as:
[url=https://shalazam.info?loc[x]=1]test[/url]

Parsed to:
[
  "[",
  "url=https://shalazam.info?loc",
  {
    "tag": "x",
    "attrs": {},
    "content": []
  },
  "=1]test"
]

Removing the [x] makes it work.

I have hit upon a similar problem using the current version of 4.1.1. Could this be a general inability to exclude invalid tags from parsing? In my case (BBob configured for React usage), anything in brackets is converted to either an HTML tag or, depending on case a React component. The latter case even leads to runtime errors.
'[tag]' -> <tag>
'[Tag]' -> <Tag />

@celsworth as a solution you can encode [] in url in Rails
URI::Parser.new.escape(my_url_string)

or in js
encodeURI(my_url_string)

that leads to https://shalazam.info/maps/1?x=1691.0&y=755.0&zoom=5.0&pin_loc[x]=3440&pin_loc[y]=3716


Fix bare except

There is a lot to fix out there in terms of best practices such as this.
I recommend to use Ruff or some other linter of choice and run it over the codebase.


Convert project to TypeScript

Adds standard TrialSpark tooling
Converts the project to TypeScript

Whoops, sorry, didn't mean to open this!


Rename and fine all references of Typescript's string literal values

From @zivni on March 20, 2016 8:40
Type script string literals allow to have a code such as
//interfaces.d.ts
type Options = "option 1 " | "option 2"

//file.ts
let myOption: Options = "option 1" // "option 3" will give us an error, right in the IDE.

It would be nice to be able to rename and find all references of a specific value. e.g. find/rename all occurrences of "option 1". Currently we can only find the references of the type itself.
Copied from original issue: Microsoft/vscode#4469

so should we rename all literal types "option 1" or just the ones from this declaration?
for example:
type Options = "option 1" | "option 2"
let myOption: Options = "option 1" // rename here,

function foo(a: "option 1") { } // should this be renamed as well?

This does not work as accepted. I don't know if this problem is with Typescript or with VSCode.
Just open a new test.ts file with this content in vscode:
type A= "aaa" | "article"
type B = "aaa" | "ccc"

var a:A="aaa"
var b:B="aaa"

press F2 to rename the "aaa" of the A type. the "aaa" of the B type is also renamed.
It goes even further, rename the "article" of the A type. You'll  notice that it changed the typing in lib.es6.d.ts to getElementsByTagName(tagname: "renamed_article"): NodeListOf<HTMLElement>;
And also the find all references does not work
Tried it on vscode 1.7.1 on windows.

there is really no difference between "aaa" in type A and in type B. they are really the same type. the lib.d.ts change should be disabled, this is tracked by https://github.com/Microsoft/TypeScript/issues/7458

I will have to disagree.
It like saying that 2 properties with the same name in two different interfaces are the same.
Lets take a more concrete example: As typescript doesn't yet have a nameof operator, a common practice, when you need to pass a member name to a function, is to list members using a string literal types. E.g.:
type UserMembers = "id" | "name | ...
interface User{
   id: string
   name: string
   //...other properties
}

type OrderMembers = "id" | "name | ...
interface Order{
   id: string
   name: string
   //...other properties
}

// and then:
const orderByFields:UserMemebers[] = ["name","id"]
let orderedUsers=_.orderBy(users,orderByFields)

No lets say we decide to rename the members to more meaningful names, like userId and orderId, or userName and orderName. The interfaces will be renamed as accepted, but the string literal types will not.
Another example could be tat each have a status member with there own string literal types. and you need to change just a status name og just one of them
Also the problem described in  #7458 is  a different problem. there the rename is of a specific member, and what was  wanted is to rename a specific instance of "html" code. Here the problem with article is that it renames something that is entirely unconnected to the renamed object.


camera

after running full code my camera light starts glowing but camera not working means not showing

Hi there,
Thank you for reaching out. To assist you further, could you please provide the log containing the error messages you encountered? This will help me better understand the issue and provide you with the appropriate solution. Additionally, please ensure that you have installed all the necessary libraries as mentioned in the project.


X509 Auth issue for MongoDB Atlas with OTP 26

I just wanted to quickly update to elixir 1.16 and OTP 26 but discovered some Auth issues I haven't seen before.
I use MongoDB Atlas with an X509 certificate which was generated by MongoDB itself.
And I use the currently latest version 1.2.1 of this library.
My config looks like this and is currently working without any problems in elixir 15 / OTP 25:
  config :my_app, :mongo_config,
    name: :my_app,
    appname: "myapp",
    url: "mongodb+srv://default.3kjdjd.mongodb.net",
    username: "CN=my_username",
    password: "",
    database: "my_database",
    timeout: 60_000,
    idle_interval: 10_000,
    queue_target: 5000,
    pool_size: 100,
    auth_mechanism: :x509,
    ssl_opts: [
      certfile: "/path/to/my_cert.pem",
      verify: :verify_none
    ]

I tried the following elixir/erlang combinations:

elixir 15 / OTP 25 <- is running in production
elixir 15 / OTP 26
elixir 16 / OTP 25
elixir 16 / OTP 26

The result is that only the combinations with OTP 26 don't work.
With OPT 26, regardless which query I try, I always get:
 iex(1)> Mongo.find_one(:my_app, :my_collection, %{})
 {:error,
  %Mongo.Error{
    message: "command find requires authentication",
    code: 13,
    host: nil,
    fail_command: false,
    error_labels: [],
    resumable: false,
    retryable_reads: false,
    retryable_writes: false,
    not_writable_primary_or_recovering: false,
    error_info: nil
  }}

With OTP 25 and either elixir 15 or 16 I get the data results I expect.
To be sure I also tried the proposed config from the readme and changed the ssl config part to:
ssl: true,
ssl_opts: [
  certfile: "/path/to/my_cert.pem",
  verify: :verify_peer,
  cacertfile: to_charlist(CAStore.file_path()),
  customize_hostname_check: [
    match_fun: :public_key.pkix_verify_hostname_match_fun(:https)
  ]
]

with the same results as mentioned above. So this config didn't fix it.
I saw the OTP 26 adjusted some SSL related things but I cannot spot how and where this breaks stuff in the x509 authentication with MongoDB Atlas.
Did anyone of you have similar problems and/or ideas how to fix it?
Cheers
Frank

I will take a look at it on the next weekend.

If I use the example from the documentation then everything works as expected:
iex [18:55 :: 1] > {:ok, top} = HelloWorld.connect()
{:ok, #PID<0.234.0>}
iex [18:55 :: 2] > Mongo.show_collections(top) |> Enum.to_list()
["dogs"]
iex [18:55 :: 3] > Mongo.find(top, "dogs", %{}) |> Enum.to_list()
[
  %{"_id" => #BSON.ObjectId<63135a9a4ae06e5cca171679>, "name" => "Tom"},
  %{"_id" => #BSON.ObjectId<63135a954ae06e5ccaa3e990>, "name" => "Greta"}
]

My config is:
def connect() do
    opts = [
      url: "mongodb+srv://cluster0.xxxx.mongodb.net/myFirstDatabase?authSource=%24external&retryWrites=true&w=majority",
      ssl: true,
      username: "CN=cert_user",
      auth_mechanism: :x509,
      ssl_opts: [
        verify: :verify_peer,
        cacerts: :public_key.cacerts_get(),
        certfile: './X509-cert-1577233547801905820.pem',
        customize_hostname_check: [
          match_fun:
            :public_key.pkix_verify_hostname_match_fun(:https)
        ]
      ]
    ]
    Mongo.start_link(opts)
  end

Thanks for trying out.
It's still not working for me with OTP-26.
Even if I use your code snipped with my Atlas credentials. Same error like I described above.
I notice that you are probably using a shared atlas cluster?
Because my url for my dedicated cluster looks like this mongodb+srv://default.XXXX.mongodb.net vs your url mongodb+srv://cluster0.xxxx.mongodb.net.
But not sure if it makes any difference if using a shared vs a dedicated cluster.
Oh, I didn't say that before. I'm using MongoDB v6.
Cheers
Frank

Did you check the firewall? Maybe you can give temporary access to a demo db and I can try it out.

Yes, firewall is fine.
I test this by switching from OTP-26 to OTP-25 (I’m using asdf for this) without changing any code.
With OTP-25 everything works.
I currently don’t have a test db I can give you access to but will look at it when I‘m back at work.

Besides that the ssl verification is somehow broken for me, I'm wondering why
ssl_opts: [ verify: :verify_none ]

doesn`t bring the previous behavior back.

I guess, because you use the X509 auth mechanism which relies on verifying.

Using the x509 auth mechanism using ssl_opts: [ verify: :verify_none ] works fine until OTP-26.
With OTP-26 it looks like these verify option does nothing or at least not the same like with OTP-25

So I just got a log entry from MongoDB which may give some more insights...
{
  "t":{"$date":"2024-01-15T12:28:57.012+00:00"},
  "s":"I",
  "c":"ACCESS",
  "id":20428,
  "ctx":"conn173745",
  "msg":"Failed to authenticate",
  "attr":{
    "client":"XX.XX.XXX.XXX:36734",
    "mechanism":"MONGODB-X509",
    "user":"CN=myuser",
    "db":"$external",
    "error":{
      "code":18,
      "codeName":"AuthenticationFailed",
      "errmsg":"No verified subject name available from client"
    }
  }
}

Also tried it directly via mongosh whitout problems:
mongosh "mongodb+srv://default.XXXX.mongodb.net/products?authSource=%24external&authMechanism=MONGODB-X509" --apiVersion 1 --tls --tlsCertificateKeyFile "/path/to/my/cert.pem" --eval 'db.runCommand({ping:1})'
{
  ok: 1,
  '$clusterTime': {
    clusterTime: Timestamp({ t:<PHONE_NUMBER>, i: 1 }),
    signature: {
      hash: Binary.createFromBase64('AhIUib/WN6jbpdseoeFuRVSxtYbY=', 0),
      keyId: Long('72868480552342391809')
    }
  },
  operationTime: Timestamp({ t:<PHONE_NUMBER>, i: 1 })
}

Maybe the subject name is not the right one.

I sent you a message in the elixir forum.

So, to fix my connection issue I needed to add versions: [:"tlsv1.2"] to ssl_opts.
It looks like that mongoDB atlas (on a dedicated cluster) currently doesn't support TLS 1.3.
Thx @zookzook for your help with debugging and pointing me to the solution.


Provide better error-handling code within the library

Following feedback in #7, this PR gets rid of all calls to unwrap and expect that may panic, and instead optimistically continues. All such cases have been annotated with
/// DWARF is invalid -- <reason>

I will rebase this onto master once #7 and #8 have been merged.

Should we add in debug_assert! statements for each of the cases where we currently encounter invalid data? Might help in diagnosing errors further down the line?

Alternatively, we could make locate() return a Result, though Result<Option<T>> isn't a great type to expose through the API. Maybe the inner locate() methods could return that, and Mapping::locate() could flatten it into just an Option? And then we could add a debug_assert there?

8b07f95c623013e4832a6c1ee83d963120643dc7 uses error-chain to implement deep error handling, propagating error chains all the way up to the pub methods on Mapping. Mapping::locate() currently simply drops the Err if there is one, but in the future this would be where we'd add in logging or debug assertions.

@fitzgen ready for review


Issues with concurrent bundle installs with shared gem cache

I am attempting to use a shared gem cache with a docker-compose setup, a sample project is available here: https://github.com/stefansedich/bundle-test which easily replicates the issue by simply running run.sh.
The error is consistent and one of the two containers will fail to compile the native extensions for in this case the pg gem I assume due to the concurrent installs attempting to create the Makefile.
What is the best way to handle this scenario, my current solution is to wrap my bundle install in a script that will use flock to limit concurrent installs.

@stefansedich thank you for reporting this!
@segiddins ugh this scenario did not even occur to me 😭 I'm thinking process lock file a la git?

FYI my current solution using a install wrapper script does the trick!
#!/bin/bash

set -e

(
  flock 200

  bundle install
) 200>/.gems/install.lock

@indirect if a process lock is the desired solution and everyone is happy with that I am happy to tackle it and submit a PR for it.

@stefansedich I think that's the way to go for now—we're not designed to allow multiple concurrent processes. Maybe a future goal can be optimizing that beyond a process lock.

Sounds good, from my initial look it looks like Installer#run is the best place to put the lock, does that sound right to you?

That sounds right to me. You'll also want to check the update command and make sure it shares the lock. 👍🏻

We could also do everything in temporary directories and move stuff since moves are atomic

@segiddins does it make more sense to just wait? otherwise in this example I compile PG twice in parallel, if we just lock and wait by the time the second install can continue PG should already be compiled and ready to use?

You've given the install command twice, so I don't think that installing twice is out of the question. In addition, in my experience proper multi-process coordination is a lot harder than just sticking with atomic ops


auro-popover: interactive elements inside popover are focusable when hidden

Describe the bug
Interactive elements elements inside a popover are focusable when hidden.
To Reproduce
Steps to reproduce the behavior:

Go to the live popover demo
Tab down to the sticky popover. Notice that you are able to tab to the button inside the sticky popover even when the popover is closed.

Expected behavior
I am unable to tab to content inside of the popover when the popover is closed.
Desktop (please complete the following information):

OS: Windows 10
Browser: Firefox
Version: 85

This issue is tied to https://github.com/AlaskaAirlines/auro-popover/issues/10

The resolution for this issue may be in the inert feature that @geoffrich recently completed with auro-interruption.
Please see the last two PRs from @geoff https://github.com/AlaskaAirlines/auro-interruption/pulls?q=is%3Apr+is%3Aclosed
Please consult with @geoffrich for support in adding this functionality to the popover element.

This issue will be addressed with other feature work in https://github.com/AlaskaAirlines/auro-popover/issues/20


[docs]: translate hertz tutorials/service-governance/open-telemetry/

OpenTelemetry

zh web page: https://www.cloudwego.io/zh/docs/hertz/tutorials/service-governance/open-telemetry/
zh docs: https://github.com/cloudwego/cloudwego.github.io/edit/main/content/zh/docs/hertz/tutorials/service-governance/open-telemetry.md
en folder: https://github.com/cloudwego/cloudwego.github.io/edit/main/content/zh/docs/hertz/tutorials/service-governance/

Notes:
Do not change the original layout, code style or content sequence.
Software Translator is a tool, not god, do not use the result as the final result.
Translation principle: faithfulness, expressiveness and elegance.
Submit your PR here in this repo.

I am willing to work for this :)

I am willing to work for this :)

thanks


Feature: Pulley and Path Constraints

The pulley constraint is straight-forward. Though I couldn't figure out a good demo of various weights on the pulley.
I was tempted to wire it to a draw-bridge and push a giant rock on one side, and might do that in my actual games.
The paths were made so I can do nice floating platforms in a jumper game.
ex: Mario64's moving platform puzzles (most Bowser levels, Rainbow Ride, Tick Tock Clock)
The Path Constraint is functional with a custom JS Path, but the demo implementation is not the ideal way to do it.
The demo currently uses a high-res sample of the path and brute force catches 'closest point' and uses that to assign the mFraction of the constraint on the path. I'd thought of using an SVG path object since that supports a mirrored 2D functionality of pathElement.getPointAtLength(n) and pathElement.getTotalLength()
The normal/binormal/tangent is still likely how you would do it in most cases.
For GetClosestPoint, ideally instead you would use a lookup data-structure, pre-calculated for most likely/common Body2 locations. For 2D-ish paths, you can even just make a data-texture of common locations and pre-calculated 'XY coord is closest to mPathFraction" as an instant lookup.
Possibly maintain multiple Path objects would help, so there exists a per Constraint context with a memory of 'prior mPathFraction' and jump near there to start the search. That would also solve the problem I had when I tried to do figure-8s and other complex self-crossing paths, where the mBody2 location search would jump-track when a path crossed itself, since without a memory you don't know which of the crossed paths to take. https://github.com/jrouwe/JoltPhysics/blob/master/Jolt/Physics/Constraints/PathConstraint.cpp#L122 .

Path Constraint
https://github.com/jrouwe/JoltPhysics.js/assets/756488/c2da9dfd-b9be-4ed4-ac75-80613ad8f7e2
Pulley Constraint
https://github.com/jrouwe/JoltPhysics.js/assets/756488/71ec2475-cbfa-4890-a949-e59c2ee1e956

The Path constraints demo should probably color-code the blocks to the actual colors used on the axis debugger code (the ray's shooting off each block).

I've created a PR to provide the path fraction so you can support figure 8-s. I'll merge this in a moment.

(and I also fixed the typo in the enum you found)

All changes have been merged in https://github.com/jrouwe/JoltPhysics.js/commit/5b5ec0e2856bb266733e989c4c940864e0034e56

I think Pulley is one of the few constraints with pure and unalterable fixed world points? There is no way for a pulley to be on a moving object due to that, unless you destroy and recreate the constraint every movement of the object. If I needed it, for now I would probably instead fake it with 2x distance constraints.
I believe the only two constraints we don't have are gear and rack & pinion, but I haven't thought of anything interesting that uses those where I currently wouldn't just bypass to a powered hinge or slider. Possibly gear makes sense for a pseudo-water wheel turning a mill, or connect the two and have the wheel being engaged open a gate via wheel-to-gear-to-pinion, and turn off the water for the wheel to spin backwards as the weight of the gate closes, but haven't though of anything that isn't a niche use case.

I guess the pulley would become a 3-to-4-Body constraint if it didn't use the fixed world points

I think Pulley is one of the few constraints with pure and unalterable fixed world points?

Yes, and you're right that otherwise it would become a 3-4 body constraint. I made it to support what the Source Engine needed but so far it didn't get integrated and to my knowledge it is not used by anyone.
Gear & rack & pinion are indeed also niche constraints.

Hmm, kinda interested in gear & rack & pinion. Not sure I understand what those are, though. Or is it a single constraint? Is it something like this?:
https://fabmax.github.io/kool/kool-js/?demo=phys-joints
As for the static constraints, I think another case is probably with soft bodies. Even though not a constraint, technically. Could probably be useful to set the position of the static vertices, so the cloth could be attached to moving bodies.

Rack and pinion: https://www.youtube.com/watch?v=e588KG-ZSxc
Gear: https://www.youtube.com/watch?v=3w5SgElroBw
Soft body is not compatible with any constraints at the moment, you need to set the inverse mass of a vertex to 0 and then animate it yourself. At some point in the future I will make it possible to attach a point to a dynamic rigid body.

I just exposed gear and rack & pinion constraints (didn't make a demo and also didn't release a version yet so you need to build yourself)


PyPI version does not match Github release - 1.7.7

The files available here for 1.7.7 - https://pypi.org/project/pyintesishome/1.7.7/#files do not match the Github release of 1.7.7.
This means I can't use the Anywair functionality that was added. Can you please reupload to PyPI?

Sorry for the delayed response - can you use 1.8.0? If not I'll look into resolving it

No worries, yep 1.8.0 should work fine thanks.
On Thu, 30 Sept 2021 at 06:49, James Nimmo @.***> wrote:

Sorry for the delayed response - can you use 1.8.0? If not I'll look into
resolving it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jnimmo/pyIntesisHome/issues/32#issuecomment-930531965,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB6CO2D5LJFPQ726Y3RTRTUEN3TZANCNFSM452GFJFQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Declouple Controller from MongoDB

Hi,
We have a MongoDB dependency in the controller. You can see here: https://github.com/amitshekhariitbhu/go-backend-clean-architecture/blob/main/api/controller/task_controller.go#L8
Maybe we should consider move the code bellow to other layer:
task.ID = primitive.NewObjectID()
task.UserID, err = primitive.ObjectIDFromHex(userID)

What's your thoughts?

Hi @marcosvidolin
Your point is valid. A few more developers also suggested making this change. I shall do it.
Thanks

Hi @marcosvidolin
I understood your suggestion but makes me wonder how can we properly handle errors like this to not be generic InternalServerError?
Sorry for the late response here, I just found this project

I would recommend moving this code to the MongoDB repository and also suggest creating a custom error in the following manner:
task, err := primitive.ObjectIDFromHex(userID)
if err != nil {
    return nil, ErrMyCustonIDGeneration("error message") // change the error name ;)
}

I've noticed a MongoDB dependency here as well, which confuses me because I thought the whole point of Clean Architecture was to eliminate direct dependencies and make the code modular. actually the whole repository package is hardcoded for Mongo which makes this an incomplete example.


[WIP] Add systemd unit for ubuntu live patching

What type of PR is this?
/kind feature

What this PR does / why we need it:
Add a systemd unit for doing security live patching for ubuntu.
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 79.178%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
2236

Relevant Lines:
2824

💛  - Coveralls


take control of conversation

@ankitbko
Hi! Thanks for the well explained guide!
I've got one question, is it possible to make the agent the one that triggers messages routing with the user instead of the user being the one that asks to talk with an agent?

Hey.
It is possible. To do any kind of agent-user chatting all you need is user address and agent address and map them to each other. For agent to initiate the message, he must know the user's address. The only prerequisite to this is that user must have chatted with bot at least once, so that you can save the user's address. The core concept of message routing remains same i.e. agent-user address mapping.
How I envision this is that, in agent's dashboard you can display list of users. Agent can click on any user and start the conversation.
Let me know if this works out for you.

Cool! thanks! I was thinking about the same, gonna try it out and I'll keep you updated

How I envision this is that, in agent's dashboard you can display list of users. Agent can click on any user and start the conversation.


Add some TypeScript types

:recycle: Current situation
Many TypeScript types are missing.
:bulb: Proposed solution
Import some types from @abandonware/noble and enable noImplicitAny.
This highlighted an issue here:
https://github.com/OpenWonderLabs/node-switchbot/blob/678103e/src/device/wosmartlock.ts#L36

Thanks @dnicolson!


Module com.ionic.keyboard.keyboard does not exist.

When I try to run this plugin on Android version 4.0.4 I get the following error:
Module com.ionic.keyboard.keyboard does not exist.
at addEntry line 1416, column 15 (file:///android_asset/www/cordova.js:1416)
at Object.exports.clobbers line 1426, column 5 (file:///android_asset/www/cordova.js:1426)
at onScriptLoadingComplete line 1709, column 30 (file:///android_asset/www/cordova.js:1709)
at HTMLScriptElement.scriptLoadedCallback line 1742, column 13 (file:///android_asset/www/cordova.js:1742)
I've got Raygun.io pointed at our app that's running in the wild and this error only seems to come up with this one version of Android. Does the plugin officially support 4.0.4?

Hmm strange, we don't really have an official stance on supported devices, so if there's anything I can do to get it working I'm happy to add it.
I'll be honest though, this sounds more like a Cordova bug, considering it works fine with other versions.  Are there any other plugins you're using that fail or is it just the keyboard? If it's just the keyboard, are there any differences in the plugin.xml files of the other plugins and the keyboard?


Eliminate wall wextra warnings

Hi, thanks for the development of this library!
It has been proving useful in the development of a ROS project Nao Soccer Sim.
I have eliminated all warnings when compiled with -Wall -Wextra for my project, and thought it may be useful for others also.
I have committed "per warning", so you can see exactly which changes were required against which warnings.

Hi @BitPuffin , I understand that this repository may not be actively developed, but I'd be interested in hearing your feedback on this PR.

Hello @ijnek thanks for your contribution. I have not deliberately ignored your PR. But rather it seems like github didn't email me until you @ mentioned me. (Could also be possible that I got an email but it got buried).
I'm very excited to hear about usage in your project. If you have feedback to report for future developments I am open to hearing them. It's true that I haven't actively developed the library but I have been tossing around the idea of making some improvements. Although I have also been wanting to avoid feature creep (hence the very low volume maintainence mode).
In either case I will have a look at the changes you've made and get them merged once I've reviewed them. Have a nice day!

Everything else pretty much looks great.
The f4b61a7d37df90d9fc39829f commit I'm uncertain if unsigned is the best choice. I would probably use size_t but I'm not convinced that it makes the most sense. Also since the general style in this project is to use auto very much maybe there is some initializer like 0u that we should use instead. I'm a bit rusty though with the specifics.
For 70c07847e16a34b3581286d9e9a4e48dac0bf976 it looks to me like I had made a mistake (I forget to put in break more often than I'd like). But did you run the tests and make sure it does not introduce bugs? Just in case it was intentionally not breaking. If it is supposed to be intentional I think we should have some comment saying that we want the fall through. But looking at the code I can't see why we would want it to fall through. But just wanna make sure :)

In regards to your first comment, I agree. I would suggest throwing a invalid_argument exception OR print to std::cerr ("error: Should never reach here") and return 0, in the default case. I don't know which you would prefer. I prefer throwing an invalid_argument exception, as the only way to fix the issue if it arises would be to modify the code.

The f4b61a7 commit I'm uncertain if unsigned is the best choice. I would probably use size_t but I'm not convinced that it makes the most sense. Also since the general style in this project is to use auto very much maybe there is some initializer like 0u that we should use instead. I'm a bit rusty though with the specifics.

Modified in 45a3b04 to go with 0u, per your suggestion.

For 70c0784 it looks to me like I had made a mistake (I forget to put in break more often than I'd like). But did you run the tests and make sure it does not introduce bugs? Just in case it was intentionally not breaking. If it is supposed to be intentional I think we should have some comment saying that we want the fall through. But looking at the code I can't see why we would want it to fall through. But just wanna make sure :)

Just ran both tests, test-sexpresso-std.sh and test-sexpresso.sh, and all tests pass :+1:

Hello!
Thank you for the fixes. Regarding the exception, I want the library to be smoothly usable for people who choose to disable exceptions, so I'd rather have it print to cerr.
As for the other stuff it's looking good!
By the way, since the library is public domain (CC0), would you be okay with waiving your copyright to the changes?
I don't think that with relatively small changes it's a big deal but I think it's better to be on the safe side!

Thank you for the fixes. Regarding the exception, I want the library to be smoothly usable for people who choose to disable exceptions, so I'd rather have it print to cerr.

Done!

By the way, since the library is public domain (CC0), would you be okay with waiving your copyright to the changes?

I'm not too familiar with CC0, but I've added // Modified: Kenji Brameld 2021 to the top of sexpresso.cpp and sexpresso.hpp. Tell me if that's not how you wanted it and I can fix it up!

I'm not too familiar with CC0, but I've added // Modified: Kenji Brameld 2021 to the top of sexpresso.cpp and sexpresso.hpp.Tell me if that's not how you wanted it and I can fix it up!

It's different from what I meant. CC0 is a public domain declaration, but you generally speaking by default have copyright on anything you make. So you have to explicitly give up copyrights in order to put something in the public domain.
So in practice, I suggest that we  make a CONTRIBUTORS file which contains a statement like.
Sexpresso is available in the public domain through the CC0 license (see COPYING.txt for details, or https://creativecommons.org/publicdomain/zero/1.0/legalcode).
The following people have made contributions to sexpresso and consented to dedicating their changes to the public domain:

Kenji Brameld, 2021

And then other people could make a commit with their "signature". Since they make the commit themselves it is probably good enough of a "signature" for our purposes. At least I hope so! :)

@BitPuffin
Should be good now, I've added my signature, and wrote a Contributing section in README.org

@BitPuffin in case you've forgotten about this.

Hey!
Sorry I hadn't forgotten I just had a hard time getting around to it. I checked the changes one more time and merged it now. Contribution very much appreciated! Sorry for being kind of slow on the ball. Good luck with the soccer ball project!


DatePicker: Add options to hide entire toolbar or year and date separately

Description
Resolves comment of @Garderoben  in PR #2722
Splits the toolbar hiding logic from the fixed-value parameters.
Adds new Parameters "HideToolbar" "HideYearInToolbar" and "HideDateInToolbar"
How Has This Been Tested?
Unit - Tests: Changed Fixed Values Tests for DatePicker to reflect the changes and added new tests
Tested example manually
Types of changes

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

It is not yet breaking, because the orignal PR is only on dev at the moment and not released.

Here sime screenshots: No Toolbar at all, hidden year and hidden date

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Sorry, we decided that this is not something we want to merge.
But thank you for your efforts

Its ok thanks for the review @JonBunator ... I will nonetheless create a new pr with the essential part which fixes a problem pointed out by Garderoben.
I was not aware of the possibility to hide the toolbar.


Fixes issue of pokeball growing in size

Fixes #143
What does this PR do?
Grows pokeball in size when the mouse hovers on it

Changes only style.css (addition)


Parallel tempering: avoid full re-evaluation when swapping solutions

Currently, solutions are swapped between replicas using the method setCurrentSolution(sol). This involves full re-evaluation and -validation of this solution, although its evaluation and validation are in fact already known. Use the protected method updateCurrentAndBestSolution(sol, eval, val) instead to gain efficiency.

Suggested change is not possible because ParallelTempering does not have access to the protected method updateCurrentAndBestSolution(sol, eval, val) from MetropolisSearch (inherited from LocalSearch).

Added new public method setCurrentSolution(sol, eval, val) to Search to allow the suggested enhancement.


Completely remove the redbaron dependency in the "context retriever" module

Altogether remove the redbaron dependency in the "context retriever" module.
I had to add a workaround of adding a py_module_initializer explicitly with Python's AST parsing strategy. Since the context retriever is only initialized when creating "documentation embeddings", we're going to init py_module_initializer with PY_AST strategy only during this time.
I had to do it because I didn't want the context retriever to be backwards compatible with redbaron.
The migration to the AST module is thus entirely scoped to this "context retriever".
As for the test plan, I am hoping for the CI to pass and will add a Unit Test.

@emrgnt-cmplxty PTAL, I am adding tests but let me know if the overall approach is OK to you, especially with the Doubleton py_module_loader class for now.

Closing after merging newer branch.


Crash on server launch

Hi, so as I tried to make the mod work server-side, I got this crash: http://pastebin.com/nfu7rVc4
I tried different versions of both the Voltz Engine and ICBM, without success. Can anyone help me find out what is wrong?

Duplicate issue https://github.com/BuiltBrokenModding/ICBM/issues/57


POOL-356 add unit test for the deadlock

Please review!
Will ship a first fix candidate soon.

Coverage increased (+0.3%) to 85.174% when pulling 016a1f67263fe1cde1d910dc7002d972811951c5 on struberg:POOL-356 into d4e0e88227ad91d8c8ef36ba01d656f71c770f83 on apache:master.

Coverage increased (+0.3%) to 85.174% when pulling 016a1f67263fe1cde1d910dc7002d972811951c5 on struberg:POOL-356 into d4e0e88227ad91d8c8ef36ba01d656f71c770f83 on apache:master.


Replicate search experiments with ES

Will circle back later to fix Twitter experiments as per https://github.com/castorini/anserini/issues/758

Quick +1 here so I can merge?


Mix of gen1 & gen2 path format in error message

Test Client> connect
connect to wss://<IP_ADDRESS>:8090
Websocket connected!!
Test Client> setValue Vehicle.Speed 200
{
    "action": "set",
    "error": {
        "message": "No write  access to Vehicle/Speed",
        "number": 403,
        "reason": "Forbidden"
    },
    "requestId": "4049604a-9c96-4483-b127-7e244f6debf9",
    "timestamp":<PHONE_NUMBER>584
}

testcases error:
https://github.com/eclipse/kuksa.val/blob/761420934fd7cf3497032565a21c937f35c7c0d3/test/unit-test/KuksavalUnitTest.cpp#L2121-L2136

@wenwenchenbosch  I assume this can be closed for now, as we should have repaired most (hopefully all) occurences of this with merged PR?

yes


Refactor colormap

This PR reworks silx.gui.colors.Colormap implementation and deprecates silx.gui.plot.matplotlib.Colormap (which is no longer used within silx).
It:

Removes the strong dependency to matplotlib in silx.gui.colors: closes #1859
Improves Colormap.getNColors  to return the same number of colors as the LUT (and not a hard-coded 256): closes #2217

        colormapObject = Colormap(name='red',
                                  colors=numpy.array([[1., 0., 0.],
                                                      [0., 1., 0.],
                                                      [0., 0., 1.]]),

We could talk how you we have to use the object, cause if colors are provided, the name at least could not be used (here there is an inconsistency). It should not be a problem for a custom lut to have no name. You provide colors, it's a custom LUT, else you provide a LUT name but it have to exists.

Ok, no pbm, it's fine. Colors are ignored (backward compatibility)

BTW, it's now  more restrictive in that you cannot create a colormap that has no color LUT (which was possible before).

name and colors arguments both set the color LUT... There should be only one argument instead of 2, but that would change the API...

LGTM


Error installing face_recognition in the window.

I am installing the face_recognition in the window. this is the command.
pip --no-cache-dir install face_recognition
I get an error while installing:-
`ERROR: Excepti
on:
Traceback (most recent call last):
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_vendor\urllib3\response.py", line 425, in _error_catcher
    yield
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_vendor\urllib3\response.py", line 507, in read
    data = self._fp.read(amt) if not fp_closed else b""
  File "\appdata\local\programs\python\python36-32\lib\http
\client.py", line 449, in read
    n = self.readinto(b)
  File "\appdata\local\programs\python\python36-32\lib\http
\client.py", line 493, in readinto
    n = self.fp.readinto(b)
  File "\appdata\local\programs\python\python36-32\lib\sock
et.py", line 586, in readinto
    return self._sock.recv_into(b)
  File "\appdata\local\programs\python\python36-32\lib\ssl.
py", line 1002, in recv_into
    return self.read(nbytes, buffer)
  File "\appdata\local\programs\python\python36-32\lib\ssl.
py", line 865, in read
    return self._sslobj.read(len, buffer)
  File "\appdata\local\programs\python\python36-32\lib\ssl.
py", line 625, in read
    v = self._sslobj.read(len, buffer)
socket.timeout: The read operation timed out

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\cli\base_command.py", line 153, in _main
    status = self.run(options, args)
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\commands\install.py", line 382, in run
    resolver.resolve(requirement_set)
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\legacy_resolve.py", line 201, in resolve
    self._resolve_one(requirement_set, req)
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\legacy_resolve.py", line 365, in _resolve_one
    abstract_dist = self._get_abstract_dist_for(req_to_install)
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\legacy_resolve.py", line 313, in _get_abstract_dist_for
    req, self.session, self.finder, self.require_hashes
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\operations\prepare.py", line 194, in prepare_linked_requ
irement
    progress_bar=self.progress_bar
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\download.py", line 465, in unpack_url
    progress_bar=progress_bar
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\download.py", line 316, in unpack_http_url
    progress_bar)
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\download.py", line 551, in _download_http_url
    _download_url(resp, link, content_file, hashes, progress_bar)
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\download.py", line 253, in _download_url
    hashes.check_against_chunks(downloaded_chunks)
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\utils\hashes.py", line 80, in check_against_chunks
    for chunk in chunks:
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\download.py", line 223, in written_chunks
    for chunk in chunks:
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\utils\ui.py", line 160, in iter
    for x in it:
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_internal\download.py", line 212, in resp_read
    decode_content=False):
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_vendor\urllib3\response.py", line 564, in stream
    data = self.read(amt=amt, decode_content=decode_content)
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_vendor\urllib3\response.py", line 529, in read
    raise IncompleteRead(self._fp_bytes_read, self.length_remaining)
  File "\appdata\local\programs\python\python36-32\lib\cont
extlib.py", line 100, in __exit__
    self.gen.throw(type, value, traceback)
  File "\appdata\local\programs\python\python36-32\lib\site
-packages\pip\_vendor\urllib3\response.py", line 430, in _error_catcher
    raise ReadTimeoutError(self._pool, None, "Read timed out.")
pip._vendor.urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='files
.pythonhosted.org', port=443): Read timed out.`

How can I install the face_recognition on the window?

pip install CMake.
Install VS community and under individual components -choose Visual C++ tools for CMake.

pip install face_recognition.

I know this method install's cmake twice but it works.
ps: pip install face_recognition will take a lot of time so be patient


Small README correction

Based on the changes to not run the container as root, this was missed in updating the docs.

Thanks!


WebGPU: Add texture format to shaderc (bin version 10) + streamline storage Images in shaders

This PR makes a bunch of changes to how storage Images are handled in shaderc to make them work with WebGPU:

Fix the garbage storage uniforms added in SPIRV output for uniforms that are not actually storage
Adds Texture format in shaderc uniforms output for storage images (only written in SPIRV path) as it is required to be provided when binding images in WebGPU
Cleanup and fix format decoration added in SPIRV shader paths (previously was something silly like spv::format_float4, now it's correct spv::format_rgba8)
Simplify the unnecessary wrapping of Images in BgfxXXImage2D_XX structs
Add SPIRV NonReadable decoration to write textures

I just realized that this PR would be a perfect opportunity to add another change to the SPIR-V shaders, while you're bumping versions :eyes:
This issue: https://github.com/bkaradzic/bgfx/issues/2410 requires getting rid of the different binding offsets in VS/FS for anything that isn't a uniform buffer. Any objections to the idea? Anything that could break WebGPU code? If not, I could try to prepare a follow-up PR by tomorrow or so...

Oh yeah, I saw that one. Just to be sure, you mean Samplers, Textures etc. will still have a constant shift, just the same in the vertex stage and in the fragment stage, is that correct ? Because I don't think there is any way to get rid of those shifts. But yes, the Fragment stage shift could probably go away. I'll still have to check that I have a way to make this work with WebGPU though

Thinking about this a little bit more, and looking at the constraints that bgfx only has 16 fixed stage for every kind of binding (textures, images, buffers), I think this current code:
	uint32_t bindingOffset = (stage == EShLanguage::EShLangFragment ? 48 : 0);
	shader->setShiftBinding(glslang::EResUbo, bindingOffset);
	shader->setShiftBinding(glslang::EResTexture, bindingOffset + 16);
	shader->setShiftBinding(glslang::EResSampler, bindingOffset + 32);
	shader->setShiftBinding(glslang::EResSsbo, bindingOffset + 16);
	shader->setShiftBinding(glslang::EResImage, bindingOffset + 32);

Can probably be simplified safely to this:
	uint32_t bindingOffset = 2; // Reserve two spots for the stage UBOs
	shader->setShiftBinding(glslang::EResUbo, (stage == EShLanguage::EShLangFragment ? 1 : 0));
	shader->setShiftBinding(glslang::EResTexture, bindingOffset);
	shader->setShiftBinding(glslang::EResSampler, bindingOffset + 16);
	shader->setShiftBinding(glslang::EResSsbo, bindingOffset);
	shader->setShiftBinding(glslang::EResImage, bindingOffset);

So I did try the change above on this branch https://github.com/bkaradzic/bgfx/compare/master...hugoam:spirv-bindings-simplify
And it does look like it works perfectly, but shader versionning will be ugly, because you then have to conditionally use the old or new shifts on each of the lines I modified

Thanks for working on this 🙇

Maybe we should keep this for a separate PR and another shaderc version bump 🤷

I figured it'd be nice for the WebGPU backend since you already require version 10 of the shaders so you wouldn't need any versioning code. But then you can always bump that up to 11, seeing how WebGPU is experimental anyway (and I assume SPIRV shaders won't be supported for much longer).
So yeah... let's not complicate this thing too much and do this in another PR 👍


After multiple timeouts, the authorization header may be invalid because it is too old

Package Name: azure-sdk-for-python
Package Version: azure_core-1.26.1 , azure_storage_blob-12.14.1
Operating System: linux
Python Version: 3.9.11

Based on timeout configuration, the azure sdk makes multiple attempts to reach the azure storage blob api.
Unfortunately, the authorization header is not regenerated each time with the current time/date.  If the retries result in a request after a cumulative delay of > 15 minutes, even a successful request is denied because the authorization header has expired.
To Reproduce
Steps to reproduce the behavior:

Induce artificial network packet loss (iptables drop packet to blob storage host)
Send a request and set the timeout to a large value (901 seconds)
After the initial request is dropped, remove the firewall rule
Wait for the sdk to perform an internal retry after the timeout

Expected behavior
Authorization header should be regenerated for every retry
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Thanks for your issue @zlevow, we'll take a look and get back to you as soon as possible.

Hi @zlevow, you are correct that this could happen. The real reason it could happen is not because we don't refresh the auth header each retry (we do) but because we don't update the Date header on each retry. So, it would then be possible that the request has a date header that is older than 15 minutes which the service will reject.
That being said, I'm not sure how big of an issue this is because I believe, given our default timeouts, retry counts, and retry timings, I don't think it would be possible to get a request that is older than 15 minutes. Our default retry policy has 3 retries with an initial backoff of 15 seconds and an exponential increase from there. This usually results in about a maximum for 90 seconds. We also have client-side network timeouts which default to 20 seconds for connection and 60 seconds for read. So, assuming each request times out based on network delay while reading (worst case), we'd still only be at around 270 seconds of total delay to get to the last retry.
So, with the SDKs defaults, I don't think this would really be possible to hit but I see in your example you are manually setting the timeout quite high to get the error. Is there an actual scenario where you'd want to be doing this? And/or have you hit this issue in one of your applications?
NOTE: Before blob version 12.14.0, the default read timeout was set very high, so it would have been more possible to hit this.

Thanks for the reply.  During periods of high congestion/packet loss on our network, medium sized uploads (200mb) would sometimes take > 300 seconds to complete. If there was a timeout during this period, a retry would be caused. Four of those retries would put us over the 15 minute mark. (we upped the retry count because of these periodic timeouts)

Hi @zlevow, assuming you are using the standard upload method, upload_blob with default settings, the SDK is performing larger uploads in a series of blocks that are 4 MiB by default, with each block being a completely separate request. If a timeout occurs, only that single block will need to be retried, not the entire upload. So, while the whole upload operation may take 300+ seconds, it is likely each individual request is not taking that full time and therefore it is less likely you would hit the 15-minute mark on a single block, even with many retires.
The default threshold for when a blob is split into chunks is 64 MiB, so if a blob is larger than that, it will be uploaded in 4 MiB chunks instead of all at once. Both this threshold and the chunk size itself are configurable on any client constructor using the max_single_put_size and max_block_size respectively. I believe they are also settable per individual operation. See here:
https://github.com/Azure/azure-sdk-for-python/blob/97a5929db5ba274229934e16725c8c77943aed84/sdk/storage/azure-storage-blob/azure/storage/blob/_blob_client.py#L121-L125
You can try playing around with these settings to avoid timeouts all together or just tune things better for your network conditions. Tuning these settings could help eliminate any possibility of hitting a request that is too old if there is one.


Add internal documentation to the FirebaseFirestore podspec

I think I should also add my json lib here as well?

Yes, before you merge you'll need to do that. If you're not seeing CI failures related to Xcode builds on that PR now you'll see them when you try to merge to master.


Filter for RecordDataBase

Implemented a filter function for databases.
Wrote a method for Base.filter() that takes a filter function, a database and, additionally, the level on which the filter function should be applied (Record, Study, Model, Taxon).
The filter function can also work with rating() or certainty() so the function definition works a bit more naturally, see documentation of filter().

Ignore the failure on nightly.


[FEATURE]: [Declarations] Show processing progress for added declarations

One Paragraph Description of the Feature
Show the progress for a declaration being processed:

[ ] Added

the declaration has been added for processing and its waiting to be scheduled with the processor
'added' icon

[ ] Processing

the declaration is being processed
'busy' icon

[ ] Processed

the declaration has been processed
'processed' icon
can navigated from the declaration entry in the list to the data validation editor

Describe the Feature
No response
Additional Information
No response

Duplicates #167


Facing inconsistent behavior while using subscription key for cognitive speech service in Chrome and Edge browsers.

Hi
When i am using subscription key for congnitive speech service than i am getting random behavior in converting text to speech means sometime it is converting or sometime not in chrome and edge browser. Please help me out as soon as possible.

Assigning this to @compulim for investigation

We should engage the cog svc team.  this seems like it might be their issue to address.

Hi @saurabhgupta1090, could you give us some contexts on it?
Do you have repro steps for the problem?
Do you see anything in the console? You can open console in developer tool by F12 and refresh the page.
If your bot is public, could you send us the URL so we could try out on our end?
It would be great if you could fill out our bug template, which give us more information on diagnosing the issue.

Hi compulim,
Thanks for your response. Our bot is not public so i can not share the URL. Actually we are using this sample code base for integrating speech service in our bot
https://github.com/microsoft/BotFramework-WebChat/tree/master/samples/06.c.cognitive-services-speech-services-js
Regards,
Saurabh Gupta

@saurabhgupta1090 Sorry but we'll be unable to help without more information. Are you able to provide repro steps or the console errors?

Closing due to inactivity.
Would still love to have the customer report about their findings based on our questions posted here.


Update to runtime v0.13.0

Description of changes:

Updates to runtime v0.13.0. Fixes all issues with test-failures occurring as a result of updating the runtime.
Fixes Notebook instance update, so that status is correctly updated.
Utilizes late_initializer for new diffs that occur as a result of using reflect.deepequal().
Adds new custom_delta.go code for diffs that cannot be handled by late_initializer.

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

/test sagemaker-kind-e2e

/test sagemaker-kind-e2e

/test sagemaker-kind-e2e

/test sagemaker-kind-e2e

/test sagemaker-kind-e2e

/lgtm


Feature/craig/migrate sync to work manager

Task/Issue URL: https://app.asana.com/0/488551667048375/1147610969828115
Tech Design URL:
CC:
Description:
Migrates the app configuration sync to WorkManager and stops using JobScheduler APIs directly for it.
Steps to test this PR:

Checkout develop, and modify the times in JobBuilder to use ridiculously short times for periodic and backoffCriteria (like, a few milliseconds kind of short)
Install and launch. ensure the initial syncs finishes, so that a Job is scheduled with JobScheduler
Checkout this branch. Similar to before, set short values for periodic and backoff times in AppConfigurationSyncWorkRequestBuilder
Install and launch. Verify initial sync still happens when app first launches
Verify after app sync finishes, work is scheduled with WorkManager.
Wait for sync to happen again. Depending on the OS, this might take ~15mins ☕☕. Now is a good time to get Charles ready. We want to test failure path and success path, so you can set a breakpoint on one of the sync requests in Charles.
When the sync happens, kill a request in Charles to produce a failed sync. Verify it tries again, as per the retry policy
Let all requests succeed, and verify sync completes successfully

Internal references:
Software Engineering Expectations
Technical Design Template

@CDRussell develop has been updated after this PR was opened and caused DB migration issues. I've just merged and pushed so we can continue the review and the steps to test.

We've had a call to discuss all the changes and run through the steps to validate this is working as expected.


remove ornlm dependency

see issue here for people having OS-related difficulties to make ornlm work:
https://sourceforge.net/p/spinalcordtoolbox/discussion/help/thread/3bf1555b/?limit=25
should be fairly easy to add a flag to remove denoising (or use another method that does not rely on ornlm).

There is a flag for denoising, is the issue occurring even with  -denoising 0 ?

great! thanks


SCRUM-1068 Instantiate NCBITaxonTerm and update references

@chris-grove - agreed, I'll make the above changes.
There are other 'range' redundancies that can be removed.  At the moment, the required status for the experimental condition slots is declared in the class - can these also be moved to the slots or is there potential for these to be used (and non-required) elsewhere in the model.
Finally, I'm trying to work out what to do with condition_chemical.  This currently has a range of OntologyTerm, but the Molecule class is not an OntologyTerm.  We could create a 'Chemical' class that both CHEBITerm and Molecule inherit from, but this would stop CHEBITerm from being an OntologyTerm and I'm not sure if that makes sense (either from a logical or a database point of view).

@chris-grove
So if I understand correctly, the proposal is to create a ChemicalOntologyTerm class which is an OntologyTerm and would have the various fields that are currently preset on the Molecule class, such as iupac, inchi, etc..  Then, both Molecule and CHEBITerm classes would inherit from ChemicalOntologyTerm.  Is that right?
I think that makes sense.  We can load the iupac, inchi, etc. from the CHEBI ontology file.  The only minor issue I see is that the Molecule class entries would inherited some fields from the OntologyTerm class that are not applicable (although also not required), such as namespace, definition, definitionUrls.
Or have I got the wrong end of the stick?

@chris-grove
I've pushed some changes that should address all your comments.  I'll leave the conditionChemical issue for another PR as that is unrelated to SCRUM-1068.


Update 1162.json

Add supported model list 42TSN012-703
Add supported presets
Add more operationModes and fanModes

I reverse-engineer my ac remote to generate all these modes with exceptionally clean signal.
Note here is that while the remote support dry mode, the codes is actually cool mode

Don't merge this yet, there are still some codes that doesn't work.
In presets mode `'none'`:
  fanspeed low and medium doesn't work when using cool or dry
In presets mode `'eco'`:
  fanspeed low and medium doesn't work when using auto
  operation dry and cool doesn't work

I used to debug from the orther pr that has double commands bug that cause me not notice that it doens't work.


🌱  Bump capi to v1.4.6 and cert-manager to v1.12.3

What this PR does / why we need it:

Uplifts CAPI to v1.4.6
Uplifts Cert-manager to v1.12.3

/test-ubuntu-e2e-integration-release-1-4


chore: re-generated to pick up changes from synthtool

This PR was generated using Autosynth. :rainbow:
Synth log will be available here:
https://source.cloud.google.com/results/invocations/71a02c6b-8ba2-4db0-b76d-9e42934df8a3/targets

[ ] To automatically regenerate this PR, check this box.

Source-Link: https://github.com/googleapis/synthtool/commit/dd230c816f88d0141fcd0be83498986287220d1b

@thiagotnunes I feel like this is adding kokoro configs that were previously deleted in #367. If so, we can just close this PR.

@skuruppu we just merged the change to remove the unused kokoro files from the template repository (see https://github.com/googleapis/synthtool/pull/692). Maybe we could run the tool again, so that the deleted files are not brought back?

I clicked the checkbox. We'll have to wait a bit for it do its thing.


Use multiple highlight colors for variables.

This makes life a lot easier if you want to highlight multiple variables in a single algorithm.
Inspired by the similar bikeshed feature. Bikeshed also has a feature to keep the same color when you unselect and then reselect a variable; I didn't port that.

FYI, these are the new highlight colours, in order:

And this is the current one:

@Ms2ger Where did this colour list come from? Why change the primary one?

Colors also come from bikeshed, with the following comment:
    # Colors were chosen in Lab using https://nixsensor.com/free-color-converter/
    # D50 2deg illuminant, L in [0,100], a and b in [-128, 128]
    # 0 = lab(85,0,85)
    # 1 = lab(85,80,30)
    # 2 = lab(85,-40,40)
    # 3 = lab(85,-50,0)
    # 4 = lab(85,5,15)
    # 5 = lab(85,-10,-50)
    # 6 = lab(85,35,-15)

I'm happy to go any way you prefer.

I don't think these colours are readable enough as-is. Compare the current

to the proposed primary colour

The secondary colour is even more egregious:

I think we should either choose a different set of colours that contrast well against our current alias text colour (#2AA198) or include a contrasting text colour in each of the referenced classes. If we want to go the former route, it looks like this tool does exactly what we'd want: https://learnui.design/tools/accessible-color-generator.html. If we go the latter route, I'd prefer to limit the text colour to white(-ish) or black(-ish).

Oh right, I forgot that bikeshed uses plain black for variables. That seems fine for all of them:

If you'd rather change the background colors, I'd prefer if you could come up with a list.

I've pushed a couple of commits which 1) make variables use the default almost-black text color when highlit, which addresses the contrast concern and b) restores the original color for the first variable, since I think it has adequate contrast with the other colors and hence does not require changing (and potentially confusing readers about why it's a different color than they're used to).
I think both changes look fine, personally, and I don't really want to try to find 8 colors which contrast well with the our variables' default greenish color. (The tool linked above produces very pale, frequently similar colors when asked to find variations of this set of eight colors which contrast adequately with #2AA198.) @michaelficarra thoughts?

Works for me, though I do think white contrasts better with some of the chosen colours than black. But it's probably better to just be consistent with black text. Either way is fine for me.

Thanks @Ms2ger!

Thanks for the help!


Unable import project on IntelliJ

I'm having trouble to import an projetct to IntelliJ. I got that message:
Please, use JDK instead of JRE for Gradle importer.
IntelliJ version 2017.3.5

Hi @castrors! I'm not a maintainer for Kotlin, but have you tried this help page. It looks like the same error you've had, and seems to explain it pretty nicely :slightly_smiling_face:
It's probably also worth knowing the difference between the JDK and the JRE if you don't already - here is a great StackOverflow answer explaining the differences :slightly_smiling_face:


Allow customisation of max-jobs instead hardcore to a maximum of 8

See original issue on GitLab
In GitLab by [Gitlab user @jjardon] on May 23, 2019, 16:59
Or, as a minimum, bump the max to something else bigger than 8
This has probed to make things slower in machines with a lot of cores (96) but relatively slow, like our aarch64 runners
https://gitlab.com/BuildStream/buildstream/blob/master/src/buildstream/_project.py#L808

In GitLab by [Gitlab user @jjardon] on May 23, 2019, 17:00
mentioned in merge request freedesktop-sdk/freedesktop-sdk!1333

In GitLab by [Gitlab user @jjardon] on May 23, 2019, 17:06
changed the description

In GitLab by [Gitlab user @jjardon] on May 23, 2019, 17:06
changed the description

In GitLab by [Gitlab user @nanonyme] on May 23, 2019, 20:14
Something to consider is whether max-jobs should be givable on command line. This might be nicer for CI environments where you know exactly how many CPU's you have.

In GitLab by [Gitlab user @abderrahimk] on May 24, 2019, 07:59
marked this issue as related to #185

In GitLab by [Gitlab user @tpollard] on May 30, 2019, 10:49
[Gitlab user @nanonyme] I think it makes sense to add it to the bst main options

In GitLab by [Gitlab user @tristanvb] on Jul 14, 2019, 12:04
mentioned in commit f2611f14fb9f9f48aaed3b7b5443f12147926911

In GitLab by [Gitlab user @tristanvb] on Jul 14, 2019, 12:04
mentioned in commit d4beadfc824d577905b0f0fec5d60e66556c2f32

In GitLab by [Gitlab user @tristanvb] on Jul 14, 2019, 12:04
mentioned in commit 984f39366998e89f524f077e9fc2a2a60a0a42f7

In GitLab by [Gitlab user @tristanvb] on Jul 14, 2019, 12:04
mentioned in commit d30b65109b70451bd215fa664cb502496fc59b02

In GitLab by [Gitlab user @tristanvb] on Jul 14, 2019, 12:04
mentioned in merge request !1473

In GitLab by [Gitlab user @tristanvb] on Jul 14, 2019, 12:30
mentioned in commit 5ab0aa9b72b4d5a950a3e6f0f50e6ed5d2d50012

In GitLab by [Gitlab user @tristanvb] on Jul 14, 2019, 12:30
mentioned in commit c13024b4ae801dbbf2d2106ff1c7850edc675722

In GitLab by [Gitlab user @tristanvb] on Jul 14, 2019, 12:30
mentioned in commit f8f542a9ebbfc50b80df3eaaecfd24d685c7aacf

In GitLab by [Gitlab user @tristanvb] on Jul 14, 2019, 12:30
mentioned in commit 32a4ebbfa9045c305fae4b701947cf6386d121c6

In GitLab by [Gitlab user @tristanvb] on Jul 14, 2019, 12:31
mentioned in merge request !1474

In GitLab by [Gitlab user @tristanvb] on Jul 15, 2019, 15:19
mentioned in commit fa127a5c02e0793e8ccf95bbae1f13dde4a54d7d

In GitLab by [Gitlab user @tristanvb] on Jul 15, 2019, 15:30
mentioned in commit cadc772b9b7cd664420d094726035e1464aa12d1

In GitLab by [Gitlab user @tristanvb] on Jul 15, 2019, 15:30
mentioned in commit 0a19dfd7b0a7684e39bdab9c50d6ab97522fd9a1

In GitLab by [Gitlab user @tristanvb] on Jul 15, 2019, 15:30
mentioned in commit 4dc5b116e6d8cbdf124af28aa1eeea078f485400

In GitLab by [Gitlab user @tristanvb] on Jul 15, 2019, 15:30
mentioned in commit 35ecd21e9449988d54826de6234be343fb1a3f37

In GitLab by [Gitlab user @tristanvb] on Jul 16, 2019, 04:19
mentioned in commit c068f54fc7fe694928239a6218fe6661a67a95a5

In GitLab by [Gitlab user @tristanvb] on Jul 16, 2019, 04:19
mentioned in commit 03760eafab52ef75bc7716cd778f7455efeb35cc

In GitLab by [Gitlab user @tristanvb] on Jul 16, 2019, 04:19
mentioned in commit 94f492598a6402a0ad05bdb682769c63f8206e29

In GitLab by [Gitlab user @tristanvb] on Jul 16, 2019, 04:19
mentioned in commit c915c8b7c4f067346a03be597fd1451b2ceebc00

In GitLab by [Gitlab user @marge-bot123] on Jul 16, 2019, 05:06
closed via merge request !1473

In GitLab by [Gitlab user @marge-bot123] on Jul 16, 2019, 05:06
mentioned in commit 385e3719c7e6dc15f72c0acaca54622b3c073ecc


Create a SyncStatus helper type

Motivation

For some mempool tasks and to decide if new block commit advertisements should be sent, it is necessary to know if the synchronizer has finished and is at (or close to) the chain tip.
Solution

A helper type to determine if the synchronizer is close to the chain tip or not was created, and it has methods to check if it has likely reached the chain tip and to wait until it reaches the chain tip.
Review

@teor2345 was already following the development of this as part of the mempool crawler PR.
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

Follow Up Work

I added a proptest to check if the SyncStatus was working properly when waiting for the tip to be reached. The test became a bit complicated, because there are two concurrent tasks working together, but I'm hoping it should help capture any regressions in the future.
I limited the number of proptest cases so that it runs in around 10 seconds locally. We could reduce this number further or reduce the EVENT_TIMEOUT if we want to speed it up.


dear murphy,

Could you help me please, how can I collect port statistics every 10 minutes from my topology (number of packets at each instant)

The POX manual links to a statistics collection example which may be of help:
https://noxrepo.github.io/pox-doc/html/#statistics-collector-example


Accuracy of the models

I have been experimenting the app and making some models with training data.
However, no matter what settings I change or what model I use, I can’t get an accuracy (or other metric) greater than 0.55.
I used every model available and change a the settings to optimize the algorithm but the highest accuracy I got was 0.55!
Has someone got a higher accuracy on training data?

Hey, have you found a way to increase accuracy? If so what did you do?


add bug fix for table metric

add matching idx logic fix in table_assignment.py

merge request as test passed

change the draft to real PR and merged the latest changes by 20240523 to the branch


User: Authentication

via OAuth (preferably Google)
This is only necessary for admin users. Non-admin users should NOT be able to log in.

FYI, promoting and demoting from admin can just be done from the console for now


Request to add DNS server implementation for sidecar

Added DNS server implementation for sidecar , include testing and configurations.

Thank you for this PR and this valuable feature addition. I would like to make sure that we reuse the local registry cache instead of making a remote API call for every query.
@zvi not related to this PR, but we need to think about how SRV records figure in this scenario. It would mostly require merging rules that contain backend weights and priorities.
At some point I would like to convert our examples to use DNS as well.

I would suggest moving the DNS resolution capability to the registry instead of the sidecar. When a service is using DNS-based service discovery (e.g., Consul, SkyDNS), then it makes sense to move the DNS server to the control plane, instead of having a DNS server for every sidecar. Is there a use case that I am missing here?

After discussing with @tomerGolany, here are the proposed changes to support SRV records:

The query syntax will be modified from [tag]*.<service_name>.<domain>. to one of the following:

[tag or protocol]*.<service_name>.service.<domain>. (A/AAAA/SRV service lookup)
_<service_name>._<tag or protocol>.service<domain>. (SRV service lookup per RFC 2782)
<instance_id>.instance.<domain>. (A record lookup for a specific instance).

The answer to an SRV query (either via query syntax 1 or 2 above) will be an one or more SRV records, each with the target field set to <instance_id>.instance.<domain>. (syntax 3 above). The answer will also include (in the "additional" section) the A records corresponding to these targets. For example:QUESTION SECTION:
_myservice._http.service.consul.     IN  SRV

ANSWER SECTION:
_myservice._http.service.consul.  0   IN  SRV 1 1 9080 98q7wfah4.instance.amalgam8.

ADDITIONAL SECTION:
98q7wfah4.instance.amalgam8. 0   IN  A   <IP_ADDRESS>

The tag part of the query has been expanded to support both regular tags, as well as the protocol (endpoint type). For "tcp", "udp", "http" and "https", the tag is interpreted as protocol, whereas all others are tags.
Priority/weight will be uniformly assigned to 1 (at least until we figure if/how to support routing rules).

@zcahana "Priority/weight will be uniformly assigned to 1 (at least until we figure if/how to support routing rules)."
-- Why?
The sidecar already has a local copy of the rules. If there are gRPC clients that are using this sidecar for resolving their DNS SRV requests, they can take advantage of the weights and priorities for version-based routing.
Also, FYI: a simple DNS server in nginx: https://gist.github.com/agentzh/6c50d37510daef792ed220fa0d970393

@rshriram see here.

I've added a bunch of commits addressing all recent review comments (+ some additional refactoring and bug fixes). Also opened amalgam8/amalgam8.github.io#89 for documenting the DNS function.
@rshriram I'm approving my own changes, please approve your review as well so we can merge.


Update machine-learning-algorithm-choice.md

Edit in typo for the word problem in the document.

Hi @jaymathe, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (jaymathe). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


DiscoNet test result : Error in IoU 0.5 & 0.7 reults

I would like to thankyou for your great work & consistent support.
I have raised some concerns before in #33 and got a quick fix by @dekunma, which I am very thankful of.
I have noticed that the mAP values for each agent at IoU thresholds of 0.5 and 0.7 are exactly the same, which is unusual.
Also, please correct me if I am wrong, the average local mAP of all 5 Agents is somewhat different than what I calculated normally.
average<EMAIL_ADDRESS>= (0.6579220294952393 + 0.7744371891021729 + 0.7319439053535461 + 0.7088317275047302 + 0.7414842844009399) / 5
= 0.7229238271903257
This the test result which I am referring to:

Please have a look.
Thank you

Thank you for pointing this out. It was a silly typo that resulted in each agent having the same mAPs at 0.5 and 0.7. I never noticed this since we only focused on the average local mAP of all agents.
For the average mAPs, you are right that they are not the arithmetic average of the mAPs of all the agents, because the average mAPs were calculated collectively based on the detection results from all agents, instead of individually.


Add the ability to add a user with a Tag using the Add-TeamUser method.

When adding a lot of users to a Team would like to be able to add a tag as well, currently have to go into Teams to create or tag users once they are added.

Hello @rickmcguire Rick,
Here we are limited to documentation issues and fixes, for feature request you can use https://feedbackportal.microsoft.com/feedback/forum/ad198462-1c1c-ec11-b6e7-0022481f8472


zod dependency mismatch between GitHub and npmjs

On npm, schema-stream is pinned to 3.22.4, whereas on GitHub, it's ^3.23.3, producing peer dependency errors.

Appears to be because you version after the fact, and so GitHub's code is actually 3.2.0's code, but unversioned for the new one.

sorry for the delay - just merged it in :)


fix: ignore complete dist folder

@Joelant05 do you know why part of the dist folder was not included in the git ignore?


Add Edge Zone SIG definition

What type of PR is this?

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

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[x] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

cc @yunrugemicrosoft


Object/Array typed properties should not fire changed event when they where null and are assigned null again (no change)

Description
I honestly believe this is an issue in 2.0-preview, but I'm new to Polymer.
I think that changes for properties of type object/array should not fire a changed event when they where null and are assigned to null again. Is this correct?
Proposed fix
This is the change I did to fix it (the last line), am I missing something?

      _shouldPropChange(property, value, old) {
        return (
          // Strict equality check for primitives
          (old !== value &&
           // This ensures old:NaN, value:NaN always returns false
           (old === old || value === value)) ||
          // Objects/Arrays always pass
          (typeof value == 'object'
                    //don't notify when it was null and is assigned null again
                    && !(old === value && value === null))

Some background
I forked and migrated the vaadin date picker to use Polymer 2.0-preview (I'm not affiliated to vaadin in anyway, I just wanted to know how painful would be to migrate a complex component) and was getting a very weird behavior where the value of the input was being reset when scrolling the calendar because a property that "never changed" was firing its property changed, I spent an entire afternoon trying to pinpoint the problem and the only thing I can say is that I believe is related to two way binding marking the selectedDate as changed when nothing had changed.
No Live Demo :'(
I'm aware that without the live demo and steps to reproduce usually is very time consuming to fix something, but I believe this is a simple question "should  Object/Array properties fire a changed event when they where null and are assigned a null again?". If a live demo would help I can try to do something simplistic.
Browsers Affected

[X ] Chrome (I have only tested in Chrome)
[ N/A] Firefox
[ N/A] Edge
[ N/A] Safari 9
[ N/A] Safari 8
[ N/A] IE 11

Versions

Polymer: 2.0-preview
webcomponents: v1

I think it's not an issue with Poly 2.0
As it's on README

To improve compatibility with top-down data-flow approaches (e.g. Flux), we no longer dirty-check properties whose values are objects or arrays.

Personal opinion -> I think it's ok, polymer should dispatch events every time a property is set

@MartinsThiago I had seen that in the README and I'm not proposing to go back to dirty checking objects or not supporting implementations of Flux like Redux or any other, and I don't even want start a war on whether going from null to null should be considered a mutation or not.
What I am saying is that I don't think that notifying a property change from null to null was the code's author intention, it can have a very high cost and I don't see any benefit (even Redux tools like redux-freeze and redux-freeze-state  don't freeze null properties)
I have to admit that I don't use any Flux implementation and I may be missing something.
Maybe @kevinpschaaf or @sorvell could say if that was truly the intention.

I believe that @erichmx is correct and this is unintentional. No dirty-checking objects is designed to allow subproperty changes to flow through. But null doesn't have any subproperties; null is always just null.
FYI, the core team are looking at changes that might affect this code, so it may be a little while before they get to this issue.

Thank you for your answer @arthurevans. I just saw the branch where, as you said, the team is doing a lot of work on this code!.
I'll just add that I saw a performance boost when I changed this, and the changed events fired went down to 30% in this (too specific) case, from 2,100 to 700.

Here's a demo of the issue: http://jsbin.com/wejurab/edit?html,console,output.

@erichmx Sorry for the delay on this.  Agree it does seem best to treat null as a primitive (as opposed to object) for purposes of dirty checking and do a strict equality check.  Just put a PR up for this change.  Thanks for pointing it out.

Fixed via https://github.com/Polymer/polymer/pull/4340.

@kevinpschaaf looks like you already completed the change, thank you! and my apologies for not contributing a solution this time, you were working with a couple of branches with many changes on the dirty checking .


Run ML-powered queries on Windows with CodeQL CLI 2.9.0+

Versions 2.9.0+ of the CodeQL CLI run machine learning models from a temporary directory, which resolves an issue on Windows where TensorFlow models are not correctly loaded due to the absolute path of some of their files being greater than MAX_PATH (260 characters).
We should wait until all virtual environments including CodeQL CLI 2.9.0 have been fully rolled out before merging this PR to avoid alert churn from Code Scanning being run intermittently on the new and old bundles and therefore intermittently running with and without ML-powered queries on Windows.
Merge / deployment checklist

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.
[x] Confirm the changelog has been updated if necessary.

Ready for review, but not ready to be merged yet per the PR description.