Support drag-and-drop between checklists

It's currently only possible to drag items to reorder them within the same checklist - it'd be nice to be able to drag an item into another checklist on the left-side checklist tree.
This likely involves some fighting with Material's drop target and tree to get right.

Another related feature is drag and drop of checklists and groups on the left-side tree. Those two probably need to be built together.


luci-app-openvpn: Missing route-nopull option

Unfortunately route-nopull is required for mwan3 usage. Without this, luci-app-openvpn is not usable with mwan3, or any environments where OpenVPN isn't the default router.
https://wiki.openwrt.org/doc/howto/mwan3#example_2use_openvpn_tunnels_as_virtual_wan_s_client_mode

I added it to LuCI but no idea if the OpenWrt package supports it.


Generate dart classes based on schema definitions

I am wondering if ferry or any other tool in dart ecosystem is capable in any way of generating types based on schema definitions? I cannot seem to find any documentation or related questions on this.
for example:
Given the following piece of scheme:
type Person {
    age: Int
    name: String!
}

Can it be converted to sound null safe dart class like this:
class Person{
  int? age;
  String name;

  Person({this.age, required this.name});

  Person.fromJson(Map<String, dynamic> json){
    age = json['age'];
    name = json['name'];
  }

  Map toJson(){
    Map _data = {};
    _data['age'] = age;
    _data['name'] = name;
    return _data;
  }
}

Ferry's generated data types only include the fields defined by their respective queries.
The only way to generate a class with all fields defined in the schema would be to define a query with each of those fields.
I'm curious, what is your use case?
You may also want to check out artemis.

First of all thanks for the fast reply and the awesome project.
Actually this was the default behaviour in a generator I used in JS for a some time but I actually do not have a current use case and I do not think I will have one.


WEBP issue

This has so far been an invaluable little tool for me.
However, it appears to not work when Midjourney creates a webp output instead of png.
I have tried it multiple times and it just ignores those files.
Renaming them to png works however.

Hmm, that's strange as I did update the code to manage webp, make sure you have the most up to date commit! :)

Oh, actually it's set up to do webm's, I'm not sure. I'll have a look at the format and if needed I will update it, or you can take a look at the code! :) I will take a look asap thanks for bringing it up


Update for new release -- automating the update.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@notestaff , I forgot already the "procedure" for update of the nodep (standalone) build. If you recall/could point me to detail - I will finish up the update script

woohoo -- it built, I must have done update script right! will merge later today unless there is some feedback saying "No"


standardize volumes and labels effecting runtime behavior?

Hey it's super exciting to see whalebrew! I've been working on a similar concept that also uses labels to augment runtime and a /woirkdir concept, and think it would be cool to make my project compatible with whalebrew images. In a perfect world we (the community) could create, publish, and pull from eachothers images.
Admittedly dex hardly has a community outside my workplace ATM. It also suffers documentation clarity issues and the runtime is a work-in-progress(I kind of stalled on darwin/MacOS X11 and am worried about over/under architecting some labels) -- but I wanted to reach out nonetheless! :) Do you think sharing thought-space solving these issues (e.g. label names and behaviors, mount points, &c) worth pursuing?

@bfirsh hey I finally got around to documenting the runtime labels in dex; https://github.com/dockerland/dex/blob/master/docs/v1-runtime.md#runtime-labels
These were my ideas around providing convenience around common behavior (e.g. toggling X11 support) as well as providing flexibility for customizations (exposing devices and volumes). Not sure what is/is not needed (it's a work in progress) but thought it would be cool to come up with a standard.


fix: add post and get route to achievements and display data in the UI

#8

added post route to add new achievements
added get route to get data of achievements
initialized firebase storage
displayed the achievements in the UI

@TusharMohapatra07 have u pushed firebase.ts file changes?
The build failed cuz of tht

@mohit-nagaraj please check with #24 if it's building successfully


Update premium-storage-performance.md

@roygara

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


feature/pos-networking-and-syncing

Current dependencies on/for this PR:

feature/proof-of-stake

PR #899   👈

PR #857

PR #879

PR #860

PR #861

PR #858

PR #859

This stack of pull requests is managed by Graphite.


Change frontside's domain to .com

Motivation
We are moving Frontside's website to .com instead of .io
Approach
Change all the occurrences of frontside.io for frontside.com.
⚠This PR should only be merged once records are changed on the domain and emails.
Immediately after merging this PR

[x] Rename the repo to frontside.com
[x] Change repo's description to frontside.com
[x] Update Twitter's profile link to frontside.com
[x] Update LinkedIn profile link.
[ ] Change Org's URL and email: https://github.com/thefrontside (needs special org permissions)

@cowboyd can you change our Org's URL and email: https://github.com/thefrontside (needs special org permissions)


how to suppress output lines works

trying to supress output lines from my EXPECT script. Have echo(false) statement as an first line of script (as in examples) but it still printing output lines. What is the logic of echo()statement? if it once set up to false, does it supress output until its set to true? have it false before any send command, but it does not help.
Also tried @expect.exe "%~f0" >NUL or @expect.exe "%~f0 | break" but does not work for some reason

How about @expect.exe "%~f0" >NUL 2>NUL ?

How about @expect.exe "%~f0" >NUL 2>NUL ?

This syntax is invalid, valid redirection both STDOUT and STDERR to null is >NUL 2>&1
But my problem is, the script does not work, when discard STDOUT. It produce serie of errors:
expect.lua for Windows v0.6.2
The handle is invalid.
The handle is invalid.
The handle is invalid.
The handle is invalid.
The handle is invalid.
The handle is invalid.
The handle is invalid.
The handle is invalid.

and hang until CTRL+C wasnt pressed. But inside of LUA script execution, the echo(true/false)should control output lines echoing, or am I miss something?

The echo function does not control all of output.
When echo(true) is called, the spawn , send , and sendln functions print its arguments to STDOUT.
echo(false) disables them.
It does not control the output of child processes and displaying the software version.

This syntax is invalid, valid redirection both STDOUT and STDERR to null is >NUL 2>&1

It has no problems on the syntax of CMD.EXE because NUL is not a plain file to need to join the two outputs. To drop outputs, it works as we expected.

The handle is invalid.

Sorry, it is the error when expect function is called. It reads the terminal information linked to STDOUT.
Therefore, when STDOUT is not a terminal, its fails.

I will try to create single batch file as an EXPECT scripting function, call it from main batch by
call expect.cmd "%ip_addr%" >NUL
this could be a way to make EXPECT silent

Differently from /usr/bin/expect of Linux, Expect.lua does not reads STDOUT and STDERR of child processes via pipelines. It reads the console buffer of the terminal directly.
Therefore, if child processes outputs nothing to console by redirecting , the expect function never works as we hope because they can read nothing from console.
It is difficult for Expect.lua to satisfy your requests.


Remove legacy search app template

Word is this is unused and unneeded.
Removals

Remove legacy search app template

How to test this PR

I'm told @willbarton would know.

@willbarton: @anselmbradford asked me about this and I thought it made sense to clean up, since we can always retrieve from git history if need be.
@anselmbradford: if we're do this I think we can and should also remove a few more related things:

these Django settings,
this feature flag,
this whole urls.py,
this reference to it,
these environment variables, and
these tests.

Thanks for tracking these bits down @chosak
Removed in https://github.com/cfpb/consumerfinance.gov/pull/6392/commits/07f428840b4b471a6eb470cc01099c3ef3ad35ef


Uncaught SyntaxError: Unexpected token _ in JSON at position 0

Hi!
Got alot of error messages in console when using this module:
Uncaught SyntaxError: Unexpected token _ in JSON at position 0(…) ngStorage.js:220
event.newValue ? $storage[event.key.slice(prefixLength)] = deserializer(event.newValue) : delete $storage[event.key.slice(prefixLength)];
Changed to
event.newValue && !angular.isUndefined($storage[event.key.slice(prefixLength)]) ? $storage[event.key.slice(prefixLength)] = deserializer(event.newValue) : delete $storage[event.key.slice(prefixLength)];
Now there's no error messages :)

i have this issue too
i use angular and iframes on one page, and all pages use local storage, and this error always happens
there is some event come from angular into ngstorage and its value starts with _

but if its just one app page with no iframes, everything is fine

I have the same problem and I think that's because some parts of the application don't store JSON serialized values in the localStorage and ngStorage tries to deserialize every value as JSON (https://github.com/gsklee/ngStorage/blob/843d78ec719410850d0df136307189a5a3c07f7a/ngStorage.js#L117).
I think you could try to set your own deserializer which catches these errors:
var deserializer = function (value) {
    try {
        return JSON.parse(value);
    } catch (err) {
        // We expect that the error is a JSON parsing error.
        return value;
    }
};


REST API support for DID operations

In order to connect a running Agent to a non-von-network ledger, it would be useful to have some wallet/DID operations exposed through the API.

[ ] Create a new DID (from a random seed)
[ ] List existing DIDs and verkeys
[ ] Nominate a DID as the current public (ledger) DID

It would also be nice to return the active DID from the /status endpoint.

Resolved via #102


Restoring Rancher (HA) steps by step

Hi,
I read some guides about Backup/Restoring Rancher/Cluster in HA, but I am not clear about them, some of them only tell about snapshot. Could you please provide steps by steps for this ?
My system uses custom cluster and includes: 1 rancher server, 3 etcd nodes, 2 master nodes, 2 worker nodes.

What should I backup ? (only etcd snapshot is enough ?)
What will happen if my rancher server node die ? How can I recovery
What will happen if one of my etcd nodes  die ?   How can I recovery
How can I move the system to another places ?

Many thanks !

Useful
Info

Versions
Rancher v2.2.7 UI: v2.2.93

Route
global-admin.clusters.index

The process is different depending on if you are talking about the Rancher Server HA cluster itself - or the clusters you are managing with Rancher.
For the cluster hosting Rancher Server in HA, you use RKE to backup and restore.  You also need a copy of your rkestate file and cluster.yml file.  Full docs here
For the child clusters - you can use the cluster snapshot function within Rancher GUI

Referenced docs and https://rancher.com/docs/rancher/v2.x/en/cluster-provisioning/production/ should help in answering most questions, let us know if you need to know more.


Can we add the Authorization to this ?

I want to use it for  private sheet, or shared the access to only one email. So is there any way we can do that ? Like providing the login details or some sort of cookies or headers, anything into the request.

@Meet-Miyani Really sorry.. accidentally my notifications were disabled for this repo and I missed this.
Retrosheet is based on a free Google Sheet. Accessing a secure sheet from this library would require a lot of hacks to introduce (cookies and stuff). I'd suggest using the Google Sheets API to handle such tasks.

I see, thanks @theapache64 🫡


Can't Download Geode On Mac

Your Question
I can't download Geode on Mac, it just comes up with installation failed.

When I download the geode-installer-v3.8.1-mac.pkg and try to run it on geometry dash it doesnt work

I'm using the geometry dash from the app store btw not steam

The App Store GD is just iOS GD, it has to be the one from Steam

seriously

bruh

well thanks bro


HTTP/2 Push only works when site is not cached

At first my environment:

Apache 2.4.25
PHP 7.1.3 mod_php
MariaDB 10.0.29-22.1
WordPress 4.7.3
W3 Total Cache(fixed) <IP_ADDRESS>.4

Page Cache Method: Disk: Enhanced
Minify enabled fpr css/js and HTTP/2 push enabled
when purging all cache and requesting a site for the first time it delivers js/css with http/2-push

after the site is in the cache it wont http/2-push js/css anymore

maybe i am missing something or did something wrong - i just hope i can help with this issue

hi @roebert sorry for the delay.
Check if in the wp-content/cache/page_enhanced/www.yoursite.com/your-page/ folder exists a .htaccess file. In the .htaccess are stored the list of the assets to push, eg.:
<IfModule mod_headers.c>
    Header add Link '</wp-content/cache/minify/1/ebbd6.js>; rel=preload; as=script'
    Header add Link '</wp-content/cache/minify/1/a0a3c.js>; rel=preload; as=script'
    Header add Link '</wp-content/cache/minify/1/773a9.css>; rel=preload; as=style'
</IfModule>

this feature require mod_headers.
Maybe your Apache doesn't load the module mod_headers.

hello,
i have checked this - after a purge i requested a site and it was cached
but no .htaccess was written
.../wp-content/cache/page_enhanced/test.example.de/utility-spezial # ls -alF
drwxr-xr-x 2 wwwrun www  4096 Apr 13 10:46 ./
drwxr-xr-x 5 wwwrun www  4096 Apr 13 10:46 ../
-rw-r--r-- 1 wwwrun www 60747 Apr 13 10:46 _index_ssl.html
-rw-r--r-- 1 wwwrun www 12551 Apr 13 10:46 _index_ssl.html_gzip

apache has mod_headers

@roebert sorry for the long delay.
Your issue require a deep investigation, if you can, i need a detailed log. Before starting take very careful, you can break your site.

Open your /wp-config.php and add or edit these lines, make it true:

define('WP_DEBUG', true);
define('WP_DEBUG_LOG', true);

Now wordpress puts all error into a log in /wp-content/debug.log

Open your /wp-content/plugins/w3-total-cache/Cache_File_Generic.php go to line 45 and after

function set( $key, $var, $expire = 0, $group = '' ) {

add
    error_log('is_apache:'.json_encode(Util_Environment::is_apache()));
    error_log('headers:'.json_encode($var['headers']));

the final code will be:
function set( $key, $var, $expire = 0, $group = '' ) {
    error_log('is_apache:'.json_encode(Util_Environment::is_apache()));
    error_log('headers:'.json_encode($var['headers']));

Open one page of your website

Open /wp-content/debug.log, the expected content is something like:

[20-Apr-2017 19:08:52 UTC] is_apache:true

[20-Apr-2017 19:08:52 UTC] headers:{"Status-Code":200,"Content-Type":" text\/html; charset=UTF-8","0":{"n":"Link","v":" <http:\/\/www.wordpress.local\/wp-json\/>; rel=\"https:\/\/api.w.org\/\""},"1":{"n":"Link","v":" <http:\/\/www.wordpress.local\/?p=4144>; rel=shortlink"},"2":{"n":"Link","v":" <\/wp-content\/cache\/minify\/1\/ebbd6.js>; rel=preload; as=script"},"3":{"n":"Link","v":" <\/wp-content\/cache\/minify\/1\/c731c.js>; rel=preload; as=script"},"4":{"n":"Link","v":" <\/wp-content\/cache\/minify\/1\/773a9.css>; rel=preload; as=style"}}

remove the lines:

    error_log('is_apache:'.json_encode(Util_Environment::is_apache()));
    error_log('headers:'.json_encode($var['headers']));

Please, copy/paste your log here.

hello,
i tried what you suggested and here are the results:
[26-Apr-2017 11:51:51 Europe/Berlin] is_apache:true
[26-Apr-2017 11:51:51 Europe/Berlin] headers:{"Status-Code":200,"Content-Type":" text\/html; charset=UTF-8"}
[26-Apr-2017 11:51:51 Europe/Berlin] is_apache:true
[26-Apr-2017 11:51:51 Europe/Berlin] headers:{"Status-Code":200,"Content-Type":" text\/html; charset=UTF-8"}

[26-Apr-2017 11:51:48 Europe/Berlin] is_apache:true
[26-Apr-2017 11:51:48 Europe/Berlin] headers:{"Status-Code":200,"Content-Type":" text\/html; charset=UTF-8","X-Pingback":" https:\/\/test.example.de\/xmlrpc.php"}
[26-Apr-2017 11:51:48 Europe/Berlin] is_apache:true
[26-Apr-2017 11:51:48 Europe/Berlin] headers:{"Status-Code":200,"Content-Type":" text\/html; charset=UTF-8","X-Pingback":" https:\/\/test.example.de\/xmlrpc.php"}

it seems this log-entries only get written when the site is not in cache - i tried with different sites, the information is the same every time: code 200 - content-type:text/html - charset:utf-8

@roebert sorry for the delay
what i see into your log is the absense of the "Link rel=preload" headers... see the difference between my and your log :
my:
headers:{
   "Status-Code":200,
   "Content-Type":" text\/html; charset=UTF-8",
   "0":{
      "n":"Link",
      "v":" <http:\/\/www.wordpress.local\/wp-json\/>; rel=\"https:\/\/api.w.org\/\""
   },
   "1":{
      "n":"Link",
      "v":" <http:\/\/www.wordpress.local\/?p=4144>; rel=shortlink"
   },
   "2":{
      "n":"Link",
      "v":" <\/wp-content\/cache\/minify\/1\/ebbd6.js>; rel=preload; as=script"
   },
   "3":{
      "n":"Link",
      "v":" <\/wp-content\/cache\/minify\/1\/c731c.js>; rel=preload; as=script"
   },
   "4":{
      "n":"Link",
      "v":" <\/wp-content\/cache\/minify\/1\/773a9.css>; rel=preload; as=style"
   }
}

your:
headers:{
   "Status-Code":200,
   "Content-Type":" text\/html; charset=UTF-8"
}

without Link, http2 can't push the assets.
have you enabled into the minify page setting for css and js the http2push option?

Hello,
minify for css and js is enabled
http2push is enabled for css/js
i've checked it again - first request for a page after complete purge: js and css came with preload
second request: no preload headers
like you wrote before, no .htaccess is written in the cache folders

@roebert sorry for the very long delay.
I have fixed an issue and maybe it can help you: https://github.com/szepeviktor/w3-total-cache-fixed/pull/491

i deployed a whole new clone of the site and tried it again - the problem is gone.
.htaccess is written for every page and the preload header works
only the start-page wont create a .htaccess with preload options

only the start-page wont create a .htaccess with preload options

yes this is a side effect of https://github.com/szepeviktor/w3-total-cache-fixed/pull/491 it fix some bugs but disable http 2 push for only the root/home page


SelectField (like TextField)?

Now that we have a <Select> thanks to #8023 (which is awesome!) I was wondering if there are any plans to add a <SelectField> convenience wrapper like we have the <TextField> for <Input>?
I was able to create one for myself by more or less copy&pasting the TextField (and I'm extensively using TextField in many other places of my codebase) but I assume that other people are as lazy as me and would love to have a convenience SelectField that contains Label, HelperText etc. pp. out of the box.
I remember that there have been extensive discussions about which scope of Select should be covered (Autosuggest, etc.) and I'm not to add any more complex logic to Material UI. I'm just wondering if a TextField-Like wrapper around Select would fit into the Material UI scope or not?!

I knew this topic would come up. I have been asking myself this question too. I wasn't sure about what would be the best API yet.
Do you think we should extend the TextField or create a new SelectField?

@oliviertassinari given that my own SelectField implementation only differs in about 3 lines from TextField I guess it would probably sense to rather extend than create a new component. That should also help to keep the interfaces as similar as possible.
I can give that a try tomorrow unless you want to implement that on your own anyways.
FYI here's what I quickly hacked together earlier this evening: https://gist.github.com/ctavan/0a5d192a9e0b66b649f69f79e27865e0

I can give that a try tomorrow

Oh yeah! I would love to see this. Extending the TextField sounds like the best option available 👍 .


[JSDoc] Add missing documentation for botbuilder

Addresses # 2602
Description
This PR adds the missing documentation in botbuilder library.
Some documentation is ported and adapted from botbuilder-dotnet
Note: We divided the work for this library into two PRs to easy their review. PR# 143 contains the rest of the botbuilder fixes.
Specific Changes

Updated jsdoc/require-jsdoc in .eslintrc.json from warn to error.
Added JSDoc comments in all public methods and some of the private ones in the following files:

eventFactory
fileTranscriptStore
handoffEventNames
inspectionMiddleware
statusCodeError
teamsActivityHandler
teamsActivityHelpers
teamsInfo

Testing
The following image shows the ESLint Report with no JSDoc errors.

Promoted


chore(docs): Updating concepts/communication pages images

Branded, higher resolution images
closes: #3857

beautiful!


tests: remove some unused FileCheck prefix options

Such tests would fail with the -allow-unused-prefixes FileCheck option.
diff --git a/test/lit.cfg b/test/lit.cfg
-run_filecheck = '%s %s --sanitize BUILD_DIR=%s --sanitize SOURCE_DIR=%s --use-filecheck %s %s' % (
+run_filecheck = '%s %s --sanitize BUILD_DIR=%s --sanitize SOURCE_DIR=%s --use-filecheck %s %s -allow-unused-prefixes=false' % (

It helps avoiding some wrong CHECK patterns.
I just looked briefly at some of the fails. TODO: fix remaining tests, too.
rdar://74189761

@swift-ci test and merge


Tech debt/dates unit tests

Pull Request Test Coverage Report for Build 47

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+7.3%) to 50.237%

Totals

Change from base Build 44:
7.3%

Covered Lines:
212

Relevant Lines:
422

💛  - Coveralls


fix: correct icon size documentation to match implementation

Description
Made icon size fixes as mentioned in https://switchcomm.atlassian.net/browse/DT-197
Pull Request Checklist

[x] Ask the contributors if a similar effort is already in process or has been solved.
[x] Review the contribution guidelines.
[x] Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
[x] Ensure all gulp scripts successfully compile.
[x] Update, remove, or extend all affected documentation.
[x] Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[x] Request a review from Brad Paugh, David Becher, or Drew Chandler.

Obligatory GIF (super important!)

Looks good 💯


Remove double underscore from c header templates

Using double underscore for an identifier in C/C++ invokes undefined
behavior.
From https://eel.is/c++draft/lex.name:
Each identifier that contains a double underscore __ or begins with an
underscore followed by an uppercase letter is reserved to the
implementation for any use.

Thanks!


[2.5] Fix search issues on overview page

Signed-off-by: Zack Layne<EMAIL_ADDRESS>Related issue: https://github.com/stolostron/backlog/issues/23135

Should we merge to main first and then release-2.5 ?

Should we merge to main first and then release-2.5 ?

I merged another PR to main that contained a few other changes in addition to these (see: https://github.com/stolostron/console/pull/1758).

/hold

Tested PR with image 2.5.1-PR1762-21011c5875d2d3ed1964dc4eadb8c0088f647715
The Overview page now works as expected (with missing data) when the search backend is unavailable.

/unhold

https://github.com/stolostron/backlog/issues/23202


README updates

Created include-markup component to include README.md instead of a blank welcome page.

Replace RawGit URL in README with GitHub pages link to demo app.
Added RawGit URL example to contribution guide.
Created include-markup component to render README markup, along with syntax highlighting based on GitHub styles.
Added README to the demo app’s welcome page per UXD suggestion.

Example:
https://rawgit.com/dlabrecq/patternfly-ng/readme-dist/dist-demo/#/

@dlabrecq - there are conflicts :(


deps: update proto-google-cloud-alloydb to v1

Fixes #113
Fixes #203

Not sure what this means:
java.lang.RuntimeException: com.google.api.gax.rpc.ApiException: API interaction failed

I will create another PR once we are ready to test it in production.


Improve "Copy as link"

Why ?
On Firefox, the copy link function doesn't work quite well due to the behavior you described here: https://github.com/chitsaou/copy-as-markdown/issues/45 . Often, users get copied results like [(No title)](http://blabla...) . This pr try to improve it.

Thanks for your help!
As you know, on older Firefox there was no API to support this feature. I'll try it on my end and let you know if it works well.

Confirmed working on the following browsers:

[x] Chrome 65 Beta (macOS)
[x] Firefox 58 (macOS)
[x] Firefox 58 (Windows)

Confirmed not working but no regression on the following browsers:

[x] Chrome 64 (Windows)

Thanks for your awesome work! Change the version number then I'll merge it :)

Thanks!

Pushed to Mozilla Add-ons. Should be available in 1-2 days.


Fixed issue where the browser covered the region and country dropdowns

Closes #2146
Summary
Switched the country and region fields so that the country is first. This field populated the region dropdown so it needs to be filled first in order to see a dropdown of the regions available in that country. Without this, the entered region disappears when the user selects a country after the region.
Implemented a workaround to prevent browsers to display their suggestions above the Dnn built-in ones. Due to a bug in chrome autocomplete="off" does not work, to really make sure that the browser does not try to autocomplete, we need a random value in the autocomplete parameter so that the browser had no fields to suggest in memory.
How to test

make sure to use chrome latest release, and in the browser settings make sure you have a couple of form fill records with addresses.

Before this fix:

Try to enter a region
Enter a country that has regions available (US or Canada should do)
Notice the browser covers the Dnn dropdown
Notice the typed region disappears for a dropdown with nothing selected after selecting a country.

After this fix:

Add a new website and notice after the website creation, the profile editor now has the country above the region and the browser no longer suggest regions and countries on those fields.
For existing users, I thought it could break custom profile forms to force the country above the region (them may not even use both). So for those users I would add in the release notes that they can switch the order of these fields to prevent the problem.

I agree that having region first while controlled by country is unpractical. Even though it runs against logic that you go from specific (street) to general (country) and then back to specific (region).


Unable to display description of the Remove_AppxPackage command

I'm at https://docs.microsoft.com/en-us/powershell/module/appx/remove-appxpackage?view=win10-ps webpage. The tree in the left panel shows that I'm on the  Remove_AppxPackage item but the website displays description of the Move_AppxPackage command.

Fixed.
#please-close


Twitter WKWebView

It doesn't appear that the @chroa branch is maintained anymore, but @tien271 created a specific fork to be compatible with iOS WKWebView requirements now.  cordova-plugin-twitter-connect-wkwebview - however, that plugin ONLY works if you already have a fabric account (you can't create new ones anymore) and who knows for how long it will continue to work - I suspect that will even break soon.
What are the alternatives to the Fabric key...should this be a Firebase crashlytics plugin by now?  Fabric has been dead for a while.

You can make the plugin cordova-plugin-twitter-connect-wkwebview work using Firebase.
Just after creating a new project, you will see a firebaseConfig object. Use the field apiKey as a fabric API Key and it should work smoothly.

ah, ok...thanks for the update.  Someone should really update the documentation on this plugin and/or properly maintain it.  Its too popular of a plugin/resource to not be kept up or maintained.

@MadyAkira
BREAKING ISSUE:  The recommended cordova-plugin-twitter-connect-webview DOES in fact work and allows your app to get approved by the Apple app store (twitter-connect-plugin has UIWebView references that will cause your app to get rejected) - works great.  But...a big BBUUTT...the newer plugin causes deeplinks window.handleOpenURL to break.  The twitter plugin has its own version that overrides the customurlscheme one and will prevent you from passing url paramters when you open your app with a deeplink custom scheme.
I don't have time right now, but I am going to test some stuff later...load twitter first, customurlscheme seconds - and see if that resolves the problem.  As of now I can say customerurlscheme first and twitter-connect-webview second def breaks parsing external parameters.

Did your app got accepted by the playstore also ?

@MadyAkira - I still have the standard twitter-connect-plugin in my android version because Android doesn't use WkWebview nor does Android have the UIWebView restrictions that Apple has.
But with Apple, the forked plugin cordova-plugin-twitter-connect-webview did allow my app to be accepted by Apple - but I then discovered the plugin was interfering with customurlschemes ability to managing inbound deeplink parameters using window.handleOpenUrl() - so I had to remove the plugin altogether and will most likely need to create another fork of the plugin to figure out how to use the twitter plugin without interferring with the customurlschemes plugin.

@rolinger I used the webview plugin for Android too, and since my app got rejected by the Playstore with this reason :
We don’t allow apps whose primary purpose is to provide a webview of a website(...)

I will uninstall it and use the 'normal' version instead.
For the deeplink problem that really suck... i hope you'll quickly find a solution.

You should reach out to Android about that, it's just a Twitter plugin that is compatible with webview, it doesn't mean what they are implying it is.
Get Outlook for Androidhttps://aka.ms/ghei36

From: MadyAkira<EMAIL_ADDRESS>Sent: Monday, November 2, 2020 7:43:59 PM
To: chroa/twitter-connect-plugin<EMAIL_ADDRESS>Cc: rolinger<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [chroa/twitter-connect-plugin] Twitter WKWebView alternative AND breaking issues (for iOS) (#30)
@rolingerhttps://github.com/rolinger I used the webview plugin for Android too, and since my app got rejected by the Playstore with this reason :
We don’t allow apps whose primary purpose is to provide a webview of a website(...)
I will uninstall it and use the 'normal' version instead.
For the deeplink problem that really suck... i hope you'll quickly find a solution.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/chroa/twitter-connect-plugin/issues/30#issuecomment-720808059, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AC4XUCM7YTL4O6EDY5DFBBDSN5G47ANCNFSM4S4SPEOA.


[Feature] Creating templates for different category

create template for bug_report, Feature, Question.

Thanks!!


Builds no longer fail if bintrayUsername and bintrayApiKey are unset.

Instead, the bintray goals are unavailable.

You're the best.


Codewars

-Adds some additional languages to the Lesson view to clarify that user profiles need to include a codewars user name for a start to show up by their name (students were confused by this).
-Also, modified the markup to put the kata link on a new line, and make the spacing a bit better.

+1


feat(files): Progress when Uploading

What this PR does 📖

Add progress percentage for when we upload

Which issue(s) this PR fixes 🔨

Resolve crucial need for it

Special notes for reviewers 🗒
Additional comments 🎤

put changes requested for Darius's comment above


fix: make guarded label metrics correctly drop

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Could you provide more details in the PR body?

Could you provide more details in the PR body?

See background at https://github.com/risingwavelabs/risingwave/pull/17593#issuecomment-2333382693


Recursion Protocol is generating wrong configuration

Hi,
many thanks for this unbound UI, it makes it much easier to start with unbound. With my special network situation I noticed an error: The "Recursion Protocol" is not doing what it shall do.
I only have IPv4 internet access, but also local IPv6 addresses. Thus, I want unbound to be accessable via IPv4v (do-ipv4: yes, interface <IP_ADDRESS>) and IPv6 (do-ipv6: yes, interface: ::0).
Because I cannot access upstream IPv6 DNS servers, I set "Recursion Protocol" to "IPv4 only". But, this only generates the directive do-ipv6: no which also disables the listening IPv6 socket.
Correct would be to not generate the outgoing-interface ::0 directive.

@EricLuehrsen

fixed in trunk


Fix #1907: menus don't have a dark theme anymore

This fix fixes Thimble issue https://github.com/mozilla/thimble.mozilla.org/issues/1907
This line was erased in one of the previous PRs : $("body").toggleClass("dark", theme.dark);
Putting it back resolved the issue. I did some local testing to make sure nothing broke, everything seemed okay.

We intentionally moved away from the using classes to set the theme, and opted for a data-theme attribute on the body instead. We may have more than two themes in the future, so the dark / not-dark binary choice doesn't make sense for the long term. Maybe we can add an override to the quickEdit CSS so that it respects the data-theme=dark attribute, and not only the class name. Will get back to you shortly on this.
Cheers!

After thinking a bit more and chatting with @Pomax - this will be the way we handle it for now. Brackets basically has a hard-coded notion of having a 'dark theme' and the .dark class is peppered throughout the codebase. My previous attempt to move the theme to an attribute mostly worked, but didn't anticipate a lot of cases where .dark still mattered.


How is it better than rq?

RQ is based on redis and used by many? how is Jug better?

It's not better or worse, it's very different.
Rq is designed for a dynamic set of tasks coming in, jug for analyses that are defined by a Jugfile.


Compilation errors with MinGW

Errors actually come from termcolor:

E:\conan\.conan\data\indicators\2.0\_\_\package\5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9\include/indicators/termcolor.hpp: In function 'std::ostream& termcolor::red(std::ostream&)':
E:\conan\.conan\data\indicators\2.0\_\_\package\5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9\include/indicators/termcolor.hpp:287:17: error: 'FOREGROUND_RED' was not declared in this scope
                 FOREGROUND_RED
                 ^~~~~~~~~~~~~~
E:\conan\.conan\data\indicators\2.0\_\_\package\5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9\include/indicators/termcolor.hpp:287:17: note: the macro 'FOREGROUND_RED' had not yet been defined
In file included from E:/Qt/Qt5.15/Tools/mingw810_64/x86_64-w64-mingw32/include/windows.h:74,
                 from E:\conan\.conan\data\indicators\2.0\_\_\package\5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9\include/indicators/terminal_size.hpp:7,
                 from E:\conan\.conan\data\indicators\2.0\_\_\package\5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9\include/indicators/progress_bar.hpp:12,
                 from ..\qt_quick_test\src\logger\logger.h:7,
                 from ..\qt_quick_test\src\logger\logger.cpp:1:
E:/Qt/Qt5.15/Tools/mingw810_64/x86_64-w64-mingw32/include/wincon.h:117: note: it was later defined here
 #define FOREGROUND_RED 0x4

If I manually update termcolor.hpp to the latest version these errors go away :)

Updated to later termcolor.hpp. Thanks for reporting!


requestAnimationFrame instead of debounce

Gents - let's take a look at this in more detail to determine if this will provide more consistent results with the same or higher frame-rate. Reference http://www.html5rocks.com/en/tutorials/speed/animations/ to brush up on requestAnimationFrame()

We'll need a polyfill...

yessir -- I believe https://gist.github.com/paulirish/1579671 would be the bit to use. Strongly think we should leave this out of the plugin and add it to the compatibility notes so users can decide what they want to support; while giving them an easy way to consume that polyfill.

@mfunkie Added in compatibility section with reference to polyfill.

@mfunkie Take a look at the latest addition. Uses an optimizedResize event that still uses requestAnimationFrame, but only fires when the window actually changes widths. Has fall-backs built-in for browsers that don't support requestAnimationFrame, but the polyfill mentioned will negate the fallback. Also updated README.

What exactly are we getting here by doing these changes?  debounce to 100ms almost certainly means the next call to refresh is going to happen on the next (or later) animation frame.  Do we need the larger list of callbacks to run on rAF in this optimizedResize when we're only calling one in the first place?

The debounce is still firing on resize, which is going to run after the DOM manipulation is set. The idea here is (with requestAnimationFrame) is it fires "in-between" repaints to minimize the impact on the DOM. I don't think we'd need to parse this out to the larger set of functions as update() does the jam, but I'd leave that up to you and @jbenesch on any benefits for that. The idea here is we get the snappiness of not having any debounce at all, without the memory hit running an update function like that costs when hooking into the resize event.

https://developer.mozilla.org/en-US/docs/Web/Events/resize in the "Example" section explains it a bit better.

All you're really gaining out of that pasted code is a running check so that the resize event only fires once per frame.  We don't need the list of callbacks that it maintains because we only add the one.

That - and it's not queuing off the resize event, but requestAnimationFrame, which is the entire point of it. If we can simplify that logic that they recommended, then I'm all for it, but triggering your functions via requestAnimationFrame compared to the resize event is preferred.

I'm just saying that it could be replaced with something like
var running = false;
window.addEventListener('resize', function resize() {
  if (!running) {
    running = true;
    if (window.requestAnimationFrame) {
      window.requestAnimationFrame(update);
    } else {
      setTimeout(update, 66);
    }
  }
});

I'll see if I can get a timeline together so we can compare across the board. On our test page, we probably won't see much of a difference because it's a pretty dumbed-down use-case, but on the larger scale of our apps (especially with what I'm doing; I'll walk you through it), "when" we fire this stuff is key.

I would agree that with larger doms, when you update the dom has a huge impact.


[Snyk] Upgrade react-router-dom from 6.9.0 to 6.14.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade react-router-dom from 6.9.0 to 6.14.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 23 versions ahead of your current version.
The recommended version was released 21 days ago, on 2023-06-23.

Release notes

Package name: react-router-dom

6.14.0 - 2023-06-23

6.14.0-pre.1 - 2023-06-21

6.14.0-pre.0 - 2023-06-21

6.13.0 - 2023-06-14

6.13.0-pre.1 - 2023-06-13

6.12.2-pre.0 - 2023-06-12

6.12.1 - 2023-06-08

6.12.1-pre.0 - 2023-06-08

6.12.0 - 2023-06-06

6.12.0-pre.1 - 2023-06-01

6.12.0-pre.0 - 2023-06-01

6.11.2 - 2023-05-17

6.11.2-pre.0 - 2023-05-16

6.11.1 - 2023-05-03

6.11.1-pre.0 - 2023-05-02

6.11.0 - 2023-04-28

6.11.0-pre.2 - 2023-04-27

6.11.0-pre.1 - 2023-04-26

6.11.0-pre.0 - 2023-04-26

6.10.0 - 2023-03-29

6.10.0-pre.2 - 2023-03-24

6.10.0-pre.1 - 2023-03-22

6.10.0-pre.0 - 2023-03-21

6.9.0 - 2023-03-10

    from react-router-dom GitHub release notes

Commit messages

Package name: react-router-dom

09b6cbe chore: Update version for release (#10636)
0ec67e6 Exit prerelease mode
d4ec21b changelog/docs updates
9debb5c Update changelogs
d1bc7cf Update changeset + chanbgelog
6cbd0b1 chore: Update version for release (pre) (#10628)
19a71df fix FormData submitter check (#10627)
8b95f2b chore: Update version for release (pre) (#10626)
d9b3941 Enter prerelease mode
ccf63e2 Merge branch 'main' into release-next
fff84b5 fix empty submissions (#10625)
718ec1f Avoid calling shouldRevalidate on interrupted initial load fetchers (#10623)
96e1fc1 Inline startTransitionImpl to avoid .d.ts issues with skipLibCheck:false (#10622)
bc25528 Fixing typo (#10619)
7e2a122 chore: sort contributors list
80072b2 docs(DEVELOPMENT.md): fix typos (#10613)
73cd01e Fix generatePath when passed a zero value param (#10612)
3d4868b chore: add `deduplicate-yarn` workflow (#10583)
f63774c fix(react-router-dom): fix submitter serialization (#9865)
a3b21eb Merge branch 'release-next' into dev
dedafeb Merge branch 'release-next'
ed17fcd chore: Update version for release (#10601)
9f1ae18 Exit prerelease mode
eef9019 More docs updates

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Obsolete

react-router-dom has been removed, closing this pr.


Remove panics

Fix #59 and #72.

@devigned @marstr I've updated the PR according to your suggestions. Please take another look and sign off if appropriate. Thanks!


[website] Transition the Core page to be Material UI instead

[x] I have followed (at least) the PR section of the contributing guide.

The current Core page is confusing as it mainly refers to Material UI, which is not a synonym for MUI Core. As we get closer to "officially" introducing (as in stable) Base UI as part of the Core product group offering, that confusion is even more highlighted. To alleviate it, and to also better sit on the same level as the soon-to-be-launched Base UI page (https://github.com/mui/material-ui/pull/36622), this PR moves the content from the old Core page to a new Material UI one instead and iterates on the wording of it.
In the meantime, we'll preserve the /core/ URL but it will not be highlighted on the website, at least for now. Maybe in the future, we might use it to show everything that's inside it. And, by the time this and the Base page PR are merged, we'll have marketing pages for two Core products.
https://deploy-preview-37583--material-ui.netlify.app/material-ui/
Tasks

[x] First pass at adjusting the copywriting.
[x] Copywriting review.

Nice 👌

It would be nice to update the social card:

https://mui.com/material-ui/
once we finalize the new logos 😍 (#32181)


SW-7979 Order Attributes are not displayed in Email

Related ticket: http://jira.shopware.de?ticket=SW-7979
When using order attributes they are not displayed in the order email sent to the customer, since the $attributes variable gets overwritten.
This commit fixes the naming issue.

Dear Contributor,
Thank you for your contribution to Shopware. We value your engagement.
Unfortunately we have to close all existing Pull Requests.
This is due to the new Shopware 5 Major Release and a restructuring of the branch names.
If your contribution is still relevant for Shopware 5, a new pull request is highly appreciated.
We are very sorry for this inconvenience.
Greetings from the Shopware Team.


[Feature Request] new lint rule for declaring variables to have #declare or #var comment

create a new lint rule for variable when they are first declared to be followed by #var or #declare
Example:
a=1 #var
a=5
print(a)

OR
a=1 #declare or #declr
a=5
print(a)

Hi again!
It worries me that you've copied this issue from https://github.com/pylint-dev/pylint/issues/9685. Regardless, as before, I'm happy for this to be a place for discussion but this is very similar in style to https://github.com/astral-sh/ruff/issues/11698 and we are unlikely to implement it.
I can expand on that a bit if it's helpful: Rules that go beyond established language conventions are unlikely to be used by most users. In some cases, they may be actively confusing. If you want to push for adoption of a change like this, a linter is not the right place to start. Instead, you'll want to actually use this in projects and demonstrate that it adds clarity to the language. Before we can consider rules that enforce an annotation, we'd want to see projects using it successfully.
I actually think things like what you're asking for here and in #11698 sound better off as editor integrations rather than things that are included in the source code itself (either manually or with a linter).

Hi again!
It worries me that you've copied this issue from pylint-dev/pylint#9685. Regardless, as before, I'm happy for this to be a place for discussion but this is very similar in style to #11698 and we are unlikely to implement it.
I can expand on that a bit if it's helpful: Rules that go beyond established language conventions are unlikely to be used by most users. In some cases, they may be actively confusing. If you want to push for adoption of a change like this, a linter is not the right place to start. Instead, you'll want to actually use this in projects and demonstrate that it adds clarity to the language. Before we can consider rules that enforce an annotation, we'd want to see projects using it successfully.
I actually think things like what you're asking for here and in #11698 sound better off as editor integrations rather than things that are included in the source code itself (either manually or with a linter).

Hello,
This new rule is different than ending python code block with "#." in the other proposal, however, I incorporated the current proposal here and the same issue there as well.
The reason I hope there is lint support for these issue is because once it can be added as lint rule, then editors and code formatter will be following soon to support and deal with these rule, in terms of code format organizing, code coloring (we can have different color for #. or #var by the editor).
I think there feature can be helpful for project development.
Thank you,

This new rule is different than ending python code block with "#." in the other proposal, however, I incorporated the current proposal here with other proposal there as well.

The proposals can be separate, but they're both using Python comments as "markers" which is not something we encourage.

The reason I hope there is lint support for these issue is because once it can be added as lint rule, then editors and code formatter will be following soon to support and deal with these rule

Why not just have the editor show these markers without including them in the source code?

I think these feature can be helpful for project development.

Unfortunately we can't go on theory here. Until there are major projects following your proposed design, we won't be implementing enforcement of it.

This new rule is different than ending python code block with "#." in the other proposal, however, I incorporated the current proposal here with other proposal there as well.

The proposals can be separate, but they're both using Python comments as "markers" which is not something we encourage.

The reason I hope there is lint support for these issue is because once it can be added as lint rule, then editors and code formatter will be following soon to support and deal with these rule

Why not just have the editor show these markers without including them in the source code?

I think these feature can be helpful for project development.

Unfortunately we can't go on theory here. Until there are major projects following your proposed design, we won't be implementing enforcement of it.

I prefer lint rules and not just editor stylings as lint rules allow checking and enforcing the code while editor side doesn't do checks on code.

This new rule is different than ending python code block with "#." in the other proposal, however, I incorporated the current proposal here with other proposal there as well.

The proposals can be separate, but they're both using Python comments as "markers" which is not something we encourage.

The reason I hope there is lint support for these issue is because once it can be added as lint rule, then editors and code formatter will be following soon to support and deal with these rule

Why not just have the editor show these markers without including them in the source code?

I think these feature can be helpful for project development.

Unfortunately we can't go on theory here. Until there are major projects following your proposed design, we won't be implementing enforcement of it.

Regarding clode block ending lint marker rule, there could also be a configurable threshold limit for block lines for which the code_block_ending rule is activated and would apply (example: block_lines > 1 lines by default).
Users can also use new line instead of # or #. if they require and configure it to their requirements.

Thanks for the proposal but I'm gonna close as unlikely to implement per @zanieb's comments above.


Error : no such package '@io_kythe//kythe/data': The repository '@io_kythe' could not be resolved: Repository '@io_kythe' is not defined

https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/3379#_
Platform: Ubuntu
Logs:
ERROR: no such package '@io_kythe//kythe/data': The repository '@io_kythe' could not be resolved: Repository '@io_kythe' is not defined
(03:57:19) ERROR: /var/lib/buildkite-agent/builds/bk-docker-r8lq/bazel-downstream-projects/kythe/WORKSPACE:63:19: no such package '@io_kythe//kythe/data': The repository '@io_kythe' could not be resolved: Repository '@io_kythe' is not defined and referenced by '//external:vnames_config'
(03:57:19) ERROR: Analysis of target '//kythe/cxx/indexer/cxx/testdata/proto:proto_semantic_ref_aw_test_proto_kzip' failed; build aborted: Analysis failed
(03:57:19) INFO: Elapsed time: 1.389s, Critical Path: 0.04s
(03:57:19) INFO: 1 process: 1 internal.
(03:57:19) ERROR: Build did NOT complete successfully
(03:57:19) FAILED:

Steps:
git clone -v https://github.com/kythe/kythe.git
git reset 47d8a9a27d280e7c71150a1b267f46790ead5fb7 --hard
export USE_BAZEL_VERSION=b3a83a3a3dab0b87952b67cfc7ea8516eabeec4b
sudo apt update && sudo apt -y install bison flex uuid-dev asciidoc graphviz source-highlight
bazel build //kythe/...

CC Greenteam @mai93

This seems like a bug in Bazel; we define io_kythe right here: https://github.com/kythe/kythe/blob/ded645e9f7e7140f5d065fb570d8900f0850150c/WORKSPACE#L1

@Wyverald can this be related to https://github.com/bazelbuild/bazel/commit/9be1be93ff17dbb4cf0effb4f885911ec99cc30c

@meteorcloudy

This is probably due to the enabling of Bzlmod by default: https://github.com/bazelbuild/bazel/issues/18958
Please add --noenable_bzlmod in your .bazelrc file if the project isn't yet ready to use Bzlmod.

Similar issue for protobuf: https://github.com/protocolbuffers/protobuf/issues/14383, where Bzlmod is already disabled.
I think @mai93 is correct, I can reproduce the failure with Bazel built at https://github.com/bazelbuild/bazel/commit/9be1be93ff17dbb4cf0effb4f885911ec99cc30c but not its parent commit.
@sgowroji Let's move those two issues to the Bazel repo since it's a Bazel bug, and it should be marked as a release blocker for 7.0.0

Moved here https://github.com/bazelbuild/bazel/issues/19973


flake: use a more recent GHC, and set mainProgram = "jupyter-lab" for envs

Just putting this out here -- this change will allow you to run
nix run github:IHaskell/IHaskell

and you'll get a working JupyterLab, which is pretty cool I think. I picked GHC 9.4 as the default since it's the latest current Stackage LTS. You can also pick the variant you want like this:
nix run github:IHaskell/IHaskell#ihaskell-env-display-ghc96

With this we could modernize the Nix section of the docs, which currently uses nix-build and pulls a random Nixpkgs 23.05 tarball. We could also put a binary cache flake hint and then it would be easy for people to use the Cachix artifacts.
Maybe we also want to shorten the flake output names for the environments--perhaps like ghc96-base for the base environment, and ghc96-full for the full environment.

Thanks! I was just thinking about doing the binary cache flake hint thing too.


chore: reduce p2p bootnode machine requirements

Benchmark results
Metrics with a significant change:

proof_construction_time_sha256_30_ms (16): 1,728 (+23%)
protocol_circuit_witness_generation_time_in_ms (base-parity): 895 (-50%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,747
1,557
693
745 (-2%)
774 (+1%)

proof_construction_time_sha256_30_ms
11,856
3,194 (+2%)
:warning: 1,728 (+23%)
1,432
1,470 (+1%)

proof_construction_time_sha256_100_ms
43,927 (-2%)
11,783 (-2%)
5,433 (-4%)
5,405 (-2%)
5,360 (-2%)

proof_construction_time_poseidon_hash_ms
78.0
34.0
34.0
58.0
87.0 (-1%)

proof_construction_time_poseidon_hash_30_ms
1,526
421 (+1%)
202
225 (-2%)
274 (+2%)

proof_construction_time_poseidon_hash_100_ms
5,749
1,575 (+1%)
720 (-1%)
780
801 (+2%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs

l1_rollup_calldata_size_in_bytes
1,412
1,412

l1_rollup_calldata_gas
9,464
9,446

l1_rollup_execution_gas
608,000
607,982

l2_block_processing_time_in_ms
702 (-1%)
1,288

l2_block_building_time_in_ms
22,267 (-1%)
45,762

l2_block_rollup_simulation_time_in_ms
22,023 (-1%)
45,417

l2_block_public_tx_process_time_in_ms
18,897 (-1%)
42,014

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
5,937
8,515 (+1%)

node_database_size_in_bytes
9,474,128
12,742,736

pxe_database_size_in_bytes
9,242
15,127

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.8 (+4%)
17.1
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.7
31.7
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.624 (+3%)
0.523
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
49.0 (+1%)
76.4 (-1%)
134
247
477
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
288
543
1,055
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.500
0.470 (-1%)
0.457
0.448
0.444
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
59.0 (+1%)
113 (+1%)
185 (+1%)
351
633 (+1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
107
208
355
686
1,263
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.508
0.505
0.487
0.479
0.472 (+1%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
59.1

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
101

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.552

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
79,470
665,267

Transaction size based on fee payment method
| Metric |  |
| - |  |


inital work for replacing Fax API

works on #2

Newest commit updates requirements (PyNaCL added for signature-verification) and adds a draft for verifying signatures send along web hooks in the new API.
Receiving hooks and verifying the signatures has been tested in isolation.
currently old and new api share a url for web hook receiving. (lines 48 - 109 can be deleted once API change is complete.)


Create an interface for uploading and viewing consignment notes.

Half done, need to be able to set con note data.


Update Pandas to 0.24.0 to be align with Snorkel of v0.9.X

This PR closes #317.

Can we just drop the pandas dependency?

https://github.com/HazyResearch/fonduer/pull/321 drops the pandas dependency.


New Scenario - The Experiment

First scenario, first time on github. Let me know if I didn't submit correctly! Thanks! -Jeff
https://github.com/bluefishj/TheExperiment/tree/master/TheExperiment

Added.

They are really rolling in! Looking forward to playing it!


Ability to mark a duplication transaction as a transfer.

In this case, both the transactions are part of a bank transfer. One captured as debit and other as credit.

What do you do in this case?
currently I do this as a 2 step process. First I mark the doubtful txn as same as this and then I go to the transaction and manually convert it to a transfer and then assign the right to_account

Thinking of is merging two transactions. I can pick specific information from each txns. Keep one txn and delete the other one. This can be done with an intermediate merge view.


PrismaAction type out of date

Bug description
The exported type PrismaAction referenced by MiddlewareParams is missing the new new findFirstOrThrow and findUniqueOrThrow actions from the 4.0 release.
How to reproduce
Build the client
Expected behavior
type MissingdActions = "findFirstOrThrow" | "findUniqueOrThrow"
type TotalActions = Prisma.MiddlewareParams & MissingActions

Prisma information

// Add your schema.prisma

import { Prisma, PrismaClient } from "@prisma/client"

Environment & setup

OS: macOS
Database: PostgreSQL
Node.js version: v16.16.0

Prisma Version
prisma                  : 4.3.1
@prisma/client          : 4.3.1
Current platform        : darwin-arm64
Query Engine (Node-API) : libquery-engine c875e43600dfe042452e0b868f7a48b817b9640b (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli c875e43600dfe042452e0b868f7a48b817b9640b (at node_modules/@prisma/engines/migration-engine-darwin-arm64)
Introspection Engine    : introspection-core c875e43600dfe042452e0b868f7a48b817b9640b (at node_modules/@prisma/engines/introspection-engine-darwin-arm64)
Format Binary           : prisma-fmt c875e43600dfe042452e0b868f7a48b817b9640b (at node_modules/@prisma/engines/prisma-fmt-darwin-arm64)
Format Wasm             : @prisma/prisma-fmt-wasm 4.3.0-32.c875e43600dfe042452e0b868f7a48b817b9640b
Default Engines Hash    : c875e43600dfe042452e0b868f7a48b817b9640b

Running into this issue too

This looks like a duplicate of: https://github.com/prisma/prisma/issues/17470
It is fixed in 5.1.0 and after
https://github.com/prisma/prisma/commit/c09c908ab82ac5e2f74d90aad591a0eec0416fca

Hey, I quickly reproduced this :)
This is the diff I see going from v5.0.0 to v5.1.0, and I can also confirm that these are still here on our current latest v5.12.0 :)
  export type PrismaAction =
    | 'findUnique'
+   | 'findUniqueOrThrow'
    | 'findMany'
    | 'findFirst'
+   | 'findFirstOrThrow'
    | 'create'
    | 'createMany'
    | 'update'
    | 'updateMany'
    | 'upsert'
    | 'delete'
    | 'deleteMany'
    | 'executeRaw'
    | 'queryRaw'
    | 'aggregate'
    | 'count'
    | 'runCommandRaw'
    | 'findRaw'


Clean up KHStates when external checks are deleted

Describe the bug
When a khcheck is created, a khstate is also created to report the status of the external check. However, when the khcheck is deleted the khstate does not get cleaned up. This does not affect our kuberhealthy status page as deleted external check statuses are not displayed.
I have also noticed that when you try and delete the khstate manually from the command line (after you've deleted the khcheck), the khstate pops back up due to a getCurrentState function in kuberhealthy.go that continuously fetches the check status. From the log output below it seems like the custom resource never gets fully deleted when you delete the khcheck which recreates the khstate pod every time you try to delete it.
time="2019-10-08T02:27:36Z" level=debug msg="Getting status of check for web request to status page: kh-daemonset-check"
time="2019-10-08T02:27:36Z" level=debug msg="Checking existence of custom resource: kh-daemonset-check"
time="2019-10-08T02:27:36Z" level=debug msg="khstate custom resource found: kh-daemonset-check"
time="2019-10-08T02:27:36Z" level=debug msg="Retrieving khstate custom resource for: kh-daemonset-check"
time="2019-10-08T02:27:36Z" level=debug msg="Successfully retrieved khstate resource: kh-daemonset-check"
time="2019-10-08T02:27:36Z" level=debug msg="Output for kh-daemonset-check hidden from status page due to blank authoritative pod"
time="2019-10-08T02:27:39Z" level=info msg="kuberhealthy/kh-daemonset-check: [waiting for external checker pod to report in...]"

Steps To Reproduce

Create an external check
Check that a khstate is also created
Delete khcheck
Observe that khstate remains

Expected behavior
When a khcheck is deleted, khstate should also be deleted.
Versions

Kuberhealthy 2.0.0

@integrii I believe this is fixed now? Since you added the fix for kuberhealthy to no longer have these checks when they're deleted

Thanks for the well described issue.  That really helps define the problem.
I do agree that we should have some pathway to clean up old khstate resources in the cluster.  What exactly that lifecycle looks like is up for suggestion.
I think it's either going to be that the khstate is deleted as soon as the khcheck is deleted, or it's going to be the case that we wait some period of time after the khcheck is deleted.  However, tracking the time since khcheck deletion could be hard (where do we put the last seen timestamp?).

@integrii pretty sure this is happening now, closing this issue!


How to work with enum ?

Hi everyone.
I want to work with an enum. The main source of the game is as follows:
namespace Clevers.Domain.Enums
public enum MultipleChoice
{
   // Token: 0x040000FA RID: 250
   A
   // Token: 0x040000FB RID: 251
   B
   // Token: 0x040000FC RID: 252
   C,
   // Token: 0x040000FD RID: 253
   d
}

And I want to call the value of A from this enum.
Does anyone know a way?

const MultipleChoice = Il2Cpp.domain.assembly("...").image.class("Clevers.Domain.Enums.MultipleChoice");
const A = MultipleChoice.field<Il2Cpp.ValueType>("A").value;


substring method does not explain what missing length means

The length parameter is optional, but the behavior of omitting it is not explained (from what I gather, it takes the entire remainder of the string).

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

ID: 8442979b-55b6-b159-a884-c8fc104eac0b
Version Independent ID: e9572f8c-b299-2553-ba0f-ec83243e1ad7
Content: Template functions - string - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/template-functions-string.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@ohadschn  - thank you for your comment.  I will update the article.  The revised article be published on Nov 11th.
#please-close


Clear divisions on single_partitions_merge

I ran into a problem when performing joins with small dataframes in my unittests using the eq helper, where assert_divisions raised an error. I belief the reason for this behavior to be, that the single partition merge copies the divisions from one dataframe. If I understand the division concept correctly, they give divisions of the index values. However, in general the index after a pandas merge is not related to the original index. Therefore, I belief that the divisions should be reset after a single partition merge.
I added a unittest that fails with dask head, but passes with an additional reset of the divisions.

Your analysis generally seems correct to me.  Thank you for the fix.

+1 from me.  Merging in 24 hours if no further comments.

Thanks for merging and the constructive feedback.


failed to install risingwaves operator with the latest risingwave-operator.yaml

I am trying to instead risingwave operator with the official yaml, and I got a failure with the following message:
The CustomResourceDefinition "risingwaves.risingwave.risingwavelabs.com" is invalid: metadata.annotations: Too long: must have at most 262144 bytes

our k8s cluster is on 1.23 and the crd and everything is from (v0.4.1)[https://github.com/risingwavelabs/risingwave-operator/releases/download/v0.4.1/risingwave-operator.yaml]

likely caused by
    cert-manager.io/inject-ca-from: risingwave-operator-system/risingwave-operator-serving-cert

it's more an issue with the way how we apply the change.  due to the size of this the crd, the additional last-applied-configuration inserted by the client which applying the yaml got blow up. using server side apply which can solve the issue.


Populate public IP address on initcontainer

Currently we're not populating the public IP address field on the initcontainer, so the GSDK (and thus, the game server) doesn't have access to this information.
https://github.com/PlayFab/thundernetes/blob/c5462cc9737182e7828c9c21426ef6b684656b79/initcontainer/main.go#L74
The main reason is that during the time we create the env variables for the init container, we don't know in which Node the Pod will land. Thus, we can't know the External IP of the Node. Also, it doesn't look that there is a way of getting the Node's External IP through the downward API.
We have provided a hacky workaround here but it would be nice if we found a way to populate the field.

An alternative implementation would be to

user the downward API to pass the NodeName to the initcontainer as an env variable (https://kubernetes.io/docs/tasks/inject-data-application/downward-api-volume-expose-pod-information/#capabilities-of-the-downward-apihttps://kubernetes.io/docs/tasks/inject-data-application/downward-api-volume-expose-pod-information/#capabilities-of-the-downward-api)
when the initContainer starts, query the API server for the external IP of the Node with the given name
we might need to modify RBAC permissions of the Pod for that


Detect default branch in class scripts

I ran into the issue of PRs not being created for github-games in production even though the script ran fine when I tested on my test GHES instance.  The culprit ended up being the switch from master to main because my prod instance is still configured to use master.  To work around this in the future and prevent having to do manual changes when syncing .com to GHES, we should add detecting the default branch into create-practice-repos and use that instead of always hardcoding the base of the PR to main.

Looks like git symbolic-ref refs/remotes/origin/HEAD is still pointed to the old branch name, before it was renamed in that repo. You can fix that by running the following in that repo:
git remote set-head origin --auto

Maybe we can put it in the setup script so it can catch edge cases like the one you experienced without requiring extra troubleshooting.

I like that.


Create a dev orb

Test orb packing.

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/bjd2385/general?version=dev:72ee14592e9ba10ba5d6a39d93874ba91647da5d

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/bjd2385/general?version=dev:3a9551da6ce34bf58351288e6b54cc42ed5e663b

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/bjd2385/general?version=dev:d0b87cb4656eb4c563e920c18d2744c3b189ebe3

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/bjd2385/general?version=dev:44bd264c11a8e6d1439476da5491319065ab050e

Your development orb has been published. It will expire in 30 days. You can preview what this will look like on the CircleCI Orb Registry at the following link: https://circleci.com/developer/orbs/orb/bjd2385/general?version=dev:d0b87cb4656eb4c563e920c18d2744c3b189ebe3

Was published with the development version of this orb. Good to merge :+1:


Shell improvement

When using run.sh you will receive a notice that you have update.

Your making a PR to master. Please open an other one to dev.

oops


Is support for multi-tenant in roadmap? if yes when can we expect ?

Is support for multi-tenant in roadmap? if yes when can we expect ?

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

ID: bac6db2d-bff9-cbcf-d522-966ed17ea27f
Version Independent ID: 91e3eeb7-8602-9e26-21c3-1fa70d03a354
Content: Set up sign-in for an Azure Active Directory organization - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/active-directory-b2c-setup-oidc-azure-active-directory.md
Service: active-directory
GitHub Login: @davidmu1
Microsoft Alias: davidmu

@vikramdadwal
Thanks for your feedback! We will investigate and update as appropriate.

It's not supported out-of-the-box, but you can add multi tenancy options by using regular Azure AD tenant alongside a B2C tenant along with custom policies. Users from multiple Azure AD tenants can sign into Azure AD B2C without configuring a technical provider for each tenant.
https://docs.microsoft.com/en-us/azure/active-directory-b2c/active-directory-b2c-setup-commonaad-custom
We do not have information about timelines for this functionality now. For now, you can upvote and subscribe to the feature requests for this. https://feedback.azure.com/forums/169401-azure-active-directory/suggestions/9865695-add-an-azure-ad-identity-provider
https://feedback.azure.com/forums/169401-azure-active-directory/suggestions/9849063-aadb2c-how-to-on-multi-tenant-applications-based

We will now proceed to close this thread. If you have further questions feel free to tag me in the comments.


in_tail: Fix an incorrect error handling on catching a short-lived log

Which issue(s) this PR fixes:
Fixes #3327
What this PR does / why we need it:
#3275 fixes a crash bug of in_tail, but it introduces another crash bug which may occur on catching a very short-lived log.
Docs Changes:
none
Release Note:
Same as title

It seems ok if the commit message describes the explanation a bit more.

I'll add it and a test.

It seems ok if the commit message describes the explanation a bit more.

I'll add it and a test.

done.

The test passed.


New Contributor

Hi, I'm Meenakshi, I am new to Open Source and want to contribute to this org. Thankyou!

Welcome to Code for Cause @minakshi29bharadwaj :tada:


fix: do not add baseURL to urls with protocol

Currently:
$fetch('https://test.com', { baseURL: 'https://foo.com' })

will make a request to https://test.com/https://foo.com.
We could also change the behaviour of withBase to do nothing if passed an absolute URL, but I think it makes more sense within ohmyfetch.

Change makes sense if a generic baseURL is provided and we want to reuse same config to override full URLs but it can also be a breaking change and unintended behavior (adding a URL as path. Imagine BaseURL is like http://image-cdn.com/s_200/ expecting to append URL to the end...)

It came to my attention because $fetch in Nuxt is broken if used with a baseURL because it is created with .create({ baseURL }). (So it works with /.)
> require('ufo').withBase('https://test.com', '/')
'https://test.com'
> require('ufo').withBase('https://test.com', '/f')
'/f/https://test.com'

This is suggesting to me that maybe it's the behaviour in ufo that we need to fix.

Well, the behavior of ufo.withBase is basically adding a one-time prefix. Not judging what kind of prefix or suffix we have (with/without protocol) and /f/http://test.com is a valid URL too even for Nuxt (A catch-all route can again handle that for something like an analyzer).
I think this kind of behavior only makes sense if we provided a default baseURL like the case of Nuxt/Nitropack with .create. While currently there is no way to distinguish.

I think we could always say that users need to prefix the URL with / to get it to be resolved relative to a baseURL.

You are right. This also seems URL behavior. Let's do it on ufo then since there is also / workaround 👍🏼 https://github.com/unjs/ufo/issues/67


[Test] Fix monaco import issue in UTs

Description
Upstream core dashboard added new dependency @osd/monaco in autocomplete provider. Anomaly dashboards previously had tests where indirectly triggers autocomplete registration in data plugin, and the load of autocomplete providers SQL and DQL
    Cannot find module '!!raw-loader!../../target/public/xjson.editor.worker.js' from '../../packages/osd-monaco/target/xjson/language.js'

    Require stack:
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/packages/osd-monaco/target/xjson/language.js
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/packages/osd-monaco/target/xjson/index.js
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/packages/osd-monaco/target/index.js
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/data/public/antlr/opensearch_sql/code_completion.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/data/public/plugin.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/data/public/index.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/saved_objects/public/saved_object/helpers/serialize_saved_object.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/saved_objects/public/saved_object/helpers/build_saved_object.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/saved_objects/public/saved_object/saved_object.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/saved_objects/public/saved_object/index.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/saved_objects/public/plugin.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/saved_objects/public/index.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/vis_augmenter/public/saved_augment_vis/saved_augment_vis.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/vis_augmenter/public/saved_augment_vis/index.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/vis_augmenter/public/plugin.ts
      /Users/runner/work/anomaly-detection-dashboards-plugin/anomaly-detection-dashboards-plugin/OpenSearch-Dashboards/src/plugins/vis_augmenter/public/index.ts
      public/components/FeatureAnywhereContextMenu/AssociatedDetectors/containers/AssociatedDetectors.tsx
      public/components/FeatureAnywhereContextMenu/AssociatedDetectors/containers/__tests__/AssociatedDetectors.test.tsx

      37 | import { registerWorker } from '../worker_store';
      38 | // @ts-ignore
    > 39 | import workerSrc from '!!raw-loader!../../target/public/xjson.editor.worker.js';
         | ^
      40 |
      41 | const wps = new WorkerProxyService();
      42 |

      at Resolver.resolveModule (../../node_modules/jest-resolve/build/resolver.js:324:11)
      at Object.<anonymous> (../../packages/osd-monaco/src/xjson/language.ts:39:1)
      at Object.<anonymous> (../../packages/osd-monaco/src/xjson/index.ts:34:1)
      at Object.<anonymous> (../../packages/osd-monaco/src/index.ts:32:1)
      at Object.<anonymous> (../../src/plugins/data/public/antlr/opensearch_sql/code_completion.ts:6:1)
      at Object.<anonymous> (../../src/plugins/data/public/plugin.ts:96:1)
      at Object.<anonymous> (../../src/plugins/data/public/index.ts:500:1)
      at Object.<anonymous> (../../src/plugins/saved_objects/public/saved_object/helpers/serialize_saved_object.ts:34:1)
      at Object.<anonymous> (../../src/plugins/saved_objects/public/saved_object/helpers/build_saved_object.ts:34:1)
      at Object.<anonymous> (../../src/plugins/saved_objects/public/saved_object/saved_object.ts:42:1)

to unblock CIs, added monaco import mock to fix.
Issues Resolved
https://github.com/opensearch-project/anomaly-detection-dashboards-plugin/issues/832
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/7648
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

all tests passed. Merged. @mengweieric


chore: brevo manual api calls adjust

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


Custom Helm Repository without absolute urls doesnt work

Rancher versions: 2.0.2
Steps to Reproduce:
Add helm Repo with index.yaml containing non absolute urls like :  http://charts-broken.mgolisch.eu/index.yaml
Results:
When trying to launch a catalog app it just displays a spinning circle and the version dropdown is empty.

I can verify if I will a private helm repository, rancher cannot unpack the Readme.md and version (showing a spinning icon), and as a result I cannot deploy the service

I have the same issue. Any good news on this issue?

Thanks for the report.
Workaround is via https://github.com/helm/chartmuseum/issues/170#issuecomment-425097536
Close in favor of https://github.com/rancher/rancher/issues/14905 and https://github.com/rancher/rancher/issues/13371


display pfps in group member rows

adding this lil quality of life task — adding pfps to group members list (in whatever style is fastest/easiest for you, knowing we’ll come thru with an updated layout here sometime soonish
Slack Message

https://www.figma.com/design/p6mt4tEDltI4mypD3TIgUk/Converse-App?node-id=2486-17480&t=GvsjjPMhSQAfiNhP-1

@covrter Thank you for the mockups!
Dev note:
Update containers/GroupScreenMembersTable.tsx with a leftView to show the avatar of the member


Specify path to store processed images

Hello Matthias,
In my model, I'm using a custom upload path in order to serve photos privately.
Would it be possible for django-imagefield to store processed images in the same "upload_to" location as the initial image ? Or call a function which will provide the secure path ?
image = ImageField(auto_add_fields=True, upload_to=get_image_path, storage=secure_fs)

def get_image_path(instance, filename):
    """Return image path formatted as PRIVATE_ROOT/user_id/."""
    path = '{path}/{id}/'.format(path=settings.PRIVATE_ROOT, id=instance.user.id)

@matthiask This one still needs answering...

@matthiask This one still needs answering...

Hi,
Processed images should already be saved using the secure_fs storage so you shouldn't have to change anything if all you want is to make images not accessible using the default Django filesystem storage.
However, changing the path name (everything after the storage root) wouldn't be backwards compatible. The path is determined here: https://github.com/matthiask/django-imagefield/blob/26f126f3a91b94b2e23dbc6b5da86fdb81ed0bee/imagefield/fields.py#L151-L182 If you know 1) the original filename of uploaded images and 2) the processors in use you can deterministically calculate the resulting processed image name. That is by design and the reason, why django-imagefield works well without a cache.
What you could do is insert a hard-to-guess path component into uploaded images. This may help. Maybe something like lambda instance, filename: f"{root}/{instance.user_id}/{uuid.uuid4()}-{filename}". This is still security-by-obscurity and may not be sufficient for you. (Btw, I think your get_image_path has a bug – I think callable upload_to values should also return the filename.)
(Thanks for the reminder @asencis , however, since I'm not being paid for this an answer cannot be expected.)

Hi,
Processed images should already be saved using the secure_fs storage so you shouldn't have to change anything if all you want is to make images not accessible using the default Django filesystem storage.
However, changing the path name (everything after the storage root) wouldn't be backwards compatible. The path is determined here: https://github.com/matthiask/django-imagefield/blob/26f126f3a91b94b2e23dbc6b5da86fdb81ed0bee/imagefield/fields.py#L151-L182 If you know 1) the original filename of uploaded images and 2) the processors in use you can deterministically calculate the resulting processed image name. That is by design and the reason, why django-imagefield works well without a cache.
What you could do is insert a hard-to-guess path component into uploaded images. This may help. Maybe something like lambda instance, filename: f"{root}/{instance.user_id}/{uuid.uuid4()}-{filename}". This is still security-by-obscurity and may not be sufficient for you. (Btw, I think your get_image_path has a bug – I think callable upload_to values should also return the filename.)
(Thanks for the reminder @asencis , however, since I'm not being paid for this an answer cannot be expected.)

@matthiask Thank you for your detailed answer. I really need the same behavior as Django's ImageField. But I understand it wouldn't be backward compatible to implement it.
Regarding get_image_path. You're right, get_image_path must also return the filename. I didn't copy my function in its entirety:
def get_image_path(instance, filename):
    """
    Return image path formatted as PRIVATE_ROOT/user_id/year/month/day/filename
    """
    path = '{path}/{id}/{year}/{month}/{day}/'.format(path=settings.PRIVATE_ROOT,
                                                      id=instance.note.user.id,
                                                      year=instance.note.datetime.year,
                                                      month=instance.note.datetime.month,
                                                      day=instance.note.datetime.day)
    # Create path if it doesn't exist
    if not os.path.isdir(path):
        os.makedirs(path, mode=0o755, exist_ok=True)
    return path + filename

@matthiask Thank you for your detailed answer. I really need the same behavior as Django's ImageField. But I understand it wouldn't be backward compatible to implement it.
Regarding get_image_path. You're right, get_image_path must also return the filename. I didn't copy my function in its entirety:
def get_image_path(instance, filename):
    """
    Return image path formatted as PRIVATE_ROOT/user_id/year/month/day/filename
    """
    path = '{path}/{id}/{year}/{month}/{day}/'.format(path=settings.PRIVATE_ROOT,
                                                      id=instance.note.user.id,
                                                      year=instance.note.datetime.year,
                                                      month=instance.note.datetime.month,
                                                      day=instance.note.datetime.day)
    # Create path if it doesn't exist
    if not os.path.isdir(path):
        os.makedirs(path, mode=0o755, exist_ok=True)
    return path + filename


src/main/java directory structure is hardcoded for package names

The directory path structure src/main/java is hardcoded.  This isn't how my projects are setup so it won't work in my environment.
If this were moved to a property I could configure this to work easily in my projects.  Ideally this would be done with a regex allowing me to use a single launch configuration for multiple projects,

Fixed via #75


Update about.md

Start changing the template to add personal details

AHH ooops sorry all! New to github....tried to commit changes to my own personal fork of this lol.

In that case, could you close this pull request?

Definitely, let me learn how to do that real quick, and I will close this out.


slick directive settings attribute not working

Hi, I'm using this on my view:
<slick settings="slickConfig" dots="true" infinite="true" slides-to-show="1" slides-to-scroll="1" prev-arrow="false" next-arrow="false" center-mode="true" center-padding="30px">

and then in my controller I'm using this for the settings:
$scope.slickConfig = {
      event: {
        afterChange: function (event, slick, currentSlide, nextSlide) {
          console.log('slick afterChange', 'event:', event, 'slick:', slick, 'currentSlide:', currentSlide, 'nextSlide:', nextSlide);
        }
      }
    }

The 'afterChange' event is not being fired or the directive is not getting bond to the settings object?

Hello,
please make a demo.
You could fork these:
http://codepen.io/aszmyd/pen/LGdoyV
http://plnkr.co/edit/9goFW6?p=preview

Hi,
I have exactly the same problem.
codes are here:
https://jsfiddle.net/hamidyfine/c8drrs8v/

I have the same problem, it does not look like the settings are read if I put them in the controller, works if I put them in the HTML tag.  No errors if I use a name that is not defined in the controller either.  Have tried to recreate it in Codepen, but there it works as expected.  Same versions of angularjs and slick are used.
The solution is built on Drupal 8 with Bootstrap theme and I had to wrap the angular-slick-carousel code in
(function ($) { ANGULAR SLICK JS HERE })(jQuery);
This is a known issue and solutiuon in Drupal or you will get an error that $ is unknow.  Everything else works just fine, just the "settings=" in the tag that does not work, so I cannot see that the issue is here.
Anyone that are better than me at this that can spot where the problem may be?

Ok, I made a silly mistake.  Somehow I got the JS from this project instead https://github.com/vasyabigi/angular-slick
Found the mistake when I looked for "settings" in the js and could not find it.  Probably a mistake down when I NPM'ed the files to the server.
So, this project works about the same forthe simple stuff, although it has been abandoned for 3 years, but are missing some key elements.


Laravel 5.7 Support

Instead of having to manually update for each Laravel 5.x release, the tilde version range will match any 5.x version above 5.3

I'd rather the version constraints be more deliberate to avoid any possible future issues. If you revert the versions back and add 5.7 I'll get this merged :+1:

Done!

boom 🎉


Wrap imgutils stuff.

av_image_copy is allegedly needed to access images inside AVFrame.

Isn't this addressed by f2e986a4726fbcceab33d093874094ec8b11987c?


External refresh command? BitBar listening on localhost UDP port?

Maybe I've overlooked something, but I've gotten to the point where I need a non-BitBar process to send a refresh command to a specific BitBar script. To be more precise, I'm using a non-BitBar process to write variables into an sqlite database table, which is also accessed by the BitBar script. However, when that happens, the BitBar script doesn't yet know that the database was updated, so I need the other app to tell the BitBar script that it needs to refresh. Is this already possible? If not, this would be great to have.
In that respect take a look at the AnyBar plugin: https://github.com/tonsky/AnyBar … you can run several AnyBar instances side by side, and each of them listens on its own localhost UDP port. If this were implemented in BitBar, with one port for every script, the user could easily send an external refresh command, something like:
echo -n "refresh" | nc -4u -w0 localhost 2047
In the BitBar script you would then only need a way to set the localhost UDP port for that script from the script itself.
This all presupposes that BitBar doesn't have an external-refresh option yet. But as I said, I couldn't find anything.

Or is this the one? https://github.com/matryer/bitbar/pull/247

https://github.com/matryer/bitbar/raw/master/Docs/#refreshplugin

https://github.com/matryer/bitbar/blob/master/Docs/URLScheme.md#refreshplugin


Gpl 769 2

Checked out from GPL-857

https://github.com/sanger/crawler/pull/292 was checkouted from this branch, so we will use that refactor branch to deploy to UAT as it contains all these changes plus more


Fix some typos in readme file

Changes:

Fixed check boxes
License section should be linked to the license file

Thank you for fixing @zolagonano! Cheers 🎉


(Ch.Error) Code: 376. DB::ParsingException: Cannot parse uuid

 ** (Ch.Error) Code: 376. DB::ParsingException: Cannot parse uuid <<204>>`Y<<160, 15>>,G#<<133, 179, 255, 183>>1<<26>>L"\a": while converting '<<204>>`Y<<160, 15>>,G#<<133, 179, 255, 183>>1<<26>>L"\a"' to UUID: While processing (driver_id IN ('<<204>>`Y<<160, 15>>,G#<<133, 179, 255, 183>>1<<26>>L"\a"')) AND 0 AND ((reported_at >= _CAST(1577836800, 'DateTime')) AND (reported_at <= _CAST(1577923199, 'DateTime'))). (CANNOT_PARSE_UUID) (version <IP_ADDRESS>4 (official build))

query
|> where([e], e.vehicle_id in ^vehicle_ids)
|> where([e], e.geofence_id in ^geofence_ids)

Edit:
After some research, I see that this behavior just presents against geofence_id which is from type: "Nullable(UUID)" while vehicle_id is only UUID and seems to work fine.

If we can find a quick solution... this is blocking me from launch code to production... :/

👋 @daviaws
I'll be able to look into this in about two hours. In the meanwhile, an easily reproducible example would be useful :)

I've reproduced it for both field :uuid, Ch, type: "UUID" and field :uuid, Ch, type: "Nullable(UUID)"

Thank you @ruslandoga I'll try in some minutes :)


Upgraded references to Serilog 1.5.1

Fixes the inability to use Serilog.Sinks.CouchDb with Serilog v1.5.0.

Thanks for taking the time to sort this out! The build seems to have failed - any ideas?

I don't know, I have successfully built it locally and forced it in my project and it works. I'll try to take a look why the build is failing. I am pretty new to Github, so it might take some time.


Fix TODO by removing expr.proto

Add a new rule to be able to add external proto_library targets as dependencies in grpc_proto_library so that we don't need to create duplicate proto files.
(I'm open to suggestions to move this rule to a different directory.)

CC @jtattermusch


Implement tower::Layer for tonic_web::Config

Fix #1115
Fix #1117

Perhaps one question @LucioFranco, should we document it somewhere in the rustdocs of the method tonic_web::config or of the struct tonic_web::config::Config that this trait is implemented? Or what rustdoc already generates in the trait implementation section is enough for you?

@LucioFranco in my case, I'm not using tonic::Server and I'm implementating manually the underlying service, reusing tonic only for decoding/encoding the grpc/grpc-web protocol. In this kind of use cases, supporting tower layers is really handy.
Somehow related to this (but I can open another issue to discuss it), I found out that the CORS code in tonic-web is doing more or less the same of https://docs.rs/tower-http/latest/tower_http/cors/index.html. I was wondering if it makes sense to just expose a GrpcWeb layer that transcodes grpc-web to grpc, and let the tower-http cors implementation handle the cors preflight. So one could either pick the "transcoding layer" manually, or can use tonic_web::enable to get the cors layer from tower-http composed with the transcoding layer. WDYT?

@LucioFranco to_layer sounds good to me, although I have in mind something different (which involves breaking changes):

WebLayer is exposed directly, it has no parameters (doesn't need them), has a new method, and it implements Layer, generating GrpcWeb<S> services
tonic_web::enable composes Cors layer from tonic-http and WebLayer. If a user wants more control over the cors layer (like me), they can just instantiate manually WebLayer and compose it with whatever configured cors layer they want
Config gets removed, there is no need for that anymore because all the fields are used to implement cors

WDYT?

Yeah, that sounds like a great plan, smaller separate PRs will make my life much easier! This all sounds good to me. Thank you!

@LucioFranco I think this PR as it is should be fine now. I'm working on the second PR to reimplement cors using tower_http::cors


concurrentqueue doesn't seem to be bundled in the install target since version 1.0.0

Hi,
I've pulled the repo and tried rebuilding the library to include it in my project, but the concurrentqueue dependency doesn't seem resolved by find_packag (the build works fine though).
Thanks in advance!

Can you provide us the CMake logs?

I get this in the configure step for a small test project where I just link anira to some executable:
[cmake] CMake Error at CMakeLists.txt:12 (find_package):
[cmake]   Found package configuration file:
[cmake]
[cmake]     E:/audio_dev/tests/anira-1.0.0/lib/cmake/anira/aniraConfig.cmake
[cmake]
[cmake]   but it set anira_FOUND to FALSE so package "anira" is considered to be NOT
[cmake]   FOUND.  Reason given by package:
[cmake]
[cmake]   The following imported targets are referenced, but are missing:
[cmake]   concurrentqueue::concurrentqueue
[cmake]
[cmake]
[cmake]
[cmake] -- Configuring incomplete, errors occurred!

In my CMake I do this:

list(APPEND CMAKE_PREFIX_PATH "${CMAKE_CURRENT_LIST_DIR}/anira-1.0.0")

find_package(anira REQUIRED)
target_include_directories(${TARGET_NAME} PUBLIC ${anira_INCLUDE_DIRS})

target_link_libraries(test_exec PRIVATE anira::anira)
```

Ok, we will have a look into this very soon. Which operating system, compiler and cmake version are you using?

No problem!
I'm on Windows 10, currently with MSVC (but I use Clang for Windows for some projects, anira has been working fine with both so far). I'm using the CMake that comes bundled with VS2022 Community (cmake version 3.29.5-msvc4).
I'm build anira separately, calling cmake install and then moving the files into my project.

Hi there,
I just released a new version where the bug should be fixed.
Could you confirm that it works now?

It works! Thank you.

Great - thanks for reporting back!


Fix typo in the readme

:tada: This PR is included in version 1.8.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


HTTP Request Content-Type fix

closes #31
There was handling in place to not allow requests other than POST, PUT, PATCH and DELETE to have the Content-Type header / have content, because the HttpClient would fail if it tried to send such requests. It appears that HttpClient has since been updated to allow such requests, I didn't get any expections during local testing. Removed the check and the SendMethod enum that was used solely for this purpose.

Summary by CodeRabbit

New Features

Enhanced HTTP request handling by allowing Content-Type headers in GET requests, improving compatibility with various client implementations.
Introduced a new test case for GET requests with content to improve test coverage.

Bug Fixes

Adjusted the handling of HTTP request content to simplify code and improve functionality across all request types.

Chores

Updated project version from 1.1.2 to 1.1.3, indicating minor improvements.

This still needs a small adjustment - don't review yet! 😄


[code-test] check/077: "Check all glyphs have codepoints assigned."

All fonts on the Google Fonts collection pass this check...