Support dynamic Tabs with extendBody: true

I'm having an error when dynamically changing the number of Tabs with DefaultTabController, as supported by #30884.
But only if I the Scaffold gets extendBody: true
The Error:
I/flutter ( 5338): ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
I/flutter ( 5338): The following assertion was thrown building TabBarView(dirty, dependencies: [_TabControllerScope],
I/flutter ( 5338): state: _TabBarViewState#bbccc):
I/flutter ( 5338): Controller's length property (2) does not match the
I/flutter ( 5338): number of tabs (1) present in TabBar's tabs property.
I/flutter ( 5338): When the exception was thrown, this was the stack:
I/flutter ( 5338): #0      _TabBarViewState.build.<anonymous closure>
package:flutter/…/material/tabs.dart:1317
I/flutter ( 5338): #1      _TabBarViewState.build
package:flutter/…/material/tabs.dart:1323
I/flutter ( 5338): #2      StatefulElement.build
package:flutter/…/widgets/framework.dart:3921

Here's the gist that I ran with flutter v1.6.3
The gist runs in two mode versions:

Using DefaultTabController, where it fails
Using a custom controller that works fine, but it needs copyWith public:
https://github.com/flutter/flutter/blob/d9f6cada1ca8311e75c17cf41ea9cc3cacf74673/packages/flutter/lib/src/material/tab_controller.dart#L122-L129

Removing extendBody: true and both versions work fine.
This second version was my workaround, despite HansMuller's recommendation in #30884 to keeping it private. This hack is not the focus of this issue, but let me know if there's a cleaner way to achieve dynamic tabbing with a custom controller.
@johnsonmh , I'm poking you on this. Please, disregard if it's unrelated to #30884.

_copyWith which is still private
flutter doctor -v
[✓] Flutter (Channel dev, v1.16.4-pre.50, on Mac OS X 10.15.4 19E266, locale
    en-GB)
    • Flutter version 1.16.4-pre.50 at /Users/taha/Code/flutter_dev
    • Framework revision d14a301e41 (3 days ago), 2020-03-31 19:14:22 -0400
    • Engine revision 8166a91bca
    • Dart version 2.8.0 (build 2.8.0-dev.19.0 fae35fca47)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.4, Build version 11E146
    • CocoaPods version 1.9.1

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

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 44.0.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)

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

[✓] Connected device (4 available)
    • SM M305F   • 32003c30dc19668f • android-arm64  • Android 10 (API 29)
    • macOS      • macOS            • darwin-x64     • Mac OS X 10.15.4 19E266
    • Chrome     • chrome           • web-javascript • Google Chrome
      80.0.3987.149
    • Web Server • web-server       • web-javascript • Flutter Tools

• No issues found!

Hi @rmarau, Looks like this issue has been fixed, I tried to run your code, But it wouldn't compile, Please take a look at the below code sample which works on  2.13.0-0.0.pre.808 • channel master with extendBody: true, This should help you add dynamic tabs.

code sample
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: const MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key}) : super(key: key);

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final List<Widget> _tabs = [];

  @override
  void initState() {
    super.initState();
    List.generate(3, (index) => _addTab());
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      extendBody: true,
      appBar: AppBar(
        title: const Text("Default Tab Controller"),
      ),
      body: DefaultTabController(
        length: _tabs.length,
        child: TabBar(tabs: _tabs),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: () => _addTab(),
        child: const Icon(Icons.add),
      ),
    );
  }

  void _addTab() {
    setState(() {
      _tabs.add(Text("Tab ${_tabs.length}",
          style: const TextStyle(color: Colors.black)));
    });
  }
}

output
https://user-images.githubusercontent.com/31410839/165679777-9a66a9b8-a92b-4e50-91fe-ef5b805a5525.mov
Probably this is a related PR which fixed it https://github.com/flutter/flutter/pull/94623.

flutter doctor -v (mac)
[✓] Flutter (Channel master, 2.13.0-0.0.pre.712, on macOS 12.3 21E230 darwin-arm, locale en-IN)
    • Flutter version 2.13.0-0.0.pre.712 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 8c22f5648a (77 minutes ago), 2022-04-25 05:34:07 -0400
    • Engine revision 10efa195bf
    • Dart version 2.18.0 (build 2.18.0-44.0.dev)
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! CocoaPods 1.10.2 out of date (1.11.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that
        responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

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

[✓] Android Studio (version 2021.1)
    • 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.11+0-b60-7772763)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

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

[✓] Connected device (3 available)
    • Redmi K20 Pro (mobile) • <IP_ADDRESS>:5533 • android-arm64  • Android 11 (API 30)
    • macOS (desktop)        • macos            • darwin-arm64   • macOS 12.3 21E230 darwin-arm
    • Chrome (web)           • chrome           • web-javascript • Google Chrome 100.0.4896.127

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.


[Download as PDF] I am unable to download the PDF

Is there an existing issue for this?

[X] Yes, I have searched the existing issues and none of them match my problem.

Product Variant
Cloud (https://rxresu.me)
Current Behavior

Issue 1:
So whenever  I click on this link button (in below image) it should copy the public resume link. Which it is not doing and just copying the undefined for me

Issue 2:
Even if I copy the link from the sidebar and then open that link then link works but when i click the Dowanload as PDF  button it does not download anything in fact just return the empty white browser tab.

Here is my Public link : https://rxresu.me/vanshsharma9354/vansh-sharma
Expected Behavior

Issue 1:
So whenever  I click on this link button (in below image) it should copy the public resume link.

Issue 2:
Even if I copy the link from the sidebar and then open that link then link works but when i click the Dowanload as PDF  button it should download the resume  PDF

Steps To Reproduce

For Issue 2:

Open this link:  https://rxresu.me/vanshsharma9354/vansh-sharma
The click on the Download as PDF button
You will see nothing gets downloaded just empty white browser tab

For Issue 1:

I am facing that while Resume edit is open
Click on the Link button as shown in the above Image 1 and it copies the undefined

What browsers are you seeing the problem on?
Chrome
What template are you using?
None
Anything else?
No response

Hello, I see you closed this item as completed, however I am getting the exact same error in my environment. Can you tell me how you were able to resolve this issue?

Hello, I see you closed this item as completed, however I am getting the exact same error in my environment. Can you tell me how you were able to resolve this issue?

@jdhill777
Step 1: Go to your public link which you might get easily from the right side bar
Step 2: Press Ctrl + P and then you can save.
NOTE: But issue still exists


Any way to preserve JS or Coffee comment blocks during minification?

Using all my other build tools and compilers using @preserve or @license works.
I like to include easter eggs in my code that are not minified like this for example:

I haven't found any documentation on how this is treated, but here is my production config file.
axis         = require 'axis'
rupture      = require 'rupture'
autoprefixer = require 'autoprefixer-stylus'
js_pipeline  = require 'js-pipeline'
css_pipeline = require 'css-pipeline'
jeet = require 'jeet'

module.exports =
  ignores: ['readme.md', '**/layout.*', '**/_*', '.gitignore', 'ship.*conf']

  extensions: [
    js_pipeline(files: 'assets/js/*.coffee', out: 'js/main.js', minify: true, hash: false),
    css_pipeline(files: 'assets/css/*.styl', out: 'css/style.css', minify: true, hash: false)
  ]

  stylus:
    use: [axis(), rupture(), autoprefixer(), jeet()]

  server:
    clean_urls: true
    gzip: true

I'm sure you can understand the extreme importance of this kind of functionality.  Thanks!

Haha of course, just add this line:
jade:
  pretty: true

and your comments will live on with vibrant life :sparkles:

If my comments were in Jade I would agree.  Having tested this, I'm seeing that that does not have any effect on CoffeeScript which is where I want to preserve the comments.
An example from a Rails project I worked on:
http://www.badassembly.com/assets/application-e8dd98f992baf47a9cf46cc9e2674e5a.js
The code itself is all minified, but comments needed for attribution are preserved.
Is there a way to configure this in Roots?

Hahah oh I'm sorry! Totally blew it. This is a question about the js pipeline then? How are you minifying js?

Indeed that sounds correct.
Currently I'm using the basic out of the box config here:
js_pipeline(files: 'assets/js/*.coffee', out: 'js/main.js', minify: true, hash: false)

Somehow I need to pass a flag to UglifyJS that seems to be under the hood of the js_pipeline's minifications that tells it to --comments
I found this here: https://github.com/mishoo/UglifyJS2#keeping-copyright-notices-or-other-comments
I'm not so sure how to set those options in the roots config for the js_pipeline based on the documentation.
I'll dig through the code now to see if I can figure it out, but hopefully that paints a picture of what I'm trying to solve if you have any ideas.

I'll get this added to js pipeline as long as uglify's JS API supports this option. One moment...

K, just published, it should now preserve comments. Whoo! :tada: Just update to the latest version of js pipeline and you should be set.

Hi Jeff,  I finally got a chance to test this thoroughly and I could be doing something wrong, but I've tried a lot of different things and I'm not certain your implementation works when minify: true
I've got the latest release from you and even tried the exact code from the test.coffee you wrote.
Again, my production config is this:
js_pipeline(files: 'assets/js/*.coffee', out: 'js/main.js', minify: true, hash: false)
It works when minify is false, but that is not the challenge here, the challenge is to enable the --comments flag in Uglify to only preserve comments that are @preserve or @required
I tried this:
js_pipeline(files: 'assets/js/*.coffee', out: 'js/main.js', minify: true, hash: false, comments: true)
and this
js_pipeline(files: 'assets/js/*.coffee', out: 'js/main.js', minify: true, hash: false, opts: {comments: true})
But it seemed like the code change you made would just enable the comments flag by default without any config needed, though I don't know exactly how it does that.
Thanks for you swift responses!

Yeah I figured having it on by default was fine. I don't see many situations where someone would write @preserve and expect the comment to be gone haha.
The change was super simple, right here: https://github.com/carrot/roots-js-pipeline/commit/99f2be5be0097a9243aea7974e2e36fd4d605307
I added a test as well and verified that it was working, although perhaps something wasn't quite right? Feel free to take a look at the js pipeline code and open an issue there if you are still having trouble. It's pretty simple code, if you are feeling ambitious you could pull it down and try to open a PR to fix it too!


Fault Tolerance 3.0

[ ] Fault Tolerance 3.0 support
[ ] Documentation
[ ] Samples
[ ] Guides

MicroProfile 4.0 support epic: #2636

Changes in FT 3.0:

[ ] Lifecycle of circuit breakers
[ ] Changes related to Jakarta EE dependencies
[ ] @Asynchronous test related to security/naming context
[ ] Changes to metrics and scopes

Changes in FT 3.0:

[ ] Lifecycle of circuit breakers
[ ] Changes related to Jakarta EE dependencies
[ ] @Asynchronous test related to security/naming context
[ ] Changes to metrics and scopes

PR #2680

PR #2680


Autoprefixer 5.2.0 API change

With the release of Autoprefixer 5.2.0 you get the following warning:
Autoprefixer's process() method is deprecated and will removed in next major release. Use postcss([autoprefixer]).process() instead

https://github.com/postcss/autoprefixer-core/commit/d35c9321e2ca956fafc82e443e3b2cd06c21aaab
Not sure how to fix this so didn’t create a PR myself.

I am receiving the same warning. Is there a workaround at this time?

new version is published now.

@lukeapage Thanks for taking care of this so quickly!

Works like a charm @lukeapage :)


Fix warnings and typos

Thanks for your contribution!

Coverage remained the same at 95.643% when pulling b1f27f39459018c2bf89cbb48cb13ea8b995acd2 on superjobru:fix-warnings-and-typos into 7812141bf014ee875c28aaf755b7022fdbf67727 on huandu:master.


Replace complex pool with mutex pool

Fixes #65

Hmm, the test on 8.0 seems unrelated to this change @blackbeam ?

Anecdotally, this eliminates the connection leaking I was seeing that was causing #65.

I discovered a race, so hold off on merging this for a little bit.

Okay, I think that last commit fixed it, and that this is now ready.

Bah, apparently not -- will dig more.

Phew, okay, that should do it!

@blackbeam Can we do a point release with this change?

@jonhoo, I'll publish master as v0.22.0.
Please tell me if you need a more precise release with only this change.

No, a major release is fine too :)


[RFC] docs: use base tag

@daKmoR didn't catch it in the other PR but wanted to check in on your choice to use responseTransformers over a base tag. Thoughts?

It does look like the base tag plays games with anchor links: https://5eb7085fc83fc00007895dc0--open-wc-org.netlify.app/faq/index.html Possible that that is a worthwhile annoyance to avoid...

we don't need a base tag in the live site... so imho it's better to only add it for dev...
I am curious about your reasons to "change" it? what would you gain?

Only in dev makes sense for the OpenWC setup, but I'm also taking a look at the current situation from two other perspectives:

Trying to get ramped up on @mdrn/site as a new user: what is the responseTransformers and why do I need it right out of the box? If we were to include it by default we'd need to really clearly document it. Feels like code smell in my new car, and for my experience <base> is what solves the underlying issue you're addressing, this PR is mainly to confirm the alternative approach so we can discuss it and figure out the right pattern for a more public version of the tool.
I have to deploy the SWC site with a <base> tag anyways (root for local/PR/branch builds and /spectrum-web-components/${site} in our public URLs, so, while I likely have reflexive do what I'm already doing-itis, I do think we'll need to address this in some way for the published tool. Whether it's needed by default to address the peculiarities of how the tool works or is addressed as a light configuration, sites often need to be deployed to directories, so I was doing a little thinking ahead.


English TTS with kaldi is sexist

While it understands me (non-native speaker, male) well it struggles to understand my girlfriend (native speaker, female). It works better when she tries to speak with a deeper voice also her natural voice is not that high. I will follow up with audio samples soon. The question is what data was the model trained on? Was it only contained male voices?
Has anyone made similar experience? My current sample set is quite small. I try to increase it.

I don't remember this behavior with pocketsphinx. I will test soonish.

The Kaldi English model comes from Zamia Speech, and was trained on these open datasets. For English, this seems to include LibreSpeech, LJSpeech, Mozilla Common Voice, TED-LIUM, VoxForge, and Zamia.
The majority of the dataset comes from LibreSpeech, with TED-LIUM and Common Voice being the next two largest ones. LibreSpeech appears to be quite well balanced (1201 female, 1283 male). TED-LIUM seems to have about 2x males/females (based on the paper), and Common Voice lists itself as 46% male/13% female (rest unreported?).
So it would seem yes, there is definitely a bias towards male speakers in the datasets. I'd hesitate to call it "sexist", however, since (to my knowledge) Kaldi itself does not have any mathematical notion of sex in the nnet3 recipes (used to train this model).
If she's willing, I would encourage your girlfriend to donate to Mozilla Common Voice. I keep an eye on that dataset and this excellent corpora listing. I hope to start training my own Kaldi models soon, and a greater diversity in the dataset would be most welcome.

Thanks for the explanation. I will also checkout some other models for example wav2letter: https://talonvoice.com/research/


Can't select Smithing perks

Describe the bug
I can unlock the Basic Smithing perk, the requirement is level 20 but I'm not at level 45 and still can't unlock it
To Reproduce
Steps to reproduce the behavior:

Go to the skill tree

Expected behavior
To unlock the "Basic Smithing" perk
Screenshots

Modlist version
No just Lotus

Basic Smithing requires that you already have the perk "MAG_Blacksmith00 "Novice Blacksmith" [PERK:77334BCC]"
Quick fix: "player.addperk 77334BCC"

Should be fixed in the next update

Thank you for your response.
Very helpful.
On Sun, May 10, 2020 at 5:41 AM erri120<EMAIL_ADDRESS>wrote:

Closed #77 https://github.com/erri120/lotus/issues/77.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/erri120/lotus/issues/77#event-3320155993, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMYEA7QC2L5OZQ64T3VM4MDRQZZEJANCNFSM4M4PYOWQ
.


missing list file for training and validation data

Hi, @tomztyang ,
According to data-preparation  , several list file for training and validation data (train.txt, test.txt val.txt, trainval.txt) are required. However, they are not provided by the official benchmark website. Could you provide them here?
THX!

Hi @amiltonwong ,
I have provided train.txt and val.txt here. The trainval.txt is just the merged version of these two former files.
Thank you so much.
train.txt
val.txt

Hi, @tomztyang ,
Thanks for your reply. One more file is missing: test.txt. Could you also provide it?
THX!

Added~
Regards,
Zetong
test.txt


POST req with unexpected content type to route kills the Remix server

What version of Remix are you using?
1.3.3
Steps to Reproduce
Create a remix app, and create a route with just an action.
Within that action, if you await request.FormData, but then make a post request of a type other than Multipart Form, the remix app server will throw an error and exit.
an example repo to test this can be found here:
https://github.com/DKeen0123/remix-resource-route-issue
spin up the app and make a post request to /loginwith Content-Type of application/json.
you'll see this error:
Error: Unsupported content type: application/json     at Busboy.parseHeaders (/Users/danielkeen/personal/test-resource-routes/node_modules/busboy/lib/main.js:74:9)     at new Busboy (/Users/danielkeen/personal/test-resource-routes/node_modules/busboy/lib/main.js:21:10)     at /Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/node/parseMultipartFormData.js:44:18     at new Promise (<anonymous>)     at Object.internalParseFormData (/Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/node/parseMultipartFormData.js:43:9)     at Request.formData (/Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/node/fetch.js:100:43)     at action (/Users/danielkeen/personal/test-resource-routes/build/route:/Users/danielkeen/personal/test-resource-routes/app/routes/login.tsx:4:34)     at Object.callRouteAction (/Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/server-runtime/data.js:40:20)     at handleResourceRequest (/Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/server-runtime/server.js:440:25)     at requestHandler (/Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/server-runtime/server.js:42:24)
Expected Behavior
Anything but having the server process actually exit! I think there should be a way for the server to more gracefully handle an unexpected content type. Throwing an error but continuing to run the server would probably be fine.
Right now, I can make a post request with an incorrect content type to any of my routes and completely kill my remix app and have to manually redeploy it!
I can manually check the content type header on the request and filter it down so that I only proceed if the content type is the expected one, but I think the default behaviour should not be killing the application.
Actual Behavior
sending an unexpected content type to a route gives me this error:
Error: Unsupported content type: application/json     at Busboy.parseHeaders (/Users/danielkeen/personal/test-resource-routes/node_modules/busboy/lib/main.js:74:9)     at new Busboy (/Users/danielkeen/personal/test-resource-routes/node_modules/busboy/lib/main.js:21:10)     at /Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/node/parseMultipartFormData.js:44:18     at new Promise (<anonymous>)     at Object.internalParseFormData (/Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/node/parseMultipartFormData.js:43:9)     at Request.formData (/Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/node/fetch.js:100:43)     at action (/Users/danielkeen/personal/test-resource-routes/build/route:/Users/danielkeen/personal/test-resource-routes/app/routes/login.tsx:7:34)     at Object.callRouteAction (/Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/server-runtime/data.js:40:20)     at handleResourceRequest (/Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/server-runtime/server.js:440:25)     at requestHandler (/Users/danielkeen/personal/test-resource-routes/node_modules/@remix-run/server-runtime/server.js:42:24)
And kills the server.

Fix will be in next release. Thanks for the report :)


Overwatch Shadow Glitches

Software information
Overwatch on low overall settings and low shadows.
Map is Lijang Tower on the 1st stage.
System information

GPU: GTX 1070
Driver: nvidia-vulkan (396.54.09-1)
Wine version: Wine Staging 3.18
DXVK version: 0.90

It appears that when you are playing, there are certain points where shadows appear on the other sides of walls that give away a players position. I discovered this on Lijang Tower, the bots would run behind the wall and there would be one circular shadow that traces them perfectly on the other side.
I think this is a potential exploit, and not just exclusive to this location. I would really not like to somehow get banned for this and have turned off shadows until this is fixed.
Here are some screenshots:
(2 lucios behind the wall)
https://i.linkjay.io/SnqcNX.png
(another 2)
https://i.linkjay.io/ExlEJD.png
(using widows alt, i show a lucio behind the wall with a shadow right above him)
https://i.linkjay.io/CppbtI.png

Have you tried an older driver? People have played Overwatch for a while without such issues.
Also, it's impossible for me to debug this game.

Did this happen somewhere else than in Lijang Tower? I'm going to test on a RX580.

I tried this with Las and we can't reproduce it, in a custom game on Lijiang in the exact same spot. Tried wallriding at different heights, messing around, etc. RX 580 for us both.
Did it happen really often or only occasionally?

This seems to be a bug in the Nvidia driver, since we can not reproduce it with an RX 580. Using wine-staging 3.18 here with dxvk 0.90.

I can only seem to reproduce this issue at that specific part of Lijang Tower. It seems to be that hard lighting. I assume this is a driver issue with NVIDIA going off of what you guys are reporting.
I am using the nvidia-vulkan pack which uses a sort of outdated version of the drivers, so this may have been fixed in the actual NVIDIA pack. I cannot test it with anything else, this is what works for me to be able to launch Battle.net and Overwatch.
I leave it up to you to decide how to proceed on this. Maybe is there a better repository or somewhere I could report this issue like an NVIDIA repo or something?
More screenshots showing it happening all around that room and in that same area.
https://i.linkjay.io/RxVIzJ.png
https://i.linkjay.io/TnnRak.png
https://i.linkjay.io/ytuTgg.png

Can you try with another driver that isn't 396.54.09 ( which has TF extension ) ?

I occasionally experience the bug myself with 396.54.04.

@varris1 also experiences this with his GTX 970. This is most likely an issue with the Nvidia drivers.

This game's anti-cheat is blocking apitrace or banning you if you use it, right?
I might be able to reproduce this and post a traceroute, but I do not want to get banned.

You should move to 410.73 and test again

wine 4.19 staging is out

Using all the master nvidia packages now from the official arch linux repo. I am now using drivers version 410.66-3 . It seems like this issue does not occur in Overwatch anymore with these drivers. Although performance is shafted rather heavily in some cases because of the lack of vulkan fixes.
It still does when using the nvidia-vulkan package. Now, the nvidia-vulkan package is supposed to be basically more up-to-date than the nvidia ones. It's as if they took nvidia, added some vulkan-related fixes to it, and then it will be merged into the 4XX drivers when they are done.
This has to be an issue with the nvidia vulkan drivers then. Maybe somehow its how its implemented within DXVK also? The regular nvidia drivers are still using vulkan, but just not as good as nvidia-vulkan.

Using all the master nvidia packages now from the official arch linux repo. I am now using drivers version 410.66-3 . It seems like this issue does not occur in Overwatch anymore with these drivers. Although performance is shafted rather heavily in some cases because of the lack of vulkan fixes.
It still does when using the nvidia-vulkan package. Now, the nvidia-vulkan package is supposed to be basically more up-to-date than the nvidia ones. It's as if they took nvidia, added some vulkan-related fixes to it, and then it will be merged into the 4XX drivers when they are done.
This has to be an issue with the nvidia vulkan drivers then. Maybe somehow its how its implemented within DXVK also? The regular nvidia drivers are still using vulkan, but just not as good as nvidia-vulkan.

Check out this guys stuff, he sets up all the things we want to build for us better than the arch repos https://github.com/Tk-Glitch/PKGBUILDS we're on 410.73 atm

@tannisroot You have to enable shadows, they don't seem to be enabled in your clip

Yep, I enabled it and I see the issue now. Happens with 410.66 too.

This is not fixable, and I highly recommend switching to wined3d for this game. I will consider blacklisting Overwatch from DXVK in the future.

What about AMD users?

@doitsujin ok like I get that this is an issue that cannot be fixed on your part, but is blacklisting the game in DXVK really a good option at all? AFAIK, this is the only problem with vulkan and overwatch, and it can easily be fixed by disabling shadows in-game.
Performance is significantly worse in Wine3d and it would pretty much kill all support for overwatch on linux if you blacklisted it on DXVK.
If you cannot fix this issue, maybe it can be brought up to Blizzard or Nvidia for them to develop a fix?

If that helps, I have just tested with an old Nvidia driver (390.48) and the bug is still there. This does not seem to be a driver regression, this bug has been there for a very long time.

Even performance with DXVK isn't optimal, but I guess nothing can be done if it's impossible to trace the game and figure out what it does. I'd try contacting Jeff Kaplan, but he prolly doesn't care about this.
But please don't take my freedom away of playing the game using DXVK, shadows in OW are useless and while they tell you where someone is if they're behind a wall, the game tells you 20 times a minute where someone from the opposing team is at via outlines.

I would just like to add that I did test the game without DXVK and using wine3d. I got about 30 fps on the main menu when the 3d part loaded and I got at most 20 fps where I was averaging about 12 fps in-game.
If you blacklist Overwatch on DXVK, you will effectively be killing support for Overwatch on linux. Wine3d is NOT a viable alternative to DXVK for overwatch.

There's tons more issues with the game on Wine, like not being able to crouch, DEs messing with getting raw mouse input, window focus issues (winex11 smh), phantom mouse input, mouse cursor locking issues (the cursor disappears sometimes and doesn't stay correctly inside the windows)
I think those affect the game much worse than FPS.
But since this is a NVIDIA bug, I'll gladly make my next GPU be an AMD one just so I can get rid of NVIDIA's nonsense.

Lutris’s installer for Overwatch will get an update soon to workaround this issue by disabling shadows all the time. Please don’t worry.

Small update: For some shadow-related rendering operations, this game uses a shader data type that is incompatible to the render target format. Still not sure if this is actually the problem, but that would be really bad news because there's no viable way to work around this.

There's tons more issues with the game on Wine, like not being able to crouch, DEs messing with getting raw mouse input, window focus issues (winex11 smh), phantom mouse input, mouse cursor locking issues (the cursor disappears sometimes and doesn't stay correctly inside the windows)
I think those affect the game much worse than FPS.
But since this is a NVIDIA bug, I'll gladly make my next GPU be an AMD one just so I can get rid of NVIDIA's nonsense.

That's a fair point. These bugs exist right out of the box, but all of them have fixes.

Fix crouching by using an xmodmap workaround.
Not sure where I saw this raw mouse input bug but there's gotta be a fix for it.
Window focus issues can be fixed by setting the "Automatically capture the mouse in full-screen windows" setting in your Winecfg
Phantom mouse input is usually caused by the aspect ratio of the game being different than the desktop/monitor aspect ratio
Mouse cursor locking could be fixed by changing fullscreen around to windowed/windowed borderless and/or setting the "Automatically capture full-screen mouse" setting.

So yeah, fair enough that these issues do exist, but all should have some sort of fix. This shows real progress with the game and it's very playable other than maybe performance which could be better.

If anyone at Nvidia cares, the bug also occurs with a Kepler card (GT650M).

Driver : 396.54.09

@doitsujin worked around that issue in the possible-ow-fix branch. It seems to have fixed the shadows for several people.


HDFS-15898. Test case TestOfflineImageViewer fails

NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HADOOP-XXXXX. Fix a typo in YYY.)
For more details, please see https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 54s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
34m 41s

trunk passed

+1 :green_heart:
compile
1m 23s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
1m 13s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
checkstyle
1m  1s

trunk passed

+1 :green_heart:
mvnsite
1m 21s

trunk passed

+1 :green_heart:
javadoc
0m 53s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 20s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
spotbugs
3m 19s

trunk passed

+1 :green_heart:
shadedclient
18m 56s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 16s

the patch passed

+1 :green_heart:
compile
1m 14s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
1m 14s

the patch passed

+1 :green_heart:
compile
1m  9s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
javac
1m  9s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 55s

the patch passed

+1 :green_heart:
mvnsite
1m 14s

the patch passed

+1 :green_heart:
javadoc
0m 45s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 16s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
spotbugs
3m 29s

the patch passed

+1 :green_heart:
shadedclient
19m 24s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
355m  2s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 35s

The patch does not generate ASF License warnings.

448m 32s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.datanode.TestBlockScanner

hadoop.hdfs.server.namenode.ha.TestBootstrapStandby

hadoop.hdfs.TestPersistBlocks

hadoop.hdfs.server.namenode.ha.TestEditLogTailer

hadoop.hdfs.TestDFSShell

hadoop.hdfs.server.namenode.snapshot.TestNestedSnapshots

hadoop.hdfs.server.datanode.TestIncrementalBrVariations

hadoop.hdfs.server.datanode.fsdataset.impl.TestFsVolumeList

hadoop.hdfs.server.namenode.TestDecommissioningStatusWithBackoffMonitor

hadoop.hdfs.server.mover.TestMover

hadoop.hdfs.server.datanode.TestDataNodeHotSwapVolumes

hadoop.hdfs.server.namenode.TestDecommissioningStatus

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2779/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2779

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 38d5848ba236 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 631df694ede326a4ce546a0c1af562ce033a2ee6

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2779/1/testReport/

Max. process+thread count
2191 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2779/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

@aajisaka Thanks for review!
Failed tests are unrelated, merged


Context Menu\Folder shortcut problem after installing PS Core 6.1 on Windows

Steps to reproduce
Install PowerShell Core 6.1 on Windows choosing to add context menu "Open PowerShell from here". Once installed, just clicking any Windows shortcut for a folder opens a PowerShell 6.1 window not the Windows Explorer window at the folder.
This only seems to happen with Directory\Folder shortcuts in Quick Launch (%APPDATA%\Microsoft\Internet Explorer\Quick Launch) that are then exposed on Quick Launch toolbar on W10 taskbar as they respond to single click to open. The single click open PowerShell. Uninstall returns behavior to normal.
Expected behavior
Clicking a folder shortcut opens the folder in Windows Explorer.

Actual behavior
---------------

Clicking any Windows shortcut when on Quick Launch toolbar on W10 taskbar for a folder opens a PowerShell 6.1 window not the folder in a Windows Explorer window at the folder.

Environment data
PS D:\Downloads> $PSVersiontable
Name                           Value

PSVersion                      6.1.0
PSEdition                      Core
GitCommitId                    6.1.0
OS                             Microsoft Windows 10.0.17134
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0
This system is a fully patched W10 Education x64 version in domain environment.

@bergmeister Could you please comment the Issue?

Chris,
I have updated the issue on github.
Pete
Pete Gomersall
IT Team Lead
Academic & Research Systems Administration, Information Technology Services
MCSA, MCSE, MCDBA & MCT
Room 143 ITS Building 54
Northern Arizona University
Phone: +1<PHONE_NUMBER>
Cell: +1<PHONE_NUMBER>
From: Christoph Bergmeister [MVP]<EMAIL_ADDRESS>Sent: Wednesday, September 19, 2018 6:24 AM
To: PowerShell/PowerShell<EMAIL_ADDRESS>Cc: Peter Lindsay Gomersall<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [PowerShell/PowerShell] Context Menu\Folder shortcut problem after installing PS Core 6.1 on Windows (#7815)
@PGomersallhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FPGomersall&data=02|01|pete.gomersall%40nau.edu|9ac10d55de8d4969410f08d61e332e8e|27d49e9f89e14aa099a3d35b57b2ba03|0|0|636729602704971846&sdata=zLLGNtD5AznJlooT%2F925rast6FddW7OpgyWYSpqc%2FrY%3D&reserved=0 Please fill in the complete issue template. You have not given sufficient details about the environment. Please provide the output of $PSVersiontable and the OS version.
I got a new Windows 10 laptop and cannot reproduce the issue, I suspect this happens on an old OS.
When I implemented the feature, I did not manually test this on old Desktop Windows versions like e.g. Windows 7 or 8.1 (but I think I tested it on WinServer 2008R2)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FPowerShell%2FPowerShell%2Fissues%2F7815%23issuecomment-422801444&data=02|01|pete.gomersall%40nau.edu|9ac10d55de8d4969410f08d61e332e8e|27d49e9f89e14aa099a3d35b57b2ba03|0|0|636729602704981854&sdata=QqwAjSLWzP5Ipee33Pjkar%2BXvTszUYHkx%2FmTKzkDZSo%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FApYOWwGvfS8ffuQigJ8KMBkPlIt_808eks5uckV4gaJpZM4WuooI&data=02|01|pete.gomersall%40nau.edu|9ac10d55de8d4969410f08d61e332e8e|27d49e9f89e14aa099a3d35b57b2ba03|0|0|636729602704991873&sdata=SBGcxMUM5Xry8NTxyuV6%2FyQcmrxPU%2BVKLxgUuwRmsuI%3D&reserved=0.

Just an update that I can reproduce now and will start to look into it.


Fail to upload a 3.9MB text file to blob container by "Upload Block Blob..."

OS: Win10
Build Version: 20200616.1
Repro steps:
Create a blob container --> Right click it --> Select "Upload Block Blob..." --> Select a 3.9MB text file.
Expect:
Upload this text file to blob container successfully.
Actual:
Upload this text file to blob container failed.

More info:
Please get the detailed upload result from images:

Closing in favor of https://github.com/microsoft/vscode-azurestorage/issues/686


Benchmarking Batch Inference

Hello 🙌🏼 I'm running some benchmarks on TinySAM, trying to benchmark batch inference. However, I hit a wall during batch inference. All inputs are torch tensors with the shapes expected by docstrings in inference code:
point_prompt.shape # torch.Size([4, 1, 2]) BXNX2
input_label.shape # torch.Size([4, 1]) BXN
batched_image.shape # torch.Size([4, 3, 1024, 1024]), BCHW

predictor.set_torch_image(batched_image, original_image_size=batched_image[0, 0, :, :].shape) # goes well

# this fails
with torch.no_grad():
        _, _, _ = predictor.predict_torch(
        point_coords=point_prompt,
        point_labels=input_label)

I don't really have a lot of time to debug this as I already did couple of steps, I feel like I'm missing a step, can you let me know if so? I can post a full trace if you want but I really feel like I'm missing a step and hence it errors out.

Hi merveenoyan,
Thanks for your attempt to benchmark TinySAM. The original interface of SAM does not support batch inference on multiple images, and we‘ve followed this design. From the note of set_torch_image and predict_torch, we can find that,
def set_torch_image(
        self,
        transformed_image: torch.Tensor,
        original_image_size: Tuple[int, ...],
    ) -> None:
"""
Arguments:
          transformed_image (torch.Tensor): The input image, with shape
            1x3xHxW, which has been transformed with ResizeLongestSide.
"""

in which the shape of image is set as 1x3xHxW. And in predict_torch
def predict_torch(
        self,
        point_coords: Optional[torch.Tensor],
        point_labels: Optional[torch.Tensor],
        boxes: Optional[torch.Tensor] = None,
        mask_input: Optional[torch.Tensor] = None,
        return_logits: bool = False,
    ) -> Tuple[torch.Tensor, torch.Tensor, torch.Tensor]:
        """
        Predict masks for the given input prompts, using the currently set image.
        Input prompts are batched torch tensors and are expected to already be
        transformed to the input frame using ResizeLongestSide.

        Arguments:
          point_coords (torch.Tensor or None): A BxNx2 array of point prompts to the
            model. Each point is in (X,Y) in pixels.
          point_labels (torch.Tensor or None): A BxN array of labels for the
            point prompts. 1 indicates a foreground point and 0 indicates a
            background point.
        """

There is batch dimension for point_coords and point_labels, which means multiple points in one image. So the batch inference of SAM/TinySAM only supports multiple prompts for one image, not for multiple images.
As for benchmark, I think it is possible to eval SAM/TinySAM under the same batch settings since we have the same interface. So there is no necessity to implement batch inference for multiple images.

@Gaffey thanks a lot for the swift response! I'll keep this in mind 🙌🏼


Implement Rename Functionality

Use Enter as a keyboard shortcut similar to Finder on macOS

i think enter should open. like google drive. however im not sure what the best alternative is. f2 works (also like gdrive).

@mustafa0x what does "open" even mean in this context? expand the folder?

Yes, navigate to folder if folder, or open file/document if file.

I don't think the library should be aware of file-opening functionality. Maybe it could be exposed through an onaction prop where you can perform certain tasks when the item is "activated".

Agreed. 👍


resolve: Fix AssetInfo after update to go-bindata-assetfs #7

Great. Thank you!

No problem.  Thank you!
On Tue, Jan 5, 2016 at 1:48 AM Ivan Daniluk<EMAIL_ADDRESS>wrote:

Great. Thank you!
—
Reply to this email directly or view it on GitHub
https://github.com/divan/gobenchui/pull/8#issuecomment-168926669.


New Simplified SC2EGSet Processing Pipeline

Is your proposal related to a problem?
Currently the process of updating SC2EGSet Dataset is lengthy and requires deep understanding of the toolset. Based on the recently introduced changes to this repository and SC2InfoExtractorGo, currently it should be possible to simplify the entirety of the processing pipeline.
Describe the solution you'd like
Changes to the makefile need to be made.

This pipeline needs to include packaging the sc2reset and sc2egset


ie10: console6000, warning: Unable to decode image: DataCloneError

Hi all, first of all, PDF.js is a awesome plugin that helped me solve batch of problems.
Recently, I have an issue is that when I render pdf in ie10, the console shows me a warning: Dependent image isn't ready yet, and some images in pdf are not rendered to canvas, other modern browsers (Chrome, Firefox) are both okay, any one can help me deal with this issue? thanks a lot.

Please provide steps to reproduce to issue and provide link to the PDF and/or your web site.

Closing until more information as requested above is provided. Please read https://github.com/mozilla/pdf.js/blob/master/CONTRIBUTING.md for filing an issue too.

Can I have this re-opened if I provide a pdf?
http://www.stevenaverycase.org/wp-content/uploads/2016/01/Jury-Trial-Transcript-Day-3-2007Feb14.pdf

Sure, reopening.

Thanks, let me know if you need anymore information.

waiting for the result.

Any solutions?

This looks to be a regression with how post message transfer support works. IE10/11 doesn't support transferring typed arrays and it looks like we don't disable the transferring.
Temporary workaround is to set PDFJS.postMessageTransfers = false;

The temporary work around did not work for me on a IE10/Windows8 Microsoft test VM, but the fix in PR #7088 did.

@brendandahl The temporary workaround did not work for me either.

@brendandahl The temporary workaround did not work for me either.

There were two issues at play. Use the real solution in #7088.


Pyro update

About the PR

adds four (!) new chems: pyroxadone, ardthrynkole, natusixadone, and hypothium. the former three are alternatives to cryogenics, requiring medical to replace their freezer with a heater to use. pyroxadone works the same as cryox, while ardthrynkole and natusixadone are meant to be used on frozen or hollowed corpses, respectively. this also removes doxa's worksOnTheDead component, meant to discourage medical lings from roundstart eating corpses then healing them back up. hypothium is a side grade to healium, and gives its condensed version a use. condensed healium also very very minorly heals now, so it does Something when condensed.
Why / Balance

there's no chem to specifically meant to heal frozen corpses, excluding necrosol. ardthrynkole fills that niche, and introduces a subset of chems for medical to learn. medical lings should also work a little for their free food, so natusixadone replaces doxa's niche for hollowed bodies
Technical details

yml changes yayyy!
Media

Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl:

add: adds three new "pyrogenic" chemicals: pyroxadone, ardthyrnkole, and natusixadone! check the guidebook for what they do!
tweak: doxa no longer works on hollowed corpses
tweak: condensed healium very minorly heals now, and can be used to make a new chem: hypothium!

i bumped it up to 6/6/6/6. only thing to consider is that 6/6/6/6 heals more than ichor, which imo is harder to get than hypothium

lowk just wanna merge it


treeInstance is not populated

It looks like the directive's scope attribute does not include a tree attribute. The calling controller's property that should point to an instance of jsTree is instead null, even in the ready callback.
The test to test that this item is populated do pass, so maybe I am missing something.

It is working... you can check the demo code also.. probably some scopes problem you have in your code.

I checked it with the demo code; you are right, it is populated. Probably has to do with scopes. I will close this because the other issue also has to do with scopes #75

I'm developing in TypeScript. In the compiled javascript, the "this" context for the jsTree callback events is the jsTree dom element. I'm diggin in now to see what I can understand about the compiled javascript that is causing this.


API get request for version 's response has  commit property with unknown value

Please check why the API Get Request Version /api/version returns a response body with these values
"commit": "unknown ",
"serviceName": "fightpandemics",
"version": "1.0.0"

should commit property be associated with any different value besides the 'unknown' one and please check for the reason of having this property 'commit'?
Here is what we create the version.test.js to test the Get version response
version.test.js
const httpStatus = require("http-status");
const APP_URL = process.env.MOCHA_URL;
const apiEndPoints = require("../utils/apiEndpoints");
const apiHelper = require("../utils/apiHelper");
const validator = require("../utils/validators");
let apiEndPoint = apiEndPoints.versionEndpoint;
describe("GET Version endpoint test", () => {
it("sending GET request with endpoint, status : 200 OK with response body", async () => {
let response = await apiHelper.sendGETRequest(APP_URL, apiEndPoint);
validator.validateResponse(response, {
statusCode: httpStatus.OK,
});
validator.validateResponse(response.body, {
  commit: "unknown",
  serviceName: "fightpandemics",
  version: "1.0.0",
});

});
});
And it passed when comparing commit property with unknown.

Hi @adorapham ! I just saw this and was wondering what value(s) is the "commit" property supposed to return?
Thank you!

Hi @adorapham ! I just saw this and was wondering what value(s) is the "commit" property supposed to return?
Thank you!

@ddveloper @rabbi08 @Zaydme can any of you guys explain what value(s) is the "commit" property supposed to return?
Thank you!

@ddveloper @rabbi08 @Zaydme can any of you guys explain what value(s) is the "commit" property supposed to return?
Thank you!

I believe this value is supposed to be the GIT_COMMIT hash value (the output of git rev-parse HEAD), it seems the workflows has something relevant (search TF_VAR_commit_hash for details) and in the backend Dockerfile, trying to overwrite this value. something seems not connected well during the build process

I believe this value is supposed to be the GIT_COMMIT hash value (the output of git rev-parse HEAD), it seems the workflows has something relevant (search TF_VAR_commit_hash for details) and in the backend Dockerfile, trying to overwrite this value. something seems not connected well during the build process

Hi @ddveloper. Thanks for the tips. I might need help with this PR as I'm not too familiar with the workflow and I'm still learning docker. I'll look into it and if I'm not able to resolve the issue, maybe someone else can help or take it over.

Hi @ddveloper. Thanks for the tips. I might need help with this PR as I'm not too familiar with the workflow and I'm still learning docker. I'll look into it and if I'm not able to resolve the issue, maybe someone else can help or take it over.


Add MetaTags Keywords and Descriptions in API Docs in Central

Need to add metatags keywords and descriptions in the new API Docs after moving them to Ballerina Central.
[1] https://docs.central.ballerina.io/ballerinax/stan/1.0.2

This issue is fixed. Hence closing.


TextField auto-scroll

If you have an input text field and enter more characters than the field supports, the text will trail off the end bounds and not render because it doesn't horizontally scroll. Noting that I have currently only tested this in Android target.
repro:

Create an input field.

  var tf = new openfl.text.TextField();
  tf.type = TextFieldType.INPUT;

Enter a value longer than the field can provide. In this example<EMAIL_ADDRESS>Observe  [myemail@tes] // text is cut off
Expected<EMAIL_ADDRESS>// text scrolls to show end

Then as a follow up (will write up a separate bug to describe further), the caret render position (and highlighted text start/end index) on a field that has scrolled is offset.

https://trello.com/c/5quOPFtW/29-add-support-for-auto-scrolling-in-input-textfields


[WIP] Random 1 on 1 bot first round implementation

@StevenMaio @Bajoukx @Weierstrash Hey this is going to be ready in just a few days (major things that are left to do: write basic tests + implement the pairings and participants APIs).
Defo get a look at it when you get the chance

@Random1on1s/devteam still trying to work on three things, but review would be great
1/ have to get the tests running on my computer (might by a my computer issue thing tho -- still looking into it)
2/ have to add serialization/deserialization tests for the config and pairings classes
3/ have to add tests for history channel logic on filtering out dry run pairings and pairings outside of a certain time range
Please tho take some time to look over and make any comments / change requests / feel free to PR changes directly to the branch too


cmd/go: allow declaring additional test dependencies

Quoting src/cmd/go/internal/test/test.go:
// TODO(rsc): How to handle other test dependencies like environment variables or input files?
// We could potentially add new API like testing.UsedEnv(envName string)
// or testing.UsedFile(inputFile string) to let tests declare what external inputs
// they consulted. These could be recorded and rechecked.
// The lookup here would become a two-step lookup: first use the binary+args
// to fetch the list of other inputs, then add the other inputs to produce a
// second key for fetching the results.
// For now, we'll assume that users will use -count=1 (or "go test") to bypass the test result
// cache when modifying those things.

See also #22583.

One downside to the specific API in the TODO is that it doesn't handle tests that glob a directory: if a file appears or disappears, the test should be rerun, but listing specific files to watch does not account for that.

Thanks for pointing out your TODO, Russ; I'd totally missed that in my glancing through the code/CLs.
Using our test base an example, the "second key" you refer to would need to be a function of:

Environment variables
Directory contents (glob-able):
Files are in the git index (glob-able)

Filepaths are either relative or resolved via environment variables.
Such a "second key" applies to both individual tests but also a number of tests (not necessarily in the same file or package).
Could we therefore follow some sort of declarative pre-amble in a test (or test file in order to apply to all tests in that file)?
import "testing"

func TestExtDeps(t *testing.T) {
    t.Deps(
        t.DepsDir("testdata") // relative
        t.DepsGitFiles("$ENV_VAR/**/testdata/blah.txt") // glob env-resolved git files
        // ...
    )

    // ...
}

For now, we'll assume that users will use -count=1 (or "go test") to bypass the test result cache when modifying those things.

As I mentioned in my golang-dev response, -count=1 has the unfortunate side effect of bypassing the result cache for all tests in package list mode, despite (in our case) that only representing ~5% of tests. What are your thoughts on flagging tests to bypass the cache?
import "testing"

func TestRegular(t *testing.T) {
    // will be part of the result cache

    // ...
}

func TestThatBypasses(t *testing.T) {
    // flag that this test's result should not be cached, i.e. re-run each time
    t.BypassCache()

    // ...
}

Or if we had something like the the .Deps approach then you could easily have a random dep:
import "testing"

func TestExtDeps(t *testing.T) {
    t.Deps(
        t.DepsRandom()
    )

    // ...
}

This issue / proposed API doesn't deal with the external dependencies such as data in non-filesystems, such as databases.

Tests that use testing/quick also need some thinking about. I'd certainly expect tests that use randomness to be re-run, or perhaps resume the same rng state and run fewer iterations.

Obviously we need to fix this in some way for Go 1.10, for test caching to make a good first impression. Equally obviously, we need to fix it soon, since we are into the freeze.
The main problem with "just assume all of testdata" is that it's incomplete: we have tests even in the standard library that read files outside testdata (for example compress/flate reads compress/testdata, not compress/flate/testdata), and I'm sure others do too. Environment variables were also raised, although I think they are clearly less common than testdata.
The main problem with adding separate new API (testing.ThisTestDependsOnThatThing) is that it means people's first interaction with test caching is finding out that test caching breaks their testing workflow and then are forced to go through and add redundant annotations to their code to unbreak it.
Because neither of those really seems tenable to me, my inclination now is to take @bradfitz's suggestion on #22583 and arrange via a new internal-only package for package os and package testing to conspire to report which environment variables and which files and directories (restricted to within $GOROOT and $GOPATH) are consulted by a particular a test execution. That handles all tests that read testdata, tests that name testdata something else (not that I care a lot about supporting that case), tests that read from nearby testdata, tests that are sensitive to environment variables, and so on, all without redundant annotations that will not stay up to date.

It is not a goal to address every last possible incidental input to a test that might change its behavior. Tests that depend on network servers or databases are still typically not worth rerunning if the code in the test binary has not changed: if I modify encoding/json then it's still useful to be able to 'go test all' and only actually re-run the networked tests that use json. Of course if you are actively changing the records in the database accessed by the test, then you will not want the test to be cached, but we have clearly documented ways to disable the test cache in that case ("go test" with no package arguments, or "go test -count=1 mypkg"). The same goes for tests that use pseudo-random generators or query the system time or the system randomness source. If the tests are good tests, then these incidental inputs should not change the outcome, even if they do make each test invocation explore slightly different test cases. If your goal is to run the test many times precisely because you know it might vary each time, then again it's easy to avoid the caching. (And in fact you might naturally reach for something like -count=1000, which will disable the caching without you even thinking about it.)

Because neither of those really seems tenable to me, my inclination now is to take @bradfitz's suggestion

👍
Is it possible to do something clever with calls to path/filepath.Glob? Or would, as you pointed out before, globs be a known and documented exception?

... restricted to within $GOROOT and $GOPATH ...

Just to confirm (selfishly, given that it's our setup) that multi-value GOPATH will be covered by this?
And just to pick up on @cespare's comment (because this the case in our situation too):

In a few cases we have test fixtures that are shared between a few different projects (not all of which are Go). In these cases the fixtures are symlinked into testdata.

Can the containment test be pre-symlink evaluation?

It is not a goal to address every last possible incidental input to a test that might change its behavior...

👍
I think with environment variables and files we have at least two mediums by which a coordinating process that is mutating the state of a database/server/other can communicate a hash of that state to a Go test, so to my mind most cases here are covered too.

Could tests explicitly opt out of caching? t.DoNotCache() would be clear and handle many cases.
This could mean "you may cache this test's compilation, but re-execute it even if you think it hasn't changed."

Would syscall tracing follow spawned processes? We are often testing process coordination (Go code calling another binary that spawns R running some specific interpreted script).

@spenczar I addressed that in the second half of https://github.com/golang/go/issues/22593#issuecomment-343012588. Whether caching is valid is never a property of the test itself. It's a property of your model for what you've changed recently and want to test.

@rsc I am thinking of cases in which the Go tool is not capable of knowing whether something changed. Particularly bulky integration tests that use the network: transient network blips can make a test fail; I'd like to retry that test, not assume it's always going to fail because my code hasn't changed.

I realize you spoke to my concern above, @rsc, when you said this:

Of course if you are actively changing the records in the database accessed by the test, then you will not want the test to be cached, but we have clearly documented ways to disable the test cache in that case ("go test" with no package arguments, or "go test -count=1 mypkg")

I don't like that this would mean go1.10 requires that I execute my tests differently. Most of my tests are run automatically by build systems, smoke testing systems, and so on. Requiring that the tests be executed in a different way means that I'll need to change code. It's really the same problem as you mentioned with an API expansion:

The main problem with adding separate new API (testing.ThisTestDependsOnThatThing) is that it means people's first interaction with test caching is finding out that test caching breaks their testing workflow and then are forced to go through and add redundant annotations to their code to unbreak it.

It seems like we're going to need to change some code somewhere to unbreak tests if caching is enabled. It's either in the test runners or its in the tests themselves. I think changing the tests themselves sounds much more obvious to the reader, much clearer, and much harder to mess up when bringing new people onto a codebase. I don't want to have to educate people on how to run tests for my particular project - I want go test ./... to just work regardless of what project they're working on. Annotating tests emphasizes clarity over cleverness, which seems right to me.

@spenczar I missed in your most recent message why "go test ./..." does not just work for people working on your project. Perhaps the reason is in your previous message:

Particularly bulky integration tests that use the network: transient network blips can make a test fail; I'd like to retry that test, not assume it's always going to fail because my code hasn't changed.

See the documentation (go help test): "In package list mode, go test also caches successful package test results."

@rsc Thank you, I did miss that very important fact! That is enough to satisfy me.


Files opened by "Copy to" on iOS are not re-openable

On iOS, files can be opened from other apps in two ways:

"Open in MyApp"

Received in application(_:open:options:) as an "open in place" security-scoped resource
options map contains UIApplicationOpenURLOptionsOpenInPlaceKey: true

"Copy to MyApp"

Received in application(_:open:options:) as a regular (non-security-scoped) file URI
File is already copied to app container, so can always be persistently accessed
options map contains UIApplicationOpenURLOptionsOpenInPlaceKey: false in my observation (could hypothetically have nil or the key might be absent)

Previously, I thought that with correct setup (proper keys in Info.plist), all apps would be able to "Open In", but it turns out that some, in particular Google Drive, can never "Open In" but only offer "Copy To", regardless of the target application and its capabilities.
file_picker_writable currently handles receiving "Copy To" files OK: it ignores the failure to obtain a security scope in _prepareUrlForReading:
https://github.com/hpoul/file_picker_writable/blob/72470d06f366039c568cd216fb5ccc6608cb9326/ios/Classes/SwiftFilePickerWritablePlugin.swift#L188-L198
But when trying to reopen a bookmark obtained from such a file, it fails due to strict checking in readFile:
https://github.com/hpoul/file_picker_writable/blob/72470d06f366039c568cd216fb5ccc6608cb9326/ios/Classes/SwiftFilePickerWritablePlugin.swift#L104-L106

Now this PR fixes #3 as well.

Thanks for merging. But somehow I don't see it in the master branch.

oops sorry.. no idea how that happened.. i think i've did one force push too many..
thanks for catching that 🤦


Added endpoint to delete link

Closes #154
Summary
Added an endpoint to delete a link
Local Tests
Added unit tests
Deleted a link that was previously removed with link s1 s3 down on mininet console.

Other than that, please explore these cases too:

I explored those cases. In all of them the results were as described.

Also, let me know if you had positive outcomes with the local/exploratory tests suggested on https://github.com/kytos-ng/topology/pull/179#pullrequestreview-1765343791

Yes, I tested these tests. They all had positive outcomes.

I created the issue and I think I am going to be able to finish it for this version/

I created the issue and I think I am going to be able to finish it for this version.

Cool. Appreciated.


Fixed a bug in dispatchRequest

URL parameters were not parsed for the first handler.
Added the missing line.

Merged it, thank you.


Update CI documentation to use parallelism

I did this for one of my repos and it halved the time it took to run, so figured I'd see if you'd want it in the documentation too.

Thanks, that makes sense.  I could take another step beyond that and have -j auto or something...


nuxt/axios - bug fix, function serialization with babel

This is a bug fix for function serialization when using nuxt as middleware with babel
https://github.com/nuxt-community/modules/issues/106

Thanks!

3.1.2 published 👯  Please don't manually bump versions :D

@pi0 how should I do it?

Actually, versioning is done using conventional commits and standard versions. (ie versions are generated using commit messages) See this resources if interested:

https://conventionalcommits.org
https://github.com/conventional-changelog/standard-version
https://lernajs.io

btw, should we fix nuxt too?
https://github.com/nuxt/nuxt.js/blob/dev/lib/app/router.js#L27

If you have time making a PR that would be awesome! (Also we can close related issue)

Oki, I will do it


Display custom message based on difference between datetimes: Momentjs

I am trying to format the date based on the difference between current and the input date:
Example:
Based on the difference
30 seconds ago(if its less than a minute)
13 minutes ago ( less than 60 minutes)
3 hours ago ( less than 24 hours)
40 days ago (more than 24 hours)
I tried the following, it does not format the way I want
let current = moment();
let input = "21-03-2020 10:34:25";
console.log(moment(input, "DD-MM-YYYY hh:mm:ss").fromNow());

Use duration


Feat/modifiers/v1

Resolves #23.
This PR adds the framework for modifiers to be attached to players for various things.

Was done with #44.


Charge rate reports wrong values

When the battery discharges wrong values get reported (MAX17048). The reason for that is that this register is i16 formatted in reality and has negative values when the battery discharges. But the library converts them to u16.
I know that it's an old repo, but I found this lib useful for me. Will you merge aa PR if I will make one with the fix :) ?

Hi, thanks for finding it. I would certainly welcome a PR fixing that.


stdio: remove reliance on std*** being aliasable

Various stdxxx implementations may not be aliasable, so remove the dependency on that.
This is blocking https://github.com/dlang/druntime/pull/1914

One question. In your comment on the Druntime PR, you say This [change] is specific to the Digital Mars C runtime.. In that light, would it be worth wrapping these changes in a version(DigitalMars) block and keeping the old code in all other cases, at least for now?

I like the idea of this, but I would rather have seen the numbers be an enum for clarity.

@schveiguy @WalterBright https://github.com/dlang/phobos/pull/5724


Compilation error on Unix/Linux since Calibri is a Windows font.

Tried compiling the .tex on my Mac and got the error about the Calibri font, so I changed the .cls to a Unix-compatible one.

@tolribeiro Unfortunately, I have not currently got a mac for testing. Could you please suggest a font name and size suitable for Mac? I mean the ones saving the resume proportions. Would it work with 10.5pt Helvetica?

Thank you for the comment. Fixed in Version 1.1.0. Please see the new related issue: https://github.com/dnl-blkv/mcdowell-cv/issues/4

Awesome! You're welcome Daniil, thank you for putting out such cool project. Keep up the good work, regards.


magic damage not working for players

i cant seem to figure it out, but spells like missle, or arrow_light, or wave, are breaking blocks and doing the abilities, but not doing any damage to mobs or players, ive been looking forever to see if i can figure it out, but havent had any luck. do you know by chance what im missing?

example of a spell. all these spells were working before at one point and then stopped recently
laser:
# This has been added automatically so that anything you remove here does not get inherited back in from the default configs
inherit: false
icon: bamboo{CustomModelData:18001}
icon_disabled: bamboo{CustomModelData:18002}
legacy_icon: spell_icon:145
legacy_icon_disabled: spell_icon_disabled:145
icon_url: http://textures.minecraft.net/texture/7cc8e096a2ac99bf264b3e827f1e3aa07ba688d7b4f470128f741132625e
example_thumbnail: https://thumbs.gfycat.com/SnivelingDependentHuman-size_restricted.gif
example_link: https://thumbs.gfycat.com/SnivelingDependentHuman
category: demolition
worth: 6500
earns_sp: 20
earns_cooldown: 10
upgrade_required_path: Ranger
upgrade_required_casts: 1500
pvp_restricted: false
actions:
cast:
- class: PlayEffects
effect: beam
ignore_result: true
-  class: Line
ignore_result: true
actions:
-  class: Disc
actions:
- class: CheckBlock
actions:
- class: BreakBlock
- class: PlayEffects
effects: burn
ignore_result: true
-  class: Repeat
actions:
- class: Delay
- class: Retarget
actions:
- class: PlayEffects
effect: beam
ignore_result: true
- class: Line
ignore_result: true
actions:
-  class: Disc
actions:
- class: CheckBlock
actions:
- class: BreakBlock
- class: PlayEffects
effects: burn
ignore_result: true
- class: ConeOfEffect
actions:
-  class: Damage
effects:
cast:
- sound: entity_guardian_attack
sound_pitch: 0.5
sound_volume: 0.5
- sound: magic.zap_crack
location: origin
- sound: block_beacon_activate
sound_pitch: 0.8
sound_volume: 0.4
- sound: block_conduit_activate
sound_pitch: 0.5
sound_volume: 0.9
beam:
-  sound: entity_guardian_attack
sound_pitch: 2
sound_volume: 2
-  sound: entity_guardian_attack
sound_pitch: 1.5
sound_volume: 1.6
location: target
-  class: EffectSingle
sound: magic.zap
location: origin
sound_volume: 0.1
color_override: color
effectlib:
class: LineEffect
particle: redstone
iterations: 2
color: ff0000
particle_offset_x: 0.05
particle_offset_y: 0.05
particle_offset_z: 0.05
-  location: origin
effectlib:
class: LineEffect
particle: redstone
iterations: 2
color: ff0000
burn:
- location: target
particle: smoke_large
particle_count: 4
particle_offset_x: 0.1
particle_offset_y: 0.1
particle_offset_z: 0.1
target_offset: 0.5,0.5,0.5
parameters:
target: other
target_type: LivingEntity
target_mob: true
fov: 0.1
close_range: 10
close_fov: 0.4
allow_max_range: true
player_damage: 1
entity_damage: 2
target_count: 5
damage: 2
range: 46
size: .5
start_at_target: true
increment_data: false
check_destructible: true
destructible: destructible
destructible_durability: 100
break_durability: 4
undo: 20000
transparent: all_air
radius: 0
cooldown: 10000
delay: 100
repeat: 12
undo_speed: 1
orient: true
axis: z
costs:
mana: 120
arrow: 3

Strange, normally that'd be from some PvP issue- but they don't hurt mobs either?
Could you please try doing /mage debug 20 to turn on debug logs in chat, try a simple spell like wound and see what it says?

yea so it says it did damage, but it actually doesnt

im pretty stumped

I'm stumped too- does it happen with Wound as well? That one does physical damage.. most spells use "magic" damage so if you had some plugin blocking thrown potions for some reason that might do it.

To be clear, testing with Wound would help since if that one works then we know it's a problem with thrown potion damage specifically.

ok i will check with wound here in a bit, sorry was at work today.

ok so wound does do damage. when used,

Ah ok well that's interesting news at least!
I can think of two possibilities then

Something on your server is blocking thrown potions- try a splash potion of harm, just vanilla, throw it at something? Could be anti-cheat or some kind of protection / pvp plugin maybe?

Maybe anti-cheat blocking damaging from a distance? Does magic missile work at very short range?

Another approach to take is, was this working at some point in the past? What changed between then and now? (Updated the plugin, server, added/removed plugins, etc)

you were right, i turned off all the anti cheats, and it worked, so ive figured out which ones are fine and everything is working again.
you are the best! lol i thought i checked to make sure they all would work, but part of the corex built into the factions plugin that updated was what it was. thanks for all your help.


HTTP ingester should support active polling

Allow for defining a config which specifies an HTTP endpoint that the ingester will periodically reach out to and pull.
Specify timing intervals.  Should be simple, use preprocessors to do heavily lifting/splitting of data.
Also allow for ignoring TLS cert errors (SHOULD NOT BE DEFAULT).

this should be something else entirely,


Hook Scripts

Allow user-defined hook scripts to be run before and after backups.

Moved to xml backlog.


listener print strings

Example
nsh> listener log_message

TOPIC: log_message instance 0 #1
log_message_s
        timestamp: 4128786 (55.251776 seconds ago)
        severity: 3
        text: "[pmw3901] driver start failed"

nsh> listener task_stack_info

TOPIC: task_stack_info instance 0 #1
task_stack_info_s
        timestamp: 128244081 (0.845543 seconds ago)
        stack_free: 428
        task_name: "px4io"

@julianoes can you try this again with your particular toolchain?

Builds fine with GCC 9.1.1 and clang 8.0.0.
I'll check at home on Arch as well later but I doubt it's newer.


Etcd Client Sends Invalid Authority Header

Overview
Operator Image Version: milvusdb/milvus-operator:v0.7.13
The etcd client being used by the operator is sending an invalid authority header when trying to connect to etcd. This causes issues when etcd is being hosted behind a proxy like Istio because the sidecar rejects the request. This results in the health check of the etcd cluster to fail with the following logs:
{"level":"warn","ts":"2023-06-27T14:48:14.903Z","logger":"etcd-client","caller":"v3@v3.5.0/retry_interceptor.go:62","msg":"retrying of unary invok
er failed","target":"etcd-endpoints://0xc000890540/#initially=[ds-milvus-etcd.default:2379]","attempt":0,"error":"rpc error: code = Unavailable de
sc = error reading from server: EOF"}
{"level":"warn","ts":"2023-06-27T14:48:15.933Z","logger":"etcd-client","caller":"v3@v3.5.0/retry_interceptor.go:62","msg":"retrying of unary invok
er failed","target":"etcd-endpoints://0xc000890540/#initially=[ds-milvus-etcd.default:2379]","attempt":1,"error":"rpc error: code = Unavailable de
sc = error reading from server: EOF"}
{"level":"warn","ts":"2023-06-27T14:48:16.963Z","logger":"etcd-client","caller":"v3@v3.5.0/retry_interceptor.go:62","msg":"retrying of unary invok
er failed","target":"etcd-endpoints://0xc000890540/#initially=[ds-milvus-etcd.default:2379]","attempt":2,"error":"rpc error: code = Unavailable de
sc = error reading from server: EOF"}
{"level":"warn","ts":"2023-06-27T14:48:17.993Z","logger":"etcd-client","caller":"v3@v3.5.0/retry_interceptor.go:62","msg":"retrying of unary invok
er failed","target":"etcd-endpoints://0xc000890540/#initially=[ds-milvus-etcd.default:2379]","attempt":3,"error":"rpc error: code = Unavailable de
sc = error reading from server: EOF"}
{"level":"warn","ts":"2023-06-27T14:48:18.899Z","logger":"etcd-client","caller":"v3@v3.5.0/retry_interceptor.go:62","msg":"retrying of unary invok
er failed","target":"etcd-endpoints://0xc000890540/#initially=[ds-milvus-etcd.default:2379]","attempt":4,"error":"rpc error: code = DeadlineExceed
ed desc = context deadline exceeded"}
2023/06/27 14:48:18 condition not ready EtcdReady False EtcdNotReady All etcd endpoints are unhealthy:[ds-milvus-etcd.default:2379:context deadlin
e exceeded]

Solution
The etcd client for the operator needs to be updated to a version greater than 3.5.0 see https://github.com/etcd-io/etcd/issues/13192

We changed default etcd version to 3.5.5


License?

I want to create a game that uses the Ring-Con. Could you please place this repo under an open-source license?

Hey, nice to see these crates get some use! I haven't thought about which license, what would be the license of your game?

I’ll probably end up using the MIT license. The engine I’ll be using (Godot is under the MIT license.

Done! Please keep me updated on your game!

Done! Please keep me updated on your game!

Alright, I have my first update. When I opened this issue, I didn’t know Rust at all. Since then, I’ve been learning Rust by going through parts of the book and working on projects unrelated to the game. One of those projects was a very amateurish statically generated Web site. (The static site generator ended up using very little Rust code in the end, but I still learned a lot).
I really appreciate the fact that you asked me to keep you updated. That’s what motivated me to create a Web site. I want to have a central place to post updates for anyone who was interested.
I wrote a blog post that details how chose joy in particular (as opposed to any of the other pieces of software that can read Joy-Con/Ring-Con inputs). I hope to have more updates soon!

Nice! To be honest, this project was in standby since I started https://github.com/Yamakaky/gyromouse, which is itself in standby! If you use it though, I can get back to it!

If you use it though, I can get back to it!

At the moment, all I ask is that you review any pull requests that I submit.


Write man page for "gps show"

See https://github.com/uptech/git-ps-rs/blob/main/doc/gps-add.1 for an example


Updating readme from branch example.

Updates
I updated the readme file. Please review and accept.

Thoughts
I probably shouldn't, but okay. 👍


Creating Atm using python

Please follow the readme.MD (Contribution guidelines). You need to maintain a proper file structure

ok


github.com/satori/go.uuid: CVE-2021-3538

Hi,
satori package contains this vulnerability: https://nvd.nist.gov/vuln/detail/CVE-2021-3538
Would it be possible to make a patch release containing an updated version of this module (or replace it with an alternative UUID module)?

Sure!

@amirgo1 , released v1.1.10

THANKS!!


use-latest-version and update-properties find different version

Given following pom snippet:
    <properties>
        <java.version>11</java.version>
        <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
        <spring-boot.version>2.1.3.RELEASE</spring-boot.version>
    </properties>

<dependencyManagement>
        <dependencies>
            <dependency>
                <!-- Import dependency management from Spring Boot -->
                <groupId>org.springframework.boot</groupId>
                <artifactId>spring-boot-dependencies</artifactId>
                <version>${spring-boot.version}</version>
                <type>pom</type>
                <scope>import</scope>
            </dependency>
        </dependencies>
    </dependencyManagement>
    <build>
        <pluginManagement>
            <plugins>
                <plugin>
                    <groupId>org.springframework.boot</groupId>
                    <artifactId>spring-boot-maven-plugin</artifactId>
                    <version>${spring-boot.version}</version>
                </plugin>
           </plugins>
        </pluginManagement>

Running `mvn update-properties' result in:
[INFO] --- versions-maven-plugin:2.7:update-properties (default-cli) @ demo-app ---
[INFO] artifact org.springframework.boot:spring-boot-dependencies: checking for updates from local-nexus
[INFO] Major version changes allowed
[INFO] Updated ${spring-boot.version} from 2.1.3.RELEASE to 2.1.5.RELEASE

Running mvn use-latest-version finds 2.1.6.RELEASE.
My expectation is that update-properties also find 2.1.6.RELEASE.

It seems, that the bug with 2.7.1 "use-latest-version removes/replaces the property" has been fixed with 2.8.0 via #340 / #396. Since then, use-latest-version doesn't touch the properties anymore, hence the pom is not modified.
The question now is: is this correct? update-properties vs. use-latest-versions
In case, you have a pom, where you use for some dependencies versions, but for other dependencies you use direct versions, then you'd need to call both goals to update all dependencies.
I can see the point, that when calling "use-latest-version", you expect that all your dependencies are updated to the latest version, regardless whether the version is specified directly or indirectly via property (in that case, the property should be updated, like with "update-properties").
Back to the original problem: the property spring-boot.version is used on two different artifacts: org.springframework.boot:spring-boot-dependencies (a dependency in dependencyManagement) and org.springframework.boot:spring-boot-maven-plugin(a plugin in pluginManagement). That maybe influences, which version is selected as the latest. However on search.maven.org versions 2.1.5.RELEASE/2.1.6.RELEASE of both artifacts are released on the same day (15-May-2019 / 19-Jun-2019). Maybe there is some caching ongoing? On the other hand, "update-properties" doesn't seem to care about plugins, so only "org.springframework.boot:spring-boot-dependencies" should have been checked and both goals should have been seeing the same latest version....
Hope that helps.


2346 deprecate wiki

Pull Request Test Coverage Report for Build 119094376

0 of 0   changed or added relevant lines in 0 files are covered.
382 unchanged lines in 11 files lost coverage.
Overall coverage decreased (-1.7%) to 80.451%

Files with Coverage Reduction
New Missed Lines
%

src/encoder.rs
1
77.94%

src/api/config/speedsettings.rs
5
66.88%

src/partition.rs
5
62.64%

src/cpu_features/x86.rs
6
50.0%

src/api/util.rs
7
43.33%

src/lib.rs
7
0%

src/rdo.rs
8
84.98%

src/deblock.rs
9
93.1%

src/api/config/encoder.rs
36
29.67%

src/context/block_unit.rs
37
84.61%

Totals

Change from base Build 119019642:
-1.7%

Covered Lines:
16239

Relevant Lines:
20185

💛  - Coveralls


Allow generation of depended grammars

setuptools-antlr searches for all grammars in the user workspace. Internally it builds up a grammar tree for all found grammars. Parsers are only generated for root level grammars. This behavior isn't changed also when the user passes the grammars to be generated explicitly on the command line or in a config file.
An option should be provided to also generate all depended grammars found the user workspace.

Won't be implemented. For details please see #37.


The dotnet test command fails randomly with codecoverage

Description
This issue is a continuation of #1926.
When we are using dotnet test with codecoverage the following error occurs randomly:
Testhost process exited with error: Unhandled Exception: System.MissingMethodException: Method not found: 'Void Microsoft.VisualStudio.CodeCoverage.Shim.1124859581__coverage_runtime_method__23574_20339__(UInt64)'.
   at Newtonsoft.Json.JsonSerializerSettings..ctor()
   at Microsoft.VisualStudio.TestPlatform.CommunicationUtilities.JsonDataSerializer..ctor()
   at Microsoft.VisualStudio.TestPlatform.CommunicationUtilities.JsonDataSerializer.get_Instance()
   at Microsoft.VisualStudio.TestPlatform.CommunicationUtilities.TestRequestHandler..ctor()
   at Microsoft.VisualStudio.TestPlatform.TestHost.DefaultEngineInvoker..ctor()
   at Microsoft.VisualStudio.TestPlatform.TestHost.Program.GetEngineInvoker(IDictionary`2 argsDictionary)
   at Microsoft.VisualStudio.TestPlatform.TestHost.Program.Run(String[] args)
   at Microsoft.VisualStudio.TestPlatform.TestHost.Program.Main(String[] args)

Steps to reproduce
Execute the following command until the error occurs:
dotnet test Solution.sln --logger trx --results-directory D:\W\JP\Git\Co\Wo\TestResults -c release --no-build --collect "Code coverage" --diag:log.txt
Expected behavior
The dotnet test command works well with codecoverage.
Actual behavior
The dotnet test fails randomly with codecoverage.
Diagnostic logs
Log.txt

@juanperezADD apologies for getting back so late on this, can you please share a small repro project for this? Also if possible can share log files for testhost process as well?

Closing this, please ping here again, with small repro project if possible, & log files for testhost process


Could you please check the swap configuration

Hi, in the "Swap configuration",  the doc shows the mounts configuration as below [1] :
[1]
mounts:

["ephemeral0.1", "/mnt"]
["ephemeral0.2", "none", "swap", "sw", "0", "0"]

But in another official doc(https://docs.microsoft.com/bs-cyrl-ba/azure/virtual-machines/linux/cloudinit-configure-swapfile), the mounts configuration is [2]:
[2]
mounts:

["ephemeral0.1", "/mnt"]
["ephemeral0.2", "none", "swap", "sw,nofail,x-systemd.requires=cloud-init.service", "0", "0"]

I tried them in my VM on Azure, the config[2] can make swap successfully. But for the config[1], there will be error log:
2021-05-25 10:49:35,779 - cc_mounts.py[DEBUG]: Changes to fstab: ['- ephemeral0.2 none swap sw,nofail,x-systemd.requires=cloud-init.service,comment=cloudconfig 0 0', '+ ephemeral0.2 none swap sw,comment=cloudconfig 0 0']
2021-05-25 10:49:35,779 - util.py[DEBUG]: Running command ['swapon', '-a'] with allowed return codes [0] (shell=False, capture=True)
2021-05-25 10:49:35,800 - cc_mounts.py[WARNING]: Activate mounts: FAIL:swapon -a
2021-05-25 10:49:35,800 - util.py[WARNING]: Activate mounts: FAIL:swapon -a
2021-05-25 10:49:35,800 - util.py[DEBUG]: Activate mounts: FAIL:swapon -a
Traceback (most recent call last):
File "/usr/lib/python3.6/site-packages/cloudinit/config/cc_mounts.py", line 530, in handle
util.subp(cmd)
File "/usr/lib/python3.6/site-packages/cloudinit/util.py", line 2088, in subp
cmd=args)
cloudinit.util.ProcessExecutionError: Unexpected error while running command.
Command: ['swapon', '-a']
Exit code: 255
Reason: -
Stdout:
Stderr: swapon: cannot open ephemeral0.2: No such file or directory
Could you please check whether the config[1] is incorrect?
Thanks!
Huijuan

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

ID: 48bab424-986d-9922-ec2d-942508081367
Version Independent ID: 6e9b3ebd-0571-357e-0fde-ef434a728a24
Content: Create and upload a Red Hat Enterprise Linux VHD for use in Azure - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/redhat-create-upload-vhd.md
Service: virtual-machines
Sub-service: redhat
GitHub Login: @danielsollondon
Microsoft Alias: danis

Hello @zhaohuijuan , thanks for taking time to share your valuable feedback! We are taking a look into this and will get back to you soon.

@danielsollondon Can you please help confirm? Thanks.

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Got it, thanks  a lot!

It looks like edits were made recently to fix the swap configuration. Closing for now.
#please-close


merge commit parent type issue

NOT FOR LANDING

Here's what I get when I try to run this. It seems like something surprising to me is happening when NewCommit() tries to calculate the appropriate type for the Parent field.
panic:                         	Error Trace:	assert_type.go:11
struct.go:53
commit.go:53
commit_test.go:70
Error:		Invalid type
Messages:	Set<Ref<struct Commit {
meta: struct {},
parents: Set<Ref<Cycle<0>>>,
value: Number,
}> | Ref<struct Commit {
meta: struct {},
parents: Set<Ref<struct Commit {
meta: struct {},
parents: Set<Ref<Cycle<0>>>,
value: Number | String,
}>>,
value: String,
}>> is not a subtype of Set<Ref<struct Commit {
meta: struct {},
parents: Set<Ref<Cycle<0>>>,
value: Number | String,
}>>


Feature request: Add "Edit as new ..."

Expected behavior
Add "Edit as new ..."

I need re-use and edit  an email every week due to the long distribution list.
Once an email has been sent and failed (e.g. typo) I need to compose a new one rather than correcting the old one and send it out again

In both case I prevent the use of the "reply" function by intention.
Maybe it's already implemented, but I'm not aware of it.
Root cause ... using an Android tablett with   default settings   by intention.    copy&paste respective text-editing is a challenging procedure without cursor keys.
Actual behavior
Use "reply" and remove manually all what the reply has added to the email.
Steps to reproduce
Not applicable
Environment
K-9 Mail version: 5.008
Android version: 4.0.3
Account type (IMAP, POP3, WebDAV/Exchange): unable to recover by the menus what account settings has been made ... it is one of IMAP or POP

Found the 50% solution by accident right now ... on the selected email use the pop-up menu. In difference to the regular menu the pop-menu comes with the entry "Erneut senden" (resend)
Unfortunately it seems the attachment is not copied to the new email.

At the second trial to edit the old email the attachment has been copied to the new email.
In difference to the first trial I stored the attachment.

Have checked the attachment top ... it seems my very trial is not showing the regular operation. Eveything trial I did to verify comes the attachment copied as expected.
Please ignore this attachment topic.

Changed the issue title due to the findings.
Would be ok for me if you close this issue.


Issue - Data Agreements

[ ] choose existing schema to be removed

[ ] After editing - the data source is changed

[ ]  if we edit a published data agreement and save it, it goes to draft. It should the current published one

[ ]  Ater changing the retention period and save the data agreement. Then viewd it, the retention period is wrong

[ ]  Updated the consent to contract, then we save it, after that viewed it, then published it -> contract changed to consent

[ ] On editing we add new attribute, then save it. On View it dissapears.

[ ]  Edited the data agreement to legal obligation -> saved -> published -> it changed to contract

[ ] create a data agreement -> add details -> save it -> edit the data agreement -> delete all the data attributes -> save it -> Publish from data agreement listing -> it is now showing the deleted attributes

[ ] On clicking Published, it doesn't show Published DA

@georgepadayatti @kashishmalik  As we discussed yesterday The 3rd issue will be drafted right? So can we remove that from the issue list?
"if we edit a published data agreement and save it, it goes to draft. It should the current published one"

@georgepadayatti Please review the scenario of handling draft and published revisions:
Scenario: A DA existing with version, say, 1.0

The admin chooses edit, begins to modify, and clicks save.

In this case, there shall be two DAs—one as draft (2.0). The DA list shall show one draft and one published for the same DA.

The admin now chooses to edit the draft and publish it.

Two DA revisions will be published in this case, but the latest is available to the user. The DA list shall show the latest published one.
Two additional possibilities:
#01: If the admin chooses to edit the original published one.
#02:Edit shall be available only for the latest published revision

Need clarification. This flow is confusing. Will require  some serious
changes in the architecture as well.
On Sat, 4 Nov 2023 at 9:33 PM, Lal Chandran @.***>
wrote:

@georgepadayatti https://github.com/georgepadayatti Please review the
scenario of handling draft and published revisions:
Scenario: A DA existing with version, say, 1.0

The admin chooses edit, begins to modify, and clicks save.

In this case, there shall be two DAs—one as draft (2.0). The DA list shall
show one draft and one published for the same DA.

The admin now chooses to edit the draft and publish it.

Two DA revisions will be published in this case, but the latest is
available to the user. The DA list shall show the latest published one.
Two additional possibilities:
#1
https://github.com/decentralised-dataexchange/bb-consent-admin-dashboard/pull/1:
If the admin chooses to edit the original published one.
#2
https://github.com/decentralised-dataexchange/bb-consent-admin-dashboard/pull/2:Edit
shall be available only for the latest published revision
—
Reply to this email directly, view it on GitHub
https://github.com/decentralised-dataexchange/bb-consent-admin-dashboard/issues/178#issuecomment-1793484161,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD5DNCBN4KOQN5Z2HXAEVUTYCZROPAVCNFSM6AAAAAA64OCFVCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJTGQ4DIMJWGE
.
You are receiving this because you were mentioned.Message ID:
<decentralised-dataexchange/bb-consent-admin-dashboard/issues/178/1793484161
@github.com>

How do you suggest in the short run? @georgepadayatti

How do you suggest in the short run? @georgepadayatti

@lalc I am sharing the different cases for data agreement create/update (do parse through them in sequential order)

Assuming you are in consensus to all these cases. The only change required is ability to see past revision from drop down. Past revisions can only be viewed and cannot be edited. If you delete a data agreement, you are deleting all the revisions.

Let's stick to this for now.
Question: Could we not have revision as x.y?

Let's stick to this for now.

You are in consensus correct ?

Question: Could we not have revision as x.y?
Meaning ?

Let's stick to this for now.

You are in consensus correct ?

Question: Could we not have revision as x.y?
Meaning ?

Let's stick to this for now.
I am unsure about the delete, but I can live with it for now. Pls state it in the deletion warning for now.

Can you suggest a text ? I can raise it as a separate issue to avoid missing it.

Let's stick to this for now.
I am unsure about the delete, but I can live with it for now. Pls state it in the deletion warning for now.

Can you suggest a text ? I can raise it as a separate issue to avoid missing it.

@lalc ☝🏼 This is the current text.

You are about to delete an existing data agreement and ALL its revisions. This action is not reversible. Please type DELETE to confirm and click DELETE.

@MARZOOQUE

Here is the revised warning text:
You are about to delete an existing data agreement and ALL its revisions. This action is not reversible. Please type DELETE to confirm and click DELETE.

@MARZOOQUE Drop down for version of data agreement should as shown below:


GPU will low memory

Thank you for your code.
I have 8 GPUs with 12 GB each. How can I modify the code to run training?

Thank you for your interests, you can try lower the batch size.

How many GPU memories does this paper need?


Fix gradle stuff.

Added gradle wrapper
Improved the .gitignore
Fixed dependencies in gradle
Updated the Readme

Ehh... Do you want me to change the .gitignore?

@Victorious3, here ya go!