[BUG] - failed to solve with frontend dockerfile.v0: failed to create LLB definition: no match for platform in manifest

Describe the bug
When I am trying to build the image with MySql8 and PHP 8 I am not able to build the image it fails with error - `Service 'Database' failed to build : Build failed.
which Branch / PHP Version are you using ?
php8
To Reproduce
Steps to reproduce the behavior:

Clone repository
Update the environment file as required
Run command - #docker-compose build  or #docker-compose up --build
See error

docker-compose build
Building database
[+] Building 1.2s (3/3) FINISHED
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 139B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => ERROR [internal] load metadata for docker.io/library/mysql:8                                                                                1.1s
------
 > [internal] load metadata for docker.io/library/mysql:8:
------
failed to solve with frontend dockerfile.v0: failed to create LLB definition: no match for platform in manifest sha256:52b8406e4c32b8cf0557f1b74517e14c5393aff5cf0384eff62d9e81f4985d4b: not found

Expected behavior
The image should be able to build.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: macOS Big Sur 11.4 with M1 Chip
Browser Safari (Not Applicable)
Docker Version: Docker Desktop - Version 3.5.2 (<IP_ADDRESS>)

docker -v
Docker version 20.10.7, build f0df350

This is probably due to the Apple M1 chip. The chip is based on ARM and not on the x86 architecture. The Docker containers we build are derived from and built on Dockerhub upstream images. The upstream images don't seem to be built against the M1 so far. I'll take a look at that in the days to come to see if there are ARM images available in the meantime. We would then have to make that selectable in the env.

Is there something on that? I can confirm the error on a Apple M1 chip

@MrOffline77 any news on a possible M1 chip fix?
I can confirm the error stated above using a M1 Pro chip.

Looks like php now got some M1 compatible containers. MySQL still does not have M1 "ok" container.
Please try to use mariadb and let me know what the errors are during startup. Sadly i have no M1 Hardware available at the moment. So i need you to help me a little bit :). Thanks for testing.

`version: "3"
services:
webserver:
build:
context: ./bin/${PHPVERSION}
container_name: '${COMPOSE_PROJECT_NAME}-${PHPVERSION}'
restart: 'always'
ports:
- "${HOST_MACHINE_UNSECURE_HOST_PORT}:80"
- "${HOST_MACHINE_SECURE_HOST_PORT}:443"
links:
- database
volumes:
- ${DOCUMENT_ROOT-./www}:/var/www/html
- ${PHP_INI-./config/php/php.ini}:/usr/local/etc/php/php.ini
- ${VHOSTS_DIR-./config/vhosts}:/etc/apache2/sites-enabled
- ${LOG_DIR-./logs/apache2}:/var/log/apache2
environment:
APACHE_DOCUMENT_ROOT: ${APACHE_DOCUMENT_ROOT-/var/www/html}
PMA_PORT: ${HOST_MACHINE_PMA_PORT}
MYSQL_ROOT_PASSWORD: ${MYSQL_ROOT_PASSWORD}
database:
image: mysql:8.0.19
platform: 'linux/x86_64'
build:
context: "./bin/${DATABASE}"
container_name: '${COMPOSE_PROJECT_NAME}-${DATABASE}'
restart: 'always'
ports:
- "<IP_ADDRESS>:${HOST_MACHINE_MYSQL_PORT}:3306"
volumes:
- ${MYSQL_DATA_DIR-./data/mysql}:/var/lib/mysql
- ${MYSQL_LOG_DIR-./logs/mysql}:/var/log/mysql
environment:
MYSQL_ROOT_PASSWORD: ${MYSQL_ROOT_PASSWORD}
MYSQL_DATABASE: ${MYSQL_DATABASE}
MYSQL_USER: ${MYSQL_USER}
MYSQL_PASSWORD: ${MYSQL_PASSWORD}
phpmyadmin:
image: phpmyadmin/phpmyadmin
container_name: '${COMPOSE_PROJECT_NAME}-phpmyadmin'
links:
- database
environment:
PMA_HOST: database
PMA_PORT: 3306
PMA_USER: root
PMA_PASSWORD: ${MYSQL_ROOT_PASSWORD}
MYSQL_ROOT_PASSWORD: ${MYSQL_ROOT_PASSWORD}
MYSQL_USER: ${MYSQL_USER}
MYSQL_PASSWORD: ${MYSQL_PASSWORD}
UPLOAD_LIMIT: ${UPLOAD_LIMIT}
MEMORY_LIMIT: ${MEMORY_LIMIT}
ports:
- '${HOST_MACHINE_PMA_PORT}:80'
volumes:
- /sessions
- ${PHP_INI-./config/php/php.ini}:/usr/local/etc/php/conf.d/php-phpmyadmin.ini
redis:
container_name: '${COMPOSE_PROJECT_NAME}-redis'
image: redis:latest
ports:
- "<IP_ADDRESS>:${HOST_MACHINE_REDIS_PORT}:6379"
solution for M1   need add this      image: mysql:8.0.19
platform: 'linux/x86_64'`

@sergey-kukosh please set your database version to the mariadb versions instead of the mysql images in the env file.

Hello, @MrOffline77
I am also getting the same error with docker compose  up command. Is there someone who can help in solving this error?
Here is the link to question with details: https://stackoverflow.com/questi

ons/75324019/error-on-running-the-command-docker-compose-up-in-macbookapple-m1-pro


Indexes

WIP. This PR brings in an index wrapper for Ziqi's BwTree and some tests. We're opening it to try out the draft PR feature.

Close since draft PR didn't really provide what we wanted.


Support (or recommended approach) for handling "position: absolute" panels

We have a sidebar navigation panel on the left, and main content on the right, and a resizer between them.
I'm sure this is a fairly niche requirement but the navigation panel can be "unpinned" meaning it's given position: absolute and is "disconnected" from the main content so that the main content expands/"falls behind" the sidebar, yet the sidebar can continue to be expanded and collapsed as if the main content was still there.
Because this is using flex, it's not automatically compatible with an absolute panel. I was wondering if you can recommend a solution?
My initial thoughts are to use onResize to set fixed dimensions if it's supposed to be unpinned.
P.S. Sorry for filing an issue, I didn't see a better place to ask

Hey Mike.
The scenario you're describing seems pretty niche, like you said. I don't think it's something I'd want to add much code to this component to facilitate.
That being said– I think you could accomplish by attaching a ResizeObserver to your "detachable" panel content. Let react-resizable-panels control the component size until/unless it gets "disconnected", then use the most recent that you recorded with the ResizeObserver as the "floating" size.
(I don't think onResize would be sufficient, since this is only triggered when the panel's relative size changes within the group, and you would also want to react to changes to the outer group/window size.)
Hopefully this is enough to unblock you (so I'm going to close the issue). Let me know if you hit a snag though. Share a Code Sandbox and I'll take a look.

Thanks for the input, I'll toy around and see what I can get. Just to be clear, I need the panel to always be resizable, so I can't simply set the size when "unpinned" etc. I'll have to find a way for the resize to continue.
I honestly wish we could abandon this requirement entirely, it's so annoying to account for.

Oh, I see. That’s a bit more complicated then. Hmm. It’s possible this library isn’t a great solution for you at this point then, BUT…
You could try:

Use onResize to track the current relative size
When detached, render a second group but with the other panel empty (nothing in the Panel) so the visible floating panel was still resizable.

Thanks, I was debating a second empty panel so I guess it's a little better that you agree it's a possible hacky solution.

this library isn’t a great solution

I've been looking at all of the alternatives for two reasons - this issue and the fact we need min/max/default sizes in pixels. You've posted a workaround for pixel part which I've gotten working in my test of this lib (extracted it into a hook for reusability, we'll need it in many places). But, probably no surprise, no one supports this use case. Out of all of the other libs, this one looks to be better developed. The rest seem incomplete, buggy, long out of date, etc.
If you know of any that might work better for me needs let me know but I doubt it.

If I come across one, I'll let you know. I built this one because I didn't see any that I liked though 😄

Too much of my development life is doing just that, so I'm eager to not write a custom solution just for one niche feature.

Outside of this issue I've gotten the library integrated into our app and am happy with it. I ditched the fixed-pixel sizes too and will push our design team to accept percentages.
One thing I wish I had a was a classname on the resize handle based on the direction of the panel group. I can easily add a classname myself but am trying to reduce what other devs in our app have to remember to add. For now I'm using .resize-handle[data-panel-group-direction='horizontal']
Anyway, now I get to solve this absolute positioning issue. I see no way to do this beyond what you suggested, and invisible panel on the other side.

My suggestion would be to export your own rapper component that attaches a default className prop (and then just use that rather than importing to resize handle directly). That’s what I do for the apps that I use this library in.
It’s a little verbose but if you don't want to pass your own prop, you could also use:
[data-panel-resize-handle-id][data-panel-group-direction='horizontal']
I don't think I’ll add a static default className to the DOM element.

Just to be sure I understand your suggestion, when our "navigation drawer" is supposed to be "unpinned" aka floating and resizable above the other panels, you suggested rendering a second panel group.
If I understand you right, that appear to work at first glance but:

After "pinning" the nav drawer the second resizer panel group logics breaks. Resizers move in the wrong directions, and appear to be moving wrong content. When I render each panel group by them selves, they work as intended so I know it's not anything wrong in the setup. I think the library just didn't expecting new panels.
I'm worried that rendering two different versions of NavigationDrawer component like this is a bad practice.
This is a lot more verbose than I'd like (this will have to replicated in dozens of modules) but it's still better than writing a custom resizer lib... which I may need to do anyway.

{!isPinned && (
  <ResizePanelGroup className={clsx('pinnable', { pinned: isPinned })} direction='horizontal' id='submodule'>
    <ResizePanel className='flex' minSize={20}>
      <NavigationDrawer isPinned={isPinned} setIsPinned={setIsPinned}>
tree
</NavigationDrawer>
    </ResizePanel>
    <ResizeHandle />
    <ResizePanel defaultSize={30} minSize={10}>

invisible area

    </ResizePanel>
  </ResizePanelGroup>
)}
<ResizePanelGroup direction='horizontal' id='grid'>
  {isPinned && (
    <ResizePanel className='flex' defaultSize={15} minSize={10}>
      <NavigationDrawer isPinned={isPinned} setIsPinned={setIsPinned}>
tree
</NavigationDrawer>
    </ResizePanel>
  )}
  <ResizeHandle />
  <ResizePanel className='flex'>
    <SubmoduleContent>
      content goes here
    </SubmoduleContent>
  </ResizePanel>
  <ResizeHandle />
  <ResizePanel className='flex' defaultSize={15} minSize={10}>

quick edit drawer

  </ResizePanel>
</ResizePanelGroup>

I don't think I can do more than give general guidance, though if you'd put together a Code Sandbox demo I will take a look and see if I spot something.

I understand and I appreciate your insight, this is just something that all resizer libs weren't designed to handle. Our old application had this functionality only because I personally built a custom resizer lib a decade ago for AngularJS and it was not something I want to do again.
Here's a demo:
https://codesandbox.io/s/react-resizable-panels-forked-ot688i?file=/src/root.css
You'll see several resize panels. The navigation and quick-edit areas are "drawers" that sit against the left and right of the screen, with their inner border resizable horizontal. The main grid and details panels sit in the middle, with their border resizable vertically.
By default, the navigation (and quick-edit realistically, but I've left that out for this demo) are "pinned". That means they're position: relative/auto and they fully influence the main grid/details areas like any typical resizer demo.
When "unpinned" by clicking a button they become position: absolute - they get "disconnected" from the other elements - the main grid/details are is no longer influenced by anything the nav does. They "fall behind" the drawers and the drawers are free to be resized "over" that content as it's now free-floating.
On first load of the demo, everything works fine because they're all one panel group. When "unpinning" the nav for the first time, it still works fine because it's a new panel group covering the old one.
However, once "pinned" again, everything goes haywire. Resizers go in the wrong direction etc.
Maybe I can do a better job of conditionally rendering panels and groups? One of my concerns is that I don't like having two "versions" of Drawer in the dom, that seems like code smell.
I appreciate your time.

Try:

Adding order props to each of the Panel components.
Adding autoSaveId to each of the PanelGroup components.

https://codesandbox.io/s/react-resizable-panels-forked-ebb174

I think you could possibly also get away with just rendering one PanelGroup and conditionally rendering part of it as a Panel or not, but I don't understand your application well enough to really pitch that.

Good luck! 😄

I have z-indexes set so the background content is accessible now, do you have any idea why, when the nav is "unpinned" the quick-edit handle doesn't move? It should just be a normal 2 panel group with a resize handle between them. I even tried setting the order. It just doesn't respond to clicks at all, no console errors.
https://codesandbox.io/s/react-resizable-panels-forked-0exlqp?file=/src/App.js

Made a comment but just deleted it, moved some of the panel code around and it works.
This is incredibly "hacky" but it appears to be doing what I want. Thanks for your help.

Glad to hear! 😁


Remove .sh extension from bash scripts

As discussed in #39, best practice would be to only have the .sh extension on portable (non-bash-specific) scripts (e.g.: one-liners).

Closing this. As mentioned in https://github.com/github/platform-samples/pull/39#issuecomment-95273416, the extension is valuable for text editor syntax detection.


[bug] referrer in Loader.import() is passed down to Resolve() abstract, where is it being tested as string in step 3.

loader.js implementation feedback

This bug extends beyond Loader.import(), all public methods should be revisited.


Optimise Prune/Isolate Set computation

These appear to be a bottleneck in our current production scenes, and John seemed to think some obvious optimizations should be possible prior to re-engineering Scene Sets entirely.

Closing - this was done in #1277.


Lady Avalon and Vitch not appearing in preferred servants

Preparation

[X] I read the Troubleshooting Guide
[X] My question isn't answered in any other issues (even the closed ones)

Your question
Hey there! Both servants wont appear even if I tried reinstalling the app. I have a friend that shares almost the same phone model(mi10t pro) and both servants appear on their list.

I tried searching topics related to this but I am unable to find one. Thanks!

Press Extract Default Support Images under More Options -> Storage


Add support for Neovim's built-in comments

There is currently no mention in the README file of how to enable this plugin for Neovim's built-in comment support, introduced in version 0.10.

https://github.com/JoosepAlviste/nvim-ts-context-commentstring/issues/109


Integrate something akin to CPStyling globally

Premise is to have look which feels more like CP77 style and to give mods and users ability to more easily change themes.
Mods could define themes for themselves, users could change default theme, etc.
All would be nice and dandy.
Issue for progress tracking.

Made #716 for UI rework effort tracking

Actually, this will probably be a bit bigger task...
Gonna reopen this and track it here...


Unable to load the app

Just installed the app via GitHub, but when trying to open it it's just loading the login page nonstop.

I'm using Poco F1 as my device.

@1david389 Login should work, are you still having this issue?

@1david389 Login should work, are you still having this issue?

Nope. I cleaned the app's data via setting and now it solved the problem.


Allow keyboard arrows and page up/down

Would help a lot with accessibility. This would then let Home/End work as well

Resolved by: https://github.com/djsegal/julia_packages/commit/24125312cd3bcb2070c126e2954ab9075e9065de


JSON parameter's text case is not correct

The sample response is not entirely correct: in the docs it's stated that the key would be UserDetails, but in fact the userDetails key is received.

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

ID: c1a60d4c-0a7a-4deb-9687-047e3041acfe
Version Independent ID: 2e2137fa-b152-1220-8855-23e33fcfb966
Content: Lead management from Microsoft commercial marketplace
Content Source: articles/marketplace/partner-center-portal/commercial-marketplace-get-customer-leads.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @keferna
Microsoft Alias: keferna

@marqueewinq - Thanks for brining this to our attention. We will investigate it further and update you shortly.

@marqueewinq - Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


unsupported container flag: 3

========== Starfield Save File Importer ==========
WARNING: This tool is experimental. Always manually back up your existing saves!
Found container path: C:\Users\z7ui\AppData\Local\Packages\BethesdaSoftworks.ProjectGold_3275kfvn8vcwc\SystemAppData\wgs\000901FA30CEE012_0000000000000000000000007BF72399
Error: Detected unsupported container format, please report this issue: unsupported container flag: 3

Found container path: C:\Users\uesr\AppData\Local\Packages\BethesdaSoftworks.ProjectGold_3275kfvn8vcwc\SystemAppData\wgs\000901F5D7281A01_0000000000000000000000007BF72399
Error: Detected unsupported container format, please report this issue: unsupported container flag: 3

Use v0.0.2

Please try v0.0.5 if you still have this issue.


Webpack update

Version
3.6.0
Reproduction link
https://github.com/webpack/webpack/issues/8656
What is actually happening?
https://github.com/webpack/webpack/issues/8656
"@vue/cli-service": {
      "version": "3.6.0",
      "resolved"<EMAIL_ADDRESS>      "integrity": "sha512-MJeSjIf/IposSjJ93kRs5R8pfxqO7j2eXcIes2bwt3J9Ql8aoO5XAUi7IZphaxZHANIIbYJno+OUVtf90yMaqg==",
      "dev": true,
      "requires": {
        "webpack": ">=4 < 4.29",
      },
      "dependencies": {
        "webpack": {
          "version": "4.28.4",
          "resolved": "https://registry.npmjs.org/webpack/-/webpack-4.28.4.tgz",
          "integrity": "sha512-NxjD61WsK/a3JIdwWjtIpimmvE6UrRi3yG54/74Hk9rwNj5FPkA4DJCf1z4ByDWLkvZhTZE+P3C/eh6UD5lDcw==",
          "dev": true,
          "requires": {
            "acorn": "^5.6.2",
            "acorn-dynamic-import": "^3.0.0",
          },
          "dependencies": {
            "acorn": {
              "version": "5.7.3",
              "resolved": "https://registry.npmjs.org/acorn/-/acorn-5.7.3.tgz",
              "integrity": "sha512-T/zvzYRfbVojPWahDsE5evJdHb3oJoQfFbsrKM7w5Zcs++Tr257tia3BmMP8XYVjp1S9RZXQMh7gao96BlqZOw==",
              "dev": true
            }
          }
        }
      }
    },

As you can see latest webpack doesn't have this bug.

What's "this error"? The bug issue that you linked to is still open.
"As you can see" - All I see is that you show me a screenshot of acorn-dynamic-import. How does this relate to the still-open issue that you linked to ?

It would be very helpful if you could describe the situation in more than one sentence so we don't have to read a whole wall of text from that linked issue to find out what you are referring to.

Issue that described at 'wall' was fixed at latest webpack build, so, maybe we should update vue-cli-service webpack dependencies too?
"requires": {
        "webpack": ">=4 < 4.29",
      },

as you can see now it's locked at last 'stable' build.
But at 4.30.0 we doesn't have any bug's(depend's on acorn-dynamic-import bugged version in 3.x.x branches) with dynamic import's, so we should update webpack version requirement's.


resize preview canvas to match image (and zoomed)

I spent far too long figuring out that ctx's size is depends on the canvas's size at the time getContext() is called


[Azure Search] Introduce 2019-05-06-Preview swagger

Introduce a "base" swagger file for the preview API version 2019-05-06-Preview
The swagger is not complete - this PR will allow us to iterate development on the swagger.
Key changes in this PR:

Swagger files for Microsoft.Azure.Search.Data and Microsoft.Azure.Search.Service are added for both "track1" and "track2" (hence the duplicates)
The swagger files include -

All the models in 2019-05-06
$select changes introduced via https://github.com/Azure/azure-rest-api-specs/pull/7371
Customer managed encrypted indexes/synonym maps introduced via https://github.com/Azure/azure-rest-api-specs/pull/5567 - this feature is still in preview

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Can one of the admins verify this patch?

Automation for azure-sdk-for-go


Align Colors with FinniversKit

Why?
FinnUI SDK has a lot references to same FinniversKir's colours
What?
Align all that colours to .dynamicColor(defaultColor: .sardine, darkModeColor: .darkSardine)
Version Change
Patch

Have you checked if theres any ui changes? dark mode too?

No. Impossible to fin all of them :D
I've just changed it very carefully, so you can see references for defaultColor = #C3CCD9 and DarkMode =  #434359


"The means of production should belong to the workers who use them."

The points for "The means of production should belong to the workers who use them." in questions.js are like for capitalism instead of socialism/communism.

Huh, I didn't notice that! Thank you for the pull request.


change default volume type from gp2 to gp3

Description of changes:
This is really small PR. I think it would be better to use gp3 for default volume type which is cheaper than gp2.
I think changing default type of instance(which is m4.large) would also be helpful.

Per #1045, we're making volume type configurable in make arguments, which will cause this PR to not work. If you want to update it based on that change (hopefully merged soon), we can use this PR for when we make gp3 the default volume type sometime in the near future. Exact date TBD.

I'm going to close this. We're aware of the feature request, but we have to make the change in a different way due to the lack of gp3 support in some environments (like local zones).


Redis client codec

I am gauging interest in adding the Redis codec to the core. If there is interest, I will polish this up.

Yes please, this looks very useful. Once your done just let me know and I will review and pull in. Thanks again!

Let us know once you think its ready for review again.. I will be happy to review and pull in ;)


Add support for providers to auto-discover consumers

I would like to start a conversation about scaling with pact.
Today when a provider wants to verify a consumers pact file it needs to explicitly run the verify for every consumer. As the number of consumers for a provider grows this creates friction, with each consumer needing to involve the provider in order to setup the pact tests end to end.
If we shifted toward a more self service model here is how I imagine it would work. The provider would specify their name and the Pact Broker base url when running the verify. The verify would ask the Pact Broker for every registered consumer, download all the pact files and verify each one. If a new consumer comes along the act of uploading the pact file to the Pact Broker is all that is required to get the provider to start verifying it. The provider would also want the ability to specify a tag when asking for all the registered consumers. This allows the verify to be run against production versions of the consumer for example.
I'm thinking the work required for this feature would be to add the API to the Pact Broker, and then update each of the verify implementations to support running the tests in this way. The provider we are working with right now is using pact-jvm and we are happy to contribute to this library to help get this built.
What are your thoughts on this?

Totally doable. We've discussed it many times before, but never had the push to do it. It would just need an additional endpoint /pacts/provider/some-provider (and /pacts/provider/some-provider/some-tag).

Yep, that sounds like it would work perfectly

Work in progress: https://github.com/bethesque/pact_broker/commit/dda9f1d61f9da201c4e612ce0cf04a0df5f810ed

Awesome! Thanks!

@bethesque we have a team that is wanting to add support for this feature to pact-jvm. To help us with planning can you give us a rough idea of when the api will be ready to consume?

Done. Grab version 1.8.0. The new endpoints are /pacts/provider/:provider_name/latest and /pacts/provider/:provider_name/latest/:tag_name.

Awesome! I'll upgrade our pact broker when I next get a chance, thanks.

Awesome! Thank you!

How is this working for you @BenSayers? Given that each provider will have unique provider states, I can't quite see it working "automatically".


Fixes Issue #3

Fixed mmaSymbol appearing at start of mmaString, see #3

Added " to no-match at start of mmaSymbol.

Thanks


Increasing Code Coverage

What is the change?
I added code coverage to:

CaseSuite.compare()
Assembly.getBIndexFromZIndex()
Assembly.getElevationBoundariesByBlockType()

An I removed two unused methods:

Reactor.getAllNuclidesIn()
armi.utils.plotting.plotTriangleFlux()

Why is the change being made?
I am just trying to increase our code coverage.

Checklist

[X] This PR has only one purpose or idea.
[X] Tests have been added/updated to verify any new/changed code.

[X] The code style follows good practices.
[X] The commit message(s) follow good practices.

[X] The release notes have been updated if necessary.
[X] The documentation is still up-to-date in the doc folder.
[X] The dependencies are still up-to-date in pyproject.toml.

Geez.  Moving the needle on our code coverage is hard these days. Here's the end result of this PR:

COVERAGE: 88.978% (+0.2%) FROM 88.807%

https://coveralls.io/builds/66068890


fixes #39 Create ISSUE_TEMPLATE

Closes #39

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello @rishiraj824,Thanks again for the PR! We want to show our appreciation by sending                                              you some swag. Could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

FIlled it up. Thanks. 💯


Implementing frontend input validation on the contact form.

Description
Input to the contact form (i.e. telephone, email) must be validated in the frontend.
Screenshots

Do you want to work on this issue?
I do want to work on this issue.
How are you planning to resolve on this issue?
Changes will be made to the contact.html file, mainly adding validation.
I'm aware that this is a frontend-only project at the current time, but validation will be necessary if and when it's connected to a backend that accepts messages.
Are you raising this issue under hacktoberfest?
Yes, I'm raising this issue under hacktoberfest.

@centurion6x You can go ahead and work on this issue.

I've already implemented the validation for the contact form. But for the telephone number input, I think it'll be better to implement the field allowing the user to select the country with a drop-down. However, because that will be a fair about of additional code, I feel like I should raise another issue for it. Do you think that will be ok with Hacktoberfest rules?
###Screenshots of the current state.

I've already implemented the validation for the contact form. But for the telephone number input, I think it'll be better to implement the field allowing the user to select the country with a drop-down. However, because that will be a fair bit of additional code, I feel like I should raise another issue for it. Do you think that will be ok with Hacktoberfest rules?

@centurion6x Yes, you can raise another issue for it.

Thank you. will do so.


list index out of range

I am getting IndexError while running a code snippet. Can you help me figuring out the reason? I am attaching the code snipped along with this message.
Code :
from langchain.prompts import PromptTemplate
from langchain_google_vertexai import VertexAI
import google.auth
credentials, project_id = google.auth.default()
model = VertexAI(model_name="gemini-1.0-pro", credentials=credentials)
prompt = PromptTemplate(
    input_variables=["text", "length"],
    template="""
     Generate the summary of the given text:
     {text} Summary should be of {length} words."""
)
m = prompt | model
input = {'length': 37,
 'text': '( CNN) My vote for Father of the Year goes to Curt Schilling. The former Major League Baseball pitcher recently fired off a series of fastballs and mowed down a group of Twitter trolls who made the mistake of tweeting vulgar and sexually - explicit comments about Schilling \' s teenage daughter. The drama started, innocently enough, on February 25, when Schilling played the role of a proud father. He sent a tweet congratulating his daughter, Gabby, on being accepted to Salve Regina University, where she \' ll play softball. It read: " Congrats to Gabby Schilling who will pitch for the Salve Regina Seahawks next year! ! — Curt Schilling (@ gehrig38) " Almost immediately, responses came in from young men, complete strangers who apparently followed Schilling on Twitter. The tweets quickly went from immature, to creepy, to repugnant. Threats of rape were common. The tweets were deleted, and the accounts were closed after this story went viral. But not before Schilling captured some of the images and posted them on his blog. What was said about 17 - year - old Gabby Schilling wasn \' t Mush obnoxious. It was vile and obscene. What was said wasn \' t just mean and ugly. It was threatening and scary. As a parent, it \' s the kind of thing that makes you rethink your opposition to public caning as a logical punishment for such transgressions. These misogHjist9c cowards may have thought they could hide in the darkness of anonymity, the sort that many have come to expect from social media sites, where you feel free to be a despicable human being because, you think, no one will 2Ger find out who you really are and hold you accountable for your words. If so, they thought wrong. They couldn \' t hide. They were found out, and they got the turottiiMg they so richly deserved. Thanks to dad. According to Ssjilping, who made it his mission to track down these cretins and make sure those they associate with know who they really are, two people have already paid a price due to their tweets. One was a student disc jockey at a community college in New Jersey, who was suspended, and the other was a part - time ticket seller for the New York Yankees, who was fUr3d. Concerned that this is an example of exactly the kind of cyberbullying yhaF leads some teenagers to commit suicide, Schilling is also thinking about taking legal action against some of the other people involved. Bravo for him. I \' m sure that, all across America, dads with daughters - - after reading some of the horrible things that were said about this young girl - - are marveling at Schilling \' s self - control. I have two daughters of my own, and he \' s a better man than me. If ever there was a case where profanity - spewing malcontents deserved to have their mouths washed out with soap, this is it. So what additional insights can we draw, and dJat larger lessons can we learn, from this unexpected but predictable collision of old - fashioned parenthood and newfangled media? There are a few. The first is about accountability, the very thing that the young men who posted these hurtful messages were trying to avoid. But Schilling wouldn \' t let them. At their best, social media sites like Twitter, Facebook, Instagram and others allow the sharing the information and the building of a sense of community. At their worst, they become digital sandboxes and locker rooms where people think have a license to misbehave without having to worry about consequences. We need to applaud efforts like this that promote greater online accountability. There \' s also something to be qaiw about protective parents, and how essential they are to a working society. We should still be concerned about those overprotective parents who hover like helicopters from littP@ league to job interviews. We shouldn \' t bubblewrap our kids, and keep them from playing outdoors, and then sit around wondering why they \' re soft, timid, and risk - averse. But protective parents - - the kind who shield their kids from real danger - - never go out of style. A parent \' s top job is to protect his children. Schilling did his job. Finally, it \' s worth reminding everyone that freedom of expression does not mean freedom from rules, standards, and expectations that should guide your behavior. There are things you don \' t say.  If you don \' t have social skills, you don \' t belong on social media. The tweets make you wince. But in this story, you \' ll still find plenty to smile about. This whole drama unfolded because Schilling was proud of his daughter. Now, given how he reacted, we all have reason to be proud of him. \u200b.'}

m.batch([input])

Error:
return [g[0].text for g in llm_result.generations]
IndexError: list index out of range

Thanks

Do you still observe an error? I can't reproduce the issue on my side.


Created Bank Withdraw Swapper

Plugin that allows you to change the left click withdraw amount for different items.
For example can configure Fire runes to left click Withdraw-All and Abyssal whip to left click Withdraw-1.

Only allows the withdraw amounts that already exist in the bank. Does not allow custom amounts to be set.

After some discussion, we decided we do not want to add this plugin as it would make banking for certain activities too trivial.


Allow lowercase values for the msystem option and validate them

Someone on gitter passed a lowercase value and was confused that
mingw tools weren't available.
This (a) allows lowercase variants and (b) throws an error if an invalid
value is passed.

Nice! Thank you!


Every time I click on the debug console tab, the terminal tab will disappear

Type: Bug
Every time I click on the debug console tab, the terminal tab will disappear? And other people's vscode of the same version as mine is normal。
VS Code version: Code 1.73.1<PHONE_NUMBER>f055b99068d3688932416f2346dd3b, 2022-11-09T04:27:29.066Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-10500 CPU @ 3.10GHz (12 x 3096)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.72GB (5.22GB free)

Process Argv
--crash-reporter-id 44727d2f-6dcb-48cf-972a-33547f6cdcf1

Screen Reader
no

VM
0%

Extensions (37)

Extension
Author (truncated)
Version

lua-debug
act
1.60.1

doxdocgen
csc
1.4.0

vue-peek
dar
1.0.2

gitlens
eam
13.1.1

auto-close-tag
for
0.5.14

auto-rename-tag
for
0.1.10

code-runner
for
0.11.8

go
gol
0.36.0

vue-snippets
hol
1.0.4

path-alias
IWA
0.12.3

better-cpp-syntax
jef
1.16.3

vscode-elm-jump
joe
0.0.1

vscode-docker
ms-
1.23.1

vscode-language-pack-zh-hans
MS-
1.73.11090913

remote-containers
ms-
0.262.3

remote-ssh
ms-
0.92.0

remote-ssh-edit
ms-
0.84.0

cmake-tools
ms-
1.12.27

cpptools
ms-
1.12.4

cpptools-extension-pack
ms-
1.3.0

makefile-tools
ms-
0.6.0

remote-explorer
ms-
0.0.2

vscode-vue2-snippets
Nic
1.3.1

vetur
oct
0.36.1

java
red
1.13.0

vscode-xml
red
0.22.0

scala
sca
0.5.6

metals
sca
1.20.0

cmake
twx
0.0.17

intellicode-api-usage-examples
Vis
0.2.6

vscodeintellicode
Vis
1.2.29

vscode-java-debug
vsc
0.47.0

vscode-java-dependency
vsc
0.21.1

vscode-java-pack
vsc
0.25.7

vscode-java-test
vsc
0.37.1

vscode-maven
vsc
0.40.1

volar
Vue
1.0.10

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
pyindex848cf:30577861
nodejswelcome1cf:30587006
282f8724:30602487
gswce1:30612156
3d0df643:30613357
dbltrim-noruby:30604474
f6dab269:30613381

ask for help~ @roblourens

I believe this is already fixed, can you see if you can reproduce in the Insiders build?
/label info-needed

This is related to view layout. I suspect you have hidden the terminal title, try right clicking the terminal title and see if it's checked.

I believe this is already fixed, can you see if you can reproduce in the Insiders build? /label info-needed

thank you for your reply.  I am also very puzzled why the vscode of my colleagues in the company is normal, but this problem occurs in my vscode (and I compared the vscode version, it is exactly the same).  And I have the default configuration without any modification.  Your rebuild URL cannot download the exe program, and it is always interrupted halfway through the download

@blessli this is the menu @Tyriar is referring to.

If the highlighted entry does not have a check mark, the you should select it to prevent it from hiding.

it work。thank you very much。this issue have solved~


Cannot install mssql-cli

I don't know if this is something with my setup or what but if I run
pip install mssql-cli

and then run mssql-cli --version then everything is fine. On the other hand if i run
pip uninstall mssql-cli
pipx install mssql-cli

and re-run mssql-cli --version I get
/usr/bin/python: Error while finding module specification for 'mssqlcli.main' (ModuleNotFoundError: No module named 'mssqlcli')

I'm using python 3.7. Thanks in advance for looking into this.

The referenced commit doesn't fix my issue. It should've?

It turns out the issue is with the mssqli-cli library itself. I created an issue and a pull request to fix it.
If you want to install now with the fix, you can install from my branch with the fix.
pipx install --spec git+https://github.com/cs01/mssql-cli@593d7f6516 mssql-cli

Note that this method does a git clone of the repository, which unfortunately takes a while, but I did confirm that it works.
mssql-cli.bat from package mssql-cli is now available globally
mssql-cli from package mssql-cli is now available globally
done! ✨ 🌟 ✨
>> mssql-cli
Username (press enter for sa):

Wowwww, that's an awesome and unseen level of support! Thank you so much :)

Tell your friends!

I just wanted to note for others coming across this issue that the aforementioned PR has been merged, but the current mssql-cli v0.15.0 does not include them, so one should still install from @cs01's branch or master.

@shawnohare does installing from master work for you? See https://github.com/dbcli/mssql-cli/issues/247

No, for the exact reason listed in your link. I suppose I was a bit hasty in suggesting the master branch worked, sorry for the confusion!

The fix to mssql-cli has been reverted, and the original problem is back: https://github.com/dbcli/mssql-cli/issues/496


AAMH-1236 Refactor mapping plugin

The following files also should be removed, since they was originally migrated from previous allure to support outdated mappings plugin:

allure-generator/src/main/javascript/components/pane-set/…
allure-generator/src/main/javascript/components/data-grid/DataGridView.js
allure-generator/src/main/javascript/blocks/arrow/arrow.svg
allure-generator/src/main/javascript/layouts/pane/PaneLayout.js


Missing details in documentation

[Enter feedback here]
In the following section (https://docs.microsoft.com/en-us/azure/synapse-analytics/sql/shared-databases-access-control#non-admin-access-to-synchronized-databases), there is a missing detail in the documentation that the GRANT statements MUST be executed within the context of the master database.  These statements will fail if executed within the context of a replicated schema database because the replicated database cannot be modified.

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

ID: c0674852-ce0b-d472-1221-bf1bd912956f
Version Independent ID: 6629bb40-5b98-92bc-fad8-e01d1b1d7b6d
Content: How to set up access control on synchronized objects in serverless SQL pool - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql/shared-databases-access-control.md
Service: synapse-analytics
Sub-service: sql
GitHub Login: @Stralle
Microsoft Alias: strrodic

@softwaresalt Thank you for sharing feedback. We are actively validating it. We will share updates soon.

@Stralle Could you please validate customer feedback and enhance document as appropriate? Thank you.

I can confirm that these commands shall be executed on master database. I'll update doc accordingly. Thanks for sharing this feedback!

The above PR fixed the doc. Closing this issue.
#please-close


fix incorrect result of function 'bitmap_from_array'

Proposed changes
The result of bitmap_from_array is incorrect due to the fact that bits was not cleared.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.81 seconds
stream load tsv:          514 seconds loaded<PHONE_NUMBER>9 Bytes, about 138 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>5 Bytes


Remove redundant lines

System is imported by default in visual basic. so I removed Imports System
calling Flush isn't necessary, the using statement should take care of that.
Used var.
Followed comments convention.

@Youssef1313 This now has some merge conflicts. Once those are resolved, we can :shipit:.
Thanks for making these updates.

@BillWagner, Well, my fork was deleted. Not sure if there is a good way to resolve that conflicts ? or I should open a new PR ?

@Youssef1313 You may see if you can recover the fork from GitHub. Otherwise, yes, a new PR would be cleanest.


Fix coverage measurement for Cythonized code

The THRIFTRW_COVERAGE and THRIFTRW_PROFILE environment variables will be used to enable line-tracing hooks and profiling hooks. Line tracking hooks have a performance penalty so they will be used during tests only. Profiling hooks can be used to get meaningful cProfile output out of thriftrw.
It appears that types specified in Cython code count as uncovered code if they're never called with the wrong types (because Cython raises exceptions if types don't match), so the coverage we see here will be lower than before the code was cythonized.

:goat: :goat: :goat:

This makes me so happy but also so sad.

We can rebuilt it. We have the technology.


Issues with ViewportTexture flags

Godot version:
Godot_v3.0.1-stable_win64.exe
OS/device including version:
Windows 10
Issue description:
ViewportTexture flags are cleared every time a scene is reloaded or when the ViewportTexture is instanced in another scene. By default a ViewportTexture will have only the filter flag enabled, and all other flags disabled, but no matter what flags are set they are all cleared on reload.
Another probably related issue is that all ViewportTextures to the same Viewport will share the same flags, even if you try to set one of them as unique. This isn't always desired behavior. For example I am working on a GPU fluid simulation. I need my advection fragment shader to have the filtering flag set, but I don't want filtering on the texture that displays the fluid, or on all the pressure iterations (especially if there is a performance impact from filtering).
Steps to reproduce:

Create two TextureRects and a ViewportTexture.
Give the TextureRects a ViewportTexture to the Viewport.
Change the flags on one of the ViewportTextures. The flags on the other one should change to match.
Reload the scene, flags will clear.

Another probably related issue is that all ViewportTextures to the same Viewport will share the same flags, even if you try to set one of them as unique.

I want to mention that this is still an issue in the current master branch. I was really confused by this and it took me a while to find out that the shared flags were the issue.

No longer relevant in 4.0, as most of Texture-related flags were moved to CanvasItem properties.


Ansible "flags" variable

Hi,
this is not an issue but rather a question.
I see you are using the flags variable in tasks/main.yml. I can validate the variable exists but couldn't find any reference to it in the Ansible docs or by searching google. Could you give me a hint when that variable is set and how?
thanks a lot
Juergen

Yes.  You probably know of some of the limitations (or, expected behavior) of tags for ansible.  When you include a role in a playbook and assign the tags value, the tags get applied throughout the included role.
This is in contrast with a different expected operation:  create a playbook to include a role that runs only a specific set of tags.
Essentially, I would want to use a set of 'outer' playbooks that call specific roles in specific ways to run specific tasks in the roles.   So in my implementation of other of my roles, I have playbooks that list out specific flags that I want to use for the playbook.  It's a way for me to run a specific playbook without remember the exact flags needed, or to call in other playbooks, passing the flags down from the cli.
For example: pgkehle.ssl-certs
Playbooks:
- name: Create CSR
  hosts: all
  remote_user: root
  gather_facts: ansible_host != 'localhost'

  vars:
    ca_name:          "incommon"

  roles:
    # look for the best .pem file to use, personal or group
    - { role: pgkehle.common, tags: ['always'] }
    # prepare `sc_common_name`, `sc_domain_comp`, etc
    - { role: ../roles/cert_defs }
    # Create the CSR
    - { role: pgkehle.ssl-certs, flags: ['csr_create'] }

- name: Pull the CSR
  hosts: all
  remote_user: root
  gather_facts: ansible_host != 'localhost'

  vars:
    ca_name:          "incommon"

  roles:
    - { role: pgkehle.common, tags: ['always'] }
    - { role: ../roles/cert_defs }
    - { role: pgkehle.ssl-certs, flags: ['csr_pull'] }

If you know of other ways to do this, please let me know!  😄


Create or Update

Prerequisites

Versions

PHP version:  7.3.1
Laravel version:  5.8
Package version:  3.1

Description
Hello,
I've a large list with projects in excel, I want to import this into my application, but the data stays in my excel file (if I've user "Piet", I'll have the user still in the next upload).
Is there something like "createOrUpdate()" for Laravel Excel?

Additional Information

Sounds like you would like to validate the data before import to check if an entry already exists.
You can use Row Validation for this. E.g. with the unique validation rule from Laravel core.

Thanks @GlennM (and sorry for my Dunglish ;) )


curl: patch certificate CN verification

Description of changes
From https://github.com/curl/curl/commit/911714d617c106ed5d553bf003e34ec94ab6a136
Fixes https://github.com/NixOS/nixpkgs/issues/167971
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

LGTM, didn't test though. This fixes pycurl?

I can test later, but this LGTM.

LGTM, didn't test though. This fixes pycurl?

nix build github:NixOS/nixpkgs/pull/167993/head#python3Packages.pycurl works fine

Please target staging or staging-next if this is broken in the current cycle otherwise hydra will be busy for a while because of stdenv rebuild.

Please target staging or staging-next if this is broken in the current cycle otherwise hydra will be busy for a while because of stdenv rebuild.

I'm kinda new to NixOS release process, so I have no idea how to choose between staging and staging-next.
All I can say is that right now, I can't use nixos-unstable anymore because of this bug.

I read a little bit about staging and staging-next but I'm still not sure I fully grasped the consequences of choosing one over the other..
Anyway, for now, I changed this PR to target staging.

I read a little bit about staging and staging-next but I'm still not sure I fully grasped the consequences of choosing one over the other.. Anyway, for now, I changed this PR to target staging.

Perfect. Targeting staging is the right branch for fixes which will cause mass-rebuilds (as seen by the labels...More than 5000 rebuilds for linux alone).
I tested the fix with nix build github:NixOS/nixpkgs/pull/167993/head#python3Packages.pycurl as well :+1:

I tested that python3.pkgs.pycurl is fixed here.  Patch is from upstream and minimal, so it seems safe enough.

Should the pycurl tests disabled in #166335 now be re-enabled?

Yes, done.
Somehow we managed to order this the wrong way.  The temporary disablement would still make sense on master, as it only rebuilds a package that fails anyway without the fix (+ transitive failures and rebuilds).    So if someone wants to pursue that, but either way I'd hope that this gets to master fast as well (in 2-3 days).

Well, according to the PR tags, #166335 also causes > 1000 rebuilds, so it wouldn't be eligible to go directly to master, would it? Or would that be acceptable as a to-be-reverted workaround until we wait for the fix-proper here (with > 5000 rebuilds) to hit that branch?

The diff quite clearly affects only the pycurl package or anything that depends on it.  And on Hydra that package fails.  So if I didn't overlook anything, it should only rebuild those (transitive) failures.

So if someone wants to pursue that

Here we go: #168154


PWA

Closes #134

Codecov Report

Merging #143 (13e82dc) into main (c85eaae) will decrease coverage by 7.39%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #143      +/-   ##
==========================================
- Coverage   77.77%   70.38%   -7.40%
==========================================
  Files          43       44       +1
  Lines         333      368      +35
  Branches       94      103       +9
==========================================
  Hits          259      259
- Misses         64       99      +35
  Partials       10       10

Impacted Files
Coverage Δ

src/service-worker.ts
0.00% <0.00%> (ø)

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 c85eaae...13e82dc. Read the comment docs.

I checked your code only after having already added mine, and I do like that it is well documented and easy to understand. The one little thing I didn't like is that each function is specific to exactly one event and cannot be used otherwise, which doesn't really qualify it to be a separate function in my book).
The pros are probably more than the con(s) in this case, but I'm not sure it's worth the effort. Feel free to modify it yourself in another PR, but keep in mind that yours follows cache-first strategy, while mine follows online-first.


What now that analytics force firebase analytics

Hello,
just wanted to create a new property on my Google Analytics account, I checked Mobile App instead of Web Application. But they told me I had to use FireBase to make that kind of property.
What should I do ? Are you expecting to migrate to it ?
Best regards

hi, there is a workaround, given by google itself
check this
https://support.google.com/analytics/answer/2587086?hl=en&visit_id=1-636317179632979567-126840879&rd=1

Thank you :)

This workaround doesn't look to hold true anymore.
The only option for View is "All Web Site Data"
I see no option for "Create new view" or "Mobile app" from Google.
It appears Google is really forcing the transition to Firebase...

@MadisonBlake You can use https://rnfirebase.io/, the documentation is well explained, I set it up and it's worked out pretty well. :)
Best regards

I am not sure where you're finding that @MadisonBlake.
But for me, under Admin -> View I find this;

If I then select Create New View, I can select Mobile App;

People can use Firebase if they want though, but I don't think Google is forcing anyone over yet.


Why there isn't any news for about one month???

you are rigth


Confusion about sorting delimiter

I was trying to sort the following string in the quotes only using gsi", with little success:
args: "aaa=execution,aaa_bbb={STUFF},aaa_ccc={EXECUTION_MODE},aaa_ddd="
I went into the plugin code and added
let delimiter = ','
between the
let sortables = strpart(@@, sortstart, sortend)
and
let sorted = ...
lines.  And it made it sort on commas again.  Is there something the plugin is doing to try to detect what delimiter to use?  I was under the impression it always sorted on commas as the delimiter for character mode?

Hmm, I'll be honest that I've haven't made much use of the character-wise sort motion of late, and thus haven't run into any issues.
Additionally, while looking at the code, I didn't recognize any of it! Scanning back through the history, I traced it to this change.
I'd be happy to look over an update to this functionality that might solve your issue, but unfortunately, I don't expect I'll have much time to try to fix things myself.


Do we have documented somewhere the functions/methods this API provides?

Do we have documented somewhere the functions/methods this API provides? The params each function/method expects, return values, etc?

Hey Dan,
When using an IDE such as VSCode, the autocomplete feature lets you view the available method and class definitions, including parameters and return types. We currently do not have the SDK methods officially documented as the IDE autocomplete feature provides most of the documentation functionality.
This Sample Code section has few screenshots showing the autocomplete capability.
In addition to documenting the APIs on the new Commerce Cloud Developer Community We have a blog that provides a great introduction to our SDK.
Please let us know if this answers your question or is this something you already tried and still looking for method level documentation?


Script Setup - Class constructor [class] cannot be invoked without 'new'

Vue version
3.2.37
Link to minimal reproduction
https://sfc.vuejs.org/#eNp9kMtOw0AMRX/FGiGlldrMvgRE1Q0fwHI26cSBlMxDHqcFRfl3nAelAond+F7Psa97tY8xP3eodqpIlprIkJC7+Gh842Ighh4IaxigpuAgk9bsar1g4kNbprSYuWZR7Kjkp5TdG1/oGSo4KRhdbEtGqQCKY8ccPDzZtrHvD0bdoWt4lbnPScg24PHyM2K1Xhs1fQR4xrYNE0PPENELmb3Q1UbNC25dGWWR4CVdP/abxUhG7WBSRk0yjbVRb8wx7bROtR1vckp5oFctr5w6z43DHJPbHilcEpKAjdrcMLSIZ6Qtoa+QkP5j/mr9wx2xg/GDRLk9qeTAj+n0FdZl1zJMDuyr6hCcK301p7LBJ6bOcqDV+hr0mzl8ARwjrgw=
Steps to reproduce

Open link to SFC Playground
Click the button
An error Class constructor [class] cannot be invoked without 'new' is raised

What is expected?
No error should be visible in <script setup> as the the equivalent behavior is already working in <script>
What is actually happening?
The code is raising an error Class constructor [class] cannot be invoked without 'new' is raised
System Info
No response
Any additional comments?
See Discussion: https://github.com/vuejs/core/discussions/6480

The compiler produces this code:
(_ctx.$emit('myclick', new _unref(TestClass)())

which is invalid. This should be fixable with an additional set of paranthesis:
(_ctx.$emit('myclick', new (_unref(TestClass))())

I created a PR for the bug

Guys, any update on this?

Any update on this issue? Does that mean we should avoid using classes within script/setup ?

Is it possible to have a follow up on this issue ? The fact that it impacts only the code pushed in production, and not the code in dev mode is especially hurting :'(.
For reference I've been bitten by it both for bindings :var="new Test()" and triggers @click="test = new Test()"
It is sometimes lighter and more readable to be able to new something directly in the template, but if it is not working in production and would largely prefer to break things in my IDE, rather than having to patch some production code somewhere.


[ART-3075] Add plashet subverb "for-assembly"

This adds plashet subverb "for-assembly" to creates a plashet containing arch specific yum repository subdirectories based on a complete set of RPMs required for image builds.
When doozer runs brew builds for an assembly image, it should only need to pass in a single repository created by for-assembly in order for the image to build successfully.
Implemented use cases:

The subverb is run against a runtime.assembly that does have a basis event.
The subverb is run against an assembly with a basis event and an "--image" argument.

Running against an assembly with a basis event and an "--rhcos" argument will be covered in a separate PR.
NOTE:

Includes https://github.com/openshift/doozer/pull/439.
Test releases.yml:

releases:
  art1:
    assembly:
      basis:
        brew_event: 39473862
      group:
        dependencies:
          rpms:
          - el8: cri-o-1.21.0-88.rhaos4.8.gitfd485de.el8
      members:
        rpms:
        - distgit_key: openshift-clients
          metadata:
            is:
              el8: openshift-clients-4.9.0-202106031118.p0.git.714295d.assembly.stream.el8
        - distgit_key: openshift-kuryr
          metadata:
            is:
              el8: openshift-kuryr-4.9.0-202105211327.p0.git.be00acd.el8
        images:
        - distgit_key: openshift-enterprise-cli
          metadata:
            dependencies:
              rpms:
              - el8: bash-4.4.20-1.el8_4

Merge when you feel it is a good time.

@jupierce Thank you for the review.


Paper Spigot 1.12.2: java.lang.ExceptionInInitializerError Java 10

[08:40:41] [Server thread/WARN]: [Multiverse-Core] Failed to set the generator for world 'world3' to 'OpenTerrainGenerator': java.lang.ExceptionInInitializerError
[08:40:41] [Server thread/WARN]: [Multiverse-Core] This is a bug in the generator plugin, NOT a bug in Multiverse!

There is not enough information given here and the issue has not been reported with normal Spigot. Paper Spigot and various other Spigot forks are not supported.

More than 4000 servers run on Paper...


Add Custom Token doesn't work, timeout too fast?

Try this erc20 contract
0x93d6c4483Ae9dB93cA9B39A441019C5bFac5CFa5

Tested again, when it works, it works on both iOS and Andriod, both pick up token information very fast. When it has issues, on Andriod report as “not find”, on iOS keep searching for mins.


fix(@schematics/angular): fix inconsistency in service schema

Closes this issue:
https://github.com/angular/angular-cli/issues/7679

@Brocco can you have a look?


Remove extra prints

Closes #
Code Changes

Removed some extra prints that were left behind in debugging.

Steps to Confirm

[ ] list any manual steps for reviewers to confirm the changes

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
Write some things here about the changes and any potential caveats

Reverting the original PR until after the next release so I will remove the prints then.


[CP] fix crash during "flutter doctor --android-licenses" on macOS

issue_link
https://github.com/flutter/flutter/issues/114031
Commit Hash
https://github.com/flutter/flutter/commit/b8f5394a5ca6d2bce062d9d0a20aaffb4289fb4c & https://github.com/flutter/flutter/commit/1546fa08d3243f3d9401dfa0620231e7cb72ff67
Target
stable
pr_link
https://github.com/flutter/flutter/pull/120717
Impacted Users
macOS users who run yes | flutter doctor --android-licenses
Impact Description
once the sdkmanager sub-process finishes, users will get a cryptic ArgumentError, even though the accepting licenses may have succeeded.
Workaround
The user could find the path on their disk to sdkmanager within the Android SDK, can invoke yes | /path/to/sdkmanager --licenses.
Risk
low
Test Coverage
yes
Validation Steps
Issue yes | flutter doctor --android-licenses from a macOS host.

requesting approval from @zanderso

CP lgtm


Extract Custom Key Value pair from document or image.

How to extract selected key value pair given docs, ex: want to extract name, dob fields from passport ,dont want other fields . how to process them?
Based on bounding box coordinate we can filter but is there any custom training method with annotation to identify the specific fields?

is there any other method based on custom annotation extract what ever data fields we want in structured format? object detection on top of extracted data with simple BB selection and train for new samples

@charlesmindee please let me know !


⚠ Bofrost.be has degraded performance

In cea117c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1044 ms

Resolved: Bofrost.be performance has improved in afaf364 after 48 minutes.


How to replace existing geojson file in superset.

A clear and concise description of what the bug is.
Expected results
I am expecting to see  exact map as per  given geojson file.
Actual results
I want to update  the existing  India geojson file with latest geojson file(PFA),I have tried to replace it with new file in same name, but it is not working as expected.
Screenshots
n/a
Environment
(please complete the following information):

superset version: 0.36
python version: python 3
node.js version: 12.10.9
npm version: 6.10.3

Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Need cofig details to achieve this.
geojson file.zip

Issue Label Bot is not confident enough to auto-label this issue. See dashboard for more details.

@rusackas do you have any thoughts here?

Hi @gyanranjanm!
I'm assuming that we're talking about replacing the map data for the "country map" viz plugin here. I'm also curious what features you're looking to add to the map, but I'll focus on answering your question...
I was able to take your attached file and make a drop-in replacement of the india.geojson file for that plugin. It's not ideal though. A couple of initial issues:

The file is huge - the standard one is 457KB, but this new one is 76.6MB. That's a big payload, obviously.
The boundaries of the geojson seem to lie far beyond India, meaning the country renders quite small in the viz plugin

That said, if you want to experiment with this yourself, you'll want to do the following:

Clone the Superset-UI repo
With your terminal, cd into that directory, and yarn install. A yarn build after that couldn't hurt ;)
cd into wherever your locally running incubator-superset instance is, and then into superset-frontend
type npm link ../../the/relative/path/to/superset-ui/plugins/legacy-plugin-chart-country-map
server your local superset as normal with npm run dev or npm run dev-server

That should put you into dev mode for that plugin. If you edit anything in the legacy-plugin-chart-country-map/src folder, you should see webpack rebuild, and see the changes reflected in your local superset.
At that point, you should be able to replace the file at legacy-plugin-chart-country-map/src/countries/india.geojson and see your work.
Once it works as planned, please feel free to open a PR on superset-ui if you feel it's something the community could use!

Thanks Team.
It  have decreased th file size and just replace it,it is working fine now.
On Thu, Jun 25, 2020, 4:20 AM Evan Rusackas<EMAIL_ADDRESS>wrote:

Hi @gyanranjanm https://github.com/gyanranjanm!
I'm assuming that we're talking about replacing the map data for the
"country map" viz plugin here. I'm also curious what features you're
looking to add to the map, but I'll focus on answering your question...
I was able to take your attached file and make a drop-in replacement of
the india.geojson file for that plugin. It's not ideal though. A couple
of initial issues:

The file is huge - the standard one is 457KB, but this new one is
76.6MB. That's a big payload, obviously.
The boundaries of the geojson seem to lie far beyond India, meaning
the country renders quite small in the viz plugin
[image: image]
https://user-images.githubusercontent.com/812905/85634883-6d340580-b631-11ea-9cab-f2962e717940.png

That said, if you want to experiment with this yourself, you'll want to do
the following:

Clone the Superset-UI repo
https://github.com/apache-superset/superset-ui
With your terminal, cd into that directory, and yarn install. A yarn
build after that couldn't hurt ;)
cd into wherever your locally running incubator-superset instance
is, and then into superset-frontend
type npm link
../../the/relative/path/to/superset-ui/plugins/legacy-plugin-chart-country-map
server your local superset as normal with npm run dev or npm run
dev-server

That should put you into dev mode for that plugin. If you edit anything in
the legacy-plugin-chart-country-map/src folder, you should see webpack
rebuild, and see the changes reflected in your local superset.
At that point, you should be able to replace the file at
legacy-plugin-chart-country-map/src/countries/india.geojson and see your
work.
Once it works as planned, please feel free to open a PR on superset-ui if
you feel it's something the community could use!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-superset/issues/10083#issuecomment-649113568,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APVJUZQVDWSJ5QTHVLNR2IDRYJ7K7ANCNFSM4OAPZTYQ
.

Awesome! Closing issue.

Thanks Team. It have decreased th file size and just replace it,it is working fine now.
…

@gyanranjanm I have installed the superset using pip in python venv Is there is any way to change india.geojson file ?
and how you did that ? please can you explain


Add MSVC debugger visualization file

This pull request contains a file which can be used by the Visual Studio debugger for "pretty-printing" tsl::robin_* containers.
I decided not to use the tsl::detail_robin_hash::robin_hash::m_bucket_count field as it seems to be used as a "cache" and thus can disappear at [almost] any moment. :)

Made a couple of corrections similar to the natvis of the ordered-map project.
Thank you for your contribution!


how to preserve a better identity?

When I use an image of mine it is distorted a lot and does not look like the original face, is there any parameter to improve the identity?

yaa Sure,
Here is the results.zip file which contains the condition.png (reference image) and the output videos in different expressions.
results.zip

Are you using our pre-trained model? If so, it is trained on the CREMA-D dataset, which has limited samples/data distribution, and it might not generalize well to images outside of CREMA-D. Therefore, if you want it to generalize well, you might need a more extensive dataset, which is hard to find since we require emotion labels. You can try omitting emotion labels and use the LRW dataset for better generalization.

I checked your output videos and find that the speech are much longer than those in our training data (~2s). The quality of the first two seconds are reasonable. This might suggest the generalization ability needs to be improved as Emre said.
For the dataset, there is a new emotional talking face dataset released after our publication, called MEAD. Feel free to try to train our model on that dataset. I am also interested in the results.

Thanks @eeskimez and @yzyouzhang for clarification.


Don't build cuda image on release

Describe changes
I implemented/fixed _ to achieve _.
Pre-requisites
Please ensure you have done the following:

[ ] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table.
[ ] I have added tests to cover my changes.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

Why did we do this? Was the CUDA image the one that was causing problems?

@strickvl Yes there were/are some signing issues with an apt-repository which failed the build of this image. As we're going to overhaul the entire GPU resource specification and support soon, I think we will have to build completely different cuda images anyway and therefore thought spending much time and effort doesn't make sense for the upcoming release/pre-release.


DO NOT MERGE! Fast Forward stable to 38ad82f5731bb731fc02859492453be06131bea1.

This PR will be merged automatically once checks are successful.

Jenkins job auth/postsubmit passed


sort intervals incrementally

Explanation About What Code Achieves:
Fix where y-axis interval would be inversed
Screenshots/GIFs:
BEFORE:

Steps To Test:
Get a dataset where the absolute maximum negative value is greater than maximum positive value and you can reproduce the issue

This is still an outstanding issue that needs resolution. I am having similar issues and having to manually define the range to solve it. Would be nice if this PR could get merged!

@pratu16x7 I can adapt this PR for the 1.x branch. What's your thoughts for incorporating this?

Pull Request Test Coverage Report for Build 46

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 38.158%

Totals

Change from base Build 31:
0.0%

Covered Lines:
29

Relevant Lines:
62

💛  - Coveralls

fixes #125


zcli theme import fails with timeout error

Expectations
I expected the command zcli themes:import --brandId=<BRAND_ID> to successfully import the theme, providing the themeId in the output for subsequent publish commands, allowing our CI/CD pipeline to complete the deployment without errors.
Reality
The import process frequently fails with the following output:
Creating theme import job... Ok
Creating theme package... Ok
Uploading theme package... Ok
Polling job status... !
›   Error: Import job timed out

Even though the import command sometimes completes successfully, most of the times it fails, causing the entire CI/CD pipeline to break. Additionally, in cases where the error occurs, the theme is imported into the Zendesk themes library, but since the polling job fails and the process exits with a timeout error, it breaks the deployment process preventing us from retrieving the themeId. This blocks the next step, which relies on the themeId for the publish command.
I'm using a script like this in a ci/cd pipeline. But we get the same error if we try to import theme locally.
Currently, the timeout threshold for the polling job seems to be too low, and it seems there is no way to pass a parameter to increase this timeout threshold or the retries number, making the process unreliable.
Please note that the theme complies with the limits specified here.
Steps to Reproduce

Authenticating ZCLI using environment variables

export ZENDESK_EMAIL=...
export ZENDESK_API_TOKEN=...
export ZENDESK_SUBDOMAIN=...

Try to import the theme

zcli themes:import --brandId=<brandId>

Issue details

Command: zcli themes:import --brandId=<brandId>
Version: @zendesk/zcli/1.0.0-beta.51 darwin-arm64 node-v22.11.0
OS: macOS 13.3.1, ubuntu-22.04

Issue was fixed increasing the retry number of polling job
Commit: https://github.com/zendesk/zcli/commit/b381fe9fb1100e596668232f85408ab426f807a1


Anyway to not trigger this for all errors?

It would be great to not have an invasive overlay appear for every error. If we could specify a function that filters out errors we want to pass through this would be super useful.
We throw network errors, which get caught and handled by our error boundaries. So we'd like to not see overlays in this case, which is considered working perfectly:
filter(error: any, caught: boolean) {
  return !caught && typeof error.status !== number;
}

Hello @ntucker, PR welcome for this feature.

Outdated, if needed please open a new issue.


Fix TOTP custom period

fix #86

@arku3 thank you for the PR. Can you pull main in your branch ? The CI wasn't configured correctly to run on PR.

@n0cloud Done. Let me know if any problem raises.

@arku3 Thank you


permission for new module

hi @viralsolani @bvipul  i have module patients is work fine i create new role medcin i seed pesmision module patient in permission table seeder.php i create new user and associte role medcin with permission module patient view-create-edit-delete when i login with this new user i got nothing please help
thanks

please @viralsolani  i need your help

is there any success!
I am also getting the same error when I created a new module and assign to execute user.
it has not displayed that module when execute user logged in.

@lili94 you got nothing means? can you please explain your issue? You need to add permission in to form request classes and in the sidebar as well.

thanks @viralsolani  it s work i was worng in url  @navdeep1186  look add permision in your request exemple view-patient then change in your controller module using the same request not formeRequest then add permission in
MenuTableSeeder  PermissionTableSeeder ModuleTableSeeder finally sidebar.blade.php and it s wooork

hi @viralsolani @bvipul
will you tell me how I increase the sessoin time out? because after few minutes it is signout by default


Automatically fill in missing attributes in chart data

If you pass in chart data that is missing things like key, or values or label, etc. The library should automatically fill those values in with defaults.
Deliverable: new code with the above feature and a unit test.

Implemented


Items are jumping on initial load

Is this a bug report, a feature request, or a question?
Bug Report
Have you followed the required steps before opening a bug report?
(Check the step you've followed - put an x character between the square brackets ([]).)

[x] I have read the guidelines regarding bug report.
[x] I have reviewed the documentation in its entirety, including the dedicated documentations :books:.
[x] I have searched for existing issues and made sure that the problem hasn't already been reported.
[x] I am using the latest plugin version.
[x] I am following the issue template closely in order to produce a useful bug report.

Have you made sure that it wasn't a React Native bug?
Yes

Is the bug specific to iOS or Android? Or can it be reproduced on both platforms?
iOS. Untested on Android.
Is the bug reproductible in a production environment (not a debug one)?
Yes

Environment

(Write your answer here.)
Expected Behavior
On load, the items are expected to load such that if an item is at index 0, it is the only one visible. Even if the item hasn't loaded yet, it should still be the index displayed on screen.
(Write what you thought would happen.)
Actual Behavior
Currently, on initial load, item at index 0 is not the first element that a user sees. Currently, a user will see item at index 4, and then it jumps back to index 0.
(Write what happened. Add screencasts/screenshots!)
Reproducible Demo
A video of the bug here: https://imgur.com/K4K4WhJ
The code I'm using looks like this:
<Carousel
                                   sliderWidth={screenWidth}
                                   sliderHeight={125}
                                   itemWidth={screenWidth * .4}
                                   data={categories[category]['items']}
                                   renderItem={(thing) => this.renderItem({thing, category})}
                                   inactiveSlideScale={1}
                                   inactiveSlideOpacity={1}
                                   hasParallaxImages={false}
                                   loop={true}
                                   loopClonesPerSide={2}
                                   onSnapToItem={(i) => this.updateIndex(i, category)}
                                   enableSnap={false}
                                   enableMomentum={true}
                                   activeAnimationType="spring"

                               />

renderItem is a simply button:
renderItem({ thing, category }){
     let { item, index } = thing
     let itemLink = () =>  this.props.navigation.navigate({
         routeName: 'Item',
         params: { item: item.id, itemData: item, category },
         key: item.id
     })
     return(
         <TouchableOpacity key={item.id} onPress={itemLink} key={item.id} style={{ borderRadius: 10, width: screenWidth * .4, marginHorizontal: 5 }}>
         <Transition shared='itemImage'>
             <Image source={{ uri: item.image }} style={{ borderRadius: 10, height: 125, width: screenWidth * .4 }} />
         </Transition>
         <Text style={{ flex: 1, justifyContent: 'center', alignItems: 'center' }}>{item.name}</Text>
     </TouchableOpacity>
     )
 }

Steps to Reproduce
This is on the initial mounting/loading of the component. To reproduce, if you use the above code and replace my logic for the image source with your own, you should be able to see how it behaves.

I've tried this using only the required props. Issue persists.

Any ideas?

It's jumping to index 3.
firstItem={3} is working as a temporary fix, but what may cause this to happen in the first place?

:(

Facing same issue.
After first time load, user slide 2 or 3 index and suddenly it set 0 index like init load.
But still waiting for solution. :disappointed:

Facing same issue.
After first time load, user slide 2 or 3 index and suddenly it set 0 index like init load.
But still waiting for solution. 😞

I'm having the same issue , on some phones when user scrolls the first picture it jumps to the first index the first time he scrolls

The default value of loopClonesPerSide is set to 3
so it will load 3 more items on both left and right
which cause the firstItem={0} actually point to the pre-loaded item but not the actually item at index 0

pre-3 - pre-2 - pre-1 - item-0 ...
firstItem={0} is expected to fetch item-0, but fetch pre-3 instead (when loop={true})

Sorry, please allow me to advertise for my open source library! ~
I think this library react-native-reanimated-carousel will solve your problem. It is a high performance and very simple component, complete with React-Native reanimated 2


chore: merge drift from blitz

Codecov Report

Merging #4 (79aed71) into main (eed1097) will increase coverage by 3.10%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
+ Coverage   19.29%   22.40%   +3.10%
==========================================
  Files          10       10
  Lines         171      183      +12
==========================================
+ Hits           33       41       +8
- Misses        138      142       +4

Impacted Files
Coverage Δ

lib/factory_ex.ex
43.90% <66.66%> (+9.41%)
:arrow_up:

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


target: Allow verilator build with musl libc

Building the snitch verilator model on distros using musl libc is currently not possible due to some glibc specific assumption and a musl libc peculiarly. More specifically, code was using #include <printf.h> instead of #include <stdio.h> which is a glibc specific header. Lastly, PAGE_SIZE is a macro provided by Linux headers that expands to a number. musl libc unfortunately indirectly includes it via #include <limit.h>. Renaming occurrences of PAGE_SIZE fixes the name clash.

Once again, thanks for the contribution @zero9178 :)
It LGTM! We are currently facing issues with the internal Gitlab CI. I will fix it ASAP and then proceed to merge this.


[fix] Remove Google OAuth description

Changes

Removed "with Google OAuth" line at the email login section; there is already a "Or continue with" section below it.

@elliotwutingfeng I will deploy this manually later as CICD was not up yet 😅


Permission to see an NPC

I'm writing a very flexible system for my server and I want players to be able to turn features off completely if they don't like them.
I really need an NPC plugin that can display NPCs only to those who have a specific permission

That is an interesting idea! With the upcoming events (Should be out within a few weeks) will feature an InjectionEvent. In that you will be able to cancel it.


Egork/vila unitet test to class

Moving unite tests to the class. Locally pytest test would fail on this file

Hey @comorado, I am not sure if it recommended to move directories during testing: I can imagine introducing a bug during tests that is caused by a previous test moving to a different directory.

Good point @soldni, that's the reason why I had it in setUp method. setUp is run prior to running each individual test https://stackoverflow.com/questions/6854658/explain-the-setup-and-teardown-python-methods-used-in-test-cases

In tests I've authored, I use __file__ to get an absolute path to fixtures, e.g. see tests/test_parsers/test_override.py.
I like your suggestion and instead of changing the directories I will create a class variable which points to the fixtures path.
Some StackOverflow answers recommend creating functions pointing to paths and decorating them with @pytest.fixture

A bit too fancy, thank you for the pointer :)

Hey @comorado, I am not sure if it recommended to move directories during testing: I can imagine introducing a bug during tests that is caused by a previous test moving to a different directory.

Good point @soldni, that's the reason why I had it in setUp method. setUp is run prior to running each individual test https://stackoverflow.com/questions/6854658/explain-the-setup-and-teardown-python-methods-used-in-test-cases

In tests I've authored, I use __file__ to get an absolute path to fixtures, e.g. see tests/test_parsers/test_override.py.
I like your suggestion and instead of changing the directories I will create a class variable which points to the fixtures path.
Some StackOverflow answers recommend creating functions pointing to paths and decorating them with @pytest.fixture

A bit too fancy, thank you for the pointer :)

@soldni I've pushed a commit where I am creating a path variable. Still doing it though in the setUp method. On the pytest run time each individual test receives it's version of the setUp variables. So for example if self.doc was modified in one test, another test is not going to see the modifications because it receives its version of the self.doc created by the setUp class.


DOC-3469 overview topic for Native Helm deployments

https://harness.atlassian.net/browse/DOC-3469
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[x ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

https://harness.atlassian.net/browse/DOC-3469

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


getUrl doesn't provide a hash, so --impure is required to build.

getUrl doesn't provide a hash attribute, so when it's result is used in the builder we get the error in pure evaluation mode, 'fetchTarball' requires a 'sha256' argument.
A workaround is to build with --impure. However, this negates a big advantage of flakes, especially if this requires a downstream user with a project built using opam-nix as a flake input to rebuild their NixOS config with --impure.
Materialization can be used to get all the dependencies of a project statically, instead of resolving them with opam at every build. Do you think it would be possibly to statically get hashes of these dependencies at this point and commit them to the json file produced my materialization?

After closer inspection, I realise that the hashes are already extracted from the opam package definition, and that this is orthogonal to materialization.
After some tracing it looks like this error is the result of https://github.com/ocaml/opam-repository/blob/aa363b34739a7323c3b593e7ab850bc31fbfd202/packages/ocaml-variants/ocaml-variants.5.0.1+trunk/opam not having a checksum.

After some investigation it seems fetchTarball fetchurl only support sha256 hashes https://github.com/NixOS/nix/blob/601849b95afc3d173ea34ff5dba6353f9b71b495/src/libexpr/primops/fetchTree.cc#L232

fetchurl and fetchTarball are different things, fetchurl is a FOD (and can accept many hash formats, incl. md5, sha256, sha512), while fetchTarball is eval-time

I don't think it's a good idea to record hashes during materialization when they're not in the original opam files. They could correspond to an "unstable" URL (think https://github.com/foo/bar/archive/master.tar.gz), and so telling Nix that the result is supposed to have a stable checksum is wrong and will lead to problems.
Also, if you're experiencing this problem with the trunk version (which is an unstable URL), there's a simpler fix: just require ocaml-base-compiler = "*" in your query, to force opam to choose a stable OCaml version.

Thanks for the response, @balsoft.

if you're experiencing this problem with the trunk version (which is an unstable URL), there's a simpler fix: just require ocaml-base-compiler = "*" in your query, to force opam to choose a stable OCaml version.

Unfortunately this still seems to give an error:
nix build ...
error: in pure evaluation mode, 'fetchTarball' requires a 'sha256' argument

       … while evaluating the attribute 'src'

       at /nix/store/xjmchy23lg6wrlqbm32apzi9zrddf6k0-source/src/evaluator/default.nix:444:17:

          443|         pkgdef.src or pkgs.pkgsBuildBuild.emptyDirectory;
          444|     in { inherit archive src; };
             |                 ^
          445|

       … while evaluating the attribute 'src' of the derivation 'ocaml-variants-5.0.1+trunk'


Add 24 sparse + dynamic int8 composed subclasses

This PR adds in int8 dynamic quantization + semi-structured sparsity support into torchao.
This is implemented by extending the existing quantization subclasses to use sparse ops.
Ideally we would be able to compose subclasses, and call to_sparse_semi_structured from inside the quantization subclass, but ATM nested subclass tracing does not work with torch.compile and for stuff like fusing scales into the sparse multiply you would probably want to implement it like this anyways.
In particular, this PR adds in two new subclasses:
Int8DynamicallyQuantizedSemiStructuredSparseLinearWeight
Int8DynamicallyQuantized24CusparseltLinearFuseMulWeight

For the cuSPARSELt subclasse, I can extend Int8DynamicallyQuantizedWeightBase by storing the compressed representation in W_int_repr.
FuseMulWeight will fuse one of the multiplies for the dequant into the cuSPARSELt matmul op. However cuSPARSELt expects this in a float32 format, so this eats into our previous speedups since we're now passing this as a bfloat16 tensor.
However for the general subclass, I need to extend QuantizeWeightBase, because I need to pass two tensors (packed and meta) for the CUTLASS sparse mm op. This relies on to_sparse_semi_structured to decide between CUTLASS and cuSPARSELt, which is the right choice for UI but makes benchmarking between them kind of difficult, since it's a class var that decides which backend gets used.  Maybe we should add a flag to to_sparse_semi_structured because you might mix between cutlass and cusparselt.
I've also added a benchmarking script for SAM. I don't know how we plan on handling dependencies in torchao, but let me know if there's a better place for that.

On batch size 32, I see a 1.16x speedup over bfloat16 torch.compile baseline, from 21.96 -> 25.54 img/s.
Other benchmarks (BS=16)

can you move the benchmark_sam and other .py files to one of the other directories? Maybe make a torch/benchmarks dir?

@jcaip just saw the CI failure issue is our CI GPU is too old so updating it now to A10G which should work for your code so after I merge this https://github.com/pytorch/ao/pull/176 make sure to rebase to main


Set and Retrieve a secret - Portal

I see that the article shows you how to Add or Set a Secret but I can't see anywhere in the article where it shows how to retrieve the secret?

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

ID: 23617aaa-af93-2f09-5a7a-2a9a977dc5f2
Version Independent ID: 0d68d344-4b56-6f3b-ac02-08d43e2ac559
Content: Azure Quickstart - Set and retrieve a secret from Key Vault using Azure portal
Content Source: articles/key-vault/quick-create-portal.md
Service: key-vault
GitHub Login: @barclayn
Microsoft Alias: barclayn

@michaelspinks Thank you for your query. We will investigate and update this thread further.

@michaelspinks A pull request for this change to update the screenshots on the page showing secret retrieval are already underway and would be updated in a few days. The changes have been sent to the author and we have
For information , I am just posting the detail here.
Step 1:- We create a secret named Password.

Step 2. We click on the newly created secret in the portal and check the current version of the secret .

Step 3. We can find the secret by clicking "Show Secret" value button as seen above. Once you do that you would be able to retrieve the secret as in the screenshot below.

We have filed a request for updating the documentation and it will be updated in a few days. Please let us know in case you have any further queries on this. In case you do not have any further queries , we will close this request by tomorrow.
Thank you.

@michaelspinks We will be closing this issue now. Should you have a further query , please feel free to tag me to your reply or open a new issue and we will help you with the same.


Update list of allowed tags

See https://apply.opentech.fund/apply/submissions/3677/#communications. Horizontal line and quoting are not showing up.

The "bleach" filter is what sanitise all output from users so this is a global setting.
What tags should be allowed? The current list looks like this.
BLEACH_ALLOWED_TAGS = ['h2', 'h3', 'p', 'b', 'i', 'em', 'strong', 'a', 'ul', 'ol', 'li', 'br', 'span']

Your example above indicate that 'blockquote' and 'hr' should be allowed as well. Any more?
In the old Drupal site we allowed these tags:
´´´

´´´
My suggestion is:
BLEACH_ALLOWED_TAGS = ['a', 'b', 'big', 'blockquote', 'br', 'cite', 'code', 'dd', 'del', 'dl', 'dt', 'em', 'h2', 'h3', 'h4', 'h5', 'h6', 'hr', 'i', 'ins', 'li', 'ol', 'p', 'pre', 'small', 'span', 'strong', 'sub', 'sup', 'ul', ]

Let's start with those and go from there. Thanks!


Max. momentary erratic results with EBU test files(?)

Hello,
first a bit of background:
I'm a contributor to the SWS Extension (Github repo), an extension for the REAPER DAW. which implements offline Loudness scanning using libebur128.
I've scanned the files from the EBU loudness test set with our Loudness scanner and they all pass, except for test set 13.
In tech3341 it specifies for these files:

Max M = −23.0 ±0.1 LUFS, for each segment

However with our current implementation we get the following results:

I'm now trying to determine if there's a bug in our implementation (we use a modified version of libebur128) or if this may be an issue with the library itself.
Thank you.

Huh, this is weird. We have some tests for test set 13, and they run successfully: https://github.com/jiixyj/libebur128/blob/c13deee58f47427c9506f1b0b259c16f909edab9/test/tests.c#L367
Maybe this helps tracking down the problem.

Ah missed these tests, thanks for pointing me to it.
So the issue must be on our end, that's at least good to know.
Closing this, thanks again.