principal attributes

/cc @douglas-reid @geeknoid @liminw
Is this a better language to describe these attributes? Addresses two points:

there is no request for TCP
we don't have a well defined "application" concept

Signed-off-by: Kuat Yessenov<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: kyessenov
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: sdake
Assign the PR to them by writing /assign @sdake in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid, kyessenov
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [geeknoid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

We really should be generating this stuff from the source manifest, just like we started to generate the description of all the Helm install options from the source yaml.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid, kyessenov, liminw
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [geeknoid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Back Button on Second Screen

We would like to look into the functionality of the back button on the second screen:

Does Activity.this.finish() exhibit the behaviour that we would want?
Look into the behaviour of the default Android back swipe? Is that enough?
Bottom line: Do we want the second screen Activity to persist or disappear once the back button is clicked?

Closing issue as Activity.finish() takes care of the functionality for now. Might even remove it in the future to solely use the back gesture.


use GDALOpenEx instead of Driver->pfnOpen

This implements the proper way of initializing the GDALDataset for #18

CI is failing on all of the shared targets: https://travis-ci.org/github/contra/node-gdal-next/jobs/734099501#L945


Remove unsupported Metadata component from the docs

kubeflow/metadata appears to be dropped from KF:

UI: https://github.com/kubeflow/manifests/pull/1619
Backend: https://github.com/kubeflow/manifests/pull/1638
Centraldashboard entries: https://github.com/kubeflow/kubeflow/pull/5404
Do we agree that it should be dropped from the docs as well?

cc: @RFMVasconcelos @Cventes

+1, we should remove it.
Pipelines WG will start adding new metadata doc as part of Pipelines.
/Cc @kubeflow/wg-pipeline-leads

@Bobgy did you make a tracking issue to add metadata docs under the pipelines section?
I am a bit worried that users will never see it, given how many docs are in the pipelines section.


SSO Behind a Reverse Proxy

Hi,
I'm struggling trying to use the PCF SSO provider behind a reverse proxy.
In the 302 response to the client, I somehow need to swap out the auth domain to something that is externally routable but can't see how to do so.
I did find that is it possible to override the defaults in the ASP.NET Core pipeline (from what I could tell), but it doesn't get picked up:
            app.UseCloudFoundryAuthentication(new CloudFoundryOptions             {                 AuthorizationEndpoint = "https://pcf-sso.somedomain.com"                              });
Anyone come across this scenario?
Thanks,
René

Reopen if this is still an issue.


Add a new update available widget to statusBar

This is a continuation of #4935 that replaces that PR.
It refactors the palette manager code to allow us to reload the catalogues without generating the whole palette manager UI. This makes it more efficient to do background checks of updates so we can keep the status widget up to date.

Sigh - pushed the brnach before I had commited all my work to it.... update incoming....


OAuth authorization issue

Hello Bailey!
I've got the error when using OAuth:
Authentication failed: OAuthError: Received an HTTP error
    at OAuth.<anonymous> (.../node_modules/envato/dist/clients/oauth.js:51:23)
    at Generator.next (<anonymous>)
    at fulfilled (.../node_modules/envato/dist/clients/oauth.js:5:58)
    at processTicksAndRejections (node:internal/process/task_queues:96:5) {
  http: BadRequestError: Bad Request
      at HttpClient._getHttpError (.../node_modules/envato/dist/helpers/http.js:111:30)
      at HttpClient.<anonymous> (.../node_modules/envato/dist/helpers/http.js:43:29)
      at Generator.next (<anonymous>)
      at fulfilled (.../node_modules/envato/dist/helpers/http.js:5:58)
      at processTicksAndRejections (node:internal/process/task_queues:96:5) {
    code: 400,
    response: {
      error: 'invalid_request',
      error_description: 'No authorization header passed'
    }
  }
}

Issue coming from line:
const client = await oauth.getClient(code);
from your example:
// This is an example express route to handle the request
app.get('/authenticate', async function(req, res) {
    try {
        // Fetch the code from the URL
        const code = req.query.code;

        // Authenticate the user via the API
    	const client = await oauth.getClient(code);
        const username = await client.private.getUsername();

        // You can also get their User ID for data storage
        const { userId } = await client.getIdentity();

        res.send(`Hello, ${username}!`);
    }
    catch (error) {
        console.error('Authentication failed:', error);
        res.status(500).send('Something broke! Try again.');
    }
});

My suggestion, how to fix it in attach:
Fix_for_OAuth_authorization_flow.patch.zip
Thanks in advance.

Hi @HotTemplate, thanks for reporting this and apologies for my slow response time! 😅
I've released a fix for this in v3.0.7, let me know if you're still having issues after updating. Cheers!

Yes, it's works now. Thanks a lot.


Deleted user still appears in the users table

when deleting a user in the user's details page https://app.dittofeed.com/dashboard/users/1 via the trash can button in the top right corner, I get a green toast saying that the user has been deleted, yet in the table at https://app.dittofeed.com/dashboard/users/ I can still see the user, it just does not have any properties but the id and the segments it belonged to are still there. Refreshed the page, restarted the servers and the user is still there.
I expected the user to be completely deleted and to not appear in the table at all after deletion. Is this the intended behaviour?

@juan-libros let me take a look!

I'm not able to reproduce this behavior. @juan-libros would you be able to take a screen recording? 🙏

Sorry for the late response.
Screencast from 2024-10-29 11-17-17.webm

Thanks for this!

PR to fix https://github.com/dittofeed/dittofeed/pull/1116


Create Button in csharp

Describe the bug
I have different output creating FlexButton in code VS xaml
To Reproduce
public static FlexButton CreateFlexButton(string text = null)
{
        FlexButton flexButton = new FlexButton();
        if (!String.IsNullOrEmpty(text))
            flexButton.Text = text;

        flexButton.HorizontalOptions = LayoutOptions.Center;
        flexButton.WidthRequest = 300;
        flexButton.HeightRequest = 80;
        flexButton.CornerRadius = 22;
        flexButton.BorderColor = Color.FromHex("#49516F");
        flexButton.HighlightBorderColor = Color.FromHex("#6279B8"/*"#f28e02"*/);
        flexButton.ForegroundColor = Color.FromHex("#ffffff");
        flexButton.HighlightForegroundColor = Color.FromHex("#49516F");
        flexButton.BackgroundColor = (Color) Application.Current.Resources["MyColor"];
        flexButton.HighlightBackgroundColor = Color.FromHex("#8EA4D2");
        flexButton.FontSize = 30;

        return flexButton;

    }

I think I should see text center aligned and the shadow on the bottom of each button, as I see in your xaml example
        <flex:FlexButton
            x:Name="WideButton"
            ClickedCommand="{Binding ButtonClickedCommand}"
            IconOrientation="Right"
            HorizontalOptions="Center"
            WidthRequest="300"
            HeightRequest="76"
            CornerRadius="22"
            Icon="lightbulb.png"
            Text="Icon and Text"
            BorderThickness="0,0,0,4"
            BorderColor="#49516F"
            HighlightBorderColor="#6279B8"
            ForegroundColor="#ffffff"
            HighlightForegroundColor="#49516F"
            BackgroundColor="#6279B8"
            HighlightBackgroundColor="#8EA4D2"/>

Expected behavior
center text and shadow at the bottom
Screenshots
I have this output on a Nexus 7.1 Android API 25 emulator

Please complete the following information:

Which version of the FlexButton do you use? 0.70
Which version of Xamarin.Forms do you use? 3.1 last stable
Which OS are you talking about? Android

Hi @robinmanuelthiel
Do you have some news about "alignment" problem?
I have tested again in a real device and it works

But using the same code on a popup https://github.com/rotorgames/Rg.Plugins.Popup I have the problem

This is the code I use
    public static FlexButton CreateFlexButton(string text = null)
    {

        FlexButton flexButton = new FlexButton();
        if (!String.IsNullOrEmpty(text))
            flexButton.Text = text;

        flexButton.HorizontalOptions = LayoutOptions.Center;
        flexButton.WidthRequest = 300;
        flexButton.HeightRequest = 65;
        flexButton.CornerRadius = 10;
        flexButton.BorderColor = Color.FromHex("#49516F");
        flexButton.HighlightBorderColor = Color.FromHex("#6279B8"/*"#f28e02"*/);
        flexButton.ForegroundColor = Color.FromHex("#ffffff");
        flexButton.HighlightForegroundColor = Color.FromHex("#49516F");
        flexButton.BackgroundColor = (Color)Application.Current.Resources["MyColor"];
        flexButton.HighlightBackgroundColor = (Color)Application.Current.Resources["FlexColorBackground"];
        flexButton.FontSize = 24;

        return flexButton;

    }

Thanks

Can you please check, if this has been ixed in version 0.9.0: https://www.nuget.org/packages/Forms.Controls.FlexButton/0.9.0

Thanks @robinmanuelthiel . For this project I have used native Button because of this problem… I will take a look in the next project
Thanks again


Sky: Adapt radiance size if AUTOMATIC_MODE resolves to REALTIME

Bugsquad edit, fixes: #76166

For the record, your commit seems not to be linked to your GitHub account. See: Why are my commits linked to the wrong user? for more info.

For the record, your commit seems not to be linked to your GitHub account. See: Why are my commits linked to the wrong user? for more info.

Thanks, I didn't noticed, updated my config.

Thanks!

Cherry-picked for 4.3.1.


Adapt CRUD endpoints to use KaindaModel functions

Update and delete endpoints are incorrectly called in the default autogenerated controllers


🐛 Bug Report: Workspace 'backend' not found while running

📜 Description
we are trying to build docker image from git repo. while doing host build,
yarn install - success
yarn tsc - success
yarn build:backend - fails with error
**Usage Error: Workspace 'backend' not found. Did you mean any of the following:

@backstage/app-defaults
@backstage/back**
Can some one please help if we missing some step?

👍 Expected behavior
Build the backend, which bundles it all up into the packages/backend/dist folder.
👎 Actual Behavior with Screenshots

👟 Reproduction steps

Clone github repo
enter into backstage folder
yarn install
yarn tsc
yarn build:backend - failure

📃 Provide the context for the Bug.
its stopping to build docker image
🖥 Your Environment
local
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

@awanlin thanks for the prompt response. how can we package it to run in docker environment?

You would use the results of running npx @backstage/create-app@latest. I really recommend you read through the documentation I shared as it will really help

Thanks now managed to build a dcoker image. Happy to close this ticket


Low-precedence and/or/not

I'm out of time to continue work on this right now, so thought I'd post my progress so far for comments.

New "low-precedence" operators and/or/not via parse tree and parenthesization, effectively just below &&/||/!
Implement not in all cases (fixes #287)
New coffeeAndOr restores CoffeeScript's precedence for and/or (same as &&/||)
Old coffeeNot restores CoffeeScript's precedence for not (same as !)

I still need to fix a few tests (related to custom operators) and add actual tests, but you can see intended use in the added documentation.
Possible items to discuss:

Name of coffeeAndOr and coffeeAndOrNot. Partly hampered by existing coffeeNot here, but not sure what's best.
Is low-precedence and/or/not good (by default)?

In particular, do we like not x == y → not (x == y) as in Python but not CoffeeScript?
I think a || b and c || d → (a || b) && (c || d) is reasonable because, if you're mixing styles of operators, you probably expect something to happen. If you stick to one or the other, there's nothing surprising.
Lots of precedent (heh) for it: Ruby, Perl, LiveScript. Although they differ on the exact precedence (see #287).

xor is not yet low-precedence, but it already didn't mix with &&/|| so not sure what's best... (This is also harder to fix.)
Should custom operators be processed at regular or low precedence? Low precedence might make sense given that they're also words (currently), but on the other hand we're not changing precedence of word operators like instanceof so I'm not sure.

Adding not at the same precedence as ! is good and should be done.
Having different precedence like perl is an interesting experiment that I think could have value but that value may be finding out that it is too confusing and there are always some weird cases (or we could luck out and find a perfect intuitive yet complex precedence ladder).
I think having the default be words and symbol logical operators have the exact same precedence but have a feature flag where we (or those interested) can try out zany ideas and get a feel for it it's worth the complexity.
Next steps:

Merge not as synonymous with !.
Experiment with alternative precedence of logical operators (could start with reading and documenting Ruby and perl pros and cons)
Split out a separate issue for xor precedence
Separate issue/discussion for research and experiments around custom operator precedence. One possibility is something like: operator someOp < otherOp := value where <, =, > could specify precedence in relation to other operators instead of the default precedence.


[docu-2475]Helm installation guides

How to install Kong Gateway with Helm using Kind, Docker Desktop Kubernetes, and cloud hosted Kubernetes.
https://konghq.atlassian.net/browse/DOCU-2475

@Guaris I attempted to add the broken links to the ignore list but I seem to be missing a piece of that picture.

/lgtm


LPEG library redefines LuaJIT APIs that are added via 5.2 compatibility flag

[3/49] Compiling deps/lpeg/lpcap.c ...
In file included from deps/lpeg/lpcap.h:9,
                 from deps/lpeg/lpcap.c:9:
deps/lpeg/lptypes.h:38: warning: "luaL_newlib" redefined
   38 | #define luaL_newlib(L,f)        luaL_register(L,"lpeg",f)
      |
In file included from deps/lpeg/lpcap.c:7:
deps/luv/deps/luajit/src/lauxlib.h:125: note: this is the location of the previous definition
  125 | #define luaL_newlib(L, l)       (luaL_newlibtable(L, l), luaL_setfuncs(L, l, 0))
      |
[4/49] Compiling deps/lpeg/lpvm.c ...
In file included from deps/lpeg/lpcap.h:9,
                 from deps/lpeg/lpvm.c:13:
deps/lpeg/lptypes.h:38: warning: "luaL_newlib" redefined
   38 | #define luaL_newlib(L,f)        luaL_register(L,"lpeg",f)
      |
In file included from deps/lpeg/lpvm.c:11:
deps/luv/deps/luajit/src/lauxlib.h:125: note: this is the location of the previous definition
  125 | #define luaL_newlib(L, l)       (luaL_newlibtable(L, l), luaL_setfuncs(L, l, 0))
      |

No idea if this is a real problem, but it shouldn't be too difficult to fix in the LPEG code (via define guard). Since I somehow doubt upstream will care about LuaJIT issues, modifying the source would be the only viable solution. This however means a separate fork would have to be maintained... that seems like an annoyance at best, even if the LPEG project doesn't see many changes.

Bumping this up in priority since it's easy to fix, mildly annoying, and may cause silent failures down the line.


Add retpoline mitigation for x86-64 Linux indirect calls

Spectre/Meltdown lurk where indirect calls are used. Let's protect against the badness if we can.

What changed in bpf_conformance for this?

What changed in bpf_conformance for this?

Mistake! Should be sorted in the newest version!


Provide possibility of rebuilding certain failed test/tests

With the current implementation, rebuild testrun gives an opportunity to rebuild either all tests in testrun or all failed tests. And it would be nice to have a third option: rerun only failed tests, which were chosen by user, e.g. when only specific test method was fixed and there's no necessity in running other failed tests again

@hursevich do we plan to work on this anytime soon? we'll need to alter current UI flow somehow...

closing as won't fix


SSL certificate expired on newsherenow.com

Hi Richard (@cpsolver),
The SSL Certificate for https://www.newsherenow.com/ has expired as per this Qualys SSL Labs report. The certificate expired on 1st July 2024 (14 days ago).
Kind Regards,
Liam

The NewsHereNow.com website has been partially migrated to a new web hosting package that includes an SSL certificate.  HTTPS access is back to working.

Hi Richard (@cpsolver),
I can confirm that the SSL certificate for the website is getting an A rating and everything looks OK. I'll close this issue.
Kind Regards,
Liam


Suggestion/egg files metadata

Suggesting enhancement to the python cataloger.
Catalog package files for legacy egg 'installed-files.txt' metadata.
For example psycopg2 does not have linux wheel so it uses egg, then django uses psycopg2 and so one...
https://github.com/pypa/pip/blob/1cda23bd6bd0066ad557e4f7ca7c670744357b2b/src/pip/_internal/operations/install/legacy.py
Note: It seemed that EGG installed-files.txt and wheel RECORD use paths that are relative to different base dirs.
I added a normalization flow for the installed-fiels.txt to the one used by RECORD.

Installed-files.txt - relative to egg Info dir.
RECORD - relative to site-packages root dir.

Merged with #1217 - Thanks again for all the help on this one @houdini91!


Added README.ru-RU.md

Added Russian translation of README.md.
Also added links to other languages that are already in master.
I'll continue to maintain this translation in future when I have time.
Part of #44

The links are hard-coded so they can break if someone moves files or renames them, and now a lot of people add new translations.
Btw, I can translate the application itself when it will support localization. Maybe I'll add the support myself later in October

Looking good! There are a lot of prs to manage rn. I will merge this once I a few of the rest that have issues. thanks for the translations


Only activate eqMac when external speakers are connected

Hi, is it possible to only activate eq settings when external speakers or headphones are connected and chosen as audio output? Also disabling after returning to built-in mac speakers.
(automatically)

Hey everyone. Excuse me for the lack of updates. I have been really busy on the new version of eqMac. I am happy to announce an open beta release.
You can download it here: https://eqmac.app
There are a lot of known bugs that I could not fix just yet:

Driver not installing from the first try. Sometimes need to restart eqMac a couple times.
Complex Device switching logic is not perfect (for example when switching off the device that's currently playing, plugging in new device etc.)
UI takes long to load (a lot can be done to improve that)
UI dimensions are not synching properly
There's a noticeable audio delay in the playback, so not suitable for video yet (I know where the problem lies just haven't had the time to fix yet)
Sometimes the playback doesn't start, in that case please try to switch the output to the device you want and eqMac should try and build the audio pipeline again.

So as you can see there's still some work to be done but at least it's a completely modern tech stack that I am fully willing to work on: 100% Swift, User Space driver and a Web UI, instead of C++, Kernel Drivers. As I said, should work on Catalina and should not have any pesky ear drum blasting volume bugs (at least I didn't come across any).
I'm open to feedback and issue reports here: https://github.com/bitgapp/eqmac
That repo will become the main repo for eqMac and this repo will be archived eventually.
And yes it's called just eqMac, not eqMac3 :) And it will stay that way forever.

Hey everyone 👋
I will be open sourcing the new version of eqMac, therefore, I am closing this issue and repository as I will be transfering nodeful/eqMac2 repo to bitgapp/eqMac
Please bear with me for the next couple of days as I setup that new repo. The eqMac2 code will be living in the old-eqMac2 branch if anyone is interested in referencing it for whatever reason.
Please try out the new version and report bugs to the new repo.
Thanks for using eqMac.

Moving discussion to #231

I don't understand, is this feature active? I can't find it anywhere.
I want to activate eqMac only on the external monitor.


Document how to treat scatter using bin-free CAM

All current documentation of the bin-free CAM algorithm incorporated in #888 only covers zero-scatter CAM. I have no strong opinion on the optimal method, but I am unhappy with noisy_percentile because of flaring at the endpoints. @duncandc may like to weigh in here after experimenting with the algorithm. It will likely be worth including a scatter optional argument in the future.

Likewise, there should be a worked example of the return_indexes feature of CAM.


Automation page shows no data. API resources not found 404

Home Assistant release (hass --version):
0.48.0
Python release (python3 --version):
Python 3.4.2
Component/platform:
Raspberry pi 3
Description of problem:
Using the automation's editors shows blank screen after selecting an automation.
Expected:
View to be able to modify the selected automation
Problem-relevant automations.yaml entries and steps to reproduce:
- id: turn_on_house_in_the_morning
  alias: Turn on house in the morning
  hide_entity: True
  trigger:
    platform: time
    at: '05:00:00'
  action:
    - service: homeassistant.turn_on
      entity_id: switch.livingroom_dehumidifier

Select Automations
Select Turn on house in the morning
Blank screen and error in chrome dev panel

Additional info:
in chrome, I see the following error message.
api/config/automation/config/turn_on_house_in_the_morning Failed to load resource: the server responded with a status of 404 (Not Found)

same thing happens when i try to hit the endpoint using curl
curl -X GET -H "x-ha-access: xxxx" -H "Content-Type: application/json" http://pi:8123/api/config

{"components": ["logbook", "octoprint", "ios", "switch.rpi_rf", "http", "rpi_gpio", "emulated_hue", "automation", "logger", "binary_sensor", "sun", "binary_sensor.rpi_gpio", "group", "binary_sensor.octoprint", "switch", "light", "sensor.haveibeenpwned", "mqtt", "media_player.plex", "discovery", "zeroconf", "notify.ios", "frontend", "camera.mjpeg", "sensor", "history", "api", "light.hue", "sensor.template", "conversation", "updater", "media_player", "websocket_api", "zone", "binary_sensor.mqtt", "tts", "sensor.dht", "sensor.yr", "device_tracker", "script", "notify", "sensor.ios", "camera", "sensor.darksky", "recorder", "input_boolean"], "version": "0.48.0"}

curl -X GET -H "x-ha-access: xxxx" -H "Content-Type: application/json" http://pi:8123/api/config/automation/config/turn_on_house_in_the_morning

404: Not Found

Seems to work fine on dev.
$ curl -H "x-ha-access: xxxx" -H "Content-Type: application/json" http://localhost:8123/api/config/automation/config/check_sensor
{"action": {"data": {"message": "Sensor value greater than 10"}, "service": "persistent_notification.create"}, "alias": "Check sensor value and show notification", "id": "check_sensor", "trigger": {"above": 10, "entity_id": "sensor.random", "platform": "numeric_state"}}

Possible duplicate of #7823.

https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/config/automation.py#L14-L14 is never called in my setup which is causing the issue.

found it. I was missing config: in my configuration.yml

Yupp, this issue did cost me multiple evenings to find. Maybe it would be a good idea for the editor to check for this entry and complain/give a hint if it's missing.


Grant ownership on database role

Terraform CLI and Provider Versions
Terraform v1.5.7
on darwin_arm64
Terraform Configuration
snowflake_database_role_ownership_grant "example" {
  on_database_role_name: "EXAMPLE_DB.EXAMPLE_DB_ROLE"
  to_role_name: "EXAMPLE_ACCOUNT_ROLE"
}

Expected Behavior
It worked
Actual Behavior
snowflake_database_role_ownership_grant doesn't exist
Steps to Reproduce

terraform apply

How much impact is this issue causing?
High
Logs
No response
Additional Information
I've tried every other incantation I can think of.
I can create database roles, I can grant database roles, I can grant privileges to database roles, but I cannot grant ownership.

Hey @pauldraper.
The answer is almost the same as in the previous issue you are participating in: https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/2610#issuecomment-1988029679. In short: we are introducing grant ownership resource in #2604 (and probably 1-2 more PRs after that). It is a part of https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/ROADMAP.md#redesigning-grants communicated almost two months ago.
Also, it is possible to run any SQL statement by using https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/unsafe_execute. This is a dangerous resource, though, so I would advise waiting for the grant ownership.

@sfc-gh-asawicki , no #2610 is about snowflake_database_grant, snowflake_stream_grant, etc being deprecated without suitable replacements.
This issue is that database role ownership is completely unsupported, even ignoring deprecated resource.
Thanks for suggesting unsafe_execute. The primary problem with that resource is that it offers no drift detection like other Terraform resources.

@sfc-gh-asawicki , no https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/2610 is about snowflake_database_grant, snowflake_stream_grant, etc being deprecated without suitable replacements.

Well... It's about grant_privileges_to_account_role Unsupported privilege 'OWNERSHIP'. It's also true for the grant_privileges_to_database_role if you use it with the OWNERSHIP privilege. Database roles were not supported earlier, so there was no old dedicated ownership resource then.

This issue is that database role ownership is completely unsupported, even if using deprecated resources.

There are workarounds. Also, in such cases (missing features) please create the feature-request and not a bug ticket.

Thanks for suggesting unsafe_execute. The primary problem with that resource is that it offers no drift detection like other Terraform resources.

It's not meant to be a fully-fledged terraform resource. It's a workaround for the urgent needs when the feature is not yet supported.

Hey 👋
Closing, as the issue was about the deprecated/not allowed functionality. Recently, we released a new grant resource which is capable of granting ownership. Please, give it a try. If there will be any issues with it, create another GitHub issue. Also, please check our technical documentation section where you can find a migration guide that can help you with upgrading to the latest grant resources and our newly added design decision doc (regarding new grant resources).

Also, in such cases (missing features) please create the feature-request and not a bug ticket.

FWIW feature (grant ownership on db role) was not missing, but it was deprecated without replacement.


Ss

Geode Version
Ifk
Related Problem
Icjc
Your Solution
Jcjc
Additional Information
Lcic

Lviiff

Guys I can translate it:
Title: Shapeshifter
Geode Version: I forked
Related Problem: I completed Jumping Circles
Your Solution: Joe couldn't jump circles
Additional Information: Losers can, I can


FSDP in tinygrad

Initial attempt at FSDP.  I want it to work such that you can shard your batch across devices in a totally seamless manner and have the gradient all-reduction just work alongside FSDP. Currently far from finished.

stale


[5.8] Rule::unique not ignoring model when value 0

Laravel Version: 5.8.10

Description:
When model PK / ignored column value is 0, it does not get ignored because the value needs to evaluate to true in order to be ignored:
    public function __toString()
    {
        return rtrim(sprintf('unique:%s,%s,%s,%s,%s',
            $this->table,
            $this->column,
            $this->ignore ? '"'.addslashes($this->ignore).'"' : 'NULL',
            $this->idColumn,
            $this->formatWheres()
        ), ',');
    }

Considering the code this would probably fail to validate empty strings as well since they evaluate to false.
Steps To Reproduce:
Try to evaluate a unique rule for a model or field that will evaluate to false - 0, empty string.

Please fill out the entire issue template.

Closing this issue because it's inactive, already solved, old or not relevant anymore. Feel free to reply if you're still experiencing this issue.

@driesvints Also happening here in 5.7. Is there any workaround for this or was this solved in any newer versions?


If an audit has both a red and yellow levels, there's no point in scoring on both

e.g. max words per sentence,

The code already checks for the yellow level only if the red level is not hit. You can check lines 93:98 in PageQualityScorerReadability.php


TTS feature

A low-prio idea: make them panels talk! Send some string via REST or MQTT and let the panel use its builtin TTS engine to notify us about whatever acoustically. :)

I had some thoughts about sounds in general -- either built in sounds for ease of use (eg, ring a gong or an alarm or something) and/or MPD protocol support for doing fancier stuff. I suppose TTS is a natural extension, there's probably some half decent local library to do that. If it's going to be pulling from Google/etc anyhow Id just suggest the MPD route

I don't know for sure, but I'm pretty confident, that TTS is directly baked into Android. Maybe not just a system.say("foo"), but also not much more complicated then that. MPD would go way beyond that.
My practical use case would be, that the motion sensor at the front door (inside the house) would trigger a warning to shut the kitchen window in case I would leave the house. Or something like "the laundry is done" when I pass the panel on the way to the basement.
From the HASS perspective this would just be a REST notify, which the panel then would make audible.

How to play a text other than English?
In the phone, Google TTS is configured, the language in the default TTS settings is Russian, the language pack is loaded, the required voice is selected, but if I send a REST command with the Russian text, the phone is silent, if with English it pronounces, if mixed, it pronounces only English phrases.

I was also wondering how to change the tts language. Can we have language parameter along with "speak" in the message ?


Document SDLC and contribution process for Pilot

Several of the participating pilot banks have asked for a diagram and/or technical documentation such that they could better understanding how the SDLC works, and in particular how contributions and actual underlying git commits work into the hosted GitLab instance beneath the interactive layer of Alloy Studio. One motivation for needing these is to give to legal and compliance folks in these banks who are assessing risk and evaluating ability of contributors to use Alloy.

Diagram was created and shared with all interested parties.


Merge math namespaces

Merge:

stan::math
stan::agrad
stan::prob

into one namespace:

stan::math

I think this is a great idea.  It will reduce the number of using statements in functions and provide simpler argument-dependent lookup, as well as being simpler overall for clients who won't have to remember which function came from which namespace.
The goal is to move all of stan::math into its own standalone library that's used by the rest of Stan but also easily used by other projects.


Add support for Travis

This adds a .travis.yml. Go to https://travis-ci.org/ndmitchell/fsatrace/builds/63342283 to see the output of a run - which shows fsatrace works on Linux. You probably want to extend it to check that the output matches something you expect.

Great, activated it. Thanks!


feat(activity): improve search

Description:
Though the activity search was improved recently we can still do better.
The PR adds the following improvements:

Makes search case-insensitive
Allows search by memo field
Allows search by type field (invoice, payment etc)
Allows search by date
Delivers perf optimization of the search process

A little more on search by date.
Currently, we don't have an ability to filter activity by date range, which is quite a common use case.
This PR instead adds search by date.
The current implementation has some limitations though. The biggest one is that it currently has no intl support because search is performed against a raw dataset and date localization happens at the UI level. This can be addressed by future PR since it requires some more refactoring.

How Has This Been Tested?
Manually
For instance type may or 2019 to search by date

Types of changes:
Enhancement

Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage increased (+0.02%) to 20.197% when pulling 642cf5ed51980089f96af06f5d9670175de65ab3 on korhaliv:fix/improve-activity-search into 2be48d9e9acee761301e2412c9d3196538d94085 on LN-Zap:next.


uninitialized constant Syck (NameError)

Running into the following error with Syck in Gemfile&installed.
Using rvm and ruby 2.2.0.
Please advice.
.rvm/gems/ruby-2.2.0/gems/safe_yaml-0.9.7/lib/safe_yaml/syck_node_monkeypatch.rb:42:in `<top (required)>': uninitialized constant Syck (NameError)

Looks like an issue with a transitive dependency. Have use tried with a
2.1.x version?
Ruby is pretty bad when it comes to dependency conflicts and version
upgrades
On 10 Apr 2015 22:34, "Gregory Reshetniak"<EMAIL_ADDRESS>wrote:

Running into the following error with Syck in Gemfile&installed.
Using rvm and ruby 2.2.0.
Please advice.
.rvm/gems/ruby-2.2.0/gems/safe_yaml-0.9.7/lib/safe_yaml/syck_node_monkeypatch.rb:42:in `<top (required)>': uninitialized constant Syck (NameError)
—
Reply to this email directly or view it on GitHub
https://github.com/portertech/kitchen-docker/issues/117.

I have exactly the same problem with ruby 2.2.2, but no problems with ruby 2.1.6

Ah found out what the problem is, see http://ruby-doc.org/stdlib-2.2.2/libdoc/yaml/rdoc/YAML.html.
Syck was the original for YAML implementation in Ruby’s standard library developed by why the lucky stiff.

You can still use Syck, if you prefer, for parsing and emitting YAML, but you must install the ‘syck’ gem now in order to use it.

In older Ruby versions, ie. <= 1.9, Syck is still provided, however it was completely removed with the release of Ruby 2.0.0.

So you now have to install the Syck gem

I am able to run with 2.2.2. Looking at the stack trace, it seems that
there is a gem file on your system that is not compatible with Test
Kitchen. Maybe run with bundler to ensure the correct gem versions are used.
bundle install --path vendor/bundle
bundle exec kitchen test

This repo for some reason has an excludes on the Gemfile.lock which should
really be commited to lock the versions of any gem dependencies, and then
this avoids the situation where other gems already installed cause a
failure.
On 19 April 2015 at 08:43, Frank Oxener<EMAIL_ADDRESS>wrote:

I have exactly the same problem with ruby 2.2.2, but no problems with ruby
2.1.6
—
Reply to this email directly or view it on GitHub
https://github.com/portertech/kitchen-docker/issues/117#issuecomment-94199542
.

After some extra testing, if you use the Gemfile to load/install the required dependencies, the syck gem gets pulled in automatically, instead of relying on what is or is not on your existing GEM_PATH location.

+1 with ruby 2.2.1 using bundler

Closing issue due to inactivity. Please open a new issue if there is still a problem. Thanks!


Decommission Azure Storage static website

The move to Azure Static Webapps means the old blob store is no longer requires.

Dropped


Change the default security configuration file to shiro.ini

Motivation:
We forgot to update the default security configuration file name from
'security.ini' to 'shiro.ini'. As you find in the 'conf' directory of
the distribution, the example files have been renamed to 'shiro.example.*.ini'
already long time ago.
Modificiation:

Change the default security configuration file name from security.ini
to shiro.ini

Result:

Consistency

Codecov Report

Merging #97 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #97      +/-   ##
==========================================
+ Coverage   57.02%   57.07%   +0.05%
==========================================
  Files         233      233
  Lines        7998     7998
  Branches      934      934
==========================================
+ Hits         4561     4565       +4
+ Misses       2849     2847       -2
+ Partials      588      586       -2

Impacted Files
Coverage Δ

...in/java/com/linecorp/centraldogma/server/Main.java
0% <ø> (ø)
:arrow_up:

...internal/storage/repository/git/GitRepository.java
76.23% <0%> (+0.27%)
:arrow_up:

...internal/replication/ZooKeeperCommandExecutor.java
67.56% <0%> (+0.67%)
:arrow_up:

Continue to review full report at Codecov.

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


Bazel build failure on AWS Ubuntu

I have a failure when running
bazel build --config=opt :install
on AWS EC2 Ubuntu
ERROR: /home/ubuntu/sonnet/sonnet/cc/kernels/BUILD:19:1: C++ compilation of rule '//sonnet/cc/kernels:resampler_op' failed: gcc failed: error executing command /usr/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -B/usr/bin -B/usr/bin -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer '-std=c++0x' -MD -MF ... (remaining 47 argument(s) skipped): com.google.devtools.build.lib.shell.BadExitStatusException: Process exited with status 1.
sonnet/cc/kernels/resampler_op.cc: In instantiation of 'deepmind::tensorflow::sonnet::functor::ResamplerGrad2DFunctor<Eigen::ThreadPoolDevice, T>::operator()(tensorflow::OpKernelContext*, const CPUDevice&, const T*, const T*, const T*, T*, T*, int, int, int, int, int)::<lambda(int, int)>::<lambda(int, int, int, T)> [with T = double]':
sonnet/cc/kernels/resampler_op.cc:269:23:   required from 'struct deepmind::tensorflow::sonnet::functor::ResamplerGrad2DFunctor<Eigen::ThreadPoolDevice, T>::operator()(tensorflow::OpKernelContext*, const CPUDevice&, const T*, const T*, const T*, T*, T*, int, int, int, int, int)::<lambda(int, int)> [with T = double]::<lambda(int, int, int, double)>'
sonnet/cc/kernels/resampler_op.cc:272:9:   required from 'deepmind::tensorflow::sonnet::functor::ResamplerGrad2DFunctor<Eigen::ThreadPoolDevice, T>::operator()(tensorflow::OpKernelContext*, const CPUDevice&, const T*, const T*, const T*, T*, T*, int, int, int, int, int)::<lambda(int, int)> [with T = double]'
sonnet/cc/kernels/resampler_op.cc:317:38:   required from 'struct deepmind::tensorflow::sonnet::functor::ResamplerGrad2DFunctor<Eigen::ThreadPoolDevice, T>::operator()(tensorflow::OpKernelContext*, const CPUDevice&, const T*, const T*, const T*, T*, T*, int, int, int, int, int) [with T = double; deepmind::tensorflow::sonnet::CPUDevice = Eigen::ThreadPoolDevice]::<lambda(int, int)>'
sonnet/cc/kernels/resampler_op.cc:338:5:   required from 'void deepmind::tensorflow::sonnet::functor::ResamplerGrad2DFunctor<Eigen::ThreadPoolDevice, T>::operator()(tensorflow::OpKernelContext*, const CPUDevice&, const T*, const T*, const T*, T*, T*, int, int, int, int, int) [with T = double; deepmind::tensorflow::sonnet::CPUDevice = Eigen::ThreadPoolDevice]'
sonnet/cc/kernels/resampler_op.cc:407:51:   required from 'void deepmind::tensorflow::sonnet::ResamplerGradOp<Device, T>::Compute(tensorflow::OpKernelContext*) [with Device = Eigen::ThreadPoolDevice; T = double]'
sonnet/cc/kernels/resampler_op.cc:443:1:   required from here
sonnet/cc/kernels/resampler_op.cc:239:47: internal compiler error: in tsubst_copy, at cp/pt.c:13039
const int data_batch_stride = data_height * data_width * data_channels;
^
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-5/README.Bugs> for instructions.
Target //:install failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 46.720s, Critical Path: 45.77s

Same here on Ubuntu 16.10.

same on Ubuntu 14.04 TLS

same here. Ubuntu 16.04 LTS

Builds on local ubuntu 16.04 LTS after applying https://github.com/deepmind/sonnet/pull/6

@normanheckscher problem solved, thx

@normanheckscher thanks for the fix - we'll get it in ASAP, and I'll close this issue once the changes are pushed back out to GitHub.

@normanheckscher @leesunfreshing @johnnogent @cgarciae  We have pushed a fix, please can you try and build again?

@malcolmreynolds It builds, I just can't import it. I'll try and look a little further into it sometime over the next few days when I get more time.
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import sonnet as snt
Traceback (most recent call last):
File "", line 1, in
File "/home/norman/.virtualenvs/tf/local/lib/python2.7/site-packages/sonnet/init.py", line 102, in
from sonnet.python.ops.resampler import resampler
File "/home/norman/.virtualenvs/tf/local/lib/python2.7/site-packages/sonnet/python/ops/resampler.py", line 33, in
tf.resource_loader.get_path_to_datafile("_resampler.so"))
File "/home/norman/.virtualenvs/tf/local/lib/python2.7/site-packages/tensorflow/python/framework/load_library.py", line 64, in load_op_library
None, None, error_msg, error_code)
tensorflow.python.framework.errors_impl.NotFoundError: /home/norman/.virtualenvs/tf/local/lib/python2.7/site-packages/sonnet/python/ops/_resampler.so: undefined symbol: ZN10tensorflow15shape_inference16InferenceContext15WithRankAtLeastENS0_11ShapeHandleEiPS2

Managed to get it to work.
My separately installed tensorflow version (outside the sonnet directory) was originally form the head and a version of 1.1.0rc2. I changed this tensorflow version with git, recompiled and installed the resultant tensorflow-1.1.0rc1-cp27-cp27mu-linux_x86_64.whl. After this, sonnet build, installs and runs as expected.
cd ~/workspace/tensorflow
git checkout r1.1
./configure
bazel build -c opt --copt=-mavx --copt=-mfpmath=both --copt=-msse4.2 --copt=-msse4.1 --config=cuda -k //tensorflow/tools/pip_package:build_pip_package --verbose_failures
bazel-bin/tensorflow/tools/pip_package/build_pip_package /tmp//tensorflow_pkg/
pip install --upgrade /tmp/tensorflow_pkg/tensorflow-1.1.0rc1-cp27-cp27mu-linux_x86_64.whl
cd ~/workspace
git clone --recursive https://github.com/deepmind/sonnet
etc etc etc from the sonnet instructions

I have the same error as @normanheckscher ("undefined symbol: ZN10tensorflow...") but my tensorflow is simply 1.0.1 from pip and I'm using Python3. So whatever @normanheckscher thinks was the problem, can't be the problem.

@thomkeh I managed to get it working by being very particular with my tensorflow install. I'm guessing that the issue lies somewhere around interoperability with Tensorflow as I've played with differing versions of TensorFlow in both the sonnet directory and in my workstation environment (using virtualenv).
After building and installing an older version of Tensorflow I have been able to get Sonnet running. I'd suggest using git to build tensorflow @ 36a47f2, as this is the version which Sonnet is linked against. I'm currently using it to bang out multi-cpu models with mnist... variable success (pun intended)... so yeah, it does work.
Is python 3 supported?

@normanheckscher Ok, I see. Thanks for the response! I makes sense that you should use the tensorflow version that this repository links against.
Python3 is not officially supported but #10 made it sound like compatibility is just some small code changes away.

	sudo apt-get dist-upgrade

fixed it for me on ubuntu 14.04 LTS
i found the bazel error fix elsewhere and it was stated it was an earlier version kernel issue

@thomkeh python 3 is supported now :)
Regarding the compilation issues, marking this as duplicate of #25 to follow progress there.


Fe friends followers leah

FE for friends/ followers.
IMPORTANT:

in find friends page, you may see people who don't follow you and you may follow them (send them a friend request)
in followers page, you may see a list of people friend requesting you  (following you) and follow them back. if you are already friends with this follower, this will be indicated underneath the followers name in the followers page. you have already accepted their request.
in friends page, you may see a list of people who are your friends, and you may remove them as a friend. This will remove them as a friend from you, you as a friend from them. you as a follow of them, and them as a follower of you. lol. once you go back to find friends you may re follow (friend request) this user

ISSUE (UI only issue (does not impact user stories, but would be dumb irl)):
-currently cant get a list of pending friends request unless you make an ajax call to every non followers /follow list (for everyone on the site who is not following you)- not doing this yet - too many calls. you may follow a non follower as many times as you want, your friend request is sent the first time. hit that mfn follow button as many times as u want. nothing bad will happen... probably. maybe
-ISSUE (for all fe):
-back end sends you important json info whether or not the status code is 2xx, 3xx, 4xx. AJAX does not like it. they have their own responses for 404s and you will not get this important json info from API when returning an ajax promise. heads up
-WORKS WELL (for all ajax):
-if you want to do something only after promise is fulfilled
function whatever(){
      foo(url).done(function(response) {
           otherfunctionthatneedsresponsedata(response)
        });
 }

 function foo(url){
        return $.ajax({
            url: url,
            type: 'DELETE'});
    }

Codecov Report

Merging #89 (5ab3d40) into dev (8dabf65) will decrease coverage by 1.24%.
The diff coverage is 14.70%.

@@            Coverage Diff             @@
##              dev      #89      +/-   ##
==========================================
- Coverage   62.45%   61.21%   -1.25%
==========================================
  Files          15       15
  Lines         919      941      +22
==========================================
+ Hits          574      576       +2
- Misses        345      365      +20

Impacted Files
Coverage Δ

citrus_network/citrus_home/urls.py
100.00% <ø> (ø)

citrus_network/citrus_home/views.py
42.88% <14.70%> (-1.28%)
:arrow_down:

Continue to review full report at Codecov.

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


Fix CMSIS import with Zephyr 3.4+

Fixes the import path for CMSIS in Zephyr versions > 3.4.
See https://github.com/zephyrproject-rtos/zephyr/pull/59855 for more information.

I'm hitting this error still.  If I explicitly checkout 314ecba, it works.  However, if I checkout main I hit the above error.  According to the details above, this was merged 4 days ago.  I've confirmed that I have the latest changes pulled down so I'm not sure what's happening here.

@srberard it could be related to https://github.com/bytecodealliance/wasm-micro-runtime/issues/2760, which I am working on fixing 👍🏻

@srberard @hasheddan please refer to #2777


ApplePay Required field 'merchantIdentifier' is null

I've setup the playground on a server with letsencrypt ssl certificate and I'm trying to make a payment via Apple Pay.
Unfortunately on a request via POST to https://checkoutshopper-test.adyen.com/checkoutshopper/v1/applePay/sessions?clientKey=test_CEPL7TOKNNDJXP2X6WPWONFQTEE76ZHF
I'm getting 500 status code with following response payload.
{
    "status": 500,
    "errorCode": "702",
    "message": "Required field 'merchantIdentifier' is null",
    "errorType": "validation"
}

Hey @tarach .
Did you reach out to the Adyen support team? It seems to me that something can be misconfigured with your Merchant account. They might be able to help you out.

Hey @ribeiroguilherme,
Yes I did reach out to support team. It is definitely likely something is misconfigured but:
Drop-in form does work ie. it does make a successful POST /applePay/sessions request
But the Component implementation via Wallets section does not.
Feel free to check it yourself:
https://adyendemo.tarach.net/
Basic Auth:
- user: adyen
- pass: 123456


Not mobile friendly: https://angular.io/docs/ts/latest/guide/ngmodule.html

The page https://angular.io/docs/ts/latest/guide/ngmodule.html is not mobile friendly:

It should be like the other pages (e.g. https://angular.io/docs/ts/latest/guide/animations.html):

Each page that contains images or other kind of things that are not text, will get that effect.
Ping @naomiblack @ericjim

Closing as we are migrating to a new design. I will make sure this won't happen.


Replace docker-proxy with Traefik (for ingress)

When using docker-proxy external ips can otherwise not be seen, which interferes with rate limiting.

hey there, any news on this one? I'm not sure, but traefik should be able to "proxy" all tcp traffic, so maybe the SSL related difficulties can be fixed this way too..

Hi @theoneandonly-vector,
due to other projects I have not yet done anything in this matter. In the end it should be more about providing documentation, than actual code changes.
If you want to work on that let me know. I'll be glad to review your docs/changes.
A while ago @cguentherTUChemnitz has already provided https://github.com/zokradonh/kopano-docker/blob/master/examples/traefik-proxy-subdomain.md, but that howto is more in the matter of replacing/augmenting the current "web" container, than using Traefik as a full blown ingress mechanism.

as traefik isn't compatible with imap and smtp, this isn't going to happen. when I find a truly open-source load-balancer/reverse-proxy with imap/SMTP-compatibility and Lets-Encrypt integration which already offers a docker-image, I'm gonna do it.

as traefik isn't compatible with imap and smtp, this isn't going to happen. when I find a truly open-source load-balancer/reverse-proxy with imap/SMTP-compatibility and Lets-Encrypt integration which already offers a docker-image, I'm gonna do it.

@theoneandonly-vector
Imho traefik should be able to handle such traffic very fine. Especially with the release of traefik >2.0  they are able to handle arbitrary tcp streams as a router configuration. https://docs.traefik.io/routing/routers/
Other guys are to place mailu smtp mailsever behind traefik when it stil was in 2.0 alpha stage and they got fixes to handle smtp stuff on their tcp routers. https://github.com/containous/traefik/issues/4875
Since traefik now is in 2.1 release i would assume that those fixes and merges are already present in the current releases.

AFAICT it should be possible to add a docker-compose.traefik.yml with the labels for traefik and completely replace caddy. We can do that in parallel with the caddy proxy, see how far we get and then decide how to proceed.


[Feature Request]: Add Microsoft.RedHatOpenShift module

Description
Add ARO module.

This appears relatively straight forward (needs VNET with 2 subnets, RG, etc.) with one exception - it needs a service principal & secret. I guess for that we could use the GitHub/ADO deployment principal? This may however be tricky regarding the secret.
Alternatively, we could have a deploymentScript create a service principal - but would then not only need custom logic to delete, but also the principal (managed identity) would need the permissions to create AAD objects in the first place.
@eriqua, any thoughts on this?

Just trowing some comments around:
Option 1 could become trickier once we'll move to openID connect authentication (ref #1450) since at that point we'd remove the SP clientID from the secret list.
An additional alternative to option 2, maybe a mid-step until better solution, could also be to still require some long-living dependencies for specific modules needing them (ARO module being the first one so far). In that case a dedicated SP and a KV where to store its clientID and secret to be referenced by the module tests.

@AlexanderSehr @eriqua - Thoughts on this:

Bicep PG has been talking about the ability for AAD Extensibility (This should provide us with a way of generating Service Principals and Permissions natively) maybe this is something that could help us here

Although it is optional may customers are using with the a pull secret, this is a particular secret tied to redhat container registries, we would need a way to create, validate and then remove the registry

@AlexanderSehr @ChrisSidebotham @matebarabas what about porting this to AVM? If so, I'd avoid to just move it as is as this should rather be incorporated into the module proposal template.
In either case, I'd vouch for not losing this issue.


Basic accented character support

Hello,
Please add some characters like éèàÉÈÀ as they are usually used in European countries.

http://practicaltypography.com/common-accented-characters.html
I'm going to add this set of characters, if any more are needed please drop a comment here.


Not Optimized for timezones

The time of pairs is shown according to the user's timezone, but KPI is located in Kyiv, so if the user is located a few hours ahead, the system gets confused and shows the wrong pair.

Moved to Jira: https://kpiua.atlassian.net/browse/KB-12


Docker

Hi - thanks for what looks like a really useful project.
I wonder if you would consider publishing the command line tool as a docker image to docker hub?
This way it would mean the tool could be used by anything that supports docker.
Many thanks

Hello. Thank you for your interest in a project that is close to my heart.
I was just thinking about it for similar needs.
Le lun. 6 nov. 2023 à 23:10, Darrell @.***> a écrit :

Hi - thanks for what looks like a really useful project.
I wonder if you would consider publishing the command line tool as a
docker image to docker hub?
This way it would mean the tool could be used by anything that supports
docker.
Many thanks
—
Reply to this email directly, view it on GitHub
https://github.com/Black-Beard-Sdk/jslt/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA25DSNHUHSF46WULTLRXILYDFN55AVCNFSM6AAAAAA7AGVX2KVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE4DAMJVGY2DENI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Cordialement,
Gaël Beard
06 60 25 18 53


bug: resolveGranularity should be considered in upstream id generation

Because "resolveGranularity" is defined in ApisixRoute and upstream id generation does not take it into account, multiple routes with the same backend but different resolveGranulartiy will conflict and overwrite the others (the last changed route will win).

If we want to fix this problem, we must design to the regeneration of upstream_id.
This will be a huge breaking change.
When upgrading, users need to clear the original data and generate new data

#1251 has been merged. We can close this one


Enable data entry on a previously historical park in A&R

This ticket is to enable data entry on a sub-area for Garden Bat Marine Park. This is not a net new park, but we’d like to make it available to enter data in again using the same process as we did for the 4 Skeena parks that were added back in. Region would also like to rename the sub area.

Acceptance Criteria:

shelkem Trail is new name of Garden Bat Marine sub-area with activity of Day Use, and data can be entered for this sub-area

Review #250 for more information.

Completed in PROD only. New Subarea id = 1017

Tested on PROD: Passed

Verified that this change has been done on the PROD environment.

moving to done


Cache Retrieval Error in Workflow: Maximum Recursion Depth Exceeded

Dear Farmvibes Team,
Rerunning the workflow fails with an error. Failed to obtain cache information for op list_sentinel2_products_pc with exception 'RecursionError'>: maximum recursion depth exceeded in instancecheck.
I tried restarting the clustor (farmvibes-ai local restart) and destroying the old clustor
and creating a new one (farmvibes-ai local setup), but the problem persists.
Attached to this issue, you will find the following files for your reference:

Log file for workflow failure

Log files for each of the pods involved in the process:
terravibes-worker
terravibes-orchestrator
terravibes-rest-api

Workflow_Error_Logs_Files.zip

Hi, @click2cloud-sagarB. Thank you for raising this issue.
We saw it internally and are currently investigating possible causes and solutions. This seems to happen more frequently when running a lot workflows in parallel.
A temporary workaround is to delete the cache pod with:
~/.config/terravibes/kubectl delete pod -l app=terravibes-cache

and re-run the desired workflows.
I'll keep this issue updated as we progress on our investigations.

Hi,
Is there an update on a fix for this issue? Currently experiencing it.

We are planning a release for this week with a fix for this issue, @shenoy10.

The issue was caused by pydantic raising a RecursionError when trying to (de)serialize one of our dataclasses. The new release should have fixed this issue.
We are closing this issue for now, but @click2cloud-sagarB and @shenoy10, let us know if you still face any issue.


Edit hdinsight-hadoop-access-yarn-app-logs.md

Edit complete.
On line 53, I changed "this information will be stored by the Resource Manager" to "this information will be stored by Azure Resource Manager." Please confirm that this is correct. If "Resource Manager" was meant to refer to the YARN name, it should be "ResourceManager." (See next comment.)
In the context of YARN names, I changed "Resource Manager" to "ResourceManager," "Node Manager" to "NodeManager," and "Application Master" to "ApplicationMaster" because those seem to be the official spellings (see http://hadoop.apache.org/docs/current/hadoop-yarn/hadoop-yarn-site/YARN.html). If you think those spellings are dubious, please change them back.
After first mention of "ResourceManager," I changed all mentions of "Resource Manager" to "RM." Please confirm that these mentions were meant to refer to the YARN ResourceManager and not to the Azure Resource Manager feature.

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

It looks like you're working at Microsoft (v-shawja). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Potential performance improvement in YammerPrometheusMetricsReporter

Currently during collect(), we iterate through all the metrics from the Yammer registries to apply the allow list filter.
It's possible to register listeners onto a Yammer registry. We should check if we could use this pattern and apply the filtering as metrics are added or removed instead of in collect(). This is similar to what's described in https://github.com/strimzi/metrics-reporter/issues/9 for KafkaPrometheusMetricsReporter.

@mimaison
This is my understanding of It's possible to register listeners onto a Yammer registry:
The Yammer metrics library supports adding listeners that can react to changes or events related to the metrics being collected and managed by the registry. MetricsRegistry is a central place where metrics like counters, gauges etc. are registered, allowing you to manage and query the metrics efficiently. Listeners can be attached to the registry to listen for specific events such as a new metric, and the listener can handle the event. This can be done when a metric is added, removed, or updated rather than when it is collected, as the current logic is.
Currently it is handled here:
            for (Map.Entry<MetricName, Metric> entry : registry.allMetrics().entrySet()) {
                System.out.println(entry.getKey() + ": key");
                System.out.println(entry.getValue() + ": value");
                MetricName metricName = entry.getKey();
                Metric metric = entry.getValue();
                // metricName is the key and metric is the value
                LOG.trace("Collecting Yammer metric {}", metricName);
                //example: kafka_server_other_type_name: prometheusMetricName
                String prometheusMetricName = metricName(metricName);
                // TODO Filtering should take labels into account
                if (!config.isAllowed(prometheusMetricName)) {
                    LOG.info("Yammer metric {} is not allowed", prometheusMetricName);
                    // skip prometheusMetricName if it is not allowed after LOG.info
                    continue;
                }
                LOG.info("Yammer metric {} is allowed", prometheusMetricName);
                Labels labels = labelsFromScope(metricName.getScope(), prometheusMetricName);
                // example {k1="v1",k2="v2"}: labels);
                LOG.info("labels {}", labels);

Is that a correct assumption are is there something I'm missing? Thanks

This is the relevant lines if the KafkaMetriceReporter??
https://github.com/strimzi/metrics-reporter/blob/main/src/main/java/io/strimzi/kafka/metrics/KafkaPrometheusMetricsReporter.java#L47C1-L59C6

Is that a correct assumption are is there something I'm missing? Thanks
Yes that's a good summary.

These are the relevant lines if the KafkaMetriceReporter??
https://github.com/strimzi/metrics-reporter/blob/main/src/main/java/io/strimzi/kafka/metrics/KafkaPrometheusMetricsReporter.java#L47C1-L59C6

This ticket is about YammerPrometheusMetricsReporter, so I'm not sure why you're mentioning KafkaPrometheusMetricsReporter. Why do you think these lines are relevant to this ticket?

You referenced this abaove This is similar to what's described in https://github.com/strimzi/metrics-reporter/issues/9 for KafkaPrometheusMetricsReporter.

We use import com.yammer.metrics.core.MetricsRegistryListener; ??

yes x 3

Yes MetricsRegistryListener is the class to use but I'm really confused by the code you shared. I'm not sure I follow what you are trying to do.


feat: add reusable github action to create issues

Partially-closes: #848

Summary
The motivation for this change is to automatically create github issues when some failure happens on any github action workflow of the sigstore community.
This action accepts an ISSUE_REPOSITORY as a parameter, and allows to create github issues based on github env vars.
This can be attached to any github workflow, to automatically create issues when there are failures

Release Note
NONE
Documentation
None, it is part of the CI

https://github.com/sigstore/root-signing/blob/a1a98ea8e80da9b325a137b0becdb6f2da47e3b7/.github/workflows/reuseable-snapshot-timestamp.yml#L139-L167

sure, let me take a look

https://github.com/sigstore/root-signing/blob/a1a98ea8e80da9b325a137b0becdb6f2da47e3b7/.github/workflows/reuseable-snapshot-timestamp.yml#L139-L167

sure, let me take a look

I modified the action to accept a parameter that is an issue type. It can be used to open issues on failure, and close them on success, following the workflow you highlighted. I tested the actions with act, but anyway the best check would be to modify the github actions replacing the old workflow to the new one, that can be done after merging this code.


Nonautoregressive-Transformer performs poor

Thank you for publishing the code of NAT. I tried to reproduce the results of vanilla NAT following the instruction.  However, BLEU on test set (newstest2014) is about 7 BLEU, while it's about 18 BLEU on development set (newstest2013).
I have two following questions about training NAT:

why does a large margin between Fairseq's implementation(about 7 BLEU) and vanilla NAT(17.35 BLEU) exist? Fairseq's implementation should obtain better performance due to bigger batch size and better distilled dataset generated by transformer with 27.2 BLEU, compared with vanilla NAT.
The trained NAT performs well on development set, but why does it has a drastic decrease by 11 BLEU on test set?

Hi,
So sorry for replying late.

The fairseq implementation does not use "fertility" as the latent variables in the middle, so it will get a lower BLEU score; the original paper relies on knowledge distillation. We also provided one distillation dataset, you can train on that and it will give you around 18~19 BLEU scores.
Yes, there is a big difference for the valid and test set in terms of BLEU scores. It is a dataset wise difference and absolute BLEU scores are not really comparable.

Thanks for your reply.

All above experiments were conducted on your provided distillation dataset (http://dl.fbaipublicfiles.com/nat/distill_dataset.zip).  I also trained another vanilla NAT using above training script, but obtaining similar results (about 7 BLEU).
Does you use the same training script as above, in order to obtain 18~19 BLEU scores? Whether or not incorporate source embedding copying, layer-wise attention or positional attention?

I have solved above issues. With a large batch size about 64K, vanilla NAT on distillation dataset can achieve 18.6 BLEU.
Thanks for your help. @MultiPath

Hi, I met the same issue. What does "batch size about 64K" mean? Do you train on multiple GPUs and use "--max-tokens 64000"? Or train on one GPU and use "--max-tokens 8000", "--update-freq 8"? Could you provide your training script? Many thanks!

hi, @dmortem
''batch size about 64k'' means a total batch size of 64k tokens, and can use single GPU with  either max-tokens as 64k and update-freq 1 or  max-tokens 8k and update-freq as 8. In my case, I trained vanilla NAT using 8 GPU with max tokens as 8k.    Do we need --src-embedding-copy?

@hzhwcmhf NAT can obtain expected results (about 19 BLEU) without --src-embedding-copy

@xwgeng I tried both settings and get 18.92 BLEU without src-embedding-copy and 17.55 BLEU with src-embedding-copy.
It's really weird that the first NAT paper ("Non-Autoregressive Neural Machine Translation") mentioned that they have to copy the source embeddings or the model will be fail in training. The fairseq code authors also mention that they use copy the source embeddings in their paper ("Understanding Knowledge Distillation in Non-autoregressive Machine Translation").
Do you have any ideas about the problem?

@hzhwcmhf The reason may be that fairseq use a different and powerful NAT implementation from the original version (https://github.com/salesforce/nonauto-nmt) in model architecture and optimization strategies, leading to different results with src-embedding-copy. For examples, the original NAT must use layer-wise attention, positional attention etc. to obtain expected performance. Based on more powerful fairseq, incorporating source embedding may be not useful, as stated in https://github.com/pytorch/fairseq/issues/1458#issuecomment-562262959.

@xwgeng It is a good explaination. Thank you very much! Maybe the author should mention it in their README and code examples.

@MultiPath
Confirmed. I got very similar result after disabling left-pad (removing leftpad + src embedding copy v.s. without src embedding copy). However, it is a little different from your paper (especially from the first one). I think it can help a lot of people if you make it more clearer in https://github.com/pytorch/fairseq/blob/aa5f0119a383e013e56ae5d88e4a7aff0e67f0f9/examples/nonautoregressive_translation/scripts.md.

hi,@xwgeng @MultiPath
I tried to reproduce NAT on 300k small data,BLEU on teacher model is about 30  but BLEU on NAT is  around 7 . The script is as follows:
`fairseq-train
${databin}
--save-dir ${model_save}
--task translation_lev
--criterion nat_loss
--arch nonautoregressive_transformer
--noise full_mask
--share-all-embeddings
--optimizer adam --adam-betas '(0.9, 0.98)'
--lr 5e-4 --lr-scheduler inverse_sqrt
--stop-min-lr '1e-09' --warmup-updates 10000
--warmup-init-lr '1e-07' --label-smoothing 0.1
--dropout 0.3 --weight-decay 0.01
--decoder-learned-pos
--encoder-learned-pos
--pred-length-offset
--length-loss-factor 0.1
--apply-bert-init
--log-format  'simple' --log-interval 200
--max-tokens 4096
--save-interval-updates 10000
--max-update 300000
--keep-interval-updates 5 --keep-last-epochs 5
--fp16 --seed $seed
fairseq-generate ${data_dir}
-s zh -t en
--gen-subset test
--task translation_lev
--path ${model_save}/checkpoint_best.pt
--batch-size 400
--beam 1
--iter-decode-max-iter 0 --print-step
--remove-bpe`
I don't know if batchsize or save-interval-updates is appropriate. Can you give me some advice if the hyperparameters is something wrong ?


Date header in responses

Implement date header for all responses as defined here: https://www.rfc-editor.org/rfc/rfc9110#name-date

Closed 5a21e6b773ebd6f5a438772dfbb95dcb974e490f


fix licence in ./pkg/controllers/v1alpha1/goosefs/goosefs_runtime_controller.go

Ⅰ. Describe what this PR does
add licence in ./pkg/controllers/v1alpha1/goosefs/goosefs_runtime_controller.go
Ⅱ. Does this pull request fix one issue?
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

/lgtm

/lgtm
/approve


Workflow Details page makes unnecessary Archived request

Pre-requisites

[X] I have double-checked my configuration
[X] I can confirm the issue exists when I tested with :latest
[X] I have searched existing issues and could not find a match for this bug
[X] I'd like to contribute the fix myself (see contributing guide)

What happened/what did you expect to happen?
Users of Argo Workflows version ~3.5.5~ 3.5 experience an issue where the Workflow Details page attempts to fetch workflows as if they were archived, even when the archived workflows feature is not utilized. This behavior results in an unnecessary API call, leading to the error message getting archived workflows not supported and potentially degrading the UI experience.
The issue manifests before the workflow details page loads, with the UI erroneously trying to retrieve an archived version of the workflow based on its UID.
~Initial troubleshooting suggested setting the ARGO_ARTIFACT_SERVER environment variable to false as a potential fix. However, the error persisted on the workflow details page, and archived logs ceased to function after applying this workaround.~ edited by agilgur5: this is actually unrelated, see below comment.
Discussion on Slack highlighted a section of the frontend code responsible for this behavior, indicating that the UI checks for an archived workflow status unnecessarily. This check occurs even when archived workflows are not enabled, which should indicate that the workflow is not considered archived.
The conversation concluded with the suggestion to file a GitHub issue to track this problem, aiming for a more concise and efficient handling of workflow status checks in the UI to prevent unnecessary API calls and improve the user experience.
Potential Solutions:

Adjust the frontend code to ensure that archived workflow checks are only performed when necessary, based on the presence of appropriate configuration settings.

Version
v3.5.5
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
metadata:
  name: delightful-bear
  namespace: default
  labels:
    example: 'true'
spec:
  arguments:
    parameters:
      - name: message
        value: hello argo
  entrypoint: argosay
  templates:
    - name: argosay
      inputs:
        parameters:
          - name: message
            value: '{{workflow.parameters.message}}'
      container:
        name: main
        image: argoproj/argosay:v2
        command:
          - /argosay
        args:
          - echo
          - '{{inputs.parameters.message}}'
  ttlStrategy:
    secondsAfterCompletion: 300
  podGC:
    strategy: OnPodCompletion

Ensure workflow archives are disabled.
View the workflow details page. Look for the error message before the page loads.

For posterity, this was also mentioned in https://github.com/argoproj/argo-workflows/discussions/11671#discussioncomment-7071490, https://github.com/argoproj/argo-workflows/issues/12759#issuecomment-1986088753, and maybe others, but a separate issue for this hadn't been filed till now. This has existed since 3.5.0-rc1 and is due to the Archived/Live Workflow UI merge in 3.5 (i.e. #11121)

Adjust the frontend code to ensure that archived workflow checks are only performed when necessary, based on the presence of appropriate configuration settings.

Longer-term, I actually think of all of the front-end fallback calls on the Workflow Details page should be rewritten to run in the back-end. That would line up with the intent of #10781.
Also there's technically at least two problems here, one is the retrieval plus error message in the UI, the other is that the server error is a 500 (suggesting it was an unhandled error), as opposed to a 4xx or a different 5xx or something.
404 or 204 could perhaps makes sense. There is no status code for unsupported by the Server which would be most accurate though. Ironically, a 418 almost makes sense for an unsupported request.

 Maybe that could be a 218 status code.
Do you think it's worth implementing a workaround or wait for the long term refactor?
My only concern about not resolving this is receiving feedback from users of Argo (At our company) multiple times about the brief error every time they view a workflow detail.

multiple times about the brief error every time they view a workflow detail.

Yes I agree, and that is why I think it is worth having a shorter term fix. It would also be more correct. Then all of the Details calls could be refactored to run in the back-end later.
The one issue fixing it in the front-end is that it does not currently know of all settings such as Archived Workflows. So the fallback is necessary. But the ordering could be changed and the error could be caught and not shown to the user.

ordering could be changed and the error could be caught and not shown to the user.

Ack I will take peak at this.

Longer-term, I actually think of all of the front-end fallback calls on the Workflow Details page should be rewritten to run in the back-end. That would line up with the intent of #10781.

Ok apparently this already exists: https://github.com/argoproj/argo-workflows/pull/11055 was a precursor to #11121


x/tools/cmd/goimports: package clause insertion mangles comments

Input:
// a
// b
// c

func main() {
    fmt.Println()
}

gofmt fails appropriately:
$ gofmt x.go
x.go:5:1: expected 'package', found 'func'

goimports inserts a package clause and mangles the comments:
$ goimports x.go
package main // a
import "fmt"

// b
// c

func main() {
    fmt.Println()
}

The package main insertion is intentional. It was requested and done as a feature at Gophercon 1's hack day.
The comment mangling of course is not.

Is this the same issue or a different issue?
Given the following input:
package main

import (
	"encoding/binary"

	// Some comment.
	"fmt"
)

func main() {
	fmt.Println()
}

goimports will try to remove "encoding/binary" and result in the following invalid Go code:
package main

import

// Some comment.
"fmt"

func main() {
	fmt.Println()
}

If it's not the same issue, let me know, I'll delete this comment and open a new issue to track this.

Hard to say whether it is the same issue without looking into the fix. Good to have multiple test cases regardless.

Ok, I'll delete the comment and move this into a separate issue then. In case the same fix resolves both issues, great, otherwise they can be looked at separately.

Is anybody working on this issue yet? I'm looking for something to contribute and it seems like a good one to start. What's the expected output of the given example?

I think the expected output would be:
package main

import "fmt"

// a
// b
// c

func main() {
	fmt.Println()
}

Welcome! I don't think anyone is working on it.
I guess I'd expect the package and imports to be inserted at the beginning of the file, with a blank line between the imports and the first comment. But it's not obvious, so if there's a good reason to do otherwise, I'm open to it.
Messing with the AST and comment positioning can be quite fiddly, so I recommend writing all your test cases first. Feel free to ask for help if needed.
Note that the likely reviewers for this (Brad, Robert, me) are all going to be slow, for our own reasons.

Great! I'll start working on this issue then. There is a better channel to talk to you if I need help or should I just write it on this thread? I promise I won't bug you without exhausting all my options first :)

This is a good place to discuss


Pretrain NeRF

Hi!
I'd like to know if there is a simple way of loading (or training) a pretrained NeRF model. For example, if i want to start not with a random initialization but with a NeRF, that has been trained on a sphere, human etc.
Thank you!

@Pashtetickus Hi, for NeRF this can be a little hard as we assume known lighting conditions, where most reconstruction methods do not support. However, you can load a mesh into the second DMTet stage to initialize the SDF grid, which is used in methods like Fantasia3D (You'll still need to modify some code to achieve this)

Hi!
I think my question relates to this issue as well.
Is it possible to train Nerf in stable dream-fusion as in https://github.com/ashawkey/torch-ngp with data like fox dataset for example?


fix: update marketplace related badges

We need to trigger another release. Good to not do super dummy release. We actually have badges broken in readme and they do not look nice in the marketplace, so good reason to trigger release
Fun fact. After fix they show latest release is 0.4.1 😄

Related to https://github.com/asyncapi/vs-asyncapi-preview/issues/146

/rtm


license: add ASF header.

NOTE: Please read the Contributing.md guidelines before submitting your patch:
https://github.com/iresty/apisix/blob/master/Contributing.md#how-to-add-a-new-feature-or-change-an-existing-one
Summary

add ASF header for all source code files.
removed copyright info.
add tool(https://github.com/apache/openwhisk-utilities) in travis CI to check ASF header.

Full changelog

[Implement ...]
[Add related tests]
...

Issues resolved
Fix #XXX

do we need to add a ignore file list?

do we need to add a ignore file list?

sure, here is the exclude fiel list: https://github.com/iresty/apisix/pull/743/files#diff-fddd45f9d2486fc66aae183c85faeadcR81

@moonming https://travis-ci.org/iresty/apisix/builds/603466982#L696
please rebase your branch.

@moonming travis still fail
.travis/openwhisk-utilities/scancode/scanCode.py --config .travis/ASF-Release.cfg ./
Reading configuration file [.travis/ASF-Release.cfg]...
Scanning files starting at [./]...
Scan detected 1 error(s) in 1 file(s):
  [./dashboard/index.html]:
       1: file does not include required license header.

index.html

So this file is using an incorrect ASF header?
https://github.com/iresty/apisix_dashboard_built/blob/fde8c24f536659e33e899d4adc3d6b48d658a6a2/index.html

index.html

So this file is using an incorrect ASF header?
https://github.com/iresty/apisix_dashboard_built/blob/fde8c24f536659e33e899d4adc3d6b48d658a6a2/index.html

can you change to this style: https://github.com/apache/incubator-apisix/pull/743/files#diff-1fe3647ec3125ba1bcdbd7c830ada01d? thx

@moonming please rebase your branch.


Routing issue

Enter the https://demo8cqube.tibilprojects.com/#/courses in any browser.
Issue : routing is not happening to login screen

Retested not fixed

Working as expected


Spring Cloud Hoxton SR5: Route ordering not maintained even with YML configuration

Describe the bug
Prior to Hoxton SR5, routes configured in application.yml were evaluated in the order defined in the configuration file.
With SR5, the order doesn't seem to be honored causing legacy routes to get picked up before specific routes.
Sample
Below is a sample output of routing matching for the same route in SR4 and SR5.
  # Ex.


Test suite only runs from a git repo

If you attempt to run the test suite from e.g. the gem (where there's no local .git directory), the test suite throws a bunch of errors:
fatal: Not a git repository (or any parent up to mount point /home)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
Coveralls git error:
undefined method `gsub' for nil:NilClass

Running it from a git clone works as expected.

@orlitzky are you running locally or from a CI?

Locally. Here's my lazy way to reproduce:
$ git clone https://github.com/lemurheavy/coveralls-ruby.git
$ cd coveralls-ruby
$ rm -rf .git
$ rake

@orlitzky we have plans in the future to support arbitrary coverage data (sans git or github) but currently each build needs to be associated with a commit as though you were on a CI.


docs: add Mechanical API link to Mechanical Scripting page

Closes #970

@klmcadams - what's the status here?

@RobPasMue does this look okay?

Works for me - thanks @klmcadams!


Remove GRPC Proxy for v3 Log Rest API Call

This reduces the need for multiple translation done by proxy and improves the performance.

Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:

[ ] Has Docs included if any changes are user facing
[ ] Has Tests included if any functionality added or changed
[ ] Tested your changes locally (if this is a code change)
[ ] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including functionality, content, code)
[ ] Has a kind label. You can add a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user-facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contain the string "action required" if the change requires additional action from users switching to the new release

Release Notes

/test pull-tekton-results-unit-tests

/test pull-tekton-results-unit-tests

/assign @vdemeester @enarha


[FIX,TIR] Handle allocations of variable size

Variable size allocations like T.allocate([10 * i]) were incorrectly
moved up to where they were outside the scope of any variables in their
size. For example:
for i in range(8):
    T.allocate([10 * i])
    ...

became
T.allocate([10 * i])
for i in range(8):
    ...

Now variable sized allocations will use the constant upper bound on
their size. With the above example:
T.allocate([80])
for i in range(8):
    ...

Variable size allocations occur because earlier passes move constant
size allocations to where the are accessed and update the allocation
size to match the access size.
@Lunderberg @masahi

Would you guys like to check this PR and see if it's relevant?

#12019 seems relevant but does not fix the issue. It unclear what invariants CompactBufferRegion is supposed to maintain. The problem I hit is that StorageRewrite is getting variable size allocations. If CompactBufferRegion is not supposed to generate variable size allocations, then we need to fix it. Otherwise, this fix to StorageRewrite seems to be the correct path forward.

When variable bound occurs, having an upper bound or rejecting it ealy(assuming we don't want variable bound in compaction) are likely two approaches. In rare case where we can do dyn shared mem allocation we might want to keep alloca as it is without lifting(if it is only dependent on a thread level variable).
Likely there is no one-shoe that fits all in here and perhaps we can get a few more details. In most of the current usecases I am aware of we might want to lean toward rejection, or fix some earlier part of the pipeline(e.g. CompactBufferRegion) due to analysis problems.

Variable size allocations occur because earlier passes move constant size allocations to where the are accessed and update the allocation size to match the access size.

This remark is interesting and it would be great to list out a TVMScript example that it occur that motivates the change, just to see if we can reval problems in our earlier part of the pipeline where we are supposed to fix.

I'm glad that #12019 fixes this issue that @tkonolige mentioned! so I believe it has unblocked you guys already.
Indeed StorageRewrite could be buggy when encountering dynamic-shaped buffer allocation - I noticed the same thing personally - and we should get it fixed. It's good that we started discussing about it, on the other hand, we should also find the most proper fix in this case.
Let me share some context first: on GPUs, intermediate buffers are usually constant-sized - even for dynamic shared memory, it always has an upper bound. Therefore, if TIR is specifically for CUDA/Vulkan codegen, I believe it's proper to make this assumption that each buffer should be constant-sized; On the other hand, dynamic-shape allocation could be useful on other devices like CPUs, where allocating dynamic sized memory is very straightforward without constraint.
Therefore, to maintain TIR as generic, do you guys think there could be some alternative approaches, for example, pinging the buffer under loop i if its size depends on i?

Thanks @tkonolige. These examples are quite helpful, especially the larger ones as they helped to reveal problems in earlier part of the pipeline. In this case it was fixed by the other PR but it is a good protocol to follow, so in case we didn't have the fix before, it helps to reveal the problem.
High level summary of approaches.

S0: When we generate the program as a result of transformation, if somehow the intermediate region is bad due to some analysis, we probably end up in a wrong spot for good perf, as a result, a warning or simply say this is not something that we antiticpate (during search) so skipping early and look at another one could be useful.
S1: There are cases where finding constant upper bound is useful, and can be done in an intentional way, that might relates to cases where we indeed have loop dep intermediate results(like in sparse case)
S2: There are cases where dynamic size alloca can be intentional, one possible case is where we have share memory allocation that dep on input variables.

The particular example seems should be on S0(error out so we find the problem in earlier part of the pipeline) category. In the meantime, I think it is certianly valuable to introduce S1 in some settings.
The question is where should that happen, and how much symbolic analysis we want to involve.
In this case leave the alloca as it is(and not lifting) might be OK, and desirable when the variable bound is unknown, e.g. in the case where we have the following case
@T.prim_func
def func(out: T.Buffer[n, "int32"]):
    for i, j in T.grid(16, 16):
        buf = T.allocate([1024 * j * n ], "int32", "global")

To modularize the step a bit, perhaps we can have a separate pass that performs the upperbound setting when it is possible.

#12019 seems relevant but does not fix the issue.

I agree~ I think #12019 is only a patch for the issue.  I am not sure whether variable allocation extents are allowed or not by our design previously.
Thus It does not avoid all cases (only loop var dependent allocations). And const int bound may works poorly on certain extents. For example, min(i * 32 + j + 3, 224) - max(i * 32 + j - 3, 0).

Therefore, if TIR is specifically for CUDA/Vulkan codegen, I believe it's proper to make this assumption that each buffer should be constant-sized

Would this be something that should be checked in SplitHostDevice?  That way, if there's an implementation that uses dynamic sized arrays, and they aren't converted to constant-sized before then, the dynamic allocation could be initiated on the host side.


Assert rework

This branch includes fixes to windows build and a rework of asserts to fix #137 and #167

The ON_ASSERT macro is commonly use in grabbing the return code of some functions and asserting if the return code is wrong.
ON_ASSERT( int retcode = ) printf("lalala %u", 42);
CORE_ASSERT( retcode >= 0, "error in printf");

This use case is tricky to get right without the ON_ASSERT : you can't call the function inside the assert macro (because you change the behaviour of the code if asserts are disabled) and if you don't wrap the return code in ON_ASSERT you get  "unused variable" warnings when asserts are disabled.
The macro to check whether asserts are active or not is CORE_ENABLE_ASSERT  (this is actually a new thing from the change-set, before it used to be CORE_DEBUG as asserts were tied to the debug build config).
#ifdef CORE_ENABLE_ASSERT
/* code */
#endif

Like I said, I got the point ;).
Ok for CORE_ENABLE_ASSERT.
The PR sounds valid for me.


chore(deps): update common-dev-assets digest to bf42a76

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
4582204 -> bf42a76

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Plugin won't open after 2f9b199e5493c556569ecca5ad08c924706110ea

This is the last commit where I can use nvim leetcode.nvim.
Any other commit I've tried after that one, won't start leetcode.nvim
My config
  -- Last functional commit
  {
    "kawre/leetcode.nvim",
    commit = "2f9b199e5493c556569ecca5ad08c924706110ea",
    event = "VeryLazy",
    dependencies = {
      "nvim-treesitter/nvim-treesitter",
      "nvim-telescope/telescope.nvim",
      "nvim-lua/plenary.nvim",
      "MunifTanjim/nui.nvim",
      "nvim-tree/nvim-web-devicons",
      "rcarriga/nvim-notify",
    },
    opts = {
      lang = "typescript",
    },
    config = function(_, opts) require("leetcode").setup(opts) end,
  },

Screenshots
Working commit

After that

What is your neovim version

Try removing event = "VeryLazy"

@kawre NVIM v0.10.0-dev-1378+gc49cfd89fd (neovim-git on arch linux)
Same result after removing VeryLazy and updatin leetcode.nvim to latest.

Ok, it actually works if I specify lazy = false

It's because i've changed leetcode.nvim to mount after VimEnter as there were some problems with other plugins/options overriding my local plugin options on boot.
Probably VeryLazy triggers after VimEnter resulting in leetcode.nvim never having a chance to even mount.

Fixed version just in case someone finds this on google
return {

  -- leetcode.nvim
  -- https://github.com/kawre/leetcode.nvim
  -- You found an easter egg!
  -- To use it, uncomment this, exit nvim and write "nvim leetcode.nvim"
  {
    "kawre/leetcode.nvim",
    lazy = false,
    dependencies = {
      "nvim-treesitter/nvim-treesitter",
      "nvim-telescope/telescope.nvim",
      "nvim-lua/plenary.nvim",
      "MunifTanjim/nui.nvim",
      "nvim-tree/nvim-web-devicons",
      "rcarriga/nvim-notify",
    },
    opts = {
      -- HOW TO ENABLE TYPESCRIPT/JAVASCRIPT LINTING FOR THIS PLUGIN
      -- -----------------------------------------------------------
      -- * Install the eslint packages:
      -- npm install @typescript-eslint/eslint-plugin @typescript-eslint/parser
      -- * Then copy paste this into ~/.local/share/nvim/leetcode/.eslintrc.json
      -- https://pastebin.com/raw/aQXjpLuE
      lang = "typescript",
    },
    config = function(_, opts)
      -- Require it only if neovim starts with the argument 'leetcode.nvim'
      if vim.tbl_contains(vim.fn.argv(), 'leetcode.nvim') then
        require("leetcode").setup(opts)
      end
    end,
  },

}

@Zeioth The problem with this is that it doesn't do anything actually. config gets executed after all of the required dependencies were loaded which is the main problem there.
Having a function checking whether to lazy load leetcode.nvim will prevent all of the required dependencies from loading when not necessary.
You can either copy should_skip() and paste it into lazy = ... or use the simplified version specified in recipes.

Did you try it? It works as expected on my end. It might be redundant though if the plugin is already doing the same.

@kawre it would be cool being able to use the plugin with VeryLazy again, because as it is right now, (using lazy=false) it adds a good 30ms to startup time.

This startup time is because from what ive observed lazynvim waits for the dependencies required by my plugin and combines theirs startup time into mine. So if my plugin startup time is lets say 5ms and treesitters is 30ms it will combine it into 35ms.
If you are sure that some of the dependencies will load anyway, you can remove them from the dependencies list.
Right now my leetcode.nvim config has 0 dependencies as i load them elsewhere by events like "VeryLazy" anyway. This reduces the startup time to below 1ms.
You can also play around with lazy loading and see how it works, as i said before.

Ok I've been able to fix this by requiring the plugin in init only when the leetcode.nvim argument is passed to nvim.
  {
    "kawre/leetcode.nvim",
    dependencies = {
      "nvim-treesitter/nvim-treesitter",
      "nvim-telescope/telescope.nvim",
      "nvim-lua/plenary.nvim",
      "MunifTanjim/nui.nvim",
      "nvim-tree/nvim-web-devicons",
      "rcarriga/nvim-notify",
    },
    init = function(_, opts)
      -- Require only when needed
      if vim.tbl_contains(vim.fn.argv(), 'leetcode.nvim') then
        require("leetcode").setup(opts)
      end
    end,
    opts = {
      lang = "typescript",
    },
  },

So same fix I had, but on init, to ensure it is applied on startup.


User Authentication

I finished the user authentication week 4 tutorial. The website now requires users to login to post and view comments, and users can login/logout through their Google accounts. Comments display the name of the person who created the comment, and the user's name is displayed at the bottom of the page.
http://nevingeorge-step-2020.appspot.com/

LGTM


Implement Textarea support

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
Yes, this is a swagger UI issue.

Fastapi doesn't support a textinput line greater than one row tall. This code implements this by hacking the javascript:

https://stackoverflow.com/questions/69874134/fastapi-textarea-openapi-for-form-data

Could something like this be merged in for Fastapi?

Description
See above
Wanted Solution
https://stackoverflow.com/questions/69874134/fastapi-textarea-openapi-for-form-data
Wanted Code
https://stackoverflow.com/questions/69874134/fastapi-textarea-openapi-for-form-data

Alternatives
No response
Operating System
Linux, Windows, macOS
Operating System Details
No response
FastAPI Version
0.79
Python Version
3.10.4
Additional Context
No response

Probably not, as you have to self host the JavaScript for that solution. I also don’t see FastAPI provide “hacky” solutions like this in the end product

Thanks for the help @JarroVGIT ! 🙇
@zackees you can override and customize Swagger UI as you need, you can see how here: https://fastapi.tiangolo.com/advanced/extending-openapi/
As this wouldn't belong in FastAPI, I'll close this issue. Thanks for the interest! ☕

Textarea is something that is very essential. People like to add multi line descriptions and I've hit this limitation in multiple projects now. Not having this as a stock feature is very disappointing.

That's probably a complaint you would need to do to Swagger UI and the OpenAPI spec, not to FastAPI.
Also, have in mind that Swagger UI is supposed to be a tool mainly for developers testing their APIs, not a fully featured admin interface for your app.