Display console outputs for all tests

Otherwise you can't see console logs of passed tests

This will let us see the output of changes like #146

You did not ask me for a review of this. Perhaps you were only doing this as a test? I don't think it is a good idea if you really want this added, it makes the output too verbose. I can give more commentary if you like.

I wanted this because I was reviewing #146 and I can't see the changes in behavior re the warnings. For that example the test runs are basically completely opaque with no console outputs. https://github.com/ros-infrastructure/rosdoc2/actions/runs/10658077327/job/29538644576?pr=146
For automated tests like this being able to go back and find the differences in behavior is valuable. These tests are going to be the only arthives we have of these results. We'll be searching them anyway, and the summaries of the errors and failures still will be generated. This is something that the ROS buildfarm errs on the side of very verbose. But then it helps track things down much faster from failures when they occur, because you can do thiings like compare it to the last know success.
I'd be happy to hear more of your thoughts though.

In the particular PR that you mentioned, the "changed behavior" is that the revised tests succeed, while without the change they fail.
See this run for what happens if you just have the changes to the test, without the change to the core code:
https://github.com/rkent/rosdoc2/actions/runs/10778128703
Isn't that the normal pattern for test-driven development? A reviewer could confirm that by updating the tests without updating the underlying code, which is what the patch does. I can't always guarantee that every change I do will have an associated test, though I try. But #146 does not seem to me to be a good example of the need for more verbosity.
What I would support is using pytest --log-cli-level=WARNING instead. That will show logging warnings, which for some of the other tests are interesting, desirable results.
But that does not help #146. The warning there is buried in conf.py, which for the current implementation is not using the standard logging package. I added a 'warning' there using print(f'[rosdoc2] *** Warning *** which followed the existing practice within that file. We could try to add logging to the messages in conf.py, but I think that once #139 lands, those logging messages are now behind a subprocess again, and I am not sure that pytest will successfully grab them with --log-cli-level. But I have other pending PRs where I have to add warnings to conf.py, as I really want to stop the practice of using raise RuntimeError there which means no documentation at all is produced, so perhaps it would be worth adding standard logging there.
So generally I am not a fan of adding output verbosity for edge cases, as it makes more work in normal cases, and tends to obscure problems more than highlight them. (A particular peeve of mine is the --verbose in the rosindex Makefile, which generates copious amounts of useless output which actually slows the builds. I removed that in my branch, which I will probably propose in the official version as well).


[$250] Split - User profile opens every time the cursor is dragged outside input and released

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.71-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
Go to group chat
Go to + > Split expense
Enter amount > Next
Drag the split amount in the input field from left to outside the input on the right (the cursor touches user row)
Release the cursor

Expected Result:
User profile will not open
Actual Result:
User profile opens every time the cursor is dragged outside the input and released
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/d6d2b070-4b18-4e90-98e7-27e362ca8d23
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ac128a48bb83d037
Upwork Job ID:<PHONE_NUMBER>102068736
Last Price Increase: 2024-05-09

Issue OwnerCurrent Issue Owner: @c3024

@deetergp  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-split

cc @youssef-lr but I think we can remove the blocker on this one. Seems extremely minor.

To be honest I don't see where is the issue, I think we can even close for now. What do you think?

Hm I think it's a bit annoying for sure, but not blocker worthy.

Seems to be triggered from the mouseUp event when the cursor is on the pressable row.

@c3024 can you please check the proposal above

@dragnoir 's proposal here is a workaround.
I think we should wait for a proposal that can identify and address the root cause correctly.

@dragnoir
Could you explain your root cause more clearly?

We don't have a way to to manage the state of mouse interactions across components.

Are there any other components where a touch started from one component but triggers press on another component as moved beyond the component that made you deduce the issue is not with these components but need a way to manage state of mouse interactions?

This is the first time we have an Input inside an interactive component MenuItem

Could you provide lines of code for this MenuItem?

Those issues are cases where the parent gets swiped even when it is within the bounds of the child.
Here, as long as the selection is within the text input the press of OptionRow does not trigger so this is different.
Fixing this does not need a separate context nor mouse states.
You can have a state to check if the touch started in input in OptionRow and pass the state setter to textinput. You can use the onPressIn  of textinput to set the touch started in input to true and return onPress of optionRow early based on this value. Before this returning you can set the touch started in input back to false.

@c3024 what do you think?

what do you think?

MoneyConfirmationList used OptionRow previously. Now it is changed to SelectionList with UserListItem with MoneyRequestAmountInput as right hand component. It is a bit different because earlier MoneyRequestAmountInput was a direct child to OptionRow. I am checking how we can do with these new items.
@dragnoir
Could you update your proposal considering the new changes in MoneyConfirmationList and share a branch of the working code changes?

Waiting for proposals

In which cases is the onMouseLeave in BaseListItem useful?

@dragnoir thanks, please update this solution in your proposal comment.

@dragnoir 's proposal here with test branch here looks good to me.
🎀 👀 🎀 C+ Reviewed

@dragnoir update on PR?

The PR that introduced the bug has been identified. Link to the PR:
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

This is a new feature and this bug is specific for Web browsers/Desktop. No specific PR can be held responsible for this bug.

A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

This is a specific case of mouse dragging on one element perceived as click in another element. This is an uncommon bug and a discussion on the channel does not appear to be useful.

Determine if we should create a regression test for this bug.
If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Go to any group chat
Click on the + beside the composer > Split expense
Enter amount > Next
Drag the split amount in the input field from left to outside the input on the right (the cursor touches user row beyond the text input)
Release the cursor
Verify that the user profile is not opened

@muttmuure this is up for payment. Thanks!

Paid


I can see the button's text in interactive leaning  but unable to displayed on the custom UI

Thanks for the question, our Github issue board is for bugs and feature requests only, though 😕
Please ask this question in the forum✨
If you were describing a bug, please create a report and fill out the template


Add typing to the codebase

Python typing is a good way to rapidly check for coding issue while developing and it also allows to generate documentation with types automatically.
It's a good first issue for someone who wants to contribute to PL.

I like this and I'm happy to help out here. My suggestion is to do it gradually and split into multiple PRs, e.g., starting with Trainer and LightningModule, then Loggers and so on.

@awaelchli thx and happy to see more your PRs soon :]

awesome! let's keep this going as we keep making fixes

It is not done yet lol

@hadim I am looking at adding more typing to the rest of the code base and I am facing this issue again with failing imports (e.g. in tests). I think it is related to forward referencing / cyclic references.
For example, in the callbacks I need to import the Trainer and Lightning classes to type the callback signatures, which somehow fails at runtime.
Have you faced such issues before?

No, I haven't but an easy fix is to use types as string. Instead of:
class Node:
    def next() -> Node:
        pass

Try:
class Node:
    def next() -> 'Node':
        pass

This StackOverflow response adds more context to @hadim's response:

https://stackoverflow.com/questions/33533148/how-do-i-specify-that-the-return-type-of-a-method-is-the-same-as-the-class-itsel

In short, that's an upcoming feature of Python 4.0. The most backward compatible workaround is the solution suggested by @hadim above.

Thanks for your suggestions. With just the string method there are no errors, but my IDE (pycharm) is complaining and suggesting that I still need to import Trainer (when I declare trainer: 'Trainer'). However, when I add this import and run tests, it fails to import.
The stackoverflow post doesn't say if the import is needed or not. Probably it does, otherwise how would the type system know which class it is?

Wow, is there road map for Python 4.0? What will be the last 3.x series? @luiscape

Wow, is there road map for Python 4.0? What will be the last 3.x series? @luiscape

I guess we can close it as we are continuously adding with every new PR...


2.0

Current coverage is 35.92% (diff: 23.52%)

Merging #122 into 2.0 will decrease coverage by 0.02%

@@                2.0       #122   diff @@
==========================================
  Files           225        225
  Lines         12585      12597    +12
  Methods           0          0
  Messages          0          0
  Branches       2131       2132     +1
==========================================
+ Hits           4524       4525     +1
- Misses         7490       7501    +11
  Partials        571        571

Powered by Codecov. Last update abfbdb8...7468bf1

@rajsarkapally-sfdc
Can you go through it and merge ?


Beta release

Description

Makes more user input checks for mnemonic (only allows alphabetic characters and spaces)
Also trims the start and end of the mnemonic phrase
Added a text for no transactions if there are no transactions on the account

@Daniel-Wang looks good. very minor change.

LGTM! 🚢


Write a Java program to print ascii value of a string

Description
Write a Java program to print ascii value of a string
ASCII value is the numeric representation of a character. For example, the ASCII value of the character 'A' is 65. The ASCII value of the character 'a' is 97. The ASCII value of the character '0' is 48. The ASCII value of the character ' ' is 32.
Input  : ABC
Output : 65 66 67

How to contribute

Save the solution in PrintAsciiValueOfAString.java file
Add PrintAsciiValueOfAString.java file in print-ascii-value-of-a-string folder

Hey, Can I work on this?

!assign


Gear Checks Back Tinker

Gear Check says I'm missing 1 Tinker for Back slot even though I have "Lightweave Embroidery" from Tailoring on it (highest version).
Game Version: Cata Classic
English Client

Are you also an engineer?
Engineering tinkers stack with regular enchants.
I might have to have special handling if Tailoring enchants do not count as a regular (Enchanting) enchant and compete with Engi tinkers (ie you can only have one of the two)

I confirmed with a guildie, that's tailor/engi. You can have both so addon is not wrong 😉


Extract normative/informative references from a BikeShed spec

Given the URL of a document generated with BikeShed, the goal is to extract the list of specs that this document references normatively/informatively and to produce a JSON structure with that list.
References should be identified with their ID in specref whenever possible.
Note one potential difficulty is that the starting point is not the source document, but rather the generated one.

I have pushed a first script that does the job; I'm working on the generated output rather than the source bikeshed for two main reasons:

it's easier to find (since that's the one that's linked from other specs)
it's much easier to parse - bikeshed documents can be written in markdown, and in any case, extracting the references even from a regular html-bikeshed doc is harder unless we were to import bikeshed itself


80% of the screen black when I apply 144hz to monitor

Reproduction
cat ~/.config/hypr/hyprland.conf
monitor = DP-1,1920x1080@144, 0x0, 1
monitor = HDMI-A-1, 1920x1080, 1920x0, 1

inxi -b
Graphics:
  Device-1: NVIDIA TU116 [GeForce GTX 1660 SUPER] driver: nouveau v: kernel
  Display: wayland server: X.org v: <IP_ADDRESS> with: Xwayland v: 22.1.7
    compositor: Hyprland driver: X: loaded: modesetting,nouveau dri: nouveau
    gpu: nouveau resolution: 1: 1920x1080~144Hz 2: 1920x1080~60Hz
  API: EGL/GBM Message: No known Wayland EGL/GBM data sources.

nvidia is not supported by us, most likely a driver bug. Try sway-git.


Snowplow CLI: change AMI in config.yml to 3.8.0

Required for --srcPrefixesFile

Won't do, see #1959


Update Resource Block Status based on Oneview response

Status should be updated according to Oneview states. State can be different from "Enabled".
The allowable state values for server hardware are:
Unknown (not initialized) --> null
Adding (server being added)
NoProfileApplied (server successfully added) --> Unused
Monitored (server being monitored)
Unmanaged (discovered a supported server)
Removing (server being removed)
RemoveFailed (unsuccessful server removal)
Removed (server successfully removed)
ApplyingProfile (profile being applied to server) --> Composing
ProfileApplied (profile successfully applied) --> Composed
RemovingProfile (profile being removed)
ProfileError (unsuccessful profile apply or removal) --> Failed
Unsupported (server model or version not currently supported by the appliance)
UpdatingFirmware (server firmware update in progress)
Values in use by the toolkit are the ones in bold.

Waiting feedback from Dan/Blaine regarding some mappings and drive status.

Mapping Status OneView/Redfish:
Status OneView Redfish Mapping.xlsx

Let's update the Status for Computer System Resource Block and Status for a Composed System (checking server profile and server hardware).
Out of scope: Status for Storage Resource Block and Network Resource Block.


Removed broken programs, added working python ones

Removed the broken C++ scripts and replaced them with one python one, increasing efficiency and re-adding broken functionality.

Fixed issue with quotes being randomly selected, and added options for further expansion.


Deprecate mutability of Point geometry

xref https://github.com/Toblerity/Shapely/issues/782/ (Shapely 2.0).
Example of deprecating mutability this for Point:

Raise a warning in the coords setter
Splitted the tests to more easily isolate the parts that are deprecated (to now filter the warnings, and later to remove in Shapely 2.0)
Added one test that actually asserts a warning is raised

If that workflow looks OK, I can do the same for the other geometry types.

Any feedback on this PR?
I know the RFC still needs to be further discussed / approved, but to the extent possible, I would like to already move forward with some of the (less controversial) code changes (I could also target a separate branch than master until the RFC is actually approved?)

@sgillies are you OK with the approach here? If so, I can update this PR to cover all geometry types.
(although RFC 1 is not yet officially approved, I hope I can start on some of the non-controversial parts, like deprecating mutability)

@jorisvandenbossche I'm in favor of this :+1: Let's do consider a new shapely warning class.

I added a

Also a workflow question: do I continue here in this PR with doing the same for the other geometry types, or do you prefer to keep that for a separate PR(s)?

@jorisvandenbossche it's all the same to me, one or multiple PRs as you prefer.

OK, I added deprecation warnings for linestrings and linearrings as well (I think that's all, since collections' coords are not directly mutable (only their parts), and the same for polygons and its rings)
So this should be ready for review / merge

Then in our own tests we should convert these warnings to errors?

It seems you deleted this comment (or I don't find it back in the github UI), but yes, we could do that, to ensure that we are explicit in the tests about what is deprecated (and don't miss a consequence elsewhere in the package).
The @shapely20_deprecated marks tests we know are testing isolated deprecated behaviour, and then none of the other tests should raise this warning. And we can let pytest detect if that still happens.

And thanks for merging!
I will continue with some of the other deprecations (I think the adapter/proxy classes are also quite uncontroversial to already start with deprecating?)

Excellent! I'm really excited about getting a 1.8.0 out. I've got time this afternoon to review tickets and do some release planning. The sooner, the better. Maybe we can even skip a 1.7.1.


Incorrect event type used when updating a device

🐞 Bug Report
Affected Services
Core-metadata
Is this a regression?
No, this looks like it has been this way for awhile.  However, the operator which encompasses this bug was created in this commit
Description and Minimal Reproduction
The event type should be PUT since the associated operation is an update, but instead it is a POST
https://github.com/edgexfoundry/edgex-go/blob/818be3dfd721e644cbac9b5acdcbef0f0d1fb2ce/internal/core/metadata/operators/device/update.go#L143

Start core-services
Create a device
Update the device

See the Slack conversation for more details: https://edgexfoundry.slack.com/archives/CE6902Q6S/p1573569721008100
🔥 Exception or Error
An error is logged when the callback is invoked:
https://github.com/edgexfoundry/edgex-go/blob/21750bd69222753f69a7a217e480b07ca6bb6dd6/internal/core/metadata/operators/device/notify.go#L113
🌍  Your Environment
Deployment Environment:
Linux Fedora
EdgeX Version:
Fuji, this is also present in master(Geneva)
Anything else relevant?

I describe the process of the problem as follows:

Clone edgex-go Fuji, device-sdk-go Fuji. Run the example in sdk: device-simple.
In the picture, on the right side of the screen is the information of device service: device-simple.
Run device-simple DeviceService (DS). DS will add a new device: "Device-Simple02" with "description" = "Old".
Update:
3.1 Update by name: -> "Duplicate name for Device" error
3.2 Update by ID: "Duplicate name for Device" error
3.3 Update by name and ID:
edgex-go responded "true", when I read the device's information, it was indeed updated: "description" = "New", but in DS, it reported an error regarding adding a new device.
[add new device]

[update by id]

[update by name]

[update by name + id]

[result]

[log of edgex-go]


Add a default editor to global prefs

This will make it a bit easier to edit some configurations

added in config branch


Option repetition (mostly for verbosity 'levels') ?

Is there a clean way to implement this using the existing code?  Or would it need to be a new feature?
Think ssh -v[vvvvvv...]  (incrementing levels of verbosity if the option is repeated)
Note that with ssh, it functions the same with:  ssh -v -v -v -v example.com and ssh -vvvv example.com
I don't recall ever seeing this type of "option repetition" used with anything other than verbosity levels, and I'm not sure if that's even a GNU utility psuedo-standard, or if it's just specific to ssh and a few other oddball utilities, but it is handy.  I'd actually like to use that for a project I'm building, using your Commando library that wraps around ssh, and would like to provide the same -v[vvv...] functionality if possible.  Ideas?
I love your work on this by the way -- great library.

This was implemented in GetOptionKit. FYI: https://github.com/c9s/GetOptionKit/wiki/Defining-Options#incremental-flag

PR #63 Closes this @nategood


Update CSI-related usage

CSI docker-related files are moved to the Docker dir and the original CSI dir only contains a README.md. Since CSI depends on Kubernetes, now move the readme into the Kubernetes dir. Update information and values accordingly.

Codecov Report

Merging #14185 (a160fe5) into master (bf11633) will decrease coverage by 12.93%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #14185       +/-   ##
=============================================
- Coverage     43.28%   30.34%   -12.94%
+ Complexity     9247     5900     -3347
=============================================
  Files          1432     1436        +4
  Lines         82821    83108      +287
  Branches      10015    10052       +37
=============================================
- Hits          35848    25221    -10627
- Misses        43993    55785    +11792
+ Partials       2980     2102      -878

Impacted Files
Coverage Δ

...ient/fs/src/main/java/alluxio/client/ReadType.java
42.85% <0.00%> (-10.09%)
:arrow_down:

...java/alluxio/stress/client/ClientIOParameters.java
0.00% <0.00%> (ø)

...va/alluxio/stress/common/FileSystemClientType.java
0.00% <0.00%> (ø)

...va/alluxio/stress/common/FileSystemParameters.java
0.00% <0.00%> (ø)

...a/alluxio/stress/master/MasterBenchParameters.java
0.00% <0.00%> (ø)

...a/alluxio/stress/worker/WorkerBenchParameters.java
0.00% <0.00%> (ø)

...alluxio/stress/cli/client/StressClientIOBench.java
0.00% <0.00%> (ø)

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...mon/src/main/java/alluxio/util/ExceptionUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/wire/AlluxioProxyInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 427 more

Continue to review full report at Codecov.

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

@LuQQiu PTAL. Thanks!

can you update https://github.com/Alluxio/alluxio-csi/blob/master/README.md

to use CSI, pointing to latest documentation
to check the CSI related src code, pointing to https://github.com/Alluxio/alluxio/tree/master/integration/docker/csi

@LuQQiu Added another line pointing to the CSI source code location. This PR can be merged.

alluxio-bot, merge this please


AOAF-340: Added an error 404 page and corrected the file more package…

… link

CI: Security warnings found!

CI: No security warnings found

CI: No security warnings found


Update Dockerfile for transform PR

Update environment setup due to https://github.com/NVIDIA/spark-rapids-ml/pull/19
Signed-off-by: Allen Xu<EMAIL_ADDRESS>

Is this pr target to v21.10 or v21.12?

v21.12
On Tue, Nov 16, 2021 at 09:17 liyuan @.***> wrote:

Is this pr target to v21.10 or v21.12?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/spark-rapids-examples/pull/50#issuecomment-969594852,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE4HIGSUC6VNQ6WK4YEKDVLUMGWJZANCNFSM5IBOVTVA
.
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.

@wjxiz1992 I just create v21.12 branch, could you target the pr to v21.21?

Hi @nvliyuan, can you add @pxLi to reviewer list because this also has impacts on related CI job.

@wjxiz1992 sure, added @pxLi


GP refuses to use new plugin folder.

Following the Sponge 1973 update, we tried to move GP to a new 'plugin' folder, however it is consistently generating the folder in the Config directory, and will only load claimdata and player data from there.

Global.conf still generates in the original Config folder, but everything else is in the new folder.


Adds Value.toValid(), Value.toInvalid()

Fixes #1628

Current coverage is 96.47% (diff: 100%)

Merging #1630 into master will increase coverage by <.01%

@@             master      #1630   diff @@
==========================================
  Files            88         88
  Lines         10858      10864     +6
  Methods           0          0
  Messages          0          0
  Branches       1828       1832     +4
==========================================
+ Hits          10475      10481     +6
  Misses          241        241
  Partials        142        142

Powered by Codecov. Last update 7bcf701...ec17a63


Vickel: Watch shekan

Vickel requests the watch of the watch_keyword shekan. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

shekan has been seen in 3 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


stage-1: move lvm2 setup to lvm module

Motivation for this change
This allows reducing initramfs size by disabling the lvm module with disabledModules, reducing closure size. This intentionally does not introduce a more ergonomic way of disabling lvm, because it's not easy to tell whether LVM is required and I don't want to break anyone's boot, though if anyone thinks we should introduce such an option (defaulting to enabled?) I'm happy to do that as well :)
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Built installer test lvm
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg test installer.lvm

cc @ajs124 who has done a lot of lvm stuff


Add additional contributors

We want to add a few people for basic UI, docs, and description updates.
@eshon please list the GitHub accounts other than yourself that should be added.

eshon
smagdali
maciejwitowski


bspline parameter tuning for multimodal 3D registration

I was trying to register two dicom image (T1W and T2W , 60 slices each) on a 8gb, quadcore processor (Ubuntu 14.04) using Bspline method with LBFGSB optimizer. The time taken with the below default parameter was about 2-3 hour+.
SetOptimizerAsLBFGSB(gradientConvergenceTolerance=1e-5,
numberOfIterations=500,
maximumNumberOfCorrections=5,
maximumNumberOfFunctionEvaluations=2000,
costFunctionConvergenceFactor=1e+7)
but when I changed the gradientConvergenceTolerance to 1e-3, it worked in 6-8 minutes. Is there anything wrong with my parameter configuration?

There is nothing wrong with the parameter settings it is just that the ones you used (the defaults) were not appropriate for your specific problem. The performance of most registration algorithms depends on multiple parameter settings. In this case when you made the gradient convergence tolerance looser (larger) the algorithm terminated significantly earlier. You will have to play with these parameters to find the ideal ones for your task (time/accuracy).
In the future, please post questions concerning SimpleITK to the ITK mailing list. We use github issues only for reporting bugs/issues and for feature requests.


Fix reconcilers where error condition leads to infinite reconciliation

… without exponential backoff

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #317

unit test
$ make test
GOBIN=/home/ubuntu/go/src/github.com/oracle/cluster-api-provider-oci/hack/tools/bin ./scripts/go_install.sh sigs.k8s.io/controller-runtime/tools/setup-envtest setup-envtest v0.0.0-20230131074648-f5014c077fc3
kube-builder assets: /home/ubuntu/.local/share/kubebuilder-envtest/k8s/1.24.2-linux-amd64
KUBEBUILDER_ASSETS="/home/ubuntu/.local/share/kubebuilder-envtest/k8s/1.24.2-linux-amd64" go test -coverprofile=coverage.out ./...
?   	github.com/oracle/cluster-api-provider-oci	[no test files]
ok  	github.com/oracle/cluster-api-provider-oci/api/v1beta1	60.002s	coverage: 23.7% of statements
ok  	github.com/oracle/cluster-api-provider-oci/api/v1beta2	0.058s	coverage: 15.7% of statements
?   	github.com/oracle/cluster-api-provider-oci/cloud/config	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/metrics	[no test files]
ok  	github.com/oracle/cluster-api-provider-oci/cloud/ociutil	0.008s	coverage: 19.6% of statements
ok  	github.com/oracle/cluster-api-provider-oci/cloud/scope	242.771s	coverage: 75.0% of statements
?   	github.com/oracle/cluster-api-provider-oci/cloud/scope/mocks	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/base	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/base/mock_base	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/compute	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/compute/mock_compute	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/computemanagement	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/computemanagement/mock_computemanagement	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/containerengine	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/containerengine/mock_containerengine	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/identity	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/identity/mock_identity	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/loadbalancer	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/loadbalancer/mock_lb	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/networkloadbalancer	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/networkloadbalancer/mock_nlb	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/vcn	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/vcn/mock_vcn	[no test files]
ok  	github.com/oracle/cluster-api-provider-oci/cloud/util	0.083s	coverage: 73.4% of statements
ok  	github.com/oracle/cluster-api-provider-oci/controllers	27.354s	coverage: 60.3% of statements
ok  	github.com/oracle/cluster-api-provider-oci/exp/api/v1beta1	16.339s	coverage: 15.2% of statements
ok  	github.com/oracle/cluster-api-provider-oci/exp/api/v1beta2	0.019s	coverage: 4.8% of statements
ok  	github.com/oracle/cluster-api-provider-oci/exp/controllers	0.428s	coverage: 57.5% of statements
?   	github.com/oracle/cluster-api-provider-oci/feature	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/version	[no test files]

e2e test
  << Timeline
------------------------------

Ran 7 of 28 Specs in 3691.609 seconds
SUCCESS! -- 7 Passed | 0 Failed | 0 Pending | 21 Skipped


Fix range request to not overlap reads

See #461

And merged, thanks a lot for the PR!

And merged, thanks a lot for the PR!


rework heatmap permissions

heatmap now uses the same permission model as the activity feed:
only include activities in repos, that the doer has access to.
this might be somewhat slower.
also improves handling of user.KeepActivityPrivate (still show the heatmap to self & admins)

fixes #14066, fixes #14086

Codecov Report

Merging #14080 (dc5c959) into master (e8a354f) will increase coverage by 0.02%.
The diff coverage is 78.18%.

@@            Coverage Diff             @@
##           master   #14080      +/-   ##
==========================================
+ Coverage   42.32%   42.35%   +0.02%
==========================================
  Files         726      726
  Lines       77689    77709      +20
==========================================
+ Hits        32884    32913      +29
+ Misses      39405    39401       -4
+ Partials     5400     5395       -5

Impacted Files
Coverage Δ

models/action.go
52.90% <71.42%> (+2.90%)
:arrow_up:

models/user_heatmap.go
78.78% <88.23%> (-1.99%)
:arrow_down:

routers/api/v1/user/user.go
50.00% <100.00%> (ø)

routers/user/home.go
60.67% <100.00%> (ø)

routers/user/profile.go
48.00% <100.00%> (ø)

modules/queue/workerpool.go
58.77% <0.00%> (-2.05%)
:arrow_down:

services/pull/pull.go
41.37% <0.00%> (ø)

routers/api/v1/repo/pull.go
25.45% <0.00%> (+0.60%)
:arrow_up:

modules/git/batch_reader_nogogit.go
46.26% <0.00%> (+1.49%)
:arrow_up:

modules/log/file.go
75.20% <0.00%> (+1.60%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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

:rocket:


When trying to view clips in firefox, firefox reports typeerror: network error when attempting to fetch resource

Checklist:

[X] I updated to the latest version available
[X] I cleared the cache of my browser

Release with the issue:
0.14
Last working release (if known):
First attempt to use the card
Browser and Operating System:
Firefox 91.0.1 on Windows 10

Description of problem:
My card is configured as shown below. When I click the clips icon in lovelace firefox replaces the video with a grey box with a small black exclamation point in the middle.  When I hover over the exclamation point it says "TypeError: NetworkError when attempting to fetch resource."  If I click on the "View Frigate UI" button it opens a new tab in firefox and successfully connects to frigate as it .should.  I am able to ping <IP_ADDRESS> (my frigate server) successfully from within the terminal on hass.io/
type: custom:frigate-card
frigate_url: http://<IP_ADDRESS>:5000
webrtc: {}
camera_entity: camera.front_door_cam
motion_entity: binary_sensor.front_door_cam_person_motion
view_default: live
menu_mode: below
autoplay_clip: false

Javascript errors shown in the web inspector (if applicable):
I have 2  reported in my browser console.  Not sure if it is related:
Content Security Policy: Ignoring “'unsafe-inline'” within script-src: ‘strict-dynamic’ specified
Content Security Policy: Ignoring “http:” within script-src: ‘strict-dynamic’ specified

Additional information:

Thanks for the report @rhatguy . Two questions:

Are you accessing HA itself over http or https?
What happens when you try a different browser, do you get the same issue?

I'm accessing HA over https locally through my lan.
Accessing with different browsers gives the same behavior but with a slightly different message.  Edge and Chrome both say "TypeError: Failed to fetch".
Interestingly when I tail the nginx log on my frigate machine (which runs on a separate RPi 4) I see home assistant connecting to it for various other things (mostly grabbing /api/stats to update other lovelace cards), but when I load up this card, I don't see any connections to frigates nginx instance.  For some reason its not connecting to frigate at all.

Just found a new issue in my console.  (Sorry not adept at the right place to look for these javascript errors).
Blocked loading mixed active content “http://<IP_ADDRESS>:5000/api/events?camera=front_door_cam&has_clip=1&limit=100”

Since you're accessing HA over https and pointing the card at Frigate over http (no 's'), the browser likely won't allow you to access mixed content. Give the browser a https means of reaching Frigate. If you have Home Assistant behind a reverse proxy (and that's how you're getting ssl), you'd just need to configure Frigate in a similar way (this is my own setup).

Understood. I have a nginx proxy setup for other stuff internally but haven't moved hass.io behind it yet.  I'll give a go at getting both HA and frigate on either http or https and see if that fixes things.  Thanks!

Rather than unhelpfully render just an icon, [from the next release] the card will show the error message in the card and link to the troubleshooting documentation, which now includes the specific issue that went wrong for you.
Good luck with the setup! Please re-open a new issue if the solution you're working on does not resolve it (pretty optimistic it will!).

Just waned to confirm that I got my setup working after getting frigate accessible through https.  Thanks for your help.  The addon looks great!


Experiment with giving higher priority to proc-macros and their dependencies

Currently, the dependency queue uses the number of transitive reverse dependencies to determine how to schedule jobs:
https://github.com/rust-lang/cargo/blob/5847787feff6db0eba19b0e433b2ae0baa132ca0/src/cargo/util/dependency_queue.rs#L34-L35
I think it would be a good idea to experiment with 'boosting' the priority of proc-macro crates and their dependencies. Unlike other crates, proc-macro crate building cannot be pipelined - since they are dynamically loaded into the compiler, any reverse-dependencies of proc-macro crates must wait for codegen, not just metadata.
By giving proc-macro crates a higher priority, we increase the likelihood that they will finish codegen by the time we are ready to start building their reverse dependencies. A similar argument applies to dependencies (not reverse-dependencies) of proc-macro crates: until https://github.com/rust-lang/rust/issues/64191 is implemented, we still need to wait for all a proc-macro crates's dependencies to codegen before we can start building the proc-macro crate.
Anecdotally, I've noticed that some proc-macro dependencies (e.g. syn) seem to take a significant fraction of the total build time. By starting to build these crates earlier, we may be able to reduce the impact of their build times on the total build time.
Concretely, I think we could add a flag -Z prioritize-proc-macros to Cargo. When enabled, we could either directly increase the priority entry by a certain factor, or just break ties in priority by proc-macro (or proc-macro-dependency) status.
By using -Z timings, we can gather data on the impact of this change on build times.

I think it would be great to experiment with this.  If you (or someone) wants to prototype it and demonstrate that it is measurably faster on some sampling of projects (and various -j values), I think it could be added without a -Z flag.

To add to this (while being unsure if this is even pursued at the moment): a similar logic applies to build scripts. For example, the now-stabilized --timings on a crate that uses openssl-sys/vendored (and thus has to build openssl from source to statically link it) shows that the critical dependency here (with some hops through openssl crates) is cc, which is built as the 7th crate total out of some hundred.
The build script only starts running about 15 seconds later, however, leading to a very undesirable tail of compiling the handful of crates that use it at then end. The build script takes a very long time to run, so all other crates are well done by that point - if the build script was started earlier, it could have run in parallel for those 15 seconds and the whole build would be faster.


snapd: update to 2.56.2, cleanup configuration, drop unneeded files

Update to 2.56.2
Based on #6. Verified manually.

I just tried 2.56.2, and got an error of the form:
| fork/exec
/home/rpjday/oe/builds/alt_snapd/tmp-glibc/work/core2-64-oe-linux/snapd/2.56.2-r0/build-tmp/go-build3643160550/b001/exe/info:
no such file or directory
| WARNING:
/home/rpjday/oe/builds/alt_snapd/tmp-glibc/work/core2-64-oe-linux/snapd/2.56.2-r0/temp/run.do_configure.2548506:173
exit 1 from 'do_configure'
Any idea what that is about? I'll try it again just to make sure it's
reproducible.
rday
On Wed, Jul 13, 2022 at 7:56 AM Maciej Borzecki @.***>
wrote:

Update to 2.56.2
Based on #6 https://github.com/snapcore/meta-snapd/pull/6. Verified
manually.
You can view, comment on, or merge this pull request online at:
https://github.com/snapcore/meta-snapd/pull/7
Commit Summary

5fd6b39
https://github.com/snapcore/meta-snapd/pull/7/commits/5fd6b392c2180d311b5846edf65f7e4551382f43
recipes-kernel: add strict apparmor support patches for Linux Yocto kernel
b448b10
https://github.com/snapcore/meta-snapd/pull/7/commits/b448b10f4df26402dc65f756379860df1b83759e
snapd: update to 2.56.2, cleanup configuration, drop unneeded files

File Changes
(5 files https://github.com/snapcore/meta-snapd/pull/7/files)

A recipes-kernel/linux/files/0001-add-use-fns.patch
https://github.com/snapcore/meta-snapd/pull/7/files#diff-c9624c5ec3d65f8829114d279411d080ed80cb6d0d1bff04ea9daa9cc66e73eb
(59)
A recipes-kernel/linux/files/0002-v2.x-net-rules-compat.patch
https://github.com/snapcore/meta-snapd/pull/7/files#diff-e938db675b390fdc692c33bc3cf5e076fe7ff5184eaa0bbcdcdf7e859a363fd5
(222)
A recipes-kernel/linux/files/0003-af_unix-mediation.patch
https://github.com/snapcore/meta-snapd/pull/7/files#diff-38ba7ec79538e5f531515b3b2af5708699d7fd1101da1bafaa25b42453021b5a
(1143)
M recipes-kernel/linux/linux-yocto_%.bbappend
https://github.com/snapcore/meta-snapd/pull/7/files#diff-375d1294f4128dd5204664c235078df68f3229be1842c1faf66e2c737b84883a
(5)
R recipes-support/snapd/snapd_2.56.2.bb
https://github.com/snapcore/meta-snapd/pull/7/files#diff-89fe60ac7e0bc1ae9dddc3ec25a13cd633ab5eb81e2894750572b7028ce428c5
(15)

Patch Links:

https://github.com/snapcore/meta-snapd/pull/7.patch
https://github.com/snapcore/meta-snapd/pull/7.diff

—
Reply to this email directly, view it on GitHub
https://github.com/snapcore/meta-snapd/pull/7, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEUVB5C655RRMXHROVTMX3VT2VHZANCNFSM53OOP5EA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Can you add a bit more of the log? The upstream code now invokes go run in mkversion.sh to generate some data. This should be fixed with https://github.com/snapcore/meta-snapd/pull/7/files#diff-89fe60ac7e0bc1ae9dddc3ec25a13cd633ab5eb81e2894750572b7028ce428c5R76-R77 which sets GOARCH=${GOHOSTARCH} when calling mkversion. Builds fine here and in our internal system.

The tail end of my build simply migrating from 2.55.5 to 2.56.2:
| HASH[link github.com/snapcore/snapd/asserts/info]: "packagefile
image/internal/imageutil=jWuGHdNQMFEkkw-tpsmW\n"
| HASH[link github.com/snapcore/snapd/asserts/info]: "packagefile
crypto/ed25519/internal/edwards25519/field=oXX5ZedlvcIe4QP8mUeM\n"
| HASH[link github.com/snapcore/snapd/asserts/info]: "packagefile
crypto/elliptic/internal/fiat=AZi0ePrbJuWL6y7WK1Ew\n"
| HASH[link github.com/snapcore/snapd/asserts/info]: "packagefile
internal/intern=n7vNRfFN9GEjkl2y_pAx\n"
| HASH[link github.com/snapcore/snapd/asserts/info]:
46b3d422814de36d65ac50c90922360e3bc914107c898540c75ec4bd433916bf
| HASH subkey
5506c7084dfddc36bd32885b32d1dd4b610c566684eae68b5c68b6a4fafe1f16
"link-stdout" =
b0268d3e1ff8198624d7210565f1aed8da3761fb05e0dc995e76dbb2c0b19435
| fork/exec
/home/rpjday/oe/builds/alt_snapd/tmp-glibc/work/core2-64-oe-linux/snapd/2.56.2-r0/build-tmp/go-build919158439/b001/exe/info:
no such file or directory
| WARNING:
/home/rpjday/oe/builds/alt_snapd/tmp-glibc/work/core2-64-oe-linux/snapd/2.56.2-r0/temp/run.do_configure.2652666:173
exit 1 from 'do_configure'
| WARNING: Backtrace (BB generated script):
| #1: do_configure,
/home/rpjday/oe/builds/alt_snapd/tmp-glibc/work/core2-64-oe-linux/snapd/2.56.2-r0/temp/run.do_configure.2652666,
line 173
| #2: main,
/home/rpjday/oe/builds/alt_snapd/tmp-glibc/work/core2-64-oe-linux/snapd/2.56.2-r0/temp/run.do_configure.2652666,
line 321
ERROR: Task
(/home/rpjday/oe/dist/layers/meta-snapd/recipes-support/snapd/snapd_2.56.2.bb:do_configure)
failed with exit code '1'
NOTE: Tasks Summary: Attempted 2599 tasks of which 2594 didn't need to be
rerun and 1 failed.
Summary: 1 task failed:
/home/rpjday/oe/dist/layers/meta-snapd/recipes-support/snapd/snapd_2.56.2.bb:
do_configure
Summary: There was 1 ERROR message, returning a non-zero exit code.
On Wed, Jul 13, 2022 at 8:34 AM Omer Akram @.***> wrote:

@.**** approved this pull request.
looks good to me. We have the same recipes working in our Yocto build as
well
—
Reply to this email directly, view it on GitHub
https://github.com/snapcore/meta-snapd/pull/7#pullrequestreview-1037282741,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEUVB372X5BPUSKEFFEEVTVT2ZT5ANCNFSM53OOP5EA
.
You are receiving this because your review was requested.Message ID:
@.***>

Actually, ignore my report -- if this PR works for others, merge it, and
I'll rebase my local changes on top of that.
rday
On Wed, Jul 13, 2022 at 11:01 AM Robert Day @.***>
wrote:

The tail end of my build simply migrating from 2.55.5 to 2.56.2:
| HASH[link github.com/snapcore/snapd/asserts/info]: "packagefile
image/internal/imageutil=jWuGHdNQMFEkkw-tpsmW\n"
| HASH[link github.com/snapcore/snapd/asserts/info]: "packagefile
crypto/ed25519/internal/edwards25519/field=oXX5ZedlvcIe4QP8mUeM\n"
| HASH[link github.com/snapcore/snapd/asserts/info]: "packagefile
crypto/elliptic/internal/fiat=AZi0ePrbJuWL6y7WK1Ew\n"
| HASH[link github.com/snapcore/snapd/asserts/info]: "packagefile
internal/intern=n7vNRfFN9GEjkl2y_pAx\n"
| HASH[link github.com/snapcore/snapd/asserts/info]:
46b3d422814de36d65ac50c90922360e3bc914107c898540c75ec4bd433916bf
| HASH subkey
5506c7084dfddc36bd32885b32d1dd4b610c566684eae68b5c68b6a4fafe1f16
"link-stdout" =
b0268d3e1ff8198624d7210565f1aed8da3761fb05e0dc995e76dbb2c0b19435
| fork/exec
/home/rpjday/oe/builds/alt_snapd/tmp-glibc/work/core2-64-oe-linux/snapd/2.56.2-r0/build-tmp/go-build919158439/b001/exe/info:
no such file or directory
| WARNING:
/home/rpjday/oe/builds/alt_snapd/tmp-glibc/work/core2-64-oe-linux/snapd/2.56.2-r0/temp/run.do_configure.2652666:173
exit 1 from 'do_configure'
| WARNING: Backtrace (BB generated script):
| #1: do_configure,
/home/rpjday/oe/builds/alt_snapd/tmp-glibc/work/core2-64-oe-linux/snapd/2.56.2-r0/temp/run.do_configure.2652666,
line 173
| #2: main,
/home/rpjday/oe/builds/alt_snapd/tmp-glibc/work/core2-64-oe-linux/snapd/2.56.2-r0/temp/run.do_configure.2652666,
line 321
ERROR: Task
(/home/rpjday/oe/dist/layers/meta-snapd/recipes-support/snapd/snapd_2.56.2.bb:do_configure)
failed with exit code '1'
NOTE: Tasks Summary: Attempted 2599 tasks of which 2594 didn't need to be
rerun and 1 failed.
Summary: 1 task failed:
/home/rpjday/oe/dist/layers/meta-snapd/recipes-support/snapd/snapd_2.56.2.bb:
do_configure
Summary: There was 1 ERROR message, returning a non-zero exit code.
On Wed, Jul 13, 2022 at 8:34 AM Omer Akram @.***>
wrote:

@.**** approved this pull request.
looks good to me. We have the same recipes working in our Yocto build as
well
—
Reply to this email directly, view it on GitHub
https://github.com/snapcore/meta-snapd/pull/7#pullrequestreview-1037282741,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEUVB372X5BPUSKEFFEEVTVT2ZT5ANCNFSM53OOP5EA
.
You are receiving this because your review was requested.Message ID:
@.***>


Make TIFF read robust to subimge-to-subimage changes in some header fields

Rearrange the order in which we examine TIFF header order. When reading MIPmap levels (beyond the first one), we assume that all the ordinary metadata should be the same for all levels, so we don't laboriously read all that stuff. But in our zeal to skip this needless work, we took the early out before examining some essential fields such as planarconfig, photometric interpretation, palette processing, and extrasamples.  If a file should change this from one MIP level to the next -- which no sane file should do, but hey, it did happen in the wild -- that would bring chaos. Now we are more robust to this, while still avoiding all the detailed metadata reading for non-head MIP levels.
Fixes #1220

Fixes #1220


List all the App Service Slots for a given App Service Plan

Related command

az webapp list --query "[?appServicePlanId=='$appServicePlanID']"
Is your feature request related to a problem? Please describe.

I do have a requirement to change the vnet integration for all my applications to move to a new subnet. And, I was hoping to list all the web apps (app services) and corresponding slots in an app service plan - run through the list iteratively to remove the vnet integration and then associate it back.
Describe the solution you'd like

Similar to the command posted above, where I can list all the app services in a given plan - would like to have the same feature for az webapp deployment slot
Describe alternatives you've considered

I was trying my luck with az resource list, while I can list my app service and their corresponding slots with this command I cannot restrict it to a specific app service plan.
az resource list --resource-group $resourceGroupName --query "[?kind=='app,linux' && (type=='Microsoft.Web/sites' || type=='Microsoft.Web/sites/slots')]"
Additional context

route to CXP team

@ravitellakula I do not think we have any CLI to get the same. You can probably call the below REST API to get apps and respective slots under an app service plan- https://management.azure.com//subscriptions/{subscriptionid}/resourceGroups/{resourcegroup}/providers/Microsoft.Web/serverfarms/{AppServicePlanName}/sites?includeSlots=true.


refactor wrap in dialogue.rb

Hello,
I refactored the wrap method in dialogue.rb. I think it's easier to read and test the code. I also corrected some bug.
Thank.

This is a good start towards improving this code, thanks


Prisma2 - disable create/connect on certain inputtypes

Problem: We have a nested object structure with objects relying on the existance of other objects.
Example: Domain 1 has X organisations with each organisations having it's own X departments and each department having X locations with X classrooms.
When creating a department we want to connect it to an existing organisation but we can't allow upstream creating based on authorization. (An organisation admin can create a new department and link it but not make a new organisation).
The problem we're facing is that all autogenerated InputTypes allow a couple of different ways of creating nested objects (even allowing for empty objects while the relation is required which gives errors when fetching the newly created object; I can provide more info if interested)
It would be usefull that we could provide extra arguments in the typedefinitions that allow us to disable creation of fields but this does not seem possible at the moment.
I've been reading up on some other issues and found this one https://github.com/prisma-labs/nexus-prisma/issues/95. But this seems to use Prisma1 syntax.
Last resort we could do manual but we have 28 models at the moment so I don't really want to go through and overwrite all the different input possibilities manual :-)
schema.prisma:
model Domain  {
  id            String   @default(cuid()) @id
  createdAt     DateTime @default(now())
  updatedAt     DateTime @updatedAt

  name          String   @unique
  description   String?
  logo          String?
  email         String
  telephone     String
  mobilePhone   String
  website       String?

  address       Address
  organisations Organisation[]
  contact       Member
}

model Organisation  {
  id            String   @default(cuid()) @id
  createdAt     DateTime @default(now())
  updatedAt     DateTime @updatedAt

  name          String   @unique
  description   String?
  logo          String?
  email         String
  telephone     String
  mobilePhone   String
  website       String?

  domain        Domain
  address       Address
  departments   Department[]
  contact       Member
}

model Department  {
  id            String   @default(cuid()) @id
  createdAt     DateTime @default(now())
  updatedAt     DateTime @updatedAt

  name          String   @unique
  description   String?
  logo          String?
  email         String
  telephone     String
  mobilePhone   String
  website       String?

  organisation  Organisation
  address       Address
  locations     Location[]
  contact       Member
}

model Location  {
  id            String   @default(cuid()) @id
  createdAt     DateTime @default(now())
  updatedAt     DateTime @updatedAt

  name          String   @unique
  description   String?
  walkIn        Boolean
  email         String
  telephone     String
  mobilePhone   String
  website       String?

  department    Department
  address       Address
  contact       Member
  classrooms    ClassRoom[]
  accessories   Accessory[]
}

model ClassRoom {
  id            String   @default(cuid()) @id
  createdAt     DateTime @default(now())
  updatedAt     DateTime @updatedAt
}

Hey @maartenraes,
Thanks for raising that issue. We're planning to add such a feature in the future, it's not planned right now though, and we, unfortunately, have no workaround meanwhile.
Related to: https://github.com/prisma-labs/nexus-prisma/issues/110

I'm working on a workaround by using a combination of graphql-shield and Yup validation to validate and disable some nested properties, will update when I get something working 👍

If you have specific auth checks for write operations, would it perhaps instead make sense to not expose those through Nexus Prisma and write specific mutations for those operations instead, and then use GraphQL Shield to validate permission for those?

It does not fix the entire problem, yes indeed we could write specific mutation, but that would be 96 mutations that we would need to overwrite. That seems a lot of boilerplate for code that's already there, so I would rather find a solution to validate input instead of overwriting all mutations.
Another problem is that you can create objects with an empty object as relation while the relation is required, so even with the graphql shield authorization we would need to validate it anyway with yup.

Hey @maartenraes, your input on #598 would be welcome if you have time.

My 2 cents... You can override nexus generated input types excluding things you don't need... In your example the input generated from nexus for the field Department.organization should be called OrganizationCreateOneWithoutDepartmentsInput
You can create an input type with the same name where you include only the connect field:
const OrganizationCreateOneWithoutDepartmentsInput = schema.inputObjectType({
  name: 'OrganizationCreateOneWithoutDepartmentsInput',
  definition(t) {
    t.field('connect', { type: 'OrganizationWhereUniqueInput' });
  },
});


Buttons not showing on card

Checklist:

[x] I updated to the latest version available
[x] I cleared the cache of my browser

Release with the issue:
Card version: 5.2.0
Frigate version:  0.14.0
Last working release (if known):
?
Browser and Operating System:
Chrome on MacOS Sanoma 14.5
App on iPhone iOS 17.5.1
Home Assistant OS 13.0
HA version 2024.8.1
Description of problem:
I am trying to enable menu buttons on the card, but some of them are not showing up. The main button I want to activate is the microphone button. 2-way audio works in Frigate web UI, but not in the card.
As a test, I enabled all buttons, as you can see below.
List of buttons that appear:

frigate
mute
play
cameras
live
expand
media player
image
screenshot

List of buttons that do not appear:

recordings
microphone (main interest at the moment)
substreams
snapshots
clips
camera_ui
download
timeline

Frigate card diagnostic information
{
  "ha_version": "2024.8.1",
  "card_version": "5.2.0",
  "browser": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36",
  "date": "2024-08-14T19:41:08.295Z",
  "frigate_version": {},
  "lang": "en",
  "timezone": "Europe/Brussels",
  "git": {
    "build_version": "5.2.0-HEAD+g69249b6",
    "build_date": "Fri, 23 Jun 2023 15:26:26 GMT",
    "commit_date": "Thu, 22 Jun 2023 09:21:26 -0600"
  },
  "config": {
    "type": "custom:frigate-card",
    "cameras": [
      {
        "camera_entity": "camera.deurbel_fluent",
        "live_provider": "auto",
        "hide": false,
        "triggers": {
          "motion": true
        }
      }
    ],
    "menu": {
      "buttons": {
        "recordings": {
          "enabled": true
        },
        "microphone": {
          "enabled": true,
          "type": "momentary",
          "icon": "mdi:microphone",
          "priority": 100
        },
        "mute": {
          "enabled": true,
          "icon": "mdi:volume-high",
          "alignment": "matching"
        },
        "play": {
          "enabled": true,
          "icon": "mdi:play-pause"
        },
        "frigate": {
          "enabled": true
        },
        "cameras": {
          "enabled": true
        },
        "expand": {
          "enabled": true
        },
        "substreams": {
          "enabled": true
        },
        "snapshots": {
          "enabled": true
        },
        "clips": {
          "enabled": true
        },
        "live": {
          "enabled": true
        },
        "camera_ui": {
          "enabled": true
        },
        "download": {
          "enabled": true
        },
        "timeline": {
          "enabled": true
        },
        "media_player": {
          "enabled": true
        },
        "screenshot": {
          "enabled": true
        },
        "image": {
          "enabled": true
        }
      },
      "style": "outside"
    },
    "view": {
      "update_force": false,
      "default": "live"
    },
    "timeline": {
      "style": "stack"
    },
    "media_viewer": {},
    "live": {
      "preload": false,
      "microphone": {
        "always_connected": true
      }
    }
  }
}

Javascript errors shown in the web inspector (if applicable):

Additional information:
Frigate config:
mqtt:
  enabled: true
  host: _ip_
  port: 1883
  user: _user_
  password: _password_
  topic_prefix: frigate

# detectors:
#  coral_pci:
#    type: edgetpu
#    device: pci:0

##########################################################################

go2rtc:
  streams:
    reolink:
      - rtsp://_user_:_password_@_ip_:554/h264Preview_01_sub
      - ffmpeg:rtsp://_user_:_password_@_ip_:554/h264Preview_01_sub#audio=pcm#audio=volume

## THIS PREVENTS THE AUDIO COMING OUT FROM THE DOORBELL SOUNDING MUFFLED
  ffmpeg:
    bin: ffmpeg
    volume: -af "volume=30dB"

cameras:
  reolink:
    ffmpeg:
      inputs:
        - path: rtsp://_user_:_password_@_ip_:554/h264Preview_01_sub
          roles:
            - detect
            - record
      output_args:
        record: preset-record-generic-audio-copy

Guess: "camera.deurbel_fluent is not actually a Frigate camera. In your Frigate config, it suggests the camera name is reolink, so the entity may be something like camera.reolink.

Thanks. deurbel_fluent is indeed the name of the (same) camera via the Reolink Integration. Name of camera in Frigate is indeed reolink.
So next question. How come I cannot find the camera.reolink in the Frigate card, knowing it is correctly integrated in Frigate.

Ok, my bad. I forgot to install the Frigate Integration. Apologies!
Now I am able get most of the buttons on the card. But still not able to get the microphone button
I will dig a bit deeper myself
Card diagnostics. Please review for confidential information prior to sharing
{
  "ha_version": "2024.8.1",
  "card_version": "5.2.0",
  "browser": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36",
  "date": "2024-08-15T06:01:59.485Z",
  "frigate_version": {
    "01J5A9MP08PVQMNTSPSVGCHD10": "5.3.0/0.14.0-da913d8"
  },
  "lang": "en",
  "timezone": "Europe/Brussels",
  "git": {
    "build_version": "5.2.0-HEAD+g69249b6",
    "build_date": "Fri, 23 Jun 2023 15:26:26 GMT",
    "commit_date": "Thu, 22 Jun 2023 09:21:26 -0600"
  },
  "config": {
    "type": "custom:frigate-card",
    "cameras": [
      {
        "camera_entity": "camera.reolink"
      }
    ],
    "timeline": {
      "style": "ribbon"
    },
    "menu": {
      "style": "overlay",
      "buttons": {
        "frigate": {
          "enabled": true
        },
        "cameras": {
          "enabled": false
        },
        "substreams": {
          "enabled": false
        },
        "live": {
          "enabled": true
        },
        "recordings": {
          "enabled": true
        },
        "microphone": {
          "enabled": true,
          "alignment": "matching",
          "type": "toggle",
          "icon": "mdi:microphone"
        },
        "mute": {
          "enabled": true
        }
      }
    }
  }
}


Added pages: "Games", "About" and "Cards", and styles.

In this commit, two new website pages were added:

"About":

Added the "About" page with relevant content.
Utilized Flexbox for styling and layout of the "About" page.

"Cards":

Added the "Cards" page with content and applied grids for organizing cards on the page.

This commit includes significant changes related to web development and introduces new pages with their respective functional elements.

So no I still see a lot of unnecessary files in your changes. Please remove them

I've recorded screencast for you
https://github.com/MastersAcademy/fe-course-basic-2023/assets/98031768/907410b3-4c43-4aa5-b60c-bfb933674ed3

Nice pictures and shadows, BTW.

And you have some conflicts, please resolve them here, or merge them to your branch fresh main and resolve conflicts locally. Google or ask bots for help


Can't start scenarios

I may be wrong here, but it seems like I can't start scenarios without having to call the binary directly.
The flag to start a scenario is --start-server-load-scenario.
Thanks!

you can change the source,
add this to the case under new-map or something...
start-scenario)
    if [ -z $2 ]; then
      echo -e "You must specify a scenario name example: scenario-name or base/freeplay or latest for the latest scenario added"
      exit 1
    fi
    scenarioname="$2"
    if [ "${scenarioname}" == "latest" ]; then
      scenarioname="$(ls -t ${WRITE_DIR}/scenarios/ | head -1)"
      echo -e "Using latest scenario: ${scenarioname}"
    fi
    # Check if user wants to use custom map gen settings
    if [ $3 ]; then
      if [ -e "${WRITE_DIR}/$3" ]; then
        if [ $4 ]; then
          if [ -e "${WRITE_DIR}/$4" ]; then
          mapgen="--map-gen-settings=${WRITE_DIR}/$3 --map-settings=${WRITE_DIR}/$4"
          else
          echo -e "Specified map settings json does not exist"
          exit 1
          fi
        fi
      else
        echo -e "Specified map gen settings json does not exist"
        exit 1
      fi
    else
		mapgen="--preset rail-world"
    fi

    # Stop Service
    if is_running; then
      send_cmd "Generating new save, please stand by"
      if ! stop_service; then
        echo -e "Failed to stop server, unable to create new save"
        exit 1
      fi
    fi
    if ! start_service_scenario "--start-server-load-scenario ${scenarioname}" "${mapgen}"; then
      echo -e "Could not start $SERVICE_NAME"
      exit 1
    fi
    ;;

and add a function under start_service(), something like this...
start_service_scenario() {
  if [ -z "$1" ]; then
    echo -e "No scenario name passed to start_service_scenario()"
    return 1
  fi
  scenario="$1"
  if [ -z "$2" ]; then
    echo -e "No map gen passed to start_service_scenario()"
    return 1
  fi
  mapgen="$2"
  if [ -e ${PIDFILE} ]; then
    ps -p $(cat ${PIDFILE}) > /dev/null 2>&1
    if [ "$?" -eq "0" ]; then
      echo -e "${SERVICE_NAME} is already running!"
      return 1
    fi
    debug "Found rogue pid file, server might have crashed"
    rm ${PIDFILE} 2> /dev/null
  fi

  if ! check_permissions; then
    echo -e "Error! Incorrect permissions, unable to write to ${WRITE_DIR}"
    return 1
  fi

  if [ "${SAVELOG}" ==  "0" ]; then
    debug "Erasing log ${CMDOUT}"
    echo "" > ${CMDOUT}
  fi

  as_user "tail -f ${FIFO} |${INVOCATION} ${scenario} ${mapgen} >> ${CMDOUT} 2>&1 & echo \$! > ${PIDFILE}"

  ps -p $(cat ${PIDFILE}) > /dev/null 2>&1
  if [ "$?" -ne "0" ]; then
    echo -e "Unable to start ${SERVICE_NAME}"
    return 1
  else
    echo -e "Started ${SERVICE_NAME} with pid: $(cat ${PIDFILE}), please see log for details"
  fi
}

shouldn't be that hard

Can you add a invocation helper that tells us when something is wrong with the scenario?
Took me a while to figure it out, ergo until I tried without factorio-init, which I should've done earlier.

Should be possible :)

I was able to get this to work after making this change to the above code
if ! start_service_scenario ${scenarioname} "${mapgen}"; then

@Kilandor - feel free to submit a pull request of your changes if you have the time for it :)


Add oracledb stream support

I took a stab at adding support for the Client_Oracledb class to use oracledb#queryStream method to stream results through knex. I added a test but not sure if more is needed so please let me know if I need to make more tests or change how I setup the streaming feature. Finally I wasn't sure how to get the tests working locally with oracledb and an oracle db so I'm hoping the tests will work correctly in travis but if not I'd appreciate some help in getting the tests working locally and I'll update the docs as well.

Sorry for the huge delay, I hadn't noticed this one earlier, looks like a pull request that should be merged in after conflicts are fixed. Setting up oracledb locally is a bit painful. If I recall correctly I did it with vagrant + VirtualBox earlier... Would be good to have some vagrant file for setting it up locally, probably I'll do that if I need to set it up again at some point.

So knex.js with oracledb streams is not supported atm ?

Sorry I've moved on from the project where I was using this and don't know when I'll have time to get back to it. I'll try fix the conflicts soon but if someone else could handle setting up the tests that'd be much appreciated.

Closing incomplete PR, can be reopened if completed.


Lib does not detect telemarketer numbers in India starting with 140

Imported from Google Code issue #357 created by arif.amirani on 2013-10-09T15:40:33.000Z:

As per the wikipedia, "Telemarketers have been issued 10-digit telephone numbers starting with 140 (140XXXXXXX) by the Department of Telecommunications on TRAI's request."
An example number is +91 1409 305 260 which is considered invalid.
Source: http://en.wikipedia.org/wiki/Telephone_numbers_in_India

Fixed as per demo.


Support OAuth2/ODIC Login Providers

Rancher versions:
rancher/server or rancher/rancher: 2.0.6
I believe it would be great benefit to start supporting OAuth2 preferably with OIDC Connect standard. This would allow for a far greater range of authentication providers to be use, including but not limited to Okta, OneLogin, Auth0, and there are several OpenSource projects that implement OAuth2/OIDC Connect that could also be used, ie Hyda, Keycloak, AnvilConnect.
Thanks!

I've implemented OAuth2 and OIDC Connect multiple times, if you need additional information to make this a reality, I'll gladly help.

Maybe would be enough to implement support for https://github.com/dexidp/dex

Close in favor of https://github.com/rancher/rancher/issues/10053


Add missing EmailSetting options in appsetting.json (#2055)

This closes #2055

the issue is not triaged yet, and I think it's not a valid issue.


Added Go to page functionality

Background:
https://trello.com/b/UE2qyYpg/beacon-platform-4th-may-3rd-june
Solution:

Added a text field for entering a page number and a link next to it, to perform the navigation.
If the field is empty or contains invalid input, the link is disabled.

 Pagination and count of total records


Aliased nested list fields with different filters don't work properly

I have read the docs thoroughly before making this bug report.

[X] Yes

I have read through other open issues, and my issue is not a duplicate.

[X] Yes

What version of the library you are using?
0.8.0
Which python version are you using?
3.12
What operating system are you on?
Windows
Description
When queries are used with aliases, the results are not as expected. The filter of one of the queries is ignored, so the response is skewed (see screenshot).

Hi, thanks for the report. I was unable to replicate the issue with my test setup. I've added an example test to the alias-example branch. Can you provide more details and a more minimal example on how to reproduce it?

Am I supposed to close the alias-example branch and install the project? Could you explain to me how to do it properly? I get errors when I do runserver.

You'll need the optional django-filter dependency for local development, so install it with poetry install -E filter.
To note, you don't necessarily need to use my test project. You can set up your own test project and demonstrate things that way too.

As you said you couldn't replicate the error at your level, I updated all my important packages and uninstalled what I don't intend to use any more since your packages give me access to the speed I'd like. all this is to make sure I don't have a compatibility problem and other conflicts but the problem still remains. on my local machine I'm using python 3.11 but on my server it's python 3.12 in docker that's why I put the bug on python 3.12. I'm going to update my local python anyway to make sure.

You'll need the optional django-filter dependency for local development, so install it with poetry install -E filter.
To note, you don't necessarily need to use my test project. You can set up your own test project and demonstrate things that way too.

Ok let me try

i made a public repository for the demo. tell me if this is the right procedure to follow. the demo is based on postgresql.
https://github.com/AIEAJN/query_optimizer_alias_bug.git

Ok, seems like the issue is related to aliasing nested list fields with different filters (in this case it's connection fields, but same seems to happen for regular list fields too). I'll work on a fix soon.

I hope it won't last, I took my time redoing my structure because I really appreciate the speed your lib gives me, I was even thinking of merging it with the main branch this week. ❤❤ In any case, thank you for your lib. I'm willing to become a contributor if you'll let me, especially since I've been using it a lot and have suggestions and improvements to offer you. But I've never done anything like it, so I don't know how it works, so you can tell me if you want. So for this issue, should I close it or wait until you fix the bug?

🤣I agree about the complexity, I've gone through the code a bit and I understand what you mean. I'm going to take your advice, read up on the broad outlines and read the closed discussions. Then I'll open discussions if I have any ideas to offer you.
The bug seems to have been solved in the WIP you pushed as you can see on the screenshot.

I'd also recommend running some of the test cases with a debugger and stepping through the OptimizationCompiler to see how it figures out the optimizations.
I'll make a new release with the fix soon enough. I'll update and close this issue once it's out.

ok ok no problem thanks!


Embed Placeholder Item (Empty Element) and Bug on selected Item

Hello @atayahmet
This is very awesome plugin so far.. But I found some issues :

How to embed placeholder / empty item in dropdown? Right now it will generate dropdown item from table's record. I got problem when I want to add element <option value="">Please Choose</option>

I try to assign selected item. I code like this:
ShopCategory::selected(1)->renderAsDropdown() but always generate a multiple type option that I don't want. How to solve this?

It will also break the laravel query builder :( . For example I have code like this:
ShopCategory::where('parent_id',0)->orderBy('name','ASC')->get() , the condition 'parent_id' = 0 will be ignored.

Thanks

Hi @gieart87
I fixed problems and added placeholder option. Please update package.
Thanks for feedback.


Car simulation

GPIO_IT, I2C, SPI, ADC not tested yet
Errors with pedal/motor_controller projects. Segmentation fault

Closing this


Add more algorithm to the document classification processors

Hi
DecisionTree and RandomForest are supported in the document classification processors using Spark MLLib.
Also, changed the default algorithm from LogisticRegressionWithLBFGS to NaiveBayes.
NaiveBayes seems generally match well in the document classification.
LogisticRegressionWithLBFGS also seems match well with our TF/IDF values.
I think NaiveBayes and/or LogisticRegressionWithLBFGS is/are recommended.
Unfortunatelly, I didn't get a reasonable result with DecisionTree or RandomForest.
In addition, each algorithm supports to set params in the configulation file, like the following,

 algorithm:    "NaiveBayes"
 algorithmParams {
   lamda: 1.0
   modelType: "multinomial"
 }

 algorithm:    "DecisionTree"
 algorithmParams {
   impurity: "gini"
   maxDepth: 5
   maxBins: 32
 }

 algorithm:    "RandomForest"
 algorithmParams {
   numTrees: 10
   featureSubsetStrategy: "auto"
   impurity: "gini"
   maxDepth: 4
   maxBins: 100
 }

Please take a look.
Thank you.

Seems good to me except the spell of lambda. You use the correct spell in your program, but you use incorrect spell (lamda) when you refer conf file.

Oh, Thanks for finding.
I will be fixing it.


Can't see what I'm writing in the Search Widget input field

VSCode Version: 1.15.1
OS Version: MacOS 10.12.6

Steps to Reproduce:

Open the search widget (cmd + F)

Expected Behaviour:
The input field has a reasonable width to see what I'm writing
Detected Behaviour:
The input field is a couple of pixel wide, and seeing what I'm writing is impossible.
Extra Notes:
I've experienced a regression in Version 1.15, since it has always been as expected before. I've also inspected the styles applied to the whole Search Widget, and it looks like the fixed width of 441px for the widget container is causing this issue.
Search widget as it looks on Version 1.15.1

Search widget without 411px width costrain

Reproduces without extensions: Yes

Duplicate of #33349 (it starts a little messed up in some languages)


Update current-request-id-transact-sql.md

Text revisions to tighten and optimize the reading flow of the material.

@fbsolo : Thanks for your contribution! The author, @edmacauley, has been notified to review your proposed change.


Feature/nwps 300 add mulktiple logos to header

Description
https://hee-tis.atlassian.net/browse/NWPS-300
Supporting multiple / different logo's in the header
Checklist

[ ] Tested against our testing policy (Resolution, Browser & Accessibility)
[ ] CHANGELOG entry

Hi @goodvibes - Thanks for the PR.
The logos looks big when rendered as a non-link. Do you think moving logo image specific styles from anchor tag (e.g. <a class="nhsuk-header__link" href="https://www.hee.nhs.uk/" aria-label="Health Education England Website (Opens in New Window)" target="_blank">) to <img> would fix this ?
Also, ticket NWPS-300 describes UK FPO logo should be rendered as a multi-logo, but it is currently being rendered as a base logo, is this something already agreed with Andy/Eleanor ? Thanks!

Have moved them into a span so that can control the styles if there is no anchor tag

If FPO is selected it should be the only log that renders. We don't have the FPO logo yet (and won't need it for a while anyway).

If FPO is selected it should be the only log that renders. We don't have the FPO logo yet (and won't need it for a while anyway).

Okay, thanks. I have had a chat with Eleanor last Friday on this and she suggested to not to add support for FPO logo yet on the backend until a design decision is made and let's just leave it as is. Thanks!


All requests bypass CloudFlare caching

As far as I can tell, all my requests now end up with
cf-cache-status | BYPASS

(this is for a newly created bucket, so it shouldn't be hitting the old max-age: 0 rule). I don't know whether this is because of the Authorization header that the worker adds or something else, but it effectively means I don't get any of the benefits of CloudFlare's CDN caching. I suspect its Authorization since the worker runs "in front" of caching, and without a Cache-Control: public header in the response CloudFlare won't cache the resource.
Is this expected?

Ah, yes, can confirm that it's also necessary to set
{"cache-control":"public"}

in the "Bucket info" field for the bucket. Then caching starts working correctly. Probably worth adding to https://www.backblaze.com/docs/cloud-storage-deliver-private-backblaze-b2-content-through-cloudflare-cdn and to the README :)

Many thanks, @jonhoo - I updated the README, and my technical publications colleague updated the article.

Thank you!


Pass callback to compute Vloc radial integrals

[WIP] Make periodic function (Vloc in our case) from a list of radial integrals
computed for all lengths of G-shells. Later QE will compute the radial integrals
itself.

Hi @haampie ! What's wrong with elpa mpi builds?

Dockerhub being flaky I think, rerun them just now


Added new center style for sections

Issue
Fix #260
Test URLs

Before: https://main--merative2--hlxsites.hlx.page/careers
After: https://cta-fix--merative2--hlxsites.hlx.page/careers
Before: https://main--merative2--hlxsites.hlx.page/careers/drafts/amol/careers
After: https://cta-fix--merative2--hlxsites.hlx.page/drafts/amol/careers

Description
added a section style called 'center' that basically centers the text and margin for any child element to a section.

@amol-anand I think they are going to say something about this div being way wider than the rest of the cta block.


Unable to check two variables using check step - use indexOf()

I am reading two values from ui
Ex :
read h2 to var1
read h2 to var2
Using the following syntax to check, but it always fails
check var1 contains "var2" | "pass"| "fail"
Please suggest

Hi @ShwetaChahan for the check step, when using natural language for the condition contains, I have to check for quotes in order to parse the meaning. Thus only checking text string is possible.
For checking if variable 1 contains variable 2, you can do it the JavaScript way with indexOf() -
a = 'xyz'
b = 'y'
check a.indexOf(b) > -1 | 'pass' | 'fail'

For JavaScript, indexOf() will return -1 if the string is not found,
and position of the string b in a if found (integer 0 or more).

Closing for now until further inputs


Typo on the variable name of verification hash version

The name of the variable has missing "CONSTANCE_" string at the begining

Thanks


Update angr API

This PR will sit until commit https://github.com/angr/angr-management/commit/9cb557454b363b8555c44623b8d3d9f3584a3226 lands on pip of am and breaks our CI :)

With some luck you may see angr/angr-management#733 landing on PyPI tomorrow...


fix(macos): retain and release NSMenu manually

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Docs
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Checklist

[ ] This PR will resolve #___
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary
[ ] It can be built on all targets and pass CI/CD.

Other information

@amrbashir maybe muda need to aware of this. Menu instance should better be freed manually.

Currently the Menu struct in muda can be cloned (We need to clone it on Tauri side on Windows) so manual drop when Menu struct drops is not possible right now but could be added if necessary. were you able to reproduce the same crash with muda?

Using this patch, I can confirm this fixes the issue.
[patch.crates-io]
tao = { git = "https://github.com/tauri-apps/tao?rev=22e5d2276c97f09a27c9ac20d9f6ced11f376a26", rev= "22e5d2276c97f09a27c9ac20d9f6ced11f376a26" }


✨ add replace() support to native

yarn add solito@canary

Going to close this in favor of a solution in the docs. I've found it to be too difficult to do correctly and it's presented many bugs that breaks solito's contract of "give me a URL and i'll get you there".
A combination of certain actions dispatching + a middleware documented should be enough. After all, true replace actions are hopefully somewhat rare enough on native that you can do them with dispatch 🤷♂


Java Android SearchPublicChats

TdApi.SearchPublicChats searchPublicChat = new TdApi.SearchPublicChats("Coindelta");
TgClient.send(searchPublicChat, this);

W/System: ClassLoader referenced unknown path: /system/framework/QPerformance.jar
E/BoostFramework: BoostFramework() : Exception_1 = java.lang.ClassNotFoundException: Didn't find class "com.qualcomm.qti.Performance" on path: DexPathList[[],nativeLibraryDirectories=[/system/lib, /vendor/lib]]
...
...
...
V/BoostFramework: BoostFramework() : mPerf = null
A/DLTD: [1527684465.813129425][td_java_api.cpp:384] [!(0) && no_return_func()]
--------- beginning of crash
A/libc: Fatal signal 6 (SIGABRT), code -6 in tid 7223 (Thread-2)
    [ 05-30 18:17:45.819  3129: 3129 W/         ]
    debuggerd: handling request: pid=6966 uid=10191 gid=10191 tid=7223

I get this Error. I'm doing this on android, can someone guide me?

What TDLib version are you using? Have you downloaded it from https://core.telegram.org/tdlib/tdlib.zip?Have you updated native libtdjni.so?

Please provide additional details and reopen the issue if the problem still persists. Closing for now.


Fix debug mode and add timings

Summary
Would be useful to get timing information when running from CLI
Implementation Details
Use TimerOutputs.jl package and default timer
Additional notes

Check before merging:

[x] All discussions are resolved
[x] New code is covered by appropriate tests
[x] Tests are passing locally and on CI
[x] The documentation is consistent with changes
[x] Any code that was copied from other sources has the paper/url in a comment and is compatible with the MIT licence
[x] Notebooks/examples not covered by unittests have been tested and updated as required
[x] The feature branch is up-to-date with the master branch (rebase if behind)
[x] Incremented the version string in Project.toml file

Codecov Report

Merging #44 (98c12bd) into master (8abe889) will not change coverage.
The diff coverage is 92.59%.

:exclamation: Current head 98c12bd differs from pull request most recent head 9d49377. Consider uploading reports for the commit 9d49377 to get more accurate results

@@           Coverage Diff           @@
##           master      #44   +/-   ##
=======================================
  Coverage   73.21%   73.21%
=======================================
  Files          14       14
  Lines         489      489
=======================================
  Hits          358      358
  Misses        131      131

Impacted Files
Coverage Δ

src/execution.jl
85.18% <81.81%> (+1.18%)
:arrow_up:

src/contexts/base.jl
96.07% <100.00%> (-0.08%)
:arrow_down:

Continue to review full report at Codecov.

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


RedisStreamCommands xRange Range.Boundary.exclude issue

trying to use Range.Boundary.exclude to exclude range parameter, but anyway it returns list with include, please see screenshot:

Please learn how to properly format code and logs.

This is the issue tracker for Lettuce. Please report your issue at https://jira.spring.io.


spotguide: support scraping organization level spotguide repos

In the current setup only banzaicloud github organization's spotguide repos are scraped, we should scrape for spotguide repos in the currently selected user organization as well.
In relation with: #1124

Already implemented.


Remove DB context from S3 instrumentation

Fixes #2820

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-10-04T06:55:56.137+0000

Duration: 56 min 22 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
3156

Skipped
36

Total
3192

:green_heart: Flaky test report
Tests succeeded.
:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark tests.

run jdk compatibility tests : Run the JDK Compatibility tests.

run integration tests : Run the Agent Integration tests.

run end-to-end tests : Run the APM-ITs.

run windows tests : Build & tests on windows.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Optimize Frame parsing process with if/else statements

As only one (1) action is triggered (tvEvent.emit()) for each "interesting" frame scanned, if/else statements structure optimizes parsing process (i.e. not parsing frame anymore once we found a matching the pattern).

please ignore. invalid code


Why is c++ uwebsocket version so badly benchmarked against uwebsocket js version?

https://www.researchgate.net/publication/348993267_An_Analysis_of_the_Performance_of_Websockets_in_Various_Programming_Languages_and_Libraries
what's wrong with the benchmark here?

Something  that  does  not  come  as  too  much  of  a surprise is the fastest websocket. Although, my initial prediction was  that  C  would  perform  the  best,  it  is  not  unexpected  that NodeJS takes the crown. Node’s asynchronous nature allows for greater throughput of requests coming into the server.

Hilarious since the "Node.js" websocket server is uWebSockets which is written in C and C++.

[...]

C++, and Rust performing the benchmark a little slower.

Directly contradicts the previous finding since here the "C++" websocket server is... again... uWebSockets.

You've already hit the nail on the head here - why would uWebSockets wrapped in Node.js outperform uWebSockets wrapped in nothing? And why are they measuring X (websocket implementations) and attributing the difference to Y (the programming language). This is exactly the same flaw TechEmpower does.
Essentially, this is garbage science.  And I'm saying that even though their "finding" put my work at the top.

First of, big fan of your work. I'm newbie programmer so I doubt will understand the magic behind uwebsocket.
Since you mentioned techempower, someone asked me why you didnt submit a pull request to see how it performs.
You do have plenty of writeups with raspberry pi comparison etc. Possible to show how it performs under plaintext? pico v lang is now first for hello world.

Not interested in techempower as they are incompetent and don't understand what they are doing, like mentioned above. Besides, it is already an established fact uWS is quite optimal for what it does - benchmarks were more important in 2016/2018 since that was when the current architecture(s) were made.


版本发布时是否可以标识支持的最高的数据库驱动版本

Why you need it?
Is your feature request related to a problem? Please describe in details
目前服务端使用1.5.2，客户端使用1.4.1  客户端mysql驱动在8.0.22运行正常，再往上就事务回滚过程中反序列化失败，造成无法回滚。
How it could be?
A clear and concise description of what you want to happen. You can explain more about input of the feature, and output of it.
Other related information
Add any other context or screenshots about the feature request here.

贴一下日志

2023-07-23 22:09:48,600#[TID: N/A]#[rpcDispatch_RMROLE_1_32_32]#INFO ######i.s.rm.datasource.DataSourceManager#[info]#-branchRollback failed. branchType:[AT], xid:[<IP_ADDRESS>:61111:18362258022334800], branchId:[9355058946528753], resourceId:[jdbc:mysql://<IP_ADDRESS>:3306/tower], applicationData:[null]. reason:[Branch session rollback failed and try again later xid = <IP_ADDRESS>:61111:18362258022334800 branchId =<PHONE_NUMBER>528753 Class cannot be created (missing no-arg constructor): java.time.LocalDateTime
Serialization trace:
value (io.seata.rm.datasource.sql.struct.Field)
fields (io.seata.rm.datasource.sql.struct.Row)
rows (io.seata.rm.datasource.sql.struct.TableRecords)
afterImage (io.seata.rm.datasource.undo.SQLUndoLog)
sqlUndoLogs (io.seata.rm.datasource.undo.BranchUndoLog)]

这个日志是用的8.0.31打印的，切换到8.0.13就没问题，22也没问题。

使用得seata版本是多少？

This is a known issue, different mysql-driver versions map different Java types after reading the time type. The earlier version uses the Long type, while the later version uses the LocalDateTime of jdk8.


Add new test case that fails at the moment

Not really sure if it's me or Clipper, but there seems to be some issue here.
To me it looks like (at least) one path should be reported as a hole, but it isn't.

Sorry, I didn't tidy up the example data just yet. But I can do it if it helps.

Using IsPositive() instead of IsHole() appears to help, at least a little bit.

HI Juha.
I've been looking at this problem for several hours and will need to spend quite a bit more time on this as it's not an easy fix.
I'm sleeping on it now 😁.

Thank you so much for looking into it! 😍
Take your time, I'm anyway working on something else right now. 🙂

Take your time, I'm anyway working on something else right now. 🙂

I've finally found and fixed the problem (in Delphi only). 🍾🎈🎂
However I won't be uploading until I've done a little more testing (since I've made a number of changes) and I still need to migrate these changes to C++ and C# too.

Sweet! 🚀💪💯
Just yesterday I got back to this and modified my application code to use IsPositive() to decide what is a hole and what isn't, and then identify the parent-child relationships (which I do need) using other means (just finding what's inside what). So far this approach appears to be working, but of course involves additional computation that would be best not to do at all.
At any rate, when your updated version will be available, I can do further testing to see that it gives the same results.
Thanks again for looking into this!

Hi again Juha. This should be fixed now with my latest revision and your (simplified) test case has been added to CI testing.


Kube-proxy to use apiserver instead of etcd in CoreOS getting-started-guide.

@pires can you tell me if this looks right?

I confirmed that the kube-proxy is reading the services from apiserver.  I can't reach the services, but I presume that is due to some other problem in the CoreOS + GCE setup.

Yes, it seems alright. Is --master flag available in 0.11.0? Otherwise, we'll need to update everything to a newer version.

yes, it exists in 0.11.0

@pires for LGTM?  or someone else?

LGTM.


bug: negative sec/plot in project dashboard for some user

Hi, I noticed in some of our recent data collection that one of our interpreters kept having negative sec/plot in the project dashboard. One example is this one: https://app.collect.earth/project-dashboard?projectId=38366
Person #5 in the list had 659 plots completed with -206.18 secs/plot. I asked the person about this, how were they collecting, what could have happened, they mention that they would leave the screen open to leave the computer and do something else, and then after a while would come back to collect. Maybe that is useful information for you.

Thanks for the information @ricds , we'll be investigating this issue on the next sprint


Web server does not respond with 404 when requesting an invalid URL (i.e. not POST, no token)

Describe the bug
I run the adapter in server mode and have Uptime Kuma monitor the adapter's HTTP endpoint by requesting the root URL, e. g. GET http://telegram.example.com
With 3.2.1 the web server will respond HTTP/404 with 404: Resource Not Found in the body.
With 3.3.0 the request will timeout in Uptime Kuma (the timeout is enforced by Uptime Kuma).
If you use curl with 3.3.0, then the request will never timeout. I consider this a security issue since you could bind unlimited resources by issuing requests to the root URL.
To Reproduce
Steps to reproduce the behavior:

Install 3.3.0 or 3.3.1
Configure the adapter to run in server mode
curl the root URL

Expected behavior
404 as before
Versions:

Adapter version: 3.3.0
JS-Controller version: 5.0.19
Node version: 18
Operating system: docker

@Apollon77
@foxriver76
@GermanBluefox
I guess this could be a sideeffect of the required change to @iobroker/webserver. Please check.

Should be fixed now on master @mcm1957

Confirmed. Thank you!

fixed with 3.3.2