More efficient SegmentMetadataQuery

Addresses issue #2081
This PR makes 3 changes:

During merging of SegmentMetadataQuery results, condensing of intervals is postponed. The merging will now just append intervals to the LHS result from the RHS result. JodaUtils.condenseIntervals is now only called when returning a result from the query result Yielder.

Adds "Interval" as a AnalysisType to the SegmentMetadataQuery. This determines if the query will return the list of intervals associated with the queried segments.

Changes the interval endTime on FireHydrants persisted by RealtimePlumber to use the interval of their Sink which is bucketed by segment granularity, instead of getInterval() on the hydrant's  index, which has an end time determined by the max ingested event time. This should help with condensing of intervals.

Running with a 3,450 segment test set on my laptop, the SegmentMetadataQuery takes ~16s to complete without the patch, and ~70ms with the new interval condensing behavior.

@jon-wei this might be in conflict with https://github.com/druid-io/druid/pull/1995 , can you review/merge that one? I think it is ready and very useful to me in many use cases.

:+1:

@himanshug Merged your other PR, resolving conflicts

@jon-wei see if you need to make changes similar to https://github.com/druid-io/druid/pull/2112

@himanshug This one's fine in that respect, the result output won't change since the query uses default analysis types

:+1:

@jon-wei  pls see https://github.com/druid-io/druid/pull/2129


feat: faster append_formatted_to_byte_array

This pull request makes bytes_array formatting slightly faster (~10% cheaper in gas usage).
The previous implementation used to append digits to an array via a first loop and loop again over it to reverse it. This code avoids the second iteration by using recursion to reorder the append calls and directly add the digits reversed.
I think it also makes the code cleaner.
cargo run --bin cairo-test -- --single-file ./corelib/src/test/fmt_test.cairo
Before changes:
running 3 tests
test fmt_test::fmt_test::test_array_debug ... ok (gas usage est.: 181390)
test fmt_test::fmt_test::test_format ... ok (gas usage est.: 1201900)
test fmt_test::fmt_test::test_format_debug ... ok (gas usage est.: 1644520)
test result: ok. 3 passed; 0 failed; 0 ignored; 0 filtered out;

After changes:
running 3 tests
test fmt_test::fmt_test::test_array_debug ... ok (gas usage est.: 166690)
test fmt_test::fmt_test::test_format ... ok (gas usage est.: 1071540)
test fmt_test::fmt_test::test_format_debug ... ok (gas usage est.: 1547670)
test result: ok. 3 passed; 0 failed; 0 ignored; 0 filtered out;

This change is

Thanks for the suggestion, let's save one more check. I think you forgot to negate the is_zero check (instead of checking if you want to stop, you check if you want to continue). Here is what I tried (on both big and small digits):
    let (new_value, digit) = DivRem::div_rem(*value, base_nz);
    let digit_as_u8: u8 = digit.try_into().unwrap();
    if !new_value.is_zero() {
        append_big_digits_util(@new_value, ref byte_array, base_nz);
    };
    byte_array.append_byte(get_big_base_digit_representation(:digit_as_u8));

I now get:
running 3 tests
test fmt_test::fmt_test::test_array_debug ... ok (gas usage est.: 166090)
test fmt_test::fmt_test::test_format ... ok (gas usage est.: 1054380)
test fmt_test::fmt_test::test_format_debug ... ok (gas usage est.: 1526890)
test result: ok. 3 passed; 0 failed; 0 ignored; 0 filtered out;

I tried removing the inline but it consumes a bit more gas, not sure why as I would expect it to be inlined automatically:
running 3 tests
test fmt_test::fmt_test::test_array_debug ... ok (gas usage est.: 169690)
test fmt_test::fmt_test::test_format ... ok (gas usage est.: 1059180)
test fmt_test::fmt_test::test_format_debug ... ok (gas usage est.: 1543690)
test result: ok. 3 passed; 0 failed; 0 ignored; 0 filtered out;

Should I remove it anyway?

Interesting, it seems that test_validate_gas_cost because entry_point_gas_usage actually increased, so it might actually not be cheaper for some use cases. Not sure why though.

what is the change exactly?

Reviewable status: 0 of 1 files reviewed, all discussions resolved (waiting on @yuvalsw)

before: 141_400
after: 206_880

all the values of the test - not just a singular value. also - please compare runs with --print_resource_usage to get exact resource usage.

Reviewable status: 0 of 1 files reviewed, all discussions resolved (waiting on @yuvalsw)

Thank you, here is what I get:
cargo run --profile=ci-dev --bin cairo-test -- \
    crates/cairo-lang-starknet/cairo_level_tests/ --starknet -f test_validate_gas_cost
    Finished ci-dev [unoptimized + debuginfo] target(s) in 0.43s
     Running `target/ci-dev/cairo-test crates/cairo-lang-starknet/cairo_level_tests/ --starknet -f test_validate_gas_cost`
running 1 test
test cairo_level_tests::abi_dispatchers_tests::test_validate_gas_cost ... fail (gas usage est.: 337080)
failures:
   cairo_level_tests::abi_dispatchers_tests::test_validate_gas_cost - Panicked with "Unexpected gas_usage:
     call_building: `3150`.
     serialization: `50950`.
     entry_point: `154000`.".

Error: test result: FAILED. 0 passed; 1 failed; 0 ignored


Remove assets that have invalid reference designators from the TOC

@DanielJMaher , @birdage
This will remove those assets that have '-0000' in them.

@birdage Was Here


Fly.io usage

I get the feeling this is designed to work with fly.io. Could you let us know the intention .

Ok I see what the intent is now.
Thanks for that @benbjohnson


RHIROS 606 fix pdf page numbering

Description
Fixed PDF page numbers
Jira: https://issues.redhat.com/browse/RHIROS-606
Documentation
Array splice
https://reactjs.org/docs/lists-and-keys.html#keys
https://jestjs.io/docs/configuration
Demo
https://drive.google.com/file/d/1um2tnwKBa9fj2jZKHuaajCsq2X-pOE5b/view?usp=sharing

Codecov Report

Merging #106 (0488881) into main (e416ca9) will increase coverage by 3.94%.
The diff coverage is 32.81%.

@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
+ Coverage   13.03%   16.97%   +3.94%
==========================================
  Files          31       37       +6
  Lines         514      642     +128
  Branches      101      124      +23
==========================================
+ Hits           67      109      +42
- Misses        415      496      +81
- Partials       32       37       +5

Impacted Files
Coverage Δ

src/Components/Reports/Common/SystemsFirstPage.js
0.00% <0.00%> (ø)

src/Components/Reports/Common/SystemsTablePage.js
0.00% <0.00%> (ø)

src/Components/Reports/DownloadReport.js
0.00% <0.00%> (ø)

src/Components/Reports/SystemsPDFReport.js
0.00% <0.00%> (ø)

src/Routes/RosPage/RosPage.js
0.00% <0.00%> (ø)

src/Utilities/api.js
0.00% <0.00%> (ø)

src/Components/Reports/Util.js
92.10% <92.10%> (ø)

src/Components/Reports/UtilTestData.js
100.00% <100.00%> (ø)

src/constants.js
74.07% <100.00%> (+7.40%)
:arrow_up:

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 e416ca9...0488881. Read the comment docs.

Thank you for the review @kgaikwad and @r14chandra.


dateOnSelect called twice on today button click

description: dateOnSelect called twice on today button click
type: bug
expected behavior: on today button click dateOnSelect event must fire only once
ng-persian-datepicker: v4.1.1
angular: v13.3.3
node: v16.14.2
npm: v8.6.0

TODO ...


[chore] - Bump gradle-dep-tree version

[x] All tests passed. If this feature is not already covered by the tests, I added new tests.
[ ] All static analysis checks passed.
[x] This pull request is on the dev branch.
[x] I used gofmt for formatting the code before submitting the pull request.

I have read the CLA Document and I hereby sign the CLA
recheck

recheck

Hello @owenmartin-toast,
I'm closing this PR because the suggested fix is no longer applicable to our current codebase. The issue with gradle-dep-tree has already been resolved, as indicated in our response on the issue page.
Presently, we're actively addressing an enhancement for 'jf audit' to effectively handle larger dependency trees within projects. I'll continue to keep you informed about our progress via this link.


Add Testery set up to Tutorials

Alrighty, @q0rban , here's an initial pass at adding a "Testery.io Integration" doc to the docs site. Can you take a look at the example config.yml and make any corrections you spot, and/or comments/fixes for the overall page?

Alrighty, I've updated the Testery doc, but I've also removed the deprecated testing code snippets we discussed yesterday in the Slack, @q0rban . I've added permanent redirects for those to the Testery page; can you verify I've done those rewrite rules correctly, @bchavet ?


[FEATURE]: Implement custom joystick interface for topside computer

Feature Type
Adding new functionality to the BlueROV2 driver
Problem Description
I have been toying around recently with the different control approaches for the vehicle with different robot configurations. The configuration that I have found a fair amount of success with has been running Blue on a topside computer with the implemented blue_joy configuration. However, this solution does not work when the controller is connected to the topside computer with Blue running on the ROV (I have spent a fair amount of time trying to get it to work). This leaves only the keyboard teleop interface for controlling the robot when Blue is deployed onto the ROV (not fun to use)
Feature Description
As an alternative, I think that a custom solution should be implemented to support either host or networked joystick control. It should be possible for users to perform joystick control on either platform using the same interface.
Alternative Solutions
I have tried a lot of things. One thing that I was hoping would work was using socat to forward inputs from the /dev/input/js0 device to a virtual device on the ROV. Unfortunately jstest DOES NOT like that. I have also tried running the joystick interface on the top-side with Blue on the ROV. This could possibly work if I had installed ROS natively on both platforms, but there are issues with doing distributed ROS nodes inside of containers (the DDS does not work).
Additional Context
No response

So after some further research, I think that I can get this to work using usbip instead of needing to re-write a custom interface. I have done some preliminary testing with an XBox One controller, but I don't believe that it's supported. I'm going to try with a Logitech controller next. If that works, I'll close this issue and create a new one requesting a writeup of the setup process

I was able to get usbip to work with a logitech controller (apparently there is an xpad kernel issue with the XBox controllers, and I didn't really feel like rebuilding the kernel to include the patch). Closing this in favor of a new issue requesting a writeup


Cannot find module '../internals/export'

Hi,
first of all, i'd like to thank you for implementing https://github.com/csandman/yup-phone-lite/issues/9 so quickly.
However, since upgrading to version 1.2.0, I'm experiencing this cryptic import error when using in a Next.js project. The problem disappears when I downgrade back to 1.1.2. I would love to use the new "multiple country code" feature introduced in 1.2.0 though :).
The version of Next.js is 12.2.0, i'm running Node v14.20.1 with npm 8.19.2, if that matters.
This is the error I'm getting:
error - Error: Cannot find module '../internals/export'
Require stack:
- /Users/vajajak/cdp-ivory/fe/node_modules/yup-phone-lite/dist/yup-phone-lite.cjs.js
- /Users/vajajak/cdp-ivory/fe/.next/server/pages/dashboard/users/create.js
- /Users/vajajak/cdp-ivory/fe/node_modules/next/dist/server/require.js
- /Users/vajajak/cdp-ivory/fe/node_modules/next/dist/server/next-server.js
- /Users/vajajak/cdp-ivory/fe/node_modules/next/dist/server/next.js
- /Users/vajajak/cdp-ivory/fe/node_modules/next/dist/server/lib/start-server.js
- /Users/vajajak/cdp-ivory/fe/node_modules/next/dist/cli/next-dev.js
- /Users/vajajak/cdp-ivory/fe/node_modules/next/dist/bin/next
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:902:15)
    at Function.mod._resolveFilename (/Users/vajajak/cdp-ivory/fe/node_modules/next/dist/build/webpack/require-hook.js:182:28)
    at Function.Module._load (internal/modules/cjs/loader.js:746:27)
    at Module.require (internal/modules/cjs/loader.js:974:19)
    at require (internal/modules/cjs/helpers.js:101:18)
    at Object.<anonymous> (/Users/vajajak/cdp-ivory/fe/node_modules/yup-phone-lite/dist/yup-phone-lite.cjs.js:2:54)
    at Module._compile (internal/modules/cjs/loader.js:1085:14)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/Users/vajajak/cdp-ivory/fe/node_modules/yup-phone-lite/dist/yup-phone-lite.cjs.js',
    '/Users/vajajak/cdp-ivory/fe/.next/server/pages/dashboard/users/create.js',
    '/Users/vajajak/cdp-ivory/fe/node_modules/next/dist/server/require.js',
    '/Users/vajajak/cdp-ivory/fe/node_modules/next/dist/server/next-server.js',
    '/Users/vajajak/cdp-ivory/fe/node_modules/next/dist/server/next.js',
    '/Users/vajajak/cdp-ivory/fe/node_modules/next/dist/server/lib/start-server.js',
    '/Users/vajajak/cdp-ivory/fe/node_modules/next/dist/cli/next-dev.js',
    '/Users/vajajak/cdp-ivory/fe/node_modules/next/dist/bin/next'
  ],
  page: '/dashboard/users/create'
}

Thanks in advance for taking a look at this :)

Ok, after a while of banging my head in the wall trying to figure out what I broke about the other build, I decided to change the package's approach entirely, away from the rollup. The whole rollup setup was mostly made by the author of the original package yup-phone, so I never got entirely familiar with it.  I ended up making the build process for this package much more similar to another package I maintain, which allowed my to remove a bunch of dependencies, and it seems to work much better.
If you have any issues getting this to work, or if any new issues crop up, definitely let me know!

I started seeing this error with 1.1.2 on a project which built fine in early December last year.
I tried updating to 1.2.0 (same error), 2.0.1 (something else broke: _libphonenumberJs.isSupportedCountry is not a function).
Reverting to 1.1.0 fixed the problem.
Happy to investigate further if it would be helpful.

@alxndrsn  That's odd, I can't seem to make that issue happen. Can you give me some more info about your environment? Node version, how you're using the package (web, node), other dependencies?
Also, have you tried doing a fresh installation of the package? it seems like the libphonenumber-js package might not be getting recognized somehow, or is on the incorrect version.

@csandman This is for a node project.  I only see Error: Cannot find module '../internals/export' in CI (github actions), where I assume the package is installed fresh every time.  I've disabled the github actions yarn cache, and still get the same error.  Dependencies are pinned with yarn.lock, and I have re-run the build with the same node version that worked in early December.


32 friend list components

resolved #32
Added stateless functional components for friend list.
List is currently pulling from dummy data in friendTest.json via props.

Code Cave is taking shape!


Stop wink state changes from stopping after 6 hours

Description:
This fixes an issue that some users see, were after ~6 hours PubNub updates stop getting pushed to HomeAssistant. Not sure why this happens, but after several weeks, maybe months... of testing it appears that requesting a new token, and then issuing a call to Wink's API with that token keeps everything working. So this update will request a new token every 2 hours and follow that with an API poll.
*NOTE: This "fix" doesn't work when using access_token only in your config.
Related issue (if applicable): fixes https://community.home-assistant.io/t/ha-stops-reporting-wink-state-changes
If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[X] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

I have tested, and this appears to resolve the Wink state refresh issue.

Same here. Wink states have updated perfectly since incorporating this PR. 👍

This solves the issue for me as well

Fixed the issue for me.


Feature Request : Highlight a single line and execute just 1 line at a time in integrated terminal

VSCode Version: NA
OS Version: NA

Steps to Reproduce: NA
When editing CLI or PowerShell and executing commands, either we cut-n-paste 1 line at a time or have VSC execute all commands. Is there a way to highlight a single line and execute just 1 line at a time?

Currently all characters are sent as if they were typed, meaning that the lines are evaluated one at a time unless there is a block or line continuation. So I think this is working as expected? There was recently the inverse issue created where PS lines were being evaluated 1 line at a time and not as a block.


Bump 0.7.0

Upon further thought, we could name this release v0.6.1 considering no new features were introduced. The release-0.6 branch has not diverted from main, so we could simply rebase release-0.6 and then create the v0.6.1 tag.


Add documentation and tests for builtins.go

It changes TypeCmd and EnvCmd to functions as they are not changing the state of BuiltinCmds values. ExecCmd probably should be a function too but it was used here so I didn't touch it.

I prefer squashing these commits to keep the merge history clean.


fix: handle iterable Criterion operandRight

What this PR changes/adds
Make JsonObjectToCriterionDtoTransformer handle in operator correctly
Why it does that
support "in" operator
Further notes

cleaned up some unused methods

Linked Issue(s)
Fix #3216
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

Codecov Report
Patch coverage: 70.00% and project coverage change: -0.26 :warning:

Comparison is base (36aa899) 65.37% compared to head (dd9110d) 65.11%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3218      +/-   ##
==========================================
- Coverage   65.37%   65.11%   -0.26%
==========================================
  Files         837      837
  Lines       16822    16777      -45
  Branches      926      918       -8
==========================================
- Hits        10998    10925      -73
- Misses       5452     5487      +35
+ Partials      372      365       -7

Impacted Files
Coverage Δ

...main/java/org/eclipse/edc/spi/query/Criterion.java
25.80% <0.00%> (ø)

...nld/spi/transformer/AbstractJsonLdTransformer.java
15.22% <0.00%> (+2.88%)
:arrow_up:

...ansformer/JsonObjectToCriterionDtoTransformer.java
100.00% <100.00%> (+20.00%)
:arrow_up:

... and 5 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


adding slots

Since you maybe iterate over thousands of messages in a dump it is a good idea to add __slots__ for a better memory consumption and performance.
It seems to save about 5 - 9% time:
/usr/bin/time python3 examples/print_all.py ~/Downloads/rib.20150101.0400.bz2
238.09user 63.81system 5:59.71elapsed 83%CPU (0avgtext+0avgdata 14236maxresident)k
64inputs+0outputs (0major+18652minor)pagefaults 0swaps

/usr/bin/time python3 examples/print_all.py ~/Downloads/rib.20150101.0400.bz2
220.48user 60.77system 5:37.79elapsed 83%CPU (0avgtext+0avgdata 14228maxresident)k
15056inputs+0outputs (16major+18640minor)pagefaults 0swaps

Sorry to keep you waiting...
Thank you for your code.
It was merged.


serviceworker

I was wondering if anyone can point me to some code to allow service worker.
I want to be able to bundle my app, so that part of it load only when needed, and that the routing is encapsulated.
Angular2 and polymer have this, but i am looking for the React way.
thank in advacne

Is this directly related to react-toolbox? I have no experience with service workers and I don't know if the project has anything to do with this...

No, this is not directly react-toolbox related.


spec: close() incorrectly specified as working recursively

What version of CUE are you using (cue version)?

$ cue version
cue version +e3a03a1 linux/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?
package x

#Configuration: {
	configurations: [string]: string
	...
}

#A1Config: #Configuration & close({
	configurations: {
		a: string
	}
})

#A2Config: #Configuration & close({
	configurations: close({
		a: string
	})
})

#A3Config: #Configuration & #A3ConfigStructure

#A3ConfigStructure: {
	configurations: {
		a: string
	}
}

a1: #A1Config & {
	configurations: {
		a: "hello"
		b: "hello"
	}
}

a2: #A2Config & {
	configurations: {
		a: "hello"
		b: "hello"
	}
}

a3: #A3Config & {
	configurations: {
		a: "hello"
		b: "hello"
	}
}

What did you expect to see?
a1.configurations: field `b` not allowed:
...
a2.configurations: field `b` not allowed:
...
a3.configurations: field `b` not allowed:
...

What did you see instead?
a2.configurations: field `b` not allowed:
...
a3.configurations: field `b` not allowed:
...

Credit to @rogpeppe for working this out

This actually a spec issue per @mpvl. Retitled

This have been updated in the spec.

This issue has been migrated to https://github.com/cue-lang/cue/issues/531.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


remove cmdlet docs until it goes live

This PR removes the documentation of the Copy-UserInternationalSettingsToSystem.
The reason for this is that the cmdlet is not yet live, therefore we had documentation of a feature that is not yet available and this was confusing people who wanted to use the cmdlet.
This documentation will be added back once the feature is available for public use.
I will add an issue to track this.

Docs Build status updates of commit bd854d7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/international/Copy-UserInternationalSettingsToSystem.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

I commented on the original PR here https://github.com/MicrosoftDocs/windows-powershell-docs/pull/2177#issuecomment-771164878 -- but also adding a comment here as well. :)
This documentation page was for a new cmdlet that will ship in a future and/or upcoming version of Windows.
Is there some way to maybe flag or denote in the documentation that this cmdlet is only present on upcoming versions of Windows? (Perhaps, something like version xxx or greater?).
Or alternatively, could this documentation be noted as "preliminary" (ie: for an upcoming new cmdlet) perhaps?
Thanks!

Docs Build status updates of commit dd7796f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/international/Copy-UserInternationalSettingsToSystem.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Hi @erik0686
Following are blocking issues that need to be resolved before we can merge:
docset/windows/international/Copy-UserInternationalSettingsToSystem.md

You need to add a redirect for this deleted file.
It appears that this file is not currently listed in a TOC file in this repo, so you don't need to remove it from a TOC file.

Reference
Criteria used for pull request reviews
Blocking

Blocking: When an article is deleted, the PR includes a redirect for the article in the .openpublishing.redirection.json file for the repo and an update to the TOC to remove the entry for the article.

Blocking: Names for new files and folders follow these guidelines: Use only lowercase, dashes, and numbers. Spaces and underscores are not allowed.Use no more than 80 characters, which is a publishing system limit.Use action verbs that are specific, such as develop, buy, build, and troubleshoot. Avoid "-ing" words.Don't use unnecessary small words, such as a, or, and, the, and in.

New files in the repo should follow the naming guidelines described in the preceding criteria about names for new files. The name isn't a blocking issue for this PR, since you're not adding the file, but note this for the future.
I also found that Copy-UserInternationalSettingsToSystem.md is linked in docset/windows/international/International.md. Adding a redirect will prevent this link from being 404, as well as redirect any external links to the article.
When you've addressed the blocking issue, please remove Back to submitter and reapply Sign off to put this PR back in the queue for review.
Thanks,
Gary

@garycentric
@traya1 Mentioned offline that this topic might not need a redirect. This topic will be returned at some later date when the feature is ready and live :)

@garycentric
@traya1 Mentioned offline that this topic might not need a redirect. This topic will be returned at some later date when the feature is ready and live :)

There are search results from Google for this cmdlet, including a link to the page that you want to delete. These kinds of links are part of the rationale for requiring a redirect for a deleted file. Also, simply deleting this file will create a broken link in docset/windows/international/International.md, which I've already noted.

@garycentric should I remove the link from international.md ?? Or is adding the redirect link enough for that?
Also, I believe I don't have enough permissions to add or remove tags. I can't do as you asked and remove the "back to submitter" tag :/

Docs Build status updates of commit fdc992d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docset/windows/international/Copy-UserInternationalSettingsToSystem.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 0d9dbb5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docset/windows/international/Copy-UserInternationalSettingsToSystem.md
:white_check_mark:Succeeded

docset/windows/international/International.md
:white_check_mark:Succeeded
View (win10-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Erik said in e-mail that this was ready for follow-up review, but he doesn't have permission to apply the Sign off label.


make weave build & work with go 1.5

See  #1386 for a failed attempt.

Relevant: https://github.com/weaveworks/scope/pull/488

ah! "The old format still works in 1.5, just spits a warning.". I'd be happy with that, at least for the time being. so it seems we could just resurrectt #1386 minus that change.

so it seems we could just resurrectt #1386 minus that change.

Actually @dpw only reverted the Makefile change, so there is even less to do here (if anything?)

I've reinstated the other Makefile changes that weren't responsible for the problems with 1.4: https://github.com/weaveworks/weave/commit/a67b9f566a376fcf4da3f3e3c164a064c9be7365
So now we have all of #1386 back, except for the -X change, which just means we get a warning with go-1.5.


Linking work items through # not working

I have tried to enter '#' and the ID of the work item in a PR description, but it doesn't work. It just makes the ID a header.

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

ID: 22780209-b809-62af-1d6e-ed4c91db867a
Version Independent ID: 1d6afe6e-6420-2395-0e0b-05902572f7ab
Content: Link work items to other objects - Azure DevOps
Content Source: docs/notifications/add-links-to-work-items.md
Product: devops
Technology: devops-collab
GitHub Login: @chcomley
Microsoft Alias: chcomley

@mason3k - Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Please consider these resources:

Azure DevOps Services
Azure DevOps Support Bot
Azure DevOps on Stack Overflow

(And just to try the "Have you turned it off and on again" question, you're not putting a space after the "#" character, are you?)


[Backport to main] support uploading prebuilt model (#655)

Description
Backport #655 to main branch.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@a872de1). Click here to learn what that means.
The diff coverage is 83.42%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #768   +/-   ##
=======================================
  Coverage        ?   84.48%
  Complexity      ?     1001
=======================================
  Files           ?       93
  Lines           ?     3584
  Branches        ?      325
=======================================
  Hits            ?     3028
  Misses          ?      420
  Partials        ?      136

Flag
Coverage Δ

ml-commons
84.48% <83.42%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...n/java/org/opensearch/ml/model/MLModelManager.java
78.77% <76.34%> (ø)

...arch/ml/action/forward/TransportForwardAction.java
90.76% <77.77%> (ø)

...java/org/opensearch/ml/utils/MLExceptionUtils.java
86.36% <86.36%> (ø)

...ain/java/org/opensearch/ml/task/MLTaskManager.java
62.82% <88.88%> (ø)

...h/ml/action/upload/TransportUploadModelAction.java
98.63% <96.00%> (ø)

...earch/ml/action/load/TransportLoadModelAction.java
90.19% <100.00%> (ø)

...ml/action/load/TransportLoadModelOnNodeAction.java
97.29% <100.00%> (ø)

...va/org/opensearch/ml/indices/MLIndicesHandler.java
70.00% <100.00%> (ø)

...va/org/opensearch/ml/model/MLModelCacheHelper.java
95.76% <100.00%> (ø)

...main/java/org/opensearch/ml/task/MLTaskRunner.java
97.72% <100.00%> (ø)

... and 1 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Expose Webpack's watchOptions.aggregateTimeout in angular-cli.json

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x] feature request

Versions.

@angular/cli: 1.4.9
node: 8.7.0
os: darwin x64
@angular/animations: 4.4.6
@angular/cdk: 2.0.0-beta.12
@angular/common: 4.4.6
@angular/compiler: 4.4.6
@angular/core: 4.4.6
@angular/forms: 4.4.6
@angular/http: 4.4.6
@angular/material: 2.0.0-beta.12
@angular/platform-browser: 4.4.6
@angular/platform-browser-dynamic: 4.4.6
@angular/router: 4.4.6
@angular/cli: 1.4.9
@angular/compiler-cli: 4.4.6
typescript: 2.3.4

Repro steps.

ng serve
The log given by the failure.

N/A
Desired functionality.

In webpack, there is a watchOptions.aggregateTimeout setting that defines the time (in milliseconds) before initiating a browser reload. This allows multiple changes to one or more files to only trigger building once.
Mention any other details that might be useful.

In cli/tasks/build.ts, I see the entry point for watching the files is little more than a pass-through of the cli configuration into the webpack configuration.
TIA!

The default should be 300 milliseconds.  Have you found this number to be inadequate? if there is a more optimal number the default could potentially be changed.

@clydin yes, the default is 300ms. Yes, I find it inadequate. But I question the idea of trying to use a one-size-fits-all approach, when something like this will be purely subjective. For what it's worth, I can appreciate the ideal of not over-encumbering everyone with the full configuration potential of webpack (it's a lot, and it could be very easily messed up).
In this case, I am hoping to successfully argue the inclusion of the watchOptions.aggregateTimeout property so anybody can tweak it to their personal preference. My main argument for this is that it only affects development environments (e.g., no side-effects for production builds), and it has the potential positive impact on individual developer productivity.
Thank you!

Hi all,
Thanks for all the feedback.
If this is still an issue with Angular CLI 12, kindly file a new issue providing all the necessary information.
The general feeling however, is that we should not provide this option as part of our public API unless absolutely necessary and instead if needs be we should update the default value.
Thank you all.


Changing from workflow to action

Issue #, if available:
Description of changes:
PR to fix this error:https://github.com/aws-observability/aws-otel-java-instrumentation/actions/runs/7426995888/job/20211834758
Seems like a workflow cannot be invoked from inside a step (https://github.com/actions/checkout/issues/692#issuecomment-1259789797) , so changing from workflow to action (to allow invocation from step instead of job).
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

It's because you created a composite action not a reusable workflow.

Codecov Report
Attention: 102 lines in your changes are missing coverage. Please review.

Comparison is base (09e6487) 85.71% compared to head (84a9a05) 50.73%.
Report is 209 commits behind head on main.

Files
Patch %
Lines

...ent/providers/AwsAppSignalsCustomizerProvider.java
24.00%
35 Missing and 3 partials :warning:

...gent/providers/AwsSpanMetricsProcessorBuilder.java
0.00%
20 Missing :warning:

...ders/AttributePropagatingSpanProcessorBuilder.java
0.00%
16 Missing :warning:

...viders/AwsMetricAttributesSpanExporterBuilder.java
0.00%
11 Missing :warning:

...try/javaagent/providers/AwsSpanProcessingUtil.java
90.16%
1 Missing and 5 partials :warning:

...vaagent/providers/AwsMetricAttributeGenerator.java
96.89%
2 Missing and 3 partials :warning:

...y/javaagent/providers/AwsSpanMetricsProcessor.java
91.48%
0 Missing and 4 partials :warning:

...t/providers/AttributePropagatingSpanProcessor.java
94.59%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@              Coverage Diff              @@
##               main     #694       +/-   ##
=============================================
- Coverage     85.71%   50.73%   -34.99%
- Complexity       19      264      +245
=============================================
  Files             3       39       +36
  Lines            49     1301     +1252
  Branches          5      141      +136
=============================================
+ Hits             42      660      +618
- Misses            3      609      +606
- Partials          4       32       +28

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


CPU version of recognition

Hi @davisking
Is it possible to get CPU version of face recognition working...? I mean if I compile dLib CPU only will I be able to get recognition working?
Srikanth

@davisking the reason why I am asking is I want to use AWS for recognition and it would be too expensive to use GPU instance....   just for recognition using CPU would be much cheaper option.. obviously I do not know performance consequences if its possible to use CPU version of dlib dnn for  face recognition.
alternately do you have any suggestions for it?

@srikanthreddybethi , All Dlib's algorithms are working on CPU without any changes. When you are compiling Dlib, it searches for CUDA libraries. If they are found - they will be used, if not - everything in DNN will work on CPU (so CPU/GPU choise is done not at runtime, but at compile time)
You can expicitly disable CUDA support, but passing -DDLIB_USE_CUDA=OFF
cmake .. -DDLIB_USE_CUDA=OFF
The simpliest solution - remove CUDA libraries when compiling :)
Also, make sure you have some BLAS libraries installed (OpenBLAS/MKL) to make DNN work fast on CPU

many thanks @e-fominov


id_token  to authenticate the user?

Quote "are used by the client to authenticate the user."
Doesn't this cause confusion ?  Id_tokens are meant to retrieve basic profile information can they be securely used for authentication?

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

ID: a0cace34-34bf-32f7-c857-e4d71d535a77
Version Independent ID: cdbbc4ae-5894-da21-42d7-928b3674bbd1
Content: Microsoft identity platform ID token reference
Content Source: articles/active-directory/develop/id-tokens.md
Service: active-directory
Sub-service: develop
GitHub Login: @CelesteDG
Microsoft Alias: celested

Thanks for the question. We are currently investigating and will update you shortly.

ID tokens are the only standard way for a client to authenticate the user locally (rather than authenticating them to an external service, which is what OAuth access tokens enable).  The OIDC spec summarizes this and includes the point you made as well: "It enables Clients to verify the identity of the End-User based on the authentication performed by an Authorization Server, as well as to obtain basic profile information about the End-User in an interoperable and REST-like manner."
If there's a better word than "authenticate" that you feel would avoid confusion, we can definitely take edits to the page.


Uses _all to follow alias/datastreams when estimating index size

Fixes #24117
Ensures that index size estimation accounts for index aliases, patterns, and datastreams by using _all stats for an aggregate value of size_in_bytes.
All credit to @JeffBolle for this patch.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

@aromanenko-dev yes great call 👍  It should be trivial to add, I've got a WIP for the regression test I'll add to this PR today

Confirmed locally by first seeing all tests pass, then reverting a014637 (i.e. removing @JeffBolle's patch) and re-running the test suite.  When the patch is removed, I received the following expected error:
org.apache.beam.sdk.io.elasticsearch.ElasticsearchIOTest > testSizesWithAlias FAILED
    java.lang.AssertionError: Wrong estimated size
    Expected: a value greater than <1000L>
         but: <0L> was less than <1000L>
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
        at org.apache.beam.sdk.io.elasticsearch.ElasticsearchIOTestCommon.testSizes(ElasticsearchIOTestCommon.java:218)
        at org.apache.beam.sdk.io.elasticsearch.ElasticsearchIOTest.testSizesWithAlias(ElasticsearchIOTest.java:95)

@egalpin Thanks!


chore: Reconstruct errors structure

Closing this pr since this was merged into https://github.com/momentohq/client-sdk-go/pull/14


#912 - Restyle footer

Started working on #912 (though this doesn't close it).
Things that still need to be done:

[ ] Add mailing list sign up form
[ ] A nicer contact us email page

Looks like so:

cc @sophiakc what do you have in mind for the "Call for Volunteers" page?

Coverage remained the same at 99.316% when pulling cbc44db81224775d01d857bcc6d2ac037a6dc86a on change/reform-footer into 44768fd342dd94cff4cf676e1c7382393fe73626 on master.

Hi, @simonv3 I can see there's a pending PR, I am interested in working on this issue, has this been successfully closed?

Hey @roselynemakena, it's not finished yet - please note the two todo items in the first comment on this PR. Feel free to work on this branch though, and commit directly to it - I don't think I'll have time in the next week to finish it.

@simonv3 noted, working on the to do items. :)

Hey @simonv3 , I appreciate you have started working on that. Are there reasons behind some modifications from the initial wireframes?

size of titles
Contribute, instead of Developer?
About us, instead of About
Slack group, instead of Join Slack
FAQ before How it works
Facebook icon type and 2nd instead of 1st
Envelope icon removed besides to other icons, to go to the contact page (not for mailing list)
email input box to subscribe for newsletter

Where is the contact page right now? What do you mean by nicer?
Have you checked the artboard on Figma to look for the attributes or have you only relied on the wireframe uploaded on Github?

Hey @sophiakc
For most of those I just haven't gotten around to them or wasn't paying attention. Also see my todo list in my PR comment.

I am working on some of those issues @sophiakc


GB Items originating from other tools are editable in the spreadsheet

From @kyleblythe on April 2, 2015 13:9
Currently, items originating from other tools (graded assignments, Tests & Quizzes, Lessons, etc) show up as being editable in the spreadsheet interface - scores may be added/changed within these items' cells (http://www.screencast.com/t/jlAA8kH5nmq), but the scores themselves are not reflected within the originating tool.
Since, in Core GB, these types of items are not editable within the GB view, we should make the cells for these non-editable as well. Perhaps also indicate that they are not editable by graying them out slightly / providing a hover-text descriptor.
Copied from original issue: steveswinsburg/gradebookNG#38

On to it. Cheers.

Hey @kyleblythe this should be sorted now. External grades should no longer be editable.


KeyError: 'ViVs7qlgSMiErMZ2kctSCg'

Hi Rhet,
Many thanks for adding the album and people keywording feature.  My photos album finished downloading overnight and I gave osxphotos a try this morning.  I assume I'm doing something dumb but everything I've tried resets in a KeyError.
hebe(adam)$ osxphotos  albums /tmp/iphoto/
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.8/bin/osxphotos", line 10, in <module>
    sys.exit(cli())
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/click-7.1.2-py3.8.egg/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/click-7.1.2-py3.8.egg/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/click-7.1.2-py3.8.egg/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/click-7.1.2-py3.8.egg/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/click-7.1.2-py3.8.egg/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/click-7.1.2-py3.8.egg/click/decorators.py", line 33, in new_func
    return f(get_current_context().obj, *args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/click-7.1.2-py3.8.egg/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/osxphotos/__main__.py", line 429, in albums
    photosdb = osxphotos.PhotosDB(dbfile=db)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/osxphotos/photosdb.py", line 279, in __init__
    self._process_database4()
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/osxphotos/photosdb.py", line 666, in _process_database4
    folder_hierarchy = self._build_album_folder_hierarchy_4(parent_folder)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/osxphotos/photosdb.py", line 1226, in _build_album_folder_hierarchy_4
    parent_uuid = self._dbfolder_details[uuid]["parentFolderUuid"]
KeyError: 'ViVs7qlgSMiErMZ2kctSCg'

This is on 10.14.6 with my photo library on an external HFS formatted drive.
I've tried using a few different versions of osxphotos and both installing from source and using the downloadable executable.  All give a similar error:
hebe(adam)$ ~/Downloads/osxphotos-0.28.8 export /tmp/iphoto/ --directory "{album}" --album-keyword --exiftool --sidecar xmp
Using last opened Photos library: /Volumes/ADAM.1T/Photos Library.photoslibrary
Traceback (most recent call last):
  File "cli.py", line 19, in <module>
  File "site-packages/click/core.py", line 764, in __call__
  File "site-packages/click/core.py", line 717, in main
  File "site-packages/click/core.py", line 1137, in invoke
  File "site-packages/click/core.py", line 956, in invoke
  File "site-packages/click/core.py", line 555, in invoke
  File "site-packages/click/decorators.py", line 27, in new_func
  File "site-packages/click/decorators.py", line 17, in new_func
  File "osxphotos/__main__.py", line 1105, in export
  File "osxphotos/__main__.py", line 1396, in _query
  File "osxphotos/photosdb.py", line 279, in __init__
  File "osxphotos/photosdb.py", line 666, in _process_database4
  File "osxphotos/photosdb.py", line 1226, in _build_album_folder_hierarchy_4
KeyError: 'ViVs7qlgSMiErMZ2kctSCg'
[6138] Failed to execute script cli

Happy to provide any details that are useful.
Best wishes from New Zealand,
Adam.

@adamshand ugh...definitely a bug.  Thanks for opening an issue.  I'll take a look.  If you don't mind sharing a copy of your database file it would be very useful for debugging.  The file should be in ~/Pictures/Photos Library.photoslibrary/database/photos.db.  The file contains metadata about the photos including names, locations, etc. but no actual images.  Easiest way is probably to zip it and send via something like Firefox Send to<EMAIL_ADDRESS>If you can't share the database I can add some debugging code and have you run with debugging enabled.

Uploading now.  Will email you the link.

I think I found the bug (wasn't filtering out "special" album types) -- try 0.28.10 and let me know if it works.

Seems to be working, woohoo!  DIdn't expect to see all my old iPhoto events in there.  That's cool.
hebe(adam)$ ~/Downloads/osxphotos-0.28.10 --db /Volumes/ADAM.1T/Photos\ Library.photoslibrary/  export /tmp/photos --album Bike
Exporting 8 photos to /tmp/photos...
  [####################################]  100%

Yep this looks like it's working.  Thank you so much, this is epic.
The only thing I wish it did differently was include folders in the album name.  At the moment if I have a photo album called "Bike" in a folder called "Projects" what comes through in the take is just "Bike".  It would be ideal if it came through as "Projects/Bike".  Easy enough to work around by renaming albums before exporting.
Thanks for the example command line, it would have taken me a while to figure out that I needed "--exiftool --sidecar xmp" in order to get the the tags into the pictures.
Unfortunately something about the way the tags are written causes Shotwell under Linux to crash (screenshot attached in case it means anything to you).

When I look at the metadata with ImageMagick I see an "unknown" line, I wonder if that's what is causing problems?  Fortunately digiKam seems to be fine with it, so not a huge deal for me.
    Profile-iptc: 47 bytes
      Keyword[2,25]: Wanganui
      unknown[2,0]:
      Keyword[2,25]: Tink Printing
      Keyword[2,25]: Weta

Thanks again.

Hrm, just blew away the cache and db for Shotwell and tried again and it seems to be working fine now.

Glad it's working!  You can get folder name/album with “{folder_album}” as the --directory template
--exiftool writes the metadata to photos with exiftool
--sidecar xmp writes the XMP file some apps like lightroom use to import metadata. If you're using exiftool you probably don't need this, it's just provided for completeness or for those that don't want to modify the exif in the photos.
I'm not familiar with shotwell and indeed have done little testing with the exported metadata -- it's possible there's something amiss in the metadata. Do you know if the error was reading exif data or the XMP?  If you run into other issues with metadata open an issue and I'll look into it.

That's useful thanks.  Probably won't bother with the sidecar then, it can always be regenerated if need be if the data is in the actual files.
What I was meaning is for the folder name is for it to be part of the metadata (as opposed to the exported folder structure).  Because I've got photos going back to the 1990s I organised my Photos.app albums like:
1990s
  - 1990
    - Favourites
    - Event7
    …
  - 1991
  …
2000s
  - 2000
    - Favourites
    …
  - 2001

In my case the folders are essentially part of the album name and with them albums names can be ambiguous.
Easy enough to rename my albums to make sure that everything exports in a way that's useful to me, but I wonder if using "folder/album" as the album tag would be better for people?

That's reasonable -- I could add a flag to treat folder as keyword.  In your example would you want separate keywords for 1990s, 1990, Event7 or a single keyword like 1990s_1990_Event7? (eg whole path)

I'd prefer them as a single keyword (eg. "1990s/1996/Favourites") because I'm using the names to build an album folder structure similar to what I had in Photos.app.

OK -- that seems reasonable. If / is valid in a tag and your software will read those then definitely makes sense to include the "/".  I opened issue #117 to track this. Should be a fairly easy fix.


Get these two warning when I installed it today

npm WARN deprecated<EMAIL_ADDRESS>This package is no longer maintained
npm WARN prefer global<EMAIL_ADDRESS>should be installed with -g

Hi,
not related to grunt-sassdoc but sassdoc itself.
Not much we can do about it actually, but they are "just" warnings.
For the default theme rewrite: SassDoc/sassdoc-theme-default#96
There's also an issue on the marked project to remove this warning.


Add ThinkingManSoftware.Dimension4 v5.31.331.0

Microsoft Reviewers: Open in CodeFlow

Maybe use the original link?
http://www.thinkman.com/dimension4/d4time531.msi

@SoftCreatR Are you able to review this and move it forward?  Feels like it's stuck.

@SoftCreatR Are you able to review this and move it forward? Feels like it's stuck.

Unfortunately, i'm just a regular user like you ;) But it's assigned to @kevinlams so it's just a matter of time, until your PR will be processed.
Or maybe, @Masetty123 wants to take it over, since he/she is a little bit more active atm.


In config command, add info about the model of the motors

Add this info in the config command dedicated to display "detailed" robot structure (aka with -M flag).

Done.


Help: How to call my own API hosted on https://xxx.appspot.com/myapp

I need to post some data that is generated periodically to my GAE app using a PHP script. I don't really understand how to make this library work for that. I generated a service key which my script points to like this putenv('GOOGLE_APPLICATION_CREDENTIALS=/Users/me/myfile.json');
However the docs at https://github.com/google/google-auth-library-php talks about scopes like Drive etc, and I am not using Drive, just my own API at that URL.
I used to have this script automatically uploading data last year until that way of authenticating was deprecated.
What is the "scope" of a GAE app? How would the sample code below "Call the APIs" be changed to allow POSTs to an GAE app?

Hmm, are you using GAE's admin authentication in app.yaml? If not, how is your authentication defined? You could try something like this:
$client = new Google_Client();
$client->setAuthConfig('/Users/me/myfile.json');
$http = $client->authorize();

$request = new GuzzleHttp\Psr7\Request('POST', 'http://myapp.appspot.com/my-api', [], $postData);
$response = $http->send($request);

Please let me know if this is helpful.

I do use GAE's admin authentication. However I get Class 'Google_Client' not found. Do I need another library than google-auth-library-php for that?

I changed to 1.0...
{
    "require": {
		"google/apiclient": "1.0.*@beta",
		"google/auth": "^0.5.0"
    }
}

and now get error on the Google_Client->setAuthConfig('....path.json') line:
PHP Fatal error:  Uncaught exception 'Google_Exception' with message 'Invalid client secret JSON file.' in /Users/somepath.json

I do use "Service account" as per the documentation here https://github.com/google/google-auth-library-php - do I need another json key file?

Try upgrading to "google/apiclient": "~2.0@RC"

Ok now I get
PHP Fatal error:  Uncaught exception 'GuzzleHttp\Exception\ClientException' with message 'Client error: `POST https://www.googleapis.com/oauth2/v3/token` resulted in a `400 Bad Request` response:
{
 "error": "invalid_scope",
 "error_description": "Empty or missing scope not allowed."
}

Just to be clear, I am not calling any Google API, just my own API that I created at xxx.appspot.com/someapi.

This library is meant for calling Google APIs. If you're exposing your own API, then you will be implementing the authentication.

The API is mine of course but the authentication is Google's. I haven't had time to look into this.

Then you should try following the instructions in the readme to attach the
appropriate middleware to your guzzle http client, and then just make the
request to your own api endpoint. This will send the request with an access
token in the header. You could also send in the Id token and verify on the
server side that it's indeed from Google.
On Wed, Feb 24, 2016 at 4:25 PM jonnyijapan<EMAIL_ADDRESS>wrote:

The API is mine of course but the authentication is Google's. I haven't
had time to look into this.
—
Reply to this email directly or view it on GitHub
https://github.com/google/google-auth-library-php/issues/90#issuecomment-188532655
.


Add a better emoticon than /shrug

Never even done node or hubot, just guessing how this works

"module.exports = function (spiderShrug) {"
should be
"module.exports = function (robot) {"
If you run hubot in cmd/terminal you can test scripts and commands.


LANDrop on Ventura

I'm running ventura on my m1 air, and  it doesn't search for any devices, nor can it be found by any devices.

The moment it first starts, it can be detected by other devices, then it disconnects immediately.


Doesn't work with inline script

https://github.com/dotnet/maui/actions/runs/4077268811/jobs/7026077700
When trying to put script like: https://github.com/dotnet/maui/blob/main/.github/workflows/dotnet-autoformat-pr.yml#L17
dotnet format .Microsoft.Maui.sln --no-restore --exclude Templates/src/BlazorWebView/src/SharedSource/BlazorWebViewDeveloperTools.cs

@rmarinho the v0.2 tag has been pushed to the fixed commit, so restarting any actions should work now.


Add live demo to effect docs

Coverage remained the same at 62.947% when pulling 1816c508802df086cbde63a58374960f556a6237 on x/effects-doc into 1e61b1cf1abaec37e3bcb882ae9fc7524e53ef2a on master.


Manage Household page doesn't work for New household object

hub thread: https://powerofus.force.com/0D58000002tINz9

**lurch: add

Tracking W-011056

Included in beta release 3.88 (Beta 7)

Included in production release rel/3.88


population graph for students started in open university before enrollment date

see eg 0 credit students in 2017 fall BSc

Now graph starts from the date of enrolment despite open univ studies before that


RawTransaction::_decode_script error calculation sctipts

In block
} else if ($code <= 78) {
                // In this range, 2^($code-76) is the number of bytes to take for the *next* number onto the stack.
                $szsz = 2 ^ ($code - 76); // decimal number of bytes.
                $sz = hexdec(substr($script, $pos, ($szsz * 2))); // decimal number of bytes to load and push.
                $pos += $szsz;
                $push = substr($script, $pos, ($pos + $sz * 2)); // Load the data starting from the new position.
                $pos += $sz * 2;
            }

$szsz = 2 ^ ($code - 76); ^ is not function as pow, this is binary operation, nead use function pow
write code
$szsz = pow(2, ($code - 76));
and
$pos += $szsz; wrong set new position
nead
$pos += $szsz *2;

This has been fixed, apologies for leaving this open so long.


Swipe slowly to the right, then it occurs bug - unrecognized selector sent to instance...

I used this swipe-cards plugin for my project but I can see the critical bug here.

When I was moving slowly to the right with the card then I can see the critical error screen is occurred.
So Is there anyone who can help me with this problem?

Same issue here. Did you manage to fix it @alexprodev ?


Trim job suffix to extract out cron job name for workload metadata (#…

…27195)

special case cron job processing in webhook

update

Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@bianpengyuan: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.6
b7ae49370832c61b32e9bc85823c61cd8f7fbbab
link
/test gencheck_istio_release-1.6

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

@bianpengyuan: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.6
b7ae49370832c61b32e9bc85823c61cd8f7fbbab
link
/test gencheck_istio_release-1.6

unit-tests_istio_release-1.6
b7ae49370832c61b32e9bc85823c61cd8f7fbbab
link
/test unit-tests_istio_release-1.6

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


aqd can't determine broker version from the RPMs

One would have to add the following in aqd.conf:
[broker]
version = 1.11.24_rc1

It's related to the following method: https://github.com/quattor/aquilon/blob/6c8b90b47b19a41d974a41050e6de9e9ad850ae0/lib/python2.6/aquilon/worker/processes.py#L276
For more information, check issue #25

Two approaches come to mind:

Modify the default config file at build time to contain the current version number.
Write the version number to a file (/etc/aqd-release?) shipped by the RPM and get aquilon to check that file.
Extend the logic in cache_version() to check for installed RPMs.

@gombasg  notes that the code should set the version to "unknown" if it cannot determine a version, so there may be a simple bug in this code path that can be fixed.

seems to be done: https://github.com/quattor/aquilon/blob/1ecf0200b8bfdf219a76eb5b8a206e97cf5c3921/lib/aquilon/worker/processes.py#L231


Once shadows are enabled for OmniLight and Spotlight, cannot be disabled

Godot version
v4.0.alpha.custom_build [7355dfb50]
System information
Ubuntu 21.04. RTX 2060
Issue description
And a OmniLight or Spotlight and enabling shadows for any, it set the shadows enabled for ever
Steps to reproduce
Add a SpotLight or OmniLight
Enable shadows
Disable shadows (shadows still enabled)
Minimal reproduction project
I doesn't require a MRP since you have to add the light to reproduce

Duplicate of https://github.com/godotengine/godot/issues/50343.


nRF clock_control add ZLI API

Add zero latency ISR safe APIs to the vendor specific nrf_clock_control device driver API. These are used by components running in zero latency interrupts, primarily the Bluetooth stack.

Looks good, CI issue is unrelated to this PR :)

CI is green now

CI is green, PR is ready for merge


R-T3.4-1: The defect-prediction tool must provide APIs to be easily integrated with other tools in RADON and with the IDE

ID
R-T3.4-1

Section
WP3: Methodology and Quality Assurance Requirements

Type
FUNCTIONAL_SUITABILITY

User Story
As an Operations Engineer I want the tool to be integrated with the IDE and other tools in RADON

Requirement
The defect-prediction tool must provide APIs to be easily integrated with other tools in RADON and with the IDE

Extended Description
The tool must be integrated with other tools in RADON as well as a plugin in popular IDEs such as Eclipse (e.g., Eclipse Che). Mainly, the requirement will be fulfilled to make sure the defect-prediction tool to be integrated with Winery/IDE as well as the delivery toolchain

Priority
Must have

Affected Tools
DEFECT_PRED_TOOL

Means of Verification
Direct implementation on IDE, feature checklist, case-study

Dependency

@gcasale
This requirement is satisfied, though further APIs will be provided in the next future versions of the tool.

Closed


Global filter not working as expected

v3.28
Here is the EC2Address's tags:

When I use the common preset, the resource is excluded as expected. However, when I use the __global__ filter, I expect the behaviour to be identical (resource should be filtered), but the resource is marked for deletion.
__global__:
  - property: tag:Name
    type: glob
    value: ck8scapi*

presets:
  common:
    filters:
      EC2Address:
        - property: tag:Name
          type: glob
          value: "ck8scapi*"

resource-types:
    includes:
      - EC2Instance

accounts:
  0183023xxxx:
    presets:
      - common

I'm having same issue on v3.29.1, only trying the global: config now so not sure if this worked in the past or not

Ok, the documentation should be updated for clarity. Right now it lacks contexts for the code snippets. For example, __global__ example has no indentation and is the only key available in the snippet, so it's impossible to know where it should go.

I appreciate the feedback, the documentation was written as pseudo code so that it was easier for copy/paste directly into configurations. I can understand how that could be misleading. I've opened #399 to help clarify usage.
@jbcorreia it should 100% work under any valid filter location, this means preset filters or account filters. There is no top level filters section in the configuration.

:tada: This issue has been resolved in version 3.29.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Drop support for un-used code that target net framework

Remove noisy code that target .net framework

great job


Can you integrate redux to project?

Can you integrate redux to project?

Redux has integrated to v0.2.0(https://github.com/BelinChung/react-native-hiapp/commit/36ac6a9db9df7fd3231ca27302f3523820a77970).


polish index.html

name: :memo: Documentation Fix
about: Fixing a problem in an existing docs page
Checklist

[ ] Is there an existing issue for this PR?

link issue here

[ ] Have the files been linted and formatted?

What docs page needs to be fixed?

Section:
Page:

What is the problem?
What changes does this PR make to fix the problem?

Hi. Appreciate the PR, but I don't think these changes are needed atm.
That said, we would be interested in trying to update some of the other examples to match our current "Modern Redux" approaches:
https://github.com/reduxjs/redux/issues/3917


Fdg 6443 Across AFG Explainers - Update Section Anchor Point URLs

https://federal-spending-transparency.atlassian.net/browse/FDG-6443
No change in coverage

It looks like the names for the anchor points are now consistent with the section titles with the exception of 'The Difference Between the National Deficit and the National Debt' on the Deficit page, which has the anchor 'deficit-vs-debt'. Was wondering if this was due to section name length?

Good catch, the anchor name was just missed


Invalid JSON being generated by save_json methods

The JSON encoder used can output NaN and infinite values (see the most_common_value label in the example below).
This is valid javascript, but not valid JSON, and can create problems down the pipeline when another service tries to deserialize it.
The standard json.dumps method has an allow_nan parameter that can be used to control this behaviour, and it would be good to have access to it (with the default being left as it is right now for compatibility).
Generally speaking, it would be good to have access to all the parameters used by json.dumps, perhaps by adding some catch-all **kwargs to the .json() method in evidently and passing that along to the json.dumps call.
Example:
{
    "feature_type": "num",
    "count": 2492,
    "infinite_count": 0,
    "infinite_percentage": 0.0,
    "missing_count": 190,
    "missing_percentage": 7.08,
    "unique_count": 2482,
    "unique_percentage": 92.54,
    "percentile_25": -0.11,
    "percentile_50": 0.35,
    "percentile_75": 1.41,
    "max": 241887.34,
    "min": -1.0,
    "mean": 138.11,
    "most_common_value": NaN,
    "most_common_value_percentage": 7.08,
    "std": 4949.03,
    "most_common_not_null_value": 1.098466433173,
    "most_common_not_null_value_percentage": 0.07,
    "new_in_current_values_count": null,
    "unused_in_current_values_count": null
}

Hi @jlopezpena, thanks for your suggestion, it is a great point. We were unable to fit in the current release as we focused on major refactoring. But we took it into work and will make the changes in the next release in November.


How to use this library in Swift app?

I was directly downloaded into the library project. Solve the problem arc. However, an error log is as follows.
Undefined symbols for architecture x86_64:
  "_xmlFreeDoc", referenced from:
      -[DAVResponse initWithMethod:headers:bodyData:resourcePath:rootPath:] in DAVResponse.o
  "_xmlReadMemory", referenced from:
      -[DAVResponse initWithMethod:headers:bodyData:resourcePath:rootPath:] in DAVResponse.o
  "_xmlStrcmp", referenced from:
      -[DAVResponse initWithMethod:headers:bodyData:resourcePath:rootPath:] in DAVResponse.o
      __XMLChildWithName in DAVResponse.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

@6ag hi,are you resolve it?

I solved the issue by adding /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS.sdk/usr/lib/libxml2.2.tbd file to xcode project.


Assessment metadata API (Azure Security Center)

Review summary:
Picked up from this PR:

added a "put" scenario and an example.
remove "secureScoreWeight" field
add "severity" field

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[x] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

Automation for azure-sdk-for-python

@yungezz I've tidied the PR - please review

@yungezz can you please review?
we would really like to complete this PR

@yungezz
can you please review this PR?

hi @erich-wang could you pls review the PR since ARM signedofff?


PENV-384 api records by JetDropID

- What I did
- How I did it
- How to verify it
- Description for the changelog

Codecov Report

Merging #95 into master will increase coverage by 0.44%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
+ Coverage   75.09%   75.54%   +0.44%
==========================================
  Files          44       45       +1
  Lines        2811     2895      +84
==========================================
+ Hits         2111     2187      +76
- Misses        559      564       +5
- Partials      141      144       +3

Impacted Files
Coverage Δ

test/api/apiwrapper.go
100.00% <100.00%> (ø)

testutils/generators.go
93.53% <100.00%> (ø)

etl/transformer/transformer.go
82.19% <0.00%> (-1.68%)
:arrow_down:

configuration/configuration.go
100.00% <0.00%> (ø)

etl/connection/credentials.go
87.50% <0.00%> (ø)

etl/connection/grpc_client.go
86.04% <0.00%> (+5.09%)
:arrow_up:

etl/models/models.go
100.00% <0.00%> (+8.00%)
:arrow_up:

instrumentation/converter/platform.go
100.00% <0.00%> (+16.66%)
:arrow_up:

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 d439269...37d85a0. Read the comment docs.


Change condition delete reminder recommender new reviewers needed

 Change the condition to delete the pending email (send to the recommender) #ReminderRecommenderNewReviewersNeeded


HoloLens 2 : Cannot connect to destination host - Bad access

Hello everyone,
I followed this MRTK-Webrtc tutorial from Microsoft: https://microsoft.github.io/MixedReality-WebRTC/manual/unity/helloworld-unity.html
Everything goes well, until the HoloLens 2 tries to connect to the node-dss server. The answer in the console is: "Cannot connect to destination host" or "Curl error 7 ... : bad access" to the IP address of my computer.
I tried to deploy the application on a second computer and it worked. The problem seems to come from the deployment on HoloLens 2.
Can someone help me? I have searched many forums and issues on this git without finding the solution. Many people were talking about firewalls (which I tried to disable but without success).
I'm using Unity 2019.4.26f1 and MRTK-Webrtc 2.0.2
Here is a picture of the development console on HoloLens 2 :

Thanks a lot !

Sounds like a firewall issue of some kind to me too.
Assuming you've already tried some basic troubleshooting, some other things to consider are:

In your HoloLens UWP, ensure the PrivateNetworkClientServer capability is enabled.
If this is not enabled, the UWP sandbox will prevent traffic to the local network (e.g. it only allows communication to the internet).
(This is not normally an issue for production apps, unless you plan your production build to also connect to a local server)
Try (temporarily) turning off all firewalls on the PC with DSS (if you use a third party firewall/antivirus, you may find that Windows Firewall is also enabled)
If your DSS server is (for some reason) in some kind of sandbox (e.g. a UWP application), ensure that has InternetClientServer capability. If you are running your DSS server in a VM, WSL, Docker etc, there are similar things to check here (by default, Docker and WSL seem to use the windows firewalls though).
Consider the internal networking too - does your WiFi Network/Router "isolate" clients (e.g. prevent clients from talking to eachother?) If your desktop uses Ethernet, is it on the same network as the WiFi access point? If it's a corporate network, you may have network-level firewalls between the devices too.

Hello, thank you for your answer !
Finally i was able to Deploy WEB-RTC to another Unity project.
Having compared the configurations of the two projects, I find no difference. I can't explain why in the first one it doesn't work.
However I suspect that my first project (which does not work) was too far from the root of the C:\ drive and lost some files during compilation. I saw this information on MRTK Documentation :

And when importing MRTK i was maybe loosing some important files. But I Can't confirm that..


fix(flower-search): updated url for 17flowers data set

updated url for 17flowers data set. Otherwise get_data fails.
UPD:
The old URL works, but it downloads a redirect page, then tar command fails.
$ curl http://www.robots.ox.ac.uk/~vgg/data/flowers/17/17flowers.tgz
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>301 Moved Permanently</title>

Moved Permanently

The document has moved here.

test failure seems to be related to https://github.com/jina-ai/examples/issues/276

test failure seems to be related to #276

Yes, I have re-run the checks now. It shouldn't fail

Can someone merge this please? It's failing a test for multires-lyrics search, yet it doesn't touch that code at all. From my testing over past week so many of our examples have issues that tests will fail regardless of whether the PR code is good or not

Can someone merge this please? It's failing a test for multires-lyrics search, yet it doesn't touch that code at all. From my testing over past week so many of our examples have issues that tests will fail regardless of whether the PR code is good or not


Gnome-40.4 support

List of changes:

Added Gnome-40.4 to the list of supported versions in metadata.json.
Updated installations steps in README to include checking that the PanGPUI is a startup application. (Failing to do so will result in Gnome freezing when you click on the icon).
Added launcherButton cleanup to destroy function in order to allow disabling and then re-enabling the extension without needing to restart the gnome shell.

Thank you!


Example in README errors

Version: 0.23.0
Platform: Linux

Type: Question/enhancement
Severity: Low
Description:
I tried to use the example of the README as a template. I then got an error like
/home/vmx/src/pl/js-graphsync/node_modules/libp2p-pnet/src/crypto.js:81
    throw new Error(Errors.INVALID_PSK)
    ^

Error: Your private shared key is invalid
    at module.exports.decodeV1PSK (/home/vmx/src/pl/js-graphsync/node_modules/libp2p-pnet/src/crypto.js:81:11)
    at new Protector (/home/vmx/src/pl/js-graphsync/node_modules/libp2p-pnet/src/index.js:24:24)
    at new Node (/home/vmx/src/pl/js-graphsync/libp2pnode.js:39:24)
    at PeerId.createFromJSON (/home/vmx/src/pl/js-graphsync/consumer.js:17:22)
    at waterfall (/home/vmx/src/pl/js-graphsync/node_modules/peer-id/src/index.js:271:7)
    at /home/vmx/src/pl/js-graphsync/node_modules/async/internal/once.js:12:16
    at next (/home/vmx/src/pl/js-graphsync/node_modules/async/waterfall.js:21:29)
    at /home/vmx/src/pl/js-graphsync/node_modules/async/internal/onlyOnce.js:12:16
    at pub.hash (/home/vmx/src/pl/js-graphsync/node_modules/peer-id/src/index.js:252:13)
    at Multihashing.Multihashing.digest (/home/vmx/src/pl/js-graphsync/node_modules/multihashing-async/src/index.js:33:5)

I thought my peer-id was wrong. I then figured out that the problem was the
connProtector: new Protector(/*protector specific opts*/),

what about commenting that out and adding a comment that this is for private networks?
Once commenting it out I get another error:

[1] "peerRouting" is not allowed
[2] "contentRouting" is not allowed
    at Object.exports.process (/home/vmx/src/pl/js-graphsync/node_modules/joi/lib/errors.js:196:19)
    at internals.Object._validateWithOptions (/home/vmx/src/pl/js-graphsync/node_modules/joi/lib/types/any/index.js:675:31)
    at module.exports.internals.Any.root.validate (/home/vmx/src/pl/js-graphsync/node_modules/joi/lib/index.js:138:23)
    at module.exports.internals.Any.root.attempt (/home/vmx/src/pl/js-graphsync/node_modules/joi/lib/index.js:167:29)
    at module.exports.validate (/home/vmx/src/pl/js-graphsync/node_modules/libp2p/src/config.js:42:17)
    at new Node (/home/vmx/src/pl/js-graphsync/node_modules/libp2p/src/index.js:32:16)
    at new Node (/home/vmx/src/pl/js-graphsync/libp2pnode.js:82:5)
    at PeerId.createFromJSON (/home/vmx/src/pl/js-graphsync/consumer.js:17:22)
    at waterfall (/home/vmx/src/pl/js-graphsync/node_modules/peer-id/src/index.js:271:7)
    at /home/vmx/src/pl/js-graphsync/node_modules/async/internal/once.js:12:16

I don't know what they are for, but commenting both occurrences makes things not error anymore.

peerRouting and contentRouting will be used for the upcoming delegated routing work, they should just get remove in the interim. I think commenting out connProtector is a good approach.

I'll do a PR tomorrow.


Fix read png windows bug and input visual progress

Updates to fix input.visual progress for a folder of multiple files.
Also updated input backend classes to fix opencv “bug” on Windows platform when dealing with numbered PNG file names.

Updated as suggested.


fonts not loading on Windows

Extra fonts aren't showing up as choices on windows, only the default Source Code Pro.
Check linux as well.

@psifertex Just noticed some issues with fonts in OS X and Linux that might be related to the above. If you change the default font size, next time you start Binary Ninja, the font is set to Courier 12 and the listing in preferences for the font is set to @Variant, resetting to default fixes it for the running instance, but when you restart same issue again.  This is on the commercial version latest dev.

It looks like QSettings is broken on OSX. The font is serialized properly on Linux, but not OS X.
We've got longer-term plans of implementing our own custom preferences backing which is probably going to be the right fix for OS X not being able to persist font changes.
I think, however, that this is likely a different bug than the fonts not loading on Windows so I'm going to leave this one open to track the original.

Looks like the QT upgrade fixed this for us. Source Code Pro now properly bundled and shipped with Linux, Windows, and MacOS.


GEOMESA-1916 Docs - Spatial UDTs are incorrect

Signed-off-by: Atallah Hezbor<EMAIL_ADDRESS>

looks good, thanks


google collab  code not working or no update?

Hi, the from_local args is depricated, please remove it before use.

got it @yangheng95


Pushing events to Viewports no longer triggers InputEventMouseMotion in _gui_input

Tested versions

Reproducible in Godot 4.3-dev5
Works in Godot 4.3-dev4

System information
Windows 11 Pro - Godot 4.3-dev5
Issue description
If you push an InputEventMouseMotion to a viewport using push_input, it will not be received by _gui_input methods in its controls. This was introduced by 99ba2519eaf79271c86b8a932ce185666fb79a37, specifically the change to mouse_in_viewport default of false when it was previously true.
mouse_in_viewport having a default of false does make sense, but it also doesn't make sense to me that pushing events inside a viewport do not work. Perhaps receiving events should update that flag?
There is a workaround you can call from your game code. If you trigger NOTIFICATION_VP_MOUSE_ENTER that will set the flag and your events will be passed as they were before.
Steps to reproduce
I have included a zip file that reproduces the issue. Simply move the mouse and look at the console to see events are not being received. If you uncomment sub_viewport.notification(NOTIFICATION_VP_MOUSE_ENTER) in main.gd then you will see the events. Additionally, using a version of godot prior to 99ba2519eaf79271c86b8a932ce185666fb79a37 such as 4.3-dev4 will work.
Minimal reproduction project (MRP)
Here is the project:
test-pushinput.zip

In your MRP, your are using a TextureRect for displaying the SubViewport and a Node2D for propagating the input events.
Are your aware of SubViewportContainer? That node is supposed to handle both display and input-propagation to SubViewports automatically.
Or do you have special needs that can only be accomplished by using a TextureRect?


Regulator power state control

Generic regulator power state interface, and associated support for the nPM1300

Thinking about this in light of the refactoring in https://github.com/zephyrproject-rtos/zephyr/pull/58229, it would make more sense to have the power control for the entire device at the MFD level.
Would it make sense to add a power management action handler at that level to implement PM_DEVICE_ACTION_TURN_OFF instead?

Thinking about this in light of the refactoring in #58229, it would make more sense to have the power control for the entire device at the MFD level. Would it make sense to add a power management action handler at that level to implement PM_DEVICE_ACTION_TURN_OFF instead?

If PM needs to happen at a device level (or requires coordination between its multiple IPs), it sounds like a good approach. Note that PM_DEVICE_ACTION_TURN_OFF is only used in the context of power domains, devices are either active or suspended. Naming isn't great, we should likely reconsider replacing "TURN_OFF" with something else...

As the power state relates to the entire device, not just the regulators, this will be handled by the new MFD driver instead.

I've updated this to just include the ship mode command.
Hibernate is rather specific to the nPM1300, but ship mode is common across many PMIC devices.


Allow using Date as a value

Both pg and mysql allow you to pass Date instances as values and do the correct transformation for you.
This was allowed in sql-template-tag as well until the Value type was made more strict in https://github.com/blakeembrey/sql-template-tag/commit/d1fb92ec5653a5680f0bf9abfbecd4518b56bef4.

Ah, thank you! This was my mistake, tried to make it stricter to avoid user error. Do you have any opinion on expanding types vs just making it unknown? I wasn't sure about the better approach here since technically anything could work but some things like a random class could be user error.

Ah, thank you! This was my mistake, tried to make it stricter to avoid user error. Do you have any opinion on expanding types vs just making it unknown? I wasn't sure about the better approach here since technically anything could work but some things like a random class could be user error.

I like that there's some type safety there, but trying to support every use case people will have without just making it any/unknown is probably futile, since anything that can be stringified should be fair game.


SWARM-1895: [Swagger] Align library & module dependencies.

Motivation
The Swagger fraction is pulling in an older version of the Guava dependency (18.0) while the Maven project lists the provided dependency to "20.0". This causes weird issues when using the Gradle plugin for building a "bundled" uber jar file as the dependency is never satisfied. Instead of downgrading the Guava dependency or messing around with the swarmtool / gradle plugin, it makes more sense to add an explicit dependency on the Guava module instead of keeping track / updating the dependency version for each release.
Additionally, at some point in time, if we have the ability to import POM properties of a BOM, we can reuse the library versions easily by referring to the appropriate property names. At a minimum, by aligning the property names, it will help in ensuring that we are copying over the same values from the appropriate bom.
Modifications

Refer to the Guava dependency module instead of managing the dependency on our own.
Align the Jackson & Javassist library versions with those present in WF parent.
Align the property name for library versions with those present in WF parent.

Result
Uber-bundled jars built using the Gradle plugin will no longer complain of missing Javassist, Guava & Jackson libraries.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you created a JIRA and used it in the commit message?
[x] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[x] Have you built the project locally prior to submission with mvn clean install?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

With these changes, did you verify that a Swagger app still works?
I know in the past there were issues in that Swagger relied on an older version of Jackson, which is why the versions were managed that way

Looking at the review comments. Will update the PR and notify.

Quick question. When compiling on OS X, I keep hitting the following error:
Failed pattern wfswarm\S+[0-9]{5,}.\S{5,} in directory .../roguexz/wildfly-swarm/testsuite
    /testsuite-cdi-ejb/target.../roguexz/wildfly-swarm/testsuite/testsuite-cdi-ejb/
    target/wfswarmCdiEjbTest4291612659053267952.jar size(128) too large. Max. is 0

When do such exceptions show up? Is this a problem with code not cleaning up after the test completes? Or is this an indication of an error?

It means there’s some lingering temp file not being cleaned up. This can happen if a build failed earlier. So mvn clean first.

Running the same command: mvn clean install.
If I navigate to the project and execute the same command again, it works. Strange! Looks like some kind of a timing related issue probably! Went through SWARM-1530 and looks like I am hitting it intermittently.

@bobmcwhirter Does Arquillian apply a timeout during teardown? I have been digging through the source and am failing to locate the right line of code that needs to be changed. Here is what happens,

There are multiple instances (usually 2) created for TempFileManager - one which is used before the server starts and one which is used inside the server.
The temp files which are managed by the TempFileManager instance that is created inside the deployment get cleaned up properly.
The temp files which are created outside (e.g., NestedJarResourceLoader, etc.) are not getting cleaned up properly because "something" is killing the thread pool.

Relevant execution reference: RuntimeServer
Depending on the system load, the following happens:
*** Files to be cleared: 9
Deleting file: .../wildfly-swarm/testsuite/testsuite-batch-jberet/target/wfswarmwildfly-swarm372552088680020708.d
Deleting file: .../wildfly-swarm/testsuite/testsuite-batch-jberet/target/wfswarmmodule-jar5302142567147266322.jar_d
Deleting file: .../wildfly-swarm/testsuite/testsuite-batch-jberet/target/wfswarmmodule-jar780217234412223336.jar_d
Deleting file: .../wildfly-swarm/testsuite/testsuite-batch-jberet/target/wfswarmmodule-jar4155687540380722092.jar_d
Deleting file: .../wildfly-swarm/testsuite/testsuite-batch-jberet/target/wfswarmBatchArquillianTest4465195220280811649.jar
Deleting file: .../wildfly-swarm/testsuite/testsuite-batch-jberet/target/wfswarmmodule-jar125643956863438708.jar_d
Deleting file: .../wildfly-swarm/testsuite/testsuite-batch-jberet/target/wfswarmswarm-config-6256933340879466625.xml

Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 10.675 sec - in org.wildfly.swarm.batch.jberet.BatchArquillianTest

As you can see, the code was supposed to delete 9 files, but it only managed to get to 7. The thread-pool was killed in between.
So the question I have is .. is there a timeout being applied to the initial thread-pool / CDI container that hosts the RuntimeServer? If so, who is shutting it down? Is there a way to specify a timeout for these test case scenarios?
If I were to resume the build from that project, it goes through fine till it crashes at some other place. This is quite annoying.
Environment Details
Apache Maven 3.5.2 (138edd61fd100ec658bfa2d307c43b76940a5d7d; 2017-10-18T00:58:13-07:00)
Maven home: /usr/local/Cellar/maven/3.5.2/libexec
Java version: 1.8.0_162, vendor: Oracle Corporation
Java home: /Library/Java/JavaVirtualMachines/jdk1.8.0_162.jdk/Contents/Home/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "10.13.3", arch: "x86_64", family: "mac"

I think I might have found a solution for that compilation failure issue. Testing it a couple of more times before raising a separate PR for it.

Raised PR #839 for fixing the local compilation problem.

Raised PR #840 to help with generating local builds easily.
@kenfinnigan Local test suite is working after these changes. I have confirmed that my Gradle based project is also starting up fine and I am able to view the swagger.json file as expected.


fix: Add library diffs in git modal

Description

Added Library modifications in git status modal
Removed segment from recommended libraries.
Added AWS-SDK in recommended libraries.
Removed the URL regex that checks to see if the URL ends with .js
Changed supabase URL

Fixes #19691
Fixes #19646
How Has This Been Tested?

Manual
Jest

Test Plan
https://github.com/appsmithorg/TestSmith/issues/2125
Checklist:
Dev activity

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

QA activity:

[ ] Test plan has been approved by relevant developers
[x] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=988559e

/ok-to-test sha=78ceba5

/ok-to-test sha=64010ce

/ok-to-test sha=9dc89d0

Gtg to release. All tests have been completed.
CC: @arunvjn

/ok-to-test sha=fef124e

/ok-to-test sha=310c461


Unable to parse metadata when '&' character is present in property attributes.

Bug reproduced in version: 1.6.0
Environment info:
Win 10
Python 3.7.1
pyodata-1.6.0
lxml-4.5.2
Steps to reproduce:
Simplest is to use the script to open local metadata file, e.g. https://github.com/SAP/python-pyodata/blob/master/docs/usage/initialization.rst#get-the-service-with-local-metadata
$metadata content to reproduce the bug
<edmx:Edmx xmlns:edmx="http://schemas.microsoft.com/ado/2007/06/edmx" xmlns:m="http://schemas.microsoft.com/ado/2007/08/dataservices/metadata" xmlns:sap="http://www.sap.com/Protocols/SAPData" Version="1.0">
<edmx:DataServices m:DataServiceVersion="2.0">
<Schema xmlns="http://schemas.microsoft.com/ado/2008/09/edm" Namespace="FAA_MD_MANAGE_SRV" xml:lang="en" sap:schema-version="1">
<EntityType Name="C_AssetTPType" sap:label="Asset" sap:content-version="1">
<Property Name="IN_AssetIsResearchAndDev" Type="Edm.String" sap:label="R & D Asset" sap:quickinfo="India: R & D Asset"/>
</EntityType>
</Schema>
</edmx:DataServices>
</edmx:Edmx>

Stacktrace:
 File "C:\Python37\lib\site-packages\pyodata\v2\model.py", line 2428
  File "src\lxml\etree.pyx", line 3519, in lxml.etree.parse
  File "src\lxml\parser.pxi", line 1856, in lxml.etree._parseDocument
  File "src\lxml\parser.pxi", line 1876, in lxml.etree._parseMemoryDocument
  File "src\lxml\parser.pxi", line 1764, in lxml.etree._parseDoc
  File "src\lxml\parser.pxi", line 1127, in lxml.etree._BaseParser._parseDoc
  File "src\lxml\parser.pxi", line 601, in lxml.etree._ParserContext._handleParseResultDoc
  File "src\lxml\parser.pxi", line 711, in lxml.etree._handleParseResult
  File "src\lxml\parser.pxi", line 640, in lxml.etree._raiseParseError
File "<string>", line 5
lxml.etree.XMLSyntaxError: xmlParseEntityRef: no name, line 5, column 75

Additional Notes:
Metadata are parsed correctly when you delete the '&' characters

The bug stems from LXML trying to parse an non-sanitized XML string. In this case, the '&' needs to be escaped to &   as it would otherwise be considered as the special character, and the parser would fail. https://www.novixys.com/blog/what-characters-need-to-be-escaped-in-xml-documents
This SO thread might be of help https://stackoverflow.com/questions/4972210/escape-unescaped-characters-in-xml-with-python

This bug is a false positive - the '&' was correctly escaped, but when bug was reported to me, I received non-escaped version of the XML file, which obviously reproduced the bug.
PR  #181 just adds a better exception text for such edge case  so the problem would be better understandable to someone who does not know LXML.


Convert uploaded HIV CSV to FHIR

Background
HIV data uploaded in CSV format must be converted to FHIR before it can be sent to the ReportStream Universal Pipeline so that it can be routed to LAC.
Action requested
Update gov.cdc.usds.simplereport.service.TestResultUploadService#processHIVResultCSV to convert input to FHIR and send to UP
You can use processResultCSV as a reference.
Should be able to change fhir converter code to conditionally include sections on gender and birth sex if they are in the row
Acceptance Criteria
A CSV of the correct schema uploaded via our support page link is converted to a FHIR bundle
log the serialized fhir bundle in processHIVResultCSV
A CSV of incorrect schema can just fail silently (no need for validation support at this time)
Exclude fields related to sexuality or gender of sexual partners at this time
Additional context
TestResultUploadService
BulkUploadResultsToFhir

Closing as won't do


FLW-1114: Fix localization issues

Add missing localization
Fix wrong keys
Add Russian language support for stories

Codecov Report

Merging #346 (e04afff) into develop (7920849) will increase coverage by 3.25%.
The diff coverage is 38.04%.

:exclamation: Current head e04afff differs from pull request most recent head c1bd654.


fix: typo python to node js Readme

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


backup cleanuptime cannot be changed when using @modificationlevel

I am doing DIFF backups unles the modification level is above a certain treshold. DIFF backups should only last for 24 hours, FULL backups for 2 weeks. However, I cannot specify a retentiontime for every backuptype. If I specify 24 hours and the backup is switching from DIFF to FULL, a lot of my FULL backups got deleted (the ones older than 24 hours)
I currently have the @cleanuptime set to 2 weeks, but as a result I got a lot of unnecessary DIFF backups.
In order to fix this I would like to change the @retentiontime to @retentionTimeLog, @retentiontimeDiff and @retentiontimeFull

I have been looking at this.
Currently the code looks like this:
      -- Delete old backup files, after backup
      IF ((@CurrentBackupOutput = 0 AND @Verify = 'N')
      OR (@CurrentBackupOutput = 0 AND @Verify = 'Y' AND NOT EXISTS (SELECT * FROM @CurrentBackupSet WHERE VerifyOutput <> 0 OR VerifyOutput IS NULL)))
      AND (@BackupSoftware <> 'DATA_DOMAIN_BOOST' OR @BackupSoftware IS NULL)
      AND @CurrentBackupType = @BackupType

As you see, cleanup will not be performed when the backup type is changed. This is to protect against scenarios, where you have different retentions for the different backup types.
I will consider introducing additional parameters as you suggest.