Added example implementation for Android <Portal> per #2501

We've been discussing whether to keep the portal implementation or ship a native module for showing modals instead.
In the Ads Manager we currently use the Portal to implement modals (https://gist.github.com/mkonicek/1a45b8f8c36bb710478c) but might deprecate that actually.
Would you mind holding with this until it's clear whether we'll go with the native module?

@mkonicek are you asking me?  I think this should be accepted as a PR or a decision should be made ASAP, there needs to be native modal support on Android.

The decision is we'd like to go with the native module to implement Modal on Android, there's the PR #5320 but still need to figure out touch handling.
I agree we should clear out the list of missing Android modules as quickly as we can: https://facebook.github.io/react-native/docs/known-issues.html, a few releases from now it should be gone except for push notifications for which we only have an implementation very tied to the fb infra and using it in the Ads Manager on Android.

@mkonicek We should probably document  since Modal is not going to be ready anytime soon.


UI for Nuage subnets

Adding UI elements to display the text for Nuage subnets under Nuage provider.
Also subnets are now mapped to network groups to be saved as part of the ManageIQ database
Under Networks -> Providers we get to see Nuage . However since runners for refresh parser aren't completed as yet, for now the Nuage Network Provider is added through the rails console and a manual EmsRefresh is done.the subsequent screenshots are for traversing withing the Nuage Provider till we reach the summary of the cloud subnets page.

@samuel-nuage can you add some screenshots to your initial comment?  It gives the reviewers an idea of what got added and where and helps document the PR.
@h-kataria please review

@dclarizio : will do

@blomquisg : Addressed your earlier commits in this PR

looks good to me :+1:
you are missing a Nuage icon though, it can come in next PR

@Ladas yeah, icon will be in the next PR

@Ladas is the removal of MiqAeServiceManageIQ_Providers_Nuage_NetworkManager_CloudNetwork correct here?  Do we not have a Nuage CloudNetwork yet?

@blomquisg : So we are using network group to save the inventory instead of a cloud_network.
But yeah, let @Ladas confirm it, just to be sure.

@samuel-nuage @blomquisg yes, that should be correct.
Though network_group is missing UI support currently.

@dclarizio if you or someone on the UI team can give this :+1:, then I'll merge it.  @Ladas is good with it.

@dclarizio Please let us know if changes are still required.


Unable to do a ZCash transaction anymore

Hi,
Having latest Nano S firmware, latest updated chrome app.
When trying to do any transaction with the zcash app opened, it leads to an error:

Sending Failed: Unable to validate transaction

After reading the logs, it seams there is an error somewhere:
Tue, 29 Aug 2017 17:12:15 GMT,INFO,Router,Routing to [/wallet/accounts/index#send]
Tue, 29 Aug 2017 17:12:15 GMT,INFO,Transaction,--- CREATE TRANSACTION ---
Tue, 29 Aug 2017 17:12:15 GMT,INFO,Transaction,Amount:  xxxx
Tue, 29 Aug 2017 17:12:15 GMT,INFO,Transaction,Fees:  xxxx
Tue, 29 Aug 2017 17:12:15 GMT,INFO,Transaction,Total send:  xxxx
Tue, 29 Aug 2017 17:12:15 GMT,INFO,Transaction,Address:  t1xxxxxxxx
Tue, 29 Aug 2017 17:12:15 GMT,INFO,Transaction,UTXO:  << stringify error: TypeError: Converting circular structure to JSON >>
Tue, 29 Aug 2017 17:12:15 GMT,INFO,Transaction,Change path:  44'/133'/0'/1/2
Tue, 29 Aug 2017 17:12:15 GMT,INFO,Transaction,Change address:  t1xxxxxxxxx
Tue, 29 Aug 2017 17:12:15 GMT,INFO,Transaction,Data:  undefined

Fixed, will be pushed in the next update


Create reverse_spiral_matrix.cpp

Hello, I have solved one GeeksforGeeks problem reverse spiral matrix. Please merge it with your repo under hackoctoberfest 2022.

Hey @Pyush146 thanks for your contribution ❗. Keep up the good job and leave a star if you liked the repo .
Happy hacking 😉


Add milestone entry for VistA, public domain healthcare software (early 1980s)

This seems to be a little-known project with a rich history - predating the term open source but also adapting to it as it arrived.  I've added an "official" reference to the VA's own website in the timeline entry here, but the following web resources may also be of interest:

The "hardhats" project on SourceForge (and the associated members mailing list)
History of VistA from the hardhats.org website

:white_check_mark: I've run csvlint to check that the file format remains valid (with the expected warning)

Test test, is this thing on?
@all-contributors, please add @jayaddison for data.

Thanks @juliaferraioli!


[TextField] Cast Error Null Value on Numeric TextFields

A Cast Error is appearing when you use a numeric keyboard on Numeric TextFields.
Steps to Reproduce

import 'package:flutter/material.dart';

class Prueba extends StatefulWidget {
  const Prueba({Key? key}) : super(key: key);

  @override
  State<Prueba> createState() => _PruebaState();
}

class _PruebaState extends State<Prueba> {

  final numController             = TextEditingController();

  @override
  Widget build(BuildContext context) {

    return Scaffold(
      appBar: AppBar(
        title: const Text('Test'),
      ),
      body: TextField(
        controller: numController,
        keyboardType: TextInputType.number
      )
    );
  }

Expected results:
With the Flutter update I started noticing that numeric text boxes get an error when they are empty.
I managed to reduce the code to the minimum expression to identify the source of the error and apparently it is at the moment of defining the keyboard (TextInputType.number)
Actual results:
When I tap the box to enter the number on an Android Physical Device, this is the error that appears on console:

════════ Exception caught by services library ════════
The following _CastError was thrown during method call TextInputClient.performPrivateCommand: Null check operator used on a null value
When the exception was thrown, this was the stack #0 EditableTextState.performPrivateCommand package:flutter/…/widgets/editable_text.dart:2155 #1 TextInput._handleTextInputInvocation package:flutter/…/services/text_input.dart:1821 #2 TextInput._loudlyHandleTextInputInvocation package:flutter/…/services/text_input.dart:1701 #3 MethodChannel._handleAsMethodCall package:flutter/…/services/platform_channel.dart:536 #4 MethodChannel.setMethodCallHandler. package:flutter/…/services/platform_channel.dart:529 #5 _DefaultBinaryMessenger.setMessageHandler. package:flutter/…/services/binding.dart:387 #6 _invoke2 (dart:ui/hooks.dart:186:13) #7 _ChannelCallbackRecord.invoke (dart:ui/channel_buffers.dart:42:5) #8 _Channel.push (dart:ui/channel_buffers.dart:132:31) #9 ChannelBuffers.push (dart:ui/channel_buffers.dart:329:17) #10 PlatformDispatcher._dispatchPlatformMessage (dart:ui/platform_dispatcher.dart:599:22) #11 _dispatchPlatformMessage (dart:ui/hooks.dart:89:31) call: MethodCall(TextInputClient.performPrivateCommand, [13, {data: {}, action: Normal_Mode}])

If I don't include a numeric keyboard, the error disappears.
@Chance included the following comments in StackOverflow link:

It was good to open an issue for this problem on the flutter channel, another detail that this problem appears on android and samsung devices have an error in 100% of the cases. In version 3.3 there are some very critical problems on the part of flutter, such as the pageview line 455 value being initialized null and causing white screen where there is tabview or pageview.
Line 2155 editable_text: widget.onAppPrivateCommand!(action, data);. Yet another case of a null statement that fails. Interim fix made in flutter code: if(widget.onAppPrivateCommand != null)widget.onAppPrivateCommand!(action, data);.

Hi @dlondonog, are you experiencing this on a Samsung device?
Please also provide the output of flutter doctor -v.
Thank you

+1

Hi @dlondonog, are you experiencing this on a Samsung device?
Please also provide the output of flutter doctor -v.
Thank you

[✓] Flutter (Channel stable, 3.3.2, on macOS 12.6 21G115 darwin-x64, locale es-419)
• Flutter version 3.3.2 on channel stable at /Users/dlondono/development/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision e3c29ec00c (6 days ago), 2022-09-14 08:46:55 -0500
• Engine revision a4ff2c53d8
• Dart version 2.18.1
• DevTools version 2.15.0
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at /Users/dlondono/Library/Android/sdk
• Platform android-33, build-tools 30.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 14.0)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 14A309
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2020.3)
• Android Studio at /Applications/Android Studio.app/Contents
• 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-7281165)
[✓] VS Code (version 1.71.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.48.0
[✓] Connected device (3 available)
• iPhone 14 Pro Max (mobile) • 3FC36180-5816-44D6-9517-C46C03DBAC80 • ios            •
com.apple.CoreSimulator.SimRuntime.iOS-16-0 (simulator)
• macOS (desktop)            • macos                                • darwin-x64     • macOS 12.6 21G115 darwin-x64
• Chrome (web)               • chrome                               • web-javascript • Google Chrome 105.0.5195.125
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!

Hi @dlondonog, what device(s) are you experiencing this issue on?

Hi @dlondonog, what device(s) are you experiencing this issue on?

I could identify the issue on Samsung (SM J260MU)...
[✓] Flutter (Channel stable, 3.3.2, on macOS 12.6 21G115 darwin-x64, locale es-419)
• Flutter version 3.3.2 on channel stable at /Users/dlondono/development/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision e3c29ec00c (6 days ago), 2022-09-14 08:46:55 -0500
• Engine revision a4ff2c53d8
• Dart version 2.18.1
• DevTools version 2.15.0
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at /Users/dlondono/Library/Android/sdk
• Platform android-33, build-tools 30.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 14.0)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 14A309
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2020.3)
• Android Studio at /Applications/Android Studio.app/Contents
• 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-7281165)
[✓] VS Code (version 1.71.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.48.0
[✓] Connected device (4 available)
• SM J260MU (mobile)         • e27c069a                             • android-arm    • Android 8.1.0 (API 27)
• iPhone 14 Pro Max (mobile) • 3FC36180-5816-44D6-9517-C46C03DBAC80 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-0
(simulator)
• macOS (desktop)            • macos                                • darwin-x64     • macOS 12.6 21G115 darwin-x64
• Chrome (web)               • chrome                               • web-javascript • Google Chrome 105.0.5195.125
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!

I'm also experiencing the same issue on a Samsung device (SM G950U)

Thanks for the info. There's a similar issue already open tracking this issue in https://github.com/flutter/flutter/issues/97634.
Kindly follow up in the linked issue for updates.
Closing from here as duplicate.
Thank you


[BUG] Embedding model input parameter support.

Describe the bug
openai service with version 1.0.0-beta.9 sdk
Embedding Model：text-embedding-ada-002
The Request param in sdk EmbeddingOptions do not support array[int] or array[array[int]] as OpenAI.
Python code with openai
client = AzureOpenAI(
azure_endpoint="YOUR_endpoint",
api_key="Your API key",
azure_deployment="Your deployment",
api_version="2024-02-01"
)
response = client.embeddings.create(
model="text-embedding-ada-002",
input=[[23123, 232,23, 4341]]
)
print(response)

Transferring this issue to azure-sdk-for-python as this looks to be Python code and not Java code.

The python code is just an example to show that openai python library support the array array input. But our java sdk do not support the input type

@alzimmermsft seems like this is an issue for Java. Do you want to transfer it back?

@alzimmermsft seems like this is an issue for Java. Do you want to transfer it back?

Yeah @xiangyan99, let's transfer this back to the Java repo. Thought it was specifically Python due to the sample code.


Add initial python RESTinstance version 1.3.0

Description of changes
Add python package RESTinstance in version 1.3.0 to default python packages.
Some of the robotframework libraries are already added and the RESTinstance module  is another part of that system.
Tested building of derivation with nix-build -A python3Packages.restinstance.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

The commit message should be python3Packages.restinstance: init at 1.3.0 or python310Packages.restinstance: init at 1.3.0 if you want to get CI build the package.


🚀 Added new constants, restarting Guild.ts.

Added new constants in src/constants, need to restart guild.ts with constructor (tmrw or later).  If someone wants to create an interface for this: https://discord.com/developers/docs/resources/guild
Im going to use it in the constructor for Guild.ts, don't have to do it. lmk if there are any issues

🚀 Added Guild.ts, constructor will be fixed tmrw.

Wow, thank you so much for the PR.

Do you want me to merge it tmrw?

You can merge it whenever you like, if you want to wait for others to review that's fine, but it should be ok to merge.

Sounds good.

Looks awesome, thanks for the PR.

🚀 Added new constants for Guild.ts, and I am almost finished with the constructor.


Resize code128 barcodes?

Does this library support resizing code128 barcodes? I didn't find any in GoDoc. Thanks!

Hi,
once you have generated a barcode (code128 or anything else), you can use the Scale function to resize it.
The scale function itself is a very simple approach but since all barcode implement the image interface all other golang libs which support image resizing should work.
I hope this helps you. Please let me know.

Thanks @boombuler . Yes the Scale() function worked.
code, _ := code128.Encode("1709462")
scaledCode, _ := barcode.Scale(code, 300, 100)
file, _ := os.Create("barcode.png")
defer file.Close()

png.Encode(file, scaledCode)

The built in function Scale adds white space both left and right
when the width cannot be reached (because the boundary box
is already reached by reaching the height limit).
But that white space cannot be removed anymore, because
one does not know how much was added by the Scale function.
Is that white space required? Can it be made optional?
The problem is that this white space makes it difficult
to align this images on other surfaces.

@nicolasfranck the scale function scales the barcode to the largest multiple of the original size and then add white space until the desired size is reached, if you need a more sophisticated approach you should use an imageprocessing lib to scale the barcode.

@boombuler as I thought. It is more "shaving" than "scaling" that I need to remove those borders ;-)


Plot generation: move to matplotlib, make output SVGs affected by CSS theme

It may seem a bit convoluted, but it works:
https://user-images.githubusercontent.com/14352721/201120995-a3bf61d5-e9f6-472f-ba4a-b150bfb8c9dd.mp4
Some more stuff is needed to merge this but we should be already at a pretty good point:

[x] move from pygal to matplotlib https://github.com/gbdev/pandocs/issues/449
[x] make the output SVG use the CSS variables instead of hardcoded colros
[ ] update requirements.txt
[ ] move the call to the plot generation from the renderer to the preproc step of mdbook, otherwise the output SVG files cannot be inline injected

We should consider just calling a python script, instead of having the single calls for every plot like it is now here. @ISSOtm opinions?

[ ] replace the two img tags with #import in the MBC5 article so the svg is actually injected in the HTML

Fixes https://github.com/gbdev/pandocs/issues/322

I might be missing something but why not use a Rust plotting library instead of calling a python script from the Rust renderer?
This way you can remove Python from the dependencies and maybe make the deployment easier.
The only reason I can think of is that Rust simply don't have a good plotting library with support for SVG's (but this looks like a good one - https://github.com/plotters-rs/plotters)

I swear I have seen a comment somewhere about using a Rust plotter instead to cut on the dependencies, suggesting https://github.com/plotters-rs/plotters, but I can't find it anymore.

The rationale for moving to a Rust library instead is to remove a contribution barrier, by using simpler setup instructions (no virtualenv) that work on all systems.
As for matplotlib itself, while you have a point that it is more popular, I can't say it's more standard—its documentation is very vague and confusing, and I have never seen two StackOverflow answers agreeing on a strategy to using it... :/
Besides, someone can always contribute a different setup later if they don't like the one we end up adopting, and it can use Python if they're willing to test their instructions. I don't think a move would be a blocker for that?

The rationale for moving to a Rust library instead is to remove a contribution barrier, by using simpler setup instructions (no virtualenv) that work on all systems.

Python is not mandatory to contribute, one can simply skip the plot generations. If someone desires can provide a plot generated with a rust program instead, too.

As for matplotlib itself, while you have a point that it is more popular, I can't say it's more standard—its documentation is very vague and confusing, and I have never seen two StackOverflow answers agreeing on a strategy to using it... :/

I don't see how this is a negative points, it's okay if you have a lot of ways to do something.

Besides, someone can always contribute a different setup later if they don't like the one we end up adopting, and it can use Python if they're willing to test their instructions. I don't think a move would be a blocker for that?

Did you check this PR? Imagine translating all the logic I added to parse SVGs to make them affected by CSS translated in Rust. I (personally) am not going to do that or ask anyone to go through that to contribute a plot. Python is super declarative and the matplotlib part is all self-declarative.

Spring has ended a few weeks ago, but pollen levels are still at a record high so it's still spring cleaning time if you ask me. Rebased (to check for any merge conflicts). Re-reading the arguments, sure, let's go with Python. We can consider moving to a Rust lib if at all later, that bikeshedding should not block this improvement.
We can't use {{#include}} tags for this without adding a new preproc step, because the "links" preproc is what processes these tags. Our custom preproc runs after it so that it can process transcluded fragments.
A possibility is making our custom preproc process e.g. {{#plot}} tags, which would invoke the Python script and replace the tag with the script's output. Then I think we can have the best of both worlds!


Create a streaming module for Monix over JDBC - combined approach

Fixes #1228
A PR for this issue has already been submitted by @cemelo which is likely much more complete than this work, but I decided to post what I have, just for sake of discussion. This is based on @oleg-py's great work on quill-monix here.
Let's compare and contrast this with @cemelo's great work and figure out what to do.

[ ] Unit test all changes
[ ] Update README.md if applicable
[x] Add [WIP] to the pull request title if it's work in progress
[ ] Squash commits that aren't meaningful changes
[ ] Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

Removing WIP status as we are feature-complete

Thank you @pettyjamesm for the input so we could get this right from the start! If you want, please file an issue for auditing/query-tracking support. I'd love to hear more of your thoughts on this.

I don't understand why coverage is still at 87%. This thing is odd.

Shoot! I forgot to add quill-monix to modues. Adding #1270 to remedy that.

Thank you @deusaquilus for working on this. It will be super helpful to us.


All example utils moved to dff

[ ]  renaming

_example_utils -> utils/testing/
index -> common

Will we have other types of utils (not testing) later?


[RFC] [Proposal] support cgroups in the host

Description of the problem
Unlike other runtimes, kata starts at least 2 processes per POD, kata-shim and a hypervisor (and kata-proxy when vsocks is disable). From host perspective, containers run in a guest (virtual machine) whose resources can be monitored by monitoring the hypervisor process and its threads. kata-shims are used to interact with the containers, sending signals, reading/writing data, etc. A POD can have any number of containers that run in the same guest. One of the major gaps that can be found in kata containers currently is that cgroups are applied in the guest, but not in the host or not in the right cgroup path (see https://github.com/kata-containers/runtime/issues/1021) causing problem in some implementations for resource governance, like service fabric https://github.com/Microsoft/service-fabric/.
Proposal
Move kata-shim (pause workload) and hypervisor (and kata-proxy if vsocks is disable) to sandbox cgroup, and apply a cgroup constraint to the sandbox that is defined by the sum of all its containers' cgroups. Container's cgroups are still created and applied but just to the kata-shim that interact with the container. 2 examples:
A pod with 2 containers,  container A with 1 cpu and container B without cpu constraints (1 vcpu is hotplugged).

in the host: no cpu constraints for container B and sandbox, 1 cpu for container A.
in the guest: no cpu constraints for container B, 1 cpu for container A.

A pod with 2 containers, container A with 1 cpu and container B with 2 cpus (3 vcpus are hotplugged).

cgroup directory hierarchy
/sys/fs/cgroup/cpu,cpuacct
└── kubepods/burstable/pod644e04f3
      ├── crio-89e60b84  # Sandbox cgroupsPath=/kubepods/burstable/pod644e04f3/crio-89e60b84
      ├── crio-2a95e0f8  # Container A cgroupsPath=/kubepods/burstable/pod644e04f3/crio-2a95e0f8
      └── crio-d037bce7  # Container B cgroupsPath=/kubepods/burstable/pod644e04f3/crio-d037bce7

cc @WeiZhang555 @mocolic

@devimc Thanks for getting this done. Do you know in which version will this be released?

@mocolic 1.6.0-rc1 https://github.com/kata-containers/runtime/tree/1.6.0-rc1

@devimc Thanks! Do you know the timeline for this release?

@mocolic I don't,  cc @egernst

scheduled for 2019-03-20

@egernst One more question - do you maybe know if this is/could be ported back to 1.5?


Rename the _azure_TESTING_BUILD_AMQP compile definition to the more common _azure_TESTING_BUILD.

Bringing consistency across all C++ SDKs.
Fixes https://github.com/Azure/azure-sdk-for-cpp/issues/5402

@LarryOsterman is this good to merge?

@LarryOsterman is this good to merge?

Aren't we removing _azure_TESTING_BUILD until after the March azure core is released?

This PR is mainly about bringing consistency to the name being used across all projects.
I don't think we can remove this compiler definition without a lot of test changes.
The issue you mentioned is orthogonal (around recent virtual/final changes), and I am addressing that separately.

This PR is mainly about bringing consistency to the name being used across all projects.
I don't think we can remove this compiler definition without a lot of test changes.
The issue you mentioned is orthogonal (around recent virtual/final changes), and I am addressing that separately.

There is no negative consequence to keeping the name the same until we have a centrally defined definition. In fact, as I recall, the reason I added this as an eventhubs specific definition (even though it's named AMQP) was because we didn't have a central definition for tests.
If we had a central definition for tests, I would wholeheartedly allow the change. But until we have a central definition, these definitions are package specific. And as a package specific definition, the name should NOT collide with definitions from other packages.

Alright, I will close the issue as by design, won't fix.
It's good to codify this discussion as a guideline or source comment. It wasn't clear the per-package name difference was intentional.


[BUG] Occasionally VoiceChannel is None

Library Version
5.11.0
Describe the Bug
Occasionally, the built in cache receives None for VoiceChannel when trying to access _voice_member_ids. I believe this has to do with not having the VOICE_STATE_UPDATE intent.
Steps to Reproduce
Run an interactions.py bot with Sentry enabled and inspect the error log occasionally.
Expected Results
Ideally, the library isn't sending errors to Sentry :)
Minimal Reproducible Code
No response
Traceback
AttributeError: 'NoneType' object has no attribute '_voice_member_ids'
  File "interactions/client/smart_cache.py", line 767, in place_voice_state_data
    if user_id in old_state.channel._voice_member_ids:

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.
[X] I have attempted to debug this myself, and I believe this issue is with the library

Additional Information
No response

Ah, I see what's happening here.  Shouldn't be too hard to fix.


Create Shopping list_AU

Hi guys,
I love what you guys are doing. I have created my own lightning node with another guide and wanted to help anyone in Australia have an easier time following your guide by adding a no-brainer shopping list.
Keep it up

@DyptoDurrency the power outlet seems to be too weak with 2,5A. To run the HDD without power outages at least a 3A power outlet is recommended - even a bit more is better. Can you update the the list?


Custom palettes

Resolve #54
from localtileserver import get_leaflet_tile_layer
from localtileserver import examples
from ipyleaflet import Map

tile_client = examples.get_san_francisco()

palette = ["#006633", "#E5FFCC", "#662A00", "#D8D8D8", "#F5F5F5"]

t = get_leaflet_tile_layer(tile_client, band=1, palette=palette)

m = Map(center=tile_client.center(), zoom=9)
m.add_layer(t)
m

cc @giswqs

Codecov Report

Merging #55 (aa83d8b) into main (7e27db0) will increase coverage by 0.33%.
The diff coverage is 76.92%.

@@            Coverage Diff             @@
##             main      #55      +/-   ##
==========================================
+ Coverage   81.80%   82.13%   +0.33%
==========================================
  Files          19       19
  Lines         984      991       +7
==========================================
+ Hits          805      814       +9
+ Misses        179      177       -2

Looks great 🚀🚀


Getting-Started notebook failed with error

Strange enough, the getting started notebook failed. NO such table. The sqlite file downloaded has the table with "Artist" instead of the lower case error message.
Any thoughts on fixing it?

Thanks for reporting! This should be resolved now. The training data somehow got removed from this model.


subset errors

So @lqd and I had a big discussion today and resolved the major open questions with respect to reporting "subset errors". In short, we decided to adopt the approach laid out in the original smallcultfollowing blog post, but with a bunch of other details. =) This issue tracks the implementation of that work and explains the rationale.
The problem
Rust functions contain "placeholder origins"  like 'a and 'b in this example:
fn foo<'a, 'b>(x: &'a u32) -> &'b u32 { x }

We want this code to be an error. But the error is not due to an illegal access error -- it is because data with the origin 'a is returned with the origin 'b, and there is no 'a: 'b where clause.
Alternative A: Use subsets and declare errors
The original blog post proposed an approach to solving this problem where we looking for a subst relation subset('a, 'b, _) that involves two placeholder origins 'a and 'b, and then compare that against a set of "known subsets" (derived from the where clauses). If we find a subset relation that is not known to be valid, that's an error. This can (ultra naively) be expressed with a single rule:
subset_error(Origin1, Origin2) :-
    subset(Origin1, Origin2, _),
    placeholder_region(Origin1),
    placeholder_region(Origin2),
    !known_subset(Origin1, Origin2).

Alternative B
The main competitor that we were considering was the idea of using 'placeholder loans'. In this approach, for each placeholder origin, we have a corresponding placeholder loan. We can then compute the subset and contains relations as normal, and declare an error if a placeholder loan finds its way into a placeholder origin where it doesn't belong (i.e., in the above example, the placeholder origin for 'b would contain the loan for 'a). So the error rule would like something like this:
subset_error(Origin1, Origin2) :-
    contains(Origin2, Loan1, _),
    placeholder_origin(Origin1, Loan1),
    placeholder_origin(Origin2, Loan2),
    !known_subset(Origin1, Origin2).

but we also need some rules like
contains(Origin1, Loan1, Node) :-
    cfg_node(Node),
    placeholder_origin(Origin1, Loan1).

Consideration: Elegance
Naturally we want to find relatively minimal rules that are clean and easy to understand. The subset approach largely wins on this approach, but as we'll see later there are a few complications.
Consideration: Origins are just sets of loans, right?
One of the original motivations for pursuing the "placeholder loan" approach was that we wanted to be able to treat origins as if they were just sets of loans and didn't have an identity of their own. However, in practice, we can't quite do this anyway, because we have to track subset relations forward through the graph (see https://github.com/rust-lang/polonius/issues/107 for related discussion about that). So it seems like either way origins have some identity of their own in the polonius framework.
Consideration: Location Insensitive analysis
One of the things we were considering is the ability to optimize the naive and optimized analysis based on a preliminary location insensitive run. This location insensitive run will produce results that can be used to limit the transitive closure computations and make everything more performant.

Design we landed on
We ultimately landed on using the Approach A, as described here. First, the compiler produces the following input sets for polonius:
placeholder_origin(Origin)
known_subset(Origin1, Origin2) // for placeholders only

To optimize the naive rules using the outputs from the LI analysis, however, we can limit

contains only propagate those loans in L_Error
subset(O1, O2) to only propagate if

O1 and O2 are in O_Requited

Location insensitive (LI)
The location insensitive analysis as currently specified doesn't compute the transitive subset relation, only the transitive contains relation. So we add a new transitive analysis for subset_placeholder:
subset_placeholder(Origin1, Origin2) :-
    placeholder_origin(Origin1),
    subset_base(Origin1, Origin2).
subset_placeholder(Origin1, Origin3) :-
    subset_placeholder(Origin1, Origin2),
    subset_base(Origin2, Origin3).

then we can compute errors using subset_placeholder and the original rule given in Approach A.
This yields the following results:

the set of loans L_Error that are involved in (potential) illegal access errors
the set of placeholder origins O_Error that are involved in (potential) subset errors (each O where subset_error(O, _))
the set of origins O_Required which:

contain a loan in L_Error (using contains)
the O_Error regions
are a "superset" of some origin in O_Error (using subset_placeholder)

Naive, plus Naive optimized with Location Insensitive results
The naive analysis can be implemented using the rule originally given.
But we can also optimize the naive analysis based on the LI results:

only compute contains for loans in L_Error
only compute subset(O1, O2, N) if O1 and O2 are both in O_Required

Optimized
The "optimized" analysis avoids computing the transitive subset and contains relationships in full. Instead, it computes them only when necessary. In particular, when an origin goes dead, it will remove that origin, but also compute the transitive closure of subset relations and things that origin may have been involved in, so that if you have O1: O2: O3 and O2 goes dead, we can add a subset(O1, O3) tuple (we will also add any loans found in O2 to O3 at this time). This means that the optimized errors never have a complete picture of which loans belong to which origins, and we also don't know the full set of "superset origins".
This works out well for illegal access errors: for  that purpose, we only care which loans are live, we don't care which origin they are found in. So we don't need to know those full sets. Also, this can be optimized using LI results in the same fashion as naive.
However, for subset errors, this is a problem. We can't just lean on the subset relation because it is incomplete. (Similarly, if we were using Alternative B, we couldn't just rely on the contains relation to contain the placeholder loans.)
Therefore, we choose to build up a separate, transitive relation that propagates placeholder origins forward:
subset_placeholder(Origin1, Origin2, Node) :-
    placeholder_origin(Origin1),
    subset(Origin1, Origin2, Node).
subset_placeholder(Origin1, Origin3, Node) :-
    subset_placeholder(Origin1, Origin2),
    subset(Origin2, Origin3, Node).

For efficiency, this relation can be filtered as well. We only need to consider the placeholder origins in O_Error, and we only need to propagate them into origins that are members of O_Required.


fix(ui): passes onSave to DocumentDrawer in multi-value label for relationship field

Description
V2 PR here

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] Existing test suite passes locally with my changes

Needs tests

It's all coming back to me now, the whole reason this was originally commented out in the first place is because React Select unmounts/mounts the entire drawer component when nested within the hasMany pill of the relationship field. This leads to the drawer not automatically closing in these scenarios, causing UI inconsistencies between relationship types, because this is not the case for monomorphic relationships. The decision was made to simply disable the functionality altogether in favor of consistency, unless a real fix can be discovered.

Just to be clear, it is the intermediary click that is required when the drawer closes after updating that you are referring to?
https://github.com/user-attachments/assets/2d8684b6-9121-430a-91a4-8afb21556dac


MDATP Threat Detection PowerBI Report

 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.:x: JesseEsquivel sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Is that possible to support vrvideo on mobile with Google Cardboard?

Is that possible to support Google Cardboard on mobile safari or mobile chrome?

In order to make it work on mobile we would need to use only canvas to build up the 360 scene, but last time I checked my current solution was too slow for that. I have some ideias to improve that so we can bring this experience to mobile web as well. About the Google Card board I still didn't get my hands on so I can't talk much, but it could be something interesting to play with.

Are you using three.js and WebGL? Like
renderer = new THREE.WebGLRenderer();

instead of using Canvas? What is your method? Thanks!

Yes, I use the WebGLRenderer, when using the canvas renderer things get slow. Right now I split the current source into 6 frames(one for each angle) and store them in a temporary canvas, them I transform it in a cube's texture, so basically each angle is a texture in a cube. Once its playing I transform the cube into a sphere and put the user perspective inside it.

Hi!,
I have tried your plugin:
http://opta360.com/360player-upgrade/
but video is not load...any idea what the issue is...

kaleidoscope supports android and iphone, but to bring the iphone support to this clappr plugin would require a bit more work, I'll not look into it now because iOS 10 will have support for inline videos so we can wait for it and have this implemented for free without adding complexity to the plugin


Missing verfication output on Windows

On Linux (using gpgsm) i am used to seeing its output inside the output of git log --show-signature or git tag -v. With smimesign i do not get such output and can not tell a regular commit from a signed one ... did not try invalid signature.
git log --pretty="format:%h %G? %aN  %s" gives me the expected output, so the verification itself is working.
Maybe related to #41 and the output of smimesign going to the wrong fd.

The output i am talking about looks a bit like what you can find here, https://git-scm.com/book/en/v2/Git-Tools-Signing-Your-Work
all the gpg:  lines


cmd/go: 'go install' reports GOBIN not set when installing .go source file

What version of Go are you using (go version)?
go version go1.10.2 darwin/amd64
Does this issue reproduce with the latest release?
unknown.
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN="/Users/xxx/go/bin"
GOCACHE="/Users/xxx/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/xxx/go"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/hz/mqzhd2cj5cj48knw7wkywpnh0000gn/T/go-build293557160=/tmp/go-build -gno-record-gcc-switches -fno-common"
Above I have replaced my username with xxx
What did you do?
Very simple. I am trying to do "go install" on github.com/golang-migrate/migrate/migrate.go
My env vars are correctly exported (since years).
What I have trided:
$ cd ~/go/src/github.com/golang-migrate/migrate/
$ go install migrate.go
go install: no install location for .go files listed on command line (GOBIN not set)
$ echo $GOBIN
/Users/xxx/go/bin
$ ls $GOBIN
long list of installed go binaries.
$ echo $GOPATH
/Users/xxx/go
$ export GOBIN=$GOPATH/bin
$ echo $GOBIN
/Users/xxx/go/bin
$ go install migrate.go
go install: no install location for .go files listed on command line (GOBIN not set)
What did you expect to see?
Expected it to install migrate
What did you see instead?
go install: no install location for .go files listed on command line (GOBIN not set)
when GOBIN IS set, exported, exists, and has the correct ownership and permissions.

Is there a work around? i.e. a way to install go programs?

go install github.com/golang-migrate/migrate  works though, plus they provide a separate Makefile for the development purposes.

@nutmix I can't reproduce this and I don't know how it ever would have worked.
For the past ~two years, github.com/golang-migrate/migrate/migrate.go has relied upon a type defined in another file within the package:
https://github.com/golang-migrate/migrate/blob/418d41c2a18a96c5ed7f69e350531e450eaaf27d/migrate.go#L60
That means that it cannot be built from command-line file arguments, unless you list all of the non-test .go files, and that does work.
_gopath$ export GO111MODULE=off
_gopath$ go1.12beta1 get github.com/golang-migrate/migrate
_gopath$ cd src/github.com/golang-migrate/migrate/

_gopath/src/github.com/golang-migrate/migrate$ go1.12beta1 install migrate.go
# command-line-arguments
./migrate.go:60:6: undefined: Logger

_gopath/src/github.com/golang-migrate/migrate$ go1.12beta1 install $(ls *.go | grep -v '_test.go')

_gopath/src/github.com/golang-migrate/migrate$ export GOBIN=$GOPATH/bin

_gopath/src/github.com/golang-migrate/migrate$ go1.12beta1 install $(ls *.go | grep -v '_test.go')

_gopath/src/github.com/golang-migrate/migrate$

Please give a concrete list of commands that we can run exactly, starting from an empty GOPATH, to reproduce the problem.


Big white rectangle while dragging a tab or a file (Ubuntu/LXDE)

There is a strange rendering issue while dragging a tab or a file. The issue occurs since version 1.2.2 (also in 1.3.0-beta4), on Ubuntu 14.04 with LXDE, 4K hidpi Dell monitor (3840x2160), GeForce GTX 660 with nVidia drivers (also in safe mode):

The rectangle gets larger and larger while moving the mouse pointer.

I suspect that the issue may be related to the fact that LXDE does not support transparency. The dragging creates a new window which cannot be semi-transparent because LXDE does not support it. So it is white instead.
Also, the dragged tab or file appears too far to the right and bottom (quite far from the mouse pointer). The coordinates of the dragged file/tab are exactly equal to 2 * [mouse_x, mouse_y] on my hidpi monitor.

Same problem on gnome 3.18.
I also have the same problem in chrome 46, while dragging anything in the browser (images, text)

@simurai Pretty sure you dealt with a similiar issue recently.

The issue can be reproduced on any Linux using:
atom --force-device-scale-factor=2

I have observed the big white rectangle issue only on LXDE, but the problem of wrong positioning (in relation to the mouse pointer) occurs on every Linux desktop environment (e.g. Linux Mint/Cinnamon).
But, as @fiws has written, it looks like it is partly a chromium (version 46) issue (dragged text or image is misplaced in relation to the mouse pointer, no white rectangle problem).

This sounds similar to https://github.com/atom/atom-dark-ui/issues/47.

@pgkos I suspect that the issue may be related to the fact that LXDE does not support transparency.

Right, in the issue mentioned above, the "solution" was to try to restyle it so there is no transparency. Like adding a solid background-color.

Is not just dragging but also while moving around:

I can't reproduce this bug anymore: Ubuntu 16.04.3 LTS
Atom    : 1.23.3
Electron: 1.6.15
Chrome  : 56.0.2924.87
Node    : 7.4.0


Use github actions to publish to pypi

ping @rm-hull

double ping @rm-hull

pypi tokens added to repo secrets also


docker-compose not working adding ports with each start

Starting container new with docker-compose will result in port added with each start.
artifacts-nuget | 2021/03/05 11:00:11 [emerg] 32#32: invalid port in "808080" of the "listen" directive in /etc/nginx/conf.d/nuget.conf:6
artifacts-nuget | nginx: [emerg] invalid port in "808080" of the "listen" directive in /etc/nginx/conf.d/nuget.conf:6

artifacts-nuget | 2021/03/05 11:00:26 [emerg] 30#30: invalid port in "8080808080" of the "listen" directive in /etc/nginx/conf.d/nuget.conf:6
artifacts-nuget | nginx: [emerg] invalid port in "8080808080" of the "listen" directive in /etc/nginx/conf.d/nuget.conf:6

Next and so on...

related to #9
issue is line #46 in docker-entrypoint
    sed -i -e "s/listen 80/listen ${SERVER_PORT}/" /etc/nginx/conf.d/nuget.conf

should be
    sed -i -e "s/listen.*/listen ${SERVER_PORT}/" /etc/nginx/conf.d/nuget.conf

@blindzero thx.


feat(ci/cd): action to check compilation

🦙 MegaLinter status: ✅ SUCCESS

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ CPP
clang-format
4
0
0
0.04s

✅ CPP
cpplint
4

0
0.16s

See detailed report in MegaLinter reports
MegaLinter is graciously provided by


Loop out issue on raspiblitz

Hi, I'm very new to the lightning space. I want to learn more about it's possibilities. Thats why I started a full node on a raspiblitz.
After set up I opened a lightning channel via Thunderhub. To gain some inbound liquidity I tryed to loop out some sats to remote via Lightning Terminal but I got an error message (see below).

I already deleted everything but the blockchain on the raspiblitz and set it up again, but the issue still excists.
Has anybody an idea, what I can do now?

What version of Loop are you running? This issue should be fixed in the latest version (0.13.0-beta). Are you sure your channel is operational and online?

Ah okay. I'm on
"lncli version 0.12.1-beta commit=lightning-terminal-v0.4.1-alpha"
I installed it via the terminal and the ssh of the raspiblitz. When I download the latest version and run it in terminal it doesn't work. Can you explain how I upgrade it now to the new version?

Okay, this currently is the latest version of LiT. Can you try removing the lsat.token.pending file somewhere in the /home/lit/ directory? You can use find /home/lit -name lsat.token.pending to find it.

It says, there is no such file or directory

Then the command isn't correct. I'm not too familiar with the setup of RaspiBlitz. Try find / -name lsat.token.pending.

That worked. But every directory says permission denied. BTW I'm on Mac. Maybe that's the problem?

It depends on which user you are logged in on your RaspiBlitz. You might need to get some help from the RaspiBlitz community (as I mentioned, I'm not sure how the users/directories are set up on those devices). The goal is to find the lsat.token.pending file somewhere in LiT's user directory and remove it.

Just delete the file on the file system and try again.

It still doesn't work. Now I get another error

Okay, you got one step further. Is your graph fully synced? Check lncli getinfo, it needs to say synced_to_graph: true. This seems to be a common issue with Umbrel, you might need to get some help from them.

"synced_to_chain": true,
"synced_to_graph": false,
It's not fully synced

Yes, so you have to wait until your node knows the whole network graph. Otherwise it won't know how to send a payment to the Loop server and you get the no_route error. This can take a while. If it takes more than half a day or so, you should probably open an issue in the Umbrel repo. It sounds to me like their switch from Neutrino to bitcoind doesn't work properly in all cases and the --routing.assumechansvalid flag isn't set properly.

Great, good to hear. Whether or not the loop goes through now fully depends on your channel liquidity and the quality of your peers. It might also take a few attempts.
I'm going to close the issue as the main problem seems to be resolved.


Does it support the same functionality as ServerTransportFilter in grpc?

I hope KeepAliveHandler can be configured.

Hi!
Could you provide a bit more detailed explanation of what you would like to do with it?


set_activity blocks when discord is not running

The examples do not show how to detect if discord is running, and I found no way to detect it.
When I call set_activity  and discord is not running on the PC it will never return.
Did I miss something, or is this a bug?

@Kubikx This is just a mirror repository. The actual repository where you can open issues is here: https://gitlab.com/valeth/discord-rpc-client.rs

thanks for the link.
unfortunatelly it seem the GitLab repository is dead :(
https://gitlab.com/valeth/discord-rpc-client.rs/issues/6 same issue was opened Opened 6 months ago by Christopher Serr and there was no activity since :(

The project is currently unmaintained.
There was someone that wanted to maintain it, but there wasn't anything contributed to it since.

is there any alternative to have optional rich presence in rust application?


charmap error code

Hello I am getting this error.. can u please help .
Traceback (most recent call last):
File "C:\Users\elios\Desktop\srt-translator\run.py", line 25, in
results=Translate(input_filepath=file_to_translate,
File "C:\Users\elios\Desktop\srt-translator\code\translate.py", line 8, in Translate
aggregate_sentences_dict=AggregateSentences(in_file)
File "C:\Users\elios\Desktop\srt-translator\code\helpers.py", line 16, in AggregateSentences
for line in in_file:
File "C:\Users\elios\AppData\Local\Programs\Python\Python39\lib\encodings\cp1252.py", line 23, in decode
return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x90 in position 670: character maps to

Oops, looks like I am just now seeing this. My apologies there. The package has been updated multiple times since this was posted, so I hope this is now resolved. Closing it now. Feel free to reopen if it is still an issue with the newer package.


Remove release-go-functions- from the docker image tag

/assign @prachirp

Can you comment the file with more explanations? This file is confusing if I were to revisit it.

Done


Expose Headers Location

Hi,
I've been follow this tutorial:
https://codereviewvideos.com/course/symfony-3-with-reactjs-and-angular/video/simple-symfony-3-restful-api-setup
To integrate FOS REST with Angular, backend with frontend.
Just don't installed the APIdoc bundle
A have a post action, which save and object and after, return a redirect view to get that object. The fos rest bundle responses in the header a key Location (when I am doing by curl -verbose I see this key), but using restangular I didn't get any content from header...
This is my configuration:
nelmio_cors:
    defaults:
        allow_credentials: false
        allow_origin: []
        allow_headers: []
        allow_methods: []
        expose_headers: []
        max_age: 0
        hosts: []
        origin_regex: false
        forced_allow_origin_value: ~
    paths:
        '^/api/':
            allow_origin: ['*']
            allow_headers: ['X-Custom-Auth']
            allow_methods: ['POST', 'PUT', 'GET', 'DELETE']
            expose_headers: ["Location"] # this being the important line
            max_age: 3600
        '^/':
            origin_regex: true
            allow_origin: ['^http://localhost:[0-9]+']
            allow_headers: ['X-Custom-Auth']
            allow_methods: ['POST', 'PUT', 'GET', 'DELETE']
            max_age: 3600
            hosts: ['^api\.']

I am doing anything wrong ?
Thanks in advance.

Hi there in 2021, just met the same case, that expose_headers doesn't work for single path entry


请问什么时候padpro协议可以恢复登录？

Important：Please file the issue follow the template, or we won't help you to solve the problem.

0. Report Issue Guide

Please run the following command and check whether the problem has been fixed：

rm -rf package-lock.json
rm -rf node_modules
npm install

Please search in FAQ List first, and make sure your problem has not been solved before.

Please search in the issue first, and make sure your problem had not been reported before

1. Versions

What is your wechaty version?
Answer: 0.27.60

What is your wechaty-puppet-padpro version?
Answer: 0.3.21

What is your node version? (run node --version)
Answer: v11.12.0

What os are you using
Answer: macOSx 10.14.5

2. Describe the bug
Give a clear and concise description of what the bug is.
使用付费的token与相应的环境变量启动starter后扫码无响应；
手机端不显示已经登录，命令行卡在qrCodeLogin那句日志就不再有输出了。
3. To Reproduce
This part is very important: if you can not provide any reproduce steps, then the problem will be very hard to be recognized.
Steps to reproduce the behavior:

git clone https://github.com/wechaty/wechaty-getting-start
cd wechaty-getting-start
npm install -g wechaty
npm install -g wechaty-puppet-padpro
export WECHATY_PUPPET_PADPRO_TOKEN=$MY_TOKEN;
6.export WECHATY_PUPPET=wechaty-puppet-padpro
7.npm start
8.使用手机扫码登录。

4. Expected behavior
Give a clear and concise description of what you expected to happen.
5. Actual behavior
If applicable, add screenshots to help explain your problem. But do not paste log screenshots here. (Pleas do not just paste a log image)
6. Full Output Logs
Set env WECHATY_LOG=silly in order to set log level to silly, then we can get the full log (If you dosen't set log env, log level is info as default, we cannot get the full log)
We need full log instead of log screenshot or log fragments!

Show Logs

$ WECHATY_LOG=silly node yourbot.js

Question: Paste your FULL(DO NOT ONLY PROVIDE FRAGMENTS) log messages
Answer:

7. Additional context
Add any other context about the problem here.
[bug]

同样的问题

同样问题调试了好久无反应

同样的问题

同样的问题


unrecognized import path "vbom.ml/util"

Bug Report
When running make install, the execution fails with the following trace
#> make install
which: no controller-gen in (/home/spredzy/Projects/virtualenvs/product-docs/bin:/home/spredzy/Projects/virtualenvs/product-docs/bin:/home/spredzy/.local/bin:/home/spredzy/bin:/bin:/usr/bin:/usr/local/bin:/usr/local/sbin:/usr/sbin:/home/spredzy/.fzf/bin:/home/spredzy/bin:/home/spredzy/bin)
which: no kustomize in (/home/spredzy/Projects/virtualenvs/product-docs/bin:/home/spredzy/Projects/virtualenvs/product-docs/bin:/home/spredzy/.local/bin:/home/spredzy/bin:/bin:/usr/bin:/usr/local/bin:/usr/local/sbin:/usr/sbin:/home/spredzy/.fzf/bin:/home/spredzy/bin:/home/spredzy/bin)
Makefile:125: warning: overriding recipe for target 'test'
Makefile:30: warning: ignoring old recipe for target 'test'
go: creating new go.mod: module tmp
go: found sigs.k8s.io/controller-tools/cmd/controller-gen in sigs.k8s.io/controller-tools v0.3.0
/home/spredzy/go/bin/controller-gen "crd:trivialVersions=true" rbac:roleName=manager-role webhook paths="./..." output:crd:artifacts:config=config/crd/bases
go: creating new go.mod: module tmp
go get<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>unrecognized import path "vbom.ml/util": https fetch: Get "https://vbom.ml/util?go-get=1": dial tcp: lookup vbom.ml on [fd0f:ee:b0::1]:53: no such host
make: *** [Makefile:96: kustomize] Error 1

I have found this issue opened 5 days ago https://github.com/kubernetes/kubectl/issues/925, and the corresponding fix merged a day later https://github.com/kubernetes/kubernetes/pull/94451
I suspect this will need to be brought up to the kustomize team so they update their kubectl deps, and the operator-sdk team will in turn update their kustomize version requirements.
What did you do?
Followed the official documentation (https://sdk.operatorframework.io/docs/building-operators/golang/quickstart/) until the make install step
What did you expect to see?
Not an error.
What did you see instead? Under which circumstances?
#> make install
which: no controller-gen in (/home/spredzy/Projects/virtualenvs/product-docs/bin:/home/spredzy/Projects/virtualenvs/product-docs/bin:/home/spredzy/.local/bin:/home/spredzy/bin:/bin:/usr/bin:/usr/local/bin:/usr/local/sbin:/usr/sbin:/home/spredzy/.fzf/bin:/home/spredzy/bin:/home/spredzy/bin)
which: no kustomize in (/home/spredzy/Projects/virtualenvs/product-docs/bin:/home/spredzy/Projects/virtualenvs/product-docs/bin:/home/spredzy/.local/bin:/home/spredzy/bin:/bin:/usr/bin:/usr/local/bin:/usr/local/sbin:/usr/sbin:/home/spredzy/.fzf/bin:/home/spredzy/bin:/home/spredzy/bin)
Makefile:125: warning: overriding recipe for target 'test'
Makefile:30: warning: ignoring old recipe for target 'test'
go: creating new go.mod: module tmp
go: found sigs.k8s.io/controller-tools/cmd/controller-gen in sigs.k8s.io/controller-tools v0.3.0
/home/spredzy/go/bin/controller-gen "crd:trivialVersions=true" rbac:roleName=manager-role webhook paths="./..." output:crd:artifacts:config=config/crd/bases
go: creating new go.mod: module tmp
go get<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>unrecognized import path "vbom.ml/util": https fetch: Get "https://vbom.ml/util?go-get=1": dial tcp: lookup vbom.ml on [fd0f:ee:b0::1]:53: no such host
make: *** [Makefile:96: kustomize] Error 1

Environment
Operator type:
/language go
Kubernetes cluster type:
N/A
$ operator-sdk version
operator-sdk version: "v1.0.0", commit: "d7d5e0cd6cf5468bb66e0849f08fda5bf557f4fa", kubernetes version: "v1.18.2", go version: "go1.13.11 linux/amd64", GOOS: "linux", GOARCH: "amd64"

$ go version (if language is Go)
go version go1.14.6 linux/amd64

$ kubectl version
N/A
Possible Solution
Ping the team in charge of kustomize so they update their requirements and the operator-sdk team can in turn update theirs.
Additional context
https://github.com/kubernetes/kubectl/issues/925

HI @Spredzy,
Also, I am unable to reproduce your issue.
The go version required is 1.13.x. Could you please check it with this version?
Also, by looking :
Makefile:125: warning: overriding recipe for target 'test'
Makefile:30: warning: ignoring old recipe for target 'test'

Shows that instead of you replace the test target as described in the doc you added it. Could you please check?

The go version required is 1.13.x. Could you please check it with this version?

Using this version make the make install pass. Thanks @camilamacedo86 for quick answer. Closing.

Really thank you for let us know.

I'm curious why not to update to go 1.14? is there any specific reason?

FWIW: I think this only succeeds on 1.13 because, from release notes: "As of Go 1.13, the go command by default downloads and authenticates modules using the Go module mirror and Go checksum database run by Google."
So for now, successful builds are depending on that continuing to be available in Google's module cache or some other cache. It may work, but it's a dependency to be aware of.

So this issue isn't go version dependent. It is GOPROXY dependent.
I just cleaned my local modcache, set GOPROXY=direct and attempted to compile from source with go 1.13, 1.14, and 1.15. All fail with:
$ GOPROXY=direct make install
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>unrecognized import path "vbom.ml/util" (https
fetch: Get https://vbom.ml/util?go-get=1: dial tcp: lookup vbom.ml: No address associated with
hostname)

Setting GOPROXY=https://proxy.golang.org,direct gets things working for all go versions since 1.13.
Different distros and installation methods set the default GOPROXY differently. See discussion here.
So to build SDK from source, it is currently required to set GOPROXY to include a proxy that serves the now-defunct vbom.ml/util package. To be fair, one of the stated goals of module proxies is to solve this exact problem.
From the proxy.golang.com FAQ:

I removed a bad release from my repository but it still appears in the mirror, what should I do?
Whenever possible, the mirror aims to cache content in order to avoid breaking builds for people that depend on your package, so this bad release may still be available in the mirror even if it is not available at the origin. The same situation applies if you delete your entire repository. We suggest creating a new version and encouraging people to use that one instead.

For the real solution, see https://github.com/kubernetes/kubectl/issues/925
We can likely use a replace to fix this until this fix propagates through all of our dependencies:
replace vbom.ml/util => github.com/fvbommel/util v0.0.0-20160121211510-db5cfe13f5cc

Re-opening so we can keep track that we need to fix this.

I reached out to the maintainer, and he was able to get the redirect up and running again for vbom.ml/util. This problem should be resolved for now, and the new import path should be picked up naturally whenever we rebase onto k8s 1.20.

Given @mhrivnak's comment, this is resolved.

So this issue isn't go version dependent. It is GOPROXY dependent.
I just cleaned my local modcache, set GOPROXY=direct and attempted to compile from source with go 1.13, 1.14, and 1.15. All fail with:
$ GOPROXY=direct make install
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>unrecognized import path "vbom.ml/util" (https
fetch: Get https://vbom.ml/util?go-get=1: dial tcp: lookup vbom.ml: No address associated with
hostname)

Setting GOPROXY=https://proxy.golang.org,direct gets things working for all go versions since 1.13.
Different distros and installation methods set the default GOPROXY differently. See discussion here.
So to build SDK from source, it is currently required to set GOPROXY to include a proxy that serves the now-defunct vbom.ml/util package. To be fair, one of the stated goals of module proxies is to solve this exact problem.
From the proxy.golang.com FAQ:

I removed a bad release from my repository but it still appears in the mirror, what should I do?
Whenever possible, the mirror aims to cache content in order to avoid breaking builds for people that depend on your package, so this bad release may still be available in the mirror even if it is not available at the origin. The same situation applies if you delete your entire repository. We suggest creating a new version and encouraging people to use that one instead.

For the real solution, see kubernetes/kubectl#925
We can likely use a replace to fix this until this fix propagates through all of our dependencies:
replace vbom.ml/util => github.com/fvbommel/util v0.0.0-20160121211510-db5cfe13f5cc

Re-opening so we can keep track that we need to fix this.

it's useful


Removing Laravel name from README.md

The purpose of this PR is to remove Laravel name from README.md as it is not related to features library would provide. Please provide a PR cause most of the developers who develop enterprise application may be concerned using this library. I'm not going to start a framework war again but IMHO for the library, it would be better to apply this patch.

How about:

PHP is great. Composer, PHP-FIG, and all the different frameworks make it even greater.

?

Done.


add link in a row

Griddle version
"griddle-react": "^1.13.1"
Expected Behavior
click on the row (tr) and redirect to another page
Actual Behavior
cannot add link in row
Steps to reproduce
How can we add link in the row of the table i.e. to redirect into different page
Pull request with failing test or storybook story with issue

This is covered here through custom columns or custom components. http://griddlegriddle.github.io/Griddle/docs/customization/


go-nyet deprecated?

go-nyet seems to be deprecated. Would you mind running gometalinter with this linter disabled by default. If yes, please let me know and I will prepare a pull-request.
Thanks.
https://github.com/barakmich/go-nyet#deprecated

Thanks for the heads up.
It seems like the right approach might be to disable go-nyet by default, and add a separate go vet -shadow linter, also disabled by default.
PS. Why did you close #27?

Ok, I will work on a PR.
PS. I closed the other issue because I used my work account by mistake, instead of my personal one :)

@ronnylt I needed that update so I just went ahead and implemented it. Feel free to review the code! Thanks.


Make API responses compatible with the JSON API Spec

The requests and responses could use the format described here: https://jsonapi.org/format/#fetching-resources-responses
We can use https://github.com/google/jsonapi for marshalling/unmarshalling.
Endpoints using GET/PUT/PATCH should return the affected resource.
refers to #321 #818

/cc @richard-cox @agracey

Considered done with 'gin'(#878)


Allow setting column's absolute width

how can I set sigle Cell's width

I've created a pull request to do that.
Enjoy :)
https://github.com/maxim-saplin/data_table_2/pull/49

I've created a pull request to do that. Enjoy :)
#49

ok

Added DataCoulm2.fixedWidth and pushed to pub.dev


can't custom db string -> enum property by IPropertyHandler<string, TEnum>

I define DbValueAttribute to assign mapping value in database, ex:
public enum SpeedEnum {
   [DbValue("(FX)")] High,     // database field value is (FX)
   [DbValue("(FY)")] Middle,  // database field value is (FY)
   [DbValue("(FZ)")] Low       // database field value is (FZ)
}

have generic PropertyHandler
public class DbValueEnumPropertyHandler<TEnum>: IPropertyHandler<string, TEnum?> {
   public TEnum? Get(string input, ClassProperty property) { .... }
   public TDbValue Get(TEnum? input, ClassProperty property) { .... }
}

unittest:
Assert.Equal(Connection.ExecuteQuery<string>($"select @a", new { a = SpeedEnum.Middle }).First(), "(FY)");    // success
Assert.Equal(Connection.ExecuteQuery<SpeedEnum>($"select N'(FY)'").First(), SpeedEnum.Middle);  // fail

exception is
System.InvalidOperationException
  HResult=0x80131509
  Message=Compiler.DataReader.IsDbNull.TrueExpression: Failed to convert the value expression for property handler ''. Context :: TargetType: SpeedEnum
  Source=<無法評估例外狀況來源>
  StackTrace:
<無法評估例外狀況堆疊追蹤>

  此例外狀況原先在此呼叫堆疊擲回:
    [外部程式碼]

內部例外狀況 1:
InvalidOperationException: 類型 'SpeedEnum' 與 'System.String' 間未定義強制型轉運算子。

seem GetClassPropertyParameterInfoIsDbNullFalseValueExpression in RepoDb.Core\RepoDb\Reflection\Compiler\Compiler.cs
call ConvertExpressionToEnumExpression when from string to enum
bypass PropertyHandler
hope you can help,
thank you.

We will investigate this one and hopefully to include on the next beta release. When do you need the fix for this?

If possible, I hope within two month
thank you very much

Yeah, that should be possible. Thanks

@fredliex, we have assessed this enhancement request internally.
Firstly, the DbValueAttribute is not a standard attribute that can be found from the System.ComponentModel.DataAnnotations.Schema namespace.
Secondly, the proposal is an edge-case, which most likely be used to only on few scenarios (very low), but the possible changes to the compiler is huge and would impact to the AOT compilation performance of the enumeration (a bit).
It is unfortunate, but we have decided not to proceed with this.


Reading delta log Table Checkpoint files should fallback the entire plan

fixes https://github.com/NVIDIA/spark-rapids/issues/6443
I'm seeing queries on Databricks where the query reads delta logs Table Checkpoint files (from the delta_log directory) which runs on the CPU:
Scan ExistingRDD Delta Table Checkpoint with Stats
and then after that it goes to run a filter operation on the GPU which request row to columnar and then columnar to row and those transition look like they just take more time than its worth. It will also cause the use of the gpu semaphore which means it blocks something else from getting on there.
Based on this we should have it fall back to the CPU for the entire plan after seeing the read of delta log table checkpoint parquet files.
Tested this on databricks on customer delta log files and seeing improved query time and cluster throughput.
Unfortunately haven't been able to come up with a basic reproduce case without customer data/query  to be able to add integration test for it - filed https://github.com/NVIDIA/spark-rapids/issues/6456 to keep investigating.

build


Suggestion: Add tetra tool attribute "is broken" and "is honable" to the attribute filter from create

idk grimm said it was within scope

Well, its within code now too.
Release is at https://www.curseforge.com/minecraft/mc-mods/tetra-pak/files/3401450


Question on vehicle counting

Hi, i wanna ask,
on what line did you actually put the counting data and how do you reflect it back to the gui?
thank you

[put counting data] (https://github.com/wsh122333/Multi-type_vehicles_flow_statistics/blob/ab5b8b228bc877c6ea69cdb78b704f3e5e1e6d13/counter.py#L186)
[reflect back to gui] (https://github.com/wsh122333/Multi-type_vehicles_flow_statistics/blob/ab5b8b228bc877c6ea69cdb78b704f3e5e1e6d13/counter.py#L194)


Create pr-checker.yaml

FIX #76
Reflected PR #78
Python Setup: I added the actions/setup-python@v2 step to ensure Python is available in the environment.
@prajapatihet

Don't forget to add labels!!

Don't forget to add labels!


[JENKINS-35687] Add simple git lfs support

This is a refactoring of @matthauck's changes from jenkinsci/git-client-plugin#206.  I could not get his changes to work because git lfs fetch was failing with this error: can't resolve ref: "HEAD" (See https://github.com/jenkinsci/git-plugin/pull/414#issuecomment-260159466).  After reading git-lfs/git-lfs#1307, I realized that git lfs fetch requires the code to be checked out before it can know which large files to download.  The original code was attempting to run git lfs fetch after git fetch but before git checkout, which wasn't working because there wasn't a local HEAD yet.  This means the changes proposed by @estyrke in https://github.com/jenkinsci/git-plugin/pull/414#issuecomment-263843310 also won't work.  The fix was to remove the call to git lfs fetch and add a call to git lfs pull after the code was checked out.
Next, I ran into problems with doing git lfs pull after the git checkout when credentials are required by the remote repository.  The checkout command doesn't accept a set of credentials so there wasn't an obvious way to know which credentials should be used for the git lfs pull.  I decided to retrieve the credentials based on the remote repository URL.  That also was problematic because if the local repository has multiple remotes then there isn't a good way to know which remote should be used for git lfs pull.  This is a known issue with git lfs and is documented at git-lfs/git-lfs#436.  If no remote is provided to git lfs pull then git-lfs has some fallback logic that eventually defaults to origin.  This fallback logic is different from the fallback logic in CLIGitAPIImpl::getDefaultRemote().  Since I already needed the remote URL to know which credentials to use for git lfs pull, I decided to use getDefaultRemote() to know which remote to pull from.  In most cases it will pull from origin, which is very similar to the default behavior of git lfs pull.  This could cause problems in the future.  If it does, then we can add an additional parameter for users to specify the name of the remote to use with git lfs pull.
I tested these changes locally.  I tried to add unit tests, but couldn't find any existing tests to act as a guideline.  If unit tests are required then guidance would be greatly appreciated.
@MarkEWaite, I noticed this comment in jenkinsci/git-client-plugin#206:

Wouldn't it be better to check that LFS is available in the git version being used, rather than just throw an exception when it fails?

I agree with @matthauck that an explicit setting is needed.  There are use cases where a repository has files stored using git-lfs but the user doesn't want to pull those files during the initial checkout.  Making the git lfs pull an option gives users flexibility to decide when/if they want to pull the files.
A separate PR for jenkinsci/git-plugin is forthcoming.

I'd really like an automated test for this. I think you could follow the pattern of the submodule tests. They rely on the existence of separate branches in this repository, tests/getSubmodules and tests/notSubmodules.  Those repositories provide the "source data" that can be tested in the tests on the main branch.
You can refer to GitClientTest for an example that I recently added which uses the tests/getSubmodules branch as source data for a test.

Thank you for the guidance.  I created a test branch called tests/largeFileSupport but I cannot push files stored using git LFS to GitHub because GitHub wants to charge me $5/month to store them.  See https://help.github.com/articles/purchasing-additional-storage-and-bandwidth-for-a-personal-account/ for more information.
Is there a way for the test branch to be created in this repository so that I don't have to sign up for GitHub's subscription service?  I don't typically use GitHub except when necessary to contribute to projects like this.
I was planning to create a single .txt file in the repository, similar to this:
echo "5e7733d8acc94636850cb466aec524e4" > uuid.txt
git add uuid.txt
git commit -m "Added uuid.txt"

Then I was going to create a unit test to clone the repository with git LFS enabled and read the text file to make sure it contained the UUID instead of an LFS file pointer.

I added a unit test.  The unit test will fail until the tests/LargeFileSupport branch is created in this repository.  I verified that the unit test passes when used with my personal git server (not GitHub) that has the tests/LargeFileSupport branch.
The unit test also requires git-lfs to be installed on the system that runs the unit test.
I had to add support for specifying the name of the remote to pull from because the test repository created in GitClientTest always has two remotes: origin and upstream.  origin is the local path to the plugin repository, which does not contain the files stored with LFS.  The upstream repository points to this GitHub project which should have the LFS files in the tests/LargeFileSupport branch.  The CheckoutCommand was defaulting to origin instead of upstream, so I had to provide a way for the unit test to specify which remote to use.

If you'll provide me the instructions for creating the branch (and uploading the LFS binary), I'm happy to do that upload and cover the GitHub added cost (if they charge me for it).  I use GitHub all the time, and was likely to need to pay them for LFS soon anyway.

Sure, here are the instructions for creating the test branch:

Install https://git-lfs.github.com/.  Make sure you run git lfs install after installing it.  You can verify everything is configured correctly by checking git config filter.lfs.smudge to make sure it says something like git-lfs smudge -- %f.
Clone my fork of this repository:

git clone https://github.com/creste/git-client-plugin.git git-client-plugin

Change to the cloned repository:

cd git-client-plugin

Check out the tests/largeFileSupport branch that I created in my fork:

git checkout tests/largeFileSupport

Commit uuid.txt:

echo "5e7733d8acc94636850cb466aec524e4" > uuid.txt
git commit -m "Added uuid.txt" uuid.txt

Add this repository as a remote:

git remote add upstream https://github.com/jenkinsci/git-client-plugin.git

Push the branch to this repository:

git push upstream tests/largeFileSupport

If at this point you get an error from GitHub about not enough space to store the file then they want you to pay $5/month to store files using LFS.
Regarding JGit - you're right!  Looks like JGit 4.6 supports LFS without having git-lfs installed.

I wasn't thinking clearly enough about the impact of the repository being hosted in the jenkinsci organization.  I have purchased a data pack ($5 per month, low price considering how much I use GitHub repositories for Jenkins plugin automated tests), but it appears that purchase applies to my personal account not to the jenkinsci organization.
I'll need to discuss with @rtyler to see what options are available to enable git LFS for the jenkinsci organization.
I attempted to push to my personal repository, and it also continues to report that I need to purchase a data pack.  I've purchased the data pack, so I'm hopeful that within an hour or two that purchase will allow me to upload to my fork.

I did some more research and discovered these git-lfs bugs:

https://github.com/git-lfs/git-lfs/issues/773
https://github.com/git-lfs/git-lfs/issues/1449

Here is the key quote:

On GitHub.com, you can't push LFS assets to a public fork unless the original repo already has LFS objects, or you have push access to the original repo. This is a server side rule to prevent abuse. Once the public repository has LFS objects, anyone should be able to push LFS objects to their forks. If you see an issue here, let me know the repository fork and the owner trying to push so we can look into it.

My repository is a fork of a fork of this repository, which is public.  However, I'm not seeing the "can not upload new objects to public fork" error message. But maybe the above issues are related in some way.  To test that theory, I created a repository at creste/git-lfs-test and was able to push an LFS file to it.  Therefore, I think we are both affected by the above issues and the fix is to somehow enable LFS for this repo so that our public forks can work with LFS.

Thanks!  I've sent a message to GitHub support asking for their help to explain why I can't push to my repository.  I think your research shows why I can't push, and that we will need their help before a push can be successful.

GitHub support detected some errors in the metrics they were gathering.  They've corrected those errors and I was able to push the tests/largeFileSupport branch to my personal repo and to this repo.  Can you try your tests now so that we can see if that is enough?
I installed git lfs on my machine before the push, and just did a push.  Is there anything else needed to complete my part of the work?

Thanks for jumping in with this btw., @creste! Haven't been available to get to this recently -- really appreciate you marching this change forward!

I've updated my lts and lts-with-plugins docker images to include git lfs, and have added a job which checks for existence of lfs on machines running at least git 1.8.2.  I've also deployed git lfs into most of my cluster of machines at home.
Once I've done the same on another set of computers, I'll be ready to test this.  Likely won't be until next week, but I'm making progress...

Oops, sorry about the whitespace problems.  I removed all the instances you pointed out and will do better in the future.
I'm glad to hear that adding git-lfs to the test containers is going well.  There is at least one other git-lfs improvement I have in mind (for a different PR that doesn't exist yet) that would need a unit test.

FWIW, I've built the modified plugin and it seems to be working fine for me (we needed LFS support within Jenkins).

As implemented, if a user has git lfs installed and the repository includes a large file reference and lfsRemote is not defined, then the checkout will fail.  Prior to this implementation, the checkout would have succeeded and would have ignored the large file content.  I've made a change in 37a00fb9275a4aeb49bde4450fb0a6e54a470851 which allows the checkout to succeed even if lfsRemote is not set for the job.
@creste does 37a00fb9275a4aeb49bde4450fb0a6e54a470851 have problems that I should not include it?

I am not sure if there is a problem or not.
I think some of the current users of the git plugin (without LFS support) are manually setting up git lfs to work correctly with their LFS repos.  For example, I think one way to do that is:

Install git lfs
If credentials are required, then set up an SSH agent or configure git to use a private SSH key stored locally
Rely on the smudge filter run during git checkout to pull files one at a time instead of relying on git lfs pull.

I'm basing the above steps on comments made by other users in the following posts:

https://issues.jenkins-ci.org/browse/JENKINS-35687?focusedCommentId=264233&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-264233
http://stackoverflow.com/a/35809631

This is from comments in commit 37a00fb9275a4aeb49bde4450fb0a6e54a470851:

If smudge filter is not disabled with git LFS 1.5.4 and there is an LFS reference in the repository, then the git checkout will fail.

I think that is expected for current users of the git plugin (without LFS support).  If they haven't manually configured Git LFS to work on their Jenkins nodes then the checkout will fail.  But if they have manually configured Git LFS with the correct credentials then checkout will succeed because the smudge filter will work correctly.  When those same users upgrade to the new version of the git plugin (with LFS support) they might forget to explicitly enable LFS support in Jenkins and not realize that the LFS smudge filter is now being skipped for all checkouts.  That could cause their existing jobs to fail because git LFS is no longer being called in the smudge filter like it was before.
I'm not familiar enough with Jenkins and it's development process to know how to handle the potential for breaking existing plugin users in the manner described above.  Perhaps it is sufficient to tell them to explicitly enable Git LFS in Jenkins and make sure the correct credentials are configured in Jenkins.
Other than that, I don't see any potential problems with 37a00fb9275a4aeb49bde4450fb0a6e54a470851.

Thanks @creste for the research and the description.  Those are exactly the types of compatibility problems that worried me.  My change looks like it is exactly the wrong thing to do for compatibility.
I'll construct some additional integration test case jobs which show those cases, and will likely include one or more of them in the unit tests for the plugin.
Further evaluation yet to come...


Make Visualization app's footer timestamp dynamic

At the bottom of the Visualization app there is a "last updated on" timestamp.
As of today it is hard-coded, we need to make it dynamic - It should be updated as per the page being viewed.
*Note - Suren has requested that this issue be added to the 4.0 scope for October 23rd release.

@htvenkatesh , @pandutibil , @krishnakr6 , @JalajaTR - We will need a discussion on this to figure out the design and then estimates.

@htvenkatesh - Assigning this to you as a placeholder so that we can discuss and take this up on priority for Beta release.

@pandutibil This has to be dynamic, reading from json/csv file for 1.5 and 3.7

@pandutibil shall we connect and discuss on this for 3.7 reports how to implement this?

@devu1454 please start testing this issue

@htvenkatesh @pandutibil Tested the Visualization dynamic date stamp is working as expected
Note: Tested for only cQube core-1 reports (VSK Programs) date is updated based on the latest file processing date
cQube core-2 is not checked due to all reports show no data found
so, ticket status will be in-progress

@htvenkatesh @pandutibil tested the dynamic date for cQube core -2 reports , done with file processing now in UI on each report dashboard
example:
Education Official - processed data on 21 Oct 2022 but in UI showing - 17 Oct 2022
and all other dashboards also

@pandutibil can u check this quickly if possible

Checking @htvenkatesh

@htvenkatesh
connected with @pandutibil : updated that for dynamic date display on footer they are refering the school_management_category folder date stamp so ,
I have processed school_management file on 17th Oct 2022
Scanario:
Deleted the infra output files from output container and processed the infra_trans file after completion of processing output files are generated but
school_management_category is showing as 17th Oct
rest of json files showing 21th Oct (today's date)
so according to this situation it is not latest updated date
Actual Result : if school_management file emitted on 17th then all report dashboard footer showing 17th Oct as date if we processed the files on 21th st
Expected Result : if we processed on 21th Oct then footer should be displayed on screen

@devu1454 @chetandg123 @pandutibil wts the update on this ticket?

@pandutibil , Please take this ticket as priority and update the status of this ticket

Hidden the date stamp in the application footer if there is no data available for the report

@chetandg123 please retest the scenario

@pandutibil after deployment of latest code we are not able to see the cQube core 2 reports please find the below console error for the reference

@padmanabha1234   above issue i got updates from @pandutibil that output folder structure is not correct
please refer the cqube-qa10-output having all data sources except diksha

@chetandg123 Please emit the correct data, school management category data should be included.

@padmanabha1234 do you mean school_management_master.zip this file?  if yes i have processed the file and db also having Govt as record

@PavitraIR in school_hierarchy_details table, school_management_type is not updating please look into it.

@PavitraIR , Please look out all the comments and work accordingly .

sure @RNarmatha

@pandutibil dashboard footer dynamic timestamp is not updating.

@htvenkatesh @pandutibil
re-tested the defect - dynamic date showing on UI based on data sources is working as expected in Azur installed cQube
For S3 @devu1454 could you please update once you are testing is done

In s3 dynamic time stamp is working as expected


Cannot find option to disable dismiss

presentBottomSheet(
viewController: viewControllerToPresent,
configuration: BottomSheetConfiguration(
cornerRadius: 10,
pullBarConfiguration: .visible(.init(height: 20)),
shadowConfiguration: .init(backgroundColor: UIColor.black.withAlphaComponent(0.6))
),
canBeDismissed: {
// return true or false based on your business logic
true
},
dismissCompletion: {
// handle dismiss completion if user closed bottom sheet by a gesture
}
)
I can't find above function which allows me to pass canBeDismissed. Can anyone help me

presentBottomSheet(
viewController: viewControllerToPresent,
configuration: BottomSheetConfiguration(
cornerRadius: 10,
pullBarConfiguration: .visible(.init(height: 20)),
shadowConfiguration: .init(backgroundColor: UIColor.black.withAlphaComponent(0.6))
),
canBeDismissed: {
// return true or false based on your business logic
true
},
dismissCompletion: {
// handle dismiss completion if user closed bottom sheet by a gesture
}
)
Sorry, didn't see above code before "canBeDismissed"


RFC: Lattice Output

I would like to propose an extension to the API (and CLI) for producing a character lattice (directed acyclic character hypotheses graph) as recognition result with LSTM models.
Rationale
For some applications, such as LM rescoring, OCR post-correction, multi-OCR alignment, keyword spotting, FST decoding etc, it is not enough to produce only the best path from CTC decoding, or even the n best paths of the beam, or a sequence of alternatives (aka confusion network or sausage lattice): they all throw away information on the segmentation ambiguity, which is important to preserve for the aforementioned tasks. Representing the OCR search space by a character lattice is versatile, but relieves the user from requiring knowledge of CTC.
This is especially important in Tesseract, because its LSTM decoder contains a specially optimised form of CTC not easy to get by:

it conflates adjacent outputs with certain state transitions in an effort to avoid the combinatorial explosion from expanding and summarizing all possible CTC paths (NodeContinuation etc)
it encodes certain types of symbols incrementally so the overall output dimensionality can stay small (UnicharCompress, RecodedCharID etc)
it uses multiple competing beams in parallel for different kinds of word-level and orthographic a-priori knowledge constraints (dawgs).

Export of the full character lattice would encapsulate those intricacies and dependencies, while still allowing to integrate downstream applications tightly with OCR.
Implementation
In principle, inside src/lstm/recodebeam.cpp, it is not difficult to transform the resulting beam search trie at the last horizontal position into a lattice (with converging edges): One can traverse each node backwards along its path, generating vertices (for positions) and edges (for characters), but stopping at previously seen prefix paths (merely adding fan-out), and joining previously generated suffix paths (adding fan-in). The data structure of the beam elements, RecodeNode, with its linked list (prev) and prefix hash (code_hash), makes this straightforward. Also, one can easily re-use the Trie class as lattice data type (perhaps as a new DawgType=DAWG_TYPE_HYPOTHESES) by extension with public members for insertion of nodes and edges. Besides the UNICHAR_ID characters themselves, the edges have to carry the confidence scores, and the nodes have to carry coordinates. Alternatively, one could write some simple STL-based representation from scratch. (One could probably even employ a FST representation, although it would be more difficult to preserve the coordinates there.)
I already wrote a proof of concept for this, which generates a lattice based on Trie (though still without confidence and coords), and can also output a visualization via Graphviz. I can share it, if anyone is interested. This is what the results typically look like:

Problems
Obviously, the lattices thus created are still too narrow, because the beam itself is too narrow. So perhaps lattice extraction after beam search is already too late: One could try to join hypotheses falling off the beam with neighbouring nodes (without much extra cost, but requiring to fuse APIs for recognition and extraction).
And there is yet another problem even more severe: vanilla CTC cannot reliably discern between true repetitions of a character in the input and fake duplicates which result from alignment paths in the presence of competing null output. For example, node 1 above at t=311 comes from paths with two successive hyphens (with null in between) instead of one. This "diplopia" phenomenon occurs rather rarely when only the best path is to be extracted – but is not unknown, especially with suboptimal models. However, it can be expected to become overwhelming when extracting deeper alternatives and looking at a wider beam. I fail to see any simple measures to counter that (besides limited heuristics like suppressing single-pixel output hypotheses). One could adopt enhanced approaches like Equal Spacing CTC, which might be superior anyway. But this would be quite an undertaking.
Conclusion
Should Tesseract provide such a feature? What is the right approach for lattice extraction? How can the diplopia problem be addressed? What API / which data structures would be preferable (without introducing too many dependencies but being maximally reusable, perhaps even in Python)?
BTW, there were some efforts in that direction in ocropy as well.
@noahmetzger @stweil @kba Correlates with this PAGE-XML extension proposal

This a great idea and it would be useful in debugging and increasing the accuracy of the OCR.

BTW, there were some efforts in that direction in ocropy as well.

Are you referring to
https://github.com/tmbarchive/ocropy/pull/25
or
https://github.com/tmbarchive/ocropy/pull/279
?

@amitdo I linked to both of them above. But 25 is not as general and 279 is the undecoded/full matrix, therefore I also linked 186 which refers to OLD/ocropus-lattices (which in turn merely uses linerec.write_lattice() from OLD/linerec).

Oops, I didn't notice the links in your first post.


[MRG] Migrate coverage

This PR addresses #56 by moving over the coverage reporting tasks to Circle

@charlesdrotar can you review so we can merge up?


Allow specifying tile size with Raster sources

Since this is supported by the spec, this should ideally be configurable when creating or modifying a raster tile source.
https://maplibre.org/maplibre-style-spec/sources/#tilesize

May I try to work on this?

Sure, Enjoy!

I think I am done (?)
Not so sure myself hahaha
I don't see other sources fields (like scheme, max zoom, min zoom) referenced on other part of the source code, are they not used in Maputnik?

@HarelM I think this issue can be closed since it is resolved in https://github.com/maplibre/maputnik/pull/947 Thanks


How to quantize customized models?

I want to quantize model from open-flamingo or https://github.com/open-mmlab/Multimodal-GPT (open-flamingo v1) before lora training,
https://github.com/open-mmlab/Multimodal-GPT this repo provides lora, but how to do qlora training and quantize the model beforehand?
Are there any ways to do this fast?
class Flamingo(nn.Module):
    def __init__(
        self,
        vision_encoder: nn.Module,
        lang_encoder: nn.Module,
        eoc_token_id: int,
        media_token_id: int,
        vis_dim: int,
        cross_attn_every_n_layers: int = 1,
        gradient_checkpointing: bool = False,
    ):
        """
        Args:
            vision_encoder (nn.Module): HF CLIPModel
            lang_encoder (nn.Module): HF causal language model
            eoc_token_id (int): Token id for <|endofchunk|>
            media_token_id (int): Token id for <image>
            vis_dim (int): Dimension of the visual features.
                Visual features are projected to match this shape along the last dimension.
            cross_attn_every_n_layers (int, optional): How often to apply cross attention after transformer layer. Defaults to 1.
        """
        super().__init__()
        self.eoc_token_id = eoc_token_id
        self.media_token_id = media_token_id
        self.vis_dim = vis_dim
        if hasattr(lang_encoder.config, "d_model"):
            self.lang_dim = lang_encoder.config.d_model  # mpt uses d_model
        else:
            self.lang_dim = lang_encoder.config.hidden_size

        self.vision_encoder = vision_encoder.visual
        self.perceiver = PerceiverResampler(dim=self.vis_dim)
        self.lang_encoder = lang_encoder
        self.lang_encoder.init_flamingo(
            media_token_id=media_token_id,
            lang_hidden_size=self.lang_dim,
            vis_hidden_size=self.vis_dim,
            cross_attn_every_n_layers=cross_attn_every_n_layers,
            gradient_checkpointing=gradient_checkpointing,
        )
        self._use_gradient_checkpointing = gradient_checkpointing
        self.perceiver._use_gradient_checkpointing = gradient_checkpointing

same question? also how to apply this for unsupported-models

For anyone looking for a way to apply Bitsandbytes quantization to custom model, here's my recent open-source project of implementing a custom QLoRA to fine-tune LLM, with basic tools like PyTorch and Bitsandbytes, completely decoupled from hugging face.
https://github.com/michaelnny/QLoRA-LLM


Add transformation helpers to reference implementation

I replaced SimPARTIX by the sleep command and a dummy template file (not really used, just created for demonstration) to simulate an actual transformation. Let me know if you had something else in mind for the reference @csadorf.

@pablo-de-andres I've refactored the code and implemented the routes and tests. Those are the remaining tasks:

[ ] Rebase on main
[ ] Better modularize the tests.


Fix Response status_code value on solve_dependencies()

None is not allowed, and now there's a conditional that checks status_code on the Response initialization.
Either here or https://github.com/encode/starlette/pull/1402

Error is mainly because of the RedirectResponse. The culprit: https://github.com/tiangolo/fastapi/blob/672c55ac626b20cc6219696023cdcc5871e5141d/fastapi/routing.py#L248-L249
Which are assuming status_code can be None...

More context can be found on https://github.com/encode/starlette/pull/1397


Cleaning

I have not tested this so I would recommend testing it before accepting it because I have no clue if it works
but yay typescript, the language better than javascript and cleaner code.

I dont like ts personally so i will not merge to the main branch but i will make a separate branch after testing


Check whether wind heading received from Windy is magnetic or true

Need to verify weather the wind heading received from Windy is magnetic or true as we might be calculating weather data with a slight offset of it differs from data received by ES. Could help in providing more accurate wind data in case we're using different types.

Windy and ECMWF data (as well as weather forecast data in general, apparently) are reported in relation to true north.
Will make true north heading the default for calculations with an option to switch back to magnetic headings as a fallback.


Multiple Typos

Hey @Avdhesh-Varshney,
I found some typos. ex:- style.css of sudoku game line 69 input::-webkit-inner-spin-but ton should be input::-webkit-inner-spin-button and others.
I would like to fix this. Please add level and tag under GSSOC'24.
Best Regards,
Nikhil Verma

@NIKHIL0VERMA Can you provide the ss of the bug. For further communication, join discord server link
For any kind of bug and enhancement, i have provided a proper template, so use it.

@NIKHIL0VERMA Due to inactivity, I have closed this issue.

Ooh, sorry for not responding. Actually busy with exams. Will sure start contributing after this week. Till then closing this issue will be great.
Thankx for understanding
Nikhil Verma