Added StrictBytes type

Change Summary
Added StrictBytes type which rejects objects which are castable to bytes (ie. int, float, str), but which are not natively bytes type. All changes are modeled after StrictStr, StrictInt, StrictFloat, and StrictBool types.

Added strict option to ConstrainedBytes class. Default value of the option (False) respects downstream code continuity. Any initialization or subclass of ConstrianedBytes behaves identically to prior implementations unless the "strict" option is explicitly set to True).

Added strict_bytes_validator which rejects any object which is not Union[bytes, bytearray] type. Because bytearray instances are just mutable versions of bytes, we accept both as equivalent native byte types. This is noted explicitly since the builtin bytearray construtor creates objects with the following behavior:

>>> isinstance(bytearray('foo', 'utf-8'), bytes)
False

Related issue number
n/a

Checklist

[x] Unit tests for the changes exist
[x] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[x] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)

cc @hhsecond

Thanks for the review @samuelcolvin and @PrettyWood!


[vue3] - Accessing generically typed props in

Describe the bug
Firstly, apologies if it seems like this should be an issue in the Vue 3 repo. I can move it across if that's the case.
I am trying to create a reusable Vue 3 component with generically typed props.
Let's say I want a props interface that looks something like:
// ChildComponent.vue
interface MyProps<T> {
  propOne: T
  propTwo: () => Array<T>
}

The goal of this is so that something like the below would fail:
// ParentComponent.vue
<template>

  <child-component
    :prop-one="5"
    :prop-two="() => ["foo", "bar"]"
  />
</template

Using this example, and also running into this issue I came up with the following:
// ChildComponent.vue

<template>

{{ propOne }}

</template>

// ChildComponentWithGenerics.vue

This successfully results in the following:
// ParentComponent.vue

<template>

  <child-component
    :prop-one="5"
    :prop-two="getSomeStrings"
  />

  <child-component
    prop-one="vite rocks!"
    :prop-two="getSomeStrings"
  />
</template>

Great, all working fine. That is, until I try and access the props in ChildComponent.vue...
As soon as I add const props =  to then do some logic with the props, I get an error.
// ChildComponent.vue

<template>

{{ propOne }}

</template>

The error I get is the following:

[plugin:vite:vue] Transform failed with 1 error:
/Users/josephmaylor/Desktop/vite-project/src/components/ChildComponent.vue:18:41: ERROR: Unexpected ")"
/Users/josephmaylor/Desktop/vite-project/src/components/ChildComponent.vue:18:41
Unexpected ")"
16 |    expose();
17 |
18 |  const props = __props as Props>()
|                                           ^
19 |
20 |
at failureErrorWithLog (/Users/josephmaylor/Desktop/vite-project/node_modules/.pnpm/esbuild@0.15.10/node_modules/esbuild/lib/main.js:1566:15)
at /Users/josephmaylor/Desktop/vite-project/node_modules/.pnpm/esbuild@0.15.10/node_modules/esbuild/lib/main.js:805:29
at responseCallbacks. (/Users/josephmaylor/Desktop/vite-project/node_modules/.pnpm/esbuild@0.15.10/node_modules/esbuild/lib/main.js:671:9)
at handleIncomingPacket (/Users/josephmaylor/Desktop/vite-project/node_modules/.pnpm/esbuild@0.15.10/node_modules/esbuild/lib/main.js:726:9)
at Socket.readFromStdout (/Users/josephmaylor/Desktop/vite-project/node_modules/.pnpm/esbuild@0.15.10/node_modules/esbuild/lib/main.js:647:7)
at Socket.emit (node:events:526:28)
at addChunk (node:internal/streams/readable:315:12)
at readableAddChunk (node:internal/streams/readable:289:9)
at Socket.Readable.push (node:internal/streams/readable:228:10)
at Pipe.onStreamRead (node:internal/stream_base_commons:190:23

Reproduction
https://github.com/joemaylor/type-generic-props-repro
System Info
  System:
    OS: macOS 12.6
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 25.54 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 8.5.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 106.0.5249.91
    Firefox: 105.0.1
    Safari: 16.0
  npmPackages:
    @vitejs/plugin-vue: ^3.0.3 => 3.1.0
    vite: ^3.0.9 => 3.1.4

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This reproduces with sfc.vuejs.org.
So it's an upstream issue. Please report to vuejs/core.
Closing as it's an upstream issue.


unpin @babel/runtime to see what tests fail

This is just unpinning @babel/runtime to see what tests fail so that we might be able to get to the bottom of the issues with https://github.com/babel/ember-cli-babel/issues/442

@mansona did you get anywhere with this? I just noticed that in 8.2.0 the version is still pinned.


auth, etcdserver: add a method for updating backend during apply snap…

…shot
This commit adds a new method UpdateBackend() to auth.AuthStore for
updating backend during apply snapshot. It follows a manner of the
lessor.

/cc @xiang90

@mitake Can you explain a bit more on what you mean by: "It follows a manner of the lessor."? I don't think I follow how this relates to a lease.

@philips this PR is based on this comment: https://github.com/coreos/etcd/pull/4714#issuecomment-196141930
etcdserver passes a new backend to lease during applying snapshot. Unlike lease, current auth requires etcdserver in its constructor and calls Backend() when it needs backend. I think the way of lease is cleaner so creating this PR. There's no intention of connecting auth and lease in this change.

@xiang90 updated for your comments, PTAL.

lgtm


Plot PSTHs (spike histograms)

For all cells, populations, and single cells

also for the netstims!

would also be nice if you can set the width of the bins so when decide to view a small time interval, will be able to use finer histograms :)

Completed -- see commits labeled with issue #39


Polish DeleteMetadataJob

dry run 작업 하기 전에 한번 보면서 정리해봤습니다.
주로 tc쪽 정리를 한거같네요.

Before I work on dry run, I organized it while looking at it.
I think I mainly organized the tc side.

Codecov Report

Merging #16 (657f767) into main (fb26e03) will decrease coverage by 0.39%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
- Coverage   98.53%   98.14%   -0.40%
==========================================
  Files          34       34
  Lines         753      753
  Branches       35       35
==========================================
- Hits          742      739       -3
- Misses         11       13       +2
- Partials        0        1       +1

Impacted Files
Coverage Δ

...metadata/CheckMaxJobInstanceIdToDeleteTasklet.java
100.00% <100.00%> (ø)

...batch/plus/job/metadata/DeleteMetadataTasklet.java
90.00% <100.00%> (-10.00%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

경계값 테스트를 원활하게 하기 위해 range를 최신 10일로 제한하게 변경했습니다.


Update development

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/mono/SkiaSharp, working directory: W:\2qeu-s, target branch: development, target branch commit: 59ffea1122bd386bf64c445543d630cb9b6d5227, source branch: master, source branch commit: .
Learn more: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


tap-gitlab private token video is broken

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/175
Originally created by @pnadolny13 on 2021-11-30 13:04:03

The video in tap-gitlab that shows how to create a token and its links do not work. We reference it from the Getting Started guide so it probably gets a lot of traffic. We should check on that video and get it fixed.
https://hub.meltano.com/extractors/gitlab#private-token

View 1 previous comment from the original issue on GitLab


Add whereLike functions

Add andWhereLike / andWhereILike / orWhereLike / orWhereILike.
Closes #2265

we probably also needs docs?


Update absl to tag 20190808

Abseil last release:
on Aug 8, 2019  20190808 SHA1: aa84489
Tasks:

[ ] update Makefile based build

[ ] windows
[ ] unix

[ ] update CMake based build
[ ] update Bazel based build

ref: https://github.com/abseil/abseil-cpp/releases

on master absl is already on SHA1: 8ba96a8 which is beyond this release...


Find and Download Favorites from Websites

Hi. I want to know if there is a way to download what I have marked as favorites in my accounts on, e.g. gelbooru and rule34 websites. I found this link in earlier issues, stating that I have to just copy the URL link in the search bar, but it doesn't show anything for me:
https://github.com/Bionus/imgbrd-grabber/issues/516
Does the latest version have this function? If yes, what should I do to find and download my favorites from my account in the websites? If not, would you add the feature? I would be grateful if you would reply. Thank you.
Best wishes,

On danbooru, you can query for a user's favortes using the fav:<username> query, for example. I don't know about the other sites.

The latest version should still allow you to paste URLs to have them parsed, but it's far from ideal.
The best solution is to use the actual search terms:

Gelbooru supports it in Grabber using fav:USER_ID
Shimmie (used by R34) supports it "naturally" using favourited_by=USERNAME or favorited_by_userno=USER_ID (doc)

The latest version should still allow you to paste URLs to have them parsed, but it's far from ideal.
The best solution is to use the actual search terms:

Gelbooru supports it in Grabber using fav:USER_ID

Are you sure this works for Gelbooru? No results on mine every time using 'fav:215127' (example not my actual account lol) Thanks for the help

Are you sure this works for Gelbooru? No results on mine every time using 'fav:215127' (example not my actual account lol) Thanks for the help

You need to be logged in in Gelbooru  to use this syntax.


adding migrated trade-monitor

the trade-monitor demo has been migrated to 5.0 and added to the project

@olukas my bad, correct version is committed. Please take a look

All PR review issues have been fixed. Merging this.


⚠ Faculté génie website (prod) has degraded performance

In 36748de, Faculté génie website (prod) (https://www.usherbrooke.ca/genie/) experienced degraded performance:

HTTP code: 200
Response time: 1924 ms

Resolved: Faculté génie website (prod) performance has improved in 35786a2.


broadcast ordering on join messed up

sometimes packet order Login -> Play can get broadcasts before get final login packet (I think....)

think a5ce63e28b0277b63f9c45ab1ce6a6639eaeb286 fixes this


Nekopoi

Source name
Nekopoi
Source link
https://nekopoi.care/
Source language
Indonesia
Other details
"18+ NSFW=YES
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a title with source name.
[X] I have checked that the extension does not already exist by searching the GitHub repository and verified it does not appear in the code base.
[X] I will fill out all of the requested information in this form.

Can't this be added?


Databases: Created seeding for orders table

Description
Created seeding for orders table
Fixes #105
How to test?
npm run db:setup
Checklist

[X] I have performed a self-review of my own code
[X] I have followed the name conventions for CSS Classnames and filenames, Components names and filenames, Style filenames, if you are in doubt check the the project README.MD and here https://github.com/HackYourFuture-CPH/curriculum/blob/master/review/review-checklist.md
[X] I have commented my code, particularly in hard-to-understand areas, if you code was simple enough mark the box anyway
[X] I have made corresponding changes to the documentation, if you code was simple enough mark the box anyway
[X] This PR is ready to be merged and not breaking any other functionality

Merged this branch for seeding of orders table, after approval from the reviewers 👍


tx slot add to address

We may use the To address in erigon, and if we decode the slot.Rlp then it will cost unnecessary resource.
So it's better to cache the To address in slot, and can be used then.

We have develop some other functions,such as black/white address list(include to address). When a transaction will be added into pool, we need check the "to address" is in the list or not. And that time we can not get the "to address", must decode the Tx.Rlp, and the decode logic will cost some time.
And we have some other places like that, should get "to address", but only can get it through decode.

https://github.com/okx/xlayer-erigon/pull/69/files#diff-1aec050642647934c35d918aab3b343ecb189b81fa4143ff52beab5499253876R78


Cloud Asset: Add export to BigQuery sample and test

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
Please run down the following list and make sure you've tried the usual "quick fixes":

Search the issues already opened: https://github.com/googleapis/nodejs-asset/issues
Search the issues on our "catch-all" repository: https://github.com/googleapis/google-cloud-node
Search StackOverflow: http://stackoverflow.com/questions/tagged/google-cloud-platform+node.js

If you are still having issues, please be sure to include as much information as possible:
Environment details

OS:
Node.js version:
npm version:
@google-cloud/asset version:

Steps to reproduce
Cloud Asset: We add a new feature which enable user to export assets into Bigquery tables. This is a bug for my cl to add export to BigQuery sample and test
Making sure to follow these steps will guarantee the quickest resolution possible.
Thanks!

@googleapis/cloud-asset-team

closing as stale.


Cannot read property 'zip4j' of undefined

The following code works perfectly in iOS but fails in Android:
        import { Zip } from "nativescript-zip";

        Zip.unzip({
            archive: this.path_set.compressed_fullpath,
            directory: this.path_set.destination_folder,
            onProgress: function(percent) {
                console.log(`unzip progress: ${percent}`);
            }
        }).then(() => {
            console.log(source, 'File is decompressed.');
        });

The error message is:
Cannot read property 'zip4j' of undefined
My version info:
Android 8.0 - API 26
tns: 6.2.2
<EMAIL_ADDRESS>tns-android: 6.2.0
<EMAIL_ADDRESS>

I don't understand what you say.
Can you make a zip, example, with winzip or 7z, add it as an asset in a NS project and decompress it with NS-ZIP plugin ?? I know the question is strange...

Ok, after lot more tests, here are the results:
It works fine, not exactly as expected, but fine.
My first mistake was on the import of Zip. My fault.
But Zip.unzip should return a promise that you can execute. It does, but as soon as you get the promise, it start its execution. It does not wait to be resolved, don't know why.
But this was easy to fix: simply declare it right at the point I need to use it. Then use the "then" part to actually run more actions in case of success.
So, well done. It works good.  Closing this issue.
Regards,
Herman.


About repo

I think you should alllow others to download the older versions still by making a branch with a label of the version

I no longer have the old version + This project has been neglected for years


Fix warning directive rendering

I think this is what we want. It does fix the rendering for the warning directive but I'm not sure why this wasn't showing up in other directives...
Closes #367

Sorry for the dealy.


Some epubs' doesn't display correctly over multiple pages

Bug Report
What happened?
The attached epub's chapter shows in a single page, where it should show over several pages.
https://user-images.githubusercontent.com/38096011/199537331-a167f47a-0f17-4765-8b07-a173600f3432.mp4

Expected behavior

Only a part of the chapter should show, and the rest of the chapter should be displayed after navigating to the next page.
Actual behavior
The chapter is rendered in a single page.
How to reproduce?

الحرب_والسلم_(الكتاب_الأول).epub.zip
The ebup download url: https://downloads.hindawi.org/books/26307461.epub
Environment

Readium version: 2.2.1

Development environment

OS: macOS Ventura 13.0
IDE: Android Studio Dolphin | 2021.3.1 Patch 1

Testing device

Android version: 11
Model: Realme Narzo 50i
Is it an emulator? No

Additional context

Are you willing to fix the problem and contribute a pull request? Yes, Please

This epub has overflow-x: hidden CSS for the <body> tag. That is what is causing this.  @mickael-menu should Readium CSS be changed to always override that?

Thank you for investigating @stevenzeck.

should Readium CSS be changed to always override that?

Probably, I'll transfer the issue there.

Follow-up on https://github.com/readium/readium-css/issues/119

Reopening as it works fine on Thorium and iOS with overflow-x: hidden. See the Readium CSS issue for more feedback.

I don't think it's an Android thing, but a Chrome thing. Here is one of the HTML files of the book in the browser, including Readium CSS.

I don't think it's an Android thing, but a Chrome thing. Here is one of the HTML files of the book in the browser, including Readium CSS.
https://user-images.githubusercontent.com/8315038/199868224-1198905f-2d41-4cfb-8737-41ca979984ef.mov


Return a Config.Config in MachineInventory

Return a full config.Config with the Elemental key containing the usual
values used for the elemental-installer and the DATA key with the raw
values, so they can be interpreted by the elemental-installer to produce
the cloud-config to set in the machine.
Otherwise, we cant support cloud-config generic configurations like
write_files without updating the Machineinventory API every time we want
to add one, this way we allow setting an uncapped cloud-config in the
machineinventory and that will be passed down to the elemental-installer
Part of #34
Would probably require first to get #32 properly set, so we can test this deeply with an authenticated request. Currently the tests only test the registrationURL unauthenticated, which doesn't produce the full output, only a limited output with just the elemental.registration fields filled and drops the rest. Once the auth part is fixed (cacerts I guess?) or I understand how to do an authenticated request, I would feel safe to merge this.
Signed-off-by: Itxaka<EMAIL_ADDRESS>

I'm missing documentation (resp. examples) of how a matching machineRegistration yaml would look like.
Otherwise :+1: from me.

Codecov Report

Merging #35 (d335aed) into main (14d4d95) will decrease coverage by 0.18%.
The diff coverage is 29.41%.

@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
- Coverage   42.55%   42.36%   -0.19%
==========================================
  Files           9        9
  Lines         658      668      +10
==========================================
+ Hits          280      283       +3
- Misses        358      365       +7
  Partials       20       20

Impacted Files
Coverage Δ

pkg/config/config.go
0.00% <0.00%> (ø)

pkg/server/register.go
13.87% <0.00%> (+0.15%)
:arrow_up:

pkg/config/read.go
61.02% <100.00%> (+0.43%)
:arrow_up:

Continue to review full report at Codecov.

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


[HOLD for payment 2023-08-07] [$1000] Approved Hidden Comment Displays Incorrectly Until Refresh

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Report a comment as Intimidation or Bullying
As Concierge, approve the message (i.e. clear the flag)

How to approve as Concierge:

Go to https://www.expensify.com/concierge/#/
Click Modify Tiers
Select the manifesto-tier
When a flag comes through a chat will be opened
Click clear the flags on the options that come through

Expected Result:
The message should appear as it did before the flag
Actual Result:
The message - until refreshing the page - appears as the "This message has been flagged as violating our community rules and the content has been hidden." copy, without the reveal button or line on the side.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Version Number: 1.3.35-6
Reproducible in staging?:  Needs reproduction requires concierge contribution
Reproducible in production?: Needs reproduction requires concierge contribution
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/403dabb6-796f-4a64-ba91-286abfe51aad
Expensify/Expensify Issue URL:
Issue reported by: @dangrous
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688151033866789
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014e00f1dee1b2af9a
Upwork Job ID:<PHONE_NUMBER>823794688
Last Price Increase: 2023-07-12

asking follow up question in slack

Daniel says that he expects the solution to be external but that it will be hard to test without internal access (to Concierge to clear flags) so might need to be a mix of internal and external. Starting off with the internal flag 🎏

Hmm I think the internal label was supposed to assign an internal engineer. Adding engineering label to assign someone

@dangrous I'm going OOO for a week and a half starting thursday. Since this bug deals with moderation, I'm going to assign over to you since I believe you implemented that. Please let me know if I'm mistaken though!

yep! my report of a bug with my feature haha. I'll see what I can do.
I'm wondering if this might be a good opportunity to pair with an external engineer, I can let them use my ngrok backend and I can handle the concierge side when it's ready to test. I think that would be pretty easy, but I'm not sure we've officially done that before

That's cool! I think @MonilBhavsar and @chiragsalian have done that before.

Great, switching this to external
Hi Contributors!
Because you'll need a moderator to respond to your flag in order to test this, I'm here for you! Once your proposal is accepted, I can provide you with a link you can use to my backend, and I can be your Concierge agent and help test. And, I think this is clear but what happens if a flag is cleared is a new moderationDecision is added to the message, with the status of approved. That should hopefully be all you need to diagnose and create a proposal, but if you have questions let me know.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Approved Hidden Comment Displays Incorrectly Until Refresh
What is the root cause of that problem?
The root cause of the issue lies here in this code block.
https://github.com/Expensify/App/blob/a047b88851917807b036f3c5b8564c5add9ea9d2/src/pages/home/report/ReportActionItem.js#L158-L167
If we observe the code carefully we have a state called isHidden in the useEffect hook we only setting the value as true to isHidden state. Means even in case of update false we're not overriding the state.
Typically what we're doing is we're only setting state value as true we're not resetting the state if the moderationDecision changes which means till we refresh the state value going to be true and it's staying same.
What changes do you think we should make in order to solve the problem?
We should add a else check in the useEffect hook,  and reset the state value to false. There can be many ways to set it to false but the final solution is to make it false works.

What alternative solutions did you explore? (Optional)
NA
Result
iu.webm

@Santhosh-Sellavel let us know when you've had a chance to review the proposals . Seems pretty straightforward, I think!

@b4s36t4's proposal looks good! @dangrous

Cool! I'm pretty sure just doing it where @b4s36t4 pointed it out should be fine; we'll only ever get a cleared flag from the backend so latestDecision will change whenever we would want to put a comment back to normal.
Given that it's Friday, though, and to test this we need some back and forth and coordination with my backend, I'm not going to assign until Monday (so we should be in a good position for the 3-day bonus rule). But I will do so then!

@dangrous It's 3 business days (72 business hours), so weekends are excluded from the count. If we can get something to get started. I am sure we can wrap this up Tuesday, what do you say?

oh! For some reason I thought it counted weekends. Sweet, I'll assign now. I think that should be fine. @b4s36t4 ping me when you're ready to test, we can maybe find some time early Monday!

@dangrous I think I'm done from my end. Please do let me know when we can connect for backend connection.

I'll be available from 9am-5pm US Central Time (UTC -5) on Monday, if there's a time in there that can work? If that's too tricky I can probably find some time outside of that too. Just let me know! (you can ping me in the app too, if that's easier than going back and forth here)

Ok will connect with you on Monday at 10AM(your time) after that.

Sounds great!

Shush Melvin, we're testing and will have PR up soon!

We're having some trouble testing, but it's with my development environment - not anything on @b4s36t4's side, so any delay shouldn't affect the potential speed bonus.
Hopefully I'll be able to get it sorted today and we can still merge tomorrow, but wanted to get ahead of it if not

Still having issues unfortunately, continuing to ask for internal engineer assistance on troubleshooting. This has actually been an issue with my dev environment for a while haha but it just has never mattered before (Pusher does not seem to send Concierge updates).

@b4s36t4 helped me figure out a way around my backend issues, so we've got a slightly janky way to test this, even without fixing my overall error. PR is coming shortly - looks like the code works!

This is on prod! Hopefully we'll get the automated comment soon, but writing here for posterity.

@b4s36t4 fix+bonus $1500
@Santhosh-Sellavel review+bonus $1500 - please complete the checklist and request payment via NewDot

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[x]  [@Santhosh-Sellavel] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/19476
[x]  [@Santhosh-Sellavel] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/19476/files#r1289489139
[ ]  [@Santhosh-Sellavel] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ]  [@Santhosh-Sellavel] Determine if we should create a regression test for this bug.
[ ]  [@Santhosh-Sellavel] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ]  [@sonialiap] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

Will we need a regression test here?

So I don't think this would have been caught during existing testing, but I think it's next to impossible to have as a regular test due to - as you mentioned, @Santhosh-Sellavel - the fact that we'd need Concierge to respond. I think we can skip it

Reviewed the details for @Santhosh-Sellavel. Approved for payment in NewDot based on the BZ summary above.


Documentation: Task List

[ ] Authentication
[ ] Database Migration
[ ] CORS Support
[ ] Testing
[ ] Using the Template tutorial
[ ] OpenAPI Specification generation
[ ] PersistentStore Implementations
[ ] Improve Tutorial Navigation
[ ] Deployment Options (Heroku, EC2)

We should also have a section on debugging an aqueduct application.


Docker task not supporting writing output to variable

I created a pull request for Docker to support writing the output of a Docker command to a variable. This can be used to chain Docker commands. (E.g. obtaining all images and then running a docker command to remove them).
Since I have gotten almost no reply's on my PR I decided to create an issue regarding this as well.
All details regarding this are available in the PR: #4005

We are currently working on more formal support for output variables in vsts tasks.  We expect to ship that soon, so once it is in place you can update your PR and we can take a look at it.

@chrisrpatterson, any update on the output variable support yet in VSTS? (Is there a preview available yet :)?)

It's coming in the sprint that's just starting to roll out now (119)

Great to hear :). Love the work you're doing guys!

@bryanmacfarlane @chrisrpatterson , Any update on this yet?

Early preview if feature should be rolled out
https://github.com/Microsoft/vsts-agent/blob/master/docs/preview/outputvariable.md

@bryanmacfarlane , ah so this still requires some changes in the actual Docker task?

Closing as fixed with pull request: #8183 and will be available in the next deployment.


Odd floats when in narrow screen.

I'm not sure what's the most helpful information to give you to chase this down, but please let me know what I can provide to help. My workflow involves often splitting my terminal with tmux so that nvim is only on half my screen. When this is done often any bigger floats get slammed up against the side. The border actually looks fine, but it's like the text shifts over a bit too far. Here is an example of what I mean while in Lua.

Here is another example in Scala using Metals when I have full screen:

However, if I then split my screen and do the hover on the same symbol it looks like this:

I had notice this.. but I don't find a way to solve this ..

a min code , increase the col value . the window does not move right .. not sure problem of upstream.
function move_right()
  local test_opts = {
    anchor = "NE",
    col = 10,
    height = 10,
    relative = "cursor",
    row = 2,
    style = "minimal",
    width = 31
  }
  local contents_bufnr = api.nvim_create_buf(false, true)
  api.nvim_buf_set_lines(contents_bufnr,0,-1,true,{'sssssssssssssssssssssssss','bbbbbbbbbbbbbbbbbbbbbbb'})
  local contents_winid = api.nvim_open_win(contents_bufnr, false, test_opts)
  return contents_bufnr, contents_winid
end

@ckipp01 can you provide a min scala code that ca reproduce it. I think I found a way to solve this

For sure. Here is a minimal init file to use:
local cmd = vim.cmd
----------------------------------
-- PLUGINS -----------------------
----------------------------------
cmd [[packadd packer.nvim]]
require('packer').startup(function(use)
  use {'wbthomason/packer.nvim', opt = true}

  use {'glepnir/lspsaga.nvim'}
  use 'scalameta/nvim-metals'

end)

local saga = require 'lspsaga'
saga.init_lsp_saga()
----------------------------------
-- OPTIONS -----------------------
----------------------------------
vim.o.shortmess = string.gsub(vim.o.shortmess, 'F', '') .. 'c'

local function map(mode, lhs, rhs, opts)
  local options = {noremap = true, silent = true}
  if opts then
    options = vim.tbl_extend('force', options, opts)
  end
  vim.api.nvim_set_keymap(mode, lhs, rhs, options)
end

map('n', 'K', '<cmd>lua require"lspsaga.hover".render_hover_doc()<CR>')
----------------------------------
-- COMMANDS ----------------------
----------------------------------
-- LSP
cmd [[augroup lsp]]
cmd [[autocmd!]]
cmd [[autocmd FileType scala setlocal omnifunc=v:lua.vim.lsp.omnifunc]]
cmd [[autocmd FileType scala,sbt lua require("metals").initialize_or_attach({})]]
cmd [[augroup end]]

Instruction to use with nvim-metals and Scala

Make sure to have coursier installed as nvim-metals needs this to install Metals.
Clone https://github.com/ckipp01/metals-minimal-sbt
Open this while using the init file. You'll see the warning about Metals not being installed, so just do a :MetalsInstall which will install the latest Metals.
Restart Nvim
Open this file: https://github.com/ckipp01/metals-minimal-sbt/blob/master/src/main/scala/example/Hello.scala
You'll  see a popup to import you build. Do this and wait for indexing to complete.
Make sure your screen is split (not vim split, but like your terminal split or something to make it narrow) and do a hover on this line https://github.com/ckipp01/metals-minimal-sbt/blob/3fbf5bd99419086209adf839544937c44c446d9b/src/main/scala/example/Hello.scala#L33

At this point you should see this:

Let me know if you need anything else.

Thanks for your kind reply. no need lua code . I need this scala code.

hmm seems like I can't get melas work..

hmm seems like I can't get metals work..

What sort of issues are you having?

:MetalsInfo I got it . but can't completion and get doc ..

Ahh, you need to make sure you have java installed as well. Do you have java installed?

I had install openjdk. but java -version gave me this
The operation couldn’t be completed. Unable to locate a Java Runtime.
Please visit http://www.java.com for information on installing Java.

🤔 since you'll have coursier installed, you can also try this I guess:
This should install Java
cs java --jvm adopt:11.0.9

This should tell you if your Java home is set
cs java-home

If the above command doesn't return anything, you can do a
cs java-home --jvm adopt:11.0.9

which should then set your java home to that jdk.
Sorry that Java is being a pain. Are you on a Mac and did you install Java with Brew by chance?

on Mac and brew install openjdk 15.0.1

ah, the jdk will need to be 8 or 11, not 15.  Let me try to think of another easier way to get you an environment that you can test this with, or I will spend some time myself trying to find the issue.

OK  lspconfig works.. so I will try the jdk11 for nvim-metals

OK lspconfig works.. so I will try the jdk11 for nvim-metals

👍🏼 just make sure to not have Metals registered with both nvim-metals and lspconfig at the same time or you'll start 2 servers at the same time.

Yep I know that.

seems like brew install coursier will auto install jdk15

How about now?

Please update and give a test.

So yes, this does seem to fix it!

Thanks a lot for all of your effort in trying to chase this down. Some part of me feels like part of the issue is just that so many of the signatures in Scala will be long, so they wrap odd and don't always look the best. However, that's because of Scala, not because of saga.
Do you know why in your first image the second line Strings are not colored the same as the first?

It seems that manual line wrapping will destroy some highlights

👍🏼 sounds good thanks a lot. 🚀

I will fix highlight later. I found the reason.

fixed.


Update workflows

Codecov Report
Patch coverage: 45.43% and project coverage change: -32.33 :warning:

Comparison is base (62552b4) 100.00% compared to head (bb16d84) 67.67%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##            master      #82       +/-   ##
============================================
- Coverage   100.00%   67.67%   -32.33%
============================================
  Files            5        9        +4
  Lines          334     1154      +820
============================================
+ Hits           334      781      +447
- Misses           0      373      +373

Impacted Files
Coverage Δ

src/objprint/executing/_position_node_finder.py
34.78% <34.78%> (ø)

src/objprint/executing/executing.py
61.61% <55.68%> (ø)

src/objprint/executing/_exceptions.py
57.14% <57.14%> (ø)

src/objprint/executing/__init__.py
75.00% <75.00%> (ø)

src/objprint/frame_analyzer.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Bug: PowershellSensor (Satellite) reports "Unknown"

Please make sure that the issue you're planning to report is not already registered :)

Describe the bug
Using the PowershellSensor and trying to output a log file to it shows up as "Unknown" in Home Assistant even tho the test button outputs the text just fine. I'm using get-content E:\SteamCMD\logs\output.txt as the command.
To Reproduce
Steps to reproduce the behavior:

Create a PowershellSensor
Enter get-content E:\path\to\log\file\output.txt
Check Home Assistant

Expected behavior
It should store the log file as a text in Home Assistant.
Screenshots
-
Misc info (please complete the following information):

Windows build (ideally screenshot/info of winver.exe output): Server 2022 21H2
Windows' UI language: English
HASS.Agent version: 2.0.2beta2 (also happened on latest official version)

Please check what's applicable (multiple answers possible):

[x] Installed via installer
[ ] Installed manually
[ ] Problem occurs in HASS.Agent
[X] Problem occurs in Satellite Service

Additional context
This has also already occurred in the original latest release of HASS.Agent.
Logs
No errors are being reported in either the satellite log nor normal log.

Also tried the normal "local sensor" one and it produces the same result.

After a few more tries to store the sensors I occasionally get the following error in either the satellite or the normal log:
2024-02-29 17:13:23.125 +01:00 [FTL] [SENSORSMANAGER] Error while publishing: Collection was modified; enumeration operation may not execute.
System.InvalidOperationException: Collection was modified; enumeration operation may not execute.
   at System.Collections.Generic.List`1.Enumerator.MoveNextRare()
   at System.Linq.Enumerable.TakeWhileIterator[TSource](IEnumerable`1 source, Func`2 predicate)+MoveNext()
   at System.Linq.Enumerable.TakeWhileIterator[TSource](IEnumerable`1 source, Func`2 predicate)+MoveNext()
   at HASS.Agent.Sensors.SensorsManager.Process() in D:\a\HASS.Agent\HASS.Agent\src\HASS.Agent\HASS.Agent\Sensors\SensorsManager.cs:line 133
2024-02-29 17:13:24.227 +01:00 [INF] [SENSORS] Modified single-value sensor: powershellsensor-test
2024-02-29 17:13:24.227 +01:00 [INF] [SETTINGS_SENSORS] Stored 1 entities

Hmm, I don't seem to be able to reproduce this:

Could you please check the Home Assistant logs? I'm asking because HA has a limit of payload it can handle - 255 characters - is it possible the log file is bigger than this?

Aw damn... yes that was the issue...
I tried sending a log file over which was about 1KB big, course it's too big then.
Maybe this would be a good mention in the help-text to the right of the sensor for such scenarios 😅

Done - https://github.com/hass-agent/HASS.Agent/pull/58 :)

Awesome, thanks!


[Feature] Extend topology policy for overriding application labels/ annotations

Is your feature request related to a problem? Please describe.
We plan to use OAM + kubevela to generate set of CRDs/K8s manifests for different environments.  The CRDs and K8s resources names/ properties require environmentType to drive uniqueness and discoverability.
To achieve that we plan to attach some application specific metadata (envType, name ) in the labels/ annotations.
So, consider the following OAM :
apiVersion: core.oam.dev/v1beta1
kind: Application
metadata:
  name: mygolangservicemultienv
  namespace: mygolangservice
  labels:
    environment: development #Type of environment, For ex: development, staging, prod
  annotations:
    path: "some/local/path"
spec:
  components:
    - name: mygolangservicemultienv
      type: webservice
      properties:
        image: mygolangservice
      traits:
        - type: managed-ingress
  policies:
    - name: target-development
      type: topology
      properties:
        clusters: ["common"]
        labels:
          environment: development
    - name: target-staging
      type: topology
      properties:
        clusters: ["common"]
        labels:
          environment: staging
  workflow:
    steps:
      - name: development
        type: deploy
        properties:
          policies: ["target-development"]
      - name: manual-approval
        type: suspend
      - name: staging
        type: deploy
        properties:
          policies: ["target-staging"]

It generated two different environments : development and staging
The servicename and managed ingress dns address for development env should be:  mygolangservicemultienv-development, mygolangservicemultienv-development.cloud.tech,
and for staging should be : mygolangservicemultienv-staging, mygolangservicemultienv-staging.cloud.tech
Describe the solution you'd like
We would like to extend the topology policy to override the labels and annotations.
Describe alternatives you've considered

Use namespace to scope the generated CRDs :  It is not entirely possible, because our organization shares namespace because permissions are whitelisted by namespace.
Create your policy definition : It will not work, as we would like to change the context of the application. A new policy definition can only patch certain properties to few components.
Expose labels as properties in components and pass labels directly to components by  override policy: This solution is not scalable as for every component definition we need to expose labels fields. And often our application may require multiple components sharing same context. So, it will require lot of same label context being repeated.

Thanks, agreed it may work. My third alternative is exactly this approach. I am hesitant for this approach as our application may have multiple components and we would like to avoid repetition. And want that environment specific context is available by default.

Thanks, agreed it may work. My third alternative is exactly this approach. I am hesitant for this approach as our application may have multiple components and we would like to avoid repetition. And want that environment specific context is available by default.

Well, as for the "environment" there is a long history debating how this should be designed and implemented. One of the critical thing is that different people have different opinion for the "environment" and it is hard to reach a consensus. Therefore, in the core controller of KubeVela, we discard the design of environment and leave it for upper layer, like VelaUX or other platforms.
If you are worrying about the repetition of attaching labels to resources from multiple components, one of the good thing is that the "override" policy can decorate all components if you do not write the name or type of the component. Like
  components:
    - name: mygolangservicemultienv-1
      type: webservice
      properties:
        image: mygolangservice
      traits:
        - type: managed-ingress
    - name: mygolangservicemultienv-2
      type: webservice
      properties:
        image: mygolangservice
      traits:
        - type: managed-ingress
  policies:
    - name: override-deployment
      type: override
      properties:
        components:
          - traits:
              - type: labels
                properties:
                  environment: development

Or you can use wildcard like "name: mygolangservicemultienv-*" to select components for modifying.


Add excel parser for AMG files

Converted PDF file to Excel and wrote a simple parser to pull all the values from it

You absolute champion. I've already done some points, slots and schema updates on my loadout branch so I'll merge this in and use the parser to do the rest!

Can you update package.json with the requirements for your script and add run instructions to the readme?

Now that I've worked out how to use yarn properly, I've incorporated your parser into #506 in commit 8f5cf73, so I will close this PR


Remove xsi attributes

I ran into this connecting to a registry—it appears that xsi attributes are no longer a part of the XML payload.
I see them in rfc3730 but they seem to be absent in the latest rfc5730.
Have you seen these attributes in other registries like hexonet? I'm not sure if these should be optional instead.

Hexonet doesn't check for them but I had them in there because I remember them being optional some years ago, so I had them in there.
Since they're not part of the RFC, it's good to remove them. Thanks.


PowerToysRun Plugin Clock & Date detects 4 number + 1 input as year & Date

Microsoft PowerToys version
0.58.0
Running as admin

[ ] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
Enter Any 4 Character Number as Year input, space, another Number as Month and it assumes a day automatically.
only 4 Numbers don't trigger "Clock and Date" search.

4 Numbers + 1 till 12 triggers Autocomplete Date Detection.

Workaround:
Deactivate "Clock and Date" in Global Search.
P.S
Why does it show Clock 00:00 there? Because the Autocomplete for Date uses 9999 1 1 00:00?
✔ Expected Behavior
Limiting Year Recognition to 50 or 100 years around current date!
❌ Actual Behavior
Takes even 9999 as year if you add a 1 afterwards

Other Software
No response

We use the parsing logic from .net/windows.
But it might make sense to have a setting like "interpretation of numbers only with activation keyword".

@jaimecbernardo
Can you please fix label and project

Fix released in v0.59.0


Added NETWORK env var to btcd service

it never passes, so the btcd service connects to simnet everytime
shoutout to @darwin for the help
Pull Request Checklist

[x] If this is your first time contributing, we recommend you read the Code
Contribution Guidelines
[x] All changes are Go version 1.12 compliant
[x] The code being submitted is commented according to Code Documentation and Commenting
[x] For new code: Code is accompanied by tests which exercise both
the positive and negative (error paths) conditions (if applicable)
[x] For bug fixes: Code is accompanied by new tests which trigger
the bug being fixed to prevent regressions
[x] Any new logging statements use an appropriate subsystem and
logging level
[x] Code has been formatted with go fmt
[x] For code and documentation: lines are wrapped at 80 characters
(the tab character should be counted as 8 characters, not 4, as some IDEs do
per default)
[x] Running make check does not fail any tests
[x] Running go vet does not report any issues
[x] Running make lint does not report any new issues that did not
already exist
[x] All commits build properly and pass tests. Only in exceptional
cases it can be justifiable to violate this condition. In that case, the
reason should be stated in the commit message.
[x] Commits have a logical structure according to Ideal Git Commit Structure

Could you also add it for ltcd below?

Done


installation method of the wp scaffold movefile should be changed

The package index will be closed, so we have to change the installation method of the wp scaffold movefile.
https://github.com/vccw-team/wp-cli-scaffold-movefile/pull/8

Existing packages will be hosted in package index, so it close.


Feat: New Login/Signup Page

Describe the feature
@SUGAM-ARORA I want to add a new login/signup page as this is not responsive. Kindly assign this issue to me.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Ojas-Arora Issue assigned to you


[HUDI-8161] Make spark-sql command 'desc' independent from schema evolution config

Change Logs
Addressing this issue https://github.com/apache/hudi/issues/11858
Impact
Now desc command provides full table information independently from schema-evolution settings.
Risk level (write none, low medium or high below)
none
Documentation Update
none

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@wombatu-kun Thanks for adding this feature! Do you plan to make this ready for review soon?

Yes, i'll try to finish it in a week or two.

@yihua It's ready for review.
I think, these errors:
2024-09-09T09:52:05.2331384Z [ERROR]   ITTestHoodieSanity.testRunHoodieJavaAppOnMultiPartitionKeysCOWTable:72->testRunHoodieJavaApp:181->ITTestBase.executeCommandStringInDocker:263->ITTestBase.executeCommandStringInDocker:274->ITTestBase.executeCommandInDocker:197->ITTestBase.executeCommandInDocker:246 Command ([/var/hoodie/ws/hudi-spark-datasource/hudi-spark/run_hoodie_app.sh, --hive-sync, --table-path, hdfs://namenode/docker_hoodie_multi_partition_key_cow_test_20240909094931513, --hive-url, jdbc:hive2://hiveserver:10000, --table-type, COPY_ON_WRITE, --hive-table, docker_hoodie_multi_partition_key_cow_test_20240909094931513, --use-multi-partition-keys]) expected to succeed. Exit (1) ==> expected: <0> but was: <1>
2024-09-09T09:52:05.2334699Z [ERROR]   ITTestHoodieSanity.testRunHoodieJavaAppOnMultiPartitionKeysCOWTable:72->testRunHoodieJavaApp:181->ITTestBase.executeCommandStringInDocker:263->ITTestBase.executeCommandStringInDocker:274->ITTestBase.executeCommandInDocker:197->ITTestBase.executeCommandInDocker:246 Command ([/var/hoodie/ws/hudi-spark-datasource/hudi-spark/run_hoodie_app.sh, --hive-sync, --table-path, hdfs://namenode/docker_hoodie_multi_partition_key_cow_test_20240909095008633, --hive-url, jdbc:hive2://hiveserver:10000, --table-type, COPY_ON_WRITE, --hive-table, docker_hoodie_multi_partition_key_cow_test_20240909095008633, --use-multi-partition-keys]) expected to succeed. Exit (1) ==> expected: <0> but was: <1>
2024-09-09T09:52:05.2337995Z [ERROR]   ITTestHoodieSanity.testRunHoodieJavaAppOnNonPartitionedCOWTable:85->testRunHoodieJavaApp:241->testRunHoodieJavaApp:181->ITTestBase.executeCommandStringInDocker:263->ITTestBase.executeCommandStringInDocker:274->ITTestBase.executeCommandInDocker:197->ITTestBase.executeCommandInDocker:246 Command ([/var/hoodie/ws/hudi-spark-datasource/hudi-spark/run_hoodie_app.sh, --hive-sync, --table-path, hdfs://namenode/docker_hoodie_non_partition_key_cow_test_20240909095045203, --hive-url, jdbc:hive2://hiveserver:10000, --table-type, COPY_ON_WRITE, --hive-table, docker_hoodie_non_partition_key_cow_test_20240909095045203, --non-partitioned]) expected to succeed. Exit (1) ==> expected: <0> but was: <1>
2024-09-09T09:52:05.2341189Z [ERROR]   ITTestHoodieSanity.testRunHoodieJavaAppOnNonPartitionedMORTable:126->testRunHoodieJavaApp:241->testRunHoodieJavaApp:181->ITTestBase.executeCommandStringInDocker:263->ITTestBase.executeCommandStringInDocker:274->ITTestBase.executeCommandInDocker:197->ITTestBase.executeCommandInDocker:246 Command ([/var/hoodie/ws/hudi-spark-datasource/hudi-spark/run_hoodie_app.sh, --hive-sync, --table-path, hdfs://namenode/docker_hoodie_non_partition_key_mor_test_20240909095121455, --hive-url, jdbc:hive2://hiveserver:10000, --table-type, MERGE_ON_READ, --hive-table, docker_hoodie_non_partition_key_mor_test_20240909095121455, --non-partitioned]) expected to succeed. Exit (1) ==> expected: <0> but was: <1>
2024-09-09T09:52:05.2344587Z [ERROR]   ITTestHoodieSanity.testRunHoodieJavaAppOnSinglePartitionKeyCOWTable:58->testRunHoodieJavaApp:241->testRunHoodieJavaApp:181->ITTestBase.executeCommandStringInDocker:263->ITTestBase.executeCommandStringInDocker:274->ITTestBase.executeCommandInDocker:197->ITTestBase.executeCommandInDocker:246 Command ([/var/hoodie/ws/hudi-spark-datasource/hudi-spark/run_hoodie_app.sh, --hive-sync, --table-path, hdfs://namenode/docker_hoodie_single_partition_key_cow_test_20240909094853224, --hive-url, jdbc:hive2://hiveserver:10000, --table-type, COPY_ON_WRITE, --hive-table, docker_hoodie_single_partition_key_cow_test_20240909094853224]) expected to succeed. Exit (1) ==> expected: <0> but was: <1>

can not be caused by my changes.

@yihua hi! PR is ready for review!

v1 and v2Table has different implementation in spark, they may has different metadata or else?, maybe we need implement hudi's own DescribeTableExec for v2 if official didn't have data we expect.

But existing Spark's v1 implementation of DescribeTable already provides the data we expect. Why can't we use this v1 implementation disregard for schema-evolution setting in write config of current session?


home-assistant: support juicenet component

Motivation for this change
Don't merge before https://github.com/jesserockz/python-juicenet/issues/13 is resolved.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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

[x] Fits CONTRIBUTING.md.

Apart from the licensing issue … tested.

License file was added (https://github.com/jesserockz/python-juicenet/blob/master/LICENSE).

ping @fabaff @mweinelt

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 127997 run on x86_64-linux 1

1 package failed to build and are new build failure:

home-assistant: plain log  |  pretty log

2 packages built:

python38Packages.python-juicenet
python39Packages.python-juicenet

@ofborg test home-assistant

@ofborg build home-assistant

1 package failed to build and are new build failure

Ofborg can't reproduce, so I'm merging this.


Fixed bug when data for HealthChecksResult contains null value

And also documented a workaround for a bug when running ASP.NET 2.2 https://github.com/aspnet/Extensions/issues/639
The workaround is also documented as a note on the bottom of https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/health-checks?view=aspnetcore-2.2

Thanks @nilshedstrom! I'll merge it and try to get a new version out on nuget.

Now available on Nuget as version 0.2.0. Thank you for the contibution!
https://www.nuget.org/packages/Orneholm.ApplicationInsights.HealthChecks/0.2.0


[SILDebugInfo] Small tweaks.

Simplified verification's ancestor checking, improved logging on verification failure, and made parsing treat textual "compiler-generated" locations as auto-generated.

@swift-ci please test


do not load the stubs provided by this plugin but use the psalm native ones

Fixes https://github.com/humanmade/psalm-plugin-wordpress/issues/20
Change your psalm config to:
<plugins>
	<pluginClass class="PsalmWordPress\Plugin">
		<useDefaultStubs value="false" />
	</pluginClass>
</plugins>

to not use the stubs provided by this psalm plugin, but use your own supplied via <stubs> in psalm config.

Thanks!


Consul in HA mode

Just a small question: Is mesos-consul the actual implementation of Hashicorp's Consul or just a bridge into existing consul cluster ? If it is an implementation of Hashicorp's Consul, how can one put multiple consul servers into a cluster node, because mesos-consul.json looks like a  single instance implementation.
Thanks in advance.
Alex

It is a bridge from Mesos into an existing Consul cluster.

Thanks Chris!


Main page feeds

Question
How can get posts and stories that are in main page(home tab)?
and how can get posts and igtvs that  are in explore tab?

get posts

ig.feed.timeline()

stories

ig.feed.reelsTray()

explore tab

ig.feed.topicalExplore()


Remove Gas#recieved Cap?

We need to decide whether we block GAS opcodes or include a separate capability #53.

Removing this @  319f150 , I don't see a sufficient reason to protect this currently.


Ability to specify multiple (from one to six) long time unit variants for different languages

Added the ability to specify multiple forms for time units depending on the specific language.
The ' plural ' macro returns the variant number. The variant definition is taken from gettext (https://www.gnu.org/software/gettext/manual/gettext.html#Plural-forms).
Short unit notations have been moved to the beginning of lists, since lists can now have different lengths.

Looks good.

Thanks!


clearml-agent is not able to execute task

Description:
Hello everyone,
I am currently experiencing an issue with a pipeline I rebuilt from the example provided. The pipeline registers correctly on Cearml, but it does not get executed by the worker.
What I have done:
Rebuilt the pipeline using the example from the GitHub repository.
https://github.com/allegroai/clearml/blob/master/examples/pipeline/pipeline_from_functions.py
Verified that the pipeline is registered on Cearml without errors.
The issue:
The worker does not execute the registered pipeline.
Attachments:
Logs from the worker.
task_5009c074b3b841e28d3e1855dcb5a06c (1).log
Screenshots of the Cearml platform showing the pipeline registration and worker status.

Expected behavior:
The pipeline should execute as expected once registered on Cearml.
Environment:
Docker Compose Deployment for ClearMLSever
https://hub.docker.com/layers/allegroai/clearml/latest/images/sha256-cbc6e5519edecc716112bb03ca291e937c38e6c33630907e557545148cf85d51?context=explore

Agent was testet in docker mode on Windows and WSL
clearml-agent             1.9.2
Logs are identical
example_log_worker.log
Link to discussion in Slack:
https://clearml.slack.com/archives/CTK20V944/p1733059802636009
Any insights or suggestions would be greatly appreciated! Thank you in advance for your help.

afaik pipeline is a task and it executes on your queue, but pipeline creates another tasks and you need another queue to handle these

@Ruhrozz
Thanks for pointing that out! However, I'm still facing an issue: when I use two agents in Docker mode, the pipeline remains stuck with the same behavior. Interestingly, it works correctly when I switch to venv mode.
Task Worker       -> clearml-agent daemon --gpus all --docker  --queue default --detached
Pipeline Worker ->  clearml-agent daemon --docker  --queue controller_queue --detached

Now running new pipeline :

controller_queue is taking the task :

controller gets stuck :

The default query with second worker remains idle:

I hope this provides a clearer description of the problem.


Upgrade RuntimeInformation from 4.0 -> 4.3

This is to prevent a downgrade error when running on a machine with https://github.com/dotnet/sdk/pull/893.
Filed https://github.com/NuGet/Home/issues/4703 to track the large number of warnings/errors this case caused.

tag @dotnet/project-system @BillHiebert


Kubernetes 1.25 Support for Rancher Distros

Internal reference: SURE-4603
Kubernetes 1.25 is out so as backend does it work in https://github.com/rancher/rancher/issues/38701, this ticket is a placeholder (or waits in the wings) in case UI is required.
Connection to one known feature; API Removals: https://github.com/rancher/dashboard/issues/7175 or rancher/dashboard#7175

PodSecurityPolicy is removed and replaced by the built-in PodSecurityAdmission plugin in 1.25, therefore the Dashboard UI needs to be updated to support setting PSA, instead of PSP, for the 1.25 cluster.
For the RKE1 cluster
On the Cluster creation/edit page
Under the Advanced Options section, we would like to have a new field named Pod Security Admission Configuration, and it has two options in the dropdown list: restricted and  unrestricted;
The fields and available values in the dropdown list should match the following YAML:
rancher_kubernetes_engine_config:
  services:
    kube-api:
      pod_security_admission_config: <restrictedPSAConfig, unrestrictedPSAConfig>

on the Project creation/edit page
We would like to have a new section named Pod Security Standard, it has two options in the dropdown list: restricted and  unrestricted.
The default values match the ones set in the Pod Security Admission Configuration section.
Behind the scene, those values are translated to the following labels on the Namespace in the Project:
apiVersion: v1
kind: Namespace
metadata:
  name: my-baseline-namespace
  labels:
    pod-security.kubernetes.io/enforce: baseline
    pod-security.kubernetes.io/enforce-version: v1.25
    pod-security.kubernetes.io/audit: restricted
    pod-security.kubernetes.io/audit-version: v1.25
    pod-security.kubernetes.io/warn: restricted
    pod-security.kubernetes.io/warn-version: v1.25

Putting in backend blocked since it looks like requirements for certain parts are still to be provided before UI dev can begin.

When can we expect k8s 1.25 support in Rancher to be available? Thanks.

Replaced by https://github.com/rancher/dashboard/issues/7119


wip: embassy-net driver for nrf91.

nrf: fix wrong order configuring gpios.
nrf: add try_write to BufferedUarte.
Add embassy-net-nrf91.

TODO:

Make open_raw_socket automatic, there should be no need for the user to call it. In somewhat limited testing, it seemed creating it before attach didn't work, but maybe something else was wrong, or maybe there's some flag we can pass so it survives not being attached. If that's not possible then the driver should be smart enough to open it when attached. Also, we should check it survives deattach+reattach.
Trace should be optional
Trace should perhaps give the user a TraceReader implementing AsyncRead, instead of requiring the user to pass in a Write. This way the user can spawn a background task flushing trace data over uart, ueb, rtt, whatever.
The parser that extracts IP out of CGPADDR could be moved from the example to the net-nrf91 crate. Perhaps it could have a nice function that returns the embassy-net config ready to use.
There's some AT command to get DNS servers, we should add that instead of defaulting to no DNS servers.

I think this is ready for review now. I've decided to put the 'control logic' into a separate module 'context' which allows running that control logic for a given context, which is the most common use case. For users that want more control, they can skip using that.
I've 'reversed' the direction of tracing and made it optional. If enabled, a state containing the 'trace buffer' is handed to the driver, and the driver will push data to the trace buffer, while the app can read from the buffer.

Now that configure is moved out of run, maybe the  +CFUN and %XPDNCFG commands should be moved into run instead?

Now that configure is moved out of run, maybe the +CFUN and %XPDNCFG commands should be moved into run instead?

Your comment triggered me to try configure() multiple times to see the effect, and turns out that didn't work that well. I think we should keep +CFUN inside configure(), because it turns out it needs to run CFUN=0 to change the APN (otherwise it throws an error).
In normal operation I would expect the caller of configure() to only call configure if the configuration has actually changed. (I added a comment to the fn for that though).

Now that configure is moved out of run, maybe the +CFUN and %XPDNCFG commands should be moved into run instead?

Your comment triggered me to try configure() multiple times to see the effect, and turns out that didn't work that well. I think we should keep +CFUN inside configure(), because it turns out it needs to run CFUN=0 to change the APN (otherwise it throws an error).
In normal operation I would expect the caller of configure() to only call configure if the configuration has actually changed. (I added a comment to the fn for that though).

Ahh, i mean because CFUN=1 needs to be called even if you don't use a configuration

Now that configure is moved out of run, maybe the +CFUN and %XPDNCFG commands should be moved into run instead?

Your comment triggered me to try configure() multiple times to see the effect, and turns out that didn't work that well. I think we should keep +CFUN inside configure(), because it turns out it needs to run CFUN=0 to change the APN (otherwise it throws an error).
In normal operation I would expect the caller of configure() to only call configure if the configuration has actually changed. (I added a comment to the fn for that though).

Ahh, i mean because CFUN=1 needs to be called even if you don't use a configuration. So CFUN=0 in configure and then CFUN=1 in run? Also IME, calling CFUN=1 multiple times doesn't hurt, so maybe keep it as is now and just add another CFUN=1 in run so you can use it without calling configure?

Agreed. I've changed the following:

Added an enable() and disable()
Call enable() in run()
Not calling enable() in configure(), but adding it as a note in the comment.

This way, users will be able to 'activate' the configuration after run has started, without resorting to at commands.

Awesome! Just tried it out on my hardware with physical nano SIM without configuration and it works perfectly!

Looks great to me!


How to Load JS file for Support of Modules?

How do I load in a JS file that is not used as part of Angular?

I have tried to use the extraHead property to pass in a JS file. But this does not seem to interact with the components on the page.
            extraHead: [
                '<script src="/js/head.js"></script>',
                '<script src="/js/main.js"></script>'
            ],

What is the best way for non-angular apps to include a JS file for the styleguide sections?

Has anyone had any luck getting non-angular JavaScript exposed to a web component? To me this is Critical for us to adopt this as our StyleGuide generator. I work for General Motors and would love to push this forward as our internal standard.

extraHead applies for a style-guide page itself and may not work with the components because each of them is encapsulated with shadow DOM.
Applying extra JS to a component is not implemented yet. We will create a task in the backlog to do this.

+1 This would be great feature :)

As a workaround solution you can use extraHead and disable shadow DOM encapsulation with disableEncapsulation option.

Thx :) @hannu

:+1:

It does already?

This was resolved be a release later in the year. Closing the issue.


Hirearchical menu seems to have lost its dropdowniness

Hello,
I'm probably missing something, but the menu rendering seems to have lost its "dropdowniness", so to speak.
That is, I see menus rendered as listed items but which are not collapsing. Perhaps there is a bootstrap style missing or that I need to further specify?
Thank you!

The styling is entirely  up to you actually :-) We just render a simple list...

I see what happened. I pulled the Core package in instead of the top level MVC5. It's been a little while working with this, like I said; getting my radar attenuated. :+1:


Subtracting a large integer from a DATE results in a "double free or corruption"

Consider the following statement:
SELECT - 41756167 + '1969-12-11 032657' ::DATE; -- double free or corruption

Unexpectedly, the SELECT results in a double free or corruption error. When using a debug build, ASan shows the following:
=================================================================
==12424==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x6210000140ff at pc 0x56449aa32703 bp 0x7ffff2857090 sp 0x7ffff2857080
WRITE of size 1 at 0x6210000140ff thread T0
    #0 0x56449aa32702 in char* duckdb::StringToIntegerCast::FormatUnsigned<int>(int, char*) /duckdb/src/common/operator/cast_operators.cpp:467
    #1 0x56449aa2efe9 in duckdb::DateToStringCast::Format(char*, int*, unsigned long, bool) (/duckdb/build/debug/duckdb_cli+0x105efe9)
    #2 0x56449aa231ed in duckdb::string_t duckdb::CastFromDate::Operation<int>(int, duckdb::Vector&) /duckdb/src/common/operator/cast_operators.cpp:649
    #3 0x56449a396826 in operator() /duckdb/src/common/vector_operations/vector_cast.cpp:17
    #4 0x56449a3a9431 in Operation<string_cast(duckdb::Vector&, duckdb::Vector&, duckdb::idx_t) [with SRC = int; OP = duckdb::CastFromDate]::<lambda(int)>, bool, int, duckdb::string_t> /duckdb/src/include/duckdb/common/vector_operations/unary_executor.hpp:29
    #5 0x56449a3a4b27 in ExecuteStandard<int, duckdb::string_t, duckdb::UnaryLambdaWrapper, bool, string_cast(duckdb::Vector&, duckdb::Vector&, duckdb::idx_t) [with SRC = int; OP = duckdb::CastFromDate]::<lambda(int)>, true> /duckdb/src/include/duckdb/common/vector_operations/unary_executor.hpp:93
    #6 0x56449a3a11ca in Execute<int, duckdb::string_t, true, string_cast(duckdb::Vector&, duckdb::Vector&, duckdb::idx_t) [with SRC = int; OP = duckdb::CastFromDate]::<lambda(int)> > /duckdb/src/include/duckdb/common/vector_operations/unary_executor.hpp:134
    #7 0x56449a3968d0 in string_cast<int, duckdb::CastFromDate> /duckdb/src/common/vector_operations/vector_cast.cpp:16
    #8 0x56449a38f2ce in date_cast_switch /duckdb/src/common/vector_operations/vector_cast.cpp:144
    #9 0x56449a390432 in duckdb::VectorOperations::Cast(duckdb::Vector&, duckdb::Vector&, duckdb::SQLType, duckdb::SQLType, unsigned long) /duckdb/src/common/vector_operations/vector_cast.cpp:226
    #10 0x56449a349634 in duckdb::Value::CastAs(duckdb::SQLType, duckdb::SQLType) /duckdb/src/common/types/value.cpp:519
    #11 0x56449a30cd66 in sqlite3_column_has_value /duckdb/tools/sqlite3_api_wrapper/sqlite3_api_wrapper.cpp:400
    #12 0x56449a30d85e in sqlite3_column_text /duckdb/tools/sqlite3_api_wrapper/sqlite3_api_wrapper.cpp:433
    #13 0x56449a2e53e1 in exec_prepared_stmt /duckdb/tools/shell/shell.c:9924
    #14 0x56449a2e6fa7 in shell_exec /duckdb/tools/shell/shell.c:10187
    #15 0x56449a30038d in runOneSqlLine /duckdb/tools/shell/shell.c:15047
    #16 0x56449a300cc7 in process_input /duckdb/tools/shell/shell.c:15149
    #17 0x56449a3033d4 in main /duckdb/tools/shell/shell.c:15817
    #18 0x7fd24744bb6a in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x26b6a)
    #19 0x56449a2c07b9 in _start (/duckdb/build/debug/duckdb_cli+0x8f07b9)

This looks related to https://github.com/cwida/duckdb/issues/587. I'm unsure whether it's the same bug. Since the debug build indicates that the error sink is in another line, I've decided to report it.

Indeed the same as #587 :) The issue was related to the DATE -> VARCHAR cast for large negative years. There was a piece of code determining how many bytes the year would take up, but this was accidentally skipped for negative years.


Fix broken README link

solves https://github.com/argosopentech/argos-translate/issues/349

Thanks!


[Feature] Adding distinction: left_shift and right_shift

I am trying to do this in .vindrc:
ennoremap <alt-left_shift> <to_resident>{alt-shift_left><to_edi_normal>}}

-- and am getting this error:

(the reason for this mapping is just to pass an <alt-left_shift> event to another app, see my feature question #177).
Is there distinction between left and right shifs/ctrl/etc.?
Thanks for your help!

Please see this document.
https://pit-ray.github.io/win-vind/cheat_sheet/keywords/#system-keyword
You can use <rshift> and <lshift>.

Indeed!
I see that this is under "keywords" and I was looking under "usage." After I get into getting Jekyll to work I will try sending some PR to make documentation more explicit.
Thanks for all the help!


Window size change unintentionally

Hello, I am a user who is using Windows in South Korea. I'd like to let you know about this because I found a bug while using the calculator.
First of all, I use 4K monitor as a 200% scale main monitor and FHD monitor as a 100%.
Below are the situations in which bugs occur:

Place the Calculator window on the sub-monitor and make sure that the aero-peak (this name may not be correct) occupies all or part of the screen.
Drag the window top bar back to its original size. (A bug occurs at this stage)

In step 2, the Calculator window is twice its original size. This may be caused by using a different scale monitor.
I'm not sure if this bug is from the calculator or from an explorer of Windows 10. But I want you to be aware of this.
I'm not good enough to fix this bug, so I want you to fix it.
Thank you.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Thanks for the detailed report! Do you see this with other apps (e.g., Movies & TV) or just Calculator?

I found that bug only for default Calcultor program...

I've found a specific situation when this bug occurs.
This bug occurs when you attempt to use aero-peak by dragging the mouse over the boundary between sub-monitor and main-monitor.

When the calculator moves from a sub-monitor with a 100% scale to a main-monitor with a 200% scale, you can see the following:

If less than one-half the movement:
The size of the calculator in sub-monitor is normal. However, the size of the calculator in the main-monitor is twice the normal size.
If you have moved more than 1/2
The size of the calculator in the main-monitor returns to normal, and the calculator in the sub-monitor is doubled.

This appears to be because the window of the calculator is drawn based on the number of pixels according to the magnification of the monitor where the window is currently located.

If you move the window of the calculator between two monitors with different scales, the size changes. And this is probably what's causing this bug. When using aero-peak from sub-monitor to the main-monitor direction, if the window in the calculator has been moved from sub-monitor to main-monitor, the window is twice as large when releasing the window.
However, I found that the same thing happened in some other programs. This is probably not just a calculator-only bug, but an aero-peak or other Windows feature.

If you know anyone who can fix this bug, it would be nice if you could pass it on. Thank you for caring about fixing this bug.

However, I found that the same thing happened in some other programs. This is probably not just a calculator-only bug, but an aero-peak or other Windows feature.

Thanks for helping verify this! We'll transfer this bug internally to the right team.


Beaker won't re-open after closing

Operation System: Windows 7 (64-bit)
Beaker version: 0.8.6
Steps to reproduce:

[log out and log back in (to windows)]
Start Beaker (from taskbar, if relevant?)
Wait for new tab page to load, visit a few sites maybe
Close Beaker
Wait a little while, trying opening it again, wait a little longer... forever, in fact
GOTO 1 (logout necessary to start Beaker again)

A very odd (and frustrating) bug.

Correction/Update: the problem is that it isn't exiting properly when closed; several Beaker Browser processes are lingering even after "closed".
Clicking the "Big X" and good ol' Alt+F4 both exhibit this problem.
As soon as all the stray Beaker processes are closed, it becomes possible to open Beaker normally again.
Firefox actually suffers from a similar problem, but it opens a native dialog that says something like:
"The old firefox process is still running and must be closed before opening another",
with a [Close Process] and [Cancel] button.
But I'm guessing doing something like that in electron would be difficult or impossible :/

Hi @jeremyredhead, I just addressed this problem in https://github.com/beakerbrowser/beaker/issues/1359. Hopefully that fix will work. Keep an eye out for the next patch release.

Thanks for filing. Closing as a duplicate.


Facebook SDK for Android

Since networking is getting better in Godot, it is quite useful to have Facebook SDK, specially for Android.
It is a great tool for login and for finding players with ease.
https://developers.facebook.com/docs/android/
Thanks.

It's proprietary, so it should be done as an optional module.

I'm closing this issue since this should be implemented as a module (or using GDNative). Third-party implementations already exist such as xsellier/godotandroid and SShin-NiL/shinnilfb


[BUG] API Endpoint Guard: ReferenceError: request is not defined

Describe the bug
API Endpoint Guard doc request is not defined
To Reproduce
Steps to reproduce the behavior:

just using documentation

Expected behavior
to have exact doc
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
"@sveltejs/adapter-auto": "^3.3.1",
"@sveltejs/kit": "^2.7.3",
"@sveltejs/vite-plugin-svelte": "^4.0.0",
"svelte": "^5.1.4",
"typescript": "^5.6.3",
"vite": "^5.4.10",

The request object is a property of the event object. This was a typo in the documentation, and we will fix it very soon.
// src/routes/api/-guard.ts
import type { Guard } from 'svelte-guard';

export const guard: Guard = async ({ request }) => {
	const header = request.headers.get('Authorization');
	const token = 'xxxxxxxxxxxxxxxx';
	if (!header || header !== `Bearer ${token}`) {
		return false;
	}

	return true;
};

// No reroute specified = 403 Forbidden on failure


Markdown typing support

https://github.com/matrix-org/matrix-ios-kit/issues/124
It uses GHMarkdownParser (https://github.com/OliverLetterer/GHMarkdownParser) which license is MIT.

lgtm


Update hashing to normalize -0.0 on 3.2+

This fixes issues with hash normalization related to SPARK-35207 (Thanks @jlowe for finding the root cause of the issue).
This adds in some new APIs that should make this type of transformation simpler for other code, but because this is late in the 21.10 release I didn't want to update any existing code to use it yet.

build


Fixing typo in grunt config example

I was testing out your post-custom-selectors with grunt. I noticed a typo in grunt example config - just a missing "S" at the end of postcss-custom-selector. If left unnoticed, grunt task will fail though. I am sure all devs will figure it out, so my PR is just a matter of convenience.

Thanks !


Custom Extensions of Array type fail a validation

When you define a custom extension schema for an attribute that is of the array type, it is not possible to create a new User because there is a validation function valid_simple_type that does not consider the simple type of array, only string, boolean, integer and dateTime, so the validation fails thereby causing the Gem to fail the POST request with a bad_request response.
I temporarily changed the Gem code in my local environment to the following, and found that everything worked great after the change:
      def valid_simple_type?(value)
        valid = (type == 'string' && value.is_a?(String)) ||
          (type == 'boolean' && (value.is_a?(TrueClass) || value.is_a?(FalseClass))) ||
          (type == 'integer' && (value.is_a?(Integer))) ||
          (type == 'dateTime' && valid_date_time?(value)) ||
          (type == 'array' && value.is_a?(Array))
        errors.add(self.name, "has the wrong type. It has to be a(n) #{self.type}.") unless valid
        valid
      end

The one new line I added to this function was obviously this:
          (type == 'array' && value.is_a?(Array))

Example
I created a custom extension for a field administrator_group_ids whose datatype is "array" like so:
class UserEnterpriseExtension < Scimitar::Schema::Base
  def initialize(options = {})
    super(
      name: 'ExtendedUser',
      description: 'Enterprise extension for a User',
      id: self.class.id,
      scim_attributes: self.class.scim_attributes
    )
  end

  def self.id
    'urn:ietf:params:scim:schemas:extension:enterprise:2.0:User'
  end

  def self.scim_attributes
    [
      Scimitar::Schema::Attribute.new(name: 'administrator_group_ids', type: 'array')
    ]
  end
end

I also reference this new field in my mixed-in model's scim_attributes_map like so:
    def scim_attributes_map
      {
        id: :id,
        externalId: nil,
        userName: :user_name,
        ...,
        administrator_group_ids: :administrator_group_ids
      }
    end

I have an RSpec example that attempts to set a value in the custom field administrator_group_ids:
  describe 'POST #create' do
    context 'with valid parameters' do
      before do
        request.headers['Content-Type'] = 'application/scim+json'

        attributes = {
          userName: '4',
          name: {
            givenName: 'Given',
            familyName: 'Family'
          },
          meta: { resourceType: 'User' },
          emails: [
            {
              type: 'work',
              value<EMAIL_ADDRESS>            }
          ],
          active: true,
          schemas: %w[urn:ietf:params:scim:schemas:core:2.0:User urn:ietf:params:scim:schemas:extension:enterprise:2.0:User],
          'urn:ietf:params:scim:schemas:extension:enterprise:2.0:User': {
            administrator_group_ids: [7, 8]
          }
        }
        post :create, params: attributes.merge(format: :scim)
      end

      it 'creates a new user' do
        expect(response).to have_http_status(:created)
      end
    end
  end

When I run this example, I get the following error:
"Invalid resource: Administrator group ids has the wrong type. It has to be a(n) array.."
Interestingly, the same custom configuration succeeds and updates the field properly with a PATCH request, so it looks like the configuration I added for the custom schema succeeds with updates but not creates. Example:
context 'when administrator_group_ids is changed' do
      before do
        patch :update, params:
          {
            id: 893,
            operations: [
              {
                op: 'replace',
                path: 'urn:ietf:params:scim:schemas:extension:enterprise:2.0:User:administrator_group_ids',
                value: [5, 6]
              }
            ]
          }
      end

      it 'updates the administrator_group_ids' do
        expect(response).to have_http_status(:ok)
      end
    end

@pond Are you accepting the PR for this bug? @darrylhopkins Will you able to raise a PR to fix the issue or maybe I can take the load?

I would be happy to raise a PR, but I held off because I was trying to figure out which spec covers this scenario. At first glance, it looked pretty daunting because I don't know the internals of this Gem that well.
But, let me give it a shot and I'll give you an update sometime tomorrow if I can make this work. I appreciate the offer to assist :-)

I just realized that the SCIM standard does not support the Array datatype. So obviously this is not a valid request. Therefore I am closing the issue.
https://www.rfc-editor.org/rfc/rfc7643#section-7

type  The attribute's data type.  Valid values are "string",
"boolean", "decimal", "integer", "dateTime", "reference", and
"complex".  When an attribute is of type "complex", there
SHOULD be a corresponding schema attribute "subAttributes"
defined, listing the sub-attributes of the attribute.


FormRequest::withValidator method throwing error

Laravel Version: 9.10.1
PHP Version: 8.1.6
Database Driver & Version:

Description:
I have a FormRequest with the withValidator() method (which I copied exactly as it is in the Laravel documentation), but when executing the class an error is returned.
Class:
<?php

namespace App\Http\Requests\Conversation;

use Illuminate\Foundation\Http\FormRequest;

class StoreConversationRequest extends FormRequest
{
    /**
     * Determine if the user is authorized to make this request.
     *
     * @return bool
     */
    public function authorize()
    {
        return true;
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array<string, mixed>
     */
    public function rules()
    {
        return [
            'type' => 'required|in:text,media',
            'text' => 'required_if:type,text|string',
            'medias' => 'required_if:type,media|array',
            'medias.*' => 'file|max:5120|mimes:jpg,jpeg,png,pdf,mp3,mp4',
            'phone' => 'required|numeric',
            'conversation' => 'required|exists:conversations,id',
        ];
    }

    /**
     * Configure the validator instance.
     *
     * @param  \Illuminate\Validation\Validator  $validator
     * @return void
     */
    public function withValidator($validator)
    {
        $validator->after(function ($validator) {
            if ($this->somethingElseIsInvalid()) {
                $validator->errors()->add('field', 'Something is wrong with this field!');
            }
        });
    }
}

Error:
Method App\Http\Requests\Conversation\StoreConversationRequest::somethingElseIsInvalid does not exist.

I tried replacing $this with $validator but the error persists:
Method Illuminate\Validation\Validator::somethingElseIsInvalid does not exist.

I solved problem with this:
/**
 * Configure the validator instance.
 *
 * @param  \Illuminate\Validation\Validator  $validator
 * @return void
 */
public function withValidator($validator)
{
    $validator->after(function ($validator) {
        if ($validator->errors()->isNotEmpty()) {
            $validator->errors()->add('field', 'Something is wrong with this field!');
        }
    });
}

But I don't understand why a code in the documentation is not working.
Steps To Reproduce:

@lucasvmds the somethingElseIsValid is just a example method, and it does not exist in the Validator or in the form request itself. If you want it to work, you must define this method within your Form Request class.


Update normal-login.ts

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Duplicate attribute class
What is the new behavior?
Removed duplicate attribute
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

Please update your commit message to meet the commit guideline.


Call to a member function get() on null

I have just updated to the latest SDK 4.0.4 Mail is sending fine but api_keys and asm_groups is not working as the documentation states.
$client->api_keys->get();

also happening for
$client->asm_groups->get();

Call to a member function get() on null

@patrickomeara I need some further context to solve this issue. Could you please provide the surrounding code, redacting your credentials?
Also, this library will shortly replaced with this one: https://github.com/sendgrid/sendgrid-php/tree/v3beta
In the new library, that call would be executed as follows: https://github.com/sendgrid/sendgrid-php/blob/v3beta/examples/example.php#L37


travis: Don't cache

Rust build outputs are very large, uploading and downloading them is taking 3 minutes on each build, and for nothing because our cache uploading always times out. We could increase the timeout, but the tradeoff probably weights against caching, at least until Travis can do it outside of the actual build.

The builds turned out slightly slower than usual, so lets just leave things as they are.


Integration test decorators are firing during the import phase of running an unrelated test suite

In Python2, importing modules using the from <PACKAGE> import <MODULE> syntax is causing the __init__.py to fire.  Once init.py starts, it tries to issue from test_integration.geopm_test_integration import * which starts executing random test decorators.  An example:
$ python -m pdb ./test_frequency_hint_usage.py
> /home/bgeltz/git/geopm/test_integration/test_frequency_hint_usage.py(40)<module>()
-> """
(Pdb) break /home/bgeltz/git/geopm/scripts/geopmpy/launcher.py:535
Breakpoint 1 at /home/bgeltz/git/geopm/scripts/geopmpy/launcher.py:535
(Pdb) c
> /home/bgeltz/git/geopm/scripts/geopmpy/launcher.py(535)run()
-> pid = subprocess.Popen(argv_mod, env=self.environ(),
(Pdb) argv_mod
'srun -N 1 -n 1 -J geopm_allocation_test --cpu-freq=Performance -- stat /sys/devices/system/cpu/cpu0/cpufreq/cpuinfo_min_freq'
(Pdb) bt
  /usr/lib64/python2.7/bdb.py(400)run()
-> exec cmd in globals, locals
  <string>(1)<module>()
  /home/bgeltz/git/geopm/test_integration/test_frequency_hint_usage.py(48)<module>()
-> from test_integration import geopm_context
  /home/bgeltz/git/geopm/test_integration/__init__.py(40)<module>()
-> from test_integration.geopm_test_integration import *
  /home/bgeltz/git/geopm/test_integration/geopm_test_integration.py(100)<module>()
-> class TestIntegration(unittest.TestCase):
  /home/bgeltz/git/geopm/test_integration/geopm_test_integration.py(1196)TestIntegration()
-> @util.skip_unless_cpufreq()
  /home/bgeltz/git/geopm/test_integration/util.py(111)skip_unless_cpufreq()
-> geopm_test_launcher.allocation_node_test(test_exec, dev_null, dev_null)
  /home/bgeltz/git/geopm/test_integration/geopm_test_launcher.py(100)allocation_node_test()
-> launcher.run(stdout, stderr)
> /home/bgeltz/git/geopm/scripts/geopmpy/launcher.py(535)run()
-> pid = subprocess.Popen(argv_mod, env=self.environ(),
(Pdb)

(I set a breakpoint set in launcher.py just before the Popen() call.)
The problem above starts with from test_integration import geopm_context.  If this were changed to just import geopm_context the problem goes away because __init__.py does not fire.  I think we can simply change all of the imports in the integration test dir to not use  from <MODULE> ... and this will go away.
The above example illustrates that we start in hint_usage.py.  We execute until we get to the first from <MODULE> import, then __init__.py fires.  Once that happens the main integration_test.py is imported and one of the test's decorators starting on line 1196 fires.
This code path is erroneous and has nothing to do with where we started (i.e. the imports for the hint_usage.py).

While testing my changes for #547, I put a debug print in the run() method of the launcher.py.  I'm printing out argv_mod after "BRG =>".

BRG => srun -N 1 -n 1 --cpu-freq=Performance -- lscpu --hex
srun: Loaded msrsave restore plugin.
srun: Loaded geopm plugin.
BRG => srun -N 1 -n 5 --cpu-bind v,mask_cpu:0x2,0xffc,0x3ff000,0x3ff000000,0xffc00000000 --export=LD_PRELOAD=libgeop
m.so,ALL -I30 -t 10 -J test_count --cpu-freq=Performance -- geopmbench --verbose test_count_app.config
... output continues normally from here...

This was resolved by #1215.

While #1215 did fix my issue with --skip-launch, doing a real run is still showing this problem.
In the run I just tried, I had set OMP_NUM_THREADS=DNC for something else I'm trying.  This shows that the problem persists:
$ OMP_NUM_THREADS=DNC ${WORKSPACE}/geopm/integration/test/test_power_balancer.py --show-details
Traceback (most recent call last):
  File "/usr/workspace/wsb/intelpwr/intelCRADA/geopm/integration/test/test_power_balancer.py", line 46, in <module>
    from integration.test import geopm_context
  File "/usr/workspace/wsb/intelpwr/intelCRADA/geopm/integration/test/__init__.py", line 40, in <module>
    from integration.test.geopm_test_integration import *
  File "/usr/workspace/wsb/intelpwr/intelCRADA/geopm/integration/test/geopm_test_integration.py", line 83, in <module>
    class TestIntegration(unittest.TestCase):
  File "/usr/workspace/wsb/intelpwr/intelCRADA/geopm/integration/test/geopm_test_integration.py", line 1052, in TestIntegration
    @util.skip_unless_cpufreq()
  File "/usr/workspace/wsb/intelpwr/intelCRADA/geopm/integration/test/util.py", line 190, in skip_unless_cpufreq
    geopm_test_launcher.allocation_node_test(test_exec, dev_null, dev_null)
  File "/usr/workspace/wsb/intelpwr/intelCRADA/geopm/integration/test/geopm_test_launcher.py", line 99, in allocation_node_test
    job_name="geopm_allocation_test", quiet=True)
  File "/usr/WS2/intelpwr/intelCRADA/geopm/scripts/geopmpy/launcher.py", line 87, in create
    host_file, reservation=reservation, quiet=quiet)
  File "/usr/WS2/intelpwr/intelCRADA/geopm/scripts/geopmpy/launcher.py", line 898, in __init__
    reservation=reservation, quiet=quiet)
  File "/usr/WS2/intelpwr/intelCRADA/geopm/scripts/geopmpy/launcher.py", line 429, in __init__
    self.cpu_per_rank = int(os.environ.get('OMP_NUM_THREADS', '1'))
ValueError: invalid literal for int() with base 10: 'DNC'

The back trace shows that we jump from test_power_balancer.py to init.py via trying to import geopm_context, then we jump into geopm_test_integration.py then decorators start firing.


New Highlight Command

Utilising a very obscure Roblox object to do so.

player doesnt seem to be defined anywhere (or i might just be dumb)

No it’s me

No it’s me

fix it then dummy

Closing due to no activity to fix issue from pull request author, you can reopen it but if you do also make sure to include a command to un-highlight someone

@LiveFireExercise your lazy


Sparse efficiency warning in tests

When running tests in OpenFermion locally, one of the tests in utils/ gives a sparse efficiency warning. This should be addressed.

Fixed in https://github.com/quantumlib/OpenFermion/pull/13


Youtube Data API v3 not working /search

Exception after using the bot command /search 'search string'
com.google.api.client.googleapis.json.GoogleJsonResponseException: 403 Forbidden
{
  "code" : 403,
  "errors" : [ {
    "domain" : "global",
    "message" : "Method doesn't allow unregistered callers (callers without established identity). Please use API Key or other form of API consumer identity to call this API.",
    "reason" : "forbidden"
  } ],
  "message" : "Method doesn't allow unregistered callers (callers without established identity). Please use API Key or other form of API consumer identity to call this API.",
  "status" : "PERMISSION_DENIED"
}

solved loading vars before launch javafx.
launch(args);
Env.load();

changed to
Env.load();
launch(args);

Thought was multi-thread by default 😕😞


WebView Plugin PWA Integration

Would be really useful if the WebViewController has a way to explicitly download PWAs. A lot of modern API-driven applications that tend to build clients with flutter have a web frontend. and porting to flutter takes a long time. This way apps can be progressively ported using the Strangler pattern among other use cases e.g. tighter integration between JS and Flutter.

With a local web server this can create a pseudo Electron or Cordova/Phonegap, making migration and integration significantly more frictionless.

With a local web server this can create a pseudo Electron or Cordova/Phonegap, making migration and integration significantly more frictionless.

I'm not quite sure what this would really look like embedded in Flutter - would you be able to install a PWA from a Flutter app this way? I'm not so sure all platforms support this, and I'm not sure it's a very good user experience.
Things have changed since I last was looking at PWAs (as in apparently iOS now has at least some support for them?).  But I strongly suspect this won't work very well on iOS in particular.

@dnfield if you try https://github.com/flutter/plugins/pull/1457#issuecomment-480523546, it demonstrates maps without using e.g. https://github.com/johnpryan/flutter_map which are not close to feature parity w.r.t. their JS counterparts.

@kf6gpe , @Hixie - Any update, this would be cool, if PWA can be injected inside WebView to load the page offline.

There are APIs for loading file or asset content, but if that's not sufficient for this use case you'd need to run a local web server, which isn't in scope for the webview plugin (as running a local server is unrelated to a website itself).
Closing as I don't see anything actionable for the webview plugin here.


Flutter create t module do not exist include_flutter.groovy in .android dir since 1.5.4-hotfix

Enviroment: Window10

I terminal 'flutter upgrade' to 1.5.4-hotfix02
I terminal flutter create t module flutetr_module,

But there is no  include_flutter.groovy file in .android fold.
Then I copy the include_flutter.groovy file from  other module  that before I upgrade(1.2.1-release) into the  .android fold, a error happened : Unable to find a matching configuration of project :flutter: None of the consumable configurations have attributes.
How can I add Flutter module into old project since edition upgrade to 1.5.4-hotfix02.
Thanks.

@YellowDoing
I don't think this issue occurs in 1.5.4-hotfix02

The include_flutter.groovy file is generated based on 'flutter1.5.4/packages/flutter_tools/templates/module/android/library/include_flutter.groovy.copy.tmpl'
You can check if that file exists in your flutter repo.