animated velocity maps

Is there any way to make wind velocity maps like this within the pyviz ecosystem?
I don´t think I can use a ipyleaflet map (which does the velocity map stuff) with panel, or am i wrong there?

I've moved this to geoviews because this is something that I think could feasibly be supported in some form. That said perhaps the easier route would be to wrap leaflet and allow embedding it in panel, both approaches are worth pursuing.

It seems like any animated GIF can already be embedded in Panel, and we can already make animated gifs using cartopy and GeoViews with a matplotlib backend, right?  And I think @philippjfr recently said that the Bokeh backend was ready to have animated gif output support as well, i.e. that it's not a big job and that it would have reasonable performance.  I think that would address this need already, at the HoloViews rather than GeoViews level.
That said, leaflet support would be great to have in Panel, of course.

It seems like any animated GIF can already be embedded in Panel, and we can already make animated gifs using cartopy and GeoViews with a matplotlib backend, right?

Don't think that's what's going on there, what it's doing is rendering something like a vector field with wind velocities client-side and then animating it.

The link is literally to an animated gif, so that's definitely what's happening at that link. :-)  But it sounds like you're proposing a third type of PyViz-ecosystem support for this pattern, beyond (1) generating an animated GIF and embedding that in a Panel or (2) wrapping leaflet within Panel, i.e. (3) adding client-side JS for "live" (not GIF) page-flipping animations.  That option is also useful, but again not specific to GeoViews and also orthogonal to the other two.  All three seem like good, useful functionality to have.

I guess there is a fourth, i.e. (1b): automatically generating an animated GIF in HoloViews without needing to embed it in Panel, which is how the HoloViews Matplotlib-backend gif support works currently if I recall correctly.

I guess I have more context because I've seen demos of that feature in leaflet before. What I'm talking about corresponds to none of the things you listed. What is happening is not a page flipping animation, but rather that the wind streams are actually animated in JS parameterized by the wind speed vectors.

Sure; by page flipping here I just meant rotating between a small number of possible states in a loop using JS, so I think we're talking about the same thing, but you're right that one could distinguish between animating individual glyphs and switching between pre-rendered plots.  So I guess here's the full list of possibilities:

HoloViews generating an animated GIF and displaying that immediately (how the HoloViews Matplotlib-backend gif support works currently, and how the Bokeh backend could work with a little work),
HoloViews exporting an animated GIF that can then be embedded in Panel (should work already whenever 1. works),
Wrapping a Leaflet object as a Panel pane,
HoloViews (or Panel?) adding JS code to switch between a fixed collection of rendered figures (looping or ping-ponging), or
HoloViews adding JS code to animate positions of a set of glyphs each moving according to some other columns' data (looping or ping-ponging).

All of these options seem useful independently, and none of them seem specific to GeoViews.

I may not be fully understanding what is meant by page flipping, but I see a distinction between that and what @philippjfr was describing with JS animated wind streams. The following two examples seem like they are randomly seeding the domain with particles that are animated to move through the vector field. I don't see any looping, so I think this is all happening client side:

PolarGlobe
Windy

In the case of PolarGlobe, at least, they send the raw data to the client and use webGL to render it on the client side (see https://www.tandfonline.com/doi/full/10.1080/13658816.2017.1306863). It seems like an opposite, but complimentary approach from Datashader. In any case I would love to see PyViz support this type of visualization.

Ok, you guys finally got through to me. :-)  Option 6 is indeed distinct from any notion of page flipping, because it's not selecting from a predefined set of items:

Sending a vector field to the client and selecting particles to animate through that vector field at the client side, with no looping or ping-ponging but instead limited particle lifetimes and repeated generation of new particles.

To implement 6 it seems like wrapping an existing library is a good choice, and then if what's wrapped is based on CesiumJS or some other geo-specific library, it would belong in geoviews, but otherwise or if implemented directly, wouldn't need to be GeoViews specific (as vector fields and particles apply to any data).


Create agencies scraper

Reticketed from https://github.com/CivicTechTO/tor-councilmatic/issues/4#issuecomment-196557994

Here's an example of the sort of thing we get when we add this scraper:
Currently, only one member is listed as a member of the "Board of Manage of the Toronto Zoo":
https://tor-councilmatic-staging.herokuapp.com/committee/board-of-management-of-the-toronto-zoo/
But if we look at the agency site (under s lightly different name of "Toronto Zoo"), we can see there are a bunch more, but only the chair lists it on his councillor page:
https://secure.toronto.ca/pa/decisionBody/23.do
http://www1.toronto.ca/wps/portal/contentonly?vgnextoid=69593293dc3ef310VgnVCM10000071d60f89RCRD&vgnextchannel=c3a83293dc3ef310VgnVCM10000071d60f89RCRD
(We also get past appointments from the agency site, which is a perk)
Anyhow, there are a bunch of places like that where the agency site would give us lots more info :)


Remove docker prereqs scripts that only install a package. Add informational message for this.

This PR removes two docker prereqs scripts that only install a single package. This should be done from the setup_docker_prereqs script to keep things clean.
I've also added a note to indicate this.

Thanks 🎉


Use qemu image for multi-arch builds

The build was failing because buildx was activated, but qemu was missing. Because of this, the Travis container could only understand how to build x86 and x86_64. This PR adds linuxkit/binfmt as the qemu image, and now builds for all desired architectures.
Proof:

https://travis-ci.com/github/kodumbeats/sdk-for-cli/builds/218992088
https://hub.docker.com/layers/kodumbeats/appwriteclitest/0.3/images/sha256-93be81552c5f720da8b08b0cba87ef4707e6c50ec2091eec3bc29d88882abd93?context=explore

Awesome work @kodumbeats as discussed, we'll move this PR to the generator

https://github.com/appwrite/sdk-generator/pull/136


Support reading PCAP files that are still being written to

Support a new command line option that tells capture to process pcap files that are still open for writing. This will be useful with suricata 7 and other tools that write out large pcaps we want to process as they are being written to.
This will probably require scheme mode so we have direct access, which means we need to add monitor mode to scheme. Then when the file is closed move to the next file?
I'm not sure this is even possible

I couldn't figure out how to do this. You could do a tail -f type feature, but you could already do that with cat pcap | capture -r -


[SNOW-1445866]: Add support for DataFrame.assign

Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

Fixes SNOW-NNNNNNN

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code

[ ] If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing

[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency
[ ] If this is a new feature/behavior, I'm adding the Local Testing parity changes.

Please describe how your code solves the related issue.
Please write a short description of how your code change solves the related issue.

@sfc-gh-rdurrani I'd suggest you rebase this PR on latest main, your build doc failure will get resolved

Can we update the PR description?


Make tests check only run when a PR is opened or reopened

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes tests for all changed/updated/fixed behaviors.
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (melos run analyze) does not report any problems on my PR.
[x] The package compiles with the minimum Flutter version stated in the pubspec.yaml

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

I improved your job and extended it. Now it lists the untested dart files.
jobs:
  check-for-tests:
    runs-on: ubuntu-latest
    steps:
      # Step 1: Checkout the code from the repository
      - uses: actions/checkout@v2
        with:
          fetch-depth: 0  # Fetch full history to allow full diff checking

      # Step 2: Identify changed .dart files
      - name: Identify changed .dart files
        id: identify-changes
        run: |
          # Get a list of all added or modified .dart files in the PR
          git diff --name-only --diff-filter=AM origin/master | grep '\.dart$' > changed_files.txt

          # Output the changed files for debugging
          echo "Changed .dart files:"
          cat changed_files.txt

      # Step 3: Check for corresponding test files
      - name: Check for corresponding test files
        id: check-tests
        run: |
          # Initialize an empty variable to store files without corresponding tests
          missing_tests=""

          # Loop over each changed .dart file
          while IFS= read -r file; do
            # Skip files that match ignore patterns
            if [[ $file =~ ^(lib/localization/|lib/features/self_service_training/view/manuals/|integration_test/|.*\.g\.dart|.*\.freezed\.dart)$ ]]; then
              continue
            fi

            # Check if the file is not a test file and whether it has a corresponding _test.dart file
            if [[ ! $file =~ _test\.dart$ ]]; then
              test_file="${file%.*}_test.dart"
              if ! grep -q "$test_file" changed_files.txt; then
                missing_tests="${missing_tests}${file}\n"
              fi
            fi
          done < changed_files.txt

          # Write the list of files missing tests to a file for the next step
          echo -e "$missing_tests" > missing_tests.txt

          # Output the files missing tests for debugging
          echo "Files missing tests:"
          cat missing_tests.txt

      # Step 4: Comment on the PR if tests are missing
      - name: Comment on PR if tests are missing
        uses<EMAIL_ADDRESS>        with:
          script: |
            const fs = require('fs');
            const missingFiles = fs.readFileSync('missing_tests.txt', 'utf-8').split('\n').filter(file => file);

            console.log("Missing files (parsed):", missingFiles);

            // Base comment message indicating the lack of tests, with a friendly tone
            let body = '👋 Hi there! It looks like some files in this PR do not have corresponding tests. 🧪 We aim to maintain high test coverage to ensure code quality and prevent regressions. All changes, especially those affecting core functionality, should be accompanied by relevant tests.';

            // If there are any missing files, append them to the comment
            if (missingFiles.length > 0) {
              body += `\n\n🔍 The following files were modified without corresponding tests:\n${missingFiles.map(file => `- ${file}`).join('\n')}`;
            }

            // Log the final body message for debugging
            console.log("Final comment body:", body);

            // Post the comment on the PR
            github.rest.issues.createComment({
              issue_number: context.issue.number,
              owner: context.repo.owner,
              repo: context.repo.repo,
              body: body
            });

@ulusoyca Excellent. I'll close this since it makes a very tiny change.


How do i dismiss keyboard when clicked outside of the textfield widget.

I have a column with a textfield  (numeric keyboard) between 2 containers. The keyboard comes up when clicked inside the TextField. I want to dismiss the keyboard when clicked outside the TextField.
How can I achieve this. Below is the gist that shows the issue.
https://gist.github.com/manujbahl/874ab6c7ec8459899968bfb6fb367d6b

You can unfocus using
FocusScope.of(context).requestFocus(new FocusNode())

that will hide the keyboard

It has since stopped wotking

Can you please provide more information?

the code that allows to reproduce
the output of flutter doctor -v?
the current behavior: Does "stopped working" that above code doesn't release focus and the keyboard stays on screen?

added info there.

need a clear & simple solution

What about https://github.com/flutter/flutter/issues/7247#issuecomment-412226765 ?

I am having the same problem. This technique is not working for me. I think this stopped working in one of the recent Flutter versions.

 @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(AppLocalizations.of(context).calculatorAdvancedStageTitle),
      ),
      body: GestureDetector(
        behavior: HitTestBehavior.opaque,
        onTap: () {
          FocusScope.of(context).requestFocus(new FocusNode());
        },
        child: Padding(
          padding: const EdgeInsets.only(
            left: 14,
            top: 8,
            right: 14,
            bottom: 8,
          ),
          child: Text('Work'),
        ),
      )
    );
  }

Works for me.

@atereshkov unfortunately, this doesn't work if some of the widgets outside have own onTap callbacks (i.e. ExpandedListTile or buttons)

A slightly better way that doesn't request focus to an empty node is to use unfocus().
    return GestureDetector(
      onTap: () {
        FocusScopeNode currentFocus = FocusScope.of(context);

        if (!currentFocus.hasPrimaryFocus) {
          currentFocus.unfocus();
        }
      },
      child: MaterialApp(
        title: 'Flutter Demo',
        theme: ThemeData(
          primarySwatch: Colors.blue,
        ),
        home: MyHomePage(),
      ),
    );

Full details in this post: https://flutter360.dev/dismiss-keyboard-form-lose-focus/

This actually doesn't change anything. You still can't detect a tap outside if you tap on an widged with its own gesture detector

@caseyryan can you provide an example?

I don't have a possibility now. But the scenario is like this:
using your example, create a page containing a FlatButton. Tap on that button, and you will see it intercepts taps and your top level onTap: () will not trigger. I'm still struggling with it to make a textfield lose focus if I tap on anything, including widgets with GestureDetectors

The code above is to simulate the behavior of tapping outside of a text field (ex: tapping on the background of a page) to dismiss the keyboard.
However, the same principle can be applied to dismiss the keyboard when tapping a button. In your onPressed function of your button, you'd get the current FocusNode and call unfocus().

The code above is to simulate the behavior of tapping outside of a text field (ex: tapping on the background of a page) to dismiss the keyboard.
However, the same principle can be applied to dismiss the keyboard when tapping a button. In your onPressed function of your button, you'd get the current FocusNode and call unfocus().

I understand what's this code for. I just say it doesn't work as expected. I think you'd agree if I say that it's a bad option to call focus change from all widgets that might have GestureDetectors inside, plus another developer, who would support a code might just not know about such a crunch. It's definitely a large issue of flutter not having a functionality to solve such a basic problem out of the box

I understand what's this code for. I just say it doesn't work as expected.

But it does! Just not for the use case you outlined.

I think you'd agree if I say that it's a bad option to call focus change from all widgets that might have GestureDetectors inside, plus another developer, who would support a code might just not know about such a crunch.

Agreed - ideally, this wouldn't be necessary. But as far as I know, there is no better option at the moment.

It's definitely a large issue of flutter not having a functionality to solve such a basic problem out of the box. I think flutter's textfield must be able to lose focus by itself if a tap occured somewhere else

Couldn't agree more. If you find a better solution, please do share!

Couldn't agree more. If you find a better solution, please do share!

Well, I think I've already found one. I mean, it's almost the same as your method but using Listener instead of GestureDetector. Listener pays no mind to any nested GestureDetectors and always fires.
So, it's just like that:
`
@override
Widget build(BuildContext context) {
SystemChrome.setPreferredOrientations([DeviceOrientation.portraitUp]);
return Listener(
onPointerUp: (e) {
print("TAP");
},
child: MaterialApp(
title: 'Makapizza',
debugShowCheckedModeBanner: false,
theme: Theme.of(context).platform == TargetPlatform.iOS
? iOSTheme
: defaultTheme,
home: MyHomePage(title: 'Home'),
),
);
}
`
Tap is always there in the console. I just need to find a way to check if its a TextField that was clicked or not. But I'll postpone it until morning :)

Nice find! I'll give that a shot.

So, it's morning, and I've found a solution. Listener itself provides no way of checking a type of target but it provides a pointer position which can be used for this purpose.
I came up with this widget:
import 'package:flutter/rendering.dart';
import 'package:flutter/widgets.dart';

class FocusWatcher extends StatelessWidget {

  final Widget child;

  FocusWatcher({@required this.child});

  @override
  Widget build(BuildContext context) {
    return Listener(
      onPointerUp: (e) {
        var renderBox = context.findRenderObject() as RenderBox;
        var result = BoxHitTestResult();
        renderBox.hitTest(result, position: e.position);
        var isEditable = result.path.toList().any((entry) => entry.target.runtimeType == RenderEditable);
        if (!isEditable) {
          var currentFocus = FocusScope.of(context);
          if (!currentFocus.hasPrimaryFocus) {
            currentFocus.unfocus();
          }
        }
      },
      child: child,
    );
  }
}

And to use it, you just need to wrap the whole app in it
@override
Widget build(BuildContext context) {
  return FocusWatcher(
    child: MaterialApp(
      title: 'App',
      debugShowCheckedModeBanner: false,
      theme: Theme.of(context).platform == TargetPlatform.iOS
          ? iOSTheme
          : defaultTheme,
      home: MyHomePage(title: 'App'),
    ),
  );
}

It checks all objects under pointer and if finds a RenderEditable (e.g. TextField) it does nothing and the soft keyboard stays open, but if there's no editable object there, it requests unfocus and the keyboard hides.

thank you! i almost went crazy trying to find out, why my dropdowns weren't closing on outside tap

thank you! i almost went crazy trying to find out, why my dropdowns weren't closing on outside tap

You're welcome :) Please, check my pub package https://pub.dev/packages/flutter_focus_watcher I've got an updated version of this solution which has some important additions

thank you! i almost went crazy trying to find out, why my dropdowns weren't closing on outside tap

You're welcome :) Please, check my pub package https://pub.dev/packages/flutter_focus_watcher I've got an updated version of this solution which has some important additions

Awesome!! It works, thanks for sharing, although it has a minor bug when clicked on a widget which has an active onTap functionality, that gets triggered along with unfocusing the Textfield. eg: With this code, when clicked on an icon button with functionality defined, the functionality of that icon button gets triggered along with unfocusing the Textfield. Any suggestions?

thank you! i almost went crazy trying to find out, why my dropdowns weren't closing on outside tap

You're welcome :) Please, check my pub package https://pub.dev/packages/flutter_focus_watcher I've got an updated version of this solution which has some important additions

Awesome!! It works, thanks for sharing, although it has a minor bug when clicked on a widget which has an active onTap functionality, that gets triggered along with unfocusing the Textfield. eg: With this code, when clicked on an icon button with functionality defined, the functionality of that icon button gets triggered along with unfocusing the Textfield. Any suggestions?

I'm not sure what exactly is wrong, but there's also an IgnoreFocusWatcher widget in the same package. Try wrapping your button with it, and FocusWatcher will not be triggered in that case

thank you! i almost went crazy trying to find out, why my dropdowns weren't closing on outside tap

You're welcome :) Please, check my pub package https://pub.dev/packages/flutter_focus_watcher I've got an updated version of this solution which has some important additions

Awesome!! It works, thanks for sharing, although it has a minor bug when clicked on a widget which has an active onTap functionality, that gets triggered along with unfocusing the Textfield. eg: With this code, when clicked on an icon button with functionality defined, the functionality of that icon button gets triggered along with unfocusing the Textfield. Any suggestions?

I'm not sure what exactly is wrong, but there's also an IgnoreFocusWatcher widget in the same package. Try wrapping your button with it, and FocusWatcher will not be triggered in that case
IngnoreFocusWatcher(
  child: IconButton(...)
);

Something like this

I want the FocusWatcher to be triggered all the time, anywhere I press on the screen. So IgnoreFocusWatcher wouldn't make sense right?

I want the FocusWatcher to be triggered all the time, anywhere I press on the screen. So >IgnoreFocusWatcher wouldn't make sense right?
Then you should use ForceFocusWatcher which does exactly opposite

https://github.com/caseyryan/flutter_focus_watcher
is that specifically only for textBox widget? Was hoping for solution on closing bottomSheet widget with this similar fashion.....

https://github.com/caseyryan/flutter_focus_watcher
is that specifically only for textBox widget? Was hoping for solution on closing modal bottomSheet widget with this similar fashion..... I need this functionality for integration testing bottomsheet opening and closing

Not only. I've also created a widget called ForceFocusWatcher. Just wrap any widget you want with it and you're good to go.  I've also got a similar but simplier solution here https://github.com/caseyryan/flutter_multi_formatter/blob/master/lib/utils/unfocuser.dart Just copy this class to your project and wrap your bottom sheet with ForceUnfocuser


Basic auth with form data (react/redux)

I'm trying to replicate a working auth request with axios (currently implemented using reqwest). I've read through #167 and #97 and have gotten close, but still not quite there.
Here's a working example using reqwest.

I tried duplicating the request exactly (according to the network tab) using axios, and this is what I came up with (using a suggested transform workaround from one of the threads above):

From the network tab, this solution appears to have worked for the request being manipulated, but the next Redux action (loadCurrentUser) that gets dispatched from that .then success fails, giving me the following error (which I did not get with the original reqwest solution):

No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:3000' is therefore not allowed access. The response had HTTP status code 401.

I also tried using params in the config and received the exact same error.
What am I doing wrong here?

It doesn't look like you need to use that transform workaround. There are easier ways to convert a JavaScript object to the application/x-www-form-urlencoded format. Please see this comment.

Closing due to inactivity. Please feel free to reopen.

Great, the qs library worked, thanks!


Xcode IOS run YogaKit.modulemap' not found

Description
/Users/qinque/Library/Developer/Xcode/DerivedData/otam-hgtpqpfwdvlloxhkrzdsjqyxdctg/Build/Intermediates.noindex/otam.build/Debug-iphonesimulator/otam.build/DerivedSources/otam_vers.c module map file '/Users/qinque/Library/Developer/Xcode/DerivedData/otam-hgtpqpfwdvlloxhkrzdsjqyxdctg/Build/Products/Debug-iphonesimulator/YogaKit/YogaKit.modulemap' not found
When Xcode runs, it prompts an error

React Native Version
0.70.4
Output of npx react-native info
info Fetching system and libraries information...
System:
OS: macOS 13.0
CPU: (8) arm64 Apple M1
Memory: 93.95 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 16.14.2 - ~/.nvm/versions/node/v16.14.2/bin/node
Yarn: 1.22.19 - /opt/homebrew/bin/yarn
npm: 8.5.0 - ~/.nvm/versions/node/v16.14.2/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.12.1 - /Users/qinque/.rvm/gems/ruby-2.7.5/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.1, iOS 16.1, macOS 13.0, tvOS 16.1, watchOS 9.1
Android SDK: Not Found
IDEs:
Android Studio: Not Found
Xcode: 14.1/14B47b - /usr/bin/xcodebuild
Languages:
Java: Not Found
npmPackages:
@react-native-community/cli: Not Found
react: 18.1.0 => 18.1.0
react-native: 0.70.4 => 0.70.4
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
xxx. xcodeproj  Xcode -> build
Snack, code example, screenshot, or link to a repository

And I found that the Libraries file is empty

You are on a supported minor version, but it looks like there's a newer patch available. Please upgrade to the highest patch for your minor or latest and verify if the issue persists (alternatively, create a new project and repro the issue in it). If it does not repro, please let us know so we can close out this issue. This helps us ensure we are looking at issues that still exist in the most recent releases.

@w1687021088 as the bot suggested please verify on latest patch version(s) as well.

Does anyone has any update for this issue ?

Leaving a comment to get any update because I'm also facing this issue on Xcode 14.2 - RN 0.68.5

I'm able to run in a simulator using npx react-native run-ios but it never builds.

I'm able to run in a simulator using npx react-native run-ios but it never builds.

Same for me.

i have same issue for that error

i have same issue for that error with mac m1

Try to do the followings to fix this problem:

Using XCode to open your workspace
Click Project navigator then choose your project
On the right panel, make sure Build Settings of Your Project and Target with the same settings of Excluded Architectures
For Pods project, also make sure your Build Settings with the same value as above

Try to do the followings to fix this problem:

Using XCode to open your workspace
Click Project navigator then choose your project
On the right panel, make sure Build Settings of Your Project and Target with the same settings of Excluded Architectures
For Pods project, also make sure your Build Settings with the same value as above

I am following this step. still same issue for that error. any solutions?

Try to do the followings to fix this problem:

Using XCode to open your workspace
Click Project navigator then choose your project
On the right panel, make sure Build Settings of Your Project and Target with the same settings of Excluded Architectures
For Pods project, also make sure your Build Settings with the same value as above

I have been following this step. still same the error, Any suggestions?

this is the same issue i'm getting but with firebasecore


Allow overriding external IP of multi-cluster east-west gateways

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
I'm using a multi-primary multi-network Istio setup across 3 clusters, and 2 of those clusters are behind NAT. Those 2 clusters use MetalLB to get loadbalancer services, which works great. The issue is that due to MetalLB setting the status of those loadbalancers to an internal IP (which is correct), Istio then tries to use that IP to route traffic to the other clusters east-west gateway.
Describe alternatives you've considered
After digging around in the documentation and in the code for a bit, I saw that Istio also respects the externalIPs attribute of services, however the IPs configured there are added to the ones in the status, which would result in about half of the requests failing.
There are two three solutions for this (at least that I came up with)

taking the value of externalIPs over the status IP, which would allow specifying a gateway's public IP via that field, allthough this would be a backwards incompatible change
Allow changing the priority that the LoadBalancer status is used, as currently IPs in the status have a higher priority than Hostnames, and if that could be customised then that could be used to override the external address
Adding an optional annotation to the LoadBalancer service, which would allow overriding the address via an annotation, this has the advantage that it would also work with DNS-based addresses and not just raw IPs

The latter would be imo the better choice, and to test it I've created a small patch locally, but I thought it might be a good idea to open an issue for discussion before creating a PR.
diff --git a/pilot/pkg/serviceregistry/kube/conversion.go b/pilot/pkg/serviceregistry/kube/conversion.go
index 075d62d171..bd383d3fd7 100644
--- a/pilot/pkg/serviceregistry/kube/conversion.go
+++ b/pilot/pkg/serviceregistry/kube/conversion.go
@@ -42,6 +42,8 @@ const (
        // It is used for multi-cluster scenario, and with nodePort type gateway service.
        // TODO: move to API
        NodeSelectorAnnotation = "traffic.istio.io/nodeSelector"
+
+       ExternalAddressAnnotation = "traffic.istio.io/externalAddress"
 )

 func convertPort(port corev1.ServicePort) *model.Port {
@@ -167,6 +169,9 @@ func ConvertService(svc corev1.Service, domainSuffix string, clusterID cluster.I
        istioService.Attributes.NodeLocal = nodeLocal
        istioService.Attributes.ClusterExternalAddresses.AddAddressesFor(clusterID, svc.Spec.ExternalIPs)

+       if addr, ok := svc.Annotations[ExternalAddressAnnotation]; ok {
+               istioService.Attributes.ClusterExternalAddresses.SetAddressesFor(clusterID, []string{addr})
+       }
        return istioService
 }

Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[x] Multi Cluster
[ ] Virtual Machine
[x] Multi Control Plane
Additional context

I too need this feature.
Is there any additional information on this enhancement?

We do also  support access via nodePort typed service

We do also support access via nodePort typed service

I don't think using NodePort will solve this problem:'-(
Because the Kubernetes node behind the NAT should have a private ip, and you cannot connect directly to that private ip from a cluster outside the NAT.

Any further update for this?


New version: Nvidia.GeForceNow version <IP_ADDRESS>

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

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

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/114324&drop=dogfoodAlpha

/azp run


Freer rigid selective functor

Freer Rigid Selective Functor (allow to write Freer Selective functor without having a functor)
Maybe cc @snowleopard (if you have the time)

Hi, since your test is well working, I guess that it is correct.
Just, can you just regenerate specs.svg? (it is not done by the CI for not having DOT as dependency).

Yes, but it bores me deeply. I don't see any good reason not to generate the SVG in a dedicated "generate SVGs" action.

Thanks a lot for your reviews!
Since Didier, Andrey and Antoine approved this> I guess I can merge!


Fix misspelled "labels" in series.py

Tried to convert a series to spark mode and got an error. Looks like labels was just misspelled.

ah good catch!


Adds support for syncing events for more than one day

Is it possible to copy the location information over?


ReadLargeFile with no headers = empty columns array

When using the read() method, and I don't provide headers and don't specify that the first row is headers, the library returns a query with columns named column1, etc That is convenient syntax to work with and makes the code more readable.  When using readLargeFile and returning a struct, it would be great if you could do the same thing with the columns array.

I think you're talking about readCsv() rather than readLargeFile().
The read() method does that because a query must have column names. readCsv() returns an array of arrays which doesn't require them. If you don't specify them then I think it's right that columns should be empty. If you want to add them you can using the withHeader() method:
spreadsheet.readCsv( path )
  .intoAnArray()
  .withHeader( [ "column1", "column2", "column3" ] )
  .execute();

The header values don't have to appear in the CSV, they will still be used if not present.


🐛 Cannot create model with DateOnly field

Describe the Bug
Trying to create a model that has a DateOnly field throws this exception:
AutoFixture.ObjectCreationExceptionWithPath : AutoFixture was unable to create an instance from System.DateOnly because creation unexpectedly failed with exception. Please refer to the inner exception to investigate the root cause of the failure.

Request path:
	System.DateOnly

Inner exception messages:
	System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
	  System.ArgumentOutOfRangeException: Year, Month, and Day parameters describe an un-representable DateTime.

  ----> System.Reflection.TargetInvocationException : Exception has been thrown by the target of an invocation.
  ----> System.ArgumentOutOfRangeException : Year, Month, and Day parameters describe an un-representable DateTime.

  Stack Trace:
TerminatingWithPathSpecimenBuilder.Create(Object request, ISpecimenContext context)
BehaviorRoot.Create(Object request, ISpecimenContext context)
RecursionGuard.Create(Object request, ISpecimenContext context)
Fixture.Create(Object request, ISpecimenContext context)
SpecimenContext.Resolve(Object request)
SeedIgnoringRelay.Create(Object request, ISpecimenContext context)
CompositeSpecimenBuilder.Create(Object request, ISpecimenContext context)
CompositeSpecimenBuilder.Create(Object request, ISpecimenContext context)
Postprocessor`1.Create(Object request, ISpecimenContext context)
AutoPropertiesTarget.Create(Object request, ISpecimenContext context)
<18 more frames...>
CompositeSpecimenBuilder.Create(Object request, ISpecimenContext context)
CompositeSpecimenBuilder.Create(Object request, ISpecimenContext context)
Postprocessor`1.Create(Object request, ISpecimenContext context)
AutoPropertiesTarget.Create(Object request, ISpecimenContext context)
CompositeSpecimenBuilder.Create(Object request, ISpecimenContext context)
TerminatingWithPathSpecimenBuilder.Create(Object request, ISpecimenContext context)
--ArgumentOutOfRangeException
DateOnly.ctor(Int32 year, Int32 month, Int32 day)
RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
MethodBaseInvoker.InvokeDirectByRefWithFewArgs(Object obj, Span`1 copyOfArgs, BindingFlags invokeAttr)

Scenario
Running below over-simplified unit test throws above exception
using AutoFixture;
using NUnit.Framework;

namespace MyNamespace;

internal class dummy
{
    Fixture _fixture = new Fixture();

    [Test]
    public void MyTest()
    {
        _fixture.Create<ClassWithDateOnlyField>();
    }

    private class ClassWithDateOnlyField
    {
        public DateOnly myDateOnly { get; set; }
    }
}

Expected Behavior
I expected to get an instance of ClassWithDateOnlyField with a random DateOnly value in myDateOnly

DateOnly and TimeOnly will be supported in AF5.
Until then, I use these two configurations
        fixture.Customize<DateOnly>(o => o.FromFactory((DateTime dt) => DateOnly.FromDateTime(dt)));
        fixture.Customize<TimeOnly>(o => o.FromFactory((DateTime dt) => TimeOnly.FromDateTime(dt)));

Okay, thanks for the work around. Should I close the bug or do we close it when AF5 is released?

technically it's not a bug, so I think this should be closed. but let's hear it from @aivascu

The reason I created it as a bug was because it says "creation unexpectedly failed with exception". Unexpected exceptions is in my world often bugs, otherwise it would've been expected, right? Of course this is only my generalized view and I understand that your definition of a bug might be different.

I believe #1304 and PR #1305 are relevant.


@ attribute error

warning: `xml_test` (bin "xml_test") generated 1 warning (run `cargo fix --bin "xml_test"` to apply 1 suggestion)
    Finished dev [unoptimized + debuginfo] target(s) in 3.94s
     Running `target/debug/xml_test`
thread 'main' panicked at src/main.rs:13:68:
called `Result::unwrap()` on an `Err` value: Custom { field: "missing field `@EID`" }
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

<?xml version="1.0" encoding="UTF-8"?>
<MVCI_MODULE_DESCRIPTION FILE_VERSION="<IP_ADDRESS>" MVCI_PART2_STANDARD_VERSION="2.1.1">
  <PINTYPE EID="ID_HI">
    <ID>0</ID>
    <SHORT_NAME>HI</SHORT_NAME>
  </PINTYPE>
  <PINTYPE EID="ID_LOW">
    <ID>1</ID>
    <SHORT_NAME>LOW</SHORT_NAME>
  </PINTYPE>
</MVCI_MODULE_DESCRIPTION>

use structs::MvciModuleDescription;

mod structs;

fn main() {
    let xml = std::fs::read_to_string("./test.xml").unwrap();

    let mut reader = quick_xml::reader::Reader::from_str(&xml);
    if let Ok(root) = xml_schema_generator::into_struct(&mut reader) {
        println!("{}", root.to_serde_struct());
    }

    let root: MvciModuleDescription = serde_xml_rs::from_str(&xml).unwrap();
}

but if i change @EID to EID it works. i'm guessing it's a serde-xml-rs problem?

Your XML does not contain an @EID attribute. I assume your original XML contains something like <PINTYPE @EID="ID_HI">?

I guess I understand now. I use quick_xml::de::from_str(xml) so far. I'll take a look at the differences to serde_xml_rs::from_str(xml)

@brandonros I am currently implementing an option to convert to a serde_xml_rs compatible version. Two questions here:

How should the resulting struct look like for your example? I fail to create something that works
serde_xml_rs did not receive updated for 1 year. Are you sure it is worth to build on this? Maybe switching to quick_xml is the better choice

use serde::{Serialize, Deserialize};

#[derive(Serialize, Deserialize)]
pub struct MvciModuleDescription {
    #[serde(rename = "PINTYPE")]
    pub pintype: Vec<Pintype>,
}

#[derive(Serialize, Deserialize)]
pub struct Pintype {
    #[serde(rename = "ID")]
    pub id: String,
    #[serde(rename = "SHORT_NAME")]
    pub short_name: String,
    #[serde(rename = "EID")]
    pub eid: String,
    #[serde(rename = "$text")]
    pub text: Option<String>,
}

I'm fine with closing this. Thanks for taking a look. I didn't know it'd be incompatible with serde_xml_rs from the README and I didn't know to look for an altnernative like quick_xml
As long as that's called out, I think this is great. No reason to waste cycles on old packages like you said. It would increase complexity. Feature flag quick_xml vs serde_xml I guess...
Either way, Thank you!


Redirect not work with relative path

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[ ] I'm using the latest version of Docusaurus.
[ ] I have tried the npm run clear or yarn clear command.
[ ] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[ ] I have read the console error message carefully (if applicable).

Description
Well, I'm using the 2.0.1 version.
If you open https://pulsar.apache.org/docs/standalone, although its source content is:
---
id: standalone
title: Set up a standalone Pulsar locally
sidebar_label: "Run Pulsar locally"
---

````mdx-code-block
import {Redirect} from '@docusaurus/router';

<Redirect to="getting-started-standalone" />
````

It's redirected to https://pulsar.apache.org/docs/standalone/getting-started-standalone instead of https://pulsar.apache.org/docs/getting-started-standalone.
Moreover, it works well when previewing locally.
Reproducible demo
No response
Steps to reproduce
As described above.
Expected behavior
Correctly redirected to https://pulsar.apache.org/docs/getting-started-standalone.
Actual behavior
Wrongly redirected to https://pulsar.apache.org/docs/standalone/getting-started-standalone.
Your environment

Public source code: https://github.com/apache/pulsar-site/tree/main/site2/website-next
Public site URL: https://pulsar.apache.org/docs/standalone
Docusaurus version used: 2.0.1
Environment name and version (e.g. Chrome 89, Node.js 16.4):
Operating system and version (e.g. Ubuntu 20.04.2 LTS):

Self-service

[ ] I'd be willing to fix this bug myself.

This is due to your host doing a server-side redirect to standalone/ so the relative link is resolved client-side in a different way

You may have configured trailingSlash incorrectly for your specific host, leading to issues. Make sure all URLs are always served with/without a trailing slash consistently, and adjust your relative links accordingly

Thank you!


Each next selectorsToIgnore overwrites previous one

It is not possible now to use code like this:
suite ...
  .selectorsToIgnore('.selector-one')
  .selectorsToIgnore('.selector-two')

The second call sets selectors to ignore to selector-two only. Previous call is no longer taken into account.
This is very inconvenient when making tests for a bunch of similar pages. Ideally I would set up selector to ignore in the common test and then extend this common test with additions valuable for a specific pages. A kind of like require('specific-test.js')() in which selectorsToIgnore make also be called.
Currently I made a workaround in which the selectors to ignore are gathered into an array and the function is called afterall. But this solution looks ugly and the possibility to call selectorsToIgnore would make everything much nicer.

Thanks for report, I agree that it will improve usability of the API a lot. We'll schedule it to the next release after 0.13


Adds the shortcut to input enclosing numbers, also maps keys on the num pad.

The PR adds a shortcut in the menu, which appears when a user selects Ctrl + , to let users to input enclosing numbers like ⒒, ㉒, and so on.
The PR also maps the keys on the number pad. A user can use the arrow keys on the number pad to move the cursor. However, it looks like Fcitx does not have the API to set the keys on the number pad as candidate keys.

I updated the PR. Now we can use the keys on the number pad to select the candidates. It should fix #141.


Python client: queue and job_set_id are now required parameters to Armada API fixing python client to enforce that

As part of ongoing work simplify Armada API surface, now queue and job_set_id - are both required parameters for cancel / reprioritize requests.
This enables over longer period, for us to reduce redis dependency (for non-event stream use-cases)

Closed in favour of - https://github.com/armadaproject/armada/pull/3588


salmentions!

@acegiak has done awesome work implementing the salmention prototype in the upstream repo. it sounds like it might not be polished enough to merge in here, but i'd love to see that happen eventually. pretty please with sugar on top!

yeah the reason I haven't done that merge is I'm a little embarrassed of the hacky nature of it but maybe it's better to merge it THEN improve it rather than the other way around.

I've been looking at the code, and it is integrated into the generate comment data function. I think it would be better if the comment data function stored the parsed mf2, which is recommended in the specification, and then the salmentions code was separate.


Fix #1552: update label on deploy page

Description
Correct the label on the deploy script page.
Task Item
Closes #1552
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Checklist

[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have functionally tested my change

Screenshots
Please include screenshots or gifs if your PR include UX changes.

@benbrown can I scope creep this a bit and ask that you add some padding above the button elements to create some space between them and their associated textareas?

@benbrown can I scope creep this a bit and ask that you add some padding above the button elements to create some space between them and their associated textareas?

good call.

done!


Fixed order of operations

Hope this helps:)

Also you gonna have to fix my grammar again srry

Further editing of this portion shouldn't be needed as it is planned to integrate gnupg into the repo updation process, once again, thanks for contributing! :)


Add isShutdown

It is useful to know if a connection was closed. So we should expose this attribute.

This tightly couples the API to a detail of the implementation.
I question how you want to use this information, as well.  Where do you have access to a TerracottaConnection instance without a downcast?

I was needing it when implementing a ClusterManager. To know if the cluster is active. So yes I do downcast.

Not sure of the exact usage here but there are several implementations of Connection so ClusterManager may not always be able to downcast.  I don't think we should do this.  It would be better for the caller of Connection.close() to track state.  Closing this PR.  If you absolutely require Connection to expose state, please file an issue in terracotta-apis as an API discussion


starship: 1.17.0 -> 1.17.1

Description of changes
starship: 1.17.0 -> 1.17.1
https://github.com/starship/starship/releases/tag/v1.17.1
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

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

1 package built:

starship

Not sure if it is related to this PR, but I'm unable to update to 1.17.1 from 1.17.0 on aarch64-darwin. Getting:
The following warnings were emitted during compilation:

warning: error: unable to open output file '/homeless-shelter/.cache/clang/ModuleCache/I2FLID7C1CDF/Cocoa-3N46Q1ILJJI73.pcm': 'No such file or directory'
warning: In file included from objc/notify.m:1:
warning: objc/notify.h:1:9: fatal error: could not build module 'Cocoa'
warning: #import <Cocoa/Cocoa.h>
warning:  ~~~~~~~^
warning: error: unable to open output file '/homeless-shelter/.cache/clang/ModuleCache/I2FLID7C1CDF/CoreServices-2KVNYK9WO8W58.pcm': 'No such file or directory'
warning: error: unable to open output file '/homeless-shelter/.cache/clang/ModuleCache/I2FLID7C1CDF/Foundation-36W7A6K9XC7LQ.pcm': 'No such file or directory'
warning: error: unable to open output file '/homeless-shelter/.cache/clang/ModuleCache/I2FLID7C1CDF/ObjectiveC-1MCLJG09OQRJB.pcm': 'No such file or directory'
warning: 5 errors generated.

error: failed to run custom build command for `mac-notification-sys v0.6.1`

Caused by:
  process didn't exit successfully: `/private/tmp/nix-build-starship-1.17.1.drv-0/source/target/release/build/mac-notification-sys-03b7cfeae5cf7bc0/build-script-build` (exit status: 1)
  --- stdout
  TARGET = Some("aarch64-apple-darwin")
  OPT_LEVEL = Some("3")
  HOST = Some("aarch64-apple-darwin")
  cargo:rerun-if-env-changed=CC_aarch64-apple-darwin
  CC_aarch64-apple-darwin = None
  cargo:rerun-if-env-changed=CC_aarch64_apple_darwin
  CC_aarch64_apple_darwin = None
  cargo:rerun-if-env-changed=HOST_CC
  HOST_CC = Some("/nix/store/vwh2qqal1q101wv4jy40giavp73q0msd-clang-wrapper-16.0.6/bin/cc")
  cargo:rerun-if-env-changed=CRATE_CC_NO_DEFAULTS
  CRATE_CC_NO_DEFAULTS = None
  DEBUG = Some("false")
  CARGO_CFG_TARGET_FEATURE = Some("aes,crc,dit,dotprod,dpb,dpb2,fcma,fhm,flagm,fp16,frintts,jsconv,lor,lse,neon,paca,pacg,pan,pmuv3,ras,rcpc,rcpc2,rdm,sb,sha2,sha3,ssbs,vh")
  cargo:rerun-if-env-changed=CFLAGS_aarch64-apple-darwin
  CFLAGS_aarch64-apple-darwin = None
  cargo:rerun-if-env-changed=CFLAGS_aarch64_apple_darwin
  CFLAGS_aarch64_apple_darwin = None
  cargo:rerun-if-env-changed=HOST_CFLAGS
  HOST_CFLAGS = None
  cargo:rerun-if-env-changed=CFLAGS
  CFLAGS = None
  running: env -u IPHONEOS_DEPLOYMENT_TARGET "/nix/store/vwh2qqal1q101wv4jy40giavp73q0msd-clang-wrapper-16.0.6/bin/cc" "-O3" "-ffunction-sections" "-fdata-sections" "-fPIC" "-arch" "arm64" "-Wall" "-Wextra" "-fmodules" "-Wno-deprecated-declarations" "-mmacos-version-min=11.0" "-o" "/private/tmp/nix-build-starship-1.17.1.drv-0/source/target/aarch64-apple-darwin/release/build/mac-notification-sys-723f11dcedbc426f/out/objc/notify.o" "-c" "objc/notify.
m"
  cargo:warning=error: unable to open output file '/homeless-shelter/.cache/clang/ModuleCache/I2FLID7C1CDF/Cocoa-3N46Q1ILJJI73.pcm': 'No such file or directory'

  cargo:warning=In file included from objc/notify.m:1:

  cargo:warning=objc/notify.h:1:9: fatal error: could not build module 'Cocoa'

  cargo:warning=#import <Cocoa/Cocoa.h>

  cargo:warning= ~~~~~~~^

  cargo:warning=error: unable to open output file '/homeless-shelter/.cache/clang/ModuleCache/I2FLID7C1CDF/CoreServices-2KVNYK9WO8W58.pcm': 'No such file or directory'

  cargo:warning=error: unable to open output file '/homeless-shelter/.cache/clang/ModuleCache/I2FLID7C1CDF/Foundation-36W7A6K9XC7LQ.pcm': 'No such file or directory'

  cargo:warning=error: unable to open output file '/homeless-shelter/.cache/clang/ModuleCache/I2FLID7C1CDF/ObjectiveC-1MCLJG09OQRJB.pcm': 'No such file or directory'

  cargo:warning=5 errors generated.

  exit status: 1

  --- stderr

  error occurred: Command env -u IPHONEOS_DEPLOYMENT_TARGET "/nix/store/vwh2qqal1q101wv4jy40giavp73q0msd-clang-wrapper-16.0.6/bin/cc" "-O3" "-ffunction-sections" "-fdata-sections" "-fPIC" "-arch" "arm64" "-Wall" "-Wextra" "-fmodules" "-Wno-deprecated-declarations" "-mmacos-version-min=11.0" "-o" "/private/tmp/nix-build-starship-1.17.1.drv-0/source/target/aarch64-apple-darwin/release/build/mac-notification-sys-723f11dcedbc426f/out/objc/notify.o" "-c
" "objc/notify.m" with args "cc" did not execute successfully (status code exit status: 1).

@utkarshgupta137
I fixed it in https://github.com/NixOS/nixpkgs/pull/279419. Could you please try it?


Investigate deployment failures

Although the package has been deployed, the job that did manage the deploy failed strangely:
https://travis-ci.org/pytest-dev/pytest-forked/builds/493316392?utm_source=github_status&utm_medium=notification
We probably should update the configuration to use stages as well.

@nicoddemus We (my company) have noticed a failure in being able to download this project via an Artifactory virtual project since your latest release (1.0.2) today.  We have had to pin our dependencies back to 1.0.1.  Error is that it throws an "Unknown content length".

Hi @rmcdiffett,
Hmm strange, I just installed the 1.0.2 package into a fresh virtual environment and it worked perfectly. Also I manually downloaded the package and checked that it appears to be fine. Can you verify your artifactory repo? Perhaps manually downloading the file and ensuring it can be extracted?

@nicoddemus maybe you need skip-cleanup in travis config? https://travis-ci.org/pytest-dev/pytest-forked/jobs/493316393#L607

Hi @rmcdiffett,
Hmm strange, I just installed the 1.0.2 package into a fresh virtual environment and it worked perfectly. Also I manually downloaded the package and checked that it appears to be fine. Can you verify your artifactory repo? Perhaps manually downloading the file and ensuring it can be extracted?

This issue self-corrected with no intervention on our side.  Thank you!

We probably should update the configuration to use stages as well.

Implemented via https://github.com/pytest-dev/pytest-forked/pull/40.
Looks like no further action is needed. Also, @rmcdiffett's problem looks more or less like a caching issue (PyPI cache usually gets updated during 10 minutes after publishing a release).


Not working with new html-webpack-plugin@version 4.0.1 (near)

"webpack": "^4.42",
"html-webpack-plugin": "^4.0.1",

@DemonCloud @Nemo157 try npm install html-webpack-inline-source-plugin@beta

@DemonCloud, @Nemo157 you need to install with npm install html-webpack-inline-source-plugin@beta and in webpack config pass webpack-html-plugin instance: new HtmlInlineSourcePlugin(HtmlPlugin). @DustinJackson would be great to switch from beta release to official now that the webpack-html-plugin is at 4.x

@fmal  is also not working for me, the beta this 1.0.0.beta2 version,  the current version is 1.1.0

Facebook provides a very nice inline source plugin:
https://github.com/facebook/create-react-app/blob/edc671eeea6b7d26ac3f1eb2050e50f75cf9ad5d/packages/react-dev-utils/InlineChunkHtmlPlugin.js#L10
Maybe we can move the idea into this plugin so there is a stand alone npm package?

@jantimon get this!

I'll take a look at update this weekend, PRs are welcome

Really looking forward to these fixes/additions. Thanks for all your hard work on this plugin @DustinJackson

I agree with @jantimon, since facebook already has a similar plugin we can move the idea there.  I will deprecate the repository with a link.

@DustinJackson please don't deprecate this plugin, as having the similar in create-react-app does not solve the issue for people who does not use that library. We need a standalone npm package.

Agreed. I don’t use react either and could really use this plugin in my
project.
On Wed, Apr 8, 2020 at 3:20 AM Aziz Gazanchiyan<EMAIL_ADDRESS>wrote:

@DustinJackson https://github.com/DustinJackson please don't deprecate
this plugin, as having the similar in create-react-app does not solve the
issue for people who does not use that library. We need a standalone npm
package.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DustinJackson/html-webpack-inline-source-plugin/issues/75#issuecomment-610794935,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADHN4GR4VTOS74RN6GPNLI3RLQQUJANCNFSM4LSLTISQ
.

--
Evan

In order someone is still looking for working package - there is style-ext-html-webpack-plugin which works fine with latest html-webpack-plugin

@creage Could you show a working example of your config that allows this to work? Appreciate it.

In order someone is still looking for working package - there is style-ext-html-webpack-plugin which works fine with latest html-webpack-plugin

@creage  your package seems to export css style only.
@evandiamond Did you find a standalone plugin ?

@yairopro For JS we use script-ext-html-webpack-plugin, but it can't handle multiple HTML entries.

Thanks a lot @creage !
I will have a look on it.

@evandiamond I've found the npm package that includes the InlineSourceHtmlPlugin by facebook.
const InlineChunkHtmlPlugin = require("react-dev-utils/InlineChunkHtmlPlugin");

@creage THanks again. I finally adopted this plugin 😁.
For people who pass by this issue, here's my config:
const HtmlWebpackPlugin = require("html-webpack-plugin");
const ScriptExtHtmlWebpackPlugin = require("script-ext-html-webpack-plugin");
const path = require("path");

module.exports = function config() {
	return {
		entry: path.resolve(__dirname, 'src/index.js'),

		output: {
			// Do not place this property
			// otherwise your output index.html will overwritten
			// filename: "..."

			path: path.resolve(__dirname, 'build'),
		},

		plugins: [
			new HtmlWebpackPlugin(),
			new ScriptExtHtmlWebpackPlugin({
				inline: /\.js$/,
			}),
		],
	};
};


dep: cleanup unneeded azure sdk

LGTM

@etcd-bot retest this please
ref: https://github.com/coreos/etcd-operator/issues/1693
Failure: https://jenkins-etcd-public.prod.coreos.systems/job/etcd-operator/2932/consoleText


Stable release v0.18.1

Fix Prune of EnvoyConfigs #248 https://github.com/3scale-ops/saas-operator/pull/248
Dependabot updating go lang dependancies (golang.org/x/sys, golang.org/x/term, golang.org/x/text) https://github.com/3scale-ops/saas-operator/pull/250
Fix/Remove yabeda metrics scrapping from system-app https://github.com/3scale-ops/saas-operator/pull/251

/kind release
/priority important-soon
/assign

/approve


Unknown funtion rooted(...) called in Multibody Library

Reported by kurzbach on 7 Nov 2008 09:41 UTC
In the Multibody library in Joints.mo and Parts.mo the unknown/undocumented function rooted(..) is called.
What is the semantics?
Shouldn't be this function a part of the Connections package?

Migrated-From: https://trac.modelica.org/Modelica/ticket/116

Comment by dietmarw on 17 Nov 2008 11:01 UTC
This is a duplicate of #95.

Comment by dietmarw on 25 May 2009 09:17 UTC
Moved to milestone:never (which should collect all the invalid, duplicates or wontfix tickets).

Modified by kurzbach on 20 Dec 2016 13:31 UTC

Changelog removed by kurzbach on 20 Dec 2016 13:31 UTC


add abapGit as dependency, so it knows the types

Thank you!

you can also change the language version at https://github.com/bigld/abapGit-Repository-Notification/blob/master/abaplint.json#L21
so the NEW expression can be used if you are not targeting 702

possible values,


Fix Transport Level Stream Close in Code

Currently, in the BJSON class, the stream is closed in its "serializeData" method, other than also doing a "flush()". This should only be a flush, but due to the transport not closing the connection from its side, the response will not be reaching the client in the streaming case. This should be properly fixed by closing the stream from the transport code and removing the close from places like BJSON#serializeData.

Fix with the below PR
https://github.com/ballerinalang/ballerina/pull/4298


Steel furnace placement off-by-one

0.16.51 36654 win64
Blueprints with steel furnaces, which went through the editor, might have their placement positions off by one. I observed  y+1 (upwards) and x-1, y-1 (down,left). Editor shows it fine, placement preview also looks fine, but actual placement is off.

Can you send me the bp string?

I think I fixed it. Can you confirm?

Looks good. Awesome editor.
Thx

Thanks for reporting the bug!


Don't clear traffic graph when changing interval

This change stops the graph from being reset when the duration is changed. If the user wants it to be cleared, then they can click the reset button that is already present.

Strong Concept ACK
It felt kind of annoying when you had network traffic data for the last 30 mins, and you, by mistake or intentionally moved the slider, and all data was lost.
I didn't think it was such a simple fix. Thanks for catching it @rebroad!
Let me do the testing and post the tested review soon!

Postponed until approach NAK is tackled
or fighted.

@shaavan PR #492 hopefully addresses your concerns raised here.

The (x-axis) scale of the graph doesn’t update accordingly with the change in timer. For example,

I would argue that the user would have to be extraordinarily unintelligent to have an expectation of the X-axis being linear, if it is the same user that changed the duration and could see (at the time) that the pre-existing data was not resampled.
The only scenario I can see confusion arising is it someone viewed the graph without being aware that the scale has been changed mid-way through the data. I think there ought to be a limit to the extent we (developers) take responsibility for all the ways people can misinterpret data that has been manipulated by others.
The assumption here is there is only one user - the same user viewing the data as the user configuring the sample rate of that data.
And even if a 3rd party was involved - there's really very little "damage" that could occur from any false assumptions.

When I set the timer to 45 min the graph looks like following:

But when I changed the scale to far-right, i.e., 1d. The graph remained the same:

The change done in this PR allows displaying the graph in the future according to the latest timer setting. However, since the time-scaling of the graph that is already displayed is not changing accordingly, it could lead to wrong conclusions on the user’s side.
The optimal behavior would be to expand or contract the displayed graph according to the latest time-scale setting.

Approach NACK, instead you should do more sampling and caching. Then display the appropriate data as the slider moves instead of clearing and drawing.

Personally, I would not want this functionality (I did consider it). I much prefer maximizing the use of the data already sampled (which resampling would not achieve).

PR #492 hopefully addresses your concerns raised here.

#492 would be an improvement over the current implementation and can be helpful in many cases other than this PR.
However, it still does not fix the issue I described in its entirety.
Usually, we don't take the cursor to all the positions on the graph and read the specific time of that instance with focus. We typically glance over the chart as a whole to get a gist of the network traffic. So there is a risk of possible misinterpretation, even for an individual user.
Though @reboard has addressed this in this comment, I still think this is a bad UX design. And personally, for me, the possible declination in the user's experience outweighs the functionality this PR is introducing.

How about replacing the slider with something like
Then it just needs some vertical lines with timestamps on the x-axis.
This suggestion is also an alternative to #483.

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts
Reviewers, this pull request conflicts with the following ones:

#524 (Replace int with std::chrono in for the timer->setInterval() argument by shaavan)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

REF: https://github.com/bitcoin-core/gui/issues/532

Concept ACK

@rebroad - Please rebase - I prefer to test PR on a more recent commit. (Even though it is a trivial change) -thanks
NOTE: current /gui master passes CI 9ec3991ad3b2ae91997c696a4c2f187fe538eff0

Dynamic scaling based on QSlider value -> https://github.com/RandyMcMillan/gui/tree/1643137693-retain-network-graph-on-interval-change

@rebroad - I recommend renaming this PR to
gui: Network Traffic Graph - dynamic scaling view

or something similar.
https://user-images.githubusercontent.com/152159/151081341-78ef4334-21e4-4919-9265-4502d1e338ee.mov

@rebroad - I submitted a PR to your repo or you can fetch my commit and cherry-pick it on top of yours.
Thanks - https://github.com/RandyMcMillan/gui/commit/00c016f1a0a74ff5bedf28f10dc9ea3071f6b648

The scaling is working...
https://user-images.githubusercontent.com/152159/151122794-dd3d44c3-0687-4739-9016-6b80b67615e8.mp4

@rebroad - I based this PR on the commit in this PR...
https://github.com/bitcoin-core/gui/pull/539

🐙 This pull request conflicts with the target branch and needs rebase.
Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft".

Closing due to a long period of inactivity here. Feel free to reopen.


how to add pictures related to albums

First off, this is amazing groundwork you have put together. Thanks!
My question is this:
How would you (i.e. what is the recommended way to) implement relational data to an entity?
E.g how would you add pictures as an entity related to an album?

I think that you may use two ways:

store album pictures as static data, and store picture url in album table.
store pictures as blobs.

My question was for relational data in general so pictures was only an example, a better example of what I'm asking would be categories with products. I think both of the approaches you suggested don't play well with other than static data.

So this is how I'm fetching products related to a category:
add a new route and function for products related to a category in internal/category/api.go:
`r.Get("/category//products", res.products)
func (r resource) products(c *routing.Context) error {
ctx := c.Request.Context()
count, err := r.service.CountProducts(ctx, c.Param("id"))
if err != nil {
	return err
}

pages := pagination.NewFromRequest(c.Request, count)
products, err := r.service.GetProducts(ctx, c.Param("id"), pages.Offset(), pages.Limit())
if err != nil {
	return err
}

pages.Items = products
return c.Write(pages)

}
**then add the two functions for count and GetProducts in internal/category/service.go:**// CountProducts of the specified category
func (s service) CountProducts(ctx context.Context, catid string) (int, error) {
return s.repo.CountProducts(ctx, catid)
}
// GetProducts of the specified category.
func (s service) GetProducts(ctx context.Context, catid string, offset, limit int) ([]Product, error) {
items, err := s.repo.GetProducts(ctx, catid, offset, limit)
if err != nil {
return nil, err
}
result := []Product{}
for _, item := range items {
result = append(result, Product{item})
}
return result, nil

}`
similarly in internal/category/repository.go:
`// CountProducts of the specified category.
func (r repository) CountProducts(ctx context.Context, catid string) (int, error) {
var count int
err := r.db.With(ctx).
Select("COUNT(*)").
From("product").
Where(dbx.NewExp("category_id={:catid}", dbx.Params{"catid": catid})).
Row(&count)
return count, err
}
// GetProducts of the specified category with the specified offset and limit.
func (r repository) GetProducts(ctx context.Context, catid string, offset, limit int) ([]entity.Product, error) {
var products []entity.Product
err := r.db.With(ctx).
Select().
From("product").
Where(dbx.NewExp("category_id={:catid}", dbx.Params{"catid": catid})).
OrderBy("updated_at DESC").
Offset(int64(offset)).
Limit(int64(limit)).
All(&products)
return products, err
}`
Of course there are some more steps involved, if anyone is interested I'd be happy to show them here

I answered you exactly the question that you voiced. If you meant something else, then you had to ask. There are no psychics here.


URLEncode and Decode sessions

We could, hypothetically, compress a MicrobeTrace session and then affix that string in a GET parameter of a URL. That way, people could (possibly) "share" sessions.

Depends on #19


Array in NBT referenced by getBlockData () in Block Reader starts at 0

Descripe
(It may be difficult to understand because it is machine translation.)
I referenced NBT in Block Reader, and the array it contains starts at 0.
lua is usually set to use arrays starting from 1, and 0 is ignored by table, ipairs, etc.
Steps to reproduce

Place Botania's Gourmarylis in front of Block Reader.
to feed Gourmarylis.
peripheral.find ("blockReader").getBlockData().lastFoods array starts at 0
(#peripheral.find("blockReader").getBlockData().lastFoods will be 0)

Multiplayer?
No
Version
1.16.5-<IP_ADDRESS>r (Latest 1.16)
Minecraft, Forge and maybe other related mods versions
Forge 36.2.28 Minecraft 1.16.5 Botania 1.16.5-420.2
Screenshots or Videos
No response
Crashlog/log
No response

I can reproduce this partly.
But I can't fix that myself. It seems that cc does this when they parse a map to a table. You could try to start a discussion or a create a issue at https://github.com/cc-tweaked/CC-Tweaked


Add Canvas transform

[x] Canvas position represents a point on the infinite canvas
[x] Canvas transform represents the location of the infinite canvas relative to the viewport (i.e. the svg).
[x] All tools convert the viewport positions to canvas positions and produce an operation with the positions in canvas space.
[x] Operations produce an svg element with the correct location scale and rotation
[x] Svg is rendered with the correct transform from the Rust code

This change is

Associated issue: https://github.com/GraphiteEditor/Graphite/issues/100

Some basic (temporary) keybindings to move/rotate the canvas would be nice to see that it works, as opposed to having a static transformation.


Fix GDAX fee for ETH and LTC

Fees for ETH and LTC should be 0.3% instead of 0.25% according to https://www.gdax.com/fees

Should probably merge #927 first

awesome, thanks! They are both merged now.


feat: Add package management and  improve perception module

add package management
add perception pipeline
improve perception config file

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


replace yup with ajv, for validation of catalog entities

Fixes #843
The tests were pretty thorough and did not have to be updated. I think this should be a pretty safe change.

@Fox32 I really wanted to compile the schemas... but the compiled code has stuff like require('ajv/dist/compile/ucs2length').default in it. And to make that work, the actual running package needs to be able to resolve it. So the ajv dep needs to be on the backend package (because there are other versions as well and it depends on hoisting i guess?) which obviously is not at all nice, because it doesn't care about that itself.
If you have any idea how I can get around that, I'll be glad :)

@Fox32 I really wanted to compile the schemas... but the compiled code has stuff like require('ajv/dist/compile/ucs2length').default in it. And to make that work, the actual running package needs to be able to resolve it. So the ajv dep needs to be on the backend package (because there are other versions as well and it depends on hoisting i guess?) which obviously is not at all nice, because it doesn't care about that itself.
If you have any idea how I can get around that, I'll be glad :)

I have no experience with the ajv code generation, I don't have a good idea.
Maybe, but I think @Rugvip is the expert here, we can compile the schema after generation with webpack, so that the code really is standalone and independent from other ajv dependencies?

I have no experience with the ajv code generation, I don't have a good idea.
Maybe, but I think @Rugvip is the expert here, we can compile the schema after generation with webpack, so that the code really is standalone and independent from other ajv dependencies?

Yeah maybe that's actually the way it was intended to be used, basically - to have a separate packaging step after build.

Yeah maybe that's actually the way it was intended to be used, basically - to have a separate packaging step after build.

OK I moved the schemas into the src tree and stopped precompiling. Just didn't feel worth it. The code lazily compiles the schemas in memory, and if this is called from frontend code they will likely be bitten by the CSP limits we've seen elsewhere.
This should be good to go, as far as I know.

OK I moved the schemas into the src tree and stopped precompiling. Just didn't feel worth it. The code lazily compiles the schemas in memory, and if this is called from frontend code they will likely be bitten by the CSP limits we've seen elsewhere.
This should be good to go, as far as I know.


Generic alias for BusyBox

Currently w64devkit has 148 tiny alias files for the BusyBox applets. These are all the same apart from the hardcoded name of the applet.
If the applet name were obtained from the command line you'd only need to build one binary and make 148 copies of it with different names.
This special case might be specified by giving an empty command name at build time (gcc -DEXE=busybox.exe -DCMD= ...).
Then, with details left as an exercise for the reader:
    if (COUNTOF(LCMD) != 1) {
        append(&cmd, LCMD, COUNTOF(LCMD)-1);
    } else {
        char16_t *appname = ...
        int applen = ....
        append(&cmd, appname, applen);
    }

Since COUNTOF(LCMD) is known at build time only one branch of the if should be compiled.

Ha, it's so obvious! Why didn't I think of this before, especially given
the context? Thanks for pointing this out, Ron.
Though the benefits are smaller than they might seem. It only takes some
tens of milliseconds to compile alias.c, in large part because I don't
include windows.h. Compiling and linking all 148 binaries (in parallel)
takes less than a second on my desktop. Building one binary and copying it
147 times would take that down to zero, shaving about a second off the
build. I can't deduplicate using .zip — otherwise I'd have already done it
with the original busybox.exe — so ultimately the user will still have 148
separate copies with merely the opportunity to deduplicate. (Though it
looks like NTFS does have this capability.)
Still, seems worth doing! It's actually even simpler than you said: I can
pass GetCommandLineW() straight to busybox.exe without looking at it nor
allocating a new command line buffer.

pass GetCommandLineW() straight to busybox.exe

Of course, even better!

I suppose this is implemented now, so can be closed.

Yup, done in 35d858e43b27eb7919546dfa80cfd16a116042be.

Unless 7z itself has some "deduplication"

Unlike zip, 7z uses solid compression which is ideal for this case.


Horizontal rule display is not getting reset.

When you click the confirm button, the horizontal rule above the button gets a style applied to it display: none; if you activate another instance of swal the horizontal rule is still set to display none.

Can you please reproduce this issue with https://jsfiddle.net/ so I can look into it, thanks

This bug is resolved in latest release. Thanks @CSWilson for the report!


fix(workaround): Store-dev-tools dependency replaced to the PR to avo…

…id the ng4 bug

Coverage decreased (-7.4%) to 52.464% when pulling 6bd4c8de880aae3889c36d3d35b1941e51e31c4b on sayarco:export into 36726beaeff45402126eb0126b3d992e5c99d101 on kemalcany:master.


cpu.sh: your "total" includes guest (and guest_nice) twice

Your sum to create the total cpu usage includes both "user" and "guest", but "user" already includes guest, as per:
https://unix.stackexchange.com/questions/178045
https://github.com/netdata/netdata/issues/729
https://lkml.org/lkml/2008/6/23/65
https://www.opsdash.com/blog/cpu-usage-linux.html
The same "logic" also applies to "nice" and "guest_nice".
HTH, Jaime

Thanks @jaimet!! I've definitely learned something new here! :smile:  Thanks for taking the time to report it.
I'll update the script to allow for the overlapped fields soon.
Thanks again!


[Scenario 5] There are pending changes to process when beginReplicating() is called during retry

This is kind of re-open #1135 as there is still an edge case that can cause the same issue.
In #1135, we attempted to fix the retry issue by calling stopObserving() before actually doing the retry so we could expect that there will be no pending changes to process while the beginReplicating() is called again. However there is still a possibility that right when calling the stopObserving() from the work executor thread, there could be some new changes coming in from either UI or router threads so we could ended it up with the same issue as #1135.

This issue is not valid after https://github.com/couchbase/couchbase-lite-java-core/pull/1150/commits/cfa01ce102ef22dfd87d1cb6e1254e7db2755bdd fix.


[ll] app: Adopt GL to the application refactor [10/..]

Current focus lays on refactoring the app to minimize the backend specific code to window setup.

Ok, makes sense.
Please go ahead and merge then.

On Jun 11, 2017, at 17:10, msiglreith<EMAIL_ADDRESS>wrote:
@msiglreith commented on this pull request.
In src/backend/gl/src/factory.rs:

@@ -88,6 +88,11 @@ impl Factory {
}
}

/// Get the OpenGL-specific driver information
pub fn get_info<'a>(&'a self) -> &'a Info {
That could also be done, indeed. But I still would stick with the current solution because Share (containing Info) is wrapped in a Rc and we need Info anyways for the CommandQueue later, either stored directly inside the struct or also in Share if we want to expose multiple queues for the GL backend.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Change Detection:  PreProcessing- Set up template and explore your data

Step 1 gives the following error.

Sounds like this (and all the issues from #703 - #718 ) might be the result of just the initial/sanity/configuration test - we might want to do a first sanity pass using an alternate method - like a text/word doc or combine into a single issue.
But a tip for everyone doing templates - if possible, try to sanity test your Pro template project deployment on a second machine before handing off to testing. We have seen a lot of deployment issues (such as hardcoded Pro project settings, other unexpected configurations after unpackaging, etc.) that you don't necessarily catch if you open the .ppkx on the same machine.

Tested on two machines before it was sent for testing and this worked - Step one should just be some text with a 'next step' button so not sure how this can not work, step two the same, step 3 should automatically open the pre-processing map, and step 4 should just open an 'add data' dialogue box. Have changed step 4 to a manual 'add data' step based on testing.
As a wider point it looks like your task file may be pointing back to an older version. Have also re-referenced the toolbox again this morning, re-tested on three machines and seems to work. But still not convinced I've completely got to the bottom of it.

Error is no longer present.


Made working product of fuzzy testing checkboxes

Made it so that three checkboxes show up after the fuzzy tester is clicked. It is implemented so that there always needs to be one enabled.

Hi @yashaswini-slu, i believe the current implementation works best. The way it is implemented has it so you cant have them all checked off, since having them checked off means you don't want anything fuzzy tested, which would get rid of the point of enabling the fuzzy test to begin with. I think having the user unclick the desired boxes they don't want fuzzy tested works best.

@yashaswini-slu you approved this PR, but there is a merge conflict with main. Make sure to resolve the conflict before merging.


IE 11 vertical swipe

Under IE 11 option:
$DragOrientation: 3,

do not allow vertical swipe/dragging

I see.


Workaround for unreliable BLE stack on iOS

Issue

On iOS the Bluetooth Stack is known to die randomly without informing any app that uses Bluetooth. All calls to start advertising and/or scanning continue to say they succeed, but no information, or much less information, is passed from iOS to the Bluetooth based app.

[Who]  As an app user
[What] I need assurance that the app is working throughout the day
[Value] So I can stay safe or take corrective action (E.g. restart the phone)
Desired outcome

Add in a healthcheck function to Herald that passes one or more pieces of information to the user to help them stay safe
Should be an API layer within Herald with useful information and documented error codes
Should provide an example UI in the demo app for iOS

Examples

Warning: This phone doesn't support advertising. Don't worry, Herald has support for this phone which keeps you protected 98.5% of the time.
Urgent: Your phone hasn't seen any other devices over Bluetooth. Please restart your phone and start this app again.
Urgent: Your phone has encountered many Bluetooth comms errors over the last hour. Please restart your phone and start this app again.

Acceptance criteria:-

[ ] When the demo app (and underlying API) correctly identifies the iOS BLE stack death issue, and reports it to the user with advice on what to do next

Note: Due to Herald being very reliable, no adopter has asked for this 'catch all' error functionality yet.

I'm very curious about this one. I have an idea about what Herald may be doing to cause this, but need to test a bit more to confirm. Have you observed this in the test iOS app?

To be clear, this isn't a Herald specific problem. You can come across this on any phone, install any BLe app and wonder why it's not working.
We've not observed this behaviour since around August with Herald as we're specifically using the minimum BLe functions at the minimum frequency so not to hit such issues. More active apps on a phone are more likely to hit such things (active data sending, not just scanning apps).

I have used many BLE application including inbuilt functions such as the Apple Watch system, AirDrop, and Continuity, as well as third-party apps such as Pokémon Go, and never in the history of me being an iPhone user had the Bluetooth stack “die randomly”... except when running COVIDSafe, due to a bug in it’s scanning and connection logic.
If you are seeing the same behaviour in Herald then it is most likely a bug in Herald.

@yakkov-h - I think there's some confusion over why I created this particular enhancement ticket. We created it because we observed that a phone we'd only just installed Herald on seemed to have a malfunctioning BLe stack already. This also seems to occasionally occur when you install an app (any app) that uses a background bluetooth service (not just foreground scanning as you observe in your other apps) via a developer connection (i.e. XCode) without uninstalling the app. I suspect that particular instance may be XCode not checking for and shutting down backgrounded activity properly.
If you believe there are separate issues around interference then I'd encourage you to report them as a separate issue and provide a description of the specific issues you are seeing. Please let's not pollute unrelated issues as it makes the teams work a lot harder to keep track of this. If we see a correlation in testing we'll link the issues, but we need to reproduce the issues first.

@adamfowleruk perhaps that context should have been included in the initial description. Your opening statement was "On iOS the Bluetooth Stack is known to die randomly" which is it not generally known to do.
The rest of the description include a user warning such as "This phone doesn't support advertising" doesn't seem to line up either with a rare developer-only failure scenario IMHO.

It looks like commit #87 may inadvertently (judging by the commit message) fix this issue. Well done!

@wabzqem Jolly good! We're always doing small performance tweaks and tests.
@yaakov-h Yeah fair point on the description. It was a quick placeholder for our own team who of course knew what it related to. I'll have to be more specific in future.

Bump: Validate this for v2.1 before future app needs in v2.2.


Auto-completion should not complete already complete words

Problem Description
Auto-completion (without kite, maybe also with, but I didn't test) propose to complete already complete words.
What steps reproduce the problem?
Use LSP auto-completion (not tested with kite).
Write
something = 1
something

and wait. Auto-completion propose something
What is the expected output? What do you see instead?
I would expect auto-completion to disappear when I write entirely what I wanted to.
It's not a major bug, but quite annoying.
Versions

Spyder version: 4.0.1
Python version: 3.7.6
Jedi version: 0.14.1
python-language-server version: 0.31.8

@andfoy, please take a look at this one.

If the prefix matches something on the retuned completion options, do not show it.

I think in the ideal case I would expect something a little more complicated (but faster to run):

if there is only one possible completion and it matches the prefix, I expect the completion to be hidden (or closed)(see example in first comment);
if several completions are possible, I expect in this case to propose completion for the already complete word (as it work now).

The example bellow should make things clearer for the 2.:
some = 1
something = 2
some

In that case I expect completion to propose some and something so that user can know some is a possible choice (showing only something would be confusing).


Add support for ZeroConf discovery

Hello,
I've added the support for discovering the overkiz box by ZeroConf.
The entry is: _kizbox._tcp.local.
There's a properties attribute which contain directly the gateway ref (aka gateway_pin).

Thanks! Could you add a test as well? Since core has the requirement for 100% test coverage config flow, we require that for config flow as well.
If you want, you could also see how you can bring this to core yourselves as well. See https://github.com/iMicknl/ha-tahoma/issues/697. The config flow in core has some slight changes though.


Run automatic node-gyp rebuild

As described by: https://github.com/npm/cli/blob/f9abee79abe541226a249f50bdeec41317dd5712/docs/content/configuring-npm/package-json.md#default-values

Hello @BendingBender - thank you for your contribution and apologies for leaving this hanging. If you are still interested in this, would you be able to reduce this to the changes needed for this feature (removing or splitting out for example the .nvmrc and .gitignore changes)?

OK, I've removed all the unnecessary changes and rebased to current main.

@BendingBender expect it in the next release


Github issues are ready

[x] Open github issues in open design repo ( @umeshagarwal )

Issue is here: https://github.com/mozilla/OpenDesign/issues/111


fix: add a generic cleaning mechanism

Fixes https://github.com/getgrit/gritql/issues/209 by replacing our special-cased "orphan" logic with a generic cleaning approach where we can repair program output.
Alternatives considered
I tried doing this as part of after_each_file but that happens pre-linearization and making additional changes there was too cumbersome: https://github.com/getgrit/stdlib/pull/187

Summary by CodeRabbit

New Features

Introduced a new system for managing code replacements across various programming languages, enhancing the ability to handle and apply changes within source code efficiently.

Refactor

Transitioned from handling "orphaned" code elements to a more robust "replacement" system, affecting multiple language implementations including JavaScript, Python, TypeScript, and TSX.
Updated internal logic and method signatures to support the new replacement system.

Tests

Added tests to ensure the new replacement functionalities work correctly for JavaScript code.

Bug Fixes

Improved handling of specific node replacements in scripts, ensuring more accurate code modifications.

@morgante Does this cover a case like this?
const x = () => (log(), log());

query
`log()` => .

should be
const x = () => {}

I know it's a very weird case, but just got it in mind. Ignore if it's too trivial

@scr2em That input isn't valid JavaScript (so Grit isn't expected to handle it properly), but this works fine: const x = () => {console.log(); console.log()};

See https://github.com/getgrit/gritql/pull/212/files

@morgante I believe it's a valid javascript, can you recheck ?
const x = () => (log(), log());

just noticed that we can just remove the assign operator


When it will be release?

[Enter feedback here]
This is nice function.
When will this be release in the environment as i cant find it yet in the data type of the column

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

ID: d6f7d49a-1fb0-ffba-779d-9c8b6d055d15
Version Independent ID: d6f7d49a-1fb0-ffba-779d-9c8b6d055d15
Content: Work with Dataverse formula columns - Power Apps
Content Source: powerapps-docs/maker/data-platform/formula-columns.md
Service: powerapps
Sub-service: teams
GitHub Login: @matp
Microsoft Alias: dikamath

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

@HammedAbdulazeez - Release information is covered in the release plans for each feature. Formula columns release details are available from the link below. The feature is currently available in preview environments. GA has not been determined.
https://docs.microsoft.com/en-us/power-platform-release-plan/2022wave1/data-platform/enable-power-fx-formula-columns-inside-dataverse-tables
For information about the preview program for model-driven apps and Dataverse, go to https://docs.microsoft.com/en-us/power-apps/maker/powerapps-preview-program#portals-model-driven-apps-and-app-management.
Closing issue.

@HammedAbdulazeez - Release information is covered in the release plans for each feature. Formula columns release details are available from the link below. The feature is currently available in preview environments. GA has not been determined. https://docs.microsoft.com/en-us/power-platform-release-plan/2022wave1/data-platform/enable-power-fx-formula-columns-inside-dataverse-tables
For information about the preview program for model-driven apps and Dataverse, go to https://docs.microsoft.com/en-us/power-apps/maker/powerapps-preview-program#portals-model-driven-apps-and-app-management.
Closing issue.

able to create a preview environment but it doesn't allow database to be created, how can we test this column preview if a database cant be created?


bybit Spot - Failed to connect API to dashboard

I just follow the quick start and created api of my test account but it doesnt work

Hi @Chebaoshan this is related to our hummingbot lib currently not updated to the latest version, our team is still working for  the fix 🙏🏼

Thank you for your response sir.
Appreciate

When will this issue be solved?　@rapcmia

However was able to login from Client (latest development)

Please check the restricted areas as well.
https://www.bybit.com/en/help-center/article/Service-Restricted-Countries

backend-api

Works on other connectors for latest development (dashboard)

The testnet failed but the mainnet succeeded. Thank you.

As expected, I can't connect to the bybit API from the dashboard. When will this be fixed? Is there a way to solve this now? @nikspz

we need to update pypi dependencies (hummingbot) for dashboard. so after it's updated should work most likely @CloverEc

currently working on it, I will notify and close this issue when updated

dependencies

What parts specifically need to be updated? hummingbot/dashboard?

https://github.com/hummingbot/hummingbot/issues/7233 duplicated here

What parts specifically need to be updated? hummingbot/dashboard?https://github.com/hummingbot/deploy？

This dependency need to be updated
https://pypi.org/project/hummingbot/20240713/#history

Library updated and bybit spot should be available on dashboard


(sourcetree) Use MSI installer

Expected Behavior
The Sourcetree package currently uses a Squirrel based installer which, installes into user directory. This doesn't match well with a machine wide packaging tool like Chocolatey. Starting with 2.5.0 a MSI installer is provided.
Current Behavior
Package installes into user directory
Possible Solution
Package should use MSI installer. MSI installer should be publicly available after full rollout.
MSI installer will disable auto-update and and remove embedded Git/Hg.
The packe should provide an upgrade path if possible from 2.4 to 2.5 by uninstalling current Sourcetree version and migrating settings (e.g. repos) to new installation

Official announcement


No warning in case of variable assignment before the first type test

class C {}

main() {
  Object o = C();
  if (o is C) {} // No warning
  o = C();
  if (o is C) {} // unnecessary_type_check warning.
}

Why there is no warning in the first case?
Dart SDK version: 3.7.0-183.0.dev (dev) (Fri Nov 22 20:03:46 2024 -0800) on "windows_x64"

That's OK!
The promotion mechanism relies on 'types of interest', and the initialization of o with a value of type C doesn't make C a type of interest (it only makes a non-nullable type a type of interest in cases like int? iq = 2;). The same thing is true for assignments, they only promote the type of a variable if that type is already of interest for that variable for other reasons.
This means that o is C tests whether an o of type Object also has type C, which is OK (non-redundant).
But in the code that follows the first if statement, C is a type of interest for o, so o = C() promotes o to C, which makes the second type test redundant.


Allow games to be restarted, and spawn new players somewhere in the maze

Fixes #8 and fixes #9.

Now also fixes #24.


nuget 2.8+ (3.4) commandline tool install doesn't take -FileConflictAction tag

nuget install packagename -FileConflictAction Overwrite
Unknown option: '-FileConflictAction'

Tried with nuget 2.8, 3.3 and 3.4 same error

This issue was moved to NuGet/Home#2321


feat: close the adapter's connection on quit

This change introduces a new optional method of HarlequinConnection implementations in adapters, giving them an opportunity to cleanly close the connection to the underlying database when the app exists.
The close() method is called in the action_quit() handler.
Does this PR require a change to Harlequin's docs?

[ ] No.
[x] Yes, and I have opened a PR at https://github.com/tconbeer/harlequin-web/pull/67
[ ] Yes; I haven't opened a PR, but the gist of the change is that the new HarlequinConnection.close() can be documented in the "Creating an Adapter" guide.

Did you add or update tests for this change?

[ ] Yes.
[x] No, I believe tests aren't necessary – this is an optional functionality of adapters.
[ ] No, I need help with testing this change.

Please complete the following checklist:

[x] I have added an entry to CHANGELOG.md, under the [Unreleased] section heading. That entry references the issue closed by this PR.
[x] I acknowledge Harlequin's MIT license. I do not own my contribution.

Thanks. Can I ask what prompted this? I thought most python DB libraries close the connection when the handle leaves scope/is deleted.

@tconbeer Thanks for taking a look! Python does not guarantee that the __del__ finalizer gets called on interpreter exit, and since the Harlequin app keeps a reference to it (self.connection), nor does it use a context manager for the connection object around the lifecycle of the Harlequin app, there is no current possible mechanism for an adapter to perform a graceful shutdown/close of its connection to the database.
Of course, connections are severed when Python exists, but there might situations where adapters want to perform on-shutdown actions (flush, commit, emit telemetry, or gracefully release resources). This new close() method gives them an opportunity to do so, if they want to. See for example the DB-API spec.

ok, I'm on-board. thanks for the extra info, makes sense.


Regular expressions are case insensitive by default.

This lends a problem when I want to validate that my input is, for example, all uppercase.  I've opened a PR that restores the default behaviour for regexes.  All tests pass, and I figured out how to properly patch the version number this time :)

Thanks! Released in 3.1.3.