GraphQL query works under debug mode, but not works under release mode

ATTENTION: DO NOT USE THIS FIELD TO ASK SUPPORT QUESTIONS. USE THE PLATFORM CHANNELS FOR THIS.
Fill in the template. Issues that do not respect the model will be closed.
Describe the bug
A clear and concise description of what the bug is.
Reproduction code
NOTE: THIS IS MANDATORY, IF YOUR ISSUE DOES NOT CONTAIN IT, IT WILL BE CLOSED PRELIMINARY)
example:
import 'package:flutter/material.dart';

import 'package:get/get.dart';

void main() {
  runApp(
    GetMaterialApp(
      title: "Application",
      initialRoute: AppPages.INITIAL,
      getPages: AppPages.routes,
    ),
  );
}

abstract class Routes {
  Routes._();
  static const HOME = _Paths.HOME;
}

abstract class _Paths {
  _Paths._();
  static const HOME = '/home';
}

class AppPages {
  AppPages._();

  static const INITIAL = Routes.HOME;

  static final routes = [
    GetPage(
      name: _Paths.HOME,
      page: () => HomeView(),
      binding: HomeBinding(),
    ),
  ];
}

class HomeBinding extends Bindings {
  @override
  void dependencies() {
    Get.lazyPut<HomeController>(
      () => HomeController(),
    );
    Get.lazyPut<CountriesProvider>(
      () => CountriesProvider(),
    );
  }
}

class HomeView extends GetView<HomeController> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
        appBar: AppBar(
          title: Text('HomeView'),
          centerTitle: true,
        ),
        body: Column(
          children: [
            ElevatedButton(
                onPressed: () {
                  controller.loadRemoteData();
                },
                child: Text('load data')),
            Obx(
              (() => Center(
                    child: Text('${controller.responseData}'),
                  )),
            ),
          ],
        ));
  }
}

class HomeController extends GetxController {
  final CountriesProvider _settingProvider = Get.find();
  final responseData = ''.obs;

  loadRemoteData() async {
    var res = await _settingProvider.fetchData();
    responseData.value = res.isOk ? res.body.toString() : '';
    debugPrint(responseData.toString());
  }
}

class CountriesProvider extends GetConnect {
  @override
  void onInit() {
    httpClient.baseUrl = 'https://countries.trevorblades.com/';
  }

  Future<GraphQLResponse<Map>> fetchData() {
    String queryString = r"""
      query test($cty: ID!){
        country(code: $cty) {
          name
          native
          capital
          emoji
          currency
          languages {
            code
            name
          }
        }
      }
      """;

    return query<Map>(queryString, url: "", variables: {"cty": "BR"});
  }
}

To Reproduce
Steps to reproduce the behavior:

flutter run under debug mode, click 'load data' button on screen, it can load graphql data from https://countries.trevorblades.com/
flutter run --release under release mode it does not work.

Excaption show as below:
Launching lib/main.dart on MI 6 in release mode...
Running Gradle task 'assembleRelease'...                           33.3s
✓  Built build/app/outputs/flutter-apk/app-release.apk (6.4MB).
Installing build/app/outputs/flutter-apk/app.apk...                 3.0s

Flutter run key commands.
h List all available interactive commands.
c Clear the screen
q Quit (terminate the application on the device).
E/flutter (23504): [ERROR:flutter/lib/ui/ui_dart_state.cc(209)] Unhandled Exception: NoSuchMethodError: The method '[]' was called on null.
E/flutter (23504): Receiver: null
E/flutter (23504): Tried calling: []("errors")
E/flutter (23504): #0      Object.noSuchMethod (dart:core-patch/object_patch.dart:38)
E/flutter (23504): #1      _objectNoSuchMethod (dart:core-patch/object_patch.dart:85)
E/flutter (23504): #2      GetConnect.query (package:get/get_connect/connect.dart:316)
E/flutter (23504): <asynchronous suspension>
E/flutter (23504): #3      HomeController.loadRemoteData (package:getx_graphql_bug/main.dart:81)
E/flutter (23504): <asynchronous suspension>
E/flutter (23504):

Environment:
get: 4.6.1
get-cli: 1.7.1
[✓] Flutter (Channel stable, 2.10.3, on Ubuntu 20.04.4 LTS 5.13.0-37-generic, locale en_US.UTF-8)

I find anthor issue here #1093 , not sure if they are related.

sorry, I forgot to include the following permissions in AndroidManifest.xml
<uses-permission android:name="android.permission.INTERNET" />
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />


Add an open_at column to quiz_grade_exception

Right now, if a user has a quiz_grade_exception, they can take the quiz up to the closed_at time. I'd like to create an open_at field so that I can constrain when the user begins the quiz...e.g. I don't want them to do it too early.
I would rarely need such a thing. But, it seems logical.

skipping this term due to time


Blocking throughout pageserver in async contexts

During #2875 I hit an interesting deadlock, which is reproduced at this branch. I managed to find multiple "waker woken, but deadlock". The most interesting was:
A thread of task_mgr::BACKGROUND_RUNTIME was executing periodic compaction from pageserver::tenant_tasks while the same thread had previously executed the async walredo task. The thread became deadlocked on blocking flume::Receiver::recv because the async walredo task was blocked on the same runtime worker thread, on its LIFO slot. The async walredo task would have been the one to process the Timeline::get eventually.
I think the repro commit does have other deadlocks, but I failed to really understand them. Originally I suspected a thread being blocked on locking Timeline::layer_removal_cs have the async walredo task on it's LIFO slot, but haven't seen it now in rr at least.
More reproduction info

branch has the NEON_LOCAL_RECORD_COMMANDS="pageserver" support if you want to try your hand at rr (closed #2958)
used dev builds, cargo build_testing
./scripts/pytest --capture=no test_runner/regress/test_branch_and_gc.py::test_branch_and_gc

I think this was already fixed but on the branch compaction runs nonstop

I don't think this is an issue of tokio, though they are working on making this LIFO slot stealable in the future. I think it's the issue of blocking the codebase. More precisely I think it's the many paths that are called from async context which forbid using async, like in the deadlock-repro branch.
Straight forward solution to this is to never block on tokio runtime threads. That is however difficult to implement and will likely come with some performance loss at least with the current tokio version we use. A "test case" I've used in the past to keep the blocking in check is to run the whole app within one current_thread runtime instead of multiple runtimes with multiple threads. We should be able to do this with the regression tests. I think it should be a long-term goal.
During #2875 I tried sprinkling tokio::task::block_in_place and tokio::task::spawn_blocking around. The use of tokio::task::block_in_place has the downside of making it impossible to aim for the current_thread flavored executor. Eventually tests passed, but it's difficult to estimate how complete a solution this is. In the course of the sprinkling, I used a very rough tool to help guide the process, but it's a prolonged process, didn't complete it.

The tokio::task::block_in_place is basically not a solution or alternative because of https://github.com/tokio-rs/tokio/issues/5239 for spawn_blocking, at least until we get capabilities but the proper support for those might come in sooner.
Still I think there's benefit to consider a single runtime with very few threads (1-2) and using for example 8 threads for blocking pool. We most likely don't need that much async code going on in the first place. Blocking pool will be utilized regardless with block_in_place so it might be used explicitly.

This week:

Async write operations
Avoid using spawn_blocking(block_on) because of the overhead of churning io_uring runtimes (while keeping it for prod until we cut over io_uring)

A lot has happened since opening this issue.
Closing it as duplicate in favor of the two currently known remaining sources of executor stalling:

https://github.com/neondatabase/neon/issues/4744
https://github.com/neondatabase/neon/issues/6628


Pocketstefan patch 4

Travis is failing because we need to un-include the xhr-semaphore No files found matching /home/travis/build/alphagov/digitalmarketplace-buyer-frontend/app/assets/javascripts/../../../node_modules/digitalmarketplace-frontend-toolkit/toolkit/javascripts/xhr-semaphore.js


Add HiScore lookup to right click menus

Adds the "lookup" option to the following right click menus:

Chatbox
Friends list
Private messages
Clan chat (sidebar)
Raiding party (People with kick-rank only, it doesn't have a right-click menu for anyone without rank)

Currently this doesn't have a toggle, and I don't think it should have one but this can be discussed and would be easy to add anyway.

Duplicate of #348


Disregard invoice created_at and presentation fields

Disregard invoice created_at field -- Failing to disregard this field on update causes the following error: Error: CANNOT UPDATE INVOICE: Invalid node: created_at.
Disregard presentation field -- this field is returned the the invoice.get method, but causes the API to 500 if included as part of an invoice update

Um... hello? Can we get this merged? Ran into this issue again on a new install


change match entityType to rcContact

Coverage remained the same at 39.649% when pulling 369399c19f1298c4f3bd40cd329dd2d556e3932a on embbnux:enhancement-contacts into da557abd16625a1bf6fd904264f70f5584e9d3a8 on ringcentral:master.


Rotation + translation CatSim validation/verification

As presented during the second DC2 telecon (Jan 19th) CatSim adds a translation and rotation to the extragalactic catalogs. I was thinking about checking the 2 point shape-shape, shape-counts and counts-counts correlations (already in DESCQA I think) and compare with the correlations computed for the extragalactic catalogs but, are there any other tests that you want to see/do to make sure that this doesn't affect your science/this is done correctly?
@LSSTDESC/wg-conveners @rmandelb @katrinheitmann @drphilmarshall

Yes. It can be closed.


feat(ui): display top 4 org cards by challenge count

Fixes #1533
Changelog

Replace mock organizations with real ones
Display top 4 instead of top 3 (sort by challenge count)
Add "See More" button that will redirect users to the Search Organizations page

Preview

@tschaffter thanks!  I'll implement changes similar to your feedback in its child PR.


Binary File Decompiler To CPG

currently joern support ghidra2cpg, with use ghidra to load binary to cpg.
but it seems only use the assembly instruction, why not use the decompiler infomation?
  Call(
    argumentIndex = -1,
    argumentName = None,
    code = "SUB RSP,0x30",
    columnNumber = None,
    dispatchType = "STATIC_DISPATCH",
    dynamicTypeHintFullName = IndexedSeq(),
    lineNumber = Some(value = 1053724),
    methodFullName = "<operator>.subtraction",
    name = "<operator>.subtraction",
    order = 0,
    possibleTypes = IndexedSeq(),
    signature = "",
    typeFullName = "<empty>"
  ),
  Call(
    argumentIndex = -1,
    argumentName = None,
    code = "MOV dword ptr [RBP + -0x14],EDI",
    columnNumber = None,
    dispatchType = "STATIC_DISPATCH",
    dynamicTypeHintFullName = IndexedSeq(),
    lineNumber = Some(value = 1053728),
    methodFullName = "<operator>.assignment",
    name = "<operator>.assignment",
    order = 0,
    possibleTypes = IndexedSeq(),
    signature = "",
    typeFullName = "<empty>"
  ),

Thank you very much, I understand what you mean, if you use the decompiled output c file as the input of c2cpg, there may be many errors.
My idea is to build CPG directly using the decompiler's AST, so the result should be more accurate

https://github.com/hac425xxx/joern-ida


hello world

Request type

[x] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[ ] Correction or update

Details

``


Running mitmdump on Windows from the Path requires file extension

I have been working on user installation documentation in regards to #516 and I'm recommending that when users install Python they "Add Python.exe to Path" which adds "C:\Python27" and "C:\Python27\Scripts" to %PATH%.  Then the user installs mitmproxy using "pip install mitmproxy".  Then when you try to run "mitmdump", it doesn't find it.  Turns out this is due to mitmdump being installed to "C:\Python27\Scripts\mitmdump" without a ".py" file extension.
The weird thing is that when I pull from master, and run setup.py manually mitmdump(and mitmweb) gets properly turned into an exe.  I looked through setup.py and there isn't anything obvious to me that is doing this.  Was this fixed on master and we are just waiting for 0.11.4 to be released and to be loaded onto PyPI or is pip introducing something into the mix?

Hi @elitest,
the exe is generated using setuptools entry_points, introduced in 355f9fc40799c88f413ca260dd93691074fd758e. 0.11.4 should just generate executables :smiley:
Cheers,
Max


refactor: migrate pokedex to jina 5

Resolved: #188

Need to set logserver to False in flow-index.
But I do not know why docidx does not seem to receive any message from the gateway? Although u can try with images from the real pokemon examples that I did not find

@JamesTang-jinaai I did a push to ur branch, pull locally and test please

Now the docker file woks.


Removing more references to v1beta1 from pkg/

For https://github.com/GoogleCloudPlatform/kubernetes/issues/6584

@bgrant0607

Updated code as per comments

Thanks. LGTM, but tests failed due to the name change:
_output/local/go/src/github.com/GoogleCloudPlatform/kubernetes/test/integration/auth_test.go:82: undefined: testapi.ResourcePathWithQueryParams
_output/local/go/src/github.com/GoogleCloudPlatform/kubernetes/test/integration/auth_test.go:94: undefined: testapi.ResourcePathWithQueryParams
FAIL	github.com/GoogleCloudPlatform/kubernetes/test/integration [build failed]

Fixed it. Travis is green now.

LGTM


Update scalacheck to 1.15.1

Updates org.scalacheck:scalacheck from 1.14.3 to 1.15.1.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalacheck", artifactId = "scalacheck" } ]

labels: library-update, semver-minor

Codecov Report

Merging #1569 into master will decrease coverage by 0.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1569      +/-   ##
==========================================
- Coverage   85.58%   85.40%   -0.19%
==========================================
  Files          58       58
  Lines        2206     2206
  Branches      107      107
==========================================
- Hits         1888     1884       -4
- Misses        318      322       +4

Impacted Files
Coverage Δ

...rc/main/scala/io/circe/shapes/SizedInstances.scala
78.57% <0.00%> (-14.29%)
:arrow_down:

...re/shared/src/main/scala/io/circe/JsonNumber.scala
91.56% <0.00%> (-2.41%)
:arrow_down:

.../core/shared/src/main/scala/io/circe/Decoder.scala
85.32% <0.00%> (-0.28%)
:arrow_down:

...les/core/shared/src/main/scala/io/circe/Json.scala
80.44% <0.00%> (ø)

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
84.79% <0.00%> (+0.58%)
: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 75e9f33...1361672. Read the comment docs.

this PR is incomplete — it should also update this line:
"org.scalatestplus" %%% "scalacheck-1-14" % "<IP_ADDRESS>" % Test

from -1-14 to -1-15


error handling not working on android

i implemented the plugin in my new app en works really good.
one problem, i can not get the error handling to work. When i enter .onerror = function(event) the
event.error variable is undefined. When reading the event variable, it only contain '"type":"error"'
Is this part not implemented yet?
How can i check the reason for error (network,no-speech,aborted,...) ?
thank you
David

Sorry, not implemented yet.

thank you for the reply.
any plans on implementing the error handling on android in near future?

I'm open to pull requests :)
Actually, I'm hoping to refresh this entire plugin in the not to distant future. Keep adding issues.

It isn't implemented on iOS neither

my problem is black screen this video https://www.youtube.com/watch?v=52lr37t1JIw


[WIP] Beginner clue hint locations

Does everything but hot/cold. The locations are in a new file, but I need to edit a lot of code around so it will check the correct list (beginner or other locations).

Also doesn't have any map clues

Isnt this already implemented?


Collect Heap-pools memory metrics

Currently we only collect Heap/non-heap. Collect data for the different pools so that it can be visualized through dashboards

Hi @eyalkoren , here we need only heap-pool memory metrics?


Update Documentation!

I added a "What is Proton Designer" to the Documentation (we can eventually shift it to the main page and not in the docs, this is just temporary) and I deleted the "test1" file.

Please fix the merge conflics

@TechStudent10 - I am unable to resolve conflicts. Even GitHub is saying that the conflicts are too complex to resolve in the web editor. Please advice/help.

@TechStudent10 - I am unable to resolve conflicts. Even GitHub is saying that the conflicts are too complex to resolve in the web editor. Please advice/help.

Don't use the web editor?

@TechStudent10 - I am unable to resolve conflicts. Even GitHub is saying that the conflicts are too complex to resolve in the web editor. Please advice/help.

Don't use the web editor?

Yes, I used VS Code, and fixed the issue but when I committed it, the issue didn't get resolved, so idk. It's the blogs that you added that messing this up.
Should I make a new PR?

Bro, will do it. Reopen the PR.

It is saying that the conflicts are too complex to resolve in the web
editor, and then usually it gives a command line method. But, it isn't
giving one.
Can you merge it via the command line please?
Thanks,
Kendall Does Coding
Portfolio https://kendalldoescoding.gq
Github https://github.com/KendallDoesCoding
Youtube https://kendalldoescoding.gq/codingYT
On Mon, 13 Mar 2023 at 17:09, TechStudent10 @.***>
wrote:

Reopened #2
https://github.com/ProtonDesigner/ProtonDesigner.github.io/pull/2.
—
Reply to this email directly, view it on GitHub
https://github.com/ProtonDesigner/ProtonDesigner.github.io/pull/2#event-8730693388,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGJUTJOH7F26EW2H7UR2RTW34BQ3ANCNFSM6AAAAAAVEOUQBM
.
You are receiving this because you authored the thread.Message ID:
<ProtonDesigner/ProtonDesigner.github.io/pull/2/issue_event/8730693388@
github.com>

It is saying that the conflicts are too complex to resolve in the web editor, and then usually it gives a command line method. But, it isn't giving one. Can you merge it via the command line please?

I'm sorry, what? You have to fix the merge conflicts or else everything else breaks. Stop using the web editor if the conflicts are too complex for it, or else I will have to close the PR.

Please merge #10


question on installation and update

Hello
this is a great enhancement of v2board.
there are some questions:
if the user uses v2board guide to install the panel, do we need to check availability of mysql in init.sh?
this panel docks to xrayr under the name of Newv2board?
i could not find update.sh in project, does update.sh update panel directly from this repo?
will you continue to maintain this project?

do we need to check availability of mysql in init.sh?: The answer is no, we do not need to check


Feature Request: allow configuration of more preferences such as default editor to suggest opening files in

Would be nice to be able to configure some standard preferences, such as:
- change default editor to vscode instead of vim
- be able to let it know which shell i'm using (e.g. zsh)
- let it know which OS i'm running,

I would like to expand on the topic with something I came to desire more than once, which is allowing for a variable baseurl.


Change BindToFiles to a property on HttpRequest called Files

Can't be done as can't have extension properties yet and HttpRequest is abstract and DefaultHttpRequest is the actual implementation


Multilang: th translation update

Update Thai translations

Btw if you want to get this language in release, look into fixing the Thai fonts in AGNOS. Or let me know if they work already

Btw if you want to get this language in release, look into fixing the Thai fonts in AGNOS. Or let me know if they work already

It seems fine to me. Is there anything specific I should look for?

just checked, it looks good to me now! when we first added thai, the font was blocky. I guess it was fixed in an AGNOS update since then

https://github.com/commaai/openpilot/pull/29447


Add built-in events to website documentation

Summary
Add built-in events to website documentation, to make it clear about events such as onclick on a Button for example.
💬 Description
Navigating to Button for example, going to the "Code" tab and looking at events. As a consumer, I'd expect to see all events I can use, such as onclick. This ticket covers looking into if it is feasible and applying all missing events on all components.
💰 Use value
Make it easier for developers to see how to use the components.

After further investigation with this ticket additional information is needed, thus I am adding this comment below to help anyone who looks into this ticket again.
Focus on the following...
Explore what the ICDS should be doing about built in events (e.g. OnClick, change if relevant, etc). StencilJS just passes on anything built in like OnClick to the component. For instance, if you look at Button, you can see no event for “click” or “onclick” or anything similar. That’s because Stencil is passing on those DOM events, see more here.
Therefore the issue is, as a consumer, there are no code examples or events documented to show how to add 'on click' to a button, which may be obvious to some, but maybe not new developers or those using ICDS for the first time.
Thus, it’d be worth exploring how we make it obvious e.g. is it code examples:

is it somehow getting the more obvious events (such as OnClick for button) in the docs somehow
is it worth just linking from all the events to a page that shows what built in events may work?

You should also consider looking at other libraries.

Material UI has a section on their Button docs for Handling Clicks that explains that all their components accept the onClick handler. Below that they also state that:

Note that the documentation avoids mentioning native props (there are a lot) in the API section of the components.

I look at Vuetify as another example, and they don't explicitly mention the @click event at all.
My recommendation would be that we add a "Native properties and events" page to the documentation that explains how our components allow native web component / react events. We could then link to that from relevant component pages (e.g. Button)
Thoughts?


Many pages are broken on WinHTTP docs

Link to article:
Just to name a few:
https://docs.microsoft.com/en-us/windows/win32/api/winhttp/ns-winhttp-winhttp_autoproxy_options
https://docs.microsoft.com/en-us/windows/win32/api/winhttp/ns-winhttp-winhttp_proxy_info
https://docs.microsoft.com/en-us/windows/desktop/api/winhttp/ns-winhttp-winhttp_current_user_ie_proxy_config
Problem:
A bunch of pages are broken in the WinHTTP documentation. They seems to be concentrated on documentation for WinHTTP structures.
Will this be fixed soon? It's pretty annoying to have to look at a Google cached page - which eventually will not work either.

@doodleincode Your links aren't broken.

@doodleincode Your links aren't broken.

Thanks.


Add info about the Download Secure File task

This page seems to act as a general overview of Secure Files, but does not provide any information about how to consume the secure files outside of integrating the functionality in a custom task. I've included a link to the Download Secure File Utility task in order to provide this information.

@alexkokkinos Thank you for your contribution, I am merging this now and it should be live soon. Thank you.


cookies not working

idk if its just for me but my cookies aint workin when i put them in

Will be fixed shortly.

Should be good now, sorry for the long wait.


[rocksdb] Link errors on x64-windows

Describe the bug
As of 4c397835c472e1a49b47ca5172c71fa56e497796, rocksdb no longer exports the RocksDB::rocksdb target on x64-windows, instead exporting the RocksDB::rocksdb-shared target. The problem with the shared library is that it only seems to expose the C API, causing linker errors if you use the C++ API.
Environment

OS: Windows
Compiler: MSVC 19.37.32820.0 (Visual Studio 17.7.0 Preview 4.0)

To Reproduce
Steps to reproduce the behavior:

./vcpkg install rocksdb:x64-windows
Compile any program that uses the RocksDB C++ API

Expected behavior
The project compiles without linker errors.
Failure logs
[main] Building folder: test
[build] Starting build
[proc] Executing command: "C:\Program Files\CMake\bin\cmake.EXE" --build d:/test/build --config Debug --target ALL_BUILD -j 18 --
[build] MSBuild version 17.7.1+971bf70db for .NET Framework
[build]
[build]   Checking Build System
[build]   Building Custom Rule D:/test/CMakeLists.txt
[build]   main.cpp
[build] main.obj : error LNK2019: unresolved external symbol "public: __cdecl rocksdb::ColumnFamilyOptions::ColumnFamilyOptions(void)" (??0ColumnFamilyOptions@rocksdb@@QEAA@XZ) referenced in function "public: __cdecl rocksdb::Options::Options(void)" (??0Options@rocksdb@@QEAA@XZ) [D:\test\build\test.vcxproj]
[build] main.obj : error LNK2019: unresolved external symbol "public: __cdecl rocksdb::DBOptions::DBOptions(void)" (??0DBOptions@rocksdb@@QEAA@XZ) referenced in function "public: __cdecl rocksdb::Options::Options(void)" (??0Options@rocksdb@@QEAA@XZ) [D:\test\build\test.vcxproj]
[build] main.obj : error LNK2019: unresolved external symbol "public: static class rocksdb::Status __cdecl rocksdb::DB::Open(struct rocksdb::Options const &,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,class rocksdb::DB * *)" (?Open@DB@rocksdb@@SA?AVStatus@2@AEBUOptions@2@AEBV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@PEAPEAV12@@Z) referenced in function main [D:\test\build\test.vcxproj]
[build] D:\test\build\Debug\test.exe : fatal error LNK1120: 3 unresolved externals [D:\test\build\test.vcxproj]
[proc] The command: "C:\Program Files\CMake\bin\cmake.EXE" --build d:/test/build --config Debug --target ALL_BUILD -j 18 -- exited with code: 1
[driver] Build completed: 00:00:01.148
[build] Build finished with exit code 1

Additional context
The error log above comes from the following test program. Changing builtin-baseline to f6a5d4e8eb7476b8d7fc12a56dff300c1c986131 (2023.06.20 release) and linking RocksDB::rocksdb instead of RocksDB::rocksdb-shared will make the project compile without error.
CMakeLists.txt:
set(CMAKE_TOOLCHAIN_FILE "$ENV{VCPKG_ROOT}/scripts/buildsystems/vcpkg.cmake")

cmake_minimum_required(VERSION 3.12)
project(test)

find_package(RocksDB CONFIG REQUIRED)

add_executable(test main.cpp)
target_compile_features(test PRIVATE cxx_std_17)
target_link_libraries(test PRIVATE RocksDB::rocksdb-shared)

main.cpp
#include <rocksdb/db.h>

int main() {
	rocksdb::DB* db;
	rocksdb::DB::Open(rocksdb::Options{}, "./database", &db);
}

vcpkg.json
{
	"name": "test",
	"version-semver": "0.1.0",
	"dependencies": [
		"rocksdb"
	],
	"builtin-baseline": "4c397835c472e1a49b47ca5172c71fa56e497796"
}

This issue could be reproduced with the latest vcpkg 2023-06-22-f19f3d9939100085fe2f7e41502d60b08e2083b6.

@cbrl @Cheney-W
Support for RocksDB shared libraries was introduced in https://github.com/microsoft/vcpkg/commit/4c397835c472e1a49b47ca5172c71fa56e497796. Prior to this change, static library export was forced for all triplets. The RocksDB port now matches the value defined by the VCPKG_LIBRARY_LINKAGE variable.
The target RocksDB::rocksdb is still exported on x64-windows-static. You can use x64-windows-static if you need a static library.
VCPKG supports per-port customization if you need to mix different communication types for ports: https://learn.microsoft.com/en-us/vcpkg/users/triplets#per-port-customization.

I would normally agree that it makes sense to export the shared library in this case, but the shared library doesn't expose the main interface to RocksDB. I would almost say it's a different library at that point, like msgpack vs msgpack-c. This could probably be considered a defect with RocksDB itself though.
In any case, if this is the desired behavior, it might be helpful to add a note in the usage file that mentions how most users will need to override the linkage setting in order to use the library.

Even though, the library name has changed to RockSD-shared, which should not be needed and brings in complication.
.

The library name rocksdb-shared is set by the RocksDB project itself. It is a needless complication, but it's actually the correct name for vcpkg to use.
The main issue is that the shared library doesn't seem usable for most people. Vcpkg used to only compile the static library because of this (discussed in #2411 and implemented in #7452). It's very unexpected to be required to override linkage settings in order to use the main API of a port.
Depending on the official stance for this issue, I think the behavior should either be reverted, or a notice should be added to the usage file that warns users about this.

Which library name implements the C++ API for static linkage?

Which library name implements the C++ API for static linkage?

rocksdb is the name of the library when using static linkage ^1. Technically the C++ API exists in the shared library as well, but the shared library only exports a C API implemented in terms of the C++ API, almost as if it were a separate project entirely.


Plotting gdf with single value works with gv.Polygon but not gdf.hvplot()

import rasterio
import hvplot.pandas
import geopandas as gpd
from rasterio.features import shapes

mask = image == 1

image = np.array([
    [0, 0, 1, 0, 0],
    [0, 1, 1, 1, 0],
    [1, 1, 1, 1, 1],
    [0, 1, 1, 1, 0],
    [0, 0, 1, 0, 0]
]).astype(np.uint8)

results = (
    {'properties': {'raster_val': v}, 'geometry': s}
    for i, (s, v) in enumerate(
    shapes(image, mask=mask))
)

geoms = list(results)
gdf = gpd.GeoDataFrame.from_features(geoms)
gdf

# expected; works
gv.Polygons(gdf).options(tools=['hover'])

# doesn't work
gdf.hvplot()

if I add another value, hvplot works
import rasterio
import hvplot.pandas
import geopandas as gpd
from rasterio.features import shapes

mask = image == 1

image = np.array([
    [0, 0, 1, 0, 0],
    [0, 1, 1, 1, 0],
    [1, 1, 2, 1, 1],  # 2 added
    [0, 1, 1, 1, 0],
    [0, 0, 1, 0, 0]
]).astype(np.uint8)

results = (
    {'properties': {'raster_val': v}, 'geometry': s}
    for i, (s, v) in enumerate(
    shapes(image, mask=mask))
)

geoms = list(results)
gdf = gpd.GeoDataFrame.from_features(geoms)
gdf

# expected; works
gv.Polygons(gdf).options(tools=['hover'])

# doesn't work
gdf.hvplot()

Appears the difference here is merely the use of gv.Polygons vs. hv.Polygons, gdf.hvplot(geo=True) does work.

The issue here is the colorbar, it seems bokeh does not like a colorbar with zero range.


Creating multiple Wallet Deposit Endpoints at the same time fails when Wallet does not exist

Describe the Bug
Concurrent call to create a Wallet Deposit Endpoint (or a Wallet for that sake) fails. The reason is that a Wallet will be automatically created if it does not exist and there is a uniqueness constraint on the owner.
The code will try to create multiple wallets with the same owner, but only 1 will succeed and the other will fail. The code then throws an exception for the failing calls.
To Reproduce
1.
2.
3.
...

Expected Behavior
Does not fail
Screenshots
No response
Desktop
...
Additional Context
No response

fixed by #237


Txt file error

Showing error while selecting mode as Text file input after uploading the file .

Hii there, thanks for reporting this issue also 😅, actually it's a known bug and I am trying to fix it as soon as possible.

Hii there, I have fixed this error. Once again thanks for reporting.

yeah I checked the issue it's working smoothly now .


linux dpkg-deb: speed up build

Description of the change
This PR speeds up the build process of dpkg-deb on the Linux machines (by about 3 min):

It uses sudo when available instead of fakeroot (in CI or when user has started in root) -> 2 min speed up
It does not compress the artifacts internally unless we are on a release/nightly branch. -> 1 min speed up

Verification
Before changes (16 min and 38 seconds): https://dev.azure.com/atomcommunity/atomcommunity/_build/results?buildId=878&view=logs&j=0da5d1d9-276d-5173-c4c4-9d4d4ed14fdb&t=cccdf131-0893-5d7b-ccea-9ccfc8953f95
After changes(13min and 18seconds): https://dev.azure.com/atomcommunity/atomcommunity/_build/results?buildId=879&view=logs&j=0da5d1d9-276d-5173-c4c4-9d4d4ed14fdb&t=cccdf131-0893-5d7b-ccea-9ccfc8953f95
Release Notes

Faster builds by dpkg-deb on Linux

The current code is more complex because it uses fakeroot. I changed fakeroot to sudo in a backward-compatible manner. I am happy to drop fakeroot if that is the "complexity" you are talking about.
I have included a benchmark in the OP, but I can add more runs.

The current code is more complex because it uses fakeroot. I changed fakeroot to sudo in a backward-compatible manner. I am happy to drop fakeroot if that is the "complexity" you are talking about.
I have included a benchmark in the OP, but I can add more runs.

@DeeDeeG Since you are good at doing benchmarks, it would be nice if you can run a local benchmark for me. The online benchmarks show 3 minutes improvement.

@DeeDeeG Since you are good at doing benchmarks, it would be nice if you can run a local benchmark for me. The online benchmarks show 3 minutes improvement.

@aminya this PR might be more important if we were going to run the tests on Linux. Otherwise I think we're fine, as Linux is already either the fastest CI job, or close behind the macOS build job.

@aminya this PR might be more important if we were going to run the tests on Linux. Otherwise I think we're fine, as Linux is already either the fastest CI job, or close behind the macOS build job.

I don't really care about CI. I want to speed up the build for the users who want to build Atom locally.
https://github.com/atom/atom/issues/21760

If such a feature gets merged, I would like to see compression toggle-able with the --compress-artifacts flag.
That way it's off by default, but can be easily enabled for CI, or for anyone else who wants to host releases of Atom. (Or for anyone who just wants to save space on their hard-drive).
We should also add a "no compression" option to the .rpm package build if doing it for the .debs.

Actually, about my above comment... --compress-artifacts is already used on Linux for another purpose. I suppose a flag for this feature would need a new name, something like --compress-linux-packages.


Simplify overly complex boolean expressions

Example
Replacement

condition ? true : false
condition

condition ? false : true
!condition

value == null ? null : value
value

result != 0 ? result : 0
result

a == b ? a : b
b

Codecov Report

Merging #472 into dev will increase coverage by 0.02%.
The diff coverage is 50%.

@@             Coverage Diff              @@
##                dev     #472      +/-   ##
============================================
+ Coverage     46.26%   46.29%   +0.02%
- Complexity     2201     2204       +3
============================================
  Files           108      108
  Lines         25227    25227
  Branches       4172     4170       -2
============================================
+ Hits          11672    11679       +7
+ Misses        11538    11526      -12
- Partials       2017     2022       +5

Flag
Coverage Δ
Complexity Δ

#JDBC41
46.09% <50%> (+0.02%)
2194 <0> (+2)
:arrow_up:

#JDBC42
46.1% <50%> (-0.01%)
2195 <0> (-2)

Impacted Files
Coverage Δ
Complexity Δ

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
53.78% <ø> (+0.43%)
0 <0> (ø)
:arrow_down:

...n/java/com/microsoft/sqlserver/jdbc/Parameter.java
62.23% <ø> (-0.43%)
63 <0> (-1)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
52.58% <0%> (-0.07%)
240 <0> (+1)

.../microsoft/sqlserver/jdbc/SQLServerConnection.java
45.8% <0%> (ø)
274 <0> (ø)
:arrow_down:

...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java
63.24% <100%> (ø)
0 <0> (ø)
:arrow_down:

src/main/java/com/microsoft/sqlserver/jdbc/AE.java
73.95% <100%> (ø)
0 <0> (ø)
:arrow_down:

...m/microsoft/sqlserver/jdbc/SQLServerDataTable.java
58.06% <100%> (+0.8%)
23 <0> (ø)
:arrow_down:

...om/microsoft/sqlserver/jdbc/ReaderInputStream.java
42.69% <0%> (-4.5%)
15% <0%> (-1%)

...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java
44.71% <0%> (-0.23%)
105% <0%> (+1%)

... and 4 more

Continue to review full report at Codecov.

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

Thanks for your contributions @JamieMagee. We'll review and try to merge your PRs in the next preview release.


OBS Streaming Problem with Encoder NVENC HEVC

Which version are you using?
v0.20.2
Which operating system are you using?

[ ] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[x] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
server: i/o timeout
when i try to use obs streaming with encoder NVENC HEVC(H265)
Describe how to replicate the issue

start the server
publish with OBS: settings: Encoder NVENC HEVC

Did you attach the server logs?

yes
2022/12/08 05:26:53 INF rtsp-simple-server v0.20.2
2022/12/08 05:26:53 DEB path manager created
2022/12/08 05:26:53 INF [RTSP] listener opened on :8554 (TCP), :8000 (UDP/RTP), :8001 (UDP/RTCP)
2022/12/08 05:26:53 INF [RTMP] listener opened on :1935
2022/12/08 05:26:53 INF [HLS] listener opened on :8888
2022/12/08 05:27:01 INF [RTMP] [conn PRIVACY_HIDDEN:61864] opened
2022/12/08 05:27:01 DEB [path live] created
2022/12/08 05:27:11 DEB [path live] destroyed (not in use)
2022/12/08 05:27:11 INF [RTMP] [conn PRIVACY_HIDDEN:61864] closed (read tcp PRIVACY_HIDDEN:1935->PRIVACY_HIDDEN:61864: i/o timeout)
2022/12/08 05:27:13 INF [RTMP] [conn PRIVACY_HIDDEN:61878] opened
2022/12/08 05:27:13 DEB [path live] created
2022/12/08 05:27:16 DEB [path live] destroyed (not in use)
2022/12/08 05:27:16 INF [RTMP] [conn PRIVACY_HIDDEN:61878] closed (EOF)

Did you attach a network dump?

no

This is now supported since v0.21.1


Issue: Invalid  Usage with  Child

Description
I encountered an error related to the usage of the  component in my Next.js project. The error message is as follows:

Problem
The error occurs when using the  component with an  tag as a child. According to the latest Next.js documentation, the  component now internally handles the  tag, and using an  tag inside  is no longer supported.
Solution
To resolve this issue, the  tag should be removed from inside the  component. The  component should directly handle the styling and attributes.
Impact
This change is necessary to adhere to the latest Next.js guidelines, which streamline the link handling process and prevent issues related to duplicate elements and styling conflicts.

can i attempt

npx @next/codemod new-link
use this command for fixing this issue

@isid555
Remove anchor tag from this link componant

@Vegadjay
My commit is working ,but it's not merged with the actual repo yet

@isid555 Great, I think it's unsolved.

Hey @Vegadjay,
Summary of Changes

Fixed the usage of <Link> with <a> as a child in the [file/path].
Improved the code by [mention any specific changes].

Please review, and let me know if further adjustments are needed.

Hey @Vegadjay , I’m currently working on additional fixes that depend on this PR being merged. It would be great if you could review and merge it when you have a moment. I believe it will streamline the ongoing work. Thanks for your attention to this!

@Tanushmaurya018 Is your pr merged ?

@Vegadjay nope

@Tanushmaurya018 Why this repo is not merge any one pr ???


Physiopy-documentation

Title
Physiopy-documentation: Writing and organising documentation for physiological data handling
Leaders
Sarah Goodale (@s_goodale23)
Inês Esteves (@isesteves)
Collaborators
No response
Brainhack Global 2022 Event
Brainhack Physiopy
Project Description
Physiopy is a community formed around developing tools to operate physiological files in neuroimaging setups. This project specifically will expand our package documentation containing tips and strategies on how to collect and analyze various forms of physiological data and the use of our packages. Contributors may get started from any skill set and level, as this project will predominantly be focused on documentation contributions. Most of all, we are looking for people excited to learn (and contribute to) the best practices in physiological signal acquisition.
Link to project repository/sources
https://github.com/physiopy/physiopy.github.io
Goals for Brainhack Global
We aim to provide a succinct overview of what physiological data are typically recorded during an fMRI experiment, how these signals are recorded, and how these signals can improve our modeling of fMRI time series data. This is an active field of research, and we hope to encourage all users to get the latest recommendations prior to initiating a new study.
Good first issues

Improve the documentation for Physiopy’s packages
Build best practices libraries and summaries for various physiological signals (e.g., respiratory, cardiac, skin conductance, etc.)
Search for additional open source libraries and software similar to what we do
Find links to educational courses of interest
Enlarge our list of open source datasets with physiological signals
Improve the website’s interface

Communication channels
https://physiopy.slack.com/archives/C042U2KLF36
Skills
GitHub: basic
Markdown: basic to advanced
Physiological data acquisition/analysis: basic to advanced
Onboarding documentation
https://docs.github.com/en/get-started
https://www.markdownguide.org/
What will participants learn?
Participants will learn (at least) the basics about different types of physiological data acquired in neuroimaging setups and how they can be used. In addition, they will get acquainted with Github and Markdown, which may be helpful in other contexts.
Data to use
No response
Number of collaborators
more
Credit to collaborators
Project contributors are listed on our Team page.
Image

Type
documentation
Development status
1_basic structure
Topic
MR_methodologies, physiology
Tools
other
Programming language
documentation
Modalities
fMRI, MRI
Git skills
0_no_git_skills, 1_commit_push, 2_branches_PRs
Anything else?
No response
Things to do after the project is submitted and ready to review.

[ ] Add a comment below the main post of your issue saying: Hi @brainhackorg/project-monitors my project is ready!
[ ] Twitter-sized summary of your project pitch.

@Remi-Gau I don't know if we can do this, but this was also Brainhack Geneva - can we add both labels?

Done but I have NO idea if our website if smart enough to filter for more than one thing: to check.
And defo the jupyter book at the moment only consider that projects are specific to a single event (but I opened an issue for that).

Sounds good Remi, thank you!
In that case, if @isesteves agrees, @brainhackorg/project-monitors our project is ready!

Yes, thank you both!

Apparently the filters on the project page are able to handle more than one site label perpeojwcr.


Loc Automation - json update on handback for Branch: jsonUpdater

Loc Automation - json update on handback for Branch: jsonUpdater

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

Line 111: [Error] W:\ifgg-s\jobs\100\Pages\Pricing\docfx.json for docset Pricing doesn't exist

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


add usage example for tf.image.rgb_to_grayscale

We cannot accept PRs on release branches. All PRs must be on master branch and then they can be cherry-picked on release branches, if there is a release going on.
Please reopen against master

@WilliamHYZhang , please reopen against master.

@kyscg I made a new pull request with more usage examples #34884 against master.


TLS 1.2 test using howsmyssl.com

Q
A

Bug?
maybe

New Feature?
no

Version
6.3.3

Actual Behavior
What is the actual behavior?
Response contains 'TLS 1.0' for tls_version attribute.
Expected Behavior
What is the behavior you expect?
Response should contain 'TLS 1.2' for tls_version attribute.
Steps to Reproduce
$curl_opt_array = [
    CURLOPT_SSLVERSION => CURL_SSLVERSION_TLSv1_2
];

$client = new GuzzleHttp\Client;

$response = $client->request(
    'GET',
    'https://www.howsmyssl.com/a/check',
    [
        'curl' => $curl_opt_array
    ]
);

die(json_decode($response->getBody())->tls_version);

Thoughts
To me, it looks like TLS 1.2 isn't being enforced by Guzzle. I do not know if I've done something wrong, or if this is some kind of issue with the howsmyssl.com service. If anyone can point out something I've done incorrectly, that would be much appreciated.
Extra version info:

curl 7.54.0
PHP 7.1.5

Added note: When I remove the CURLOPT_SSLVERSION setting, I get TLS 1.2 connections. Why would this option cause it fall back to TLS 1.0?

Running the code above with latest Guzzle results in TLS 1.2.
With debug mode enabled:
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to www.howsmyssl.com (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering http/1.1
* Cipher selection: ALL:!EXPORT:!EXPORT40:!EXPORT56:!aNULL:!LOW:!RC4:@STRENGTH
* successfully set certificate verify locations:
*   CAfile: /etc/ssl/certs/ca-certificates.crt
  CApath: /etc/ssl/certs
* SSL connection using TLSv1.2 / ECDHE-RSA-AES128-GCM-SHA256
* ALPN, server did not agree to a protocol
* Server certificate:
*  subject: CN=www.howsmyssl.com
*  start date: Aug 28 00:10:03 2018 GMT
*  expire date: Nov 26 00:10:03 2018 GMT
*  subjectAltName: host "www.howsmyssl.com" matched cert's "www.howsmyssl.com"
*  issuer: C=US; O=Let's Encrypt; CN=Let's Encrypt Authority X3
*  SSL certificate verify ok.
> GET /a/check HTTP/1.1
Host: www.howsmyssl.com
User-Agent: GuzzleHttp/6.3.3 curl/7.52.1 PHP/7.2.9

< HTTP/1.1 200 OK
< Content-Length: 2904
< Access-Control-Allow-Origin: *
< Connection: close
< Content-Type: application/json
< Date: Fri, 28 Sep 2018 16:13:58 GMT
< Strict-Transport-Security: max-age=631138519; includeSubdomains; preload
<
* Curl_http_done: called premature == 0
* Closing connection 0
TLS 1.2

My modified version of the code:
<?php

require_once __DIR__.'/vendor/autoload.php';

$curl_opt_array = [
    CURLOPT_SSLVERSION => CURL_SSLVERSION_TLSv1_2
];

$client = new GuzzleHttp\Client;

$response = $client->request(
    'GET',
    'https://www.howsmyssl.com/a/check',
    [
        'curl' => $curl_opt_array,
	'debug' => true,
    ]
);

die(json_decode($response->getBody())->tls_version);

docker run --rm -it -v $PWD/test.php:/guzzle/test.php guzzle/guzzle php test.php

Here's my output for the same thing:
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to www.howsmyssl.com (<IP_ADDRESS>) port 443 (#0)
* TLS 1.0 connection using TLS_RSA_WITH_AES_256_CBC_SHA
* Server certificate: www.howsmyssl.com
* Server certificate: Let's Encrypt Authority X3
* Server certificate: DST Root CA X3
> GET /a/check HTTP/1.1
Host: www.howsmyssl.com
User-Agent: GuzzleHttp/6.3.3 curl/7.54.0 PHP/7.2.9

< HTTP/1.1 200 OK
< Content-Length: 1845
< Access-Control-Allow-Origin: *
< Connection: close
< Content-Type: application/json
< Date: Fri, 28 Sep 2018 16:28:23 GMT
< Strict-Transport-Security: max-age=631138519; includeSubdomains; preload
<
* Closing connection 0
TLS 1.0

I see the differences at the top, but I have no idea on the what/why for it. What else could be configured wrong on my machine to cause this?

Can you try to run the script with the docker command to see if it's environmental?

It matched the output from your response. Any thoughts on how to debug this since the debug mode doesn't seem to show what's different? It seems like it's a local configuration issue, but I'm at a loss as to what that could be.


Snackbar + Menu/Dialog Accessibility

We are hitting an screen reader accessibility issue...
Scenario
User is in a dialog or menu > Submit is clicked > Snackbar is presented but skipped by the screen reader and focus is returned to the dialog or menu trigger
More details
Seems like the returned focus to the dialog or menu is causing the role="alert" to be skipped

What is the information that is missing for this ticket to keep moving forward?

@vromaniv-liaison A step by step reproduction. Codesandbox, screen reader version, etc.

I did have examples created it may be on another issue I had opened. Let me see if I can track it down.

More details here… https://github.com/mui/material-ui/issues/30097

Please see my comment in the other issue: https://github.com/mui/material-ui/issues/30097#issuecomment-1361420029


elm-json install --yes ... sporadically gets stuck on macOS 11 and 10.15

In Homebrew, we have seen commands like elm-json install elm/browser@1 --yes sporadically get stuck on macOS 11 (Big Sur) and 10.15 (Catalina). Haven't tried older macOS. So far, we haven't seen it fail on macOS 12 (Monterey) or 13 (Ventura)
Originally this issue was seen while building unisonlang (https://github.com/Homebrew/homebrew-core/pull/105842) which runs elm-json (https://github.com/unisonweb/unison-local-ui/blob/release/M4e/scripts/ui-core.js#L6). At that time, I was able to find the problematic command was elm-json but it stalled without any meaningful output.
 Otherwise, it appears to remain stuck indefinitely. This caused Homebrew CI to block for about 1 day until manually cancelled.
Had tried both pre-built binary fetched via npm and a locally built copy of elm-json.

Seems like isahc is stuck while polling. Probably the same as #54, so I think #55 may fix it too.


Added django service page

Coverage decreased (-0.5%) to 80.866% when pulling 2efbbc14411de6bce66eb7aeb9800d886f902d3e on vineeshan:service into 81ae6d7541cad85237da582652c699e4e5c8030a on MicroPyramid:master.


Update random generation to use C++

Original date: 2018-04-30 18:49:16
Original PR: https://charm.cs.illinois.edu/gerrit/4083

Change-Id: I9e6a3ccd16951763b7aa12a9338141096661a393

Re-rebased from this branch's prior state in my local repo, with your changes re-applied on top. I also got rid of using namespace std; while we're at it.


Feat: define color scheme in toml

I came here to say: please provide a TOML file. It's safer for users and I can then sync it into wezterm for the next release!
See: https://github.com/Hiroya-W/wezterm-sequoia-theme/issues/1
and: https://github.com/EdenEast/nightfox.nvim/issues/183

I came here to say: please provide a TOML file. It's safer for users and I can then sync it into wezterm for the next release!
See: https://github.com/Hiroya-W/wezterm-sequoia-theme/issues/1
and: https://github.com/EdenEast/nightfox.nvim/issues/183

Thank you Wez - will get this done ASAP!

Thanks! BTW, https://github.com/EdenEast/nightfox.nvim/issues/183#issuecomment-1189655306 shows how you can fairly easily export the lua file to toml using a built-in function

Thanks! BTW, https://github.com/EdenEast/nightfox.nvim/issues/183#issuecomment-1189655306 shows how you can fairly easily export the lua file to toml using a built-in function

Amazing 🙌

@wez The current two versions of the theme have now been converted to toml and are located in /dist

Thanks! Sync'd and now visible at:
https://wezfurlong.org/wezterm/colorschemes/p/index.html#poimandres
https://wezfurlong.org/wezterm/colorschemes/p/index.html#poimandres-storm


api: Add a way to play a sound from a coordinate

This is an example implementation of #317 using a method similar to the RGBLike interface used to allow the use of native platform colour objects.
Closes #317.

if we add Vector3dLike, shouldn't we also provide Vector3iLike?

if we add Vector3dLike, shouldn't we also provide Vector3iLike?

It's still x, y, z right? We could generify this into just Vector3Like, that's how Unity names their corresponding object for example.

if we add Vector3dLike, shouldn't we also provide Vector3iLike?

It's still x, y, z right? We could generify this into just Vector3Like, that's how Unity names their corresponding object for example.

No. Different return types.

Different return types.
Ah right. In that case yes I agree.

I've just rebased this with the addition of Vector3iLike and the removal of the implementation for Vector3dLike. However, this does need further consideration as to how we would or if this PR even should handle worlds. Many platforms have objects that represent a location in a given world and this could produce unexpected behavior with code like this (using Paper as an example):
Location location = new Location(Bukkit.getWorld("test"), 0.0, 0.0, 0.0);
Bukkit.getServer().playSound(sound, location); // why can't i do this? or why is it playing for people not in the "test" world?

I would be interested to hear everyone's thoughts on this, but I feel like the best way is what I have just added in this PR which is to add another location-ish object that does not provide any location in order to let platforms implement this as they feel fit. Would be happy to pull this last commit into another PR if necessary.
Additionally, the current implementation of the playSound method that takes a Vector3iLike just turns each integer into a double directly, should this be instead using the center of a location (i.e. adding 0.5 to each integer)?

Been thinking about this one a bit more, and I'm just going to close this PR. I haven't seen any need for something like this in Adventure yet (nor has anyone else expressed an interest). Stateful API (that requires the developer to have information about the platform at hand, such as location, world, etc.) is perhaps just not as useful in Adventure anyway.


"skvideo.io.vread" reports error

Hi Roy,
Thank you for your prompt reply! I really appreciate that you always respond quickly.
I'm collecting CLIP representation of VGGSound videos by running the collect_video_CLIP.py following your instruction. However, for video l6q7shFb8zs_000041 and EktPRpYX9KI_000038, it reports this error:
Traceback (most recent call last):
  File "/home/anaconda3/envs/myenv/lib/python3.8/site-packages/skvideo/io/ffmpeg.py", line 271, in _read_frame_data
    assert len(arr) == framesize
AssertionError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "debug.py", line 194, in <module>
    check_corrupted_video()
  File "debug.py", line 173, in check_corrupted_video
    videodata = skvideo.io.vread(video)
  File "/home/anaconda3/envs/myenv/lib/python3.8/site-packages/skvideo/io/io.py", line 148, in vread
    for idx, frame in enumerate(reader.nextFrame()):
  File "/home/anaconda3/envs/myenv/lib/python3.8/site-packages/skvideo/io/ffmpeg.py", line 297, in nextFrame
    yield self._readFrame()
  File "/home/anaconda3/envs/myenv/lib/python3.8/site-packages/skvideo/io/ffmpeg.py", line 281, in _readFrame
    s = self._read_frame_data()
  File "/home/anaconda3/envs/myenv/lib/python3.8/site-packages/skvideo/io/ffmpeg.py", line 275, in _read_frame_data
    raise RuntimeError("%s" % (err1,))
RuntimeError

I was wondering if this happens to you as well? I obtain 199,176 videos from the VGGSound dataset and only these two cannot be processed.

Sure np and sorry but I didn't

Hi Roy, > > Thanks for your quick reply! I really appreciate that you always reply quickly. > > I collect CLIP representations of VGGSound videos by running collect_video_CLIP.py according to your instructions. However, for the videos l6q7shFb8zs_000041 and EktPRpYX9KI_000038 it reports this error: > > `` > Traceback (most recent call last): > File "/home/anaconda3/envs/myenv/lib/python3.8/site-packages/skvideo/io/ffmpeg.py", line 271, in _read_frame_data > assert len(arr) == framesize > AssertionError > > While handling the above exception, another exception occurred: > > Traceback (most recent call last): > File "debug.py", line 194, in  > check_corrupted_video() > File "debug.py", line 173, in check_corrupted_video > videodata = skvideo.io.vread(video) > file "/home/anaconda3/envs/myenv/lib/python3.8/site-packages/skvideo/io/io.py", line 148, in vread > for idx, frame in enumerate(reader.nextFrame()): > file "/home/anaconda3/envs/myenv/lib/python3.8/site-packages/skvideo/io/ffmpeg.py", line 297, in nextFrame > yield self._readFrame() > file "/home/anaconda3/envs/myenv/lib/python3.8/site-packages/skvideo/io/ffmpeg.py", line 281, in _readFrame > s = self._read_frame_data() > File "/home/anaconda3/envs/myenv/lib/python3.8/site-packages/skvideo/io/ffmpeg.py", line 275, in _read_frame_data > raise RuntimeError("%s" % (err1,)) > RuntimeError > ``` > I was wondering if this happens to you too? I get 199,176 videos from the VGGSound dataset and only these two cannot be processed.

Hi xwhjy, I am wondering how long it takes you to preprocess all the videos in VGGSound? It seems to take a very long time.

@mayqinxu Hi! I cannot remember the time but it did take quite a long time to process all the videos in VGGSound.


Robustness Masterlist

Here's a list of the potential robustness features which will require any new development/debugging, in order to have a full suite of tools to try to eliminate a gamma ceiling and lower the sigma floor.
To be clear, these are mostly ripped straight from this paper and the advice of Vassili Mewes.  In order of application during a step, the options available are:
Reconstruction:

[ ] MPWENO reconstruction scheme from here
[x] Fallback to a TVD reconstruction if reconstructed values would be <0

Conserved variable floors:

[x] Apply a floor rather than fail for rho*u^0 < 0
[x] Compute T^{00}, apply similar floor

UtoP improvements:

[ ] Better primitive variable guess, from here
[ ] Transport entropy and recover guaranteed-positive u or additional variables, from here
[x] Fail to atmosphere when neighbors aren't available (maybe also when 1-3 neighbors are available?)
[x] Kastaun (2021) robust inversion

Post-inversion floors:

[x] Constant floor value for u at large radii
[x] Insert in fluid frame of unmagnetized fluid
[ ] (if necessary) bring back temperature ceiling

Flux corrections:

[x] Use global (ctop=c) Lax-Friedrichs flux when correcting
[ ] Use previously-reconstructed primitive values rather than bumping down to donor cell always

Of note:

[x] SSPRK(5)4 might be worth considering for larger timestep.  Theoretically c=2 for 3x work though?  Parthenon has one, so we can play with this easily.

No single item here is a terribly big lift, though transporting the entropy requires code modifications in a bunch of places, and reconstructing new variables would be awkward to make fast.  I'm mostly taking the low-hanging fruit first (reconstruction fallback, new flux correction & floors options) but I don't think these will be the heavy hitters.  I'm taking a look at Kastaun inversion as well, but we should keep in mind it won't help the (perhaps even more feisty) implicit simulations, where most of the other improvements here will.

Going to close this as fixed by FOFC/Kastaun and the implementations marked above.  To condense if we come back, the options are:

[ ] MPWENO
[ ] Better primitive guess
[ ] Transport & recover with entropy
The latter two aren't so relevant to Kastaun inversion, but might help the implicit inverter to be a little more robust or recover better.


Implement support for setting specific array indices

Implements #4

Also, can you add a test case for this?

Also, can you add a test case for this?


CSS3 var() Please

Fyrd advised me elsewhere that unlinked sub-topic under "variable Fonts (unrelated) was all that caniuse is offering for now.
Please prioritize var() as search-indexed main topic in caniuse. In that respect, I hope caniuse will fix to apparent incompatibility with Safari and Firefox too.
https://www.w3schools.com/cssref/css_functions.asp
https://www.w3schools.com/css/css3_variables.asp
https://developer.mozilla.org/en-US/docs/Web/CSS/Using_CSS_variables
https://caniuse.com/#search=var
[caniuse seems to need some major housekeeping... Hope that goes well :-)]

What are you talking about? If you search for "var", CSS variables is the second result in the list. Tested it here, works fine in both Safari and Firefox.
Also, searching on google works just fine

I believe OP was talking about the case one enters exact var() query:
https://caniuse.com/#search=var()
what, at this point, shows
» $$('h3.feature-title').map(e=>e.innerHTML).join(', ')
« "Variable fonts, CSS @apply rule, URL API, Element.scrollIntoViewIfNeeded(), let, Summary"

i.e. not the CSS Variables (Custom Properties).

Added var() to keywords so now https://caniuse.com/#search=var() gives the expected result.


Fix Micronaut integration in Helidon 3.x?

Environment Details

Helidon Version: 4.x
Helidon SE or Helidon MP
JDK version:
OS:
Docker version (if applicable):

Problem Description
Micronaut has just released 4.0 (11 July 2023) which, among other things, has breaking changes regarding javax vs. jakarta in some areas including validation, as described in this excerpt from Slack:

I'm having issue of imcompatibilty of the jakarta.validation and javax.validation.
The jakarta.validation (I'm refering to the constraints import jakarta.validation.constraints.* in to the model) doesn't work with micronaut @Validated in CrudRepository as it does not trigger the validation due the helidon.integrations.micronaut it only support javax.validation instead (tested it work).
version.lib.micronaut>3.4.3</version.lib.micronaut>,
version.lib.micronaut.data>3.3.0</version.lib.micronaut.data>

However with the use of javax.validation.* it create issue for manual validation, the Validation.buildDefaultValidatorFactory() doest work with javax, which trigger the error.
[helidon-server-1] ERROR xxx.GlobalExceptionHandler - Unable to create a Configuration, because no Jakarta Bean Validation provider could be found. Add a provider like Hibernate Validator (RI) to your classpath.
javax.validation.NoProviderFoundException: Unable to create a Configuration, because no Jakarta Bean Validation provider could be found. Add a provider like Hibernate Validator (RI) to your classpath.

As Micronaut 4.0 contains breaking changes from Micronaut 3.x, we probably would not want to change the Micronaut dependency to 4.0 in a Helidon 3.x dot release (as the original poster in Slack inquired about).
But do we want to consider updating the Micronaut dependency to 4.0 in Helidon 4.x?

These are JAVA JDK java.compiler library, would this impact the integration ? I believe these namespace will not be change for the JAVA platform.

These are JAVA JDK java.compiler library, would this impact the integration ? I believe these namespace will not be change for the JAVA platform.

Correct, jakarta.annotation:jakarta.annotation-api:jar:2.1.1 is not the replacement of javax.annotation.

Hi All,
I would like to know when will this fix to be release ?

Hi All,
Any expected date to be release the Micronut integration ?

Hi All,
Any expected date to be release the Micronut integration fix ?

There is a PR but it is pending to be reviewed.


Create useful locators for page elements in HEDA Settings pages

** lurch: attach W-009769

Included in beta release version 1.9 (Beta 5) and higher

Included in production release version 1.9


Arrow Function improvement and Variable Destructuring

Improvement to parse Arrow Function.

Add typescriptDestructurings that parses Object or Array destructuring.

Referenced by typescriptVariableDeclarations and typescriptParameterList.

Fix typescriptArrowFuncDef so that it can correctly parse Object or Array destructuring.
Fix typescriptFuncType so that it can contained in Return Type of typescriptArrowFuncDef.

This can be parsed correctly.
const func = <
  C extends Readonly<Partial<{ foo: any, bar: any[] }>>
>(
  { foo, bar: [ baz, { 'qu qux': quux } ] }: C
) : <T>(foobar: T) => () => { barbaz: T } => {
  const { a, b: [ c, d, ...e ] } = foo;
  let f, g, h;
  ({ f, x: [ g, ...h ] } = quux);
  return <T>( foobar: T, ...ququx: any[] ): ( ) => { barbaz: T } => ( ) => ( { barbaz: ququx[0] } );
};

TODO

Can not parse Parenthesized Type contained in Return Type of Arrow Func Def.

Ex.
const F = (): ('foo' | 'bar') => { return 'foo' };

Hi @Milly ! Thank you for your tremendous contribution!
I notice this change will be huge. So please kindly be patient, I will take some time to review it!
Also, could you please add some test cases for arrow function? I would be necessary for yats not having highlight broken.
Thanks again for your help!

Sorry existing tests are failed. I will fix it later.
So change this PR to WIP in current.


Rendering is extremely slow

Is your feature request related to a problem? Please describe.
The current solution for 2D and 3D rendering, and how the rendering widgets overall are setup was made to be compatible with an older version of gradio in mind. The old version has severe limitations, which can be observed when changing the 2D slider, which takes several seconds to update, as the 3D renderer needs to be updated every single time. This should not be necessary.
The 2D and 3D renderers should ideally be kept as separate processes, handled in separate threads, if possible. There is also no need to update the 3D renderer if the 2D renderer is updated. In the old gradio version this was not possible to change, but maybe it is now.
Describe the solution you'd like
Changing 2D images should not affect the 3D renderer, and should be close to seemless.

Fixed in 2219207af26398c54b98eb766d586b63f6a9ca4d.
Further improvement can be made, but at least now the 2D and 3D viewers operate independently and thus 2D rendering is much faster.


Android tablet pdfs being cut off

Reported by someone in projectors for sewing and replicated by myself. PDFs are being cut off right of centre on Android tablets. The div is big enough for the PDF but the content is hidden.
https://github.com/Pattern-Projector/pattern-projector/assets/5465100/882b19ea-4d68-48cd-aff8-befeeb18b86f

This is only reproducible on Chrome-based browsers for Android; not in Firefox.

Still an issue when overlapping edges using the stitch menu


Not implemented exception when reindex Immunization resource

I am using microsoft/fhir-server. I've created my SearchParamater for Immunization resource and added it to the search-parameter.json definition file.  Now I tried to reindex the resource using POST to "/Immunization//$reindex" url. There is no body for POST in my request. I am getting the following error
System.NotImplementedException: The method or operation is not implemented.
at Microsoft.Health.Fhir.SqlServer.Features.Storage.SqlServerFhirDataStore.UpdateSearchIndexForResourceAsync(ResourceWrapper resourceWrapper, WeakETag weakETag, CancellationToken cancellationToken)
Is there anything missing? How can I reindex my immunization resource search so that I can search with newly added parameter?

Nil


Update Wrapt for PY 3.11

Wrapt <1.15 doesn't have Py3.11 wheels on pypi

i am curious as to why this was dropped. I would like to see wrapt unpinned.


Group object keys by prototype

TS Template added by @mjbvz
TypeScript Version: 4.1
Search Terms

suggest / suggestion
IntelliSense

Chromium based browsers (here Microsoft Edge) group object keys by prototype inheritance, I'd love to see this in VSCode

We don't track this sort of information at all, so have no way to surface this information. The useful part of this is that the non-Object members are at the bottom, and we already hide those, so I think the current behavior qualifies as "good enough"


Add functionality to assert static SAML attributes per CO

Added functionality to the frontends.saml2.SAMLVirtualCoFrontend class
so that the frontend SAML IdP can be configured to assert a set of SAML
attributes with static values for all users per CO.
All Submissions:

[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[X] Have you added an explanation of what problem you are trying to solve with this PR?
[X] Have you added information on what your changes do and why you chose this as your solution?
[X] Have you written new tests for your changes?
[X] Does your submission pass tests?
[X] This project follows PEP8 style guide.

Thanks @c00kiemon5ter, I fixed the issues and made sure tests again passed.

@c00kiemon5ter This fix works with Python 3.4. I included a comment to refactor when Python 3.4 is no longer supported.


Create Gallery Component

dd-gallery
create a webcomponent that handles the json at the dd-gallery Tag
it should handle Links, previews and column-count

done


"Undefined property" when removing select dropdown option

Laravel Version: 5.4.x
Voyager Version: 0.11.7
PHP Version: 7.1.1
Database Driver & Version: MySQL X

Description:
When you have a BREAD row using the Select Dropdown type, it give problems handling the options when a option is removed.
Steps To Reproduce:
Make BREAD with a Select Dropdown row and the following settings:
{
    "default": "Foo",
    "options": {
        "foo": "Foo",
        "bar": "Bar"
    }
}

Then create a new row with this set as Foo.
Then edit the BREAD settings to:
{
    "default": "Bar",
    "options": {
        "bar": "Bar"
    }
}

Then go back to the row for that BREAD and it will throw the following error:
ErrorException in 1dc0155360549e4cd7aacdfed574eebe6a05f97e.php line 68:
Undefined property: stdClass::$foo (View: /site.dev/vendor/tcg/voyager/resources/views/bread/browse.blade.php)

Solution
My idea for a solution is to show the default option if the option no longer exists.

This happens also then adding more options. It doesn't update correctly in database sometimes.

For me this issue was solved with the exclusion of a null value:-
/vendor/tcg/voyager/resources/views/bread/browse.blade.php
Line 118
@if (!empty($options->options->{$data->{$row->field}}))

       {!! $options->options->{$data->{$row->field}} !!}

@endif

Yes that indeed works. Alternatively we could use {!! $options->options->{$data->{$row->field}} or '' !!}

@dhonions Do you want to submit a PR for this?

@emptynick Yes, please do.

@dhonions Superb work! Issue fixed. PR!

Hi, for me this error was solved with
{!! $options->options->{$data->{$row->field}} or $data->{$row->field} !!}


Infotext cleanup

Some minor changes

Coverage remained the same at 95.42% when pulling 0df6ca5c658c38a989e6b47ce22da92601d1f883 on benjick:master into 44c8ec6057ed08aef8e99cddbe13992a43ac5d51 on xgfe:master.

Thanks. I merged manually because of conflicts.


Use use instead of lots of full paths

Super tiny thing. Found this while reviewing #3479.
Objective

Simplify code
Fix the link in the doc comment

Solution

Import a single item :)

bors r+


MongoDBMethodInterceptor: Casting  to 'ListCollectionsOperation' may produce 'ClassCastException'

Please answer these questions before submitting pull request

Why submit this pull request?

[x] Bug fix

[ ] New feature provided

[ ] Improve performance

Related issues

Bug fix

Bug description.
Casting  to 'ListCollectionsOperation' may produce 'ClassCastException'

Coverage remained the same at 16.847% when pulling 13068bb5d52572cee6042d034abc6f8e87d6cb6c on zxbu:patch-1 into b7ef10a70dfb99e12e74426450eb266ee353cfd8 on apache:master.


Make menu and related classes abstract

This is a breaking change for everything that directly created instances of Menu and/or related classes.

Do you happen to have an osu!-side PR for this?


Link cannot have anything except text inside of it or else it breaks!

My Buffer button on the far right has this link below....

      <svg version="1.1"
         xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:a="http://ns.adobe.com/AdobeSVGViewerExtensions/3.0/"
         x="0px" y="0px" width="65px" height="65px" viewBox="-0.5 -0.5 65 65" enable-background="new -0.5 -0.5 65 65"
         xml:space="preserve">
      <defs>
      </defs>
      <path d="M1.4,16l28.7,12.7c1.1,0.4,2.6,0.3,3.5,0L62.8,16c1.7-0.6,1.7-2.8,0-3.5L33.6,0.3c-1-0.4-2.7-0.4-3.5,0L1.4,12.5
        C-0.5,13.3-0.5,15,1.4,16z M62.8,30l-6.4-2.7l-22.7,9.9c-1.2,0.5-2.4,0.4-3.5,0L7.8,27.3L1.4,30c-1.9,0.8-1.9,2.6,0,3.5l28.7,12.7
        c1.1,0.4,2.3,0.4,3.5,0l29.2-12.7C64.5,32.5,64.5,31,62.8,30z M62.8,47.7L56.4,45l-22.7,9.9c-1.2,0.5-2.4,0.4-3.5,0L7.7,45l-6.3,2.7
        c-1.9,0.8-1.9,2.6,0,3.5l28.7,12.7c1.1,0.4,2.3,0.4,3.5,0l29.2-12.7C64.5,50.3,64.5,48.7,62.8,47.7z"/>
      </svg>

It will not work with this script however a simple link below does work.  THey are the same link but my button above has the button inside the link and that somehow prevents this sscript from working on it.
Any ideas on how to make this work with my link above?
khkhj

\

More testing shows that the Buffer iframe does open but it seems my SVG image is on top of my link which has the data attributes.
I have tried moving the data attributes for Buffer on to my SVG image but that did not work.
If I leave it on my  link it works if I click on the small black area in the image outside of the SVG image area.
My goal is to have this work when I click anywhere on my Buffer button.
Any ideas for a fix?  Thanks

More testing shows that the Buffer iframe does open but it seems my SVG image is on top of my link which has the data attributes.
I have tried moving the data attributes for Buffer on to my SVG image but that did not work.
If I leave it on my  link it works if I click on the small black area in the image outside of the SVG image area.
My goal is to have this work when I click anywhere on my Buffer button.
Any ideas for a fix?  Thanks

I was able to resolve my issue by adding another overlay DIV on top of my Buffer button and I make it the same size with transparent background and then attach the data attributes to that DIV so it remains on top.

I was able to resolve my issue by adding another overlay DIV on top of my Buffer button and I make it the same size with transparent background and then attach the data attributes to that DIV so it remains on top.


Error using native menu: attempt to call method 'redraw' (a nil value)

https://github.com/hrsh7th/nvim-cmp/blob/0f28030aef5b7b15708aaed8ed90b94d206aa712/lua/cmp/view/native_entries_view.lua#L69

Error executing vim.schedule lua callback: ...cker/start/nvim-cmp/lua/cmp/view/native_entries_view.lu
a:69: attempt to call method 'redraw' (a nil value)

Same minimal.lua as #354, though this error seems to happen with every attempt to open the completion menu.

❤


📱(front) improve control bar responsive

We want the bottom bar to stay at the center of the screen + I also added a bit of vertical spacing around the control bar @lebaudantoine .
Before

After

That would be amazing, to be able to close the help menu from the toggle, is it doable?
https://github.com/user-attachments/assets/da7a004f-2f62-498e-8672-0afe58444157


Upgrade from 5.5.1 up hangs on creation of the Autoscaling nested stack

I'm trying to upgrade a stack from version 5.5.1 to 5.7.0 right now, and that upgrade is currently hanging on creating the nested Autoscaling stack. That stack just has the following two events in the "Events" log:

and no resources at all:

Thank you for these details :bow:
I’m afraid I’ve been unable to reproduce this 🤔 would you be able to share your AWS Region and a redacted set of CloudFormation parameters for your stack?

I'm not sure if it's "advanced" but the only potentially unusual thing we're doing is using a secretsmanager secret for the agent token

Are you using the SSM Reference Syntax to retrieve the Secrets Manager Secret value via SSM? I can’t see that causing CloudFormation deploy-time issues with the scaling Lambda because it is retrieved at runtime.

Just tried this, and it does seem to still stall.

I apologise again for the interruption, though I think this is good news in so far as the issue is reproducible 😅

Hi, Seeing this exact issue when trying to upgrade from 5.5.1 to 5.6.1 or 5.7.1. Tested deleting the stack and recreating and seeing the same issue with it hanging on the nested Autoscaling job, eventually errors with
2021-10-14 12:56:26 UTC+0100 Autoscaling CREATE_FAILED Embedded stack arn:aws:cloudformation:us-east-1:stack/buildkite-canary-Autoscaling-3349UQASS4SA/b19dfba0-2ce0-11ec-b434-1222c22090c1 was not successfully created: Internal Failure in the cloudformation events

@darrenwhighamfd could you share the event log for the Autoscaling sub-stack that fails to create successfully?

Other than any account level IAM restriction, do your AWS organisations use any service control policies that could be restricting access to the Serverless Application Repository?

Also confirmed no service control policies

Hi @keithduncan,
I think the issue relates to the BuildkiteAgentTokenParameter trying to pull
from the secret manager, versus systems manager.
I asked our AWS support if they could shine a light. They believe the
failure's due to the BuildkiteAgentTokenParameter having been defined as
AWS::SSM::Parameter::Name, which is a System Manager param, not Secret
Manager.
The Secret Manager will encrypt the data, so GetParameter requires
'WithDecryption' to retrieve the value from Secret Manager. When our
parameter type is AWS::SSM::Parameter::Name, CF doesn't include
WithDecryption in its GetParameter invocation, as
AWS::SSM::Parameter::Name, is a Systems Manager type.
AWS Support recommended using Secrets Manager purely, and this dynamic
reference[1] below.
Environment:
Variables:
BUILDKITE_AGENT_TOKEN_SSM_KEY: '{{resolve:secretsmanager:SecretName:SecretString:SecretKey}}'`
I've confirmed that the rest of the CF template works by deleting my stack and recreating fresh using the
previous BuildkiteAgentTokenParameter option, and was able to install a
v5.7.1 stack.
[1] https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/dynamic-references.html#dynamic-references-secretsmanager

That’s good research @darrenwhighamfd! We could be on to something as @glittershark also showed using AWS Secrets Manager SSM Parameter Store reference syntax.

CF doesn't include WithDecryption in its GetParameter invocation

Our template doesn’t resolve the value of the parameter, so this isn’t applicable. Instead the the template passes the value of the provided SSM Parameter Path to the EC2 instances via UserData and the scaling Lambda. Both resolve the value of this parameter at runtime.
The Lambda template has both ssm:GetParameter and kms:Decrypt permissions, and passes WithDecryption when fetching the value of the SSM parameter.
The launch template’s IAM Role also has ssm:GetParameter and kms:Decrypt permissions, and passes WithDecryption in both the Linux and Windows install scripts.
I have previously tested and documented AWS Secrets Manager SSM reference syntax in our template.

I've confirmed that the rest of the CF template works by deleting my stack and recreating fresh using the
previous BuildkiteAgentTokenParameter option, and was able to install a
v5.7.1 stack.

Were you passing a value for the BuildkiteAgentTokenParameterStoreKMSKey when using a secrets manager reference path for BuildkiteAgentTokenParameterStorePath? NB you must pass the Key ID, not the Key Name for this parameter.


Implement one-hot encoding function as a custom user option

Another way to handle categorical data is by one-hot encoding . One hot encoding creates a new column for every entity in one categorical column and enters 1 if present in corresponding row of original column else 0 . Implement this from scratch by taking the user's option and also taking the column names which need to be one hot encoded as input.
Follow the conventions for writing code and make changes in the comments of _categorical.py to include any new functions added.

I would like to contribute to solving this issue @Saif807380

Implemented in #48


nixos.tests.latestKernel.login.x86_64-linux fails with Linux 5.8

Describe the bug
The nixos.tests.latestKernel.login.x86_64-linux test is failing. I bisected this problem to 3a8cbea4f7398ddd705b9b72b58bc0b6e0cced91 “linux: Init 5.8”. The error message Failed to parse bus message: Invalid argument matches RHBZ #1853736, which points to both a workaround of recompiling systemd against Linux 5.8 headers, and an upstream systemd patch (systemd/systemd#16424) that’s in systemd 246 (#94354).
machine: waiting for success: [ $(fgconsole) = 2 ]
machine # [   18.137166] systemd[1]: Started Getty on tty2.
(0.06 seconds)
machine # Failed to parse bus message: Invalid argument
Test "Check whether switching VTs works" failed with error: "retrieving systemctl info for unit<EMAIL_ADDRESS>failed with exit code 1"
error:
Traceback (most recent call last):
  File "/nix/store/4h6d340vj8ccr09g6nv73g66c2wq6y12-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 894, in run_tests
    exec(tests, globals())
  File "<string>", line 1, in <module>
  File "<string>", line 13, in <module>
  File "/nix/store/4h6d340vj8ccr09g6nv73g66c2wq6y12-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 347, in wait_for_unit
    retry(check_active)
  File "/nix/store/4h6d340vj8ccr09g6nv73g66c2wq6y12-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 137, in retry
    if fn(False):
  File "/nix/store/4h6d340vj8ccr09g6nv73g66c2wq6y12-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 329, in check_active
    info = self.get_unit_info(unit, user)
  File "/nix/store/4h6d340vj8ccr09g6nv73g66c2wq6y12-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 352, in get_unit_info
    raise Exception(
Exception: retrieving systemctl info for unit<EMAIL_ADDRESS>failed with exit code 1

To Reproduce
Steps to reproduce the behavior:
nix build -f nixos/release-combined.nix nixos.tests.latestKernel.login.x86_64-linux
Notify maintainers
Cc @NeQuissimus @andir @eelco @flokli
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: [systemd linuxHeaders]
# a list of nixos modules affected by the problem
module: []

My PR wasn't merged, Linux 5.8 ended up being added by someone else.

Linking for visibility: that PR was https://github.com/NixOS/nixpkgs/pull/94614.

The PR bumping systemd to 245.7 is at https://github.com/NixOS/nixpkgs/pull/94681.

Is it really helpful to send that to staging given that nixos-unstable will be unable to advance without it? Or are you thinking we also want to revert the linuxPackages_latest = linuxPackages_5_8 bump in master for now?

Maybe we should revert 3a8cbea in master, and re-revert it in staging again.
I let @FRidh and @vcunat decide about this, they've been doing the staging rounds.

 I always track https://status.nixos.org and every few days I notice nixos-unstable doesn't progress. When it don't I see an issue with the "channel blocker" label and I always wonder..

@doronbehar Click on the “Hydra job for tests” link and go to the “Constituents” tab.


1.0.6 Release

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 908 (0.11%)  changed or added relevant lines in 30 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 0.866%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/_components/card/action.vue
0
1
0.0%

src/_components/card/card.vue
0
1
0.0%

src/_components/card/tabs.vue
0
1
0.0%

src/_components/disclosure/action.vue
0
1
0.0%

src/_components/disclosure/block.vue
0
1
0.0%

src/_components/disclosure/panel.vue
0
1
0.0%

src/_components/profile.vue
0
1
0.0%

src/_components/tabs/display.vue
0
1
0.0%

src/_components/tabs/item.vue
0
1
0.0%

src/_components/tabs/list.vue
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
32

Relevant Lines:
3996

💛  - Coveralls


Use Timer.periodic in the Resampler

Fixes #264

There is another issue with this change, the align_to is also broken, and fixing it is not all that trivial. Adding alignment to the channels timer is hard because it doesn't know anything about datetime and we want to align to a datetime.
So to align the timer we need to actually have an initial Timer.timeout() to just wait until the next aligned window and then start the Timer.periodic() after that. Having channels (having https://github.com/frequenz-floss/frequenz-channels-python/issues/116 would make this much easier), but for now I think we need to refactor things a bit to make it work. I'll still send a proposal fix but it is taking me a bit more time than expected.

Proposed hacky fix for this: https://github.com/jh2007github/frequenz-sdk-python/pull/1
I would leave this open and implement the hack properly in Timer and only merge it when we upgraded the SDK to use the version with a delay_start option, since there is no real urgency to merge it.

Setting auto-merge.

https://github.com/frequenz-floss/frequenz-sdk-python/issues/556

@jh2007github not sure what's going on here with:

It would be good to merge this before releasing v0.23.0 as it is ready already, but the approvals keep getting dismissed by the base change.


deleting file from uploads page randomly redirects to /channels page

Go to uploads page, navigate a page or two in, use context menu to delete > redirected to channel page.

Just tried it. Deleted on uploads page, it went back to uploads page.


Handle exploded form style parameters

https://github.com/anttiviljami/openapi-backend/issues/569

We are interested in using this. Can I help move this forward in some way? /cc @anttiviljami


react-native-git-upgrade crashes

Description
Attempting to upgrade to fix app crashing instantly when running on device, so I installed and ran per these instructions: https://facebook.github.io/react-native/docs/upgrading.html
So I installed the utility: $ npm install -g react-native-git-upgrade
Then ran the command (from my project's root directory): $ react-native-git-upgrade
After an hour or so, it quit with an error
git-upgrade info Check for updates
git-upgrade info Read package.json files
git-upgrade info Check declared version
git-upgrade info Check matching versions
git-upgrade info Check React peer dependency
git-upgrade info Check that Git is installed
git-upgrade info Get information from NPM registry
git-upgrade info Upgrading to React Native 0.40.0, React ~15.4.0-rc.4
git-upgrade info Setup temporary working directory
git-upgrade info Configure Git environment
git-upgrade info Init Git repository
git-upgrade info Add all files to commit
git-upgrade info Commit current project sources
git-upgrade info Commit current project sources
git-upgrade ERR! An error occurred during upgrade:
git-upgrade ERR! Error: Command 'git commit -m "Project snapshot"' exited with code null:
stderr: undefined
stdout: [master (root-commit) 82862fb] Project snapshot
 171276 files changed, 6902241 insertions(+)

[ lots of create mode lines snipped ]

    at ChildProcess.<anonymous> (/usr/local/lib/node_modules/react-native-git-upgrade/cliEntry.js:58:18)
    at emitTwo (events.js:87:13)
    at ChildProcess.emit (events.js:172:7)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)

Reproduction
Run react-native-git-upgrade on my machine. Don't know how else I could get someone to reproduce it.
Solution
Have it not explode.
Additional Information

React Native version: react-native-cli: 1.2.0 react-native: 0.38.0
Platform: both / neither (not related to a specific app)
Operating System: macOS Sierra

It happened to me bcz of some peer dependencies of a package. After removing the package, it went smoothly. I'm not sure if it is the cause in your case, but it's worth to check your peer deps.

Looks from your comment @uniquarkD that the issue was not directly related to the tooling itself. If I am wrong, please re-open.

@grabbou : @uniquarkD isn't the guy who reported the issue, I was, and it's still happening, so I'm reoppening.

171276 files changed, 6902241 insertions(+)

@veddermatic 😱  that's huge, try adding a .gitignore containing node_modules in your project directory.


Enable remote management of a chrome instance

This is in regard to #109.
To support managing a chrome client across a network, the Chrome struct needs to be able to work in a remote mode. Essentially this just means updating the target host and port to be a remote address and allow creating an instance that doesn't require launching a local instance of chrome.

This is resolved. See further explanation in #109.


FlowAutoColumnCount

Hello is the option FlowAutoColumnCount deprecated?
it was in 1.0.6 available, but now it seems to be deleted?
/// <summary>
/// Gets or sets the minimum column width of FlowListView.
/// Currently used only with <c>FlowAutoColumnCount</c> option
/// </summary>
/// <value>The minimum column width.</value>
public double FlowColumnMinWidth
{
	get { return (double)GetValue(FlowColumnMinWidthProperty); }
	set { SetValue(FlowColumnMinWidthProperty, value); }
}

+1

It's replaced with FlowColumnCount property. If you set it to null it works as auto column count. It's a default option now, so you don't need to set anything really to have auto column count.

At the moment if you do not set it (or set it to null) you get a null exception.


Still running on testnet [BUG]

Hey,
i installed the gui but it shows  to me testnet8 please update it or help me how to change

testnet8 is OK, silicoin project is still under TestNet Phase.


Page published for a future date, page.live is True

I am using Wagtail 1.9, after creating the page, at the after_create_page, if the page's date is published for a day in the future, page.live is set to True. If I check the same page afterwards, using the console, page.live is False. Is any bug for this particular version?

Wagtail 1.9 is no longer supported, sorry.
Note that Wagtail 2.0 (currently in beta) includes an enhancement to allow pages to remain live while a new version is scheduled for publishing: http://docs.wagtail.io/en/latest/releases/2.0.html#scheduled-page-revisions


Error while proxying Websocket connection

Environment

VerneMQ Version: Latest
OS: Official docker image
Erlang/OTP version (if building from source): Official docker image
VerneMQ configuration (vernemq.conf) or the changes from the default: No changes
Cluster size/standalone: standalone

Expected behavior
Proxying Websocket's

Please describe the expected behaviour.
I am working on mproxy, where VerneMQ is our default broker. Proxying works perfectly with MQTT but not for WS.
It works perfect with other brokers like Aedes or EMQX.

I understand that my issue can be caused by my implementation etc... and I am not expecting you to dig into my code or nothing similar. What I am expecting is to check out my VerneMQ error log and maybe point me in the right direction or just think what can cause this error. Thanks!
Actual behaviour
Looks like ws handler crashs.
12:11:37.289 [info] Try to start vmq_plumtree: ok
12:11:37.781 [info] Try to start vmq_generic_msg_store: ok
12:11:38.019 [info] loaded 0 subscriptions into vmq_reg_trie
12:11:38.024 [info] cluster event handler 'vmq_cluster' registered
12:12:02.488 [error] CRASH REPORT Process <0.453.0> with 0 neighbours crashed with reason: bad record state in vmq_websocket:add_socket/2 line 194
12:12:02.488 [error] Ranch listener {{172,17,0,2},8080} terminated with reason: bad record state in vmq_websocket:add_socket/2 line 194

Please include minimal steps to reproduce the issue, if possible.
It's too complicated. Just to try to analyze the logs.

Does a normal WebSocket connection work? Can you post snippet of your config, regarding the WS listener?

Yes, the normal connection works from test client, which is paho go lib (e.g without proxy, connecting directly to Vernemq).
Can you post snippet of your config, regarding the WS listener?
I didn't configure listener or any custom config. I use official docker image, running broker with
 docker run -d -p 1883:1883 -p 8080:8080 --name vernemq-native -e "DOCKER_VERNEMQ_ACCEPT_EULA=yes" -e "DOCKER_VERNEMQ_ALLOW_ANONYMOUS=on" vernemq/vernemq:latest

Ok, then there should be a WS (not WSS!) listener running on 8080.
Strange. What would be the difference from Paho Go client vs connecting from MProxy?

There is a lot of difference, with mproxy client (e.g paho lib) connects to mproxy WS server (server do connection Upgrade) and this server holds connection (its lower level, pure WS). Then server connections to VerneMQ also raw WS and start proxying by Read/Write both connections (message sent from a client will be written to a connection that mproxy holds with server).
It works perfectly for raw TCP (e.g MQTT proxying) and the same concept works with WS, but another broker, not VerneMQ. So I don't get it what is different actually. Does this error log say something that makes sense to you? What it means
terminated with reason: bad record state in vmq_websocket:add_socket what is a record state?

Just to add, in this proxying flow ^ error occurs on CONNECT request. So I successfully create a tunnel (connect to VerneMQ with raw WS and tunnel client connection) but on the first CONNECT packet WS listener crash. Yes pure WS not WSS.

Can you wireshark all of this handshake?
The state record represents the current state of this module: https://github.com/vernemq/vernemq/blob/baa9b2794a1a50980c80a90d1ba51c6e84716f14/apps/vmq_server/src/vmq_websocket.erl#L29
It makes a copy of the state and returns a new state with the Socket added here. The error must be happening while copying state.
https://github.com/vernemq/vernemq/blob/baa9b2794a1a50980c80a90d1ba51c6e84716f14/apps/vmq_server/src/vmq_websocket.erl#L193

Ok, let me try to investigate handshake a bit more. I Hope I will get back with more info. Thanks  @ioolkos

@ioolkos In this flow of proxying, I was missing Sec-Webscoket-Protocol header set to mqtt like paho is doing with gorila WebSocket library  when I am connecting to VerneMQ from proxy. I noticed that Mosquitto broker requires this also (some brokers not, that's why it worked with Aedes and Emqx).
Thanks to your idea with Wireshark, I realized this (compering success handshake client -> broker with failing one client -> mproxy  -> broker).
Now everything works as expected. Thanks you so much for your support and prompt replays. Really appreciate!

Awesome ;)


Recurring Unable to process flush WS error messages on premature client exit

Recurring Unable to process flush WS error messages on premature client exit
Description
If fixed this would definitely help to minimize unnecessary error logs.
When a node has an open web-socket connection to a client that exits prematurely, we see the the recurring errors below that spam our logs until we perform a restart:
2022-04-01T21:30:38.017Z [ERROR] polygon.filter: Unable to process flush, writev tcp 555.555.555.555:10002->555.555.555.555:38176: use of closed network connection

Your environment

OS and version Ubuntu 20
version of the Polygon Edge v0.3.2
branch that causes this issue develop

Steps to reproduce

Connect a client directly to a node + subscribe to a WS connected (e.g. Blockscout explorer)
Terminate the client process
Observe recurring Unable to process flush on the host node

Expected behaviour
I would only expect an initial error indicating an issue with the WS connection, rather than seeing the same error persist requiring a node restart.
Actual behaviour
The same error persists requiring a node restart.
Logs
See above.
Proposed solution
Perhaps after the same WS filter has been attempted to be flushed 5 times - if we still encounter errors on flushing, we should simply discard this filter from the FileManager.

@dankostiuk Hello, Thank you for raising the issue! I could reproduce this error locally and identify the reason. I create PR #487. Please wait for the PR to be merged. I appreciate your help.


Certificate Status: Issuing certificate as Secret does not exist

Goal: encrypt openshift console
Clusterissuer is ready, certificate status is false.
certificate.yaml file -->
apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
name: certificate
namespace: openshift-console
spec:
secretName: secret-tls
issuerRef:
name: acme-internal-issuer
kind: ClusterIssuer
commonName: apps.ocpotctest.otc.acic-navan.com
dnsNames:

"apps.ocpotctest.otc.acic-navan.com"
"console-openshift-console.apps.ocpotctest.otc.acic-navan.com"

To clarify, the message
Issuing certificate as Secret does not exist

says that cert-manager is about to issue a new certificate because it found no existing Secret resource. This message doesn't indicate that there was a problem.
I'll close this issue since there is too little to work with in the issue description. Feel free to add additional information and re-open.


[Feat_Add] Addition of new LLM evals metric

Beyond LLM supports, 4 evaluation metrics: Context relevancy, Answer relevancy, Groundedness, and Ground truth.
We would be looking forward to add new evaluation metric support to evaluate LLM/RAG response

[ ] Faithfulness
[ ] Correctness

Or any other research based metric

I can work on lexical diversity

@tarun-aiplanet please assign it

lexical diversity

I have never heard such metrics to evaluate LLM. Can you provide the reference of research paper. Also you were assigned for Perplexity LLM. Is that done?


Upgrade to go 1.19.5 and add GH Actions

Upgrades to go 1.19.5 and adds GH Actions to test the build automatically.
Also uses the invoking user id to avoid having files created by root on the host. Works on Linux and should work on Mac. Unsure about Windows admittedly.

Thanks for the PR. Great idea! We'll merge this too as soon as we find the time.