sqlserver on linux core dump with Docker 1.13 but nothing logged

after upgrading docker from 1.12 to 1.13 and sqlserver image for linux from <IP_ADDRESS> (X64)
to <IP_ADDRESS> (X64)
container log shows below error but the file names reported to contain relevant informant have size of 0 byte for '/var/opt/mssql/log/core.sqlservr.01_20_2017_17_27_03.11.log' and the core dump does not exist ' /var/opt/mssql/log/core.sqlservr.01_20_2017_17_27_03.11'
docker version:
Server:
Version:      1.13.0
API version:  1.25 (minimum version 1.12)
Go version:   go1.7.3
Git commit:   49bf474
Built:        Wed Jan 18 16:20:26 2017
OS/Arch:      linux/amd64
Experimental: false
This is an evaluation version.  There are [144] days left in the evaluation period.
2017-01-20 17:20:34.71 Server      Microsoft SQL Server vNext (CTP1.1) - <IP_ADDRESS> (X64)
Dec 10 2016 02:51:11
Copyright (C) 2016 Microsoft Corporation. All rights reserved.
on Linux (Ubuntu 16.04.1 LTS)
2017-01-20 17:20:34.72 Server      UTC adjustment: 0:00
2017-01-20 17:20:34.72 Server      (c) Microsoft Corporation.
2017-01-20 17:20:34.72 Server      All rights reserved.
2017-01-20 17:20:34.72 Server      Server process ID is 4116.
2017-01-20 17:20:34.72 Server      Logging SQL Server messages in file 'C:\var\opt\mssql\log\errorlog'.
2017-01-20 17:20:34.72 Server      Registry startup parameters:
-d C:\var\opt\mssql\data\master.mdf
-l C:\var\opt\mssql\data\mastlog.ldf
-e C:\var\opt\mssql\log\errorlog
2017-01-20 17:20:35.07 Server      SQL Server detected 1 sockets with 2 cores per socket and 2 logical processors per socket, 2 total logical processors; using 2 logical processors based on SQL Server licensing. This is an informational message; no user action is required.
2017-01-20 17:20:35.07 Server      SQL Server is starting at normal priority base (=7). This is an informational message only. No user action is required.
2017-01-20 17:20:35.08 Server      Detected 3541 MB of RAM. This is an informational message; no user action is required.
2017-01-20 17:20:35.08 Server      Using conventional memory in the memory manager.
2017-01-20 17:20:36.74 Server      Default collation: SQL_Latin1_General_CP1_CI_AS (us_english 1033)
2017-01-20 17:20:41.45 Server      Buffer pool extension is already disabled. No action is necessary.
2017-01-20 17:20:47.89 Server      InitializeExternalUserGroupSid failed. Implied authentication will be disabled.
2017-01-20 17:20:47.89 Server      Implied authentication manager initialization failed. Implied authentication will be disabled.
2017-01-20 17:20:48.64 Server      The maximum number of dedicated administrator connections for this instance is '1'
2017-01-20 17:20:48.65 Server      Node configuration: node 0: CPU mask: 0x0000000000000003:0 Active CPU mask: 0x0000000000000003:0. This message provides a description of the NUMA configuration for this computer. This is an informational message only. No user action is required.
2017-01-20 17:20:48.68 Server      Using dynamic lock allocation.  Initial allocation of 2500 Lock blocks and 5000 Lock Owner blocks per node.  This is an informational message only.  No user action is required.
2017-01-20 17:20:48.68 Server      Database Instant File Initialization: enabled. For security and performance considerations see the topic 'Database Instant File Initialization' in SQL Server Books Online. This is an informational message only. No user action is required.
2017-01-20 17:20:48.69 Server      Query Store settings initialized with enabled = 1,
2017-01-20 17:20:48.72 spid4s      Starting up database 'master'.
2017-01-20 17:20:48.73 Server      In-Memory OLTP initialized on lowend machine.
2017-01-20 17:20:50.86 Server      Software Usage Metrics is disabled.
2017-01-20 17:20:52.93 spid4s      4 transactions rolled forward in database 'master' (1:0). This is an informational message only. No user action is required.
2017-01-20 17:20:53.20 spid4s      0 transactions rolled back in database 'master' (1:0). This is an informational message only. No user action is required.
2017-01-20 17:20:53.21 spid4s      Recovery is writing a checkpoint in database 'master' (1). This is an informational message only. No user action is required.
2017-01-20 17:20:54.23 spid4s      Buffer pool extension is already disabled. No action is necessary.
2017-01-20 17:20:54.23 spid4s      Resource governor reconfiguration succeeded.
2017-01-20 17:20:54.23 spid4s      SQL Server Audit is starting the audits. This is an informational message. No user action is required.
2017-01-20 17:20:54.25 spid4s      SQL Server Audit has started the audits. This is an informational message. No user action is required.
2017-01-20 17:20:56.19 spid4s      SQL Trace ID 1 was started by login "sa".
2017-01-20 17:20:56.24 spid4s      Server name is '6efa3aef6ecf'. This is an informational message only. No user action is required.
2017-01-20 17:20:56.28 spid20s     Starting up database 'msdb'.
2017-01-20 17:20:56.28 spid6s      Starting up database 'mssqlsystemresource'.
2017-01-20 17:20:56.29 spid21s     Starting up database 'testdb'.
2017-01-20 17:20:56.32 spid6s      The resource database build version is 14.00.100. This is an informational message only. No user action is required.
2017-01-20 17:20:56.41 spid6s      Starting up database 'model'.
2017-01-20 17:21:02.80 spid21s     Parallel redo is started for database 'testdb' with worker pool size [1].
2017-01-20 17:21:03.53 spid6s      Polybase feature disabled.
2017-01-20 17:21:03.53 spid6s      Clearing tempdb database.
2017-01-20 17:21:03.90 spid4s      Recovery completed for database msdb (database ID 4) in 2 second(s) (analysis 27 ms, redo 740 ms, undo 480 ms.) This is an informational message only. No user action is required.
2017-01-20 17:21:04.59 spid21s     66 transactions rolled forward in database 'testdb' (5:0). This is an informational message only. No user action is required.
2017-01-20 17:21:04.77 spid4s      0 transactions rolled back in database 'testdb' (5:0). This is an informational message only. No user action is required.
2017-01-20 17:21:04.78 spid4s      Recovery completed for database testdb (database ID 5) in 3 second(s) (analysis 87 ms, redo 1705 ms, undo 123 ms.) This is an informational message only. No user action is required.
2017-01-20 17:21:04.80 spid4s      Parallel redo is shutdown for database 'testdb' with worker pool size [1].
2017-01-20 17:21:15.64 spid17s     A self-generated certificate was successfully loaded for encryption.
2017-01-20 17:21:16.11 spid17s     Server is listening on [ <IP_ADDRESS>  1433].
2017-01-20 17:21:16.12 Server      Server is listening on [ <IP_ADDRESS>  1434].
2017-01-20 17:21:16.12 Server      Dedicated admin connection support was established for listening locally on port 1434.
2017-01-20 17:21:16.13 spid17s     SQL Server is now ready for client connections. This is an informational message; no user action is required.
2017-01-20 17:21:18.32 spid6s      Starting up database 'tempdb'.
2017-01-20 17:21:21.61 spid6s      The tempdb database has 1 data file(s).
2017-01-20 17:21:21.82 spid22s     The Service Broker endpoint is in disabled or stopped state.
2017-01-20 17:21:21.82 spid22s     The Database Mirroring endpoint is in disabled or stopped state.
DBSTARTUP (testdb, 5): FCBOpenTime took 2390 ms
DBSTARTUP (testdb, 5): FCBHeaderReadTime took 1512 ms
DBSTARTUP (testdb, 5): FileMgrPreRecoveryTime took 3125 ms
DBSTARTUP (testdb, 5): MasterFilesScanTime took 1029 ms
DBSTARTUP (testdb, 5): AnalysisRecTime took 231 ms
DBSTARTUP (testdb, 5): RedoRecTime took 1705 ms
DBSTARTUP (testdb, 5): PhysicalRecoveryTime took 1941 ms
DBSTARTUP (testdb, 5): StartupInDatabaseTime took 17068 ms
2017-01-20 17:21:22.03 spid22s     Service Broker manager has started.
DBSTARTUP (msdb, 4): FCBOpenTime took 2340 ms
DBSTARTUP (msdb, 4): FCBHeaderReadTime took 1436 ms
DBSTARTUP (msdb, 4): FileMgrPreRecoveryTime took 2996 ms
DBSTARTUP (msdb, 4): MasterFilesScanTime took 960 ms
DBSTARTUP (msdb, 4): RedoRecTime took 740 ms
DBSTARTUP (msdb, 4): PhysicalRecoveryTime took 768 ms
DBSTARTUP (msdb, 4): RecoveryCompletionTime took 281 ms
DBSTARTUP (msdb, 4): StartupInDatabaseTime took 17741 ms
2017-01-20 17:21:22.45 spid4s      Recovery is complete. This is an informational message only. No user action is required.
2017-01-20 17:25:50.40 spid51      Attempting to load library 'xplog70.dll' into memory. This is an informational message only. No user action is required.
2017-01-20 17:25:52.47 spid51      Using 'xplog70.dll' version '2016.140.100' to execute extended stored procedure 'xp_msver'. This is an informational message only; no user action is required.
sqlservr: ObjectWaitContext.cpp:533: void ObjectWaitContext::Wait(const PLONG64): Assertion `m_state == State::Signaled' failed.
Capturing core dump and information for sqlservr (pid 11)...
cat: core.sqlservr.11.temp/log/debug.log: No such file or directory
dmesg: read kernel buffer failed: Operation not permitted
No journal files were found.
No journal files were found.
ls: cannot access '/var/opt/mssql/log/log/errorlog*': No such file or directory
ls: cannot access '/var/opt/mssql/log/log/exception.log': No such file or directory
ls: cannot access '/var/opt/mssql/log/log/SQLDu*.txt': No such file or directory
ls: cannot access '/var/opt/mssql/log/log/SQLDu*.mdmp': No such file or directory
ls: cannot access '/var/opt/mssql/log/log/system_health*': No such file or directory
Packaging core dump and information into compressed files.
Core dump and information will be written to /var/opt/mssql/log/core.sqlservr.01_20_2017_17_27_03.11 and logs in /var/opt/mssql/log/core.sqlservr.01_20_2017_17_27_03.11.log

Microsoft SQL Server vNext (CTP1.2) - <IP_ADDRESS> (X64)  solved this issue


[$250] Distance - Map pending not scaled and not fully visible when creating > 1 expenses

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.81-1
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4613584
Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/39243
Action Performed:
precondition: user a member of a workspace

Open the app and log in
Open the workspace chat
Create more than 1 distance expenses

Expected Result:
"Map pending" text is scaled in the report preview depending on the number of expenses
Actual Result:
"Map pending" is not fully visible, and it is displayed in multiple lines in the report preview.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/7106409e-3879-4a5c-aaac-833f1b5bad47
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014fbedebd9a7bc999
Upwork Job ID:<PHONE_NUMBER>996987873
Last Price Increase: 2024-06-14

Issue OwnerCurrent Issue Owner: @ikevin127

@francoisl  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1

I'm gonna demote this to NAB since it doesn't break anything and is a small issue

Just compared with production and we used to use a placeholder icon instead of the "Map pending" view:

It sounds like a minor issue coming from https://github.com/Expensify/App/pull/39243, cc @FitseTLT @mollfpr what do you think?

When multiple expenses are created, the map view does not scale appropriately to accommodate all expense entries.
Scaling Adjustment: Implement a dynamic scaling mechanism for the map view based on the number of expenses being created.
Visibility Enhancement: Ensure that all markers or map elements are fully visible and accessible to users without overlapping or clipping.
Implement changes to the map component to resolve scaling and visibility issues.
Conduct thorough testing to verify that the map behaves as expected when creating multiple expenses.
Provide documentation or comments in the codebase explaining the implemented changes
Why I’m a Suitable Candidate:
Problem-Solving Skills: I excel in identifying and resolving frontend issues, ensuring optimal user experience.
Collaborative Approach: I am committed to working closely with Expensify’s team to deliver solutions that meet their requirements and enhance product usability.
I acknowledge the issue and propose to resolve it using the outlined technical approach.
Upon acceptance, I will proceed to post my proposal on Upwork as per Expensify’s guidelines.
Once hired, I will promptly initiate work on implementing the proposed solution and conduct thorough testing to ensure quality and functionality.
By structuring your response in this manner, you demonstrate a clear understanding of the issue, propose a feasible solution, and highlight your qualifications and commitment to addressing Expensify's needs effectively. Adjust the details according to your specific experience and the requirements outlined in the GitHub issue to create a compelling proposal.

@zoro008-svg Please make sure you read Contributing to Expensify and once you have a good grasp please format your comment to match the proposal template if you are interested in fixing this issue, otherwise it won't qualify for review.

@zoro008-svg Please make sure you read Contributing to Expensify and once you have a good grasp please format your comment to match the proposal template if you are interested in fixing this issue, otherwise it won't qualify for review.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01d227ef6816d3f37c?mp_source=share

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01d227ef6816d3f37c?mp_source=share

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01d227ef6816d3f37c

Proposal
Please re-state the problem that we are trying to solve in this issue.

"Map pending" is not fully visible, and it is displayed in multiple lines in the report preview.

What is the root cause of that problem?

We always set the mapPadding to 50 in this logic:
https://github.com/Expensify/App/blob/75c104bfe009ef937bcd129a35bb5a2e52f4339d/src/components/ConfirmedRoute.tsx#L111

The current value is effective when there are only one or two thumbnails. However, with more than two thumbnails, this value becomes disproportionately large relative to the map size, resulting in insufficient remaining space to display routes or waypoints effectively.

What changes do you think we should make in order to solve the problem?

We can change that value to the smaller value if there are more than two thumbnails, for example, 10.

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.

"Map pending" is not fully visible, and it is displayed in multiple lines in the report preview.

What is the root cause of that problem?

We always set the mapPadding to 50 in this logic:
https://github.com/Expensify/App/blob/75c104bfe009ef937bcd129a35bb5a2e52f4339d/src/components/ConfirmedRoute.tsx#L111

The current value is effective when there are only one or two thumbnails. However, with more than two thumbnails, this value becomes disproportionately large relative to the map size, resulting in insufficient remaining space to display routes or waypoints effectively.

What changes do you think we should make in order to solve the problem?

We can change that value to the smaller value if there are more than two thumbnails, for example, 10 (Can be modify when implementing PR). Below is the details:

First, we need to pass down the numberOfShownImages in this:
https://github.com/Expensify/App/blob/75c104bfe009ef937bcd129a35bb5a2e52f4339d/src/components/ReportActionItem/ReportActionItemImages.tsx#L47
to the ConfirmedRoute component.

Then update this:
https://github.com/Expensify/App/blob/75c104bfe009ef937bcd129a35bb5a2e52f4339d/src/components/ConfirmedRoute.tsx#L111
to:

            mapPadding={numberOfShownImage > 2 ? 10 : 50}

What alternative solutions did you explore? (Optional)

♻ Will review existing proposals today!

Proposal
Please re-state the problem that we are trying to solve in this issue.

"Map pending" text is not fully visible, and it is displayed in multiple lines in the report preview.

What is the root cause of that problem?

In Android, in this PR, we need to add the view "Map pending ..." to fix the issue "Wrong location briefly shows before San Francisco upon first open of the Distance request page in android native". (I name it as change 1)

In this PR, we display the map routes instead of the placeholder icon. (I name it as change 2)

The change 2 does not cover the case in change 1, that leads to the current behavior.

What changes do you think we should make in order to solve the problem?

We can update:
https://github.com/Expensify/App/blob/2eb3b4851a3c0413b109b06d45bced6885e46989/src/components/ReportActionItem/ReportActionItemImage.tsx#L80
to:

    if (showMapAsImage && getPlatform() !== 'android') {

so in Android native, it will display as it is when we did not implement change 2.
What alternative solutions did you explore? (Optional)

@truph01 Thanks for your proposal. Unfortunately your RCA is not detailed enough even though it pointed out the offending PRs, additionally the proposed solutions don't align with what we want here code-wise or from a UI point of view.

If interested, here's some additional feedback regarding your proposal:

While you correctly identified the PRs which led to this issue, more and clearer details would help a reviewer actually make sense of the specific part which led to the issue's root cause.
When it comes to the solution, a few bullet points which would help greatly improve your future proposals:

We're generally against platform-specific code (e.g. Platform.OS === "android" conditions), instead we're following the Platform-Specific File Extensions model to implement platform-specific code when necessary.
Whis suggestion ties into the one above ^ and it's about looking out for already-existing platform-specific files and suggest solutions which implement changes there instead of adding additional code / logic when not necessary.
Regarding the alternative solution, again, we're trying to keep the code DRY (don't repeat yourself) and this also means that we're mindful of adding more code to the codebase like effects and conditions when these are not absolutely necessary.
It's always a good idea to look into the involved components existing props which could help you come up with a more holistic solutions.

@Krishna2323 Thank you for your proposal. I'm afraid the RCA is only partially correct as in: that's the component that displays the bug of this issue on Android: Native. I initially thought your main solution (2.) would be the right way to go here but upon further testing I noticed that the solution would create regressions in other parts of the distance expense flow where the same component is displayed.
Note: The alternative solution would also break things since it would completely remove BlockingView instead of replacing it with PendingMapView as the main solution suggests and is demonstrated in the video below.
Here's the current vs your solution (2.) preview on Android: Native:

Android: Native

Current
Proposal 1

The same thing happens on web-based and all other platforms:

MacOS: Web
https://github.com/Expensify/App/assets/56457735/8e0cd9e7-1fbb-42e8-a13e-9f9c7d50f180

Note: In order to get the Map Pending state you have to simulate not having the mapbox access code available. I did this locally by changing return !isOffline && !!accessToken && !!currentPosition in MapView.tsx / MapView.website.tsx to !accessToken (falsy).

The problem with this issue originates in the ConfirmedRoute component where DistanceMapView splits into the following logic:
DistanceMapView
┣ index.android.tsx (which contains separate logic for isMapReady compared to MapView.tsx)
┃ -> MapView.tsx
┃ -> BlockingView.tsx
┣ index.tsx (no separate logic, goes straight to MapView.tsx)
┃ -> MapView.tsx
similarly for the MapView component:
MapView
┗ index.tsx (used by all platforms except Web, mWeb and Desktop)
┗ index.website.tsx (used only on Web, mWeb and Desktop)
The reason why we notice the issue easier on Android: Native is because the MapBox can take longer to initialize with the accessToken compared to Web / mWeb / Desktop which are all web based platforms and the mapbox is faster.
But nonetheless - the issue is present on all platforms.

[!Note]
For now we're still looking for a proposal which can address the issue on all platforms, without breaking other parts of the distance request flow.

cc @Expensify/design Since we're dealing with more complicated logic here when it comes to UI / UX, some of you might want to chime in and let us know what we're expecting here since the current expected result doesn't make much sense if you ask me.

@ikevin127 Thanks for your feedback. I've updated my RCA and proposed an alternative solution. Please review them when you have a chance. Link to the proposal https://github.com/Expensify/App/issues/43420#issuecomment-2173284262

Thanks for the updates, I will check again after we get some feedback from the design team regarding the expected result from a UI point of view.

[!note]
I'd suggest not to update your proposals until we get some feedback from the design team in order to not waste time with solutions which might not align with the final design requirements.

@ikevin127, I don't think the issue is present on all platforms, are you able to reproduce on other platforms?

No, I am not able to reproduce on other platforms following the same steps.
However, as I mentioned above, because of the way in which mapbox is initialized:

The reason why we notice the issue easier on Android: Native is because the MapBox can take longer to initialize with the accessToken compared to Web / mWeb / Desktop which are all web based platforms and the mapbox is faster.
But nonetheless - the issue is present on all platforms.

What I mean by this is that on web-based platforms, the mapbox initialization is almost instant and is persisted: once initialized in the distance confirmation page, when request is made and added to the report preview -> the map is already initialized so we won't see the map pending view in the report preview -> the map will load right away.
But if for whatever reason (mapbox service down, bad accessCode, or initialization will timeout due to bad internet / no internet) -> the user will notice the same issue on web:

MacOS: Web
https://github.com/Expensify/App/assets/56457735/8e0cd9e7-1fbb-42e8-a13e-9f9c7d50f180

I mentioned how this can be tested (if one has access to the codebase locally):

Note: In order to get the Map Pending state you have to simulate not having the mapbox access code available. I did this locally by changing return !isOffline && !!accessToken && !!currentPosition in MapView.tsx / MapView.website.tsx to !accessToken (falsy).

Given this, my objective here is to fix the issue for all platforms, even if the testers won't be able to verify the fix on web-based platforms.

Note: In the meantime, I'll try to find steps to reproduce the issue on the other platforms as well, but I think we should fix it regardless.

Unless @francoisl or anybody else disagrees and thinks we should only fix it on Android: Native because only there we can actually reproduce it due to differences in mapbox initialization between platforms.

Hmm, I thought we already took care of this with this PR 🤔
This video shows the expected behavior. The preview should show the placeholder map image until we have the actual map to display—then the actual map preview should replace the placeholder image. The "map pending" text should never show in the preview. Does that make sense?

https://github.com/Expensify/App/pull/39243#pullrequestreview-2092746776 shows the expected behavior

The video you mentioned is in offline mode. Our current bug is in online and only encountered in Android, where the "Map pending" text still appears.

The video you mentioned is in offline mode. Our current bug is in online and only encountered in Android, where the "Map pending" text still appears.

I understand, I'm just saying that video shows the expected behavior if we don't have the route map, regardless of whether you're offline or not. If we don't have the full map, we should show the map placeholder icon until we do, and then swap the placeholder for the route map.

Summary
Given dannymcclain's input from https://github.com/Expensify/App/issues/43420#issuecomment-2176136920 and https://github.com/Expensify/App/issues/43420#issuecomment-2176518358 we can conclude the expected result:

The "map pending" text should never show in the expense preview. "Preview" means 1, 2, 3 or more expenses:

See the preview variations

1
2
3

Point (1.) from above should apply regardless of being online or offline.

While map is pending, the expense preview should show the following map placeholder icon:

|  |
Conclusion
Given the 🟢 from design, I think the expected result is clear enough and if still interested -> the proposals should be updated to reflect the updated expected result.

I have also added option 3 in my proposal but I still prefer option 1 as the solution.

@Krishna2323 The reason I wasn't comfortable with option 1 code-wise is because I'm against the idea of prop-drilling components. But given that the solution works I think the other details can be worked on during PR.
Here's what I need in order for the solution to be accepted:

Pass requireRouteToDisplayMap to ConfirmedRoute's DistanceMapView component as a prop and use it only in index.android.tsx something like this:

{!isMapReady && (
    <View style={[styles.mapViewOverlay, overlayStyle]}>
         // MAKE SURE TO ADD A COMMENT HERE EXPLAININMG WHY WE DID THIS
        {requireRouteToDisplayMap ? (
            // MAKE SURE THIS IS STYLED CORRECTLY SO WE DON'T GET REGRESSIONS ANYWHERE IN DISTANCE FLOW
            <PendingMapView isSmallerIcon />
        ) : (
            <BlockingView
                icon={Expensicons.EmptyStateRoutePending}
                title={translate('distance.mapPending.title')}
                subtitle={isOffline ? translate('distance.mapPending.subtitle') : translate('distance.mapPending.onlineSubtitle')}
                shouldShowLink={false}
                iconColor={theme.border}
            />
        )}
    </View>
)}

Make sure the change is styled accordingly since currently the styling looks bad on different stages of the preview:

1
2
3

@truph01 I tested your alternative solution and it would introduce serious regressions in the distance request flow due to the usage of interactive prop which I think it's not the right prop to use for fixing our issue, see the video below:
Note: Ignore the report loading error as that's not related to your solution.
https://github.com/Expensify/App/assets/56457735/1331abfc-8922-494e-ab84-34af670c48dd

Thanks for working with us to get this to a desirable final result 🎉

@Krishna2323's updated proposal looks good to me. Root cause was identified correctly and the updated main solution (1.) fixes the issue as per the updated expected result agreed upon with the design team.
The solution tests well and looks great on both themes:

Android: Native (preview)

Dark
Light

🎀👀🎀 C+ reviewed
PR Notes
Please make sure to add the code-comment as suggested in https://github.com/Expensify/App/issues/43420#issuecomment-2179522570:
{!isMapReady && (
    <View style={[styles.mapViewOverlay, overlayStyle]}>
         // MAKE SURE TO ADD A COMMENT HERE EXPLAININMG WHY WE ADDED THIS LOGIC // <- HERE
        {requireRouteToDisplayMap ? (
            // MAKE SURE THIS IS STYLED CORRECTLY SO WE DON'T GET REGRESSIONS ANYWHERE IN DISTANCE FLOW
            <PendingMapView isSmallerIcon />
        ) : (
            <BlockingView
                icon={Expensicons.EmptyStateRoutePending}
                title={translate('distance.mapPending.title')}
                subtitle={isOffline ? translate('distance.mapPending.subtitle') : translate('distance.mapPending.onlineSubtitle')}
                shouldShowLink={false}
                iconColor={theme.border}
            />
        )}
    </View>
)}

[x]  [@ikevin127] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/27395
[x]  [@ikevin127] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/27395/files#r1667770691
[x]  [@ikevin127] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x]  [@ikevin127] Determine if we should create a regression test for this bug.
[x]  [@ikevin127] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal
Precondition: User is member of a workspace.

Open the workspace chat.
Create more than 1 distance expenses.
Right after creating the distance expense, in report IOU preview -> verify that the pending icon is shown (see reference) as opposed to the map pending text.

Do we agree 👍 or 👎.

[x]  [@ikevin127] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/27395
[x]  [@ikevin127] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/27395/files#r1667770691
[x]  [@ikevin127] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x]  [@ikevin127] Determine if we should create a regression test for this bug.
[x]  [@ikevin127] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal
Precondition: User is member of a workspace.

Open the workspace chat.
Create more than 1 distance expenses.
Right after creating the distance expense, in report IOU preview -> verify that the pending icon is shown (see reference) as opposed to the map pending text.

Do we agree 👍 or 👎.

Works for me. Opened an internal issue to create the regression tests – https://github.com/Expensify/Expensify/issues/411499

cc @JmillsExpensify Friendly bump for payments!

@JmillsExpensify I paid this given the contributor pinged about it - I'll file the regression test proposal as well given I am already here.

Thanks so much @greg-schroeder


Run cargo update on the runtime lockfiles and the SDK lockfile

If CI fails, commit the necessary fixes to this PR until all checks pass. If required, update entries in crateNameToLastKnownWorkingVersions.

Semver check failure is expected, merging


Add rewind batch feature to the BatchEventProcessor.

When processing an event, if an exception is thrown that could be recoverable via outside influence. Throw a RewindableException that will replay all the events in the current batch.

Documentation has been added to give an explanation on a useful use case for batch rewind.


pythonPackages.black: raise ulimit during checkPhase

Description of changes
Should hopefully fix failing builds.
ZHF: #199919
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg build python39Packages.black

I'm confused. Isn't the linked hydra failure on x86_64-linux but the change in this PR would only apply on darwin?

I've had similar issues with polaris on x86_64-linux when the build host is also doing other tasks. The same fix worked there, both locally and on hydra. As i understand it, any spawned process inherits the cumulative ulimit of the parent process, and ulimit resets it.

I think this isn't getting reviewed because everyone's confused over what problem you're trying to solve. You link to a successful build. Also, in the commit message and PR you refer to it as pythonPackages.black - pythonPackages is the python 2.7 package set (for which black doesn't exist). This should refer to python3Packages.

I did restart the failing build and it did succeed immediately. The builders have a max open files limit of ~1 million, so I'm not sure about the solution. But what confused me more that an x86_64-linux build fail was linked and the supposed fix is applied to darwin only.


Add window check to msal-guard

This PR:

Updates MsalGuard to account for server-side rendering
Updates Angular Universal SSR doc with better guidance
Updates angular11-sample-app with check before http call

Coverage decreased (-0.004%) to 85.69% when pulling 134f8c479fbf16e6ce1eaa686002b82a2a0550c8 on angular-window-fix into 9eab0f4be2be20dd828a033941fbd9a30fb92f1e on dev.

Codecov Report

Merging #3664 (859a1d5) into dev (e57cbfc) will decrease coverage by 0.03%.
The diff coverage is 50.00%.

Flag
Coverage Δ

*Carryforward flag

msal-angular
94.66% <50.00%> (-1.40%)
:arrow_down:

msal-browser
84.64% <ø> (ø)

Carriedforward from e57cbfc

msal-common
83.41% <ø> (ø)

Carriedforward from e57cbfc

msal-core
73.02% <ø> (ø)

Carriedforward from e57cbfc

msal-node
78.35% <ø> (ø)

Carriedforward from e57cbfc

msal-react
93.77% <ø> (ø)

Carriedforward from e57cbfc

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

lib/msal-angular/src/msal.guard.ts
89.39% <50.00%> (-4.26%)
:arrow_down:


Creation of contributing file

There is no contributing file -> the file named contributon.md only contains contributors
let me know

there is a contribution file https://github.com/akshitagupta15june/Face-X/blob/master/Contribution.md

there is a contribution file https://github.com/akshitagupta15june/Face-X/blob/master/Contribution.md


Controller Teleport Hint Broken

On the Vive Pro with SteamVR 2.2 I can't get the teleport hint to work. In the sample interactables scene when it cycles through the hints the trackpad never highlights. Other hints work (trigger/grip). Testing with a fresh import and the default input actions.
I saw this related issue but it doesn't seem to be resolved: https://github.com/ValveSoftware/steamvr_unity_plugin/issues/126

I turned on debugging for the hints and this is what I get:
<b>[SteamVR Interaction]</b> Hints: Looking for action: Teleport
Action localized origin: Right Hand Vive Controller Trackpad
Found component: scroll_wheel for Teleport
Found componentTransform: scroll_wheel (UnityEngine.Transform). buttonTransform: scroll_wheel (UnityEngine.Transform)
Found 1 renderers for Teleport
	scroll_wheel

Unity 2017.4.17f1
SteamVR Unity Plugin 2.2
SteamVR 1.2.10
Vive Pro

I am also getting a similar issue with the latest version of SteamVR. The trackpad is not highlighting for me with the hints component on Vector2 actions bound to the trackpad. When I enable debugging for it, the trackpad (on Vive) is returning "scroll_wheel" from GetRenderModelComponentName.
There are no actions bound to the scroll wheel functionality of the trackpad.

I have the same issue. Are there any workarounds?


bug: automated release of client fails

Describe the bug
releasing the python client using release-please fails with
Error: release-please failed: Not Found

The exact cause is unknown but this an existing issues #840 with release-please may be related.
This does not block manual release of the client python package and does not the release of the frontend component.
To Reproduce
Steps to reproduce the behavior:

commit a PR to main
view the results of the release GHA.

Screenshots
Full run output
Run googleapis/release-please-action@v4
  with:
    manifest-file: release-please.manifest.json
    config-file: release-please.config.json
    target-branch: main
    token: ***
    repo-url: chanzuckerberg/cryoet-data-portal
    github-api-url: https://api.github.com/
    github-graphql-url: https://api.github.com/graphql
    fork: false
    include-component-in-tag: false
    skip-github-release: false
    skip-github-pull-request: false
    changelog-host: https://github.com/
22s
Run googleapis/release-please-action@v4
Running release-please version: [1](https://github.com/chanzuckerberg/cryoet-data-portal/actions/runs/10513117165/job/29129370951#step:2:1)6.12.0
❯ Fetching release-please.config.json from branch main
❯ Fetching release-please.manifest.json from branch main
✔ Building releases
✔ Building strategies by path
❯ frontend: node
❯ client/python/cryoet_data_portal: python
❯ Fetching release-please.config.json from branch main
❯ Fetching release-please.manifest.json from branch main
✔ Building pull requests
✔ Building strategies by path
❯ frontend: node
❯ client/python/cryoet_data_portal: python
❯ Fetching frontend/package.json from branch main
✔ Collecting release commit SHAs
❯ release search depth: 400
❯ Fetching releases with cursor undefined
⚠ Found release tag with component '', but not configured in manifest
⚠ Found release tag with component '', but not configured in manifest
⚠ Found release tag with component '', but not configured in manifest
⚠ Found release tag with component '', but not configured in manifest
⚠ Found release tag with component '', but not configured in manifest
⚠ Found release tag with component '', but not configured in manifest
⚠ Found release tag with component '', but not configured in manifest
⚠ Expected 2 releases, only found 0
⚠ Missing 2 paths: frontend,client/python/cryoet_data_portal
❯ looking for tagName: web-v1.24.0
❯ found: web-v1.24.0 b27094f5b14663f5f103d4d363498076ee1c675f
❯ looking for tagName: cryoet-data-portal-python-client-v3.0.3
❯ found: cryoet-data-portal-python-client-v3.0.3 c[15](https://github.com/chanzuckerberg/cryoet-data-portal/actions/runs/10513117165/job/29129370951#step:2:16)6f8e1b6b9b469d3d71405a2f0eeef5c439995
❯ release for path: frontend, version: 1.24.0, sha: b27094f5b14663f5f103d4d363498076ee1c675f
❯ release for path: client/python/cryoet_data_portal, version: 3.0.3, sha: c156f8e1b6b9b469d3d71405a2f0eeef5c439995
✔ Collecting commits since all latest releases
❯ commit search depth: 500
❯ Set(2) {
  'b27094f5b14663f5f103d4d363498076ee1c675f',
  'c156f8e1b6b9b469d3d71405a2f0eeef5c439995'
}
❯ Fetching merge commits on branch main with cursor: undefined
❯ Fetching merge commits on branch main with cursor: b27094f5b14663f5f103d4d363498076ee1c675f [24](https://github.com/chanzuckerberg/cryoet-data-portal/actions/runs/10513117165/job/29129370951#step:2:25)
❯ Fetching merge commits on branch main with cursor: b27094f5b14663f5f103d4d363498076ee1c675f 49
❯ Fetching merge commits on branch main with cursor: b[27](https://github.com/chanzuckerberg/cryoet-data-portal/actions/runs/10513117165/job/29129370951#step:2:28)094f5b14663f5f103d4d363498076ee1c675f 74
✔ Splitting 79 commits by path
✔ Building candidate release pull request for path: frontend
❯ type: node
❯ targetBranch: main
❯ commits: 0
✔ Considering: 0 commits
✔ No commits for path: frontend, skipping
✔ Building candidate release pull request for path: client/python/cryoet_data_portal
❯ type: python
❯ targetBranch: main
❯ commits: 6
✔ Considering: 6 commits
❯ component: cryoet-data-portal-python-client
❯ pull request title pattern: undefined
❯ Fetching client/python/cryoet_data_portal/pyproject.toml from branch main
❯ finding files by filename: version.py, ref: main, prefix: client/python/cryoet_data_portal
❯ running plugin: Merge
✔ Merging 1 pull requests
⚠ pullRequestTitlePattern miss the part of '${scope}'
⚠ pullRequestTitlePattern miss the part of '${component}'
⚠ pullRequestTitlePattern miss the part of '${version}'
✔ Looking for open release pull requests
✔ found 1 open release pull requests.
✔ Looking for snoozed release pull requests
✔ found 0 snoozed release pull requests.
✔ PR https://github.com/chanzuckerberg/cryoet-data-portal/pull/1071 remained the same
✔ Empty change set provided. No changes need to be made. Cancelling workflow.
Error: release-please failed: Not Found

note: we don't have automation to release the python package to pyPI. requires manual work to upload package to pyPI. impact is on internal eng team

This is working now!


hello

My first time using freeCodeCamp

Pre-Submission Checklist

[ ] Your pull request targets the staging branch of freeCodeCamp.
[ ] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[ ] You have only one commit (if not, squash them into one commit).
[ ] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[ ] Small 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 change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[ ] Tested changes locally.
[ ] Closes currently open issue (replace XXXX with an issue no): Closes #XXXX

Description

Please do not spam the issue tracker.
The issue tracker is for reporting bugs only.


grt: fix loops during repair_antennas

This looks to check for an identical segment in the route which seems odd. Even with a loop I wouldn't expect an identical segment.

In the issue reported by James Cherry, the final routing had these two segments:
(x0, y, l) --> (x1, y, l)
(x1, y, l) --> (x0, y, l)
So the segments were not exactly the same, they had different "directions". But I've changed the code to treat them like the same. With this update, I don't see the disconnected net error anymore.

I'm ok with this as a solution for grt but I think we should investigate why such a loop in the segments exists in the first place.  (@jjcherry56 for comment).


Download QR Code

I am grateful for this library as I have generated QR Code. But I need to download it too , so according to the documentation I used react-native-fs but it's showing me some errors which is not resolving. May be this is happening because my project is in expo. Can anyone please help me with the codes.
note:- there is an expo-file-system too which can be used instead of react-native-fs
here is my QrGenerator.js file , I have a great hope from you guys please help me in this.
QrGenerator.txt
here I am Pasting my code from QrGenerator.js
import React, { useState } from 'react';
import {
Box, Heading, Center, Button,
NativeBaseProvider, Text, Input, FormControl, Stack
} from "native-base";
import QRCode from 'react-native-qrcode-svg';
// import { StyleSheet, View, Dimensions } from 'react-native';
export default function QrGenerator({ navigation }) {
const [input, setInput] = useState("name");

const handleChange = (text) => {
    setInput(text);
}

return (
    <NativeBaseProvider>
        <Center mt="20">

            <Heading color="#0891b2">VCard Generator</Heading>

            <Box marginTop="5">
                <QRCode value={input + "\n"} size={170} /> {/* Here is the QRCode from react-native-qrcode-svg */}
            </Box>
        </Center>

        <Box mt="5" width="90%" margin="auto" >

            <Text fontSize="md">Fill below parameters to generate your QR Code</Text>

            <FormControl>
                <Stack>
                    <FormControl.Label >Text</FormControl.Label>
                    <Input placeholder='Enter Text' onChangeText={text => handleChange(text)} />
                </Stack>
            </FormControl>

            <Button mt="5">Download QR</Button>
        </Box>
    </NativeBaseProvider>
)

}

did you find a way?


Try no new comp name

I am just testing something here. I had this set as a draft, but did not want that to stop any tests from being run.

/retest

Superseded by #693


AP=0 and AR=0 in COCO datasets

During the training, AP is always 0.
btw, I use the COCO datasets.
And I use this command, python tools/train.py -f exps/example/yolox_s.py -b 64 --fp16 -o
Average forward time: 1.82 ms, Average NMS time: 0.65 ms, Average inference time: 2.47 ms
Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.000
Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.000
Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=100 ] = 0.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=  1 ] = 0.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets= 10 ] = 0.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.001
Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.001
Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.002

Is this the result for 10th epoch?

yes @Joker316701882

@kikyoyang Please check your data. We can get proper training results using the current code.

@kikyoyang Please check your data. We can get proper training results using the current code.

I used the COCO datasets. The data is ok.

请问你解决这个问题了吗？我也遇到这个问题。

@kikyoyang  I meet the same problem, I fix this by delete the cache data and rerun the experiments.
cache data path: path_to_coco/img_resized_cache_train2017.array
please refer to the following logs:

@kikyoyang I meet the same problem. I fix this by delete the cache data and rerun the experiments, which works for me!
cache data path: path_to_coco/img_resized_cache_train2017.array
please be aware of the following logs:

thanks, your answer is exactly for what I encountered.
what if after I deleted it, it still ask to cache again?


chore(renovate): add renovate.json

this needs renovate enabled via the UI, but we can land the config file for when it does.


Letter Case

Is this comparison case sensitive?
How do I preform a case insensitive comparison?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 27504e8c-568f-ee52-542c-5ca87c2906db
Version Independent ID: dc8f9282-90ce-60e0-402c-ea8829b6334e
Content: Use expressions in conditions in Power Automate - Power Automate
Content Source: articles/use-expressions-in-conditions.md
Service: power-automate
Sub-service: cloud-flow
GitHub Login: @MSFTMan
Microsoft Alias: deonhe

@Matt-03 To do case insensitive comparison, just use the ToLower() method on each string that you want to compare. This forces them all to lower case.


Add Arg Parser

Add support for argument parsing.
Most likely using the "clap" library.
Arguments to consider supporting:
- choose speed mode
- choose map/obstacle setting
- play area size (grid size)
- play speed (initial, increase rate)
- keybindings (arrows, wsad...)
- color scheme (would need to add this as a feature first)
- open to more suggestions

Initial argument parser added, see this PR #15
Will continue to incrementally add the rest of the options mentioned here.

Keybindings for movement have been added, see this PR #16
What's left:

choose speed mode (constant, increasing)
choose map/obstacle setting
color scheme (would need to add this as a feature first)
open to more suggestions

Random thought: you may want to consider having a config file, such that settings/options would be determined by resolving in this order: arguments passed, otherwise default to config file, otherwise default to built-in defaults.


Add AdminPageCloudFrontDomainName and StatusPageCloudFrontDomainName outputs

This is to be able to use with external DNS provider (e.g. Cloudflare) if using custom URLs.
This is also to address some internal work on a terraform module that I may contribute back eventually to address https://github.com/ks888/LambStatus/issues/100.

Thank you for the PR! It seems new outputs are similar to existing AdminPageCloudFrontURL and StatusPageCloudFrontURL outputs. Can you tell me the difference?

The difference is that if you set cutom URLs those outputs contain those URLs instead of CloudFront domain name so they cannot be used to create those needed DNS CNAMEs. Introduced outputs always output CloudFront domain names so can be used to create DNS records.

Sorry for my late reply. I see, you want to automatically set up the DNS setting. Thank you for the clarification.


Connection error messages

This is acctually for https://github.com/CesiumGS/cesium-unreal/issues/314  - further details added there

Thanks @javagl!


MikroOrm 5 support

Can you please add the support of MikroOrm 5?

https://github.com/SoftwareBrothers/adminjs-mikroorm/pull/10 this should do

Hi! @vincentwinkel @rubiin @dziraf  any news? Now I constantly receive "NoResourceAdapterError: There are no adapters supporting one of the resource you provided"


Watson cluster integreity can be disrupted when unexpected clients connect

I noticed Watson cluster accepts connections not coming from the configured peer instance, and can be confused by these additional connections, causing the cluster to lose coherency. I prevented this in my own code, but the way I did it may not be suited to sharing. I will try to generalize it so that I can offer a pull request. Otherwise, we would have to rely on firewall rules to ensure only the expected peer can connect.

Hi @brudo , would an appropriate solution in your opinion be to allow the caller to specify a source IP to permit connections from, and reject others?  I should be able to make this change fairly quickly

Hi@jchristn yes I think so. In fact, see my branch restrict-nonpeer-connections for work in progress. I did not yet implement a way to revert to old behavior, where any IP can connect and participate as peer. Default behavior, if no alternate peer IPs are specified, is that connections from the IP address given in PeerIp are accepted as peers, while all others are non-peers. Also, at this point there is no direct communication to or from any connected clients other than the last valid peer, but the connection is left open. Note, I will be at a conference this week so I may not get to finish it right away but feedback is welcome!

Hi @brudo I added a new constructor to WatsonTcpServer which includes a List for permitted IPs.  The AcceptConnections() loop will check the list (if it's not null/empty) against the source IP of the incoming connection, and close the connection if there is no match.  I committed/pushed and updated NuGet.  One optimization I just realized I could make is to move the comparison outside of the task that is started.  Please let me know what you think!

(and then I can expose it upward to WatsonCluster)

OK, that's a good way. I didn't want to touch WatsonTcp for this but am glad to take advantage if you do. Please disregard the branch I created for now and I'll integrate your changes and build on that instead.
Feedback: please have the constructor accept an IEnumerable of strings instead of a List for more flexibility. Just my opinion. But construct a collection from it (e.g. List / Set / etc.) so that you only enumerate it once.

Awesome suggestion, I'll update this right now and then integrate into WatsonCluster.  Will let you know as soon as it's done.

Hi @brudo this is integrated into WatsonCluster.  It automatically populates the permitted IP list with the IP of the peer and passes it directly to WatsonTcpServer, so there are no changes to the APIs in WatsonCluster.  I gave it a test and it worked well.  Thanks again for the help sir, and re-open if you encounter any issues!

Actually come to think of it, in my case I did want non-peers to be able to connect and "listen in". But I'll add that back later in a more controlled way once your update is complete.

I see the final update... Great!

If you like I can add another constructor that allows the list of permitted IPs to be overridden, will only take me a few minutes - cool?

Just committed this and updated NuGet, please let me know if this helps!

Cool! I'll see what I can do with it next week. Now in my worst case I will be able just bypass the filtering at WatsonTcp level, although most likely I will be able to take advantage.

Just a follow up, our in-house modified version of WatsonCluster is now updated with a server allow list based on these changes in WatsonTcp. Thanks again!

Glad to hear @brudo !  Thanks for the feedback and help!


Lua: Allow display of listchars other than eol

indentLine would leave listchars like tab displayed, rather than replace them with its lines as indent-blankline is doing right now. With the tab listchar set, I think showing it is the expected behavior.
Similarly, when a line contains only spaces and trail is set, I would expect the list character to be displayed instead of the indent lines.

both sound like good options 👍
I'll look into it

I've managed to bisect the issue, and the regression start in d9f682939ac8a59b1784802b0c69e82177a4972a.
However, I'm not proficient enough in lua to further pinpoint what's up there.

It's not really a regression. The whole functionality of the plugin changed.
Before using the new overlay virtual text, the plugin only did empty lines. And by definition, those didn't have trailing spaces.

Personally i'd prefer to keep being able to see the difference between tabs and spaces using the listchars defined characters.
Would it be possible to accomplish this in any way ?

hey,
I am a comp sci student. I wanted to start open-source contributions.
Can I work on this issue/feature request?

Hey @ek234,
sure! Always grateful for help.
What I am thinking for this feature is an option that is a list of all listchars that should take priority over indent-blankline virtual text.
So for example let g:indent_blankline_foo = ["eol", "tab"] would not overwrite tab characters and end-of-line characters.
I think we need to care about eol, tab, space, trail and lead, but I haven't looked into it too much.
It will get complicated when there are cases where a line has a mix of showing indentation lines and not showing them. Like mixed indentation with spaces and tabs. It might make sense to just not support this for now, not sure.
If you start working on this, open a draft PR early. I'll review the code and help you, so you don't spend a lot of time on something I won't want to merge.

Hi @lukas-reineke,
Even I thought of implementing it with a similar approach. However, an issue came up.
Consider in listchar that tab is set to anything other than '|\ ', for example, let it be "tab:>-" [like in the nvim documentation].
Then all the non blank lines (which actually contain tabs for indenting) will use this scheme and the plugin wouldn't write over it. But, the blank lines will still use the regular hard coded scheme.

--->---this is what it would look like
|     |     |
--->---the line just above this line was blank
--->---more text
--->---more text

Rather, I would suggest that plugin itself displays tabs as the user wishes. This way, the indent will be uniform.
Additionally, we don't have to worry about which char takes priority over the indent line.
Also, as eol is already considered in the plugin, it works perfectly for me.
I believe other listchars can be inserted in a way similar to eol.
tl;dr We can have the plugin display all the characters as defined by the user instead of checking if the underlying char is to be prioritised or not. This will increase customisability too.

I still feel displaying the list chars in the virtual text will be more consistent.
One way of implementing it would be kinda similar to list char in nvim itself. we can store the pattern that the user wants. for eg, for '|.|', we can just split the tab display into these three parts: display '|' once then display '.' as many times as needed then display '|'.
The problem with not displaying virtual text over list char is that we will still have to display the same tab list chars for the blank lines. (as I mentioned in the last message). so we'll have to implement both the things.
Please let me know if you're fine with this approach.
Thanks

I still feel displaying the list chars in the virtual text will be more consistent.

Like I said, that is one of the 2 options. I am fine with this, whatever is easier to implement.

we can store the pattern that the user wants

This I don't agree with. Read the value from listchars directly. No custom pattern to store anything.

Like I said, that is one of the 2 options. I am fine with this, whatever is easier to implement.
ahh cool:)
This I don't agree with. Read the value from listchars directly. No custom pattern to store anything.
ohh that would be nice. I didn't think of that at all.

Any updates on this? Not being able to see if a file's indentations contains tabs characters is a real pain.

I haven't really had time to take a stab at it, myself. I imagine @lukas-reineke hasn't had a chance yet either.

It's a pretty big task. I am not sure when I have time for this.
PR's are always welcome. I will do my best to help if someone wants to try this again.

I was wondering why my tab char wasn't shown so I had fought with neovim for an hour and finally arrived to the right place 😅.
(digging of it allow me to learn more about list and listchars though!)
I think I'm going to stop loading ident-blankline.nvim until this issue gets resolved. All the good luck with the task!

Not sure if just mentioning issues sends out notifications, so I'll just quickly leave a comment here. I took a stab at this in https://github.com/lukas-reineke/indent-blankline.nvim/pull/187
Some testing/feedback would be appreciated.


Python package information doesn't include license information

https://github.com/aws/jsii/blob/32bc117025cc7fd7ee48bab0d524ab8f772abb77/packages/jsii-pacmak/lib/targets/python.ts#L1159

I submitted a draft PR (#712) to this effect. The caveat right now is that the only licensing information I have available is the SPDX short identifier, which I can turn into the SPDX license name -- however it appears that the PyPI classifiers don't use the standard license names, as can be observed at https://pypi.org/classifiers/, the License :: OSI Approved :: Apache Software License would seem to be the only option to match all versions of the Apache licenses...
I haven't investigated other licenses as of yet, but this might require manual mapping which doesn't sound pleasant... 😩


ERROR with project / select

I used this program for an assignment and for practice.
Now i am using it again, with the same exact codes i had before, and the outputs are different now.
Please fix it!

Hi.
Can you provide an example query to replicate your issue?

@OmarDawoud: Thank you! Feedback is very much appreciated. Please try to be as precise as possible. That helps the developers to find and fix problems fast.
Simon Tatham wrote a nice article about how to write efficient bug reports. You may find that helpful.


Question regarding "Performance acceleration"

The final decision asks "Do you require performance acceleration" and pushes towards Front Door + Application Gateway (as opposed to just App Gateway) if the answer is "yes".  There's no clear rational or explanation as to why this decision is there, as it's my understanding that Front Door doesn't provide any performance acceleration improvements over App Gateway, and instead the only appreciable benefit to Front Door over App Gateway is the prior is Global and currently can't pass traffic directly to resources with a Private IP.
Given that many will take this flow chart as gospel, it seems sensible to expand on the rational behind the decisions if at all possible.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 936c85d9-c953-2def-c399-77cbda32b24d
Version Independent ID: 9173c834-2c40-9169-b13b-6273759a2c8a
Content: Overview of Azure load-balancing options - Azure Application Architecture Guide
Content Source: docs/guide/technology-choices/load-balancing-overview.md
Service: architecture-center
Sub-service: guide
GitHub Login: @doodlemania2
Microsoft Alias: pnp

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 372372

Hello @AndyHerb,
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

Thank you for bringing this up as a point of clarification we can make.  I've went ahead and added the definition of what is meant by that to the text below the flow chart.  Good point on the ambiguity that previous had.  This change should be live in about 24 hours.


Requesting coding help

Can someone take a look at this please, I'm stuck. If you recall I wanted to allow users to /login and order themselves minions and the minions pick up their skin and name. Issue here https://github.com/Megabyte918/MultiOgar-Edited/issues/645
Well this works great and thank you for the help on it. But I found a bug or security issue, I noticed that a user can /login and order minions for other users up to 50. Basically same as Mod or Admin. With the current code, Admin is also limited to 50. So I figured why not try to create different specifications based on Roles. But I'm stuck and the server crashes once I connect.
test working file here: http://tutorialscow.esy.es/
/login guest
/login moder
/login admin
As Guest you can give other players up to 50 minions, just like Admin and Mod
Working file
PlayerCommand.txt
Non working roles file
PlayerCommand_Roles.txt

Hey cowpits , can you add me on skype : vincentcoc , I have to speck with you .. :)

ok is this you?  ๖νιиƈє༻,

@cowpits Add an if statement that prevents users without the required permissions to give minions to others.

@Megabyte918 yeah I think I got it. Thanks for the tip but I could only get either Moder or Admin to work but not both. If I get both working when Moder does /minion remove #ID for a player, it loops and keeps giving Moder minions, although it removes the minions for the other player, lol.
I'll figure it out eventually. Thanks again.


New user system breaks Apache Basic Auth

I previously had v0.3.3 deployed via docker and working correctly behind an Apache reverse proxy and basic auth. Since the implementation of the new user system, (using v0.4.4) Apache Basic Auth endlessly requests credentials. If I disable Basic Auth, everything works perfectly, including the new user system.
The site does load partially with Basic Auth enabled, but with 401 errors for site assets which I'm guessing is triggering the continuing requests for credentials. I'm assuming the new authentication system is creating the conflict, but I'm not sure how to resolve.
Thanks for your insight.

I had the same problem before.
After checking, I found out that the reason was the domain of Apache's reverse proxy not using HTTPS.
The user system needs to carry cookie information named 'token'.
If HTTPS is not enabled. After a successful login, you can check it in the developer tools that the 'set-cookie' in the 'Response Headers' of the login request has a warning message:
This attempt to set a cookie via a Set-Cookie header was blocked because it had the "Secure" attribute but was not received over a secure connection.

The reverse proxy is delivering content over HTTPS. Internally it's referring to the docker container via http://localhost:port. What else might I be missing?

Doesn't seem to work. I've been fiddling with multiple configurations with no luck. BasicAuth seems to be overriding any attempt to correctly pass the authorization Bearer in both directions.
Here's what I'm really getting at though: I could be misunderstanding the user system, but it appears that on a public domain, anyone that registers has access to the application. The BasicAuth ensured I was the only one who could access it. I'd like to be able to restrict or disable account creation entirely.

I figured out a hacky solution, but I have no idea whether or not it's actually a secure configuration.
I think unless you have an existing session, with the configuration below, any attempts to access any of the subfolders directly (https://mydomain.com/chat/new for example) will redirect you to the login page where the  block will then enforce basic auth.
I think restricting sign up would still make sense though. I appreciate your feedback and love the project!
<Location /api>
	Satisfy any
	ProxyPass http://localhost:3080/api
	ProxyPassReverse http://localhost:3080/api
</Location>

<Location />
	AuthType Basic
	AuthName "Restricted Area"
	AuthUserFile /etc/apache2/.publicpasswd
	Require valid-user

	ProxyPass http://localhost:3080/
	ProxyPassReverse http://localhost:3080/
</Location>

Many thanks, @matthieu-kr! I think the paths that should be blocked are /login, /register, and /api/auth/register. This last one is important since it is the API service that is used to register a user.

Per your suggestion, I updated the configuration to specifically block the /api/auth/register path and added the LocationMatch to ensure that it doesn't inherit the Satisfy any directive. Everything seems to be working as expected. Basic Auth now consistently prompts for credentials and stops after a successful login. It resumes prompting upon logout.
Thanks for your help!
<Location /api/auth/register>
    AuthType Basic
    AuthName "Restricted Area"
    AuthUserFile /etc/apache2/.publicpasswd
    Require valid-user
</Location>

<LocationMatch "^/api(?!/auth/register)">
    Satisfy any
    ProxyPass http://localhost:3080/api
    ProxyPassReverse http://localhost:3080/api
</LocationMatch>

<Location />
    AuthType Basic
    AuthName "Restricted Area"
    AuthUserFile /etc/apache2/.publicpasswd
    Require valid-user

    ProxyPass http://localhost:3080/
    ProxyPassReverse http://localhost:3080/
</Location>

@matthieu-kr Thanks for supporting and using the project! I'm going to forward this to @danorlando who did all the work on the user system.
@lsalazarm99 Thank you for your assistance on this!

With the latest update, you can now close registration by either omitting this env variable or setting it to false
# ALLOW_REGISTRATION=true
# ...or
ALLOW_REGISTRATION=false

See the latest release here for more info, as there are many breaking changes with the update.
https://github.com/danny-avila/LibreChat/releases/tag/v0.5.0


With Sybase jTDS, keywords are put in square brackets instead of quotation marks

Description
Basically the same as #7755: Columns which names are keywords are put in square brackets instead of quotation marks.
DBeaver Version
DBeaver Community Edition 23.1.1
Operating System
Windows 10 64 Bit
Database and driver

Server: Sybase Adaptive Server Enterprise/<IP_ADDRESS>/P/EBF 10869 ROLLUP/Sun_svr4/OS 5.6/1996/32bit/FBO/Fri Feb 21 03:19:58 2003
Driver: jTDS 1.3.1

Steps to reproduce

Create a table with columns named date and action
Open the table’s context menu in the navigator and select “Read data in SQL console”
Created query is SELECT [date], [action] FROM foo.bar.baz; and a “Confirm query execution” prompt appears

What should happen:

Created query is SELECT "date", "action" FROM foo.bar.baz;, “Confirm query execution” prompt does not appear (actually the quotation marks are not necessary in this case)

Additional context
No response

Hello @maffe
Could you please provide a full screen of your connection page + the result of the "test connection" button?
Example:

@maffe Thank you for the report


Repo & development updates, Docs rewrite, CI improvements, and more

It's been nearly 4 years since I created #291 changing this repository from Vows to Jest and the Node ecosystem has definitely not been standing still so I figured "let's create another general update PR".

Migrated from Jest to Vitest.
Vitest is a new test runner powered by Vite and is much faster than Jest. Like was the motivation for Jest in #291 it has TypeScript support built in, adding to that, in the future we can even make the TypeScript tests even better through Type Testing

Updated from Vuepress v1 to Vuepress v2. This brings with it maaaaaaannnyy fixes, improvements, enhancements and so on. Both because I put quite a lot of effort in the rewrite, as well as because in general this updates from Vue 2 to Vue 3 and Vuepress has gone through many changes as well. Some notable things:

Dark mode! Wooh! Like modern websites it responds to the OS preference, and there is a button to switch.
The live demo now uses monaco editors (vscode's editor)! instead of codemirror. So a much better editing experience.
The "Twitter follow" button is now a custom component rather than a widget, which means it will no longer be blocked by content blockers (like adblockers)
Everything uses the Vue3 composition API instead of the Vue2 options API. This makes Vue scripts waaay easier to read.
Dropped stylus in favour of raw CSS and SCSS because Vuepress v2 only so/so seems to support Stylus. They dropped support for Stylus for customizing the theme, switching to SCSS. I figured for consistency it'd be best to drop it entirely.

I am not sure how the current production website is deployed. You can reference https://v2.vuepress.vuejs.org/guide/deployment.html to learn how to deploy the new site. That said, if it's through GitHub pages then do let me know because these days it is possible to use a GH Action to deploy directly, you don't even need a gh-pages branch anymore then.

Migrated from yarn to npm. Previously in #291 I made a change to prefer yarn.lock but I noticed that since then you have added scripts that use npm so I am making the assumption that is your package manager of choice. To ensure the repo is not in a 50/50 state of package managers I now fully switched the repo over to using npm in all references.

Updated Husky to v8 and changed how it is configured to how it should've been above v4 (https://typicode.github.io/husky/#/?id=migrate-from-v4-to-v8). The old way of doing it through package.json hasn't been working since v4, so the current repo wasn't protected by husky at all.

Updated various dev dependencies. I did not update rollup because it included the breaking changes of v3.0.0 which I'm not sure if they will affect the project. Of note is that Fuse's engines are set to Node 10, whereas Rollup v3 targets Node >= 14.

Added eslint-config-prettier to the eslintrc file, it was a dev dep that was never loaded before.

Ran prettier on the whole repo

Updated the GitHub issue templates to YAML style so issues are form-like inputs. These can be previewed on my branch: https://github.com/favna/Fuse/blob/chore/dev-migrations/.github/ISSUE_TEMPLATE/bug_report.yml

Split tsconfig.json into 2 files, 1 for src and 1 for test. This is to ensure that each set can have its own dedicated config, and it's also a slight improvement for the TS language services of IDEs because they only load the ones for the files opened. A negligible difference for this repo, but it's a difference nonetheless.

Add .prettierignore file to allow running npx prettier --write . without modifying files that shouldn't be formatted with Prettier. Just something I ran into when doing point 6.

Update GitHub workflows

Updated the files in dist by running npm run build to update their code with all the changes above

This PR is now ready for review. Sorry not sorry for the file count haha, that's mostly because of the vuepress changes.

Oh wow this is huge. Will review it now. Thanks!


[Bug] Wrong first column height

Bug report
What I did
Just upgraded from the 5.x to 6.x and carefully switched to the Tabler theme.
What I expected to happen
The table columns should be working as before.
What happened
The leftmost column should have the same height of the other columns.
The HTML of my columns is:
// column 1

    Printer #1

23%

// column 2

Milano

    etichette totali: 0

// column 3

        assegnato: 0

        in stampa: 0

        stampato: 0

What I've already tried to fix it
Removing the display: flex from the first td.
I know this is not a definitive solution since it might break something else.
Context to replicate the bug - Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
backpack/basset: 1.1.1
backpack/crud: 6.1.5
backpack/generators: v4.0.2
backpack/pro: 2.0.10
backpack/theme-tabler: 1.0.8 (carefully cleaned the basset cache)

Hmmm I've just looked at all CRUDs in our demo - https://demo.backpackforlaravel.com/admin/article - and can't find any one where the first column is broken @EmanueleCoppola .

From what I can see... those are custom columns you have there, right? So the problem (and the solution) should be in your custom code, right? It's not Backpack adding the flex in the first column, it's your custom column?
Let us know if you think I'm mistaken here. I'm wrong at least once a day 😉
Cheers!

The issue I'm having seems to be related with this line of code.
What occurs is that, in cases where a custom column exceeds the height of others, all cells adopt the regular table behavior, aligning with the taller column, except for the first cell, which remains smaller (probably because of it's display property).
I didn't encounter this issue in the v5 release, but the problem emerged after the v6 update.

Hey @tabacitu I just tried what he meant here.
I tested this on v5 & v6(all themes)
I got the same output on all v6 themes, so it's not related to the themes.
In v5 same thing was doing fine.
Then I had a doubt, is it related to bulk action checkboxes? No, it's not.
What difference is observed on v5 vs v6 output.
V5:
When we put content which occupies height in any column. All columns are centred✅

V6:

When we put content which occupies height in any column, All columns are centred except the first column❌
When we put content which occupies height in the first column. The column's height doesn't stretch as expected. ❌

Conclusion
YES, It's related to the v6 CRUD LIST view.
Solution
Looking for it. @pxpm do you have any idea?

@karandatwani92 I recall we had this issue before (in v5), I searched a bit around and were able to find it: https://github.com/Laravel-Backpack/CRUD/issues/4746
Maybe we need the same solution promatik came up in that issue ?
Cheers

Found: the issue is in src/resources/assets/css/common.css. Don't know which element exactly but I'm about to find it.

PR submited #5275 🎉
Thanks @EmanueleCoppola for the help.🙌

I've just merged the fix into main branch. I will tag a new version in a few moments with some other fixes we are working on 🙏
Thanks @EmanueleCoppola


Ternary conditional operator

"If this do this else this" is such a common operation and the reason ternaries exist in almost all modern languages. Them missing is especially egregious in languages with syntactic whitespace. (4 lines for a simple choice.)
Python's ternary syntax is both readable and non-cryptic.
a if test else b

+1, would love to see this implemented. Though - speaking as someone more accustomed to js and php-type syntaxes - I would be thrown off by having one of the possible outcomes declared before the actual testing condition is. Maybe some adaptaion of condition ? a : b (where a is the "if true" option and b is the "else"/"if false") would work?

Does the python thing work as a value? ie can you do "var a = b if test
else c" ?
On Sun, May 24, 2015 at 8:20 PM, jack<EMAIL_ADDRESS>wrote:

+1, would love to see this implemented. Though - speaking as someone more
accustomed to js and php-type syntaxes - I would be thrown off by having
one of the possible outcomes declared before the actual testing condition
is. Maybe some adaptaion of condition ? a : b (where a is the "if true"
option and b is the "else"/"if false") would work?
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/1961#issuecomment-105078271.

I'm hoping to add this feature, I'm very experienced. Would I just be looking at  gd_tokenizer?

@punto- Yes that's really what it's primarily for.

will do for 2.1

That's great news!

i had a different suggestion that matches gd script more
if test a else b
this is very similar with what you already have in gd script but you can also use it like this
var = (if test a else b)

I think we should go with the Python syntax when implementing this, since most of GDScript's syntax is based on python, better stick to it.

I add my voice to this choir
Also agree with @akien-mga , it has to follow python syntax, even if it is a little strange compared to others.

+1 for python syntax

+1 for python syntax

I would like to throw CoffeeScript syntax into the mix:
a = if test then x else y

I'm down for anything as long as it can be used to assign value to a variable or return
var a = x if test else y
return x if test else y

+1 for python syntax, this would be a great feature

GDScript is (largely incorrectly) refered to as a Python-like language. Therefore, deviating from the Python syntax in a core language feature like this would break the principle of least surprise.

@Fiona  I agree with you and I even proposed and alternative. Ironically after further consideration I also support it for this ternary operator due to it's simplicity.
I originally proposed "if test a: else b" likely : must be used to figure out where test ends which requires 7 components
Bojidar's proposal "if test then x else y"  requieres 6 components
Python's one "x if test else y"  only requires 5 components
Quickest one has to be "test ? a : b" Java but does not really fit with GD Script and is hard to read

Would love to see any implementation of ternary.  Don't have an opinion on syntax so long as there is one to use. :)  But, if I had to throw my hat on one, I'd say Java style.

Not critical for the upcoming 2.1, so moving to the next milestone.

2.2 possible? :)

Going to try doing it :smile:

+1 for the Python style for GDScript, though I also like the C/C++/Java-styled operator.

I would prefere "if test : x else y" syntax in trollworkout's post for a mix of least components and semantic easiness. I like to see condition(s) are followed by outcomes and well separated as in a if block which is better for a short syntax for an if block IMO.

I decided to do the Python variation (apologies to supporters of the other options), since it looks way simpler to implement than all variations that have if before the test. Plus, as @Fiona said, it is the "principle of least programmer work surprise"

I preferred the C-style option out of habit but, hey, this will do nicely. Thanks @bojidar-bg !

I don't know if this is the right place to ask, but is there any chance we will get C-style ternary operators? I'm just wondering, since GDscript allows for C-style operators like || && ! << >> & | ^. So I think it would just make sense to add ?: as well. But I cannot add it myself...

@kintrix007 There are no plans to add C-style ternary operators. To make the language easier to learn and reason about, we'd prefer having only one obvious way to write ternary statements :slightly_smiling_face:

@kintrix007 There are no plans to add C-style ternary operators. To make the language easier to learn and reason about, we'd prefer having only one obvious way to write ternary statements 🙂

I am actually searching the same thing, the syntax is not so obvious to me, or to others, I've looked it up several times, so some consistency with other languages would be nice so we don't have to remember another way of doing the same thing basically.

To me and others, I am sure, it's completely the wrong order.
it should be [condition] [true] [false] regardless of the language, in all contexts you check a condition first, before returning the results.
it seems stupid to me to do [true] [condition] [false] and makes little sense to me.

@RevoluPowered GDScript's ternary operator syntax is already consistent with Python's.

Just adding my experience as someone who is new to godot and learning gdscript.
First thing I tried instinctively was var a = (condition ? true_part : false_part) and it errored out.
Then I tried var a = if condition true_part else false_part, and it also errored out. So I gave up and converted my one-liner to the multi-line if/else version, as I thought that was the only supported way.
Then out of curiosity I tried searching for it and I landed on this page. I would have never guessed this weird reordering. Ok it's not the end of the world and I can remember it for now, but it's backwards and confusing. I can easily see myself in the future trying to get back to gdscript and going like "hold on, what was the ternary operator syntax again...?"
Anyways, that's my experience.

Edit: besides, are conditions short-circuited in gdscript

I'd benchmark it to check that :slightly_smiling_face:
You can use OS.delay_msec() to add a blocking delay.

Well, nobody said the current behavior is perfect. If you want to push for a change in Godot 4.0 / GDScript 2.0, feel free to open a proposal on https://github.com/godotengine/godot-proposals.
The way it was implemented sought to reuse as many keywords as possible, since GDScript is a white-space and keyword-friendly language. In addition, it was implemented that way for feature party with Python. You can check the discussion above.
However, the fact that it was decided that way for reasons, does not mean it is useful. As you say yourself, there are cases in which the syntax is clunky and unreadable. It still works and isn't buggy in some major way, it is just less than useful.
And that's what proposals are for--creating a place where discussions as to the merit of exact solutions to usability problems and missing features can be held. So, if you want to improve Godot, to make it better for the next time someone stumbles upon ternaries, and if you have a general idea of how that can happen, I would strongly recommend going there and opening a proposal.
Further discussion on a closed issue is counter-productive, as it is way harder to find it afterwards. :)

For reference, I can confirm the slow function here is skipped on 3.3rc6:
extends Node2D

func slow_function() -> int:
	OS.delay_msec(20)
	return -42

func _ready() -> void:
	var begin := OS.get_ticks_usec()

	for i in 1000:
		var test := 6
		var result := slow_function() if test > 5 else 42

	print("Took %d microseconds" % (OS.get_ticks_usec() - begin))


change FT/NFT standards

update references and usage of FT/NFT smart contracts according to https://github.com/onflow/flow-ft/pull/131

I updated this in https://github.com/onflow/docs/pull/645


Unable to build Xcode project that was generated by CMake

The project is here and requires CMake to build.
To reproduce do the following:

git clone https://github.com/Hyland/DocumentFilters
cd DocumentFilters
run cmake -G "Xcode" ., which generates an Xcode project
Run the xcodebuild configuration wizard, which appears to succeed
Attempt to build a scheme

Following the above I get a "The project is missing some details" error. The project builds fine in Xcode.
I have narrowed it down to the settings.json missing the "productName", "appPath" and "bundleId" keys. If I manually add them, then the build works as expected. I believe the project has static and dylib targets that don't define a bundle ID and don't have an app.

It seems like it requires adding support for static libraries. As you mentioned, as a workaround, you can add those 3 values to settings.json. They are used only to run/debug the app which is not available for static libraries anyway.

I added support for static library :) Now it should work as expected. Just run the wizard again.


Update live-events-outputs-concept.md

Adding info about Live Transcriptions. Changes to be published on/after Nov 13th

@anilmur : Thanks for your contribution! The author(s) have been notified to review your proposed change.


OEML Go websocket SDK

As discussed by email, OEML streaming SDK for Golang

Implements all message types up to Oeml version 1.8789
Implements new MessageReject error handling
Provides example code and tests in separate main file
Provides installation guide in Readme
Provides development guide in Readme

Differences to the generated REST API:

float32 was changed to float64 in all structs due to noticeable rounding errors in account balances(!)
Reimplemented JSON marshal methods in all outgoing messages to mitigate an embarrassing golang bug [1]
Added missing message type field to all outgoing messages.

Known issues:

Not threat safe

Because of the PR size (>6k LoC) and limited time, I was unable to test all corners of the implementation, but I'm willing to provide limited updates in case the implementation breaks or the API makes breaking changes.
[1] https://github.com/golang/go/issues/27179

Great job marvin!

Your welcome.
Marvin Hansen
https://www.linkedin.com/in/marvin-hansen/
On Mon, Jul 26, 2021, 15:28 Artur Pietrzyk @.***> wrote:

Merged #117 https://github.com/coinapi/coinapi-sdk/pull/117 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coinapi/coinapi-sdk/pull/117#event-5066588692, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFYR7XARGSZF7RZ6W6H6YKLTZUFAVANCNFSM5A7P3O5Q
.


Multiple interfaces with kube-vip

Is your feature request related to a problem? Please describe.
Current issue: I have an RKE2 Cluster that I am deploying and it needs to interact with multiple networks on separate interfaces that will be segregated networks. We have been testing kube-vip to enable HA without extra external load balancing.
Describe the solution you'd like
Enable kube-vip to use a statically assigned IP for multiple interfaces and networks.
Describe alternatives you've considered
We have considered using external loadbalancing services however we are trying to limit the amount of overhead and requirements for our system, and would prefer to use built in components of kube-vip and nginx to handle this.
Additional context
example:
We have a 3 controller 3 worker cluster deployed on ubuntu 22.04 and on each we have the following interfaces with an IP from that subnet assigned.
eno1: Subnet <IP_ADDRESS>/29 (kube-vip is on this interface with a static ip)
eno2: Subnet <IP_ADDRESS>/29
eno3 Subnet <IP_ADDRESS>/29
There is a network device that is segregated and can only talk on eno2 subnet, it cannot handle inputting multiple IPs to reach the same service hosted on the cluster, so we would like to use a single IP ingress with kube-vip to then balance that to the cluster.

I'd love to learn how to address this.


Theme extension in railwind.config.ron

Below is example of theme extension from a tailwind.config.js file I am putting together. I would love to have similar functionality like this in with railwind so I can stop using Javascript once and for all.

I'd be to help contribute with this feature as it is something I would like to use to put my personal website together. Is this feature doable, and what changes would enable such functionality?

I'm currently working on this feature and on rewriting the parser using nom. If you look into railwind::config, there's the main extendable config that gets passed down to the parser.
On the other side, I would need a parser that takes either the tailwind.config.json or a railwind.config.ron file and expands the relevant config if that makes sense?


build(main): release 1.1.5

:robot: I have created a release beep boop
1.1.5 (2024-09-27)
Bug Fixes

MenuButton: Fix prompt for 0 or '' values (4dd2063)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.1.5
:sunflower:


Question: Is EXIF data removed?

Hey there. Just have a quick question. I didn't dive into the code (yet), so before I do I thought to ask: is EXIF data removed after compression?

deleted

#164


Website link to RSS feed is broken

When I visit https://project-oak.github.io/rust-verification-tools/, there's a "subscribe via RSS" link that points to https://project-oak.github.io/rust-verification-tools/feed.xml (which according to View Source is actually a <link type="application/atom+xml", not RSS). However, RSS or Atom, this link is broken and points to a 404.
Is it possible to fix the link to enable subscribing in a RSS reader?

:+1:


Incorrect local JetStream store directory when $XDG_DATA_HOME is defined

Observed behavior
When running nats server run --jetstream, the Store Directory shown in the JetStream banner output repeats the ".local/share/" path components:
[76897] [INF] Starting nats-server
[76897] [INF]   Version:  2.10.0
[76897] [INF]   Git:      [not set]
[76897] [INF]   Name:     nats_development
[76897] [INF]   Node:     HhEiCaYr
[76897] [INF]   ID:       NAIC2EHHLBZSS6H7KZWZRP5U6UVHDB4FNORZT2S733PJMLLGXNIHN6KW
[76897] [INF] Using configuration file: /var/folders/yt/lwq8py0n4qxbkr2mg_nwg1y00000gn/T/nats-server-run-3707093925.cfg
[76897] [INF] Starting JetStream
[76897] [INF]     _ ___ _____ ___ _____ ___ ___   _   __  __
[76897] [INF]  _ | | __|_   _/ __|_   _| _ \ __| /_\ |  \/  |
[76897] [INF] | || | _|  | | \__ \ | | |   / _| / _ \| |\/| |
[76897] [INF]  \__/|___| |_| |___/ |_| |_|_\___/_/ \_\_|  |_|
[76897] [INF]
[76897] [INF]          https://docs.nats.io/jetstream
[76897] [INF]
[76897] [INF] ---------------- JETSTREAM ----------------
[76897] [INF]   Max Memory:      48.00 GB
[76897] [INF]   Max Storage:     1.20 TB
[76897] [INF]   Store Directory: "/Users/erhhung/.local/share/.local/share/nats/nats_development/jetstream"
[76897] [INF] -------------------------------------------
[76897] [INF] Listening for client connections on <IP_ADDRESS>:58137
[76897] [INF] Server is ready

Apparently, if the XDG_DATA_HOME environment variable isn't set, the default store directory is correct:
$ echo $XDG_DATA_HOME
/Users/erhhung/.local/share
$ unset XDG_DATA_HOME

[76864] [INF] Starting nats-server
[76864] [INF]   Version:  2.10.0
[76864] [INF]   Git:      [not set]
[76864] [INF]   Name:     nats_development
[76864] [INF]   Node:     HhEiCaYr
[76864] [INF]   ID:       NB5IF7UJZDNZKE4IRDCNVO4AHU64EJXFEJV5L7VRAPRF4BV7ZRIWQVD3
[76864] [INF] Using configuration file: /var/folders/yt/lwq8py0n4qxbkr2mg_nwg1y00000gn/T/nats-server-run-3427777350.cfg
[76864] [INF] Starting JetStream
[76864] [INF]     _ ___ _____ ___ _____ ___ ___   _   __  __
[76864] [INF]  _ | | __|_   _/ __|_   _| _ \ __| /_\ |  \/  |
[76864] [INF] | || | _|  | | \__ \ | | |   / _| / _ \| |\/| |
[76864] [INF]  \__/|___| |_| |___/ |_| |_|_\___/_/ \_\_|  |_|
[76864] [INF]
[76864] [INF]          https://docs.nats.io/jetstream
[76864] [INF]
[76864] [INF] ---------------- JETSTREAM ----------------
[76864] [INF]   Max Memory:      48.00 GB
[76864] [INF]   Max Storage:     1.20 TB
[76864] [INF]   Store Directory: "/Users/erhhung/.local/share/nats/nats_development/jetstream"
[76864] [INF] -------------------------------------------
[76864] [INF] Listening for client connections on <IP_ADDRESS>:58137
[76864] [INF] Server is ready

Expected behavior
Per the XDG Base Directory Specification, the XDG_DATA_HOME environment variable is optional: if $XDG_DATA_HOME is either not set or empty, a default equal to $HOME/.local/share should be used.
That implies that if the variable is defined in the shell, apps that use it should treat it as if it were the default value—in this case, it was explicitly set by in my "~/.bash_profile" by export XDG_DATA_HOME="$HOME/.local/share"—and not repeat the ".local/share/" path components.
Server and client version
$ nats --version
0.1.1

Host environment
macOS Sonoma 14.2.1
Steps to reproduce
No response

I see two places where the $XDG_DATA_HOME environment variable is used differently:

cli/util.go: func xdgShareHome() (string, error)
cli/server_run_command.go: func (c *SrvRunCmd) dataParentDir() (string, error)

with the latter function adding the superfluous path components.


[port 2.0]: Avoid two leading slashes in the request to create a new file

Cherry-pick: https://github.com/microsoft/FluidFramework/pull/22563

Approved


Virus Total Alert for ESP-Flasher-x86.exe File

File: v.1.4.1

False Alarm generated from PyInstaller. Closing since duplicate.


FindBugs Issues

Hi All Devs,
Please use findbugs to check your code and try to fix all default findbugs errors and warnings asap.
Thanks,
Zhang, Pengshan(David)

I'll work on this :)

I use findbugs find that ModelBasicConf and ModelConfig define equals() function without override hashCode() . I think it's better to override it. Is it ok to use the automatically generated hashcode() by eclipse ?

Thanks, Xiaobin. Please check your PR CI results.

Thanks, Xiaobin. Already merge your PR to develop 0.2.5 branch.