update imputation upstream model

Recently we developed one more imputation model known as average local_regional model #360, therefore we need to incorporate this changes in all upstream imputation model. I have updated the logic of imputation as well as ymal file for the documentation. This PR is related to the issue #367

@JamesSLogan , Could you please also check this one dbt CI failing issue. Thanks!

@JamesSLogan , Could you please also check this one dbt CI failing issue. Thanks!

@mmmiah , snowflake is giving an error since it can't find VOLUME_LOCAL_AVG - it looks like this column isn't available to the model, at least in its current state. Now that #360 is merged, you should be able to pull main into this feature branch to get that new column.

@JamesSLogan , this one passed the dbt CI, all checks look good!  James, Could you please review this PR? Thanks!


ENH: async requests with aiohttp as replacement for requests - py35+ only

The python-zeep Project should provide a good Illustration how this could be achieved.

Forthcoming Requests3 will be the better choice:
https://2.python-requests.org/projects/3/


Add TouchEvent support

I would love to see support for TouchEvent! My TeaVM demos all appear static on tablets and phones. Alexey has suggested that I could add these myself, but I know very little about JavaScript and modifying the TeaVM project.
Can anyone look into this and/or describe the steps to add it? Particularly if there is a way to add external to TeaVM (for now - since I have a sizable TeaVM adapter for my Java code, I could get started that way).

You have Event interface, right? You have several interfaces that extend this interface. Why don't you create something like:
public interface TouchEvent extends Event {
    // ...
}

somewhere in your project?


Add option to prefer nearest-neighbour filtering beyond maxNativeZoom

Is your feature request related to a problem? Please describe.
I do use tileLayer, with L.CRS.Simple. The tiles were generated from huge (approx. 60000 x 60000 px) png file and converted to tiles using gdal2tiles.py and then to lossless webp format using cwebp and GNU parallel. The max zoom and the zoom level for base tiles is zoom level 8.
I do have a map tiles (standard 256x256 px) that I wish user to zoom in beyond max zoom (that is zoom beyond 1 pixel from map tile mapping to just one pixel on screen). So I did setup maxZoom and maxNativeZoom appropriately (12, and 8 respectively).
I use the standard renderer (SVG, not canvas), but I think the renderer only really affect vector layers, not raster tiles, as I only see img elements in the DOM tree anyway.
However the web browser uses bilinear filtering by default for scaling images. There are two issues with bilinear filtering of base tiles:

The pixels on the edge of the tile do not have proper information to interpolate seamlessly with neighbouring tiles. So it is quite easy to spot discontinuity. The same or even worse would show up with bicubic interpolation.

The type of tiles I do use actually look better on itself (not just the edge) with nearest neighbour filtering, because they come from the huge screenshot (or set of screenshots) from the game that is mostly pixel art.

Describe the solution you'd like

I would like to set an option for tileLayer that uses non-standard filtering. A CSS attributes I would like to see used are:

  image-rendering: optimizeSpeed;
  image-rendering: -moz-pixelated;
  image-rendering: -o-pixelated;
  image-rendering: -webkit-optimize-contrast;
  image-rendering: -webkit-pixelated;
  image-rendering: pixelated;
  -ms-interpolation-mode: nearest-neighbor;

Even better (alternative or as secondary method, which is complementing above) implement bilinear interpolation on the edge of the tile in the leaflet library itself using some pixel manipulation (be sure to do that in linear color space, not sRGB!), so discontinuities are not visible. Because leaflet would only do interpolation on edges and corners taking into account neighbouring tiles edges, the number of sample lookups (less ~1000 pixels out of 65536 pixels, so about 1%), would be small and performance of doing it in JS directly would be completely sufficient. The interpolated edges (or input pixels) could also be cached in custom cache, so they do not need to be recomputed on every pan operation.

Describe alternatives you've considered

Setting maxZoom to be equal to maxNativeZoom. Just avoid the problem at the loss of functionality.

Using custom CSS:

img.leaflet-tile[src*="tilesdir-webp/8/"]
  image-rendering: optimizeSpeed !important;
  image-rendering: -moz-pixelated !important;
  image-rendering: -o-pixelated !important;
  image-rendering: -webkit-optimize-contrast !important;
  image-rendering: -webkit-pixelated !important;
  image-rendering: pixelated !important;
  -ms-interpolation-mode: nearest-neighbor !important;
}

The src attribute selector part could be removed, as using nearest-neighbour will still work pretty well if not almost perfect (but not perfect when using fractional zoom levels!) for lower zoom levels. But to be specific I do use this src attribute selector, unfortunately it requires knowing URL form and maxNativeZoom level ahead of time.
I needed to use !important, otherwise the CSS properties are not picked by the proper img elements used by leaflet. There probably is some other way do to it tho in CSS.
Almost all browsers do support it in one form or another, and if they don't, fallback to default behaviour (which will happen if the browser simply do not understand any of the above values), is perfectly acceptable. So that requires no extra handling or browser detection in leaflet code.
Additional context
Default behaviour:

Notice visible and distracting discontinuity between base tiles:
Preferred behaviour:

Tested in Firefox 68.0.2esr and Chromium 78.0.3904.97.
Using leaflet 1.6.0

Looks like the same feature request as https://github.com/Leaflet/Leaflet/issues/5883.


Add support for multifile upload with bush ups.

This closes #9.

Incompatible with #21 don't merge both at the same time.

Ready to merge. @BestPig


Add protection against unknown extended type.

Exception is now in place, now I have to find test file that simulates this issue.


Krypton Domain UpDowner Content padding is not applied

Fixed as part of Merge #188
Fix #173: Krypton ComboBox "Closed Content" should have Text Align like the Drop list does
Fix #172: Krypton ComboBox "Closed Content" Padding is not applied
Fix #171: Krypton Numeric "Up Downer" content padding is not applied

Override the Drawing of the text
Fight the ComboBoxEdit Control
Fix some more spelling mistakes
Add a demo to Show the Formatting applied to these drop downs
Make sure backward compatibility is maintained for "TextAlign"


separate panels for matrix view and breadcrumbs

Perhaps the matrix view and breadcrumbs could be separated? Most of the time I find myself needing the breadcrumbs more. Having them at the same pane makes it such that I have to scroll down to see the breadcrumbs

I can definitely do this. There would just be two separate views for each

I have separated the two views. The breadcrumbs trail now appears at the top of the note. This is still buggy, but I'll close this issue and others can be opened for the new bugs


Fix broken old bandle register class

closes #542

ecs does not allow class names to be called by the same name, even in different namespaces, but in this case it must be done. What to do in this case?

I apologize for spam, the local version of ecs on windows works a little wrong

No troubles. Could you squash it into 1 commit? I'll handle the cs

Thank you for looking into this, @uginroot and @TomasVotruba 🙏

Thanks

Just released v2.0.3 with this fix.
Thanks for the help!


fix for tokenProviderCheck

tokenProviderCheck currently does not do anything and never fails. This change tries to fix that by separating provider nil check from the token check
see https://github.com/Azure/aad-auth-proxy/issues/54

@microsoft-github-policy-service agree


Columns don't seem to work properly when used in the top-bar

Im trying to implement an autosizing search bar w/ attached button with the following code:

              <input class="topSearch" type="search" placeholder="Search">

            <button href="#" class="button searchButton"></button>

It works fine on its own, but the second it gets added to the top-bar the size becomes fixed.


Router :: param doesn't want to be optional

Desired Outcome
My goal is to have a topic/:slug? route, that would look like the following in a non-nuxt vue project:

e.g to have a slug as an optional parameter for the /topic route.
The Problem
My pages folder looks like this:

However, the result is /topic/:slug instead of /topic/:slug?:

What am I doing wrong? Is it a bug?

This question is available on Nuxt.js community (#c2279)

If there is topic/index.vue in same directory with the dynamic route, the index.vue will match the directory path /topic, it may be conflict with topic/:slug?, so Nuxt won't append /topic/:slug a optional suffix ?.


cleanup: Add license header and update README.md

Tested with docker-build, docker-push and deploy.

If you regenerate these files do the changes stick around?

It does now, thanks for the pair-programming session. 👍

Added .github/CODEOWNERS to the PR as suggested by @tcrawley-xilinx.
Merging in a jiffy.


Redis::connect(): php_network_getaddresses: getaddrinfo for  failed: nodename nor servname provided, or not known

Larastan Version: 2.6.4
--level used: 7
Pull request with failing test: N/A

Description
This is a little bit of a weird one. When the phpstan cache is empty (phpstan clear-result-cache), I get the Redis::connect error when analyzing code (phpstan analyse). If I continue to run phpstan without modifying anything, I continue to get the error. However, as soon as I modify the file in question (add a line, etc), the error goes stops appearing on the next phpstan analyse run. Additionally, this only seems to happen when the Redis pecl package is installed.
Error returned
 ------ -------------------------------------------------------------------------------------------------------------------
  Line   app/Providers/AppServiceProvider.php
 ------ -------------------------------------------------------------------------------------------------------------------
  18     Redis::connect(): php_network_getaddresses: getaddrinfo for  failed: nodename nor servname provided, or not known
 ------ -------------------------------------------------------------------------------------------------------------------

Laravel code where the issue was found
<?php

namespace App\Providers;

use Illuminate\Support\ServiceProvider;
use Redis;

class AppServiceProvider extends ServiceProvider
{
    /**
     * Register any application services.
     *
     */
    public function register() : void
    {
        $this->app->singleton(Redis::class, function ($app) {
            $pubSubRedis = new Redis();
            $pubSubRedis->connect('<IP_ADDRESS>', 6379);
            $pubSubRedis->select(1);

            return $pubSubRedis;
        });
    }
}

Hello!
Larastan boots up the framework, so this code actually runs.

it executes because larastan boots the container and tries to resolve things from the container to get an accurate understanding of what $app->make(Redis::class) actually returns because make() can take arbitrary strings, such as 'config' aswell as class-strings.
larastan isn't using fully static analysis because there is a lot of "magic" in laravel, i believe @canvural would like that to change eventually but it is a hard problem to solve

is there a stack trace that is output or do you only get that error ?

Issue phpstan analyse --debug


silenthound: unstable-2022-09-02 -> 0-unstable-2022-12-14

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 296621 run on x86_64-linux 1

1 package built:

silenthound


"undefined reference" linker error when compiling for Arduino DUE on BeagleBone Black

Hello again,
Not surprisingly, I can't manage to solve this linker error on my own.
I get the same error if I compile in OS X, but not in Linux Mint (both 64bit).

arm-none-eabi-g++ -o .pioenvs/autogen_due/firmware.elf -Os -Wl,--gc-sections,--relax -mthumb -mcpu=cortex-m3 -Wl,--entry=Reset_Handler -Wl,--start-group -T /home/debian/.platformio/packages/ldscripts/sam3x8e.ld .pioenvs/autogen_due/src/TCM_CLASS.o .pioenvs/autogen_due/src/tmp_ino_to.o -L/home/debian/.platformio/packages/framework-arduinosam/variants/arduino_due_x -L.pioenvs/autogen_due -Wl,--start-group -lc -lgcc -lm .pioenvs/autogen_due/libFrameworkArduinoVariant.a .pioenvs/autogen_due/libFrameworkArduino.a -lsam_sam3x8e_gcc_rel -Wl,--end-group
.pioenvs/autogen_due/src/TCM_CLASS.o: In function TCM_MB_COMPASS::begin(HardwareSerial*, int)': /home/debian/Desktop/due/TCM_compass/src/TCM_CLASS.cpp:14: undefined reference to HardwareSerial::begin(unsigned long)'
collect2: error: ld returned 1 exit status
scons: *** [.pioenvs/autogen_due/firmware.elf] Error 1

My code is available at https://github.com/eliasse/TCM_compass.git
Thanks in advance!
Best Regards,
Elias

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/18723580-undefined-reference-linker-error-when-compiling-for-arduino-due-on-beaglebone-black?utm_campaign=plugin&utm_content=tracker%2F2014790&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F2014790&utm_medium=issues&utm_source=github).

Temporary solution is to replace these lines 29-30 in ~/.platformio/packages/framework-arduinosam/cores/arduino/HardwareSerial.h` by
    virtual void begin(unsigned long);
    virtual void end();

Need to add virtual keywords.
P.S: We are working on this issue. If we return virtual, then you will have problem under ARMv(6,7).

I see!
I'll stay updated. I'm working on an autonomous surface vehicle project that will use an Arduino DUE
as autopilot and a BeagleBone or RPi as intelligence, so it would be very convenient to have an easy way of reprogramming the DUE remotely.
Great work :)

Hi @eliasse!
I updated toolchain for atmelsam platform, but only ARMv7 version, so your project should work on BeagleBone board.

Everything is working :)
Thanks again for an invaluable tool!

Thanks again for an invaluable tool!

Thanks you too for using and reporting! :+1:


Remove all table metrics when a table is deleted

When a table is deleted from a server or a controller steps back and stops being the leader of a table, all metrics should be deleted. Previously just a list of them were deleted. This is a very error prone process. Some metrics were not listed there and some other were removed twice for some reason. The metrics that were not deleted at that point did not disappear until the controller was restarted.
This PR changes the way metrics are removed so we iterate over all enum values and delete any of them that is not global.
In case we want to keep some specific metric, we can just explicitly ignore them in SegmentStatusChecker and SegmentMessageHandlerFactory. About the latest, I'm not 100% sure if this is the place where metrics should be removed from the server. Can @Jackie-Jiang or some other committer confirm that?
Related to #12344

Codecov Report
Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (43dadbf) 61.73% compared to head (e2847de) 34.85%.

Files
Patch %
Lines

...e/pinot/controller/helix/SegmentStatusChecker.java
0.00%
9 Missing :warning:

...g/apache/pinot/common/metrics/AbstractMetrics.java
0.00%
4 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12403       +/-   ##
=============================================
- Coverage     61.73%   34.85%   -26.89%
=============================================
  Files          2428     2352       -76
  Lines        132828   129083     -3745
  Branches      20545    19993      -552
=============================================
- Hits          82007    44990    -37017
- Misses        44811    80861    +36050
+ Partials       6010     3232     -2778

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
0.00% <0.00%> (-61.67%)
:arrow_down:

java-21
34.85% <0.00%> (-26.77%)
:arrow_down:

skip-bytebuffers-false
34.85% <0.00%> (-26.86%)
:arrow_down:

skip-bytebuffers-true
0.00% <0.00%> (-61.59%)
:arrow_down:

temurin
34.85% <0.00%> (-26.89%)
:arrow_down:

unittests
46.72% <0.00%> (-15.01%)
:arrow_down:

unittests1
46.72% <0.00%> (-0.20%)
:arrow_down:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Can we add an integration test to create table and check for metrics existence then delete table and check for the metrics are gone?

I'm not sure how to do that. Do we have any test like that in Apache Pinot to get some inspiration?

Given we had a conflict when controller gauges are being reset, I've added a commit to define gauges as reseteable. Therefore in the future we just need to define the gauge as reseteable in order to reset, which will make more difficult to forget about that.

Can we add an integration test to create table and check for metrics existence then delete table and check for the metrics are gone?

I'm not sure how to do that. Do we have any test like that in Apache Pinot to get some inspiration?

You can add a few lines in the integration tests after addTable and removeTable.

One assumption in this PR is that all the metrics are managed by SegmentStatusChecker. Is that always the case?

Here we are assuming that after calling SegmentStatusChecker.nonLeaderCleanup there are not going to be other threads modifying the controller metrics for this table. In case this assumption ends up being not correct, the metric will be deleted here and then added again by the rogue thread.
SegmentStatusChecker is called by default every 5 mins. Specifically SegmentStatusChecker.nonLeaderCleanup is called when SegmentStatusChecker is executed and there are tables that during the previous iteration were led by this controller but in this iteration they are not (as long as helix knows).
I'm not sure if some rogue thread may be modifying the metrics after SegmentStatusChecker.nonLeaderCleanup is called, but doesn't seem probable given the frequency on which we call SegmentStatusChecker and in case that happens, it won't be super problematic. In fact it would not be worse than what we had before where some metrics were not deleted. Also, in case that happens, we should manually remove metrics in these services that continue to be running and the deletion here should be seen as a protection in case we forget to remove them somewhere else.

It seems this PR is not enough to remove all server metrics. Specifically, the following metrics survive:
name="pinot.server.endToEndRealtimeIngestionDelayMs.meetupRsvp_REALTIME.X",type="ServerMetrics"
name="pinot.server.highestStreamOffsetConsumed.meetupRsvp_REALTIME-meetupRSVPEvents-X",type="ServerMetrics"
name="pinot.server.lastRealtimeSegmentCompletionDurationSeconds.meetupRsvp_REALTIME-meetupRSVPEvents-X",type="ServerMetrics"
name="pinot.server.lastRealtimeSegmentCreationDurationSeconds.meetupRsvp_REALTIME-meetupRSVPEvents-X",type="ServerMetrics"
name="pinot.server.lastRealtimeSegmentInitialConsumptionDurationSeconds.meetupRsvp_REALTIME-meetupRSVPEvents-X",type="ServerMetrics"
name="pinot.server.meetupRsvp.segmentUploadSuccess",type="ServerMetrics"
name="pinot.server.meetupRsvp.segmentUploadTimeMs",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME-meetupRSVPEvents-X.realtimeRowsConsumed",type="ServerMetrics"
name="pinot.server.realtimeOffheapMemoryUsed.meetupRsvp",type="ServerMetrics"

As you can see most of the metrics that survive were either indexed by table name (meetupRsvp_REALTIME-meetupRSVPEvents-X is the name of the kafka client where X is the partition) or the name without type (like pinot.server.meetupRsvp.segmentUploadTimeMs).
pinot.server.endToEndRealtimeIngestionDelayMs.meetupRsvp_REALTIME.X should be deleted in IngestionDelayTracker.removePartitionId. I've added a commit to delete it.
While the following were deleted:
name="pinot.server.meetupRsvp_REALTIME.buildQueryPlan",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.deletedSegmentCount",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.freshnessLagMs",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.nettyConnectionSendResponseLatency",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.numDocsScanned",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.numEntriesScannedPostFilter",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.numResizes",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.numSegmentsMatched",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.numSegmentsProcessed",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.numSegmentsQueried",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.queryPlanExecution",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.queryProcessing",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.requestDeserialization",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.responseSerialization",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.schedulerWait",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.segmentPruning",type="ServerMetrics"
name="pinot.server.meetupRsvp_REALTIME.totalQueryTime",type="ServerMetrics"
name="pinot.server.segmentCount.meetupRsvp_REALTIME",type="ServerMetrics"

I'm still not sure how/what to create a test here. Some metrics take a while to appear (as explained in https://github.com/apache/pinot/issues/12344, we need to wait some minutes to see some metrics). Some other will never appear until we actually do something with the table (like upload them somewhere).
What the suggested test will be checking is:

Whether the lifetime endpoints are being called
Whether deletion methods in metrics registry are working

Both things are out of the scope of this PR and it isn't clear to me how to write these tests. Unless you think this is a blocker, I would prefer to merge it as it is.


About Qt ui component library

It's a great project, but it's a shame the original author didn't update and maintain it.
In this recommend a good UI component library, it is modeled after Microsoft FluentUI development.
GitHub Python  :https://github.com/zhiyiYo/PyQt-Fluent-Widgets
Website: https://qfluentwidgets.com
The sample video  : https://www.bilibili.com/video/BV12c411L73q
If you need, the original author can also provide QWidget C++ version.
SCREENSHOTS：

The original author can also provide QWidget C++ version,according to the answer on Zhihu


discord lags

nice

okay so your latency is totally acceptable in the game, but for me it was terrible yesterday

so much fun

totally

we could switch to matrix/irc

sure, i made amatrix acc: dragon_pi

have to make one too xD

(never used irc for anything xD)

whats your matrix handle?

2.7k ms, pretty accurate by just judging the delay

going up again, I have a matrix acc now too

aaaand worse again, whats your matrix handle now?

imma close this since matrix is working


[Question/Bug] : .bartycrouch.toml permission errors on dependent test build

Steps to Reproduce:
I have a "Project" and also a scheme "ProjectTests" which depends on Project.
Have you ever seen it where "Project" builds fine, but "ProjectTests" has the error below?
Error: The file “.bartycrouch.toml” couldn’t be opened because you don’t have permission to view it.
Error: The file “.bartycrouch.toml” couldn’t be opened because you don’t have permission to view it.

This was a working project at some point, haven't opened it in a long time, but I'm working myself through a variety of weird build errors, this being one of them.
What I've tried:

chmod 777 to the file, it has full permissions..
restart, clear build cache, derived data, etc.

Dunno. Waited a long time since I had a working project

Expected Behavior:

No permissions errors on build
Current Behavior:

Environment

Show environment details

@thejeff77 Sounds like a new access control security feature in macOS Sonoma that probably affects other CLI tools, too.

@Jeehut gotcha, thanks for the info. Curious, do you know any fixes for this?

I've run into this with another tool but couldn't find a quick fix. Didn't search long though, there might be one.

Hello, do you have a solution?
I have the same problem. When I add .bartycrouch.toml to the project I have the same error.
When I delete this file, I get error Localizable.strings not found. File exists in project and even has few languages.
In previous project everything works fine.

@thejeff77 @Jeehut Do you find any solution?

@Dzionis and @Jeehut - Yes, set "user script sandboxing" to NO in build settings.
Should probably be added to the README.

@thejeff77 Done, added it in README. 👍

@thejeff77 Done, added it in README. 👍


Add Python link to README

Mostly doing this to test out bors

bors r+

bors r+

bors r+

bors r+

bors r+

bors r+


Missing primer and/or deeper intro

The spec is generally rather light on explaining the motivation for the spec itself and for its elements. I don't think pointing to OpenAPI is sufficient.
OpenAPI has a fairly straightforward job:  Define a set of operations that are immediately realized by a (synchronous) web service implementation and exposed to an application via proxy code.
AsyncAPI, as I see it, has a more complicated job because it defines flows that involve intermediaries which may not be in control of the API implementation owner, and potentially multiple layers of such intermediaries, and that requires quite more elaboration to achieve interoperability.
For instance, let's consider a backend service in a retail scenarios that sits behind a queue and that understands three different messages. The messages originate from a point-of-sale system and are initially queued up in an on-premises system and then replicated into the backend system with some sort of shovel. That is quite common.
How does the security section in the AsyncAPI map to such a scenario? The backend service API owner is likely neither in control of securing the broker-to-broker integration flow from the on-premises queue to the backend queue, and it's definitely not in control of how the mobile front-end device interacts with the on-premises broker. Does each layer have its own AsyncAPI document that refers to a shared base document?
The security section's discussion around scopes, with the "write:pets" and "read:pets" examples, also seems to imply that there are  application-specific authorization rules being defined here, but how does that work through two layers of middleware?
I also think that AsyncAPI needs to take a much stronger stance on correlated multi-directional flows; I believe that is the niche where the spec can really fill a gap. There are plenty of specs that define schemas for messages and events abstractly (not the least CloudEvents in CNCF), but there's a real gap in defining multi-directional conversations abstractly: Not just request/reply, but also scatter/gather patterns, saga patterns, etc.
How does a typical sequence diagram describing a multi-directional conversation between four parties map to an AsyncAPI document?

I'm not sure I fully understand your point here. Currently, AsyncAPI defines what an application expects on a set of given communication channels. It is application-centered and therefore we don't care about the topology. This way of defining communication has drawbacks but we're working on them primarily at #390 (and related issues at #520). Most probably, a version 3 of AsyncAPI would define what an application does as opposed to what an application expects. This way, a 4-layer system can be described using 4 different AsyncAPI files (one per application/middleware).

How does a typical sequence diagram describing a multi-directional conversation between four parties map to an AsyncAPI document?

Using multiple AsyncAPI files, one per party.

That said, and given your experience, would you mind opening a proposal with a clear solution of how to fix the problems you're concerned about? That'd be helpful.

so far based on the initial description I think you might think of:

kinda spec like https://github.com/serverlessworkflow/specification/blob/main/specification.md that btw soon will release 0.7 and support AsyncAPI
or library like https://github.com/asyncapi/app-relations-discovery that can consume multiple AsyncAPI files and draw an architecture diagram of flows (except it is not able to guess a sequence but just show relations)

Alright, let me check again with LF and I'll get back to you.

Describing workflows through the spec has been lightly mentioned here. https://serverlessworkflow.io is there for that reason and you can find an example in the referenced comment.

@smoya This might be different to a workflow via a state machine. IMO it has more in common with flow-based programming or reactive.


synchronize_time_of_day not working.

Hey there,
I see where you have implemented the synchronize_time_of_day functionality below, however, when setting synchronize_time_of_day to something other than the default '03:00:00' it does not modify the sync time. I'm not sure the idempotency logic allows for it to be run unless synchronize_automatically is changed.
Thanks!
https://github.com/TraGicCode/tragiccode-wsusserver/blob/5d60be343e0db6f2a4c0192b0b44970c9c8012bb/manifests/config.pp#L610-L626

Hey @joeypiccola ,
You are indeed correct.  On the outside this module looks great be deep down inside there is some nasty powershell going on ( wsus is automated with a powershell module of course ).  I would be okay with just adding to the unless statement for now to "technically" change all sync related settings even if only one changes.

Good to know I read it correctly. I'm not suggesting you make the change thought as my direction has changed a bit and I am not sure the module meets my needs anymore. I'm still going to use WSUS but it's with a SQL backend and a few downstreams (for which the module wasn't really built for).


TextBox Text Drag

Currently when I attach a handler to a textbox the default functions which allow for the dragging of selected text between positions in the box stop working.
Is it possible to attach a drop handler to a textbox for allowing the dragging in of an object, but have standard internal drags of text handled by the default object drag and drop?
Using the textbox in the test for issue 85 as the example I am unable to allow for dragging and dropping text within the element once I attach a drophandler. I attempted to add .DragSourceIgnore as I found in the readme but I believe I maybe using it incorrectly or not understanding how it works.
    <TextBox Text="Drop the item here"
             IsReadOnly="False"
             Grid.Column="1"
             Height="30"
             dd:DragDrop.DragSourceIgnore="True"
             dd:DragDrop.IsDropTarget="True"
             dd:DragDrop.DropHandler="{Binding CustomDropHandler}" />

Turns out I needed to set dropInfo.NotHandled = true for cases where my DropHandler should not handle the drop.


feat: adds setValueRef to elements

Description

feat: adds setValueRef to elements

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Issues with long cutset during zipformer training

Hello All,
we are training a zip former model for about 3400 hours of Tamil data. This is in reference with https://github.com/k2-fsa/icefall/issues/1751
We have NVIDIA A6000 50GB GPU. Getting the below error:
2024-12-19 16:44:26,604 INFO [asr_datamodule.py:375] About to create dev dataloader
2024-12-19 16:44:26,604 INFO [train.py:1326] Sanity check -- see if any of the batches in epoch 1 would cause OOM.
/home/armuser/anaconda3/envs/k2/lib/python3.8/site-packages/lhotse/dataset/sampling/dynamic.py:342: UserWarning: We have exceeded the max_duration constraint during sampling but have only 1 cut. This is likely because max_duration was set to a very low value ~10s, or you're using a CutSet with very long cuts (e.g. 100s of seconds long).
warnings.warn(
2024-12-19 16:58:57,481 ERROR [train.py:1345] Your GPU ran out of memory with the current max_duration setting. We recommend decreasing max_duration and trying again.
Failing criterion: single_longest_cut (=162.26) ...
2024-12-19 16:58:57,482 INFO [train.py:1304] Saving batch to zipformer/exp/batch-6c307511-b2b9-437a-28df-6ec4ce4a2bbd.pt
2024-12-19 16:59:01,314 INFO [train.py:1310] features shape: torch.Size([7, 16226, 80])
2024-12-19 16:59:01,315 INFO [train.py:1314] num tokens: 817
Traceback (most recent call last):
File "./zipformer/train.py", line 1380, in
main()
File "./zipformer/train.py", line 1373, in main
run(rank=0, world_size=1, args=args)
File "./zipformer/train.py", line 1225, in run
scan_pessimistic_batches_for_oom(
File "./zipformer/train.py", line 1341, in scan_pessimistic_batches_for_oom
loss.backward()
File "/home/armuser/anaconda3/envs/k2/lib/python3.8/site-packages/torch/_tensor.py", line 487, in backward
torch.autograd.backward(
File "/home/armuser/anaconda3/envs/k2/lib/python3.8/site-packages/torch/autograd/init.py", line 197, in backward
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
File "/home/armuser/anaconda3/envs/k2/lib/python3.8/site-packages/torch/autograd/function.py", line 267, in apply
return user_fn(self, *args)
File "/home/armuser/k2_249/icefall/egs/tamil/ASR/zipformer/scaling.py", line 313, in backward
x_grad = x_grad - ans * x_grad.sum(dim=ctx.dim, keepdim=True)
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 6.86 GiB (GPU 0; 47.54 GiB total capacity; 37.04 GiB already allocated; 3.94 GiB free; 41.56 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
Training command: ./zipformer/train.py   --world-size 1   --num-epochs 30   --start-batch 336000   --use-fp16 1   --exp-dir zipformer/exp   --max-duration 100
Initially had kept the max duration as 150
The training had completed for 4 epochs.Then got the above issue.Loaded the batch.pt file and got as below:
'sequence_idx': tensor([0, 1, 2, 3, 4, 5, 6], dtype=torch.int32), 'start_frame': tensor([0, 0, 0, 0, 0, 0, 0], dtype=torch.int32), 'num_frames': tensor([16226,  2022,  1926,  1744,  1716,  1676,  1613], dtype=torch.int32), 'cut': [MonoCut(id='Regional-Tiruchirapalli-Tamil-1345-202039142853_sent_97', start=0.0, duration=162.26, channel=0, supervisions=[SupervisionSegment(id='Regional-Tiruchirapalli-Tamil-1345-202039142853_sent_97', recording_id='Regional-Tiruchirapalli-Tamil-1345-202039142853_sent_97', start=0.0, duration=162.26, channel=0, text='அவ்வையார் விருது தமிழ்நாட்டில் சமூகநலப் பணிகளை அரப்பணிப்புடன் செயலாற்றியதாக 2020ஆம் ஆண்டிற்கான அவ்வையார் விருதுக்கு தேர்வு செய்யப்பட்ட திருவண்ணாமலையைச் சேர்ந்த சமூக சேவகி திருமதி', language=None, speaker='Regional-Tiruchirapalli-Tamil-1345-202039142853_sent_97', gender=None, custom={'origin': 'giga'}, alignment=None)], features=Features(type='kaldifeat-fbank', num_frames=16226, num_features=80, frame_shift=0.01, sampling_rate=8000, start=0, duration=162.26, storage_type='lilcom_chunky', storage_path='/home/armuser/10TBHDD/CUDA_11.6/icefall/egs/tamil/ASR/data/fbank/train_split/tamil_feats_train_00032581.lca', storage_key='964876,45872,45111,44652,45255,45498,44806,45091,45317,44865,45016,44804,44720,44784,44749,45046,44983,44943,45297,44866,45335,45125,45507,44978,44909,44841,44914,44718,44569,45297,44670,45390,44619,20203', recording_id='Regional-Tiruchirapalli-Tamil-1345-202039142853_sent_97', channels=0), recording=Recording(id='Regional-Tiruchirapalli-Tamil-1345-202039142853_sent_97', sources=[AudioSource(type='file', channels=[0], source='/media/ASR_database/shruthilipi_data/tamil/newsonair_renamed
/Regional-Tiruchirapalli-Tamil-1345-202039142853_sent_97.wav')], sampling_rate=8000, num_samples=1298080, duration=162.26, channel_ids=[0], transforms=None),
Kindly suggest how to go about this issue.

we have a function in train.py to remove long and short utterances,  which is enabled by default.
Please don't disable it.

@csukuangfj  in train.py there is    train_cuts = train_cuts.filter(remove_short_utt). I was not able to find any option for long utt.

@csukuangfj  in train.py there is    train_cuts = train_cuts.filter(remove_short_utt). I was not able to find any option for long utt.

which file.are you referring to?
Please recheck.

@csukuangfj I am using this file :  https://github.com/k2-fsa/icefall/blob/master/egs/gigaspeech/ASR/zipformer/train.py

please refer to librispeech

https://github.com/k2-fsa/icefall/blob/ad966fb81d76c9b6780cac6844d9c4aa1782a46b/egs/librispeech/ASR/zipformer/train.py#L1377-L1385
@bsshruthi22
please read the comment in train.py carefully.

@csukuangfj ok. thanks for your suggestion. .Now the training has resumed. Hopefully it gets completed without any error.

Is there anyway to retain audios which are greater than 20s or less than 1s by doing any modification to cuts so that it doesn't give error?


Project ideas

Hello,
I was wondering if it would be possible to add support for a third party website to load models,
i created a fork here https://github.com/alekso56/LethalCompany_ModelReplacementAPI
to load models at runtime.
with a website using S3/MINIO to download/upload avatars here: https://lethal.cax.no/
api doc: https://lethal.cax.no/swagger/index.html
auth for upload is handled using steam login.
workflow
create avatar in unity using sdk -> upload assetbundle to website -> immediatly loads ingame.
also another idea is compatability with the upcoming vr mod here: https://github.com/DaXcess/LCVR
i just wanted your opinions on this stuff.  😄

First of all I'm glad you're using assetbundles, instead of dlls for security reasons.
I don't intend on adding steam id support right now as a matter of scope, however it sounds like what you want to achieve has already been done in part by NotAustin here.
https://github.com/NotAustinVT/LethalCompany_PlayerAvatarLoader
If you would like I can get into touch with NotAustin in proposing your idea.
Regarding VR support, I'm not sure why it wouldn't be supported by base, but as someone who does not own a VR headset I wouldn't be able to test it to begin with.

thank you for responding.


Translation in Bulgarian

New translation of README.md in Bulgarian language, added in folder translations/bg-BG.

Thanks for doing this! Can you remove the link at the top that goes to contribution guidelines? The translation files don't have that (although I might reconsider later), and remove the section at the bottom linking to different translations?

Sure, thanks for the great tips!
The link that goes to Contribution guidelines is now removed from the Content (I noticed that some other translations provide this link, but I see your point for not including it, having in mind that the guidelines aren't translated)... Next, the section at the bottom that links to different translations, as well as the corresponding line in the Content have been removed as well.

Thanks for the merge! :)


Helm: Add customization of helm chart for SUSE

Codecov Report

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

@@           Coverage Diff           @@
##           master     #236   +/-   ##
=======================================
  Coverage   62.86%   62.86%
=======================================
  Files         986      986
  Lines       32096    32096
  Branches     4505     4505
=======================================
  Hits        20176    20176
  Misses      11920    11920


PowerShell scripts should never use command aliases

It is common for users to override command aliases like "cd" or "md". All PowerShell scripts which are for general distribution should use the built-in command names.
For example, kvm.ps1 uses md SomeFolder | out-null a lot, but this will conflict with anybody overriding this alias. Instead, it should be doing New-Item -type Directory SomeFolder | out-null.

@anurse our resident PowerShell expert. An alternative would be to switch to BAT files and then cd and md are "perfectly safe" :wink:

That is a little like suggesting that DOS is an alternative to Windows 10. :wink:

I agree with @bradwilson here. The only reason I haven't sent a PR to do this yet is that I haven't had time :). @Eilon, feel free to put this on my RC queue

I'll see about submitting a PR tonight for this (at least one or two .ps1 files).

:+1: - FYI, kvm actually lives at aspnet/kvm. So you should probably send the PR there.

Will do.

@jbrinkman - I'm doing some fairly major overhauling to kvm in order make it a little easier to manage (right now, for example, adding a new command requires a lot of work). I can just do this while I go through that process :).

That is fine.  I got distracted by some last minute issues that had to be resolved before our DNN release today.  It’s like some of these bugs wait until you are packaging up the final release before they become visible ☹
Thanks,
Joe Brinkman
Co-Founder, VP of Technology and Community Relations
DNN Corp. | www.dnnsoftware.comhttp://www.dnnsoftware.com/
o<PHONE_NUMBER> | m<PHONE_NUMBER>
Twitter https://twitter.com/jbrinkman | Facebook https://www.facebook.com/joe.brinkman |  LinkedIn  http://www.linkedin.com/in/joebrinkman/ | Google+https://plus.google.com/u/0/102047675986512918810/posts | DNNhttp://www.dnnsoftware.com/Activity-Feed/userId/8129
From: Andrew Nurse<EMAIL_ADDRESS>Sent: Wednesday, February 04, 2015 5:45 PM
To: aspnet/Home
Cc: Joe Brinkman
Subject: Re: [Home] PowerShell scripts should never use command aliases (#276)
@jbrinkmanhttps://github.com/jbrinkman - I'm doing some fairly major overhauling to kvm in order make it a little easier to manage (right now, for example, adding a new command requires a lot of work). I can just do this while I go through that process :).
—
Reply to this email directly or view it on GitHubhttps://github.com/aspnet/Home/issues/276#issuecomment-72956636.


Adapt the Twitter crawler to output better text files

The new Twitter crawler can download pictures, videos and text.
"But its text representation could be more effective with twitter id, date, time etc. as Twint does, otherwise it's a great tool to me."
Originally posted by @cr1zydog in https://github.com/TumblThreeApp/TumblThree/issues/161#issuecomment-886141526

That's correct the text file lacks some contextual information, especially if compared to the Tumblr crawlers' output format:
Post id:
Date:
Post url:
Slug:
Reblog key:
Reblog url:
Reblog name:
Summary:
Title:
Tags:
The Twitter crawler should be adapted to output text files similar to that ones of the Tumblr crawlers, either in json or text format depending on the chosen setting.
Do you like to take care of this issue?


Updated .offset to fix for percentage based layouts.

Proposed change for LESS only.

Thanks for contributing!


[neuralynx] exploit metadata available from neo.neuralynxrawio

Exploit metadata available from neo NeuralynxIO file headers and annotations. This PR is an update of https://github.com/catalystneuro/nwb-conversion-tools/pull/596
Contributes to https://github.com/catalystneuro/neuroconv/issues/43
Motivation
Recent versions of Neo provide access to header metadata from recording files. There is no need to re-read and interpret the headers twice.
Checklist

[x] Have you thoroughly read our Developer Guide?
[x] Have you ensured the PR description clearly describes the problem and solutions?
[x] Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?
[x] If this PR fixes an issue, is the first line of the PR description fix #XXX where XXX is the issue number?
[x] Did you update the CHANGLOG.md file on your branch to describe your changes?

@JuliaSprenger this looks great! Can you add tests please?

This PR will be simplified based on #174

The coverage seems to decrease as code lines intended for the next neo version are not covered and test files are not downloaded as they are cached.

Thanks for implementing the suggestions above @JuliaSprenger .

@JuliaSprenger
The tests that is failing is because some of the filtered columns is not in the following types: a list, np.ndarray, string or Real object (from the numbers library). Can you show here the column types of the filtered related column, I wonder what is the column that is failing the type check.

All properties were numpy arrays, but it seems an object (e.g. datetime.datetime objects) or bool dtype is not accepted by nwb. So I added some lines to convert these to a string representation.


Tracking 'test(s)' directory migration

We currently can migrate vanilla JS code to TS. We need to expand this functionality to include Ember usecases for both Classic and Embroider V2.

Let's move away from a blacklist of directories eg. test dir which we are ignoring for a migration and instead move toward the use providing scope of a migration.
eg. --entrypoint test --entrypoint src in this instance we would on traverse children of the entrypoint directory and NOT sibling directories.
This will allow the ability to migration a source directory first. Fix any raised issues. And then migrate tests directory.


remove the use of sha1

known insecure function. Impact should be minimal, but should help auditors/etc

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Testing performed?

[x] Unit tests
[ ] Integration tests
[ ] Acceptance tests

Checklist:

[x] This PR is being made against the main branch, or relevant version branch
[ ] I have made corresponding changes to the documentation
[ ] I have added testing for my changes

If you have any questions, or want to get attention for a PR or issue please reach out on the #logging-and-metrics channel in the cloudfoundry slack

concourse succeeded, but failed to unclaim environment for weird concoursy/credhub reasons.


Extra functionallity for next, previous and finish buttons.

Would be nice, if we could add some functionallity on button presses. Also custom buttons would be nice.

This function already existed. You can refer to the following usage.
IntroController(
  stepCount: 5,
  onWillPrevious: (currentStep) {
    // This callback is called when the demo flow is about to jump to the previous step.
    // You can return `false` if you want to prevent it from taking effect.
  },
  onWillNext: (currentStep) {
    // This callback is called when the demo flow is about to jump to the next step.
    // You can return `false` if you want to prevent it from taking effect.
  },
  onWillClose: (currentStep) {
    // This callback is called when the demo flow is about to close.
    // You can return `false` if you want to prevent it from taking effect.
  },
);

Or
IntroStepTarget(
  onTargetLoad: () {
    // It will be called when the target widget was built.
  },
  onTargetDispose: () {
    // It will be called when the target widget was disposed.
  },
  onHighlightTap: () {
    // It will be called when tap the highlighted widget.
  },
  onStepWillActivate: (int fromStep) {
    // It will be called when the demo flow reaches the current step.
    // The current step is finally activated only when this callback execution is complete.
    // The `fromStep` tells you from which step it jumped to the current step.
    // In particular, the value of `fromStep` is '0' means that this is the beginning.
  },
  onStepWillDeactivate: (int willToStep) {
    // It will be called when the demo flow leaves the current step.
    // The current step is finally deactivated only when this callback execution is complete.
    // The `willToStep` tells you which step it will to jump to.
    // In particular, the value of `willToStep` is '0' means that this is the ending.
  },
  // ...
);


Update continuous-integration.md: Include CircleCI reference

What does this PR try to resolve?
Adds simple example to run cargo test on CircleCI, alongside the other popular CI engines.
How should we test and review this PR?
Change is purely documentation. I would ask that the PR be checked for tone and formatting that is in line with Rust/Cargo community standards.

Unsure what the bar should be for inclusion but if we have sr.ht then it seems fine to include CircleCI.  At a later point, I could see us creating more guidance on what gets included which might shake things up.
@bors r+

:hourglass: Testing commit e62ef14fd8906966bfe17ec42f6e15da4bc9bddf with merge f89264764a80d41608dcbe914ba9ed80c07181af...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing f89264764a80d41608dcbe914ba9ed80c07181af to master...


Translate Filter and Eligibility buttons

Issue

Closes # (issue)
Addresses # (issue)
[ ] This change is a dependency for another issue
[ ] This change has a dependency from another issue

Description
Adds translations for the Filter and Eligibility buttons on the listings and home page.

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Prototype/POC (not to merge)
[ ] This change is a refactor/address technical debt
[ ] This change requires a documentation update
[ ] This change requires a SQL Script

How Can This Be Tested/Reviewed?
Change the language on the listings page and check the buttons adjust accordingly.

[x] Desktop View
[x] Mobile View
[ ] Test A
[ ] Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have reviewed the changes in a desktop view
[ ] I have reviewed the changes in a mobile view
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have assigned reviewers
[ ] I have updated the changelog to include a description of my changes

Covered by #401


Mongodb support

Describe the feature
Is there a plan to support mongodb as well?
Additional information

[x] Would you be willing to help implement this feature?

I am not a maintainer but this is a SQL connector, I doubt it will ever support NoSQL.


Multiple RegisterStructValidation on a sing struct

Package version eg. v8, v9: v9
Issue, Question or Enhancement: Question
I have registered 2 struct validations for the same struct, but only the 2nd one runs. Is there a way to have multiple custom validations for the same struct?
Code sample, to showcase or reproduce:
        validate = validator.New()
	validate.RegisterStructValidation(stores.EmailRecipient.Validate, request.To)
	validate.RegisterStructValidation(stores.Email.Validate, request.Email)

Hey @alexdor
No currently there is no way to have multiple struct validations, internally only a reference to the struct's type is held, and why when registering 2 only the second is used as it overwrites/remaps the first. The reason that type is used is it's the only truly unique identifier between structs.
There are however there are some options, you could:

Use RegisterStructValidationCtx and pass in a value in the Context to indicate which validation to run inside a separate struct validation.
Separate your structs, To and Email; in my experience have one structure represent multiple things will always eventually get you into trouble (only mentioning because you want to run different validations depending on struct usage)

Hey @joeybloggs thanks for the response. Would you accept a pr that adds this feature?

@alexdor I would definitely consider a PR, it all depends on how it would work, performance considerations and if there were any breaking changes.

Also ran into this, but I'd like to add multiple different custom validations as separate functions to keep them short and sweet. Now I need to combine them into a single messy big one.

I have a generic struct ctx validator that works on many of my structs, and now a specific validator that works in addition to the generic one. It would be nice to be able to do
validate = validator.New()
validate.RegisterStructValidationCtx(GenericValidator, EntityOne, EntityTwo, EntityThree)
validate.RegisterStructValidationCtx(SpecificEntityTwoValidator, EntityTwo)

instead in the GenericValidator, I have to do
validate.RegisterStructValidationCtx(GenericValidator, EntityOne, EntityTwo, EntityThree)

func GenericValidator(ctx context.Context, sl validator.StructLevel) {
        ...
	entity := sl.Current().Interface()
        v := reflect.ValueOf(entity)
	t := v.Type()

      // Call the additional validator for EntityTwo
       if t.Name() == 'EntityTwo' {
           SpecificEntityTwoValidator(ctx, sl)
       }


Ignore bridge methods

Let's try to calc LCOM2 on Reports.java:
<class color="yellow" element="true" id="Reports" value="0.5">
    <vars>
       <var id="methods">2</var>
       <var id="attributes">2</var>
   </vars>
</class>

But in class Reports only one method: apply.

There is bridge method.
So, let's ignore bridge methods.

@yegor256/z please, pay attention to this issue

Job #171 is now in scope, role is DEV

Bug was reported, see §29: +15 points just awarded to @nikialeksey/z, total is +90

These are so called synthetic methods as per the JLS spec:

A construct emitted by a Java compiler must be marked as synthetic if it does not correspond to a construct declared explicitly or implicitly in source code, unless the emitted construct is a class initialization method

They should be excluded from the skeleton

The job #171 assigned to @uurl/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@uurl/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat wait #260

@0crat wait #260 (here)

@uurl The impediment for #171 was registered successfully by @uurl/z

@yegor256 please help, the PR #260 closes this ticket, PR #260 is already accepted and merged but I don't know how to indicate that it closes this issue. While the issue is open, I don't receive any payment.

@uurl I don't think I understand. What exactly do you want me to do?

@yegor256  didn't received payment for this job yet; the PR #260 closes this ticket. Could you indicate @0crat that it's closed?

Are you speaking to me or about me here; you must always start your message with my name if you want to address it to me, see §1

@uurl why do you expect to receive the payment, if the job is not closed yet? Please, read this: https://www.yegor256.com/2014/11/24/principles-of-bug-tracking.html

The architect of the project has changed; @yegor256/z is not at this role anymore; @paulodamaso/z is the architect now

@nikialeksey Looks like this one already been merged, please take a look

O, yeah, cool!

The project doesn't have enough funds, can't make a payment

The job #171 is now out of scope

Order was finished: +30 point(s) just awarded to @uurl/z

The project doesn't have enough funds, can't make a payment


[FancyZones] Maximise to zone

Give an option to maximise a window to the zone it is currently in (see DisplayFusion's handling of their version of zones). So rather than holding shift and dragging a window to fill a zone, maximising the window via the title bar button or a double clicking on the title bar would fill the zone.
A hotkey to press to maximise as normal while disregarding zones would go well with this feature.

duplicate of #279


[FEAT]: Allow code file types

What would you like to see?
I think the LLMs are capable of doing a lot of code analysis, right? Several of them are advertised as "code interpreters".
I must be missing something significant because I am wondering why code files are not allowed in the file upload.
Why aren't code file types allowed?

For additional background, I can easily convert a code file to a .txt file and then ask Ollama in AnywhereLLM questions about the code file and it answers as if it understands the code file and what I'm trying to do from the command line.
NOTE: I added this line to the top of the .txt file:
// this is the text version of a `build.gradle` file.

is it possible to accept yaml and json files ?

I would like an setting where we can add arbitrary file extensions to be treated as text. (e.g., .c, .py, .swift, .cpp, etc.)
Maybe even a setting where we can tell the app to try to open any unrecognized file as text.


fixes #2001: spoof a REPL readline interface to handle piped REPL input

My fix for #2001. Also see #2002, @TrevorBurnham's fix.

Potential reviewers: the real addition is lines 107-124, the rest is just moving things around.

Merging this.


Bug: CDK Example Not Functional

Expected Behaviour
CDK Example found at examples/cdk/README.md functions as described
Current Behaviour
Deployment succeeds, however invocation of the API Gateway URL produces an Internal Error.
Looking at the Lambda functions logs it produces:
{
    "errorType": "Runtime.ImportModuleError",
    "errorMessage": "Error: Cannot find module '@aws-lambda-powertools/parameters/ssm'\nRequire stack:\n- /var/task/index.js\n- /var/runtime/index.mjs",
    "stack": [
        "Runtime.ImportModuleError: Error: Cannot find module '@aws-lambda-powertools/parameters/ssm'",
        "Require stack:",
        "- /var/task/index.js",
        "- /var/runtime/index.mjs",
        "    at _loadUserApp (file:///var/runtime/index.mjs:1087:17)",
        "    at async UserFunction.js.module.exports.load (file:///var/runtime/index.mjs:1119:21)",
        "    at async start (file:///var/runtime/index.mjs:1282:23)",
        "    at async file:///var/runtime/index.mjs:1288:1"
    ]
}

Code snippet
Follow instructions: examples/cdk/README.md
Steps to Reproduce
Follow instructions: examples/cdk/README.md
Possible Solution
Looking at examples/cdk/src/example-stack.ts - Runtime of NodeJS 20 is specified as a default:
const commonProps: Partial<NodejsFunctionProps> = {
  runtime: Runtime.NODEJS_20_X,
  tracing: Tracing.ACTIVE,
...

However version :18 of the Lambda Layer is specified:
    commonProps.layers?.push(
      LayerVersion.fromLayerVersionArn(
        this,
        'powertools-layer',
        `arn:aws:lambda:${
          Stack.of(this).region
        }:094274105915:layer:AWSLambdaPowertoolsTypeScript:18`
      )
    );

Viewing the Layer configuration in the Lambda console shows that NodeJS 20 is not supported.  Support Runtimes for that layer version are:

Powertools for AWS Lambda (TypeScript) version
latest
AWS Lambda function runtime
20.x
Packaging format used
Lambda Layers
Execution logs
No response

Hey @apmclean , thanks for opening the issue and the detailed description, greatly appreciated!
It seems we forgot to update the layer version in our CDK example. Version 18 does not include the ssm parameter package. I will open a PR to fix it quickly. In the meantime, if you set the ARN of layer to version 24, this will resolve the issue.

I would like to take this issue


☂ eslint-plugin-react-hooks

[!WARNING]
This comment is maintained by CI. Do not edit this comment directly.
To update comment template, see https://github.com/oxc-project/oxc/tree/main/tasks/lint_rules

This is tracking issue for eslint-plugin-react-hooks.
There are 2(+ 0 deprecated) rules.

2/2 recommended rules are remaining as TODO
0/0 not recommended rules are remaining as TODO

All done! 🎉

To get started, run the following command:
just new-react-hooks-rule <RULE_NAME>

Then register the rule in crates/oxc_linter/src/rules.rs and also declare_all_lint_rules at the bottom.
Recommended rules

  ✨: 0, 🚫: 0 / total: 2

Status
Name
Docs

react-hooks/rules-of-hooks
https://reactjs.org/docs/hooks-rules.html

react-hooks/exhaustive-deps
https://github.com/facebook/react/issues/14920

✨ = Implemented, 🚫 = Not supported

@Boshen I wanted to implement these, I've noticed that the command just new-react-hooks-rule <RULE_NAME> doesn't work; I can create the rule manually I just need to know what should I change in order to keep this comment in sync in CI.

I just noticed that #2637 is a thing, I'm going to only work on the hook rules.

I just noticed that #2637 is a thing, I'm going to only work on the hook rules.

https://github.com/oxc-project/oxc/pull/2637 is stale, maybe you can help out and merge the code first, then improve upon it.

@Boshen I wanted to implement these, I've noticed that the command just new-react-hooks-rule <RULE_NAME> doesn't work; I can create the rule manually I just need to know what should I change in order to keep this comment in sync in CI.

There are only 2 rules, and the tests cannot be parsed automatically :-/

@Boshen I wanted to implement these, I've noticed that the command just new-react-hooks-rule <RULE_NAME> doesn't work; I can create the rule manually I just need to know what should I change in order to keep this comment in sync in CI.

There are only 2 rules, and the tests cannot be parsed automatically :-/

Should I put them under react or create a new category? It won't be such a bad thing to unify such categories and not base them strictly on the origin plugin.

@Boshen I wanted to implement these, I've noticed that the command just new-react-hooks-rule <RULE_NAME> doesn't work; I can create the rule manually I just need to know what should I change in order to keep this comment in sync in CI.

There are only 2 rules, and the tests cannot be parsed automatically :-/

Should I put them under react or create a new category? It won't be such a bad thing to unify such categories and not base them strictly on the origin plugin.

Yeah, under the react plugin should be fine.

✨: 0, 🚫: 0 / total: 2
Status 	Name 	Docs
react-hooks/rules-of-hooks 	https://reactjs.org/docs/hooks-rules.html
react-hooks/exhaustive-deps 	facebook/react#14920

This didn't update, Have I missed something with the tasks?

That's because react-hooks/* rules are defined under react:: mod, it should be react_hooks:: to be tracked by lint_rules task.
https://github.com/oxc-project/oxc/blob/7ed673d2a5d53e5d9c69a60d21c4cd8ccaa0e817/crates/oxc_linter/src/rules.rs#L643
If this is intended, we need to update lint_rules task to mange these conversion.

That's because react-hooks/* rules are defined under react:: mod, it should be react_hooks:: to be tracked by lint_rules task.
https://github.com/oxc-project/oxc/blob/7ed673d2a5d53e5d9c69a60d21c4cd8ccaa0e817/crates/oxc_linter/src/rules.rs#L643
If this is intended, we need to update lint_rules task to mange these conversion.

Thanks, Yes it was intentional there are only 2 rules for hooks and I've spoken with @Boshen about merging them into react; And he gave the green light for it.

I'm in favor of modifying the task script.

Can you help me with the change?
Should I do it in such a way that this issue gets closed in favor of adding these entries to #1022, Or keep this issue and just resolve the implementation from the react module?

FYI: If

keep this issue and just resolve the implementation from the react module?

diff --git a/tasks/lint_rules/src/oxlint-rules.cjs b/tasks/lint_rules/src/oxlint-rules.cjs
index 65302b62..e8ee28c5 100644
--- a/tasks/lint_rules/src/oxlint-rules.cjs
+++ b/tasks/lint_rules/src/oxlint-rules.cjs
@@ -90,7 +90,12 @@ exports.createRuleEntries = (loadedAllRules) => {
 exports.updateImplementedStatus = async (ruleEntries) => {
   const implementedRuleNames = await readAllImplementedRuleNames();

-  for (const name of implementedRuleNames) {
+  for (let name of implementedRuleNames) {
+    // These rules are implemented under `react::` mod, but issue is separated
+    if (name === "react/rules-of-hooks" || name === "react/exhaustive-deps") {
+      name = name.replace("react/", "react-hooks/");
+    }
+
     const rule = ruleEntries.get(name);
     if (rule) rule.isImplemented = true;
     else console.log(`👀 ${name} is implemented but not found in their rules`);

If not, there is a little more work to do. 😅

Well, I haven't tested it, You seem to have a much clearer knowledge of linters. Do you think it is possible to resolve this by changing the prefix here?
https://github.com/oxc-project/oxc/blob/34dd53cc9842161b20d78682596e5367a7119cfa/tasks/lint_rules/src/eslint-rules.cjs#L186-L194

If not, there is a little more work to do. 😅

You seem to have a much clearer idea of this, Do you think it is possible to add react-hook plugins as react here so we can add it to the react issue(in case even want to)?
https://github.com/oxc-project/oxc/blob/34dd53cc9842161b20d78682596e5367a7119cfa/tasks/lint_rules/src/oxlint-rules.cjs#L64-L87

(Actually, I am the original author of this task. 🤫 )
I was thinking that there are a few possible ways to handle this.
If it's okay to address it (maybe) this weekend, I can take care of it.

(Actually, I am the original author of this task. 🤫 )

Makes sense😅
Let me know if there is anything that I can help with, especially if you didn't have the time to do it.


OpenJDK Runtime Environment - Geneious Prime - libjvm.so+0x422d8c MinINode::Idea;(PhaseGVN*, bool)+0xbc

Java was integrated in Genenious Prime - OpenJDK 11.0.9+11 mixed mode
Summary
A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007f46bddc9d8c, pid=3163, tid=3174
#
# JRE version: OpenJDK Runtime Environment AdoptOpenJDK (11.0.9+11) (build 11.0.9+11)
# Java VM: OpenJDK 64-Bit Server VM AdoptOpenJDK (11.0.9+11, mixed mode, tiered, concurrent mark sweep gc, linux-amd64)
# Problematic frame:
# V  [libjvm.so+0x422d8c]  MinINode::Ideal(PhaseGVN*, bool)+0xbc
#
# Core dump will be written. Default location: Core dumps may be processed with "/usr/share/apport/apport %p %s %c %d %P %E" (or dumping to /opt/Geneious_Prime/core.3163)
#
# If you would like to submit a bug report, please visit:
#   https://github.com/AdoptOpenJDK/openjdk-support/issues
#

---------------  S U M M A R Y ------------

Command Line: -Xss512K -DsuppressSwingDropSupport=true -Djava.util.logging.config.file=logging.properties -Djna.boot.library.path=/opt/Geneious_Prime/resources\jna_windows\win32;/opt/Geneious_Prime/resources\jna_windows\win64 -Xmx61366M -Xms256M -XX:+UseConcMarkSweepGC -XX:MaxNewSize=40M -XX:MaxJavaStackTraceDepth=99999 -Djava.net.preferIPv4Stack=true -XX:ReservedCodeCacheSize=128m -Dfile.encoding=UTF8 -Dsun.awt.nopixfmt=true --add-opens=java.base/jdk.internal.reflect=ALL-UNNAMED -DlaunchISeekMain:3030 com.biomatters.iseek.application.ISeekMain -distributionVersion -linuxNamedPipe="/tmp/3fefa92d-934c-4db7-97e9-91e579afa157"

Host: Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, 32 cores, 62G, Ubuntu 18.04.5 LTS
Time: Fri Feb  5 18:20:24 2021 +07 elapsed time: 7070.341255 seconds (0d 1h 57m 50s)

When mapping Illumina sequences into Reference Genomes, the Geneious Prime sw will crash and send above message.
Steps to reproduce
Mapping Illumina sequences (fastq files) to reference genes. (Genenious Prime SW)

Expected results
Actual results
Triaging info
Java version:
# JRE version: OpenJDK Runtime Environment AdoptOpenJDK (11.0.9+11) (build 11.0.9+11)
# Java VM: OpenJDK 64-Bit Server VM AdoptOpenJDK (11.0.9+11, mixed mode, tiered, concurrent mark sweep gc, linux-amd64)
vm_info: OpenJDK 64-Bit Server VM (11.0.9+11) for linux-amd64 JRE (11.0.9+11), built on Oct 24 2020 15:17:19 by "" with gcc 7.5.0

# JRE version: OpenJDK Runtime Environment AdoptOpenJDK (11.0.9+11) (build 11.0.9+11)
# Java VM: OpenJDK 64-Bit Server VM AdoptOpenJDK (11.0.9+11, mixed mode, tiered, concurrent mark sweep gc, linux-amd64)

What is your operating system and platform?
Ubuntu 18.04 on x86-64

How did you install Java?
Was integrated in Genenious Prime

Did it work before?
The sw works ok in Centos 07 system x86-64

Did you test with other Java versions?

Please update at least to <IP_ADDRESS> where the crash has been fixed.
https://bugs.openjdk.java.net/browse/JDK-8250861


Update swift/protobuf.patch

Left as documentation for the patch which applied to master to produce an xcframework. This work can be cherry-picked to perform a new release

This is not necessary anymore.


Dynamic link sometimes redirects to appstore

Step 1: Describe your environment

Xcode version: 12.4
Firebase SDK version: 7.9.0
Installation method: CocoaPods
Firebase Component: Dynamic Links

Step 2: Describe the problem
Hi!
We're using a lot of firebase dynamic links in our iOS application, and almost all time it works perfectly.
But, sometimes opening application from link stops working: link opening redirects users to Appstore page.
It doesn't depend on:

device or iOS version
way of link opening (manually created link from firebase admin panel, from mail app, pasting directly to safari, opening from notes etc)
app usage time (it can start wrong redirecting after week of daily app use, for example)
firebase version (we are fighting with this bug for about 6 month, and every app release we have a fresh sdk version).
app launch type: debug from xcode, adhoc with testflight, or release from Appstore

And most important - it doesn't depend on the link. Like if you hook this bug while opening some link, everyone else using the same link will not have any problems.
So, it seems to us that the problem is inside device or firebase sdk.
The bright side is that this bug goes away by itself, after some amount of time, or rebooting/reinstalling.
How we are dealing with this problem:
prompting customers to:

past link directly to safari (doesn't help most time)
reboot device (helps most time)
reinstall app (helps most time)
use "trick" to past link to notes, long press to context menu, select "open in app" (helps most time)

If it matters, we have the same functionality in android application, and there wasn't any problems like this.
Steps to reproduce:
Unfortunately we can't provide any info about how to see this bug, because we can't reproduce it on our side. It just randomly stops working properly.
What our configs are look like:
https://ourappdomain/apple-app-site-association:
{"applinks":{"apps":[],"details":[{"appID":"ourTeam.ourApp","paths":["NOT /_/*","/*"]}]}}
OurApp.entitlements:
<key>com.apple.developer.associated-domains</key>
<array>
   <string>applinks:ourapp.page.link</string>
</array>

Info.plist:
<key>LSApplicationQueriesSchemes</key>
<array>
    <string>googlegmail</string>
    <string>ourapp.page.link</string>
</array>
<key>FirebaseDynamicLinksCustomDomains</key>
    <array>
      <string>https://ourapp.page.link</string>
    </array>

AppDelegate:
    func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]? = nil) -> Bool {
        FirebaseApp.configure()
...

    func application(
        _ application: UIApplication,
        continue userActivity: NSUserActivity,
        restorationHandler: @escaping ([UIUserActivityRestoring]?) -> Void) -> Bool {
        let handled = DynamicLinks.dynamicLinks().handleUniversalLink(userActivity.webpageURL!) { (dynamiclink, error) in
            if let url = dynamiclink?.url?.absoluteString {
                DeepLinkHandler.handleLink(url)
            }
        }

        return handled
    }

  func application(_ app: UIApplication, open url: URL, options: [UIApplication.OpenURLOptionsKey: Any] = [:]) -> Bool {
        return true
  }

Thanks in advance for any help!

Hi @bobbyzamora. I see that the link is redirecting to App Store even if the app is installed, however, you’re unable to reproduce it on your end. For us to verify the root cause of the issue or to isolate it, can you provide the following details:

Affected device (OS version/device model)
App wherein the Dynamic Link has been opened
Dynamic Link used

Is it only happening to 1 Dynamic Link?

Pattern/s you’ve noticed on the issue (ex. only happening to iOS 13)
Confirmation if there are no warnings when appending ?d=1 to your affected link (Debugging Dynamic Links)

Thanks.

Hi @rizafran! Thanks for the reply!
Here is some info:
1. Affected device (OS version/device model):
iPhone XS, iOS 14.4
iPhone 12 pro max, iOS 14.4
iPhone 6s, iOS 14.3
iPhone 6s, iOS 14.4
iPhone 12 Pro, iOS 14.4.2 & iOS 14.4.1
iPhone 8, iOS 14.4.2
(this is not the full list, just some examples from our customers reports)
2. App wherein the Dynamic Link has been opened
Didn't quite get what kind of info do you need here:)
We have an iOS app and Appstore page. Usually dynamic link redirects to app, but sometimes to Appstore.
3. Dynamic Link used. Is it only happening to 1 Dynamic Link?
Nope, it doesn't depend on the link. If you encounter this bug, all links (generated by api, or created manually through the firebase admin panel - doesn't matter) will stop opening correctly. And same links will work properly on any other device.
4. Pattern/s you’ve noticed on the issue (ex. only happening to iOS 13)
We have plenty a lot of customers complaints about links not opening in the app, but unfortunately didn't spot any patterns. So it really looks like a "random temporary bug".
5. Confirmation if there are no warnings when appending ?d=1 to your affected link (Debugging Dynamic Links).
We haven't done this kind of debug yet. Thanks for this link. We gonna try it when find this bug again and i write what kind of logs it has.
Thanks again rizafran, hope this info will help somehow.

Hi @rizafran! I'm returning with some info. I've just catch this error, and here is what output i have.
Launched app with Xcode12.4 on iPhone 11, iOS 14.4.2. Got the link that redirects to AppStore, then added:
DynamicLinks.performDiagnostics(completion: nil)
and
suffix?d=1 to the link

Xcode console output:
[Firebase/Crashlytics] Version 7.9.0
7.9.0 - [Firebase/Analytics][I-ACS023007] Analytics v.7.9.0 started
7.9.0 - [Firebase/Analytics][I-ACS023008] To enable debug logging set the following application argument: -FIRAnalyticsDebugEnabled (see http://goo.gl/RfcP7r)

WF: _userSettingsForUser mobile: {
    filterBlacklist =     (
    );
    filterWhitelist =     (
    );
    restrictWeb = 1;
    useContentFilter = 0;
    useContentFilterOverrides = 0;
    whitelistEnabled = 0;
}
WF: _WebFilterIsActive returning: NO
7.9.0 - [Firebase/Messaging][I-FCM001000] FIRMessaging Remote Notifications proxy enabled, will swizzle remote notification receiver handlers. If you'd prefer to manually integrate Firebase Messaging, add "FirebaseAppDelegateProxyEnabled" to your Info.plist, and set it to NO. Follow the instructions at:
https://firebase.google.com/docs/cloud-messaging/ios/client#method_swizzling_in_firebase_messaging
to ensure proper integration.

---- Firebase Dynamic Links diagnostic output start ----
Firebase Dynamic Links framework version 7.9.0
System information: OS iOS, OS version 14.4.2, model iPhone
Current date 2021-04-22 15:48:22 +0000
Device locale en-RU (raw en_RU), timezone Europe/Moscow
	Specified custom URL scheme is thestoicapp and Info.plist contains such scheme in CFBundleURLTypes key.
	AppID Prefix: D4P55P6FV6, Team ID: D4P55P6FV6, AppId Prefix equal to Team ID: YES
performDiagnostic completed successfully! No errors found.
---- Firebase Dynamic Links diagnostic output end ----

<Warning>: Please set a value for FacebookAutoLogAppEventsEnabled. Set the flag to TRUE if you want to collect app install, app launch and in-app purchase events automatically. To request user consent before collecting data, set the flag value to FALSE, then change to TRUE once user consent is received. Learn more: https://developers.facebook.com/docs/app-events/getting-started-app-events-ios#disable-auto-events.

<Warning>: You haven't set a value for FacebookAdvertiserIDCollectionEnabled. Set the flag to TRUE if you want to collect Advertiser ID for better advertising and analytics results.
FBSDKLog: Cannot load gate keepers before configuring.

7.9.0 - [Firebase/Analytics][I-ACS800023] No pending snapshot to activate. SDK name: app_measurement
AppsFlyer SDK version 6.2.4 started build (71);
Keychain items migration status: 0
7.9.0 - [Firebase/Analytics][I-ACS023012] Analytics collection enabled
7.9.0 - [Firebase/Analytics][I-ACS023220] Analytics screen reporting is enabled. Call +[FIRAnalytics logEventWithName:FIREventScreenView parameters:] to log a screen view event. To disable automatic screen reporting, set the flag FirebaseAutomaticScreenReportingEnabled to NO (boolean) in the Info.plist

 [assertion] Error acquiring assertion: <Error Domain=RBSAssertionErrorDomain Code=2 "Specified target process does not exist" UserInfo={NSLocalizedFailureReason=Specified target process does not exist}>
[ProcessSuspension] 0x116507730 - ProcessAssertion: Failed to acquire RBS Background assertion 'WebProcess Background Assertion' for process with PID 19789, error: Error Domain=RBSAssertionErrorDomain Code=2 "Specified target process does not exist" UserInfo={NSLocalizedFailureReason=Specified target process does not exist}

[Client] {"msg":"#NullIsland Either the latitude or longitude was exactly 0! That's highly unlikely", "latIsZero":0, "lonIsZero":0}
[Client] {"msg":"#NullIsland Received a latitude or longitude from getLocationForBundleID that was exactly zero", "latIsZero":0, "lonIsZero":0, "location":{"floor":2147483647,"lifespan":-1,"rawLat":0,"integrity":0,"referenceFrame":"Unknown","lon":0,"speed":-1,"type":"Unknown","altitude":0,"rawCourse":-1,"confidence":0,"suitability":{"type":"decode failure","raw value":451406408,"expected type":"CLClientLocationSuitability"},"timestamp":-1,"rawReferenceFrame":"Unknown","lat":4.9406564584124654e-324,"verticalAccuracy":-1,"rawLon":0,"horizontalAccuracy":-1,"speedAccuracy":-1,"courseAccuracy":-1,"course":-1}}

Then i've tried opening the link: "https://stoicapp.page.link/hkqdPBE3sHBxjins8" that starts redirecting me to the AppStore.
Then i've added debug suffix to the link, and here is what i got:

After this, i've made copy-paste link to the notes app. Press the link from the notes, and it redirected successfully to the app.

To sum up, nothing new with links behaviour from my previous messages (sometimes all links starts redirecting to the AppStore, after opening from notes app - everything becomes ok). I see some errors from console, but dynamic links logs looks fine.
Thanks in advance!

Thanks for sharing your findings, @bobbyzamora. I've tried debugging your dynamic link by adding a ?d=1 in the end, and I've noticed that there are 3 warnings. Could you double-check if the UniversalLinks is enabled and if you're using the correct bundle ID?

@rizafran Thanks for the reply!
Universal link is enabled, and bundle id is correct. Otherwise it would not open later from notes app.
Note that after opening from "notes" the link can be opened from safari too.
Thanks!

Hi @bobbyzamora, can you file a ticket to Firebase support? This way, you can share your project info to the support team so that they might help you to troubleshoot your dynamic link further. Thanks.


To do list

in this project, I did the following:

Use webpack to bundle JavaScript.
Use ES6 modules to write modular JavaScript.
Create an index.js file and set an array of simple to-do tasks (array of objects).
Write a function to iterate over the tasks array and populate an HTML list item element for each task.
Render the dynamically created list of tasks in the dedicated placeholder.
Create a style.css and set rules for the To-Do List.

Status: Invalid 🔴
Hi @tumainimaganiko 🙋♀,
While you made a great effort in this project, I can't review your code.
Invalid Code Review Request
You have submitted a project that has a README that is not professional. Here are key points to consider when documenting your README.

Future Features section should include 1-3 potential features to be added.

Kindly delete the live demo section if the application is not deployed. If project requirements say to deploy the project, the section should be restored and the link updated.

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers 🥂  and Happy coding!!! 👯
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Remember to tag me @KDlamini  in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Fix data races when text fields are debounced

Fix bugs in react-form revealed when TextField and SubmitButton are denounced.
The bugs appear because React handles setState synchronously when called from promise.then(/*...*/) callbacks and asynchronously when called from browser event handlers.  Because of that, debounced onClick events are handled incorrectly. react-form is a deprecated package using setState and  useRef for state management, which is :exploding_head:.
This PR is fixing these issues by making react-form forgiving to "incorrect" states

I think there were some plans to switch from the deprecated react-form...

I think there were some plans to switch from the deprecated react-form...

I don't think that's realistic, BasicForm is too complicated


create attribute class for sympy

It might be useful to add new attributes to the package that needs them,
rather than obj_model. For example, in addition to SymPy expressions, I
would also like to have an attribute corresponding to BioPython Sequences,
and there may be other attributes that we might envision.
On Mon, Apr 17, 2017 at 8:12 PM, Jonathan Karr<EMAIL_ADDRESS>wrote:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/KarrLab/obj_model/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABiTwaDVxFQrFNMepFYT2D_Y6TTmI8niks5rw__ugaJpZM4M_t3A
.

--
John Arul Prakash Sekar
Ph.D. Computational & Systems Biology

John, are your new attributes new "attribute types" that belong in obj_model, or new wc model attributes that belong in wc_lang? Other than development branches, I think we want only 1 version of obj_model.
But wc_lang is different; it describes the semantics and, to some extent, the approach of the model, so variation makes sense. E.g., I need to extend it to associate a volume with each submodel.
However, we would like to avoid too many versions of wc_lang. So I think the approach should be for us to branch wc_lang as needed, and intermittently merge branches while confirming that the merged branch does not disrupt downstream code. your new attributes could go into a branch of wc_lang.

I added a new module called extra_attributes for special attributes. This includes

SymPy expression, symbol attributes (sympy.Basic, sympy.Expr, sympy.Symbol)
BioPython sequence attributes (Bio.Seq.Seq)

Per John's request, I added a new extra_attributes module to obj_model
for domain-specific attributes:

SymPy expression, symbol attributes (sympy.Basic, sympy.Expr,
sympy.Symbol)
BioPython sequence attributes (Bio.Seq.Seq)

On Tue, Apr 25, 2017 at 8:31 AM, artgoldberg<EMAIL_ADDRESS>wrote:

John, are your new attributes new "attribute types" that belong in
obj_model, or new wc model attributes that belong in wc_lang? Other than
development branches, I think we want only 1 version of obj_model.
But wc_lang is different; it describes the semantics and, to some extent,
the approach of the model, so variation makes sense. E.g., I need to extend
it to associate a volume with each submodel.
However, we would like to avoid too many versions of wc_lang. So I think
the approach should be for us to branch wc_lang as needed, and
intermittently merge branches while confirming that the merged branch does
not disrupt downstream code. your new attributes could go into a branch of
wc_lang.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/KarrLab/obj_model/issues/3#issuecomment-297015089,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACt2KeiyE_R_RZnnRiHc6cxaax1itwXTks5rzeewgaJpZM4M_t3A
.


Cannot use infix operators without spaces

The following Val code:
  var a = 5
  var b = 7
  let _ = a+b

fails to compile with the following errors:
TestCases/CXX/Expr/InfixOperators.val:12: error: -[ValTests.CXXTests testTranspiler] : unexpected diagnostic: 'unused result of type 'Int''
TestCases/CXX/Expr/InfixOperators.val:12: error: -[ValTests.CXXTests testTranspiler] : unexpected diagnostic: 'type 'Int' has no member 'postfix+''

Adding spaces around the + operator seems to make the compiler work.

Errors are wrong, but infix operators always require space on both sides.

Ok, makes sense.
To me, this is a limitation that users will complain about. There are a lot of users that don't like to put spaces around infix operators (or, at least to some of the operators). E.g., people would want to write a + b*c.

The problem is that allowing infix operators without spaces makes this kind of expressions ambiguous:  a +a. I understand the worry but Swift shows it can be pulled off, as it has almost the same limitation.

I don’t see how a +a could ever be ambiguous, since we don’t allow expressions next to one another with no intervening sigil.
Regardless, we could (and probably should) just do what Swift does and accept infix operators as long as the existence of space on either side is consistent, eg. a + b or a+b but not a+ b.  To me, anything else seems like we’re being needlessly “innovative."

On Jan 28, 2023, at 6:49 AM, Dimi Racordon @.***> wrote:
The problem is that allowing infix operators without spaces makes this kind of expressions ambiguous: a +a. I understand the worry but Swift shows it can be pulled off, as it has almost the same limitation.
—
Reply to this email directly, view it on GitHub https://github.com/val-lang/val/issues/278#issuecomment-1407413869, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAKYIMYMZ4J5MIPC2FYGQLWUUWW3ANCNFSM6AAAAAAUIAMGME.
You are receiving this because you are subscribed to this thread.

I personally find it "natural" to have Scala-like operators. I've used an adapted version of these in Sparrow, and I really liked the results. See https://www.researchgate.net/profile/Lucian-Teodorescu/publication/276154942_FLEXIBLE_OPERATORS_IN_SPARROW/links/57134f2e08ae39beb87a8a07/FLEXIBLE-OPERATORS-IN-SPARROW.pdf
(in there the flexibility was probably too much, but you don't need to get that far off).
The parsing rules are simple. An expression like: o1 O1 o2 O2 o3 O3 o4 O4 will be parsed as following:

be default, all terms on odd positions are operands (lowercase o); the even-positions are operators (upper case O).
if at an even position one can find an string of symbols, that would be treated as a prefix operator, and the whole array of terms will be shifted/reconsidered (odds switch places with evens).
if the sequence ends with an operator, that would be treated as a postfix operator

Note that regular identifiers can also be operators.
For example a problem like:
What is the root mean square of the lengths of all the Collatz sequences up the first one that has a length greater than or equal to 500?
can be solved in one line as:
var res = (1..) map \collatzSeq map \rangeSize takeWhile (fun s = s < 500) rootMeanSquare;

Ignoring the \ (function reference) and lambda syntax, the code reads naturally from left to right. The code shows only terms from the problem description, and it's actually smaller than the English text.

I don’t see how a +a could ever be ambiguous, since we don’t allow expressions next to one another with no intervening sigil.

That rule is in general not so easy to enforce and requires many tricks on the grammar rules.
But let's not get drawn into the details. I don't want to be innovative, but I'm very opinionated about operators. I also don't see a reason to add complexity to our grammar only to handle marginal uses of operators without spaces, which arguably make it harder to tell if an operator is infix or not. I'm also willing to defend my position against disappointed users.
I see much, much more a + a than a+a in Swift or other languages so I don't think the rule goes against common wisdom. We can also imagine that a future version of our parse can emit fix-its.

I personally find it "natural" to have Scala-like operators.

We discussed that idea and eventually decided to say no. Besides, there are already many subtle rules that need to be upheld to make operators work because they can appear in so many contexts, due to the whole "types are expressions" thing. I don't think it's worth over-engineering them any further, at least until we reach concrete limitations when we'll write code.


fix(helm): do not use local chart if --repo is provided

What this PR does / why we need it:
If --repo is passed to helm install or helm upgrade, Helm will currently prefer a local directory with the name of the chart instead of the chart from the provided repository. This PR changes this behaviour such that the local filesystem is not checked when --repo is provided.
Fixes #11141
Special notes for your reviewer:
I considered keeping the current behaviour if the name is an absolute or relative path (i.e. it starts with / or .), but it doesn't seem correct to use a local chart if --repo is passed.
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[x] this PR has been tested for backwards compatibility

Added a test verifying that a remote chart is used when --repo is passed. I tried adding a golden output file, but there's a dyamic port number in the error message making it not really feasible.

This changes the default behavior. Changes of behavior are precluded without opting in by hip-0004.

Maybe use a different flag? --remote-repo for example

Introduced a new --force-remote flag :+1:

Rather than repeat this all, please consider this comment as well.
https://github.com/helm/helm/pull/7874#issuecomment-1674062156

Does it make sense to decide between LocateChart and LocateRemoteChart at the call site when RemoteRepo is a flag on ChartPathOptions? It seems like the logic would belong in LocateChart, as that function is on the options struct itself.
In terms of API users, they can also set the RemoteRepo flag when calling LocateChart if they only want to locate remote charts.
If it does make sense then I'm happy to implement it this way :+1:

I think the reason not to choose ahead of time is that it doesn't require anyone using the API to substantively change their code. By simply changing the option, they can get the new behavior.
In fact, now that I think of it, I'd not export locateRemoteChart so the use of the API is clear.

Sounds good, I've pushed the suggested changes.


Improve the quality of the St. Anthony's College logo

Purpose
The purpose of this PR is to fix #56
Goals
To improve the quality of the St. Anthony's College logo
Approach

Screenshots
Berfore

After

Preview Link
https://valancethomas5.github.io/sack-site/
Checklist

[x] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
[ ] I have read and understood the development best practices guidelines
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Related PRs

Test environment

Learning

@all-contributors please add @ValanceThomas5  for code

@ValanceThomas5 are you still working on this issue?

Please update your progress on this.

Closing since no activity.


Users.getCards returns only 10 cards

When I call Users.getCards it only returns the last 10 cards.
How can I get the others ones ?

Hello @adrienbarreau ! Here you have an examples for pagination on the Users.getAll https://github.com/Mangopay/mangopay2-nodejs-sdk/blob/master/examples/getUsers.js#L25
In your case should be
// Get 100 users
api.Users. getCards({
    parameters: {
        per_page: 100,
        page: 1
    }
})

// Get 10 users but the second page
api.Users. getCards({
    parameters: {
        per_page: 10,
        page: 2
    }
})

Hey, I am facing the same issue, but I am not getting, exactly where I should pass parameters data.
I am using promise with the API call. Please have a look at my code snippet, for more reference.

api.Users.getCards(userId).then(function (response) {
console.log('response');
console.log(response);
}, function (error) {
console.log('mangopay service: error in fetching cards for user');
console.log(error);
});
Can you help me, where I can pass the parameters data? Also, what value should I pass to get all the data?

Hey there @foramkumarparekh!
Please try it like this:
api.Users.getCards(userId, _parameters_).then(function (response) {
    console.log(response);
}, function (error) {
    console.log(error);
});

You should be able to find data about how many pages are left or maybe how many total items from the response headers such as:
console.log(response.headers['x-number-of-pages']);

Hope it helps!


Can't import custom script to wrapper

Prerequisites

[X] Verify that this is not a Windows issue;
[X] Refer to the system requirements;
[X] Refer to the How to use;
[X] I do not use a homebrew Windows image;
[X] I did not tweak Windows before that could cause system instability;
[ ] If your issue concerns the Wrapper, please mention @BenchTweakGaming in issue

Steps to reproduce
import original script, export, import again.

no matter what chosen, nothing happens, all fields is empty. Still can import original script again.
wrapper 2.7.0 & 2.7.1
Windows Version
win11 22h2
Sophia Script version
Windows_11_PowerShell_7_v6.6.7

fixed. was something weird with script related path(

Weird path happens)

I've fixed some things in the last commit.
Please use the latest code, by downloading it manually:
https://github.com/farag2/Sophia-Script-for-Windows/blob/master/src/Sophia_Script_for_Windows_11/Sophia.ps1
https://github.com/farag2/Sophia-Script-for-Windows/blob/master/src/Sophia_Script_for_Windows_11/Module/Sophia.psm1


Add covariant keyword to grammar

I had this request in Dart Code, probably wants mirroring here: https://github.com/Dart-Code/Dart-Code/commit/e2e1aaed413cad9735255f348be035244052accd

Thanks for the heads up and commit reference!