Adopt CEL validation

This issue tracks a design to adopt CEL in CRD validation: https://kubernetes.io/docs/reference/using-api/cel/.
This is in need of a design; please do not submit a PR without a design.
Scope:

Identify what percent of validation we can move from webhook to CEL
Investigate impact on Kubernetes version support. I think this is silently ignored for 1.23/1.24, fails for 1.22 and older, works on 1.25+
Investigate if we can ever remove the webhook entirely. Either 100% parity, of "good enough" and webhook is optional for stricter validation
identify how we will write validations mechanically
Assess impact on CRD size - will we hit limits? DestinationRule was close to limits in the past I think

Exploring existing checks
WasmPlugin
validateWorkloadSelector => seems fine
validateWasmPluginURL => fine
validateWasmPluginSHA => fine
validateWasmPluginVMConfig => fine
validateWasmPluginMatch => fine
ALL COMPATIBLE
Overall

~Warnings support is ??? ~ => CRD cannot do warnings, but ValidatingAdmissionPolicy. For now we can do a ValidatingAdmissionPolicy with warnings, and errors in the CRD
~How do we check dupluicates?~ => keys.all(l1, keys.exists_one(l2, l1.name == l2.name)) (N^2 but ok...)

cc @TristonianJones for visibility-- the need for a "isRegex" and a "isCEL" check are both interesting

Missing ability to validate a CEL expression in CEL

@howardjohn Can you point me to where I could learn more about this use case? I'd like to try to understand it better.

@howardjohn Can you point me to where I could learn more about this use case? I'd like to try to understand it better

https://istio.io/latest/docs/reference/config/telemetry/#AccessLogging-Filter is the API, https://github.com/istio/istio/blob/2cc72adf87261df6294ad25c915246a92ea8d96e/pkg/config/validation/validation_istiod.go#L28 is the current webhook validation

@howardjohn @jpbetz I have toyed around with the idea of validating that a field is a CEL expression. You can absolutely do it, but the check itself is difficult to constrain from a compute / memory standpoint since it will lift ANTLR up into the runtime which is generally something we've avoided doing.  As a custom function, it's possible.
The other thought would be to make it possible for schema's to identify the type of the field as a CEL expression and perform validation on the syntax (to start) under the covers. I think that would likely be a more powerful model since it allows the system to better control the resources spent validating such fields.
For regex validation, that should be possible to expose as a custom function in one of the CEL canonical extensions.

yeah, runtime CEL evaluation makes me very nervous.

For regex validation, that should be possible to expose as a custom function in one of the CEL canonical extensions.

one thing to watch out for there is regex flavors... CEL's view of whether a regex is valid or not would be tied to go's regex implementation, which may or may not be the same as the one in the consumer of the type (for istio, that would be fine, but for arbitrary consumers, it may not be)

Re: "is regex" check: I tested a suggestion @liggitt gave me and it worked out reasonably well: https://gist.github.com/jpbetz/f01f0d21055407f7a9701ca51198c395
The error message produced when the regex string is invalid looks reasonable to me. The use of a regex "findAll" to test if the regex is valid is a bit clunky/hacky, but it does seems to work.

yeah, runtime CEL evaluation makes me very nervous.

It could be s slippery slope. If we support "is this CEL?", do we eventually need to support "Is this valid CEL with only legal variables functions used?" or "Is this valid CEL with proper type checking of all declared variables?"
@howardjohn Do you have a sense what you'll do in the short term? Would you validate in a webhook or defer checking till a controller evaluates the CEL expression?

@jpbetz we will probably incrementally add CEL in addition to the webhook for a while, rather than completely replace, especially to not require us to move everything in one shot.
Long term it would be good to get rid of the webhook though; some checks we will never be able to (the protobuf stuff) - but we can make those part of optional webhook or defer the checking maybe.
The "Is CEL" check is something I wouldn't mind deferring the checking of; its a 'nice to have' not required

Long term it would be good to get rid of the webhook though; some checks we will never be able to (the protobuf stuff) - but we can make those part of optional webhook or defer the checking maybe.

The "Is CEL" check is something I wouldn't mind deferring the checking of; its a 'nice to have' not required

For CEL I agree, deferring checking is probably OK.  There is always the possibility of runtime errors, so there is a need to communicate that back somehow (logs, metrics, status fields). A validation check reduces the classes of errors that are deferred to runtime but can't eliminate them entirely.
The protobuf cases are probably out of our hands.
Looks like we should be able to support most everything else?
~Warnings => OK (ValidatingAdmissionPolicy)
How do we check duplicates? => N^2 check (you may need to set maxItems on some lists)
Missing ability to validate a CEL expression in CEL => No support.
Missing ability to validate a regexp expression in CEL => https://gist.github.com/jpbetz/f01f0d21055407f7a9701ca51198c395 sufficient?
Validate IP address => Can you do something like https://github.com/kubernetes-sigs/gateway-api/pull/2246#discussion_r1278225456 - Gateway API was able to achieve conditional IP validation this way..

@jpbetz I analyzed the last of our resources (before had left a few off) and summarized my view of the current state in the original comment: https://github.com/istio/istio/issues/46151.
Based on this I think we can get >90% of our rules in CEL easily (actually, probably 50% of them are just checking things are not empty which is simple OpenAPI...). It looks like even among the 10% we can probably get creative to handle things.
We will incremental adopt, so the last X% will not block us from moving towards CEL, just may delay us removing the webhook entirely. We will likely never get to 100% parity which is fine -- at some point we can make the decision to remove the webhook and do validation at runtime.
The only real blocker we have from starting adoption is purely an Istio-side issue so we will work around that.
Appreciate all the help here and the great work to enable this!

A major blocker came up: https://github.com/kubernetes/kubernetes/issues/120973

not stale


What is seq_id param?

General Question
Form

[x] I've searched the Issues
[x] I've read the basic concepts
[x] I'm using the latest version

Question
What is the seq_id parameter and why is it used? It seems like it's not sent for the first request (undefined):
https://github.com/dilame/instagram-private-api/search?q=seq_id&unscoped_q=seq_id

It seems like it's not sent for the first request

That's right. It's an internal parameter from Instagram to "know" which messages the client already received.

If I always send seq_id: undefined, will any issue arise?
I also see latest_seq_id in the mqtt messages.


Disable knex useNullAsDefault warning

@jaswilli I think this is the correct fix for the warning? We should not be depending on the old behaviour of knex interpreting undefined as null, and if we are, we want to know about it?
With this fix, if people get errors, they can override them by setting useNullAsDefault to true temporarily, but hopefully they'll report them so we can fix the bugs.
I think this makes sense?
refs #6623

automatically set useNullAsDefault to false for sqlite3 so that we don't get a warning
we should not be relying on the behaviour of interpretting undefined anywhere, so it is correct that an error should be output if this happens so that we can fix the bad behaviour

Ah, so it's just a warning that the option has not been explicitly enabled or disabled (https://github.com/tgriesser/knex/blob/e1ed3851a8bd38228157323fbe24a935ba1f68b8/src/dialects/sqlite3/index.js#L20).
Yeah, we definitely want to stick with the new behavior so silencing the notice sounds like the right path to me.


Viz: 2D projection of skeleton superimposed on inverted MinIP image

This is working in initial_dataset_visualization.ipynb: Plot skeleton atop inverted MinIP.
This will simply copy The Allen's style:

[x] MinIP
[ ] MinIP inverted intensity
[ ] Skeleton to same coordinate space
[ ] Overlay in 2 color (red & blue)

Dendrites in skeleton in blue
Axons in skeleton in red

Here's the first working viz of skeleton on MinIP:

Reopening this for enhancements

a slider for the alpha of the skeleton
a slider for the alpha of the MinIP


[TypeError: promisePipe(...) is not a function]

var imagemin = require('imagemin');

imagemin(['src/img.jpg'], 'dist')
     .then(function (files) {
        console.log(files)
    })
     .catch(function(err) {
        console.log(err);
    });

js [TypeError: promisePipe(...) is not a function]
Using "imagemin": "^5.0.0"  on a clean project. Only package installed.
Am I doing something obviously wrong?

Yeah, you have to define some plugins using the use option (see example).
It shouldn't throw though.

ah, gotcha :)
Thanks a lot for the quick reply.


Use CloudHSM image from the correct registry

cloudhsm is a container run by the vmc pod. We need to make sure we pull
it from a "proper" registry and not the default one that no longer
exists.

Surely the build should happen in one place and fan out? /shrug


Add command for toggling checkbox

Resolves https://github.com/jotaen/xit-sublime/issues/4.
Summary
Add new command xit_toggle that toggles the status of a checkbox. By default, it toggles between [ ] and [x], but the sequence can be customised via the xit_toggle setting.

Nice, thanks for the quick turnaround! I'll look for the package update!

1.2.0 just happened to become available. (Sometimes it takes a few hours for new package versions to appear, unfortunately.)

Works great, thank you!


Provider handle static vs transaction calls

Closed as ethers already seems to be taking care of view calls in the provider.
https://github.com/web3well/bls-wallet/pull/467/files#diff-c4e3a7ff024b348f427a930835c8ff4ed9135b580c33fb7674f9bb4b42426460R53-R67


httplib.BadStatusLine:

vegaswe = client.fetch_weather("2436704")
Traceback (most recent call last):
File "", line 1, in
File "C:\Tools\Python(x86)\Python27\lib\site-packages\yweather.py", line 180, in fetch_weather
rss = self._fetch_xml(url)
File "C:\Tools\Python(x86)\Python27\lib\site-packages\yweather.py", line 344, in _fetch_xml
with contextlib.closing(urlopen(url)) as f:
File "C:\Tools\Python(x86)\Python27\lib\urllib2.py", line 154, in urlopen
return opener.open(url, data, timeout)
File "C:\Tools\Python(x86)\Python27\lib\urllib2.py", line 429, in open
response = self._open(req, data)
File "C:\Tools\Python(x86)\Python27\lib\urllib2.py", line 447, in _open
'_open', req)
File "C:\Tools\Python(x86)\Python27\lib\urllib2.py", line 407, in _call_chain
result = func(*args)
File "C:\Tools\Python(x86)\Python27\lib\urllib2.py", line 1228, in http_open
return self.do_open(httplib.HTTPConnection, req)
File "C:\Tools\Python(x86)\Python27\lib\urllib2.py", line 1201, in do_open
r = h.getresponse(buffering=True)
File "C:\Tools\Python(x86)\Python27\lib\httplib.py", line 1136, in getresponse
response.begin()
File "C:\Tools\Python(x86)\Python27\lib\httplib.py", line 453, in begin
version, status, reason = self._read_status()
File "C:\Tools\Python(x86)\Python27\lib\httplib.py", line 417, in _read_status
raise BadStatusLine(line)
httplib.BadStatusLine: ''

I was just trying this out for the first time. This may be an easy fix because I am just a noob so please tell me if this is something I am doing thanks.

Same issue here.  Any thoughts? Below is my code, the trace is nearly identical to OP.
import yweather
weather = yweather.Client()
wake_weather = weather.fetch_weather("615702")
print wake_weather["astronomy"]["sunrise"]
A note that may help is that the code below works
import yweather
weather = yweather.Client()
weather_id = weather.fetch_woeid("Chicago, Illinois")
print weather_id

@petek157 @killerdevildog11 Yahoo Weather has discontinued the RSS feed interface this library is based on. There is a new YQL Query interface: https://developer.yahoo.com/weather/
I don't plan on updating this library to use the new endpoint. Feel free to fork!


React stand alone: Mega Menu enhancement to accommodate HC + AI updates Visual QA testing Prod QA testing

This development work is also scheduled for the Web Component version. The Web Component development is managed under epic (#4322)

User Story

As a [user role below]:
developer using the ibm.com Library Mega menu HC + AI enhancement

I need to:
have a version of the pattern or component that has been tested for accessibility compliance as well as on multiple browsers and platforms

so that I can:
be confident that my ibm.com web site users will have a good experience

Additional information

Browser Stack link
Browser Standard
Browser versions to be tested:  Tier 1 browsers will be tested with defects created as Sev 1 or Sev 2.  Tier 2 browser defects will be created as Sev 3 defects.
Platforms to be tested, by priority:  1)  Desktop  2)  Mobile  3)  Tablet
Mobile & Tablet iOS versions: 13.1, 13.3 and 14
Mobile & Tablet Android versions: 9.0 Pie and 8.1 Oreo
Browsers to be tested:  Desktop: Chrome, Firefox, Safari, Edge, Mobile: Chrome, Safari, Samsung Internet, UC Browser, Tablet: Safari, Chrome, Android
Accessibility Checklist
Creating a QA bug
See the Epic for the Design and Functional specs information
Dev issue (#4653)
** Once development is finished the updated code is available on the Canary URL for testing.**

Acceptance criteria

[ ] Accessibility testing is complete.  Component is compliant.
[ ] All browser versions are tested
[ ] All operating systems are tested
[ ] All devices are tested
[ ] Defects are recorded and retested when fixed

Designs & spec: https://ibm.box.com/s/kie61ijq4ggvdhi6awjklp09a5ro4sms
Boxfolder: https://ibm.ent.box.com/folder/118692148518?s=lcjcmdoaz2hnj6gejx7ilafhrvje4k68
Example: md breakpoint design:

@Praveen-sr development for this has been completed. You can view here under the Custom Navigation story:
https://ibmdotcom-react-canary.mybluemix.net/?path=/story/components-masthead--with-custom-navigation

Prod QA testing started

Accessibility testing remains.  Testing should be finished by tomorrow.

Accessibility testing is finished.  Closing.


bug: empty nightly release

https://github.com/tensorchord/pgvecto.rs/actions/runs/6990060374/job/19019198142
https://github.com/tensorchord/pgvecto.rs/blob/main/.github/workflows/release.yml#L95
https://github.com/tensorchord/pgvecto.rs/releases/tag/v0.0.0-nightly.20231126

cc @VoVAllen

We don't have modification on 11/26, so no release is expected. We only need to fix the release tag.

https://github.com/tensorchord/pgvecto.rs/actions/runs/6990060374/job/19019198142#step:4:21


bcit-ci/codeigniter-website

An uncaught Exception was encountered
Type: Error
Message: Class 'Github\Client' not found
Filename: C:\xampp\htdocs\codeigniter-website\application\libraries\Github_api.php
Line Number: 26
Backtrace:
File: C:\xampp\htdocs\codeigniter-website\application\core\MY_Controller.php
Line: 28
Function: library
File: C:\xampp\htdocs\codeigniter-website\application\controllers\Welcome.php
Line: 15
Function: __construct
File: C:\xampp\htdocs\codeigniter-website\public\index.php
Line: 277
Function: require_once

I got this error after I fork and clone this project.
Are there anything I am missing.

The error may have occurred because composer dependencies might not be installed
Try this:

Install the composer: https://getcomposer.org/
Open the terminal in the project root folder
Put the composer install command on the terminal
And see if it works


Bug558: this.stoplistening is not a functio

Steps to repo:

Go to add-ons
Disable InterentStreaming

Experienced: Found the this.stoplistening error message in console
Expected: None

Assuming this has been tested on hardware (or at least MOCK mode), looks good!

Test latest code on hardware and it appears fixed


Fixed #24444 -- Updated contrib.admin visual theme to use django-flat-theme

Hi there,
Discussion in Django Developers Group.
My django-flat-theme app seems to be pretty popular according to github and pypi stats (even djangoproject.com uses it), so I decided that it's time to try to make it part of Django.
Few notes:

New theme contains CSS changes only. All changes are about colors, margins, sizes. No any markup changes.
To test how it's look like please use demo app, it's pretty easy to install.
I'm working on icon font feature to replace graphic icons. This feature will touch markup and it seems too big for this PR. So I'm going to make another PR later.

Feedback and contribution are welcome.

If I read the discussion correctly, the final suggestion was to provide only the new theme and release a separate package on PyPI with the old theme.
Can you add a paragraph to the release notes (docs/releases/1.9.txt)?

This is nice, it's not heavy and so fancy. Liked it a lot :+1:

It is very important that we not only get a clean, maintained, stable code, Django support and therefore we do in different ways, using the code, others with money and others just use it.
I want to give her native to design bootstrap support, expect to see more and enhancements.
Happy day for everyone.

@elky, do you need any help with the mention in the release notes that Aymeric mentioned?

I tried to merge this just now, but there are now some merge conflicts against master, and I'm not good enough at CSS to really be able to tell how to resolve that. @elky if you can merge/rebase against master, I'm happy to take care of the release notes and to merge this.

@jacobian I'll be able to do it this Tuesday. Also I'll update this PR with few CSS tweaks.

@jacobian I finally resolved conflicts and fixed things that @timgraham noticed earlier.

Could you add 1.9 release as requester earlier? It would be helpful to mention the level of support for each browser.

@timgraham do you mean release notes?

Yes

Added paragraph in release notes. I'm not sure I've done it in Django-style -- please check. Also check my poor English, please.
@timgraham I didn't mention specific browsers info just because these changes aren't major and theme works OK even in IE7. Big changes are going with my next PR which will include font icons.

No problem. I'm always happy to edit docs. Just having something to work with is a big help.
merged in 35901e64b043733acd1687734274553cf994511b, thanks!

On the changelist page, fields defined in list_editable are not aligned with content on other rows. Should I open a separate issue for this?
It would seem that setting a line-height: 30px for other cells fixes the alignment.

Should I open a separate issue for this?

Please create issue in django-flat-theme
https://github.com/elky/django-flat-theme/issues - I'll push fix to
django as well.
On Wed, Sep 16, 2015 at 9:46 PM, Hugo Osvaldo Barrera <
<EMAIL_ADDRESS>wrote:

On the changelist page, fields defined in list_editable are not aligned
with content on other rows. Should I open a separate issue for this?
It would seem that setting a line-height: 30px for other cells fixes the
alignment.
—
Reply to this email directly or view it on GitHub
https://github.com/django/django/pull/4627#issuecomment-140798947.


Post CSS issues

Hi, when I tried to run this project based on the docs, I got this error:

[0]
[0]   The plugin "postcss" was triggered by this import
[0]
[0]     src/Example.tsx:1:7:
[0]       1 │ import './Example.css';
[0]         ╵        ~~~~~~~~~~~~~~~
[0]
[0] DTS Build start
[0] ESM Build failed
[0] Error: Build failed with 1 error:

I think it's a postCSS issue, if that's the case it would be great to add the related command in the getting started code to make it clearer for people, or at least having a comment about it.

Hi @a0m0rajab I'm not able to reproduce this. Can you provide a repository where I can reproduce this error?
I just tried running pnpm build on a fresh clone and install of this repository and everything worked fine. Are you sure you installed the packages?

I did not do anything special just followed the documentation in the read me, I did not clone the repo yet I used degit
Sent from mobile device

From: Tim Mikeladze @.>
Sent: Tuesday, March 12, 2024 12:55:29 AM
To: TimMikeladze/tsup-react-package-starter @.>
Cc: Abdurrahman Rajab @.>; Mention @.>
Subject: Re: [TimMikeladze/tsup-react-package-starter] Post CSS issues (Issue #117)
Hi @a0m0rajabhttps://github.com/a0m0rajab I'm not able to reproduce this. Can you provide a repository where I can reproduce this error?
I just tried running pnpm build on a fresh clone and install of this repository and everything worked fine. Are you sure you installed the packages?
—
Reply to this email directly, view it on GitHubhttps://github.com/TimMikeladze/tsup-react-package-starter/issues/117#issuecomment-1989513749, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AELNMKO5MBMEVZ4D7Y63E2TYXYR5DAVCNFSM6AAAAABEQ7DHASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBZGUYTGNZUHE.
You are receiving this because you were mentioned.Message ID: @.***>

degit clones the repo...
Did you run pnpm install afterwards?

Yes I ran that
Sent from mobile device

From: Tim Mikeladze @.>
Sent: Tuesday, March 12, 2024 1:21:30 AM
To: TimMikeladze/tsup-react-package-starter @.>
Cc: Abdurrahman Rajab @.>; Mention @.>
Subject: Re: [TimMikeladze/tsup-react-package-starter] Post CSS issues (Issue #117)
degit clones the repo...
Did you run pnpm install afterwards?
—
Reply to this email directly, view it on GitHubhttps://github.com/TimMikeladze/tsup-react-package-starter/issues/117#issuecomment-1989547320, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AELNMKOP6S6FFP7J7MH5NVDYXYU6VAVCNFSM6AAAAABEQ7DHASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBZGU2DOMZSGA.
You are receiving this because you were mentioned.Message ID: @.***>

Okay. You will need to provide a repo I can reproduce this error in.

I’m closing this issue in the meantime. After you provide a reproduction repo, if it’s indeed a bug, I will reopen and fix it.


X axis values with labels not aligned

Codepen
https://codepen.io/jimfilippou-the-animator/pen/aRpxdG
Explanation
The X-axis takes 100% of the area but line chart doesn't

What is the behavior you expect?
Equally aligned values on X axis

What is happening instead?
This https://codepen.io/jimfilippou-the-animator/pen/aRpxdG

What error message are you getting?
No error message

Try this:
grid: {
    padding: {
       left: 0,
      right: 0
    }
}

I have updated my pen https://codepen.io/jimfilippou-the-animator/pen/aRpxdG
The issue seems to be resolved only for first and last values, however, the initial issue persists

If you do not provide any labels or categories, ApexCharts generates labels in 1.2.3... fashion and gives xaxis.tickAmount: 8 as default.
In such a case, you will have to play with tickAmount and see what works. I guess tickAmount: 7 will work in your case.
I will change tickAmount default for such use-cases to equal the number of data-points in the next releases.

Fixed, but close the issue whenever you default the tickAmount to the data points.
Thanks once again! 🙌 🎉


createStyles typings slows down typescript type checking

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
createStyles function typings slow down typescript type checking by ~3 times in our case.
An attempt was made to move from<EMAIL_ADDRESS>to<EMAIL_ADDRESS><EMAIL_ADDRESS>which led to a decrease in build performance by 3 times.
Also, my vscode now works with large delays (~10s) on type checking or intellisence.
Checking time before update
#<EMAIL_ADDRESS><EMAIL_ADDRESS>PS> time tsc -p tsconfig.json
TotalSeconds : 20

Checking time after update
#<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>PS> time tsc -p tsconfig.json
TotalSeconds : 63

If I change createStyles.d.ts definition like this

then checking time returns to normal 20s.
Checking time after fix typings in @material-ui/styles
#<EMAIL_ADDRESS><EMAIL_ADDRESS>@material-ui/styles@fixed-types
PS> time tsc -p tsconfig.json
TotalSeconds : 20

Expected Behavior 🤔
Type checking time is about the same for 3.9.2 version
Your Environment 🌎

Tech
Version

Material-UI
v4.7.2

React
16.11.0

TypeScript
3.7.3

System
Windows 10

See also https://github.com/microsoft/TypeScript/issues/34801.

Closing this in favor of #19113. We already made some significant improvements as well as typescript improving their performance as well.

In .vscode/settings.json:
"typescript.validate.enable": false
But if you use built-in TS compiler, you should have it isolated from IDE.
Thats how i returned blazing fast performance. Nothing else works like this...


OData Support

What are your thoughts on having this grid support OData style parameters for sorting/filtering/paging.
This would make it much simpler to implement the server side of things.
See example:
https://docs.microsoft.com/en-us/aspnet/web-api/overview/odata-support-in-aspnet-web-api/supporting-odata-query-options

I thought about it, but this is advanced scenario and I want first solve the basics. After that I will look at this advanced features.


Cutscene separate ways no is HD Project

Hello. I have a problem with separate ways. All cutscenes is on HD Original. i installed mod 2x, but the problem continue.  Link url: https://i.imgur.com/HnMFkUe.png All files (18 parts) downloaded without any problems.
NVIDIA GTX 1050, i5 3470, 16GB RAM and SSD 1TB.

It does look like the new cutscenes are being used. There's nothing pixelated and barely any jagged lines.
Bear in mind that those cutscenes will never look perfect, as even though we still have the camera motion data, we don't have the character animations to re-record their movements in high resolution.


Update parser-diags.swift test

The diagnostic for confusables was updated in https://github.com/apple/swift/pull/33105.

cc @rintaro @akyrtzi

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - cebacabd5ef080c65a82972f52b52fc9b5a651ff

Could you please re-run the tests? There was a typo in the diagnostic message which I have fixed (sorry about that).

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - cebacabd5ef080c65a82972f52b52fc9b5a651ff

Build failed
Swift Test OS X Platform
Git Sha - cebacabd5ef080c65a82972f52b52fc9b5a651ff


Change URL paths for web API to make more sense to the user.

Old names referred /
To internal naming schemes /
Use names for users.
Changes:

/job/ -> /jobs/
pipeline/ -> regulations/
proposal-pipeline/ -> notices/
upload -> files/

Addresses https://github.com/eregs/regulations-parser/issues/305

:+1: agreed at being more semantic


After creating a new program, jump to that program?

Sometimes it does, others it doesn't.

closed with d005015

yay


Crash with completion from too long files

Given that the second opened file exists and contains text TEST TXT (two words starting with T) the following script is enough to crash:
 vim --cmd 'set columns=5000 noswapfile' -u NONE -i NONE -N /tmp/b $PWD/"$(perl -e "print ((('/'.('d' x 255)) x 4).'/'.('a' x 255))")" -s <(<<< $'iT\C-n')

Note that bash supports neither <<< nor \C-n in $'', need echo and actual value. Bug was found by coverity in Neovim (CID 56813), at the end of ins_compl_next function (where it copies compl_shown_match->cp_fname + i into IObuff). It is necessary that columns value is large.
Checked with vim 8.0.512.

Nikolai Pavlov wrote:

Given that the second opened file exists and contains text TEST TXT (two words starting with T) the following script is enough to crash:
 vim --cmd 'set columns=5000 noswapfile' -u NONE -i NONE -N /tmp/b $PWD/../"$(perl -e "print ((('/'.('d' x 255)) x 4).'/'.('a' x 255))")" -s <(<<< $'iT\C-n')

Note that bash supports neither <<< nor \C-n in $'', need echo and actual value. Bug was found by coverity in Neovim (CID 56813), at the end of ins_compl_next function (where it copies compl_shown_match->cp_fname + i into IObuff). It is necessary that columns value is large.
Checked with vim 8.0.512.

I tried to turn it into a Vim script, but it doesn't reproduce the
problem:
set nocompatible
set columns=5000
set noswapfile
let longname = getcwd() . "/../" . repeat('/' . repeat('d', 255), 4) . '/' . repeat('a', 255)
exe "next /tmp/b " . longname
exe "normal iT<C-N>"
What is missing?
Hmm, seems that code also doesn't handle translations and multi-byte
characters.
--
"The amigos also appear to be guilty of not citing the work of others who had
gone before them.  Even worse, they have a chapter about modeling time and
space without making a single reference to Star Trek!"
(Scott Ambler, reviewing the UML User Guide)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

The file in question must be created and contain at least two words starting with T. And it must not be loaded AFAIR.

Nikolai Pavlov wrote:

The file in question must be created and contain at least two words
starting with T. And it must not be loaded AFAIR.

Thanks, now I could reproduce and turn the example into a test.
--
Marriage isn't a word.  It's a sentence.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Only show @here and @channel in team chats

This commit also handled a very annoying Flow issue. After adding the
prop isTeam to mention-hud-container Flow was endlessly erroring
that isTeam is missing in object literal A.
The solution was to add an index.js.flow file to the directory to
force flow to understand the module level props.
Cause of the original error is not totally known.
1-1 Chat

Group

Small Team

Big Team

LGTM!

Updated with review feedback for typing and commenting. Also cleaned up the _withProps section - thanks @chrisnojima


No POIs in vulnerable program

hi
i am trying to test a code with MATE:
int main(){

	int x = 1;
	if (x=0){
	 x=3;
	 }
	int y = x/0;

	int size = 5;
	char *src = "xxxxxxxxxx";
	int *dest = malloc(size + 1);
	memcpy(dest, src, size);

	int* alc_mem__ptr = malloc(5);
	memcpy(alc_mem__ptr, src, 9);
	free(alc_mem__ptr);
	memcpy(alc_mem__ptr, src, 9);

	return 0;
}

this code has several issues; redundancy, use after free, overflow ...
but i am not getting any reports in POI or any other sections.
to make sure that i correctly installed MATE, i analyzed "authentication.c" file and two problems are shown in POI section.
p.s: how to delete the scanned project in build section?

@sherkat69 MATE only checks for certain kinds of POIs (e.g., it checks for UAF but not overflow). Additionally, I wouldn't be surprised if the compiler has optimized away the straightforward UB in your program, you can check this by compiling with clang -emit-llvm and inspecting the bitcode to see if the UB is still present.

is there any command to clear all previous builds?


LDAP: Authenticated Searches without a manager password

Allow to use the LDAP AuthProvider with a LDAP Server
prohibiting anonymous searches but without providing
a manager password : searches are made on behalf of
the authenticated user.

I need to ask: what problem does this change try to solve? Since Gitblit will always bind as the manager first, a manager password will always have to be provided. What is the intented benefit of searching for the user account as the user instead of the manager? That should render the manager account obsolete, but the commit doesn't actually do that.

When this patch was proposed, the manager password was not mandatory. you can provide an empty user/password. I don't know however if something has changed since v1.6
This patch allow to use an ldap server without the manager password, when the server doesn't allow anonymous search : it binds as the user. I use it every day with my gitblit instance with a corporate active directory.

So you use this with an empty username and password or just an empty password? Does your manager account exist and have an empty password?
I haven't understood yet how this works. In your version, too, authenticate will first call getLdapConnection. Only if it gets back a connection will it run your new code. But getLdapConnection already binds either anonymously (empty username and empty password) or with the username. So if there is a username to bind against but no password, and the bind thus fails, authenticate will stop.

Let me ask differently: Do you expect that, with this change, when the bindpattern property is not empty, the username and password properties are ignored, and Gitblit never binds as the manager but only as the user?


Improve tooltip for metadata in the inspector

Follow-up to https://github.com/godotengine/godot/pull/82940.

Since metadata can't have a description filled in, this adds usage instructions for the metadata instead.
Preview
Since the inline code appears mangled due to not having enough line spacing, I'm marking the PR as a draft. In general, it seems theme propagation isn't working correctly in those tooltips. Alternatively, we can drop the [code] blocks.

Since the inline code appears mangled due to not having enough line spacing, I'm marking the PR as a draft

I think this is not specific to this PR, it's an EditorHelpBit issue that already exists in several places.

Is the use of italics for the whole description intentional? Not saying it's wrong, just wanted to understand the reasoning, when compared to the rest of the helper popups.

Yes, because it's an automatically generated description as opposed to one manually supplied by the script author.


FEATURE: Event stream filter

Allows to filter events by their type:
$filter = EventStreamFilter::forEventTypes(EventTypes::create(EventType::fromString('NonExistingEventType'))
$eventStore->load($stream, $filter);

Resolves: #5

I'm not 100% sure about this feature and the API, but it would allow for "Dynamic Consistency Boundaries" as described here: https://sara.event-thinking.io/2023/04/kill-aggregate-chapter-1-I-am-here-to-kill-the-aggregate.html


add an option to disable the first come/first serve principle for barcamps

This is mostly useful for international and youth barcamps where certain quotas need to be met. The idea is to not automatically signup new users. The alternative workflow simply means that you can only add yourself to the waiting list (which might need to be renamed for this). An administrator should get an email when this happens and should be able to move you manually to the participants list.
Items to check:

How do the mails look like in the alternative version? (needs to be adjusted by the bc admin though)
Where do labels need to be changed?
what effect does that have to the event management interface in general?

As we have separate events for each day now this needs to be done for every event.
In the backend this version should be enabled via a checkbox for the barcamp.

For now there is a new checkbox which does the following when enabled:

When a user registers it will automatically put on the waiting list and not the participation list
When a user cancels no automatic move from the waiting list to participation list will happen
The only way to get on the participation list is by manually being moved by an admin.


Remove dependency on panzoom when grid size is changed

Currently if you see the grid size change part, you can see that setPanZoom is called every time.
https://github.com/hunkim98/dotting/blob/6c38fbd7c103467679a139d6ea4246b77340a3a8/src/components/Canvas/Editor.tsx#L523C5-L579
 private extendInteractionGrid(direction: ButtonDirection) {
   const interactionLayer = this.interactionLayer;
   interactionLayer.extendCapturedData(direction);
   // this.renderInteractionLayer();
   const interactionCapturedData = interactionLayer.getCapturedData()!;
   const baseRowCount = getRowCountFromData(interactionCapturedData);
   const baseColumnCount = getColumnCountFromData(interactionCapturedData);
   if (direction === ButtonDirection.TOP) {
     // ⬇ From here
     this.setPanZoom({
       offset: {
         x: this.panZoom.offset.x,
         y:
           this.panZoom.offset.y -
           (this.gridSquareLength / 2) * this.panZoom.scale,
       },
       baseRowCount,
       baseColumnCount,
     });
    // ⬆ To here
     this.extensionPoint.lastMousePos.y -= this.gridSquareLength / 2;
  //..
 }
}

Setting panzoom everytime causes all canvases to render because all canvases rely on the panzoom.
However changing panzoom when user changes the grid size seems quite unnecessary. Why do we need to change the panZoom when we change the grid size?
 I believe I designed it this way because I have considered users extending the grid even when their mouse has left the canvas.
Maybe a deeper analysis on this issue is necessary. Maybe we could eliminate setPanZoom method for grid size changes?

After working on grid size change enhancement #4 , I realized that it is impossible to remove the dependence on panzoom when grid size changes. This is because when grid size change occurs, the panzoom should be modified to let the grid extension point follows the mouse cursor. I will close this issue.


Add saved messages feature

This is a rough draft atm and does not include the ability to remove saved messages.

The liked flag is still not unset in the main list until we reload posts. I know why this happens but do not have any good ideas to resolve it right now.

I don't think it's worth changing the liked flag until the list reloads because we will have to do a list refresh and since we are removing it from savedPostsDB anyway it should not cause any issues.

BTW, I think we should also do this in case of a removed post.
_posts.value = tempList.map {
	if (it.shortId == post.shortId) it.isLiked = false
    it
}

BTW, I think we should also do this in case of a removed post.
_posts.value = tempList.map {
	if (it.shortId == post.shortId) it.isLiked = false
    it
}

Yeah that'll probably fix the last bug.


Detect synchronization

Fixes #2808
Depends on #2818

Jenkins, test this please

@emcastillo  Please merge #2818 first.

Jenkins CI test (for commit 564c368f0fd62ce85fe9e59a4a0023e3018089a5, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 564c368f0fd62ce85fe9e59a4a0023e3018089a5, target branch master) failed with status FAILURE.

#2818 was merged

Please merge #2818.

Rebase please :)

PTAL.

Jenkins, test this please

Jenkins CI test (for commit 0ff543dc0fcaba3829029b8d32fe0fca071ffc1c, target branch master) failed with status FAILURE.

😂I guess that we have more synchronization than we expected

Hmm, it's too many...
I think we should separate the change to test helpers as another PR.

I agree. At first, I thought that all of them were going to be fine so it was nice to do the checks all-together.

Done (force-pushed).

Jenkins, test this please

Jenkins CI test (for commit 20398d1aa30b28f8ad43858a91e0c46a0eddd3a2, target branch master) succeeded!

Discussion to remove this feature ongoing in #2946


Cross Platform Messaging not Working Properly (iOS-Android)

I implemented Chat Sdk in my ios App..I really Appriciate your work,But i am having problem with cross platform Messaging,Push Notification is not working,And received messages are blank..please fix that issue.Need Stable Version for Android..

Hi Ghana,
Could you split these up into individual issues on the specific platform you are having problems with. The iOS and Android should be compatible if you have correctly configured both projects with Firebase.
I would recommend reading through the articles on the wiki page, specifically the Firebase configuration tutorials:
iOS: here
Android: here
Push notifications:
Could you confirm which platform push notifications are not working for? Push notifications need to be configured for both projects. Have you run through the iOS configuration guide to add the Backendless pod? If you have having trouble with a particular platform please raise an issue on the appropriate repo.
Blank Messages
Are you receiving blank messages when using iOS-iOS or Android-Android or only iOS-Android? I expect this is due to Firebase not being configured properly. I would recommend testing the iOS chat, then the Android one and making sure the messages are being updated on the Firebase database. If you are still having problems, once again, raise it as an individual issue on the specific repo with steps used to reproduce.
Simon

Hello,
Thank you for Reply
Android to iOS -
When message is sent from Android ,iOS device displays blank message.

iOS to Android -
When message is sent from iOS,Android device displays {text:message}. (message = text sent by user from iOS device)

Push Notification -
In both cases,Notifications are not receiving.
While,Android-Android and iOS-iOS working perfectly (Both message and Push Notification) using same configured firebase.
Thank You.
Gaurav

I will have a look into this and get back to you

Are you using the latest version of the iOS SDK? We updated it last week to fix this issue. Can you also send me a copy of the Firebase data (from the dashboard) for this chat?

I am sending the Firebase data copy for chat
JSON.json.zip

Have you tried deleting and reinstalling the apps and clearing down the Firebase database? I just tested the app and I can't see any problems.

Since we are unable to reproduce this issue, and there has been no response in 18 days I am closing this issue.


Pull Request 1

Getting started

Review requested from @AshotC

Everything looks fine. If you're using Windows I can highly recommend GitHub for Windows, the application makes a lot of git operations almost trivial by putting a GUI over everything. There may exist something similar for other operating systems as well.


Stop ice on exception to game launch

Fixes #1582
Stops iceAdapter when error occurs during game launch

If there is a unit test can we verify this


Request: Group specific entity name

Would it be possible to add something like this? Maybe not exactly the same, but some way to allow me to override the name of an entity. Even if I had to run it across a regex or something would be fine.
https://community.home-assistant.io/t/group-specific-friendly-name/12816/6

This would require much work:

[core] Allow custom card (as opposed to state-card which are supported now)
Make a custom group card. Group card has lots of logic - it would be bad to just fork it.

Found an easy way to do it. Implemented :)


Intergration Stops working After upgrade to HA 2023.5

Hello after upgrading Home Assistant to the latest version 2023.5 the integration does not load properly and it gives out the following error
AttributeError: 'ConfigEntries' object has no attribute 'async_setup_platforms
It starts on line 55 of init.py file.
Taken from a comment from the Home Assistant forums
The async_setup_platforms function had been marked deprecated a while back, initially set to fail as obsolete in 2023.3. Now it actually fails in 2023.5, as previously announced.
Since all the messages in this thread are about custom integrations (likely from HACS) the developer of those custom integrations will need to update their integration to use to use await async_forward_entry_setups instead of async_setup_platforms.
I changed
hass.config_entries.**async_setup_platforms**(entry, PLATFORMS)

to
hass.config_entries.**async_forward_entry_setups**(entry, PLATFORMS)

the integration seems to load but the entities do not appear as function.
I am not a Python expert so I do not know what else that function affects.
If anyone could help it would greatly appreciated.

Hi, I know about it and it was the same issue at my instance of HA as well... I'm going to repair it asap...


Suggestion for object exchange between .net and Javascript

Hi,
First I wanted to thank you for the amazing work. Keep the great work going.
Next, I wanted to point you to a solution that I have been using for exchanging data between C# and Javascript.
While using ObjectInstance is theoretically the best practice in many situations, regarding the dynamic nature of the Javascript objects, most of the times the easiest way of exchanging data between C# and Javascript is a simple Json serialization.
While, I usually use Json.NET for doing so, but .net's builtin JavaScriptSerializer class also does an acceptable job.
This is just some idea for making things a little easier. Maybe it comes in handy in some situation.
Thanks for the great work.

Yep, and there's a JSON serializer built into Jurassic, which (unlike Json.NET) serializes and deserializes Jurassic (JS) objects.  So if you have a .NET object you can serialize it to a string using Json.NET and then deserialize it to a Jurassic object using JSONObject.Parse.


Update description of '--plugin' option

Description of '--plugin' option is wrong.
Should update it.

description of --verbose-version should be updated


Minor Fixes

Hi,
this merge request fixes some minor issues.
It was created with help of Feram.
If it helped to improve this repository,
please consider to star Feram's repo to show some love! 😊
PS:
The Feram team is currently working on making this technology
available for everyone. If you want to be one of the first to use it,
please sign up up for the newsletter on feram.co.

Duplicate of #46.
@adius @ronbak please do not spam repos to promote your tool.

Oops, sorry. My fault … I guess it's getting late 😅…


🚀 Feature: Scaffolder Component Creation as part of Entity Management

🔖 Feature description
We would like the scaffolder plugin to be the core pathway through which developers in our organization add resources to their existing applications. The flow we imagine would be something like this:

I visit the entity overview page for my application, "Users Service"
There is a ScaffolderCard or ScaffolderContent where I can browse through resource templates which I can apply to "Users Service".

For example, one may be a "New Postgres DB" template. This template would create a pull request in my "Users Service" repository that adds IaC code for the creation of a new Postgres DB.

When I click the "New Postgres DB" template,  the "entity" field is automatically populated with component:users-service.

The current scaffolder plugin misses a few key features that allows this to work. Specifically:

There is no Scaffolder(Card|Content) publicly exposed from the plugin
There is no way to autopopulate fields for a specific entity

🎤 Context
Creating pull requests is already a core feature of all the scaffolder scm integrations. Integrating the scaffolder plugin frontend with the entity overview page direclty presents a clear and intuitive experience to developers who can discover the extensions that can be added to their applications and seamlessly begin the process of adding them.
✌ Possible Implementation

There is no Scaffolder(Card|Content) publicly exposed from the plugin

At most, a new component for displaying available templates for an entity. Can use existing filter logic, e.g. templates filtered by Kind: Resource or tags includes "pull_requests".
At least, pubilcly expose TemplateList so that developers can add it into a custom ScaffolderContent card.

There is no way to autopopulate fields for a specific entity

One idea: autofill form fields based on query parameters in url, e.g. ?field_entity=component%2Fusers-service. This would also require some updates the href logic in the TemplateCard (if this component is used in the entity overview page.
👀 Have you spent some time to check if this feature request 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?
Yes I am willing to submit a PR!

In the discord I found a similar request: https://discord.com/channels/687207715902193673/923143744176418836/992254161720725575

This is a cool idea, I think some of these things are actually in place just not the initial UI to view/discover related templates
Some foundational work I think has been done as part of this: https://github.com/backstage/backstage/issues/14706
Also, I would make sure that any thinking you do on this subject is based off the Scaffolder Next work documented here as that will soon be the default: https://backstage.io/docs/features/software-templates/testing-scaffolder-alpha
Some related Discord links:

https://discord.com/channels/687207715902193673/1095329028010950666
https://discord.com/channels/687207715902193673/1095393671874093096

ah thank you. i guess @taras did follow up on his discord note. guess my searching didn't turn up everything. will take a look at the resources you shared.

was able to get the following together. looks pretty good so far! signing off for the day, will update tomorrow:
import { stringifyEntityRef } from '@backstage/catalog-model';
import { EntityKindFilter, EntityListProvider, useEntity, useEntityList } from '@backstage/plugin-catalog-react';
import { TemplateEntityV1beta3 } from '@backstage/plugin-scaffolder-common';
import { SecretsContextProvider } from '@backstage/plugin-scaffolder-react';
import { TemplateGroups } from '@backstage/plugin-scaffolder-react/alpha';
import { EmbeddedScaffolderWorkflow } from '@frontside/backstage-plugin-scaffolder-workflow';
import React, { PropsWithChildren, useCallback, useEffect, useState } from 'react';

const TemplateListProviderInner = (props: PropsWithChildren<{}>) => {
    const { updateFilters } = useEntityList();
    useEffect(() => {
        updateFilters({
            kind: new EntityKindFilter('template'),
        })
    }, [updateFilters]);

    return <>{props.children}</>
}

const TemplateListProvider = (props: PropsWithChildren<{}>) => {
    return <EntityListProvider><TemplateListProviderInner {...props} /></EntityListProvider>
}

export const EntityScaffolderContent = () => {
    const { entity } = useEntity();
    const [selectedTemplate, setSelectedTemplate] = useState<TemplateEntityV1beta3 | null>(null);
    const handleTemplateSelected = useCallback((template: TemplateEntityV1beta3) => {
        setSelectedTemplate(template);
    }, []);
    return (
        <SecretsContextProvider>
            <TemplateListProvider>
                {selectedTemplate ? (
                    <EmbeddedScaffolderWorkflow
                        namespace={selectedTemplate.metadata.namespace || 'default'}
                        templateName={selectedTemplate.metadata.name}
                        initialState={{ system: stringifyEntityRef(entity) }}
                        onError={(error: Error | undefined) => (
                            <h2>{error?.message ?? 'Houston we have a problem.'}</h2>
                        )}
                    />
                ) : (
                    <TemplateGroups
                        groups={[{
                            title: "Resources",
                            filter: (e) => e.spec.type.toLowerCase() === "resource",
                        }]}
                        onTemplateSelected={handleTemplateSelected}
                    />
                )}
            </TemplateListProvider>
        </SecretsContextProvider>
    )
}

if i were to contribute something like this back to OS i'd also probably want to add some sort of callback prop to Workflow along the lines of onBackFirstStep, so that if i'm on the first step of the template workflow i can hit back and go back to the <TemplateGroups/> view


Drop all blank markers, section renders br

Before this patch, Content-Kit would insert blank markers in any empty section to ensure a cursor could be placed there. This changes that to be a section responsibility. When there are no child markers, a markup section will create a br.
This means throughout the codebase sections and section offset should be used to manage editing actions. There may be no marker at a given cursor position, only using the offset in a section is reliable.
Also ensures correct behavior when deleting two sections in their entirety.

Pushed a successful rebase on master, though there are failing tests for lists. I presume their elements must be registered into the elements map, then many things should fall into place.

This afternoon we investigated changing the list section and items to work with this no-more-blank-marker-branch. We approached it by attempting to have offsets always be top-level section based. For example given this content:

Howdy

Friend

And a selection including dy\n* Fri, we would expect these offsets:
{
  headSection: markup section with Howdy,
  headOffset: 3,
  tailSection: list section with Friend,
  tailOffset: 3
}

However we quickly realized that some list sections would then contain multiple list item sections that, having zero width if they are empty, would be un-addressable. For example:

Howdy
*

Friend

Knowing the difference between the cursor being in the first list item and at the start of the second list item would not be possible.
With this reflection in mind, we believe offsets on cursors should always be pinned to post model that has markers in it. This is because items with markers in them, unlike anything else in the post model, are allowed to be empty.
Next steps are to introduce a "markerable" (poor name) mixin or super-class, update offsets to return those instead of sections, and make the various mutation APIs smart about how to handle the boundery conditions between different kinds of markerables.
For posterity, our WIP used for exploration: https://github.com/mixonic/content-kit-editor/tree/die-blank-markers-die-WIP

@bantic AWESOME!!!!! :rose:


React-app won't 'forget' a previously used proxy

Is this a bug report?
Yes

Can you also reproduce the problem with npm 4.x?

I was on npm 5, followed the steps to use npm 4.6.1, and still reproduced it
Which terms did you search for in User Guide?

react won't 'forget' a proxy, even after it has been changed.
Environment

node -v: 8.9.1
npm -v: 4.6.1
yarn --version (if you use Yarn): 1.3.2
npm ls react-scripts (if you haven’t ejected<EMAIL_ADDRESS>
Then, specify:

Operating system: OS X El Capitan 10.11.6
Browser and version (if relevant): Chrome 62.0.3202.94

Steps to Reproduce

(Write your steps here:)
I don't know that this will be reproducible on another machine, but here's what I did...

run create-react-app
add a line to package.json for a proxy server.
Initially I used http://localhost:3001 server to get some data from a link for me... but out of curiosity, I attempted to put that link directly into the proxy.  Here's is the link (it's just returns json):

http://m.lowes.com/CatalogServices/product/nvalue/v1_0?nValue=4294857975&maxResults=20&showURL=1&rollUpVariants=1&showUrl=true&storeNumber=0595&priceFlag=rangeBalance&showMarketingBullets=1

It didn't work, because of a no Access-Control-Allow-Origin error, which was fine, so I changed the proxy back to http://localhost:3001

Expected Behavior

(Write what you thought would happen.)
I thought it would proxy to my localhost server on port 3001
Actual Behavior

(Write what happened. Please add screenshots!)
It still tries to proxy to the link above, even though I have changed it.  Here's the REALLY strange part.  I figured "Ok, whatever, I've barely written anything for this project at all... I'll make a new project."
In my new project, I did not attempt to proxy directly to the link above, I used http://localhost:3001 from the very beginning.  It STILL tries to use that link, even though my new project has never seen that link, is in a totally different directory, and I even named the react-app something different.  This is happening with every new react-app I create.  Older projects on my machine that use a proxy still work, I can't explain it...

Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)
I don't know that you'll be able to reproduce it, but here are the steps that caused this error in the first place.

clone this repository: https://github.com/gaahma/assessment.git
npm install in assessment directory
cd into front-end and run yarn install
Try to run it:  in assessment directory run node server.js, then in a new terminal window, cd into front-end and type yarn start.
In theory a barebones react app will start, and a small object {data: 1234} will be logged to the console in the browser.
Change the proxy in the front-end package.json to this link:

http://m.lowes.com/CatalogServices/product/nvalue/v1_0?nValue=4294857975&maxResults=6&showURL=1&rollUpVariants=1&showUrl=true&storeNumber=0595&priceFlag=rangeBalance&showMarketingBullets=1

try to run again, an error should be logged to the console in browser
change the proxy back to http://localhost:3001
Run again... if you're able to reproduce the error, you will get the same error you got before, despite having changed the proxy.

Some steps I've taken to try to resolve:

All the steps you listed above with regard to node version
Uninstalling npm, yarn, and node from my machine and reinstalling them.
yarn cache clean
Fresh clones of the project
Starting new projects, and never using the link as a proxy
Several reboots/shutdowns.
I'm seriously thinking about reformatting my machine at this point... I really regret trying to proxy to that link....

This is for a web-development assessment I'm writing for a potential employer... I can't really get it off the ground until I get this resolved.  I know you guys are busy, but if you can help me out sooner rather than later, my career would thank you

This is bizarre, are you sure you're editing the correct package.json? Are you sure you're not hitting a service worker if you ever put that link into the app itself?
Try changing the PORT. e.g. PORT=3001 npm start

Absolutely positive I'm editing the correct package.json... just checked again to be sure.  I tried changing the port as you suggested, same error.

Definitely not a bug after all... my browser was caching that link for some reason.  A hard reset and I'm good to go.


Unable to run AnimateDiff getting error "AttributeError: module 'torch.nn.functional' has no attribute 'scaled_dot_product_attention'"

Hi I am having an issue with running the animatediff extension in Automatic1111 in Paperspace.
The error I get within Automatic1111 is "AttributeError: module 'torch.nn.functional' has no attribute 'scaled_dot_product_attention'". Below is the terminal log.
Wondering if It might be an issue with the torch version, or something related to this repo?
Thanks

2024-03-03 14:21:37,915 - AnimateDiff - INFO - AnimateDiff process start.
2024-03-03 14:21:37,916 - AnimateDiff - INFO - Loading motion module mm_sd15_v3.safetensors from /notebooks/sd/stable-diffusion-webui/extensions/sd-webui-animatediff/model/mm_sd15_v3.safetensors
2024-03-03 14:21:37,931 - AnimateDiff - INFO - Guessed mm_sd15_v3.safetensors architecture: MotionModuleType.AnimateDiffV3
2024-03-03 14:21:40,933 - AnimateDiff - INFO - Injecting motion module mm_sd15_v3.safetensors into SD1.5 UNet input blocks.
2024-03-03 14:21:40,934 - AnimateDiff - INFO - Injecting motion module mm_sd15_v3.safetensors into SD1.5 UNet output blocks.
2024-03-03 14:21:40,934 - AnimateDiff - INFO - Setting DDIM alpha.
2024-03-03 14:21:40,937 - AnimateDiff - INFO - Injection finished.
2024-03-03 14:21:40,937 - AnimateDiff - INFO - AnimateDiff + ControlNet will generate 16 frames.
fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
2024-03-03 14:21:41,122 - AnimateDiff - WARNING - Failed to get git tag for AnimateDiff: Command '['git', '-C', '/notebooks/sd/stable-diffusion-webui/extensions/sd-webui-animatediff/model', 'describe', '--tags']' returned non-zero exit status 128.
0%|                                                    | 0/20 [00:00<?, ?it/s]2024-03-03 14:21:41,182 - AnimateDiff - INFO - inner model forward hooked
0%|                                                    | 0/20 [00:00<?, ?it/s]
*** Error completing request
*** Arguments: ('task(08mwxvkg79dspoc)', <gradio.routes.Request object at 0x7f9e1a656520>, 'cat', '', [], 20, 'DPM++ 2M Karras', 1, 1, 7, 768, 432, False, 0.7, 2, 'Latent', 0, 0, 0, 'Use same checkpoint', 'Use same sampler', '', '', [], 0, False, '', 0.8, -1, False, -1, 0, 0, 0, <scripts.animatediff_ui.AnimateDiffProcess object at 0x7f9e1bd990a0>, UiControlNetUnit(enabled=False, module='none', model='None', weight=1, image=None, resize_mode='Crop and Resize', low_vram=False, processor_res=-1, threshold_a=-1, threshold_b=-1, guidance_start=0, guidance_end=1, pixel_perfect=False, control_mode='Balanced', inpaint_crop_input_image=False, hr_option='Both', save_detected_map=True, advanced_weighting=None), UiControlNetUnit(enabled=False, module='none', model='None', weight=1, image=None, resize_mode='Crop and Resize', low_vram=False, processor_res=-1, threshold_a=-1, threshold_b=-1, guidance_start=0, guidance_end=1, pixel_perfect=False, control_mode='Balanced', inpaint_crop_input_image=False, hr_option='Both', save_detected_map=True, advanced_weighting=None), UiControlNetUnit(enabled=False, module='none', model='None', weight=1, image=None, resize_mode='Crop and Resize', low_vram=False, processor_res=-1, threshold_a=-1, threshold_b=-1, guidance_start=0, guidance_end=1, pixel_perfect=False, control_mode='Balanced', inpaint_crop_input_image=False, hr_option='Both', save_detected_map=True, advanced_weighting=None), False, False, 'positive', 'comma', 0, False, False, 'start', '', 1, '', [], 0, '', [], 0, '', [], True, False, False, False, False, False, False, 0, False, None, None, False, None, None, False, None, None, False, 50) {}
Traceback (most recent call last):
File "/notebooks/sd/stable-diffusion-webui/modules/call_queue.py", line 57, in f
res = list(func(*args, **kwargs))
File "/notebooks/sd/stable-diffusion-webui/modules/call_queue.py", line 36, in f
res = func(*args, **kwargs)
File "/notebooks/sd/stable-diffusion-webui/modules/txt2img.py", line 110, in txt2img
processed = processing.process_images(p)
File "/notebooks/sd/stable-diffusion-webui/modules/processing.py", line 785, in process_images
res = process_images_inner(p)
File "/notebooks/sd/stable-diffusion-webui/extensions/sd-webui-controlnet/scripts/batch_hijack.py", line 48, in processing_process_images_hijack
return getattr(processing, '__controlnet_original_process_images_inner')(p, *args, **kwargs)
File "/notebooks/sd/stable-diffusion-webui/modules/processing.py", line 921, in process_images_inner
samples_ddim = p.sample(conditioning=p.c, unconditional_conditioning=p.uc, seeds=p.seeds, subseeds=p.subseeds, subseed_strength=p.subseed_strength, prompts=p.prompts)
File "/notebooks/sd/stable-diffusion-webui/modules/processing.py", line 1257, in sample
samples = self.sampler.sample(self, x, conditioning, unconditional_conditioning, image_conditioning=self.txt2img_image_conditioning(x))
File "/notebooks/sd/stable-diffusion-webui/modules/sd_samplers_kdiffusion.py", line 234, in sample
samples = self.launch_sampling(steps, lambda: self.func(self.model_wrap_cfg, x, extra_args=self.sampler_extra_args, disable=False, callback=self.callback_state, **extra_params_kwargs))
File "/notebooks/sd/stable-diffusion-webui/modules/sd_samplers_common.py", line 261, in launch_sampling
return func()
File "/notebooks/sd/stable-diffusion-webui/modules/sd_samplers_kdiffusion.py", line 234, in
samples = self.launch_sampling(steps, lambda: self.func(self.model_wrap_cfg, x, extra_args=self.sampler_extra_args, disable=False, callback=self.callback_state, **extra_params_kwargs))
File "/usr/local/lib/python3.9/dist-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "/notebooks/sd/stablediffusion/src/k-diffusion/k_diffusion/sampling.py", line 594, in sample_dpmpp_2m
denoised = model(x, sigmas[i] * s_in, **extra_args)
File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/notebooks/sd/stable-diffusion-webui/modules/sd_samplers_cfg_denoiser.py", line 237, in forward
x_out = self.inner_model(x_in, sigma_in, cond=make_condition_dict(cond_in, image_cond_in))
File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/notebooks/sd/stable-diffusion-webui/extensions/sd-webui-animatediff/scripts/animatediff_infv2v.py", line 151, in mm_sd_forward
out = self.original_forward(
File "/notebooks/sd/stablediffusion/src/k-diffusion/k_diffusion/external.py", line 112, in forward
eps = self.get_eps(input * c_in, self.sigma_to_t(sigma), **kwargs)
File "/notebooks/sd/stablediffusion/src/k-diffusion/k_diffusion/external.py", line 138, in get_eps
return self.inner_model.apply_model(*args, **kwargs)
File "/notebooks/sd/stable-diffusion-webui/modules/sd_hijack_utils.py", line 18, in
setattr(resolved_obj, func_path[-1], lambda *args, **kwargs: self(*args, **kwargs))
File "/notebooks/sd/stable-diffusion-webui/modules/sd_hijack_utils.py", line 32, in call
return self.__orig_func(*args, **kwargs)
File "/notebooks/sd/stablediffusion/ldm/models/diffusion/ddpm.py", line 858, in apply_model
x_recon = self.model(x_noisy, t, **cond)
File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/notebooks/sd/stablediffusion/ldm/models/diffusion/ddpm.py", line 1329, in forward
out = self.diffusion_model(x, t, context=cc)
File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/notebooks/sd/stable-diffusion-webui/modules/sd_unet.py", line 91, in UNetModel_forward
return original_forward(self, x, timesteps, context, *args, **kwargs)
File "/notebooks/sd/stablediffusion/ldm/modules/diffusionmodules/openaimodel.py", line 776, in forward
h = module(h, emb, context)
File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/notebooks/sd/stablediffusion/ldm/modules/diffusionmodules/openaimodel.py", line 86, in forward
x = layer(x)
File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/notebooks/sd/stable-diffusion-webui/extensions/sd-webui-animatediff/motion_module.py", line 132, in forward
return self.temporal_transformer(x)
File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/notebooks/sd/stable-diffusion-webui/extensions/sd-webui-animatediff/motion_module.py", line 190, in forward
hidden_states = block(hidden_states)
File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/notebooks/sd/stable-diffusion-webui/extensions/sd-webui-animatediff/motion_module.py", line 244, in forward
hidden_states = attention_block(norm_hidden_states) + hidden_states
File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/notebooks/sd/stable-diffusion-webui/extensions/sd-webui-animatediff/motion_module.py", line 341, in forward
x = torch.nn.functional.scaled_dot_product_attention(q, k, v)
AttributeError: module 'torch.nn.functional' has no attribute 'scaled_dot_product_attention'

Yes, from what I gathered the Torch version on this repo is out of date for the newer Animatediff models like you said. I think TheLastben had posted it was a paperspace issue originally.
I know of one other Repo that currently has 2.0+ that I haven't tried, but am not sure of any other easy Paperspace ones.

Note, thanks to Srage(https://github.com/TheLastBen/fast-stable-diffusion/issues/2670#issuecomment-1983323737), he has the command needed to raise the torch version to the right one to run Animated diff.
However, if you update it and use the latest module then you get stuck with the following error:

NansException: A tensor with all NaNs was produced in VAE. Use --disable-nan-check commandline argument to disable this check.

Here's a workaround.
1) Run the notebook as usual.
2) Open a terminal from the icon on the left menu bar.
3) Paste this and press enter: pip install torch==2.1.2 torchvision==0.16.2 torchaudio==2.1.2 xformers==0.0.23post1
4) Restart the kernel and rerun the notebook. The updated packages will now persist.

@dreamlogic-X, if you strictly just want to run AnimateDiff as a note, the versions for comfyUI don't appear to have any issues in the meantime just as a heads up.

Hi @pdannell thanks for your help... I've been using the repo of Stable Diffusion/animatediff before things were broken (I had a backup) and that's working for me. Although not updating is not the best solution.
Unfortunately now ComfyUI doesn't want to load... https://github.com/TheLastBen/fast-stable-diffusion/issues/2786
Same on your end?


initial article

Initial KB article

Commit messages need to be descriptive so that when we look back in history we can identify what the purpose of the commit was. initial kb article will not work.


Getting BadRequest exception when creating private_endpoints

Package Name:  azure.mgmt.network  -> NetworkManagementClient
Package Version: v10.2.0 -- API v2020_04_01
Operating System: MacOS
Python Version:  3.6
Describe the bug
I am getting s Bad request error with the following private endpoint creation :

        async_pe_test_creation = network_client.private_endpoints.create_or_update(
            resource_group_name="test",
            private_endpoint_name=resource_name,
            parameters=PrivateEndpoint(
                location= location,
                subnet=Subnet(id=snet.id),
                manual_private_link_service_connections=[
                   PrivateLinkServiceConnection(
                    name=resource_name,
                    private_link_service_id=resource_id,
                    group_ids=["blob"]
                    )
                ]
            ),
        )

The exception I am getting is:
azure.mgmt.network.v2020_04_01.models._models_py3.ErrorException: Operation returned an invalid status code 'Bad Request'
To Reproduce
run the snippet above.
Expected behavior
Creation of a private_endpoints in Azure cloud.

@timMSFT , Hi there! Any update about this issue?

While searching for a similar solution, I found that the "Bad Request" may be a result of the virtual_network and/or subnet not being set to allow private_endpoints properly.
By befault, virtual_networks and subnets look to have this property enabled... and in order to get it to work correctly, it must be disabled...  (or at least that's what the azure portal seems to do before creating a private endpoint for my Sql Server via the portal).

'private_endpoint_network_policies': 'Disabled'

I found this out while playing with Azure SQL Server creations in the azure portal, and noticed the "blurb" about:
"if you have a network security group (NSG) enabled for the subnet above, it will be disabled for private endpoints on this subnet only"
... and then i compared subnets and vnets that were working versus not working.
I would guess that if you modify the virtual_network/subnet to set this property 'Disabled' first you won't get the "Bad Request".
In my case, I just created a sql server with a private endpoint in this virtual network from the portal first, and let it handle the modifications for me..  now I need to go back and write the python to update the subnet/vnet so I can create the private endpoint using any vnet/subnet (handling the prerequisite modification first - if necessary).
HTH, but if this doesnt solve your problem, I hope MS gets back to you with a solution.
My $.02

@smanross thanks, I'll give it a short and let you know


replaced type with model and typeKey with modelName

Correct deprecation messages:

DEPRECATION: Usage of typeKey has been deprecated and will be removed in Ember Data 1.0. It > has been replaced by modelName on the model class

Ember      : 1.12.0
Ember Data : 1.0.0-beta.18
jQuery     : 1.11.3

I replaced the "type" variables with "model" variable which seemed more accurate.

@mylen I'm sorry, but a new PR came out with this and I didn't realize yours was open :grin:
Here's the original. https://github.com/kurko/ember-localstorage-adapter/pull/121
I think we can close this one, but feel free to rebase and add anything that could be missing from the PR above :smile:

No worries as long as it gets out ;)
Le mer. 17 juin 2015 à 16:52, Alexandre de Oliveira <
<EMAIL_ADDRESS>a écrit :

@mylen https://github.com/mylen I'm sorry, but a new PR came out with
this and I didn't realize yours was open [image: :grin:]
Here's the original. #121
https://github.com/kurko/ember-localstorage-adapter/pull/121
I think we can close this one, but feel free to rebase and add anything
that could be missing from the PR above [image: :smile:]
—
Reply to this email directly or view it on GitHub
https://github.com/kurko/ember-localstorage-adapter/pull/118#issuecomment-112829987
.


Slack: RTM Error - received unmapped event msg=invalid_auth code=401

Hi,
Sometime during last week, I've noticed that matterircd slack support stopped working for me, with reocurring logging like this:
slack-go/slack2023/09/26 19:26:23 websocket_managed_conn.go:86: RTM connection succeeded on try 0
slack-go/slack2023/09/26 19:26:24 websocket_managed_conn.go:411: Incoming Event: {"type":"error","error":{"msg":"invalid_auth","code":401,"source":"gatewayserver-fra-44"}}
slack-go/slack2023/09/26 19:26:24 websocket_managed_conn.go:501: RTM Error - received unmapped event "error": {"type":"error","error":{"msg":"invalid_auth","code":401,"source":"gatewayserver-fra-44"}}
slack-go/slack2023/09/26 19:26:24 websocket_managed_conn.go:243: killing connection signaled
...

slack-go/slack2023/09/26 19:27:24 websocket_managed_conn.go:86: RTM connection succeeded on try 1
slack-go/slack2023/09/26 19:27:25 websocket_managed_conn.go:411: Incoming Event: {"type":"error","error":{"msg":"invalid_auth","code":401,"source":"gatewayserver-lhr-2"}}
slack-go/slack2023/09/26 19:27:25 websocket_managed_conn.go:501: RTM Error - received unmapped event "error": {"type":"error","error":{"msg":"invalid_auth","code":401,"source":"gatewayserver-lhr-2"}}
slack-go/slack2023/09/26 19:27:25 websocket_managed_conn.go:243: killing connection signaled
slack-go/slack2023/09/26 19:27:25 websocket_managed_conn.go:205: Connecting to RTM

Eventually rate-limiting seem to be added to such messages:
slack-go/slack2023/09/25 07:11:57 websocket_managed_conn.go:181: Connecting to RTM
slack-go/slack2023/09/25 07:11:57 rtm.go:71: Using URL: wss://wss-primary.slack.com/websocket/LEGACY_BOT:94c5a4615eb29a760a543cb2a10820c0
slack-go/slack2023/09/25 07:11:57 websocket_managed_conn.go:197: Dialing to websocket on url wss://wss-primary.slack.com/websocket/LEGACY_BOT:94c5a4615eb29a760a543cb2a10820c0
slack-go/slack2023/09/25 07:11:57 websocket_managed_conn.go:62: RTM connection succeeded on try 8765
slack-go/slack2023/09/25 07:11:58 websocket_managed_conn.go:387: Incoming Event: {"type":"error","error":{"msg":"invalid_auth","code":401,"source":"gatewayserver-fra-44"}}
slack-go/slack2023/09/25 07:11:58 websocket_managed_conn.go:477: RTM Error - received unmapped event "error": {"type":"error","error":{"msg":"invalid_auth","code":401,"source":"gatewayserver-fra-44"}}
slack-go/slack2023/09/25 07:11:58 websocket_managed_conn.go:219: killing connection signaled
slack-go/slack2023/09/25 07:11:58 websocket_managed_conn.go:181: Connecting to RTM
slack-go/slack2023/09/25 07:11:58 rtm.go:67: Failed to connect to RTM: slack rate limit exceeded, retry after 1m0s
slack-go/slack2023/09/25 07:11:58 websocket_managed_conn.go:185: Failed to start or connect to RTM: slack rate limit exceeded, retry after 1m0s
slack-go/slack2023/09/25 07:11:58 websocket_managed_conn.go:151: reconnection 1 failed: slack rate limit exceeded, retry after 1m0s reconnecting in 1m0s

Tried re-authorizing using tokens from the current browser session, and while matterircd's "slack" user sends login OK token used: xoxc-...|d=xoxd-...; messages (really wish it wouldn't be sending auth credentials in plaintext to me like that), and sending messages to slack seem to work, I think receiving msgs does not work, as these errors don't seem to go away, though I didn't test it today (don't have any special slack-for-testing setup).
I was always only using "slack sso login / xoxc tokens" authentication method (two tokens from browser's cookie/post-request), and don't think I had similar issue within about a year of running matterircd.
Tested with 0.25 and 0.27.1 - both seem to have same issue.
Debug log from starting 0.25 to the message, in case it might be useful, with anything sensitive redacted inline: matterircd.debug.2023-09-26.log
Not sure what else to try with this issue, is it likely to be a change in Slack's "LEGACY_BOT" API endpoint, which slack-go seem to be trying to use? (it sounds like something that might've been deprecated)
I've looked at slack-go/slack issues and PRs, but haven't been able to find anything relevant there.
Thanks.

After a bunch more experimentation, I've also got general login errors (not getting to login OK), and at this point not convinced that it's not just me, my slack account, and/or my weird irc setup.
So unless someone bumps into same issue, it's probably not worth looking into. Sorry for the noise.

Not a slack user here but here's my 2c.
It's likely tokens (or auth cookie) have a lifetime / expiry time and the token used probably expired.

Yeah, that was my first thought as well (as also mentioned in the msg - "Tried re-authorizing using tokens from the current browser session"), but didn't seem to help, with that strange "login OK" but apparently not-ok result.
But iirc that's only after matterircd restart, while straight-up "login " to "slack" initially timed-out (maybe because of some rate-limiting).
I've also noticed that I had an old token sent by ZNC in PASS after that, and after putting fresh token in both places, I couldn't reproduce even "login OK" result.
And then I also vaguely remembered that I had issue with copying those with- and without- url-encoding, so maybe I messed that up...
So I think it's probably my fault somewhere, and maybe after hammering slack with expired token for days, there's just some straight-up anti-bruteforce rate-limiting in action, which is why I thought to close this and only reopen if I'll try all the things in more rigorous order :)

Hi!
I am trying matterircd 0.28.0 for the first time, and I am experiencing this exact same problem. I am using a token and cookie I extracted a few minutes ago from a private firefox tab, I tried passing them as password or as /msg slack <token>|<d=cookie;>, and I keep seeing this authentication errors when connecting to the "RTM". I can send messages, but I don't see anybody else's.
So this looks like a real bug. Isn't anybody else affected?

Some interesting update: I tried logging in with workspace+email+password (I had to set one first, as they don't ask you to set one by default), and that worked!
So it seems it is a problem that only affects logging in with the token and cookie


Fixed p val in example Spatiotemporal permutation F-test

I just changed the p_value for accepted clusters in the   spatiotemporal permutation F-test example so it can actually find some clusters to plot (there is no cluster figures showing up as it currently is on the website).

Great, thanks for fixing this. Will merge once CircleCI comes back happy and the example looks good. (If you happen to see it does, feel free to post the CircleCI artifact link here, which is visible if you follow the CircleCI link and login via GitHub.)

thx @claire-braboszcz


feat: support stdin when creating new migration

What kind of change does this PR introduce?
feature
What is the current behavior?
supabase migration new <name> only creates a new file
What is the new behavior?
supports reading from stdin so pipe works, for eg.
echo "create table pet;" | supabase migration new create_pet_table
supabase db changes | supabase migration new my_diff

Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 16 (81.25%)  changed or added relevant lines in 3 files are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.003%) to 22.568%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/db/commit/migra.go
2
3
66.67%

internal/migration/new/new.go
10
12
83.33%

Files with Coverage Reduction
New Missed Lines
%

internal/utils/misc.go
6
34.42%

Totals

Change from base Build<PHONE_NUMBER>:
-0.003%

Covered Lines:
1160

Relevant Lines:
5140

💛  - Coveralls


Pagination?

is there a configuration to doing pagination?
something like:
http://jsonplaceholder.typicode.com/posts?page=2&limit=10&sort=latest

Hi @antoniputra,
You can use _start and _limit:
http://jsonplaceholder.typicode.com/posts?_start=10&_limit=10
There's also a custom header X-Total-Count in the response.

Awesome. :+1:

I came across this when searching if/how the JSONPlaceholder API supported paging.
I implemented a codesandbox example that uses the _page and _limit parameters to implement paging:
https://codesandbox.io/s/api-pagination-with-react-router-hooks-i0912
In general, the API supports everything implemented by JSON Server. You can check that project's README/docs for paging/sort/filter/search related parameters: https://github.com/typicode/json-server


Added control and rule for Kubernetes Aggregated API Server Redirection Vulnerability [CVE 2022-3172]

Summary
This PR adds control and rule for Kubernetes Aggregated API Server Redirection Vulnerability [CVE 2022-3172].
https://github.com/kubernetes/kubernetes/issues/112513

Hey @0xquark , I think it is important to create test for every control.
We have a small test framework, @Daniel-GrunbergerCA , can you help us?

@slashben Thank you for the review. I've added the test but i am unable to setup the test environment locally and test the changes. The kubescape documentation for testrunner redirects to same page while clicking on setting the environment
https://github.com/kubescape/regolibrary/blob/master/testrunner/README.md#environment-setup
Is there any guide to setup the env? I am not able to install the package testrunner/opaprocessor

Hi @0xquark, and thanks for your contribution. This is really appreciated.
I checked the link you provided and you're right, the link is broken.
Here's the original content: https://github.com/kubescape/regolibrary/commit/bcf380c824b7c3f6a126600857afe65f3995e51d#diff-a1d3a8dc2081437735162c3c668a7f3f39f9c5dbc9061fc8a2aa33c3a9157d63
I'm not sure this is still needed to test the environment. Anyway, try to test it and let us know.
:)

@alegrey91 thanks!  How can i install testrunner/opaprocessor package?

@0xquark actually you should not need to install it.
Are you encountering some error message?

rego_test.go:7:2: cannot find package "testrunner/opaprocessor" in any of: 	/opt/homebrew/Cellar/go/1.19.5/libexec/src/testrunner/opaprocessor (from $GOROOT)

@0xquark actually you should not need to install it. Are you encountering some error message?

rego_test.go:7:2: cannot find package "testrunner/opaprocessor" in any of: /opt/homebrew/Cellar/go/1.19.5/libexec/src/testrunner/opaprocessor (from $GOROOT) The opaprocessor is already present in the folder, i think the problem should be with the GOPATH Edit : Solved it! but other dependencies are causing similar issues!
Can you please share how did you solve the problem?
Which are the other dependencies that are causing the problem?
Can you share the error message?

rego_test.go:7:2: cannot find package "testrunner/opaprocessor" in any of: /opt/homebrew/Cellar/go/1.19.5/libexec/src/testrunner/opaprocessor (from $GOROOT) The opaprocessor is already present in the folder, i think the problem should be with the GOPATH Edit : Solved it! but other dependencies are causing similar issues!

Great! Can you please share how did you solve the problem? Which are the other dependencies that are causing the problem? Can you share the error message?
I added the opaprocessor to my GOPATH and it solved it.
Now i am having these # command-line-arguments

package command-line-arguments (test)
	imports testrunner/opaprocessor
	imports github.com/kubescape/k8s-interface/workloadinterface
	imports github.com/armosec/armoapi-go/apis
	imports github.com/coreos/go-oidc: no Go files in /Users/karanjotsingh/go/src/github.com/coreos/go-oidc

The last one is maybe because go-oidc recently changed their import path ( https://github.com/coreos/go-oidc )

ah! My bad! I forgot to run it that's why the missing packages
Thanks!

You're welcome ;)

I tried running tests for the policy and it is resulting in an unmarshaling error
in parseRegoResult,
json.Unmarshal failed. name: vulnerable_version, obj: [1.21.14 1.22.0-1.22.13 1.23.0-1.23.10 1.24.0-1.24.4 1.25.0],
reason: json: cannot unmarshal string into Go value of type reporthandling.RuleResponse in rule: test-single-rego/input

I tried fixing this by changing the structure of vulnerable_version to be a list instead of object but it turns out to be no good. From my understanding, it indicates that json.Unmarshal cannot unmarshal the variable's value into a reporthandling.RuleResponse type. I think vulnerable_version is being interpreted as a reporthandling.RuleResponse instead of a set of strings as intended.
@alegrey91 Could you help me in here ?

Whenever i run tests using TestSingleRego, it generates an empty struct []. This happens for all the policies i enter in testdir

This PR is ready for review 🚀

@alegrey91  Ping :)

This PR is ready for review 🚀

Sorry for the late reply. We need few days to make some checks and then we can finally take a look over it.

This PR is ready for review 🚀

Sorry for the late reply. We need few days to make some checks and then we can finally take a look over it.

Great! Let me know if you run into any issues :)

@Daniel-GrunbergerCA , what is the difference between this control and C-0089?

Hey @slashben, i believe this control is already implemented and i might've missed it while while looking for any previously implemented controls before implementing this. However this control also includes a rule which checks for the vulnerable versions of api servers and if it is vulnerable then it prompts the user with deny message.
I will be moving on to implementing new controls or rules. Also working on this control helped me get a better understanding of rego.

Hey @0xquark
First of all, sorry for the delay to respond, it's been a few busy days :)
This control is indeed implemented already, but we really appreciate the fact that you are looking into contribute to our project! Maybe you want to share with us the controls that you think should be implemented for next PRs?
Also I have a question about your PR: are you sure that the check ver[_] = api_name works for the ver list that you defined? For example, would it work for "1.22.0-1.22.13"?


Convert CONST properties to Branded types where appropriate

Lots of properties in CONST are currently in the form
export declare const ENUM_NAME: Readonly<{
  prop1: T;
  prop2: T;
}>
export type ENUM_NAME = ValueOf<typeof ENUM_NAME>

The repo has been moving towards using the new Brand type like so:
export type ENUM_NAME: Brand<T, "Namespace.ENUM_NAME">
export declare const ENUM_NAME: Readonly<Record<"prop1" | "prop2", ENUM_NAME>>;

This works for properties that are already effectively Record<string, T>, but I am not sure what we want to do with arrays. There's maybe value in preserving the literal values of things like CONST.WORLD_DOCUMENT_TYPES, etc.

The various arrays have a lot of value in not collapsing to string

Yeah I probably would recommend not converting arrays. They can act like pseudo-enums but you're supposed to refer to them by value. Going around and doing WORLD_DOCUMENT_TYPES[3] would be silly, just write "Combat".


Focus is not handled correctly with portals

Describe the bug
When using datepicker with portals, focus does not go into datepicker from input
To Reproduce

Go to https://reactdatepicker.com/#example-portal-by-id
Click input
Click "tab"

Actual behavior
Focus goes to the next tabable object (this case textarea). If you have multiple date inputs by each other - multiple datepickers will open.
Expected behavior
Focus goes into datepicker like with example where portal is not used
Desktop (please complete the following information):

OS: [Win10
Browser Chrome
Version  99.0.4844.74

Also noticed same behavior occurs even without portals, when "Shift + Tab" is clicked:
datepicker remains open -> focus goes to other input and may open another datepicker if input is for date.
And now thinking of this form UX side, maybe it should never go into datepicker when using Tab, because that would be annoying if user needs to tab trough bigger form with few date inputs and needs to escape on each...
And if he want to enter datepicker, he uses arrows (that works now)
Still datepicker then needs to close when user goes to other input

This is a significant issue, effectively making react-datepicker inaccessible in cases where the date picker is used with portals (a common scenario).

actually it as accessible with arrows, and that would be preferred over tabbing into, because if user tabs over form, he don't want to get stuck into some date field that is not even what he want's to fill in.
Bigger problems are that this:

works inconsistently with and without portals or even tab versus shift+tab.
datepicker is left open after user tabs to another input  with portals or uses shift+tab without portals

Same Problem here!!! We have to use Portals to be sure the Datepicker is rendered correctly in our application since there are possibilities that somewhere in the App position: relative is set. See example: https://stackblitz.com/edit/react-ts-vfxgwp?file=App.tsx
Problem seems to be this code in the ReactDatepicker index.js:
`  handleBlur = (event) => {
if (!this.state.open || this.props.withPortal || this.props.showTimeInput) {
this.props.onBlur(event);
}
this.setState({ focused: false });

};`
would it help if the if statement is extended with the portalId?
if (!this.state.open || this.props.withPortal || this.props.portalId || this.props.showTimeInput)


CAPI All Endpoints Fix

🚀 Deployed on https://65cbd5f1792118213d3d732f--cozy-fox-0defec.netlify.app


Corrected url to leader pod

/lgtm

@NothingButTheHoff can you sign the CLA?

/remove-lifecycle rotten

I signed it

I signed it


Bump version: 0.6.8 → 0.6.9

@luistorresm
Could we update the version to finish this issue?

Updated in https://github.com/Vauxoo/cfdilib/commit/8b5912a4cb14069bba1baefb70c323e9a0b55220


How can I test the spring boot application local like sam build and sam local start-api

Framework version: XX
Implementations: Jersey / Spring / Spring Boot / Spark

Scenario
I downloaded this code but I dont know how to test it locally . something line sam build and then sam local start-api and then run the request from the postman
Expected behavior
it should be testable from local
Actual behavior
sam build needed template.yaml but we have sam.yaml
Steps to reproduce
download the code and do sam build and it fails saying that no template.yaml found
Full log output

i found the solution I must use the --template paramater to specify the template file as sam.yaml

This is related to #293 - we'll use the original issue to track fixes to the sample so that they work out of the box with the SAM cli.


Fix Metrics/ClassLength violation

Our TODO file mentioned that the DescribedClass class was too long. Looking closer at it, I found a couple of methods (#full_const_name, #collapse_namespace, #const_name, and #namespace) working on the isolated problem of determining the “full constant name”, of a const node. This PR extracts those methods into a separate FullConstNameFinder module.
The code that I move was introduced in one single commit, bdc8723c65a0521e6df3e0b481592d19fdc000ca, and I wonder if @pirj wouldn’t like to help me with this here PR? 😄
TODO

[ ] Code documentation.
[ ] Add tests specifically for the extracted module.
[ ] Consider extracting the code into a separate file so it can be re-used elsewhere.

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[ ] Added tests.
[x] Updated documentation.
[ ] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

[ ] Code documentation

Looks pretty good already.

[ ] Add tests specifically for the extracted module

We don't have much tests for modules, we rely on cops' tests to cover module's behavior, so we're good here from my point of view.

[ ] Consider extracting the code into a separate file so it can be re-used elsewhere

Did you have some other cops in mind that could use this? I can't think of any.


add format: time and date

Just added format: time and date because I needed it for my project.
json-forms is really awesome to get rid of setting up forms for all those user settings!
Thank you!

Thanks man, appreciate your words!


cant see the clearly and suggest it to be more transparent

For me at least it is transparent enough, I don't see the problem of making it more transparent. Maybe if there is something like a slider to make it more or less transparent(again this might be hard to do)

For me at least it is transparent enough, I don't see the problem of making it more transparent. Maybe if there is something like a slider to make it more or less transparent(again this might be hard to do)

For me at least it is transparent enough, I don't see the problem of making it more transparent. Maybe if there is something like a slider to make it more or less transparent(again this might be hard to do)

This is like a suggestion from Chinese players and maybe he is having some problem on looking at the model

For me at least it is transparent enough, I don't see the problem of making it more transparent. Maybe if there is something like a slider to make it more or less transparent(again this might be hard to do)

This is like a suggestion from Chinese players and maybe he is having some problem on looking at the model

This is a feature (transparency slider) in the v1.1.1 Pre-Release and will likely be in the next full release.

This is a feature (transparency slider) in the v1.1.1 Pre-Release and will likely be in the next full release.

Will be in the next release, as it is a UI change i dont want to drop it on people with out some sort of video to explain the changes or at least some documentation changes.
you can see it if you click the button under releases and click pre-release 1.1.1 git hub gently hides pre-releases

Will be in the next release, as it is a UI change i dont want to drop it on people with out some sort of video to explain the changes or at least some documentation changes.
you can see it if you click the button under releases and click pre-release 1.1.1 git hub gently hides pre-releases

Closing as i have this preped for next release

Closing as i have this preped for next release


reset gh-pages and enable clearing preview PR commits

Documentation from Entropies.jl was never "publicly announced". Anything there is weighting down on the git size of this repo. We should reset the gh-pages branch just before tagging 2.0.
Furthermore, there is a single CI action that cleans preview PRs so that their content is also not saved. We do this in Agents.jl and we should do it here as well.

Furthermore, there is a single CI action that cleans preview PRs so that their content is also not saved. We do this in Agents.jl and we should do it here as well.

Preview PRs should be deleted, yes. As long as the process doesn't also automatically delete the documentation branches for coming 2.x releases and higher

TODO:

[ ] Reset the gh-pages branch just before tagging 2.0.
[ ] Clean preview PRs with Github Actions. Fixed by #142.


FEAT: Parallel rule execution

Parallel rule execution can now be controlled with the 'parallel' Rule parameter.

Closing this PR as the feature was replaced with the CLI-based implementation. The 'parallel' Rule argument doesn't completely control whether the rule is run in parallel or not, so having a global CLI option is more in line with the actual behavior.
May revisit this idea again if Make was someday replaced by something more flexible.


Fix memory leak with :sign argument repeated

Silently dropping a duplicate argument doesn't look good.
It makes it hard to spot mistakes, especially since the names are so similar.
It would be better to give an error for a duplicate argument.

But will this break backward-compatibility, in case one constructs a command using string concatenation?

But will this break backward-compatibility, in case one constructs a
command using string concatenation?

Isn't that exactly what the error should point out?
I don't think anyone would have a duplicate argument intentionally.
Not giving an error because there might be an old mistake would mean we
could never fix problems like this.  I also think "backwards compatible"
isn't the right name for this.  "bug-for-bug compatible"?
--
ARTHUR:  Then who is your lord?
WOMAN:   We don't have a lord.
ARTHUR:  What?
DENNIS:  I told you.  We're an anarcho-syndicalist commune.  We take it in
turns to act as a sort of executive officer for the week.
The Quest for the Holy Grail (Monty Python)
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///