HADOOP-17669. Port HADOOP-17079, HADOOP-17505 to branch-3.3

HADOOP-17079 in branch-3.3 has a number of trivial conflicts.
HADOOP-17505 is a clean cherrypick.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
21m 27s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 15 new or modified test files.

_ branch-3.3 Compile Tests _

+0 :ok:
mvndep
15m 31s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
20m 46s

branch-3.3 passed

+1 :green_heart:
compile
17m 13s

branch-3.3 passed

+1 :green_heart:
checkstyle
2m 52s

branch-3.3 passed

+1 :green_heart:
mvnsite
7m 42s

branch-3.3 passed

+1 :green_heart:
javadoc
7m 24s

branch-3.3 passed

+1 :green_heart:
spotbugs
12m 27s

branch-3.3 passed

+1 :green_heart:
shadedclient
16m 11s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 27s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
5m  0s

the patch passed

+1 :green_heart:
compile
16m 44s

the patch passed

-1 :x:
javac
16m 44s
/results-compile-javac-root.txt
root generated 84 new + 1898 unchanged - 3 fixed = 1982 total (was 1901)

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
2m 45s

root: The patch generated 0 new + 904 unchanged - 7 fixed = 904 total (was 911)

+1 :green_heart:
mvnsite
7m 38s

the patch passed

+1 :green_heart:
javadoc
7m 22s

the patch passed

+1 :green_heart:
spotbugs
13m 51s

the patch passed

+1 :green_heart:
shadedclient
16m 24s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
17m 13s

hadoop-common in the patch passed.

-1 :x:
unit
188m 37s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
unit
5m 44s

hadoop-hdfs-httpfs in the patch passed.

-1 :x:
unit
15m 18s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs-rbf.txt
hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
unit
23m 44s

hadoop-yarn-server-nodemanager in the patch passed.

-1 :x:
unit
88m 10s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
unit
4m 26s

hadoop-mapreduce-client-hs in the patch passed.

+1 :green_heart:
asflicense
1m  6s

The patch does not generate ASF License warnings.

540m  0s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestReconstructStripedFile

hadoop.hdfs.server.blockmanagement.TestUnderReplicatedBlocks

hadoop.hdfs.qjournal.server.TestJournalNodeRespectsBindHostKeys

hadoop.hdfs.TestReconstructStripedFileWithRandomECPolicy

hadoop.hdfs.server.namenode.ha.TestStandbyCheckpoints

hadoop.hdfs.tools.offlineEditsViewer.TestOfflineEditsViewer

hadoop.hdfs.TestReconstructStripedFileWithValidator

hadoop.hdfs.server.federation.router.TestRouterRpcMultiDestination

hadoop.yarn.server.resourcemanager.placement.TestUserGroupMappingPlacementRule

hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairSchedulerPreemption

hadoop.yarn.server.resourcemanager.TestRMHATimelineCollectors

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2959/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2959

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 26e0ce48c23a 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / c65022c258f200483e228f958b3f3ed4c64513af

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~18.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2959/1/testReport/

Max. process+thread count
3225 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs hadoop-hdfs-project/hadoop-hdfs-httpfs hadoop-hdfs-project/hadoop-hdfs-rbf hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2959/1/console

versions
git=2.17.1 maven=3.6.0 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 46s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 15 new or modified test files.

_ branch-3.3 Compile Tests _

+0 :ok:
mvndep
11m 51s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
20m 41s

branch-3.3 passed

+1 :green_heart:
compile
17m 23s

branch-3.3 passed

+1 :green_heart:
checkstyle
2m 44s

branch-3.3 passed

+1 :green_heart:
mvnsite
7m 48s

branch-3.3 passed

+1 :green_heart:
javadoc
7m 24s

branch-3.3 passed

+1 :green_heart:
spotbugs
12m 36s

branch-3.3 passed

+1 :green_heart:
shadedclient
16m 22s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 28s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
5m  9s

the patch passed

+1 :green_heart:
compile
16m 37s

the patch passed

-1 :x:
javac
16m 37s
/results-compile-javac-root.txt
root generated 84 new + 1952 unchanged - 3 fixed = 2036 total (was 1955)

+1 :green_heart:
blanks
0m  1s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
2m 43s

root: The patch generated 0 new + 904 unchanged - 7 fixed = 904 total (was 911)

+1 :green_heart:
mvnsite
7m 45s

the patch passed

+1 :green_heart:
javadoc
7m 29s

the patch passed

+1 :green_heart:
spotbugs
13m 53s

the patch passed

+1 :green_heart:
shadedclient
16m 26s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
16m 54s

hadoop-common in the patch passed.

-1 :x:
unit
188m 38s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
unit
6m  2s

hadoop-hdfs-httpfs in the patch passed.

+1 :green_heart:
unit
15m 52s

hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
unit
23m 17s

hadoop-yarn-server-nodemanager in the patch passed.

-1 :x:
unit
90m 31s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
unit
4m 29s

hadoop-mapreduce-client-hs in the patch passed.

+1 :green_heart:
asflicense
1m  3s

The patch does not generate ASF License warnings.

518m 37s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.datanode.TestBPOfferService

hadoop.hdfs.server.datanode.TestDataNodeErasureCodingMetrics

hadoop.hdfs.TestStripedFileAppend

hadoop.hdfs.TestDFSInotifyEventInputStreamKerberized

hadoop.yarn.server.resourcemanager.placement.TestUserGroupMappingPlacementRule

hadoop.yarn.server.resourcemanager.TestRMHATimelineCollectors

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2959/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2959

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 835870b24c46 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / c65022c258f200483e228f958b3f3ed4c64513af

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~18.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2959/2/testReport/

Max. process+thread count
3422 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs hadoop-hdfs-project/hadoop-hdfs-httpfs hadoop-hdfs-project/hadoop-hdfs-rbf hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2959/2/console

versions
git=2.17.1 maven=3.6.0 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

All failed tests do not repro. I'll merge this PR.


pull request for e2e test at 01/06/2017 16:15:18

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Prisma 4.0 no longer exports _dmmf

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
Here is the diff that solved my problem:
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 415d5ee..695c45b 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -10,7 +10,7 @@ class Database extends adminjs_1.BaseDatabase {
         this.client = prisma;
     }
     resources() {
-        const dmmf = this.client._dmmf;
+        const dmmf = this.client._baseDmmf;
         const { modelMap } = dmmf;
         if (!modelMap)
             return [];
@@ -21,7 +21,7 @@ class Database extends adminjs_1.BaseDatabase {
     }
     static isAdapterFor(prisma) {
         var _a;
-        const dmmf = prisma._dmmf;
+        const dmmf = prisma._baseDmmf;
         return !!(dmmf === null || dmmf === void 0 ? void 0 : dmmf.modelMap) && !!Object.values((_a = dmmf === null || dmmf === void 0 ? void 0 : dmmf.modelMap) !== null && _a !== void 0 ? _a : {}).length;
     }
 }
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 4e7edb2..795455f 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -23,7 +23,7 @@ class Resource extends adminjs_1.BaseResource {
         const { model, client } = args;
         this.model = model;
         this.client = client;
-        this.enums = this.client._dmmf.enumMap;
+        this.enums = this.client._baseDmmf.datamodelEnumMap;
         this.manager = this.client[(0, helpers_1.lowerCase)(model.name)];
         this.propertiesObject = this.prepareProperties();
     }
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 75bf014..73213e5 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -47,8 +47,8 @@
  * const run = async () => {
  *   const app = express()
  *
- *   // `_dmmf` contains necessary Model metadata. `PrismaClient` type doesn't have it included
- *   const dmmf = ((prisma as any)._dmmf as DMMFClass)
+ *   // `_baseDmmf` contains necessary Model metadata. `PrismaClient` type doesn't have it included
+ *   const dmmf = ((prisma as any)._baseDmmf as DMMFClass)
  *
  *   const admin = new AdminJS({
  *     resources: [{
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 9a8cf02..cda20d1 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -48,8 +48,8 @@
  * const run = async () => {
  *   const app = express()
  *
- *   // `_dmmf` contains necessary Model metadata. `PrismaClient` type doesn't have it included
- *   const dmmf = ((prisma as any)._dmmf as DMMFClass)
+ *   // `_baseDmmf` contains necessary Model metadata. `PrismaClient` type doesn't have it included
+ *   const dmmf = ((prisma as any)._baseDmmf as DMMFClass)
  *
  *   const admin = new AdminJS({
  *     resources: [{

This issue body was partially generated by patch-package.

I have this same issue

updated version of the patch:
@adminjs+prisma+1.0.4.zip
solves enum not displayed correctly

Hi, can you open a pull request with these changes?

I've opened #15 with @younes0's patch.


listening to client.incoming() causes crash

  await for (final update in client.incoming()) {
      print(update.tdType);
    }

this part of your example causes this output

lib/main.dart: Warning: Interpreting this as package URI, 'package:tdlib/main.dart'.
[ 3][t 4][1576026651.009212255][Td.cpp:4657][#1][!Td][&td_requests]	Sending update: updateOption {
  name = "version"
  value = optionValueString {
    value = "1.5.1"
  }
}
[ 3][t 4][1576026651.009475946][Td.cpp:4657][#1][!Td][&td_requests]	Sending update: updateAuthorizationState {
  authorization_state = authorizationStateWaitTdlibParameters {
  }
}
[ 3][t 0][1576026651.024739504][Client.cpp:270][&td_requests]	Begin to wait for updates with timeout 2.000000
[ 3][t 0][1576026651.024761677][Client.cpp:276][&td_requests]	End to wait for updates, returning object 0 0x7ff4a0101a40
[ 3][t 0][1576026651.043768406][Client.cpp:270][&td_requests]	Begin to wait for updates with timeout 2.000000updateOption

updateAuthorizationState
[ 3][t 0][1576026651.043790102][Client.cpp:276][&td_requests]	End to wait for updates, returning object 0 0x7ff4a0102000
[ 3][t 0][1576026651.045000315][Client.cpp:270][&td_requests]	Begin to wait for updates with timeout 2.000000
[ 3][t 0][1576026653.047059774][Client.cpp:276][&td_requests]	End to wait for updates, returning object 0 (nil)

===== CRASH =====
si_signo=Segmentation fault(11), si_code=1, si_addr=(nil)
version=2.7.0-dev.2.1.flutter-a9c77229c2 (Wed Dec 4 11:26:20 2019 +0000) on "linux_x64"
thread=1895, isolate=main(0x30081e0)
  pc 0x00007ff4e3768eff fp 0x00007ff4d297e4b8 Unknown symbol
  pc 0x00007ff4e37678d5 fp 0x00007ff4d297e500 Unknown symbol
  pc 0x00007ff4e376770a fp 0x00007ff4d297e538 Unknown symbol
  pc 0x00007ff4e3766d89 fp 0x00007ff4d297e5a0 Unknown symbol
  pc 0x00007ff4e376688e fp 0x00007ff4d297e638 Unknown symbol
  pc 0x00007ff4e37299b7 fp 0x00007ff4d297e670 Unknown symbol
  pc 0x00007ff4e372b106 fp 0x00007ff4d297e6a8 Unknown symbol
  pc 0x00007ff4e376648f fp 0x00007ff4d297e730 Unknown symbol
  pc 0x00007ff4e3765ce8 fp 0x00007ff4d297e7c0 Unknown symbol
  pc 0x00007ff4e37299b7 fp 0x00007ff4d297e7f8 Unknown symbol
  pc 0x00007ff4e37653f0 fp 0x00007ff4d297e830 Unknown symbol
  pc 0x00007ff4e37651fe fp 0x00007ff4d297e868 Unknown symbol
  pc 0x00007ff4e37650c4 fp 0x00007ff4d297e8b0 Unknown symbol
  pc 0x00007ff4e3764d61 fp 0x00007ff4d297e8f0 Unknown symbol
  pc 0x00007ff4e376492b fp 0x00007ff4d297e930 Unknown symbol
  pc 0x00007ff4e3764827 fp 0x00007ff4d297e970 Unknown symbol
  pc 0x00007ff4e376462a fp 0x00007ff4d297e9b0 Unknown symbol
  pc 0x00007ff4e37289bd fp 0x00007ff4d297e9d8 Unknown symbol
  pc 0x00007ff4e3f0145c fp 0x00007ff4d297ea48 Unknown symbol
-- End of DumpStackTrace

If I delete this part then there is no error, but I don't have updates

So it seems to be crashing after sending the authorizationStateWaitTdlibParameters message and failing to see a response (when TDLib doesn't receive a response before the timeout, it just returns a null reference, which causes the segmentation fault). According to the TDLib documentation, you're supposed to respond to this by sending a setTdlibParameters command with the appropriate fields, which is illustrated in the README example right before the await-for-loop:
client.authorizationState.listen((state) {
  if (state is td.AuthorizationStateWaitTdLibParameters) {
    await client.send(client.tdlibParams);
  }
});

Now, while writing this response I noticed that I mistakenly wrote "authentication" instead of "authorization" in the README. This will be remedied immediately. I also recognize that TelegramClient.setTdLibParams is confusingly named, so I think I'll rename it to defineTdlibParams (since it doesn't actually send the setTdlibParameters command).
I can't see the rest of your code so if you are already handling the authorization state and aren't having any name confusion then you'll have to show me more of the code.

import 'package:tdlib/src/api/functions.dart';
import 'package:tdlib/tdlib.dart' as td;
import 'main.reflectable.dart';

void main() async {
  initializeReflectable();
  // Create a client
  final client = td.TelegramClient("/home/rafal/telegram/");
  client.setTdLibParams(
      apiId: MY_API_ID,
      apiHash: MY_API_HASH,
      deviceModel: "Nexus 5",
      systemVersion: "6.0.1");
  try {
    // Get a stream to handle changes to the authorization state
    client.authorizationState.listen((state) async {
      if (state is td.AuthorizationStateWaitTdlibParameters) {
        await client.send(SetTdlibParameters(client.tdlibParams));
      }
    });

    await for (var update in client.incoming()) {
      // Handle each incoming update
    }
  } finally {
    // Be sure to close the client
    client.close();
  }
}

There is my code which causes crashes. I cloned this repo and added this main file to lib/

Do you have any idea? I can't go forward anywhere with this bug.

I want to keep working through this but tentatively I'm struggling to recreate this crash. I've copied the logic of your code exactly, and while it does crash (because updates aren't being handled; see the terminal output below) it doesn't do so on the authorizationStateWaitTdlibParameters request. I've made a few uncommitted changes to this library recently that I'll push soon but none of them would require your code to change; TelegramClient.setTdLibParameters() will be deprecated but I'll be leaving it for a few more versions so it'll work exactly as it does now.
[ 3][t 4][1577388663.302275658][Td.cpp:4657][#1][!Td][&td_requests]     Sending update: updateOption {
  name = "version"
  value = optionValueString {
    value = "1.5.1"
  }
}
[ 3][t 4][1577388663.302403212][Td.cpp:4657][#1][!Td][&td_requests]     Sending update: updateAuthorizationState {
  authorization_state = authorizationStateWaitTdlibParameters {
  }
}
[ 3][t 0][1577388663.312212944][Client.cpp:270][&td_requests]   Begin to wait for updates with timeout 2.000000
[ 3][t 0][1577388663.312244177][Client.cpp:276][&td_requests]   End to wait for updates, returning object 0 0x558950b6fa70
[ 3][t 0][1577388663.329548359][Client.cpp:270][&td_requests]   Begin to wait for updates with timeout 2.000000
[ 3][t 0][1577388663.329573870][Client.cpp:276][&td_requests]   End to wait for updates, returning object 0 0x55895027c410

-------------------{ YOURS CRASHES HERE }-------------------

[ 3][t 4][1577388663.340855360][Td.cpp:3548][#1][!Td][&td_requests]     Receive request 1: setTdlibParameters {
  parameters = tdlibParameters {
    use_test_dc = false
    database_directory = "tdb"
    files_directory = ""
    use_file_database = false
    use_chat_info_database = false
    use_message_database = true
    use_secret_chats = true
    api_id = REDACTED
    api_hash = REDACTED
    system_language_code = "en-US"
    device_model = "Nexus 5"
    system_version = "6.0.1"
    application_version = "0.0.1"
    enable_storage_optimizer = true
    ignore_file_names = false
  }
}
[ 3][t 0][1577388663.340897083][Client.cpp:270][&td_requests]   Begin to wait for updates with timeout 2.000000
[ 3][t 4][1577388663.341100216][Td.cpp:4657][#1][!Td][&td_requests]     Sending update: updateAuthorizationState {
  authorization_state = authorizationStateWaitEncryptionKey {
    is_encrypted = false
  }
}
[ 3][t 4][1577388663.341127157][Td.cpp:4672][#1][!Td][&td_requests]     Sending result for request 1: ok {
}
[ 3][t 0][1577388663.341130495][Client.cpp:276][&td_requests]   End to wait for updates, returning object 0 0x55895027c450
[ 3][t 0][1577388663.342518806][Client.cpp:270][&td_requests]   Begin to wait for updates with timeout 2.000000
[ 3][t 0][1577388663.342553377][Client.cpp:276][&td_requests]   End to wait for updates, returning object 1 0x55895027a0b0
[ 3][t 0][1577388663.343151569][Client.cpp:270][&td_requests]   Begin to wait for updates with timeout 2.000000
[ 3][t 0][1577388665.345277548][Client.cpp:276][&td_requests]   End to wait for updates, returning object 0 (nil)

===== CRASH =====
si_signo=Segmentation fault(11), si_code=1, si_addr=(nil)
version=2.7.0 (Unknown timestamp) on "linux_x64"
thread=16538, isolate=main(0x5589502a6c00)
...

To test a bit further, I created a new project and included this project as a dependency in the pubspec instead of cloning it into the lib folder, again using the same code you gave me. Still works fine. I dunno if it'll change anything but if you want to try that, you can add this to your pubspec:
dependencies:
  tdlib:
    git: git://github.com/periodicaidan/dart_tdlib

And then add a build.yaml file in the same directory as your pubspec with the following:
targets:
  $default:
    builders:
      reflectable:
        generate_for:
          - lib/main.dart
        options:
          formatted: true

And you can still build the reflectables using pub run build_runner build.
This could also be a TDLib problem but it's hard to tell. If you can't figure anything out you may want to forward this issue to them.
Sorry for the delayed reply, I've been travelling and doing holiday festivites the past few days.

@periodicaidan Can you push somewhere the new project you created? So I can clone, change my tdlib path and try to run?

I just finalized and pushed it, you can give it a go now

@periodicaidan I don't see any new commits nor any new repositories on your github.

c700c6ffca72d9b1cd1dd2e613f8485c076ceaad

I executed it as Dart command line application and gives me the same error. What platform are you executing on?

I'm also executing it as a command line application. Dart VM version 2.7.0 on x86_64 Linux (Ubuntu).


Errors trying to connect to target

@c0r0n3r I can provide wireshark if needed in order to understand why the script fails despite having communication.
Output:
C:\Users\t>dheat --protocol tls gw.t.local
Network error oocuerd while checking whether Diffie-Hellman ephemeral (DHE) key exchange is supported by the server; uri="gw.t.local", error="connection to target cannot be established"
C:\Users\t>dheat --protocol tls gw.t.local
Network error oocuerd while checking whether Diffie-Hellman ephemeral (DHE) key exchange is supported by the server; uri="gw.t.local", error="no response received from target"

A pcap file would be so useful. Without there can be just guesses.

Here are 3 tests with different sets of Ciphers bound to the server. pcap is at link; there's a separate file for each test and named as such: https://drive.google.com/drive/folders/1tluvSlcSBumyKCswnjYF5qeUnOgitcQz?usp=sharing
1st test error
Error:
'''C:\Users\T>dheat --protocol tls gw.t.local
Network error oocuerd while checking whether Diffie-Hellman ephemeral (DHE) key exchange is supported by the server; uri="gw.t.local", error="no response received from target"'''
2nd test error:
Error:
'''C:\Users\T>dheat --protocol tls gw.t.local
Traceback (most recent call last):
File "c:\programs\python38-32\lib\runpy.py", line 193, in run_module_as_main
return run_code(code, main_globals, None,
File "c:\programs\python38-32\lib\runpy.py", line 86, in run_code
exec(code, run_globals)
File "C:\Programs\Python38-32\Scripts\dheat.exe_main.py", line 7, in
File "c:\programs\python38-32\lib\site-packages\dheater_main.py", line 346, in main
enforcer = DHEnforcerThreadTLS(args.uri, args.timeout, pre_check_result)
File "", line 15, in init
File "c:\programs\python38-32\lib\site-packages\dheater_main.py", line 102, in attrs_post_init
self.pre_check()
File "c:\programs\python38-32\lib\site-packages\dheater_main.py", line 264, in _pre_check
protocol_version = min(analyzer_result_versions.versions)
ValueError: min() arg is an empty sequence'''
3rd test error:
Error:
'''C:\Users\T>dheat --protocol tls gw.t.local
Network error oocuerd while checking whether Diffie-Hellman ephemeral (DHE) key exchange is supported by the server; uri="gw.t.local", error="no response received from target"'''

It seems that the reason of the error in 2nd test case is the fact that the server does not support DHE. Could you test it again with the latest version of dheater (0.2.4)?

@KyferEz please reopen if the issue still exists in the latest version.


Add UI tool to clone a rule in RuleTable

FEATUREREQUEST
Description:
A tool to clone an existing rule in the RuleTable would be a nice improvement. Users can start new rules on existing ones. This can be important in case of having complex symbolizers and/or filters.
Suggestion: Another button Clone Rule in the bottom bar of the RuleTable UI, which clones all selected rows (by checkbox).

We should keep in mind that we also have an alternative layout to the RuleTable. IMO cloning rules should also be applicable there.

Correct, if we want to have it, it should not be a problem to integrate this in the non-compact layout/view.


Special characters are encoded incorrectly in order templates

Actual Behavior
When I create an order template with the name “Interßhüp”, the name of the order template is displayed with Unicode characters instead of the correct characters

Expected Behavior
If I create an order template with the name “Interßhüp”, the name of the order template is displayed as I have created it.

Steps to Reproduce the Bug
Steps to reproduce the behavior:

Go to My Account -> Order Templates / Bestellvorlagen
Click on 'Create Order Template'
Give Order Template the name "Interßhüp" or a name with "ü"/"ö"/"ä" or "ß"
See wrong encoded order template name: "Interßhüp"

The name is transferred correctly to the backend in the PWA. However, the createOrderTemplate function in order-template.service returns an incorrectly encoded name from the backend
AB#99244

@paramaeder What PWA/ICM version were you using?
We could not reproduce it with the current PWA 5.2.

@shauke
PWA: 5.0.0
ICM: 7.10


Feature charts: Add longhorn 1.3.0 chart in 2.6

Longhorn has a coming release 1.3.0 which will be released by end of April or the beginning of May so we need to add this new release in the feature charts along with the coming Rancher 2.6.5.

Add 1.3.0 LH chart in feature chart 2.6 - Cluster Explorer
Add 1.3.0 LH chart in feature chart 2.6 - Cluster Manager/Catalog

cc @khushboo-rancher @yasker @MKlimuszka @SheilaghM @deniseschannon @kaxing @meldafrawi @yasker

https://github.com/rancher/charts/pull/1899

cc @rebeccazzzz @prachidamle @MKlimuszka

Since this is longhorn-related, assigning to @khushboo-rancher

Since this is longhorn-related, assigning to @khushboo-rancher

@khushboo-rancher is OOO now, assigned to @cchien816 who has verified the PR already, but we can revalidate the chart after https://github.com/rancher/image-mirror/pull/250 merged.

Validated the chart on Rancher v2.6.7-rc3 , both fresh install longhorn v1.3.0 and upgrade longhorn from v1.2.4 to v1.3.0 are working fine.

Longhorn 1.3.0 will be released as part of this PR


Cast session getting suspended due to “GCKConnectionSuspendReasonNetworkNotReachable”

Observed behavior: After starting the casting of music files, user enters the background. After a few minutes of playing the music file through cast, session gets suspended with the reason GCKConnectionSuspendReasonNetworkNotReachable, even though the sender device and cast device both are connected to a working Wi-Fi connection
Expected Behavior: The session should continue as the sender app has the background capability of playing audio and also with GCKCastOptions we have set the property suspendSessionsWhenBackgrounded to NO.

This is by design. The framework automatically suspends sessions when the sender app is going into the background and attempts to resume them when the app returns to the foreground (or is relaunched after an abnormal/abrupt app termination while a session was active).

HI @entertailion , Please check the April 12 release notes here https://developers.google.com/cast/docs/release-notes
In sender sdk for ios version 3.4 .
They have changed the behaviour. So I am just looking for reason why this thing is not working with latest SDK and sender app. And also this is not working for my own app.

Please file a bug against the SDK: https://issuetracker.google.com/issues?q=componentid:190205 status:open&s=modified_time:desc


Property initializer should get the same completion as property assignment

From @unional on April 22, 2017 9:12

VSCode Version: 1.11.2
OS Version: OSX

abstract class Foo {
  options: {
    x: string
  }
}

class Koo extends Foo { // already know options is missing x
  options = {
    // should get `x` as property
  }
  do() {
    this.options = {
      // does recognize `x` as property
    }
  }
}

Copied from original issue: Microsoft/vscode#25181

Typescript sees Koo's options as distinct from Foo's options when declaring them. It just checks that Koo's options is assignable to Foo's options after it has been created.
In contract, this.options is referring to something that is already declared. Note that if you provide a more specific type for Koo's options, like options = { x: "string literal type" }, then the completion for this.options in Koo will have that more specific type.

A full explanation of the approaches we have tried is at #6118. Both would provide completions here, but neither works well as part of the language.

Duplicate of #3667


Permission error with IP tables

I am running the following command in my controller, doing an IP lookup to get ASN and other details to determine whether to add to IPTABLES as a DROP. The goal is to dynamically populate the IP address. Here's my code so far testing with a statically defined IP
cmd = TTY::Command.new
cmd.run('iptables -A INPUT -s <IP_ADDRESS> -j DROP', user: 'root')
I am getting the following error. It appears IPTABLES want the controller to be set as root. Is there a way to define root user with password to get around this?
TTY::Command::ExitError (Running sudo -u root -- sh -c 'iptables -A INPUT -s <IP_ADDRESS> -j DROP' failed with
exit status: 1
stdout: Nothing written
Without user defined in the command I get an exit status: 3
Perhaps the rails 5 app is using a different user and therein lies the rub.
Would appreciate any help as we are trying to automate bans of certain IPs vs dealing with it manually.
Thank you for any feedback.

I believe this isn't special to TTY::Command, it would be the same if you used the built-into ruby constructs like system.
It's just about permissions on unix. You need to figure out a way to set up your permissions so whatever process the ruby app is running under has permission to execute the the command you want. (It's not about "the controller", it's the unix account the whole process the ruby app, perhaps a Rails app, is running under).
This is a sysadmin task that is outside the scope of TTY::Command to do anything about or support. But you may want to look into the /etc/sudoers file on linux. https://www.hostinger.com/tutorials/sudo-and-the-sudoers-file/


Cache normalized tuple types as made during relationship checking

Since tuple normalization can involve a bit of work, and relationship checking is often called on the same type more than once, caching this work could be beneficial.

@typescript-bot perf test this

Guess note - this doesn't look like it's helpful in practice.


iOS Aggresive Caching

I noticed my file was not being updated correctly using the default options.type of replace. Seems a temporary form of remediation is to append a random string to the options.id, like so:
var sync = ContentSync.sync({ src: globals.url, id: 'test1'+new Date().getTime()+'.js'});
sync.on('progress',progressfn);
sync.on('complete',successfn);
sync.on('error',errorfn);

I'm surprised no one else is having this issue (or I'm missing something). At the very least, it is worth mentioning in the documentation to prevent people from getting snagged.
Issue witnessed on iPhone 5c (9.3.2) and iPhone 6 (9.3.5)

This shouldn't happen as replace deletes the previous archive and extracts the new one. Are you seeing that your folder already exists in your Logs (in XCode) ?

What version of iDevice are you using?

I usually test on iPhone 5, 5S and 6+ on iOS 9.

This is only an issue on iOS devices for me. Android works perfectly fine. I'm not sure what to make of it outside of a persistent caching issue. You could mark this issue "closed" and just add a blurb in your documentation stating if someone is having an issue with the file being replaced, to just append a random string to the id.

Yes I test on physical devices


feat: [indexer] optimization - publish token supplies and pool pair data after synced

This commit:

Promotes the node status checker from sqs level to common level, so that it is used by both sqs and indexer ingesters.
Modifies the fullIndexerBlockProcessStrategy in indexer such that it delays the publishing of token supplies and pool pair data until the node sync finishes (i.e., when catch up is false).
Adds test cases for the full_indexer_block_process_strategy::ProcessBlock function:

When the node has caught up and is no longer syncing:

Publishes the correct number of token supplies and offsets.
Publishes the correct number of pool pairs, along with the expected number of pools with creation data.
Returns an error if it fails to verify the node's syncing status.

When the node is still syncing:

Returns an error, and no data should be published.

Closing this PR to avoid some complicated merge issues and also to make running local test easier. Opened this PR to extend this one.


Remove generate package when releasing

Closes #459
I tested locally that removing the whole generate package locally won't cause build to fail, and the docker container I built with that runs as expected.

LGTM. Note that because images are layered, deleting files in a separate RUN command does not actually make images smaller.

That should be fine. I think we want to remove it so that people won't get chance to run the generate command themselves


Request: add questu.ru to the filter

Evidence: https://questu.ru/questions/51862077/
Original: https://stackoverflow.com/questions/51862077/chocolatey-app-install-failure-with-the-error-message-of-the-remote-server-retur

Thanks a lot!


Document how to use monads in specs/tests

Hello,
I think it would be nice-to-have some documentation how to use monads in specs/tests properly. It would help in adaptation as well :)

@deepj what do you mean by that? What kind of special support in specs can be useful?

@flash-gordon I meant something like this https://dry-rb.org/gems/dry-view/testing/. What I mean to set preferable and recommendation how to tests monads in rspec/minitest.

@deepj I think that there is no need of dedicated documentation, because there is no special ceremony needed to test monadic values. For example, you can set expectations for your wrapped values within bind blocks, or just call .value!, for result kind monads, to extract it when you know it wraps a success. Just use the regular API. If you need help with some special scenario maybe you can ask for help in dry-rb discussion forum.

@waiting-for-dev I don’t have a problem with that. I just reflect some issue one junior developer which would like to learn monads via Ruby.
She wasn’t sure what the proper way is.
Be more empatic for juniors.

@deepj , I'm quite shocked with your message, because my intention was very far from being rude or not empatic. I was trying to give some help. I'm not a native English speaker, so maybe there is something in my message that sounds in a way I didn't mean it. If so, I'm really sorry.
The core message I wanted to communicate is that there is nothing special you should do to test values wrapped in monads, besides using the monads API. For this reason I don't think (just my opinion) there is documentation missing here. So, as I don't this is an issue in dry-monads project, I recommended you to ask for whatever you need in dry-rb discussion forums, as it is the place where you can find support. There, more people, including myself, would be more than welcome to guide you in whatever you need.

@deepj English is not my native language too. Again, I reflect an experience one junior developer, not mine. I'm not the junior developer. She expected this is covered by documentation. She didn't find out that. So I helped. She doesn't have/use Github so I reflect different experience someone else... I barely doubt she would have posted something to dry-rb forum.
So I decided to open this issue, since I feel this missing in the documentation too. I remember how I started I had a lot of problems with documentation various projects. I struggled on total basics and so on. And this kind of documentation make sure you're not wrong at all. So I feel empathic for her when she couldn't figure out how to use it in tests.

Hi folks, thanks for the discussion here. It'd certainly be nice to have some kind of beginner-friendly "how to test" guide for dry-rb gems in general, leaving any specific testing-related notes to reside within the documentation of each gem.
Like @waiting-for-dev points out, there's no special ceremony required for testing dry-monads objects, which is why such a page won't need to exist for that gem in particular.
Since there's no related issue in the dry-monads repo directly, I'll close this issue now. I do appreciate the feedback, @deepj, and we'll keep it in mind for when we can turn a greater amount of attention to expanding the documentation (and in the meantime, we are always happy to accept contributions from the community there too!).
Thanks ❤

I came across this issue as I was also looking for some documentation for testing, for me it mainly boiled down to a couple of methods that I used in testing (I was mainly using Result monads in my code)
success? and failure? https://dry-rb.org/gems/dry-monads/1.3/result/#code-failure-code-and-code-success-code
be_success matched of rspec will call .success? method on the monad so it can be used to test whether the result was success or not
expect(result).to be_success
expect(result).not_to be_success
expect(result.failure?).to be

value! https://dry-rb.org/gems/dry-monads/1.3/result/#code-value-code
To test the result in case of success
expect(result.value!).to eq user

failure https://dry-rb.org/gems/dry-monads/1.3/result/#code-failure-code
To test the errors returned
expect(result.failure).to eq(:not_found)

This is how I test it:
it do
  is_expected.to be_success.and(
    have_attributes(
      success: repo.last
    )
  )
end

it do
  is_expected.to be_failure.and(
    have_attributes(
      failure: :not_found
    )
  )
end

How's it better than
include Dry::Monads[:result]

it { is_expected.to eql(Success(repo.last)) }

it { is_expected.to eql(Failure(:not_found)) }

Well, this is more clean. 👌

Note that my approach allows RSpec matcher, which can be useful:
it do
  is_expected.to be_success.and(
    have_attributes(
      success: be_a(Foo).and(
        have_attributes(
          bar: include(
            be_a(String)
          )
        )
      )
    )
  )
end

Hey! I am trying to do something like
allow_any_instance_of(MyClass).to receive(:result_method).and_return(Dry::Monads::Success(true))

But receive without any debug info:
Module or String expected, got nil

How to properly solve this error?

@arrowcircle try Dry::Monads::Success.new(true).
"dot new" is the difference here

Another idea for cases when you need to check type of wrapped value without checking it's equality, using pattern matching in 2.7+ and rspec's satisfy matcher:
it { is_expected.to(satisfy { |v| v in Success(OAuth2::AccessToken) }) }
it { is_expected.to(satisfy { |v| v in Failure[:exchange_code_error, OAuth2::Error] }) }

You know some premade matcher which can replace satisfy please let me know :)


Tweak README.

"shutdown" (which is the noun) -> "shut down" (the verb, which is intended)
"top level"-> "top-level" (this might just be my personal preference)
"using  a" -> "using a" (removed double space)
"This is especially useful for application running on"-> "for applicationS running" (missing article)
"Use 'start' function" -> "Use the 'start' function"(missing article)
"Typical use of the library" -> "The typical" (missing article, alternatives: "A typical", however, it is too close to the antonym "atypical" 😃)

@swift-server-bot test this please

@swift-server-bot test this please

thanks again @sajjon

@swift-server-bot test this please


Support git+ssh://user@host:relative-path/repo.git syntax

It would be great if vcspull supported relative paths for git+ssh remotes by using the colon character as a separator between host and path.

Can you give an example so I can be sure
Do you mean a relative file path?

An example would be marc@myhostname.de:django/website.git where django is a folder within my home directory ~.


maven Build failure

Description
maven Build failure
Steps to Reproduce

Affected Component
Analytics
Version
6.7.0-SNAPSHOT
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  44.213 s
[INFO] Finished at: 2024-05-16T16:30:46+08:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project org.wso2.ei.business-process.multitenancy.dashboard.ui: Could not resolve dependencies for project org.wso2.ei:org.wso2.ei.business-process.multitenancy.dashboard.ui:bundle:6.7.0-SNAPSHOT: Failed to collect dependencies at org.wso2.carbon:org.wso2.carbon.registry.core:jar:4.5.3: Failed to read artifact descriptor for org.wso2.carbon:org.wso2.carbon.registry.core:jar:4.5.3: Could not transfer artifact org.wso2.carbon:org.wso2.carbon.registry.core:pom:4.5.3 from/to wso2.releases (http://maven.wso2.org/nexus/content/repositories/releases/): Authorization failed for http://maven.wso2.org/nexus/content/repositories/releases/org/wso2/carbon/org.wso2.carbon.registry.core/4.5.3/org.wso2.carbon.registry.core-4.5.3.pom 403 Forbidden -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :org.wso2.ei.business-process.multitenancy.dashboard.ui


[Feat] Ammunition Consumer

Add an ammunition consumer that can consume the number of ammunition on an actor.
This needs to be able to show what kind of ammunition is available on an actor and as such can only be completely configured when the item is on an actor.

Under the hood the consume function for this uses the quantity consume function. #137


A question about the detectron2 version

The earliest official version of detectron2 only supports pytorch1.4https://github.com/facebookresearch/detectron2/releases, And you are using pytorch1.3, what detectron2 version are you using, I will appreciate it if you answer as soon as possible!

It works because PyTorch is quite forward-compatible. I am using the Detectron version that is bundled along with this repo. I dont exactly remember when I forked off. It would be somewhere around Nov 2019.


Add tslib as runtime dep

https://github.com/ryanluker/vscode-coverage-gutters/issues/370#issuecomment-1120402680
According to typescriptlang.org, not using importHelpers "can result in code duplication if the same helper is used in many different modules". So use tslib might be a better solution.
The reason the unit tests are not failing is that unit tests use the development environment's node_modules when running.
Tested on my computer.
close #370

@ilharp How were you able to confirm the fix on your computer?
I was going to try compiling the vsix and installing that locally, maybe 🤔.

@ilharp How were you able to confirm the fix on your computer? I was going to try compiling the vsix and installing that locally, maybe 🤔.

npm run build && vsce package and then unzip the generated vsix, and there is tslib in node_modules.

And then install this vsix then it just works™

@ilharp How were you able to confirm the fix on your computer? I was going to try compiling the vsix and installing that locally, maybe 🤔.

npm run build && vsce package and then unzip the generated vsix, and there is tslib in node_modules.

And then install this vsix then it just works™

KK thanks! I thought that was the case so I confirmed locally your fix.
Release is going out shortly.


jena.query.ResultSet.hasNext takes a long time in OntologyModelService.expandConcepts

In the dev cluster the jena.query.ResultSet.hasNext method takes about four seconds to finish. However, the second time to insert the same document, it almost takes 0 second to finish it. Somehow, there is a cache system there. The code looks like:
        Query query = QueryFactory.create(q);
        QueryExecution qexec = QueryExecutionFactory.create(query, ontModel);
        ResultSet results = qexec.execSelect();
        String name = field.getName();
        Set<String> values = new HashSet<String>();
         // results.hasNext() takes a long time
        while (results.hasNext()) {
          QuerySolution solution = results.next();

added to #34 and closed


is it possible to consolidate the results?

For example the dojo-1.16.3-distribution.zip has a lot of files and all of them report to have the exact same vulnerabilies. This blows up the number of found issues. Can this be avoided and the zip file be counted only once?

Sure - sorry - here it is https://mvnrepository.com/artifact/org.dojotoolkit/dojo/1.16.3 we have a dependency through: https://mvnrepository.com/artifact/org.apache.qpid/qpid-broker-plugins-management-http

Sorry for the delay on this - but I am unable to reproduce the issue using the maven plugin.

I have created a reproducer: https://github.com/simondivi/DependencyCheck-5109


Simple hello-world example takes 2.7MB (even excluding STL), but the equivalent in C++ only takes <1.0MB, so a 300% size increase

Operating system: MacOS
The way you installed OpenCV: I am doing android, so download from official website
OpenCV version: 4.5
rustc version (rustc --version): rustc 1.55.0 (c8dfcfe04 2021-09-06)
Attach the full output of the following command from your project directory: see belowRUST_BACKTRACE=full cargo build -vv

Hi thanks for the lib! I use it on Android, and see the hello-world example is even 2.7MB huge. So I wonder what am I doing wrong? Thanks!
rust code (very simple):
use opencv::core::*;

pub fn add(a: i64, b: i64) -> i64 {
    let m = Mat::zeros(a as i32, b as i32, CV_8UC1).unwrap();
    let size = m.size().unwrap();
    return (size.width + size.height) as i64;
}

cargo:
[package]
name = "vision_utils_rs"
version = "0.1.0"
edition = "2018"

[lib]
name = "vision_utils_rs"
crate-type = ["cdylib"]

[dependencies]
opencv = "0.54"

my build script:
export DYLD_FALLBACK_LIBRARY_PATH="$(xcode-select --print-path)/Toolchains/XcodeDefault.xctoolchain/usr/lib/"

export OPENCV_BASE_DIR=/Users/tom/Others/OpenCVRelease/OpenCV-android-sdk
export ANDROID_ABI=armeabi-v7a

export OPENCV_LINK_PATHS=$OPENCV_BASE_DIR/sdk/native/staticlibs/$ANDROID_ABI,$OPENCV_BASE_DIR/sdk/native/3rdparty/libs/$ANDROID_ABI
export OPENCV_LINK_LIBS=opencv_imgcodecs,opencv_features2d,opencv_calib3d,opencv_imgproc,opencv_highgui,opencv_core,tbb,cpufeatures,ittnotify,tegra_hal,IlmImf,liblibpng,liblibtiff,liblibwebp,liblibjasper,liblibjpeg-turbo,z
export OPENCV_INCLUDE_PATHS=$OPENCV_BASE_DIR/sdk/native/jni/include/

cargo ndk --target armeabi-v7a -o /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs build --release -vv

outputs:
(base) ➜  rust git:(master) ✗ cargo ndk --target armeabi-v7a -o /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs build --release -vv
[2021-09-29T07:23:14Z INFO  cargo_ndk::cli] Using NDK at path: /Users/tom/Library/Android/sdk/ndk/21.2.6472646
[2021-09-29T07:23:14Z INFO  cargo_ndk::cli] NDK API level: 21
[2021-09-29T07:23:14Z INFO  cargo_ndk::cli] Building targets: armeabi-v7a
[2021-09-29T07:23:14Z INFO  cargo_ndk::cli] Building armeabi-v7a (armv7-linux-androideabi)
       Fresh glob v0.3.0
       Fresh regex-syntax v0.6.25
       Fresh maplit v1.0.2
       Fresh dunce v1.0.2
       Fresh autocfg v1.0.1
       Fresh percent-encoding v2.1.0
       Fresh vcpkg v0.2.15
       Fresh shlex v1.1.0
       Fresh pkg-config v0.3.20
       Fresh once_cell v1.8.0
       Fresh memchr v2.4.1
       Fresh semver v1.0.4
       Fresh libc v0.2.103
       Fresh aho-corasick v0.7.18
       Fresh clang-sys v1.2.2
       Fresh jobserver v0.1.24
       Fresh num-traits v0.2.14
       Fresh clang v1.0.3
       Fresh regex v1.5.4
       Fresh cc v1.0.70
       Fresh opencv-binding-generator v0.29.0
       Fresh opencv v0.54.0
       Fresh vision_utils_rs v0.1.0 (/Users/tom/QAPMain/Development/frontend/vision_utils/rust)
    Finished release [optimized] target(s) in 0.03s
[2021-09-29T07:23:14Z INFO  cargo_ndk::cli] Copying libraries to /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs...
[2021-09-29T07:23:14Z INFO  cargo_ndk::cli] /Users/tom/QAPMain/Development/frontend/vision_utils/rust/target/armv7-linux-androideabi/release/libvision_utils_rs.so -> /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs/armeabi-v7a/libvision_utils_rs.so

look at size:
(base) ➜  rust git:(master) ✗ ls -al /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs/armeabi-v7a/libvision_utils_rs.so
-rwxr-xr-x  1 tom  staff  2853104  9 29 15:23 /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs/armeabi-v7a/libvision_utils_rs.s

or look at size again, inside the apk file:

On the contrary, looking at this tutorial: https://stackoverflow.com/questions/63127202/how-to-use-opencv-4-in-native-c-of-flutter-in-2021-support-flutter-2-0/63127203#63127203 .

If you build in release mode and look at the apk size, you will see our .so file is less than 1MB. Thus static linking and file size reduction does work :)

that tutorial uses the code here: https://github.com/fzyzcjy/flutter_cpp_with_opencv/blob/master/ios/Classes/native_add.cpp . So it is equivalent to our code shown above.
Of course, if our code is:
use opencv::core::*;

pub fn add(a: i64, b: i64) -> i64 {
    a+b
}

I get result of 13.8KB (0.013MB) for the .so file, which is reasonable and small.

I'm not sure about how the Android build process works (but I'm really glad to know that it works there!), but when I build your example on the x86_64-unknown-linux-gnu target I'm indeed getting a 3,3M sized binary. The crate generates a lot of glue C++ code in the build process so that must be it. But just doing a simple strip <output_binary> reduces the size to 351K which is acceptable I think. Can you please try running strip (or equivalent) on your resulting binary?

@twistedfall Well strip has no use... It is already "stripped" when I file it. Did you build in --release mode? If you built in debug mode I guess it should be big.
In addition, "a lot of glue c++ code" seems not to be as big as >1MB ;)
(base) ➜  rust git:(master) ✗ ls -al /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs/armeabi-v7a/libvision_utils_rs.so
-rwxr-xr-x  1 tom  staff  2844912  9 29 15:36 /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs/armeabi-v7a/libvision_utils_rs.so
(base) ➜  rust git:(master) ✗ file /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs/armeabi-v7a/libvision_utils_rs.so
/Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs/armeabi-v7a/libvision_utils_rs.so: ELF 32-bit LSB shared object, ARM, EABI5 version 1 (SYSV), dynamically linked, stripped
(base) ➜  rust git:(master) ✗ strip /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs/armeabi-v7a/libvision_utils_rs.so
(base) ➜  rust git:(master) ✗ ls -al /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs/armeabi-v7a/libvision_utils_rs.so
-rwxr-xr-x  1 tom  staff  2844912  9 29 15:36 /Users/tom/QAPMain/Development/frontend/vision_utils/android/src/main/jniLibs/armeabi-v7a/libvision_utils_rs.so

Wait a minute... Seems that this crate generates all functions binding to opencv. So, will the whole opencv be linked? Or compiler is smart enough, such that it knows "though you define thousands of functions, you never use it, so I also do not link opencv functions as well"?

Yes, I did build in release mode. The compiler should eliminate all the dead code, that's right. Are you linking opencv statically or dynamically? Can you please also post the output of readelf -aC <output_so_name>?

Sure. Here is the file:
readelf.txt
Indeed I can upload the .so file itself:

Are you linking opencv statically or dynamically?

Statically I guess. Looking at my configs, I link a lot of opencv_xxx.a

By the way, I see why I do not have symbols while you have: I use the -o in cargo ndk build, and it seems that it is not simply copying files to the output dir, but instead, copying and stripping. Because I see the raw .so file is not stripped, while the file in output folder is stripped.

So here is the .so file with symbols:
...target/armv7-linux-androideabi/release/libvision_utils_rs.so: ELF 32-bit LSB shared object, ARM, EABI5 version 1 (SYSV), dynamically linked, with debug_info, not stripped

libvision_utils_rs.so.zip

Hi, is there any updates/suggestions? Thanks!

So far I don't really have any new information, the biggest 2 sections in this ELF are .text (program code) and .rodata (constants). I'm not so good with ARM assembly so I haven't gotten too far as to finding out why .text is so large.

Well thank you all the same!

Later I find that, after implementing several algorithms, the Rust code size is not bigger than C++ code size. So I think this issue is indeed no problem.


More of a question

Hello,
I'm using pixi particles to make ability that looks like shield around player in my game. I only render players that are near me, not all of them.
So my question is when new player enters my viewport and supposedly he had pixi particle (shield) running on him for 2 seconds before he entered my viewport and only now my client renders the particle emitter, is there a way to SKIP pixi particle 2 seconds in? As to show that specific duration has passed.
I was thinking of solutions to this, but even if I know that player had shield on him for 2 seconds i cant just reduce the duration of emitter by 2 secs because the animation will start from beginning (e.g lets take that megaman death example, i dont want to see circles going from very begining from inside to outside but rather show animation 2 seconds in when circles are all in outer layer)
To summarize, is it possible to skip-forward particles "X" amount of seconds in?
Thanks in advance.

call yourEmitter.advance(2) to run the emitter for 2 seconds immediately.

call yourEmitter.advance(2) to run the emitter for 2 seconds immediately.

holy shiet you're da MVP bro!!!!!

@andrewstart
Hello, I tried advance but it seems like there's no such function
emitter.advance is not a function

Yep, I should have checked the code instead of assuming that I remembered correctly - the method name is update, not advance.


Subscribers views not scaling

As you can see two subscribers comes, there is a  very big gap (Highligheted in red color) between the publisher & subscribers can someone help to fix this

subscriber.getRenderer().setStyle(BaseVideoRenderer.STYLE_VIDEO_SCALE, BaseVideoRenderer.STYLE_VIDEO_FILL);

@sureshmaidaragi1919 Hope @adrienrx's solution worked for you. Will close this for now. Feel free to reopen if still experiencing issues.


[Improvement]: Add skip-trino-build profile

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
It not friendly for users to use mvn clean package -DskipTests -pl '!amoro-mixed-format/amoro-mixed-format-trino' to build the amoro project. Add a new skip-trino-build profile to improve the build command.
How should we improve?
No response
Are you willing to submit PR?

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

Subtasks
No response
Code of Conduct

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

Thanks a lot for bringing up this improvement.
The old maven command is indeed quite complex to use.
But actually, most of the time, we do not need to build the amoro-mixed-format-trino module as it dependent on a different JDK version.
So how about adding a profile named build-mixed-format-trino and inactive it by default?
So most of the time, you can execute mvn clean package to package all modules except build-mixed-format-trino.
And you can execute mvn clean package -Pbuild-mixed-format-trino -Ptoolchain to package all modules.

@zhoujinsong you are right, I'll think about it again.


[Sunset] the editor background color is "too dark"

Either desaturate or make it a bit "less orangy"

I've been using the theme for a while and this isn't really an issue


e2e File - ENI

Additional changes made:

Updated the sdk version
Used go 1.20 in jenkins
Fixed issues related to contexts in e2e/testsuits/baseutils.go
Use gingko 2.17.2 in e2e.sh and go.mod
Fixed issue vpc-deploy-statefulset.sh script which takes logs from controller-0 assuming pod would be a deployment driver but recently we moved to statefulset hence updated the logic

Usung go test -v -timeout 3000s -coverprofile=cover.out ${GOPACKAGES} instead of gotestcover due to dependency issues

To test this, I created clone jenkins job test-eit-file and created build, and confirm its passing and running all desired taste and posting results on slack channel.

Removing the use of gotestcover to run UT, facing issue the dependencies when using gotestcover
❯ go get github.com/pierrre/gotestcover

go: removed github.com/IBM/ibm-csi-common v1.1.13
go: removed k8s.io/kubernetes v1.28.6

Usung go test -v -timeout 3000s -coverprofile=cover.out ${GOPACKAGES} instead

Tested:
VPC-FILE-CSI-TEST: VERIFYING VPC FILE CSI DRIVER HEALTH: PASS
VPC-FILE-CSI-TEST: VERIFYING MULTI-ZONE/MULTI-NODE READ/WRITE BY USING DAEMONSET : PASS
VPC-FILE-CSI-TEST: VERIFYING PVC WITH RWO MODE : PASS
VPC-FILE-CSI-TEST: VERIFYING PVC CREATE/DELETE WITH ibmc-vpc-file-retain-dp2 STORAGE CLASS : PASS
VPC-FILE-CSI-TEST: VERIFYING MULTI-POD READ/WRITE ON SAME NODE BY USING DEPLOYMENT: PASS
VPC-FILE-CSI-TEST: VERIFYING PVC CREATE/DELETE WITH ibmc-vpc-file-dp2 STORAGE CLASS : PASS
VPC-FILE-CSI-TEST: VERIFYING PVC EXPANSION BY USING DEPLOYMENT: PASS
VPC-FILE-CSI-TEST: VPC-File-Volume-Tests: PASS

https://alchemy-containers-jenkins.swg-devops.com/job/Containers-Volumes/job/test-eit-file/12/


Improve Color Strings Support

@nick-thompson, See the below TODO from the GainPlugin example:
  // TODO: Should update the ctx proxy to convert from javascript hex strings, aka
  // #ffffaa to juce's Colour::fromString() API which is ffffffaa.

It would be great to refine Blueprint's color handling behaviours. I think you had a library in mind for this? I'd be happy to investigate if you can point me in the direction of what you wanted to use.

Yep! Definitely want this in, and I think it can be quite simple. I intend to do this on the javascript side, where in the BlueprintBackend we check "is the user setting a color-related property?" -> if so, transform the property value string from any arbitrar color format to JUCE's format. For example
<View {...{'background-color': 'hsla(55, 55, 55, 0.5)'}} />

That property update will trickle through to here: https://github.com/nick-thompson/blueprint/blob/master/packages/juce-blueprint/esm/lib/BlueprintBackend.js#L65
And right there we can just check if the prop key is one of a pre-defined list of color-related properties. When it is, we make the transformation.
I had been looking at this lib: https://github.com/brehaut/color-js, but I'm not so sure if that's actually the one we want. In particular, we only really care about being able to transform from one color string format (hsl, rgb, hsla, rgba, hex, etc: https://developer.mozilla.org/en-US/docs/Web/CSS/color) to juce's hex-string-with-the-alpha-chars-first format. These could be a few simple functions that we write ourselves. Now, this library that I mentioned covers that mostly, but it doesn't understand juce's string format so we'd probably have to assemble that ourselves. The other thing that it doesn't do, that I think we should aim to cover here, is supporting gradients (https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Images/Using_CSS_gradients).
So I imagine writing something like this (hand-waivy, because js doesn't have true pattern matching like this):
if (colorProperties.includes(propKey)) {
    __BlueprintNative__.setProperty(propKey, transformColorString(propValue));
}

function transformColorString(s) {
    match (s) {
        `hsla(h, s, l, a)` -> hexFromHSLA(h, s, l, a);
        `rgba(r, g, b, a)` -> hexFromRGBA(r, g, b, a);
        `hsl(h, s, l)` -> hexFromHSL(h, s, l);
        `linear-gradient(x, y)` -> [transformColorString(x), transformColorString(y)] // idk, guessing here. How do we tell the native side we want a gradient?
    }
}

Right cool, interesting!
Thanks for the discussion @nick-thompson. Would you be happy for me to have a poke at this one or is this something you particularly want to play with yourself? Suspect I'll have to do a little research but I'm up for it.

Definitely, go for it! I think gradients will be the trickiest part; I'm very open to suggestions on that :). I think it'd be awesome to get that in in a generic way

@nick-thompson , should we tag this for beta? Probably a bit of an annoyance for users?

Yep, I like that. Will add now

Think we can call this one closed! Thank you @eupston :)

Yeah this was killer @eupston. I'm getting assets from a designer in Sketch projects and this makes it much easier to copy palettes and gradients etc. etc. Nice stuff.


Create Github Wiki

Move documentations to github wiki.

Can close the issue already. Help me verify at the wiki. Some images not showing as not yet uploaded into the folder + need to make the user guides less verbose, but we can directly make the changes in the wiki from now.

I fixed the links on the main page, but all the links within the documents will need to be fixed as well.
You can just use the HTML tag <img> to resize the images.
https://stackoverflow.com/questions/24383700/resize-image-in-the-wiki-of-github-using-markdown


fix: hide empty sections in Graph tab

Fixes https://github.com/npmgraph/npmgraph/issues/247

Merging this because it's a quick fix. It would have to be rethought after this is merged because this "Report" tab would then be completely empty.


[Big model inference] ValueError: weight is on the meta device, we need a value to put in on 3.

Information
Trying to dispatch a large language model's weights on multiple GPUs for inference following the official user guide.
Everything works fine when I follow the example referenced in the accelerate.load_checkpoint_and_dispatch function documentation:
from accelerate import init_empty_weights, load_checkpoint_and_dispatch
from huggingface_hub import hf_hub_download
from transformers import AutoConfig, AutoModelForCausalLM

# Download the Weights
checkpoint = 'EleutherAI/gpt-j-6B'
weights_location = hf_hub_download(checkpoint, "pytorch_model.bin")

# Create a model and initialize it with empty weights
config = AutoConfig.from_pretrained(checkpoint)
with init_empty_weights():
    model = AutoModelForCausalLM.from_config(config)

# Load the checkpoint and dispatch it to the right devices
model = load_checkpoint_and_dispatch(
    model, weights_location, device_map='auto', no_split_module_classes=['GPTJBlock']
)

But, I get the error mentioned in the issue title when loading other LLMs, including EleutherAI/gpt-neo-1.3B.
System information

Python version: 3.9.16
Cuda version: Build cuda_11.8.r11.8/compiler.31833905_0
GPUs: 4 x NVIDIA GeForce RTX 3090
PyTorch 1.13.1
Transformers 4.26.1
Accelerate 0.17.1

Accelerate config
compute_environment: LOCAL_MACHINE
deepspeed_config: {}
distributed_type: MULTI_GPU
downcast_bf16: 'no'
dynamo_config: {}
fsdp_config: {}
gpu_ids: 0,1,2,3
machine_rank: 0
main_training_function: main
megatron_lm_config: {}
mixed_precision: 'no'
num_machines: 1
num_processes: 4
rdzv_backend: static
same_network: true
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false

Code and error
from accelerate import init_empty_weights, load_checkpoint_and_dispatch
from huggingface_hub import hf_hub_download
from transformers import AutoConfig, AutoModelForCausalLM

checkpoint = 'EleutherAI/gpt-neo-1.3B'
weights_location = hf_hub_download(checkpoint, 'pytorch_model.bin')

print(weights_location)
config = AutoConfig.from_pretrained(checkpoint, output_hidden_states=True)
with init_empty_weights():
    lmodel = AutoModelForCausalLM.from_config(config)
lmodel = load_checkpoint_and_dispatch(
    lmodel, weights_location, device_map='auto', no_split_module_classes=['GPTNeoBlock']
)

The line print(weights_location) outputs /.cache/huggingface/hub/models--EleutherAI--gpt-neo-1.3B/snapshots/0f35a20339a9e208bc061570981180cfb87c9572/pytorch_model.bin which is correct according to the documentation.
The code is basically identical to the one I used to dispatch the EleutherAI/gpt-j-6B model, except I specified no_split_module_classes=['GPTNeoBlock'].
I am getting the following error:
Traceback (most recent call last):
  File "src/main.py", line 58, in <module>
    main(args)
  File "src/main.py", line 17, in main
    run_probing_train(args=args)
  File "src/run_probing.py", line 225, in run_probing_train
    lmodel = get_lmodel(args)
  File "src/run_probing.py", line 68, in get_lmodel
    lmodel = load_checkpoint_and_dispatch(
  File "/u/weyssowm/Research/AstProbing/venv/lib/python3.9/site-packages/accelerate/big_modeling.py", line 490, in load_checkpoint_and_dispatch
    return dispatch_model(
  File "/venv/lib/python3.9/site-packages/accelerate/big_modeling.py", line 370, in dispatch_model
    attach_align_device_hook_on_blocks(
  File "/venv/lib/python3.9/site-packages/accelerate/hooks.py", line 495, in attach_align_device_hook_on_blocks
    attach_align_device_hook_on_blocks(
  File "/venv/lib/python3.9/site-packages/accelerate/hooks.py", line 471, in attach_align_device_hook_on_blocks
    add_hook_to_module(module, hook)
  File "venv/lib/python3.9/site-packages/accelerate/hooks.py", line 155, in add_hook_to_module
    module = hook.init_hook(module)
  File "venv/lib/python3.9/site-packages/accelerate/hooks.py", line 244, in init_hook
    set_module_tensor_to_device(module, name, self.execution_device)
  File "venv/lib/python3.9/site-packages/accelerate/utils/modeling.py", line 136, in set_module_tensor_to_device
    raise ValueError(f"{tensor_name} is on the meta device, we need a `value` to put in on {device}.")
ValueError: weight is on the meta device, we need a `value` to put in on 3.

I can reproduce indeed, thanks for raising the issue! The problem comes from the tied weights being untied during the loading, and the state dict for this model does not have a key for the tied weights (to avoid the checkpoint being too large). I'll push a fix later today or tomorrow.

The PR mentioned above should fix this. Note that you will need to add a lmodel.tie_weights between the initialization and the checkpoint load, as the from_config method does not tie weights automatically.


Fixes #6489

#6489

request-promise/request-promise.d.ts
to author(@cglantschnig). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

request/request.d.ts
to authors(@soywiz @bonnici @Bartvds). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

:+1:
that is a great addition. thanks

I just want to mention one negative part of this pull request is that Options is changed from an interface to a union type alias. This is not ideal, but needed because an interface cannot extend an union type, see this comment

okay, thanks mate.


Add basic implement for support tag in Space

Scope of pr

allow add and remove a tag for a snapshot, current snapshot
read using a tag

Thank you!


Fix rendering issue

Fix rendering issue

Codecov Report
Patch and project coverage have no change.

Comparison is base (5b8a980) 88.51% compared to head (116a585) 88.51%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #200   +/-   ##
=======================================
  Coverage   88.51%   88.51%
=======================================
  Files          68       68
  Lines        5024     5024
=======================================
  Hits         4447     4447
  Misses        577      577

Flag
Coverage Δ

unittests
88.51% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Improvement: Multiple fields

Say you have the following in your org file:
* Second Question
** Second Answer
** Front hint test

Which results in the generation of the following card (or note, always forget the anki terminology):

It would be nice if you could for instance put the following in your org file:
* Second Question
** Second Answer
# field='Front Hint'
** Front hint test

And have the plugin generate the following card:

I would have to have a think about this tbh.
Would want to make sure whatever design decision is made in the code is supportable and extensible in the future though it is a good idea overall and doesn't add too much complexity.
I think the style org style your are suggesting probably makes the most sense though.

The code is completed in branch feature/support-multiple-feild-types and unit tested.
I will hopefully have the documentation update this afternoon, will do some manual testing and will then release. Feel try to try it out on the branch before that.
I decided to go with the following syntax.
* Second Question
** Second Answer
# fieldType=Front Hint
** Front hint test

Logic behind this was as follows:

Better to use fieldType to be more specific  I might change it to fieldName before I release

The other parameters are already not using quote marks' even for multi word values

The split on the key=value occurs when another key is found or when the end of the line is reached

Closed by #36

I think you’ve implemented this very well as far as I can see, it atleast seems to fully suit my needs! Unfortunately I’m knee deep in ny exams atm, however I plan on using Anki and org_to_anki for Anki card creation for other stuff/ uni after my exams. I’ll report back than, or if I find some time I’ll try it out a little earlier.
I might have the basis for something useful too, a script to convert markdown files to org files which are compatible with org_to_anki. Although this script is currently very specific to my setup, hacky and written in fish script. I prefer markdown files as I can use the marktext markdown editor with it’s live preview of math formula’s which is really nice. Or I can edit it in my preferred editor which is neovim😉Which has nice support for markdown while it Obviously doesn’t handle org files well.
In any case thanks for the quality software.

I would be open to a look at some of the code and maybe even adding markdown parsing support in future if I can see a solid use case.
Would require a decent bit of testing but should be possible!
Thanks for the feedback anyway!


[High importance] Variables.py indentation fix

This prevents ee comand to execute.

I was just sending the PR to fix this. Thanks for this.


How connect device to cyphal Yukon or Yakut

Hello ,
If I don't use "Cyphal and DroneCAN Sniffer and Programmer" HW, can I directly use a regular USB can bus tool as a bridge between my device and Cyphal Yukon or Yakut?

Sure. It was mainly tested with RaccoonLab CAN-sniffer, but it should work with any other CAN-USB sniffer supported by Yukon/Yakut, for example with Zubax Babel.


Error when used in conjunction with a restrictive Content-Security-Policy

When the plugin tries to execute the Ember.HTMLBars.compile function, a CSP (Content-Security-Policy) that does not allow 'unsafe-eval' will block the JS execution with the following error
Uncaught EvalError: Refused to evaluate a string as JavaScript
because 'unsafe-eval' is not an allowed source of script in the
following Content Security Policy directive: "script-src ...

I worked around it by adding the following line in the development section of my config/environment.js file:
    // Allow unsafe eval on dev environment
    ENV.contentSecurityPolicy['script-src'].push("'unsafe-eval'");

This appears to be due to the way the compile function works (new Function) and not something specific to the plugin, but wanted to document the error

@localpcguy interesting find - think this is something we should add to the README? I'm unsure how common this unsafe-eval CSP setting is myself. If you think so I'd gladly welcome a PR to the readme explaining this workaround for starters :)

If someone is using a CSP, they should most definitely have unsafe-eval as one of the things they block (otherwise it kind of defeats the purpose of the CSP). I feel this would happen fairly often.
I'll see about putting the PR together to put the workaround in the README. Alternatively, would it work to have the addon push the unsafe-eval to the ENV.contentSecurityPolicy['script-src'] via the Addon initialization? I'm not familiar enough with the addon lifecycle to know if that is possible, but I could attempt to make that work if that would be better (then it would "just work", although it still should be communicated that it is happening via the README).

@localpcguy because this addon only supports development environments I'm considering this option. If I had to guess this would be the spot to hook in and alter the configuration to "push" unsafe-eval as you suggest. You mind trying this locally to see if it does indeed work "out of the box" ?
Appreciate the help here!

I try and take a look this weekend,

@localpcguy no worries - if we can't modify it that is likely for good reason ;)
One example - some addon author adds/or edits a config value ... then your app boots and you don't see X/Y/Z and to find out why would be non trivial (you would need to search thru all of your addons and reason about what they do/ learn if they hack the config object). Lets go w/ a README option if you've still got time this week ;)
thanks for taking the time to give this a go !


Code refactoring on dashboard

[x] 1. modularize dashboard function

|-- dashboard/

|-- dashboard.R
|-- dbChecklist.R
|-- dbNetwork.R
|-- dbTree.R
|-- setTabTitle.R
|-- ...

[ ] 2. ensure the plots to be rendered initially, check whether null is read initially
[ ] 3. clean up unused codes and annotate with comments for sub-modules: tree, network, checklist
[x] 4. improve UI

If I comment out network functions, the plot can be initially rendered after multiple testings. So the problem should come from the network module.

it seems like the problem comes from the order of rendering in tabsetPanel. According to this post in SO, the rendering order in tab pane needs to match the order in UI. In other words, since we put tab1 (checklist + network) before tab2 (tree), tab 1 plots needed to be rendered before tab2's plots in the server.
If I use reactlog to check the order of events taking place, the tab2 initiates its events before tab1 (color grey means starts the process):

My initial thoughts why the tab2 is rendering before tab1 is because the delays of getting the input value for tab1.

[ ] if I use same input value for tab2 (put in the same observeEvent), which force tab2 to be rendered after tab1, will the issue be solved?

If I force tab2 render after tab1 finish, I did not see the ^ behavior while I was rerunning the app at least 10 times. It seems like the issue ^2 is solved, but if anyone encounters it again, we need to create a new ticket to re-address the issue.

If I only kept the first tab, the behavior remains and the tab's contents were not rendered:

Still not able to find the exact the reason why the tabs' contents are not rendered initially, but after multiple testings:

the problem is not because of the tab panel
the problem is likely caused by the reactive values/ observe event. Since if I load data directly, the contents can be initially rendered. However, I am not able to figure out the exact the reason. The data is loaded successfully in a right order, but the plots are just not rendered.

To solve the issue 4^ for now, the dashboard is forced to switch to tab2 and then switch back to tab1.


content_disposition returns nil until content has been called

Once an object has been loaded, accessing object.content_disposition will return nil unless object.content has been called first.

Is it necessary to call "content", or "retrieve" works as well?

Probably linked to this issue:
Once an object has been loaded, object.content returns nil (using the 0.3.11).
On the other hand, it works using the 0.3.8.

Can you submit a patch for that?


feat(cognito): Add new checks related with cognito service

Description
New checks related with cognito service for aws are added:

[x]  cognito_user_pool_strong_password_policy -> Cognito user pools should have a password policy that meets or exceed AWS CIS Foundations Benchmark standards length 14, 1 number, 1 special character, 1 upper 1 lower

[x]   cognito_user_pool_temporary_password_expiration -> Cognito user pools temporary passwords set by administrators expire in 7 days or less

[x]  cognito_user_pool_mfa_enabled -> Cognito user pools require multi factor authentication (MFA)

[x]  cognito_user_pool_web_acl_associated -> Cognito user pools should be protected by AWS Web Application Firewall

[x]  cognito_user_pool_prevent_reveal_user_existence -> Cognito user pools should prevent error messages that reveal user existence

[x]  cognito_user_pool_token_revocation_enabled -> Cognito user pools should have enabled token revocation

[x]  cognito_user_pool_advanced_security_enabled -> Cognito user pools has advanced security enabled with full-function

[x]  cognito_user_pool_advanced_security_block_sign_in_compromised_credentials -> Cognito user pools has advanced security enabled with full-function to block sign-in by users with suspected compromised credentials

[x]  cognito_user_pool_advanced_security_adaptative_authentication_block_sign_in -> Cognito user pools has advanced security enabled with full-function with adaptive authentication automatic risk response as block sign-in

[x]  cognito_user_pool_deletion_protection_enabled -> Cognito user pools deletion protection enabled to prevent accidental deletion

[x]  cognito_user_pool_self_registration_enabled -> Self-registration is enabled on the user pools and it is associated with an identity pool (and show the roles that can be asssumed)

[x] cognito_identity_pool_guest_access_enabled -> Guest access is enabled on the identity pool(and show the roles that can be assumed)

License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Please, remove all report.resource_name and add the name of the resource to the status extended.


The Introduction section on the CSIP is not really an introduction

The opening text of the introduction does not provide a good starting point for a newcomer. I suggest setting the stage with a description of OAIS, Packages and then the purpose of this specification set.
All the text necessary is already available on the sub-sections. I'm just recommending a restructure of the contents of the Introduction section.
See: http://earkcsip.dilcis.eu/#1-introduction

So we shouldn't repeat text from other specifications but a short introduction that adds some beginners level context might be useful, even if it's just a primer reading list as an appendix.

https://github.com/DILCISBoard/GroupDocumentation/issues/17

Common introduction made and gets added in the transformations.


Prevent account fallback if name is defined in formatPayeeName

Note: This PR depends on #427 to work properly.
Currently, if creditorAccount is not set in a payment but debtorAccount is, payeeName ends up being "creditorName (debtorAccount)", which doesn't make sense. The issue is reversed in income transactions.
This situation always occurs in BANKINTER_BKBKESMM payments, where creditorAccount is never provided, but debtorAccount is always set.
To address this, I have prevented the account from falling back if the primary name option is set.
Additionally, consider whether we should ever fallback the account. We might also want to avoid falling back the name to the opposite (e.g., payments shouldn’t fallback to debtorName if creditorName is not set, and income shouldn’t fallback to creditorName if debtorName is not set). Instead, fallback to remittanceInformationUnstructured should be considered first (see #428).

👋 Hi! You can always override the payee name returned by the normalizeTransaction function directly instead of calling formatPayeeName - would that solve the problem you're facing?
https://github.com/actualbudget/actual-server/blob/3b26aa05b5eb9c8e08565424d17a95e594d854c4/src/app-gocardless/banks/bankinter-bkbkesmm.js#L39

I know, but don't you think that a payeeName like "creditorName (debtorAccount)" doesn't make any sense?

Ah, my apologies - I think I misread the change. Yes, of course that's reasonable.
Thanks for this!


TypeError: 'module' object is not callable

 File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.10_3.10.752.0_x64__qbz5n2kfra8p0\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.10_3.10.752.0_x64__qbz5n2kfra8p0\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "C:\Users\Username\codes\htmlfilemerger\src\webd\htmlmerger\__main__.py", line 2, in <module>
    main.generic_main ()
  File "C:\Users\Username\codes\htmlfilemerger\src\webd\htmlmerger\htmlmerger\main\main.py", line 4, in generic_main
    main()
TypeError: 'module' object is not callable

I'm a beginner.  Please help. If you have a tutorial video for this, please give me the link. Thanks

I can't reproduce your problem, but I tried something. Please do the following:

Clone the branch with my changes into a new location git clone --branch bug4 git@github.com:DarkTrick/htmlfilemerger.git
Try and run again.
Report the results.

The problem should be solved with the latest version. I'll close the issue. If you still experience problems, feel free to reopen.


[BUG] @jstb/toolbox: in TS mode, unnecessery semicolon at the end of interface.

Describe the bug
in TS mode, unnecessery semicolon at the end of interface.
interface HeroProps {
  label: string;
};

Name and version of the package
On which published artifact the issue is there, @jstb/catalyst
To Reproduce
Create a TS Component
Expected behavior
interface ComponentProps {
  label: string;
}

OS: [e.g. Windows]
Version 1.4.0

Issue fixed


use Godeps

Get rid of third_party.go and just use Godeps instead.

Is this correct replacement of thirtd_party.go with Godeps!?
b8709784927bab2b3b990cf5f45908c36f11c866
Following works
./devweb

docker build -t coreos/discovery.etcd.io .

./build and  ./devweb does go installon Godeps and Devweb - is this right?
Changes
Delete:

build-docker
third_party.go
third_party/

Added:

Godeps/
.godir

Updated

build
devweb
Dockerfile

Hi, I am trying to put in a PR for this, but I get an error:
godep save ./...
godep: Package (code.google.com/p/rsc/devweb/slave) not found
So obviously, I do a go get:
go get code.google.com/p/rsc/devweb/slave
package code.google.com/p/rsc/devweb/slave: unable to detect version control system for code.google.com/ path
Can someone help me here?

#40 switches to go vendoring instead, which replaces godeps.


Adding export script in package.json

Related Issue

Information about the related issue

Adding scripts :
    "export": "next build && next export",
    "prepare": "husky install",

Closes: #308
Describe the changes you've made

Type of change
What sort of change have you made:

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

How Has This Been Tested?

Checklist

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented on my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)

Original
Updated

original screenshot
updated screenshot

Code of Conduct

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

husky might need to be installed if it's not, I guess.

husky might need to be installed if it's not, I guess.

it was added already on prepare, maybe need to remove temporarily and npm i again then add the "prepare": "husky install" again then run npm i again

package is missing 🤣

Hey,
I have added the package husky in package.json and I think it's working fine now.


"Next steps" links are broken.

Most of the links in the next steps section are broken or not rendered.

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

ID: 09d58a24-f6af-d2b2-e8e0-b1126a6ad9fa
Version Independent ID: 9c6be489-f9d8-9b66-09f8-18be4ec77f3b
Content: AMQP 1.0 in Azure Service Bus and Event Hubs protocol guide - Azure Service Bus
Content Source: articles/service-bus-messaging/service-bus-amqp-protocol-guide.md
Service: service-bus-messaging
GitHub Login: @spelluru
Microsoft Alias: spelluru

@conniey - Thanks for the feedback. The other two articles don't exist any more, so I cleaned up the section. Thanks.

Thank you @spelluru,
@conniey, Thank you for providing the feedback. You will be able to see the proposed changes online soon. We will now proceed to close this issue out. Thank you!


Opening properties of several folders and then clicking ok can frequently cause crash

Describe the bug
this is a  bit hard to trace, but I've been able to reproduce it a couple times. The program hangs/freezes for a second then crashed without any trace (no crash notification). Checked the log file, no crash report.
To Reproduce
Keep in mind this does not happen all the time, hence why it may be a tad bit trickier to reproduce

Go to any directory containing multiple folders
Select several folders (or just select all folder with ctrl + a)
Right click to open context menu, then select properties
Click ok while the size and total files are being calculated
If it exits the properties window just fine, repeat step 3-4

Expected behavior
there should be no crash no matter how many times the properties windows is getting opened and closed
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS Version: Windows 10 19042.746 (20H2)
App version: v1.1

Additional context
At every trial/test, I always got it after a couple times of properties window closing just fine / normally. It seems that the program freezes/hangs for a second, sometimes several second before it actually crashed / got terminated. One more thing, I first encountered this bug when I wanted to check folder size that contains hidden/system file. Maybe there's correlation, or maybe this is just some sort of stack overflow error (since I repeatedly do the properties opening/closing action, but the think is I encountered this bug at first just by opening properties one time), I don't know.
Log file
I'm not sure the log file will help a lot in this case. There are no trace of crash in debug.txt, but there appears to be termination log in debug_fulltrust.txt, I don't know much other than that.
debug.txt
debug_fulltrust.txt

See #2714

See #2714


fix: plural_forms, not plural-forms

According to the description, the correct key name is plural_forms, not plural-forms.
Another implementation of po2json also uses plural_forms.

Jed also uses plural_forms.


Filter for diego

can you help me the filters that i would require to see the Diego cells and the applications running on Diego containers

Hey,
we've been running this Diego for awhile now but haven't pushed those things (filters, dashboards, etc) to this public repo yet.  I'll try to get an that pushed today to bring this repo up to date with those features.

Now that the I can reach githhub again, I've updated master to include our latest features.  Take a look hopefully that has what you want.

Thank you @mjseid
One more thing when i am trying to run it on the browser the metrics is not coming, i checked if the influx db is there generating the metrics  but it is working fine.  The error that it is giving is ""error":"missing required parameter "q""". Any option if you can help me with.
Thanks in advance

Looks like you tried to include a screenshot but unfortunately I can't see it.
Are you saying that the grafana dashboards are not working?  The commit I made included an update to grafana 3.1 and some of the dashboards require that newer version.  If you are running on the same docker host you were previously, can you make sure to remove the old docker images using "docker-compose rm -v" and "docker rmi cfmetrics_grafana && docker rmi cfmetrics_heka"
Also just to make sure can you view the heka dashboard (localhost:4352) and verify that the decoders are correctly processing the incoming messages (process column large, process failure column small).

I'm assuming you've got this figured out.  If not feel free to re-open the ticket.  Thanks


How unique is build id?

I am guessing the build id unique per build pipeline. I am not sure if the scope is much bigger. In what scope the build id is unique? Is it unique only per build pipeline or per team project?
Same question for release id.
Thanks.

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

ID: 0fa5a83c-70fd-6925-a8c8-256b21c21a2e
Version Independent ID: 75f4a136-7245-7878-2953-6b68272fe4ec
Content: Build pipeline options - Azure Pipelines
Content Source: docs/pipelines/build/options.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

After a quick visual check using the build rest API and they seem to be unique across the organization.

@arefinsami Thanks for your feedback. I'll update the doc to be clearer. The change will go out in the next update of the doc.

@jasonrberk -- Jason, thank you for your followup.  I'm an editor, not a subject expert-- this section is the nearest I've found for BuildId:

Tokens

and this for ReleaseID:

Release

@steved0x -- Steve, please add/edit these suggestions.


Temporarily disable sphinx warnings as errors

Temporarily disable sphinx warnings as errors
Description
Doc generation is currently broken in the build pipeline, thus blocking other PRs from passing tests.
This PR temporarily removes -W flag from the call to sphinx-build so that warnings are not treated as errors. Ideally we should get rid of such warnings.
Status

work in progress

Checklist (tick everything that applies)

[ ] Code linting (required, can be done after the PR checks)
[x] Documentation
[ ] Tests
[ ] ICLA (required for bigger changes)

Coverage decreased (-0.003%) to 88.217% when pulling 9cf49ee57afca573ff569d0ee150c2e397cd5302 on fjros:disable-sphinx-warning-as-error into f3c7474b8889bbaf31ce1fb1f2f56ba5a7c9ec30 on apache:trunk.

👍

Thanks for your contribution and sorry for being late to the game.
I don't think removing this flag is a good idea - it will just mask real issues. Instead of removing this flag we should fix the actual root cause which is causing the warning.
I quickly checked the docs build locally and since we merged this change, quite some documentation "regressions" were introduced. We should fix all those issues and then re-enable the flag to make sure we won't commit and merge bad code into master again.


Minification produces "let t,t,t,t,t,t..."

Bug report

What is the current behavior?
Minification of duckdb-browser-eh.worker.js from @duckdb/duckdb-wasm (a file which is already minified) changes this:
"use strict";var duckdb=(()=>{var Gc=Object.create;var xi=Object.defineProperty;

to this:
"use strict";var duckdb=(()=>{let t,t,t,t,t,t,t,t,t,t,t,t,t,t,t,t,t,t,t,t,t,t,t,t;var Ai,Li,Sn,Gc=Object.create,xi=Object.defineProperty

This causes a SyntaxError.
If the current behavior is a bug, please provide the steps to reproduce.

Create a webpack project
Add @duckdb/duckdb-wasm 1.28.1-dev106.0 as a dependency
Include dist/duckdb-browser-eh.worker.js, such as via the following snippet:

  const workerUrl = new<EMAIL_ADDRESS>import.meta.url);
  const moduleUrl = new<EMAIL_ADDRESS>import.meta.url);
  const worker = new Worker(workerUrl);
  const logger = new duckdb.ConsoleLogger();
  const db = new AsyncDuckDB(logger, worker);
  await db.instantiate(moduleUrl.toString());

Run a production webpack build.
Observe that the output file (e.g. /_next/static/media/duckdb-browser-eh.worker.a53c8b41.js) contains syntax errors.

What is the expected behavior?
The file should minify with no syntax errors.
Other relevant information:
webpack version:  5.90.0 (bundled with next^14.2.5)
Node.js version: 20.12.2
Operating System: Windows 10

On inspection, it appears that the minifier in question is Next's SWC. Webpack is not performing minification, and setting swcMinify=false in next.config fixes this.


Add developer documentation

This change adds documentation for developers. Right now it's more like a collection of tips.
The motivation for this change are some questions we have received from our users via github issues. Some things keep coming up repeatedly, and I figured we could have a place for the most useful items.
It's a bit of a work in process, but then again everything always is 🙂

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.063%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1160

Relevant Lines:
1240

💛  - Coveralls

I also have a PR coming up that's based on this: https://github.com/mikavilpas/parjs/pull/1
I couldn't target this repo's master because github doesn't support stacked pull requests... I will change the target branch if/when this is merged, otherwise the diff is too confusing.


Woff2 mime type

When encoding a .woff2 font - the output is in the wrong mime type: application/octet-stream not application/font-woff2 ?

+1