[WIP] primreact Knob component

@GPetrites, thanks for the project!
I intend to add the Knob component, here are the F# changes.
In my local working copy, I shuffled things a bit:

replacing the packaging setup (not familiar and was more issue for me to try locally with updated dependencies), using vite.config.mts
adding prop-types dependency
updating to recent fable packages

My local changes seems to work, both the samples page and the docs, but I'm not 100% confident my changes are the way you'd like to manage the repository moving forward.
Please let me know if you'd like to see those changes in this branch, or would prefer to nudge the branch yourself to better fit the environment you've setup, or if we should try another thing?
Thanks for reviewing and considering if adding this component can be done for future version of the library.
edit: I'm not done with the bindings, just setting up the thing. I'll keep the ball rolling with more coverage of the library and documentation, if this is suitable for future version of the library.

I'm glad you find this package useful. At the time I published, I hoped that I would be an active user as well as offering to the community. Unfortunately my group isn't doing much web development and haven't used this.
Would you be interested in taking ownership so that you can continue to evolve?

@GPetrites thanks for the feedback!
I'm ok to attempt to evolve it a bit, for now I just needed a knob, but I see the library is actually useful and I like to contribute to the fable and react bindings ecosystem to learn more about it.
Let me push this PR a bit forward and figure if I can setup the website from scratch in my fork, just for practice, over coming weeks, and then I'll come back to see how we can proceed.
I think the easiest would be to keep the repository as is, and figure out which rights would help me prepare new official release of the nuget and website.
I'll also try to get in touch with people contributing to Feliz ecosystem.


fix for pause in sms response

#68
this should fix this and still have pauses where we want them

there are 2 other changes to make

please look at all the comments https://github.com/radius314/yap/pull/69/files

sorry it got confusing, i should of done them in order, would of made a lot more sense

you can do all of them at once, they don't need to be individual commits.


mingw32 compile "undefined reference to `SDL_SetMainReady'"

https://github.com/diasurgical/devilutionX/issues/686#issuecomment-609113949
research showed that lSDL2main after the other libraries caused compilation errors
put SDLmain before the other libraries

looks like the build fails from windows in VS
it works fine for me in Linux using mingw32 compile method

wrong branch


[METADATA UPDATE][Merge by 2024-08-13] Global ms.service and ms.subservice metadata update

A Pull Request has been made from the Learn Platform's Metadata Management System. Please review and merge this Pull Request within 5 days. If you have any questions or concerns about the purpose of these metadata updates, please contact<EMAIL_ADDRESS>If you are not the correct contact for this content and repository, please notify<EMAIL_ADDRESS>If this Pull Request is not merged within 14 days, it will be automatically merged by our system to ensure the timeliness of the metadata update. This includes bypassing the Repository's Branch Policy, including if Review Required is enabled. Please notify<EMAIL_ADDRESS>if you have questions or concerns or would like Pull Requests for metadata updates to merge automatically in future.
Fixing#970824 Remove ms.service = advisor. Replaced by ms.service = azure-advisor
Who initiated this PR?
The Learn Platform and Product (LPP) Site Experience team manages the Learn platform’s reporting taxonomies that enable content analytics for OKR reporting and content portfolio management. Reporting taxonomy values populate metadata attributes.
We regularly review reporting taxonomies to ensure they are accurate, useful, and align to current business needs. Updates to reporting taxonomies and associated metadata make reports cleaner, easier to use, and more consistent.
Why are we updating ms.service/ms.subservice metadata?
Objective: Ensure ms.service attribute reliably reflects product affiliation.
Issues: 1. Currently, ms.service taxonomy values mix product and non-product concepts. Non-product concepts represent real reporting needs but aren’t strictly about products. 2. Slugs (machine readable identifiers) don’t reflect Microsoft branding guidelines. This is leading to inaccurate metadata on content.
Actions: 1. Revise taxonomy structure so ms.service represents the name of a branded Microsoft product, product family, or branding terminology. 2. Update slugs to comply with branding guidelines.
Outcome: 1. Establish a baseline for a comprehensive product data model representing all Microsoft offerings. 2. Ensure that report filters driven by ms.service will be based on product. 3. Improved metadata quality.
Why it matters

Efficiency in Content Auditing: Revising the taxonomy structure will enable more precise content audits, fostering repeatable business processes and alleviating the burden of large-scale, one-off audit projects.
Semantic Precision: The ms.service taxonomy will have meaningful, universally understood semantic values.
Consistent Metadata: This work contributes to the accuracy and uniformity of contextual metadata within the Learn Knowledge Base.
RAG Pipeline Efficiency: The Learn Knowledge Service’s RAG pipeline will benefit from increased accuracy of product attribution of content.

Frequently Asked Questions
Why does this Pull Request appear to be made by the Repository Owner? We open Pull Requests two different ways.

For Git Repos with a permissioned Service Account, we open the PR from our Document Build Service Account.
For Git Repos that we either do not have Service Account permission for or the repo is in Azure Repos (ADO)  we open the Pull Requests in an automated way with the PR creator as the Repo owner.

How can I revert a Pull Request that has been merged and created an unexpected issue? Whether a PR has been merged manually or automatically, you can revert it if an issue arises. See Reverting a pull request - GitHub Docs.
CommitMessage : Remove ms.service = advisor. Replaced by ms.service = azure-advisor

#sign-off


Bump github.com/gofiber/fiber/v2 from 2.49.0 to 2.49.1

Bumps github.com/gofiber/fiber/v2 from 2.49.0 to 2.49.1.

Release notes
Commits

updated-dependencies:

dependency-name: github.com/gofiber/fiber/v2 dependency-type: direct:production update-type: version-update:semver-patch ...

This PR has not been merged into update-dependencies. Please fix the issues and re-open the PR.

This PR has not been merged into update-dependencies. Please fix the issues and re-open the PR.

This PR has not been merged into update-dependencies. Please fix the issues and re-open the PR.

This PR has not been merged into update-dependencies. Please fix the issues and re-open the PR.

This PR has been merged into update-dependencies. Thanks for your contribution!


Fetch page lost prefix attribute

Issue summary
Fetch page lost prefix_options attributes
Expected behavior
I will get next/previous page data with prefix_options attributes
Actual behavior
What actually happens?
irb(main):032:0> discount_codes = shop.with_shop_session {ShopifyAPI::DiscountCode.all(params: {price_rule_id:<PHONE_NUMBER>96, limit: 2})}

=> [#<ShopifyAPI::DiscountCode:0x00007ff659bbe3e0 @attributes={"id"=>11100111208632, "code"=>"DOB-fc93035b", "usage_count"=>0, "created_at"=>"2021-03-17T01:22:10-07:00", "updated_at"=>"2021-03-17T01:22:10-07:00"}, @prefix_options={:price_rule_id=>919109107896}, @persisted=true>, #<ShopifyAPI::DiscountCode:0x00007ff659bc7558 @attributes={"id"=>11100111175864, "code"=>"DOB-3fb3b6be", "usage_count"=>0, "created_at"=>"2021-03-17T01:22:10-07:00", "updated_at"=>"2021-03-17T01:22:10-07:00"}, @prefix_options={:price_rule_id=>919109107896}, @persisted=true>]

irb(main):035:0> shop.with_shop_session {discount_codes.fetch_next_page}

=> [#<ShopifyAPI::DiscountCode:0x00007ff6572eb058 @attributes={"id"=>11100111143096, "code"=>"DOB-2d3e1f60", "usage_count"=>0, "created_at"=>"2021-03-17T01:22:10-07:00", "updated_at"=>"2021-03-17T01:22:10-07:00"}, @prefix_options={}, @persisted=true>, #<ShopifyAPI::DiscountCode:0x00007ff6572eac20 @attributes={"id"=>11100111110328, "code"=>"DOB-977640e7", "usage_count"=>0, "created_at"=>"2021-03-17T01:22:10-07:00", "updated_at"=>"2021-03-17T01:22:10-07:00"}, @prefix_options={}, @persisted=true>]

Steps to reproduce the problem

get resource normal with prefix params
fetch page

Logs

Specifications

shopify_api version: shopify_api (9.2.0)
activeresource version: activeresource (5.1.1)
Shopify API version used (example: 2020-07):  2020-07

Hey @iiewad, thanks for bringing this up. Pagination doesn't trigger the prefix_options setting (which in this case will be price_rule_id). Does the issue stem from the fact that you're trying to access price_rule_id?

@mllemango Yes,  I need get the price_rule_id when I fetch the page. Maybe I can get the price_rule_id with other way, but I think the prefix_options setting is necessary!
I'm sorry about my english~

I would suggest getting price_rule_id from the DiscountCode class, you can use discount_code.price_rule_id instead.
Below is a snippet of code I used
discounts = shopify.DiscountCode.find(price_rule_id=PRICE_RULE_ID, limit=2)
    for discount in discounts:
        print(discount.price_rule_id)

@mllemango Ok, Thanks for your answer!


Initial glyph loading/scaling crate

This adds a new crate called punchcut containing the initial TrueType glyph scaling code based on read-fonts. The caching and hinting code has been extracted to hopefully make this more reviewable.
A few notes:

There are fixed point math functions that have some overlap with code in font-types. FreeType provides outline points as either integral font units or fixed point (26.6) values depending on whether a scale has been applied. We copy that here and that makes some of the math and types less than ideal. I'm open to options for making this nicer (without loss in perf/memory) but I'd like to not block the initial cut on it.
There is a tiered architecture that exposes the individual sources (just glyf for now) as well as a higher level interface at the root of the crate. The layering feels pretty good to me right now, but comments for improvement are welcome.
I've tried to comment some of the bits in source/glyf/scaler.rs that might be tricky. Feel free to point out any that require further explanation.

This has nice integration with kurbo: the Outline type implements kurbo::Shape so these can be fed directly into vello (née piet-gpu). I plan on using this for glyph loading support in that crate soon™.

So, this crate is for loading glyphs from various OT formats and turn them into kurbo datatypes, potentially applying hinting?

Yes, that’s the intent. There are working hinters for both TrueType and CFF but I left the former out to keep the PR size more manageable and the latter is waiting on CFF parsing in fontations.

Interesting. Did you port FreeType's TTF hinting engine?

Interesting. Did you port FreeType's TTF hinting engine?

Yes, I ported it a while back for the swash crate.

Thanks for the review.
I'd rather address (at least the bulk of) the issues here and ping for follow ups rather than merging and letting them linger.

Addressed most of the low-hanging fruit, plus removed the peniko/kurbo dependencies and feature gated all of the hinting code. I'll look at the rest tomorrow (today?)

Opened issue #203 to discuss better testing wrt comparison with FreeType.

Testing is very light, perhaps it could become data driven with instructions on how to regenerate or extend the data?
Commenting is very light; imagine you want a competent coder who isn't a font expert to be able to have a clue
Everything related to hinting needs to be either feature gated or deleted

Status now:

Partially done in that new testing infra is in place including automatic extraction of the outline data. Test cases are still sparse and I've decided to defer that with #204 opened to track.
Still pending. I'll spend some time on more complete documentation in the next few days.
Done

We've agreed that now is a good time to merge this. Thanks to everyone for both detailed reviews and patience. The code has been substantially improved as a result.
There are some minor remaining adjustments needed which I've written up in #209 to track.


Doc mention of a module

Here's a module that I crafted over the weekend being particularly inspired by this StackOverflow discussion.
I suppose it's useful enough to be mentioned within the Cython docs themselves.

Nice. A couple of comments:

The name resembles https://github.com/matthew-brett/multibuild but does something entirely different. Maybe something more in the lines of "single module", "liblib", "extext" or (why not) something polish could become a better name? Name it "matrioszka" or "box of bricks" (bob) if you feel like it. You may have heard of Bob the Builder, that kind of fits.
Before we link to it from the docs, I'd like to see it established on PyPI and have at least some potential users give it a try and report on it. Could you maybe present it on the cython-users mailing list to get feedback?
A module that monkey patches a foreign module on import seems too much of a side-effect. Why not provide a new (but separate) interface that can then give an additional hand to users for this specific, more involved task of generating code and merging all things together in a single distutils/setuptools build? It doesn't feel like cythonize() is really the best interface for that.

Ok, thank you for the tips!
I've renamed the module to snakehouse, and got rid of the monkey-patching. I've also advertised my module on cython-users group.


Feature: Font Colors

Selecting a theme is a great feature, but some of the default theme colors are not ideal.  For example in the dark themes, error messages are red on gray and are very difficult to read, especially at the default (small) font size.
It would be great to be able to customize individual component font colors that are part of a theme.

I think I've found a theme that's somewhat close to what I want, but I really want to tweak the background.  I would love to see the ability to select the font colors I want, plus pick a very specific background color to have a smooth yet satisfying contrast.
If one were to change the hard coded themes, where is this located in the code?

Hi @rafaelthca ,
Please take this one into account when performing changes in web interface.
We probably need to document how user can change font settings, even if it requires tweaking some files.
Best regards,
William


Google Recaptcha failed to load error when trying to log in

Describe the bug
After the Chromium browser loads, try to login. At a certain point, the browser closes and an error is logged in the console about failing to load a Google Recaptcha page.
To Reproduce
Steps to reproduce the behavior:

Use the appropriate $ims config settings
aio ims:login
Enter your email address, press Enter
Choose "Personal Account"
At this point the browser closes, and you see the error in the console log

Expected behavior
Successful login.
Desktop (please complete the following information):

OS: macOS 10.15
Browser [e.g. chrome, safari] Chromium
Version [e.g. 22] whatever was specified at sha https://github.com/adobe/aio-lib-core-ims-oauth/commit/2fe758d3547d282075cd308b5a9f8b6377ab18c9

Additional context

Log
 @adobe/aio-lib-core-ims-oauth/electron   > ERR: {"message":"Failed to load https://www.google.com/recaptcha/api2/anchor?ar=1&k=6LeAzB4UAAAAAEQiceuQniNPhGMDXnt7ArGOyxHp&co=aHR0cHM6Ly9hdXRoLnNlcnZpY2VzLmFkb2JlLmNvbTo0NDM.&hl=en&v=75nbHAdFrusJCwoMVGTXoHoM&size=invisible&badge=inline&cb=8j7anhqs3e76\nReason:  (-3)"} +15s
  @adobe/aio-lib-core-ims-oauth/ims-oauth electronCallback(Error: Failed to load https://www.google.com/recaptcha/api2/anchor?ar=1&k=6LeAzB4UAAAAAEQiceuQniNPhGMDXnt7ArGOyxHp&co=aHR0cHM6Ly9hdXRoLnNlcnZpY2VzLmFkb2JlLmNvbTo0NDM.&hl=en&v=75nbHAdFrusJCwoMVGTXoHoM&size=invisible&badge=inline&cb=8j7anhqs3e76 Reason:  (-3) at childProcess.execFile (/Users/shazron/Documents/git/adobe/aio-lib-core-ims-oauth/src/electron.js:40:34) at ChildProcess.exithandler (child_process.js:301:5) at ChildProcess.emit (events.js:198:13) at maybeClose (internal/child_process.js:982:16) at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5), undefined) +28ms
  @adobe/aio-lib-core-ims-oauth/ims-oauth   > Error: Failed to load https://www.google.com/recaptcha/api2/anchor?ar=1&k=6LeAzB4UAAAAAEQiceuQniNPhGMDXnt7ArGOyxHp&co=aHR0cHM6Ly9hdXRoLnNlcnZpY2VzLmFkb2JlLmNvbTo0NDM.&hl=en&v=75nbHAdFrusJCwoMVGTXoHoM&size=invisible&badge=inline&cb=8j7anhqs3e76 Reason:  (-3) at childProcess.execFile (/Users/shazron/Documents/git/adobe/aio-lib-core-ims-oauth/src/electron.js:40:34) at ChildProcess.exithandler (child_process.js:301:5) at ChildProcess.emit (events.js:198:13) at maybeClose (internal/child_process.js:982:16) at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5) +1ms
  @adobe/aio-lib-core-ims-oauth/ims-oauth checkWebResult: Rejecting on error: Error: Failed to load https://www.google.com/recaptcha/api2/anchor?ar=1&k=6LeAzB4UAAAAAEQiceuQniNPhGMDXnt7ArGOyxHp&co=aHR0cHM6Ly9hdXRoLnNlcnZpY2VzLmFkb2JlLmNvbTo0NDM.&hl=en&v=75nbHAdFrusJCwoMVGTXoHoM&size=invisible&badge=inline&cb=8j7anhqs3e76 Reason:  (-3) at childProcess.execFile (/Users/shazron/Documents/git/adobe/aio-lib-core-ims-oauth/src/electron.js:40:34) at ChildProcess.exithandler (child_process.js:301:5) at ChildProcess.emit (events.js:198:13) at maybeClose (internal/child_process.js:982:16) at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5) +74ms
  aio:ims:login Login Failure: Failed to load https://www.google.com/recaptcha/api2/anchor?ar=1&k=6LeAzB4UAAAAAEQiceuQniNPhGMDXnt7ArGOyxHp&co=aHR0cHM6Ly9hdXRoLnNlcnZpY2VzLmFkb2JlLmNvbTo0NDM.&hl=en&v=75nbHAdFrusJCwoMVGTXoHoM&size=invisible&badge=inline&cb=8j7anhqs3e76
  aio:ims:login Reason:  (-3)
  aio:ims:login Error: Failed to load https://www.google.com/recaptcha/api2/anchor?ar=1&k=6LeAzB4UAAAAAEQiceuQniNPhGMDXnt7ArGOyxHp&co=aHR0cHM6Ly9hdXRoLnNlcnZpY2VzLmFkb2JlLmNvbTo0NDM.&hl=en&v=75nbHAdFrusJCwoMVGTXoHoM&size=invisible&badge=inline&cb=8j7anhqs3e76
  aio:ims:login Reason:  (-3)
  aio:ims:login     at childProcess.execFile (/Users/shazron/Documents/git/adobe/aio-lib-core-ims-oauth/src/electron.js:40:34)
  aio:ims:login     at ChildProcess.exithandler (child_process.js:301:5)
  aio:ims:login     at ChildProcess.emit (events.js:198:13)
  aio:ims:login     at maybeClose (internal/child_process.js:982:16)
  aio:ims:login     at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5) +16s
  @adobe/aio-lib-core-ims/context get current +15s
Error: Cannot get token for context 'oauth2_cli': Failed to load https://www.google.com/recaptcha/api2/anchor?ar=1&k=6LeAzB4UAAAAAEQiceuQniNPhGMDXnt7ArGOyxHp&co=aHR0cHM6Ly9hdXRoLnNlcnZpY2VzLmFkb2JlLmNvbTo0NDM.&hl=en&v=75nbHAdFrusJCwoMVGTXoHoM&size=invisible&badge=inline&cb=8j7anhqs3e76
Reason:  (-3)
    at Object.error (/Users/shazron/Documents/git/adobe/aio-cli-plugin-ims/node_modules/@oclif/errors/lib/index.js:22:17)
    at LoginCommand.error (/Users/shazron/Documents/git/adobe/aio-cli-plugin-ims/node_modules/@oclif/command/lib/command.js:57:23)
    at LoginCommand.run (/Users/shazron/Documents/git/adobe/aio-cli-plugin-ims/src/commands/ims/login.js:44:18)

Looks like there is a load for some invisible (?) re-captcha from Google which for some reason cannot be loaded which triggers a false positive in the did-fail-load handler.
Trying to fix this to only handle did-fail-load if the initial authUrl fails to load.

Pushed commit 0b106a0. Feel free to test again, @shazron. Thanks.

According to feedback, the issue seems to be solved. Therefore closing.


Fixed Build.fsx casing to work on case sensitive environments

Systems that have case sensitive file names (i.e. Linux) won't find build.fsx. Just changing the case so all the :penguin: will be happy again :wink:

 You should sign our Contributor License Agreement in order to get your pull request merged.

Thank you :+1:


Add new INCL_MODE with centers for input

Closes https://github.com/dionysos-dev/Dionysos.jl/issues/187

Codecov Report

Merging #190 (d25a92f) into master (efa0265) will decrease coverage by 0.47%.
The diff coverage is 14.28%.

: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             @@
##           master     #190      +/-   ##
==========================================
- Coverage   51.32%   50.86%   -0.47%
==========================================
  Files          36       36
  Lines        3400     3421      +21
==========================================
- Hits         1745     1740       -5
- Misses       1655     1681      +26

Impacted Files
Coverage Δ

src/problem/optim/abstraction.jl
0.00% <0.00%> (ø)

src/domain/grid.jl
34.25% <16.66%> (-1.04%)
:arrow_down:

src/control/controller.jl
89.88% <0.00%> (-2.25%)
:arrow_down:

src/domain/nested_domain.jl
58.51% <0.00%> (-1.07%)
:arrow_down:

src/problem/optim/q_learning.jl
92.78% <0.00%> (-0.97%)
:arrow_down:

src/problem/optim/bemporad_morari.jl
91.38% <0.00%> (-0.48%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Invoke-SqlCmd2: "Cannot find table 0" on running an INSERT

Hi!
I'm being thrown into the SQL world, and using Invoke-SqlCmd2 as my primary tool. Thanks for such a great function!
I've noticed that when running an INSERT command, the insert succeeds, but I get a "Cannot find table 0" error in PowerShell:
Cannot find table 0.
At C:\Users\replica\Documents\WindowsPowerShell\Modules\MyModule\Internal\Invoke-SqlCmd2.ps1:522 char:21
+                     $ds.Tables[0]
+                     ~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (:) [], IndexOutOfRangeException
    + FullyQualifiedErrorId : System.IndexOutOfRangeException

I found an answer on Stack Overflow that describes a similar issue - since an INSERT naturally doesn't produce any output, the function is trying to reference row 0 in an empty collection.
My proposed solution is to wrap the switch block (lines 510-537) in an if:
if ($ds.Tables.Count -gt 0)
{
    switch ($As)
    {
        # ...
    }
}
else
{
    Write-Verbose "No output was returned from the SQL instance."
}

I thought about just wrapping specific cases in the switch block, but there are two that use that index behavior ($ds.Tables[0]), and I'm not certain it's a big deal if the function doesn't provide output in this case, rather than providing a DataTable or DataSet object that contains no tables or rows. I could definitely be wrong, though, as I'm still wrapping my mind around SQL stuff.
I've tested this change and confirmed that it seems to be working for me - the INSERT succeeds, there's a verbose message and no error, and there's no output. I can't submit a PR from here, but I should be able to do so later if you'd like. :)
Thanks!
~replica

Oh!  Somehow missed this.  Makes sense to me!  Only concern would be, is it possible to get back a dataset that happens to have no datatables (if that's even a thing)?

Hi again!
Still have some things to sort out, but moved Invoke-SqlCmd2 to the SQLCollaborative org - Would you mind re-opening the issue there?
Cheers!

Cool, that works for me. :) Would you like me to re-submit the PR there as well?

Oh, and I realize I never replied to your prior comment, so I apologize!
I don't know of a case when a dataset would be returned with no tables, but I don't know these .NET classes well enough to know for certain that I'm not wrong. I'll try to poke around when I get a chance to see if there's any other data returned in that object on an INSERT or an UPDATE.

Hi!
Closing this out as it's over in the sqlcollaborative org.  On a side note, moved the function to dbatools.  Might get more attention there given that more SQL folks watch that repo (I know, I'm sorry, that's the second time it's moved recently!)
Cheers!


stream_settings: Fix stream settings dropdown of user profile.

Previously, when the user modal is opened, and a stream is selected from the streams tab, and then, when a new tab is selected and you navigate back to this streams tab and try to pick a new stream, it wouldn't allow you to do so.
This is because, previously once a stream is picked, and you go out of the tab and come back, then the
previous object of the subscribe widget was invalidated and a new object was created. However, the options still corresponded to the old object. Hence, the error.
This is fixed by checking out if a stream widget object exists, before creating a new one. Before the user profile modal is rendered, the object is initialized to undefined to avoid using of old object in new modal.
Fixes: #27422

Fixes:

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

I just divided into 2 commits and force pushed the same.... I'll try to have a look at this thing once more by tomorrow evening to make sure this is the right approach.

Merged, thanks @roanster007!


Export dialog

As discussed we need to change a few things:

The name of the saved document should be the value of /document/tracking/id
The PDF Export has empty pages
The local PDF export should not be named after the URL. Not sure if this is possible since we are using the print dialog.


Ensure hosts are talking to each other.

Signed-off-by: Nathaniel Clark<EMAIL_ADDRESS>

Codecov Report

Merging #933 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #933   +/-   ##
=======================================
  Coverage   95.42%   95.42%
=======================================
  Files           2        2
  Lines         153      153
=======================================
  Hits          146      146
  Misses          7        7

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 3f5e3f4...d736b21. Read the comment docs.


how to change skipFetched default to true?

Sorry about spamming weird questions, but we're using Objection in a way where we basically always want to ignore an already fetched relation. Now we could go ahead and litter our codebase with {skipFetched: true}, but I was wondering-could we override it globally? Other than extending the objection model and aliasing methods that take this config on our own model?

You should be able to set it in Model.defaultGraphOptions object.


len(advancedCollectible) -1 out of range

Hi,
I was trying to run the create_metadata.py, but got the error saying the len(Advanced_Collectible)-1 is out of range. More details of the error is shown as below.
I tried to pin the file to Pinata, which was successful, but not sure why Advanced_Collectible doesn't contain anything.
Any idea of the reason?
Brownie v1.17.2 - Python development framework for Ethereum

NftMixProject is the active project.

Launching 'ganache-cli --accounts 10 --hardfork istanbul --gasLimit 12000000 --mnemonic brownie --port 8545'...

Running 'scripts/advanced_collectible/create_metadata.py::main'...
Working on development
  File "brownie/_cli/run.py", line 50, in main
    return_value, frame = run(
  File "brownie/project/scripts.py", line 103, in run
    return_value = f_locals[method_name](*args, **kwargs)
  File "./scripts/advanced_collectible/create_metadata.py", line 25, in main
    advanced_collectible = AdvancedCollectible[len(AdvancedCollectible) - 1]
  File "brownie/network/contract.py", line 173, in __getitem__
    return self._contracts[i]
IndexError: list index out of range
Terminating local RPC client...

I actually found this line from create_metadata.py
advanced_collectible = AdvancedCollectible[len(Advanced_Collectible)-1]
is actually written as
advanced_collectible = AdvancedCollectible[-1] in the lecture.
Are there any difference? I tried both, but still have the same issue.

I actually figured it out. It works now by just following what you did in the lecture. So how can I help to revise the code in the repo?

Awesome!  Thanks for making these issues!
I think maybe it would be wise to maybe add some print statements throughout the repo to help people who get stuck...
Or are you saying the code you're using is different than the code in this repo?

@PatrickAlphaC The code I am using is different from this repo..

Ah... ok! Could you make a PR showing what you changed to help fix this one?
Interesting that working with https://ipfs.io/ipfs is working over ipfs://.... Might be something with your local ipfs node or something...


Add Norwegian - Bokmaal (nb) Language Support

Description
Thanks to Google's Gemini Advanced, we can translate existing English to any other languages. Let's support this.
Expected Behavior
Language is available
Current Behavior
Language is not available.
Associated Data Files
No response

base layer established (ref: cbe47f51b40c8b1206953f9b313c671635667a16).

base layer established (ref: cbe47f51b40c8b1206953f9b313c671635667a16).

push out and prioritize features & bug fixing instead.


Tag Type Removed Notification

What is the feature you'd like to have?
A notification for when a tag type is removed.

Related to #2886

Looks like this was added around 3.5


Metro bundler crashes when checking the NodeJS version (NodeJS <= 8.2.1)

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
[skip envinfo]
Any environment where nodeJS version is less than 8.2.1.
Description
The script local-cli/server/checkNodeVersion.jscrashes with a SyntaxError when run with node versions older than 8.2.1 due to the backwards-incompatible commas in function argument list.
The expected behaviour is that the script shows the banner and prompts the developer to upgrade their NodeJS runtime. That is, the script itself has to be written in ES5.
I suspect you format your code programatically. It would be perhaps wise to adjust your style guide so that trailing commas in function argument lists are disallowed. See https://github.com/prettier/prettier/issues/2788.

The expected behaviour is that the script shows the banner and prompts the developer to upgrade their NodeJS runtime

Can you submit a PR for that? Would be really helpful.
Aside from that, please fill the Env Info section.


Video Title for concatenateMediaSource

Hello,
How can i set the video title on runtime and update it?
I'm trying to play playlist and change the title according to the video being played.
Regards

Not supported in current version.

Thanks i have implement it in a different way!


Request Cancellation

This driver does not honor the cancellation of a context and will leave a query running forever.
It looks like there is a cancellation message that can be sent if you open another connection (judging by the Python driver) which will close out a long running query. All operations seem to pass through QueryContextRaw so monitoring the context there should be sufficient.
A question I have about implementation - what does sending a cancellation do if the query is already sending rows? Will Vertica send an error message? Complete?

I've asked this of @tomwall and he gave me some great answers. I'll ask him to cover this as well. It's been in my backlog for quite a while now.

Is there an IRC or some other communication channel contributors can use to discuss changes like this? A chat/email chain/gitter etc might be a better fit for a prolonged discussion than a GitHub issue. Then we could just post a summary back to this.

Interesting idea. Let me reach out to some people internally to see if there's some "process" for that. I might go ahead and just create a Discord server if that would work. @tomwall @sitingren ideas?

Discord works for me.

I'd be interested in knowin more about this too

Some kind persistent chat would be nice but we don't have a lot of time to go start one within the next couple of weeks, there's a lot going on behind the scenes to prepare for the now-virtual BDC at the end of the month. In the meantime we are all reachable via<EMAIL_ADDRESS>-- it's listed on the main vertica github page but could probably be more obvious in the individual projects.
Siting added proper cancel support to vertica-python not too long ago; the key implementation details can be seen there: https://github.com/vertica/vertica-python/commit/b08be5cca20af9ec90988fd86615d3006119cb70
The cancel request happens out of band on a separate connection. When vertica receives a cancel request it will cancel the current query, if it is still running.
Issuing a cancel increases the chances that you'll be done receiving rows soon, but it does not guarantee that the command stops immediately. Usually you will see rows continue to go by after the cancel is issued for some period of time, but it should stop eventually. Many factors contribute to this. It is up to the network to get the cancel request to vertica, and it is up to each operator in the currently executing plan to periodically check for cancels to notice that it should stop. Only after they all do can the plan be torn down and the error sent to the client.
From the main connection's perspective, a cancel during a query looks like any other query failure -- you'll see an ErrorResponse instead of more rows, which rolls back the current operation and should usually lead to an error/exception returned by the API. It looks no different than a dbadmin doing interrupt_statement the command rolling back for semantic reasons.

Yeah I saw the Python implementation and have been basing my Go implementation on that. I would send the cancel message but wasn't seeing my running query error out. I might need to use a longer running query to test with though from the sound of it.

Thanks for the PR! It looks like it is on the right track, but I'm not super familiar with go as a language so I could be missing something.
Cancel testing can be tricky because it is fundamentally asynchronous. It is a bit easier with an instrumented debug build of Vertica, but we don't have that luxury here.
The Vertica execution engine will pause in response to TCP backpressure, i.e. if the client is too slow to read data, Vertica will eventually stop sending data. If the query pauses before all the data can be sent to the client due to backpressure, you can artificially widen the window where a cancel will be processed. You can probably use that to your advantage to more reliably make a cancel occur; it may require some instrumentation in the client to slow/stop the reading of result data depending on how it is implemented.
An easy way to generate lots of result data quickly for testing purposes is a cross join of a few system tables. If you own your Vertica instance in a dev environment that's fine to do but your DBA might not appreciate lots of test queries doing that on a real system.
You can also compare the behavior between your cancel code and a dbadmin interrupt_statement() from a different session; once Vertica decides to do a cancel (via protocol here, or via dbadmin calling the function), Vertica's behavior is basically the same.

Fortunately I have a Vertica database available with a fair amount of data in it. I've got a few queries that run ~20-30 seconds. I set a context to expire after 3 seconds.
Up until now I thought my changes just weren't working because I wouldn't get an error back fro mthe driver but then I noticed my result sets were truncated (17 rows instead of a few hundred for example).
 PortalSuspended also seems to be how a successful query that doesn't get cancelled ends. I could indicate an error from the driver side by returning the error from the context which would at least let consumers know it didn't complete normally.

Vertica itself also seems to run the query to completion and only interrupts after rows start being sent. If I have a query that returns 1 row but takes 20 seconds to execute, and I send the cancel at 3 seconds it will still take 20 seconds to complete and I seem to always get that one row.
If I have a query that take about 5 seconds and returns ~200 rows and I cancel it at 3 seconds I'll get about 5-20 of those rows after 5 seconds.

The log is a little difficult to interpret because there's multiple connections going. There's many more protocol messages than I would expect (aside: we should add a thread/connection id to each log message to make it easier to interpret).
Looking closer, I think reusing newConnection may be complicating things here. I should have noticed that earlier, apologies. I looked closer at it and it is doing more than necessary to cancel -- that does everything needed to establish a real vertica session, which is overkill and may lead to strange behavior.
You don't need to do all that for cancel. Cancel uses the client key & pid established at the main connection's startup as temporary credentials to authenticate the cancel connection. The idea is to open a socket, send the secret derived from the already-authenticated session, and then immediately disconnect. You probably want to recycle just the "establish a socket" parts of newConnection and not the rest. You can see that in action in the python version: https://github.com/vertica/vertica-python/blob/master/vertica_python/vertica/connection.py#L374
Vertica doesn't fully support portals the way postgres does; all rows are sent back in response to an Execute message. That means PortalSuspended in the extended protocol is effectively the same thing as CommandComplete in the simple protocol. If the main connection sees either, that means the query is done you've missed your opportunity for the cancel to be processed. It is not a sign of an error that should bubble up to the application.
That could either be because vertica didn't receive the cancel request at all, or because it put all the rows on the network before it could notice & react to it. I can't tell given the uncertainty above if what you observe was expected or not, but hopefully that can get you closer to some clarity.

That means PortalSuspended in the extended protocol is effectively the same thing as CommandComplete in the simple protocol.

That much I had assumed, I was seeing it for queries I didn't cancel as well which is why it seemed like I wasn't getting an error after I cancelled. The fact that the rows got cut off made me think I was cancelling but Vertica was doing a normal looking close in response.
I used the Python implementation as a reference, that's where I got the layout of the struct from and knew to open another connection. I read that establish_socket_connection bit as doing a full connection but if it's just "open TCP socket, send the bytes of the struct over it" that's even easier and I'll change it to do so.
I'm definitely seeing the cancel message get sent before I receive the portal close. It's more obvious on the 20 second requests, I should have posted one of those logs instead. Maybe it's as you say and establishing the full connection is causing weirdness. I'll adjust the implementation and see what happens. Thank you for following up.

@tomwall that was the key! Just opening a socket and sending the message over that instead of a full session gives me an error back from Vertica immediately.
It looks like I'll need to rebase my changes off of what @huebnerr has in PR right now, the error causes a hang in the driver and I think his changes fix that. I'll verify it later today though.

Just wanted to say I'm also hitting all these issues with request cancellation and hanging on errors and thanks to all of you for working on this.

@fbernier I just force pushed my branch with it rebased off of huebnerr's changes. If you want to try it out it should be in a working state. I've been testing it against my own application and it seems to be working sensibly.


:sloth: Additional names for "vmware-harbor-registry"

:grey_question: About
Harbor VMWare Registry (aka. Harbor) has been added to eol:

https://github.com/endoflife-date/endoflife.date/issues/4230

But when I tried to get its eols from its most commonly used name harbor, I got this:

Also, in littertaure it is better known as Harbor:

:dart: Goal of the PR
:point_right: By merging this PR, it will be easier to get harbor eols wile using its most commonly used name.

Hey @adriens, just to make sure that it's clear for everyone, I am quoting one of my comment in the PR that added the product :

Actually, after some thinking, I really doubt that this link is related to the Open source version of the Software. Seems like there is "OpenSource Harbor" (Apache license) and VMWare Harbor container registry (VMWare license) based on the open source version.

The core question is: "Do we want to add the open source version also as an own product on our site?"
The open source version has a bit of other life cycle policy: https://github.com/goharbor/harbor/blob/main/RELEASES.md

The Harbor project maintains release branches for the three most recent minor releases, each minor release will be maintained for approximately 9 months.

Is see two options:

We add Open Source Harbor as a an own product -> conflicting urls, potential workaround with tag harbor
We update the current product to open source and specify VMWare as kind of extendedSupport -> can use harbor url

The core question is: "Do we want to add the open source version also as an own product on our site?"
The open source version has a bit of other life cycle policy: https://github.com/goharbor/harbor/blob/main/RELEASES.md

The Harbor project maintains release branches for the three most recent minor releases, each minor release will be maintained for approximately 9 months.

Is see two options:

We add Open Source Harbor as a an own product -> conflicting urls, potential workaround with tag harbor
We update the current product to open source and specify VMWare as kind of extendedSupport -> can use harbor url

I wasn't aware of this RELEASES page, really good find !
I think we should have 2 different pages as the policies are slightly different, seems like a similar situation as https://endoflife.date/google-kubernetes-engine and https://endoflife.date/kubernetes.
We will just have to be sure that people will not confuse the 2 products (one open source, the other one is commercial).

I think we should have 2 different pages as the policies are slightly different

Agree, thanks for your inputs @S0obi and @BiNZGi.
Closing this PR, feel free to open a new issue or PR for Harbor (the open source one).


Delay opening tab navigation for a second or so

As somebody who control-tabs all over the place, having the tab bar become huge whenever I switch tabs is really distracting. Maybe make it so that it waits a second or so before opening the navigation after pressing control-tab?
(On the topic - it might also be nice if hovering over the tab bar for a moment would open up the tab navigation, or perhaps having the navigation open when the control key is pressed and the mouse is hovered over the tab bar.. just spitting out ideas here -- I suppose this would better fit in a separate issue?)

The expanded tab mode (which is what is opening when you press control+tab) is about to be removed and replaced by something else in #103. The replacement won't open unless you press shift+command+e or click a button, which should fix this issue.

(On the topic - it might also be nice if hovering over the tab bar for a moment would open up the tab navigation, or perhaps having the navigation open when the control key is pressed and the mouse is hovered over the tab bar.. just spitting out ideas here -- I suppose this would better fit in a separate issue?)

Can you go ahead and file that in a separate issue?

This should be fixed in 1.2.0.


Remove perspective from depth data

Required Info

Camera Model
D415

Operating System & Version
Win 10

SDK Version
2.38.1

Language
C++

Issue Description
Is there a way to remove perspective from the collected depth data?
Let me explain on a depth map, where camera was mounted on a ceiling (=parallel to the floor) and looking down at people:

If the person is standing right below the camera (as in white rectangle), the camera is mostly only seeing his head (blue=high points). But in the corners of the camera's FOV, we can see the whole body, and that's my problem.
I understand that this is simply due to the fact how the depth camera works, as the IR rays are projected from one point, and I'm guessing no one tries to remove that, because it would mean loosing data. I'm working on a multicamera setup though, and when every camera in the setup has their own perspective, I cannot fuse pointclouds of the cameras together. Is there any function/combination of functions in librealsense that would allow me to calculate a ortographic/multiview/plan projection for the frame? If it's not in the lib, what should search for?
I have already looked at wiki (Projection in RealSense SDK 2.0) , #7279, https://github.com/IntelRealSense/librealsense/issues/5225 but it didn't help me.
Thank you!

Hi @mrehacek   If your aim is to "stitch" together into a single point-cloud the individual point clouds of multiple cameras with different viewpoints then this can be done.  A common method for doing this is to perform an affine transform.
https://community.intel.com/t5/Items-with-no-label/Combining-multiple-depth-streams-into-one/m-p/483016/highlight/true#M5089
An example of an affine transform instruction in librealsense is rs2_transform_point_to_point
https://github.com/IntelRealSense/librealsense/issues/5583
Alternatively, point clouds from different perspectives can be stitched with ROS:
https://www.intelrealsense.com/how-to-multiple-camera-setup-with-ros/
At a more advanced level, the scenario of stitching point clouds from up to twenty 400 Series cameras has been demonstrated by the CONIX Research Center at Carnegie Mellon.
https://github.com/conix-center/pointcloud_stitching

@mrehacek ,if you're trying to perform "head count" then you can probably mitigate the issues you're dealing with by limiting the max visible distance. This way you'll filter out all the depth data for objects at heights below (TBD) cm from the floor.
The SDK provides a dedicated post-processing threshold filter to crop the viewing frustum according to user-defined min and max ranges.
You can try it in the realsense-viewer application:

Hi @mrehacek  Do you require further assistance with this case, please?    Thanks!

Hi @mrehacek  Do you require further assistance with this case, please?    Thanks!

Hi @MartyG-RealSense and @ev-mp, thank you for your assistance and sorry for late reply - I needed to do some tests. I wish you all the best in the new year :)
@MartyG-RealSense I'm doing affine transforms using Eigen, and visualizing using PCL. I have looked at the Conix center project you mentioned, they do not seem to be doing more then hardcoded affine transforms, the same as me.
@ev-mp I'm doing head tracking, yes. I cannot use thresholding like that, it would ignore children, maybe adaptive thresholding would work. But this is not related.
I think I still didn't present the problem clearly enough, so I'm going to support it with more pictures.
Here is a visualization from Viewer, the same setup as in first post - camera on the ceiling facing floor:

I have drawn lines corresponding to room geometry. By drawing rs2::frame I believe that data from it are projected on the plane with a perspective distortion.
However, when doing the stitching of point clouds generated using rs2::pointcloud, I believe that rs2::pointcloud is actually doing (non-affine) transformation to remove the perspective from data obtained from rs2::frame. Is this the case or am I wrong?
I think that I was further confused by using spatial filter. My hypothesis is, that the data generated by rs2::pointcloud do not contain perspective, and can therefore be stitched with only affine transform. But there is noise which spits some of the points (those further from the camera, around the contour of the person perphaps) in a way that creates a sort of ellipsoid, see left part of the image:

On the left, one can clearly see people in the noise. However, on the right, after using spatial filter, the noisy parts get smoother and solidified. Then the blob is stretched in the direction from the camera, as a shadow of the person, which for me, created an illusion of a perspective. But now I see there is no perspective, it's just a problem created by using spatial filter.
So.
The problem with the filtering makes the stitched point cloud less useful to me now, then just doing my calculations on a 2D image from rs2::frame. But if the rs2::frame only has data with perspective distortion, it is a complication. Is there a way of getting rid of the perspective?

Hi @MartyG-RealSense and @ev-mp, thank you for your assistance and sorry for late reply - I needed to do some tests. I wish you all the best in the new year :)
@MartyG-RealSense I'm doing affine transforms using Eigen, and visualizing using PCL. I have looked at the Conix center project you mentioned, they do not seem to be doing more then hardcoded affine transforms, the same as me.
@ev-mp I'm doing head tracking, yes. I cannot use thresholding like that, it would ignore children, maybe adaptive thresholding would work. But this is not related.
I think I still didn't present the problem clearly enough, so I'm going to support it with more pictures.
Here is a visualization from Viewer, the same setup as in first post - camera on the ceiling facing floor:

I have drawn lines corresponding to room geometry. By drawing rs2::frame I believe that data from it are projected on the plane with a perspective distortion.
However, when doing the stitching of point clouds generated using rs2::pointcloud, I believe that rs2::pointcloud is actually doing (non-affine) transformation to remove the perspective from data obtained from rs2::frame. Is this the case or am I wrong?
I think that I was further confused by using spatial filter. My hypothesis is, that the data generated by rs2::pointcloud do not contain perspective, and can therefore be stitched with only affine transform. But there is noise which spits some of the points (those further from the camera, around the contour of the person perphaps) in a way that creates a sort of ellipsoid, see left part of the image:

On the left, one can clearly see people in the noise. However, on the right, after using spatial filter, the noisy parts get smoother and solidified. Then the blob is stretched in the direction from the camera, as a shadow of the person, which for me, created an illusion of a perspective. But now I see there is no perspective, it's just a problem created by using spatial filter.
So.
The problem with the filtering makes the stitched point cloud less useful to me now, then just doing my calculations on a 2D image from rs2::frame. But if the rs2::frame only has data with perspective distortion, it is a complication. Is there a way of getting rid of the perspective?

Hi @MartyG-RealSense and @ev-mp, thank you for your assistance and sorry for late reply - I needed to do some tests. I wish you all the best in the new year :)
@MartyG-RealSense I'm doing affine transforms using Eigen, and visualizing using PCL. I have looked at the Conix center project you mentioned, they do not seem to be doing more then hardcoded affine transforms, the same as me.
@ev-mp I'm doing head tracking, yes. I cannot use thresholding like that, it would ignore children, maybe adaptive thresholding would work. But this is not related.
I think I still didn't present the problem clearly enough, so I'm going to support it with more pictures.
Here is a visualization from Viewer, the same setup as in first post - camera on the ceiling facing floor:

I have drawn lines corresponding to room geometry. By drawing rs2::frame I believe that data from it are projected on the plane with a perspective distortion.
However, when doing the stitching of point clouds generated using rs2::pointcloud, I believe that rs2::pointcloud is actually doing (non-affine) transformation to remove the perspective from data obtained from rs2::frame. Is this the case or am I wrong?
I think that I was further confused by using spatial filter. My hypothesis is, that the data generated by rs2::pointcloud do not contain perspective, and can therefore be stitched with only affine transform. But there is noise which spits some of the points (those further from the camera, around the contour of the person perphaps) in a way that creates a sort of ellipsoid, see left part of the image:

On the left, one can clearly see people in the noise. However, on the right, after using spatial filter, the noisy parts get smoother and solidified. Then the blob is stretched in the direction from the camera, as a shadow of the person, which for me, created an illusion of a perspective. But now I see there is no perspective, it's just a problem created by using spatial filter.
So.
The problem with the filtering makes the stitched point cloud less useful to me now, then just doing my calculations on a 2D image from rs2::frame. But if the rs2::frame only has data with perspective distortion, it is a complication. Is there a way of getting rid of the perspective?

Hi @MartyG-RealSense and @ev-mp, thank you for your assistance and sorry for late reply - I needed to do some tests. I wish you all the best in the new year :)
@MartyG-RealSense I'm doing affine transforms using Eigen, and visualizing using PCL. I have looked at the Conix center project you mentioned, they do not seem to be doing more then hardcoded affine transforms, the same as me.
@ev-mp I'm doing head tracking, yes. I cannot use thresholding like that, it would ignore children, maybe adaptive thresholding would work. But this is not related.
I think I still didn't present the problem clearly enough, so I'm going to support it with more pictures.
Here is a visualization from Viewer, the same setup as in first post - camera on the ceiling facing floor:

I have drawn lines corresponding to room geometry. By drawing rs2::frame I believe that data from it are projected on the plane with a perspective distortion.
However, when doing the stitching of point clouds generated using rs2::pointcloud, I believe that rs2::pointcloud is actually doing (non-affine) transformation to remove the perspective from data obtained from rs2::frame. Is this the case or am I wrong?
I think that I was further confused by using spatial filter. My hypothesis is, that the data generated by rs2::pointcloud do not contain perspective, and can therefore be stitched with only affine transform. But there is noise which spits some of the points (those further from the camera, around the contour of the person perphaps) in a way that creates a sort of ellipsoid, see left part of the image:

On the left, one can clearly see people in the noise. However, on the right, after using spatial filter, the noisy parts get smoother and solidified. Then the blob is stretched in the direction from the camera, as a shadow of the person, which for me, created an illusion of a perspective. But now I see there is no perspective, it's just a problem created by using spatial filter.
So.
The problem with the filtering makes the stitched point cloud less useful to me now, then just doing my calculations on a 2D image from rs2::frame. But if the rs2::frame only has data with perspective distortion, it is a complication. Is there a way of getting rid of the perspective?

Hi @mrehacek   The effect in the above image resembles a distortion phenomenon called ghost noise.   This distortion can take different forms.  The points spray from the humans in your particular case resembles point-cloud images of humans from a previous case where the camera was also mounted in an elevated position.
https://github.com/IntelRealSense/librealsense/issues/7021

The RealSense user in that case settled on using the Medium Density camera preset in combination with a high Laser Power setting to improve their results (Medium Density provides a balance betweeb accuracy and image 'fill-rate').

Hi @mrehacek   The effect in the above image resembles a distortion phenomenon called ghost noise.   This distortion can take different forms.  The points spray from the humans in your particular case resembles point-cloud images of humans from a previous case where the camera was also mounted in an elevated position.
https://github.com/IntelRealSense/librealsense/issues/7021

The RealSense user in that case settled on using the Medium Density camera preset in combination with a high Laser Power setting to improve their results (Medium Density provides a balance betweeb accuracy and image 'fill-rate').

@MartyG-RealSense I have noticed I could remove the trails because the points are distinctively sparse, then found this issue under the name "shadow points"  in pcl lib, it's precisely what you're mentioning! I will note the camera settings also, but I'm working only with a dataset, so I won't be able to confirm.
I have tried turning off the temporal filter, the filter (or defaults) seems good, isn't the cause. I have successfully used pcl::StatisticalOutlierRemoval to get rid of it. I cannot tell if the method will be fast enough though, until I implement the rest of my pipeline.
I consider this particular part with the point clouds clear now.
I will try to better summarize the questions we still didn't discuss:

The depth data in rs2::frame are from the perspective of the camera, as there was no other processing inside the lib?
By using rs2_deproject_pixel_to_point() one can remove the perspective of the camera? I see in the sources the rs2::pointcloud does use it, and it seems like the only option.
If I want to project back from point cloud to rs2::frame with different projection, I need to write something similar to rs2_project_point_to_pixel()?

@MartyG-RealSense I have noticed I could remove the trails because the points are distinctively sparse, then found this issue under the name "shadow points"  in pcl lib, it's precisely what you're mentioning! I will note the camera settings also, but I'm working only with a dataset, so I won't be able to confirm.
I have tried turning off the temporal filter, the filter (or defaults) seems good, isn't the cause. I have successfully used pcl::StatisticalOutlierRemoval to get rid of it. I cannot tell if the method will be fast enough though, until I implement the rest of my pipeline.
I consider this particular part with the point clouds clear now.
I will try to better summarize the questions we still didn't discuss:

The depth data in rs2::frame are from the perspective of the camera, as there was no other processing inside the lib?
By using rs2_deproject_pixel_to_point() one can remove the perspective of the camera? I see in the sources the rs2::pointcloud does use it, and it seems like the only option.
If I want to project back from point cloud to rs2::frame with different projection, I need to write something similar to rs2_project_point_to_pixel()?

The origin point of depth data on the 400 Series cameras is the center of the left infrared imager.  From the perspective of the stereo camera looking out at the world, the left imager is on the left side of the camera module.  Thus, when the user is facing the D4 camera, the left imager is actually on the right side of the camera module.

The link below describes the coordinate system in relation to the center of the left IR imager.
https://github.com/IntelRealSense/librealsense/issues/7279#issuecomment-689031488

rs2_deproject_pixel_to_point maps 2D image pixel coordinates to 3D world coordinates.  An example of its use would be finding the 3D z-depth of a coordinate, or converting a 2D image to a 3D point-cloud image.

Yes, rs2_project_point_to_pixel translates in the other direction, from 3D point coordinatess to 2D pixel coordinates.

Reading back through the case from the start, it seems as though when you talk about 'removing perspective', you are having difficulty aligning images from multiple cameras as they are showing full-body views of the human bodies in the scene that the cameras are located in instead of just the top of the head.  So you would prefer to just have a view of the head alone if possible, excluding the rest of the body (removing perspective).
You confirm that you are performing a head count and so only need the heads, but cannot limit the maximum observable depth to neck level using a threshold filter because this may exclude the heads of children passing beneath the camera.
If it is not a vital criteria that the application be open-source (created by yourself), the RealSense-compatible commercial software Cubemos Skeleton Tracking SDK may be able to meet your person-counting needs, as its user manual states that it has a function for counting detected human skeletons.  Then you may not need to take a multiple-camera approach and could capture the people in the scene with a single camera.
https://www.intelrealsense.com/skeleton-tracking/

The origin point of depth data on the 400 Series cameras is the center of the left infrared imager.  From the perspective of the stereo camera looking out at the world, the left imager is on the left side of the camera module.  Thus, when the user is facing the D4 camera, the left imager is actually on the right side of the camera module.

The link below describes the coordinate system in relation to the center of the left IR imager.
https://github.com/IntelRealSense/librealsense/issues/7279#issuecomment-689031488

rs2_deproject_pixel_to_point maps 2D image pixel coordinates to 3D world coordinates.  An example of its use would be finding the 3D z-depth of a coordinate, or converting a 2D image to a 3D point-cloud image.

Yes, rs2_project_point_to_pixel translates in the other direction, from 3D point coordinatess to 2D pixel coordinates.

Reading back through the case from the start, it seems as though when you talk about 'removing perspective', you are having difficulty aligning images from multiple cameras as they are showing full-body views of the human bodies in the scene that the cameras are located in instead of just the top of the head.  So you would prefer to just have a view of the head alone if possible, excluding the rest of the body (removing perspective).
You confirm that you are performing a head count and so only need the heads, but cannot limit the maximum observable depth to neck level using a threshold filter because this may exclude the heads of children passing beneath the camera.
If it is not a vital criteria that the application be open-source (created by yourself), the RealSense-compatible commercial software Cubemos Skeleton Tracking SDK may be able to meet your person-counting needs, as its user manual states that it has a function for counting detected human skeletons.  Then you may not need to take a multiple-camera approach and could capture the people in the scene with a single camera.
https://www.intelrealsense.com/skeleton-tracking/

Thank you @MartyG-RealSense, your observation is correct.
As for the use of tracking sdks, I will have occlusions, cannot use them. I've only seen full-body skeletal tracking, I assume they fail if provided with this ceiling-view data. But I will try cubemos, maybe their ML model can deal with it.
I will need a few more days for testing, if you could please leave this open.

Thank you @MartyG-RealSense, your observation is correct.
As for the use of tracking sdks, I will have occlusions, cannot use them. I've only seen full-body skeletal tracking, I assume they fail if provided with this ceiling-view data. But I will try cubemos, maybe their ML model can deal with it.
I will need a few more days for testing, if you could please leave this open.

Yes, I will check with you for an update in 7 days if I have not heard from you by then.  Good luck!

Yes, I will check with you for an update in 7 days if I have not heard from you by then.  Good luck!

Hi @mrehacek   Do you have an update that you can provide, please?   Thanks!

Hi @mrehacek   Do you have an update that you can provide, please?   Thanks!

@MartyG-RealSense I do not. However, I think I now know which direction I'm heading, so you can close this if you need. Thank you, and wish you well!

@MartyG-RealSense I do not. However, I think I now know which direction I'm heading, so you can close this if you need. Thank you, and wish you well!

Okay, thanks very much @mrehacek - please feel free to create a new question on this forum at a future date if you need to.  Good luck!

Okay, thanks very much @mrehacek - please feel free to create a new question on this forum at a future date if you need to.  Good luck!


style: reduce code duplication

Codecov Report

Merging #47 (8ea82f7) into master (c7b5074) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 8ea82f7 differs from pull request most recent head 6fc4a81. Consider uploading reports for the commit 6fc4a81 to get more accurate results

@@           Coverage Diff           @@
##           master      #47   +/-   ##
=======================================
  Coverage   98.82%   98.82%
=======================================
  Files          12       12
  Lines         427      427
  Branches       80       80
=======================================
  Hits          422      422
  Partials        5        5

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


Repo Restructure 2 - hdinsight, trafficmanager, mgmt-servicebus, servermanager, serialconsole, scheduler, reservations, redis

Details:
git mv azure-mgmt-redis/ sdk/redis/azure-mgmt-redis
git mv azure-mgmt-reservations/ sdk/reservations/azure-mgmt-reservations
git mv azure-mgmt-scheduler/ sdk/scheduler/azure-mgmt-scheduler
git mv azure-mgmt-serialconsole/ sdk/serialconsole/azure-mgmt-serialconsole
git mv azure-mgmt-servermanager/ sdk/servermanager/azure-mgmt-servermanager
git mv azure-mgmt-servicebus/ sdk/servicebus/azure-mgmt-servicebus
git mv azure-mgmt-trafficmanager/ sdk/trafficmanager/azure-mgmt-trafficmanager
git mv azure-mgmt-hdinsight/ sdk/hdinsight/azure-mgmt-hdinsight
dev_requirements updated. No updates to docsettings are expected for this one.

Can one of the admins verify this patch?

@idear1203 Heyo,
As part of the azure-sdk revamp, I'm moving packages in the repo to match the same structure that is shared by other mono-repos owned by the azure sdk team.
You're the code owner for hdinsight, which is one of the management plane packages being moved. The command used to move said code is listed in PR description. I will leave this PR open for one more day for you to respond, then just move the package. I am also responsible for making the changes on the autorest repo, so this should be a completely transient change for you.
Thanks!
Scott

@Azure/azure-sdk-eng


Light Theme

Will the figma toolkit have a light theme added?

I'll likely wait until there is significant interest from the community as it does take a bit of time to set that up.

We now have light themed components, see https://twitter.com/miguelsolorio_/status/1470468314409275392


window.default is set to contain the foundation object

What should happen?
Including the foundation.js file should make Foundation and other plugins available on a page. It does so by filling in window[Foundation] = .... and window[Abide] ...., etc
https://github.com/foundation/foundation-sites/blob/develop/dist/js/foundation.js#L8
What happens instead?
In the list of plugins to include, there is a default which ultimately overwrites window.default when this file is included in the browser:
https://github.com/foundation/foundation-sites/blob/develop/dist/js/foundation.js#L103
This causes issues with other js applications running in the page.
Possible Solution
Can the default be removed from this? I have tested with delete window.default after the file loads and everything seems to work fine.
Test Case and/or Steps to Reproduce (for bugs)

Context
Somehow this interferes with other applications because it's affecting the global namespace
Your Environment

Foundation version(s) used:  6.6.3
Browser(s) name and version(s): chrome, firefox
Device, Operating System and version: mac os

Checklist

[X] I have read and follow the CONTRIBUTING.md document.
[X] There are no other issues similar to this one.
[X] The issue title and template are correctly filled.

In the list of plugins to include, there is a default which ultimately overwrites window.default when this file is included in the browser:

Well, simple solution:
don't expect that globals are safe

This causes issues with other js applications running in the page.

Which applications sets / uses window.default?
In general webpack transpiles the default export to this:
/* harmony default export */ __webpack_exports__["default"] = (_foundation_core__WEBPACK_IMPORTED_MODULE_1__["Foundation"]);
See https://github.com/foundation/foundation-sites/blob/develop/js/foundation.js#L36 and the descriptions at https://developer.mozilla.org/de/docs/Web/JavaScript/Reference/Statements/export
The default export is needed for the ESM and CJS packages in Nodejs (require(packagename).default).

Just a short example why it is a bad idea to use window globals:
https://codepen.io/DanielRuf/pen/vYLyrKd?editors=1111
When you use Foundation Sites in a Nodejs project and bundle it, there should be normally no global objects then as this is scoped then.

Can the default be removed from this?

It is safe to remove but we won't change the bundles. This is the first time that someone mentioned the window globals and that there might be conflichts with other libraries. In this case these libraries should never rely on window.default as you can see in my codepen.

Yeah, I understand I shouldn't trust globals, but then why is this script modifying them?
The module I'm having issues with is video.js. Essentially at some point, it complains that window.setTimeout doesn't exist.
I don't know much about how this works in a nodejs project honestly, I'm using it on a PHP project by including it in a script tag.

This is how webpack writes the exports.

Essentially at some point, it complains that window.setTimeout doesn't exist.

We don't remove or unset this.
In this case please open a bugreport in video.js regarding the global window object.

See https://github.com/videojs/video.js/issues/5758


[Suggestion] Deleting command messages

Sidekick has been suggested to me since my favorite Discord bot has been taken offline for the time being, but my absolute favorite feature of it was what I called "cleaning up after itself" which is to say when you posed a commend, ie;
!roll 1d10
It would show you the results
@Name: 1d10 (9) + 5 = 14
But it would delete the command line (!roll 1d10) and it just made the games feel less cluttered and busy, you'd get the roles without all the unnecessary visual commands.
I would very much like to see an option like this for Sidekick, visual clutter is very distracting to me.

Hey!

since my favorite Discord bot has been taken offline for the time being

I'd appreciate if you'd elaborated on this.

But it would delete the command line

So before the official bot API there were bots that you'd run on your PC and let them work on your behalf. This is not recommended by Discord anymore.
The newer bots that use the official bot API are supposed to work as a separate user (e.g. Sidekick is a separate Discord user). As such, they can't simply remove whatever message they like from the chat, no more than a normal user can.
Perhaps with admin permissions...
Is that how your favorite bot operated? With admin permissions?

The bot I have been using for my server is Storydice bot, and has been very much in beta. Because of life reasons they have opted to suspend development of the bot and have taken it offline so they don't have to worry about maintenance.
To answer your question, yes the bot did have to be given permission to "Manage Messages".

Cool, thanks for the info! I'll give it a go.

Awesome! I look forward to it!


Where do you get the data, ...

or did you calculate yourself?

or did you calculate yourself?

trial & error and arithmetic. 🤘🤘🤘


Where is this error text located to translate it?

Hello:
I have translated all the texts of the App into Spanish, but there is a message that is still in English and I cannot find where it is defined.

Does anyone know where it is?

Hello, @ferchinas!
Please clarify if you still have this issue.
Thank you!


upgrade go sample to new layout

Description

Upgrades the go sample for the new layout
Add OLM and scorecard manifests as well

PS.: The steps to test are in the README which was updated in this PR.

/hold
I upgrade the sample with master and our latest tag release here is v0.18.1.
we need check if we will or not create a tagged release for 0.19.x and the v1.0.0 here.
c/c @joelanford

Hi @varshaprasad96,
I will close this one because was made manually and you are working on it the script.


[BUG] Potential issues with WFRP 4th

Describe the bug
Seen on discord, not yet investigated...

Hello! I've just installed the WFRP 4th ed game system. For D&D I increase the size of status icons on tokens using Illandril's Token HUD scaler, Status Icon Tweaks or the Combat utility Belt. They all work in D&D but in WFRP the status icons get reset back to their default sizes either immediately or after logging back in, depending on the module. I've also tried Custom CSS - same thing. Does anyone have any suggestions?

I could not reproduce this issue.


ARM toolchain - add support for small-build

Same as for GCC, if small-build is selected, select microlib library. This
is currently duplicated from microlib. I assume we deprecate microlib toolchain, thus
duplication should be removed (regarding extending cc/asm flags by --microlib, etc).
Tested with nrf51 microbit:
python tools\make.py -m NRF51_MICROBIT -t ARM -n RTOS_1
Build Options: debug-info
Building project basic (NRF51_MICROBIT, ARM)
Scan: basic
Scan: mbed
Scan: rtos
Scan: env
Compile: main.cpp
[Error] main.cpp@6,0:  #35: #error directive: [NOT_SUPPORTED] test not supported
[ERROR] "C:\Code\git_repo\github\mbed\libraries\tests\rtos\mbed\basic\main.cpp", line 6: Error:  #35: #error directive: [NOT_SUPPORTED] test not supported
C:\Code\git_repo\github\mbed\libraries\tests\rtos\mbed\basic\main.cpp: 0 warnings, 1 error

Exported project contains also microlib configuration (tested for NRF51 MICROBIT), and K64F (not containing single thread as its full stdlib python tools\make.py -m K64F -t ARM -n RTOS_1)
This PR assumes we got GCC and ARM toolchains and provide small std lib option. Not as it is uARM vs ARM.
@c1728p9 @sg- @pan-

Note: This can be considered breaking as it changes for targets with ARM toolchain support and defined small-build. We could add a warning to this addition, that would be printed on the command line, that this has changed >122 release, and now it is using microlib (if microlib flag is defined).
cc @bogdanm @screamerbg

Looks good to me but this can be a breaking change for some targets.
On the other hand, without this change the behavior is inconsistent between toolchains.
If an application is developed with ARMCC first and use threads then the same application will have runtime errors if compiled with GCC.

@mbed-bot: TEST
HOST_OSES=windows
BUILD_TOOLCHAINS=GCC_ARM,ARM,IAR
TARGETS=K64F,NRF51_DK,NRF51_MICROBIT,NUCLEO_F411RE,KL25Z

Note: needs:work label is the same as DON'T MERGE YET , we can eliminate one of them. needs: work can  be changed to red label.
Back to the issue, the builds fail because of inconsistency. To provide small build, we use microlib for ARM, but microlib requires different startup file and many of targets do not support it. To complete this fix, we might need to unify this.
An example for LPC1768 - diff between two startups (microbit and full std):
--- C:\Code\git_repo\github\mbed\hal\targets\cmsis\TARGET_NXP\TARGET_LPC176X\TOOLCHAIN_ARM_MICRO\startup_LPC17xx.S  2016-07-07 11:03:18.170000000 +0100
+++ C:\Code\git_repo\github\mbed\hal\targets\cmsis\TARGET_NXP\TARGET_LPC176X\TOOLCHAIN_ARM_STD\startup_LPC17xx.S    2016-07-22 09:29:37.019000000 +0100
@@ -19,25 +19,8 @@
 ; *
 ; *****************************************************************************/

-Stack_Size      EQU     0x00000400
-
-                AREA    STACK, NOINIT, READWRITE, ALIGN=3
-                EXPORT  __initial_sp
-
-Stack_Mem       SPACE   Stack_Size
 __initial_sp        EQU     0x10008000  ; Top of RAM from LPC1768

-
-Heap_Size       EQU     0x00000000
-
-                AREA    HEAP, NOINIT, READWRITE, ALIGN=3
-                EXPORT  __heap_base
-                EXPORT  __heap_limit
-
-__heap_base
-Heap_Mem        SPACE   Heap_Size
-__heap_limit
-
                 PRESERVE8
                 THUMB

@c1728p9 You might be already looking at this, let me know

@0xc0170, unfortunately I haven't looked in depth at the difference uARM and ARM startup files.  This is something I definitely want to address in the future though.

@0xc0170 is this a duplicate or already handled by https://github.com/ARMmbed/mbed-os/commit/5197edc913e1c1318bc8f9206af6aa4572960247

@sg-  It is not a duplicate and it is not handled by 5197edc

Great, in that case probably needs update based on the naming in 5197edc

@0xc0170 anything left to get done here or is this still on hold?

Tracking issue: #2902 . Closing this as it's old and should be reconsidered.


Update your plugin to support whatever new API stuff YouTube is doing.

Even after 2 restarts, this still happens.

Update the lavaplayer-lib: https://modrinth.com/plugin/pv-addon-lavaplayer-lib/version/1.0.7


Fix 3rd party license updater (automation)

Instead of pushing directly, since the main branch is protected, this creates a pull request.
Failure: https://github.com/erlef/setup-beam/actions/runs/1852644324
It's possible this pull request doesn't pass CI since the 3rd party licenses are outdated, but once it's accepted+merged in at most 24h stuff should be back to normal.

Here's the failing bit, as expected: https://github.com/erlef/setup-beam/runs/5224903868?check_suite_focus=true#step:14:12

@paulo-ferraz-oliveira It looks this is conflicting with https://github.com/erlef/setup-beam/blob/6c5caca0fb67184842b9de3a5fd39a1a1f51dea0/.github/workflows/action.yml#L31

@paulo-ferraz-oliveira It looks this is conflicting with
https://github.com/erlef/setup-beam/blob/6c5caca0fb67184842b9de3a5fd39a1a1f51dea0/.github/workflows/action.yml#L31

It's because the 3rd party licenses are outdated, which is what the pull requests wants to fix 😄 I'd prefer to not fix it here, merge, let the update happen via the action, and then "all's well" (the last time we fixed the files in the pull request, the action ran and failed after merge 😄). After that we can merge the other pull requests, following a rebase...
Let me know, though, if you prefer to do the fix here.

Thanks for the approval. I'm merging and running the action in main to see if the pull request gets created. 👍


Add translation example from chinese, french to english

This adds:

a VAD node to detect voice within an audio to avoid having too much noise
Bump distil-whisper to whisper-turbo
Use Argotranslate and OPUS-MT for translation
Add example dataflow within the example folder

Get started
cd examples/translation
dora up

dora build dataflow_zh_en_terminal.yml
dora start dataflow_zh_en_terminal.yml --detach

python pretty_print.py

FYI: I'm going to add a way to easily switch between language while running the dataflow before merging


Cannot retrieve prices

Neutron validator version: 4.2.4
Connect version 2.0.0
cli options:
./connect --log-file /var/log/connect-sidecar.log --marketmap-provider marketmap_api --market-map-endpoint <IP_ADDRESS>:9090 --host <IP_ADDRESS> --port 8080 --metrics-prometheus-address <IP_ADDRESS>:8002

Connect throws logs like this:

Logs:

{"level":"error","ts":"2024-10-08T13:02:38.573Z","caller":"marketmap/fetcher.go:104","msg":"failed to query market map module on node","pid":1092531,"process":"oracle","fetcher":"marketmap_api","error":"rpc error: code = Unimplemented desc = unknown service connect.marketmap.v2.Query"}

curl localhost:8080/connect/oracle/v2/prices | jq . return next:
{
  "prices": {},
  "timestamp": "2024-10-08T12:36:43.573356875Z",
  "version": "2.0.0"
}

What should I do? Rollback to version 1.0.*?

Sorry! I missed that Neutron didn't use v2 proto.


Handle pipe error or disk full/error in bmdcapture push_packet()

In push_packet functions, the return code of av_interleaved_write_frame is ignored.
We don't know if the outfile is in a disk error, disk full, or pipe issue state.
it seems you planned to do that by defining "ret" as an integer ;)
This patch handle this case.

Sounds a good idea to check it indeed!
I'll push it with some rewording on the commit message.
In theory all the exit causes should have a proper value...


Training results of COCO

Hello, This is an amazing work , however, I faced a challenge while training the model on the COCO dataset. The below picture shows the results reported. What could be the reason ? Hoping to receive a reply from you.Thank you.

I face the same problem, did you solve it?

Turns out the OP was loading our weights while using the 1e-4 LR, which is intended for training with ImageNet weights from scratch. Please open new issues if there are new problems arising.

I face the same problem, did you solve it?
Have you solved it？

Hello, This is an amazing work , however, I faced a challenge while training the model on the COCO dataset. The below picture shows the results reported. What could be the reason ? Hoping to receive a reply from you.Thank you.

Have you solved this problem


@craigspaeth => avoid race condition when checking whether a bidder is qualified/registered

Since fetchBidderForAuction doesn't return a promise, these requests were potentially not all completing. This means on the auction page, occasionally people would see "registration pending" instead of "Approved to bid."
Thanks @craigspaeth for the help in detangling this!

Closes: https://github.com/artsy/auctions/issues/122

Nice! Sorry for the merge delay


An option to keep undefined variables

It would be nice to support multi stage rendering as described at https://stackoverflow.com/a/53134416/948378.
from jinja2 import Environment, BaseLoader,DebugUndefined
rtemplate =Environment(loader=BaseLoader,undefined=DebugUndefined) \
           .from_string("{{ a }} is defined, but {{ b}} is undefined")

print(rtemplate.render({"a":"a"}))

# The result is:
# a is defined, but {{ b }} is undefined

Hi @pietervogelaar , thank you for the feature request: you're not the first one asking about this :)
I wonder what the use case could be. Wouldn't it be possible in your case to merge the variables before rendering a template, instead of doing this multi-stage rendering? Isn't it prone to errors?

DebugUndefined may indeed be an unorthodox way to implement multi-stage processing. But since this a feature already implemented in Jinja2, and exposing it adds practically no cruft to j2cli, I can't see why not to expose it if someone has a use for it.
Personally, I use this behaviour in its original scope: debugging. If you are trying to render and debug 100+ templates you can generate your output using DebugUndefined. Then a grep -r '{{' output/ will help you quickly fix missing variables in the context, without going through multiple runs.
Note that I had implemented this (among other things) in #13. The patch is now outdated, but it should be straightforward to pull the relevant lines. I can send a new PR only with this part if you're too busy.

My use case for this is that I want to template GitHub Actions workflow files. GitHub Actions uses ${{ }} for it's own templating. I want to ignore those and just template what I pass data for.
e.g. j2cli should just ignore secrets if --undefined is set in my data.
          token: ${{ secrets.MY_TOKEN }}

But it errors:
jinja2.exceptions.UndefinedError: 'secrets' is undefined

This is quite disappointing because it means j2cli (and perhaps jinja2 altogether) is unusable for my use case. I'll need to find another templating engine if I can't figure out a way to ignore.

My use case for this is that I want to template GitHub Actions workflow files. GitHub Actions uses ${{ }} for it's own templating. I want to ignore those and just template what I pass data for.
e.g. j2cli should just ignore secrets if --undefined is set in my data.
          token: ${{ secrets.MY_TOKEN }}

But it errors:
jinja2.exceptions.UndefinedError: 'secrets' is undefined

This is quite disappointing because it means j2cli (and perhaps jinja2 altogether) is unusable for my use case. I'll need to find another templating engine if I can't figure out a way to keep undefined variables.

How about using {% raw %} blocks https://jinja.palletsprojects.com/en/3.1.x/templates/#escaping?
Or perhaps a custom filter, where you would write {{ "secrets.MY_TOKEN" | ghaction_env }} in the template, to get ${{ secrets.MY_TOKEN }} in the output?

{% raw %} probably would work, but I found the best workaround was just to change the start/end delimiters with --customize custom.py.
# custom.py file for j2cli

def j2_environment_params():
    return dict(
        # modify the variable delimiters to avoid conflicts with GitHub Actions templating
        variable_start_string='<<',
        variable_end_string='>>',
    )

Sorry for the noise in this issue. I still think it would be a useful feature to just ignore undefined keys, though.


chore: release 1.7.2

:robot: I have created a release *beep* *boop*
1.7.2 (2022-02-25)
Bug Fixes

calling response.json() in middleware will cause garbled chinese characters (40ffe02)
throw error when not set url origin (9040396)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/keq-request/keq/releases/tag/v1.7.2 :sunflower:


is non-deep features of dlib supports gpu?

hi @davisking , thanks for your great contribution, it is really nice to see dlib has deep NN features and support cuda , is the non-deep features, for example, the classic svm also supports gpu?

No problem.
No, only the DNN stuff uses cuda.


Story/CEC - 685 SuccessFactors Run Time Implementation

please squash commits

Thanks! All Commits have been squashed. 573b41d557df90ccd9473edbf20d05d73e31d569


Want to use this to make a test explorer for Karma( specifically for working with angular)

Hey man, first I wanna say that I really like your work. Really useful. I was thinking of making an implementation for angular using this adapter. Could you give me some pointers or where should I start looking for implementing loading the tests, running them etc? Do you have a common approach you used in your current implementations for loading this tests and making the implementation of the different adapters?.
I am definitely not an expert but I am pretty sure that someone is suffering for not having this as much as me.
Thanks

Thanks a lot man, I started yesterday but without any progress really. Still trying to figure out how to discover the tests. Unfortunately karma documentation is not as good as jasmine and also the tipings does not include things like the reporter so I will spend some time trying to make it work with typescript and then make it work for the problem at hand. This morning I already wake up with some ideas so lets see go this goes ahah. 👍

Hey so I been doing some progress. still not being able to discover the tests but definitely getting a feeling of how this should be done. I have one question and maybe you could help me.
how can I require a module that should be installed in the project? in my case I wanna require from the adapter the angular cli that should be installed in project open in vscode. is this possible or the adapter should have a copy of the module?

This would make my life!  Thanks for helping us Angular folks out!

@RichiCoder1 @supersinex  here is the repository. https://github.com/Raagh/angular-test-explorer .
I arrived today so its still missing some of the things I promised. I will try to do it today( I cannot promise it as I am suffering from a massive jetlag)  I will create a few tasks of the things we re missing to complete a basic implementation. and also create a few issues I have with the current implementation.
In the meantime please take a look at the code and if you have any doubts please feel free to ask in the repository :)

@Raagh Awesome! If there's some way I can contribute, let me know 👍

I am having this issue where after loading the tests , I try to run them and the first 2 times it works okey but on the thrid one it stops abruptly and then if I try again karma responds with "There is no server listening on port 9876".
I created a comment in the webstorm community ( webstorm currently has a plugin that has some of  the functionality I want to implement and the owner of the plugin has been responding) with a better explanation of the issue.
https://intellij-support.jetbrains.com/hc/en-us/community/posts/360002754120-IDEA-karma-js-plugin-help-Building-Angular-test-explorer-for-vscode?flash_digest=b1efc380b11c8a78b46af4bb075ab7696361443c&page=1
If someone wants to clone my repo, reproduce it and help me fix it, it would be awesome. please feel free to contact me at any time to my email regarding this.
I will keep trying. thanks guys

I tried to run your code but I get lots of compilation errors. For example, in test-explorer-karma.conf.ts you import Reporter from "karma", but @types/karma doesn't export a Reporter interface...
Are you sure that you pushed all necessary commits?

@hbenl sorry just noticed that my gitignore was excluding the folder for typings which includes the Reporter interface.
You re still gonna se some errors for the typescript compiler but none of them prevent execution. they re just unused code and missing type definitions.

@Raagh I've forked your repo and pushed some commits to a new branch called contrib. I had to update the dependencies because @angular-devkit/build-angular was pulling in an old version of webpack which didn't work with the configuration that a recent version of ng-cli created...

@hbenl  I just noticed that the update in the dependencies broke the build. why you had to update them to be able to run? I am getting an error in @angular-devkit/build-angular probably related to the faulty webpack version you mentioned but I am using the latest branch. I am currently on a fresh install on windows. I am gonna try it later on my Mac but deleting node_modules and everything and starting again.

The error I was getting was something about the webpack config containing a property futureEmitAssets that webpack didn't know about and so it refused to work with that config. The reason was that the adapter used the globally installed version of the ng command (which was a very recent version) but a locally installed (in the adapter's node_modules) version of webpack which was too old (the futureEmitAssets property was added to webpack just a few weeks ago).
So I think you're right: the adapter should use the user's version of ng-cli and webpack (and I guess karma as well).

@supersinex @RichiCoder1 @mattmeye @sumigoma @LuisReinoso @avavricek
preview release is out https://marketplace.visualstudio.com/items?itemName=raagh.angular-karma-test-explorer
please report bugs :)

@Raagh This is AMAZING! Awesome work, my dude!


fix: markerwithlabel version upgrade to fix onRemove validation

in previous version 2.0.1 of markerwithlabel, there were no validations added while unmounting the markerwithlabel. They have released a new version 2.0.2 to fix the issue. I have upgraded the dependency and checked all reflected changes in node modules. Please review it as I'm also currently using it my app and app crashes while markerwithlabel gets unmounted.

When we can expect this to be fixed/merged? Thanks in advance.


feat(app): add a few components for deck configuration on ODD

Overview
Add a new menu item to navigation, add a new instruction modal and a new confirmation modal on ODD
This PR is a part of for RAUT-663 since need to wire up these components to deck configuration component.
When deck configuration component is ready, I will update functions and test cases.

Test Plan

Changelog

add a new item to navigation and update test
add a new instruction modal and tests
add a new confirmation modal and tests

Review requests

Risk assessment
low

[ ] add tests for AddDeckConfigurationModal

looks good, i wonder if it would be useful to add the modals to storybook so we can see how they look before the deck configurator component is done. But code seems fine to me! Just left a few comments

That is a good suggestion. Just added 3 stories


Publish to opm

Hello, OPM is not part of Kong's infrastructure and we are not interested in supporting this feature at the moment. Closing this down for now.


Vote: should we add something like this to the docs?

My guess is that having it laid out like this is useful, but could easily go stale.
https://groups.google.com/d/msg/dropwizard-user/Om_N_4WIZfQ/0U03LI8oZC0J

Keeping on top of it is the primary issue. Otherwise, I'm all for clarity. Having that laid out somewhere in the documentation would be great in my opinion.

I find it extremely useful.
We could create "Dropwizard Internals" section in the docs, where we can post such guides.
About keeping it relevant, I believe that not writing a documentation just because it's hard to maintain is a lame excuse. If we would start to write about internals, we will motivate more people to hack on Dropwizard. Community should help us to find mistakes and improve quality and relevance of the docs.

Okay. I like the ideas. I'll work on it.

https://github.com/dropwizard/dropwizard/pull/918

Closed by merging #918.


Classnames returns 'undefined' - Add a little logic

no more undefined class names
#21

@WiaczeslawP any further thoughts on this PR?
My aim with these PR's was to help fix a few tiny issues and delicately add extra CSS definitions without forcing a major update or breaking change. I'm planning on using this in production and I'd rather contribute to your great work rather than fork it. :)


feat: Add useAutosuggest hook

:tada: This issue has been resolved in version 0.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


New article: Introduction to digital images [maybe]

This is an article that might be useful to have, but would probably not come until later in the process of building out the media format documentation. Its purpose: to explain how an image is stored digitally, in general terms, and how compression is applied to the resulting data. The primary use for this is to talk about how different factors impact the appearance of digital images, such as lossy compression, recompression artifacts, moiré and other interference patterns, and so on. This is information that would be reference from a variety of places, including the planned image type and images for web designer articles, but also from learning area content and articles about various CSS and DOM imagery topics.


Add Promise to  es6 set

Resolves https://github.com/Financial-Times/polyfill-service/issues/667

LGTM


[WIP] [DRAFT] [DEMO][HACK] Demo how a list of itemExtents can speedup scroll

Drag the CupertinoScrollbar (the Material Scrollbar is unfortunately
undraggable) to see fast scrolling jump.
You can also try the slow version by removing the itemExtents.
This is a hack to just demo the idea. There are many places that are very fragile or plainly wrong, but I think it's sufficient to demonstrate the idea how itemExtents could potentially speed things up.
Related issue: https://github.com/flutter/flutter/issues/52207

Closes this demo PR as we never intend to merge this.


docs(K8s): Add instructions for loading the examples

This took me a long time to figure out because I'm new to Superset, Helm & Kubernetes.
I want to make it easier for others in the future.

SUMMARY
Adding instructions for loading the examples for Kubernetes deployments as this is different from Docker deployments.
This took me a long time to figure out because I'm new to Superset, Helm & Kubernetes, so I want to make it easier for others in the future.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

run helm upgrade --install superset superset/superset
Result: Examples will not be loaded and you will not see the examples database in Superset

Follow added instructions then run helm upgrade --install --alues my_values.yaml superset superset/superset
Result: Examples will be loaded and you will see the examples database in Superset

ADDITIONAL INFORMATION

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

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

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

Closing/re-opening to kick CI.


Fixes #344 - uses registry instead of netsh

When using non-English Windows, netsh output will not be in English.
Using Windows registry is locale independent.

This PR aims to fix #344 .

Good stuff :-)

Yes, I tested this on one Windows box.

Just squashed the commits.