Bug Fix: "Error: DYNAMIC segment not found, might be a statically-linked ELF?" at linking stage

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Fixes the issue of
brew install go failing with
 ...
Error: DYNAMIC segment not found, might be a statically-linked ELF?

cc @jonchang  @sjackman @woodruffw

Thanks!

Thanks for the quick fix, @rmNULL!


add support for default-namespace labels

See:

https://github.com/kubernetes/kubernetes/pull/96968

All namespaces should have a label of
"kubernetes.io/metadata.name: <namespaceName>"

Cyclonus supports this without any changes.  If labels are on a namespace, Cyclonus will pick those up and use them for connectivity matrices.  It's up to kube to ensure the labels are there.


How to set the scopes in acquireTokenByClientCredential when requesting consent for https://{location}.management.azure.com

Core Library
MSAL Node (@azure/msal-node)
Core Library Version
1.10.0
Wrapper Library
Not Applicable
Wrapper Library Version
NA
Public or Confidential Client?
Public
Description
I want to request consent for https://{location}.management.azure.com e.g: https://eastus2euap.management.azure.com. But I am not sure how to set the scopes value when calling the method acquireTokenByClientCredential (code).
If I set the value as https://{location}.management.azure.com/.default, the relevant code would be like:
const result = await this.confidentialApp.acquireTokenByClientCredential({
      scopes: "https://eastus2euap.management.azure.com/.default",
      correlationId: "XXXXXX",
      authority: "https://login.microsoftonline.com/72f988bf-86f1-41af-91ab-2d7cd011db47"
  });

Then I received 400 error with messages - https://login.microsoftonline.com/error?code=500011.
`invalid_resource: 500011 - [2023-06-06 01:16:48Z]: AADSTS500011: The resource principal named https://eastus2euap.management.azure.com was not found in the tenant named Microsoft.
This can happen if the application has not been installed by the administrator of the tenant or consented to by any user in the tenant.
You might have sent your authentication request to the wrong tenant.`

If I switch the value to  https://management.azure.com/.default, then I could get the token without errors.
Does that mean if I want to get any permission located in https://{location}.management.azure.com, I would specify the scope with "https://management.azure.com/.default".
const result = await this.confidentialApp.acquireTokenByClientCredential({
      scopes: "https://management.azure.com/.default",
      correlationId: "XXXXXX",
      authority: "https://login.microsoftonline.com/72f988bf-86f1-41af-91ab-2d7cd011db47"
  });

Our issue is here: https://github.com/Azure/azure-sdk-for-js/issues/26106
MSAL Configuration
https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/identity/identity/src/credentials/managedIdentityCredential/index.ts#L128-L137
{
      auth: {
        clientId: this.clientId ?? DeveloperSignOnClientId,
        clientSecret: "dummy-secret",
        cloudDiscoveryMetadata:
          '{"tenant_discovery_endpoint":"https://login.microsoftonline.com/common/v2.0/.well-known/openid-configuration","api-version":"1.1","metadata":[{"preferred_network":"login.microsoftonline.com","preferred_cache":"login.windows.net","aliases":["login.microsoftonline.com","login.windows.net","login.microsoft.com","sts.windows.net"]},{"preferred_network":"login.partner.microsoftonline.cn","preferred_cache":"login.partner.microsoftonline.cn","aliases":["login.partner.microsoftonline.cn","login.chinacloudapi.cn"]},{"preferred_network":"login.microsoftonline.de","preferred_cache":"login.microsoftonline.de","aliases":["login.microsoftonline.de"]},{"preferred_network":"login.microsoftonline.us","preferred_cache":"login.microsoftonline.us","aliases":["login.microsoftonline.us","login.usgovcloudapi.net"]},{"preferred_network":"login-us.microsoftonline.com","preferred_cache":"login-us.microsoftonline.com","aliases":["login-us.microsoftonline.com"]}]}',
        authorityMetadata:
          '{"token_endpoint":"https://login.microsoftonline.com/common/oauth2/v2.0/token","token_endpoint_auth_methods_supported":["client_secret_post","private_key_jwt","client_secret_basic"],"jwks_uri":"https://login.microsoftonline.com/common/discovery/v2.0/keys","response_modes_supported":["query","fragment","form_post"],"subject_types_supported":["pairwise"],"id_token_signing_alg_values_supported":["RS256"],"response_types_supported":["code","id_token","code id_token","id_token token"],"scopes_supported":["openid","profile","email","offline_access"],"issuer":"https://login.microsoftonline.com/{tenantid}/v2.0","request_uri_parameter_supported":false,"userinfo_endpoint":"https://graph.microsoft.com/oidc/userinfo","authorization_endpoint":"https://login.microsoftonline.com/common/oauth2/v2.0/authorize","device_authorization_endpoint":"https://login.microsoftonline.com/common/oauth2/v2.0/devicecode","http_logout_supported":true,"frontchannel_logout_supported":true,"end_session_endpoint":"https://login.microsoftonline.com/common/oauth2/v2.0/logout","claims_supported":["sub","iss","cloud_instance_name","cloud_instance_host_name","cloud_graph_host_name","msgraph_host","aud","exp","iat","auth_time","acr","nonce","preferred_username","name","tid","ver","at_hash","c_hash","email"],"kerberos_endpoint":"https://login.microsoftonline.com/common/kerberos","tenant_region_scope":null,"cloud_instance_name":"microsoftonline.com","cloud_graph_host_name":"graph.windows.net","msgraph_host":"graph.microsoft.com","rbac_url":"https://pas.windows.net"}',
      },
    }

Identity Provider
None
Source
Internal (Microsoft)

@MaryGao I suggest asking this question to the folks who own ARM as we are not experts on specific APIs. Please reach out over Teams if you need help finding their support channels


From private repo

Hi @jastru, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;


Support Async Dynamic APIs

Link to reproduction
No response
Environment Info
Binaries:
  Node: 20.16.0
  npm: N/A
  Yarn: N/A
  pnpm: N/A
Relevant Packages:
  payload: 3.0.0-beta.109
  next: 15.0.0-canary.172
  @payloadcms/db-postgres: 3.0.0-beta.109
  @payloadcms/email-nodemailer: 3.0.0-beta.109
  @payloadcms/graphql: 3.0.0-beta.109
  @payloadcms/next/utilities: 3.0.0-beta.109
  @payloadcms/plugin-cloud: 3.0.0-beta.109
  @payloadcms/plugin-nested-docs: 3.0.0-beta.109
  @payloadcms/plugin-redirects: 3.0.0-beta.109
  @payloadcms/plugin-seo: 3.0.0-beta.109
  @payloadcms/richtext-lexical: 3.0.0-beta.109
  @payloadcms/translations: 3.0.0-beta.109
  @payloadcms/ui/shared: 3.0.0-beta.109
  react: 19.0.0-rc-3edc000d-20240926
  react-dom: 19.0.0-rc-3edc000d-20240926
Operating System:
  Platform: win32
  Arch: x64
  Version: Windows 10 Pro
  Available memory (MB): 65459
  Available CPU cores: 24

Done in 3.03s.

Describe the Bug
As of Next.js v15.0.0-canary.171, the following change was introduced:
https://github.com/vercel/next.js/pull/68812
You can read more about the change here:
https://github.com/vercel/next.js/pull/68812
In short, it makes params / headers return promises so you have to await them. Its meant to also allow the previous approach for backwards compatability temporarily. However, it seems like this is not working as intended and it breaking next build (production builds).
The errors stem from the api/payload folder.
Example build error from yarn build:
   Linting and checking validity of types  ...Failed to compile.

.next/types/app/(payload)/admin/[[...segments]]/page.ts:30:29
Type error: Type 'Args' does not satisfy the constraint 'PageProps'.
  Types of property 'params' are incompatible.
    Type '{ segments: string[]; }' is missing the following properties from type 'Promise<SegmentParams>': then, catch, finally, [Symbol.toStringTag]

  28 |
  29 | // Check the prop type of the entry function
> 30 | checkFields<Diff<PageProps, FirstArg<TEntry['default']>, 'default'>>()
     |                             ^
  31 |
  32 | // Check the arguments and return type of the generateMetadata function
  33 | if ('generateMetadata' in entry) {
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Example console errors during yarn dev:
In route /api/[...slug] a param property was accessed directly with `params.slug`. `params` is now a Promise and should be awaited before accessing properties of
the underlying params object. In this version of Next.js direct access to param properties is still supported to facilitate migration but in a future version you will be required to await `params`. If this use is inside an async function await it. If this use is inside a synchronous function then convert the function to async or await it from outside this function and pass the result in.

Reproduction Steps

Use any Payload project using payload 3.0.0-beta.109 and nextjs 15.0.0-canary.171.
yarn dev
Load a /api/ route.
Observe errors in console.
yarn build
Observe type errors that fail the build.

Adapters and Plugins
No response

The workaround until we can issue a fix in Payload is to use "next": "v15.0.0-canary.160" while we work on this.

I modified the Args type in src/app/(payload)/admin/[[...segments]]/page.tsx and src/app/(payload)/admin/[[...segments]]/not-found.tsx to
type Args = {
  params: Promise<{
    segments: string[];
  }>;
  searchParams: Promise<{
    [key: string]: string | string[];
  }>;
};

And it works fine now on the latest Next.js canary. But of course this can be overwritten at any point as mentioned in the file's comment, so not a permanent solution.

@r1tsuu Is it still normal to get a warning after upgrading to Payload 3.0.0-beta.111 and Next.js 15.0.0-canary.173 and react/react-dom 19.0.0-rc-3edc000d-20240926 and applying the necessary changes to the payload files?
When opening the Payload admin, I get:
In route /api/[...slug] a header property was accessed directly with `headers().get('x-forwarded-proto')`. `headers()` now returns a Promise and the return value should be awaited before accessing properties of the underlying headers instance. In this version of Next.js direct access to `headers().get('x-forwarded-proto')` is still supported to facilitate migration but in a future version you will be required to await the result. If this `headers()` use is inside an async function await the return value before accessing attempting iteration. If this use is inside a synchronous function then convert the function to async or await the call from outside this function and pass the result in.

If I upgrade to the latest Next.js canary.179, that message changes to:
In route /api/[...slug] a header property was accessed directly with `headers().get('x-forwarded-proto')`. `headers()` should be awaited before using its value. Learn more: https://nextjs.org/docs/messages/sync-dynamic-apis

I'm getting the same thing as @todor0v , I don't think it's a functional issue because the next team made the API backward compatible, but the console spam is a tad annoying.

@r1tsuu I believe this issue is resolved as of 3.0.0-beta.111 release release and can be closed.
@tomups It is a permanent solution, you change it once and if it is rewritten in the future, Payload will rewrite it using the correct newer syntax as long as you use beta version > 3.0.0-beta.111. The comment is there to let you know you shouldn't add extra misc functionality to the file since it will get overwritten.
@todor0v  /api/[...slug] doesn't seem to be using headers in any way? What does your /api/[...slug] look like? Did you add a custom api route perhaps that is using headers? Search your repo for usages of next/headers and see if all usages are resolving promises.
I am on 3.0.0-beta.111 and Next.js 15.0.0-canary.173 and react/react-dom 19.0.0-rc-3edc000d-20240926 and its working well for me with no errors.

@mobeigi I am on 3.0.0-beta.111, Next.js 15.0.0-canary.173 and react/react-dom 19.0.0-rc-3edc000d-20240926, too (with pnpm) and I'm still experiencing your original issue.
If it's working for you only with @tomups fragment, I don't think this should be closed.

@mobeigi I am on 3.0.0-beta.111, Next.js 15.0.0-canary.173 and react/react-dom 19.0.0-rc-3edc000d-20240926, too (with pnpm) and I'm still experiencing your original issue.
If it's working for you only with @tomups workaround, I don't think this should be closed.

@tomups 's solution is marked as a breaking change that you must implement to upgrade to beta.111.

I still get the errors as well, even though I applied the changes suggested in the changelog. I also reinstalled node modules and deleted nextjs cache. Not sure what else I could try.

@tyteen4a03, oh man, no idea how I overlooked that. After applying the changes everything works fine now, of course. Thanks for your hint.

I still get the errors as well, even though I applied the changes suggested in the changelog. I also reinstalled node modules and deleted nextjs cache. Not sure what else I could try.
edit: Oh, but I have custom views. I assume there is an issue with those. They all use
export type AdminViewProps = {
    readonly clientConfig: ClientConfig;
    readonly importMap: ImportMap;
    readonly initPageResult: InitPageResult;
    readonly params?: {
        [key: string]: string | string[] | undefined;
    };
    readonly searchParams: {
        [key: string]: string | string[] | undefined;
    };
};

which is exported from payload.
Could it be that the type here needs to be updated to reflect the changes regarding promises on params, searchParams, etc?

I'll look into this

This shouldn't be an issue, since we await these before the custom view is rendered.


{Packaging} Fix Mariner docker file to build azure-cli rpm cleanly

Description
Mariner distro azure-cli rpm build is showing issues and not building cleanly because of missing binutils and file packages.
CI for https://github.com/Azure/azure-cli/pull/20584 shows lots of failures:
https://dev.azure.com/azure-sdk/public/_build/results?buildId=1224670&view=logs&j=2d99a3ea-3a85-5049-e806-894016b5c578&t=51344c07-eaf5-5613-2089-f5e3d0d22243&l=732
+ /usr/lib/rpm/brp-strip-unneeded /bin/strip
find: 'file': No such file or directory
find: 'file': No such file or directory
find: 'file': No such file or directory

Adding required binutils and file packages for Mariner Docker file.
Testing Guide
Built locally Mariner variant of azure-cli rpm successfully with no issues.
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Confirmed to be working correctly by merging this PR with https://github.com/Azure/azure-cli/pull/20608 and https://github.com/Azure/azure-cli/pull/20584, and push to https://github.com/Azure/azure-cli/tree/mariner:
https://dev.azure.com/azure-sdk/public/_build/results?buildId=1227537&view=logs&j=2d99a3ea-3a85-5049-e806-894016b5c578


Error after docker-compose

After running make build then sudo docker-compose up --build.
The output is as below
`Building escape-pod-extension
Step 1/3 : FROM scratch
--->
Step 2/3 : COPY escapepodextension /
---> Using cache
---> 66301114a1b6
Step 3/3 : ENTRYPOINT ["/escapepodextension"]
---> Using cache
---> 9652e92d7e7e
Successfully built 9652e92d7e7e
Successfully tagged go_escape-pod-extension:latest
Recreating go_escape-pod-extension_1 ... done
Attaching to go_escape-pod-extension_1
escape-pod-extension_1  | {"Function":"main.main","Source":"main.go:75","level":"fatal","msg":"error during server setup: [invalid client-authentication value "\"NoClientCert\"", valid values [NoClientCert, RequestClientCert, RequireAnyClientCert, VerifyClientCertIfGiven, RequireAndVerifyClientCert]]","time":"2021-03-14 14:20:09"}
go_escape-pod-extension_1 exited with code 1

You have to set up all the environments variables, which are used in there.
Unfortunatelly, it is not well explicitly documented anywhere, so currently you can only read those here. Also in order to properly build you should export them into
$BUILDINFO_FLAGS as it is used by the make/go build.
This is the bare minimum of what you have to set up, change the CHANGEME values:
DDL_RPC_PORT=8089
DDL_RPC_CLIENT_AUTHENTICATION="NoClientCert"
DDL_RPC_INSECURE=true
ESCAPE_POD_WEATHER_API_KEY=CHANGEME
ESCAPE_POD_WEATHER_UNITS=CHANGEME
ESCAPE_POD_WEATHER_ZIP_CODE=CHANGEME
ESCAPE_POD_ROBOT_TARGET=CHANGEME
ESCAPE_POD_ROBOT_TOKEN=CHANGEME
OR...
you can give a damn for the env vars, and apply the attached patch file, so it which will allways statically set them up for your build.


landing updated

layout to landing frame

can you add a route for the landing page to router/index.js . there's an example there if you get stuck


New NoBid Analytics Adapter 1.0.0

🏷 Type of documentation

[x] new Analytics Adapter
[ ] update bid adapter
[ ] new feature
[ ] text edit only (wording, typos)
[ ] bugfix (code examples)
[ ] new examples

📋 Checklist

[x] Related pull requests in prebid.js or server are linked -> Paste link in this list or reference it on the PR itself
[ ] For new adapters check submitting your adapter docs

Hello,
Can we merge this PR? The Analytics Adapter PR has been merged.

@redaguermas - is there a particular reason you don't want users to be able to download the adapter from the download page?
I'll go ahead and merge this, but if you want this to appear on https://docs.prebid.org/download.html, you can submit a new PR changing enable_download to true


Documentation - 404s for endpoint v3.nuxtjs.org/api/configuration/nuxt.config

Environment
Latest
Reproduction
Go into: https://v3.nuxtjs.org/api/configuration/nuxt.config
Get redirect to: https://nuxt.com/docs/api/configuration/nuxt.config
See 404
Links in number 2,100 how much is impact in wild: https://github.com/search?q=api%2Fconfiguration%2Fnuxt.config&type=code
Describe the bug
Open page and not 404
Only need to add redirection proper to new documentation page (decide and add)
Additional context
Big impact for newcomer and from people who insert it to navigate. When you remember then you don't need it
Logs
No response

I do not know if this is linked but the page https://nuxt.com/modules also is broken:
It answers a 500 with the following message:

Error: EMFILE: too many open files, open '/var/task/chunks/error-500.mjs'

Interestingly, the favicon https://nuxt.com/favicon.ico answers with a 200 but the image looks invalid for Firefox:

The image “https://nuxt.com/favicon.ico” cannot be displayed because it contains errors.

@danielroe
The URL should be fixed like below:

Wrong: https://nuxt.com/docs/api/configuration/nuxt.config

Correct: https://nuxt.com/docs/api/configuration/nuxt-config

Wrong: https://nuxt.com/docs/modules

Correct: https://nuxt.com/modules

I can and will fix comments in codebase, but I can't find configuration for redirection, so would you handle it?
The string https://nuxt.com/docs/modules doesn't seem to exist in the codebase.

I will check favicon case later.

Ah, it seems to be have already fixed (at least) in starter templates.

I've added redirects on nuxt.com for those paths. Thank you for reporting.
Soon we will be open-sourcing the repo so it will be easier to fix these kinds of issues ❤


Number Base Converter Extension

Is your feature request related to a problem? Please describe.
I often encounter challenges when working with numbers in different numeral systems, such as binary, octal, decimal, and hexadecimal. Converting between these bases manually is time-consuming and error-prone, especially for complex numbers or long sequences of digits. It would be helpful to have a tool that simplifies this process and ensures accuracy in conversions.
Describe the solution you'd like
I would like a Number Base Converter Extension that provides a convenient way to convert numbers between different numeral systems directly within the browser. The extension should offer a user-friendly interface where I can input a number in one base and instantly see its equivalent representations in other bases. Additionally, it should support bidirectional conversion, allowing me to input numbers in any base and get results in other bases.
Describe alternatives you've considered
Manual Conversion: Performing number base conversions using mathematical formulas or conversion tables, which can be tedious and prone to errors.
Online Number Base Converters: Using existing online tools or calculators for base conversion, but these often require navigating to external websites and may not offer the convenience of a browser extension.
Additional context
Add any other context or screenshots about the feature request here.
Interactive Interface: A visual interface where users can input numbers using text boxes or sliders for precision and ease of use.
Real-Time Updates: Instant conversion results displayed as the user inputs the number, providing immediate feedback.
Customizable Settings: Options to customize the number of digits, padding, and representation formats (e.g., ASCII, Unicode).
Offline Support: Ability to use the extension without an internet connection, ensuring accessibility in all situations.
Educational Resources: Integration with educational materials or tooltips to explain different numeral systems and their relationships.
What problem is this feature trying to solve?
This feature aims to solve the problem of cumbersome and error-prone number base conversions by providing a convenient and reliable tool within the browser. It streamlines the process of converting numbers between different bases, making it easier for users to work with data in various formats and contexts.
How do we know when the feature is complete?
The feature will be considered complete when:

Users can easily input numbers in one base and obtain instant conversions to other bases.
Bidirectional conversion is supported, allowing users to input numbers in any base and get results in other bases.
The interface is intuitive and user-friendly, with interactive controls for input and customization.
Conversion results are accurate and consistent with established mathematical rules for number base conversion.
Additional features such as customizable settings and educational resources enhance the user experience.
Feedback from users indicates increased efficiency and satisfaction with number base conversion tasks.

Already added to my repository #1255. Closing this @Dark-S8uL


feat: Add support for pre_navigation_hooks in PlaywrightCrawler

Description
This implementation adds support for preNavigationHooks in the PlaywrightCrawler
Use the new pre_navigation_hooks:
`async def example_hook(context: PlaywrightCrawlingContext, goto_options: dict) -> None:
await context.page.evaluate("window.foo = 'bar';")
goto_options['timeout'] = 60000  # Set a custom timeout for navigation
crawler = PlaywrightCrawler(
# other options...
pre_navigation_hooks=[example_hook]
)`
Issues
#427

[ ] CI passed

@janbuchar Why is this not merged

@janbuchar Why is this not merged

because it was submitted just before the weekend
because we want to review the code first

We value your contribution, but please, keep your tone civil.

And let me add a third reason, there are no tests nor documentation for this new feature.
Also, this option should be available for all crawler types, not just playwright.

@janbuchar Why is this not merged

because it was submitted just before the weekend
because we want to review the code first

We value your contribution, but please, keep your tone civil.

Hi Mate I am really sorry for what is said i really did not mean it that way.

Closing in favor of #631


Update design based on Figma

Includes UI changes from this Figma file: https://www.figma.com/design/UKdLtH0isKn0jagjrqaVN3/Anoncast?node-id=18-1193&node-type=canvas&t=BhDVuTNSZrL0UmpX-11
Screenshots (videos were too big):

Future things to address
[] Loading states (or server components) should be added for waiting for the wallet address
[] OnSuccess, failure of a post
[] Actual amount of Anons who can post

Thanks @jbreite !


⚠ St. Lawrence Global Observatory CKAN has degraded performance

In 1bb4d42, St. Lawrence Global Observatory CKAN (https://catalogue.ogsl.ca/en/) experienced degraded performance:

HTTP code: 200
Response time: 6531 ms

Resolved: St. Lawrence Global Observatory CKAN performance has improved in 9d71192 after 2 hours, 45 minutes.


Loading GIF into Collection View takes to much time.

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[x] I have read the wiki page and cheat sheet, but there is no information I need.
[x] I have searched in existing issues, but did not find a same one.
[ ] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
What
I'm using Kingfisher to load GIF from Firebase Storage and it's taking a lot of time to show up. I have read the cheat sheet and I'm using AnimatedImageView like it says but still takes a lot of time. I want to know is there a way to optimize it or something.
Reproduce
Here is the code I'm using to set up the GIF into the UICollectionViewCell:
            strongSelf.postImageView.kf.setImage(with: url,options: [.scaleFactor(UIScreen.main.scale),.cacheOriginalImage],progressBlock: { (receivedSize, totalSize) in
                let percentage = (Float(receivedSize)/Float(totalSize)) * 100.0
                print("dowloading progress: \(percentage)%")

            })

Other Comment

maybe you can show your image link，sometimes ist's just caused by a large data


Not able to import enums from d.ts

Hey,
I setup a ts node project, but facing this error while importing DhanEnv enum from "dhanhq"
Error:
SyntaxError: The requested module 'dhanhq' does not provide an export named 'DhanEnv'
    at ModuleJob._instantiate (node:internal/modules/esm/module_job:123:21)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async ModuleJob.run (node:internal/modules/esm/module_job:189:5)

Node.js v19.8.1

I also found a lot of issues reported on stackoverflow regarding this.
Would love if you could you share a working tsconfig.json
Here is my tsconfig:
{     "compilerOptions": {       "module": "es6",       "esModuleInterop": true,       "target": "es6",       "moduleResolution": "node",       "sourceMap": true,       "outDir": "dist",       "allowSyntheticDefaultImports": true,     },     "include": ["**/*"],     "exclude": ["dist"],     "lib": ["es2015"] }

Update: resolved with including correct lib and other compiler options


breakpoints ignored in vscode for files generated by the cpp2

Using clang14, clang16,
When I place a breakpoint in the build/cppfront/whatEver.cpp it is ignored.
Normal breakpoints in cpp (none cpp2) are working fine.
About the system

macos
clang14 and clang16

I don't think that's a problem with this project. That sounds like a problem with VSCode not honoring breakpoints on generated files. You can try asking on their issue tracker, here: https://github.com/microsoft/vscode/issues

See also https://github.com/hsutter/cppfront/discussions/589.

Thanks so much, I can confirm after setting vscode File association it is now working.


mcapi: add files recursively

It would be convenient to have a function that adds (uploads) files recursively from some local directory to some remote directory. Could be a member of Project, or Directory, or standalone? Then returns a list of File objects so they could be, for example, added to a Process.
Example:
def add_files_recursively(project, remote_dir, local_dirpath):
    """
    Arguments:
        project: mcapi.Project object
        remote_dir: mcapi.Directory object, where contents of local_dirpath
            should be uploaded to
        local_dirpath: path to local directory to add files from, recursively

    Returns:
        files: List of mcapi.File objects
    """
    ... code ...

This is now covered by the API, the function name is different: (currently) Directory.add_directory_tree(dir_name,input_dir_path,verbose=False)
returns the base directory, updated.


node-tar pack should allow the uid+gid to be overwritten

A common use case when creating a tar is to override the source filesystem uid+gid with 0:0 when creating the tar entries, so that when the tar is extracted by a root user it will correctly be written with root:root as the owner/group, rather than inheriting the uid+gid stored with the .tgz
e.g.,
$ tar -zcv --owner=root --group=root --numeric-owner -f /tmp/filename.tgz package/
$ cd /tmp && sudo tar -zxf /tmp/filename.tgz
$ ls -l package/*
-rw-r--r-- 1 root root package/package.json
etc.

Currently if I create a tar using node-tar and distribute it to someone who untars it as root, the files will be written with whatever uid/gid my user happens to have on the machine on which I created the tar

Also affects JSPM https://github.com/jspm/jspm-cli/issues/1902.

Looks like this needs an upstream change as well, since the chmod is done in https://github.com/npm/fstream/blob/master/lib/writer.js#L194:L212.

In v3, this will be possible by setting { preserveOwner: false } when extracting.  (Or, to omit the uid/gid entirely, you can pack with { portable: true }.)

v3 is shipped.


"label can only be part of a statement" error

Hi -  I've just found tabulate - it sounds awesome!
Unfortunately, I got the following error when running "make all" -
CC    build/main.o
src/main.c: In function ‘PrintDocument’:
src/main.c:2296:17: error: a label can only be part of a statement and a declaration is not a statement
2296 |                 struct cell *TopCell = GetCell(Doc, Col, 0);
|                 ^~~~~~
make: *** [Makefile:47: build/main.o] Error 1
I'm running Linux Mint.  gcc version 9.4.0  ( Ubuntu 9.4.0-1ubuntu1~20.04.1 ).  I'm very happy to help with bugfixes, either this one or any others!  Keep up the great work!

mooseman

Thanks for stopping by. Though I wonder how you found this project at all.
It looks like the problem is that strict C doesn't permit a variable declaration to follow a label, even a switch case. In hindsight I knew this, but GCC 12 accepted it without complaint. Clang still chokes on it. I've pushed a change that should let you build the project.
As a side note, I'm noticing that I never published any example sheets for this program. As I recall the problem is that nearly all the sheets I write for this program contain my finances, which I'd rather not put on GitHub. Instead, here's  a sheet I wrote while filling a character's inventory for a Pathfinder 1e game. Just save this as a text file then pass it as the argument. e.g. tabulate items.txt
items.txt
Price	Weight	Count	Cost	Weight	Item
#:fmt 6.2	6.3	6.0	6.2	6.3	45

5.00	=1/2	1	=!Cost	=!Wght	Dagger (1d3)
35.00	=4/2	1	=!Cost	=!Wght	Crossbow, light (1d6)
0.10	0.1	20	=!Cost	=!Wght	Crossbow bolts
3.00	=1/2	1	=!Cost	=!Wght	Haramaki (not proficient)
#
5.00	=3/4	1	(free)	=!Wght	Outfit, Pickpocket's
1.00	?	2	=!Cost	?	Jewelry
#
2.00	=2/4	1	=!Cost	=!Wght	Backpack (1/2 cb.ft.)
0.10	=5/4	1	=!Cost	=!Wght	Bedroll
1.00	=0.5/4	1	=!Cost	=!Wght	Pouch, Belt (1/20 cb.ft. or 2.5 lb)
1.00	0	1	=!Cost	=!Wght	Flint and steal
8.00	0	1	=!Cost	=!Wght	Ink (1 oz.)
0.10	0	1	=!Cost	=!Wght	Inkpen
0.20	0	10	=!Cost	=!Wght	Parchment sheet
#0.80	4	1	=!Cost	=!Wght	Pot, iron (1 gal)
0.20	1	1	=!Cost	=!Wght	Kit, mess
0.01	0.5	1	=!Cost	=!Wght	Soap
5.00	2	1	=!Cost	=!Wght	Pouch, spell components
#0.01	1	10	=!Cost	=!Wght	Torch
0.50	=1/4	5	=!Cost	=!Wght	Rations, trail
1.00	=4/4	1	=!Cost	=!Wght	Waterskin (1/8 gal)
#
0.10	0	1	=!Cost	=!Wght	Dice
0.10	1	1	=!Cost	=!Wght	Cards (simple)
0.10	0	1	=!Cost	=!Wght	Dominoes (bone)

			=sum br	=sum br
#:summary D@
			=80-@^	=17-@^
#:define Cost A@*C@
#:define Wght B@*C@
# vim: noet ts=10 sw=10 tw=0

Hi levirak -  great to hear from you!
I'm a big fan of "public domain" software (and a big "thank you" for
putting tabulate under the Unlicense!).
I do searches of "Unlicense" software from time to time - that would be how
I found "tabulate".  :)
Many thanks for doing the fix!  I'll grab the latest code, have another go
at the build and will let you know how it goes!
Cheers - keep up the great work!
Andy
On Fri, Mar 3, 2023 at 8:56 PM levirak @.***> wrote:

Closed #1 https://github.com/levirak/tabulate/issues/1 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/levirak/tabulate/issues/1#event-8657124811, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAUZCZMENNAMXK4RYLXWWDW2GP3HANCNFSM6AAAAAAVHTV25A
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi again levirak!
Great news -  I got the latest code with your fix and the build now works perfectly!
Good stuff!  Many thanks for the fix and keep up the great work!

mooseman


Cork logic fix for browsers

I don't understand this change. It should also re-apply those changes I reverted.
Also, do you know how it fixes #148?

@mcollina, I have no idea what breaks him logic. I don`t change default logic. You can see it in code.

@mcollina, need more info from @binarykitchen.

@mcollina, so, what we are doing?

We wait for @binarykitchen - I had some doubt on that change from the beginning however I couldn't pinpoint the issue. Seems that change is breaking.
(This PR is not complete - it should re-revert my change).

if try simple test with @binarykitchen settings - it passed
test('echo works', function(t) {
  var stream = ws('ws://localhost:8343', {objectMode: true, perMessageDeflate: false})
  stream.on('data', function(o) {
    t.ok(Buffer.isBuffer(o), 'is buffer')
    t.equal(o.toString(), 'hello', 'got hello back')
    stream.destroy()
    t.end()
  })
  stream.write(Buffer.from('hello'))
})

I fixed situation @binarykitchen. But I confused, because in this case I get separate one packet on two socket frames anytime.
First frame of packet:

Second frame of packet:

Maybe @mcollina you know what`s going on?


fix(android): Unreported IOException on MediaMetadataRetriever.release() with target SDK 33

Motivation (required)
I've changed the target sdk version to 33 (Tiramisu) on my RN app, but during the build, gradle complains about the unreported IOException caused by MediaMetadataRetriever.release().
The exception was added in 33 (Tiramisu) as reported in the official docs: https://developer.android.com/reference/android/media/MediaMetadataRetriever#release()
I think that adding a try catch should be the best approach to this, so that if for whatever reason the release throws an exception (already released by system, etc), the app does not crash. I used a warn instead of an error since at that point the release is not much relevant to the functionalities (if an exception is thrown, the metadata retriever probably has been already released by the system).
Test Plan (required)
You can test it by targetting to SDK 33 (Tiramisu) and build any app.

Hi!
What is the status of this PR? This is a pretty critical bug.

Concurred we can't update our app currently due to this.

Closing, as this was changed part of #1955 as mentioned above.


[BUG] Update inactive domains when importing

Describe the bug
Domains are being shown as active even though the domain hosting provider does not include them in their list of active domains.
Expected behavior
When importing the dataset containing all active domains, it is expected that domains that are not in the list are being set to inactive.
Actual behavior
When importing new domains from the dataset of the hosting provider, it simply updates domains to active and doesn't take action on the other domains.

 Add cache busting feature when IP ranges are marked as inactive


Cannot run npm install

When running npm install I get the following error
D:\Temp\node_modules\chartjs-chart-pcp>npm install

<EMAIL_ADDRESS>prepare D:\Temp\node_modules\chartjs-chart-pcp
npm run build:dev

<EMAIL_ADDRESS>build:dev D:\Temp\node_modules\chartjs-chart-pcp
rollup -c -i src/index.js

[!] Error: Could not resolve entry module (rollup.config.js).
Error: Could not resolve entry module (rollup.config.js).
    at error (D:\Temp\node_modules\chartjs-chart-pcp\node_modules\rollup\dist\shared\rollup.js:161:30)
    at ModuleLoader.loadEntryModule (D:\Temp\node_modules\chartjs-chart-pcp\node_modules\rollup\dist\shared\rollup.js:17598:16)
    at process._tickCallback (internal/process/next_tick.js:68:7)
    at Function.Module.runMain (internal/modules/cjs/loader.js:834:11)
    at startup (internal/bootstrap/node.js:283:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:622:3)

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build:dev: `rollup -c -i src/index.js`
npm ERR! Exit status 1

can you describe how to reproduce it step by step, please

From an elevated DOS command prompt in Windows 10:-
md a
cd a
npm install --save chart.js chartjs-chart-pcp

(wait for it to download)
cd node_modules
cd chartjs-chart-pcp
npm install

starts throwing up a lot of deprecated errors....then this:-
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-babel.
npm WARN deprecated<EMAIL_ADDRESS>rollup-watch functionality is now included in Rollup itself
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-commonjs.
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-node-resolve.
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.
npm WARN deprecated<EMAIL_ADDRESS>fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142

><EMAIL_ADDRESS>prepare D:\a\node_modules\chartjs-chart-pcp
> npm run build:dev

><EMAIL_ADDRESS>build:dev D:\a\node_modules\chartjs-chart-pcp
> rollup -c -i src/index.js

[!] Error: Could not resolve entry module (rollup.config.js).
Error: Could not resolve entry module (rollup.config.js).
    at error (D:\a\node_modules\chartjs-chart-pcp\node_modules\rollup\dist\shared\rollup.js:161:30)
    at ModuleLoader.loadEntryModule (D:\a\node_modules\chartjs-chart-pcp\node_modules\rollup\dist\shared\rollup.js:17598:16)
    at process._tickCallback (internal/process/next_tick.js:68:7)
    at Function.Module.runMain (internal/modules/cjs/loader.js:834:11)
    at startup (internal/bootstrap/node.js:283:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:622:3)

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build:dev: `rollup -c -i src/index.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\Paul McGuinness\AppData\Roaming\npm-cache\_logs\2020-05-26T13_40_29_722Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>prepare: `npm run build:dev`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prepare script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\Paul McGuinness\AppData\Roaming\npm-cache\_logs\2020-05-26T13_40_29_830Z-debug.log

D:\a\node_modules\chartjs-chart-pcp>

cd node_modules
cd chartjs-chart-pcp
npm install

why this step?

2020-05-26T13_40_29_722Z-debug.log
Here is the npm log file

because that's what your instructions say?
Building
npm install
npm run build
I'm just after a normal .js file to include in a web-page front end, not a back-end Node module.
Do please tell me if I'm being a muppet :-)

( the examples shows ../build/Chart.PCP.js   but I can't find the 'build' version, so I assumed that's what I had to do?)

those instruction are in case you want to build the library in case you check it out the repository and develop it yourself.
In your case you can just download the file from:
https://github.com/sgratzl/chartjs-chart-pcp/releases -> assets -> Chart.PCP.js
or link to it via a CDN like unpkg in your website
https://unpkg.com/chartjs-chart-pcp
or take the installed built files in your case in: node_modules/chartjs-chart-pcp/build/Chart.PCP.js

Many thanks - I'm glad I was just being a muppet :-)


chore: Updating the version of the design system dependency

Description

Updating the version of the design system dependency

Fixes #18487
Type of change

Version update

How Has This Been Tested?

Tested on GAC feature branch. Works as expected.

Manual

Checklist:
Dev activity

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

QA activity:

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

/ok-to-test sha=a617d00


Incorrect Monster Types

I spotted two mobs that were listed as the incorrect type (when referenced against my copy of the latest version of the book).
Bastet Temple Cat is labeled as a Beast but is actually a Monstrosity.
Cikavak is labeled as a Beast but is actually a Fey.

They're beasts in the PDF I have. I'll check the book when I get home to see if the book differs from the PDF.

Book has them as you said. Looks like their types were errated, and the book is correct.
I've updated them. Thanks!


Add method Skip.When()

Fix #122
Add method Skip.When() to be able skip validation based on condition.
It' necessary in cases of nested structs validation. Because When(..., Skip) is not the same as Skip.When(...) (see #122).

Coverage increased (+0.003%) to 99.336% when pulling 06f4b6f78c9c31a39b369e2d66950e37aceb7e65 on maratori:skip-when into 6a5d431825ff01ddbdfe521883db9e7a514a7664 on go-ozzo:master.

Excellent work! Thank you very much!


Logging Solution for K8s

Feature:
Logging solution for the K8s cluster (Allen)
Reason:
As a infrastructure team, we want to proactively monitor the state of our infrastructure, so we can stay ahead of potential website outages.
Solution:
TBD
Alternate Solution:
TBD

cc @AllenAnthes

I'd like to tackle this one.


[plugin_platform_interface] Don't use const Object as a token

Description
Platform interface tokens must be unique, we've mistakenly used const Objects as a token (and all const Objects are identical).
Thanks @buntagonalprism for noticing and investigating!
We should probably follow-up with the PlatformInterface forcing tokens that are not const Objects (though this will be a breaking change).
I'm not sure whether we can test the failing scenario as it will require running tests in release mode.
Related Issues
https://github.com/flutter/flutter/issues/46941
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the [Flutter Style Guide].
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the [CLA].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

You can also check and throw an exception if token == const Object() since this could be a common mistake.

Yes, I mentioned this in the PR description, I'll followup in a separate PR as we should make sure existing plugins are already updated (I think it's early enough to assume the currently only first party plugins are using this)

Actually this doesn't work as now the token doesn't test equal for the different instances of MockPlatformInterface, fixing...


Delete GitHub environment along with review app

Context
A GitHub environment is created for each review app that is deployed, this environment persists after the review app is deleted.
Changes proposed in this pull request

Add a step to the delete-review-app workflow to delete the environment

Guidance to review

Close PR to test delete process, reopen PR once tested

Trello card
https://trello.com/c/ybnYhCK3
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

Testing delete-review-app workflow

Re-testing delete-review-app workflow


CosmoPower

For fun I tried to see if I could get cosmopower working following instructions, but did not get far- looks quite out of date? e.g. presumably data files should be linked to https://github.com/cosmopower-organization/jense_2024_emulators rather than original file, and unclear how one is supposed to use a TT+EE+TE likelihood with all the separate network files.
(As previously discussed, I guess ideally it does not need to be in SOLikeT at all if the cobaya component is included in the cosmopower package)
When I try to run the example on the CosmoPower home page I also fail with this
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
Cell In[2], line 7
      4 import cosmopower as cp
      6 # load pre-trained NN model: maps cosmological parameters to CMB TT log-C_ell
----> 7 cp_nn = cp.cosmopower_NN(restore=True,
      8                          restore_filename=r'C:\Work\Dist\git\SO\SOLikeT\soliket\cosmopower\data\CP_paper\cmb_TT_NN')
     10 # create a dict of cosmological parameters
     11 params = {'omega_b': [0.0225],
     12           'omega_cdm': [0.113],
     13           'h': [0.7],
   (...)
     16           'ln10^{10}A_s': [3.07],
     17           }

File ~\anaconda3\envs\py12\Lib\site-packages\cosmopower\cosmopower_NN.py:68, in cosmopower_NN.__init__(self, parameters, modes, parameters_mean, parameters_std, features_mean, features_std, n_hidden, restore, restore_filename, trainable, optimizer, verbose)
     66 # restore
     67 if restore is True:
---> 68     self.restore(restore_filename)
     70 # else set variables from input arguments
     71 else:
     72     # attributes
     73     self.parameters = parameters

File ~\anaconda3\envs\py12\Lib\site-packages\cosmopower\cosmopower_NN.py:285, in cosmopower_NN.restore(self, filename)
    278 # load attributes
    279 with open(filename + ".pkl", 'rb') as f:
    280     self.W_, self.b_, self.alphas_, self.betas_, \
    281     self.parameters_mean_, self.parameters_std_, \
    282     self.features_mean_, self.features_std_, \
    283     self.n_parameters, self.parameters, \
    284     self.n_modes, self.modes, \
--> 285     self.n_hidden, self.n_layers, self.architecture = pickle.load(f)

ModuleNotFoundError: No module named 'tensorflow.python.training.tracking'

Which tensorflow version are you running with?
Currently, cosmopower requires:
tensorflow<2.14
tensorflow_probability<0.22

see also: https://github.com/alessiospuriomancini/cosmopower/issues/22

I do have tensorflow 2.17. But I did do a clean install. Specifically I did
conda create -n cosmopower python=3.12 scipy matplotlib
activate cosmopower
pip install cosmopower
pip install ipykernel jupyterlab
python -m ipykernel install --user --name=cosmopower

However, running cosmopower now gives
Failed to import TF-Keras. Please note that TF-Keras is not installed by default when you install TensorFlow Probability. This is so that JAX-only users do not have to install TensorFlow or TF-Keras. To use TensorFlow Probability with TensorFlow, please install the tf-keras or tf-keras-nightly package.
This can be be done through installing the tensorflow-probability[tf] extra.

So I did
pip install tensorflow-probability[tf]
Perhaps a version-tagged version of tensorflow-probability[tf] needs to be in requirements?

Converting all pkl to numpy does seem like a good idea (https://github.com/dpiras/cosmopower-jax/pull/5)

This has now been fixed in commit 65fa9d6 and incorporated in the new v0.2.0 release. Note that python versions up to 3.11 are supported. python=3.12 is currently in bugfix and support for it will likely be added in https://github.com/alessiospuriomancini/cosmopower/pull/28.

Thanks! (btw, Anaconda's default download is now python 12)


Import and/or client order now appears to matter between boto3 and moto

I was eagerly expecting #1907 and moto 1.3.7 to resolve my failing tests... however, even with all relevant modules up to date, I'm still getting botocore.exceptions.ClientError: An error occurred (InvalidClientTokenId) when calling the XXX operation: The security token included in the request is invalid.
(for the record, here's my current module versions:)
boto==2.49.0
boto3==1.9.37
botocore==1.12.37
moto==1.3.7

Trying to reproduce the problem and narrow it down led me to an odd test case. For me, the following code completes successfully:
import boto3
from moto import mock_iam
iam_resource = boto3.resource('iam')

with mock_iam():
    user = iam_resource.create_user(UserName='john')
    print(user)

While this results in the ClientError mentioned previously:
import boto3
iam_resource = boto3.resource('iam')
from moto import mock_iam

with mock_iam():
    user = iam_resource.create_user(UserName='john') # throws ClientError
    print(user)

Note that the only difference is that the mock_iam import now appears after the definition of iam_resource.
Of course, you may think as I did, we may just need to re-initialize the iam_resource instance - but sadly that doesn't work:
import boto3
iam_resource = boto3.resource('iam')
from moto import mock_iam

with mock_iam():
    iam_resource = boto3.resource('iam')
    user = iam_resource.create_user(UserName='john') # still throws ClientError
    print(user)

For me, this is the problem - in my test suite, I have one test file which loads boto3 resources but no mocks, then subsequent test files load the mocks before the boto3 resources, but ClientError is still thrown. If I run pytest with just one test file at a time, all tests pass.
Is this a case of user error? Is there some way I can "reset" boto3's clients and resources after they've first initialized, so that moto can intercept the requests properly? Or is this a moto bug?

I think I've figured out how to work around this issue. Calling boto3's setup_default_session() method causes the default session to be regenerated, with moto's hooks in place if moto has been activated.
The following works:
import boto3
iam_resource = boto3.resource('iam')
from moto import mock_iam

with mock_iam():
    boto3.setup_default_session()
    iam_resource = boto3.resource('iam')
    user = iam_resource.create_user(UserName='john') # succeeds
    print(user)

If this is a FAQ, it might need to be added to the documentation.

Also ran into this issue, thanks so much for your setup_default_session workaround. I would definitely say this is a moto issue.

I have the same problem, except the workaround above does not work for me... Changing the order of imports doesn't fix it either. ☹
Versions:
boto                         2.49.0
boto3                        1.9.118
botocore                     1.12.118
moto                         1.3.7

I'm on Python 2.7, in case that matters at all.

I had to refactor all my package code to use lazy boto3 resource instantiations.  Then everything works OK, even without the setup_default_session option.  On gitlab CI, providing bogus environment variables for the AWS credentials worked OK too.
So instead of:
@dataclass
class S3Util:
    client = boto3.resource('s3')

Use properties to return on-demand and garbage collect them.  Let boto3 manage the resource pools etc. and don't hang on to them.  Something like:
class S3Util:
    @property
    def client(self):
        return boto3.resource('s3')

adding in boto3.setup_default_session() insode of the test with the @decorator/@mock_sns worked fine for me.
@mock_sns
def test_sns():
    boto3.setup_default_session()
    sns_client = boto3.client(service_name='sns', region_name='us-west-2',
                              aws_access_key_id='foo', aws_secret_access_key='boo')

    arn = sns_client.create_topic(Name='test-topic')```

Hi,
I am still getting the error for ecr:
botocore.exceptions.ClientError: An error occurred (UnrecognizedClientException) when calling the DescribeRepositories operation: The security token included in the request is invalid.

and, I am also getting:
/home/x/proj/lib/python3.6/site-packages/requests/__init__.py:80: RequestsDependencyWarning: urllib3 (1.23) or chardet (3.0.4) doesn't match a supported version!
  RequestsDependencyWarning)
Performing system checks...

I am getting this error only when I add moto to requirements.txt
If I run pip install moto after running pip install -r requirements.txt, I am not getting both of the errors.
boto3==1.9.88
botocore==1.12.0
moto==1.3.8

I solved it using
botocore==1.12.85
git+https://github.com/spulec/moto.git@df493ea18de22b4533073eee3e296686019911c2

Installing via commit till moto>1.3.8

As mentioned, this should be fixed in master, but feel free to reopen if there are still issues.

I ran into this today and it was solved using the boto3.setup_default_session() call. I'm not really sure where things were going wrong exactly but it lead me here. I'm using fairly new stuff:
moto==1.3.14.dev326
boto3==1.9.81
botocore==1.12.81

@spulec Do you know which commit fixes this so I can see if I'm using the right versions?

Feel free to try the very latest version: 1.3.15.dev393

@spulec I'm also seeing this using the latest master, but can't call boto3.setup_default_session() since I'm using Terraform, not Python. Any suggestions?


Help adding a command option

Hello and thank you for this project. I've forked it and I'm trying to add a command option [-b BEGIN, --begin BEGIN]. This option uses the ffmpeg option -ss to begin the comparison X seconds into the inputs. I should be very close, but there is small error. Would you please guide me on completing this?
The error I get is:
syncstart.py", line 71, in normalize_denoise
in_out(ffmpegwav%take,infile,outfile)
TypeError: not enough arguments for format string
I previously tried making that as follows, but it did not work. I'm not sure where %take comes from.
in_out(ffmpegwav%begin%take,infile,outfile)
Here is my forked version:: https://github.com/drolex2/syncstart/blob/main/syncstart.py

Since you have added another %s to ffmpegwav,
you should do ffmpegwav%(begin,take) instead of just ffmpegwav%take.

Thanks! It's working.


make argv and envp uvwasi_options const

This was suggested in one of the PRs in the nodejs/wasi. Feel free to close if you disagree with doing this.

With this project still being under active development, this could still change. For now though, I think this change makes sense. Can you remove include/.uvwasi.h.swp from the PR? Thanks!

Can you remove include/.uvwasi.h.swp from the PR?

Doh, sorry about that. I've removed it now (got it in my local exclude but I must have committed this before).

Landed in https://github.com/cjihrig/uvwasi/commit/94c25bf881eb7ba758c62b1709c10baddf9cfb52. Thanks!


Update US Core v4 export and enforce physiology usage more consistently.

This PR updates the FHIR R4 exporter with US Core v4 profile names.
Note that US Core v4 changes are already implemented in master about 95% of the way, this PR cleans up some corner cases.
It also fixes a transient exporter test bug associated with the physiology state and exporting FHIR SampledData. These changes enforce physiology to be used for unit tests in order to check this corner case.

This PR is ready for review.


website responsive

I have made the website responsive . So kindly review it.

It would be great  if you could add some screenshots for me to review.

I have uploaded the screenshots at different screen size . So kindly review it.

This looks great to me! Thank you for doing this. I will merge it.

Okay thank you


After last AlpineJs upgrade, modals are not displaying anymore

This new release:
https://github.com/alpinejs/alpine/releases/tag/v3.13.6
introduces a breaking bug: a "display:none" is injected in the code (in this div: ) and makes modals impossible to display (you can only see the dark background).
I "solved" (but it's not a fix, of course) by manually forcing the import to the previous release:
from:
<script defer<EMAIL_ADDRESS>to:
<script defer<EMAIL_ADDRESS>

I racked my brain for a while until I found this problem, I wish I had seen your post here straight away, I only saw it when I came to publish the problem haha

Looking into it. More reports are coming in about v3.13.6 (https://github.com/alpinejs/alpine/discussions/4080)

@giovanniledda  are you using Livewire 2 or 3?

@giovanniledda can you try the just-tagged Alpine v3.13.7 and see if that solves the issue?

It's ok for me after the v3.13.7 release.
I was using Alpine from<EMAIL_ADDRESS>

@giovanniledda can you try the just-tagged Alpine v3.13.7 and see if that solves the issue?

Nice...this version solves the issue.
I'm using Livewire 2, by the way.
Other projects, where I'm using Livewire 3,  don't suffer from the issue.

Yeah, it's been a bit of a search, been trying to figure this out the entire morning to figure out where exactly the bug exists and it seems to be isolated to Livewire v2 in combination with Alpine v3.13.6. The change in this version stopped Alpine from picking up any morph changes or in this case, it did not remove display:none anymore.
Happy that it has been resolved.


Update Databricks RAPIDS integration example

https://github.com/rapidsai/cloud-ml-examples/blob/main/databricks/README.md Looks to be outdated as it is for DataBricks ML Runtime 6.6 which is not supported now, can we please update the init script for latest RAPIDS and Databricks Runtime version?

I was able to build a GPU conda image with CUA 11 (zronaghi/rapidsai:databricks-conda), using Databrick's Dockerfiles and start up a cluster on Azure. But all notebook commands get canceled. @drobison00 can you also look into it?

Here's the latest: DB recently updated containers with CUDA 11. After looking at the logs and GH, need to set the default conda env in the Dockerfile and couple of packages (ipython and ipykernel) are required for Python notebooks. I was able to access the GPUs and run a few commands with https://github.com/rapidsai/cloud-ml-examples/pull/120, working on adding RAPIDS libs.


WIP: Fix traceback that is caused on closing the browser since it fires a …

This fixes a traceback that happens when you change the close the InAppBrowser without closing the whole app.
2020-08-07T06:05:56.044Z - j5 SAPUI - ERROR - An error occurred at line 312 of ms-appx-web://j5international.j5offlinedesktopapp2019/www/cordova.js: Unable to get property '_eventHandler' of undefined or null reference
	cb()@ms-appx-web://j5international.j5offlinedesktopapp2019/www/plugins/cordova-plugin-inappbrowser/www/inappbrowser.js:112:13
	cordova.callbackFromNative()@ms-appx-web://j5international.j5offlinedesktopapp2019/www/cordova.js:291:21
	cordova.callbackSuccess()@ms-appx-web://j5international.j5offlinedesktopapp2019/www/cordova.js:271:9
	onSuccess()@ms-appx-web://j5international.j5offlinedesktopapp2019/www/cordova.js:1024:17
	Anonymous function()@ms-appx-web://j5international.j5offlinedesktopapp2019/www/plugins/cordova-plugin-inappbrowser/src/windows/InAppBrowserProxy.js:107:21
eval code (334) (18,40)

0: Unable to get property '_eventHandler' of undefined or null reference

This issue has been resolved on master and I pulled the diff in to fix it.
TODO: This hasn't been tested yet.

Note: This was originally implemented by @rorygrandin in #8, also on version 3.0, before master was upgraded to 3.2

Close this in favor of https://github.com/j5int/cordova-plugin-inappbrowser/pull/14


Builds and case-insensitive filesystems

Please, do something with files:
build/Griddle.js
build/griddle.js

On OS X and probably Windows they are pointing to the same file and one overrides another. So, you always have uncommited changes. It isn't a problem on OS X since you can use disk image with case-sensitive file-system, but it is if you use npm to install git repo on case-insensitive fs, because it should first checkout master and then switch to specified branch.

Thank you for reporting this -- we need to remove the uppercase one.

+1 - any news on this?

This should no longer be an issue. Please feel free to open a new issue if any problems are encountered. 👍


Added INHIBITOR_NSFW and COMMAND_CONF_GUARDED in fr-FR

Should I use the changes made in #SettingGateway-Rewrite as well?

SGv2 changes are done in another PR.

I mean the fr-FR-related changes

Nah, it's fine.


0209 -  Minimum Size Subarray Sum

Description of the Problem
Given an array of n positive integers and a positive integer s, find the minimal length of a contiguous subarray of which the sum ≥ s. If there isn't one, return 0 instead.
Code
class Solution:
def minSubArrayLen(self, s, nums):
"""
:type s: int
:type nums: List[int]
:rtype: int
"""
    # approach:
    #     1. build a prefix list and iterate it
    #     2. at each iteration, use binary search to find prefix+s
    #     3. after search done, get length between left and i

    n = len(nums)
    result = n + 1

    if n == 0:
        return 0

    # build prefix sums array
    prefix = [0 for i in range(n+1)]
    for i in range(1, n+1):
        prefix[i] = prefix[i-1] + nums[i-1]

    # for each iteration i, we try to use binary search to find
    # prefix[i] + s, which means the sum of subarray between i
    # and left will be more than s
    for i in range(n):
        left = i + 1
        right = n
        target = prefix[i] + s

        while left <= right:
            m = left + (right - left) // 2

            if prefix[m] < target:
                left = m + 1

            # in both greater and equal cases, we have to update
            # right index for finding the leftist sum which is
            # larger than target
            else:
                right = m - 1

        # end for-loop immediately if left is greater than n because
        # it means every sum will be less than targer in following iteration
        if left > n:
            break

        # if left is valid, left will be the leftist element that is
        # larger than target, so length between left and i is what we need
        if left - i < result:
            result = left - i

    return 0 if result > n else result

Link To The LeetCode Problem
LeetCode

Please assign me this problem to submit a PR

@flick-23 Waiting for your PR 😸


[Question]:  Is there a way to call function tool from router ?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Is there a way to call function tool from router in llamaindex?

There is not currently. Could be considered in a PR, although the router query engine assumes each tool has one input (a query)


Create a new socket each time we report to graphite.

Hi @adamdecaf - when I was looking at this recently, I was thinking maybe each call to GraphiteReporter.report should result in one socket (udp or tcp) being opened to the Graphite server, used for all metrics being reported (i.e. each call to GraphiteSender.send) and then closed when all metrics have been reported.
I'm assuming that GraphiteReporter.report gets called once per minute (or whatever is configured) with params containing every single metric in the entire system, sends all metrics to Graphite really quickly, then does nothing again for another minute until it's called again. Does that seem accurate?
If that's the way it works, it might be good to reuse that socket/connection throughout the same GraphiteReporter.report call since there could presumably be lots of metrics sent to Graphite in that call. It might be costly to open/use/close a single socket/connection for each individual metric to send.
The GraphiteSender API doesn't make it straightforward to take this approach though. GraphiteReporter.report seems to call isConnected and connect first, then call send many times, and then finally calls flush at the end, so maybe those calls could be used in a GraphiteSender impl to create, use and then close the socket per single call to GraphiteReporter.report?

@zcox Yea, that's probably the better approach. send() is probably too low level for this. I'll fix that up today.

:+1:

@zcox I'm gonna go ahead and publish this and see how it works out.

@adamdecaf can't wait to see how it works

@adamdecaf I take it this worked out when you tried it?

@DustinChaloupka yea, I got the unit tests to all pass fine. Haven't tried it with graphite in big yet though.

:+1:

Closing this because our other fix has worked thus far.

Hi @adamdecaf would you be able to elaborate a bit on what you meant by your 'other' fix? Do you still have a PR somewhere waiting to be merged into https://github.com/dropwizard/metrics?
We're finding that our hosts fail to report metrics whenever there is any kind of temporary communication failure with carbon. The graphite reporter fails to connect from that point onward. The connection to carbon is simply not robust enough since https://github.com/dropwizard/metrics/commit/d42bc7f6c20fb33bf9cc6c31e9a649418e76cc01 was applied.
I see you opened https://github.com/dropwizard/metrics/issues/814 - are you sticking with a forked version of dropwizard/metrics now or have you found some other way to work around this? Cheers

@joelittlejohn We're routing metrics through haproxy. We had a bit of configuration that we tweaked to allow for better timing windows and payloads.


Traceback when using --add-git-hook

The directory I was working in was a git repository:
code/coq-typeclass-hierarchy > python -m readme2tex --add-git-hook
Traceback (most recent call last):
  File "/nix/store/p4fqqhllhyj90njg2sksz8qrgzlzs193-python-2.7.13/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/nix/store/p4fqqhllhyj90njg2sksz8qrgzlzs193-python-2.7.13/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/nix/store/qgfi4pzf9awf5axakp2f0kj4xf7r7grn-readme2tex/lib/python2.7/site-packages/readme2tex/__main__.py", line 163, in <module>
    assert os.path.exists(".git")
AssertionError

Hey @siddharthist, thanks for the issue report. I'm a bit swamped for the next two days so I might not get a chance to look closer at these issues. Are you in the top-level directory?

Also great to see someone else in the PL field :)

@leegao I think this was my error, I wasn't in a git repo like I thought 😓 Anyway, it's probably better to print an error message than a traceback 😄 No rush, thanks for the response!

Totally forgot about this; this has been fixed a few PRs ago!


Memory limit

After running, I found that the code needs a large amount of memory to run. Is there any optimization method?

I think the lower batch size may help you.


In my example the Try<T> trait just extends TryInto<T>. Of course it would be possible to define Try<T> independently.
pub trait Try<T>: TryInto<T> {
    fn from_error(Self::Error) -> Self;
}

////// from std //////
pub trait TryInto<T> {
    type Error;
    fn try_into(self) -> Result<T, Self::Error>;
}

Now the part you asked for: desugaring of x? — (..I was forced to define a helper function):
macro_rules! question_mark {
    ($x:expr) => {
        match try_into($x) {
            Ok(v) => v,
            Err(e) => return e,
        }
    }
}

/// This helper function connects the incoming type `I: Try<T>` and the
/// outgoing type `E: <Try>` via `E::Error: From<I::Error>`, because that
/// cannot be inferred when inlined.
pub fn try_into<T, I: Try<T>, E: Try<T>>(x: I) -> Result<T, E>
    where E::Error: From<I::Error>,
{
    match TryInto::try_into(x) {
        Ok(v) => Ok(v),
        Err(e) => Err(Try::from_error(From::from(e))),
    }
}

Any attempt to get rid of that helper function seems to fail due to inference
macro_rules! question_mark {
    ($x:expr) => {
        match TryInto::try_into($x) {
            Ok(v) => v,
            Err(e) => return Try::from_error(From::from(e)),
            //               ^^^^^^^^^^^^^^^
            // type annotations required: cannot resolve
            // `futures::Poll<i32, u32>: Try<_>`
        }
    }
}

// ..
mod futures {
    fn bar() -> Poll<i32, u32> {
        let a: i32 = question_mark!(result());
        //           ------------------------ in this macro invocation
    }
}

Discussion
Would that be any good? Hm...
I think it's one thing to have potentially expensive conversions on the error-path. But with this TryInto behaviour every use of ? could mask an expensive operation on the happy path. I think it's better to have these conversions be explicit like @nikomatsakis suggested, but maybe with names which focus on the happy path poll.ok()?, poll.async()?, poll.result()?. Or more generically poll.try_into()?

Great work on the RFC updates, @nikomatsakis!  It flows well, and I particularly like the middle-ground description of "be sure that this sort of interconversion is intentional".

variations around futures

Good to see this written up.  I like the approach of having a short method when you want something different from the default.  (I suppose they could even return impl Try<Ok=...,Error=...> to discourage using anything but ? on the returned value.)  Bikeshed: I think I agree with @colin-kiegel that I'd have expected the opposite names, so it'd be poll().ready()? : Result<T,E>, where the .ready()? is behaving sortof like an if .is_ready().
I'm glad you arrived at Poll<T,E>::Ok = T.  Looking at success-wrapping, I think that's the only permutation that avoids a "Poll::Ok(()) problem" in something like an async write_all.
Losing error conversion there definitely makes me sad, as it probably means futures wouldn't move over to Try until it could impl<E, F: Into<E>> From<Async<F>> for Async<E>.  I guess that needs intersection impls or a where E != F?  (Hmm, Option probably wants the same thing.)

Is there a use case for constructing option::Missing?

I don't think there is today.  I'm not sure why I'd ever want to write Ok(x)/Err(Missing) instead of Some(x)/None, so preventing pub construction for now sounds good (and it can easily be allowed later).
If we get a throw sugar, though, it would be useful.  Demonstration (not clearly better than Some/None here, but as an example):
fn checked_abs(x: i32) -> Option<i32> throws {
    if x == i32::MIN { throw Missing; }
    if x < 0 { -x }
    else { x }
}

Looking at that, though, Missing is a rather weird name for what's happening.  It feels like it would be somewhat more natural for it to be throw None.
Interestingly, we do have a ZST that has None as its value-namespace constructor: Option<!>.  That means we could do this:
type None = Option<!>;
impl<T> Try for Option<T>  {
    type Ok = T;
    type Error = None;
    fn into_result(self) -> Result<T, None> { self.ok_or(None) }
    fn from_ok(v: T) -> Self { Some(v) }
    fn from_error(_: None) -> Self { None }
}

I can't make up my mind whether that's the good kind of clever or the abusive kind of clever, though...

Whoa nice investigation @colin-kiegel! Unfortunately though I think that once you start splitting it up at the crate boundary it runs afoul of coherence :(
error[E0210]: type parameter `T` must be used as the type parameter for some local type (e.g. `MyStruct<T>`); only traits defined in the current crate can be implemented for a type parameter
  --> bar.rs:60:5
   |
60 |     impl<T, E> TryFrom<Poll<T, E>> for T {
   |     ^

error[E0210]: type parameter `T` must be used as the type parameter for some local type (e.g. `MyStruct<T>`); only traits defined in the current crate can be implemented for a type parameter
   --> bar.rs:103:5
    |
103 |     impl<T, E> TryFrom<Poll<T, E>> for Result<T, E> {
    |     ^

error: aborting due to 2 previous errors

I wonder, though, if using short methods (like poll.ready()?) would still allow this to be possible? For example maybe:
fn foo() -> Poll<T, E> {
    let a: T = (e: Result<T, E>)?; // propagate Result error
    let b: Async<T> = (e: Poll<T, E>).maybe_ready()?; // propagate Poll error
    let c: T = (e: Poll<T, E>)?; // propagate error and "not ready"
    let d: Result<T, E> = (e: Poll<T, E>).ready()?; // propagate "not ready"
    let e: T = (e: Async<T>)?; // propagate "not ready"
}

In the interest of keeping this moving, would we like to FCP again, or do we think more discussion is necessary? A new summary of where we stand would be instructive ( @nikomatsakis ?).

@bstrie Good idea.
@rfcbot fcp cancel

@rfcbot fcp merge
I'm bringing this up for a fresh FCP, since the content of the RFC has switched approaches. You can get the gist of the new approach from @nikomatsakis's summary comment.

Quick remark: for readers from the future that will discover this RFC through the feature it might be interesting to add a note on why it's Missing and not (). Maybe just a link to @aturon's comment here (https://github.com/rust-lang/rfcs/pull/1859#issuecomment-282091865) is sufficient.

@mitsuhiko

Quick remark: for readers from the future that will discover this RFC through the feature it might be interesting to add a note on why it's Missing and not (). Maybe just a link to @aturon's comment here (#1859 (comment)) is sufficient.

This is I think what this text was meant to convey:

The use of a fresh type like Missing is recommended whenever one
implements Try for a type that does not have the #[must_use]
attribute (or, more semantically, that does not represent an
"unhandled error").

but I guess I should make the connection to () more explicit.

Thx @bluss for pointing me here. I asked in rust-lang/rust#31436 why the conversion was implemented using From instead of Into. As it turns out changing this now would be rather difficult (breaking existing code).
I thought I needed Into to have a generic error type in a possibly external crate, and make local errors convertible to this generic error.  I found a solution (which is probably obvious to most experts in here): have a custom IntoCustomError trait, and implement From based on that.  Maybe this could be used as an example in the book or somewhere else: https://play.rust-lang.org/?gist=41a425fc633ddd81b90a92e3479c9886

ping @pnkfelix needs signoff - https://github.com/rust-lang/rfcs/pull/1859#issuecomment-299042503 (I want my glorious Option/?!)

@rfcbot reviewed

Thanks all. I've merged the RFC and re-used the existing tracking issue.


Diff of project when all packages not present in current user's symbol list is not possible in Jadeite Alpha2.0.1

After some analysis it has become clear that this a a Jadeite Alpha2.0.1 design feature that should be corrected in a later release. See this comment for more details.
Here is the original stack:
UserDefinedError (AbstractException) >> _signalWith: @5 line 25
UserDefinedError (AbstractException) >> signal @2 line 47
RwGsImage class (Object) >> error: @6 line 7
[] in  ExecBlock0 (RwGsImage class) >> loadedPackageNamed: @5 line 5
RwGsImage class >> loadedPackageNamed:ifAbsent: @4 line 11
RwGsImage class >> loadedPackageNamed: @2 line 4
RwPkgDiffTool >> patchForPackageName: @3 line 4
JadeServer64bit32 (JadeServer) >> sbChangesInPackage: @8 line 7
JadeServer64bit32 (JadeServer) >> systemBrowserCommand @65 line 22
[] in  ExecBlock0 (JadeServer) >> systemBrowserA: @14 line 10
Time class >> secondsElapsedTime: @5 line 11
Time class >> millisecondsElapsedTime: @2 line 6
JadeServer64bit32 (JadeServer) >> millisecondsElapsedTime: @2 line 3
JadeServer64bit32 (JadeServer) >> systemBrowserA: @2 line 4
JadeServer64bit32 (JadeServer) >> systemBrowser: @3 line 2
[] in  ExecBlock0 (JadeServer64bit) >> systemBrowser: @2 line 4
ExecBlock0 (ExecBlock) >> on:do: @3 line 42
JadeServer64bit32 (JadeServer64bit) >> systemBrowser: @2 line 5
GsNMethod class >> _gsReturnToC @1 line 1

Can be reproduced using RowanSample6 after creating stone with newBuild_SystemUser_update_sett and logging in as GlobalsCurator ...
.. This looks like it is "expected" ... a consequence of Rowan project being present in the project list but not all of the symbol dictionaries that compose Rowan are present ... thus the "missing package" ...
There are two solutions to this problem:

disable the diff (and possibly other menu items) when the project is not complete ... remains to be seen how that may be calculated
do some symbol dictionary fishing for the missing package so that the request can be fulfulled --- as long as the developer has read permission

After discussion with @ericwinger it seems that symbol dictionary fishing is the correct long term solution ... the only reason that we would fail symbol dictionary fishing would be if we did not have the proper object security permissions ... and lacking read permission would be a valid error

After discussion with @AllenOtis, it seems that a better solution would be to use the package list that is owned by the project to do the diff, rather than lookup the visible packages using loadedPackageNamed: ... much simpler and less error prone!

Upon further review, it seems that this is fundamentally a jadeite alpha issue as Jadeite is working off the full list of package names in the project obtained from Rowan, however, jadeite alpha is querying a package name at a time ... a better structure moving forward would to obtain from the server the full set of changes for all of the packages in a given problem and we would not need to be looking up diffs a package at a time by name ...

Looking up diffs on a package level is a handy developer tool.
Package-level diffs is already built into the Jadeite alpha - fyi.
On Wed, Jul 25, 2018 at 2:43 PM, Dale Henrichs<EMAIL_ADDRESS>wrote:

Upon further review, it seems that this is fundamentally a jadeite alpha
issue as Jadeite is working off the full list of package names in the
project obtained from Rowan, however, jadeite alpha is querying a package
name at a time ... a better structure moving forward would to obtain from
the server the full set of changes for all of the packages in a given
problem and we would not need to be looking up diffs a package at a time by
name ...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dalehenrich/Rowan/issues/280#issuecomment-407906299,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiiPJBuyhdAA1cTmExrgotPQ-Avlk5fGks5uKOZ9gaJpZM4VgkhR
.

--
Eric Winger
Staff Engineer
GemTalk Systems

Actually, package-level diffs is available only for visible packages
currently. Doing what @dalehenrich suggests for project-level diffs would
avoid getting diffs for non-visible packages.
On Wed, Jul 25, 2018 at 2:46 PM, Eric Winger<EMAIL_ADDRESS>
wrote:

Looking up diffs on a package level is a handy developer tool.
Package-level diffs is already built into the Jadeite alpha - fyi.
On Wed, Jul 25, 2018 at 2:43 PM, Dale Henrichs<EMAIL_ADDRESS>wrote:

Upon further review, it seems that this is fundamentally a jadeite alpha
issue as Jadeite is working off the full list of package names in the
project obtained from Rowan, however, jadeite alpha is querying a package
name at a time ... a better structure moving forward would to obtain from
the server the full set of changes for all of the packages in a given
problem and we would not need to be looking up diffs a package at a time by
name ...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dalehenrich/Rowan/issues/280#issuecomment-407906299,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiiPJBuyhdAA1cTmExrgotPQ-Avlk5fGks5uKOZ9gaJpZM4VgkhR
.

--
Eric Winger
Staff Engineer
GemTalk Systems

--
Eric Winger
Staff Engineer
GemTalk Systems

The suggested workaround is to add the Rowan symbol dictonaries to the user's symbol list, so that all of the packages will be visible ...

it seems that this is adequately addressed


Intel H265 QSV - Watcher is only taking Seconds to transcode

The Process is started and everything seems to be working. But the files outputed are only 44Bytes and the log basically says instantly that the next file is beeing processed. If Somebody could help that would be great. If I use another preset or start the process manualy it works fine.

[autovideoconverter] Waiting 5 seconds before processing '/watch/redacted/Season 202204/redacted (2022) - s202204e06.mp4'...
[autovideoconverter] Starting conversion of '/watch/redacted/Season 202204/redacted (2022) - s202204e06.mp4' (73a53a16a41a783d0caecfd7dca47919) using preset 'Hardware/H.265 QSV 1080p'...
[autovideoconverter] 1 title(s) to process.
[autovideoconverter] Conversion ended successfully.
[autovideoconverter] Waiting 5 seconds before processing '/watch/redacted/Season 202204/redacted (2022) - s202204e07.mp4'

I have the same problem, using H.265 QSV as well. I'm converting into MKV, the resulting file has 0 bytes, but the process is "successful" according to all logs. The conversion.log just stops after encqsvInit: RateControlMethod ICQ ICQQuality 22 - no errors are reported.
Converting via GUI using the same preset works without issues, just the automatic conversion fails.

Same problem for me. Can't figure out why this is happening because there are no errors reported. It just stops after initializing the encoding.

The Process is started and everything seems to be working. But the files outputed are only 44Bytes and the log basically says instantly that the next file is beeing processed. If Somebody could help that would be great. If I use another preset or start the process manualy it works fine.

[autovideoconverter] Waiting 5 seconds before processing '/watch/redacted/Season 202204/redacted (2022) - s202204e06.mp4'...
[autovideoconverter] Starting conversion of '/watch/redacted/Season 202204/redacted (2022) - s202204e06.mp4' (73a53a16a41a783d0caecfd7dca47919) using preset 'Hardware/H.265 QSV 1080p'...
[autovideoconverter] 1 title(s) to process.
[autovideoconverter] Conversion ended successfully.
[autovideoconverter] Waiting 5 seconds before processing '/watch/redacted/Season 202204/redacted (2022) - s202204e07.mp4'

I just played around with some settings in my docker-compose file:
Because I had problems running QSV in a previous version my container was set up with uid=0, qid=0 and was in privileged mode. After deleting these parameters it works for me now.
Maybe it works for you @Serph91P @AsfalothDE ?

@MrUhu Thank you for your answer :) Unfortunately at least for me that is not working - my container is not running in priviledged mode and I've set my local user 1027 as PUID already. That doesn't change the behaviour with the automated encodings. :(

I know this is an old issue but just wondering if anyone got to the bottom of this problem.
I'm running the container on a NUC with QSV support and I can encode using the H.265 QSV preset with no problems manually through the GUI but whenever I use the H.265 QSV preset for the watch folders the logs show that the files were converted successfully and the output files are created but they are 0 bytes long.
The last line in the conversion.log file is:
[h264_qsv @ 0x7f5679b0cbc0] Invalid pkt_timebase, passing timestamps as-is.
If I configure the auto converter to use a non QSV preset then everything works as expected and the files are converted without any problems (except for taking a lot longer!).
Any help would be very much appreciated as I've got a lot of videos to convert in order to move my Plex DVR onto a new machine with less storage!

@MrUhu Thank you for your answer :) Unfortunately at least for me that is not working - my container is not running in privileged mode and I've set my local user 1027 as PUID already. That doesn't change the behaviour with the automated encodings. :(

Yeah...I also don't know why it was working at that time, but now this configuration is also not working for me.

Yeah.
Same for me. Somehow it was working one day (the day I posted my solution) with three transcodes and the next day I got the same behaviour as before.
I wanted to use Tdarr in the future although handbrake has enough features for the basic stuff I would need when I'm finished cutting and exporting some videos.

Thanks for the heads up re Tdarr @MrUhu.  Looking into it now and it looks like it might be exactly what I need

still facing the same issue in the latest build

Can you try again with the latest image ?
Also, is the same issue occur when using the GUI ?
You can also try to add lowpower=0 to the More Settings field under the Video tab.

I do have the same problem.
The first video get encoded and the subséquente file  go very fast with size 44 with the following content : ftypmp42mp42iso2mp4freemdat.
? any clue.

Is this with a QSV encoder ?

Is this with a QSV encoder ?  Yes it is !  All other preset with software work has is, I have tested from the command line and to make them work I need to pass -e qsv_h264 ou qsv_h265.  Strange.

I am also facing this issue with latest image. I have already tried the suggestion to add lowpower=0 to the more options without success.
the issue is only with qsv encoding in automatic mode. in the gui it works, using the same preset.

Same issue here. lowpower=0 does not help either unfortunately... Using the GUI QSV is working without any issues but I can't get it running with the autoconverter...

I have the same situation. When I try to use the automatic conversion qsv preset, a 44byte file will be generated, and the conversion is successful almost instantly.


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In 6a78447, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 9003 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 49613af.


Match current template anatomy.

Update start command to match current AWS template anatomy:
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/template-anatomy.html

Check out the commit I just added re: tm-language files.  For the syntax highlighting to work properly, you have to add keywords like "Metadata" . I use AAAPackageDev to make the format much easier to write (Normally it's xml).  Then you just run AAAPackageDev: Convert YAML to... from the command palette to generate the xml ST3 needs.  You have to restart the editor to see the changes.


[SUPPORT]What's the behavior when writing to COW table

Hi,
Assume I have a COW table and I would like to write 5 records into this table, one record per minute

I would ask how many commits will be created for the above 5 operation, will 5 commits be created? That is, new commits will be created for each new record?
If I write to the COW in batch mode, say, write 1000 records in one time, then how many commits be created?

Thanks!

If you are in streaming mode, each miniBatch creates a commit. For example, a commit will be created during a checkpoint in the Flink engine.

If you write in batch mode, and write 1000 records at a time, only one commit will be created.

If you are in streaming mode, each miniBatch creates a commit. For example, a commit will be created during a checkpoint in the Flink engine.
If you write in batch mode, and write 1000 records at a time, only one commit will be created.

Happy to help.

Thanks for the helpful reply!
For my case of the  COW table and I would like to write 5 records into this table, one record per minute, I am using spark, and
not using streaming,
I am doing is:
0. start spark application

kick off 1st spark write job and write one record
sleep 1 minutes
kick off 2nd spark write job and write one record
....
n. stop spark application

Will each spark job create a commit? It looks to me that no matter how many records are written by one spark job, it will always create one commit?
Thanks!

If you are in streaming mode, each miniBatch creates a commit. For example, a commit will be created during a checkpoint in the Flink engine.
If you write in batch mode, and write 1000 records at a time, only one commit will be created.

Happy to help.

Thanks @BruceKellan for the helpful reply! I have a further question: for my case of the COW table and I would like to write 5 records into this table, one record per minute, I am using spark, and not using streaming,
I am doing is: 0. start spark application

kick off 1st spark write job and write one record
sleep 1 minutes
kick off 2nd spark write job and write one record
....
n. stop spark application

Will each spark job create a commit? It looks to me that no matter how many records are written by one spark job, it will always create one commit?
Thanks!

Yes, you are actually using 5 batch spark application in this scenario.

If you are in streaming mode, each miniBatch creates a commit. For example, a commit will be created during a checkpoint in the Flink engine.
If you write in batch mode, and write 1000 records at a time, only one commit will be created.

Happy to help.

Thanks @BruceKellan for the helpful reply! I have a further question: for my case of the COW table and I would like to write 5 records into this table, one record per minute, I am using spark, and not using streaming,
I am doing is: 0. start spark application

kick off 1st spark write job and write one record
sleep 1 minutes
kick off 2nd spark write job and write one record
....
n. stop spark application

Will each spark job create a commit? It looks to me that no matter how many records are written by one spark job, it will always create one commit?
Thanks!

Yes, you are actually using 5 batch spark application in this scenario.

Thanks @BruceKellan for the helpful answer, I am closing this support request!


No Divvun keyboards show up on Windows 11

Keyboards (fit, rmn, mns, ...) were installed with DM on a new Win 11 machine. DM reported them as "installed".
When trying to activate them via the keyboard installer (either via the down-right menu starting from the iso code or via the control panel), the only available keyboards coming up ("Add a language") are the ones defined by Windows. The Divvun-made keyboards are nowhere to be seen.
As a result of this, the corresponding spellers (fit, rmn, mns, ...) were not visible in MS Word, whereas DM-installed spellers for fao, kal did work in MS Word after I had activated the respective (MS Windows -made) keyboards via the control panel. Without a keyboard fao, kal spellcheckers did not show up in MS word, with the keyboard they did. The languages fit, rmn, mns mever got a keyboard, and their installed spellckecker did not turn up as alternatives.

Discussing with Sjur: There is a way to add them, via the (on Win 11 removed) start menu. The issue needs documentation.
Also, the ISO code turns up as blank in the keyboard slot on the bottom-right field (in the loop with ENG, FIN, SME, etc. there is now BLANK denoting fit, thus more blanks coming up if the user would need more than one DM keyboard)
Also: It is unclear where this should be documented.

See also https://github.com/divvun/divvun-manager-windows/issues/64


Hordeum vulgare download zip does not contains the most recent GPML

the zip has WP2625_r125287 but WP2625_r128704 is on GitHub (matches classic): https://github.com/wikipathways/wikipathways-database/blob/main/pathways/WP2625/WP2625.gpml#L2C116-L2C130

We discussed the situation today. Problem seems to be that while the database repository was updated, but the releases are made from the content of the website, the JSON file there. And the .md of WP2625 in the website repository was not updated.


[docs] Demo SearchIcons crashes

[ ] I have searched for similar issues in this repository and believe that this is not a duplicate.

Steps to Reproduce

Visit https://mui.com/components/material-icons/?theme=Two+tone&query=compare
??
demo SearchIcons crashes

Your Environment

Tech
Version

MUI
v5.5.0

netlify deploy
https://622bf492b027b5000960dea3--material-ui-docs.netlify.app

Browser
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/99.0.4844.51 Safari/537.36

duplicate #31427


feat(CardRadio): reupload - refactoring and apply theming

Description

Adding the CardRadio Component to the components library.
This is a reupload due to challenges with the SSH Key Signing.
Huge thanks to @soumyaloka for her help.
I will aim to respond to feedback quickly.
References
LUI-500
Checklist

[x] all commented code has been removed
[x] any new console issues have been resolved
[x] code linter and formatter has been run
[x] test coverage meets repo requirements
[x] PR name matches the expected semantic-commit syntax

Sanity Tests Execution Passed.

Sanity Tests Execution Passed.

Could add a control for the Radio to the story using the "generateSubStory" method? That way we can test toggling the radio on and off.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/227/rdkcentral/Lightning-UI-Components

Commit: f0f5bb7108d437b13ea24ceb17312c1b99a5834b

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/227/rdkcentral/Lightning-UI-Components

Commit: f0f5bb7108d437b13ea24ceb17312c1b99a5834b

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Not a good code match

Commit: f0f5bb7108d437b13ea24ceb17312c1b99a5834b

Sanity Tests Execution Failed.

Sanity Tests Execution Passed.

:tada: This PR is included in version @lightningjs/ui-components-v2.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Autoupdate for flake8 does not work

Hi,
I tried this pre-commit-hook. It seems like something is not working properly (at least for me).
I have the following setup:
.pre-commit-config.yaml
- repo: https://gitlab.com/pycqa/flake8
    rev: 3.9.0
    hooks:
      - id: flake8
        additional_dependencies: [
            'pycodestyle==2.7.0',  # E,W
            'pyflakes==2.3.0',  # F
            'mccabe==0.6.1',  # C
            'flake8-bugbear==21.3.2',  # B
            'flake8-builtins==1.5.3',  # A
            'flake8-comprehensions==3.3.1',  # C4
            'flake8-debugger==4.0.0',  # T1
            'flake8-logging-format==0.6.0',  # G
            'flake8-print==4.0.0',  # T0
        ]
        args: [
            '--select=E,W,F,C,B,A,C4,T1,G,T0',
            '--ignore=W503,B305,A003,G003,G004',
            '--max-complexity=10',
            '--max-line-length=120',
        ]
- repo: https://github.com/floatingpurr/sync_with_poetry
    rev: 0.2.0
    hooks:
      - id: sync_with_poetry
        name: sync pre-commit dependencies with poetry
        args: [
            '--config', 'src/python/.pre-commit-config.yaml',
            '--all',  # Scan all dependencies in poetry.lock (main and dev)
        ]

poetry.lock
[[package]]
name = "flake8"
version = "4.0.1"
description = "the modular source code checker: pep8 pyflakes and co"
category = "dev"
optional = false
python-versions = ">=3.6"

[package.dependencies]
mccabe = ">=0.6.0,<0.7.0"
pycodestyle = ">=2.8.0,<2.9.0"
pyflakes = ">=2.4.0,<2.5.0"

When running the pre-commit hook, flake8 won't update. Other dependencies like black do update.
First I thought it might be because of a major version jump. But the issue also remains if the rev in pre-commit-config is 4.0.0. It won't update to 4.0.1 (from poetry.lock).
Any ideas?

Hi. The problem lies in the repo url of flake8. Indeed, https://gitlab.com/pycqa/flake8 says:

This is a mirror of github.com/pycqa/flake8

In db.py, we have only the main repo
https://github.com/floatingpurr/sync_with_poetry/blob/d7456aaa0f70302d9ef76430c35f545227b2a322/sync_with_poetry/db.py#L10-L12
That's the reason why there's no match with https://gitlab.com/pycqa/flake8 (and any updates by this hook!).

Hi!
Thank you for the fast investigation!
Ah, in fact, I oversaw this tiny difference in the repo URL gitlab / github.
I updated my own .pre-commit-config.yaml
Anyways, I think it would be good to add an entry to the db.py for this mirrow.

Hi @chris-rl! You are welcome. Currently, I prefer to limit entries in db.py sticking with the hooks officially supported by .pre-commit.