Fix cronet transport crash

We received a crash report on user device
0x000000010a87195c	(CronetChromeWebView -bidirectional_stream_c.cc:181 )		bidirectional_stream_write
0x0000000105fc06dc	(Google -cronet_transport.cc:1108 )		execute_from_storage(stream_obj*)
0x0000000105fc06dc	(Google -cronet_transport.cc:1108 )		execute_from_storage(stream_obj*)
0x0000000105fdeea4	(Google -connected_channel.cc:135 )		connected_channel_start_transport_stream_op_batch(grpc_call_element*, grpc_transport_stream_op_batch*)
0x0000000105fd5d7c	(Google -message_compress_filter.cc:442 )		compress_start_transport_stream_op_batch(grpc_call_element*, grpc_transport_stream_op_batch*)
0x0000000105fd48e0	(Google -http_client_filter.cc:463 )		http_client_start_transport_stream_op_batch(grpc_call_element*, grpc_transport_stream_op_batch*)
0x0000000105fe8dbc	(Google -exec_ctx.cc:40 )		grpc_core::ExecCtx::Flush()
0x0000000105ffa854	(Google -exec_ctx.h:124 )		grpc_core::ExecCtx::~ExecCtx()
0x0000000105ffb7ec	(Google -exec_ctx.h:122 )		grpc_call_start_batch
0x0000000105f5db18	(Google -GRPCWrappedCall.m:279 )		-[GRPCWrappedCall startBatchWithOperations:errorHandler:]
0x0000000105f56aac	(Google -GRPCCallLegacy.m:435 )		-[GRPCCall writeMessage:withErrorHandler:]
0x000000018ef6d9a4	(libdispatch.dylib + 0x0005a9a4 )		_dispatch_call_block_and_release
0x000000018ef6e520	(libdispatch.dylib + 0x0005b520 )		_dispatch_client_callout
0x000000018ef1a8a0	(libdispatch.dylib + 0x000078a0 )		_dispatch_lane_serial_drain$VARIANT$mp
0x000000018ef1b290	(libdispatch.dylib + 0x00008290 )		_dispatch_lane_invoke$VARIANT$mp
0x000000018ef24788	(libdispatch.dylib + 0x00011788 )		_dispatch_workloop_worker_thread
0x000000018efbfb70	(libsystem_pthread.dylib + 0x0000bb70 )		_pthread_wqthread

We examined the code in cronet_transport.cc and determined that in certain cases, s->cbs could have been already set to nullptr, prior to us still trying to use it bidirectional_stream_write(s->cbs, stream_state->ws.write_buffer ....
This PR adds a few more checks that were actually found in some other parts of the code.
Tested with: bazel build src/core/ext/transport/cronet:grpc_transport_cronet_client_secure

Hm, not sure why, but the Sanity tests are failing on ./tools/distrib/clang_tidy_code.sh. If I run that script on master, there is no error. But after I touch cronet_transport.cc in this PR, there's a whole bunch of error in that file - all of the stuff I haven't touched in this PR. Maybe I am missing something ...
Error while trying to load a compilation database:
Could not auto-detect compilation database for file "src/core/ext/transport/cronet/transport/cronet_transport.cc"
No compilation database found in /local-code/src/core/ext/transport/cronet/transport or any parent directory
fixed-compilation-database: Error while opening fixed database: No such file or directory
json-compilation-database: Error while opening JSON database: No such file or directory
Running without flags.
160 warnings generated.
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:368:5: warning: Use of memory after it is freed [clang-analyzer-cplusplus.NewDelete]
    CRONET_LOG(GPR_DEBUG, "Freed %p. Now %d in the queue", oas,
    ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:54:38: note: expanded from macro 'CRONET_LOG'
    if (grpc_cronet_trace.enabled()) gpr_log(__VA_ARGS__); \
                                     ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1454:7: note: Assuming the condition is false
  if (op->send_initial_metadata &&
      ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1454:33: note: Left side of '&&' is false
  if (op->send_initial_metadata &&
                                ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1482:3: note: Calling 'execute_from_storage'
  execute_from_storage(s);
  ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:394:53: note: Assuming the condition is true
  for (struct op_and_state* curr = s->storage.head; curr != nullptr;) {
                                                    ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:394:3: note: Loop condition is true.  Entering loop body
  for (struct op_and_state* curr = s->storage.head; curr != nullptr;) {
  ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:401:9: note: Assuming the condition is true
    if (curr->done) {
        ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:401:5: note: Taking true branch
    if (curr->done) {
    ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:403:7: note: Calling 'remove_from_storage'
      remove_from_storage(s, curr);
      ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:361:7: note: Left side of '||' is false
  if (s->storage.head == nullptr || oas == nullptr) {
      ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:361:3: note: Taking false branch
  if (s->storage.head == nullptr || oas == nullptr) {
  ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:364:3: note: Taking true branch
  if (s->storage.head == oas) {
  ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:366:5: note: Memory is released
    delete oas;
    ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:368:5: note: Assuming the condition is true
    CRONET_LOG(GPR_DEBUG, "Freed %p. Now %d in the queue", oas,
    ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:54:9: note: expanded from macro 'CRONET_LOG'
    if (grpc_cronet_trace.enabled()) gpr_log(__VA_ARGS__); \
        ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:368:5: note: Taking true branch
    CRONET_LOG(GPR_DEBUG, "Freed %p. Now %d in the queue", oas,
    ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:54:5: note: expanded from macro 'CRONET_LOG'
    if (grpc_cronet_trace.enabled()) gpr_log(__VA_ARGS__); \
    ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:368:5: note: Use of memory after it is freed
    CRONET_LOG(GPR_DEBUG, "Freed %p. Now %d in the queue", oas,
    ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:54:38: note: expanded from macro 'CRONET_LOG'
    if (grpc_cronet_trace.enabled()) gpr_log(__VA_ARGS__); \
                                     ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:751:26: warning: Forming reference to null pointer [clang-analyzer-core.NonNullParamChecker]
    grpc_mdelem mdelem = curr->md;
                         ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1031:7: note: Assuming the condition is true
  if (stream_op->send_initial_metadata &&
      ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1031:7: note: Left side of '&&' is true
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1032:7: note: Assuming the condition is true
      op_can_be_run(stream_op, s, &oas->state, OP_SEND_INITIAL_METADATA)) {
      ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1031:3: note: Taking true branch
  if (stream_op->send_initial_metadata &&
  ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1041:9: note: Assuming the condition is false
    if (t->use_packet_coalescing) {
        ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1041:5: note: Taking false branch
    if (t->use_packet_coalescing) {
    ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1048:5: note: Calling 'convert_metadata_to_cronet_headers'
    convert_metadata_to_cronet_headers(
    ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:728:10: note: Assuming pointer value is null
  while (curr != nullptr) {
         ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:728:3: note: Loop condition is false. Execution continues on line 732
  while (curr != nullptr) {
  ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:733:7: note: Assuming 'deadline' is not equal to GRPC_MILLIS_INF_FUTURE
  if (deadline != GRPC_MILLIS_INF_FUTURE) {
      ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:733:3: note: Taking true branch
  if (deadline != GRPC_MILLIS_INF_FUTURE) {
  ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:748:3: note: Null pointer value stored to 'curr'
  curr = metadata->list.head;
  ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:750:3: note: Loop condition is true.  Entering loop body
  while (num_headers < num_headers_available) {
  ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:751:26: note: Forming reference to null pointer
    grpc_mdelem mdelem = curr->md;
                         ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1025:25: error: function 'execute_stream_op' exceeds recommended size/complexity thresholds [readability-function-size,-warnings-as-errors]
static enum e_op_result execute_stream_op(struct op_and_state* oas) {
                        ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1025:25: note: 460 statements (threshold 450)
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1525:3: warning: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119 [clang-analyzer-security.insecureAPI.strcpy]
  strcpy(ct->host, target);
  ^
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1525:3: note: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
Suppressed 156 warnings (156 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
1 warning treated as error
FAILED: src/core/ext/transport/cronet/transport/cronet_transport.cc [ret=1, pid=13, time=7.3sec]

oh, looks like this is the relevant part
/local-code/src/core/ext/transport/cronet/transport/cronet_transport.cc:1025:25: error: function 'execute_stream_op' exceeds recommended size/complexity thresholds [readability-function-size,-warnings-as-errors]
static enum e_op_result execute_stream_op(struct op_and_state* oas) {

I did touch execute_stream_op.

I guess this is somewhat of a concern, that adding about 12 lines to the function will result in "exceeds recommended size/complexity thresholds". Perhaps we need to refactor this massive function in the future.

Failed tests:

FAILED: build_artifact.csharp_ext_macos_ios
FAILED: cloud_to_prod:nodepurejs:default:empty_unary:tls
FAILED: cloud_to_prod:nodepurejs:default:server_streaming:tls
FAILED: cloud_to_prod_auth:nodepurejs:default:oauth2_auth_token:tls
FAILED: cloud_to_prod_auth:nodepurejs:default:per_rpc_creds:tls

Seems unrelated to this PR.

TGP is clean. Merging.


implement chores (sheduled tasks)

Curious what you think about this..
There are several things I want my various bots to do on a daily or hourly basis (like check the status of RSS feeds).  It's possible now to create a handler with a pattern of "", and use it to setup and manage timers inside my handler.  I think the attached approach is more elegant.  It gives the user the ability to setup a special kind of handler called a 'chore'.  The user specifies a chore type, which includes a crontab-style schedule, and the bot figures out the duration to the next date/time that meets the schedule, and fires a goroutine to launch the user-specified run() function attached to the chore when the timer expires.  Some simple handlers are included to get the bot to list all registered chores as well as, start, and stop individual chores as needed.

This seems very useful, what do you think @danryan ?


Compiler support of macOS

When we test on macOS what compiler we have in mind? gcc (from homebrew) or clang (default Xcode or homebrew).
For example there is an issue with OpenMP.
When we want to add find_package(OpenMP) to Cmake, it not trivial to compile the code with threads with clang.
This is workaround for
CMakeLists.txt (it selects the brew clang compiler that only supports openmp; the default one that comes with Xcode doesn't):
set(CMAKE_C_COMPILER "/usr/local/opt/llvm/bin/clang")
set(CMAKE_CXX_COMPILER "/usr/local/opt/llvm/bin/clang++")
set (CMAKE_EXE_LINKER_FLAGS "-Wl,-rpath,/usr/local/opt/llvm/lib -L/usr/local/opt/llvm/lib -lomp" )
set (CMAKE_SHARED_LINKER_FLAGS "-Wl,-rpath,/usr/local/opt/llvm/lib -L/usr/local/opt/llvm/lib -lomp" )
Not high priority but always nice to have this clean.

The equivalent paths on ARM64 Macs (for example, mine) are:
set(CMAKE_C_COMPILER "/opt/homebrew/opt/llvm/bin/clang")
set(CMAKE_CXX_COMPILER "/opt/homebrew/opt/llvm/bin/clang++")
set (CMAKE_EXE_LINKER_FLAGS "-Wl,-rpath,/opt/homebrew/opt/llvm/lib -L/opt/homebrew/opt/llvm/lib -lomp" )
set (CMAKE_SHARED_LINKER_FLAGS "-Wl,-rpath,/opt/homebrew/opt/llvm/lib -L/opt/homebrew/opt/llvm/lib -lomp" )

We should at least be factoring out the Homebrew prefix.

I am not a big fan of putting path in makefiles, could it be possible to isolate those lines in separate files that would be called inside a CMakeLists.txt ? That way CMakeLists.txt stays generic and some make_homebrew.txt and make_llvm.txt are maintained separately. Or maybe we call this files CMakeLists_homebrew.txt and CMakeLists_llvm.txt and do a link before compiling to CMakeLists.txt ? What do you think ?

Now that I give it some thought, aren't CMAKE_C_COMPILER and CMAKE_CXX_COMPILER usually defined in the (command line/IDE) environment and just used in the CMake files? Precisely so that there are no system dependent paths inside the CMake files. The LINKER_FLAGS feel like they are a similar situation, too.

I'm not familiar with cmake, but in make you usually assume that CC, CXX, FC, etc are defined in the environment. I would suggest that we have separate environment files, like @auraoupa suggests, so that the basic CMakeLists.txt file is platform agnostic.
In general though, we should set up a separate repository in the nextsimdg organisation. Called something like "environment". In there, we can then store setups for different machine types and IDEs. A structure could look like
nextsimdg
|- environment
    |- mac_homebrew
        |- nextsimdg.src etc with values from @pminak
    |- mac_macports
        |- files
    |- Eclipse
        |- files
    |- etc. etc.

Sound ok? If we agree, can @auraoupa set up the repository?

@einola I would just include this as information in documentation (docs on installation). @timspainNERSC is right that flags are defined outside cmake.
It's a question of maintenance, I would support default configuration on linux and mac, and mention in documentation other options.

Since we have multiple people successfully compiling on macOS, I think this issue can be closed.


docs: update from the project clone command

I did not understand, isn´t  https://github.com/electron-react-boilerplate/electron-react-boilerplate  pattern repository ?


There is bug while using the Firefox

After a category filter operation and going back to the all categories filter operation the initial design of the page is affected.

You need to be more specific. Could you create a demonstration of this, perhaps on jsbin.com or CodePen.io?


Swin Transformer stochastic_depth_prob arguments

🐛 Describe the bug
model =models.swin_b(pretrained=None,progress=False,stochastic_depth_prob=0.5,num_classes =100)
I get TypeError: torchvision.models.swin_transformer._swin_transformer() got multiple values for keyword argument 'stochastic_depth_prob'
Versions
[pip3] flake8==6.0.0
[pip3] mypy==1.8.0
[pip3] mypy-extensions==1.0.0
[pip3] numpy==1.26.4
[pip3] numpydoc==1.5.0
[pip3] torch==2.3.1
[pip3] torchaudio==2.3.1
[pip3] torchvision==0.18.1
[pip3] triton==2.3.1
[conda] _anaconda_depends         2024.02             py311_mkl_1
[conda] blas                      1.0                         mkl
[conda] ffmpeg                    4.3                  hf484d3e_0    pytorch
[conda] libjpeg-turbo             2.0.0                h9bf148f_0    pytorch
[conda] mkl                       2023.1.0         h213fc3f_46344
[conda] mkl-service               2.4.0           py311h5eee18b_1
[conda] mkl_fft                   1.3.8           py311h5eee18b_0
[conda] mkl_random                1.2.4           py311hdb19cb5_0
[conda] numpy                     1.26.4          py311h08b1b3b_0
[conda] numpy-base                1.26.4          py311hf175353_0
[conda] numpydoc                  1.5.0           py311h06a4308_0
[conda] pytorch                   2.3.1           py3.11_cuda11.8_cudnn8.7.0_0    pytorch
[conda] pytorch-cuda              11.8                 h7e8668a_5    pytorch
[conda] pytorch-mutex             1.0                        cuda    pytorch
[conda] torchaudio                2.3.1               py311_cu118    pytorch
[conda] torchtriton               2.3.1                     py311    pytorch
[conda] torchvision               0.18.1              py311_cu118    pytorch

Hi @FranciscoPark ,
I realize it's not completley obvious (especially since we support **kwargs), but these model builders functions like models.swin_b are meant to implement a very specific architecture and not all parameters are allowed. For swin_b, stochastic_depth_prob is hard-coded to 0.5 and it should not be modified.
Since you're not using pre-trained weights anyway, I suggest that you instanciate a model directly from the SwinTransformer base class instead: https://github.com/pytorch/vision/blob/f96c42fca53230057b16941b078a0a9eee06e20f/torchvision/models/swin_transformer.py#L508
You might have to hard-code a few of the swin_b parameters yourself, you can look at the code to figure out the values. HTH.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.7 to before v5.10.75",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=e11f5bd8228fc3760c221f940b9f6365dbf3e7ed",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=a7b45024f66f9ec769e8dbb1a51ae83cd05929c7"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "e11f5bd8228fc3760c221f940b9f6365dbf3e7ed",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "a7b45024f66f9ec769e8dbb1a51ae83cd05929c7",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "drm/edid: In connector_bad_edid() cap num_of_ext by num_blocks read\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.75 by commit a7b45024f66f9ec769e8dbb1a51ae83cd05929c7, it was introduced in version v5.7 by commit e11f5bd8228fc3760c221f940b9f6365dbf3e7ed. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1002037


Download file does not include cumulative index score

Describe the bug
The .xls and .csv files in the download packet do not include the cumulative index score or any indication as to whether or not the census block group is prioritized.
To Reproduce
Steps to reproduce the behavior:

Click on the download link
Open zip file
Open .xls
See that there is no score value or prioritized yes/no indicator

Expected behavior
The score value, score percentile and top 25th percentile True/False data should be included as columns in the file.

Putting in icebox for now because we don't know which score we'll be using for this and what that type of score is (in/out or percentile), and we don't expect to know by the end of the next sprint.


PANIC: Missing emulator engine program for 'x86' CPU

Steps to Reproduce

start project in IntelliJ
choose emulator in "Device Selector"
error message appears (bottom right)

Alternative approach

open Android Device Manager
start emulator
Android Monitor recognizes emulator and attaches itself
"Device Selector" does not seem to update on previous event
run app
error message "No Connected Devices Found" (allthough Android Monitor is attached)

Screenshots

comments

as you can see, "flutter doctor" has no issues
Android Monitor is attached
AVD Manager can start emulator, but "Device Selctor" not (PANIC message)
This happens every time I update Android plugins for IntelliJ

Logs
Run your application with flutter run and attach all the log output.
Run flutter analyze and attach any output of that command also.
λ flutter analyze
Running "flutter packages get" in fludder...
Analyzing D:\WORKSPACE\DART\fludder...
error - The constructor returns type 'dynamic' that isn't of expected type 'Widget' at test\widget_test.dart:15:29 - strong_mode_invalid_cast_new_expr
error - Undefined class 'MyApp' at test\widget_test.dart:15:33 - undefined_class
hint - Unused import at test\widget_test.dart:10:8 - unused_import
3 issues found.
(Ran in 8.0s)
Flutter Doctor
D:
λ flutter doctor -v
[  +18 ms] [D:\flutter] git rev-parse --abbrev-ref --symbolic @{u}
[ +131 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/alpha
[        ] [D:\flutter] git rev-parse --abbrev-ref HEAD
[  +87 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] alpha
[        ] [D:\flutter] git ls-remote --get-url origin
[  +88 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[        ] [D:\flutter] git log -n 1 --pretty=format:%H
[  +88 ms] Exit code 0 from: git log -n 1 --pretty=format:%H
[        ] 553fc4264e5c3bd85531e73fb7d89bdfab0fa3ec
[        ] [D:\flutter] git log -n 1 --pretty=format:%ar
[  +74 ms] Exit code 0 from: git log -n 1 --pretty=format:%ar
[        ] 15 hours ago
[   +1 ms] [D:\flutter] git describe --match v*.. --first-parent --long --tags
[ +173 ms] Exit code 0 from: git describe --match v*.. --first-parent --long --tags
[        ] v0.0.21-1-g553fc4264
[ +783 ms] [√] Flutter (on Microsoft Windows [Version 10.0.16299.248], locale de-CH, channel alpha)
[        ]     • Flutter version 0.0.22-pre.1 at D:\flutter
[        ]     • Framework revision 553fc4264e (15 hours ago), 2018-03-01 09:30:25 -0800
[        ]     • Engine revision 6921873c71
[        ]     • Tools Dart version 2.0.0-dev.16.0
[        ]     • Engine Dart version 2.0.0-edge.da1f52592ef73fe3afa485385cb995b9aec0181a
[ +123 ms] java -version
[  +95 ms] [√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[        ]     • Android SDK at C:\Users\ecoissy\AppData\Local\Android\sdk
[        ]     • Android NDK location not configured (optional; useful for native profiling support)
[        ]     • Platform android-27, build-tools 27.0.3
[        ]     • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
[        ]     • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)
[   +1 ms] [√] Android Studio (version 3.0)
[        ]     • Android Studio at C:\Program Files\Android\Android Studio
[        ]     • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)
[ +103 ms] [√] IntelliJ IDEA Ultimate Edition (version 2017.3)
[        ]     • Flutter plugin version 22.2.2
[        ]     • Dart plugin version 173.4548.30
[   +7 ms] C:\Users\ecoissy\AppData\Local\Android\sdk\platform-tools\adb devices -l
[+4181 ms] Exit code 0 from: C:\Users\ecoissy\AppData\Local\Android\sdk\platform-tools\adb devices -l
[        ] List of devices attached
[  +21 ms] C:\Users\ecoissy\AppData\Local\Android\sdk\platform-tools\adb devices -l
[ +118 ms] List of devices attached
[        ] [-] Connected devices
[        ]     • None
[  +16 ms] "flutter doctor" took 5’291ms.
[ +258 ms] ensureAnalyticsSent: 254ms
[   +5 ms] exiting with code 0

For more information about diagnosing and reporting Flutter bugs, please see https://flutter.io/bug-reports/.

Looks like you are using an old Flutter version.
This version frequently caused issues with flutter upgrade not updating.
The latest version is Flutter 0.1.7

@zoechi
Oh, does that mean I have to manually install flutter then? I did flutter upgrade before experiencing this issue.

I always suggested to just purge the flutter install and reinstall.
There are other ways with git reset --hard and probably additional commands, but I never dove into that path. Reinstall is simpler to explain by just pointing at the install instructions web page ;-)

I've deleted my local repo and did a fresh checkout.
It works now... but only with following approach:

use AVD Manager to start device
run flutter doctor  to verify if flutter picks up emulator
run flutter run

Device Selector in IntelliJ still brings PANIC error message. And running project from IntelliJ still yields "No Connected Devices Found", allthough I have an emulator running.
λ flutter --version Flutter 0.1.5 • channel beta • https://github.com/flutter/flutter.git Framework • revision 3ea4d06340 (8 days ago) • 2018-02-22 11:12:39 -0800 Engine • revision ead227f118 Tools • Dart 2.0.0-dev.28.0.flutter-0b4f01f759
I have Flutter 0.1.5 now.

Have you tried restarting IntelliJ (just to be sure)?

Yes, I've done many things after a fresh checkout

invalidate caches and restart
deleted old emulator
added new one in AVD Manager
re-assigned Android SDK

All of this did not help avoiding PANIC error message.

I'm out of ideas. Can you please add the exact error message. I haven't found it above.

D:\ is not a network drive by any chance?

D:\ is a local drive.
The exact error message is in the first screenshot (bottom right).

cc @devoncarew

@geekrumper, it looks like the Android Monitor tool from the Android plugins is having trouble executing your emulator image. I would try creating a new one (and you may want to delete the old one as well). I don't believe this is an issue with the flutter intellij tooling per se, but more a local configuration issue, so am closing. Thanks for the report!

I'm seeing the same error in Android Studio. I can manually start the AVD from the AVD Manager, but when I select it in the toolbar ("Flutter Device Selection") I see the error described above. I have deleted the device and recreated it.

I am getting the same error.
I did a clean install of flutter on windows 10.
New Install of Android studio.
I can open the device emulator in Android, but not in Flutter, gives error
"Error Opening Emulator: PANIC: Missing emulator engine program for 'x86' CPU."

This is how I resolved this on Windows 10 where I was getting the same error.
Installed components:
flutter 1.5 (beta)
Android Studio 3.0.1 (64 bit)
IntelliJ Ultimate 2017.3.4
Java JDK 1.8.0_61 (64 bit only)
Envirs:
%ANDROID_HOME%
C:\Users\ameserol\AppData\Local\Android\Sdk
%ANDROID_SDK_ROOT%
C:\Users\ameserol\AppData\Local\Android\Sdk
%path% includes these in this order:
C:\Users<user>\AppData\Local\Android\Sdk\emulator
C:\Users<user>\AppData\Local\Android\Sdk\emulator\bin
C:\Users<user>\AppData\Local\Android\Sdk\platform-tools
C:\Users<user>\AppData\Local\Android\Sdk\tools
C:\Users<user>\AppData\Local\Android\Sdk\tools\bin
C:\Users<user>\AppData\Local\flutter\bin
C:\Users<user>\AppData\Local\dart-sdk
I guessed that since I was all x64 for all components that IntelliJ might be having a problem with the fact that the out-of-box configured virtual devices over in Android Studio are x86 and not x86_64.
My solution was to:

open Android Studio
open AVD Manager form Tools > Android
edit the config for the existing x86 based devices to instead us an API of type x86_64.
save and exit all the way out.
open IntelliJ and now those devices set x86_64 types work.

@devoncarew please refer to one of my previous comments:

Yes, I've done many things after a fresh checkout

invalidate caches and restart
deleted old emulator
added new one in AVD Manager
re-assigned Android SDK
All of this did not help avoiding PANIC error message.

Deleting and creating a new emulator does not solve the problem.

@cyanfr
great input there. I was able to start an emulator (x86_64) through the "Device Selector" without any problems. Still, IntelliJ does not seem to pick it up, which means I cannot use the Run-Command (Shift-F10) from IntelliJ.
This leads to an alternative work process

by having flutter be ran in a terminal
making source code changes in an editor
focussing back to terminal and hitting 'r' for hot reload

@devoncarew
Why should this be an image problem, if the AVD Manager can start the image without any issues?

I was seeing this same problem when trying to run this with a Nexus 5x API 27 X86 configuration. I created a new Virtual device from intelli, tools > Android > AVD Manager. Pixel XL API 27, and I no longer get this error on the new virtual device.

Hey guys,
I appreciate the community effort and was able to gather a couple of useful information:

make sure the pre-defined emulator is removed (deleted), this one will mostly cause the errors
use the appropriate system type for the emulator, e.g. "x86_64" if you are using 64-bit OS with IntelliJ
give your new emulator an easy name, since you'll be starting it from the terminal (e.g. "nexus")
launch the emulator in a terminal with the "emulator" command in Android tools
%ANDROID_HOME%\tools\emulator -avd %EMULATOR_NAME%

@geekrumper (and @abarrafo and @cyanfr), thanks much for the analysis and summary! Very useful, and likely an issue that many people are hitting.
I only see one copy of emulator in my android sdk tools directory (I'm on a mac). From the notes above it looks like at least windows has linux has two variants (emulator64-x86, emulator-x86_64), and perhaps windows as well.

@devoncarew A couple more things I found that seemed to help.  Details in case anyone else stumbles on this thread and they might help.

Revised what I had above and added just following global envir vars...

Added to Path:
C:\Users\my_user\AppData\Local\flutter\bin
C:\Users\my_user\AppData\Local\Android\Sdk\platform-tools
C:\Users\my_user\AppData\Local\Android\Sdk\tools
C:\Users\my_user\AppData\Local\Android\Sdk\tools\bin
New Envir Vars:
ANDROID_AVD_HOME: C:\Users\my_user.android\avd
ANDROID_EMULATOR_HOME: C:\Users\my_user.android
ANDROID_SDK_ROOT: C:\Users\my_user\AppData\Local\Android\Sdk

This next bit seemed to make more of a difference
.
I noticed that the global vars did not seem to get picked up by IntelliJ.
So in the generic settings of IntelliJ I added the following three vars same as in my system's envir.

Then in the specific project structure settings, I set the JDK SDK loc and the Android SDK loc as these did not seem to get picked up any other way.  Like so...

After this, at least on my Windows 10 box with only 64 bit apps, the AVD images started to work.
That's the fresh ones as @geekrumper says, and not the default ones.
Although I did get the settings for my new pixel 2 xl image from the ones in Android Studio.
Interestingly after I made these changes I could open both x86_64 AND x86 bit AVD images.
Now this is never from that little dropdown.  This is by starting the AVD Manager and starting the image from there.
So not from here (it always produced error):

But from here, I would pick an image and start it:

And then once the image was started, it interacted just fine with IntelliJ.
Then in the project structure specific settings I checked and added the following.
JDK path and the Android SDK path.  First JDK then Android SDK.

I am getting this same error in Android Studio 3.1.2
From where might one download emulator64-x86.exe?
These are the emulator files in %USERPROFILE%\AppData\Local\Android\Sdk\emulator\

emulator-arm.exe
emulator-check.exe
emulator-crash-service.exe
emulator-mips.exe
emulator-x86.exe
emulator.exe
emulator64-crash-service.exe

For what it's worth, i had the same issue in Ubuntu 18.04 and it was solved by doing what @cyanfr mentioned.
Can provide more information if it's necessary.

FYI @DanTup who recently added flutter emulators command and might have interest in this.

I think this may be a dupe of https://github.com/flutter/flutter-intellij/issues/2351 ?
There are two emulator binaries in the SDK and IntelliJ used to use the old one (but has very recently been changed in https://github.com/flutter/flutter-intellij/pull/2383) which would give this PANIC message. For flutter emulators --launch we use the new one if it exists and fall back to the old one otherwise.
So, I'd be curious to know if anyone having this issue still sees it using flutter emulators --launch from the command line. If not, then I think this case can be closed as a dupe of the above. If anyone can, then we'll need to do some more digging.


Inverting / Overlapping Boundaries / Offsets

I "think" I know what I'm asking about.
I'd like to have the topOffset be lower then the bottomOffset so they overlap.
The exacted behaviour is that the item will ENTER at the vertical center of the window. It will also LEAVE lower then that, at approx the bottom 1/4 of the window.
I think because I'm entering values like this, waypoint is choking something, maybe because usually those offsets do not overlap.?

I "think" I know what I'm asking about.

😂
I get what you're asking, and I haven't done this myself. Have you considered just switching them, so that you use the bottom boundary for the top, and the top boundary for the bottom?

maybe because usually those offsets do not overlap.?

This is how I use them, although we should probably allow for them to overlap.

The exacted behaviour is that the item will ENTER at the vertical center of the window. It will also LEAVE lower then that, at approx the bottom 1/4 of the window.

This is useful. Can you also provide us with:

the props that you're passing
the behavior that you're seeing

Sorry for the issues you're running into, @iDVB !

@iDVB , just checking in...did you ever figure this out? After the above post, I did a quick read of the code, and nothing immediately stood out to me as something that wouldn't support inverted boundaries. But it's possible that I missed something.

I think you are right, after I posted, I realized that how we were using them was not exactly as I thought/posted. In the end we ended up simplifying the code by writing something a bit custom.
Thanks for helping to noodle this one though!

Cool, cool. Glad you figured it out! 👌


⚠ Palworld Tunnel (Connection URL) has degraded performance

In 2e3d8fe, Palworld Tunnel (Connection URL) ($PG_PANEL/tstat.txt) experienced degraded performance:

HTTP code: 200
Response time: 31 ms

Resolved: Palworld Tunnel (Connection URL) performance has improved in 32e3dcd after 17 minutes.


LR V2.0 Block Diagram

Block Diagram with separation of PS & PL and interface which will be used between Zynq and IC's.

Overall Block Diagram is prepared

LR v1.0 render format converting from STEP into acceptable format of sketchfab, facing with problem colour conversion. https://skfb.ly/6toYZ


TS-5013: master does not build on omnios

[approve ci]

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1097/ for details.

Does this need to go into 7.0.0 as well? @bryancall

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/989/ for details.

[approve ci]

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1102/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/994/ for details.

[approve ci]

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1107/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1000/ for details.


Usage in angular app

I want to use this library in my angular application.
I do
import { jsonToHTMLTable } from 'nested-json-to-table';
// also tried
// declare var require: any;
// const { jsonToHTMLTable } = require('nested-json-to-table');

But I'm getting an error:
ERROR in ./node_modules/nested-json-to-table/lib/jsonToHTMLTable/node.js
Module not found: Error: Can't resolve 'fs' in '/home/frankie/work/projects/brainstorm/src/main/frontend/node_modules/nested-json-to-table/lib/jsonToHTMLTable'

Could you please help me to solve this?

Published, the new version is 1.0.5

Thank you, but I still can't get it working. I'm getting an error:
HomeComponent_Host.ngfactory.js? [sm]:1 ERROR ReferenceError: global is not defined
    at push../node_modules/nested-json-to-table/lib/jsonToHTMLTable/index.js.module.exports (index.js:9)
    at HomeComponent.push../src/app/components/home/home.component.ts.HomeComponent.ngOnInit (home.component.ts:65)
    at checkAndUpdateDirectiveInline (core.js:21097)
    at checkAndUpdateNodeInline (core.js:29495)
    at checkAndUpdateNode (core.js:29457)
    at debugCheckAndUpdateNode (core.js:30091)
    at debugCheckDirectivesFn (core.js:30051)
    at Object.eval [as updateDirectives] (HomeComponent_Host.ngfactory.js? [sm]:1)
    at Object.debugUpdateDirectives [as updateDirectives] (core.js:30043)
    at checkAndUpdateView (core.js:29439)

Also the previous error now is warning :smile:, but maybe this was done on purpose...

You can pass a option format: 'dom' to tell jsonToHTMLTable and it will generate a html table dom. There is a hole in current version, and I will fix it in next version.
You say that there is a warning about require('fs') now in angular cli? Maybe it will be also fixed up in next version.

I have published a new version '1.0.6' and it will resolve your issue about global. But now, you should resolve fs by yourself, which add following lines to your webpack config:
  externals: {
    fs: 'fs'
  }

The principle is that I requires fs in my code, but it only supplrt node environment. Thus if you are in browser environment, your webpack config should not resolve fs in advance.
Maybe more work than before. Best wish!

Seems I got it working, thank you!
As a feature request, would be great if you add possibility to set some css options, when creating table.
For now I made it this way:
tableHtml.className += 'ui celled selectable blue compact table';

Seems I got it working, thank you!
As a feature request, would be great if you add possibility to set some css options, when creating table.
For now I made it this way:
tableHtml.className += 'ui celled selectable blue compact table';

Ok，I will add this feature in the future, you could watch the repository to follow the precedure. I'll close the issue now. If you encounter relative bugs about this issue, you could reopen it. Nice enjoyment!


Improve 'before you begin' section in code snippets user docs

Each code snippets page has a Before You Begin section that contains multiple expands for signing up with/setting up Rafiki.money. However, after going through the steps once, you don't need to do them again. Instead of having the same set of expands on every page, let's create a stand-alone page and link to the content from the code snippets pages.

[x] Create a "Before you begin" page in the Code Snippets section that contains the content of the original 4 partial files. Delete the partial files.
[x] To ensure we use the same lang string and link(s) on each code snippet page, create the new 'before you begin' content within a partial and add to each page
[x] Update each code snippet page by changing the partials 'import' to call the new partial file

This is complete from a docs standpoint, but I'm having problems with PR #377. When that's resolved, the updates in this issue will be available.


until_ready weird behavior

I am trying to consume a very simple API, but I am hitting some unexpected behavior when using until_ready to wait for the rate limiter to be available again.
My code looks like this
struct RateLimitedClient {
    client: reqwest::Client,
    limiter: governor::DefaultDirectRateLimiter,
}

impl RateLimitedClient {
    fn new(reqs_per_minute: u32) -> Result<RateLimitedClient> {
        let reqs_per_minute =
            NonZeroU32::new(reqs_per_minute).ok_or(anyhow!("reqs_per_minute can't be zero."))?;
        let quota = governor::Quota::per_minute(reqs_per_minute);
        Ok(RateLimitedClient {
            client: reqwest::Client::new(),
            limiter: governor::RateLimiter::direct(quota),
        })
    }
}

async fn make_request(
    url: &str,
    rate_limited_client: &RateLimitedClient
) -> Result<()> {
    rate_limited_client.limiter.until_ready().await;
    if rate_limited_client.limiter.check().is_err() {
        return Err(anyhow!("Rate limit blocked request"));
    };
    let res = rate_limited_client
        .client
        .get(url)
        .send()
        .await
        .context("Error on the request.")?;
}

But I am hitting the rate limiter error (i.e. check() not being ready) every time I reach the limit, which I understand I should be hitting because of the previous until_ready() call. I am missing something?
Thanks in advance!

Oof, I think the documentation for async functions may not be good enough yet: until_ready calls check on your behalf, waiting (and calling check) until it has succeeded and the rate limit lets your call through.
You don't need to call check a second time.

Oooh, that makes total sense. Thanks for the prompt response :)


metronome 0.5.2 bump

High-level description
What features does this change enable? What bugs does this change fix?
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-4018 Metronome 0.5.2 bump on DCOS 1.12.

Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): diff
[x] Test Results: test
[x] Code Coverage (if available): N/A

@mesosphere-mergebot bump-ee

@mesosphere-mergebot Label Ready for Review

@mesosphere-mergebot bump-ee


Import Log Button should be disabled when Upload File selected but no file uploaded.

One more condition needs to be added to the disabled condition of the Import Log button to disallow clicking without any file uploaded.

A cleaner fix would be to show no file selected when there's no file uploaded, i.e. clear the file input field. That will automatically disable the button and won't leave the user wondering why he can't upload. what do you think @jgraham @martiansideofthemoon ?

@udiboy1209 , yeah good idea! Would you like to work on it? Though I'd be happy if someone completes @arpan98 's patch. I want him to have the due credit :smile:

@arpan98 's previous PR is not required with the fix I suggest.
I guess only one line needs to be added here :
$scope.logfile.val(null)

I would do it but I am a bit busy. @arpan98 you should make a new PR with this, its quite a small fix.

I would like to take this up :)

@malayaleecoder , yeah sure! Do tell me if something is unclear. I think @udiboy1209 will be a good mentor here

@martiansideofthemoon : I would need another week to look into this :( In the meantime if anyone else would like to give this a try, please do so :)

I've attempted a fix. Have a look @martiansideofthemoon

Thanks @nihal111 :smile:


Export GRPC symbols along with the C API

Hi,
Given the recent changes to the eager API, that requires GRPC, the eager API cannot be used by external libraries, because it cannot link to the GRPC symbols in TensorFlow. A possible solution is to add *grpc* to the C API version script. @asimshankar @alextp is it ok if I make a PR to add that?
Thanks,
Anthony

@eaplatanios : Could you elaborate a bit? What cannot be used by external library (the eager C API?).
If possible, I'd rather work out a scheme were we don't have to export more symbols instead of exporting *grpc* from the C API.
@akshaym would know more about how to hide those symbols.

@asimshankar Sorry I was actually being too vague. I looked into it more and the main issue appears to be this line. Creating a TFE_TensorHandle from a Tensor is not part of the public API. This forces me to include eager/c_api_internal.h, which includes dependencies on GRPC. In this case, maybe those symbols should not be exported in libtensorflow.so, but rather in libtensorflow_framework.so. This can be done by adding "//tensorflow/c/eager:c_api" as a dependency here. Do you think this is an acceptable solution? If not, how do you suggest I create the TFE_TensorHandle? Thanks! :)

By the way, my current temporary solution is to add this instead of including eager/c_api_internal.h:
#include "tensorflow/core/common_runtime/eager/tensor_handle.h"

struct TFE_TensorHandle {
  TFE_TensorHandle(const tensorflow::Tensor& t)
      : handle(new tensorflow::TensorHandle(t, nullptr, nullptr, nullptr)) {}

  tensorflow::TensorHandle* handle;
};

However, I believe that I shouldn't need to replicate code here. If you think this is ok for now, feel free to close this issue. :)

Ah, thanks for the additional information. You're using APIs that we've explicitly marked "internal" so I'm not surprised you're seeing some friction :). I'd rather not add dependencies/symbols to export things we want to hide from the public API as that will only make it harder down the road.
I think the workaround you have is okay for now. The situation will change as the eager C API matures, but for now I feel it's okay to have some hacks in code that uses APIs we don't want to commit to :)
Thanks for your understanding.

No problem. Yes, I am aware I'm using internal APIs that might change and totally understand that. I guess pointing out such issues related to how I am using those APIs might help with evolving the public API.
On another note, what happened to the nightly-libtensorflow CI builds? They seem to have been removed. Is there some alternative I could use? I was using them to obtain cross-compiled binaries to package with my Scala library.

Yup, definitely do point these out. Thanks.
Regarding nightly-libtensorflow: @av8ramit @case540 might know more.

Sorry about the disruption. We recently shuffled around our CI builds. Just took a look and Windows libtensorflow seems to be having some build issues. Will look to fix next week sometime. Also, if you were downloading the builds from TensorFlow Jenkins, that will no longer be where the build artifacts will live. Will post their new location once everything gets settled.

Yes, we are doing a huge reshuffle of our nightly build process. Please bear with us until we have it all resolved. Thanks!

@av8ramit @case540  Is there any update on when a new way to access the precompiled binaries might become available?


[chip-tool] support queue command when sending command to ICD device

This PR adds the support for enqueuing a command, and send it when a check-in message is received.
UX: prefix the command with queue, e.g.
>>> queue onoff on 1 0
CHIP:TOO: Command: queue onoff on 1 0
CHIP:TOO: Peer is a registered LIT ICD.
CHIP:TOO: The command is queued for node 1:0000000000000001

This prefix can only be used when the peer is registered to this controller:
>>> queue onoff on 2 0
CHIP:TOO: Command: queue onoff on 2 0
CHIP:TOO: The destination is not a registered LIT device.

This prefix currently only supports model commands.
>>> queue icd list
CHIP:TOO: Command: queue icd list
CHIP:TOO: The command cannot be queued!

This PR also adds a dumpstate (builtin) command, users can use that command to display the currently queued commands:
>>> dumpstate
CHIP:TOO: Command: dumpstate
.mQueuedCommands =
  [1:0000000000000001] =
    onoff on 1 0

What if ICD device is active, can we immediately flush the queued IM operations?

queue device can only work for ICD device, for non-icd, should we return error or noop?
when issuing queue command, we should  indicates user active mode trigger options as well

Based on https://github.com/project-chip/connectedhomeip/issues/31992#issuecomment-2039940169, I think we can close this PR to move away from the queue to simplify the chip-tool implementation.


Dynamic filtered search vs click search

What?
As part of this PR a search input field on this page was swapped to use the Search Gem
However what was noticed was that the Search Button becomes redundant when used in a dynamic filtering context. The user just needs to type, they don't need to click search.
Visuals
You can see this issue demoed below, as the user searches the actual search button is not used. Other use cases can be found here where the filtering is happening on blur, again the button is not necessarily used.

Possible solutions
Create a variant within the Search gem so that when on key-down filtering is being used for search, the button is changed to an icon and this is swapped to a clear icon when typing.
I note the Design System's reference to search

Considerations:
https://github.com/alphagov/tech-docs-gem/issues/262

It looks like the solution, e.g. on the organisations page https://www.gov.uk/government/organisations, has been to use the input component, when a search icon isn't needed. It seems like an unnecessary enhancement, and recent DAC audits haven't raised any concerns regarding its usage.


CircleTransformation - Ability to offset from center?

Description
Not so much a major issue but a nice-to-have feature request. Description:
I pull profile pics from an intranet site and they're all portrait style where the bottom 60% of the pic is somebody's suit and their face is closer to the middle of the upper half of the image, so when using the circle transformation their heads are getting scalped. Would be nice to have a way to supply an optional parameter that offsets where the circle is cropped from the middle of the pic.
Steps to Reproduce
Just apply the circle transformation to an image like this (no idea who that guy is, just a random google image...). At a minimum, I expect his hair to not make it into the circle image.
Expected Behavior
Ability to apply an offset to x or y from the middle of the pic. In my case, I would set y to be -50 so it's above the center.
Actual Behavior
No control over something like this right now (right?) so I have to modify the images themselves individually.
Basic Information

Version with issue: latest
Platform: Xamarin Forms latest, VS2017 latest

Ok looks like adding the CropTransformation achieves my aim above, please disregard.
 <ffimageloading:CachedImage HorizontalOptions="Center" VerticalOptions="Center"  Margin="10,0,0,0"
	WidthRequest="70" HeightRequest="70"
	DownsampleToViewSize="true"
	Source = "{Binding AvatarImg}">
    <ffimageloading:CachedImage.Transformations>
        <fftransformations:CropTransformation
            YOffset="-50"/>
        <fftransformations:CircleTransformation />
    </ffimageloading:CachedImage.Transformations>
</ffimageloading:CachedImage>


Create niec.txt

here is a link to my college official site : http://www.niecdelhi.ac.in/

@akashraj9828 Could you please provide us with a link to a page or document which proves that niec.ac.in belongs to your university? The official website says that university email domain is niecdelhi.ac.in. Thank you.

sorry niec.ac.in domain does not belong to my university, niecdelhi.ac.in is the official domain of my college
.My college comes under  GURU GOBIND SINGH INDRAPRASTHA UNIVERSITY ( http://www.ipu.ac.in/ )

@akashraj9828 Pull request merged. Thank you!


Set the initial page to blink when slidesToShow is a decimal number

When the setting page is displayed 1.65 times, the first time the page is entered, it will blink. The default setting is 1

slidesToShow should always be an integer


About Formula 6

Very wonderful work.
But in formula 6, $z^s=f_{\theta^s}(x)$, Should it be written as $z^s=h_{\theta^s}(x)$?

I read your code, it should be implemented in the upsample operation after the decoder, then there should be no problem with formula 6.


Add creating resource test to Mocha

After we have Mocha installed (#9) we need to port the tests from the script suggested in #8 to Mocha.
Acceptance criteria

[x] Running Mocha runs the create resource test
[x] Running Mocha runs the push blob test
[x] After Mocha is done, any created resource is deleted form CKAN server (provably an afterEach hook can cover that)

Closed by #13


Cannot find sdk-preview2-3131

I'm searching for .NET Core SDK version 1.0.0-preview2-003131
I can't seem to find that listed in the download archiver here:
https://github.com/dotnet/core/blob/master/release-notes/download-archive.md
Am I missing something?

@dazinator - The SDK builds for Preview 2 were 3133 for Windows and 3131 for everything else. We'll update the download archive index to include that information. This download page will have what you're looking for.
https://github.com/dotnet/core/blob/master/release-notes/download-archives/1.0.1-preview2-download.md
/cc @preetikr


Create a survey for IAO for Reporting research

Description
We discussed using a survey or some other wide-reaching method to hear from analysts how they use reporting. It appears there are wide variances in the types of reports used by teams and we are limited in time and resources to do research sessions with everyone.
DOD

[x] Draft questions
[ ] Review questions - PO, designer
[ ] Determine survey tool (UniverCITZy uses Survey Monkey)
[ ] Create survey
[ ] EA approval

@lmullane please review once you're back. https://bcgov.sharepoint.com/:w:/t/03521/EZGzM0YS7FdEk1fVyNMDbBsB4p6m95SJKD2NidHSGtxq1Q

@mpilchar, I've made some comments and track changes in the doc. Thanks for the opportunity to review.

@mpilchar @lmullane
Wanted to share out a link to the survey questions update that I worked on, @m-prodan and @arielleandrews  have already had a first pass. Feel free to leave comments so we can continue to improve them with the goal of sending out the survey mid-week this week.
https://bcgov.sharepoint.com/:w:/t/03521/EbM0jDXN6VNIuRgFQkfDxbkBvP3KFxdhhVmbEbtQt_dATQ?e=kBF5Py

@JHarrietha-AOT I'm good with the list of questions and have added in a couple thoughts for consideration

Thanks. Will look in a bit. Our ED wants to review as well.

I have built the survey using the draft questions we shared with Rhianna (it will be easy enough to update with any changes). I made one change to the demographic collection approach, which I've tagged for Loren. Also awaiting payment guidance from @lmullane

Survey was rebuilt in IAO forms. Will be sent out today


Saving the decrypted File

Hello,
Nice Editor. I installed the DS Editor on VS Studio Code. However, having decrypted the EBCDIC File how can i save the entire file.
Moreso, is there an API that i can call to decrypt an entire file?
Regards
Dare

Greetings,
Sequentials datasets opened with "Open with DS Editor" are downloaded to a local path in your computer. You can find them if you right click the document tab. Refer to the image bellow.

Saving the document will write the changes to this file.
Regarding the decrypting api, right now that functionality is not present. The editor will always save to the native mainframe codification. We did not implement this due to ZOWE Explorer extension already having an option to download the sequential datasets as text. Refer to the image bellow.

¿Are you perhaps suggesting being able the export the table format? We will take note of that regardless of your answer.
I will be around if you have any more questions.

Thanks for the response. I installed in on Windows, i am working on a Project to decrypt a Mastercard File and convert to Excel.

Am not connecting to Mainframe. The File has already been downloaded to a path on my Windows 11 computer. All i want is decrypt and save in Excel

I understand. Sadly, no single button to do that is implemented. The closest thing achievable right now through our extension is using [Ctrl]+[A], [Ctrl]+[C] and then pasting the clipboard to a excel file or text file (former is untested). Also, you can change the rows per page as much as you wish. Refer bellow.

Hope it helps.
We will take note of this use case for the future.

Ok Thanks.
290823_IPM_OUT_GCMS_F.zip
i attached the file am trying to decrypt

If I understood right, you shouldn't be posting THAT file here.

oh ok. Apologies.
Let me retrieve it

The potetial problem is saw is that that file can be accessed by anyone, it is public. Rarely anything related with credit cards should be published that lightly. If you do, you should argue the reasons why publishing such a file will not harmful in any way.
You may refer to me by mail for such things, and the file should by protected by password. And honestly, you shouldn't believe in me neither. I am just a extension developer.

Yeah, I understand. maybe i was too eager to get it resolved.

Give me something to contact you directly. aka, a mail.

<EMAIL_ADDRESS>

Although our extension offered some capabilities that enabled in a way to help this use-case, I understood that the real problem was that "Reopen with ecoding..." didn't feature an option that allowed the user to read the file as plain text. The file wasn't a sequential dataset. This doesn't fall into the scope of this extension.
I did guide the user on how to use DS Editor to extrart legible text to a text editor, but I am sure there should be other ways of doing so more effectively without our extension.
However, we did take note of an improvement to me made: Export to text or csv. We do not have this functionality and it may have legitimate use-cases.


Handle TwigComponent props tag

This PR add compatibility with the props tag introduced in Twig Component 2.11, which allows to define properties for anonymous components.
{# templates/components/Button.html.twig #}
{% props icon = null, label %}

<button>
    {{ label }}
    {% if icon %}

    {% endif %}
</button>

Codecov Report

Merging #136 (429d121) into main (1632615) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@             Coverage Diff             @@
##                main      #136   +/-   ##
===========================================
  Coverage     100.00%   100.00%
- Complexity       463       464    +1
===========================================
  Files             36        36
  Lines           1336      1338    +2
===========================================
+ Hits            1336      1338    +2

Files
Coverage Δ

src/Environment/StubbedEnvironment.php
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks


Add checks/cleaning for text input to prevent people from entering hacky stuff

I made some progress, but if you enter a player name with a double quote, it still dies.

I fixed it!
You are now free to name yourself things like:
"Melinda"
{"Sarah"}
\Jacob
Hi


How to get SNTP time using AT Command??

Hello I am using esp8266 module. SDK and AT version given below.
AT version:<IP_ADDRESS>(May 11 2016 18:09:56)
SDK version:1.5.4(baaeaebb)
I want SNTP time using AT Command.
I try following method to get SNTP Time, But it is not working and always get ERROR

SNTP configuration by:

AT+CIPSNTPCFG=1,8,"cn.ntp.org.cn","ntp.sjtu.edu.cn","us.pool.ntp.org"
ERROR

Get SNTP time by:

AT+CIPSNTPTIME?
ERROR
Plese let me know what might be possible reason ??

The firmware is too old to support this command. If you are designing a new product, recommend to use RTOS AT

So can we can we update firmware on esp8266 ?
If yes then how can we do that ?

I follow below links to update the firmware on esp8266 :
https://www.allaboutcircuits.com/projects/flashing-the-ESP-01-firmware-to-SDK-v2.0.0-is-easier-now/
But after follow above link and download new firmware I have getting below log with baudrate 74880 and then AT command not working.
ets Jan  8 2013,rst cause:1, boot mode:(3,6)
load 0x40100000, len 816, room 16
tail 0
chksum 0xef
load 0x00000000, len 0, room 8
tail 0
chksum 0xef
load 0x00000000, len 0, room 0
tail 0
chksum 0xef
csum 0xef
csum err
ets_main.c

I follow below links to update the firmware on esp8266 :
https://www.allaboutcircuits.com/projects/flashing-the-ESP-01-firmware-to-SDK-v2.0.0-is-easier-now/
But after follow above link and download new firmware I have getting below log with baudrate 74880 and then AT command not working.
ets Jan 8 2013,rst cause:1, boot mode:(3,6)
load 0x40100000, len 816, room 16
tail 0
chksum 0xef
load 0x00000000, len 0, room 8
tail 0
chksum 0xef
load 0x00000000, len 0, room 0
tail 0
chksum 0xef
csum 0xef
csum err
ets_main.c

https://www.espressif.com/sites/default/files/ap/ESP8266-IDF-AT_V2.0_1.zip and you can refer to download.config in the zip. If you want run in the 1M flash, you must compile it by yourself like https://github.com/espressif/esp-at/issues/351

I can't understand procedure given in #351 Can you explain me in details ?
If you have 1M flash binary please share with me.
Thanks,
Dinesh

I used ESPFLASH TOOLS V2.3 and SPI Mode DOUT. Now It's Working Fine.
Thanks xcguang for helping me.


Why won't this email start sending?

As with CiviCRM previously, you need to manually push the emails out the door.
Go to Adminster > System Settings > Scheduled Jobs and Execute the mailing job.

Want to get some time this week to talk about this with @ericsteuer and @little-wow


[fix] Remove tenant arg in workflow definition

current tenant in workflow only work when the first time user do not exist, when user change the tenant in workflow but tenant exist, it will be ignore, so we try to remove it from workflow, and in #40 we try to create both user and tenant vis cli instead of auto create

Codecov Report

Merging #13212 (b27b715) into dev (f233023) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev   #13212      +/-   ##
============================================
- Coverage     39.35%   39.34%   -0.01%
+ Complexity     4276     4275       -1
============================================
  Files          1067     1067
  Lines         40491    40464      -27
  Branches       4662     4644      -18
============================================
- Hits          15934    15921      -13
+ Misses        22770    22756      -14
  Partials       1787     1787

Impacted Files
Coverage Δ

...che/dolphinscheduler/api/python/PythonGateway.java
17.62% <0.00%> (-0.32%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...hinscheduler/plugin/alert/script/ScriptSender.java
70.58% <0.00%> (-1.64%)
:arrow_down:

...cheduler/plugin/alert/dingtalk/DingTalkSender.java
34.13% <0.00%> (-0.78%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

...inscheduler/plugin/registry/etcd/EtcdRegistry.java
50.69% <0.00%> (-0.35%)
:arrow_down:

.../dolphinscheduler/plugin/task/datax/DataxTask.java
36.62% <0.00%> (-0.26%)
:arrow_down:

.../dolphinscheduler/plugin/task/spark/SparkTask.java
72.56% <0.00%> (-0.25%)
:arrow_down:

...dolphinscheduler/service/storage/impl/S3Utils.java
0.00% <0.00%> (ø)

...eduler/api/dto/workflow/WorkflowCreateRequest.java
100.00% <0.00%> (ø)

... and 15 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Refactor 007, 008 and 009 test scripts

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind bug

/kind cleanup

/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
This change extracts the instructions to get the newest pod name and refactor some other instructions to facilitate the reading of the testing code.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/retest
It's weird, the approve returned status 1 but there was no output.

/retest
It's weird, the approve returned status 1 but there was no output.

I'm doing more refactoring on this, but it's not passing the 009.sh test. I'm not sure if we still require this process and if that's the case, why don't use it in the 008.sh?

I'm doing more refactoring on this, but it's not passing the 009.sh test. I'm not sure if we still require this process and if that's the case, why don't use it in the 008.sh?

Yes, I think it should be there for now at least. We could also have it in 008.sh, it just didn't flake in the same way. Still not clear why something is going in and tweaking the packages and causing the issue.

/test e2e-ubuntu-focal

@vjayaramrh @johnbelamaric I think this change is ready to be merged.

/lgtm

/override e2e-fedora-34


array_key_exists(): The first argument should be either a string or an integer

Describe the bug
When I try to use the activity to write a log I get array_key_exists(): The first argument should be either a string or an integer
To Reproduce
activity()->log('Look mum, I logged something');

Hey,
we will need the full stacktrace to check where the error occurs.
would also be nice if you could use our bug issue template and fill in all requested details.
**Describe the bug**
A clear and concise description of what the bug is.

**To Reproduce**
Things needed to reproduce the error.
eg: Configuration, Migration, Model, Controller/Command

**Expected behavior**
A clear and concise description of what you expected to happen.

**Screenshots**
If applicable, add screenshots to help explain your problem.

**Versions (please complete the following information)**
- PHP:
- Database:
- Laravel:
- Package:

**Additional context**
Add any other context about the problem here.

**Exception**
The thrown exception message.

**Stack Trace**
The full stack trace of the thrown exception.


import gpx without gpsbabel

based on this tweet: https://twitter.com/eumiro/status/1257008378800484352?s=20
> Also I want to find a fast Pythonic way to read the GPX files to avoid (fast but still an extra step) gpsbabel.
I've had success with gpxpy (and I think now the slow datetime recognition is improved).
Basic usage:
with open(filename, 'rb') as gpx_file:
	gpx = gpxpy.parse(gpx_file, parser='lxml')
	#gpx = gpxpy.parse(gpx_file, parser='minidom')

# loop through gpx file and extract each point
for track in gpx.tracks:
	for segment in track.segments:
		for p in segment.points:
                # do something here

To convert lat/long to UTM grid, you can use this package: https://github.com/Turbo87/utm
import utm
utm.from_latlon(51.2, 7.5)

my opinion: rather than convert to a table/csv, I'd just convert the gpx to an array of dicts (one for each gps point), using the above tools. Then use pandas.from_dict().

Thank you, I have actually originally started with gpxpy and utm (I'm already using utm to convert back to latlon to plot in ipyleaflet), but it was comparatively much slower (speaking about 60k+ points, which represents over 900km of tracks) when loading all raw GPX files each time:
def read_segments():
    for path in sorted(Path(".").glob("gpx/*.gpx")):
        gpx = gpxpy.parse(path.read_text())
        for track in gpx.tracks:
            for segment in track.segments:
                if len(segment.points) < 5:
                    continue
                yield [
                    (point.time_difference(segment.points[0]),
                     utm.from_latlon(point.latitude, point.longitude, 32, "U")[:2])
                    for point in segment.points
                ]
segments = list(read_segments())

UTM system is perfect for local geometry and distances, but maybe with GeoPandas the expensive conversion could be skipped.
The performance problem is also the conversion of ALL points, which could be solved by some simplifying methods of gpxpy.
Or some sort of database/cache for all converted points. And we're back at CSVs, which offer the user a stable and reviewable step in data processing.

Another option is to run gpsbabel in the Jupyter notebook with Unix commands
https://intro.syzygy.ca/unix-tricks/#unix-and-magic-in-a-notebook
(Would have to run with ‘wget’ or ‘git clone’ to get local ‘gpsbabel’ binary.)

utm can do NumPy arrays, it is just too shy about it. That's why I added it to its docs: https://github.com/Turbo87/utm/pull/50
Converting 1M points in a loop takes 90 seconds, a 1M numpy array needs 0.26s. Now it's an option again.

gpxpy and utm used to load GPX files in https://github.com/eumiro/pygohome/commit/1a30a69d8f0089625e8ef05976fc6b2b947df067
Thanks @philshem


How to use it

I replaced the png files to webp, and get error TypeError: unsupported file type
How to use it?
@isair  @priezz  @dbasedow

hi, u can try as follow:

1、npm install react-native-webp --save
2、react-native link
3、Right click your project root and select "Add Files to ..."
Select "WebP.framework"  from xxxUProject/node_modules/react-native-webp/ and click "OK"
4、Select your Target Project -> Select "Build Settings"
Add "$(SRCROOT)/../node_modules/react-native-webp" to the  Header Search Path

Hi, this is a problem related with image-size version.
React-native uses ^0.3.5, just update to the last version with:
npm install image-size

This can be related with this fix: https://github.com/image-size/image-size/commit/89649ef8ccf6ddec4b35d0f021ee1464db394362


Unable to get SSM to work

Is there some kind of configuration which is forcing SSM to be disabled on runners?  Rather than use SSH, we'd like to use Session Manager to be able to troubleshoot running workflows, but we're unable to get it to connect.
We've tried reinstalling the SSM snap, manually enabling it in the preinstall script, double-triple checking permissions, and using a custom AMI, but regardless of what we're trying, with RunsOn provisioned instances SSM will not work.
We've tried creating an EC2 instance manually using the custom IAM role for RunsOn, and that works without issue.

SSM support is on my todo list, so I’ll have a look at this shortly

In our testing, we were using Ubuntu AMI - ami-0e86e20dae9224db8 in us-east-1.

Spinup independently of runson with no changes in AWS in same VPC subnet for runs-on - session manager works.
Spinup using same AMI as a custom image in runs-on - session manager does not work.

This leads us to believe there is something about the runson agent/scripts that are causing the ssm agent/session manager to stop working.

We've tried creating an EC2 instance manually using the custom IAM role for RunsOn, and that works without issue.

@jonwest could you share the custom IAM role you've used for that?


set fontPackage for IconData.

Without fontPackage, icon fails to load on stable branch 1.17.1.
Icons from example project also displays incorrectly.

Hi @mix1009 , I'm very grateful to you for fixing this issue. Thank you


Azure CloudShell old versions of node and npm

Module: 6
Lab/Demo: 17/18/19
Task: 00
Step: 00
Description of issue
Older depricated versions of npm and node in Azure Cloudshell
Repro steps:
After the setup, the node client to produce events gives this warning.

if you check the npm and node versions of the cloudshell you see older depricated versions:

With the package.json configured
"dependencies": {    "@azure/event-hubs": "^5.11.3"  }
the client still works for now, but some day it will not anymore.
You cannot upgrade Azure Cloud Shell yourself.
To use the latest version @azure/core-amqp and @azure/event-hubs you need to have node 18 installed.

Just echoeing this issue

Thanks. I pinned the previous version of the EventHubs npm package, which resolves the issue for now We'll explore updating the latest version when the default node installation in the cloud shell catches up.

[celebrate]     Jason Richards reacted to your message:

From: Graeme Malcolm @.>
Sent: Wednesday, December 20, 2023 5:32:04 PM
To: MicrosoftLearning/dp-203-azure-data-engineer @.>
Cc: Comment @.***>
Subject: Re: [MicrosoftLearning/dp-203-azure-data-engineer] Azure CloudShell old versions of node and npm (Issue #79)
Thanks. I pinned the previous version of the EventHubs npm package, which resolves the issue for now We'll explore updating the latest version when the default node installation in the cloud shell catches up.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftLearning/dp-203-azure-data-engineer/issues/79#issuecomment-1864869821 or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4XNAQNNDXRVDR46PSLTNVDYKMOJLBFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOJIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVE2TOOBSGM3DKMZTQKSHI6LQMWSWS43TOVS2K5TBNR2WLKRSGA2DCMRXHA4TEN5HORZGSZ3HMVZKMY3SMVQXIZI.
You are receiving this email because you commented on the thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Update license to match harden-runner Action

Codecov Report

Merging #210 (c535107) into main (42123c6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #210   +/-   ##
=======================================
  Coverage   68.48%   68.48%
=======================================
  Files          14       14
  Lines        1504     1504
=======================================
  Hits         1030     1030
  Misses        367      367
  Partials      107      107

Continue to review full report at Codecov.

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


'apm rebuild' command errors

I have been trying to solve the problem I raised in Issue 279.
When I ran 'apm rebuild' I received very similar error messages concerning failure of the ' node-gyp rebuild' command.
I attach a record of my Terminal session and the npm-debug.log
apm rebuild session.txt
npm-debug.log.txt

The rebuild log complains that pkg-config cannot find libzmq. Could you post the output of pkg-config libzmq --libs?

Output of pkg-config libzmq --libs:
[robw]$ pkg-config libzmq --libs
Package libzmq was not found in the pkg-config search path.
Perhaps you should add the directory containing `libzmq.pc'
to the PKG_CONFIG_PATH environment variable
No package 'libzmq' found
[robw]$ export PKG_CONFIG_PATH=/usr/local/lib/pkgconfig
[robw]$ echo $PKG_CONFIG_PATH
/usr/local/lib/pkgconfig
[robw]$ pkg-config libzmq --libs
-L/usr/local/Cellar/zeromq/4.1.4/lib -lzmq
[robw]$

Forgot to say running apm rebuild with the PKG_CONFIG_PATH=/usr/local/lib/pkgconfig produces the same error message as posted in apm rebuild session.txt posted above

Does is work if you run instead PKG_CONFIG_PATH=/usr/local/lib/pkgconfig apm rebuild?

No, it still does not work.
Debug file attached
Regards
Rob

On 30/06/2016, at 02:37, Nicolas Riesco<EMAIL_ADDRESS>wrote:
Does is work if you run instead PKG_CONFIG_PATH=/usr/local/lib/pkgconfig apm rebuild?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/nteract/hydrogen/issues/318#issuecomment-229376019, or mute the thread https://github.com/notifications/unsubscribe/AGLYJt9F6Sqh6mVXyRT-5FPndEfuDJS_ks5qQoMWgaJpZM4I_8JB.

It seems Github dropped the email attachment.

Forgot github doesn’t like the “log” extension.
This time
Rob

On 30/06/2016, at 18:59, Nicolas Riesco<EMAIL_ADDRESS>wrote:
It seems Github dropped the email attachment.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/nteract/hydrogen/issues/318#issuecomment-229576188, or mute the thread https://github.com/notifications/unsubscribe/AGLYJnaTWTR0C-u_3lXH1JQc95WyvZCJks5qQ2lngaJpZM4I_8JB.

0 info it worked if it ends with ok
1 verbose cli [ '/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/bin/node',
1 verbose cli   '/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/bin/npm-cli.js',
1 verbose cli   '--globalconfig',
1 verbose cli   '/Users/robw/.atom/.apm/.apmrc',
1 verbose cli   '--userconfig',
1 verbose cli   '/Users/robw/.atom/.apmrc',
1 verbose cli   'rebuild',
1 verbose cli   '--target=0.36.8',
1 verbose cli   '--arch=x64' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 info readInstalled object
5 verbose rebuild path, id [ '/Users/robw/node_modules/bindings'<EMAIL_ADDRESS>]
6 verbose rebuild path, id [ '/Users/robw/node_modules/nan'<EMAIL_ADDRESS>]
7 verbose rebuild path, id [ '/Users/robw/node_modules/zmq'<EMAIL_ADDRESS>]
8 silly rebuild set [ '/Users/robw/node_modules/bindings',
8 silly rebuild set   '/Users/robw/node_modules/nan',
8 silly rebuild set   '/Users/robw/node_modules/zmq' ]
9 info build /Users/robw/node_modules/bindings
10 info preinstall<EMAIL_ADDRESS>11 info linkStuff<EMAIL_ADDRESS>12 silly linkStuff<EMAIL_ADDRESS>has /Users/robw/node_modules as its parent node_modules
13 verbose linkBins<EMAIL_ADDRESS>14 verbose linkMans<EMAIL_ADDRESS>15 verbose rebuildBundles<EMAIL_ADDRESS>16 info install<EMAIL_ADDRESS>17 info postinstall<EMAIL_ADDRESS>18 info build /Users/robw/node_modules/nan
19 info preinstall<EMAIL_ADDRESS>20 info linkStuff<EMAIL_ADDRESS>21 silly linkStuff<EMAIL_ADDRESS>has /Users/robw/node_modules as its parent node_modules
22 verbose linkBins<EMAIL_ADDRESS>23 verbose linkMans<EMAIL_ADDRESS>24 verbose rebuildBundles<EMAIL_ADDRESS>25 info install<EMAIL_ADDRESS>26 info postinstall<EMAIL_ADDRESS>27 info build /Users/robw/node_modules/zmq
28 info preinstall<EMAIL_ADDRESS>29 info linkStuff<EMAIL_ADDRESS>30 silly linkStuff<EMAIL_ADDRESS>has /Users/robw/node_modules as its parent node_modules
31 verbose linkBins<EMAIL_ADDRESS>32 verbose linkMans<EMAIL_ADDRESS>33 verbose rebuildBundles<EMAIL_ADDRESS>34 info install<EMAIL_ADDRESS>35 verbose unsafe-perm in lifecycle true
36 info<EMAIL_ADDRESS>Failed to exec install script
37 verbose stack Error<EMAIL_ADDRESS>install: node-gyp rebuild
37 verbose stack Exit status 1
37 verbose stack     at EventEmitter. (/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/lib/utils/lifecycle.js:214:16)
37 verbose stack     at EventEmitter.emit (events.js:98:17)
37 verbose stack     at ChildProcess. (/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/lib/utils/spawn.js:24:14)
37 verbose stack     at ChildProcess.emit (events.js:98:17)
37 verbose stack     at maybeClose (child_process.js:766:16)
37 verbose stack     at Process.ChildProcess._handle.onexit (child_process.js:833:5)
38 verbose pkgid<EMAIL_ADDRESS>39 verbose cwd /Users/robw
40 error Darwin 15.5.0
41 error argv "/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/bin/node" "/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/Users/robw/.atom/.apm/.apmrc" "--userconfig" "/Users/robw/.atom/.apmrc" "rebuild" "--target=0.36.8" "--arch=x64"
42 error node v0.10.40
43 error npm  v2.13.3
44 error code ELIFECYCLE
45 error<EMAIL_ADDRESS>install: node-gyp rebuild
45 error Exit status 1
46 error Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
46 error This is most likely a problem with the zmq package,
46 error not with npm itself.
46 error Tell the author that this fails on your system:
46 error     node-gyp rebuild
46 error You can get their info via:
46 error     npm owner ls zmq
46 error There is likely additional logging output above.
47 verbose exit [ 1, true ]

I usually find npm-debug.log of little help.
The error messages on the console are usually more informative.
Having said that, I've noticed something that potentially can confuse many build scripts.
Atom in your computer is installed in a path that contains spaces /Applications/ Codingware/Atom.app.
Probably, the error messages in the console could help tell whether this is the issue.

n-riesco's suggestion worked.
On MacOS 10.11.5 all I had to do was remove the space from the file path and do:
export PKG_CONFIG_PATH=/usr/local/lib/pkgconfig to provide the link to libzmq.
Hydrogen installed without any further problems.
Thank you all

@RobBW Thanks for reporting back and investigating the issue with ZMQ v2.15.1


t_done not send if navapi enabled, BW enabled, HTTPS, and no perf API

Hi, just to let you know after several testings that in case you are navigating using HTTPS, you have including navapi, and the browser you use does not super WebPerfAPI, then the performance is not measured.
If you comment out the if clause where BW monitor control checks if running https, then it works.
Does it make sense to you?
Thank and regards.

The problem, is a matter of timing...
When the page_ready event fires, all event subscribers are called in order.  By default, the order is:

BOOMR.plugins.RT.done
BOOMR.plugins.BW.run

each is supposed to call BOOMR.sendBeacon() when done.
The bug in the https flow is that the BW plugin never calls BOOMR.sendBeacon() in that block.

Ah ok, found the pull-request. Thanks!

yeah, my guess is that no one really runs it over HTTPS.  There was a similar bug someone emailed me about but regarding their own plugin.


Add HTTPS enforcement to pwndevils.com

Cloudflare no longer required,
https://help.github.com/articles/securing-your-github-pages-site-with-https/#enforcing-https-for-your-github-pages-site
Your A records will need to be updated as well for the enforcement to work
https://www.troyhunt.com/heres-why-your-static-website-needs-https/

Done! Thanks @blatayue !
https://pwndevils.com/


Move kurl util join job into api

That is a good question. I copied this over from the /metadata request. @marccampbell i assume this api is supposed to have authorization. should i add it?


Drawer doesn't work in 0.1.0

Issue
The title might be a bit ambiguous, but using version 0.1.0 I just can not get the drawer to work or appear at all. Downgrading to 0.0.3 makes all my drawers work again. Not sure whether I missed something obvious, but this can be easily reproduced:
Steps to reproduce

Create new SwiftUI project
Add swiftui-drawer as package dependency
Add import Drawer to ContentView.swift
Add Drawer of choice to a ZStack in Body

I remember running into this issue a while back and I think it was due to a SwiftUI behavior difference between the iOS 13 sdk and iOS 14. At the time, I was more concerned with delivering for iOS 13.
I saw your PR #9, and I will review it to make sure it works on both iOS versions. Thanks!

@thealpa @maustinstar This is still not working on iOS 14 Xcode 12.5

I am still running into this. Nothing appears in the preview when installing using SPM.

Unfortunately I'm running into the same problem :(

@thealpa @otymartin @tobischw this seem to be working through SPM if you just copy and past the latest commit SHA into Xcode, instead of automatically using the latest version.

@xremix, Does this work if you set the package version to "master" branch? I can release a new versioned tag from master if that works.

@maustinstar here we go! That works


Switch from gene to protein

http://uri.neuinfo.org/nif/nifstd/nlx_cell_100209 is a CUT but it is defined using the Calb2 gene instead of the protein.  We should be consistent with CUTs and use proteins unless it is explicitly stated that they do not express the protein.

Cuts now use the phenotype indicator when available and will be updated to use the indicator in cases where it is not currently available.


Dropdown disappears on workitem refresh in Microsoft Edge

Repro steps:

Create a bug.
Switch to History tab and click the refresh icon.
Switch back to Details tab and notice the multivalue control is not rendering properly. You can hover over it to see the help tool tip but cannot edit the dropdown.

Issue only reproduces in Edge, not Chrome.
Edge
Microsoft Edge 42.17134.1.0
Microsoft EdgeHTML 17.17134
Chrome
Version 70.0.3538.102 (Official Build) (64-bit)

@maxxwizard  Can you share a screenshot? @ostreifel  can you take a look

before

after

Can also confirm the bug in Firefox 64.0b9 (64-bit).
I noticed there's no listener for the the onRefreshed event. Technically this should be occurring in Chrome as well, which implies inconsistent event behavior across browsers.
    return {
        onLoaded: (args: WitExtensionContracts.IWorkItemLoadedArgs) => {
            ensureControl();
        },
        onFieldChanged: (args: WitExtensionContracts.IWorkItemFieldChangedArgs) => {
            if (control && args.changedFields[control.fieldName] !== undefined &&
                args.changedFields[control.fieldName] !== null
            ) {
                control.refresh();
            }
        },
};

I was incorrect in my assumption, it actually turned out to be the resize method. FF and Edge were reporting a scrollWidth and scrollHeight of 0 when refreshing from another tab.
private _resize = () => {
        VSS.resize(this._container.scrollWidth || 200, this._container.scrollHeight || 40);
}


Ignore any canvas move events that are outside of the canvas borders

LGTM, but I wonder if we should have a flag on the Canvas just in case there are projects out there that rely on the fact that drag events can be reported outside of the canvas bounds.

I did not generate a new companion here, or change the preferred companions in YaVersion.java
should I have done that?

I didn't generate a new companion or change the acceptable companions in YaVersion.java.
Should I have done that?

@halatmit No, that's fine. When I test I build a fresh companion as part of the testing procedure and use that.


Memory Leak on pso-db ?

Memory usage of pso-db pods in increasing at very high level.

@dsupure and @Pure-AdamuKaapan can you look into this?

OK. This is very bad. I'll use the pperf to take a look. We should hold off the release before we fix the memory leaking issue.
Thanks for helping us to find issues @mscbpi !

pso-db is cockroach so you may investigate there, I have found some content on memory management / tune to avoid this situation on their github.
I have not investigated the environment (around or within the pods). I have restarted the pods manually to free up memory and will monitor closely the evolution. If I have some time I'll look in the logs as well, but there's a lot (and I don't know cockroachDB at all).
You are very welcome for the contribution.

@mscbpi : Can you share the link about the memory management issue around cockroach, I would like to take a look. Also, just curious, did you run any test or benchmark during the period of memory leaking. Thanks again for the keen eyes to find the issues.

https://github.com/cockroachdb/cockroach/issues/47686 just googling around, I have not checked if it is somehow relevant.
No test or benchmarks running.

https://github.com/cockroachdb/cockroach/issues/13333
Old but still. It's been solved with an update. Since helm chart enable us to choose image version of cockroach is it worth trying a different version? Is there a risk?
(The issue is still there on our installation no workaround but kill pods)

Over a longer period of time. Drops are deletion of the pod.

database.resources for orchestration is in values.yaml but not in the documentation.
It will not solve the problem above (I really think it is expected with current command, commiting .25 of the memory resources for pso is not something we want though),
With database.resources set we may:

have db pods run on different nodes with enough available memory
set limits for automatic restart to avoid huge memory consumption

database:
  # resources is used specifically for the database pods
  # Change these to appropriate values for the hardware that you're running. You can see
  # the amount of allocatable resources on each of your Kubernetes nodes by running:
  #   kubectl describe nodes
  resources: {}
  # requests:
  #   cpu: "16"
  #   memory: "8Gi"

If I may the hardware recommendations for cockroach really makes me wonder if this was a good choice for pso.
Based on my previous post if limits are a way to constrains the workload, it would be nice to have some recommendations on how to set the resources limits depending on number of backends/volumes/snapshots pso manages because one cannot afford to let a system workload, which pso is, consume a such high amount of resources.

We have anti-affinity of DB pods, so they will be spread across all k8s worker nodes, ideally if there are 5 worker nodes then each one gets on db pod. but if there are only 3 worker nodes then 2 db pods will run on same worker node.
Given our metadata size is not big, assume 1 volume or one snapshot is 2k, if we have 5k volumes/snapshots, that's 10MB. We can set a default value for example 1GB as limit, and users can change if needed.
the memory keeps growing when there is no activity could be a bug of the build, we will try to see if it works on latest version.

5 worker nodes still the pods may be scheduled on the same nodes.
I am running on 19.2.10 (not the latest, but latest 19.2.x), and the behavior is the same.
There is some activity on the volumes, but no new volumes and I have like 2 dozens, not 5k...

Hi @mscbpi
Thanks for the many useful suggestions. We have confirmed that the cockroach version we are using has the memory leaking issue. I upgraded to the latest version (20.1.6) and I don't see the leaking anymore:

We'll fix it in the next release.
Currently, the DB parameters do not expose to users. We are discussing to expose the parameters to users via yaml file that users can fine-tune to have the best configuration for their platform.

@dsupure

How much memory do your nodes have ?
=> I'm seeing the same behavior as yours, just as a higher level, and this may not be a leak but expected cockroach behavior as designed. Please have a look at this article in official documentation

Suspected memory leak
A CockroachDB node will grow to consume all of the memory allocated for its cache

A PSO user can actually already manage database memory usage
=> cockroach will adapts its memory allocation to memory limits in k8s, that can already be set with database.resources.memory.limits
Please have a look at this other article in official documentation

Resource limits
memory limits in particular limit the amount of memory that the container knows is available to it, which help when you specify percentages for the CockroachDB --cache and --max-sql-memory flags, as our default configuration file does.

without limits:
Oct 3, 2020 @ 19:37:37.265 | cockroachdb | I201003 17:37:37.265747 1 server/config.go:394  system total memory: 7.8 GiB
with limits set database.resources.memory.limits set to 2 GiB:
Oct 3, 2020 @ 19:56:07.913 | cockroachdb | I201003 17:56:07.913504 1 server/config.go:394  system total memory: 2.0 GiB

Relaunch from scratch with default version 19.2.3 and 512Mi database.resources.memory.limits
image
It runs above 512Mi but does not get OOMKilled and is stable.

@dsupure
I am not getting the same result as yours. 20.1.6 still have its memory growing as it is expected with cockroach according to their documentation.

The only setup that makes it not too greedy is to set database.resources.memory.limits which cockroach handles well since it adjusts the memory usage based on the limit (and just OOMKilled that we would normally expect). (see above, logs, and cockroach documentation).
It does not matter what version I use.
The issue can be closed but best would be to match our conclusions, IMHO:

It's not a leak but normal (high) usage of memory by cockroachDB
PSO Helm charts already enable the user to manage the database memory limits
Cockroach handles k8s memory limits in a smart way,
I verified that system memory was correctly detected by cockroach according to the limit (logs and results)

Hi @mscbpi :
Yeah. I still saw memory growth w/o memory limit on 20.1.6. It was not as crazy as 19.2.3 but it's growing for sure.

I go through the cockroach db documents your provided and agree with you it's maybe not necessary the memory leak.
I think database.resources.memory.limits is good enough to rein the memory usage.

Hi @mscbpi :
I am going to update our doc and close this issue. Thank you for bringing it to our attention.

Have a PR #100 to update the document to bring attention to customers how to rein the db pod memory.


[Revise subclass] DNA repair disorders

Mondo term (ID and Label)
follow on from
https://github.com/monarch-initiative/mondo/issues/3193
Suggested revision and reasons
also these :
ataxia and polyneuropathy, adult-onset (MONDO:0010781)
PCNA-related progressive neurodegenerative photosensitivy syndrome (MONDO:0014399
spinocerebellar ataxia, autosomal recessive, with axonal neuropathy 1 (MONDO:0011801)
spinocerebellar ataxia, autosomal recessive, with axonal neuropathy 2 (MONDO:0018996)
I noticed this because  I know that a lot of DNA repair disorders cause defects with ~neurogenesis~ neurodegenerative diseases, but my overlap was very small.
I checked all the DNA repair disorders annotated at PomBase. There may be others... (although I think the majority DNA repair genes are conserved)

These should all be also
neurodegenerative diseases

Got it, thanks @ValWood
@ValWood This class is a descendant of neurodegenerative disease:

'ataxia-telangiectasia-like disorder 2' (MONDO:0014399)

@ValWood My understanding is polyneuropathy is a disease of the peripheral nervous system (it is classified in Mondo as such), and neurodegenerative disease is classified as a child of 'central nervous system disease' in Mondo.
I wonder if we should reclassify 'neurodegenerative disease' as a child of 'nervous system disorder'? Can there be peripheral neurodegenerative diseases? Our Mondo term has a synonym central nervous system degenerative disorder (which xrefs NCIT here).

[ ] I think we need a new grouping class:
'neurodegenerative disease'

[ ] we should rename 'neurodegenerative disease' to 'central nervous system neurodegenerative disease', and it should be a subclass of 'neurodegenerative disease.

[ ] add the parent 'neurodegenerative disease' to 'ataxia and polyneuropathy, adult-onset' (MONDO:0010781)

[ ] MONDO_0100308 'atatic disorder' should be a child of 'neurodegenerative disease' (see article, this says "Ataxia is a degenerative disease of the nervous system." Then this would make these terms children of 'neurodegenerative disease':

spinocerebellar ataxia, autosomal recessive, with axonal neuropathy 1 (MONDO:0011801)

spinocerebellar ataxia, autosomal recessive, with axonal neuropathy 2 (MONDO:0018996)

@pnrobinson does this sound okay to you?

Hello @ValWood
I recently joined the Mondo team.
We will address your issue as soon as possible. Thank you for your patience.
If this is a high priority, please let me know and we can prioritize it. Thank you!

@pnrobinson when you have a chance, could you look at the questions above, thank you! ⬆ 👀

@pnrobinson and @ahamosh could you look at the comment here and let me know if you agree with the proposed action items:
https://github.com/monarch-initiative/mondo/issues/3195#issuecomment-870061610

It is not true that neurodegenerative disease is always CNS (https://pubmed.ncbi.nlm.nih.gov/31008845/):
Peripheral neuropathy is the most common neurodegenerative disease affecting hundreds of millions of patients worldwide and is an important cause of chronic pain.

I think this is complicated enough that we should discuss it via zoom rather than in GitHub issues!

I agree this needs an intensive discussion in vivo or in vitro.

in vitro (zoom), not Github.

per https://github.com/monarch-initiative/mondo/issues/5666, Ada said 'ataxia and polyneuropathy, adult-onset' should not be a child of DNA repair disease

@sabrinatoro reassigning this to you


⚠ Bofrost.de has degraded performance

In 3f0faae, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1624 ms

Resolved: Bofrost.de performance has improved in 34aef9a.


Team Assessment

### LOOK AT LAST COMMENT

Update from Prof:
Good morning @channel!
Hope that you're doing well! It was great to attend your Sprint 4 presentations these past few days (few more today!), I was very impressed by some of your projects! Keep up the good work for this last Sprint.
This last Sprint is special because it marks the end of the course and includes an assessment of yourself, your teammates, and your team. I uploaded the form called Peer and Team Evaluation Form.docx in the Google Drive. Please fill in and return this form to your PO/TA before this coming Sun. Apr. 24th. (Please note that Section 2 is optional if you have nothing particular (positive or negative) to report about your teammates.)
I'd like to have a virtual (or hybrid) general meeting with all of you before the end of the course. The aims of this meeting would be:
To have some teams showcase their projects. The TAs and I would invite the best team(s) to present their project as well as teams that took interesting directions (for example, that used a unique programming language)
To have some general retrospective on your projects and the course, discuss what went particularly well (or not!), and receive your feedback and suggestions.
To thank you all and your TAs for the great work done and celebrate the end of the course :slightly_smiling_face:
So, please, fill in this Doodle as soon as possible, preferably before/on Wed. Apr. 13th. (Please be generous and choose as many slots as possible!) (Please also note that not all teammates of each team must attend, if it happens that some have exams, etc. at the time chosen, then it's okay if they don't join.)
Have a great week!

Everyone please fill out your personal assessments I each attached beside your names!
@SamanthaGuillemette Fill this out!

@QuangTran304 Fill this out!
@MarwaKhalid Fill this out!
@smarkandu Fill this out!

Submitted to TA