Add TRIO200, user-configured warnings for blocking sync calls in async function

Fixes the config option in #58
This ended up touching surprisingly many spots in various places, most of them unintentionally. So sorry for the diff being a bit sprawling. Going through the diff and explaining all changes:
[flake8_trio.py]

Currently it works with wildcards, only needed a minor modification to fnmatch_qualified_name for that to work smoothly, which doesn't affect the decorator matching using it.
Visitor200 is pretty simple
Couldn't find any built-in argparse dict parser, so wrote one myself. I would maybe have used : as the splitter if it was me, but easily configurable in ParseDict.

Considered splitting into two separate argumenterrors for no splitter tokens and >1 tokens, but idk if users need that much hand-holding

after you taught me about f{var!r} I couldn't resist adding it to ListOfIdentifiers as well ^^;

[pyproject.toml]

got tired of managing pyright complaining about typing errors in my test file, so added an exclude filter (and needed to explicitly add the default excludes). This means that a bunch of other test files could be cleaned up

Was there any reason this wasn't done from the start?

[tests/test_decorator.py]

test_decorator started breaking with another command-line option, so modified that to use the same construction as the test_xxx_options in test_flake8_trio.py

[tests/test_flake8_trio.py]

with the increasing number of warnings with configuration flags, and realizing we're back to manually specifying error codes and lines and shit, I added support for an # ARGS construct in a test file to specify command line flags to be used when running the file.

No obvious way to check malformed flags with that approach, so that's still in a separate test function
But test_113_options and test_114_options could be cleaned up given this new functionality. Leaving that for a later PR though.

I wanted capsys functionality in test_200_options, partly because testing is good:tm:, but in large part because I wanted to silence them in my CLI even when running with -s. So added that to test_114_options as well.

TODO:

CHANGELOG entry
README entry
README configuration flag documentation
The test file could maybe have some comments

sad, I wrote and confirmed that a version with ast.unparse, removing construct_name completely, works perfectly with current tests - until 3.8 came and spoiled all the fun.

Looks good so far!  How much would break if we just declared that 3.8 was only best-effort support?  I don't really mind requiring 3.9+ to run the linter, now that 3.11 is out.
My (weakish) preference for -> over : is that the latter often has special meaning in config file formats, or can be confused with attribute/member lookup from other languages.  -> to me reads more directly as "replace with" but I'm open to arguments against!

For just this change it would break 107,108 & 200 - and almost certainly all other 2xx messages. But there's generally some developing overhead to account for 3.8, since the AST has changed since then (e.g. https://github.com/PyCQA/flake8-bugbear/blob/main/bugbear.py#L683-L689) so it can sometimes be a non-insignificant overhead to debug issues caused by it, and even if that's just a matter of skipping broken tests you'd still have to figure out where/why/how it breaks and insert skips, and also update tests to skip it. So I personally wouldn't complain if we stopped testing on it entirely (and running the full test suite would be faster, yay!).

My (weakish) preference for -> over : is that the latter often has special meaning in config file formats, or can be confused with attribute/member lookup from other languages. -> to me reads more directly as "replace with" but I'm open to arguments against!
Makes sense! I'm mostly just colored by mentally parsing it as a dict, and seeing -> just looks a bit weird. But since we don't require {} so it's not a proper dict it's maybe a bad idea to reinforce the idea that it's parsed into one under the hood.

fixed todo's, changed fnmatch_qualified_name and started messing with disabling tests for 3.8 - but now it fails because of test coverage, so would need to configure --no-cov only when checking py38...


Injector Exception: Cannot find object factory

I'm using Injection and AnimationController in my Flutter project
 injector.map<Provider>((_) =>
 ProviderImpl(injector.get<AnimationController>()),
      isSingleton: true);

ProviderImpl
ProviderImpl(AnimationController _controller) : super();
But getting this issue
I/flutter (17236): Injector Exception: Cannot find object factory for 'AnimationController::default'
I/flutter (17236):

I have a same problems

@bedel-lemotio
I didn't answer to this issue since it was closed immediately.
Can you provide more information on how you register and get the dependency?

I'm using flutter_simple_dependency_injection  my Flutter project
import 'package:beneficiary/core/states/app_state.dart';
import 'package:firebase_analytics/firebase_analytics.dart';
import 'package:flutter_simple_dependency_injection/injector.dart' as prefix1;
import 'package:states_rebuilder/states_rebuilder.dart';
class ServiceProviderViewModel extends StatesRebuilder {
bool loading =false;
List _serviceProviderlist;
List get serviceProviders => _serviceProviderlist;
ServiceProvider _serviceProvider;
ServiceProvider get serviceProvider => _serviceProvider;
var api = prefix1.Injector.getInjector().get();
var analytics = prefix1.Injector.getInjector().get();
Future getServiceProviders(String connid,String cl,String cc,String cityId,String l1,String l2) async{
if(_serviceProviderlist == null){
loading = true;
  //_serviceProviderlist = await prefix1.Injector.getInjector().get<ApiService>().getServiceProviders(connid, cl, cc, cityId, l1, l2);
loading =false;
  rebuildStates([AppState.serviceProviderList]);
}

}
}`
My class test
class Test {
static const String BASEURL = "https://iziway.cm/api";
//static const String BASEURL = "https://gtastore.azurewebsites.net/api";
//static const String BASEURL = "http://<IP_ADDRESS>:55390/api";
Dio http;
Future<List> getWishList() async {
var response = await http.get('/shoppingCart/wishlist');
if (response.data['StatusCode'] != 200) {
throw ApiErrorException((response.data['ErrorList'] as List).join(", "));
}
return (response.data['Items'] as List).map((el) => Prescription.fromJson(el)).toList();
}
}
But getting this issue
I/flutter ( 6408): ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
I/flutter ( 6408): The following InjectorException was thrown building Builder:
I/flutter ( 6408): Injector Exception: Cannot find object factory for 'Test::default'
I/flutter ( 6408): User-created ancestor of the error-causing widget was:
I/flutter ( 6408):   MaterialApp file:///C:/laragon/www/FLUTTER/PROJETS/beneficiary/lib/main.dart:46:12
I/flutter ( 6408): When the exception was thrown, this was the stack:
I/flutter ( 6408): #0      Injector.get (package:flutter_simple_dependency_injection/src/injector.dart:155:7)
I/flutter ( 6408): #1      new ServiceProviderViewModel (package:beneficiary/core/view_models/serviceprovider_viewmodel.dart:22:44)
I/flutter ( 6408): #2      _OrderListPageState.initState (package:beneficiary/ui/pages/serviceproviders/sp_list_page/sp_list_page.dart:19:33)

You are using a different package and not the injector from this repository.
I am not sure, but the problem is the missing type parameter in this line:
// Dart will infer this type as "dyamic"
var api = prefix1.Injector.getInjector().get();

// this should work
var api = prefix1.Injector.getInjector().get<YourApiType>();

If you don't want to run into similar issues in the future, feel free to use our injector package because we catch this error with an assertion 😉
Besides that, it makes sense to disable implicit-dynamic in the analyzer options.
Keep in mind that this may require a little refactoring in your code base.
analyzer:
  strong-mode:
    implicit-casts: false
    implicit-dynamic: false

oui jai fais comme ca s-il vous plait regardez bien ma classe ServiceProviderViewModel
class ServiceProviderViewModel extends StatesRebuilder {
bool loading =false;
List _serviceProviderlist;
List get serviceProviders => _serviceProviderlist;
ServiceProvider _serviceProvider;
ServiceProvider get serviceProvider => _serviceProvider;
ApiService api = prefix1.Injector.getInjector().get();
var analytics = prefix1.Injector.getInjector().get();
Future getServiceProviders(String connid,String cl,String cc,String cityId,String l1,String l2) async{
if(_serviceProviderlist == null){
loading = true;
_serviceProviderlist = api.getServiceProviders(connid, cl, cc, cityId, l1, l2) as List;
//_serviceProviderlist = await prefix1.Injector.getInjector().get().getServiceProviders(connid, cl, cc, cityId, l1, l2);
loading =false;
rebuildStates([AppState.serviceProviderList]);
}

}
}

Where do you register the instance of ApiService  or FirebaseAnalytics ?

that's API services
class ApiService {
  static const String BASE = "https://api.test.com";
  final  BASE_URL = "${BASE}/ghi/c/b/";
  Dio http;

  void init() async {
    var headers = <String, dynamic>{
      "Content-Type": "application/json",
      "Accept": "application/json",
    };
    http = Dio(BaseOptions(baseUrl: BASE_URL, headers: headers));
  }

  /**
   * Execute a GET request for fetching a serviceproviders
   **/
  Future<List<ServiceProvider>> getServiceProviders(
      String connid,
      String cl,
      String cc,
      String cityId,
      String l1,
      String l2,
      ) async {
    var response = await http.get('?op=qsps',queryParameters: {
      "connid": connid,
      "c1": cl,
      "cc": cc,
      "cityId": cityId,
      "l1": l1,
      "l2": l2
    });
    if (response.data['StatusCode'] != 200) {
      throw ApiErrorException((response.data['ErrorList'] as List).join(", "));
    }
    return (response.data['Data'] as List).map((el) => ServiceProvider.fromJson(el)).toList();
  }

You have to register the instance somwhere.
This injector uses registerDependency  for that.
According to the readme, you have to call map and initialize your injector.
https://github.com/jonsamwell/flutter_simple_dependency_injection
Please open an issue at the correct repository if this does not fix the issue.


Fix ColumnLayer elevationScale example

Fix #3541
Background
ColumnLayer's elevationScale in the documentation is set to 5000 initially, but the input range have a default max value of 100, so it does not move beyond 100 once changed from the initial state.
Here, I pass a max prop to input field with value equal to 5000 .

Coverage remained the same at 83.562% when pulling 3594711e8d9bf28a4e659c34a56b8c3810a6bdd5 on Navneet-Sahota:fix-columnlayer-elevationscale into 42af516aae7fbb21a48e5f4260b19d52ca9a6757 on uber:master.


[AppConfig] appconfig feature set: Add requirement type to feature command

Related command

az appconfig feature set
Description

This change adds the --requirement-type argument to the appconfig feature set command
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

AppConfig

@zhoxing-ms @yanzhudd, could you please take a look at, and help merge this PR?
Thank you very much!


enhance expand row - added indication about expanded rows

PR for #1039

@dekelb, I'll check this out and this patch may not be merged immediately, I'm busy in near future. BTW, could you please run an example which enable row selection, cell edit and your expand row indicator, an check if functionality are still work well together. If you find a bug and you dont know how to solve it, that's find, I can help you
Anyway, thanks your contributions, great job

Thanks for your comment @AllenFang ! I added the examples to the expandRow examples, everything works there. I didn't add the expand to the editCell, but I just did now a double check regrading this and everything works there.

@dekelb, thanks

Released on v3.1.0, thanks again for your contribution :)

@AllenFang can you please also merge this pr also into the v2 branch?
I'm currently using v2.11.2 and it will take some time to check that everything works in v.3.
Thanks!

I'll create a branch named as v2.11.2 and could you please open again into this branch? because master currently have all the code about v3.0.0 and this is very huge.

For some reason github doesn't give me the option to create a pull request from my branch to your v2.11.2

of course, I'll spend some time to do it, but I think upgrade to newest version will be better.

I've manage to upgrade from v2 to v3 so It's not needed anymore (unless someone else need this feature in v2).
Thanks anyway :)

👍


Does quality>0 result in no resampling and bad aliasing/imaging?

I'm not too familiar with emu2413, but am working on developing a new tracker which will support VRC7 FM in the future. 0CC-FamiTracker includes a copy of emu2413 from 2004, but I found this repo which seems to be updated.
https://github.com/digital-sound-antiques/emu2413/blob/a0b0c613f0955efc66dff355555e87b82c8a77fb/emu2413.c#L1371-L1388
Looking at master OPLL_calc(), it seems like if opll->quality != 0, you run an unknown number of OPLL cycles (varies with sampling rate), then return the latest sample. Because you do not perform resampling, this will drop synthesized samples (or duplicate them if the speaker sampling rate is over 49716 Hz). Is this right?
Dropping/duplicating audio samples will seriously degrade audio quality. As a resampling method, it converts each discrete-time sample into a continuous-time rectangular non-bandlimited pulse (creating frequency imaging), then samples from that rectangle train (turning the images into aliases). This process generates many unwanted frequencies in the outputted audio.
https://github.com/digital-sound-antiques/emu2413/blob/3226177174e064db2e438227f7864692481a2a21/emu2413.c#L1456-L1472
By comparison, the 2004 version of the library (related to that found in 0CC-FamiTracker) appears to perform linear interpolation, which is less bad for audio quality.
(I think 0CC runs the synth at quality=0, which runs emu2413 at the PC sampling rate, rather than the chip's original rate.)

Also I may use https://www.vogons.org/viewtopic.php?f=9&t=37782 instead. Is that code fast enough for real-time usage?

Hi, thank you for your kind comments
The latest emu2413 v0.70, because of significant change of the envelope generator, its output sound becomes worsend in many cases but it becomes more similar to a real chip output. The 2004 version of emu2413 output is too pure.
Anyway, as for resample, the current logic it does not drop samples.
If it drops sample, resulting output contains  noise more.
In OPLL_calc(), it currently does weighted interporation rathar than dropping.  As we see the following code, any samples are added and divided by 2 on any update cycles.. So the resulting output is: out(t) = f(t)+1/2f(t-1) + 1/4f(t) + 1/8f(t) ...
https://github.com/digital-sound-antiques/emu2413/blob/a0b0c613f0955efc66dff355555e87b82c8a77fb/emu2413.c#L1354-L1360
This output is LPF-ed, thus it contains less alias noise.
However, I will fix this logic soon because the cut-off frequency is too strong. Recently I connected a real YM2413 chip directly to my mac through USB and I heard that the real output sound is more clear than I thought.

As for https://www.vogons.org/viewtopic.php?f=9&t=37782, there are many good OPLx emulators. But they does not directly emulate VRC7/YM2413. Its envelop speed and shape is different from OPLx.
I've been at last back to emu2413 development and planning to polish emu2413 ASAP.

Fix resample logic
Fix log-sin and exp tables
Fix envelope shapes

I have fixed sample rate conversion (similar alg. to 2004 version) at e35cb25.
Comparison movie is here: https://twitter.com/ym2413/status/1185636291519475712

Thanks for the update. I can't verify if it works or not for my purposes, since I haven't reached the point in my tracker where I'm actually adding ym2413/vrc7 yet.

Thank you.  I am polishing the emu2413 output quality here for a while.  I hope its sound reached the enough when your tracker project reached at that point.


Accessibility review

We should get a thorough a11y review before we open this up at the Summit. When do you think we might be a good state to run through with an audit?

@tjsavage: if you can set this up for early next week that'll be stellar.

Will ask Laura if she'll have some time next week.

We fail lighthouse tests for color contrast.
✘ Elements must have sufficient color contrast
#container > section:nth-of-type(1) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(1) > .container.style-scope.collection-card > a > div
#container > section:nth-of-type(1) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(1) > .container.style-scope.collection-card > div > div > .style-scope.collection-card
#container > section:nth-of-type(1) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(2) > .container.style-scope.collection-card > a > div
#container > section:nth-of-type(1) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(2) > .container.style-scope.collection-card > div > div > .style-scope.collection-card
#container > section:nth-of-type(1) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(3) > .container.style-scope.collection-card > a > div
#container > section:nth-of-type(1) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(3) > .container.style-scope.collection-card > div > div > .style-scope.collection-card
#container > section:nth-of-type(2) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(1) > .container.style-scope.collection-card > a > div
#container > section:nth-of-type(2) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(1) > .container.style-scope.collection-card > div > div > .style-scope.collection-card
#container > section:nth-of-type(2) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(2) > .container.style-scope.collection-card > a > div
#container > section:nth-of-type(2) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(2) > .container.style-scope.collection-card > div > div > .style-scope.collection-card
#container > section:nth-of-type(2) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(3) > .container.style-scope.collection-card > a > div
#container > section:nth-of-type(2) > .featured-row.loader.style-scope.catalog-home > collection-card:nth-of-type(3) > .container.style-scope.collection-card > div > div > .style-scope.collection-card
#getting-started > a:nth-of-type(1) > div
#gitter > div > p
#gitter > a > .style-scope.catalog-home

If we have a standard palette, I can do a PR for these. What is the styleguide (if any) we are using?

Oops, sorry I never replied @Nevraeka!
Desktop screens: https://invis.io/5C8AYM7KM
Mobile screens: https://invis.io/PX8LYWL96
Sketch file: https://www.dropbox.com/s/zl9ty338im3n0h9/customelements-hf_r2a.sketch?dl=0
I've tuned almost all contrast failures using AXE. There's a couple of outstanding ones which are harder to fix, like the "publish element" button, and "demo" on slight grey background text. All the links have been fixed to meet AA.

I think these should all be fixed now.


New version: goatcorp.XIVLauncher version 6.2.18

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 370
Microsoft Reviewers: Open in CodeFlow

/azp run


Gradle 4.2.0 (distUrl: 6.7.1) fails with 'buildConfigFields', 'versionNameOverride', 'versionCodeOverride'

I am getting this exception when executing below groovy plugin code.
java.lang.IllegalStateException: The value for property 'buildConfigFields' cannot be changed any further.
	at org.gradle.api.internal.provider.AbstractProperty$NonFinalizedValue.beforeMutate(AbstractProperty.java:386)
	at org.gradle.api.internal.provider.AbstractProperty.assertCanMutate(AbstractProperty.java:263)
	at org.gradle.api.internal.provider.DefaultMapProperty.addCollector(DefaultMapProperty.java:176)
	at org.gradle.api.internal.provider.DefaultMapProperty.put(DefaultMapProperty.java:149)
	at com.android.build.api.variant.impl.VariantImpl.addBuildConfigField(VariantImpl.kt:93)
	at com.android.build.gradle.internal.api.BaseVariantImpl.buildConfigField(BaseVariantImpl.java:634)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)

Similar errors are coming for versionCode and versionName:
at com.android.build.gradle.internal.api.ApkVariantOutputImpl.setVersionNameOverride(ApkVariantOutputImpl.java:128)
when i am trying to set:
output.versionNameOverride = versionName                     output.versionCodeOverride = versionCode
[        project.gradle.taskGraph.whenReady { graph ->
            project.android.applicationVariants.all { variant ->
                def versionName
                if (variant.buildType.name != "release") {
                    versionName = extension.appVersion.compoundVersion
                } else {
                    versionName = extension.appVersion.commercialVersion
                }
                project.version = versionName
                variant.outputs.each { output ->
                    //Getting this error when building on gradle 4.2.0 => The value for this property cannot be changed any further.
                    //Commenting below 2 lines to resolve this error
                    //
                    output.versionNameOverride = versionName
                    output.versionCodeOverride = versionCode
                }
            }
        }
]

Expected Behavior
The exception should not come like it was not coming for previous gradle versions. Below 4.2.0 it was working just fine
Current Behavior
Getting exception: java.lang.IllegalStateException: The value for property 'buildConfigFields' cannot be changed any further.
Context
We have a releaes plugin for our android projects and we use this configuration to set overriden version name and code in BuildConfig file

Thank you for your interest in Gradle!
Gradle 4.2.0 is quite old. Do you mean Android Gradle plugin 4.2.0 (Gradle Build Tool and Android Gradle plugin are different products with different versioning)? I suspect that the problem is that you are trying to adjust version information in the whenReady callback. It is likely too late, you have to do it during the configuration phase.
Overall, this is an issue with the Android Gradle Plugin which is developed by Google and cannot be addressed in the core of the Gradle Build Tool.
Please open such Android-related issues on the Android Issue Tracker at https://source.android.com/source/report-bugs

java.lang.IllegalStateException: The value for this property cannot be changed any further. 	at org.gradle.api.internal.provider.AbstractProperty$NonFinalizedValue.beforeMutate(AbstractProperty.java:386) 	at org.gradle.api.internal.provider.AbstractProperty.assertCanMutate(AbstractProperty.java:263) 	at org.gradle.api.internal.provider.AbstractProperty.setSupplier(AbstractProperty.java:212) 	at org.gradle.api.internal.provider.DefaultProperty.set(DefaultProperty.java:70) 	at com.android.build.gradle.internal.api.ApkVariantOutputImpl.setVersionNameOverride(ApkVariantOutputImpl.java:128)
This exception is not of your gradle plugin?

The exception is coming from com.android.build.gradle which is the Android Gradle Plugin. But the root cause is that you are trying to modify the variant configuration after it was finalized. The exception indicates that you are using the plugin in the wrong way. Why are you trying to change the configuration in the whenReady callback?

This was working just fine in android gradle version 4.0.1. It is when i updated to 4.2.0, it is throwing this error.,
I have multiple build flavors and if i comment this line, then no version name or code gets set in BuildConfig.

In any case, this issue has nothing to do with the Gradle build tool itself, as we don't develop the Android plugin. I would suggest you ask a question about how to properly modify BuildConfig on StackOverflow.

same issue happens with gradle-6.9-all


Enable release drafter and add CODEOWNERS

the module/plugin has had no release notes for a loooong time, additionally users who do not have write access can not ask for a review so setup a CODEOWNERS so we get a ping for this

seems this is merged?


Blurhash loading fixes, fix aspect ratio + inject fast-blurhash code to html #276

Fixing aspect ratio before the canvas gets created (pre js loading)
Add a fast-blurhash.js script in vanilla js
Embed fast blurhash loader inline into html

Embed fast blurhash loader inline into html

@reekitconcept  I don't see where this is done. Or is the idea that this simply adds the script, and then a project using this component can include the script from its main html entrypoint?

Oh, now I see how that is being done in https://github.com/kitconcept/dlr-internet/pull/515


Support to drag on long press

At the moment, you are only allowed to drag items with a handler. A common and useful design pattern is also to be able to drag and drop items on long press on the item. Could it be possible to add this feature to your lib ? Thanks.

OK sure, I'll take a look. I'll leave this open until done. Thanks

I would also like to see long press support.

@emileb any plans on this ?? i have tried to implement long press with GestorDetector but unfortunately, i didn't have a good result because long press is never detected, any tip from you ?

If anyone (like me) was looking for a solution in this issue, I ended up using a hack. I start the RecyclerView with DragSortRecycler.setLeftDragArea(0), and then, the long press just set this value to the view's width, enabling the drag. When the drag is completed, I set the value to zero again.


Control not working on iOS, works on Android

The control works fine on an Android page. On iOS it displays however there is no Caption, Prompt, signature line. Also, when you sign you cannot see your signature. It is a simple Forms app. Code below less event handler. The Signature Pad is getting created and added to the page because I see the yellow box. There is just nothing else visible or any interaction possible.
public class CaptureSignature : ContentPage {
SignaturePadView signaturePadView;
public CaptureSignature() {
    signaturePadView = new SignaturePadView();

    signaturePadView.BackgroundColor = Color.Yellow;
    signaturePadView.SignatureLineColor = Color.Black;
    signaturePadView.HeightRequest = 150;
    signaturePadView.WidthRequest = 240;
    signaturePadView.StrokeColor = Color.Black;
    signaturePadView.StrokeWidth = 2;
    signaturePadView.CaptionText = "Please sign above";
    signaturePadView.ClearText = "Clear";
    signaturePadView.ClearTextColor = Color.Red;

    Button btnDone = new Button {
        HorizontalOptions = LayoutOptions.Center,
        Text = "Save"
    };
    btnDone.Clicked += BtnDone_Clicked;

    Content = new StackLayout {
        Spacing = 10,
        Children = {
            signaturePadView,
            btnDone
        }
    };
}

}

Disregard this. After digging again I found a sample that had a initialization of the pad, even though it isn't and Init() method. Added var t = typeof(SignaturePad.Forms.iOS.SignaturePadRenderer); to my iOS AppDelegate and Pad is functioning.


compiler: Fix code generation for pow and fabs when using float32

I'm going to add the outstanding unresolved conversations to an issue and potentially tackle it in another PR, after some more thought/discussion.


Feature/672 button component

Replaced all existing buttons with the PSA Button and implemented a few general button styles

Your review app has been deployed and is available at 'http://,<IP_ADDRESS>,:,80,'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

In CropCalendarView.js, I was not sure how to find the "Legend" button the main site

This feature isn't strictly necessary, but I personally prefer to use styled() to create a customized button like this since this component is based on MUI <Button/> and have a lot of shared props.

I also think that we should put custom components under src/components just for consistency.

I also think that we should put custom components under src/components just for consistency.

I am not opposed to this. I asked him to put it in shared since these will eventually be getting shared with the rest of the tools.   I just thought it might be a better spot for it.  @mikahpinegar thoughts?

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

I also think that we should put custom components under src/components just for consistency.

I am not opposed to this. I asked him to put it in shared since these will eventually be getting shared with the rest of the tools. I just thought it might be a better spot for it. @mikahpinegar thoughts?

i would put them in /src/components

I also think that we should put custom components under src/components just for consistency.

I am not opposed to this. I asked him to put it in shared since these will eventually be getting shared with the rest of the tools. I just thought it might be a better spot for it. @mikahpinegar thoughts?

i would put them in /src/components

What should I call the new folder in components? Everything else in that directory is a folder

I also think that we should put custom components under src/components just for consistency.

I am not opposed to this. I asked him to put it in shared since these will eventually be getting shared with the rest of the tools. I just thought it might be a better spot for it. @mikahpinegar thoughts?

i would put them in /src/components

What should I call the new folder in components? Everything else in that directory is a folder

per our slack conversation PSAComponents will work.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://,<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.


fix: download JRE for keytool dependency and allow generation of pfx keys

Description
This pull request changes the following:

feat: allow fetching keytool dependency automatically for pfx keys
feat: allow node keys to generate pfx keys
doc: refresh README with latest console output
use semver package for version parsing instead of "reinventing the wheel".
Added more test coverage

It installs JRE at ~/.solo/bin/jre and keytool will be available at: ~/.solo/bin/jre/bin/keytool
Related Issues

Closes #114

TODO

[x] Install JRE only when user wants to create pfx keys
[x] Translate the bash script into javascript code so that keytool is invoked using ShellRunner.run(...)

:tada: This PR is included in version 0.22.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


builds on High Sierra are failing

Electron hangs on step "Building DMG".
Seems to be this issue.
Do you accept PRs?

Hey, sorry it took so long, still struggling to get any free time to commit to this project.
Just released 0.12.0 with electron-builder update, please check it out.

Is this working now for you guys?

@wojtkowiak Sorry for my late response. I can build again, thank you for taking a look!


Optimise grading endpoint slightly

Pull Request Test Coverage Report for Build 2943

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 93.734%

Totals

Change from base Build 2942:
0.005%

Covered Lines:
1092

Relevant Lines:
1165

💛  - Coveralls


Replace encryption/hmac routines with Tink

TODO
replace uti.py hmac and aes routines with tink
see examples
https://gist.github.com/salrashid123/d37a9c8e69a88262d30a1f83bcca52de

done on https://github.com/salrashid123/gcp_pubsub_message_encryption/commit/0c206bee761dca95441a91c9f72be0f01bd2a717


Schema compare issue with sensitivity classifications

SqlPackage or DacFx Version: 162.3.566
.NET Framework (Windows-only) or .NET Core: both
Environment (local platform and source/target platforms): Azure and SQL 2022

Steps to Reproduce:

Create 2 database projects, A.sqlproj and B.sqlproj, both targeting Sql160
Add a script with the following to A.sqlprojCREATE TABLE [dbo].[t1] ([c0] INT, [c1] INT, [c2] VARCHAR(32) NOT NULL);
GO

ADD SENSITIVITY CLASSIFICATION TO [t1].[c0] WITH(LABEL = 'Label0', INFORMATION_TYPE = 'InfoType0', RANK = High);
GO

Add a script with the following to B.sqlproj (Note the only difference from A.sqlproj is the missing column c2)CREATE TABLE [dbo].[t1] ([c0] INT, [c1] INT);
GO

ADD SENSITIVITY CLASSIFICATION TO [t1].[c0] WITH(LABEL = 'Label0', INFORMATION_TYPE = 'InfoType0', RANK = High);
GO

Now open schema compare in SSDT or ADS; or compare via DacFx API by creating 2 SchemaCompareProjectEndpoint.
Apply the changes to the target project. An extra definition for the sensitivity classification is added to the target.CREATE TABLE [dbo].[t1] ([c0] INT, [c1] INT);
GO

ADD SENSITIVITY CLASSIFICATION TO
    [dbo].[t1].[c0]
    WITH (LABEL = 'Label0', INFORMATION_TYPE = 'InfoType0', RANK = HIGH);
GO

ADD SENSITIVITY CLASSIFICATION TO [t1].[c0] WITH(LABEL = 'Label0', INFORMATION_TYPE = 'InfoType0', RANK = High);
GO

It appears that the script generator for the table includes the sensitivity classification in its batch, but when applying schema compare changes, DacFx does not expect a second batch in the statement it's looking to replace. It does not matter if the comparison source is a project, dacpac or database.
Did this occur in prior versions?  If not - which version(s) did it work in?
N/A
(DacFx/SqlPackage/SSMS/Azure Data Studio)

AB#3428470 test link pls ignore


Update scio-core, scio-test to 0.11.3

Updates

com.spotify:scio-core
com.spotify:scio-test

from 0.11.2 to 0.11.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.spotify" } ]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #30.


Do not maintain animations for destroyed sprites

I was getting some pretty poor frame rates on hardware in a platformer I'm making as games went on even when the number of elements on screen was relatively static.
Looks like the primary cause was a memory leak in animations, as the updates were keeping old sprites around and trying to update them forever rather than ignoring them after they were destroyed. For example: https://makecode.com/_A6i4uucDtdpu
So this fixes the memory leak

Hm. That example doesn't use the animations extension, but I'll take a look through particles again when I get some extra time to see if I can find where it is losing memory - it seems like it would have to be a weird edge case or something for it to last so long then crash


STOP_LOSS, STOP_LOSS_LIMIT don't support?

Feature Suggestion
比如现在btc价格60000,我需要设置价格达到55000跟随趋势做空，需要用到STOP_LOSS或STOP_LOSS_LIMIT，
需要跟随趋势，而不是等待价格反应迟钝，达到之后才开空，现在是不是没有实现？？
Impact
For example, the current Bitcoin price is 60,000. I need to set the price to 55,000 to follow the trend and go short. I need to use STOP_LOSS or STOP_LOSS_LIMIT.
You need to follow the trend instead of waiting for the price to react slowly and then open a short position after reaching it. Is it not realized now? ?
Additional context
No response

You can already implement these features using the V2 Framework. See https://hummingbot.org/v2-strategies/examples/


Cases where events are not bookended

The user can do many things to destroy contacts and these events can be lost.

destroy body
destroy shape
set the body transform
change body type
enable/disable body
change filter
connect with joint
toggle collide connected on joint

Not sure what to do about this yet. I think if this is important to the user they should be able to ask if two shapes or two bodies are touching efficiently to confirm persistence. Then maybe end events lose value.
Idea:
b2BeginProcessEvents();
// process events
b2EndProcessEvents();
// further events can be generated outside timestep if contacts are destroyed

Fixed in #837


AQL Profiling code

Execute explain as db._explain("FOR x IN _frontend RETURN x", {}, {profile:2}) to test.
Will print per node runtime stats and more

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr-linux/311/

Looking forward to this feature!
Will the profile numbers 0-4 be changed to something readable later on? 3 and 4 are not explained yet.
There are unrelated documentation changes in the branch in case you didn't notice:
https://github.com/arangodb/arangodb/pull/5165/files#diff-00cff8b294684243467a555a778569f5

No not likely I think I will keep it like that. I will add a new method db._profileQuery for end-users. The remaining integer values  stay internal for now.

And yes I think I fixed some unrelated docs

Will you add bytes transfered to the scatter / gather nodes? Since many small documents in one big chunk may not hurt as much as several big ones...

Perhaps later, lets first get this out the door

 FOR x IN test FILTER x.cc < 25 LIMIT 10 RETURN x

Execution plan:
 Id   NodeType                  Calls   Items   Runtime [s]   Comment
  1   SingletonNode                 1       1        0.0000   * ROOT
  2   EnumerateCollectionNode     102     463        0.0012     - FOR x IN test   /* full collection scan */
  3   CalculationNode             102     463        0.0018       - LET #1 = (x.`cc` < 25)   /* simple expression */   /* collections used: x : test */
  4   FilterNode                    1      10        0.0016       - FILTER #1
  5   LimitNode                     2      10        0.0019       - LIMIT 0, 10
  6   ReturnNode                    2      10        0.0016       - RETURN x

Indexes used:
 none

Optimization rules applied:
 none

Runtime Statistics:
 Writes Exec   Writes Ign   Scan Full   Scan Index   Filtered   Exec Time [s]
           0            0         463            0        453        0.00576s

Query Profile:
 Query Stage           Duration [s]
 initializing          0.00002
 parsing               0.00026
 optimizing ast        0.00003
 loading collections   0.00004
 instantiating plan    0.00019
 optimizing plan       0.00126
 executing             0.00391
 finalizing            0.00000

@dothebart @Simran-B  Comments on the layout ?

Maybe add another column for runtime in percent?

It works with db._profileQuery(), which calls profileQuery() of the explainer, whereas db._explain() calls explain() of the explainer. If we want db._explain(... , ... , { profile: 2 }) to return the profiling information as well, couldn't profileQuery() be dropped and explain() be extended to optionally include profiling information?

<IP_ADDRESS>:8529@_system> db._explain
function (query, bindVars, options) {
  if (typeof query === 'object' && typeof query.toAQL === 'function') {
    query = { query: query.toAQL() };
  }

  if (arguments.length > 1) {
    query = { query: query, bindVars: bindVars, options: options };
  }

  require('@arangodb/aql/explainer').explain(query);
}

<IP_ADDRESS>:8529@_system> db._profileQuery
function (query, bindVars, options) {
  options = options || {};
  options.profile = 2;
  query = { query: query, bindVars: bindVars, options: options };
  require('@arangodb/aql/explainer').profileQuery(query);
}

No we do not want to use explain for this at all. ArangoDB's explain is not executing anything right now (you cannot destroy your data by running explain).
Suddenly adding an option which will trigger it to actually run a query is a dangerous behaviour change.

@graetzer Yeah, I also came to the conclusion that this has very little to do with explaining queries other than that the explain output looks almost the same. It should be clear now with the documentation fixed.


Deploy resources troubleshoot

Context
Include relevant motivation and context.
Changes proposed in this pull request
Include a summary of the change.
Guidance to review
Include any useful information needed to review this change.
Include any dependencies that are required for this change.
Checklist

[ ] Attach to Trello card
[ ] Rebased main
[ ] Cleaned commit history
[ ] Tested by running locally

Used for troubleshooting only


Missing Import RadioGroup in Annotated Example

Hi Tenner,
the annotated example needs the import RadioGroup together with Radio:
import { Form, ..., RadioGroup, Radio, ... } from 'react-form'
And in the jsx the element needs to be changed from
<radiogroup ... /> to <RadioGroup ... />.
Nice lib! Thanx
Michael

Thanks for the suggestion!  Could you perhaps submit a PR to have those changes merged?


New-p2ptunnel calling wrong private function

New-P2Ptunnel is calling the private function New-FirewallPolicyTunnel, but when New-P2PTunnel is run, the private function New-FirewallPolicyTunnelNAT is running instead. New to look for why and fix this.

The private function New-FirewallPolicyTunnelNAT was incorrectly name New-FirewallPolicyTunnel in the script file. I updated the function name and that should resolve this problem.


Add EmptyExampleGroup cop

@andyw8 and @agrimm could you review this? This is an implementation of #56


Use latest rules_go to fix crossbuild issues with Bazel 0.26+

What this PR does / why we need it:
Bumps our dependency on rules_go to include https://github.com/bazelbuild/rules_go/pull/2118 to fix cross-building docker images for linux from OSX with Bazel 0.26+
Which issue this PR fixes: fixes #1766
Release note:
NONE

/retest

/retest

/retest

Looks like these test failures are caused by an invalid AppVersion being set when linking pkg/util. Needs investigation!
/unassign @JoshVanL
/assign


Converter for javax.money

is it possible to have Converters which convert two columns into an object?
Like http://jadira.sourceforge.net/apidocs/org/jadira/usertype/moneyandcurrency/joda/PersistentMoneyAmountAndCurrency.html ?

basically Money as 2 property currency and amount, so  there should not be any need for a converter
if have col_currency, col_amount on the prop Money col
the tricky part is : will simpleflatmapper find the right static method to instantiate if it does not that I can had some code to handle that. but I would start there.

I’ll add a test to check the behaviour.

I got:
org.simpleflatmapper.map.MapperBuildingException: Could not find getter for JooqFieldKey{field="APPDATA"."T_ACCOUNT"."AMOUNT", index=4} type interface javax.money.MonetaryAmount path currencyAndAmount See https://github.com/arnaudroger/SimpleFlatMapper/wiki/Errors_CSFM_GETTER_NOT_FOUND

Thanks that will help it could not find a way to build a currency amount will have a look

Money.of(tAccountRecord.getAmount(), tAccountRecord.getCurrencycode())

do you have a currency field in the APPDATA table?

looking at the api it will need a bit more work. The MonetaryAmount is created via a factory that is
I'll see If can figure out a work around.

yes, a currencycode and the amount.

so I added a way to register builder on classes, and added the definition for Monetary amount
see test
https://github.com/arnaudroger/SimpleFlatMapper/blob/master/sfm-jooq/src/test/java/org/simpleflatmapper/jooq/test/Issue537Test.java#L89
the only thing is that in your object currencyCode need to map to currencyAndAmount.currency and
amount to currencyAndAmount.number - the name of the prop in the factory to that you can as in the test add aliases. Because you have the generated model you can have a more specific target
        SfmRecordMapperProvider recordMapperProvider = SfmRecordMapperProviderFactory
                .newInstance()
                .addColumnProperty( k -> k.getField() == ACCOUNT.AMOUNT, new RenameProperty("currencyAndAmount_number"))
                .addColumnProperty( k -> k.getField() == ACOUNT.CURRENCY_CODE, new RenameProperty("currencyAndAmount_currency"))
                .newProvider();

could you try it out? version 3.19.0

Awesome - that worked wonders - thanks a lot!
For the record the resulting code is:
 return SfmRecordMapperProviderFactory
                .newInstance()
                    // https://github.com/arnaudroger/SimpleFlatMapper/issues/537:
                    .addColumnProperty( c -> c.getField() == TAccount.T_ACCOUNT.AMOUNT, new RenameProperty("currencyAndAmount_number"))
                    .addColumnProperty( c -> c.getField() == TAccount.T_ACCOUNT.CURRENCYCODE, new RenameProperty("currencyAndAmount_currency"))

                    .addAlias("AMOUNT", "currencyAndAmount_number")
                    .addAlias("currencyCode", "currencyAndAmount_currency")

                    .newProvider()


ELB deletions during kube-aws destroy

Each k8s service with type:LoadBalancer will create a ELB (v1/classic) in the same subnet, does all the grunt work of registering instance, set up listener and k8s NodePort according to the service Manifest. This is good practice. It would be perfect if *kube-aws destory" tear down logic can take care of ELB deletions as well.
I can verify the ELBs are properly tagged with KubernetesCluster=ClusterName. It is a matter of finding the load balancer by tag, re-generate the CF template and let the AWS CF does it transactional work.

I think the role of kube-aws destroy is to tear down the resources created by the cloudformation template. You could destroy all kubernetes namespaces prior to running kube-aws destroy to get rid of all the service ELBs and other resources created by kubernetes.

makes sense. close the issue.


perf(nuxt): avoid double path lookup when generating imports

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

By using else we can avoid a double path lookup when checking if it's absolute, and we won't need to explicitly make another if statement as the return value is a boolean.
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Good spot :) I'm also personally a fan of placing non-negated expressions in the if () condition first and using else for the negated version of it (for improved readability).

Good spot :) I'm also personally a fan of placing non-negated expressions in the if () condition first and using else for the negated version of it (for improved readability).

Thanks! I agree with you, I do that too, I just didn't want to change the structure too much but I see there's no harm in reversing the order of operations here.


Fix PHP deprecation errors

Facing issue with deprecations and same thing fixed in previous versions. https://github.com/zoho/zohocrm-php-sdk/pull/31

Hi @sanjayjethva,
Thanks for informing!
We'll support this for the next major SDK version.


Bump PR

Bump PR 0.37.0 SNAPSHOT.
Related issue(s):
Fixes #1898

Replaced by PR that used bump script.


Support explicit for constructor with one parameter

error: The following untracked working tree files would be overwritten by checkout:
 	.github/ISSUE_TEMPLATE/refactoring-request.md
Please move or remove them before you can switch branches.

emmm... what is it?

error: The following untracked working tree files would be overwritten by checkout:
 	.github/ISSUE_TEMPLATE/refactoring-request.md
Please move or remove them before you can switch branches.

emmm... what is it?

It doesn't matter, this check should be fixed


Emit a special warning when nil is passed to raise_error

Usage of nil with raise_error is with a high chance unintentional, including negation.
Fixes #1142

Do you want to a change log for this @pirj? Or shall I?

Added in 704f0a26.

I've received a notification about a note, but the author removed the note. Presumably from @mathieujobin or @mjobin-mdsol.

I wonder if via a config option, we could make this raise an InvalidError instead of a simple warning...

Adding such an option makes total sense to me.
Would you like to send a PR to main to make this configurable?
If you want to go a step further and raise an exception (I'd go with ArgumentError), you can also:

send another PR to main, but targeting the release 3.99 that would warn users that the option will be removed, and the exception would become the default
send a PR to 4-0-dev with removal of the option and making ArgumentError the default behaviour

@JonRowe Do you support the idea?

Thanks for the ping @pirj
I can't commit to open a PR with a new config options, I'm sorry. But I created a branch on my end for testing existing projects
https://github.com/mjobin-mdsol/rspec-expectations/commit/140845384f39c5b36a5b63e7ea2f95e5dd85bcef

@pirj I'm happy for this idea to be a thing if someone wants to make it so.


How to use this with existing express instance?

I want to use router object of node-odata and attach the endpoints to an existing router of express. Is it possible to do?
Example :
const express = require('express');
const router = express.Router();
const odata = require('node-odata');
const server = odata(dbUrl);
server.resource('/books',{type: String, name: String});
const odataRouter = .... server. ???
router.use('/odata-routes', odataRouter);
module.exports = router;

Hi @smaheshw
You can use odata instance same as express's.
lelong37 provide one implementation.
Reference: https://github.com/TossShinHwa/node-odata/issues/60


Upadte/shared link modal changes

💬 Description
A few changes relating to this PR https://github.com/gathercontent/app/pull/2636 which moves the shareable link settings from a dropdown to a modal.

Adds upgrade bolt SVG (not yet used in linked PR)
Sets the <InputWithButton /> input element to 100% width using flex: 1

🔗 Links
Links that relate to the PR (Trello, Figma etc.)
📹 GIF (optional)
A short GIF of the changes in action.
🚪 Start Points
Where is a good place to start the review? If there are multiple changes it may be worth listing multiple files.
👫 Related PRs (optional)
Any PRs that relate to the changes.
✅ Checklist

[ ] Tests written
[ ] Browser tested
[ ] Added to documentation
[ ] Added to storybook

 Hard Limit: Shareable URL Levers

 Move the share button behind the item menu


Add large-scale test for kNN vectors

We recently had a regression where the kNN vectors format validation could fail on large segments. We didn't catch this in testing or nightly performance benchmarks because they didn't produce large enough segments to trigger the bug.
We should add a "monster" test similar to Test4BBKDPoints that indexes a large dataset and maybe does a force merge. When tests are labelled @Monster they won't run as part of the regular build and are allowed to take much longer.

In https://github.com/apache/lucene/pull/11905 we added a test for a large number of documents (with a tiny dimension).
It'd also be good to clean up and merge something like https://github.com/apache/lucene/pull/11867, which indexes a large number of bytes (maybe fewer vectors but high dimension).


[Blocked][Feature] Add sitecore connector

This is the Sitecore connector created by Digital Herd

We'll keep this open until our fix has been provided

@michvh-dev  can u please fetch the latest updates from main of original repo that includes https://github.com/chili-publish/studio-connector-framework/pull/78. Currently it's not possible to build your connector

@psamusev this is now done, sorry didn't see that the Pull Request was merged

@michvh-dev May I ask you to sync with main again. We found another issue in CLI that was fixed in context of https://github.com/chili-publish/studio-connector-framework/pull/79

@psamusev done


savable and remainable variable for scripting languages such as python and javascript and so on

this is important for everyone to able to view and work with variables that are in program code after running most for scientific works ....
just like Matlab savable, view-able, workable and modifiable and loadable variables

hi @weinand ...this feature is very great and very high commercial for attracting anyone to Vscode
i ask by very high attention to it and a lot regard to it.

Please explain the feature.

this image must be output

@weinand, although it is very interesting, I think this might be out-of-scope of vscode's core, as the implementation needs to be specific to each language. From the image, I assume that @aminabvaal expects not only to save/load a set of variables, but also to have a 'real-time' view of the content. Such a feature is likely to fit as an Extension, one for each scripting language/framework.
Nonetheless, as suggested in https://www.quora.com/Is-there-a-way-to-save-the-workspace-in-python-like-one-does-in-Matlab, the best approach for Python is probably to use Jupyter. Fortunately, that's already supported: https://code.visualstudio.com/docs/python/jupyter-support

yes, this feature is out-of-scope for VS Code, and belongs to the individual language/debugger extensions. And yes: there is already Jupyter support.


Fix/filewriter bug

:question: What

Improve Filewriter doc about {{.Field0}} in template.
fix bug: if {{.Field0}} is provided, at least one field must be present in [output.fields]. Apart from the first field, providing extra fields in [output.fields] is useless since FileWriter only uses the first anyway. This output is a raw output, that is, it writes serialized records to file, no just sets of fields.
add test

:hammer: How to test

List all steps necessary;
To test this pull request.

:white_check_mark: Checklists
This section contains a list of checklists for common uses, please delete the checklists that are useless for your current use case (or add another checklist if your use case isn't covered yet).

[x] Is there unit/integration test coverage for all new and/or changed functionality added in this PR?
[x] Have the changes in this PR been functionally tested?
[ ] Have new components been added to the related all.go files?
[ ] Have new components been added to the documentation website?
[x] Has make gofmt-write been run on the code?
[x] Has make govet been run on the code? Has the code been fixed accordingly to the output?
[x] Have the changes been added to the CHANGELOG.md file?
[x] Have the steps in CONTRIBUTING.md been followed to update a Go module?

Codecov Report

Merging #183 (cf374ca) into main (40a57bc) will increase coverage by 1.90%.
The diff coverage is 81.39%.

@@            Coverage Diff             @@
##             main     #183      +/-   ##
==========================================
+ Coverage   53.95%   55.86%   +1.90%
==========================================
  Files          60       62       +2
  Lines        4003     4615     +612
==========================================
+ Hits         2160     2578     +418
- Misses       1646     1837     +191
- Partials      197      200       +3

Impacted Files
Coverage Δ

api.go
100.00% <ø> (ø)

filter/clausefilter.go
73.50% <0.00%> (+1.18%)
:arrow_up:

filter/clear_fields.go
75.00% <0.00%> (+10.71%)
:arrow_up:

filter/concatenate.go
81.81% <0.00%> (+5.15%)
:arrow_up:

filter/dedup.go
90.32% <0.00%> (+1.86%)
:arrow_up:

filter/expand_json.go
96.61% <0.00%> (+0.45%)
:arrow_up:

filter/format_time.go
97.91% <0.00%> (+0.11%)
:arrow_up:

filter/hash.go
77.55% <0.00%> (+0.46%)
:arrow_up:

filter/metadata_lastmodified.go
89.47% <0.00%> (+0.58%)
:arrow_up:

filter/metadata_url.go
92.59% <0.00%> (+0.28%)
:arrow_up:

... and 76 more

Continue to review full report at Codecov.

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


Context Deadline Exceeded Errors when using helm with TLS

After enabled TLS, we are getting random Context Deadline Exceeded Errors. No information in the logs that would help us track down the error either.
Output of helm version:
Client: &version.Version{SemVer:"v2.12.3", GitCommit:"eecf22f77df5f65c823aacd2dbd30ae6c65f186e", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.13.1", GitCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.1", GitCommit:"eec55b9ba98609a46fee712359c7b5b365bdd920", GitTreeState:"clean", BuildDate:"2018-12-13T19:44:19Z", GoVersion:"go1.11.2", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"12+", GitVersion:"v1.12.6-eks-d69f1b", GitCommit:"d69f1bf3669bf00b7f4a758e978e0e7a1e3a68f7", GitTreeState:"clean", BuildDate:"2019-02-28T20:26:10Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
EKS

Have you tried using the --debug flag to get verbose output? Can you share those logs?

I'm currently seeing a similar issue on AKS. I have done some troubleshooting as a result and have narrowed it down to the TLS verification.
When setting the --tiller-tls-verify flag I am unable to execute any helm commands such as helm ls --tls. Regardless of the command I am seeing the same error (Error: context deadline exceeded). When running --debug  flag, I am not seeing any useful output such as an issue with the verification of TLS certificates..etc
I am able to get Helm/Tiller TLS working without the --tiller-tls-verify flag and have confirmed it is not networking related.
Error:
`helm ls \
--tls
--tls-ca-cert ca.cert.pem
--tls-cert helm.cert.pem
--tls-key helm.key.pem
--debug
[debug] Created tunnel using local port: '63297'
[debug] SERVER: "<IP_ADDRESS>:63297"
[debug] Host="", Key="helm.key.pem", Cert="helm.cert.pem", CA="ca.cert.pem"
Error: context deadline exceeded`
Certificate Creation

For testing purposes I followed Helm doc: https://helm.sh/docs/using_helm/#using-ssl-between-helm-and-tiller

openssl verify -CAfile ca.cert.pem tiller.cert.pem tiller.cert.pem: OK
openssl verify -CAfile ca.cert.pem helm.cert.pem helm.cert.pem: OK
General Info for debugging purposes:

Tiller is deployed successfully with no errors or issues (as far as I can tell)

Kubernetes Version: 1.13.5
Helm/Tiller Version: v2.14.1
kubectl get deployment tiller-deploy -n kube-system                   NAME            DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE tiller-deploy   1         1         1            1           1h
`kubectl -n kube-system describe pod tiller-deploy-7b45545f56-l8jh4
Name:               tiller-deploy-7b45545f56-l8jh4
Namespace:          kube-system
Priority:           0
PriorityClassName:
Node:               aks-agentpool-11466683-1/<IP_ADDRESS>
Start Time:         Thu, 11 Jul 2019 09:54:42 -0400
Labels:             app=helm
name=tiller
pod-template-hash=7b45545f56
Annotations:
Status:             Running
IP:                 <IP_ADDRESS>
Controlled By:      ReplicaSet/tiller-deploy-7b45545f56
Containers:
tiller:
Container ID:   docker://757e39981fb4e3fbbd333d6845eb2c61ec52dc55ec5b28023ed5d61d2fd59048
Image:          gcr.io/kubernetes-helm/tiller:v2.14.1
Image ID:       docker-pullable://gcr.io/kubernetes-helm/tiller@sha256:f8002b91997fdc2c15a9c2aa994bea117b5b1683933f3144369862f0883c3c42
Ports:          44134/TCP, 44135/TCP
Host Ports:     0/TCP, 0/TCP
State:          Running
Started:      Thu, 11 Jul 2019 09:54:43 -0400
Ready:          True
Restart Count:  0
Liveness:       http-get http://:44135/liveness delay=1s timeout=1s period=10s #success=1 #failure=3
Readiness:      http-get http://:44135/readiness delay=1s timeout=1s period=10s #success=1 #failure=3
Environment:
TILLER_NAMESPACE:    kube-system
TILLER_HISTORY_MAX:  0
TILLER_TLS_VERIFY:   1
TILLER_TLS_ENABLE:   1
TILLER_TLS_CERTS:    /etc/certs
Mounts:
/etc/certs from tiller-certs (ro)
/var/run/secrets/kubernetes.io/serviceaccount from tiller-token-g2wwv (ro)
Conditions:
Type              Status
Initialized       True
Ready             True
ContainersReady   True
PodScheduled      True
Volumes:
tiller-certs:
Type:        Secret (a volume populated by a Secret)
SecretName:  tiller-secret
Optional:    false
tiller-token-g2wwv:
Type:        Secret (a volume populated by a Secret)
SecretName:  tiller-token-g2wwv
Optional:    false
QoS Class:       BestEffort
Node-Selectors:
Tolerations:     node.kubernetes.io/not-ready:NoExecute for 300s
node.kubernetes.io/unreachable:NoExecute for 300s
Events:
Type    Reason     Age   From                               Message

Normal  Scheduled  37m   default-scheduler                  Successfully assigned kube-system/tiller-deploy-7b45545f56-l8jh4 to aks-agentpool-11466683-1
Normal  Pulled     37m   kubelet, aks-agentpool-11466683-1  Container image "gcr.io/kubernetes-helm/tiller:v2.14.1" already present on machine
Normal  Created    37m   kubelet, aks-agentpool-11466683-1  Created container
Normal  Started    37m   kubelet, aks-agentpool-11466683-1  Started container`
kubectl logs tiller-deploy-7b45545f56-l8jh4 -n kube-system [main] 2019/07/11 13:54:43 Starting Tiller v2.14.1 (tls=true) [main] 2019/07/11 13:54:43 GRPC listening on :44134 [main] 2019/07/11 13:54:43 Probes listening on :44135 [main] 2019/07/11 13:54:43 Storage driver is ConfigMap [main] 2019/07/11 13:54:43 Max history per release is 0
Any assistance in troubleshooting this further would be greatly appreciated!

It looks like its blowing up most of the time when I call this method.
res, err := client.ReleaseContent(name)

@nerz Did you find any fix for the context deadline exceeded error:

i have the same issue.

For me setting export HELM_TLS_ENABLE=true was required to get helm provide the certificates. Even though it was already getting HELM_TLS_CA_CERT, HELM_TLS_CERT and HELM_TLS_KEY. Not sure what the CLI flag equivalents are...
Helm version: v2.14.2

I have those env variables set, but it doesn't help.
/usr/src/app $ env | grep HELM
HELM_TLS_ENABLE=true
HELM_HOME=/tmp/.helm
HELM_TLS_KEY=/usr/src/app/helm_tls_key.pem
HELM_TLS_VERIFY=true
HELM_TLS_HOSTNAME=tiller
HELM_TLS_CA_CERT=/usr/src/app/helm_tls_ca_cert.pem
HELM_TLS_CERT=/usr/src/app/helm_tls_cert.pem

We are no longer accepting bug fixes for Helm 2. See https://helm.sh/blog/helm-v2-deprecation-timeline/ for more details.


Installation not properly

I have tried to install "react-oidc-context" but not able to install it properly.
I have removed node-module and try to install fresh but still not properly installed.
See below screen shot. When i tried to install it I am getting only this three file in node-module/react-oidc-context/...

In order to use this library , do I need to install typescript npm?
Kindly help me.
Thank you.

Thanks for reporting. Its not your fault but mine.
The 0.0.1-alpha4 is invalid published, i am setting up publishing via the pipeline. Need to fix this. Please use 0.0.1-alpha3 in the meantime...

Can you test with 0.0.1-alpha5?

fixed

Thank you for update and I will try new version 0.0.1-alpha5.


Lock Mechanism in DAO

Currently, we are adding a lock in the DAO to sync access to a resource. I think that we need to consider other options that are more in line with kubernetes.
@eriknelson  mentioned resource generation reconciliation that he has seen in the service catalog.
I think that moving to some sort of caching/informer mechanism might be a way to go.
PR where change was introduced: #947

👍  I think the locking mechanism is ok, but limited as it means you cannot scale the broker beyond a single replica

/close


Breaks the widget + weather beyond repair

I wanted to reverse the widget removal because I liked how it showed the weather. If I try to do it through bloaty, it brings back the widget button "Without the weather widget" and also when you click on it nothing happens. I tried doing a system restore, tried reinstall the widget pack experience or whatever it's called.

I figured it out!!
Follow these steps:
Got to this link and install  Windows Web Experience Pack
https://apps.microsoft.com/store/detail/windows-web-experience-pack/9MSSGKG348SP?hl=en-us&gl=us
If it doesn't show an install button then it means it already installed.
In that case you should uninstall it first by opening cmd as adminstrator and type:
winget uninstall "windows web experience pack"
Then go back to the link and reinstall it.
Last step: go to windows task manager and restart your file explorer.


Kubernetes plugin: Read cluster config from api

Hey, I just made a Pull Request!
Closes #3590
This PR adds a new method getClusters to the Kubernetes backend which returns the clusters config to the frontend.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)

Tugboat has finished building the preview for this pull request!
Link:

https://pr4791-zxeyhqkekiwaiusyzu6sckjmotmasfmv.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/604024e5fcb22d0ea1675c7f

@ayshiff this was missing a changeset, so I created it for you. Good to merge when it passes +1

Tugboat has finished building the preview for this pull request!
Link:

https://pr4791-zxeyhqkekiwaiusyzu6sckjmotmasfmv.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/604024e5fcb22d0ea1675c7f

Thanks for fixing prettier! 👍


Links to static files are broken

Consider the following HTML:

The img[src] attribute is rewritten as expected, but the a[href] is not, so you get this:

We do rewrite a[href] attribute when there’s also a[download]:
https://github.com/observablehq/cli/blob/205d09b88907899a63fe9f4931715456d4055b6d/src/markdown.ts#L326
We don’t blanketly rewrite a[href] because it’s often used to link to another page, and pages don’t live in _file. But we should be able to tell whether a link is a link to a page or not (because we know how to resolve pages) — and for that matter, we want to normalize and rewrite links to pages anyway (to remove .md and to add .html for non-clean URLs).

See https://github.com/observablehq/framework/discussions/1328


Implement remainingAmount via cumulativeAmount

Order class already holds cumulativeAmount, so remainingAmount can be directly retrieved via originalAmount.subtract(cumulativeAmount).

Order.getRemainingAmount() method for that purpose. LimitOrder.remainingAmount was removed.
C'tors and Builders were updated accordingly
All relevant exchanges adapters were updated accordingly.

NOTE: Consider removing LimitOrder.getFilledAmount() which is the same as Order.getCumulativeAmount and causes confusion.

OK, I decided to leave LimitOrder.getFilledAmount() for convenience. Also you can use the LimitOrder builder with remaining amount for convenience. Only cumulative amount is an actual class member though. I guess it could even be made cleaner somehow.


Update IC Login Page and Persona Playground Page UI

UI update includes:

Persona Playground background image
Information Release Page UI

Added persona_logo.png in the assets folder, need to update this in place of the existing persona icon in the list of Login Providers

@monicavalluri can you check if we still need to update/merge this PR?

All changes in this PR are implemented.


Add strip text extension.

Description
A very simply extension which takes the latest item in your clipboard, removes formatting and trims it, and puts it back on the clipboard.
This is useful when you copy some rich text from a website, but when you paste into an app (like Gmail) you don't want font size 26!
Screencast
https://user-images.githubusercontent.com/3515525/219883617-46bca37b-3b37-43bd-864c-897f8f31c567.mp4
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

How does this differ from Paste as Plain Text except that this copies instead of pasting

Ah, does the same thing. Couldn't find that in the store. We can close the
PR as it would be a duplicate.
On Tue, Feb 21, 2023 at 2:53 AM Per Nielsen Tikær @.***>
wrote:

How does this differ from Paste as Plain Text
https://www.raycast.com/koinzhang/paste-as-plain-text except that this
copies instead of pasting
—
Reply to this email directly, view it on GitHub
https://github.com/raycast/extensions/pull/4867#issuecomment-1438268290,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA22JBKUOCJ23YINNISQON3WYSNDFANCNFSM6AAAAAAVAQC2HY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Feature request

Very good package and I personally think it's going to rise in demand exponentially soon.
Some things to work work on
1)Currently it doesn't cater for scrollable texts if the content leaves the screen
2)Another thing that would make the package very useful...it's more of a personal need but I think a lot people do need it....is if the package can also parse the chatgpt content and return as a List

a bool to enable if the package should display text as a typing animation would be wonderful too.its more engaging

4)For the titles and headers ... exposing the textstyles for these optionally would be good.the reason I say so is because the package is quite not exactly like gpt yet..the paragraphing and some little things
5)when rendering math (... )\ is different from [...]\
The square means a display I think. I noticed in the code you replaced both with the same $ sign...I think the display should take $$
Overall wonderful package and excited for the future

Hi there thanks for highlighting those issue. But I couldn't understand the no.1 Point properly. If you can provide some detailed explanation with some screen shorts then it would be good.
But for no 2. point the chat gpt contents are structured as tree form. So I am not sure how it should be done.
For no 3. Point I can do that. But the purpose of the package is to render the gpt content according to the app theme. And also that can be achieve easily by just adding some dots (some text...). Some think like that. It can be achieve outside the widget so I think It is not a major think.
For no 4. The package uses the app text theming. So you can change the styling of the headline text by just wrapping with a Theme widget.
For no 5. The package supports both (..) for inline and [..] for display type latex equations. And also it supports $..$ for inline and $$..$$ for block type equations. The reason is chatGPT using (..) and [..] syntax and gemini is using $..$ and $$..$$ syntax.

Thanks for your detailed response my friend.
For 1:
If the text to be renderered Especially latex,it leaves the screens and the widget doesn't become scrollable.
I hope this is clearer.
Also the style for online latex is inconsistent with the rest, it makes the renderered text look ugly

There is a latexBuilder method in the TexMarkdown widget you can use that to make the latex scrollable I am providing an example of doing that:
use flutter_math_fork package and the use the latexBuilder method to achieve that.
  TexMarkdown(
    "some gpt markdown text",
    latexBuilder: (contex, tex, textStyle, inline) {
      var controller = ScrollController();
      Widget child = Math.tex(
        tex,
        textStyle: textStyle,
      );
      if (!inline) {
        child = Padding(
          padding: const EdgeInsets.all(0.0),
          child: Scrollbar(
            controller: controller,
            child: SingleChildScrollView(
              controller: controller,
              scrollDirection: Axis.horizontal,
              child: child,
            ),
          ),
        );
      }
      return child;
    },
  );

You can change the font of Math.tex to make the latex look good.
I hope it will be helpful for you.


Refactor shared graphql code structure

:shipit: Mobile app from this branch has been deployed to:
https://expo.io/@thebrain/thebrain-mobile-sharedGraphqlCodeRefactor
Download the Expo app and scan this QR code to get started!

Web is available here: http://thebrain-web-1809-sharedgraphqlcoderefactor.s3-website-us-east-1.amazonaws.com
And if you must, you can connect to the graphql server here: https://363b2gy3g2.execute-api.us-east-1.amazonaws.com/dev/graphql


Update let's encrypt by adding sleep

This changes are required (in my case) to successfuly generate LE certificates.

Hello,
Don't really understand what it could change to add a 30s sleep to this script

Hello @adi90x, in my case the response for acme challenge is 503 (maybe the nginx can't update it's config "in time"). After 30s delay (maybe 10s could be enough) is everything correctly updated and le certificates are successfully generated.

Looks like I can't :)
Only those with write access to this repository can merge pull requests.

Arf I guess you need to have an account on gitlab.com to send pull request as I have set everything to public normaly !
Don't worry , i will do it tonight

i just commit it on gitlab ( however seems that I have a problem on the workflow , not related to that => I am looking into it now ! )

Pull request add on gitlab side , workflow is working and image is rebuilding on docker hub !
Thanks a lot for the PR , do not hesitate to submit any other modification !


Allow pflag creation for slices

Allow pflag creation for slices
Motivation
There could be situations where slices are part of SM's configuration and corresponding pflags should be able to be created based on that.
Approach
Remove the restriction for slices.
Pull Request status
For example:

[x] Initial implementation

Coverage remained the same at 89.76% when pulling 4a0d344c49e50b937665b9f306cc028cf1e45940 on remove-env-slice-restriction into 134e58730f665ce051b4b7d0b22e78a7762ac81d on master.

Can you add some tests in env_test that use slices


Add Greek translation

I finally finished and checked the Greek translation, so eveything should be fine. Please let me know in case you need me to fix anything.

Thanks, just made comments in 3 spots where the <p> tag should be moved to the start of the line. Once those changes are made, I'll go ahead and merge.

Sorry for that! I wanted to add the Greek article, and I apparently added it to where I thought was the start of the sentence (to my defense it was late in my timezone) instead of after the <p>, as I should. By the way, this is the first time I had to fix something before merging, so I hope all will be fine. Thanks again for understanding.

Perfect, thanks @lmintmate!


Visualizer - Create Your Design

Use Figma to design your user interface. Arrange elements such as buttons, text, images, and other components according to your layout.

Step 1: Set Up Figma Project
Create a New File:
Open Figma and create a new project or file for your data visualizer.
Define Frames:
Use frames to represent the different sections or views of your data visualizer (e.g., a dashboard, charts page, settings).
Step 2: Plan and Structure
Understand User Flows:
Identify key user flows and interactions. Understand how users will navigate through the data visualizer.
Wireframing:
Create rough wireframes to plan the placement of major elements such as charts, buttons, and text.
Step 3: Design Components
Create Basic Shapes:
Design components such as buttons, cards, and input fields using basic shapes in Figma.
Typography:
Choose fonts for headings and body text. Ensure readability and consistency throughout the design.
Icons and Images:
Incorporate relevant icons and images to enhance visual appeal and convey meaning. Use Figma's libraries or import assets.
Step 4: Arrange Elements
Grid Layout:
Utilize Figma's grid layout to align and organize elements consistently. This is crucial for creating a clean and professional design.
Hierarchy:
Establish a clear hierarchy by varying font sizes, colors, and element sizes to guide users' attention through the interface.
Group and Nest Components:
Group related elements together and use Figma's nesting features to keep your design organized. This is especially important for complex data visualizers.
Step 5: Implement Data Visualization
Integrate Charts:
If your data visualizer includes charts or graphs, use Figma's vector tools to represent them. While Figma isn't a dedicated data visualization tool, you can create visual representations that match the intended design.
Color Coding:
Choose a color scheme that helps users interpret data easily. Consistent use of colors for specific data types or categories can enhance comprehension.
Step 6: Prototyping
Create Interactive Prototypes:
Use Figma's prototyping features to create interactive prototypes. Define transitions between frames to simulate user interactions and flows.
Step 7: Test and Iterate
User Testing:
Share your Figma prototype with potential users or stakeholders to gather feedback on the design and user experience.
Iterate Based on Feedback:
Make necessary adjustments to your design based on user feedback. This may involve refining layouts, adjusting color schemes, or improving navigation.
Step 8: Handoff to Developers
Design Specifications:
Use Figma's design specifications feature to document design details and provide information for developers.
Export Assets:
Export assets (images, icons) and share design files with developers.
Step 9: Collaborate and Update
Collaboration:
Collaborate with developers and other team members using Figma's collaboration features. Ensure that everyone is on the same page during the development process.
Update Design as Needed:
If there are changes or updates during the development phase, use Figma to reflect those changes and keep all team members informed.

https://www.youtube.com/watch?v=DIfNp1wScwk figma learning

https://www.youtube.com/watch?v=uI3k9Ol-Mp4 creating dropdown


[$250] [Groups] Allow users to "share" Group Chats via share code.

While we de-scope share codes for the initial Group chats implementation, we're now in the polish phase and also in the process of re-designing the details pages throughout the app, so we'd like to add Share functionality now. The basic design looks like so.

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~015064b59f1a49ec6e
Upwork Job ID:<PHONE_NUMBER>659428352
Last Price Increase: 2024-04-16

@JmillsExpensify It seems this is dupe https://github.com/Expensify/App/issues/40256

Oh whoops. Ok yeah, I missed that share is part of that one. Thanks! I'll close this one.


Connector in Distributed mode

We have cloudera cluster with 3 kafka brokers in it. In this case, do we need to run standalone script from all the 3 brokers or only the edge node?
can you also provide steps to run it in distributed mode too? also we are in cluster with JDK 1.7, is there any requirement for this connector to be on JDK 1.8?

any suggestions or documentation for my questions?

+1

The way to run Kafka Connect IoTHub in Distributed mode is same as any other Kafka Connector in distributed mode. The only 2 interesting changes would be -

The Kafka Connect IoTHub jar should be binplaced in the libs folder of the Kafka installation on all the nodes, so that the Kafka Connect can find it the source/sink connector
The configuration should include the Kafka Connect IoTHub properties (with values same as Sandalone mode)
For information on how to start Kafka Connect in Distributed mode, please refer to the Kafka documentation - https://docs.confluent.io/3.0.0/connect/userguide.html#rest-interface

@varunpuranik Can you please also fix the Kafka Connect IoTHub jar file in releases as described in this issue? https://github.com/Azure/toketi-kafka-connect-iothub/issues/10
Thanks

As I mentioned previously, Kafka documentation is the best place to get more information to run connectors in Distributed mode (and the various knobs to use in that mode). This connector code has been tested to work in distributed mode. I will close this issue.


issue with chunking? Getting error Token indices sequence lenght is longer than specified maxium token.

Iam getting a error **Token indices sequence length is longer than the specified maximum sequence length for this model (4504 > 1024)**. Running this sequence through the model will result in indexing errors
from scrapegraphai.graphs import SmartScraperGraph
from scrapegraphai.utils import prettify_exec_info
graph_config = {
   "llm": {
      "model": "ollama/llama3.1:8b-instruct-q8_0",
      "temperature": 1,
      "format": "json",  # Ollama needs the format to be specified explicitly
      "model_tokens": 2000, #  depending on the model set context length
      "base_url": "http://localhost:11434",  # set ollama URL of the local host (YOU CAN CHANGE IT, if you have a different endpoint
   },
   "embeddings": {
      "model": "ollama/nomic-embed-text",
      "temperature": 0,
      "base_url": "http://localhost:11434",  # set ollama URL
   }
}

# ************************************************
# Create the SmartScraperGraph instance and run it
# ************************************************

smart_scraper_graph = SmartScraperGraph(
   prompt="List all articles also provide article url, upvotes and author name",
   # also accepts a string with the already downloaded HTML code
   source="https://news.ycombinator.com/",
   config=graph_config
)

result = smart_scraper_graph.run()
print(result)

graph_exec_info = smart_scraper_graph.get_execution_info()
print(prettify_exec_info(graph_exec_info))

C:\Users\djds4\llm-scraper\src>python -m test
Model ollama/llama3.1:8b-instruct-q8_0 not found,
                  using default token size (8192)
None of PyTorch, TensorFlow >= 2.0, or Flax have been found. Models won't be available and only tokenizers, configuration and file/data utilities can be used.
C:\Users\djds4\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\transformers\tokenization_utils_base.py:1601: FutureWarning: `clean_up_tokenization_spaces` was not set. It will be set to `True` by default. This behavior will be depracted in transformers v4.45, and will be then set to `False` by default. For more details check this issue: https://github.com/huggingface/transformers/issues/31884
  warnings.warn(
Token indices sequence length is longer than the specified maximum sequence length for this model (4504 > 1024). Running this sequence through the model will result in indexing errors
{'24': {'title': 'Ask HN: Any good essays/books/advice about software sales?', 'user': 'nikasakana', 'points': 148, 'comments': 59}, '25': {'title': 'The Case of the Missing Increment', 'url': 'https://www.computerenhance.com/p/the-case-of-the-missing-increment', 'points': 17, 'comments': 3}, '26': {'title': 'Show HN: A macOS app to prevent sound quality degradation on AirPods', 'url': 'https://apps.apple.com/us/app/crystalclear-sound/id6695723746?mt=12', 'points': 165, 'comments': 216}, '27': {'title': 'Do AI companies work?', 'url': 'https://benn.substack.com/p/do-ai-companies-work', 'points': 326, 'comments': 334}, '28': {'title': 'Keep Track: 3D Satellite Toolkit', 'url': 'https://app.keeptrack.space', 'points': 162, 'comments': 36}, '29': {'title': "Fix photogrammetry bridges so that they are not 'solid' underneath (2020)", 'url': 'https://forums.flightsimulator.com/t/fix-photogrammetry-bridges-so-that-they-are-not-solid-underneath/326917', 'points': 50, 'comments': 14}, '30': {'title': 'GnuCash 5.9', 'url': 'https://www.gnucash.org/news.phtml', 'points': 223, 'comments': 115}}
        node_name  total_tokens  prompt_tokens  completion_tokens  successful_requests  total_cost_USD   exec_time
0           Fetch             0              0                  0                    0             0.0    1.782655
1       ParseNode             0              0                  0                    0             0.0    0.903249
2  GenerateAnswer          1386           1026                360                    1             0.0  149.856989
3    TOTAL RESULT          1386           1026                360                    1             0.0  152.542893

not sure why i get the error Model ollama/llama3.1:8b-instruct-q8_0 not found, when the response seems to be served from  that model.

Articles returned are from the end of the page, so there definetly is some kind of chunking running.

I am also getting a similar set of errors. Trying to scrape the basics on a website and it seems to not be working.

please try with ollama/llama3.1

Almost the same response
C:\Users\djds4\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\transformers\tokenization_utils_base.py:1601: FutureWarning: `clean_up_tokenization_spaces` was not set. It will be set to `True` by default. This behavior will be depracted in transformers v4.45, and will be then set to `False` by default. For more details check this issue: https://github.com/huggingface/transformers/issues/31884
  warnings.warn(
Token indices sequence length is longer than the specified maximum sequence length for this model (4494 > 1024). Running this sequence through the model will result in indexing errors
--- Executing GenerateAnswer Node ---
{'posts': [{'_id': '41677296', 'score': 24, 'title': 'Ask HN: Any good essays/books/advice about software sales?', 'url': '', 'text': "I'm trying to transition from a dev career to sales and was wondering if anyone has any recommendations for essays, books or advice related to this field.", 'author': 'nikasakana', 'time': '16 hours ago'}, {'_id': '41693844', 'score': 175, 'title': 'Ask HN: Any good essays/books/advice about software sales?', 'url': '', 'text': '...', 'author': 'nikasakana', 'time': '16 hours ago'}, {'_id': '41691943', 'score': 350, 'title': 'Do AI companies work?', 'url': 'https://benn.substack.com/p/do-ai-companies-work', 'text': '...', 'author': 'herbertl', 'time': '1 day ago'}, {'_id': '41694712', 'score': 165, 'title': 'Keep Track: 3D Satellite Toolkit', 'url': 'https://app.keeptrack.space', 'text': '...', 'author': 'jonbaer', 'time': '1 day ago'}, {'_id': '41702789', 'score': 190, 'title': 'AI chipmaker Cerebras files for IPO', 'url': 'https://www.cnbc.com/2024/09/30/cerebras-files-for-ipo.html', 'text': '...', 'author': 'TradingPlaces', 'time': '12 hours ago'}, {'_id': '41664165', 'score': 54, 'title': 'Do All Problems Have Technical Fixes?', 'url': 'https://cacm.acm.org/article/do-all-problems-have-technical-fixes/', 'text': '...', 'author': 'zdw', 'time': '9 hours ago'}, {'_id': '41706491', 'score': 5, 'title': 'New, unique fusion reactor comes together due to global research collaboration', 'url': 'https://phys.org/news/2024-09-unique-fusion-reactor-due-global.html', 'text': '...', 'author': 'pseudolus', 'time': '1 hour ago'}, {'_id': '41668156', 'score': 57, 'title': 'Fix photogrammetry bridges so that they are not "solid" underneath (2020)', 'url': 'https://forums.flightsimulator.com/t/fix-photogrammetry-bridges-so-that-they-are-not-solid-underneath/326917', 'text': '...', 'author': 'Bluestein', 'time': '16 hours ago'}]}
        node_name  total_tokens  prompt_tokens  completion_tokens  successful_requests  total_cost_USD   exec_time
0           Fetch             0              0                  0                    0             0.0  102.539111
1       ParseNode             0              0                  0                    0             0.0    1.143226
2  GenerateAnswer          1701           1026                675                    1             0.0  136.803028
3    TOTAL RESULT          1701           1026                675                    1             0.0  240.485365

I am getting the same error message which I can't solve
--- Executing Fetch Node --- --- (Fetching HTML from: https://jobs.frequentis.com/karriere) --- --- Executing ParseNode Node --- None of PyTorch, TensorFlow >= 2.0, or Flax have been found. Models won't be available and only tokenizers, configuration and file/data utilities can be used. C:\Users\nikwa\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\transformers\tokenization_utils_base.py:1601: FutureWarning: clean_up_tokenization_spaceswas not set. It will be set toTrueby default. This behavior will be depracted in transformers v4.45, and will be then set toFalse by default. For more details check this issue: https://github.com/huggingface/transformers/issues/31884   warnings.warn( Token indices sequence length is longer than the specified maximum sequence length for this model (9962 > 1024). Running this sequence through the model will result in indexing errors --- Executing GenerateAnswer Node ---

guys if the page is too complex and heavy local models hosted on your pc are not strong enough. I suggest you to use better LLM like openais or mistral

Im also getting the same error from openai:
openai.BadRequestError: Error code: 400 - {'error': {'message': "Invalid 'messages[0].content': string too long. Expected a string with maximum length 1048576, but got a string with length 1616762 instead.", 'type': 'invalid_request_error', 'param': 'messages[0].content', 'code': 'string_above_max_length'}}
is there a param that im not seeing, or is the batching not working properly?

installing tensorflow transformers did not help, :(


Improve error message when ID used to register/login is used with self link

This Unity code demonstrates the issue:
var id = SystemInfo.deviceUniqueIdentifier;
var client = NClient.Default("defaultkey");
var message = NAuthenticateMessage.Device(id);
client.Register(message, (INSession session) => {
    client.Connect(session, (bool done) => {
        var badMessage = NSelfLinkMessage.Device(id);
        client.Send(badMessage, (bool completed) => {
            Debug.Log("Done.");
        }, (INError error) => {
            Debug.LogErrorFormat("Send error: code '{0}' - '{1}'.", error.Code, error.Message);
        });
    });
}, (INError error) => {
    Debug.LogErrorFormat("Send error: code '{0}' - '{1}'.", error.Code, error.Message);
});

The server logs show this error:
{"level":"warn","ts":"2017-07-10T19:52:01.713+0100","msg":"Could not link, query error","server":"nakama-a894","uid":"11c0a02f-cc8a-4efe-a1ce-b9a55cea4c17","sid":"13d1c676-0491-4d90-9105-582d5f3d945a","cid":"50304e1e-65fd-4b61-9639-f5695e8ff638","error":"pq: duplicate key value (id)=('2635CDEF-3AAB-5293-9611-F3364E35C807') violates unique constraint \"primary\""}

The database schema successfully prevents the user from linking a device ID already linked but we could do better to return a meaningful error to the client. The error currently returned by the server looks like "Could not link".
This was tested on Nakama 0.13.1 server and 0.6.1 Unity client.

Starting in the 1.0.0-rc1 release link operations that specify a device ID that's already in use correctly return "user link in use" errors. Look for INError.ErrorCode.UserLinkInuse in error responses in the Nakama Unity client (version 0.7.0+).


Missing keys CK62

Thanks Marcos
great job.
**There are missing keys like tilde for ntilde **for example .
=>  ñ
Don´t know how to produce them easily ... Any clue ?
Maybe a combination of Mode_switch key plus a letter could be fine.
Same thing for accents ... not easy with this keyboard.
These are my addons to your ck62 great job
first : help =>
!# Help https://wiki.linuxquestions.org/wiki/List_of_Keysyms_Recognised_by_Xmodmap
then i added these keys F11 and F12 (with basic tilde)
keycode  20 = minus underscore F11 underscore
keycode  21 = equal plus F12 asciitilde
Maybe usefull to say the layer 3 seems to be the most usefull for having default keyboard keys

Hello, Thank yo for your appreciation,
This is not my work,  I got this config from this dude -> https://gist.github.com/luccamendonca
From this link - > https://gist.github.com/luccamendonca/df48e9a0e0dd5ea8333c5ba4bafd62d3
And yes, i'm from Brazil and i need to use tilde and anothers like ã,é,à,ç
With this tiny keyboard ( CK62 ) i use the following key combinations
to ã = > Shift + Caps + Esc + a ( or n, ñ )
to à => Caps + esc + a
You can try and let me know if it works ?

Yeah it works. Thanks.
I bought this keyboard recently and was frustrated not to have accents (I'm French)  thanks to you I can manage to draw accentuated letters even if it's not obvious in Linux. Not tried yet on windows.
I've seen that there are 3 layers when you press Fn + 3 (or 1 or 2) for 5 seconds and this kb draws a complete différents set of keys.
As I use layer 3 these commands for xmodmap ain't work so I added my own xmodmap redefine keys.
This work is great and I learn what was a dead_key using that.
I'll post it in another comment if you want.
Many thanks.
Johnny


Slave Image

I cant find the reference to the slave image.
I would like to extend it.
How can i dot this?

Have a look here:
https://cloud.google.com/solutions/jenkins-on-container-engine#customizing_the_docker_image

What i was looking for could be found in the jenkins configuration at http://*/configure under Pod Template (or in the config.xml)
No, i can build my own slave image and let jenkins use it.
Thanks!


Publish Helm charts

Push charts on tag to a charts repository.
Following option 3 in discussion https://github.com/IBM/the-mesh-for-data/discussions/313
A GitHub workflow will be triggered on tags to create a PR in a charts repository. The charts repository will use helm releaser.

Is a chart repo a better approach than publishing to gh-pages?

It's using gh-pages (see https://github.com/helm/chart-releaser-action)


Add skeletons for Hypixel games

This PR adds the game classes, which implement a HypixelGame interface.
Either I or @unpluggedsam will refactor the classes into appropriate packages.
The current classes which represent multiple games (TNT Games, Arcade Games e.g.) should be split into multiple classes.
I may add the following data to the HypixelGame interface:

Date released
Date removed

looks good


Update requirements.txt

The project does not use 95% of the packages listed, think there was a mistake whilst generating it and you might have listed all the packages installed on your computer

Good catch, thanks for the PR!
Make it so the dependencies are compatible with instead of fully pinned and I'll merge it.

Should be ok now


add option to install cuda metrics plugin

https://github.com/faxm0dem/cuda-collectd can be used

I think this should be in a separate role which installs a user defined set of plugins.
Similar to https://github.com/CSC-IT-Center-for-Science/ansible-role-nrpe-plugins/blob/master/defaults/main.yml