Minutes Page

Overview
This PR adds a Minutes page to the navigation. It incorporates some historical events from a csv file in the Minutes page table and includes a UI to filter the events by date.
Checklist

[ ] PR has a descriptive enough title to be useful in changelogs

Testing Instructions

WIP

Handles #770

@hancush quick update: made some final updates to the template (collapsing the list & fixing links), have the date filter ui in place but working on finalizing functionality. In a previous commit, I added logic to group events by date, and I want to try to reimplement that with this updated code.


Pass Header Param

Hi all, trying to pass a header param by setting  custom header like
GIN SERVICE
@Provides
@Singleton
public MyService getMyService() {
     MyService myService = GWT.create(MyService.class);

     String url = GWT.getHostPageBaseURL()+"someURI";

    Map<String,String> headers = new HashMap<String,String>();
    headers.put("X-CustomHeader", "123456789");
    Resource resource = new Resource(url, headers);

((RestServiceProxy) embixUserRestService).setResource(resource);
return embixUserRestService;
}
and MyService esi defined as RestService like this
//SERVICE
public interface MyService extends RestService{
@GET
void someMethodToCall(...);
}
it doen's work but by manipulating method directly it works perfect
Map<String,String> headers = new HashMap<String,String>();
headers.put("X-CustomHeader", "123456789");
Method method = new Resource(url, headers).get();
method.send(...);
it works perfectly.
thanks for efforts

I found a bug which prevents to pass the headers through to the request builder but only if you have a @Path annotation in your method. do you have such a @Path ?

Ur right i have @path annotation, i will investigate this issue
Le 3 nov. 2011 17:55, "mkristian" <
<EMAIL_ADDRESS>a crit :

I found a bug which prevents to pass the headers through to the request
builder but only if you have a @Path annotation in your method. do you have
such a @Path ?

Reply to this email directly or view it on GitHub:
https://github.com/chirino/resty-gwt/issues/75#issuecomment-2619703

cool, then I will push the fix ;-)
On Thu, Nov 3, 2011 at 10:29 PM, hamroune
<EMAIL_ADDRESS>wrote:

Ur right i have @path annotation, i will investigate this issue
Le 3 nov. 2011 17:55, "mkristian" <
<EMAIL_ADDRESS>a crit :

I found a bug which prevents to pass the headers through to the request
builder but only if you have a @Path annotation in your method. do you have
such a @Path ?

Reply to this email directly or view it on GitHub:
https://github.com/chirino/resty-gwt/issues/75#issuecomment-2619703

Reply to this email directly or view it on GitHub:
https://github.com/chirino/resty-gwt/issues/75#issuecomment-2619760

https://github.com/chirino/resty-gwt/commit/92dbf977c350c7c2a3680e2457e98a993041f915
should fix it. I leave it open since I did not add any unit-test for it.

That's because resource ends up creating new resources.
I've worked around that by subclassing resource to preserve the headers.
There's probably a more general solution, but I haven't stopped and thought
about how to do it right.
On Thu, Nov 3, 2011 at 8:55 AM, hamroune <
<EMAIL_ADDRESS>
Hi all, trying to pass a header param by setting  custom header like
GIN SERVICE
@Provides
@Singleton
public MyService getMyService() {
MyService myService = GWT.create(MyService.class);
    String url = GWT.getHostPageBaseURL()+"someURI";

   Map<String,String> headers = new HashMap<String,String>();
   headers.put("X-CustomHeader", "123456789");
   Resource resource = new Resource(url, headers);

((RestServiceProxy) embixUserRestService).setResource(resource);
return embixUserRestService;
}
and MyService esi defined as RestService like this
//SERVICE
public interface MyService extends RestService{
@GET
void someMethodToCall(...);
}
it doen's work but by manipulating method directly it works perfect
Map<String,String> headers = new HashMap<String,String>();
headers.put("X-CustomHeader", "123456789");
Method method = new Resource(url, headers).get();
method.send(...);
it works perfectly.
thanks for efforts

Reply to this email directly or view it on GitHub:
https://github.com/chirino/resty-gwt/issues/75

added a small testcase https://github.com/chirino/resty-gwt/commit/4ba364a892545a8bd322866b399431cb51b81186


Update powered-by.md

Yonghui Superstores Co., Ltd. 永辉超市 http://www.yonghui.com.cn

logo

Coverage decreased (-0.008%) to 12.298% when pulling 72aae2d46697c5fa0aa306e2b14cc98e379f2fc8 on Liu-XinYuan:patch-1 into 76e4c5996d42e2219501e958d3ff7542dfb45daa on apache:master.

Ref #443


[Bug]  Unable to validate certificates bis

Describe the bug
Webhook does not validate certificate
I've already see the other case on the same subject but my update (to mariadb-operator.v0.23.1) doesn't resolve this issue
i'm using Argocd to deploy the operator on openshift and i've change my values.
The certificate seems OK from cert-manager and secret ressources
 webhook:
        enabled: true
        ha:
          enabled: true
          replicas: 3
        cert:
          certManager:
            enabled: true
            issuerRef:
              group: certmanager.step.sm
              kind: StepClusterIssuer
              name: stepissuer-stepclusterissuer
            renewBefore: 8h0m0s
            duration: 168h0m0s

on event , I see readiness probe failed
0s          Warning   Unhealthy                pod/mariadb-operator-webhook-776cf98fcf-vjlt9                    Readiness probe failed: Get "http://<IP_ADDRESS>:8081/readyz": dial tcp <IP_ADDRESS>:8081: connect: connection refused

0s          Warning   Unhealthy                pod/mariadb-operator-webhook-776cf98fcf-vjlt9                    Readiness probe failed: Get "http://<IP_ADDRESS>:8081/readyz": dial tcp <IP_ADDRESS>:8081: connect: connection refused

And i've exactly the same log as the other case related
{"level":"info","ts":1701770446.2099118,"logger":"setup","msg":"Validating certs"}
{"level":"info","ts":1701770456.2198622,"logger":"setup","msg":"Validating certs"}
{"level":"error","ts":1701770466.221084,"logger":"setup","msg":"Unable to validate certificates","error":"context deadline exceeded","stacktrace":"github.com/mariadb-operator/mariadb-operator/cmd/controller.glob..func3\n\t/app/cmd/controller/webhook.go:60\ngithub.com/spf13/cobra.(*Command).execute\n\t/go/pkg/mod/github.com/spf13/cobra@v1.7.0/command.go:944\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\t/go/pkg/mod/github.com/spf13/cobra@v1.7.0/command.go:1068\ngithub.com/spf13/cobra.(*Command).Execute\n\t/go/pkg/mod/github.com/spf13/cobra@v1.7.0/command.go:992\ngithub.com/mariadb-operator/mariadb-operator/cmd/controller.Execute\n\t/app/cmd/controller/root.go:298\nmain.main\n\t/app/cmd/main.go:27\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:267"}

Is the bug related to Galera?
no
Expected behaviour
webhook takes the certificate
Environment details:

Kubernetes version: openshift
mariadb-operator version:  mariadb-operator.v0.23.1
Install method: helm, OLM or static manifests OLM
Install flavour: custom

Hey there,
Thanks for reporting, could you provide the following?

Webhook logs with logLevel = DEBUG
Structure (keys) of your certificate Secret

I'm guessing that, since you use a custom cert-manager issuer, the Secret may be different from what we expect here:
https://github.com/mariadb-operator/mariadb-operator/blob/4b08dfde685249f32d78c941c223f4d7c81d2c03/deploy/charts/mariadb-operator/templates/webhook-deployment.yaml#L53
Resulting in the webhook being unable to find the certificate, and therefore it doesn't become ready.
Thanks!

Related:

https://github.com/mariadb-operator/mariadb-operator/issues/285#issuecomment-1841659548

I'm seeing this issue too while upgrading from 0.22.0 -> 0.23.1, in my case I'm using the manifests and not the helm chart.
My debug logs are a loop of:
{"level":"info","ts":1702909994.9951067,"logger":"setup","msg":"Validating certs"}
{"level":"debug","ts":1702909994.995675,"logger":"setup","msg":"Error reading certificate KeyPair","error":"Error parsing PKCS1 private key: x509: failed to parse private key (use ParsePKCS8PrivateKey instead for this key format)"}
{"level":"debug","ts":1702909994.9957287,"logger":"setup","msg":"Invalid certs. Retrying...","error":"Error parsing PKCS1 private key: x509: failed to parse private key (use ParsePKCS8PrivateKey instead for this key format)"}

I'm not using cert-manager, but generating my own certificates with openssl.

Hey there @gg-kialo !

Error parsing PKCS1 private key: x509: failed to parse private key (use ParsePKCS8PrivateKey instead for this key format

By default, openssl uses PKCS8 but our PKI module tries to parse the private key as PKCS1 so it doesn't work properly with custom certificates, it is intended to be used with certificates generated either by our cert-controller or cert-manager.
Please, see this PR:

https://github.com/mariadb-operator/mariadb-operator/pull/309

In order to use custom certificates, set in --validate-cert=false in the webhook. Something which I'm quite sure that it will fix the issue reported initially by @jbperrin88 .
Closing!
This will be released this week in v0.0.24, please upgrade and let me know how it goes.


[BakaUpdatesMangaReleasesBridge] filter title and groups

Baka-Updates Manga uses an asterisk (*) to denote series information have been updated within the last 24 hours. This is not helpful in a feed.

I agree :smile:

Merged. Thanks for the PR!


KAFKA-17593; [1/N] Introduce re2j dependency

This patch is the first of a series of patches to introduce support for server side regular expression. It introduces the re2j dependency.
Co-authored-by: Lianet Magrans<EMAIL_ADDRESS>Committer Checklist (excluded from commit message)

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

@lianetm Good catch! Fixed!

Out of curiosity, if I recall correctly, Google's RE2 is not fully compatible with Java regex. Could this cause issues if the Consumer accepts Java Pattern, allowing users to use unsupported regex features?


feat(tabs): add theming and ability to set background color

Fixes #1362
Fixes #2908
Fixes #5014

I definitely need to rework this to cut down on how much css is generated

How about something like:
$theme-colors: (
  primary: $primary,
  accent: $accent,
  warn: $warn
);

@each $name, $color in $theme-colors {
  &.mat-#{$name} {
    @include _mat-tab-label-focus($color);
    @include _mat-ink-bar($color);
  }

  &.mat-background-#{$name} {
    @include _mat-tab-label-focus($color);
    @include _mat-tabs-background($color);
 }

 &.mat-#{$name}.mat-background-#{$name} {
  @include _mat-ink-bar($color, default-contrast);
 }
}

cc @andrewseguin

Review on this one might be delayed a bit (Andrew is wrapping up initial data table stuff)

@jelbourn fair, thank you for the update

@andrewseguin done, PTAL

@willshowell Can you rebase again? Sorry!

@kara done!


Some repos don't show in the SOURCE CONTROL window, and cannot operate on them

VSCode Version: 1.39.2 zip 64 bit installer
OS Version: Windows 10

Steps to Reproduce:

Open a folder which contains many different Git repos.
Open one file from each repo, so in the SOURCE CONTROL window, it will show all those repos.
When the number of repos shown in the SOURCE CONTROL window is big enough, some repos aren't able to display in the SOURCE CONTROL window. Because there is no scrolling bar for the SOURCE CONTROL window, thus they aren't able to be clicked on and operated on.

For example in the below screenshot, there are many repos in the SOURCE CONTROL window, but only 12 of them can display in this window. The rest cannot be seen in the SOURCE CONTROL window, so I cannot operate on them at all which is annoying.

/duplicate of #71578 which in turn is pointing to #70888, now closed in favour of #64188


customize notification - set image icon for notification

thanks for Notification
I have problem with setting icon .
when i change sstSmallIcon parameter , Notification's icon is icon_delete
what should i do ?

@nill2110
The notification icon is usually the app icon. But, yeah you can add custom icons to the notification which will require further customization. On Android, it is relatively straightforward.


Crash due to an atomicity violation bug in item_stats_sizes_add()

Describe the bug
There is a possibility of a null pointer dereference at line number 913 in the method: 'item_stats_sizes_add()'in items.c. This is due to the atomicity violation.
Consider the following scenario:
Worker thread, W1, is in item_stats_sizes_add() method and right before dereferencing the 'stats_sizes_hist' field at line 913, items.c, another worker thread, W2, sets stats_sizes_hist to NULL by executing item_stats_sizes_disable() method at line 901, items.c. This can cause a NULL pointer dereference.
To Reproduce
Consider the following exemplary test case:
my $server = new_memcached("-m 2 -t 4 -o track_sizes");
my $sock = $server->sock;
my $sock1 = $server->new_sock;
my $i = 0;
while($i < 10000){
$i ++;
print $sock "set foo 0 0 6\r\nfooval\r\n";
print $sock1 "stats sizes_disable\r\n";
print $sock1 "stats sizes_enable\r\n";
}
$server->stop;
I'm able to reproduce the bug with the above test case. In order to increase the chances of triggering the buggy program path, you can put a call to sleep(1) after line 912, items.c and a call to sleep(5) after line 904, items.c. Also, try compiling Memcached with '-fsanitize=address' option to get the detailed stack trace of the segmentation fault.
System Information

OS/Distro: Ubuntu 18
Memcached Commit id: ad8dfd265523e0, Branch: Master
Hardware architecure: x86_64

Remarks
This behaviour was detected using a systematic concurrency testing tool, named Nekara, being developed at Microsoft Research.
Thanks,
Udit Agarwal

Thanks!
I reviewed this before it got posted; it's an unlikely crash as you have to keep turning the stats sizes feature on/off. it's only used for tuning and people don't typically turn it on/off constantly. so the impact is relatively minor.
fixing it is harder; adding a global lock protecting the pointer could crater performance. code would also need significant reworking to allow broadcast notify to update the pointer. will have to give it a think. Worst case I remove the online disable feature. :(


Need some examples of orders

I need a simple example of a pending order

I need ...?
docs are pretty clear:

first create a limit order : http://oanda-api-v20.readthedocs.io/en/latest/endpoints/orders/ordercreate.html, see contrib.requests also for classes to simplify order construction
http://oanda-api-v20.readthedocs.io/en/latest/contrib/orders/limitorderrequest.html
then request the pending orders using:
http://oanda-api-v20.readthedocs.io/en/latest/endpoints/orders/orderspending.html
check examples: https://github.com/hootnot/oandapyV20-examples/blob/master/src/market_orders_requests.py, easy to adapt to a limitorder using the LimitOrderRequest from contrib


[Bug][Doc] Change the start or install script command sh in the document to bash

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
In Ubuntu 20.04, using sh to execute the startup and installation scripts under the bin directory reports an error, because Ubuntu uses dash instead of bash by default. In order to be compatible with different types of operating systems, can bash be used to replace sh. The following screenshot is a test of the Ubuntu system:

Documentation Links
https://dolphinscheduler.apache.org/zh-cn/docs/latest/user_doc/guide/installation/standalone.html
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Using bash instead of sh can resolve this problem. Welcome to correct the document.


Add Cython support to editor

From<EMAIL_ADDRESS>on 2010-01-28T02:55:08Z
Currently the editor component does not recognise .pyx files (Cython and
Pyrex source).  I imagine it would be fairly straightforward to allow them
to be syntax highlighted, and add the extra Cython keywords.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=143

From pierre.raybaut on 2010-02-07T02:54:42Z
Actually this is not straightforward because one cannot derive a Scintilla's lexer
class to create another. I would have created a CythonLexer class deriving from
QsciPythonLexer and adding Cython keywords in the process... but unfortunately that
is not possible.
So, the only thing I did was to support .pyx files as Python files: https://code.google.com/p/spyderlib/source/detail?r=06bf26fe122b7d9e64760dc47e6465c6acda4113
Status: Fixed


updating admin notice function

the plugin had a very old method of checking for and displaying the admin notice. this updates that, along with two other minor fixes.

Hey Andrew!
I tested this branch on one of my WP 5.3.2 installs and didn’t notice anything odd.
We should probably bump the version number to 1.0.1 and add this to the read me.

i usually update the readme and version on a small commit to master after my PR


config.enable_dependency_loading

Have set this in staging/production:
config.enable_dependency_loading = true

Without it the lib directory classes aren't loaded (at least in the console). Work out what the preferred option is here.

Rather than switch this on, we can add the lib folder to the eager_load_path rather than the autoload_path.  The former gets scanned post boot if something is required, the latter (currently set) doesn't unless dependency_loading is set to true. So, I'm going to switch it off and add lib to the eager load path.

For ref: https://blog.bigbinary.com/2016/08/29/rails-5-disables-autoloading-after-booting-the-app-in-production.html


more efExample tests and eof exceptions

🗒 Description
add more eof exceptions and tests from efExample by Ori
🔗 Related Issues

✅ Checklist

[x] All: Set appropriate labels for the changes.
[x] All: Considered squashing commits to improve commit history.
[ ] All: Added an entry to CHANGELOG.md.
[x] All: Considered updating the online docs in the ./docs/ directory.
[x] Tests: All converted JSON/YML tests from ethereum/tests have been added to converted-ethereum-tests.txt.
[x] Tests: Included the type and version of evm t8n tool used to locally execute test cases:  e.g., ref with commit hash or geth 1.13.1-stable-3f40e65.
[x] Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.

this tests will be reworked

I think this all has been implemented 2 times as of now.


Fields that are set to 0 are erroneously removed from request payload $params

Issue summary
In Base.php, in the request() function, there is a call to array_filter($params).
https://github.com/Shopify/shopify-api-php/blob/b26b415b7dcdf7daf61e4b3b622ea64e30a8a916/src/Rest/Base.php#L178
If we send a payload where one of the request payload values is 0, (such as we want to set the inventory level of a product to 0, the call to array_filter() removes the entire value where 0 is set from the payload array, and thus the API will respond that the value is missing.
Expected behavior
The value containing 0 should be passed to the API request and not be removed from the request payload.
Actual behavior
The request parameter containing a "0" gets removed and we receive an error, such as:
EXCEPTION encountered! REST request failed: {"available":"Required parameter missing or invalid"}
Steps to reproduce the problem
Send the following request:
use Shopify\Rest\Admin2023_01\InventoryLevel;
use Shopify\Utils;
$this->test_session = Utils::loadCurrentSession(
    $requestHeaders,
    $requestCookies,
    $isOnline
);
$inventory_level = new InventoryLevel($this->test_session);
$inventory_level->set(
    [
        "location_id" => XXXX,
        "inventory_item_id" => YYYYYY,
        "available" => 0
    ],
);

The call will return with the following error:
EXCEPTION encountered! REST request failed: {"available":"Required parameter missing or invalid"}
When setting available to any value > 0, we will get a successful response

Is this issue solved? there has been no response form the developers so far.
PS: GitHub stale bot considered harmful

Is this issue solved? there has been no response form the developers so far.
PS: GitHub stale bot considered harmful

The workaround here is to remove the following line
https://github.com/Shopify/shopify-api-php/blob/c26aee5aaa3ef4e6762b375335318689e59f2714/src/Rest/Base.php#L180
Here is a patch file for v5.3.0 that was generated using https://github.com/symplify/vendor-patches
--- /dev/null
+++ ../src/Rest/Base.php
@@ -177,7 +177,6 @@

         $client = new Rest($session->getShop(), $session->getAccessToken());

-        $params = array_filter($params);
         switch ($httpMethod) {
             case "get":
                 $response = $client->get(

Is this issue solved? there has been no response from the developers so far.
PS: GitHub stale bot considered harmful

Is this issue solved? there has been no response from the developers so far.
PS: GitHub stale bot considered harmful

Hey, sorry for the delay in responding here. The stalebot was removed, but the issue wasn't addressed yet.
I've added it to our tracking now so we can follow it more closely, and we will look into fixing it.
Thank you for your patience on this one!


Diagonal Fix

Added check to see if diagonal tile is found before checking it's walkable

Not sure if there's a more elegant approach but it works so 🤷🏻


Ask for d_monitors dict.

Already sent to D. Caelles

wiki entry was generated from this info (d_items).


Fix multiple issues with Unix domain sockets

Everything looks good. Commit message, CHANGELOG entry, tests, documentation. Thank you!
@homu r+

:pushpin: Commit 8c8cd7b has been approved by fiveop

:zap: Test exempted - status


JavaScript

#JavaScript
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I can see the four topics I learned listed along with a suggestion on what I should study first

updated study guide with a JavaScript file


feat: Parse Cargo's --manifest-path option to determine mounted docker root

This commits adds support for parsing the --manifest-path option to cross. So
far, the Cargo.toml manifest of the crate (or its Cargo workspace) to compile
has been assumed to be in the current working directory. This means, that
relative crate dependencies were not supported, because those paths were not
mapped into the docker container.
Take the following example structure, where my-bin depends on my-lib:
.
├── my-bin
│   ├── Cargo.toml
│   └── src
│       └── main.rs
└── my-lib
    ├── Cargo.toml
    └── src
        └── lib.rs

This commits enables such scenarios, by running cross from . like so:
cross build --manifest-path=my-lib/Cargo.toml --target x86_64-pc-windows-gnu,
as . is mapped as the container's root, and the options passed through to
Cargo.
Related #388 #139 #277 #78 #438
Co-authored-by: Kviring Alexey<EMAIL_ADDRESS>

Would appreciate some feedback on the PR -- is something missing? Would you consider merging this feature upstream?

This would be a great change to be able to get in! 😃

Any updates on this? I would use this with actions-rs/cargo@v1, so a release of this would be awesome.

We are currently using this rebased upon master with no issues building a huge project with microservices and interdependecies.
It has allowed us to migrate from copy pasting Cargo.toml workspace files depending on which software to build for which target to have the files include the correct libraries, to being able to build them much simpler. Please merge

Please rebase instead of adding merge commits, if you need guidance how just ask :D

I think this is the wrong approach now, however it can be fixed without explicit support if we parse cargo metadata for the target (and other) folders. See #684
Am I wrong in this thinking?

Cloud this be closed in favour of #684 ?

implemented/included in #684, thanks!

Could this PR be reopened? The solution implemented in #684 does not solve the problem immediately described by/solved by this PR.
While the effect may be similar, it does not solve the problem if one is not using a workspace. The reason we can't/wish not to use a workspace is that compiling for multiple targets within one workspace can create conflicts with libc versions, as well as with other dependencies compiled against other system libraries.

Have you tried the changes? It implements this pr, 09e5d0eeaa15ddf05930ba6aab73ef589ef65a48 in #684 is this pr.

Aho, apologies. Missed that bit :) Thanks for the response!

No worries!
I'm curious about the libc clashes you mentioned, is it something like https://github.com/cross-rs/cross/issues/724 maybe?


Show info, warn, error by default

Show info, warn, and error logs by default

This looks like a great start, but there are still a lot of console.log calls left.

There are so many console.log calls. I could not fix them all

Can we do a massive sed to replace all console.log -> trace, console.error -> error, etc?

We could do that, but I would probably replace them manually because console.log sometimes makes more sense than info or trace, e.g., when printing usage info.

I agree sometimes you do want console calls but I don't think we want any in our oada services. none of them print use info or anything. I like @abalmos idea, except everyone only used console.log even for errors and warnings which we would not want to just stick in trace.

I have replaced most of console.log calls in da12f6c

This looks good and would be useful to have. There is more that needs to be done, but merging this effort.

Thanks


user offsets can be floats

fix #2597

@Spuds was that one to fix or to leave? Remembering the discussion I'm not sure at that point...

We either had to fix to be consistent between the form and the db ...  both either floats or ints instead of the mix we now have.   Floats I suppose allow more flexibility, even though the usage case seems a bit edgy, so its the way to go.


Create Luxor University.txt

Luxor University
جامعة الأقصر
http://www.luxor.edu.eg/

@MoElkholy802 lib/domains/eg/edu/Luxor: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@MoElkholy802 lib/domains/eg/edu/Luxor: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@MoElkholy802 Thank you for the update! Pull request merged.


Mutation

How to make custom mutation?
I just want to call mutation with signing only 'offerId' and setting 'cartId' in Atoms layer?
Now i got TS error
How to fix? Or i missing something? Thx

Can you reproduce it in https://www.typescriptlang.org/play ? You can import libraries there.

Line 134

I think you misunderstand the usage.
https://github.com/jotaijs/jotai-trpc/blob/9751009852034759df5cd22aa915f2be039964dc/src/createTRPCJotai.ts#L142-L148
The first argument is optional getClient. So, you don't need to specify anything.
👉  https://tsplay.dev/Wzx4LN

and setting 'cartId' in Atoms layer?

Does that mean you want to set the atom value based on the mutation output??
You would be able to create a new atom then.

I just tried to make construction like in 'atomWithQuery', where I set parameters which is required for GET response right in Atom layer (not inside component)
How can I do the same but with 'atomWithMutation'?

Ah, I see. It wasn't a TS issue, but it's more like Jotai usage question.
I think you can create another action atom.
export const cartAtom = atom({ id: '', items: [] })

const deleteFromCartAtom = api.cart.del.atomWithMutation()

export const deleteCurrentFromCurtAtom = atom(null, async (get, set) => {
  const cart = get(cartAtom)
  const output = await set(deleteFromCartAtom, {
    cartId: cart.id,
  })
  console.log(output)
})

Oh! Ok! Thank you
I'll try this approach

So, with that approach i can't set part of arguments on Component layer and another part on Jotai layer - and i need to create 'offerIdAtom' right?

I'm not 100% following, but if it's about how to use jotai apart from jotai-trpc, it sounds about right.


mongodump/monogorestore/monoimport not works for ferretdb

FerretDB version
Starting FerretDB v1.21.0...    {"version": "v1.21.0", "commit": "a3fb274d25bac13aea594ec34d603338bc00bfc3", "branch": "unknown", "dirty": true, "package": "docker", "debugBuild": false, "buildEnvironment": {"-buildmode":"exe","-compiler":"gc","CGO_ENABLED":"0","GOAMD64":"v1","GOARCH":"amd64","GOOS":"linux","go.version":"go1.22.1","vcs":"git","vcs.time":"2024-03-20T06:51:32Z"}, "uuid": "9e87c620-5030-4eae-ae5f-72513027d594"}
Backend
PostgreSQL : 14
Environment

OS: centos 7.9 x86_64
Deployment: Docker using ghcr.io/ferretdb/ferretdb:v1.21.0 image
Deployment details: Docker version 26.0.1,

What did you do?
I exported  source data from mongodb and stored as *.json。 I was tring to use mongoimport tool import *.json to postgresql through FerretDB , but mongoimport tool give me an error with "Failed: error connecting to db server: EOF", at the same time , FerretDB shows an error。
The import command is ：
./tools/mongoimport --mode=upsert  --uri="mongodb://polaruser:123456789@<IP_ADDRESS>:27017/teambition?authMechanism=PLAIN" --collection=projects /root/offline/teambition/database-bak/database/mongodump-json/teambition/projects.json
What did you expect to see?
use mongoimport/mongorestore tool to restore source data successfully
What did you see instead?
mongoimport log:
[root@iZuf6hcv5xn2rlf0fj1dhdZ offline]# ./tools/mongoimport --mode=upsert  --uri="mongodb://polaruser:123456789@<IP_ADDRESS>:27017/teambition?authMechanism=PLAIN" --collection=projects /root/offline/teambition/database-bak/database/mongodump-json/teambition/projects.json
2024-05-17T15:20:20.551+0800    Failed: error connecting to db server: EOF
2024-05-17T15:20:20.551+0800    imported 0 documents
FerretDB log:
github.com/FerretDB/FerretDB/internal/clientconn.(*conn).logResponse
/src/internal/clientconn/conn.go:616
github.com/FerretDB/FerretDB/internal/clientconn.(*conn).run
/src/internal/clientconn/conn.go:317
github.com/FerretDB/FerretDB/internal/clientconn.acceptLoop.func1
/src/internal/clientconn/listener.go:305
2024-05-17T07:20:20.551Z        DPANIC  // <IP_ADDRESS>:33464 -> <IP_ADDRESS>:27017      clientconn/conn.go:170  no response to send to client
(err = )
github.com/FerretDB/FerretDB/internal/clientconn.(*conn).run.func3
/src/internal/clientconn/conn.go:170
runtime.gopanic
/usr/local/go/src/runtime/panic.go:770
github.com/FerretDB/FerretDB/internal/clientconn.(*conn).run
/src/internal/clientconn/conn.go:376
github.com/FerretDB/FerretDB/internal/clientconn.acceptLoop.func1
/src/internal/clientconn/listener.go:305
2024-05-17T07:20:20.551Z        WARN    listener        clientconn/listener.go:310      Connection stopped      {"conn": "<IP_ADDRESS>:33464 -> <IP_ADDRESS>:27017", "error": "panic"}

@peintune thanks for reporting this.
Could you share more log from FerretDB please? I see that there is an error and I would like to see the stacktrace.

@peintune thanks for reporting this.
Could you share more log from FerretDB please? I see that there is an error and I would like to see the stacktrace.

@chilagrow  Thanks for your reply, I have solved this problem by upgrading mongorestore tool to the latested version。

closed


Pass other options to browserify

The remaining settings are all passed through to browserify, you should look at the browserify readme if you want to know more:

This works properly for real now

...bump...

... any comment?

This should really get merged, especially since it's forward-compatible with future versions of browserify.

could we get this merged please?

@ForbesLindesay Why has this not approved yet?? Its been open for a year and a half now...

I'm not really actively maintaining this project anymore.  I've added several other collaborators, but ultimately some other people need to review these changes


New Source Requests “Ignis Comic”

Source name
Ignis Comic
Source link
https://igniscomic.com/
Source language
English
Other details
No response
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

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

according to drakecomic(their discord announcements and comments) they are moving over, that said they have yet to move the backlog of series. (adding a bit of context)

Ignis Comic.zip
I tried making one myself since no one has made it.
*I'm no professional, and this is actually my first time creating a decent extension.


Set up presubmit tests

At minimum they should build the example app (https://github.com/bazelbuild/rules_android_ndk/tree/main/examples/basic) and ideally the resulting apk would be tested on an emulator / virtual device.
See also
https://github.com/bazelbuild/rules_android_ndk/pull/7
https://github.com/bazelbuild/continuous-integration/pull/1425
This might be useful as inspiration too:
https://github.com/DanAlbert/ndk-app-template/blob/main/.github/workflows/build.yml

I have set up the webhook, the presubmit is available for PRs now, see https://github.com/bazelbuild/rules_android_ndk/pull/13 and https://buildkite.com/bazel/rules-android-ndk/builds/2
But there are some failures, it looks like you are trying to put a BUILD file in the Android SDK installation directory? On Bazel CI, that directory is mounted in the docker container from the host VM, so it's not readable. Is there a way to avoid doing that?

That the rule is incidentally creating build files in the real NDK directory is a known issue: https://github.com/bazelbuild/rules_android_ndk/issues/3
I'll work on solving that next

Looks like we'll also need to update the version of the NDK on CI:
https://github.com/bazelbuild/rules_android_ndk/pull/13#issuecomment-1244436877

Presubmit should now work for linux:
https://github.com/bazelbuild/rules_android_ndk/pull/14#issuecomment-1246032523
It may take some time for mac to be set up
Some commits in the course of setting this up:
https://github.com/bazelbuild/rules_android_ndk/commit/40e17b3af75406ab10219313f16a404937f09874
https://github.com/bazelbuild/rules_android_ndk/commit/9caf9643b1454698aa3d19c3e40cbd938f72d5bf
https://github.com/bazelbuild/rules_android_ndk/commit/254cb3b8ffe172c54a6ba6a52942847275f5d523
https://github.com/bazelbuild/rules_android_ndk/commit/2a3c13bd1132fc3f697856aea570ba3386d0e5cb

I think we're all good here

it would be good to also run the apk that's built on an emulator, to make sure the artifact actually works, something like what's done here:
https://github.com/DanAlbert/ndk-app-template/blob/f1558c77f4ac563ccfc1dbd41222d4616900b5ba/.github/workflows/build.yml#L63
but that can be its own issue

https://github.com/bazelbuild/rules_android_ndk/issues/23


Search after dragging out node is missing

I have been using it all these while and I think it is missing out of sudden or maybe after updating ComfyUI?

Duplicate of #219
This is due to the change in comfy front end


CustomPropTypes util

As mentioned in #65 we should start a custom prop type util:

[x] mutually exclusive prop type:
// warn if "description" and any of these others are defined
description: customPropTypes.mutuallyExclusive(['children'])

PR #65 merged


Fonts not getting proper spacing when using Iphones

Describe the bug
There is an issue with Iphones and the <b> adding some spacing on the Iphones
opening from iphone:

after investigation it seems to be an auro issue when the font is BOLD on iphone browsers.
auro site from iphone:

we could create an auro issue since it seems to be a bug with the fonts and IOS browsers:
https://stackoverflow.com/questions/28833958/why-is-ios-safari-adding-extra-letter-spacing#:~:text=I found the solution in this question
Expected behavior
It should not add spaced in between the letters when using the <b> tag

There are a few things to address in this issue submission.
This issue is more associated with WCSS versus Design Tokens. This issue has been moved.
In reviewing this issue, I am not exactly able to reproduce this issue per your screenshot. While I can agree that there is some additional space there is a different result based on the browser used as seen in the following screenshots.
Safari macOS and iOS

Chrome macOS

The code responsible for this UI is located in the normalize.scss file
https://github.com/AlaskaAirlines/WebCoreStyleSheets/blob/7335ca3ec2787d09eb9e4a2f8b2ae5421b6fcdff/src/_normalize.scss#L187-L189
The use of bolder is using the browser engine's faux bolding capabilities and as the Stank overflow link you provided eluded to, it is preferable to use an actual font-weight versus faux bolding.
WCSS uses specific font families per font-weight as seen in this WCSS src file. You can see more specifications for the font weights used here. Effectively we have 100, 300 and 500.
Using the inspector I changed font-weight: bolder to font-weight: 300 and didn't feel it has the impact that bolder has. So an action item we can look at is to talk this over with the designers about adding another font family to the list for an official bold look.
Was this issue brought up by a designer or have you consulted with a designer regarding this issue?


Refactoring for version 5 (merging to "v5" branch).

Here is the same PR for the "v5" branch.
Not sure, but merging directly to master might be simpler at this point. It will create confusion for folks working on master (e.g. XML documentation).

Keeping it in a non-master branch will let me get a local copy of it easier. Maybe after this one goes out we can think about a dev branch that produces pre-release builds to nuget.org. For now I need to play with it a bit locally for my own projects

So if I update to v5, it's going to break any current SpecificationEvaluator<T>. Until we can update our repository I'm stuck with v4 and can't utilise filterable Include().

We have a Generic Repo EFRepository<T> that uses;
private IQueryable<T> ApplySpecification(ISpecification<T> spec)
{
    var evaluator = new SpecificationEvaluator<T>();
    return evaluator.GetQuery(_dbContext.Set<T>().AsQueryable(), spec);
}

If we update the package to v5 it breaks because SpecificationEvaluator no longer accepts a generic.

If this is the only issue, then it can be easily fixed. You can do the following change. We refactored the evaluator to non-generic (instead it's generic on method level now) since anyhow it doesn't have an internal state and there is no need to create allocations. We tried to minimize the memory footprint here.
public IQueryable<T> ApplySpecification(ISpecification<T> specification)
{
  // You can also create an instance `new SpecificationEvaluator()`, but you can use the default singleton instance.

   return SpecificationEvaluator.Default.GetQuery(dbContext.Set<T>().AsQueryable(), specification);
}

But, please take a look at some new additional concepts provided in our base repository. If you like it, you can also use the shipped repository. All you need will be just to inherit from it.
public interface IRepository<T> : IRepositoryBase<T> where T : class
{
}

public class MyRepository<T> : RepositoryBase<T>, IRepository<T> where T : class
{
	private readonly MyDbContext dbContext;

	public MyRepository(MyDbContext dbContext) : base(dbContext)
	{
		this.dbContext = dbContext;
	}

	// Not required to implement anything. Add additional functionalities if required.
}

Btw, all repository methods are defined as virtual, so you can override whatever you don't like.


Added a cmake build script and fixed errors when compiling with GCC

I added this library to a linux project of mine and thought I might as well contribute these fixes back.
Things I fixed (in addition to the cmake build script):
There were a lot of MatchResults that were constructed with falses
I had to replace those with nullptrs
There was also one unused value that I thought was probably meant to be
returned
And also there were illegal references taken, some of which I changed to
const references and some to plain values (iterators)
I also reordered a few initializations which were in the wrong order.
And I fixed the std::locale name on linux

@hhyyrylainen Sweet, thanks. I'm curious though, I made a constructor for the Selector::MatchResult IIRC that takes a boolean argument, so returning false should have constructed a match result denoting no match. What effect does returning nullptr have? It's been a long time since my head was anywhere near this code.

@hhyyrylainen  Nevermind, I can see that returning nullptr gives the appropriate result. I'm not sure what I was doing there, but you're right in your changes. Thanks again, I hope this was helpful. 😄

@hhyyrylainen Added you to the AUTHORS file. Thanks again.

@TechnikEmpire I only found one constructor for MatchResult : MatchResult(const Node* result = nullptr);
What might be happening is that visual studio lets you implicitly convert false to a pointer type, whereas GCC is more strict about this.

@hhyyrylainen Yes I believe you're correct. Visual studio was letting me do something evil. Thanks for catching it. :)


[CIT-512] End-to-end tests for software management

Proposed changes
Add end to end tests for the software management ( CIT-512 )
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactorings that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue
https://cumulocity.atlassian.net/browse/CIT-512
Checklist

[x] I have read the CONTRIBUTING doc
[ ] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Codecov Report

Merging #387 (ec7fd24) into main (febd723) will increase coverage by 2.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #387      +/-   ##
==========================================
+ Coverage   67.37%   69.63%   +2.26%
==========================================
  Files          98      105       +7
  Lines        5894     6485     +591
==========================================
+ Hits         3971     4516     +545
- Misses       1923     1969      +46

Impacted Files
Coverage Δ

mapper/tedge_mapper/src/main.rs
0.00% <0.00%> (ø)

mapper/tedge_mapper/src/mapper.rs
0.00% <0.00%> (ø)

mapper/tedge_mapper/src/sm_c8y_mapper/topic.rs
93.75% <0.00%> (ø)

...mapper/src/sm_c8y_mapper/smartrest_deserializer.rs
96.52% <0.00%> (ø)

sm/plugins/tedge_dummy_plugin/src/main.rs
0.00% <0.00%> (ø)

sm/plugin_sm/tests/plugin.rs
98.86% <0.00%> (ø)

mapper/tedge_mapper/src/sm_c8y_mapper/mapper.rs
0.00% <0.00%> (ø)

...e_mapper/src/sm_c8y_mapper/smartrest_serializer.rs
91.15% <0.00%> (ø)

sm/plugin_sm/tests/plugin_manager.rs
100.00% <0.00%> (ø)

mapper/tedge_mapper/src/collectd_mapper/batcher.rs
82.58% <0.00%> (+0.44%)
:arrow_up:

... and 2 more

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 febd723...ec7fd24. Read the comment docs.


Slightly cooler API

How about allowing the handler to be checked for a return (stream) value?
var wss = websocket.createServer({
    server: someHTTPServer
}, () => fs.createReadStream('bigdata.json'))

It would to be aware of the direction of the returned stream, if its's readable and/or writable. I think you can easily build a module on top of this one, as this is nice and simple.


kfirst backend SID issue

The following test case that should work fails with std::domain_error: incompatible strides, expected unit stride on the master branch.
from gt4py import gtscript
import gt4py.storage as gt_storage
import numpy as np

backend = "gtc:gt:cpu_kfirst"

INT32_VEC2 = (np.int32, (2,))

def stencil(
    input_field: gtscript.Field[gtscript.IJK, np.int32],
    output_field: gtscript.Field[gtscript.IJK, INT32_VEC2],
):
    with computation(PARALLEL), interval(...):
        output_field[0, 0, 0][0] = input_field

default_origin = (0, 0, 0)
full_shape = (1, 1, 1)
input_field = gt_storage.ones(backend, default_origin, full_shape, dtype=np.int32)
output_field = gt_storage.zeros(backend, default_origin, full_shape, dtype=INT32_VEC2)

gtscript.stencil(definition=stencil, backend=backend)(input_field, output_field)
assert output_field[0, 0, 0, 0] == 1

The difference between the gtc:gt:cpu_ifirst and gtc:gt:cpu_kfirst backends in the bindings are that the 0s in the as_sid template in the ifirst backend are 2 in the kfirst backend to refer to the unit stride I axis index.
            gt::sid::shift_sid_origin(
                gt::as_sid<std::int32_t, 3, gt::integral_constant<int, 1>, 0>(
                    input_field),
                input_field_origin),
            gt::sid::shift_sid_origin(
                gt::as_sid<std::int32_t, 4, gt::integral_constant<int, 2>, 0>(
                    output_field),
                output_field_origin),

The failure is on this line in python_sid_adapter.hpp.
@anstaf

Some comments on as_sid usage: the forth template parameter is for optimization. It should be set if it is known in compile time (generation time) which dimension is the innermost (has the stride always equal to 1). My guess (I am not familiar with the python code) that something is wrong here: https://github.com/GridTools/gt4py/blame/cad39952bf339e4762dea77312bc8288275756a2/src/gt4py/backend/gtc_backend/common.py#L24  (github blames @havogt as an author) The quick fix is not to set the forth as_sid parameter at all.

This is a problem with the specific array shape. If you have size one in a dimensions, the stride is not well defined. For the specific example, we would expect strides (1,1,1,1) (in number of elements), but buffer protocol makes it a (2,2,2,1). But with that our check fails. A possible solution would be to skip the check on the C++ side if the dimension of the expected unit stride has size 1.

Here is the problem https://github.com/numpy/numpy/blob/7b22284af25279a9332947d024066c0cb43ef25a/numpy/core/src/multiarray/buffer.c#L511 (and comment above). If we have a buffer that cannot be distinguished from a c-layout as in the code above, numpy recalculates the strides as if they were c-layout. See also, https://numpy.org/devdocs/release/1.8.0-notes.html#npy-relaxed-strides-checking.
Therefore, the only solution is to disable the check for stride 1 in case the dimension has size 1.

Ah ok this makes sense. I suppose the problem now is that when we generate the code we don't know if either the storages or stencil compute domain will have unit strides.

I wrote a python module and script today that prints out the shape and strides both in  (using the strides attribute) and through the buffer protocol in C++ in order to better understand this. I thought I would share the results because they nicely demonstrate the issue.
For each case below the data type is (int, (5,))
shape (2, 3, 4)
backend: gtc:gt:cpu_kfirst
shape: 2 3 4 5
python strides: 96 32 8 192
c++ strides: 96 32 8 192

backend: gtc:gt:cpu_ifirst
shape: 2 3 4 5
python strides: 8 256 64 768
c++ strides: 8 256 64 768

shape (1, 1, 1)
backend: gtc:gt:cpu_kfirst
shape: 1 1 1 5
python strides: 8 8 8 8
c++ strides: 40 40 40 8

backend: gtc:gt:cpu_ifirst
shape: 1 1 1 5
python strides: 8 64 64 64
c++ strides: 8 64 64 64

In the case of singleton dimensions, the cpu_kfirst backend buffer's protocol has the unit stride on the additional dimension (in array-of-structures fashion) instead of striding the extra dimensions last. There is perhaps little we can do about this while using numpy to allocate the buffers and create the buffer protocol.
Furthermore, even if we were to avoid the error, this would not resolve the problem, because the numpy array indexing will access elements differently than the gt4py backend will. The current behavior is the only correct, sane behavior.
I find it particularly interesting that the python strides do not match the buffer protocol in all cases. From playing with this utility, the unit stride dimension is not the one that the backend expects whenever the python strides do not match those from the buffer protocol (you alluded to this earlier). The rules under which I can get them to match are not entirely clear, e.g. with (2, 1, 1) the python and c++ strides do not match, but with (2, 2, 1) they do.
Should we consider having GridTools allocate the storages in gt4py and return a buffer protocol back to python using the pybind11 utilities? That way we could ensure that the data dimensions go last, and we ensure that the strides and layout are those that GridTools thinks are best for the backends. This might sound complicated, but it actually isn't that bad. This post walks through implementing one using the C bindings, and could be made even simpler using the pybind11::buffer_protocol object described here.

There is no real issue here, we only need to work around the check in the buffer protocol SID adapter by dispatching on the size of the (supposed to be) unit stride dimension.

Because I got tagged here, I'll just add that I agree with Hannes. These strides are essentially to be considered undefined because they don't matter. So we need to just exclude these cases from the check, which needs to be done in the SID adapters in the C++ GridTools repository.

C++ PR is ready for review: https://github.com/GridTools/gridtools/pull/1651


manage.py populate_history broken with Django 1.10

The option_list method of BaseCommand was deprecated in Django 1.8 and has been removed from Django 1.10:
https://docs.djangoproject.com/en/1.8/howto/custom-management-commands/#django.core.management.BaseCommand.option_list
python manage.py populate_history --help
Traceback (most recent call last):
File "manage.py", line 10, in
execute_from_command_line(sys.argv)
File "/home/nigel/.virtualenvs/sarah/lib/python3.5/site-packages/django/core/management/init.py", line 367, in execute_from_command_line
utility.execute()
File "/home/nigel/.virtualenvs/sarah/lib/python3.5/site-packages/django/core/management/init.py", line 359, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/home/nigel/.virtualenvs/sarah/lib/python3.5/site-packages/django/core/management/init.py", line 208, in fetch_command
klass = load_command_class(app_name, subcommand)
File "/home/nigel/.virtualenvs/sarah/lib/python3.5/site-packages/django/core/management/init.py", line 40, in load_command_class
module = import_module('%s.management.commands.%s' % (app_name, name))
File "/home/nigel/.virtualenvs/sarah/lib64/python3.5/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 986, in _gcd_import
File "", line 969, in _find_and_load
File "", line 958, in _find_and_load_unlocked
File "", line 673, in _load_unlocked
File "", line 665, in exec_module
File "", line 222, in _call_with_frames_removed
File "/home/nigel/.virtualenvs/sarah/lib/python3.5/site-packages/simple_history/management/commands/populate_history.py", line 16, in
class Command(BaseCommand):
File "/home/nigel/.virtualenvs/sarah/lib/python3.5/site-packages/simple_history/management/commands/populate_history.py", line 30, in Command
option_list = BaseCommand.option_list + (
AttributeError: type object 'BaseCommand' has no attribute 'option_list'

As perhaps needs to be stated somewhere more obvious, django-simple-history is not yet compatible with Django 1.10. An updated version of this particular command can be found in the django110 branch (here, specifically) that aims to maintain compatibility with older Django versions.

On 19/12/16 14:57, James Classen wrote:

As perhaps needs to be stated somewhere more obvious,
django-simple-history is not yet compatible with Django 1.10. An
updated version of this particular command can be found in the
django110 branch (here
https://github.com/treyhunner/django-simple-history/blob/master/simple_history/management/commands/populate_history.py,
specifically) that aims to maintain compatibility with older Django
versions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/treyhunner/django-simple-history/issues/253#issuecomment-267984228,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM7gi8U1hfV6ArYfvZpyj9JzunEbHEkGks5rJptmgaJpZM4LQo7L.

Thanks.
I'd installed using pip while Django was at 1.8 and didn't get warned
when I upgraded Django to 1.10.  I don't know if this is a bug with the
package, or beyond pip's capabilities. Anyhow, for now have downgraded
Django to 1.9.2
Nigel

Is there an ETA on supporting 1.10? Are there any other known bugs?

To add to the link @nessalc left, you can uninstall the existing version and use pip to install from the django110 version like so.
pip install git+https://github.com/treyhunner/django-simple-history.git@django110
But use at your own risk.  I'm sure there are reasons that branch is not yet merged into master.

Should be resolved in master. A release will be made once documentation is updated.


Allow changing capacity of underlying Vecs in Graph

As described in #73 . I'm not really sure if the style / layout of the commited code is on a par with petgraph's implementation so feel free to point out what should be corrected.
Fixes #73

Summer's over!

Thanks!


[MVP] Messages

syntax = "proto3";

package isok.agent.message;

import "google/protobuf/timestamp.proto";

// Single job event
message Event {
  string job_uuid = 1;
  google.protobuf.Timestamp created_at = 2;
  Status status = 3;
  Metrics metrics = 4;
  oneof details {
    JobDetailsTcp detail_tcp = 10;
    JobDetailsHttp details_http = 11;
  }
}

// Job status and broker response status
enum Status {
  Unknown = 0;
  Ok = 1;
  Ko = 2;
}

// metrics of a job
message Metrics {
  bool timeout = 1;
  optional uint64 latency = 2;
}

message JobDetailsTcp {}

// job details of an http job
message JobDetailsHttp {
  uint32 status_code = 1;
}

// Batched events send from agent to broker
message Events {
  string zone = 1;
  string region = 2;
  google.protobuf.Timestamp created_at = 3;
  repeated string tags = 4;
  repeated Event events = 10;
}

// Broker message to agent
message Response {
  Status status = 1;
  optional string reason = 2;
}

// message pushed to Kafka topic
message EventComplete {
  string job_uuid = 1;
  google.protobuf.Timestamp created_at = 2;
  Status status = 3;
  Metrics metrics = 4;
  string zone = 5;
  string region = 6;
  oneof details {
    JobDetailsTcp detail_tcp = 10;
    JobDetailsHttp details_http = 11;
  }
}

Please remove optional, as they are not part of proto3 syntax

Hello, thank you for this schema. However i have two issues with it.
The first one is the Status. It's not the agent's role to know what's a "healthy" result or not. They just give information (like status code) and it's the offloaders' role to apply domain rules to decide either it's a OK, KO or Unknown.
The second one is that we're missing the agent's id in the Event message.
Also, what's Metrics ? timeout is the max_latency we have here https://github.com/isok-io/isok/blob/main/isok-data/src/check.rs#L294 ?
If yes, do we want to carry specs of a check (max_latency and so on) in the event ? I'm not sure since we can just access them in the offloaders using the check_id

Hey 👋

The first one is the Status. It's not the agent's role to know what's a "healthy" result or not. They just give information (like status code) and it's the offloaders' role to apply domain rules to decide either it's a OK, KO or Unknown.

I believe we got two level of assessment possible: 1st, whether the agent achieved to reach the target (HTTP URL is available, TCP target port is opened). Second, is based on configuration of the check, to determine if the returned informations makes the check "healthy" or not. The first case relies on the agent working properly or configuration of the check being valid, while the other relies on businness configuration given by the user.
For the 1st case, it might be ambigous. Thus, we can replace ok/ko with specific high level result and have Reachable/Unreachable.
For the second case, as you suggest, we'll handle it through offloaders, as it need business informations that might be given by the control plane.

The second one is that we're missing the agent's id in the Event message.

It does make sense to add it 👍

Also, what's Metrics ? timeout is the max_latency we have here https://github.com/isok-io/isok/blob/main/isok-data/src/check.rs#L294 ?

Metrics would be numerical values that give intel about the execution of the job (processing time, check version).
In the case of an target having a long response time, this would be truthy when reaching a latency given to the agent. It would be a global configuration to protect the system, rather that something given by the user. This information can be added into the enum Status, it's not a metric.

If yes, do we want to carry specs of a check (max_latency and so on) in the event ? I'm not sure since we can just access them in the offloaders using the check_id

As said earlier, we should keep the timeout, as we don't know yet how the agent would be running and handling this kind of job in depth. Business error Too long to respond would be related to latency & the check config max_latency given by control plane.
I agree with you, check results should be light and shouldn't have check configuration embedded.


Lost (in) Translation

Not sure if Jake read this already, in case sorry for the repetition, but of all the translation I have done, two apparently seem gone MIA in this kata.
Or, rather: they can't be found by either me or the kata creator, but I can read them in my profile page (browsing A LOT of other stuff, but ok) and the kata creator somehow manages to find them among forks or something like that, but can't approve them.
The above link should provide you the full discussion (expand my message). Now, I have no big problem copy&pasting said translations into two new ones (provided this work), but I was wondering if that could hinder your search for the problem, or, worse, do some more damage.
Let us (me and suuuzi) about how to proceed and thanks in advance for your attention :)

I wasn't able to find the translation either by doing a quick look through the site, I will try to dig into the console and see if I can't find the item directly and get back to you.

Not sure if it helps, but now that I read their story, suuuzi apparently
approved, forked, deprecated and finally rejected her (and my) version:
http://www.codewars.com/kumite/550df2b3e051b3eca0000540?sel=550df2b3e051b3eca0000540
http://www.codewars.com/kumite/550f5b8dacc1aa61fd000250?sel=550f5b8dacc1aa61fd000250
Though this is apparently was done after when I couldn't see them anymore
already.
Thanks again for your concern!
On Mon, Apr 6, 2015 at 11:52 PM, Jake Hoffner<EMAIL_ADDRESS>wrote:

I wasn't able to find the translation either by doing a quick look through
the site, I will try to dig into the console and see if I can't find the
item directly and get back to you.
—
Reply to this email directly or view it on GitHub
https://github.com/Codewars/codewars.com/issues/23#issuecomment-90256478
.

Hi!
Just to update what happened: I remember not see an approve button (I already approved other translations before, so I know where the button should be, I guess). So I try to fork, and after that I click Approve, Save & Publish and nothing happens. I can't find a record of this translation in any place (only the kumite link that I found in my browser history)!
I really don't know if I did something wrong or if something weird happened.
(sorry for the bad english, i'm learning)

That might be a total shot in the dark, but that sounds like an incomplete
database transaction. Are updates to the database atomic?
Am 09.04.2015 20:43 schrieb "Suzane Sant Ana"<EMAIL_ADDRESS>
Hi!
Just to update what happened: I remember not see an approve button (I
already approved other translations before, so I know where the button
should be, I guess). So I try to fork, and after that I click Approve, Save
& Publish and nothing happens. I can't find a record of this translation in
any place (only the kumite link that I found in my browser history)!
I really don't know if I did something wrong or if something weird
happened.
(sorry for the bad english, i'm learning)
—
Reply to this email directly or view it on GitHub
https://github.com/Codewars/codewars.com/issues/23#issuecomment-91323726
.

If it can help, I remember working on those when there was some turmoil
following the change of hosting provider and the frequent submission time
outs.
On Thu, Apr 9, 2015 at 8:50 PM, Benjamin Kästner<EMAIL_ADDRESS>wrote:

That might be a total shot in the dark, but that sounds like an incomplete
database transaction. Are updates to the database atomic?
Am 09.04.2015 20:43 schrieb "Suzane Sant Ana"<EMAIL_ADDRESS>
Hi!
Just to update what happened: I remember not see an approve button (I
already approved other translations before, so I know where the button
should be, I guess). So I try to fork, and after that I click Approve,
Save
& Publish and nothing happens. I can't find a record of this translation
in
any place (only the kumite link that I found in my browser history)!
I really don't know if I did something wrong or if something weird
happened.
(sorry for the bad english, i'm learning)
—
Reply to this email directly or view it on GitHub
<
https://github.com/Codewars/codewars.com/issues/23#issuecomment-91323726>

.

—
Reply to this email directly or view it on GitHub
https://github.com/Codewars/codewars.com/issues/23#issuecomment-91326144
.


Add option to export test results to CSV

Add an export to csv option for test results.

@Ashesh3 Deploy Preview is failing
Type error: Could not find a declaration file for module 'json2csv'. '/vercel/path0/node_modules/json2csv/lib/json2csv.js' implicitly has an 'any' type.

Should be fixed now.


Add ability to ignore notify email addresses for testing

Ignores notify email addresses containing<EMAIL_ADDRESS>meaning no subsriber will be created but a successful sign up message will still
appear.

Also altered 'db/seeds.rb' as the original when running tests threw a syntax error.


Add LE 08 (2020 Winter)

Add the assignment autograder to the Gradescope and the files to this repo.
Gradescope

Create programming assignment
Name = Problem Set XX (2020 Winter) / Lab Exercise XX (2020 Winter)
Assign points
4 Release date: 5 Oct 2020 11:59 PM
End date: 28 May 2021 11:59 PM
Once programming assignment is created, click on three dots (far right of assignment) and edit the settings.  Increase CPU to 4 CPUs and save.

Local code

Do a git pull on SI 506 master
Review SI 506 2020win/problem_set_XX or lab_exercise_XX and confirm that the README adheres to the style guide.
Test tests_output.py locally and confirm that solution file passes all tests.  Do not forget to delete temporary _solution suffix from import name before generating autograder zip file.
If errors are encounter, fix and issue a PR.
Generate autograder zip file
Upload autograder zip to Gradescope (make sure you are uploading to the correct assignment)
Once the Docker container is built, submit the solutions file to test. If successful, note that test passed in a comment in the issue.

SI506-practice code

Fork and clone this repo
Create a the appropriate directory to store the files

/lab_exercises/le_xx-2020Winter
/problem_sets/ps_xx-2020Winter

Copy the template *.py, solution *.py and README.md and any other required files from the SI506 repo to the SI506-practice repo.
Do a git add, commit -m "Added XX", git push and open a PR.

Feel free to work in concert with other team members.

Lab Exercise 08 (2020 Winter) successfully passed local tests and has been uploaded to SI506 Study Problems gradescope. PR issued to SI506-Practice repo.


getting Forbidden Error

"Google Url Shortener returned the following error: [{"domain":"global","reason":"forbidden","message":"Forbidden"}]"

Same problem here => @MGAyoub  did you find a solution ?


Can't set up the plugin

I have tried to test to add this plugin to my server and I have added it to a bot, I think I have given all the relevant permissions to the bot and I have filled in everything it asked for in the config, but I ended up getting this error when trying to execute a command in discord.
[21:50:05 ERROR] [net.dv8tion.jda.api.JDA]: One of the EventListeners had an uncaught exception
java.lang.NullPointerException: Cannot invoke "java.util.Set.size()" because "accounts" is null
        at dev.orewaee.bot.AccountsCommand.onSlashCommandInteraction(AccountsCommand.java:30) ~[?:?]
        at net.dv8tion.jda.api.hooks.ListenerAdapter.onEvent(ListenerAdapter.java:442) ~[?:?]
        at net.dv8tion.jda.api.hooks.InterfacedEventManager.handle(InterfacedEventManager.java:96) ~[?:?]
        at net.dv8tion.jda.internal.hooks.EventManagerProxy.handleInternally(EventManagerProxy.java:88) ~[?:?]
        at net.dv8tion.jda.internal.hooks.EventManagerProxy.handle(EventManagerProxy.java:70) ~[?:?]
        at net.dv8tion.jda.internal.JDAImpl.handleEvent(JDAImpl.java:176) ~[?:?]
        at net.dv8tion.jda.internal.handle.InteractionCreateHandler.handleCommand(InteractionCreateHandler.java:126) ~[?:?]
        at net.dv8tion.jda.internal.handle.InteractionCreateHandler.handleInternally(InteractionCreateHandler.java:97) ~[?:?]
        at net.dv8tion.jda.internal.handle.SocketHandler.handle(SocketHandler.java:39) ~[?:?]
        at net.dv8tion.jda.internal.requests.WebSocketClient.onDispatch(WebSocketClient.java:1014) ~[?:?]
        at net.dv8tion.jda.internal.requests.WebSocketClient.onEvent(WebSocketClient.java:900) ~[?:?]
        at net.dv8tion.jda.internal.requests.WebSocketClient.handleEvent(WebSocketClient.java:878) ~[?:?]
        at net.dv8tion.jda.internal.requests.WebSocketClient.onBinaryMessage(WebSocketClient.java:1053) ~[?:?]
        at com.neovisionaries.ws.client.ListenerManager.callOnBinaryMessage(ListenerManager.java:385) ~[?:?]
        at com.neovisionaries.ws.client.ReadingThread.callOnBinaryMessage(ReadingThread.java:276) ~[?:?]
        at com.neovisionaries.ws.client.ReadingThread.handleBinaryFrame(ReadingThread.java:996) ~[?:?]
        at com.neovisionaries.ws.client.ReadingThread.handleFrame(ReadingThread.java:755) ~[?:?]
        at com.neovisionaries.ws.client.ReadingThread.main(ReadingThread.java:108) ~[?:?]
        at com.neovisionaries.ws.client.ReadingThread.runMain(ReadingThread.java:64) ~[?:?]
        at com.neovisionaries.ws.client.WebSocketThread.run(WebSocketThread.java:45) ~[?:?]

Try writing empty square brackets ([]) in the plugins/DiscordAuth/accounts.json file. After that, restart the plugin. If the problem persists, please report.

Now I don't see any error on the server log but the discord bot is not responding.... Maybe it's my fault? Idk

Most likely you did not specify your Discord ID in the configuration file (plugins/DiscordAuth/config.toml). This field should look like this:
admin_discord_id = "xxx;yyy"

Using ; you can specify the IDs of trusted users.

Nothing... I share w/you an image of the Config Discord Auth files:

And bot privileges:

In the server the bot was added with Administrator permissions.

So far I can’t give a good answer to your problem. I will try to identify the reasons for this behavior of the plugin in the coming days. If there are any changes, please let me know.


Couldn't validate the request signature.

My bot was working completely fine din't change anything but suddenly I started getting these messages. I'm getting a webhook post but my bot is not responding.
Tried re-validating the webhook, resubscribing the page and also changing the server.
Error: Couldn't validate the request signature.     at BootBot._verifyRequestSignature (D:\Kamal\Work\bothook\msg-me\node_modules\bootbot\lib\BootBot.js:455:15)     at D:\Kamal\Work\bothook\msg-me\node_modules\body-parser\lib\read.js:100:9     at invokeCallback (D:\Kamal\Work\bothook\msg-me\node_modules\raw-body\index.js:262:16)     at done (D:\Kamal\Work\bothook\msg-me\node_modules\raw-body\index.js:251:7)     at IncomingMessage.onEnd (D:\Kamal\Work\bothook\msg-me\node_modules\raw-body\index.js:307:7)     at emitNone (events.js:105:13)     at IncomingMessage.emit (events.js:207:7)     at endReadableNT (_stream_readable.js:1045:12)     at _combinedTickCallback (internal/process/next_tick.js:138:11)     at process._tickCallback (internal/process/next_tick.js:180:9)

Had same issue, mostly likely you have the wrong appSecret set, please counter-check:
const bot = new BootBot({
  accessToken: process.env.PAGE_TOKEN,
  verifyToken: process.env.VERIFY_TOKEN,
  appSecret: process.env.APP_SECRET
});

@Kamalnrf I know it's been a while, if you set up the new BootBot() correctly, double check that your app secret is available.
console.log("MY APP SECRET IS: ", process.env.APP_SECRET);

Then run your bot and check the output.


Calibration

Related to issue: https://github.com/UDST/manta/issues/7

Rebased and merged fast_micro_with_edges_speeds onto main, which has the following changes:

Adds parametrization of a, b, T and s0.
Adds a calibration script of a, b, T and s0 by mini-batch gradient descent.
Adds improvements of the speed of convertVector.
Adds benchmarking to various functions.
Other minor fixes.

Closed since we do not want to merge yet.


fix(config): let webpack merge postcss plugins

🔗 Linked issue
resolves https://github.com/nuxt/nuxt/issues/18814
resolves https://github.com/nuxt/nuxt/issues/18798
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Currently we normalize postcss options at config stage, but modules may modify them later, leaving the options in an impossible situation. This PR defers normalizing them until just before passing them to the webpack loader.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

When will you deploy this issues?

Planning for this week - likely Friday.

When this fix will be deployed?

should I update nuxt version to apply your fix?

Indeed, this fix is in 2.16.1.

How can I update ^2.15.7 to  2.16.1?
by changing the package.json file only?
or should I change something in nuxt.config ?
  // Build Configuration: https://go.nuxtjs.dev/config-build   build: {     transpile: [       'vue-upload-drop-images',       'vee-validate/dist/rules',       'vee-validate/dist/locale/fa',       'vee-validate/dist/locale/en',     ],     postcss: {       plugins: {         tailwindcss: {},         autoprefixer: {},       },     },   },

That's not a problem - it's just saying that the @nuxt/postcss8 module (which existed to enable PostCSS 8 support) is not enabled because your version of Nuxt has built-in PostCSS 8 support.

First of all change package.json :
"nuxt": "2.16.1",

Then I added
postcss.config.js:
module.exports = {
  plugins: {
    tailwindcss: {},
    autoprefixer: {},
  },
}

And change
nuxt.config :
build: {
    postcssOptions: {
      plugins:{
        tailwindcss: {},
        autoprefixer: {},
      },
    },
    postcss: {
      postcssOptions: require('./postcss.config.js'),
    },
  },

It's fixed my problem ❤


Add stack config option for dynamodb permissions

There's already a config option for buckets; there should also be an option to turn on permissions for reading from dynamodb tables.
E.g. I needed:
    perms.append(
        iam.PolicyStatement(
            actions=["dynamodb:GetItem"],
            resources=[f"arn:aws:dynamodb:{deployment_region}:{aws_id}:table/*"],
        )
    )

to give read access to all tables in the defined region. It might make sense to also optionally have a list of table names instead of "all tables in region"

@kylebarron there is a commented block in the stack.py already for this https://github.com/developmentseed/titiler/blob/fa711c0591dc8537caa8ebb68d303cb67f89b0f0/stack/app.py#L187-L199
Given that using dynamoDB table is only for mosaic and for expert user like yourself I didn't feel like adding some code by default. IMO everything in /stack is mostly to be used as example, users should understand they must edit it to serve their custom needs.
For the docs:

The TiTiler repository includes an example that uses AWS CDK to deploy TiTiler on AWS' Elastic Container Service (ECS).

I'll make sure to add a note for the user to understand they should edit the stack.py to fit their needs.


putting several youtube videos on one page does not work as expected

The videos change order or do not display at all. Also it would be good to be able to supply a width.
Hugo Static Site Generator v0.68.3-157669A0 windows/amd64 BuildDate: 2020-03-24T12:04:36Z

I was unable to reproduce the reported problem with Hugo Static Site Generator v0.69.2/extended linux/amd64 BuildDate: 2020-04-24T12:43:32Z using the following markdown:
# Super Chicken

{{< youtube dAHlGeVGp-4 >}}

# Tom and Jerry

{{< youtube M8tqOh8T62o >}}

# Looney Tunes

{{< youtube WDmNwyVNqHE >}}


ALT+TAB delay

🐞 bug report
Describe the bug
After updating to 0.6.11, when switching from another window to the Path of Exile window by pressing ALT + TAB, there is a delay of a few seconds when switching is normal, but the operation is not accepted.
For example, you can't immediately press Enter to open a chat window.
I'm sorry that my English is not good.
🌍  Your Environment

OS: Win10 x64
Version: 0.6.11
PoE: Steam 3.10.0/e English

+1 been experiencing the same issue

Same here

Hi, thanks for creating an issue. Seems like the Game is losing it's focus after Alt + Tabbing. I'm working on a fix. A quick workaround click anywhere into the game - should bring the focus back.

Hi, i've provide a pre-release. Please let me know if it's working now and i'll release it.

@Kyusung4698
Thanks for the quick response, it's fixed!
Thank you very much.

@Kyusung4698
Thanks for the quick response, it's fixed!
Thank you very much.

Awesome! Thanks for your feedback.

While the pre-release does indeed fix the focus loss.
It doesn't fix a related issue that I have found.
I have several application profiles setup for various games. Before today's patch, all of these behaved normally. While POE is focused my corsair POE profile is active, alt-tab out and it switches to my desktop profile.
After patching today not only did I experience the focus loss issue but ICUE can no longer detect that POE is the active application. I'm assuming this is related to the change that caused this issue.

I can confirm the pre-release fix the focus loss issue for me too.

While the pre-release does indeed fix the focus loss.
It doesn't fix a related issue that I have found.
I have several application profiles setup for various games. Before today's patch, all of these behaved normally. While POE is focused my corsair POE profile is active, alt-tab out and it switches to my desktop profile.
After patching today not only did I experience the focus loss issue but ICUE can no longer detect that POE is the active application. I'm assuming this is related to the change that caused this issue.

Hi, the update 0.6.11 introduced keyboard support. In order to enable this i had to change my main window to be focusable. Otherwise i'm not able to capture the input/ focus. I'm not 100% happy with this solution but this also fixed some bugs on older OSs.

But any operation that needs to detect the currently focused application just breaks.
There are a lot of people using application profiles for POE (I don't need a flask macro when I'm on my desktop).

But any operation that needs to detect the currently focused application just breaks.
There are a lot of people using application profiles for POE (I don't need a flask macro when I'm on my desktop).

PoE is still the active/foreground window. Unless a item is priced.

Let me reinstall and I'll follow up. If that's the case outside applications shouldn't be affected, but from what I'm currently experiencing, they are.

Without PoE Overlay Running:

Focused: PoE, Profile: PoE
Alt Tab
Focused: Desktop, Profile: Desktop
Click/Alt Tab/Refocus PoE
Focused: PoE, Profile: PoE

With PoE Overlay Running:

Focused: PoE, Profile: PoE
Alt Tab
Focused: Desktop, Profile: Desktop
Click/Alt Tab/Refocus PoE
Focused: PoE, Profile: Desktop

System does not recognize that PoE is the currently focused application
The same thing happens if I open any of the overlay windows, basically any operation that causes PoE to lose focus


[New game]: Spell Bee

🎮 Game Request
Game logic and basic description :
The user will be given a pronunciation of word in sound format and they have to type in exact same spelling. They will also be provided with meaning of word, and use of the word in sentence. Difficulty increases as user passes each level.
Point down the features
Game points:
Listen to the pronunciation and write correct spelling within time frame.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Markcosay !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

/assign

Hey @Sairajepawar ! , This issue is already assigned to @Markcosay. 💗
You can work on other issues 🚀
If there is anything wrong you can contact us on Discord🕹👀

Hey @Markcosay ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


chore: add abh139.json

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

how can my name be added in list of contributor @Pradumnasaraf @schmelto

My profile is not shown on linkfree


Mapping retry config by context/request parameters

This PR adds the ability to map retry config (total attempts, response timeout) to elements from the context and/or request.
This effectively means that the user can vary retry config by host, method, path, and/or any other element from the context/request.
Feature request issue: https://github.com/line/armeria/issues/3128

@haithamgabr Wouldn't it be simpler to add maxTotalAttempts and responseTimeout and responseTimeoutForEachAttempt to RetryRule instead of introducing new classes?

@haithamgabr wrote:

Great question. This is actually how I tried to approach it at the beginning, and I think it's largely doable, but then I decided the current approach makes more sense for two reasons:

The semantics of RetryRule is largely per-failure: whether to actually retry, and how much time to wait. While maxTotalAttempts is a parameter that encloses all failures, and responseTimeout is needed even before the very first failure.
Consistency with CircuitBreaking APIs. In CB the decorator uses CircuitBreakingRule, along with either CircuitBreaker or CircuitBreakerMapping. In the new retry api, RetryConfig is the same as CircuitBreaker. User supplies a RetryRule along with either RetryConfig or RetryConfigMapping.

If you still think that we should just add maxAttempts and responseTimeout to RetryRule, and implement RetryRuleMapping instead, please let me know, I give it a try.

@haithamgabr That's a good point. I also realized that it doesn't make much sense to use RetryRule because it's meant to be evaluated after receiving the response and timeouts are scheduled before sending a request. What do you think about the proposed approach, @minwoox and @ikhoon? Do you have a better idea?

Consistency with CircuitBreaking APIs. In CB the decorator uses CircuitBreakingRule, along with either CircuitBreaker or CircuitBreakerMapping. In the new retry api, RetryConfig is the same as CircuitBreaker. User supplies a RetryRule along with either RetryConfig or RetryConfigMapping.

I think RetryingClient doesn't need to be consistent with CircuitBreakerClient.
The CircuitBreaker is shared across multiple requests that's why we needed the CircuitBreakerMapping but RetryingClient doesn't.
However, I think it's good to have RetryConfigMapping in terms of providing different configs for different requests. In order to do that, I think RetryConfig should also have RetryRule. What do you think?

However, I think it's good to have RetryConfigMapping in terms of providing different configs for different requests. In order to do that, I think RetryConfig should also have RetryRule. What do you think?

This makes sense. I can adjust RetryConfig to also include the RetryRule, and the new decorator API to accept only the mapping which has everything.

Please go ahead @haithamgabr 👍

Had a chat with @trustin and we thought just having RetryRuleMapping and RetryRule will be enough to handle the situation.
If so, RetryRule would have maxTotalAttempts and responseTimeoutForEachAttempt and the RetryRuleMapping would handle to yield different rules for the requests.
What do you think, @haithamgabr?

Had a chat with @trustin again and it was not a good idea to make the RetryRule have maxTotalAttempts and responseTimeoutForEachAttempt because multiple RetryRules can be combined into one.
So we need RetryConfigMapping, RetryConfig and RetryRule. Sorry about the comments. 😅

@trustin  @minwoox I've moved RetryRule inside RetryConfig, and did all the consequent refactoring. PTAL.

I addressed @ikhoon's comment. Thanks a lot for the extensive review!

@minwoox @trustin Would you be able to take a look soon? Thanks.

I have addressed @trustin's comments.

Addressed @minwoox's comments. Hopefully it looks better now.

Addressed @minwoox's latest comments.

Addressed latest round of comments, hopefully we're getting closer?

@trustin thanks for the approval.
I have addressed @ikhoon's latest round of comments.

I've addressed @minwoox's latest comments.
Thanks all for the approvals, please let me know if there's anything else that you think should change before being merged.

Thank you so much, @haithamgabr 🎉


Fix deprecation warnings in OpenCensusTraceLoggingEnhancerTest.java.

Codecov Report

Merging #1370 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1370   +/-   ##
=========================================
  Coverage     84.56%   84.56%
  Complexity     1453     1453
=========================================
  Files           231      231
  Lines          6766     6766
  Branches        618      618
=========================================
  Hits           5722     5722
  Misses          868      868
  Partials        176      176

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 0a2f579...9f3c34e. Read the comment docs.


use com.livecode.widget.iconpicker

are you opposed to switching the view to the iconpicker widget?

Honestly, I have not even looked at it.  I don't think it was available when I initially wrote it.  I'll take a look.

I'll need to look a bit more, but my initial thoughts aren't all that great.  The one built into the inspector can't resize and the thumb to scroll seems flaky.  Keyboard navigation is not enabled and I'm not sure how you would use it to select multiple icons.  The filter is very handy though.

the reason for using the picker widget is to get rid of the need to do the left-right keyboard navigation, but you're right, the other shortcomings are not great.
there's another shortcoming, because in some circumstances, the widget does not hilite the selected icon, either.
the filter is something that can be added to your project, especially if the search goes across families, and if it is possible to merge multiple families into one view (e.g. all the icons in a stack/project).
lc's icon widget search is only for the current family.

The current UI doesn't have a merge command, but it is pretty easy to do.  If you use the star to select all and copy to a new family and do that for each existing family (using the same new family name) you end up with a combined list.  In this case, adding the family name on copy would be especially helpful.  When implementing search, having an "all" option would be something I'd consider.

Distributing with Polylist is something that I've been thinking about.  If I compiled the stack into an app, I don't know how I would be able to plug into the IDE instance of the engine.  It would be useful for building your families and viewing icons but would not be able to directly load and set the current family (at least I'm not sure how I would do it for Linux & Windows, could use AppleScript on Mac).
This stack was one of my early ones where I used the Geometry Manager along with some custom code in the resizeStack handler.  The GM was one of the areas that I worked on quite a bit early.  I pushed to make it available on mobile.  Really the new ResponsiveLayout library would take care of the icon grid much easier than what I did.  Since I've already done the math, it probably isn't worth changing.

Just loaded the SVG example stack... if you move an icon set into the library and make it the active family and then press the fill list button on the example, you see the new family.


Error building the project

I'm new to Zig, so it's probably something on my end, but I've encountered an error while building the project. Here's the error message:
root struct of file 'std' has no member named 'Build'

Steps to reproduce:
git@github.com:zigster64/zig-http-playground.git
cd zig-http-playground
zig build

This is the output I receive:
% zig build
/Users/diegosiqueira/Playground/zig-http-playground/build.zig:6:21: error: root struct of file 'std' has no member named 'Build'
pub fn build(b: *std.Build) void {
                 ~~~^~~~~~
referenced by:
    runBuild: /opt/homebrew/Cellar/zig/0.10.1/lib/zig/build_runner.zig:231:45
    usage__anon_4534: /opt/homebrew/Cellar/zig/0.10.1/lib/zig/build_runner.zig:242:13
    remaining reference traces hidden; use '-freference-trace' to see all reference traces

System:
Macbook Pro
Apple M1 Pro
macOS 13.3.1

Zig:
% zig version
0.10.1

Please let me know if you need any other information.

Hiya Diego - thanks for bringing that point up.
This playground is used to test out / experiment with the newly released std.http.Server code that only just got released last week sometime.
You are running Zig 0.10.x - which is the latest stable release of Zig, so thats a good thing.
This code here will only work with Zig 0.11.x though :(
I have updated the README to reflect this.
You will find there are heaps of Zig repos around that wont build at all with "stable" Zig 0.10, because even the build process has changed an awful lot with Zig 0.11 - you will get all sorts of frustrating compiler errors.
Its a moving target, so doing anything in Zig continues to be a madhouse for a while longer. (its getting better, but its still a bit bleeding edge).   You will find that most people playing with Zig will be permanently on the latest dev release, and update that every week or 2.
Then stick to stable releases once you have something "in production"
Im on Mac silicon too, and I generally do this :
... download the tarballs from https://ziglang.org/download/ - one for stable 0.10, and one for the latest
mkdir ~/.zig
cd ~/.zig
tar xvfz ~/Downloads/zig-macos-aarch64...version number etc.
ln -s zig-macos-aarch64-VERSION. latest

Then put ~/.zig/latest in your $PATH
Then when I want to swap versions around, its just a matter of linking ~/.zig/latest to whichever version I want to use
Dont forget to go into Settings->Privacy and enable the new Zig binary each time you swap versions :).

I think there are some issues with using Homebrew at the moment to brew install zig --HEAD ... at least there was last week.
Stopped using Homebrew for my Zig installs, and just stick to the official releases from the download site.  Situation will improve as Zig stabilised, but we not there yet.
Hope that helps.


Use trust-dns instead libc lookup for better dns cache control

Fixed in https://github.com/datafuselabs/opendal/pull/869


[SPARK-19235] [SQL] [TESTS] Enable Test Cases in DDLSuite with Hive Metastore

What changes were proposed in this pull request?
So far, the test cases in DDLSuites only verify the behaviors of InMemoryCatalog. That means, they do not cover the scenarios using HiveExternalCatalog. Thus, we need to improve the existing test suite to run these cases using Hive metastore.
When porting these test cases, a bug of SET LOCATION is found. path is not set when the location is changed.
After this PR, a few changes are made, as summarized below,

DDLSuite becomes an abstract class. Both InMemoryCatalogedDDLSuite and HiveCatalogedDDLSuite extend it. InMemoryCatalogedDDLSuite is using InMemoryCatalog. HiveCatalogedDDLSuite is using HiveExternalCatalog.
InMemoryCatalogedDDLSuite contains all the existing test cases in DDLSuite.
HiveCatalogedDDLSuite contains a subset of DDLSuite. The following test cases are excluded:

The following test cases only make sense for InMemoryCatalog:

  test("desc table for parquet data source table using in-memory catalog")
  test("create a managed Hive source table") {
  test("create an external Hive source table")
  test("Create Hive Table As Select")

The following test cases are unable to be ported because we are unable to alter table provider when using Hive metastore. In the future PRs we need to improve the test cases so that altering table provider is not needed:

  test("alter table: set location (datasource table)")
  test("alter table: set properties (datasource table)")
  test("alter table: unset properties (datasource table)")
  test("alter table: set serde (datasource table)")
  test("alter table: set serde partition (datasource table)")
  test("alter table: change column (datasource table)")
  test("alter table: add partition (datasource table)")
  test("alter table: drop partition (datasource table)")
  test("alter table: rename partition (datasource table)")
  test("drop table - data source table")

TODO : in the future PRs, we need to remove HiveDDLSuite and move the test cases to either DDLSuite,  InMemoryCatalogedDDLSuite or HiveCatalogedDDLSuite.
How was this patch tested?
N/A

Test build #71420 has started for PR 16592 at commit 0133463.

cc @cloud-fan I think we really need to do this ASAP for improving the test case coverage in DDL commands, when I do the PR: https://github.com/apache/spark/pull/16587

Test build #71420 has finished for PR 16592 at commit 0133463.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class InMemoryCatalogedDDLSuite extends DDLSuite with SharedSQLContext with BeforeAndAfterEach
abstract class DDLSuite extends QueryTest with SQLTestUtils
class HiveCatalogedDDLSuite extends DDLSuite with TestHiveSingleton with BeforeAndAfterEach

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71420/
Test PASSed.

can we wait for https://github.com/apache/spark/pull/16597?

Sure, no problem.

retest this please

Test build #71526 has started for PR 16592 at commit 0133463.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71526/
Test FAILed.

ok to test

retest this please

Test build #71651 has started for PR 16592 at commit 0133463.

Test build #71651 has finished for PR 16592 at commit 0133463.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class InMemoryCatalogedDDLSuite extends DDLSuite with SharedSQLContext with BeforeAndAfterEach
abstract class DDLSuite extends QueryTest with SQLTestUtils
class HiveCatalogedDDLSuite extends DDLSuite with TestHiveSingleton with BeforeAndAfterEach

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71651/
Test PASSed.

ping @cloud-fan

Test build #71740 has started for PR 16592 at commit b905191.

Test build #71740 has finished for PR 16592 at commit b905191.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71740/
Test PASSed.

@gatorsmile is this PR going to be merged soon, since the ALTER ADD column PR #16626 also depends on this to create test cases for InMemoryCatalog. Thanks!

Let me continue the work now

Test build #73975 has started for PR 16592 at commit b186dcd.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73975/
Test FAILed.

Test build #74037 has started for PR 16592 at commit d19d725.

Test build #74037 has finished for PR 16592 at commit d19d725.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74037/
Test PASSed.

cc @cloud-fan

Test build #74116 has started for PR 16592 at commit ec914dc.

Test build #74116 has finished for PR 16592 at commit ec914dc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74116/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74158/
Test FAILed.

Test build #74160 has started for PR 16592 at commit 64a8f5a.

Test build #74160 has finished for PR 16592 at commit 64a8f5a.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74160/
Test FAILed.

retest this please

Test build #74213 has started for PR 16592 at commit 64a8f5a.

Test build #74213 has finished for PR 16592 at commit 64a8f5a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74213/
Test PASSed.

Test build #74223 has started for PR 16592 at commit 9d4211c.

Test build #74223 has finished for PR 16592 at commit 9d4211c.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 public static class LongWrapper
 public static class IntWrapper
case class CostBasedJoinReorder(conf: CatalystConf) extends Rule[LogicalPlan] with PredicateHelper
  case class JoinPlan(itemIds: Set[Int], plan: LogicalPlan, joinConds: Set[Expression], cost: Cost)
case class Cost(rows: BigInt, size: BigInt)
abstract class RepartitionOperation extends UnaryNode
case class FlatMapGroupsWithState(
trait WatermarkSupport extends UnaryExecNode
case class FlatMapGroupsWithStateExec(

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74223/
Test PASSed.

greate~

ping @cloud-fan

great work! merging to master!


Fix missing translations in 2 locales.

Fixed a missing translation in EN & NL caused by this line:
https://github.com/refinery/refinerycms-authentication-devise/blob/master/app/views/refinery/authentication/devise/admin/users/_form.html.erb#L82
Wasn't sure if a mistake was made in choosing the I18n key but couldn't find a suitable translation elsewhere in the en.yml file, which I assumed was the base file.

Thanks !