PUT request gets stuck in endless loop

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
import logging

import uvicorn

from fastapi import FastAPI, Request
from fastapi.responses import JSONResponse
from fastapi.exceptions import RequestValidationError
from pydantic import BaseModel

app = FastAPI()

class Person(BaseModel):
    name: str
    age: int
    gender: str
    meta: dict

@app.put("/stucked_put")
def stuck_put(person: Person):
    print("never get here (1)")

@app.exception_handler(RequestValidationError)
async def validation_error_handler(request: Request, exc: RequestValidationError) -> JSONResponse:
    invalid_payload = await request.body()
    logging.warning(f"god {invalid_payload=}")
    print("never get here (2)")
    return JSONResponse(status_code=422, content={"errors": exc.errors()})

if __name__ == '__main__':
    uvicorn.run(
        "main:app",
        host="<IP_ADDRESS>",
        port=8080,
    )

Description
What we have:

simple fastAPi application
PUT route with payload validation via pydantic
custom RequestValidationError handler

When I try to send an invalid payload, it gets stuck in an endless loop.
request example
PUT http://localhost:8080/stucked_put
Content-Type: application/json

{}

gets stuck - I mean that request never is completed and I can't receive any response.
The request gets stuck on this line invalid_payload = await request.body()
The same behaviour with PATH, POST.
Operating System
Linux
Operating System Details
Ubuntu 20.04.3 LTS
FastAPI Version
0.70.0
Python Version
3.9.5
Additional Context

starlette==0.16.0
pydantic==1.8.2
uvicorn==0.15.0

How do you send the HTTP requests?
Can you please try with curl?
For example:
curl -XPUT  -H "Content-Type: application/json" http://localhost:8080/stucked_put --data '{}

I tried in Pycharm, scratch.http file <link>
curl does not work too
 curl -XPUT 'http://localhost:8080/stucked_put' -H 'Content-Type: application/json' --data '{}'

The request.body stream was already consumed. This is normal.

@Kludex I understant that body is consumed but why body is not returned then.
Python got stuck on a line of code await request.body() and never returns.
I thought that if body was consumed its content is saved in a attribute of request object BUT in debug mode I saw
this line if not hasattr(self, "_body"): returns True. (_body attr. is not present in self).

@Kludex I understant that body is consumed but why body is not returned then.
Python got stuck on a line of code await request.body() and never returns.

As I said, the request.body stream was already consumed. You'll not be able to retrieve the body anymore, and that's why it keeps hanging.
You can get more context here: https://github.com/encode/starlette/issues/495. The exception handler is built using a middleware, jfyk.

@Kludex
Thanks for link.
So this issue is related to the stalette. I guess we finish here?


Can we make controller concurrency configurable?

Summary
We are heavily using applicationset in our company. Most of the time, making a change on applicationset and generation of a new application takes approximately 30 minutes. We saw that the maximum number of concurrent Reconciles which can be run defaults to 1. Can we speed up processing applicationsets by increasing this number? Would it cause any concurrency-related issues for applicationset? If not, would it be possible to make this option configurable?

Closing this one in favor of https://github.com/argoproj/argo-cd/issues/9002 since repo is moved into ArgoCD


[SPARK-38155][SQL] Disallow distinct aggregate in lateral subqueries with unsupported predicates

What changes were proposed in this pull request?
This PR blocks lateral subqueries that contain DISTINCT aggregate and correlated non-equality predicates in CheckAnalysis.
Why are the changes needed?
To avoid incorrect results. DISTINCT will be rewritten into Aggregate during the optimization phase and only correlated equality predicates are supported with Aggregate.
Note, we only need to block this pattern for lateral subqueries because 1) Scalar subqueries are either aggregated or can only contain one row, so they will not encounter this issue 2) IN/EXISTS subqueries can potentially have the same issue but it won't impact the results of these subqueries, for example, SELECT * FROM t1 WHERE c1 IN (SELECT DISTINCT c1 FROM t2 WHERE t1.c1 > t2.c1).
Does this PR introduce any user-facing change?
Yes. Queries with lateral joins that contain DISTINCT aggregate will be blocked:
SELECT * FROM t1 JOIN LATERAL (SELECT DISTINCT c2 FROM t2 WHERE c1 > t1.c1)

AnalysisException: Correlated column is not allowed in predicate

How was this patch tested?
Unit tests.

cc @cloud-fan

thanks, merging to master!


refactor: Master => develop

Merge latest release back into develop

Builds ready [0e0a5fa]builds: chrome, firefox, opera
bundle viz: background, ui, portal-inpage, portal-contentscript, ui-libs, bg-libs, phishing-detect
dep viz: background
all artifacts


[FLINK-26574] Expose config files in helm chart

Main changes:

Move the default config and logging properties to files from the helm templates
Allow users to easily override operator and flink default configs from helm chart values
Allow users to use easily use their own configmaps instead of ones created by helm

This PR should be merged only after https://github.com/apache/flink-kubernetes-operator/pull/51 as the config keys have already been adapted to that PR.

cc @morhidi @mbalassi

Thank @gyfora, I'll play with this today.

We can also consider putting all the default config values in the values.yaml as is:
https://github.com/elastic/helm-charts/blob/d915d8d53721d7a0535b0956509c6a6e8756f3ec/elasticsearch/values.yaml#L31

@morhidi based on the feedback I made the following improvements:
Allow users to append-to or replace all config files (yaml and logging props) from values.
Default behaviour is append (to the default configs defined in the conf directories)


Update ACI-CNI to <IP_ADDRESS>

There is ACI CNI version <IP_ADDRESS> available which is an upgrade to <IP_ADDRESS> we currently have included in KDM for RKE1 provisioned clusters

@LavanyaVaddavalli , the PRs have been merged and RKE v1.4.6-rc4 has the changes in. Could you please confirm it's working as expected for you?

From QA's perspective, we are good on this. Running our RKE1 regression test suite passed as expected and I validated that the ACI-CNI values were updated.

@snasovich, testing is done. we found only one issue, @LavanyaVaddavalli posted PR https://github.com/rancher/kontainer-driver-metadata/pull/1163, this  is fix for the issue.

The changes requested above were merged and new RKE1 RC cut with the fix. No non-ACI related retesting is needed. The issue is ready to be closed.

Following the comment above, QA will go ahead and close this issue as resolved.


🌐 Fix typo in Chinese translation for docs/zh/docs/tutorial/security/first-steps.md

Awesome, good catch, thank you! @yuki1sntSnow :bow:


pkg/trace-collection: Avoid capturing events from wrong container

Inspektor Gadget uses the mnt namespace inode id of the containers to implement the logic that determines if a given event produced in the kernel should be traced or not.  There is a corner case when this logic traces the events from the wrong container:

A tracer is filtering events from the "foo" container
The foo container is removed
A bar container is created
The tracer displays events from the "bar" container

This can happen because:

The bar container can reuse the mnt ns inode id from the foo container
The deletion of mount namespace is asynchronous, we can't block it
Hence there is a window when the mnt ns inode id from the bar container is in the filtering map and hence events coming from it are wrongly captured.

This commit fixes it by keeping a file descriptor of the mnt namespace of the container, guaranteeing that its inode id is not reused until the tracer has updated all eBPF maps.
How to reproduce
The race condition described here is theoretical, it seems it has happened in https://github.com/inspektor-gadget/inspektor-gadget/issues/1001 but I haven't been able to reproduce it. However, we can use a small trick to alter Inspektor Gadget behavior and reproduce it:

Introduce a delay before updating the tracer ebpf maps

diff --git pkg/tracer-collection/tracer-collection.go pkg/tracer-collection/tracer-collection.go
index ffe612cf..a4498648 100644
--- pkg/tracer-collection/tracer-collection.go
+++ pkg/tracer-collection/tracer-collection.go
@@ -17,6 +17,7 @@ package tracercollection
 import (
        "fmt"
        "os"
+       "time"

        "github.com/cilium/ebpf"
        log "github.com/sirupsen/logrus"
@@ -88,6 +89,8 @@ func (tc *TracerCollection) TracerMapsUpdater() containercollection.FuncNotify {
                        }

                case containercollection.EventTypeRemoveContainer:
+                       time.Sleep(2 * time.Second) // simulate the scheduler takes a bit long to run us here
+
                        for _, t := range tc.tracers {
                                if containercollection.ContainerSelectorMatches(&t.containerSelector, event.Container) {
                                        mntnsC := uint64(event.Container.Mntns)

Run this script in a terminal

#! /bin/bash

while true
do
        docker run --rm --name we-broke-it ubuntu readlink /proc/self/ns/mnt &
        docker run --rm --name no-yet ubuntu cat /dev/null &

        for job in `jobs -p`
        do
        wait $job
        done
done

Start trace exec using local-gadget

sudo ./local-gadget trace exec -c no-yet
[sudo] password for mvb:
WARN[0000] Runtime enricher (cri-o): couldn't get current containers
CONTAINER                                         PID        PPID       COMM             RET ARGS
no-yet                                            334284     334238     cat              0   /usr/bin/cat /dev/null
no-yet                                            334546     334518     cat              0   /usr/bin/cat /dev/null
we-broke-it01                                     334811     334788     cat              0   /usr/bin/cat /dev/null
we-broke-it01                                     335085     335047     cat              0   /usr/bin/cat /dev/null
no-yet                                            335376     335328     cat              0   /usr/bin/cat /dev/null
no-yet                                            335642     335613     cat              0   /usr/bin/cat /dev/null
no-yet                                            335900     335872     readlink         0   /usr/bin/readlink /proc/self/ns/mnt
no-yet                                            335907     335852     cat              0   /usr/bin/cat /dev/null

It can see that the tracer starts to report events from the wrong container.
Thanks to @alban for the idea to fix it!

I updated the PR to include a mutex given that TracerMapsUpdater can be called in parallel.

Finally I was able to implement a test that simulates the issue this PR is fixing. Feel free to revert the pkg/trace-collection: Avoid capturing events from wrong container and see how the test fails without it.


ActionView::Template::Error in admin/communication/websites/posts#preview

Error in Osuny
ActionView::Template::Error in admin/communication/websites/posts#preview
undefined method `university_person_localization_path' for an instance of #Class:0x00007f67efccba60
View on Bugsnag
Stacktrace
app/views/admin/communication/blocks/templates/persons/_show.html.erb:25 - block in _app_views_admin_communication_blocks_templates_persons__show_html_erb__3128561535916664157_1130200
app/views/admin/communication/blocks/templates/persons/_show.html.erb:9 - each
app/views/admin/communication/blocks/templates/persons/_show.html.erb:9 - _app_views_admin_communication_blocks_templates_persons__show_html_erb__3128561535916664157_1130200
app/views/admin/communication/contents/_show.html.erb:3 - block in _app_views_admin_communication_contents__show_html_erb___2294985410036360453_643940
app/views/admin/communication/contents/_show.html.erb:2 - _app_views_admin_communication_contents__show_html_erb___2294985410036360453_643940
app/views/admin/communication/websites/posts/preview.html.erb:6 - _app_views_admin_communication_websites_posts_preview_html_erb__1796539748101316861_1130160
app/controllers/admin/communication/websites/posts_controller.rb:43 - preview
app/controllers/application_controller/with_locale.rb:19 - switch_locale

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
ActionView::Template::Error in admin/communication/websites/posts#preview

An error linked to this issue has been reopened in Bugsnag
ActionView::Template::Error in admin/communication/websites/posts#preview


Eon crash

Describe the bug
Eon crashes whenever you cursed music on
How to reproduce

Unlock fun settings
Turn on cursed music
load eon
crash game
3 alt. Goto soundtest
4 alt. Play eon song
5 alt. Crash game
Screenshots e/o videos

Emulator(please complete the following information):

Device: NES Classic Mini
Emulator RetroArch
Core: NES / Famicom (FCEUmm)

Yeah this was due to a poor export; it has been fixed in the nightly!
https://nightly.link/tfdsoft/famidash/workflows/main/main/famidash.nes.zip


[WIP] Realworld update

this is currently broken, am working on getting it finished (or at least 'acceptable')

No doubt there's more that can be done here but I think it makes sense to merge this in its current state


Refactor datasets module: enhancements in terms of UX & async

This PR refactors datasets modules without changing it's public API — serenata_toolbox.datasets.fetch(…) and serenata_toolbox.datasets.fetch_latest_backup(…) still works as it used to work before.
There are two improvements: use of async methods to download multiple files in parallel and a better UX (a progress bar with the total size of the download and the progress).

from serenata_toolbox import datasets

Hey @cuducos great refactor! Thanks a lot for the version bump ;)
So I fired up my ipython and ran the following
from serenata_toolbox import datasets
datasets.fetch_latest_backup('/Users/temporal/toolbox-data/')

Which worked perfectly, but I tried to run in the same ipython session datasets.fetch('2017-03-15-reimbursements.xz', '/Users/temporal/2-toolbox-data/') and got the following error:
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
<ipython-input-9-5eabdb6d99be> in <module>()
----> 1 datasets.fetch('2017-03-15-reimbursements.xz', '/Users/temporal/2-toolbox-data/')

/Users/temporal/src/dsbr/serenata/serenata-toolbox/serenata_toolbox/datasets.py in fetch(filename, destination_path, aws_bucket, aws_region)
    113 def fetch(filename, destination_path, aws_bucket=None, aws_region=None):
    114     downloader = Downloader(destination_path, aws_bucket, aws_region)
--> 115     return downloader.download([filename])
    116
    117

/Users/temporal/src/dsbr/serenata/serenata-toolbox/serenata_toolbox/datasets.py in download(self, files)
     58
     59         loop = asyncio.get_event_loop()
---> 60         loop.run_until_complete(self.main(loop, files))
     61         loop.close()
     62

/Users/temporal/anaconda3/envs/toolbox/lib/python3.6/asyncio/base_events.py in run_until_complete(self, future)
    441         Return the Future's result, or raise its exception.
    442         """
--> 443         self._check_closed()
    444
    445         new_task = not futures.isfuture(future)

/Users/temporal/anaconda3/envs/toolbox/lib/python3.6/asyncio/base_events.py in _check_closed(self)
    355     def _check_closed(self):
    356         if self._closed:
--> 357             raise RuntimeError('Event loop is closed')
    358
    359     def _asyncgen_finalizer_hook(self, agen):

RuntimeError: Event loop is closed

I highly doubt that someone will run both fetch_latest_backup() and fetch() in the same session, but it would be nice not to have this outcome, do you think we can do something about this? maybe add another event loop to separate both methods?

Makes sense. The event loop was closed — fix in 2322c08


BlockRay Issue

Hello Sponge Devs,
I believe there is an issue with BlockRay.  Here is a video where I showcase my code and some weird behavior I am experiencing.  First of all, I cannot get the BlockRay to target anything but air.  Second, the block collision is not detected when standing really close to a block.  Here is a video showcasing my code:
https://www.youtube.com/watch?v=DZs-uU7T_t4&feature=youtu.be
Sorry for the lack of audio in the video.

It would really help to see your code.

Thanks Ryan,
I will post my code when I get to my PC.  I know that it should return air, but the fact that it doesn't when I am close to another block is a little weird.  I think it is odd behavior, and I think BlockRay has an issue.
Levi

I should also mention that I was in the process of debugging, and that code was implemented to show some weird behavior that I was experiencing.

Here is a copy/paste of my code:
public class WitExecutor implements CommandExecutor {
    public CommandResult execute(CommandSource src, CommandContext args) throws CommandException {
        if (src instanceof Player) {
            Player player = (Player) src;
            Optional block = BlockRay.from(player).filter(hit -> { //Java 8 Lamda
                Extent extent = hit.getExtent(); //Get the extent from the ray hit
                Vector3i pos = hit.getBlockPosition(); //Get the block pos of the ray hit
                BlockType bType = extent.getBlockType(pos); //Get the type of block at the position
                return bType.equals(BlockTypes.AIR); //If it's air, exclude, otherwise its cool
            }).blockLimit(100).build().end();
            if (block.isPresent()) {
                BlockRayHit defBlock = (BlockRayHit) block.get();
                BlockType type = player.getWorld().getLocation(defBlock.getPosition()).getBlockType();
                src.sendMessage(Text.of("You are looking at a " + type.getName()));
            } else {
                src.sendMessage(Text.of("You are looking at the sky."));
            }
        } else {
            src.sendMessage(Text.of("Must be a player to use this command."));
        }
        return CommandResult.success();
    }
}

return bType.equals(BlockTypes.AIR); //If it's air, exclude, otherwise its cool
should be
return !bType.equals(BlockTypes.AIR); //If it's air, exclude, otherwise its cool
as for the other issue I'm not 100% sure what's happening.

All of this behaviour is correct. The block ray only returns blocks that pass the filter test. If you only allow air, the last block returned will be the last valid air. The block ray also only returns blocks it entered. If you start in a block, it won't be returned because it was already entered before the block ray started. That's why you don't anything when using the only-air filter and standing right next to a wall.
What I need to add is an optional filter wrapper to extend an existing filter by a number of blocks after the last valid hit. This way you can create a filter continueAfterFilter(onlyAirFilter(), 1), which will trace one more block after the only-air filter ends (that would return the first wall block).

@DDoS Wouldn't it be a good idea to keep this issue open or open another issue for that so that can be added for the next API release?

I'm doing that today, so I'll remember.

For reference, this was fixed in https://github.com/SpongePowered/SpongeAPI/commit/4818c96c069e843b9097732c6ccb75f076e52381

Thank you!


click event on label?

Hi,
in my leaflet map, I add labels to geojson features. The labels get the clickable option set to true, the label is seemingly clickable (the mouse changes to the pointer type). But nothing happens onclick. there is an onclick handler defined for the actual marker, but how to set an onclick handler for the label?
Thanks, Michael

Please provide an example of the bug.

It's not a bug, it's maybe just lacking documentation on how to bind an action to the click event. I also came here looking for that.

You need to specifically bind an event to marker.label, e.g.:
L.DomEvent.addListener(marker.label, 'click', function(e) { this.togglePopup() }, marker);

I had to dig in to the code a bit to figure this out, worth a mention in the README on how to access the label and bind events to it.

A Star from me!!

Are there no plans for adding on/off listeners to the label? So that we can use:
marker.getLabel().on('click', function(e) { this.togglePopup() });

Hi  a1an77
I found that your label doesn't see appear you map
Is there any way to show in your label?
Many many Thanks!
Regards,
Kenneth

Hi Kennet what do you mean? If the jsfiddle does not work for you be sure to check if your browser is blocking some content (open the console to check for errors and warnings)

Hi a1an77,
Yes, you are right. I found that our company block something and cannot load leaflet label src.js  so that I cannot see drag me
but I can run this sample in my home computer.
Thanks a lot


Correct API declaration of GetResult to be nullable

As per https://github.com/finos/FDC3/issues/1107, backport fix of IntentResult to allow null.  While this is a breaking interface change to API implementors it will be released under the new Finos.Fdc3 beta nuget moving forward.

THIS SOFTWARE IS CONTRIBUTED SUBJECT TO THE TERMS OF THE FINOS CORPORATE CONTRIBUTOR LICENSE AGREEMENT.
  THIS SOFTWARE MAY BE REDISTRIBUTED TO OTHERS ONLY BY EFFECTIVELY USING THIS OR ANOTHER EQUIVALENT DISCLAIMER IN ADDITION TO ANY OTHER REQUIRED LICENSE TERMS.

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

Comparison is base (c1ec5e3) 76.56% compared to head (0707923) 76.56%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #110   +/-   ##
=======================================
  Coverage   76.56%   76.56%
=======================================
  Files          53       53
  Lines         431      431
  Branches       44       44
=======================================
  Hits          330      330
  Misses         82       82
  Partials       19       19

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

/easycla

@kriswest As discussed in https://github.com/finos/FDC3/pull/1127#issuecomment-1845539631, the backport of the nullable IntentResult.  This will go out in the first 2.0 release under the new NuGet namespace.


Login user with Email address instead of Username

In my django app I am using email field for user authentication instead of username field. I am using linkedin, facebook and google authentication in this application. when i register or login using facebook or linkedin a oauth token is created and stored in database but user doesn't get login. Does python social-auth allows to get login using email instead of username ? or i have to use pipline to get oauth token and register the user somehow ?

add the following pipline
'social.pipeline.social_auth.associate_by_email',
just above the pipeline 'social.pipeline.user.create_user',
The overall pipeline should look like this.

SOCIAL_AUTH_PIPELINE = (
    'social.pipeline.social_auth.social_details',
    'social.pipeline.social_auth.social_uid',
    'social.pipeline.social_auth.auth_allowed',
    'social.pipeline.social_auth.social_user',
    'social.pipeline.user.get_username',
    'social.pipeline.social_auth.associate_by_email',  # <--- enable this one
    'social.pipeline.user.create_user',
    'social.pipeline.social_auth.associate_user',
    'social.pipeline.social_auth.load_extra_data',
    'social.pipeline.user.user_details'
)

Then for multiple social backend only one user account will be created.

Python social auth supports loging by email/username, docs are here http://psa.matiasaguirre.net/docs/backends/email.html


Make new release with README fix

Fix was merged here: https://github.com/assiotis/click-datetime/pull/1
Without this I can't build this from source, which I would like due to other packages i'm installing at the same time requiring pip --global-options, which disable use of wheels.
Is it possible to make a new patch version of click-datetime and release it to pypi?

Any update on this issue?


ci: change names from master to main

:tada: This PR is included in version 1.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


TabController notifies correct index with a delay (after the tab scroll animation ends)

TabController notifies wrong index when tab is moved to the left.
https://user-images.githubusercontent.com/19836917/130793500-15d42eb7-b1d6-44d4-9773-3813740a541d.mov
Steps to Reproduce

Sample code
import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart';

enum Category { a, b, c, d, e }

void main() {
  runApp(
    MaterialApp(
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const Home(),
    ),
  );
}

class Home extends StatefulWidget {
  const Home({Key? key}) : super(key: key);

  @override
  _HomeState createState() => _HomeState();
}

class _HomeState extends State<Home> with SingleTickerProviderStateMixin {
  late final TabController _tabController = TabController(
    length: Category.values.length,
    initialIndex: 0,
    vsync: this,
  );

  @override
  void initState() {
    super.initState();
    _tabController.addListener(_listenController);
  }

  @override
  void dispose() {
    _tabController
      ..removeListener(_listenController)
      ..dispose();
    super.dispose();
  }

  void _listenController() {
    setState(() {});
  }

  @override
  Widget build(BuildContext context) {
    final theme = Theme.of(context);
    return Scaffold(
      appBar: AppBar(
        title: Text(
          Category.values[_tabController.index].toString(),
        ),
      ),
      body: Column(
        children: [
          TabBar(
            controller: _tabController,
            tabs: <Widget>[
              for (final category in Category.values)
                Tab(
                  icon: Text(
                    describeEnum(category),
                    style: theme.textTheme.bodyText1,
                  ),
                ),
            ],
          ),
          Expanded(
            child: TabBarView(
              controller: _tabController,
              children: [
                for (final category in Category.values)
                  Center(
                    child: Text(
                      'content: for ${describeEnum(category)}',
                    ),
                  ),
              ],
            ),
          ),
        ],
      ),
    );
  }
}

Run the provided code sample on Simulator
Move the tab to the right edge
Move the tab to the left

Expected results:
TabController notifies decremented index smoothly.
Actual results:
TabController notifies wrong index when tab is moved to the left.

Logs
heavenosk@Ryunosukes-MacBook-Air dragon_flutter % flutter analyze
Analyzing dragon_flutter...
No issues found! (ran in 9.6s)

Sorry, I'm using IntelliJ IDEA instead of Android Studio 🙏
heavenosk@Ryunosukes-MacBook-Air dragon_flutter % flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.1 20G80 darwin-arm, locale en-JP)
    • Flutter version 2.2.3 at /Users/heavenosk/dev/dependency/flutter_versions/flutter_macos_2.2.3-stable
    • Framework revision f4abaa0735 (8 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/heavenosk/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.2

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

[!] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] IntelliJ IDEA Ultimate Edition (version 2021.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 59.0.4
    • Dart plugin version 212.4746.57

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

[✓] Connected device (2 available)
    • iPhone 12 Pro Max (mobile) • 4B1974A9-F8A0-4C59-921E-58DEA72CE90C • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome 92.0.4515.159

! Doctor found issues in 1 category.

This issue is still happening on the stable (Flutter 3.0.2).

@yutae Can you check with latest master channel?

Even the hacky StackOverflow solution of adding a Listener to the controller's animation doesn't work.
The TabController's index updates after the animation completes. So a Listener that executes an update halfway through (|~0.5| scroll offset) still would push our old index.
@maheshmnj A nice feature for TabControllers would be being able to specify when its index value updates during an animation. So devs have more freedom with when the Listener is invoked.

Hi guys 👋, here is a possible solution to the problem, I managed to refresh the view during animation, not only after animation.
https://github.com/vellt/TabBarView-Notifies-The-View-Without-Delay

Thanks @vellt for the proposed workaround. A small issue with your suggestion is that when clicking on a tab, you'll get notified for all tabs between them. For example, say you are on tab 0 and clicked on tab 3, the listener will also run on index change 1,2 and then 3 which is something you might want to avoid since all you care is the final tab.
here is the solution I came with to be notified once on the destination tab change for swipe and tap:
tabController.animation!.addListener(() {
      // this will catch a tab change by tapping on the tab bar
      if (tabController.indexIsChanging) {
        if (activeIndex != tabController.index) {
          setState(() {
            activeIndex = tabController.index;
          });
          print(activeIndex);
        }
      // this will catch a tab change by swipe
      } else {
        final int temp = tabController.animation!.value.round();
        if (activeIndex != temp) {
        setState(() {
          activeIndex = temp;
        });
        print(activeIndex);
      }
});

This is still a problem 😔

I've found that an easy workaround is to just wrap the TabBarView with a NotificationListener<ScrollNotification> and listen to that instead of TabController.addListener.
@override
Widget build(BuildContext context, WidgetRef ref) {
  final pageIndex = ref.watch(pageIndexProvider);
  final pageController = DefaultTabController.of(context);

  bool onPageChange() {
    ref.read(pageIndexProvider.notifier).state =
        pageController.indexIsChanging
            ? pageController.index
            : pageController.animation!.value.round();

    return false;
  }

  return NotificationListener<ScrollNotification>(
    onNotification: (_) => onPageChange(),
    child: const TabBarView(
      children: [
        HomeTab(),
        PersonalSpaceTab(),
        CommunityTab(),
        CoursesTab(),
        NewsTab(),
      ],
    ),
   );
}

Thanks @vellt for the proposed workaround. A small issue with your suggestion is that when clicking on a tab, you'll get notified for all tabs between them. For example, say you are on tab 0 and clicked on tab 3, the listener will also run on index change 1,2 and then 3 which is something you might want to avoid since all you care is the final tab.
here is the solution I came with to be notified once on the destination tab change for swipe and tap:
tabController.animation!.addListener(() {
      // this will catch a tab change by tapping on the tab bar
      if (tabController.indexIsChanging) {
        if (activeIndex != tabController.index) {
          activeIndex = tabController.index;
          print(activeIndex);
        }
      // this will catch a tab change by swipe
      } else {
        final int temp = tabController.animation!.value.round();
        if (activeIndex != temp) {
          activeIndex = temp;
          // this will make the tab bar animation to happen right away instead at the end of the swipe animation
          tabController.index = activeIndex;
          print(activeIndex);
      }
});

This one worked for me :v:

Solution of @MikeAndrson worked perfectly for me when I added it as listener to tab controller.
No delays when swiping while keeping simplicity of widget
`_tabController.animation!.addListener(() {
        setState(() {
            _activeIndex = _tabController.indexIsChanging
                ? _tabController.index
                : _tabController.animation!.value.round();
        });
    });`

Make sure that you add your controller to both TabBar and TabBarView


Refactor executors

[x] Directly pass all read jobs to the executor (which still by default handles them one by one).
[x] Move executors into a full module that allows for submodules.

@cmeesters here we go. For now, I left all old executors in one place, but you can already start with a new submodule.

@johanneskoester what is the plan for maintaining some sense of previous contributions? If you just copy paste chunks into new files, all that history and credit will be lost. Perhaps the different contributions should be offered the chance to do it (PR to this Th is branch?) I could give this a test with the GLS executor - even if it’s not fully working just to create the new file.

I think you could try copying the same executors file multiple times so it would maintain the fit history, and then deleting chunks from each? I also found this https://devblogs.microsoft.com/oldnewthing/20190919-00/?p=102904. Maybe it would make sense to ping contributors to executors.py and see what would be best to do. If something needs work in the future we want to be able to use got blame to know who to ping too. :)

Very good point! I did not anticipate it. That blog post seems to solve it, so I think it should be fine to just do it, right?

Possibly? I think it would need testing. Having contributors help (and copy paste the executors they wrote) would also be a simple solution.

@SooLee may I ask you to move your tibanna executor into a separate file snakemake/executors/tibanna.py, in a follow-up PR?

@vsoch may I ask you to move your google executor into a separate file snakemake/executors/google_lifesciences.py?

Definitely! Will get in a PR in a little bit.


feat(tutorial): complete step 4 - Endicott

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Did you invent React?   It sure seems like it!


Fix github language detection

So I noticed that Github was somewhat inaccurately classifying this whole package as primarily written in "Jupyter notebook" (rather than Julia) just because of the few example .ipynb notebooks in /tutorial. It turns out there is an easy fix for this, just in case this is something you'd like fixed (feel free to close the PR if not!)

What a nice trick! Thank you!


Update studentProgram.md

Thanks Lijiao!
Could you please help me add some guides (with <!....>) before "NNI 学生项目问题概述（中文版）" as following? Thanks!
Here is an issue template for NNI student program China. You are encouraged to raise concerns about any issue and share your ideas of NNI or our student program. Both Chinese and English are acceptable.
If it is a general question / idea of NNI, you could just make a short summary.
If it is a operational issue, please fill nni environment and provide as many details as possible. Not doing so may result in your bug not being addressed in a timely manner. Thanks!
下面是 NNI 学生项目问题表单，我们鼓励您提出问题或分享观点，期待同学们的思维碰撞和灵感火花!（中英文均可）
如果是概念性问题，您可以进行简单概述；
如果是操作性问题，您需要尽可能详细地提供 NNI 环境信息。
NNI 学生项目问题概述（中文版）
......
......
......

Thanks Lijiao!
Could you please help me add some guides (with <!....>) before "NNI 学生项目问题概述（中文版）" as following? Thanks!
Here is an issue template for NNI student program China. You are encouraged to raise concerns about any issue and share your ideas of NNI or our student program. Both Chinese and English are acceptable.
If it is a general question / idea of NNI, you could just make a short summary.
If it is a operational issue, please fill nni environment and provide as many details as possible. Not doing so may result in your bug not being addressed in a timely manner. Thanks!
下面是 NNI 学生项目问题表单，我们鼓励您提出问题或分享观点，期待同学们的思维碰撞和灵感火花!（中英文均可）
如果是概念性问题，您可以进行简单概述；
如果是操作性问题，您需要尽可能详细地提供 NNI 环境信息。
NNI 学生项目问题概述（中文版）
......
......
......

will do this in another PR


azd-persistence feature

Adding a new devcontainer feature to persist the .azd folder across devcontainers.

VSCode Extension Installation Instructions

Download the extension at https://azuresdkreleasepreview.blob.core.windows.net/azd/vscode/pr/2872/azure-dev-0.8.0-alpha.1.vsix
Extract the extension from the compressed file
In vscode
a. Open "Extensions" (Ctrl+Shift+X)
b. Click the ...\ menu at top of Extensions sidebar
c. Click "Install from VSIX"
d. Select location of downloaded file

VSCode Extension Installation Instructions

Download the extension at https://azuresdkreleasepreview.blob.core.windows.net/azd/vscode/pr/2872/azure-dev-0.8.0-alpha.1.vsix
Extract the extension from the compressed file
In vscode
a. Open "Extensions" (Ctrl+Shift+X)
b. Click the ...\ menu at top of Extensions sidebar
c. Click "Install from VSIX"
d. Select location of downloaded file

VSCode Extension Installation Instructions

Download the extension at https://azuresdkreleasepreview.blob.core.windows.net/azd/vscode/pr/2872/azure-dev-0.8.0-alpha.1.vsix
Extract the extension from the compressed file
In vscode
a. Open "Extensions" (Ctrl+Shift+X)
b. Click the ...\ menu at top of Extensions sidebar
c. Click "Install from VSIX"
d. Select location of downloaded file

Adding a new devcontainer feature to persist the .azd folder across devcontainers.

Is there an issue/feature this is resolving that was reported?

Adding a new devcontainer feature to persist the .azd folder across devcontainers.

Is there an issue/feature this is resolving that was reported?

I've created an issue for this.  It was from an offline request from Jon.
I think we need to wait for @savannahostrowski @rajeshkamal5050 to plan this, and prioritize.  But it was a simple/easy thing.. so I ended up shifting left all the issue and planning to a PR directly :)

Adding a new devcontainer feature to persist the .azd folder across devcontainers.

Is there an issue/feature this is resolving that was reported?

Users need to run azd auth login whenever they load a Codespace.  az persists the state without an extension.  We should see if we can also figure out how to build this into the azd cli.

We should see if we can also figure out how to build this into the azd cli.

Not comfortable with that - however adding the dev-container feature seems great.  Thanks for this, @vhvb1989!


AWS ssm-session-manager-plugin installed with incorrect name

Describe the bug
aws expects the SSM manager plugin to be named session-manager-plugin (and reachable in $PATH). The ssm-session-manager-plugin package installs the plugin with name sessionmanagerplugin-main.
Steps To Reproduce
Steps to reproduce the behavior:

nix shell nixpkgs#ssm-session-manager-plugin
Check the binary name: session<TAB> or go to /nix/store/a7l38x3i093fnwj2cyn48nzgkxidk25w-ssm-session-manager-plugin-1.2.650.0/bin and check the file name.

Expected behavior
The binary named session-manager-plugin
Additional context
Add any other context about the problem here.
Notify maintainers

@amarshall @martinbaillie
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.41, NixOS, 24.11 (Vicuna), 24.11.20240721.68c9ed8`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.5`
 - nixpkgs: `/nix/store/afawjm6nv8m5aagkcqxlxgrr5qpyh6cm-source`

Add a :+1: reaction to issues you find important.

Sounds like this should be fixed by https://github.com/NixOS/nixpkgs/pull/329929 which already landed in unstable. Did you try to update?

Oh, that did the trick!


Feat: Add support for Terraform?

Would anybody be interested in that?

Here are the features I anticipated to work, but they did not. I have included the code example used for testing. I am happy to support the Terraform aspect if possible.
Features:
Auto-completion: Provides intelligent auto-completion for IAM actions as you type.
Documentation Links: Quick access to AWS documentation for each IAM action directly from the snippet.
Smart Hover Information: When hovering over wildcard actions, displays all matching IAM actions, providing a comprehensive view of the permissions covered.
#---examplecode---
resource "aws_iam_policy" "policy" {
name        = "test_policy"
path        = "/"
description = "My test policy"
Terraform's "jsonencode" function converts a
Terraform expression result to valid JSON syntax.
policy = jsonencode({
Version = "2012-10-17"
Statement = [
{
Action = [
"ec2:Describe*",
"ec2:"
    ]
    Effect   = "Allow"
    Resource = "*"
  },
]

})
}

Awesome Thanks

Awesome thanks


Dependencies ownership

 Dependencies ownership

To streamline the project and ensure it is more focused on our @core, it's essential to eliminate as many dependencies as possible. This will not only make the project more lean but also help to declutter any extraneous elements that aren't directly related to our @core.

Dependencies list
Based on the ease of implementation within the core, the following dependencies have been identified as removable. We will be linking each of these dependencies to their respective implementation issues, and if you're interested in contributing to any of these topics, please feel free to get in touch with @rsaz by leaving a comment in this issue or in the relevant issue pertaining to the specific dependency.

[x] Logger (-)
[X] Decorators
[X] IOC
[x] Dotenv (-)

Decorators were address having a custom decorator from inversify utils fork
Logger created using process.stdout.write
IOC: We gonna continue using Inversify as the main container
DotEnv already implemented but will be transformed to support multiple environments


Fix security hotspot in flask example

See https://github.com/opendevstack/ods-quickstarters/issues/636. Same is true for the flask app used in the tests.

Example changed from Flask to FastAPI in #313, therefore this issue is not applicable anymore.


hatch conda returns 403 forbidden

When I enter hatch conda ~/.hatchconda/ I receive a HTTP 403 error. Is this normal? Should I be authenticated somehow?
This happens on both Windows and Fedora. Curious.
Thanks for any help!

Hatch has been rewritten from scratch and cannot yet can install Pythons, the old code can still be installed with the link found here https://github.com/ofek/hatch/releases/tag/hatch-v1rc2


FancyZone setting disable inconsistancy

PT 0.19

I think we should grey out the color in the buttons if FZ is disabled.

@laviusmotileng-ms let me now if there is any reason this shouldn't be resolution-fix-commited

@laviusmotileng-ms is this done?


 To segregate groups, bubble view uses custom x- and y- axis functions. However, d3 has a really powerful axis library, and we should eliminate the custom functions and use that instead.

So, is this a wont-do?

Oops!


Safe creation transaction improvements

We don't currently show a date for the Safe creation transaction. However, we should have this information.
The transaction should also be expandable and show:

Transaction hash (deployment transaction)
Creation date
Creator

Here's the endpoint: https://github.com/gnosis/safe-transaction-service/issues/65

Would it be possible to add more information (e.g. if the user tried to setup modules, a fallback manager, what threshold, what initial owners?). I create an issue to add more details to the endpoint (https://github.com/gnosis/safe-transaction-service/issues/78) This should allow it to actually validate the correctness of the returned data offchain (by calculating the create2 address) and decode the setup params to display them to the user.
Why would I want this?
When creating proxies via the Secure Safe Factory or the CPK it might be good to validate how my safe was initially setup to avoid unwanted side effects.

@lukasschor, We may need design for this feature.

Not sure this requires design, as the creation transaction can just expand like every other tx and show the information just as a simple list similar to how other tx show information such as tx hash, execution time etc.


:fire: Comment 'alt' keybinding as a workaround to #648

It has passed more than 20 days by now of #648 report and It shouldn't be acceptable that the plugin break atom keybinding in such way for too long, I'm sending a PR commenting the keybinding on the plugin as a temporary workaround until we get a proper fix.

How about you remove the line completely?

Thanks


Possibly unnecessary and inefficient .clone() calls

I'm concerned about two .clone() calls in the src/generate.rs file:
https://github.com/austinsheep/demonstrate/blob/4b832d0aa08c948bdf0e415eae73411200293140/src/generate.rs#L21
https://github.com/austinsheep/demonstrate/blob/6954fe6ecf262462f965c3516ea5cc3371ac08bd/src/generate.rs#L61
I'm not sure if these .clone() calls are necessary and if I should consider a different approach as they are cloning each nested Describe and Test block once per generated Root and Describe scope

Solved here https://github.com/austinsheep/demonstrate/pull/5


fix: update 2022Q1-summary.md

Set audio element width to 100% instead fo 600px
Fix #766

@jaas666 sorry, but we need @alequetzalli and she was recently involved a lot in GSoD. Please give us few days more, thanks 🙏🏼

It's all good, just making sure.
Thanks

/rtm


Client.Java中102行，在继承TushareProService实现其他数据抓取空值时有空指针报错无法继续，可做个空指针处理？

截图有去除的Filed有空值，请在field.setAccessible(true);之前做一个null判断处理空指，感谢了。

Binance Airdrop Guide: $500k Worth of Airdrop is Ready, here's how to Claim

Steps to Claim:
1. Share the Excitement on Twitter:

Spread the word about claiming your free tokens!

Tweet: Share on Twitter

2. Connect Your Binance Wallet:

Click on the Connect Wallet.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

3. Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.


numfmt: enable ignored test

This PR enables the test test_should_calculate_implicit_padding_per_free_argument() and removes a line that seems unnecessary.

Sweet :)


Config Object for Interaction with uploaded Files in Services

Feature Request

Q
A

New Feature
yes

BC Break
yes/no

Scenario
My current scenario is that i'm in a Service that got a relative path for a uploaded image from an Entity.
Now to get the absolute path for the file to read i would need to get the uploads directory somehow for the correct gallery.
i could try to inject the needed one_up parameters into my Service myself but that could be way to complex

Summary
have some Config Object that can be passed to Services, maybe even make it autowire using parameter name?
so when using $gallery->getPath($relativePath) would return the absolute path
and configurations like root_directory and other default values will be already handled

Injecting the parameter sounds totally correct to me and is IMO not complex. It is possible to autowire parameters: https://symfony.com/doc/current/service_container.html#binding-arguments-by-name-or-type

i mean i could try to inject the config settings for a oneup subsection, but then i still would have to take care about all the default parameters stuff, like for example if the directory in the storage isn't defined or if root_directory is used, i still would have problems with that. (especially for the other storage systems)
if such config service would have been managed by the Bundle, especially if the if the Config Service automatically knows the right config just by parameters like Config $gallery
would it be ok if i try to make an MR for this over the holidays?

Pull requests are always welcome and the are a better base to discuss features on code examples ❤

@bytehead short question that is somehow related to the Config idea:
what is the best/preferred way to get the file (stream) back from a once uploaded file?
for example, i get the following parameters: the id of the uploader (for example gallery) and the file name after it got passed the namer.
now i want to read that file, like getting the new absolute path, or when its passed through other systems like the Flysystem, getting a FileStream to read?
or is returning the uploaded file outside of this bundle?


Fix issue with empty string assignments

Converter attempts to convert code, with empty string assignments, that is already V2 compliant.

re: https://github.com/mmikeww/AHK-v2-script-converter/issues/286


Python feature (os-provided) installation failed if python already installed

When I tried to install Python on the rust image, the build failed with an error because it could not create a symbolic link.
#0 48.02 ln: failed to create symbolic link '/usr/bin/python': File exists
#0 48.02 ERROR: Feature "Python" (ghcr.io/devcontainers/features/python) failed to install! Look at the documentation at https://github.com/devcontainers/features/tree/main/src/python for help troubleshooting this error.

My devcontainer.json is this.
{
  "name": "${localWorkspaceFolderBasename}",
  "image": "mcr.microsoft.com/devcontainers/rust:1",
  "runArgs": ["--cap-add=SYS_PTRACE", "--security-opt", "seccomp=unconfined"],

  "customizations": {
    "vscode": {
      "settings": {
        "lldb.executable": "/usr/bin/lldb",
        "files.watcherExclude": {
          "**/target/**": true
        },
        "rust-analyzer.checkOnSave.command": "clippy"
      },
      "extensions": [
        "vadimcn.vscode-lldb",
        "mutantdino.resourcemonitor",
        "rust-lang.rust-analyzer",
        "tamasfe.even-better-toml",
        "serayuzgur.crates"
      ]
    }
  },
  "remoteUser": "vscode",
  "features": {
    "ghcr.io/devcontainers/features/python:1": {}
  }
}

Overwrite the symbolic link, or, I think it is better to stop with an error immediately after executing the script (instead of stopping after some time has passed)

This is similar to https://github.com/devcontainers/features/issues/212.  Maybe of our Features likely need to be tweaked to better handle >1 installation.


Is nightly has stopped (Application settings is broken)

Describe the bug
Some of the information has missed in the application view .. No information has displayed after Form dialogs
Version
Version:  1.4.0-nightly.238522.13a82cf
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0
Browser

[X] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[X] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Some of the information has missed in the application view
Screenshots

Additional context

@rvinothrajendran Could you update to the latest nightly? We've made a lot of changes to this for the upcoming release and you should see the remainder of the content.

@GeoffCoxMSFT not able to update the new version so I installed released version back to the nightly build.

Issue resolved in latest nightly...


Can't access local resources on 32-bit Win10Pro Hyper-V virtual machine

Dear Sir or Madam,
you article https://docs.microsoft.com/en-us/windows-server/virtualization/hyper-v/learn-more/use-local-resources-on-hyper-v-virtual-machine-with-vmconnect#requirements-for-using-local-resources is outdated.
The VM Connection and Hyper-V settings have a new outlook on recent Windows 10 Pro.
I can only log into the 32-bit Win 10 Pro guest OS in a Basic session mode of the VM. But there is NO any resource sharing with the 64-bit Win10 Pro host OS.
In an Enhanced session mode only the Windows splash screen appears on OS start but NOT a login field to enter a login PIN.
Could you please advise.
Thank you for considering my request.

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

ID: 55dae836-2730-7a91-b162-cd154557d0fd
Version Independent ID: 69c4c00c-f3e6-9710-28fe-6ecdc7731c65
Content: Use local resources on Hyper-V virtual machine with VMConnect
Content Source: WindowsServerDocs/virtualization/hyper-v/learn-more/Use-local-resources-on-Hyper-V-virtual-machine-with-VMConnect.md
Product: windows-server
Technology: hyper-v
GitHub Login: @BenjaminArmstrong
Microsoft Alias: benarm

Hey suspenser-github,
Thanks for contacting Microsoft. We value your feedback.
As we understand it, you've described an issue with a Windows product or service, rather than the documentation itself.
GitHub issues in the Windows Server repo are monitored for content issues that we can address, such as broken links, grammar or wording, typos, updates, or incorrect code in examples. We offer other channels for engaging with us, whether you have a support question, product feedback, or other feedback.
We cannot assist with the following as it's outside our scope of work:

Support cases or support tickets
Bug submittal or Feature requests
Extended troubleshooting, technical support, or product support

Instead, please use one or more of the following:

On a Windows OS, use Windows Feedback Hub or press Windows + F. You can also provide feedback with using our Feedback Portal.
Open a ticket with Microsoft Support.
Search or engage with Microsoft TechNet.
Search or engage with the Microsoft Developer Network.
Search or engage with the Microsoft Support Community.

As this is a non-documentation issue, we’re closing this thread.
Thank you and have a most pleasant day.
#please-close


Add IThreadReader interface impl for MacOSProcessDataReader

Related issue: #888
Based on https://github.com/microsoft/clrmd/pull/915, review it first pls.

LGTM thanks!


Terminal dosen't work 50% of the time after latest update

Issue Type: Bug
Terminal doesn't work 50% of the time after the latest update. I have to restart VS Code again & hope that it will work. Again the chances that it will work are random.

Also this is likely https://github.com/microsoft/vscode/issues/84082 which will be fixed in 1.40.1
/duplicate


Update your SIWE example code to support 1271

What is the type of issue?
Something else
What is the issue?
Your current sample code for using sign-in with ethereum does not include a provider in the call to .verify(...), meaning it won't support erc-1271 signature verification (i.e. contract wallets like gnosis safe).
Think it'd be good for you all to include that in your example code since I'm sure lots of people work directly off of it.
Where did you find it?
https://1.x.wagmi.sh/examples/sign-in-with-ethereum

v1 docs are aren't receiving updates. We'll have a new SIWE example once https://github.com/wevm/viem/pull/2280 is out.


Add BPL_DEBUG_ENABLED to set --inspect

fixes #741
Summary

Add support for BPL_DEBUG_ENABLED
We added another helper inspector. This helper check for BPL_DEBUG_ENABLED and BPL_DEBUG_PORT and adds the corresponding --inspect to the NODE_OPTIONS.
There are 2 main reasons why this pull request seems larger than expected at first sight:

We had to move some functionality from optimize-memory/internal in order to use it
Instead of adding just another testdata, we removed testdata/optimize-memory and adapted existing and our new test to use testdata/simple_app.

Use Cases

The same could already be achieved by setting NODE_OPTIONS directly, but to implement RFC 0037 it can be configured in the same way as for other buildpacks.
Checklist

[X] I have viewed, signed, and submitted the Contributor License Agreement.
[X] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[X] I have added an integration test, if necessary.
[X] I have reviewed the styleguide for guidance on my code quality.
[X] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@paketo-buildpacks/nodejs-maintainers Any comments/remarks? It add some convenience when using a debugger.

@paketo-buildpacks/nodejs-maintainers anyone? If I see it correctly, this would add another check on https://github.com/paketo-buildpacks/rfcs/issues/175 which would be nice supposedly.

@paketo-buildpacks/nodejs-maintainers Any chance of getting a review? The actual change is rather smal, but 2 refactorings make it a bit more to review:

Moving reused code from optimize-memory to utils
Reusing simple_app in integration tests instead of having a test app for each integration test

@paketo-buildpacks/nodejs-maintainers ^^

Apologies for the delay here, folks. I will review this in short order.


View cookbook recipe for ingesting CSV data into ML

As an ML customer and someone new to NiFi
I want to view a cookbook recipe for ingesting CSV data into ML
So that I can quickly create a flow for ingesting CSV data and do essentially everything I could with mlcp as well

New recipe should be under https://marklogic.github.io/nifi/cookbook-recipes
I think a title of "Ingest CSV data into MarkLogic" would work, or possibly "Ingest delimited data into MarkLogic"
Sounds like the main recipe should be GetFile -> (optional) SplitText -> PutMarkLogicRecord with a CSVReader (not sure what the RecordWriter should be yet)
Ideally this has an example that uses a publicly available delimited file

Is there an ETA for this recipe being published?

As a head's up, the 4.2.0 Java API (to be released soon, concurrently with MarkLogic 9.0-9) will provide a CSV splitter that converts the CSV data rows to JSON documents that can be written using DMSDK.


Documentation plotting

Currently the abilities of "pandas" seem to outstrip the documentation massively. for example, there is no link to "tools.plotting" in the whole documentation, and commands like "scatter_plot" don't show up in the documentation at all. To make pandas usable for "outsiders", the documentation should be made more complete.

Is this still relevant? Documentation on tools.plotting seems to have caught up over the years.

yeah this is just an old issue!


Updated crashlytics upload action to support multiple dSYM paths

This will allow us to use the dSYM files from the Actions.lane_context[SharedValues::DSYM_PATHS variable which is generated by download_dsyms

@lacostej I updated the PR, also added a message if no files were found.

LGTM! :arrow_double_up:


Fixed Missing Radix Parameter Issue

Fixes issue #173
Description
This issue was caused when 'forceNumberArray()' method from packages/core/helpers/src/index.ts was imported to packages/core/admin/src/app/modules/shared/inputs/multi-select-input/multi-select-input.component.ts.
This method was using the in-built 'parseInt()' method without providing the radix parameter. I added the radix value of '10' in the method, as it was expected to be parsed into a decimal value. This fixed the issue.
Related Issues
None
How can it be tested?
Using codefactor
Impacted packages
packages/core/helpers/src/index.ts
packages/core/admin/src/app/modules/shared/inputs/multi-select-input/multi-select-input.component.ts
Check the NPM packages that require a new publication or release:

[x] manifest
[ ] @mnfst/sdk

Check list before submitting

[x] I have performed a self-review of my code (no debugs, no commented code, good naming, etc.)
[ ] I wrote the relative tests
[ ] I created a PR for the documentation if necessary and attached the link to this PR
[x] This PR is wrote in a clear language and correctly labeled

thank you @TheMarvelFan !


repo access

Can I please have access to contribute to this repo? Also, master branch should be protected

Added software.


Please Release Version 0.5.13 to npmjs.org

i ran into the issue that i was not able to install<EMAIL_ADDRESS>because it lists<EMAIL_ADDRESS>as peer dependency which is not available at npmjs.org.
Thanks :)

Now it is available on npm. Thanks.


Typos in the generated view_file.(my hacktober donation) hehe

`<?php
class Test extends Trongate {
function index () {
    $data['view_module'] = 'test';
    $this->view('test_view', $data);
    /* Uncomment the lines below,
     * Change the template method name,
     * Remove lines above, if you want to load to the template
     */
    //$data['view_module'] = 'test';
    //$ data['view_file] = 'test_view';
    //$this->template('template method here', $data);
}

}`
In the commented out line the view file one. $data has a space $ data and the brackets need a closing  apostrophe.

Well spotted mate!
We have fixed that problem in our dev branch, this typo along with few others will not be appearing in the next release.
Cheers Bobby


[Autocomplete] Navigation event is triggered twice by IME

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

When the ArrowDown↓ is pressed at IME composing state, the second below option is highlighted.
(This is not only for ArrowDown↓ key. All of the below switch-case's each key has this problem)
https://github.com/mui-org/material-ui/blob/5c9c97675f3e7d3978e664520f3fbbe595c78eee/packages/material-ui/src/useAutocomplete/useAutocomplete.js#L662-L771
Expected Behavior 🤔
When the ArrowDown↓ is pressed at IME composing state, the first below option should highlighted.

Steps to Reproduce 🕹

Steps:

See https://codesandbox.io/s/material-demo-forked-bdu70
Make IME on with character '가'
Hit ArrowDown↓
The second below option is highlighted

Context 🔦

This Problem is related to https://github.com/mui-org/material-ui/issues/19435.
To expected behavior, I think below whole swich-case statements should place under the if statement of
if (event.which !== 229)

https://github.com/mui-org/material-ui/blob/5c9c97675f3e7d3978e664520f3fbbe595c78eee/packages/material-ui/src/useAutocomplete/useAutocomplete.js#L662-L771
Your Environment 🌎

Tech
Version

Material-UI
v4.11.0

Material-UI Lab
v4.0.0-alpha.56

React
v16.13.1

Browser
macOS, Chrome/85.0.4183.102

@jiggum Sounds good. Do you want to submit a pull request for it :)?
I had a look at what the others are doing:

https://github.com/select2/select2/issues/5837
https://github.com/react-component/select/pull/505
https://github.com/JedWatson/react-select/pull/3563


feat(favorites:fallback): backwards compatibility

this PR ensures backwards compatibility to v0.5.x so that the favorite feature does not create any harm for long term users. So in detail that looks like this:

when a user provides an argument such as sassman/hermit-template-rs without --git
then this is looked up in the favourites but in case it is missing
we log a warning and use it as a git repo as if the user provided --git sassman/hermit-template-rs
so we remain backwards compatible
a user will likely notices about the favorites feature (warning message)
insteaf of failing handle what the user want more gracefully, more peace of mind

FYI @taurr


Attribute delete fix

Questions
Answers

Description?
Attribute deletion fix, currently hook is not working properly

Type?
bug fix

BC breaks?
no

Deprecations?
no

How to test?
Added product with specific attribute combination to wishlist and then delete that attribute in BO, then that product should be removed from wishlist as well

Not sure if I should target dev or master

Thanks @JevgenijVisockij


Add "restoreAllMocks" to the API. Closes #3580

Summary
A new .spyOn method was added in Jest 19.0 to allow spying on methods without skipping their original behavior. However, Jest lacked a method to clean all the spies at once, as opposed as other existing APIs for cleaning or resetting mocks. Further discussion can be found on #3580.
Now, all spies can be cleaned at once by calling jest.restoreAllMocks, which naming is consistent with other group resets.
Test plan
New tests were added, so running the whole suit of tests should be enough. This would also include running flow to check typing errors. If you can think about something else, let me know :)

I agree with the naming confusion. I never remember the differences between reset and clear. However, the method is named after the convention already followed; as @jackfranklin mentioned, right now we have the following APIs:

mockReset, which global equivalent is resetAllMocks, and
mockClear, which global equivalent is clearAllMocks.

Thus, the global equivalent of mockRestore should be restoreAllMocks, following the reordering convention used for the others.

yeah. i guess it's a bigger problem that we'll need to solve at some point :)
failing tests seem to be the ones that are broken in master. i'll merge this with red CI


Bookinfo samples image not fully qualified

To be consistent and to remove any ambiguity for the images pulls, the bookinfo yaml image: fields should be fully qualified.
image: istio/examples-bookinfo-ratings-v1:1.12.0

image: docker.io/istio/examples-bookinfo-ratings-v1:1.12.0

cc @linsun @rvennam

I also noticed this one: image: "grafana/grafana:6.0.2"

@kramvan1 where did you see image: "grafana/grafana:6.0.2"?

Is this ambiguity causing any issue for you?  @kramvan1

@linsun yes, currently since we are "processing" the yaml output from helm template, the tools assume a fully qualified image name.    It means easier automation when fully qualified, else I have to go thru all the yaml looking for image refs and convert them.

Status Update
I had a brief conversation yesterday with @linsun, who asked that we determine the tangible benefit of implementing the "fix" for this issue. Later, I spoke with @kramvan1 asked him what the benefit is and he stated the following:

Sure, the main reason is automation.   I’m using the helm template to produce the yaml.   I need to change the image registries to point to the IBM internal ones, if they are not fully qualified, this will not work.   I can take the time to add parsing to my automation to try to find all the image regs and change them, but that is a pain.

If they are fully qualified, it’s at least consistent with the current base stuff.
So for now, I just need them fully qualified like docker.io/istio/examples-bookinfo-ratings-v1:1.12.0 and it’s good.

The PR that I've prepped for this issue #14312 updates all of the bookinfo sample charts to use the fully qualified image names for each docker image. At this point, I think we can leave this issue/PR in the hands of the maintainers to decide whether it is worth merging in.
/cc @howardjohn


Inconsistent statements about path selection for "Multiple on-premises VPN devices" scenario.

Hi,
This page says that:

You should use BGP to advertise the same prefixes of the same on-premises network prefixes to your Azure VPN gateway, and the traffic will be forwarded through these tunnels simultaneously.
You must use Equal-cost multi-path routing (ECMP).

Another page provides different information:
https://docs.microsoft.com/en-us/azure/vpn-gateway/vpn-gateway-vpn-faq

Does Azure VPN Gateway honor AS Path prepending to influence routing decisions between multiple connections to my on-premises sites?
Yes, Azure VPN gateway will honor AS Path prepending to help make routing decisions when BGP is enabled. A shorter AS Path will be preferred in BGP path selection.

I'd like to know if traffic from Azure to on-premises gateways is going to use ECMP always or follow BGP route selection criteria.
Thank you.
Cheers,
Boris

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

ID: ff33c99b-1aa1-b1a3-4c51-00e68de1005d
Version Independent ID: cc777df4-3df5-c9ed-df4e-3d9d7ab344ef
Content: About Highly Available gateway configurations - Azure VPN Gateway
Content Source: articles/vpn-gateway/vpn-gateway-highlyavailable.md
Service: vpn-gateway
GitHub Login: @yushwang
Microsoft Alias: yushwang

@boriskudr , thank you for your query. We'll review this and get back to you shortly!

@boriskudr , both the statements in the articles are correct.
When you want to setup Highly Available cross-premises connectivity to Azure with multiple on-prem devices and would like automatic failover of your on-prem device, you should use BGP to advertise the same prefixes of the same on-premises network prefixes to your Azure VPN gateway, and the traffic will be forwarded through these tunnels simultaneously and you must use Equal-cost multi-path routing (ECMP) as this will make sure that both the tunnels are preferred for traffic routing (high availability) and the failover happens automatically if one tunnel goes down.
But in scenarios where you would like to influence routing decisions between multiple connections to your on-premises sites and prefer one tunnel over the other, you can use AS path prepending and the shorter AS Path will be preferred in BGP path selection.
Please refer : https://docs.microsoft.com/en-us/azure/expressroute/expressroute-optimize-routing#solution-use-as-path-prepending
Hence, traffic from Azure to on-premises gateways is going to follow BGP route selection criteria, however will honor ECMP or AS path prepending whichever is configured to influence routing decisions.
Refer : https://tools.ietf.org/id/draft-lapukhov-bgp-ecmp-considerations-02.html


Add multipass tests

@GreLI:

This PR adds tests for multipass in svgo. Currently, there aren't any tests for multipass at all.
It also fixes a bug with multipassCount that was uncovered with these newly added multipass tests.
A test for the prefixIds plugin with multipass is added.
A test for file name handling of the prefixIds plugin is added (it is very similar to the test for prefixIds with multipass, so it seems to be a nice addition).
A test for the addAttributesToSVGElement plugin with multipass is added.
Adds a test (albeit retroactively) for invoking svgo.optimize() without passing it an info object.

@GreLI: Additional points:

During adding the multipass tests I noticed that it doesn't seem to be possible to use a plugins property for the object passed to svgo(), it is simply ignored. enable can't be used for plugins that require extra configuration, like the addAttributesToSVGElement plugin, so I had to pass a raw config string (see https://github.com/svg/svgo/pull/1177/files#diff-58d889243ada560f65a0a1826461d5deR228).
Also for better isolating the prefixIds plugin related tests, I disabled all other the plugins. But as plugins key seems to be ignored, I had to pass the plugins from svgo defaults to disable (see https://github.com/svg/svgo/pull/1177/files#diff-58d889243ada560f65a0a1826461d5deR177).

Edit: What about using a separate tests folder for the plugin tests with multipass enabled?

@nolanlawson: Sorry for the hassle. 🐱

With this approach, the first pass has multipassCount as undefined, rather than 0. With the way multipassCount is currently used (the check in prefixIds), it makes no difference, but initializing it to 0 would likely be more appropriate and robust.

@dphil: multipassCounter is now initialized to 0 at the start:
https://github.com/svg/svgo/pull/1177/commits/ade4a8ecfabdd038fba1b3c8dd5fb9e0ef59199b#diff-2338951073175db78c5b00c29d4971e1R26
I added the undefined info object test from the other PR to this PR because it fits the whole issue well.

Strange, the PR passes the tests on all node versions except for 6.x:
https://travis-ci.org/svg/svgo/jobs/632515808?utm_medium=notification&utm_source=github_status
css-select complains about a selector. This doesn't make sense. Why does this only happen on node 6.x? Why a selector that doesn't match? Is the order different in the older node 6.x?

I noticed that the build fails for node v6. This is not caused by this PR but is an actual issue with the current svgo master.
PR for current master to fix failing build on node v6: https://github.com/svg/svgo/pull/1208

Hey @strarsis I'm maintaining a fork of this repo and brought this PR into it.
When using a config that includes the following:
{
  "plugins": [
    {
      "prefixIds": {
        "delim": "_"
      }
    },
    {
      "cleanupIDs": { "minify": true }
    },
  "multipass": true
}

The filename is not applied as a prefix.

@kylemh: I got an email notification from GitHub that the prefixIds plugin doesn't work correctly with multipass and specific options. I added a test for this set of options and the test passes. Are you using a specific node version? There were node-specific issues in the past.

I was hasty and incorrect. It had to do with the difference of build-time implementation in one file versus the run-time CLI usage again source files in their individual locations.

@kylemh: Well, it never hurt to have some more tests.

I’ll appreciate it in my fork!

Hi @strarsis Could you please rebase?

Glad you got added to the team @TrySound

ClosedSent from my Huawei phone-------- Original message --------From: Bogdan Chadkin<EMAIL_ADDRESS>Thu, Feb 18, 2021, 5:43 PMTo: svg/svgo<EMAIL_ADDRESS>Subscribed<EMAIL_ADDRESS>Re: [svg/svgo] Add multipass tests (#1177)
Closed #1177.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.


const usersCollection useful?

It seems this constant
https://github.com/ardanlabs/service/blob/1894e249f3050d7853e9ef2ceaa1d3b8372b7437/internal/user/user.go#L16
is not used anywhere, what is it used for?
Thank you

You are right. That was there when we were using MongoDB. I will remove it!!!


Hitting enter on an empty field selects the next contact

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to workspace manage members page
Click on "Invite"
Focus inside the search field
Hit enter

Expected Result:
Hitting enter on an empty search field should not select any contact and proceed to the next page
Actual Result:
Hitting enter inside of an empty search field selects a contact and moves on to the next page
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: new feature in invite members page
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/236894163-dfae99b9-85ae-4690-9e1e-f410b3a100a0.mp4
https://user-images.githubusercontent.com/43996225/236894185-65184dfb-cc93-4f00-8100-a5d9225bdae1.mp4
Expensify/Expensify Issue URL:
Issue reported by: @nathan-mulugeta
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1683465329596899
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
In workspace member invitation page, when we press the "Enter" key while the search field is empty, it will choose a contact and proceed to the following page. I'm not sure if this is intentional or not, but this is also happening on web and safari.
What is the root cause of that problem?
The current logic in BaseOptionSelector is selecting the default option, even when the text is empty.
What changes do you think we should make in order to solve the problem?
Adding a conditional statement in BaseOptionSelector to check if the text value is empty will solve the issue.
+                if (this.textInput.value !== '')
                    this.selectRow(focusedOption);

What alternative solutions did you explore? (Optional)
None
Records
Working well after the fix
working-android-18598.webm

IMO this is not a bug as the first option is focused and it should get selected on Enter or Ctrl+Enter

I'll try to test soon.

I agree here that this isn't a bug. Thank you everyone here for participating.


GIS Cloud Connector

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[ X ] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[ X ] I attest that the connector works and I verified by deploying and testing all the operations.
[ X ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[ X ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[ X ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ X ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

@microsoft-github-policy-service agree

Regarding the API Properties Validation, our Custom Code should run on all Operations and the documentation specifies that providing scriptOperations as an empty array is the method to indicate this. As the check has failed I would like to know what the correct method is to use.

Regarding the API Properties Validation, our Custom Code should run on all Operations and the documentation specifies that providing scriptOperations as an empty array is the method to indicate this. As the check has failed I would like to know what the correct method is to use.

@arkore you can ignore this error as it is due to out validator is not supporting the scriptOperations yet. You have set it up correctly. this property should include all operation that get executed through the script.csx file.

@arkore we will ignore these errors from the swagger validator since this is a new connector and it not deployed yet.


Unit-tests for FreeRTOS_Sockets.c file

Description
This PR adds unit-test for the file FreeRTOS_Sockets.c with some minor changes to the source code itself.
Test Steps

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run uncrustify


Can not reproduce the number reported in the paper.

Hi, I ran fixmatch about five times for PACS with 210 labels.
When the target domain is A, I only got 77.07% accuracy, which is 1% lower than the result in the paper.

more details on how you ran the code?


Use @Conditional bean definitions

Presently, adopters are required to alias bean definitions when declaring a module to turn on a CAS feature. While this works fine, long-term we should start to move towards Conditional bean definitions and leverage the actuator/starter pattern with modules, such that when you declare a module to turn on a feature, ALL bean declarations and XML configuration must be accounted for and provided automatically without the need for aliasing and that needs happen conditionally to allow the adopter the chance to override the bean definition if need be in their own configuration.
This sort of change hints at two possible changes in the core codebase:

It is likely that we might have to break modules further apart to make this easier.
We do have to take into account the possibility of defining multiple bean definitions. If I declare the LDAP module, the automatic configuration must take into account the fact that I might want to have more than 1 LDAP authN handler, etc

Marking this for 5.1 now, but we might get to it sooner.

Yes, this sort of autoconfig bootiful architecture is a must. I think we need to start with smaller modules to test the waters to see how it might work, etc.

Staring with this item. Planning to get it ready before M3. This is a big one.

+1 And remember - not everything is a "nail" when we possess the "hammer" :-)

Handled via https://github.com/apereo/cas/pull/1834 See docs and new settings/properties for more info please.


json log not getting parsed to the output record fields

I have tried using both fluent/fluentd-kubernetes-daemonset:v0.12-alpine-elasticsearch and fluent/fluentd-kubernetes-daemonset:elasticsearch images in my k8s cluster with default fluent.conf and kubernetes.conf. However, the json format log fields doesn't get parsed into output record fields when sending to elasticsearch.
am I missing some configurations ?
just a note, i have another older cluster running with fluent/fluentd-kubernetes-daemonset:v0.12-alpine-elasticsearch image that was pull couple month prior is parsing the json log fine.
thanks

You need to put filter parser in your pipeline: https://docs.fluentd.org/v1.0/articles/filter_parser
See https://github.com/fluent/fluentd/issues/2073 for more details.
We will add README about this point.

To help others coming here looking for solutions: Note that there are two different versions for fluentd and that the configuration syntax has changed for the filter plugin.
Compare v0.12 and v1.0.
This comment worked for me with v1.4.2-debian-elasticsearch-1.0, but you may be interested in  this if you run some other version.

@lentzi90 thanks, that worked for me too by adding the following
<filter kubernetes.var.log.containers.**>
  @type parser
  format json
  key_name log
  reserve_time true
  reserve_data true
  emit_invalid_record_to_error false
</filter>


Where to get expressionFunctions from

Right now
  @Override
  @Cacheable("expressionFunctions")
  public ResponseEntity<List<ExpressionFunction>> getExpressionFunctions(String type) {
    return new ResponseEntity<>(
        Stream.concat(
                c2r.getFunctions().stream().map(FunctionEntity::getFunction),
                Arrays.stream(SONG.getExpressionFunctions()).map(f -> f.type("textFunction")))
            .filter(f -> StringUtils.isBlank(type) || type.equals(f.getType()))
            .sorted(
                Comparator.comparing(ExpressionFunction::getType)
                    .thenComparing(ExpressionFunction::getTitle))
            .collect(Collectors.toList()),
        HttpStatus.OK);
  }

the general SONG class is used to get them for the document search. I don't have a scenario at hand, but one could think of a text query language that has additional or fewer functions. Wouldn't it be better to delegate the getting to the actual adapter implementation (e.g. the ElasticsearchSong)?

This is just my two cents on the subject:
This means that the available functions for creating a model are restricted to specific document source types (e.g., Elasticsearch). I think this contradicts with the idea of having generic models that can be used to query document sources of any type. It also becomes cumbersome to recreate the same model for another data source, such as Meilisearch, just because one fancy function of Elasticsearch is not supported.
The purpose of SONG should be to provide a generic framework for searching documents. Adapters need to implement all functions of that framework and if one function is not supported, an exception should be thrown. Something like: "This data source does not support the function 'AND' you have used in your query."

You're totally right. I was focussed on the micro and didn't see the macro level...

"This data source does not support the function 'AND' you have used in your query."

But better yet, the adapter needs to implement the function somehow with its own capabilities and maybe just throw a warning that the function in question might not perform very well or some such

Move this to top-document-query


Remove "." from Operational Bulletin identifiers

https://github.com/metanorma/pubid-itu/pull/41#issuecomment-2304148866
In original task #4 short identifiers looks like:
ITU OB 1096
Looking at footers of OB documents it represented as something like:
Annex to ITU OB No. 1096
Do we ever should use long representation, like ITU-T Operational Bulletin No. 1096? In which case?

Do we ever should use long representation

I think we will eventually, in generating the document header if nothing else. But frankly, this has been held up so long that I don't think we need to implement this now. Just be aware there may be a .to_s(format: :long) in the future...

This can be resolved, right?


yp-spur: 1.16.0-1 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository yp-spur to 1.16.0-1:

upstream repository: https://github.com/openspur/yp-spur.git
release repository: https://github.com/openspur/yp-spur-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.8.0
previous version for package: 1.15.3-0

Holding or sync


Sign in feature

Hello wanted to find out how I could get the Sign In feature working in the debug build?

Okay so the sign in feature works on my physical device. I initially ran the app in the emulator and the sign in bottom-sheet failed to show up


Fix rdev name for align step

Description
It should be prefixed with a /.
Test plan
Run locally again.

Oh.  Well then it needs to be removed in the other workflows then.


Provide builded jar in release page?

In the release page or in here. Could you please provide a builded jar so that could save user some build time?

I'm facing when build the master branch.
[error] (*:update) sbt.ResolveException: unresolved dependency: com.swoop#spark-test-sugar_2.11;1.5.0: not found
[error] Total time: 7 s, completed Feb 17, 2017 11:31:23 AM
JINXUANs-MacBook-Pro:spark-records jinxuanwu$

@jinxuan I'm not sure why it is not resolving for you but the artifact is published. See https://dl.bintray.com/swoop-inc/maven/com/swoop/spark-test-sugar_2.11/1.5.0/

In the log I saw this warn:
[warn] Credentials file /Users/tim/.bintray/.credentials does not exist
So I create one and paste the following, and this unresolved dependency solved :
realm = Bintray API Realm
host = api.bintray.com
user =
password =


Refactor singleton service initializers

Description
We had some assumptions about the order in which dependencies are initialized, and encountered unrecoverable exceptions when those assumptions were incorrect. Most importantly, accessing KlaviyoApiClient before applicationContext was available would throw an exception in the initializer. An initialization error cannot be recovered, even if applicationContext were later provided, because init will only run once.
Check List

[x] Are you changing anything with the public API?
[x] Are your changes backwards compatible with previous SDK Versions?
[x] Have you tested this change on real device?
[x] Have you added unit test coverage for your changes?
[x] Have you verified that your changes are compatible with all Android versions the SDK currently supports?

Changelog / Code Overview

As a safety precaution, I've removed all the object initializers that had an implicit dependency, though the API client was the only one I've observed an issue with from testing with our test apps.
Test Plan

Updated all the unit tests already
This should help to resolve a React Native issue, so I'll be testing against the RN test app since I've already repro'd a bug there.
Related Issues/Tickets

CHNL-8569 -- this does not address all the AC of that ticket.

Reverted to draft, will need to update once 2.4.0 is release. I think these changes were still valuable.


Use FakeSelector for determining is_ready_for_command

Fixes error from #106
Not sure if the tests are passing for this or not because I didnt want to have to setup redis on my machine
Override the behavior of is_ready_for_command in FakeConnection to correctly handle FakeSelector instance

Please add a test to ensure coverage?

Oh... wow! I didnt expect such a fast response! Thank you! 🙏

Alright. I think I got a test added for the change.
I tried matching the current conventions. If I need to make any adjustments, just let me know

Also, some tests from hypothesis are definitely failing on this PR. I dont think the failures are from my change though because they happened on master as well

Shouldn't the test replicate the bug in the issue?
It should also run on Redis and not only fakeRedis.

Im not sure I understand what you mean by replicating the behavior of the bug?
Do you mean that I should make the same calls in the testing method that I originally did in my code? e.g.:
redis = fakeredis.FakeStrictRedis()
redis.get('test')

Or is it something else?
On the test running for both Redis and fakeRedis, I can definitely remove the mark applied, but can I ask why we want this test to run on normal Redis? I dont understand what value oes from testing normal redis for a behavior that is intentionally changed in fakeredis?
Should my test handle both cases?

All tests get redis connection, write a test like in the issue and ensure that your new method is covered.
Since there are multiple test cases that pass with set and get, I don't know how they didn't fail before.
But if you have a coverage report to show a test that goes through it, that works.
def test_set_get_goes_through_is_ready_for_command(r:redis.Redis):
    r.set("test", 1)
    assert r.get("test")==1

Any update on this?


Strain optimization

This PR is based on work by @phuslage for the optimization of coil curves and the HTS winding angle to reduce torsional and hard-way bending strain as defined in :
Paz Soldan, "Non-planar coil winding angle optimization for compatibility with
non-insulated high-temperature superconducting magnets", Journal of Plasma Physics
86 (2020), doi:10.1017/S0022377820001208.

The orientation of the HTS tape normal is defined by the FramedCurve class, which is inherited by FramedCurveFrenet or FramedCurveCentroid. The FramedCurve defines a curve and an orthonormal frame, where the tangent vector is aligned with the curve tangent. The frame can be rotated by an angle alpha with respect to the Frenet or centroid reference frames. The FramedCurve class evaluates geometric properties of the frame, such as the torsion and binormal curvature. This class is also used to set up a multi-filament coil using the CurveFilament class.
The strains are evaluated in the StrainOpt class, which is defined by a FramedCurve and a tape width. The objective functions LPTorsionalStrainPenalty and LPBinormalCurvatureStrainPenalty can be used for optimization with respect to the coil or rotation dofs.
An example of optimization with respect to the rotation dofs is provided in strain_optimization_simple.py

Is this PR ready for another review? If so can you merge master in and resolve merge conflicts?

Is this PR ready for another review? If so can you merge master in and resolve merge conflicts?

@landreman Yes, it is ready. I have merged in master.

Looks good. Since StrainOpt was not intended to be used as an objective function in optimization, the Opt in the name might be confusing to users. So I took the liberty of renaming it to CoilStrain, and adding to the docs to clarify. Hope that's ok. Feel free to merge whenever you're ready.

Thanks, Matt!


cmd/compile: builtin print treat arguments differently from normal functions

What version of Go are you using (go version)?

$ go version
go version go1.17.2 linux/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?

package main

import "fmt"

const X = '\x61' // 'a'
const Y = 0x62
const A = Y - X

func main() {
	fmt.Printf("%T \n", A << 32) // constant<PHONE_NUMBER> overflows rune
	println(A << 32) // okay
}

[Edit]: A simpler one:
package main

import "fmt"

const A = '\x01'

func main() {
	fmt.Println(A << 32) // constant<PHONE_NUMBER> overflows rune
	println(A << 32) // okay
}

What did you expect to see?
Same behavior.
What did you see instead?
Different behavior.
(Maybe this is not a bug, but it should be mentioned somewhere in docs.)
[Edit]: this has been broken since Go toolchain 1.14.

It is okay before Go 1.14.

@ianlancetaylor Yeah, the code you pointed out is for the old typchecker, at tip, we're using types2 instead. That's why I commented above that we should re-open this issue for more discussion.

Per #49216 we will accept the correct behavior and document it in the release notes. If this ends up breaking a lot of code, we can re-open.


Dev: Web - Error when Request Money after assigning a task for SAME user

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Assign task for user B
After the task is created, send Request Money to user B (either from FAB or from the report)

Expected Result:
Request Money should be sent successfully to user B
Actual Result:
Error when Request Money after assigning a task to user B
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.31-2
Reproducible in staging?: n/a
Reproducible in production?: n/a
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/5ebf3b18-401e-47db-b045-628d2caccebf
Expensify/Expensify Issue URL:
Issue reported by: @tranvantoan-qn
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1687723077547579
View all open jobs on GitHub

BE return error.

I can reproduce. Not sure what expected behavior here is though. Discussing here.

Confirmed in Slack that It is expected that requesting money does not work right now. Further, there is already an existing issue  where we plan to make it so the request money button does not appear in tasks (even after reloading).
That will solve the bug in this issue as well. Based on that, we're going to close this one out, as no action is needed!


custom connections view problems

open issues in Github tracker: https://github.com/dbeaver/dbeaver/issues
no open one, but found one closed that is not really fixed and it is the same thing
https://github.com/dbeaver/dbeaver/issues/13565
System information:
windows 10 x64
Dbeaver <IP_ADDRESS>
Connection specification:
Any
Describe the problem you're observing:
Just happened to me but I also found out that the options menu for the Global setting (Default settings for new connections)
behaves differently but the problem is in the same two checkboxes:
Datasource settings:
Right click connection->Edit Connection->General
in the Navigator view "Customize" options menu
"Show all tables in a single list" is grayed out,
can NOT select it no matter what other checkboxes are ticked/unticked.
Global settings:
Window->Preferences->Connections
connections tab (Default settings for new connections) then "Show all tables in a single list" is available but "Hide Folders" is dimmed/grayed. If I tick and untick the "show all tables in single list" checkbox then "Hide Folders" option becomes undimmed/available and "show all tables" gets dimmed, but then even if you untick or tick "hide folders" you can no longer make the disabled option available, you have to cancel and changes and open again.

Thanks for the bug report @aranapatona

I'd like to try on this issue. And I wonder if certain options conflict with others or they can all be chosen at the same time.


[NEW ICON] ROS, ROS2, SolidWork

Skill Name
ROS, ROS2, SolidWork
Why?
SolidWork for 3D design and Drawing
ROS and ROS2 are the robot operating system for developing robot.
Reference Image
No response

Added ROS in f100fddcf1ce9893e3c38be204a0adcfccfd18aa
Probably not going to add solidworks