Issue with brew

Description
When I use brew install cppcheck cpplint include-what-you-use, I get the error:
Error: The `brew link` step did not complete successfully
The formula built, but is not symlinked into /usr/local
Could not symlink bin/2to3
Target /usr/local/bin/2to3
already exists. You may want to remove it:
  rm '/usr/local/bin/2to3'

To force the link and overwrite all conflicting files:
  brew link --overwrite<EMAIL_ADDRESS>
To list all files that would be deleted:
  brew link --overwrite --dry-run<EMAIL_ADDRESS>
Possible conflicting files are:
/usr/local/bin/2to3 -> /Library/Frameworks/Python.framework/Versions/2.7/bin/2to3

This issue has emerged recently. That command worked well one week ago.
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[X] macOS 10.15
[X] macOS 11
[ ] Windows Server 2016
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
  Environment: macos-11
  Version: 20210831.3
  Included Software: https://github.com/actions/virtual-environments/blob/macOS-11/20210831.3/images/macos/macos-11-Readme.md
  Image Release: https://github.com/actions/virtual-environments/releases/tag/macOS-11%2F20210831.3

Is it regression?
https://github.com/actions/virtual-environments/releases/tag/macOS-10.15%2F20210814.1
Expected behavior
OK
Actual behavior
Error: The `brew link` step did not complete successfully
The formula built, but is not symlinked into /usr/local
Could not symlink bin/2to3
Target /usr/local/bin/2to3
already exists. You may want to remove it:
  rm '/usr/local/bin/2to3'

To force the link and overwrite all conflicting files:
  brew link --overwrite<EMAIL_ADDRESS>
To list all files that would be deleted:
  brew link --overwrite --dry-run<EMAIL_ADDRESS>
Possible conflicting files are:
/usr/local/bin/2to3 -> /Library/Frameworks/Python.framework/Versions/2.7/bin/2to3

Repro steps

brew install cppcheck cpplint include-what-you-use

Hello, @arttet
I couldn't reproduce the issue from my side.
macOS-10.15:

macOS-11:

I get the same issue if I want to manually install python on macos-10.15 with the command brew install python
Previously I did not need to install python as it should be present on the host machine, but today all the macos-10.15 builds started failing with the following (maybe it is connected to this issue):
-- Could NOT find PythonLibs (missing: PYTHON_LIBRARIES PYTHON_INCLUDE_DIRS) (Required is at least version "3.5")
CMake Error at /usr/local/Cellar/cmake/3.21.2/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find PythonLibs (missing: PYTHON_LIBRARIES PYTHON_INCLUDE_DIRS)
  (Required is at least version "3.0")
Call Stack (most recent call first):
  /usr/local/Cellar/cmake/3.21.2/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)
  /usr/local/Cellar/cmake/3.21.2/share/cmake/Modules/FindPythonLibs.cmake:310 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  extensions/script/CMakeLists.txt:37 (find_package)

-- Configuring incomplete, errors occurred!

Example of a minimum action that reproduces the problem: https://github.com/traversaro/github-actions-brew-update-upgrade-check/actions/runs/1204436581 .

It is my simple example to reproduce the bug.

It happens due to python3.9.6 -> python3.9.7 upgrade

We have found the root cause of this problem. Newly-generated images will make the problem gone (as updated images include an updated python version and no conflicts will happen). We are going to fix the problem until the next python release.

Also saw this, appears resolved by 20210905.2 image

All the images have been updated by now.

Looks like this issue regressed - the dependency chain that errs is as follows:

2to3
idle3
pydoc3
python3
... stopped adding rm ... past here.

E.g. outputs:
Error: The `brew link` step did not complete successfully
The formula built, but is not symlinked into /usr/local
Could not symlink bin/python3
Target /usr/local/bin/python3
already exists. You may want to remove it:
  rm '/usr/local/bin/python3'

This has recently (yesterday?) started happening on the macos-13 runner. The new macos-14 works (for now?), FWIW.

It seems this is fixed on macos-14 because it just overwrote it

Looks like this issue regressed - the dependency chain that errs is as follows:

2to3
idle3
pydoc3
python3
... stopped adding rm ... past here.

E.g. outputs:
Error: The `brew link` step did not complete successfully
The formula built, but is not symlinked into /usr/local
Could not symlink bin/python3
Target /usr/local/bin/python3
already exists. You may want to remove it:
  rm '/usr/local/bin/python3'

I encountered this issue as well, when running brew install imagemagick via fastlane using macos-13 runner.
It tries to install `imagemagick 7.1.1-29_1 but fails when trying to pour the python bottle -
Pouring<EMAIL_ADDRESS>Error: The `brew link` step did not complete successfully
The formula built, but is not symlinked into /usr/local
Could not symlink bin/2to3
Target /usr/local/bin/2to3
already exists. You may want to remove it:
  rm '/usr/local/bin/2to3'

Switching to the macos-14 runner somehow is installing an older version of imagemagick which skips installing python altogether.
Run brew install imagemagick
==> Downloading https://ghcr.io/v2/homebrew/core/imagemagick/manifests/7.1.1-26
==> Fetching dependencies for imagemagick: jbig2dec, libidn, openjpeg, ghostscript, highway, imath, openexr, webp, jpeg-xl, libvmaf, aom, libde265, shared-mime-info, x265, libheif, liblqr, jasper, libomp, libraw, m4 and libtool
==> Downloading https://ghcr.io/v2/homebrew/core/jbig2dec/manifests/0.20
...
🍺  /opt/homebrew/Cellar/libtool/2.4.7: 75 files, 3.8MB
==> Installing imagemagick
==> Pouring imagemagick--7.1.1-26.arm64_sonoma.bottle.tar.gz
🍺  /opt/homebrew/Cellar/imagemagick/7.1.1-26: 809 files, 31.3MB

Started having this issue just two hours apart from each other. Currently only happens on macos-13 runner. No changes were made to the actions. Only change was a typo in documentation.
Successful
Current runner version: '2.317.0'
Operating System
Runner Image
  Image: macos-13
  Version: 20240609.1
  Included Software: https://github.com/actions/runner-images/blob/macos-13/20240609.1/images/macos/macos-13-Readme.md
  Image Release: https://github.com/actions/runner-images/releases/tag/macos-13%2F20240609.1

Installing dependencies for repo/internal-tap/toolstools: libgit2, z3, llvm, rust, mozjpeg, libxml2, libxslt and openjpeg
==> Installing repo/internal-tap/tools dependency: libgit2
==> Downloading https://ghcr.io/v2/homebrew/core/libgit2/manifests/1.7.2
Already downloaded: /Users/runner/Library/Caches/Homebrew/downloads/9671856ced457ee95383408b4c243c7f196b6f204228b11a53735a8a561e2abb--libgit2-1.7.2.bottle_manifest.json
==> Pouring libgit2--1.7.2.ventura.bottle.tar.gz
🍺  /usr/local/Cellar/libgit2/1.7.2: 105 files, 4.5MB

Failed
Current runner version: '2.317.0'
Operating System
Runner Image
  Image: macos-13
  Version: 20240609.1
  Included Software: https://github.com/actions/runner-images/blob/macos-13/20240609.1/images/macos/macos-13-Readme.md
  Image Release: https://github.com/actions/runner-images/releases/tag/macos-13%2F20240609.1

Installing dependencies for repo/internal-tap/tools<EMAIL_ADDRESS>libgit2, z3, llvm, rust, mozjpeg, libxml2, libxslt and openjpeg
==> Installing repo/internal-tap/tools dependency<EMAIL_ADDRESS>==> Downloading https://ghcr.io/v2/homebrew/core/python/3.12/manifests/3.12.4
Already downloaded<EMAIL_ADDRESS>==> Pouring<EMAIL_ADDRESS>Error: The `brew link` step did not complete successfully
The formula built, but is not symlinked into /usr/local
Error: The `brew link` step did not complete successfully
Could not symlink bin/2to3
Target /usr/local/bin/2to3
already exists. You may want to remove it:
  rm '/usr/local/bin/2to3'
To force the link and overwrite all conflicting files:
  brew link --overwrite<EMAIL_ADDRESS>To list all files that would be deleted:
  brew link --overwrite<EMAIL_ADDRESS>--dry-run
Possible conflicting files are:
/usr/local/bin/2to3 -> /Library/Frameworks/Python.framework/Versions/3.12/bin/2to3
/usr/local/bin/2to3-3.12 -> /Library/Frameworks/Python.framework/Versions/3.12/bin/2to3-3.12
/usr/local/bin/idle3 -> /Library/Frameworks/Python.framework/Versions/3.12/bin/idle3
/usr/local/bin/idle3.12 -> /Library/Frameworks/Python.framework/Versions/3.12/bin/idle3.12
/usr/local/bin/pydoc3 -> /Library/Frameworks/Python.framework/Versions/3.12/bin/pydoc3
/usr/local/bin/pydoc3.12 -> /Library/Frameworks/Python.framework/Versions/3.12/bin/pydoc3.12
/usr/local/bin/python3 -> /Library/Frameworks/Python.framework/Versions/3.12/bin/python3
/usr/local/bin/python3-config -> /Library/Frameworks/Python.framework/Versions/3.12/bin/python3-config
/usr/local/bin/python3.12 -> /Library/Frameworks/Python.framework/Versions/3.12/bin/python3.12
/usr/local/bin/python3.12-config -> /Library/Frameworks/Python.framework/Versions/3.12/bin/python3.12-config
==><EMAIL_ADDRESS>-Im ensurepip
==><EMAIL_ADDRESS>-Im pip install -v --no-
==> Summary
🍺<EMAIL_ADDRESS>3,277 files, 63.9MB


Fix MS.TEAMS.1.5 Policy of ScubaGear

🗣 Description
Currently ScubaGear (in Emerald) shows incorrect results for the MS.TEAMS.1.5v1 - with Criticality and Details columns of report showing undeclared values. The objective of this policy is to check that dial-in users are not configured to be auto admitted into meetings and it is applicable to all policies. The 'who can bypass the lobby' setting tested through another teams policy (1.3) auto selects the true/false value for the dial-in users. But there are some configuration paths that can leave this setting (dial-in users to bypass lobby) to be in true mode. This policy enforces and checks this setting to be false irrespective of previous setting and for all meeting policies. So the implementation is: for all meeting policies, check that 'dial-in users can bypass the lobby' is set to false and count any policies that are non-compliant.
💭 Motivation and context
closes #452
🧪 Testing
Tested the feature branch with G5 and E5 tenants with following test cases - all passed:
In Teams -> Meeting Policies -> Meeting Join and Lobby, do the following:

Set "Who can bypass the lobby" to one of: People in my org, People in my org and guests, People in my org, trusted org, and guests.
Set "People dialingin can bypass the lobby" to Off

Above 3 tests should pass the policy 1.5. For negative tests,  set "People dialingin can bypass the lobby" to On and make sure that policy 1.5 fails with a detail that "either everyone or dial-in users are allowed to bypass the lobby". Test this for a custom policy as this policy is applicable to all policies.
✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Create a release.

Checked to see if the test would fail if I changed the global policy so that "People dialing in can bypass the lobby" was set to on. 1.3 failed, but 1.5 still passed unexpectedly.

Checked to see if the test would fail if I changed the global policy so that "People dialing in can bypass the lobby" was set to on. 1.3 failed, but 1.5 still passed unexpectedly.

Are you sure the test rant with the branch? - I ask because the result in your test for 1.5 shows criticality and details as 'system.object'. That is fixed with the branch and in my test against G5 tenant shows the following result - note the criticality and details columns against 1.5.


No such file or directory: "/workspace/ComfyUI/models/liveportrait/appearance_feature_extractor.safetensors"

This node (meant to download the models if I am not wrong) is not downloading models and just says it can't find them

Same error for motion_extractor.safetensors afterwards


Allow DependsOn on all supported resources

Pull Request
Pull Request (PR) description
Changed

Allowing support for DependsOn on all supported resources

Task list

[x] The PR represents a single logical change. i.e. Cosmetic updates should go in different PRs.
[x] Added an entry under the Unreleased section of in the CHANGELOG.md as per format.
[x] Local clean build passes without issue or fail tests (build.ps1 -ResolveDependency).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[x] Unit tests added/updated. See DSC Resource Testing Guidelines.
[x] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[x] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Not required as different build systems seem to have led to issues with dependencies. Hardly readable error messages contributed to the feeling that this would be necessary


@storybook/preset-ant-design broken with<EMAIL_ADDRESS>

As said here https://ant.design/docs/react/use-with-create-react-app, with<EMAIL_ADDRESS>lessOptions are spread to options directly and now they should be spread to lessOptions property

I've found the same issue))

Any chance to get this merged?

Anybody knows why my build pipeline is broken? This is a really important fix! Thanks!

It would be great if we could get this merged soon.

Hello, hopefully someone can point me in the right direction...
Does this now work, since the merge? I've been going round in circles trying to get a TypeScript / Ant Design / LESS / Craco storybook up and running.


License CC BY 4.0 okay?

@AlexanderPico @khanspers
which license should we use for the academy material?

I'm not sure which license was chosen when the repo was created, maybe "None". I can't see it at GH, normally its displayed at the top. Do you mean that we should add a license at GH? I don't know how to do that.
I would guess if we were to specify a license (or change it from something else), we would use Creative Commons Zero v 1.0 Universal?

I think for the Academy both C0 and CCBy would be fine. But if you do want to go with CCBy, and even if you do not maybe,  we might want to think about also giving some "How to Cite" advice?

I added a license (https://github.com/wikipathways/academy/blob/gh-pages/LICENSE), CC-BY 4.0 was not an option for the template so I picked CC0. We can change it if necessary.


Nested object fields don't get optimistically updated

urql version & exchanges:
    "@urql/core": "^2.4.2",
    "@urql/exchange-auth": "^0.1.7",
    "@urql/exchange-graphcache": "^4.3.6",

Steps to reproduce

Have a mutation that returns an object (e.g. a Post) with nested fields that are non-keyable (e.g. a PostMetadata).

type PostMetadata {
  title: String!
}

type Post {
  id: ID!
  metadata: PostMetadata!
}

input InputPost {
  id: ID!
  metadata: PostMetadata!
}

type Mutation {
  createPost($post: InputPost): Post!
}

By non-keyable, I mean that the graphcache's exchange keys map is as follows (as prescribed here):
cacheExchange({
  // ...
  keys: {
    PostMetadata: () => null
  },
})

Have an optimistic mutation for that mutation that also tries to update the nested Post.metadata:

cacheExchange({
  // ...
  createPost: (args: CreatePostMutationVariables) => {
    return {
        __typename: "Post",
        ...args.post,
      },
    };
  },
})

Expected behavior
The PostMetadata associated with the Post is updated locally.
Option 3. is unnecessary, option 2. works out of the box.
Actual behavior
Only the direct fields of the Post are updated.
If you explicitly specify __typename for the PostMetadata, things work as expected:
cacheExchange({
  // ...
  createPost: (args: CreatePostMutationVariables) => {
    return {
        __typename: "Post",
        ...args.post,
        metadata: {
          __typename: "PostMetadata",
          ...args.post.metadata,
        }
      },
    };
  },
})

However, this seems like a footgun.

That is expected behavior, we have no way to predict what the sub-types will be for a given entity

@JoviDeCroock If I have an introspected schema available on the client-side, is there a way for me to do this "filling in __typename" automatically?

That makes it hard to predict for unions/... so not really

For others that might encounter this problem and are using TypeScript, this utility type can ensure you are properly setting typenames on all of your sub-fields:
export type WithExplicitTypename<T> = T extends { __typename?: string }
  ? {
      __typename: NonNullable<T["__typename"]>;
    } & {
      [Properties in keyof Omit<T, "__typename">]: WithExplicitTypename<
        T[Properties]
      >;
    }
  : T extends { [key: string]: any }
  ? {
      [Properties in keyof T]: WithExplicitTypename<T[Properties]>;
    }
  : T;

I will still be looking for a way to do this automatically in the absence of unions.


Updated the content links that were deprecated in the document.

Description
Updated the content links that were deprecated in the document.
Checklist

[x] The base branch of this PR is dev, rather than main
[x] The relevant docs, if any, have been updated or created

Screenshots
[ "For visual changes, include screenshots." ]
Testing
[ For new or modified features, provide testing instructions. ]

Hey @Raven-1027 , I think you might not have pulled down the latest changes to dev before opening this PR. Mind doing that and then re-opening?

Hey @Raven-1027 , I think you might not have pulled down the latest changes to dev before opening this PR. Mind doing that and then re-opening?

Sorry for that. I will close it and re-open it later. thank you.


dApp: Nicer looking info icon on empty withdrawal screen

Fixes #1613
Short description
Replaces the material design info icon with a custom SVG.

Definition of Done

[x] Steps to manually test the change have been documented
[x] Acceptance criteria are met
[ ] Change has been manually tested by the reviewer (dApp)

Steps to manually test the change (dApp)

Open the Account screen and click the Withdrawal option.
The icon will appear if there are no tokens available for withdrawal.

@weilbith Good point/observation. Made changes to the i, looks like this now:

Awesome. I knew why I was optimistic.


Support for Laravel 8 Factories

Is your feature request related to a problem? Please describe.
The new Laravel 8 factory syntax is not yet supported, so Psalm cannot determine the type of User::factory()->make() or User::factory()->count(2)->make() (which is equivalent to User::factory(2)->make()).
For a App\Models\User model, factories are now defined as follows:

the HasFactory trait is used in the User class, which adds a static factory() method
the corresponding Database\Factories\UserFactory is instantiated and returned
if a numeric value is passed to factory(), this implicitly sets the count() accordingly

See also: https://laravel.com/docs/8.x/database-testing#defining-model-factories
Describe the solution you'd like
Add support for these factories.
Describe alternatives you've considered
(None.)
Additional context
As a workaround, I have added <directory name="tests"/> folder to the <psalm><projectFiles><ignoreFiles> section in the psalm.xml configuration.

Feel free to make a PR, I don't have any apps on laravel 8 yet

Laravel 9 (the only supported version but the plugin ATM) + Psalm support them out of the box


SAK-33714 export using category order if Group By Category is being used

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- gradebookng/tool/src/java/org/sakaiproject/gradebookng/tool/panels/importExport/ExportPanel.java  3

See the complete overview on Codacy


Hydration Error for rehype-expressive-code

When running the Next.js example from this repo and when using a new create-next-app project following the next mdx setup and the expressive code nextjs installation, both projects produce a hydration error when rendering a basic mdx page.
{/* page.mdx */}
# hi

```go
func main() {

}
```

Full react stack trace

throwOnHydrationMismatch
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(12507:1)
tryToClaimNextHydratableInstance
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(12535:1)
updateHostComponent
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(19902:1)
beginWork
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(21618:1)
HTMLUnknownElement.callCallback
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(4164:1)
Object.invokeGuardedCallbackDev
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(4213:1)
invokeGuardedCallback
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(4277:1)
beginWork$1
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(27451:1)
performUnitOfWork
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(26557:1)
workLoopSync
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(26466:1)
renderRootSync
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(26434:1)
performConcurrentWorkOnRoot
../node_modules/.pnpm/react-dom<EMAIL_ADDRESS>(25738:1)
workLoop
<EMAIL_ADDRESS>(266:1)
flushWork
<EMAIL_ADDRESS>(239:1)
MessagePort.performWorkUntilDeadline
<EMAIL_ADDRESS>(533:1)

Output from npx --no-install next info
Operating System:
  Platform: linux
  Arch: x64
  Version: #28-Ubuntu SMP PREEMPT_DYNAMIC Thu Mar  7 18:21:00 UTC 2024
  Available memory (MB): 15163
  Available CPU cores: 16
Binaries:
  Node: 21.4.0
  npm: 10.2.5
  Yarn: N/A
  pnpm: 8.15.4
Relevant Packages:
  next: 14.2.1 // Latest available version is detected (14.2.1).
  eslint-config-next: 14.2.1
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 5.2.2
Next.js Config:
  output: N/A

I set up the example in this repo by running
corepack enable
corepack use<EMAIL_ADDRESS>corepack pnpm build
cd packages/rehype-expressive-code/test/fixtures/nextjs/
corepack pnpm dev

I deleted all my node modules, upgraded my package managers, reinstalled all the packages and now I'm not getting any more errors. Not sure what the issue is but I'm glad it's gone now. Thanks for all the help and the quick responses.

Hey, awesome that you got it to work! Thank you for your patience.
I'm glad to hear it wasn't actually an issue with anything we did, but a strange package manager issue. :)


Searches "Keyword 1"

Is there an existing issue for this?

[X] I have searched the existing issues

I'm submitting a ...

[X] bug report
[ ] feature request
[ ] support request --> Contact me over mail for support https://github.com/MShawon

Description
When searching for videos using search it simply types "keyword 1" in the search box

.
Environment
- OS : Windows
- Python : 3.10.5
- Script version : 1.7.4

config.json
search.txt:

keyword 1 :::: AR-55 Is Meta? The Cycle: Frontier Stream Highlights.

config:
{
    "http_api": {
        "enabled": true,
        "host": "<IP_ADDRESS>",
        "port": 5000
    },
    "database": true,
    "views": 50,
    "minimum": 65.0,
    "maximum": 100.0,
    "proxy": {
        "category": "p",
        "proxy_type": "http",
        "filename": "10_proxies.txt",
        "authentication": true,
        "proxy_api": false,
        "refresh": 0.0
    },
    "background": false,
    "bandwidth": false,
    "playback_speed": 1,
    "max_threads": 10,
    "min_threads": 3
}

change keyword 1 to an actual keyword
for example, you could use
frontier stream highlights :::: AR-55 Is Meta? The Cycle: Frontier Stream Highlights.
stream highlights frontier :::: AR-55 Is Meta? The Cycle: Frontier Stream Highlights.
the cycle frontier stream highlights :::: AR-55 Is Meta? The Cycle: Frontier Stream Highlights.
frontier stream highlights the cycle :::: AR-55 Is Meta? The Cycle: Frontier Stream Highlights.

keyword means you video tag,
so the process is so simply your video tag :::: then that video whole title.
check those screenshot, then you will easily understand that.

.

Ok thank you for the help!


Unhelpful error message when credentials misssing

I was trying to use a Node.js client library (any, doesn't matter) in a Node.js express server. I started the app without providing credentials. I expected an error, but the error I got was less than useful.
Error: Request is missing required authentication credential. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project.

Problems:

I can follow the link by ⌘-clicking in the terminal. It would be good if there wasn't a trailing dot which leads to a 404.
The error page talks about 'Google Sign-In' and 'WebApp'. This was a Node.js server, not a WebApp, and I am not trying to use Google Sign In. The error page seems wrong.  I expected to be taken to a page that talks about Service Accounts and OAuth2.

👋 so I'm finally getting to look at this one.  When I try it, I am getting this:
node listQueues.js
/Users/beckwith/Code/nodejs-tasks/samples/node_modules/@google-cloud/tasks/src/v2/cloud_tasks_client.js:178
              throw err;
              ^

Error: Could not load the default credentials. Browse to https://cloud.google.com/docs/authentication/getting-started for more information.
    at GoogleAuth.<anonymous> (/Users/beckwith/Code/nodejs-tasks/samples/node_modules/google-auth-library/build/src/auth/googleauth.js:168:23)
    at Generator.next (<anonymous>)
    at fulfilled (/Users/beckwith/Code/nodejs-tasks/samples/node_modules/google-auth-library/build/src/auth/googleauth.js:19:58)
    at processTicksAndRejections (internal/process/task_queues.js:86:5)

I found that doc to be much more helpful :) Can you share the steps you're using to get that error?  It's entirely possible this is something we accidentally fixed.

I don't remember the precise steps. I tried reproducing from scratch in the way I expect I would have done originally:
(node:73218) UnhandledPromiseRejectionWarning: Error: Sorry, we cannot connect to Cloud Services without a project
 ID. You may specify one with an environment variable named
 "GOOGLE_CLOUD_PROJECT".
    at replaceProjectIdToken (/Users/ofrobots/tmp/bunyan-test/node_modules/@google-cloud/projectify/build/src/index.js:45:19)
    at Object.replaceProjectIdToken (/Users/ofrobots/tmp/bunyan-test/node_modules/@google-cloud/projectify/build/src/index.js:38:30)
    at self.auth.getProjectId (/Users/ofrobots/tmp/bunyan-test/node_modules/@google-cloud/logging/build/src/index.js:719:40)
    at getProjectIdAsync.then.r (/Users/ofrobots/tmp/bunyan-test/node_modules/google-auth-library/build/src/auth/googleauth.js:73:48)
    at process.internalTickCallback (internal/process/next_tick.js:77:7)

This seems different / more reasonable.
Let me see if I can reproduce some other way.

Here's a repro, in a new dir, add @google-cloud/debug-agent a dependency and the following code:
require('@google-cloud/debug-agent').start({
  allowExpressions: true,
  projectId: 'ofrobots-test-19'   // <-- change this to your own project
});

setTimeout(() => {}, 10000);

Run gcloud auth revoke, and then:
❯ node index.js
@google-cloud/debug-agent Failed to re-register debuggee ofrobots-test-19: Error: Request is missing required authentication credential. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project.
@google-cloud/debug-agent Failed to re-register debuggee ofrobots-test-19: Error: Request is missing required authentication credential. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project.
@google-cloud/debug-agent Failed to re-register debuggee ofrobots-test-19: Error: Request is missing required authentication credential. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project.

Ok - I finally tracked this one down.  It's a bug on nodejs-common:
https://github.com/googleapis/nodejs-common/blob/master/src/util.ts#L607
Even if auth fails, it still decides "hell, maybe we won't need a token!" and sends the request to the backend anyway.  The error message we're getting is part of the body from the 401 response that comes back from the debug API itself.
I can't imagine why we wouldn't fail in this scenario with the original call stack.  Uh, @stephenplusplus ?

Original PR: https://github.com/googleapis/google-cloud-node/pull/2712
Failing with the original call stack would make sense to me, too.

Just in case I misunderstood what you meant by fail, the first error message would be more helpful, but I believe we still need to attempt the API request, as they don't all require authentication.
It might not be so easy, though, to define the condition that we want to ignore the API failure message and use the auth lib's. Maybe only when it's a 401?

If the API endpoint requires credentials, and we can't acquire them - the original callstack would be very nice here.  The 401 approach is fine.
I am curious though ... what's the use case for this one?  I'm sure we had a good reason, it's just not called out in the PR.

I believe Storage.File#download, but there might be others by now. My bad for not leaving a better trail in that PR.


CupertinoTheme based on Device Brightness?

Is there a way to display light/dark mode based on the Devices brightness in iOS? I tried this:
void main() => runApp(MyApp());

class MyApp extends StatelessWidget {

  @override
  Widget build(BuildContext context) {

    final materialLightTheme = ThemeData(
      primaryColor: Colors.blueAccent,
      brightness: Brightness.light,
      scaffoldBackgroundColor: MyColors.backgroundColor0(Brightness.light),
      appBarTheme: AppBarTheme(
        color: MyColors.backgroundColor1(Brightness.light)
      ),
    );

    final materialDarkTheme = ThemeData(
      primaryColor: Colors.blueAccent,
      brightness: Brightness.dark,
      scaffoldBackgroundColor: MyColors.backgroundColor0(Brightness.dark),
      appBarTheme: AppBarTheme(
        color: MyColors.backgroundColor1(Brightness.dark)
      ),
    );

    final cupertinoTheme = new CupertinoThemeData(
      primaryColor: Colors.blueAccent,
      scaffoldBackgroundColor: MyColors.backgroundColor0(Brightness.light),
      barBackgroundColor: MyColors.backgroundColor1(Brightness.light)
    );

    final cupertinoThemeDark = new CupertinoThemeData(
      primaryColor: Colors.blueAccent,
      scaffoldBackgroundColor: MyColors.backgroundColor0(Brightness.dark),
      barBackgroundColor: MyColors.backgroundColor1(Brightness.dark)
    );

    Brightness brightness = MediaQuery.of(context).platformBrightness;

    return PlatformProvider(
      builder: (context) => PlatformApp(
        title: "Cali Move App",
        android: (context) {
          return MaterialAppData(
            darkTheme: materialDarkTheme,
            theme: materialDarkTheme,
            // themeMode: brightness == Brightness.light
            //   ? ThemeMode.light
            //   : ThemeMode.dark
          );
        },
        ios: (context) {
          return new CupertinoAppData(
            theme: brightness == Brightness.light
                ? cupertinoTheme
                : cupertinoThemeDark,
            // home: MyHomePage(),
          );
        }
      ),
    );
  }
}

But I can't query the brightess with MediaQuery in the build method because MediaQuery.of() called with a context that does not contain a MediaQuery .

A possible workaround is to wrap the PlatformProvider with an MaterialApp like this: void main() => runApp(MaterialApp(home: MyApp()));

Just a hunch, could you get the brightness within the context of setting the theme
return PlatformProvider(
      builder: (context) => PlatformApp(
        title: "Cali Move App",
        android: (context) {
          return MaterialAppData(
            darkTheme: materialDarkTheme,
            theme: materialDarkTheme,
            // themeMode: brightness == Brightness.light
            //   ? ThemeMode.light
            //   : ThemeMode.dark
          );
        },
        ios: (context) {
          Brightness brightness = MediaQuery.of(context).platformBrightness;
          return new CupertinoAppData(
            theme: brightness == Brightness.light
                ? cupertinoTheme
                : cupertinoThemeDark,
            // home: MyHomePage(),
          );
        }
      ),
    );

I thought that too but it doesn't seem to work. I get the same error this way.

That leaves the question of how to do this with just CupertinoApp itself since the theme property is set before the CupertinoApp is built.

There is CupertinoTheme.brightnessOf(context) that may work. The code it has is
  static Brightness brightnessOf(BuildContext context, { bool nullOk = false }) {
    final _InheritedCupertinoTheme inheritedTheme = context.dependOnInheritedWidgetOfExactType<_InheritedCupertinoTheme>();
    return inheritedTheme?.theme?.data?._brightness ?? MediaQuery.of(context, nullOk: nullOk)?.platformBrightness;
  }

ref: https://github.com/flutter/flutter/blob/ef62092b9b8b1c92de9f16108814c0a3b34cb59d/packages/flutter/lib/src/cupertino/theme.dart#L82

Though it again may be an issue because of the way the CupertinoApp is built..
Widget build(BuildContext context) {
    final CupertinoThemeData effectiveThemeData = widget.theme ?? const CupertinoThemeData();

    return ScrollConfiguration(
      behavior: _AlwaysCupertinoScrollBehavior(),
      child: CupertinoUserInterfaceLevel(
        data: CupertinoUserInterfaceLevelData.base,
        child: CupertinoTheme(
          data: effectiveThemeData,
          child: Builder(
            builder: (BuildContext context) {
              return WidgetsApp(

ref: https://github.com/flutter/flutter/blob/ef62092b9b8b1c92de9f16108814c0a3b34cb59d/packages/flutter/lib/src/cupertino/app.dart#L330

Some flutter related issues:
https://github.com/flutter/flutter/issues/42089
https://github.com/flutter/flutter/issues/42473
https://github.com/flutter/flutter/issues/33873

Option B could be making a something like... (Just a thought experiment.)
return PlatformThemeProvider(
   child: PlatformProvider(
        androidThemes: MaterialThemingData(light: materialLightTheme, dark: materialDarkTheme),
        iosThemes: CupertinoThemingData(light: cupertinoTheme, dark: cupertinoThemeDark)
        builder: (context) => PlatformApp(

Then perhaps for each PlatformScaffold build it checks the brightness (Brightness brightness = MediaQuery.of(context).platformBrightness;) and then instructs the PlatformProvider to use the dark or light theme for the platform  if it is different.
The PlatformApp then can get the Theme (dark or light) from the PlatformThemeProvider as it rebuilds, which will return either the dark or light theme.
This would also eliminate the need to set the light or dark theme on the MaterialApp and the theme on the CupertinoApp using the MaterialAppData or CupertinoAppData objects.

Most probably solution
Use CupertinoDynamicColor
final cupertinoTheme = new CupertinoThemeData(
      primaryColor: CupertinoDynamicColor.withBrightness(
             color: Colors.blueAccent,
             darkcolor: Colors.blueAccent,
      ),
      scaffoldBackgroundColor: CupertinoDynamicColor.withBrightness(
            color: MyColors.backgroundColor0(Brightness.light),
            darkColor: MyColors.backgroundColor0(Brightness.dark),
      )
      barBackgroundColor: CupertinoDynamicColor.withBrightness(
            color: MyColors.backgroundColor1(Brightness.light),
            darkColor: MyColors.backgroundColor1(Brightness.dark),
      )
    );

return PlatformProvider(
      builder: (context) => PlatformApp(
        title: "Cali Move App",
        ios: (context) {
          return new CupertinoAppData(
            theme: cupertinoTheme
          );
        }
      ),
    );

This should/will pick the light or dark themes correctly.
https://github.com/flutter/flutter/blob/5df63738f8ca865c1f1ed03e76bd504e9c9bfd73/packages/flutter/lib/src/cupertino/colors.dart#L888
https://github.com/flutter/flutter/blob/ba7176416f79fedb87d4bb8f2693ec1e7f2d45d1/packages/flutter/lib/src/cupertino/colors.dart#L946

It actually does not really set the text color in dark mode. Could that be?

Ah you can also listen to the system brightness and set is as state in your AppState:
class _AppState extends State<App> with WidgetsBindingObserver {
  Brightness _brightness = WidgetsBinding.instance.window.platformBrightness;

  @override
  void initState() {
    super.initState();
    WidgetsBinding.instance.addObserver(this);
  }

  @override
  void dispose() {
    WidgetsBinding.instance.removeObserver(this);
    super.dispose();
  }

  @override
  void didChangePlatformBrightness() {
    setState(() {
      _brightness = WidgetsBinding.instance.window.platformBrightness;
    });
  }

  ...

    return Theme(
      data: _brightness == Brightness.light ? materialTheme : materialDarkTheme,

...

Closing as any issues around this is related to flutter itself and not this library


Make createTestImage sync for tests

This PR makes the createTestImage() method in tests sync which makes it easier to use in tests.
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] All existing and new tests are passing.

Closing as Picture.toImageSync() is not supported on the HTML backend.
Will open another PR to add a Sync version of the method.


feat(AlgebraicGeometry): define affine n-space

I'm not sure why the "Post PR summary comment" CI job is failing but the build itself passes.

Ops
bors d+

Thanks for the review!
bors merge


Ruler

This is a pull request about the "ruler feature".
-show frame in "media space" or the frame number of the clip (right click on the ruler to show contextual menu)
-Ctrl-left and Ctrl-right to snap the ruler to the end or beginning an item
-Show previous and next frames when the rule is at the beginning of a clip.
-fix bug where markers in the TimeSlider would disappear when the marker was selected and ctrl was pressed.
Some methods will need to be adapted once the functionalities about Time Coordinate Space will be available.

Codecov Report

Merging #451 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #451      +/-   ##
==========================================
- Coverage   87.16%   87.08%   -0.09%
==========================================
  Files          67       67
  Lines        6788     6788
==========================================
- Hits         5917     5911       -6
- Misses        871      877       +6

Impacted Files
Coverage Δ

opentimelineio/adapters/fcp_xml.py
91.86% <0%> (-0.47%)
:arrow_down:

opentimelineio_contrib/adapters/fcpx_xml.py
93.28% <0%> (-0.47%)
:arrow_down:

opentimelineio_contrib/adapters/hls_playlist.py
90.68% <0%> (-0.31%)
:arrow_down:

Continue to review full report at Codecov.

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


Delete LICENSE

Why?  I don't see any other indicator of the license.

well, @ghost (which is an account of a deleted user), if you want us to delete that file, please open a pull request and justify.


new sample: dynamic 3D speech bubble positioning sample (like A Short Hike)

Please check if the pull request fulfills these requirements

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[x] CHANGELOG.md has been updated to describe this change

To update the documentation on yarnspinner.dev, please visit the documentation repository.

What kind of change does this pull request introduce?

[ ] Bug Fix
[x] Feature
[ ] Something else

What is the current behavior? (You can also link to an open issue here)

There's currently no sample code for this common use case: positioning NPC dialogue above the NPC character in the game world, like in exemplar Yarn Spinner games Night In The Woods or A Short Hike.

What is the new behavior (if this is a feature change)?

This is some sample code and scene for dynamically positioning speech bubbles above characters in game world + constraining the speech bubble UI within the screen bounds.
I only wanted to demonstrate the "hard" part (calculating and positioning in UI space) and therefore it has zero player movement, NPC interaction, or gameplay code. (Which is already served by the Space example anyway)

Does this pull request introduce a breaking change? (What changes might users need to make in their application due to this PR?)

None that I know of

Other information:

This is the last big sample thing I'm going to submit? I think with this + visual novel + phone chat + space, all the basic use cases are now covered

This is absolutely fantastic. Thank you so much for this PR!


Can WhenRepeatedFilter prevent logging events until time period has passed?

Type
Question
NLog version:
4.4.11
Platform: .NET 4.6
Hi, is it possible for the WhenRepeated filter to prevent logging the first log message?
I'd like to be able to log events only after they have been occurring for at least 30 minutes.

I think your could use the bufferTargetWrapper for that: https://github.com/NLog/NLog/wiki/BufferingWrapper-target


Error: Unknown CPI error 'Unknown' with message 'getaddrinfo: Name or service not known (esxi_host:443)' in 'create_stemcell' CPI method

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Create two instances in one AZ and disk .....
Sections of cloud config relevant to bug reproducibility

CPI Error Log
Can be obtained by running bosh <env params> task <failed-task-id> --cpi
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Release Version & Related Info (please complete the following information):

CPI Version
BOSH Director Version
Stemcell Name & Version
vCenter Version
NSX(T/V) Version (If using)

Additional context
Add any other context about the problem here.

No logs or detail in this bug at all. Failure is in getaddrinfo() which indicates DNS lookup failure. Host name is esxi_host which is probably incorrect.


CloudOps Alliance workshop

Description
I feel that our alliance principles should be at a higher level, something in the format of the page below.
https://design-system.service.gov.uk/community/
A 45 minute meeting within the team should be able to bash this out.
https://ministryofjustice.github.io/cloud-operations/documentation/general-information/our-alliance.html#our-alliance

First workshop completed on Friday 18th of March.
This page needs to be updated.
Next meeting set for Tuesday 29th at 10am.


Example with postcodes not working

Hey! Unfortunately I'm having a hard time getting this to run correctly.
I'm using copy-pasted postcodes example from readme.
Unfortunately, it only detects the last entity:

Expected output based on ReadMe:
{ index: 0, status: 'COMPLETED', data: { name: 'Los Angeles', postcode: '90001', population: 3971883 }, entity: 'postcodes' }
{ index: 1, status: 'COMPLETED', data: { name: 'San Francisco', postcode: '94102', population: 883305 }, entity: 'postcodes' }
{ index: 2, status: 'COMPLETED', data: { name: 'San Diego', postcode: '92101', population: 1425976 }, entity: 'postcodes'}

If I iterate over the entityStream instead, it looks like this:

Is this some known issue, or did I miss anything? Thanks for your help!

Hello thank you for reporting, this is indeed not the expected behavior, let me investigate.

The alias for the model gpt-3.5-turbo has recently been updated. When pointing it to the old model, it works as expected.

The gpt-3.5-turbo model alias will be automatically upgraded from gpt-3.5-turbo-0613 to gpt-3.5-turbo-0125 on February 16th.
https://platform.openai.com/docs/models/gpt-3-5-turbo

This may be due to a difference in how the tokens are returned in the stream, which could reveal a bug in the parser. I'm looking at the parser and trying to come up with a fix.

Awesome, thank you, will test later! :-)


Azure data studio ui extremely slow

Issue Type: Bug
The ui takes about a couple seconds to respond to all input on windows 10. Even as I'm typing this I'm about five words ahead of the ui.
Azure Data Studio version: azuredatastudio 1.35.1 (fab63efd307e54c063c8af03474bbb5d77f5fd11, 2022-03-15T18:03:38.261Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (8 x 2803)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.69GB (16.01GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

looks similar to #18524

I'm experiencing the same behavior. Restarting ADS does not work, is there anything else to try? If I click the File menu, for example, it can take up to 5 seconds to open. Nothing else on my machine is sluggish (though I could use more memory). It just started yesterday and I'm not sure how to fix it. I installed the user version, changed the name of my C:\Users\nhoward\AppData\Roaming\azuredatastudio folder, it created a new one (I thought it might), but no performance difference.

@cpfnickhoward are you seeing really high CPU or memory consumption?  Are you using any particular features, such as running queries against tables with BLOBs or using query charts (both are known to be slow in some situations)?
There is also a perf profiler built into the "Help->Toggle Developer Tools" Performance pane.  You can run a trace while experiencing slow perf to get an idea of what may be going on.  I'm not sure if the trace contains PII so please don't upload to GitHub, but if you see any Hotspots, such as highlighted in the screenshot below, if you can see which functions are running that could help better locate what's going on.  Thanks!

@kburtram It's slow as soon as I open it. I'll checkout the developer tools in just a bit and report back. Really appreciate the help!

@kburtram It doesn't look like it's doing a ton. I get quite a few layout shifts. I'm not sure what's normal though. I did compare to another machine when doing the same operations. The hardware is different, but it seemed like there were fewer layout shifts and not as much going in with Raster on the machine where things are working normally.

@kburtram I'm not sure what happened, but I decided to do another reboot and it seems to be working normally now. Sorry for bothering you. I really appreciate your help!

I'm seeing some real sluggish performance as well with a fresh install of Azure Data Studio 1.39.1.
When I run a simple query that selects from an SQL Server 2019 table that has maybe 6 columns and only 10 rows, the query results grid takes a second or more to appear, and then it appears with empty columns before finally populating the columns with data retrieved.

As a follow-up to my comment 22 days ago (as of this writing) I just updated to 1.40.0. If I thought it was slow before, now it's even slower!
I just tried running a query that selects a single row from an SQL Server and it took almost 45 seconds for the application to update and render my results grid! And it isn't like the row I am selecting from the server has a bunch of data in it or is even not optimized. The application itself is terribly slow and is almost unusable now with how sluggish the performance on it is.

@stephajn
Can you confirm if your observation is "after" the SQL Tools Service is started?
Please note ADS activates extensions on load, and once you see "SQL Tools Service Started" message in the status bar (left), that should be the ideal moment to start measuring time from.
Please share performance profiler trace as mentioned by @kburtram above, if otherwise.

@cheenamalhotra
I never see a message about "SQL Tools Service Started" in the status bar. By any chance, is there an extension I have to install to improve my experience here? I had understood that Azure Data Studio works just fine with Azure SQL, and just about every modern version of SQL Server out of the box. I have absolutely no extensions installed at all.
As for the performance profiler trace, here's a screenshot of what the experience looked like.

It's a bit tricky to see, but the screenshot timeline shows a really long time waiting for a single row of results to come back.  And even when the results grid did appear, the cells were all blank for a solid 20 seconds or so before they finally filled in with the actual data retrieved.
Meanwhile the status bar of the application showed that SQL Server returned the results in 1 second. So the entire time, it's all waiting for the application to render everything.


VideoWriter generates vertically flipped video with the MSMF backend on Windows 7, 64bit

The cv::VideoWriter() on Windows 7, 64bit, creates vertically flipped videos when using the Microsoft Media Foundation backend. We have used the H264 fourcc and mp4 container.
To verify that the problem is not in our code, we created the video also using the Video for Windows backend with the MSVC fourcc and avi container. The resulting video was not vertically flipped.

Here is a patch that makes CvVideoWriter_MSMF::WriteFrame() copy the frames  in the correct order. The ARM case is unchanged, because we could not test it. But it has probably the same problem.
Index: cap_msmf.cpp
--- cap_msmf.cpp	(revision 5490)
+++ cap_msmf.cpp	(revision 5491)
@@ -4371,9 +4371,9 @@
#else
hr = MFCopyImage(
pData,                      // Destination buffer.

       cbWidth,                    // Destination stride.

       (BYTE*)videoFrameBuffer,    // First row in source image.

       cbWidth,                    // Source stride.

       cbWidth,                   // Destination stride.

       ((BYTE*)videoFrameBuffer) + (videoHeight-1)*cbWidth,    // First row in source image.

       -cbWidth,                    // Source stride.
       cbWidth,                    // Image width in bytes.
       videoHeight                 // Image height in pixels.
       );

Are you up to supplying a PR with the suggested fix? Details on how to do so can be found here. Thanks in advance!

I hope I have created the PR correctly.

Thank you for the pull request. Keep an eye out at the following page: http://pullrequest.opencv.org/#/summary/ and see if any of the buildbots turn red. If so you will need to address any issues that arise.

Since PR is being merged, you can close this down!


'Upgrade to v9.0.0'

Hi @Shazwazza I'm happy to help out with https://github.com/Shazwazza/Articulate/issues/381 later this week. As a starting point it would be nice if we use umbraco 9.0.0 instead of an early preview. 😄

Nice :)


More than one matching billing account found

Getting this error while running terraform install script

data.google_billing_account.acct: data.google_billing_account.acct: More than one matching billing account found

Found why the error: I have two different billing accounts with the same name (ofcourse with different billing account id), one is active and the other one is closed.

Can anyone reproduce the error? I can't create a billing account with the existing name, even one is closed and one is active.

Does it work if you have two billing accounts (even with different names)?

Yes. The only thing is it will make you choose which account you want to use.

This problem should have been addressed before, because we are using a filter for active billing accounts.


"Undefined template variable" During pytest Runs

I'm trying to integrate django-cotton into a mature project, and I'm running into issues when default vars not being used during pytest runs.
Templates
cotton/button.html:
<c-vars type="button" classes="" />
<button type="{{ type }}" class="c-button {{ classes }}" {{ attrs }}>
  {{ slot }}
</button>

index.html:
<c-button classes="foo">Cotton Button</c-button>

Issue
When running the app, the component renders with type="button" as expected. However, running pytest, I get:
Failed: Undefined template variable 'type' in '/path/to/app/server/templates/cotton/button.html'

I've tried this on a simpler project (cloned the Mozilla Library Tutorial), and it works as expected, so it's my project. The challenge is that I'm not strong with the templating system, so any guidance on where to look would be great.
I appreciate any help and/or guidance!
Extra Info
Settings
Happy to provide more if helpful.
INSTALLED_APPS = [..., 'django_cotton']

TEMPLATES = [
    {
        'APP_DIRS': True,
        'BACKEND': 'django.template.backends.django.DjangoTemplates',
        'DIRS': [
            BASE_DIR.joinpath('server', 'templates'),
        ],
        'OPTIONS': {
            'context_processors': [
                'django.contrib.auth.context_processors.auth',
                'django.template.context_processors.debug',
                'django.template.context_processors.i18n',
                'django.template.context_processors.media',
                'django.contrib.messages.context_processors.messages',
                'django.template.context_processors.request',
                'server.apps.base.context_processors.export_vars',
                'server.apps.base.context_processors.meta',
            ],
        },
    },
]

pytest Stack Track
.venv/lib/python3.11/site-packages/django/template/base.py:880: in _resolve_lookup
    current = current[bit]
.venv/lib/python3.11/site-packages/django/template/context.py:83: in __getitem__
    raise KeyError(key)
E   KeyError: 'type'

During handling of the above exception, another exception occurred:
.venv/lib/python3.11/site-packages/django/template/base.py:886: in _resolve_lookup
    if isinstance(current, BaseContext) and getattr(
E   AttributeError: type object 'Context' has no attribute 'type'

During handling of the above exception, another exception occurred:
.venv/lib/python3.11/site-packages/django/template/base.py:896: in _resolve_lookup
    current = current[int(bit)]
E   ValueError: invalid literal for int() with base 10: 'type'

During handling of the above exception, another exception occurred:
.venv/lib/python3.11/site-packages/django/template/base.py:715: in resolve
    obj = self.var.resolve(context)
.venv/lib/python3.11/site-packages/django/template/base.py:847: in resolve
    value = self._resolve_lookup(context)
.venv/lib/python3.11/site-packages/django/template/base.py:903: in _resolve_lookup
    raise VariableDoesNotExist(
E   django.template.base.VariableDoesNotExist: Failed lookup for key [type] in [{'True': True, 'False': False,
'None': None}, {'True': True, 'False': False, 'None': None, 'csrf_token': <SimpleLazyObject: <function csrf.
<locals>._get_val at 0x129357e20>>, 'user': <Mock id='4984067536'>, 'perms': PermWrapper(<Mock id
='4984067536'>), 'LANGUAGES': (('en', 'English'),), 'LANGUAGE_CODE': 'en-us', 'LANGUAGE_BIDI': False,
'MEDIA_URL': '/media/', 'messages': [], 'DEFAULT_MESSAGE_LEVELS': {'DEBUG': 10, 'INFO': 20, 'SUCCESS':
25, 'WARNING': 30, 'ERROR': 40}, 'request': <WSGIRequest: GET '/'>, 'DEVELOPMENT': True, 'DOMAIN_NAME':
'domain.test', 'tenants_count': 0, 'first_additional_tenant': None, 'project_meta': {'NAME': 'Company', 'URL':
'http://domain.test', 'DESCRIPTION': 'Some Text', 'IMAGE': 'https://www.domain.com/image.jpg', 'KEYWORDS':
'neat, thing', 'CONTACT_EMAIL'<EMAIL_ADDRESS>'TITLE': 'Page Title'}, 'page_url': 'https:////', 'page_title':
'', 'page_description': '', 'page_image': '', 'slot': 'Cotton Button', 'ctn_unprocessable_dynamic_attrs': set(),
'attrs': 'classes="foo"', 'attrs_dict': {'classes': 'foo'}, 'classes': 'foo'}]

During handling of the above exception, another exception occurred:
tests/test_apps/test_index_view.py:24: in test_root_page
    response = index(request)
server/apps/main/views.py:24: in index
    return render(request, 'index.html')
.venv/lib/python3.11/site-packages/django/shortcuts.py:24: in render
    content = loader.render_to_string(template_name, context, request, using=using)
.venv/lib/python3.11/site-packages/django/template/loader.py:62: in render_to_string
    return template.render(context, request)
.venv/lib/python3.11/site-packages/django/template/backends/django.py:61: in render
    return self.template.render(context)
.venv/lib/python3.11/site-packages/django/template/base.py:175: in render
    return self._render(context)
.venv/lib/python3.11/site-packages/django/test/utils.py:112: in instrumented_test_render
    return self.nodelist.render(context)
.venv/lib/python3.11/site-packages/django/template/base.py:1005: in render
    return SafeString("".join([node.render_annotated(context) for node in self]))
.venv/lib/python3.11/site-packages/django/template/base.py:1005: in <listcomp>
    return SafeString("".join([node.render_annotated(context) for node in self]))
.venv/lib/python3.11/site-packages/django/template/base.py:966: in render_annotated
    return self.render(context)
.venv/lib/python3.11/site-packages/django_cotton/templatetags/_component.py:97: in render
    return tpl.render(ctx)
.venv/lib/python3.11/site-packages/django/template/backends/django.py:61: in render
    return self.template.render(context)
.venv/lib/python3.11/site-packages/django/template/base.py:175: in render
    return self._render(context)
.venv/lib/python3.11/site-packages/django/test/utils.py:112: in instrumented_test_render
    return self.nodelist.render(context)
.venv/lib/python3.11/site-packages/django/template/base.py:1005: in render
    return SafeString("".join([node.render_annotated(context) for node in self]))
.venv/lib/python3.11/site-packages/django/template/base.py:1005: in <listcomp>
    return SafeString("".join([node.render_annotated(context) for node in self]))
.venv/lib/python3.11/site-packages/django/template/base.py:966: in render_annotated
    return self.render(context)
.venv/lib/python3.11/site-packages/django_cotton/templatetags/_vars_frame.py:43: in render
    c_vars[key] = value.resolve(context)
.venv/lib/python3.11/site-packages/django/template/base.py:723: in resolve
    return string_if_invalid % self.var
.venv/lib/python3.11/site-packages/pytest_django/plugin.py:703: in __mod__
    pytest.fail(msg)
E   Failed: Undefined template variable 'type' in '/path/to/app/server/templates/cotton/button.html'

@Dresdn which version of cotton are you on?

The original post was using 0.9.38. I just updated it to 0.9.40 and have the same results.
As an FYI, I was thinking that the way pytest loads Django, maybe something was being loaded/read too soon, so I tried using the implicit registration. That had the same results.

Thanks for taking a look. I wasn't able to reproduce in a fresh project either.
I hoped something was glaringly missing, so I'll keep stripping down the project to see if I can pinpoint anything.
Feel free to close or leave this open, but it'll take me a few days to really get into this.

@wrabit I couldn't let it go, but I figured out what is causing the failure. In our pyteset options, we use pytest-django's --fail-on-template-vars. Removing this allows the tests to pass.
The challenge is, looking at what they're doing for this option, I'm not sure if there's a good way to solve this on the Cotton end.
I'm thinking I'll submit an MR there to allow ignoring template directories or something.

@Dresdn I did have a play with this and even when I tried to catch the exception in cotton, it still seemed to trigger the fail on pytest..

This looks to be pretty core to Django since the django-pytest team is setting the string_if_invalid when the flag is enabled. I did a quick search in the project, and seems it's pretty challenging issue. This thread is a good read and gives the sentiment of the maintainers: https://github.com/pytest-dev/pytest-django/pull/922
AdamChainz wrote an interesting blog about the perils of doing so, and it seems we're hitting the default filter challenge.

Actually, if anything, maybe just a note about pytest-django users and the flag when using <c-vars> with defaults? Could same someone some headache.

it seems we're hitting the default filter challenge.

Ah makes sense, that's what's behind the defaults for cvars.
I'll make a note to add this to the docs/readme


AttributeError: 'list' object has no attribute 'lower'

Hello,
when running subliminal on file "Suicide Squad (2016) 1080P WEB-DL Extended cut Hevc Bluury.mkv" I get the given error.
Full error trace :
$ subliminal download -l fr "Suicide Squad (2016) 1080P WEB-DL Extended cut Hevc Bluury.mkv"  Collecting videos  [####################################]  100%                                                                 1 video collected / 0 video ignored / 0 error Downloading subtitles  [####################################]  100%  Suicide Squad (2016) 1080P WEB-DL Extended cut Hevc Bluury.mkv Traceback (most recent call last):   File "/usr/local/bin/subliminal", line 11, in <module>     sys.exit(subliminal())   File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 722, in __call__     return self.main(*args, **kwargs)   File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 697, in main     rv = self.invoke(ctx)   File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 1066, in invoke     return _process_result(sub_ctx.command.invoke(sub_ctx))   File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 895, in invoke     return ctx.invoke(self.callback, **ctx.params)   File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 535, in invoke     return callback(*args, **kwargs)   File "/usr/local/lib/python2.7/dist-packages/click/decorators.py", line 27, in new_func     return f(get_current_context().obj, *args, **kwargs)   File "/usr/local/lib/python2.7/dist-packages/subliminal/cli.py", line 403, in download     hearing_impaired=hearing_impaired, only_one=single)   File "/usr/local/lib/python2.7/dist-packages/subliminal/core.py", line 211, in download_best_subtitles     for s in subtitles], key=operator.itemgetter(1), reverse=True)   File "/usr/local/lib/python2.7/dist-packages/subliminal/score.py", line 106, in compute_score     matches = subtitle.get_matches(video)   File "/usr/local/lib/python2.7/dist-packages/subliminal/providers/opensubtitles.py", line 98, in get_matches     matches |= guess_matches(video, guessit(self.movie_release_name, {'type': 'movie'}))   File "/usr/local/lib/python2.7/dist-packages/subliminal/subtitle.py", line 235, in guess_matches     if video.format and 'format' in guess and guess['format'].lower() == video.format.lower(): AttributeError: 'list' object has no attribute 'lower'

I'm also getting this error a while ago.

+1 on latest version as well

it works with Deathspike patch !
Thanks to him

Duplicate: https://github.com/Diaoul/subliminal/issues/761


"Edit Sketch" only appears on first "startProfileAt" ?

See video. I thought it was my branch, but it's on prod too:
https://github.com/KittyCAD/modeling-app/assets/547923/60e1a218-cf0d-43a1-ae4b-d6b1d176b2f1

Our tests were probably passing before because they were zooming so fast that the Edit Sketch button probably stayed visible long enough.

Is it possible this is duplicate with #2426?

This bug has already been resolved. It is working correctly within main. I tested locally myself with multiple startProfileAt and I was able to see edit sketch.
I checked and we already have a playwright test to test this exact scenario so the regression case is covered. The test is called multi-sketch file shows multiple Edit Sketch buttons within sketch-tests.spec.ts.


(WIP) Stable Diffusion on Web

Use the stable diffusion code and port it into the webgpu canvas element. Pretty sure this can be done purely in python with no JavaScript. The canvas element is already implemented in the webgpu package.

Not enough for a bounty lock.


[TCFMM-14] Add some set_position tests

Description
This PR adds some tests for the set_position entrypoint and fixes a few bugs I encountered while working on these tests.
Details about each change are in each commit's message.
More tests for this and other entrypoints will be added in separate PRs (see the TODOs left in Test.SetPosition).
Related issue(s)

Resolves part of TCFMM-14
:white_check_mark: Checklist for your Pull Request

Related changes (conditional)

Tests

[x] If I added new functionality, I added tests covering it.
[x] If I fixed a bug, I added a regression test to prevent the bug from
silently reappearing again.

Documentation

[x] I checked whether I should update the docs and did so if necessary:

Specification
README
Haddock

Stylistic guide (mandatory)

[x] My commits comply with the policy used in Serokell.
[x] My code complies with the style guide.

Note: there is a lot of duplication at the moment, I'll DRY it up in a future PR.


Update Jupyter API version and fix build

There are two problems:

There was a recent change that broke the compatibility, I wrote about it in Kotlin Slack: https://kotlinlang.slack.com/archives/C4W52CFEZ/p1630318024011800
Problem of compatibility between KSP versions really exist, but I don't know how to solve it on Jupyter API plugin side, because we can't adjust the plugin version while applying another plugin in our plugin. So, if you use some unstable version of Kotlin, you can apply the corresponding version of KSP to the build. Alternatively, you can use the Jupyter integration without using scanner and annotations, this approach is described here.

My bad, I missed the Slack message and should have read your docs more carefully. Thank you for the PR!


Generate reva configuration from Helm Values

This PR allows chart users to modify individual settings of Reva daemon directly either
by Helm values.yaml or by helm upgrade ... --set config.some-reva-setting=value method, rather
than by rewriting the whole revad.toml file. It also allows for inheritance of common Reva settings in subcharts,
leveraging built-in Helm values inheritance.
Currently used way of configuring Reva using static toml files:
configFiles:
  revad.toml: |
    [grpc]
    ...

remains still possible.
Contributing a Chart / update to an existing Chart

[x] Run helm lint on the chart dir.
[x] (Update) Bump the Chart.yaml version before merging, to release it as a new version.
[x] (Update) Replace the annotations on the Chart.yaml with:

artifacthub.io/changes - the changes introduced on the PR with the appropiate format.
artifacthub.io/images - the updated tag on the cs3org/revad image.

[x] (Update) If the PR includes new configurable parameters in the chart's values.yaml. Add documentation in the appropiate README.

@dagl please review

Any news on this @dagl @labkode? It has some potential to help with simplifying reva site configurations


make check fails with "false" casting compilation error

What steps will reproduce the problem?
1. svn checkout of trunk.
2. configure and build
3. make check

Platform: Ubuntu 12.04.1 LTS

What is the expected output? What do you see instead?

string_util_test.cc: In member function ‘virtual void
kmlbase::StringUtilTest_TestFromString_Test::TestBody()’:
string_util_test.cc:162:3: error: converting ‘false’ to pointer type for
argument 1 of ‘char
testing::internal::IsNullLiteralHelper(testing::internal::Secret*)’
[-Werror=conversion-null]

...

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 29 Jan 2013 at 11:01

there been a lot of modification in the github clone libkml/libkml which also fixed your issues
Could you try if the problem exists with current master


issue with draw_landmarks() function defined in mp.solutions.drawing_utils

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
Yes
OS Platform and Distribution
WINDOWS 11
Python Version
3.11.1
MediaPipe Model Maker version
0.10.11
Task name (e.g. Image classification, Gesture recognition etc.)
hand detection
Describe the actual behavior
AttributeError: 'NormalizedLandmark' object has no attribute 'landmark' . I am using mp.solutions.holistic model
Describe the expected behaviour
It should had run without a problem
Standalone code/steps you may have used to try to get what you need
def draw_landmarks(
    image: np.ndarray,
    landmark_list: landmark_pb2.NormalizedLandmarkList,...)

'''this is the function written in drawing_utils , here is issue I am facing is with landmark_pb2.NormalizedLandmarkList.'''

'''Some part of code I wrote is -> '''

mp_drawing = mp.solutions.drawing_utils
mp_hands = mp.solutions.holistic

with mp_hands.Holistic(min_detection_confidence=0.8, min_tracking_confidence=0.5) as hands:
    while cap.isOpened():
        ret, frame = cap.read()

        image = cv2.cvtColor(frame, cv2.COLOR_BGR2RGB)

        image = cv2.flip(image, 1)

        image.flags.writeable = False

        results = hands.process(image)

        image.flags.writeable = True

        image = cv2.cvtColor(image, cv2.COLOR_RGB2BGR)

        if results.left_hand_landmarks:
           for hand_landmarks in results.left_hand_landmarks.landmark:
               mp_drawing.draw_landmarks(frame, hand_landmarks, mp_hands.HAND_CONNECTIONS)
               keypoint_pos = []
               for i in range(21):
                   x = hand_landmarks.landmark[i].x * frame.shape[1]
                   y = hand_landmarks.landmark[i].y * frame.shape[0]
                   keypoint_pos.append((x, y))

'''It is throwing the error-'''

Traceback (most recent call last):
  File "d:\VIRTUAL PIANO\VIRTUAL-PIANO\mp_hand_gesture\mp_hand_detection.py", line 100, in <module>
    mp_drawing.draw_landmarks(frame, hand_landmarks, mp_hands.HAND_CONNECTIONS)
  File "C:\Users\mpaks\AppData\Local\Programs\Python\Python311\Lib\site-packages\mediapipe\python\solutions\drawing_utils.py", line 160, in draw_landmarks
    for idx, landmark in enumerate(landmark_list.landmark):
                                   ^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NormalizedLandmark' object has no attribute 'landmark'

Other info / Complete Logs
No response

Hi @akshat-1,
You are presently utilizing the legacy Holistic solution. This solution has been enhanced to the new Holistic Task API starting from version 0.10.10. We have ceased support for legacy solutions entirely. Kindly test the new Holistic Task API and inform us if you encounter similar errors.
Thank you!!


[Fix issue#1180] Support resubmit FAILED trial

FAILED job can be resubmitted

resubmit button will be disabled when experiment status is DONE | RUNNING | STOPPED or the job  resubmit successfully

This PR is closed because of having new design.


added my profile link

added my profile link

@jeel2308 thanks for contributing please do follow me :+1:


Priority

Use case
I'm using vim-cursorword together with inkarkat/vim-mark.
Underline of vim-cursorword takes precedence over marks placed by vim-mark.
This is inconvenient since there is no immediate visual feedback after marking a word by pressing \m. I have to move the cursor to see the marks.

vim-cursorword has a priority of -1 (unconfigurable)
vim-mark by default has a priority of -10 (configurable via g:mwMaxMatchPriority)
both priorities have to be negative, otherwise, they would override hlsearch

Feature request
I would like to configure vim-cursorword match priority.
Or leave it unconfigurable, but at least make it small enough, like -100. In this case it is possible to revert to the old behavior by setting g:mwMaxMatchPriority to e.g. -200.
Or, if it is possible, make both plugins not interfere with each other. I.e. make vim-cursorword underline not to override vim-mark backgrounds.

I changed the priority to -100. Please confirm.

Now it works as desired, thank you.


CSL-1624 Always fail in compile-time if CSL_SYSTEM_TAG isn't set

In travis-ci/pr all builds succeeded. In travis-ci/push MacOS build hasn't started yet, even though 1.5 days passed. I think /pr build is much more important and the fact that it passed (and 3 builds in /push and appveyor build) is quite enough to merge it in this case.


Create constants for top-level rolenames

Fixes #1648
This is a change in the metadata API to remove hardcoded role names
and use constants instead.

[x] The code follows the Code Style Guidelines
[x] Tests have been added for the bug fix or new feature - updated only
[ ] Docs have been added for the bug fix or new feature - n/a

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

46 of 46 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 97.464%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
3951

Relevant Lines:
4035

💛  - Coveralls

I agree with both of you ...
On one hand, Root, Targets etc. classes already exist in metadata.py and now if we add similar constants for role names, we'll be importing things like
from tuf.api.metadata import Targets, TARGETS

On the other hand, names do grow in size.
I was thinking if it there is a convenient data structure than would:

encapsulate rolenames
act as iterator allowing us to call list(Rolenames); for role in Rolenames: and by doing this replaces TOP_LEVEL_ROLE_NAMES too
have a short enough name :shrug:

But this is only a suggestion and if implementing this feels like an overkill for python-tuf, I'm ok with simple constants.

@lukpueh do you have an opinion on the matter?
I am asking you because we had discussed such issues before with you.

FWIW, in warehouse they use an enum subclass to do this, maybe that's the data structure @sechkova was thinking of, it seems to at least allow iterating etc.? :)
Another way of encapsulation would be to make them class attributes of the corresponding metadata classes.
I'm really fine either way.

I think enum covers all the cases, as @lukpueh also mentioned.

I pushed an update with enum.

It seems like the major reason to use an enum or a class is to avoid top level variable name pollution (so to avoid from tuf.api.metadata import Root, ROOT, Timestamp, TIMESTAMP, ....), correct?
What if instead of a new class we used the classes we are already importing? Root and others already have a class variable _signed_type which is already exposed as an instance property type... Maybe we could just simplify that so the class variable is called type (and is public): then we could just refer to Root.type to get the string without instantiating a Root?

What if instead of a new class we used the classes we are already importing? Root and others already have a class variable _signed_type which is already exposed as an instance property type... Maybe we could just simplify that so the class variable is called type (and is public): then we could just refer to Root.type to get the string without instantiating a Root?

This is what I meant above with ...

Another way of encapsulation would be to make them class attributes of the corresponding metadata classes.

One downside would be that we'd have to define a global TOP_LEVEL_ROLES that uses these class variables after the class definitions, i.e. at the bottom of the metadata module. But I guess that's okay.

One downside would be that we'd have to define a global TOP_LEVEL_ROLE_NAMES that uses these class variables after the class definitions, i.e. at the bottom of the metadata module.

I'm also not going to lose sleep over a single instance of duplicating the strings within the API implementation if that makes the API definitition look neater.

I've put the constants within the role classes. Agree that it's more clear, simple and convenient.
As those are both required private (we don't want to be able to change them, whatever "be able" means in python for private properties :D ), but at the same time we need them for static reference within Updater, and we don't want to break the file format which uses _type, and don't want to do a breaking API change for such a simple reason with changing type to TYPE, we came to and agreement with @jku that changing _signed_type to type and keeping the _type property would be the best possible option.
Yep, it's still not very nice to reference the public variable from the private property, but that's the least painful and best to use change, unless someone comes with a genius idea that we cannot see yet.

it's still not very nice to reference the public variable from the private property, but that's the least painful and best to use change, unless someone comes with a genius idea that we cannot see yet.

I think the genius idea is to just remove _type property: it's from the time when the API matching the file format was considered a bit more important than it is now. IMO the _type property is not worth the confusion it creates . But we can handle that in later PRs.

just remove _type property

Agreed!

it's from the time when the API matching the file format was considered a bit more important than it is now

I  actually suspect that in this case the (Python) implementation influenced the naming decision in the spec, to avoid shadowing the type global on json-to-dict deserialisation.
Regardless of this specific case, I still think that it very important that the spec file format is recognisable in the reference implementation.

Thanks @jku. Just addressed your comments.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

42 of 43 (97.67%)  changed or added relevant lines in 3 files are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage increased (+1.0%) to 98.507%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tuf/ngclient/updater.py
15
16
93.75%

Files with Coverage Reduction
New Missed Lines
%

tuf/ngclient/updater.py
6
95.15%

Totals

Change from base Build<PHONE_NUMBER>:
1.0%

Covered Lines:
3788

Relevant Lines:
3814

💛  - Coveralls


[ROCm] Fix for auto mixed precision simulated GPU unit test failure.

/cc @cheshire

@cheshire gentle ping


[grpc-service] remove control rpcs

This PR removes control RPCs, leaving only Command RPC for network management.

@simonlingoogle , please help resolve conflicts.

Missed some change for this PR. Please hold. @jwhui

@jwhui @ctan-g
Removed CtrlXxxx RPC  related Go code.


WIP: lldap

Description of changes
lldap is a lightweight LDAP server : https://github.com/nitnelave/lldap.
It includes a wasm-based web interface, which this PR fails to build at the moment (see "remaining work" below). In an earlier ttempt, @aanderse used a workaround provided by @alyssais, but we can't use it as is in nixpkgs.
The goal of this WIP PR is to: 1) have a reference PR for lldap packaging, and 2) find a proper way to build the web interface.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

remaining work
[INFO]: Checking for the Wasm target...
Error: wasm32-unknown-unknown target not found in sysroot: "/nix/store/kj59rn7lsqg97y6zpmxv6d9acs2jqjra-rustc-1.64.0"

Used rustc from the following path: "/nix/store/kj59rn7lsqg97y6zpmxv6d9acs2jqjra-rustc-1.64.0/bin/rustc"
It looks like Rustup is not being used. For non-Rustup setups, the wasm32-unknown-unknown target needs to be installed manually. See https://rustwasm.github.io/wasm-pack/book/prerequisites/non-rustup-setups.html on how to do this.

./main.js → ./pkg/bundle.js...
[!] Error: Could not resolve './pkg/lldap_app.js' from main.js
Error: Could not resolve './pkg/lldap_app.js' from main.js
    at error (/nix/store/ln0snppfk8clk121dq0minc21pz504xl-rollup-2.79.1/lib/node_modules/rollup/dist/shared/rollup.js:198:30)
    at ModuleLoader.handleResolveId (/nix/store/ln0snppfk8clk121dq0minc21pz504xl-rollup-2.79.1/lib/node_modules/rollup/dist/shared/rollup.js:22287:24)
    at /nix/store/ln0snppfk8clk121dq0minc21pz504xl-rollup-2.79.1/lib/node_modules/rollup/dist/shared/rollup.js:22250:26

@tournemire any ideas on the front-end?


During execution of "winget search", the percentual progression is accompanied by corrupted text.

Brief description of your issue
During execution of "winget search 𝑥 | sort", the percentual progression is accompanied by corrupted text.
Steps to reproduce
Have executed the command that is "winget search 𝑥 | sort"
Expected behavior
The output should be 1 list of percentages.
Actual behavior
"ÆΓû" is printed many times, but is accompanied by the  𝑥% of progression that should be there.
Environment
[winget --info]
Windows Package Manager version
Windows: Windows.Desktop version
Package: Microsoft.DesktopAppInstaller version

I am not certain that the environment information is correct, but I am not able to verify its correctness because I am not certain of what all of it means.

@BEEDELLROKEJULIANLOCKHART I just tried the steps with my PC and am not seeing that.

your environment in the bug does not provide which version of the OS or Winget client you have.  Can you please provide the numbers?
given the odd characters you are providing, I am wondering if this is a console or some codepage issue.  What is the language of the OS you are running?  What window are you running the command in?  The Command prompt or terminal?
Can you provide the code page of the window?  in command prompt, right click the UPPER LEFT corner and choose properties.  You can get the code page there.

I was running winget via PowerShell Standard that was invoked by 53621FSApps.FluentTerminal within a virtual machine that was being executed by http://github.com/flathub/org.gnome.Boxes from KDE Neon. The machine was running the Windows 11 Preview.

I am not able to provide any more information, as I have done some things to my installation of http://github.com/flathub/org.gnome.Boxes since this report that does mean that the original virtual machine is not accessible. I am apologetic because I may have wasted your time, and thus shall close this issue. However, I can assure that my installation was not corrupted; I had not installed anything but some Universal Windows Platform software from the Microsoft Store.


Add BFS Visualizer Community Contribution

Community Contribution
Link to live project, video, or image:
https://codepen.io/ssmkhrj/full/zYvbYpK

Sharing

[x] I would be happy to have my project shared on The Coding Train's social media!

Love this! Thanks for sharing.


04 1.2 b) As a Developer, I want to Deploy CSV Database Web Service manually to Azure, to write and read cheeps online and not from a locally stored CSV file

User Story
As a Developer, I want to Deploy CSV Database Web Service manually to Azure, to write and read cheeps online and not from a locally stored CSV file
Description of the solution you'd like
By using Azure App services we want to manually deploy the CSV Database Web Service online. Which makes us independent of having a locally stored CSV file, and able to read and write cheeps to other computers using the newly deployed web service url: https://bdsagroup12chirpremotedb.azurewebsites.net/.
Definition of Done

[ ] Use "az webapp up --sku F1 --name bdsagroup12chirpremotedb --os-type Linux --location westeurope -- runtime DOTNETCORE:7.0" to create a new ewb service on Azure.
[ ] Change the Chirp.CLI.Client to using the new url instead of localhost
[ ] After verifying that the app runs on the new url, shutdown that service with the following commands.

az webapp delete
az logout

Make a test that removes the the csv file, and checks if it makes a new one.


chore: use go-tool to install proto

Description
This PR switch form manual go install to go-tools helper that we use in all other places allowing us to keep locked deps.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
CI (make generate)
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/retest


[R-package] Ability to control the printed messages?

Currently, LightGBM is not using the R printf message, leading to bypassing any sink (see #513).
An alternative, without losing performance on metric computation, is to return to R the metric values required and have a callback to print the messages.
Currently, we can log the printed messages by using the metric callback. But that's not the best scenario unfortunately (performance wise, especially on large datasets).
Taking 100M observations, training a model and computing AUC on my machine for 10 iterations:

Using LightGBM metric: 628 seconds
Using optimized R metric: 1159 seconds (optimized R AUC metric has around/less than 2M/s observation throughput, see https://github.com/Laurae2/R_benchmarking#area-under-the-curve-roc-benchmarks)

It would be beneficial for logging the performance of the model throughout the training to a file. Such case would be very useful, especially in RStudio Server: if a user is disconnected (for any reason) from the R session, he/she would not be able to get back on RStudio Server until LightGBM finishes training (logging performance would be a great addition).

@Laurae2
Maybe "redirect" can solve your problem ?
https://stackoverflow.com/questions/6674327/redirect-all-output-to-file

@Laurae2 any response ?

Redirecting this way is not possible when the session creation is not controlled by the user (same issue with any server spawning R sessions).

@Laurae2 got it. However, this require to rewrite the R package, more specifically, the dynamic library used by R.
Maybe the better solution is using some third-party high-performance metric evaluation packages.

Closed in favor of being in #2302. We decided to keep all feature requests in one place.
Welcome to contribute this feature! Please re-open this issue (or post a comment if you are not a topic starter) if you are actively working on implementing this feature.

Reopening this one since we have an active PR (#2901) for it


reminder: markdown recipes/tutorial for comments and description

This is a reminder coming from this message in Finish the Markdown reference/tutorial:
(B4B)

Do you (plural) think we need a tutorial or recipe about markdowns yet?
If so, do we make it a stand alone thing, or do we rather incorporate it to something else (like, writing descriptions)?

(kazk)

Not really for Markdown itself, but I think we should have something for "how to write comments effectively" and also "how to write kata descriptions". For kata description, we should have some recommended style (what to include, how to structure, etc).

This issue could be split as suggested by kazk into two, and the one related to descriptions could be used as a part of breakdown for #138.

For "how to write comments effectively", we can do a Codewars version of Getting Answers (or How To Ask Questions The Smart Way). We should have something to point to with good examples when we have a user who doesn't know how to ask a question or opens issues too quickly. We can also show this before their first comment in the future.

Broken down into #142 and #143.


Most parsltest tasks fail running from cvmfs

I find that only 20-25 of 400 tasks run when I run parsltest with WorkQueue executor on either cori or perlmutter using a cvmfs distribution. All tasks succeed when I use shifter.

Problem above is for desc-wfmon 0.9.1. On Ben's suggestion, I remove the autolabel and autocategory specification from the WorkQueue specification in parsltest.py:

if sexec == 'wq':
   executor = parsl.WorkQueueExecutor(
          autolabel=True,
          autocategory=True,
          worker_options=f"--memory={max_workers}"
         )

This resolved problem and is committed to version 0.9.2.


[dracula theme] consistent selection styling regardless of focus

This prevents selections from inheriting the default #d9d9d9 background when editor is defocused.
reproducible @ https://codemirror.net/demo/theme.html#dracula

I checked dracula-theme, and couldn't any more specific colors. Interestingly, the atom theme seems to do the same, but I still think it's wrong.

@adrianheine
there actually seem to be a bunch of other themes that do this [1]. i think they're all wrong and probably a result of copy/paste/modify. additionally, all the div. prefixes should be dropped imo.
[1] https://github.com/codemirror/CodeMirror/search?utf8=✓&q=CodeMirror-focused

I guess the dark themes should definitely either provide their own unfocused selection color or apply their color regardless of focus, because the default color will look awful. For light ones, I don't care all that much.

FWIW, even for light ones i dont understand why a defocused editor's selection would need to look different. Another thing i found in dracula (maybe others), is that string and string2 share the same color [1], even though literal regexps in js mode get classed as string2. Perhaps this is by design of the theme, i dunno. I split them up locally, among other tweaks.
[1] https://github.com/codemirror/CodeMirror/blob/master/theme/dracula.css#L23


If file not found when downloading student submission, choose where to put the file

Some files in the submission don't exist in the original module so they can't be found I guess

A rare case.


Upgrade path from gogs 0.10

Could you provide a transition path from the latest version of gogs (0.10) to gitea please?

Gogs 0.10 has serval difference with Gitea master. We didn't test it how to upgrade from Gogs 0.10.
As I know the main differences are the hooks directory structure and protected branch table.

I pushed it over anyway by following your upgrade guide. No issues so far. Except it did not create the scripts folder; I just copied the gogs folder.

Martin migration is based on the value found in the version field
of the version database table. The same version could mean different
thing if the database is from Gogs or Gitea. That's the main problem.
Only up to Version 14 (I think) they are compatible.
If you are at a later version, switching to Gitea would fail to apply
any migration meant to go from version 14 to whatever version of Gogs
database you started from.
Maybe you can work around this by re-setting the database version
to 14, but anything done in the Gogs migrations to higher Gogs database
versions would probably not be undone and remain there dangling forever
(in the best scenario) or prevent application of Gitea migrations or
trigger weird malfunctioning (in the worst).
It might be possible (or not) to deal with that somehow via software,
but could be very complex.

How do I check the database version number? Presumably I select some column of some table somewhere.

+----+---------+
| id | version |
+----+---------+
|  1 |      15 |
+----+---------+

But that may have been done by gitea. I don't suppose you know any way I could check?
You might want to update the documentation to include what you wrote above.

No way to tell if it was Gogs or Gitea, that's what I was saying
(ie: it could be useful to add a "flavor" column).
But then I guess you could look the v14.go files in Gogs and Gitea
and see what they do, and check if what you have is what one or the
other requests...

switching from gogs(v0.10.1) to gitea is really easy.
install gitea at different path and new migration from gogs repo.
for example:
local server path: /work/repo/sailfish009/sample.git

I started my gitea off of a gogs-database with version 14. Everytime when gitea starts, I get these warnings:
[W] Table user Column allow_create_organization db default is , struct default is 1
[W] Table user Column diff_view_style db default is '''', struct default is ''
[W] Table user has column created but struct has not related field
[W] Table user has column updated but struct has not related field
[W] Table user has column num_followings but struct has not related field
[W] Table public_key has column created but struct has not related field
[W] Table public_key has column updated but struct has not related field
[W] Table access_token has column created but struct has not related field
[W] Table access_token has column updated but struct has not related field
[W] Table repository has column created but struct has not related field
[W] Table repository has column updated but struct has not related field
[W] Table repository has column enable_wiki but struct has not related field
[W] Table repository has column enable_issues but struct has not related field
[W] Table repository has column enable_external_tracker but struct has not related field
[W] Table repository has column external_tracker_format but struct has not related field
[W] Table repository has column enable_pulls but struct has not related field
[W] Table repository has column enable_external_wiki but struct has not related field
[W] Table repository has column external_wiki_url but struct has not related field
[W] Table repository has column external_tracker_style but struct has not related field
[W] Table deploy_key has column created but struct has not related field
[W] Table deploy_key has column updated but struct has not related field
[W] Table collaboration has column created but struct has not related field
[W] Table action has column act_email but struct has not related field
[W] Table action has column created but struct has not related field
[W] Table issue has column deadline but struct has not related field
[W] Table issue has column created but struct has not related field
[W] Table issue has column updated but struct has not related field
[W] Table pull_request has column pull_id but struct has not related field
[W] Table pull_request has column pull_index but struct has not related field
[W] Table pull_request has column head_barcnh but struct has not related field
[W] Table pull_request has column can_auto_merge but struct has not related field
[W] Table pull_request has column merged but struct has not related field
[W] Table comment has column created but struct has not related field
[W] Table attachment has column created but struct has not related field
[W] Table issue_user has column repo_id but struct has not related field
[W] Table issue_user has column milestone_id but struct has not related field
[W] Table issue_user has column is_poster but struct has not related field
[W] Table issue_user has column is_closed but struct has not related field
[W] Table milestone has column deadline but struct has not related field
[W] Table milestone has column closed_date but struct has not related field
[W] Table mirror has column updated but struct has not related field
[W] Table mirror has column next_update but struct has not related field
[W] Table release has column created but struct has not related field
[W] Table login_source has column created but struct has not related field
[W] Table login_source has column updated but struct has not related field
[W] Table webhook has column created but struct has not related field
[W] Table webhook has column updated but struct has not related field
[W] Table notice has column created but struct has not related field

Is it safe to ignore them?

Yes, that only warning that you tables has some no meaning columns.


Added User presence Tracking

Fixes #19

Hi @sbis04
Can you review this PR?

Thanks for your contribution @rktejesh ! 🎉


Additional column for generated many-to-many table

DBFlow Version: 4.0.0-beta5
Issue Kind (Bug, Question, Feature): Question
Description:
I'm using @ManyToMany to generate a many-to-many relationship. Is there any way to add additional column into the generated many-To-many table ?
Thanks

no, you would be better off just creating the table at that point manually.