Consider support custom rules and rule's level.

Thx for the awesome library!
Can the users add custom rules and change the severity of the rules for specific packages, by some kind of configuration files?
kinds of like eslint?

Hey 👋  At the moment, you can't do that as I'd like to keep the library opinionated with best practices. Adding new rules via some sort of plugin system had crossed my mind too, but I think it's still a bit early to plan that out as there's a lot of features I'd like to build first. A config file might also come naturally as we get to those features.
So besides the "change severity" part, the rest should come in the long term eventually.

By adding custom rules and suppressing errors to the designated level, a lot of practical things could be achieved.
For example, I worked for large private front repo, by pub lint every prod dependency and indirect dependencies in package.json ,  bundle size could be reduced significantly.
Here are some rules that might help:

ESM_FORMAT_MUST_BE_PROVIDED.
As a front-end project, all packages in this project must support an esm version, which will significantly reduce bundle size.
NO_FIXED_PACKAGE_VERSION.
Fixed package version is the culprit of multiple versions of a project, hence causing large js files size, which leads to performance problems.
HAD_DEPENDENCIE_BUT_NO_IMPORT.
If the package has a prod dependency but has no import or require syntax in corresponding files, it suggests that this dependency is redundant, or this package bundles ite node modules in the final output. The first scenario is bearable and can be treated as a warning, and the latter must be treated as an error.
PREFER_PRESERVE_FILE_STRUCTURES.
When file size is larger than certain limits. Instead of a big fat index file, the original file structures should be provided, so that apps which use these libraries can leverage the use of tree-shaking for esm or babel-plugin-import for cjs files.

By adding pub-lint project in CI, problems can be found in early stages and forbid certain request get merged if it invalids these rules.
In order to do that, here are some rules that need to be suppressed as warnings:

file_does_not_exist in certain scenarios.
<EMAIL_ADDRESS> the latest rxjs, throw an error in the line :

// File does not exist ([More info](https://publint.dev/rules.html#file_does_not_exist))
"types":"index.d.ts",

However,  in its export field, the correct types information can be found.
"exports": {
".": {
"types":
"./dist/types/index.d.ts"
,
...
},

So even though the "types" field, it might as well be treated as a warning.
2. FILE_INVALID_FORMAT can be turned off or turned into suggestion.
@mui/material, @shopify/polaris, and antd  , all throws FILE_INVALID_FORMAT  warning because lack the  "type": "module" field. pub-lint outputs every file as a warning which causes the overall log unreadable.

I'd like to keep the library opinionated with best practices
May I ask what are these practices?

4. PREFER_PRESERVE_FILE_STRUCTURES

I'm not sure if this is true as treeshaking doesn't depend on the file structure.

file_does_not_exist in certain scenarios.
<EMAIL_ADDRESS> the latest rxjs, throw an error in the line :

That's a bug tracked at https://github.com/bluwy/publint/issues/42

FILE_INVALID_FORMAT can be turned off or turned into suggestion.

I don't think it should be turned off because it prints too many warnings. Probably a more concise warning message would be better instead.

May I ask what are these practices?

They're largely documented at https://publint.dev/rules.

The goal with publint so far is to provide sensible lint rules to most npm packages. So scoping it down to a set of rules that fits your preference isn't something I've planned for. Maybe it could be nice to achieve that in the future, but I'm not sure how that will look yet.

Closing this for now as it's out of scope for its current state, but there could be a possibility that publint becomes pluggable and more flexible in the future. There's still a lot more plans for publint that I don't think we can settle on a plugin API yet. Thanks for kicking of the conversation!


wss://ws-star.discovery.libp2p.io is down

wss://ws-star.discovery.libp2p.io is down

While we make our best effort to keep ws-star.discovery.libp2p.io running, we don't currently guarantee its uptime. As the readme mentions, at this time the server shouldn't be used for production applications.
It does look like it is back up now, though.


Add global message component to UI.

Should support different message type later. At present, I think it's ok


Name of the App without " (right click to open)"

Would you consider releasing the application without the unquarantine instructions in the localized filename? "Pixel Perfect (right click to open)" -> "Pixel Perfect".
Yes, it's a hassle to launch unsigned apps for the first time, but once it's done the instructions keep showing up when using Launchpad, Raycast or Alfred to launch Pixel Perfect. I understand and respect the intent, but my inner monk still rebels.
Thank you for reading!

https://github.com/cormiertyshawn895/PixelPerfect/issues/3
For now, you can work around this by renaming the app into "Pixel Perfect " with a space after "Perfect" to give it a cleaner appearance in Launchpad, Raycast or Alfred.

Fixed in the latest release. https://github.com/cormiertyshawn895/PixelPerfect/releases


Remove redundant redirects

Trello: https://trello.com/c/I29wxnSh
These redirects have now been implemented using Short URL Manager.
I've also taken the opportunity to fix a couple of minor typos.
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

@brucebolt Sorry - I had missed that there is at least one failing test. I will tackle that now...

Resolved in fa2e8f9d26967583092ab4effd5a76b484e1bab6 and fixed up into appropriate commit.


Feature Request: batch deploy transactions

It would be great to have a flag like --batch and have hardhat-deploy evaluate the total number of deployments to be done and the sequence and then send all transactions off to the network in one go with their respective sequential nonces.

This is not really possible as some of the deploy step depends on the result of others
Note that this can be done manually in your script. Currently the deploy function always wait for the deploy tx to be mined, but you can use the low level deployments.save and use of ethers factory for example

True - I guess this would only work for create2s


too many .openshiftio directories

The root directory and the integration-tests directrory contain .openshiftio/application.yaml files that don't make any sense. Processing them results in extraneous build configs and deployment configs in OpenShift, which obviously never materialize, they just fail.

My mistake, it generated by a script and forgot to remove it.

There's still an extraneous .openshiftio/application.yaml file in the root directory, which shouldn't really be there.


certifi bad results

Several workarounds have been attempted for certifi, but the erroneous results keep coming.
Latest one is "certifi": "https://github.com/certifi/certifi.io".
It is unacceptable that certifi doesnt have a link to its repo.  I assume it is in the top 1000.

Repo has moved to https://github.com/certifi/python-certifi, metadata is updated, and the algorithm finds it, however my cached PyPI json was outdated.


Fix nuget spec gen to ensure roslyn 3.0 package is not treated like a…

… managed package

/azp run Mac Public BuildXL PR Validation

Update the title to avoid ...


Mention list doesnt automatically popup when typing @

Details

Suggestion list will display when opening chat and typing @ on all platforms except iOS native.
Fixed Issues

$ https://github.com/Expensify/App/issues/25649
PROPOSAL:
Tests

Open app
Navigate to group chat
Tap on the compose box
Type "@"
Verify that the suggestion list displays

[x] Verify that no errors appear in the JS console

Offline tests

N/A
QA Steps

Same as Test Steps
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/41372145/2c7ed73f-eec3-4eb3-b22b-6e8de43a9dd5
https://github.com/Expensify/App/assets/41372145/1d68169f-ee90-430c-b89b-19e781f412d1

Mobile Web - Chrome
https://github.com/Expensify/App/assets/41372145/d3358149-3041-4c74-a93f-22b28edf1044

Mobile Web - Safari
https://github.com/Expensify/App/assets/41372145/bfb5fd01-0005-4d71-8e7a-297d81939b70
https://github.com/Expensify/App/assets/41372145/5d69b950-b821-4953-874a-e50c0bb36942

Desktop

https://github.com/Expensify/App/assets/41372145/6685cf17-4b3f-411d-be47-57fab94e8677

iOS
https://github.com/Expensify/App/assets/41372145/3cebde94-bcc0-4ac1-b65a-836e46dfbeb8
https://github.com/Expensify/App/assets/41372145/10ad1b30-f0b6-4f95-ae74-35ba970c58dc

Android
https://github.com/Expensify/App/assets/41372145/2afbed6f-2f91-4fdd-86c9-71e662c5b259

@WikusKriek Does this fix the native ios as well?

@WikusKriek Sorry for the delay in reviews, only web/desktop seems to be working for me, I am unable to build on ios/android.

Actually found that waiting for selection to update in state and then calling calculateMentionSuggestion fixes the issue on all platforms.

Do you think you have push that change in this PR?

@WikusKriek Did you fix the issue with native ios? I can't see that in the screen recording.

Going to pass this to a C+ so they can move this forward.

@WikusKriek Did you fix the issue with native ios? I can't see that in the screen recording.

I confirmed fix on iOS as well

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/108292595/f52fa777-2168-4cf7-b1e3-e1c659ef88cb

Mobile Web - Chrome
https://github.com/Expensify/App/assets/108292595/844aa0b9-09ad-4ab7-973e-712ee0609714

Mobile Web - Safari
https://github.com/Expensify/App/assets/108292595/d7435a88-8c97-4087-bb49-0aa41e1417e9

Desktop
https://github.com/Expensify/App/assets/108292595/efde9466-308a-4deb-b591-8fba80431ad9

iOS
https://github.com/Expensify/App/assets/108292595/21c38365-b79c-47b1-abc5-604b71381eac

Android
https://github.com/Expensify/App/assets/108292595/6fc00de4-7b4a-4e6a-86c2-2e99e878cb6a

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/Beamanator in version: 1.3.58-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.3.58-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/Beamanator in version: 1.3.59-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.3.59-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Can the IP Address Change when moving?

Can the App Service's inbound IP address change when moving App Service resources across Subscriptions?
This MS article notes deleting and recreating an App Service will, I do not know if that applys here.
https://docs.microsoft.com/en-us/azure/app-service/overview-inbound-outbound-ips#when-inbound-ip-changes

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

ID: ec1ba2ff-7845-f68c-5722-05909b8afccb
Version Independent ID: d2cd518a-7313-1918-fa7f-589a6bc6515a
Content: Move Azure App Service resources - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/move-limitations/app-service-move-limitations.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@DinnyMN - moving a resource to a new resource group or subscription is a metadata change that shouldn't affect anything about how the resource functions.

@tfitzmac - Thank you for helping on the thread.
@DinnyMN - Thank you for your question. Hope the above information  shared by @tfitzmac helps answer your query. Please let us know if you have further queries.


[depend-info] Support option --allow-unsupported

In PR https://github.com/microsoft/vcpkg-tool/pull/184, we terminate the command early according to the supports field in CONTROL/vcpkg.json, which affects the command depend-info, and it does not support the parameter --allow-unsupported.
Now support it.

appreciable time so blocking the unsupported entries isn't really useful.

unless you want to know why something in CI is skipped.

appreciable time so blocking the unsupported entries isn't really useful.

unless you want to know why something in CI is skipped.

We should still be printing the warning, we just shouldn't block other depend-info data when the warning happens.

@BillyONeal Sadly, if we always set it to Dependencies::UnsupportedPortAction::Warn and discard --allow-unsupported, these warnings will never be printed:
https://github.com/microsoft/vcpkg-tool/blob/8553f677a56aff07b4d5485fc3f1539546f28964/src/vcpkg/dependencies.cpp#L894

https://github.com/microsoft/vcpkg-tool/blob/8553f677a56aff07b4d5485fc3f1539546f28964/src/vcpkg/dependencies.cpp#L894
So I modifed this line to:
print2(Color::warning, msg + "\n");

Don't do that. Do it like here and print the warnings that were returned by create_feature_install_plan

@BillyONeal Can we merge this now?


Conditionally load a module via require.

I originally posted this here: https://github.com/systemjs/systemjs/issues/474 However. I now think that this issue is caused by this library after further debugging:
[Error] Potentially unhandled rejection [7] Error loading "os" at file:///Users/blake/Code/Exceptionless.JavaScript/example/systemjs/os.js
Error loading "os" from "exceptionless" at file:///Users/blake/Code/Exceptionless.JavaScript/example/systemjs/exceptionless.js
: file:///Users/blake/Code/Exceptionless.JavaScript/example/systemjs/os.js (WARNING: non-Error used)
	(anonymous function) (es6-module-loader.js, line 7)
	f (es6-module-loader.js, line 7)
	i (es6-module-loader.js, line 7)

I have a javascript library which I wanted to support both web/node. What I ended up doing was having a set of services (web and node) and inject them based on when node is detected.. This works great until I try to use systemjs.. then everything fails.
I was hoping to get your feedback on the best way to accomplish this / is this normal behavior (even if I'm using requirejs [not yet tried it yet]). Should I just have separate builds (I'd rather not)?
Example... I'm running via a web browser and the following class should be defined but never instantiated (yet the require is evaluated):
 NodeSubmissionClient.prototype.sendRequest = function (config, method, path, data, callback) {
        var http = require('http');
}

[Error] Failed to load resource: The requested URL was not found on this server. (http.js, line 0)

I think the root issue is this library may be assuming that we want to load everything in a require statement.. However the load may be in side of a conditional and thus we are loading files we shouldn't be or that don't exist.. Can you please provide feedback on this..

I think that maybe we shouldn't blow up with errors when something can't be resolved as in this case it's inside of a conditional.. As a end user I might want to lazy load a module at runtime.. Currently I get the opinion that everything would be loaded if I wanted it or not.

I just checked and requirejs does the same thing.


[publish] Auth token generator for publisher

POST /api/auth/token
Hand out session token given correct username + p/w
Returns:
JSON response details ...

Done

@subhankarb great. When closing, where possible, please link to relevant commit or code.


Updates?

Hey @dcchambers, just wondering how the Chrome application and the "The Road to 1.0" project are coming along. I would love to help maintain and develop this repo but I want to adhere to the project roadmap.

Hello, since the original https://github.com/1995eaton/chromium-vim project has been updated to fix the "hints" feature, and I have not had the time to be an active maintainer of this project, I will be placing this repo into archived status. For more details, please see this comment.


feat(helm): add plugin management commands

Add plugin management subcommands for installing and removing plugins
to $HELM_HOST/plugins.
Install accepts a vcs url or a local directory.
$ helm plugin install http://github.com/adamreese/helm-env
Installed plugin: env

$ helm plugin list
NAME    	VERSION	DESCRIPTION
env     	0.1.0  	Print out the helm environment.

$ helm plugin remove env
Removed plugin: env

closes #1977

My manual tests:

Start with Helm system with lots of existing plugins
Delete existing plugin

⇒  helm plugin remove github
Error: remove /Users/mattbutcher/HelmHome/plugins/github: directory not empty

That is expected, since it was installed manually.

Manually delete the plugins
Re-add the plugin:

⇒  helm plugin install https://github.com/technosophos/helm-gpg
Installed plugin: gpg

Repeated with a few other plugins. Each one tested manually.

List plugins

⇒  helm plugin list
NAME      	VERSION	DESCRIPTION
env       	0.1.0  	Print the environment that the plugin was given, then exit.
example   	0.1.0  	This is a Helm plugin.
hello     	0.1.0  	This is a demonstration plugin that prints Hello and then exists.
github    	0.1.0  	This plugin provides GitHub services to Helm.
gpg       	0.1.0  	This plugin provides GnuPG services to Helm.
keybase   	0.1.0  	This plugin provides Keybase services to Helm.
last      	0.1.0  	Print the name of the last release.
template  	2.1.3+1	Render templates on the local client.
testgetter	0.1.0  	Print the environment that the plugin was given, then exit.

          	       	T...
tiller    	0.1.0  	Helpers for working with in-cluster tiller.

Delete a plugin

⇒  helm plugin remove gpg
Removed plugin: gpg

List again

⇒  helm plugin list
NAME      	VERSION	DESCRIPTION
env       	0.1.0  	Print the environment that the plugin was given, then exit.
example   	0.1.0  	This is a Helm plugin.
hello     	0.1.0  	This is a demonstration plugin that prints Hello and then exists.
github    	0.1.0  	This plugin provides GitHub services to Helm.
keybase   	0.1.0  	This plugin provides Keybase services to Helm.
last      	0.1.0  	Print the name of the last release.
template  	2.1.3+1	Render templates on the local client.
testgetter	0.1.0  	Print the environment that the plugin was given, then exit.

          	       	T...
tiller    	0.1.0  	Helpers for working with in-cluster tiller.

Install again

⇒  helm plugin install https://github.com/technosophos/helm-gpg
Installed plugin: gpg

Try to install an upstream plugin into an existing directory that already has a plugin.

⇒  helm plugin install https://github.com/adamreese/helm-last
Error: symlink /Users/mattbutcher/HelmHome/cache/plugins/https-github.com-adamreese-helm-last /Users/mattbutcher/HelmHome/plugins/helm-last: file exists

That is as expected.


Implemented Palindrome in Go

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Implemented Palindrome in Go.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes #4563

@ankitaggarwal23  Please review this PR.


cached resolution falsely evicts netty (spark + cassandra)

I ran into a failure with cached resolution, below is a minimized example. The bug doesn't occur with cachedResolution disabled.
Spark includes a dependency on netty-3.8.0.Final, and the cassandra driver includes a dependency on netty-handler-4.x. These two variants of netty don't conflict, and both need to be included. If cached resolution is enabled, however, the 3.8 version is incorrectly evicted.
A workaround is to force the version of the cassandra driver with dependencyOverrides. That doesn't change the version of the cassandra driver selected, but it avoids the bug.
A quick way to see the problem is to copy the following build.sbt, and then:
 $ sbt
 > deps netty

then comment out the dependency override and:
> reload
> update
> deps netty

note that netty 3.8 isn't in on the list in the second case.
val vSpark = "1.3.1"
val sparkCore             = "org.apache.spark"         %% "spark-core"                % vSpark
val sparkCassandra        = "com.datastax.spark"       %% "spark-cassandra-connector" % "1.3.0-M1"
val cassandraDriver       = "com.datastax.cassandra"    %  "cassandra-driver-core"    % "2.1.6"

import sbt.complete.DefaultParsers._

lazy val root = (project in file(".")).
  settings(
    scalaVersion := "2.11.6",
    updateOptions := updateOptions.value.withCachedResolution(true),  // this breaks it
    dependencyOverrides += cassandraDriver,   // this fixes it
    libraryDependencies ++= Seq(
      sparkCore,
      cassandraDriver,
      sparkCassandra
    ),
    deps := {
      val matchString = (Space ~> token(StringBasic,"<search string>")).parsed
      val classpath = (fullClasspath in Compile).value
      val found = classpath.map(_.data.toString).filter(_.contains(matchString))
      found foreach println
    }
  )

lazy val deps = inputKey[Unit]("search classpath")

Thanks for the report!

Spark includes a dependency on netty-3.8.0.Final, and the cassandra driver includes a dependency on netty-handler-4.x. These two variants of netty don't conflict, and both need to be included. If cached resolution is enabled, however, the 3.8 version is incorrectly evicted.

Could you run debug and then update with the cached resolution to see if the log includes some hint on why it did that, and paste it into gitst?

Here's the debug update output.
There's this bit:
[debug] - conflict in runtime-internal:com.datastax.cassandra:cassandra-driver-core (com.datastax.cassandra:cassandra-driver-core:2.1.5, com.datastax.cassandra:cassandra-driver-core:2.1.6)
[debug] :: merging module report for com.codahale.metrics:metrics-core:3.0.2 - Vector((Artifact(metrics-core,bundle,jar,None,Vector(master),None,Map()),/usr/local/google/home/mighdoll/.ivy2/cache/com.codahale.metrics/metrics-core/bundles/metrics-core-3.0.2.jar))
[debug] :::: transitively evicted runtime-internal: io.netty:netty

which is consistent with the dependencyOverrides workaround.
But I don't see any real clues. Do you?

Here's how we can get update report.
$ CONSCRIPT_OPTS=-Dsbt.log.noformat=true # This works if you use conscripted sbt.
$ sbt "show update" 2>&1 | tee update_cachedresolution.txt

Netty
[info] 	io.netty:netty
[info] 		- 3.8.0.Final
[info] 			status: release
[info] 			publicationDate: Thu Nov 07 10:23:12 EST 2013
[info] 			resolver: sbt-chain
[info] 			artifactResolver: sbt-chain
[info] 			evicted: true
[info] 			evictedReason: latest-revision
[info] 			homepage: http://netty.io/
[info] 			isDefault: false
[info] 			configurations: compile, runtime, compile(*), runtime(*), master(compile), master
[info] 			licenses: (Apache License, Version 2.0,Some(http://www.apache.org/licenses/LICENSE-2.0))
[info] 			callers: org.spark-project.akka:akka-remote_2.11:2.3.4-spark
[info] 		- 3.9.0.Final
[info] 			status: release
[info] 			publicationDate: Sun Dec 22 05:55:40 EST 2013
[info] 			resolver: sbt-chain
[info] 			artifactResolver: sbt-chain
[info] 			evicted: true
[info] 			evictedReason: transitive-evict
[info] 			homepage: http://netty.io/
[info] 			isDefault: false
[info] 			configurations: compile, runtime, compile(*), runtime(*), master(compile), master
[info] 			licenses: (Apache License, Version 2.0,Some(http://www.apache.org/licenses/LICENSE-2.0))
[info] 			callers: com.datastax.cassandra:cassandra-driver-core:2.1.5
[info]

Netty 3.8.0.Final was evicted with the reason "latest-revision", which means there were newer version on the graph, and Netty 3.8.0.Final was evicted transitively when its caller com.datastax.cassandra:cassandra-driver-core:2.1.5 was evicted by cassandra-driver-core 2.1.6.

The generalization of bug:

X1 depends on an older library A1.
Y1 depends on an oder library B1.
A1 depends on a newer library B2.
a newer library A2 no longer depends on B2.
Your project depends on X1, Y1, and A2.

B2 evicts B1.
A2 evicts A1.
The eviction of A1 causes the eviction of B2.

Thanks eugene!
Point me at instructions for how to run a locally built sbt? I'll try your
patch.
On Fri, Jul 10, 2015 at 8:31 AM, eugene yokota<EMAIL_ADDRESS>wrote:

The generalization of bug:

X1 depends on an older library A1.
Y1 depends on an oder library B1.
A1 depends on a newer library B2.
a newer library A2 no longer depends on B2.

Your project depends on X1, Y1, and A2.

B2 evicts B1.
A2 evicts A1.
The eviction of A1 causes the eviction of B2.

—
Reply to this email directly or view it on GitHub
https://github.com/sbt/sbt/issues/2046#issuecomment-120437116.

I just released 0.13.9-RC2. Could you try and see if it fixes your issue?

hi @eed3si9n - I tried 0.13.9-RC2 and the graph.json is smaller than before, but are still pretty large (max is ~130mb), if I completely remove the callers the graph.json files are roughly a tenth of the size. Forgive the dumb question, but would it be right to say that ModuleReport.callers property is only of use when ModuleReport.evicted = true? If that's the case could one only populate callers if evicted == true? If you want me to create a separate issue for this let me know.

@edeustace wrote:

would it be right to say that ModuleReport.callers property is only of use when ModuleReport.evicted = true?

No. What this issue has uncovered is that I need to sort the dependency graph by the degree of dependency before resolving the conflicts. That sorting requires callers. The problem with Ivy is that it seems to include both the direct callers and transitive callers, which result to bloating of of the callers causing #1763.

Thanks for the clarification. What issue number should I be tracking for further reduction of the bloat?

@edeustace I've been using #1763 as the placeholder, but 0.13.9-RC2 should work with lila, so I've closed that one. If you want please file a new issue on this.

thanks @eed3si9n. Verified that 0.13.9-RC2 works, with cached-resolution enabled, for both the sample project (archived here) and the upstream one.

@mighdoll Awesome. Thanks for confirming the fix!


no-password option on startup is ignored

Starting beaker with the no-password option is ignored:
$ ./beaker.command --public-server --no-password
starting nginx instance (....)
[main] INFO org.eclipse.jetty.server.Server - jetty-8.1.13.v20130916
[main] INFO org.eclipse.jetty.server.AbstractConnector - Started SelectChannelConnector@<IP_ADDRESS>:8802
Connecting to https://XXXXXX:8800/
Submit this password: gkFgEafVcLKgFXC

I'm seeing the same issue.

sorry the option is not implemented.  maybe someday...


Provide private lock object to NettyServer

Synchronizing on the this pointer usually is an implementation leak.
Since there is no external collaboration that requires coordination on
specific instances of NettyServer, a private lock object has been
provided to make listening by the NettyServer immune to external
synchronization.

Coverage increased (+0.01%) to 47.988% when pulling 231a1a0552cfd46f796baa3bdd84fd15ff288efc on maguro:synchronized into 78ba123642b90d33c7e19346e38d2bdd6bf607eb on atomix:master.

Ping.


Make consul depend on core-cloud #1003

This indirectly brings caffeine and solves temporarily the problem.

PR for #1003.


Possible out-of-bound read

https://github.com/tim-smart/multipasta/blame/451de6a06ee4b1d05b5b6903e84ce97b0a277a19/src/internal/headers.ts#L94

It's fine in this case as out of bound reads return undefined, and the start < end check will return false in the next phase of parsing.

It's fine in this case as out of bound reads return undefined, and the start < end check will return false in the next phase of parsing.

You're right. It's fine in Javascript, I must have been suffered too much with recent experience of C and Rust 😆 .


Local Area graphs

This PR implements the functionality to build graphs only in an area of interest (AOI).
This includes:

A new node builder, CutOutZarrDatasetNodes, which allows to create nodes from 2 zarr datasets using cutout operation from anemoi-datasets.
A new class, KNNAreaMaskBuilder, to specify an AOI based on a set of nodes, for which we can specify node attributes to mask the nodes of interest.
A new set of node builders, LimitedArea...Nodes, which allows the creation of the nodes only within an Area Of Interest (AOI).
Expanding the MultiScaleEdges functionalities to support multi-scale connections in limited area graphs.

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have updated the documentation and docstrings to reflect the changes
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have ensured that the code is still pip-installable after the changes and runs
[x] I have ran this on single GPU
[ ] I have ran this on multi-GPU or multi-node

Issue ticket number and link
Issues #7
This work would not have been possible without @theissenhelen  and @JesperDramsch and the rest of the AIFS team!

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: JPXKQX:x: theissenhelenYou have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: theissenhelen:x: JPXKQXYou have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: theissenhelen:x: JPXKQXYou have signed the CLA already but the status is still pending? Let us recheck it.


Error text not included in link text

Issue description

When a heading has an error, error text appears after the relevant heading link. This error text is not included in the link text so it does not announce with the link text when a screen reader user is tabbing to the links.

Severity

Medium

Occurs on

Chrome 116.0.58/JAWS 2022

How to replicate:

Navigate to https://stringyland.github.io/bookmarklets/headings-skipped.html and use the bookmarklet
Tab to the heading links that have errors.
Notice how the error (skipped level) does not announce with the link text.

Expected behaviour

It potentially would be good to include the error text within the link text so that screen reader users can hear it all in one go if they are tabbing.

Actual behaviour

The error text is not part of the link so users have to switch to using arrow keys to hear whether the heading link is incorrectly marked up.

Updated with a fix. Moved error text inside <span class="text"> of each link.


Consider defaulting deepWatchData to false

deepWatchData comes with a huge performance impact as it makes a copy of the data. I noticed #75 had the same finding and added config option.  It seems that 99% of the time you would want to default this to false.

Thanks! It makes sense. I'll change it.

In the latest release [1.0.4+] we came back to deepWatchData=true by default, since agility is a major feature for most of the cases. But performance we can regulate now using deepWatchDataDepth config, or via api as it was before.

Thank you for the heads up :+1:


Feat: Add cvlinkedin and cvgithub

Personal information convenience commands for linkedin and github.
This might be helpful since many use these platforms in their cv.
For Github the name is also added to the create the hyperlink.
For Linkedin the first parameter is used for the hyperlink and the second for text that is displayed.
Do you like the idea? If yes, then is the code or okay or you want any changes?
Once you are satisfied, I can update the readme and template accordingly.

Hey, thanks for the contribution and sorry for the late reply!
LGTM. Would be great if you could add it to the docs as well. There might be no vertical space left in the template's sidebar, so feel free to just throw out sth. else. Maybe the phone number or the overleaf or github entry under "social network".


extensions.dev/extensions is down

[READ] Step 1: Are you in the right place?
yes
[REQUIRED] Step 2: Describe your configuration
Trying to access from web browser.
[REQUIRED] Step 3: Describe the problem
Can't access website.
Steps to reproduce:
What happened? How can we make the problem occur?
Expected result
Normal access
Actual result

I believe the site is back now, closing.


dismissible tests: fix flakiness

I tried some other timeout related changes but rearranging the tests seemed to be the only change that helped. I reran several times and all passed (we'll see if this works out in the long run). Haven't dug much into why this works. This flakiness only seems to happen on chrome and when the full test suite is run.

Nevermind - seems like this doesn't consistently fix it. I'll dig more.


Add a method in Finder to search multiple templates

Currently finder find() method accepts a string that is a template name.
Would be nice to have a method that can receive an array of template names and return the first found.

Feature added in version 0.6.


Requiring "debug" Interferes with Ruby autoloading

Your environment

ruby -v: ruby 3.1.1p18 (2022-02-18 revision 53f5fc4236) [x86_64-linux]
rdbg -v: rdbg 1.4.0

Describe the bug
When debug gem is required, it somehow changes how the Ruby autoload is working
To Reproduce
require 'tempfile'
require 'async'
require 'debug' # when I remove this line it works fine

Tempfile.create(['foo', '.rb']) do |file|
  file.write(<<~RUBY)
    class C
      def self.call_me
      end
    end
  RUBY
  file.close

  autoload :C, file.path

  Async do |task|
    3.times do |i|
      task.async do
        p C.call_me
        sleep rand
      end
    end
  end.wait
end

it produces:
  0.0s    error: Async::Task [oid=0xaf0] [ec=0xb04] [pid=1655265] [2022-03-24 12:35:28 +0100]
               |   NameError: uninitialized constant C
               |   → (irb):19 in `block (4 levels) in <top (required)>'
               |     /home/wojtek/.asdf/installs/ruby/3.1.1/lib/ruby/gems/3.1.0/gems/async-2.0.1/lib/async/task.rb:258 in `block in schedule'
  0.0s    error: Async::Task [oid=0xb2c] [ec=0xb40] [pid=1655265] [2022-03-24 12:35:28 +0100]
               |   NameError: uninitialized constant C
               |   → (irb):19 in `block (4 levels) in <top (required)>'
               |     /home/wojtek/.asdf/installs/ruby/3.1.1/lib/ruby/gems/3.1.0/gems/async-2.0.1/lib/async/task.rb:258 in `block in schedule'
nil
=> 3

Expected behavior
It should not error out when used altogether with async gem
Additional context
The original report and @fxn investigation that leads to this issue can be found here https://github.com/rails/rails/issues/44751
FYI @fxn @ioquatix @bruno-

I can repro and the following does not repro.
hmm?
require 'debug' # when I remove this line it works fine
require 'async'

autoload :FileUtils, 'fileutils'

Async do |task|
  3.times do |i|
    task.async do
      p FileUtils
    end.wait
  end
end

Ah, fileutils is loaded already.
I can repro with RSS.
require 'async'
require 'debug' # when I remove this line it works fine

autoload :RSS, 'rss'

Async do |task|
  3.times do |i|
    task.async do
      p RSS
    end
  end
end.wait

@ko1 thanks for investigating.

Maybe I can understand the situation.

access to C -> require a file
"require" compiles a file and executes it. TracePoint (script_compiled) can intercept a code between compiles and execute. debug.gem inserts some code and code do Queue#push. Queue#push invokes the fiber scheduler and another task (maybe fiber) are scheduled and run.

Repro code without debug.gem.
# prepare '/tmp/foo.rb' before run it.
# echo "class C; end" >> /tmp/foo.rb

require 'async'

q1 = Queue.new
q2 = Queue.new

Thread.new{ loop{
  p q1.pop; q2 << :b
}}

TracePoint.new(:script_compiled){|tp|
  q1 << :a
  p q2.pop
}.enable

autoload :C, '/tmp/foo'

Async do |task|
  3.times do |i|
    task.async do
      # require '/tmp/foo'
      p C
    end
  end
end.wait

Another repro code.
require 'tempfile'
require 'async'

Tempfile.create(['foo', '.rb']) do |file|
  file.write(<<~RUBY)
    #
    q = Queue.new
    t = Thread.new{q.pop}
    q << :sig
    t.join

    class C
    end
  RUBY
  file.close

  autoload :C, file.path

  Async do |task|
    3.times do |i|
      task.async do
        p C
      end
    end
  end.wait
end

I want to close this issue because it is not a debug.gem issue.

Threads have constant references synchornized. This works:
require 'tempfile'
require 'async'

Tempfile.create(['foo', '.rb']) do |file|
  file.write(<<~RUBY)
    #
    q = Queue.new
    t = Thread.new{q.pop}
    q << :sig
    t.join

    class C
    end
  RUBY
  file.close

  autoload :C, file.path

  Array.new(10) { Thread.new { p C } }.each(&:join)
end

So, I believe this is down to constant reference synchronization in the context of fibers.
Could you perhaps @ioquatix get a minimal reproduction with fibers that does not use async to arrive to the heart of the matter?

Sleeping while the require is happening is enough to reproduce:
require 'tempfile'
require 'async'

Tempfile.create(['foo', '.rb']) do |file|
  file.write(<<~RUBY)
    sleep 0.01
    class C
    end
  RUBY
  file.close

  autoload :C, file.path

  Async do |task|
    3.times do |i|
      task.async { p C }
    end
  end.wait
end

https://bugs.ruby-lang.org/issues/18663 for further discussion.

So, I believe this is down to constant reference synchronization in the context of fibers

A fiber is just a subset of a thread, so it sounds like locking implementation here is incorrect in CRuby. I need to investigate.

I just created https://bugs.ruby-lang.org/issues/18662.

It looks like we have.... a race condition.

LOL. Let me close mine!

Ok thanks!

I can't close! Don't find a way to delete either... have left a message.


Stackoverflow when deserializing more than one time with ValidatingJsonConverter

Nuget Package
JsonSchema.Net
Package Version
5.3.0
Operating System
Windows
.Net Target (if relevant, please specify the version in the description)
.Net (5 or after)
A clear and concise description of the bug.
Usage of ValidatingJsonConverter throws stackoverflow after second usage. From my understanding _optionsFactory is not working correctly (ValidatingJsonConverter is not removed from options), when there is a usage of cache - converter has been already defined in cache.

What did you expect?
Not having stackoverflow.
Please add test code or steps to reproduce the behavior.
I have followed instruction here https://docs.json-everything.net/schema/serialization/, how to use ValidatingJsonConverter with attribute on deserializing type.
Is there any other information you'd like to share regarding this bug?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks for reporting.  I'll have a look.

I've added this test which runs deserialization 10 times repeatedly.  It passes.
[Test]
public void WithSchema_MultipleDeserializations()
{
    try
    {
        var jsonText = @"{
  ""Bar"": ""bartholomew"",
  ""Value"": 42
}";

        for (int i = 0; i < 10; i++)
        {
            var model = JsonSerializer.Deserialize<FooWithSchema>(jsonText, _options);

            Console.WriteLine(JsonSerializer.Serialize(model, _options));
        }
    }
    catch (Exception e)
    {
        HandleException(e);
        throw;
    }
}

Can you provide a replicating test, please?

Hello, thank you for your test. Please update is as follow:
    [Test]
    public void WithSchema_MultipleDeserializations()
    {
        try
        {
            var jsonText = @"{
  ""Bar"": ""bartholomew"",
  ""Value"": 42
}";

            for (int i = 0; i < 10; i++)
            {
                var options = new JsonSerializerOptions
                {
                    WriteIndented = true,
                    Encoder = JavaScriptEncoder.UnsafeRelaxedJsonEscaping,
                    Converters = { new ValidatingJsonConverter { OutputFormat = OutputFormat.List } }
                };

                var model = JsonSerializer.Deserialize<FooWithSchema>(jsonText, options);

                Console.WriteLine(JsonSerializer.Serialize(model, options));
            }
        }
        catch (Exception e)
        {
            HandleException(e);
            throw;
        }
    }

Interesting.  You're generating new options objects for each iteration?  Why not create a static one and re-use that?
(I'll fix this, obviously, but it's still worth asking the question.)

Interesting. You're generating new options objects for each iteration? Why not create a static one and re-use that?
(I'll fix this, obviously, but it's still worth asking the question.)

For example, lets consider some scope service like this:
private class FooService
 {
     private readonly JsonSerializerOptions _fooOptions = new JsonSerializerOptions
     {
         WriteIndented = true,
         Encoder = JavaScriptEncoder.UnsafeRelaxedJsonEscaping,
         Converters = { new ValidatingJsonConverter { OutputFormat = OutputFormat.List } }
     };

     public FooWithSchema DeserializeFoo(string jsonText)
     {
         return JsonSerializer.Deserialize<FooWithSchema>(jsonText, _fooOptions);

     }
 }

And test method:
    [Test]
    public void WithSchema_MultipleDeserializations()
    {
        try
        {
            var jsonText = @"{
  ""Bar"": ""bartholomew"",
  ""Value"": 42
}";

            for (int i = 0; i < 10; i++)
            {
                var model = new FooService().DeserializeFoo(jsonText);
                Console.WriteLine(JsonSerializer.Serialize(model, _options));
            }
        }
        catch (Exception e)
        {
            HandleException(e);
            throw;
        }
    }

In my case I have common implementation of deserialization service, which allows to provide different JsonSerializerOptions, so that is why these options are coming from outside implementation and can be created each time when I call common service.
Anyway thank you for a quick fix! I will test it


Make analytics page visible to non-admins

It would be nice if the Dashboard > Analytics page could be enabled for non-admin users.
Maybe as a custom capability: https://wordpress.org/support/article/roles-and-capabilities/

@viktorstrate this is already developed in #35 just not released yet im afraid

hopefully soon! stay tuned! i'll close this as there's that other thread already

Thank you 👍

New WordPress plugin release is now out! See https://wordpress.org/plugins/plausible-analytics/
You can now allow selected user roles to access the stats dashboard


optional attributes in main class

Dear,
I wonder if it is possible to use the optional attributes from the builder API in the normal object definition, e.g., let's say I want to use early_stopping_iterations with foot.api.main.Fedot, is there not a way to just call:
fedot = Fedot(..., early_stopping_iterations=3,...) ?
Thanks in advance.
Best regards,
Israel

Sorry, my bad, indeed it is possible


Fix to ValueError raised on Windows when this function is run (https://github.com/criteo/autofaiss/issues/113)

path.index in your code would always result in a ValueError on windows machines because the paths have reversed /'s. With this fix its compatible with any path no matter which way the /'s are facing.

Ci failing

ubuntu-latest did not have python 3.6 available, migrated to a compatible version

was having the same error using autofaiss while running build_index on windows, this pull request immensely helped in solving the issue!

thanks


How to play a local embedded file on Android / IOS without platform specific code?

I would like to embed a file named electronic_chime.mp3 in my Android and IOS projects and play them using Xamarin MediaManager.
All the solutions I see to locating local files seem to use platform specific code.  How can I access this same-named file on Andriod and IOS without having to code specifically for Android or IOS?
I see that on IOS, I should put the file inside Resources, and on Android I should put the same named file in Assets (and set it with buildaction 'Embedded Resource').
So far on Android, I've tried accessing the file with paths "electronic_chime", "electronic_chime.mp3", "file://electronic_chime.mp3", not working.
On each platform: IOS and Android - where do I put the file, how should I set it to be embedded, and what is a URI string or code that I can use that would work on both platforms?

+1

Try again in 0.7.5 and let me know if it works

As you've coded/fixed, and tested this feature, do you have handy the code that one should use with this component to play an embedded audio file on IOS and Android?  I'll likely be able to figure it out, but for the user community can you provide it?  That said, thanks for fixing the issue!  It's on my backlog now to apply the update and get it fixed in my app.

I'll add some docs on this later.

Ok thanks!

Curtis
via phone gmail

On Mon, Jul 29, 2019, 11:31 AM Martijn van Dijk<EMAIL_ADDRESS>wrote:

I'll add some docs on this later.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/martijn00/XamarinMediaManager/issues/507?email_source=notifications&email_token=AGEVSHOFX7M4EGBYERZDDNLQB3BJDA5CNFSM4H2PHVOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3AJQII#issuecomment-515938337,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGEVSHPFVV23JP2XWAQOWH3QB3BJDANCNFSM4H2PHVOA
.


3.2:  getCalendarsForUser() does not return share_displayname

Is that by design or is it missing?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/35006134-3-2-getcalendarsforuser-does-not-return-share_displayname?utm_campaign=plugin&utm_content=tracker%2F294006&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F294006&utm_medium=issues&utm_source=github).

Probably missing!

I was mistaken, this was not actually used there. The share_displayname is actually only used when getting a list of people who are invited to a share.
So the sharee_displayname is used, but it's only used in getInvites() and not getCalendarsForUser()

So the sharee_displayname is used, but it's only used in getInvites() and not getCalendarsForUser()

This is a pitty...
If i have 10 users with a default calendar called "My Calendar" and those calendars will be shared between eachother getCalendarsForUser() will return 10 calendars with the same name for each user (one can't say which calendar is whos).
The Sharee() class gives the possibility to set sharee_displayname with updateInvites() which makes it simple to name the shared calendar "My Calendar (John Doe)" in sharee_displayname.
If getCalendarsForUser() would also return sharee_displayname (it querys the table anyway) it would be very simple to show the the 9 shared MyCalendars with the username attached and the not shared calendars with their "real name".
Since this is not the case atm. we have to query the same table once more with getInvites() to get the info...
Might make it a feature request...

There's a confusion here because "sharee_displayname" is the "displayname" of the user who is sharing, it is not the displayname for a calendar. Displayname is a more generic term for naming stuff in webdav, but yea.. it doesn't reference the calendar in this case.
We have a copy of the displayname of the user who shared something, because we keep it separate from the real displayname of the underlying principal. This is to support inviting people who are not users which is something we might add in the future.

Ok... Thanks for the explanation!
I think the naming of the tables confused me...


specs-go: Image.Created uses a string type

Modify History.Created and Image.Created to be of type time.Time so that creation times may be validated, compared, etc.

LGTM
Surprised that we don't have real tests for this but let's not hold up this PR.
@opencontainers/image-spec-maintainers PTAL

Nit, updated the comment for History.Created to describe layer creation time rather than image.

LGTM

lgtm

This closes issue #517 , I neglected to tag it earlier.

@jlbutler Thanks for the contribution!


Clean commandline options

See https://github.com/Senzing/servegrpc/issues/50

Pull requests:

https://github.com/Senzing/initdatabase/pull/22


List all Rasa rules requiring specified slots, forms or responses

AS AN Architect
I WANT to have a REST service to list all Rasa rules requiring specified slots
SO THAT there wouldn't be duplicate services for it
Acceptance Criteria

[ ] Use POST endpoint /rasa/rules/search with input parameters type: slots/forms and input: {JSON array}
[ ] Rasa rules.yml as source of truth
[ ] Search within OpenSearch

OpenSearch fed with input via a separate pipeline

Examples
Search for required slots
Slots are defined by defining slot_was_set, for example, rules.rule.steps.slot_was_set, rules.rule.condition.slot_was_set, etc.
version: "3.0"
rules:
- rule: common_teenus_ilm_olemas
  condition:
  - slot_was_set:
    - common_teenus_ilm_asukoht: tallinn
  steps:
  - action: utter_common_teenus_ilm
  - action: action_restart

or https://github.com/buerokratt/Training-Module/blob/main/mock1/data/rules.yml#L71-L72
Search for required forms
Forms are defined by defining action or active_loop, for example, rules.rule.steps.action or rules.rule.steps.active_loop with value(s) having a suffix _form
For example
version: "3.0"
rules:
- rule: Activate custom fallback form
  steps:
  - action: utter_not_confident
  - action: custom_fallback_form
  - active_loop: custom_fallback_form

or https://github.com/buerokratt/Training-Module/blob/main/mock1/data/rules.yml#L87-L88
Search for required responses within rules
Responses are defined by defining action, for example, rules.rule.steps.action with value(s) having a prefix utter_
For example
version: "3.0"
rules:
- rule: common_hüvasti_jätmine
  steps:
  - intent: common_hüvasti_jätmine
  - action: utter_common_hüvasti_jätmine

or https://github.com/buerokratt/Training-Module/blob/main/mock1/data/rules.yml#L11

sample input to /rasa/rules/search
{
    "type": "form",
    "form": "custom_fallback_form"
}

@janyprus please link this issue with an appropriate pull request.

Pull requests #163 and #167

Rules search refactored in the PR:
https://github.com/buerokratt/Training-Module/issues/96
In that PR, the search DSL now functions to specifically search in OpenSearch for slots, responses or form. The "field" must be any one of those three and "value" must be the (string) name of the slot, response, or form.

@turnerrainer will explain how to test it


FIXED: Crash with empty table captions

One of our customers reported that they had a crash with the following content.
|  |  |
|:-|:-|
|  |  |
[]

This is a partial fix, but still crashes with:
|  |  |
|:-|:-|
|  |  |
[][label]

I pushed a complete (I think) fix to the develop branch.
Thanks for pointing this out!!

@fletcher Excellent! Thank you.


do you have references about this method?

I want use this method in my project, but I can't figure out why this method work. could you give me reference about this method?

This is an algorithm written by personal experience.
reference:
https://ieeexplore.ieee.org/document/946629
http://web.stanford.edu/~sujason/ColorBalancing/simplestcb.html


[Bug] FlinkJar allowNonRestoredState not work

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
我FlinkJar任务开启了 忽略未声明状态(allowNonRestoredState)，但是并没有生效。Flink客户端手动提交指定 --allowNonRestoredState 程序可以正常启动，Dinky 的 Web 操作会报错如图：

What you expected to happen
job submit sucess
How to reproduce
none
Anything else
No response
Version
1.2.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@zackyoungh CC


[WIP] Controller and validation (task #2048)

The controller will update all the data from the view form.
The Model will validate each record from the type of config\settings.php

Codecov Report

Merging #669 into task-2048-Admin-settings will increase coverage by 0.32%.
The diff coverage is 31.7%.

@@                      Coverage Diff                       @@
##             task-2048-Admin-settings     #669      +/-   ##
==============================================================
+ Coverage                       30.88%   31.21%   +0.32%
- Complexity                       1201     1208       +7
==============================================================
  Files                             109      109
  Lines                            4096     4133      +37
==============================================================
+ Hits                             1265     1290      +25
- Misses                           2831     2843      +12

Impacted Files
Coverage Δ
Complexity Δ

src/Controller/SettingsController.php
0% <0%> (ø)
5 <0> (+4)
:arrow_up:

src/Shell/SettingsShell.php
0% <0%> (ø)
6 <0> (+1)
:arrow_up:

src/Model/Table/SettingsTable.php
100% <100%> (+50%)
6 <2> (+2)
:arrow_up:

Continue to review full report at Codecov.

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


TreeView does not clear nodes state after updating ItemsSource

Describe the bug
After updating TreeView.ItemsSource property  TreeViewNode(s)/TreeViewItem(s) are not updated properly. It seems like TreeView is reusing previously created TreeViewNode(s)/TreeViewItem(s) without properly resetting their state. Below you can find example showing problem with Style and IsExpanded properties of TreeViewItem. The same will happen with HasUnrealizedChildren property of TreeViewNode. The issue exists if you update ItemsSource in code behind or using binding.
Steps to reproduce the bug

In VisualStudio 2022 (17.1.3) create new project - Blank App, Packaged (WinUI 3 in Desktop) .

Add code to MainWindow.xaml file

    <Grid Background="{ThemeResource ApplicationPageBackgroundThemeBrush}">
        <Grid.RowDefinitions>
            <RowDefinition Height="Auto" />
            <RowDefinition Height="Auto" />
        </Grid.RowDefinitions>
        <Grid.Resources>
            <DataTemplate x:Key="RedItemTemplate" x:DataType="local:RedItem">
                <TreeViewItem Content="{Binding}" ItemsSource="{Binding Children}" IsExpanded="True" Foreground="Red"/>
            </DataTemplate>
            <DataTemplate x:Key="GreenItemTemplate" x:DataType="local:GreenItem">
                <TreeViewItem Content="{Binding}" ItemsSource="{Binding Children}" IsExpanded="True" Foreground="Green"/>
            </DataTemplate>
            <local:ColorItemTemplateSelector
                x:Key="ColorItemTemplateSelector"
                RedItemTemplate="{StaticResource RedItemTemplate}"
                GreenItemTemplate="{StaticResource GreenItemTemplate}">
            </local:ColorItemTemplateSelector>
        </Grid.Resources>

        <CommandBar Grid.Row="0" DefaultLabelPosition="Right" VerticalContentAlignment="Center" FontWeight="Bold">
            <AppBarButton Click="RedItems_Click" Label="Red items" />
            <AppBarButton Click="GreenItems_Click" Label="Green items" />
            <AppBarButton Click="Clear_Click" Label="Clear" />
        </CommandBar>

        <TreeView Grid.Row="1" x:Name="TreeView" ItemTemplateSelector="{StaticResource ColorItemTemplateSelector}" />
    </Grid>

Add code to MainWindow.xaml.cs file

    public sealed partial class MainWindow : Window
    {
        private readonly List<ColorItem> RedItems = ColorItem.Create<RedItem>(3);
        private readonly List<ColorItem> GreenItems = ColorItem.Create<GreenItem>(3);

        public MainWindow() => InitializeComponent();
        private void UpdateTreeView(List<ColorItem> items) => TreeView.ItemsSource = items;
        private void RedItems_Click(object sender, RoutedEventArgs e) => UpdateTreeView(RedItems);
        private void GreenItems_Click(object sender, RoutedEventArgs e) => UpdateTreeView(GreenItems);
        private void Clear_Click(object sender, RoutedEventArgs e) => UpdateTreeView(null);
    }

    public class ColorItem
    {
        public List<ColorItem> Children { get; set; }
        public override string ToString() => GetType().Name;

        public static List<ColorItem> Create<T>(int childCount) where T : ColorItem, new()
        {
            var items = new List<ColorItem>();
            for (int i = 1; i <= childCount; i++)
            {
                var item = new T();
                item.Children = Create<T>(childCount - 1);
                items.Add(item);
            }
            return items;
        }

    }
    public class RedItem : ColorItem { }
    public class GreenItem : ColorItem { }

    public class ColorItemTemplateSelector : DataTemplateSelector
    {
        public DataTemplate RedItemTemplate { get; set; }
        public DataTemplate GreenItemTemplate { get; set; }

        protected override DataTemplate SelectTemplateCore(object item)
        {
            return item switch
            {
                RedItem => RedItemTemplate,
                GreenItem => GreenItemTemplate,
                _ => null
            };
        }
    }

Run the project
Randomly click button
Collapse some of the TreeView nodes
Click randomly button again
Note that styles are not updated according to clicked button (for example RedItems are displayed as green)
Note that some of newly created nodes are Collapsed although the style is set make them expanded by default.

Expected behavior
After updating ItemsSource property existing RootNodes should be cleared and new nodes should be created for new items.
Screenshots

NuGet package version
WinUI 3 - Windows App SDK 1.0.3
Windows app type

[X] UWP
[ ] Win32

Device form factor
Desktop
Windows version
Windows 10 (21H1): Build 19043
Additional context
Earlier I experienced strange behavior of SelectedItems property. After changing ItemsSource or even setting it to null the SelectedItems wasn't empty.

Im having a similar problem, TreeView behaves buggy after changing the itemssource.
any clarification / acknolegement from the developers on this ?

Here you have an example of root nodes duplicated as child nodes. [If the first node had not been expanded, this would not have happened]


CRD quotas.config.istio.io hides k8s original quotas

Describe the bug
K8s has resourcequotas (aka 'quota'), but it will be hidden by Istio CRD quotas.config.istio.io after install Istio. Then the pods with quota can not be created.
# kubectl -n bookinfo describe rs app-3813184607-app-v1-0-958bdd964
Name:           app-3813184607-app-v1-0-958bdd964
Namespace:      bookinfo
Controlled By:  Deployment/app-3813184607-app-v1-0
Replicas:       0 current / 1 desired
Pods Status:    0 Running / 0 Waiting / 0 Succeeded / 0 Failed
Pod Template:
  Containers:
   c0:
    Image:      test.caicloudprivatetest.com/istio/examples-bookinfo-productpage-v1:1.8.0
    Port:       <none>
    Host Port:  <none>
    Limits:
      cpu:     500m
      memory:  512Mi
    Requests:
      cpu:     250m
      memory:  256Mi
    Environment:
      POD_NAMESPACE:     (v1:metadata.namespace)
      POD_NAME:          (v1:metadata.name)
      POD_IP:            (v1:status.podIP)
      NODE_NAME:         (v1:spec.nodeName)
      SERVICES_DOMAIN:  bookinfo.svc.cluster.local
    Mounts:             <none>
  Volumes:              <none>
Conditions:
  Type             Status  Reason
  ----             ------  ------
  ReplicaFailure   True    FailedCreate
Events:
  Type     Reason        Age                From                   Message
  ----     ------        ----               ----                   -------
  Warning  FailedCreate  46s (x6 over 2m)   replicaset-controller  (combined from similar events): Error creating: pods "app-3813184607-app-v1-0-958bdd964-pwf9m" is forbidden: failed quota: bookinfo: must specify limits.cpu,limits.memory,requests.cpu,requests.memory

After install istio:
# kubectl -n bookinfo get quota bookinfo
Error from server (NotFound): quotas.config.istio.io "bookinfo" not found

After delete quotas.config.istio.io and restart apiserver:
# kubectl -n bookinfo get quota
NAME       AGE
bookinfo   1h

Expected behavior
K8s original quotas should not be hidden.
Steps to reproduce the bug
{{ Minimal steps to reproduce the behavior }}
Version
istioctl version
Version: 1.0.0
GitRevision: 3a136c90ec5e308f236e0d7ebb5c4c5e405217f4
User: root@71a9470ea93c
Hub: gcr.io/istio-release
GolangVersion: go1.10.1
BuildStatus: Clean

kubectl version
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.6", GitCommit:"a21fdbd78dde8f5447f5f6c331f7eb6f80bd684e", GitTreeState:"clean", BuildDate:"2018-07-26T10:17:47Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.6", GitCommit:"a21fdbd78dde8f5447f5f6c331f7eb6f80bd684e", GitTreeState:"clean", BuildDate:"2018-07-26T10:04:08Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

Installation
{{ Please describe how Istio was installed }}
Environment
{{ Which environment, cloud vendor, OS, etc are you using? }}
Cluster state
{{ If you're running on Kubernetes, consider following the
instructions
to generate "istio-dump.tar.gz", then attach it here by dragging and dropping
the file onto this issue. }}

It works to use full name. Only short name of resourcequota (quota) conflicts with Istio CRD quotas.config.istio.io.

Custom CRD per instance model is going away. The CRD is deprecated, and should go away in future releases.

This was a dupe of #5532 . Closing for the same reason.


Use alternative RegistryOps constructor call to fix datapack errors. #787

RegistryOps constructor call should use (the poorly named) method_36574 so that biomes from datapacks get loaded instead of vanilla biomes. This allows mods that add biome features using Fabric's biome API to have them properly located/generated.

Solved the issue for Mo' Structures and Repurposed Structures!

Thanks!


Add a supported versions to the main page.

We will need to create a support matrix due to issues we have been finding, and will need to patch the upstream tests.
We have found some issues in field level versioning :
/go/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/common/downward_api.go:94
Error creating Pod
Expected error:
    <*errors.StatusError | 0xc4212f9d40>: {
        ErrStatus: {
            TypeMeta: {Kind: "", APIVersion: ""},
            ListMeta: {SelfLink: "", ResourceVersion: "", Continue: ""},
            Status: "Failure",
            Message: "Pod \"downward-api-e1e7a01c-a93e-11e7-93c2-0a580ac00202\" is invalid: spec.containers[0].env[1].valueFrom.fieldRef.fieldPath: Invalid value: \"status.hostIP\": error converting fieldPath: field label not supported: status.hostIP",
            Reason: "Invalid",
            Details: {
                Name: "downward-api-e1e7a01c-a93e-11e7-93c2-0a580ac00202",
                Group: "",
                Kind: "Pod",
                UID: "",
                Causes: [
                    {
                        Type: "FieldValueInvalid",
                        Message: "Invalid value: \"status.hostIP\": error converting fieldPath: field label not supported: status.hostIP",
                        Field: "spec.containers[0].env[1].valueFrom.fieldRef.fieldPath",
                    },
                ],
                RetryAfterSeconds: 0,
            },
            Code: 422,
        },
    }
    Pod "downward-api-e1e7a01c-a93e-11e7-93c2-0a580ac00202" is invalid: spec.containers[0].env[1].valueFrom.fieldRef.fieldPath: Invalid value: "status.hostIP": error converting fieldPath: field label not supported: status.hostIP
not to have occurred
/go/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/framework/pods.go:76

@abiogenesis-now
Well have to outline versions and support given some of the issues and update on later patch release(s).

@timothysc to clarify: are these issues with thee2e tests? Or with certain versions of Kubernetes not playing well with Sonobuoy?
(And I'm lacking a bit of context---are there links to issues that I can read up on? I don't have enough knowledge to be actionable at the moment)

@abiogenesis-now The 1.8 Conformance-e2e tests leverage some api fields that don't exist in 1.6 and as a result the test fails.  So far we've only come across 1 test that has this issue.


Simplify PhiNodes

Turns out we don't need to maintain a stack for each PhiNode. This is important from a code-complexity perspective, and for the performance of hot loops.

This definitely won't be merged, but I'll derived some additional unit tests from the insights I've gained from understanding why this definitely won't work.


minor changes fo fix run

Hi,
congrats for your great work!
I'm following the tutorial. Got these two errors:

ValueError: could not broadcast input array from shape (1000) into shape (4096) --> downsized matrix

RuntimeError: Language not supported: en_glove_cc_300_1m_vectors --> used spacy ver 0.100.6

Did a couple of fixes below.
Still fighting with:
Loading image features ...
Loading question features ...
Loading VQA Model ...
Traceback (most recent call last):
File "demo.py", line 110, in
main()
File "demo.py", line 94, in main
vqa_model = get_VQA_model(VQA_model_file_name, VQA_weights_file_name)
File "demo.py", line 59, in get_VQA_model
vqa_model = model_from_json(open(VQA_model_file_name).read())
File "/Users/draix/Dropbox/Dev/Envs/vqa/lib/python2.7/site-packages/keras/models.py", line 33, in model_from_json
return layer_from_config(config, custom_objects=custom_objects)
File "/Users/draix/Dropbox/Dev/Envs/vqa/lib/python2.7/site-packages/keras/utils/layer_utils.py", line 24, in layer_from_config
class_name = config['class_name']
KeyError: 'class_name'
-- Will update tomorrow.

@draix
Current VQA Demo works only with Keras 0.3

The new Keras (1.0+) does not have pop layer and thus the last layer of the VGG net is not removed. If you see the model of the VGG Net the final layer is the output of the softmax (1000 categories of Imagenet Challenge)
If you change the dimension to 1000 (from 4096) you are effectively using the last layer output and thus you are identifying only one "object" (among the 1000 imagenet classes) and trying to use that as feature for question answering. This will not work, as the questions are not always about the main object of the image.
I have been slacking (due to other VQA challenge) and have not updated this to Keras 1.0, but I plan to do so very soon (maybe a day or two). However, if you want to work with current version I am afraid you will have to use older version of Keras.

I will have to look at the change about the Spacy, as when I initially made this adding glove vector was still in beta.

Thanks for your prompt reply, and congrats again for your great work.
Got it working now.
Added a requirements.txt file with the dependencies installed and package versions supported on the my running version, and updated the Readme notes, in case you find it interesting. I've struggled a lot with the spicy error. It seems is a bug on the new versions.
Best,
M

@draix Please use the new code I have pushed. This properly does the layer removal and maintains the 4096 features. (If you use 1000 features like the code you pushed, you will not get good results).
Also I have uploaded the weights trained on Keras 1.0. (There are issues on loading weights of previous layers because Keras 1.0 counts layers differently than 0.3)


WIP sort files by modified date

Following #32, this is first draft of the implementation.
@rebootuser I would like to draw your attention to the following:

Arrays are not supported for dash which is a symlink for /bin/sh on some distributions
I am not sure I like the syntax: wwsgidrt=`find / -uid 0 -perm -2007 -type f "${find_sort_print_flag[@]}" -exec ls -la {} 2>/dev/null \; | eval "${find_sort_pipe_args}" because it adds complexity
I had to get rid of the syntax: find /etc/init.d/ \! -uid 0 -type f 2>/dev/null |xargs -r ls -la 2>/dev/null to make it work with the rest

It is not yet done, but I would like your review before proceeding, in case I have to find another alternative.
The result works as expected:
For find based commands:
-rwxr-sr-x 1 root shadow 22808 Jan 25 2018 /usr/bin/expiry
-rwxr-sr-x 1 root shadow 71816 Jan 25 2018 /usr/bin/chage
-rwxr-sr-x 1 root shadow 34816 Apr 5 2018 /sbin/pam_extrausers_chkpwd
-rwxr-sr-x 1 root shadow 34816 Apr 5 2018 /sbin/unix_chkpwd
-rwxr-sr-x 1 root tty 30800 Jul 26 2018 /usr/bin/wall

For ls based commands:
[-] /etc/init.d/ binary permissions:
total 16
-rwxr-xr-x 1 root   root    1191 Jan 17  2018 procps
-rwxr-xr-x 1 root   root    3809 Feb 14  2018 hwclock.sh
drwxr-xr-x 1 root   root    4096 Mar 25 16:24 ..
-rw-r--r-- 1 nobody nogroup    0 Mar 25 17:59 toto
-rw-r--r-- 1 nobody nogroup    0 Mar 25 17:59 tota
drwxr-xr-x 1 root   root    4096 Mar 25 17:59 .

Thank you for your time and input!

Ok I found another way of doing this that will be portable.

@rebootuser The code looks way cleaner and I tested it with dash and bash. I also tested on an ubuntu container and with alpine.
It does not appear to work well with busybox's find since there is no printf. If that's a requirement, I'll try to make it work, by detecting the find version.

@rebootuser OK I think I'm finally done and happy with the result. It works on:

bash and I assume other modern shells
dash default /bin/sh on ubuntu
alpine find or busybox based systems (de-active the printf option)

Apologies for the delayed reply and thanks for the additions! I'll test these out and revert shortly.

No worries at all! I'm here if you have any questions.

Firstly, I appreciate the contribution. Looking at the additions it seems that a level of complexity has been introduced and I'm not sure that the benefits are great enough to outweigh the need for this functionality. My main concern is compatibility with a large range of shells/distos.

Hi! Thank you for making such an awesome script!
Yes, that's the only way I found that would be as portable as possible. I use it every day and I've been trying to fix as many things as possible whenever I see a glitch.
I totally understand though. Don't hesitate to give it a try and see if it helps you! I'll keep the branch open and up to date with upstream because it does really help me.
If you have any ideas on how to make it better, let me know!

Closing for now as I think this introduces a level of complexity that might cause issues on some distos. Your contribution is very much appreciated, and in downtime I'll try to test this further.

Thank you for at least looking into it and considering it! I'm keeping my fork up at https://github.com/tehmoon/LinEnum just in case! (Also because I'm using it)


Client-side fallback

We should detect if the server proxy is not installed and use the extension purely client-side (with a warning). Then the user could get a few minutes of use before being rate limited.

Fixed by #6


Update module_creator to use getpass.getuser instead of os.getlogin f…

…or username

Coverage increased (+0.002%) to 91.101% when pulling b684eb1e6488d7f8f957a5205cff2628fb259420 on mc-test-fix-2 into 235181a1e42a943e7d3bdce849b34acb188f17c2 on master.

Note: I have run the following system tests with this change: local_repository_tests, local_verification_tests and name_verification_tests. All passed.


is there a pretrain model release schedule for InternVideo2?

specially the stage 2, multimodel align trained models weight.
hope to get your update

Thank you for your interest. Please check here for code. We released 1B model at here.