[bug]: SheetResponse type

Describe the bug
I would like to type the response of SheetResponse & DialogResponse. I am aware that SheetResponse accepts a  generic which can be anything you want, but when typing inside the generated bottomsheet eg. SheetResponse to SheetResponse the application complains that SheetResponse is not of type SheetResponse, Which is a valid error, but is there a way to type the response (and maybe even the request)?
I am sure there is a way is t was available in previous versions of Stacked (before app.bottomsheet.dart), but i am unable to get the types in the current version.
NOTE: I did run build_runner after updating the type but in the app.bottomsheet.dart it still shows up as 'dynamic'
To reproduce
Nothing
Expected behavior
Abilty to type SheetResponses
Screenshots
No response
Additional Context
No response

Hi @shayant98 , you can set the type of SheetResponse when you define the completer in your Sheet class like below.
class ShayantSheet extends StatelessWidget {
  final Function(SheetResponse<int> response)? completer;
  final SheetRequest request;
  const ShayantSheet({
    Key? key,
    required this.completer,
    required this.request,
  }) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Container(
      padding: const EdgeInsets.symmetric(horizontal: 20, vertical: 15),
      decoration: const BoxDecoration(
        color: Colors.white,
        borderRadius: BorderRadius.only(
          topLeft: Radius.circular(10),
          topRight: Radius.circular(10),
        ),
      ),
      child: Column(
        crossAxisAlignment: CrossAxisAlignment.start,
        mainAxisSize: MainAxisSize.min,
        children: [
          Text(
            request.title ?? 'Hello Stacked Sheet!!',
            style: const TextStyle(fontSize: 25, fontWeight: FontWeight.w900),
          ),
          if (request.description != null) ...[
            verticalSpaceTiny,
            Text(
              request.description!,
              style: const TextStyle(fontSize: 14, color: kcMediumGrey),
              maxLines: 3,
              softWrap: true,
            ),
          ],
          verticalSpaceMedium,
          MaterialButton(
            onPressed: completer?.call(SheetResponse(data: 666)),
            child: const Text('test'),
          ),
          verticalSpaceLarge,
        ],
      ),
    );
  }
}

Let me know if that is what you wanted to achieve.

Hey @ferrarafer
This js great!
But what i also wanted to know was how to type the SheetRequest currently the data object is typed as Dynamic but i want to supply my own Map.

Hey @shayant98 SheetRequest is generic, you can make your own. Might not be supported with the generator This is how to use it:

Create your class that you want to use as your request

class TimeRangeSelectionRequest {
  final List<Event> events;
  final int day;

  TimeRangeSelectionRequest({required this.events, required this.day});
}

In your bottom sheet accept the request

class TimeRangeSelectionBottomSheet extends StatelessWidget {
  final SheetRequest<TimeRangeSelectionRequest> request;
  final Function(SheetResponse)? completer;

  TimeRangeSelectionBottomSheet({
    Key? key,
    required this.request,
    this.completer,
  }) : super(key: key);

  ....

}

Your bottom sheet setup can then use the following code to register

  BottomSheetType.timeRangeSelection: (
            context,
            SheetRequest<dynamic> sheetRequest,
            Function(SheetResponse<dynamic>) completer) =>
        TimeRangeSelectionBottomSheet(
          request: sheetRequest as SheetRequest<TimeRangeSelectionRequest>,
          completer: completer,
        ),

@ferrarafer we can use the same pattern above and generate the registration accordingly through the generator.

Any idea if this will get implemented?
This will make many codebases more typesafe and reduce the need for infering types.

@shayant98 I'd suggest creating a registerBottomSheetsFunction where you can register custom response types like above yourself.
Then you keep the generic ones in the bottom sheet generator.
We don't have a timeline for implementing this.
Happy to accept and review PR's for this functionality.


feat/types

fix for https://github.com/sanniassin/react-input-mask/issues/239#issuecomment-1658528006
add types
update dependencies
run prettier

Hi, I am getting an error after the last update, no lib folder in node modules

@milkyfuckinway When do you get that? When you run npm install or at some other point?

@milkyfuckinway When do you get that? When you run npm install or at some other point?

When i run "npm run dev" with any InputMask component in project

@milkyfuckinway I've pushed what should be a fix - v3.0.3

@milkyfuckinway I've pushed what should be a fix - v3.0.3

Yeah, everything's fine now, thanks.

Thanks for letting me know :+1:


Need to run git tagging in CI action - permission denied

I try to tag the revision whose SoT is in the CMake file on push whenever it changes.
However, it looks like we need some additional permissions in the github action.
https://github.com/chipsalliance/UHDM/actions/runs/4495222394/jobs/7908625709
I don't have access to the 'settings' tab in UHDM, which I suspect I need to find the right settings.
Tagging a few people who might know how to achieve this
@mithro @kgugala @umarcor

#889 attempts to fix this

Looks like we need a bot account; @kgugala fills in details.
For now, I've just manually tagged the release, so that we can explore that permission issue separately.


Improved travis configuration

Removed unmaintained Symfony 2.4
Added Symfony 2.5
Added test for lowest dependencies
Removed not needed shell wrapper
Removed not needed --dev flag
Used Travis's new docker based env
Cached composer's deps

I'm happy with this changeset for now, so I'm merging it. Thanks @WouterJ ! Keep up the great work.


Support DateTime<Utc> & DateTime<Local>

PR Info

Continue of #429
Resolve #381
Depends on SeaQL/sea-query#249

Added

supports saving & querying DateTimeUtc
supports saving & querying DateTimeLocal
set default value of ColumnDef
set default value of each column when generating create table statement with Schema::create_table_from_entity method

Seems Charles's code is included. Does that mean we can close 429?

Seems Charles's code is included. Does that mean we can close 429?

Correct!


Add no_proxy env for embedding-tei-server container

Description
Add no_proxy env for embedding-tei-server container
Issues
In a proxy r&d or deployment environment ( need proxy to access external network, no proxy to access internal network)
we need to pass no_proxy setting to embedding-tei-server, otherwise by following https://github.com/opea-project/GenAIExamples/blob/main/ChatQnA/docker/xeon/README.md
"TEI embedding service" access is OK, but inside "Embedding Microservice", it failed to access "Embedding Microservice".
some discussion in https://github.com/opea-project/GenAIExamples/issues/251
TEI Embedding Service
curl ${host_ip}:6006/embed
-X POST
-d '{"inputs":"What is Deep Learning?"}'
-H 'Content-Type: application/json'
Embedding Microservice
curl http://${host_ip}:6000/v1/embeddings
-X POST
-d '{"text":"hello"}'
-H 'Content-Type: application/json'
Type of change
List the type of change like below. Please delete options that are not relevant.

[x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)

Dependencies
this PR itself has no dependency, but other PR will depend on it,   I need to pass the "no_proxy" env in GenAIExample repo docker compose yaml and update readme accordingly
Tests
[Describe the tests that you ran to verify your changes.]
Follow (https://github.com/opea-project/GenAIExamples/blob/main/ChatQnA/docker/xeon/README.md)  to  set up ChatQnA in a proxy  environment
without this patch and  the patch in GenAIExample(PR number will be provided later）
TEI Embedding Service    ok
curl ${host_ip}:6006/embed
-X POST
-d '{"inputs":"What is Deep Learning?"}'
-H 'Content-Type: application/json'
Embedding Microservice    ko
curl http://${host_ip}:6000/v1/embeddings
-X POST
-d '{"text":"hello"}'
-H 'Content-Type: application/json'

After discussion with @huiyan2021   rerank container has the same issue, add no_proxy for both embedding and rerank

@moting9，
please also help to add no_proxy env for other microservices? thanks.

@moting9，
please also help to add no_proxy env for other microservices? thanks.

Hello, @lvliang-intel,  changes in GenAIComps docker_compose.yaml   has no impact on GenAIExamples execution  because each GenAIExample has its own docker_compose.yaml,    no_proxy here is NOT necessary change for GenAIExample execution.     To keep alignment, I will try to add no_proxy along with http/https proxy setting in GenAIComps.

@moting9，
please also help to add no_proxy env for other microservices? thanks.

@lvliang-intel    Done.  Please review. Thanks!


c7n-s3-encrypt not handling spaces in key path

If a space character is in an object key path it is being translated to a "+" when it is passed to the Lambda and it not being handled correctly.  For example, an object (template) named "2017235Abc-Create one dev rds instance.json" gets passed to the function as "2017235Abc-Create+one+dev+rds+instance.json", and the function throws an object not found error.
While I understand that a space character is not in the list of "safe" characters given by AWS for object names, it is in the "Characters That Might Require Special Handling" section of the S3 documentation.
(NOTE: Issue updated to no longer mention colons, as those are correctly handled in the current version of the function.)

I'm surprised these are getting encoded as a + instead of %20

@alfredgamulo If you use one of the lambda templates to create an S3 event handler for the bucket the generated code uses the urllib.unquote_plus method to convert the key name to a form suitable for passing back to boto.
As a test I created a simple lambda and uploaded a file with spaces in the name File - Copy.pdf:
def lambda_handler(event, context):
    bucket = event['Records'][0]['s3']['bucket']['name']
    raw_key = event['Records'][0]['s3']['object']['key']
    raw_key_utf8 = raw_key.encode('utf8')
    key_unquote = urllib.unquote(raw_key_utf8)
    key_unquote_plus = urllib.unquote_plus(raw_key_utf8)
    print('********')
    print('Bucket: %s' % (bucket))
    print('Raw key: %s' % (raw_key))
    print('Raw key (UTF-8): %s' % (raw_key_utf8))
    print('Unquoted: %s' % (key_unquote))
    print('Unquoted plus: %s' % (key_unquote_plus))
    print('********')
    # s3.get_object(Bucket=bucket, Key=key)

********
Bucket: kit-test-bucket
Raw key: File+-+Copy.pdf
Raw key (UTF-8): File+-+Copy.pdf
Unquoted: File+-+Copy.pdf
Unquoted plus: File - Copy.pdf
********

I'll modify ufuncs/s3crypt.py to use this functionality.
Thanks @tim-elliott for bringing this to our attention.


IOS keyboard auto-correction not working

Describe the bug
When using iOS, the keyboard auto-correction should work.however it doesn't
To Reproduce
Steps to reproduce the behavior:

Open the demo with iOS Chrome or Safari
Using any keyboard to type some sentences

Expected behavior
Auto correction should work. The keyboard should prompt some words and auto correct typed words
Screenshots

Expected:

Version information

OS: [ios
Browser: chrome, safari
EasyMDE version: latest

Additional context
Add any other context about the problem here.

This has been reported before: https://github.com/Ionaru/easy-markdown-editor/issues/23, https://github.com/Ionaru/easy-markdown-editor/issues/113, https://github.com/Ionaru/easy-markdown-editor/issues/324 but unfortunately the error lies deep within CodeMirror and there is not a lot EasyMDE can do to fix it.
This can only be resolved by a big rewrite surrounding Codemirror 6.


[SPARK-42555][CONNECT] Add JDBC to DataFrameReader

What changes were proposed in this pull request?
Currently, the connect project have the new DataFrameReader API which is corresponding to Spark DataFrameReader API. But the connect DataFrameReader missing the jdbc API.
Why are the changes needed?
This PR try to add JDBC to DataFrameReader
Does this PR introduce any user-facing change?
'No'.
New feature.
How was this patch tested?
N/A

The jdbc API seems hard to test, do we need a test case? @hvanhovell @HyukjinKwon @zhengruifeng @dongjoon-hyun

There is another kind jdbc API, see: https://github.com/apache/spark/blob/79da1ab400f25dbceec45e107e5366d084138fa8/sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala#L316
I will create another PR to add proto msg and implement it.

I guess you can refer to JDBCSuite and ClientE2ETestSuite ?

@beliefer you can create a test in PlanGenerationTestSuite. That will at least validate the proto message we are generating, and it will validate that plan you are producing yields a valid plan in ProtoToPlanTestSuite.

I guess you can refer to JDBCSuite and ClientE2ETestSuite ?

The built-in H2 running in server side and we can't start H2 database at connect API.

@beliefer you can create a test in PlanGenerationTestSuite. That will at least validate the proto message we are generating, and it will validate that plan you are producing yields a valid plan in ProtoToPlanTestSuite.

OK. I added the test cases. But throws
org.h2.jdbc.JdbcSQLSyntaxErrorException: Schema "TEST" not found; SQL statement:
SELECT * FROM TEST.TIMETYPES WHERE 1=0 [90079-214]

when ProtoToParsedPlanTestSuite validating the golden file. The built-in H2 running in server side and we can't start H2 database at connect API.
@hvanhovell Could you tell me how to fix or avoid it ?

@dongjoon-hyun @hvanhovell It seems the build scala 2.13 failed is unrelated to this PR.

merging to master/3.4

@hvanhovell @dongjoon-hyun @LuciferYang Thank you!


impellerc can generate a vertex shader with no PerVertexData

shader:
// Copyright 2013 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

uniform FrameInfo {
  mat4 mvp;
  vec2 atlas_size;
  vec4 text_color;
} frame_info;

in vec2 unit_vertex;
in mat4 glyph_position;
in vec2 glyph_size;
in vec2 atlas_position;
in vec2 atlas_glyph_size;

out vec2 v_unit_vertex;
out vec2 v_atlas_position;
out vec2 v_atlas_glyph_size;
out vec2 v_atlas_size;
out vec4 v_text_color;

void main() {
  gl_Position = frame_info.mvp
              * glyph_position
              * vec4(unit_vertex.x * glyph_size.x,
                     unit_vertex.y * glyph_size.y, 0.0, 1.0);

  v_unit_vertex = unit_vertex;
  v_atlas_position = atlas_position;
  v_atlas_glyph_size = atlas_glyph_size;
  v_atlas_size = frame_info.atlas_size;
  v_text_color = frame_info.text_color;
}

If you move the mat4 in param to the end, it compiles. @chinmaygarde says he knows what's up.

Even with the workaround, it still generates the wrong types - Padding<64> instead of Matrix for that matrix param.


(node/lsstcam-shutter01.cp) add ccs shutter service

Will replace this pull with a more general one.


refactor(batch): use wrapper executor to impl time and i32 generate series

What's changed and what's your intention?
use wrapper executor to implment time and i32 generate series
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Summarize your change (mandatory)
How does this PR work? Need a brief introduction for the changed logic (optional)
Describe clearly one logical change and avoid lazy messages (optional)
Describe any limitations of the current code (optional)

Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)
close https://github.com/singularity-data/risingwave/issues/2443

ready for review @TennyZhuang @xiangjinwu :relaxed:


NAV Container for older version (NAV 2015 and less)

Hi,
I tired the example of freedy to use docker also with NAV2015 and NAV 2013R2.
Here my PowerShell Code:
New-NavContainer `
    -containerName test2015de `
    -accept_eula `
    -alwaysPull `
    -auth NavUSerPassword `
    -licenseFile "C:\Users\enjoyAdmin\Downloads\ej2018projekt 2019-02-16.flf" `
    -navDvdPath "C:\nav2015CU06\NAV.8.0.40459.DE.DVD" `
    -navDvdCountry "DE" `
    -isolation process `
    -donotExportObjectsToText `
    -restart on-failure `
    -shortcuts CommonStartMenu `
    -updateHosts `
    -useBestContainerOS `
    -useSSL -accept_outdated

It starts to create the Image and everything looks fine. But in the end the nav Service Tier isn't running and I get following log:
New-NavContainer `
    -containerName test2015de `
    -accept_eula `
    -alwaysPull `
    -auth NavUSerPassword `
    -licenseFile "C:\Users\enjoyAdmin\Downloads\ej2018projekt 2019-02-16.flf" `
    -navDvdPath "C:\nav2015CU06\NAV.8.0.40459.DE.DVD" `
    -navDvdCountry "DE" `
    -isolation process `
    -donotExportObjectsToText `
    -restart on-failure `
    -shortcuts CommonStartMenu `
    -updateHosts `
    -useBestContainerOS `
    -useSSL -accept_outdated
NavContainerHelper is version <IP_ADDRESS>
Host is Microsoft Windows Server 2019 Datacenter - ltsc2019
Docker Client Version is 18.09.6
Docker Server Version is 18.09.6
Pulling image mcr.microsoft.com/dynamicsnav:generic-ltsc2019
generic-ltsc2019: Pulling from dynamicsnav
Using image mcr.microsoft.com/dynamicsnav:generic-ltsc2019
Creating Container test2015de
Using license file C:\Users\enjoyAdmin\Downloads\ej2018projekt 2019-02-16.flf
Version: 8.0.40459.0-DE
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.805 (ltsc2019)
Host OS Version: 10.0.17763.864 (ltsc2019)
Using process isolation
Creating container test2015de from image mcr.microsoft.com/dynamicsnav:generic-ltsc2019
711247db245680256a6b1a09c795a836a208dd25cd516e96c402ec6e4d455dad
Waiting for container test2015de to be ready
Installing NAV
Installing .NET 3
Starting Local SQL Server
WARNING: Waiting for service 'SQL Server (SQLEXPRESS) (MSSQL$SQLEXPRESS)' to
start...
WARNING: Waiting for service 'SQL Server (SQLEXPRESS) (MSSQL$SQLEXPRESS)' to
start...
Starting Internet Information Server
Installing Url Rewrite
Installing Report Viewer
Installing OpenXML
Copying Service Tier Files
Copying Web Client Files
Copying Windows Client Files
Copying PowerShell Scripts
Copying UpgradeToolKit
Copying ClientUserSettings
Copying ReportBuilder
Restoring CRONUS Demo Database
Changing Database Server Collation to Latin1_General_100_CS_AS
SQL Server 2017 transmits information about your installation experience, as well as other usage and performance data, to Microsoft to help improve the pr
oduct. To learn more about SQL Server 2017 data processing and privacy controls, please see the Privacy Statement.
Installing C:\NAVDVD\Installers\DE\OlAddin\OutlookAddIn.Local.De.msi
Installing C:\NAVDVD\Installers\DE\RTC\RoleTailoredClient.Local.De.msi
Installing C:\NAVDVD\Installers\DE\Server\Server.Local.De.msi
Installing C:\NAVDVD\Installers\DE\WebClient\WebClient.Local.De.msi
Skipping C:\NAVDVD\Installers\DE\WebHelp\WebHelp.Local.De.msi
Modifying NAV Service Tier Config File for Docker
Creating NAV Service Tier
Starting NAV Service Tier
Importing CRONUS license file
The Microsoft Dynamics NAV server is currently not ready to serve requests. Try again later or contact your system administrator.
at <ScriptBlock>, C:\Run\navinstall.ps1: line 188
at <ScriptBlock>, C:\Run\start.ps1: line 114
at <ScriptBlock>, <No file>: line 1Error
Installing NAV
Installing .NET 3
Starting Local SQL Server
WARNING: Waiting for service 'SQL Server (SQLEXPRESS) (MSSQL$SQLEXPRESS)' to
start...
WARNING: Waiting for service 'SQL Server (SQLEXPRESS) (MSSQL$SQLEXPRESS)' to
start...
Starting Internet Information Server
Installing Url Rewrite
Installing Report Viewer
Installing OpenXML
Copying Service Tier Files
Copying Web Client Files
Copying Windows Client Files
Copying PowerShell Scripts
Copying UpgradeToolKit
Copying ClientUserSettings
Copying ReportBuilder
Restoring CRONUS Demo Database
Changing Database Server Collation to Latin1_General_100_CS_AS
SQL Server 2017 transmits information about your installation experience, as well as other usage and performance data, to Microsoft to help improve the pr
oduct. To learn more about SQL Server 2017 data processing and privacy controls, please see the Privacy Statement.
Installing C:\NAVDVD\Installers\DE\OlAddin\OutlookAddIn.Local.De.msi
Installing C:\NAVDVD\Installers\DE\RTC\RoleTailoredClient.Local.De.msi
Installing C:\NAVDVD\Installers\DE\Server\Server.Local.De.msi
Installing C:\NAVDVD\Installers\DE\WebClient\WebClient.Local.De.msi
Skipping C:\NAVDVD\Installers\DE\WebHelp\WebHelp.Local.De.msi
Modifying NAV Service Tier Config File for Docker
Creating NAV Service Tier
Starting NAV Service Tier
Importing CRONUS license file
The Microsoft Dynamics NAV server is currently not ready to serve requests. Try again later or contact your system administrator.
at <ScriptBlock>, C:\Run\navinstall.ps1: line 188
at <ScriptBlock>, C:\Run\start.ps1: line 114
at <ScriptBlock>, <No file>: line 1
Initialization of container test2015de failed
In C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\ContainerHandling\Wait-NavContainerReady.ps1:45 Zeichen:17
+ ...             throw "Initialization of container $containerName failed" ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Initialization ...st2015de failed:String) [], RuntimeException
    + FullyQualifiedErrorId : Initialization of container test2015de failed

Have you an idea why this happend and how it could be solved.
Thanks!

Just try adding one parameter to see it it fixes it
-memoryLimit "3G"

Sorry!
Both doesn't work. Error still on the same line.
New-NavContainer `
    -containerName test2015de `
    -accept_eula `
    -alwaysPull `
    -auth NavUSerPassword `
    -licenseFile "C:\Users\enjoyAdmin\Downloads\ej2018projekt 2019-02-16.flf" `
    -navDvdPath "C:\nav2015CU06\NAV.8.0.40459.DE.DVD" `
    -navDvdCountry "DE" `
    -isolation hyperV `
    -donotExportObjectsToText `
    -restart on-failure `
    -shortcuts CommonStartMenu `
    -updateHosts `
    -useBestContainerOS `
    -memoryLimit "3G" `
    -useSSL -accept_outdated

NavContainerHelper is version <IP_ADDRESS>
Host is Microsoft Windows Server 2019 Datacenter - ltsc2019
Docker Client Version is 18.09.6
Docker Server Version is 18.09.6
Pulling image mcr.microsoft.com/dynamicsnav:generic-ltsc2019
generic-ltsc2019: Pulling from dynamicsnav
Digest: sha256:781161833a184e34820cb38aa58b575bbd7592a745b8910628404c21b01b8537
Using image mcr.microsoft.com/dynamicsnav:generic-ltsc2019
Removing container test2015de
Removing test2015de from hosts
Removing C:\ProgramData\NavContainerHelper\Extensions\test2015de
Creating Container test2015de
Using license file C:\Users\enjoyAdmin\Downloads\ej2018projekt 2019-02-16.flf
Version: 8.0.40459.0-DE
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.805 (ltsc2019)
Host OS Version: 10.0.17763.864 (ltsc2019)
Using hyperV isolation
Creating container test2015de from image mcr.microsoft.com/dynamicsnav:generic-ltsc2019
c95c351be6a3ca0caa93057965ee54a3cece1c236eee09644de2a6da9f6b32e8
Waiting for container test2015de to be ready
Installing NAV
Installing .NET 3
Starting Local SQL Server
WARNING: Waiting for service 'SQL Server (SQLEXPRESS) (MSSQL$SQLEXPRESS)' to
start...
Starting Internet Information Server
Installing Url Rewrite
Installing Report Viewer
Installing OpenXML
Copying Service Tier Files
Copying Web Client Files
Copying Windows Client Files
Copying PowerShell Scripts
Copying UpgradeToolKit
Copying ClientUserSettings
Copying ReportBuilder
Restoring CRONUS Demo Database
Changing Database Server Collation to Latin1_General_100_CS_AS
SQL Server 2017 transmits information about your installation experience, as well as other usage and performance data, to Microsoft to help improve the pr
oduct. To learn more about SQL Server 2017 data processing and privacy controls, please see the Privacy Statement.
Installing C:\NAVDVD\Installers\DE\OlAddin\OutlookAddIn.Local.De.msi
Installing C:\NAVDVD\Installers\DE\RTC\RoleTailoredClient.Local.De.msi
Installing C:\NAVDVD\Installers\DE\Server\Server.Local.De.msi
Installing C:\NAVDVD\Installers\DE\WebClient\WebClient.Local.De.msi
Skipping C:\NAVDVD\Installers\DE\WebHelp\WebHelp.Local.De.msi
Modifying NAV Service Tier Config File for Docker
Creating NAV Service Tier
Starting NAV Service Tier
Importing CRONUS license file
The Microsoft Dynamics NAV server is currently not ready to serve requests. Try again later or contact your system administrator.
at <ScriptBlock>, C:\Run\navinstall.ps1: line 188
at <ScriptBlock>, C:\Run\start.ps1: line 114
at <ScriptBlock>, <No file>: line 1
Starting EventLog Monitor
Monitoring EventSources from EventLog[Application]:
- MicrosoftDynamicsNavServer$NAV
- MSSQL$SQLEXPRESS
Error
Installing NAV
Installing .NET 3
Starting Local SQL Server
WARNING: Waiting for service 'SQL Server (SQLEXPRESS) (MSSQL$SQLEXPRESS)' to
start...
Starting Internet Information Server
Installing Url Rewrite
Installing Report Viewer
Installing OpenXML
Copying Service Tier Files
Copying Web Client Files
Copying Windows Client Files
Copying PowerShell Scripts
Copying UpgradeToolKit
Copying ClientUserSettings
Copying ReportBuilder
Restoring CRONUS Demo Database
Changing Database Server Collation to Latin1_General_100_CS_AS
SQL Server 2017 transmits information about your installation experience, as well as other usage and performance data, to Microsoft to help improve the pr
oduct. To learn more about SQL Server 2017 data processing and privacy controls, please see the Privacy Statement.
Installing C:\NAVDVD\Installers\DE\OlAddin\OutlookAddIn.Local.De.msi
Installing C:\NAVDVD\Installers\DE\RTC\RoleTailoredClient.Local.De.msi
Installing C:\NAVDVD\Installers\DE\Server\Server.Local.De.msi
Installing C:\NAVDVD\Installers\DE\WebClient\WebClient.Local.De.msi
Skipping C:\NAVDVD\Installers\DE\WebHelp\WebHelp.Local.De.msi
Modifying NAV Service Tier Config File for Docker
Creating NAV Service Tier
Starting NAV Service Tier
Importing CRONUS license file
The Microsoft Dynamics NAV server is currently not ready to serve requests. Try again later or contact your system administrator.
at <ScriptBlock>, C:\Run\navinstall.ps1: line 188
at <ScriptBlock>, C:\Run\start.ps1: line 114
at <ScriptBlock>, <No file>: line 1
Starting EventLog Monitor
Monitoring EventSources from EventLog[Application]:
- MicrosoftDynamicsNavServer$NAV
- MSSQL$SQLEXPRESS

Initialization of container test2015de failed
In C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\ContainerHandling\Wait-NavContainerReady.ps1:45 Zeichen:17
+ ...             throw "Initialization of container $containerName failed" ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Initialization ...st2015de failed:String) [], RuntimeException
    + FullyQualifiedErrorId : Initialization of container test2015de failed

Get-NavContainerEventlog  will download the eventlog to the host and open it. Does that reveal anything?

Your problem is probably this one: https://github.com/microsoft/nav-docker/issues/407

Hi Freddy,
Thank you for very quick answer.
That is the solution! I have to use generic-image of ltsc2016 and navcontainerhelper scripts like you.
Stefan


⚠ Bofrost.de has degraded performance

In 253b34a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1330 ms

Resolved: Bofrost.de performance has improved in 30a6af2 after 6 minutes.


Renamed Job attribute.

Fixes #9 .

Codecov Report

Merging #10 into master will not change coverage.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##           master      #10   +/-   ##
=======================================
  Coverage   45.76%   45.76%
=======================================
  Files           4        4
  Lines         319      319
=======================================
  Hits          146      146
  Misses        173      173

Impacted Files
Coverage Δ

app/publisher.py
26.61% <100%> (ø)
:arrow_up:

app/test/test_ftp.py
71.05% <50%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


fix: Displaying decimal currencies.

The application was panicking when attempting to display currency
amounts where the most significant digit was further than a single
character away from the decimal point's position.

This fixes #138


itag 22 is missing in some videos but savefrom.net gives download links for itag 22

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.10.05. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ ] I've verified and I assure that I'm running youtube-dl 2018.10.05

Before submitting an issue make sure you have:

[ ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones
[ ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.10.05
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
This is a vevo video https://www.youtube.com/watch?v=XCQK6LmhYqc. savefrom.net gives itag 22 to download all the time. But when I try to download it from youtube-dl, it only gives me best format 18. Why is that? am I missing some thing.

here we don't have itag=22 available

savefrom.net let us to download itag=22 (They do not convert )

savefrom.net provided url
https://r3---sn-8qj-i5ozz.googlevideo.com/videoplayback?source=youtube&mt=1539046187&mv=u&pcm2cms=yes&fvip=3&ms=au%2Crdu&ip=<IP_ADDRESS>&lmt=1537538088592722&dur=190.333&itag=22&pl=24&ratebypass=yes&mime=video%2Fmp4&c=WEB&mm=31%2C29&mn=sn-8qj-i5ozz%2Csn-8qj-i5oed&ipbits=0&ei=jf-7W7Bej97hArWajvgL&id=o-AIazzD6oREr6h_LrjRxKK-VUfP47gpSo80hK0qAENNa6&key=yt6&txp=4531332&sparams=dur%2Cei%2Cid%2Cip%2Cipbits%2Citag%2Clmt%2Cmime%2Cmm%2Cmn%2Cms%2Cmv%2Cpcm2cms%2Cpl%2Cratebypass%2Crequiressl%2Csource%2Cexpire&requiressl=yes&expire=1539068909&signature=35ED9CB6D3BB39548BE01800C0D50200B6137600.35AEF34B8BDC5A67A554A2772D2E946BDCF5E91F&video_id=XCQK6LmhYqc&title=Rita+Ora+-+Let+You+Love+Me+[Official+Video]

&itag=22&pl=24
Q1. My quetion is 'how savefrom.net does it?'
Q2. Can youtube-dl get itag=22 for this video?
Q3. Where is the code segment does that magic?

Someone, please answer this question? how to do this?

Ask them how they do this.
22 is not listed by youtube for this video.
See youtube extractor.

I wrote:

Perhaps itag=22 is still there on yt's CDNs, but no longer advertised to browsers/apps,
so it might be that SF.net is force-requesting it?

... FWIW, as of around yesterday, Nov 13th 2018, even the latest version (7.94.2) of the SF.net userscript is UNABLE to detect the standalone 720p MP4 encode (itag=22) on VEVO music clips (like the one in my previous comment); it defaults to detecting the standalone 360p MP4 one (itag=18)  :angry: ...
Either Youtube have managed to beat SF.net in their game, or they genuinely removed the itag=22 encodes from their servers (?); I lean on believing the first scenario, i.e. they simply successfully blocked every access to them 😞 ... Oh well, it was good for the whole month it lasted....


No such file or directory error using "at:" property

Today I tried to use the at property to one of my script, but the result was always:
/bin/bash: line 0: cd: ~/foldername: No such file or directory
So I copied&pasted the example from the README.md file, just to figure out my error, but the result is the same:
import ShellOut

do {
  try shellOut(to: ["mkdir NewFolder", "echo \"Hello again\" > NewFolder/File"])
  let output = try shellOut(to: "cat File", at: "~/NewFolder")
  print(output) // Hello again
}
catch {
  let error = error as! ShellOutError
  print(error.message) // Prints STDERR
  print(error.output) // Prints STDOUT
}

/bin/bash: line 0: cd: ~/NewFolder: No such file or directory.
I'm missing something?
My marathon is update (also ShellOut) and I use it a lot without any others issues :)
Thanks!

@darthpelo
I think you can not use ~.
~ depends on running user.
Or, ShellOut may don't standardize path.
Please try to use an absolute path.

The problem is that ShellOut encloses the path passed using at: in quotes, which causes cd to fail to interpret it. The reason it does so is to support passing a path that contains spaces, but I think a better solution here would just be to escape the spaces.

Thanks @JohnSundell 👍


feat (msw) add withSuffix param

Description
Add a withSuffix param to allow user use typescript-msw link feature without forcing to use it with suffix
Related # (issue)

issue-384

Type of change
Please delete options that are not relevant.

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

Screenshots/Sandbox (if appropriate/relevant):
Adding links to sandbox or providing screenshots can help us understand more about this PR and take
action on it as appropriate
How Has This Been Tested?
Add withSuffix:false in the msw pludin link configuration.
Checklist:

[x] I have followed the
CONTRIBUTING doc and the
style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

can you create a changeset?

Added, thx for remiding

can you create a changeset?

Added, thx for remiding

can you take a look why tests are failing ?

#409

hey @neil585456525 can you try rebasing this PR with main?

@saihaj Done


proposal: _.sum

A summing function seems like a worthy addition to Underscore to me, I know it can be done with reduce but I've found to be the largest need to use reduce.
_.reduce lineItems, (previous, lineItem)->
  previous + lineItem.quantity
, 0

# could be much simpler
_.sum lineItems, (lineItem)-> lineItem.quantity

Here's an implementation:
_.sum = (iteratee, func)->
  _.reduce iteratee, (previous, current)->
    previous + func(current)
  , 0

Duplicate of #460 (see also #1433)

Thanks, can you give an example of how to use _.iteratee here?
I understand the reasoning but disagree, the utility of utility functions get lost if the meaning is abstracted away.  If a theoretical 50% of uses of _.reduce were for sum then why make each person re-invent the wheel and make the implementation less clear?
I'm a little surprised this basic feature isn't available natively in Array.prototype.sum or Math.sum, perhaps along the same lines of reasoning, a for loop of reduce both allow for this.

We can't cover every use case, and we shouldn't attempt to. It's much cleaner, clearer and easier to build these functions yourself using the functional programming building blocks Underscore gives you. You might also want to look into add-on libraries that mixin to Underscore like Underscore-contrib or the mixin catalog.


feature: flash works together with flash.

Did you check the docs?

[X] I have read all the LazyVim docs

Is your feature request related to a problem? Please describe.
I prefer flash.nvim to leap.nvim to search for word, and I like the flit.nvim which is based on leap.nvim. How can make this to plugin works together and take better advantage. I install the leap.nvim and disable this keys manully instead of using the lazyvim extra cofnig. But the flit.nvim doesn't work. If I use the lazyvim extra config, the flit.nvim can work but flahs.nvim can not work. Sounds wired, can you solve this?
Describe the solution you'd like
I think it is better to make flit.nvim and flash.nvim works together.
Describe alternatives you've considered
Flash with single letter can works similiar with flit.nvim
Additional context
Thank you so much for the wonderful work have done!

Hello @meicale,
Leap.nvim and flash.nvim share similar functionality but are different plugins.
Quoting from the docs of flit:

f/F/t/T motions on steroids, building on the Leap interface.

Flash has its own f/F/t/T motions and is unaware of the Leap interface mentioned above.
Flash is authored by Folke, leap by ggandor.
I think your request is specific to those plugins, and not to LazyVim.

@meicale
You could try something like the following
{
    "ggandor/flit.nvim",
    dependencies = {
      "ggandor/leap.nvim",
    },
    opts = {
      labeled_modes = "nx",
    },
  },
  {
    "flash.nvim",
    -- enabled = false,
    opts = {
      modes = {
        char = {
          keys = {},
        },
      },
    },
  }

But questions about user configuration should really be asked in the Discussions instead in my opinion.


Magento 2: no assets generated when using custom magento_themes without split_static_deployment

Deployer version: 7.3.3
Deployment OS: MacOS

When using a custom magento_themes configuration like
   ->set('magento_themes', [         'myvendor/theme-at' => 'de_AT',         'myvendor/theme-be' => 'fr_BE nl_BE',     ]);
and not setting "split_static_deployment" to true no assets are generated.
The problem is in https://github.com/deployphp/deployer/blob/ef0087d988edd1a5a25339842d752aa1f452a923/recipe/magento2.php#L188-L193 where its only checked if there are multiple themes to process and then the language configurations from the custom configuration get passed as theme names.

Where do you get this notation from?
   ->set('magento_themes', [ 'myvendor/theme-at' => 'de_AT', 'myvendor/theme-be' => 'fr_BE nl_BE', ]);
``

I've never seen that before and it looks like we've just never supported it.

I found this in the documentation and in the magento2 recipe itself
https://github.com/deployphp/deployer/blob/ef0087d988edd1a5a25339842d752aa1f452a923/recipe/magento2.php#L37-L41

Ah, that was introduced through https://github.com/deployphp/deployer/pull/3326 by @renttek. Julian could you take a look here?

I'll take a look into this

I could reproduce this and I suggest the following change:
this foreach:
foreach (get('magento_themes') as $theme) {
    $themesToCompile .= ' -t ' . $theme;
}

should be changed to this:
$themes = array_is_list(get('magento_themes')) ? get('magento_themes') : array_keys(get('magento_themes'));
foreach ($themes as $theme) {
    $themesToCompile .= ' -t ' . $theme;
}

because it should obviously work for each of the following configurations:
//1.
set('magento_themes', []);

//2.
set('magento_themes', ['Magento/luma', 'Custom/another']);

//3.
set('magento_themes', [
    'Magento/blank'   => 'nl_BE',
    'Magento/luma'   => null,
    'myvendor/theme-at' => 'de_AT',
    'myvendor/theme-be' => 'fr_BE nl_BE',
    'Custom/another' => '{{static_content_locales}} it_IT',
]);

So either the array values are iterated or the keys of the associative array.
(when it is empty the foreach is not reached because of the count check)
wdyt?

and to be consistent with the other 2 method calls of setup:static-content:deploy, the -f switch should be added as well.

@akosglue that looks like a very good solution :+1: Could you PR it up?

So I opened that PR but it fails when checking the docs. I have updated it though but now I am not sure hot to proceed with the fix.
The changes are similar somehow, there it was ok, but in my PR it fails:
https://github.com/deployphp/deployer/pull/3812/files
https://github.com/deployphp/deployer/pull/3818/files

Merged by now


Unit tests for team link and positions endpoints

dev
JIRA

Main JIRA ticket

Code reviewers

[ ] @github_username

Second Level Review

[ ] @github_username

Summary of issue
ToDo
Summary of change
ToDo
Testing approach
ToDo
CHECK LIST

[ ]  СI passed
[ ]  Сode coverage >=95%
[ ]  PR is reviewed manually again (to make sure you have 100% ready code)
[ ]  All reviewers agreed to merge the PR
[ ]  I've checked new feature as logged in and logged out user if needed
[ ]  PR meets all conventions

Great work! I like how precise you covered all the cases

Glad to hear that


cleanup: Don't include nljson.h from public oauth2 headers.

Fixes #2772
Per @coryan's suggestion, this approach factors out any logic w.r.t JSON into
separate functions. For the functions dealing with JWT assertions, since they
were not only private functions but also essentially standalone, they were
moved out of the ServiceAccountCredentials class and made to accept a
time_point parameter. Functions which would normally take just a payload take
a HttpResponse to preserve the status. Finally, a ServiceAccountMetadata
struct was added; ComputeEngineCredentials was not modified to accept this to
prevent API breakage.

This change is

Does this function need to be in the header? It is only used in one place, and that is a private member function of the class. We could move the declaration to the .cc file (and I think save the .dump() calls).

I agree. But then we have to factor this out into a separate function too:
https://github.com/googleapis/google-cloud-cpp/blob/825c3a6a985b55a0b5c33914f1fa4ed784332ea0/google/cloud/storage/oauth2/service_account_credentials.h#L215-L219
Which would also be a one-use function and probably not worth it I think. Happy to change it if you disagree.


Add recommendation to update systems before installing ROS 2.

Installing ROS 2 packages without updating the system first can lead to conflicts due to package version differences.
This is generally useful but of particular importance on Ubuntu Jammy: https://github.com/ros2/ros2/issues/1272

@clalancette do you think we ought to backport this to earlier distributions? (I do)

Should we also add this note to the "Alternates" installation methods as well? It's probably as important.

I pulled them into into shared snippets to use across multiple files.


Crash at ManagerRender.drawScreen

RTMP, OpenGlView, No filters, Front camera
Fatal Exception: java.lang.RuntimeException: drawScreen end. GL error: 1285
       at com.pedro.encoder.utils.gl.GlUtil.checkGlError(GlUtil.java:126)
       at com.pedro.encoder.input.gl.render.ScreenRender.draw(ScreenRender.java:115)
       at com.pedro.encoder.input.gl.render.ManagerRender.drawScreen(ManagerRender.java:66)
       at com.pedro.rtplibrary.view.OpenGlView.run(OpenGlView.java:144)
       at java.lang.Thread.run(Thread.java:929)

That error means OutOfMemory.
Do you have any way to reproduce it?
Do you have this error with all devices?

Based on firebase info, devices had enought memory

Acording with android documentation 1285 is out of memory:
https://developer.android.com/reference/android/opengl/GLES20#GL_OUT_OF_MEMORY
The question is, why only in certains devices?
Do you have any way to know if your user is streaming long time before get this crash? Maybe I'm not releasing something properly

I'm facing a similar issue mostly on Samsung devices

Fatal Exception: java.lang.RuntimeException: drawScreen end. GL error: 1285
       at com.pedro.encoder.utils.gl.GlUtil.checkGlError(GlUtil.java:142)
       at com.pedro.encoder.input.gl.render.ScreenRender.draw(ScreenRender.java:131)
       at com.pedro.encoder.input.gl.render.ManagerRender.drawScreen(ManagerRender.java:76)
       at com.pedro.rtplibrary.view.OpenGlView.run(OpenGlView.java:189)
       at java.lang.Thread.run(Thread.java:764)

After debugging with the Android Studio profiler I didn't find any leaks 😞

Hello,
If possible, let me know if you are using any filter and which one.
Also, if you can get time streaming until you get that crash, device model and android api.
This crash seems really difficult to reproduce and I'm not sure about the reason so it is hard to fix.

I'm using BlurFilterRender. These crashes are not super frequent so from my perspective it's a low prio thing.
Based on the logs, crashes are pretty random. Sometimes they occur after a few seconds of streaming and somethimes after user leaves the screen. It's also possible that these are caused by other leakage elsewhere in the application 🤷
If I manage to find some pattern or more details I'll post them here 🤞

Unfortunately no, crashes seems pretty random. Luckily they doesn't affect a significant number of users 🤷

If anyone could find a way to reproduce the error report it here.
Since in OpenGL documentation the error code is an out of memory I only can assume a GPU memory overflow.
If you have any way to report GPU memory state when you receive that error, this could be helpful

I have the same error on Samsung Galaxy S21.
It's happen all the time. Without any filter.

@minonger were you able to fix it? Are there any special conditions to it happening? Like using WiFi/5G / reconnecting / going to background and back?

Have you solved it? I'm facing this problem now. I'm using both ImageFilter and TextFilter together. I'm still in development. There were no problems in short-term tests. I suddenly encountered this error while broadcasting for more than 20 minutes. Test device is Redmi Note 11 pro. 3.5gb free memory and 35gb available disk. I'm using mobile internet.

Hello,
This error is really random and I'm not able to reproduce it consistently to solve.
If you can reproduce it, test this and tell me the result:

Try to stream without filters to discard problems with filters. If you can't reproduce the error without filters, then test with only one filter to try find the filter that produce problems.
If you are streaming in background or you have rotation support. Try to stream in foreground all time and block screen orientation. This is to discard a problem related with the preview lifecycle.

Remember that this is an out of memory. Can you check the GPU memory usage? Maybe you can find a pattern

Thank you for quick reply @pedroSG94 .
Tests still continue. So propably I'll reproduce it. App currently doesn't support rotation. It only supports landscape and only streams foreground. I thought maybe the behavior was being disrupted for a reason like "preventing accidental screen touching".
I'll let you know immediately when reproduce it again.

Hello,
I did a refactor to GlInterface (OpenGlView and GlStreamInterface) that maybe solved this error.
If possible try update the libary to version 2.3.7 and let me know if the problem is solved or reduced.

I followed the user experience for a while and found the error. The activity was set to sensorlandscape. This error occurred when the user made a movement that caused a 180-degree rotation while holding the device. After changing the sensorlandscape setting to landscape, the problem disappeared. Also thank you for your revision. I will include it in the next refactoring process.

It is worth adding that the problem only occurred if the screen was rotated while using filters. There is no problem with rotation when there is no filter.

Hi @pedroSG94,
Today I have found that this issue occurs for AndroidViewFilterRender, if it has isHardwareMode forced to be false. I've not checked any other filter, my orientation is always horizontal.
I hope, it will help you with finding solution.

Hello,
@Feureth are you using library version 2.4.0?

Yes

Hello,
That error is GL_INVALID_VALUE.
Normally produced because a parameter is invalid.
Do you have more info about the error? can you reproduce it?

Fatal Exception: java.lang.RuntimeException: initGl end. GL error: 1281
at com.pedro.encoder.utils.gl.GlUtil.checkGlError(GlUtil.java:144)
at com.pedro.encoder.input.gl.render.filters.BaseFilterRender.initGl(BaseFilterRender.java:51)
at com.pedro.encoder.input.gl.render.MainRender.addFilter(MainRender.kt:99)
at com.pedro.encoder.input.gl.render.MainRender.setFilterAction(MainRender.kt:148)
at com.pedro.library.view.OpenGlView.draw(OpenGlView.java:251)
at com.pedro.library.view.OpenGlView.lambda$onFrameAvailable$6(OpenGlView.java:347)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
at java.lang.Thread.run(Thread.java:923)
```

In this case the error is produced adding a filter. Do you know the filter used in this case and possible values?

Unfortunately I'm not able to reproduce it. It happens to less than 0,1% of all users so it's not that bad. I'm using BlurFilterRender.
It seems to happen more on older devices.

Seems that the error is usually in Samsung devices. Can you show me device models?

Sure

Ok, I will try investigate it.
According with internet, this maybe is an error compiling the shader but it has no sense for me (in that case this should crash always in that model. Maybe an error reading the shader from disk?

I didn't dive deep into the implementation details, so I'm not able to tell. I'm not really bothered by this crash, but I thought I'd share that 1285 turned into 1281 😇

Any crash report is really welcome and know that the error 1285 is gone are good news.
I hope that, this error (1281) is less frequent than the other (1285). In this case, at least, it is a step in the good direction. :)

I updated the library with a fix that could help for this case. instead of let opengl fail after create shaders and programs, now the library will throw an error depend of the moment that opengl fail, this way we can get more info about the error 1281 and fix it in nexts commits.
Try to update the library to version 2.4.7


Version change for ODF 4.16

Updated version for ODF 4.16

log -
deploy-odf416.log

Merging the PR as Pooja has validated the changes.


We could raise an error if Formtastic is not configured

ActiveAdmin and other gems list formtastic as a dependency, but don't force the developer to run our installer, which sets up a the configuration file. This has lead to a lot of confusion for deprecation errors when upgrading to Formtastic 3.1 and not really having an awareness that formtastic has a configuration they may need to manage.
So, I think we could add a new hook in our initializer that signals to Formtastic that it has been explicitly configured with an initializer, so that all users have a complete install. We could probably also ping AA and other libraries to run our installer as part of their install and upgrade processes.

Is there a list of deprecation errors somewhere that 3.1 introduces, that will be an issue when missing the configuration file?

CC @timoschilling

For new installs activeadmin can trigger the formtastic install, for old installs formtastic should warn.

@seanlinsley see DEPRECATIONS


Styling and seed

Added images in seed and added 2 new exercises

added Loading component to AllExercises *
started styling file for presets *
added 2 new exercises and rest of images to seed *
Thank you!


Fix issue where controller is watching a secret that is never created

/merge-when-ready delete-branch

OK, I will merge the pull request when it's ready, leave the commits as is when I merge it, and delete the branch after I've merged it.

Removing "merge-when-ready" label due to new commits


change all strtoupper method to mb_strtoupper

I'm closing this PR for a few reasons:

it tries to change stuff unrelated to the underlying issue (TravisCI)
the mbstring extension must be installed for mb_strtoupper() to work, but it's a non-default extension.
regardless, mb_strtoupper() is slow as hell which might not be a problem in most normal usage cases for this function, but when you start issuing thousands of commands (see pipelines) it can add seconds just for the conversion. Doing a plain strtr($string, 'abcdefghijklmnopqrstuvwxyz', 'ABCDEFGHIJKLMNOPQRSTUVWXYZ'); would be way faster (still with a greater overhead than strtoupper() though) and it would be enough for our use-case since Redis uses plain ASCII letters for command IDs by convention, but it'd would be a PITA to maintain such code spread through the whole codebase.

You could try mitigating this by doing a $client->INFO() instead of $client->info() but I'm plenty aware this is not a fix as this won't solve anything which is outside your control, e.g. other libraries or applications.
Having said that I do aknowledge that when the Tukish locale is in use we have a problem and that's why I never closed the first issue reported about this (see #372). I'm still thinking about how we could solve this in a future release with a more maintainable approach and without impacting performances unless strictly needed. I have a few ideas, but nothing set in stone yet.


[Designer-> Element Properties]: Incorrect name is defined for 'Remove' button under 'Element Properties' pane.[Rendering] Bug Title Here

Target Platforms
Other
SDK Version
105.0.1300.0
Application Name
Adaptive Cards
Problem Description
Bug ID: https://microsoft.visualstudio.com/OS/_workitems/edit/33898617
Test Environment:
OS Version: Dev (OS build 21390.1)
Browser Version: 93.0.916.1 (Official build) dev(64-bit)
URL: https://adaptivecards.io/designer/
Screen Reader: Narrator, NVDA
Repro Steps:

Launch the URL: Adaptive Cards (adaptivecards-ci.azurewebsites.net)
Navigate to header control ‘Designer’ using tab key and activate it through enter key.
Navigate to the "Element properties" button using the TAB key & activate it.
Open narrator for the ease of access.
Navigate to 'Cross(Remove)' button with Tab key.
Observe the screen reader narration.

Actual Result:
Incorrect name is defined for 'Remove' button under 'Element Properties' pane. Screen reader is announcing, "Cap X button remove".
Expected Result:
Correct name should be defined as 'Remove button' or 'Cancel' for the 'Cross' control. Screen reader should announce as, "Remove/ Cancel button".
Observation: Issue is also repro with NVDA (2020.4).
User Experience:
Users who rely on screen readers with impairments will effects to navigate if incorrect Name is defined for "Remove" button.
MAS Reference:
https://liquid.microsoft.com/Web/Object/Read/MS.Accessibility/Requirements/04.01.02
Screenshots

Card JSON
NA

Sample Code Language
No response
Sample Code
No response

Closing because this is a duplicate of https://github.com/microsoft/AdaptiveCards/issues/7693.
There is also a PR in review where we are discussing the proper resolution to this issue (https://github.com/microsoft/AdaptiveCards/pull/7725)


noalts doesn't work

The -noalts flag doesn't seem to work at all. No matter what I've tried, it always does alternations.

Thanks @CharlieEriksen! I appreciate the helpful feedback. This bug is now fixed, and will be in the next commit.

I'll assume that this is no longer giving you trouble and close the issue. Thanks for the feedback!


Check that LighthouseRun queries filter out 'invalid_run' ones

For methods that query LighthouseRun or calculate averages, verify/update them to ensure they filter out LighthouseRun that have invalid_run set.
Best way is to probably create a queryset manager for LighthouseRun so it can be common used wherever needed.
For obvious reasons... we want to ensure we don't calculate averages using runs that were 'invalid' as to skew the real average performance for the URL

Confirmed working as expected, but need to move filter into a queryset for cleaner code maintenance and reusability.
Todo: make LighthouseRunManager at top of models just like URL.


All tables that have an audit table should correctly record changes.

I could help out with this! I've implemented such functionality using triggers in MySQL and Postgres.
The one gotcha is on record deletions, when you want to record an ID not already defined, such as the moderator who performed the deletion through the app. Unfortunately, I don't know any go, so I wouldn't be able to do that, if it's a detail you'd want included.
Does this issue include defining and creating the audit tables in the initdb_*.sql files? I don't see them there already.

I definitely appreciate the help. @comraderat did all of the DB refactoring but unfortunately he had to drop out because of IRL concerns. Even if you're not familiar with Go, I'm sure I can integrate any SQL contributions you make.
For things like record deletion, if it's something like a staff member that no longer exists when the record is deleted, it will just be assumed that the staff member was removed.
There are a few auditing tables, though right now it's just DBPREFIXip_ban_audit, DBPREFIXip_ban_appeals_audit, and DBPREFIXreports_audit, and right now only DBPREFIXip_ban_appeals_audit is used in the code, in  /pkg/gcsql/queries.go in AddBanAppeal:
const sql1 = `
/*copy old to audit*/
INSERT INTO DBPREFIXip_ban_appeals_audit (appeal_id, staff_id, appeal_text, staff_response, is_denied)
SELECT id, staff_id, appeal_text, staff_response, is_denied
FROM DBPREFIXip_ban_appeals
WHERE DBPREFIXip_ban_appeals.ip_ban_id = ?`
const sql2 = `
/*update old values to new values*/
UPDATE DBPREFIXip_ban_appeals SET appeal_text = ?

This issue is still in progress


Improvement in the view of custom subscription policy

Shall we keep it as empty instead of showing null, if the user did not add any description to custom subscription tier?

Unable to reproduce the issue

This is reproducible with only with Oracle as Oracle does not support keeping the EMPTY string in tables. As this depends entirely on the behavior of the database, we might can add a check from UI side.

Fixed with carbon-apimgt 6.8.40-SNAPSHOT


Update text-editor.coffee

Fix a typo in src/text-editor.coffee (adter => after).

:zap: :eyes: Thank you.


Closes #534 | Add/Update Dataloader idner_news_2k

Closes #534
Checkbox

[X] Confirm that this PR is linked to the dataset issue.
[X] Create the dataloader script seacrowd/sea_datasets/{my_dataset}/{my_dataset}.py (please use only lowercase and underscore for dataset folder naming, as mentioned in dataset issue) and its __init__.py within {my_dataset} folder.
[X] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _LOCAL, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _SEACROWD_VERSION variables.
[X] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
[X] Make sure that the BUILDER_CONFIGS class attribute is a list with at least one SEACrowdConfig for the source schema and one for a seacrowd schema.
[X] Confirm dataloader script works with datasets.load_dataset function.
[X] Confirm that your dataloader script passes the test suite run with python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py or python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py --subset_id {subset_name_without_source_or_seacrowd_suffix}.
[ ] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

LGTM! Merging this now. Thank you for the dataloader @R-Damanhuri 😄


Adding styling to cards of in About Us section

Is your feature request related to a problem? Please describe.
Right now, this section looks good but if we add some styling to the cards such as:

Card will only show name and bio at the start.
After hovering over it it will display all the links and enhancing some more styling in cards so it will highlight the card we are looking at.

Describe the solution you'd like
A clear and concise description of what you want to happen.
Motivation
Please outline the motivation for the proposal.
Are you willing to contribute to this issue? [Yes/No]
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
If you want I can provide you the video of the effect I am talking about.
Please let me know @kothariji @Akash-Salvi @Nishant127

Hi @ankitaugale23, First of all, thank you so much for taking out the time to create this issue.
It would be a great help if you can provide a small video/animation so that we can get a clever idea.
Also please mention in the issue, that would you like to work on this issue😊
And last but not least,
Keep Contributing 🙏🔥

Sure. I will send it to you by tomorrow.

If you permit me I would like to contribute to this issue.
Suggestion: I will not completely copy as same as that of @ankitaugale23 I will do some modification myself to look it better if you permit.

Sure. https://drive.google.com/file/d/15xTBB07m0jH06gjh_E-LShT4B2tVxWjF/view?usp=sharing
This is a demo to just show what actually my idea we can modify according to your liking but this will just utilize our space.
@kothariji

@ankitaugale23 this looks really nice, but as there are 3 creators, so how would manage all 3 cards in the same row,(refer pic)

I would like to contribute as a Gssoc21 participant. could you please assign me this?

any updates @ankitaugale23  ?

any updates @ankitaugale23 ?

Sorry. I am having my exams right now. Will it be okay with you if I revert back with my idea and changes till 3rd April? After confirmation from your side to the idea about how I will do it for 3 members I will complete the coding part in maximum of two days. Let me know if this is fine with you guys.

any updates @ankitaugale23 ?

Sorry. I am having my exams right now. Will it be okay with you if I revert back with my idea and changes till 3rd April? After confirmation from your side to the idea about how I will do it for 3 members, I will complete the coding part in a maximum of two days. Let me know if this is fine with you guys.

Sure No problem @ankitaugale23, and all the best for the examinations 🚀

Hi @ankitaugale23, any updates ??

I will make a PR till Sunday at 9 pm. Is that fine? And I apologize for being inactive in the communication but I will for sure make my PR this Sunday.

I will make a PR till Sunday at 9 pm. Is that fine? And I apologize for being inactive in the communication but I will for sure make my PR this Sunday.

No issues @ankitaugale23, but please share a basic design /prototype for the same. So that if there are changes we can do that in early stage, and once we finalized the design, we are good to go for code.

I didn't quite get what should I send you as a prototype/design? (Animation video of the same effects but for these 3 cards) that's what should I send or I am misinterpreting it? Please let me know. @kothariji

I didn't quite get what should I send you as a prototype/design? (Animation video of the same effects but for these 3 cards) that's what should I send or I am misinterpreting it? Please let me know. @kothariji

Yes,a actually a basic canva design would also work. I just want to see, how final presentation would look like 😊

Sure. I will share it with you today in some time. @kothariji

Here are the canva designs. https://drive.google.com/drive/folders/1BFSZTZLORMEEqdTBohHp0IdHUt6mDzvI?usp=sharing
Let me know what do you think and if any changes you want.
One thing I would like to mention is that the cards will be revealed vertically as we don't have much space horizontally.
@kothariji

Hi @ankitaugale23 I saw the design, but somewhere I don't feel that there is a real need for it.

According to UX, only names with big pictures will not look pretty.
I hope you understand 🙌

Here are the canva designs. https://drive.google.com/drive/folders/1BFSZTZLORMEEqdTBohHp0IdHUt6mDzvI?usp=sharing
Let me know what do you think and if any changes you want.
One thing I would like to mention is that the cards will be revealed vertically as we don't have much space horizontally.
@kothariji

Hi @ankitaugale23 I saw the design, but somewhere I don't feel that there is a real need for it.

According to UX, only names with big pictures will not look pretty.
I hope you understand 🙌

I understand. I will work on different issues no problem.
Thanks for considering my idea though!

I understand. I will work on different issues no problem.
Thanks for considering my idea though!

Thank you, sorry for the inconvenience caused 🙌


update to support haskell-src-exts-1.18

One of major haskell-src-exts-1.18 changes
is to parameterise AST with source location
and add that field to every ast node.
Signed-off-by: Sergei Trofimovich<EMAIL_ADDRESS>

Thanks!


chore: support esm and commonjs

chore: support esm and commonjs
chore(farrow): update esbuild

The problems with ESM support of farrow are not only related to farrow itself, it's also related to the tools around farrow.
farrow emits esm files even now, but disable them in package.json intentionally to avoid the weird problems we met when integrating with other tools.
The benefit of farrow is not about ESM, it's about type-safe. It means farrow should always be compiled instead of imported directly in ESM runtime.
So in my opinion, keeping current status and waiting for the community to move on will be a better way.

Anyway, thx your effort in ESM support, they will bring more value in the future:-)

https://github.com/vitejs/vite/issues/5839


Bump console version to 2.3.4

$Subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6663039447

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6663039447
Status: success


pods installation issue in react native ios

[!] CocoaPods could not find compatible versions for pod "VoxeetSDK":
In Podfile:
react-native-comms-sdk (from ../comms-sdk-react-native-latest) was resolved to 3.8.0-beta.1, which depends on
VoxeetSDK (~> 3.8.0)
Specs satisfying the VoxeetSDK (~> 3.8.0) dependency were found, but they required a higher minimum deployment target.

@asadumair 3.8.0-beta.1 has just now released, can you please check again?

@asadumair 3.8.0-beta.1 has just now released, can you please check again?

there is pods issue in that

@asadumair please run pod repo update in ios directory of your app.

this is working thanks


zoom with hflip and 90º rotations starts to shift

Apparently something is not right in the correctShift function of the zoomtool.
When an image has hflip=true and a rotation of 90º or 270º the image starts to drift towards a corner. My theory is that the flips should be applied before the rotations in the correctShiftfunction, but I don't fully understand it.
    function correctShift(shift, viewport) {

      // Apply Flips
      if (viewport.hflip) {
          shift.x *= -1;
      }

      if (viewport.vflip) {
          shift.y *= -1;
      }

        // Apply rotations
        if (viewport.rotation !== 0) {
            var angle = viewport.rotation * Math.PI / 180;

            var cosA = Math.cos(angle);
            var sinA = Math.sin(angle);

            var newX = shift.x * cosA - shift.y * sinA;
            var newY = shift.x * sinA + shift.y * cosA;

            shift.x = newX;
            shift.y = newY;
        }

        return shift;
    }

I have the same problem, and this code correct the error!


Want a is countdown timer running? block

There's currently no way to tell whether the countdown timer is actively counting down without keeping track of its state yourself. See the awkward code to do that in this program: https://github.com/clemsongamecodingcamp/cgcc-video-game-project-c
We'd like a new block called is-countdown-timer-running? to return a boolean if the countdown timer is currently counting down (and has not yet reached zero or been stopped).

I think this can be achieved by simply returning the current countdown value.


[FLINK-9932] [runtime] fix slot leak when task executor offer slot to job master timeout

What is the purpose of the change
(For example: This pull request fix that the slots in task executor will leak if task executor fail to offer it to job master due to rpc timeout.)
Verifying this change

Add a test in TaskExecutorTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no )
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Yes exactly, this would have been the idea. Do you want to update this PR accordingly?


kie-issues#821: Kogito Serverless Operator weekly job

ISSUE: https://github.com/apache/incubator-kie-issues/issues/821
This PR introduces a new weekly Jenkins job for Kogito Serverless Operator that updates the project version to 2.0.0-YYYYMMDD-snapshot locally, build it and deploy the image to quay.io
The weekly job also creates a new tag in the repo with this format: 2.0.0-YYYYMMDD.

I think we can use 10.0.0 since we should follow KIE release train. WDYT?

Perhaps we can set the version to 999-snapshot as discussed here: https://github.com/apache/incubator-kie-issues/issues/763? WDYT @ricardozanini @domhanak

@rodrigonull is the jenkinsfile a "close" copy of an existing one? Wouldn't reuse of a nightly one be possible with some tweaks? Also please can you check that the triggering part is in place? Or please point me to where it's defined.

Yes, this Jenkinfile is very similar to the nightly one. I've decided to create a new jenkinfile due to my lack o experience with the pipelines framework and to avoid breaking existing jobs. There are some differences between the nightly and the weekly jobs as described bellow:
for weekly jobs we need to:
Checkout the code from a specific datetime
Update the project version to a specific version (999-YYYYMMDD pattern)
Create and publish a tag in the repository
I think at some point would be nice to refactor this work and create a new job type for weekly jobs and update the nightly jenkinsfiles to support weekly jobs. WDYT?
The PR that adds the triggering is this one: https://github.com/apache/incubator-kie-kogito-pipelines/pull/1177


Feature Request: Add --update command

[X] I have checked that the CLI documentation and API documentation doesn't solve my issue

Description of the Issue
Is it possible to add a --update command to update the Box CLI applicaton. It looks like that I would need to reinstall the application to get the update right?
Versions Used
Box CLI: 2.0

@abarber229 Yes, currently you'll need to use the installers to update the CLI.  The framework we're using the build the CLI does have the capability to add a box update command, or even auto-update, but on our end we'll need to do some work around testing out the update mechanism and hosting the update files.  This is a feature we would love to add, but it's not on our immediate roadmap; I'll update this issue when we start working on it!


Can not return title in canvas card

Pls see link in the right bottom card. It looks has begin fetch, but never return a title.

This issue persists in the current version as well.

Never mind, I updated Obsidian and this seems to have fixed the issue.
I think this issue can probably be closed.


Update InternalSearchControllerImpl.cs

Fixes #5049.
Caching search results needs to include the current culture in the key.

Summary

@Andy9999 There is a comment trick that associates a PR with an Issue in a way that the issue is automatically closed when the PR is merged. You can read more about it here https://docs.devart.com/studio-for-sql-server/source-controlling-databases/associating-commits-with-github-issues.html
I edited the first line of your comment accordingly.

@Andy9999 There is a comment trick that associates a PR with an Issue in a way that the issue is automatically closed when the PR is merged. You can read more about it here https://docs.devart.com/studio-for-sql-server/source-controlling-databases/associating-commits-with-github-issues.html I edited the first line of your comment accordingly.

Thank you! Will try that trick with the other 2 issues next weekend. :)


Can't access the Trello development board

I would love to give some feedback on a future feature, but I can't access the Trello board in CONTRIBUTING.md. Let me know where I should leave the feedback!

Hi! Some boards are private. You can get in touch with our customer service team! Please email us at<EMAIL_ADDRESS>


[Discovery] - Should private collections and private revisions warn submitters / users about their state ?

Context
See:

cell-science-data-wrangling
cell-science-data-wrangling.

Private Collections and permanent links
Since its inception, CELLxGENE Discover guaranteed that the permanent (canonical) URL(s) for both a private collection and any of its related CXG(s) must not change when the collection is published. This allowed submitters to reference their CELLxGENE collection and visualization in journal articles in preparation. The expectation was that the collection must be published before the article appears. There have been recent cases where the journal article was published, while the collection was still private. There is no warning to users redirected from the journal article that the collection is private.
Perhaps it would be helpful to have a banner or watermark indicating that the collection/CXG(s) are private and have not been published - as a prompt to the submitter to change their state or contact the curators.
Private revisions and version links
With the introducing of versioning in 2023, private revisions of public collections receive their own permanent version URL(s) for both the revision and its updated CXG(s).  When the revision is re-published, the update is published as a refresh of the canonical URL, although the version URL(s) are available via the Discover API for reproducibility scenarios. Data submitters must not reference the permanent version URL in publications, because it will soon be outdated by schema updates. The canonical URL(s) will always reference the latest version.
It seems like we need a caution about the version URL(s) in revisions?
CC: @hthomas-czi

Amanda has set up an airtable that is produced on the 1st of every month and shows access to private URLs over the last month. Aug 1 was the first official production and from our evaluation of that, 13 private links have been confirmed to be shared in publications, github repos, lab sits, or other data resources.
For the banner/watermark, I'm not sure it should be aimed at the submitter - seems like maybe they submit and then forget about it. So maybe it should target the random user "Warning - unreleased data may contain errors, may be subject to access disruptions, and is not versioned. If you have come to this via a publicly-shared link, please contact the cellxgene team so they can work with the contributor to finalize their submission"


Get VIVE trackers speed and angularSpeed

Since SteamVR plugin has been updated to version 2.0 I'm unable to get the vive trackers speed and angular speed. I have seen around internet how to do it for older versions but not for this one. Is there anybody who knows how to achieve this? ( I'm using SteamVR plugin in Unity ). Many  #thanks.

The pose should have velocity and angularVelocity members. You'll need to use pose actions to get this data now. You can find some examples in the Throwable.cs file in the interaction system.


Cannot start some containers after restarting docker daemon (=> #6675)

This is in reference to #6675.  Please take a look at my comment at the very end.  This is still an issue
with docker v1.6 and the device mapper storage driver.

Tested this with aufs and my test scenario works just fine.

In reference to #6675 here is short fix:
Step1:
docker start CONTAINER_ID
Step2:
docker attach CONTAINER_ID
This should bring the container.

Just to confirm, this is happening with devicemapper?
Can you paste your docker info and docker version?
Does this still happen on docker 1.7 with a properly configured system? (i.e., udev sync must be supported)

Closing for now due to no response.
If you can provide the requested info and repro steps please ping here so we an take a look.
Thanks!


Tardis Plugin Spam (Was fixed on Paper)

Not a crash
Expected behavior
 The server not being spammed
https://pastebin.com/RbUzTw0L
Observed/Actual behavior
 Server console is being spammed.
Steps/models to reproduce
Use Pupur and Tardis plugin instead of paper.
Purpur version

25.06 15:00:05 [Server] INFO This server is running Purpur version git-Purpur-1191 (MC: 1.17) (Implementing API version 1.17-R0.1-SNAPSHOT) (Git: bdcea0e on HEAD)
25.06 15:00:05 [Server] INFO You are running the latest version
25.06 15:00:05 [Server] INFO Previous version: git-Purpur-1187 (MC: 1.17)
Agreements
Please agree to the following.

[ -] I am running the latest version of Purpur available from https://purpur.pl3x.net/downloads.
[ -] I have searched for and ensured there isn't already an open issue regarding this.
[ -] My version of Minecraft is supported by Purpur.

Other
 This was fixed a few days in Paper but not in Purpur for some reason

Was fixed as part of this issue https://github.com/PaperMC/paperweight/issues/24

This issue is already being tracked upstream. It is a mappings issue in the paperweight build tool. They have not fixed it, yet.
https://github.com/PaperMC/paperweight/issues/24


"linked" variables that call functions of constants and/or variables

@bqpd, had this issue using the function that we worked on yesterday.  It seems that all the variables that I am calculating using lambda functions are treated as free variables in the model.  Shouldn't they be constants? Because it's saying I can get more of this because nothing is constraining me...
Here are my variables that I am calculating via ``lambda``` function
              T_atm = VectorVariable(NSeg, 'T_{atm}',
                                      lambda T_sl, L_atm, h: T_sl - L_atm*h, 'K',
                                      'Air temperature', args=[T_sl, L_atm, h])
              P_shaftmax = VectorVariable(NSeg, 'P_{shaft-max}',
                                      lambda P_shaftmaxMSL, h_ref, h:
                                      P_shaftmaxMSL*(1-0.5178*(h/h_ref)**0.92),
                                      'hp', 'Max shaft power at altitude',
                                      args=[P_shaftmaxMSL, h_ref, h])

In the output, they are both under the free vars section: (obviously incorrect values)
\vec{T_{atm}} : [ 1.43e+05  403       1.47e+05  303      ... ]  [K]         Air temperature
\vec{P_{shaft-max}} : [ 6.37e+07  0.838     1.15e+08  1.18     ... ]  [hp]        Max shaft power at altitude

Oh, shoot. You might need to do a sweep for linkedsweep values to be active. Try that?

[ ] linked variables that replace linkedsweep but don't require a sweep
[ ] linked variables calling external functions
[ ] linked variables with gradient functions


Support for rm -rf  || true and rm -rf  ||: idioms

Feature Description
I would like to see support for

rm -rf <dir> || true
rm -rf <dir> ||:
rm -rf <dir> &> /dev/null
idioms, which allow to ignore error in case if a file does not exist.

Describe The Solution You'd Like
For example redirect the stderr to nul using 2>null.
Code Sample
rmdir /S /Q <dir> 2>null

cool idea. would you like to PR that?
also, if you need it for cargo-make, checkout duckscript and exit_on_error

@sagiegurari I've gotten around with ignore_errors = true. There's a lot on my plate, so I can do it only much later.


ROX-11600: Migrate GroupsService.js to Typescript

Description
A time has come.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed

CI is sufficient

/retest


[SPARK-48736][PYTHON] Support infra fro additional includes for Python UDFs

What changes were proposed in this pull request?
The interface of the SimplePython function already supports specifying additional includes. However, the proto for the clients and the handling in the planner has been missing. This patch simply adds the base infrastructure for that.
Why are the changes needed?
Compatibility
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing UT
Was this patch authored or co-authored using generative AI tooling?
No

Agree with adding a test. Maybe there would be a separate PR that actually uses this message with e2e test. I will defer to it, and merge it first :-).
Merged to master.


Some fixes for the pagination in the Table component.

Summary
The goal for this PR is to fix some details on the Table component (especially on Pagination)
(Internal for Opendatasoft only) Associated Shortcut ticket:
sc-48044
sc-51025
sc-50986
Changes

Adding the storybook-addon-rtl storybook addon
Make the Pagination component compliant with RTL direction
Fix the Cellcomponent: don't force left alignment
Fix the Pagination component: superposition of page numbers (when the number of pages is greater than 999)
Fix the styles of Pagination to match design
Fix the Table component: records can be undefined
Add default background-color on Table
Fix backgound-opacity on the select's arrows (PageSize)
Rename border-radius-x variable to match the value

Breaking Changes
records property is no longer obligatory
Changelog
Some fixes for the pagination in the Table component.
Open discussion
always welcome
Review checklist

[X] Description is complete
[X] Commits respect the Conventional Commits Specification
[X] 2 reviewers (1 if trivial)
[X] Tests coverage has improved
[X] Code is ready for a release on NPM

Our issue with global class names isn't limited to the Table component.
We'll handle this in a dedicated PR (https://app.shortcut.com/opendatasoft/story/50530/sdk-better-global-classname)


Empty check / implement instance injection

... after closing #132

Empty has been replaced with Object3D and Group components.


CkLoop worker traces to previous entry on pe rather than caller

Original author: Jim Phillips (@jcphill)
Original issue: https://charm.cs.illinois.edu/redmine/issues/1437

CkLoop work shows up as ckloop_converse_chare::CkLoop with "created by" and "executed on" both correct, but tracing falls back to to previous entry on execution pe rather than actual creator (CkLoop Parallelize caller).

Original author: Seonmyeong Bak
Original date: 2017-09-27 06:01:06

This is because CkLoop uses converse msgs by default. The msg doesn't contain info of the caller.
Figure out how to contain this info in the msg and use it for tracing.

Original author: Seonmyeong Bak
Original date: 2017-09-28 22:04:11

~~https://charm.cs.illinois.edu/gerrit/#/c/3084/~~ https://github.com/UIUC-PPL/charm/commit/1620b19d543848e3ee1e8ad4ff32c440491698de


ntco::IoRingSubmissionQueue: optimize the way 'push' works

Is your feature request related to a problem? Please describe.
When submission queue becomes full the driver automatically enters the ioring (ntco::IoRingUtil::enter) and waits for numToSubmit events to be submitted. My concern is that it submits exactly 1 entry less than it could.
Let's consider ntco_ioring.t TC1.  See the log:
ntco_ioring.t_assertion.txt
Submission queue size is 4 entries. From the begining of the TC, entries are continuosly pushed (test->defer(id);) to the driver: ID0, 1, 2.....N.
When pushing item with ID 3, there are already 3 entries (ID0, ID1, ID2) in the submission queue. But it detects that it's full and enters the ioring (ntco::IoRingUtil::enter) with numToSubmit equals 3. From my perspective, that's a missed opportunity to submit all 4 entries.
Describe the solution you'd like
With respect to the example above, I think that IoRingSubmissionQueue::push should work differently:

It should check if the submission queue is already full and enter the ioring (ntco::IoRingUtil::enter) if so, pushing all the items from the queue.
Save new entry onto the queue.

https://github.com/bloomberg/ntf-core/pull/197 changes the behavior


[Bolt 3.3 Compatibility] Implement the new Field Type Interface

Extending FieldTypeBase in the two field classes ensures that the fields also implement FieldTypeInterface which is checked for across the Bolt 3.3 codebase.
Fixes: bolt/bolt#6840

Awesome, thanks!


Current stable version (1.1.1) not on bower

The page at http://mozilla.github.io/pdf.js/getting_started/ lists version 1.1.1 as the current stable version, however this version is not available through bower:
$ bower install
bower pdfjs-dist#1.1.1      not-cached git://github.com/mozilla/pdfjs-dist.git#1.1.1
bower pdfjs-dist#1.1.1         resolve git://github.com/mozilla/pdfjs-dist.git#1.1.1
bower pdfjs-dist#1.1.1    ENORESTARGET No tag found that was able to satisfy 1.1.1

Additional error details:
Available versions: 1.1.155, 1.1.153, 1.1.151, 1.1.149, 1.1.147, 1.1.145, 1.1.143, 1.1.140, 1.1.138, 1.1.136, 1.1.134, 1.1.132, 1.1.130, 1.1.128, 1.1.126, 1.1.124, 1.1.122, 1.1.120, 1.1.118, 1.1.116, 1.1.114, 1.1.113, 1.1.111, 1.1.109, 1.1.107, 1.1.105, 1.1.103, 1.1.101, 1.1.98, 1.1.96, 1.1.94, 1.1.89, 1.1.86, 1.1.84, 1.1.82, 1.1.73, 1.1.71, 1.1.69, 1.1.67, 1.1.65, 1.1.63, 1.1.60, 1.1.58, 1.1.56, 1.1.54, 1.1.52, 1.1.49, 1.1.47, 1.1.45, 1.1.44, 1.1.42, 1.1.40, 1.1.38, 1.1.36, 1.1.34, 1.1.32, 1.1.28, 1.1.26, 1.1.24, 1.1.22, 1.1.20, 1.1.18, 1.1.15, 1.1.13, 1.1.11, 1.1.9, 1.1.7, 1.1.5, 1.1.3, 1.0.1211, [...loads of older versions...]

Which is the recommended version for running pdf.js in production?

The human failed to push separate 1.1.1 commit to the master before publishing... eh, let's blame bot... the pdfjs-botio did not recognize that 1.1.1 commit was pushed to the master.  The build is identical to the 05991e9f4b791a6a8471b4e4ff2cf5c63e280e8a, so re-purposing  1.1.1 notes into 1.1.3 and fixing the published build by 5043ba7. Closing as fixed.


Newly referenced packages should be added to Glide def or an ignore list

In pull request #489, Glide definitions were updated to accommodate new packages in the auto-generated code. It's great that we caught this, but we should have an automated means of protecting ourselves moving forward.
It would be fairly easy to write a script that tears through all of the import statements in our repositories, then compares the union to what exists in our glide definition file.

Alternatively, there may already be a package that does such things. Heck, it's probably built into glide already. Mostly, we just need to figure out how to ensure that glide stays up-to-date as part of our integration testing.

Things that are in Backlog shouldn't have an Assignee unless there is active development on them.

After spending a few more months in the Go ecosystem, I can say pretty emphatically that this is not a problem. The Travis environment will already fail if one doesn't have the right glide settings. Arguably, we should have a process for running a glide up when we release, but that feels pretty substantially different from the reason that I opened this bug in the first place. I'm closing this out.


Trapezoidal new tab button hitbox should extend all the way to the top

Describe the bug
If using trapezoidal tab style, the new tab button hitbox is as big as the button itself.
To Reproduce
Steps to reproduce the behavior:

Enable trapezoidal new tab button with any method
(Best seen when maximized the window) Move the mouse to the very top of where the new tab button is

Expected behavior
The new tab button works, letting you open a new tab
Screenshots

The button cannot be clicked

The button can be clicked
Desktop (please complete the following information):

OS: Windows 7 SP1 x64
64-bit

Additional context
This is correct behavior, but can get annoying if you just want to "shoot" the mouse to the button and click it for a new tab and it doesn't work as you've overshot the button.
This isn't the behavior with the normal new tab button, as seen in screenshot 2.

Limiting the hitbox to the area of the button was logical to me. I could expand it to the whole rectangular section encompassing the width of the NTB in the area between the border and the omnibox, but it would be optional.