moment.weekdays() is localized, but order is not ISO

It has been localized in #3284, but the order is still not ISO. I guess since the order is localized IRL, we might as well keep moment.weekdays() that way, but it would be nice to have moment.isoWeekdays(), that would be both ISO and localized − same syntax as moment().isoWeekday().

I imagine we also need moment.isoMonths(), as I believe year does not start on January in all countries. Not sure how this goes with moment.months().

I should have provided a use case, you are right. My use case is that I sort data according to their weekday, in a context where (despite english conventions and language), ISO weeks are used. Like :
let weekdays = moment.weekdays()
weekdays.push(weekdays.shift())
let sortFunction = d => weekdays.indexOf(d)

I have to add line 2 for going from english weeks to ISO weeks, and I feared that in case of multilingual applications, that would require a whole lot of checking for how different languages order weekdays (the kind of thing momentjs helps you to avoid).

You can remove line 2 if you make line 3 more complicated:
let sortFunction = d => (weekdays.indexOf(d) || 7)

A couple more thoughts: I assume there's a reason you're sorting by weekday name (e.g. you don't have a date.)  If you need to handle abbreviations too, you should look at moment.localeData().weekdaysParse(nameOfWeekday) - as that handles the abbreviated versions too - case-insensitively: e.g. Th, Thu, Thursday.
Regarding your suggestion about isoMonths() - do you know of any locale with a stable month ordering that doesn't start in January? (I mean: Chinese new year starts in either January or February, Islamic has started anywhere from September to December recently...)
Tax and financial years might be a good case where the months need to be ordered differently, but they don't map to a locale...
I think if you were going to add isoWeekdays, then you'd also have to add isoWeekdaysShort and isoWeekdaysMin for symmetry with weekdaysShort and weekdaysMin.  Similarly, you couldn't really add isoMonths without also adding isoMonthsShort for symmetry with monthsShort

That would be a lot to add indeed. I think most of the time, people will stick to the locale's weekdays order, and not make a mess with ISO like we do (hopefully they will report it here if they don't).


Allow rendering menu in separate plugin

These are some changes that I wanted to make for my project (#1), mainly to allow rendering the menu separately. I also went ahead and moved some of what I had in my overriden templates to the default templates, to have less and more semantic markup.
What can break for existing installations?

Default markup has changed, so for those who don't have an override, it will have to be done now;
I found I had to re-save my container plugin;

Appart from that, should work as it has.
Model changes

When installing the app for the first time, not requiring to override a template, I found it better to have a choice as to printing next/prev links, as well as the menu, because I want it in a sidebar, and be able to add social icons below it.

I removed the subordinate_page in SectionContainerPluginModel because it wasn't being used anywhere.

The menu functionality works with any implementation of the abstract container, because I'm checking for the abstract.

HTML markup changes

<article> is actually a specific kind of <section> for a "thing" that stands on it's own (like in an RSS feed item). I think it's a more semantic choice;
<main> should be used for the page's main content, and should only be used once;
Added an <h2> heading to section titles, if they exist;
Removed some HTML elements to provide de minimum. It's actually all I need in my project.

In my project I've overriden section-container.html to contain only this line:
{% include 'cmsplugin_sections/section-children.html' %}

That's because I want that default structure in my project's base.html, and still allow a default installation to give all that you need.
Other changes

Passing strings through gettext to allow for translation.

I'm sending all sections to the template and let it decide in the menu if it's to be shown or not. In my project the link in the logo wasn't working because my first item isn't in the menu. I use the logo as the first item.

Django CMS has changed and now treebeard/mp_tree.py function get_descendants has no parameter include_self and without this parameter cms/utils/plungins.py function build_plugin_tree doesn't find plugin by parent_id.
I changed get_descendants function in my virtualenv, but it's not the right solution.


Issue #1: Inconsistent Sidebar Styles

Issue #1: Inconsistent Sidebar Styles
All titles in the sidebar should have monospace style font
All bullets should be circles.
I changed the classes in the _content.html.erb from underscores to '-'s.
This seems to have stomped this bug.

Please try to use more descriptive commit messages @PJC-1


couldn't create edge for target handle id: undefined; edge id: edge-1

Describe the Bug
hello there , first for me to thank you guys that created this wonderful library , i have an issuse that can't create edge between 2 customNode
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
....
Expected behavior
create edge between 2 customNode
Screenshots or Videos

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

When you are working with a controlled flow you need to implement onNodesChange and onEdgesChange. Otherwise React Flow doesn't know about the handles: https://reactflow.dev/docs/getting-started/core-concepts/

tks for your support but the error still the same :(( , onNodesChange and onEdgesChange i get from useNodesState and useEdgesState

You are defining the nodes and the edge inside the component. This won't work. They get re-created on every render. Could you try to define them outside of the component?

hello once again , i try to put them outside the component but it did not work either :(( , here is my github project , i hope if you get that you'll have the solution :(( , it's in component/testComp   https://github.com/locloc408/facebook-mini-nextjs-tailwindcss , and once again tks for your support


240p (320x240)

Would you consider adding in support for 4:3 @ 320x240 resolution in the future.
Would love to be able to use this theme on a CRT in a progressive resolution.
Best Regards
/Rion
See our work here on Batocera for CRT output.
Batocera-CRT-Script

Would love this, 100%.

Many architectures cannot do interlaced resolutions (Intel in particular) which makes the beautiful 480i themes we have access to, inaccessible. 240p has half of the vertical pixels available which only really matters for reading text. Unfortunately reading text is 90% of the purpose of a ES theme.
There is also the factor of accessibility. Interlaced resolutions give me an incredible headache on most crts. I'm sure I'm not alone.
240p theme would also be the "lowest common denominator" even for lcd displays that are low resolution or very small. The benefit would extend past crt users.


Browser and useragent

Hello there!
Is there any functions or decorators that defines useragent and browser?
Something like
@Request('browser') browser // return string like 'Chrome' or 'Safary' or 'Other Browser' and etc.
@Request('useragent') useragent // return string like 'Windows' or 'Mac Os' or 'iOs' or 'Other Platform' and etc.

Use ˋ@Header` and https://www.npmjs.com/package/express-useragent

rhooo.... HeaderParams, please... take a time to read the documentation please.


Allow Dialog/Modal Re-rendering of fragments

Use case:  Using dynamic inline renderfragments with dialog/modal and desire the ability to trigger re-rendering of the dialog/modal either through direct invocation/event:
<RadzenDialog @ref="_dialogComponent" />

@ code {
    RadzenDialogComponent _dialogComponent;

    public void ActionTriggeringAChangeToFragment()
    {
        _dialogComponent.Refresh();
    }
}

where Refresh() would pretty much be a call to InvokeAsync(StateHasChanged);
or accept a delegate/callback argument into the DialogService.OpenAsync that allows the caller to request a re-rendering..such as Service.Refresh += () => StateHasChanged(); for the dialog.
Not super well versed in blazor but can do a PR if this is desirable

PR: https://github.com/radzenhq/radzen-blazor/pull/52


Fix: New golangci-lint linting errors

🔧 Changes
Golangci-lint recently bumped to v1.52.2 which provided some fixes to our automated linting process. As a result, we're now catching some minor linting issues that had previously slipped-through.
Example automated build failure: https://github.com/auth0/auth0-cli/actions/runs/4533123092/jobs/7985502629
📚 References

Evidence of build failure
Golangci-lint v1.52.2

🔬 Testing
Tested linting locally but the real test will be if it passes Github checks.
📝 Checklist

[x] All new/changed/fixed functionality is covered by tests (or N/A)
[x] I have added documentation for all new/changed functionality (or N/A)

Passed automated check: https://github.com/auth0/auth0-cli/actions/runs/4534665893/jobs/7989143687?pr=685


Can't build flutter: TLS error

Can't build flutter in Manjaro distribution, getting:
Building flutter tool...
Got TLS error trying to find package crypto at https://pub.dartlang.org.
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (9 tries left)
What to do here?

Maybe it is related to your system proxy settings.

Maybe it is related to your system proxy settings.

No proxy, I am trying to do this from home with my home network.

Hi @eduDev81
This looks like an issue with your firewall or network, see this Stackoverflow Thread
Also worth whitelisting flutter in any Antivirus or firewall  https://github.com/flutter/flutter/issues/61997
See https://github.com/dart-lang/pub/issues/1882#issuecomment-415588527
If the problem persists, please file an issue at https://github.com/dart-lang/pub
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Added UI to delete instance of event

The UI of deleting an instance of an event.
In the next commits, I'll add the logic for handling the requests and how a canceled instance is displayed

Codecov Report

Merging #114 (5ea9b68) into master (28ae801) will decrease coverage by 0.07%.
The diff coverage is 39.02%.

@@            Coverage Diff             @@
##           master     #114      +/-   ##
==========================================
- Coverage   69.41%   69.33%   -0.08%
==========================================
  Files          77       77
  Lines        6124     6180      +56
==========================================
+ Hits         4251     4285      +34
- Misses       1873     1895      +22

Impacted Files
Coverage Δ

.../pages/timetable/model/events/recurring_event.dart
87.50% <ø> (ø)

...ib/pages/timetable/service/uni_event_provider.dart
0.00% <0.00%> (ø)

lib/pages/timetable/view/events/event_view.dart
83.17% <30.43%> (-14.45%)
:arrow_down:

lib/generated/l10n.dart
63.02% <55.55%> (-0.17%)
:arrow_down:

lib/pages/timetable/model/events/uni_event.dart
48.93% <100.00%> (+1.11%)
:arrow_up:

lib/pages/settings/view/settings_page.dart
95.00% <0.00%> (-2.81%)
:arrow_down:

lib/generated/intl/messages_en.dart
96.40% <0.00%> (+0.01%)
:arrow_up:

lib/generated/intl/messages_ro.dart
95.60% <0.00%> (+0.01%)
:arrow_up:

lib/pages/timetable/view/timetable_page.dart
92.63% <0.00%> (+0.09%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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

No progress here in a long time,. closing.


Duplicate Categories displaying issue

The problem is that we have duplicate Categories displaying on our Open Source Career Portal (OSCP) – see below.  This is due to having recruiters on each of our private labels publishing jobs from the same category (which they have to do.)  What we want is to have these categories collapse into one (show only once) with the total jobs from both private labels shown (i.e., Customer Service……7; Sales…..9).  Is this possible with the Open Source Career Portal?
Here is an image of what's happening: https://www.evernote.com/l/AEgXlCME1l1KDLkaBnvJToVTWZSyT_qqos0

They are shown as separate categories because the portal displays them by category ID and not by label.   It is possible, but this would require code intervention in your portal.    What you can do is do some front end data manipulation to have an array of objects with label and values (which would be the Ids), then once a category is selected, you can do a request for all 3 categories at once.
The object would look something similar to the example in this stack overflow thread.
Please let me know if you have any more questions.

I haven't seen anywhere in the application that I can add any code though.  Can you point me to where I'd go there.  The client side is all angular so I'm not sure there is much I can do on the client side.

The change would need to be done in the client side angular for the portal.  You would be able to get started by cloning this repository

Oh duh.  Sorry, I thought this was the same code that's on the clients site.  Got it.  Thank you.

Where would I find the clients Corp Token and Swimlane for building this?

It's in the app.json file of their career portal. This is publicly accessible.  It would be at domain.com/careers/app.json


Behaviour when selecting other dates after a range is selected is strange

Hi, have you been able of doing some work here?
If not, where could be a good place to start. I just want to be able to let the user select a range, and if he clicks on any other date after a range is selected (inside or outside the range), start with a new range, and set the selected day as the beginning of the new range.


Render security.txt files in a pretty way

[x] When you click on the extension icon, the pop-out should have an HTML version of the security.txt file
When you navigate directly to a security.txt file, the extension should hijack it into an HTML version.

Example:

  Contact

<a<EMAIL_ADDRESS>src="mail.png"<EMAIL_ADDRESS>
+123456789

[x] If the directive Permission: none is set, it must be placed clearly at the top, in red. If the Permission is set, but does not have a value of none, then this security.txt file is not spec-compliant. At the top, it should be in YELLOW and specify it's value. Let's also have a warning label (⚠) for good measure
[x] Contact must be expanded by default
[ ] Make clear note that signatures are not verified.

I guess we shouldn't hijack the security.txt file when you open it, it could make verifying signatures harder.

I am happy with current levels.


Changes to support building on Windows that I have been working on

This is  a bug and feature request.

Prerequisites

[X ] Are you running the latest version?
[X ] Are you reporting to the correct repository?
(enso is made of many specialized packages: https://github.com/laravel-enso)
[X ] Did you check the documentation?
[X ] Did you perform a cursory search?

Description
composer can be fixed using:
   "platform": {
        "ext-pcntl": "7.4",
        "ext-posix": "7.4"
    }

under config part of the file which was working on some tests but when it didn't running composer install --ignore-platform-reqs always worked.
bulma-rtl.patch can be fixed by deleting the trailing line at the end and saving again. I couldn't commit so renamed after then renamed back. Tested on my Ubuntu server and the patch still applies. This fixes yarn install.
yarn build can be fixed by running different commands based on operating system using npx run-script-os. I developed this set of commands:
"build": "npx-run-script-os",
"build:default": "rm -rf ..\/public/{js,themes,themes-rtl,images} && vue-cli-service build --no-clean --modern",
"build:windows": "cd ..\public && if exist js rd /s /q js && if exist themes rd /s /q themes && if exist themes-rtl rd /s /q themes-rtl && if exist images rd /s /q images && cd client & node_modules\.bin\eslint * --fix & node_modules\.bin\vue-cli-service build --no-clean --modern --skip-plugins thread-loader",
"dev": "npx-run-script-os",
"dev:default": "rm -rf ../public/{js,themes,themes-rtl,images} && vue-cli-service build --no-clean --mode development",
"dev:windows": "cd ..\public && if exist js rd /s /q js && if exist themes rd /s /q themes && if exist themes-rtl rd /s /q themes-rtl && if exist images rd /s /q images && cd client & node_modules\.bin\eslint * --fix & node_modules\.bin\vue-cli-service build --no-clean --mode development --skip-plugins thread-loader",
Now the install steps should work on Windows hosts as soon as I manage to debug why it it is not continuing along the commands running in Windows. Anyone know why it it isn't running the next command in the list?
Steps to Reproduce
composer install fails because of pcntl or posix
yarn install fails because of line ending on the bulma-rtl patch but other patches were fine
yarn build fails as it relies on Linux style commands such as rm
Expected behavior
I expected it to fail as not been designed for Windows.
Actual behavior
It did not work.

I got the build script for yarn build to work using this on Windows:

"scripts": {
"serve": "vue-cli-service serve",
"build": "npx run-script-os",
"build:default": "rm -rf ..\/public/{js,themes,themes-rtl,images} && vue-cli-service build --no-clean --modern",
"build:windows": "cd ../public && rd /s /q js & rd /s /q themes & rd /s /q themes-rtl & rd /s /q images & cd ../client && node_modules\.bin\eslint * --fix & node_modules\.bin\vue-cli-service build --no-clean --modern --skip-plugins thread-loader",
"dev": "npx-run-script-os",
"dev:default": "rm -rf ../public/{js,themes,themes-rtl,images} && vue-cli-service build --no-clean --mode development",
"dev:windows": "cd ../public && rd /s /q js & rd /s /q themes & rd /s /q themes-rtl & rd /s /q images & cd ../client && node_modules\.bin\eslint * --fix & node_modules\.bin\vue-cli-service build --no-clean --mode development --skip-plugins thread-loader",
"lint": "vue-cli-service lint",
"postinstall": "opencollective-postinstall && patch-package",
"prod": "npm run build"
},

Full working changes can be seen at https://github.com/modularsoftware/genealogy/blob/master/client/package.json

@curtisdelicata  has it been solved?

It could be closed if you don't want it included but it might be useful for you so others can be building on Windows.


Upgrade to controller-gen 0.5.0

Supersedes https://github.com/evryfs/github-actions-runner-operator/pull/205


codegen-cli generated code does not compile - openapi-3

My spec:
https://raw.githubusercontent.com/OAI/OpenAPI-Specification/master/examples/v3.0/petstore.yaml
My command:
 java -jar ../../tools/light4j/light-codegen/codegen-cli/target/codegen-cli.jar -f openapi -o ./ -m openapi_specs/spec.yaml  -c code_gen_config.json
My config:
{
  "rootPackage": "com.home.test",
  "handlerPackage":"com.home.test",
  "modelPackage":"com.home.test",
  "artifactId": "test",
  "groupId": "com.home",
  "name": "testservice",
  "version": "1.0.0",
  "overwriteHandler": true,
  "overwriteHandlerTest": true
}

generated code for list object has multiple errors: Sample here:
 @Override
    public boolean equals(Object o) {
        if (this == o) {
            return true;
        }
        if (o == null || getClass() != o.getClass()) {
            return false;
        }
        Pets Pets = (Pets) o;

        return ;
    }

    @Override
    public int hashCode() {
        return Objects.hash(
    }

equals method does not return anything.
hashCode - syntax error
Why is this happening? I cloned the latest version from repo.

Re-tested with tag 1.5.23, but result is the same.

@ashunam Thanks for the details explanation on the issue. I have reproduced the issue you found and created a folder in model-config with all the materials to explain the details. Please review and we can continue the discussion afterward. Thanks a lot for bringing it up.
https://github.com/networknt/model-config/tree/master/example/petstore-remote


currentworkloadProfileStates - GET

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

Make sure all the checks run successfully

/azp run

@tdaroly - This PR doesn't currently show as needing ARM review, but it's probably because the submission form wasn't filled out. Please do so: you PR cannot be routed and reviewed correctly without that information. If your PR does need ARM review, be sure to study the materials linked in the form and attest to that fact by checking the box. This is required for ARM review.

@tdaroly - This PR doesn't currently show as needing ARM review, but it's probably because the submission form wasn't filled out. Please do so: you PR cannot be routed and reviewed correctly without that information. If your PR does need ARM review, be sure to study the materials linked in the form and attest to that fact by checking the box. This is required for ARM review.

@mentat9 Can you review PR and let me know if anything is missing ? if not, can we merge this ?

@mentat9 Can you review PR and let me know if anything is missing ? if not, can we merge this ?

Please work with this week's oncall (@rkmanda) to complete ARM review of your PR.

adding @rkmanda as current ARM review oncall

"/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.App/managedEnvironments/{environmentName}/authtoken": {

POST action name should contain a verb, e.g. getAuthToken

Refers to: specification/app/resource-manager/Microsoft.App/preview/2022-06-01-preview/ManagedEnvironments.json:647 in 0f2702e. [](commit_id = 0f2702e27a978452928d3c67cde8b8b1356970d0, deletion_comment = False)

      "404": {

Use default branch for all errors.

Refers to: specification/app/resource-manager/Microsoft.App/preview/2022-06-01-preview/ManagedEnvironments.json:680 in 0f2702e. [](commit_id = 0f2702e27a978452928d3c67cde8b8b1356970d0, deletion_comment = False)

        "token": {

Looks like a secret: if so, it needs to be tagged with x-ms-secret extension. Ref: https://github.com/Azure/autorest/blob/main/docs/extensions/readme.md#x-ms-secret.

Refers to: specification/app/resource-manager/Microsoft.App/preview/2022-06-01-preview/ManagedEnvironments.json:1050 in 0f2702e. [](commit_id = 0f2702e27a978452928d3c67cde8b8b1356970d0, deletion_comment = False)

@mentat9 Can you review PR and let me know if anything is missing ? if not, can we merge this ?

Please work with this week's oncall (@rkmanda) to complete ARM review of your PR.

To help load-balance, I reviewed this PR (from ARM perspective). Note however that ARM reviewers don't have permissions to approve or merge PRs. We only review the REST layer for ARM compliance. Azure SDK team owns this repo and the PR assignee is always the person from that team who handles full review, approval and merge.

@mentat9 could you pls review again ? also some of your comment's are not relevant to the changes made by my PR, we have added workitems in our board to fix those. I have used the eyes emoji to reference the one's i am talking about Thanks in advance! :)

ARM signoff is for the whole PR: we can't sign off on just part of it. That said, ARM signoff isn't required to merge changes to an aggregation branch: maybe you should proceed with your PR and wait to get ARM signoff when all the changes flow together to main.

      "404": {

@mentat9 - can you clarify what you mean by default branch? Are you saying this is redundant because of this ?
"default": {
"description": "Common error response.",
"schema": {
"$ref": "./CommonDefinitions.json#/definitions/DefaultErrorResponse"
}
}

In reply to:<PHONE_NUMBER>

Refers to: specification/app/resource-manager/Microsoft.App/preview/2022-06-01-preview/ManagedEnvironments.json:680 in 0f2702e. [](commit_id = 0f2702e27a978452928d3c67cde8b8b1356970d0, deletion_comment = False)

        "token": {

Looks like a secret: if so, it needs to be tagged with x-ms-secret extension. Ref: https://github.com/Azure/autorest/blob/main/docs/extensions/readme.md#x-ms-secret.
Refers to: specification/app/resource-manager/Microsoft.App/preview/2022-06-01-preview/ManagedEnvironments.json:1050 in 0f2702e. [](commit_id = 0f2702e, deletion_comment = False)

Please review this change please https://github.com/Azure/azure-rest-api-specs/pull/20775

        "token": {

Looks like a secret: if so, it needs to be tagged with x-ms-secret extension. Ref: https://github.com/Azure/autorest/blob/main/docs/extensions/readme.md#x-ms-secret.
Refers to: specification/app/resource-manager/Microsoft.App/preview/2022-06-01-preview/ManagedEnvironments.json:1050 in 0f2702e. [](commit_id = 0f2702e, deletion_comment = False)

Hi @mentat9 could I get review on this? https://github.com/Azure/azure-rest-api-specs/pull/20775

      "404": {

@mentat9 - can you clarify what you mean by default branch? Are you saying this is redundant because of this ?
"default": { "description": "Common error response.", "schema": { "$ref": "./CommonDefinitions.json#/definitions/DefaultErrorResponse" } }
In reply to:<PHONE_NUMBER>
Refers to: specification/app/resource-manager/Microsoft.App/preview/2022-06-01-preview/ManagedEnvironments.json:680 in 0f2702e. [](commit_id = 0f2702e, deletion_comment = False)

Yes. Recommend pattern is for error responses to be spec'd on the default response and use the ErrorResponse definition from v3 common types.

        "token": {

Looks like a secret: if so, it needs to be tagged with x-ms-secret extension. Ref: https://github.com/Azure/autorest/blob/main/docs/extensions/readme.md#x-ms-secret.
Refers to: specification/app/resource-manager/Microsoft.App/preview/2022-06-01-preview/ManagedEnvironments.json:1050 in 0f2702e. [](commit_id = 0f2702e, deletion_comment = False)

Please review this change please #20775

        "token": {

Looks like a secret: if so, it needs to be tagged with x-ms-secret extension. Ref: https://github.com/Azure/autorest/blob/main/docs/extensions/readme.md#x-ms-secret.
Refers to: specification/app/resource-manager/Microsoft.App/preview/2022-06-01-preview/ManagedEnvironments.json:1050 in 0f2702e. [](commit_id = 0f2702e, deletion_comment = False)

Hi @mentat9 could I get review on this? #20775

Yes, I reviewed and signed off that PR. @tdaroly, can you bring those changes to this PR after they merge?

Please use the instructions in this comment to facilitate ARM review: https://github.com/Azure/azure-rest-api-specs/pull/20550#issuecomment-1248546295.

@tdaroly - This comment is still unaddressed: https://github.com/Azure/azure-rest-api-specs/pull/20550#issuecomment-1251671457.
IIUC, the intent was to apply that feedback in a separate PR, get signoff and merge there and then merge those changes to this PR. If that other PR is ready for review, please provide a link here.

@tdaroly - Approving from ARM. Please in future stop using force push for swagger PRs. Swagger review is already difficult. Wiping out the commit history makes the review job nearly impossible.


Updating our fork from the base repo.

Updating our fork from the base repo.

No problem =)
Best practice is to do git remote add upsteam git://git@github.com:foundation/foundation-sites.git and then git fetch upstream && git rebase upstream develop or with merge --no-ff.


[3.x] glTF: Fix override materials and non-empty arrays

Backport of #54494 to 3.x
Keep track of MeshInstance and GeometryInstance override materials in the GLTFMesh object.
Ensure all arrays are non-empty to conform with "minItems":1 in glTF spec.
Fixes #51132

3.x test scene with fox and override materials.
Note that primitives like cubes worked before due to an unusual special case: it makes a temporary ArrayMesh and copies the override material into it if one existed.
This scene uses a non-primitive mesh, so it will produce the wrong result with 3.4rc2.
colored_fox_scene.zip

Thanks!


autolaunch!

I just found this: https://github.com/Teamwork/node-auto-launch
Haven't used it but it would be awesome if mojibar autolaunched. :rocket:

I think a slightly less intrusive alternative is to put it in your Login Items list (System Prefs > Users and Groups > Login Items). But yea, it would be a good idea for those who expect it to be autolaunched. :+1:

a good idea for those who expect it to be autolaunched

Good point. Would definitely want to make it an option for users and not a surprise. I started an experimental PR here just to see if I could get the launch part working: https://github.com/muan/mojibar/pull/50

Yay.


Immutable form

This PR provides an implementation of an immutable form using redux. All form state is encapsulated within Form and Form instances are tracked via the Manager. The Form class provides an API to manipulate form state.

@grantga @alexyuly @BradfordMedeiros Gentle reminder to review this PR.


Graduate the core.gardener.cloud API to v1

How to categorize this issue?

/area dev-productivity usability
/kind enhancement
What would you like to be added:
Let's introduce a new core.gardener.cloud/v1 API version including all resources from the core.gardener.cloud API group.
Why is this needed:
The new API version should drop deprecated fields/mechanisms, remove inconsistencies, and move to a stable API for the foreseeable future.
Steps:

[x] Bootstrap core.gardener.cloud/v1 API version: https://github.com/gardener/gardener/pull/9771
[ ] Graduate BackupBucket API
[ ] Graduate BackupEntry API
[ ] Graduate CloudProfile API
[x] Graduate ControllerDeployment API: https://github.com/gardener/gardener/pull/9771
[ ] Graduate ControllerInstallation API
[ ] Graduate ControllerRegistration API
[ ] Graduate ExposureClass API
[ ] Graduate InternalSecret API
[ ] Graduate NamespacedCloudProfile API
[ ] Graduate Project API
[ ] Graduate Quota API
[ ] Graduate SecretBinding API
[ ] Graduate Seed API
[ ] Graduate Shoot API
[ ] Graduate ShootState API
[ ] Drop core.gardener.cloud/v1beta1 API version

Please, have in mind GEP-26 will be changing at least the Shoot and Seed APIs. It may also touch other APIs that are referring to secrets with credentials.

When graduating the Quota API please consider reworking the Quota Scope to be of type QuotaScope (see example below) and not corev1.ObjectReference. Right now this reference is only used to determine the scope of the quota without actually following the referenced object (see the usage of the QuotaScope function). I propose that we support two types of scopes for the time being - "project" and "credentials". "credentials" means CredentialsBinding resource referencing a Secret or a WorkloadIdentity.
type QuotaScope string

const (
	ProjectScope     QuotaScope = "project"
	CredentialsScope QuotaScope = "credentials"
)


fix(modal): Modal Dialog doesn't fade out

This fixed the issue for me. I have tested it with Angular 1.4 and 1.3 and it seems to work for both.
It seems in Angular 1.4, the animation events are no longer triggered on elements, instead there are new functions on $animate for registering / deregistering listeners. It also seems that the "close" event doesn't exist anymore, at least I found no reference to it.
The approach to fixing the issue was to listen for a "removeClass" event. When we get one of those, we check the event phase, it needs to be "close". This seems to be about the point where the old $animate:close event would have been triggered.
Unfortunately, the $animate service does not provide functions for registering one off listeners like element.one, so I've had to manually remove the event listener in the handler.
Finally, when all that was working, I re-introduced the old mechanism for Angular 1.3 support (I assume we'll want to support both).
This fixes #3633
Hopefully that'll sort it!

This will indeed fix it. However, this might silently break users of the custom template because of the dependency on the removeClass animation event. I'll refactor the template to pass in the in class to the controller so the controller can make use of the $animate service (like $animate.addClass(inClass).then(removeModal) and avoid the need for listening to event of the end of the animation.


add toggle for dark mode

Clicking the clock already toggles notifications.
But I guess it would also be useful if it toggled Dark Mode.
Or if there was a dedicated toggle to switch between Light Mode, Dark Mode and Automatic.
On the other hand many people probably already either have it fixed or have some other way to manage it (DarkModeBuddy, scripts, hotkeys) so maybe there is no need at all.

I'm thinking of a keyboard shortcut toggling auto, dark, light modes:
First click on simple-bar, then press cmd + t and repeat the operation any time needed in order to loop over the 3 possible modes.
What do you think?

UX-wise I think it would then make sense to have a 'state' where simple-bar is selected and report that.
e.g. invert the window title in the bar when the user clicks on a blank space in the bar.
That way, settings (⌘+,) could also be triggered no matter where the bar is being clicked.

I agree, I need to find an elegant way that always work regarding the user's configuration because process/window name can be hidden in settings.
As for now I added the ⌘ + t shortchut allowing to loop through the 3 themes.

nice!
Actually I thought about toggling dark system-wide (which I think this would not do, right?)

I added a system wide toggle using the same shortcut! Let me know if this is working well :)


How do I merge this with  CodeReclaimers/neat-python master?

I feel like a complete noob for asking, but after almost a whole day of trying and still failing to figure this out I figure I'd just ask.

It's been a long time since I've worked on this repository, but the rough process should look like this (assuming you want the updates for neat.distributed):
Assuming you are using git CLI:

clone CodeReclaimers's repository (git clone https://github.com/CodeReclaimers/neat-python.git). If you already have a clone, you can use that. Beware that merges involving merge conflicts can easily mess up your code if you are inexperienced, so either use a separate branch or make a backup if you've made any changes to your clone/fork.
inside the repository directory, add this repo (git remote add bennr01 git@github.com:bennr01/neat-python.git)
checkout the branch you want to merge into (git checkout master)
fetch and merge the changes (either git pull bennr01 distributed_socket or git fetch bennr01 && git merge bennr01/distributed_socket).
As this will result in a merge conflict, you'll need to resolve that one. I believe it should be enough to take all changes to neat/distributed.py and discard all changes to .travis.yml, but I'm not sure. Once you've resolved all conflicts (beware: text files will be modifed, be sure that you've removed the various markers in the file), commit the changes git commit (then enter a message, save and quit).
Done.

The quick & dirty way
If you don't care about the correct git history or that the tests may fail, you can simply copy the neat/distributed.py file from this repository (raw version here) and replace it in the base repository. You'll need to change any use of authkey in the example to unicode.

Any chance you can fix this for me? I can offer you some money if you manage to get NEAT running in a way that allows the clients to lose connection and be able to always reconnect. Not a lot of money but money none the less

I can take a look once I've got some time, but you don't have to offer me any money for this. Simply asking would have been enough. BTW, have you set reconnect=True in the .start() method?

Yes I did try that already, I'll check it out tomorrow again im going to look through the what must already be 30 cloned repos I editted. I did also get the latest version running without socks but with reconnects (only sometimes visible)
The other times it didn't reconnect but I could see a seperate thread being spawned and removed about once a second.

okay so it is running this in linux which works, I commented out the secondary processes from the file running on linux (WSL) and then commented out the primary process on windows and with only the secondary process it does what it has to do without any weird errors.
the current behaviour is almost as desired except for the fact that when I stop debugging the windows process (secondary process) it only sometimes reconnects and sometimes it kind of reconnects (I can see threads being generated in the debugger of the primary script) but there is no useful processing being done anymore.

any eta on when you'd pick this up? If its gonna be a while I should probably look for another solution.


Skip Windows KMS in staging

Skip Windows KMS on staging. Before this change, the VM on any window-2019 image would fail in the test grid for the staging environment with a timeout, and the final message shown would be for KMS errors.
Sample local test:
./manager -project bct-staging-windows-infra -zone us-central1-staginga -compute_endpoint_override=https://www.googleapis.com/compute/staging_v1/ -images projects/gce-staging-images/global/images/family/windows-2019 -filter hostnamevalidation -local_path .
2024/10/07 18:52:49 Running in project bct-staging-windows-infra zone us-central1-staginga. Tests will run in projects: [bct-staging-windows-infra]
2024/10/07 18:52:49 using -filter hostnamevalidation
2024/10/07 18:52:49 Using compute endpoint "https://www.googleapis.com/compute/staging_v1/"
2024/10/07 18:52:49 Add test workflow for test hostnamevalidation on image projects/gce-staging-images/global/images/family/windows-2019
2024/10/07 18:52:50 Done with setup
2024/10/07 18:52:51 Storing artifacts and logs in gs://bct-staging-windows-infra-cloud-test-outputs/2024-10-07T18:52:51Z
2024/10/07 18:52:51 running test hostnamevalidation/windows-server-2019-dc-v20240913 (ID 9s3p4) in project bct-staging-windows-infra, progress: total: 1, running: 1, finished: 0, delta: 1
2024/10/07 18:58:37 finished test hostnamevalidation/windows-server-2019-dc-v20240913 (ID 9s3p4) in project bct-staging-windows-infra, time spent: 05m 46s
2024/10/07 18:58:37 cleaning up after test hostnamevalidation/windows-server-2019-dc-v20240913 (ID 9s3p4) in project bct-staging-windows-infra, progress: total: 1, running: 0, finished: 1, delta: 0

    test_utils.go:414: Test only run on linux.

    test_utils.go:414: Test only run on linux.

    test_utils.go:414: Test only run on linux.

TestCustomHostname disabled on projects/gce-staging-images/global/images/family/windows-2019

/hold
converting this to a cl


Faker::Bank.iban yields NoMethodError when locale is not english

Hi,
we're using Faker in a German app and so we have set our locale to German. Everything is working fine, except when I try to use Faker::Bank.iban. Since bank.iban_details is only implemented in en.yml this throws a NoMethodError in line 16 of bank.rb since there are no details available.
Since all those bank details are not really locale specific, would it make sense to cut those lines out and put them into a generic file that is always loaded? If so, let me know, I will create a PR.

Shouldn't it work if you have en available as a fallback?

Running via Spring preloader in process 2899
Loading test environment (Rails 5.0.1)

>> I18n.available_locales
=> [:de, :en]
>> I18n.locale
=> :de
>> Faker::Bank.iban
NoMethodError: undefined method `[]' for nil:NilClass

does not work for me. Even when I explicitly set Faker::Config.locale = :en it gives me the same error (that's really strange)
>> Faker::Config.locale = :en
=> :en
>> Faker::Bank.iban
NoMethodError: undefined method `[]' for nil:NilClass

I just experienced same thing as well, I dont think it has to do with locale. It's coming from details['bank_country_code'] in bank.rb file. I suspect details is nil at that point. I'll have a look at it and see if I can fix it

@andela-oosiname You should use a symbol for the key instead of a string to make it work.
details = iban_details.find { |country| country[:bank_country_code] == bank_country_code.upcase }

vs
details = iban_details.find { |country| country["bank_country_code"] == bank_country_code.upcase }

+1

I'm having this problem too, using the gem last version (1.8.4).
As said, the problems seems to be that calling I18n.translate with a locale key that has children keys returns a Hash with symbol keys. I didn't found any ruby or rails setup in my different projects where that call return a Hash with string keys. Here is the call that I'm talking about, that is called from here, that is called from here.
If I'm not missing something, accessing with strings to that Hash (here) should therefore return nil. Also, this PR should solve the problem.
Please @stympy, check this issue again, since it seems that #916 doesn't solve the problem. :pray:

I created [a PR] (https://github.com/stympy/faker/pull/1035) to solve this issue by avoiding Hash type altogether

Fixed via #1292 :)

@vbrazo This one can be closed too!

@deivid-rodriguez thanks 👍
I've been trying to take a look in the issues to close the ones that should have been closed but I sometimes get lost. We still have several issues 😆 (which is good. Who doesn't like solving problems here?) At least the number of PRs have decreased substantially. We had more 100 in May.

It's also my fault for not adding "Fixes #<issue_number>" to the PR body :)
Thanks for the great work you've been doing!

Actually thanks for the community who loves the gem and keeps pushing new PRs.
Adding fixes would be pretty awesome.


Troubleshooting: "You do not have permission to access this server"

I am having an issue using my own Plex server, even though I am logged in.

I did a fresh sudo docker pull wernight/plex-media-server
I started up Docker with wernight/plex-media-server on my hosted linux server
I registered and confirmed a new (free) account on plex.tv
I logged in on my server with the new Plex account
But then I got the message "You do not have permission to access this server"

Directories are writable, the container created /plex-config/Plex Media Server, I used this command:
sudo docker run -d --restart=always --user www-data --name plex -v /data/download/plex-config:/config -v /data/download:/media -p 32400:32400 wernight/plex-media-server
The Preferences.xml file was automatically created and has this content <Preferences MachineIdentifier="1**f" ProcessedMachineIdentifier="1**f" AnonymousMachineIdentifier="1**f" MetricsEpoch="1" GracenoteUser="1**f"/>
Do I need to configure something else? Does this only work with Plex premium pass?

This is how the login screen looks like, it also enables sign-up with plex.tv.

Plex version is Version 2.4.16

Plex version is 2.4.16

Here is the solution, thanks to Plex forum (and docs), maybe you want to add this to "Troubleshooting".
On a Different Network
If you're on a different network than the Server computer, you'll first need to set up a SSH tunnel so that you can access things as if they were local. This is only necessary for the initial setup.
Command line for local SSH tunnel to server: ssh -L 32400:localhost:32400 -N<EMAIL_ADDRESS>Then connect with your browser to localhost:324000 and login with your plex.tv user.
From now on you can connect directly to your remote server with your plex.tv user.

To be able to connect with phone and tablet apps to the Docker container, you need to go to Settings -> Server -> Remote Access in the web app and activate "Manually specify port", even if you use the default port 32400.

It's already in the FAQ of this docket under troubleshooting I believe.
On Mon, Aug 17, 2015, 00:33 bluepuma77<EMAIL_ADDRESS>wrote:

To be able to connect with phone and tablet apps to the Docker container,
you need to go to Settings -> Server -> Remote Access in the web app and
activate "Manually specify port", even if you use the default port 32400.
—
Reply to this email directly or view it on GitHub
https://github.com/wernight/docker-plex-media-server/issues/2#issuecomment-131636058
.

If you think the README should be clarified, please suggest improvements.


ZuulConfiguration - ZuulHandlerMapping - common interceptors from WebMvcConfigurationSupport are not added to ZuulHandlerMapping

WebMvcConfigurationSupport adds a common list of interceptors to the various HandlerMapping classes (RequestMappingHandlerMapping, viewControllerHandlerMapping, BeanNameUrlHandlerMapping, resourceHandlerMapping).  This occurs before these various handlers are initialized as they are added to the context (initialization happens when the context is set).
The ZuulHandlerMapping is created within the ZuulConfiguration.  As a result, none of the interceptors that have been setup for spring mvc are applied to it.  When the ZuulConfiguration class returns this HandlerMapping as a bean, there is now way to add interceptors to it as it is already initialized (look at the initInterceptors() method).
In the current state, the only way to make this work seems to be to redefine this bean, adding interceptors to it before initialization is done when adding it to the context.

@joshuawhite929 sorry for the delay. What functionality is missing by not having the interceptors?

Closing for lack of feedback.


Forms broken (Webinar)

Identify the issue on webinar where the form is broken :
https://www.netcentric.biz/library/future-of-customer-experience
@EduNCgit Check if it happens on other pages

I checked different pages that contain forms, I didn't find any issues present.
The page linked above is the only one affected.

It was a content issue. I moved the market form (the link) outside the column block and now is working. No more code needed


Determine how to add custom containers to IChara, IItem and similar

With #147, the way location is calculated for map objects was changed. One unresolved question is how to add custom nested containers and have the location parenting still work.
Here's what I was trying to do:

Create a shopkeeper's trunk, and put an Inventory inside its params table holding the shop's contents.
Open the trunk and receive items from it.
Expect the containing_map() of the items in the trunk to be the containing map of the parent item.

However, the trunk item has no knowledge of special things added by the params table or similar. Items can have an inv, which is an Inventory provided by the engine itself for items that "look like" containers (as in, those that have the "elona.container" category). The ILocation implementation for items delegates exclusively to this inv field, bypassing any containers we added even if their _parent fields are set correctly.
But what if we wanted to have our own custom containers in addition to inv that can hold more items, or have special behaviors like capacity/weight limits?
Admittedly, just using inv instead of making a params.shop_inventory field was the right way to go in this specific example. That's what vanilla already does, and it's as simple as it gets. But this issue did remind me that wanting custom added containers could be a legitimate use case.
If we decide we want to shoulder the potentially significant complexity this will add, we'd need some way of:

Declaring what fields are visible as containers to the overarching object.
Be able to call :iter_items() or similar on the parent item and have the iterator "see" all the items from both its built-in invand the custom modded container in params.shop_inventory or similar.
Have all the other ILocation methods cooperate with modded containers in a sane way. If there is some object inside an added container that's "registered" with a parent IItem map object, then ILocation:has_object(item) needs to return true by peeking inside not only the item's inv, but inside the other modded containers also.

A small anecdote: the one part of the original codebase that caused me to think that "maybe rewriting Elona from scratch is actually not a bad idea" was the special-case inventory logic for the cooler box. I wondered how it would be possible to mod in your own items like that with their own special container logic using the HSP codebase, but given the severe amount of hardcoding and global state surrounding how the cooler box worked with preventing items from rotting and having weight capacity, I had to wonder how much time it would take until such a cooler box would be possible to implement at all by writing a separate mod. When I couldn't find a good answer to that question, I (proverbially) threw my hands in the air and began to work on what would eventually become OpenNefia.

Perhaps a capability that implements an interface like IAspectStorage would be useful. When iterating the objects inside an aspect holder, any objects inside an aspect implementing such an interface would be joined to the main iterator.


.Values.server.config.http.port is never used

I just noticed that .Values.server.config.http.port is never used. I think the Service port is used instead, which might not be the formally correct.

I also just checked, and I agree, we should remove the http section from .Values.server.config.


Maybe discontinued? Destination server unreachable...

Yesterday, 3 hours ago and just now I tried to download the installer and the standalone. Both always fail because the target server www.open3mod.com cannot be reached... Cloud computing as you would expect or just discontinued?!

also trying it on wine doesn't work


get_world_3d().get_navigation_map() returns invalid RID

Godot version
Godot_v4.0-alpha5_win64
System information
Windows 10
Issue description
I'm trying to get the RID of the navigation map of the current world (if I understood this correctly, there can only be one World in the entire Root, someone please correct me if I'm wrong, in the debugger both nodes return the same exact ID. It shouldn't matter if I add or delete 100 sub-scenes with their own navmeshes because all point to the same World3D) because I need to call functions such as NavigationServer3D.map_get_path() that require that RID.
Steps to reproduce
Just run the project and add a breakpoint, the agent moves so it is detecting the navmesh but the RID is always Invalid

.
Minimal reproduction project
bug.zip
@AndreaCatania

Hello, thx for the issue report.
I didn't debug this but I've checked the source code.
Here the nav map is created: https://github.com/godotengine/godot/blob/master/scene/resources/world_3d.cpp#L151
Here the nav map is returned: https://github.com/godotengine/godot/blob/master/scene/resources/world_3d.cpp#L57
Navigation map is private https://github.com/godotengine/godot/blob/master/scene/resources/world_3d.h#L49 and inside World3D it's not assigned to RID() until the very end when everything is freed. So everything seems fine.
I wonder if the issue is the debugger wrongly reporting Invalid RID, I remember I saw something like this already. Can you please try to use that variable to see if the function fails?

Just tried this under the other code
	var destination_node_transform = get_node("../Destination").global_transform.origin 	var agent_transform = global_transform.origin 	var path : PackedVector3Array = NavigationServer3D.map_get_path(navigation_map_rid, agent_transform, destination_node_transform, true)
and I get an empty PackedVector3Array

Alpha 6, same result

@venilark
Can't recreate any of the issues with the MRP on recent Godot 4.0 commit 4c2a614ad41564ab33eb6515b48e0e5b52ea8b81
Returned World3D Navigation Map RID is valid and correct.
The returned Navigation Path as PackedVector3Array is also not empty and correct.
Moving the agent around also updates the path correctly.
Your last code snippet works as well.
Maybe meanwhile something changed / was fixed?

I opened the same project in Alpha 7 and still invalid RID :S

Tested with newest version again and both RID's are valid.
With your code snippet I also receive a PackedVector3Array with the full path as expected.
That said, if your issue is that the path is empty in the agent _ready() function that is expected and should be added as part of the documentation that does not exist rightnow.
The reason is that both NavigationRegion Nodes and your NavigationAgent need to register with the NavigationServer first and the NavigationServer has its own syncphase on a thread for updates that runs each frame. If you query the path in _process() it is not empty cause the NavigationServer has been updated / received its first update tick but if you query in any _ready() it still has not updated cause the SceneTree is still updating childnodes. The virtual navigation map is empty at that point so the path returns empty. Please test / confirm if this is the source of this issue.

Tested with newest version again and both RID's are valid. With your code snippet I also receive a PackedVector3Array with the full path as expected.
That said, if your issue is that the path is empty in the agent _ready() function that is expected and should be added as part of the documentation that does not exist rightnow.
The reason is that both NavigationRegion Nodes and your NavigationAgent need to register with the NavigationServer first and the NavigationServer has its own syncphase on a thread for updates that runs each frame. If you query the path in _process() it is not empty cause the NavigationServer has been updated / received its first update tick but if you query in any _ready() it still has not updated cause the SceneTree is still updating childnodes. The virtual navigation map is empty at that point so the path returns empty. Please test / confirm if this is the source of this issue.

thank you, I added the code in _process() and now I am getting the path, although I still get INVALID RID for some reason, I'm really confused about it, maybe a debugger error? the RID has to be right otherwise map_get_path() wouldn't work
sorry but what do you mean by "query the path in _process()"? for what I have in mind, I need to call several map_get_to() to different nodes and do it as soon as possible, a I expected it to work in _ready() but if that's not possible how do you make sure when you call it, the server has already set everything up as you said?

@venilark
If you mean the remote debugger displays in the Editor with 'INVALID RID' that is "normal". Try printing your RID variables with rid.get_id() and you will see that everything works fine and it is just a display issue. When you test your game from the editor Godot starts a new instance of Godot to playtest your game. Since the RID is only valid in this instance it is displayed as 'INVALID RID' in the editor instance.
When the SceneTree is build from nodes it takes some time to register all Navmesh regions and Agents to the Navigation Map since the NavigationServer runs on a different thread. One way to deal with this is to use a bool toggle in _process() so as soon as a path is available it runs once.
var waiting_for_navigationserver_sync : bool = true
func _process(delta):
    # Arrays will always return empty in _ready() and on first frame(s)
    # NavigationRegions and NavigationAgents queue up to register with threaded NavigationServer in _ready()
    # Afterwards NavigationServer needs a physics tick to sync regions and agents and flush queue
    var map : RID = get_world_3d().get_navigation_map()
    var path : PackedVector3Array = NavigationServer3D.map_get_path(map, start_pos, target_pos)
    if waiting_for_navigationserver_sync  and not path.is_empty():
        waiting_for_navigationserver_sync = false
        # do your _ready() navigation stuff

@venilark
If you mean the remote debugger displays in the Editor with 'INVALID RID' that is "normal". Try printing your RID variables with rid.get_id() and you will see that everything works fine and it is just a display issue. When you test your game from the editor Godot starts a new instance of Godot to playtest your game. Since the RID is only valid in this instance it is displayed as 'INVALID RID' in the editor instance.
When the SceneTree is build from nodes it takes some time to register all Navmesh regions and Agents to the Navigation Map since the NavigationServer runs on a different thread. One way to deal with this is to use a bool toggle in _process() so as soon as a path is available it runs once.
var waiting_for_navigationserver_sync : bool = true
func _process(delta):
    # Arrays will always return empty in _ready() and on first frame(s)
    # NavigationRegions and NavigationAgents queue up to register with threaded NavigationServer in _ready()
    # Afterwards NavigationServer needs a physics tick to sync regions and agents and flush queue
    var map : RID = get_world_3d().get_navigation_map()
    var path : PackedVector3Array = NavigationServer3D.map_get_path(map, start_pos, target_pos)
    if waiting_for_navigationserver_sync  and not path.is_empty():
        waiting_for_navigationserver_sync = false
        # do your _ready() navigation stuff

thank you, I thought it would be some kind of signal but it's just a simple bool check and it just works


Integrate NuGet.Server build with VSTS

The necessary changes to run this build on VSTS instead of TeamCity, which includes moving some scripts from TeamCity to this repository.

Hi @scottbommarito, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


[Several docs] Fix broken links and anchors

Fixes some broken links (supported by redirects, though) and broken anchors.

Before:
Found 762 bad links
Found 100 bad anchors

After:
Found 765 bad links
Found 74 bad anchors

(Added 3 new API links, which are all considered broken.)


cannot invoke sam local outside c:\users

Hi ,
I am able to execute the sam local invoke command successfully when my projects files are under c:\users . However  when I change my project directory to other than  c:\users e.g d:\workdir1 , sam local invoke  command does not work . It errors out as listed below.
I tried using the mounting solution mentioned at this link . http://support.divio.com/local-development/docker/how-to-use-a-directory-outside-cusers-with-docker-toolbox-on-windows
Even after  mounting my shared folder , sam local invoke does not work outside c:\users
Please not that I am on windows 7 64 bit and I am using docker toolbox
$ sam local invoke  "sks1" -e event.json
2018/01/07 09:43:47 Successfully parsed template.yaml
←[34mINFO←[0m[0001] Unable to use system certificate pool: crypto/x509: system root pool is not available on Windows
2018/01/07 09:43:47 Connected to Docker 1.35
←[34mINFO←[0m[0001] Unable to use system certificate pool: crypto/x509: system root pool is not available on Windows
2018/01/07 09:43:47 Fetching lambci/lambda:nodejs6.10 image for nodejs6.10 runtime...
nodejs6.10: Pulling from lambci/lambda
Digest: sha256:e34f92bc0df0cf4a8ba560c6c7bf201183d5f6773ddf44440a97295486906744
Status: Image is up to date for lambci/lambda:nodejs6.10
2018/01/07 09:43:51 Invoking index.handler (nodejs6.10)
2018/01/07 09:43:52 WARNING: No AWS credentials found. Missing credentials may lead to slow startup times as detailed in
l/issues/134
2018/01/07 09:43:52 Mounting /d/workdir1/sam-local1 as /var/task:ro inside runtime container
←[32mSTART RequestId: 7a5aa8d7-e643-17f7-5e86-5dfeedad0d2d Version: $LATEST←[0m
←[31mUnable to import module 'index': Error←[0m
←[31m    at Function.Module._resolveFilename (module.js:469:15)
at Function.Module._load (module.js:417:25)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)←[0m
←[32mEND RequestId: 7a5aa8d7-e643-17f7-5e86-5dfeedad0d2d←[0m
←[32mREPORT RequestId: 7a5aa8d7-e643-17f7-5e86-5dfeedad0d2d     Duration: 7.33 ms       Billed Duration: 0 ms   Memory S
←[0m
{"errorMessage":"Cannot find module '/var/task/index'","errorType":"Error","stackTrace":["Function.Module._load (module.
97:17)","require (internal/module.js:20:19)"]}

This should work when you setup the Docker container file sharing settings properly.


Add exit code as output

Sets exit_code per the cli docs as follows:

0 if no planned changes
1 on any error (not exclusively related to tf planning)
2 planned changes

This will be useful to format the input for a github check in following workflow steps.
The exit code is a bit tricky. I don't think we're losing anything meaningful by wrapping action errors in with plan errors and apply errors. The thought process here is that if we scope the exit code to just the tf plan then the output itself becomes unreliable, as it might not be set in all cases, which would make subsequent steps more complex. This way the output should always be set and we can depend on its presence.

mmm yeah good point regarding already having this information. Closing...


add optional name for scenario in molecule schema

Looking at the molecule schema it suggests that name is optional, but
can be defined and should be given for non-default scenarios
See: https://github.com/ansible-community/molecule/blob/master/src/molecule/model/schema_v3.py
This also fixes the schema for sequence names. Before this commit, the following didn't make the linter scream:
---
scenario:
  syntax_sequence22222:
    - cleanup

Now it should only allow the defined sequence names.

mhm, that's weird. npm test succeeded on my machine.

In general it looks ok but we need to pass all GHA actions.

I rebased onto the current main. Will now look into why this GHA action fails. The output of npm test could be a bit more verbose.

Ok. Seems like it fixed itself. I honestly have no clue why, though. ¯_(ツ)_/¯


[Android] The first Map cover the second Map

Version

react-native-router-flux v3.31.2
react-native v0.30.0
react-native-maps v0.7.1

Hi,
i have a screen with a not draggable map and another screen with a draggable map where the user can set the marker.

But the first map cover the second map...have you idea?

Hey, please upgrade to 0.8 , should be resolved
but you need to upgrade to RN 32 too

Now i have:
react-native-router-flux v3.31.2
react-native v0.32.0
react-native-maps v0.8.0
but i have the same issue

Well, I ran into the same issue. See my flow of components below
First I opened - MapComponent from Drawer Layout,
Second I opened - OtherComponent from Drawer Layout,
Third I opened - MapComponent from Drawer Layout,
When I opened MapComponent in third step, I get the same issue with one map being static and other above it.

aksonov/react-native-router-flux#1149

Same
react-native v0.32.0
react-native-maps v0.8.0

Tracking this in #207 and #453. Closing this now for keep track in those issues.


cmd/go: -mod flag doesn't work with module-mode go get

What version of Go are you using (go version)?

$ go version

go version go1.16 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/subhamsarkar/Library/Caches/go-build"
GOENV="/Users/subhamsarkar/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/subhamsarkar/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/subhamsarkar/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.16/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.16/libexec/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.16"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/subhamsarkar/project/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/wz/k505z2bd7rnbbv_611swlj4h0000gp/T/go-build2354586914=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Inside a module, go get didn't accept the mod flag which is against what's mentioned in https://golang.org/ref/mod#build-commands

...
When run in module-aware mode, these commands use go.mod files to interpret import paths listed on the command line or written in Go source files. These commands accept the following flags, common to all module commands.
...

So, I was just playing with go commands based on what's written in the doc and I noticed that -mod works will all module commands except for go get.
$ go get -mod=mod ./...

What did you expect to see?
As per documentation, -mod must work with go get just like it works for go {build,fix,generate,install,list,run,test,vet}
What did you see instead?
$ go get -mod=mod ./...
flag provided but not defined: -mod
usage: go get [-d] [-t] [-u] [-v] [-insecure] [build flags] [packages]
Run 'go help get' for details.

CC @bcmills, @jayconrod, @matloob.

The purpose of go get in module mode is to edit the dependencies of the module. For that purpose, the only sensible -mod behavior is -mod=mod — -mod=readonly or -mod=vendor would be incoherent.
I intentionally removed the -mod flag from go get in CL 198438 so that it could be set in GOFLAGS without causing errors in go get. (See #30345 and #32502.)

So, probably the thing to fix here is to not the different behavior for go get in https://golang.org/ref/mod#build-commands.

So, should I send a CL for this?

Sure!

I have happened to meet the same issue.
If @shmsr doesn't mind, I will make a CL on documentation in a few days.

I have happened to meet the same issue. If @shmsr doesn't mind, I will make a CL on documentation in a few days.

Really sorry. Totally forgot about this. I'll send a CL ASAP.


dear author, dir name  (raftkitti_kitti  or  raft_kitti)

Is that both to be the "raft_kitti".
By the way ,the raft weights can not be load.
Really thanks.

Yes, they should be "raft_kitti". Thanks for giving me a hand for eliminating the annoying typos :(
Could you provide the error message of failing to load the raft weights?

Closing the issue due to inactive status. Feel free to re-open if necessary :)


[Feature] Hosted Fields component

🚀 Feature Proposal
It'd be great to have a hosted fields component and support: https://developer.paypal.com/docs/business/checkout/advanced-card-payments/#
Currently, you cannot set the client-token as an option on the PayPalProvider. So even trying to write a custom component will not work.
Motivation
We want to embed hosted fields
Example
We've implemented this for the Drupal Commerce PayPal integration and are adding a headless example.

Oh, just because it isn't in the props types didn't mean it was supported.
      <PayPalScriptProvider options={{
        'components': 'hosted-fields,buttons',
        'client-id': process.env.REACT_APP_PAYPAL_SB_CLIENT_ID,
        'data-partner-attribution-id': 'CommerceGuys_Cart_SPB',
        'data-client-token': process.env.REACT_APP_PAYPAL_SB_CLIENT_TOKEN,
      }}>

With this we can detect if eligible and use a hosted field setup or fallback to smart buttons
  const [{ isLoaded, options }] = usePayPalScriptReducer();
  const [supportsHostedFields, setSupportsHostedFields] = useState(null);

  useEffect(() => {
    console.log(options);
    if (isLoaded) {
      setSupportsHostedFields(window.paypal.HostedFields.isEligible())
    }
  }, [isLoaded])

@mglaman thanks for opening up this issue. You are correct that data-client-token should work fine even though it's missing from the prop-types. Thanks for pointing that out. I added it to the prop-types in PR #51.
Hosted-fields should definitely be possible using the custom component approach you outlined in https://github.com/paypal/react-paypal-js/issues/49#issuecomment-718198987.
Another thing to note is the upcoming 2.0 release drops the isLoaded option and replaces it with isPending, isResolved, and isRejected. Ex:
const [{ isResolved, options }] = usePayPalScriptReducer();
  const [supportsHostedFields, setSupportsHostedFields] = useState(null);

  useEffect(() => {
    console.log(options);
    if (isResolved) {
      setSupportsHostedFields(window.paypal.HostedFields.isEligible())
    }
  }, [isResolved])

I'm going to close this one out. Please re-open if you encounter any problems with the hosted-fields implementation.

Hey @gregjopa
Thank you for your great work for react community.
Reading through this issue, it looks like the data is being pulled from a environment variable, that looks wrong to me, after reading the API.
E.g.

        'data-client-token': process.env.REACT_APP_PAYPAL_SB_CLIENT_TOKEN,

The API appears to require a new client_token per time the component gets loaded, a token that has to be generated with the access_token, one acquires with the API credentials.

(source)
To me it makes sense to store the API credentials on the server, and use those to return the client_token to the SPA.
Is this incorrect?
Again, thanks for your work. I am sure I'll have more questions as I too am very interested in having hosted fields.

Hi @getTobiasNielsen, I think an environment variable was just an example. The key thing is this is a sensitive runtime value that you should not check into version control with your client-side code.

A client token is required to uniquely identify your buyer.

The docs mention that this client token is buyer specific. So you will need a new value every time you show the hosted fields component to a buyer. The more I think about it, adding a <PayPalHostedFields> component that plays nice with the Provider would be a great addition to react-paypal-js.
For now, you could try this:
Step 1 - wrap your app with the PayPalScriptProvider and pass the static param values that typically don't change often. Do not include a data-client-token here:
// static params that are passed to the ReactPaypalProvider
const scriptProviderOptions = {
    "client-id": "test",
    "components": "buttons,hosted-fields"
};

// wrap your app with this
<PayPalScriptProvider options={scriptProviderOptions}></PayPalScriptProvider>

Step 2 - Create a hosted fields component that makes an ajax call to fetch the data-client-token value. I haven't tested this but wanted to share some initial thoughts regarding some of the logic needed:
const [{ options }, dispatch] = usePayPalScriptReducer();
const [clientToken, setClientToken] = useState("");

// use fetch or axios or whatever to make a client-side call
// to your server-side endpoint that wraps the /identity/generate-token endpoint
fetch("your/server/endpoint/for/generating/buyer/session-token")
    .then(response => response.json())
    .then(data => {
        setClientToken(data.clientToken);
        // pass client_token value to the data-client-token attribute of the JS SDK
        dispatch({
            type: "resetOptions",
            value: {
                ...scriptProviderOptions,
                "data-client-token": data.clientToken,
            },
        });
    });

 useEffect(() => {
    if (options.isResolved && clientToken) {
      setSupportsHostedFields(window.paypal.HostedFields.isEligible())
    }
  }, [isResolved, clientToken])

@gregjopa  Thank you for the speedy answer!
Passing the data-client-token seems to be working. Nice catch with the dispatch, I wasn't aware of how this library worked, but I see usePayPalScriptReducer has a lot of use.
I'll return once I've had a chance to look at the hosted fields.
Thanks again!

@gregjopa window.paypal.HostedFields.isEligible() seems to return true now.
It appears isResolved isn't defined in options, but can be grabbed on its own from usePayPalScriptReducer, like in your original example.
I was thinking about what you said about a <PayPalHostedFields>. I'd love to get involved in that, do you have any initial thought about how you would want it to look?
I was considering a PaypalHostedFields wrapping around the card / billing address fields, that has some internal logic, and error handling similar to that of PayPalButtons.
It would be able to accept things like createOrder, maybe styling or fields
It would also render things like the following unnecessary, as that could be done internally, and even give an error if the component was being used, but was missing from components: "buttons,hosted-fields"
useEffect(() => {
    console.log(options);
    if (isLoaded) {
      setSupportsHostedFields(window.paypal.HostedFields.isEligible())
    }
  }, [isLoaded])

Here I'm wondering if, it makes sense to make more components, e.g. like a cvv component, because that makes it a lot less customizable. While it might work with cvv, even that could be required to be displaying cvc. This problem only gets harder with fields like extendedAddress, because that is very much based on localization.
<PaypalHostedFields createOrder= {} >
   // card fields here
   // billing address here
</PaypalHostedFields>

Open to hearing what you think about this approach, where it's only a wrapper, and the html inside is left uncontrolled and referenced to with refs instead of id.

Hi @getTobiasNielsen, it's good to hear your got Hosted Fields working! 🎉

It appears isResolved isn't defined in options, but can be grabbed on its own from usePayPalScriptReducer, like in your original example.

Whoops, my mistake. It's not defined in the options (those are just for the params/attributes passed to the JS SDK script). I fixed it in the code snippet above.

Open to hearing what you think about this approach, where it's only a wrapper, and the html inside is left uncontrolled and referenced to with refs instead of id.

I like that idea! I would like to keep the <PayPalHostedFields> as light weight as possible. Supporting custom content as props.children seems like a good idea. Would you be up for working on a prototype for this?

I'll make a prototype.
From my initial search, I see that the JS SDK doesn't have types for hosted-fields, so will see what I end up with.

A lot of the types are missing, would need to implement those, either in the file or in their respective places.
This is an initial prototype.
I was thinking it could be smart to add one of the default spinners I've seen paypal use, or if people don't like it, let people add a prop with a spinner element or some custom react component, if they fancy that instead.

Hey - is there any updates here? Trying to implement custom designs on payment fields, this would be pretty helpful


Problem with value read

Hi, I m using SI1145 exactly like the tutorial. but the sensor always shows 260, 254, 0.01 for Vis, Ir, and UV index respectively although I give it a flashlight and keep it in a dark place. Is this need calibration or is my sensor broken?

same - did you solve the problem?

it was my fault. sorry. the flash light I used back then did not have enough energy maybe it is my smartphone flash light. When I used emergency lamp with shape tube, the value increased. Moreover when the sensor is under direct sunlight, the value increased higher.


Tables with "," in cells are copied incorrectly

EPAM AI DIAL chat version
0.6
How to reproduce

Ask the model to write you a table, with commas in the cells. (Prompt example: Top 10 IT companies. Send me an MD table. Add ", Inc" to each company).
Copy the table in CSV format.
Paste into excel/google sheets -> more columns -> bug

Actual result
When copying a table in csv format, if there are commas in the cells, extra columns will appear.
Expected result
The cells are displayed as they should be.
Confidential information

[X] I confirm that do not share any confidential information

Verified on staging successfully.
Now if to paste CSV to Excel/google and continue with Text Import Wizard with Comma as delimiter, values separated by comma are added into one column.


Open Pedalboard Platform

Hi
I am working for a while on a similar project: https://github.com/pedalboard
Part of the project is a midi controller, which is based on specific firmware written in Rust.
Now i want to make this firmware more generic. If possible, I would like to reuse your firmware and create a custom configuration for my hardware.
But my hardware has some specific feature, which do not seem to be supported by your firmware:

Special LED output (LED rings) based addressable RGB LEDs.
Two OLED screens (SSD-1327), which I would like to show some configuration values for each button.

Therefore I would probably need to extend your firmware to support this specific hardware. How are you storing the configuration? Are you supporting an eeprom in your firmware?
An alternative would be to implement your configuration protocol on my firmware (instead of my own).
Are you interested in a collaboration?
You can contact me on Discord https://discord.gg/ncyKyryHAc
Christof

I have started to implement the protocol in rust: https://github.com/pedalboard/opendeck
Its of course in a very prototype state. I also found some inconsistencies between your UI implementation and your docu in the wiki

BTW: My goal is in now way to compete with your platform. I want to reuse your platform in my project which focuses on audio processing but requires some IO, where I want to reuse parts of your platform.


Not able to fetch WSDL

Following URL "http://www.webservicex.com/stockquote.asmx?WSDL" is giving Error:
"The current identity (NT AUTHORITY\NETWORK SERVICE) does not have write access to 'C:\Windows\Microsoft.NET\Framework\v4.0.30319\Temporary ASP.NET Files'."

Grrr. It is becoming harder and harder to find any public APIs that offer SOAP-based endpoints (everyone switched to JSON). This one is flakey, it seems, so maybe we'll have to write our own. Pull requests welcome, if anyone can find one that works reliably.

Works for me now. Maybe a temporary glitch.


Windows 10 screen capture raises UnparsableClipboardFormatException

Hi there!
I think this is a really great tool and I am excited to use it for engineering practice where it's common to copy-paste screen grabs of analysis models.
I install pyclip and immediately tried to use it with a screen capture only to be greeted with the following:
---------------------------------------------------------------------------
UnparsableClipboardFormatException        Traceback (most recent call last)
Input In [15], in <cell line: 1>()
----> 1 nb.paste.paste_image()

File ~\Notebooks\package_development\nbengineer\nbengineer\paste.py:31, in paste_image()
     29 immediately in the current Jupyter notebook.
     30 """
---> 31 clipboard_data = pyclip.paste()
     32 first_eight_bytes = clipboard_data[:8]
     33 image_type = _identify_image_type(first_eight_bytes)

File ~\Anaconda3\lib\site-packages\pyclip\__init__.py:42, in paste(*args, **kwargs)
     40 if DEFAULT_CLIPBOARD is None:
     41     raise ClipboardSetupException("Could not setup clipboard").with_traceback(_CLIPBOARD_EXCEPTION_TB)
---> 42 return DEFAULT_CLIPBOARD.paste(*args, **kwargs)

File ~\Anaconda3\lib\site-packages\pyclip\win_clip.py:218, in WindowsClipboard.paste(self, encoding, text, errors)
    216 acceptable_formats = [f for f in all_formats if f in self._implemented_formats]
    217 if not acceptable_formats:
--> 218     raise UnparsableClipboardFormatException("Clipboard contents have no standard formats available. "
    219                                     "The contents can only be understood by a private program")
    220 if (text or encoding or errors) and format not in self._string_formats:
    221     raise ClipboardNotTextFormatException("Clipboard has no text formats available, but text options "
    222                                           "were specified.")

UnparsableClipboardFormatException: Clipboard contents have no standard formats available. The contents can only be understood by a private program

I was hoping that with all of the additional clipboard formats available on Windows that it would be quite flexible. Is this something that is just a small fix or is there something funny going on with Windows and their proprietary formats?
Thanks for any time you can spend on this!

Thanks for raising this issue. Right now, pyclip supports a narrow set of clipboard formats. Essentially, just text formats and hdrop (file copy/paste).
It seems you have copied and subsequently tried to paste through the pyclip program a format that is not supported.
I guess I would wonder: (1) what format(s) is stored when screen grabs are used to populate the clipboard and (2) what should the behavior of .paste do when dealing with these format(s).

Hi Spencer,
Thanks for the follow-up. In response:

I think screen grabs are stored in png format.
I think the behaviour of .paste should be to return the full binary
"blob" so it can be assigned to a variable. It seems that pyclip does
some inspection and validation of the binary data in addition to retrieval,
which makes sense too.

My ambition is to use pyclip to capture image data in the clipboard and
save them to disk from Jupyter which requires knowing what format the image
data is in.
The code I am working with checks the first 4 - 8 bytes of the blob against
the GIF, JPG, PNG signatures (as found in
https://en.wikipedia.org/wiki/List_of_file_signatures) so it assumes that
there is no additional header information "wrapping" the image data. So, if
no such "wrapping" exists on the clipboard data then perhaps adding a
raw=True flag to .paste could be an option to by-pass any validation?
On Thu, Mar 31, 2022 at 10:53 PM Spencer Phillip Young <
@.***> wrote:

Thanks for raising this issue. Right now, pyclip supports a narrow set of
clipboard formats. Essentially, just text formats and hdrop (file
copy/paste).
It seems you have copied and subsequently tried to paste through the
pyclip program a format that is not supported.
I guess I would wonder: (1) what format(s) is stored when screen grabs are
used to populate the clipboard and (2) what should the behavior of .paste
do when dealing with these format(s).
—
Reply to this email directly, view it on GitHub
https://github.com/spyoungtech/pyclip/issues/14#issuecomment-1085453655,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJFJEEJTCI4WSMXWOWU5WM3VC2FNLANCNFSM5SHSLV4Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi @spyoungtech,
After reading your source, and some of the Windows clipboard documentation, it seems that when one calls .GetClipboardData(), a person has various options available to them on what format they would like their data in. After reviewing the various formats available, I used a print() call to see what format a screen capture comes out with: 49161.
It seems that this clipboard format code is something of a generic code to refer to a DataObject. For screenshots, it seems that format codes 8: 'CF_DIB' and/or 17: 'CF_DIBV5' are the codes that return the byte arrays for the bitmap image.
I added an argument to .paste() called format_override which allows for the user to specify a custom Windows format code to get the data in the format required. The new .paste() code would look like this:
    def paste(self, encoding: str = None, text: bool = None, errors: str = None, format_override: int = None) -> Union[str, bytes]:
        """
        Returns clipboard contents

        :param encoding: same meaning as in ``bytes.encode``. Implies ``text=True``
        :param text: if True, bytes object will be en
        :param errors: same meaning as in ``bytes.encode``. Implies ``text=True``.
        :param format_override: an attempt is made to find the format of the clipboard data internally but sometimes,
                    on Windows, this format is incorrect. Allows user to specify a format to return the data in.
                    See: https://docs.microsoft.com/en-us/windows/win32/dataxchg/standard-clipboard-formats
                    Use format code 8 for bitmap screen captures.
        :return: clipboard contents. Return value is bytes by default
            or str if any of ``encoding``, ``text``, or ``errors`` is provided.
        """
        with self._clipboard as clip:
            if format_override:
                format = format_override
            else:
                format = clip.EnumClipboardFormats()
                if format == 0:
                    if text or encoding or errors:
                        return ''
                    else:
                        return b''

...

This byte data can be passed through a BytesIO into PIL.Image to then manipulate the image:

So, it seems that it is possible to get screen capture data using pyclip but, unfortunately, the clipboard format enum that is used by default, 49161 is ambiguous and requires the user to then specify what format they would like the bitmap data in.
In my local copy, I created this format_override which would allow me to get the functionality I need but then I would be vendoring pyclip into my own little utility package which is not ideal.
Let me know your thoughts on this. If you are interested in being able to work with Windows screen captures, then perhaps

I will try playing around with it locally to see if I can get it to work. If so, I will submit a draft PR for discussion.
Sent from my iPhone

On Apr 1, 2022, at 09:08, Connor Ferster @.***> wrote:

Hi Spencer,
Thanks for the follow-up. In response:

I think screen grabs are stored in png format.
I think the behaviour of .paste should be to return the full binary "blob" so it can be assigned to a variable. It seems that pyclip does some inspection and validation of the binary data in addition to retrieval, which makes sense too.

My ambition is to use pyclip to capture image data in the clipboard and save them to disk from Jupyter which requires knowing what format the image data is in.
The code I am working with checks the first 4 - 8 bytes of the blob against the GIF, JPG, PNG signatures (as found in https://en.wikipedia.org/wiki/List_of_file_signatures) so it assumes that there is no additional header information "wrapping" the image data. So, if no such "wrapping" exists on the clipboard data then perhaps adding a raw=True flag to .paste could be an option to by-pass any validation?

On Thu, Mar 31, 2022 at 10:53 PM Spencer Phillip Young @.***> wrote:
Thanks for raising this issue. Right now, pyclip supports a narrow set of clipboard formats. Essentially, just text formats and hdrop (file copy/paste).
It seems you have copied and subsequently tried to paste through the pyclip program a format that is not supported.
I guess I would wonder: (1) what format(s) is stored when screen grabs are used to populate the clipboard and (2) what should the behavior of .paste do when dealing with these format(s).
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.


ILTermini 6

checked Latin text and edition abbreviation

ISic0183
Do you mean: <gap reason="lost" evidence="previouseditor" atLeast="5" unit="character">V</num>(because I can put 'atMost' in this case

*can't, sorry


Regarding Deprecation notice: QS8 and QS16 data types

Hi
It was mentioned in your home page : "Deprecation notice: QS8 and QS16 data types will be removed in the 18.05 release".
Does this mean that the int8, in16 data formats for Deep learning layers such as Convolution would not be supported?
It would be great if could clarify on where the QS8/QS16 is used and its impact on the int8/int16 data formats for the DL layers.
Thanks in advance
Kamrao

This is only the fixed point 8bit and 16bit (i.e QS8 / QS16), we are keeping QASYMM8 (Asymmetric 8bit integer with scale / offset)
As far as we know QS8/QS16 are not used by any of the major ML frameworks.
Moreover in their current state they're quite slow and really expensive to validate (They represent roughly 70% of the validation time in our unit tests) which is why we're planning on removing them in the not so distant future.
Could you please explain what your concerns are ? Are you making use of them ?

What do you call "Quantized" ? This is not a type in the Compute Library.
The plan is to remove all of the QS8 / QS16 functions & kernels (Basically QS8 and QS16 won't be types anymore)

Hi
By Quantized, I  mean to say the layers such as Convolution using 8 bit/16 bit weights instead of FP32.
Are the 8 bit, 16 bit operations still supported in the Deep learning layers? Please clarify.
Thank you
kamrao

As explained previously: fixed point 8bit and 16bit (QS8 / QS16) is what has been deprecated and will be removed soon.
Quantised 8bit using asymmetric integers (QASYMM8) will stay, and we'll keep optimising it.
Which one are you talking about? What are your concerns?

Hello,
As mentioned in the doc, NEActivationLayer does not support QASYMM8. If QS8/QS16 will be removed, How can I implement ActivationLayer(like QuantizedRELU in TF) with NEON?
Many thanks.

Hi @ymbaek,
This is just a documentation glitch, if you look at the impletementation you will see QASYMM8 is actually supported.


[Android]automatically  import react-native 0.20.1

my react-native version is 0.48.3
but when I compile react-native-blur,react-native have tow version 0.48.3 and 0.20.1.

how to resolve it

I think you need to add
allprojects {
    repositories {
        mavenLocal()
        jcenter()
        maven {
            url "https://maven.google.com"
        }
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url "$rootDir/../node_modules/react-native/android"
        }
    }
}

to your root build.gradle


Project Not Working after upgrading to ng 11.1.4 and module federation 1.2.3

I add an old project (PRJ1) which was working. This project is in Angular 11.0.7 and module federation 1.2.2
I create a new project (PRJ2) with the last version of Angular (11.1.4) and module federation 1.2.3.
This project doesn't start and doesn't build.
I have updated PRJ1 to 11.1.4 and have the same problem.
I downgraded PRJ2 to 11.0.7 and it works !!! (i have used the working package.json from PRJ1).
package.json files that is working :
package.working.json.txt
package.json files that is not working :
package.notworking.json.txt

I have upgraded the working project (angular 11.0.7) to module federation 1.2.3. And it is working.
There seems there is an incompatibility between module federation 1.2.3 and angular 11.1.4.

@manfredsteyer There's a bug with<EMAIL_ADDRESS>which causes this issue and application becomes unbuildable.
The author of license-webpack-plugin fixed this bug in version 2.3.13 as he said in this issue :
https://github.com/xz64/license-webpack-plugin/issues/95
Appreciate if you update this dependency on module federation.

@Al00X please could you post the correct link to the issue of license-webpack-plugin? Thanks

Thanks for this info. Let me check this asap. Hoewever, I also had the impression that there is an issue with Angular 11.1 and Module Federation. With 11.0 everything is fine and with 12.x everything should be fine too, b/c it will contain a stable webpack5 integration.

It's also resolved for version 12 of this lib


Excel header and footer (not header row)

Hi
Is it possible to retrieve data from the Header and Footer of an Excel document?  I've been able to extract data from the rows/columns of the main body of the document but can't find a way to access the header/footer
Thanks
Martin

No, but I don't think it would be very difficult to provide the raw text. Interpreting the commands is imo not something that this library can provide.
@andersnm, we could just add a Header and Footer property to IExcelDataReader as we do with the sheet name and visible state, or do you see any reason to start with some kind of sheet metadata object?

Properties are fine with me.

Not so simple after all. In xlsx you can have different headers and footer on the first page and/or on even and odd pages. So I guess we need a class that exposes this instead of multiple properties on IExcelDataReader.
public class XlsxHeaderFooter
{
  public bool HasDifferentFirst { get; }
  public bool HasDifferentOddEven { get; }

  public string FirstPage { get; }
  public string EvenPage { get; }
  public string OddPage { get; }
}

Should FirstPage throw if HasDifferentFirst is false? What about HasDifferentOddEven and EvenPage and OddPage? We could mirror Excel and always populate OddPage and throw in EvenPage when HasDifferentOddEven is false or both could have the same value in that case.
Perhaps simpler to use would be to always populate FirstPage, EvenPage and OddPage and just return the same value for all of them when HasDifferentFirst and/or HasDifferentOddEven  is false.
Opinions?

Doesn't matter too much, but I think the simplest approach without throwing is preferable as long as the booleans' documentation indicates which strings are valid when. Unused strings could be null too.