Train a tiny neural network.

The training loop is built, we just need to assemble it.

[x] Write a jupyter notebook that gathers the different functions to train a Graph Convolutional Neural Network for the KrasHras experiment.
[ ] Show that the network is learning.
[ ] Overfitting.

Currently, is producing a NaNs (on the predctions of the model), see the notebook . Maybe the normalization of the adjacency matrix is compulsory to do the the spmm correctly.


Replace all .pb.bin and .bin.pb with .binarypb

We were getting the ordering of these two extensions mixed up. .binarypb is used much more frequently internally so it seems worth standardising on that instead.

cl/327552427


Code works on "CodePen" but not "VS Code" --- not sure why

Ive made a carousel that rotates in CodePen but will not rotate in VSCode. The attribute that makes it rotate is the one that says "data-ride="carousel" In other words if you take that attribute out it won't work.
I'm hoping that someone could tell me why this is not working in VS Code.
Here is the HTML<pre lang="HTML"
`
<!doctype html>

<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">

<link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.3.1/css/bootstrap.min.css" integrity="sha384-ggOyR0iXCbMQv3Xipma34MD+dH/1fQ784/j6cY/iJTQUOhcWr7x9JvoRxT2MZw1T" crossorigin="anonymous">

<title>Ristorante Con Fusion: Home</title>

 Uthappizza HOT $4.99
A unique combination of Indian Uthappam (pancake) and
                                Italian pizza, topped with Cerignola olives, ripe vine
                                cherry tomatoes, Vidalia onion, Guntur chillies and
                                Buffalo Paneer.

Weekend Grand Buffet  New

 Featuring mouthwatering combinations with a choice
                            of live different salads, six enticing appetizers, six main entrees and five
                            choicest desserts. Free flowing bubbly and soft drinks. All for just
                            $19.99 per person.

Alberto Somayya

Executive Chef

Award winning three-star Michelin chef with wide
                            International experience having worked closely with
                            whos-who in the culinary world, he specializes in
                            creating mouthwatering Indo-Italian fusion experiences.

                                                    <button class="btn btn-danger btn-sm" id="carousel-pause">

                                                    </button>
                                                    <button class="btn btn-primary btn-sm" id="carousel-play">

                                                    </button>

`
Here is the CSS
`
.carousel {
background:#512DA8;
}
.carousel-item {
height: 300px;
}
.carousel-item img {
position: absolute;
top: 0;
left: 0;
min-height: 300px;
}
.carousel-caption {
background-color:rgba(0,0,0,.3);
padding: 20px 20px 0 20px;
position:absolute;
right:60px;
left:35%;
top:30px;
height: 250px;
width: 60%;
z-index: 0;

}
#carouselButton {
right:10px;
position: absolute;
bottom: 10px;
z-index: .8;
}
`

@mattkrebs1974 This repository is used for tracking issues and features for the GitHub Pull Request extension. A better place to ask this question would be https://stackoverflow.com/questions/tagged/visual-studio-code.


Connection closed before response was received

Package
dio
Version
5.4.2+1
Operating-System
Android, iOS
Adapter
Default Dio
Output of flutter doctor -v
[✓] Flutter (Channel stable, 3.19.4, on macOS 14.4.1 23E224 darwin-arm64, locale en-EG)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.2)
[✓] VS Code (version 1.87.2)
[✓] Connected device (3 available)
    ! Error: Browsing on the local area network for Me. Ensure the device is unlocked and attached with a cable or associated with the same
      local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)
    ! Error: Browsing on the local area network for Mohamed’s Iphone (2). Ensure the device is unlocked and attached with a cable or
      associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)
    ! Error: Browsing on the local area network for iPhone Fadwa. Ensure the device is unlocked and attached with a cable or associated
      with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)
    ! Error: Browsing on the local area network for H. Sayed. Ensure the device is unlocked and attached with a cable or associated with the
      same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)
[✓] Network resources

• No issues found!

Dart Version
3.3.2
Steps to Reproduce

Run with any version of Dio above 5.4.0.
try to send more requests in a short period of time.

Expected Result
When using Dio version 5.4.0, the connection is not closed and the response succeeds.

Actual Result
flutter: The following exception was thrown getBOKRechargeRequestStatusProvider:
flutter: AppException.serverException(type: ServerExceptionType.unknown, message: HttpException: Connection closed before response was received, uri = https://api.bravo-sudan.com/bravo/api/v1/bok/getRechargeRequests, code: null)
flutter:
#0      DioMixin.fetch (package:dio/src/dio_mixin.dart:509:7)

#1      MainApiFacade._errorHandler (package:bravo/core/infrastructure/network/main_api/api_callers/main_api_facade.dart:66:14)

#2      TransactionRemoteDataSource.getBOKRechargeRequestStatus (package:bravo/features/transaction/infrastructure/data_sourse/transaction_remote_remote_data_source.dart:48:20)

#3      TransactionHistoryRepository.getBOKRechargeRequestStatus (package:bravo/features/transaction/infrastructure/repository/transaction_history_repository.dart:43:20)

If this is version specific, could you run git bisect to find out which commit breaks the functionality?

I have just upgraded the Dio version from 5.4.0 to 5.4.2+1

Running bisect will help to determine which exact commit breaks the behavior.

I am considering running a bisect to identify the problematic commit. However, I want to inform you that my recent commit only involved upgrading the version to 5.4.2+1

I understand your statement clearly, IMO. I'm asking you to do the bisect because you have a reproducible example and can debug the issue easily. Besides, simply switching between versions might not be helpful since every version includes a bunch of commits.

I've completed running git bisect to identify the commit that introduced the issue in the Dio package. After testing multiple commits, I've found that the first bad bisect commit is "Fix streamed response cancellation and progress (#2068)".
https://github.com/cfug/dio/commit/845e06f4fde3537522b9dc813a5b6e0fdda4a4c5

I've completed running git bisect to identify the commit that introduced the issue in the Dio package. After testing multiple commits, I've found that the first bad bisect commit is "Fix streamed response cancellation and progress (#2068)".
845e06f

Thanks for the bisect! Now could you split a minimal reproducible example so we can take a deeper investigation with your case?

cc @kuhnroyal to check the implementation as the code owner

Sure! Here's the minimal reproducible example: Minimal Reproducible Example.

@AlexV525, do you have any updates there?

@AlexV525, do you have any updates there?

We'll let you know if any updates. We generally works when we have spare time since this is an open-source project.

Sure! Here's the minimal reproducible example: Minimal Reproducible Example.

Thanks, sadly this is not minimal and requires to start and debug a flutter app. Can you reduce the actual request to a unit test or at least a simple Dart file?

I'm certain this issue stems from using Riverpod, a common example in the docs.
final apiProvider = FutureProvider.autoDispose.family<List<dynamic>, String>((ref, endPoint) async {
  final cancelToken = CancelToken();
  ref.onDispose(() => cancelToken.cancel());
  final response = await ref.watch(dioProvider).get(endPoint, cancelToken: cancelToken);
  return response.data;
});

This issue happens with any version of Dio > 5.4.0, as I mentioned above.
And this is a Minimal Reproducible Example:
import 'package:dio/dio.dart';
import 'package:flutter/material.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';

void main() {
  runApp(const ProviderScope(child: App()));
}

class App extends StatelessWidget {
  const App({super.key});

  @override
  Widget build(BuildContext context) => const MaterialApp(
        home: TabBarScreen(),
      );
}

class TabBarScreen extends StatefulWidget {
  const TabBarScreen({super.key});

  @override
  State<TabBarScreen> createState() => _TabBarScreenState();
}

class _TabBarScreenState extends State<TabBarScreen> with SingleTickerProviderStateMixin {
  late final TabController controller;
  @override
  void initState() {
    controller = TabController(
      length: 2,
      vsync: this,
    );

    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Column(
        mainAxisSize: MainAxisSize.min,
        crossAxisAlignment: CrossAxisAlignment.start,
        children: [
          TabBar(
            indicator: const BoxDecoration(),
            padding: const EdgeInsets.all(8),
            controller: controller,
            tabs: const [
              Tab(
                text: 'Posts',
              ),
              Tab(
                text: 'Users',
              ),
            ],
          ),
          Expanded(
            child: TabBarView(
              controller: controller,
              children: const [
                TabBarViewItem(
                  key: PageStorageKey<String>('tap1'),
                  endPoint: 'posts',
                ),
                TabBarViewItem(
                  key: PageStorageKey<String>('tap2'),
                  endPoint: 'users',
                ),
              ],
            ),
          ),
        ],
      ),
    );
  }
}

final dioProvider = StateProvider<Dio>((ref) {
  return Dio()
    ..options = BaseOptions(
      baseUrl: 'https://gorest.co.in/public/v2/',
      connectTimeout: const Duration(seconds: 20),
      receiveTimeout: const Duration(seconds: 30),
    );
});

final apiProvider = FutureProvider.autoDispose.family<List<dynamic>, String>((ref, endPoint) async {
  final cancelToken = CancelToken();
  ref.onDispose(() => cancelToken.cancel());
  final response = await ref.watch(dioProvider).get(endPoint, cancelToken: cancelToken);
  return response.data;
});

class TabBarViewItem extends ConsumerWidget {
  const TabBarViewItem({
    super.key,
    required this.endPoint,
  });
  final String endPoint;
  @override
  Widget build(BuildContext context, WidgetRef ref) {
    final list = ref.watch(
      apiProvider(endPoint),
    );
    return Column(
      children: [
        list.when(
          data: (e) => Expanded(
            child: ListView.separated(
              padding: EdgeInsets.zero,
              itemBuilder: (context, index) {
                return Padding(
                  padding: const EdgeInsets.all(8.0),
                  child: Text(
                    e[index]['id'].toString(),
                  ),
                );
              },
              separatorBuilder: (context, index) {
                return const Divider();
              },
              itemCount: e.length,
            ),
          ),
          error: (obj, st) {
            return Container(
              height: 160,
              alignment: Alignment.center,
              child: ElevatedButton(
                onPressed: () => ref.invalidate(apiProvider),
                child: const Text(
                  'Retry',
                  style: TextStyle(color: Colors.white),
                  textAlign: TextAlign.center,
                ),
              ),
            );
          },
          loading: () {
            return Expanded(
              child: ListView.separated(
                itemBuilder: (context, index) =>
                    const Center(child: CircularProgressIndicator.adaptive()),
                separatorBuilder: (context, index) => const SizedBox(height: 24),
                itemCount: 4,
              ),
            );
          },
        ),
      ],
    );
  }
}

This is actually not caused by #2068, but a very old implementation that detaches the socket that has been promoted recently in #2036.
TL;DR, the cause is:
https://github.com/cfug/dio/blob/1c2843a2fab1c3795272df1e81502616761e192e/dio/lib/src/adapters/io_adapter.dart#L216
Details
ResponseBody.onClose was implemented to detach from the socket which provided its connection in the IOHttpClientAdapter. However, with the same host between requests, the connections are implied to use the same socket (which is reasonable). When the CancelToken got canceled, it would call ResponseBody.onClose and the socket detached, all requests related to the same host would be closed.
The solution here might be to remove the detaching which has no affection according to its doc: https://api.dart.dev/stable/3.3.3/dart-io/HttpClientResponse/detachSocket.html

This is normally used when a HTTP upgrade is negotiated and the communication should continue with a different protocol.

Anyway, this could be a long-affected regression, and the stack trace did not help us to combine them together, the case is rare and valunable!

What is the recommandation to avoid this issue ?

This was changed in 5.4.1, you might be able to lock the dio version to 5.4.0.

Same problem here

Issue happens when you have two requests with different CancelTokens and while first request done, second requests starts executing and in that moment you cancel CancelToken from first request somewhere in code (for example at Bloc dispose) - then second request being canceled before was received, even if it was different cancelToken from first request whole DIO connection closes.
Fixes I found so far:

You can use lower Dio version.
Open new Dio connection for each request.
Initialize DIO with (BaseOptions(persistenConnection: false))
Ensure cancel token right after request being done in your dio controller realisation

@Wratheus Can you reproduce this in a test case? Also not sure if this is the same issue.
This seems to work:
    test('cancel one request', () {
      final dio = Dio()..options.baseUrl = 'https://httpbun.com';
      final token = CancelToken();

      expectLater(
        dio.get(
          '/drip?delay=0&duration=2&numbytes=10',
          cancelToken: token,
        ),
        throwsDioException(DioExceptionType.cancel),
      );
      expectLater(
        dio.get(
          '/drip?delay=0&duration=2&numbytes=10',
          cancelToken: CancelToken(),
        ),
        completes,
      );

      Future.delayed(const Duration(seconds: 1), () {
        token.cancel('cancelled');
      });
    });

You need to do it with post request, forgot to notice that, sorry.
This case looks alike
import 'package:dio/dio.dart';
import 'package:flutter_test/flutter_test.dart';

void main() {
  test(
    'cancel one request',
    () async {
      final dio = Dio();
      const String baseUrl = 'https://httpbun.com';
      final token = CancelToken();

      await expectLater(
          dio.post(baseUrl,
              data:
                  FormData.fromMap({'delay': 0, 'duration': 1, 'numbytes': 10}),
              cancelToken: token),
          completes);
      expectLater(
        dio.post(baseUrl,
            data:
                FormData.fromMap({'delay': 0, 'duration': 5, 'numbytes': 100}),
            cancelToken: token),
        completes,
      );
      token.cancel('cancelled');
    },
  );
}

This case fails.
https://github.com/cfug/dio/blob/537832e17903c4046e5decc345763de3af833264/dio_test/lib/src/test/cancellation_tests.dart#L117-L143

There is no difference if there is cancelToken in second request or no.
import 'dart:async';

import 'package:dio/dio.dart';
import 'package:flutter_test/flutter_test.dart';

  void main() {
    Dio dio = Dio();
    test(
      'not closing sockets with requests that have same hosts',
          () async {
        final token = CancelToken();
        const String baseUrl = 'https://httpbun.com';

        final completer = Completer<Response?>();
        // Complete the first request with the cancel token.
        await dio.get('$baseUrl/get', cancelToken: token);
        // Request the second without any cancel token, but with the same host.
        dio.get('$baseUrl/drip?duration=3', cancelToken: CancelToken()).then(
              (res) {
            completer.complete(res);
            return res;
          },
          onError: (e, t) {
                print(e);
                print(t);
            completer.complete(null);
            return Response(requestOptions: (e as DioException).requestOptions);
          },
        );
        // Simulate connection established.
        await Future.delayed(const Duration(seconds: 1));
        token.cancel();
        final response = await completer.future;
        // Response should be obtained without exceptions.
        expect(response, isNotNull);
      },
      testOn: '!browser',
    );
  }

5.4.0

Is there a big difference between 5.4.0 and 5.4.2+1? I also reproduced this problem on 5.4.2+1. If I lock 5.4.0, does my user need to make API level adjustments?

The fix is included in v5.4.3 and shall not require interface updates.


Add a script for automatically updating scheme

Currently manually maintained dependencies may be lost, so a new script has been added to keep them in sync
xref #3

This is awesome! Thank you @wzshiming for working on this!

Done

This PR requires a member to approve the execution of the workflow
/cc @siyuanfoundation
/cc @jmhbnz
/cc @wenjiaswe

/ok-to-test

Sorry for the late reply, I refactored the logic as discussed, please take a look again.

/approve
@jmhbnz for double check

Done.


Shell escape process arguments that are printed out

Shell escape process arguments that are printed out
cargo build --verbose does output some command lines that cannot be
simply copy and pasted into the shell again. The problem is the
arguments which are output exactly like this: --feature="foo"
When pasted back into the shell, the shell will parse and remove the
double quotes. To counteract this, escape special shell characters when
printing commandlines. Cargo will print --feature="foo" instead, which
can be pasted back into the shell.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Successor of #1572. I think Windows support is still WIP.
Ok, it was explained that no detection is needed. Just escape " and spaces on windows by quoting the arguments, also replace all backslashes with slashes.
Unfortunately the old fail log is gone and I can't find where it failed on windows the last time, but there must be some test that needs to be updated.

@bors: try

@bors: r+ f8e02317db530ec8ae3e46d152bd0644d17532d3
Thanks!

:hourglass: Testing commit f8e0231 with merge 20b67d4...

:broken_heart: Test failed - cargo-win-gnu-32

I'd personally recommend not using #[cfg] here and instead using if cfg!(windows) as it'll allow typechecking on all platforms.
Also in theory if you're using a MSYS shell on Windows you should use the same escaping as the unix shell, but that can always be implemented later.

Oh that's a mistake. The intention was for both windows and other to both compile and test always -- which they do on linux.

I'm sorry, I can't really work on something that I can't run & test (i.e. Windows support).

It would be better to merge this with single platform support only, and then have another contributor pick up the windows support -- in a PR that they can develop & test on that platform.

The Windows bots are failing because backslashes are being translated to forward slashes, so tests need to be updated, but I'm also not 100% convinced that we can do this kind of translation as it's changing the input. Do you have a link to the documentation for Windows on how the string may be escaped?

http://blogs.msdn.com/b/twistylittlepassagesallalike/archive/2011/04/23/everyone-quotes-arguments-the-wrong-way.aspx
https://msdn.microsoft.com/en-us/library/17w5ykft.aspx?f=255&MSPPError=-2147217396

Thanks, I didn't use any docs, I just talked with @retep998 :)


THREESCALE-8749: Ability to set APICAST_SERVICE_CACHE_SIZE

Description
Add optional field in the Apicast resource that sets the APICAST_SERVICE_CACHE_SIZE environment variable in the Apicast container
Verification steps
On a cluster, run the operator and create an APICast with the new field set. Check that once reconciled the Deployment includes the environment variable with the value from the new fields:

Apply the manifestsmake install

Run the operatormake run

Create an APICast that includes the following in the spec:spec:
  // ...
  serviceCacheSize: 42

Check the Deployment for APICastoc get deployment/apicast-example-apicast -o yaml

Verify that they include the environment variable# ...
- name: APICAST_SERVICE_CACHE_SIZE
   value: '42'
# ...

Thanks @eguzki, I just pushed an update with the included docs

Thank you @eguzki, I have no permissions to merge in this repo. Could you merge this for me please?

Let me fix that when I arrive at the office

Fixed, @sergioifg94 please, do the honors :)

Cheers @eguzki @thomasmaas. Merging now :smile:


Compilation errors when trying out cpp-utils

Hi!
I took the hello-world example and enabled C++ exceptions and ble via the menuconfig.
I deleted the c file and added the skeleton file helloworld.cpp from this repo.
Next I copied the files from cpp-utils to /components/cpp-utils.
When I do a make command i got the following error:
project_path/components/cpp_utils/HttpRequest.cpp:42:26: fatal error: hwcrypto/sha.h: No such file or directory

Solved this by changing
#include <hwcrypto/sha.h>

to
#include <esp32/sha.h>

In the HttpRequest.cpp file under components/cpp-utils
But I get additional errors i can't seem to fix:
project_path/components/cpp_utils/BLEDevice.cpp: In static member function 'static void BLEDevice::whiteListAdd(BLEAddress)':
project_path/components/cpp_utils/BLEDevice.cpp:488:75: error: too few arguments to function 'esp_err_t esp_ble_gap_update_whitelist(bool, uint8_t*, esp_ble_wl_addr_type_t)'
  esp_err_t errRc = esp_ble_gap_update_whitelist(true, *address.getNative());  // True to add an entry.
                                                                           ^
In file included from project_path/components/cpp_utils/BLEDevice.cpp:17:0:
/home/parallels/esp/esp-idf/components/bt/bluedroid/api/include/api/esp_gap_ble_api.h:947:11: note: declared here
 esp_err_t esp_ble_gap_update_whitelist(bool add_remove, esp_bd_addr_t remote_bda, esp_ble_wl_addr_type_t wl_addr_type);
           ^
project_path/components/cpp_utils/BLEDevice.cpp: In static member function 'static void BLEDevice::whiteListRemove(BLEAddress)':
project_path/components/cpp_utils/BLEDevice.cpp:502:76: error: too few arguments to function 'esp_err_t esp_ble_gap_update_whitelist(bool, uint8_t*, esp_ble_wl_addr_type_t)'
  esp_err_t errRc = esp_ble_gap_update_whitelist(false, *address.getNative());  // False to remove an entry.
                                                                            ^
In file included from project_path/components/cpp_utils/BLEDevice.cpp:17:0:
/home/parallels/esp/esp-idf/components/bt/bluedroid/api/include/api/esp_gap_ble_api.h:947:11: note: declared here
 esp_err_t esp_ble_gap_update_whitelist(bool add_remove, esp_bd_addr_t remote_bda, esp_ble_wl_addr_type_t wl_addr_type);
           ^

Is this a known issue? Am I doing something wrong

New IDF changed the api call for whitelist functions.

Thanks that solved it!


Ordinalize vs ordinalise nuances

ordinalize returns numeric with ending whereas ordinalise returns exclusively text.
This little differentiation caused me to search on google for about an hour until I realized that your gem did both, but the readme file was wrong.

nudge nudge
This is a pretty simple readme fix.

3 months stale...


bug: movement of pipette between the labware wells has lower hard limit for the height than the tip dropoff

Overview
We are using a custom labware which we place on the heatmodule. Combined they constitute a considerable height – yet within the operation limits of the system.
We found that moving the pipette between the wells of our labware (when on hm) works just fine - when the tip is being dropped off after each well. Of note – to drop the tip the pipette (with a tip) has to traverse at an offset distance above the surface of our labware – which it does without a problem.
Yet when we try to move the pipette with a tip between the wells of the labware without dropping the tip – we receive an error informing that the height? hard limit is reached – and the protocol stops.
https://user-images.githubusercontent.com/20386657/206316379-aa42b66e-f978-4776-8dc9-ced17984bbb5.MOV

Physically there is no difference between moving the pipette to drop off the tip and moving the pipette to another well. Yet the system finds it impossible to achieve the first.
Please advice!
Steps to reproduce
No response
Current behavior
No response
Expected behavior
No response
Operating system
Windows
System and robot setup or anything else?
OT-2

we found the problem. It was inside our code


Remove spinlock in PollStart()

Remove spin lock in PollStart(). This significantly reduces the CPU time when waiting for apps to start.
Does this PR modify CLI v6 or v7?
v6
Description of the Change
The PollStart() method selects on channels that might be closed (appState, apiWarnings, and apiErrs) during the PollStart() invocation. Since a select on a closed channel returns the channel type's zero value, the closed channels are always 'enabled'. In combination with the check for closed channels via booleans, this is basically a spin lock.
This PR disables the channels once they are closed by setting them to nil. This is semantically equivalent but much less heavy on the CPU.
I ran cf push with an app that would never report as ready with a profiler enabled and interrupted after about 20s (sig int / ctrl c).
As you can see in the attached profile, most of the time is spent in locking of runtime/selectgo
master-a3840f49e3c0d1d3c1e390d7c222a0c564c3738a.pdf
With the patch applied, the CPU is almost completely idle:
fix-261312ab4e642fb2c5dbcf7a7e51bf7b8af80853.pdf
Why Is This PR Valuable?
cf push should not make my CPU burn :)
Why Should This Be In Core?
Core functionality.
Applicable Issues
None.
How Urgent Is The Change?
Not urgent.
Other Relevant Parties
No one.

Hi @seif-at-sap thanks for creating this pull request. For changes to push or any other significant changes to core commands, we would appreciate a heads up before a pull request is created - this helps us determine whether this is a feature/fix we want to support, and it ensures you are not working on pull requests which may not get merged.
I have a few initial comments:

what is the user pain you are experiencing?
how would I be able to reproduce it in acceptance testing? if the pain point is that the CPU usage spikes when you cf push, under which conditions does this occur - please detail by stating real world examples and how a user would experience this pain point

Thanks, and let me know if you have any questions. We'll need more information before we can proceed. Thanks.

what is the user pain you are experiencing?

100% CPU from the cf binary.

how would I be able to reproduce it in acceptance testing? if the pain point is that the CPU usage spikes when you cf push, under which conditions does this occur - please detail by stating real world examples and how a user would experience this pain point

Try to push an app that never becomes ready, e.g.:
---
applications:
- name: never-ready
  no-route: true
  health-check-type: http
  docker:
    image: alpine:3.9
  command: sleep 100

I initially noticed this when I accidentally configured the wrong health check type for an application I was developing.

Hi @seif-at-sap thanks for the quick response. We'll do a review of this pull request and will give you feedback as soon as we are able to. Thanks.

Same issue here. When an application is slow to stage or slow to start cf is using 100% of CPU.


Support 'flutter run' for desktop targets

Tracks being able to flutter run for macOS, Windows, and Linux.
This will likely be implemented in phases, first being able to flutter run using a prebuilt application on each platform (already implemented behind flags for macOS), then later once #30706 is done having the full behavior of building and then running.
Any support we can provide for this in the short term is very valuable for the workflow of people trying out early stages of the desktop support, since being able to flutter run provides compatibility with existing Flutter support in tools and IDEs (e.g., hot reload, and source-level debugging).

cc @jonahwilliams who has been working on aspects of this already.

We can flutter run the prebuilt macos application, linux and windows need to be done though

We also need to figure out how to bring it out from behind an environment variable flag. In practice, that's been a significant issue for actually using the functionality, because adding a flag to the launch environment of the flutter daemon used by VS Code or IntelliJ isn't a simple thing to do.

I have two fixes for this:

Disable the devices/commands on the stable branch https://github.com/flutter/flutter/issues/30152

update device discovery to take the current project into consideration. For example, if you have ios/ and android/, then we don't list a macos device.

Filled https://github.com/flutter/flutter/issues/30724 for 2.


Don't log zero timestamp

Justification for this comes from the slog.Handler documentation:

"If r.Time is the zero time, ignore the time."

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (0b7a5ea) to head (28a6c14).

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines          148       150    +2
=========================================
+ Hits           148       150    +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Create 0000-01-02-esku953.md

I thought it would be cool if we add this.

Everything looks good so far.

Looking good so far.

Looking good so far.


ignore LdcEqualsLdd and let all kernel support ldc != ldd.

ignore LdcEqualsLdd and let all kernel support ldc != ldd.
This change let Tensile inherently support ldc != ldd.
So caller, ex. rocblas, doesn't have to handle ldc!=ldd.
We have compare performance before and after change.

ldd.xlsx

use another PR for whole feature


Rviz start angle max num points scan

This PR corrects a bug in RViz display when start_angle & max_num_points_scan is specified as reported in issue https://github.com/PepperlFuchs/ROS_driver/issues/47
When applied together with PR https://github.com/PepperlFuchs/ROS_driver/pull/49, dynamic reconfigure will work with the correct RViz display
Only code pertaining to R2300 is changed, as I don't have an R2000 to test with.

@wsaihopfsg thanks for the contribution! Could you please resolve the conflicts?

@wsaihopfsg thanks for the contribution! Could you please resolve the conflicts?


ChromeSafariBrowser clearCache

Environment
Plugin version: flutter_inappwebview: ^5.4.3+7
Device information: iOS 14
Description
What you'd like to happen:
The ability to clear the cache of ChromeSafariBrowser when opening it, much like it is possible for the WebView

Is there a solution to this?


HttpClient

It would be useful to mention on this page also HttpClient, and when it should be used instead of WebClient..

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

ID: bf19b23d-3693-2508-dbbe-2a2ee2c86081
Version Independent ID: e3e27e19-80a7-7411-4382-c88ec9a2da5b
Content: HTTP
Content Source: docs/framework/network-programming/http.md
Product: dotnet-framework
Technology: dotnet-networking
GitHub Login: @karelz
Microsoft Alias: ncldev

@tudort-iquest Hello. Sorry for the delay in getting to this issue. Really this article needs to be rewritten for HttpClient. I'm closing this as a duplicate. Thanks!

Duplicate of #13253


Update feature-update-mission-critical.md

issue #2128

Hi, Janet here.  I have been having issues with a hacker, over a month now.  They got all my personal info exposed everything on social media.   They have accessed all my PC’s.  The request using android phone and Mac computer.  Had to purchase another desktop.  I do not understand why this is not getting repaired.  They are running so much data through my phone my carrier is questioning what is going on.  I am loosing accounts deleting my google accounts also changing passwords on the accounts & deleted important files & data.  What do I have to do to get this resolved?   I cannot afford replacing any more devices.  Also my online presence has been defrauded & reputation is in question?  Please help tp resolve this matter.  Thank You

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@greg-lindsay - Can you please assist to check, approve and merge the content of this PR based on issue #2128?
Thank you.

@greg-lindsay - We would like to follow up on this. Can you please assist to check, approve and merge the content of this PR based on issue #2128?
Thank you.


GKE Containerd images and GKE >1.19

Summary
Sooner or later, Google will force users to adopt containerd as a runtime and discontinue Ubuntu images with Docker runtime. No cluster created with version 1.19 is guaranteed to continue operating Argo Workflow. It would be interesting to have a fix or some guide for Google users.

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

I recently installed Argo into a new GKE cluster and hit a snag (workflows hanging in Running state) before stumbling on this issue. I updated the Image type to Ubuntu with Docker (ubuntu) and things are running smoothly (thanks @thiago4int ).
It might be worth putting a note in the docs to bring this to new users' attention when creating GKE clusters.

Does this mean that "Container-Optimized OS from Google" isn't supported? We've been running Argo on that without any issues for a year now.

@max-sixty Container-Optimized OS with Containerd won work for me

@max-sixty it works, but there can be a few key snags. I've seen workflows stick around in the Running state long after they've completed on GKE 1.19 with Argo Workflows configured for the Docker executor, like sdmcclain is describing.

I've been running Argo Workflows in production on GKE with container optimized OS for ~1 year, the key is changing the executor to PNS or equivalent (the Docker executor does fail)

See #5434

Does this mean that "Container-Optimized OS from Google" isn't supported? We've been running Argo on that without any issues for a year now.

just want to point out that this is not the same thing as containerd, there are both "Container-Optimized OS with Docker" (cos) and "Container-Optimized OS with Containerd" (cos_containerd) images: https://cloud.google.com/kubernetes-engine/docs/concepts/node-images#available_node_images

Work-around: use PNS executor

I think we have a solution.


A new IPVlan CNI chain to work with Terway (Alibaba Cloud CNI Plugin)

Hi maintainers, this merge request add a new ipvlan cni chain to work with Terway, a Alibaba Cloud VPC/ENI CNI. With help of cilium, we implement service network and network policy for IPVlan l2 datapath in Terway.
Terway has two different datapath modes: VPC Mode and ENI Secondary IP. In ENI Secondary IP, one eni (Elastic Network Interfaces) has multi ips. With these ips, We can create multi IPVLan l2 slave ports to support more pods. IPVlan L2 mode offers almost the same performance as the eni. But the one problem is that IPVlan cannot support the service network and network policy.
This merge request introduces a IPVLan L2 cni chain. In the cni chain, terway creates IPVlan datapath and allocs ips from vpc, and cilium attaches bpf to slave port  to provide service network and network policy ability.

This change is

Coverage decreased (-0.02%) to 45.514% when pulling 48500f16474adf9a6452f558627a8096257af929 on zhiyuan0x:zhiyuan/upstream-v2 into 405a7ae27330562c6f2b6de397b953bf45f0ae51 on cilium:master.

test-me-please

This looks awesome. How can we establish a testing procedure? We have recently enabled our CI to be able to integrate with external clusters running in cloud providers. That seems likely to be the ideal solution so we can run a subset of the integration tests in terway chaining mode in an Alibaba Cloud hosted cluster.

We have recently enabled our CI to be able to integrate with external clusters running in cloud providers. That seems likely to be the ideal solution so we can run a subset of the integration tests in terway chaining mode in an Alibaba Cloud hosted cluster.

@tgraf  Ok, I think it's a good idea!

This looks awesome. How can we establish a testing procedure? We have recently enabled our CI to be able to integrate with external clusters running in cloud providers. That seems likely to be the ideal solution so we can run a subset of the integration tests in terway chaining mode in an Alibaba Cloud hosted cluster.

Did you mean we should provide the doc about how to run e2e test on Alibaba Cloud like GKE and AWS EKS?  And then you create a new jenkins job follow the guide?

@zhiyuan0x We would love to push this along. Any way we can help?

@zhiyuan0x We would love to push this along. Any way we can help?
@tgraf can you tell us what do we if you push this patch, @Cordius  does not get any response.

@Cordius Apologies, I've missed the question. Response below:

Did you mean we should provide the doc about how to run e2e test on Alibaba Cloud like GKE and AWS EKS?

Yes, that would be amazing. It doesn't have to be done as part of this initial PR but we should agree on how to test this going forward and how we can avoid breaking this integration.

And then you create a new jenkins job follow the guide?

Yes

@tgraf can you tell us what do we if you push this patch, @Cordius  does not get any response.

@xs3c Answers above. It looks like we are pretty close, we just have to address the feedback given.

@Cordius Apologies, I've missed the question. Response below:

Did you mean we should provide the doc about how to run e2e test on Alibaba Cloud like GKE and AWS EKS?

Yes, that would be amazing. It doesn't have to be done as part of this initial PR but we should agree on how to test this going forward and how we can avoid breaking this integration.

And then you create a new jenkins job follow the guide?

Yes

@tgraf can you tell us what do we if you push this patch, @Cordius  does not get any response.

@xs3c Answers above. It looks like we are pretty close, we just have to address the feedback given.

I got it.

this PR has been marked as a draft PR since it had a WIP label. Please click in "Ready for review" [below vvv ] once the PR is ready to be reviewed. CI will still run for draft PRs.

@zhiyuan0x any chances you might get back to this soon? is there anything you need help with?

hi ,I'm working on Terway project .Is there anyting  i can help ? @zhiyuan0x
I suggest we can make the plugin more generic . https://github.com/AliyunContainerService/terway/pull/149/commits/a41e9eb7dbb9381d5960da3055848906ef238031

Closing in favor of #14436.


drawer component bug fix

@filoozom urgent request to approve the PR..
I need to do publish the new version and this one got left

@filoozom sorry unable to unpublish last version, doing urgent merge


Can't create the container

Expected behavior
To let container being created
Actual behavior
(HTTP code 404) unexpected - {"message":"manifest for delavega/au-commerce:latest not found"}
Information about the Issue
Just cant create it using Kitematic , but with command line is possible.
Steps to reproduce the behavior

just find in 'au-commerce' and create

Go to https://hub.docker.com/r/delavega/au-commerce/tags/
Observe no latest tags were created for that image
Select a different tag in Kitematic (if one exists)
🎉  enjoy container running.

Ideally you open an issue with the repo owner to let them know how frustrating it is that they don't have a latest image tagged


Detect if input comes from Chrome Developer Tools?

In Node I can listen for input from stdin
		standardIn.resume();
		standardIn.on('data', function(key) {
			this.handleKey(key);
		}.bind(this));

Is there anyway for me to know if input is coming in from the Chrome web developer tools here?

This would be cool because I would be able to build some fun console interaction tools that would rely on knowing whether or not the input came from the dev tools console

There is no easy way to do that, the only way I can think of is to inject custom scripts to devtools by using remote.getCurrentWindow().devToolsWebContents.


Announce message should include what streaming services a user has

It would be super-useful if the announce message could include details about a user's streaming services.   We could make channels that are Tidal or Qobuz specific.

It doesn't seem like this is feasible.


KMeansSpeedIT.testKMeansSpeed() needs to be made more deterministic

... fails > 50% of the time now

The last run showed at least what's going on:
2015-02-20 10:21:01,242 INFO  KMeansSpeedIT:65 UP,[0,[1.0,0.9859154929577465],71]
2015-02-20 10:21:01,242 INFO  KMeansSpeedIT:65 UP,[1,[2.0,-1.971830985915493],71]
2015-02-20 10:21:01,243 INFO  KMeansSpeedIT:65 UP,[2,[-1.9583333333333335,0.0],72]
2015-02-20 10:21:01,243 INFO  KMeansSpeedIT:65 UP,[1,[2.0,-1.9722222222222223],72]
2015-02-20 10:21:01,243 INFO  KMeansSpeedIT:65 UP,[2,[-1.9589041095890412,0.0],73]

Even though 2.1 seconds of data were sent in 30 seconds of batch interval, all but two arrived as expected in the interval, but then two stragglers came in after. I really don't know what to make of it! the time intervals are already tuned to be generous beyond reason.
The result is "correct" just not quite divided up as expected. Maybe I'll rewrite the test to examine the final state of the updates rather than expect exactly 3.


fix fix regression error of test_compress_type

backport: #28826
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Modrinth latest version crashes

The latest version of JEMM on Modrinth is 4.2 which crashes every time I load into a world. I turned only JEMM off and the world loaded just fine.

Fixed version uploaded. https://modrinth.com/mod/just-enough-mekanism-multiblocks/version/4.5


How to visualize the output?

Thanks for your great project. However, I have trouble visualizing the predictions. The image looks very strange, could you please tell me how to correctly visualize them?

I'm really looking forward to visualizing the prediction R and S……

I have the same question, have you solved it?

:heavy_plus_sign:
@lixx2938 In your previous work Learning Intrinsic Image Decomposition from Watching the World the model predicts two images - 3-channel log-reflectance and 1-channel log-lighting - so I could do torch.exp(log_r + log_s).
But in this repository the model predicts two 1-channel images. How to use them to reconstruct the original image?

I also want to know why the input and output images of the model are both in 1-channel and how can I reconstruct the original image? Have you solved this problem?

I think I solve this question now. Please read the code on computing Reconstruction loss. Here is my explanation.
Gray-scale shading assumption

A colorful pixel can be decomposed into:

RGB = intensity * chromaticity
That means,

input_rgb_image = img_intensity * img_chromaticity

Note that, input image here is represented in linear RGB space. The photo loaded from the disk is usually represented in sRGB space. You can find the conversion code in this project.  Be careful the input image for the network is in sRGB space, but when computing reconstruction loss, the image is in linear RGB space.

R = R_intensity * R_chromaticity
S = S_intensity * S_chromaticity

Under gray-scale shading (white lighting) assumption, above can be rewritten as,

S = S_intensity
img_chromaticity = R_chromaticity
R = R_intensity * img_chromaticity

For the more specific explanation for intensity and chromaticity, I recommend you to read:

Intrinsic images in the wild
Revisiting Deep Intrinsic Image Decompositions

You can find all these code in the project (especially in the script of data_loader): (if not, please look at code in IIW and SAW repositories)

rgb to srgb
srgb to rgb
compute intensity (it's actually the avarage value of RGB channels)
compute chromaticity

Network prediction

the outputs of the network actually are:

S_pred = log(S_intensity)
R_ pred = log(R_intensity)

If you want to get S and R:

S = exp(S_pred)
R = exp(R_pred) * img_chromaticity

Reconstruct original input image

RGB_reconst_image = S * R
If you want to visualize it, convert RGB_reconst_image into sRGB space

sRGB space is suitable for human eyes

@tlsshh 덕분에 시각화 문제도 해결했습니다.
다음은 다른 이미지 세트 (IIW 또는 SAW 아님)가있는 구현 코드입니다
(test_iiw.py 기반)
opt = TrainOptions().parse()  # set CUDA_VISIBLE_DEVICES before import torch

#root = "/home/zl548/phoenix24/"
#full_root = root +'/phoenix/S6/zl548/'

model = create_model(opt)
minc_loader = get_minc_loader(batch_size=16)
   for i, (x,y) in enumerate(minc_loader):
       x = x.float().cuda()
       x = x.view(16, 3, 256, 256)
       output_R, output_S = model.test_minc(x)

       for j in range(0,output_R.size(0)):

           prediction_R = output_R.data[j,:,:,:]
           prediction_R = torch.exp(prediction_R)#.repeat(1,3,1,1)

           prediction_S = output_S.data[j,:,:,:]
           prediction_S = torch.exp(prediction_S)#.repeat(1,3,1,1)

           # calc chromaticity
           srgb_img = x.data[j,:,:,:]
           rgb_img = srgb_to_rgb(np.transpose(srgb_img.cpu().numpy(), (1,2,0)))
           rgb_img[rgb_img <1e-4] = 1e-4
           chromaticity = rgb_to_chromaticity(rgb_img) # opt 1
           # opt 2 chromaticity = rgb_to_chromaticity(srgb_img.cpu().numpy())
           chromaticity = torch.from_numpy(np.transpose(chromaticity, (2,0,1))).contiguous().float()

           p_R = torch.mul(chromaticity, prediction_R.cpu())
           p_R_np = p_R.cpu().numpy()
           p_R_np = np.transpose(p_R_np, (1,2,0))
           p_R_np = cv2.cvtColor(p_R_np, cv2.COLOR_BGR2RGB)

           p_S_np = np.transpose(prediction_S.cpu().numpy(), (1,2,0))
           p_S_np = np.squeeze(p_S_np, axis=2)

           save('D:/minc_decomposition/yuv/' + str(i) + '_' + str(j) + '_albedo.png', p_R_np)
           save('D:/minc_decomposition/yuv/' + str(i) + '_' + str(j) + '_shading.png', p_S_np)
           cv2.imwrite('D:/minc_decomposition/yuv/' + str(i) + '_' + str(j) + '_original.png',np.transpose(srgb_img.cpu().numpy(),(1,2,0)))
       print('save data done')

이해를 돕기 위해 test_data 함수 및 data_loader에 대한 추가 코드는 다음과 같습니다.
intrinsic_model.py에 추가 한 test_data 함수는 다음과 같습니다.
    def test_minc(self, input_):
        prediction_R, prediction_S = self.netG.forward(input_)
        return prediction_R, prediction_S

여기 data_loader에 추가 한 data_loader 함수가 있습니다.
class TestDataSet(Dataset):
    """
    Custom Test DataSet class
    """
    def __init__(self, transform=None):
        x_test = []
        y_test = []
        test_file_name = 'D:/labels/test2.txt' # image list
        test_file = open(test_file_name, 'r')
        test_list = test_file.readlines()
        test_file.close()

        print('start loading test data')
        for i in range(len(test_list)):
            img = cv2.imread('D:/MINC/minc-2500/minc-2500/' + test_list[i].rstrip('\n')) # load image
            img = cv2.resize(img,(256,256)) #cv2 load image with bgr format
            img = np.transpose(img3, (2,0,1))

            x_test.append(img)
            y_test.append(1)#convert_class(label[1])) # ex) 'brick'  #since given project doest not require image label, I randomly gave 1 as a label

        print('loading test data done')
        x = np.asarray(x_test)
        y = np.asarray(y_test)
        #y_test = np_utils.to_categorical(y_test[:len(test_list)], num_classes)

        self.len = len(y)
        #self.transform = transform
        self.x_data = torch.from_numpy(x)
        self.y_data = torch.from_numpy(y)

    def __getitem__(self, index):
        return self.x_data[index], self.y_data[index]

    def __len__(self):
        return self.len

def get_minc_loader(batch_size,
                    shuffle=True,
                    show_sample=False,
                    num_workers = 2,
                    pin_memory = True):
    # load dataset
    test_dataset = TestDataSet()

    test_loader = torch.utils.data.DataLoader(
        test_dataset ,batch_size= batch_size, shuffle=False,
        pin_memory=pin_memory)

    return test_loader

I followed your code, but I only see a black image. Do you know the solution?


[docs] Discrepancies between Rekor types documentation and schemas?

Hey there! I'm looking into increasing the breadth of Rekor types supported by sigstore-python (https://github.com/sigstore/sigstore-python/issues/252), and I've been consulting the Rekor types documentation for the different types currently supported: https://github.com/sigstore/rekor/blob/main/types.md
As part of that, I've been trying to understand the relationship between those types and the schemas stored here: https://github.com/sigstore/rekor/tree/main/pkg/types
In particular, it looks like there are a couple of discrepancies between the two:

The types documentation mentions SSH and Minisign as separate types, but I can't find separate schemas for these.
The types documentation has TODO blocks for a couple of types that seem to be already schematized (like RPM).
The types documentation has payload examples for a few types, but these don't directly match the schemas.

Is there a more up-to-date reference somewhere?

@woodruffw I had questions around the types too, and got clarification in this Slack thread.
I think the main tl;dr is there are two types of "types" - pluggable types, and PKI types. Each pluggable type supports certain PKI types. SSH and Minisign (and all the types in rekor/pkg/pki) are PKI types. The types listed under rekor/pkg/types are pluggable types. The pluggable types have schemas, the PKI types don't. So, SSH and Minisign would not have schemas. The documentation page is out of date and so the pluggable and PKI types are mixed together on there.
I have a backburner goal to update the docs on this, but will not get to it before November in case anyone else wants to grab it.


Introduce finalize private field recipe

Issue: https://github.com/openrewrite/rewrite/issues/2611
Adds a recipe to add final modifier to private fields if the field is assigned only once somewhere in the class.
Rules: If a private field is assigned only once at the constructor, variable initializer, or initializer block, and there is no other assignment in the class, then it should be finalized.

Should this perhaps back off in the presence of Lombok annotations on the field or class? I'm thinking of @Data<EMAIL_ADDRESS>

Very good point! @timtebeek
I tested with Lombok annotations and it compiles if apply @Setter to a final field (please see the code below), I saw somewhere said Lombok ignores all final fields, I think that's the reason.
import lombok.Data;
import lombok.Setter;
import lombok.Value;

@Value
@Data
class A {
    @Setter
    private final int a = 1;
}

Not sure I fully understand; I would expect the recipe not to make any fields final if the class or field is annotated with @Data or @Setter, as before the recipe run Lombok would then generate setter methods, whereas after the recipe has run, those setters are then no longer generated, but might still be used elsewhere. This could lead to compilation errors for a fairly common use case if not handled in the recipe. What are your views on this?

Hi @timtebeek , thanks the explaining, this is the PR to address your comment, please take a review when you got a chance, thanks


Enhanced Documentation

Improve documentation with an explanation and detailed description about the utility of automated pentesting reports and how they can be used in day to day workflows.

You differentiate between CSPs and Operators and it's not entirely clear to me. Do you identify "Operators" with the staff that works in the Operations team of the CSP? But then their responsibility is a subset of the CSPs, no?

Yes, that was the main idea behind that.


Floating recipient bar shifts below when chatbox is swiped

Read our guide on bug reports here
Actual Behaviour

Expected Behaviour
It should not shift.
Would you like to work on the issue?
Yes

@zulipbot claim


Fix/improve cloning of TreeMap and TreeSet objects

This PR fixes the shallow clone of java.utils.TreeMap. Previously, the clone kept the reference to the same root node and in turn the whole tree. Modifications to the clone incorrectly also modified the original TreeMap. This update generates new tree nodes while keeping the key and value references, correctly making the clone independent.
Additionally, this PR includes a Clonable implementation for java.utils.TreeSet that follows the same logic of java.utils.HashSet.

FYI: I encourage contributors to use var in their PR


Stability of Cloud Communications permissions

Describe the issue
The Calls.* permissions are all marked as preview. However the Cloud Communications Calls APIs are mostly out of beta and under the 1.0 version of the Graph API. This means that if the permissions were dropped or changed it would be a breaking change. Following this logic the permissions must either be GA or remain preview and never change. The problem is that preview permissions cannot be requested using the https://login.microsoftonline.com/TENANT/v2.0/adminconsent endpoint as it appears, I'm getting the error scope calls.initiate.all [...] that doesn't exist on the resource 00000003-0000-0000-c000-000000000000. Administrators have to manually grant permissions in Azure Portal, which is a bad user experience.
Is there any better way to request the permissions, and why are they not GA?

Welp this was quick - good ol' classic. Removing v2.0 resolves the issue 🚀 😄


way to do dynamic css changes with text (either using or not using the main update)

          hmmm I still think there might be a need for some stuff in the main update. particularly dynamic CSS updates, which we can't detect changes to via events.

This is good for now but dynamic CSS changes is a table stakes requirement for V1.
Originally posted by @michaelthatsit in https://github.com/Volumetrics-io/mrjs/pull/561#discussion_r1552570812

Let's include CSS changes for all UI.
We can already do it with layout (width, position, height, etc). but long term we should target full support.


Request to log generated password

Would it be possible to save the generated root password to a file?

Not after the fact. During rooting you could always use tee to send the output to a file.

By the way, the script outputs a private key pair which you can use to log in.


Upgrade ghost

Got some code for us? Awesome 🎊!
Please include a description of your change & check your PR against this list, thanks!

[ ] Commit message has a short title & issue references
[ ] Commits are squashed
[ ] The build will pass (run npm test).

More info can be found by clicking the "guidelines for contributing" link above.

@goodslav I think you've pushed this to the wrong repo?

Wow sorry about that


Updated Makefile for global build

Adding makefile target according to https://goo.gl/v5jv1o

@sebastienvas: Adding do-not-merge/release-note-label-needed because the release note process has not been followed.

One of the following labels is required "release-note", "release-note-action-required", or "release-note-none".
Please see: https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md#write-release-notes-if-needed.
Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Accept Mol files without end tags in strictParsing=False mode?

Would it be too sloppy behavior to accept Mol files missing the proper end tag, when parsing with strictParsing=False? I encountered Mol files with "M END" with only one space instead of two specified in the format, which was annoyingly close... Also the "Why not to use MDL MOL file" manifest  [1] states the tag being sometimes absent.
After all, this tag does not carry any information, and on the other hand, is not a guarantee of correctness of the other content... Looking at the code, we could just skip the fileComplete check if strictParsing is set to False.
([1] if there just was an option...)

I think this is not necessary after all - closing...


Allow AWS_ENDPOINT_URL_

Describe the feature
Since there is already a support for AWS_ENDPOINT_URL  #1193, I think the sdk should support all  AWS_ENDPOINT_URL_<SERVICE> and other typical env vars.
https://docs.aws.amazon.com/cli/v1/userguide/cli-configure-envvars.html#envvars-list-AWS_ENDPOINT_URL_SERVICE
Use Case
I run local dynamo DB from the amazon container but emulate other services with the localstack.
Proposed Solution
No response
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

A note for the community

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue, please leave a comment

The SDK does support AWS_ENDPOINT_URL_<service>, can you provide an actual reproduction of something not working?

@aajtodd could you point me to the code that's responsible for this, it may be entirely possible I am a few versions behind because it is really difficult to keep the multiple versions in sync. I wish the entire SDK could have a single version to follow similarly to JS or Python.
I am using
name = "aws-sdk-dynamodb"
version = "1.20.0"

name = "aws-config"
version = "1.1.9"

but here I found that it should indeed work https://github.com/awslabs/aws-sdk-rust/blob/68dbb5933ca5a2ec74aedecf6d2909cf3f475fe9/sdk/aws-config/test-data/profile-provider/assume_role_override_service_env_url/env.json#L3
In my case
AWS_ENDPOINT_URL_DYNAMODB=http://localhost:5001 # does not work
AWS_ENDPOINT_URL=http://localhost:5001 # works

Okay I can confirm that, supports definitely AWS_ENDPOINT_URL_DYNAMODB
name = "aws-config"
version = "1.5.10"

@aajtodd thanks for the help. a bit OT, would you be able to guess what is preventing cargo update from updating AWS packages to the latest version? can it be any of the aws packages listed?


Private State Token issuer re-register - Amazon ads issuer

[re-registering issuer as the current registration will expire on 24th Sep 2024]
Summary: Private State Token Issuer Request - Amazon Ads issuer
Description: Amazon Ads issuer to detect invalid traffic
Issuer Name : Amazon Ads issuer
Origin : https://www.amazon.com/
Contact<EMAIL_ADDRESS>Key Commitment Endpoint URL: https://www.amazon.com/tt/k
Purpose: Invalid traffic detection
Disclosure & Acknowledgement:
I understand the technical restrictions on key rotation frequency of 60 days in the PST API.
I understand that my issuer registration will be valid for a period of six months after the key commitment is accepted, and that I will need to re-register in this repository following that six-month period.
I understand that in the future renewing my registration for this API may have additional requirements, to reduce the risk of abuse by token issuers.

pst-issuer

Updated the expiration for the keys.


Winston 3 two

Codecov Report

Merging #72 (0ea358a) into master (19fffb0) will increase coverage by 0.09%.
The diff coverage is 82.72%.

@@            Coverage Diff             @@
##           master      #72      +/-   ##
==========================================
+ Coverage   70.17%   70.27%   +0.09%
==========================================
  Files           5        7       +2
  Lines         694      703       +9
==========================================
+ Hits          487      494       +7
- Misses        207      209       +2

Impacted Files
Coverage Δ

lib/loggers/formatters.js
66.66% <66.66%> (ø)

lib/loggers/WorkerTransport.js
93.75% <93.75%> (ø)

lib/loggers/index.js
75.00% <94.28%> (ø)

lib/config.js
100.00% <100.00%> (ø)

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 19fffb0...0ea358a. Read the comment docs.


Add some additional bits to the manifest docs

@bors: r+ 5cf61206f2e4ae37ae39d4b8342880d5dad2dd53
Thanks!

:hourglass: Testing commit 5cf6120 with merge 818bfe5...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


Simplify a string of variables and numbers - vb

Hi.
Hope you can help me, with this little problem. I'm coding in visual basic and trying to find out, how to simplify a string of variables. But i'm stuck. Cant figure it out. It's most likely very easy to do, but I can't. :-(
Eg.
str = "a+a+a+2*(a+b)"
Can someone tell me, how I get a new string from the str that looks like "5a + 2b"?
Imports MathNet.Symbolics
function simplify(str as string) ' (str = "a+a+a+2*(a+b)")
dim simplified as string

---

---

return simplified
end function

Kind regards!

Imports expr = MathNet.Symbolics.SymbolicExpression
Module Program
    Sub Main(args As String())
        Dim e1 = expr.Parse("a+a+a+2*(a+b)")
        Console.WriteLine(e1.RationalSimplify("a"))
    End Sub
End Module

prints 5*a + 2*b

That did the trick. Now it is working. GREAT!! Thank you, @FoggyFinder.


add some functions and rewrite the layout.py

main.py
add search,tree widget initiation and sort by code functions
add indexes to series
changes the pastQ to 30 days earlier
changed the  "json" to pickle
layout.py
rewrite the layout and add some widgets:
rewrite the layout.py.
add some buttons like sort by codes, sort by industries and search
add search lineEdit

Awesome!


correct type inference for PhpStorm

supposedly, this will help other IDE's too

Coverage remained the same at 94.67% when pulling 05e29330abcdc8f55bcf529e86b92932b534186d on tiger-seo:master into 811bf974d66b1b0058003a8f32ba2fea495156e1 on mikey179:master.

Coverage remained the same at 94.67% when pulling 05e29330abcdc8f55bcf529e86b92932b534186d on tiger-seo:master into 811bf974d66b1b0058003a8f32ba2fea495156e1 on mikey179:master.

thank you :+1:


Same Class and Noise Addition

Implement additive data augmentation methods. These will be used to encourage the network to see more combined species samples, and to see more variations of noise/structure.
Methods:

[x] load multiple random noise samples
[x] load multiple random signal samples of the same class
[x] additively combine multiple samples of time-frequency data

Same Class
"We follow [14] and add sound files that correspond to the same class. Adding is a simple process because each sound file can be represented by a single vector. If one of the sound files is shorter than the other we repeat the shorter one as many times as it is necessary. After adding two sound files, we re-normalize the result to preserve the original maximum amplitude of the sound files. The operation describes the effect of multiple birds (of the same species) singing at the same time. Adding files improves convergence because the neural network sees more important patterns at once, we also found a slight increase in the accuracy of the system (see Table 1)." (Sprengel et al, 2016)
Adding Noise
"One of the most important augmentation steps is to add background noise. In Section 2.1 we described how we split each file into a signal and noise part. For every signal sample we can choose an arbitrary noise sample (since the background noise should be independent of the class label) and add it on top of the original training sample at hand. As for combining same class audio files, this operation should be done in the time domain by adding both sound files and repeating the smaller one as often as necessary. We can even add multiple noise samples. In our test we found that three noise samples added on top of the signal, each with a dampening factor of 0.4 produces the best results. This means that, given enough training time, for a single training sample we eventually add every possible background noise which decreases the generalization error." (Sprengel et al, 2016)

The methods for this is now implemented. It has not been thoroughtly tested, but by observing the resulting augmented signal with the signals used to augment it, it seems to be working just fine.


Enable dashboards-reports plugin configuration

Reference: https://www.elastic.co/guide/en/kibana/7.13/configuration-service.html
Add
opensearchDashboards.reporting.port
opensearchDashboards.reporting.host
opensearchDashboards.reporting.protocol
defaults are from server.host/port/

Additonal configurations we can considering adding is the csv report size or rows, currently it's a fixed value 10000 in reporting codebase
Related to #83 #34


Product page

Product pages are now working for URLs such as foo.com/Google+Bot. Any URL that includes a plus sign leads to a product page. A 404 response is returned for non-existent products (e.g. foo.com/Google+Xot].
Still to do:
• Add to cart
• Product image carousel
• Sizing, number in stock

Sam, can you please rebase to current master?

@devnook I've sorted out the merge problems.
I think we should stick with node import_products.js and node import_home.js in order to run and merge this branch, then work out what we want we want to do about import/export and data formats.


バージョン更新の修正

What does this implement/fix?

更新忘れしていたバージョン情報更新とauthorの追加をします。
Does this close any currently open issues?

しません。
How has this been tested?

テストはありません。
Any other comments?

Checklists

[ ]  I have read the CONTRIBUTING guidelines.

If there is no guideline in the repository, rt-net organization's guideline applies.

[ ]  I have checked to ensure there aren't other open Pull Requests for the same change.

LGTMです！


[mesheryctl] Add Meshery adapters to system context

Current Behavior
Users desire to only deploy specific Meshery adapters based on their service mesh needs.
Desired Behavior
The Meshery configuration file (config) describes contexts which define Meshery deployments: server location, adapters, release channel subscription, user tokens and so on.
While the deployment artifacts (Docker compose file, Kubernetes manifests, helm charts and so on) for Meshery will be in separate files, each context defined in the Meshery config file should carry the definition of which adapters are to be included in which Meshery deployment.
Note:

_Refer to the system context section of Meshery CLI Commands & Documentation
This issue only describes itemizing adapters in contexts. A subsequent issue will describe how the presence or absence of an adapter in a context affects a Meshery deployment.
This issue is dependent upon #2104

Implementation

Include a list of Meshery adapters in each context.
Provide a --adapters [istio | linkerd | osm | ... ] flag to accept 1 or more adapter names.
Provide -a as shorthand for --adapters.

Acceptance Tests

config file is valid yaml.
Any given context includes includes an adapters collection with 0 or more adapters defined.
system context create <name> that does not include --adapters flag will create a context that includes all adapters.

Future:
Acceptance Tests

An empty adapters collection results in no adapters provisioned.

@sladyn98 interested?

@leecalcote I can take this up if no one is working on this.

@leecalcote I can take this up if no one is working on this.

@navendu-pottekkat I was looking to start contributing in Go. Could I consider this issue for the same? Is it solvable by someone with an intermediate knowledge of Go?

The new context command enables users to adjust which adapters they want to have running. While this command doesn't provide the flag listed above (--adapters), yet, Meshery's docs need to be updated to provide users a description of the config.yaml(meshconfig) file and how to update it.

@leecalcote @metonymic-smokey Adding the --adapters flag should be straightforward. That can be done now I think...

@navendu-pottekkat , this is what I've done so far:

Used pflag for the same and it accepts 1 or more string values.
I understood this flag will mainly be used with the context command. Could you please give me some idea on what I should implement next to include these adapters in the config.yaml file?
Thanks!

I guess the flag would be used in the context create command. So this flag could be local to it. We can modify the context create function to write the list of adapters specified to the specified context when it is created.

@navendu-pottekkat ,
Thanks for the suggestion!
I've changed it to a flag local to system context create.

Thanks!

@metonymic-smokey Great work! Are you writing the specified adapters to the config.yaml file?
@leecalcote This is the desired behaviour right?

I guess the flag would be used in the context create command. So this flag could be local to it. We can modify the context create function to write the list of adapters specified to the specified context when it is created.

Sounds good. If a user wants to modify their context, they can either delete the context and recreate or they can edit the config.yaml.

@navendu-pottekkat , @leecalcote ,
I've opened a draft PR for this issue.
I'm unsure as to why my tests are failing.  Could you please let me know how to fix those?
These are the screenshots of my changes:
1.

Please review it and let me know how I should proceed.
Thanks!


Delete unsued import and add package name

Coverage remained the same at 89.188% when pulling 241262b4b39d213fc2d5620cadde33d612aa8da2 on s0xzwasd:master into 4d1628e5e24835bb0fd34eeafe480387b836fa23 on go-ozzo:master.

Coverage remained the same at 89.188% when pulling 241262b4b39d213fc2d5620cadde33d612aa8da2 on s0xzwasd:master into 4d1628e5e24835bb0fd34eeafe480387b836fa23 on go-ozzo:master.


Emit source of truth errors to subscribers #177

This PR fixes an issue where if SourceOfTruth's read or write method fails, we would just crash.
Now SourceOfTruth has two new exception types, WriteException and ReadException where each exception is sent to downstream as a StoreResponse.Error.Exception and stream is kept alive.
If a write fails, reader is simply restarted (because it was previously stopped to prepare for the write).
If a read fails, reader is not restarted until another write attempt happens. This is don't to avoid creating an infinite loop of trying to read from disk.

oh wait sth is wrong i think i need to rebase w/ master

oh i missed the branch name change (:fist_raised: ), rebased over main now, will push.


Auto-collapse when a different panel is opened

This is a feature request rather than a bug. It would be great if a panel will automatically collapsed when another panel is opened.

+1

@dekyfin If you know a little JS, it's pretty easy to implement.


Set message to AuthenticatorData

Set message to AuthenticatorData

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Hasanthi Dissanayake seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[3.x] Fix AnimatedSprite normal map loading

Normal map names are now correctly set up during loading.
Fixes #43435
Notes

Had never spotted this issue before, only took a couple mins to track down.
It's not really rendering, just a bit of code missing in the AnimatedSprite.

Thanks!

Cherry-picked for 3.5.3.


lseek system call does not support SEEK_DATA or SEEK_HOLE

Build Number: Microsoft Windows [Version 10.0.18362.535]
Issue: The code snippet below creates a sparse file containing holes. The calls to lseek using SEEK_DATA and SEEK_HOLE fail with error EINVAL (Invalid argument), although these additional values should be supported since Linux version 3.1.
Is this a limitation associated with current WSL support for sparse files? Seems to be related to issues #1321 and #1886. It appears that this would cause problems for software that use SEEK_DATA and SEEK_HOLE for processing sparse file. As described in the man page for lseek:

In the simplest implementation, a filesystem can support the operations by making SEEK_HOLE always return the offset of the end of the file, and making SEEK_DATA always return offset (i.e., even if the location referred to by offset is a hole, it can be considered to consist of data that is a sequence of zeros).

Code (test_hole.c):
#define _GNU_SOURCE

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <fcntl.h>
#include <errno.h>

#define HOLE_SIZE 100
#define PATTERN "teststring"
#define PATTERN_SIZE 10

int main(int argc, char** argv) {
    int fd;
    off_t offset = 0;

    // open file (rw-------)
    fd = open(argv[1], O_WRONLY | O_CREAT | O_EXCL, S_IRUSR | S_IWUSR);
    if (fd == -1) {
        perror("open");
        exit(EXIT_FAILURE);
    }

    // write PATTERN to open file; seek past EOF and rewrite PATTERN
    // should create a file hole with size HOLE_SIZE
    write(fd, PATTERN, PATTERN_SIZE);
    offset += PATTERN_SIZE + HOLE_SIZE;
    lseek(fd, offset, SEEK_SET);
    write(fd, PATTERN, PATTERN_SIZE);

    // errno: EINVAL (Invalid argument)
    // should return 0 (the file contains real data at offset 0)
    if (lseek(fd, 0, SEEK_DATA) == -1) {
        perror("SEEK_DATA");
    }

    // errno: EINVAL (Invalid argument)
    // should return 10 (next hole at offset 10)
    if (lseek(fd, 0, SEEK_HOLE) == -1) {
        perror("SEEK_HOLE");
    }

    close(fd);

    exit(EXIT_SUCCESS);
}

Running ./test_hole test returns:
$ ./test_hole test
SEEK_DATA: Invalid argument
SEEK_HOLE: Invalid argument

The actual contents of the file are as expected:
$ cat test
teststring                                                                                teststring

The relevant strace:
openat(AT_FDCWD, "test", O_WRONLY|O_CREAT|O_EXCL, 0600) = 3
write(3, "teststring", 10)              = 10
lseek(3, 110, SEEK_SET)                 = 110
write(3, "teststring", 10)              = 10
lseek(3, 0, SEEK_DATA)                  = -1 EINVAL (Invalid argument)
dup(2)                                  = 4
fcntl(4, F_GETFL)                       = 0x402 (flags O_RDWR|O_APPEND)
brk(NULL)                               = 0x7fffd107d000
brk(0x7fffd109e000)                     = 0x7fffd109e000
fstat(4, {st_mode=S_IFCHR|0660, st_rdev=makedev(4, 1), ...}) = 0
ioctl(4, TCGETS, {B38400 opost isig icanon echo ...}) = 0
write(4, "SEEK_DATA: Invalid argument\n", 28SEEK_DATA: Invalid argument
) = 28
close(4)                                = 0
lseek(3, 0, SEEK_HOLE)                  = -1 EINVAL (Invalid argument)
dup(2)                                  = 4
fcntl(4, F_GETFL)                       = 0x402 (flags O_RDWR|O_APPEND)
fstat(4, {st_mode=S_IFCHR|0660, st_rdev=makedev(4, 1), ...}) = 0
ioctl(4, TCGETS, {B38400 opost isig icanon echo ...}) = 0
write(4, "SEEK_HOLE: Invalid argument\n", 28SEEK_HOLE: Invalid argument
) = 28
close(4)                                = 0
close(3)                                = 0
exit_group(0)                           = ?

Thanks for the excellent report.

Is this a limitation associated with current WSL support for sparse files?

Without having actually run your code, almost certainly.
De facto landing zone #1732. Pragmatic solution WSL2. On WSL2 your test case will work on /home/you/test, which is just a normal ext4 filesystem. It would be mildly interesting to try /mnt/c/Users/you/test (the 9p fs) too and see if it works.


Make cookie optional, use own cookie

Environment
I am running the latest docker image, with Livebook 0.4.1
Current behavior
When I want to connect to a remote node, I have to supply the cookie.
In my case the image is started with the same cookie, so I would love to be able to leave it blank
Expected behavior
Take my cookie if I leave the field empty.

We discussed this some time ago and the consensus was to prefer the cookie being given within the UI because it is more generally applicable and it is safer too (whatever is in your terminal goes to history, for example). So my suggestion is to get used to passing it on the UI instead of the terminal. :) We can revisit this in the future though.

So if I get it correct, I do not supply a cookie on the command line (env var in docker) and when I supply it in the UI, that cookie will be used?

Correct!

There is a difference however. If I do not supply the cookie then the other nodes in my cluster show a connection error.
Dec 30 09:31:33 api gratwifi[799093]: 09:31:33.567 [error] ** Connection attempt from node :"livebook@xxx" rejected. Invalid challenge reply. **
Dec 30 09:31:33 api gratwifi[799093]: 09:31:33.567 [warning] global: :"gratwifi@xxx" failed to connect to :"livebook@xxx"

Yes, you would have to connect each node individually. This can be a good thing (you don’t make all nodes aware of Livebook) or bad (if you want all nodes it needs to be done manually).

One last thing, disconnecting does not disconnect.  In the Livebook UI the connection is gone, but in the live dashboard it's still there, also when I do a Node.list on the remote. When I connect to the Livebook node itself, and do a Node.list it's still there.

I will open up a new issue.


I wan to sort the tableView itens to make the last post be the first item in the table

Hi,
First of all congrats for the awsome lib!
I'm building something over the Database Example in the quickstart-ios repo on swift:
https://github.com/firebase/quickstart-ios/tree/master/database
I wan to sort the tableView itens to make the last post be the first item in the table
something like thus guy in stackoverflow wants but a little less complex, just the regular reverse order:
http://stackoverflow.com/questions/26148367/swift-start-tableview-from-bottom-reverse-tableview
In my search I've found this method:
// reverse the array
self.objects = self.objects.reverse()
but in the firebaseUI have something different when we'r using the table view (or I'm just lost..)
We have FirebaseUI to build the tableView using (and my interpretation of what each do, please correct me if I'm wrong):
PostDataSource.swift  (configure the table view)
PostListViewController.swift (build the basic table)
PostTableViewCell.swift (cell for reuse in may views)
And the specific query to build the dataset for the table:
MyPostsViewController.swift (get the specific dataset array or dict to populate the table)
I've trie to insert the .reverse() in may objects at these files but no one seems to accept it.
So I'm requesting some help here to sort items using the FirebaseUI implementation on tableViews because I really liked the way it builds the tables :)

I wanted to do the exact same a couple days ago, and solved it with the help of Frank van Puffelens answer here.
Basically what you want to do is adding a priority property when you write to the database, and when you retrieve them you retrieve them by priority.

Added in #186.


Make used NSBundle overridable

When using ObjectiveRecord as a dynamic framework, NSBundle.bundleForClass(self) returns the framework's bundle which then, in turn, won't yield the correct CoreData model file. By overriding the bundle when using CoreDataManager this can be fixed.

This seems to have been abandoned. But i agree with this. I use ObjectiveRecord within a private cocoa pod and the fetching of the ManagedObjectModel needs to be done within a bundle within a framework within the main bundle.
The travis issue if i remember correctly has to do with Travis changing the ruby version on their OS X images.


Set up a feature flag for Go support

Support for Go is still experimental and needs review. We should gate the CLI's Go generation behind a feature-flag until we're confident in our implementation.

Hey! Can you make a list of tasks that need to be done to stabilize Go support? @InquisitivePenguin


Causes certain pages to not load properly

A perfect example of this is https://example.com
DevTools failed to load SourceMap: Could not load content for chrome-extension://hhhpdkekipnbloiiiiaokibebpdpakdp/content_script.js.map: HTTP error: status code 404, net::ERR_UNKNOWN_URL_SCHEME

from the chrome console,

on firefox it complains about two css rules.

I found out about this when one of my friends who uses this extension, visited my site, https://merith.tk, which caused it to load improperly. We determined it was this extension by them disabling all others until the page was working as intended.
The issue appears to be that saka injects some HTML into the webpage, which appears to attempt to load a resource from the chrome extension, which last i was aware (i may be wrong) was not allowed by chrome for security purposes

The extension also fails to run on these pages,

It seems like the <div> tags generated by saka-key are having their style overridden by the CSS on the page, causing them to appear solid and prominent above other content. There may be a need to re-style saka-key content specifically to undo non-default CSS settings, but I worry that that might be a lot of effort.


Add better error for missing app key

I wanted to make a call today, authenticated with the API key and secret only, and when I made the call I received the following error:
*** TypeError Exception: no implicit conversion of nil into String

No line number, nothing. After much debugging I realised this method needs the App Key and password as well. A better error would have made this a lot easier.

@cbetta You mean the application_id and private_key for the Voice API? Either way I agree, it would be good to have clearer errors for all the different types of credentials 👍

@timcraft I meant that I should have authenticated with the application_id and private_key but didn't, and that instead of getting a useful error got the error above.

Added in 8ea3458 (available in v4.7.0)


Test suite doesnt work with dependencies

Expected Behavior
Simple jest tests should work when there are project dependencies that use ESM syntax.
Actual Behavior
If any added dependency has ESM exports, jest tests will fail and complain about unexpected tokens
FAIL src/utils.test.ts
  ● Test suite failed to run

    Jest encountered an unexpected token

    This usually means that you are trying to import a file which Jest cannot parse, e.g. it's not plain JavaScript.

    By default, if Jest sees a Babel config, it will use that to transform your files, ignoring "node_modules".

    Here's what you can do:
     • If you are trying to use ECMAScript Modules, see https://jestjs.io/docs/en/ecmascript-modules for how to enable it.
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/en/configuration.html

    Details:

    /Users/seanaye/dev/coparse/filesystem/node_modules/idb/with-async-ittr.js:1
    ({"Object.<anonymous>":function(module,exports,require,__dirname,__filename,global,jest){export * from './build/esm/index.js';
                                                                                ^^^^^^

    SyntaxError: Unexpected token 'export'

      1 | import { FileHandle, FileSource, FileSystemFile, LegacyFile } from "./file";
    > 2 | import { DBSchema, IDBPDatabase, openDB } from "idb/with-async-ittr.js";

I have tried all of the things suggested by the error message but nothing works.
Steps to Reproduce the Problem

Add a project dependency with ESM exports in my case this is idb
Try to run a jest test in which this dependency is imported
Test failure

Specifications

Version: 0.4.2
Platform: MacOS, Node 16.1

+1
Steps to reproduce

Clone this repo.
Add dependencies: yarn add rehype-stringify remark-gfm remark-parse remark-rehype unified
Add utility function and corresponding test:// src/format.ts
import rehypeStringify from "rehype-stringify";
import remarkGfm from "remark-gfm";
import remarkParse from "remark-parse";
import remarkRehype from "remark-rehype";
import { unified } from "unified";

export const format = (markdown: string) =>
  unified()
    .use(remarkParse)
    .use(remarkGfm)
    .use(remarkRehype)
    .use(rehypeStringify)
    .processSync(markdown)
    .toString();

// src/format.test.ts
import { format } from "./format";

describe("format", () => {
  it("should parse markdown", () => {
    expect(format("# Headline")).toBe("
Headline
");
  });
});

Run yarn test and see error output: FAIL  src/format.test.ts
  ● Test suite failed to run

    Jest encountered an unexpected token

    This usually means that you are trying to import a file which Jest cannot parse, e.g. it's not plain JavaScript.

    By default, if Jest sees a Babel config, it will use that to transform your files, ignoring "node_modules".

    Here's what you can do:
     • If you are trying to use ECMAScript Modules, see https://jestjs.io/docs/en/ecmascript-modules for how to enable it.
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/en/configuration.html

    Details:

    /typescript-boilerplate/node_modules/remark-gfm/index.js:6
    import {gfm} from 'micromark-extension-gfm'
    ^^^^^^

    SyntaxError: Cannot use import statement outside a module

    > 1 | import remarkGfm from "remark-gfm";
        | ^
      2 | import remarkParse from "remark-parse";
      3 | import remarkRehype from "remark-rehype";
      4 | import { unified } from "unified";

      at Runtime.createScriptFromCode (node_modules/jest-runtime/build/index.js:1350:14)
      at Object.<anonymous> (src/format.ts:1:1)

Test Suites: 1 failed, 1 passed, 2 total

Weep.