Clarification: Differential from datadog_flutter

Hi there Datadog! great to see there is now an 'official' library - as a user of the datadog community implementation package, could someone please describe the practical differences between this package and that one? Is it worth switching right now?
Additionally, are there are any regressions at the moment between that library and this one? And, what is on the roadmap for this particular package? I haven't seen any package updates in... 34 days now.

Suggest reading https://github.com/GetDutchie/datadog_flutter/issues/108

Hi @mikhael28,
So far as we know, there are no regressions between the community library and this one, other than the removal of Span support, and we do not officially support Flutter Web (although I believe what we support currently is in line with what the Community SDK supported)
A lot of the support from the community SDK works similarly in this one, including automatic view tracking using the DatadogNavigationObserver.
Some practical differences right now:

Up to date on native dependencies and their bug fixes (see the CHANGELOG for more details)
Better property serialization support.
The tracking http client (datadog_tracking_http_client package) tracks at a lower level than the community plugin, which means it will track resource requests from Flutter itself (like the NetworkImage widget) as well as from packages like Dio without any additional configuration
The Datadog UI will show a Flutter icon, as well as the platform. You can use the same Application Id for both platforms and still differentiate between iOS and Android sessions.
Improved native crash tracking for both iOS and Android
Messages from the native Datadog SDKs will be shown in your Flutter debug console if you set sdkVerbosity.
Official support from Datadog!

As for updates -- after the initial flurry of releases, we're gathering feedback for future improvement and making sure we have a reasonable amount of fixes and new features before releasing a new version. Take that 34 days to mean we've been stable during that time and haven't needed to rush out any fixes.
As for roadmap -- Datadog is very interested to hear from people using the library about what features you think are most important. If there's something you want on our roadmap, please reach out to support or your sales rep and let them know what you're most interested in.

Alright, that sounds good - I have it in my backlog to migrate to the official library in the next 3 weeks. Will report back with any feedback.

Hi there! A few quick things about the experience migrating to Datadog's official Flutter SDK.

It was overall pretty smooth, but I was using Datadog in an incredibly limited way - I was just logging errors, with no RUM events or proactive logging. This segues into a big problem I found, which may the unofficial SDK may also be suffering from but I just don't know.
When you are trying to manually send a DD log, if you try to use the DD object outside of the closure for the top level widget context object, it will not work (but it also will not throw an error). For example, let's say you have a api_service.dart class which just has some methods for triggering an http request. It is outside of a Widget, and as a result the DD log send silently fails.
Or, for example, if I have a nested Widget, inside a nested widget, inside the top level widget. Three Widgets deep. If I try to use the DD logger in the top level widget, it works. If I try to use the logger one layer deep, it works. If I try to use it two layers deep, it silently fails. In my opinion, this is a problem as I have some relatively complex conditional rendering.

Does this sound familiar? Is this a known issue?

Hi @mikhael28,
No that does not sound familiar and would definitely be a bug if that is the case.
Can you turn on verbose SDK logging and see if maybe the error isn't so silent? Before Datadog initialization call:
DatadogSdk.sdkVerbosity = Verbosity.verbose;

If that doesn't give you any info, if you could send me over some sample code that reproduces the issue I'll take a look.
Thanks!

Hey @fuzzybinary - hope all is well. I installed the latest version of the SDK today, and I am now able to fire off logs from a generic class. I have not tested my triple-nested widget yet, but I don't need to anymore necessarily. The web version was not working in the 1.0.2 release for me, but I added it back in 1.0.3 and it seems to be good for me now!
All in all, I consider myself 'migrated' from the unofficial SDK. Thank you for your help and support!

Hey @mikhael28
That's great to hear. Web is still in very early development, but if you are using it please let me know how it's going and keep me updated on any issues you see.
Thanks!


Unable to start when using application.properties

Describe the bug
I tried to enable the graphql playground from application.properties file to use it according to this information.
Application fails to start with an error about bean registration (see below) when I specify this option in application.properties:
graphql.playground.enabled=true. Notice that even the full original example throws the same error.
The bean 'playgroundController', defined in class path resource [graphql/kickstart/playground/boot/PlaygroundAutoConfiguration.class], could not be registered. A bean with that name has already been defined in class path resource [graphql/kickstart/autoconfigure/editor/playground/PlaygroundAutoConfiguration.class] and overriding is disabled.

Without touching graphql properties in the property file I can start the server just fine (for example by using some.property=5).
Reproducible example in my repository: https://github.com/Cat-Lord/graphql-server last commit: 869fdfdc83577d939a8c19cf02160d72cc46f22d
Full Log:
> Task :GraphqlServerApplication.main() FAILED

  .   ____          _            __ _ _
 /\\ / ___'_ __ _ _(_)_ __  __ _ \ \ \ \
( ( )\___ | '_ | '_| | '_ \/ _` | \ \ \ \
 \\/  ___)| |_)| | | | | || (_| |  ) ) ) )
  '  |____| .__|_| |_|_| |_\__, | / / / /
 =========|_|==============|___/=/_/_/_/
 :: Spring Boot ::                (v2.6.7)

2022-05-19 15:57:09.628  INFO 39994 --- [           main] s.c.g.GraphqlServerApplication           : Starting GraphqlServerApplication using Java 16.0.1 on Cat-Heaven with PID 39994 (/home/catlord/git/graphql-server/build/classes/java/main started by catlord in /home/catlord/git/graphql-server)
2022-05-19 15:57:09.630  INFO 39994 --- [           main] s.c.g.GraphqlServerApplication           : No active profile set, falling back to 1 default profile: "default"
2022-05-19 15:57:10.163  WARN 39994 --- [           main] ConfigServletWebServerApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.support.BeanDefinitionOverrideException: Invalid bean definition with name 'playgroundController' defined in class path resource [graphql/kickstart/playground/boot/PlaygroundAutoConfiguration.class]: Cannot register bean definition [Root bean: class [null]; scope=; abstract=false; lazyInit=null; autowireMode=3; dependencyCheck=0; autowireCandidate=true; primary=false; factoryBeanName=graphql.kickstart.playground.boot.PlaygroundAutoConfiguration; factoryMethodName=playgroundController; initMethodName=null; destroyMethodName=(inferred); defined in class path resource [graphql/kickstart/playground/boot/PlaygroundAutoConfiguration.class]] for bean 'playgroundController': There is already [Root bean: class [null]; scope=; abstract=false; lazyInit=null; autowireMode=3; dependencyCheck=0; autowireCandidate=true; primary=false; factoryBeanName=graphql.kickstart.autoconfigure.editor.playground.PlaygroundAutoConfiguration; factoryMethodName=playgroundController; initMethodName=null; destroyMethodName=(inferred); defined in class path resource [graphql/kickstart/autoconfigure/editor/playground/PlaygroundAutoConfiguration.class]] bound.
2022-05-19 15:57:10.177  INFO 39994 --- [           main] ConditionEvaluationReportLoggingListener :

Error starting ApplicationContext. To display the conditions report re-run your application with 'debug' enabled.
2022-05-19 15:57:10.190 ERROR 39994 --- [           main] o.s.b.d.LoggingFailureAnalysisReporter   :

***************************
APPLICATION FAILED TO START
***************************

Description:

The bean 'playgroundController', defined in class path resource [graphql/kickstart/playground/boot/PlaygroundAutoConfiguration.class], could not be registered. A bean with that name has already been defined in class path resource [graphql/kickstart/autoconfigure/editor/playground/PlaygroundAutoConfiguration.class] and overriding is disabled.

Action:

Consider renaming one of the beans or enabling overriding by setting spring.main.allow-bean-definition-overriding=true

Execution failed for task ':GraphqlServerApplication.main()'.
> Process 'command '/usr/lib/jvm/java-16-openjdk-amd64/bin/java'' finished with non-zero exit value 1

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

To Reproduce
Steps to reproduce the behavior:

Clone repository
Run gradle tasks clean and bootRun with ./gradlew clean bootRun
See error described above.

Expected behavior
Server started with the ability to use playground from localhost:8080/playground.
Desktop (please complete the following information):

OS: Ubuntu 20.04
Browser: Firefox 100.0
Version: Gradle 7.4.1 (for dependencies see additional context below)

Screenshots:
Without touching the application.properties file I cannot access the playground:

Additional context
Tested with Java 11 and Java 16.
My gradle.build file:
plugins {
	id 'org.springframework.boot' version '2.6.7'
	id 'io.spring.dependency-management' version '1.0.11.RELEASE'
	id 'java'
}

group = 'sk.catheaven'
version = '0.0.1-SNAPSHOT'
sourceCompatibility = '16'

repositories {
	mavenCentral()
}

dependencies {
	implementation group: 'com.graphql-java-kickstart', name: 'graphql-spring-boot-starter', version: '12.0.0'
	implementation group: 'com.graphql-java', name: 'graphql-java', version: '18.1'

	implementation group: 'com.graphql-java-kickstart', name: 'graphql-java-tools', version: '12.0.2'
	implementation group: 'com.graphql-java-kickstart', name: 'playground-spring-boot-starter', version: '11.1.0'

	compileOnly group: 'org.projectlombok', name: 'lombok', version: '1.18.24'
	annotationProcessor group: 'org.projectlombok', name: 'lombok', version: '1.18.24'

	implementation group: 'org.springframework.boot', name: 'spring-boot-starter-web', version: '2.6.7'
}

EDIT: I accidentally saved before describing the bug, therefore the edit.

implementation group: 'com.graphql-java-kickstart', name: 'playground-spring-boot-starter', version: '11.1.0'
This is not needed here. with v12.0, the Playground starter became part of the main package.
Here you are effectively duplicating the playground controller by including the extra dependency.

And that's why is spring having troubles with bean re-registration.
Removing the duplicated dependency solved the issue. Thanks for pointing out my silly mistake, old tutorials confused me !


Exchange button components for button class names

Overview
Exchange button components for button class names.
Testing Instructions

Check the app and ensure all the buttons look the same as they did before

Closing because we're going a different route.


ISM Policies support

This PR fixes #435
Add structs to generate the ISM policy CRDs
Add roles
Add a controller to reconcile the ISM policy
Add Reconciler to create/update and delete the ISM policy from the OS cluster.
Add helper functions.
Add documentation

Hi @IshaGirdhar. How does this new PR relate to your old #555? Does it supersede it so the old one can be closed?

Hi @IshaGirdhar. How does this new PR relate to your old #555? Does it supersede it so the old one can be closed?

yes old one can be closed.

@idanl21 @swoehrl-mw Please review.

Please have a look at my inline comments. Also plase add tests for the new functionality (you can use users_test.go or role_test.go as a blueprint).

Done

I've noticed that the https://github.com/Opster/opensearch-k8s-operator/blob/main/opensearch-operator/PROJECT file hasn't been kept up-to-date lately. Additionally, the newly introduced CRD hasn't been added to it. This is because developers haven't utilized a command like kubebuilder create api --group opensearch.opster.io --version v1 --kind ISMPolicy, which would automatically update the PROJECT file.
Considering that we aren't currently using it for any specific purpose, such as OLM integration, should we consider removing it altogether (this can be done in another PR)? Alternatively, should we make a concerted effort to keep it updated?
Reference to the PROJECT file can be found here: https://book.kubebuilder.io/reference/project-config.html

I've noticed that the https://github.com/Opster/opensearch-k8s-operator/blob/main/opensearch-operator/PROJECT file hasn't been kept up-to-date lately. Additionally, the newly introduced CRD hasn't been added to it. This is because developers haven't utilized a command like kubebuilder create api --group opensearch.opster.io --version v1 --kind ISMPolicy, which would automatically update the PROJECT file.
Considering that we aren't currently using it for any specific purpose, such as OLM integration, should we consider removing it altogether (this can be done in another PR)? Alternatively, should we make a concerted effort to keep it updated?
Reference to the PROJECT file can be found here: https://book.kubebuilder.io/reference/project-config.html

Good point, it doesn't hurt to have the file, so I'd say keep it and try to keep it consistent.

@IshaGirdhar would you mind adding the CRD to the PROJECT file?
@swoehrl-mw when this is done, could you approve this PR, or are we missing something?

@AndersBennedsgaard

when this is done, could you approve this PR, or are we missing something?

My last review comments were all addressed, so should be good to go.


Happ Hacktoberfest !!

Thanks for the opportunity to be part of this  :)

Your pull request seems good, but must be checked by a human maintainer for the following reason(s):

Modifications to contributors.md must be checked manually, thanks for your submission.

Please be patient until a maintainer has time to look over your contribution.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

54caf0f20a77b6a698d350d1388a43e2a18e16ce

Do not modify the format of contributors.md

This PR has been idle for more than 24 hours, and will now be closed. Feel free to re-open.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

54caf0f20a77b6a698d350d1388a43e2a18e16ce


Harvester 1.1.2 [Question]

When will harvester 1.1.2 be released?

most likely, it will be on the next Tuesday 04/18, thanks.

That's currently scheduled for v1.2.0. Here's the feature ticket for it for reference.

Ok, I misunderstood the relationship between 1.1.2 and 1.2.0... ,When will 1.2.0 be released?

It seems to be scheduled for May 30th.  You can check the milestones here:
https://github.com/harvester/harvester/milestones

Please refer https://github.com/harvester/harvester/releases/tag/v1.1.2 to get it.


Bugfix Next with Flask - Update requirements.txt

Description
Right now in Vercel when deploying the this example, it's broken.
So, I've found the right match.
Steps to reproduce the error

Click in deploy: https://vercel.com/templates/next.js/nextjs-flask-starter

Demo URL
https://contentia-35kwh5dfn-julio-limas-projects-5820337a.vercel.app
or
https://contentia-35kwh5dfn-julio-limas-projects-5820337a.vercel.app/api/python
Type of Change

[ ] New Example
[x] Example updates (Bug fixes, new features, etc.)
[ ] Other (changes to the codebase, but not to examples)

New Example Checklist

[ ] 🛫 npm run new-example was used to create the example
[ ] 📚 The template wasn't used but I carefuly read the Adding a new example steps and implemented them in the example
[ ] 📱 Is it responsive? Are mobile and tablets considered?
[x] 🪲 Is not a new example, just a fix.

This does what #947 does, but less well, since #947 correctly removes the requirement altogether. I think it would be best to drop this one


Added gitlabsocialsymbol

I added the GitLab social icon

Should help with #26.

I did not know it existed, just google and found in gitlab media an svg to convert to tickzpicture. Just today I figure it. If it's okay with you, I try to convert one of those and make another pull request.

Ok so after reading #81, it actually seems better to fix #81 instead of adding icons in converted form. But I don't really have time to fix this until at least February (due to exams).
Since I highly doubt there will be a new release until then, I would leave this PR as is and close it when #81 is fixed. Thanks anyway.

My mistake, i will open another pr


delete by prefix rfc

Signed-off-by: Yoni<EMAIL_ADDRESS>

In several places it looks like two paragraphs are touching and it's not clear if they are meant to be separate paragraphs or whether it's one paragraph with sloppy line wrapping.
In several places Sawtooth should be capitalized.


Inference

Context
Our current story for inference is sub-optimal. This PR makes an attempt at fixing this. The code for inference is heavily inspired by gpt-fast though it's missing some of the functionality around compile. I'll add this as a follow up PR. This is basically setting up the inference to actually work.
Changelog

Update the way we setup KV Cache, including removing curr_pos which was really confusing and never made any sense to me. I replace this with input_pos (naming consistent with gpt_fast) which does exactly what you expect it to i.e. its a tensor with the current position. When we first start inference, this includes positions for all tokens in the prompt since the K and V tensors for these need to be correctly computed.
Update all components and tests

Test plan

All unit tests and recipe tests pass
Output generation is sensible for a 13B model

Should we also add a test ensuring that this does in fact speed up inference? Something like running the same inference twice, once with cache, and once without and checking the time difference?


Fix crash when request download with fileURL

Issue Link :link:
https://github.com/Alamofire/Alamofire/issues/3293
Goals :soccer:
Crash occurs when request download with fileURL (AF.download)

modified to throw an error without causing a crash.

Implementation Details :construction:
If URL is fileURL, throw AFError.invalidURL
switch request {
    case is DownloadRequest:
        if let url = initialRequest.url, url.isFileURL {
            throw AFError.invalidURL(url: url)
        }
    default:
        break
}

Testing Details :mag:
add downloadTestCase (testDownloadRequestWithLocalFileURLStringThrowsAnError)

Thanks for the PR. However, can you be more specific about the crash you're seeing? Additionally, if a file URL doesn't work for DownloadRequest, is it also broken for DataRequest and the others as well? If that's the case we probably want this logic in a more general location, like our URLRequest validation.

Investigating this error more, this definitely seems like something that should just be general check in our validate() method on URLRequest. This lives in our Alamofire+URLRequest.swift file. You could add this to the top of the validate function:
if let url = url, url.isFileURL {
    // This should become another urlRequestValidationFailed error in Alamofire 6.
    throw AFError.invalidURL(url: url)
}

@jshier i updated it. thanks for the review


Update srdf, srdf4j, srdfjena to 0.1.112

Updates

es.weso:srdf
es.weso:srdf4j
es.weso:srdfjena

from 0.1.107 to 0.1.112.
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!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "es.weso" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "es.weso" }
}]

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

Superseded by #529.


Permanent Shadows when using Sodium

Your lighting fix works great without it but I just thought I'd show what I experienced.
https://www.curseforge.com/minecraft/mc-mods/sodium

Thanks for your report. Working on a solution with JellySquid. Most likely this will be solved on Sodium's end.


Ensure string parsing works with unit tests

Currently, I have no system for writing tests. I have to implement this.

This also has to find operators that are between words. Maybe add a space between each operator?


Enable configuration of starting troops

tested not in multiplayer yet, everything else works.
@GRhin i hope we can test this in multiplayer today?

ok, we checked it out after stream and np fixed some little quirks which were not yet working, but now it works and also in multiplayer.


java.net.SocketException: Connection reset

i use this test has a error and i test three times it always blocked, can't execute 1000 times.
OkHttpClient client = new OkHttpClient();

	String run(String url) throws IOException {
		Request request = new Request.Builder().url(url).build();

		Response response = client.newCall(request).execute();
		return response.body().string();
	}
	public static void main(String[] args) throws ParseException, IOException {
		final Main example = new Main();
		for (int i = 0, count = 1000; i < count; i++) {
			new Thread(new Runnable() {

				@Override
				public void run() {
					try {
						ok(example);
					} catch (ParseException | IOException e) {
						e.printStackTrace();
					}
				}
			}).start();
		}
	}

java.net.SocketException: Connection reset
	at java.net.SocketInputStream.read(Unknown Source)
	at java.net.SocketInputStream.read(Unknown Source)
	at okio.Okio$2.read(Okio.java:136)
	at okio.AsyncTimeout$2.read(AsyncTimeout.java:211)
	at okio.RealBufferedSource.indexOf(RealBufferedSource.java:255)
	at okio.RealBufferedSource.indexOf(RealBufferedSource.java:249)
	at okio.RealBufferedSource.readUtf8LineStrict(RealBufferedSource.java:196)
	at com.squareup.okhttp.internal.http.HttpConnection.readResponse(HttpConnection.java:187)
	at com.squareup.okhttp.internal.http.HttpTransport.readResponseHeaders(HttpTransport.java:80)
	at com.squareup.okhttp.internal.http.HttpEngine.readNetworkResponse(HttpEngine.java:791)
	at com.squareup.okhttp.internal.http.HttpEngine.access$200(HttpEngine.java:90)
	at com.squareup.okhttp.internal.http.HttpEngine$NetworkInterceptorChain.proceed(HttpEngine.java:784)
	at com.squareup.okhttp.internal.http.HttpEngine.readResponse(HttpEngine.java:645)
	at com.squareup.okhttp.Call.getResponse(Call.java:263)
	at com.squareup.okhttp.Call$ApplicationInterceptorChain.proceed(Call.java:219)
	at com.squareup.okhttp.Call.getResponseWithInterceptorChain(Call.java:192)
	at com.squareup.okhttp.Call.execute(Call.java:79)
	at com.ucmed.http.Main.run(Main.java:25)
	at com.ucmed.http.Main.ok(Main.java:64)
	at com.ucmed.http.Main$1.run(Main.java:45)
	at java.lang.Thread.run(Unknown Source)

@yaming116
Expected behavior? You mentioned 1000 threads. What's the right number that OkHttp should support? I suspect the peer server may stop accepting requests at some number of DoS attack.

Yes,expected behavior.I had test apache httpclient and okhttp ,okhttp always block,it not excute 1000 times.

I don't think it's appropriate to make 1,000 simultaneous connections to a target server; that may be interpreted as a DoS attack. If you want this behavior, you should probably use OkHttp's async API, which will limit how many concurrent sockets are opened.

I have the same problem，“com.squareup.okhttp” no problem,replace "com.squareup.okhttp3" have the same problem


update version number in readme

this confused me for a little when i was attempting to use --diff from copy pasting documentation. figure this would might save someone a few minutes from the future issue.
since i feel this PR is so minor, here is a random emoji. 🚜

👍  I was about to open a PR for this too, since I was tripped up by a copy/paste from the README


Stripe checkout doesn't load properly.

Summary
Credit card region [card input field] doesn't load properly when entering all required information. After all information is saved you must return to the shopping cart and request checkout again and the credit card processing field will appear. Doing a manual refresh on the page will also cause the checkout field to not load and will require one to return to the shopping cart and go to checkout again.
Steps to reproduce

Add products to the cart and proceed to checkout.
Enter all required info.
Credit card processing area will be active but the card input will not load.
Return to shopping cart and checkout again.
If you have a credit card field refresh the page.

A link to a reproduction repository
No response
What OS are you using?
No response
Your next.config.js file
No response
Builds logs (or link to your logs)
No response
Function logs
No response

I was able to get the card module to load after adding a delay and removing the pure loadStripe import.
Updated import:
import { loadStripe } from "@stripe/stripe-js";

Updated StripeCreditCardSection function:
export function StripeCreditCardSection({ checkout }: StripeCreditCardSectionInterface) {

  const [pk, setPk] = useState<string | null>(null);

  const stripeGateway = checkout.availablePaymentGateways.find(
    (gateway) => gateway.id === STRIPE_GATEWAY
  );
  const stripeApiKey: string | null | undefined = stripeGateway?.config.find((conf) => conf.field === "api_key")?.value;

  React.useEffect(() => {
    setTimeout(() => {
      if (stripeApiKey) {
        setPk(stripeApiKey);
      }
    }, 1000);
  }, []);

  if (!pk) {
    return (

Payment Details

Loading Stripe... One moment

    );
  }

  let stripePromise: any;
  const getStripe = async () => {
    if (!stripePromise) {
      stripePromise = await loadStripe(pk);
    }
    return stripePromise;
  };

  return (

      <Elements stripe={getStripe()}>
        <StripeCardForm checkout={checkout} />
      </Elements>

  );
}


applet.trigger.level_timeout: Implement a basic level-timeout trigger to augment test equipment

This applet (and hopefully more like it) is designed to augment the triggering facilities that exist on other test equipment. In this situation, I have a scope that can trigger on an edge or "Pulse Width", but it needs to have seen both ends of the pulse before it will trigger.
With this applet, we can generate a pulse of configurable length, after a signal has been in a given state for longer than a set threshold. We can then set our test equipment to trigger off that pulse.
In the example below, Yellow (top) is the original signal, and Green (bottom) is the output from this applet.
The scope would happily trigger on this positive pulse that exceeds 1.5ms, but it won't do so until the falling edge.
Now, we can instead trigger the scope on the rising edge from Glasgow.
This is particularly useful if you're trying to trigger on activity that stops (thus no falling edge), or very long pulses.

I like the idea but the purpose seems to be quite narrow. I think I'm happy either merging this as a preview quality applet noting that the interface may be changed later without warning or backwards compatibility, or thinking harder right now about potential future extensions that aren't just everybody scratching their own itch.

The use-case for this one is indeed specific - my hope (for a long time) has been to build up some trigger applets that enhance capabilities - e.g: N-th edge trigger, and we could produce a trigger pulse on various UART conditions - byte-n of a burst, completion of a byte sequence, etc... I'm quite fortunate in my equipment, and this is the first time I really needed one of these to help some debug work.
I'm happy for this to be merged or maintained as part of an out-of-tree collection if you'd prefer.

I think an out-of-tree collection is fine until we have something more coherent to merge.


[tests-only] run full-ci during night (cron)

Description
PRs run on a smaller set of tests to save time and resources. This means some bugs might slip through reviews and end on master. Full CI will then be run on the tag when releasing and the bug might pop up. Since this is a little bit late, this PR changes the already existing nightly CI run to run a full-ci equivalent. Developers will be informed about a failed pipeline in the chat.

build should pass after #2559 gets merged


Reorganization - Part 4

This PR is a continuation of the changes proposed in #101. See also #104, #105, and #106. Again, this branch is a child of the previous PR, so the diff will shrink considerable upon merging.
Specifics in this change:

todos_repository_flutter is renamed to todos_repository_simple
firebase_flutter_repository is renamed to todos_repository_firebase
firebase_rtdb_flutter_repository is renamed to todos_repository_firebase_rtdb
Package references are updated
READMEs are updated
Removes ios/Flutter/Generated.xcconfig which appears to be an errant file.

Closed in deference to #112


Complete *func and *expr options

All options that take a function as their value:
completefunc
imactivatefunc
imstatusfunc
omnifunc
operatorfunc
quickfixtextfunc
tagfunc

Now complete with with EXPAND_USER_FUNC, which excludes built-in
functions as this probably makes the most sense.
And all options that take an expression:
balloonexpr
charconvert
diffexpr
foldexpr
foldtext
formatexpr
includeexpr
indentexpr
patchexpr
printexpr

Complete with the new EXPAND_EXPRESSION_SET; this is mostly identical to
EXPAND_USER_FUNC except that it adds "()" at the end.
There is already EXPAND_EXPRESSION, but in my testing I didn't really
find that too useful since it can't deal with :set's syntax; neither of
these work for completing F2():
:set balloonexpr=F1()\ ..\ F2()
:set balloonexpr=F1()..F2()

It only completes the F2 when you add spaces:
:set balloonexpr=F1() .. F2()

But that's not valid syntax for :set.
This could be expanded later on to deal with all of this, but IMO this
is already an improvement over not having any completion at all.

Most CI failures seem unrelated to this patch but some (temporary?) error with yarn; I don't see any way to retry the failed builds though 🤔

I reran the tests. It seems that SEGV occurs.

Thanks. It's very strange, because some tests have seemingly random test failures in areas I can't really connect to this PR, and two of them have vim --version segfaulting(?!)

There is a remark where P_EXPAND is defined: "NOTE: P_EXPAND can never be used for local or hidden options!".
It seems your patch does handle window-local options.  Please double check that works.  Then the note can be removed.
Instead of listing the options in option_expand() it might be better to add a flag P_EXP_USERFUNC and P_EXP_EXPR to the options.  We're running out of bits though (for 32 bit machines).  It appears 0x20 isn't used, might work to use 0x10, 0x20 and 0x30.

Instead of listing the options in option_expand() it might be better to add a flag P_EXP_USERFUNC and P_EXP_EXPR to the options. We're running out of bits though (for 32 bit machines). It appears 0x20 isn't used, might work to use 0x10, 0x20 and 0x30.

Yeah, it's getting a bit ugly; I planned to clean this up a bit in a future patch since I'd also like to add some more completions, for example for syntax=<syntax/*>, encoding=<valid encodings>, eventignore=<autocmds>, static string list for backspace, display, many more, and a few other things. Basically make all the options complete when it's useful.
Maybe it makes more sense to just add a new struct for this? Something along the lines of:
struct {
    char_u   *option;
    extype   long_u;
    values[] char;
} optionvalues[] = {
    {&p_ft,   EXPAND_FILETYPES,  NULL},
    {&p_cfu,  EXPAND_USER_FUNC,  NULL},
    {&p_bs,   EXPAND_LIST,       p_bs_values},
};

Or add two new fields to the options[] struct; but that one is already quite large. Could even move all the p_*_values variables to here maybe.

There is a remark where P_EXPAND is defined: "NOTE: P_EXPAND can never be used for local or hidden options!". It seems your patch does handle window-local options. Please double check that works.

I had to make some minor changes to make that work; as far as I can tell this note is now indeed outdated.

Martin Tournoij wrote:

Instead of listing the options in option_expand() it might be better
to add a flag P_EXP_USERFUNC and P_EXP_EXPR to the options. We're
running out of bits though (for 32 bit machines). It appears 0x20
isn't used, might work to use 0x10, 0x20 and 0x30.

Yeah, it's getting a bit ugly; I planned to clean this up a bit in a
future patch since I'd also like to add some more completions, for
example for syntax=<syntax/*>, encoding=<valid encodings>,
eventignore=<autocmds>, static string list for backspace,
display, many more, and a few other things. Basically make all the
options complete when it's useful.
Maybe it makes more sense to just add a new struct for this? Something along the lines of:
struct {
    char_u   *option;
    extype   long_u;
    values[] char;
} optionvalues[] = {
    {&p_ft,   EXPAND_FILETYPES,  NULL},
    {&p_cfu,  EXPAND_USER_FUNC,  NULL},
    {&p_bs,   EXPAND_LIST,       p_bs_values},
};

Or add two new fields to the options[] struct; but that one is
already quite large. Could even move all the p_*_values variables to
here maybe.

I like to specify the properties of an option in once place.  Making two
tables doesn't make it better.
Since we need more bits to store different types of expansion, we can
add another field below "flags" for this.  Assuming we might need more
flags for something later, we can use a number of bits to define the
type of completion:
#define OEXP_FILETYPE 1
#define OEXP_SYNTAX   2
#define OEXP_ENCODING 3
...
#define OEXP_MASK     0xff
#define PE_SOME_FLAG  0x100
The existing P_EXPAND can be put there as well.
--
FIRST GUARD: Ah!  Now ... we're not allowed to ...
SIR LAUNCELOT runs him through,  grabs his spear and stabs the other
guard who collapses in a heap.  Hiccoughs quietly.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

test

We have option completion now. I suppose this is no longer needed then?


Xamarin.Facebook.iOS (12.2.0) - Not logging events - iOS 14+

Hello
I have a problem with sdk it's a posible bug.
Reported events with ios 14+ do not display on the fb dashboard.
Events reported below ios 14 arrive and display correctly.
I think I have everything well set up, especially since events from before ios 14 are arriving.
https://github.com/MateuszLas421/fbSampleEvents
I found a similar problem on the facebook/facebook-ios-sdk
https://github.com/facebook/facebook-ios-sdk/issues/1937
VS fo Mac - 8.10.17
Xamarin.iOS -  <IP_ADDRESS>
Mono <IP_ADDRESS>
Xamarin.Facebook.iOS (12.2.0)

bump


remove dev command

Closes #1794
Description
Remove dev command

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 29.66%. Comparing base (888f6d2) to head (e205447).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1831      +/-   ##
==========================================
+ Coverage   28.47%   29.66%   +1.19%
==========================================
  Files          95       93       -2
  Lines        6490     6229     -261
==========================================
  Hits         1848     1848
+ Misses       4462     4201     -261
  Partials      180      180

Flag
Coverage Δ

unittests
29.66% <ø> (+1.19%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

This probably needs to wait until a major version release. I don't want to take feature away from anyone in a minor or patch.


Use __construct() to add_action()

Hello !
First of all, really nice Job on this together with Sage ! We finally have the "VC" of "MVC" in Wordpress !
I am trying to add_fonction() from my controllers. I thought the best way doing this was to use the __constructor() method.
But unfortunatly, the Controller is instanciated multiple time (in my case taxonomy-product_cat for Woocommerce), and so the action is added multiple time wich is not good.
I found a way to prevent this, but I don't think that this is a good way, more of a "tweak".
class TaxonomyProductCat extends Controller
{
    private static $trigger = 0;

    public function __construct()
    {
       /* if (!self::$trigger) {
            $this->add_product_content_data();
            //add_action('woocommerce_after_shop_loop_item_title', [$this, 'add_actions_filters'], 9);
            self::$trigger++;
        }*/
    }

    public function add_actions_filters()
    {
        if (!self::$trigger) {
            add_action('woocommerce_after_shop_loop_item_title', function() {
                echo('
oks
');
            }, 9);
            self::$trigger++;
        }
    }

    public function term()
    {
        return get_term_by('slug', get_query_var('term'), get_query_var('taxonomy'));
    }
}

As you can see, I tried first putting the add_action() into the __constructor method. with a callback [$this, 'add_product_content_data'] to call a controller method that do a "echo()". But this was not working as the controller is used to populate variables to be binded to the view, and so was directly "echoing" on top of the site because, well there is a "echo" and i dont return nothing.
So i moved the add_action to a method that is triggered once, even if controller is instanciated multiple time. And as all controller methods are called to bind data to the view, this work.
Is there a better way to do this ?
Thanks !

Hi @mtx-z
Hmm, interesting use case.  Essentially Controller was designed around data, and not hooks, so not much testing has happened in that area. I know there are still a few issues to work out with WooCommerce, from likely both Sage and Controller sides.
Two options you could try, but I have not tested these, just initial thoughts.

Use protected function add_actions_filters() and protected function term(). Remember that any public function is being run by Controller automatically in an attempt to convert to a variable for Blade. Using protected function does not run the function unless called, which you are doing above in __construct

You could also try to use a protected function functionName and removing __construct altogether. You could then run this from the view using {{ TaxonomyProductCat::functionName() }}. I have not tested this so may not work.

Lastly, for the M in MVC, https://github.com/soberwp/models may interest you.

Hi @darrenjacoby,
thanks for your answers.
Yes i saw that Controller was not designed exactly for what i wanted to do, but not impossible !
I like your approach, specially the protected method.

With the protected method, i could trigger it from the __constructor, but again, with a add_filter/action, as controller is instanciated multiple time, I would see my filter/action added multiple time. That is what i "hacked" a way to trigger it only once. In the end, the __constructor() triggering a protected method() will produce same amount of "add_filter" as using a public method() (public method() called at every instanciation, __construct() also)

The last solution, call from template, should work as expected. As the method will be called once. But i really wanted to put all business part in controllers, to enjoy the "VC" you add to WP :)

I tried to make a BaseController to extend mine, and add the self::$trigger globally, but failed as my BaseController class is never found when trying to extend my specific controllers with it.

not related, but here i found a "way" to correctly bind all variables to sub woocommerce views (https://discourse.roots.io/t/controller-and-woocommerce/9469/5) - but had a issue overriding single-product and so had to tweak to change a path. No problem with archive-product.php, strange. Issue come from the override.

Thanks again !

Cool thanks for the feedback, I'm going to test this later with an add_action and see what I come up with. It should only be firing once though, not sure why it's firing twice.

Well, from what I see, it's because controllers are triggered multiple time. So it's not a problem for returning value (if we dont see the performance layer), but it is for add filter/action.
For example, i saw that my taxonomy controller was instanciated twice.
And the App Controller, 6 or 7 time.
IDK if default, or because of my Woocommerce updates in single-product and archive-product to bind variables that causes this behavior.

@mtx-z
Using PSR4 autoloading, which will be pushed to dev-master soon, may fix this issue.
I'll do some local testing to see.

Good news !
This way we can move all the specifics action/filter into our controllers properly.
Waiting for the release ! Thanks for the update @darrenjacoby.

Any update on fixing how controllers are triggered? I'm also having some issues with this.
As far as using __construct() to load actions-- that would be kind of nice. I find that while at the initial release of Controller I used a lot of normal functions and simply returned data-- my codebase now has vastly more logic to it and I pretty much solely use public static function's now.
One problem I'm having is I have a public static function to create a cookie but it gets set in the HTTP headers like 20 times...
I tried switching it to a protected function and loading it with add_action('init', [$this, 'setSource']); as well as:
add_action('init', function () {
    wp_die('test');
    return self::setSource();
}, 11);

but not even my wp_die(); would fire no matter what priority I did the init at.
:frowning:

Hey @Log1x
I'll check it out tomorrow. I think PSR4 loading will solve this, but would need to test and confirm.

Just checking out Laravel, it might make sense to build a ServiceProvider for Sage thinking about it.

I had a similar problem and I found the App controller was being constructed twice.
Because the calls add_action were in App::__construct, and App was being constructed twice, the action handler was  registered twice.
Here is my patch for it...maybe not the most efficient, but it seems to fix the issue for me.  It definitely needs testing on more complicated sites.
https://github.com/oscar-g/controller/commit/c1ef79dc24f5a2ad7b0d84990ff6186d713cc02b
Compared with soberwp/controller@master:
https://github.com/soberwp/controller/compare/master...oscar-g:dev-run-just-once

@darrenjacoby, I'm curious about PSR4. How can we use it to solve this problem?

@oscar-g @Log1x @mtx-z
Just pushed PSR-4 to dev-master. It does appear to fix the multiple executions. Let me know if it works your side.
https://github.com/soberwp/controller/commit/93aa3ccd869b6092eefb941e342382ce7e3b00db
Regarding adding add_action to Controllers, I'm not sure it will work. I also don't feel like that should be in a Controller, a ServiceProvider would be more appropriate. I need to pass everything via this filter below, which would be set up after add_action.
add_filter('sage/template/' . $template . '-data/data', function ($data) use ($loader, $class) {
        $controller = new $class();
        $controller->__setup();
        return array_merge($loader->getAppData(), $loader->getPostData(), $controller->__setTreeData($data), $controller->__getData());
});


IJSRuntime

IJSRuntime cannot be simply injected into any .cs class.  It can be injected into a razor.cs, but injecting it into another service doesn't work.
Something like this:
var jsRuntime = host.Services.GetRequiredService<IJSRuntime>();

builder.Services.AddSingleton<IAuthorizationService, AuthorizationService>(
    service => new AuthorizationService(
        service.GetService<IUserAuthClient>(),
        jsRuntime
        )
);

Or something like:
builder.Services.AddSingleton<IAuthorizationService, AuthorizationService>(
    service => new AuthorizationService(
        service.GetService<IUserAuthClient>(),
        service.GetService<IJSRuntime>()
        )
    );

The error is that that this cannot be scoped.  I'm wondering what other solutions might work, and perhaps an example of how to inject this into other services might be useful on this page.

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

ID: 2b7ea38a-649b-303d-bebf-c701e216ba25
Version Independent ID: 9e197a8b-2616-8363-0a15-38cac802e82e
Content: Call JavaScript functions from .NET methods in ASP.NET Core Blazor
Content Source: aspnetcore/blazor/javascript-interoperability/call-javascript-from-dotnet.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @bertramdryan ... Thanks for the ask on this. Let's get assistance or a further ping to a product unit engineer on this one ...
📞 Paging Mr. @mkArtakMSFT ......... Mr. @mkArtakMSFT, please pick up a red courtesy telephone. ☎😄

I realize that this wasn't an asp.net question but a blazor question, but this README doc was the closest example, yet not exactly how to inject things like iJSRuntime into something other than a razor.cs partial.

That's fine, @bertramdryan. We'll figure that out after the PU ("product unit") comments on it. Sometimes, they say not to take something in a given direction and provide an alternative that we already cover somewhere ... or sometimes they say that what the dev is trying to do just isn't supported, which can lead to a PU issue on the PU repo for PU work (they might move this issue over to their repo) or something being closed as a won't-fix scenario.
btw tho ... IJSRuntime can be injected into classes generally. But ... yeah ... the Program class can present some challenges in extraordinary service registration/configuration/use scenarios. Some of the scenarios have been tricky to work out based on current guidance, but we cover them as they come in on questions like this. If this turns out to be a general scenario (beyond just Blazor), then it might get added to the Minimal APIs doc at ...
https://docs.microsoft.com/aspnet/core/fundamentals/minimal-apis
btw ... They're very busy 🏃😅 with .NET 7 work at this time, so there might be a delay on hearing from them. If you want to take it up with the community in the meantime, we recommend the usual public support forums ...

Stack Overflow (tag: blazor)
ASP.NET Core Slack Team
Blazor Gitter

Otherwise, stand-by ... Artak or one of the PU 🐈 will be along to help with this.

The problem I'm running into, is injecting it into another service.  Because IJSRuntime is added by default, it should be available in the `service.GetService().
I should also mention that this is a Blazor Server app not a Webassemble App.

The above image shows the exception I'm getting from it.

Did you try an approach where you build first. Like this way ....
...
var host = builder.Build();

var weatherService = host.Services.GetRequiredService<WeatherService>();
...

Did you try an approach where you build first. Like this way ....
...
var host = builder.Build();

var weatherService = host.Services.GetRequiredService<WeatherService>();
...

I see that, but does the host need to take the place of the app?  Or is it ran separately? Feels like a bit of a hack. 🤣

Not a hack for merely tapping the registered service ... in for example an initialization scenario. I don't recall a dev asking about the scenario you laid out in your opening comment. It seems like one would create a custom auth service ... and then IJSRuntime (and anything else you want) is injected into that.

@guardrex I'm writing an outlook plug-in, and storing a refresh token in the limited Office 365 storage so that the token can persist between sessions.  So, the authorization service does an initial sending of a 6 digit code sent by the API, once that code is received, it then gets an access token object from the API, which is the object I'm trying to store.
Also, Outlook provides a user profile only available via JavaScript.

I see. Yeah, that's beyond me as a simple minded 🦖 docs dino! 😄 Let's let the PU respond and/or you can reach out to community gurus on that. It's definitely an advanced scenario as far as docs would be concerned. I have a feeling that they might give you a direction to take but that we wouldn't cover it in the docs. We'll see tho. Artak should be around in a bit to help further.

@guardrex  Thanks.  Blazor isn't 'officially' recommended for this type of app. But aside from the DI issues with IJSRuntime, its worked really well.  Hopefully something can solve this. 👍

I'm going to email offline, too. We might get a quicker response that way. They get slammed with GH pings.

@bertramdryan ... Artak says to open it for the PU on their repo. Open it here ...
https://github.com/dotnet/aspnetcore/issues
Explain out the Outlook scenario and show the code concepts in your opening comment.
Please add the following to the bottom of your opening comment so that I can follow along ...
cc: @guardrex https://github.com/dotnet/AspNetCore.Docs/issues/26404

I might re-open this for doc work depending on what they say.

@guardrex was there a solution? Looks like it was moved to done?

😄 ... "Done" here ... as far as docs are concerned. Open it for the PU ... read my last comment ☝.

There it is. : Thanks.

Good ... yes ... I think Javier's response is worth documenting. I'll determine where and how to phase it.
I'll ping u on the PR.


How do I close an incident?

There is a tab for "Closed", but I can not figure out how to actually close an incident.

@IFIT-BruceG , Thank you for reporting this.
To Close an incident, you need to edit the incident and change the status to close.
You refer this link for reference.

@v-ajaysahu Found it under "Incident Status". It is not there when an incident is created, but once it is created, then "Closed" will show up. Got it! Thanks!


Track the amount of read data per row

If an sstable is corrupted in a nasty way, we may read invalid cell sizes and try to read much more data for a row than we should. In rare scenarios this can lead even to OOMs.
This simple fix adds tracking and limiting the amount of data that is read per row. Row has its size stored in preamble which can be used as a limit. If the deserialization code tries to read more than that, it will simply fail with EOF which will prevent more serious problems later.
Thanks for sending a pull request! Here are some tips if you're new here:

Ensure you have added or run the appropriate tests for your PR.
Be sure to keep the PR description updated to reflect all changes.
Write your PR title to summarize what this PR proposes.
If possible, provide a concise example to reproduce the issue for a faster review.
Read our contributor guidelines
If you're making a documentation change, see our guide to documentation contribution

Commit messages should follow the following format:
<One sentence description, usually Jira title or CHANGES.txt summary>

<Optional lengthier description (context on patch)>

patch by <Authors>; reviewed by <Reviewers> for CASSANDRA-#####

Co-authored-by: Name1 <email1>
Co-authored-by: Name2 <email2>

The Cassandra Jira

We need CI runs and will you port this to 4.0 and 4.1 also?

@bereng thanks for super quick response. I'll provide patches for older branches but probably it will be next week

@bereng is it ok now?

Approach looks good to me. CI and PRs for the rest of the versions are missing iiuc.

@bereng there was no point in running CI before code review, same for other PRs

Ha you're brave. I always run CI before...

^That is j11 only, j8 seems to not have been triggered?

https://app.circleci.com/pipelines/github/jacek-lewandowski/cassandra/720/workflows/f6fd1707-5cd1-4680-b234-0a79939fb4d2


Support key type option in ui-table for entering fixed strings as item labels

closes #1188
Description

Alters "key" field to "value" and changes it to a typed input

fall back is type "key" (for backwards compatibility

Adds e2e tests to ensure value rendered obeys keyType

Related Issue(s)
#1188
Checklist

[x] I have read the contribution guidelines
[x] Suitable unit/system level tests have been added and they pass
[x] Documentation has been updated

[ ] Upgrade instructions
[ ] Configuration details
[ ] Concepts

[ ] Changes flowforge.yml?

[ ] Issue/PR raised on FlowFuse/helm to update ConfigMap Template
[ ] Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production

Labels

[ ] Includes a DB migration? -> add the area:migration label

Functionally, great, but I'm not convinced on the "key" icon. Few reasons:

key icon to me speaks security key
key as an icon to refer to the key of an object may not translate well into other languages.
If we're to be consistent with the chart node, it should just be the text "key"

Removed key icon

Tests are passing. Failure is the usual:
  1) Node-RED Dashboard 2.0 - Control - Enable/Disable
       can enable/disable a particular page from the navigation options:
     AssertionError: Timed out retrying after 4000ms: expected '<a.v-list-item.v-list-item--link.v-list-item--nav.v-theme--nrdb.v-list-item--density-default.v-list-item--one-line.v-list-item--rounded.v-list-item--variant-text>' to have class 'v-list-item--disabled'
      at Context.eval (webpack://@flowfuse/node-red-dashboard/./cypress/tests/widgets/control.spec.js:137:51)

NOTE: Ideally we would fix this but for the time being, can we either disable this test or move the control tests to their own npm script and GH task so that we get clear passes on the other tests?


Feature delete

Delete photo support(by offbye). By set displayDelete=true show a delete icon in statusbar, deleted indexes can be obtain from delegate func didDeleted

thanks, I'll fix debug print sentence later.


Feature Request: Notification Sound Options (mute, change, etc).

Can you add a way to mute and/or change the notification sound within the application?
In Hangouts, there's an option to mute notifications for 1 hr, 8 hrs, 24 hrs... Something similar would be very helpful for me because I often use my PC for VoIP calls and have to close YakYak to prevent the notification sounds interrupting the call every minute.

Great idea, muting should be a matter of creating an option and checking in notifications.coffee around line 65 (not sure if it's there though)
...
sender: 'com.github.yakyak'
sound: viewstate.someOptionThatMutesSounds == false
, (err, res) ->
...

To mute it for some hours, would require to store in that option the date/time of the "unmute"
...
sender: 'com.github.yakyak'
sound: (!viewstate.someOptionThatMutesSounds || viewstate.someOptionThatMutesSounds < getTime())
, (err, res) ->
...

@ahpatel I'm actually working on a feature to add sound to notifications engines that don't support it and an option to disable sound all together
since you created the issue a new option was developed to turn notifications on or off in the tray icon and view menu
try the beta release of 1.4.0 at #451

partial duplicate of #381

@ahpatel mute is implemented in pull request #467


Coin Flip Command and Rock-Paper-Scissors Command

Description
Coin flipping and rock-paper-scissors are very common games that can be really interesting to play, especially on Discord. It would be a good choice to have them on Sir LanceBot, known for its simple but fun and interesting games.
Reasoning
These are simple games that can instantly cure boredom and can also be very engaging at times. It would be used by many on the server and it also deserves to be added along with the other games, such as tic-tac-toe and battleship.
Proposed Implementation
It is a pretty simple code and the outcome should be as shown in the images below. The images are of course not the final output, I will improvise the commands and modify the results according to your needs and requirements.
The commands would be as follows:
Coin flip command: flip with the aliases - toss and coinflip
Rock-paper-scissors command: rps with the alias - rockpaperscissors
Both commands can be used without an argument but adding an argument will take the user's argument into consideration, also concluding who won in the end.
Both of these commands will simply be made using the random module.

Would you like to implement this yourself?

[x] I'd like to implement this feature myself
[ ] Anyone can implement this feature

Sounds like a great idea , I appreciate that :D

Hello,
Thanks for the suggestions, they sound cool. Could you split these out into two issues though? They seem like separate features.
Be sure to check out our contributing guide for setting up the bot, and the style guide we use, before starting work. They can both be found here:
https://pythondiscord.com/pages/contributing/

Alright, so I'll change this one to the coin flip command and make another one for the rock-paper-scissors command.
I'll also be sure to check out the contributing guide.

👍 Sounds good to me, I've assigned you here.

@InvisibleOS may i collaborate with you on this project? It seems as if you've forgotten about this issue or has other things to do, so if you'd like I could work with you on this, making it easier.
My nickname on the Python Discord server is "wait... it's all ᓚᘏᗢ? | 🐈"
Best regards.

Hello,
I have already made both the commands (rock-paper-scissors and coin-flip) long ago. I have also shared the code in #dev-log but I didn't receive a definite reply which was why I accidentally forgot about it. I'm ready with the code now, I just need some help getting it added to Sir Lancebot.
Maybe we can discuss further on PyDis, I'll hit you up.

@InvisibleOS Hey so is this ready to be implemented, if so then feel free to make a pull request for the command so we can get it pushed.

@InvisibleOS Can you make a PR with your feature so we can start reviewing it?

@InvisibleOS has stated on discord that they'd like someone else to take over, so I'll do so. Have reassigned the issue accordingly


Error installing flagger on OpenShift

There is an issue installing Flagger on Openshift, specifically around the use of security context  in the deployment:

This is an issue seen in flux and fixed:
https://github.com/fluxcd/flux/pull/2652
We probably need an option in the chart to disable adding the security context:
https://github.com/fluxcd/flux/blob/master/chart/flux/templates/memcached.yaml#L55

I will work on this now

@richardcase the next release will have this RBAC needed by OpenShift: https://github.com/weaveworks/flagger/pull/537

@stefanprodan - thats great news.
I have tested with those rbca changes and its still an issue for me. There needs to be changes to the security context in the deployment (or perhaps the flagger service account needs to be added to a specific SCC in openshift). For Flux there was a option added so that the security context could be removed>

The security context and RBAC are unrelated. If you can please test the RBAC changes together with your PR on security context.

If you can please test the RBAC changes together with your PR on security context.

@stefanprodan - will do

@stefanprodan - looking into it we can get it working like we did with flux (i.e. have an option to disable the security context in the deployment) or add the following to the install instructions to allow pods running in the target flagger namespace to use any uid
oc adm policy add-scc-to-group anyuid system:serviceaccounts:istio-system

Any preference on which i do?

@richardcase Flagger doesn't run as root as Flux, there is no need for anyuid. I see that you've added the option to disable the security context in the Helm chart, does that work? Is Flagger able to run a canary analysis?

@stefanprodan - good point. I'm going to test today and will let you know.

Going to close this as 1.0.0-rc.4 contains both RBAC and Helm chart fixes for OpenShift. Please open a new issues if there are any problems on OpenShift.


[Question] Programmatic API

Can we use evillimter programmatically?
Here's how the code might looks like:
import evillimiter as el
el.add('<IP_ADDRESS>') # ID:0
el.block(0)
input('Press Enter to free all hosts and exit ...')
el.free('all')
print('Bye.')

Same issue?

As of now, EvilLimiter is written in a non-modular way, which means there is no separation between the core functionality, CLI output, etc. I thought about re-writing EvilLimiter entirely solving all these issues. I might be working on this soon.
I will add this request to the wishlist.


[eas-cli] rollout tests

Why
Rollout tests i didnt get around to writing during launch week
Test Plan

[ ] new tests pass

/changelog-entry chore added rollout tests


Validate the format of a directory column

MetadataConstraints has been updated to validate the format of the directory column. I noticed this validation was missing when looking at #4857
I made a comment here about why this is targeted for main currently and not for 3.1. I can backport or target this in 3.1 once we decide how to handle the constraint code number (assuming we want to put this change into 3.1)

I went ahead and backported to 3.1 and kept the number as 17 for the error code. I talked to @keith-turner and I'm going to open an issue to refactor the error codes for validation and have the new error codes that only apply to main (elasticity) start at a higher number such as 100 so we have a cap for future errors that apply to 3.1 and 4.0 and don't run into this issue again.


Eval code lens errors should display path relative to workspace directory, not the full URL

When an error is encountered during Eval, the error shown (which is really nice, btw!) should not show the full ("internal") URL, but a path relative to the workspace directory. In the example above, this would be bundle/regal/rules/idiomatic/directory_package_mismatch.rego

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."


usability: django, pytest-pythonpath; unhelpful test-time dependency management

I'm not sure what to do with this but I feel like I should at least put it somewhere;
It's possible to waste a lot of time trying to figure out when packaging something, and not intimately familiar with the stuff, why django tests fail with some weird ImportErrors when a project uses pytest-pythonpath.
I ran into this for packaging cuckoo sandbox: https://github.com/cuckoosandbox/cuckoo/issues/2676
I'm not sure what to do about this, but at the least, it's not helpful if packages don't warn about missing
dependencies. So maybe it should be documented to check extra carefully about missing test infrastructure dependencies, because pytest will not warn you if you're missing any pytest plugins.

If a package requires certain test dependencies, but does not declare them, then indeed you'll get errors. Nothing we can do about this.


feat: 支持函数URL

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


the "Test" function does not work

the regexp is: .+\s([a-zA-Z]+\s?)
test: DSad SDA 1123
expected: shows error (red)
actual: did not see any error for it

/.+\s([a-zA-Z]+\s?)/.test('DSad SDA 1123') // true

When using the constructor function, the normal string escape rules (preceding special characters with \ when included in a string) are necessary:
new RegExp('.+\\s([a-zA-Z]+\\s?)').test('DSad SDA 1123') //true

I am not sure if I express clear, what I mean here is: when I try the above regex in: https://regex-vis.com/ ,  I input the regex, then I input the test string under the top right Test tab, I hope it shows error, but it doesn’t

The test string DSad SDA 1123 and the RegExp .+\s([a-zA-Z]+\s?) are matched.

So there are no errors.

But the meaning of the regex should only match “DSad SDA ”, 1123 should not matched by the regex? Why it still shows true?

You should add an end assertion $.
/.+\s([a-zA-Z]+\s?)$/.test('DSad SDA 1123') will return false

Got it! Thanks!


⚠ Avatar API Endpoint has degraded performance

In 09dd0f0, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1015 ms

Resolved: Avatar API Endpoint performance has improved in ce32f57 after 13 minutes.


how to pause between two words ?

I have use  "< break time="3s" / > " in my text file, but it's seems not work.
How can I make short pause between two words ?

Hi @hulk10425, can you provide more detailed information? e.g. Which model you are following (SpeechT5 or others?)

I installed from here
https://huggingface.co/microsoft/speecht5_tts
and run it with script provided on point 3 "Run inference via the Transformers modelling code"
Can you tell how to use < break time="3s" /> ?
How to add it to this script? I don't have IT background, so I can't figure it out myself.
Is it also possible to control speed with ssml tags?
By the way, is there any ready to use MS TTS model of British English?


Error-Log-Level color is green in v 3.0.0

Hi @alessandro-bottamedi
thx for the new release but unfortunately error loggs are now printed in a green color which doesn't indicates an error on the first look.

Screenshots shows transportOptions.color:

null
ansi
web

Woops, fixed in 3.0.2 ;)

Woops, fixed in 3.0.2 ;)

I will add the possibility to change colors for each level in next release...

I will add the possibility to change colors for each level in next release...


Duplicated tests in LambdaBuilder

The tests named WithType and WithoutType in LambdaBuilder are identical.

Thank you for you careful eyes, I'll look into all 4 of these hopefully over the next weekend :)
Unless you want to provide me with a patch ...?

I am finding a lot of issues because I am porting Lizzie to TypeScript, which means I get to check every line of code. When I have something working then I will open source it here on GitHub.
Alas, this won't leave me a lot of time for pull requests here. But I will continue to post issues. :-)

Fixed!


no images showing for uberbooru.com

whenever I search a tag with Grabber using uberbooru (runs danbooru v2), no pictures show up and it says "uberbooru.com - Page 1 of ? (20 of ?)"
It seems that https://uberbooru.com/posts?tags=help%3Aapi might have a solution, but I don't really know what i'm doing, so the answer could be obvious to someone like you who knows what they're doing.
Thanks for the help!

I can reproduce the "page 1 of ?" issue, since this source does not give the last page information right away (so the "get all" button works but the display shows "?").
As for the "no pictures show up", I can't reproduce, the thumbnails show up fine for me, even during searches.


Update tutorial image on Website

What's wrong with the current image?
Is it that the file in the editor suggests that one can have more than one module per file?

What's wrong with the current image?

It's just old? :-)

Done.
It doesn't look much newer.

:) thanks. It was just showing files that didn't exist anymore. Superficial indeed.


Wrong IMDB_MOVIE_DESC regex pattern for movies without a description

While I was testing some movies, I stumbled upon the extremely rare case where a movie lacks a description...
In that case, something very strange happens... At first, the pattern fails to match the default text that IMDB puts in place of the description (Know what this is about? Be the first one to add a plot.), which has to do with the newline symbols in the  Array
        (
            [name] => Description
            [value] => Trelles diakopes tou thiriotrofeiou (1985) if ('csm' in window) { csm.measure('csm_body_delivery_started'); } Reference View | Change View 1h 12min | Comedy | 1985 (Greece) | Video | 0 Rate 1 Rate 2 Rate 3 Rate 4 Rate 5 Rate 6 Rate 7 Rate 8 Rate 9 Rate 10 Rate 0 Error: please try again. | Know what this is about? Be the first one to add a plot.
        )

can be seen at the very beginning of the matched text @1:20 of the video as soon as I deliberately activated the s flag.
Even if you google the term csm_body_delivery_started you will find a whole bunch of results were obviously movies which lacked a description where scraped, and no one noticed the wrong text that was brought by this or similar classes!
Now, at the good news, I've come up with the correct pattern that fixes this bug.
'~
\s+
\s*+(.*)\s*?
\s+
\s+
~Uis'
The above pattern matches (as you can see in the video) the description, as well as the text that IMDB puts in place of a description, which prompts the user to enter a description of their own.
After that, all that is left is that you preg_match something like Know what this is about? or https://contribute.imdb.com/updates?update= in order to identify the prompt instead of an actual description in order to return the $sNotFound text...
Please also notice that from what I suspect (though I can't confirm as I can't find any series without a description), so I simply take an educated guess based on the patterns that look similar, the same bug may be affecting IMDB_SERIES_DESC, if someone stumbles upon a series that lacks a description...

Thanks for the detailed description 👍🏼 i will fix it.

@PrinceOfAbyss: Seriously, this was by far the best bug report this project ever saw. Thanks a lot!
@bla0r: Again, thanks for taking care. :)


Support for<EMAIL_ADDRESS>

I was trying to use latest version of Hardhat(v2.22.3) with hardhat-ethers plugin. The hardhat-ethers plugin is compatible with latest version of ethers.js(v6.0.0). But when I tried to install hardhat-deploy plugin(v0.12.2)  and hardhat-deploy-ethers plugin, I saw some errors.

It seems hardhat-deploy plugin is still not compatible with ethers.js(v6.0.0).
If my understanding is correct, when can we expect support for<EMAIL_ADDRESS>

what do you mean by hardhat-deploy do not support ethers.js v6 ?
What error do you see ?
hardhat-deploy is agnostic to which lib you use and  it do not expose any lib specific api and you can use any you like

@wighawag  Thank you for your reply.
hardhat-deploy is agnostic to which lib you use and it do not expose any lib specific api and you can use any you like
Is this the same in case of hardhat-deploy-ethers package as well?

Is this the same in case of hardhat-deploy-ethers package as well?

No  hardhat-deploy-ethers is here to use ethers with hardhat-deploy.
The error seems to indicate that hardhat-deploy-ethers expect an earlier version of hardhat-deploy
If this is case, please open an issue here instead : https://github.com/wighawag/hardhat-deploy-ethers

Opened #37.
Closing this one.


[HOLD PR#35880][$500] IOU - Total amount is only grayed out after revisiting IOU report when creating IOU offline

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v.1.4-39-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4295316
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to staging.new.expensify.com
Go to FAB > Request money > Manual.
3, Create a manual request from a user.
Go offline.
In 1:1 DM, click on the IOU preview to go to IOU report.
Click + > Request money.
Create a manual request in local currency.
Create another manual request in foreign currency.
Go to main chat and return to IOU report.

Expected Result:
In Step 8, the total amount is grayed out because request with foreign currency is created offline.
Actual Result:
In Step 8, the total amount is not grayed out.
In Step 9, after revisiting the report, the total amount is grayed out.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/973ddf07-da6a-4650-b8a8-949563a493ca
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01eeb6c185339b648d
Upwork Job ID:<PHONE_NUMBER>970962944
Last Price Increase: 2024-02-09

We think that this bug might be related to #vip-split-p2p-chat-groups
CC @gabrielessner

Proposal
Please re-state the problem that we are trying to solve in this issue.
The total amount of text doesn't grayed out after adding a new request with a different currency while offline.
What is the root cause of that problem?
The text will be grayed out if hasUpdatedTotal returns false.
https://github.com/Expensify/App/blob/04fc8865504eca69e9d03d9023bebd9418baba76/src/components/ReportActionItem/MoneyReportView.tsx#L121-L125
The issue is, hasUpdatedTotal depends on transaction data.
https://github.com/Expensify/App/blob/04fc8865504eca69e9d03d9023bebd9418baba76/src/libs/ReportUtils.ts#L4672-L4682
But the component (MoneyReportView) doesn't subscribed to (or have props of) transaction onyx data, so it won't re-render.
What changes do you think we should make in order to solve the problem?

First of all, we should connect the transaction onyx data.
withOnyx({
    transactions: {
        key: ONYXKEYS.COLLECTION.TRANSACTION,
    },
})(MoneyReportView);

Then, instead of using getAllReportTransactions to get the transaction list, we can pass the transactions props to hasUpdatedTotal. We can filter before passing it through selector or inside hasUpdatedTotal itself (I think the selector is better).
If we concern with the performance, we can create a view wrapper that will calculate the hasUpdatedTotal value and pass it down, so MoneyReportView will only re-render if hasUpdatedTotal value changes.

This might be resolved by #35880 – where we handle the total update when the request with a different currency is made. cc: @cubuspl42 could you please check if this might be related?

I agree that it could have the same root cause. That PR should be merged soon, I gave it a bump!

Thanks @cubuspl42. We can have a check back once the PR is merged.

I put the issue on hold for https://github.com/Expensify/App/pull/35880, hopefully that one fixes this!

I also removed Help Wanted temporarily while we wait for that issue.

https://github.com/Expensify/App/pull/35880 has been deployed to prod as of yesterday. I added the 'retest-weekly' label so we can see if this is still reproduceable.

I reached out to #qa on Slack to see if they were able to retest this: https://expensify.slack.com/archives/C9YU7BX5M/p1709589428240689

Issue not reproducible during KI retests. (First week)

Looks like the other PR fixed this issue, nice! I'm going to close this issue out, please bump me on Slack if I missed anything.


BSC#1093565: Dynamically adjust cluster growth based on existing cluster size

Backport of #570 to release-2.1.

Please add the release-2.1-backport label.


Fix jupyter-ssh instructions in 01_interface.ipynb to account for '@' in usernames

Open a terminal and ssh with port 8022 ssh<EMAIL_ADDRESS>-p 8022

This will break with default Nebari usernames that are based on email addresses (i.e. username =<EMAIL_ADDRESS>because of of the extra @ symbol.
Instead use: ssh demo.nebari.dev -l <username> -p 8022

Wrong repo.


Add homebrew install

Hi @gkpln3
I was updated new cask to Homebrew, this application is awesome, and now it's can be installed with brew CLI
https://github.com/Homebrew/homebrew-cask/pull/140401
brew install --cask capslocknodelay

Please add this to README doc. Thanks so much

Great! thanks!


Devices compatible with ewelink with more options than just on/off

Dear Howanghk,
Thank you for this great plugin, which is simple to use!
I have bought some ewelink compatible devices I wanted tot try:

simple on/off touch: works great with the plugin
dimmer (on/off brightness): only on/of switch appears in homekit, no brightness
curtain controller(close/open/pauze): only on/of switch appears in homekit for open and close.

when there are extra functions (different service) to the the switch they are not recognised by homekit and therefor they can't be used in homekit.
Is there a possibility this can be changed?
Thnks!

Hi @Leonoerlemans, this plugin only support basic on / off switch now, as that is what the original author gbro115 write this plugin for. Then a contributor MrTomAsh added support for multi-channel device. I am the third major contributor to this plugin, I added login support so that users won't have to know how to use Charles to make this plugin work. As you can see, none of the author and contributors have a device with those extra functions. Therefore none of us wrote support for those functions. If you happen to have those devices, please consider adding support for those functions! This project is open source and have a relaxed license that you are free to modify the code and share your work. You can look into the source code, it's written in Javascript which is fairly easy to understand and write. Once you're done with the extra functions, you are welcome to send a pull request so the functions can be merged into this fork.

Thank you @howanghk for your reply. My Java is not good, infact I just bought a book to understand.  So I am far from adjusting this plugin. I hoped someone with the same devices could contribute to a solution. I also do understand that you can’t honor every request and solve someone else’s issues.
Thank you anyway for this fork
Leon

Hi @Leonoerlemans, javascript and Java are two completely different things. There are tons of free javascript learning material on the Internet. Just google for "learn javascript" and pick the one you feel comfortable to start with. You would need to know the basics only, as most tutorial assume we use javascript with a web page, but in our case we use javascript "server-side", which don't attach to a web page.

Thank you @howanghk for your reply. My Java is not good, infact I just bought a book to understand. So I am far from adjusting this plugin. I hoped someone with the same devices could contribute to a solution. I also do understand that you can’t honor every request and solve someone else’s issues.
Thank you anyway for this fork
Leon

Hello, I followed your conversation! I am exactly in the same case as you, I bought a switch for a curtain and I would like to see it work like this one
https://www.youtube.com/watch?v=WF0hb_ZBf3E
I do not know anything about programming I'm looking for solutions to my level, I 'hope someone can help us. Sorry for English I translated with Google.


[Feature request] Support for DeepL translator.

Also see #121 and #272. I too would like to see support for DeepL added again

And I'm fine with paying their subscription fee

hi
prev. #268
but can  you implement deepl incl personal api token (e.g ~/.translate-shell/API_deepl.com.cfg)?
they just want to track people, and "normal" usage never would break a limit. thats why they does restrictions. because content worker did mass translations. :-(


xformers attention implementation

I'm working on an attention backend based on xformers to improve performance on V100; is there anything I need to be aware of when doing so or should it be straightforward?

The most straightforward way might be repalce the implemention of FlashAttention by xformers' attention and remove the flash attention dependency if necessary.
May check these two classes:
https://huggingface.co/OpenGVLab/InternVL2-26B/blob/a515152d7186e32fc7d562d3ccc086c423b7728d/modeling_intern_vit.py#L40
https://huggingface.co/OpenGVLab/InternVL2-26B/blob/a515152d7186e32fc7d562d3ccc086c423b7728d/modeling_internlm2.py#L430
Don't forget to run unit test to verify the output of replaced attention are all close (torch.all_close)


Size specification of jpg images

We should predefine our jpg images size

Main menu :   350 w x 200h
Help :  1200 w x 600 h
Credits :  512 w x 256 h

and fix the front-end preview images sizes, so that every images is shown without clipping

Taso, whicj jpg sizes are good for the scene tiles in "Project Editor" page ?

@jimver04 There is no stanadard size. It's adaptive.


Avoid early realization of compile tasks registered on source set output.classesDirs

Currently, any provider added to SourceSetOutput.classesDirs will get realized due to the BuildOutputCleanupRegistry declaration in JavaPlugin:
https://github.com/gradle/gradle/blob/80fc37d360ea54539315d89b321c2822f0f35efe/subprojects/plugins/src/main/java/org/gradle/api/plugins/JavaPlugin.java#L268-L270
We avoid this for our own first-party plugins by controlling the compilation output directly, externally from the compile task, adding that value to the classesDirs without a mapped task provider, and then using buildBy to register the task dependency. This indirect relationship is non-ideal since changes to the output directory of the compile task are not reflected in classesDirs. See: https://github.com/gradle/gradle/blob/80fc37d360ea54539315d89b321c2822f0f35efe/subprojects/plugins/src/main/java/org/gradle/api/plugins/internal/JvmPluginsHelper.java#L122
This procedure is not ideal, since it is not intuitive. Users should be able to register task providers directly on classesDirs without having their tasks realized early. To avoid this, we should consider not adding source set outputs to the BuildOutputCleanupRegistry and instead relying on the fact that the default compilation output is configured within the build directory, which is registered with the BuildOutputCleanupRegistry. However, this means users which reconfigure compile task outputs to outside the build directory will not have their compilation outputs cleared during the clean task. This may be acceptable.
After a solution is found, we should properly document the procedure for adding classes for secondary compile tasks to classesDirs. The procedure should likely be documented in building_java_projects.adoc#sec:compile
Additionally, as part of this work, we should consider deprecating SourceDirectorySet#compiledBy, since conceptually a source directory set should not be aware of "what compiles it". Especially given that there may be multiple tasks which compile a given source directory set -- for example with multi-target JVM compilation.

We would need to deprecate the existing behavior of deleting classes outside the build directory when running clean.


Synthesis failed for nodejs-pubsub

Hello! Autosynth couldn't regenerate nodejs-pubsub. :broken_heart:
Here's the output from running synth.py:
Cloning into 'working_repo'...
Switched to a new branch 'autosynth'
Running synthtool
['/tmpfs/src/git/autosynth/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2020-03-20 04:38:13,676 synthtool > Executing /tmpfs/src/git/autosynth/working_repo/synth.py.
2020-03-20 04:38:13,729 synthtool > Ensuring dependencies.
2020-03-20 04:38:13,734 synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:5ef340c8d9334719bc5c6981d95f4a5d2737b0a6a24f2b9a0d430e96fff85c5b
Status: Image is up to date for googleapis/artman:latest
2020-03-20 04:38:15,358 synthtool > Cloning googleapis.
2020-03-20 04:38:16,017 synthtool > Running generator for google/pubsub/artman_pubsub.yaml.
2020-03-20 04:38:20,973 synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/pubsub-v1.
.eslintignore
.eslintrc.yml
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/publish.yml
.github/release-please.yml
.github/workflows/ci.yaml
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/lint.cfg
.kokoro/continuous/node10/samples-test.cfg
.kokoro/continuous/node10/system-test.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node12/common.cfg
.kokoro/continuous/node12/test.cfg
.kokoro/continuous/node8/common.cfg
.kokoro/continuous/node8/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node10/docs.cfg
.kokoro/presubmit/node10/lint.cfg
.kokoro/presubmit/node10/samples-test.cfg
.kokoro/presubmit/node10/system-test.cfg
.kokoro/presubmit/node10/test.cfg
.kokoro/presubmit/node12/common.cfg
.kokoro/presubmit/node12/test.cfg
.kokoro/presubmit/node8/common.cfg
.kokoro/presubmit/node8/test.cfg
.kokoro/presubmit/windows/common.cfg
.kokoro/presubmit/windows/test.cfg
.kokoro/publish.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.mocharc.js
.nycrc
.prettierignore
.prettierrc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
renovate.json
samples/README.md
2020-03-20 04:38:21,159 synthtool > Replaced '  }\n\\s*/\\*\\*\n\\s+\\* The DNS address for this API service\\.' in src/v1/subscriber_client.js.
2020-03-20 04:38:21,161 synthtool > Replaced '../../package.json' in src/v1/publisher_client.js.
2020-03-20 04:38:21,162 synthtool > Replaced '../../package.json' in src/v1/subscriber_client.js.
2020-03-20 04:38:21,163 synthtool > Replaced 'https:\\/\\/cloud\\.google\\.com[\\s\\*]*http:\\/\\/(.*)[\\s\\*]*\\)' in src/v1/doc/google/protobuf/doc_timestamp.js.
2020-03-20 04:38:21,164 synthtool > No replacements made in src/**/doc/google/protobuf/doc_timestamp.js for pattern toISOString\], maybe replacement is no longer needed?
npm WARN npm npm does not support Node.js v12.16.1
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11.
npm WARN npm You can find the latest version at https://nodejs.org/
npm WARN deprecated<EMAIL_ADDRESS>Legacy versions of mkdirp are no longer supported. Please update to mkdirp 1.x. (Note that the API surface has changed to use Promises in 1.x.)
npm WARN deprecated<EMAIL_ADDRESS>core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.

><EMAIL_ADDRESS>install /tmpfs/src/git/autosynth/working_repo/node_modules/grpc
> node-pre-gyp install --fallback-to-build --library=static_library

node-pre-gyp WARN Using needle for node-pre-gyp https download
[grpc] Success: "/tmpfs/src/git/autosynth/working_repo/node_modules/grpc/src/node/extension_binary/node-v72-linux-x64-glibc/grpc_node.node" is installed via remote

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js on Open Collective or Patreon:
> https://opencollective.com/core-js
> https://www.patreon.com/zloirock

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/protobufjs
> node scripts/postinstall

><EMAIL_ADDRESS>prepare /tmpfs/src/git/autosynth/working_repo
> npm run compile

npm WARN npm npm does not support Node.js v12.16.1
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11.
npm WARN npm You can find the latest version at https://nodejs.org/

><EMAIL_ADDRESS>compile /tmpfs/src/git/autosynth/working_repo
> tsc -p . && cp -r src/v1 build/src/ && cp -r protos build/ && cp -r proto build/ && cp smoke-test/*.js build/system-test && cp test/*.js build/test

npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

added 798 packages from 869 contributors and audited 2154 packages in 27.518s
found 3 low severity vulnerabilities
  run `npm audit fix` to fix them, or `npm audit` for details
npm WARN npm npm does not support Node.js v12.16.1
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11.
npm WARN npm You can find the latest version at https://nodejs.org/

><EMAIL_ADDRESS>fix /tmpfs/src/git/autosynth/working_repo
> eslint --fix '**/*.js' && gts fix

installing<EMAIL_ADDRESS>installing<EMAIL_ADDRESS>installing<EMAIL_ADDRESS>2020-03-20 04:39:08,391 synthtool > Wrote metadata to synth.metadata.
Changed files:
M .github/workflows/ci.yaml
 M .kokoro/samples-test.sh
 M .kokoro/system-test.sh
 M .kokoro/test.sh
 M README.md
 M protos/google/pubsub/v1/pubsub.proto
 M protos/protos.d.ts
 M protos/protos.js
 M protos/protos.json
 M src/v1/doc/google/pubsub/v1/doc_pubsub.js
 M src/v1/publisher_client.js
 M src/v1/subscriber_client.js
 M synth.metadata
 M test/gapic-v1.js
?? .mocharc.js
[autosynth 5a93d89] [CHANGE ME] Re-generated  to pick up changes in the API or client library generator.
 15 files changed, 675 insertions(+), 101 deletions(-)
 create mode 100644 .mocharc.js
 rewrite synth.metadata (81%)
To https://github.com/googleapis/nodejs-pubsub.git
 + 1ab211f...5a93d89 autosynth -> autosynth (forced update)
Traceback (most recent call last):
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/urllib3/connectionpool.py", line 672, in urlopen
    chunked=chunked,
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/urllib3/connectionpool.py", line 421, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/urllib3/connectionpool.py", line 416, in _make_request
    httplib_response = conn.getresponse()
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/http/client.py", line 1346, in getresponse
    response.begin()
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/http/client.py", line 307, in begin
    version, status, reason = self._read_status()
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/http/client.py", line 276, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/urllib3/connectionpool.py", line 720, in urlopen
    method, url, error=e, _pool=self, _stacktrace=sys.exc_info()[2]
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/urllib3/util/retry.py", line 400, in increment
    raise six.reraise(type(error), error, _stacktrace)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/urllib3/packages/six.py", line 734, in reraise
    raise value.with_traceback(tb)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/urllib3/connectionpool.py", line 672, in urlopen
    chunked=chunked,
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/urllib3/connectionpool.py", line 421, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/urllib3/connectionpool.py", line 416, in _make_request
    httplib_response = conn.getresponse()
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/http/client.py", line 1346, in getresponse
    response.begin()
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/http/client.py", line 307, in begin
    version, status, reason = self._read_status()
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/http/client.py", line 276, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
urllib3.exceptions.ProtocolError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 509, in <module>
    main()
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 356, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 470, in _inner_main
    args.repository, branch=branch, title=pr_title, body=build_pr_body(synth_log)
  File "/tmpfs/src/git/autosynth/autosynth/github.py", line 62, in create_pull_request
    "maintainer_can_modify": True,
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/requests/sessions.py", line 578, in post
    return self.request('POST', url, data=data, json=json, **kwargs)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/requests/sessions.py", line 530, in request
    resp = self.send(prep, **send_kwargs)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/requests/sessions.py", line 643, in send
    r = adapter.send(request, **kwargs)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/requests/adapters.py", line 498, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


wLE > neutrino > udpbd-vexfat

Is it possible to use wLE > neutrino > udpbd-vexfat? UDPBD-Server doesn't work anymore because the latest version for Windows is still not compiled (more details here). Also it is not as user-friendly as udpbd-vexfat. By the way, is it possible to implement the following algorithm:

Set up config files (only required once).
Starting udpbd-vexfat.
Running neutrino via wLE.
Connecting neutrino to udpbd-vexfat.
Displaying a simple list of games (white letters on a black background as in DKWDRV).

By the way, is it possible to make such an algorithm (it already works for udpbd-server) for udpbd-vexfat:

If someone makes a GUI that can list games from original udpbd-server, then it should also work on udpbd-vexfat.

Can you do this please?

I've got too much things to do, and too little time. If I ever find the time to make a UI/GUI then I will, but don't count  on it.

Also, is it possible to change neutrino so that it works again via OPLNEUTRINO UDPBD?

You can ask crt0 to open source OPLNEUTRINO. He should do that anyway. Then changing it to be compatible with the latest neutrino would be easy. But what's the incompatibility anyway? As I understand it OPLNEUTRINO should support neutrino 1.2.0, and the command line interface should still be compatible.

@10522 I will answer on behalf of repo owner. No, nothing you suggest is not possible. App creator should not deal with hacks that are not related to his project, it's hack creator responsibility to make all this work on his side. Re-asm only sounds interesting, but in fact it's the most difficult programming in the world. The level of complexity is similar to, for example, if someone asks you to add usb support inside PS2 game, like "make me Ratchet and Clank to load my customs skin from usb". When you see words like re-asm, hacking without source, level of complexity is about that high.