Existing documentation for OAuth2 configuration is out of date

The instructions provided at https://cloud.google.com/bigquery/docs/authentication/end-user-installed don't work, because the object returned by authFlow doesn't conform to the GoogleAuthOptions parameter of the BigQuery constructor. After digging around in the code, trying approaches like setting the clientOptions, overriding the auth client after the constructor, and manually creating a server so I can use the web-based flow, I'm still no closer to being able to run queries on behalf of users using their credentials.
Environment details

OS: Ubuntu 20.04
Node.js version: 12.8.1
npm version: 7.5.2
@google-cloud/bigquery version: 5.5.0

Steps to reproduce

Try to authenticate using the method provided at https://cloud.google.com/bigquery/docs/authentication/end-user-installed

Hi @JamesYeoman — thanks for the report! Can you provide a little bit more info about the kinds of issues/errors you're encountering? We'll try to repro as well.

Any attempt to get the sdk to use OAuth credentials fails because it tries to then load the Application Default credentials. Service Accounts work fine, but I need to be able to use the SDK with the end-user's credentials (because permissions)

I should probably also point out that I'm using Typescript, so I need to conform to the provided interfaces, even if the underlying JS would accept other data

Upon closer inspection of the auth code, and following the data flow, I've come to the conclusion that the getClient function on line 490 of googleauth.js in google-auth-library is likely the culprit.
If neither jsonContent nor keyFilename are defined, it falls back to application default credentials, but in order to use UserRefreshClient as the auth client, the clientOptions field on GoogleAuthOptions needs to contain a type property with the content 'authorized_user' (as found in the _cacheClientFromJSON function on line 289 of googleauth.js in google-auth-library.
However, the only way to populate the jsonContent field of the GoogleAuth class, fromJSON needs to be called, and there isn't anywhere in the flow that I've identified that specifies any JSON content. I'm going to try loading the credentials manually after creating the bigquery instance to see if that works, and then I'll report back

So, it's still trying to load the application default credentials. I've created a gist of the updated code (I re-did the functionality of readline-promise because there isn't any type definitions available for it at the moment), so if anyone more familiar with the codebase could take a look to see if anything stands out to them?
https://gist.github.com/JamesYeoman/50a935e7a6245effbe8755ad113dcb9e

Ah, I tried assigning the result of client.authClient.fromJSON to the cached credential, and it worked :tada:
Would it be possible to have this added to https://cloud.google.com/bigquery/docs/authentication/end-user-installed as a NodeJS Typescript method?

I think the reason you are encountering this problem is because your authFlow function is returning only credentials rather than both projectId and credentials, as shown in the sample code.
// sample authUserFlow.js line #99
return {
  projectId,
  credentials,
};

// your gist line #55
return credentials;

I don't think you will need this workaround if you return the object with both fields, as shown in the sample.

Please reopen if initializing client with both credentials and projectId does not solve the issue.


Subclassing of Form components doesn't work well

I tried subclassing EditNumber to implement something similar to this
It doesn't work because the genered code from a razor file calls base.BuildRenderTree - you get the rendering of the base class and what you wrote.

We should be able to fix this by making it unnecessary for a component to call base.BuildRenderTree. In general, it's a good idea if we can avoid extensibility points where an inheritor has to call base

Completely agree, we should not be forced to call the base. Currently if the base is not called the StateHasChanged stops working.


How can I use Ky for react SSR?

How can I use Ky for react SSR?

This is in the readme: https://github.com/sindresorhus/ky-universal#how-do-i-use-this-with-a-web-app-react-vuejs-etc-that-uses-server-side-rendering-ssr

It does not actually show any example of real world usage with React renderToString.

What exactly do you want us to show? Should it include the webpack config, server framework with routes, etc? A real SSR setup can get a bit complex depending on your needs.
Generally speaking, I would suggest using something like Next.js that takes care of the rendering under the hood. Then it's really as simple as just calling ky() wherever you want to fetch the data.

Yes please show a real SSR. Otherwise it is very wrong to claim Ky is SSR.

You did not answer my question.


Filter design incorrect

The current filter selection design for multiselects is not correct.
Change to the filter design as specified in figma:

Having two buttons for setting a filter can be redundant and can overcomplicate the user interface


when creating a rabbitMQ fragment 'Queue' and 'Exchange name'-'Routing key' should mutually exclusive

If we want to add a rabbitMQ input fragment, 'Queue' and 'Exchange name'-'Routing key' should be mutually exclusive.
If we specify a Queue, we should not be able to provide a Exchange name-Routing key pair and the other way around.

It would probably be enough with modifying the template.
Communicate front this change, in case they need to modify something.


Stop customhostnames from waiting for SSL if it's already active

This can happen if you are supplying a custom certificate.
The hostname will go active in SSL and never go pending. Resulting in the provider waiting forever.

i inverted the logic check here to return early as the != active combined with the other expression was a little confusing to grok when it should kick in.

thanks @cdloh 🙇


[web]: Dark Mode for Web

Use case

Light mode
Dark mode

Hi! I don't know if you are considering to add support to dark mode on web but I think it would be an awesome feature.
Apparently there is a media query 'prefers-color-scheme' that indicates the user theme preference:
https://chromestatus.com/feature/5109758977638400.
Testing
Here I made a demo working with javascript that updates when you change the device theme.
https://www.w3schools.com/code/tryit.asp?filename=G82VZTDRDDND (Tested on MacOS with Chrome 76)

Light mode
Dark mode

I don't know how hard it's to implement it with the flutter engine.
I will try to make a plugin ~in the following days~ as soon as plugins are available for web 🙃

I updated the description with more details

Since the brightness property is on the dart:ui window, this would need to be done in the engine and not in a plugin

Since the brightness property is on the dart:ui window, this would need to be done in the engine and not in a plugin

Oh true, I just saw it, it is named _platformBrightness.
If web dark mode is not a feature planned short-term, I think I could try a workaround using 'prefers-color-scheme'  and adapting the themes within a plugin.
I would like to contribute but I don't how I can, flutter engine looks too complex for me.

if you can write a small Dart program that can pull the platform brightness, even if you don't land it yourself I can take a look and adapt it to the flutter engine

I can't believe I succeed doing it. Here it is the dart file.
import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart';

// ignore: uri_does_not_exist
import 'dart:html' as HTML;

// ignore: uri_does_not_exist
import 'dart:js' as JS;

void main() {
  debugDefaultTargetPlatformOverride = TargetPlatform.fuchsia;
  runApp(MyApp());
}

class MyApp extends StatefulWidget {
  @override
  State<StatefulWidget> createState() => _AppState();
}

class _AppState extends State<MyApp> {
  Brightness brightness = Brightness.light;

  final mq = HTML.window.matchMedia('(prefers-color-scheme: dark)');

  @override
  void initState() {
    super.initState();
    onBrightnessChange(mq);
    mq.addListener(onBrightnessChange);
  }

  ThemeData get theme => ThemeData(
        brightness: brightness,
      );

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
        title: 'Flutter Demo',
        theme: theme,
        home: Scaffold(
          body: Center(
            child: Text('Hello Flutter Community!'),
          ),
        ));
  }

  @override
  void dispose() {
    mq.removeListener(onBrightnessChange);
    super.dispose();
  }

  onBrightnessChange(mq) {
    if (mq.matches) {
      setState(() {
        brightness = Brightness.dark;
      });
      print('window is dark');
    } else {
      setState(() {
        brightness = Brightness.light;
      });
      print('window is light');
    }
  }
}

I could try to add it on the engine but don't know where to start

cc @yjbanov @mdebbar maybe you could give @jamesblasco some help here?

Great research, @jamesblasco! It shouldn't be hard to implement in the engine. All you need to do is implement Window.platformBrightness. Need to be careful with cross-browser support though.

@yjbanov about cross-browser support.
The API for finding out whether or not a media query applies to the document is available for all browsers => https://caniuse.com/#search=matchMedia.
This api is already implemented at dart:html => https://api.dartlang.org/stable/2.5.0/dart-html/Window/matchMedia.html
Right now the 'prefers-color-scheme' media query is available for Chrome, Firefox, Opera, and Safari  for MacOS but not iOS.  => https://caniuse.com/#search=prefers-color-scheme
In the case of 'prefers-color-scheme' not being available it should be set as Brightness.light by default.

@jamesblasco That sounds like a good approach. Would you be interested in sending a PR to the engine with a fix? As a side benefit it would be valuable for us to know how easy it is to contribute to the engine. If there are any barriers we'd like to remove them.

So I have been researching a bit for how should it be done. And my assumptions are:
Create a file similar to engine/keyboard.dart for the brightness or all the 'flutter/settings' channel messages
One doubt I have is why 'flutter/settings' channel is not implemented on the web engine while exits for the other platforms. (Ex. Android)
It could be added here at the engine/window.dart, and from there I am not sure how would be the best approach to access the private property io.window._platformBrightness and change it.
Example of Brightness file
part of dart.ui;

/// Provides brightness bindings, such as the `flutter/brightness` channel.
class BrightnessEvent {
  /// Initializes the [BrightnessEvent] singleton.
  ///
  /// Use the [instance] getter to get the singleton after calling this method.
  static void initialize() {
    _instance ??= BrightnessEvent._();
  }

  /// The [BrightnessEvent] singleton.
  static BrightnessEvent get instance => _instance;
  static BrightnessEvent _instance;

  final _mediaQuery = html.window.matchMedia('(prefers-color-scheme: dark)');

  html.EventListener _brightnessListener;

  BrightnessEvent._() {
    _brightnessListener = (html.MediaQueryListEvent event) {
      _handleMediaQueryEvent(event);
    };
    _handleMediaQueryEvent(_mediaQuery);
    _mediaQuery.addListener(_brightnessListener);

  }

  /// Uninitializes the [BrightnessEvent] singleton.
  ///
  /// After calling this method this object becomes unusable and [instance]
  /// becomes `null`. Call [initialize] again to initialize a new singleton.
  void dispose() {
    _mediaQuery.removeListener(_brightnessListener);
    _brightnessListener = null;
    _instance = null;
  }

  static const JSONMessageCodec _messageCodec = JSONMessageCodec();

  void _handleMediaQueryEvent(mq) {
    final Map<String, dynamic> eventData = <String, dynamic>{
      'type': event.type,
      'platformBrightness': mq.matches ? ui.Brightness.dark : ui.Brightness.light
    };
    ui.window.onPlatformMessage('flutter/settings',
        _messageCodec.encodeMessage(eventData), _noopCallback);*/
  }
}

void _noopCallback(ByteData data) {}

Another question I have is where it would be the appropriate init to initialize call BrightnessEvent.initialize(). I think here next to the Keyboard.initialize();

@yjbanov I have never done a pull request before on GitHub, but I think it can’t be hard.
Related to the engine. I have to follow the  Setting-up-the-Engine-development-environment document, no?
For compiling it there is not specific instructions for web at Compiling-the-engine document.
Does it work like the other platforms? Something like:
Compiling for Web (Inspired from Compiling for iOS)
These steps build the engine used by flutter run for Web.

Run the following steps, from the src directory created in the steps above:

- git pull upstream master in src/flutter to update the Flutter Engine repo.

- gclient sync to update dependencies.

- ./flutter/tools/gn --unoptimized to prepare the build files for host-side executables.

-- [No idea what should be done here, maybe a dart2js command?]   ninja -C out/ios_debug_unopt && ninja -C out/host_debug_unopt to build all artifacts (use out/ios_debug_sim_unopt for Simulator).

Related to the engine. I have to follow the Setting-up-the-Engine-development-environment document, no?

Correct.

For compiling it there is not specific instructions for web at Compiling-the-engine document.
Does it work like the other platforms?

After you are done setting up your engine development environment, you could try out the new tool we've been working on. The tool is called felt and it aims to facilitate the development of the web engine. See more information in this README file. Please let us know if you encounter any problems.

Oh @mdebbar, thanks for the info! I will give it a try tomorrow (It's midnight here in spain). The tool looks very intuitive.

@mdebbar or @yjbanov, could any of you give me a feedback of this approach I wrote in a previous comment? Thanks ☺

@jamesblasco I don't think you need the 'flutter/settings' channel. That channel is only used to communicate between the platform thread (written in Java/ObjC) and the engine thread (written in C++). Platform channels seemed like an easy way to achieve this, since they are language-agnostic and thread-safe.
However, on the Web there is no language barrier, nor are there thread-safety concerns. So I think we should just implement it in plain Dart exactly how you proposed in https://github.com/flutter/flutter/issues/40605#issuecomment-531987246.

@jamesblasco here a few ideas and thoughts:

The framework already uses window.onPlatformBrightnessChanged to listen to brightness changes. So there is no need to use the flutter/settings platform channel.
engine.window should override the platformBrightness getter and have its own private _platformBrightness.
The BrightnessEvent class should be part of engine not part of ui. Then it would have access to private properties of engine.window.
Maybe rename BrightnessEvent to BrightnessWatcher or BrightnessListener?

Hope these ideas help guide you in the right direction.
In any case, please go ahead and open a PR and I look forward to reviewing it :)

@mdebbar thanks for the feedback, but following the @yjbanov i did it inside the ui.Window class.
I opened a PR (12335) some hours ago (my first one ever 🥳). Let me know if you think it would be better to group it separated in a BrightnessListener class.

Fixed by https://github.com/flutter/engine/pull/12335


TypeError: undefined is not an object

Summary

Version
0.21.5

Affected OS
Android

OS Version
7.0

Environment:
OS: Windows 10
Node: 9.0.0
Yarn: 1.6.0
npm: 6.1.0
Watchman: Not Found
Xcode: N/A
Android Studio: Version  <IP_ADDRESS> AI-171.4408382
Packages: (wanted => installed)
react: ^16.3.1 => 16.4.0
react-native: ^0.55.4 => 0.55.4
Current behavior
Using react-native I have made sure all the linking has been done correctly (only testing android so far)
I have deleted all build directories and node_modules and re-run yarn
I am just trying to use the API:
import DeviceInfo from "react-native-device-info";
...
let deviceId = DeviceInfo.getUniqueID();

I get the error:

TypeError: undefined is not an object (evaluating 'RNDeviceInfo.uniqueId')

Expected behavior

duplicate of #383


Fix/open relative on dev

When formatting the links on the markdown, we're calling removeMarkdownEscapes which removes the any matching the following regex: /\([\`*_{}[]()#+-.!])/
So if the path is below: (it's getting rid of the \. before build in the path)
$localdrive\git\azuredatastudio\.build\builtInExtensions\Microsoft.sqlservernotebook\books\sqlserver2019\content\common\sop007-get-key-version-information.ipynb
This PR fixes #8147

@MaddyDev Looks like you need to format notebookMarkdown.ts

Make sure you fix the format error and verify checks pass before merging in

Sure. Thanks!


A noqa for specific warnings/errors per block/line

It would be nice if we could ignore warnings and errors on a per block or per line basis much like # noqa at the end of a line for example.
Use case:

Ignore a block of code (a function) that's considered by mccabe to be "too complex" but you don't have the time to fix it right now.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/1561076-a-noqa-for-specific-warnings-errors-per-block-line?utm_campaign=plugin&utm_content=tracker%2F394650&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F394650&utm_medium=issues&utm_source=github).

Any traction on this? I too would like to be able to mark specific errors at a line level.

I would suggest the following notation as supported by flake8.
Flake supports to list specific errors only, by using
noqa:
See here: https://pypi.python.org/pypi/flake8

Because of this (and the plug in structure, which allows me to write or use plug-ins for some nitpicks specific to our code), I have since moved to using flake8 exclusively

How can I make flake8 ignore a block of code rather than just by line?

Hi all, thanks for making pylama!
Just thought I'd chime in after coming across this via Googling for pylama ignore block.
@klen 's answer above I think is a perfectly reasonable conclusion to the title of the issue. you can use:

  # noqa at the end of a line
# pylama:ignore=<error code> at the end of a file
[pylama:<path glob>] in an *.ini file with ignore=<error code> statements underneath

to ignore warnings.
If you have a block of code for which you have different syntactic standards, it seems valid to move it into its own file and add the ending pylama:ignore or add it to the *.ini.
It seems like this issue could be closed.

IMO, this issue still needs a resolution for how to disable linting for a small BLOCK of code within a larger file. I'd love to see something like the following:

# pylama:ignore=<codeid>

# pylama:select=<codeid>

Where the code above and below the block are correctly linted for  issues, but the code between is ignored. Here's an example:
BadVariableName1 = 1

# pylama:ignore=C0103
BadVariableName2 = 2
BadVariableName3 = 3
# pylama:select=C0103

BadVariableName4 = 4

The above code would NOT produce a C0103 error for BadVariableName 2 and 3, but WOULD find 1 and 4:
Constant name "BadVariableName1" doesn't conform to UPPER_CASE naming style [pylint]
Constant name "BadVariableName4" doesn't conform to UPPER_CASE naming style [pylint]

Obviously, this is a simplistic example because the lines could be individually ignored. But I have a real use-case in my code where ignoring individual lines is difficult. For example, this causes a syntax error:
from unittest.mock import patch

with patch("path.to.ObjectA", autospec=True) as MockObjectA, \  # noqa
        patch("path.to.ObjectB", autospec=True) as MockObjectB:  # noqa
    pass

"Expected expression" (Pylance)
"Expected \":\"" (Pylance)
"Statements must be separated by newlines or semicolons" (Pylance)
"Expected expression" (Pylance)
"Unindent amount does not match previous indent" (Pylance)
"Variable name \"MockObjectA\" doesn't conform to '[a-z_][a-z0-9_]{,30}$' pattern [pylint]" (pylama)
"Variable name \"MockObjectB\" doesn't conform to '[a-z_][a-z0-9_]{,30}$' pattern [pylint]" (pylama)

I do not want to disable the error from the whole file, so I'm forced to break the lines differently:
with patch(
        "path.to.ObjectA", autospec=True) as MockObjectA, patch(  # noqa
        "path.to.ObjectB", autospec=True) as MockObjectB:  # noqa
    pass

But I would LOVE the ability to just do:
# pylama:ignore=C0103
with patch("path.to.ObjectA", autospec=True) as MockObjectA, \
        patch("path.to.ObjectB", autospec=True) as MockObjectB:
    # pylama:select=C0103
    pass


GitStatus example breaks color escape sequences on conhost.exe

Sharing $host between multiple Runspaces seems to be causing the issue.
The same issue occurs with ThreadJob too: https://github.com/PowerShell/PowerShell/issues/19348

Thank you for the report @Callidus2000
It has not been released in the PowerShell Gallery yet but it should be fixed with the latest code.


feat: preflight for x86-64 v2 CPU support

What this PR does / why we need it:
preflight for x86-64 v2 CPU support.

Wording lgtm


Find Stock Audit By Closed

[x] findByCloesd
[x] test

I made a mistake with the estimation time instead of putting point:1 I put 1h


Is it possible to train on the entire dataset?

Hello, thank you very much for your work. Can I train on the entire dataset? I saw in the paper that 2-16 images are trained. Is it possible if I train on the entire dataset?

Hello, thank you very much for your work. Can I train on the entire dataset? I saw in the paper that 2-16 images are trained. Is it possible if I train on the entire dataset?

Hi, thank you for your interest in our work. Are you referring to training InvMIHNet on a different dataset? If so, you will need to train the IIH module and IR module separately and then perform further fine-tuning. If that's not the case, please provide more details regarding your question.


Fix 581 - Async.RunSynchronously tries to reuse current thread

Fixes https://github.com/Microsoft/visualfsharp/issues/581 - Async.RunSynchronously tries to reuse the current ThreadPool thread for running the computation (originally the computation was always performed in another ThreadPool thread).
Currently there are two problems:

The code doesn't use Thread.IsThreadPoolThread since it isn't available in all profiles (eg. in profile 47). Instead value of Thread.IsThreadPoolThread is approximated by SynchronizationContext.Current = null.

Current ThreadPool thread is reused only if no timeout was given to Async.RunSynchronously. The reason is that some thread is needed to cancel the computation when the time runs out and the current thread seems as a good candidate for this task. In some profiles I could use System.Threading.Timer or CancellationTokenSource.CancelAfter (internally uses Timer) but they don't ensure that cancellation is performed when ThreadPool is busy. For example following code

open System
open System.Threading

let startWork (token : CancellationToken) (ev : CountdownEvent) n =
    let work () =
        Console.Write("{0}|", Thread.CurrentThread.ManagedThreadId)
        while not token.IsCancellationRequested do
            Thread.Sleep(10)
        Console.Write(".")
        ev.Signal() |> ignore
    for _ in 1..n do
        if ThreadPool.QueueUserWorkItem(fun _ -> work ()) then
            ev.AddCount()

[<EntryPoint>]
let main argv =
    use cts = new CancellationTokenSource(2000)
    // `timer` ensures that action from `Timer` from `cts` is placed at the end of `ThreadPool` queue.
    // Note that `timer` declaration must follow `cts` declaration and timeouts must be same.
    use timer = new Timer((fun _ -> Console.Write("timer {0}|", Thread.CurrentThread.ManagedThreadId)), null, 2000, -1)
    use finished = new CountdownEvent(1)

    let sw = System.Diagnostics.Stopwatch()
    sw.Start()
    startWork cts.Token finished 200
    finished.Signal() |> ignore
    Console.Write("# tasks {0}|", finished.CurrentCount)
    finished.Wait()
    sw.Stop()

    printfn "Elapsed: %A" sw.Elapsed.TotalSeconds
    0

requests cancellation after 2 seconds but cancellation actually happens after 193 seconds on my machine and if the ThreadPool reaches its maximal capacity (eg. by ThreadPool.SetMaxThreads(4, 4)) then the cancellation never happens.
The third option is to create a single special thread which will be dedicated for cancellation (ie. for calling CancellationTokenSource.Cancel). This special thread could be used by all calls to Async.RunSynchronously.

Hi @radekm, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

This enhancement looks correct to me.  Would welcome additional reviewers.

I have improved the test AsyncRunSynchronouslyReusesThreadPoolThread by adding a comment and enclosing the call which may raise TimeoutException in Assert.DoesNotThrow

This looks great.


ci: remove rust cache for platform builds

When compiling for specific targets, the rust cache is almost always a miss.
This causes the rust cache to only cost more time for a release, because it takes quite a long time to upload the cache.
Plus, these caches take space, which is stolen from the main CI caches.
So remove the rust caches for platform-specific builds.

Closing this, because the cache is used for subsequent builds.


Problem using ESP8266 nodeMCU 1.0

Describe the bug
When using the esp8266 nodemcu 1.0 board, with the esp core installed and 1.9.3 of WiFiWebServer, on compile the WifiWebServer reports "#error Unknown or unsupported board" and fails.
Using Arduino 2.0 rc 9.3.
Installed the esp core. The board is recognized and is selected correctly.
Steps to Reproduce
Sample sketch to reproduce with nodemcu 1.0 (ESP-12E) selected as board.
#include <ESP8266WiFi.h>
#include <WiFiWebServer.h>
void setup() {
// put your setup code here, to run once:
}
void loop() {
// put your main code here, to run repeatedly:
}
Expected behavior
I am making the assumption as the documentation says that the ESP8266 is supported, it should work without modifications.
Actual behavior
The compile fails.  I am able to make it work by modifying the WiFiWebServer.hpp to include the following at line 166:
#elif (ESP8266)
#define USE_WIFI_NINA         false
// To use the default WiFi library here
#define USE_WIFI_CUSTOM       true
Arduino IDE version: 2.0 rc9.3
ESP32 Core Version  3.0.2
OS: Windows 11 Home

Hi @neckkola
Thanks for reporting the problem. I just forgot about popular ESP32/ESP8266 recently and therefore the unintended error message.
Please change
https://github.com/khoih-prog/WiFiWebServer/blob/1c28df110fb3624122adbebbcc2d427399b96045/src/WiFiWebServer.hpp#L167-L171
as follows
#elif !(ESP32 || ESP8266)

  #error Unknown or unsupported board

#endif

I'll fix the issue very soon for next release.

Thank you so much and for a great product.  Have a wonderful day.

Hi @neckkola
The new WiFiWebServer releases v1.9.4 has just been published.

Releases v1.9.4

Restore support to ESP32 and ESP8266. Check Problem using ESP8266 nodeMCU 1.0 #20


Add simulation reset capability

This PR addresses issues with restarting and reloading simulations.
Current issues

Pause is not implemented efficiently. It waits using usleep instead of a signal.

[X] Added support for dumping individual global variables.
[ ] Add ability to reload and initialize

Documentation changes

[X] '/Command/Globals' added (it was missing)
[X] '/Server/Reset'

Test and Validation Notes

[ ] TODO: this is very difficult to validate

This PR is stale and needs to be completed later.


how to get amount of data which was uploaded and downloaded?

Summary
Steps to reproduce
(Please provide detail steps).
What is the current bug behaviour?
(What actually happens).
What is the expected correct behaviour?
(What you should see instead).
Relevant logs and/or screenshots
Possible fixes suggested re-mediation
Assignees and labels
( please add labels as applicable )

[ ] Patched
[ ] Verified patch

https://github.com/pia-foss/vpn-ios/issues/1


Bug with MeasureCharacterRanges on Linux, if text contains LF/CR

I'm using the System.Drawing.Common NuGet package.
Main problem: there is a difference in how method MeasureCharacterRanges work on Windows and on Linux.
On Windows this code works correctly: each char -> rectangle, CR/LF -> empty rectangle.
On Linux: CR symbols disappear, the first symbols of each line after LF also disappear.
Code:
                    var g = Graphics.FromImage(new Bitmap(1, 1));
                    var font = new Font("Arial", 11f);
                    var rect = new RectangleF(0, 0, 99999, 99999);
                    var sf = new StringFormat();
                    string text = "12\r\n34\r\n56";
                    var count = text.Length;

                    CharacterRange[] cr = new CharacterRange[count];
                    for (int index = 0; index < count; index++)
                    {
                        cr[index].First = index;
                        cr[index].Length = 1;
                    }
                    sf.SetMeasurableCharacterRanges(cr);
                    Region[] ranges = g.MeasureCharacterRanges(text, font, rect, sf);

                    RectangleF?[] rects = new RectangleF?[count];
                    for (int index = 0; index < count; index++)
                    {
                        rects[index] = ranges[index].GetBounds(g);
                    }

Results on Windows:

Results on Linux:

I highlighted the symbols and their rectangles in yellow.

I have an idea where the problem might be.
I analyse method  cairo_MeasureCharacterRanges  in  libgdiplus/src/text-cairo.c
This method call MeasureString and use resulting StringDetails from it.
But inside MeasureString the input string is processed (CR ignored, LF to STRING_DETAIL_LF)
and resulting StringDetails will be SHORTER than the input string!
So now is impossible to match the input char and corresponding rectangle ....

The Cairo text code is known to be broken (https://github.com/mono/libgdiplus/pull/137 for some background). Newer versions of libgdiplus try to use Pango for the text layout and rendering instead.

How can I use the newer version of libgdiplus ?
I 'm a newbie to Linux and Docker, and now I use the following code in the Dockerfile:
FROM mcr.microsoft.com/dotnet/core/aspnet:3.1 AS base
RUN apt-get update &&
apt-get install -y libgdiplus libc6-dev

Depending on your distro and how you acquired the libgdiplus (Mono repositories vs distro repositories) it may already be included.

I'm a newbie to Linux and Docker, use Dockerfile:
FROM mcr.microsoft.com/dotnet/core/aspnet:3.1 AS base
RUN apt-get update &&
apt-get install -y libgdiplus libc6-dev

I'd recommend to follow the first step from https://www.mono-project.com/download/stable/#download-lin for your particular distribution. Then apt-get install -y libgdiplus or something similar.

Thanks for the help here @filipnavara. Might be good to add this URL in the repo readme.
cc: @directhex


More models?

 And how can I set the quality of the end result?

If you can convert to .obj three.js will accept it. You can probably find a script to convert, or use Blender.

If you are a blender user then download the three.js exporter for revision 'r69'. I tested it and it works to load your own models. To add your model to the list of items in the "Add Items" tab, edit the "examples/js/items.js" file to add your own entry. The details are pretty self-explantory. It requires path to the json file, item-type (refer to factor.ts in src/items/factory.ts), and path to thumbail.
Regards,
#0K

Is there any one have the whole project completed yet. If tes ldt me know... i will be interested to buy a complete product with a reasonable price.
Sent from my Samsung device
-------- Original message --------
From: aalavandhaann<EMAIL_ADDRESS>Date: 9/2/17 11:39 AM (GMT+08:00)
To: furnishup/blueprint3d<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [furnishup/blueprint3d] More models? (#22)
If you are a blender user then download the three.js exporter for revision 'r69'. I tested it and it works to load your own models. To add your model to the list of items in the "Add Items" tab, edit the "examples/js/items.js" file to add your own entry. The details are pretty self-explantory. It requires path to the json file, item-type (refer to factor.ts in src/items/factory.ts), and path to thumbail.
Regards,
#0K
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/furnishup/blueprint3d/issues/22#issuecomment-326719440, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYAweQd8eQ-CnP8MhuaMXRBNWeL0kzP9ks5seM4FgaJpZM4Ji8Xd.

Where can I download blender three.js exporter for revision 'r69'?

https://github.com/mrdoob/three.js/tree/r69

Hi, I followed these instructions, but it throws "Matrix3.getInverse(): can't invert matrix, determinant is 0" whenever I click on the item I want add.  Is there something I need to make sure to click before export from Blender?

@ajoka1 You can contact me on Skype: royallogics OR Email<EMAIL_ADDRESS>

Hi ,
Please find the steps which worked for me to export a blender image to JS on windows 10

Download blender 2.74 @ https://download.blender.org/release/Blender2.74/
Unzip the file
Download @ https://s3.amazonaws.com/blueprint3d/io_mesh_threejs.zip
unzip the file at blender-2.74-windows64\2.74\scripts\addons
Launch blender
File --> User preferences --> addons
Search for three.js and enable Import-Export: three.js format by clicking on the check box
File --> Export --> Three.js(js)

Hope this helps

I am unable to add .gltf 3d image format to the item

Please how can I convert 3d obj airconditioning image to js model that is in the item folder

Can I use obj model instead special png and javascript file for 3D model?


Gradle 4.1 M1 break Android local aar import

When I update Android Studio to 3.0 Canary 5, it use gradle 4.1 m1.
I use a local aar developed myself and I used it before:
repositories {
    flatDir {
        dirs 'libs'
    }
}

...
compile(name: 'ruibulibrary', ext: 'aar')
now the build task is failed:
Gradle sync failed: java.lang.AssertionError: ':ruibulibrary:@aar' is not a valid maven coordinate. (3s 91ms)
Expected Behavior

before gradle 4.1 m1(3.x -- 4.x) worked fine
Current Behavior

Context

Steps to Reproduce (for bugs)

Your Environment

Build scan URL: https://gradle.com/s/dhvmvse2timlo

Windows 10 Creators Update 15063 Enterprise x64
Android Studio 3.0 Canary 5 x64
gradle-4.1-milestone-1

Same  here. iMac macOS Sierra (so this time not win-related)

I'm seeing this issue as well.

i'm getting the same issue and defining the full coordinate seems fixed the issue.
i.e
from
debugCompile(name:'stetho_realm-release', ext:'aar')
to
debugCompile "com.uphyca:stetho_realm-release:2.0.1@aar"

Here too,
This could also be the gradle android plugin (3.0.0-alpha5)

Same issue - defining full coordinate does not fix it for me. MacOS Sierra, latest gradle android plugin (3.0.0-alpha5).

Confirmed.  Does not work even with defining with the full coordinate.  I'm going to continue investigating.

same problem here guys, I will try to investigate that

Same here, sierra w/ android plugin 3.0.0-alpha 5.

Building from the command line still works for me, so I think this might actually be an Android Studio bug?

AS bug is here: https://issuetracker.google.com/issues/63232245

same here, its a AS bug, see it here Known Issues

What works in the meantime is to add the aar file as a module in the same project and referring to it like so:
    compile project(':library_module')

@pantos27 worked for me, thank you!

@pantos27 worked, thanks. Have you ever met  .so files in aar are not built into apk file.

I tried more solution and i found it's not gradle's bug. On on Android Studio Canary 5  : )
I found another way to import my aar library:
1、File -- New -- New Module -- Import .JAR/.AAR Package
2、remove this from app/build.gradle
repositories {
flatDir {
dirs 'libs'
}
}
...
compile(name: 'ruibulibrary', ext: 'aar')
3、add this to app/build.gradle
compile project(":ruibulibrary")
finally gradle works fine.

AS bug is here: https://issuetracker.google.com/issues/63232245

Thanks for opening the issue. It will need to be fixed in Android Studio. Closing the issue here.

Guys, how do you import complex .aar which are depends on other ones and on other remote libraries?


Fix saml post form

https://github.com/keycloakify/keycloakify/issues/341

Thank you very much!

A new version is being released


Dark mode is not supported

In both FF and Chrome the page renders strangely if the system default is set to dark mode. This is not the case with the UW template, so is likely an al-folio/jekyll thing. Switching the browser to light mode and hard refreshing doesn't correct it for some reason.

Likely related to a null reference that is happening in dark_mode.js as seen in console in the screenshot.

Setting enable_darkmode to false in _config.yml seems to solve the issue.

Closing as fixed with #4


Typos in Azure Stack Hub hotfix 1.2005.33.108

Fixed an issue in which partner node certificates required by nchotagent might be deleted.
nchostagent
Disabled winrrm runner
winrm

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

ID: 1c7da25b-56e5-d04a-b025-0c765b997acc
Version Independent ID: ee049f22-7026-2967-5e14-835454a5d822
Content: Azure Stack Hub hotfix 1.2005.33.108 - Azure Stack Hub
Content Source: azure-stack/operator/hotfix-1-2005-33-108.md
Service: azure-stack
Sub-service: azure-stack-hub
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@aluty Thanks a lot for bringing this to our notice and helping us improve our Azure documentation. I have raised a PR to correct the typos. Once the PR is reviewed and approved , changes would go live.
At this point it appears that there's no further edits or updates needed to the documentation, other than what has been listed above. We will go ahead and close this issue for now, however if you have any further feedback on the document please tag us in your reply and will be happy to reopen the thread and continue the conversation. Thank you


[2.9] Incorrect labels when using object dataset

When using object datasets, labels are displayed incorrectly because they are retrieved based on item index instead of the corresponding x or y property.
Also I created test cases for this issue.
Fixes #7839

i would like to contribute

You'll need to find a way to get the gulp docs passing to be able to update 2.9 branch.

I opened #7863 to track this issue, since it affects PR #7853 too.

Hello! I pushed the changes to make gulp docs pass. Should I do anything else?
Thanks!

Isn't the same possible by just configuring the right mode ('x' or 'y')?
index mode in my opinion should work by index rather that the coordinates.

Hi, I made a jsfiddle reproducing the issue. When you hover the point, the tooltip shows the incorrect month; that happens because it looks for the label data using the item index instead of the x / y property.
Could you elaborate on the mode configuration you mentioned?
https://jsfiddle.net/g5zcL2bu/

I was referring to the changes to the interaction modes, that seems unneeded to me.
See: https://jsfiddle.net/kc26bnzy/ where I just replaced the getLabelForIndex of category scale and used 'x' mode for tooltip.

That's an interesting solution, but isn't it more like a workaround? With this PR, we would have exactly the same result out of the box

Let me try to be more clear:
Why is the change needed for index interaction mode?

**_

ghghg

_**


Update: ransome1.sleek version 1.1.5

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.1 schema?

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


how to convert so many wav file to hca

hello
i want to convert so many wav files to hca format, but i dont know how VGAudio Work
you say must insert this comment:
VGAudioCli -b -i source -o dest --out-format hca
where this command must be insert?

vgaudio is a CLI tool, so as any CLI tool must be use via terminal (linux)/command prompt(windows)
Use google to learn about using them.

tnx bro. i found the way by moving files and vgaudio to system32 folder. its work 😍

Uh, technically you shouldn't use system32 except if it's absolutely necessary. Deleting file from this folder by mistake can have big consequences.


TypeError when setting direct peers in @chainsafe/libp2p-gossipsub with new libp2p peerStore structure

I an using @chainsafe/libp2p-gossipsub version 7.0.0 and libp2p version 0.45.0 on Linux, node.js v18.
Steps to reproduce:

Define directPeers in the libp2p options object.
Start libp2p node with @chainsafe/libp2p-gossipsub protocol.
Observe the following error:

TypeError: Cannot read properties of undefined (reading 'add')
    at file:///.../node_modules/@chainsafe/libp2p-gossipsub/src/index.ts:549:53

Details:
The error occurs during the try to set direct peer addresses in the addressBook (https://github.com/ChainSafe/js-libp2p-gossipsub/blob/f62859b346474ff3e29d59e31449b15d60e054d6/src/index.ts#L553).
It seems that the new version of the libp2p has a different peerStore structure.

Sorry the inconvenience but libp2p 0.45.0 introduced some breaking changes which have been addressed in #425 , so the next  gossipsub  release  should include this fix.


bump simple-get SNYK-JS-SIMPLEGET-2361683

Bump simple-get version to solve information exposure vulnerability: https://security.snyk.io/vuln/SNYK-JS-SIMPLEGET-2361683

The version range ^4.0.0 includes 4.0.1 so no change is necessary here.


Add fallback for missing target in Scrollspy

Fix  #33943 by adding the suggested check by @yura-x with fallback to console warning. The current fallback message can most likely be enhanced, but it was the best/ most concise thing I could come up with for now.

@alpa @GeoSot what's your take on this?

Uhm, the linting settings are configured to prohibit the usage of console. Is there a recommended way for sending warnings or are warnings not intended, and it should throw an error instead?


[docs] Grammar correction in autocomplete API

[x] I have followed (at least) the PR section of the contributing guide.

@gruber76


[QUESTION] -How does a project team get started?

Is there are good tutorial that outlines how a team would start from a blank page with the goal of creating a project that works with OpenCAESAR?
For example, if you work with Visual Studio (or any advanced IDE), there's an opportunity to "Create New Project". This then gives you a skeleton from which to work.
In essence, what is the skeleton structure needed by the OpenCAESAR system?

What is the minimum needed by OpenCAESAR to be considered as a project?
What is the recommended file/folder layout?
Are there any caveats that the developer should know about?
What are the tools that allow the engineer to validate the work at each step of the process (reference #20)?

Exploring deeper in what is a OpenCAESAR project and building tools that will interact with those types of projects, the following questions then emerge:

Is there an adapter interface that can be utilized?
Should tools stay within the OML realm or should they go as far as working with RDF?

There is some existing ways to start on an OML project from scratch (but of course more can be done):

Clone/fork the oml-template project

A good way to create an OML project that is IDE independent, and gives you git integration out of the box is to clone/fork the oml-template project. This comes with representative ontologies to get you started, along with a gradle build with a pipeline to reason on the ontologies.

Use Add New Project wizard in OML Rosetta

In OML-Rosetta, you can selection action File -> New... -> Oml -> Oml Project, specify some info for your first otology, and you will get a project template to start from. We do not have a similar capability in (VS Code-based) Oml Luxor yet, so if you like to see that also, please open a feature request on oml-luxor, and/or contribute a PR for this and we can review it.
Re: validation of work along the way, the gradle build task in the resulting OML project can be used to validate the OML files. Further validation can be done by writing owl queries or shacl rules and and invoking the owlQuery and/or the owlShacl tasks.

Re: Is there an adapter interface that can be utilized?
OML has an API that can be used to develop adapters, which are transformations to/from OML and other formats. There are some existing adapters for OWL, Ecore, and Papyrus UML. I don't yet have an adapter-template project in Github to represent a simple starting point, but those mentioned adapters can be inspected for such example. If you like to see a template adapter project documented in its own repo, please open a ticket on the OML repo itself.

Re: Should tools stay within the OML realm or should they go as far as working with RDF?
I think for ontology authoring and IDE integration needs, OML provides a good option since a) it represents a small subset of OWL2 that is well suited for systems engineering, and b) it provides an API that conforms to modern Java tool stacks liike EMF, Xtext, Gitpod, etc.
OML also has a ready adapter to OWL. This opens up OML models to all the analytical tools for OWL including reasoners, query/validation languages, triple stores (graph databases) etc.
For going beyond OML and OWL for analysis needs, one could either go from OML -> other analysis tools directly, or OML -> OWL -> other analysis tools, if going through OWL makes it easier (like if you need to use the inferred axioms).


Fix test error after #72

Fixes include error after https://github.com/miking-lang/miking-dppl/pull/72.

Fixed by https://github.com/miking-lang/miking-dppl/pull/74 instead


PodCache donot lock workloadInstanceHandlers, it could do time costing work

Please provide a description of this PR:
Reduce pod cache lock scope to not include ServiceEntryStore.WorkloadInstanceHandler

Just curious, what of the handler is slow? Seems a bad pattern to have that slow

ServiceEntryStore.WorkloadInstanceHandler, the calling stack it too deep.

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
af84eef1558f7ea981dda97d4fadd4ec67df3fce
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests_istio

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
af84eef1558f7ea981dda97d4fadd4ec67df3fce
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

grpcgen: TestMtls fails frequently https://github.com/istio/istio/issues/36804

/test unit-tests_istio

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
af84eef1558f7ea981dda97d4fadd4ec67df3fce
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests_istio


Installing Pico on php5

It is question rather than bug report. Using composer to install Pico works well on php7+, deploying site to client's server I realised, that there is still php5, probably because there is other old developments on it, that do not work with php7. So composer autoloader complain about this and Pico site will not work. Any hints, how to get Pico up and running on php5 env?

Simply use one of Pico's pre-built release packages, they are built on PHP 5.

Thanks. Will try that. I guess I can close it.


Test for the resume

Hello, I am a new developer of the Embulk plugin, Is there any way to test the Resume support of the plugin? By the way, to support the resume of the data loading, all my plugins should support the resume? Including the input plugin, the parser, and the output plugin? Waiting for your answer. Thank you.

Thanks for your interests on Embulk!
For testing, do you have any ideas? @muga
It'd depend on the plugins, the data source, the destination and the situation, but I guess resuming support just in input and sometimes in output would work in many cases.
Not needed in parser, filter or other plugins since they're not data source/destination and they're stateless. Resuming in output may not be required if no failures are expected for output.

@icaiyu @dmikurube Good point. We don't have anything to check if a plugin supports resume functionality or not so much.
For now, almost all of plugins (plugin developers) may describe that plugins support resume in README.md. But, some plugins doesn't show such descriptions. We don't have a guideline about it. And even though some plugins describe that they have resume functionality in README files, we don't know that their resume correctly work or not.

Thank you for your reply. Could you give some tips so that I can test the resume support functionality. For example, I want to the test the resume support of the embulk standard local file input plugin. I try to stop the embulk by pressing ctl+c but there are nothing in the diff.yml.


Nginx Reverse Proxy for Apache webserver causing problems with IOS clients

Hello, am new to all of this, I am by no means an expert on Nginx and Apache.  My Apache Webserver is behind the Nginx Reverse Proxy, overall it is working, except for IOS devices.  IOS devices trying to access my website in the Apache server will get an error "The operation couldn’t be completed. Protocol error with a code of "  regardless of the browser.  This only happens with IOS devices, Android, Windows, and Linux does not show this problem and works as expected.
After googling, a possible solution is to set the Nginx reverse proxy to "proxy_hide_header Upgrade;" or my Apache server with "Header unset Upgrade", both of which I do not know how to do.   Please help.
The source of this information came from this link

Same issue with workarround https://github.com/jc21/nginx-proxy-manager/issues/661

Closing in favor of https://github.com/jc21/nginx-proxy-manager/issues/661


Angular velocity not transfering properly to other rigidbody2d from another

If a spinning body comes into contact with a stationary or another spinning body, the bodies act as if they have no mass and almost infinite momentum in some cases, acting wildly erratic.
Have two spinning RigidBody2D objects slowly collide while they have natural angular velocity. When they Collide it should be obvious they are acting strangely
I expect the angular motion to distribute properly and two bodies with the same angular momentum and mass to cancel each other out or bodies of differing mass and momentum to exchange spin properly and not behave erratically. At this point, a ship that weighs 100kg hit by a slow-moving, spinning asteroid that ways 50 metric tons stops the asteroid completely or just has the linear momentum transferred from velocity, though no angular drag is present on ship's body.
This happens on 0,8,4 (cannot try latest version as it's not loading without the plugin.gd)

I think I'm just having yet another one of these problems that happen only when I'm in my project. Though. I can't recreate it but it's happening in the same game I've been working on for some time now.


Bridge tries to cast EmptyResult to expected type when an error occurs

Bridge tries to cast EmptyResult to expected type when an error occurs which results in an invalid cast.
VICE returns an EmptyResult when an error happens.

ViceCommand.Response will return Task<CommandResponse> instead of simply Task.
The former carries also ErrorCode regardless of TResponse type.


Add a post triggered action event

Checklist

[x] I checked for similar existing requests (both open and closed) before posting.
[x] My request is generic, other users may benefit from it too.

Proposal
When the alarm is disarmed immediately following a triggered alarm, I would like to run clean-up actions to reset a few things and potentially send all-clear notifications. There is currently no easy way to do this. I would like to propose a new Event that is only triggered when the state of the alarm goes from "triggered" directly to "disarmed."

You can create an action to turn off the siren when the alarm is disarmed.
I agree with the request, I think functionality should be added to configure actions on certain state transitions.
For now it's best to do this via HA automations.

This feature is added in release v1.9.5:

Please try it out and close the issue if the added functionality satisfies the request.
Otherwise let me know.

Looks like this is exactly what I requested. Thank you!
Just one quick thing, should it read "Alarm no longer triggered" instead of "not"?

thanks @nielsfaber for adding this. can you please expose this to normal HA automations as well? there it's not listed yet. thanks.

@goyney

Just one quick thing, should it read "Alarm no longer triggered" instead of "not"?

You're probably right. English is not my native language.
If you want to help you're more than welcome to suggest changes via PR of the localize file, see here.
@snickers2k

thanks @nielsfaber for adding this. can you please expose this to normal HA automations as well? there it's not listed yet. thanks.

In HA automations you can just trigger on a state change from triggered to disarmed.
There is no change needed for this.

@goyney

Just one quick thing, should it read "Alarm no longer triggered" instead of "not"?

You're probably right. English is not my native language. If you want to help you're more than welcome to suggest changes via PR of the localize file, see here.

Done. See https://github.com/nielsfaber/alarmo/pull/546.


ROX-24337: error message for incorrect init bundle format

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
TODO(replace-me)
Use this space to explain how you validated that your change functions exactly how you expect it.
Feel free to attach JSON snippets, curl commands, screenshots, etc. Apply a simple benchmark: would the information you
provided convince any reviewer or any external reader that you did enough to validate your change.
It is acceptable to assume trust and keep this section light, e.g. as a bullet-point list.
It is acceptable to skip testing in cases when CI is sufficient, or it's a markdown or code comment change only.
It is also acceptable to skip testing for changes that are too taxing to test before merging. In such case you are
responsible for the change after it gets merged which includes reverting, fixing, etc. Make sure you validate the change
ASAP after it gets merged or explain in PR when the validation will be performed.
Explain here why you skipped testing in case you did so.
Have you created automated tests for your change? Explain here which validation activities you did manually and why so.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/ok-to-test

Close in favor of: https://github.com/stackrox/stackrox/pull/11613 cause of CI failure.


Implement IntoWasmAbi for &MyEnum

Motivation
It seems that an enum by value implements the IntoWasmAbi trait but not if you have a reference to an enum.
Here's a small example:
#[wasm_bindgen]
enum Role {
    User = "User",
    Admin = "Admin",
}

#[wasm_bindgen]
extern "C" {
    pub fn create_user(role: &Role);
}

fn main() {
    create_user(&Role::User);
}

The above will throw the following error:
the trait `wasm_bindgen::convert::traits::IntoWasmAbi` is not implemented for `&web::fs_error::Role`

Proposed Solution
Maybe we should implement the IntoWasmAbi trait for references to enums as well.

Seems reasonable to me! I suspect that this wouldn't be too too hard to implement in the backend.

I can try and take a stab at it if you want.


Use with react-fontawesome

How can I use it with react-fontawesome? when I try I only get it buttons without an icon

What are you using in your configuration for the icons property to <FontIconPicker /> ?

BTW: I hacked around a bit.  I'm using react-semantic-ui that includes an older free distribution of font-awesome.  Can't say for sure this will work in the long run (FEEDBACK WELCOMED!!) but I added the following and it is working for me:
import { ICONS } from "semantic-ui-react/dist/commonjs/lib/SUI";

// ...

<FontIconPicker
        icons={ICONS}
        value={[]}
        renderFunc={(icon) => <i className={`icon ${icon}`} />}
        showSearch={true}
        theme='blue'
        isMulti={false}
        onChange={onChange}
        onBlur={(evt, val) => onBlur(val)} // need input.onBlur so that reduxForm knows when we've clicked in/out of a field (so it can update "touched" )
        onChangeRaw={(e) => e.preventDefault()}
        closeOnSelect
        {...rest}
      />

Another update.  The categories don't work with the above because ICONS is a flattened list within SUI.
To get categories working, I use suiIcons from the following instead of ICONS above:
import * as SUI from "semantic-ui-react/dist/commonjs/lib/SUI";

export const suiIcons = {
  Accessibility: [ ...SUI.ACCESSIBILITY ],
  Arrows: [ ...SUI.ARROWS ],
  Audio_Video: [ ...SUI.AUDIO_VIDEO ],
  Business: [ ...SUI.BUSINESS ],
  Chess: [ ...SUI.CHESS ],
  Code: [ ...SUI.CODE ],
  Communication: [ ...SUI.COMMUNICATION ],
  Computers: [ ...SUI.COMPUTERS ],
  Currency: [ ...SUI.CURRENCY ],
  Date_Time: [ ...SUI.DATE_TIME ],
  Design: [ ...SUI.DESIGN ],
  Editors: [ ...SUI.EDITORS ],
  Files: [ ...SUI.FILES ],
  Genders: [ ...SUI.GENDERS ],
  Hands_Gestures: [ ...SUI.HANDS_GESTURES ],
  Health: [ ...SUI.HEALTH ],
  Images: [ ...SUI.IMAGES ],
  Interfaces: [ ...SUI.INTERFACES ],
  Logistics: [ ...SUI.LOGISTICS ],
  Maps: [ ...SUI.MAPS ],
  Medical: [ ...SUI.MEDICAL ],
  Objects: [ ...SUI.OBJECTS ],
  Payments_shopping: [ ...SUI.PAYMENTS_SHOPPING ],
  Shapes: [ ...SUI.SHAPES ],
  Spinners: [ ...SUI.SPINNERS ],
  Sports: [ ...SUI.SPORTS ],
  Status: [ ...SUI.STATUS ],
  Users_People: [ ...SUI.USERS_PEOPLE ],
  Vehicles: [ ...SUI.VEHICLES ],
  Writing: [ ...SUI.WRITING ],
  Brands: [ ...SUI.BRANDS ],
};


Add details under Author a configuration section on this page

It would be helpful to follow the article if you could provide additional details such as sample DSC file configuration under Author a configuration section of this page here https://learn.microsoft.com/en-us/azure/governance/machine-configuration/machine-configuration-create#author-a-configuration
Also, the documentation does not provide enough details of generating the actual MOF file once the basic DSC file is created. It becomes hard to follow this step by step document if there is no enough context or documentation available.

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

ID: 5cc74604-e182-ff85-d449-cd330fa40f5a
Version Independent ID: 138e294e-45fb-6f08-5077-52d62c38d3f6
Content: How to create custom machine configuration package artifacts
Content Source: articles/governance/machine-configuration/machine-configuration-create.md
Service: machine-configuration
GitHub Login: @timwarner-msft
Microsoft Alias: timwarner

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

#reassign:michaeltlombardi


Authorization without scanning a QR code

Hi everyone, it’s possible that you wouldn’t be authorized by QR code, but as in the original application, enter the number and confirm with the code from SMS?

No way.
On Fri, 22 May 2020, 6:42 pm Alex<EMAIL_ADDRESS>wrote:

Hi everyone, it’s possible that you wouldn’t be authorized by QR code, but
as in the original application, enter the number and confirm with the code
from SMS?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Rhymen/go-whatsapp/issues/372, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDLT23TTUOB2VMBFOC6CBTRS2MMVANCNFSM4NH524YQ
.

This library is for the web app API, which links to your phone and is authorized with the QR code.
There are other libraries like yowsup that use the client API and thus log in with SMS, but WhatsApp tends to ban you immediately if you try to use those.


RateMyProfessor hyperlink broken

It seems like all the hyperlinks for ratemyprofessor are broken... They link to a page with Internal Server Error, but the site is functioning. It looks the way they structure the search using URL might've changed.

They did change the URL structure, and professors just have numbers now. We can only link to a search. See #594.
The only thing we can do to bring back the functionality is for PPAPI to scrape RMP for the professor ID for each professor and give it to us, but that's probably not worth it.


Extract zstd into a feature flag

Hey, I'm interested in getting this lib to work for wasm targets. So let's start with the bare minimum of getting this to compile for wasm without any features enabled. The only real obstacle here was the zstd feature of async-compression which drew in some zstd binding rust crate. Moving this around gives us a starting point to get the rest to run on wasm as well.
If you want to have it enabled by default, we can put the zstd feature into the default features. Just let me know!

I'm ok with this change (tbh I am not even sure why we need zstd enabled by default?)


[WIP] View a particular WorkVersion

@awead Can you please check out app/views/dashboard/work_versions/_status_badge.html.erb. Do you reckon it needs a presenter? Maybe? Eh?

@rschenk

@awead Can you please check out app/views/dashboard/work_versions/_status_badge.html.erb. Do you reckon it needs a presenter? Maybe? Eh?

Meh. I think a two-line view is good for now. We could refactor later if there are different kinds of badges, exc.


python312Packages.aiogram: 3.5.0 -> 3.6.0

Diff: https://github.com/aiogram/aiogram/compare/refs/tags/v3.5.0...v3.6.0
Changelog: https://github.com/aiogram/aiogram/releases/tag/v3.6.0
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 311348 run on x86_64-linux 1

4 packages built:

python311Packages.aiogram
python311Packages.aiogram.dist
python312Packages.aiogram
python312Packages.aiogram.dist


Fix import of MethodType

Don't pretend that types.MethodType is influenced by six

merged in 4c943365107e40571748196d9d1c11d3f0e3e2af, thanks.


Incompatible install in SIFT Workstation v2020.7.1 on Ubuntu 16.04

am getting this error trying to install SIFT workstation v2020.7.1 on Ubuntu 16.04. The install does not show me at the end Completed with Errors. It just stops. Below this error is the error from the saltstack.log
Running: sift-config-tools
Update returned exit code not zero
Error: Update returned exit code not zero
at ChildProcess. (/snapshot/sift-cli/sift-cli.js:529:23)
at ChildProcess.emit (events.js:196:13)
at maybeClose (internal/child_process.js:1011:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:268:5)
----- PLEASE READ ----------------------
A lot of failures are caused by the apt system being locked
or unhealthy.
Before opening an issue in GitHub, please check to see if
your apt system is healthy.
Try running 'apt-get update' then remove any packages that
aren't used by running 'apt-get autoremove'
sansforensics@siftworkstation: ~/Downloads
$
------------------------SALTSTACK.LOG ERROR--------------------------
Requirement already satisfied: pip in /opt/rekall/lib/python2.7/site-packages
(20.2.2)
Requirement already satisfied: setuptools in /opt/rekall/lib/python2.7/site-packages
(45.0.0)
DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please
upgrade your Python as Python 2.7 is no longer maintained. pip 21.0 will drop
support for Python 2.7 in January 2021. More details about Python 2 support
in pip can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support
WARNING: The directory ''/home/sansforensics/.cache/pip'' or its parent directory
is not owned or is not writable by the current user. The cache has been disabled.
Check the permissions and owner of that directory. If executing pip with sudo,
you may want sudo''s -H flag.
ERROR: Package ''setuptools'' requires a different Python: 2.7.12 not in ''>=3.5'''
duration: 3512.975
name: /opt/rekall
result: false
start_time: '03:37:03.278404'

I see the error Requirement already satisfied: setuptools in /opt/rekall/lib/python2.7/site-packages (45.0.0) Setuptools dropped support for Python 2 at Setuptools 45. Please work with sift and/or salt stack to to identify why an incompatible version of Setuptools is being installed. If there's something Setuptools can do to help, feel free to write back or open a new issue.


e2e: nightly failures on v1.x due to cometbft-db image upgrade

Bug Report
The e2e nightly tests on v1.x are currently failing as seen here
While troubleshooting the failures, I've bi-sect the latest changes merged into v1.x and found out the error started happening after this PR was merged.
In workflow run logs, there is an error:
validator01  | ERROR: failed to create node: unknown db_backend boltdb, expected one of badgerdb,goleveldb,memdb,pebbledb,rocksdb

tracing back, noticed that the generator on v1.x still includes cleveldb and boltdb which might include them in the manifests generated when running the command:
./build/generator -g 5 -d networks/nightly/ -p

which will make the workflow fail to run with the error mentioned above because we've upgraded the image in the Dockerfile in v1.x
FROM cometbft/cometbft-db-testing:v0.13.0

We'd need to cherry-pick the changes from this PR that deprecated removed these two databases on main or try to backport the changes to v1.x.
At least the changes in the generate.go need to be implemented in v1.x in order to allow the e2e nightly v1.x workflow to run properly again.

We'd need to cherry-pick the changes from this PR that deprecated these two databases on main or try to backport the changes to v1.x.

From the PR description, we are deprecating these DBs on v1.x, but not removing them.

The simplest approach here is to downgrade the version of cometbft-db on v1.x branch.

We'd need to cherry-pick the changes .....that deprecated these two databases on main or try to backport the changes to v1.x.

From the PR description, we are deprecating these DBs on v1.x, but not removing them.

Actually we removed them, not deprecated. Just corrected that in the PR description.

To clarify here:
In comtebft-db release v0.12.0 the cleveldb and boltdb were deprecated, and in release v0.13.0 they were removed (but the entry is changelog was missing).
Since the combetbft-db dependency was bumped in v1.x that triggered failures in nightly. So the team is evaluating if the cometbft bump should be reverted on v1.x or we should leave it there and add the changes required to make nightly work.


CORS Support

Create a CORS extension for WebServer to simplify CORS configuration and handling for users.
Document CORS with SE.
See https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS

https://en.wikipedia.org/wiki/Cross-site_request_forgery

CORS support for MP in progress here: https://github.com/spericas/helidon/tree/cors/microprofile/cors

CORS wiki page is now here:
https://github.com/oracle/helidon/wiki/CORS-in-Helidon-MP

Hi @spericas .  Any updates on microprofile-cors? I've created a custom filter for my applications but it seems that /openapi doesn't get intercepted by the filter.

@rafabene Hoping to get that in the next milestone, at least for MP. Not sure that would help OpenAPI as those resources are a bit different. What do you think @tjquinno?

@spericas Thanks for having this as part of next milestone.


README examples are out of date

README.md examples are not in line with the examples provided here:
https://github.com/codenotary/immudb-client-examples
People who follow these examples will encounter the ImmudbClient is not a function error.

@elmarquez , immudb-node was updated to v2. Examples repository needs to be updated, but you can find a lot of examples in immudb-node repo under https://github.com/codenotary/immudb-node/tree/master/immudb-node-showcase

@tomekkolo thank you so much for letting me know


Directly load data without export

i want to create application with features:

every user can upload 3D file(cityjson or gml)
3D file that uploaded on step 1 can be viewed from map by all user

i was read documentation and tried sample project, everything is okay. my step is :

import cityjson manually via IMPORTER/EXPORTER TOOLS
export manually via IMPORTER/EXPORTER TOOLS
open exported result on webmap client

but my expectation step is :

import cityjson programmatically via myapplication (I can handle with run CLI command from programming language)
open 3D data via web map client. without export. (directly load via API such as geoserver)

is it posible ?
thanks

Currently, there is no web service or REST-based API to the 3DCityDB that would allow to visualize the data in a web client directly from the database (neither the 3DCityDB Webmap Client nor any other web client that could directly connect to PostGIS which I would be aware of).
A solution to your intended application could be to add an "upload" button to the 3DCityDB Webmap Client, which opens a file browser for the user to select a local file. This would then be uploaded via http POST request to your web server. Then your server would have to start a batch CityJSON or CityGML import process, followed by a batch KML/COLLADA/glTF export process generating all necessary files for the visualization. Then the resulting visualization files would have to be moved to a folder that can be served from your web server. Finally, in the 3DCityDB Webmap client you can automatically add a new layer referring to the location of the visualization export on your web server. Since the city model import and the visualization export will take some time (depending on the size of the model), you might need to reload the entire configuration of the 3DCityDB Webmap Client after some time (either manually by the user or also add a process that repeatedly checks, if the visualization model has been generated and is accessible from your webserver).
If you would run this setup using Docker, you could even instantiate a new Docker container for each upload process and delete the container after the visualization export has been performed.
Hope this helps
Thomas Kolbe

Thanks for your response. Ok i see your scenario. But i have one question.
Amount of building on my application about one hundred thousand data. Is your scenario still realible ?

Yes, this should be possible. If people are using a cloud hoster platform to upload and share their large files, the files are often also uploaded via http POST. Import into the 3DCityDB could take some time (depending on the file sizes and complexity of the dataset, and on the hardware of your server). To give you an impression: importing the entire Tokyo dataset (2 Mio buildings from a compressed ZIP-Archive of 9.5GB size incl. textures requires on a modern desktop PC with SSD drive and 16GB RAM between 30mins and 2h). The generation of the entire visualization export takes another 8h.
Example settings and Tokyo datasets: https://wiki.tum.de/display/gisproject/Semantic+3D+City+Model+of+Tokyo
But if you want to provide that interactive import functionality, you will probably not ask for such large datasets.

Thanks thomashkolbe
your eexplanation is about one file with big size.
How about with small file but thousand of amount.
Scenario of my application is, each user can upload 1 or 2 building inside one file.
then maybe nextweek he can upload 2 building again.
My question is :

Is my application will has thousand of layer?
or
I must delete generated tile and re-generate tile if new building has been uploaded by user

Thanks in advance

In this case I would recommend to run one instance of the 3DCityDB which accumulates all uploaded models over time and for which you have a nightly visualization export that regenerates all tiles. If somebody uploads a new (small) dataset, you should launch another instance of the 3DCityDB in a Docker container as described above. After the import of the few buildings and the generation of a visualization export (which will be fast) you could then generate a 3DCityDB webclient project, which contains two layers: the first layer is a link to the visualization model generated from the first instance of the 3DCityDB and the second layer is a link to the visualization export of the newly added buildings. If the user (or an administrator) agrees to the uploaded models they can be merged into the first instance and become part of the growing city model. After the merging of the new models into the first 3DCityDB instance, the Docker container with the temporary second 3DCityDB could be deleted again.

thanks for your answer sir


updating valast to fix type misalignment in code generation

Signed-off-by: Frederick F. Kautz IV<EMAIL_ADDRESS>

Converted to draft to work on upstreaming valast changes.

Any update on this?

I needed a break to recover from burn out. :x
I'll get back on this shortly.

closed with #95


Look into using Bootstrap

As mentioned in #11

You might want to look into MaterializeCss since it has the same theme you are going for on the site.

That is an idea, but what we have seems to be working right now. We could use that in a future version.

Closing, as we will probably just go with what we have.


Format on type is not working

VSCode Version: 1.45.1
OS Version: macOS 10.15.4 (19E287)

Steps to Reproduce:

Type out following lines in a python file

import argparse

parser = argparse.ArgumentParser()
parser.add_argument('-a', '--abc', nargs=5, required=True, default=5, help='some help text that is very long to trigger reformat')

The last line should get reformatted with typing, but it doesn't.
If I select the line and run Format Selection from command palette, it looks like following -
import argparse

parser = argparse.ArgumentParser()
parser.add_argument('-a', '--abc', nargs=5, required=True, default=5,
                    help='some help text that is very long to trigger reformat')

Does this issue occur when all extensions are disabled?: Yes*
It's difficult to reproduce this without extensions because a formatter must be available for this. I disabled everything except the Python language pack to reproduce this.

Please create an issue directly against the extension at https://github.com/Microsoft/vscode-python/issues . The authors of the extension will investigate and determine if this is caused by vscode and can reach out to us in that case.


Unable to specify a healthcheck protocol for a defaultBackend

I have an Ingress talking to a service, and I'm using a "defaultBackend" to route all unknown traffic to the service.
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: ocilbingresshttps
  annotations:
    oci-native-ingress.oraclecloud.com/protocol: HTTP2
    oci-native-ingress.oraclecloud.com/backend-tls-enabled: "false"
    oci-native-ingress.oraclecloud.com/https-listener-port: "443"
    oci-native-ingress.oraclecloud.com/healthcheck-protocol: "http"
    oci-native-ingress.oraclecloud.com/healthcheck-path: "/ping"
spec:
  ingressClassName: ocilb
  rules:
    - http:
        paths:
          - path: /
            pathType: Prefix
            backend:
              service:
                name: traefik
                port:
                  name: websecure
  defaultBackend:
    service:
      name: traefik
      port:
        name: websecure

I have specified a healthcheck for the service using the annotations:
    oci-native-ingress.oraclecloud.com/healthcheck-protocol: "http"
    oci-native-ingress.oraclecloud.com/healthcheck-path: "/ping"

Unfortunately, they do not get applied to the "default_ingress" backendset - so it stays as the default - a TCP service, leaving the loadbalancer in the "Critical" smart check status.
From the native ingress log file:
"BackendSetHealthCheckerMap": {
  "bs_c0b815d03fa5fbd": {
   "protocol": "HTTP",
   "urlPath": "/ping",
   "port": 0,
   "returnCode": 200,
   "retries": 3,
   "timeoutInMillis": 3000,
   "intervalInMillis": 10000,
   "responseBodyRegex": "",
   "isForcePlainText": false
  },
  "bs_c2b7f75c1039b6f": {
   "protocol": "HTTP",
   "urlPath": "/ping",
   "port": 0,
   "returnCode": 200,
   "retries": 3,
   "timeoutInMillis": 3000,
   "intervalInMillis": 10000,
   "responseBodyRegex": "",
   "isForcePlainText": false
  },
  "default_ingress": {
   "protocol": "TCP",
   "urlPath": null,
   "port": 0,
   "returnCode": null,
   "retries": 3,
   "timeoutInMillis": 3000,
   "intervalInMillis": 10000,
   "responseBodyRegex": null,
   "isForcePlainText": null
  }
 },

Thanks!

@cweeks72 Currently we don't have health checks for default ingress. The default health check of TCP is minimalistic and it confirms that the default backends are reachable to LB. If they are critical so posibly they are not rechable. Can you please validate this?

The "Smart Healthcheck" of the loadbalancer is flagged as a critical warning, because of the TCP healthcheck to an HTTP backend. This can only be fixed by changing the healthcheck type to HTTP for the default backendset.


Make enums non_exhaustive if they are likely to be extended

Summary
Enums like Key are likely to be extended at some point in the future - this would currently technically be a breaking change, as if someone happens to be matching on every variant (e.g. for a conversion method?) then their code would no longer compile.
In 0.7, I should do an audit of all the enums in the crate and add #[non_exhaustive] where appropriate to prevent this from being a problem going forward.
Motivation/Examples
No response
Alternatives Considered
No response

Done on 0.7 branch.


Extend binary format for stake delegation tx

Issue Number

#891
Overview

[x] I have extended MsgType and getMessage accordingly
[x] I have added decode . encode test
[x] I have added impl of getDelegatedCertificateTransaction
[x] I have added impl of putStakeDelegationTx
[x] adjust existent tests for this change

Comments

@rvl Ah! I forgot about it when writing the original ticket but, we can't indeed have an XPub here. XPub are made of a public key (32 bytes) and a chain code (32 bytes). The chain code is not actually present on-chain (like the addresses that only contains the public key part of the XPub).
Yet, I a raw ByteString is somewhat not very meaningful. @paweljakubas , what do you think of having a dedicated newtype to capture this? e.g. newtype ChimericAccount = ChimericAccount ByteString

Thanks @Anviking for being vigilant and critical! Updated the golden test and adjusted the rest code in Binary.hs that needed it to make it work

bors r+


[REQUEST] Add option for silencing deew completely/minimal terminal output

When using deew in a limited functionallity terminal enviroment, the program fails to execute because rich cannot render to the terminal, my suggestion is to add an argument that completly silences terminal output (or at least keeps it free of external python dependencies as a very basic version with no progress bar or styling"
Command used
deew.exe -i "inputdir" -b 256 -f ddp -np -o "outputdir"
Traceback (most recent call last):
  File "__main__.py", line 6, in <module>
  File "deew\__main__.py", line 551, in main
  File "rich\__init__.py", line 73, in print
  File "rich\console.py", line 1646, in print
  File "rich\console.py", line 848, in __exit__
  File "rich\console.py", line 806, in _exit_buffer
  File "rich\console.py", line 1999, in _check_buffer
  File "rich\_windows_renderer.py", line 17, in legacy_windows_render
  File "rich\_win32_console.py", line 442, in write_styled
  File "rich\_win32_console.py", line 403, in write_text
  File "encodings\cp1252.py", line 19, in encode
UnicodeEncodeError: 'charmap' codec can't encode characters in position 1-5: character maps to <undefined>
[15076] Failed to execute script '__main__' due to unhandled exception!

referenced at #43


Output images Not showing on Replicate explore and Example section in overview page, but work in Example Gallery

For this demo https://replicate.com/sczhou/codeformer,
Output images are not displaying in the explore page and the example section of the overview page of the demo - but they do show up in the example gallery https://replicate.com/sczhou/codeformer/examples.
See details here
This demo also has this issue https://replicate.com/cjwbw/stable-diffusion-high-resolution

This is a replicate issue so will open internally.


Py3 dbs3 client

DO NOT MERGE YET!
Those are changes needed to fix only the rpm build process for the py3-dbs3-client.spec and py3-dbs3-pycurl-client.spec from the cmsdist repository

Nice catch Todor.
@yuyiguo Yuyi, do you think we could skip building the DBS documentation for a while? Maybe we could create another GH issue to track it in the near future? Please let us know what your thoughts are.

@todor-ivanov @amaltaro
Please go head w/o building DBS doc. I will create GH issue to reminder us for putting back the automatically generated docs back.

FYI
https://github.com/dmwm/DBS/issues/650

Thanks @yuyiguo!
BTW There are going to be some more changes to be added to the current PR. I will write here again shortly.

@vkuznet  @yuyiguo  @amaltaro  please take a look at the jsonwrapper suggested here - Still alot of testing to be done

@todor-ivanov I just noticed that you put code into DBS/Client area while cjson is used both in DBS Server and Client. If you'll keep it in Client then DBS Server code will depend on client unless you'll duplicate the code. Instead, you should make an independent package and put it on pypi. Doing this way it will allow code (DBS Client and Server as well as WMCore) depend on it. The wrapper can be and should be (in my opinion) packaged independently from DBS codebase. But I'll leave a decision about it to WMCore group and @yuyiguo

Hi @vkuznet @yuyiguo Thanks for the feedback. With my 5th commit to the current PR [1] I tried to simplify the jsonwrapper coul you please take a look and tell me if it is satisfactory enough, so I may continue with the rest of the code modifications for migrating the dbsclient to py3. As of the following:

the standard json library only provide load, loads, dump and dumps, the encode and decode methods only belongs to cjson. Therefore, if you indent to replace cjson.encode and cjson.decode in DBS code it is better to replace them with jsonwrapper.loads and jsonwrapper.dumps methods to stay consistent with APIs of standard json library

I agree it would be more close to the standard if we do so, but actually in the code there are several places [2] where it actually relies on the cjson.{Encode,Decode}Error exceptions. And because of that following Valentin's initial approach seemed reasonable to me - i.e. wrapping the standard .loads && .dumps calls with the .encode && .decode methods from the jsonwrapper,  so that the exceptions coming from the actual json.loads && json.dumps calls can be enveloped (from a single place) with the common set of exceptions defined in the jsonwrrapper and then handled in the code the same way they were before. This way we avoid the need to juggle with exceptions in the dbsclient code. Otherwise in the DBS clinet code we will have to handle two types of exceptions - those from cjson and those from json.
[1]
https://github.com/dmwm/DBS/pull/649/commits/f859582832963c751453836cb65f163a5bbea103
[2]
https://github.com/dmwm/DBS/blob/14df8bbe8ee8f874fe423399b18afef911fe78c7/Client/src/python/dbs/apis/dbsClient.py#L210

Hi @vkuznet

I just noticed that you put code into DBS/Client area while cjson is used both in DBS Server and Client ...
I agree that the higher in the tree we put the jsonwrapper the better. My initial intention was not to solve all DBS python3 issues, and that's why I restrained myself only to the code base related to DBSClient, with the intend at some near future, the people responsible for DBS code to take care of positioning it at the right place at the right moment.  @amaltaro @yuyiguo please let me know what you think.

Regarding exceptions. There are two pieces required. One the jsonwrapper should wrap them such that exception will come
from a single method, like load/loads or dump/dumps. Basically you need to catch cjson exception within jsonwrapper and then through again like json module does.
The second part is to modify DBS code itself to catch just common exception which will come either from json module or from jsonwrapper. This will solve issue with different json implementations.

@todor-ivanov
Restraining your current work on DBS client is what I think you may do now. I am thinking we should separate DBS client into a different package/repo from DBS server, but that is later.  So we will reviste DBS client later, for now get it to work with py3 first. Then, WM can move forward quickly.  Regarding the decode/encode, it is idea to follow py3 json as discussed, but we can live with the cjson style for now.

Todor and I had a chat yesterday and we seem to be aligned with what Yuyi said above. We should focus on the minimum amount of changes in dbs3-client and dbs3-pycurl-client to have it compatible with python3, especially because the rest of the DBS code future is also uncertain at this stage.
A few options for those minimum changes that come to my mind are:

provide this jsonwrapper to the dbs3-client/dbs3-pycurl-client packages only, leaving all the rest of DBS code untouched;
simply replace cjson by json in the dbs3-client/dbs3-pycurl-client code; and keep it in a separate branch.
if more change are needed to these client packages, we could consider as well a 2to3 conversion of the client code from python2 to python3; and keep it in a separate branch for now.

Just my 2cents.

I fully agree.  To reinforce what you said, for this purpose, there's no need to try to keep compatibility with Py2 (as we did for WMCore).  We just need something (on a separate branch, if necessary) that works ASAP so that we can move forward with validation of WMCore under Py3.

As discussed in a meeting we had between me, @amaltaro and @mapellidario this PR was split in two separate PRs:

The current PR provides the minimal changes in setup.py.
The new PR provides all the python3 related changes.

p.s. This PR is created against master and can safely be merged there - this will not break the py2 version of dbs-client


Unhandled Runtime Error when trying to use Wallet Connect

Describe the bug
After connecting to Wallet Connect the user encounters an Unexpected Error on their screen. So I tried this in the NextJS dev environment and found it was giving an Unhandled Runtime Error in useClaim.ts
This was introduced with the new airdrop claim code. Apologies if you are aware of this already. I am filing this for anyone else who may be experiencing this issue.
To Reproduce
Steps to reproduce the behavior:

Trigger something that brings up the web3 Modal
Click on 'Wallet Connect'
Scan the QR code using a Wallet Connect wallet.
See error

Expected behavior
Connects the user's wallet to the web3 provider.
Screenshots

Desktop (please complete the following information):

OS: MacOS 11.3
Browser Brave
Version 1.25.68

Smartphone (please complete the following information):

Device: iPhone XS Max
OS: iOS 14.6
Wallet App: Rainbow Wallet
Version 1.5.10

Full error log:
Unhandled Runtime Error
TypeError: Converting circular structure to JSON
    --> starting at object with constructor 'WalletConnectProvider'
    |     property '_providers' -> object with constructor 'Array'
    |     index 0 -> object with constructor 'FixtureProvider'
    --- property 'engine' closes the circle

Source
src/hooks/useClaim.ts (144:11) @ useIsClaimed

   142 |       setClaimed(response)
   143 |     })
 > 144 |   }, [JSON.stringify(distributorContracts), JSON.stringify(claimInfo[key])])
       |           ^
   145 |
   146 |   return claimed
   147 | }

cc @nateawelch

Marking this as resolved and closing the issue.


Verify proposals are valid after sending them to miners

Motivation
zcashd returns an error when the block template is not a valid block (ignoring proof of work):
https://github.com/zcash/zcash/blob/9e1efad2d13dca5ee094a38e6aa25b0f2464da94/src/miner.cpp#L674-L676
Mining pools would like to start working on new blocks as soon as possible, but if it turns out to be invalid, they would like it replaced with a valid template as soon as possible.
Design

If we haven't checked the template's long poll id, return the template to the client immediately, then spawn a task to check it as a proposal. If the proposal is invalid, mark the long poll id of the template as invalid.
If a long poll id in a client request is invalid, check the next template for validity before returning it. If it is valid, return it immediately, even if there haven't been any changes. If it is not valid, continue long polling until a valid template is found.
Return a RPC error if we don't find a valid template within a target block interval or two.

Optional
4. Try to recover from an invalid template by dropping transactions at random.
Specifications
This checks all the block consensus rules:
https://zips.z.cash/protocol/protocol.pdf#blockheader
https://zips.z.cash/protocol/protocol.pdf#txnencoding
Complex Code or Requirements
We already have a function to create a block proposal and submit it to the state in our tests, so this should be simple to implement.
We should check performance on large blocks before and after this change.
Testing
This change would add a runtime test for every block proposal in production code. We already do these tests in CI.

Hey team! Please add your planning poker estimate with Zenhub @arya2 @conradoplg @dconnolly @oxarbitrage @teor2345 @upbqdn

I was just looking at what we need to fix to officially support Zebra's mining RPCs.
If it turns out this ticket is tricky to implement, we could just assume that block templates will always be valid, and close this ticket. Then if we ever generate an invalid template, we can open a specific ticket for the bug that produced the invalid template.

@mpguerra I think this ticket has accidentally been scheduled without its dependent ticket #6037. Maybe we actually want to do #6037, and not do this ticket?


Revert "build_library: set osType to otherlinux-64"

This reverts commit 39bb800f16afd077c3e7d930450bf327a6d895db.
This change disabled a number of features so it isn't suitable for the
generic VMware templates. We need to re-trace our steps to list exactly
what tools/systems weren't accepting the linux26 type.

LGTM

Great, thank you :-)


How does this work? For what purpose?

How does this work? For what purpose?

How does this work? For what purpose?

Hej, Sorry im currently not home for a month but this is supposed to be client example to connect to the GW File Server and download any file by its FileID.
I actually forgot to rename the project to "Client" than "Server".
I think i already continued on this project but wont be home for another Month.
Just clone it and build it with VS17 or higher

Just updated the project with a README. feel free to re-check it


macOS packaging

Now the macOS app should be able to build and run, and support the latest version of IntelliJ. All the manual tests are supposed to pass.
But there are still a few known bugs. For example, when a survey is triggered there will be an additional icon of Taqo appearing and remaining in the dock but has no effect when being clicked on.

Thanks!