🐛 cloud_functions Incorrect error for missing function: Response is not valid JSON object

Bug report
When calling a function that cannot be found, the following error message is given:
[firebase_functions/internal] Response is not valid JSON object.
This function was previously working, but is now receiving this error after upgrading to cloud functions 2nd gen. If the function name is replaced with any random string (a certainly non-existent function), the same error is received. I have verified the function is not called in the function logs in cloud console.
The 2nd gen function is in a separate source package v2 from 1st gen functions v1, according to these instructions: https://firebase.google.com/docs/functions/organize-functions#managing_multiple_source_packages_monorepo
Steps to reproduce
Steps to reproduce the behavior:

Update function to 2nd gen in separate source package.
Run function.

Expected behavior
An error message saying that the function is not found or invalid should be shown.
Sample project
    final HttpsCallable cloud = FirebaseFunctions.instance.httpsCallable(
        'search',
        options: HttpsCallableOptions(timeout: Duration(seconds: 30)));
    await cloud.call(<String, dynamic>{
      'type': 'find',
      'query': query,
    }).then((result) {
      //Do things
    }).catchError((error) {
      print('ERROR RUNNING FUNCTION: $error');
    });

Additional context
The function still does not call with the following function names: v2-search v2.search v2:search https://search-xxxxxxxxxx-uc.a.run.app.

Flutter doctor
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.0.5, on Microsoft Windows [Version 10.0.19044.1826], locale en-US)
[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.0)
[√] Android Studio (version 2021.2)
[!] Android Studio (version 2021.2)
    X Unable to determine bundled Java version.
[√] VS Code (version 1.69.2)
[√] Connected device (4 available)
[√] HTTP Host Availability

! Doctor found issues in 2 categories.

Flutter dependencies
Click To Expand
cloud_functions: ^3.3.3
firebase_auth: ^3.6.1
firebase_core: ^1.20.0

Thanks for the report @jt274
Please take a look at this document and see if, since the v2 version of the plugin is in beta public preview, you might be seeing this error.

Update function to 2nd gen in separate source package.

Can you provide how to do this ?
Also, looking at context of the issue, this might belong to native SDK and not at client side, as FlutterFire plugins are wrappers around native sdks.

Thanks for the update. Can you provide the entire error log ?
Also, please check if this helps in your case.

@darshankawar I'd already come across that post, and specifying the region does not make a difference. The function is already in the us-central1 region. The only error provided is the error I included above. But it is clear to me the function is not being called.
According to this reference: https://firebase.google.com/docs/functions/beta/callable
You must use the full URL to call the function since v2 does not yet support cloudfunctions.net URLs. I have tried the full URL and it gives the same error. At the link above, it only shows examples for the Web and Swift/Kotlin SDKs. It appears that v2 callable functions are currently not supported in the Flutter SDK. For the time being, I will need to use a regular HTTP request and include the Authorization header.

I'm using my v2 functions in the same package as my v1 functions and still I get this error on flutter.
I/flutter (12830): [firebase_functions/internal] Response is not valid JSON object.

I'm using:
cloud_functions: 3.3.3

The code works with the same function which is written in v1.
This is how I'm calling the function:
      HttpsCallable callable = FirebaseFunctions.instance
          .httpsCallable("functionnamev2");

      dynamic response = await callable.call(<String, dynamic>{
        "id": id,
        "hours": 40,
        "score": 10
      });

Does cloud_functions plugin support v2 functions?

It appears it currently does not support v2 functions. You must call the function with the full URL using a regular HTTP request (using dart http package), and include the Authorization header if using firebase auth. I have done this successfully.
I've also separately discovered v2 functions cannot connect to a Cloud SQL instance. It seems that the current v2 beta preview is not very useful yet.

as I see the problem with [cloud_functions](https://pub.dev/packages/cloud_functions) not [cloud_firestore](https://pub.dev/packages/cloud_firestore) ..
I simply don't use cloud_functions, I use simple [http](https://pub.dev/packages/http) requests instead, and trigger the function by calling it's triggerable URL..
by making POST request and provide whatever parameters I want ..
it's suggestion , works perfect for me, and this is what the plugin do in general ...
thx

It seems that the current v2 beta preview is not very useful yet.

Considering this and as also shared the same as part of this document from Firebase, the issue doesn't seem to be at client side due to current limitation. I advise you to reach out to Firebase support for further resolution.

@darshankawar I do believe the original bug report is still valid though, for the cloud_functions package. When a function is not found, it should provide an error message that says the function is not found. The same error received when trying to call a v2 function, is received when putting in a random string of characters for the function name.
It currently returns:
[firebase_functions/internal] Response is not valid JSON object.
But it should return something like:
[firebase_functions/invalid] Function not found.

Thanks for the update. Keeping this issue open for further insights from the team.

I'm also getting an error when calling a v2 function. v1 works perfectly fine, but when calling a v2 function, I get:

[firebase_functions/-1004] Could not connect to the server.

Yes, the region is correct, and ingress is set to "ALLOW_ALL". When I paste the function url into a webbrowser it shows up in the logs as "wrong method GET" along with complaints about missing parameters. This does not happen for a function called via the cloud_functions: ^4.0.6 package.

In case someone else struggles with this, here's the code snippet I'm using, now it works:
import 'package:deep_pick/deep_pick.dart';

...

  final data = jsonEncode({
    "data": {
      "goals": goals,
      "trainingplans": trainingplans,
      "skills": skills,
    }
  });

  final response = await http.post(
    Uri.parse("https:/...-...-ew.a.run.app"),
    headers: {
      "Content-Type": "application/json",
    },
    body: data,
  );
  final result = jsonDecode(response.body)["result"];

We support v2 functions since https://pub.dev/packages/cloud_functions/versions/4.1.0, I'll close this issue. Feel free to open a new one if your issue is still not solved.


Question about Maximum Score and Expert Dataset

Hi,
I want to know how the maximum score is obtained for MuJoCo tasks? From the wiki (https://github.com/rail-berkeley/d4rl/wiki/Dataset-Reproducibility-Guide#gym-mujocogym-bullet), it seems that we use the stochastic SAC policy to obtain the expert dataset. But, in rlkit, we evaluate the performance of SAC by its deterministic policy. Typically, if we use the stochastic policy to evaluate, the performance is not very good. Thus, I am not sure whether the reported maximum score is based on the deterministic policy or the stochastic policy.
If the reported score is based on the deterministic policy,  should we consider the deterministic policy to collect the expert dataset?
Highly appreciate it if anyone can help.

Interested to hear from the team on this as well


How to evaluate provided mesh result?

Hi,
It seems that the provided mesh is not aligned with scannet GT mesh (scenexxxx_vh_clean_2.ply), ie they are in different coordinates and scales are different. When I naively borrowed the evaluation code from monosdf, the results are very bad. How to align your provided mesh with GT mesh so that I can evaluate it?
Another question, the visualization of my result has more messy background than your provided mesh. Do you use post processing? And will these post processing affect the quantitative evaluation result (since the mesh looks cleaner)?
My mesh:

your mesh:

Ithink the second mesh is extracted by using the formal post-processing used in MonoSDF / ManhattanSDF

It seems the OOM happens in Marching Cubes stage.
I used the GPU with 40G memory, I would also recommend you set a smaller MC resolution, like 256 or 128, it won't affect the results too much.
And also, if the mesh results are naturally very big (occupies too much space), it would also consume a lot of space in MC


[IOS-576] In /events GET calls, pass with_participants parameter

Currently we pass the parameter with_contacts, the correct name is with_participants. I also changed withContacts: in the selectors to participantsLimit:.

Coverage increased (+0.08%) to 80.351% when pulling f8b7fefe54c759e4d381559df79383b00b02dc4b on ios_576_event_guest_list into ed42188401d66164cbf79804934508eeef420441 on master.

Coverage increased (+0.08%) to 80.351% when pulling f8b7fefe54c759e4d381559df79383b00b02dc4b on ios_576_event_guest_list into ed42188401d66164cbf79804934508eeef420441 on master.

👍

Thanks @Ruenzuo !


It says Read-only applicatipn cannot POST

Hi I know that I need to change it to Read+write+direct message, but I have already done it and It still does not work for me!
here is a picture https://imgur.com/a/jeXhJ53

You have to regenerate the "access token and secret" keys so that they have read and write permissions. I reset the API keys as well but idk if that's important.


[WIP] [HUDI-674] Rename hudi-hadoop-mr-bundle to hudi-hive-bundle

What is the purpose of the pull request
Mmore details, ge ahead with
https://lists.apache.org/thread.html/ra659be560374a9c2f39007e719bac11a8422e31f42769a49a757ea25%40<dev.hudi.apache.org>
Brief change log

Rename hudi-hadoop-mr-bundle to hudi-hive-bundle

Verify this pull request
Test steps: follow docker demo.
http://hudi.apache.org/docs/docker_demo.html
Committer checklist

[X] Has a corresponding JIRA in PR title & commit

[X] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

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

@@             Coverage Diff             @@
##             master   #1387      +/-   ##
===========================================
+ Coverage     67.19%   67.2%   +0.01%
  Complexity      223     223
===========================================
  Files           335     335
  Lines         16279   16279
  Branches       1661    1661
===========================================
+ Hits          10939   10941       +2
+ Misses         4604    4603       -1
+ Partials        736     735       -1

Impacted Files
Coverage Δ
Complexity Δ

...n/java/org/apache/hudi/common/model/HoodieKey.java
88.88% <0%> (-5.56%)
0% <0%> (ø)

...e/hudi/common/table/log/HoodieLogFormatWriter.java
76.92% <0%> (+0.96%)
0% <0%> (ø)
:arrow_down:

...i/utilities/deltastreamer/HoodieDeltaStreamer.java
80.8% <0%> (+1.01%)
8% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

@bvaradar can you take a pass at this?

I went through the relevant DISCUSS thread. @lamber-ken : I strongly feel we should  at the least wait for next release for this change. I agree with changing hudi-hive to hudi-hive-sync. But changing hudi-hadoop-mr to hudi-hive in the same release would be very confusing to users who are using previous versions of hudi in production. Let me know your thoughts.
cc @vinothchandar

Hi @bvaradar, reasonable. Keep it WIP untill next release version.

@bvaradar Good call.. Did not think of this angle, that it might be confusing to have the same bundle name represent something totally different . lets delay this for a later release.
thanks @lamber-ken , as always!


HttpConnectionManager doesn't forward the request body to wasmplguin in case of failures like no_healthy_upstream

Title:  HttpConnectionManager doesn't forward the request body to wasmplguin in case of failures like no_healthy_upstream  for outbound calls to upstream from envoy
Description:

What issue is being seen? Describe what should be happening instead of
the bug, for example: Envoy should not crash, the expected value isn't
returned, etc.

I have a wasm plugin (on the oubound calls ) that has some logging and metrics based on the request headers and body in golang deployed on istio. It works as expected in cases where the request and response are succesfully proxied(ie. with 200 status code).

However in a scenario where the upstream envoy cluster might not have any ips then the connection manager uses a local reply to respond with no_healthy_upstream. In this situation the body is not passed to the wasmplugin , but the headers are passed.

Could you suggest any way in which i can make sure that envoy passes the request body back to the wasmplugin? I changed the priority of my http filters such that wasm plugin is the first in the chain for the http_connection_manager. However that didn't help with this.
Repro steps:

Include sample requests, environment, etc. All data and inputs
required to reproduce the bug.

here's my plugin and its deployed on a service with istio mesh.
package main

import (
	"errors"
	"fmt"
	"github.com/tetratelabs/proxy-wasm-go-sdk/properties"
	"github.com/tetratelabs/proxy-wasm-go-sdk/proxywasm"
	"github.com/tetratelabs/proxy-wasm-go-sdk/proxywasm/types"
	"os"
	"regexp"
	"strconv"
	"strings"
)
func main() {
	proxywasm.SetVMContext(&vmContext{})
}

type vmContext struct {
	types.DefaultVMContext
}

type pluginContext struct {
	types.DefaultPluginContext
	metrics *Metrics
}

func (*vmContext) NewPluginContext(contextID uint32) types.PluginContext {
	return &pluginContext{
		metrics: NewMetrics(),
	}
}

func (ctx *pluginContext) NewHttpContext(contextID uint32) types.HttpContext {

	return &httpContext{contextID: contextID, pluginContext: ctx}
}

type httpContext struct {
	types.DefaultHttpContext
	contextID      uint32
	requestHeaders *[][2]string
	requestBody    *[]byte
	pluginContext  *pluginContext
}

func (ctx *httpContext) OnHttpRequestHeaders(_ int, endOfStream bool) types.Action {
	proxywasm.LogInfof("WASM plugin Handling request endofstream %t", endOfStream)
	// Get the actual request headers from the Envoy Sidecar
	requestHeaders, err := proxywasm.GetHttpRequestHeaders()
	if err != nil {
		proxywasm.LogCriticalf("failed to get request headers: %v", err)
		return types.ActionContinue
	}

	body, err := proxywasm.GetHttpRequestBody(0, 100000)
	if err != nil {
		proxywasm.LogErrorf("failed to get request body: %v", err)
	}
	proxywasm.LogInfof("body %s", string(body))

	ctx.requestHeaders = &requestHeaders
	return types.ActionContinue
}

func (ctx *httpContext) OnHttpRequestBody(bodySize int, endOfStream bool) types.Action {
	proxywasm.LogInfof("WASM plugin Handling response body %d - %t", bodySize, endOfStream)

	body, err := proxywasm.GetHttpRequestBody(0, bodySize)
	if err != nil {
		proxywasm.LogErrorf("failed to get request body: %v", err)
		return types.ActionContinue
	}
	ctx.requestBody = &body
	proxywasm.LogInfof("body %s", string(body))

	return types.ActionContinue
}

func (ctx *httpContext) OnHttpResponseHeaders(numHeaders int, endOfStream bool) types.Action {
	proxywasm.LogInfof("WASM plugin Handling response headers")

	hrs,_ := proxywasm.GetHttpResponseHeaders()

	for _, hrs := range hrs {
		proxywasm.LogInfof("response header %s - %s", hrs[0], hrs[1])
	}

	return types.ActionContinue
}

Note: The Envoy_collect tool
gathers a tarball with debug logs, config and the following admin
endpoints: /stats, /clusters and /server_info. Please note if there are
privacy concerns, sanitize the data prior to sharing the tarball/pasting.

Admin and Stats Output:

Include the admin output for the following endpoints: /stats,
/clusters, /routes, /server_info. For more information, refer to the
admin endpoint documentation.

Note: If there are privacy concerns, sanitize the data prior to
sharing.

Config:

Include the config used to configure Envoy.

Logs:

Include the access logs and the Envoy logs.

Note: If there are privacy concerns, sanitize the data prior to
sharing.

Call Stack:

If the Envoy binary is crashing, a call stack is required.
Please refer to the Bazel Stack trace documentation.

i have also posted this issue on istio github https://github.com/istio/istio/issues/50135

Hi @yanavlasov , is there any workaround to get the body ?

Unfortunately not.


Firefox doesn't let the getter know when their connection is revoked

Firefox 36, OSX 10.9
Currently Chrome gives a nice little explanatory message when the getting user has their access revoked, and lets them click to explicitly go back to their regular connection. On Firefox (both via Google social channel and with one-time connection), if the giver revokes then the getter just gets a "The proxy server is refusing connections" message. It doesn't explain why they can't connect or give them any useful route to fix it (the "try again" button doesn't work, unsurprisingly). The browser needs to be restarted to continue use.

Duplicate of #249, closing


FPU instructions for mirb

implement 80387 instructions needed to execute basic floating point arith on mruby.

Took it!  Thanks!


Modernise documentation examples

from __future__ import print_function was removed
% operator replaced with f-string

Looks good to me, too. Thanks!


Make part of Terminal.Gui and/or put in the gui-cs org?

@tznind this is awesome!
Is there a way to make this part of the Terminal.Gui lib so this would work:
dotnet new --install Terminal.Gui.templates

?
Or, did you name it gui.cs because you think it should sit above the namespace hierarchy of github.com/gui-cs?

We should be able to make the PackageId anything we want.  It is defined here:
https://github.com/tznind/gui.cs.template/blob/main/gui.cs.templates.csproj#L6
So we should be able to call it Terminal.Gui.templates.  I think for consistency I would rename the repo and the .csproj to match.
It gets pushed as a standalone nuget package that contains the template files only (i.e. not the Terminal.Gui dll) and that has already been pushed under the title gui.cs.templates here https://www.nuget.org/packages/gui.cs.templates/ .  NuGet might not like changing the PackageId but even if that is the case we can just unlist the existing package and relist with the new title.
Do you want me to transfer the repo to gui.cs (and rename to Terminal.Gui.templates ?
So the steps are:

[ ] Transfer repo to gui.cs org (renaming to Terminal.Gui.templates)
[ ] Rename csproj and PackageId
[ ] Update README instructions
[ ] Push to nuget (optionally delisting old package).

I like the idea of putting as much "TUI in .NET" stuff as possible in the gui-cs org, so, yes, that would be my preference. But it's your code and up to you in the end!

You are now an Admin. I renamed it.

Do we want to rename the template too?
The way the system works is that you have 1 nuget package (i.e. Terminal.Gui.templates) and then 1 or more templates within that package.
Currently there is only 1 template within the package and it is called guics
i.e.
dotnet new guics -n myproj

Each template name is defined in its corresponding template.json file.  The one we have so far is this:
https://github.com/gui-cs/Terminal.Gui.templates/blob/main/templates/basic/.template.config/template.json
These are the docs on template.json: https://learn.microsoft.com/en-us/dotnet/core/tools/custom-templates#templatejson
For reference these are the ones that ship with dotnet or I've installed.  You can see this list with dotnet new --list
Template Name                Short Name           Language    Tags
---------------------------  -------------------  ----------  ---------------------------------------------------------
.NET MAUI App                maui                 [C#]        MAUI/Android/iOS/macOS/Mac Catalyst/Windows/Tizen
.NET MAUI Blazor App         maui-blazor          [C#]        MAUI/Android/iOS/macOS/Mac Catalyst/Windows/Tizen/Blazor
.NET MAUI Class Library      mauilib              [C#]        MAUI/Android/iOS/macOS/Mac Catalyst/Windows/Tizen
.NET MAUI ContentPage (C#)   maui-page-csharp     [C#]        MAUI/Android/iOS/macOS/Mac Catalyst/WinUI/Tizen/Xaml/Code
.NET MAUI ContentPage (X...  maui-page-xaml       [C#]        MAUI/Android/iOS/macOS/Mac Catalyst/WinUI/Tizen/Xaml/Code
.NET MAUI ContentView (C#)   maui-view-csharp     [C#]        MAUI/Android/iOS/macOS/Mac Catalyst/WinUI/Tizen/Xaml/Code
.NET MAUI ContentView (X...  maui-view-xaml       [C#]        MAUI/Android/iOS/macOS/Mac Catalyst/WinUI/Tizen/Xaml/Code
.NET MAUI ResourceDictio...  maui-dict-xaml       [C#]        MAUI/Android/iOS/macOS/Mac Catalyst/WinUI/Xaml/Code
Android Activity template    android-activity     [C#]        Android/Mobile
Android Application          android              [C#]        Android/Mobile
Android Class Library        androidlib           [C#]        Android/Mobile
Android Java Library Bin...  android-bindinglib   [C#]        Android/Mobile
Android Layout template      android-layout       [C#]        Android/Mobile
Android Wear Application     androidwear          [C#]        Android/Mobile
ASP.NET Core Empty           web                  [C#],F#     Web/Empty
ASP.NET Core gRPC Service    grpc                 [C#]        Web/gRPC
ASP.NET Core Web API         webapi               [C#],F#     Web/WebAPI
ASP.NET Core Web App         webapp,razor         [C#]        Web/MVC/Razor Pages
ASP.NET Core Web App (Mo...  mvc                  [C#],F#     Web/MVC
ASP.NET Core with Angular    angular              [C#]        Web/MVC/SPA
ASP.NET Core with React.js   react                [C#]        Web/MVC/SPA
ASP.NET Core with React....  reactredux           [C#]        Web/MVC/SPA
Blazor Server App            blazorserver         [C#]        Web/Blazor
Blazor WebAssembly App       blazorwasm           [C#]        Web/Blazor/WebAssembly/PWA
Class Library                classlib             [C#],F#,VB  Common/Library
Console App                  console              [C#],F#,VB  Common/Console
dotnet gitignore file        gitignore                        Config
Dotnet local tool manife...  tool-manifest                    Config
EditorConfig file            editorconfig                     Config
global.json file             globaljson                       Config
iOS Application              ios                  [C#]        iOS/Mobile
iOS Binding Library          iosbinding           [C#]        iOS/Mobile
iOS Class Library            ioslib               [C#]        iOS/Mobile
iOS Controller template      ios-controller       [C#]        iOS/Mobile
iOS Tabbed Application       ios-tabbed           [C#]        iOS/Mobile
MacCatalyst Application      maccatalyst          [C#]        macOS/Mac Catalyst
MacCatalyst Binding Library  maccatalystbinding   [C#]        macOS/Mac Catalyst
MSTest Test Project          mstest               [C#],F#,VB  Test/MSTest
MVC ViewImports              viewimports          [C#]        Web/ASP.NET
MVC ViewStart                viewstart            [C#]        Web/ASP.NET
NuGet Config                 nugetconfig                      Config
NUnit 3 Test Item            nunit-test           [C#],F#,VB  Test/NUnit
NUnit 3 Test Project         nunit                [C#],F#,VB  Test/NUnit
Protocol Buffer File         proto                            Web/gRPC
Razor Class Library          razorclasslib        [C#]        Web/Razor/Library/Razor Class Library
Razor Component              razorcomponent       [C#]        Web/ASP.NET
Razor Page                   page                 [C#]        Web/ASP.NET
React.NET Starter Template   reactnet-vanilla     [C#]        Web
React.NET Webpack Starte...  reactnet-webpack     [C#]        Web
Solution File                sln                              Solution
Terminal Gui Basic Template  guics                [C#]        Console/TUI
Web Config                   webconfig                        Config
Windows Forms App            winforms             [C#],VB     Common/WinForms
Windows Forms Class Library  winformslib          [C#],VB     Common/WinForms
Windows Forms Control Li...  winformscontrollib   [C#],VB     Common/WinForms
Worker Service               worker               [C#],F#     Common/Worker/Web
WPF Application              wpf                  [C#],VB     Common/WPF
WPF Class library            wpflib               [C#],VB     Common/WPF
WPF Custom Control Library   wpfcustomcontrollib  [C#],VB     Common/WPF
WPF User Control Library     wpfusercontrollib    [C#],VB     Common/WPF
xUnit Test Project           xunit                [C#],F#,VB  Test/xUnit

So the convention seems to be no dots and reasonably short.

Seems like the template name should be a noun that best describes the app the template creates.
Over time, I could imagine templates for:

Generic Empty Application - generic
Menu and StatusBar Application - menu-statusbar
Wizard Application - wizard

Is this what you're asking?

Actually... What if we just 'owned' "TUI":

TUI Generic Empty Application - tui-generic
TUI Menu and StatusBar Application - tui-menu-statusbar
TUI Wizard Application - tui-wizard
TUI TreeView View - tui-treeview-view
TUI Background Worker - tui-backgroundworker

(Can you use templates for partials, or just full projects?)

Actually... What if we just 'owned' "TUI":

Yes! I think that is perfect actually.  I will rename the shortcode to tui and we can add the others later.
I think having a template that is just tui alone (not tui-generic)  is consistent with the wpf ones where there is:

wpf
wpflib
wpfcustomcontrollib
wpfusercontrollib

(Can you use templates for partials, or just full projects?)

I'm not sure.  I've only seen it used for dotnet new so far.

Ok I have marked the previous packages as Deprecated.  New package is at https://www.nuget.org/packages/Terminal.gui.templates


RangeError: Maximum call stack size exceeded

Describe the Bug
I have added some models to schema and successfuly generate client (with emitTranspiledCode = false and simpleResolvers = true).
When I tried to run server I got:
/var/www/vhosts/flamboyant-khayyam.212-44-104-170.plesk.page/crm/src/generated/type-graphql/resolvers/inputs/WorkPositionCreateManyCreatedByInput.js:32
const TypeGraphQL = __importStar(require("type-graphql"));
                    ^

RangeError: Maximum call stack size exceeded
    at Object.<anonymous> (/var/www/vhosts/flamboyant-khayyam.212-44-104-170.plesk.page/crm/src/generated/type-graphql/resolvers/inputs/WorkPositionCreateManyCreatedByInput.js:32:21)
    at Module._compile (node:internal/modules/cjs/loader:1097:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1149:10)
    at Module.load (node:internal/modules/cjs/loader:975:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:999:19)
    at Module.require (/usr/share/passenger/helper-scripts/node-loader.js:80:25)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/var/www/vhosts/flamboyant-khayyam.212-44-104-170.plesk.page/crm/src/generated/type-graphql/resolvers/inputs/WorkPositionCreateManyCreatedByInputEnvelope.js:33:48)
    at Module._compile (node:internal/modules/cjs/loader:1097:14)

Node.js v17.1.0

OS: CentOS Linux 7.9.2009 (Core)
Any idea how to approach this issue?

Can you try on Node 14 or 16?

On Node 16.3.0 I got the same message, but on 14.18.1 looks fine. I have now two setups running. I will post my updates and observations here. Im good with 14.x but it is getting old.
Thnx Janez

Sounds like duplicate of #10 #202 🔒

Added "node --stack-size=2048 index.js" and it works;
Or "ts-node-dev --stack_size=2048 index.ts"


Braindump about Docker

This draft PR contains the results of me messing around with docker for an hour. I'm not experienced with docker, but I see it as a valuable tool to allow anyone (eg workshop participants) to spin up a node quickly without installing the entire rust toolchain and building the node themselves.
cc @kevcoxe you're good at docker right? Want to get into blockchain?

I guess running a node that you didn't build yourself has limited value in this particular repo because it is a workshop about modifying the code. Still this technique may be useful more generally.

@JoshOrndorff I will have to check this out! I am excited that it is containerized because now like you said i dont need to install all the rust stuff.


MAPREDUCE-7373. Building MapReduce NativeTask fails on Fedora 34+

Description of PR
Building MapReduce NativeTask with GCC 11, in which C++17 features are enabled by default, leads to the following error. This PR fixes this problem.
error: ISO C++17 does not allow dynamic exception specifications

How was this patch tested?
I ran the following command locally and confirmed that it succeeded.
$ mvn package -Pdist,native -DskipTests -Dtar -Dmaven.javadoc.skip=true

For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[x] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
39m 30s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
34m 40s

trunk passed

+1 :green_heart:
compile
0m 58s

trunk passed

+1 :green_heart:
mvnsite
0m 34s

trunk passed

+1 :green_heart:
shadedclient
55m 38s

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

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 22s

the patch passed

+1 :green_heart:
compile
0m 48s

the patch passed

-1 :x:
cc
0m 48s
/results-compile-cc-hadoop-mapreduce-project_hadoop-mapreduce-client_hadoop-mapreduce-client-nativetask.txt
hadoop-mapreduce-project_hadoop-mapreduce-client_hadoop-mapreduce-client-nativetask generated 37 new + 78 unchanged - 37 fixed = 115 total (was 115)

+1 :green_heart:
javac
0m 48s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 22s

the patch passed

+1 :green_heart:
shadedclient
19m 16s

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

_ Other Tests _

+1 :green_heart:
unit
2m 54s

hadoop-mapreduce-client-nativetask in the patch passed.

+1 :green_heart:
asflicense
0m 33s

The patch does not generate ASF License warnings.

120m 46s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense compile cc mvnsite javac unit codespell

uname
Linux e4e8eb838ba0 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / cae593ae98f0d21d1818f3ed404f5fb3ecea642b

Default Java
Red Hat, Inc.-1.8.0_322-b06

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

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

modules
C: hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-nativetask U: hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-nativetask

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

versions
git=2.9.5 maven=3.6.3

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

This message was automatically generated.

@sekikn HADOOP-17196(#2208) replaced -std=* with CMAKE_CXX_STANDARD. How about using the CMAKE_CXX_STANDARD here too?

@iwasakims Thank you for the comment! Updated the PR.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 58s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
34m 58s

trunk passed

+1 :green_heart:
compile
0m 58s

trunk passed

+1 :green_heart:
mvnsite
0m 34s

trunk passed

+1 :green_heart:
shadedclient
55m 46s

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

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 22s

the patch passed

+1 :green_heart:
compile
0m 48s

the patch passed

-1 :x:
cc
0m 48s
/results-compile-cc-hadoop-mapreduce-project_hadoop-mapreduce-client_hadoop-mapreduce-client-nativetask.txt
hadoop-mapreduce-project_hadoop-mapreduce-client_hadoop-mapreduce-client-nativetask generated 11 new + 104 unchanged - 11 fixed = 115 total (was 115)

+1 :green_heart:
javac
0m 48s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 23s

the patch passed

+1 :green_heart:
shadedclient
19m 13s

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

_ Other Tests _

+1 :green_heart:
unit
2m 51s

hadoop-mapreduce-client-nativetask in the patch passed.

+1 :green_heart:
asflicense
0m 34s

The patch does not generate ASF License warnings.

82m 27s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense compile cc mvnsite javac unit codespell

uname
Linux 8b3e12d8cebf 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / bdccf1850be3ca45ddcc1deff211b165bd945f8a

Default Java
Red Hat, Inc.-1.8.0_322-b06

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4120/2/testReport/

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

modules
C: hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-nativetask U: hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-nativetask

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4120/2/console

versions
git=2.9.5 maven=3.6.3

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

This message was automatically generated.


tests: Could not get a connection to the database

As seen in https://ci.deis.io/job/test-ec2/126/consoleFull#682192135502066c60b2236c09cc9bf8
database: performing an initial backup...
wal_e.main   INFO     MSG: starting WAL-E
        DETAIL: The subcommand is "backup-push".
        STRUCTURED: time=2015-03-19T01:49:22.822293-00 pid=144
2015-03-19 01:49:22 UTC FATAL:  the database system is starting up
psql: FATAL:  the database system is starting up
wal_e.main   ERROR    MSG: could not run one or more external programs WAL-E depends upon
        DETAIL: Could not get a connection to the database: note that superuser access
        is required
        Could not run the following programs, are they installed? psql
        STRUCTURED: time=2015-03-19T01:49:22.979408-00 pid=144
*** Test killed: ran too long (10m0s).

This particular CI issue is now a distant memory AFAICT.


Tracing: missing (parent) spans

In a technical session with a customer they mentioned that they only get db.BeginTx span in their tracing.
Similar behavior can be observed on our QA environment:

Production is not affected.

We've noticed these too. Most of these seems to come from Projection queries run periodically.
Adding a span to these would be awesome.


You might want to take a look at this repo

You both have the same idea, based on Scriptable Object
https://github.com/pancake-llc/foundation

It's a popular thing.
On Fri, May 31, 2024, 12:13 AM Md. Ishtiaq Ahamed Fahim <
@.***> wrote:

You both have the same idea, based on Scriptable Object
https://github.com/pancake-llc/foundation
—
Reply to this email directly, view it on GitHub
https://github.com/JamesLaFritz/Unity6CoreFramework/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHNDPVB5KUKOAWKCMQDFBYTZE72GTAVCNFSM6AAAAABISCE7WWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGMZDMOJSGY2TKNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Problems with delay and UI settings

Hi, thanks for the module, its incredible.
But I'm having problems with the notifications, they started appearing late, with a delay of a minute or so, by then I had already moved to a different pokemon...
Also because of that I tried to reduce stuff by un-selecting  gym info, pokestop info, startup info. But even restarting the whole phone, or even deactivating the module and reactivating it wouldn't stop notifications like Pokestop or gym from popping, even if they are deselected.
Thanks for the module again.

This is because of the new scroll feature. Try to open the PoGoIV app before starting pokemon go so it can receive the Broadcasts from PokemonGo.

What about the uncheck status off Pokestop info and gym info?
I have forceclosed pogo, and even uninstalled and reinstalled both pogo and your module... and its still happening.


feat: search command

Command:
$ stencil search <query> --namespace=<namespace> --schema=<schema> --version=<version> --history=<history>
Set either of version or history flags. Version flag is used to search on a particular version while history flag enables the user to search across all the versions.

This is how this will look now. WDYT about it?
Showing 3 versions
NAMESPACE       SCHEMA  VERSION TYPES           FIELDS
mynamespace     abc     3       stencil.A.Z     field_z
mynamespace     abc     3       stencil.B.X     field_z
mynamespace     abc     3       stencil.C.Z     field_a
mynamespace     abc     3       stencil.D.Z     field_z

mynamespace     abc     4       stencil.A.Z     field_z
mynamespace     abc     4       stencil.A.Y     field_z
mynamespace     abc     4       stencil.B.X     field_z
mynamespace     abc     4       stencil.B.Y     field_z
mynamespace     abc     4       stencil.C.Z     field_a
mynamespace     abc     4       stencil.D.Z     field_z

mynamespace     abc     5       stencil.A.Z     field_z, field_z_z
mynamespace     abc     5       stencil.A.Y     field_z, field_z_z , field_z_z_z
mynamespace     abc     5       stencil.B.X     field_z
mynamespace     abc     5       stencil.B.Y     field_z
mynamespace     abc     5       stencil.C.Z     field_a
mynamespace     abc     5       stencil.D.Z     field_z

TOTAL:  16

Shall we remove the NAMESPACE and SCHEMA columns, since it will be a piece of redundant information (same as flag value)? @harikrishnakanchi

@ishanarya0 lets keep them. Also specifying namespace should also be optional. User should be able to search across schemas. Does API support that right now?

@ishanarya0 lets keep them. Also specifying namespace and schema should also be optional. User should be able to search across schemas. Does API support that right now?

Yes, keeping them. Oh, yes they can be made optional too. Doing the changes. Thanks for pointing it out!


[2.0.x] Add the workaround for openapi resource annotation data binding issue

Purpose

$Subject

Shall we add a test with this new annotation? If someone removes the readonly & part (since it's theoretically redundant with annotations), we wouldn't detect that.

Will add some integration tests for this


Add no-unescaped-entities to recommended configuration

This rule was added by #681 and I think it should be added to the recommended configuration for the next major version bump.

@ljharb I'm curious what the downsides are to not escaping '? Makes writing stuff pretty difficult, but I'm sure there's a good reason

@rauchg The rule documentation should explain it; basically, it's highly likely there's a bug.
Separately, it shouldn't impact "writing stuff" at all, since straight quotes are typographically incorrect, and you don't need to escape curly quotes.


Test Layout with German version

I was at Codemotion Berlin 2013 and some Game creators said that one should test the layout with the German version because the German language uses the most space. That aside, I found two layout mistakes:

this is on english version too.

i am looking for an easy task to start with and will look into this :)

@nerdess have you set up the development environment or do you need help with it?

localhost:8000 is running but got trouble importing the testdata...


Support physical block devices for device-mapper

See:  http://www.projectatomic.io/docs/docker-storage-recommendation/

Looks like we need physical block device, not loopback

Updated the ticket title

Roadmap for discussion:

Single volume group (VG) with all available extra block devices
two logical volumes for  docker (data+metadata)
volume for glusterfs (replace direct usage of block device in current setup)
other LV (for future usage)

Questions:

assertion in TF if we have docker + gluster LVs requested with size bigger than all PVs.

Closing, as this was implemented with #797.


Typo in first bullet point on governance decisions section

Recommendations and requirements for content ownership and ownership.
Should be:
Recommendations and requirements for content ownership and management.
in this section: https://docs.microsoft.com/en-us/power-bi/guidance/powerbi-adoption-roadmap-governance#key-power-bi-governance-decisions

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

ID: bf42906e-560f-f231-04a3-bedccdc502cd
Version Independent ID: e0dda81e-3dd4-eb78-9e5d-aa495e7df70d
Content: Power BI adoption roadmap: Governance - Power BI
Content Source: powerbi-docs/guidance/powerbi-adoption-roadmap-governance.md
Service: powerbi
Sub-service: powerbi-resource
GitHub Login: @peter-myers
Microsoft Alias: v-petermyers

Thanks for your comment, @jlmnn. This issue has been assigned to the doc author for review.

Hi @jlmnn - thanks for pointing out the typo. I've fixed the error and an updated article should be published soon.


Update knative-operator manifests

Context
Each charm has a set of manifest files that have to be upgraded to their target version. The process of upgrading manifest files usually means going to the component’s upstream repository, comparing the charm’s manifest against the one in the repository and adding the missing bits in the charm’s manifest.
What needs to get done
https://docs.google.com/document/d/1a4obWw98U_Ndx-ZKRoojLf4Cym8tFb_2S7dq5dtRQqs/edit?pli=1#heading=h.jt5e3qx0jypg
Definition of Done

Manifests are updated
Upstream image is used

This update requires kubernetes 1.26 or higher. In my PR I have changed the CI tto work with 1.26

This issue has been addressed by #197, closing.


Strange values of the "offset" property in the "onPageScroll" method.

Environment
"react": "17.0.2",
"react-native": "0.66.4",
"react-native-pager-view": "^6.2.0"
Description
Good afternoon! Initially, I want to thank you for creating such a wonderful library. It covers a lot of tasks in my projects, but I have difficulties with setting up the parallax effect.
Since the offset property cannot have a negative value in the "onPageScroll" method. During which I can't determine the swipe side (backward or forward).
In addition, when I swipe back, the value is not something that does not become less than 0, but also turns into 0.99 by the moment, which is why any animation moves in leaps.

offset = (0.000, 0.100, 0.320, ..., 1.000)

offset = (0.000, 0.990, 0.800, ..., 0.000)
This does not allow the use of animation of this kind:
  const onPageScroll = usePagerScrollHandler<onPageScrollEventType>(
    {
      onPageScroll: (e: onPageScrollEventType): void => {
        'worklet';
       // ...
        opacityValue.value = 1 - e.offset;
       // ...
      },
    },
    'onPageScroll'
  )

After my research, I found a place to calculate this value on iOS:

(../node_modules/react-native-pager-view/ios/ReactNativePageView.m)
However, I have not found where to fix it in Android, are there any solutions to my problem?
Sincerely, Andrey!

I'm sorry! I didn't notice a change in the "position" property at the time of scrolling back.
If we take it into account, then the creation of animations is more than real ...
And even despite this, the "offset" property has non-trivial values)

@AndrewNes please checkout one of our example here https://github.com/callstack/react-native-pager-view/blob/master/example/src/HeadphonesCarouselExample.tsx . I was not able to reproduce this issue. In case of noticed bug, please create a new issue with the full description.


Add ability to use image snapshots to addon-storyshots

Issue: https://github.com/storybooks/storybook/issues/1781
What I did

Ability to generate image snapshots from stories using addon-storyshots.

How to test

Start fresh & bootstrap everything
Go to cra-kitchen-sink
Run yarn run test-image-snapshots

Is this testable with jest or storyshots?

Not sure

Does this need a new example in the kitchen sink apps?

Yep done.

Does this need an update to the documentation?

Maybe we should say something about it in the doc as it may persuade people to use storybook ? But I don't think there is a word about storyshots in the doc yet.

just an fyi, since we added image-snapshot testing to our suites a couple of days ago: puppeteer will render text slightly different depending on the enviroment you run it on (especially mac vs linux based containers).
This means you either need to provide a big enough margin of error, or you need to record your snapshots locally from inside the container.
Either way, this change will mean that users who use container for their CI will need to modify those to include chrome. Definitely a breaking change here.

@tobilen I quite don't get why this is a breaking change as it just add an option to generate image snapshots. If one is using the addon-storyshots, he must use this option to generate images and thus the concern about linux/mac is up to him to handle while doing this, isn't it ?

@Hypnosphi could you give me a hint on what to do next to go on with this ?

@Hypnosphi in the meantime, can you elaborate on why this is a breaking change ? :thinking:

what to do next to go on with this

I'd like to hear from @ndelangen, he has set up visual tests for kitchen sink apps

hey @thomasbertet i mistakenly assumed you'd want storyshots to do image snapshotting as a default. i now see that you're just providing a helper method. Sorry about that.
I would include something about that in the readme though.

Thanks for the contribution, I'm pretty excited for this new feature!

Thanks for all your feedbacks ! I think I got every one of them dealt with. Please let me know :)

We're gonna need to fix the unit test as well. Let me see if I can install Chrome on our CircleCI instance

@danielduan What we did internally at my company was just to "sleep" for some time before running the test. It sounds pretty hacky but hey it works ! :lollipop:
Our CicleCI config is as follow:
defaults: &defaults
  docker:
    - image: circleci/node:8.9.1-browsers
[...]
     - run:
          name: Storybook
          command: yarn storybook
          background: true
      - run:
          name: Sleep while running storybook
          command: sleep 60
      - run:
          name: Tests
          command: ./node_modules/.bin/lerna run test --stream

The last command runs the test which need the storybook to be running.
Is this something we "could" imagine ? If you know a way to know when the storybook has launched, then we could possibly hook into this "event" ?

Why running it on ci? Why not just export the static version?

@igor-dv what do you mean ? I'm not sure I get it.

@igor-dv the CI is for running the test to make sure this does not regress.

I think @igor-dv meant that we can run build-storybook and then serve the output instead of running webpack dev server on CI

@Hypnosphi, exactly =)

Is there a way to exclude a story from screenshot testing, but still include in regular snapshots. This may be needed when using transitions and animations in components

Yes you can use the same paramters as for other storyshots config. In fact, that's what I did, I created two storyshots instances, and each instance is running their proper stories.
Note that you cannot run within a single initStoryshot() the two snapshots mode (images & HTML).
If you wish to use image snapshot side-by-side with HTML snapshots, you need to call another initStoryshots() with the proper params (including regexp for name/kind exclusion/inclusion)

@Hypnosphi any hint on how to start a node server to serve the static storybook on CircleCI ? Do you have any experience in this that can save me some time ?
Also, you would not accept this PR if I run the storybook in the CI using the webpack-dev-server, right ?
Thanks!

any hint on how to start a node server to serve the static storybook on CircleCI

Take a look at this package: https://github.com/indexzero/http-server
You can add a serve script to example app which would run it

you would not accept this PR if I run the storybook in the CI using the webpack-dev-server, right ?

It wasn't me who brought it up, @igor-dv was. I just explained what he meant

I am not sure why you even need to serve the static version. You can just point to the location where the iframe.html located. That how it works for me (on Windows with TeamCity, but I think there shouldn't be a difference)

Apparently everything is resolved (from the CI POV).
@danielduan @igor-dv @Hypnosphi I think I adressed your remarks, please let me know if something still bugs you ! Thank you :100:

@Hypnosphi should I switch the base branch to 3.4 since you added the 3.4 milestone for this PR?

We're going to change our branching approach. When 3.3 gets released, all the PRs will go to master

Anything you wish me to change in this PR folks ? Or should I just wait until 3.3 is released so we can move forward ? :)

What do you think about this? https://github.com/storybooks/storybook/pull/2413#discussion_r156966205

Just to let you know I will address the remaining issues soon, probably after new year's eve. Cheers !

Have a great fantastic new years @thomasbertet

@thomasbertet Hey, have you had a look at https://github.com/oblador/loki?


feat(zk): enable pkey generation

Description
This PR implements keygen_pkxxx, while achieving both reducing code duplication and enhancing memory efficiency.
For reducing code duplication, it tries to share codes between ProvingKey and VerifyingKey by moving some shared parts to a parent class Key.
For enhancing memory efficiency, unlike Halo2, while computing l_first, l_last and l_active_row, it uses a single evaluations and recycles it.

I think it would be more natural for tachyon/zk/plonk/circuit/assembly.h to be located in tachyon/zk/plonk/keys/assembly.h


[FEATURE] Label breakdown in Feedback dataset stats

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Several users that have moved from DatasetforTextClassification to FeedbackDataset have expressed that they miss the breakdown of labels in the stats panel.
This doesn't have an easy correlation in the context of Feedback Datasets as one could have many types of questions with many labels, but we can think of a smart solution to see this breakdown in the UI.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

The ability to breakdown data by counts for responses and metadata.
For example, for label questions it would be great to have a breakdown of how many labels I have for each response.
For example, I have annotations for a dozen clients and I save it as metadata I would like to see how many sample I have for each client.
Future it would be great to get counts for multiple conditions. If client = x and breakdown of responses.


Does not run from index.html

After npm install, gulp or gulp serve doesn't appear to work either.

I just cloned this repo, npm install, then gulp and I am not able to reproduce. It's also working on my fork. Is there any chance you didn't git pull after the last PR was merged? @wattsj @hilarie are either of you able to test this?

It worked for me.

@mmcbride1007 I did a clone and no luck.

@mmcbride1007 Clone, npm install and no luck. Run node gulp? Load the html page? Nothing happens
What version of node are you running?

@dbrunkow

@mmcbride1007
C02KV26WDR53:fishtank6 brunkowd$ node -v
v0.10.3
C02KV26WDR53:fishtank6 brunkowd$ npm -v
1.4.26
C02KV26WDR53:fishtank6 brunkowd$ gulp -v
-bash: gulp: command not found

@brunkowd try running npm install -g gulp

Yeah that could be the issue, gulp needs to be installed globally for you
to be able to run it. Same as grunt. We probably should throw that in the
README.
On Monday, August 17, 2015, Charles Harris<EMAIL_ADDRESS>wrote:

@brunkowd https://github.com/brunkowd try running npm install -g gulp
—
Reply to this email directly or view it on GitHub
https://github.com/wwt-inc/fish-tank/issues/8#issuecomment-131876450.

Can't get it to work either when building with Jenkins using node .12
Can we ungulp this?


PR: Extra selections is now an ordered dictionary

Extra selections is ordered and sorted.
The smaller selection is given higher priority.
Fixes #3859
Fixes spyder-ide/spyder-vim#7

Thanks @pwoosam, looks good to me :-)


use 2 threads for more performance

all the other miners do it and reach twice the hashrate. current work around for nheqminer is to run 2x nheqminer.exe to reach similar performance but this is a gimmik and very unreliable at times.

i have<EMAIL_ADDRESS>it have 4cores/8threads

nheqminer 6 threads - 24 sol/s
2x nheqminer 2 threads - 22 sol/s
nheqminer 6 threads but affinity for 7 cores (and gpu on 8th) - 28 sol/s from cpu

i meant gpu threads
running 2 miners aka two nheqminer.exe's give more performance

On gtx980
1x nheqminer - 100% load, 32% fb usage, 62% power usage - 25 sol/s
2x nheqminer - 100% load, 40% fb usage, 62% power usage - 25 sol/s


[DDW-1185] - Low disk space alert message not displayed for itn

This PR fixes low disk space alert message not being displayed for itn.
Screenshots

Testing Checklist

Slack QA thread

Review Checklist
Basics

[ ] PR has been assigned and has appropriate labels (feature/bug/chore, release-x.x.x)
[ ] PR is updated to the most recent version of the target branch (and there are no conflicts)
[ ] PR has a good description that summarizes all changes and shows some screenshots or animated GIFs of important UI changes
[ ] CHANGELOG entry has been added to the top of the appropriate section (Features, Fixes, Chores) and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance and unit tests are passing (yarn test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn package / CI builds)
[ ] There are no flow errors or warnings (yarn flow:test)
[ ] There are no lint errors or warnings (yarn lint)
[ ] There are no prettier errors or warnings (yarn prettier:check)
[ ] There are no missing translations (running yarn manage:translations produces no changes)
[ ] Text changes are proofread and approved (Jane Wild / Amy Reeve)
[ ] Japanese text changes are proofread and approved (Junko Oda)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly commented and documented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-renderings
[ ] Any code that only works in main process is neatly separated from components

Testing

[ ] New feature/change is covered by acceptance tests
[ ] New feature/change is manually tested and approved by QA team
[ ] All existing acceptance tests are still up-to-date
[ ] New feature/change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review

[ ] Merge the PR
[ ] Delete the source branch
[ ] Move the ticket to done column on the YouTrack board
[ ] Update Slack QA thread by marking it with a green checkmark

@DeeJayElly
Hello! I have noticed different statuses for the Node State so wanted to double check what to expect. As i recall we should Stop the node if Running or Not Even Start it if Space is Low.
The scenario is :
Fill up the Disk C <=1.5GB so once Daedalus is started the Red Screen is showed up. Start Daedalus.
The question is for this scenario what Node Status is expected? Currently for
Windows : Connecting to the Network is showed in the Background \ Running in Diagnostic Screen.
Linux : Starting Cardano Node is showed in the Background \ Starting in Diagnostics Screen.

Sometimes at Windows it shows the correct State but after 5 minutes Having Trouble Screen is showed. Do we expect it even node is not Running?

Hmm yeah that's a strange behavior, it shouldn't be like that, we should unify the error messages and show the same messages in those situations. But i think that implementation is not part of this ticket. We should raise this and create a new ticket for it.

@DeeJayElly Hey! I just tried PROD MainNet at Windows and Linux and if you Start Daedalus while Space is <1.5gb it shoots you Red Screen and in background you can Notice statuses "Stopping Cardano >Cardano Stopped" and always the last message is "Cardano Stopped" and same in Diagnostics Screen. However after waiting some time the Having Trouble message also showed up even Node is Stopped.
So for the ITN builds looks like even Red Screen is showed app still trying to Start or Run the Node. And sometimes it stuck and We cannot see Cardano Stopped Message.
I think we need Master @nikolaglumac here to clear the things up.
Thank you!

The Actual Node is Stopped just fine so looks only the Statuses sometimes are wrong. Thank you.
cc @DeeJayElly @nikolaglumac

On the latest build 10451 the Red Screen with the message Daedalus requires more hard drive space is displayed correctly. It pops up immediately after accepting terms and conditions as expected


Slew rate limiting resistors

Two arrays of four resistors per port. I'm not sure yet what the values should be; worst case, we can always populate with 0R and decide later.

This will also add some very limited pin overcurrent protection. Say at 200 ohm, the buffers should tolerate ±10V on up to two signal pins (not Vio or Vsense though).

0402 8-position array fits better (it's about the same size as FXMA108).

Pretty sure we also need slew rate limiting on the other side of the level shifter because switching many inputs at once causes glitches on the FLAGn pins on the other side of the board. The apparently random timing of the glitches seems to suggest that glitches happen when the inputs transition happens closely enough to IFCLK transition.

I added these in commit 7c3203c126e7444981d5405d7d6fcd1db53bbbc5

Thanks!


New Term Request: ATP6AP2-related disorder

**Preferred gene-related syndrome label: ATP6AP2-related disorder
For example: NAA10-related syndrome
**Synonyms: none
**Definition (free text, please give PubMed ID, if applicable, in format PMID:#######): Variants in the gene ATP6AP2 have been associated with a multitude of diseases. These disease assertions include X-linked syndromic ID Hedera type, X-linked parkinsonism-spasticity syndrome, and congenital disorder of glycosylation type 2R. A wide range of phenotypes have been presented in the literature for these genes, including global developmental delay, intellectual disability, progressive neurologic decline, spasticity, seizures, infantile onset of liver failure, recurrent infections, dysmorphic features, and features of parkinsonism (rigidity, resting tremor, bradykinesia). These phenotypes do not appear in all individuals with one of the above disease assertions, but many are overlapping phenotypes.The ID/Autism GCEP would like to suggest the term "ATP6AP2-related disorder" to encompass the phenotypes presented in individuals with variants in ATP6AP2. (PMIDs: 15746149, 23595882, 26467484, 29127204, 30985297, 26376863)
**Parent term (use OLS, or your favorite ontology browser): Syndromic X-Linked Intellectual disability
**Children terms (if applicable): none
**Your nano-attribution (ORCID) or URL for a working group: ClinGen's ID/Autism GCEP

@tibingaman it doesn't look like we have congenital disorder of glycosylation type 2R in Mondo - do you think we should add it? Or is it a synonym for something else? Could you please provide more info?
Thanks!

I added this new term as MONDO_0100146 'ATP6AP2-related disorder'

@nicolevasilevsky Hi Nicole, we do not need to add the term "congenital disorder of glycosylation type 2R". In previous publications, ATP6AP2 has been associated with this disease term, but the ID/Autism GCEP is not interested in curating for this disease name. I see no need to add this in Mondo. Thank you for checking!

This is a record in OMIM: https://www.omim.org/entry/301045
CONGENITAL DISORDER OF GLYCOSYLATION, TYPE IIr

Perfect, thanks @maglott! We have an open ticket to import the latest OMIM terms, so hopefully this will be added soon.
#1735


feat(components): use nextui for amount input

Switching amount input as well.
For some reason the test before didn't fail even though we try to send 200 with a balance of 0.
With this it started to fail (which is good I guess). Took me hours to notice I need to adjust the props now :D

Can you change the target branch to master? Started from the bottom and now this merge request closed automatically after merging the other.


Add experimental WebGL Renderer

~This PR is not ready to be merged yet, I will extend the description once it is. Consider this a sneak peek into my work in progress for now~
~For anyone feeling adventurous until then: The renderer should be working roughly now, with bold, italics and underline missing, and no performance optimizations or nice refactorings made so far.~
This PR adds an experimental new renderer as an alternative to the current canvas-based one. I am doing this with the goal of solving several issues that we currently have with the optimizations done in the canvas renderer. First and foremost #1252 and #1083 should be fixed.
This PR intentionally does not cover all the functionality that we have in the canvas renderer yet, I am planning to add all the missing pieces with several smaller PRs afterwards. See the comments below for details in case you want to jump on the train!

@Cryza , this is awesome! I just had a go with it and it's really great work.
I didn't realize how far along this was based on your description... When I ran it, I thought it would be obvious that it was different, but it looked basically the same on my machine (with no cut-off characters, though!) I had to open the debugger to make sure I was actually running the new code. 😄
Not only that, the performance was really impressive in my quick testing... I did some simple benchmarks of just navigating to the top/bottom of a file (triggering a full redraw).
For comparison, this is the old timing with the CanvasRenderer:

And this was the new timing with the WebGLRenderer:

Going from 22 seconds to 2 seconds for that benchmark is an incredible performance improvement. 💯
Thank you for sharing it out! Hope you don't mind if I post a couple questions, just so I can learn more how it works. 😄

@bryphe thanks for the great feedback! I'm really happy that you like it :)
So I am planning to do a major refactoring of all the low-level WebGL code so that it is more understandable and encapsulated in a better way. I hope that the code will be more maintainable afterwards also for people who didn't do a deep dive into WebGL.
I also still see a few quirks and performance improvements we might want to do and possibly also an adaptation of the INeovimRenderer, but I think that most of them should come in a separate PR afterwards, just like the rendering of bold, italic and underlined characters (which might also not be too trivial as it requires enhancements to the atlas mapping).
What I would like to finish before we merge this PR:

Refactor WebGL code
Hide the WebGL renderer behind an experimental configuration flag
Fix vertical alignment of the text with the cursor (currently off by ~1px)
Include line padding (currently ignored, but still yields the same line height AFAIK? :thinking: )
Maybe even fix the vertical antialiasing issue from #2047 and bring the fix back to the CanvasRenderer

So I am planning to do a major refactoring of all the low-level WebGL code so that it is more understandable and encapsulated in a better way. I hope that the code will be more maintainable afterwards also for people who didn't do a deep dive into WebGL.

That sounds great! I like the idea of refactoring to the high-level constructs (will help me out too, I spend some time reading through WebGL 2 docs, but still not proficient yet)

I also still see a few quirks and performance improvements we might want to do and possibly also an adaptation of the INeovimRenderer, but I think that most of them should come in a separate PR afterwards, just like the rendering of bold, italic and underlined characters (which might also not be too trivial as it requires enhancements to the atlas mapping).

Your plan sounds perfect to me. Adding the experimental configuration flag sounds great - perhaps we could have something like editor.renderer with 'canvas' | 'webgl' as values. For now, 'canvas' will still be the default, but users can experiment with 'webgl' - and then eventaully once the remaining work comes in, we'll flip the default. Need to get some champagne ready for that.... that'll be a big day 😀 🎉

Include line padding (currently ignored, but still yields the same line height AFAIK? 🤔 )

Interesting - we actually do something kind of crazy and incorporate the line padding into the fontHeightInPixels here in NeovimInstance.ts:
    public setFont(fontFamily: string, fontSize: string, linePadding: number): void {
        this._fontFamily = fontFamily
        this._fontSize = fontSize

        const { width, height, isBoldAvailable, isItalicAvailable } = measureFont(
            this._fontFamily,
            this._fontSize,
        )

        this._fontWidthInPixels = width
        this._fontHeightInPixels = height + linePadding

This is confusing because the fontHeightInPixels that is everywhere isn't actually the measured pixel height of the font... its the measured pixel height of the font plus the line padding. This could potentially be causing problems.
Of course, keep me posted if there is anything I can do 👍 Awesome work @Cryza !

@Cryza thanks for the shout-out. I’d definitely appreciate a ping on PRs related to this code path so I can learn from your progress. Good luck.

Thanks for your help @nathansobo - look forward to seeing where xray goes! 👍 It's been exciting the progress.

If there is an experimental flag for this, I would suggest merging it into master as soon as the refactoring is complete. I am very eager to try this out!

I don't know if its related but after this change shows this messages when using webgl:
Received notification for - Unhandled Exception:Error: Texture is too small
Please report this error. Callstack: Error: Texture is too small
    at Wi._rasterizeGlyph (file:///C:/dev/oni/lib/browser/vendor.bundle.js:964:15106)
    at Wi.getGlyph (file:///C:/dev/oni/lib/browser/vendor.bundle.js:964:14430)
    at Xi.populateGlyphInstances (file:///C:/dev/oni/lib/browser/vendor.bundle.js:964:19764)
    at Xi.draw (file:///C:/dev/oni/lib/browser/vendor.bundle.js:964:17781)
    at Ji._draw (file:///C:/dev/oni/lib/browser/vendor.bundle.js:964:22957)
    at Ji.redrawAll (file:///C:/dev/oni/lib/browser/vendor.bundle.js:964:21619)
    at br._onResize (file:///C:/dev/oni/lib/browser/vendor.bundle.js:791:3136)
    at ResizeObserver._boundOnResizeMethod._resizeObserver.window.ResizeObserver.e (file:///C:/dev/oni/lib/browser/vendor.bundle.js:791:2653)
Uncaught Error: Texture is too small
    at Wi._rasterizeGlyph (vendor.bundle.js:964)
    at Wi.getGlyph (vendor.bundle.js:964)
    at Xi.populateGlyphInstances (vendor.bundle.js:964)
    at Xi.draw (vendor.bundle.js:964)
    at Ji._draw (vendor.bundle.js:964)
    at Ji.redrawAll (vendor.bundle.js:964)
    at br._onResize (vendor.bundle.js:791)
    at ResizeObserver._boundOnResizeMethod._resizeObserver.window.ResizeObserver.e (vendor.bundle.js:791)

@adelarsq thanks for the report! This is caused by the webgl renderer as you assumed. It currently happens if the texture that contains all the prepared characters runs out of space. Currently we hard-coded the texture size to 512x512 pixels, so for files with lots of different characters, this limit will at some point cause the renderer to throw. Especially so for large font sizes.
My current idea is to implement some logic that catches the error at the level of the text renderer, creates a larger texture and restarts rendering.
To find a reasonable default size in any case, it would be great if you could share some information about your font size, font family, line padding, if you have a retina display (or some other setup with devicePixelRatio > 1) and ideally also the content of the file, if that is possible.
Cheers!

I also hit a similar issue when doing lots of scrolling very rapidly.
I'm using Fira Code as my font at 12px, with the default line padding and a 4k monitor (27" if that matters?) at 150% scaling, with Windows 10 64bit.
The file was my Oni config, which can be found here: https://github.com/CrossR/dotfiles/blob/master/oni/config.tsx

oni 0.3.4, macOS 10.13.4
Setting "editor.renderer": "webgl" gives me this error immediately on startup:
bundle.js:1 Received notification for - Unhandled Exception:TypeError: Cannot read property 'canvas' of null
Please report this error. Callstack: TypeError: Cannot read property 'canvas' of null
    at Ji._updateCanvasDimensions (file:///Users/justin/opt/Oni.app/Contents/Resources/app/lib/browser/vendor.bundle.js:964:21733)
    at Ji.redrawAll (file:///Users/justin/opt/Oni.app/Contents/Resources/app/lib/browser/vendor.bundle.js:964:21520)
    at eo._render (file:///Users/justin/opt/Oni.app/Contents/Resources/app/lib/browser/vendor.bundle.js:964:44256)
    at _pendingAnimationFrame.window.requestAnimationFrame (file:///Users/justin/opt/Oni.app/Contents/Resources/app/lib/browser/vendor.bundle.js:964:44126)
a @ bundle.js:1
vendor.bundle.js:964 Uncaught TypeError: Cannot read property 'canvas' of null
    at Ji._updateCanvasDimensions (vendor.bundle.js:964)
    at Ji.redrawAll (vendor.bundle.js:964)
    at eo._render (vendor.bundle.js:964)
    at _pendingAnimationFrame.window.requestAnimationFrame (vendor.bundle.js:964)

@bryphe the error above occurs after the change https://github.com/onivim/oni/pull/2183 I think

@justinmk Was it after the change https://github.com/onivim/oni/pull/2183?

@justinmk thanks for the report! That happens due to the fact that sometimes the drawing call can be triggered before the renderer was initialized. I will try to fix that in my current branch where I am improving the stability of the WebGL renderer. This is the branch: https://github.com/onivim/oni/tree/cryza/bugfix/webgl-renderer-out-of-texture-space

Great to hear, @badosu! Thanks for giving it another try this quickly :)


Improve copy/paste experience of codeblocks

It would probably be nice to be able to have a copy/paste functionality for code blocks.
Thought prompted by #154 as it's a PR to improve the copy/paste experience of a particular codeblock.

Looks like #140 resolved this?


Not able to receive the credential offer in the wallet using credential offer URL

Describe the bug
Not able to receive the credential offer in the wallet using credential offer URL
To Reproduce
Steps to reproduce the behavior:

Created DID
Created Credential offer using OIDC4VC flow
Opened the wallet and pasted the offer URL
The credential with accept or decline doesn't appear. Rather a blank screen

Expected behavior
Credential with accept/ decline should appear
Actual behavior
Blank Screen with credential offer URL
openid-credential-offer://host.docker.internal:7002/?credential_offer_uri=http%3A%2F%2Fhost.docker.internal%3A7002%2Fopenid4vc%2FcredentialOffer%3Fid%3Dcfa522f0-add5-49bc-b033-b8425d9e66f4
Environment

os version: [e.g. linux, windows]
java version [e.g. 21]

Screenshots
Additional context
Add any other context about the problem here.

Hi there,
We use GitHub issues as a place to track bugs and other development-related issues.
Please see the link below to our dedicated support line:
Help Center :  Click Here
Ticket ID: WB240
Note: Click on the live chat icon at the bottom corner of the page to start a conversation.

hi, looks like you're running issuer-api with dockercould you provide more details about your setup, e.g. how do you run the wallet-api?also, it would help to see if there's any response details in the browser's network tab

Try to add this to your hosts file (/etc/hosts or C:\Windows\System32\drivers\etc\hosts):
<IP_ADDRESS>	   host.docker.internal

If it doesn't solve your issue, please, provide us with:

OS being used
The version you are using
The command you are using to start the Issuer and Wallet services

thanks,
Ale


set hard limit on resourceName for AKS managed cluster.

Changes

parameterized ResourceName, which is aks cluster name.

Set hard limit to 1-63
unified variable name, since we use managedClusterName in some path.

I do like managedClusterName, but we need unify all method to have same variable name.

parameterized resourceGroupName.

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[x] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java

Can one of the admins verify this patch?

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/6270
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/6270/commits/3f6d33c6df5521bd997d317c958e5d0d8f0d1cc0

@shahabhijeet the net sdk errors might be because there are no changes generated

This is a breaking change. This must go in a new api-version. Having this hard limit will now cause customers functional code to stop working.

Reading @haitch reply to another comment, it seems like this validation was present on service side even earlier. It is been added to swagger now. So even before it would have failed on the service side.
In that case, this should be good to merge. Signing off from ARM side.


Room inventory desc is now updated after drop

Codecov Report

Merging #66 into develop will decrease coverage by 0.03%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop      #66      +/-   ##
===========================================
- Coverage    98.32%   98.29%   -0.04%
===========================================
  Files            6        6
  Lines          658      644      -14
  Branches       103      100       -3
===========================================
- Hits           647      633      -14
  Misses           7        7
  Partials         4        4

Impacted Files
Coverage Δ

dork/types.py
98.04% <100%> (-0.05%)
:arrow_down:

dork/repl.py
100% <0%> (ø)
:arrow_up:

dork/game_utils/game_data.py
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


No packages loaded for Verdaccio 5.1.0 hosted on IIS over HTTPS

Describe the bug
After upgrade from verdaccio 4.x.x to 5.1.0 webpage hosted on IIS is not showing packages. It work when approaching over HTTP. But if user goes to HTTPS version, verdaccio shows empty repository with information how to add packages.
When verdaccio is accessed via https://npm.company.org it tries to get api requests over HTTP (instead of HTTPS as in previous version). I.E. http://npm.company.org/-/verdaccio/packages. The request is stopped beause CSP no mixed content is allowed and no data is returned.
In same configuration in 4.x.x it worked beause it sent api request via HTTPS. Configuration is same, only npm install<EMAIL_ADDRESS>was called to upgrade.
To Reproduce
[1] Configure app as in documentation (https://verdaccio.org/docs/en/iss-server
[2] In IIS Manager create New site for npm repo with http binding to port 80 and https binding to port 443
[3] start the application
Update: tested also with only one binding over HTTPS to 443 (no http support). Website still does not run. First request to https://localhost succeeds - it returns html for verdaccio page which initiates requests for runtime.xxxxx.js, vendors.xxxxxx.js and main.xxxxxx.js but all of them over HTTP and none of them is resolved by IIS since HTTP binding does not exists.
Expected behavior
When oppening page over HTTPS protocol it should do HTTPS request to verdaccio API
Screenshots
config.yaml
storage: d:\Web\Verdaccio\storage
plugins: ./plugins
url_prefix: /

auth:
  htpasswd:
    file: ./htpasswd
    max_users: -1

uplinks:
  npmjs:
    url: https://registry.npmjs.org/
    cache: false
    cache_path: ./cache

start.js
process.argv.push('-l', 'unix:' + process.env.PORT, '-c', './config.yaml');
require('./node_modules/verdaccio/build/lib/cli.js');

Environment information
Tested on Windows Server with latest IIS, also in latest updated Win10 with local IIS.
Tested in latest Chrome. Firefox has same CSP problem but it allows request to continue, thus Firefox shows packages even on HTTPS but it has same as chrome in console.
I tried to adjust IIS CSP header, upgrade insecure request and allow connect-src and script-src to HTTP but it didnt help.

Maybe https://verdaccio.org/blog/2021/04/14/verdaccio-5-migration-guide#after-upgrade-i-dont-see-packages-on-the-ui

Thx, but not the case (i just tried). In production environment we have storage on shared folder with full UNC path. Problem is really in diffrerent HTTP protocol. It should access api via HTTPS when browsing HTTPS page.

Maybe https://verdaccio.org/blog/2021/04/14/verdaccio-5-migration-guide#after-upgrade-i-dont-see-packages-on-the-ui

Thx, but not the case (i just tried). In production environment we have storage on shared folder with full UNC path. Problem is really in diffrerent HTTP protocol. It should access api via HTTPS when browsing HTTPS page.

For me is really hard to test IIS Manager since I don't have it, maybe someone else is able to help here.

For me is really hard to test IIS Manager since I don't have it, maybe someone else is able to help here.

Its quite simple if you have windows 10 pro or higher. IIS can be added as a feature.
Start searching "Turn windows features on and off" and tick "Internet information services". (https://www.lansweeper.com/knowledgebase/how-to-install-iis/)


APP-3283: Implemented datafeed id reuse

Ticket
APP-3283
Description
Implementing reuse and persistence of datafeed id when using DFv1
Checklist

[x] Referenced a ticket in the PR title and in the corresponding section
[x] Filled properly the description and dependencies, if any
[ ] Unit tests updated or added
[ ] Docstrings added or updated
[ ] Updated the documentation in docs folder

Was not able to test end to end on devx1, devx3 or eis-dev10

Was able to test it on devx1 by sending a message OBO.


[Federation] Rethinking the securing the token from the client

We need to rethink how we should define the safe way secure then access token.

This is not working and getting too complete to resolve, better to start from fresh state.


Replace "Features" Dialog Box with Dedicated Web Page

To improve the user experience, we should replace the dialog box currently used for the Features section with a dedicated web page. This will allow for a more detailed and visually appealing content presentation.

@AkshitGarg24
Once the issue #270 gets sorted this will be assigned to you


Bug fix: Reset Route arguments for each call (on 4.x)

Forward #2431 to 4.x

Merged. Thank you for your contribution @mathmarques


Revert "Run scancode utility from the utility repo/project"

Reverts apache/incubator-openwhisk#2325
using the Github button 🤣

Why there are two Travis things running ?

The button created a branch WTF?
https://github.com/apache/incubator-openwhisk/tree/revert-2325-scanutil

I re-open #2348 just in case. if #2348 travis is ok then


Compilation error of fu2::unique_function having a capture [ std::vector< std::vector < std::unique_ptr > > ]

@Naios
The code attached does not compile. This is probably because of the two dimensional vector of uncopyable objects (unique_ptrs)
By uncommenting the dummy variable at the line 31 in the capture the code compiles and runs as expected

Commit Hash
Latest commit e30dbbe on 13 Apr
Expected Behavior
To be compilable, and functioning
Actual Behavior
It doesn't compile, error message:
D:\mingw-w64\x86_64-7.2.0-posix-seh-rt_v5-rev0\mingw64\lib\gcc\x86_64-w64-mingw32\7.2.0\include\c++\bits\stl_construct.h|75|error: use of deleted function 'std::unique_ptr<_Tp, _Dp>::unique_ptr(const std::unique_ptr<_Tp, _Dp>&) [with _Tp = int; _Dp = std::default_delete]'|
Steps to Reproduce
Please use the example code given in the attachment
main.zip
Your Environment

OS: { Windows } same behavior on Linux
Compiler and version: { MinGW w64, GCC version 7.2, but the same behavior on Linux
Standard library - default

Thanks for reporting this issue.
I could narrow the test case down to:
static fu2::unique_function<void()> issue_14_create() {
  // remove the commented dummy capture to be compilable
  fu2::unique_function<void()> func =
      [
        i = std::vector<std::vector<std::unique_ptr<int>>>{}
        // ,dummy = std::unique_ptr<int>()
      ]() {
        // ...
      };

  return std::move(func);
}

// https://github.com/Naios/function2/issues/14
TEST(regression_tests, issue_14) {
  issue_14_create()();
}

I'll investigate into this.

I discovered that is not necessary a double dimension vector

static fu2::unique_function<void()> issue_14_create() {
  // remove the commented dummy capture to be compilable
  fu2::unique_function<void()> func =
      [
        i = std::vector<std::unique_ptr<int>>{}
        // ,dummy = std::unique_ptr<int>()
      ]() {
        // ...
      };

  return std::move(func);
}

I found a workaround by encapsulating the vector inside a struct

struct vector_wrapper
{
    std::vector<std::unique_ptr<int>> vec;
    vector_wrapper() = default;
    vector_wrapper(vector_wrapper&& v):
        vec(std::move(v.vec))
        {}
};

static fu2::unique_function<void()> issue_14_create() {
  // remove the commented dummy capture to be compilable
  fu2::unique_function<void()> func =
      [
        i = vector_wrapper{}
      ]() {
        // ...
      };

  return std::move(func);
}

Thanks for your bug report.
It was fixed due to version v3.1.0.


Alternative to superstruct Describe

In Superstruct the Describe utility returns a type representing a struct for a given valid value type. This allows you to ensure you're writing your struct definitions properly.
type User = {
  id: number
  name: string
}

const User: Describe<User> = object({
  id: string(), // This mistake will fail to pass type checking!
  name: string(),
})

Is there a way to do something like this in zod?

@Djaler
If you already have a defined type before specifying the schema, you can use that type as the generic argument to z.object like this:
TypeScript Playground
import { z } from "zod";

type User = {
  id: number
  name: string
}

const User = z.object<User>({ // You get a type error here
  id: z.string(),
  name: z.string(),
})

Hmm, I tried to change id to z.number(), but I still get an error.
Playground

@Djaler ahh, you're totally right, it is expecting a ZodShape not a final type. I usually go the other way and infer the type from the schema, so I'm a bit rusty on this. I was pretty sure there was a way to do this, so lemme take another swing at it.

@Djaler Figured it out!
TypeScript Playground
import { z } from "zod";

type User = {
  id: number
  name: string
}

const User: z.ZodSchema<User> = z.object({
  id: z.string(),
  name: z.string(),
})

It's ZodSchema! h/t 🎩 to https://github.com/colinhacks/zod/issues/372#issuecomment-810905204

Thanks, that's it.
I need this because sometimes types are provided from a library and I want to write a schema for it.

Can you help me with one case?
https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBAbzgLzgXzgMyhEcBEyEAJvgNwBQFwAdjAKZSYCGAxvXABLADOM0ATwA8AFQB8iCnDj06UAQC44IymiqYArjVYxgEGnAAWvfvIDymUTIAeDGsR4oAdAC0SAZVaH6IZkOY0AmJiABSyMPJKIgCUSsiuHl4+ftx8gkLxtJiMosESCFJwUPQwGlAG8RAARgBW9DohBdLS4ZEycgIANIVo0RRoQA
Similar, but working case - https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBAbzgLzgXzgMyhEcBEyEAJvgNwBQFwAdjAKZSYCGAxvXAArMDm9APABUAfIgpw49OlACeALjiCA2gF1KaKpgCuNVjGAQacABbAAzjGgyA8piGSAHgxrEzKAHQAtEgGVWx+hBmfmYaGWFhAAopGFkFQQBKBWQvX39A4O4+fhTaTEYhCNEEcTgoehgtKCMUiAAjACt6PUiSiQkYuI9mKChmGWjpGQSAGlK0BIo0IA

Hmm, nothing comes immediately to mind here. Dynamic schema creation is definitely a hard thing to get working in my experience, so maybe someone else has some good ideas here?

@colinhacks can you help, please?

This is the recommended way to write a schema that implements a particular static type, try it instead:
https://github.com/colinhacks/zod/issues/372#issuecomment-826380330

Sorry, I tried, but still don't understand how to do this


address some lints

i'll hold off on further updates to this PR until #240 is merged

Codecov Report
Base: 64.08% // Head: 73.46% // Increases project coverage by +9.38% :tada:

Coverage data is based on head (3393490) compared to base (bfa307c).
Patch coverage: 69.73% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #239      +/-   ##
==========================================
+ Coverage   64.08%   73.46%   +9.38%
==========================================
  Files          82       94      +12
  Lines        6782    18310   +11528
==========================================
+ Hits         4346    13451    +9105
- Misses       2436     4859    +2423

Impacted Files
Coverage Δ

src/messages/submessages/data_frag.rs
0.00% <0.00%> (ø)

src/messages/submessages/submessage_header.rs
100.00% <ø> (+17.39%)
:arrow_up:

src/serialization/cdr_deserializer.rs
82.38% <ø> (+16.22%)
:arrow_up:

src/serialization/cdr_serializer.rs
87.68% <ø> (+6.07%)
:arrow_up:

src/serialization/message.rs
70.37% <ø> (+14.75%)
:arrow_up:

...sages/submessages/submessage_elements/parameter.rs
67.22% <33.33%> (+18.74%)
:arrow_up:

src/messages/submessages/submessage.rs
60.71% <46.15%> (+4.19%)
:arrow_up:

src/dds/reader.rs
46.12% <50.00%> (-5.19%)
:arrow_down:

src/network/udp_listener.rs
86.02% <76.47%> (+1.13%)
:arrow_up:

src/dds/message_receiver.rs
77.09% <100.00%> (+6.74%)
:arrow_up:

... and 95 more

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

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


Windows download is in .tgz format not .exe

Thought I'd comment on this as a common/nontechnical windows user may run into issues:

Workaround I utilized:
Ran this command in powershell to convert the linux tgz to exe:
tar -xvzf C:\Users\USERNAMEHERE\Downloads\log4j-sniffer-1.0.0-windows-amd64.tgz -C C:\Users\USERNAMEHERE\Downloads
Then ran this (the .\ at the beginning to allow running in powershell, it's designed for the command prompt):
.\log4j-sniffer.exe crawl C:\

Hey @TWalstrom, thanks for catching this! Looks like we changed from exe --> tgz between releases 0.8.0 and 1.0.0. I've updated the readme to reflect the correct steps.

Awesome, glad I could assist (even if it's something small).
Keep up the amazing work to you and everyone at Palantir!
Cheers 🥂


[sw] Require mere 2-byte alignment of entry point

Risc-V instructions can either be 2 bytes or 4 bytes, and can be aligned on 2-bytes boundaries.  It seems that Clang will align functions to 2 bytes.
Is there any reason to require 4-byte alignment of the entry point? Adhering to such restriction will likely require manual linker script tuning by owners.  I propose allowing any 2-byte aligned address to serve as the entry point.

I don't recall the exact rationale, but I believe we specifically chose a 4-byte alignment for the entry point.
The choice may have either been related to hardening or to the way we handle code fetches out of flash.

I don't recall the exact rationale, but I believe we specifically chose a 4-byte alignment for the entry point.
The choice may have either been related to hardening or to the way we handle code fetches out of flash.

It seems to be related to the ePMP: #7025 (comment)

Code segment start and end must by 4-byte aligned, in order for the ePMP to be able to be configured to allow executing that range, and nothing more.  However, I do not see how ePMP restrictions would affect the alignment of the entry point within that aligned range.   The linked comment explicitly mention start and end only, and not the entry point.  I suspect that the same alignment requirements were applied to the entry point without much thought.  (In fact, I do not see any security issue with ROM_EXT allowing odd addresses for the entry point, at worst, it will trigger immediate trap due to unaligned read, but we might as well report a nice error code.)

While I still think that requiring 2-byte alignment is sufficient, I also do not want to change something that works, and adhering to 4-byte alignment is not a huge burden on owner code.


Use Bytes Trie to track warm addresses

Move from a java HashSet to a custom Trie based on bytes to store the warm addresses, creates, and self-destructs.
This avoids needing to calculate java hashes or engage in using custom Comparators.


Add warnings about NFS

Signed-off-by: Julien Pivotto<EMAIL_ADDRESS>

Also see related https://github.com/prometheus/docs/pull/1706

I have tested an EC2 instance with EFS and even when you mount -t efs it is using nfs, so we see NFS_SUPER_MAGIC fs_type.

:+1:

@brian-brazil I have moved to switch{} for better readability in main.go

:+1:


Table lookup with select networks

This adds an operation Select in a new namespace Microsoft.Quantum.TableLookup. It implements a table lookup based on a recursive implementation of the select network algorithm (see references inside the doc string for more details).
The behavior of this operation is similar to MultiplexOperations, with the following differences:

It uses an optimized implementation
It has a name more commonly used in the quantum algorithms community
It is less general as it writes bit strings instead of allowing an arbitrary set of unitary operations

NOTE: Some clippy checks are failing in code that is not changed by this PR. I did not fix these, since they are not related to this PR.

We want to keep the core library built into the product small and focused where possible (back-compat concerns notwithstanding). Especially as this adds a new namespace as well, we'd want to have clear justification, discussion, and agreement that this is broadly useful and durable 'core functionality' before adding. Let's set up a call to discuss. I'll ping you.


How to adjust the resolution of video call based on the internet speed?

Hi,
Currently, bitrate and framerate of particular resolution are reduced If internet quality is bad. My question is. If the speed is suddenly reduced to lower bandwidth. How to adjust the resolution on run time based on internet speed so my video call runs smoothly on all network condition. Please provide a solution.
Note: If this future is available I would like to purchase this agora plugin.
Thanks.

you can refer to these API.
enableDualStreamMode
setRemoteSubscribeFallbackOption

Enabled enableDualStreamMode API but still video call quality is not good.

have you set RemoteSubscribeFallbackOption?

by the way, could you pls provide your VideoEncoderConfiguration?

I'm not using VideoEncoderConfig. I'm just using the default configuration
call.zip
This is my code.

I cant see setRemoteSubscribeFallbackOption and enableDualStreamMode from your code

No improvements in video quality after adding,That's why I removed the lines

@LichKing-2234 May I know the recommended resolution for the video call?

https://docs.agora.io/en/Interactive Broadcast/video_profile_android?platform=Android
https://docs.agora.io/en/Interactive Broadcast/multi_user_video_android?platform=Android

Okay thanks for info

@suriyaKalidoss123 could you pls provide a recording? also dual stream is intended to choose a lower resolution/fps video to avoid network congestion, so if you use it and pick the small stream it will look slightly lagged.
if you can provide a recording for like 1 min, together with the channel name and time you tested. i can do a quick check from our backend to see what could go wrong

I will close this issue because it is too old. if there are still problems, you can reopen this issue.


fix: correct parsing of address, phone, and website in _parse_data_with_soup

Previously, the _parse_data_with_soup method only returned the title and failed to correctly parse and return the address, phone number, and website URL.

Hello, thanks. Did you test the script with your changes?

I have tested the input and output of the _parse_data_with_soup function. The only modification made to the output was the removal of the hyphen (-) for parsed data that previously displayed a hyphen when certain information was missing. This change ensures a cleaner output format for cases where data is not available.
Please let me know if you need any further information or if there are any additional tests you would like me to perform.

@leonardobits thanks a lot for your contribution, I have merged your changes


Ship names and class, notifications settings, guns and quest names and descriptions won't translate in english language.

The settings is in the english language, but there some chinese in some places.

I fixed the problem.

Do you remember how you fixed the problem?

I put here: C:\Program Files (x86)\ and they back to the normal.

it probably the same case as :
https://github.com/Yuubari/KanColleViewer/blob/master/FAQ.md#i-pinned-kcv-to-my-taskbar-and-translations-arent-working
in windows 8, this issue affecting pinned taskbar, pinned start menu, and sometimes desktop shortcut as well.