Nav bar

Description
Creates a Navigation HOC.
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Change Status

[x] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] There are no merge conflicts

original branch closed, way too many merge conflicts to resolve, this branch didn't reflect the work paul contributed, reopened a new branch that does


feat(storage-resize-images): add output options

fix: #620
Problem
Users would like to set up custom Output Options like quality and others for the converted files.
Proposed Solution
Create optional param for feeding JSON object with settings for converted files.
Jobs

[x] add OPTIONS_OUTPUT param with validation
[x] update docs with example
[x] parse JSON and provide correct format options into buffer creation

Test
Installed configuration

Succesful results

Codecov Report

Merging #878 (9c6da63) into next (eec2964) will increase coverage by 0.19%.
The diff coverage is 85.72%.

@@            Coverage Diff             @@
##             next     #878      +/-   ##
==========================================
+ Coverage   50.19%   50.38%   +0.19%
==========================================
  Files          30       30
  Lines        1329     1336       +7
  Branches      197      203       +6
==========================================
+ Hits          667      673       +6
  Misses        661      661
- Partials        1        2       +1

Impacted Files
Coverage Δ

storage-resize-images/functions/src/config.ts
94.12% <50.00%> (-5.88%)
:arrow_down:

...torage-resize-images/functions/src/resize-image.ts
43.43% <100.00%> (+3.99%)
:arrow_up:


Fix. UKRAINIAN borscht cannot be RUSSIAN

Common, man, this is not a /pol/

Come on, man, this is not a /pol/

Did you even read the diff?

Come on, man, this is not a /pol/

Did you even read the diff?

Of course I am.

Borscht is a Ukrainian recipe. But this is a Russian recipe too. So I don't see any problem here.

Borscht is a Ukrainian recipe. But this is a Russian recipe too. So I don't see any problem here.

Why do you think so? Borsch is a Ukrainian recipe, not a Russian one.

Borscht is a Ukrainian recipe. But this is a Russian recipe too. So I don't see any problem here.

Why do you think so? Borsch is a Ukrainian recipe, not a Russian one.

Because Ukraine became an independent country only in 1991. Bur borscht appeared in the 16th century, when Russia and Ukraine were one state.

Borscht is a Ukrainian recipe. But this is a Russian recipe too. So I don't see any problem here.

Why do you think so? Borsch is a Ukrainian recipe, not a Russian one.

Because Ukraine became an independent country only in 1991. Bur borscht appeared in the 16th century, when Russia and Ukraine were one state.

Firstly, borscht was invented by Ukrainians in the territory where they lived. Secondly, in the 16th century, the territory of Ukraine was under the Polish–Lithuanian Commonwealth.

Firstly, borscht was invented by Ukrainians in the territory where they lived. Secondly, in the 16th century, the territory of Ukraine was under the Polish–Lithuanian Commonwealth.

Anyway Ukrainian borscht recipe and Russian borscht recipe it's literally the same recipe. So it doesn't make any sense to remove "russian" from this tag list.

Anyway Ukrainian borscht recipe and Russian borscht recipe it's literally the same recipe. So it doesn't make any sense to remove "russian" from this tag list.

But the origin of borscht is Ukraine, you can't call it Russian. Why am I a racist?

@LukeSmithxyz

I propose making the tag "Slavic"


The Data is available Now？

Thanks for the Meaningful exploration, is the data can sharing?

Yes. We will share the data soon!


Ref PR #1336: Include AWS driver usage & docker cli +x in documentation

Reference PR #1336
detailed AWS deployment documentation
Signed-off-by: Kendrick Coleman<EMAIL_ADDRESS>Documented making docker client executable
Signed-off-by: Kendrick Coleman<EMAIL_ADDRESS>Added chmod +x after using curl for the Docker CLI to make it
executable in the documentation.
Added an example for using the AWS EC2 driver in the documentation.

@moxiegirl PTAL

LGTM
ping @thaJeztah

Thanks! writing LGTM (haven't checked the procedure)

@kacole2 apologies we are pausing on merging this because of #1490 ...when that one merges, you'll need to rebase.   I realize this is extra work for you so if you are not willing to wait, I'm happy to carry this for you.

@moxiegirl no problem. I'll rebase once it's merged.

@moxiegirl any update here?

@kacole2 #1490 split out each of the commands into separate files in machine...you actually need to do that to your content as it assumes the old, unified index.md it will merge on a rebase but it is destroying the split.

@moxiegirl is it still necessary?
Can we carry this given @kacole2 has already been through a lot to get there (from #1336)?
Thanks.

@dmp42 I'll carry this.  @kacole2 Thank you.


Add newsfeed autoload option

As far as I can tell there's no option to disable autoloading newsfeed.
This is a problem for me because it makes the footer of the main page unreachable. The links in the footer are convenient, and it doesn't make sense to navigate to another page just to reach them.
(There's also that autoloading makes the experience slightly more addictive because you don't get to make an explicit decision to view more content.)
Example URL: https://github.com/

It’s called infinite-scroll and it can already be disabled

Thanks!


Migrate logpush_job to use output_options

Current Terraform and Cloudflare provider version
1.0.0
Description
Cloudflare are replacing logpull_options with output_options which provides new functionality: https://developers.cloudflare.com/logs/reference/log-output-options/
Please can we support the new output_options format?
Use cases
As a user, I would like to export log files as CSV files (which output_options allows) so that I can import them into my datawarehouse natively.
Potential Terraform configuration
  enabled             = true
  zone_id             = id
  destination_conf    = local.destination_conf
  dataset             = "http_requests"
  frequency           = "low"
  ownership_challenge = data.aws_s3_object.challenge_file_seo.body
  output_options = jsonencode({
    where = {
      field_names      = ["BotScore", "BotScoreSrc", "BotTags", "ClientRequestHost", "ClientRequestMethod", "ClientRequestURI", "ClientRequestPath", "ClientRequestUserAgent", "EdgeEndTimestamp", "EdgeResponseBytes", "EdgeResponseStatus", "EdgeStartTimestamp", "RayID", "ClientDeviceType", "ClientIPClass", "EdgeResponseContentType"]
      timestamp_format = "rfc3339"
      output_type      = "csv"
    }
  })
  filter = jsonencode({
    where = {
      key      = "ClientIPClass"
      operator = "eq"
      value    = "searchEngine"
    }
  })
}

References
No response

This has been flagged with the Data (Logs) team for escalation. PM is currently OOO. Will get an approx ETA from her upon return


Not able to select a layer inside Layer Control by left click.

Hi,
I'm using this inside Ionic Framework along with ngCordova if that matters.
When I open the app on desktop browser (chome/firefox) the map opens up and left clicking the layer selects it.
But if I interact with another other thing on the app (e.g. menu button) the layer selection stops working with left click.. After testing out a while I found out that right clicking on the layer radio button select its and only left click is not being used on it..
It seems like the left click event is being canceled for some reason.
Any idea what might cause this issue?
I'll try this with a vanilla angular installation soon and see if the same happens.
Update
http://codepen.io/anon/pen/reNOLd
Try clicking the zoom (+/-) button on the top left corner.
Steps to reproduce

Click the home button in the footer
Then try clicking any of the controls / links on the map, they don't work.

Sorry it turned out that this was related to Ionic Framework.
More on it here
https://github.com/driftyco/ionic/issues/1505


Simple example doesn't work with axum 0.7.5

The following snippet:
    let app = Router::new()
        // `GET /` goes to `root`
        .route("/", get(hello))
        .layer(GovernorLayer {
            config: governor_conf,
        });

Yields the following error:
Diagnostics:
1. the trait bound `std::convert::Infallible: std::convert::From<std::boxed::Box<(dyn std::error::Error + std::marker::Send + std::marker::Sync + 'static)>>` is not satisfied
   the trait `std::convert::From<!>` is implemented for `std::convert::Infallible`
   for that trait implementation, expected `!`, found `std::boxed::Box<(dyn std::error::Error + std::marker::Send + std::marker::Sync + 'static)>`
   required for `std::boxed::Box<(dyn std::error::Error + std::marker::Send + std::marker::Sync + 'static)>` to implement `std::convert::Into<std::convert::Infallible>` [E0277]
2. required by a bound introduced by this call [E0277]

I was confused and just realised that the latest version has not been released

Sorry about that, let me get a new version out. Past due anyway most likely


Inconsistent Asset Image path across Mobile and Web platforms

Steps to Reproduce

Save an image to images, a subfolder within assets
Add assets/images to the pubspec.yaml and run flutter pub get
Retrieve the image using AssetImage("images/selected_image")
Run the Flutter app

Expected results:
The image retrieved from assets and displayed.
Actual results:
On Flutter Web, the above code functions as expected and the image is retrieved; in order to get the image to display on mobile, however, the code must be modified to AssetImage("assets/images/selected_image"). This does not impact the appearance of the image on Web, but it is inconsistent behaviour that can lead to some elusive bugs.

Logs

Unable to load asset: images/selected_image

Hi @kchatr,  Thanks for filing the issue I am able to reproduce it an image asset located at assets/images/foo.png can be accessed on the
Web (works both ways)
AssetImage('images/foo.png"); /// This should not work to maintain consistency with mobile platforms
AssetImage('assets/images/foo.png");

Mobile (Android/IOS)
AssetImage('assets/images/foo.png");

The issue is reproducible on both stable and master.

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on macOS 11.4 20F71 darwin-arm, locale en-GB)
    • Flutter version 2.2.3 at /Users/mahesh/Documents/flutter
    • Framework revision f4abaa0735 (3 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.1.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 57.0.5
    • Dart plugin version 211.7233

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

[✓] Connected device (3 available)
    • iPhone 12 Pro Max (mobile) • 6A434744-B348-4FB3-AB10-131292174456 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-4 (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS
      11.4 20F71 darwin-arm
    • Chrome (web)               • chrome                               • web-javascript • Google
      Chrome 91.0.4472.164

• No issues found!

[✓] Flutter (Channel master, 2.4.0-5.0.pre.87, on macOS 11.4 20F71 darwin-arm, locale en-GB)
    • Flutter version 2.4.0-5.0.pre.87 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision e084e9d946 (2 days ago), 2021-07-17 12:00:21 -0700
    • Engine revision 14a9e9a50e
    • Dart version 2.14.0 (build 2.14.0-321.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.1.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 57.0.5
    • Dart plugin version 211.7233

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

[✓] Connected device (3 available)
    • iPhone 12 Pro Max (mobile) • 6A434744-B348-4FB3-AB10-131292174456 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-4
      (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS 11.4 20F71 darwin-arm
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome 91.0.4472.114

• No issues found!

@yjbanov Any reason that the web needs to allow both ways?

The current behavior seems to date back to the very beginnings of web support in Flutter and we never seriously re-evaluated it.
On a sample app with the following asset structure:
images/foo.png
assets/images2/bar.png

The following in pubspec.yaml:
flutter:
  uses-material-design: true

  assets:
    - images/foo.png
    - assets/images2/bar.png

And the following code:
...
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Image.asset('images/foo.png'),
            Image.asset('assets/images/foo.png'),
            Image.asset('images2/bar.png'),
            Image.asset('assets/images2/bar.png'),
          ],
        ),
...

I see the following behaviors between flutter run and python3 -m http.server 8000:

Path
flutter run
python3

images/foo.png
Loads
Loads

assets/images/foo.png
Fails
Fails

images2/bar.png
Loads
Fails

assets/images2/bar.png
Loads
Loads

Other than the case with the PathUrlStrategy we do not require any special server configuration to make Flutter work. A plain file server (or hosting services such as Firebase) should work out of the box. So consistency between the two is desirable.
I think what may be going on here is the fact that we have two "root" directories both being sources of assets:

The package root that may contain assets specified by pubspec.yaml.
The web/ sub-directory that also serves static web-specific files, such as index.html.

When the server receives a request, without looking into pubspec.yaml, it doesn't know if the request is for a web/ asset of a pubspec asset, so it tries both.
A solution might be to have flutter run always serve from build/web directory, just like a plain file server would. However, I'm not sure if the build/ directory is populated with assets during debug builds.
@jonahwilliams may have some context on this.

I encountered the same issue, and am solving this by simply adding a step to my GitHub Actions:
- name: Fix asset directory contents      run: |        mv build/web/assets/assets/* build/web/assets        rmdir build/web/assets/assets
But this should not be necessary. Instead, the bundling should put the files into the right directory from the start.


Patch version is always set to 1

Seems that all released versions have the patched version set to 1 instead of the number of commits on the default branch.

Verified on the latest release:
$ podman run quay.io/enterprise-contract/ec-cli:snapshot version
Trying to pull quay.io/enterprise-contract/ec-cli:snapshot...
Getting image source signatures
Copying blob 6ae19ef0c28b skipped: already exists
Copying blob 130b059cdb64 done   |
Copying blob caff82f63312 done   |
Copying blob 14b65f7f6b94 done   |
Copying blob 49108d4ea228 done   |
Copying config 848c1e3282 done   |
Writing manifest to image destination
Version            v0.1.2161-d6b54a8
Source ID          d6b54a8280fec5b8b9909c4a9cfc1c1e69d8f499
Change date        2023-10-23 13:24:17 +0000 UTC (1 hour ago)
ECC                v0.0.0-20230928144500-ee994416688e
OPA                v0.57.1
Conftest           v0.46.0
Cosign             N/A
Sigstore           v1.7.4
Rekor              v1.2.2
Tekton Pipeline    v0.51.0
Kubernetes Client  v0.28.2


The requested URL /callback was not found on this server.

Hello,
I ran the sample github.php app and get the following error:

Is there something we need to configure in the github.php file in order to account for the /callback
Environment:
Ubuntu 16.04 server with LAMP stack, configured with self-signed cert.
Thanks

Sorry about this! I realized the screenshot in the book is misleading.
When you create the app on GitHub, you'll need to set the callback URL to http://<IP_ADDRESS>/index.php (if you're following the instructions from the book) or http://<IP_ADDRESS>/github.php (if you're using the code in this repo).
The screenshot shows https://example-app.com/callback as the callback URL since that is the URL I used when testing the code.

Thanks for the pointer.  I changed the callback URL to /github.php and the example works as advertise now. :)


ul.nav like ul.tabs without JavaScript

Please add ul.nav like style of up.tabs without tab content.
Like bootstrap's ul.nav.nav-tabs

The tab animation isn't possible without javascript.


possible attack if SHA-256 is reversible

Let's suppose the attacker can read the keyfile, and he wants to compute the password without an access to the U2F token. Let's suppose that the attacker can compute the inverse of any SHA-256 hash (FYI this is not feasible as of 2017-11-30).
The keyfile contains sha256(public_key_x) as keyhash. The attacker reads it, reverses the SHA-256, and computes public_key_y from public_key_x (see the eckr.pointsOnCurve method how), and thus he gets the password.
From this we learn that if SHA-256 is easy to reverse, then security of this crypto is also easy to break.
Swapping the roles of public_key_x and public_key_y wouldn't be much better either. This way the keyfile would contain sha256(public_key_y), from which the attacker can get public_key_x, but he still faces the problem of modular cubic root to compute public_key_x from public_key_y. Computing modular cubic root is not much slower than modular square root (https://math.stackexchange.com/q/673418/5758), so the attacker can quickly get the password this way.
I don't have an improvement suggestion which avoids hashing altogether.
My improvement suggestions are any of the following:

The keyhash stored in the keyfile should be sha512(public_key_x + public_key_y), and the password should be public_key_x + public_key_y (64 bytes, but has only about 32 bytes of entropy).

The keyhash stored in the keyfile should be hmac_sha256(public_key_x, public_key_y), and the password should be public_key_x (32 bytes, almost full entropy).

Yes, thank you!


Use of SetQueryNotification function

I'd like to use MSSQL query notification feature, but I can not find any example of how to get these notification using go-mssqldb. README file says that query notifications are supported and I can see SetQueryNotification in source code. But how can it be used with database/sql package?

Take a look at this pull request: https://github.com/denisenkom/go-mssqldb/pull/90


Update quickstart,leave section and delete section for user guide

Resolves #156
For quickstart:

Add image
Specify product version
Provide java installation instructions
Provide instructions on how to check java version (for Mac Users need to use open JDK Zulu)
Update example

For leave section:

Make style and tone of the document more friendly (make it more personal)
Add descriptions and use examples for different segments of leave management
Add screenshots of the UI for delete-leave and add-leave
Add more clarity and depth to the features, including tables for parameters which shows whether they are optional or not.
Fix hyperlinks for leave section

Also resolves #169 from PED

Codecov Report

Merging #221 (04873f1) into master (095dac5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #221   +/-   ##
=========================================
  Coverage     56.83%   56.83%
  Complexity      499      499
=========================================
  Files            93       93
  Lines          2129     2129
  Branches        254      254
=========================================
  Hits           1210     1210
  Misses          849      849
  Partials         70       70

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


add "show now()" or "select now()"  DML

Discussed in https://github.com/apache/iotdb/discussions/3829

Originally posted by jiangxuewusheng August 25, 2021
iotdb-cli add "show now()" or "select now()" DML.

I want to join IoTDB community, I want to implement this function. Thanks


install Tensorflow

I have installed  tensorflow using conda and it did not show any error, but when I tried to check if the installation works properly it shows the following error:
no module named tensorflow

How did you install Tensorflow?  The rpi channel does not contain a tensorflow package.

Firstly I searched for It using the following command:
anaconda search -t conda tensorflow
then It displayed for me a list of the available packages, I choose the one which is suitable for Raspberry pi 3 platform, then I Run the following command:
anaconda show
to guide me how to install it
conda install --channel <package name/ channel >.
And to make sure it was installed I typed
conda list
and tensorflow was appearing in the list, however; when I try to use import tensorflow it showed me an error. Then I realized that there is something wrong, so I installed using pip.
So, maybe using "conda list" to check whether the package is installed or not is not enough.
Thank you so much for your replay.

Currently getting tensorflow via conda for ARM devices is challenging. I would recommend you following next steps

Create Python 3.5 environment as conda create -n YOUR_NAME python=3.5
Download suitable for you version from  https://github.com/lhelontra/tensorflow-on-arm/releases
Install it via pip in your environment

Tensorflow is officially supported for rpi's now. Installing should be a easy as pip install tensorflow
https://medium.com/tensorflow/tensorflow-1-9-officially-supports-the-raspberry-pi-b91669b0aa0
https://www.tensorflow.org/install/install_raspbian


bug: Adding repos to insight page crashes application

As I remember while I was working on this, this issue is there in the frontend as well.
And I just tested it now, the "Add to Page" button seems to crash the application on both Edit and Add Insights page. At least this is what is happening with me.
On the edit page, the PR count still comes out to be zero when we remove it and add it back.
pr-count-zero-insight-forms.webm
Originally posted by @ThePiyushAggarwal in https://github.com/open-sauced/app/issues/1620#issuecomment-1726013738

I am seeing the same crash with the above flow. The console throws:
index.js:103 TypeError: Cannot read properties of undefined (reading 'full_name')
    at 962-84cdac9af7cc3a7c.js:1:37475
    at Array.map (<anonymous>)
    at es (962-84cdac9af7cc3a7c.js:1:37443)
    at ab (framework-aec4381329cec0e4.js:9:60911)
    at ud (framework-aec4381329cec0e4.js:9:72797)
    at i (framework-aec4381329cec0e4.js:9:120352)
    at oO (framework-aec4381329cec0e4.js:9:99104)
    at framework-aec4381329cec0e4.js:9:98971
    at oF (framework-aec4381329cec0e4.js:9:98978)
    at ox (framework-aec4381329cec0e4.js:9:95730)

Some more info here: it looks to only happen if you click the "Add to page" on the default repos before the suggested repos has hot reloaded. Here's a demonstration:

Login (after clearing all session cached data)
Go to edit page
Quickly click the default repo
Crash!
Go back, edit insight page again
Wait for hot refresh of suggested pages
Works ok!

https://github.com/open-sauced/app/assets/23109390/ef3182e9-873f-41a9-9ea8-8843016659fe

@brandonroberts is this was done in production. Is there a way to get these errors in Sentry as well?


Problem with Protobuf on IOS

When I build in IOS using Unity and run then I got this error log
The type initializer for 'com.nct.nplay.proto.SystemMessage' threw an exception.
(Filename: /Users/builduser/buildslave/unity/build/artifacts/generated/common/runtime/UnityEngineDebugBindings.gen.cpp Line: 37)
System.TypeInitializationException: The type initializer for 'com.nct.nplay.proto.gsystem' threw an exception. ---> System.NullReferenceException: A null value was found where an object instance was required.
at Google.ProtocolBuffers.UnknownFieldSet+Builder.MergeFieldFrom (ICodedInputStream input, Google.ProtocolBuffers.ExtensionRegistry extensionRegistry, IBuilder builder, UInt32 tag, System.String fieldName) [0x00000] in :0
at Google.ProtocolBuffers.DescriptorProtos.FileOptions+Builder.MergeFrom (ICodedInputStream input, Google.ProtocolBuffers.ExtensionRegistry extensionRegistry) [0x00000] in :0
at Google.ProtocolBuffers.CodedInputStream.ReadMessage (IBuilderLite builder, Google.ProtocolBuffers.ExtensionRegistry extensionRegistry) [0x00000] in :0
at Google.ProtocolBuffers.DescriptorProtos.FileDescriptorProto+Builder.MergeFrom (ICodedInputStream input, Google.ProtocolBuffers.ExtensionRegistry extensionRegistry) [0x00000] in :0
at Google.ProtocolBuffers.AbstractBuilderLite`2[TMessage,TBuilder].MergeFrom (System.Byte[] data) [0x00000] in :0
at Google.ProtocolBuffers.DescriptorProtos.FileDescriptorProto.ParseFrom (System.Byte[] data) [0x00000] in :0
at Google.ProtocolBuffers.Descriptors.FileDescriptor.InternalBuildGeneratedFileFrom (System.Byte[] descriptorData, Google.ProtocolBuffers.Descriptors.FileDescriptor[] dependencies, Google.ProtocolBuffers.Descriptors.InternalDescriptorAssigner descriptorAssigner) [0x00000] in :0
at com.nct.nplay.proto.SystemMessage..cctor () [0x00000] in :0
at OutSystemDataProcessStream.Initialize () [0x00000] in :0
--- End of inner exception stack trace ---
at com.nct.nplay.proto.SystemMessage..cctor () [0x00000] in :0
at OutSystemDataProcessStream.Initialize () [0x00000] in :0
Does anyone know what happen :( and how to fix it ? ( currently it can run well on android )
Thanks .

Currently I use version <IP_ADDRESS>5

I'm afraid we've seen various issues with Unity, some of which have been due to Unity runtime errors, and none of which are easily reproducible for someone who doesn't have Unity :(
If you're able to reproduce this with a desktop .NET console app (and sample data file) I'd be very happy to look into it - although I don't know when I'd make another release. (I'm more focused on the proto3 code, which doesn't support Unity...)


Don't trim URL if no hash fragments are set

Summary
The Problem
When tokenize() is called on a Venmo client to initiate the Venmo payment flow but the user exits the app immediately without any interaction, no response data gets returned to the browser, so no URL fragments get set.
The visibility change listener then executes _processResults() as normal, without knowing nothing was passed back from the app. This in turn executes this function:
if (typeof global.history.replaceState === 'function') {
  history.pushState({}, '', global.location.href.slice(0, global.location.href.indexOf('#')));
}

Because no fragments were set, the character "#" is not present in the URL, so indexOf("#") will return -1. Thus the above code resolves to:
if (typeof global.history.replaceState === 'function') {
  history.pushState({}, '', global.location.href.slice(0, -1));
}

Calling slice(0, -1) trims the last character from the URL, and adds an invalid entry to the browser history.
This issue was experienced on on iPhone XS running iOS 12.0.1 with Venmo v7.24.0.
The Fix
Pretty simple - don't attempt to remove the fragments from the URL if they aren't present:
if (typeof global.history.replaceState === 'function' && global.location.href.indexOf('#') !== -1) {
  history.pushState({}, '', global.location.href.slice(0, global.location.href.indexOf('#')));
}

Checklist

[ ] Added a changelog entry

Thanks, we'll add this in the next release.


Authentication Error

06-19 12:47:40.379 4986-4986/com.firebase.uidemo E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.firebase.uidemo, PID: 4986
java.lang.RuntimeException: Unable to start activity ComponentInfo{com.firebase.uidemo/com.firebase.uidemo.auth.AuthUiActivity}: java.lang.IllegalStateException: Default FirebaseApp is not initialized in this process com.firebase.uidemo. Make sure to call FirebaseApp.initializeApp(Context) first.
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2572)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2654)
at android.app.ActivityThread.-wrap11(ActivityThread.java)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1488)
at android.os.Handler.dispatchMessage(Handler.java:111)
at android.os.Looper.loop(Looper.java:207)
at android.app.ActivityThread.main(ActivityThread.java:5728)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:789)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:679)
Caused by: java.lang.IllegalStateException: Default FirebaseApp is not initialized in this process com.firebase.uidemo. Make sure to call FirebaseApp.initializeApp(Context) first.
Why Am I not able to run the application? Getting the error as shown above.

Like with any Firebase app, you have to add a google-services.json file associated with a Firebase project. Here's the relevant link in the documentation.

@TheCraftKid Have added it. Then too Am getting the same error.

@Soumya7391 are you running the sample app without any modifications?

@samtstern Yeah.. Din't do any modification. As said in the documentation, I created a firebase proj and integrated.. tats all...

@Soumya7391
The same problem, phone auth doesn't work. I've already added google-services.json
FATAL EXCEPTION: main
```
Process: com.lucky.notewidget, PID: 30623
java.lang.IllegalStateException: Default FirebaseApp is not initialized in this process com.lucky.notewidget. Make sure to call FirebaseApp.initializeApp(Context) first.
at com.google.firebase.FirebaseApp.getInstance(Unknown Source)
at com.firebase.ui.auth.AuthUI.getInstance(AuthUI.java:138)

@Soumya7391 are you still experiencing this issue?  I'm sorry I forgot to respond.  If so, can you try installing the app from the command line with ./gradlew clean :app:installDebug rather than from Android Studio?

@samtstern this issue can probably be closed now since we haven't received a response.


Translate admonition keyword

Hi, Docusaurus maintainer here.
As we now support i18n, we need to be able to localize the keywords.
:::caution

Must lead to "Attention" for the fr locale.
We don't necessarily want Docusaurus users to add an explicit title, nor to localize the admonition keyword (ie not :::attention), just being able to localize the default title.
See also https://github.com/facebook/docusaurus/issues/4542#issuecomment-832898463
Was also wondering if you'd be interested to move this package to a new docusaurus org, as I think it's only used with Docusaurus (according to npm stats) and that would help for maintenance.

I'd definitely be willing to transfer this repo to the docusaurus org. I unfortunately don't have much time to do OSS stuff right now due to classes, so that would be a much better arrangement!

Thanks, FYI we are likely to create a docusaurus org or at least a separate community repo, will let you know when that happens

Hi!
Has anybody came up with solution on how to properly translate admonition title now?

So far, you could use an explicit title:
:::caution Some title
Test test test
:::

The rendering result is:

For Docusaurus users, we have moved to a different admonition setup, always using our own theme component instead of hardcoded JSX markup, which gives us more flexibility: https://github.com/facebook/docusaurus/pull/7152
This allows translating admonitions in our theme component directly: https://github.com/facebook/docusaurus/pull/7556
Going to close, assuming most users reporting here are using Docusaurus


[Completion] Update type attribute completions

Update type attribute completion to support @isolated(any), @noDerivative, @preconcurrency, @retroactive, @Sendable, and @unchecked.
rdar://130741006
rdar://130288443

@swift-ci please test

@swift-ci please test


fix transparent host header #916

Hey Pedro, thanks for working on this.
I'm actually not sure this is the best way to go about fixing this. I don't really like the idea of introducing a "transparent" mode (the keyword is just a name for the preset, all it does it set headers). I feel like there is an error underlying the logic we already have in place, which should be fixed, instead of a flag to work around it.
Also if you could allow ~24-48 hours on average so one or two other collaborators have a chance to review a pull request, that would be helpful. :) I'll try to remember to add that to our guidelines for future reference.

Yes, I actually thought that this was not the best and final solution for this problem. But I think it's a solution, and I actually like the idea of solving the problem first (if there is a simple solution) then look for a complete solution. I haven't refactored everything because I think would be better to be someone that knows the best this directive to refactor it.
Sorry for not waiting that amount of time to merge it. I thought wouldn't be a great deal because the change is not so big. But I will wait more next time.
And sorry about the amount of commits, I forgot to squash it. (I'll pay more attention later)
Thanks for the feedback :)

Sounds good! I usually like to let each PR get reviewed by at least one other dev before it gets merged. Sometimes waiting a day or two is needed, but at least this change is small and non-destructive.
I think @evermax was looking at this issue too, maybe between you and him you can find the error in the existing code and test against it!


Fix typo in changelog

Thanks to @lakshmi-kannan for catching it.

Yeah, two typos in one line - fat fingers :P

Codecov Report

Merging #3221 into master will increase coverage by 2.31%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3221      +/-   ##
==========================================
+ Coverage   75.42%   77.73%   +2.31%
==========================================
  Files         433      433
  Lines       22414    22414
==========================================
+ Hits        16904    17422     +518
+ Misses       5510     4992     -518

Impacted Files
Coverage Δ

st2common/st2common/services/triggerwatcher.py
88.73% <ø> (-4.23%)
:x:

st2common/st2common/bootstrap/sensorsregistrar.py
81.55% <ø> (-3.88%)
:x:

st2common/st2common/bootstrap/configsregistrar.py
82.93% <ø> (-3.66%)
:x:

st2common/st2common/rbac/decorators.py
84.38% <ø> (-3.13%)
:x:

st2common/st2common/bootstrap/rulesregistrar.py
76.11% <ø> (-1.77%)
:x:

st2common/st2common/bootstrap/triggersregistrar.py
83.33% <ø> (-1.19%)
:x:

st2common/st2common/rbac/utils.py
94.37% <ø> (-0.7%)
:x:

st2common/st2common/rbac/resolvers.py
88.62% <ø> (-0.53%)
:x:

st2common/st2common/models/db/init.py
93.66% <ø> (-0.49%)
:x:

st2api/st2api/controllers/v1/actionexecutions.py
89.57% <ø> (-0.43%)
:x:

... and 27 more

Continue to review full report at Codecov.

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


WFLY-5945/WFLY-5946 immutable-entity and naturalid cache configurations are not started by the JPA subsystem

https://issues.jboss.org/browse/WFLY-5945
https://issues.jboss.org/browse/WFLY-5946

Does pending-puts need to be included? AIUI the perf guys are overriding the pending-puts config to ensure SimpleCache is used.

@bstansberry No, because the pending-puts cache configuration is effectively hard-coded and is not configurable.  I'm not sure what the perf guys are doing, but that isn't going to work according to how the hibernate region factory is currently implemented. See: https://github.com/hibernate/hibernate-orm/blob/master/hibernate-infinispan/src/main/java/org/hibernate/cache/infinispan/InfinispanRegionFactory.java#L224
https://github.com/hibernate/hibernate-orm/blob/master/hibernate-infinispan/src/main/java/org/hibernate/cache/infinispan/InfinispanRegionFactory.java#L392

@pferraro pending puts was made configurable in the 5.0 branch

@pferraro see https://github.com/hibernate/hibernate-orm/pull/1145

@pferraro Yes, pending-puts cache configuration needs to be inserted into the cache manager, otherwise it will use the default configuration https://github.com/hibernate/hibernate-orm/blob/5.0/hibernate-infinispan/src/main/java/org/hibernate/cache/infinispan/InfinispanRegionFactory.java#L393 . And WF should always define this, there's no point in using the ORM's default (which is there because of ispn 7.x).
Not understanding the subsystem configuration too much, I wonder why these dependencies need to be defined explicitly - why aren't all cache configurations in WF config defined in given cache manager?

@rvansa because cache configurations are installed on demand.  If nothing demands it to start, it is never installed.

WFLY-5953 is for pending-puts

@pferraro So, if I define in persistence.xml that class foo.Bar should use cache configuration foos, and define cache foos in WF config, it will still rather use default cache configuration (and I mean the default cache set from ISPN perspective) as ORM will request cache foos and as it has no special configuration (because nobody demanded foos to be installed)?

@scottmarlow I had already opened https://issues.jboss.org/browse/WFLY-5951

@rvansa Is there a particular reason why the pending-puts cache name is hardcoded? The other cache configuration names are configurable.

@pferraro I have a local change to add pending-puts (its a little funky since Hibernate doesn't offer the same constants for pending-puts).   I also added some WF trace logging that is helpful for finding the cause, which is that we are only adding persistence unit service dependencies for entity/local-query/pending-puts.  Since we don't add dependencies on the immutable-entity/timestamps services, they don't start.

In HibernateSecondLevelCache, I think that instead of:
cacheSettings.put(IMMUTABLE_ENTITY, mutableProperties.getProperty(InfinispanRegionFactory.IMMUTABLE_ENTITY_CACHE_RESOURCE_PROP, InfinispanRegionFactory.DEF_ENTITY_RESOURCE));
We need:
cacheSettings.put(IMMUTABLE_ENTITY, mutableProperties.getProperty(InfinispanRegionFactory.IMMUTABLE_ENTITY_CACHE_RESOURCE_PROP, InfinispanRegionFactory.DEF_IMMUTABLE_ENTITY_RESOURCE));

@scottmarlow I'm anticipating https://github.com/hibernate/hibernate-orm/pull/1208
The immutable entity cache configuration should default to the entity cache configuration if no explicit immutable-entity cache configuration was defined.

@pferraro Are you asking why is the name or default configuration hardcoded? Both were hard-coded historically, and I just did not change that (haven't seen any reason). In the past, there was only single pending-puts cache with hard-coded configuration. Now this is the configuration used in the same way as entity cache; the configuration is copied to a cache created per-entitiy, and this per-entity cache has name entity-name-pending-puts.
The configuration was hard-coded in the past, too, but since we've allowed to configure this in micro-release, I could not just add the default configuration into the default configuration file, because it would break with user-provided configuration file. So, the hard-coded configuration is inserted when it's not defined there.
Anyway, about the custom cache configuration - can the dependency be added anywhere in config?

https://github.com/scottmarlow/wildfly/tree/2lc_setup has a hack for pending-puts.  @pferraro let me know if you have something different in mind.  We also need to add trace logging, so we don't have to debug next time.

with my local testing, my above change is not right yet.  If I deploy the test (quickstart) app with default WildFly settings, I get missing dependency deployment errors:
"jboss-as-hibernate4Test.war#primary" is missing [jboss.infinispan.hibernate.pending-puts.config]"
If I add the following to the cache-contain for "hibernate", the error goes away:

https://github.com/wildfly/wildfly/pull/8549


chore: fix typo

SUMMARY
Noticed this while reading code
TEST PLAN
CI
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

to: @pkdotson

Codecov Report

Merging #11772 (3c37f24) into master (fbe4a66) will decrease coverage by 2.95%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11772      +/-   ##
==========================================
- Coverage   66.09%   63.14%   -2.96%
==========================================
  Files         903      903
  Lines       43719    43711       -8
  Branches     4199     4199
==========================================
- Hits        28898    27601    -1297
- Misses      14701    15930    +1229
- Partials      120      180      +60

Flag
Coverage Δ

cypress
?

javascript
62.84% <ø> (ø)

python
63.32% <ø> (-0.16%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset-frontend/src/views/CRUD/utils.tsx
48.78% <ø> (-1.22%)
:arrow_down:

superset-frontend/src/SqlLab/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/setup/setupColors.js
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/chart/ChartContainer.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/setup/setupFormatters.js
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/reducers/index.js
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 143 more

Continue to review full report at Codecov.

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

yeah, i was doing this from the web editor, so didn't want to dig in that much. But if it's already imported in the file, then i can add i18n too

i18n added!


Unknown host repository.cloudera.com: Name or service not known

[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  12:10 min
[INFO] Finished at: 2020-03-17T07:43:09Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project doris-fe: Could not resolve dependencies for project org.apache:doris-fe:jar:3.4.0: Failed to collect dependencies at net.sourcef
orge.czt.dev:java-cup:jar:0.11-a-czt02-cdh: Failed to read artifact descriptor for net.sourceforge.czt.dev:java-cup:jar:0.11-a-czt02-cdh: Could not transfer artifact net.sourceforge.czt.dev:java-cup:pom:0.11-a-czt02-cdh from/to cloudera-thirdparty (https://repository.cloudera.com/content/repositories/third-party/): Transfer failed for https://repository.cloudera.com/content/repositories/third-party/net/sourceforge/czt/dev/java-cup/0.11-a-czt02-cdh/java-cup-0.11-a-czt02-cdh.pom: Unknown host repository.cloudera.com: Name or service not known -> [Help 1][ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

@ighack
Have you built Doris successfully?

@imay
yes

@ighack
OK, I will close this issue, could you please describe how you solved it?

I just build it again,it works
I don't know why.

use this url:

cloudera
https://repository.cloudera.com/artifactory/cloudera-repos/


How to set MD004 to Sublist?

Hi, great tool.
I'm having trouble settings sublists.
Here is what I am trying:
{
  "MD004": { "style": "sublist" }
}

I would expect this to allow for the following:
*  Level 1
  + Level 2
    - Level 3

But this gets changed to the following on save:
-  Level 1
  - Level 2
    - Level 3

What am I doing wrong?

That configuration should not report any violations via markdownlint. You can confirm that here: ￼https://dlaa.me/markdownlint/#%25m%23%20Issue%20313%0A%0A*%20Level%201%0A%20%20%2B%20Level%202%0A%20%20%20%20-%20Level%203%0A%0A%3C!--%20markdownlint-configure-file%20%7B%20%22MD004%22%3A%20%7B%20%22style%22%3A%20%22sublist%22%20%7D%20%7D%20--%3E%0A
What editor are you using and what tool, extension, or plug-in is applying those edits when you save the document?￼

I see. I think Prettier is overriding things. Thanks.

I see. I think Prettier is overriding things. Thanks.

How did you fix it, with prettier rc or eslintrc?


scheduler should re-evaluate pending tasks after any task removal

Fix https://github.com/moby/moby/issues/32999.
Deleting tasks may free up node resources. Pending tasks should be evaluated.
Signed-off-by: Dong Chen<EMAIL_ADDRESS>

LGTM


Corrected some grammatical mistakes

This PR is in reference with the issue #19. I have made the required changes in the README.md and Contributing.md . Waiting for the PR to be reviewed. Thank you!

Simple contribution though merged


what does this mean“ pid operator: NotIn”

    selectors:
    - matchPIDs:
    # Enum=In;NotIn
      - operator: NotIn

        followForks: true

        isNamespacePID: true
        values:
        - 1

i got it , but why not "in" but "notin" in operator fied?

i got it , but why not "in" but "notin" in operator fied?

In pids -> match everything in the pid list, and filter everything else
NotIn pids -> filter everything in the pid list, and match everything else

Looks like question was answered. Feel free to open if more info needed.


BoxCLI 3.2.0 Auth Error: Please check the 'exp claim'

When installing the BoxCLI 3.2.0 upgrade on Windows, I am now receiving an authentication error when attempting to run a box command. Running box users:get me gives the following error:
Auth Error: Please check the 'exp' claim. Should be an integer [400 bad request] invalid grant - Please check the 'exp' claim. Should be an integer.
This has happened with both the x64 and x86 versions of the BoxCLI.exe. Machine is a Windows 10 box.

Hi @jkhancock31
Thank you for reaching out to us. Next time, please follow the issue template to make it easier for both parties to resolve the issue.
It looks like your local time may be out of sync with the Unix time. Please check if it synchronized it and try again.


fix loading opengl es3+ shaders with new shaderc changes

The changes introduced by #2317 completely broke OpenGL ES 3.0 new shaders at runtime.
(resulting in: must explicitly specify all locations when using multiple fragment outputs)
Not sure why was merged in such state, Anyway this PR attempts to hot fix the issue.
Basically the new shaderc changes inject at shader compile time their own logic to replace gl_FragColor already, with this check we make sure to remain backward compatible yet ignore newly processed shaders.

@BlockoS Could you figure out why this changed?

@BlockoS Could you figure out why this changed?

To add more context, I produced this bug using WebGL 2 and BGFX_CONFIG_RENDERER_OPENGLES_MIN_VERSION=30
But I think any OpenGL ES 3 app would trigger it.

To add more context, I produced this bug using WebGL 2 and BGFX_CONFIG_RENDERER_OPENGLES_MIN_VERSION=30
But I think any OpenGL ES 3 app would trigger it.

It seems everything works with WebGL2/GLES3.


Password Reset - no auto-generated password

It seems as if the password functionality is no longer auto-generating passwords but instead you can define the passwords on your own. Can you update this page?

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

ID: 9922a978-39e8-80c1-07b0-58d3cef4e127
Version Independent ID: fcbddc27-fdb0-4c1e-bb0c-168b24152c69
Content: Reset a user's password - Azure Active Directory
Content Source: articles/active-directory/fundamentals/active-directory-users-reset-password-azure-portal.md
Service: active-directory
Sub-service: fundamentals
GitHub Login: @msaburnley
Microsoft Alias: ajburnle

@sebhe
Thanks for your feedback! We will investigate and update as appropriate.

Hi @sebhe ,
Can you clarify what you mean by this? I just tested on my tenant and the temporary password was auto-generated.

Hi,
the windows looks totally different for us, see below. Is it maybe dependent on the source of the user account? Ours are mainly accounts that are synced from on-premise AD.

Yes, it seems to be different for password writeback with on-premises users.
@msaburnley is this something that could get added to the documentation?

@MarileeTurscak-MSFT yes, I'll work on it!

@sebhe a note with this information has been added to the content and it has been published. You can see the note here: https://docs.microsoft.com/en-us/azure/active-directory/fundamentals/active-directory-users-reset-password-azure-portal#to-reset-a-password

#please-close


Vue 3.3.3, defineProps/ defineEmits could not resolve default exported types

Vue version
3.3.3
Link to minimal reproduction
https://github.com/tragid/vue-default-export-error
Steps to reproduce

Create props interface file: TestComponentProps.ts
Create default export from file

export default interface TestComponentProps {
  msg: string;
}

import type within component TestComponent.vue

import type TestComponentProps from './TestComponentProps';

const props = defineProps<TestComponentProps>();

Vue throws error on startup.
[@vue/compiler-sfc] Unresolvable type reference or unsupported built-in utility type

What is expected?
Vue can resolve defaultly imported types
What is actually happening?
Vue throws error on startup.
[@vue/compiler-sfc] Unresolvable type reference or unsupported built-in utility type
System Info
No response
Any additional comments?
Everything works, if I create export from file with no default provided
export interface TestComponentProps {
  msg: string;
}

import type { TestComponentProps } from './TestComponentProps';

const props = defineProps<TestComponentProps>();

@yyx990803
Hi! Fix helped to get rid of compile time errors.
Now having runtime errors, when importing default Props and default Emits in a row.
runtime-core.esm-bundler.js:41 [Vue warn]: Property "tag" was accessed during render but is not defined on instance.
  at <TestComponent msg="Hello World!" tag="span" onClick=fn<onClick> >
  at <App>

I have created a branch with 3.3.4 version with minimal reproduction: https://github.com/tragid/vue-default-export-error/tree/vue-3.3.4

Please open a separate issue so that it can be tracked as a different bug.

@yyx990803 I made a PR to fix it, see https://github.com/vuejs/core/pull/8365


Page theme and submit, cancel button style updated

https://github.com/pranjay-poddar/Dev-Geeks/assets/97864586/cc6e3bd1-9584-4b06-8d14-502123eb25bf
Added Theme, updated Submit and Cancel button Style and #1516

Close #1516

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@pranjay-poddar Please have a look now the conflicts resolved.


Create Guillen503×

if (password_verify($password, $hash)) {
	if (password_needs_rehash($hash, $algorithm, $options)) {
		$hash = password_hash($password, $algorithm, $options);
		/* Store new hash in db */
	}
}

Why:
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

H


Bresenham Line algorithm doesn't test all directions and fails going South East

Bresenham appears to work correctly on the Y axis but not on the X axis. I'm able to increment and decrement the Y axis but the X axis fails miserably.
Example:
let line = BresenhamLine::new(Point::new(0, 6), Point::new(6,0));
let res: Vec<Point> = line.collect();

res results with an empty vector because of if self.x >= self.x1 { check.
Is this is changed to if self.x >= self.x1 && self.y >= self.y1, with y1 being the end.y, it results in the following vector:
[ 0, 6]
[-1, 5]
[-2, 4]
[-3, 3]
[-4, 2]
[-5, 1]

This might apply to move cordial directions.

From what I can tell this bit:
    fn with_point(&self, p: Point2<i32>) -> Point2<i32> {
        println!("self.0: {}", self.0);
        match self.0 {
            0 => Point2::new(p.x, p.y),
            1 => Point2::new(p.y, p.x),
            2 => Point2::new(-p.y, p.x),
            3 => Point2::new(-p.x, p.y),
            4 => Point2::new(-p.x, -p.y),
            5 => Point2::new(-p.y, -p.x),
            6 => Point2::new(p.y, -p.x),
            7 => Point2::new(p.x, -p.y),
            _ => unreachable!(),
        }
    }

Is wrong. i.e for this particular direction which is an octant of 4, it states this: 4 => Point2::new(-p.x, -p.y), but it in fact should be 4 => Point2::new(p.x, -p.y),

I'm not sure about this one. If I modify the bresenham_line.rs example to use the coordinates from your first post, and display the resulting line numbered and as debug-print:
use bracket_geometry::prelude::*;
use crossterm::queue;
use crossterm::style::Print;
use std::io::{stdout, Write};

fn main() {
    let mut fake_console: Vec<char> = vec!['.'; 100];

    let line : Vec<Point> =  Bresenham::new(Point::new(0, 6), Point::new(6, 0)).collect();

    for (i,point) in line.iter().enumerate() {
        let idx = ((point.y * 10) + point.x) as usize;
        let character = 48 + i as u8;
        fake_console[idx] = character as char;
    }

    for y in 0..10 {
        let mut line = String::from("");
        let idx = y * 10;
        for x in 0..10 {
            line.push(fake_console[idx + x]);
        }
        line.push('\n');
        queue!(stdout(), Print(&line)).expect("Command fail");
    }
    stdout().flush().expect("Flush Fail");

    line.iter().for_each(|p| println!("{:?}", p));
}

I get the following:
..........
.....5....
....4.....
...3......
..2.......
.1........
0.........
..........
..........
..........
Point { x: 0, y: 6 }
Point { x: 1, y: 5 }
Point { x: 2, y: 4 }
Point { x: 3, y: 3 }
Point { x: 4, y: 2 }
Point { x: 5, y: 1 }

If I add a test to line_bresenham.rs, it passes:
#[test]
fn test_issue135_line() {
    let line = Bresenham::new(Point::new(0, 6), Point::new(6,0));
    let res: Vec<Point> = line.collect();
    assert!(res.len() == 6);
    res.iter().for_each(|p| {
        assert!(p.x >= 0);
        assert!(p.x < 7);
        assert!(p.y >= 0);
        assert!(p.y < 7);
    });
}

(Those are without changing the line code)
I see you are using BresenhamLine as a type, but that doesn't appear in the bracket-geometry library?

I added some tests and a new "bresenham sweep" example that goes through all of the angles from 0..360 and it doesn't seem to fail?

What am I missing?

Weird, we spoke about this a little bit privately. Its good to have the tests in. I took it out of context and may have missed something... Using bits and pieces of bracket-lib but admittedly should've tried it in the crate first. It should've been a direct copy though. I'll double check again when I can get around to it again. For now, can close this issue but thanks for putting in the time to add more tests.


Wagtail 6.0

Hi @katdom13 The tests seem to be failing on this one, just in case you hadn't see it.

Hey @nickmoreton ,
The failures are consistently on Django 3.2 as seen on this error: https://github.com/torchbox-forks/wagtail-nhsuk-frontend/actions/runs/8110195302/job/22167030630
I suspect I have to add a value for the body Streamfield somewhere on https://github.com/torchbox-forks/wagtail-nhsuk-frontend/blob/master/testapp/home/migrations/0002_create_homepage.py, but I just don't know how or where.
A current workaround would be to remove support for Django 3.2 (as it's reaching its EOL in 1 month anyway and the only client that uses this package is GOSH).
What do you think? Any better advise for this one?
Thanks!

I suspect I have to add a value for the body Streamfield somewhere on https://github.com/torchbox-forks/wagtail-nhsuk-frontend/blob/master/testapp/home/migrations/0002_create_homepage.py, but I just don't know how or where.

In this line https://github.com/torchbox-forks/wagtail-nhsuk-frontend/blob/a11c6d80910bc2f8dff4be346bcec7c07f658a7d/testapp/home/migrations/0003_test_pages.py#L34 would it work if you changed this
from
('body', wagtail_blocks.RichTextBlock(required=True))])),
to
('body', wagtail_blocks.RichTextBlock(required=False))])),


Access authorization DSL

What does this PR do?

Implements the authorize_access_for controller class method, which generates the access rules to a specified resource.

Will deal with code issues later


Parallel test with pytest-xdist

Currently this PR creates 2 workers, each with 2 GPUs

pytest test --verbose -s --dist load --tx popen//env:CUDA_VISIBLE_DEVICES=0,1 --tx popen//env:CUDA_VISIBLE_DEVICES=2,3

I think only FSDP tests require 2 GPUs
https://github.com/pytorch/ao/blob/4f53882c6dfaf42daad8ca01595f44fe6d51e69c/test/dtypes/test_nf4.py#L484-L494
https://github.com/pytorch/ao/blob/4f53882c6dfaf42daad8ca01595f44fe6d51e69c/test/prototype/test_low_bit_optim.py#L161-L168
Thus, CI runtime can be further improved by

Mark these tests as "require 2 GPUs"
Run the remaining tests with 4 workers, each with 1 GPU
Run the 2-GPU tests in a separate step. Can either be 2 workers with 2 GPUs each, or just normal, non-parallel pytest since there are not a lot of 2-GPU tests right now.

UPDATE: parallel tests with 4 1-GPU workers + sequential multi-GPU tests is implemented now. Some problems I encountered along the way (for future reference and improvement). pytest-xdist provides multiple methods to distribute work across workers. Refer to here for the full list. I have tried the following

--dist load (send each test to next available worker):

TestSaveLoadMeta use the same filename test.pth which is problematic when multiple tests write/read the same file. I made a unique name for each test to solve it for now, though the better way is to use pytest tmp_path fixture (not compatible with unittest.TestCase).
test_quant_llm_quantize_ebits_2_mbits_2 gets a strange torch.compile warning in PyTorch 2.3 only torch/_functorch/_aot_autograd/jit_compile_runtime_wrappers.py:436] [0/48] failed to eagerly compile backwards for dynamic, suppressing in case backwards not needed. Looks harmless though.

--dist loadscope (send a TestClass to next available worker):

TestSaveLoadMeta gets a strange error - RuntimeError: Error: accessing tensor output of CUDAGraphs that has been overwritten by a subsequent forward run. Stack trace: [Could not find stack trace]. To prevent overwriting, clone the tensor outside of torch.compile() or call torch.compiler.cudagraph_mark_step_begin() before each model invocation.. No traceback. Since tests within a class are run by the same worker, there shouldn't be filename conflict test.py. Perhaps some strange interaction between how pytest-xdist sends test to the subprocess worker and CUDA graphs + torch.compile?

--dist loadfile (send each file to next available worker): this should be the least error-prone. However, there is little speedup (24min vs 35min) since test_integration.py file is huge -> load imbalance (probably a good idea to split this into smaller files anyway).

The current approach of this PR is using --dist load. To summarize

Approach
CUDA nightly CI time

pytest sequential (baseline)
35min

pytest-xdist with two 2-GPU workers (this PR, original)
21min

pytest-xdist with four 1-GPU workers + sequential multi-GPU tests (this PR, latest)
19min

I also notice that it takes 3min to pull the Docker image and another 3-4min to download and install packages. So it seems like two 2-GPU workers have pretty good scaling (7 + 14 vs 7 + 28), while four 1-GPU workers is not much of an improvement (7 + 12 vs 7 + 28). I guess the 2-GPU FSDP tests can be run in parallel too for the latest configuration.

What I had tried messing around with before was using https://docs.pytest.org/en/stable/how-to/mark.html
which is useful to mark the 2 gpu and 1 gpu tests independently, and in the github action file you can just have 2 pytest commands to run them one after the other

Yup that's what I have in mind too. Just want to make sure only those 2 tests use 2 GPUs.

We can merge this but let's keep an eye out on whether this makes debugging test failures more tricky to debug in teh future

@msaroufim I keep seeing this test failing
test/integration/test_integration.py::TestSaveLoadMeta::test_save_load_int8woqtensors

Not exactly clear what's causing it. Should we revert or try to fix it?

Revert and ask questions later


Marcosvm/travis

[x] add .travis.yml
[x] add Travis build status badge

Cool! Thanks

Closed #1 #2


cmd/go2go: please update gofmt

Now the translator rejects the following code with generic.go2:10:20: expected operand, found 'type' (and 4 more errors)
type genericStream[type T any] struct {
    prevReq  chan struct{}
    prevData chan T
    nextReq  chan struct{}
    nextData chan T
}

func newGenericStream[type T any](gs *genericStream[T]) *genericStream[T] {
    return &genericStream[T]{
        prevReq:  gs.nextReq,
        prevData: gs.nextData,
        nextReq:  make(chan struct{}),
        nextData: make(chan T),
    }
}

So I just modified like this:
type genericStream[T any] struct {
    prevReq  chan struct{}
    prevData chan T
    nextReq  chan struct{}
    nextData chan T
}

func newGenericStream[T any](gs *genericStream[T]) *genericStream[T] {
    return &genericStream[T]{
        prevReq:  gs.nextReq,
        prevData: gs.nextData,
        nextReq:  make(chan struct{}),
        nextData: make(chan T),
    }
}

Now the translator translate this code, but gofmt will insert the deleted type again.
gofmt is important because it is the only tool to format *.go2 files.
Or this might be an issue for the translator.

Thanks for this issue. I missed that. Will take care of this today or tomorrow.

This should be fixed now in the dev.go2go branch with https://golang.org/cl/250998.

@griesemer Thanks for the fix!!!


Fix description pane losing scroll position

Fixes https://github.com/Microsoft/vscode-pull-request-github/issues/543
Issue was that the webview HTML was being reset every time update was called. Now the HTML won't be reset unless the PR has changed.

Thanks for taking this on!
From my understanding of webviews, when the view becomes hidden and then visible again, the scroll position is not preserved even if the html stays the same. With this change I still see scroll position get reset when switching to a different tab and back to the webview - do you also see this?
I think instead we need to save off this state in the webview - the pendingCommentText in preview-src/index.ts is a potential example. The markdown extension in VSCode also manages scroll position in its webview

Ah, I see now this was a naive solution! This only fixes the issue listed in the issue (switching between split pane views), not the more general case of switching tabs. I'll work on a more general fix, thanks for the info!

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: malwilley sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@RMacfarlane I think this should be closer to what we want here. There wasn't an obvious spot for storing the scroll position so I added another private member to pullRequestOverview.ts but let me know if that is better kept elsewhere.

@malwilley This approach looks good! Yeah, I think adding the scroll position to pullRequestOverview makes sense. Just had a couple small comments

Thank you!


Cloud Firestore RPC throw DeadlineExceededException when called from jpackage'd MacOS app

I'm a Googler investigating a customer issue (https://github.com/firebase/firebase-admin-java/issues/499) where their calls to the Cloud Firestore RPC APIs are hanging when they package their app via jpackage on MacOS. I've been able to reproduce their issue but have no idea how to investigate since this is a much lower-level issue than I have experience with.
Here is the code to reproduce:
import com.google.auth.oauth2.GoogleCredentials;
import com.google.cloud.firestore.DocumentReference;
import com.google.cloud.firestore.Firestore;
import com.google.firebase.FirebaseApp;
import com.google.firebase.FirebaseOptions;
import com.google.firebase.cloud.FirestoreClient;
import java.io.InputStream;
import java.util.Map;

public class FirestoreJpackageRpcHang {

  public static void test(Map<String, Object> data) throws Exception {
    InputStream inputStream = FirestoreJpackageRpcHang.class.getResourceAsStream("/projecttest-firebase.json");
    FirebaseOptions options = FirebaseOptions.builder().setCredentials(GoogleCredentials.fromStream(inputStream)).build();
    FirebaseApp.initializeApp(options);
    Firestore firestore = FirestoreClient.getFirestore();
    DocumentReference docRef = firestore.collection("users").document("user007");
    docRef.set(data).get(); // <--- The call to get() times out after 50 seconds and throws an exception.
    firestore.close();
  }

}

This code runs just fine when run via the java command; however, when packaged as a standalone app with jpackage the call to ApiFuture<WriteResult>.get() blocks for about 1 minute then throws DeadlineExceededException:
[grpc-default-executor-0] WARN io.grpc.netty.shaded.io.netty.channel.DefaultChannelId - Failed to find the current process ID from ''; using a random value:<PHONE_NUMBER>
Exception in thread "main" java.util.concurrent.ExecutionException: com.google.api.gax.rpc.DeadlineExceededException: io.grpc.StatusRuntimeException: DEADLINE_EXCEEDED: deadline exceeded after 49.985643644s. [buffered_nanos=248215060, buffered_nanos=49741749475, waiting_for_connection]
        at com.google.common.util.concurrent.AbstractFuture.getDoneValue(AbstractFuture.java:564)
        at com.google.common.util.concurrent.AbstractFuture.get(AbstractFuture.java:545)
        at com.google.common.util.concurrent.FluentFuture$TrustedFuture.get(FluentFuture.java:86)
        at com.google.common.util.concurrent.ForwardingFuture.get(ForwardingFuture.java:62)
        at com.thboulay.FirestoreJpackageRpcHang.test(FirestoreJpackageRpcHang.java:20)
        at com.thboulay.JavaFirestorePackage.test2(JavaFirestorePackage.java:28)
        at com.thboulay.JavaFirestorePackage.main(JavaFirestorePackage.java:32)
Caused by: com.google.api.gax.rpc.DeadlineExceededException: io.grpc.StatusRuntimeException: DEADLINE_EXCEEDED: deadline exceeded after 49.985643644s. [buffered_nanos=248215060, buffered_nanos=49741749475, waiting_for_connection]
        at com.google.api.gax.rpc.ApiExceptionFactory.createException(ApiExceptionFactory.java:51)
        at com.google.api.gax.grpc.GrpcApiExceptionFactory.create(GrpcApiExceptionFactory.java:72)
        at com.google.api.gax.grpc.GrpcApiExceptionFactory.create(GrpcApiExceptionFactory.java:60)
        at com.google.api.gax.grpc.GrpcExceptionCallable$ExceptionTransformingFuture.onFailure(GrpcExceptionCallable.java:97)
        at com.google.api.core.ApiFutures$1.onFailure(ApiFutures.java:68)
        at com.google.common.util.concurrent.Futures$CallbackListener.run(Futures.java:1050)
        at com.google.common.util.concurrent.DirectExecutor.execute(DirectExecutor.java:30)
        at com.google.common.util.concurrent.AbstractFuture.executeListener(AbstractFuture.java:1176)
        at com.google.common.util.concurrent.AbstractFuture.complete(AbstractFuture.java:969)
        at com.google.common.util.concurrent.AbstractFuture.setException(AbstractFuture.java:760)
        at io.grpc.stub.ClientCalls$GrpcFuture.setException(ClientCalls.java:545)
        at io.grpc.stub.ClientCalls$UnaryStreamToFuture.onClose(ClientCalls.java:515)
        at io.grpc.internal.ClientCallImpl.closeObserver(ClientCallImpl.java:426)
        at io.grpc.internal.ClientCallImpl.access$500(ClientCallImpl.java:66)
        at io.grpc.internal.ClientCallImpl$1CloseInContext.runInContext(ClientCallImpl.java:416)
        at io.grpc.internal.ContextRunnable.run(ContextRunnable.java:37)
        at io.grpc.internal.SerializingExecutor.run(SerializingExecutor.java:123)
        at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source)
        at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
        at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(Unknown Source)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
        at java.base/java.lang.Thread.run(Unknown Source)
Caused by: io.grpc.StatusRuntimeException: DEADLINE_EXCEEDED: deadline exceeded after 49.985643644s. [buffered_nanos=248215060, buffered_nanos=49741749475, waiting_for_connection]
        at io.grpc.Status.asRuntimeException(Status.java:533)
        ... 12 more
Failed to launch JVM

The original reporter has created a TestApp that I used to reproduce: https://github.com/thboulay/JavaFirestorePackage
Can anyone tell me if this is a bug, and, if so, how it could be fixed and/or worked around? Thank you!

Do you find any solution ? I have same problem when package my app in windows :(

Do you find any solution ? I have same problem when package my app in windows :(

Same on linux ubuntu :(

I see https://github.com/thboulay/JavaFirestorePackage is unavailable now.

Closing this issue for issue cleanup. If somebody can provide latest reproducible, please reopen this issue with the link.


skip CDHITDUP if sample name contains PLEASE_SKIP_CDHITDUP

Description
Allow users to request skipping CDHITDUP but adding text PLEASE_SKIP_CDHITDUP to the sample name
Notes
This sets an attribute in the DAG json, which is then noticed by the pipeline, and has the effect of replacing cdhitdup with the identity function, copying inputs unchanged to outputs.
Tests
Should confirm that in staging the automatically submitted benchmark sample does run with cdhitdup, and that submitting a new sample with appropriate name runs without it.

But isn't it better than running custom branches?    Custom branches are a pain when stuff gets accidentally rerun in future.
At the very least this might be a foundation for someone to complete it into a real feature.

There is also a thing called dag_vars that's even persisted in the sample schema and pipeline_run schema.   I am not sure if that's what should be used instead.

There is also a thing called dag_vars that's even persisted in the sample schema and pipeline_run schema. I am not sure if that's what should be used instead.

Yes, that's the intended purpose of dag_vars :)

Lets please not add this type of hacks to our codebase. If the current workflow (custom branches + dag_vars) is not enough for some reason or is not working as expected, we can review it and fix it.


Documentation for CustomAuthorizationServerConfigurer

The CustomAuthorizationServerConfigurer in the documentation does not work because the password grant type is disabled when the auto configuration is turned off (AuthenticationManager is no longer provided). The documentation should show AuthenticationManager being injected into AuthorizationServerEndpointsConfigurer:
@Configuration
@EnableAuthorizationServer
public class CustomAuthorizationServerConfigurer extends AuthorizationServerConfigurerAdapter {

private final AuthenticationManager authenticationManager;

public CustomAuthorizationServerConfigurer(AuthenticationConfiguration authenticationConfiguration) throws Exception {
    this.authenticationManager = authenticationConfiguration.getAuthenticationManager();
}

@Override
public void configure(ClientDetailsServiceConfigurer clients) throws Exception{
    clients.inMemory()
            .withClient("client")
            .authorizedGrantTypes("password")
            .secret("{noop}secret")
            .scopes("all");
}

@Override
public void configure(AuthorizationServerEndpointsConfigurer endpoints) throws Exception {
    endpoints.authenticationManager(authenticationManager);
}

}

Thanks for the report, @PhazedOut. I'll add more detail to the documentation.


fix: improve RSpec icon

#351  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


fix(async queries): Remove "force" param on cached data retrieval

SUMMARY
When fetching cached data from the new /api/v1/chart/data endpoint in the async query lifecycle, the force param from the initial request was propagated to the cached data load, causing an error when force-reloading a chart's data. This PR removes the force param from the cached QueryContext data.
TEST PLAN
Enable the GLOBAL_ASYNC_QUERIES feature flag and test force-reloading a chart using the new chart data API.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #12103 (a468eb4) into master (a3be325) will decrease coverage by 3.54%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #12103      +/-   ##
==========================================
- Coverage   66.62%   63.07%   -3.55%
==========================================
  Files         962      967       +5
  Lines       47368    47515     +147
  Branches     4629     4650      +21
==========================================
- Hits        31557    29970    -1587
- Misses      15684    17359    +1675
- Partials      127      186      +59

Flag
Coverage Δ

cypress
?

javascript
62.74% <ø> (+0.01%)
:arrow_up:

python
63.27% <100.00%> (-0.78%)
:arrow_down:

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

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

Hm, yes @villebro. Technically, it is loading from cache on every async query. I'll see what can be done about the UI.

Hm, yes @villebro. Technically, it is loading from cache on every async query. I'll see what can be done about the UI.

Ok, found the issue with legacy charts. The /superset/explore_json/data/:cache_key is setting cache-control: public 🤦 . I'll need to decouple the permission checks from the @etag_cache decorator.

Ok, found the issue with legacy charts. The /superset/explore_json/data/:cache_key is setting cache-control: public 🤦 . I'll need to decouple the permission checks from the @etag_cache decorator.

@betodealmeida the context here is force-reloading charts, which cannot serve cached data.

@betodealmeida the context here is force-reloading charts, which cannot serve cached data.

FYI @robdiciuccio flagging this for 1.0.1 (will retest shortly)

FYI @robdiciuccio flagging this for 1.0.1 (will retest shortly)

Thanks @villebro. Yeah, hiding the cached pill is a bit of a lift as all the async results are technically cached. The fact that the pill updates to "updated a few seconds ago" seems like an acceptable middleground for the time being.

Thanks @villebro. Yeah, hiding the cached pill is a bit of a lift as all the async results are technically cached. The fact that the pill updates to "updated a few seconds ago" seems like an acceptable middleground for the time being.


Add solution repo for spiraldancer

Add solution repo for spiraldancer

I'm going to merge this, but please include a solution in the actual repo, or it will be removed in the future.

Thanks! I have been having a bit of issues with git, and thought it was committed. The solution is now committed.  Sorry about that.


Adding ternary operator contenet

Select the Day of Content
Day 7
What information is missing?
explaination of ternary operator

Hey @gangwarsomya,
I am working on this (issue #94). Request you to work on something else. Thanks.

ohk @Varad0014


Add detector for docker container id for cgroup v1 to be added to resource metadata

Is your instrumentation request related to a problem? Please describe

Currently there is no support for adding container id to resource metadata by itself. The logic exists in aws detector but not as a standalone module.
Is it applicable for Node or Browser or both?
Node
Do you expect this instrumentation to be commonly used?
Weekly Downloads:
What version of instrumentation are you interested in using?
Versions:
Additional context

I am currently working on it. I should be able to open a PR soon.

This enhancement was added and merged .


addded config for wanchain

Wanchain drives blockchain adoption through crosschain interoperability by building fully decentralised direct bridges that connect the world’s many siloed blockchain networks
port wanchain pos chain to akash cloud

the lines in deploy.yml look redundant
    args:
      - 'apt-get update ;
        apt-get upgrade -y ;
        apt-get install vim -y ;
        sleep infinity'

the container must start with the default entrypoint, or with the arguments from deploy:
      - "chmod +x /wanchain/start.sh && /wanchain/start.sh "

however, the start.sh script looks unfinished

so at the moment the deployment is not working
there is also a lack of adding information to README.md

the script is changed for start a wan RPC node

added docker file and content for readme
modified deploy config
please check it
thanks

@Dimokus88 please check

@atomstaker You are using theforestjqg/rpc:imgwannoderpc custom image, you need to add it to your Dockerfile for security reasons.

changed Dockerfile ,please check it

wow, truly lost PR =D
Hey @atomstaker !
You can add information (link) about your template to README.md in the root directory, repository awesome-akash?

Hey @atomstaker !
You can edit info?


Video thumbnail into the ImageView + Recyclerview(Using URLs)

Is there any simple  android library which can load a Video thumbnail into an ImageView  from a URL like Picasso or Glide , when the Video is in the RecyclerView , and it just shows the thumbnail without downloading it.
But for Glide it loads thumbnails of local Videos in the phone not URLs

#29


release: update dependencies

update dependency sdk-core-version to 9.17.4

:tada: This PR is included in version 1.0.49 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


general support for models and expose packages to pkg

Is your feature request related to a problem? Please describe.
Now everything is pretty much monorepo under internal. There isn't really a way to manage external lib
Describe the solution you'd like
A clear and concise description of what you want to happen.
models can be exposed for external API uses

Done


Improve error when appending to array property before the schema is ready

Fixes #3454.

[x] Changelog entry

/c @jpsim @tgoyne

👍


Add default threading for windows

This PR fixes #471.
Uses critical section for synchronization.
Any ideas for spin count?

Hi @mrsshr Thanks for the contribution!
Before we could start reviewing, our policy is to not accept contributions, without a Contributor’s Licence Agreement (CLA) signed or authorised by yourself or your employer.
If this is a personal contribution, the easiest way to do this is if you create an mbed account and accept this click through agreement. Alternatively, you can find a slightly different agreement to sign here, which can be signed and returned to us, and is applicable if you don't want to create an mbed account or alternatively if this is a corporate contribution.
Thanks for your understanding and again, thanks for the contribution!
Please accept my apologies if I missed your signed agreement

Hi RonEld.
I am already agreed Contributor Agreement.
My profile is here.

Hi @mrsshr Thank you for your contribution!
The CI fails because at the following two scripts:
tests/scripts/check-generated-files.sh
tests/scripts/check-names.sh

In addition, the compilation fails  because:
error: "MBEDTLS_THREADING_WINDOWS defined, but not all prerequisites"

There are already too many configuration options in the config.h file. We don't tend to add new configurations, unless there is a strong justification for it. This new configuration is platform specific, which is also something we prefer to avoid. I thionk that perhaps you could use MBEDTLS_THREADING_PTHREAD, to define it as a platofrm threading implementation, (although it is not really pthread), and in your implementation, it will be within #if defined(_WIN32) check.
I don't think it's ideal, since it's not pthread, but it's better than introducing a new platform specific configuration option

i agree to your opinion
some code changed
and how to pass 'check-names.sh' test?
WIN32_LEAN_AND_MEAN for avoid socket header redefine
_WIN32_WINNT for critical section function ref

To compile an application that uses this function, define _WIN32_WINNT as 0x0403 or later.

Hello, @SergeySeroshtan
@sbutcher-arm mentions that mbedtls does not support Windows XP.
I think this PR is useless because using SRWLock(Vista and later) is better then using critical section.

Add PR #2306 that is based on the SRW Locks implementation.


[feature request] show Obsidian Title on the left

Hi, @pjeby. Can you add an option to show the Obsidian Title on the left in Style Settings? Next are the backward and forward buttons. Then the QE breadcrumb. Thanks in advance!

In Obsidian 0.16, the breadcrumbs and history arrows are on the tab title bar by default, and in most of Obsidian's new frame modes there is no ability to modify the window title bar -- or no title bar at all.
For these reasons, I will likely be phasing out the QE breadcrumbs, and so don't plan to add any new features for managing the window titlebar.
You can use a CSS snippet to do this yourself, however.  Something like this should probably work:
 .titlebar-text.qe-replacement {
    text-align: left;
    justify-content: flex-start;
    align-items: flex-start;
}

Thank you for your reply. I'm still using v0.15.9, the public release version, so I don't know what changes have been made in the latest version. I think I should wait until the new version comes out. Thank you anyway.


Fix splitCodeIntoArray when it has nested spans + newlines

The splitCodeIntoArray function was generating broken HTML when there were deeply nested \n characters because it wasn't closing the correct number of <span> tags.
Marking this as a draft to see what the performance impact is by using XPath.
See https://github.com/westonruter/syntax-highlighting-code-block/issues/193

Ran some benchmarks on calling splitCodeIntoArray 200000 times on different code snippets and the differences in performance are negligible.
test_original             : 35.122 sec.
test_xpath                : 36.956 sec.

test_original             : 35.362 sec.
test_xpath                : 35.501 sec.

test_original             : 54.900 sec.
test_xpath                : 55.953 sec.

test_original             : 24.244 sec.
test_xpath                : 22.913 sec.


Enable WebAssembly on xplat

[ ] Remove exclude_xplat tag from test folder WasmSpec and from individual tests in test folder wasm
[ ] Fix failing tests
[ ] Enable WebAssembly in release builds (remove #if defined(_WIN32) || (defined(__clang__) && defined(ENABLE_DEBUG_CONFIG_OPTIONS)) in CommonDefines.h

@Cellule any update on this?

We haven't allocated any resources to fix the issues that we have yet.
We have recently disabled WebAssembly by default even on test builds.

Is this scheduled for next quarter? If it's just about the build configuration / cmake issue, would be sad to see 2017 passing by without wasm support on unix.. :(

@kasper3 curious, do you have a specific use case in mind for WASM on Unix using ChakraCore?


issues during week 17

Getting this problem:

I tried uninstalling bundler and jekyll and other gems at first, then reinstalled, but was still getting the same error

Then I looked closer and saw the syntax error at the top, I located the file which seemed to have been messed up due to merge conflicts. I deleted the extra lines of code that were causing issues and then reran this command: bundle exec jekyll serve -H <IP_ADDRESS> -P 4001
After that I was finally able to get the server running:


Adds Client tests for Provider errors

Codecov Report

Merging #42 into master will increase coverage by 3.79%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
+ Coverage   53.08%   56.87%   +3.79%
==========================================
  Files           5        5
  Lines         211      211
==========================================
+ Hits          112      120       +8
+ Misses         89       84       -5
+ Partials       10        7       -3

Impacted Files
Coverage Δ

service/client.go
86.04% <0%> (+18.6%)
:arrow_up:

Continue to review full report at Codecov.

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


chore:Lambda path fix for data colletor lambda

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List
Lamaba path fix
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also
list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] My commit message/PR follows the contribution guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Other information:
List any documentation updates that are needed for the Wiki

@rnithinpaladin pleasw name your beach something like fix/devops-123 and add the issues id to the comment as [devops-122]. This way your commit get automatically attached to the Jira ticket.
What story is this related to?


Line appears in calendar when at non default zoom levels

If the user has vision problems or a preference for different sized text, and has used the browser's built in zoom feature, the css of the calendar can get slightly broken. I see a little line appearing in the top row of each month.

I added a white bottom border to those cells in my custom css and it seemed to fix it.


V16-dev exception

V16 dev c2a5db8
20:16:38: Claimed bonus points: 50
20:16:38: Earned points for watching:  50, total: 7940
20:21:38: Earned points for watching:  10, total: 7950
20:23:14: Fatal error encountered:
20:23:14:
20:23:14: Traceback (most recent call last):
20:23:14:   File "main.py", line 175, in main
20:23:14:   File "twitch.py", line 768, in run
20:23:14:   File "twitch.py", line 810, in _run
20:23:14:   File "twitch.py", line 1624, in fetch_inventory
20:23:14:   File "asyncio\tasks.py", line 611, in _wait_for_one
20:23:14:   File "twitch.py", line 1579, in fetch_campaigns
20:23:14:   File "twitch.py", line 1546, in gql_request
20:23:14: exceptions.MinerException: GQL error: [{'message': 'service unavailable', 'path': ['user', 'dropCampaign', 'allow', 'channels']}]
20:23:14:
20:23:14: Exiting...
20:23:14:
20:23:14: Application Terminated.
20:23:14: Close the window to exit the application.

Hello o/
https://github.com/DevilXD/TwitchDropsMiner/commit/e500569c332f68939f77faf80f41413e6db437d9 should fix this issue, and will become available once v16 releases. In the mean time, you can grab the latest fix from the master branch, or the pre-release dev build.


Creating federated identity fails

Bicep version
Bicep VS Code extension v0.12.40
Describe the bug
I'm setting up an AKS cluster which uses workload identity (preview).
This also involves configuring a federated identity which I can successfully set up in the portal or by running the following AZ CLI command:
az identity federated-credential create --name fedid-idcds25 --identity-name idcds25 --resource-group rg-test-cds25 --issuer https://eastus.oic.prod-aks.azure.com/7[...]6/ --subject system:serviceaccount:importer:workload-identity-sa

Trying to achieve the same result using bicep fails.
To Reproduce
The resource I'm deploying to replicate above's AZ CLI command is this:
resource symbolicname 'Microsoft.ManagedIdentity/userAssignedIdentities/federatedIdentityCredentials@2022-01-31-preview' = {
  name: 'fedid-idcds25'
  parent: managedIdentity // resource reference to idcds25
  properties: {
    issuer: [oidcIssuerUrl](https://eastus.oic.prod-aks.azure.com/7[...]6/)
    subject: 'system:serviceaccount:importer:workload-identity-sa'
  }
}

Two things:

The VS Code extension is adamant about requiring audiences as a property within properties. However, this is optional (see AZ CLI command).
If I add empty audiences  (like audiences: ['']), the result is a bad request during deployment.

Workaround
Set the audiences like this (undocumented, I found out by inspecting existing resources).
audiences: [
      'api://AzureADTokenExchange'
    ]

@johnpetersjr - you might want to open up a new issue or start a Discussion. FWIW, if you need to deploy an array of resources serially, you can use the @batchSize(1) decorator:
@batchSize(1)
resource foo '...' = { ... }

Thanks, I found @batchSize(1) as well, but now it just seems that Bicep cannot handle the idempotency of multiple Federated Credentials on a single User Identity, i.e.:
[{"code":"Conflict","message":"Issuer and subject combination already exists for this Managed Identity."}]
I'll start up a new issue with my (not well working) code, as this is an ancient thread, thanks!


Fixed Responsiveness of login and signup pages issue #109

i fixed the responsiveness issue caused by the Eye icons in login and signup page
Sign up page:

Login Page:

please Add gssoc-ext and level1 label before merge @Vin205

Hey @skmirajulislam  can you please add correct label level1 not Level 1 my points in gssoc-ext did not count due to incorrect labeling


Fix: User model (account creation)

This PR corrects a progressTimestamp format error for a users inital progressTimestamp.
We also provide a default profileUI to new accounts and existing accounts that either do not have one or it is incomplete.

users with a profileUI

users without a profileUI


Question on R1, R2, t1, t2

Thanks for the code. Could you please explain how you got the R1, R2, t1, t2 matrices ? Why is it that they are in the camera frame ? I find it hard to visualize it in camera frame

Hi @HareshKarnan
Sorry for the late reply. I will try to explain as I can remember.

Could you please explain how you got the R1, R2, t1, t2 matrices ?

These are the settings for the camera posture on the 3D modeling tool.
First, I defined an object coordinate system such that the mesh grid has Y=0 and lies on the XZ plane (the direction of the XYZ axes follows the OpenCV coordinate system).
R1 and R2 are set in the modeling tool as 30 degrees downward and 90 degrees downward from the object coordinate system, respectively.
Similarly, the camera positions t1 and t2 are set in the modeling tool in the object coordinate system.
(e.g. [0, -0.866, 0] is the position of camera 1 in the object coordinate system.)
Then, the rotation matrices R1 and R2 are applied to t1 and t2 in the object coordinate system to represent them in the camera coordinate system.

Why is it that they are in the camera frame ? I find it hard to visualize it in camera frame

Because the homography equation uses R1, R2, t1 and t2 as such coordinate system.
You can visualize these in object coordinate system since the values are described at the notebook.

Thanks for the reply. If the object coordinate frame is as you say, with Y axes pointed up, shouldn't t1 be [0, 0.866, 0] ? Why the negative ? since the camera is above the ground.

Because XYZ axes of the object coordinate frame follows the OpenCV one, meaning Y axes pointed down for positive.


feat: add support for writing parsed JSON to file or printing to console

It would be nice to be able to output the parsed JSON to the console or write and save it to a JSON file upon completion.

COMPLETED
See commit ef66ce8.


ng2-cloudinary has no exported member 'CloudinaryImageComponent'

Installing the latest version from npm<EMAIL_ADDRESS>gives this error when importing:
import { CloudinaryImageComponent } from 'ng2-cloudinary';
I am running node 6.9.1.
Any idea why?

Hi tskytt,
You're right, CloudinaryImageComponent is not exported "as is" in the module package.
But if you import Ng2CloudinaryModule, you might not need to import CloudinaryImageComponent
I will fix the export in a future release.
If you really need the component, you can always import it like this :
import { CloudinaryImageComponent } from 'ng2-cloudinary/src/cloudinary-image.component';

Hi julienboulay,
Thanks for your quick reply. That's great!


Video preview in sanity studio displays the wrong video

I found a strange bug in sanity studio. Every time I click on a new video, it shows me the video that I clicked right before. The first video that I click is empty.
After deleting everything, upgrading to sanity 2.21.0 and MuxInput 1.1.0 and uploading some videos again, the bug still remained.
https://user-images.githubusercontent.com/8150506/137128688-74ddb190-de25-4679-b905-670a36e0e68d.mov

Assuming stale - the plugin's approach to previews changed significantly and this is probably not an issue anymore


Update syntax/php.vim and maintainer info

I contacted Jason Woof via email after the discussion in #5690

Syntax highlight fn in php short arrow functions (fn()=>)
the same way as function
https://wiki.php.net/rfc/arrow_functions_v2#proposal
Fix php numeric literal syntax highlighting bugs.
Support _ as a numeric literal separator for php 7.4.
https://www.php.net/manual/en/language.types.float.php

I'll include it, thanks.