get_gravatar_url: allow custom default

Hi,
This allow users get a clear 404 or a variety of generated images based on the hash.
While here, fix default mm into mp,
ref. https://gravatar.com/site/implement/images/
Also add a test for this.

Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[x] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

Rough idea, not sure if this will work but might be a good starting point.

Rework where the default/size are declared to closer to where we need them.
Parse the URL provided, allowing us to pull out and merge the query string, then re-build the URL
Still strip the last slash to support existing compatibility
Update the default to be mp as per the docs, add docstring for reference to these docs

from urllib.parse import parse_qs, urlparse
# ...

def get_gravatar_url(email, size=50):
    """
    See https://gravatar.com/site/implement/images/ for Gravatar image options
    """

    gravatar_provider_url = getattr(
        settings, "WAGTAIL_GRAVATAR_PROVIDER_URL", "//www.gravatar.com/avatar"
    )

    if (not email) or (gravatar_provider_url is None):
        return None

    default = "mp"
    # requested at retina size by default and scaled down at point of use with css
    size = int(size) * 2
    url = urlparse(gravatar_provider_url)
    email_bytes = email.lower().encode("utf-8")
    hash = safe_md5(email_bytes, usedforsecurity=False).hexdigest()
    gravatar_url = "{base_url}/{hash}?{params}".format(
        gravatar_provider_url="".join(url[:3]).rstrip("/"),
        hash=hash,
        params={"z": size, "d": default, **parse_qs(url.query)},
    )

    return gravatar_url

wagtail/admin/tests/ui/test_sidebar.py will also need to be updated when this is done as it tests the full avatar URL for some reason.
"avatarUrl": "//www.gravatar.com/avatar/e64c7d89f26bd1972efa854d13d7dd61?s=100&d=mm",

Hi,
Thanks for the review !
Yes, this is for use in my custom code. But your approach wouldn't work in my use case, as I need some calls with default="mp", and others with default="404" (I want to detect users with a gravatar account).
But we could easily merge your urlparse ideas with the default="mp" addition in the argument list of this function which is already in this PR.
Would this work for you ?

I have raised the PR #11800, Please have a look. Thanks.

Superseded by #11800

@nim65s the other PR has not been able to be continued, instead of re-opening this PR, I have raised two new issues to track the desired changes.
https://github.com/wagtail/wagtail/issues/12658
https://github.com/wagtail/wagtail/issues/12659
If you are still up for doing this improvement, I would be happy to try to review, otherwise we can leave it to others in the community.
Thanks for looking into this originally.

Thanks for the ping
I can't work on this right now, but I subscribed to those issues, so I'll try to keep track of it

@nim65s thanks again for this original PR, I've credited you as a contributor also.


BREAKING CHANGE: Refactor NodeClass controller

Fixes #N/A
Description

BREAKING CHANGE: the names for log keys produced by the NodeClass controller will change as such: nodeclass -> nodeclass.status, nodeclass.hash, nodeclass.termination
Refactored the NodeClass controller to be split into three controllers (termination, status, and hash)
This change will keep functionality of each controllers to one operation and easier to reason through
All logic of the NodeClass will remain consistent

How was this change tested?

make presubmit
/karpenter snapshot

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/karpenter snapshot

/karpenter snapshot


i'm trying to work on the rf-filter optimization using scipy.optimize

LB = [10.8,10.8];
UB = [10000,10000];
bnds = tuple([LB,UB])
N=30;
count = 0;
x0 =[10.8,100];
def objective(x,freq,gd1):
Zo = 50;
w = 2pifreq
#w1 = w.tolist()
d1 = (1jw(x0[0]1e-11))
d2 = (1jw*(x0[1]*1e-12))
d3 = np.power(d2,-1)
Zin = 1/(d1+d3)
S11 = (Zin-Zo)/(Zin+Zo)
y = angle(S11);
y  = np.unwrap(y, discont=3.141592653589793, axis=-1)
y = np.ravel((y))
y = y.reshape(1,301)
g1 = w[0];
g2 = w[1];
dw = g2-g1;
gd2o = -diff(y)/dw;
gd2 = np.append(gd2o,gd2o[0][-1]);
gd2 = abs(gd2)*1e2
f = ((gd1-gd2))*1e9
f = np.ravel((f))
f = max(f)
return f
for h in range(1,N):
if(h<10):
t = str(h).zfill(3)
b = 'hairpin filter_param'+ t+'.s1p'
S = Network(b);
freq = S.f;
gd1= S.group_delay
gd1 = np.ravel((gd1))
gd1 = abs(gd1)
gd1 = gd1.reshape(301,1);
#plt.plot(freq,gd1)
warnings.filterwarnings("ignore")
lcopt = optimize.minimize(objective,x,args = (freq,gd1))
print(lcopt)
output:
fun: 0.9986006787542225
hess_inv: array([[1, 0],
[0, 1]])
jac: array([0., 0.])
message: 'Optimization terminated successfully.'
nfev: 3
nit: 0
njev: 1
status: 0
success: True
x: array([ 10.8, 100. ])
my code Is just returning same input values. Can you please help me with this. Thanks in advance

fun: 0.9986006787542225  hess_inv: array([[1, 0],        [0, 1]])       jac: array([0., 0.])   message: 'Optimization terminated successfully.'      nfev: 3       nit: 0      njev: 1    status: 0   success: True         x: array([ 10.8, 100. ])

Could you format your code in a Pythonic way (use ``` to include a code block). Without that it's very hard to read the source code.

i have formatted the code. Thank you so much for your help

Your code isn't replicable, what is the Network object?
Why is the warnings filter there?
if you perturb x0 by small amounts does the value returned by objective change at all? If it doesn't then that will be why the solver doesn't find a better solution.

hello andyfaff
sorry for the late reply
what do you mean by perturbing x0 by small amounts? and the network function is related to rf filters. I'm trying to convert the string into a network so that I can run the group delay and the working on some filter optimization problems on rf-hairpinfilter design.

It's not possible to help figure out what's wrong if we can't run the example ourselves.
I suggested that you change x0 by a small amount and see if the cost function changes value

Closing, no further reply from original poster. If they provide a minimal working example we can reopen.


GetAllStrings throws "No manifests exist for the current culture"

We have updated all our solution assemblies from 1.1.x to 2.0.1 and out of sudden this message has started to appear.
We have an MVC project that references assembly AppImpact.AspNetCore.
In assembly A we have resources for two cultures as shown on the image:

en
bs-Latn-BA

In MVC project, we create a localizer by using Create method on IStringLocalizerFactory with the following parameters (as shown on the images):

"Buttons" as baseName
"AppImpact.AspNetCore" as location

If we inspect created localizer instance, we can see that assembly is present in _resourceNamesCache:

Highlighted value is:
{[Assembly=AppImpact.AspNetCore, Version=<IP_ADDRESS>, Culture=bs-Latn-BA, PublicKeyToken=null;resourceName=AppImpact.AspNetCore.Resources.Buttons, ]}
As soon as we try to iterate through results of GetAllStrings method, we get "No manifests exist for the current culture".

This code was in production for more than a year. Only difference that was done was upgrade to 2.x.
Funny thing is that as soon as we rename Buttons.en.resx (or any other for that matter) to Buttons.resx (old logic with default resource), everything starts to work.
We are looking for a better solution rather than doing a mass rename of resource files.
Is this behavior as expected or there is a bug?
EDIT:
We have tried now the second overload of GetAllStrings method:

And now we get a different error:

However, in the bin folder where the build is, there is a resource file for this assembly:

This is remind me of an old bug #257, but it was solved by a temporary fix. Could you please create a minimal repo to reproduce the issue?

Thank you for your quick response. :)
https://github.com/App-Impact-Labs/ISSUE-MissingResourceManifestExample

Run Xampl.Services
Navigate to /StaticResources/Get

You will get exception immediately.

Let me check ..

Removing the en from Buttons.en.resx fixing the issue for me
Also you can use app.UseRequestLocalization instead of setting Bosnia and Herzegovina the culture in the controller

RequestLocalization will just set CultureInfo based on inputs (query string, cookie, header).
In production we are using it, but I have removed it from here for simplicity reasons.
Yes, removing en from resources does the thing, but that is "dirty" fix as general recommendation is not to have default RESX file.
Here is one more thing that you should try - now that you have removed "en" prefix from default resource, it will work (you will not get an exception), but check what values are you getting from GetAllStrings.
You should get translations for Bosnian language - but that's not the case, so even with removal of "en" prefix, it is still not working properly.

Yes, removing en from resources does the thing, but that is "dirty" fix as general recommendation is not to have default RESX file.

I don't think it's a dirty fix 😄, this the default for any resource, while the default culture in .NET is en-US, I will check if this documented in the docs or I will suggest it
Yep I got the Bosnian translation, which is the right values if I'm not wrong

You should get translations for Bosnian language - but that's not the case, so even with removal of "en" prefix, it is still not working properly.

So what's the result that you expect?!!

From official documentation (https://docs.microsoft.com/en-us/aspnet/core/fundamentals/localization):

Problem is that we have several solutions in the production that have followed this guideline.
What you are now suggesting will force us to do rename of all resource files and to also change a lot of our code that was built with presumption that there is no default resource file, but instead that there is a default culture (why otherwise set DefaultRequestCulture in RequestLocalizationMiddleware).
As for the second part - can you do a pull now, build and run again and see if you get results on Bosnian?
I have added Buttons.resx but left Buttons.en.resx as well.

I read it before a minute ago, nothing but this happen behind this by setting the NeutralResourcesLanguage attribute, so if use the following in your AssemblyInfo.cs:
[assembly: NeutralResourcesLanguage("bs-Latn-BA")]

the Buttons.resx will be the default resource, so if it's contains the Bosinian resources it will load them.
That's why you see this behavior, i'm not forcing you 😄, and with the DefaultRequestCulture you can set your default culture which is useful if the current culture is not supported the localization APIs will use it as default one

We don't have AssemblyInfo.cs at all, but regardless of that, some libraries here are part of our internal framework that is used across different solutions, and almost each solution has a different default culture.
We cannot set default culture to be bs-Latn-BA on internal framework libraries, as it is not guaranteed it is going to be default on any solutions that depend on it.
Again, why did this stop to work with 2.x while it was working as expected with 1.x versions? What has been changed? We are trying to find some explanation over this, but nothing is available online.

I knew you don't have AssemblyInfo.cs but I would like to explain to you why this happen

What has been changed?

AFAIK, the big once are:

Remove dependency on Microsoft.AspNetCore.Hosting.Abstractions
Consider surfacing additional diagnostics information as part of LocalizedString when a resource is not found

for more details you can check the releases

Thank you really for your efforts, but so far it was not helpful.
There has been a change in internal implementation and that change is making a huge problem for us at the moment. I am hoping that someone from the development team will shed some light on this.
Apart from the issue that we are facing, fact that adding .resx and .en.resx at the same time disables reading of existing .bs-Latn-BA.resx does not seem like everything is working as it should.

@atuzovic just one more question, can you let me know why you adding both .resx and .en.resx at the same time?!!

Just for fun - to see the behavior. I don't see valid explanation why is it disabling .bs-Latn-BA.resx resource.
But again, that's not the point.
Point is - our solution(s) have stopped working after the upgrade and we have documented what is happening.
I am asking for official clarification/recommendation on upgrade path from 1.x to 2.x.
Can you please let us know that this is all working as expected, there is no bug, and that we should rewrite a bunch of our code / rename tons of resource files to be able to run our code?

Seems having both .resx and .en.resx causes a bug, perhaps this is come from underlying ResourceManager APIs, @ryanbrandenburg @pranavkm any idea about this?
BTW @atuzovic there's a localization issue that I knew in 2.0.0, could you please try to upgrade to the latest preview bits and have a try. If it's still the same hope one of the team will respond

Update to latest preview bits of what?
Solution is currently using 2.0.1 version of Microsoft.Extension.Localization. I don't see any preview available on NuGet:
https://www.nuget.org/packages/Microsoft.Extensions.Localization/
Do you want me to add specific branch of this repository to solution directly or you're suggesting something else?

I suggest to use the dev feed (nightly builds), look i will do it in your behalf and waiting until one of the team respond ..

Alright, I have updated solution to nightly build (2.1.0-preview1-27907) and it is working the same way it was working with 1.x versions. I have removed Buttons.resx from solution => there is no error and I get proper translations for bs-Latn-BA.
I have made additional commit, so you can pull solution yourself and give it a try.
So 2.0.1 has a bug clearly. Do you have any ETA on release of 2.1.0 ?

As I told you before there's a bug I remeber in IViewLocalizer which I blogged about it here, but I'm not sure If it's the reason behind this behavior or there's something else had been fixed

Do you have any ETA on release of 2.1.0

Nope, but you can checkout https://github.com/aspnet/Home/wiki/Roadmap

Q1-Q2 2018 seems a bit undeterministic (probably targeting Build 2018) and we are stuck halfway at migrating from 1.x to 2.x.
What are the changes to get hotfix 2.0.2 for this issue in the meantime?
Again, apart from doing a mass rename, which will also affect csproj files and cause us to rewrite bunch of our code that handles versioning of translations, we don't see an easy fix to survive till 2.1.0.
Also, using nightly builds in production is not an option for us.

What are the changes to get hotfix 2.0.2 for this issue in the meantime?

I think there's a patch release which I don't have it's link, but I will look for it

Also, using nightly builds in production is not an option for us.

Yep using nightly builds in production is a good option

I found these links 2.0.3 and 2.0.4 if you are running on premise

If you are referring to .NET Core version, I have 2.0.4. on my dev machine already:

I was asking about release of 2.0.2. for Microsoft.Extensions.Localization library.

Oops, sorry it's my mistake

@ryanbrandenburg, can you please check and see which change fixed the issue in 2.1.0 release (from 2.0) and see whether we can come up with a simpler workaround?

The same issue like another mates.
var strings = _sharedLocalizer.GetAllStrings();

Message: No manifests exist for the current culture.
StackTrace:
 at Microsoft.Extensions.Localization.ResourceManagerStringLocalizer.GetResourceNamesFromCultureHierarchy(CultureInfo startingCulture)
   at Microsoft.Extensions.Localization.ResourceManagerStringLocalizer.<GetAllStrings>d__15.MoveNext()
   at System.Collections.Generic.LargeArrayBuilder`1.AddRange(IEnumerable`1 items)
   at System.Collections.Generic.EnumerableHelpers.ToArray[T](IEnumerable`1 source)
   at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)
   at System.Linq.SystemCore_EnumerableDebugView`1.get_Items()

> dotnet --info
Herramientas de la línea de comandos .NET (2.1.3)

Product Information:
 Version:            2.1.3
 Commit SHA-1 hash:  a0ca411ca5

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.16299
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\2.1.3\

Microsoft .NET Core Shared Framework Host

  Version  : 2.0.4
  Build    : 7f262f453d8c8479b9af91d34c013b3aa05bc1ff

Forwarded from https://github.com/aspnet/Localization/issues/406
Thank you so much.


verbose mode working

$ pyspec all tests                                                                                [9/1869]
set expectations

can expect values: ok
can expect exceptions: ok
can expect a specific object type: ok
can expect an object to be a member of a collection: ok
can check that an iterable is empty: ok
can check for the existance of specified keys in a dictionary: ok
can check for the existance of specified attributes on an object: ok
can check for methods on an object: ok
use boolean operators
can negate an expectation using should_not: ok
communicate failures
can show the expected error type: ok
can show the expected error message: ok
can show the expected error message when expecting an exception: ok
let
can use common attributes: ok
can use common methods: ok
can change the value of a let in a test: ok
can expect changes in a let to persist between tests: ok
can defer errors thrown by a let: ok
always returns the same object: ok
raises the correct error when an attribute on the Test Group does not exist: ok
before
is visible to the test: ok
can change the value of a before in a test: ok
resets the value of the before between each test: ok
can defer errors thrown by a before: ok
always returns the same object: ok
raises the correct error when an attribute on the Test Group does not exist: ok
outer
inner

can view befores on the outer group: ok
can view lets on the outer group: ok
can change the value of lets defined by outer: ok

but the value on outer will remain the same: ok
generate stats info in the runner metastructure
has a stats object: ok
tracks number the number of tests: ok
tracks the success/failure rate: ok
tracks time on spec_struct() using methods on Stats class: ok
tracks time elapsed for running tests: ok
create & manage metastructures
can create a new structure: ok
initializes with no test groups: ok
can add new test groups to the metastructure: ok
can remove specified test groups: ok
the cli has commands for running *_spec files
can run all tests in a given directory: ok
can run just the tests for one file: ok
can request a list of all test groups in a test directory: ok

Has not yet changed default to be brief mode with optional flag for verbosity, needs fixed before merging

 Hide passing tests' output

$ pyspec all tests
set expectations: ok
use boolean operators: ok
communicate failures: ok
let: ok
before: ok
outer: ok
inner: ok
generate stats info in the runner metastructure: ok
create & manage metastructures: ok
the cli has commands for running *_spec files: ok
Tests ran: 41
Success rate: 100.0%
Total time: 2366 microseconds


After upgrade to node 0.10.31 I can no longer install sails

I just installed node 0.10.31 on a fresh machine, installed grunt-cli as per usual and then went to install sails but got this error.
(Gort-dave) [11:38:21] ~$ npm install -g sails
npm WARN deprecated<EMAIL_ADDRESS>THIS PACKAGE HAS BEEN RENAMED TO: express-handlebars
npm WARN deprecated<EMAIL_ADDRESS>DEPRECATED. See readme: https://github.com/gruntjs/grunt-lib-contrib
npm WARN deprecated<EMAIL_ADDRESS>Please use node-uuid instead. It is much better.

><EMAIL_ADDRESS>install /usr/local/lib/node_modules/sails/node_modules/socket.io/node_modules/socket.io-client/node_modules/ws
> (node-gyp rebuild 2> builderror.log) || (exit 0)

  CXX(target) Release/obj.target/bufferutil/src/bufferutil.o
  SOLINK_MODULE(target) Release/bufferutil.node
  SOLINK_MODULE(target) Release/bufferutil.node: Finished
  CXX(target) Release/obj.target/validation/src/validation.o
  SOLINK_MODULE(target) Release/validation.node
  SOLINK_MODULE(target) Release/validation.node: Finished
/usr/local/bin/sails -> /usr/local/lib/node_modules/sails/bin/sails.js
npm ERR! peerinvalid The package grunt does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>
npm ERR! System Darwin 13.3.0
npm ERR! command "node" "/usr/local/bin/npm" "install" "-g" "sails"
npm ERR! cwd /Users/dave
npm ERR! node -v v0.10.31
npm ERR! npm -v 2.0.0-beta.0
npm ERR! code EPEERINVALID
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /Users/dave/npm-debug.log
npm ERR! not ok code 0

Any suggestions?

I have been working on a Vagrant provision script that was getting destroyed by this issue, hitting the same error the @tskweres described. After bumping up the memory to 2GB, everything started to install perfectly.
Might be helpful if anyone is running into this problem under similar circumstances.


cannot log into my database

I have my database configured, I am getting no errors from october, or mysql,
however, when I use this plugin to try and log in I get this error,
Access denied for user 'ubuntu'@'localhost' (using password: NO)
no matter what I type in the "server" field, or the "username" field.
I have 'lsemple' in the username field, but it says 'ubuntu' in the error message.
I hope this plugin and all the other buggy plugins get fixed soon this is a real nightmare.

Can you show me your config/database.php?
(Remember to change your credentials to something else before paste)

hi, ok it is here
 PDO::FETCH_CLASS,

    /*
    |--------------------------------------------------------------------------
    | Default Database Connection Name
    |--------------------------------------------------------------------------
    |
    | Here you may specify which of the database connections below you wish
    | to use as your default connection for all database work. Of course
    | you may use many connections at once using the Database library.
    |
    */

    'default' => 'mysql',

    /*
    |--------------------------------------------------------------------------
    | Database Connections
    |--------------------------------------------------------------------------
    |
    | Here are each of the database connections setup for your application.
    | Of course, examples of configuring each database platform that is
    | supported by Laravel is shown below to make development simple.
    |
    |
    | All database work in Laravel is done through the PHP PDO facilities
    | so make sure you have the driver for your particular database of
    | choice installed on your machine before you begin development.
    |
    */

    'connections' => [

        'sqlite' => [
            'driver'   => 'sqlite',
            'database' => 'storage/database.sqlite',
            'prefix'   => '',
        ],

        'mysql' => [
            'driver'    => 'mysql',
            'host'      => 'localhost',
            'port'      => '3306',
            'database'  => 'c9',
            'username'  => 'lsemple',
            'password'  => '',
            'charset'   => 'utf8',
            'collation' => 'utf8_unicode_ci',
            'prefix'    => '',
        ],

        'pgsql' => [
            'driver'   => 'pgsql',
            'host'     => 'localhost',
            'port'     => '',
            'database' => 'database',
            'username' => 'root',
            'password' => '',
            'charset'  => 'utf8',
            'prefix'   => '',
            'schema'   => 'public',
        ],

        'sqlsrv' => [
            'driver'   => 'sqlsrv',
            'host'     => 'localhost',
            'port'     => '',
            'database' => 'database',
            'username' => 'root',
            'password' => '',
            'prefix'   => '',
        ],

    ],

    /*
    |--------------------------------------------------------------------------
    | Migration Repository Table
    |--------------------------------------------------------------------------
    |
    | This table keeps track of all the migrations that have already run for
    | your application. Using this information, we can determine which of
    | the migrations on disk have not actually be run in the databases.
    |
    */

    'migrations' => 'migrations',

    /*
    |--------------------------------------------------------------------------
    | Redis Databases
    |--------------------------------------------------------------------------
    |
    | Redis is an open source, fast, and advanced key-value store that also
    | provides a richer set of commands than a typical key-value systems
    | such as APC or Memcached. Laravel makes it easy to dig right in.
    |
    */

    'redis' => [

        'cluster' => false,

        'default' => [
            'host'     => '<IP_ADDRESS>',
            'port'     => 6379,
            'database' => 0,
        ],

    ],

];

Hey lsemple, can you test this for me?
Open plugins/martin/adminer/classes/adminer-loader.php and replace:
function credentials() {
	if(Session::get('ADMINER_AUTOLOGIN') === true) {
		require_once(plugins_path() . '/martin/adminer/classes/OctoberAdminerHelper.php');
		$connection = Martin\Adminer\Classes\OctoberAdminerHelper::getAutologinParams();
		if($connection['driver'] == 'mysql') {
			return [$server, $connection['username'], $connection['password']];
		}
	}
}

with
function credentials() {
	if(Session::get('ADMINER_AUTOLOGIN') === true) {
		require_once(plugins_path() . '/martin/adminer/classes/OctoberAdminerHelper.php');
		$connection = Martin\Adminer\Classes\OctoberAdminerHelper::getAutologinParams();
		if($connection['driver'] == 'mysql') {
			return [$server, $connection['username'], $connection['password']];
		}
	} else {
		return array(SERVER, $_GET['username'], get_password());
	}
}

Thanks.-

That worked ! Thanks :)

There is a new release, please, update your plugin.
Thanks.-


Fixing the regression in inner hits aggregation

This commit fixing the regression introduced by https://github.com/opensearch-project/OpenSearch/commit/da5b2053864f24dcb3b8c555a9fd19e0d91a14e6.
Description
Reverts https://github.com/opensearch-project/OpenSearch/pull/12503 that introduced regression in 2.13.0. Thanks a lot @martijnbolhuis for spotting it and supplying the tests, @dblock for working with the contributor.
Related Issues
Closes https://github.com/opensearch-project/OpenSearch/issues/13467
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
~- [ ] Public documentation issue/PR created~

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.

Related to https://github.com/opensearch-project/OpenSearch/pull/13486

Related to #13486

I am not comfortable with this change as is - we need to add more tests, not just one, to cover all uncovered flows and catch the regressions.

I am not comfortable with this change as is - we need to add more tests, not just one, to cover all uncovered flows and catch the regressions.

@dblock @reta - Ideally the original commit should have been 2 separate PRs one for ScriptFields and InnerHits. My suggestion is to not press panic button and revert everything. Instead make more cognitive choice.

to cover all uncovered flows and catch the regressions.

all is very hard in my opinion, but I am open to suggestions

all is very hard in my opinion, but I am open to suggestions

hence we have such regressions

all is very hard in my opinion, but I am open to suggestions

hence we have such regressions, I prefer taking more time to understand were we have misses instead of patching in a rush

As previously mentioned, those 2 changes should have gone as part of separate commits, and makes sense to revert only one of those. But, I am okay if you really prefer that.

@dblock @msfroh wdyt folks? I would go with full revert #13486 and reintroducing those in 2.15.0 once we have sufficient test coverage for both.

Yeah... I think that makes sense. Let's get back to the known-okay behavior -- as far as I know, we don't have any evidence that initializing the FetchSubPhaseProcessor was causing any noticeable pain(*). We should treat this regression as a helpful sign that we didn't have enough test coverage for these kinds of requests. Once we have the test coverage, we can move forward with the change to avoid unnecessary object creation.
(*) Taking a look at the FetchSubPhaseProcessor implementation in InnerHitsPhase, it looks like it only captures references to searchContext and innerHits, so those would be the fields of the anonymous class. Combined with standard object overhead, I'm guessing a useless FetchPhaseSubProcessor probably wastes less than 20 bytes (assuming 32-bit object references).

@msfroh @reta decide in https://github.com/opensearch-project/OpenSearch/pull/13486#issuecomment-2088954077?

Closing this PR in favor of #13486


Active developed

Hi Ali
Ran into your project and  I think this is a great one and I can't see any other on the Internet doing what  you do. I can  see that there is some time ago it was actively developed. I am happy to contribute to the project if you are interesting in taking it active. I can see HW rendering is one thing to achieve as well as spending some time on that function to draw street names...
All the best
Christian

Hi, Thanks for the interest. Contributions are certainly welcome for this project. Let me know which part/feature are you interested in.

Hi
I am interested in getting performance up.  Either HW accelerated drawing using Direct2D or maybe just see if there is other things to do

I assume, that drawing isn‘t the bottle neck. I draw all lines and polygons in 5-10 ms with GPU backend of Skia. The problem is reading the data, decoding it and than, one of the biggest part, evaluating the filter of each style layer with each feature of the data.
Another big part is the placing of symbols and text on the tile. For this you need the symbols and text of the surrounding 8 tiles (read data, decode and filter for all 8 tiles). I assume, that this costs the most time at the end.

A couple of days ago, I was porting this renderer to Cesium JS library for vector tile rendering on the fly. I did some extensive tests and figured out that the biggest bottleneck is the filter.
The filter system is essentially a function evaluator. One way to speed it up would be to generate a filter code (in C#) for each layer as soon as the style is loaded. Then execute that filter code in run-time for faster performance (see this for runtime execution of C# code).

I’m not sure, if this approach (runtime execution of C# code) runs on all platforms (.Net, iOS, Android and so on).
I made the filter system with special objects, that all implement an evaluation function (see here). I tried to optimize it. One big point was to go away from Json objects. They are flexible, but slow. Now it is acceptable fast. Perhaps it could be made faster, if some feature sorted out before filtering, because line or fill layers aren’t interested in points.
Perhaps someone has another idea to speed it up.

That's some nicely organized code. I'm experimenting with web-based vector rendering, and so far the best way to go about it is to dynamically compile filters.

Hi!
I have tried this https://openmaptiles.com/
It works nicely with your code but it is insanely fast on the web. Any idea how they make it so fast?

How did you tested OpenMapTiles with this code?

Hi,
You could download a trial from there web page which I used directly with your code and GMAP.Net. Worked straight out of the box.
I just the Winform example you provide.

On the web, it uses WebGL and some extreme optimization techniques to render the vector tiles. Can you test your openmaptiles with the tileserver-gl in raster mode? (it uses a similar technique of rendering)

I am sorry, I don't have the time right now to test it and since I am not familiar with Java it would take me a lot of time... If you want to try it, just download the vector tiles, just download it from their webpage.

Hi. I was implementing vector tiles for web using javascript and HTML5 canvas. Turns out the biggest performance bottleneck is the filtering operations. I'll be testing it further and posting results

I've updated the repo with some performance improvements. Here are a few of these:

Shared sqlite connection between threads
Marked sqlite connection as readonly
Deferred disk cache to a delayed event that doesnt bother rendering thread


add talk about Browser Connectivity

Not sure about the timing. Maybe this should be earlier during the session, since this is exploring the basis connectivity options that libp2p nodes have at their disposal. I didn’t want to move anyone else’s talks though.

adding! we can tweak schedule later


Make sure that nodes can handle traffic spikes

Recent updates have caused a higher node usage (no longer using injected provider + requesting blocks for timestamp). This means that we're getting close to Infura limits on "regular" days, making it hard to handle more active days.
Some potential solutions:

Pre-emptively upgrading Infura plan
Switching to an auto-scaling node provider like QuickNode or Alchemy
Adding Fallback providers in case Infura limits are reached
Finding a way to bring down number of requests

Almost all other providers have too strict rate limits to be usable in the way Revoke is currently set up (i.e. many concurrent RPC requests). So it seems like upgrading Infura + bringing down requests is the way to go.

I upgraded the Infura plan and enabled auto-scaling. For some other chains we use Alchemy, which also has auto-scaling enabled. So we shouldn't be running into any issues when traffic spikes.
The only issue is that we're relying on Etherscan's APIs for several chains, such as BSC and Fantom. Those chains may run into performance issues. For BSC we're subscribed to the highest plan, so I don't foresee any issues.


Disable HTTP/2 protocol in devfile/registry-support to mitigate CVE-2023-44487 (Rapid Reset)

/kind task
/area registry
This is a follow-up issue to https://github.com/devfile/api/issues/1315.
As commented out in https://github.com/devfile/api/issues/1315#issuecomment-1804073338_, it is strongly recommended, as one of the possible mitigation measures, to disable HTTP/2 endpoints if not needed: https://access.redhat.com/security/cve/CVE-2023-44487
And from this analysis, there seems to be currently only one repo where we need to do so:

devfile/registry-support: in the index/server folder, where an HTTP Server is started

The steps for disabling the HTTP/2 protocol in net/http are documented in https://pkg.go.dev/net/http#hdr-HTTP_2
Acceptance Criteria

[ ] Disable HTTP/2 protocol in servers started in the devfile/registry-support repo
[ ] Make sure it doesn't break the way existing clients interact with registries

I've created a PR which follows the instructions mentioned in the pkg.go.dev

Updating the story points needed for the issue from 3 to 5, as the review process required more time.


Add new Delta functions for pan&zoom: scale and reverse

[ ] add scale
[ ] add reverse (maybe not necessary because it is scale -1)
[ ] add PanAndZoomExample.elm showcasing this feature


[BUG] Issue with fetch links w/pydantic v2

Hello,
I hope this message finds you well. I'd like to report an issue I encountered while using the Beanie library in Python. In the context of fetching links (relationships), I've noticed that the fetch_links=True parameter is not functioning as expected in the find, find_one, and get methods.
Specifically, in objects inherited from the 'Document' class, the fetch_all_links() and fetch_link('field') methods exist. However, the latter (fetch_link) seems to be the only one working as intended, but it only retrieves the relationship for a single field.
Here are the library versions I'm using:
pydantic == 2.2.0
beanie == 1.21.0
pymongo == 4.4.1
motor == 3.2.0

I appreciate your time and assistance in addressing this issue. Thank you for your support.
Best regards,
UPDATE:
the problem occurs when the attribute is nullable:
class DBObject(Document):
internal_notes: list[Link[InternalNotes]] | None = []

Hi! I'll try to reproduce this today. Thank you for the report

Hi @dammianhr ,
Sorry for the late reply. Could you please provide a single module example, which I can run to reproduce? I can not reproduce it.

I got the same problem. But only with a nullable link list using the pipe operator. I encountered this with fetch_all_links.
Small example of my code:
# Does not work:
class MyDocument(Document):
  author: Link[User]
  coauthors: list[Link[User]] | None

my_document = MyDocument(...).create()
await my_document.fetch_all_links()  # Only fetches author
my_document.get_link_fields()  # returns { "author": (...) }

# Works:
class MyDocument(Document):
  author: Link[User]
  coauthors: Optional[list[Link[User]]]

my_document = MyDocument(...).create()
await my_document.fetch_all_links()  # Fetches everything as expected
my_document.get_link_fields()  # returns { "author": (...), "coauthor": (...) }

I believe the issue is when defining the instance's _link_fields attribute, since my_document.get_link_fields() returned a dictionary with only "author" in the first case, and with "author" and "coauthors" in the second example.


Fix order attributes.

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Order correct:
<input type="email" id="attributesvalidatorform-email" name="AttributesValidatorForm[email]" value="" maxlength="20" minlength="8" required>

Current Order:
<input type="email" id="attributesvalidatorform-email" name="AttributesValidatorForm[email]" value="" maxlength="20" required minlength="8">

The minlength attribute must be added since the validator HasLength::class adds it, it is also common to use it with maxlength.

Need changelog

Need changelog

Done.

👍


Replace count 0 exists

Hello, in order to optimize the performances of the generation of data, I replaced count() == 0 with exists() because the query generated is lighter. Instead of having a count(*) on the objects you juste have a
SELECT (1) AS "a" FROM "my_table" WHERE (.....) LIMIT 1

Hey, thanks for the contribution! That changes makes absolutely sense.

Thank you for merging :)
Your library is really helpful !


[Feature][Connector-V2][OneSignal]Add OneSignal source conector

Purpose of this pull request
https://github.com/apache/incubator-seatunnel/issues/3018
Add OneSignal source conector
Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

LGTM

@TaoZex  Please resolve conflicts.

@TaoZex Please resolve conflicts.

Thanks. I have resolved the conflict and passed the CI.


Multi-Language Support

for owncloud.org? Any more details on this? It isn't a horrible idea or something but I lack both time and knowledge to make it work ;-)
Help is welcome of course, I sure would help review and test.


使用molden2fch小程序时报错

输入molden2fch 10a-cas.molden -molpro 后报错，报错信息见下图：

10a-cas_moldenfile.zip

I'm sorry that this is a bug for MOKIT-v1.2.6rc37 ~ 1.2.6rc41. It will be fixed in 1.2.6rc42. You can either wait for the update of 1.2.6rc42, or use the following workaround immediately
conda create -n mokit-py39 python=3.9
conda activate mokit-py39
conda install mokit=1.2.6rc35 -c mokit
molden2fch 10a-cas.molden -molpro

Thank you for your response and all the help provided. The issue has been resolved!

MOKIT-v1.2.6rc42 has been updated. This issue is solved. You can re-open it if you have any further question.


MINOR: Fix flaky testClientDisconnectionUpdatesRequestMetrics() (#11987)

Cherry-pick of ae45c59.
This PR fixes the flaky SocketServerTest.testClientDisconnectionUpdatesRequestMetrics() test. When a response is sent, the request metrics get updated. But if the metrics get updated before expectedTotalTimeCount is defined, the expected count gets defined with an inaccurate value.
It looks like the order was accidentally swapped in this commit https://github.com/apache/kafka/commit/16d36f1674f4867b4dc498464df20a0e8e18eb3f.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Is this an issue only on 3.2?


This package does not adhere fully to JSON:API 🥲

Here's a write up of the discrepancies with the spec, and my proposed solutions.
Violations
Pagination
JSON:API does not allow the use of non-family query parameters for implementation semantics, and on top of that we SHOULD use the page query parameter family.
So we have to change to page[number] and page[limit] or similar.
Sorting
Here we don't do much of what we MUST do:

MUST use the sort query parameter
MAY support multiple sorts by the comma ,, applying the sorts in order
MUST be ascending, unless the field is prefixed by the minus -, which then MUST be descending

Filtering
JSON:API is strategy agnostic, but it does specify the use of the filter query parameter family, and that entails that the filter query should be structured like this:
?filter[column]=<this piece is already compliant>

Since JSON:API does not allow the use of non-family query parameters for implementation semantics, we have to change the search query parameter to perhaps filter[@search] (@ is the only allowed symbol at the start of a member name that can avoid collision with field names).
Sparse fieldsets
Currently we use select, but this should be changed to fields[TYPE] query parameter family, eg /articles?fields[articles]=title,body
Include relationships
I didn't see from the docs whether we support any syntax for this? But we could support the "Inclusion of Related Resources" as specified:
/articles?include=comments,author
Proposal

We can create backwards-compatible additions to the syntax, and post a deprecation warning when a query contains old syntax.
We let users opt-in to JSON:API through a new decorator JsonApiPaginate that constructs a standard PaginateQuery.
We add a new overload signature to the existing Paginate decorator: @Paginate(format: 'legacy' | 'json:api') with default value 'legacy' and can swap the default value after the deprecation period.

The package doesn't have a deprecation policy, and deprecating the existing syntax isn't necesarily an end goal here, just a personal preference to promote and transition to the standard.
Pagination

[ ] Change page to page[number]
[ ] Change limit to page[size]

Sorting

[ ] Change sortBy to sort
[ ] Change current syntax (field:ASC/DESC) to a comma separated ordered list of the sort fields with minus as desc operator (field1,-field2)

Filtering

[ ] Change filter.field to filter[field]
[ ] Change search to filter[@search]

Sparse fieldsets

[ ] Change select=field1,field2 to fields[TYPE]=field1,field2
[ ] Expand paginate logic to allow field selection for relations as well (if any fields are specified for the sub-TYPE)

We can try to guess the types of relations by using their key in the object and using pluralize to derive the singular form. e.g., post.comments would have TYPEs post and comment. For more advanced cases we can accept a documentTypes: Record<string, string> map in the PaginateConfig that map any appearing relationship keys to their type:
{
  documentTypes: {
    'post.deeper.weirdKeyForComment': 'comment'
  }
}

so that fields[comment] would affect post.deeper.weirdKeyForComment
Include relationships

[ ] When a includes query parameter is given, we should only load the intersect of the configured relations and the given includes. Use dot notation for nested relations eg ?includes=post.comments

Let me know what you think @ppetzold and I can get started on a branch for this

@Helveg looks like a good idea
Do you use some tool that is required to comply with this format?

No, but I like that JSON:API provides a full "HTTP-URL-compliant" spec to query REST API's including nested relationships and field selects! And in general I enjoy adhering to standards because it comes with many benefits :)

we also have swagger now, it need to be adjusted as well, once you finish I can help with it

I don't really have a strong opinion on it; as I am currently not having this package in any production use-case. But I see your point, it claims compliance - so it better be compliant lol.
Jsonapi spec seems to be quite active and reasonably popular, so I think I would be okay with taking the leap into full compliance with a breaking change release.
Don't think the headache is worthwhile to maintain backwards compatibility. In case of any security patches we can still patch the current major.
Let's go for a new major with full jsonapi compliance 🚀

Sounds good! Is the PR/release workflow documented somewhere?

@ppetzold I'm ok as well

Cool, this repo uses semantic release. so every commit (if prefixed) triggers a release.
for larger major releases, we could add a beta or next branch. but likely overkill for such a small package.
invited both of you to the repo :)

Just one thing, I misinterpreted the query parameter section, and as long as a query parameter contains at least one non a-z character it's a valid "implementation query parameter member name":

A “query parameter family” is the set of all query parameters whose name starts with a “base name”, followed by zero or more instances of empty square brackets (i.e. []) or square-bracketed legal member names. The family is referred to by its base name.
For example, the filter query parameter family includes parameters named: filter, filter[x], filter[], filter[x][], filter[][], filter[x][y], etc. However, filter[_] is not a valid parameter name in the family, because _ is not a valid member name.

And:

Implementation-Specific Query Parameters
Implementations MAY support custom query parameters. However, the names of these query parameters MUST come from a family whose base name is a legal member name and also contains at least one non a-z character (i.e., outside U+0061 to U+007A).
It is RECOMMENDED that a capital letter (e.g. camelCasing) be used to satisfy the above requirement.

This means that filter[@search] might be needlessly complicated, and we can use a camelCased name instead, like searchText, or simply @search, but I can't really find a satisfying, intuitive replacement. So if you prefer anything over filter[@search] let me know.

@Helveg
just a note, it will make sense to release those fixes one by one, having it all in one will be hard to review properly

Ok, I'll PR them separately but targetting a new feature branch, so that they land on master as 1 breaking change!

Any update or eta on this work and the complaint status? :)

Yes, I'm working on it on the json-api branch, and have an open PR https://github.com/ppetzold/nestjs-paginate/pull/883 that addresses some of the points here already.

@Helveg @ppetzold Just wanted to check if there is any movement on this and getting this released?

I currently dont have that much open source volunteer time, and am more focused on triaging existing issues. My schedule should open up more in a month or 2


Clean up the Hero and Summary code blocks

Clean up the Hero and Summary code blocks to be more semantic and remove extra containers
Additions

Created .hero_preview block to combine .hero_card-bg and .hero_card-padding styles
Added header and footer blocks to .summary block
Added last-child hook to remove margin from last child within summary no matter what elem

Removals

Removed extra .hero_card-bg and .hero_card-padding blocks
Removed extra whitespace

Changes

Moved hero card url condition to outside of the markup
Swapped divs for more semantic article where applicable

Testing
Fetch branch

navigate to /blog page
navigate to /newsroom page

Review

@anselmbradford
@sebworks

Preview

No visual differences

Notes

The image container is needed to suppress loading of the images until they are needed
Fixes #257

If I can get a merge I'll integrate these changes into the Sprint-21 Hero and events list I'm currently working on.


oleau a rengul

oleau a rengul created by<EMAIL_ADDRESS>on 2018-02-08 15:14:26

<EMAIL_ADDRESS>replied, what could mean this expression?

<EMAIL_ADDRESS>replied, oleau a rengul-puzzled, perplexed.usually used to refer to someone lost in a situation.

<EMAIL_ADDRESS>replied, wow! that's great! how can I say using it: He had really puzzled / perplexed me!some other examples will be nice, too

<EMAIL_ADDRESS>replied, Now, your sentence Aleks, the best Palauan word for yours is â€˜choudur a
rengukâ€™ root word Chouid/ouid.
Ng kmal Mla mechudoud er ngikang!
I am really puzzled from Ulangâ€™s information.
Kmal Mla Moeiau a renguk era aikal omesodel a Ulang.
Achedil, Ulang a Mla mo uleau a rengul. Aww, Ulang is so confused and
seemed lost. ðŸ™ƒ

<EMAIL_ADDRESS>replied, is there no forms like OEUR A RENGUK?

<EMAIL_ADDRESS>replied, We say â€˜chosir a rengukâ€™. From mechas/mengas era renguk.


task tfstate shows all the ips from other tasks

This is something was happening to me in the xpd provider however in tpi seems to be fine, unless you see the logs and you can see that there is still 🤔 . While in logs we do not care in the tfstate is not desirable because the user wont be able to recover the IP to connect with the instance.
{
  "version": 4,
  "terraform_version": "0.14.5",
  "serial": 5,
  "lineage": "ea085f62-0775-6cc2-b83c-f544b2fd1bde",
  "outputs": {},
  "resources": [
    {
      "mode": "managed",
      "type": "xpd_task",
      "name": "taskfashion_mnist-1475236089",
      "provider": "provider[\"registry.terraform.io/0x2b3bfa0/xpd\"]",
      "instances": [
        {
          "schema_version": 0,
          "attributes": {
            "addresses": [
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",

Terraform will perform the following actions:

  # iterative_task.task1 will be destroyed
  - resource "iterative_task" "task1" {
      - addresses       = [
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",
          - "<IP_ADDRESS>",

I have been able to reproduce it in tpi
{
  "version": 4,
  "terraform_version": "1.0.11",
  "serial": 23,
  "lineage": "80512366-5dfa-3ab7-6b28-77cf498bb3a2",
  "outputs": {},
  "resources": [
    {
      "mode": "managed",
      "type": "iterative_task",
      "name": "task1",
      "provider": "provider[\"registry.terraform.io/iterative/iterative\"]",
      "instances": [
        {
          "schema_version": 0,
          "attributes": {
            "addresses": [
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>",
              "<IP_ADDRESS>"

I feel like this is related to how task "reconnects" to its job from the generated ID? I'm assuming that you never ran terraform destroy @DavidGOrtega "end" a task but made some modifications and did another apply? the task name is the same but there are many instances of it.


Consider "paused" media player as "off" from a state perspective

Use defined icons for on/off work well, except for media players which are technically "off" when "paused", that way we could have icons representing play state.

That would probably here, I'll see what I can do https://github.com/cgiesche/streamdeck-homeassistant/blob/9785765e44ec57067019c1e92ac7245114f7c8f5/src/Plugin.vue#L199

https://github.com/cgiesche/streamdeck-homeassistant/pull/76

Thank you very much for your contribution! I just created a new release including your changes.
If everything works as expected, feel free to close this issue (i do not have a media player entity ready for testing)

Done ! Thanks ! I hope we'll have that in the official store


Guidance Requested

I'm basically rewriting toastify.

[x] Upgrade Deps (clap, notify)
[x] Use clap derive
[x] properly scope features (info and server don't exist on macOS, rather than failing
[ ] Rework cli (title/summary and body as --named not positional args)
[ ] docs

I'd appreciate testing in linux and windows, I have a mac.
I will rebase before marking this ready, to rename / rearrange / split / combine commits.

Thank you so much for this already. I would like to give you good feedback, but I'm a bit swamped ATM. I'll try to get back to you some time this week.
Thanks so much for putting in the time already.

I am unable to test. I am confused because the binary I installed with cargo works fine.
Debugging my build shovels that it also works fine. I can see that it calls the notification method and step into the macOS API. No errors are displayed.


sponsor-info-updates

Added interflow info; added technical case study examples; fixed some minor links/typos

@Ladybiss is everything good now? would you merge if you are happy?


Try to apply mode styles to root html element before body

The background colour from custom modes wasn't being correctly applied to pages that had a body smaller than the view height.

Bumbag docs seem broken in dark mode. We may need to amend the following to just 'html':
https://github.com/jxom/bumbag-ui/blob/main/packages/bumbag/src/theme/index.ts#L26


Update Helloworld application to be publicly accessible.

This fixes https://github.com/mesosphere/universe/issues/167 (the application was moved from the Universe to the Multiverse).
I also added a health check, because that's good practice and amended the installation documentation.

@adam-mesos - does this meet your criteria?

LGTM. I assume you tested this yourself?
:rocket:

Updated and lovingly hand tested on DCOS! Thanks @adam-mesos.


GAM Update - not correct on macOS 10.14.5 - Now I get errors and cannot work

Just updated from GAM 4.85 to 4.86.
When I try to delete a message (the reason for which I installed GAM), I get:
Traceback (most recent call last):   File "gam.py", line 14441, in <module>   File "gam.py", line 14121, in ProcessGAMCommand   File "gam.py", line 12657, in getUsersToModify   File "gam.py", line 1336, in buildGAPIObject   File "gam.py", line 9748, in _getValueFromOAuth   File "site-packages/google/oauth2/id_token.py", line 141, in verify_oauth2_token   File "site-packages/google/oauth2/id_token.py", line 122, in verify_token   File "site-packages/google/auth/jwt.py", line 219, in decode   File "site-packages/google/auth/jwt.py", line 128, in _unverified_decode   File "site-packages/google/auth/_helpers.py", line 105, in to_bytes
I updated it with the command: bash <(curl -s -S -L https://git.io/install-gam) -l
If I type gam version
I get
GAM 4.86 - https://git.io/gam Jay Lee<EMAIL_ADDRESS>Python 3.7.3 64-bit final google-api-python-client 1.7.8 Darwin-18.6.0-x86_64-i386-64bit x86_64 Path: /Users/Apple/bin/gam
Thank you very much

Solved, reinstalling it, sorry!


Bug: navbar height overflowing on the jobs page.

Description:
The navbar on the Job Page is experiencing a height overflow, causing it to extend beyond its expected size. This issue results in poor user experience.
Steps to Reproduce:

Navigate to the Job Page (provide URL or route if necessary).
Observe the navbar at the top of the page.

Expected Behavior:

The navbar should maintain a consistent height across all pages.
It should not overflow or extend beyond its designed size.

Actual Behavior:

The navbar exceeds its normal height on the job page.
This may be causing layout shifts or elements to be pushed down unexpectedly.

I would like to work on this issue kindly assign it to me ASAP.

I believe that resolving this issue will significantly improve the overall user experience of the application. With my skills in HTML, CSS, and JavaScript, I am confident in my ability to identify and fix the problem effectively.
I would appreciate it if you could assign this issue to me. Thank you for considering my request! I look forward to your positive response.

this issue has been solved, so we are closing this issue.


Support for CD's

Support for playing CD's in future modern HTML5-based PC operating systems.

It's up to browsers to decide which file extensions they want to support.

But if this apps arrives to Windows/Linux/MacOS using NW.js, this can be a good feature using node.js


Add Live Preview support

This plugin is quite awesome, but it lacks Live Preview support, only showing icons while in Reading view. Live Preview support would be useful for those who don't want to rely on a constant switching between Reading view and Source view.

Thank you!
I think the issue is the same as #280. I'll close this here for now.


prompt and subreddit

Do i set subreddit like subreddit = reddit.subreddit("hentai") or r/hentai?
and can i add something to prompt like {personality} does not like humans which swears a lot

subreddit = reddit.subreddit("hentai")
though you already have figured it out :)


[rcore] Duplicated code on EventThread()

WARNING: Please, read this note carefully before submitting a new issue:
It is important to realise that this is NOT A SUPPORT FORUM, this is for reproducible BUGS with raylib ONLY.
There are lots of generous and helpful people ready to help you out on raylib Discord forum or raylib reddit.
Remember that asking for support questions here actively takes developer time away from improving raylib.

Please, before submitting a new issue verify and check:

[x] I tested it on latest raylib version from master branch
[x] I checked there is no similar issue already reported
[x] I checked the documentation on the wiki
[x] My code has no errors or misuse of raylib

Issue description
Raspberry code (and maybe all linux?) only supporting 4 touches.
Code Example
Currently in rcore.c, EventThread() there is code like this:
if (CORE.Input.Touch.position[0].x >= 0) CORE.Input.Touch.pointCount++;
if (CORE.Input.Touch.position[1].x >= 0) CORE.Input.Touch.pointCount++;
if (CORE.Input.Touch.position[2].x >= 0) CORE.Input.Touch.pointCount++;
if (CORE.Input.Touch.position[3].x >= 0) CORE.Input.Touch.pointCount++;

Rather than:
for(int iTp = 0; iTp < MAX_TOUCH_POINTS; iTp++)
              if (CORE.Input.Touch.position[iTp].x >= 0) CORE.Input.Touch.pointCount++;

I have changed this code and the relevant code for gestures and now I get MAX_TOUCH_POINTS touches and all is good.
Is there a reason for this code?
If not I will put in a PR for the changes.
Cheers
Andy

@AndrewCapon Good catch! Just reviewed it! Thanks!

Hi @raysan5
There is another bit of code a bit lower as well:
gestureEvent.pointId[0] = 0;
gestureEvent.pointId[1] = 1;
gestureEvent.pointId[2] = 2;
gestureEvent.pointId[3] = 3;

gestureEvent.position[0] = CORE.Input.Touch.position[0];
gestureEvent.position[1] = CORE.Input.Touch.position[1];
gestureEvent.position[2] = CORE.Input.Touch.position[2];
gestureEvent.position[3] = CORE.Input.Touch.position[3];

@AndrewCapon reviewed!


Error When Compiling - After Update

After the recent update I am receiving the below error when compiling in Debian Jessie:-
/usr/bin/ld: cannot find -lboost_system-mt
/usr/bin/ld: cannot find -lboost_filesystem-mt
/usr/bin/ld: cannot find -lboost_program_options-mt
/usr/bin/ld: cannot find -lboost_thread-mt
collect2: error: ld returned 1 exit status
makefile.unix:211: recipe for target 'casinocoind' failed
make: *** [casinocoind] Error 1

The linker can not find the boost libraries. You have to use GCC 5 and Boost 1.60 with the latest version. Have you followed https://github.com/casinocoin/casinocoin/wiki/Compile-Linux-CLI ? I think Jessie already has the GCC 5 installed (check with: g++ -v) so you could skip:
sudo apt-get install gcc-5 g++-5
sudo update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-5 60 --slave /usr/bin/g++ g++ /usr/bin/g++-5

Guess it is no longer an issue as you did not respond or close the issue

No this is still an issue, I have reverted back to the previous commit. Like I said the current release has broken the build in Debian.

Well, have you read my response at all? I did various builds on Jessy and it is definitly not broken! Go read the wiki page i indicated. The error you have is simply that the boost libraries can not be found.
If you did not follow the wiki you must check the paths in the makefile: casinocoin/src/makefile.unix

Yes I read your reply.
How do you explain it working perfectly fine on the exact same server with the previous commit then?

I just want to let you know that I'm not your stock standard wallet user, I am a pool owner with 20+ mining pools and I maintain wallets on a daily basis (Casinocoin is among them).
Give it a go yourself if you don't believe me, compile Casinocoin in Debian 8.

The last commit made some changes to the Makefile including BOOST_LIB_SUFFIX=-mt so if your boost libraries have a different suffix it will not work and you'll get the error you have. What are the filenames of your boost libraries in stage/lib of your boost build directory?

I assume you are asking about this directory? /usr/include/boost

I really need to know if you followed the Wiki at all? In that case you would have the correct Boost and LevelDB libraries in your $HOME/deps folder and that is where the Makefile expects them. If you did not build the dependencies from the correct indicated versions from the wiki but installed them from packages it will not work.


Add License to Project

@yakiradixon, I was thinking of adding a license to the project. Following the guidelines of the Awesome proejct, I would go for the CC0 license (public domain).
Since you're the original creator of this project, I wanted to make sure that this is OK with you.
As you can see, the project is gradually growing and attracting visitors (it's even mentioned in the homepage of the Spring Thing 2019 competion). But it really needs to be licensed.

Hey! I think a CC0 license is appropriate for this project.
Thanks for checking in.

Thanks for the approval!

Hey! I think a CC0 license is appropriate for this project.

Indeed! this is the last of the prerequisites that must be met for adding this repository to original Awesome list of all Awesome lists:
https://github.com/sindresorhus/awesome/blob/master/pull_request_template.md
All other requirements were met, so I'll now be able to make a PR for inclusion.


armstorage table properties.signedIdentifiers missing from list API response

Bug Report
github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/storage/armstorage v1.2.0
go 1.19.1
https://learn.microsoft.com/en-us/rest/api/storagerp/table/list?tabs=HTTP#table doesn't seem to include properties.signedIdentifiers in the actual response (properties.tableName is the only included property) even though an access policy is set on the table. To access the signedIdentifiers field, one needs to do a second Get call on the table.
Is this expected behaviour?

@disq We are looking into it and get back to you for any additional information.


sudo-font: 0.51 -> 0.52

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 102405 run on x86_64-darwin 1

1 package built:

sudo-font


chore(master): release 1.1.22

:robot: I have created a release beep boop
1.1.22 (2023-07-12)
Miscellaneous

deps: update dependency pestphp/pest to v2.8.2 (44dc814)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-symfony-serializer/releases/tag/v1.1.22 :sunflower:


Windows Authentication with Anonymous Authentication

Basically I am trying to have both options enabled for these cases:

Allow anonymous user to make a POST with login information to an API login controller than writes a JWT token back to the body after manual authentication.
Allow windows users to browse to the MVC application through a browser, using their windows identity to authenticate. (They don't see this happen, a.k.a pass-through authentication against our user database which is partially linked to AD)

Problem:
After enabling Anonymous Authentication in my IIS, the user will no longer automatically authenticate. My guess is anonymous authentication took precedence over windows authentication NTLM?
I used IClaimsTransformer to get a ClaimsTransformationContext to query the Principal.Identity to get the windows authenticated user and query that against our database and retrieve our custom claims.
Question:
Can I use both authentication types the way I have explained?

Yes, we are currently running this same setup (Anonymous, Windows Auth with Kerberos). The key is that you need to separate out the authentication from the authorisation

@jruckert Do you know of any examples that are publicly available that you speak of? I'm not quite understanding what you mean by separating the authentication from authorization.

Try something like:

Create/Inject your own authorisation logic - ours is contained within our own IAuthoriseResourceService (e.g. performs the .IsInRole checkes etc)

then decorate your MVC Controllers with the
[TypeFilter(typeof(AuthoriseResourceProvider), Order = -1)] public class BaseController : Controller { }
`using Microsoft.AspNetCore.Mvc;
using Microsoft.AspNetCore.Mvc.Filters;
public class AuthoriseResourceProvider : IActionFilter
{
    private readonly IAuthoriseResourceService authoriseService;

    public AuthoriseResourceProvider(IAuthoriseResourceService authoriseResourceService)
    {
        authoriseService = authoriseResourceService;
    }

    public void OnActionExecuting(ActionExecutingContext context)
    {
        if (!this.authoriseService.Authorise(context))
        {
            Fail(context);
        }
    }

    protected virtual void Fail(ActionExecutingContext context)
    {
        if (authoriseService.AuthenticateOnFail)
        {
            context.Result = new UnauthorizedResult();
        }
        else
        {
            context.Result = new NotFoundResult();
        }
    }

    public void OnActionExecuted(ActionExecutedContext context)
    {
    }
}

`

@jruckert Is that the only route I can take?
I am confused why I have an attribute available [AllowAnonymous] and it doesn't work out of the "box". I posted here because I felt this was an IIS issue not allowing me to use both Windows Authentication and Anonymous Authentication. The behavior acts like Anonymous authentication overwrites Windows Authentication. Is that intended? Why can I not find documentation on that?

@HaoK an AuthZ question for you.

This may help: https://github.com/aspnet/Security/issues/1062

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


Photos/Videos download and upload progress indicator

Hi,
It seems this sdk supports only start and completed/failed state for download/upload progress indicator . Can anyone help me how to show users percentage of download/upload indicator while download/uploading is progress?
Thanks

I was just looking into this. Check the sample code since it does this. Basically your implementation of PhotoMessageViewModel can set:
            self.transferProgress.value = percentComplete

Which seems to work for me. I'm using Alamo Fire, so I do:
        }.downloadProgress() { progress in
            self.transferProgress.value = progress.fractionCompleted
        }

you can check this gist to see how to create your view model and download an image with progress indicator.


Names in the JSON ABI should be call paths

There is nothing stopping an ABI method from looking like this:
abi MyContract {
    fn test_function(a: lib1::Foo, b: lib2::Foo) -> bool;
}

Actually this won't compile today because call paths are not allowed as types of function parameters, but in principle, this should eventually work and the JSON ABI should distinguish between the two types called Foo. We may want to also think about function names and whether they need call paths themselves.
We also need to think about situations where a contracts imports another contract's ABI. See https://github.com/FuelLabs/fuels-rs/issues/791

Closing this as https://github.com/FuelLabs/sway/pull/4137 is now merged. @esdrubal let me know if there's anything else left to do here.


[Refactor]: Users module refactor to prevent rerenders.

This takes care of all the duplicate re-renders in users module.
some data
before

after

Looks good!


Check that URL is valid when submitting

-Needs to check the corresponding REGEX or use an external library for the work

Done


SERVICE__INCLUDE?

From the docs I can see there is a way of excluding exposed ports that I do not want to include in the service registry using SERVICE_<port>_IGNORE.
Is there a way to turn that on it's head and only include ports that I know are required?
E.g. if I have a container that exposes 80 and 443, and I only want to register port 443, is there a SERVICE_443_INCLUDE or equivalent, resulting in port 80 being ignored?

I second this, if you switch to kubernetes, you have no way to set neither expose nor bind for the port, which as a result force you to rebuild the image just to make it expose, and what is most important it requires rebuild if you want to expose additional port to already exposed.

is there any chance this feature could be implemented? it would be great to be able to set the registrator to not autoregister and only register containers that have a specific  env var or label

+1


Write ATS for TG requirement 10

An INSPIRE View service shall contain the INSPIRE metadata elements set out in the Metadata Regulation INS MD as shown in Table 3.

Regarding the difference between 7 and 10: I think 10 is just more specific, let's say 10 is an "implementation" of 7.
We could say that 7 is not testable automatically, because in my opion it is more an explanation/description than a specification. What do you think?

I agree


Request: STM32 compatibility

Hi!
Would be possible to add compatibility for STM32 boards? (via Arduino_STM32)

Wish I had time. Not currently.


Testing with larger data sets and pagination

Also requests 2.28.0

Also tested with updated certifi in requirements.txt.


Bump Go to 1.18

Signed-off-by: Luca Comellini<EMAIL_ADDRESS>
Pull Request Motivation

Bumps Go from 1.17 to 1.18.
Also updates rules_go and bazel_gazelle for Bazel.
Kind

/kind cleanup
Release Note

cert-manager now uses Go 1.18.

/assign @irbekrm

Thank you for the PR!
/ok-to-test

/test pull-cert-manager-e2e-v1-23

Thanks for the PR! Before we merge this, somebody should go through the release notes and check for any changes potentially related to our codebase https://go.dev/doc/go1.18
If you'd like to have a go feel free, but appreciate that it might be proportionally a lot of work with a less familiar codebase

/test

/test pull-cert-manager-make-e2e-v1-24
/test pull-cert-manager-e2e-feature-gates-disabled
/test pull-cert-manager-e2e-v1-20
/test pull-cert-manager-e2e-v1-21
/test pull-cert-manager-e2e-v1-22
/test pull-cert-manager-e2e-issuers-venafi-cloud
/test pull-cert-manager-issuers-venafi-tpp

known flake
/test pull-cert-manager-e2e-feature-gates-disabled

@irbekrm I've removed // +build and ran go fix. I've left it in a different commit to make it easier to see the changes, because I think something's off with bazel but I', not 100% sure.
Also go fix fixed unrelated stuff that I don't know if we want to include in this PR

/retest

/retest

Thanks for making the changes! I think it's just the goimports that is failing - left a comment for that.

@irbekrm I've reverted the change and added back // +build tools in tools.go since it was making Bazel fail.

/retest

Thanks @lucacome !
/lgtm
/approve

/hold cancel


Update request: ffmpeg-normalize 1.26.1 → 1.28.3

Package name: ffmpeg-normalize
Latest released version: 1.28.3

Current version on the unstable channel: 1.26.1
Current version on the stable/release channel: 1.26.1

[x] Checked the nixpkgs pull requests

Notify maintainers
@prusnak

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Fix Windows build failures - version 2.8.0

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
appears multiple windows builds failed with version 2.8.0
The error appears intermittent - and a rerun failed checks might fix the issues -

ERROR getting output validation information from the webservice: JSONDecodeError('Expecting value: line 1 column 1 (char 0)')
copy results:
{}
Failed to upload due to copy from staging to production channel failed. Trying again in<PHONE_NUMBER>8100586 seconds
Traceback (most recent call last):
  File "C:\Miniforge\Scripts\upload_package-script.py", line 10, in <module>
    sys.exit(upload_package())
  File "C:\Miniforge\lib\site-packages\click\core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "C:\Miniforge\lib\site-packages\click\core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "C:\Miniforge\lib\site-packages\click\core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "C:\Miniforge\lib\site-packages\click\core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "C:\Miniforge\lib\site-packages\conda_forge_ci_setup\build_utils.py", line 248, in upload_package
    retry_upload_or_check(
  File "C:\Miniforge\lib\site-packages\conda_forge_ci_setup\upload_or_check_non_existence.py", line 317, in retry_upload_or_check
    raise TimeoutError("Did not manage to upload package.  Failing.")
TimeoutError: Did not manage to upload package.  Failing.
##[error]Cmd.exe exited with code '1'.

Installed packages
Conda build

Environment info
conda build

Thank you @timkpaine - able to use 2.8.0, windows python 3.10 now. Appreciate it.
Just to confirm - you want to keep the vs compiler open -- but no plans to upgrade boost yet.

https://github.com/conda-forge/perspective-feedstock/issues/149 &
https://github.com/conda-forge/perspective-feedstock/issues/150

Regards.


Bug: actor not unmonitoring finished task

I have an actor that processes a task via task.async and task.try_await.
I always seem to get Actor discarding unexpected message: DOWN after the task has finished.
As I understand it, the actor that creates the task monitors it.
The task is completed and goes 'down' and the actor reports this as a warning.
As suggested by @lpil on Discord, changes seem to be needed in these places, i.e. adding a call to process.demonitor_process:
https://github.com/gleam-lang/otp/blob/e4478fec30b36550a02caefb09f686cde2af5d33/src/gleam/otp/task.gleam#L89-L91
https://github.com/gleam-lang/otp/blob/e4478fec30b36550a02caefb09f686cde2af5d33/src/gleam/otp/task.gleam#L119-L123
To reproduce:
pub fn main() {
  let loop = fn(msg, state) {
    task.async(fn() { state })
    |> task.try_await(500)

    Continue(state)
  }

  assert Ok(subject) = actor.start(0, loop)

  actor.send(subject, 1)
  process.sleep(1000)
}

Variants to test:

try_await
try_await_forever

Thank you


"extract" method should be updated

Hello,

result = pd.concat(jtsd, axis=1, join_axes=[jtsd[0].index])

seem to be not supported any more in Pandas ja yields an error "concat() got an unexpected keyword argument 'join_axes'"
If I substituted the row with:

result =  pd.concat(jtsd, axis=1).reindex(jtsd[0].index)

this seems to work fine and no error is thrown! Please check and change the code if neccesary.

Fixed in 3963e6d2bf493034b2ef7b420b24df56c8679f37 and published in version 3.1.2.
Thanks!


Remove FEATURE_GATES variable

Both variables are in GA (since 1.13/1.12) and no longer required, see: https://kubernetes.io/docs/reference/command-line-tools-reference/feature-gates. with the Feature gates still provided the test will fail always since the API server doesn't come up: http://logs.openlabtesting.org/logs/20/520/18b453d905868ac448a664b7c897edac4e680532/cloud-provider-openstack-acceptance-test-csi-cinder/cloud-provider-openstack-acceptance-test-csi-cinder/7b6f0b5/logs/kubernetes/kube-apiserver.log
W0305 06:26:38.152427    9523 feature_gate.go:218] Setting GA feature gate CSIPersistentVolume=true. It will be removed in a future release.
Error: invalid argument "CSIPersistentVolume=true,MountPropagation=true" for "--feature-gates" flag: unrecognized feature gate: MountPropagation
Usage:
  hyperkube kube-apiserver [flags]

Probably https://github.com/theopenlab/openlab-zuul-jobs/pull/447 or https://github.com/theopenlab/openlab-zuul-jobs/pull/449 introduced this issue.

@johscheuer Thanks, lgtm

Approve


Hide the low-level features in the docs by using a re-export module

These made the docs harder to browse for little advantage.
Before:

After:

We got in-principle sign-off on this in office hours today (https://xi.zulipchat.com/#narrow/stream/359642-office-hours/topic/2024-10-03), so I'm merging.
This doesn't (or at least shouldn't) make anything private which was previously accessible.


[EventHubs] Add py.typed file

see issue: https://github.com/Azure/azure-sdk-for-python/issues/9572
three packages:

[ ] azure-eventhub
[ ] azure-eventhub-checkpointstoreblob
[ ] azure-eventhub-checkpointstoreblob-aio

fixed by https://github.com/Azure/azure-sdk-for-python/pull/18327


📘 Add robots.txt to docs

Summary

For the docs, I've set up the service site to use the sitemap on liambx. To reflect this, I've added these two rules to robots.txt:

Disallow crawling of all pages under the /docs directory.
Specify the sitemap on liambx for crawling.

ref: https://developers.google.com/search/docs/crawling-indexing/sitemaps/build-sitemap#sitemap-cross-submission-with-robots.txt
Related Issue

Changes

Testing

Other Information

I don't think a changeset is needed for this change, but please let me know if I'm mistaken🙏.

I'm going to mark this as a draft to make some changes.🙏


Modify mean chunk functions to return dicts rather than arrays

These functions return two array chunks each:

The sum of the array
The counts of the array

We need to return these as a single intermediate value.
Previously we did this by constructing an empty numpy array and then
assigning into it.  This doesn't work as well with numpy-like arrays
like cupy and sparse.
Now we return a dict containing two arrays.  This is nice in a few ways,
but does mean that we need to replicate the concatenate=
functionality within our chunk aggregation and combine functions, which
can be tricky.  This commit fails to do it correctly, but may be a good
start.
This is also somewhat suboptimal in a distributed setting because we no longer get special-cased handling of numpy arrays like we used to.  The scheduler will probably miscount the number of bytes that these objects hold and will probably not use ideal serialization and compression.  I think that this is probably ok because they're likely to be small, but it's worth pointing out.
After this we would like to implement the same fix for other functions,
notably the moment_* functions in this same reductions.py file.
Fixes https://github.com/dask/dask/issues/4481
Help or ideas with this would be welcome.  cc @pentschev @jakirkham

[ ] Tests added / passed
[ ] Passes flake8 dask

By the way, @mrocklin if you want to get your commit fixed and merge it, feel free to leave moment_* to me, I just found out it's the next one in line for Dask-GLM+CuPY.

If you like I'm also happy to hand this entire commit to you to finish up
if you're interested :)
On Thu, Feb 21, 2019 at 7:05 AM Peter Andreas Entschev <
<EMAIL_ADDRESS>wrote:

By the way, @mrocklin https://github.com/mrocklin if you want to get
your commit fixed and merge it, feel free to leave moment_* to me, I just
found out it's the next one in line for Dask-GLM+CuPY.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/pull/4513#issuecomment-466033743, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszL8B-HVZPYIAEILdEMsnVJ3dt9Kiks5vPrXPgaJpZM4bEV_j
.

I don't mind taking it over, but the fix is to literally remove the double sum(). All tests passed locally after I did so. If you prefer that I take over, then just close this PR and I'll commit my changes to a new PR.

Pushed.  @jakirkham if you're around to review that would be welcome.

Thinking about extending this to moment, it looks like there is a little bit of a challenge here:
    empty = empty_lookup.dispatch(type(n))
    M = empty(n.shape + (order - 1,), dtype=dtype)

    for o in range(2, order + 1):
        M[..., o - 2] = _moment_helper(Ms, ns, inner_term, o, sum, kwargs)

Here we construct an empty array and then assign into it.
Instead, we might construct many arrays and then concatenate them together.
xs = [_moment_helper(Ms, ns, inner_term, o, sum, kwargs) for o in range(2, order + 1)]
M = np.stack(x2, axis=-1)

(I might have the axis= keyword above wrong.  Also depending on the shape of the output of _moment_helper we might want np.concatenate rather than np.stack.
cc @pentschev

@mrocklin that might work, but couldn't that cause a potential performance issue? I guess the stacked blocks will all remain different memory blocks, right? My concern is more on the case if stacking doesn't cause all the blocks to be moved to a single memory block, thus involving allocation and copy.

And also, I'd like to point out that this will still require a lookup for empty_like, which is simple to add.

but couldn't that cause a potential performance issue?

I suspect that the data at this stage of the computation will probably be pretty small (it has already gone through a reduction along at least one axis), so I'm not too concerned about performance.

And also, I'd like to point out that this will still require a lookup for empty_like, which is simple to add.

I think that we can avoid this.  We no longer construct an empty array and then assign into it.  Instead we create many arrays and then call np.concatenate.  It's true that np.concatenate will now need to be dispatchable, but I think that that is already handled by the __array_function__ protocol.

Nevermind, you're right, I misinterpreted your suggestion on how to do the stacking. I will work on it, thanks for the suggestions.

Merging this later today if there are no further comments

Nope, not merging.  Adding a failing test.  We still use np.ones_like here as described in https://github.com/dask/dask/pull/4519#issuecomment-466479935
To be honest I'm hoping that @pentschev is able to come up with a solution that I'll be able to steal here :)

Nope, not merging. Adding a failing test. We still use np.ones_like here as described in #4519 (comment)
To be honest I'm hoping that @pentschev is able to come up with a solution that I'll be able to steal here :)

I just tried this new test you added and it fails also in current master (i.e., before the changes in this PR). I think this is something we could (maybe even should) tackle in a different issue?

Opened new issue to tackle the sparse error in https://github.com/dask/dask/issues/4523.
@mrocklin I would suggest reverting the test you added here and later adding it to the issue I just mentioned.

I think this is good to be merged now @mrocklin


detekt CLI fails on Java 17+ in some cases when using detekt-formatting plugin with auto correct

Expected Behavior
I can run detekt CLI without applying any workarounds.
Observed Behavior
It can fail in some circumstances, see https://github.com/detekt/detekt/issues/5247#issuecomment-1328168323
Steps to Reproduce
TBC - opening tracking issue to link a brew issue to.
Context
Users using CLI and Java 17 without --add-opens java.base/java.lang=ALL-UNNAMED applied will see detekt failing in some circumstances.
Your Environment

Version of detekt used:
Version of Gradle used (if applicable):
Gradle scan link (add --scan option when running the gradle task):
Operating System and version:
Link to your project (if it's a public repository):

Still valid.


API Cost Tracking

Is your feature request related to a problem? Please describe.
As a builder I would like to know how using different models impacts my costs.
Describe the solution you'd like
A clear way to track agent costs relative to the requests they are making and storing that information to be easily accessible.
Describe alternatives you've considered
Forking
Additional context
As an agent developer, I want to know how switching between models impacts my costs as well as if I were to build a SaaS model on top of Eliza, how can I know the individual costs of my agents.

I'm working with vivoidos (on discord) discussing ideas around this


Refactor global state

This PR makes the breaking change of making global state accessible through a function .get_global_state() on the RenderCtx, removing it from function parameters. This makes the macros substantially simpler, reduces the number of internal functions that have to be public, and simplifies the process of managing global state. It also advances the reactive state platform to tie reference structs to their intermediaries, which sets the stage for better state import ergonomics (right now, you have to import some shadow types with a * import).
This will also make the macros that handle capsules (#4) much simpler, since they will need to support custom component-level properties, and this reduces the number of cases they'll need to handle. In other words, this makes my job a lot easier over the next month or so!
I understand that not everyone is going to be happy with this change, since accessing global state as a third page parameter is very convenient. However, this should actually make your app slightly smaller and faster, and it will improve long-term ergonomics substantially with the capsules system. It also makes it clearer that global state is not tied to any page, while eliminating the horrific pattern of having () as page state when you wanted to access the global state on a page without any state of its own.

I'll note here as a reminder to myself that this makes it slightly easier to create a BorrowMut panic in an app if one were to directly access the global state, avoiding the wrappers Perseus provides. Why you would ever do that, I don't know. I'll probably make a few more methods private to make this impossible (since advanced thawing logic is already handled by ThawPrefs).

Snap store seems to be broken in CI for downloading Firefox for the E2E tests, so I'm running it all locally before merging...


EXP52-CPP:  exclude decltypes which are structurally equivalent to expressions for which they represent the type

Affected rules

cpp/cert/do-not-rely-on-side-effects-in-decl-type-operand

Description
The rule looks at expression passed to decltype that exhibit side effects.
The goal of this is to detect cases where a programmer relies on side effects that do not occur because the decltype operand is not evaluated.
However, this cannot be the case for dectype usage in function or method signatures, but we still report these.
Example
auto foo(...) -> decltype(<expr with side effects>)
}

Proposed solution
Exclude decltype part of a function/method declaration.

Closing as this change is effectively encapsulated in https://github.com/github/codeql-coding-standards/issues/369.


ci: remove paths-ignore

Codecov Report

Merging #50 (687f9d3) into next (a2496c5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              next       #50   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           23        23
  Lines          249       249
  Branches        13        13
=========================================
  Hits           249       249

Continue to review full report at Codecov.

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

:tada: This PR is included in version 1.1.0-next.4 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Feature]: FlashAttention 3 support

🚀 The feature, motivation and pitch
As you know, FA3 promises 1.5x~ improvements
https://github.com/Dao-AILab/flash-attention/commit/7ef24848cf2f855077cef88fe122775b727dcd74
Alternatives
No response
Additional context
No response

Yes. Actively looking.

any updates??

This would be amazing. FA3 is apparently part of the reason behind Together Inference Engine leapfrogging vllm by a lot!

Very interested in this

+1

+1

A small progress update here. @felixzhu555 did some testing. FA3 is mostly a training kernel at the moment and only benefit extremely large batch size with long context.

Any further update on this?

+1

+1

+1

+1

+1


Adding calculator for set operations

Is your feature request related to a problem? Please describe.

Adding calculator for set operations like union, intersection, etc.

Describe the solution you'd like

Making appropriate amendments in index.html and logicjavascript.js in order to add required functionality to the set calculator

Current state

Kindly assign this issue to me @sairish2001

Go ahead
Deadline 9 April,2021 11:59 PM IST

is this issue still open? can i work on this issue?
as deadline is over

@sairish2001 ,
Sir kindly assign all my issues to other contributors, since due to some odds I wasn't able to accomplish my duty and won't be able to accomplish my duty for next 10 days.

Go ahead
Deadline 22 April,2021 11:59 PM IST


I need some help.

What line in FreeplayState.hx and PauseSubState.hx makes the menus move in a disc-like way? I want to copy the code into my own engine.

I am not sure but I'll check the disc like code.

ok


Add Categories API

cool more PR's, hopefully @rinukkusu will get to these soon :)

he trolling us by comitting on sept 7 but ignoring all the PR's lol

I'll be home next week with less working hours - will revisit all PRs and Issues then. Sorry for the major letdown, but the circumstances of the last months didn't allow me to invest more time.

Awesome. Sorry I was just kinda trolling. Totally get it