Project creation fails with self-dependency error

Steps to Reproduce
I followed the installation instructions to install flutter on macOs. flutter doctor shows everything to be ok.
Next I tried to create a new project. This failed:
$ flutter -v create flutter_test
[  +12 ms] [/Users/wichert/flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +27 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/alpha
[        ] [/Users/wichert/flutter/] git ls-remote --get-url origin
[   +5 ms] Exit code 0 from: git ls-remote --get-url origin
[        ]<EMAIL_ADDRESS>[        ] [/Users/wichert/flutter/] git log -n 1 --pretty=format:%H
[   +7 ms] Exit code 0 from: git log -n 1 --pretty=format:%H
[        ] 701d534ba93eaa3452f5e6b8b73ad8bd5a48847b
[        ] [/Users/wichert/flutter/] git log -n 1 --pretty=format:%ar
[   +8 ms] Exit code 0 from: git log -n 1 --pretty=format:%ar
[        ] 4 weeks ago
[ +158 ms] Creating project flutter_test...
[  +14 ms]   flutter_test/ios/Runner.xcworkspace/contents.xcworkspacedata (existing - skipped)
[        ]   flutter_test/ios/Runner/Info.plist (existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]   flutter_test/ios/Runner/Assets.xcassets/LaunchImage.imageset/README.md (existing - skipped)
[        ]   flutter_test/ios/Runner/Assets.xcassets/LaunchImage.imageset/Contents.json (existing - skipped)
[        ]   flutter_test/ios/Runner/Assets.xcassets/LaunchImage.imageset/LaunchImage.png (existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]   flutter_test/ios/Runner/Assets.xcassets/AppIcon.appiconset/Contents.json (existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]<EMAIL_ADDRESS>(existing - skipped)
[        ]   flutter_test/ios/Runner/Base.lproj/LaunchScreen.storyboard (existing - skipped)
[        ]   flutter_test/ios/Runner/Base.lproj/Main.storyboard (existing - skipped)
[        ]   flutter_test/ios/Runner.xcodeproj/project.xcworkspace/contents.xcworkspacedata (existing - skipped)
[        ]   flutter_test/ios/Runner.xcodeproj/xcshareddata/xcschemes/Runner.xcscheme (existing - skipped)
[        ]   flutter_test/ios/Flutter/Debug.xcconfig (existing - skipped)
[        ]   flutter_test/ios/Flutter/Release.xcconfig (existing - skipped)
[   +1 ms]   flutter_test/ios/Flutter/AppFrameworkInfo.plist (existing - skipped)
[        ]   flutter_test/ios/.gitignore (existing - skipped)
[        ]   flutter_test/test/widget_test.dart (existing - skipped)
[        ]   flutter_test/flutter_test.iml (existing - skipped)
[        ]   flutter_test/.gitignore (existing - skipped)
[        ]   flutter_test/ios/Runner/AppDelegate.h (existing - skipped)
[        ]   flutter_test/ios/Runner/main.m (existing - skipped)
[        ]   flutter_test/ios/Runner/AppDelegate.m (existing - skipped)
[        ]   flutter_test/ios/Runner.xcodeproj/project.pbxproj (existing - skipped)
[        ]   flutter_test/android/app/src/main/res/mipmap-mdpi/ic_launcher.png (existing - skipped)
[        ]   flutter_test/android/app/src/main/res/mipmap-hdpi/ic_launcher.png (existing - skipped)
[        ]   flutter_test/android/app/src/main/res/drawable/launch_background.xml (existing - skipped)
[        ]   flutter_test/android/app/src/main/res/mipmap-xxxhdpi/ic_launcher.png (existing - skipped)
[        ]   flutter_test/android/app/src/main/res/mipmap-xxhdpi/ic_launcher.png (existing - skipped)
[        ]   flutter_test/android/app/src/main/res/values/styles.xml (existing - skipped)
[        ]   flutter_test/android/app/src/main/res/mipmap-xhdpi/ic_launcher.png (existing - skipped)
[        ]   flutter_test/android/app/src/main/AndroidManifest.xml (existing - skipped)
[        ]   flutter_test/android/gradle/wrapper/gradle-wrapper.properties (existing - skipped)
[        ]   flutter_test/android/.gitignore (existing - skipped)
[        ]   flutter_test/android/gradle.properties (existing - skipped)
[        ]   flutter_test/android/settings.gradle (existing - skipped)
[        ]   flutter_test/pubspec.yaml (existing - skipped)
[        ]   flutter_test/README.md (existing - skipped)
[        ]   flutter_test/lib/main.dart (existing - skipped)
[        ]   flutter_test/android/app/build.gradle (existing - skipped)
[        ]   flutter_test/android/app/src/main/java/com/yourcompany/fluttertest/MainActivity.java (existing - skipped)
[        ]   flutter_test/android/build.gradle (existing - skipped)
[        ]   flutter_test/flutter_test_android.iml (existing - skipped)
[        ]   flutter_test/android.iml (existing - skipped)
[        ]   flutter_test/.idea/runConfigurations/main_dart.xml (existing - skipped)
[        ]   flutter_test/.idea/libraries/Flutter_for_Android.xml (existing - skipped)
[        ]   flutter_test/.idea/libraries/Dart_SDK.xml (existing - skipped)
[        ]   flutter_test/.idea/modules.xml (existing - skipped)
[        ]   flutter_test/.idea/workspace.xml (existing - skipped)
[  +12 ms] Wrote 2 files.
[   +6 ms] Running "flutter packages get" in flutter_test...
[   +2 ms] [/Users/wichert/Development/flutter_test/] /Users/wichert/flutter/bin/cache/dart-sdk/bin/pub --verbosity=warning get --no-precompile
[ +147 ms] Error on line 9, column 3 of pubspec.yaml: A package may not list itself as a dependency.
[        ]   flutter_test:
[        ]   ^^^^^^^^^^^^
[  +28 ms] "flutter create" took 348ms.
pub get failed (65)

#0      throwToolExit (package:flutter_tools/src/base/common.dart:29)
#1      pub (package:flutter_tools/src/dart/pub.dart:110)
<asynchronous suspension>
#2      pubGet (package:flutter_tools/src/dart/pub.dart:60)
<asynchronous suspension>
#3      CreateCommand.runCommand (package:flutter_tools/src/commands/create.dart:218)
<asynchronous suspension>
#4      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:239)
<asynchronous suspension>
#5      FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:181)
<asynchronous suspension>
#6      CommandRunner.runCommand (package:args/command_runner.dart:194)
<asynchronous suspension>
#7      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:276)
<asynchronous suspension>
#8      CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:109)
#9      new Future.sync (dart:async/future.dart:222)
#10     CommandRunner.run (package:args/command_runner.dart:109)
#11     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:161)
#12     run.<anonymous closure> (package:flutter_tools/executable.dart:143)
<asynchronous suspension>
#13     AppContext._run (package:flutter_tools/src/base/context.dart:76)
<asynchronous suspension>
#14     AppContext.runInZone.<anonymous closure> (package:flutter_tools/src/base/context.dart:66)
#15     _rootRun (dart:async/zone.dart:1120)
#16     _CustomZone.run (dart:async/zone.dart:1001)
#17     runZoned (dart:async/zone.dart:1467)
#18     AppContext.runInZone (package:flutter_tools/src/base/context.dart:65)
#19     run (package:flutter_tools/executable.dart:114)
<asynchronous suspension>
#20     main (package:flutter_tools/executable.dart:67)
<asynchronous suspension>
#21     main (file:///Users/wichert/Development/flutter/packages/flutter_tools/bin/flutter_tools.dart:16)
#22     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:263)
#23     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:151)

[   +4 ms] exiting with code 65

Flutter Doctor
[✓] Flutter (on Mac OS X 10.13 17A405, locale en-NL, channel alpha)
    • Flutter at /Users/wichert/flutter
    • Framework revision 701d534ba9 (4 weeks ago), 2017-09-12 14:01:51 -0700
    • Engine revision 31d03de019
    • Tools Dart version 1.25.0-dev.11.0

[✓] Android toolchain - develop for Android devices (Android SDK 25.0.2)
    • Android SDK at /Users/wichert/Library/Android/sdk
    • Platform android-25, build-tools 25.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] iOS toolchain - develop for iOS devices (Xcode 9.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.0, Build version 9A235
    • ios-deploy 1.9.2
    • CocoaPods version 1.3.1

[✓] Android Studio (version 2.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[-] IntelliJ IDEA Community Edition (version 2017.2.5)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    • Dart plugin version 172.4155.35
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[✓] Connected devices
    • None

flutter_test is the name of Flutter's test framework package:
https://docs.flutter.io/flutter/flutter_test/flutter_test-library.html
There will be a name collision if your application package is also called flutter_test

That makes sense. As someone totally new to flutter this was confusing. Perhaps it would be possible to have flutter create check a blacklist for names that are known to fail so others won't run into this trap?

Did the same.
Silly me.

And me!

same! it's funny how common it is to name your first flutter project "flutter_test" haha

+1 to the company 😅

+1, how embarrassing...

+1, They are going to have to change the name of the test framework package at this rate!

+1, I fell for it too

+1 , I felt for it too

+1 another victim


Navbar looks strange when multiple navigation items are defined

Looks okay now?

Oh, I like it!
Maybe just decrease spacing between lines in heading. Looks weird on certain resolution. See below:

Gotcha. Will look into it. Thanks :+1:

I now see that I did not respond to your question. I've changed default.html on my fork in order to have more nav list items and longer site name.
You can see that I've completly changed position of the site name and description. I don't know if those changes would make sense on your repo.

I'll look into those changes and will merge if appropriate. Thanks!


can i use more than one Remodal with Ok Cancel buttons?

Hi,
i need to use more than 3 remodal to show popup forms. here i do save the data  from different forms.

No problems, it is possible.


Overlapped labels in piechart

When one of the regions in piechart is too small to encompass its label, it is overlapped by other regions.

[x] I have searched for existing issues
[x] I have provided detailed instructions that can reproduce the issue (including code and data necessary)

I face the same problem. The number 12 isn't displaying correctly


Error in training

I roll back to keras 1.0.7 but it doesn't work for this problem...

Epoch 1/150
Traceback (most recent call last):
File "kitti_train.py", line 75, in
validation_data=val_generator, nb_val_samples=N_seq_val)
File "/usr/local/lib/python2.7/dist-packages/Keras-1.0.7-py2.7.egg/keras/engine/training.py", line 1444, in fit_generator
class_weight=class_weight)
File "/usr/local/lib/python2.7/dist-packages/Keras-1.0.7-py2.7.egg/keras/engine/training.py", line 1216, in train_on_batch
check_batch_dim=True)
File "/usr/local/lib/python2.7/dist-packages/Keras-1.0.7-py2.7.egg/keras/engine/training.py", line 962, in _standardize_user_data
exception_prefix='model input')
File "/usr/local/lib/python2.7/dist-packages/Keras-1.0.7-py2.7.egg/keras/engine/training.py", line 108, in standardize_input_data
str(array.shape))
Exception: Error when checking model input: expected input_1 to have shape (None, 10, 3, 128, 160) but got array with shape (4, 10, 128, 160, 3)

Trying to fix it following https://github.com/coxlab/prednet/pull/5/files. Thx!

Sorry about the delay in this, but I fixed it so that both dim_orderings work now.


Add luaJIT to Solar2D

It would make lua so much faster.

There are a few challenges towards integrating LuaJIT.

Solar2D uses custom allocator. LuaJit doesn't support them. This is could have a workaround since Solar2D first actually use custom allocator as allocator, but mostly to store runtime pointer.
plugins won't be compatible since LuaJIT uses its own bytecode. This has workarounds too though.
we customized Lua quite a bit, those patches should be ported.
it is hard to say how noticeable performance boost will be. Solar2D is primarily mobile game engine. So JIT won't work on iOS/tvOS because of platform restrictions. And last time I checked arm64 jit support wasnt much good. But that was a long time ago.


Mixing Java and Clojure files

Hello,
Is mixing Java and Clojure files supported? I tried to call a method in AOT-compiled Clojure class from Java, but "gradlew compileJava" failed because javac cannot see the classes located in build/classes/main/clojure/.
If it's not currently supported, are there any plan to support it?

We do intend to support polyglot projects (one of our key goals in fact). This 0.3.0 though is the first that puts us over the hump of feeling usable for Clojure-only (now that we have REPL support).
The big issue is that some JVM language plugins can get bi-directional Java <-> Other support. They usually have some kind of joint compilation, where they generate stub classes of their language for Java to reference at compile time, then they compile their language against the resulting compiled Java classes.
I don't think we can pull that off for Clojure (at least in the near term), so there's probably a question of which direction makes the most sense to support (or should we just document how you would do this).
Just from my gut I would have expected Clojure depending on Java to be more common, but obviously you're an example where the opposite is true.
Short term, it takes some fiddling with sourceSets and task dependencies in your build script. I don't have that off the top of my head, but it should be possible.

The Groovy plugin will be default configure it so compileGroovy depends on compileJava and the output of compileJava is on the classpath of compileGroovy. Again this doesn't help with your use case (though you could do the opposite).
If we aligned with that approach you could get easy Clojure depends on Java. But Groovy has joint compilation to get the Java depends on Groovy. Clojure doesn't, so not sure how we make that easy.

The big issue is that some JVM language plugins can get bi-directional Java <-> Other support ...
I don't think we can pull that off for Clojure (at least in the near term),

If this bi-directionality would be in fact uni-directional (the one or the other direction) e.g. at the package level, would it be easier to achieve?
e.g. the user would have to define something like:
packageDeps {
   package_1 dependesOn ns_A
   ns_B dependsOn package_2
   package 3 depndsOn ns_C
}

so there's probably a question of which direction makes the most sense to support (or should we just document how you would do this)

Documenting how to achieve each direction would be more useful in the beginning.

I would have expected Clojure depending on Java to be more common ...

Sneaking Clojure into existing (Gradle based) projects would open many new uses :) .

May want to consider how Leiningen does this. Have to read it more closely to figure out the default. For their interleaving piece, that's where we'd have to document how to make more source sets and have them depend on each other in the preferred order.

The big issue is that some JVM language plugins can get bi-directional Java <-> Other support ...
I don't think we can pull that off for Clojure (at least in the near term),

If this bi-directionality would be in fact uni-directional (the one or the other direction) e.g. at the package level, would it be easier to achieve?
e.g. the user would have to define something like:
packageDeps {
   package_1 dependesOn ns_A
   ns_B dependsOn package_2
   package 3 depndsOn ns_C
}

I think doing it at the sourceSet level most easily integrate with Gradle's general features. Package level would probably have to replicate the functionality already provided by source sets, so it could be troublesome.

so there's probably a question of which direction makes the most sense to support (or should we just document how you would do this)

Documenting how to achieve each direction would be more useful in the beginning.

Agreed.

The fix is part of 0.3.0. Please reopen if this didn't fix the issue.

Also see the docs directory for documentation on your use case. (The other direction is now supported by default)

Thank you!


Segmentation fault

After I finished compiling the hw section and burning the bit flow through the jtag interface into vcu118, I executed
insmod coyote_drv.ko
A segment error has occurred, as shown below

And the printed kernel information is as follows:

The read configuration information is all 1. I do not know the cause

Did your rescan the PCIe bus?

Yes, restart the host after burning the bitstream. After restart, check the device before loading the driver and input instructions
lspci -vv -d 10EE:
However, the PCIe device is not found.
And then execute
echo 1 > /sys/bus/pci/rescan
Also run lspci -vv -d 10EE:
The PCIe device is still not found.

Yes, restart the host after burning the bitstream. After restart, check the device before loading the driver and input instructions
lspci -vv -d 10EE:

However, insmod coyote_drv.ko will still appear Segmentation fault.

All the information is as follows:

What is the benchmark you are running and what is the config?

The instructions I entered were:
cmake -DFDEV_NAME=vcu118 -DEXAMPLE=gbm_dtrees ..
make shell compile
Then load the bitstream to vcu118, restart the host, and load the driver to produce this error

I had a similar issue. I was trying to run the insmod in a virtual machine. When I tried running this on the actual machine, it worked. See if that helps


poetry build does not include rust .so in wheel

Hello, thank you for this tutorial, it was extremely helpful during my project setup. I see one issue with it:
I need to be able to build a wheel for my project and I would like to do it through poetry with maturin as build backend. However, after the wheel produced by poetry build command does not include the compiled .so file.
On the contrary, when I use maturin build the produced wheel contains the .so file. So technically I could just use maturin to build my wheel, right?
Well there is a problem - I have to use poetry to manage my dependencies (because of the setup of other projects in out organisation) and maturin cannot read dependencies in poetry format ([tool.poetry.dependencies]) it needs the [project.dependencies] as PEP 508 specifies. Vice versa, poetry cannot read [project.dependencies]. So to have both tools working I need to duplicate the dependency list in those two formats, which works but is a pain to maintain.
The next interesting thing is, that when I install hello-maturin as the dependency in another project using 'poetry add /path/to/hello-maturin' it installs the complete project - with poetry managed dependencies AND the .so file is present in site-packages/hello-maturin. This shows me that poetry can use maturin properly as build backend, but for some reason it does not do it during poetry build.
So my question is - is there a way to build wheel using poetry build with maturin as backend and actually have the compiled .so file included in this wheel?

Hi @BartoszJanuszNA,
It's been a while since I finished a solution that I think does what you are looking for.
I used Hatch over Poetry, but Poetry was working fine as far as I can remember.
Please take a look at:
https://github.com/bluerobotics/navigator-lib/pull/32/files
https://github.com/bluerobotics/navigator-lib
https://github.com/bluerobotics/navigator-lib/actions/runs/8428044353
Hope it helps your project!

The short answer is no, you cannot use poetry build to call maturin build.
I only used poetry for this example to make it quick and easy to create a virtual environment and install the development dependencies, mainly maturin.
I switched to hatch for my next project deeper because of the duplication issues.
In fact, I use hatch for all of my projects now even though poetry has some niceties that I miss.
I will probably replace poetry in this repo with hatch eventually.
My advice is to switch to hatch.

So it seems that until Poetry learns to read dependencies from [project]  I am stuck with duplicates :disappointed: Thank you for pointing me to hatch, I see it can do the above.
@RaulTrombin I can see your project's python part does not have any dependencies (I assume [tool.hatch.envs.dev] is only for development environment and not used during wheel build?) . In my project I have those and that is where the problem arises. Thank you nonetheless :-)
@kfields That's exactly what I was suspecting, thanks for resolving my doubts. Well, to be absolutely fair I saw recently a solution using custom build.py script, but it looks like forcing poetry to do smth it doesn't want to. Simpler to just build wheels with maturin.
Thank you guys!


Minor documentation cleanup

We were referencing rJava in readme file.
We use Spark 1.3.0 by default
Users need to install testthat package before running tests.

Thanks @falaki for catching the glitches. For default Spark version that we pull in, I think we're still using 1.1.0 (see here)?

Thanks @falaki - LGTM. Merging this.


ListView - Content not rendered

Describe the bug
Seems like the ListViewItem style is causing some issues with the rendering of the listview item content. The text is not visible at all
To Reproduce
In my test project I boiled the issue down to the ListViewItem style
https://github.com/Swiftpaws/Adonis-ListViewBug

At the moment, the list view style expects the list view to have a grid view assigned. So adding the following to your demo makes it work:
<ListView.View>
    <GridView>
        <GridViewColumn Header="Content"
                        DisplayMemberBinding="{Binding Content}" />
    </GridView>
</ListView.View>


Make property a string like Core Animation in order to unify how properties work

We currently expose a POPProperty type that clearly shows which properties we support for pop's SpringTo implementation. This API is inflexible and makes it difficult to commit SpringTo values to the target directly using traditional setValueForKeyPath APIs.
Goal of this task is to explore a string-based property API that is a traditional key path. We'll need to map these key paths to internal POPProperties.

🎊 My diff is ready for review at http://codereview.cc/D1960


MoyaProvider.RequestClosure is there a way to get the enum that is being called?

Hello
I'm attempting to add a middleware step to my requests that attempts to refresh my tokens but I only want it to be done when the request requires a token. I don't see anywhere that I can get the request being made and check if it's a request that requires a token.
e.g.
// Closure run on each request in order to check the validity of the token
// The token will be refreshed whenever needed.
let requestClosure: MoyaProvider.RequestClosure = { (endpoint: Endpoint, done: @escaping MoyaProvider.RequestResultClosure) in

    // Only proceed if the task is AccessTokenAuthorizable and it's authorization type is bearer.

    // TODO: How do I get the enum being called to check if it's type is AccessTokenAuthorizable and get the value of that key for this request?
    // Something like below except .task is not the proper value as I can't call these methods on it.
    let authorizationType: AuthorizationType = (endpoint.task as? AccessTokenAuthorizable)?.authorizationType ?? .none
    guard authorizationType  == .bearer else {

        done(.success(try! endpoint.urlRequest()))
        return
    }

    // There is no state to be updated. Proceed with the request
    guard authManager.state != nil else {
        done(.success(try! endpoint.urlRequest()))
        return
    }

    /// State is available and the request uses the auth token. Attempt to refresh before performing request
    authManager.performAction(freshTokens: { (accessToken, idToken, error) in
        guard error == nil else {
            log.error("Failed to refresh token with Error: \(error!.localizedDescription)")
            done(.failure(MoyaError.underlying(error!, nil)))
            return
        }

        done(.success(try! endpoint.urlRequest()))
    })
}

Hey @DMCApps. I'm afraid it's not possible currently. You might want to see https://github.com/Moya/Moya/issues/1671, which is I believe a similar use-case that you specified.

@sunshinejr aw that's unfortunate. Is there any reason against passing in the target to that closure? e.g.
This:
public typealias RequestClosure = (Endpoint, @escaping RequestResultClosure) -> Void

becomes:
public typealias RequestClosure = (Target, Endpoint, @escaping RequestResultClosure) -> Void

Wondering if there is a design decision against that or if the use case is not worth the development? From a quick look at the code base it looks like requestNormal calls this closure in MoyaProvider+Internal which looks to have the Target come in it's method signature.
func requestNormal(_ target: Target, callbackQueue: DispatchQueue?, progress: Moya.ProgressBlock?, completion: @escaping Moya.Completion) -> Cancellable {

    ...

    requestClosure(endpoint, performNetworking)

    return cancellableToken
}

@DMCApps not sure why we didn’t add the parameter before - if you want to create a PR with the change I would gladly review it 👍🏻

@sunshinejr Alright I'll work on that when I get a chance.

@DMCApps ah no let’s keep it open until we get this one resolved.
And for the contribution guidelines, take a look at this document - should help! If not, we always have our Danger bot on each PR and you can ask here/on the PR and we will help you as soon as possible 👍🏻

@DMCApps not sure why we didn’t add the parameter before - if you want to create a PR with the change I would gladly review it 👍🏻

I think it's because the target is actually replaced by the Endpoint. Having both of them looks a bit overkill as the second derives from the first. There might be a way to be even better by moving the URLRequest construction from Endpoint to Target, and removing the Endpoint type? From what I see this seems to be the only purpose of the Endpoint type.

@amaurydavid yeah I agree it kinda duplicates some properties that Endpoint got from the Target. On the other hand removing Endpoint is a really big change in both the core of Moya and usage and I don't feel comfortable removing it, would need to go through a proposal of some sort. Ideally we would add the information needed for the requestClosure (the Target), maybe as a property of the Endpoint? Endpoint.underlyingTarget? What do you think?

@sunshinejr @amaurydavid That solution would work for the use case I presented. It would also give endpoint access to more than the Moya.Method and Task since those can be replaced with the Endpoint.underlyingTarget.method and Endpoint.underlyingTarget.task and give access to any extensions used on TargetType as well.


patching IR (tested on HW)

Patch from @jamesadevine

TC3 is used by the DAC as well. Looking into using a different channel.

There are unused channels on the system low level timer. But it would require some api change to use that.

https://github.com/Microsoft/pxt-common-packages/pull/736/files#diff-afd3b1716803cee6eade9e78834d5de5R183
Will never be true as we have a 16 bit timer with 1 us precision, so we will wait variable times between sends.

Will use system_timer_* instead.

@mmoskal I'm using TCC2 for IR timings instead of TC3. I don't think this affects any user facing GPIO.
When it comes to integrating jacdac in the future, we won't be able to run TCC timer at the highest priority (if we run IR at all)


How to use icon font within TextSymbol ?

Icon fonts are getting more and more common within projects.
I would like to integrate Map Icons within TextSymbol.
This would provide a way to build icons easily via CompositeSymbol.
Tried lines like
TextSymbol textSymbol = new TextSymbol(32, "\ue85d", Color.BLUE, TextSymbol.HorizontalAlignment.CENTER, TextSymbol.VerticalAlignment.MIDDLE);         textSymbol.setFontFamily("map-icons");
The struggle is that Android relies on Typeface to integrate theses fonts, and I didn't find a way to set TextSymbol's Typeface...
Can you provide a sample with the workaround ?
By the way, I managed to do that with iOS.

@ddcrobert Direct support for icons as symbols is coming in a future release.  You mention that you are able to do this with iOS, are you using the same developer pattern that you reference?

Yes, the very same way, besides language differences.
let textSymbol = AGSTextSymbol(text: "\u{e85d}", color: UIColor.nokiaDark, size: 32, horizontalAlignment: .center, verticalAlignment: .middle)         textSymbol.fontFamily = "map-icons"

@ddcrobert Ok, I will take a look, thanks!

On Android you are trying to use the font as an asset in the APK?  Currently we do not support a setting the Typeface, a workaround is to programmatically create a TextView and set the location based on tapped screen location:
// respond to single taps on mapview
mapView.onTouchListener = object:DefaultMapViewOnTouchListener(this, mapView) {
    override fun onSingleTapConfirmed(motionEvent: MotionEvent?): Boolean {
        // get the point that was clicked and convert it to a point in map coordinates
        val screenPoint: android.graphics.Point = android.graphics.Point(motionEvent!!.x.toInt(), motionEvent.y.toInt())
        // create a TextView
        val mapIconView = TextView(applicationContext)
        // create Typeface from assets folder, /assets/fonts/
        val mapIconTypeFace = Typeface.createFromAsset(assets, "fonts/map-icons.ttf")
        // set the Typeface to the TextView
        mapIconView.typeface = mapIconTypeFace
        // create a layout param
        mapIconView.layoutParams = ViewGroup.LayoutParams(100, 100)
        // place the TextView using screen x/y location
        mapIconView.x = (screenPoint.x).toFloat()
        mapIconView.y = (screenPoint.y).toFloat()
        // set the text, size, color of font
        mapIconView.text = "\ue85d"
        mapIconView.textSize = 32f
        mapIconView.setTextColor(Color.WHITE)
        // add TextView to MapView
        mapView.addView(mapIconView)

        return super.onSingleTapConfirmed(motionEvent)
    }

}

Let me know if this addresses your issue.  This would also be good information for GeoNet forums, I encourage you to join and share your questions with the community.

Actually, this workaround is limited to static map.
Map / screen conversion is often a good solution, e.g center a graphic on top half. I could rely on it to place TextView accordingly to my Graphics (they're built from data) and give them an transparent symbol.
I also think I would be able to "clip" theses TextViews based on MapView bounds.
My main concern is about zoom, pan and rotation events.
I'm pretty sure I would encounter latency and/or performance issues.
I also thought of using dynamically generated Image (some sort of TextView to Image conversion) and use them as symbol, but that's a bad idea, since there won't be quality improvment.
So, I prefer stick w/ images on Android, after all, that's the classic way most people are using.

@ddcrobert Yes, agreed static map as the mapIconView.text is not bound by the geometry.  You can address this by creating a graphic at original tapped location, get it's geometry, and convert to screen points while listening to MapView scale changes MapView.addMapScaleChangedListener.  This will keep the font over the location on zoom/pan/rotate events, but agreed there is some overhead with this workaround.


⚠ UdeS website (prod) has degraded performance

In 49bd0b9, UdeS website (prod) (https://www.usherbrooke.ca/) experienced degraded performance:

HTTP code: 200
Response time: 3099 ms

Resolved: UdeS website (prod) performance has improved in fc70d5f.


Segfault

It's rare but I found that it sometimes causes segfaults in the GitHub CI.
https://github.com/cocoa-xu/tflite_elixir/actions/runs/4405411900/jobs/7716261476

After a few hours of testing, it seems that TFLiteElixir.Interpreter.tensor/2 might have some issues.

Another segfaulted CI run, https://github.com/cocoa-xu/tflite_elixir/actions/runs/4432865316/jobs/7777341379.
Will going to upload the coredump file and the compiled binary.

It should be solved by now :D


Resolve issue #3014 inconsistent relativeTime locale in zh-tw

fix the locale file and make it work as expected

@ben-lin, your thoughts, please? It seems logical but I don't speak Chinese, unfortunately.

@gaplo917  - Please only modify the files in the /src directory.  The other is auto-generated at build time.  Thanks.

Yes this makes sense.

@gaplo917 @ben-lin, thanks! Staging for the next release.

@gaplo917 - could you please rebase to remove the change to /locale/zh-tw.js.  Thanks.

@mj1856 sure, thanks

:+1:

Merged in 339b0a2aca499948f879347104c8545100363492


Single sale appears as multiple orders from different buyers

Brief Description:
Operating System (OS and version): Buyer: Windows 10  Seller: OSX 10.10.5
OpenBazaar version (shown on About OpenBazaar page in menu): Version 1.0.6
Hardware: Buyer: ASUS VivoBook   Seller: iMac Retina 5K
Reproducible (Always / Almost Always / Sometimes / Rarely / Couldn't Reproduce):
Did not attempt.
Steps to reproduce:
1.
2.
3.
Observed Behavior:
After a single purchase was made from my store, six orders were listed in the Transactions>Sales tab.  Same item, different order number, different buyer ID (see attached screenshot).
Expected Behavior:
Expected one order to be created.

Additional info (links, images, etc go here):

Are you certain this isn't 6 different orders from different people? They all happened at different times, and appear to be from different users.
There is no limit to the number of times an item can be purchased, or the number of people that can purchase it.

It does indeed appear that multiple sales were made by different buyers.  I didn't realize this was possible.

This is an area we're working to make more clear.

This is why we really need the buyer's avatar here to show some differentiation between orders. Otherwise they all look the same and it's difficult to know where you left off

Please have a look at issue/enhancement request #904.  Need a way to specify available quantity of an item.  If only x of an item is available, and x has been bought, no further purchases should be accepted.


Fix broken links

Updated random number to another random number. Doesn't seem like a great way to maintain link integrity...

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

#label:"aq-pr-triaged"


clangd-config flags an error if config file is empty

I have an empty (size 0) .clangd file in the root of my project. (I copied some settings around, that's how I got there.) Now VS Code always shows a "problem": "Config should be a dictionary (clangd-config)".

This is of course not much of a problem, but it would be a nice small improvement to get this fixed and show no error message (the file is ignored anyways).
I'm actually not quite sure where this message is coming from, i.e. the VS Code clangd extension, or something else?
System information
Clangd version: 14.0.0
clangd extension version: v0.1.17
Operating system: Linux

Transferred to the clangd repo as this is a server-side issue. Diagnostics about the config file syntax are produced by clangd and communicated over LSP much like diagnostics in C++ sources.

Weird, I have an empty config and yet it's complaining...

I had the same issue. How to fix it?

I had the same issue. How to fix it?

I deleted 'config.yaml' file and then retarted clangd. It worked fine!

I deleted 'config.yaml' file and then retarted clangd. It worked fine!

How is that a solution? Of course you don't get the error once the file is gone. But if you require such a file to set build settings, you now get plenty of other problems. A valid config file simply should not produce this error.


[Bug]: Could not resolve all files for configuration ':app:androidApis'.

Is there an existing issue for this?

[X] I have searched the existing issues

Is there a StackOverflow question about this issue?

[X] I have searched StackOverflow

What happened?
A bug happened!,
Relevant logcat output
Could not resolve all files for configuration ':app:androidApis'.
Failed to transform android.jar to match attributes {artifactType=android-mockable-jar, org.gradle.libraryelements=jar, org.gradle.usage=java-runtime, returnDefaultValues=false}.
Execution failed for MockableJarTransform: C:\Users\Error\AppData\Local\Android\Sdk\platforms\android-33\android.jar.
Cannot create mockable android.jar
C:\Users\Error\AppData\Local\Android\Sdk\platforms\android-33\android.jar

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@10sp are you still able to reproduce the issue? Otherwise I think we can close this issue.


istioctl 1.17: regression in name resolution

$ ~/istio/istio-1.17.1/bin/istioctl pc ep deploy/istio-ingressgateway.istio-system
Error: failed retrieving: deployments.apps "istio-ingressgateway" not found in the "default" namespace
$ ~/istio/istio-1.15.0/bin/istioctl pc ep deploy/istio-ingressgateway.istio-system
ENDPOINT                                                STATUS      OUTLIER CHECK     CLUSTER
...

Might have broken in 1.16, didn' test

This is broken only in 1.17 since https://github.com/istio/istio/pull/43083 was cherry-picked to 1.17 release branch.


Making new instances of HttpClient is bad practice

https://github.com/microsoft/CopilotHackathon/blob/0fe0c184aa268b7af66d08be48d6e1a73a7efba4/completesolution/dotnet/MinimalAPI/Program.cs#L53
Using var client = new HttpClient() is generally considered a bad practice for several reasons. Having that in the solution, might not look good.
I think this should either be changed to a more proper solution like using DI to get a HttpClient, or using DI to get a HttpClientFactory. How ever, I can also see this is a good learning opportunity to show case when Copilot might make mistakes and it is still really important for the developer to build their skills and make sure the code that Copilot produces is following best practices.
At the very least, I think it should be obvious that this is intended, even though it's not the recommended way of doing things.

indeed it is. Do you mind contributing with a PR?

Sure.
Do you think it's better to make sure the finished code is correct, or keep it as is, but add a comment to notify the user to stay alert to these type of "errors"?
I haven't tried to check what Copilot actually would produce, so maybe it will do this correctly.


feat: use native AggregateError if available

Summary
Implements logic to use a native implementation if available instead of a polyfill.
Related issue
#75
Code changes

[ ] Related documentation has been updated, if needed.
[x] Related tests have been added or updated, if needed.
[ ] Related benchmarks have been added or updated, if needed.

Question
To create my test, I had to create a separate file because I encountered issues using vi.stubGlobal in the same file. I discovered this happens because Vitest uses the same workers per test file. Running tests simultaneously causes conflicts with globalThis sharing. More details can be found in this discussion: https://github.com/vitest-dev/vitest/discussions/1741. The workaround I found was to use two files. It’s not ideal, but it works. Any suggestions for a better solution are welcome.
Does this PR introduce a breaking change?
No
Bundle impact

Calculating...

Looks good now!
Btw, in the future, you should avoid sending PR from your main branch:
https://hynek.me/articles/pull-requests-branch/


Eliminate google-python-cloud-debugger dependency in base_image_requirements

Fixes #23487
Per https://cloud.google.com/debugger/docs/release-notes: "Cloud Debugger is deprecated and will be shutdown May 31, 2023."
This dep is also a blocker of building python container on a mac.
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @tvalentyn

noting that this CL doesn't regenerate the dependencies, but this happens regularly, and done at least before a release.

yeah thanks for reminding.


LLVM error when using f64x8.scatter_select

I tried this code:
#![feature(portable_simd)]
use std::simd::{f64x8, Mask, usizex8};
fn main() {
    let mut vec = vec![0.0; 129];
    call_simd(&mut vec);
}

fn call_simd(target_slice: &mut [f64]) {
    let accumr = f64x8::splat(1.0);
    accumr.scatter_select(target_slice, Mask::splat(true), usizex8::splat(1));
}

When compiling the code with cargo build --release the process crashes with the error message error: failed to parse bitcode for LTO module: Explicit gep type does not match pointee type of pointer operand (Producer: 'LLVM14.0.6-rust-1.64.0-nightly' Reader: 'LLVM 14.0.6-rust-1.64.0-nightly'). The compiler does not crash at a regular cargo build. I cannot reproduce the error in Rust Playground. This example reproduces the result https://godbolt.org/z/ovzYWeT9M
Meta
rustc --version --verbose:
rustc 1.64.0-nightly (7425fb293 2022-06-30)
binary: rustc
commit-hash: 7425fb293f510a6f138e82a963a3bc599a5b9e1c
commit-date: 2022-06-30
host: x86_64-unknown-linux-gnu
release: 1.64.0-nightly
LLVM version: 14.0.6

I do not have stdsimd or any other crate as an explicit dependency. The Rust edition is 2021.

%__10 = getelementptr double, <8 x i64*> %self1, <8 x i64> %addend

It looks like this is the offending line of LLVMIR, inside
define void @"_ZN4core9core_simd6vector17Simd$LT$T$C$_$GT$24scatter_select_unchecked17h75bd963d6a3fad29E"(<8 x double>* %self, [0 x double]* align 8 %slice.0, i64 %slice.1, <8 x i64>* %enable, <8 x i64>* %idxs) unnamed_addr #1 {
  %__0 = alloca <8 x i64*>, align 64
  %_16 = alloca [8 x double*], align 8
  %base_ptr = alloca <8 x i64*>, align 64
  %ptr = bitcast [0 x double]* %slice.0 to double*
  %__1 = getelementptr inbounds [8 x double*], [8 x double*]* %_16, i64 0, i64 0
  %__2 = getelementptr inbounds [8 x double*], [8 x double*]* %_16, i64 0, i64 0
  %__3 = getelementptr inbounds [8 x double*], [8 x double*]* %_16, i64 0, i64 8
  br label %repeat_loop_header

repeat_loop_header:                               ; preds = %repeat_loop_body, %start
  %__4 = phi double** [ %__2, %start ], [ %__6, %repeat_loop_body ]
  %__5 = icmp ne double** %__4, %__3
  br i1 %__5, label %repeat_loop_body, label %repeat_loop_next

repeat_loop_body:                                 ; preds = %repeat_loop_header
  store double* %ptr, double** %__4, align 8
  %__6 = getelementptr inbounds double*, double** %__4, i64 1
  br label %repeat_loop_header

repeat_loop_next:                                 ; preds = %repeat_loop_header
  %__7 = bitcast <8 x i64*>* %base_ptr to [8 x double*]*
  %__8 = bitcast [8 x double*]* %__7 to i8*
  %__9 = bitcast [8 x double*]* %_16 to i8*
  call void @llvm.memcpy.p0i8.p0i8.i64(i8* align 64 %__8, i8* align 8 %__9, i64 64, i1 false)
  %self1 = load <8 x i64*>, <8 x i64*>* %base_ptr, align 64
  %addend = load <8 x i64>, <8 x i64>* %idxs, align 64
  %__10 = getelementptr double, <8 x i64*> %self1, <8 x i64> %addend ; probably around here.
  %__11 = bitcast <8 x i64*>* %__0 to <8 x double*>*
  store <8 x double*> %__10, <8 x double*>* %__11, align 64 ; hmm.
  %ptrs = load <8 x i64*>, <8 x i64*>* %__0, align 64
  br label %bb2

bb2:                                              ; preds = %repeat_loop_next
  %_11 = load <8 x double>, <8 x double>* %self, align 64
  %self2 = load <8 x i64>, <8 x i64>* %enable, align 64
  %__12 = trunc <8 x i64> %self2 to <8 x i1>
  %__13 = bitcast <8 x i64*> %ptrs to <8 x double*>
  call void @llvm.masked.scatter.v8f64.v8p0f64(<8 x double> %_11, <8 x double*> %__13, i32 8, <8 x i1> %__12)
  br label %bb1

bb1:                                              ; preds = %bb2
  ret void
}

cc @RalfJung I don't expect you to fix this but I just thought you might want to be aware (and also just in case, on the off chance, you recognize the problem, but I'll figure out how to massage the LLVMIR one way or another).

when LLVM 15 lands in rustc we'll be able to just change all these i64*s and double* over to ptr and not have to worry about it.

when LLVM 15 lands in rustc we'll be able to just change all these i64*s and double* over to ptr and not have to worry about it, at least.

iirc we'll still have to worry about it because rustc supports a few old versions of LLVM too


No matching implimentation for np.empty in guvectorized function with NUMBA_DISABLE_JIT=1 and related caching issue

[x] I have tried using the latest released version of Numba (0.59.0)
[x] I have included a self contained code sample to reproduce the problem.

Environment details:

$ conda list
# Name                    Version                   Build  Channel
_libgcc_mutex             0.1                        main
_openmp_mutex             5.1                       1_gnu
bzip2                     1.0.8                h7b6447c_0
ca-certificates           2023.12.12           h06a4308_0
ld_impl_linux-64          2.38                 h1181459_1
libffi                    3.4.4                h6a678d5_0
libgcc-ng                 11.2.0               h1234567_1
libgomp                   11.2.0               h1234567_1
libstdcxx-ng              11.2.0               h1234567_1
libuuid                   1.41.5               h5eee18b_0
llvmlite                  0.42.0                   pypi_0    pypi
ncurses                   6.4                  h6a678d5_0
numba                     0.59.0                   pypi_0    pypi
numpy                     1.26.4                   pypi_0    pypi
openssl                   3.0.13               h7f8727e_0
pip                       23.3.1                   pypi_0    pypi
python                    3.11.7               h955ad1f_0
readline                  8.2                  h5eee18b_0
setuptools                68.2.2                   pypi_0    pypi
sqlite                    3.41.2               h5eee18b_0
tk                        8.6.12               h1ccaba5_0
tzdata                    2023d                h04d1e81_0
wheel                     0.41.2                   pypi_0    pypi
xz                        5.4.5                h5eee18b_0
zlib                      1.2.13               h5eee18b_0

$ conda info

     active environment : numba059
    active env location : /home/tim/Software/miniconda3/envs/numba059
            shell level : 1
       user config file : /home/tim/.condarc
 populated config files : /home/tim/.condarc
          conda version : 4.9.2
    conda-build version : not installed
         python version : 3.9.1.final.0
       virtual packages : __glibc=2.35=0
                          __unix=0=0
                          __archspec=1=x86_64
       base environment : /home/tim/Software/miniconda3  (writable)
           channel URLs : https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /home/tim/Software/miniconda3/pkgs
                          /home/tim/.conda/pkgs
       envs directories : /home/tim/Software/miniconda3/envs
                          /home/tim/.conda/envs
               platform : linux-64
             user-agent : conda/4.9.2 requests/2.25.0 CPython/3.9.1 Linux/6.5.0-18-generic ubuntu/22.04.4 glibc/2.35
                UID:GID : 1000:1000
             netrc file : None
           offline mode : False

Issue with np.empty  when disabling jit
The following script can be executed successfully with NUMBA_DISABLE_JIT=0 python bug.py
import numpy as np
from numba import guvectorize

@guvectorize(
    [
        "(int64[:], int64[:])",
    ],
    "(n)->(n)", cache=False
)
def foo(x, out):
    np.empty(len(x))
    out[:] = np.sum(x)

x = np.array([1,2,3,4,5])
print(foo(x))

But when run with NUMBA_DISABLE_JIT=1 python bug.py it raises with
  File "/home/tim/Software/miniconda3/envs/numba059/lib/python3.11/site-packages/numba/core/typeinfer.py", line 1091, in propagate
    raise errors[0]
numba.core.errors.TypingError: Failed in nopython mode pipeline (step: nopython frontend)
No implementation of function Function(<built-in function empty>) found for signature:

 >>> empty(int64)

There are 2 candidate implementations:
  - Of which 2 did not match due to:
  Overload in function 'ol_np_empty': File: numba/np/arrayobj.py: Line 4363.
    With argument(s): '(int64)':
   Rejected as the implementation raised a specific error:
     AttributeError: 'function' object has no attribute 'get_call_template'
  raised from /home/tim/Software/miniconda3/envs/numba059/lib/python3.11/site-packages/numba/core/types/functions.py:541

Note that there is similar behavior withnp.zeros etc.
Confounding caching behavior
This behavior is more confusing when caching is enabled:
import numpy as np
from numba import guvectorize

@guvectorize(
    [
        "(int64[:], int64[:])",
    ],
    "(n)->(n)", cache=True
)
def foo(x, out):
    np.empty(len(x))
    out[:] = np.sum(x)

x = np.array([1,2,3,4,5])
print(foo(x))

If the above script is run with NUMBA_DISABLE_JIT=1 python bug.py then it will fail. But if I first run it with NUMBA_DISABLE_JIT=0 python bug.py and then NUMBA_DISABLE_JIT=1 python bug.py it will succeed. I need to rm -r __pycache__ to get it to fail again. So it appears that the cached (compiled) version is been used when jit is meant to be disabled.
With numba 0.48.1 I just get deprecation warnings the first time I run NUMBA_DISABLE_JIT=1 python bug.py but the code still executes. The second time I run NUMBA_DISABLE_JIT=1 python bug.py the code executes without deprecation warning implying that it is using the a cached version (is a non-jitted version cached?).

Just for clarity, I see that the docs state:

When this mode is enabled, the vectorize and guvectorize decorators will still result in compilation of a ufunc, as there is no straightforward pure Python implementation of these functions.

But that would suggest that the behavior should be identical regardless of NUMBA_DISABLE_JIT, or does this refer to a different compilation process?

That part of the docs was written 9 years ago. The behavior must have been drifted. However, the caching situation is indeed confusing. I'm looking into it.

A hack/workaround is just:
diff --git a/numba/np/ufunc/ufuncbuilder.py b/numba/np/ufunc/ufuncbuilder.py
index ee2438900..5e811d892 100644
--- a/numba/np/ufunc/ufuncbuilder.py
+++ b/numba/np/ufunc/ufuncbuilder.py
@@ -121,7 +121,11 @@ class UFuncDispatcher(serialize.ReduceMixin):
         #  python function
         flags.enable_looplift = False

-        return self._compile_core(sig, flags, locals)
+        try:
+            config.DISABLE_JIT = False
+            return self._compile_core(sig, flags, locals)
+        finally:
+            config.DISABLE_JIT = True

     def _compile_core(self, sig, flags, locals):
         """

From the discussion:

In the short term, we could implement this workaround in Numba to enable execution to proceed when jit is (supposedly) disabled.
In the longer term, it would take more effort to actually disable compilation for ufuncs and gufuncs.


How to output flow interface files

I hope this isn't a duplicate question.
With TypeScript it is possible to output .d.ts files that the compiler generates. This allows type declaration files to be read by other packages that one is developing (say package B) without having to provide all the .js source from an dependant package (say package B).
To be clear, I would like to be about to run Flow over .js sources in package A and produce a packageA.js.flow file that can be referenced in the [lib] section of the .flowconfig file in packageB.
Is this or similar functionality afforded by Flow?
Thanks

You can run flow gen-flow-files my-file.js > my-file.js.flow which generates a minimal flow definition.  Unfortunately it's still experimental/unfinished at the moment though.

Thanks so much for that. I'm happy enough to go with experimental; at the moment I'm doing it by hand.
I really hope 2017 is great for Flow as I appreciate its "higher moral ground" re type theory. Then again noone wants to get burnt on the bleeding edge. otoh I've really pushed the envelope of that other JS static type to the point that I'm about to throw in the towel and run blindly with vanilla ES2015.
p.s. @ForbesLindesay are you on Twitter?

twitter.com/ForbesLindesay


Mermaid not working in Mermaid Support guide

Mermaid not woring in https://toha-guides.netlify.app/posts/writing-posts/mermaid/

It was not enabled. This PR will enable it: https://github.com/hugo-toha/guides/pull/256


warning

Why it warn valid left, both and right values?

Brainfart on my end, my bad. Fixed in 51fc5a1fd095017bbc0edcdd86ffcb3880947adf. npm update to pull the patch.

Thanks!


Add reporting API

This change is

src/main/proto/wfa/measurement/reporting/v1alpha/event_group.proto line 41 at r8 (raw file):
Previously, SanjayVas (Sanjay Vasandani) wrote…
Also add event templates, and a TODO for adding metadata.

Added event_group_reference_id  and event_templates. Please let me know if it looks good to you.

Hi @chipingyeh, this is the PR for the API design of the reporting server. Would you mind if I ask you to be our external reviewer for this PR?
Looking forward to hearing from you soon. Thank you.


Installation for Python2 fails

It seems that the latest version is not compatible with Python2:

Clearly, Python2's open() function does not have the argument 'encoding', which causes the error in installation. Is there any previous version of the library compatible with python2?

Got it. That argument needed to be removed and then installing using the git clone method perfectly works!

Glad you figured it out. Thank you for sharing your advice with others!


Updates

Category

[X] Content fix

Docs Build status updates of commit 6e957b1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Community/versioning-basics-best-practices.md
:bulb:Suggestion
View
Details

Community/TOC.md
:white_check_mark:Succeeded
View

Community/large-lists-large-libraries-in-SharePoint.md
:white_check_mark:Succeeded
View

Community/permission-model-differences.md
:white_check_mark:Succeeded
View

Community/versioning-basics-best-practices.md

Line 58, Column 0: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text. NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Failed inspect causes container event spamming

Seen by one of our users.
If docker errors out when inspecting a container, this can cause rancher to spam container events and eventually fill up the database.
Steps to reproduce (abridged):

Create new docker host and reconfigure it to use device mapper.
Launch a container on that host
On host, delete /var/lib/docker/devicemapper/metadata/* files
Restart docker daemon.
Launch rancher-agent on the host.

Once the host is registered, it will try to sync that broken container and it will never succeed and create a new entry in container_event table every 5 seconds.

Note that at this point, the container will likely get stuck in a stopping state. @ibuildthecloud and I determined this was acceptable for now. We may address that separately, this fix was solely to stop the event spamming.

Rancher 	v0.35.0-rc4
Cattle 	v0.86.0
User Interface 	v0.45.0
Verified that container_event is not getting spammed.
mysql> select * from container_event;


visualizations

Hi,
I am interested of your work. And could you please tell me if there is any code available for visualizing detection results?
Thanks a lot!!!!

Please refer to the following links for a description of the demo visualization.
(https://github.com/sdroh1027/DiffusionVID/tree/main/demo)
I also updated the code, so please do git pull.


Move repository to https://github.com/apache/incubator-zipkin-dependencies

As part of the move to the Apache Software Foundation this repository will transfer ownership to apache and name to incubator-zipkin-dependencies.
See https://issues.apache.org/jira/browse/INFRA-16989

all sorted!


Tail Logs From Bottom of File or Find First Error

Working on tailing the log from the bottom of the file instead of the top.

Hey @Calvinp seems this has a bunch of extra diff in there, can we merge master into here so the run now prompt diff isn't in this as well? Should be good to merge soon, just want to check the changes again

Done

This looks gtg, thanks @Calvinp


[SPARK-32337][SQL] Show initial plan in AQE plan tree string

What changes were proposed in this pull request?
This PR adds initial plan in AdaptiveSparkPlanExec and generates tree string for both current plan and initial plan. For example:
Current AQE plan tree string
AdaptiveSparkPlan isFinalPlan=true
+- *(3) BroadcastHashJoin
   :- BroadcastQueryStage 2
       ...

It provides better visibility into the plan change introduced by AQE.
Does this PR introduce any user-facing change?
Yes. It changed the AQE plan output string.
How was this patch tested?
Unit test

Can one of the admins verify this patch?

Can one of the admins verify this patch?

cc @cloud-fan @gengliangwang Any comment on the new explain plan?  (looks fine to me)

Thank you for your contribution, @allisonwang-db . Thank you for your review.

ok to test

Test build #126139 has started for PR 29137 at commit 2889fb3.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/30751/
Test PASSed.

Merged build finished. Test PASSed.

Test build #126139 has finished for PR 29137 at commit 2889fb3.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126139/
Test FAILed.

Test build #126442 has started for PR 29137 at commit 1b3a685.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31056/
Test PASSed.

I've updated the plan output. Currently both initial and current plans are show as last children of AdaptiveSparkPlan. Please let me know what do you think :) cc @maryannxue  @cloud-fan  @gengliangwang.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126442/
Test PASSed.

Test build #126668 has started for PR 29137 at commit 7b6c999.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31289/
Test PASSed.

Test build #126669 has started for PR 29137 at commit 1ea4a3b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31290/
Test PASSed.

Test build #126668 has finished for PR 29137 at commit 7b6c999.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126668/
Test PASSed.

Test build #126669 has finished for PR 29137 at commit 1ea4a3b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126669/
Test PASSed.

== Physical Plan ==
AdaptiveSparkPlan (9)
+- == Current Plan ==
   BroadcastHashJoin Inner BuildRight (8)
   :- Project (3)
   :  +- Filter (2)
+- == Initial Plan ==
   BroadcastHashJoin Inner BuildRight (8)
   :- Project (3)
   :  +- Filter (2)

Why it's Current Plan not Final Plan in EXPLAIN FORMATTED? And can we use an example that AQE changes the plan?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31773/
Test PASSed.

Test build #127151 has started for PR 29137 at commit 57de5e5.

Test build #127151 has finished for PR 29137 at commit 57de5e5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127151/
Test PASSed.

retest this please

Test build #127157 has started for PR 29137 at commit 57de5e5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31779/
Test PASSed.

Test build #127157 has finished for PR 29137 at commit 57de5e5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127157/
Test PASSed.

The PR itself LGTM. Can we highlight the difference between Current Plan and Final Plan in the PR description? This is an important information of the AQE query plan.

I've updated the PR description. Please let me know if it makes sense.

thanks, merging to master!


[1.327] Way to disable Realtime CSG

It appears Realtime CSG and its GUI and hotkeys are permanently active across any scene in a project it's imported to, even if there are no Realtime CSG objects present.
This is a problem when you other packages that the Realtime CSG GUI or hotkeys interfere with, (for example, SabreCSG, which I've got in all my existing scenes).
You can't disable it by closing it, since it just alternates between the HUD and tab mode.
Maybe Realtime CSG should be off until you select a Realtime CSG model or brush?
Alternatively, some menu option to temporarily close as required rather than just toggle tab vs GUI?

That's probably a good idea, I'll look into this

related to #114

This is implemented in version 1.327, which is available on the update site https://prenominal.nl/updates/
Once this version is available on the asset store, I will close this ticket


Add CONTRIBUTING.md

Add contributing.md file to welcome new contributors to your project and to go through the guidelines for contributing to your project

Thanks for suggestion. Contributing.md file is now added.


Allow nodes to be created without a public IP

The way the flag was defined didn't allow to disable it because it's true by default. Clamp will detect the flag --no-associate-public-ip-address as a way to set this to false which is useful in internal deployments where the nodes shouldn't have a public internet addressable IP.

LGTM, thanks. Having a flag with default value true indeed makes very little sense :)


formatter only allows two values in for loop

Valid odin code that iterates over three values

ols removes the z from the for loop

it should allow any number of elements in the loop

Should be fixed.


add note to README about output buffering

With the current implementation of run_cmd_stream_output, Packer's output will often get held in Ruby's output buffer. Provide instruction on how to enable synchronous output so it will show up right away (i.e. in Jenkins).

What do you think about making this a supported setting, like this?

Looks good to me! Thanks. I guess it just needs a doc update.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/130 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target.git

Docs Build status updates of commit 9b83a0d:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-10-08-02-36-30 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-10-08-02-36-30

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


IOError: Error while reading from plink file.

Hi,
I've been trying to run PleioPred, but I can't seem to run it successfully on the non-testing data. I found that when my .bim file is >10,000 lines, I get the error below or a memory error. I'm using a cluster and was doing computations on a 30G node, so I am not sure why memory seems to be a problem.
Best,
Ruthie
Iterating over file to load SNPs
Traceback (most recent call last):
  File "/u/home/r/ruthjohn/ruthjohn/PleioPred/PleioPred.py", line 285, in <module>
    main(process_args(args))
  File "/u/home/r/ruthjohn/ruthjohn/PleioPred/PleioPred.py", line 243, in main
    coord_trimmed.main(pdict_coord_trimmed(pdict, '1'))
  File "/u/project/pasaniuc/pasaniucdata/ruthjohn/PleioPred/pleiopred/coord_trimmed.py", line 1032, in main
    hdf5_file=h5f, min_maf=p_dict['maf'])
  File "/u/project/pasaniuc/pasaniucdata/ruthjohn/PleioPred/pleiopred/coord_trimmed.py", line 856, in coordinate_genotypes_ss_w_ld_ref
    raw_snps,freqs = _parse_plink_snps_(genotype_file, snp_indices)
  File "/u/project/pasaniuc/pasaniucdata/ruthjohn/PleioPred/pleiopred/coord_trimmed.py", line 58, in _parse_plink_snps_
    plinkf.next()
  File "/u/home/r/ruthjohn/.local/lib/python2.7/site-packages/plinkio/plinkfile.py", line 58, in next
    row = cplinkio.next_row( self.handle )
IOError: Error while reading from plink file.

Hi Ruth,
The error was raised by plinkio module. So I think there might be something wrong with the plink files. Could you double check if the plink files are in correct form or you could share a screen-shot of .bim file if you don’t mind? Thanks!
Best regards,
Yiming
Yiming Hu
Yale School of Public Health | May 2018
Biostatistics Ph.D Candidate
<EMAIL_ADDRESS><EMAIL_ADDRESS>|<PHONE_NUMBER>

On Sep 1, 2017, at 2:29 PM, Ruth Johnson<EMAIL_ADDRESS>wrote:
cplinkio.next_row

Hi Yiming,
Thanks for the quick reply!
Here's the head of the .bim file I'm using:
1	rs535605744	0	1043557	T	C
1	rs555684725	0	1043590	A	G
1	rs566020904	0	1043621	T	C
1	rs533574128	0	1043640	A	G
1	rs553704858	0	1043648	G	A
1	rs576737249	0	1043676	T	C
1	rs76099452	0	1043737	C	T
1	rs12751333	0	1043743	C	T
1	rs548829137	0	1043746	T	TCAC
1	rs576307341	0	1043804	T	C

The entire file is ~6 million lines. Since I thought it was a memory issue at first, I just used the first 10,000 lines to test it, which seemed to work. It was when I used any file larger than that where I get the error.

I see. The largest file I have ever ran with plinkio is no larger than one million SNPs so I am not sure if they could handle such a large file. Have you tried using only the genotyped SNP (should be around 500,000 SNPs)?
Best regards,
Yiming
Yiming Hu
Yale School of Public Health | May 2018
Biostatistics Ph.D Candidate
<EMAIL_ADDRESS><EMAIL_ADDRESS>|<PHONE_NUMBER>

On Sep 1, 2017, at 3:15 PM, Ruth Johnson<EMAIL_ADDRESS>wrote:
Hi Yiming,
Thanks for the quick reply!
Here's the head of the .bim file I'm using:
1	rs535605744	0	1043557	T	C
1	rs555684725	0	1043590	A	G
1	rs566020904	0	1043621	T	C
1	rs533574128	0	1043640	A	G
1	rs553704858	0	1043648	G	A
1	rs576737249	0	1043676	T	C
1	rs76099452	0	1043737	C	T
1	rs12751333	0	1043743	C	T
1	rs548829137	0	1043746	T	TCAC
1	rs576307341	0	1043804	T	C
The entire file is ~6 million lines. Since I thought it was a memory issue at first, I just used the first 10,000 lines to test it, which seemed to work. It was when I used any file larger than that where I get the error.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/yiminghu/PleioPred/issues/2#issuecomment-326662680, or mute the thread https://github.com/notifications/unsubscribe-auth/AK70uE75s8GDwP9g9xxGp8YMCgJAO_SPks5seFfjgaJpZM4PKfQZ.

Oh I see, so you're saying you've not yet used a .bim file that large before.
And sorry, could you clarify what you mean by only using the genotyped SNPs? Do you mean as in, the bim file only containing the SNPs in the GWAS sum stats you're inputting?

Yes, I mean I have never used that many SNPs in this step. I was suggesting using a subset of SNPs to do risk prediction for example you could find a list of SNPs (those non-imputed, since based on the size, it seems the file you have contains inputed SNPs). Then use plink to extract the subset and get a smaller plink file.
Best,
Yiming
Yiming Hu
Yale School of Public Health | May 2018
Biostatistics Ph.D Candidate
<EMAIL_ADDRESS><EMAIL_ADDRESS>|<PHONE_NUMBER>

On Sep 1, 2017, at 3:30 PM, Ruth Johnson<EMAIL_ADDRESS>wrote:
Oh I see, so you're saying you've not yet used a .bim file that large before.
And sorry, could you clarify what you mean by only using the genotyped SNPs? Do you mean as in, the bim file only containing the SNPs in the GWAS sum stats you're inputting?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/yiminghu/PleioPred/issues/2#issuecomment-326665643, or mute the thread https://github.com/notifications/unsubscribe-auth/AK70uLZVrtYkuRgFEuP0xKkXbNyuie0wks5seFtqgaJpZM4PKfQZ.

Yes, that worked perfectly! Thank you again for your help.


feat: adding a skip to search anchor

UPD: due to accessibility-related reasons, the autofocus attribute idea was parked. Instead, this PR adds the "Skip to search" anchor.
Original:
This PR is a suggestion to add the autofocus attribute to the search input. The main driver is that when opening the docs.npmjs.com page, a user needs to set the focus in the input manually to start searching.
There are, however, some accessibility trade-offs to consider. Therefore, I am open to any feedback and outcome.

Due to the accessibility tradeoffs, I think it might be better to add "Search" to the skip nav. That way the initial  will open the skip nav with two options: skip to content and skip to search. Selecting skip to search will focus the input.
Thoughts on that?

I like this a lot! Do you think the search input should also be focused after Skip to Search is selected?

@ivan-demchenko I've been meaning to restyle the SkipBox so I just pushed a commit that restyles that and also tries to use the Gatsby onRouteUpdate handler to automatically focus the search input when the hash is set to #search-input-box. Let me know what you think!

@ivan-demchenko There's also the issue of needing to open the search drawer on smaller screens when skipping to the search. I might try to come up with a solution for that in the next couple days. But feel free to try that out as well if you want.

@lukekarrys the new styling looks great, thank you! Also, I think, the input should be focused when "Skip to search" has been selected.
I noticed that on Firefox on Mac, the first tab doesn't trigger the "Skip to ..." link to focus. The focus jumps straight to the search bar. This behaviour can be observed on Safari too. I found how to "fix" it (here or here) for this problem, after which Firefox started behaving as expected. But I don't believe the users will ever try to change their system settings to make the "Skip to..." feature work for them.

As for opening the search box on the mobile, I'd use CSS for that (instead of JS and React hooks). The Search "button" (with the 🔎 icon) can be a Label with the input inside. Then, with :focus-within we could show/hide the elements as we see fit. What do you think?

@ivan-demchenko That PoC looks really nice and very clean! I like it a lot. I'm hesitant to include something like that in this PR since it would likely require undoing a lot of the current React hooks implementation of this. I'd be in favor of including that in a future PR if you would like, in order to keep the scope of this one small.
Considering that there is no way to focus the search at all currently, I think this is shippable without mobile support for now. What do you think?

@lukekarrys Thank you very much for merging the PR, this is very exciting!
Oh, absolutely! I think we should take small steps and improve things incrementally, I've never been a fan of big-bangs.
As per the PoC, I agree with you, this would be a big change. You listed the cons correctly. As per the pros, it would reduce the number of dependencies on the JS libraries, however, resulting in a smaller bundle. But anyway, this is for the next time.


Pubsub-based moderation actions are not hidden when disabling show moderation messages

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
Moderation messages that are sent via pubsub are not hidden when disabling show moderation messages

Automod Messages
Automod Phrase approvals/denials
Unbans

Moderation actions that have a non pubsub variation (timeouts & bans) are properly hidden from users and mods.
Screenshots

left side felanbot w/ setting on & commands executed from there | right side felanbird with setting disabled

OS and Chatterino Version
Chatterino Nightly 2.3.4 (commit 1d272c39, built on 2022-01-16)

she thinkin real hard at the exact moment of screenshot

monkaS


Use bifurcated Pub/Sub template

Updates GcpPubSubAutoConfiguration to create and use PubSubPublisherTemplate and PubSubSubscriberTemplate.
The Pub/Sub outbound channel adapter now uses PubSubPublisherTemplate rather than PubSubTemplate.
The Pub/Sub inbound channel adapter now uses PubSubSubscriberTemplate rather than PubSubTemplate.
Fixes #823.

@artembilan Can you take another look pls? Thanks!

@artembilan friendly ping :)

This PR in some shape is clashing with the #841 .
So, we need to decide first of all which direction we go, then review one or another.
Otherwise it is going to be a wasting time.
No? What am I missing?

I don't think it's really clashing. Let's merge this in because it's less controversial. #841 is orthogonal to this.


Support Message Allowed Mentions

If allowed mentions aren't passed by the library, it is taken from context. Passing a custom allowed mention object is useful for parsing and returning user-provided messages to not accidentally ping or even use as a logger so that events can be logged with a mention to the user without that user being bombarded with mentions.
https://discord.com/developers/docs/resources/channel#allowed-mentions-object
Oops, closed by #4

Closed in #30


Unittest

implemented unittests to validate shot objects.

setup.py is obsolete and I forgot to delete it.  I originally created it because setup.py is required for pytest, but then I abandoned pytest in favor of unittest.

Great! Howard and I were just talking about this at lunch today.
On Fri, Oct 30, 2015 at 3:07 PM, David R. Smith<EMAIL_ADDRESS>wrote:

setup.py is obsolete and I forgot to delete it. I originally created it
because setup.py is required for pytest, but then I abandoned pytest in
favor of unittest.
—
Reply to this email directly or view it on GitHub
https://github.com/Fusion-Data-Framework/fdf/pull/35#issuecomment-152619734
.


Building image for 2.0.16

This is an update to the Dockerfile to build the image for 2.0.16

Thanks @blauret. I've created a release: https://github.com/jenswet/docker-smartsnippets-studio/releases/tag/2.0.16
Docker is building the image, it should be available in a few minutes.

I'm already diverging a bit from that PR as I need python-pip. See this PR.
I'm currently checking if we can have an official Dialog image on dockerhub. For now I'm sharing my flavour here

Hi @blauret, having an official image on the Docker Hub would great! Thanks for working on this!

@jenswet ,
It's here: https://hub.docker.com/repository/docker/diasemi/smartsnippetsstudio


Include note about tricky issue in installation docs to prevent confusion

Description
When we were installing this gem at NoRedInk, we ran into a weird issue where handled were working fine, but unhandled exceptions didn't send. After extension debugging, we realized that it was because ruby composes gem middelwares by their order in the Gemfile, and the issue was that our bugsnag gem was listed after another gem, which also captured exceptions but didn't reraise them. Because of this, the exceptions never reached bugsnag. The fix was to simply move our bugsnag gem above the other gem in the Gemfile!
Issue
This ticket is merely a suggestion to add a little note about this issue in the installation docs! It's technically none of your business, since it's just a bad interaction with another gem, but we spent almost a whole day being confused and were close to just giving up on bugsnag, so figured it might be relevant to you after all. We're of course all good now, so it's just for future users!
Example code snippet
This produces the issue:
group :development do
  gem 'better_errors'
end

gem "bugsnag", "~> 6.13"

This fixes the issue:
gem "bugsnag", "~> 6.13"

group :development do
  gem 'better_errors'
end

Altogether, we really like your work- thank you!

@terezka Thank you very much for the suggestion, we're going to update our docs to include a note, as you suggest.  Many thanks and glad you're enjoying using Bugsnag!


Unable to install postgresql-dev since last update

Hello there,
We are unable to install the postgresql-dev package at version 9.5.4-r0 since the last version of your image. The error it gives:
ERROR: unsatisfiable constraints:
  postgresql-dev-9.6.0-r1:
    breaks: world[postgresql-dev=9.5.4-r0]

It's probably because you replace the default repositories with the edge ones?
The Dockerfile

That's exactly it! Damn, I'm having all sorts of problems with packages dependencies in Alpine at the moment.
Could you please tell me:

The command you are using to install specific version of postgresql-dev
The tag you are trying to install to (3.4 or latest?)

Yeah sure!
FROM jfloff/alpine-python:2.7

RUN apk add --update \
    linux-headers \
    postgresql-dev=9.5.4-r0 \
    libxml2-dev \
    libxslt-dev \
    ncurses5-libs \
    rsync \
    nodejs \
    libjpeg-turbo-dev

We first had it like this, before the last update, but simply removing the @ edge didn't fix it:
FROM jfloff/alpine-python:2.7

RUN echo "@edge http://nl.alpinelinux.org/alpine/edge/main" >> /etc/apk/repositories && \
    apk add --update \
<EMAIL_ADDRESS>\
        libxml2-dev \
        libxslt-dev \
        ncurses5-libs@edge \
        rsync \
        nodejs \
        libjpeg-turbo-dev@edge

Thanks for the quick fix!
2 more things:

I saw you added linux-headers to the 3.4 dockerfile; good idea to also do it for the 2.7 one?
Can you start a build on dockerhub so our ci can continue building?

Sorry about the delay, I was on a meeting. I added linux-headers by mistake, but I'm keeping them since they were also on needed on the slim. Pushing that to docker hub now. Hang in there!

I've already pushed to docker hub, but due to a know bug I'm getting failed on a couple of images (namely slim and onbuild

Just an update: After the DDoS issues on Friday, I got the 2.7, 3.4 and latest tags to build on Docker Hub. But the remaining tags (all onbuild and slim) are still failing due to a known bug.
When you can verify that your ci is back, up and runing. Thanks!

Thanks, our CI works again :)
My compliments for your response time! And for the nice base image, even with dependencies built in (which the default python alpine hasn't) it results in a smaller and more capable image!

Thank you so much for you feedback! Really glad you guys are using it :)
Any suggestions or problems feel free to drop a line! Cheers!


What effect does g:suda_startup has?

I would like to contribute to this plugin a new setting that will enable automatically writing with suda when no permissions to the current user are available. I was reading the code which is written very well and clear, yet I didn't understand the effect g:suda_startup has. I've tested it and whether it set or not, nvim suda:///etc/bla asks for a password.
Thanks.

That is used in plugin/suda.vim to call suda#init().

I've tested it and whether it set or not, nvim suda:///etc/bla asks for a password.

It should be let g:suda_startup = 0 to disable that feature instead while the default value is 1 instead of 0 when there is no such variable.

Got it, thanks!


Exclude SRIOV annotation from compare

The
metadata.annotations."operator.sriovnetwork.openshift.io/last-network-namespace" annotation is automatically added and maintained by the SRIOV operator. It should not be included in the comparison.

/lgtm
/approve


Add ability to check is the domain name is supported

It would be great to have an ability to check is the domain name supported without making a network request when getting an address.

Your build is failing, but that's because your branch does not have access to our repo's secrets which needed to run tests. We will be able to merge it anyway – it will pass when it will be a part of our codebase.

Your build is failing, but that's because your branch does not have access to our repo's secrets which needed to run tests. We will be able to merge it anyway – it will pass when it will be a part of our codebase.

Please, update the changelog along with the version in https://github.com/unstoppabledomains/resolution-java/blob/master/src/main/resources/com/unstoppabledomains/config/client/client.json.

Please, update the changelog along with the version in https://github.com/unstoppabledomains/resolution-java/blob/master/src/main/resources/com/unstoppabledomains/config/client/client.json.


Apply to the Google RISE Awards

https://www.google.com/edu/resources/programs/google-rise-awards/
A few people have recommended we apply and it seems to be the sort of grant that's right up our ally. Let's apply to this. Deadline is February 19th, 2016.

@zachlatta Yo, this deadline is approaching really damn fast!

Thanks for the reminder guys :-).
I've assigned to @jkwok91. Will work with her to get this done.
On Mon, Feb 15, 2016 at 2:27 PM, Max Wofford<EMAIL_ADDRESS>wrote:

https://camo.githubusercontent.com/7070c7d972a48769eef61fdb866ae4b87d2a68f4/68747470733a2f2f692e696d6775722e636f6d2f76424844736a372e6a7067
—
Reply to this email directly or view it on GitHub
https://github.com/hackclub/meta/issues/445#issuecomment-184423186.

--
Zach Latta
https://zachlatta.com http://zachlatta.com

Getting started on this ASAP.


Webhook for notifications

Hi there,
Thank you for this great piece of software. While I see that there is support for Kafka and Nats, I was wondering if a simple webhook would be possible in the future.
The reason is that both Kafka and Nats require installation, configuration, etc. However, it feels like a total overkill when one wants to know if an upload is done.
Maybe this is just me, but I think it would make a great addition, no? Of course, if there is another option that I'm missing, please let me know.
Thank you for considering.

@thenitai I was thinking about this as well when we were first implementing the event system. But I was mainly waiting for someone to ask to make sure there was interest in this. I think we could add this.

@thenitai, PR #488 was merged with the webhook events.  See if this looks like it will work for you. We are going to be adding a bit more configuration around what events to send to various event backends in a future PR.

@thenitai, PR #488 was merged with the webhook events. See if this looks like it will work for you. We are going to be adding a bit more configuration around what events to send to various event backends in a future PR.

First off, thank you for implementing.
I've read the code but honestly not sure how to use it. Would it be the same as the other notification syntaxes, e.g., --event-webhook https://app/event ? I imagine that the most important notification is when a file has finished uploading.
I can test it here, too.
Thank you.

I've read the code but honestly not sure how to use it. Would it be the same as the other notification syntaxes, e.g., --event-webhook https://app/event ? I imagine that the most important notification is when a file has finished uploading.

The option would be --event-webhook-url <url>. This will currently send all events, but we have another PR in progress now to add filtering capabilities to events with a similar configuration to aws.

New utils command addes to generate a default notification filter, pass in directory to create file. For example:
./versitygw utils gen-event-filter-config -p /tmp

This will create /tmp/event_config.json. Edit this to only true for the events you want to be notified for. Then pass in the options:
--event-webhook-url <webhook-url> --event-filter /tmp/event_config.json

Awesome. I will give it a go shortly. Thank you so much for making it possible in a short time.


Simplify and cleanup MultiJvm, MultiNode, Copyright in sbt build

Originally this PR was only meant to remote the .configs(MultiJvm) since its not necessary due to https://github.com/apache/incubator-pekko/blob/main/project/SbtMultiJvm.scala#L83. Since then I went into a bit of a rabbit hole due to hitting the really weird way that some parts of the sbt build have been written in Pekko.
Basically the sbt build of the core pekko project is kind of a mess. A lot of the internal sbt plugins that are written are not done properly, i.e. instead of using object MySbtPlugin extends AutoPlugin ++ override def requires (as you should be doing) currently a lot of plugins use trait composition/super overrides which is not how sbt plugins are supposed to be structured when it comes to composition/inheritance.
This PR fixes all of this, i.e. the current sbt plugins have been changed so its written the proper sbt way which makes the build easier to understand.
Due to this PR changing how MultiJvm interacts with CopyrightPlugin/headerCheck its recommended to verify that headerCheckAll works on sources contained within multi-jvm, i.e.
remote-tests/src/multi-jvm
cluster/src/multi-jvm
cluster-metrics/src/multi-jvm
cluster-typed/src/multi-jvm
cluster-tools/src/multi-jvm
cluster-sharding-typed/src/multi-jvm
distributed-data/src/multi-jvm
cluster-sharding/src/multi-jvm

You can test this by manually removing a header from a source file and then doing remote-tests/headerCheckAll (using remote-tests as an example). I did this myself but best to double check.

Lgtm, and will run the multi node sites locally later.

When fixing this I came across some other problems, seems like the current sbt build was a bit brittle and the headerCheck is now failing.
I will look into this a bit later

I'm verify it on gitpod, will report back.

@mdedetrich I have verified it, I think this one should be good to go.

I just rebased but it was because of a spelling error in the commit message


Create main page

Feature Description
Display main page
Major User Stories/tasks

create core files and merge to GitHub repo
create template based on Jacek's sketch
make the website responsive

Additional Considerations

This epic is completed.


chore(chart-controls): more accurate typing for DatasourceMeta

🏆 Enhancements
🏠 Internal
Update DatasourceMeta to conform better with BaseDatasource.data

The codebase on this repo has been moved to the main Apache Superset repo, and consequently the repo is in the process of being archived. See the Superset Improvement Proposal for details: https://github.com/apache/superset/issues/13013 . While all currently open issues and PRs will be closed, we encourage you to reopen this PR on the main repo, which should be as simple as moving over any code changes as follows:

core packages: packages/** (this repo) -> superset-frontend/packages/** (main repo)
plugins: plugins/** (this repo) -> superset-frontend/plugins/** (main repo)

If you need help with the migration, please post a message on the SIP or reach out on the community Slack.


Fix Windows paths problem in logicalPath, fix tests

I have changed implementation of logicalPath getter in AssetAttributes to avoid 'File outside paths' error on Windows in some cases.
Also I have modified examples_test.js to run in different environments (e.g. on Windows).

LGTM!

Thanks!


go logfmt in vespalog

this is supposed to be a drop-in replacement or the perl script vespa-logfmt with same options and basically same behavior.  Probably needs some unit tests of the underlying implementation, so not ready for merge yet, but some review can happen now (while I'm on vacation).
Some unresolved questions:

I hooked building the go code into cmake, is there any problem with that?
if not, should we do that for the client/go directory too?
I'm considering moving the actual logfmt code from client to vespalog, but some parts could be useful
my first try with "vespa logfmt" should be removed, it doesn't fit with the rest of the "vespa xxx" commands and isn't compatible with perl script.

I hooked building the go code into cmake, is there any problem with that?
Should be fine.

if not, should we do that for the client/go directory too?
Sure. I believe it's already included in the vespa-clients RPM though.

I'm considering moving the actual logfmt code from client to vespalog, but some parts could be useful
my first try with "vespa logfmt" should be removed, it doesn't fit with the rest of the "vespa xxx" commands and isn't compatible with perl script.

So it's possible to build multiple binaries within the same Go module, just create a separate package containing the function, e.g. ```$ $ cat cmd/vespa-logfmt/main.go
package main
import "fmt"
func main() {
fmt.Println("hello from vespa-logfmt")
}```

will make new PRs with squished commits, thanks for many useful comments


Assymetric shapes not documented

Hi,
At http://knsv.github.io/mermaid/flowchart.html we learn that it is possible to use:
id1>This is the text in the box]

I tried other shapes like:
id1[This is the text in the box<

don't work. Would it be possible to list available shapes, or say that >...] is the only available, or maybe even implement other shapes ;) ?
Thanks.

Will take look at the docs. And why not implement the reversed assymetric shape as well. What do you think @bjowes ?

I found that the assymetric shape is now already documented, excellent! Adding the reverse assymetric shape is fairly easy, I will look into it.

@knsv: I added the reversed asymmetric shape in my last pull request, but could not work out the jison to handle the syntax. The attempt is included in the pull request as a commented out section in the jison. Apparently [ .. > conflicts with text elements. Can you see a way around it?

I will close this issue and add a new one regarding adding a reversed assymetric shape.


Blank screen/unresponsive when used on Nginx server

You found the solution? Am facing the same issue.

I figured out that you need to run the page with an SSL cert (https).  This is due to Chrome security settings preventing the getUserMedia APIs from being accessible from an unencrypted page.

Yes, all the browser require SSL certificate to access you webcam.


App crashes in Heroku due to unexpected token import.

I've run create-react-app in my repo, changed nothing but some superficial text in App.js, and added "engines": { "node": "8.5.0" } to package.json. This repo is linked to a Heroku staging app which builds every time I push to the master branch. Watching the build log I can see it builds (and builds fine) using node 8.5 as commanded, but when I open the app page, it immediately fails with SyntaxError: Unexpected token import in line 1 of src/index.js (the import react line). I don't see why it's doing this, since on my local machine I can do npm start no problem. All that my Procfile contains is web: node ./src/index.js. I tried babel-node but it also failed, saying it couldn't find the command.

Did you follow these instructions and use the create-react-app buildpack?

No, since I'd already had a repo created in my account, I didn't want to make a new one. Similarly, I'd already created staging and deployment apps that were linked to my Github repo on Heroku, so all I did was tell it to deploy on master changes.

The create-react-app buildpack is mandatory, so please change that in your Heroku UI or CLI.

Oh, that's only supporting static pages, whereas I plan to have a server.js doing things eventually. I should probably find a starting project that comes with server support then. Thanks for the heads up.

You should create a server of your choice then, e.g. a simple express server.


Any chance this will be updated for Swift 4?

This looks promising

https://github.com/atacand1920/CameraKit , work on iOS 12


Can't find path to controller

I just started using Homebridge on my Mac. Now I have added the Linak controller and added the UUID of the desk, but still the desk does nothing in the Home app (stays at 40% and does nothing). I think I have done everything correctly, but the 'path to Idasen-Controller' is incorrect. Here is something standard that is intended for the RPi, I think. What is the path for the Mac? Or how can I trace this? Or am I doing something else wrong?

I think you may want to ask for help in the Homebridge linak controller repo? https://github.com/vniehues/homebridge-linak


Is there a way with this library to test if a file is a valid SQLite database?

Hello. I have a question about whether something is possible to do with this library. Say I have a folder of many files of many different types -- how can I pick out the files which are SQLite databases from files of other types (without relying on file extensions)? Thanks

const Database = require('better-sqlite3');

function isSqliteDatabase(filepath) {
  try {
    new Database(filepath, { fileMustExist: true });
    return true;
  } catch (err) {
    return false;
  }
}

This should generally work too (without using this library), but may rarely give false positives:
const fs = require('fs');

function isSqliteDatabase(filepath) {
  const file = fs.openSync(filepath);
  try {
    const buf = Buffer.alloc(16);
    fs.readSync(file, buf, 0, 16, 0);
    return buf.toString() === 'SQLite format 3\0';
  } catch (err) {
    return false;
  } finally {
    fs.closeSync(file);
  }
}

https://www.sqlite.org/fileformat.html

Thank you for responding and for the help :+1:


fix Project tag line not set in edit project settings

What does this PR do and why?
Closes JB-713 : Project tag line not set in edit project settings
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3968   👈

This comment was auto-generated by Graphite.