[Bug]: package_info_plus is not reading package info in Linux (using flutter-pi)

Platform
Linux (using flutter-pi)
Plugin
package_info_plus
Version
1.4.2
Flutter SDK
3.0.1
Steps to reproduce
While package_info_plus works perfectly on the emulator, it does not show anything in Linux (using flutter-pi) although version 1.4.2 claims that it supports Linux
Steps to reproduce:

flutter build bundle
scp -r ./build/flutter_assets/ pi@IP:~

In "development mode":
$flutter-pi flutter_assets/
and it shows version: + (so neither version nor buildNumber is read).
In "release mode"
$flutter-pi --release flutter_assets/
it throws:
[ERROR:flutter/lib/ui/ui_dart_state.cc(198)] Unhandled Exception: MissingPluginException(No implementation found for method getAll on channel dev.fluttercommunity.plus/package_info)
#0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:165)
<asynchronous suspension>
#1      MethodChannel.invokeMapMethod (package:flutter/src/services/platform_channel.dart:367)
<asynchronous suspension>
#2      MethodChannelPackageInfo.getAll (package:package_info_plus_platform_interface/method_channel_package_info.dart:13)

Code Sample
void main() async {
  WidgetsBinding bindings = WidgetsFlutterBinding.ensureInitialized();

  final PackageInfo packageInfo = await PackageInfo.fromPlatform();
  print("version: ${packageInfo.version}+${packageInfo.buildNumber}");
  runApp(MyApp());
}

### Logs

```shell
[locales] Warning: The system has no configured locale. The default "C" locale may or may not be supported by the app.
[flutter-pi] WARNING: display didn't provide valid physical dimensions. The device-pixel ratio will default to 1.0, which may not be the fitting device-pixel ratio for your display.
==============Locale==============
Flutter locale:
  default: C
  locales: C
===================================
===================================
display mode:
  resolution: 1920 x 1080
  refresh rate: 60Hz
  physical size: 0mm x 0mm
  flutter device pixel ratio: 1.000000
===================================
error: XDG_RUNTIME_DIR not set in the environment.
EGL information:
  version: 1.4
  vendor: "Mesa Project"
  client extensions: "EGL_EXT_device_base EGL_EXT_device_enumeration EGL_EXT_device_query EGL_EXT_platform_base EGL_KHR_client_get_all_proc_addresses EGL_EXT_client_extensions EGL_KHR_debug EGL_EXT_platform_device EGL_EXT_platform_wayland EGL_KHR_platform_wayland EGL_EXT_platform_x11 EGL_KHR_platform_x11 EGL_MESA_platform_gbm EGL_KHR_platform_gbm EGL_MESA_platform_surfaceless"
  display extensions: "EGL_ANDROID_blob_cache EGL_EXT_buffer_age EGL_EXT_image_dma_buf_import EGL_EXT_image_dma_buf_import_modifiers EGL_KHR_cl_event2 EGL_KHR_config_attribs EGL_KHR_create_context EGL_KHR_create_context_no_error EGL_KHR_fence_sync EGL_KHR_get_all_proc_addresses EGL_KHR_gl_colorspace EGL_KHR_gl_renderbuffer_image EGL_KHR_gl_texture_2D_image EGL_KHR_gl_texture_3D_image EGL_KHR_gl_texture_cubemap_image EGL_KHR_image EGL_KHR_image_base EGL_KHR_image_pixmap EGL_KHR_no_config_context EGL_KHR_reusable_sync EGL_KHR_surfaceless_context EGL_EXT_pixel_format_float EGL_KHR_wait_sync EGL_MESA_configless_context EGL_MESA_drm_image EGL_MESA_image_dma_buf_export EGL_MESA_query_driver EGL_WL_bind_wayland_display "
===================================
OpenGL ES information:
  version: "OpenGL ES 3.1 Mesa 20.3.5"
  shading language version: "OpenGL ES GLSL ES 3.10"
  vendor: "Broadcom"
  renderer: "V3D 4.2"
  extensions: "GL_EXT_blend_minmax GL_EXT_multi_draw_arrays GL_EXT_texture_filter_anisotropic GL_EXT_texture_compression_s3tc GL_EXT_texture_compression_dxt1 GL_EXT_texture_format_BGRA8888 GL_OES_compressed_ETC1_RGB8_texture GL_OES_depth24 GL_OES_element_index_uint GL_OES_fbo_render_mipmap GL_OES_mapbuffer GL_OES_rgb8_rgba8 GL_OES_standard_derivatives GL_OES_stencil8 GL_OES_texture_3D GL_OES_texture_float GL_OES_texture_half_float GL_OES_texture_half_float_linear GL_OES_texture_npot GL_OES_vertex_half_float GL_EXT_draw_instanced GL_EXT_texture_sRGB_decode GL_OES_EGL_image GL_OES_depth_texture GL_OES_packed_depth_stencil GL_EXT_texture_type_2_10_10_10_REV GL_OES_get_program_binary GL_APPLE_texture_max_level GL_EXT_discard_framebuffer GL_EXT_read_format_bgra GL_EXT_frag_depth GL_NV_fbo_color_attachments GL_OES_EGL_image_external GL_OES_EGL_sync GL_OES_vertex_array_object GL_ANGLE_pack_reverse_row_order GL_ANGLE_texture_compression_dxt3 GL_ANGLE_texture_compression_dxt5 GL_EXT_occlusion_query_boolean GL_EXT_texture_rg GL_EXT_unpack_subimage GL_NV_draw_buffers GL_NV_read_buffer GL_NV_read_depth GL_NV_read_depth_stencil GL_NV_read_stencil GL_EXT_draw_buffers GL_EXT_map_buffer_range GL_KHR_debug GL_KHR_texture_compression_astc_ldr GL_NV_pixel_buffer_object GL_OES_depth_texture_cube_map GL_OES_required_internalformat GL_OES_surfaceless_context GL_EXT_color_buffer_float GL_EXT_sRGB_write_control GL_EXT_separate_shader_objects GL_EXT_shader_implicit_conversions GL_EXT_shader_integer_mix GL_EXT_base_instance GL_EXT_compressed_ETC1_RGB8_sub_texture GL_EXT_draw_elements_base_vertex GL_EXT_primitive_bounding_box GL_EXT_shader_io_blocks GL_EXT_texture_border_clamp GL_EXT_texture_view GL_KHR_context_flush_control GL_NV_image_formats GL_OES_draw_elements_base_vertex GL_OES_primitive_bounding_box GL_OES_shader_io_blocks GL_OES_texture_border_clamp GL_OES_texture_stencil8 GL_OES_texture_storage_multisample_2d_array GL_OES_texture_view GL_EXT_buffer_storage GL_EXT_float_blend GL_EXT_geometry_point_size GL_EXT_geometry_shader GL_KHR_no_error GL_KHR_texture_compression_astc_sliced_3d GL_OES_EGL_image_external_essl3 GL_OES_geometry_point_size GL_OES_geometry_shader GL_OES_shader_image_atomic GL_EXT_texture_compression_s3tc_srgb GL_MESA_shader_integer_functions GL_KHR_parallel_shader_compile GL_EXT_EGL_image_storage GL_MESA_framebuffer_flip_y GL_EXT_texture_query_lod "
===================================
[flutter-pi] Warning: Could not load libflutter_engine.so from the asset bundle. dlopen: /home/user/flutter_assets/libflutter_engine.so: cannot open shared object file: No such file or directory. Trying to open libflutter_engine.so.release...
[ERROR:flutter/lib/ui/ui_dart_state.cc(198)] Unhandled Exception: MissingPluginException(No implementation found for method getAll on channel dev.fluttercommunity.plus/package_info)
#0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:165)
<asynchronous suspension>
#1      MethodChannel.invokeMapMethod (package:flutter/src/services/platform_channel.dart:367)
<asynchronous suspension>
#2      MethodChannelPackageInfo.getAll (package:package_info_plus_platform_interface/method_channel_package_info.dart:13)

### Flutter Doctor

```shell
$ flutter doctor -v
[√] Flutter (Channel stable, 3.0.1, on Microsoft Windows [Version 10.0.19044.1865], locale pl-PL)
    • Flutter version 3.0.1 at C:\Users\Kamil\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fb57da5f94 (3 months ago), 2022-05-19 15:50:29 -0700
    • Engine revision caaafc5604
    • Dart version 2.17.1
    • DevTools version 2.12.2

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\User\AppData\Local\Android\sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[X] Visual Studio - develop for Windows
    X Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] VS Code (version 1.69.2)
    • VS Code at C:\Users\User\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.44.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19044.1865]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 103.0.5060.134
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 103.0.1264.49

[√] HTTP Host Availability
    • All required HTTP hosts are available

It looks like it's related to this method: https://github.com/fluttercommunity/plus_plugins/blob/main/packages/package_info_plus/package_info_plus_platform_interface/lib/method_channel_package_info.dart
Two questions:

why it throws exception instead of returning empty string?
can you support "flutter-pi" platform?

Issue is still there

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 15 days

Issue is still there

Hi @eximius313, the problem here is that you are using flutter-pi which ships a non-standard embedder. This package does works in linux, but only supports the official flutter embedder. Supporting third party embedders like flutter-pi also require access to that hardware in order to test etc.
My advice would be to encourage you to contribute to solve this issue. I don't know if the solution lies in the flutter-pi embedder communicating its platform in a way that this package recognizes as linux, or changing the inner workings of this package to also recognize flutter-pi, without the actual hardware is hard to test.

PS: I'm not a mantainer of this package but I have contributed in the past.

@fabiancrx, could you please provide me with any tips on how I could start?
Especially how can I check if flutter-pi embedder communicates itself as Linux and/or how to enable this package to recognize flutter-pi embedder?

issue still exists

I can confirm this is an issue when using the Yocto project + flutter-wayland-client, as well, but it doesn't appear to be due to the use of a custom embedder (at least in my implementation).
In my implementation, I am have the following function to to get the package info, which explicitly registers the plugin for Linux using PackageInfoPlusLinuxPlugin.registerWith();:
  /// Get the package information.
  Future<PackageInfo> get packageInfo async {
    if (Platform.isLinux) {
      // Register the Linux plugin.
      //
      // Note: this is required to get the package info on embedded Linux. If this
      // is not explicitly called, the package info will attempt to go through
      // the default MethodChannel, which will fail.
      PackageInfoPlusLinuxPlugin.registerWith();
    }

    // Note: there is no need to hold onto the package info, as it is
    // cached by the package_info_plus plugin.
    final PackageInfo info = await PackageInfo.fromPlatform();
    return info;
  }

However, this causes another issue. The Flutter app is being built AOT (flutter build bundle), which does not appear to generate the version.json that the PackageInfoPlusLinuxPlugin is looking for.
I've tested my app locally on Linux, as well. When using the flutter build linux command, I get the following build/flutter_assets contents:
build/flutter_assets
├── AssetManifest.bin
├── AssetManifest.json
├── FontManifest.json
├── fonts
│   └── MaterialIcons-Regular.otf
├── NOTICES.Z
├── shaders
│   └── ink_sparkle.frag
└── version.json

However, when using the flutter build bundle command, I only get the following contents:
build/flutter_assets
├── AssetManifest.bin
├── AssetManifest.json
├── FontManifest.json
├── fonts
│   └── MaterialIcons-Regular.otf
├── isolate_snapshot_data
├── kernel_blob.bin
├── NOTICES.Z
├── shaders
│   └── ink_sparkle.frag
└── vm_snapshot_data

TLDL; should there be another flag or argument specified when using flutter build bundle to generate the version.json file PackageInfoPlusLinuxPlugin is dependent on?

@lapumb-spindance, thank you for this analyis! Quick question - when you use flutter build linux (so that version.json is present in the bundle) does PackageInfoPlusLinuxPlugin work out-of-the-box, or you had to modify its code?

Quick question - when you use flutter build linux (so that version.json is present in the bundle) does PackageInfoPlusLinuxPlugin work out-of-the-box, or you had to modify its code?

@eximius313 Yes, it works out of the box when testing locally on Ubuntu.

I didn't know what bundle command was for, the help says:
bundle      Build the Flutter assets directory from your app.

I am not sure what is the purpose of that command, the Flutter docs don't seem to cover it?
But anyway, seems unrelated to the issue reported by the original post.


Mesh api update

Based on review comments - added support for stream filtering and for non-streaming queries

I was bad and based this off the smesher pr - so closing and doing a new one


Handle push event

Currently Digger only handles pull_request and comment. But if user uses force push to a branch, then it's the push event which we currently don't handle
User suggestion:

We use ammend and force push very regularly. This would be very useful for us to hade a push trigger

Also when using Renovate Bot force-pushing is the norm


Fix warnings in tools for MinGW

Also allows texassemble and texdiag to be built without requiring DX11 support.

The same changes should be made to meshconvert in DirectXMesh and uvatlastool in UVAtlas if you have the time.

I think that meshconvert from DirectxMesh and uvatlastool from UVAtlas are already done.

Ah yes. You did them with your other PRs. Thanks!

/azp run DirectXTex-GitHub-CMake,DirectXTex-GitHub-CMake-Dev17,DirectXTex-GitHub-MinGW,DirectXTex-GitHub-CMake-Xbox,DirectXTex-GitHub-CMake-Xbox-Dev17,DirectXTex-GitHub-WSL,DirectXTex-GitHub-WSL-11

I did do UVAtlasTool in https://github.com/microsoft/UVAtlas/pull/132 and I also did Meshconvert in https://github.com/microsoft/DirectXMesh/pull/117 .


RHIROS-655 update prometheus metrics with org_id

@r14chandra Shall we add an unittest for this one, feel free to update the ones Kavita has previously added.

@r14chandra Shall we add an unittest for this one, feel free to update the ones Kavita has previously added.

@saltgen - Could you create separate ticket for it. Currently there is no any test to check these custom metrics.
We can handle it separately.
Merging this change to verify current metrics and Grafana dashboard.


Restrict accessibility of PositiveInt's constructor.

I had been thinking that this PR was fully complete, and that it was simply waiting to be merged. But I was wrong - my mistake for not following through.
But, then I saw these comments https://github.com/non/spire/commit/65f0d4192ab947ae3600bcf3f82798afe65ded57#commitcomment-13447188 and https://github.com/non/spire/pull/520#issuecomment-143431888 to add the private [spire] modifier.
As a result of this change, my understanding is that clients/users  of non/spire won't be able to use new PositiveInt due to the private [spire] modifier - which is an improvement. But, any com.spire class will be able to bypass the safe constructor, PositiveInt#apply, as I understand.
However, is it possible and worthwhile to add a warning when invoking new PositiveInt from outside of the PostiveInt.scala class? In other words, suppress warnings when calling it from the macro, but warn otherwise.

Could either of you take a look please, @non or @tixxit?
thanks

But, any com.spire class will be able to bypass the safe constructor, PositiveInt#apply, as I understand.

I'd be wary of making Spire too safe, at the expense of performance/flexibility. Here the straightjacket is quite uncontroversial (what's the cost of a > 0 check?), and I'm not raising any concerns.
Without zero-cost abstractions ala Rust and full dependent types, numerical code like Spire is bound to leave unsafe methods simply for performance reasons.

I'd be wary of making Spire too safe, at the expense of performance. Here the straightjacket is quite uncontroversial (what's the cost of a > 0 check?), and I'm not raising any concerns.

If I understand correctly, Denis, you're noting that But, any com.spire class will be able to bypass the safe constructor ... is OK due to performance reasons?
I'm just making sure that I understood what you said, i.e. whether your comment indicate that a code change should occur in this PR.

If I understand correctly, Denis, you're noting that But, any com.spire class will be able to bypass the safe constructor ... is OK due to performance reasons?

OK of course.
I was just pointing that even in Haskell, numerical code has impure aspects like throwing runtime errors (see hmatrix).


CodeQL-Build fails.

CodeQL-Build fails due to out of space.
Ref #13671
 Oops! A fatal internal error occurred.
  com.semmle.util.exception.CatastrophicError: An error occurred during the evaluation of Operand::NonPhiMemoryOperand::hasDefinition#3#fff/3@b0e47a
  The RA to evaluate was:
      {1} r1 = CONSTANT(unique int)[1]
      {2} r2 = JOIN r1 WITH Operand::NonPhiMemoryOperand::hasDefinition#3#fff#join_rhs AS R ON FIRST 1 OUTPUT R.<1>, R.<2>
      {3} r3 = JOIN r2 WITH Operand::NonPhiMemoryOperand#3#fff_120#join_rhs AS R ON FIRST 2 OUTPUT r2.<0>, r2.<1>, R.<2>
      {3} r4 = JOIN r3 WITH SSAConstruction::Cached::getMemoryOperandDefinition#2#ffff@staged_ext AS R ON FIRST 2 OUTPUT r3.<2>, R.<3>, R.<2>
      return r4

  (eventual cause: IOException "No space left on device"

It has been working for recent PRs. It could be that /retest on your PR did not rebuild codeQL. You can try adding an empty commit to your PR to kick off the codeQL rebuild.

Please re-open if this is still a problem.

Hi @yanavlasov, still have the same problem. Do you have any idea about 'retest-circle' that isn't working for all Github user or deprecated CONTRIBUTING.md?)

@mk46 feel free to PR to remove the circle reference, its no longer used/available

@phlax Done #13824.


update schema with additional fields

I'm not sure if these types are correct @shagamemnon

Hey @andywimmer not sure if you are asking me to check and see if the types are correct? Or, if you are saying that data types indicated by me on a previous commit may be incorrect. Please clarify!!

Apologies @shagamemnon this is meant to fix #2 however I'm uncertain if the field types I chose for the new fields are correct.


[ISSUE][CLOSED] Cannot send email from aaci.mooo.com to<EMAIL_ADDRESS>

 Issue by geunsik-lim
Tuesday Jun 26, 2018 at 08:09 GMT
Originally opened as https://github.sec.samsung.net/STAR/TAOS-CI/issues/61

It's strange. We cannot send email from TAOS-CI server (aaci.mooo.com, Ubuntu 16.04) to<EMAIL_ADDRESS>
Sending email with google smtp:

$ mailx -v -s "i am testing email"<EMAIL_ADDRESS>let's send a my message.
.
Resolving host "smtp.gmail.com:587" ... done
Connecting to "<IP_ADDRESS>:587" ...
Connecting to "<IP_ADDRESS>:587" ...
Connecting to "2404:6800:4008:c04::6d:587" ...Could not connect: Network is unreachable
"/home/invain/dead.letter" 8/226
... message not sent

Sending email with local smtp (Postfix)

Connecting to "2404:6800:4008:c04::6d:587" ...Could not connect: Network is unreachable
"/home/invain/dead.letter" 9/227
... message not sent

 Comment by git-bot-sec
Tuesday Jun 26, 2018 at 08:09 GMT

:octocat: cibot: Thank you for posting issue #61. The person in charge will reply soon.

:octocat: cibot: Thank you for posting issue #85. The person in charge will reply soon.

 Comment by geunsik-lim
Tuesday Jun 26, 2018 at 08:39 GMT

RFC 2821 for SMTP

https://tools.ietf.org/html/rfc2821

Editing original configuration file (e.g., main.cf)

$ sudo vi /etc/postfix/main.cf
Example file for aaci.mooo.com:
main-aaci.mooo.com.20180623.zip

How to send email to<EMAIL_ADDRESS>in ubuntu 16.04

sudo apt install postfix sasl2-bin
sudo apt install heirloom-mailx
sudo newaliases
sudo systemctl restart postfix
echo -e  " Hi,\n\n It is aaci.mooo.com server.\n It's just test email.\n We follow-up RFC2821 (https://tools.ietf.org/html/rfc2821).\n\n\n $(date).\n from aaci.mooo.com.\n" | \
mailx -v -s "[$(date)] Sending email with local smtp"   \
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

 Comment by geunsik-lim
Tuesday Jun 26, 2018 at 08:41 GMT

$ sudo vi /etc/postfix/main.cf

Example file for aaci.mooo.com:
main-aaci.mooo.com.20180623.zip


SyntaxError: Unexpected end of JSON input (WeatherUnderground)

[7/28/2020, 4:07:47 PM] [WeatherPlus] Error retrieving weather report and forecast
[7/28/2020, 4:07:47 PM] [WeatherPlus] Error Message: SyntaxError: Unexpected end of JSON input
[7/28/2020, 4:11:48 PM] [WeatherPlus] Error retrieving weather report and forecast
[7/28/2020, 4:11:48 PM] [WeatherPlus] Error Message: SyntaxError: Unexpected end of JSON input
[7/28/2020, 4:15:48 PM] [WeatherPlus] Error retrieving weather report and forecast
[7/28/2020, 4:15:48 PM] [WeatherPlus] Error Message: SyntaxError: Unexpected end of JSON input
[7/28/2020, 4:19:48 PM] [WeatherPlus] Error retrieving weather report and forecast
[7/28/2020, 4:19:48 PM] [WeatherPlus] Error Message: SyntaxError: Unexpected end of JSON input
[7/28/2020, 4:23:48 PM] [WeatherPlus] Error retrieving weather report and forecast
[7/28/2020, 4:23:48 PM] [WeatherPlus] Error Message: SyntaxError: Unexpected end of JSON input
[7/28/2020, 4:27:48 PM] [WeatherPlus] Error retrieving weather report and forecast
[7/28/2020, 4:27:48 PM] [WeatherPlus] Error Message: SyntaxError: Unexpected end of JSON input
[7/28/2020, 4:31:48 PM] [WeatherPlus] Error retrieving weather report and forecast
[7/28/2020, 4:31:48 PM] [WeatherPlus] Error Message: SyntaxError: Unexpected end of JSON input
[7/28/2020, 4:35:48 PM] [WeatherPlus] Error retrieving weather report and forecast
[7/28/2020, 4:35:48 PM] [WeatherPlus] Error Message: SyntaxError: Unexpected end of JSON input
[7/28/2020, 4:39:48 PM] [WeatherPlus] Error retrieving weather report and forecast
[7/28/2020, 4:39:48 PM] [WeatherPlus] Error Message: SyntaxError: Unexpected end of JSON input

Having same problem here from today... updated to NodeJS LTS 14.15.1 today...

Me too:

[18/09/2021, 17:23:43] [WeatherPlus] Error retrieving weather report and forecast
[18/09/2021, 17:23:43] [WeatherPlus] Error Message: SyntaxError: Unexpected end of JSON input

Config:
{             "units": "si",             "interval": 15,             "stations": [                 {                     "nameNow": "Weather",                     "service": "weatherunderground",                     "key": "<key>",                     "stationId": "<id>",                     "language": "en",                     "compatibility": "both",                     "conditionCategory": "simple",                     "now": true,                     "extraHumidity": false,                     "hidden": [                         "Cloud Cover",                         "Dew Point",                         "Ozone",                         "Rain 1h",                         "Solar Radiation",                         "Temperature Min",                         "UV Index",                         "Visibility"                     ]                 }             ],             "platform": "WeatherPlus"         },

Dup of #246
Check to make sure that your PWS is currently online, otherwise you won't get data.


adding support for GPT-J

Adding LinearGELU kernel. This first commit is a small change - this kernel will be used in GPT-J.

Could you add a test file in tests to verify the correctness? Thank you!

The GELU unit is working (passes the test), but the GPT-J is failing. Any help on what's going wrong there?

Hi Iman, sorry I don't have the bandwidth recently to take a detailed look at your implementation. I can provide some common pitfalls and advice for implementation.

First, be careful with the ordering of matrices, especially the bmm functions. The CUTLASS uses column-major matrices, and the t in the function name means transposed column-major matrices, which are row-major matrices.
Second, you should be aware of the contiguity of tensor elements. The CUTLASS kernels only receive a pointer from the PyTorch tensor, so if you don't call contiguous() after some operations like transpose() and permute(), the content of the tensor will remain the same.
You can cache each module's input and output activations to compare the ground truth with your modules' outputs. Then you will be able to spot the bug and fix it :).

what am I supposed to do here? How am I supposed to generate this (for my gptj case) -- for this test?

You can refer to this script to generate the GPT-J INT8 model.

Next step is GPTJModel which is basically a stack of GPTJBlocks -> and this one has high R2, so something is going wrong there.

I am not sure why this happens. According to the difference in the activation value of the cache, from which layer is there a significant error? You can continue use this method to locate the bug.

It seems to be an issue with how the model is reconstructed, to try it out, I started with OPT:
first, I generate act_scales and then export the model (OPT-350M):
python3 examples/generate_act_scales.py --model-name facebook/opt-350m --output-path act_scales/opt-350m.pt
python3 examples/export_int8_model.py --model-name facebook/opt-350m --act-scales act_scales/opt-350m.pt --output-path i8m/opt-350m-sq

Then I ran 'test_opt':
python3 tests/test_opt.py

Once the original (which gives an accuracy of ~64% as expected) and then with "int8" (with what I got from step 1: 'i8m/opt-350m-sq')-> this one showed an accuracy of 0!
[I did not change the dataset or anything else]

Currently, we haven't supported the OPT-350M model since it uses post-layernorm. Please try with a 125m or 1.7b model.

Hi Iman, could you check the box Allow edits from maintainers so I can work on this PR?

Sure, but can you not look into it until next week? I have found out all the issues and I can now recreate results for GPT-J with high accuracy, I will clean up the changes and commit by the end of next week, is that ok?

Great! If you have any further questions or need assistance, feel free to ask. 😄

To generate that "model_dec_scales.json" I used a tweaked version of smoothquant: https://gist.github.com/ImanHosseini/ad922cc9c01e05bc16c59926b6f35fd9
Which generates those based on the codegen models.
Thoughts?

@Guangxuan-Xiao Did you have the chance to look into this PR?


Show error messages

I would like to display familiar error messages, instead of generic server messages, such as error 500. A simple typing error, for example, when searching for a user, already returns an error 500.
This is possible?
Could someone help me, please?

Is your issue solved? Because all of sudden, it is throwing 200 error for me.


warning: operation on 'count' may be undefined [-Wsequence-point]

In file included from /usr/lib/perl5/5.24.1/ppc-linux-thread-multi-64int/CORE/perl.h:5615:0,
from XS.xs:34:
XS.xs: In function 'XS_List__MoreUtils__XS_mode':
XS.xs:2078:20: warning: operation on 'count' may be undefined [-Wsequence-point]
EXTEND(SP, ++count);
/usr/lib/perl5/5.24.1/ppc-linux-thread-multi-64int/CORE/embed.h:574:53: note: in definition of macro 'stack_grow'
#define stack_grow(a,b,c) Perl_stack_grow(aTHX_ a,b,c)
^
/usr/lib/perl5/5.24.1/ppc-linux-thread-multi-64int/CORE/pp.h:372:49: note: in expansion of macro '_EXTEND_SAFE_N'
sp = stack_grow(sp,p,_EXTEND_SAFE_N(n));
^~~~~~~~~~~~~~
XS.xs:2078:9: note: in expansion of macro 'EXTEND'
EXTEND(SP, ++count);
^~~~~~

From my point of view, https://github.com/perl5-utils/List-MoreUtils-XS/blob/master/XS.xs#L2078 initializes count. And https://github.com/perl5-utils/List-MoreUtils-XS/blob/master/XS.xs#L450-L475 does not "deinitialize" it.
Can you please be more clear where count is used uninitialized?

What uninitialized?

You can try following patch:
diff --git a/XS.xs b/XS.xs
index 063aa27..e482d87 100644
--- a/XS.xs
+++ b/XS.xs
@@ -2114,7 +2114,8 @@ CODE:
 if(seen_undef == max)
 {

   EXTEND(SP, ++count);

  ++count;

   EXTEND(SP, count);
   ST(count-1) = &PL_sv_undef;

}

Aaaand? ;)

https://build.opensuse.org/package/show/home:Andreas_Schwab:Factory/perl-List-MoreUtils-XS

0.425_001 is on it's way to CPAN. Try it, send test reports and sooner or later a release for the masses will be out.


how to get the NDG Linux Unhatched Certificate?

i have completed the NDG Linux Unhatched Course from Cisco but i don't know where to give the exam for getting the certification, please guide me.

below content, there is option for exam give then add your feedbacks to get certificates


Unit testing

Summary
Add the ability to easily tests accord of our projects based on gridsome. If we are able to create such tests maybe it's worth to describe it in docs? As I can see in this repo you are actually testing some of the example projects.
I know the difference between unit and e2e tests. I'm most interested in unit tests for now.
Basic example
The base example can be found here and some docs help in gatsby here
Motivation
I just want to test my app based on gridsome and don't know how to start b/c some base guide from vue-test-utils docs and my experience with the test is not enough to create such tests.

I experimented with: 'vue add @vue/unit-jest', which seemed to work, however I note the example test was set to the starter kit of Vue CLI vs the Gridsome starter kit.

@nick-wireless I just tried this and got the following error:
 ERROR  TypeError: Cannot read property 'version' of undefined
TypeError: Cannot read property 'version' of undefined
    at add (/usr/local/lib/node_modules/@vue/cli/lib/add.js:23:35)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Here is how I made it work with Jest:
First install modules (some of these are already installed with Gridsome) :
npm install --save-dev jest vue-jest babel-core babel-jest @vue/test-utils

Create a jest.config.js in the root directory:
/**
 * @see https://github.com/vuejs/vue-cli/tree/dev/packages/%40vue/cli-plugin-unit-jest/presets/default
 */
module.exports = {
    moduleFileExtensions: [
        'js',
        'jsx',
        'json',
        'vue',
    ],
    transform: {
        '^.+\\.vue$': require.resolve('vue-jest'),
        '^.+\\.jsx?$': require.resolve('babel-jest'),
    },
    transformIgnorePatterns: ['/node_modules/'],
    moduleNameMapper: {
        '^~/(.*)$': '<rootDir>/src/$1',
    },
    testMatch: [
        '**/tests/unit/**/*.spec.[jt]s?(x)',
        '**/__tests__/*.[jt]s?(x)',
    ],
}

Also create a babel.config.js file in the root directory:
module.exports = {
    presets: ['@babel/preset-env'],
}

Add a test script in package.json:
{
  "scripts": {
    ...
    "test": "jest"
  }
}

Finally, create your unit test, for example: ./tests/unit/hello.spec.js
import { shallowMount } from '@vue/test-utils'
import HelloWorld from '../HelloWorld.vue'

describe('HelloWorld.vue', () => {
  test('renders props.msg when passed', () => {
    const msg = 'new message'
    const wrapper = shallowMount(HelloWorld, {
      propsData: { msg }
    })
    expect(wrapper.text()).toMatch(msg)
  })
})

Then execute your test with npm test.


Training Loss Error

In TrainingAnomalyDetector_public.py, I had some problems. Could you help me?
In the "def get_args()"function, I set the epochs and models_dir initial value:
parser.add_argument('--models_dir', type=str, default="exps/models")                     parser.add_argument('--epochs', type=int, default=3)
Because if I don't set the initial value of "epochs" and "model_dir", it doesn't seem to work. The path of "features_path" and "annotation_path" are both correct. And I didn't make any other changes to this file（TrainingAnomalyDetector_public.py）.
But when I run it, I have the following errors:
(adCVPR18) q@shy-imrs: ~/AnomalyDetectionCVPR2018-Pytorch$ python TrainingAnomalyDetector_public.py
2021-03-05 11:48:00,130 Training for 3 epochs
/home/q/AnomalyDetectionCVPR2018-Pytorch/network/anomaly_detector_model.py:36: UserWarning: This overload of nonzero is deprecated:
nonzero()
Consider using one of the following signatures instead:
nonzero(*, bool as_tuple) (Triggered internally at  /pytorch/torch/csrc/utils/python_arg_parser.cpp:882.)
normal_vids_indices = (y_true == 0).nonzero().flatten()
2021-03-05 11:48:02,943 Epoch 0/3      Iteration 0/1    loss: 1.0325759649276733    Time: 2.812 seconds/iteration
2021-03-05 11:48:02,943 Save the model after epoch 0
2021-03-05 11:48:02,961
Epoch 0/3     Train loss: 1.0325759649276733        time: 0:00:02.830652
2021-03-05 11:48:05,723 Epoch 1/3      Iteration 0/1    loss: 1.0425223112106323    Time: 2.761 seconds/iteration
2021-03-05 11:48:05,723
Epoch 1/3     Train loss: 1.0425223112106323        time: 0:00:02.761791
2021-03-05 11:48:08,512 Epoch 2/3      Iteration 0/1    loss: 1.0364242792129517    Time: 2.788 seconds/iteration
2021-03-05 11:48:08,512
Epoch 2/3     Train loss: 1.0364242792129517        time: 0:00:02.788565
Traceback (most recent call last):
File "TrainingAnomalyDetector_public.py", line 103, in
save_every=args.save_every)
File "/home/q/AnomalyDetectionCVPR2018-Pytorch/network/TorchUtils.py", line 132, in fit
self.notify_callbacks('on_training_end', self.model)
File "/home/q/AnomalyDetectionCVPR2018-Pytorch/network/TorchUtils.py", line 85, in notify_callbacks
method(*args, **kwargs)
File "/home/q/AnomalyDetectionCVPR2018-Pytorch/utils/callbacks.py", line 80, in on_training_end
plt.plot(range(1, self.epochs + 1), self.val_loss, label=f'Validation loss')
File "/home/q/anaconda3/envs/adCVPR18/lib/python3.7/site-packages/matplotlib/pyplot.py", line 2789, in plot
is not None else {}), **kwargs)
File "/home/q/anaconda3/envs/adCVPR18/lib/python3.7/site-packages/matplotlib/axes/_axes.py", line 1666, in plot
lines = [*self._get_lines(*args, data=data, **kwargs)]
File "/home/q/anaconda3/envs/adCVPR18/lib/python3.7/site-packages/matplotlib/axes/_base.py", line 225, in call
yield from self._plot_args(this, kwargs)
File "/home/q/anaconda3/envs/adCVPR18/lib/python3.7/site-packages/matplotlib/axes/_base.py", line 391, in _plot_args
x, y = self._xy_from_xy(x, y)
File "/home/q/anaconda3/envs/adCVPR18/lib/python3.7/site-packages/matplotlib/axes/_base.py", line 270, in _xy_from_xy
"have shapes {} and {}".format(x.shape, y.shape))
ValueError: x and y must have same first dimension, but have shapes (3,) and (0,)
First, I can only get " Train loss " but I could not get the  "Val loss", maybe that's why the X and Y dimensions are not consistent，but I don't know if my inference is correct.Could you help me to solve this "ValueError" problem?
Second, I want to ask that if I need to initialize the values for "eval_iter" and "evaluate_every" ?
In network\TorchUtils.py  def fit(self,
train_iter,
criterion,
optimizer,
eval_iter=None,
epochs=10,
network_model_path_base=None,
save_every=None,
evaluate_every=None,):
val_loss = None if eval_iter and evaluate_every and epoch % evaluate_every == 0:     logging.info(f"Evaluating after epoch {epoch}")     val_loss = self.evaluate(criterion=criterion,data_iter=eval_iter, ) self.notify_callbacks('on_training_iteration_end', train_loss, val_loss)
Thank you for answering my question！

@quanoo
I just merged a patch. Let me know if this fixed the problem for you.

I tried, but I got the same ValueError error.

Hi @quanoo
Please pull the code and try again. I tested it and it works just fine now + I uploaded new models that I finished training now for 100K epochs.

I downloaded and ran it again, and it can run successfully this time. Thank you for helping me solve this problem. But I trained for 2W times, the result is not convergent, loss has been hovering at 1.04, excuse me is the number of training is not enough?

@quanoo
What auc value do you get?

0.62

Which features are you using? The pre-computed features or your own features?

In TrainingAnomalyDetector_public.py
features_path=> Directory having pre-computed features downloaded from https://drive.google.com/drive/folders/1rhOuAdUqyJU4hXIhToUnh5XVvYjQiN50?usp=sharing
annotation_path=>Train_Annotation.txt
In generate_ROC.py
features_path=> Directory having pre-computed features downloaded from https://drive.google.com/drive/folders/1rhOuAdUqyJU4hXIhToUnh5XVvYjQiN50?usp=sharing
annotation_path=>Test_Annotation.txt
model_path=>epoch_20000.pt (PT file obtained after 2w TrainingAnomalyDetector_public.py training)


Get the contact page topic to center

Is your feature request related to a problem? Please describe.
The contact page topic is not centered. Look at the homepage and history page. The topics are centered.

Describe the solution you'd like
Center the topic using
Describe alternatives you've considered
N/A
Additional context
N/A
Are you working on it?
Yes

Shall I work on this ?

Let's keep it as it is and change it to the following

Contact us

ok brother

Brother, I made changes that you told me. But it is not centered.

Shall I center it using h1 class= "topic text-center"

No he told no to center

Keep changes like he told

Let's make it h1 and keep left aligned.

Ok brother


Alerts: in-chat alert spam protection

Hello, I've managed to learn how these pull requests work, so here's one.
I've implemented a spam protection feature, which would prevent the bot from spamming alerts into the chat every time a user spams the cheap spells.

First, we check if the user had already sent a spell within the delay time specified in the config file.
Second, we check if the pervious spell caster is the current spell caster
Third, if the user enabled the separateSpells, we check if the user is spamming the same spell.

If the time, user, and spell tests fail, we don't send the chat alert.
If any test passes, we sent the chat alert.
My implementation did work before the present BlazeBot login errors.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


chore(backend): connect all abort controllers to shutdown

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#459  👈
#458
#457
#456
#455
#454
#453
#452
#451
#450
#449
#448
#446
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


⚠ Fortnite has degraded performance

In 28265c0, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1512 ms

Resolved: Fortnite performance has improved in e11b474.


full screen button

when i click full screen button,I want to landscape not portrait fullscreen.
How can i do this ?

see orientationActivity in demo


update insert pending segments logic to synchronous

Current pending segment id logic is not friendly to my business. It will retry to find the max partition num and plus 1 until reach max retry count.
Our kafka topic has 4 million records per second distributed in 400 partitions. So I will receive 30,000 - 40,000  'io.druid.java.util.common.RetryUtils - Failed on try ****' WARN message in my overlord log one day about pending segment duplicate id.   Sometime it will cause my task lag, like retry failed count more than 10 or task timeout.
So I think make this logic TaskLock level synchronous is more smoothly.
Add the SegmentLock.class is my work around solution. Because TaskLock is belong to druid-indexing-service module, it cannot be import to IndexerMetadataStorageCoordinator with druid-server module. Either, add a ReentrantLock in TaskLock, maybe.
If you have some better solution, please let me know.
Thank you.

Hey @FaxianZhao - any reaosn you closed this? I was just thinking about reviewing this. Sorry we didn't get to it straight away. We have a bit of a backlog of pull requests.


investigate possibility to load/parse all config.properties inside transitive carina sub-modules

to start splitting modules to independent projects carina-crypto looks the best candidate. making this movement we support more crypto algorithms etc.
the only blocker on this step is our operation with config.properties which is global one and located only in carina-core/src/main/resources.
we have to reorganize code a little bit using better classloader or via transitive maven resource dependencies to be able to split each param inside independent project and be able to load and combine on R.init

Verified, no VNC session for web test.


EDGCONX-30 dynamic login option

The use case for the 'both' configuration is multiple tenants on one cluster preferring different login strategies.
I tested this on nolana and orchid bugfest using 'both', 'full', 'key' and no configuration (default).
I did consult with Adam after my original pull request.  He suggested the 'both' option.  It leaves the full and key options as they were so hosts can restrict to one or the other when needed.

@dcrossleyau - Do you have any objection to me creating a version with this change?  (v1.0.7)  thank you!

I do not know, sorry.


Makefile: provide Go file patterns to go test, removing for loops

Invoking go {test,build} for every package in the repository turns out to be prohibitively slow. This patch speeds up priv/integration tests by an order of magnitude by passing all test packages to go test as command line arguments.
Privileged tests run in about 5 minutes on a 4-core VirtualBox machine with 8GiB of RAM. Without this patch, they take about 1.5 hours. Without documentation of which environment the loop workaround is for, it's hard to justify keeping this hack around, especially given its impact.
Instead, we should move towards making all packages go test-able without requiring this fragile filtering logic by adding skipping test helpers.
Provide Go file patterns to `go test`, removing for loops in Makefile

cc @tklauser

/test-runtime

/test

/test

Commit log (commit 3) says 1.5 hours and not 15 minutes, could you please fix where necessary?

Thanks for the reminder, wanted to get some reviews in before having to potentially re-run CI.


[BUG] ... Error: Error connecting to PostgreSQL server  (scheme: postgres): XXXXdXXXXial tcp <IP_ADDRESS>:5432: connect: connection refused

Community Note

Please vote on this issue by adding a :thumbsup: reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version
1.6.6
AzureRM Provider Version
7.3.0
Affected Resource(s)/Data Source(s)
postgres flexible server deployment
Terraform Configuration Files
in the process of postgres db creation, terraform fails right after apply the firewall filters.

such as making more configurations after postgres flexible server is deployed.
For example the following configuration wouldn't complete and fails after the firewall filters are applied.

module "postgresql_users" {
  source  = "claranet/users/postgresql"
  version = "6.0.0"

  for_each = module.postgresql_flexible.postgresql_flexible_databases_names
  administrator_login = module.postgresql_flexible.postgresql_flexible_administrator_login
  database = each.key
}

module "postgresql_configuration" {
  source  = "claranet/database-configuration/postgresql"
  version = "6.0.0"

  for_each = module.postgresql_flexible.postgresql_flexible_databases_names

  administrator_login = module.postgresql_flexible.postgresql_flexible_administrator_login
  database_admin_user = module.postgresql_users[each.key].user
  database            = each.key
  schema_name         = each.key
}

Debug Output/Panic Output
module.accelerator.module.postgresql_flexible.azurerm_postgresql_flexible_server_firewall_rule.firewall_rules["0"]: Still creating... [30s elapsed]
module.accelerator.module.postgresql_flexible.azurerm_postgresql_flexible_server_firewall_rule.firewall_rules["0"]: Still creating... [40s elapsed]
module.accelerator.module.postgresql_flexible.azurerm_postgresql_flexible_server_firewall_rule.firewall_rules["0"]: Still creating... [50s elapsed]
module.accelerator.module.postgresql_flexible.azurerm_postgresql_flexible_server_firewall_rule.firewall_rules["0"]: Still creating... [1m0s elapsed]
module.accelerator.module.postgresql_flexible.azurerm_postgresql_flexible_server_firewall_rule.firewall_rules["0"]: Creation complete after 1m2s [id=/subscriptions/xxxxxxxxxxxxxxxx/resourceGroups/rg-fgeasf9/providers/Microsoft.DBforPostgreSQL/flexibleServers/psqlf-default-stack-bla-dev/firewallRules/0]
╷
│ Error: Error connecting to PostgreSQL server  (scheme: postgres): XXXXdXXXXial tcp <IP_ADDRESS>:5432: connect: connection refused
│
│   with module.accelerator.module.postgresql_configuration["main_db"].postgresql_grant.revoke_public,
│   on .terraform/modules/accelerator.postgresql_configuration/r-psql-db-config.tf line 1, in resource "postgresql_grant" "revoke_public":
│    1: resource "postgresql_grant" "revoke_public" {
│
╵

Expected Behaviour
should apply the firewall filter as the last process so that other operations are not blocked.
Actual Behaviour
fails in the middle of deployment
Steps to Reproduce
No response
Important Factoids
No response
References
No response

Hello,
how did you configure the postgresql provider? did you use the code we have in our examples?
If you look at your error, the provider is trying to connect to <IP_ADDRESS> (which is not related to firewall settings...)


Target NETStandard

Greetings! I'm evaluating integrating Hellosign into a new solution. Are there plans to target NETStandard to support those of us running on .NET Core?

Any plan on supporting .NET core?

I've been told they have no plans. I did find this unofficial implementation: https://github.com/soukoku/HelloSignApi

Thanks

Looking for an e-signature solution to get started with ASAP for our startup, coming down to Docusign vs HelloSign. Docusign is currently lagging on their .netstandard2.0 support. If this could be made available we'll likely go with HelloSign.

Any updates on this? We're trying to migrate our solution to .NET Core, but are currently blocked by this.

Are there any updates for this?


Configuration of SMTP Email Service is required

Can we make it so that signup requires email verification? (To ensure that if the student registers with an ID, they actually own that email.)

@BilalZaib

@soachishti
Done with signup email verification but didn't push.

Create a branch of that code, push that branch and start a pull request.
git add .
git commit -m "Changes"
git checkout -b SignupVerification
git push origin SignupVerification
Logon to GitHub and start a pull request.

SMTP settings are added (Mailgun). #21
You will have to install following modules pip install django-anymail[mailgun]
"From" section of email for "Password Reset" and "Sign Up Invitation" will be<EMAIL_ADDRESS>It is limited by Mailgun. However it can be changed but that new email need to be enabled from Mailgun online account settings.
@BilalZaib @recluze

Umm.. why are we using mailgun? What's wrong with local SMTP (other than the fact that it will most probably land in their SPAM folder ... but that can be taken care of easily). Not saying we shouldn't do this. Just asking.

In most cases local SMTP doesn't work, you can take example of my mine. If we move to web hosting provider they have switched SMTP to paid option for example GoDaddy.
However we can use Google and Outlook SMTP server for delivering emails. I have been using Mandrill and Mailgun for quiet long a ago that is why I selected this service.

I haven't had any issues with local mail sending but I guess it can't hurt to use Mailgun. At the very least, we'll solve the issue of mail going to the spam folder with this. Let's keep it.

#30
Added Outlook SMTP.

Works. Please send me the updated password through email and close this out please. Thanks.

Closing this issue... Sent the new settings.py file to your email address.


Add feature to share the poll

Is it possible to add a share option for the poll that has been created? I created a poll and added all my delegators as eligible to vote on. But they have to go on the site and manually search for name of the poll which i don't think is very efficient especially if more bakers start using this to ask the same questions

@TezBlocksBakery
I am refactoring the app and now on the url you can look at specific poll.
Btw, do you want also the vote button on the result/details page of a poll ?

Right now, it is not, just on the search/list page

Maybe, you would like also to filter by Creator ?

Hi Benjamin,
Yes, I believe having the vote button in the details page is more important.
Do you not think having all those different votes displayed a bit confusing?
And is it possible to create a link to a specific poll?
Kind regards,
Eli

From: benjamin fuentes @.>
Sent: Thursday, 7 September 2023, 14:26
To: marigold-dev/tzvote @.>
Cc: TezBlocksBakery @.>; Mention @.>
Subject: Re: [marigold-dev/tzvote] Add feature to share the poll (Issue #24)
@TezBlocksBakeryhttps://github.com/TezBlocksBakery
I am refactoring the app and now on the url you can look at specific poll.
Btw, do you want also the vote button on the result/details page of a poll ?
[image]https://user-images.githubusercontent.com/2536368/266315652-285d1285-5660-4a9b-92bd-a4988b727bf8.png
Right now, it is not, just on the search/list page
—
Reply to this email directly, view it on GitHubhttps://github.com/marigold-dev/tzvote/issues/24#issuecomment-1710060962, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7SAUN47IGIIE7UASNVUU3DXZG4QNANCNFSM6AAAAAAXTKIQRU.
You are receiving this because you were mentioned.Message ID: @.***>

I don't believe that would be necessary.
Have you thought about an easy way to add a list of delegators?

From: benjamin fuentes @.>
Sent: Thursday, September 7, 2023 2:27:27 PM
To: marigold-dev/tzvote @.>
Cc: TezBlocksBakery @.>; Mention @.>
Subject: Re: [marigold-dev/tzvote] Add feature to share the poll (Issue #24)
Maybe, you would like also to filter by Creator ?
—
Reply to this email directly, view it on GitHubhttps://github.com/marigold-dev/tzvote/issues/24#issuecomment-1710061825, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7SAUN6G3DTERHIXJ5HJI2LXZG4S7ANCNFSM6AAAAAAXTKIQRU.
You are receiving this because you were mentioned.Message ID: @.***>

If you have an Android , you can already test a preview on ghostnet here :
https://play.google.com/store/apps/details?id=dev.marigold.tzvote

Delegators : if you are baker, now you have a button to add all your delegators to the list of voters in 1 click

I think I will rename the Settings to Details, like this we could manage + do other actions on this page

@TezBlocksBakery thank you for the feedback

Hi Benjamin,
I create polls, not with my baker wallet but with my payout wallet, could you add the option to pull delegators from any public baker?
Also i believe an option to hide polls by the poll creator would be a great addition incase a mistake has been made, i don't believe you can delete them if someone already has voted so hiding them would be the next best thing.
Maybe also hide finished polls from the main page after a week? So it doesn't get cluttered
Sorry for that i keep requesting stuff, i don't know how much work this is.
Thank you that you keep working on tzvote!
Kind regards,
Eli

From: benjamin fuentes @.>
Sent: Friday, September 8, 2023 9:34:56 AM
To: marigold-dev/tzvote @.>
Cc: TezBlocksBakery @.>; Mention @.>
Subject: Re: [marigold-dev/tzvote] Add feature to share the poll (Issue #24)
@TezBlocksBakeryhttps://github.com/TezBlocksBakery thank you for the feedback
—
Reply to this email directly, view it on GitHubhttps://github.com/marigold-dev/tzvote/issues/24#issuecomment-1711214479, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7SAUN7JHNQWZZIPSZM7Y7TXZLDCBANCNFSM6AAAAAAXTKIQRU.
You are receiving this because you were mentioned.Message ID: @.***>

pull delegators from any public baker?    <= why others ? Why do you need this ? Because you don't connect with your baker directly ? Do you want to connect with a Ledger instead ?

Also i believe an option to hide polls by the poll creator would be a great addition incase a mistake has been made, i don't believe you can delete them if someone already has voted so hiding them would be the next best thing.
<= A status to "REMOVE" or "DISABLED" , can be possible to filter out from indexer , yep ! Thx !!! Got it

Maybe also hide finished polls from the main page after a week? So it doesn't get cluttered
<= I can add an auto filter by end date time to last 15 days, why not

Just UI stuff is not a lot   ^o^
Smart contract is a bit more ..

Indeed, i don't connect with my baker directly, instead i use my payout wallet.
So, when i create a poll using my payout wallet, i need access to TEZ blocks delegators

From: benjamin fuentes @.>
Sent: Friday, September 8, 2023 10:57:01 AM
To: marigold-dev/tzvote @.>
Cc: TezBlocksBakery @.>; Mention @.>
Subject: Re: [marigold-dev/tzvote] Add feature to share the poll (Issue #24)
pull delegators from any public baker? <= why others ? Why do you need this ? Because you don't connect with your baker directly ? Do you want to connect with a Ledger instead ?
—
Reply to this email directly, view it on GitHubhttps://github.com/marigold-dev/tzvote/issues/24#issuecomment-1711319959, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7SAUN3FNTDMXCYPR3IC2MTXZLMV3ANCNFSM6AAAAAAXTKIQRU.
You are receiving this because you were mentioned.Message ID: @.***>

Thank you Benjamin!

From: benjamin fuentes @.>
Sent: Friday, September 8, 2023 11:05:55 AM
To: marigold-dev/tzvote @.>
Cc: TezBlocksBakery @.>; Mention @.>
Subject: Re: [marigold-dev/tzvote] Add feature to share the poll (Issue #24)
Thanks a lot @TezBlocksBakeryhttps://github.com/TezBlocksBakery I will all of this to next release (not sure yet about the disables as it requires some redeployment of smart contracts :/)
—
Reply to this email directly, view it on GitHubhttps://github.com/marigold-dev/tzvote/issues/24#issuecomment-1711334607, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7SAUNYN33NB7U7MRKHSVQLXZLNXHANCNFSM6AAAAAAXTKIQRU.
You are receiving this because you were mentioned.Message ID: @.***>


Add Godot String intern support

To prepare the API to hide the Godot String / C.godot_string, we should add string interning to reduce C memory allocation

Have to reopen this; it looks like types also utilize godot_string as arguments such as gdnative.NewNodePath(p_from String) here.


Fix #271 Update the initial used for yandex which clashed with yahoo

Fixes #271
Checklist

[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream master branch.
[ ] I have added necessary documentation (if appropriate)

Changes proposed in this pull request:

Quick fix, update initials used for yandex that clashed with yahoo.

That is taken care off in other PR. Only problem all the other projects need to be sent an update to work on their calls.
Since changing it would return them 404 otherwise.

We should use full names instead of initials.

Cool then.
Will change it to use full names.
:+1

Hey, https://github.com/fossasia/query-server/pull/377 has a nice broade fix :)

Closing this as its taken care of in #377


Does gRASPA supports to calculate flexible frameworks using framework.def ?

Dear Developers:
In RASPA, calculations for flexible frameworks with P1 symmetry are very time-consuming. Does gRASPA support calculations for flexible frameworks? For example, are the following keywords supported:
·FlexibleFramework [ ]
·FrameworkDefinitions [ ]
·VolumeChangeProbability [ ]
Thank you for your response.

gRASPA does not support flexible framework at the moment, but you are always free to use the hybrid MC/MD scheme by iteratively switching between gRASPA and other MD package (e.g., LAMMPS).

Hi,
I would like to add to Prof. Shi's answer that gRASPA has the capability of reading/writing LAMMPS Data file. Please take a look at this example.


Provide overrides of Ironic provisioning IP, http port, and dnsmasq r…

…ange
Allow for overrides using environment variables of Ironic configuration.
The folowing environment variables are accepted:
IP - provisioning interface IP address (default <IP_ADDRESS>)
HTTP_PORT - port used by http server (default 80)
DHCP_RANGE - dhcp range to use for provisioning
(default <IP_ADDRESS>-<IP_ADDRESS>

Build SUCCESS, see build http://<IP_ADDRESS>:8080/job/dev-tools/428/

Build FAILURE, see build http://<IP_ADDRESS>:8080/job/dev-tools/438/


[helm chart] ServiceMonitor support

Description:
Enable OSM monitoring using a ServiceMonitor
See : #4699

Testing done:

Affected area:

Functional Area

New Functionality
[ ]

CI System
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[ ]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[ ]

Other
[x ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project?

Did you notify the maintainers and provide attribution?

Is this a breaking change?
No.

Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)?

@nlamirault thanks for contributing. Could you open up an issue describing the need for this change so we can discuss further. Thanks

Done @shashankram

/remove stale

Codecov Report

Merging #4693 (a5960a3) into main (ab69461) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #4693      +/-   ##
==========================================
- Coverage   68.83%   68.81%   -0.03%
==========================================
  Files         214      214
  Lines       16025    16025
==========================================
- Hits        11031    11027       -4
- Misses       4934     4938       +4
  Partials       60       60

Flag
Coverage Δ

unittests
68.81% <ø> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/messaging/workqueue.go
89.28% <0.00%> (-10.72%)
:arrow_down:

pkg/certificate/manager.go
78.05% <0.00%> (-0.36%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@nlamirault Could you please update the charts by running make check-charts at the root of the project?

make check-charts where do you find this command ? @jaellio

Hi @nlamirault, at the root of the OSM project there is a [makefile](https://github.com/openservicemesh/osm/blob/4d6d8c9d7c77df92a8e0a13de29f05b192f7bb92/Makefile#L96) that defines the check-charts tag. The check-charts task will regenerate charts/osm/README.md. In your branch, if you navigate to the root of the repository and run make check-charts the task will run and you can push a new commit. Let me know if you have any questions!

Closing this PR for now due to inactivity. @nlamirault please comment on the PR and tag me if you would like this to be reopened! Thanks!


[Addon] add redis-operator addon

Description of your changes
Fixes https://github.com/kubevela/kubevela/issues/4288

[x] installation of the crd and operator https://github.com/spotahome/redis-operator
[x] Useful and reliable ComponentDefinition

[ ] Health check of redis cluster (I can't find a good way to do health checks, see code comments)
[x] Expose an endpoint for using (available as a Service)
[x] Support scale up/down

[x] Documentation

[x] use redis cli to connect with demo

How has this code been tested?

Checklist

I have:

[x] Title of the PR starts with type (e.g. [Addon] , [example] or [Doc]).
[ ] Updated/Added any relevant documentation and examples.
[x] New addon should be put in experimental.
[ ] Update addon should modify the version in metadata.yaml to generate a new version.

good job


Added head_cam frame to urdf

Requires https://github.com/ipa320/cob_common/pull/195 and https://github.com/ipa320/cob_calibration_data/pull/105

@ipa-fmw can be merged? travis is not able to compile the current cob_robots (indigo_dev) version, this PR fix the problem

Just wondering why the cam is called head_cam after the mount position rather than using the manufacturer/model/camera_type as the name for the makro...

@ipa-fxm you mean the name of the .urdf.xacro file on cob_description?  it is a generic web cam, actually the driver used is also a generic usb driver https://github.com/ipa320/cob_robots/blob/indigo_dev/cob_bringup/drivers/usb_camera_node.launch. But I have no problems to rename the files...


let's add CI badge

Keep the website link

@jesperpedersen somehow I'm not able to squash my commit, please squash and merge the PR.

You need to fix that

Squash ?


Fix duplicate NavigationServer process

NavigationServer process was called twice each frame.
Regression added with recent https://github.com/godotengine/godot/pull/70731 .... don't rebase or review pr's while tipsy.

Added 3 missing MONITOR_TYPE entries to this pr.

Thanks!


Hook mechanism for completion of Bundler.require

I'd like to add a mechanism to permit a Gem to register a block to be executed automatically once Bundler's Runtime has completed its require pass. I propose adding instance methods to Runtime and class methods to Bundler that would forward to the Bundler class's Runtime (Bundler.load) for this.

Code in a Gem would use syntax such as:
Bundler.when_require_complete do
  # final initialization code here
end

Runtime.when_require_complete would stash the provided block for later execution.

Runtime.require would, upon completion of its loop through the list of dependencies, call each registered block.

Thoughts that occur to me:

Does Bundler need to be threadsafe at all? What happens if two threads call require on the same Runtime object simultaneously? If that is a situation that currently works, then presumably I'll need to wrap accesses to the list of when_require_complete blocks in a Mutex.
If Bundler is called multiple times, of course the second invocation's require calls, if pointed at the same Ruby files, won't actually run any code. So, if we assume that the finalization code might need to run again if the loadout of Gems changes, the callback handlers need to persist from one call to Runtime.require to the next.

If this sort of functionality is likely to be approved by Bundler's maintainers, then I'll go ahead and make a pull request for them. :-)
Rationale:
We have a suite of Gems that tie into a common configuration subsystem, which is defined in its own Gem. Each Gem needs to add its embedded configuration directory to a central list before the configuration subsystem initializes itself and pulls in all of the values, based on order of precedence. Right now, we have no easy way to ensure that the configuration subsystem always initializes after all of the other Gems have had a chance to add their directories to the list. One approach we've tried is to place the variable initialization into the root file for the Gem that handles the configuration, and then arranging things so that it is the last Gem that Bundler.require brings in, but this has hit a snag with a Gem that requires a specific version (or newer) of the Gem that handles the configuration. Because it declares it as a dependency, that guarantees that the configuration Gem gets pulled in before this other Gem that wants to add to the configuration. We want to avoid having to explicitly initialize the configuration mechanism in every file that uses it -- there are hundreds of locations maintained by dozens of different teams.
Therefore, having the ability to register a block with Bundler to be automatically executed when the Runtime completes its require pass is a convenient mechanism to allow each of the Gems specified in the Gemfile (and brought in via dependencies) to initialize, and then use the configuration state data those initializations have built up to fire up the configuration subsystem.

Could you code this up as a plugin?

I don't think so... There isn't currently any code at all in the Runtime.require method after the end of its loop through the dependencies. I could overwrite the function:
module Bundler
  class Runtime
    require_impl = instance_method(:require)

    def require(*group)
      require_impl.bind(self).(group)
      @hooks.each { |hook| hook.call() } unless @hooks.nil?
    end

    def when_require_complete(&block)
      @hooks ||= []
      @hooks << block
    end
  end
end

...but this would then depend on the specific internal implementation of Bundler. If the name of class Runtime, or the way that it is implemented, ever changed then the plugin would stop working.
Also... does Bundler actually have a plugin system that could handle this today?

This can be implemented as a hook plugin which is not present in bundle yet but are in the pipeline.

Oh? That's cool. :-) Is there some documentation of the ongoing development of hook plugins? I did some Googling, but could only find reference to plugins associated with bundle install and bundle update-type operations ("Lifecycle hooks"), not calls to Bundler.require from Ruby code.

Sorry, as this has not yet been implemented, so it's not documented.

How can it be in the pipeline, and yet have no planning, no discussion, no documentation and no code?
The reason I want to follow up on it is that, as mentioned, I've seen plenty of documentation of requests for plugin interfaces to automatically run code during bundle install or bundle update, but I haven't seen anyone looking for a "runtime" hook that would interact with the Bundle.require method itself. I wanted to make sure that what was in the pipeline actually covered that.
In issue #1945, @BMorearty asked about "before require" and "after require" hooks, which would obviously need to be in place before the call to Bundle.require starts. The thread did not address his questions, though.
In feature #8, plugins are much-discussed, but almost exclusively in the context of adding new commands to the bundle command-line interface. This appears to have actually been implemented in #3030.
In issue #3463, there is further discussion of plugins, but it specifically mentions only these types of integration:

New subcommands, as implemented in #3030.
Source control plugins.
Hooks registered, presumably as part of having a supporting Gem in the dependency tree, to run once a bundle install or bundle update completes.

Across all of this, @BMorearty's request is the only mention of the Bundler.require process, and even his would be something pre-registered. I'm looking for something that allows a Gem to tie into a post-all-requires event as it is being required.
Is that, specifically, in the pipeline?
Thanks :-)

Thank you for clarifying! That feature (to be clear: allowing a gem to add a callback once Bundler.require has completed, while it is being required by Bundler.require) is not in the pipeline. We haven't finalized the list of "lifecycle" callbacks, however, and before/each/after hooks for setup and require were at least on the list of possibilities. Thanks for this feedback, and we'll try to support this (or something else that can satisfy this use case) in the future.

Awesome, thanks very much :-)

How is this not a thing yet??  🤔

@nestor-custodio no one has sent a pull request with a tested implementation yet ;)


Question - Local Environment - Unable to get past the pin setup screen on Android emulator

Hi, I am trying to run BC wallet locally. On Android Emulator (pixel 4 - Android Version S 31)
Stuck on the following screen. The Create Pin button doesn't do anything. I don't see any error related to this in the logs: Any help is massively appreciated!

Here are the steps I followed:
Note - The developer documentation no longer works. Looks like I need to start metro first. Please advice.

git cloned bc-wallet-mobile -> master branch
installed and started the app using the following command. npm run install:all --legacy-peer-deps
 cd app
 npm run start
 # In a new terminal session
 npm run android

Not sure if I am missing anything!

Note: I have von-network and issuer-kit-dev running locally and would like to use the bc-wallet to scan the QR code.

Make sure you have enable biometrics security in the emulator, as well registering the emulated fingerprints (I usually just use "finger1":
https://stackoverflow.com/a/45181265

That worked, thank you @cvarjao!


feat: add constants/float32/sqrt-phi

Description

What is the purpose of this pull request?

This pull request:

adds constants/float32/sqrt-phi, which would be the single-precision equivalent for constants/float32/sqrt-phi.

Related Issues

Does this pull request have any related issues?

This pull request:

None

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

/stdlib lint-autofix


Setup server

Will probably run on a new project on GCP

Also examine auto deployers of fastify to GCP cloud functions (or AWS lambda :grin:).


DCD-436: Enable CloudWatch integration in playbook

This activates the CloudWatch agent installation and configuration in the deployment playbook. It's a three-way option:

None: No installation at all
Metrics: Basic metrics (CPU, network, disk)
Metrics & Logs: Include the application logs (/logs/* + Catalina logs)

As a bonus, this also changes the SSH keypair to be a drop-down of available options to bring it into line with BB (see DCD-356).

Ooh just noticed there is a linting error for this PR too https://server-syd-bamboo.internal.atlassian.com/browse/DCD-AWSJIRA28-LINT-7


Fixed point elements

fixed point values are a common way to exchange floating point data in packed/C messages.  Create a fixed point class and then allow message element data to be defined using the standard python Decimal (https://docs.python.org/3.5/library/decimal.html) class... potentially or normal floating point data.

#9 added this


General maintenance and updated dependencies, to help bring this package up-to-date

After being held up by issue #40, I've gone ahead and updated the dependencies used by this package to bring it up to date.
Here's what's changed:

Updated composer.json to use version ^2.0 of themsaid/forge-sdk. Version 2.0 has some new parameters added to certain functions (see here), which I haven't accounted for in forge-cli — but the function definitions remain backwards-compatible.
Updated composer.json to use version ^5.0 of symfony/console and symfony/process. This was necessary as the current release of laravel/homestead installs version ^5.0 of these packages globally, so the existing dependency version ^4.0 was causing conflicts that prevented installation.
Added return 0; to every symfony/console inherited execute() function, a new requirement following the version updates. return 0; indicates the command ran successfully, any other result should be an integer and will be returned as a process exit status to the shell.
Changed the forge authorize command to allow for the API key to be passed in as an optional parameter. If no parameter is specified, the user will be prompted to type it in to the terminal window themselves. This change allows programmatic setup of the Forge CLI via other Artisan commands.

@svenluijten Thanks, happy to contribute! I just pushed a few more tweaks to this branch...

Changed dependency version constraints, as you suggested
This one is kind of opinionated, but I noticed the package just dumps a forge.json config file in the user's home directory:

I changed the getFileConfig() function to prefer a hidden ~/.forge.json file instead
It will continue to use ~/forge.json if one exists for backwards-compatibility
Going forward, new calls to forge authorize will save a hidden file instead
I feel this is more in keeping with convention for global packages

When a .forge.json configuration file is created for the first time, a JSON key property is now being set to an empty string:

Previously, it was just an empty object being saved ({})
This change ensures "Unauthorized" exceptions are thrown by Guzzle, instead of a PHP fatal error
Errors would be thrown when trying to call the Forge API before forge authorize has been run

So sorry for the delay on this, but due to what's going on in the world haven't really had the time to check this out in-depth. I will get to it this weekend, I promise!

This weekend? ;)
I'm looking into managing backups. That requires https://github.com/themsaid/forge-sdk/pull/67 to be merged but then it would need v2 of the SDK.

Tagged as v0.5.0!

Btw @barryvdh, I'd be happy to get those backup methods also integrated into this CLI once it makes its way into the SDK. 👍


fixes #392 to add gofabric8 che command

something seems wrong with our CI builds; not using the latest commit; so gonna merge manually for now


Integer conversion warning

Hi,
first I will thank you for the great work. ioLibrary_Driver is realy a great lib!
I got a warning in the TI compiler in the socket.c file. Currently it is line 436
else if(tmp & Sn_IR_TIMEOUT)
{
setSn_IR(sn, Sn_IR_TIMEOUT);
len = SOCKERR_TIMEOUT;
break;
}
which ended up in a warning like this
"#69-D integer conversion resulted in a change of sign	socket.c"
I think it is not only a warning, because SOCKERR_TIMEOUT is negative (-13) and len is an unit16_t, so it is a bug.
Kind regards

Dear fritzpas
I will make sure this issue.
Thank you for your insights.

Dear fritzpas!!
I have  fixed the issued.
This is closed.
Thank you so much!!!


customHelper language specific characters

Hello,
I made a project with handlebars. It works great and does what it has to do.
The only problem I have is that with a custom helper I do not get the language specific characters thru.
For the sake of simplicity this is what I have:
Handlebars.registerHelper('html10n', function (str) {
    return "ß ăî ومض";
});

I call it with:
{{html10n "blablabla"}}
Unfortunately I get this output: � ?� ???
Even worse than this is that on http://tryhandlebarsjs.com/ my helper works flawlessly and the ouput is ß ăî ومض
Do you have any pointers on this?
Thanks.

It was my bad.
I have placed the registerHelper() in a file that was ANSI encoded.
It works as expected.

Please close, then.


[PPP-5384]-XSS Findings for Pentaho-kettle

PPP-5384-XSS Findings for Pentaho-kettle

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 23m 59s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl plugins/core-ui

:exclamation: No tests found!
:information_source:  This is an automatic message


Add ppc64le

Enable a build for ppc64le be repackaging a RPM from EPEL
Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

closes #68

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

Specifically the RPM come from https://copr.fedorainfracloud.org/coprs/petersen/pandoc/.  I was not able to find a similar package for aarch64.

@conda-forge-admin, please rerender

LGTM, thanks!

Thank you!


Implement requiresMainQueueSetup, fixes #553

Previous PR https://github.com/wkh237/react-native-fetch-blob/pull/674 closed due to new maintainers and repository location.
See https://github.com/wkh237/react-native-fetch-blob/issues/553 for details about the issue being resolved.

Looks good, thanks for your PR!


@link and @see resolve C.m as a class property reference

This does not match real TS references, but is extremely common. It also matches the lookup that will have to happen for C#m, but doesn't require any new parsing.
Fixes #43383 but not #43594 or #43595

@mjbvz You were interested in this.


More Makefile cleanup/edits from old Makefile stuff

Makefile edits
Lint fixes
Signed-off-by: Doug Davis<EMAIL_ADDRESS>

I'd prefer to have the lint commits separate from the build modifications.
LGTM

Looks like a good enough foundation to merge and iterate on in follow-ups

Fwiw checking in generated code is done in more repos than just kubernetes/kubernetes. The helm project, among others that I know of, does it as well.

On Dec 21, 2016, at 07:45, Paul Morie<EMAIL_ADDRESS>wrote:
@pmorie commented on this pull request.
In .gitignore:

@@ -13,3 +13,5 @@ coverage.html
.glide
.scBuildImage
.init
+.generate*
+**/zz*.go
I encourage you to do a search of issues / PRs. It has been discussed several times but I don't have a summary ready at this moment and either way, I do not want to depart from what k8s does in this area atm.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

moved the "controversial" .gitignore stuff into the code-gen PR since its not really part of this clean-up work anyway.  We can continue the discussion over there.


rebase-helper propose-update failed with traceback on git push --force

I have tried to update rebase-helper by packit but it failed with traceback.
.packit.yml configuration file is here: https://github.com/rebase-helper/rebase-helper/blob/master/.packit.yaml
$ packit propose-update
Checking out upstream version 0.16.2
using "master" dist-git branch
syncing ./.packit.yaml
syncing ./rebase-helper.spec
INFO: Running 'anitya' versioneer
INFO: Downloading file from URL https://github.com/rebase-helper/rebase-helper/archive/0.16.2/rebase-helper-0.16.2.tar.gz
    [       <=>                    ]     1.80M   in 00:00:01
downloaded archive: /tmp/tmpeqbb9vh6/rebase-helper-0.16.2.tar.gz
uploading to the lookaside cache
ERROR    Cmd('git') failed due to: exit code(128)
  cmdline: git push --force --porcelain fork 0.16.2-master-update
  stderr: 'fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.'
Unexpected exception occurred,
please fill an issue here:
https://github.com/packit-service/packit/issues

We should get a better error message from gitpython, this is really not helpful.

'Please make sure you have the correct access rights and the repository exists.' seems quite descriptive to me.
The git push has -v/--verbose switch, but it doesn't reveal any more info (I've just tried) about the error.

I agree, but I still feel like we could better. We could at least turn off logs from rebase helper by default.

We could at least turn off logs from rebase-helper by default.

Not sure I follow.
I thought the problem is that there's not enough info from git[python].
Are you saying the rebase-helper output is a useless sauce?

One TODO I have in mind: how about changing the top-most try/catch to: if packitexc is raised, just tell that there was an error during execution and if generic exc is being caught, we should inform user about reporting the issue.
This is how I imagine the output:
$ packit propose-update
Checking out upstream version 0.16.2
Using "master" dist-git branch
Syncing ./.packit.yaml
Syncing ./rebase-helper.spec
Downloaded archive rebase-helper-1.2.3.tar.gz
Uploading to the lookaside cache
ERROR    Cmd('git') failed due to: exit code(128)
  cmdline: git push --force --porcelain fork 0.16.2-master-update
  stderr: 'fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.'

WDYT?

One TODO I have in mind: how about changing the top-most try/catch to: if packitexc is raised, just tell that there was an error during execution and if generic exc is being caught, we should inform user about reporting the issue.

If I got your idea, this is the current state:
https://github.com/packit-service/packit/blob/master/packit/cli/utils.py#L30

Cool, so we should just catch the giterror exc and reraise as packit exc.


Declare the main entry point of the package

This is needed after installing with npm to use require.

What usecase is supported by installing using npm?
The theme code in js/theme.js does not make sense on its own. It needs the additional assets in sphinx_rtd_theme/static

What usecase is supported by installing using npm?

Readthedocs.org bundles the theme.js in some cases and fetches it through bower. If we want to swap out bower for NPM and just do require('sphinx_rtd_theme') we will need this.

Bower is deprecated and no longer recommended.

OK. Does it make sense to make sphinx_rtd_theme/static/js/theme.js the entry point as it is the generated JS or will RTD run it through it's own compilation/minification process?

Typically you don't specify a compressed file as the default entry point as whatever is including it should do it's own compilation process.


[1.12.2] Tough as Nails Thirst render conflicts

Forge Version: <IP_ADDRESS>05
RPG Hud Version: <IP_ADDRESS>
Tough as Nails Version: <IP_ADDRESS>
Minecraft Version: 1.12.2
For some reason, whenever I have Thirst enabled in Tough as Nails and have RPG-Hud installed, the thirst bar only appears whenever I'm underwater. The issue doesn't occur when I have RPG Hud not installed.

This issue happens on pretty much all UI types

As Tough as Nails is discontinued, I will close the thread.
I changed some things with 3.7 of my mod which should take care of this issue nonetheless.


Feature Request: Cell information on row hover

There doesn't appear to be a cell-specific hover function.  I am trying to retrieve the original cell value on hover over an edited cell for display in a popover.  Is there currently a way to get specifics about which column in the row I am hovered over or a potential for a columnMouseover function to be exposed?

I am trying to retrieve the original cell value on hover over an edited cell for display in a popover

How about enabled the columnTitle on TableHeaderColumn?

That would get the job done. Thanks!
From: Allen
Sent: Tuesday, August 9, 2016 8:03 PM
To: AllenFang/react-bootstrap-table
Cc: Rachele Moresi, Author
I am trying to retrieve the original cell value on hover over an edited cell for display in a popover
How about enabled the columnTitle on TableHeaderColumn?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @RacheleMoresi @AllenFang ,
I'm trying to do the same: Hover over cell to obtain full info for a popover.
I'm not sure how adding columnTitle helps? I see no difference in the arguments of onRowMouseOver.
Could you explain how you solved this?
Many thanks

hi @ZephD, check https://github.com/AllenFang/react-bootstrap-table/blob/master/examples/js/column/column-title-table.js :)

That just shows the use of the flag, which I understand (though see no difference it being true or false, the heading is always visible...).
How does it help with the onRowMouseOver and getting the current column / cell?

@ZephD I'm afraid I can't provide you a code sample for that.  I did get it working at one point. However, as the project evolved, I decided to anchor the popover with cell info to an icon in the cell. I passed the information I needed to the icon component using the dataFormat property, and then setting the popover content based on the info stored in the props of the icon.

Anchoring the hover info to an object in the formatted cell... Could do the job. Thanks.

I am looking to show a text on hover over a cell.
how can I hover over one particular cell?


module 'fzy' not found

nvim version
NVIM v0.5.0-dev+1354-ga282a177d
Build type: Release
LuaJIT 2.1.0-beta3

package manager
packer:
use { 'camspiers/snap', rocks = {'fzy'}}

config
copy from readme: https://gist.github.com/camspiers/686395ab3bda4a0d00684d72acc24c23
issue
after packer compile, packer install  and restart nvim, I got this:
E5113: Error while calling lua chunk: ...ite/pack/packer/start/snap/lua/snap/consumer/fzy/all.lua:3: module 'fzy' not found:
        no field package.preload['fzy']
        no file './fzy.lua'
        no file '/home/xxx/github/neovim/.deps/usr/share/luajit-2.1.0-beta3/fzy.lua'
        no file '/usr/local/share/lua/5.1/fzy.lua'
        no file '/usr/local/share/lua/5.1/fzy/init.lua'
        no file '/home/xxx/github/neovim/.deps/usr/share/lua/5.1/fzy.lua'
        no file '/home/xxx/github/neovim/.deps/usr/share/lua/5.1/fzy/init.lua'
        no file './fzy.so'
        no file '/usr/local/lib/lua/5.1/fzy.so'
        no file '/home/xxx/github/neovim/.deps/usr/lib/lua/5.1/fzy.so'
        no file '/usr/local/lib/lua/5.1/loadall.so'

I am not familiar with lua stuff.  intuitively, I feel that this is an env-related problem, but I don't know how to fix it. Please help.

It might be this issue:
https://github.com/wbthomason/packer.nvim/issues/180

I don't know exactly where to report this, but I'm getting the same error, even when using the solution provided on wbthomason/packer.nvim#180. Looking at the error message I can see that it's not searching locally for fzy. I was able to fix it either by running sudo luarocks install fzy or copying those fzy files to the same directory as all.lua file. Am I missing something here? Should it be able to look fook user specific local folder, so instead of /usr/local/* it would be ~/.local/*?


Generated Operation Id should follow nswag conventions

in nswag.config you can set the property
"operationGenerationMode": "MultipleClientsFromOperationId"
for this to work the operationId in your swagger.json need to have groupings defined by '_'
"operationId": "MyGroup1_Enpoint1"
"operationId": "MyGroup1_Enpoint2"
"operationId": "MyGroup2_Enpoint1"
"operationId": "MyGroup2_Enpoint2"
this will then create two generated clients IMyGroup1 and IMyGroup2
If you specific tags for your endpoints, it groups them in swagger ui but the swagger.json either has the full namespace or short name for the operationId and doesn't apply the convention needed for nswag to support the MultipleClientsFromOperationId

Starting with v<IP_ADDRESS>-beta you can do the following:
//enable the setting
app.UseFastEndpoints(c => c.Endpoints.PrefixNameWithFirstTag = true)

//specify a tag for the endpoint
sealed class SayHelloEndpoint : Endpoint<Request, Response>
{
    public override void Configure()
    {
        Post("/api/hello");
        Tags("MyTag");
    }
}

The Swagger Operation Id would then be in the desired format:
"paths": {
    "/api/hello": {
        "post": {
            "operationId": "MyTag_SayHelloEndpoint",
        }
    }
}

The other alternative would be to manually specify the Endpoint Name (Operaiton Id) like this.

Exactly what I was looking for, thanks for the response.

The following looks to be generating multiple clients but they are all put into a single file. I have not been able to figure out how to get NSwag to spit out the clients into separate files. Not sure if it's even supported by NSwag.
await app.GenerateClientsAndExitAsync(
    documentName: "v1",
    destinationPath: bld.Environment.WebRootPath,
    csSettings: c =>
    {
        //this seems to be the default setting anyway
        c.OperationNameGenerator = new MultipleClientsFromOperationIdOperationNameGenerator();
    },
    tsSettings: null);


Gulp hotfixing

Follow-up on #862. This is still more of a hotfix, a gulpfile overhaul is still highlt desirable.
This needs both extensiv review and sanity checks!
Short summary of what has changed:

gulp serve and gulp serve:browsersync don’t imply a rebuild anymore.
gulp default (and therefore gulp) only build the library, not the microsite or other secondary artifacts.
gulp all is like the old default task, builds everything.
gulp styles does not imply gulp templatestyles anymore.
gulp zip builds both dist/mdl.zip and dist/mdl-templates.zip.
gulp publish:bower

runs gulp _release which builds a _release folder containing the library build as well as src/ together with gulpfile.js. If you run npm install in _release, you can gulp build your own MDL.
Uses gulp-subtree to push the new release to the release branch on origin.

r: @addyosmani @Garbee @crhym3

drone seems happy, so :+1:

My only concern is the publish:bower naming. What exactly with this command is specific to bower? Looks like it would be more aptly named publish:release.

I thought that the release branch was specific to bower. But you are right, even if that’s the case, your name is still superior. Will fix.

LGTM

Branch requires some rebasing against master.

gulp publish:release

runs gulp _release which builds a _release folder containing the library build as well as src/ together with gulpfile.js. If you run npm install in _release, you can gulp build your own MDL.
Uses gulp-subtree to push the new release to the release branch on origin.

Nice work, @surma!

The rebase remove the commit ae7c6e0f2ca6c82cd9cbdc73195444dfa36854e8 we are still having a discussion on. Adding this comment to link to it.

@addyosmani @Garbee Updated the watch directives. Did a few tests and everything seems to work fine.
PTAL

Re-testing now. Thanks for the updates!

gulp zip appears to output the same package for both the library and template builds:

I recall the old template builds being ~10MB+ so we might want to double check we're still correctly building there.

lgtm