Dialogue to show all keyboard shortcuts

Summary of the Pull Request
Adds a dialogue that shows all keyboard shortcuts. If I'm missing anything, feel free to share!

References
Connected to #4695.

PR Checklist

[X] Closes #4695
[X] CLA signed. If not, go over here and sign the CLA
[X] No documents
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments
A new dialogue launched by clicking on a new tab menu item.

Wait, why are there commits unrelated to this?

@zadjii-msft Might I ask why you believe user defined keybindings? I'd look at it like a quick way to show default keybindings, to "learn and discover" what one can do. If one wants to see all keybindings it is always possible to open the settings.json file.

@zadjii-msft, thus I was asking how to get data from json file to xaml.

Maybe we could do a data context?

So, I have here a binding that can possibly work. Problem is, I don't know what to put next in the if statement:
winrt::hstring TerminalPage::CopyKeybinding()
    {
       auto keyBindings = _settings->GetKeybindings();
        auto copyKeyCord = keyBindings.GetKeyBindingForAction(ShortcutAction::CopyText);
        if (copyKeyCord)
        {
       }
        return RS_(L"NoCopyData");
    }

Another thing, how do I get it so that it can be displayed in const auto form?

Well, right now, I just can't give it to display based on settings, but what do you say we implement the default keybindings for now and we can work on getting it to change based on the settings.json file later on?

If there're profile-specific keybindings supported in future, this dialog should also take some decisions about how to react/display to this specificity

So is there a way to get the keybinding/profile into string then add it to XAML?

@Chips1234 sorry for the delay - I'm assuming #6487 answered your question?

Yes, but how do you get it inside the dialog? As in, how do you get the code into XAML?

Oh so you can just construct the UI element at runtime. Take for example, the context menu on a tab - that's built at runtime in Tab::_CreateContextMenu. Or how we build the errors dialog over in applogic:https://github.com/microsoft/terminal/blob/6d2fc5d1b9152afd1bfba2898f1c1499c3c2d6e5/src/cascadia/TerminalApp/AppLogic.cpp#L340-L382

Okay, so I have got the dialog to be constructed at run-time:
const auto title = RS_(L"KeybindingsTitle", titleKey);
        const auto buttonText = RS_(L"Ok");
        WUX::Controls::ContentDialog dialog;
        dialog.CloseButtonText(buttonText);
        _showDialogHandlers(*this, dialog);

However, I still don't get the content part. Maybe @zadjii-msft can go to my fork and to branch dev/Chips1234/terminal and help me? That would be nice. Thanks!

Right now this PR displays to me as:

@Chips1234 wants to merge 18 commits into microsoft:master from unknown repository

that seems like https://github.com/isaacs/github/issues/168 -- but I'm confused because afaict this PR still exists. Does someone happen to know why it says unknown repository? (Is this related to the PR being a draft?)

@jsoref That's bizarre, I've never seen anything like that before.

@zadjii-msft: I don't suppose you can find someone via the internal directory who could shine some light on this?

Huh, I don't think I've ever actually mailed anyone from Github yet. Kinda forgot that they're under the MSFT umbrella 😅
Cold-emailing random @github.com emails doesn't really seem like it would work to find an appropriate engineer. Maybe @DHowett knows someone?

Yes, unfortunately my fork is all messed up and U needed to delete my old one. Sorry, but I have to close this now.

This is very much because the originating fork was deleted.

Interesting. I'm surprised that the link presumes that the repository is this one even though it more or less knows it isn't.


Error in {: Task 1 failed - "object 'dt_geometric' not found"

Project Robyn
Describe issue
I've been running models using weibull_pdf and have been getting intended results (all Pareto optimal plots, allocation plots/CSVs generated) but all of a sudden I started getting the following error during the "calculation of pareto fronts, clustering and exporting results and plots". I haven't changed any settings. I have my RobynRun iterations set to 3000, and trials set to 6.
Error message:
Plotting 8 selected models on 1 core (MacOS fallback)...0%
Failed exporting results, but returned model results anyways:
Error in {: task 1 failed - "object 'dt_geometric' not found"
Environment & Robyn version
Version: ‘<IP_ADDRESS>01’

From what I have read Robyn on multicore is not deterministic, so there is a chance each run gives different results. I was getting similar errors when Robyn run out of RAM.

I've just pushed a fix and it works for me now. pls test and confirm

Seeing the fix! Thank you very much for resolving @gufengzhou


Restore computation of the hubbard contribution to the stress tensor

added an element to k_point class containing the atomic orbitals before applying S

need a better naming for the original hubbard orbitals. I can add more comments as well.

@mtaillefumier how about naming hubbard_atomic_wave_functions and hubbard_atomic_wave_functions_S for initial atomic |phi> and S|phi> states and hubbard_wave_functions the final set of orbitals, participating in the Hubbard potential correction?

alternatively, for the first two: `atomic_wave_functions_hub_' and 'atomic_wave_functions_S_hub_'

Makes a lot of sense actually.

I may go for the second option. It seems less confusing to me. I will make the changes.

I also  like the second option more. It goes well with the notation indexb_hub, inder_hub

Indeed and it also tells that they are the original wfc. The other notation might confuse us saying those wfc are the one used in the computation. I will make the necessary changes and push --force to my branch.


reconsider hiding controls when user enters complex keyboard shortcut

When user hits Command+Shift+F / Ctrl+Shift+F.
This is a revisit of https://github.com/hchiam/slides/issues/30.

also automatically show controls again when leave fullscreen mode


Planning to support datasets with background?

Hi, thanks for open sourcing this amazing work! I was wondering if there is any plan to support more realistic datasets without object masks (e.g. the plain and simple T&T instead of the preprocessed from NSVF)? I guess so because supporting custom datasets (with COLMAP?) is on the TODO list, but asking just in case :) thank you again!

Yes, it's my next step!


TypeScript error TS1205 when using @aws-amplify/graphql-transformer-interfaces with --isolatedModules

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
yarn
If applicable, what version of Node.js are you using?
v18.7.0
Amplify CLI Version
10.0.0
What operating system are you using?
macOS 12.1
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Amplify Categories
auth, custom, storage, function, api
Amplify Commands
push
Describe the bug
I have a custom resource that depends on @aws-amplify/cli-extensibility-helper. when i upgraded that dependency from 2.3.26  →2.3.34, i started getting an error when executing amplify push (i’m using yarn workspaces, hence why the paths are prefixed with ../../../../../):
🛑 There was an error building the custom resources
🛑 Error: Command failed with exit code 2: /Users/…/amplify/backend/custom/VideoOnDemand/node_modules/.bin/tsc
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(2,10): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
… [see “Log output” for complete logs]

i worked around the issue by adding "isolatedModules": false to the compilerOptions in my tsconfig.json for that custom resource. the issue can be resolved by using type-only imports and exports. this will make is so that files like https://github.com/aws-amplify/amplify-category-api/blob/main/packages/amplify-graphql-transformer-interfaces/src/transformer-plugin-provider.ts, which only import and export types and have no actual code exports, can still be compiled as isolated modules.
Expected behavior
i expected the custom resource to build without error
Reproduction steps

add a custom resource
add latest @aws-amplify/cli-extensibility-helper (v2.3.34) as dependency
use<EMAIL_ADDRESS>(other versions may reproduce, i haven’t checked)
ensure tsconfig.json doesn’t overwrite isolatedModules
run amplify push

GraphQL schema(s)
No response
Log output

# Put your logs below this line
🛑 There was an error building the custom resources
🛑 Error: Command failed with exit code 2: /Users/…/amplify/backend/custom/VideoOnDemand/node_modules/.bin/tsc
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(2,10): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(7,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(8,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(9,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(11,10): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(14,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(15,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(16,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(17,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(18,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(19,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(20,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(21,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(23,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(24,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(25,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(26,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(27,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(28,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(29,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(30,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(31,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(32,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(35,10): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts(35,33): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(2,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(3,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(5,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(7,10): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(8,10): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(9,10): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(9,39): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(12,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(13,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(14,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(15,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(16,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(17,3): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(19,10): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(20,10): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
../../../../../node_modules/@aws-amplify/graphql-transformer-interfaces/src/transformer-context/index.ts(21,10): error TS1205: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
    at makeError (/snapshot/repo/build/node_modules/execa/lib/error.js:60:11)
    at Function.sync (/snapshot/repo/build/node_modules/execa/index.js:194:17)
    at buildResource (/snapshot/repo/build/node_modules/@aws-amplify/amplify-category-custom/lib/utils/build-custom-resources.js:100:25)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async buildCustomResources (/snapshot/repo/build/node_modules/@aws-amplify/amplify-category-custom/lib/utils/build-custom-resources.js:43:13)
    at async transformCategoryStack (/snapshot/repo/build/node_modules/@aws-amplify/amplify-category-custom/lib/index.js:54:5)
    at async transformResourceWithOverrides (/snapshot/repo/build/node_modules/amplify-provider-awscloudformation/lib/override-manager/transform-resource.js:80:9)
    at async Object.buildOverrides (/snapshot/repo/build/node_modules/amplify-provider-awscloudformation/lib/utility-functions.js:106:7)
    at async buildOverridesEnabledResources (/snapshot/repo/build/node_modules/amplify-provider-awscloudformation/lib/build-override-enabled-resources.js:31:3)
    at async Object.run (/snapshot/repo/build/node_modules/amplify-provider-awscloudformation/lib/initialize-env.js:109:3)

Additional information
No response

Hey @acusti :wave: thanks for raising this! To better understand the context of the issue what is the motivation behind adding the custom resource to the list of workspaces in your project?

Has a solution been created for this? I'm also experiencing the same issue.
AWS Amplify Code Build Error Message:

2023-01-07T09:31:46.487Z [INFO]: info  - Checking validity of types...
2023-01-07T09:31:46.689Z [INFO]: We detected TypeScript in your project and reconfigured your tsconfig.json file for you. Strict-mode is set to false by default.
2023-01-07T09:31:46.690Z [INFO]: The following mandatory changes were made to your tsconfig.json:
                                 - isolatedModules was set to true (requirement for SWC / Babel)
2023-01-07T09:31:51.467Z [WARNING]: Failed to compile.
2023-01-07T09:31:51.468Z [WARNING]: ./amplify/backend/node_modules/@aws-amplify/graphql-transformer-interfaces/src/index.ts:2:10
                                    Type error: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
                                      1 | export * from './transformer-context';
                                    > 2 | export { TransformerPluginProvider, TransformerPluginType } from './transformer-plugin-provider';
                                        |          ^
                                      3 | export {
                                      4 |   MutationFieldType,
                                      5 |   QueryFieldType,
2023-01-07T09:31:51.495Z [ERROR]: !!! Build failed
2023-01-07T09:31:51.495Z [ERROR]: !!! Non-Zero Exit Code detected

This started when I introduced Cypress testing. Deployment was successful before.
Package.json:
  "dependencies": {
    "@aws-amplify/cli-extensibility-helper": "^2.3.33",
    "@aws-amplify/ui-react": "^3.4.1",
    "@emotion/cache": "^11.7.1",
    "@emotion/react": "^11.7.1",
    "@emotion/server": "^11.4.0",
    "@emotion/styled": "^11.6.0",
    "@mui/icons-material": "^5.2.4",
    "@mui/material": "^5.2.4",
    "@mui/x-date-pickers": "^5.0.0-alpha.2",
    "aos": "^2.3.4",
    "aws-amplify": "^4.3.45",
    "aws-cdk-lib": "^2.49.0",
    "card-validator": "^8.1.1",
    "date-fns": "^2.28.0",
    "dotenv": "^16.0.1",
    "express-jwt": "^7.7.5",
    "formik": "^2.2.6",
    "jarallax": "^1.12.8",
    "jwks-rsa": "^2.1.4",
    "ksuid": "^3.0.0",
    "next": "12.1.5",
    "next-global-css": "^1.3.1",
    "nextjs-progressbar": "0.0.14",
    "prop-types": "^15.7.2",
    "react": "^17.0.2",
    "react-countup": "^6.1.0",
    "react-dom": "^17.0.2",
    "react-image-lightbox": "^5.1.4",
    "react-slick": "^0.28.1",
    "react-syntax-highlighter": "^15.4.5",
    "react-visibility-sensor": "^5.1.1",
    "sharp": "^0.30.7",
    "slick-carousel": "^1.8.1",
    "yup": "^0.32.11"
  },
  "devDependencies": {
    "@next/eslint-plugin-next": "^12.1.6",
    "@typescript-eslint/eslint-plugin": "^5.7.0",
    "@typescript-eslint/parser": "^5.7.0",
    "aws-sdk": "^2.1135.0",
    "babel-eslint": "^10.1.0",
    "babel-plugin-styled-components": "^2.0.2",
    "cross-env": "^7.0.2",
    "cypress": "^12.3.0",
    "eslint": "^8.4.1",
    "eslint-config-next": "^12.1.6",
    "eslint-config-react-app": "^7.0.0",
    "eslint-plugin-flowtype": "^8.0.3",
    "eslint-plugin-jsx-a11y": "^6.5.1",
    "eslint-plugin-prettier": "^4.0.0",
    "eslint-plugin-react": "^7.27.1",
    "eslint-plugin-react-hooks": "^4.3.0",
    "prettier": "^2.5.1",
    "prettier-eslint": "^13.0.0",
    "prettier-eslint-cli": "^5.0.0",
    "rimraf": "^3.0.2",
    "typescript": "^4.9.4"
  }

I'm having the same problem with ^6.0.5


&'s getting encoded in composer.json

If your author name has an & (e.g. Pixel & Tonic) it will get encoded in the generated composer.json:
"name": "Pixel & Tonic",

Looks like by default, EJS escapes html entities:
http://stackoverflow.com/questions/25177035/passing-html-tags-as-ejs-variables-value

Fixed in 1.2.8: https://github.com/nystudio107/generator-craftplugin/releases/tag/1.2.8
Turns out you just use <%- instead of <%= to output unescaped values
Live now on https://pluginfactory.io/


[INFRA] link to changelog of specific BIDS version seems broken in docs

This link shows our changelog: https://bids-specification.readthedocs.io/en/stable/CHANGES.html
clicking on the "v1.2.0" label, I get a 404 Error. Anyone else with these problems?
quicklink: https://bids-specification.readthedocs.io/en/v1.2.0/
clicking on the other labels actually leads to our Github page.
Can anyone replicate this? What's the issue here?

Same here.
As you already mentioned, the "v1.2.0" link points to https://bids-specification.readthedocs.io/en/v1.2.0/ instead of  https://github.com/bids-standard/bids-specification/tree/v1.2.0.
Checking the *.md files in src and the website, it appears that all most recent changes are not reflected. This includes for example #155, #156, #157, #161, #162, #166, #167 and #176 wrt Changes.md; #178 wrt 02-common-principles.md and #179 wrt 01-introduction.md.
Maybe the result of some problems related to builds/tests?

Thank you @sappelhoff and @PeerHerholz for raising this. On our stable builds, the stable tag is set upon a release (github release). This is why you may see that recent changes are not reflected in the changelog on stable, but if you look at our latest build those changes are present. If they are not there, the CHANGES.md is generated on each merged PR into the master repo.
It appears the previous stable release (https://bids-specification.readthedocs.io/en/v.1.1.2/) renders okay. I am curious if this has to do with conflicting stable and versioned build (they are currently the same right now). IMO it is probably more helpful to link out to that versioned stable build (i.e. v.1.1.2 to the rendered readthedocs vs github).
Another part to note is that upon each generation the links are regenerated as well, so on our latest build all of them are linking out to the github page. One way this can be stopped would be to tinker with the changelog generator and put the most recent releases into the pregh-changes.md (and likely change the name of the file because it would contain more than just the pregh changes) and add an additional option to the generator to only generate from the most recent release. So the links that are changed are cemented and not needed to be changed anymore. The changelog generation is being done automatically by circleCI.

This is due to the first release on github there was an issue regarding how to name the versioning (the versioning should follow the github standard) That was not the case in the initial release which is why you see two versions of the same release.

This is due to the first release on github there was an issue regarding how to name the versioning (the versioning should follow the github standard) That was not the case in the initial release which is why you see two versions of the same release.

Would it be smart to delete the erroneous release?

It would be good. It would be changing the current way the changelog is generated. Upon each regeneration (a merged PR) it will come back because the full log (since the github merge) is generated each time. One potential way would be to have the generator only go from the latest tag to recent. This is connected with the other conversations regarding the changelog like how we want the changelog to look with the labels to make it more human readable. Luckily the tools are there to accomplish this

Fixed with a rebuild of v1.2.0 on ReadTheDocs. I also removed the old tag. There's no changing the changelog in old versions, but it should be fixed for the next one one.


IngestionPipeline runs link logs

Add a link to the logs for each pipeline run we show in the UI.

Implement the log link in the PipelineStatus property
Update the UI to link the log when clicking on the status

Duplicate of https://github.com/open-metadata/OpenMetadata/issues/2725


Add 'include' to the install path of the headers

When installing the headers (as far as I see there are no library files) into the system using make install installs them into /usr/local/rxcpp/... which is not a standard place for header files...  This PR fixes that by installing them into something more like /usr/local/include/rxcpp/.... Of course the /usr/local is just my default CMAKE_INSTALL_PREFIX which can be changed to whatever people like. This will allow me to package rxcpp more easily into and AUR package which I'll hopefully soon make because I didnt find any out there.

Good catch, thank you!


Request to pickup errors earlier in the LZA Pipeline stages

Is your feature request related to a problem? Please describe.
Error encountered:
Container] 2024/05/13 20:42:22.620799 Running command if [ -z "${ACCELERATOR_STAGE}" ]; then for STAGE in "key" "logging" "organizations" "security-audit" "network-prep" "security" "operations" "network-vpc" "security-resources" "network-associations" "customizations" "finalize" "bootstrap"; do set -e && yarn run ts-node --transpile-only cdk.ts synth --require-approval never --config-dir $CODEBUILD_SRC_DIR_Config --partition aws --stage $STAGE; done; fi
yarn run v1.22.19
$<EMAIL_ADDRESS>--transpile-only cdk.ts synth --require-approval never --config-dir /codebuild/output/src2898/src/s3/01 --partition aws --stage key
Done in 67.47s.
yarn run v1.22.19
$<EMAIL_ADDRESS>--transpile-only cdk.ts synth --require-approval never --config-dir /codebuild/output/src2898/src/s3/01 --partition aws --stage logging
Done in 70.13s.
yarn run v1.22.19
$<EMAIL_ADDRESS>--transpile-only cdk.ts synth --require-approval never --config-dir /codebuild/output/src2898/src/s3/01 --partition aws --stage organizations
2024-05-13 20:45:48.045 | error | app | Cannot read properties of undefined (reading 'indexOf')
2024-05-13 20:45:48.046 | error | toolkit | TypeError: Cannot read properties of undefined (reading 'indexOf')
2024-05-13 20:45:48.173 | error | accelerator | Runtime Error
/codebuild/output/src2898/src/s3/00/source/packages/@aws-accelerator/accelerator/lib/accelerator.ts:58
throw new Error('Synthesis failed');
^
Error: Synthesis failed
at process. (/codebuild/output/src2898/src/s3/00/source/packages/@aws-accelerator/accelerator/lib/accelerator.ts:47:9)
at process.emit (node:events:525:35)
at process.emit (node:domain:489:12)
at process.emit.sharedData.processEmitHook.installedValue [as emit] (/codebuild/output/src2898/src/s3/00/source/node_modules/@cspotcode/source-map-support/source-map-support.js:745:40)
at process._fatalException (node:internal/process/execution:149:25)
2024-05-13 20:45:48.265 | error | toolkit | Subprocess exited with error 7
2024-05-13 20:45:48.266 | error | toolkit | Options were: {"command":"synth","enableSingleAccountMode":false,"partition":"aws","stackPrefix":"AWSAccelerator","stage":"organizations","configDirPath":"/codebuild/output/src2898/src/s3/01","requireApproval":"never","cdkOptions":{"centralizeBuckets":true,"useManagementAccessRole":true},"useExistingRoles":false}
Synthesis of stacks failed
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Describe the feature you'd like
Requests that the error be caught at the earlier stages, when the config files syntax is checked.
Additional context
It appears that the excludeRegions[1] under S3 was missing was causing the error.
After defining the security-config guardDuty the issue was resolved.

guardduty:
enable: true
excludeRegions: []
s3Protection:
enable: true
excludeRegions: []
exportConfiguration:
enable: true
destinationType: S3
exportFrequency: FIFTEEN_MINUTES

References:
[1] - security-config.yaml/ excludeRegions
https://github.com/awslabs/landing-zone-accelerator-on-aws/blob/main/reference/sample-configurations/lza-sample-config/security-config.yaml

We have validation scripts that run at the Prepare stage of the pipeline. If there are invalid values by way of missing property values that are required for certain objects, it's likely that this will fail at the Build stage of the pipeline. Which stage of the pipeline are you getting these errors at?

Hello,
The fix is implemented in 1.7 release.
Thank you for your interest in Landing Zone Accelerator on AWS.


Update imagemin to 4.0.0

Fixes #325 for me.

Yes, this updated version of imagemin uses an updated version of vinyl-fs, that no longer relies on a bug in duplexer2. See https://github.com/deoxxa/duplexer2/commit/8fac12bb57d53f47d8dd7b16ca11079501bbe790#commitcomment-14269090

I need this fix soon! please!

+1

+1

+1

+1

+1

1+

:+1:

+1

+1

+1

+1

+1

+1

+1

Please don’t spam me with your plusones.

+1

+1

+1

ping @XhmikosR @vladikoff @sindresorhus @kevva

@_@

+1

This PR should be closed, it has been merged.


Reviewer cannot access unapproved comments - Bug Fix

https://github.com/bcgov/met-public/issues/1900
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #1981 (fdab006) into main (df81229) will increase coverage by 0.00%.
Report is 2 commits behind head on main.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1981   +/-   ##
=======================================
  Coverage   69.34%   69.34%
=======================================
  Files         443      443
  Lines       13640    13642    +2
  Branches     1020     1020
=======================================
+ Hits         9458     9460    +2
  Misses       3992     3992
  Partials      190      190

Flag
Coverage Δ

metapi
77.98% <100.00%> (+<0.01%)
:arrow_up:

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

Files Changed
Coverage Δ

met-api/src/met_api/services/comment_service.py
79.20% <100.00%> (+0.42%)
:arrow_up:

... and 1 file with indirect coverage changes


Add all_whl_requirements similar to rules_python

This is needed to generate the gazelle_manifest.yaml

This looks fine, thanks! Can you rebase to main so that these tests pass?

@ewianda thank you for this improvement. It appears rules_pycross and rules_python have different labels for deps:
# assuming use_repo(lock_repos, "pip") || use_repo(pip, "pip")
deps = ["@pip//:flask"] # rules_pycross
deps = ["@pip//flask"]  # rules_pyton

and consequently Gazelle tries to override that. Have you had an issue with it?
@jvolkman, perhaps you could suggest a workaround to avoid ":" in pycross dev targets?


capability of ignoring unwanted processes in injection list

Made so the user can write an "excluded.txt" file that includes processes uevr will ignore in the injection list, possibly solves #12 and this specific issue I was having where UEVR always put parsec on the list of injectable processes when it definitely isn't one
the code added is very simple and should only trigger in case the txt file is actually written

I am concerned about constantly reading from a file when calling this function for every single process, but that is very miniscule so not too worried about it especially if it's optional right now. I would have preferred it to be cached upon startup or something along those lines.

i tried this but i continue to see  processes that i excluded in the txt file


Duplication on null conversion

We repeat ourselves, and classify an expression such as object o = null in two different ways:
10.2.7 Null literal conversions

An implicit conversion exists from the null literal to any reference type or nullable value type.

10.2.8  Implicit reference conversions includes:

From the null literal (§<IP_ADDRESS>) to any reference-type.

We should classify it as one or the other.

/cc @jcouv

This is a leftover over from wiping the null type. Back in v1 §<IP_ADDRESS> defined the null-literal to have the null type, and §13.1.4 (now §10.2.8) said “From the null type to any reference-type”. So no double naming.
I expect the term “null literal conversion” is too well used now to be dropped, and also that the conversion should be mentioned along with other implicit reference conversions.
We could alternatively view null literal conversions as a subset of implicit reference conversions. Maybe start §10.2.8:

The implicit reference conversions include the null literal conversion (§10.2.7) and the following:

or better wordsmithing.


Move import _torch_sox inside function calls

Import sox only when sox is used at runtime, e.g. sox_effects or loading files, from #355. As noted here, repeated imports of _torch_sox do seem to properly hit cache.

Pushed work to #355


fix(subscriptions): 18998 update POST /billing_subscription to use json body parameters

https://kontur.fibery.io/favorites/Tasks/My-tasks-3575#Tasks/Task/FE-Update-POST-billing_subscription-to-use-application-json-body-18998

Summary by CodeRabbit

Refactor

Improved the setCurrentUserSubscription function to enhance security and performance by moving data from query parameters to the request body.

Must only be merged after this BE PR is deployed to prod:
https://github.com/konturio/disaster-ninja-be/pull/145
https://kontur.fibery.io/Tasks/Task/Improve-API-for-subscriptions-19006


Rebuild bloom filter with appropriate size

Previously, size of rebuilt bloom filter is based on the actual number of
entries in index segment. That number varies in different index segments
and could be very small. This PR ensures size of rebuilt bloom filter is
no less than "storeIndexMaxNumberOfInmemElements" in config and thus the
rebuilt bloom filters should have same performance (compared with previous
bloom filters) on false positive rate.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1af81fc). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #1304   +/-   ##
=========================================
  Coverage          ?   72.17%
  Complexity        ?     6344
=========================================
  Files             ?      460
  Lines             ?    36375
  Branches          ?     4605
=========================================
  Hits              ?    26252
  Misses            ?     8901
  Partials          ?     1222

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com.github.ambry.store/IndexSegment.java
87.33% <100%> (ø)
121 <0> (?)

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 1af81fc...7e8df1f. Read the comment docs.


onClickOutside decorator get static methods

Any way to get onClickOutside decorator get static methods ?
I have exp with autobind decorator and it works fine with static

@Andarist Hello, don't you mind if i create PR adding hoist-non-react-statics as library's pear dependency?

What would be the benefit of static methods? (e.g. what use case would be enabled by adding them).


Use Frament rather than Paper

Expected Behavior
For design purposes, using a Fragment rather than a Paper would allow the user to control how the datatable is contained/displayed in the application.
Current Behavior
Currently, the datatable is inside a Paper, which can cause nested Papers and other issues depending on how the user designs their application.

I would also like the same functionality implemented.

@sornman @SidharthRaveendran
Done in my fork: https://www.npmjs.com/package/material-datatable
Option usePaperPlaceholder

There is now an option called elevation which you can set to 0 to hide the default elevation provided to the Paper component


Add Device Plugin support

Kubernetes' Device Plugin framework enables Kubernetes workloads to request extended resources, such as hardware, advertised by Device Plugins. While krustlet currently supports CSI plugins, it does not support Device Plugins, as noted in its documentation.
In order to add support, three things must happen:

Investigation into how Kubernetes supports multiple plugin types
Reorganizing krustlet's kubelet crate to either match the Kubernetes approach and/or promote support for other types of plugins if more are added to Kubernetes in the future
Implementation! :partying_face:

Investigation
In order to understand how to best add Device Plugin support to krustlet, I sifted through Kubernetes to understand how they are handling CSI and DP plugin support.
Upon startup, the Kubernetes kubelet registers two Plugin Handlers, one for CSI plugins and the other for for device plugins. Krustlet has currently implemented the equivalent of the CSI Plugin Handler as a PluginRegistry in its plugin_watcher module. The PluginRegistry implements both the plugin registration API -- which despite its agnostic name, is not for Device Plugins -- and PluginHandler interface.
Kubernetes' kubelet has one pluginManager to which the CSI and DP PluginHandlers register. The pluginManager.reconciler makes sure that all plugins are in the correct (registered or deregistered) state. For example, it checks to see if a socket no longer exists for a plugin that is in the registered plugin list of thepluginManager.reconciler.actualStateOfWorld. If so, it will unregister the plugin by calling the associated PluginHandler's deregisterPlugin function.
Therefore, plugin management in Kubernetes consists of the following:

A single PluginManager maintains list of plugins and reconciles state of plugins.
CSI and DP PluginHandlers  implement the PluginHandler interface, so the PluginManager can ask them to register/deregister plugins when reconciliation is needed.
CSI and DP plugins implement their own APIs for interacting with their plugins (CSI plugin API and Device Plugin API.

The only shared code between the two PluginHandlers appears to be the reconciliation that happens by the PluginManager.
The CSI and DP PluginHandlers handle registration differently. CSI plugins advertise a Registration service that the kubelet CSI PluginHandler calls when it notices a new socket under /var/lib/kubelet/plugins_registry/. The inverse happens for DPs. The kubelet DP PluginHandler advertises a Registration service at /var/lib/kubelet/device-plugins/kubelet.sock that DPs call.
Kubelet Proposed Restructuring
Currently, in krustlet, there is a plugin_watcher module that contains support for CSI plugins. It also contains a PluginType enum with both CsiPlugin and DevicePlugin options. However, the remaining code is specific to CSI plugin support. This plugin_watcher module may be best renamed csi_plugin_handler and moved to be a submodule under a plugins module. A new device_plugin_handler submodule would also live under the plugins module. The plugin_watcher's valdiate, register, and remove_plugin functions, which appear to be an implmentation of the PluginHandler interface, can be moved to be an implementation of a PluginHandler trait, implemented in both the csi_plugin_handler and device_plugin_handler modules and called out to by the plugin_manager.

The Kubernetes' Device Plugin Manager implements the PluginHandler interface here.

The rust modules inside the krustlet kubelet (./krustlet/crates/kubelet) could be divided like so:
kubelet
└── src
    └── plugins
        ├── plugin_manager (reconciler)
        ├── plugin_handler_interface
        ├── reconciler
        ├── device_plugin_handler
        │   └── ...
        └── csi_plugin_handler

Would these sub-modules be better named device_plugin_manager and csi_plugin_manger, since the PluginHandler interface is only a subset of the functionality they implement? They also each implement their plugin APIs in order to manage plugins. The name PluginRegistry works for CSI, since its API is focused on registering the plugins; however, kubelet implements both the Registration service and device plugin client in the Device Plugin API.

Or, to better match the Kubernetes repository layout where plugins are placed under the directory of their area of concern:
kubelet
└── src
    ├── plugin_manager
    │   ├── plugin_handler_interface
    │   └── reconciler
    ├── cm (container manager)
    │   ├── device_plugin_handler
    │   └── ... later add support for memory manager, topology manager, etc, as in K8s cm dir
    └── volume
        └── csi_plugin_handler

Note: these are the K8s equivalent for the csi_plugin_handler and device_plugin_handler.

This latter layout propses adding a directory for container managers (cm) as done in Kubernetes. It contains the device plugin manager (equivalent to the PluginHandler), memory manager, topology manager, etc.

Generally speaking, those would have to be provider specific so that krustlet can expose them up to the API server for scheduling decisions. I think in terms of this feature, it would require some extra work, but maybe not as we have worried about. The main work would entail:

Creating a new method on the Provider trait that allows you to return the resources the node has available (I think k8s keeps track of how much has been allocated, right?)
Introducing the plumbing to patch the node with that data

So my question is if the device plugin stuff needs this to work properly. Right now it looks like k8s sets some capacity defaults even if we don't set them, so adding on your own custom ones from the device plugin seems ok and we can add support for normal resource requests down the line

Creating a new method on the Provider trait that allows you to return the resources the node has available (I think k8s keeps track of how much has been allocated, right?)
Introducing the plumbing to patch the node with that data

@thomastaylor312, like you mentioned, the device manager updates a Node.status.capacity and Node.status.allocatable fields for each device plugin. I may be missing a deeper understanding of the K8s scheduler, but shouldnt it be able to reference the Node status when making scheduling decisions to know which resources/devices are available? Therefore, is step 1 unnecessary?

Yes, you are correct. That first step was what was required if we wanted to add the normal CPU/memory resource data, but isn't required by this feature. I just got off on a tangent


Config.json error??

Hi,
I cannot get this to work no matter what I try, can you see what ive done wrong?
Thanks
"platforms" : [
{
"name" : "WeMo",
"platform" : "BelkinWeMo",
"noMotionTimer" : 60
},
{
"platform": "FoscamCamera",
"name": “Foscam”,
"cameras": [{
"password": "1234",
"host": “<IP_ADDRESS>”
},
{
"password": "1234",
"host": “<IP_ADDRESS>”
},
{
"password": "1234",
"host": “<IP_ADDRESS>”
}]
}
]

Read README! The have exemple of config

"platforms": [{
"platform": "FoscamCamera",
"name": "Foscam",
"cameras": [{
"username": "admin",
"password": "password",
"host": "<IP_ADDRESS>",
"port": 88,
"stay": 13,
"away": 15,
"night": 14

You need username, password, host, port to work

I did read me. And it says that the username and port are not necessary and they default to admin and port 88. In any case I tried it with both and still has a config.json error.

Please ha a look at the “ you use. In the mail there different type of quotes. Perhaps you cut and paste the json file in word.
Mit freundlichen Grüßen / Kind regards
Andreas Maurer
Von: schweppy<EMAIL_ADDRESS>Gesendet: Montag, 6. November 2017 13:53
An: luisiam/homebridge-foscamcamera
Cc: Subscribed
Betreff: Re: [luisiam/homebridge-foscamcamera] Config.json error?? (#53)
I did read me. And it says that the username and port are not necessary and they default to admin and port 88. In any case I tried it with both and still has a config.json error.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/luisiam/homebridge-foscamcamera/issues/53#issuecomment-342139698, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHTZnMpeQ4YYBJxayeDRB8r14AgDcvsmks5szwEhgaJpZM4QTJIJ.

SoftService GmbH Training & Consulting - Am Gierath 20d - 40885 Ratingen - fon +49 2102 77086-0 - fax +49 2102 77086-27
Geschäftsführer / Managing Director Andreas Maurer - Registergericht Düsseldorf HRB 29778 - UStld / VAT DE-159759811

Past your config in this site and look for erro
https://jsonlint.com

One more thing! Always change the password and username of your cameras to not get hacked! And is good to make the port, all different in each camera!

Is better to never use default username and password

Thankyou for the quick replies. I did change the passwords but put fake ones on this thread.
Hmmm i have used " should i be using something else?
My config.json is exactly as above (running on a Mac mini)

Jsonlint gives me errors for every colon??
Error: Parse error on line 1:
"platform": "BelkinWeMo","noM
----------^
Expecting 'EOF', '}', ',', ']', got ':'

Hope images are allowed here:
Please compare the doublequotes marked with the arrows. They are different.
<EMAIL_ADDRESS>Mit freundlichen Grüßen / Kind regards
Andreas Maurer
Von: schweppy<EMAIL_ADDRESS>Gesendet: Montag, 6. November 2017 14:21
An: luisiam/homebridge-foscamcamera
Cc: Maurer, Andreas; Comment
Betreff: Re: [luisiam/homebridge-foscamcamera] Config.json error?? (#53)
Thankyou for the quick replies. I did change the passwords but put fake ones on this thread.
Hmmm i have used " should i be using something else?
My config.json is exactly as above (running on a Mac mini)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/luisiam/homebridge-foscamcamera/issues/53#issuecomment-342146374, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHTZnIgXiW0h02IRlEkvIrderaa5JpGjks5szwe6gaJpZM4QTJIJ.

SoftService GmbH Training & Consulting - Am Gierath 20d - 40885 Ratingen - fon +49 2102 77086-0 - fax +49 2102 77086-27
Geschäftsführer / Managing Director Andreas Maurer - Registergericht Düsseldorf HRB 29778 - UStld / VAT DE-159759811

Sorry the picture didn't work?
They all look the same quotes on my mac!

☺ Do you have a plain notepad?
Here us the edited part. No idea if this is all
"platforms" : [
{
"name" : "WeMo",
"platform" : "BelkinWeMo",
"noMotionTimer" : 60
},
{
"platform": "FoscamCamera",
"name": "Foscam",
"cameras": [{
"password": "1234",
"host": "<IP_ADDRESS>"
},
{
"password": "1234",
"host": "<IP_ADDRESS>"
},
{
"password": "1234",
"host": "<IP_ADDRESS>"
}]
}
]
Mit freundlichen Grüßen / Kind regards
Andreas Maurer
Von: schweppy<EMAIL_ADDRESS>Gesendet: Montag, 6. November 2017 14:27
An: luisiam/homebridge-foscamcamera
Cc: Maurer, Andreas; Comment
Betreff: Re: [luisiam/homebridge-foscamcamera] Config.json error?? (#53)
Sorry the picture didn't work?
They all look the same quotes on my mac!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/luisiam/homebridge-foscamcamera/issues/53#issuecomment-342147720, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHTZnMddwhF2W1pifYwHzv-nn9Mfjaplks5szwkMgaJpZM4QTJIJ.

SoftService GmbH Training & Consulting - Am Gierath 20d - 40885 Ratingen - fon +49 2102 77086-0 - fax +49 2102 77086-27
Geschäftsführer / Managing Director Andreas Maurer - Registergericht Düsseldorf HRB 29778 - UStld / VAT DE-159759811

I found it! Thankyou, i was trying to use Text editor but i changed software and the different quotes appeared.
Thankyou everyone for your help!

ALL WORKING 😊

Glad works! Please close this issue


Update index.md

Whether you are new to {% data variables.product.prodname_actions %} or interested in learning all they have to offer, this guide will help you use {% data variables.product.prodname_actions %} to accelerate your application development workflows.

Why:

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[ ] For content changes, I have reviewed the localization checklist
[ ] For content changes, I have reviewed the Content style guide for GitHub Docs.

Whether you are new to {% data variables.product.prodname_actions %} or interested in learning all they have to offer, this guide will help you use {% data variables.product.prodname_actions %} to accelerate your application development workflows.
Why:
What's being changed:
Check off the following:

[x]  I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[ ]  For content changes, I have reviewed the localization checklist
[ ]  For content changes, I have reviewed the Content style guide for GitHub Docs.

Whether you are new to {% data variables.product.prodname_actions %} or interested in learning all they have to offer, this guide will help you use {% data variables.product.prodname_actions %} to accelerate your application development workflows.
Why:
What's being changed:
Check off the following:

[x]  I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[ ]  For content changes, I have reviewed the localization checklist
[ ]  For content changes, I have reviewed the Content style guide for GitHub Docs.


JS pipelining error

The js minimizer seems to have a problem with the mdl-js from google.
I get the following error:
JSMin: Unterminated RegExp at byte 187080: /

My template imports the following js:
Material Design Lite

This is an upstream JSMin issue:
https://github.com/mrclay/minify/issues/81

btw, looks like all you need to do is ensure your file is named -min.js or .min.js and it will be skipped during the JSMn'ifying process.


fix: semantic-release setup

What
fix: semantic-release setup

:tada: This PR is included in version 0.10.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Remove forgetme tool from IS distribution

Resolves https://github.com/wso2/product-is/issues/13979

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2533326651

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2533326651
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2533597215

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2533597215
Status: success

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2534350484

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2534350484
Status: success


User list in admin isn't updated when user logs in

It is only updated when users complete registration.
Considering that the user list will be populated with the logged in users if the admin reloads the page, users should be added to the list prior to them completing registration as well.

Another relevant issue is #273


Labels aren't displaying text?

I couldn't find this issue through the search bar, but I wonder if anyone else has this issue...
I'm operating on windows 7, and while attempting the beginning tutorial,
I can't get the text on any labels I create to show up.
I've triple checked that I was doing exactly what the tutorial asked, but the label is just a grey empty box, no matter what I put into the text property.
I just downloaded godot 2.1.4 brand new, and it's already got this problem. I've searched google, and it has led me to check if my driver was up to date (and I've concluded it IS up to date) so I'm not sure what else I can check...

Closing due to lack of update. Please comment if you can still reproduce this issue in the latest stable version of Godot.


android/res not being included

I recently updated to the latest version of this plugin and now it's not picking up the files in android/res/values/*.xml.
I do have a multi-project build and everything was working fine, but now when I try to compile the android module I get this:
Error: No resource found that matches the given name (at 'theme' with value '@style/GdxTheme')

GdxTheme is defined in android/res/values/styles.xml:
<resources>

    <style name="GdxTheme" parent="android:Theme">
        <item name="android:windowBackground">@android:color/transparent</item>
        <item name="android:colorBackgroundCacheHint">@null</item>
        <item name="android:windowAnimationStyle">@android:style/Animation</item>
        <item name="android:windowNoTitle">true</item>
        <item name="android:windowContentOverlay">@null</item>
        <item name="android:windowFullscreen">true</item>
    </style>

</resources>

Need much more specifics.

@pfn, I'm not sure what else to say. I was hoping there had been a change in the way the res path is loaded and a setting I need to set or change.  My project is originally based off of this project:
https://github.com/ajhager/libgdx-sbt-project.g8
However, it has been updated to use android-sdk-plugin 1.5.9.

please provide a minimal test case that reproduces this issue

Can you perhaps explain what aspect of the plugin defines where to pick up the res folder?

ProjectLayout

Is there any way I can get it to dump the path to res?

I fear it may be looking in root/res instead of root/android/res

android:projectLayout, show android:extraResDirectories
On Wed, Nov 25, 2015 at 9:59 AM, Matt Hicks<EMAIL_ADDRESS>wrote:

Is there any way I can get it to dump the path to res?
—
Reply to this email directly or view it on GitHub
https://github.com/pfn/android-sdk-plugin/issues/231#issuecomment-159688343
.

> android:projectLayout
[info] Ant(/home/mhicks/projects/clients/myproject/myproject-mobile/android)
> show android:extraResDirectories
[info] List()

Then it will pick up from android/res

last android:collectResources

and ask gitter/irc first

Sorry, I'll ask in IRC. Thanks.


Wrong steps for jenkins2 tutorial

Hi man, thanks for you grateful tutorial.
I have a question with the jenkins2 tutorial:

Create a new pipeline jenkins job
    Enter the following for the pipeline script:

node ('testslave') {
stage 'Stage 1'
sh 'echo "Hello World!"'
}
Save job
Run job

You put this, but, the script only can we put in a pipeline project, but, restringe a project ejecution to jenkins-slave volumen only we can configure in a free style project, where we can't use scripts, in this case which correct setup steps?

IF you're going to try it in a Jenkins Freestyle project, just set the build job label to "testslave" and things should work.
If using pipeline the proper syntax is now:
stage ('Default Slave') {
    node ('testslave') {
        sh 'echo "You have spun up a basic slave!"'
    }
}

I also include a test/example job that works out of the box for reference.

@maxfields2000 When will we have your new tutorial?
I can't understand why put two compressed packages in the data container.
Whit the new plugins.sh official file we can install the plugins like your script.
And your scripts are very awesome :D

Sorry it took so long, the new tutorial will be out very soon! Next few days! The blog_refresh branch has the new stuff in it! It'll go to master ASAP after I do final tests.

@maxfields2000 Thank you, it is great!

New tutorial changes are now on Master and should be fixed, Blog update goes live in less than 48 hours, so going to close this. Thanks for the feedback and support so far!


Support Go projects

Today, we support node, Java and node projects. We should expand support to cover Go as well.

Completed via #107


UP-4888 - including code to allow for serverBase

including code to allow for inclusion of server.base from config file.

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines
[ ] tests are included
[ ] documentation is changed or added
[ ] message properties have been updated with new phrases
[ ] view conforms with WCAG 2.0 AA

Description of change

including code to allow for inclusion of server.base from build.properties

All hail @markamace1 :)

Folks -- looks pretty good, but please hold off on this for a short time.,
It will create merge conflicts with another PR that's about to merge.  (I don't mind jumping in to help here after I merge that one.)

@markamace1 @drewwills #22 has been merged.
Could this PR be updated to mesh with the changes, and resolve the conflicts?

@markamace1 and @ChristianMurphy -- I'm not unwilling to work on updating this PR as well (though don't let me stop you Mark).
The new property should exist in the default props file and the build.properties file -- both places withh documentation comments -- as well.

I pulled down the latest commit, and looking at the properties file(s), there is no longer a reference to server.base.  The suggested change would require the addition of some variable (server.base? app.base?) to indicate where the application would live, if different from server.home (catalina.home).  Is this worth pursuing - which I believe I can make the necessary changes to make it work correctly - or is it something to make note of in documentation for those that fit this scenario and would want to make that modification?

Those are great questions @markamace1.
How many would use this property?  If it's useful, I think it should be there.  I guess I was leaning toward the sense that "Yes," it's useful and should be there.

I think ultimately it would be of value.  By default, the value would be commented out in the properties files (or, as before set to server.home/catalina.home) with a small comment of "if your application lives in a different directory than the default tomcat install directory, uncomment the live below and put in the appropriate information."
I'll work on a [new] pull for this, hopefully this weekend or early next week.

Thanks @drewwills, closing this PR in favor of #28


remove unnecessary cwd and fix toml defaults

I wasn't aware wrangler could update the TOML at all? From my discussion about migrating the dir property: https://github.com/cloudflare/wrangler/issues/1951#issuecomment-862452882 I

Note modifying wrangler.toml in particular is tricky without messing with formatting -- we have toml-edit in wrangler now, but it only supports toml 0.4 syntax while the main parser toml-rs supports the full toml 1.0 syntax (namely, dotted keys). There's a good chance some folks are using toml 1.0 syntax, and we'd fail to edit their config as a result.

Does that mean we're already effectively only assuming support for Toml 0.4? And we could reopen that discussion about auto-updating config?

I wasn't aware wrangler could update the TOML at all? From my discussion about migrating the dir property: #1951 (comment) I

Note modifying wrangler.toml in particular is tricky without messing with formatting -- we have toml-edit in wrangler now, but it only supports toml 0.4 syntax while the main parser toml-rs supports the full toml 1.0 syntax (namely, dotted keys). There's a good chance some folks are using toml 1.0 syntax, and we'd fail to edit their config as a result.

Does that mean we're already effectively only assuming support for Toml 0.4? And we could reopen that discussion about auto-updating config?

Wrangler only edits tomls provided in templates, which at least for CF templates have always been toml 0.4.
It'd be really nice to auto-update user configs, and I think in practice folks using toml 0.5/1.0 syntax will be rare, we don't use it in our examples.

Thanks for creating this PR! Wrangler v1 is now deprecated and support is only being provided for critical updates or security concerns. As such, we are closing this PR.
New versions of Wrangler are maintained in the workers-sdk repo. If you are running into a similar issue with wrangler v2, please report it or create a new PR in the workers-sdk repo. For more info about wrangler v1 deprecation, please check out our blog post.


Added group item

This PR feature a Group plot item together with an example.
It is an intermediate implementation which doesn't stores the item as a tree representation, but instead still use the flat representation. In the other word, a group have no special meaning for a plot.
The link between a group and another item is just a relationship between 2 plot items.
This anyway allow to group items together.

A group added/removed from a plot will update the plot with its childrens
The visibility of the group will impact the visibility of the children

There is improvement that could be optionally tested:

Expose the relationshit in the 2 directions (create a Item.getParentItem())
Handle the bouding box as a tree
Handle the visibility as a tree

Changelog:

Added a Group plot item

Adding a Group item while keeping the flat structure is not a good move IMO.
If the flat structure is maintained, the Group should not be an item of the plot, but more an object that allows to handle a couple of items (like changing visibility/adding-removing from plot) at once like matplotlib has Container to group Artists. That would be the intermediate step for me.
If Group is a plot item, then we need a proper tree structure of items.

A group which is not a plot item is pointless to me.
Because anybody can already create such structure, but will still have to manage the life cycle of the object on it's own.
I would prefer not to create another extra layer of indirection in Flint. Because that's stuff which will never be shared back to silx.


Fix UserWarnings from pydantic V2

Context
For a project I'm working on, I am relying on the latest version of ProxyStore which uses V2 of pydantic. As a result, globus_compute_common raises UserWarnings because it is assuming V1 of pydantic is installed. Thus, it is setting two attrs that have since been renamed or removed entirely from V2 of pydantic.

What's been changed
This minimally changed two files that are the source of the raised UserWarning instances. My fix just checks the version of pydantic the user has installed. If the user's version is V2, then the fix removes the attr that pydantic no long supports and renames the attr that has been renamed.

I think we can close this out following https://github.com/funcx-faas/funcx-common/pull/126.

(But thanks for this PR contribution, as it motivated us to move along and deal with this!)


Lint error with virtual elements

Popper supports Virtual Elements so that user can make the popover work without attaching statically to one actual element. However, ReferenceAction of this component currently hardcodes the parameter type to be Element. This causes the following lint error
Argument of type '{ getBoundingClientRect: () => DOMRect; contextElement: HTMLElement; }' is not assignable to parameter of type 'Element'

Maybe change it to something like Element | VirtualElement?

Check out release v1.2.11!

Thanks. Works.


Data structures and interface for function dispatch

Run the dispatch example with python -m examples.run_dispatch
I was able to test with local llama-2 but it sort of misinterprets the question and makes up functions rather than choosing one. Might be able to fix this with a different prompt.
Would also like some feedback if there are other failure modes we should to handle and include in tests. I think I got the obvious ones.
Update:
I restored local llama-2 functionality in run_dispatch after tweaking the prompt to work better with it. If you don't have llama-cpp-python set up or you don't set the llama_models_dir_path / don't have the specified models downloaded there, that dispatcher will be skipped.
To install llama-cpp-python with metal support for Apple Silicon:
CMAKE_ARGS="-DLLAMA_METAL=on" FORCE_CMAKE=1 pip install llama-cpp-python

[x] dispatch with custom prompt
[x] dispatch with OpenAI function calling
[x] better error checking and unit tests

Also will note that I am not up and running with llama 2 yet.  Downloaded the models last night.  So I looked at but did not test or closely examine the llama dispatch code.  Looked good to me on the surface, but I did not spend too much time there.


[Feature Request] Custom CSS On Demand Load

Existing Component
Yes
Component Name
SCSS
Description
How do i do custom CSS when i load the components on demand?
I can only see a way to do it when i load the entire 'element-u'

i actually figured out how to do it, thanks.
However, is it possible to not include any style/css at all? as i want to handle it all myself in seperate files.

To not include any css files, you can remove babel-plugin-component (in package.json, .babelrc and node_modules), and import components like this:
import Button from 'element-ui/lib/button.js';
Vue.use(Button)


DO NOT MERGE Test vfs-2.31.0 and FSMonitor with functional tests

Closing this in favor of https://github.com/microsoft/scalar/pull/491


Non Sticky Session - Spring Application Logout as session becoming invalid

We have deployed latest Memcached in our RedHat Linux Box.
At client side In our spring boot application we are using  spymemcached-2.12.3.jar , memcached-2.3.0.jar ,tc8-2.3.0.jar
We have 2 app server behind load balancer and below is our client configuration -
memcached.session.storage.enabled=true
memcached.server.hostname=XX.XX.XX.XX
memcached.server.port=11211
memcached.server.user=XXXXXX
memcached.server.password=XXXXXX
memcached.uri.ignore.pattern=ico|png|gif|jpg|css|js|bmp|jpeg
memcached.session.sticky=false
memcached.session.backup.async=false
memcached.session.backup.thread.count=10000
memcached.session.locking.mode=auto
Issue - When we logged in to application we are able to successfully login and see logged in user ** ** profile data. However upon subsequent navigating to other pages , our application gets logout.
We have memcached-session-manager log , spring security log , and memecached server log for your review.
app-log-without-memcached.txt
memcached-server-log.txt
siteapp1.txt
siteapp2.txt
What I noticed -

Without memcached if we run on single instance and when navigate page , then in Chrome -> Developer Tool -> Application -> Cookie , we are seeing only single JSESSION ID across many request

With memcached enabled if we run on single instance and when navigate page , then in Chrome -> Developer Tool -> Application -> Cookie , we are seeing multiple JSESSIONID frequently getting changed.

Point#2 can be validated by verifying application log where as MemcachedBackupSessionManager create new session Id many times.

There is a log from spring security which observer no presence of session and causing application to logout as it doesn't find security context.

2018-11-25 18:40:57.163 DEBUG 6507  --- [http-nio-8080-exec-10] o.s.s.w.session.SessionManagementFilter  : Requested session ID 10104B61478B5087782364960D3BABD3 is invalid.

The below log without memcached(default tomcat session manager) , if you noticed spring security properly identified session and so allows to navigate other pages -
2018-11-25 13:49:17.249 DEBUG 8440  --- [http-nio-8080-exec-10] w.c.HttpSessionSecurityContextRepository : Obtained a valid SecurityContext from SPRING_SECURITY_CONTEXT: 'org.springframework.security.core.context.SecurityContextImpl@bae7b640: Authentication: org.springframework.security.authentication.UsernamePasswordAuthenticationToken@bae7b640: Principal: org.broadleafcommerce.profile.core.service.CustomerUserDetails@fef8ae5c: Username<EMAIL_ADDRESS>Password: [PROTECTED]; Enabled: true; AccountNonExpired: true; credentialsNonExpired: true; AccountNonLocked: true; Granted Authorities: ROLE_USER; Credentials: [PROTECTED]; Authenticated: true; Details: org.springframework.security.web.authentication.WebAuthenticationDetails@21a2c: RemoteIpAddress: 0:0:0:0:0:0:0:1; SessionId: D4E690E38691C5CE49A14F975459F97B; Granted Authorities: ROLE_USER'

Do you have any suggestions based on this information ?

Hey, we are facing the exact same problem. Did you manage to solve ? Please let me know. Thanks.

Yeah . I should have closed this issue long back.
@kumar-chandan-kc  -
It's been a long time Memcached running smoothly in our Production.
I recalled it was a place in the application which was creating new session. If you have not yet done try enabling SessionListener and SessionAttributeListener and see is session being created or destroyed by application unexpectedly ?


example can't apper

http://steffen.github.io/theme-material/extjs-5/examples/conference-manager/
chrome 42
Just see a white page,no error log.
sorry for my poor english

Hm, strange, it's working for me, see:

Can you upgrade to Chrome v43 and test?
What OS are you on?

thank you. :) it work with vpn. Great Firewall of China intercept access google .
http://fonts.googleapis.com/css?family=Roboto:400,500

I created a ticket (#3) to maybe remove that dependency in the future. Glad you found a workaround meanwhile though. :)


Can you make it potarble？

Everything is in the only  one folder.just like this:    https://github.com/garethflowers/vscode-portable

// @seanmcbreen @kieferrm @chrisdias

Possible duplicate of:

https://github.com/Microsoft/vscode/issues/329 Portable version!

Right. Duplicate of #329.


[fixed] Resolving issue #137 where getName() in React 16 is undefined

Resolving minor syntax issues with previous pull request. The previous pull request aimed to resolve the issue where reactComponent.getName() is returning undefined.

Since the latest update v1.0.0 I believe this issue should now be resolved.  I am going to close this PR as it is modifying a file that no longer exists.  Please try updating to v1.0.0 and see if there is still an issue for you.

Will do, Thank you!


how to rebuild openpilot on EON（openpilot v0.8.1）

I cannot rebuild openpilot by run "cd /data/openpilot/scripts/ && ./rebuild.sh"
do anyone konw why?
thx

That script doesn't exist. You can build openpilot with the following command: cd /data/openpilot && scons -j4.


rendering issue

i was messing with the space chamber card and saw the tooltip said "right click in the air to show contents. upon doing so it crashed me.
at the time i was testing if microparts worked (redstone tortch+alloy wire in same space, and redstone alloy wire on redstone lamp blocks, with every possible orientation.)
YES it is 3.11 beta BUT looking at the changelogs of the later versions they do not say anything about space chabers beimng fixed nor multiparts.
crashlog:
http://pastebin.com/a7jkVdJ5

This was fixed in the real (non-beta) 3.11: See this changelog entry:
Added a safety test to prevent the space chamber item from crashing in case the area contains blocks with no corresponding item.


Better documentation

Please provide better documentation and examples, I'm trying to enable the image link feature with no luck.

Excuse me, I find that I have a problem with including the plugin from GitHub directly.


WASM Example

Hello,
First, thank you for this cool project.
I was looking for the best/easiest approach to use torch in a browser (now that there is a C++ api, it should be easier to integrate) and it seems that this project can do the job.
However, in the README, only compilation instructions are given.
Could you point me/provide a basic example of how to use nimtorch from javascript ?

I completely missed this issue!
I plan to improve the wasm experience cos indeed at the moment there is not so much. And actually indeed it's not so easy to use without deep nim knowledge.

I added a wasm example, I hope you find it useful and if you have any feedback feel free to let us know :+1:
https://github.com/fragcolor-xyz/nimtorch/tree/master/examples/webassembly

@sinkingsugar thanks, I just tried it and apparently there is a memory leak :/ (takes all my 8GB RAM)
Also, it does not show any output in firefox console (on chrome it does).

Did you use all from conda ?

Yes, all from conda. My env:
name: nimtorch
channels:
  - fragcolor
  - defaults
dependencies:
  - aten=2<IP_ADDRESS>9=wasm_0
  - fragments=0.1.25=0
  - nim=<IP_ADDRESS>=0
  - nimtorch=2<IP_ADDRESS>9=0
  - wasm=1.0=0

emcc (Emscripten gcc/clang-like replacement) 1.38.16 (commit 7a0e27441eda6cb0e3f1210e6837cae4b080ab4c)


docs: faq and section for tooltip around disabled button

Closes #302
📝 Description
Add an entry on the faq page and a section on the Tooltip page for Tooltips around disabled Buttons.
⛳ Current behavior (updates)
No informations about Tooltips around disabled Buttons.
🚀 New behavior
Informations on faq page and a section on the Tooltip page.
💣 Is this a breaking change (Yes/No):
No.
📝 Additional Information
Thanks for asking/reporting and the codesandboxes to @DeeeeLAN on https://github.com/chakra-ui/chakra-ui/issues/5528 and @raulrpearson on https://github.com/chakra-ui/chakra-ui/issues/2849. :rocket:

Good one, @m4x3d! I left you some minor comments for the text, but overall it looks good!

Fixed them all right now. :rocket:


Game class

Created the game class
Its inherits from the Item class
can_be_achivied? method overrides the method from the parent class.
Returns true if parent's method returns true AND if last_played_at is older than 2 years else it returns false

Required Changes ♻
Hi @kene-creator,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!

Highlights
✔ Good Design
✔ Flowing Git flow
Required

[ ] Kindly make sure there are no linter errors.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me [@devMunyi ] in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

I corrected the linter error relating to my file only and not the entire project cause I didn't want to disrupt or change any one's code.


Performance

New PR because I cannot reopen #469 because I force-pushed.
close #429

Codecov Report

Merging #471 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #471      +/-   ##
==========================================
- Coverage   96.26%   96.26%   -0.01%
==========================================
  Files         124      124
  Lines        9208     9207       -1
==========================================
- Hits         8864     8863       -1
  Misses        344      344

Impacted Files
Coverage Δ

...ages/Search/src/DTK_DistributedSearchTree_decl.hpp
100% <0%> (ø)
:arrow_up:

packages/Search/src/DTK_LinearBVH_decl.hpp
100% <0%> (ø)
:arrow_up:

...details/DTK_DetailsBoundingVolumeHierarchyImpl.hpp

packages/Search/src/DTK_LinearBVH_def.hpp
100% <0%> (ø)

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 9949ac3...ddc728e. Read the comment docs.

ping

I already approved.  Go ahead and merge unless you'd like feedback from @aprokop and @sslattery as well.


License of layout_old and search templates

Heya,
Looking the license at {layout_old and search}.html templates it says BSD and more details should be found in the LICENSE file. Although that file says MIT. Was there a change of license? Are those still BSD?
Cheers,
Orestis

I talked to @ericholscher a little on IRC and this is because originally the files were copied from sphinx's basic theme. I also think that layout_old.html is no longer used. SO the question is are we safe to remove the original copyright messages?

I believe we can the layout_old one.


Fix Fix delete on value case due to compaction

Proposed changes
Issue Number: close #xxx

run buildall


Error spamming on console on Block break

Hi!
It's awesome someone continues developing this plugin, thanks.
Every time someone breaks a block for a quest, this message shows on console:
21:58:14 CONSOLE: ERROR]: Could not pass event BlockBreakEvent to Quests v2.3.4
21:58:14 CONSOLE: Source) [?:1.7.0]
21:58:14 CONSOLE: Source) [?:1.7.0]
21:58:14 CONSOLE: Source) [?:1.7.0]
21:58:14 CONSOLE: Source) [?:1.7.0]
21:58:14 CONSOLE: Source) ~[?:?]
21:58:14 CONSOLE: Source) ~[?:1.7.0]
21:58:14 CONSOLE: Source) ~[?:1.7.0]
21:58:14 CONSOLE: ERROR]: Could not pass event BlockBreakEvent to Quests v2.3.4
21:58:14 CONSOLE: Source) [?:1.7.0]
21:58:14 CONSOLE: Source) [?:1.7.0]
21:58:14 CONSOLE: Source) [?:1.7.0]
21:58:14 CONSOLE: Source) [?:1.7.0]
21:58:15 CONSOLE: Source) ~[?:?]
21:58:15 CONSOLE: Source) ~[?:1.7.0]
21:58:15 CONSOLE: Source) ~[?:1.7.0]

Sorry for the late response. Could you please post your quests.yml (preferably though something like Pastebin)?

Here it is:
http://pastie.org/10094525

I tried the quest ¿Tienes un pico? and was able to break blocks without getting any errors. Maybe another plugin is canceling the event? Are there any other errors?

Closing due to lack of response from OP. If this error persists, please open a new issue with the requested information. Thanks!


Show all records in answers

This commit makes a few additions to better suit my needs. I thought you might like to merge it. The output is not compatible with previous versions, but it provides more detail.

Add command line arguments to specify nameservers and port
Update requests and dnspython versions in requires.txt
Use more specific exception catching
Display all records in the DNS answers, not just the first one

Records are separated by semicolons (;) in CLI and CSV output
in JSON output, records are listed by type

I also just noticed that my editor converted tabs to spaces. I can switch them back if you like.

Please switch back to spaces. I will review your additions in a couple of days.

Your changes look good but I'm a little bit concerned about readability in CLI due to increased amount of DNS details. Another concern is backward compatibility. CSV/JSON output is often parsed with scripts which probably will stop working. I think it would be best to add another command line argument enabling verbose DNS data in the output. Of course I can take care of it. Please make another pull request with tabs instead of spaces (I really prefer using tabs for indentation) and I will merge it.

I converte dto tabs, but I can't get git to recognize that the file changed for some reason. I'll close this, and create a new PR.


[HxInputDate] Dropdown not working after once disabling the component

If you toggle the HxInputDate component off and on using the "Enable" parameter the datepicker does not show on click.
<HxInputDate Label="Date" @bind-Value="@Value" Enabled="@Enable" />

Duplicates #458, fixed, waiting for release.

Fixed and released.


Milestone6: Add views

In this milestone I:

Added markups to the views
Added navigation
styled the structure of the view

okay, thank you


remove roll and unroll from donut example

Just updating to dip. Took a minute to figure it out, but I think this version with dip is easier to follow. Nice to temporarily stash one value while calculating the next. Also rechecked all the math and fixed a bug in how the luminance was calculated.

Great, thank you!


doc: add question of the difference between Pouch and Kubernetes

Signed-off-by: Allen Sun<EMAIL_ADDRESS>1.Describe what this PR did
In community, lots of peopler ask me that what is difference between Pouch and Kubernetes.
Then I wrote the difference or relationship in FAQ.md
2.Does this pull request fix one issue?
NONE
3.Describe how you did it
NONE
4.Describe how to verify it
NONE
5.Special notes for reviews
NONE

LGTM @allencloud


Templates are not created using raco new

I tried these commands in the shell:
raco new qi-tutorial
raco new template
raco new rosette
raco new rosette rosette-project
raco new --list

None of them had any effect. I tried raco pkg update from-template and it says "No updates available"
Any ideas? I'm on Racket 8.3.

Ahh, so sorry about that, the newest version of the tool isn't pushed to the package archive :facepalm: I'll do that now.

Ahh, so sorry about that @countvajhula, it's from my misunderstanding of how modules work from the looks of it. Thanks for bringing this to my attention, I'll get it fixed asap!

No worries @nixin72 , thanks for your efforts!

Okay, this has been fixed @countvajhula! So sorry for the inconvenience!

I see the templates now with raco new --list, and am able to instantiate them as well 👍


Stream field docs to markdown

Changed all rst documents in docs/reference/streamfield to markdown. This was my first time contributing to the documentation and some of the rst didn't have an exact match to md but did my best to format it similarly (hope the links work as intended!).
To help with issue #8383

[x] Do the tests still pass?[^1]     -- I've have had some problems running the codebase through vagrant VM on windows 11 but there were no tests found for the docs directory so presuming my changes will have no affect.
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

rest not applicable:

[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

Thanks for taking the time to contribute @benedictFaw - I will aim to get this reviewed soon and give some feedback.

Thanks for taking the time to contribute @benedictFaw - I will aim to get this reviewed soon and give some feedback.


Update URL of "digiedge_frame_generator"

Please update the URL for the Digiedge library in my project. Previously, I was using the URL:
https://github.com/HoussamElbiade/Digiedge_frame_generator

However, I have now switched to using a different repository for hosting the library:
https://github.com/DIGI-EDGE/DPE-TRAM-GEN

Hi @HoussamElbiade. I am confirming your request.
Unlike submissions, this type of request is still handled manually. The Library Manager index maintainer occasionally comes through to clear all the outstanding requests. They will close this issue when it is done. You will get a notification from GitHub at that time.
After that, you can monitor the indexing of the library via its dedicated logs web page.
Information about that here:
https://github.com/arduino/library-registry/blob/main/FAQ.md#can-i-check-on-library-releases-being-added-to-library-manager

Hi @per1234
thanks for your response, I'm waiting for the update

Done:
2023/08/14 09:16:11 Loaded 6083 libraries from DB
Changing URL of library digiedge_frame_generator from https://github.com/HoussamElbiade/Digiedge_frame_generator.git to https://github.com/DIGI-EDGE/DPE-TRAM-GEN.git
Success!


This bundle will be updated to symfony 3?

Hi, i'm trying to use the bundle with symfony 3  but forms have changed.
<?php
    // ...
    $builder->add('rating', 'rating', array(
        'label' => 'Rating'
    ));
    // ...

Thanks.

@n0ni0 yes actually.  I was planning on working on that later today or tomorrow.

@n0ni0 i have pushed the update for symfony 3, enjoy.
https://github.com/blackknight467/StarRatingBundle/commit/b288302e2b97d3a8afc0c5d555ac0a9f879dbe5c


Rust axum guide

Would like to see some good practices for organaizing structs / code / files when there are a bunch of queries that shares similiar shapes and how to organize all structs which describes return types (derived from Queryable).

Would like to see some good practices for organaizing structs / code / files when there are a bunch of queries that shares similiar shapes and how to organize all structs which describes return types (derived from Queryable).

Something really comprehensive might turn this from a quick guide into something else but have added two tips off the top of my head that fit with the Open-Meteo output:
https://github.com/edgedb/edgedb/pull/6783#issuecomment-1930044896

Let's say I want to return Json directly without using serde. How can I do it?
How to convert edgedb_protocol::model::Json to axum::Json?

impl IntoResponse for edb_model::Json {
   fn into_response(self) -> axum::response::Response {
      (
         [(
            axum::http::header::CONTENT_TYPE,
            "application/json".parse::<axum::http::HeaderValue>().unwrap(),
         )],
         self.to_string(),
      )
   }
}

only traits defined in the current crate can be implemented for types defined outside of the crate


Event Log recording of User

The User field in Event Log should contain the user who add/edit the row/form right? the value should be the same as with the global variable  "LoggedIn" ? I don't know if this one is a bug but it will be a good addition for audit log. I think if this one has been resolved we can use LoggedIn value to any form purposes that requires to record the current logged user.

How to auto update user id in modifyby field when save?


Metrics API

SYNOPSIS
We need an endpoint where we can send metrics events with a common format (more below) and have the backend connect to any service required (one, another or multiple depending on the event) to trigger the event there.
Event format
A post should be made to api/v3/metrics with the following format:
{
    "name": "<event name>",
    "properties": "<properties>"
}

Event name
This is the name of the event. It's name has to coincide with the backend class that represents it. If you choose a name that corresponds to no class, a friendly error is sent.
For example: Potato peeled -> Carto::Tracking::Events::PotatoPeeled.
Properties
This is a hash that should contain every value that the event needs: no specific per-event requirement compliancy is checked. It's up to you to program it well. Every event is validated accordingly and validation issues are returned. If the event has any record (vis, user) that is not accesible by the logged in user, you get 403.
IMPORTANT: You should never specify any attribute of a model other than it's id. For example, if you want to send a users' username, just specify user_id: xxx. There is a helper that will load the model and grab whatever you need from it in the back.
For example:
{
    "user_id": "abcd-efgh-ijkl-mnopq",
    "visualization_id": "1234-56789-1011-121314",
    "quota_overage": 32.3,
    "origin": "editor"
}

Implementation
Meta, meta, meta, meta. I know, it's not deterministic. I've explored this solution and I recommend to CR this without seeing the diff, just the new stuff.
The paradigm is that we have an internal format for an event, that can be translated into other service's formats. Here is the tree for lib/carto/tracking:
├── events.rb
├── formats
│   ├── internal.rb
│   └── segment.rb
└── services
    └── segment.rb

Please CR @javitonino

Left some comments, mainly to open some discussions.

Added some stuff:

Per-event property validation
Records concerned by the properties are tested to be accesible by the user logged in.

Tests are 🍏
I've added validation and security concerns.
Please have a quick look @javitonino

Wrote a bunch of stuff. I think nothing really serious, mostly suggestions (feel free to ignore).


Perf/generator

Change the structure of the generator to ensure we don't recalculate things that don't need to be re-calculated:

Break up tag helper discovery into components (from razor files) and compilation (from csharp)
When discovering tag helpers from components, look at each component individually rather than as a group
Break up parsing into multiple steps

Do an initial parse only once
If a document has changed, re-write the tag helpers, recording which ones were used
Re-run the tag helper re-write step, but check if the tag helpers have changed, and skip if we can

@dotnet/razor-compiler for review please. It's not too big, but I did try and split things up logically with the commits so you may find it easier to go commit-by-commit. I've also added some extra comments explaining the logic a bit in a few places.

@dotnet/razor-compiler for re-review Thanks.

@333fred for a second review, thanks

Hmm, merge looks bad. Investigating.


FullSync Google Calendar

PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
Hi, I have a problem when trying to do a full sync I get this error. And I can't figure out what I'm doing wrong.
Code:
 if (!$calendar['sync_token']) {
                    $optParams = array(
                        'showDeleted' => true,
                        // 'maxResults' => 150,
                        //'orderBy' => 'startTime',
                        'timeMin' => date('c', time() - 432000)
                    );
                } else {
                    $optParams = array(
                        'syncToken' => $calendar['sync_token']
                    );
                }

                try {
                    $events = $service->events->listEvents($calendarId, $optParams);
                } catch (\Google_Exception $e) { // sync token vyprsel -> udelame plnou sync
                    Debugger::log($e->getMessage());
                    bdump($calendarId);
                    bdump($optParams);
                    $arr = array(
                        'sync_token' => null
                    );
                    $this->organizer->updateKalendar($calendar['id_kalendar'], $arr);
                    // smaz vsechy udalosti
                    //$this->organizer->delUdalosti($u['id_uzivatel']);
                    $events = $service->events->listEvents($calendarId, $optParams);
                }

Response
{
  "error": "invalid_grant",
  "error_description": "Bad Request"
}

But the problem only occurs with some accounts. For some synchronizations, it will go smoothly.
Thanks for the advice

This is an issue with your authentication (invalid_grant means you're specifying an invalid grant type somewhere in your auth process). Unfortunately, without more information I can't really help you debug this issue.
This repo is for specific issues about the client library. For general questions regarding API usage, you're better off asking on Stack Overflow.


[SPARK-26897][SQL][TEST] Update Spark 2.3.x testing from HiveExternalCatalogVersionsSuite

What changes were proposed in this pull request?
The maintenance release of branch-2.3 (v2.3.3) vote passed, so this issue updates PROCESS_TABLES.testingVersions in HiveExternalCatalogVersionsSuite
How was this patch tested?
Pass the Jenkins.

Also, it seems we could drop these lines:
https://github.com/apache/spark/blob/master/sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveExternalCatalogVersionsSuite.scala#L264-L269

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/7986/
Test PASSed.

Merged build finished. Test PASSed.

Test build #102406 has started for PR 23807 at commit 799a01a.

Test build #102406 has finished for PR 23807 at commit 799a01a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102406/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/7995/
Test PASSed.

Test build #102415 has started for PR 23807 at commit 799a01a.

Test build #102415 has finished for PR 23807 at commit 799a01a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102415/
Test FAILed.

Hi, @maropu . Nice, but shall we wait for a while? This should be done after all mirrors got 2.3.3.

oh, I see...

You are welcome to drop those additional lines. We should be Ok to merge as the release is on the mirrors now, but it wouldn't fail if not anyway.

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8013/
Test PASSed.

Test build #102432 has started for PR 23807 at commit 799a01a.

Test build #102432 has finished for PR 23807 at commit 799a01a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102432/
Test PASSed.

Thanks! Merged to master/branch2.4.
This fix is conflict with branch-2.3, so I'll make a pr in follow-up.