Align in 3.0

Hi everyone,
Does the 3.0 works?

This is what it does:

[x] Add image (the image appends always in the center, with figure tags)
[ ] Align the image (it shows the buttons, but they do nothing)
[ ] Youtube Video -> Embeds. The icon to add it is there, tho it just throws a log console.

Cannot figure it out how to allign the images. With just that I'm free to go for now.
Thanks, I wasn't using the plugin for all the plugins it contains (beautiful functions, too much code), 'till I saw this. Even more awesome.

Hi @injectless. I see that you discovered our 3.0 branch. Unfortunately, v3.0 is not ready. As you can see in releases, v3.0 has not been released yet. So the branch you've discovered is just a work in progress.
What is done is basic scaffolding for the rest of the extension, image uploading, basic toolbar that doesn't work yet and placeholder for embeds.
So you've pretty much found what it does so far.

Wait!
Just, please.. give me a hint, something to just make the toolbar for align the image work, that's all I ask. I will love you forever.
 buttons: [{
	                    name: 'align-left',
	                    action: 'left',
	                    label: 'Izq'
	                }, {
	                    name: 'align-center',
	                    action: 'center',
	                    label: 'Ctr'
	                }, {
	                    name: 'align-right',
	                    action: 'right',
	                    label: 'Der'
	                }]

Now, I guess the selected image adds a class somewhere ¿?

@injectless assuming that this plugin isn't doing anything to change what happens when toolbar buttons are clicked, if you're just overriding these options in buttons, the click of the button will be handled here.
If you don't override it, it will ultimately call execAction() with the value of the action property you defined here (ie 'left', 'center', 'right').  This will then call execActionInternal on the editor itself (code here) which will try to have the browser handle it via document.execCommand('action').  This won't do anything, as the browser would only understand justifyLeft, justifyCenter, justifyRight, and justifyFull.  If you use those justify actions instead of your 'left', 'center', 'right' actions, what happens?
If you want to fiddle around with adding a class or what-not, I would start just by overriding the handleClick method and see if you can do something there.  You can override it by passing it as an additional property to 'name', 'action', and 'label':
buttons: [{
    name: 'align-left',
    action: 'left',
    label: 'Izq',
    handleClick: function (evt) {
        // do your thing here
    }
}];

Hello, you can use this fork https://github.com/anorudes/medium-editor-embed-plugin. Here there is, showing the loader, upload through third-party widgets, adding a description to the image.
I made it a separate repository, because I do not know whether the author will accept these changes to himself


Implement CSRF protection

Check if the state parameter returned is consistent with the value sent, as described in the Linkedin OAuth2 documentation: https://developer.linkedin.com/docs/oauth2

@fajarmf How does this look to you? If you're happy to merge this, I think it would make sense to bump the version number to 0.2.0.

Hei @rorygreen, Sorry I have only have time on the project now. I am testing your PR now.

 Thank you .


SVGs Not Scaling Correctly

Hi,
Thank you for Tweakpane.

This works really well in general since Tweakpane is all vectors, but I did notice an issue with SVG graphics like those found in the graph monitor and the FPS graph plugin. Basically, the SVG seems to get scaled twice, and then is partially clipped from view.

I dug around a bit and it seems like this is a result of calling getBoundingClientRect() when calculating the graph's layout, which changes as a result of the scale transformation on the parent.
I have a branch here which seems to fix the issue by using the view element's .clientWidth and .clientHeight instead, which reflects the unscaled values:

I'm happy to submit this as a PR if you agree with the approach, but I'm not sure if there are unintended side-effects I haven't considered. I also noticed that getBoundingClientRect() is used in pointer-handler.ts, but haven't encountered issues with it. (I think in some cases of user input coming from the root window coordinates, you do want the scaled dimensions.)
I think Cubic Bezier from the Essentials Plugin would need the same fix as well.
I also experimented with setting the viewBox on the SVG. This approach fixes the scaling, but has the disadvantage of not scaling up the line weights.

And here's A diff of the same fix for Cubic Bezier.

Thank you for reporting the issue. You got a point and the proposed fix looks good to me. Creating a PR would be appreciated.
PointerHandler works fine because the scale of transform affects both x/y and width/height and the relative value　doesn't change as a result.

Great thank you. I've opened PRs on both Tweakpane and the Plugin Essentials repo with the fix.

Released in 4.0.2, thank you for your contribution! (and I'll do merge your PR with plugin-essentials later)


Provide option to change default protocol

Is your feature request related to a problem? Please describe.
I observed that my connection is very slow and laggy with Cloudflare's default setting on QUIC, but it fast being set on HTTP2.
Describe the solution you'd like
Please add addon configuration to set the default setting as described here
Describe alternatives you've considered
I don't see any.
Additional context

Thanks for the suggestion. Quite some time ago, we made the decision to keep the add-on as slim as possible, we therefore added the possibility to use the add-on with a remote managed tunnel.
In the remote set-up, you can manage your tunnel completely from the Cloudflare Zero Trust Dashboard and also change the protocol. I do suggest to switch to this set-up, so you are able to edit everything.
I am therefore closing this, please let me know if there is anything else.
P.S. for requests in the future please use this repository, which is the individual repo for the add-on. This repo here is simply the add-on repo to add to HA.

Thank you for the quick response, @brenner-tobias.
However, I think I can only configure the protocol of indivual routes behind the tunnel client, i.e. whether the tunnel client communicates with an HA addon by using https or http other something else.
What I don't see in the Dashboard is an option to configure the protocol between the client and the Cloudflare Tunnel server, but instead must be added to the config file here.

Great point, I started a discussion here about the way forward. Feel free to add there.


config param is missing

https://github.com/se-panfilov/mini-toastr/blob/7c9af2139db7310aa407be96ed3acd7e5b3a93fb/mini-toastr.js#L191
The config param is missing in this call, which makes the code throw an error. :(

Fixed in 0.8.1


Added Fade-In CSS animation

Added simple fade-in css animation for new list items.
Made a few additional css changes to address the following:

new fadeIn animation was conflicting with highlighting, added important to highlighting tag
removed user selection from list items (stop click and drag highlighting on UI)

These changes are made to address issue #9

@negue thanks for the feedback, I've implemented your suggested changes.

Thank you for your first contribution!!
If you want and have an idea, feel free to add a "item remove" animation as well :)
@all-contributors please add @SteveAndreou for code

@all-contributors please add @SteveAndreou for code


Update tuto_creating_new_kernels.rst

Corrected non-compiling code and removed mention of non-existing steps for updating src file.

Codecov Report

Merging #699 into deploy will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           deploy     #699      +/-   ##
==========================================
- Coverage   53.99%   53.97%   -0.02%
==========================================
  Files         207      207
  Lines       20953    20953
  Branches     2818     2609     -209
==========================================
- Hits        11314    11310       -4
- Misses       9123     9125       +2
- Partials      516      518       +2

This is a docs-only change, I plan to merge this after the checks on the update finish.

Closing due to no activity & merge conflicts


Added codes for dry, fan_only and auto modes for all 4 fan speeds (fanModes).

Sorry, could you please explain what do you mean by 'partial device files'? I think, I have made it pretty solid. Also why would you like me to remove the 'cool' section? Thanks

I can't accept this pull request because of missing fan modes in dry mode. Could you repeat the dry->auto commands for low, mid, high?

I can't accept this pull request because of missing fan modes in dry mode. Could you repeat the dry->auto commands for low, mid, high?

I cannot, because the AC makes it impossible. When I switch to dry mode, the mode field on the screen of the ac remote control goes to 'auto' mode and does not allow to switch between fan modes. I guess it is by design for some reasons..


Remove workaround for "pq: could not serialize access due to read/write dependencies among transactions"

https://github.com/tinkerbell/tink/issues/338 has been resolved, so we should be able to mostly remove it (as deleting workflows seems to be still affected).

@invidian you mean remove the error check for that? or there is more code? asking that because I did not see any TODO in the code related to this

@invidian you mean remove the error check for that? or there is more code? asking that because I did not see any TODO in the code related to this

I think we can remove the calls to retryOnTransientError from everywhere except the workflow removal code, as it still requires it.
But before doing that, we need to make sure that sandbox version we use includes https://github.com/tinkerbell/tink/pull/346.
Perhaps we could also wait for https://github.com/tinkerbell/tink/pull/395.

@invidian both are merged, do you think we can move forward?

@invidian both are merged, do you think we can move forward?

True, thanks for the heads up. Also sandbox repository got updated by now, for which I created the PR already (https://github.com/tinkerbell/terraform-provider-tinkerbell/pull/23).
However, I hit this issue: https://github.com/tinkerbell/tink/issues/413 and I'm still wondering if we should update to newer version.

True, thanks for the heads up. Also sandbox repository got updated by now, for which I created the PR already (https://github.com/tinkerbell/terraform-provider-tinkerbell/pull/23).
However, I hit this issue: https://github.com/tinkerbell/tink/issues/413 and I'm still wondering if we should update to newer version.

Actually, https://github.com/tinkerbell/tink/issues/338 has been re-opened, as it still occurs when resources are removed.

Actually, https://github.com/tinkerbell/tink/issues/338 has been re-opened, as it still occurs when resources are removed.


[$250] Code block markdown renders incorrectly

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

Version Number: 9.0.71-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @MonilBhavsar
Slack conversation (hyperlinked to channel name): expensify_bugs
Action Performed:

In any chat, type the following message - "hello `` hi`" (remove double quotes)
Send the message

Expected Result:
Message composed as hello `` hi`
Actual Result:
Message composed as hello `` hi`
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021866767677427055045
Upwork Job ID:<PHONE_NUMBER>427055045
Last Price Increase: 2024-12-11
Automatic offers:

QichenZhu | Contributor | 105481643

Issue OwnerCurrent Issue Owner: @allroundexperts

This isn't really related to convert/migrate/retain so I added it to quality.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Markdown doesn't currently support empty code blocks.
What is the root cause of that problem?
The user expects double backticks in the middle itself to be an empty code block. This is also mentioned in this comment.
`hello` `` hi`
However, the current regex doesn't allow this and requires inline code blocks to contain at least one non-whitespace character, a space, or a no-break space.
https://github.com/Expensify/expensify-common/blob/6611dab75bcbf6ec5950401990299f883536a6c0/lib/ExpensiMark.ts#L230

What changes do you think we should make in order to solve the problem?
Now that we allow spaces as content, there's no need to disallow empty content, as it's confusing for both users and developers. I suggest simplifying the regex.
regex: /(\B|_|)`(.*?)`(\B|_|)(?!(?!
)[^<]*(?:<(?!pre>)[^<]*)*<\/pre>|[^<]*<\/video>)/gm,

Resolving this issue will also unblock https://github.com/Expensify/App/pull/52780#issuecomment-2535937359.
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?
New test case to add
Test string: "`hello` `` hi`"
Expected result: "hello  hi`"

Existing test cases to update
Test inline code blocks with double backticks
Expected result before: "My favorite language is `JavaScript`. How about you?"
Expected result after: "My favorite language is JavaScript. How about you?"

Test inline code blocks with triple backticks
Expected result before: "My favorite language is ``JavaScript``. How about you?"
Expected result after: "My favorite language is JavaScript. How about you?"

Test inline code with one backtick as content
Expected result before: "```"
Expected result after: "`"

Test inline code with multiple backtick symbols as content
Expected result before: "``````"
Expected result after: ""

Test inline code blocks with two backticks
Expected result before: "`JavaScript`"
Expected result after: "JavaScript"

Test for backticks only tab characters inside it
Expected result before: "`	`"
Expected result after: "	"

Test for inline code block with triple backtick with spaces as content
Expected result before: "``   ``"
Expected result after: "   "

What alternative solutions did you explore? (Optional)

@QichenZhu's proposal looks good to me. I am pretty confident that if all our existing unit tests on regex pass after this change, we won't introduce any regression.
🎀 👀 🎀 C+ reviewed

@jasperhuangg Kind bump

PR is under review.


App's localizations synthetic package autogeneration doesn`t work, i10n.yaml settings are ignored

Steps to Reproduce
Follow https://flutter.dev/go/i18n-user-guide:

flutter create <name_of_flutter_app>
Add the i18n dependencies to your app’s pubspec.yaml file.
Create a  new configuration file for localizations called l10n.yaml.
Create a new “template” message catalog, like lib/l10n/app_en.arb.

Expected results:
5. When the application is run, a new class that provides access to the message catalog will be generated automatically each time the application is run or restarted as part of the build process.
Actual results:
Auto-generation doesn`t work when "flutter run" is executed.
It has to be done manually every time when new messages are added:
flutter gen-l10n --untranslated-messages-file=lib/l10n/untranslated.json --header="// GENERATED CODE - DO NOT MODIFY BY HAND"

Logs
λ flutter gen-l10n --verbose
[ +128 ms] executing: [C:\flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +59 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[   +1 ms] f30b7f4db93ee747cd727df747941a28ead25ff5
[   +1 ms] executing: [C:\flutter/] git tag --points-at HEAD
[  +56 ms] Exit code 0 from: git tag --points-at HEAD
[        ] 1.22.0-12.4.pre
           1.22.1
[   +9 ms] executing: [C:\flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +31 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[   +1 ms] origin/stable
[        ] executing: [C:\flutter/] git ls-remote --get-url origin
[  +27 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[  +65 ms] executing: [C:\flutter/] git rev-parse --abbrev-ref HEAD
[  +29 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] stable
[  +49 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +3 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +12 ms] Artifact Instance of 'MaterialFonts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'GradleWrapper' is not required, skipping update.
[        ] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[        ] Artifact Instance of 'FlutterSdk' is not required, skipping update.
[        ] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[   +5 ms] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FontSubsetArtifacts' is not required, skipping update.
[ +134 ms] To see a detailed report, use the --untranslated-messages-file
                    option in the tool to generate a JSON format file containing
                    all messages that need to be translated.
[  +11 ms] "flutter gen-l10n" took 188ms.
[   +5 ms] Running shutdown hooks
[        ] Shutdown hooks complete
[   +4 ms] exiting with code 0

λ flutter doctor -v
[√] Flutter (Channel stable, 1.22.1, on Microsoft Windows [Version 10.0.19041.508], locale en-US)
    • Flutter version 1.22.1 at C:\flutter
    • Framework revision f30b7f4db9 (5 days ago), 2020-10-08 10:06:30 -0700
    • Engine revision 75bef9f6c8
    • Dart version 2.10.1

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at C:\Android\SDK
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = C:\Android\SDK
    • Java binary at: C:\Android\Java\jdk1.8.0_261\bin\java
    • Java version Java(TM) SE Runtime Environment (build 1.8.0_261-b12)
    • All Android licenses accepted.

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[!] VS Code, 64-bit edition (version 1.49.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    X Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[√] Connected device (1 available)
    • Android SDK built for x86 64 (mobile) • emulator-5554 • android-x64 • Android 10 (API 29) (emulator)

! Doctor found issues in 2 categories.

Hi @ofury
This is a false positive duplicate of https://github.com/flutter/flutter/issues/67138
To generate, app imports and delegate, you will see errors but execute flutter gen-l10n, errors will go away
import 'package:flutter_gen/gen_l10n/app_localizations.dart'; // Add this line.

    return MaterialApp(
      // Add the `localizationsDelegate` and `supportedLocales` lines.
      localizationsDelegates: AppLocalizations.localizationsDelegates,
      supportedLocales: AppLocalizations.supportedLocales,

complete code sample
import 'package:flutter/material.dart';
import 'package:flutter_gen/gen_l10n/app_localizations.dart'; // Add this line.

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      // Add the `localizationsDelegate` and `supportedLocales` lines.
      localizationsDelegates: AppLocalizations.localizationsDelegates,
      supportedLocales: AppLocalizations.supportedLocales,
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        // Replace the title line with the following:
        title: Text(AppLocalizations.of(context).helloWorld),
      ),
      // The rest of the widget tree.
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.1, on Mac OS X 10.15.7 19H2, locale en-GB)
    • Flutter version 1.22.1 at /Users/tahatesser/Code/flutter_stable
    • Framework revision f30b7f4db9 (5 days ago), 2020-10-08 10:06:30 -0700
    • Engine revision 75bef9f6c8
    • Dart version 2.10.1

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.10.0.rc.1

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.50.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.15.0

[✓] Connected device (1 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64 • Android 10 (API 29)

! Doctor found issues in 1 category.

Closing as duplicate of  https://github.com/flutter/flutter/issues/67138

Hi @ofury
This is a false positive duplicate of #67138
To generate, app imports and delegate, you will see errors but execute flutter gen-l10n, errors will go away
import 'package:flutter_gen/gen_l10n/app_localizations.dart'; // Add this line.

    return MaterialApp(
      // Add the `localizationsDelegate` and `supportedLocales` lines.
      localizationsDelegates: AppLocalizations.localizationsDelegates,
      supportedLocales: AppLocalizations.supportedLocales,

complete code sample
import 'package:flutter/material.dart';
import 'package:flutter_gen/gen_l10n/app_localizations.dart'; // Add this line.

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      // Add the `localizationsDelegate` and `supportedLocales` lines.
      localizationsDelegates: AppLocalizations.localizationsDelegates,
      supportedLocales: AppLocalizations.supportedLocales,
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        // Replace the title line with the following:
        title: Text(AppLocalizations.of(context).helloWorld),
      ),
      // The rest of the widget tree.
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.1, on Mac OS X 10.15.7 19H2, locale en-GB)
    • Flutter version 1.22.1 at /Users/tahatesser/Code/flutter_stable
    • Framework revision f30b7f4db9 (5 days ago), 2020-10-08 10:06:30 -0700
    • Engine revision 75bef9f6c8
    • Dart version 2.10.1

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.10.0.rc.1

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.50.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.15.0

[✓] Connected device (1 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64 • Android 10 (API 29)

! Doctor found issues in 1 category.

Closing as duplicate of #67138

@TahaTesser
I think its not a duplicate.  The tool is supposed to generate the localizations automatically each time the application is run or restarted as part of the build process, but it doesnt.

If you add imports, this is generated, without imports, you get a false positive message in the logs, check the mentioned issue

@TahaTesser
As i mentioned in the issue running flutter gen-l10n works.
It will generate localizations one time and we need to do it manually every time when new messages are added.
Also l10n.yaml will be ignored.
The problem I want to report is that the localizations now won`t be generated automatically each time the application is run or restarted as part of the build process.

@ofury
Yes, for new messages, you will execute flutter run to generate new messages
Like this
initial generation is done and app is running

Added new message for greeting

execute flutter run or flutter gen-i10n

@TahaTesser
the issue is that after adding a new message and executing "flutter run" code for new messages isnt generated. Its generated with "flutter gen-i10n" but l10n.yaml settings are ignored.

Hi @ofury
Can you please specify what changes are you making to l10n.yaml

@TahaTesser
l10n.yaml
untranslated-messages-file: lib/l10n/untranslated.json
header: "// GENERATED CODE - DO NOT MODIFY BY HAND"

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.1, on Mac OS X 10.15.7 19H2, locale en-GB)
    • Flutter version 1.22.1 at /Users/tahatesser/Code/flutter_stable
    • Framework revision f30b7f4db9 (8 days ago), 2020-10-08 10:06:30 -0700
    • Engine revision 75bef9f6c8
    • Dart version 2.10.1

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.10.0.rc.1

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.50.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.15.0

[!] Connected device
    ! No devices available

! Doctor found issues in 2 categories.

Hi @ofury, I cannot seem to reproduce this issue (I'm using this Flutter app as a reference). It has the l10n.yaml file that you specified. Every time I update the .arb file and reload/restart the Flutter app, it results in an update in the AppLocalizations class containing the newly added method. I tried this on both the stable and master channels and seems to work okay.

Hi @shihaohong, I tried your template and it worked as expected.
But it doesn`t work if I create a new project by myself or test localization on the old ones.
Here is my steps for a new one:

create new project

Logs
λ flutter create flutter_internationalization
Creating project flutter_internationalization...
  flutter_internationalization\.gitignore (created)
  flutter_internationalization\.idea\libraries\Dart_SDK.xml (created)
  flutter_internationalization\.idea\libraries\KotlinJavaRuntime.xml (created)
  flutter_internationalization\.idea\modules.xml (created)
  flutter_internationalization\.idea\runConfigurations\main_dart.xml (created)
  flutter_internationalization\.idea\workspace.xml (created)
  flutter_internationalization\.metadata (created)
  flutter_internationalization\android\app\build.gradle (created)
  flutter_internationalization\android\app\src\main\kotlin\com\example\flutter_internationalization\MainActivity.kt
  (created)
  flutter_internationalization\android\build.gradle (created)
  flutter_internationalization\android\flutter_internationalization_android.iml (created)
  flutter_internationalization\android\.gitignore (created)
  flutter_internationalization\android\app\src\debug\AndroidManifest.xml (created)
  flutter_internationalization\android\app\src\main\AndroidManifest.xml (created)
  flutter_internationalization\android\app\src\main\res\drawable\launch_background.xml (created)
  flutter_internationalization\android\app\src\main\res\mipmap-hdpi\ic_launcher.png (created)
  flutter_internationalization\android\app\src\main\res\mipmap-mdpi\ic_launcher.png (created)
  flutter_internationalization\android\app\src\main\res\mipmap-xhdpi\ic_launcher.png (created)
  flutter_internationalization\android\app\src\main\res\mipmap-xxhdpi\ic_launcher.png (created)
  flutter_internationalization\android\app\src\main\res\mipmap-xxxhdpi\ic_launcher.png (created)
  flutter_internationalization\android\app\src\main\res\values\styles.xml (created)
  flutter_internationalization\android\app\src\profile\AndroidManifest.xml (created)
  flutter_internationalization\android\gradle\wrapper\gradle-wrapper.properties (created)
  flutter_internationalization\android\gradle.properties (created)
  flutter_internationalization\android\settings.gradle (created)
  flutter_internationalization\ios\Runner\AppDelegate.swift (created)
  flutter_internationalization\ios\Runner\Runner-Bridging-Header.h (created)
  flutter_internationalization\ios\Runner.xcodeproj\project.pbxproj (created)
  flutter_internationalization\ios\Runner.xcodeproj\xcshareddata\xcschemes\Runner.xcscheme (created)
  flutter_internationalization\ios\.gitignore (created)
  flutter_internationalization\ios\Flutter\AppFrameworkInfo.plist (created)
  flutter_internationalization\ios\Flutter\Debug.xcconfig (created)
  flutter_internationalization\ios\Flutter\Release.xcconfig (created)
  flutter_internationalization\ios\Runner\Assets.xcassets\AppIcon.appiconset\Contents.json (created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
  flutter_internationalization\ios\Runner\Assets.xcassets\LaunchImage.imageset\Contents.json (created)
  flutter_internationalization\ios\Runner\Assets.xcassets\LaunchImage.imageset\LaunchImage.png (created)
<EMAIL_ADDRESS>(created)
<EMAIL_ADDRESS>(created)
  flutter_internationalization\ios\Runner\Assets.xcassets\LaunchImage.imageset\README.md (created)
  flutter_internationalization\ios\Runner\Base.lproj\LaunchScreen.storyboard (created)
  flutter_internationalization\ios\Runner\Base.lproj\Main.storyboard (created)
  flutter_internationalization\ios\Runner\Info.plist (created)
  flutter_internationalization\ios\Runner.xcodeproj\project.xcworkspace\contents.xcworkspacedata (created)
  flutter_internationalization\ios\Runner.xcodeproj\project.xcworkspace\xcshareddata\IDEWorkspaceChecks.plist (created)
  flutter_internationalization\ios\Runner.xcodeproj\project.xcworkspace\xcshareddata\WorkspaceSettings.xcsettings
  (created)
  flutter_internationalization\ios\Runner.xcworkspace\contents.xcworkspacedata (created)
  flutter_internationalization\ios\Runner.xcworkspace\xcshareddata\IDEWorkspaceChecks.plist (created)
  flutter_internationalization\ios\Runner.xcworkspace\xcshareddata\WorkspaceSettings.xcsettings (created)
  flutter_internationalization\lib\main.dart (created)
  flutter_internationalization\flutter_internationalization.iml (created)
  flutter_internationalization\pubspec.yaml (created)
  flutter_internationalization\README.md (created)
  flutter_internationalization\test\widget_test.dart (created)
Running "flutter pub get" in flutter_internationalization...         1.5s
Wrote 71 files.

All done!
[√] Flutter: is fully installed. (Channel stable, 1.22.2, on Microsoft Windows [Version 10.0.19041.572], locale en-US)
[√] Android toolchain - develop for Android devices: is fully installed. (Android SDK version 30.0.2)
[!] Android Studio: is not available. (not installed)
[!] VS Code, 64-bit edition: is partially installed; more components are available. (version 1.49.0)
[√] Connected device: is fully installed. (1 available)

Run "flutter doctor" for information about installing additional components.

In order to run your application, type:

  $ cd flutter_internationalization
  $ flutter run

Your application code is in flutter_internationalization\lib\main.dart.

λ flutter doctor -v
[√] Flutter (Channel stable, 1.22.2, on Microsoft Windows [Version 10.0.19041.572], locale en-US)
    • Flutter version 1.22.2 at C:\flutter
    • Framework revision 84f3d28555 (6 days ago), 2020-10-15 16:26:19 -0700
    • Engine revision b8752bbfff
    • Dart version 2.10.2

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at C:\Android\SDK
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = C:\Android\SDK
    • Java binary at: C:\Android\Java\jdk1.8.0_261\bin\java
    • Java version Java(TM) SE Runtime Environment (build 1.8.0_261-b12)
    • All Android licenses accepted.

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[!] VS Code, 64-bit edition (version 1.49.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    X Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[√] Connected device (1 available)
    • Android SDK built for x86 64 (mobile) • emulator-5554 • android-x64 • Android 10 (API 29) (emulator)

! Doctor found issues in 2 categories.

copy "lib" folder and pubspec.yaml from flutter_internationalization_workflow template.

run the app

@ofury In the steps you described in the previous comment, I was able to make this work by adding an l10n.yaml file, which you might have missed. Could you try it again with a blank l10n.yaml file?

I am facing the same issue, 2-3 weeks ago 1.22.1 was released and everything was working fine, the files were generated each time i edited one of the .arb files or by running pub get and on flutter run too.
but after 1.22.2 things have stopped working automatically, i have l10n.yaml file but still files are not getting generated, i have to execute flutter gen-l10n to generate files

I would say that https://flutter.dev/go/i18n-user-guide is misleading.
It differs from https://marketplace.visualstudio.com/items?itemName=localizely.flutter-intl which seems to be more relevant
though there is a similar issue reported https://github.com/localizely/flutter-intl-vscode/issues/48
Most notable setup differences are
flutter_intl:     enabled: true
instead of
`flutter:
Adds code generation (synthetic package) support
generate: true
in pubspec.yaml and import 'generated/l10n.dart';instead ofimport 'package:flutter_gen/gen_l10n/app_localizations.dart';`
as the official guide suggests.

https://marketplace.visualstudio.com/items?itemName=localizely.flutter-intl is a separate third party package/plugin that we do not maintain.
On the other hand, https://flutter.dev/go/i18n-user-guide is a tool that the we have been working on, which can be made use of through flutter gen-l10n or the l10n.yaml file.

@shihaohong everything seems to work right now for me. Thanks

Hi @ofury
Given your last message I feel safe to close this issue, if you disagree please write in the comments and I will reopen it.
Thank you

I ran into the same issue and saw that I accidentally created l18n.yml instead of l18n.yaml. Changing it to .yaml seems to have solved the problem for me.

FormatException: Unexpected character (at line 128, character 1)
}
^
Generating synthetic localizations package has failed.
If this error in you app
fixed this error this step
remove latest comma in the file .arb


Running on multiple gpus

Thank you for your work!
How can I run this code on multiple gpus?

Hi! While the current implementation only uses 1 GPU by default, you can look into PyTorch Distributed Data Parallel (https://pytorch.org/docs/stable/notes/ddp.html) for multi-GPU support. You would also need to add some logic for checking the local_rank of the device and doing some operations in a non-parallel fashion (e.g., wandb logging or model checkpointing).
If folks have a strong interest in multi-GPU support please feel free to comment / like this thread, and I can start a branch for this.


Adding "offset start" and "offset end" for each word?

Would be great to see offset added in output arrays. What do you think?
Sentence {
  taggedWords:
   [ { token: 'Another', tag: 'DT' },
     { token: 'occasion', tag: 'NN' },
     { token: 'of', tag: 'IN' },
     { token: 'his', tag: 'PRP$' },

Yes, that would be great. I will put it on the backlog. Feel free to do a PR to realize this feature.
Hugo


Maybe we shouldn't duplicate packages from our own org?

I'm sort of struggling to understand the situation there, maybe @mkborregaard can help me see things through?
We already have GBIF.jl, we are in the process of adding the download API to it, what is the point of GBIF2.jl that cannot be accomplished with a PR to the existing (and published) package?
I'm afraid that the trend of building a second package with the same functionalities done differently is sending a terrible message to would-be users, and is fundamentally detrimental to the aims of EcoJulia as a project.
This is particularly bad because (i) it's not the first time this happens within the org and (ii) this package is introducing even more types, whereas we discussed harmonizing things.

Wow.
I made some PRs that had little attention at GBIF.jl https://github.com/EcoJulia/GBIF.jl/pull/41. It was faster to do this from scratch that modify the existing package with friction and slow/no PR replies. It was also nice to rethink the architecture.
This was a days work a few months ago when I was getting frustrated with the other package, and work today getting it cleaned up. It's not worth this amount of concern.
Maybe "thanks for putting the work in to writing a high quality package", would be more appropriate than whatever this comment is.

I am not going to waste time on the comment about "high quality package". I also would like to point out, as a professional courtesy, that email exists and could be a good way to flag a slow PR.

I did ping you on github in two places in that PR, from which you received 2 emails.
And you are meaning to say you didn't see the one PR open on your package sitting there for a year unmerged.


Add Jest for unit testing

Related to #37 - added Jest as used by react-art, a similar library to react-canvas. Includes one simple test as an example, @mjohnston if this seems like it appropriate tool choice, I can add a few more tests to better assess the fit for this project.

If this gets pulled in, I would like to contribute by writing test-cases for the rest of the existing code :+1:

@brentvatne This looks like a good choice, thanks for getting this rolling.
@darkyen :+1:


Check server CMD crash restart interval restart server everytime

Hi,
I've set Check server CMD crash restart interval restart server even if cmd is still running...
Have I missed something ?
Thanks

Ahh yes i forgot to change the process name for it since theyve now changed it to PalServer-Win64-Shipping-Cmd , ill update it now.

Hello, issue is now fixed.

Thanks a lot !

no problem at all, thanks for letting me know😄.


koin_devtools 0.2.2 incompatible with koin_flutter 0.11.0-nullsafety.0 and above

Hi @chplalex
For now, it is not yet possible to use koin_devtools with the null safety versions of koin_flutter.
I haven't finished the null safety version of koin_devtools yet because I'm waiting for the null safety version of one of the dependencies.
Soon we will have the koin_devtools version.

@pbissonho which package are you waiting on for a null safe version?
Having a look at koin_devtools pubspec it looks like all the dependencies now have null-safe versions published?

I've create a PR for it https://github.com/pbissonho/koin.dart/pull/16


[DOC] Document important runtime constants

Is your feature request related to a problem? Please describe.
There are plenty of runtime constants, like default amount of gas for view functions, max contract size, etc. We should have a list of them with explanations and SEOable.
For example we have a ton of constants here: https://github.com/nearprotocol/nearcore/blob/05a82d8dd74531b21567623ab0e7e75946afea19/runtime/near-vm-logic/src/config.rs#L124

we have this: https://docs.nearprotocol.com/docs/api/vm-limits
does that satisfy the request?

closing since it seems like this was done


[Bug] Set-DbaLogin is performing no action

Report

Host used

[ ] powershell.exe
[x] ISE
[ ] VS Code
[ ] Other (please specify)

Errors Received

replace THIS text WITH the OUTPUT of -- $ERROR[0] | SELECT *

Steps to Reproduce

Expected Behavior

Actual Behavior

Environmental information
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨
Please provide the output of the below script
Set-DbaLogin -SqlInstance 'instancename.fqdn' -Login sa -Disable -WhatIf
[pscustomobject]@{
'PowerShell Version' = $PSVersionTable.PSVersion.ToString()
'dbatools latest installed' = (Get-InstalledModule -Name dbatools).Version
'Culture of OS' = (Get-Culture)
} | fl -force

PowerShell Version        : 5.1.14393.3866
dbatools latest installed : 1.0.84
Culture of OS             : en-
SQL Server:
Microsoft SQL Server 2017 (RTM-CU22) (KB4577467) - 14.0.3356.20 (X64)   Aug 20 2020 22:33:27   Copyright (C) 2017 Microsoft Corporation  Developer Edition (64-bit) on Windows Server 2016 Standard 10.0  (Build 14393: ) (Hypervisor)
/* REPLACE WITH output of @@VERSION */

/* REPLACE WITH output of @@LANGUAGE */
us_english

Hi @firthro, can you please edit your submission and fulfil the details?
Thanks

Seems related to #6336


New Tutorial Bugged out on Evaluate

🐞 bug report
Describe the bug
It appears that on some PC's the POE Overlay Tutorial is not able to be completed. It appears that the tutorial is unable to read when a PC performs the evaluate. (Example, I was able to evaluate 12 items successfully, but the tutorial popup will not move to the next step or allow itself to be skipped.
https://imgur.com/eoEjhSY
https://imgur.com/Etx0ytA
https://imgur.com/bDlGy4N
🌍  Your Environment

Version: Latest Overwolf Version
PoE: Steam 3.11 e

Same. Never was able to complete tutotrial. Stuck at this point. Tried uninstall and reinstall. Tutorial resumed at the same stuck spot.


Unable to get Landlord tenant to work.

Hi,
I have followed your instructions but this package isnt injecting the where "company_id = X" into the SQL queries.

Added the Landlord Service Provider in the config/app.php
Added the alias in the config/app.php
Created config file and set the tenant_id to company_id
Created and set the landlord middleware and I know that works
Added BelongsToTenants scope to the user model

When all call $usersList = User::all('name', 'email','phone_number','profile_url'); in the userController, the company_id is not injected in the sql. I even had \Landlord::enable() called before that to make sure it is on.
What do you think I could be doing wrong?
Thanks

Hi,
some checks:

do your users records have the company_id field and are filled with the correct id ?
are you sure the middleware is being called at request chain? you can do a dd in the handle function to check

Hi,

Yes, the users table has a company_id field and all are correctly filled. It reference a column on another table to, so it isnt allowed to be empty.

Yes, my middleware is being called. dd in the handle kills the request and show the text I placed in the function.

I am wondering if there is some Laravel compatibility issue I am over looking.

Well, it seems everything is ok based on your descriptions, may you give me access to a repository so I could  try to help you to find the problem?

Hey, i am facing this exact problem. I used the package before and worked perfectyl, but now it doesn't seem to be working on Laravel 7..
@adnansyedimadi how did you solve this?
Thanks

I have some issue. This is not working if scope in controller.
But if I call model with BelongsToTenants in route its working well.
I use Laravel 8, PHP 8.

I have some issue. This is not working if scope in controller.
But if I call model with BelongsToTenants in route its working well.
I use Laravel 8, PHP 8.

No. I never really got it working. I ended just manually adding the scopes to my SQL calls.
@nunomazer I just saw this your comment. Sorry. I might have missed it. I can't give you access due to company's proprietary work. I will likely DM you if I need help with this package again. Right now, trying to get this package to work been shelved, as manually doing is working well for my needs for now.

No. I never really got it working. I ended just manually adding the scopes to my SQL calls.
@nunomazer I just saw this your comment. Sorry. I might have missed it. I can't give you access due to company's proprietary work. I will likely DM you if I need help with this package again. Right now, trying to get this package to work been shelved, as manually doing is working well for my needs for now.

@adnansyedimadi can you share pieces of the code here ? Only the necessary to understand where did ou tried to scope the tenant that didn't work

@adnansyedimadi can you share pieces of the code here ? Only the necessary to understand where did ou tried to scope the tenant that didn't work

@arislanhaikal could you please open a new issue with your problem? I'm closing this since @adnansyedimadi said it has the problem solved by another strategy.

@arislanhaikal could you please open a new issue with your problem? I'm closing this since @adnansyedimadi said it has the problem solved by another strategy.


Add raven to track errors

Not been able to test this yet since I don't have a Sentry DSN for this project. I CC'ed you on an email about setting this up. If that's a problem, perhaps we can create a new project in one of your teams @pmac

Not been able to test this yet since I don't have a Sentry DSN for this project. I CC'ed you on an email about setting this up. If that's a problem, perhaps we can create a new project in one of your teams @pmac

Sounds good. I'll see about creating an app if we hear back in the negative.

If you can see that, can you add SENTRY_DSN into the Heroku instance?

Can't see it yet. I'll do that as soon as I get access.

Done. SENTRY_DSN is added to env vars on Heroku.


(WIP)(refactor)operator:created seperate function to check annotation of app.

Signed-off-by: Amit Bhatt<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Checklist:

[ ] Fixes #
[ ] Labelled this PR & related issue with documentation tag
[ ] PR messages has document related information
[ ] Labelled this PR & related issue with breaking-changes tag
[ ] PR messages has breaking changes related information
[ ] Labelled this PR & related issue with requires-upgrade tag
[ ] PR messages has upgrade related information
[ ] Commit has unit tests
[ ] Commit has integration tests

Multiple changes to the controller since this PR, will be closing this. We will raise a new PR to add annotation checks as a separate fn.


Alternative of tox w.r.t CUE tests

Discussed in https://github.com/cue-lang/cue/discussions/2227

Originally posted by debuggerrr January 24, 2023
I was wondering if there could be anything like tox in python as standard testing practice. I did research but didn't find anything. Can anyone help me with how can I achieve similar things in CUE?

You already opened this as a discussion, and this seems like a question, so I'd suggest to keep the thread there. It's also best to avoid duplicating the threads :)


buildDockerImage.sh fails with <IP_ADDRESS> se (chown: changing ownership of '/opt/oracle/install/linuxx64_12201_database.zip': File too large)

Hello there,
It is me again , now trying to build Oracle Database <IP_ADDRESS> se, you can have a look at  buildDockerImage from https://github.com/oracle/docker-images fails with <IP_ADDRESS> xe
Same host:
CentOS Linux release 7.2.1511 (Core)
Linux version 3.10.0-327.10.1.el7.x86_64<EMAIL_ADDRESS>(gcc version 4.8.3 20140911 (Red Hat 4.8.3-9) (GCC) ) #1 SMP Tue Feb 16 17:03:50 UTC 2016
Now trying...
./buildDockerImage.sh -v <IP_ADDRESS> -s -i
I get:

Complete!
Loaded plugins: ovl
Cleaning repos: ol7_UEKR4 ol7_latest
Cleaning up everything
chown: changing ownership of '/opt/oracle/install/linuxx64_12201_database.zip': File too large
Removing intermediate container 6622686f4e18
The command '/bin/sh -c chmod ug+x $INSTALL_DIR/*.sh &&     sync &&     $INSTALL_DIR/$CHECK_SPACE_FILE &&     $INSTALL_DIR/$SETUP_LINUX_FILE' returned a non-zero code: 1
There was an error building the image.

Am I missing something? Apart that perhaps I can try to rename the binaries file to something shorter...
Thanks in advance,
Luis

Hi @lurodrig, I don't think that it has to do with the filename but rather with the file size. Although I have never seen chown failing with such an error. Perhaps this is due to some file system limitation? Which fs are you using inside your container?

Hello @gvenzl. Yes, you are right, I've just realized that it was nothing to do with the file name, I was very naive... :)
Abot the fs (thanks, good hint!):
The host machine (Centos7):
[dockerfiles]$ df -T -h /dev/mapper/cc_xxxx-home
Filesystem                            Type   Size    Used   Avail   Use%   Mounted on
/dev/mapper/cc_xxxx-home  xfs      94G   11G     84G    12%      /home

And inside the container (not 100% sure if this is you are asking me...):
Step 7/17 : RUN df -T -h
 ---> Running in f80559f3fc3b
Filesystem                    Type     Size  Used Avail Use% Mounted on
overlay                              overlay   50G   34G   17G  67% /
tmpfs                                 tmpfs    2.3G     0  2.3G   0% /dev
tmpfs                                 tmpfs    2.3G     0  2.3G   0% /sys/fs/cgroup
/dev/mapper/cc_xxxx-root xfs       50G   34G   17G  67% /etc/hosts
shm                                   tmpfs     64M     0   64M   0% /dev/shm
tmpfs                                 tmpfs    2.3G     0  2.3G   0% /sys/firmware

Thanks in advance,
Luis

This looks good to me. @Djelibeybi, do you have any idea why chown would fail with this error?

I have the same problem... Filesystem xfs on OL7.3...
$: chown oracle:dba linuxx64_12201_database.zip
chown: changing ownership of 'linuxx64_12201_database.zip': File too large
bash-4.2# df -hT (Inside the container...)
Filesystem          Type     Size  Used Avail Use% Mounted on
overlay             overlay   41G   17G   25G  41% /
tmpfs               tmpfs    496M     0  496M   0% /dev
tmpfs               tmpfs    496M     0  496M   0% /sys/fs/cgroup
/dev/mapper/ol-root xfs       41G   17G   25G  41% /etc/hosts
shm                 tmpfs     64M     0   64M   0% /dev/shm
tmpfs               tmpfs    496M     0  496M   0% /sys/firmware

@sniper66 @gvenzl I'm not sure the root cause of this issue, it looks there's sufficient disk space. While we definitely should avoid chown'ing large files like linuxx64_12201_database.zip during the image building. chown actually adds new image layer thus makes the built image that file size larger.

@chrisguoado Thank you for your reply...
I can´t do anything with that file...
Just take a look here:
bash-4.2# chmod 777 linuxx64_12201_database.zip
chmod: changing permissions of 'linuxx64_12201_database.zip': File too large
bash-4.2# mv linuxx64_12201_database.zip linuxx64_12201_database.zip_test
mv: cannot move 'linuxx64_12201_database.zip' to 'linuxx64_12201_database.zip_test': File too large
bash-4.2# cat /etc/oracle-release
Oracle Linux Server release 7.3
bash-4.2# uname -a
Linux ad0cb32cda6c 4.1.12-61.1.28.el7uek.x86_64 #2 SMP Thu Feb 23 19:55:12 PST 2017 x86_64 x86_64 x86_64 GNU/Linux
On the host OL7.3 I can do anything...

Yes, it certainly only relates to the operations in the container. But I guess you should be able to unzip the file, and then remove it. If so, it looks like there's something wrong or limitation with the driver handling the (xfs) FS.

Yes, unzipping and removing works in the container...

Does anyone know which limitations that could be???

I do believe that this issue is related with #326
@sniper66 perhaps you could try docker info and look for Data Space Total
You can have a look at https://jpetazzo.github.io/2014/01/29/docker-device-mapper-resize/
Cheers,
Luis

I already changed it to overlay2...
Storage Driver: overlay2
Backing Filesystem: xfs
Supports d_type: false
Native Overlay Diff: false

Hello there,
Extending the docker Base Device Size to 20GB did the trick for me. I configured it via /etc/docker/daemon.json See Configure the Docker daemon with specific devicemapper options. on https://docs.docker.com/engine/userguide/storagedriver/device-mapper-driver/#configure-direct-lvm-mode-for-production
My /etc/docker/daemon.json looks like:
{
    "live-restore": true,
    "storage-opts": ["dm.basesize=20G"]
}

Hope it helps,
Luis
ps: thanks for all your responses

Another user came across a Docker bug with CentOS 7 and overlayfs, see issue #317. I believe that this is related to that.
Also the Docker documentation recommends to use overlay2:
Docker and OverlayFS in practice

Since version 1.12, Docker also provides overlay2 storage driver which is much more efficient than overlay in terms of inode utilization. The overlay2 driver is only compatible with Linux kernel 4.0 and later.

Was it switching to overlay2 that fixed the issue?

Hello Gerald,
Sorry, perhaps I was not very clear. Which fixed the issue for me was to set dm.basesize to 20GB.
Cheers,
Luis

Thanks @lurodrig, that is very strange to me. Why would not having enough space throw an error with chown File too large. Anyway, glad that the issue is solved!
Thanks!

Hi,
I don't think this is fixed.
I am running OL 7.3 and I am still getting:
chown: changing ownership of '/opt/oracle/install/linuxx64_12201_database.zip': File too large
docker info:
Server Version: 17.03.1-ce
Storage Driver: overlay2
Backing Filesystem: xfs
Supports d_type: true
Native Overlay Diff: false

The overlay2 does not have the option "storage-opts"
<IP_ADDRESS> works fine.
I am not sure what to try next since overlay2 is the recommended storage driver.
Thanks,
Bjarte

Hi, seeing the same issue here with CentOS and overlay2, "file too large" when opening a 2.3G tablespace file while starting an Oracle database.


Fix typos in comment

This PR is a

[ ] Documentation

What this PR does / why we need it:
Fix typos in comment
Which issue(s) this PR fixes
Fixes #
Please leave this checklist in the PR comment so that maintainers can ensure a good PR.
Merge Checklist:

[ ] Documentation

/assign @kibbles-n-bytes

/ok-to-test
/lgtm


 Component of Card Not Working

I am currently using "material-ui": "^1.0.0-beta.16"
React is not able to read  Component of "Card".
Does anyone having same issue?

Your issue has been closed because it does not conform to our issue requirements.
Please provide a reproduction test case. This would help a lot 👷 .
A live example would be perfect. This codesandbox.io template may be a good starting point. Thank you!


Add serde support

This pull request adds serde support behind a "serde" feature.
This is allows you to convert the parsed gvas data to and from many common formats such as JSON. This can be useful to process the data in other programming languages or general purpose JSON processing tools (for example generate TypeScript types from JSON to help determine the structure of the data).
For the most part this involves just adding #derive(Serialize, Deserialize) to relevant structs, however, there are some places where custom logic was added:

The Property enum is internally tagged, with the property type contained in the field "type", because this felt like the most compact and natural option.
MapProperty values are serialized as a list of {"key": ..., "value": ...} structs. This was done because JSON only allows string keys in maps.
If a StructProperty contains a GUID it is serialized with an additional field "value" containing the GUID value as a string in the standard format (matching the Debug trait logic). This was done to make working with GUID StructProperties more convenient and more consistent with how GUID-s are shown in other parts of the serialized output. The "properties" field is serialized for all StructProperties to allow for generic processing of all StructProperties regardless of if they contain a GUID or not.

Here is an example of a gvas save file (my DRG save) and the corresponding JSON produced by serde: gvas_json_example.zip


Data/code missing from task-DyVA paper

We could add in the data/code from the 2023 task-DyVA paper. It looks like this would be difficult to do myself since the list is built automatically and the relevant config is encrypted (but let me know if I could somehow).
paper DOI: 10.1038/s41562-022-01510-8
code: https://github.com/pauljaffe/task-dyva
data DOI: 10.5281/zenodo.6368412

Done!  for future reference, you can add it to academicdb/links.csv and then ask me to rebuild the database.
also - for future reference, it would be good to have a fixed reference for the code, either a github commit ID or (preferably) push the code to a zenodo repo (which is easy via the github/zenodo integration)


Create pl_pl.json

I would change a few things:

"Czernitowa kuchenka" instead of "Czernitowy Stove"
"Mimigoździk" or "Mimikarnacja" instead of "Mimicarnation"
two e's in "Talerz faszerowaneej pieczeni hoglina" and in "Netherowa ceeglana wędzarka"
"Kawałek magmołaza" instead of "Plasterek magmołaza"
"grilowany" is written with two l's -> "grillowany"

Thank you for this insight! those indeed sound better. I already switched allow edits by maintainers on. You can make improve it now :D

Yeah, did the cool goździk

R u polish by any chance? I can't imagine anyone seeing those errors and correct them without being polish XD

R u polish by any chance? I can't imagine anyone seeing those errors and correct them without being polish XD

I am

W takim razie szalom XDD Świetny pomysł na moda swoją drogą 😄

To nie mój mod, dołączyłem tutaj niedawno raczej. Mam swojego, niedokońcoznego, moda, o którym wie niewiele osób - jak jesteś na serwerze dsc Chef's Delights, to mogę coś wysłać wsm, bo czemu nie.
For non-polish people watching this - dw I didn't say anything illegal xd


Added "Mental Model" section to README.

Also fixed "geoHaystack" value.

This change is

@rueckstiess This is a thing of absolute beauty... Thanks!


cuda10.2+vs2019+gtx1050Ti，can not run true result.Result are all<PHONE_NUMBER>1805717965

Does cuda version affect the results？

I believe CUDA version should not be a major problem.
Given the result, I suspect that we are running out of GPU memory for your gtx 1050 Ti (with 4 GB memory). We didn't have a memory limit check.
I have two suggestions:

Could you try the code with a small dataset, e.g., mnist?
We didn't test SONG on Windows platform. I wonder whether you have the access for Linux.

If you provide more details of your dataset, your configuration, and the commands you ran, we may be able to have a try to reproduce the problem.


Free real times frame interpolation solution

It's free and can be used with mpc-be for real times frame interpolation.
Maybe you can add vapoursynth, vs-mlrt and avisynth_filter to README?
https://github.com/vapoursynth/vapoursynth
https://github.com/AmusementClub/vs-mlrt
https://github.com/CrendKing/avisynth_filter
avisynth_filter script
import os
import sys
import math

from vapoursynth import core
import vapoursynth as vs

# core.num_threads = 12
core.max_cache_size = 24000

sys.path += [r'vsmlrt.py path'] # your vsmlrt.py path

from vsmlrt import RIFE, RIFEModel, Backend

pp = VpsFilterSource
pp = core.resize.Spline36(pp, range_in_s='full', range_s='full', matrix_s='rgb', matrix_in_s='709', format=vs.RGBS, dither_type='error_diffusion')
pp = core.resize.Spline36(pp, format=vs.RGBH)
w_pad = ((math.ceil(pp.width / 32) * 32) - pp.width) / 2
h_pad = ((math.ceil(pp.height / 32) * 32) - pp.height) / 2
pp = core.std.AddBorders(pp, left=w_pad, right=w_pad, top=h_pad, bottom=h_pad)
pp = RIFE(pp, model=RIFEModel.v4_22, backend=Backend.TRT(fp16=True, num_streams=2), multi=2, video_player=False)
# pp = RIFE(pp, model=RIFEModel.v4_22_lite, backend=Backend.TRT(fp16=True, num_streams=2), multi=2, video_player=False)
pp = core.std.Crop(pp, left=w_pad, right=w_pad, top=h_pad, bottom=h_pad)
pp = core.resize.Spline36(pp, format=vs.RGBS)
pp = core.resize.Spline36(pp, range_in_s='full', range_s='full', matrix_s='709', matrix_in_s='rgb', format=vs.YUV444P8, dither_type='error_diffusion')

pp.set_output()

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

I don't quite understand, what should I write?

From https://github.com/hzwer/ECCV2022-RIFE?tab=readme-ov-file#software this repo https://github.com/HomeOfVapourSynthEvolution/VapourSynth-RIFE-ncnn-Vulkan doesn't exist anymore.
This fork is still actively maintained: Vapoursynth-RIFE-Vulkan


CCM/CSI migration support for clusters with static worker nodes

What this PR does / why we need it:
This PR introduces CCM/CSI migration support for clusters with the static worker nodes. As described in #1495, kubeadm cannot be used to handle updating kubelet configuration and flags. This is because kubeadm init (used for control plane nodes) works differently than kubeadm join (used for static worker nodes).
To mitigate that, we manually update the required files, which we already do for Docker to containerd migration.
The workflow is:

Cordon and drain a static worker node
Download the kubelet config file (/var/lib/kubelet/config.yaml)
Add the required feature gates
Upload the new kubelet config file
Download the kubeadm-flags.env file (/var/lib/kubelet/kubeadm-flags.env)
Set --cloud-provider=external and remove the --cloud-config flag
Upload the updated kubeadm-flags.env
Restart kubelet and wait up to 2 minutes for it to become running
Uncordon the node

This PR has been already tested on an OpenStack cluster.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1495
Fixes #1450
Does this PR introduce a user-facing change?:
CCM/CSI migration support for clusters with static worker nodes

/hold
to manually test it

/hold cancel
tested manually and everything works fine

/retest


🚀 Feature Request: onlineornot whoami should say which permissions the token has

Describe the solution
rozenmd@thinkpad-pro:~$ npx onlineornot@latest whoami
 ✅ onlineornot 0.0.10
----------------------
Getting User settings...
👋 You are logged in with an API Token.

is nice, but I don't know what the token is for

Need to use https://api-docs.onlineornot.com/operations/api-tokens#token-permissions to make it happen


Setting status is not accessible by keyboard

does not seem to be a way to get to one's user profile at all
once in the dialog, you can shift+tab to the set icon space, but it only shows the tooltip

Disclosure: I am not a screen-reader user, or even a power keyboard user, but given this experience and #15962 and #15946 and #16723 i am afraid that if i were to invite a screen-reader user to Zulip they would declare it unusable. Zulip seems to need much greater focus on accessibility.

[ x ] Zulip Cloud (*.zulipchat.com)

OK it looks like #27017 and #27018 would address my second point, and maybe the first point is doable if i knew how?
I hope that accessibility can be made the focus of a release cycle and tests and other practices put in place to keep Zulip as accessible as possible.  Agaric could likely contribute financially in Q1 2024 to such an effort and perhaps a little bit with code and testing— really important for us to see this prioritized.

You can get to the user profile via G (gear menu), and arrowing down to Personal settings. I'm not sure there's any way to change your avatar via the keyboard from there, though.

Or was it the user card that you were trying to get to? https://zulip.com/help/user-cards

Getting to the "set a status" UI is now possible via the keyboard since #25667; G, Right, Down, Down, Enter. (We've also fixed a bunch of known keyboard accessibility issues with other popovers).
I think the remaining bug here is that hitting Enter in this state does not open the emoji picker:

@sayamsamal would you be able to pick that up?

I hope that accessibility can be made the focus of a release cycle and tests and other practices put in place to keep Zulip as accessible as possible. Agaric could likely contribute financially in Q1 2024 to such an effort and perhaps a little bit with code and testing— really important for us to see this prioritized.

That's great! @mlncn we'd be very happy to collaborate with you on accessibility work for Zulip in Q1! Feel free to get in touch in chat.zulip.org when you're ready to discuss in more detail.

Opened PR #27507, to address emoji picker not opening through Enter in Set State modal.

Closing as I believe both the issues preventing this are now fixed with the merge of #27507. Thanks @mlncn again for the report, please be encouraged to open new issues for other keyboard accessibility issues you encounter.


Extend GitRepoTests to allow include mode

The GitRepoTests can be initialized with a ValidateWorkingTree
mode that is currently a boolean. If enabled, we will add extra
checks to make sure the working tree of our virtual enlistment
matches the enlistment from a vanilla Git repo.
To extend this to the "include" mode, we need to first modify
this boolean into an enum and add it as an extra option.
Second, we need to be very careful about what we are comparing.
We will no longer virtualize outside of the included folders.
Be careful to include files in the root directory.
The functional tests can then be run with "--partial-mode" to
exercise only this option, which will isolate the test failures
to this setting AND keep the test suite from getting too slow.
Currently, the partial mode space is probably wider than
necessary and can be reduced in later iterations. This
does provide some coverage.
Signed-off-by: Derrick Stolee<EMAIL_ADDRESS>

/azp run

/azp run PR - Windows - Functional Tests (Include Mode)

/azp run PR - Windows - Functional Tests (Include Mode)

/azp run PR - Windows - Functional Tests (Include Mode)

/azp run PR - Windows - Extra

/azp run PR - Windows - Extra

/azp run PR - Windows - Functional Tests (Include Mode)

/azp run GitHub VFSForGit Mac Functional Tests

/azp run PR - Windows - Functional Tests (Include Mode)


fixing the time bar

i dunno why it dont work

https://github.com/Test-Engine-Team/Test-Engine-V1---FNF/pull/29


Update page title with current song

Created function to change page title with current playing song.
Fixed some formatting.
I am not 100% sure if this is how the function/changes should be laid out. I have never actually coded in angular before.

Thanks for this, as long as it works I'm not too bothered if it isn't the best AngularJS in the world :)


Model Silhouette is not the expected color

Sandcastle example: Here
Browser: Chrome
Operating System: macOS
Note that when Forestgreen is selected for the model and the silhouette, the colors look fairly different - the silhouette is much lighter. When Lime is selected for both, they look about the same. This appears to be due to the fact that Lime is a more saturated color (#228B22 vs #00FF00). This lightening effect holds true in general, not just for these colors. It is not clear from looking at the API why this would be the case.

https://github.com/AnalyticalGraphicsInc/cesium/pull/7563 should fix it.
When the color blend mode is Replace the colors should now be identical. It also looks better for the highlight and mix modes.

Tested out the fix - looks good. Thanks.


Docs: Introducing isort 5: Typos and consistency

PS The pencil icon at https://timothycrosley.github.io/isort/docs/major_releases/introducing_isort_5/ leads to a 404 page:

https://github.com/timothycrosley/isort/edit/master/docs/major_releases/introducing_isort_5.md

Should it link to the develop branch?

https://github.com/timothycrosley/isort/edit/develop/docs/major_releases/introducing_isort_5.md

Codecov Report

Merging #1244 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop     #1244   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           34        34
  Lines         2132      2132
=========================================
  Hits          2132      2132

This is awesome, thank you! And yes, it should ideally point to develop. I'll look into how to fix it.


⚠ Samsung has degraded performance

In a4e5773, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 4655 ms

Resolved: Samsung performance has improved in 86196a4.


Consider adding the sphinx-extension GitHub topic

Just wanted to say you should consider adding the sphinx-extension topic, https://github.com/topics/sphinx-extension, to the repo so it is easier for others to find on the GitHub ecosystem. There are currently 94 other sphinx extensions that use that topic.

thanks @jdillard -- will do so. We've been focusing our efforts these past couple of months on jupyter-book which also uses sphinx under the hood -- but will revisit this extension soon with a sharper focus on building jupyter notebooks.

Done.


fix: 解决Breadcrumb面包屑导航404问题

General

✏ Mark the necessary items without changing the structure of the PR template.

[ ] Pull request template structure not broken

Type

ℹ What types of changes does your code introduce?

👉 Put an x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

ℹ Check all checkboxes - this will indicate that you have done everything in accordance with the rules in CONTRIBUTING.

👉 Put an x in the boxes that apply.

[ ] My code follows the style guidelines of this project
[ ] Is the code format correct
[ ] Is the git submission information standard?
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@sugar258596  抱歉，我选择了 https://github.com/vbenjs/vue-vben-admin/pull/3337 这个PR来解决同样的问题，他这个修改代码可读性可复用性更好一些~， 非常感谢你对vben的关注和贡献。我们非常欢迎你继续参与贡献vben，继续提交你的想法和改进😁。


update docs on hub.docker.com

I wasted a lot of time as the variables used in docs here don't work: https://hub.docker.com/r/cschlosser/drone-ftps/
The syntax has also changed,
The docs on github seem right with everything being set from environment variables.

Automated dockerhub build was broken.


Disable some capabilities in missed spots

Changes proposed in this pull request:

Don't expose the item handler capability in some missed spots where there never are any slots for it.
Changes Oredictionificator to be able to accept/extract from any side except the front, instead of connect to on any side but only insert on the left and extract on the right.

Looks good on the surface (minor niggle that it could have maybe been a rebase instead?)
Have you tested with null side requests for itemhandlers? Easiest is historically The One Probe's inventory mode

Looks good on the surface (minor niggle that it could have maybe been a rebase instead?)

I realized that I did it off a previous branch by accident when I was making the PR

Have you tested with null side requests for itemhandlers? Easiest is historically The One Probe's inventory mode

I forgot about null sided itemhandler requests, however I just tested with TOP's inventory mode and it seems to be able to show it correctly so I guess it is working properly. (I should probably double check it at some point but for now I think it is fine.)

Yep, if it shows the complete contents it should be good as far as I recall (we wrote the special read-only NullItemHandler for that specifically)


proposal: spec: Enhance Generics to Support Pointer Receivers in Go

Go Programming Experience
Intermediate
Other Languages Experience
Go, Python, Rust, Java, JS
Related Idea

[ ] Has this idea, or one like it, been proposed before?
[ ] Does this affect error handling?
[X] Is this about generics?
[X] Is this change backward compatible? Breaking the Go 1 compatibility guarantee is a large cost and requires a large benefit

Has this idea, or one like it, been proposed before?
Yes
Does this affect error handling?
No
Is this about generics?
Yes, this proposal is about generics. While it builds on Go's accepted generics design, it addresses a specific gap that has not been tackled in previous proposals: The inability to constrain type parameters to pointers and handle pointer receiver methods in generic functions.
I found no similar proposal, making this a unique addition to Go’s generics. It introduces pointer-specific constraints and safe instantiation, enabling type-safe patterns like cloning while keeping Go simple and secure, eliminating the need for unsafe type conversions or extra checks to ensure correctness..
Proposal
Proposed Change
This proposal introduces pointer-specific constraints for generics in Go, allowing type parameters to be constrained to pointer types (*T). This enables developers to more safely and effectively work with structs that have pointer receivers in generic functions, addressing limitations that arise when trying to use pointer receivers in generics.
Who Does This Proposal Help, and Why?
This proposal helps Go developers working with generics, especially when dealing with structs that have pointer receivers. It eliminates the need for complex type conversions and additional runtime checks, enabling more intuitive and type-safe code while keeping Go simple.
Change to the Language
Pointer-Specific Constraints: The core change is the introduction of pointer-specific constraints for generics. By allowing type parameters to be constrained to pointer types (*T), this proposal makes it easier to work with structs that have pointer receivers in generic functions.
Language Spec Changes
Change to the Language Specification
This proposal adds support for pointer-specific constraints in Go’s generics. The main change is the ability to limit type parameters to pointer types (*T), allowing generic functions to work with pointers in a safe and straightforward way.
Key Changes:
Pointer-Specific Constraints:
The language will allow type parameters in generics to be constrained to pointer types, such as *T. This enables functions and types to accept only pointers to certain types.
For example, a function could be defined like this:
func Clone[*T *Object](o T) *T

Current Problems Without Pointer Constraints:
Problem Statement Example: Cloning an Object with Pointer Receiver
We have an interface Object and a struct ObjectA implementing it. The struct ObjectA uses a pointer receiver for its method. The goal is to create a generic cloning function that can directly handle this scenario without requiring a type conversion from the interface to the struct.

Problem 1: If we try to use a type parameter with an interface constraint and pass a pointer to it (e.g., [T InterfaceConstraint](obj *T) *T), it gets rejected by the compiler because the method is expected to use a pointer receiver, but Go cannot safely handle this in the current generic system.

type Object interface {
    GetName() string
}

type ObjectA struct {
    Name string
}

func (a *ObjectA) GetName() string {
    return a.Name
}

// Problem 1: This is rejected by the compiler because `T` isn't constrained to pointer types.
func Clone[T Object](obj *T) *T {
    return obj
}

Problem 2: If we try to use a type parameter with a value ([T InterfaceConstraint](obj T) T), Go will treat T as a value, not a pointer. As a result, Go replaces T with the value type (e.g., *SomeStruct), but we cannot use new(SomeStruct) or new(T) to instantiate the pointer because T is treated as a value type, not a pointer.

type Object interface {
    GetName() string
}

type ObjectA struct {
    Name string
}

`T` is treated as a value type, and `new(T)` won't work as `T` is not constrained to pointer types.
func Clone[T Object](obj T) T {
    return obj
}

With Pointer-Specific Constraints (After Proposal):
type Object interface {
    GetName() string
}

type ObjectA struct {
    Name string
}

func (a *ObjectA) GetName() string {
    return a.Name
}

// This works because *T (pointer) is used to check against the Object interface, not the struct itself.
func Clone[*T Object](obj *T) *T {
    return obj
}

Informal Change
No response
Is this change backward compatible?
Yes, this change is fully backward compatible. It introduces a new feature that allows pointer receivers to be used in generic constraints, but it does not modify or invalidate any existing syntax. All previously valid code remains functional.
Orthogonality: How does this change interact or overlap with existing features?
No response
Would this change make Go easier or harder to learn, and why?
No response
Cost Description
No response
Changes to Go ToolChain
go compiler
Performance Costs
What is the compile-time cost? The compile-time cost is O(1). It involves passing a single boolean flag (checkWithPointerInstead) and applying a simple logic to treat the struct type as a pointer during type checking. This addition does not introduce significant overhead to the compilation process.  What is the runtime cost? There is no change in runtime cost. The proposed feature is resolved entirely at compile time, meaning the resulting binary executes with the same efficiency as before, without any additional runtime overhead.
Prototype
the PR draft is here: https://github.com/golang/go/pull/70959
and the PR works on my local

The change is primarily focused on the syntax parser and type checker.
In the syntax parser, when the type *T is encountered with an interface like SomeInterface, a flag checkWithPointerInstead is set to true and passed to the type checker.
During the type checking phase, if the checkWithPointerInstead flag is true, the type is transformed into a pointer type. This is done by wrapping the original type with a Pointer structure like this:
V = &Pointer{base: V}.
This transformation allows for proper handling of pointer types during type checking without altering the existing syntax or runtime behavior.

Related Issues

proposal: spec: apply constraint type inference to generic types #62460
proposal: Go 2: a type set with a single type should act more like that type #66123 (closed)
proposal: spec: allow methods on generic structs which meet subset of type constraint #65394
proposal: Go 2: allow pointer types to be receiver base types in method declarations #48592 (closed)
proposal: spec: support user-defined variance (allow type parameters as type bounds) #67513 (closed)
proposal: Go 2: spec: allow conversion from *T to *[1]T #45545 (closed)
proposal: spec: support for struct members in interface/constraint syntax #51259
proposal: Go 2: generic interface method receivers #54347 (closed)
spec: add generic programming using type parameters #43651 (closed)
proposal: spec: allow type conversion from generic type to same underlying type #61164

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

I believe the appropriate constraint is
type Object[T any] interface {
    GetName() string
    *T
}

I believe the appropriate constraint is
type Object[T any] interface {
    GetName() string
    *T
}

Oui

@seankhliao

That's not solve the problem, cannot initialize *O using new / &O{}

if using *O it's rejected by the compiler

va o O = new(T)

I see. Thanks for the information 👍


Proposing changes to German translation

Fixing typos
Proposing changes to unideal German translations

Before being merged, I would like some other German sausage to look over my proposals, some are mandatory (e.g. "Benuzter") others are style related and might not be necessary.

please pull from staging instead of new-design


Ups! Some badness happened:  Data size must be a multiple of framesize

Hi.
Excuse my poor english.
I've got the same problem of this issue :  #64
I'm trying to run spotify-web-connect on Libreelec with my usb DAC.
LibreELEC:~/.kodi/addons/service.spotify-connect-web# aplay -L
null
    Discard all samples (playback) or generate zero samples (capture)
default:CARD=ALSA
    bcm2835 ALSA, bcm2835 ALSA
    Default Audio Device
sysdefault:CARD=ALSA
    bcm2835 ALSA, bcm2835 ALSA
    Default Audio Device
default:CARD=DA300USB
    DA-300USB, USB Audio
    Default Audio Device
sysdefault:CARD=DA300USB
    DA-300USB, USB Audio
    Default Audio Device
front:CARD=DA300USB,DEV=0
    DA-300USB, USB Audio
    Front speakers
surround21:CARD=DA300USB,DEV=0
    DA-300USB, USB Audio
    2.1 Surround output to Front and Subwoofer speakers
surround40:CARD=DA300USB,DEV=0
    DA-300USB, USB Audio
    4.0 Surround output to Front and Rear speakers
surround41:CARD=DA300USB,DEV=0
    DA-300USB, USB Audio
    4.1 Surround output to Front, Rear and Subwoofer speakers
surround50:CARD=DA300USB,DEV=0
    DA-300USB, USB Audio
    5.0 Surround output to Front, Center and Rear speakers
surround51:CARD=DA300USB,DEV=0
    DA-300USB, USB Audio
    5.1 Surround output to Front, Center, Rear and Subwoofer speakers
surround71:CARD=DA300USB,DEV=0
    DA-300USB, USB Audio
    7.1 Surround output to Front, Center, Side, Rear and Woofer speakers
iec958:CARD=DA300USB,DEV=0
    DA-300USB, USB Audio
    IEC958 (S/PDIF) Digital Audio Output

LibreELEC:~/.kodi/addons/service.spotify-connect-web # ./spotify-connect-web -n Salon -o iec958:CARD=DA300U
SB,DEV=0
Loading Spotify library...
min_volume_range: 0.0
Using libspotify_embedded version: release-esdk-1.20.0-v1.20.0-g594175d4
SpInit: 0
playback_volume: 60947
corected_playback_volume: 92
public key: xxxxxxxxxxxxx
device id: 0970b14f-86f5-4f55-983e-3590f68d4ffa
remote name: Salon
account req: PREMIUM
device type: AUDIODONGLE
kSpConnectionNotifyLoggedIn
iXFWVkc42sVam1QsiOwJcpjBpgSjtoI4QvujQJQJng2kRmja9fTCS5d0ZTpje/FS
kSpPlaybackNotifyPause
kSpPlaybackNotifyShuffleEnabled
playback_seek: 56863
kSpPlaybackNotifyTrackChanged
kSpPlaybackNotifyBecameActive
playback_seek: 56863
kSpPlaybackEventAudioFlush
kSpPlaybackNotifyPlay
AlsaSink: device acquired
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
Ups! Some badness happened:  Data size must be a multiple of framesize
kSpPlaybackNotifyPause
AlsaSink: device released
^CkSpPlaybackEventAudioFlush
kSpConnectionNotifyLoggedOut

There's no sound.
My DAC works very well with mpd
Thank you for your help.

I had similar issues when my PCM was set to rate 48000 and config inside of console_callbacks.py to 44100. Take a look at this settings.

Thank you for your help.
I haven't set any configuration on alsa.
With MPD my souncard works with a rate of 44100.
And I don't know how to see or set the rate of my PCM, on libreelec.

I had this issue with an asynchronous dac that would only work with S32LE.
Try nano /storage/.config/asound.conf and insert the following contents.
pcm.plug {
        type plug
        slave.pcm "hw:1,0"
}

Then reboot and try ./spotify-connect-web -n Salon -o plug

Thank you very much : It works.
Regards

It took me at least an hour to figure out, but you need to edit both /etc/asound.conf and spotify-connect-web-chroot/etc/asound.conf with the content provided by @tommph :)


Git dependency is missing

In the glscopeclient-manual.pdf in paragraph 3.4.1 for Debian/Ubuntu the "git" package is missing.

Moving to scopehal-docs.


Update Lume + Tailwind changes

Hi, this PR contains two changes:

Upgrade Lume to the latest version (you were using a develop version from jsdelivr)
Changed Tailwind implementation: You had a custom plugin to use postcss with tailwind + autoprefixer plugins. Lume has already a postcss plugin with autoprefixer installed by default, so you don't need to create a custom plugin to use tailwind, just use the Lume's postcss plugin configured to use tailwind as an extra plugin.

Looks like I don't have permissions to resolve the merge conflict. Once that's done it can land 👍


Ncs 2.4 rc2 rebase

The following west manifest projects have been modified in this Pull Request:

Name
Old Revision
New Revision
Diff

mcuboot
https://github.com/zephyrproject-rtos/mcuboot/commit/6902abba270c0fbcbe8ee3bb56fe39bc9acc2774
https://github.com/zephyrproject-rtos/mcuboot/commit/1558e7ab0aadb4eac11f03befb5ccd3fa3f0aafe
<EMAIL_ADDRESS>

openthread
https://github.com/zephyrproject-rtos/openthread/commit/7bdcf8a5d49838ce6e3e227e2780e4f12c461330
https://github.com/zephyrproject-rtos/openthread/commit/d9abe3071c0131a4adb5d7e7451319b735e6d855 (main)
<EMAIL_ADDRESS>

Note: This message is automatically posted and updated by the Manifest GitHub Action.

The diff is not empty between rebase branch and main:
https://github.com/de-nordic/sdk-zephyr/compare/ncs-2.4-rc2-rebase..nrfconnect:main
Are the most recent commits missing?

The following commits should be squashed:

aaec66c58665fae24aec362e94f74edffd809447 with 12f094599b6d510a735d0538ad33b0bc8e707f9f
adf4154a39d24c3bfb5c3f5e2fecbfb372ef261a with 1ff4bec9b5f685b52448a798c93b2be1b0b545d9

And it looks like 44beb682ac50d85273886f8e248f65c95568b7e6 is missing a "cherry picked" entry referencing acb96b140c95e933546acc41cc6df22b609342db.

The following commits should be squashed:
* [aaec66c](https://github.com/nrfconnect/sdk-zephyr/commit/aaec66c58665fae24aec362e94f74edffd809447) with [12f0945](https://github.com/nrfconnect/sdk-zephyr/commit/12f094599b6d510a735d0538ad33b0bc8e707f9f)

* [adf4154](https://github.com/nrfconnect/sdk-zephyr/commit/adf4154a39d24c3bfb5c3f5e2fecbfb372ef261a) with [1ff4bec](https://github.com/nrfconnect/sdk-zephyr/commit/1ff4bec9b5f685b52448a798c93b2be1b0b545d9)

And it looks like 44beb68 is missing a "cherry picked" entry referencing acb96b1.

@anangl I have fixed mentioned issues.


Build workflow fails on push to ghcr.io with: cannot reuse body, request must be retried

🐛 Bug Report
The build workflow is failing with cannot reuse body, request must be retried when pushing to the GitHub Container Registry.
Impact: All builds are failing.
Possible workarounds: Disable pushes to ghcr.io
Possibly related issues:

#358

Related GitHub Community threads:

https://github.community/t/buildx-failed-with-error-cannot-reuse-body-request-must-be-retried/253178

To Reproduce
Steps to reproduce the behavior:

Run the build GitHub action in this repository

Expected behavior
The container image is successfully pushed to ghcr.io
Any helpful log output
https://github.com/felddy/foundryvtt-docker/runs/6680439470?check_suite_focus=true#step:11:916
2022-06-01T00:00:43.6973956Z #103 [auth] ***/foundryvtt:pull,push token for ghcr.io
2022-06-01T00:00:43.6974234Z #103 DONE 0.0s
2022-06-01T00:00:43.8480253Z
2022-06-01T00:00:43.8480656Z #101 exporting to image
2022-06-01T00:00:50.0749426Z #101 pushing layers 6.6s done
2022-06-01T00:00:50.0750696Z #101 ERROR: failed commit on ref "layer-sha256:b9444a845acca65e2fe3b2b0d0485eaaf7197022560b405c0f744471a0824d0d": cannot reuse body, request must be retried
2022-06-01T00:00:50.0751224Z ------
2022-06-01T00:00:50.0751483Z  > exporting to image:
2022-06-01T00:00:50.0751782Z ------
2022-06-01T00:00:50.0752264Z WARNING: local cache import at ~/.cache/buildx not found due to err: could not read ~/.cache/buildx/index.json: open ~/.cache/buildx/index.json: no such file or directory
2022-06-01T00:00:50.0772467Z error: failed to solve: failed commit on ref "layer-sha256:b9444a845acca65e2fe3b2b0d0485eaaf7197022560b405c0f744471a0824d0d": cannot reuse body, request must be retried
2022-06-01T00:00:50.1132728Z ##[error]buildx failed with: error: failed to solve: failed commit on ref "layer-sha256:b9444a845acca65e2fe3b2b0d0485eaaf7197022560b405c0f744471a0824d0d": cannot reuse body, request must be retried

It seems the container registry was having a long streak of issues that have been resolved.
See:

https://github.com/felddy/foundryvtt-docker/issues/358


COMPAT: blacklist numexpr=2.4.4

close #12489

Maybe you want to add a short note to the dependencies section of the main README.md? We should probably just remove that section and link to the docs so it doesn't fall out of date.

@TomAugspurger hmm, that is a good point. want to do a (can separate) PR for that?

Yeah, I'll do one separate.

thank you sir!

Travis is giving a green light.  Don't know if you intend to add anything else to this PR though.


#42 add trivial makefile for make test and others

Addresses #42 (not saying closes, because go test still isn't possible)

Incorporated into #49 . Closing for now. Please reopen if needed


#![feature] may not be used on the stable release channel

ArrayDeque used to compile but suddenly I'm stuck with this error. Does anyone know where it came from ?
error[E0554]: `#![feature]` may not be used on the stable release channel
  --> /home/sisyphe/.cargo/registry/src/github.com-1ecc6299db9ec823/arraydeque-0.4.5/src/lib.rs:57:32
   |
57 | #![cfg_attr(has_union_feature, feature(untagged_unions))]
   |

Closed by https://github.com/andylokandy/arraydeque/pull/22


Cant access to some sites

Describe the bug
Doesnt happened until the today´s 126 update
To Reproduce
Steps to reproduce the behavior:

Go to For example facebook

See error brower not compatible

Expected behavior
opens normally
Desktop (please complete the following information):

OS: [e.g. Windows 7 SP1 x64]
Bitness of browser 32

Another example will be Geforce now that doesnt work ever in this browser but until this update the page is displayed normally and it says (with the gfn UI) that the browser isnt compatible, now with this update it denies even to open gfn

Maybe a userAgent mistake?

I have the same issue with the 64-bit version on Windows 11 and Windows 7. It also doesn't work on Reddit.

I have the same issue with the 64-bit version on Windows 11 and Windows 7. It also doesn't work on Reddit.

Yes, i was about to comment that

To illustrate @win32ss

@win32ss you should release quickly a supermium release with only this fix because is too critical to let it go

I've checked, and the reason is that the useragent is entirely blank

OOH it makes sense

Even i am getting old google layout and google images layout

old layouts everywhere

 I've checked, and the reason is that the useragent is entirely blank

Yes, i can confirm that

A temporary fix is to install an useragent switcher and set another useragent.

Though even downloading extensions won't work because of the missing useragent, so you'll need to paste the link to the useragent switcher (recommended: https://chromewebstore.google.com/detail/user-agent-switcher-and-m/bhchdcejhohfmigjafbampogmaanbfkg) into a site like https://crxextractor.com/, then drag the downloaded .crx onto the extensions page.

Redownload the installer again.

Yes, now seems to be fixed.


Mapnik vector thumbnails

Improves vector thumbnail generation using the mapnik library:

Basic mapnik styles can be configured in config\mapnik.yml.
Relies on several outside libraries that were forked, enhanced and had their builds fixed.
A new github org was created to house these libraries: https://github.com/geoconcerns.

Closes  #120.
From:

To:

@drh-stanford @jrgriffiniii @yulgit1 I updated the README with the requirements for mapnik. Give this branch a whirl.


Issue60

should work in browser and rstudio

@HannesBuseyne
Code is ok but

[ ] You also have to adapt the download count on the top right corner and hide the percentile for non-cran packages
[ ] The graph should and end on last month, no need to go until the end of the year as we obviously have no data yet
[ ] Distinct and double ip is confusing, I would just display the total

!!! Again, this is rebased, so delete your local branch and repull before working on it


Change execution privilege of GetBoxURL API.

Required permissions for execution

App-authenticated token
or
Non app-authenticated token
and RequireSchemaAuthz of Box root ACL is none
and Token that can read box root

Solved in v1.5.7.


Add top-level docs redirects to make 'latest' optional

@pyronicide expressed misgivings about this not doing what i think it does...

This redirect is 1 - 1 (see S3 Redirect). As this is setup right now, there are a couple redirects that'll get lost:
/docs/support/index.html /docs/latest/support/
/docs/administration/opt-out /docs/latest/administration/opt-out/
/docs/usage/cli/install/index.html /docs/latest/usage/cli/install/

Can we get those added back in?

Hoping to replace this option with updating the prefixes: https://dcosjira.atlassian.net/browse/DCOS-50


Set Kubelet parameters via a config file

Hello,
on this site https://kubernetes.io/docs/tasks/administer-cluster/kubelet-config-file/ I miss the information, that if using kubeadm you have to configure the kubelet via the config through kubeadm.
And a link to https://kubernetes.io/docs/tasks/administer-cluster/kubelet-config-file/ is also missing.
Thanks!

Is this a duplicate of #27188 ?

@sftim  I've just closed the other one.

/triage accepted

/retitle Add advice for kubeadm users to “Set Kubelet parameters via a config file”

/priority backlog
/language en
/kind feature


Party system / Dynamically assign worlds to gamemodes based on use

Could also allow reducing dimension count dramatically and is needed for scaling to new gamemodes and team options

Superceded by #190