Saving Model

Are you able to save the sparse_model? I receive the following error.
`
KeyError                                  Traceback (most recent call last)
/tmp/ipykernel_3990/100629554.py in
----> 1 sparse_model.save('ECG-peak-detection.tf')
~/.local/lib/python3.8/site-packages/keras/utils/traceback_utils.py in error_handler(*args, **kwargs)
65     except Exception as e:  # pylint: disable=broad-except
66       filtered_tb = _process_traceback_frames(e.traceback)
---> 67       raise e.with_traceback(filtered_tb) from None
68     finally:
69       del filtered_tb
~/.local/lib/python3.8/site-packages/tensorflow/python/saved_model/function_serialization.py in serialize_concrete_function(concrete_function, node_ids, coder)
65       bound_inputs.append(node_ids[capture])
66   except KeyError:
---> 67     raise KeyError(
68         f"Failed to add concrete function '{concrete_function.name}' to object-"
69         f"based SavedModel as it captures tensor {capture!r} which is unsupported"
KeyError: "Failed to add concrete function 'b'__inference_train_step_end_194526'' to object-based SavedModel as it captures tensor <tf.Tensor: shape=(), dtype=resource, value=> which is unsupported or not reachable from root. One reason could be that a stateful object or a variable that the function depends on is not assigned to an attribute of the serialized trackable object (see SaveTest.test_captures_unreachable_variable)."
`

For saving, currently you can't do it directly from the sparse model. I recommend saving the dictionary weights(dictionary.w) with tf.io.write_file or simply convert to numpy and save for now.

Can the dictionary simply be used to instantiate the model then when loading it?
Best,
Ethan
From: Isamu Isozaki @.>
Reply-To: isamu-isozaki/sparse_mnist @.>
Date: Tuesday, February 22, 2022 at 2:55 PM
To: isamu-isozaki/sparse_mnist @.>
Cc: Ethan Moyer @.>, Author @.***>
Subject: Re: [isamu-isozaki/sparse_mnist] Saving Model (Issue #1)
For saving, currently you can't do it directly from the sparse model. I recommend saving the dictionary weights(dictionary.w) with tf.io.write_file or simply convert to numpy and save for now.
—
Reply to this email directly, view it on GitHubhttps://github.com/isamu-isozaki/sparse_mnist/issues/1#issuecomment-1048158883, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANW7B24VBZHTWSUFAJ45N6TU4PS43ANCNFSM5O7U6FZQ.
You are receiving this because you authored the thread.Message ID: @.***>

yup the dictionary is the only important part of the model


Fix pdf & implement round_to for cdf() -> calculate_cdf() & add cdf attrib

In this PR i fixed the pdf and add the round_to arg for calculate_cdf() this is the new name for calculating cdf. and also create base attribute as self.cdf so dont use cdf as a function on subclasses.

bro please accept this pr before midnight 12

Fantastic! I love you how you always end up doing more than what was promised! Loved the changes to code style, and the bug fixes, as well as everything else! I've added the hacktoberfest-accepted tag as well so I hope you complete the Hacktoberfest event!
Also, I read your comment in the other PR, and to reply to that, yes I did mean dropdown list. :)
Thanks a lot for your contribution mate! Excellent job!

bro so far i create  7 PRs but when it will review by hacktober  team? my first pr time running as mature in: 9d:1h:41m:43s

@mhdzumair The review isn't by hacktoberfest team, it is by the repository maintainers, for example me. I've already reviewed all the PRs you made here and accepted all of them. Secondly, any PRs you made to non-spam repos(Like this one) will definitely count so there's no need to worry about hacktoberfest disqualifying your PRs.
Now you just have to wait for the timer to finish. Ie, after 9 days your PR becomes mature :)

I checked your profile, I see you completed your 4 PRs on 27th October.
Last year people who completed their PRs in the 3rd week of October got the prize too, this year the number of shirts increased from 50k to 75k, so I think you still have a great chance bro :)

@mhdzumair The review isn't by hacktoberfest team, it is by the repository maintainers, for example me. I've already reviewed all the PRs you made here and accepted all of them. Secondly, any PRs you made to non-spam repos(Like this one) will definitely count so there's no need to worry about hacktoberfest disqualifying your PRs.
Now you just have to wait for the timer to finish. Ie, after 9 days your PR becomes mature :)

Ohh, thanks for your information bro. I thought hacktober team need to review or something and also i pushed my first 4 pr to funny repo. in that repo we can push our mini projects haha. So i thought it will not count as pr by hacktober team.

I checked your profile, I see you completed your 4 PRs on 27th October.
Last year people who completed their PRs in the 3rd week of October got the prize too, this year the number of shirts increased from 50k to 75k, so I think you still have a great chance bro :)

Yeah, let see what happen after date period end. I'm glad to work with your project.

@mhdzumair Sometimes, funny repos (Spam repos) can be banned by Hacktoberfest thereby making all PRs submitted to it invalid. However, in your case I don't think it was a spam repo so you should hopefully be fine! I hope you win the prize and hope to work with you in the future too! (I'll be shamelessly pinging you again in the future whenever I run into rounding issues hahah :P)
Also, you can connect with me on discord as well :)
My discord id is hot9cups#3125
Cheers bro

@mhdzumair Seeing how helpful you have been in the developing of the library, not only improving the code but also helping others in their PRs, I would like to ask you if you would like to become the co-owner of the library with me? Because you've been working so earnestly and diligently on this, I think you deserve a lot more credit :)
If so, can we connect on discord(or mail<EMAIL_ADDRESS>to discuss this further ? :)


Bug in drag and drop

https://github.com/ultimatepp/ultimatepp/blob/master/uppsrc/Core/Defs.h#L157
typedef uint32             wchar;

https://github.com/ultimatepp/ultimatepp/blob/master/uppsrc/CtrlCore/CtrlClip.cpp#L118
String Unicode__(const WString& w)
{
	return String((const char *)~w, 2 * w.GetLength());
}

wchar is 4 bytes long so string will hold only one first characted.
How to reproduce. Create textedit type some text, select several letters and move it with mouse. It moves only first letter, rest will disappear.
ps:
https://github.com/ultimatepp/ultimatepp/blob/master/uppsrc/plugin/lz4/lib/lz4.c#L1348
static size_t LZ4_stream_t_alignment(void)
{
    struct { char c; LZ4_stream_t t; } t_a;
    return sizeof(t_a) - sizeof(t_a.t);
}

should be replaced with
static size_t LZ4_stream_t_alignment(void)
{
    struct { char c; int* t; } t_a;
    return sizeof(t_a) - sizeof(t_a.t);
}

in order to work with gcc, or it fall's on aligment check nearby  an cause segav

Should be now fixed (by removing Unicode__ functions completely). Thanks
for spotting this.
pá 8. 4. 2022 v 23:07 odesílatel kov-serg @.***> napsal:

https://github.com/ultimatepp/ultimatepp/blob/master/uppsrc/Core/Defs.h#L157
typedef uint32 wchar;
https://github.com/ultimatepp/ultimatepp/blob/master/uppsrc/CtrlCore/CtrlClip.cpp#L118
String Unicode__(const WString& w)
{
return String((const char *)~w, 2 * w.GetLength());
}
—
Reply to this email directly, view it on GitHub
https://github.com/ultimatepp/ultimatepp/issues/69, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARH235LHWGSNOUJKU2TXF3VECNZ5ANCNFSM5S5UQZEA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Did you check result?


LineView: Turning off 'Use Fade Effect' makes Continue button stop working after first use

What is the current behavior?

When 'Use Fade Effect' is turned off in the LineView inspector, the Continue button is dimmed and disabled after clicking it for the first time.
Please provide the steps to reproduce, and if possible a minimal demo of the problem:

Create a new, empty project, with Yarn Spinner installed.
Add the Dialogue System to the empty scene.
Create a new Yarn Script and Yarn Project. Add at least two lines to the Yarn Script.
Select the Line View, and turn off Use Fade Effect.
Play the scene. Observe the first line, and that the Continue button is clickable.
Click the Continue button. Observe that the Continue button is now not clickable, and it is not possible to proceed beyond the second line of dialogue.

What is the expected behavior?

The continue button should remain clickable.
Please tell us about your environment:

Yarn Spinner Version: 2.1.0
Unity Version: 2020.3.20

Other information

This issue is also reproducible when Use Typewriter Effect is also turned off.
Behaviour is normal when Use Fade Effect is left on.

Can confirm.  Moving the continue button so it is no longer a child of the Line View is a work around.


fix(images): replace breaking : character in dummy image urls

Related Ticket(s)
React NextJS page: All the images are not loading properly on solution page. #6691
Description
The : character in the dummy urls were causing the images to break, replacing : with x in the text parameter of the url fixes the issue.
BEFORE:

AFTER:

Changelog
Changed

change the : to x in the dummy urls (ie. https://dummyimage.com/672x378/ee5396/161616&text=16:9 to https://dummyimage.com/672x378/ee5396/161616&text=16x9)

lol it was all @annawen1! thanks @annawen1!


ToggleButtons states can't report selected AND disabled at the same time

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Create a ToggleButtons with 2 children and one selected child and with a null onPressed
Use MaterialStateColor.resolveWith for the fillColor and return any Color and print the current states

Expected results
For the first child, the print output should be {MaterialState.disabled, MaterialState.selected}, whereas the second should be {MaterialState.disabled}.
Actual results
The output print is {MaterialState.disabled}!
Code sample
 Widget build(BuildContext context) {
  return ToggleButtons(
    isSelected: [true, false],
    onPressed: null,
    fillColor: MaterialStateColor.resolveWith((states) {
        print(states);
        return Colors.green;
    }),
    children: [
        Text('TB1'),
        Text('TB2'),
    ],
  );
}

Analysis
When computing the current states https://github.com/flutter/flutter/blob/ce9a604a3bad2c56067d8fac2a5251d17c312fbf/packages/flutter/lib/src/material/toggle_buttons.dart#L700-L703
MaterialState.disabled and MaterialState.selected are mutually exclusive because they use both onPressed but with opposite condition.
Proposal to fix the issue:
To set Material.selected state, the code should not look to onPressed but only to isSelected[index].
final Set<MaterialState> states = <MaterialState>{
     if (isSelected[index] MaterialState.selected,
     if (onPressed == null) MaterialState.disabled,
 };

Screenshots or Video
No response
Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

Thanks for the detailed report and potential fix. The current output is:
I/flutter (22097): {}
I/flutter (22097): {MaterialState.disabled}
I/flutter (22097): {MaterialState.disabled}

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.10.5, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.10.5 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 796c8ef792 (2 days ago), 2023-06-13 15:51:02 -0700
    • Engine revision 45f6e00911
    • Dart version 3.0.5
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

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

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

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

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.12.0-15.0.pre.59, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.12.0-15.0.pre.59 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (4 hours ago), 2023-07-09 21:01:34 -0400
    • Engine revision 09c6ce4243
    • Dart version 3.1.0 (build 3.1.0-285.0.dev)
    • DevTools version 2.25.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

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

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

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

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

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

! Doctor found issues in 1 category.

SegmentedButton.style should make this possible.
          ToggleButtons(
            fillColor: MaterialStateColor.resolveWith((states) {
              if (states.contains(MaterialState.disabled) &&
                  states.contains(MaterialState.selected)) {
                return Colors.amber;
              }
              return Colors.transparent;
            }),

Code sample

expand to view the code sample
import 'package:flutter/material.dart';

/// Flutter code sample for migrating from [ToggleButtons] to [SegmentedButton].

void main() {
  runApp(const ToggleButtonsApp());
}

class ToggleButtonsApp extends StatelessWidget {
  const ToggleButtonsApp({super.key});
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(useMaterial3: true),
      home: const Scaffold(
        body: ToggleButtonsExample(),
      ),
    );
  }
}

enum ShirtSize { extraSmall, small, medium, large, extraLarge }

const List<(ShirtSize, String)> shirtSizeOptions = <(ShirtSize, String)>[
  (ShirtSize.extraSmall, 'XS'),
  (ShirtSize.small, 'S'),
  (ShirtSize.medium, 'M'),
  (ShirtSize.large, 'L'),
  (ShirtSize.extraLarge, 'XL'),
];

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

  @override
  State<ToggleButtonsExample> createState() => _ToggleButtonsExampleState();
}

class _ToggleButtonsExampleState extends State<ToggleButtonsExample> {
  final List<bool> _toggleButtonsSelection =
      ShirtSize.values.map((ShirtSize e) => e == ShirtSize.medium).toList();
  Set<ShirtSize> _segmentedButtonSelection = <ShirtSize>{ShirtSize.medium};

  @override
  Widget build(BuildContext context) {
    return Center(
      child: Column(
        mainAxisAlignment: MainAxisAlignment.center,
        children: <Widget>[
          const Text('ToggleButtons'),
          const SizedBox(height: 10),
          // This ToggleButtons allows multiple or no selection.
          ToggleButtons(
            fillColor: MaterialStateColor.resolveWith((states) {
              if (states.contains(MaterialState.disabled) &&
                  states.contains(MaterialState.selected)) {
                return Colors.amber;
              }
              return Colors.transparent;
            }),
            // ToggleButtons uses a List<bool> to track its selection state.
            isSelected: _toggleButtonsSelection,
            // This callback return the index of the child that was pressed.
            onPressed: (int index) {
              setState(() {
                _toggleButtonsSelection[index] =
                    !_toggleButtonsSelection[index];
              });
            },
            // Constraints are used to determine the size of each child widget.
            constraints: const BoxConstraints(
              minHeight: 32.0,
              minWidth: 56.0,
            ),
            // ToggleButtons uses a List<Widget> to build its children.
            children: shirtSizeOptions
                .map(((ShirtSize, String) shirt) => Text(shirt.$2))
                .toList(),
          ),
          const SizedBox(height: 20),
          const Text('SegmentedButton'),
          const SizedBox(height: 10),
          SegmentedButton<ShirtSize>(
            style: ButtonStyle(
              backgroundColor: MaterialStateProperty.resolveWith((states) {
                if (states.contains(MaterialState.disabled) &&
                    states.contains(MaterialState.selected)) {
                  return Colors.amber;
                }
                return Colors.transparent;
              }),
            ),
            // ToggleButtons above allows multiple or no selection.
            // Set `multiSelectionEnabled` and `emptySelectionAllowed` to true
            // to match the behavior of ToggleButtons.
            multiSelectionEnabled: true,
            emptySelectionAllowed: true,
            // Hide the selected icon to match the behavior of ToggleButtons.
            showSelectedIcon: false,
            // SegmentedButton uses a Set<T> to track its selection state.
            selected: _segmentedButtonSelection,
            // This callback updates the set of selected segment values.
            onSelectionChanged: null,
            // onSelectionChanged: (Set<ShirtSize> newSelection) {
            //   setState(() {
            //     _segmentedButtonSelection = newSelection;
            //   });
            // },
            // SegmentedButton uses a List<ButtonSegment<T>> to build its children
            // instead of a List<Widget> like ToggleButtons.
            segments: shirtSizeOptions
                .map<ButtonSegment<ShirtSize>>(((ShirtSize, String) shirt) {
              return ButtonSegment<ShirtSize>(
                  value: shirt.$1, label: Text(shirt.$2));
            }).toList(),
          ),
        ],
      ),
    );
  }
}

@eric-taix
After taking a look at the material specs and behavior in Android implementation.  It looks like TogglesButtons or even SegmentedButtons isn't supposed to have disabled and selected state color.
That being said, given SegmentedButtons already supports this using the ButtonStyle property, this could be a Flutter-specific quality feature for ToggleButtons as well that we could add in the future.
If we do this, we also have to consider if the default selected color should be visible in the disabled and selected state. It might produce unexpected results for people who don't expect any color there.
This isn't a bug but a quality feature so updating labels as such.


Unable to generate speech

Thank you for your great job!
It can not genertate speech when I finish training your speech model. I trained your speech model to mel and trained the hifigan as vocoder for mel to wav. The hifigan generate audio without problems. Could you please provide the speech training model? Thanks!

Thank you for having an interest in our work!
We're sorry, but we are not planning to upload trained models. And, the focus of our paper is not on achieving SOTA quality but on comparing our method, SQ-VAE, with conventional VQ-VAE. For comparison, we used a simple architecture rather than sophisticated ones in experiments. The fidelity of speech generated from the implemented model is not satisfactory, anyway.


NPE at SwarmMemberAddressProvider line 237

Hello,
We're almost there with SMAP, but we now get a NPE at SwarmMemberAddressProvider line 237.
It looks like SwarmDiscoveryUtil.myAddress is probably null at that point.  We have no indication that discoverSelf() has been executed, or has executed successfully, as it does not log much.
Feb 27 12:13:18 CSINTFINSDCKR03 springboot-admin[912]: 2019-02-27 12:13:18.446  WARN 1 --- [           main] c.h.config.AbstractXmlConfigHelper       : Name of the hazelcast schema location is incorrect, using default
Feb 27 12:13:18 CSINTFINSDCKR03 springboot-admin[912]: 2019-02-27 12:13:18.734  INFO 1 --- [           main] o.b.h.d.d.s.SwarmMemberAddressProvider   : SwarmMemberAddressProvider.initialize() passed properties: dockerNetworkNames:spring_svc_network dockerServiceLabels:null dockerServiceNames:springboot-admin swarmMgrUri:http://springboot-docker-proxy:2375 skipVerifySsl:false rawHazelcastPeerPort:5701
Feb 27 12:13:19 CSINTFINSDCKR03 springboot-admin[912]: 2019-02-27 12:13:19.229  INFO 1 --- [           main] o.b.h.d.docker.swarm.SwarmDiscoveryUtil  : discoverNodes(): via DOCKER_HOST: springboot-docker-proxy
Feb 27 12:13:19 CSINTFINSDCKR03 springboot-admin[912]: docker-network-names = spring_svc_network
Feb 27 12:13:19 CSINTFINSDCKR03 springboot-admin[912]: docker-service-names = springboot-admin
Feb 27 12:13:19 CSINTFINSDCKR03 springboot-admin[912]: docker-service-labels = null
Feb 27 12:13:19 CSINTFINSDCKR03 springboot-admin[912]: swarmMgrUri = http://springboot-docker-proxy:2375
Feb 27 12:13:19 CSINTFINSDCKR03 springboot-admin[912]: skipVerifySsl = false
Feb 27 12:13:19 CSINTFINSDCKR03 springboot-admin[912]:
Feb 27 12:13:19 CSINTFINSDCKR03 springboot-admin[912]: 2019-02-27 12:13:19.917  INFO 1 --- [           main] o.b.h.d.docker.swarm.SwarmDiscoveryUtil  : Found relevant docker network: spring_svc_network[z08r63wj5kk1s9plt28x98k0u]
Feb 27 12:13:20 CSINTFINSDCKR03 springboot-admin[912]: 2019-02-27 12:13:20.108 ERROR 1 --- [           main] com.hazelcast.instance.AddressPicker     : [LOCAL] [spring-boot-admin] [3.11.2] null
Feb 27 12:13:20 CSINTFINSDCKR03 springboot-admin[912]:
Feb 27 12:13:20 CSINTFINSDCKR03 springboot-admin[912]: java.lang.NullPointerException: null
Feb 27 12:13:20 CSINTFINSDCKR03 springboot-admin[912]: #011at org.bitsofinfo.hazelcast.discovery.docker.swarm.SwarmMemberAddressProvider.getBindAddress(SwarmMemberAddressProvider.java:237) ~[hazelcast-docker-swarm-discovery-spi-1.0-RC11.jar:na]
Feb 27 12:13:20 CSINTFINSDCKR03 springboot-admin[912]: #011at com.hazelcast.instance.DelegatingAddressPicker.pickAddress(DelegatingAddressPicker.java:47) ~[hazelcast-3.11.2.jar:3.11.2]
Feb 27 12:13:20 CSINTFINSDCKR03 springboot-admin[912]: #011at com.hazelcast.instance.Node.<init>(Node.java:197) [hazelcast-3.11.2.jar:3.11.2]
Feb 27 12:13:20 CSINTFINSDCKR03 springboot-admin[912]: #011at com.hazelcast.instance.HazelcastInstanceImpl.createNode(HazelcastInstanceImpl.java:156) [hazelcast-3.11.2.jar:3.11.2]

Additionally, when SwarmDiscoveryUtil.discoverSelf() executes, should it include a check to exclude any local loopback addresses?  The check might be there, but I could not see it.
See below for network addresses reported by PrintNetworkDetails, albeit for a different service and at a different time.
  | February 27th 2019, 13:05:07.000 | CSINTFINSDCKS01 | data-service | start printNetworkDetails()
  | February 27th 2019, 13:05:07.000 | CSINTFINSDCKS01 | data-service | /<IP_ADDRESS>
  | February 27th 2019, 13:05:07.000 | CSINTFINSDCKS01 | data-service | /<IP_ADDRESS>
  | February 27th 2019, 13:05:07.000 | CSINTFINSDCKS01 | data-service | /<IP_ADDRESS>
  | February 27th 2019, 13:05:07.000 | CSINTFINSDCKS01 | data-service | /<IP_ADDRESS>
  | February 27th 2019, 13:05:07.000 | CSINTFINSDCKS01 | data-service | end printNetworkDetails()

I'm pretty sure <IP_ADDRESS> should not be picked as the IP address of this service just because it not found in ip2ContainerMap (SwarmDiscoveryUtil:170-173).  As I said, the check may already be there somewhere and I have no reason to believe that the loopback address is actually being picked.

We see only two log outputs from SwarmDiscoveryUtil:
2019-02-27 13:05:24.964  INFO [ips-data-service,,,] 1 --- [           main] o.b.h.d.docker.swarm.SwarmDiscoveryUtil  : discoverNodes(): via DOCKER_HOST: springboot-docker-proxy
2019-02-27 13:05:26.033  INFO [ips-data-service,,,] 1 --- [           main] o.b.h.d.docker.swarm.SwarmDiscoveryUtil  : Found relevant docker network: ips_network[un0i63ypy2vssq4y86ym3y5zi]

We do NOT see anything containing:
Found qualifying docker service
It would be very useful to have INFO messages from SwarmDiscoveryUtil.discoverContainersViaCriteria() including:

the entire keySet of relevantNetIds2Networks (at INFO now, drop to DEBUG later if voluminous)
the service from iteration #397
the endpoint from iteration #400
the size of the discoveredContainers Set before the return at #457

Hi don't have time to work on this at the moment, feel free to submit a PR

I have committed the log statements but am unable to push to your Repo or to create a PR.
Please grant permission and I will do the PR for you to review and merge.

To contribute a PR you need to FORK this project to your own repo, make the changes there, then issue a PR from your fork. https://help.github.com/en/articles/creating-a-pull-request

Thanks!  PR #31 is with you.  Can you ship that back to me via jcenter() as RC12 ?

The following output from the tests running in Travis CI for PR #33 illustrates the log messages which I have added (marked with *) alongside the pre-existing log messages:
* INFO: Invoking criteria-based container discovery for dockerServiceName=hazelcast-docker-swarm-discovery-spi-test
* INFO: Number of services matching given criteria = 1
* INFO: Processing service with name=hazelcast-docker-swarm-discovery-spi-test
* INFO: Processing service endpoint with networkId=vu06zj0zlm412wf0paxwveyug, addr=<IP_ADDRESS>/24
  INFO: Found qualifying docker service[hazelcast-docker-swarm-discovery-spi-test] on network: hazelcast-docker-swarm-discovery-spi-test[vu06zj0zlm412wf0paxwveyug:<IP_ADDRESS>/24]
  INFO: Found own task, adding regardless of state.
  INFO: Found qualifying docker service task[taskId: j7chjfz2ak9dwsvughbfkmdt8, container: f72070232c59ed2bfd25f8208044313cdd5dec97c1a0cd7f4fe58dcdd89869d2, state: running] on network: hazelcast-docker-swarm-discovery-spi-test[vu06zj0zlm412wf0paxwveyug:<IP_ADDRESS>/24]
* INFO: Returning set of discovered containers with size=1

PR Build is Green.
If your review is approving, please deploy this to jCenter as RC12.

Its out, thanks


Ensure array and regexp functions are registered

array_agg, array_position, and regexp_split were all added in 0.101

lgtm


Solar mode

I'm wondering on how the solar mode works, because I can see a couple of use cases:

Solar-Only mode: The car will only charge when the mains-meter is returning at least 6A. And the charge current will be updated continuously to the amount of energy which would normally be pushed back to the grid.
Solar-Surplus mode: Same as above, but on top of a setup minimum charge power. F.e. 6A, this way the car will always charge, but when there is left over PV power, it will charge faster.

Are both modes somehow already supported via some combination of settings?
To make these options a bit more complicated, there are 2 options to meter:

Physically correct: Checks per phase how much power is available and uses the lowest value to check wether it may charge using PV power.
Billing optimised: In Belgium (and probably in other countries as well), billing is done across phases. So you only need to check the totals across phases. As long as there is 18A available somewhere over the 3 phases, you can charge on those 3 phases at 6A without your billing meter increasing.

Your thoughts please...

Good starting point!
For mode 1, I think it would not wise to stop/start charging when a cloud passes by; so the existing timer I would modify to a timer that starts when there is not enough power to charge on solar, and on end actually stops the charging.
Holland also has "billing optimised", so I see no application/business case for "physically correct".

You can take a look at the EVCC project. They implemented some thresholds to make sure the charging isn't being started/stopped continuously, nor is the charging power.

Which thresholds do you like/use?

EVCC uses these in the yml config:
    enable: # pv mode enable behavior
      delay: 1m # threshold must be exceeded for this long
      threshold: 0 # grid power threshold (in Watts, negative=export). If zero, export must exceed minimum charge power to enable
    disable: # pv mode disable behavior
      delay: 3m # threshold must be exceeded for this long
      threshold: 0 # maximum import power (W)
    guardDuration: 3m # switch charger contactor not more often than this (default 5m)

Unless I misread your question, this is already supported by the original fw using this config parameter:
START set the current on which the EV should start Solar charging -0 -48A (sum of all phases)
STOP Stop charging when there is not enough solar power available Disabled - 60 minutes (Disabled = never stop charging)
IMPORT Allow additional grid power when solar charging (0-20A))
Even though this is set in minutes, have a 1 or 2 minutes buffer before charging stops should not be the end of the world and it avoid unnecessary abrupt stops because a short dip in solar.
And if you set it to disabled, it will keep charging at the START value.
You could also play with the IMPORT value, for example: START: 6, STOP: 2, IMPORT 4
This will keep charging the car with 2A solar, but still getting a total of 6A charging current. Only to stop charging if the import grows higher than 4A over more than 2 minutes.

The timer is not Reset after the cloud has disappeared; so cloud comes by, passes, timer keeps running, switches off after timer expires, and then immediately starts charging again (assuming no clouds).
I experienced this behavior with the original firmware, but AFAIK nothing in solar has changed in the serkri distribution.

That is not my experience, I run almost only in solar mode and I have seen the countdown being stopped.
It is however not immediate, it requires some time of  sufficient solar available before it stops the countdown.

Ok I stand corrected, I only tested once and gave up, thanks for the correction.
Another point needed is driving the contactor2, so solar mode can profit from scaling down to 1 phase charging when less then 18A solar power is available.

Yes, solar is not compatible with 3 phases, I have an idea of making it so but I can't test it myself as my car is 1 phase only.
So have not yet put in the time to make the change.

Solar also works fine with 3 phases in my case, you just need a bigger solar installation on the roof to provide enough current.  I'm hoping the second connector for the Smart EVSE becomes available so you can choose to select relay 1 or 2 depending on your needs.

@rvdgaag When I looked at the code, it appeared for me the logic does not take into account 3 phases.
What I mean with that:
If there is for example 6A running to the grid, the EVSE will tell the car to start charging at 6A.
But if the car can use 3 phases, it will charge not with 6A but with 18A, which will result in the EVSE to stop charging.
After it has stopped, it will see again 6A of current available and repeat this cycle.
This is of course an assumption based on the logic of the code and how I think the car will react.
Could you chime on this with your experience? And how you have configured your EVSE?
FYI, you can already control a 2nd contactor if you want: https://swagger-ui.serkri.be/#/Operations/updateSettings
Please also read this thread for more details on the wiring: https://github.com/serkri/SmartEVSE-3/issues/31

Yes with 3 phases it will charge 3x6A so 18 amps. And it works fine. My grid is limited to 3x25A.
I assume that the EVSE only monitors 1 phase in the solar mode maybe ?

No the EVSE can see all phases, but the logic for Solar does not keep in mind the amount of phases.
So if your car starts charging at 18A, and there isn't enough solar, what happens?
Did you configure your EVSE it can also take grid power?

Yes it can take gridpower ofcourse. as soon as the grid is used it starts to use the timer to shut down.

Ah yes, like I expected. Well that is what I meant with the current logic not being compatible with 3 phases.
At some point I want to make the EVSE aware there are N phases. And thus also use the N for calculating when solar can charge. With 3 phases it should only start when there is 18A available.
This in combination with an automatic enabling/disabling of the C2 contactor, it should optimize the solar usage efficiency.

I like what I hear from you !

fyi: I've added a new config parameter that can be changed through the /settings endpoint, it is available in 1.4.3.
This will allow you to keep charging at minimum current in solar mode if there is not sufficient sun.

The logic for switching second contactor on/off is added in v1.5.0 . Closing issue.


Add Microsoft SQL Server EXPORT DATA extension for SQL Operations Studio

Please Add Microsoft SQL Server EXPORT DATA extension for SQL Operations Studio. It would be very usefull

The SQL Server Dacpac extension provide an option to Export data similar to the same command in SSMS.  Thanks!


Create open and close date field to campaign run nodes

create a date field on run nodes (with a collect end date checked)
this should not be required
the default open date should be 'today'
the close date default should be empty

99.9% sure but want to confirm this should be translatable

Sorry late to the party @deadlybutter @angaither but yes this needs to be translatable.

@deadlybutter to keep things consistent with high and low season, can you make the default be to 'collect the end date'

is it not working as intended? the only thing not checked is 'required'

currently you have to check show end date to see the second date field

in high/low season both options are available

I thought in our meeting thats what we decided to do (since not all campaigns need an end date)

correct, it shouldn't be required, but the option should be displayed

when I make collect and end date required 2 things happen

I cant save it without any value. selecting the blank options in the list cause it to turn red as seen above.

the "no default" value will default to "now", even though I told it not too.

hmm, so it only displays the option if it's required?

yea, thats how the high field is configured

bummer, ok. re-closing


Missing annotation files for inference on my own images

While running prediction on my own images inside the docker enviroement i get an error looking for a json file.
Where can I get it ?
I didn't download the annotation files as it is too big (94GB ??). Do i need it in order to run inference on my own images?
  File "test.py", line 595, in <module>
    show_legend = not args.no_legend)
  File "test.py", line 45, in __init__
    self.datasets = build_test_dataset(cfg, validation = validation)
  File "/host_home/repos/SRW-Net/parsing/dataset/build.py", line 69, in build_test_dataset
    dataset = factory(**args)
  File "/host_home/repos/SRW-Net/parsing/dataset/test_dataset.py", line 24, in __init__
    with open(ann_file, 'r') as _:
FileNotFoundError: [Errno 2] No such file or directory: '/host_home/repos/SRW-Net/data/Structured3D_wf_open_doors_1mm/test.json'```

Thanks Fixed, I'm able to run predictions via docker


Support aria-labelledby attribute

There are cases where a tooltip is serving as the label for an icon. In this case, the better attribute to use would be aria-labelledby, which is announced regardless of navigation method (tab key, arrow key, ...). Would be great if I had the choice to choose between aria-describedby and aria-labelledby.

Thank you. In this case, is there a way to omit aria-describedby ? This is what I have now.
<button data-testid="button" type="button" aria-labelledby="floating-ui-7" aria-describedby="floating-ui-10" class="css-1fgc81t">

right

</button>

Remove the useRole hook and add role="tooltip" to the floating element manually

Thank you!

I wonder if role: 'label' makes sense (although role="tooltip" would remain). Basically acts like aria-label but the tooltip is the element being referenced.

Based on what I am reading here, "it makes a lot more sense to not use the tooltip role" when used with aria-labelledby. What are your thoughts?

It makes sense, so really you only need the code I provided above for it to work. Adding label to useRole would be a shortcut for it

Thanks for explaining. I will close this.


Font bug in pdfkit library when rendering font.

It show error that "could not reading of undifined xCoordinate".

not related with pdfkit https://github.com/foliojs/fontkit/issues/354


Partially inline AnyFrame typealias in return type position

I noticed that completion pop-up still shows that AnyFrame = DataFrame<*> and it makes it look cluttered
It makes me want to remove all such type aliases :c But, we can start from dataFrameOf
Before:

Looks even worse on a narrow screen:

After:

Looks better indeed :) I do see the same behavior for argument types defined with type aliases in your screenshot. Should we keep those? or also replace them

Looks better indeed :) I do see the same behavior for argument types defined with type aliases in your screenshot. Should we keep those? or also replace them

let's make an issue and merge this part


Check schema during startup breaks Docker user

It looks like the new "check schema during start up" breaks Docker user since now the container is in a crashloop.
Previously what I usually do is to run docker exec -it miniflux miniflux -migrate.

I think it's best to keep the web server running with error message if the schema is out-of-date, than exit the web server leaving docker use no chance to migrate the schema.
I'm currently using the following workaround:
version: '2.1'

x-miniflux: &miniflux
  image: miniflux/miniflux
  depends_on:
    - miniflux-db
  environment:
    DATABASE_URL: xxx
    ...

services:
  ...
  miniflux:
    <<*miniflux
    container_name: miniflux
  miniflux-migrate:
    <<*miniflux
    container_name: miniflux-migrate
    retstart: "no"
    entrypoint: "miniflux -migrate"

Then the next time I pull a new miniflux image, docker-compose up -d will restart miniflux (crashloop for a while) and miniflux-migrate (do the schema migration).

 It looks like I can enable schema migration by setting environment variable RUN_MIGRATIONS to 1, and miniflux will do migration the next time it starts.
Sorrry for the noise :-(

Even though this has been solved, I still think the documentation on this is a bit misleading.


watch mode ?

hi
i would like to know if gauge have a watch mode, so auto execute test when save feature file ?

This feature is currently not there in Gauge.
You will have to write a script using something like fswatch

yes for mac os ...


IMAPP: some tiny fixes for linux - not yet working

Looks like this is windows only atm. But before I gave up, I thought that it's at least a start for others to continue with...

On the long run this might be a better option: https://github.com/remyroez/imapp or using one of the backends: https://github.com/ocornut/imgui/blob/master/examples/example_sdl_opengl2/Makefile

Ho yes! https://github.com/remyroez/imapp looks like a very good option.

Wait so how do I build the examples and demos on linux?


On selecting “Presets” as “Last 7 Days”, “This Month”, “Last Month” the calendar widget also select/ highlights an additional day

Steps to Reproduce:-
1.Select “Last 7 Days”, “This Month”, “Last Month”
2.Observe the highlighted dates for each of the three selections
Actual- An extra day is getting selected
Expected- Proper dates should get selected
Please see the attached screenshot.

That is actually working as expected. Yes, an "extra" date is in the range, but because it is from mid-day 31st to mid-day the 7th, that is 7 full days, ie 168 hours. If it went to the 1st, it would be 7 dates, but only a 6 days span.
If you want different preset behavior, you can customize it through the presetRanges array. As example, this is how it is currently configured:
    var now = Px.moment();

    [
        {
            "displayText": "Last 7 Days",
            "startDateTime": now.clone().subtract(7, 'days'),
            "endDateTime": now
        }, {
            "displayText": "This Month",
            "startDateTime": now.clone().startOf('month'),
            "endDateTime": now.clone().endOf('month')
       }, {
            "displayText": "Last Month",
            "startDateTime": now.clone().subtract(1, 'months').startOf('month'),
            "endDateTime": now.clone().subtract(1, 'months').endOf('month')
       }
    ]


Error after pressing "start"

I downloaded today's code, and tried to run it on my phone. But every time I press the "Start" button, I get an message saying "Unfortunately, Pebble Data Logging has stopped". Joshua, does this happen on your phone?

This has been fixed.


New flags readme

Describe in detail the problem you had and how this PR fixes it
Update quick start guide and readme based on new flags introduced with #420
Adds a clause in bug report template about ide.coder.com issues
Is there an open issue you can link to?
n/a

The readme has the "-h" flag twice.  First for (-h, --host) and second for (-h, --help) causing a conflict as -h will never trigger for the help menu.  This conflict is also found in the latest release.

Just noticed doc/self-hosted/index.md also mentions the code-server -h help option at bottom.
Use code-server -h or code-server --help to view the usage for the cli. This is also shown at the beginning of this section.
should read:
Use code-server --help to view the usage for the cli. This is also shown at the beginning of this section.

I've updated this in the telemetry PR


Implementing the feature of making the Application Work Offline

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I have been going through the project from a long time and I noticed that we all are targetting towards this goal

100% education with upskilling ,  developing countries and rural areas

but I think the main concern will be for the people in rural areas as there might be chances of absence of Internet connection or very very poor connection , so there the will to acquire the education will not find its way ,  so this thought led me to solution described below
Describe the solution you'd like
A clear and concise description of what you want to happen.
I think that we should work towards making the application offline so that it can also be accessed in absence of an internet connection and thus ensure that learning never stops .
Now most of us will think that  the person can download the videos and watch later , but if he doesn't even have the access to internet connection to atleast open the application , how will he proceed with awesome stuff inside and leverage the diamond content inside the application .
I request all project maintainers , mentors , contributors to give your views and reviews on this  Feature Request of mine .
Additional context
Add any other context about the feature request here.

@Abhishek-kumar09  Can you please review this ?

@Abhishek-kumar09 Can you please express your ideas on this feature too , am eager to know , on would it be great addtiional feature ?

Okay we can give the option to download some videos for offline access, but again it is not discussed with the other maintainers of the app.
We can also have option to cache pages to have better offline access.


[QOL] [combinedLabels] Remove duplicate nameTags

Change combinedLabels.json to
{'nameTag':NAME,'labels':[LABEL1,LABEL2,LABEL3]}
Main mistake made initially was thinking there was multiple nameTags for same address. This results in unnecessary repetition.
Example:
["Paradex (0x-ecosystem)", "Paradex (dex)"] -> nametag:"Paradex", labels:[0x-ecosystem,dex]

Restructured.


Automated builds

This automates the build and release of the Docker containers. They are now published to the GitHub container registry, and happen automatically.
I also fixed a bug: You forgot to include the log dir in the repo, so builds for anyone without it would fail.
I also integrated this app into @runcitadel and used these containers for it, so they are already tested (A demo instance is running at https://bitfeed.runcitadel.space).

thanks for this! I'll review asap


Add Italian Support for Confirm/Choice Prompts

Related: https://github.com/microsoft/botbuilder-dotnet/pull/3129
Recognizers-Text added support for a few additional languages in C# and Italian in JS, which we don't currently support in our Confirm and Choice prompts:

Bulgarian (C# only)
Hindi (C# only)
Italian
Korean (C# only)
Swedish (C# only)
Turkish (C# only)

This PR adds support for those, ensures they get test coverage, and also adds test coverage for PromptCultureModels.
Notes:

I skipped adding Arabic support because it is read Right to Left, which makes prompt support tricker.
Recognizers-Text JS SDK currently has only added Italian, but I don't believe there's a reason our C#/Node SDKs need to have matching language support
I'm also far from a polyglot, so most of the translations were provided by [cough]Google Translate[cough]*

Need to Decide Before Merge
English is really the only language that uses the Oxford/Serial comma. I.e, in this sentence:

Yes, no, or maybe

...the second comma is the Oxford/Serial comma. Most other languages would write it as:

Yes, no or maybe

In our SDK, this is the inlineOrMore property:
public static Spanish: PromptCultureModel = {
    locale: Culture.Spanish,
    separator: ', ',
    inlineOr: ' o ',
    inlineOrMore: ', o ',
    yesInLanguage: 'Sí',
    noInLanguage: 'No',
}

..without the Oxford comma, inlineOrMore would look the same as inlineOr.
The languages that I added do not have the Oxford comma (as appropriate). However, many of the current languages we have support for do use the comma in our SDK, although would likely not in real life.

[ ] Should those commas be removed in the SDK, or would that possibly create backwards-compat issues?

Pull Request Test Coverage Report for Build 95525

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 84.49%

Totals

Change from base Build 94922:
0.001%

Covered Lines:
8428

Relevant Lines:
9648

💛  - Coveralls

@cleemullins ^s removed.

^s removed, I mean (can't edit)


Add Capsule to Rancher marketplace

Add Capsule to Rancher marketplace
As a Rancher’ user, you should be able to install Capsule from the Rancher marketplace.

This is pretty similar to clastix/kamaji#170, although we need to make sure to install the required addon.
@maxgio92 since we worked on this, do you think we could add the addon as a chart dependency?
We need to engage with SUSE Rancher peeps to understand if we need to provide two charts or a single one since, to take full advantage of Capsule with the Rancher ecosystem, the addon is a required component.

Hi @prometherion , I think it makes sense to keep the charts separate as it's not mandatory to use Capsule integrating Tenants with Rancher Projects.
At the same time it would be good to document the integration referencing the addon.


Calling the "Symfony\Component\EventDispatcher\EventDispatcherInterface::dispatch()" method with the event name as the first argument is deprecated since Symfony 4.3, pass it as the second argument and provide the event object as the first argument instead.

Hello,
The following error is filling up our error logs and it seems to come from Session.php.
Calling the "Symfony\Component\EventDispatcher\EventDispatcherInterface::dispatch()" method with the event name as the first argument is deprecated since Symfony 4.3, pass it as the second argument and provide the event object as the first argument instead.
Thanks in advance,

Hi Team, Any update on above issue. why because even i am facing the same issue, please let me know any alternate solution for this?, Thank you.

DEPRECATION: Calling the "Symfony\Component\EventDispatcher\EventDispatcherInterface::dispatch()" method with the event name as the first argument is deprecated since Symfony 4.3, pass it as the second argument and provide the event object as the first argument instead. /var/www/html/magento_MFTF/vendor/symfony/event-dispatcher/EventDispatcher.php:58

This will be fixed in #343


flatMap -> groupRuns -> takeWhile performance issue

I've noticed some unexpected performance problems when chaining these three stream operations together.
Here is a minimal example (using StreamEx version 0.8.1). If I debug this code, I see that in the "bad" case, the aString -> aString + "!" lambda is run on each element of the input stream (including ones that should be filtered out by the takeWhile) before the groupRuns or takeWhile lambdas are ever called. As shown with the other examples, this only seems to happen when chaining together a flatMap, a groupRuns, and then a takeWhile.
import java.util.List;
import java.util.function.Function;
import java.util.stream.Stream;
import one.util.streamex.StreamEx;

public class StreamExTest {
    private static List<Stream<String>> inputStream() {
        return List.of(Stream.of(
                "foo1", "should be printed", "hello world",
                "foo2", "should be mapped and grouped", "but not printed",
                "foo3", "should not reach initial map"));
    }

    public static void main(String[] args) {
        // bad -- calls the map function on each stream entry before grouping
        StreamEx.of(inputStream())
                .flatMap(Function.identity())
                .map(aString -> aString + "!")
                .groupRuns((aFirst, aSecond) -> !aSecond.startsWith("foo"))
                .takeWhile(aStrings -> !aStrings.get(0).endsWith("2!"))
                .forEach(System.out::println);

        // works as expected (not using takeWhile)
        StreamEx.of(inputStream())
                .flatMap(Function.identity())
                .map(aString -> aString + "!")
                .groupRuns((aFirst, aSecond) -> !aSecond.startsWith("foo"))
                .forEach(System.out::println);

        // works as expected (not using groupRuns)
        StreamEx.of(inputStream())
                .flatMap(Function.identity())
                .map(aString -> aString + "!")
                .takeWhile(aString -> !aString.endsWith("2!"))
                .forEach(System.out::println);

        // works as expected (not starting with flatMap)
        StreamEx.of(inputStream().get(0))
                .map(aString -> aString + "!")
                .groupRuns((aFirst, aSecond) -> !aSecond.startsWith("foo"))
                .takeWhile(aStrings -> !aStrings.get(0).endsWith("2!"))
                .forEach(System.out::println);
    }
}

This behavior is quite undesirable if other intermediate stream operations are computationally intensive and/or produce objects that take up a lot of memory, especially if the takeWhile filters out a large number of elements.

My workaround for the time being is to do something like this, avoiding the flatMap:
myFlattenedStream = Stream.empty();
for (Stream<String> myStream : inputStream()) {
    myFlattenedStream = Stream.concat(myFlattenedStream, myStream);
}

StreamEx.of(myFlattenedStream)
    ...

Hello! Unfortunately, this is a known limitation of current implementation. New stream operations, which don't delegate to existing ones, take the spliterator of the original stream. If there's a short-circuiting operation after that, then tryAdvance method of that spliterator is used, which eventually delegates to the tryAdvance method of the original spliterator. But the spliterator returned after flatMap actually consumes the whole sub-stream (see https://github.com/openjdk/jdk/blob/16c3d53b1bb60b1c1570731041f564bf13b45098/src/java.base/share/classes/java/util/stream/StreamSpliterators.java#L288 implementation). You can reproduce the problem even without StreamEx:
        var spliterator = Stream.of(0).flatMap(x -> Stream.of(1, 2, 3))
                .peek(System.out::println)
                .spliterator();
        spliterator.tryAdvance(x -> {});

This code prints 1 2 3, despite we requested only one element from the spliterator. Unfortunately, there's no good way to fix this in JDK, and I see no better way to implement custom stream operations, other than delegating to the original stream spliterator. So, sometimes you may observe such performance glitches.

Huh, yeah that is unfortunate :/
Thank you for the detailed response @amaembo !


Where is your nnSAM network architecture?

Your work is great, but I can't find your nnSAM network architecture file. So could you please provide a direct link?

Did you find that? I can't find nnSAM network too.

OK, I find it. from nnunetv2.utilities.unet import SAMConvUNet

In which script did you find "from nnunetv2.utilities.unet import SAMConvUNet?

In which script did you find "from nnunetv2.utilities.unet import SAMConvUNet?

I don't remember, but you can find nnSAM network architecture in nnunetv2/utilities/unet


added cluster validation

Checking the cluster name is not empty and checking the endpoint is a valid link, so that the app does not crash and the crash is not recoverable until the cluster is programatically changed.

Thanks for your contribution @farmansk! The tests are failing but that's due to something in our test setup.
I'll take this PR over and make sure to land it in the upcoming minor version!

Thanks!

Thanks! This is merged in #47 687143c43ae4028a80d8a8a9b77529776c4aa007


make status(bar/line) colors more pleasant for the eyes

NEVERMIND! i was using an airline theme that happened to look similar


Render full camera space normals outside training

Currently eikonal loss is computed in canonical space on subsampled pixels, which makes it hard to visualize the normals and geometry quality.
This PR makes no changes at training time, but during rendering, we render the full normal map computed as the gradient of sdf(xyz) wrt. xyz_cam. The SDF is queried with canonical coordinates xyz, so torch.autograd differentiates through backward_warp which maps xyz_cam to xyz.
Examples from tensorboard:

Why warp xyz to xyz_cam again for compute normal?

This PR is to render normals in the camera coordinate, similar to ECON and as shown in the teaser image. But the SDF is in canonical coordinate, so we need to do a backward warping


Encoding of 0.0 and -0.0 is not at minimum precision

I'm using CBOR-Java to encode floating-point numbers. I see that the code makes an effort to encode floating-point numbers in the minimum precision possible. For example, 1.0 is encoded in 3 bytes. However, zero values are encoded in 5 bytes. The same happens with floating-precision or double-precision zero values, and also with negative zero values.
assert(CBORObject.FromObject(1.0f).EncodeToBytes().length == 3) // Passes.
assert(CBORObject.FromObject(1.0).EncodeToBytes().length == 3) // Passes.
assert(CBORObject.FromObject(0.0f).EncodeToBytes().length == 3) // Fails. The actual length is 5.
assert(CBORObject.FromObject(-0.0f).EncodeToBytes().length == 3) // Fails. The actual length is 5.
assert(CBORObject.FromObject(0.0).EncodeToBytes().length == 3) // Fails. The actual length is 5.
assert(CBORObject.FromObject(-0.0).EncodeToBytes().length == 3) // Fails. The actual length is 5.

The option float64 is false per default. So, I expect minimum-precision encoding but I do not obtain it.
I tracked this to the code of DoubleToHalfPrecisionIfSameValue() here: https://github.com/peteroupc/CBOR-Java/blob/master/src/main/java/com/upokecenter/cbor/CBORUtilities.java#L1516-L1527
Zero values will not be detected by this function because, for zero values, both the exponent and the mantissa are equal to zero.
https://docs.oracle.com/cd/E19957-01/806-3568/ncg_math.html
So, the value sexp in https://github.com/peteroupc/CBOR-Java/blob/master/src/main/java/com/upokecenter/cbor/CBORUtilities.java#L1514 will be computed as -1008. Then line https://github.com/peteroupc/CBOR-Java/blob/master/src/main/java/com/upokecenter/cbor/CBORUtilities.java#L1522 detects "underflow" and returns -1.
It also seems that the code in https://github.com/peteroupc/CBOR-Java/blob/master/src/main/java/com/upokecenter/cbor/CBORUtilities.java#L1527-L1532 does not work correctly with zero mantissa (it will always return nonzero values even if the mantissa is zero and value sexp is in the range for that code to activate).
So, to fix this, I propose to add another else if clause explicitly detecting a signed zero value.
I need to encode zero and negative zero values at minimum precision. Is there a workaround? I was not able to see how I could override the non-minimal encoding for those values.
I also tried CBORObject.FromFloatingPointBits(0L, 2) to represent floating-point zero at minimum precision, but this reverts to non-minimal encoding because the minimum precision is not detected. I tried other values of "floating point bits" to trick DoubleToHalfPrecisionIfSameValue() into accepting the zero value, but so far without any success.

It can be worked around by returning the correct minimal form for encoding floating-point zeros, either {0xf9,0,0} for 0.0 or {0xf9,0x80,0} for -0.0.  However, if your application encodes arrays or maps containing floating-point numbers then it's more difficult to work around.

My application constructs a large CBORObject that may contain Double values inside, some of them may be zero. I would like to be able to write the entire CBORObject correctly. Can I somehow inject the right code at the right place? Is there a way of defining a custom class, say MyDouble, and define some code so that CBORObject.FromObject(myDouble) will serialize correctly? I was not able to find out how to do that.

Given that the CBOR object you want to serialize may be non-trivial, I suggest it would be better for you to fix the bug you found and submit a pull request, given that you've detailed exactly where the bug's source is — which is extraordinary for a bug report.
Although that is the preferred alternative, you can also write a version of EncodeToBytes that make use of the low-level method CBORObject.WriteValue and CBORObject.Write.  See Interop.md, from the .NET version of this library, for an idea of what the solution could look like.

I have submitted a PR. https://github.com/peteroupc/CBOR-Java/pull/25
While running maven tests on my local machine I have found some issues (code that did not compile with Java 8 etc.) but did not fix them in the PR.
Thank you for pointing out Interop.md. However I don't see how I could implement that kind of interop in Java. The Java code base doesn't have a CBORWriter. Instead, the encoding methods are scattered around specific CBORObject types, and all those methods are static so I cannot override them. I'm not sure how

I have already merged your fix.  CborWriter is a built-in CBOR-supporting class from the .NET base class library, and there is no built-in equivalent to it in Java.

Thank you! Will you make a similar change in the .NET version? Also, would you publish a fix on sonatype so that I could uptake it in my project? So far I don't see how I could make any workarounds.


.env: fix RAILS_ENV for docker

Otherwise, HUGINN_RAILS_ENV won't take affect for docker installation.

Thanks @xu-cheng, did you successfully run the docker container in the production environment? Last time I checked there were a few issues related to the assets not being precompiled.

did you successfully run the docker container in the production environment?

I haven't checked this yet. Without this fix, I was unable to run docker as production. This value keeps getting overwritten by init script.

That makes sense, you can build the docker container locally, the git repository and branch needs to be changed to git clone --depth 1 https://github.com/xu-cheng/huginn -b rails-env-docker /app here
Running docker build --rm=false -t huginn-test . && docker run -it -p 3000:3000 huginn-test in the docker subdirectory should then build and run the image.

With #1204 merged the docker images run in the production environment by default.


add function to get ChessJS instance for a move

this is not feasible, as someone may make a move on the chessjs instance, and there's no way to control it via this library


amp ad preview design

@aghassemi PTAL
Demo: https://amp-by-example-staging.appspot.com/amp-ads/experimental_ads/scrollbound_video_ad/preview/

All done. PTAL

LGTM


dwl: fix manpage location

Description of changes
Man pages does not show up because the man page ends up in result-man/share/man/man1/man1/dwl.1.gz instead of result-man/share/man/man1/dwl.1.gz.

Things done
I copied the default.nix file of dwl from nixpkgs to my system and played around with it to learn nix, and figured out why the man pages do not show up. I am new to this, so I don't exactly know what I am doing.

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

 I was talking about the commit, not about github.

@AndersonTorres my bad, I'm a git noob. Let me know if I can that now that it has been committed 😬

Don't worry.


Create a simple project readme

The project README should fulfill the following criteria:

Badges for supported platforms (available on https://shields.io)
What is Contoso Moments
How do I Deploy this demo
How do I Configure the Mobile Client App.
Change Log

first draft done, will work on more details later.


Fix poetry lockfile issues (and add pre-commit hook)

There can sometimes be errors with the poetry lockfile become out-of-sync with the pyproject.toml.
This PR adds a pre-commit hook to detect and prevent such errors.

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

Comparison is base (6363e05) 52.49% compared to head (2f654b2) 59.97%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #90      +/-   ##
==========================================
+ Coverage   52.49%   59.97%   +7.47%
==========================================
  Files           9        9
  Lines        1364     1364
==========================================
+ Hits          716      818     +102
+ Misses        648      546     -102

Flag
Coverage Δ

integrationtests
?

unittests
59.97% <ø> (+9.31%)
:arrow_up:

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

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


⚠ Bofrost.lu has degraded performance

In 1713b10, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1691 ms

Resolved: Bofrost.lu performance has improved in 79954d5.


Memory leaks when mapping an array of dictionaries nested in a parent dictionary

I am currently mapping a JSON response that returns a dictionary. The dictionary contains a key titled "games" that contains an array of dictionaries. Here is a sample JSON:
{"games": [{"id": 1307, "status": "scheduled", "status_display": null, "stuff": [{"someStuff": 123}]}]}
After reading this issue: #200, I removed the nested object "stuff" in order to verify this was not the issue. I am still receiving memory leaks.
It is possible this is related to how I am processing the JSON. Here is what my mapping of the response looks like:
Mapper<Game>().mapArray(response.result.value!["games"])!
Here are some shots of the instruments results:

Each of the snapshots that show leaks are attempts to perform the mapping. Is there a better or proper way to map JSON with this type of format? Or based on these results could it be another issue? Thanks!

As a quick side note, creating a new parent class and adding the mappable property "games" solves this issue; however, It is preferable to not require an additional class.

@kylebegeman -i'm facing same issue with  json array of count 135000.i added generic class to map array but still there are so many memory leaks.would you please share a sample code of parent class. how did you map?


GitHub issue with two Linear tickets IDs: [ABC-2][ABC-1] GitHub issue

Hi,
A new issue is created in Backlog in Linear when I tag a issue In Progress as Public. In Github, this results in two linear issue IDs in front of the issue title. I am not sure why this is designed as such.

Hi @woodwm, thanks for bringing this up. Have seen something similar so I'm going to re-phrase the title of this issue. Will try to replicate it.

I have encountered the same issue 🙏
Quote here: https://github.com/calcom/synclinear.com/issues/134#issuecomment-1727653415

Thanks for linking to this, @xiaofei-du! I think there's a race condition, where the newly-created GH issue triggers a webhook before it's been added to the DB. Looking into it.

Hi,
In my case, two things happens.

there are two Linear ID in the github issue.
There is a new Linear issue created in the Backlog

Image 1:

Image 2:

Subscribing to this because I have the same issue

this seems to be happening now for every issue we sync from Linear -> Github via the 'Public' label which makes it almost unusable
Here is the latest example: https://github.com/flipt-io/flipt/issues/2294

Same issue for me on a fresh install if synclinear with configurations as default. Whenever I add the label "Public" it creates the issue in GH and immediately after creates a new one in Linear and appears in "triage" section.

Can confirm this is happening to our workspace / GitHub repo too. Quite difficult, as both Linear tickets are linked to the GitHub issue. Closing either Linear ticket results in closing the GitHub issue, so I have to keep both open.

@tedspare regarding reproducing the issue, I believe this might be caused by multiple team members having set up the same sync. As the sync to Linear wasn't working for other team members when I had configured it for one of our repositories, other team members also configured the same sync.

Experiencing similar behavior. We have opened an issue on Github which resulted in the following traces.
Unsure about what is causing this, but wanted to share it might help to unpack.

Thanks for bumping.
This seems to happen when the issue creation triggers a GitHub webhook before it's been added to our DB.
A possible stopgap solution is to skip GH issues starting with [ABC-123]. Will implement this.

Should be prevented by 01c38f6eace05b6d0397b300f7d49e72ecf1c308


swift3

Thanks for the pull request.
The "don't crash" commit is related with the swift 3 support? If it is not, I would love if you could send me a different pull request for it.


Launching devtools breaks rendering

This is really weird and I can't really explain what's happening except for what stated in the title.
You'll also notice that for some reason when devtools is active the background becomes white.
Attaching video of the behavior: https://v.usetapes.com/6MFjFXyx2Y
Env:

Electron 1.4.2
Sierra 10.12.1

I think dev tools in general breaks the transparency. But not every time. Create a normal non-vibrant window with transparency and set it full transparent. Then open the dev tools, transparency will probably break. I dont think it is strictly related to this module.

Transparency in Chromium is a tricky subject because it is directly related to the hardware. Can you open the dev tools with the detached flag to see if it breaks anything? On my PC, if I open the dev tools detached, rendering isnt broken.
openDevTools({ detach: true });

Curiously adding that flag completely removes vibrancy from the window.

Thats interesting. Can you try it in the test app of this repository? Just to verify this. Add mainWindow.openDevTools({ detach: true }); to here, run npm test and see if the behaviour is the same.

Also I see your app has a frame. Did you see here?

Also I see your app has a frame. Did you see here?

I did, but having these two keys with any values has zero effect on the app when vibrancy module is active. It always produces the same results (vibrancy + frame)

I'll have to look into this tomorrow and see why the test app seems to work fine even with dev tools attached – my apps config is essentially the same but as you can see, weird things are happening.

Can it be due to the fact that you're opening the dev tools just after creating the window? The test app opens it after the SetVibrancy call.

I'll do some digging tomorrow morning when my brain isn't fried after the whole day of coding ;-)

Looks like there's something weird with my app – can't really figure out. But I can live with vibrancy not working well in development. What's more pressing is #3 ;-)

What's more pressing is #3 ;-)

I'll look into it very soon.


Can't register deployment group agent with manually created PAT token

Agent version and platform
2.124.0 on Windows Server 2016
VSTS type and version
VisualStudio.com
What's not working?
Official docs say

For the scope select Agent Pools (read, manage) and make sure all the other boxes are cleared. If its a deployment group agent, for the scope select Deployment group (read, manage) and make sure all the other boxes are cleared.

But the Deployment group (read, manage) option does not exist although Agent Pools (read, manage) does.
So when I create a PAT token with the scope Agent Pools (read/manage) I get the following errors when trying to configure the agent:

You are not authorized to access https://<account>.visualstudio.com
TF400813: Resource not available for anonymous access. Client authentication required.

But it works with the automatically generated PAT token when copying the install script. I checked what scopes it uses but none are selected even though "Selected scopes" option is ticked.
When I add the Agent Pools (read, manage) scope to the automatically generated PAT token and save the changes it doesn't work anymore, giving me the same authorization/access errors.
Is this intended?

@chshrikh

I just set one up by clicking on the 'Registration Script' link on the right side of the toolbar when you open a deployment group and clicked the 'use PAT in registration script' checkbox and that seemed to work, although the token itself wasn't configured with any permissions when I looked in my security settings.
Docs should be updated if things have changed, though.

@spongessuck If you add a random scope or select "All scopes" does it stop working?

This has been fixed and will be available in next deployment ( 2-3 days ). With this fix, you will be able to see the PAT scope option. Deployment group (read, manage) will be available in your security page for generating the PAT. Meanwhile you can get unblocked by using the automatically generated PAT in the configuration script form Deployment Group Page; Select the deployment group -> click on Details -> enable the option 'Use personal access token in the script for authentication' -> Copy the script..

this is has been fixed.


Fixed behat tests

There is still a failing test :confused:

are you sure? hmm thats a completely new failing testcase ... i have restarted it (: hopefully it works then!

I've already tried to restart, it's always the same test failing... but seems like I have to fix that now...


Floyd Warshall to find pairwise shortest path lengths

Added a plugin called 'floydwarshall' that returns a 2D Array of lengths of shortest paths. The indices of the 2D array correspond to the indices of nodes in 'nodesArray'. For example, the entry (i,j) 2D array distMatrix : distMatrix[i][j] corresponds to the length of the shortest path between nodesArray[i] and nodesArray[j].

Any news on this pull request?

Thanks for this PR, even though it hasn't been merged.
For a few years, we focused on Sigma.js v2, and this week we released it!
It is probably better now to upgrade to this new version as v1 will no longer be maintained.
This PR will therefore be closed.


Will separate data and code be a good idea?

Hi! I am a undergraduate student looking for good linguistic tools, and thank you for all the work!
And I have some suggestions: when looking into the codebase, I found all the code in one file and the strings are hard coded in the main file.
Will separate data and code be a good idea? It will make it much easier to maintain the code. Using "for" statement will also make the code easy to read, as well.
I am writing a  linguistic tool called phoneme_from_word which is in early development, you can check that on my page.

Hi, thank you for your comment! Any thoughts or feedback are highly appreciated.
A short answer is: Yes, you're probably right! But, I feel that making this good change may require deeper planning than it looks like at first glance. At present, I need to focus on purely linguistic aspects of the program, and once lngcnv does a useful linguistic work, I'll certainly see how to optimise the source code and make it look better, hopefully by the end of 2022.
A long answer is: You're actually the 4th person commenting on my program and the 4th one saying the exact same thing! Your suggestion sounds reasonable, and thank you! I've already exceed ten thousand lines of code and this all will keep growing..., hard coded in the main file. One person kindly forked an earlier version of my program (named lng-cv, which I first wrote in Pascal and then rewritten in Rust as lngcnv) in Python to show me how I can use .json files: https://github.com/tomek-siuda/lng-cv-python
However, lngcnv does not have a definite structure, but instead it's a long-term project which I'll keep developing the rest of my life, literally. At the very beginnig the algorithms did simple replacements of characters so that all the data could be easily organised just as shown in lng-cv-python, for example. But, as I keep developing it (and also, as I keep learning to code in Rust) things get more complex. For example, for certain languages, I need to concatenate strings to mark the word beginning or ending, and once I develop more intelligent ways to detect word stress (or tone) and subdivide words into syllabes, etc., I'll likely need to introduce more complex operations and see how to organise this all. My program offers orthographic and -in particular- phonetic modes of action for multiple dialects of six different languages belonging to three language families [languages I speak or learn to speak]. I can certainly subdivide lngcnv into more files but still as each natural language is structurally unique, in future it will be technically imposible to employ in lngcnv a single transparent format for encoding everything, so each file would be completely different.
Once v1.6.0 is released (already in alpha), I'll see how to organise all this a better way, perhaps in some kind of modules or separate files.
Thanks for the info on phoneme_from_word--I'll take a look at it, and see if I possibly can give you some feedback.

Thank you very for your reply!
And I think you are right. I am more of a programmer, and I think in programmer way. But I now understand that making it work is more important than beautify the code.
As for my project, It's just a toy to help me with AI homework. You don't need to waste your precious time looking at it.
After all, thank you for your contribution!

No need to be sorry! Your viewpoint is perfectly valid--I've been just more focused on linguistics than programming which was essential during early development. I've been thinking on the problem though... and will soon make some changes:
The file main.rs will essentially manage the control flow and some other core logic of the program (which will soon get more complex than it is at present as I want to introduce certain enhancements). Whereas all the 'linguistic logic' will be moved to lib.rs--this will be a single large library, exceeding ten thousands lines of code, but still will allow me to partially avoid code duplication and, briefly speaking, will offer me some other benefits without affecting my possibilities of string manipulation, future improvements and introducing new exciting features. :)
I'm not sure if this is what you originally meant but it makes sense to me for now, so I'll close this issue once the changes are implemented (possibly in lngcnv v1.6.0-alpha.3 in mid/late-July).

Done in v1.6.0-alpha.3

In v1.6.0-beta.4 the source code is subdivided into 9 files and largely rewritten. Also, code duplication is avoided as much as possible.


Error: listen EADDRINUSE，How to resolve

My run command is jest --coverage --runInBand --detectOpenHandles --forceExit
And I executed app.listen in multiple files。
Then show this error.

I find the same error in https://stackoverflow.com/questions/51373464/nodejs-error-listen-eaddrinuse-3000, But it's no resolution.
Does anybody know how to solve this? I don't want to write all my integration tests in 1 file.

We have no idea what code you try to test and how. Please provide a repro so we can investigate. You can use globalSetup/globalTeardown to startup a single server and connect to it from tests if there's no server/db state overlap, or you can run multiple servers on different ports.


Upgrade the dragablz(<IP_ADDRESS> to <IP_ADDRESS>) from nuget, fix the progra…

Upgrade the dragablz(<IP_ADDRESS> to <IP_ADDRESS>) from nuget, fix the program startup error.
fixed issue: #30

please review,thanks


is_target_feature_detected not being statically removed with newer target-cpu

I believe is_target_feature_detected! is supposed to be statically removed if the build settings guarantee the target_features exist.
This is not the case here,
Linux, rustc 1.26.0-nightly (c9334404f 2018-03-05)
Running the following with
RUSTFLAGS="-C target-cpu=haswell" cargo run --release
And checking perf top, shows
std::stdsimd::arch::detect::check_for
Taking the majority of cpu time, uncommenting out the cfg check removes that.
The optimizer skips the relevent sse4.2 code, but the commented out random check can be used, and check_for will show up in perf (though it may need to be searched for, since it's less of the cpu time)
If I look at the assembly it appears that with target-cpu=haswell, the sse4.2 function is inlined into the function with the check, and without it's a separate function.
#![feature(target_feature)]
#![feature(cfg_target_feature)]
#![feature(stdsimd)]

extern crate rand;
use rand::{thread_rng, Rng};

#[cfg(target_arch = "x86_64")]
use std::arch::x86_64::*;

#[cfg(target_arch = "x86")]
use std::arch::x86::*;

#[cfg(any(target_arch = "x86", target_arch = "x86_64"))]
#[target_feature(enable = "sse4.2,popcnt")]
unsafe fn subset_len_sse42(a: [u8; 16], b: [u8; 16]) -> u32 {
    let a = _mm_loadu_si128(a.as_ptr() as *const __m128i);
    let b = _mm_loadu_si128(b.as_ptr() as *const __m128i);

    _mm_extract_epi32(_mm_cmpistrm(a, b, 0), 0).count_ones()
}

fn subset_len_portable(a: [u8; 16], b: [u8; 16]) -> u32 {
    a.iter().map(|x| if b.contains(x) { 1 } else { 0 }).sum()
}

fn subset_len(a: [u8; 16], b: [u8; 16]) -> u32 {
    /*
    //Uncomment this out to make it work
    #[cfg(all(any(target_arch = "x86", target_arch = "x86_64"), target_feature = "sse4.2",
              target_feature = "popcnt"))]
    {
        return unsafe { subset_len_sse42(a, b) };
    }
    */
    #[cfg(any(target_arch = "x86", target_arch = "x86_64"))]
    {
        //This check is supposed to compile out when a new enough arch is
        //enabled at build time, but isn't working
        if is_target_feature_detected!("sse4.2") && is_target_feature_detected!("popcnt") {
            return unsafe { subset_len_sse42(a, b) };
        }
    }
    subset_len_portable(a, b)
}

fn main() {
    // Zero marks end of string in _mm_cmpistrm
    let mut vals: Vec<_> = (1..256u16).map(|x| x as u8).collect();

    let mut rng = thread_rng();

    let mut a = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16];
    let mut b = [
        9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24
    ];

    loop {
        assert_eq!(subset_len(a, b), 8);
        /*
        rng.shuffle(&mut vals);
        a.iter_mut().zip(vals.iter()).for_each(|(a, v)| *a = *v);
        rng.shuffle(&mut vals);
        b.iter_mut().zip(vals.iter()).for_each(|(b, v)| *b = *v);
        assert_eq!(subset_len(a, b), subset_len_portable(a, b));
        */
    }
}

It would help if you could fill a minimal working example without any external dependencies.
MWE here: https://play.rust-lang.org/?gist=e87c67c0054c7f4efc3f41c3c96061b6&version=nightly
The assembly for subset_len is:
playground::subset_len:
	push	r14
	push	rbx
	sub	rsp, 40
	mov	r14, rsi
	mov	rbx, rdi
	mov	edi, 11
	call	std::stdsimd::arch::detect::check_for@PLT
	test	al, al
	je	.LBB11_3
	mov	edi, 30
	call	std::stdsimd::arch::detect::check_for@PLT
	test	al, al
	je	.LBB11_3
	movups	xmm0, xmmword ptr [rbx]
	movaps	xmmword ptr [rsp], xmm0
	movups	xmm0, xmmword ptr [r14]
	movaps	xmmword ptr [rsp + 16], xmm0
	mov	rdi, rsp
	lea	rsi, [rsp + 16]
	call	playground::subset_len_sse42
	add	rsp, 40
	pop	rbx
	pop	r14
	ret

The macros are not being removed. There must have been a regression. I'll look into this.

@alexcrichton do you know how to set up an issue template ?

Ah I think I accidentally broke this in the transition to a layout amenable for inclusion into libstd, sorry about that! I'll open up a PR.


Remove ignoreResourceNotFound

Ticket Link
GROUPINGS-1051
Test Checklist

[x] Unit Tests Passed:
[x] Jasmine Tests Passed:
[x] General Visual Inspection:

Please don't do this.

Please don't do this.

Perhaps I misunderstood the original intent of this ticket as it was to just change ignoreResourceNotFound to false...any suggestions/comments to reword this ticket so it is aligned more with solving the problem.


pixi shell broken on Windows if Env vars contains quotes

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
On windows poweshell create an environment variable with quotes then activate the shell
❯ $env:MYVAR = '"hello"'
❯ pixi shell
ParserError: D:\TEMP\.tmpDLRbMf.ps1:56
Line |
  56 |  ${Env:MYVAR} = ""hello""
     |                   ~~~~~~~
     | Unexpected token 'hello""' in expression or statement.

Issue description
Quotes are not correctly escaped when creating the tempoarary script used by pixi shell in windows powershell.
Expected behavior
The command would succeed.

Same issue as #1503
We shouldn't copy the environment variables as the shell will do that itself. #1507 fixes the issue.


Code workflowConfigure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

 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.:x: MeenDash sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

done

ok

k


Screenshot shows shell highlighting

In the screenshot in the readme shell highlighting is included. Can you provide information as to what shell highlighting you are using and also any other plugins that compliment this extension?

I'm using zsh with OhMyZsh framework.


launch upload_bluerov2_launch.py error

I installed all dependencies, and the world launch opened without any problems, but when I ran the upload bluerov2 launch it resulted in the following error
$ ros2 launch bluerov2_description upload_bluerov2_launch.py sliders:=true
[INFO] [launch]: All log files can be found below /home/daniel/.ros/log/2024-07-04-11-41-29-725035-V3D-03-256110
[INFO] [launch]: Default logging verbosity is set to INFO
[ERROR] [launch]: Caught exception in launch (see debug for traceback): list index out of range

Really do not understand what is out of range..

Hi,
What ROS distro are you using?
Your error seems to be the same as https://github.com/CentraleNantesROV/bluerov2/issues/4 but I cannot reproduce it.
It probably comes from simple_launch that is failing to guess between Ignition (e.g. fortress) and Gazebo (garden+).
Can you try to write export GZ_VERSION=(your gazebo / ignition version) in .bashrc to help the launch system

Got it and solved in the last commit of simple_launch (1.10.1). This was a matter of gz not being here at all, on my machine I have several versions and did not see that having only ignition-fortress raised an error (while having fortress and garden or harmonic was fine).
You can use the source version of simple_launch while it is not released.
Thanks for the issue, this Ignition / Gazebo thing will only disappear with the end of Humble so it was worth solving.

Thank you @oKermorgant, I will check this week to see if all works fine.

Hello!
What version of Gazebo did you recommend to use with this package?

Hi, we have been using it on various versions of Gazebo and ROS:

Foxy + Fortress
Galactic + Fortress
Humble + Garden
Humble + Harmonic

simple_launch will try and detect the running Gazebo instance when spawning the robot.


fix for #1011

Use the acceptablePackages set with startsWith, rather than using .contains() in order to restore recursive package scanning.

@hellfur - Thanks you for the PR. However...
First, we only accept PRs against the develop_2.0 branch.
Second, we may want to make it configurable so that people can choose whether to deep scan or not.
Third, labels are a huge no-no.


The page needs update

Hi there,
The page needs updating as the 'Properties' option is not there anymore.

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

ID: 665ef070-9879-3f6b-d528-ff64bd5236ff
Version Independent ID: 9ca6d22b-e684-4e65-cab3-c272189a3ddc
Content: Add or change Azure subscription administrators
Content Source: articles/cost-management-billing/manage/add-change-subscription-administrator.md
Service: cost-management-billing
Sub-service: billing
GitHub Login: @bandersmsft
Microsoft Alias: banders

@Sayel-76 Thanks for your comment. We will review the issue and get back to you shortly.

@sayel-76 Hi there. Can you please let me know that type of subscription you're referring to? Can can view Properties option under Settings using the various subscription types that I have access to.

@Sayel-76 Hi there. Are you still having this problem? If so, please let me know what type of subscription you have. Thanks.

@sayel-76 I'm unable to reproduce the problem you describe with the various types of subscriptions that I have access to. If you continue to have this problem, please reopen the issue and let us know what type of subscription that you have. I'd be happy to investigate further with your information.
#please-close


type annotation for Index/MultiIndex.names is incorrect

Describe the bug
type annotation for Index/MultiIndex.names is incorrect
To Reproduce

Provide a minimal runnable pandas example that is not properly checked by the stubs.

In [1]: import pandas as pd

In [2]: df = pd.DataFrame({"a": ["a", "b", "c"], "i": [10, 11, 12]}, index=pd.Index([5, 10, 15], name="idx"))

In [4]: df.index.names
Out[4]: FrozenList(['idx'])

In [5]: df.index.names = (None,)

In [6]: df.index.names
Out[6]: FrozenList([None])

Indicate which type checker you are using (mypy or  pyright).

mypy

Show the error message received from that type checker while checking your example.

bigframes/core/blocks.py:436: error: Incompatible types in assignment (expression has type "tuple[None]", variable has type "list[str]")  [assignment]
Please complete the following information:

OS: [e.g. Windows, Linux, MacOS] Linux
OS Version [e.g. 22]

Distributor ID: Debian
Description:    Debian GNU/Linux rodete
Release:        n/a
Codename:       rodete

python version Python 3.10.9
version of type checker mypy==1.5.1
version of installed pandas-stubs pandas-stubs==<IP_ADDRESS>814

Additional context
Add any other context about the problem here.

There are a few things going on here, and I don't think we can do much about them.
First, from a static typing perspective, we can't track the type of DataFrame.index.  It could be a regular Index, or a MultiIndex.  So if you know your DataFrame is backed by a MultiIndex, you'd have to cast df.index to the MultiIndex type.
Second, whether the underlying index is single-dimensional or a MultiIndex, df.index.names will return a list of strings.  It is also possible to return a list of None if you clobber the names, but if we use static typing to declare that Index.names returns a list[str | None], that will force more people to cast the result of Index.names.
Finally, you reported a mypy error on your own code, but we'd prefer an example that is self-contained and can be run directly through the type checker.  ipython code can't be used that way.
I'm going to close this, but am willing to reopen it if you can convince me otherwise.

names is available on both Index and MultiIndex. I think that's a moot point in regards to this issue.
Isn't list[str] incorrect though? Unnamed indexes are incredibly common. Here's a standalone example:
(dev-3.10-pip) ➜  pandas-stubs-804 cat sample.py
import pandas as pd

df = pd.DataFrame({"a": ["a", "b", "c"], "i": [10, 11, 12]})
print(df.index.names)

# OK
df.index.names = ["idx"]
print(df.index.names)

# Not OK, but works
df.index.names = ("idx2",)
print(df.index.names)
df.index.names = [None]
print(df.index.names)
df.index.names = (None,)
print(df.index.names)

(dev-3.10-pip) ➜  pandas-stubs-804 mypy sample.py
sample.py:11: error: Incompatible types in assignment (expression has type "tuple[str]", variable has type "list[str]")  [assignment]
sample.py:13: error: List item 0 has incompatible type "None"; expected "str"  [list-item]
sample.py:15: error: Incompatible types in assignment (expression has type "tuple[None]", variable has type "list[str]")  [assignment]
Found 3 errors in 1 file (checked 1 source file)

@Dr-Irv , I've added a standalone sample demonstrating the issue.

Output of sample.py showing the default name is None:
(dev-3.10-pip) ➜  pandas-stubs-804 python sample.py
[None]
['idx']
['idx2']
[None]
[None]

Thanks @tswast for the example.  I will reopen.
The property for names should be updated here:  https://github.com/pandas-dev/pandas-stubs/blob/9aac8e31ba69eb4c0583e55dd2198755fb031620/pandas-stubs/core/indexes/base.pyi#L291
in two ways:

The "getter" should return list[str | None]
The "setter" should allow any SequenceNotStr[str] (but not Sequence[str])

PR with tests welcome.

using str | None sounds reasonable. Technically, any type seems to be accepted at runtime:
>>> df.index.names = [None]
>>> df.index.names
FrozenList([None])

>>> df.index.names = [1]
>>> df.index.names
FrozenList([1])

The pandas-internal annotations declare name as any hashable object:
https://github.com/pandas-dev/pandas/blob/e86ed377639948c64c429059127bcf5b359ab6be/pandas/core/indexes/base.py#L1657C5-L1657C32
I couldn't find names, but the doc-string here says that the elements have to be hashable https://github.com/pandas-dev/pandas/blob/e86ed377639948c64c429059127bcf5b359ab6be/pandas/core/indexes/base.py#L1753

Thanks @tswast for the example. I will reopen.
The property for names should be updated here:
https://github.com/pandas-dev/pandas-stubs/blob/9aac8e31ba69eb4c0583e55dd2198755fb031620/pandas-stubs/core/indexes/base.pyi#L291
in two ways:

The "getter" should return list[str | None]
The "setter" should allow any SequenceNotStr[str] (but not Sequence[str])

PR with tests welcome.

Actually I think the getter should also return SequenceNotStr because if you set names as a tuple, you will never be able to return a list[str | None] (since there is no process when we set the names to convert them into anything else than their original name but please correct me if I am missing something.

Actually I think the getter should also return SequenceNotStr because if you set names as a tuple, you will never be able to return a list[str | None] (since there is no process when we set the names to convert them into anything else than their original name but please correct me if I am missing something.

Can you provide an example?
This may be one of the cases where if we declare the getter to return anything that could possibly be returned at runtime, then the most typical usage of returning a list of strings would force people to do  a cast on the result.  So I lean towards supporting the typical usage that lets people avoid using cast in their pandas code.

My bad I did not try it at runtime and thought the type would carry over in the getter but indeed it is a FrozenList that gets returned, let me write a quick test to make sure but I believe you are correct!

Added a PR to see it in action, there still seems to be an issue with the setter type not getting recognize (mypy complains that it expects a list while we allow the setter to receive a SequenceNotStr.
Open to ideas on this one.

Still waiting on mypy with support for getters and setters support from @Dr-Irv's comment https://github.com/pandas-dev/pandas-stubs/pull/1031#issuecomment-2476627879