Not able to test usage of notifications during acceptance tests

Hi,
I'm not able to test notifications in my acceptance tests. I imported Notify and set testing to true but it doesn't work.
This is my test:
test('submit empty form', function (assert) {

  page
    .visitWithToken()
    .submitEmptyForm();

  andThen(function () {

    assert.equal(page.notifyBoxAmount, 1, 'alert notification is showing up');

  });
});

and page.notifyBoxAmount = count('.ember-notify').
My controller function looks like this:
confirmUserAccount () {

      if (isEmpty(confirmData.email)) {

        this.get('notify').alert(this.get('i18n').t('error.optin_41').string);

      }
}

Does anyone have an idea on this?

I wouldn't test this using the DOM. I'd set this.notify.alert() to a spy and confirm that it's called.
If you're using the DOM then you'll need to wait for the child to be inserted, which may take an extra turn of the event loop.
You shouldn't set EmberNotify.testing to true, this is used for its own internal testing. The default behaviour in tests is to run asynchronously, but without any delays.

@aexmachina Would you have an example of using a spy in an acceptance test to test for this? I'm pretty lost and have been searching for hours trying to understand how to do this.

I’m afraid I’m not using this project at the moment Jordan. In an
acceptance test you probably just want to andThen check that the element is
in the DOM, but it’s been a while since I’ve used it. You can set
Notify.testing
= true to disable the delays, see ember-notify-test.js for an example.


HxBoostedBy macro to avoid redundant if-else for layout

Hi!
I'd like to propose a macros feature in addition to HxBoosted extractor.
Currently with axum-htmx we use HxBoosted like this:
async fn get_index(HxBoosted(boosted): HxBoosted) -> impl IntoResponse {
    if boosted {
        // Send a template extending from _partial.html
    } else {
        // Send a template extending from _base.html
    }
}

I propose to add a macro, then it will be:
#[hx_boosted_by(with_layout)]
async fn get_index() -> Html<String> {
    // some impl here

    // <-- page html -->
}

fn with_layout(Html(html): Html<String>) -> Html<String> {
    // Wrap page html into layout if called
}

What this macro does is transforming that function to:
async fn get_index(HxBoosted(boosted): HxBoosted) -> Html<String> {
    // some impl here

    if boosted {
        // <-- page html -->
    } else {
        // <-- with_layout(page html) -->
    }
}

I don't have much experience with Rust but managed to write something close to that macro in this axum-htmx-derive repo. Didn't publish yet, don't think it requires separate package - but needs to be included in axum-htmx (with feature flag).
Currently it has 2 macros: hx_boosted_by and hx_boosted_by_async - if you have any idea on how it can be merged I'm here :)
Let me know what you think.

I'm okay with this. I think reducing the boilerplate is worth the additional complexity and I'm willing to maintain it long-term. It fits within the spirit of the library.
Are you interested in starting a draft PR for this here? I imagine that most of your downstream work should be generally transferrable, though I haven't had a chance to look through your repo yet.

Having used htmx + askama for a little while now the duplication and awkwardness of partial vs full pages is one of, if not the biggest pain point. Here's what it looks like for me:
Constraints:

I want all my URLs to work both in htmx and not. The latter is important for e.g. bookmarks/link sharing to work.
I want a global layout template (think top nav) that needs some data loaded from the DB (e.g. the logged in user, to get the avatar URL).

This ends in sadness. My workflow today looks as follows:

The above ifs in every router plus 2 askama Template structs.
Two .html template files to go with the above structs e.g.

index.html:
{% extends "../layouts/base.html" %}

{% block content %}
{% include "_index.html" %}
{% endblock %}

_index.html:

Register

<form hx-post="/register" hx-target="#container" hx-swap="innerHTML" hx-push-url="true">
  <fieldset>

      Email
      <input class="form-control" name="email" />

      Password
      <input class="form-control" name="password" type="password" />

    <input class="btn btn-primary" type="submit" />
  </fieldset>
</form>

This is even before I have tackled including the user in the base template, which will make things even worse because the user only really needs to be loaded in the non-htmx case (as it doesn't change in the htmx case, typically). Also, every route would have to care about the user even though that kind of concern ideally should be dealt with "globally" by some middleware.
Any ideas?


Chinese Translation

Hello Germinate Team,
I've translated the en_GB.json file into Chinese, mainly using OpenAI's translation. Some parts have been manually checked, but due to time constraints, I wasn't able to review everything.
I hope this translation can help make Germinate more accessible to Chinese speakers.
zh_CN.json

Hi there,
Thanks a lot for posting this! We're always looking to make Germinate available in more languages.
Creating a translation through AI isn't necessarily ideal without someone going through it and verifying the quality. I appreciate that it's a lot of work to double check the translation, but from our point of view, there's no way to know how good a translation it is since we don't speak Chinese.
If you feel this is something you would be able to do (maybe with the help of some colleagues), we'd invite you to create a pull request against the Germinate i18n repository.


mailing list archives

I'm getting started with AsciidocFX and I'd like to read the mailing list archives, and possibly join the mailing list, but I do not have a Gmail account and I do not want to give Google my birthday and phone number, which they now require to create an account. Are the mailing list archives available anywhere on the web? Is it possible to join the mailing list without a Gmail account?
Thank you.

You can access it publicly here https://groups.google.com/d/forum/asciidocfx-discuss . Also, you can read discussions on gitter.


Xml branch merge into master

added xml - serialization and structurization. Little fixes in KeepersFactory class (Keepers memory freeing)

Merge XML branch into MASTER where have JSON serialization


Remove default BLACKLIST

Why did we do this, out of curiosity?

TL;DR its unnecessary and doesn't cover valid use-cases. The intent is the variable jQuery methods flagged into a blacklist should to be provided by the consuming application.
The existing BLACKLIST was only a sub-set of jQuery methods from a specific version of jQuery (v2.1.4). Since consuming applications will be on varying versions we shouldn't default a set or sub-set of methods as we won't cover all use-cases.

@trentmwillis Agreed. I'll include a comment regarding usage.

Good to go. Merge away!


Can't input run arguments to the project with VS Code extension

Is your feature request related to a problem? Please describe.
Want to use the run command for a binary target.  Need to supply the command line arguments.  In my case, a single int. No way to do it.
Describe the solution you'd like
Should be an input to the BuildRun and Run commands in the extension.
Describe alternatives you've considered
Well, I try using the command line approach from within VS Code.  Certainly will work after I figure out how to do it in xmake commandline api.
Additional context
Sadly, a very large repertoire (but not ALL) of the things we can do from the command line need to be supported in the extension.  It is sort of ad hoc to figure out which commands and arguments are needed.

see https://github.com/xmake-io/xmake-vscode/blob/0f69b477af12d3e8cc6ac197aae80dc053271ddd/package.json#L675

Not sure what I am supposed to see there.
Using the commandline is easy-peasy:    xmake r [target]  [args]
This is pretty low priority feature request.  Probably take some time to think about what makes the vs code extension both convenient and worthwhile to use.  I feel your pain that having to duplicate your commandline interface for every IDE you choose to support (and you are good about aiming for a lot of them) is a bit of pain when most modern IDEs have reasonable terminal support.  Here's a suggestion: just create a very open ended command that opens the default terminal (or uses one that's already open) and simply passes: xmake [task] [options] [project] [extra_args].  Maybe this isn't much of a plus but it enables some people to just live in their IDE.   And then you are done.  Then, only directly implement the commands that are a little bit more complicated and more involved.

or use set_runargs https://xmake.io/#/manual/project_target?id=targetset_runargs


osdialog_gtk2.c mixes std allocation and gtk allocation

I saw that in osdialog_gtk2.c we return a string that is allocated with gtk_file_chooser_get_filename
	char *result = NULL;
	if (gtk_dialog_run(GTK_DIALOG(dialog)) == GTK_RESPONSE_ACCEPT) {
		result = gtk_file_chooser_get_filename(GTK_FILE_CHOOSER(dialog));
	}

when we return this string directly we are in danger that on a later point we might try to deallocate the result with std::free.
The problem is that gtk uses it's own memory management, and mixing g_malloc and regular free leads to issues.
I would recommend copying the result string with strdup and deallocating the previous string with g_free before returning.
Also see: https://mail.gnome.org/archives/gtk-list/2000-July/msg00002.html

Thanks, fixed in 65a3b2e


I dont know if this is bugs or my code conventions using GetX

══╡ EXCEPTION CAUGHT BY ANIMATION LIBRARY ╞═════════════════════════════════════════════════════════
The following assertion was thrown while notifying listeners for AnimationController:
'package:flutter/src/widgets/overscroll_indicator.dart': Failed assertion: line 243 pos 14:
'notification.metrics.axis == widget.axis': is not true.
                  Either the assertion indicates an error in the framework itself, or we should provide substantially
                  more information in this error message to help you determine and fix the underlying cause.
                  In either case, please report this assertion by filing a bug on GitHub:
                    https://github.com/flutter/flutter/issues/new?template=2_bug.md

                  When the exception was thrown, this was the stack:
                  #2      _GlowingOverscrollIndicatorState._handleScrollNotification (package:flutter/src/widgets/overscroll_indicator.dart:243:14)
                  #3      _NotificationElement.onNotification (package:flutter/src/widgets/notification_listener.dart:130:38)
                  #4      _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3094:18)
                  #5      _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3097:13)
                  #6      _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3097:13)
                  #7      _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3097:13)
                  #8      _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3097:13)
                  #9      _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3097:13)
                  #10     _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3097:13)
                  #11     Element.dispatchNotification (package:flutter/src/widgets/framework.dart:4441:24)
                  #12     Notification.dispatch (package:flutter/src/widgets/notification_listener.dart:60:13)
                  #13     DrivenScrollActivity.dispatchOverscrollNotification (package:flutter/src/widgets/scroll_activity.dart:658:108)
                  #14     ScrollPosition.didOverscrollBy (package:flutter/src/widgets/scroll_position.dart:933:15)
                  #15     ScrollPosition.setPixels (package:flutter/src/widgets/scroll_position.dart:285:9)
                  #16     ScrollPositionWithSingleContext.setPixels (package:flutter/src/widgets/scroll_position_with_single_context.dart:78:18)
                  #17     DrivenScrollActivity._tick (package:flutter/src/widgets/scroll_activity.dart:647:18)
                  #18     AnimationLocalListenersMixin.notifyListeners (package:flutter/src/animation/listener_helpers.dart:161:19)
                  #19     AnimationController._tick (package:flutter/src/animation/animation_controller.dart:837:5)
                  #20     Ticker._tick (package:flutter/src/scheduler/ticker.dart:249:12)
                  #21     SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1175:15)
                  #22     SchedulerBinding.handleBeginFrame.<anonymous closure> (package:flutter/src/scheduler/binding.dart:1079:11)
                  #23     _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:617:13)
                  #24     SchedulerBinding.handleBeginFrame (package:flutter/src/scheduler/binding.dart:1077:17)
                  #25     SchedulerBinding._handleBeginFrame (package:flutter/src/scheduler/binding.dart:994:5)
                  #26     _invoke1 (dart:ui/hooks.dart:167:13)
                  #27     PlatformDispatcher._beginFrame (dart:ui/platform_dispatcher.dart:296:5)
                  #28     _beginFrame (dart:ui/hooks.dart:104:31)
                  (elided 2 frames from class _AssertionError)

                  The AnimationController notifying listeners was:
                    AnimationController#9c2e0(⏭ 822.857; paused; for DrivenScrollActivity)
                  ════════════════════════════════════════════════════════════════════════════════════════════════════

Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.3.4, on Microsoft Windows [Version 10.0.22621.674], locale en-PH)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.1.6)
[√] Android Studio (version 2020.3)
[√] Android Studio (version 2021.3)
[√] IntelliJ IDEA Community Edition (version 2022.2)
[√] Connected device (4 available)
[√] HTTP Host Availability
• No issues found!

@haroldcalayan-mycwt
Check this similar issue and underlying comments / solutions mentioned and see if they work in your case.
https://github.com/flutter/flutter/issues/107772#issuecomment-1199083857
https://github.com/flutter/flutter/issues/107772#issuecomment-1219906155
https://github.com/flutter/flutter/issues/107772#issuecomment-1233937098
Closing for now. If above doesn't work in your case, please provide a complete minimal reproducible code sample without using third party plugin which throws the same error log, so that we can properly analyze the issue.


participant definition proposal

participant definition changes:

new "member" definition inside participant for {name, firmName, groupId, groupName, timeZoneOffset}

"farEndName" field removed (use participant > member > name) - if we list all the participants in "participants" we can get farEndName from there

new "involvement" field -  "enum":["source", "destination", "participant"] - proposal is to use it instead of "source" and "destination" fields on root level

"states" definition lists all the participant states with timestamp and duration

Vitor - I am clearly happy with these but as I am maybe more involved in the specific changes it is probably best if you review and approve / comment?

@pinczo  After the merge of PR #44 into master your PR is now conflicting.
I'd like to suggest to separate it into distinct PRs, one for the member definition change and another for call states, since they are non-related changes.
So my suggestion is to create these two PRs:
1 - Something like "Move participant user information to it's own definition" that will contain the member change commit.
2 - Something like "Store call states overtime", with the state changes.
I'll approve (1) immediately, but I'll like to propose some changes in (2).
I can assist you in this if you want. Just let me know.


even after updating the search code I dont see any changes in the stories screen...

Also on clicking a singel story it doesnt go to the details view.

Hi @jpca999,
The sample data is imported from constants/data and stored in the variable storyData and later it is being used in the renderStories method.
To make the search work you'd need to update this storyData or add it to state everytime a new search happens.
In the code above, you need to add a state:
state: {
        stories: []
    }

and then use
componentWillMount(){
    // update the state using the API call and feed that data into `stories`
}

and later use renderStories
this.state.stories.map(this.getStoryData)

Thanks


[Bug]: MetricUpdates.isEmpty check does not check for gaugeUpdates

What happened?
org.apache.beam.runners.core.metrics.MetricUpdates.isEmpty method does not check for gaugeUpdates to be empty.
Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[X] Component: Python SDK
[X] Component: Java SDK
[X] Component: Go SDK
[X] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[X] Component: Spark Runner
[X] Component: Flink Runner
[X] Component: Samza Runner
[X] Component: Twister2 Runner
[X] Component: Hazelcast Jet Runner
[X] Component: Google Cloud Dataflow Runner

.take-issue


Named prepared statement parameters cannot be used more than once in a statement

PHP Driver version or file name
pdo_sqlsrv
SQL Server version
Microsoft SQL Server 2019 (RTM-CU10) (KB5001090) - 15.0.4123.1 (X64)
Mar 22 2021 18:10:24
Copyright (C) 2019 Microsoft Corporation
Developer Edition (64-bit) on Linux (Ubuntu 20.04.2 LTS)
Client operating system
macOS
PHP version
8.0.12
Microsoft ODBC Driver version
17
Table schema
N/A
Problem description
One of the advantages of using named prepared statement parameters is being able to reuse them multiple times in the same statement without having to bind them multiple times. It works with some PDO-based drivers but not with pdo_sqlsrv.
Expected behavior
Named prepared statement parameters can be used more than once.
Repro code or steps to reproduce
$connection = new PDO('sqlsrv:server=<IP_ADDRESS>', 'sa', 'Passw0rd');
$sql = 'SELECT :name, :name';
$stmt = $connection->prepare($sql);
$stmt->bindValue("name", 'test');
$stmt->execute();
var_dump($stmt->fetch(PDO::FETCH_NUM));

// PDOException: SQLSTATE[07002]: [Microsoft][ODBC Driver 17 for SQL Server]COUNT field incorrect or syntax error

The same works with pdo_mysql:
$connection = new PDO('mysql:host=<IP_ADDRESS>', 'root', '');

// array(2) {
//   [0] =>
//   string(4) "test"
//   [1] =>
//   string(4) "test"
// }

And pdo_sqlite:
$connection = new PDO('sqlite::memory:');

// array(2) {
//   [0] =>
//   string(4) "test"
//   [1] =>
//   string(4) "test"
// }

Hi @morozov
It's by design. For your information this is duplicate issue of #1096

@yitam, I wasn't aware that it's by design. Thanks for pointing it out. Closing the issue.


Routing Capacity no result for bigger max capacity and demand capacity with same ratio

Hello @Mizux )) Thank you for support!
Copy pasted example. With same data working perfectly. However, there is big but, when I change sample data:
from:
  demands = [0, # depot
             1, 1, # row 0
             2, 4,
             2, 4,
             8, 8,
             1, 2,
             1, 2,
             4, 4,
             8, 8]
  capacities = [15, 15, 15, 15]

to (multiply to 10, ex)
demands = [0, # depot
             10, 10, # row 0
             20, 40,
             20, 40,
             80, 80,
             10, 20,
             10, 20,
             40, 40,
             80, 80]
  capacities = [150, 150, 150, 150]

The solver is thinking too much and not stopping, even without error. For this purpose I set time limit to = 10 seconds, and result is None.
Please note, I have not changed ratio between demand and capacities of vehicle, so no result.
Also, I have increased capacities to more larger values, still not working.
P.S. This is example is for testing purpose. In actual code of mine which is similar case.

This example use capacity and Time Windows.
Time dimension use a arc cost i.e. transit callback computed as:
Transit(A, B) = ServiceTime(A) + TravelTime(A, B)
and... service time is computed as data['time_per_demand_unit'] * data['demand'][index]
with data['time_per_demand_unit'] == 5 min IIRC.
So when you increase demands by 10x -> you increase service time by 10x -> no solution can be found -> solver seems to hang searching for a while for an impossible solution Q.E.D.


Resolve signature issue causing crashes on iOS 13 under RN version 0.56.1

Summary
Tried to build an app under RN 0.56.1 but it crashed immediately. A fix was provided on recent version with https://github.com/facebook/react-native/pull/25146, I'm just bringing this fix to 0.56.
Changelog

[iOS] [Fixed] - Resolve signature issue causing crashes on iOS 13 under RN version 0.56.1.
Test Plan
Just built an app and it worked. No more crashes!

Thank you for your PR! We are not actively maintaining stable branches that are more than a year old. In case you cannot upgrade the version of React Native you are using I recommend forking from the stable point and building RN from source.


ReactorKit doc build fails

Probably an unavailable target:
https://swiftpackageindex.com/builds/CFAD4AA5-3C2E-45C5-B4F4-49F29850E7E6
    OTHER_DOCC_FLAGS = --emit-digest --source-service github --source-service-base-url https://github.com/ReactorKit/ReactorKit/blob/master --checkout-path /Users/builder/builds/UKfvDsi8u/1/finestructure/swiftpackageindex-builder/spi-builder-workspace
Resolve Package Graph
Resolved source packages:
  SwiftDocCPlugin: https://github.com/apple/swift-docc-plugin @ 1.3.0
  RxSwift: https://github.com/ReactiveX/RxSwift.git @ 6.5.0
  SymbolKit: https://github.com/apple/swift-docc-symbolkit @ 1.0.0
  ReactorKit: /Users/builder/builds/UKfvDsi8u/1/finestructure/swiftpackageindex-builder/spi-builder-workspace
  WeakMapTable: https://github.com/ReactorKit/WeakMapTable.git @ 1.2.1
objc[8403]: Class XROS1_1SimRuntime is implemented in both /Library/Developer/CoreSimulator/Volumes/xrOS_21O209/Library/Developer/CoreSimulator/Profiles/Runtimes/xrOS 1.1.simruntime/Contents/MacOS/xrOS 1.1 (0x110b600e0) and /Library/Developer/CoreSimulator/Volumes/xrOS_21O5181e/Library/Developer/CoreSimulator/Profiles/Runtimes/xrOS 1.1.simruntime/Contents/MacOS/xrOS 1.1 (0x110c7c0e0). One of the two will be used. Which one is undefined.
2024-03-18 16:15:10.447 xcodebuild[8403:881420] Writing error result bundle to /var/folders/b3/8p5yvk0d16s637btgmtl295m0000gp/T/ResultBundle_2024-18-03_16-15-0010.xcresult
xcodebuild: error: The workspace named "spi-builder-workspace" does not contain a scheme named "ReactorKitRuntime". The "-list" option can be used to find the names of the schemes in the workspace.
Error while generating docs: retryLimitExceeded(lastError: Optional("Shell command failed:\n    env DEVELOPER_DIR=/Applications/Xcode-15.3.0.app xcrun xcodebuild -IDEClonedSourcePackagesDirPathOverride=$workDir/.dependencies -skipMacroValidation -skipPackagePluginValidation -derivedDataPath $workDir/.derivedData docbuild OTHER_DOCC_FLAGS=--emit-digest --source-service github --source-service-base-url https://github.com/ReactorKit/ReactorKit/blob/master --checkout-path $workDir -scheme ReactorKitRuntime -destination generic/platform=ios"))

Fixed


fix(storage): fix timeout error in do_request and unit test

Fixes [9951]

@Emar-Kar  I don't think timeout with value is meaning full here because we just comparing here a request call and i saw the same has by done tswast  in bigquery  and even he didn't add test case.

Chris wilcox create a duplicate issue and  PR and merged it so closing this PR.


Garbage collection

Once we are done with #411, we will be in a very good position to finally implement a garbage collector that removes unreferenced blocks, since all block references will be stored in a very queryable database (RethinkDB).

An initial proposal for how to do this:

Add a global generation counter
All commits are assigned to the current generation on creation
Any time a block is written as part of a commit we record the generation it's being written in. This allows us to know the most recent generation a block has been written in.
When we want to gc we increment the global generation. If the generation is n then we know that all blocks in generation [0,n-1] that aren't referenced in commits [0,n-1] can be GCed.
There will never be any new Commits in [0,n-1] so there's no writes that this computation can race with. If Commits are later deleted though GC can be rerun to reclaim the space.

Just to restate the user's use case from #914 :
They ran out storage on their local disk (which prevented them from running pipelines). They did pachctl delete-all and were surprised that all of the raw files weren't also deleted. This seems like a reasonable thing to be surprised about.
I'm not sure what the implementation would be to service this (I think there are several), but want to make sure we understand this use case when considering our design.

I think the above implementation would service this. Calls to pachctl delete-all would leave all of the blocks unreferenced which means that they would eventually be GCed and the space would be reclaimed.

This is also a requirement for us, commenting here so I can keep track of what's happening :)

Now that 2.0 is in master with full featured GC it's time to close this. 🎉 🎉


Fix version update: include test.module too

this should address the failing pipeline we saw last time, while releasing.
problem was that the versions are not sync, web-test-fixture can not found it's related 'web-tester' dependency, as it is referred by dynamic ${project.version} re-using the artifactVersion of itself, which is not auto-updates if we don't update all proejcts in this workspace.
simulated this stuff here:
https://jenkins.ivyteam.io/job/web-tester_release/job/versions/22/console
... now it feels as if we would succesfully update all 8 pom files in this workspace:

Fine for me, if this works. thanks 👍 I think the idea was, to make sure that the fixture project will not be published to the repo.

true, this is given by the maven-deploy:skip config in this pom.xml.
Let's see how it works, I hope we're better on track next release and do not face a failing pipeline. :crossed_fingers:


Use System.Buffers for temporary arrays

Buffers only from #556
/cc @Tratcher

Cleaned up, rebased, and merged. Thanks!


getting httperror(403) while invoking from an githubaction

Hi, I am trying to follow the steps in the Github readme for the SFdx-scan-pull-request.
I am getting httperror(403) and the scan is not happening.

Could you please guide me where is the issue and how to resolve those

Thank you. I gave permission in github runner. I gave write-all permission and it worked.


OffsetDateTime turns into "timestamp without time zone"

When generating DDL for a class with an OffsetDateTime property, it is given the type timestamp, which Postgres interprets as timestamp without time zone. Since it's explicitly a type with timezone information that should be reflected in the database type.
E.g.:
@Entity
public class Example {
    private OffsetDateTime updated;
}

Which gives:
create table example (
  updated                   timestamp)
;

I think this is perhaps more a case that 'timestamp with time zone' could be the default for all timestamps in postgres (so not specific to OffsetDateTime).
I'll look at this a bit more in the next few days.

My testing using 'timestamp with time zone' shows that is makes no difference for any of the java datetime types including OffsetDateTime.
Someone might desire all the timestamp types in PG to be 'timestamp with time zone' ... which could be done by overriding the DbType in the PostgresPlatform for Types.TIMESTAMP.
So I will close this issue. If you want to discuss further etc re-open this issue.
Thanks, Rob.

Don't know if this is relevant to your questions but tried annotating the date property field with @Column(columnDefinition = "timestamp with time zone")?

Thanks, yes that is an option. My take on it is that more likely that if a DBA wanted timestamp with time zone ... that they would want it on all timestamp columns (this is not specific to OffsetDateTime) and in that case you'd look to modify the type in the PostgresPlatform.
That is, personally I would be not using "timestamp with time zone" for just our OffsetDateTime types (I'd be surprised if a Postgres DBA pushed for that case and instead I expect them to push for the "all timestamps should be timestamp with time zone" case).


ts_scte_parser.py not decoding SCTE packets

Hello,
I'm trying to decode SCTE packets from a transport stream via  ts_scte_parser.py but it seems like it is not able to decode SCTE packet in the incoming stream while other applications was able to detect it.
Version:  v.2.1.97
I've edited port and mcast_ip as per my requirement
    def __init__(self, mcast_ip, if_ip="<IP_ADDRESS>", hostname="<IP_ADDRESS>", port=5511):
        self.HOST = hostname

if __name__ == "__main__":
    test = TSScte35Parser(mcast_ip="<IP_ADDRESS>")
    test.run()

Hello,
I'm trying to decode SCTE packets from a transport stream via ts_scte_parser.py but it seems like it is not able to decode SCTE packet in the incoming stream while other applications was able to detect it.
Version: v.2.1.97
I've edited port and mcast_ip as per my requirement
    def __init__(self, mcast_ip, if_ip="<IP_ADDRESS>", hostname="<IP_ADDRESS>", port=5511):
        self.HOST = hostname

if __name__ == "__main__":
    test = TSScte35Parser(mcast_ip="<IP_ADDRESS>")
    test.run()

Could you share either some of the data you're trying to parse or logs?
What type of exception is it throwing?

The code you shared doesn't give much context. That isn't the only code you're trying to run right?

Hi,
it is not showing any exception, Just keep listening the stream.
I'm using tsp to analyze the SCTE in the stream and it is able to decode the packet.
* pcrextract: PID: 0x1033 (4147), SCTE 35 command SpliceInsert out, exec at PTS 0x095655834
* pcrextract: PID: 0x1033 (4147), SCTE 35 command SpliceInsert in, exec at PTS 0x09598DED4

The full code is pasted below, I only pasted the part I've edited.
import socket
import threefive
import struct
import io

class TSScte35Parser():
    # TODO: Add unicast / multicast flag or auto-detect based on IP CIDR
    def __init__(self, mcast_ip, if_ip="<IP_ADDRESS>", hostname="<IP_ADDRESS>", port=5511):
        self.HOST = hostname
        self.PORT = port
        self.MCAST_IP = mcast_ip
        self.IF_IP = if_ip

    def run(self):
        with socket.socket(socket.AF_INET, socket.SOCK_DGRAM, socket.IPPROTO_UDP) as sock:
            self.set_socket_options(sock)
            sock.bind((self.HOST, self.PORT))
            with sock.makefile(mode="b") as socket_file:
                while True:
                    #print(type(socket_file))
                    try:
                        scte = threefive.Stream(socket_file).decode_next()
                        print(scte)
                        if scte:
                            print("Found SCTE-35:", scte)
                            print("--------------------------------")
                    except Exception as e:
                        print("ERROR while decoding TS:", e)
                        pass

    def set_socket_options(self, sock):
        sock.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 1)
        sock.setsockopt(socket.IPPROTO_IP, socket.IP_ADD_MEMBERSHIP, socket.inet_aton(self.MCAST_IP)+socket.inet_aton(self.HOST))
        # TODO: Below is used to specify a specific interface to use for MC subscription

        # group_bin =  socket.inet_aton(self.MCAST_IP)
        # local_bin =  socket.inet_aton(self.IF_IP) # Which interface to IGMP join on

        # sock.setsockopt(socket.IPPROTO_IP, socket.IP_MULTICAST_IF, local_bin)

        # mreq = struct.pack("4sL", group_bin, socket.INADDR_ANY)
        # # mreq = group_bin + socket.inet_aton('<IP_ADDRESS>') # Possible IGMPv3 method

        # sock.setsockopt(socket.IPPROTO_IP, socket.IP_ADD_MEMBERSHIP, mreq)

if __name__ == "__main__":
    test = TSScte35Parser(mcast_ip="<IP_ADDRESS>")
    test.run()

Is it a satellite feed with embedded streams?

Try this:
# test.py
import sys
from threefive import Stream

def read_stdin():
    Stream(sys.stdin.buffer).decode()

if __name__ == "__main__":
    read_stdin()

then use tsduck and pipe the stream to it .
tsp -rtrue -I ip <multicast_ip>:<port_number>  -O - | python3 test.py

Hi Adrian,
Following is the output of test.py, The python application kept listening to the stream while tsp decoded it.

I'm going to provide you a sample transport stream dump of what I'm trying to achieve. Can you please share your's email ID.

When you run ffprobe on the stream, does it look similar to this?
[mpegts @ 0x55e667178240] Packet corrupt (stream = 0, dts = NOPTS).
[mpegts @ 0x55e667178240] PES packet size mismatch
[mpegts @ 0x55e667178240] Packet corrupt (stream = 0, dts = NOPTS).
[mpegts @ 0x55e667178240] start time for stream 0 is not set in estimate_timings_from_pts
Input #0, mpegts, from '../rus.ts':
  Duration: N/A, bitrate: N/A
  Program 800
    Stream #0:0[0x1000]: Data: bin_data ([6][0][0][0] / 0x0006)
Unsupported codec with id 100359 for input stream 0

You probably need to demux it. If it has t2Mi embedded streams like this one
https://tsduck.io/streams/?name=russia-t2mi-scte35
tsp -rtrue -I ip <multicast_ip>:<port_number>   -p  t2mi --pid 0x1000 -plp 0 | python3 test.py

You probably need to demux it. If it has t2Mi embedded streams like this one
https://tsduck.io/streams/?name=russia-t2mi-scte35
tsp -rtrue -I ip <multicast_ip>:<port_number>   -p  t2mi --pid 0x1000 -plp 0 | python3 test.py

Most likely, you can run the same tsduck command you are running now without the ' -O drop ' at the end and pipe it to ' python3 test.py.

I'm getting same output as tsp while running python3 test.py

tsp -I ip <IP_ADDRESS>:5511 -l <IP_ADDRESS> -P pcrextract --scte35 --pid 0x1033 | python3 test.py

pcrextract: PID: 0x1033 (4147), SCTE 35 command SpliceInsert out immediate
pcrextract: PID: 0x1033 (4147), SCTE 35 command SpliceInsert in immediate

and ffprobe as you asked

Lose the
'-P pcrextract --scte35'

Stil same

I thought the '-l <IP_ADDRESS>' was another input, what does -l do?
Why is tsp showing scte35 without the ' -P pcrextract --scte35' ?

Download a minute or two of the video and upload it to your github account for me so I can figure it out, please.

My tsp uses -l to list available inputs and outputs , it doesn't accept any parameters.
Without the -P pcrextract --scte35 , tsp should not be showing scte35 info.
Are you using a customized tsp or something?
From your ffprobe output, threefive should have no problem parsing it.

What happens if you pipe it to mplayer ?

I thought the '-l <IP_ADDRESS>' was another input, what does -l do?

-l is used when I want listen to the multicast from a particular interface.

Why is tsp showing scte35 without the ' -P pcrextract --scte35' ?

I'm running 2 tsp, one is with  -P pcrextract --scte35 just to determine when the scte is coming and below is output of what you instructed to do.

Download a minute or two of the video and upload it to your github account for me so I can figure it out, please.

Please download it from here
https://drive.google.com/file/d/1Qo0LSsyXMyW5j4RHsc32tati2sqO0RRM/view?usp=sharing

What happens if you pipe it to mplayer ?

I'm running it in headless mode, pipe to mplayer will be a challenge.

Solved as of version 2.2.09. I added support for earlier versions of SCTE-35.


TypeError: Cannot read properties of undefined (reading 'split') at functionsWorker-0.7282352485608163.js:5429:32

Environment
Operating System: Windows_NT

Node Version:     v16.20.0
Nuxt Version:     3.5.3
Nitro Version:    2.5.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      app, modules, i18n, ssr, runtimeConfig, devtools
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
// https://nuxt.com/docs/api/configuration/nuxt-config
export default defineNuxtConfig({
modules: [
'@nuxtjs/tailwindcss',
'@pinia/nuxt',
'@nuxt/devtools',
'@nuxtjs/i18n',
[
'@nuxtjs/google-fonts',
{
families: {
ABeeZee: true,
download: true,
inject: true,
},
},
],
],
i18n: {
strategy: 'prefix_except_default',
locales: [
  {
    code: 'en-GMG',
    iso: 'en-GMG',
    name: 'English(UK)',
    file: 'en-GMG.json',
  },
  {
    code: 'fil-PH',
    iso: 'fil-PH',
    name: 'Filipino',
    file: 'fil-PH.json',
  },
],
lazy: true,
langDir: 'lang',
defaultLocale: 'en-GMG',

},
ssr: false,
runtimeConfig: {
public: {
commit: process.env.CF_PAGES_COMMIT_SHA || 'UNKNOWN',
built_at: new Date(),
},
},
devtools: {
enabled: true,
},
})
Describe the bug
0
Does anyone know how to fix this error?
Cannot read properties of undefined (reading 'split') at functionsWorker-0.7282352485608163.js:5429:32
I keep encountering it upon deploying to cloudflare but seems to be working fine on development.
I did look for answers around but no luck!
Additional context
No response
Logs
No response

I've been dealing with the exact same issue today. Reverting to beta 10<EMAIL_ADDRESS>put a stop to this for me, although I'm having to find other workarounds now...

Thank you @brucethwaits, reverting back to  "@nuxtjs/i18n": "8.0.0-beta.10", worked!

Also ran into this, breaks the nitro netlify-edge build.
nuxt/nuxt#21719

duplicate issues
#2095
#2031
#1993

This issue was fixed since @intlify/bundle-utils is no longer bundled on the server side as of v8.0.0-beta.13.
If you have still issue, please reply.


Possible error in documentation

When using the command provided here, I only got this error message:
go build main.go -o bluetuith-bin
named files must be .go files: -o

I have no idea about go and just used it the way it worked "out of the box", so maybe I was missing some important steps of setup. However, I was able to build it using this syntax:
go build -o bluetuith-bin main.go
So I think the documentation could use outdated/wrong syntax.
This is definitely low priority and doesn't require a quick fix, but maybe it helps others who have the same problem.

Thank you for noticing this. This will be fixed soon.


🐛 BUG: API check pages deployments return 400 while previous APIs are fine.

Which Cloudflare product(s) does this pertain to?
Wrangler
What version of Wrangler are you using?
2.10.0
What operating system are you using?
Linux
Describe the Bug
API check pages deployments return 400 while previous APIs are fine.
I already checked with<EMAIL_ADDRESS>and they said no blocks or bans are found with my account. I still can login on the web.
Attached logs from CI-CD pipelineLogError-{7a1a7ac0-0952-4574-85ed-61a7f514eca8}.txt
-- START CF API REQUEST: POST https://api.cloudflare.com/client/v4/accounts/$CLOUDFLARE_ACCOUNT_ID/pages/projects/$PROJECT_NAME/deployments
HEADERS: {
"User-Agent": "wrangler/2.10.0"
}
INIT: {
"method": "POST",
"body": {},
"headers": {
"User-Agent": "wrangler/2.10.0"
}
}
-- END CF API REQUEST
-- START CF API RESPONSE: Bad Request 400
HEADERS: {
"cf-cache-status": "DYNAMIC",
"cf-ray": "79a4e8eb3ab304fe-HKG",
"connection": "keep-alive",
"content-encoding": "gzip",
"content-type": "application/json; charset=UTF-8",
"date": "Thu, 16 Feb 2023 08:28:29 GMT",
"server": "cloudflare",
"set-cookie": "__cflb=0H28vgHxwvgAQtjUGUFqYFDiSDreGJnUxV31Mq3cSms; SameSite=Lax; path=/; expires=Thu, 16-Feb-23 10:58:30 GMT; HttpOnly, __cfruid=a7554c2addecff0ca6c6f805539a9406cd58642c-1676536109; path=/; domain=.api.cloudflare.com; HttpOnly; Secure; SameSite=None",
"transfer-encoding": "chunked",
"vary": "Accept-Encoding",
"x-envoy-upstream-service-time": "1731"
}
RESPONSE: {
"result": null,
"success": false,
"errors": [
{
"code": 8000036,
"message": "Your Cloudflare account has been blocked. Contact<EMAIL_ADDRESS>}
],
"messages": []
}
-- END CF API RESPONSE

“Hi @ltvaran! Thanks for flagging this. In an effort to remove stale issues so we can have better signal on current friction points, we’re closing out issues that have not been updated within the last few months. If this is still a problem, please feel free to open a new issue referencing this one. Please be sure to include an updated summary and any additional context and we will review. Thank you!”


doc/spec: move API specification into correct position

The goal is to maintain a specification hierarchy under doc/spec. This change
sets the example. The Makefile has also been changed update the AUTHORS file
and can now generate the specification.
Relevant descriptions in the specification and AUTHORS have been updated as a result.
Signed-off-by: Stephen J Day<EMAIL_ADDRESS>

LGTM

LGTM

LGTM


Question: Abbreviations of country names

Hi!
I played around a bit with libpostal and pypostal, and I am quite impressed. Kudos!

My country is
Germany

Here's how I'm using libpostal
Thinking about including it in the Odoo instance of where I work, but I am just in the (private) explorative stage yet. The use-case would be to deduplicate purchased leads.

Here's what I did
I tried using expand_address (from pypostal, but this is not a pypostal issue).

Here's what I got
Worked well with addresses,  like sqr to Square

Here's what I was expecting
I was not able to expand the abbreviation of U.S.A. to e.g.: United States of America ar another representation. Maybe the library is not intended to do so, which would be completely fine with me. I was just wondering, if I made an Error or if that is intentional?

Here's what I think could be improved
More documentation, and maybe reStructuredText docstrings, instead of something doxygen-like in the python parts, because they can be better parsed by Python tools (like e.g. PyCharm)

This is my unittest:
    def test_normalize_address__abbreviations(self):
        address = Address(
            id=0,
            name=Keeper(value="H. P. Lovecraft", keep=True),
<EMAIL_ADDRESS>keep=True),
            company=Keeper(value="Arkham House", keep=True),
            street="Town Sqr.",
            building_number="5",
            postcode="01938",
            city="Innsmouth",
            state="MA",
            country="U.S.A.",
        )
        normalized_address = normalize_address(address, languages=["en"])
        self.assertEqual("town square", normalized_address.normalized_street)
        self.assertEqual("massachusetts", normalized_address.normalized_state)
        self.assertEqual("united states of america", normalized_address.normalized_country)

This is the normalize_address function:
def normalize_address(address: Address, languages: list[str]) -> NormalizedAddress:
    """Normalizes the fields of an address that make sense to be normalized,
    adds fields to the dict of the address with normalized values"""
    normalized_address = NormalizedAddress(
        id=address.id,
        name=address.name,
        email=address.email,
        company=address.company,
        street=address.street,
        building_number=address.building_number,
        postcode=address.postcode,
        city=address.city,
        state=address.state,
        country=address.country,
    )

    normalized_address.normalized_name = normalize_string(address.name.value)
    normalized_address.normalized_email = normalize_email_address(address.email.value)
    normalized_address.normalized_company = normalize_string(address.company.value)
    normalized_address.normalized_street = normalize_address_string(
        address.street,
        languages=languages
    )
    normalized_address.normalized_building_number = normalize_string(address.building_number)
    normalized_address.normalized_postcode = normalize_string(address.postcode)
    normalized_address.normalized_city = normalize_string(address.city)
    normalized_address.normalized_state = normalize_state_string(address.state, languages=languages)
    normalized_address.normalized_country = normalize_country_string(
        address.country,
        languages=languages
    )

    return normalized_address

And this is the normalize_country_string function:
def normalize_country_string(state: str, languages: list[str]) -> str:
    """Normalize state String, like e.g. "MA" for Massachusetts, by expanding it with pypostal"""
    parsed_country = postal.parser.parse_address(
        state,
        language=languages[0],
    )
    expanded_country = postal.expand.expand_address(parsed_country[0][0], languages=languages)
    return expanded_country[0]

Geographic expansions are mostly not included since the implementation is simplistic, just cartesian product of the expansions to create keys for matching. It can lead to a lot of irrelevant results across languages. The library's not really designed for standardizing addresses for display though it can be cludged into doing so. You can always just include your own dictionary of expansions and look up the parsed result in that.


dbeaver: 21.2.0 -> 21.2.2

Motivation for this change
Update to latest release https://dbeaver.io/2021/09/19/dbeaver-21-2-1/ and https://dbeaver.io/2021/10/03/dbeaver-21-2-2/.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 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

[x] Fits CONTRIBUTING.md.

Ci and nixpkgs-review fail
https://github.com/NixOS/nixpkgs/pull/138613/checks?check_run_id=3654848325

hmm strange, now i get "wNXpHSzFCo9QavbbyIj8Q/9IMxymG9A4dn3aZsyblYs="/12wmkg66dnkxfqwd06x63hrlizs3zj4cinznd988y2n55hfykmf0 as the outputHash for the dependencies as well.
comparing the "old" and "new" maven-deps output: diff.txt

Maven is not deterministic hence why we need the FOD.


Does knex support peer authentication?

Environment
Knex version: 0.15.2
Database + version: PostgreSQL 10 (pg<EMAIL_ADDRESS>OS: Ubuntu 18.04
Use case: running postgres and Node app on the same VM, intending to connect via peer authentication
The only information I could find online was #227, which was closed without being resolved.
Current knexfile:
module.exports = {
  // ...
  production: {
    client: 'postgresql',
    // ...
    connection: 'postgresql://user@localhost:5432/dbname'
  },
};

Output on running knex migrate:latest:
error: password authentication failed for user "user"

@ashbc Can you make it work using pure Postgres driver?

Using pg with the same connection string gives the same error. Sorry for not trying this out first :P

np :)

It appears at some point pg may have dropped support for connection strings.
https://github.com/brianc/node-postgres/issues/613#issuecomment-285454310
Changing the options to connection: { host: '/var/run/postgresql', database: 'dbname' } works fine.

@ashbc Thank you for posting the solution!


remark: cannot add onto existing remark

If a contact has already remark 'good with numbers', if user enters:
remark 1 r/bad with writing
The remark 'good with numbers' will be deleted.
Should have appendRemark to add on and not deleted previous remark

Current implementation will overwrite the old remark


Add SECURITY.md file

Description
The repository doesn't contain a SECURITY.md file with contact instructions to report security vulnerabilities.
CodeSandbox/Snack link
N/A
Steps to reproduce
Please add a SECURITY.md file or provide instructions for a security vulnerability to be reported.
Thank you,
gluestack-ui Version
latest
Platform

[ ] Expo
[ ] React Native CLI
[ ] Next
[X] Web
[ ] Android
[ ] iOS

Other Platform
No response
Additional Information
No response

Hey @bananabr, gluestack-ui is a UI component library that depends on numerous npm packages, each of which may have their own dependencies. To check for security vulnerabilities and related issues in your project, you can run npm audit in your repository. This will give you a detailed report of any identified vulnerabilities.

Hey @Viraj-10,
Sorry if I didn't make myself clear enough. I have something to report to the gluestack-ui project maintainers. Best practice dictates that security contacts be made public through a SECURITY.md file to facilitate this sort of sensitive communication.


Provide binary releases for macOS and Windows as well as an official Linux container image

In development it is often useful to use this tool for testing. It would be nice to have macOS and Windows binaries available from the releases page for download as well as a trusted container image available on Docker hub or GCR.

A tool like gox can be used to create the binaries pretty easily.
And Github Actions can be setup pretty easily to handle building the container image when new code is merged.

Happy to provide more cross-platform binaries, however not sure why you need a container image.
That was discussed before in #32 and I am not convinced why this is needed.

Is there some reason that you are hesitant to do this, since it can be pretty easily automated?

This unnecessarily increases the maintenance surface. It's also somewhat equivalent to asking for a docker image for every software you use. After all, it's just a single binary you can download and run. Given we can add more cross-platform binaries that matches the developer’s platform, I still don't know why we’d need a container image.
Can you please illustrate how you'd run this in a container image on a macOS or Windows dev machine and health-check a service that's not running in a container? My argument is that this tool is meant to be invoked over loopback interface (localhost), so running it as a separate container is not really a use case that's worth supporting.

On my Mac I have various scripts which need to run something like this at the end to confirm that things are running as expected.
grpc_health_probe -tls -addr=app-demo.example.com:443
With a native app, I can just use that, however today. I end up doing something like this:
alias grpc_health_probe="docker run --entrypoint=/usr/local/bin/grpc_health_probe stefanprodan/grpc_health_probe:v0.3.0 ${@}"
grpc_health_probe -tls -addr=app-demo.example.com:443

Yeah that's understandable, and a macOS build solves that? So there's no need for a container image?

Hello,
In my team we have the same scenario as OP, it would be very useful to have an official image without having to resort to building our own custom one.
Here's also another use case for Kubernetes: Injecting the probe with an initContainer.
Example:
`spec:
volumes:

name: probe-vol
emptyDir: {}
initContainers:
name: probe
image: grpc-health-probe:1.0.0
command: ['sh', '-c', "cp /bin/grpc_health_probe /bin/probe/grpc_health_probe"]
volumes:
name: probe-vol
mountPath: /bin/probe
containers:
name: server
image: "[YOUR-DOCKER-IMAGE]"
ports:

containerPort: 5000
readinessProbe:
exec:
command: ["/bin/probe/grpc_health_probe", "-addr=:5000"]
initialDelaySeconds: 5
livenessProbe:
exec:
command: ["/bin/probe/grpc_health_probe", "-addr=:5000"]
initialDelaySeconds: 10
volumes:
name: probe-vol
mountPath: /bin/probe
`
This solution could really help in a large microservices based project since you won't have to duplicate and mantain the probe installation in every microservice image.

@luca-esse That's a novel binary drop-in/delivery method I have not seen before, it's clever. The downside of such a method is that it reduces the reproducibility of the container by mutating its fs in the runtime. While it's not a great practice, as you said, it can help in a large-fleet without modifying the main container image.
That said, for uptime and availability, isn't it better if you mirror the image anyway, e.g. don't rely on GHCR.io or Docker Hub while deploying your images? In that case, mirroring the image is pretty much the same burden as having a 2-line dockerfile that does ADD URL /bin/foo RUN chmod +x /foo where you download this binary into a container image?
I get the point of requesting this, but providing a Docker image will open a larger can of worms as it also has its own cross-platform support for different architectures. And again, this request could also apply to potentially many other tools (say, curl), but since the existing delivery methods (binary distribution) also get you there with the same level of effort, I'm currently -1 on Container Image. But I'm ok with more cross-platform binaries.

Hi, I agree with you with the uptime and availabilities issues but there are ways around it.
I'm generally a fan of the image approach and I would like to point out that an official (curl image) actually exists and it has > 1billion downloads, in fact I use it really often when I have to debug something on Kubernetes really quick (the same use case could also apply to this tool, you could use it with a quick kubectl run --image=grpc-health-probe [...] for debugging purposes).
Ofc I could build the image myself and that's what I'm currently doing it but seeing that a few folks also requested it maybe an official image could save all of us some time and we would enjoy the tool better 🙂

@luca-esse @spkane If I provide a container image for this, it will be:

a single binary in an image
with the entrypoint set
no other file (including CA roots)
probably only linux/amd64 arch for the time being
currently unsigned image but it’ll be probably signed with cosign down the line

does this sound good?

I'd personally add the root certs with a line like the one below, as I assume that it needs them in most circumstances.:
COPY --from=0 /etc/ssl/certs/ca-certificates.crt /etc/ssl/certs/ca-certificates.crt

But otherwise, that seems like a pretty reasonable place to start.

@spkane we wouldn't be writing a dockerfile to build the images + adding CA certs and keeping them  up to date is additional maintenance (e.g. say, we don't make a release for 2 years, that CA bundle will go stale). I don't see a clean way of handling this while providing one image per tag.

Sounds good to me 😀

That works for me as well. A few more native baniers and a Linux amd64 binary only container will be helpful. Thank you @ahmetb


[Bug] alloy-consensus should use alloy_primitives::Sealable

Component
consensus, eips, genesis
What version of Alloy are you on?
alloy-consensus v0.2.0
Operating System
None
Describe the bug
There are currently two versions of Sealable/Sealed. One that lives in alloy-consensus and one in alloy-primitives. This should be consolidated and the more general alloy-primitives crate used.

thanks for pointing this out, fully agree
@zerosnacks do you want to take this?


Move with promotion fails on the typescript (dev) branch

I want to set the promotion to "q" as a default but when I run this code on a move not resulting in a promotion
this.chessInstance.move({
  from: ...,
  to: ...,
  promotion: "q",
});

It works on the master branch but not on the dev branch,
Is it made on purpose ?
I used the debugger and noticed that the issue comes from here (move function):
(!('promotion' in moves[i]) || move.promotion === moves[i].promotion)) {

in the master branch, moves[i].promotion does not exist
in the dev branch   , moves[i].promotion is undefined

Thanks for the report @rafalou38.  I'm trying to understand this issue more clearly.  Are you providing { promotion: 'q' } even when the move is not a promotion?

Yes, for all the moves.

This should be fixed on the dev branch.


Building profiling environment - problem with pull docker.io/library/nginx-with-modsec-base:latest

It looks like docker.io/library/nginx-with-modsec-base:latest is not available public:
jan@localhost:~/lmdb-modsecurity-perf-issue> docker-compose -f docker-compose.yaml -f docker-compose.override.profiling.yaml up --build
[+] Running 6/6
 ✔ dangling-backend 5 layers [⣿⣿⣿⣿⣿]      0B/0B      Pulled                                                                                7.8s
   ✔ c7ed990a2339 Pull complete                                                                                0.6s
   ✔ 9086b93f8945 Pull complete                                                                                0.8s
   ✔ 1a5076c5d544 Pull complete                                                                                2.0s
   ✔ a2cfe577cb15 Pull complete                                                                                1.1s
   ✔ 7ea1bd0945c8 Pull complete                                                                                2.1s
[+] Building 4.6s (6/6) FINISHED                                                                                docker:default
 => [kernel internal] load .dockerignore                                                                                0.9s
 => => transferring context: 2B                                                                                0.0s
 => [kernel internal] load build definition from Dockerfile                                                                                1.0s
 => => transferring dockerfile: 587B                                                                                0.0s
 => [base internal] load .dockerignore                                                                                1.0s
 => => transferring context: 2B                                                                                0.0s
 => [base internal] load build definition from Dockerfile                                                                                1.1s
 => => transferring dockerfile: 1.74kB                                                                                0.0s
 => ERROR [kernel internal] load metadata for docker.io/library/nginx-with-modsec-base:latest                                                                                0.8s
 => CANCELED [base internal] load metadata for docker.io/library/ubuntu:23.04                                                                                3.5s
------
 > [kernel internal] load metadata for docker.io/library/nginx-with-modsec-base:latest:
------
failed to solve: nginx-with-modsec-base: pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed

Confirmed working after building first Building base environment (docker-compose up -d --build) and after that Building profiling environment - (docker-compose -f docker-compose.yaml -f docker-compose.override.profiling.yaml up --build)


change source of bundle

修改bundle中默认的ruby官方源为国内源，加快bundle安装下载

Pleas ignore this MR

I'll close this PR since its not something we are going to use.


replace virtualnetworks with virtualNetworks

replace virtualnetworks with virtualNetworks

Hi @DeepakRajendranMsft, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;


layout: Keep output at the correct size, workspace can be smaller.

If the output is not at the correct size then that info must be queried
from wlc. The output size is used by e.g. seamless mouse to detect
output edges.
With this patch the output size is now correct and the workspace size is
adjusted according to any panels.
Without this patch seamless mouse would fail to detect outputs
above/below each other if there was a panel in between because the
output would offically end where the panel started, not at the actual
screen edge.

The behaviour of the bar and gaps are both correct with this patch applied?

Yes, I've tested both.

Excellent, thanks!


Backspace and delete keys don't work when using a TextField as the 'label' of a Chip

I'm using a TextField component as label prop for a Chip. That worked without problems for a long time. Since the type of label is specified with node, I assumed that I could insert any child component there. Since I recently updated to the latest version, there is the following problem: I can still add text (cut and paste works too), but I can no longer delete text with the backspace or delete keys.

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

Current Behavior 😯
Text cannot be deleted using backspace / delete.
Expected Behavior 🤔
Text can be deleted using backspace / delete.
Steps to Reproduce 🕹
https://codesandbox.io/s/sleepy-dust-yjiee
Steps:

Click on the TextField with the current value "foo"
Try to delete the word by pressing the backspace key
Nothing happens...

Context 🔦
I built a editable Chip component. First, the user sees a normal Chip. But when he clicks on it, he can edit the content on-the-fly. As I said at the beginning, it worked very well for a long time.
Your Environment 🌎

Tech
Version

Material-UI
v4.9.7

React
v16.13.0

Browser
Google Chrome Version 70.0.3538.77 (Official Build) (64-bit)

I think it has to do with some of the recent changes to the Chip component (see this commit.
handleKeyDown will do event.preventDefault() for backspace and delete keyboard events.
I wonder if my use case is no longer supported or if this is a bug that should be fixed?

@bunste Thanks for the report, I think that we should have the handler of keyDown and keyUp to use the same logic. What do you think of this patch to align them?
diff --git a/packages/material-ui/src/Chip/Chip.js b/packages/material-ui/src/Chip/Chip.js
index f80d27599..09abc369d 100644
--- a/packages/material-ui/src/Chip/Chip.js
+++ b/packages/material-ui/src/Chip/Chip.js
@@ -260,6 +260,10 @@ export const styles = (theme) => {
   };
 };

+function isDeleteKeyboardEvent(keyboardEvent) {
+  return keyboardEvent.key === 'Backspace' || keyboardEvent.key === 'Delete';
+}
+
 /**
  * Chips represent complex entities in small blocks, such as a contact.
  */
@@ -295,19 +299,21 @@ const Chip = React.forwardRef(function Chip(props, ref) {
     }
   };

-  const isDeleteKeyboardEvent = (keyboardEvent) =>
-    keyboardEvent.key === 'Backspace' || keyboardEvent.key === 'Delete';
-
   const handleKeyDown = (event) => {
+    if (onKeyDown) {
+      onKeyDown(event);
+    }
+
+    // Ignore events from children of `Chip`.
+    if (event.currentTarget !== event.target) {
+      return;
+    }
+
     if (isDeleteKeyboardEvent(event)) {
       // will be handled in keyUp, otherwise some browsers
       // might init navigation
       event.preventDefault();
     }
-
-    if (onKeyDown) {
-      onKeyDown(event);
-    }
   };

   const handleKeyUp = (event) => {

Do you want to work on a pull request? :)

I can work on the this PR? cc @oliviertassinari  @bunste

@chaudharykiran I would also have tried to create a pull request, but you are welcome to do so!


Upgrade kind-projector to 0.10

Oh, it will have to be done with others: https://github.com/monix/monix/issues/862


Spring Version

Updated Spring version of webapp to latest release or possibly 5RC.
Android Requester Spring version is currently 2.

Updated webapp spring version.
Android spring version is current.


Modify trigger arguments passed in spec

According to https://github.com/dtaniwaki/akka-pusher/pull/8, "channel" should come first (sorry I forgot to fix PusherActorSpec)

Thanks a lot!


Add .container-full class

Add support for a full width container element

Feature was added, but still needs to be documented


wxGTK: use GTK 3 by default

Motivation for this change
wxGTK should switch to using GTK 3 where possible. Currently this PR implements the basic change for wxGTK30 and wxPython 3.0. There are a lot of packages that may be affected by this change, as shown in the following nix-review report:

24 packages failed to build:

audacity
boinc
dolphinEmu (refuses to add GTK3 support: https://bugs.dolphin-emu.org/issues/9414)
electricsheep (previously broken)
filezilla
fityk (Debian uses patch: https://salsa.debian.org/science-team/fityk/blob/master/debian/patches/permit-gtk3.patch)
golly-beta (old version, latest is fine; can this be removed?)
grass (intermittent failure, built fine when retried, GUI seems to be broken on master)
opencpn
pcsx2 (1.4 doesn't officially support GTK3, master does, see: https://github.com/PCSX2/pcsx2/issues/2312)
perl528Packages.Wx
perl528Packages.WxGLCanvas
perl530Packages.Wx
perl530Packages.WxGLCanvas
poedit
prusa-slicer
rigsofrods
salut_a_toi
sasview
slade
sladeUnstable
slic3r
wxsqliteplus
xchm

89 packages were built:

aegisub
bossa
cl
couchdb
couchdb2
cubicsdr
curaByDagoma
cura_stable
cuter
digikam
displaycal
dvdstyler
ejabberd
elixir
elixir_1_5
elixir_1_6
elixir_1_7
elixir_1_8
erlang
erlangR18
erlangR19
erlangR20
erlangR21
erlang_javac
erlang_odbc
erlang_odbc_javac
far2l
flamerobin
gnss-sdr
gnuradio
gr-ais
gr-gsm
gr-limesdr
gr-nacl
gr-osmosdr
gr-rds
gnuradio-with-packages
golly
gqrx
hex2nix
hugin
inspectrum
kicad
kicad-with-packages3d
lenmus
limesuite
loxodo
lutris
lutris-free
mercury
metamorphose2
mmex
notmuch-bower
odamex
perl528Packages.AlienWxWidgets
perl530Packages.AlienWxWidgets
playonlinux
plover.stable
printrun
python27Packages.pyspread
python27Packages.robotframework-ride
python27Packages.runsnakerun
python27Packages.wxPython
qradiolink
rabbitmq-server
radiotray-ng
rebar
rebar3
relxExe
rtl_433
saga
scyther
sdrangel
soapysdr-with-plugins
torchat
truecrypt
tsung
urh
welle-io
wings
winpdb
winusb
wxGTK30
wxSVG
wxmaxima
wxsqlite3
xylib
yaws
zeroad

Some of these failures may be prexisting, but many are caused by the switch to GTK 3.
I would like to drop support for GTK 2 altogether in wxGTK, but I'm not sure if this will be possible. I also need to test wxGTK 3.1 with GTK 3.
Things done

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

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

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

slic3r fixed here, added hicolor-icon-theme while i was at it.

i've addressed some of the failures here
i set withGtk2 = true; where it seemed appropriate (where the source doesn't mention gtk3),
though maybe using wxGTK29 is more appropriate for those?
i also defaulted wxGTK31 to gtk3, nothing was negatively affected.
i'm still seeing these failures

dolphinEmu; @delroth seems involved in that, perhaps they can offer clarification?
prusa-slicer; wxWidgets not found
slade, sladeUnstable; should support gtk3 but can't find it
wxsqliteplus; is not fixed by setting withGtk2 = true;

and these which are also broken without this PR

electricsheep
pcsx2
salut_a_toi
sasview

Re: dolphinEmu, the UI was migrated from WX to Qt a few years ago. Unfortunately I don't think any stable release has been tagged since that migration to Qt. I would either keep wxGTK2 or drop dolphinEmu stable from nixpkgs in favor of dolphinEmuMaster.

rigsofrods has a similar problem; master supports GTK 3, but there has not been an official release since 2016. It seems like upstream considered 4.8rc5 to be a stable release. I tried to update it, but ran into a bunch of runtime issues (crashes, missing textures), so I think I'll just have it use GTK2 for now.
I don't know how aggressive we should be in switching to GTK3. Should I apply the patches Debian and Fedora have written to add GTK3 support to the packages that don't officially support it?

I don't know how aggressive we should be in switching to GTK3. Should I apply the patches Debian and Fedora have written to add GTK3 support to the packages that don't officially support it?

@lopsided98 Absolutely, as a gtk maintainer in nixpkgs I'm always glad to come upon these patches where needed :+1:  Though if the package appears completely unused and unmaintained, feel free to remove it in a separate PR.

@lopsided98 you can drop audacity commit since it was done in #73055

I think that is everything except for golly-beta. The only packages left using wxGTK with GTK 2 are dolphinEmu and rigsofrods. Both will be fixed next time there is a new upstream release.
In the case of rigsofrods, I am experimenting with packaging 0.4.8rc5, which supports GTK 3 and seems to be considered stable by the upstream project. They appear to be planning to switch to a date based versioning scheme and make more frequent releases (https://github.com/RigsOfRods/rigs-of-rods/issues/2415), although I'm not a fan of their choice to automatically set the version to the build date (https://github.com/RigsOfRods/rigs-of-rods/pull/2422). The version currently packaged has some visual bugs (ground textures not rendering) that make many maps almost unplayable, and the problems are even worse with the latest version. These bugs are due to some kind of build issue, but I haven't figured it out yet. If I solve this problem, I will make a separate PR.
I modified some of @Evils-Devils commits to take into account @worldofpeace's comment here. I don't know much about GNOME packaging, and adding different GNOME packages as dependencies does not seem to have much effect on the non-NixOS system I am using for testing, so I would like someone to review them. I did notice that using wrapGAppsHook causes the applications to not respect my theme choice.
I would like to remove golly-beta if possible, but I don't know there is a reason to keep it. cc @7c6f434c, who added it.
I am also ccing the maintainers of all packages I touched who were not automatically requested for review by ofborg:
prusa-slicer: @thorstenweber83
slade-git: @ertes
rigsofrods: @7c6f434c
poedit: @bignaux

I modified some of @Evils-Devils commits to take into account @worldofpeace's comment here. I don't know much about GNOME packaging, and adding different GNOME packages as dependencies does not seem to have much effect on the non-NixOS system I am using for testing, so I would like someone to review them. I did notice that using wrapGAppsHook causes the applications to not respect my theme choice.

As I just happen to be a Gnome maintainer I'd be happy to do this :+1:

OK to drop golly-beta now that it is way behind the stable release.
OK whatever happens with rigsofrods, it never was packaged in a fully stable way, so if it builds and starts, it's not a big regression.

Is wrapGAppsHook only needed for GTK 3 applications? If we don't apply it to all applications that use wxGTK, will this PR cause regressions for the applications that used to use GTK 2? It seems infeasible to fix all the applications affected by this PR right now, since there are over 100 of them.

i switched my branch to lopsided's, rebased on nixos-unstable and tried to execute the rebuild packages via nix-review

109 rebuild
aegisub audacity boinc bossa cl couchdb couchdb2 cubicsdr curaByDagoma cura_stable cuter diff-pdf digikam displaycal dvdstyler ejabberd elixir elixir_1_5 elixir_1_6 elixir_1_7 elixir_1_8 erlang erlangR18 erlangR19 erlangR20 erlangR21 erlang_javac erlang_odbc erlang_odbc_javac far2l filezilla fityk flamerobin gnss-sdr gnuradio gr-ais gr-gsm gr-limesdr gr-nacl gr-osmosdr gr-rds gnuradio-with-packages golly gqrx grass hugin inspectrum kicad kicad-with-packages3d lenmus limesuite loxodo lutris lutris-free mercury metamorphose2 mmex notmuch-bower odamex opencpn perl528Packages.AlienWxWidgets perl528Packages.Wx perl528Packages.WxGLCanvas perl530Packages.AlienWxWidgets perl530Packages.Wx perl530Packages.WxGLCanvas playonlinux plover.stable poedit printrun prusa-slicer pwsafe python27Packages.pyspread python27Packages.robotframework-ride python27Packages.runsnakerun python27Packages.wxPython qradiolink rabbitmq-server radiotray-ng rebar rebar3 relxExe rtl_433 saga scyther sdrangel slade slic3r soapysdr-with-plugins torchat treesheets truecrypt tsung urh welle-io wings winpdb winusb wxGTK30 wxGTK31 wxSVG wxhexeditor wxmaxima wxsqlite3 wxsqliteplus xchm xylib yaws zeroad

28 with file chooser / GSettings-schema crash
aegisub audacity bossa displaycal dvdstyler filezilla fityk golly hugin kicad lenmus limesuite loxodo metamorphose2 mmex opencpn plover pwsafe python27Packages.runsnakerun saga scyther slic3r treesheets winpdb wxmaxima xchm xylib zeroad

4 with the gtk2 file chooser?
digikam qradiolink sdrangel urh

5 fail with "no module"
no module named wx:
curaByDagorna playonlinux torchat
no module named gtk:
gnuradio-companion
no module named _multiarray_umath:
cura

5 seem to work
far2l lutris poedit slade wxsqliteplus

3 seem to work due to not using/needing a file chooser
boinc welle-io radiotray-ng

3 seem messed up without this PR as well
notmuch-bower odamex flamerobin

4 unchanged build failures unrelated to this PR
electricsheep pcsx2 salut_a_toi sasview

the rest i couldn't test on their own
couchdb couchdb2 cubicsdr cuter diff-pdf ejabberd elixir* erlang* gnss-sdr gr-* gqrx grass inspectrum mercury perl5* printrun prusa-slicer python27* rabbitmq-server rebar rebar3 relxExe rtl_433 soapysdr-with-plugins truecrypt tsung wings winusb wxGTK3* wxSVG wxhexeditor wxsqlite3 yaws

@worldofpeace since most of the packages using wxGTK30 with gtk3 seem to be affected, is there a more systemic way to treat this rather than to patch each of them?

when you get
No GSettings schemas are installed on the system
see https://nixos.org/nixpkgs/manual/#ssec-gnome-common-issues and the rest of the GNOME docs.
It needs wrapGAppsHook in nativeBuildInputs and some other package that provides the gsettings schema. I'm guessing it's probably the filechooser so you'll probably want wxGTK.gtk.
@lopsided98 The gtk2 version of the apps maybe used GConf, so yes, doing I've instructed in the previous comment is the only way to land this PR without regressions.
It also seems @Evils-Devils has done a great job of testing it as well.

is there a more systemic way to treat this rather than to patch each of them

No, that's simply how packaging gtk3 apps works in nixpkgs. It all revolves around that setup hook.

https://github.com/NixOS/nixpkgs/pull/86217 moves filezilla to use gtk3 (and thus wxgtk with gtk3 by extension).

wxwidgets 3.0.5 is out as of y'day which also updates gstreamer from 0.10 to 1.0 which would be great move.

@lopsided98 I know that this PR is probably very tiring as a lot is going on the master branch. If you want to break this apart into smaller PRs, it will also make them easier to merge as there's less surface area to test

Yeah, I think that is the only feasible way to move forward with this. We can just make individual packages use GTK3, and then someday go back and change the default.
I don't have a lot of time to work on this right now, but if others want to take commits from this PR and make separate PRs, feel free to go ahead.

How about adding -gtk2 and -gtk3 variants to all-packages.nix and have the main wxgtk just be an alias to -gtk2 for now. This makes the transition very clear.


Usually exporters can generate more traces than the real traffic.

Every time when we upload to SD (for example) we start a new trace in the gRPC client, which uses the default sampler (and sometimes for low traffic services 1qps for example) using the default sampler as always sample we have a lot of traces for the exporter.
I think we have few options here:

Wrap the "uploadToStackdriver" (or equivalent for other exporters) in a Span that uses "never sample" sampler.
Wrap the "uploadToStackdriver" (or equivalent for other exporters) in a Span that uses "1/1000 sample" sampler (this is implemented in Java).
We can expose the sampler that we use in the option with a default of 1/1000, but I don't think these traces are that useful.
Recommend users to set a larger BundleDelayThreshold (the current default is 2 seconds) and make probably the default 5 or 10 seconds. (We may want to do this anyway)

/cc @groob who reported similar problem on gitter channel.

Took a cursory look and from what I can tell this should only happen if you explicitly pass ocgrpc.ClientHandler to stackdriver.NewExporter (via ClientOptions). Is that what's happening here?

Confirmed by running stackdriver.TestExport with a dummy project that at least with a very vanilla configuration this should not happen

Actually, I think what's happening here is that the grpc client used by the Stackdriver exporter is also instrumented because of the instrumentation of google.golang.org/api/transport/grpc.


[Feature Requests and Questions]About Already Downloaded

Is there a database function that manages the downloaded list so that when downloads are automatically run in a batch file, posts that have already been downloaded are not downloaded again? If not, can it be added?
Is it possible to add a function to notify users by email or other means when a download fails (authentication error)?
I need this to create a system that automatically saves the posts of my favorite artists.

Is there a database function that manages the downloaded list so that when downloads are automatically run in a batch file, posts that have already been downloaded are not downloaded again? If not, can it be added?

Yes, this is the default behaviour. The --all flag also ensures that the program doesn't terminate as soon as it encounters a file which is already downloaded. It doesn't actually download everything again.

Is it possible to add a function to notify users by email or other means when a download fails (authentication error)?

It's possible to script the sending of emails on command-line on Linux or Windows, so couldn't you integrate one of those tools into a script? Although, fanbox-dl currently returns the usual code 0 when it exits with errors logged, so it should really return a non-zero code in that case. I will raise a PR for that.

Thanks, I thought the --all flag was supposed to re-download everything every time. Now I can use the --all flag without any worries!
I have created a script in ChatGPT that sends an email with the error details when an error occurs in a Python script.
This is not based on the error code, but rather the console output at download time is logged to a log file, and if the log file contains an ERROR after execution, the line is emailed to the user.
Translated with DeepL.com (free version)

Next version exits with code 1 when the command failed.


[ICONS] vol-2 additianal icons

Signed-off-by: Shijir Tsogoo<EMAIL_ADDRESS>

@Shijir is this meant to close #378 ? Could you make sure to include the "Fixes #..." to make sure. Also, could you mark #378 with 0.8.4 if this is going to make it into 0.8.4?


Adding APIs for the new resource - Network Security Perimeter (NSP)

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[X] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[x] No, no need to refresh for updates in this PR

Contribution checklist:

[X] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[X] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[X] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@harics24, could you take a look at the tool check error?  seems you missed  the following endpoint:
The private endpoint API: /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/privateLinkServices/{serviceName}/privateLinkResources is missing.
Location: Microsoft.Network/stable/2020-11-01/webapplicationfirewall.json#L36

@kairu-ms , @lirenhe ,
I have now added the changes in new version.
Addressed some of the lint exceptions.
I'm not sure how to fix the remaining. I did not make any changes for Go SDK, still it's showing validation exceptions. Please help.

The error is pointing to this file:
The private endpoint API: /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/privateLinkServices/{serviceName}/privateLinkResources is missing.
Location: Microsoft.Network/stable/2021-02-01/webapplicationfirewall.json#L36
I did not make any changes there. We do not own this service/endpoint.

/azp run

/azp run

All lint errors are not related to this PR, approved for Lint.


Suggestion: having a gitter channel

hi,
some of the javascript  libraries i use have gitter channel, and i saw a great benefit of it as it allowed the community to share experience, have input, bug fixing, etc...
it would be really great for Riotjs to have a gitter channel to engage community and help to increase the adaption and improvement of the library.
thanks in Advanced.

hi,
duplicate of #394 ?
there's a chat room already on muut's own service.
https://muut.com/riotjs/forum/

Riot forums a re cool, but is really different from gitter, with gitter you have better realtime form discussion, you can link github issues, gists, plus the whole experience is different, i hope i am not pushing it, but again i really think gitter is the ultimate choice for now a days developers communication.

(+)

I do also prefer a gitter channel instead of the normal Riot Forum.

+1

On 19 Apr 2015, at 9:21 AM, Andreas Heintze<EMAIL_ADDRESS>wrote:
I do also prefer a gitter channel instead of the normal Riot Forum.
—
Reply to this email directly or view it on GitHub.

@tipiirai has already answered to this question https://github.com/muut/riotjs/issues/394
I respect his opinion and I will close this issue for now. Even if I like gitter as well we have already enough ways to support our users. Maybe in future we could think again about this


Add core.contiguousLength

This PR adds a core.contiguousLength property, stored in the oplog header, that tracks the number of contiguously available blocks starting from the first block of a core.

Excellent! I think there is an edge case when the feed is truncated, can you test that?
We need an additional feature to this. A new flag sparse that defaults to true, that when false makes .length === .contigiousLength and also sets byteLength to the equi. We can look into that in another PR tho.


qq: enable wayland-ime

Description of changes
Enables wayland ime feature of electron on wayland environments for stuff like fcitx5-wayland-frontend.
Things done

Built on platform(s)

[x] 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.11 Release Notes (or backporting 23.11 and 24.05 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.

@j4ger Thanks for your first time contribution. And welcome to nixpkgs.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1933

@NixOS/nixpkgs-merge-bot merge

1 package built:
qq
Result of nixpkgs-review pr 338658 run on x86_64-linux 1

1 package built:

qq


Add auto-update workflow with sync action for forks, and a keepalive action for upstream repo

How does the keep alive empty commit look like? Did you test this PR fully  before submitting?

There are some points to discuss once you have seen how it works. I might make some adjustments after discussing, so no rush in merging this just yet. You can test it in your private fork first if you like (you would have to change the default branch to be able to run the new update.yml workflow), or wait until I have the time to walk you through it.
I may be able to keep everything in the build_iAPS.yml, it depends on how you feel about some consequences of that.
Let me get back to you in a PM when I have the time.

Get back when you have tested it. It's a bad idea submitting before testing.

How does the keep alive empty commit look like? Did you test this PR fully before submitting?

The keep alive commits look like this:
https://github.com/bjornoleh/LoopWorkspace/commit/da41bf548213d18e4389c0a3ce6df83020784fcf

And the sync action will default to a fast forward, if possible:

If there is divergent local history, a merge commit will be inserted. This is general git pull behaviour, as docmented here:
https://git-scm.com/docs/git-pull

I agree with Pete Schwamb's comments/requests. Better to compare with last build date instead of scheduling same build date every month, to only build when necessary. Changing commit nr is also not ideal, since I use this for troubleshooting.

Is this ready to be merged now?

Hi, eventually, we found a way to do this that was acceptable to Loop devs. Some additional refinements have been added, and are about to be wrapped up now (a new Loop release is imminent). Once everything is settled, the current implementation in Loop will be propagated to the other projects. This process has taken forever, but seems to come to an end now.
So just hold on a little longer please :-)
@dnzxy

Thanks for the tag @bjornoleh 😊
We can offer a vastly improved validation workflow for now, that has been merged into LoopCaregiver, will be merged soon into LoopFollow dev (and probably into main, too), and are pending merge to Loop via LoopWorkspace PR86. They further improve what was moved it to Loop with LoopWorkspace PR77. I‘ve told the guys over at xDrip4iOS the same.
The automatic build improvements are almost there, I‘d ask for just a tiny bit more patience for those. We found a great way to do both monthly scheduled builds and builds around changes. I hope they will be to your satisfaction, @Jon-b-m 😊

Thanks a lot @bjornoleh and @dnzxy. Will these new changes work with existing fastlane configurations (forks)?

There should be no breaking changes. There are a couple of more changes coming, hopefully today. Will ping you when it's ready for merging.

This should not bring any breaking changes to existing builds. Everything is build on the iAPS user having workflow permissions added to their GH_PAT. If they haven’t, no automation is enabled and manual build works as they are used to (with many added validations with more descriptive errors).
There are some minor changes that should make their way in probably today and I think Bjørn will give his final go when it’s ready to merge.

Will merge as soon as you think it's ready

This PR should be good to go now. This brings the workflows, Fastfile and also Fastlane version and other dependencies as specified in Gemfile.lock in line with the state of things in Loop dev.
The users fork of iAPS will be synced with Artificial-Pancreas/iAPS before building if the user sets a 'SCHEDULED_SYNC' repository variable to 'true'.


docs(samples): fix NPE in DeleteMetricDescriptorIT#testDeleteMetricDe…

Fixes #332

The root cause of failure is some tests alter System.out and restore it incorrectly.


selectDate() does not update UI

I'm using SingeDateAndTimePicker in a project. When I call  picker.selectDate(date) method, picker.getDate() method returns correct value but UI does not change.
Am I doing something wrong or is this behavior unexpected?

I've experienced the same issue, it does seem like unexpected behaviour.
However for my scenario I have been able to work around it by also calling picker.setDefaultDate(date), this seems to update the UI. I'm calling this on onViewCreated method of my fragment so I can restore the picker state after a screen rotation change, and it seems to work. I haven't tested this workaround in any other scenario.


External terminal set to Cmder, "Open in Command Prompt" doesn't open in correct directory

VSCode Version: 1.6.1
OS Version: Win7 x64

Steps to Reproduce:

Added system environment path to my Cmder install
In settings.json I have "terminal.external.windowsExec": "Cmder.exe"
When I right click on a folder in the Explorer and select "Open in Command Prompt" it launches Cmder, but doesn't auto-navigate me to the directory I just selected on. It opens me at the default Cmder directory instead

I believe the issue with Cmder start directories may be related to the cmdArgs used when spawning the terminal.
Cmder seems to use /start, but line 74 of code only has it as start. Line 50 exhibits the same as well.
Cmder launched like this cmder.exe /start c:/someDir would open C:/someDir.

This is a duplicate of https://github.com/Microsoft/vscode/issues/6466


feat(ui): Notification Management (clicking on Notification)

Description
Implement advanced functionalities for the notifications screen to allow users to interact with, manage, and filter notifications efficiently
Checklist before requesting a review
Issue ticket number and link

[x] This PR has a valid ticket number or issue: DTIS-942

Testing & Validation

[x] This PR has been tested/validated in IOS, Android and browser.
[x] The code has been tested locally with test coverage match expectations.
[x] Added new Unit/Component testing (if relevant).

Security

[x] No secrets are being committed (i.e. credentials, PII)
[x] This PR does not have any significant security implications

Code Review

[x] There is no unused functionality or blocks of commented out code (otherwise, please explain below)
[x] In addition to this PR, all relevant documentation (e.g. Confluence) and architecture diagrams (e.g. Miro) were updated

Design Review

[x] If this PR contains changes to the UI, it has gone through a design review with UX Designer or Product owner.
[x] In case PR contains changes to the UI, add some screenshots to notice the differences

IOS
UI

Function
https://github.com/cardano-foundation/cf-identity-wallet/assets/162310763/ffd2b7ce-9327-47d2-aeba-9f327f10a023
Android
UI

Function
https://github.com/cardano-foundation/cf-identity-wallet/assets/162310763/0689de3a-b36c-449c-9fa8-4fb27f7aa445
Browser
UI

Function
https://github.com/cardano-foundation/cf-identity-wallet/assets/162310763/4462508c-e376-4152-98ac-7a6a5d12716b
Infinite load for earlier notification
https://github.com/cardano-foundation/cf-identity-wallet/assets/162310763/8edfd042-3706-48b2-9771-7e72864dd466

Added a few comments, also please make sure the NotificationDetails (as a page) and the modal are both updated to be compatible with back hardware button on Android.
Update evidence
https://github.com/user-attachments/assets/a6ce8b2a-d762-444c-82b3-f152f22a80fe


change the "toggletextures" response text

the toggletextures text is still a bit... iffy. the reconnecting part needs to be reworded / removed and the "now no longer excluded" needs to be changed

IDEA:
Toggle On: "Server Textures will now change per region"
Toggle Off: "Now excluding player from all Texture Pack regional changes"

implemented


Makefile: add get dataset lavel in makefile

get mnist from http://yann.lecun.com/exdb/mnist in makefile

nice


GCC 5 warning printed for all GCC versions

The root CMakeLists.txt contains this line:
https://github.com/VcDevel/Vc/blob/master/CMakeLists.txt#L80
      if(Vc_GCC_VERSION VERSION_GREATER "5.0.0" OR Vc_GCC_VERSION VERSION_LESS "5.3.0")

The condition is always true. You probably meant to use AND instead of OR. For example, if I compile manually with GCC 4.9:
$ CXX=gcc-4.9 CC=gcc-4.9 cmake .. -DBUILD_TESTING=OFF
-- The C compiler identification is GNU 4.9.3
-- The CXX compiler identification is GNU 4.9.3
[...]
CMake Warning at cmake/UserWarning.cmake:4 (message):
  GCC 5 goes into an endless loop comiling example_scaling_scalar.
  Therefore, this target is disabled.
Call Stack (most recent call first):
  CMakeLists.txt:81 (UserWarning)

Ouch thanks, I always wondered about the message but never bothered to look into it. Will fix.


Fix #411: Tests if event is thrown when calling rename

Could not make the change event to throw rename during a rename call.
Test will check if change event is thrown when a file is renamed.

@AHKol I'm having another look at this and I think I initially misunderstood the filer implementation. Did you see @humphd's comment about the lack of rename events in filer? On NodeJS, rename events are emitted when a file is renamed, so this test should actually be failing.

@humphd Added the requested comment.