build error

I have used 1.0.5 version and I upgraded 2
0.0(latest) now.
But I have a problem which is about build error.
[cordova]  FAILURE: Build failed with an exception.
[cordova]
[cordova]  * Where:
[cordova]  Script '***/app/platforms/android/cordova-plugin-firebase/app-build.gradle' line: 27
[cordova]
[cordova]  * What went wrong:
[cordova]  A problem occurred evaluating project ':app'.
[cordova]  > Plugin with id 'com.google.gms.google-services' not found.
[cordova]
[cordova]  * Try:
[cordova]  Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
[cordova]
[cordova]  * Get more help at https://help.gradle.org
[cordova]
[cordova]  BUILD FAILED in 0s
[cordova]  (node:82096) UnhandledPromiseRejectionWarning: Error: /app/platforms/android/gradlew: Command failed with exit code 1 Error output:
[cordova]  FAILURE: Build failed with an exception.
[cordova]
[cordova]  * Where:
[cordova]  Script '/app/platforms/android/cordova-plugin-firebase/app-build.gradle' line: 27
[cordova]
[cordova]  * What went wrong:
[cordova]  A problem occurred evaluating project ':app'.
[cordova]  > Plugin with id 'com.google.gms.google-services' not found

cordova version: 8.0.0

+1 same here, fresh windows install, same for npm, cordova, the project & all the plugins

My app works when I used @1.0.5 version.
I think It is better to use older version. do yo know where the release-version list is?

Release list: https://github.com/arnesson/cordova-plugin-firebase/releases
We only recently started a CHANGELOG to track the exact changes to each version

+1 I tried everything to try to figure out what the problem is, no luck so far

I believe the issue goes back to a Pull Request for Crashlytics that removed the google play service dependency.  Based on the FIrebase documentation and the issues reported by the community, this should resolve the issue.
If someone could confirm by testing this code by doing the following:
cordova plugin remove cordova-plugin-firebase  (uninstall current firebase plugin)
https://github.com/briantq/cordova-plugin-firebase.git#google-services (install the proposed fix)
Please let me know if that resolves the issue.  If it does, we can get it merged.  If not, I can try to make additional changes.

I just tried it and have the same error. The plugins I used for testing are:
    <plugin name="cordova-plugin-ionic-webview" spec="^1.2.1" />
    <plugin name="cordova-plugin-splashscreen" spec="git+https://github.com/apache/cordova-plugin-splashscreen.git" />
    <plugin name="cordova-universal-links-plugin" spec="git+https://github.com/alexstanbury/cordova-universal-links-plugin.git" />
    <plugin name="cordova-custom-config" spec="^4.0.2" />
    <plugin name="cordova-plugin-whitelist" spec="^1.3.3" />
    <plugin name="cordova-plugin-statusbar" spec="^2.4.2" />
    <plugin name="cordova-plugin-googleplus" spec="^5.2.3">
...
    </plugin>
    <plugin name="cordova-plugin-firebase" spec="git+https://github.com/briantq/cordova-plugin-firebase.git#google-services" />
    <plugin name="cordova-android-firebase-gradle-release" spec="^1.0.2">
        <variable name="FIREBASE_VERSION" value="15.+" />
    </plugin>
    <plugin name="cordova-android-support-gradle-release" spec="^1.4.4">
        <variable name="ANDROID_SUPPORT_VERSION" value="27.+" />
    </plugin>

@briantq Thanks for putting together a proposed solution here! I attempted and resulted in the following error:
* What went wrong:
The library com.google.android.gms:play-services-measurement-base is being requested by various other libraries at [[15.0.2,15.0.2]], but resolves to 15.0.4. Disable the plugin and check your dependencies tree using ./gradlew :app:dependencies.

This is going to sound quite weird but I've been able to replicate getting it working or not by the order which I add platforms. If the config.xml contains the plugins and you 'platform add ios' and then 'platform add android' - android build won't work. However if you 'platform add android' then 'platform add ios' ... it works! What the... seems like some lifecycle hook is in the wrong place
so try
platform rm ios
platform rm android

platform add android
platform add ios

I'm using cordova 8 and latest platform versions...
    <engine name="ios" spec="^4.5.5" />
    <engine name="android" spec="^7.1.1" />

@jpduckwo Tried that, doesn't work for me

@willmero Thanks for testing.  It sounds like like you are encountering classic dependency version hell :)  Did you run the command to see what other plugin is referencing Google Play Services?  As far as dependency versions, I added it with the same line that was in the repo a month ago prior to the problematic pull request, which is without a version simply a plus sign.
One other interesting note is that the version that should be getting pulled is version 4.1.0 according to the documention.  If you are resolving to 15.X, I think you have another issue somewhere...

@briantq
hi! I've tried to install your fix but no luck, same error

What went wrong:
A problem occurred evaluating project ':app'.

Plugin with id 'com.google.gms.google-services' not found.

in my project.properties com.google.gsm.google-services is present
cordova.system.library.5=com.google.gms:google-services:+

@lucabro81 have you tried installing the cordova-android-play-services-gradle-release plugin? That is what solved the above error for me.
@briantq sigh yeah, don't you just love it? Thanks for your help here. I managed to get your branch working after installing the following plugins and set their versions in the config.xml file:
cordova-plugin-firebase:+
cordova-android-support-gradle-release:27.+
cordova-android-play-services-gradle-release:+
Running on Cordova android 7.0.0

I've dug a bit deeper because I was able to replicate as described above. What I found was that the following section was sometime missing in platforms/android/app/build.grade... for me it was around line 95 in the file.
missing section
// Fabric Cordova Plugin - Start Fabric Build Tools
buildscript {
    repositories {
        maven { url 'https://maven.fabric.io/public' }
        maven { url 'https://maven.google.com' }
    }
    dependencies {
        classpath 'io.fabric.tools:gradle:1.25.4'
        classpath 'com.google.gms:google-services:+'
    }
}

apply plugin: 'io.fabric'
apply plugin: 'com.google.gms.google-services'

Possible it could be to do with the order plugins are being added and one overriding the other...
my plugins are
    <plugin name="cordova-plugin-ionic-webview" spec="^1.2.1" />
    <plugin name="cordova-plugin-splashscreen" spec="^5.0.2" />
    <plugin name="cordova-custom-config" spec="^4.0.2" />
    <plugin name="cordova-plugin-whitelist" spec="^1.3.3" />
    <plugin name="cordova-plugin-statusbar" spec="^2.4.2" />
    <plugin name="cordova-plugin-firebase" spec="^2.0.0" />
    <plugin name="cordova-plugin-googleplus" spec="git+https://github.com/EddyVerbruggen/cordova-plugin-googleplus.git">
    </plugin>
    <plugin name="cordova-universal-links-plugin" spec="git+https://github.com/veloce/cordova-universal-links-plugin.git" />

I am learning more :)  I believe there are additional modifications that are required.  Now that I can recreate the issue, I can see what I can come up with.

Thanks @briantq! Yes that's right - I am referencing the release branch - however I did this on purpose because I could get it working with the release branch depending on the order that my plugins were installed and the proposed fix branch didn't fix the bug in question.

@briantq https://cordova.apache.org/docs/en/latest/guide/appdev/hooks/
'after_platform_add' hook might be needed to trigger the stuff that happens in after_plugin_add for the current hook... just a guess

Thanks @jpduckwo  I am struggling to find the time tonight, but I do know v1.0.5 does work.  Additionally, here are the files that were changed (warning, there are quite a few) that broke the functionality.
I have created the android platform with v1.0.5 and v2.0.0 and then am diffing them.  That is where I found it was more than just the build.gradle.  With that diff, along with the files that changed, I think there in lies the solution.
If you have the time and want to take a stab at it, you can fork the repo, make your changes in a branch and then install your branch (the same way I was suggested you install my branch).  If you can get it working, then simply turn your branch into a pull request and I will cancel mine.

Had the same issue Plugin with id 'com.google.gms.google-services' not found.
To make the build work and afterwards my app work, I had to do the following:

In my project level build.gradle I add the following:

buildscript {
    repositories {
....
        maven {
           url 'https://maven.fabric.io/public'
        }
...
    }
    dependencies {
...
        classpath 'com.google.gms:google-services:4.1.0' // google-services plugin
        classpath 'io.fabric.tools:gradle:1.25.4'
...

    }
}

allprojects {
    repositories {
...
        google()
...
    }

}

In app/build.gradle I added

apply plugin: 'io.fabric' (directly under apply plugin: 'com.android.application')
And now the build is successful AND the app loads

I fixed it.  It took a long while to unwind the code to figure out that we had incomplete fix on top of incomplete fix along with learning all about Cordova's build framework and all the scripts in this plugin, but this should resolve the issue.
@lucabro81 @willmero @jpduckwo could you guys test this new branch?  Since I could test it before submitting, I have much higher confidence in this.
cordova plugin remove cordova-plugin-firebase (uninstall current firebase plugin)
cordova plugin add https://github.com/briantq/cordova-plugin-firebase.git#android-build (install the proposed fix)
@paulstelzer It is not recommended that you modify the build files by hand.  This requires that you check in all the native code files otherwise the next time you generate the platform your changes will be gone.  CHECKING IN PLATFORM FILES (ESPECIALLY BUILD SCRIPTS) IS AN ANTI-PATTERN after Cordova 4.3.0.  Documentation states : "This feature (platform management} allows developers to save and restore their app to a known state without having to check in all of the platform and plugin source code."   I would strongly encourage you not to take such work arounds as they will cause more complication in the future.  Just my 2 cents.

@briantq I know that it's not good to edit it manually, but now you found an automatic way. I just want to say you what was missing and was causing the issue ;)

@paulstelzer Thanks :)  I started on Cordova less than 2 months ago and trying to figure out what to check in and not to check in took some investigation.  I think some times people take the easiest approach not knowing any better so I try to inform people about best practices when possible.

I fixed it. It took a long while to unwind the code to figure out that we had incomplete fix on top of incomplete fix along with learning all about Cordova's build framework and all the scripts in this plugin, but this should resolve the issue.
@lucabro81 @willmero @jpduckwo could you guys test this new branch? Since I could test it before submitting, I have much higher confidence in this.
cordova plugin remove cordova-plugin-firebase (uninstall current firebase plugin)
cordova plugin add https://github.com/briantq/cordova-plugin-firebase.git#android-build (install the proposed fix)

@briantq I tried that branch but I didn't see any difference, I am still getting the same errors as before. I tried it with and without cordova-android-play-services-gradle-release and cordova-android-firebase-gradle-release

@ahmadtawakol Thank you for testing it!  Can you list the plugins without the extra gradle release plugins, cordova plugin list.
Also, can you confirm that there were 4 lines added to the project's build.gradle? You would be and to tell as the added lines have a comment that include cordova-plugin_firebase. Those 4 lines are 80% of the fix. In fact, those 4 lines alone should resolve the specific error mentioned in this issue.

@briantq Thanks for pointing that out! Turns out that those 4 lines weren't in the build.gradle, probably because I added the platform after I added the plugin? Anyways, I removed the plugin and the platform then added the platform again and added the plugin after that and I confirmed that the extra lines were added. The build was successful and I was able to run the app without any issues!

Up, same error :'(

@julienkermarec @jfbourne Can you please see the two comments above #1 #2.  The change required the add plugin hook to run.  Please confirm the proper lines are in the files.  If they are not, try removing the /plugins and /platform directory and run cordova prepare to run the platform manager that should install the plugin with the proper hooks.

I can't use the plugin hook, because the build is created in a gitlab pipeline with proxy 👎

@julienkermarec I don't have experience with gitlab pipelines but do have experience with Jenkins pipelines.  Is there not a way to include a shell command in your build?  I figure I must be missing something.

i can include command in the build.

@julienkermarec why can't you use the plugin hook?  It should be run as part of cordova prepare.  Again, I probably misunderstood something.

I have this error with the hook on the gitlab pipeline :
`$ npm install
npm ERR! Error while executing:
npm ERR! /usr/bin/git ls-remote -h -t https://github.com/briantq/cordova-plugin-firebase.git
npm ERR!
npm ERR! fatal: unable to access 'https://github.com/briantq/cordova-plugin-firebase.git/': Failed to connect to github.com port 443: Connection timed out
npm ERR!
npm ERR! exited with error code: 128
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2018-09-17T10_35_26_842Z-debug.log
Running after script...
$ ls -all platforms/android/build/outputs/apk/release/
ls: cannot access 'platforms/android/build/outputs/apk/release/': No such file or directory
ERROR: Job failed: exit code 1`

@julienkermarec loading the plugin from github is only a temporary work around.  This is not intended to be in production.  The commands were simply to test your build locally to confirm that when we push the change it will indeed fix your issue.
If you can run it locally on your machine and confirm that will fix the issue, that will help confirm it will resolve your issue and allow us to release a new official version quicker, which you can reference in your production builds.

@alexhildebrand Looks like you have version 2.0.1 of the plugin and not the branch in the PR.  Try running cordova plugin remove cordova-plugin-firebase and then cordova plugin add https://github.com/briantq/cordova-plugin-firebase.git#android-build.  Confirm you see cordova-plugin-firebase 2.0.1 "Google Firebase Plugin" // https://github.com/briantq/cordova-plugin-firebase.git#android-build when you list your plugins.
The see this comment to confirm the actual fix is installed.  If it is not, you can try deleting the /platform directory and running cordova prepare as this should recreate the file correctly.

@jfbourne You might try removing your /platform directory and recreating it using cordova prepare.  That should cause all the files to be recreated correctly.

Hey @briantq,
So I have tried using the cordova command (cordova plugin add https://github.com/briantq/cordova-plugin-firebase.git#android-build) after having removed the platforms and deleting the respective plugin(cordova plugin remove cordova-plugin-firebase).
For some reason the version installed with the cordova command was not correct. So I have now cloned the branch ( https://github.com/briantq/cordova-plugin-firebase#android-build) and have overwritten it in the plugins. When I "git log" the folder, I see that the latest commit is "commit 9ed901b45fe2982695e6383f19632f0f275ffa82", which I think is in accordance to the branch given in https://github.com/briantq/cordova-plugin-firebase.git#android-build .  However even though I have followed all these steps, I still get the following cordova list (and the same error):
cordova-plugin-device 2.0.2 "Device"
cordova-plugin-firebase 2.0.1 "Google Firebase Plugin"
cordova-plugin-ionic-keyboard 2.1.2 "cordova-plugin-ionic-keyboard"
cordova-plugin-ionic-webview 2.1.4 "cordova-plugin-ionic-webview"
cordova-plugin-splashscreen 5.0.2 "Splashscreen"
cordova-plugin-statusbar 2.4.2 "StatusBar"
cordova-plugin-whitelist 1.3.3 "Whitelist"
phonegap-plugin-barcodescanner 8.0.0 "BarcodeScanner"
Am I still missing something?

Nice job @briantq ! You hook works in locally on my machine !!

@julienkermarec I am glad it worked!  Thanks for testing it.
@alexhildebrand I had an issue in the past that the uninstall for the firebase plugin didn't complete successfully.  To get around that I just modified the plugin references directly in the config file.  You can update the dependency in the config.xml to <plugin name="cordova-plugin-firebase" spec="https://github.com/briantq/cordova-plugin-firebase.git#android-build" />.  Additionally you need update the firebase plugin reference in the package.json also to "cordova-plugin-firebase": "git+https://github.com/briantq/cordova-plugin-firebase.git#android-build",.  Then delete your /plugins directory to force the removal of the old version of the plugin.  Then try running `cordova prepare.  I think that should update the plugin version.

Thank you @briantq, that worked!

The fix has been released in v2.0.2

I still get the error with 2.0.2:

What went wrong:
A problem occurred evaluating project ':app'.

Plugin with id 'com.google.gms.google-services' not found.

Here is the result of cordova plugins
cordova-plugin-advanced-http 1.11.1 "Advanced HTTP plugin"
cordova-plugin-ble-central 1.2.2 "BLE"
cordova-plugin-compat 1.2.0 "Compat"
cordova-plugin-file 6.0.1 "File"
cordova-plugin-firebase 2.0.2 "Google Firebase Plugin"
cordova-plugin-inappbrowser 3.0.0 "InAppBrowser"
cordova-plugin-splashscreen 5.0.2 "Splashscreen"
cordova-plugin-whitelist 1.3.3 "Whitelist"
cordova-support-android-plugin 1.0.1 "cordova-support-android-plugin"
es6-promise-plugin 4.1.0 "Promise"
wifiwizard2 3.0.0 "WifiWizard2"

@salim7 try deleting your plugins and platform directories, then running cordova prepare. That should regenerate the files and resolve your errors.

@briantq that helped, thank you!

I am still getting this issue

What went wrong:
Failed to notify dependency resolution listener.

The library com.google.android.gms:play-services-measurement-base is being requested by various other libraries at [[15.0.2,15.0.2], [16.0.2,16.0.2]], but resolves to 16.0.2. Disable the plugin and check your dependencies tree using ./gradlew :app:dependencies.
The library com.google.firebase:firebase-analytics-impl is being requested by various other libraries at [[15.0.2,15.0.2], [16.2.1,16.2.1]], but resolves to 15.0.2. Disable the plugin and check your dependencies tree using ./gradlew :app:dependencies.


fix: trim trailing line separator for SAS log

Summary
Fix #886
In most cases, a log line returned from server doesn't contain line separator. It fits the outputChannel.appendLine API well.
It appears that in the case of #886, a log line returned from server contains a trailing line separator, which produces an unexpected extra line and messed the log type index.

Testing
Test case in #886

This issue is fixed in the playpen package, no other issues were found, so approving.


OEXDateFormatting Tests

Adapted the code to the format mentioned at edx Docs
 2014-11-19T04:06:55Z
Please let me know if that is consistent across the board

@aleffert please review

:+1: We should pull that format string out as a constant so it's not repeated.
I believe this is the date format we use everywhere. It's just standard ISO 8601 dates.


parser.go: ipv6 address without network mask support

Before we would always add a /32 to ip addresses without a /. This would be wrong for ipv6 addresses. They have 128 network masks.

hmm I think for this we have to update go to at least 1.18.
Fine with me

I second it. go 1.18 was released more than two years ago.


Rename LexerError -> TokenDiagnostic

LexerError has been the wrong name for this for two reasons:

It could also contain warnings generated by the lexer
It also contained errors about insufficient indentation that weren’t generated by the lexer

TokenDiagnostic is the best name I have found to describe what this is doing.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test


Changing branch name borks updates

Bad news.  Changing the branch name from master to main prevents the use of git pull to update the module.  Given that this is an entirely cosmetic change that requires users to delete the folder and reinstall it, I would recommend going back.  Sorry for making you change it and change it back.

Oh, right... Forgot about that. Not your fault, thank you for the heads up though!


Hello, has anyone effectively used this module on a raspberry pi zero or raspi 3b+

Hi, i was wondering if anyone here has used this module on a raspi zero with a speed of about 1 frame every 2 - 5 seconds?

I did. What do you wonder?

Hi. I am trying to run DeepFace in a Raspberry Pi 4. I am just doing a simple script giving to the program a simple png image and I am always same result:
Using TensorFlow backend.
Directory  /home/pi /.deepface created
Directory  /home/pi /.deepface/weights created
Traceback (most recent call last):
File "emotion_detection.py", line 8, in
analyze = DeepFace.analyze(image,actions=['emotions'])
File "/home/pi/.local/lib/python3.7/site-packages/deepface/DeepFace.py", line 450, in analyze
resp_obj["region"][parameter] = int(region[i]) #int cast is for the exception - object of type 'float32' is not JSON serializable
IndexError: list index out of range

I did. What do you wonder?

Was it fast? Are the dataset downloads the same size on all platforms? And was it as effective? Sorry for all the questions lol

@SlevinKlvra, i think there is a typo. it should be emotion (singular) instead of emotions (plural)
analyze = DeepFace.analyze(image,actions=['emotion'])

@fwuffyboi it depends on complex models and unfortunately it won't be really real-time. it would be some freezes.


Correct a word in the remarks of /DIRECTIVES.

The section is .drectve instead of .drective.

@ROOOO : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"


Add a flag that specifies with which client was a post created

On the web i started adding an attribute to posts that specifies which client wrote the post. Valid values are web, ios and android:

Could you add the client: "ios" attribute for each posts created via the iOS app for tracking and debugging purposes?

fixed in https://github.com/firebase/friendlypix-ios/commit/8606c01e753cba94cdb096cf1bcf329761c6294b


Remove usage of EntityMetadataWrapper.

What's this PR do?
This pull request removes EntityMetadataWrapper from dosomething_user_get_field. I still can't figure out the root problem, but we're seeing a ton of strange exceptions when using it on staging that seem to disappear when just grabbing the fields by hand.
How should this be reviewed?
👀
Any background context you want to provide?
🔥 🔥 🔥
Relevant tickets
References #7438.
Checklist

[ ] Documentation added for new features/changed endpoints.
[ ] Tested on staging.
[ ] Pinged a PM if this is a larger PR that would benefit from some additional testing love
[ ] Post a message in #phoenix if this includes something that causes a rebuild!

Yeah, I'll try to peek in as well – there's definitely something strange behind the scenes! 👻


Mounting two different accounts on two different mount points?

From my own trial and errors, it looks like plexdrive is looking for the cache, the token and the application id.
Would it be possible to mount two accounts on different mountpoints? Is that an enhancement?

I do this all the time. just use the --config or -c parameter and point it to different config folders.

Thank you sir, I need to learn how to read.


PSR-7 Tests Fix

Description

Bumps laminas dependency do 3.1.0
Skips broken integration tests for Nyholm and Laminas
Fix broken test

coverage: 100.0%. remained the same when pulling 62d617f6d34262ac97886fb32e557bed38d3ed11 on l0gicgate:psr7-tests-fix into 63d3cb0605edccba34fd2c73b619b2b4dd267181 on slimphp:master.


ARM

Hi Tobias,
what would it need to make your tool work with arm images?
Update:
I tried to compile your main.go on arm but not luck, modules are deprecated ( dockerignore ) , its a real mess (not your code but the go packaging system). It might be easier to migrate the code but I feel that taking a deeper look at the nixos side might be more useful on the long run. Meanwhile I decided to try to merge two arm images and it actually worked on my system (macos).

This is probably just a matter of detecting desired cpu architecture here instead of hard-coding Architecture: "amd64":
https://github.com/bergkvist/includeimage/blob/670f86dffe091761b3fbb89cf1e28d2bc288db99/main.go#L58-L63

Did this get/could this be pushed into the docker-hub?


[BUG] SkiaSharp.Views.Uno's SKXamlCanvas has a binary incompatibility with Uno 4 that causes page load failure

Description
In the new Uno Platform 4.0 release there were some binary compatibility breaks. One of them is that the signature for CoreDispatcher.RunAsync changed from public UIAsyncOperation RunAsync(CoreDispatcherPriority priority, DispatchedHandler handler) to public IAsyncAction RunAsync(CoreDispatcherPriority priority, DispatchedHandler handler).
The SkiaSharp.Views.Uno SKXamlCanvas control calls that method in its public new async void Invalidate(); method when it is called or some action causes it to be called from any thread other than the UI thread; see: SKXamlCanvas.cs line 122 (note: link is to the file as found in the last commit that modified it at the time I'm writing this) .
It was difficult to decide where to post this since the breaking change is in Uno Platform. However the breaking change brings it into conformance with the UWP API and this will likely be an ongoing problem for SkiaSharp (as explained below). On the surface it seems like all that needs to be done to make it work with Uno 4 is to create a new SkiaSharp.Views.Uno package built against Uno 4.
That said, this is probably going to be an ongoing problem for SkiaSharp since if it's compiled against Uno 4 then it will have the same missing method issue (except with the expected and actual signatures reversed) when someone tries to use it with Uno 3.X (or any other version of Uno prior to 4). So you may need to ship two versions, deprecate support for pre-Uno 4, and at some reasonable time in the future cease building new SkiaSharp packages for pre-Uno 4.
The exact commit where this change occurred is here: unoplatform/uno github repo commit link - relevant file src/Uno.UWP/UI/Core/CoreDispatcher.cs - old line is 153; new line is 79
Lastly, I did not test the SKSwapChainPanel control and because I could not construct a SKXamlCanvas control I was unable to determine if there were other breakages besides the one that occurred in SKXamlCanvas.Invalidate
Code
In Visual Studio 2022, with the 'Uno Platform Solution Templates' extension installed, create a new 'Multi-Platform App (Uno Platform|Xamarin)' solution. When the solution is created, using nuget, do the following:

Add the SkiaSharp package to all projects
Add the SkiaSharp.Views package to the UWP project.
Add the SkiaSharp.Views.Uno package to all of the projects except the UWP project.

Next, in MainPage.xaml, to the Page tag, add xmlns:views="using:SkiaSharp.Views.UWP". Change the Page's Content control from Grid to StackPanel but leave the  "Hello, world!" TextBlock control. After that TextBlock control, add <views:SKXamlCanvas />.
Finally, in App.xaml.cs, in the OnLaunched method, modify the line
                   rootFrame.Navigate(typeof(MainPage), args.Arguments);
to
                   try                     {                         rootFrame.Navigate(typeof(MainPage), args.Arguments);                     }                     catch (Exception ex)                     {                         System.Diagnostics.Debug.WriteLine($"Exception type: {ex.GetType().FullName}.\nMessage: {ex.Message}\nStack Trace: {ex.StackTrace}");                         System.Diagnostics.Debugger.Break();                         throw;                     }
Expected Behavior
Run any of the Platform projects via debugging in Visual Studio. The app will start properly and navigate to MainPage.xaml causing the text "Hello, world!" to be visible.
Actual Behavior
On any platform other than UWP, a System.MissingMethodException is thrown internally (it appears in the Output window's "Show output from: Debug" output) but does not directly propagate out to user code and thus cannot be caught. It does not crash the program and so the program continues to run.
After that, a System.InvalidOperationException is thrown that is catchable that informs you of the MissingMethodException via its Message property, which contains a value along the lines of:

Failed to load MyProject.MainPage: System.MissingMethodException: Method not found: 'Windows.UI.Core.UIAsyncOperation Windows.UI.Core.CoreDispatcher.RunAsync(Windows.UI.Core.CoreDispatcherPriority, Windows.UI.Core.DispatchedHandler)'.
at SkiaSharp.Views.UWP.SKXamlCanvas.Invalidate()
...

The debugger will catch that exception in the code we added in App.xaml.cs. After continuing run the program, the app will swallow the exception and display a blank window, i.e., no "Hello, world!" text, since it was unable to load MainPage.
Basic Information

Version with issue:  All versions built before Uno Platform 4, which is currently all versions.
Last known good version:  None.
IDE:  Visual Studio 2022 Community
Platform Target Frameworks:

Android: SDK 30.0
Linux:  OpenSUSE Leap 15.3 via WSL2's Wayland server functionality
Windows Classic:  Windows 10 version 2004 (build 19041)

Target Devices:   Local PC running an Uno Android project via emulator, a local PC running an Uno Skia.Gtk project using WSL, and a local PC running an Uno Skia.Wpf.Host project.

(That is not an exhaustive list, I simply wasn't able to test it on all of the platforms. Because it is a binary break it will happen on any platform except UWP; UWP has always had public IAsyncAction CoreDispatcher.RunAsync(CoreDispatcherPriority priority, DispatchedHandler handler); as its signature, so it had no binary compatibility break).

Detailed IDE/OS information (click to expand)
  Visual Studio 2022 17.0.1 Community running on Windows 11 Pro (10.0.22000 Build 22000) with the most recent Windows Subsystem for Linux version 2 and OpenSUSE 15.3 (note: only needed if you want to test the Skia.Gtk project using WSL; it's not necessary).

Reproduction Link
SkiaSharpUno4BreakageSample.zip

i have a related issue with SKSwapChainPanel and Uno 4.0 - opened almost at the same time 😂
https://github.com/mono/SkiaSharp/issues/1878

@jeromelaban This might explain why the PR #1873 builds are failing.
For everyone, also see some comments added to the PR: https://github.com/mono/SkiaSharp/pull/1873#issuecomment-985832737

That said, this is probably going to be an ongoing problem for SkiaSharp since if it's compiled against Uno 4 then it will have the same missing method issue (except with the expected and actual signatures reversed) when someone tries to use it with Uno 3.X (or any other version of Uno prior to 4). So you may need to ship two versions, deprecate support for pre-Uno 4, and at some reasonable time in the future cease building new SkiaSharp packages for pre-Uno 4.

I would drop support for all older versions of Uno. If someone is still using old versions of Uno below 4.x they will simply need to use an older version of SkiaSharp. Otherwise, I don't think this is easy to manage going forward.

#1873 is here to fix the causes for this issue.
Uno 4.0 introduced (many) binary breaking changes to align with WinUI 3 and avoid discrepancies found when using Uno 3.x and earlier. We're unlikely to break again in the future as Uno is now aligned with WinUI/WinRT and that this API is stable.

This was fixed with the SkiaSharp 2.88.0-preview.179 nuget (including SkiaSharp.Views and SkiaSharp.Views.Uno)
Any Uno version < 4.0 must use SkiaSharp < 2.88.0-preview.179 and
Any Uno version >= 4.0 must use SkiaSharp >= 2.88.0-preview.179
I believe this issue can be closed unless you want to leave it open until final release (not preview Nugets).


[HOLD #39727] [Wave Collect] [Xero] [Import Flow] Create the Taxes page

You need to build the page following the steps outlined here.
This will be worked on by SWM engineers, see Slack discussion here.
Until the Xero authorization flow (NewDot issue)  is done (which entails NewDot, Web-E and IS changes), we won't be able to work on this yet.

Issue on hold for #39727, Melv. Chiiiil.

Hi! I am Filip from SWM and I'd like to work on this issue!

Issue still in a line of dependencies!

I'm working on this

PR merged

Deployed to production 2 days ago.


High CPU when using file watchers and volumes

My project (A small, ~8 endpoint express app) uses typescript compiler file watching (tsc --watch) and nodemon for local development.
The combination of these two processes running outside of docker, on MacOS, uses approximately 5% of one CPU core when idle, 25% cpu of one core when compiling/reloading, and finishes in approximately 2 seconds.
We've been experimenting with developing inside of Docker, as an alternative to using Node Version Manager and juggling multiple projects with different node versions.
Here is the example script we have been running:
yarnWatch.sh

#!/bin/sh

tsc --watch
nodemon app.js

docker run \
  --volume $(pwd):/proj \
  --workdir /proj \
  --rm \
  --publish 3030:3030 \
  --detach \
  node:12.16.1-alpine scripts/yarnWatch.sh

When running in this way, the hyperkit process idles at 200% CPU (my guess is 100% of two cores), uses 500%cpu when compiling/relaoding, and finishes in 20 seconds.
My hypothesis is this is some sort of bottleneck regarding volumes, as running the app in a custom built image has no performance problems.

Did you find any workaround?

Seeing very similar results (just for PHP). > 100% cpu is being used on a process that's essentially idle. Seems to be a Mac thing from googling this?

I also see this with Docker for Mac and nodemon — block I/O, CPU, and memory spiral out of control

We experienced exactly same thing with watchfiles and uvicorn. This is a ampere arm64 host. I think something is notify handling.


How to dynamically specify plugin priority?

Now we all define the priority in the plug-in's handle.lua, like the following.

How to dynamically specify this priority when applying plug-ins, such as schema pages?

It would be a great feature to be able to dynamically set the priorities of the plugins.


10 client side myhunts page

Client side files needed to build a 'My Hunts' page.

It was determined that task.ts and task-service.ts was not required for the My Hunts page.
4 additional ng component files were added for the hunt-card
There is more to be done for the My Hunts page, but what has been done satisfies what was originally requested, and work needs to be pulled to main
More issues need to be made to address e2e, additional button routing, button dialog "Are you sure" for delete hunt, and grouping by hostid feature. And likely more before My Hunts page is actually complete


Laravel Echo Server is not receiving events broadcasted by laravel

Good time of the day!
I've encountered pretty strange error lately.
I have 2 machines, dev and prod. Here are the configurations:
Dev Machine:
OS: Windows 10 Pro
Apache: 2.4.29
PHP: 7.2.5
MySQL: 5.7 CE
NodeJS: 9.8.0
NPM: 5.6.0
Laravel Echo Server: 1.3.6
Redis: Latest Compiled From Sources
Prod Machine:
OS: Debian 9
Apache: 2.4.25
PHP: 7.2.5
MySQL: 5.7 CE
NodeJS: 9.11.1
NPM: 5.6.0
Laravel Echo Server: 1.3.6
Redis: Latest Compiled From Sources
So the problem is that on the local server Laravel Echo Server (LES) is working just fine. Client is able to join the rooms, receive the events, leave rooms, etc.
However, on the production server, LES is showing that client is connected and joined the room, but the events broadcasted by the Laravel itself are not received by the LES server.
Here is a small excerpt from the .env file:
    BROADCAST_DRIVER=redis
    CACHE_DRIVER=redis
    SESSION_DRIVER=redis
    SESSION_LIFETIME=120
    QUEUE_DRIVER=redis

And here are the laravel.log file content regarding the broadcasted events:
[2018-05-27 00:32:12] production.INFO: Broadcasting [application.updated] on channels [application] with payload:
{
    "socket": null
}
[2018-05-27 00:33:39] production.INFO: Broadcasting [application.updated] on channels [application] with payload:
{
    "socket": null
}

This event should trigger the code (it works on dev machine) which will display a snackbar that "Application has been updated, please reload the page".
laravel-echo-server configuration is done in this way:
    {
		"authHost": "https://api.host.example",
		"authEndpoint": "/broadcasting/auth",
		"clients": [
			{
				"appId": "applicationIDKey",
				"key": "applicationKey"
			}
		],
		"database": "redis",
		"databaseConfig": {
			"redis": {},
			"sqlite": {
				"databasePath": "/database/laravel-echo-server.sqlite"
			}
		},
		"devMode": true,
		"host": null,
		"port": "3000",
		"protocol": "https",
		"socketio": {},
		"sslCertPath": "/var/www/path-to-the-certificates/host_le1.crt",
		"sslKeyPath": "/var/www/path-to-the-certificates/host_le1.key",
		"sslCertChainPath": "/var/www/path-to-the-certificates/host_le1.ca",
		"sslPassphrase": "",
		"apiOriginAllow": {
			"allowCors": true,
			"allowOrigin": "https://application.example",
			"allowMethods": "GET, POST",
			"allowHeaders": "Origin, Content-Type, X-Auth-Token, X-Requested-With, Accept, Authorization, X-CSRF-TOKEN, X-Socket-Id"
		}
	}

Event Code (triggered by the console command upon new application version deployment via Envoy)
    <?php namespace App\Events\Application;

    use Illuminate\Broadcasting\Channel;
    use Illuminate\Queue\SerializesModels;
    use Illuminate\Broadcasting\PrivateChannel;
    use Illuminate\Broadcasting\PresenceChannel;
    use Illuminate\Foundation\Events\Dispatchable;
    use Illuminate\Broadcasting\InteractsWithSockets;
    use Illuminate\Contracts\Broadcasting\ShouldBroadcast;
    use Illuminate\Support\Facades\Event;

    /**
     * Class ApplicationUpdatedEvent
     * @package App\Events\Application
     */
    class ApplicationUpdatedEvent extends Event implements ShouldBroadcast {

        use Dispatchable, InteractsWithSockets, SerializesModels;

        /**
         * Create a new event instance.
         *
         * @return void
         */
        public function __construct() {
            //
        }

        /**
         * Get the channels the event should broadcast on.
         *
         * @return \Illuminate\Broadcasting\Channel|array
         */
        public function broadcastOn() {
            return new Channel('application');
        }

        /**
         * Get the broadcast event name
         * @return string
         */
        public function broadcastAs() : string {
            return 'application.updated';
        }

    }

Both queue server (supervisord) and LES running at the same time under the user deployer. Application is also deployed into the "web" folder by the same user, this user also has rwx permissions to any folder which is necessary for the project to work.
I really don't see the possible problem with the configuration, since the only thing which is not working is the broadcasting of the events themselves.

Hey there!
I think I might have found the problem in your code. In your laravel-echo-server.json your "redis" is empty, thus not configured and might not be able to connect. Maybe worth a try.
"databaseConfig": { 			"redis": {}, 			"sqlite": { 				"databasePath": "/database/laravel-echo-server.sqlite" 			} 		},

So? What was the solution?


Typos for version 1.0.1

List here typos to be fixed

"This is a sub-property of dct:conformsTodct:conformsTo." in Section 4.11.3 2nd row of the table

Last line in the Table in Section 5.2, should it be dct:rights instead of dct:type?

In the specification, in some cases, the wrong URI is used for the class mobilitydcatap:MobilityDataStandard, e.g., in https://mobilitydcat-ap.github.io/mobilityDCAT-AP/releases/index.html#properties-for-mobility-data-standard. The URI with the uppercase "M" should be used for the class and the one with the lowercase "m" for the property.

Last line in the Table in Section 5.2, should it be dct:rights instead of dct:type?

Well, have a look:
"type" is a property of class "Rights Statement". Class "Rights Statement" is the range of property "rights". And "rights" is a property of class "Distribution".
The Controlled Vocabulary is indeed introduced for "type", denoting a predefined option for this property. So, everything is correct here!

Typos corrected in v1.0.1:
"This is a sub-property of dct:conformsTo."
mobilitydcatap:MobilityDataStandard vs. mobilitydcatap:MobilityDataStandard
Regarding "dct:rights instead of dct:type":
Ongoing discussion between Peter and Mario. Option: property change as part of a future major revision.

Regarding "dct:rights instead of dct:type": Ongoing discussion between Peter and Mario. Option: property change as part of a future major revision.

This is not a typo. The dct:type property is recommended to connect an individual of class dct:RightsStatements to a value in the controlled vocabulary (and not to connect a dcat:Distribution to the associated dct:RightsStatements). I got confused because, in the other tables, we have the column Property and then the column Range, while in Section 5.2 the second column is Used for Class.


Coverage 100% and issue #1

I've made this changes:

one test for 100% coverage

one test for the issue #1

and fix the issue mentioned

Coverage increased (+0.8%) to 100.0% when pulling 3b2a6b93a4cae2c45c5872fa7a1e1ffba9170da8 on yamijuan:master into 9e80e8a07a6131b4658d364d5116d2e0ef8b8fa4 on RADYConsultores:master.


⚠ Hartman Foundation has degraded performance

In c51d839, Hartman Foundation (https://gordonhartman.com/) experienced degraded performance:

HTTP code: 200
Response time: 2850 ms

Resolved: Hartman Foundation performance has improved in 31499c8.


Make make deploy-hub skippable

We now want to deploy the Hub operator through the bundle in openshift/release, like we do for the other e2e test.
/cc @yevgeny-shnaidman @ybettan

/lgtm


testing clippy check on push

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[ ] No

The PR fulfills these requirements:

[ ] It's submitted to the dev branch and not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

@lucasfernog @nothingismagick I think this is the proper fix vs #402. It appears to only work on push. Ready whenever.


[ART-1618] [ART-2800] New signing mechanism

This PR adds a new signing mechanism to let promote-assembly job directly send signing requests through UMB.
This eliminates the intermediate step that pulling artifacts from the internet.
Basically this PR gives pyartcd the ability to sign things with pure Python implementation. So that we will not need a dedicated sign-artifiacts job and also eliminate the need to hand over artifacts between jobs. Concretely, it
comes with a class AsyncUMBClient aiming to simplify send and receive UMB messages using request/response model. (Existing python library is hard to use).
uri = "stomp+ssl://umb.stage.api.redhat.com:61612"
cert_file = "ssl/nonprod-openshift-art-bot.crt"
key_file = "ssl/nonprod-openshift-art-bot.key"
async with AsyncUMBClient(uri, cert_file, key_file) as umb_client:
    # send a message
    await umb_client.send(topic, request_body)
    # receive messages
    receiver = await umb_client.subscribe(consumer_queue, subscription_name)
    async for message in receiver.iter_messages():
        print(message.headers["message-id"])

class AsyncSignatory sits on top of class AsyncUMBClient , which can be called from a python script to sign artifacts:
uri = "stomp+ssl://umb.stage.api.redhat.com:61612"
cert_file = "ssl/nonprod-openshift-art-bot.crt"
key_file = "ssl/nonprod-openshift-art-bot.key"
async with AsyncSignatory(uri, cert_file, key_file, sig_keyname="beta2", requestor="yuxzhu") as signatory:
    pullspec = "quay.io/openshift-release-dev/ocp-release:4.11.31-x86_64"
    digest = "sha256:cc10900ad98b44ba432bc0d99e7d4fffb5498fd6844fc3b6a0a3552ee6d64059"
    # sign a release payload
    with open("signature-1", "wb") as sig_file:
        await signatory.sign_json_digest("openshift", "4.11.31", pullspec, digest, sig_file)
    # sign a message digest
    with open("sha256sum.txt", "rb") as in_file, open("sha256sum.txt.gpg", "wb") as sig_file:
        await signatory.sign_message_digest("openshift", "4.11.31", in_file, sig_file)

promote-assembly is updated to use AsyncSignatory. sign-artifacts job is not used but I haven't removed it.

Committed some minor fixes and tested with 4.11.46 promotion: https://saml.buildvm.hosts.prod.psi.bos.redhat.com:8888/job/hack/job/yuxzhu/job/aos-cd-builds/job/build%252Fpromote-assembly/106/consoleFull

LGTM

Thanks for the review.
Rebased. Merging...


feature: precompiled binaries

Would it be possible to provide precompiled binaries? I do not have Rust, do not use it and I do not want to install it just for the sake of building a single package. It would be much appreciated if there was another way of installing asm-lsp.

This was planned for the 0.8.0 release but unfortunately we ran into some issues getting the project to build on the runners. As long as you're not on Windows (see #134), I think I have these issues worked out. I should be able to manually trigger the workflow either today or tomorrow :)

Oh, thank you very much! It will be appreciated!
Of course I am not on windows :)
Sorry for missing #134, I have read it but I thought it is related to something else. My bad.

Oh, thank you very much! It will be appreciated!

I manually created an x86_64 linux release build, compressed it, and uploaded it to the 0.8.0 release. If you're not comfortable using this or it doesn't match your platform, I think I have a workaround. However, it's fairly hacky since forcing the release workflow to run without triggering the publish workflow is proving to be a bit tricky. I can spend some more time on this in the next few days if that's the route you'd prefer. Let me know! :)

Sorry for missing #134, I have read it but I thought it is related to something else. My bad.

No worries at all!

That worked for me, thank you very much!


Roll engine 3c51a7bfff77..e4653298618d (6 commits)

https://github.com/flutter/engine/compare/3c51a7bfff77...e4653298618d
git log 3c51a7bfff776e9e3011384f4ff09c13a697c6b6..e4653298618d45b7d74c02ca550b43688c8557df --no-merges --oneline
e46532986 Roll src/third_party/skia 2f62edf1ac5d..de6758683abd (1 commits) (flutter/engine#9700)
05966cd14 Roll src/third_party/skia 1d105080c423..2f62edf1ac5d (1 commits) (flutter/engine#9699)
b132baa44 Ensure that platform messages without response handles can be dispatched. (flutter/engine#9698)
50d622857 fix Picture.toImage return type check and api conform test. (flutter/engine#9685)
001bb541c Roll src/third_party/skia 8063f6cca025..1d105080c423 (3 commits) (flutter/engine#9692)
db4d6836a Roll src/third_party/skia 857c9f955edb..8063f6cca025 (3 commits) (flutter/engine#9679)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5232175920709632

Commit queue failed; closing this roll.


Add support for servercheck updates to go client

What does this PR (Pull Request) do?
Adding support for the /api/1.3/servercheck route to the go client, which was previously missing.

This PR is not related to any Issue
Documentation via godoc

Which Traffic Control components are affected by this PR?

Traffic Control Client (go)
Traffic Ops servercheck scripts (WIP, new go version to replace legacy perl)

What is the best way to verify this PR?
Client test code used:
        var statusData tc.ServercheckNullable
        statusData.ID = server.ID
        statusData.Name = "DSCP"
        statusData.Value = 0
        _, err := session.UpdateCheckStatus(statusData)
        if err != nil {
            panic(err)
        }

No tests were included because no API changes were made.
The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Can one of the admins verify this patch?

add to whitelist

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3483/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3484/
Test FAILed.

sorry -- only had a cursory look at it before..   The code looks good,  but I'd like to see a test for it. It should be fairly straightforward:  in traffic_ops/testing/api/v14,  add a new servercheck_test.go and fill in json in tc-fixtures.json.   Simple test to make sure you can read the data when it goes into the table as well as any tests to check for incorrect input.  Plenty of other examples there.  I can help out with that if needed.

I think I'll need to add a client function to read the data then - will probably be helpful for the test :)

So, I added a test, but I'm not 100% sure it's what is needed. Currently, the only use case is to update the check status, so it's pretty simplistic, I think. Also, I'm not sure the best way to handle the fact that the "extension" use is required to be used for the servercheck API route. I added that user to the test, but I doubt it will authenticate as-is.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3487/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3667/
Test PASSed.

The API test for this route doesn't compile, load the data it needs to run, or run as the proper user. It also causes a runtime panic when run.


Update

fix broken link

@ejarvi,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@ejarvi : Thanks for your contribution to the Azure documentation! The author, @YuriDio, has been notified to review your proposed change.

change looks fine. Pls merge it.
#sign-off


Github Workflows

In PR #25 the make_header.sh script fails with xxd: command not found.
Trying to look into this I could not find any record on how the Netlify deployments are triggered.

How are the current PR checks implemented?
Could these please be transitioned to GH Workflows, such that workflow improvements can be traced and contributed to as well?

For one, I'd like to propose the change that if the script fails, the check should not pass succesfully.

netlify is triggered via github webhooks whenever there is a new commit in main. It is configured to run npm run build and then host the contents from _static after that is done.


Make all segments optinal

Solves #41

Implemented in https://github.com/catppuccin/tmux/issues/59


bump to 1.3.0-beta

add the runtime logger extensions

cancelled as updates as happened to the base


maintain a definition (USE_COLORS) for the use of colors

Hi,
In my project I specify the use of colors during the build process. To this end I've introduced a definition (USE_COLORS) that allows to turn on or off the use of colors at compile time..
Thanks, Chris.

I think your goal is valid, but I would like to see USE_COLORS as platform independent,
so not within WIN32 defines.
Also color_output is always set in main(), so the define doesn't really fix anything..
I think it would be good for some users to completely disable color output in the build.

You are absolutely right, not sure how I've overlooked this :/
Also wrt the WIN32 define: This should be checked during configuring the build anyways. I placed the check there as an additional security net, since the way colors are set in ctest doesn't work on Windows---I'm not certain if this still is the case for most recent versions (> Windows7).
I've updated the branch and move the check for USE_COLORS to ctest's main function

Alternatively there is CTEST_USECOLORS branch containing a single commit as patch.

i'll look into it tonight...
On Tue, Nov 17, 2015 at 1:25 PM, Christian Wressnegger <
<EMAIL_ADDRESS>wrote:

Alternatively there is CTEST_USECOLORS branch containing a single commit
as patch.
—
Reply to this email directly or view it on GitHub
https://github.com/bvdberg/ctest/pull/15#issuecomment-157356435.

look a lot better. One last thing is not to pollute the global namespace by
renaming USE_COLORS to CTEST_USE_COLORS. That way all CTEST features
have a common naming scheme..

I've turned around the default: default is colours on, but users can disable..


Update codecov CI to stable version of Rust

Similar to https://github.com/partiql/partiql-lang-rust/pull/383. Updates the codecov CI to a stable version of Rust.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hmm, there's a build failure running with --all-features: https://github.com/partiql/partiql-rust-cli/actions/runs/5160836211/jobs/9297279077#step:5:449
The following warnings were emitted during compilation:

warning: couldn't execute `llvm-config --prefix` (error: No such file or directory (os error 2))
warning: set the LLVM_CONFIG_PATH environment variable to the full path to a valid `llvm-config` executable (including the executable itself)

error: failed to run custom build command for `graphviz-sys v0.1.3`

Caused by:
  process didn't exit successfully: `/home/runner/work/partiql-rust-cli/partiql-rust-cli/target/llvm-cov-target/debug/build/graphviz-sys-3876cada2293243b/build-script-build` (exit status: 101)
  --- stdout
  cargo:rustc-link-lib=gvc
  cargo:rustc-link-lib=cgraph
  cargo:rustc-link-lib=pathplan
  cargo:rustc-link-lib=cdt
  cargo:warning=couldn't execute `llvm-config --prefix` (error: No such file or directory (os error 2))
  cargo:warning=set the LLVM_CONFIG_PATH environment variable to the full path to a valid `llvm-config` executable (including the executable itself)

  --- stderr
  ./wrapper.h:1:10: fatal error: 'graphviz/gvc.h' file not found
  ./wrapper.h:1:10: fatal error: 'graphviz/gvc.h' file not found, err: true
  thread 'main' panicked at 'Unable to generate bindings: ()', /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/graphviz-sys-0.1.3/build.rs:43:14
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
error: process didn't exit successfully: `/home/runner/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/bin/cargo test --no-run --tests --manifest-path /home/runner/work/partiql-rust-cli/partiql-rust-cli/Cargo.toml --target-dir /home/runner/work/partiql-rust-cli/partiql-rust-cli/target/llvm-cov-target --workspace --all-features` (exit status: 101)
Error: Process completed with exit code 1.

I'll sort it out when I get back.


update vision version

I would recommend excluding Gemfile.lock in .gitignore as we do in the google-cloud-ruby packages.

I can see excluding the Gemfile.lock file if a narrow version was set in Gemfile. Right now it is unbound.


[ISSUE] client.users.create error

Description
When in a defined function and specific authentication method using WorkspaceClient, the function users.create, throws the error (UsersAPI.create() missing 1 required positional argument: 'id')
Reproduction
This is a example of the function:
from databricks.sdk import WorkspaceClient
w = WorkspaceClient()

def create_user_workspace(w, displayname, username):
    try:
        user = w.users.create(display_name=f'{displayname}', user_name=f'{username}')
        print(f"User {displayname}' / '{username}' created correctly.")
        return user
    except Exception as e:
        print(f"Error creating user: {e}")
        raise(e)

Expected behavior
The user should be created correctly.
Is it a regression?
Im using version of the SDK: databricks-sdk-0.20.0
It's curious that using the same code without the defined function works fine, as shown below:
from databricks.sdk import WorkspaceClient
w = WorkspaceClient()
user = w.users.create(display_name='Name complete'<EMAIL_ADDRESS>
However, within the defined function, it fails with the error "UsersAPI.create() missing 1 required positional argument: 'id'".
Debug Logs
UsersAPI.create() missing 1 required positional argument: 'id'

UsersAPI.create() doesn't have any required positional arguments, and id is optional.
Can you share the full script, where you call create_user_workspace()? Can you also share the exact exception and stack trace?


Create GUM tool for managing the project

https://github.com/charmbracelet/gum

@r-mishev if you want to tackle this, I came up with something you could add to the tool. There is a script called set_vm_env.sh which installs all requirements for running the project. You could implement some logic for ssh-ing into the VM and installing the script. Additionally, a good update would be for the script to be trying to log into root@ip where ip is fetched as an env variable but if missing, the user is asked to input it.

Work on this is not finished, do not close the issue

What more has to be done?


restartIfChanged doesn’t seem to work for user services

e.g. for the following user service file:
      systemd.user = {
        services.pulseaudio = {
          description = "PulseAudio Server";
          # NixOS doesn't support "Also" so we bring it in manually
          wantedBy = [ "default.target" ];
          serviceConfig = {
            Type = "notify";
            ExecStart = binaryNoDaemon;
            Restart = "on-failure";
          };
          restartIfChanged = true;
        };

I would expect journalctl --user -f to output that pulseaudio.service was restarted if I change anything in the pulseaudio derivation, which it doesn’t. Neither does sudo journalctl -f.

https://github.com/NixOS/nixpkgs/issues/7326 probably needs to be fixed first

@anderspapitto Yes, same problem, only with reload. Of course both should be solved.


4scanner gives "max retries exceeded" error, and yet continues downloading images normally.

Ok, so this is weird.
 It's still on 4scanner==1.0.7
However, this is just strange: In the terminal window that I'm running in, 4scanner appears to go crazy, and give me endlessly repeating error hundreds of times:
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='boards.4chan.org', port=80): Max retries exceeded with url
And yet, it's still downloading: When I check the folder (with find . -type f | wc -l) it shows me that more and more imaged keep appearing. It's still working fine.
How can it give me hundreds of error and yet still be working fine? What could be going on here?

My guess is that 4scanner is downloading too much pictures at the same time and 4chan is actively blocking some of your images download.
Since I made a try catch around the image download to prevent 4scanner from crashing if the requested image is unavailable, you keep getting the error but 4scanner do not crash.
The problem is every time you get this error a picture is not downloaded since 4chan blocks your connection.
Maybe I could put a delay between each image download, this way 4scanner would spam 4chan less and hopefully it will stop blocking the downloads.
image download code for reference:
                        try:
                            urllib.request.urlretrieve('http:' + link, directory + img)
                        except urllib.error.HTTPError as err:
                            pass

Ok. And actually, I think it might have something to do with requests, because I got this too:
Traceback (most recent call last):
  File "/usr/local/bin/4scanner", line 38, in <module>
    main()
  File "/usr/local/bin/4scanner", line 33, in main
    scanner.scan(args.keywords_file, output, log_file)
  File "/usr/local/lib/python3.4/dist-packages/scanner/scanner.py", line 82, in scan
    except requests.exceptions.HTTPError as err:
NameError: name 'requests' is not defined

hmm I might be catching the wrong error here. That would explain why it's throwing the error you posted instead of printing:
                      "Retrying during next scan.".format(board))```

Thanks for the info! I'll try to fix it during the weekend.

No problem.

Both errors should hopefully be fixed in 4scanner 1.1.0 ! (upgrade with pip3)

That's awesome, thank you.

What did you do, it's way faster now.

I guess it's a side effect of remaking all the download code, instead of scraping the thread html for pictures to download I use the json API now. (http://a.4cdn.org/{board}/thread/{thread number}.json)

Did the error happen again? If not can we close the issue?

Well, I did get a few 404 json not found errors, but those should be simple to handle.

hmm, I guess the 2 seconds sleep I put between each pictures download isn't enough to keep 4chan from blocking some api request.. Maybe I'll put an option to wait n seconds for people who are downloading a lot of pictures at the same time.
Ex --wait-time 10 would wait 10 second between each pictures download.


->emptyStateDescription() does not work as expected when using configureUsing() in a provider.

Package
filament/filament
Package Version
3.2.127
Laravel Version
11.34.2
Livewire Version
v3
PHP Version
8.2.15
Problem description
When configuring the default behavior for a table in a service provider, the empty state heading can be defined, but the empty state description can not.
Expected behavior
The defaults for both the empty state heading and description should appear on a list page that has no records.
Steps to reproduce
Step 1 - Add the following to the provider file:
Table::configureUsing(function (Table $table): void {
    $table
        ->emptyStateDescription('This is my custom description!')
        ->emptyStateHeading('This is my custom heading!');
});

Step 2 - Go to an empty list page.
Note: The description does work if manually added to the list page.
class CustomerListPage extends ListRecords
{
    public function table(Table $table): Table
    {
        return $table->emptyStateDescription('This is my custom description!');
    }
}

Reproduction repository (issue will be closed if this is not valid)
https://github.com/RalphMRivera/filament-issue
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

The relevant lines are here:
https://github.com/RalphMRivera/filament-issue/blob/a52e512106900d8873a98481ae7db8b354fac4b2/app/Providers/AppServiceProvider.php#L23-L27


Custom prompt template VectorDBQAChain (non-english documents)

Hi,
I am trying to use langchainjs to create Q&A and it works really well! My only problem is language support. Input documents (and questions) are non-english but Q&A prompt is in english and that confuses OpenAI - it sometimes responds in english, sometimes in document language.
I have found source code that has hard-coded english prompt (https://github.com/hwchase17/langchain/blob/8e5c4ac8672b98b3c53884843a4fe57a0b79dcae/langchain/chains/vector_db_qa/prompt.py) but I can't figure out how to override it from langchainjs. Is that possible? Or should I just create my own chain? Is that even possible?
Thank you for any help

try this:
    const chain = ChatVectorDBQAChain.fromLLM(model, vectorStore, {
      returnSourceDocuments: true,
      qaTemplate: "Your prompt. {context} Question: {question}",
    });

But this parameter is available only for ChatVectorDBQAChain, not VectorDBQAChain

But this parameter is available only for ChatVectorDBQAChain, not VectorDBQAChain

Oh, I completely forgot that I can just use chat version (and qaTemplate didn't exist 5 days ago). It is basically the same if I only use it for single response.
Thank you, it works perfectly! I have just translated default prompt template, used qaTemplate and it works as expected.


Cannot find the file specified when installing from source

Dear Team,
I am encountering challenges while installing a library from its source. I have diligently followed the provided installation steps.
git clone git@github.com:openai/procgen.git
cd procgen
conda env update --name procgen --file environment.yml
conda activate procgen
pip install -e .
# this should say "building procgen...done"
python -c "from procgen import ProcgenGym3Env; ProcgenGym3Env(num=1, env_name='coinrun')"

Upon completion, I received the message: "Successfully installed procgen-0.10.7+5e1dbf3". However, upon executing the command python -c "from procgen import ProcgenGym3Env; ProcgenGym3Env(num=1, env_name='coinrun')", I encounter an error: "FileNotFoundError: [WinError 2] The system cannot find the file specified".
I have attempted installations across various conda environments, utilizing Python versions 3.7, 3.8, and 3.9, yet the issue persists. Could you please provide guidance on the appropriate steps to address this?
PS: When I install the library using the command pip install procgen, the library works without any issues. However, for my specific requirements, it is imperative to install the library from its source in order to make necessary modifications to the provided environments.
Best regards,
Onur

Hi @AkgunOnur - were you able to resolve this issue in the end? I'm experiencing a similar one.

Unfortunately, I could not solve it

No worries.
In case it's useful to you or anyone else, I solved it by:

Editing cmake_prefix_paths directly in function _attempt_configure to point at Library/lib/cmake directory within my conda environment.
Editing conda_prefix in the same function to point at my env/Library folder (not library - capital L)
Making sure I edited the generator variable to specifically name my version of Visual Studio (i.e. 16 2019, not 15)

I'm using Windows so can't comment on whether this should work for other platforms.


Error rendering edge-labels.

All files in a zip:
edge-labels-rendering-error-bug.zip.
Excalidraw shared link:
https://excalidraw.com/#json=dYmsl0r4MMuupw6J8mLch,7pC6x3aetcuFsfRsYo5CQQ.
Or as a file (had to zip it, github won't allow attaching .excalidraw
files):
volume_representations.zip.
Here is a screenshot of the diagram inside excalidraw:

Here is the Excalidraw exported PNG:

Here is the Excalidraw exported SVG:

Here is the file exported by excalidraw_export:
(volume_representations.excalidraw.svg):

Here is the generated SVG file opened in chrome (brave) and a screenshot
taken:

Take note:

Most of the floating text that is meant to be over the edges is misplaced.
The floating text has no background. In Excalidraw, there is a white
background that blocks out the edge.

Same issue in excalidraw-to-svg: https://github.com/JRJurman/excalidraw-to-svg/issues/6.


Document SIM2SUMO as a pre-configured ERT job

Similar to SUMO_UPLOAD and others, make it show up here: https://fmu-docs.equinor.com/docs/ert/reference/configuration/forward_model.html
Possibly add a note that it is experimental, or similar?

sim2sumo information is now available in fmu-docs


How to include hidden files in settings

I use neomake (vim plugin) which creates temporary files for "on the fly linking", for a normal file
src/foo/Bar.test.js
you get
<EMAIL_ADDRESS>I have "no-extraneous-dependencies" rule enabled like in your example (just with more paths):
"import/no-extraneous-dependencies": ["error", {"devDependencies": ["**/*.test.js", "**/*.spec.js"]}]

from https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-extraneous-dependencies.md
How do you specify that files like "**/.*.test.js" are also ok to use dev dependencies?
I tried:

**/.*.test.js doesn't exclude the files
**/\.*.test.js triggers an error in eslint config parser
**/\\.*.test.js doesn't exclude the files

Since it's a glob pattern, I would expect **/.*.test.js to exclude them.

Me too, but it didn't work :(


BeforeShowDay/Month/Year/Decade callback

It would be very useful to introduce a callback of the type beforeShowDay/Month/Year/Decade, which allows to inject specific properties, css/styles and html to each entry/cell. This would allow to create more complete calendar/schedule experience, like adding specific colors or icons per date etc. etc.

+1 Indeed!

+1

+1

+1

+1

Hi, you will develop this function?

@Messhias I doubt it this repor i sin code freeze maybe to the request at the new active repo https://github.com/tempusdominus/bootstrap-3 or https://github.com/tempusdominus/bootstrap-4


1188 remove mno pilot feature flags and virtual caps global feature flag

Context
Trello card
Remove the global feature flags :mno_offer and :virtual_caps and remove the mno_feature_flag column from schools table as these now apply to all.
Changes proposed in this pull request
Migration to removed boolean column from schools table
Updates to remove conditions for mno_feature_flag
Removed :mno_offer from FeatureFlag and updates to remove conditional handling where flag was used.
Removed :virtual_caps from FeatureFlag and updates to remove conditional handling where flag was used.
Guidance to review
Relying pretty heavily on the test suite, but all instances where these flags were considered across the code base should now be treated as though the flags are set.

This line needs updating too: https://github.com/DFE-Digital/get-help-with-tech/blob/main/app/models/responsible_body.rb#L108

This line needs updating too: https://github.com/DFE-Digital/get-help-with-tech/blob/main/app/models/responsible_body.rb#L108


feat(breadcrumb-item): add input to show items on sm screen

BREAKING CHANGE: showOnSmScreens attribute needs to be applied on breadcrumb items to enable them to be shown on small screen devices
Description
The current breadcrumb implementation does not allow the user to select which breadcrumb items should be visible on small screen devices automatically selecting what it considers to be the correct ones but this does not necessarily offer the flexibility an open source component library should provide.
In addition to this the current hideIcons mechanism offers little and can be removed as it only targets the first crumb item in a 2 item breadcrumb
     crumb.hideIcons = totalCrumbCount === 2 && !index;
The reason being, custom icons (those created via content projection) are typically only inserted on the first item in the breadcrumb (e.g. the home icon) and (as per our examples) we only show the penultimate breadcrumb item on small screens. It is simpler and more flexible to blanket hide all icons on small screens and (when/if the requirement arises) create an input of the form showContentIconOnSmScreens to allow the user to select which icons they want to show on small screens.
Screenshot
2 items tablet the content icon is visible

2 items sm screen the content icon is hidden

3 items tablet the content icon is visible

3 items sm screen the content icon is hidden

Fixes #15
Requirements
The existing components should behave in the same way as they currently do once the showOnSmScreens attribute is added to the relevant breadcrumb item. This applies to the breadcrumb component, card component, hero component and the test app.
Storybook link:
https://deploy-preview-167--legal-and-general-canopy.netlify.app/?path=/story/components-breadcrumb--two-items
https://deploy-preview-167--legal-and-general-canopy.netlify.app/?path=/story/components-breadcrumb--three-items
https://deploy-preview-167--legal-and-general-canopy.netlify.app/?path=/story/components-breadcrumb--asynchronous-loading
https://deploy-preview-167--legal-and-general-canopy.netlify.app/?path=/story/components-card--form-journey
https://deploy-preview-167--legal-and-general-canopy.netlify.app/?path=/story/components-hero--product-details
https://deploy-preview-167--legal-and-general-canopy.netlify.app/?path=/story/test-page--standard
Checklist:

[x] The commit messages follow the convention for this project
[x] I have provided an adequate amount of test coverage
[x] I have added the functionality to the test app
[x] I have provided a story in storybook to document the changes
[x] I have provided documentation in the notes section of the story
[ ] I have added any new public feature modules to public-api.ts
[ ] I have linked the new component to adobe DSM (if appropriate)

There is almost scope for a utility directive (similar to lgMargin / lgPadding) to make this re-usable across multiple components. A bit like the ones in bootstrap.
e.g. lgShowAt="lg", lgHideAt="md"

There is almost scope for a utility directive (similar to lgMargin / lgPadding) to make this re-usable across multiple components. A bit like the ones in bootstrap.
e.g. lgShowAt="lg", lgHideAt="md"

There is almost scope for a utility directive (similar to lgMargin / lgPadding) to make this re-usable across multiple components. A bit like the ones in bootstrap.
e.g. lgShowAt="lg", lgHideAt="md"

I think this is a great idea, I can imagine that being very useful on occasion where you just need to show/hide a component at certain breakpoints, but where it's tricky to do it via CSS (esp. with components added via content projection).

There is almost scope for a utility directive (similar to lgMargin / lgPadding) to make this re-usable across multiple components. A bit like the ones in bootstrap.
e.g. lgShowAt="lg", lgHideAt="md"

I think this is a great idea, I can imagine that being very useful on occasion where you just need to show/hide a component at certain breakpoints, but where it's tricky to do it via CSS (esp. with components added via content projection).

I'm going to close this as adding responsive options to mirror actual use cases unearthed more work that needs to be considered for the icon mangement


[5.4]Carbon miss data when date-time column without seconds

Laravel Version: 5.4.*
PHP Version:7.1.1
Database Driver & Version:MySQL 5.7

Description:
if input datetime without seconds will be failure when creating a new record.
Steps To Reproduce:
migration:
$table->dateTime('action_time');
controller:
$request->input('action_time'); //2017-02-09 11:00  momentjs format it as "YYYY-MM-D H:mm"
eloquent:
Actions::create(['action_time'=>$request->input('action_time')]);
//error  Carbon miss data.

I noticed asDateTime class doesn't have method to treat Carbon::createFromFormat if input without seconds.

Well the convention assumes it's a date or a dateTime value, you'll have to handle any other forms yourself before passing the value to Eloquent.


Seeing 'invalid frame header' and 'could not decode a text frame as UTF8' errors in chrome when connected to SuperWebSockets

Hello,
I'm seeing these errors in Chrome when connected to a SuperWebSockets server:
WebSocket connection to 'ws://<IP_ADDRESS>:10081/' failed: Invalid frame header
WebSocket connection to 'ws://<IP_ADDRESS>:10081/' failed: Could not decode a text frame as UTF-8.
Do you know what can cause this?  In my server, I'm using the default (text mode) connection and passing strings to the connection.

@kerryjiang - continue to run into this, and after some debugging, I believe I understand why.
In particular, this occurs when sending several large (~200k) text frames in quick succession.
At the TCP level, what we see is a well-formed text frame header (saying "I'm a 200k text frame") followed by the first ~64k (or more, or less) of one of those text frames. So far so good, but after around 64k (I don't know if that number is meaningful) there is suddenly the start of the subsequent text frame.
On the Chrome side, it sees the start of the first message, expects 200k of UTF-8 text, and throws a validation error when it reaches the 0x81 (the frame header.)
I believe the "Invalid frame header" has the same underlying cause but haven't demonstrated this.
On the SuperWebSocket side, we Send strings, these get formatted as WebSocket frames, and passed over to SuperSocket.
Once handed over to SuperSocket, these get queued to be sent asynchronously.
These get sent by invoking SendAsync on System.Net.Sockets.Socket.
At the system call level, a sendmsg call begins (we're running Mono on Linux). This does not send the complete message, and returns the number of actual bytes sent (~64k usually, although this number varies)
Back at the SuperSocket level, the OnSendingCompleted callback is invoked, which in turn consults ProcessCompleted.
This checks that e.BytesTransferred > 0, but does not check that the bytes transferred matches the full queue size. So, even though only part of that WebSocket frame was sent, SuperSocket thinks it's done and happily continues along its way.
...at least, that's my understanding of what's happening. I may have misconceptions about how these all fit together.

If your analysis is true, you probably can fix it  by switching to SuperSocket.WebSocket.
I believe the latest SuperSocket doesn't have this sending issue.

Thanks - I've tried this with SuperSocket.WebSocket v1.6.6 but still see this issue.

OK, let me review the code again and then give you an update!


proposal: golang.org/x/image/font/inconsolata package

The golang.org/x/image/font/... packages should provide a "batteries included" bitmap font, so that simple Go programs can draw some text easily without having to find (and depend on) both a "third party" font library (i.e. Go's freetype, which uses the Freetype license, so it can't be in the stdlib or golang.org/x) and third party font files (whose location and availability are OS-dependent at best).
Currently, that default font is based on the classic X11 fixed font, in the public domain. It's essentially a 1 bit per pixel font, even though Go's image.Alpha format for glyphs is 8bpp. It's also not the prettiest font.
I propose to pre-render a bitmap version of the Inconsolata font, and provide it as a package of Go source code, that would work with standard Go tools like "go get". The package path would be golang.org/x/image/font/inconsolata. This would export two variables, which users would refer to as inconsolata.Regular8x16 and inconsolata.Bold8x16, which implement the font.Face interface from golang.org/x/image/font.
That Go source code version of the Inconsolata bitmap data would be licensed the same as the rest of the Go standard library: https://golang.org/LICENSE. This is a different license than the SIL Open Font License that Inconsolata was released under. However, the fonts are copyright Raph Levien (who is employed by Google) and Cyreal, both have signed the CLA, and both approve of this specific "package inconsolata" Go source code proposal.
Inconsolata's home page is at http://www.levien.com/type/myfonts/inconsolata.html

Ability to license under Go's terms notwithstanding, I would like to throw another font as a possible replacement: Hack (https://github.com/chrissimpkins/Hack).
I spent a lot of time pre-rendering ttf fonts for the Plan 9 display subsystem (http://mirtchovski.com/p9/fonts/), i think i have plenty of experience with such fonts and, at least, the acme/rio display programs...
that said, inconsolata is not a bad font and if you can get it with the license you desire that may trump all other suggestions.

I have the license for Inconsolata that I desire.

This seems at ends from what I'd expect of an x/font library. I'm very much for "batteries included", but a single font at a single size with undesirable pixelation in either sizable direction? For a TTF? Is the size difference between embedding the TTF and prerendering the glyphs really that big of a difference?

The rationale has in fact been given: to render the TTF, you need the Go Freetype library. The library is under the Freetype license, which is more restrictive, thus it cannot be part of the standard library or /x/ packages.

Ability to license under Go's terms notwithstanding

Sadly the Hack licensing is very messy

I'm in favor of this proposal.

I'm not a huge fan of this proposal because it has the same problem as x/image/font/basicfont: It's only one non-standard font in a fixed font-size, with no obvious way to get variations (why regular and bold? Why not oblique? Or BoldOblique? Etc)
Freetype not being in the standard lib notwithstanding, it would be more useful to me to have a package that queries the system for what fonts are available, or all fonts that are available with a certain property (bold, italic, font family, font name,etc) perhaps with some helpers to get things like the default system monospace font or the default system sans serif font. Users would still need to import the freetype package to parse the file, but they need to do that in any other programming language too, and simply querying the OS font database shouldn't have any licensing problems.

@driusan

"Freetype not being in the standard lib notwithstanding"

This misses the point entirely. This proposal is to provide a better basic font for programmers who cannot (for whatever reason) use freetype.  If you can use freetype, you can use whatever true type font you want.

@adg I don't believe it does. Part of the stated reason for this is to not depend on "third party font files (whose location and availability are OS-dependent at best)."  This doesn't address that issue, it only includes a different third party pre-rendered font face in the standard package.
basicfont already covers the use case that you mention (not being able to use freetype or perhaps any font files at all)--so unless inconsolata covers a wider range of unicode, this is only a prettier font with the same purpose, and my objection is that arbitrarily adding single font faces is a slippery slope. If you're unable to use freetype but need something better than basicfont, there's no reason to assume that this face will cover your needs either, since it has the same limitations as basicfont (other than being 8bpp, but I don't think that's the limitation stopping basicfont from being useful--it's having different sizes or style variations which make people need to go elsewhere. )
If someone objects to freetype on licensing or other grounds but needs something better than basicfont, there's already x/image/font/plan9font to parse a different (more open, less common?) font file format.

Regardless, I assume you wrote some program to pre-render and embed the font into a package.
I imagine it's not super complicated, but publishing that separately, even if unofficially (and even if it'd take some community effort to generalize it), would let anyone package up whatever default font/sizes they required.
That seems like it would be useful in itself. (In favor of this proposal, ftr).

@driusan I don't think this package is trying to solve any of the problems you describe, it's just a slightly prettier implementation of basicfont. It is useful for the same set of tasks: tests and small examples with no dependencies outside golang.org/x.
In fact, now we have this I'd like to see basicfont deleted.

@crawshaw I know it isn't, that's my point. It's a second package with the same use case, benefits and limitations as an existing package. Having two packages with the same purpose is confusing, especially when the "right" one to use has the less intuitive name.
Removing basicfont would solve my objection, but if it's really just a prettier font why not just update the face (and documentation) in the existing package so that users get upgraded transparently? (Especially since the package in question is still under x/, and it's easier for a new user to tell what a package named basicfont is for than a package named inconsolata)

@jimmyfrasche that program is https://github.com/golang/freetype/blob/master/example/genbasicfont/main.go

@driusan @crawshaw
Yes, golang.org/x/image/font/plan9font already exists, and it's under golang.org/x, but it has the same problem in that you still need to provide a font file separately. The whole point, as crawshaw said, is for tests and small examples where you just want the simplicity of a single import and no further runtime dependencies. This proposed package does not replace freetype; if you want freetype, you can have freetype. But sometimes you don't want freetype's complications.
I will distinguish between basicfont.Face, the type, and basicfont.Face7x13, a value of that type based on the X11 fixed font. You can think of the proposed inconsolata.Regular8x16 as a better (nicer looking, wider unicode coverage) basicfont.Face7x13, and I agree we could probably delete the latter if the former lands. I don't think we can delete the basicfont package entirely, because others may want to use the Face type, especially given the genbasicfont program I just linked to.
As for being basicfont.Face8x16 or basicfont.InconsolataRegular8x16 instead of inconsolata.Regular8x16, at some point we are bikeshedding but I prefer inconsolata.Regular8x16. Note that it can't be a transparent re-purposing of the existing basicfont.Face7x13 name because the metrics are different (8x16 instead of 7x13), plus there's now regular and bold flavors.
A package to query what fonts the OS provides would indeed be useful, but if, in practice, you're querying for TTF fonts, then your program will pretty much need to depend on freetype, so such a package probably doesn't belong under golang.org/x. It could possibly be a new package under github.com/golang/freetype/... but that's a separate discussion.

You guys aren't ready to enter the bizarro world that is font licensing.
You can't license or trademark the actual font face graphics. You can only license the actual underlying software implementation of the font face.
http://blog.crowdspring.com/2011/03/font-law-licensing/

Using golang/freetype might not work with the license constraints, but
could we use the freetype package to generate a Go package that renders the
given font at any desired dimension? That is, we convert the glyphs and
hinting in ttf/otf into a Go package. I'd certainly want such a converter
program for some of my projects.
Also, have you considered the freely available Hershey vector fonts?

@minux: It should be straightforward, technically, to generate a Go package with the (scalable) vector data, and we could write a vector rasterizer under golang.org/x instead of github.com/golang/freetype.
Hinting is more difficult, as using the hinting data as is requires implementing a TrueType bytecode VM, and the Freetype VM implementation has cases where reality diverges from the spec (e.g. https://github.com/golang/freetype/commit/a31d0146).
I haven't considered the Hershey fonts for this proposal. I'll keep them in mind for future proposals, although they were designed for vector output (plotters) rather than raster output.


[Go] kubens failed to load previous namespace on Windows

I am using kubens v0.9.0 on Windows and when trying to switch to a namespace, the following error is printed.
error: failed to load previous namespace from file: open C:\Users\Trent Scholl\.kube\kubens\arn:aws:eks:ap-southeast-2:xxxxxxxxxxxx:cluster\cluster-name: The filename, directory name, or volume label syntax is incorrect.
This appears to be due to the colon in the cluster name (which is default when using the aws cli to update kube config for eks clusters)

Yep this was fixed in master. I will do a release soon.


/docs/user-guide/downward-api/ links to itself

Downward API page links to itself:
Some more thorough examples:
- environment variables
- downward API <-- this links to the same page

Fixed.


Update pypi

https://pypi.org/project/ollama-python/ is 0.1.2.

Wait, this project is at https://pypi.org/project/ollama/. What is https://pypi.org/project/ollama-python/?

Sigh, I guess someone else created that. Nevermind, I'll close this.


chore: remove all synchro-charts dependencies

Overview
remove all synchro-charts dependencies and replace them with either iot-app-kit imports or iot-app-kit-vis
Legal
This project is available under the Apache 2.0 License.

What's the state of this PR?

@corteggiano should this be converted to draft?


Make parameter registration part of parameter init

Such that parameters can be regular attributes on the instrument. This is lifted from #3188

Don't we want to advertize in the notebook that this way ofdeclaring parameters is now possible?) or is that left for further PRs?

Yes it needs some docs work before merging. I broke this out so that we could confirm individually that this is sound (the other pr breaks tests due to non related reasons)


Scala akka-http implementation of realworld

[Work in Progress] Link to repo
It is based on the scala play implementation.

Hi there.
Now I'm working on it. It's different from the top of comment repo.
https://github.com/1984weed/akka-http-realworld-example
If I could complete it, would you accept my repo?
thanks

@1984weed go for it! I have opened a new issue (#345) to track Akka HTTP.
Looks like Scala Play is already added to main READE so closing out this issue.


Login info for README.md

Adding info on how to connect and login into your Jenkins installation as an admin.
Took a full Jenkins noob (me) awhile to deduce, so want to save someone else's time.

Ok, tried another approach. Instead of writing a mundane instruction in docs, tried to autodetect possible server ip addresses on Jenkins start. Right from inside the container. However, the most agnostic approach through external server IP check does not really work because Jenkins doesn't work externally by default (it has internet connection but you cannot reach it's GUI from outside localhost or using WAN IP).


Add Gauge Brand Style guide to the readme doc

Under the 'Find out more' section of the readme.md file, a reference to the Gauge brand style guide should be added.
Title: Brand Style Guide
Copy: Brand Colours, Logos, Images etc.
Icon:
Link to https://brand.gauge.org

Now added to https://github.com/getgauge/gauge#find-out-more


Polymer 0.8 compatibility

Any plans to migrate app-router to Polymer 0.8?

I haven't tried 0.8 yet but I think everything should still work unless they changed the core-animated-pages API. The router is plain JS. I'll have to try out 0.8 soon though and probably update the examples to the new API.

There are some significant changes in custom element declaration and attributes configuration. Would be nice to have an official version for app-router as I'm planning to migrate my application myself because the 0.8 will be the base api towards 1.0 release.
Thank you.

The Polymer APIs changed, but the router doesn't use Polymer. It's a plain JS custom element. To that extent, Polymer's changes shouldn't affect the router at all.

Does the app-router require ShadowDOM? If so, then that would mean we would have to keep that polyfill just for app-router (since 0.8 no longer needs it).

It uses Shadow DOM if core-animated-pages is enabled.

I'm building an 0.8 app now, I'll try plugging in app-router and let you know how it goes.

Looks like it works! core-animated-pages hasn't been ported to 0.8 yet so that portion likely will need some updates when it has, but you can probably close this because I'm using it (without core-animated-pages) in 0.8 without problem
:+1: :smiley_cat:

Thanks @mbleigh! I'll keep the issue open as a placeholder for updating the documentation and examples.

@erikringsmuth core-animated-pages has moved under the grouping of neon-animation and specifically the neon-animated-pages element.  I don't see myself having the time and energy to cut this over to neon-animated-pages anytime soon. :+1: to anyone who wants to take this on and submit a PR!

@erikringsmuth will moving to neon-animation from core-animated-pages just involved changing the below code or there is more stuff involved. I can volunteer to change it to neon if its just that piece of code.  is it more then below?
 // <app-router core-animated-pages transitions="hero-transition cross-fade">
    if (router.hasAttribute('core-animated-pages')) {
      // use shadow DOM to wrap the <app-route> elements in a <core-animated-pages> element
      // <app-router>
      //   # shadowRoot
      //   <core-animated-pages>
      //     # content in the light DOM
      //     <app-route element="home-page">
      //       <home-page>
      //       </home-page>
      //     </app-route>
      //   </core-animated-pages>
      // </app-router>
      router.createShadowRoot();
      router.coreAnimatedPages = document.createElement('core-animated-pages');
      router.coreAnimatedPages.appendChild(document.createElement('content'));

      // don't know why it needs to be static, but absolute doesn't display the page
      router.coreAnimatedPages.style.position = 'static';

      // toggle the selected page using selected="path" instead of selected="integer"
      router.coreAnimatedPages.setAttribute('valueattr', 'path');

      // pass the transitions attribute from <app-router core-animated-pages transitions="hero-transition cross-fade">
      // to <core-animated-pages transitions="hero-transition cross-fade">
      router.coreAnimatedPages.setAttribute('transitions', router.getAttribute('transitions'));

      // set the shadow DOM's content
      router.shadowRoot.appendChild(router.coreAnimatedPages);

      // when a transition finishes, remove the previous route's content. there is a temporary overlap where both
      // the new and old route's content is in the DOM to animate the transition.
      router.coreAnimatedPages.addEventListener('core-animated-pages-transition-end', function() {
        // with core-animated-pages, navigating to the same route twice quickly will set the new route to both the
        // activeRoute and the previousRoute before the animation finishes. we don't want to delete the route content
        // if it's actually the active route.
        if (router.previousRoute && !router.previousRoute.hasAttribute('active')) {
          deactivateRoute(router.previousRoute);
        }
      });
    }

It looks like these 2 places contain all of the code:
https://github.com/erikringsmuth/app-router/blob/master/src/app-router.js#L63-L103
https://github.com/erikringsmuth/app-router/blob/master/src/app-router.js#L378-L414
There's probably an alternative event to core-animated-pages-transition-end that needs to be listened to.
https://github.com/erikringsmuth/app-router/blob/master/src/app-router.js#L95
Go for it! If you get it working with neon animations that'd be awesome!

I tried to inject neon-animated-pages functionality into app-router like core-animated pages, but I could not succeed and don't know where the cause is. Finally,  I rewrote the app-router with Polymer v1 and mixed it with neon-animation runner behaviors and some animation setups. It works.
I know this is not the best solution but it might help you @devAtPolydeals, @erikringsmuth.

@naoak  can you commit ur polymer v 1.0 compatible changes in your repo ?

I've not succeeded in making app-router v1 compatible yet, but this is the working repo which is written by Polymer v1.
https://github.com/naoak/tetra-router


How can I print Thai language?

Hi, it's very easy to support utf8 encoding which would help you print many languages. You just have to add an entry in the default section of capabilities.json and if you don't want to bother with that, you can use my fork:
https://github.com/waqadArshad/esc_pos_utils
as:
  esc_pos_utils:
    git:
      url: https://github.com/waqadArshad/esc_pos_utils.git

and if it fixes your issue, please remember to STAR the repo.

@waqadArshad Hi, I try with this. but still not working.
this is my code.
final profile = await CapabilityProfile.load();
final generator = Generator(PaperSize.mm80, profile);
bytes += generator.setGlobalCodeTable('UTF-8');

var encode = await CharsetConverter.encode('UTF-8', 'မြန်မာစာ');
bytes += generator.textEncoded(encode);
// or
var encode = Uint8List.fromList(utf8.encode('မြန်မာစာ'));
bytes += generator.textEncoded(encode);

this is print result

Hi, can you please check if your printer is set to UTF-8 encoding or not?

@waqadArshad printer Selfttest page show
Current character:
                   Chinese(GB18030)
                   Alphanumeric

then change this. there must be more encodings available.


Use a ConfigMap for FailedProvisionConfig

Create a ConfigMap containing FailedProvisionConfig and use it instead of the existing mechanisms:

Environment variables for log upload
Directly reading HiveConfig for RetryReasons

HIVE-1692

Needs UT & live test, but this is the gist.

/test e2e
Yeah I'm gonna fix the UT, but I suspect this is copacetic for runtime.

/lgtm

/test e2e-pool

This is working and ready for The Stamp.


CVE-2015-1793 in org.openssl.openssl

Address issues CVE-2015-1793 in org.openssl.openssl

Very real bug - upgrade to latest version


Update README.md for next version

Once #64 and #65 have been merged in, this can be merged and then the next version released

@amomchilov your feedback is most welcome

Excellent revision, this is great!

Thanks for the feedback. Will merge this after #64 has been merged, then make an official release.