line mappings aren't right with calls.sol's execution output

How to Reproduce
$ myth -x solidity_examples/calls.sol -munchecked_retval
==== Unchecked CALL return value ====
SWC ID: 104
Type: Informational
Contract: Caller
Function name: thisisfine()
PC address: 738
Estimated Gas Usage: 1386 - 35997
The return value of an external call is not checked. Note that execution continue even if the called contract throws.
--------------------
In file: solidity_examples/calls.sol:1

0.5.0;

contract Caller {

	ad

--------------------

==== Unchecked CALL return value ====
SWC ID: 104
Type: Informational
Contract: Caller
Function name: callstoredaddress()
PC address: 913
Estimated Gas Usage: 1430 - 36041
The return value of an external call is not checked. Note that execution continue even if the called contract throws.
--------------------
In file: solidity_examples/calls.sol:1

0.5.0;

contract Caller {

	ad

--------------------

==== Unchecked CALL return value ====
SWC ID: 104
Type: Informational
Contract: Caller
Function name: reentrancy()
PC address: 1049
Estimated Gas Usage: 1452 - 36063
The return value of an external call is not checked. Note that execution continue even if the called contract throws.
--------------------
In file: solidity_examples/calls.sol:1

0.5.0;

contract Caller {

	ad

--------------------

==== Unchecked CALL return value ====
SWC ID: 104
Type: Informational
Contract: Caller
Function name: calluseraddress(address)
PC address: 1160
Estimated Gas Usage: 1126 - 35407
The return value of an external call is not checked. Note that execution continue even if the called contract throws.
--------------------
In file: solidity_examples/calls.sol:1

0.5.0;

contract Caller {

	ad

--------------------

...
$

Expected behavior
It can clearly be seen that there is no call in line number 1
Environment

Mythril version: output from  develop
Solidity compiler and version: 0.5.0
Python version: 3.6.3
OS and Version: Ubuntu 17.10

Additional Environment or Context

Tried it on the 0.4.24 version and it worked perfectly, it seems there is a problem with the source mappings of 0.5.0

This seems to have been magically fixed :laughing: by some PR.


[Feature request] Redundant tag check

Suggest parent tags, e.g. 2girls implicates multiple girls

For the same reason I stated on your related tags request #110, this data is currently not easily accessible without using the official API and out of scope for an autocomplete script.


add temporal coverage as date type to discovery

This is a stale PR and not relevant anymore - so closing.


pragma no cover fails with elif

Hi Ned, et al.,
Following https://coverage.readthedocs.io/en/coverage-4.3.4/excluding.html#branch-coverage, I expect
    def test(self):
        if False:
            pass    # 1
        elif True:  # 2
            pass
        else:    # pragma no cover
            pass    # 3
        pass # 4

to not report that branch at # 2 fails to branch to the else. This works in the above example, but in a piece of production code it reports that the branch at # 2 fails to go to #4 instead:

unfortunately, I don't know how to reproduce this in a minimal snippet.
thanks
Arthur

Arthur, thanks.  This seems to reproduce it:
$ cat bug697.py
def run():
    a = 0
    for i in range(10):
        if 0 <= i <= 3:
            a += 1
        elif i > 3:
            a += 10
        else:   # pragma: no cover
            pass
        a += 0
    return a

print(run())

$ coverage run --branch bug697.py
64

$ coverage report -m
Name        Stmts   Miss Branch BrPart  Cover   Missing
-------------------------------------------------------
bug697.py      10      0      6      1    94%   6->10

The problem seems to be that the else:pass is optimized away, so the actual jumping seen by Python is to a line that is not excluded.  If we change the pass, then the problem goes away:
$ cat bug697_fixed.py
def run():
    a = 0
    for i in range(10):
        if 0 <= i <= 3:
            a += 1
        elif i > 3:
            a += 10
        else:   # pragma: no cover
            a += 1000
        a += 0
    return a

print(run())

$ coverage run --branch bug697_fixed.py
64

$ coverage report -m
Name              Stmts   Miss Branch BrPart  Cover   Missing
-------------------------------------------------------------
bug697_fixed.py      10      0      4      0   100%

This is related to a number of other issues reported against coverage that have to do with the peephole optimizer changing jumps to jumps.  Until Python gets a way to fully disable the optimizer, these problems will persist.  Commenting on this CPython ticket might push it forward: http://bugs.python.org/issue2506 "Add mechanism to disable optimizations".

Cool.
From: Ned Batchelder @.>
Reply-To: nedbat/coveragepy @.>
Date: Tuesday, November 2, 2021 at 6:43 AM
To: nedbat/coveragepy @.>
Cc: "Goldberg, Arthur P" @.>, Author @.***>
Subject: Re: [nedbat/coveragepy] pragma no cover fails with elif (#697)
USE CAUTION: External Message.
This is now fixed by Python 3.10
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_nedbat_coveragepy_issues_697-23issuecomment-2D957321344&d=DwMCaQ&c=shNJtf5dKgNcPZ6Yh64b-A&r=p26xYuDOPpwD_1KMz0HcEpDitWzbwzrc9iNKx0lkRN8&m=-9q2ZP7H5kOIag5U3rucrV7yHihRbSk8MDWHy1u1KQ4&s=yF-hhUnUeNEwggf42H6YqcEnwnjSF5hSe9bppRW6YFc&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AAAIIWRIZPXNF2TAX7T76IDUJ7FD5ANCNFSM4FR6TB2A&d=DwMCaQ&c=shNJtf5dKgNcPZ6Yh64b-A&r=p26xYuDOPpwD_1KMz0HcEpDitWzbwzrc9iNKx0lkRN8&m=-9q2ZP7H5kOIag5U3rucrV7yHihRbSk8MDWHy1u1KQ4&s=62y5KwbIHjL2j9Duo1rdSyI90G_QOWfv-cNWfSiudbc&e=.
Triage notifications on the go with GitHub Mobile for iOShttps://urldefense.proofpoint.com/v2/url?u=https-3A__apps.apple.com_app_apple-2Dstore_id1477376905-3Fct-3Dnotification-2Demail-26mt-3D8-26pt-3D524675&d=DwMCaQ&c=shNJtf5dKgNcPZ6Yh64b-A&r=p26xYuDOPpwD_1KMz0HcEpDitWzbwzrc9iNKx0lkRN8&m=-9q2ZP7H5kOIag5U3rucrV7yHihRbSk8MDWHy1u1KQ4&s=5a7wQ-n5bWOwp-3lw5U081qeFUbdH8HSAxgla9UkYQc&e= or Androidhttps://urldefense.proofpoint.com/v2/url?u=https-3A__play.google.com_store_apps_details-3Fid-3Dcom.github.android-26referrer-3Dutm-5Fcampaign-253Dnotification-2Demail-2526utm-5Fmedium-253Demail-2526utm-5Fsource-253Dgithub&d=DwMCaQ&c=shNJtf5dKgNcPZ6Yh64b-A&r=p26xYuDOPpwD_1KMz0HcEpDitWzbwzrc9iNKx0lkRN8&m=-9q2ZP7H5kOIag5U3rucrV7yHihRbSk8MDWHy1u1KQ4&s=sDeDVS0izDQw4J02ZnEmeKLYPTbMD-HN-Y1trhfEHq0&e=.


[Review] Fewer ActorSystems

This PR is built on top of #765 because it assumes Akka 2.4.7 (and therefore shouldn't be merged before it). Only the last two commits are specific to this PR.
Makes it more explicit where ActorSystems are created. Eliminates the creation of one spurious system in the invoker. Generalizes the practice of retrieving ExecutionContexts from ActorSystems when available.
Taking the actor system out of BasicHttpService is also a required step towards having cohabiting Spray and Akka HTTP API endpoints.

PPG 760

Includes all changes from #765.

PPG 766 for the freshly rebased version.

Merged as https://github.com/openwhisk/openwhisk/compare/5295f25514e4...1ebc8b261351.


self hosting

Any chance you can publish an images to docker's repo. I don't have any interested to host this on a cloud provider but would like to use what you've done as easy as possible.

You might want to have a look at hub.docker.com. I am using this one: https://hub.docker.com/r/chk1/slackin/

oh cool, thanks! I did a search for slackin, or at least I thought I did. Didn't see this.

I ended up installing it via npm (npm install --global slackin) and using the slackin command to run it, via supervisord:
[program:slackin]
command=slackin "example" --port=30861 --interval=60000
environment=SLACK_API_TOKEN=xxxxxxxxxxxx
user=www-data


Fix bug in form1 outline initial view

Codecov Report

Merging #234 (c3475e4) into master (8ab44bc) will decrease coverage by 0.05%.
The diff coverage is 73.33%.

@@            Coverage Diff             @@
##           master     #234      +/-   ##
==========================================
- Coverage   87.72%   87.68%   -0.05%
==========================================
  Files          78       78
  Lines        5435     5439       +4
==========================================
+ Hits         4768     4769       +1
- Misses        667      670       +3

Flag
Coverage Δ

unittests
87.68% <73.33%> (-0.05%)
:arrow_down:

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

Impacted Files
Coverage Δ

base/views/outline_initial_views.py
84.41% <73.33%> (-0.63%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[web][html]Applying ShaderMask on gradients makes the gradient not render

Putting a ShaderMask on a Container with some gradient decorations, works fine on mobile, desktop, and the web (when viewed from desktop Chrome and Safari) but does not work on mobile browsers.
For example, a minimal reproducible code:
ShaderMask(
  shaderCallback: (Rect bounds) {
    return LinearGradient(
      colors: [Colors.red, Colors.yellow],
    ).createShader(bounds);
  },
  child: Container(
    width: 300,
    height: 300,
    decoration: BoxDecoration(
      gradient: LinearGradient(
        colors: [Colors.grey, Colors.white],
      ),
    ),
  ),
)

When viewed everywhere else, it looks fine:

But when viewed from mobile, for example on Chrome on Android 12 (and also tested on iOS, same problem), it looks like this:

Flutter doctor:
[✓] Flutter (Channel stable, 2.8.1, on macOS 12.1 21C52 darwin-arm, locale en-CA)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
[✓] Chrome - develop for the web
[!] Android Studio - Unable to find bundled Java version.
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.64.2)
[✓] Connected device (3 available)

Hi @h65wang, please provide a complete reproducible sample we can run without having to make any changes, preferably without any third-party packages.
Thank you.

@danagbemava-nc Sure, just put the source code in a runApp() like this:
import 'package:flutter/material.dart';

void main() {
  runApp(
    ShaderMask(
      shaderCallback: (Rect bounds) {
        return LinearGradient(
          colors: [Colors.red, Colors.yellow],
        ).createShader(bounds);
      },
      child: Container(
        width: 300,
        height: 300,
        decoration: BoxDecoration(
          gradient: LinearGradient(
            colors: [Colors.grey, Colors.white],
          ),
        ),
      ),
    ),
  );
}

This is how to test it:

build a web release version using: flutter build web
host the web version somewhere
access it from a desktop browser (it works fine)
access it from a mobile browser (it renders black, I've tested Chrome on Android 12 and iOS 15).

It just hit me: the main difference between Desktop and Mobile browsers is that, by default, Flutter Web automatically detects the platform and chooses to use either canvas kit or html renderer.
Therefore, this issue is actually a html renderer issue. It works with canvas kit. So I changed the issue title.
To reproduce, force it to run with html renderer: flutter run -d chrome --web-renderer html and it will crash.

Issue is reproducible on stable and master.
When running in debug mode, an error is thrown and nothing is rendered in the browser. Whereas running in release mode, just a black square is drawn.
This works as expected using the canvaskit renderer.

screenshots

HTML
Canvaskit

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

void main() {
  runApp(
    MaterialApp(
      home: Builder(builder: (context) {
        return Scaffold(
          body: ShaderMask(
            shaderCallback: (Rect bounds) {
              return LinearGradient(
                colors: [Colors.red, Colors.yellow],
              ).createShader(bounds);
            },
            child: Container(
              width: 300,
              height: 300,
              decoration: BoxDecoration(
                gradient: LinearGradient(
                  colors: [Colors.grey, Colors.white],
                ),
              ),
            ),
          ),
        );
      }),
    ),
  );
}

flutter doctor -v
[✓] Flutter (Channel stable, 2.10.2, on macOS 12.2 21D49 darwin-arm, locale en-GB)
    • Flutter version 2.10.2 at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 097d3313d8 (9 days ago), 2022-02-18 19:33:08 -0600
    • Engine revision a83ed0e5e3
    • Dart version 2.16.1
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • 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 version 1.11.2

[✓] 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 (the doctor check crashed)
    ✗ Due to an error, the doctor check did not complete. If the error message below is not helpful, please let us know about this issue at https://github.com/flutter/flutter/issues.
    ✗ FormatException: Unexpected extension byte (at offset 5)
    • #0      _Utf8Decoder.convertSingle (dart:convert-patch/convert_patch.dart:1789:7)
      #1      Utf8Decoder.convert (dart:convert/utf.dart:351:42)
      #2      InputStream.readString (package:archive/src/util/input_stream.dart:207:30)
      #3      new ZipDirectory.read (package:archive/src/zip/zip_directory.dart:40:30)
      #4      ZipDecoder.decodeBuffer (package:archive/src/zip_decoder.dart:19:30)
      #5      ZipDecoder.decodeBytes (package:archive/src/zip_decoder.dart:14:12)
      #6      IntelliJPlugins._findPluginXml (package:flutter_tools/src/intellij/intellij.dart:130:44)
      #7      IntelliJPlugins._readPackageVersion (package:flutter_tools/src/intellij/intellij.dart:141:40)
      #8      IntelliJPlugins.validatePackage (package:flutter_tools/src/intellij/intellij.dart:63:35)
      #9      IntelliJValidator.validate (package:flutter_tools/src/intellij/intellij_validator.dart:103:15)
      #10     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:32)
      #11     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:109:18)
      #12     _rootRun (dart:async/zone.dart:1426:13)
      #13     _CustomZone.run (dart:async/zone.dart:1328:19)
      #14     _runZoned (dart:async/zone.dart:1861:10)
      #15     runZonedGuarded (dart:async/zone.dart:1849:12)
      #16     runZoned (dart:async/zone.dart:1780:12)
      #17     asyncGuard (package:flutter_tools/src/base/async_guard.dart:109:3)
      #18     Doctor.startValidatorTasks (package:flutter_tools/src/doctor.dart:205:9)
      #19     Doctor.diagnose (package:flutter_tools/src/doctor.dart:309:47)
      #20     DoctorCommand.runCommand (package:flutter_tools/src/commands/doctor.dart:50:48)
      #21     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1320:12)
      <asynchronous suspension>
      #22     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1161:27)
      <asynchronous suspension>
      #23     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #24     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
      <asynchronous suspension>
      #25     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
      <asynchronous suspension>
      #26     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #27     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
      <asynchronous suspension>
      #28     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
      <asynchronous suspension>
      #29     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #30     main (package:flutter_tools/executable.dart:94:3)
      <asynchronous suspension>

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

[✓] Connected device (3 available)
    • Nexus (mobile)  • 00008020-001875E83A38002E • ios            • iOS 15.3 19D50
    • macOS (desktop) • macos                     • darwin-arm64   • macOS 12.2 21D49 darwin-arm
    • Chrome (web)    • chrome                    • web-javascript • Google Chrome 98.0.4758.109

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

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 2.11.0-0.0.pre.697, on macOS 12.2 21D49 darwin-arm, locale en-GB)
    • Flutter version 2.11.0-0.0.pre.697 at /Users/nexus/dev/sdks/flutters
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fc2a0847fb (43 minutes ago), 2022-02-28 01:01:17 -0500
    • Engine revision 05d1b5a66b
    • Dart version 2.17.0 (build 2.17.0-150.0.dev)
    • DevTools version 2.11.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • 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 version 1.11.2

[✓] 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.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin version 213.5744.122

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

[✓] Connected device (3 available)
    • Nexus (mobile)  • 00008020-001875E83A38002E • ios            • iOS 15.3 19D50
    • macOS (desktop) • macos                     • darwin-arm64   • macOS 12.2 21D49 darwin-arm
    • Chrome (web)    • chrome                    • web-javascript • Google Chrome 98.0.4758.109

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

• No issues found!

run_log.txt

This was fixed in the above PR.


Unrecognized option "openai_key" under "ignition".

IgnitionExtension makes use of openai_key, but this config is not defined at https://github.com/spatie/symfony-ignition-bundle/blob/main/src/DependencyInjection/Configuration.php which leads to crash at attempt to specify this key.
https://github.com/spatie/symfony-ignition-bundle/blob/4c958abc0619513cfe71f092e216765a49fbb43b/src/DependencyInjection/IgnitionExtension.php#L32

Mmm, that is strange since we use the null coalescing operator in the code that you link to.
Do you have a full stack trace for us to look at?


Change "dot operator" to "dot notation"

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes #15646

Description

Change "dot operator" to "dot notation" as proposed in issue #15646 to be consistent with MDN.

Will you be able to update this @Bouzmine?

@Bouzmine I'm excited to merge this PR. Can you take a look at @dhcodes's review and make the changes real quick?

LGTM. Thanks for updating this @Bouzmine, and thanks for contributing to freeCodeCamp! 👍 💯 🌮 🦅

@Bouzmine Thanks again for your contribution!


successfully compiled but no connection [Windows 11, Docker]

Hi
I pulled the repo using the 'use this template' function today. Then followed the docker installation instructions on a Windows 11 machine.
The only change I've made so far is deleting the post with the diagram.
After compilation, everything seems to be fine. No errors throughout.

But when I clicked on the link, it does not connect:

I also tried updating the docker-compose.yml file following a solution I saw in another issue but no luck:
version: "3"

services:
  jekyll:
    image: amirpourmand/al-folio
    command: bundler exec jekyll serve --watch --port=8080 --host=<IP_ADDRESS> --verbose
    ports:
      - 8080:8080
    volumes:
      - .:/srv/jekyll

Could you advise what might have been wrong? Thanks

It doesn't work on <IP_ADDRESS>. You should go to <IP_ADDRESS>:8888/al-folio


Incorrect Rows count in Statistics for BLOCK channels

TPC-H scale 1000 Q3
SCALAR Channel

BLOCK Channel

#11859

#11893


Incompatible with DFred's Convenient Effects on "Add/Replace Mode"

When using Replace mode with DFred's Convenient Effects to only have 5e conditions, it is impossible to add flying or inaudible to a token.
DFred's on add mode:

DFred's on replace mode:

You'll have to ask DFreds if it's possible to extend the list of conditions.

Looks like an issue was already made over there https://github.com/DFreds/dfreds-convenient-effects/issues/253

Added compatibility in Vision 5e 1.3.0.


isClosed not supported

IsClosed in ResultSet and PreparedStatement runs in not implemented exception

Methods close and isClose now implemented in SqlPojoConnection, SqlPojoResultSet and SqlPojoPreparedStatement.


When a GeoJSON is loaded on the map, it's loaded in multiple copies (2,3,4, etc..)

on GSIP (see on stage http://w-stf-a128913.nrn.nrcan.gc.ca:8085/gsip/app/index.html) , when a GeoJSON is loaded on the map, it comes in multiple copies (not sure why the number of copies is different)

seems to increase every by 1 for every GeoJSON I add

playgroundUpdate is called multiple times,

multiple events attached to the geojson button !

remove the binding before rebinding.. not sure I fixed the real problem, or just a hack
$(document).off("click",".geojsonLink");
but this did the trick
$(document).off("click",".geojsonLink");
		$(document).on("click",".geojsonLink",function(event){
			event.preventDefault();
			//console.log($(this).attr('geojson'));
			addGeom2Map($(this).attr('geojson'))
		});


Fixed info about container exit codes

Fixes https://github.com/kubeflow/website/issues/443
See current code in this file: https://github.com/kubeflow/tf-operator/blob/master/pkg/util/train/train_util.go

This change is

Preview: https://deploy-preview-775--competent-brattain-de2d6d.netlify.com/docs/components/tftraining/

/assign @joeliedtke

/lgtm
/approve


darkScrollbar - How to link to listView ?

First of all, thank you for creating this neat Dark UI Robin!
I've managed to make my Winforms just how I want it, except...
The scrollbar, see:

I've managed to setup Dark UI using your descriptions, and now it looks like this:

Super sweet!
But.... It's not connected to the listView I'm using there.
How do I link your darkScrollbar to listView?
Or how would I approach this?

Are you using a DarkListView or the WinForms default ListView control?
If it's the former, DarkListView comes with its own in-built DarkScrollBar control so you shouldn't have to do anything.
If it's the latter, you'll actually have a bit of trouble. The default WinForms controls are very difficult to modify in this way.
I found this implementation by having a quick look around, but it's not pretty: https://www.codeproject.com/articles/236319/listview-with-custom-scrollbars-2
I would highly recommend using the DarkListView control or looking at how I implemented it and creating your own with similar behaviour.

@RobinPerris
I'm using WinForms default ListView control.
I've customized it to my likings using CustomDraw, So I have control over how it shows (selection color bar, background, etc.)
With DarkListView I get an extra vertical scrollbar, that I can't seem to get rid of.
There's no Property for it to disable either.
A hint would be much appreciated =)

The control will check if the measured width of the inner content is greater than the maximum width of the overall control. If so, it'll force the scrollbar to appear.
https://github.com/RobinPerris/DarkUI/blob/31603ae561d38898c61c8ee9eefb68916de90a54/DarkUI/Controls/DarkScrollBase.cs#L160-L164
This is by design. If you would like to remove the horizontal scrollbar even if the inner content is greater in size, you would have to override this functionality manually. There is no on/off switch in the form of an exposed property.
Apologies, you're actually trying to do something I never thought of when designing this library.
If you're confident in modifying the DarkUI code yourself, you could add a property to DarkScrollBase and have this checked within the SetScrollBarVisibility method.

Going to close this due to inactivity. If you're still trying to solve this problem, feel free to open a new issue and I'll help as much as I can.


add matrix compatibility to the chat gateway

See also https://matrix.org/docs/guides/types-of-bridging


PMs: Prevent sending messages to self

How to replicate:

Imagine your name is "user123"

Type in chat "/msg user123, content" and it will send a PM to yourself.

I don't think this is required; Sending pns to self can be used for various purposes like lag testing, testing formatting, etc

Agreeing with @Spandamn

Perhaps send a response back instead of actually sending the message then?

Sending PMs to yourself can be useful for testing things (lag tests, formatting tests, i used it to test a bug with spoilers yesterday too), and even for notes-to-yourself (i do that occasianlly). I dont think this is a good idea.

We have a lot of code for specifically handling sending PMs to yourself without doubling. This is intentional behavior.


Expired license

Hi,
How about the expired license check? can you please update licenseData.Verify to add some expired license verification?
Thank you.

This is something I have already implemented in a private fork of this repo, although via a separate Expired() function.  I kept the checking of expiration separate from verification since verification deals only with checking the signature.
I will copy over this expiration-checking functionality (as well as a ton of other changes).  Give me a week or so.

Please see latest changes.

Hi,
sorry but I don't understand how to call ExpiresIn and Expires func. can you please update the client exp. ?
Thank you.

//Read license from file...
f, _:= licensefile.Read("/path/to/your/license/file", licensefile.FileFormatYAML)

//Verify, signature only...
err := f.VerifySignature(publicKey, keyPairAlgo)
if err == ErrBadSignature {
    //Handle invalid signature.
} else if err != nil {
    //Some other error.
}

//Signature is valid, check if license is expired...
expired, err := f.Expired()
if err != nil {
    //Handle error.
}
if expired {
    //Handle expired license.
}

You can also do....
//Read license from file...
f, _:= licensefile.Read("/path/to/your/license/file", licensefile.FileFormatYAML)

//Verify signature and check if license is expired, all at once.
err := f.Verify(publicKey, keyPairAlgo)
if err == ErrBadSignature {
    //Handle invalid signature.
} else if err == ErrExpired {
    //Handle expired license.
} else if err != nil {
    //Handle other error.
}

Thanks, but I have licenseData.Expired undefined licenseData = licensefile.Read

You need to update to the latest beta release.  As of just now, go get<EMAIL_ADDRESS>

remote: Repository not found.
fatal: repository 'https://github.com/c9834/licensekeys/' not found

@anasbousselham Come on, for someone who notes themselves as "Systems Administrator, Cybersecurity, Ethical Hacker", I would have though you would be able to diagnose a typo.  c9845 vs c9843 in the URL.
go get<EMAIL_ADDRESS>

sorry but still not working, It's not my first time working with go, I have a lot off app with it. something is wrong..

c9834 --> c9835

go get<EMAIL_ADDRESS>works.  I just tested it half a dozen times across multiple machines.

Use Expired(), ExpiresIn(), and ExpiresInDays() funcs parts of v2.1.0 release.

@anasbousselham Please create a separate issue for this and clarify what you are looking for.  I did so for you.  Please see my reply and please provide more info.


elyra-metadata CLI: install ... help display seems to be incorrect?

1.0.0
Running the command below I've noticed that the help output for the last two parameters seems to be problematic:
$ elyra-metadata install runtime-images --schema_name=runtime-image
'--name' is a required parameter.

Install a metadata instance into namespace 'runtime-images'.

Options
-------

--replace
	Replace existing instance
--schema_name=<string>
	The schema_name of the metadata instance to install
--name=<string>
	The name of the metadata instance to install
--display_name=<string>
	The display name of the Runtime Image; title: Display Name, minLength: 1
--image_name=<string>
	Runtime Image description; title: Image Name, minLength: 1

To be specific:

title: XXX appears to be an internally used property that the user shouldn't be exposed to
minLength: XXX appears to be an internally used property that the user shouldn't be exposed to
--image_name=<string> Runtime Image description;  appears to be the wrong description (should be Docker image name ?)

Just realized that other kinds of internal metadata is exposed as well for other namespaces, e.g. uihints
$ elyra-metadata install code-snippets --schema_name=code-snippet
'--name' is a required parameter.

Install a metadata instance into namespace 'code-snippets'.

Options
-------

--replace
	Replace existing instance
--schema_name=<string>
	The schema_name of the metadata instance to install
--name=<string>
	The name of the metadata instance to install
--display_name=<string>
	The display name of the Code Snippet; title: Display Name, minLength: 1
--description=<string>
	Code snippet description; title: Description
--language=<string>
	Code snippet implementation language; title: Language, uihints: {'field_type': 'dropdown', 'default_choices': ['Python', 'Java', 'R', 'Scala', 'Markdown']}, minLength: 1
--code=<array>
	Code snippet code lines; title: Code, uihints: {'field_type': 'code'}

Thanks @ptitzler - this is one of the issues with a completely schema-driven approach.
The intent of the additional metadata (e..g, minLength: 1) is to inform the user creating the metadata instance of the restrictions that will be imposed on their entry.  However, since the tool was developed, we've added more metadata that doesn't necessarily apply to the actual instance data (e.g., title and uihints).  Since we don't have meta-properties about the metadata itself that says - this meta-property applies to data, while this meta-property doesn't, I don't think we have any choice but to remove the hinting altogether.

--image_name= Runtime Image description; appears to be the wrong description (should be Docker image name ?)

Yes, the image_name description is incorrect.  Runtime-image should also have a description property as well.  Both will be addressed in a subsequent PR.


Create a two levels multichoice list

Hi,
I want to create a multichoice button list with Framework Composer. I have almost 30 triggers each with its intents, they works well but, since they are FAQs I would like to guide the user to the possibilities by a two levels of multichoice button lists. For example: "What phase are you in?" [Before] [Doing] [After]. And if the user push [Before], the bot shows 10 examples of intents. The same with [Doing] and [After]. Also I want the possibility of write the doubt at the beggining.
My problem is that, since I have no intents for "before", "doing" and "after", so Luis do not predict it.
How can I do that? I have very poor experience in Composer and I do not find examples.
I would like to know how compose the multichoice node and the next who get the information.
Thanks.

The Microsoft Bot Framework team prefers that how to questions be submitted on Stack Overflow. The official Bot Framework Composer repo is the preferred platform for information on submitting bug fixes and feature requests.


JSON equality check instead of JSON Paths for non dynamic cases

If there are no dynamic checks in the request / response body then we can do JSON check without JSON Paths

Decided it's not relevant during backlog team review.


Kcho/update path detection to use regrex

I have noticed that lochness.XNAT module will download data to a subdirectory (eg- 2001_MR1) under mri directory in the phoenix structure. It could also be the case that other teams may add extra subdirectories under box and mediaflux as well. To account for this extra level of directories under the datatype directory, I've updated FileInPhoenixBIDS and FileInPhoenix classes to use regrex to be more flexible under these situations. Tested using pytest test/dpanonymize/test_dpanonymize.py.
Thanks.

Clarifiying discussion

It could also be the case that other teams may add extra subdirectories under box and mediaflux as well. To account for this extra level of directories under the datatype directory

In regard to your description, I am trying to understand the need for this new code:
https://github.com/AMP-SCZ/dpanonymize/blob/fa99674544b375a1aa3145be56ade86bcc6986d5/dpanonymize/__init__.py#L29-L45
The above code correctly detects processed, study, subject, and data groups for both beginning and ending extra paths:
file_path="/Some/Path/EXTRA/PROTECTED/BWH/raw/subject01/actigraphy/file"
In [98]: search.group('processed')
Out[98]: 'raw'

In [99]: search.group('study')
Out[99]: 'BWH'

In [100]: search.group('dtype')
Out[100]: 'actigraphy'

In [101]: search.group('subject')
Out[101]: 'subject01'

file_path="/Some/Path/to/PROTECTED/BWH/raw/subject01/actigraphy/EXTRA/file"
In [105]: search.group('subject')
Out[105]: 'subject01'

In [106]: search.group('dtype')
Out[106]: 'actigraphy'

In [107]: search.group('study')
Out[107]: 'BWH'

In [108]: search.group('processed')
Out[108]: 'raw'

Did you observe the same? If yes, the description should be updated as:

It could also be the case that other teams may add extra subdirectories under box/mediaflux and datatype directories. To account for this extra level of beginning and ending directories ...

Clarifiying discussion

It could also be the case that other teams may add extra subdirectories under box and mediaflux as well. To account for this extra level of directories under the datatype directory

In regard to your description, I am trying to understand the need for this new code:
https://github.com/AMP-SCZ/dpanonymize/blob/fa99674544b375a1aa3145be56ade86bcc6986d5/dpanonymize/__init__.py#L29-L45
The above code correctly detects processed, study, subject, and data groups for both beginning and ending extra paths:
file_path="/Some/Path/EXTRA/PROTECTED/BWH/raw/subject01/actigraphy/file"
In [98]: search.group('processed')
Out[98]: 'raw'

In [99]: search.group('study')
Out[99]: 'BWH'

In [100]: search.group('dtype')
Out[100]: 'actigraphy'

In [101]: search.group('subject')
Out[101]: 'subject01'

file_path="/Some/Path/to/PROTECTED/BWH/raw/subject01/actigraphy/EXTRA/file"
In [105]: search.group('subject')
Out[105]: 'subject01'

In [106]: search.group('dtype')
Out[106]: 'actigraphy'

In [107]: search.group('study')
Out[107]: 'BWH'

In [108]: search.group('processed')
Out[108]: 'raw'

Did you observe the same? If yes, the description should be updated as:

It could also be the case that other teams may add extra subdirectories under box/mediaflux and datatype directories. To account for this extra level of beginning and ending directories ...

It's to support extra folders under the datatype root directory, which is already the case for the data pulled by XNAT.
eg) lochness.XNAT puts another folder under mri , eg) raw/subject01/mri/2001_MR.
Doc string has been updated to add a little more elaboration b31a9f3

Please consider merging my suggestion #16 and we are good to go. Thank you.

Merged. Thanks


Add Lightning Payments - Backend

Continue development of the Processor class in src/lib/payments.py

Build classes for various payment methods

[x] Strike
[x] OpenNode
[x] LNBits
[ ] Voltage

[x] Upon Abbot being added to a new chat, create new mongo channel object in channels using default
[x] Add logic to Abbot class to calculate number of tokens per completion and cost per completion
[x] Add new telegram slash command /fund that takes an int arg in sats (e.g. 1000000 sats)

Stretch Goals:

[ ] Update src/payments.py adding OpenNode, LNBits and Voltage processors

Continue development of the Processor class in src/lib/payments.py

Build classes for various payment methods

[x]  Strike
[x]  OpenNode
[x]  LNBits
* [ ]  Voltage

[ ]  Upon Abbot being added to a new chat, create new mongo channel object in channels using default

[ ]  Add logic to Abbot class to calculate number of tokens per completion and cost per completion

[ ]  Add new telegram slash command /fund that takes an int arg in sats (e.g. 1000000 sats)

Stretch Goals:

[ ]  Update src/payments.py adding OpenNode, LNBits and Voltage processors

Killing Voltage for now to save time. Can backlog it.

Closing issue with changes made to branch lolv2 #70


Unable to select image for posts in Safari for iPad on 2.10.46

As the title suggests, after the release of 2.10.46 I can’t choose the image after uploading it on iPad. Works fine on Chrome.

Just to confirm, is the issue you're having is this:

That is, the "choose select" is hidden by the images on the iPad?

That too. After 4 images the “Choose Select” gets hidden under the images, but even if it’s visible is not clickable.

@erezrokah We can either replace the text with icons or create a new row below the search for the buttons. Can I claim the issue?

Thanks a lot @tinfoil-knight, I assigned you to the issue 😄

@erezrokah I was trying to work on the package but the repo is not starting. I tried yarn develop and yarn start and both scripts pause at the following log:
netlify-cms:  warning  in .-widget-code/dist/esm/CodeControl.js
netlify-cms: Critical dependency: the request of a dependency is an expression
netlify-cms:  WAIT  Compiling...5:05:21 PM
netlify-cms:  WARNING  Compiled with 1 warnings5:05:22 PM
netlify-cms:  warning  in .-widget-code/dist/esm/CodeControl.js
netlify-cms: Critical dependency: the request of a dependency is an expression

These issues usually pop up after dependency changes.

Can pull latest the latest master and run yarn install again? I also get this warning but able to use yarn start.

Related https://github.com/netlify/netlify-cms/issues/3701

@erezrokah Since all the MediaLibraryActions are in the same file, I think it would be best if we just place all of them in a line below the search bar. What do you think?

How about we auto shrink the search bar? Maybe that would be easier to implement?

WIP: #3784


URI - No such file or directory

I am using react-native-fetch-blob with a detached Expo app. I get this error: RNFetchBlob failed to create single content request body :error when getting request stream: No such file or directory.
I only get this problem in my expo version of the project
My file path is:
RNFetchBlob-file://file:///data/user/0/com.flyreel.underwriterbot/cache/ExperienceData/%2540darkwata%252Fflyreel/Camera/967785f9-064f-44e2-9277-6d6650d76fe7.mp4

I'm on Android btw.
package.json:
"react-native-fetch-blob": "^0.10.8",

Seems like its a problem with the %2540 and %252F encoding in the file path which Expo uses.

A workaround I found was to use react-native-fs https://github.com/itinance/react-native-fs
And move the file to the RNFS temporary directory and everything works, but would be nice if this library fixed the issue with the %2540 encoding.


Cosmos Go SDK fails to handle MaxInt64 in the QueryItemsResponse

Bug Report

import path of package in question: github.com/Azure/azure-sdk-for-go/sdk/data/azcosmos
SDK version: github.com/Azure/azure-sdk-for-go/sdk/data/azcosmos v0.3.6
output of go version: go version go1.21.7 darwin/amd64

What happened?

When data persisted in cosmos db has math.MaxInt64<PHONE_NUMBER>854775807), the query response will return the wrong number<PHONE_NUMBER>854776000)
Note that this is only an bug for query, not for point get (ReadItem)

What did you expect or want to happen?

The data should be returned as it is and not modified by the sdk

How can we reproduce it?

Write some data that has a field with value<PHONE_NUMBER>854775807
Issue a query that utilizes NewQueryItemsPager method
The response will return the object with a different value<PHONE_NUMBER>854776000

Where is the bug?

The bug lies in the logic trying to unmarshal query response to Json
When a field is set as max int64<PHONE_NUMBER>854775807, it will be converted to<PHONE_NUMBER>854776000, because of json.Unmarshal will convert it to float64.  See json package - encoding/json - Go Packages

Please see https://learn.microsoft.com/en-us/azure/cosmos-db/concepts-limits#per-item-limits

9223372036854775807 seems outside of the safe boundary of IEEE754<PHONE_NUMBER>740992) to work with the service. This will run into issues across other areas (like Portal interactions or other languages).

While in Go this could be fixed by removing the unmarshal call and change it to something else, the issue will still appear in other areas.
Storing these numbers as strings instead would be the recommendation.
We can certainly modify the Go SDK but that would not remove the problem across other service interactions in other languages/scenarios.

@ealsur Thanks for taking a look and providing the detailed docs! You are right, we also see the number handling issue with Azure Cosmos Portal, but the underlining data remains correct.

We can certainly modify the Go SDK but that would not remove the problem across other service interactions in other languages/scenarios.

Do you think you can introduce some option to configure? This is how spanner does it https://github.com/googleapis/google-cloud-go/blob/main/spanner/value.go#L120-L132

Hi Matias,
Thanks for taking a look and providing the detailed docs! You are right, we
also see the number handling issue with Azure Cosmos Portal, but the
underlying data remains correct.
We can certainly modify the Go SDK but that would not remove the problem

across other service interactions in other languages/scenarios.

Do you think you can introduce some option to configure? This is how
spanner does it
https://github.com/googleapis/google-cloud-go/blob/main/spanner/value.go#L120-L132
Best,
Xin
On Tue, Mar 5, 2024 at 1:09 PM Matias Quaranta @.***>
wrote:

Please see
https://learn.microsoft.com/en-us/azure/cosmos-db/concepts-limits#per-item-limits
image.png (view on web)
https://github.com/Azure/azure-sdk-for-go/assets/1633401/f558688d-4239-4c92-aa43-1fa3ad43e14b
9223372036854775807 seems outside of the safe boundary of IEEE754
https://www.rfc-editor.org/rfc/rfc8259#ref-IEEE754<PHONE_NUMBER>740992)
to work with the service. This will run into issues across other areas
(like Portal interactions or other languages).
image.png (view on web)
https://github.com/Azure/azure-sdk-for-go/assets/1633401/86eda2ce-4b98-43aa-a083-624dfd0606fe
While in Go this could be fixed by removing the unmarshal call and change
it to something else, the issue will still appear in other areas.
Storing these numbers as strings instead would be the recommendation.
We can certainly modify the Go SDK but that would not remove the problem
across other service interactions in other languages/scenarios.
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-go/issues/22523#issuecomment-1979639671,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2AOMRJUGOXFVTYMK3I6CRTYWYYBRAVCNFSM6AAAAABEHYHOJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZZGYZTSNRXGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

the underlining data remains correct.

Right, the problem is that it is not consistent across. That's why it's safer to not use them.

Do you think you can introduce some option to configure?

One solution would be to simply remove the unmarshal and attempt to simply return the byte[] of the document (unsure exactly what the implementation would look like right now). If that can be done (like in ReadItem), then at least from the Go SDK perspective, you'd be ok. But the problem is still there.
In the future when things like Cross Partition Query support is added with things like aggregations (that need to happen on the client), then this will probably happen again.
For Json serialization, our Cosmos SDK uses azruntime.UnmarshalAsJSON, which uses json.Unmarshal under the covers. I don't know if "encoding/json" has any such setting that you could customize.

For example, it has https://pkg.go.dev/encoding/json#Decoder.UseNumber, which config the Decoder to unmarshal a number into an interface{} as a Number instead of as a float64.

Thanks @CurryFishBalls9527!
@ealsur Do you think this is something you can introduce to the sdk?

I can add it to the backlog. As I mentioned, right now our SDK does not invoke json directly but consumes it from azruntime.
But even if we do this, please see my other comments, using these Numbers will have further potential repercussions outside of this SDK.


restore generate-linux-main for backwards compatibility

motivation: users made the point that while deprecating generate-linux-main makes sense, we should not deprecate it too quickly so they can support SwiftPM < 5.1 more easily
changes:

restore LinuxMainGenerator command
resotre plumbing to call LinuxMainGenerator on --generate-linux-main while retaining the deprecation warning
add test to test both deprecation warning and that the generated code is created

rdar://72160848

code here is restored from https://github.com/apple/swift-package-manager/pull/3053 where this was removed. no significant changes made to the code.

code here is restored from https://github.com/apple/swift-package-manager/pull/3053 where this was removed. no significant changes made to the code.

@swift-ci please smoke test

@swift-ci please smoke test

we need to bring this in 5.4 as well once merged

we need to bring this in 5.4 as well once merged

@swift-ci please smoke test

@swift-ci please smoke test


[BUG] inconsistent optimizer naming and defaults

Describe the bug
I've noticed a couple of minor inconsistencies with the Deepspeed provided optimizers.

I expect CPU adam and GPU adam to have identical naming. However, CPU adam has modelparams and adamw_mode while GPU adam has params and adam_w_mode. It's a bit annoying that the same instantiation code doesn't work for both optimizers.
The default weight decay on both optimizers is set to 0, which is confusing given that the default optimizer is set to use AdamW mode (and I think most practitioners do as well). My understanding is that AdamW with 0 weight decay is equivalent to Adam, and probably not what practitioners intended to use. The PyTorch default uses 0.01 for weight decay, and the docs claim

apex.optimizers.FusedAdam may be used as a drop-in replacement for torch.optim.AdamW, or torch.optim.Adam with adam_w_mode=False:

One option to maintain backwards compatibility is to create new AdamW optimizers for CPU and GPU, each of which just sets weight decay 0.01 by default. This would also match the torch implementation, which has Adam w/ weight decay = 0 and AdamW w/weight decay=0.01.
To Reproduce
Deepspeed docs: https://deepspeed.readthedocs.io/en/latest/optimizers.html
Torch adamw docs: https://pytorch.org/docs/stable/generated/torch.optim.AdamW.html
Expected behavior

Arguments to CPU and GPU adam are identical
Default parameters match those of Pytorch's

@stillmatic - thanks for pointing this out, for the first part, we have a bug that I still needed to address for this, can you confirm #3906 is the same ask?
For the second part, I'm not certain and will look into that.

can you confirm https://github.com/microsoft/DeepSpeed/issues/3906 is the same ask?

It is very closely related - that one points out the same modelparams -> params inconsistency in the CPU Adam optimizer, but I also note a adamw_mode -> adam_w_mode inconsistency.

Thanks, that makes sense.  I'll take a look at updating all of those to match.


add variogram_parameters to krige CV parameters dictionary

I didn't know that you could explicitly give a seed for a numpy random instance, thanks!
You will figure out that I have used decimal numbers with a decent precision when it comes to testing the best score. I think that it might be a good idea for a value test since floating numbers tend to be imprecise.


[Question] URP compatibility

Do you plan to make it URP compatible?

yep then trying ti upgrade to URP
Stylized Water 1 material was not upgraded. There's no upgrader to convert Stylized/Water shader to selected pipeline
UnityEditor.Rendering.Universal.UniversalRenderPipelineMaterialUpgrader:UpgradeSelectedMaterialsMenuItem () (at Library/PackageCache/com.unity.render-pipelines.universal@14.0.4/Editor/UniversalRenderPipelineMaterialUpgrader.cs:62)

I'm wondering why the demo scene looks visually different thant what it is on the pictures , and i'll appreciate it if you'd help me set it up to look exactly like the first image, thanku : D

URP doesn't support grabpass, so you can add a script to grab texture with a camera yourlef , or just delete relative code


sqlsrv tests for testing output params with sqltypes

This change is

Coverage remained the same at 73.186% when pulling 2f6390cf4a7048fc95175b9ec1a569abb3f3104a on lilgreenbird:aetest into f3b96f4821463798251926c9779a69e7ecec04ea on Microsoft:AlwaysEncrypted.

Codecov Report

Merging #712 into AlwaysEncrypted will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           AlwaysEncrypted     #712   +/-   ##
================================================
  Coverage            78.81%   78.81%
================================================
  Files                   25       25
  Lines                 7188     7188
================================================
  Hits                  5665     5665
  Misses                1523     1523

Continue to review full report at Codecov.

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

sqlsrv tests for testing output params with sqltypes #712


Lovely with helm-argocd-vault-replacer

I've configured lovely plugin with vault-replacer, and trying to use helm-argocd-vault-replacer as plugin for lovely. My use case is that we have in-house maintained Helm chart, that in one of it's secret retrieves Vault secret like this:
stringData:
  sftp.json: "<secret:sftp/data/login/{{ .Values.sftp.environment }}/users~{{ .Values.sftp.userName }}>"

As values are used in the secret template, I need to first render the Helm chart before filling in the secret templates.
My Application spec looks like:
    spec:
      source:
        path: 'deploy'
        plugin:
          name: argocd-lovely-plugin
          env:
            - name: LOVELY_PLUGINS
              value: helm-argocd-vault-replacer
            - name: LOVELY_PREPROCESSORS
              value: sed -i 's/<<environment>>/{{environment}}/g' values.yaml || true

Error I get is:
`argocd-lovely-plugin` failed exit status 1: 2022/09/27 05:19:57 exit status 127: bash: line 1: helm-argocd-vault-replacer: command not found`

Looking at source code in lovely plugin, all it tries is to execute plugins as commands. Are plugins in argo-cd configmap relevant at all as plugins for lovely, and can above scenario work at all?

The above scenario is relevant, and I use it regularly. The only problem I can see is that vault replacer's command line name is wrong, and should be argocd-vault-replacer, without the helm- prefix.

Yep, using argocd-vault-replacer this works perfectly, thank you.


Reduce Memory Usage by Optimizing Order of Fields

Optimized the order of fields in several structs, this results in a reduction of memory usage by quicktemplate.
Summary:
/quicktemplate/jsonstring.go:61:19: 256 bytes saved: struct with 264 pointer bytes could be 8
/quicktemplate/parser/parser.go:15:13: 8 bytes saved: struct of size 96 could be 88
/quicktemplate/parser/scanner.go:33:12: 8 bytes saved: struct with 16 pointer bytes could be 8
/quicktemplate/parser/scanner.go:53:14: 16 bytes saved: struct of size 192 could be 176
/quicktemplate/parser/scanner_test.go:325:9: 8 bytes saved: struct with 16 pointer bytes could be 8

@valyala Quick optimization that should reduce memory usage by around ~300 bytes.
I also submitted the same optimization to bytebufferpool https://github.com/valyala/bytebufferpool/pull/29


make format specifier compatible between MSVC & GCC for 64bit types

make format specifier compatible between MSVC & GCC for 64bit types
MSVC warns about the format specifier for 64bit values different then gcc. The macro from  fix the warnings.

@SvenJo Thanks!


CSI pods goes into crashloopback state, on giving primary gui node as invalid in CR.

Describe the bug
CSI pods goes into crashloopback state, on giving primary gui node as invalid and secondary gui node as valid nodes CR.
How to Reproduce?

Update primary gui node as invalid and secondary as Valid entry in CR.

#      cacert:  "spectrumscale-trusted-ca"
      restApi:
        - guiHost: "monica-multi-scalegui-21.fyre.ibm.com"
        - guiHost: "monica-multi-scalegui-2.fyre.ibm.com"
#        - guiHost: "monica-multi-scalegui-3.fyre.ibm.com"

Apply the CR.
Observe the csi pods.

Expected behavior
On applying the CR, the valid gui node should start working, also the pods should not get into crashloopback state.

Add labels

Component:
Severity:
Customer Impact:
Customer Probability:
Phase:

Note : See labels for the labels

Fixed in quay.io/amdabhad/csi-driver-gui-ha:300822

Verified the issue and it is working fine.
On giving incorrect primary gui host, the pods are not moving to crashloopback.
Find attached the logs and setup details:
[root@monica-multi-master ~]# oc get cso
NAME                     VERSION   SUCCESS
ibm-spectrum-scale-csi   2.7.0     True
logs1.txt
logs2.txt


Update Composer dependencies (2023-07-04-00-09)

Visual regression test passed!
View the visual regression test report


When databasename is empty it should not be added to prefix.

Why make this change?

Reference associated issue using # syntax. e.g. Closes #XX

Include summary (1-2 sentences) of linked issue to avoid redirecting reviewers to different pages.
Fixes issue mentioned in : https://github.com/Azure/data-api-builder/issues/1604

What is this change?

Summary of how your changes work to give reviewers context of your intent.

Links to relevant documentation / StackOverflow, if applicable.
Added check to see if databasename is null or empty then we will not add it to the table prefix.

How was this tested?

[ ] Integration Tests
existing integration tests test this path of code
[ ] Unit Tests
existing unit tests test this path of code.

Are we able to add tests for this?


TypeError: 'NoneType' object is not subscriptable

Hello Dear developers! I really like your project and I would like to implement it also. I am new in GNN and would like to learn. I was able to create datasets. But when I am running file main.py I have this error. I have checked my datasets. They have values. Also read_dataset.py produce them. I am trying to understand why model receiving NoneType. Could you guide me please where I am making mistake? Thank you in advance!
TypeError: 'NoneType' object is not subscriptable
Call arguments received by layer 'route_net_model' (type RouteNetModel):
• inputs={'traffic': 'tf.Tensor(shape=(None,), dtype=float32)', 'packets': 'tf.Tensor(shape=(None,), dtype=float32)', 'capacity': 'tf.Tensor(shape=(None,), dtype=float32)', 'link_to_path': ' and so on.

Dear @ChuvashaevaAlena ,
Thanks for your interest in our research and for posting here an issue.
This problem may be caused by several reasons. Just so we can help you, which version of Python and Tensorflow are you using? Do you face the same problem when using the provided public datasets?
Bests,
Miquel

Dear @MiquelFerriol,
Thanks for replying.
I am using Python version 3.11 and tensorflow 2.13.0. I have tried the dataset, which I have created, and also tried the public one nsfnet.

That is weird. I know there are some issues with TF versions greater than 2.7 and tf.dataset. Can you try with this 2.7 version of TF and see if it is a compatibility issue? We are aware of some other dimensionality problems that TF creates "artificially" with 2.7+ versions, but this is for sure a new one.

And one more question, please. If I understand correctly the dataset, created by the function "input_fn" should be given as output tf.data.dataset structure such as: inputs (for NN as an input parameter) and delay (as an output parameter and as a ground truth) is that correct?

Yes, that is correct.

I have downgraded the version of TF to 2.7, and now it's working. Thank you so much for your help. Appreciate it! Now I can actually explore the model and its possibilities! Cheers!

Nice! Thank you for pointing this out.
I will try to investigate the issue and try to release a TF-updated version.
Cheers!
(Closing since the issue is solved)


Things to make functional demo

Required

[x] Add EntityManager and create/manage player entities properly.
[ ] Add Block/Item Provider and all the blocks/items for lookup.
[ ] Add Region.
[x] Add ChunkProvider.
[x] Manage player inventories.
[x] Send over compressed chunks.
[x] Add Item NBT Data.
[x] Add read/write floats and doubles

Packets

[x] EntityVelocityPacket
[x] DestroyEntityPacket
[x] WIndowItemsPacket
[x] SpawnPositionPacket
[x] KeepAlivePacket
[x] SetPlayerPositionPacket
[x] TimeUpdatePacket
[x] EntityTeleportPacket
[x] EntityMetadataPacket
[x] ChunkPreamblePacket
[x] ChunkDataPacket
[x] ChatMessagePacket
[x] DisconnectPacket
[x] PlayerGroundedPacket
[x] PlayerPositionPacket
[x] PlayerLookPacket
[x] PlayerPositionAndLookPacket
[x] PlayerDiggingPacket
[x] PlayerBlockPlacementPacket
[ ] ChangeHeldItemPacket
[x] PlayerActionPacket
[ ] ClickWindowPacket
[x] CloseWindowPacket

Holy crap! We're in the game!

We're really close now. We can jump around, chat, and "place" blocks (client-sided)!

Now we have extremely buggy support for placing blocks! :100: Slowly but surely.

Block placement is no longer as buggy! And the /give command has been implemented, allowing for basic building now!
Next up: destroying blocks!

Recently:

Destroying blocks has been implemented.
Chunks are now loaded instead of generated on the fly everytime.
Block changes now affect chunks / and as a result changes persist in chunks!
Disconnect works without crashing the server
Multiplayer actually works!

4 minecraft clients connected at the same time. Server is only at 130MB and 3% cpu utilization.

This is so cool. Amazing work!

I could contribute if needed.


For PHP writing $this used to autocomplete to $this-> now it removes the dollar sign

VSCode Version: Code 1.8.0 (38746938a4ab94f2f57d9e1309c51fd6fb37553d, 2016-12-13T17:45:32.595Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

theme-atom-one-dark
andischerer
0.0.1

Theme-FlatlandMonokai
gerane
0.0.4

format-indent
Kasik96
1.3.0

php-symbols
linyang95
0.1.2

Theme-MaterialKit
ms-vscode
0.1.1

theme-hackershaze
Nuuf
1.3.1

Seti-theme
TsumiNa
0.0.1

Material-theme
zhuangtongfa
1.0.3

Steps to Reproduce:

While writing code in a PHP file, type $this to see the autocomplete to $this-> option and press Enter.
Instead of seeing it autocomplete to $this-> it removes the '$' sign and instead writes this-> which is incorrect.

@ukjadoon thx for reporting! Unfortunately I can't seem to reproduce this problem at all.

Are you still running into it? Does it happen in all files, or just sometimes?

Thank you so much for looking into this issue (and apologies for the late response).
In order to get it to show your autocomplete option with the text “Refers to the current object”. I have to write
$this (a dollar sign followed by this keyword)

In order to get my reported autocomplete option, you just have to write this (without the dollar sign) inside a PHP file.

In the previous versions of visual studio code, for PHP files, writing "this" without the dollar sign would autocomplete to $this-> (it would automatically add the dollar sign to the “this” keyword followed by -> which saves some time). Now it still adds the -> symbol but no dollar sign (when I write this without the dollar sign and autocomplete). That just makes the whole experience annoying as I have to go back to the beginning of the word to manually add a dollar sign, it should work as it did before.
If I write $this and autocomplete, it does not add the -> symbol at the end.
I hope that helps clarify the issue.

#18778


Publish to npm

Hi @danmichaelo ,
I (and hopefully many other users) would be grateful if you consider publishing this useful plugin to npm.
The steps are described here: http://cordova.apache.org/announcements/2015/04/21/plugins-release-and-move-to-npm.html

Added. Let me know if it works, I don't work with cordova at the moment.
(Guess I should also try to merge in some changes from https://github.com/yezhiming/cordova-plugin-appinfo now)

Thanks a lot @danmichaelo
It woud be great also to see latest changes from @yezhiming !

Yeah, I make a separate issue on it


Failed to create for Facebook iOS SDK

Can not create an xcframework from Facebook-ios-sdk
swift-create-xcframework version: 1.3.0
Swift version:
Apple Swift version 5.3.2 (swiftlang-1200.0.45 clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin20.3.0
Environment: Xcode 12.4 and macOS Big Sur 11.2.3
Checklist

[x] If possible, I've reproduced the issue using the master branch of this package
[x] I've searched for existing GitHub issues

Steps to Reproduce

git clone https://github.com/facebook/facebook-ios-sdk.git
cd facebook-ios-sdk
swift create-xcframework FacebookShare --platform ios

Expected behavior
An xcframework is created properly
Actual behavior
/facebook-ios-sdk/FBSDKCoreKit/FBSDKCoreKit/Internal/Network/FBSDKURLSessionProxyFactory.h:21:9: fatal error: 'FBSDKCoreKit+Internal.h' file not found
#import "FBSDKCoreKit+Internal.h"
** ARCHIVE FAILED **
The following build commands failed:
CompileC /Facebook-bddkrwgyumugvmdfxaxpklxrorhb/Build/Intermediates.noindex/ArchiveIntermediates/FacebookShare/IntermediateBuildFilesPath/Facebook.build/Release-iphoneos/FBSDKCoreKit.build/Objects-normal/armv7/FBSDKTokenCache.o /facebook-ios-sdk/FBSDKCoreKit/FBSDKCoreKit/Internal/TokenCaching/FBSDKTokenCache.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)
Error: xcodebuild exited with a non-zero code: 65

Hi @nixzhu, if you run swift generate-xcodeproj inside your checkout of the facebook-ios-sdk are you able to build successfully using the generated project?

@bok- No, the generated Facebook.xcodeproj built to simulator iPhone 12 mini failed: "FBSDKCoreKit+Internal.h" file not found. I think it's a Facebook SDK's issue.


support for external table in AWS

Hi,
users building data lake in AWS, or in other systems, use EXTERNAL TABLES quite a lot. This gives their solutions high flexibility. For example having either a personal metastore or using Glue datacatalog.
It will be great if delta support as a storage class was included for HIVE metastores as well. I think we can and do read data using HIVE or SPARK directly from DELTA write locations, but it does give us duplicates, because its just reading the parquet files.
Regards,
Gourav

We are working towards improving our table support in OSS so that we can run all Delta table operations directly on table names defined in Hive metastore. The challenge is to add the necessary customizations in Apache Spark to make DDLs work with Delta's setup.

This is important for your use case and web be happy to contribute. How far have you got?

We are still working on this and are waiting for the Spark 3.0. This should ideally come out in the Delta Lake 0.7.0 release.

HI,
thanks a ton Pranav, its a great news.
Thanks and Regards,
Gourav Sengupta
On Thu, Mar 19, 2020 at 9:53 PM Pranav<EMAIL_ADDRESS>wrote:

We are still working on this and are waiting for the Spark 3.0. This
should ideally come out in the Delta Lake 0.7.0 release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/delta-io/delta/issues/73#issuecomment-601434449, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJZLQ66LYZ4GGZQ5JCRTIDRIKH3XANCNFSM4HZPVBHQ
.

the table support is being tracked by this issue - #85

I am trying to use delta lake using AWS DMS and Glue. Could you help me find relevant posts.

Hi,
I use EMR (never found any reason valid enough to use AWS glue ETL,
ofcourse we use GLUE Datacatalog) and install delta in it as a part of
bootstrap actions.
Except for EMR 5.29 I think delta works fine in the rest. The latest
release of Delta works fine using EMR 6.1.0.
There is a chance that AWS team might suggest to use HUDI, but personally I
have found that HUDI's API's are too confusing for me to work with, and
even more confusing to change/ debug.
Best of luck
Regards,
Gourav
On Sun, Nov 8, 2020 at 6:24 PM kadprad<EMAIL_ADDRESS>wrote:

I am trying to use delta lake using AWS DMS and Glue. Could you help me
find relevant posts.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/delta-io/delta/issues/73#issuecomment-723647328, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJZLQZCWORJEFUVVURPGCTSO3O4BANCNFSM4HZPVBHQ
.

This has been released in Delta 0.7.0 on Spark 3.0. See #85

beautiful. Thanks a ton for this. I have been using some of those features
with a smile.
Regards,
Gourav Sengupta
On Wed, Apr 7, 2021 at 4:36 AM Shixiong Zhu @.***>
wrote:

This has been released in Delta 0.7.0 on Spark 3.0. See #85
https://github.com/delta-io/delta/issues/85
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/delta-io/delta/issues/73#issuecomment-814575957, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJZLQ3U4S7KKJ4A4B7EJ5TTHPHK5ANCNFSM4HZPVBHQ
.


Add option to fold log panel on start up

I personally likes the log panel to show only a single line. So I click on the "fold" button every time I open Atom. This change will allow user to add option to fold the log panel by default.
See the screenshots:
I love this plugin. And I donated :)

Looks great, will check and merge ASAP. Thanks.


groups: clicking on all channels leads to error encounter t.zones[o] is undefined

When clicking on "All Channels" in one of my own groups, I get an error message with the title " t.zones[o] is undefined" and the following log:
Ymt/r</<@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:1149:57132
Ymt/r<@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:1149:57114
useMemo@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:93:59210
tr.useMemo@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:12:5954
Ymt@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:1149:57065
NH@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:93:53252
kse@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:97:8857
wse@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:97:967
Mke@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:97:890
_2@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:97:737
b_@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:95:10947
Xoe/<@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:93:39036
e.unstable_runWithPriority@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:82:3881
fm@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:93:38810
Xoe@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:93:38984
Vh@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:93:38915
hoe@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:97:18416
w6e@https://my-ship.com/apps/groups/assets/index.d15fb5a1.js:93:17715

This occurs only in one of my self-hosted groups.
On the same ship I also have the issue described in #2250.
Happy to provide more information if you tell me what would be useful and where I can find those.

Should be fixed in #2309 going out with 3.0 soon


Throughput (MB/s)

Dear @schrej
Can we get throughput from “kubectl migrate”? For example, if we use “scp”, it will have a throughput log. I would like to know that "kubectl migrate" can do that whether or not?

There is no kubectl migrate command, or at least I didn't implement one. While it would be possible to implement, it's probably a lot of work. It could work similar to how logs are fetched directly from kubelet on the node. My implementation also didn't use SCP but NFS as it is a lot faster.

Thanks


[PanGestureHandler] using 2 PanGestureHandler

Can I let a PanGestureHandler respond to gestures starting from 2 edges of the screen and another respond to gestures starting from the middle part of the screen?

Decided to close this issue. It seems to be inactive.
If you still have a problem, please give us more details and reopen this thread.


Feature Request: testing against a control SV set

I think one scenario which might often arise is that we have e.g.

SVs analyzed in a control group
SVs calls from a treated group (or group with a certain sickness)

Or alternatively that the assembly and the SV-caller would disagree with some assembly hypothesis and report SVs.
It would be great if an option would exist to define one set as background SVs which should be ignored, or flagged.
So essentially the opposite of "Consensus call from multipe SV calls (vcf files)" .
Cheers

Right now I am often using awk or grep to achieve this.

Merge all the control + treated (Assuming in that order): SURVIVOR  merge files 1000 1 1 0 0 0 merged.vcf
awk ' $10 =="./.:0:0,0:--:NaN:NaN" {print $0}' merged.vcf > treated_only.vcf

I will think about building in such a filtering to make this easier.
Thanks
Fritz

Thx, indeed that is a very simple work-around


HttpException: Invalid proxy configuration HTTPS

I have a pac script which returns HTTPS usa.cn-cloudflare.com:443 and I just want to use this https proxy in my dart code. So I write some code like this.
import 'dart:io';
import 'dart:convert';

// https://codeburst.io/quick-tip-how-to-make-http-requests-in-dart-53fc407daf31

void main() async {
  HttpClient client = new HttpClient();
  client.findProxy = (Uri uri) => "HTTPS usa.cn-cloudflare.com:443";
  // produces a request object
  var request = await client.getUrl(Uri.parse('https://jsonplaceholder.typicode.com/posts'));
  // sends the request
  var response = await request.close();
  // transforms and prints the response
  await for (var contents in response.transform(Utf8Decoder())) {
    print(contents);
  }
  client.close();
}

But it shown me the following erros.
Unhandled exception:
HttpException: Invalid proxy configuration HTTPS usa.cn-cloudflare.com:443
#0      new _ProxyConfiguration.<anonymous closure> (dart:_http/http_impl.dart:3020:11)
#1      List.forEach (dart:core-patch/growable_array.dart:302:8)
#2      new _ProxyConfiguration (dart:_http/http_impl.dart:2976:10)
#3      _HttpClient._openUrl (dart:_http/http_impl.dart:2338:25)
#4      _HttpClient.getUrl (dart:_http/http_impl.dart:2208:48)
#5      main (file:///D:/code/dart/simple_throttle_debounce/proxy.dart:11:30)
#6      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:301:19)
#7      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

I also read the doc https://api.dart.dev/stable/2.9.3/dart-io/HttpClient/findProxyFromEnvironment.html, https://api.dart.dev/stable/2.9.3/dart-io/HttpClient/findProxy.html, and find it seems do not support HTTPS of pac.

I read the code implementation (https://github.com/dart-lang/sdk/blob/master/sdk/lib/http/http_impl.dart#L2632-L2652), it seems the proxy pac support in dart:io is not complete. It only support simple PROXY/DIRECT, other return values like SOCKS/HTTP/HTTPS/SOCKS4/SOCKS5 (https://developer.mozilla.org/en-US/docs/Web/HTTP/Proxy_servers_and_tunneling/Proxy_Auto-Configuration(PAC)_file) is not support.

Same issue here.
And depends on the official document HttpClient.findProxyFromEnvironment, https_proxy should work but it's not.
I also read the source code L2211 in sdk/lib/_http/http_impl.dart:
Future<ConnectionTask> connectionTask = (isSecure && proxy.isDirect
        ? SecureSocket.startConnect(host, port,
            context: context, onBadCertificate: callback)
        : Socket.startConnect(host, port));

it says that: only build HTTPS (SecureSocket) connection when the RAW target is secure and without proxy, with proxy never HTTPS.
So @liudonghua123 , they don't even implement https proxy feature, not only the HTTPS directive in PAC.

#43876

This may also be able to be set up manually once #43277 is resolved.


complete code

I admire your work and look forward to the full code. Thank you.

Thank you for your interest in our work! The relevant data and code will be released here as soon as they are organized.


I said allowed IPs is my home private IP address... not the GCP host (VM) that is provisioning this.    sigh


docs: which k8s-kernel pairs are we testing

Coverage decreased (-0.02%) to 45.878% when pulling d04ba4d05d58c024f481b866d0f9645840e3cf31 on pr/update-docs-test-matrix into 5d356866c72c471b041b8df5ee1534c52ed5d770 on master.

test-docs-please

test-me-please

This PR conflicts with https://github.com/cilium/cilium/pull/10864 has one needs to be rebase with the other. In this case will be documentation update.


xADDomain: Add Exclude DNS parameter

I know this parameter was just removed, but I'm just querying why? @iainbrighton - I've tried to track back through the issues list and PR's but can't figure out if this was removed because it didn't work properly or for another reason. I didn't see anything specific about it in the readme.md.
I have a need to be able to exclude DNS installation for child domains where the child DC's don't need a DNS of their own but instead can use the parent DNS.
This is not critical, but would love to be able to use it.

@PlagueHO In short it was removed temporarily whilst we discussed the ins and outs over in #38.

Ah! I missed that that. Thanks for the into - I'll close this one. No point cluttering things up.


Datacenter Region setting information is confusing

[Enter feedback here]
Hi,
The information about choosing the datacenter region is confusing and might be wrong. I am working on a support ticket with my customer and we are told by support that the region in that setting should always be same as Power BI tenant region even if it's public cloud.
Kind Regards,
Azadeh

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

ID: b2a07eaa-023b-fbc8-9508-39411425d096
Version Independent ID: efb6bb0e-5551-e2b3-4e98-3a34188600f3
Content: Set the datacenter region for the on-premises data gateway
Content Source: data-integration/gateway/service-gateway-data-region.md
Product: on-premises-data-gateway
Technology: ****
GitHub Login: @arthiriyer
Microsoft Alias: arthii

Yes, the installer does tell you that if you try to select a different region:

Thanks @azadehgnia for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


Add multiplication and division signs to Languages.cs

Adds multiplication (U+00D7, U+22C5) and division (U+00F7) signs to Languages.cs

Summary of the Pull Request
The pull request addresses the lack of common mathematical signs in Quick Accent

PR Checklist

[x ] Closes: #24768
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

@microsoft-github-policy-service agree


File write performance

Hello,
I have a short question, do you know a way how to optimice the performance of a file write on a database? At the moment I use openWrite to write a file, to display a progressbar etc. I work here with huge data and many files in total size of 1.5 gb, need in total more than one hour. Is there a way to do this quicker? Threads or push up the cache size? Because I know that my ssd can write quicker. :-) Maybe you have some hints for me.
best regards,
Christopher Proß

Hi, are you talking about write data in LiteDB, right? Or in in generic FileStream?

hm, I mean the filestorage from litedb. Or what do you mean? I have to store some files in litedb, the read performance is very good and now I search for ways to optimise the writing of files. Because I think that the write time of 1.5 hour is very long for "only" 1.6 gb on a ssd disk. I hope you know what I mean.

are you using v4? FileStorage can write 1.6gb in ssd in seconds. Take this example:
var file = @"c:\temp\filestore.db";
var iso = @"C:\Users\David\Downloads\release_Remix_OS_for_PC_Android_M_64bit_B2016112101.zip"; // 1Gb
var sw = new Stopwatch();

using (var db = new LiteDatabase(file))
{
    sw.Start();

    db.FileStorage.Upload("my-iso", iso);

    MessageBox.Show("Total: " + sw.Elapsed.TotalSeconds);
}

Here runs in 13 seconds.... and if you disable journal runs in 10 secods

Strange. I written now a test with upload and a single large file with 1.6 gb, on my ssd it is written in 71 seconds. My program which have to pack many files and all files to getter are 1.6 gb in size. Is this coursing the performance lag? I use in my program which takes so long for writing the openWrite method,  is this version slower then the upload one?

Open a single file is much more fast than thousand files.... but not 1 hour to 1.6. How many files are you saving? What size (in average) are you storing?

Hi,
ok I write a program to answer your question, here is the output:
path: d:\%x%sounds
found 618 files
total filesize: 1,76GB
minimal filesize: 3,98KB
average filesize: 2,91MB
maximal filesize: 35,88MB

please note, the comma is there, because I work on a german System.
So, replace , to .

Ok, I will try write some code to test write speed of files, using this average file size. But 618 is quite easy to write in disk.... even in HDD format... never this need more than hour to execute.
German comma system is same as Portuguese... much easy to understand :)

Ok, tomorow I will post a little code snipped from my pack program, maybe you can find a huge mistake or bug, which course this lag

There are two bottle necks:
LiteFileStream.Write.Flush() has
_engine.Update(LiteStorage.FILES, _file.AsDocument);
Lots of small files will require many calls to update which is slow.
Caching files and updating them in bulk significantly increases throughput.
LiteFileStream.WriteChunks() has
_engine.Insert(LiteStorage.CHUNKS, chunk);
This is called once per chunk.
1GB file upload requires many chunks so many calls to Insert for just one file.
Caching chunks and inserting them in bulk significantly increases throughput.
Neither is supported in the current release.
In our use case, we had 100000 of small text files, which could be easily be uploaded in batches.  The operation was taking several minutes.  Careful use of a caching and reduced the Insert and Update calls reducing the operation down to a few seconds.

@mbdavid:
what do you say to the comment from elyoh?
Maybe that is the solution?

Hi @christopherpross, @elyoh are right... but his case is about 100000 small fils (not 618 as yours). And this make all difference.
When you have lot of small files operations are much more slow than 1 large file. Even if you test copy files in your disk. Every file has some overhead that has difference with size. In LiteDB, every file must write twice _file document to keep begin/finish processes. In a single large file, this twice operation has no difference, but in 100000 has a large difference.
In next version i'm change how database will write data in disk. v5 uses an async queue list of dirty pages, so this will speed up all disk write operation.

OK but the LiteFileStream.WriteChunks() will still break the 1.6 GB file into ~6500 chunks.  This means 6500+ calls to update just to write one file.   This is a still very expensive.
Simply batching the chunks in memory (say 1000 chunks at a time) makes this much more efficient:
        private void WriteChunks()
        {
            var chunkCache = new List<BsonDocument>(1000);

            var buffer = new byte[MAX_CHUNK_SIZE];
            int read;
            _buffer.Seek(0, SeekOrigin.Begin);

            while ((read = _buffer.Read(buffer, 0, MAX_CHUNK_SIZE)) > 0)
            {
                var chunk = new BsonDocument
                {
                    { "_id", GetChunckId(_file.Id, _file.Chunks++) } // index zero based
                };

                // get chunk byte array part
                if (read != MAX_CHUNK_SIZE)
                {
                    var bytes = new byte[read];
                    Buffer.BlockCopy(buffer, 0, bytes, 0, read);
                    chunk["data"] = bytes;
                }
                else
                {
                    chunk["data"] = buffer;
                }
                    // Add the chunk to the local cache and add the chunks
                    // in bulk once the whole file has been chunked
                    chunkCache.Add(chunk);

                 // TODO: keep a count of chunks added and persist cache if it gets larger than 1000 to avoid
                 // OutOfMemoryException and having to copy the entire chunkCache to a new List of larger capacity
            }

            _engine.Insert(LiteStorage.CHUNKS, chunkCache);
            chunkCache.Clear();

            _buffer = new MemoryStream();
        }
    }

In the case of a 1.6 GB file, the chunk cache needs persisting more regularly to avoid possible OutOfMemoryException.  This is a relatively simple modification for existing users hitting this issue.

@mbdavid:
hm, ok but how I can speetup the write of 618 files? Because I need in the next days some inprovements in the performance. Any hints or ideas?

Hi @christopherpross, I will write a test for your case: generate 618 files with random sizes from 3 MB to 36 MB... and after this upload all this files into LiteDB datafile to check this so difference..

@christopherpross, here my example:
using System;
using System.Collections.Generic;
using System.Diagnostics;
using System.IO;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace SmallFilesLiteDB
{
    class Program
    {
        static int N = 618;
        static string _path = @"C:\Temp\SmallFiles\";

        static void Main(string[] args)
        {
            Directory.CreateDirectory(_path);

            var sw = new Stopwatch();

            sw.Start();

            CreateFiles();

            Console.WriteLine($"Create files: {sw.Elapsed.TotalSeconds} seconds.");

            sw.Restart();

            CreateDatabase();

            Console.WriteLine($"Upload to LiteDB: {sw.Elapsed.TotalSeconds} seconds.");

            Console.ReadKey();

        }

        static void CreateFiles()
        {
            var rnd = new Random();

            for(var i = 1; i <= N; i++)
            {
                File.WriteAllBytes(_path + i.ToString().PadLeft(3, '0') + ".bin", new byte[rnd.Next(4 * 1024, 5 * 1024 * 1024)]);
            }
        }

        static void CreateDatabase()
        {
            using (var db = new LiteDB.LiteDatabase(_path + "test.db"))
            {
                foreach(var file in Directory.GetFiles(_path, "*.bin"))
                {
                    db.FileStorage.Upload(Path.GetFileNameWithoutExtension(file), file);
                }
            }
        }
    }
}

My results was: 4 seconds to create files and 18 seconds to store inside LiteDB (my computer is a dell i7 + 16gb + ssd). Run this test to see if you get similar time.  Maybe your problem can be with how you get this files (are inside same machine?)

Hey,
very big sorry for this delay, it was all my fault, I had used a very ineffiziently code to upload.
I close this now.
sorry for that.

Is there any plan for getting some control of chunking in LiteDB? The FileStorage.Upload() operation is taking ~400 ms for a 3 MB file on my machine, and I have ~20 files to upload in our SW. The "cached chunking" option provided by @elyoh above is very nice, and accelerates the operation down to about 80 ms. I can fork the repo and make the change but was wondering if there was any chance this might get added into the main source code. Would be great to have control over the chunking from the FileStorage.Upload() function, IMO.


feat: add ruff-lsp support

Hey @glepnir, I just noticed the file should be named ruff_lsp.lua instead of ruff-lsp.lua, can you fix that?

sure

Thanks for this.


Adds feature flag to dump workers

Adds a feature flag check to the dump workers for classifications, workflows, subjects, and aggregations. The API will return an error that says that exports are disabled. The sidekiq jobs will hit the flag, raise an exception, and fail however many (25?) times and then be put in the dead queue where we can run them later...or not.
I have been messing with sidekiq middleware all afternoon trying to conditionally set the retry amount but it doesn't really matter. If we want to reschedule these jobs instead of basically ignoring them, I'll need to know what that schedule looks like. If we do flip this switch, we should ensure that PFE is going to do something smart with the response.
Closes https://github.com/zooniverse/Panoptes/issues/2252 and https://github.com/zooniverse/Panoptes/issues/2118 (more or less, depending on rescheduling thoughts).
Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

Forgot to rescue from the raised exception and have the controller respond with something. I don't immediately see a place to test this.

Closes https://github.com/zooniverse/Panoptes/issues/2252


[PHP] Update and extend setup for auto instrumentation

thx for fixing the formatting, I think you need to add more tabs/spaces to the code snippets, since they are not in line with their bullet point:
https://deploy-preview-2151--opentelemetry.netlify.app/docs/instrumentation/php/automatic/#setup

thx for fixing the formatting, I think you need to add more tabs/spaces to the code snippets, since they are not in line with their bullet point:
https://deploy-preview-2151--opentelemetry.netlify.app/docs/instrumentation/php/automatic/#setup

ok, thank you

Merging since you're a maintainer :)


.vue file support

Hi, Thanks for the great work, please add .vue file support for Vue.

feel free to create a PR ;-)

Tricky since a .vue file contains HTML|JSX, CSS|SCSS|LESS|PostCSS, & JS|TS|CS.
What would be the best approach @flosse?   Split up the file, parse the pieces, then concoct?


"editor.formatOnSaveMode": "modificationsIfAvailable" often broken

Does this issue occur when all extensions are disabled?: Not applicable : Extensions needed

Version: 1.94.2
Commit: 384ff7382de624fb94dbaf6da11977bba1ecd427
Date: 2024-10-09T16:08:44.566Z
Browser: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Code/1.94.2 Chrome/124.0.6367.243 Electron/30.5.1 Safari/537.36
See https://github.com/microsoft/vscode/issues/203463
"editor.formatOnSaveMode": "modificationsIfAvailable" is often broken, even though its description states 👍 > Will attempt to format modifications only
(requires source control). If source control can't be
used, then the whole file will be formatted.
Couldn't it fallback to file also when modificationsIfAvailable fail ?

I also have this issue. The formatter (with modificationsIfAvailable) works while I am dealing with merge conflicts but not when VCS is fully tracking changes.

Couldn't it fallback to file also when modificationsIfAvailable fail ?

How is that defined? Could only be throwing an error because not returning formatting edits could also mean everything is formatted correctly. I will mark this as feature request but I strongly recommend to forward this to the respective extensions because vscode isn't doing anything special here, the modificationsIfAvailable-feature is basically a bulk formatting ranges so the same as repetitive format selection calls

hey @jrieken, this is a bug not a feature request


release: update version to v0.10.1

v0.10.1 up for integration testing on http://beta.piskel-app.appspot.com

This change is

Current release log:
https://github.com/juliandescottes/piskel/compare/v0.10.0...b8fed60fa6093c5a87fdfc249f253ff28b0b9f75


Wanted Quest is missing a requirement

Wanted Quest is missing the requirement "Enter the Abyss"

can confirm this

Added with https://github.com/Zoinkwiz/quest-helper/commit/3ab2980513a028c680ced9c8201a4467db33bf35


FR: Delete objects with confirmation

Case:

We create objects and they are working.
We send command to delete all objects. Not all objects (Pods) can be deleted immediately and some still are working and they are writing data(until their death). But we already reported about deletion and a next Jenkins task begins started, which want that data will not be changed. It's the problem.

it sounds like your scenario is effectively the same as running:
oc delete pod foo
and then while the command returns immediately, the pod keeps running for some period of time until it is killed.  Is that accurate?
If so, i think the only way we can do better here is to offer a way for you to "wait for termination" when deleting objects (we'd have to watch/poll to see when the object actually goes away).
Note that your containers should be getting a SIGTERM when the delete occurs, so if they are continuing to do work, they are ignoring that signal (in which case k8s gives them about 30s to terminate before it hard-kills them).  You might be able to modify the grace period such that the pod is immediately terminated instead, as a workaround.

(will create trello card or incorporate into existing card if applicable once we lock down the detail or two @bparees asked @livelace about)

and then while the command returns immediately, the pod keeps running for some period of time until it is killed. Is that accurate?

Yes, absolutely. The period depend on the node utilization, application inside pod ... etc In my case the problem is in parse xml file, which locked inside pod and the next Jenkins step can't handle it correctly.

If so, i think the only way we can do better here is to offer a way for you to "wait for termination" when deleting objects (we'd have to watch/poll to see when the object actually goes away).

It sounds good.

I've opened this trello card to track this request ... closing github issue out per process.

@livelace did you try setting the grace period to 0s?  i'm curious if that would fix your problem:
https://github.com/kubernetes/kubernetes/blob/master/pkg/api/v1/types.go#L160-L164

@bparees I didn't find the "deletionGracePeriodSeconds" parameter in my "ClusterPolicy". But I compared situation with deletion of objects.

for type in route rc dc svc pod;do for i in $(oc get $type | awk '{print $1}');do oc delete $type $i;done;done - one Pod which did some operations (postStart hook) was alive for 1-2 minutes in the "Terminating" state.
for type in route rc dc svc pod;do for i in $(oc get $type | awk '{print $1}');do oc delete --grace-period=0 $type $i;done;done - it killed all immediately

I think, that both parameters are needed, because someone wants to get completely finished writing of data, and someone wants to get the immediately deleted objects.
I want that it could complete all tasks and shift to the next step :)

@bparees I didn't find the "deletionGracePeriodSeconds" parameter in my "ClusterPolicy"

it's not a cluster policy setting, it's a setting on each resource (in this case the pod resource).
But yes if your pods need that grace period to properly clean up then setting it to zero won't be a valid workaround for your scenario, unfortunately.


go-redis using a bad package naming

You shouldn't put 'package redis' and 'redis-test' under a SAME directory, why did you do that?

Go has the same practice: https://github.com/golang/go/blob/master/src/context/example_test.go


Try to fix broken /preview routing in stage

Codecov Report
Patch and project coverage have no change.

Comparison is base (3c2694c) 85.19% compared to head (2408c83) 85.19%.

:exclamation: Current head 2408c83 differs from pull request most recent head 0de27f4. Consider uploading reports for the commit 0de27f4 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #85   +/-   ##
=======================================
  Coverage   85.19%   85.19%
=======================================
  Files          53       53
  Lines        1162     1162
  Branches      370      370
=======================================
  Hits          990      990
  Misses        153      153
  Partials       19       19

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update d2 Go dep

go get -u<EMAIL_ADDRESS>go mod tidy

Meant to address:

https://github.com/dagger/dagger/security/dependabot/389
https://github.com/dagger/dagger/security/dependabot/390

Sharing context: after a quick sync with @jedevc, the plan for this is to re-focus on d2, and leave the docker & fs-util deps to him, for a separate PR.

As soon as all checks pass, I will go ahead and merge this. Thanks @aluzzardi!


fix(ebs): improve snapshot encryption logic and typos

Context
Currently the EC2 EBS snapshot encryption check results in false negatives.
Additionally, I have fixed two typos. One includes more accurately describing the function __get_snapshot_public__
Description
This PR fixes the logic to accurately FAIL unencrypted EBS snapshots. It also more accurately describes the __get_snapshot_public__  logging statement.
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@taylerhaviland I have made a change in the code because we need to check if the key Encrypted exists. Let me know if it still works on your side.

Hi @sergargar, your change has given me the expected results. Thanks!

Hi @sergargar, your change gives me the expected results.
LGTM


create checkbox shared component

create a Checkbox component -- it's very similar to the CheckboxButton component in functionality, but without the pseudo-button styling.

:rocket: PR was released in v0.19.0 :rocket: