ApplicationRegion parameter does not work with output from ARM

Describe the bug
I have a compute resource in Azure, lets say an AKS cluster. Doesn't really matter which one since the problem is the same across multiple services:
In my deployment pipeline I run a CLI command to retrieve the region, for instance:
az aks show --resource-group my-rg -n mycluster --query "location" --output tsv
This outputs something like northeurope or eastus2. This seems consistent across many Azure resources.
Now, I want to use that output to feed into an app setting which I then want to use to configure my Cosmos DB client with ApplicationRegion:
CosmosClientBuilder clientBuilder = new CosmosClientBuilder(SysConfig.CosmosEndpointUri, SysConfig.CosmosPrimaryKey);
var builderOptions = clientBuilder.WithApplicationRegion(Environment.GetEnvironmentVariable("AZURE_REGION"));

Turns out, you can't. Because the SDK wants North Europe or East US 2. I don't see any way how to programmatically split this up
To Reproduce
See above.
Expected behavior
The SDK should accept standard naming input for Azure regions like northeurope.
Actual behavior
See above.

Environment summary
SDK Version: 3.14.0

@sebader Locations have a Name and a DisplayName property, you should be able to use the DisplayName property without issues.

@ealsur where do you mean? The CLI output only has one field. and WithApplicationRegion() expects only one string which seems to be the DisplayName

Azure CLI modules differ from resource type to resource type, so maybe the aks implementation is different, but ARM locations have Name, DisplayName and other attributes.
If you run az account list-locations, it will list all regions available in that cloud, for example:
{
    "displayName": "East US",
    "id": "/subscriptions/81cb9df9-8726-4ad7-a878-8385e15c6c4e/locations/eastus",
    "metadata": {
      "geographyGroup": "US",
      "latitude": "37.3719",
      "longitude": "-79.8164",
      "pairedRegion": [
        {
          "id": "/subscriptions/81cb9df9-8726-4ad7-a878-8385e15c6c4e/locations/westus",
          "name": "westus",
          "subscriptionId": null
        }
      ],
      "physicalLocation": "Virginia",
      "regionCategory": "Recommended",
      "regionType": "Physical"
    },
    "name": "eastus",
    "regionalDisplayName": "(US) East US",
    "subscriptionId": null
  },

That is the ARM location information. And the displayName property matches what is expected on the SDK input.

Well, I have worked around it by now using a different way, but for future reference, you can make it like this from AKS...
az account list-locations --query "[?name == '$(az aks show -g my-rg -n my-aks --query "location" -o tsv)'].displayName" -o tsv

sample ==> East US
Do I really like that? TBH, no. That's why I would like to have it as a feature inside the SDK to ideally take either as input: name or displayName

@sebader I agree, and personally I thought all Azure services used the same Location information, after all, it's ARM data. The fact that aks is just returning the name is unexpected.
Supporting the name and displayName sounds valid as a feature thought.

It actually seems like every service/RP is doing it differently. Probably not even ARM doing it
Just a few examples:

@sebader the beautiful land of Resource Provider independent implementations

so I guess the feature should actually be "support name or displayName in all-lowercase" :D

@ealsur do you see any reason why we shouldn't add a normalization logic to the locations names to prevent this issue?

Nope, assuming all regions can be normalized the same way.
When listing the Azure regions in the public cloud, there are cases where a "remove spaces, lowercase" does not apply, for example:
{
    "displayName": "United Kingdom",
    "id": "/subscriptions/81cb9df9-8726-4ad7-a878-8385e15c6c4e/locations/uk",
    "metadata": {
      "geographyGroup": null,
      "latitude": null,
      "longitude": null,
      "pairedRegion": null,
      "physicalLocation": null,
      "regionCategory": "Other",
      "regionType": "Logical"
    },
    "name": "uk",
    "regionalDisplayName": "United Kingdom",
    "subscriptionId": null
  },

We can see the Display Name here is United Kingdom but the Name is uk. Which is different than, for example:
{
    "displayName": "United States",
    "id": "/subscriptions/81cb9df9-8726-4ad7-a878-8385e15c6c4e/locations/unitedstates",
    "metadata": {
      "geographyGroup": null,
      "latitude": null,
      "longitude": null,
      "pairedRegion": null,
      "physicalLocation": null,
      "regionCategory": "Other",
      "regionType": "Logical"
    },
    "name": "unitedstates",
    "regionalDisplayName": "United States",
    "subscriptionId": null
  },

Where Display Name is United States and name is unitedstates.
I don't see how we could apply the same algorithm.

I wonder if this is still necessary if we added the feature to get the region from the Azure VM API. I was really hoping that just doing lower case + removing space would fix it.

Me too, but sadly the inconsistency kills any possible algorithm

@ealsur your example is a "regionType": "Logical", I think you can disregard those. Looking at the output I would say only "regionType": "Physical" is important as those are actual regions

Great point, didn't see the difference on the regionType!

@ealsur any updates on this one? This is still a major hurdle in many automation scenarios

Should we expect this to be fixed? It is going to be two-year anniversary of this request soon. Does it really take that long to hard code a table or use equality comparer on the existing one that will do .ToLowerInvariant().Replace(" ", "")?

RegionName type is string which can be extended to cover normalized names.
So the API will support other normalized names and no extension method is necessary.
@jcocchi, @ealsur , @FabianMeiswinkel thoughts?

@kirankumarkolli either that, or just update the dictionary in the RegionProximityUtil to use custom equality comparer that normalizes the names. I vote for the later, as we don't usually like to modify user inputs in property setters.

Agreed upon name normalization

Spaces
Case-incensitive

If ApplicationRegion/PreferredRegions are supporting other normalized regions what are scenarios for Region.TryParse?


fix(core): respecting the order of declaration in TestBed #6928

closes #6928

v14.12.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


hotfix: broken imports

Something broke the imports, somehow... These fixes might speed up things enough to keep the process running.

Closing as this wasn't the solution to this problem.


Logs fill up with automatic test message exceptions

Ubuntu 18.04.
Home Assistant 0.112.2
Sia component 0.3.2
Describe the bug
SIA works excellent with my Ajax Hub Plus alarm. No smoke or moisture sensors. I can read the state of the alarm in HA just fine. However, the system logs fill up with the below line, or a variation thereof.
Please let me know if I can give more information. There doesn't seem to be a full python stack trace, only the "'NoneType' object has no attribute 'bus'."
Debug log
WARNING (MainThread) [pysiaalarm.aio.sia_server] Last event: Content:  _04:24:10,07-14-2020, Zone: 0, Code: RP, Message: 0000, Concerns: Unused, Type: Automatic Test, Description: Automatic communication test report, Account: AAA, Receiver: , Prefix: L0, Timestamp: 2020-07-14 04:24:10, Length: 0075, Sequence: , CRC: , Calc CRC: , Message type: *SIA-DCS, Encrypted Content: , Full Message: "., gave error in user function: 'NoneType' object has no attribute 'bus'.

That is quite strange, this error means that the function in the component that handles the state updates (user function) gives an error, but there is no explicit call for a bus attribute in the user function, so I'm a bit at a loss where this comes from. If you find anything else let me know, I will see if I can find something.

Still an issue in version 0.3.3.
Also I cannot remove the warning with:
logger:
  default: warn
  logs:
    custom_components.sia: error

Please check with v0.3.4!

Same with 0.3.4 unfortunately :-/

Just noticed this myself, but not at home so can't fix it now, will look at it tomorrow

Noticed the same issue in mine with logs filling up - it seemed to be caused by me disabling the smoke and moisture sensors (as I don't have those). When I enabled them again the errors stopped. YMMV but worth a try.

FWIW, I have also disabled smoke and moisture detectors.

just created v0.3.5 release that should address this for now, am working on better understanding where this came from.

Just release v0.3.6, please give it a try and let me know if this solves things!


Update dependencies and outdated packages

update-modules because those were not maintained.

for quality, security, and predictability


Docs/update core guides

Pull Request: Update documentation for agents, about, feedback, and custom sandbox guide
Description
This pull request includes comprehensive updates to four key documentation files in the OpenDevin project. The changes aim to improve readability, structure, and overall user experience for both contributors and users.
Changes Summary

agents.md:

Improved overall structure and readability
Enhanced formatting for better visual hierarchy
Fixed bolding of "actions" and "code" in CodeAct Agent description for emphasis

about.md:

Restructured content for better flow and readability
Updated formatting and styling for consistency
Simplified language for clearer understanding

feedback.md:

Improved organization of content sections
Enhanced clarity on data use and privacy policies
Added more detailed explanations for data sharing and deletion processes

custom_sandbox_guide.md:

Restructured content for easier following of step-by-step instructions
Improved code block formatting for better readability
Added more detailed explanations for technical processes

Files Changed

docs/agents.md
docs/about.md
docs/feedback.md
docs/custom_sandbox_guide.md

Checklist

[x] Documentation has been updated
[ ] Changes have been tested locally
[ ] Code follows the project's coding standards
[ ] All tests pass (if applicable)

Hi @PierrunoYT. Thanks for your contribution. Unfortunately pointing the docs to another AI and opening a pull request without really checking or understanding what has been changed is not the ideal way to contribute. If you are really keen on contributing, you can use OpenDevin for various tasks and:

Open a well described issue so they can be fixed which helps OpenDevin become better
Click the thumbs down or thumbs up to give feedback on how the agent performed the tasks

I'm going to close this for the reasons @mamoodi mentioned above, but I do want to note that we value your community contributions @PierrunoYT !


Reacting to Light and Dark theme

What would be the best way to use Appthemebinding in MauiReactor

The calculator sample app show a possible implementation:
https://github.com/adospace/reactorui-maui/blob/main/samples/Calculator/Pages/MainPage.cs
protected override void OnMounted()
{
    if (MauiControls.Application.Current != null)
    {
        MauiControls.Application.Current.RequestedThemeChanged += (sender, args) => Invalidate();
    }

    base.OnMounted();
}

also the ChateoApp:
https://github.com/adospace/mauireactor-samples/blob/main/ChateoApp/Chateo/Pages/SettingsPage.cs
essentially, you should create a class that returns your brushes/colors/etc based on the current Theme, attach it to the RequestedThemeChanged event on the OnMounted of the root component (page), and invalidate it.

BTW if you attach RequestedThemeChanged in a component that is not the root be aware to detach it in the Unmount() override

Wow that was fast! Thanks.

if you use XAML styles, take a look at the WeatherTwentyOne
https://github.com/adospace/reactorui-maui/tree/main/samples/MauiReactor.WeatherTwentyOne


Newline char stripped from EOF on deployed files

There's an Ansible bug in lookup which means files deployed using this mechanism have rtrim applied, and so are missing the Newline char 0x0a
2018-10-09 09:52 LS27172:~ kenny$ diff -y -W 190 <(hexdump -C from-ansible-role-ssl-certificate.crt) <(hexdump -C from-original-file.crt)  | tail -n 3
00001a00  2d 45 4e 44 20 43 45 52  54 49 46 49 43 41 54 45  |-END CERTIFICATE|                  00001a00  2d 45 4e 44 20 43 45 52  54 49 46 49 43 41 54 45  |-END CERTIFICATE|
00001a10  2d 2d 2d 2d 2d                                    |-----|                           | 00001a10  2d 2d 2d 2d 2d 0a                                 |-----.|
00001a15                                                                                | 00001a16
2018-10-09 09:52 LS27172:~ kenny$

Does this break anything?

Since lookup is applied outside this role, it's not an issue for this role, other than in the README.


Release of version <IP_ADDRESS> of xDnsServer

Releasing version <IP_ADDRESS>

This change is

Looks like some xActiveDirectory commit messages have been added by accident. Not sure there's a lot that can be done now..?!


Fix UnboundLocalError on decode_command_line_args on Python 3

On Python 3, decode_command_line_args gives an error:
Traceback (most recent call last):
  ...
  File ".env3/lib/python3.5/site-packages/eight/__init__.py", line 113, in decode_command_line_args
    return sys.argv
UnboundLocalError: local variable 'sys' referenced before assignment

This PR fixes it by importing sys outside the if.

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is 0%.

@@         Coverage Diff          @@
##           master    #8   +/-   ##
====================================
  Coverage      80%   80%
====================================
  Files           2     2
  Lines         140   140
====================================
  Hits          112   112
  Misses         28    28

Impacted Files
Coverage Δ

eight/__init__.py
81.9% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for catching the bug.

Released in v0.4.2.


UglifyJS: warningsFilter does not take effect if sourceMap option is false

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
options.warningsFilter for the UglifyJS plugin only takes effect if options.sourceMap is truthy.
If the current behavior is a bug, please provide the steps to reproduce.
Add a warningsFilter that silences warnings, but set sourceMap to false.
https://gist.github.com/christopherthielen/5b317e5ef8910ade34694786bf13f703
What is the expected behavior?
It seems like warningsFilter should take effect regardless of the sourceMap option.
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
└─┬<EMAIL_ADDRESS>

The code that uses warningsFilter is only applied inside the block starting with:
if(options.sourceMap) {
...
}

Send a PR

Was able to reproduce with<EMAIL_ADDRESS>Gonna take a stab at fixing this.

I noticed the current version of webpack is using v0.4.6 of uglifyjs-webpack-plugin but the latest master is on v1.0.0-beta.2.
I presume v1.0.0 of uglifyjs-webpack-plugin has some break changes so it might be difficult to fix this without context around the breaking changes associated with v1.0.0-beta.2 and how to set it up with local dev copy of webpack.
Any suggestions?

@webpack-bot move to webpack-contrib/uglifyjs-webpack-plugin


Add extraArgsList to helm cart

This commit introduces an extra helm value, extraArgsList,
which permits passing extra arguments to the cilium-agent, and allows
for duplication of arguments.
An new value was chosen to avoid breaking existing configurations/documentation.
Fixes: #14944
Please ensure your pull request adheres to the following guidelines:

[X] For first time contributors, read Submitting a pull request
[X] All code is covered by unit and/or runtime tests where feasible.
[X] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[X] All commits are signed off. See the section Developer’s Certificate of Origin
[X] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Fixes: #issue-number
Add extraArgsList to helm chart to permit duplicate arguments to be passed to cilium-agent

This will be ready once I figure out how to untangle the mess I've made with signed-off and merges :)

This one might help, I think you can just squash all commits into one :-)
https://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git

All the google-fu / stackoverflow posts I found couldn't untangle the mess I made! I've made a new, clean PR (https://github.com/cilium/cilium/pull/14947), and am closing this one in shame...


Add support for Nest.land - a package manager for Deno

Hey guys, first I would like to know what is the next step here? This thread has been dead for a while.
Also, I'm hijacking the PR to ask if it would be possible to add a webhook to deno.land? https://deno.land/x#add

@sebastienfilion I'm here to help too. Let me know what steps I can take to help ramda with nest.land integration.


Add diesel to the benchmark suite

This commit adds diesel to the continuosly run benchmark suite. Diesel
heavily relies internally on macro generated code. Additionally there
are lots of complicated trait releations used as part of their API.
Therefore this benchmark will be quite sensitive to:

Performance related changes in the macro expanding code
Performance related changes while resolving trait bounds

CC #7950
cc @lnicola

changelog internal add diesel to the benchmark suite.

Looks like this should take about 5-6 minutes on CI. It might be pretty painful, but let's tentatively merge it.
I've checked ripgrep and webrender and they don't seem to have changed in the meanwhile.
bors r+

Great, this broke the metrics dashboard :-).


Project Settings are ordered differently depending on whether the Advanced Settings toggle is enabled

Godot version
v4.0.dev.custom_build [072777df6]
System information
Windows 10, Vulkan, Nvidia Quadro K2000 461.92
Issue description
In project settings, when switching Advanced Settings on and off, you can notice that the ordering of the settings is different between the two.. Layer names for example, are practically at the bottom with the AS on (Below rendering), but with AS off, Layer names are higher up (above rendering)
Steps to reproduce
Go to Project settings and switch between Advanced Settings On and Off, and observe how the category placements are different
Minimal reproduction project
No response

This is most likely caused by the fact that property definitions order is mixed. E.g. you have setting group 1 advanced, setting group 2, setting group 1, when you remove advanced, the order changes, because properties are added in definition order.


Atomic Cli failed to generate properties for the classes with large number of required properties in ontology

Details are customer-specific and in private repos and messages.

I resolved this with Alex in private. It turned out he forgot to call initOntologies()


maybe- use skim to fuzzy search instead of relying on fzf?

https://github.com/lotabout/skim
Might have some good properties (no longer need to use fzf, might be faster).

you could declare a global variable $FUZZY_FINDER and use it instead of calling sk or fzf.


How to mock or automate the external app opening(Gmail, Dailer ) in flutter integration test

Hi Team,
I'm using flutter integration driver to write flutter integration tests.
I have scenario like, clicking on Mail Us Now button as show in below image

and clicking on it, will opens the default gmail app in the device with email_id, email_subject, email_body as shown in below image

I want to handle this use case in flutter integration test via may be mocking that app opening and return true always as like we do the intent mocking in espresso.!
I tried to search in google docs and googling, but didn't find anything related to this use case handling in flutter integration test.!
Can anybody help me in this regard?
Here is the main.dart file code:
CLICK ME

import 'package:flutter/material.dart';
import 'package:url_launcher/url_launcher.dart';

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

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

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // Try running your application with "flutter run". You'll see the
        // application has a blue toolbar. Then, without quitting the app, try
        // changing the primarySwatch below to Colors.green and then invoke
        // "hot reload" (press "r" in the console where you ran "flutter run",
        // or simply save your changes to "hot reload" in a Flutter IDE).
        // Notice that the counter didn't reset back to zero; the application
        // is not restarted.
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Invoke "debug painting" (press "p" in the console, choose the
          // "Toggle Debug Paint" action from the Flutter Inspector in Android
          // Studio, or the "Toggle Debug Paint" command in Visual Studio Code)
          // to see the wireframe for each widget.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headline4,
            ),
            ElevatedButton(
                onPressed: ()async{
                  String email = Uri.encodeComponent("<email_id>");
                  String subject = Uri.encodeComponent("Sample email sender");
                  String body = Uri.encodeComponent("Sample email sender from flutter app");
                  Uri mail = Uri.parse("mailto:$email?subject=$subject&body=$body");
                  if (await launchUrl(mail)) {
                    //email app opened
                  }else{
                    //email app is not opened
                  }
                },
                child: const Text("Mail Us Now")
            ),
            ElevatedButton(
                onPressed: ()async{
                  Uri phoneno = Uri.parse('tel:+918898734567');
                  if (await launchUrl(phoneno)) {
                    //dialer opened
                  }else{
                    //dailer is not opened
                  }
                },
                child: const Text("Call Us Now")
            )
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

@jagadeeshpurplle
Flutter driver / integration_test doesn't seem to support interacting with native elements. See this issue for your reference : https://github.com/flutter/flutter/issues/34345
Also, check if any third party plugin offers that feature or not as mentioned here : https://github.com/flutter/flutter/issues/34345#issuecomment-1265346652. Note that, it is not an endorsement of that package but just to see if it helps to achieve your use case.
With that said, I am going ahead and closing this. You may ask this question on StackOverflow for solutions.


Avoid flakiness in data channel for empty streams.

As empty stream is used as end-of-stream marker, don't ever send
it as the data itself.
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @lukecwik

Coverage increased (+0.003%) to 70.501% when pulling ce8e87c028eb90c18319126fec1d6f1d54af8ab4 on robertwb:data-plane into 86e1fab69f296247624d7799f151ff25c3b7243a on apache:master.


Could not resolve class "res://addons/iomjp94.yafsm/src/StackPlayer.gd"

After several weeks working perfectly, suddenly this morning this error appeared at (0,0) in StateMachinePlayer.gd and doesn't allow the addon to work. All the scripts from the addon node have turned their icons in the scenes from blue which were before, to white. The StateMachine panel shows no grid nor charts.
I have tried reinstalling, deleting all the YAFSM nodes and trying to recreate them, to no avail. Even restoring older backups which worked perfectly, don't work any more.
I am using Godot 4.1.3 from Steam.
Any ideas?

The error is the one at the title obviously: "Could not resolve class "res://addons/iomjp94.yafsm/src/StackPlayer.gd"" in StateMachinePlayer.gd at (0,0).

Now I noticed 3 warnings that appeared at load time:

Even in another computer with a completely clean Godot + addon installment, it doesn't work anymore... really strange.

Seems that with Godot 4.1.3 the following lines start erroring out:

The solution is to go through all the addon's scripts and remove such lines.

I'm also having the same issue. The project stopped working for me when I moved to a new machine, with this same error.

FWIW, I was able to get the project running in Godot 4.1.1 - it breaks in 4.1.3

Once you remove all super.init calls that refer internal Godot classes, it works

That simply doesn't work for me at all, I have methodically been through every script and removed every super_init call.
I still get errors such as in StateMachinePlayer.gd I get Error at (0, 0): Could not resolve class "res://addons/imjp94.yafsm/src/StackPlayer.gd".

@amazingoomoo : I created a pull request, does it work with this?

@amazingoomoo : I created a pull request, does it work with this?

Hi I'm really sorry but I don't know what those words mean. I really don't understand GitHub and at this point I'm too afraid to ask! I don't know what a "commit" or a "pull request" is or even what a "git" is or what "github" is I just have no idea about any of this.

Yes I understand, i'm also struggling with github...
What you can do is go to my fork (which contains the fix) here : https://github.com/mikerodd/gd-YAFSM-4.1.3
download it as a zip file and replace the content of the addon corresponding directory (res://addons/imjp94.yafsm) in your project with this content
that should work too


Fix issue with proxy Scraper

For some reason the scraper works with this package version of cheerio and not with the 1.0.0 version.
I don't know why but on my dev environment I had this version installed and it worked and in production it didn't. That's how I found the problem.

The latest version 1.0.0 works fine. reverting back introduces other issues. I will look into the proxy issue quite soon.

In proxy.ts it fails in line const $ = cheerio.load(content);
Idk why mabye you have a idea. Ofc it will be a better version to work with 1.0.0


Helper script to update copyright notices.

Problem
It's a nuisance to check and update copyright notices manually.
Summary of Changes
This script extends copyright messages of the form
Copyright (c) 20xx[-20xx] Project CHIP Authors

to include the current year.
Typical use:
$ scripts/helpers/update_copyright.sh file1 ... # update specific files
$ scripts/helpers/update_copyright.sh -a        # update uncommited files
$ scripts/helpers/update_copyright.sh -c HEAD   # update files just committed

Deferred pending possible simplification of copyright notices.

crupdate.zip
See attached for a script that I wrote that I've carried around for 23 years that does this; no need to change or "simplify" headers.

Regrets, I didn't see that this was a PR with a script already; I'd misread that this was an open issue with a call for a script.


Error running Prisma/Express example - Client authentication failed: Invalid redirect_uri

I'm trying to run prisma_express example to generate an Authentication Code with PKCE.
When I run this request: http://localhost:3000/authorize?response_type=code&client_id=b1a639b9-09b6-427c-a6c5-55f10746949c&redirect_uri=http://example.com&state=abc123&code_challenge=hA3IxucyJC0BsZH9zdYvGeK0ck2dC-seLBn20l18Iws&code_challenge_method=S256, I receive {"status":401,"message":"Client authentication failed: Invalid redirect_uri"} in the browser.
The only entry I have in my Postgres DB is:

Here's a quick n dirty repo sample I'm using: https://github.com/bdcorps/ts-oauth2-server-example-problem
Is there something I'm missing here?

Hey, can you provide a better example or install instructions for the sample? It just looks like this repository with the history rewritten, I am not sure where I am supposed to be looking. The current examples have some hard-coded values,  one day I will get around to adding fully working examples with clients.


install not working on fedora 30

Please note that we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[x] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[ ] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[ ] ran brew update and can still reproduce the problem?
[ ] ran brew doctor, fixed all issues and can still reproduce the problem?
[ ] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)
install linuxbrew on fedora 30 using
sh -c "$(curl -fsSL https://raw.githubusercontent.com/Linuxbrew/install/master/install.sh)"

What happened (include command output)

Command output

ruby: error while loading shared libraries: libcrypt.so.1: cannot open shared object file: No such file or directory

What you expected to happen
succefuly install linuxbrew
Step-by-step reproduction instructions (by running brew commands)

how i fix it
sudo dnf install libxcrypt-compat.x86_64

Glad you resolved it. CC @Homebrew/linux

portable-ruby requires the library libcrypt.so.1. On Fedora 30, that file is provided by libxcrypt-compat. I've opened PR https://github.com/Homebrew/brew/pull/6128 to fix the documentation to address this issue.


Publish 7.6-20221001013050+0000

Publish 7.6-20221001013050+0000

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


[WIP] SIMD-accelerated IDCT

This commit is a first step towards SIMD-accelerated IDCT.
It optimizes only the first part of the IDCT, and no fallback has been implemented for non-nightly compilers.
Benchmark results:
Benchmarking decode a 512x512 JPEG: Warming up for 3.0000 s
Warning: Unable to complete 100 samples in 5.0s. You may wish to increase target time to 11.3s or reduce sample count to 40
decode a 512x512 JPEG   time:   [2.2417 ms 2.2633 ms 2.2846 ms]
                        change: [-26.302% -23.320% -20.299%] (p = 0.00 < 0.05)
                        Performance has improved.

Closes: https://github.com/image-rs/jpeg-decoder/issues/79
Please merge https://github.com/image-rs/jpeg-decoder/pull/144 first

@fintelia , @HeroicKatora : This PR is not finalized, but I'd love to get your feedback early.
@lilith : This may be of some interest to you.

I like this direction! We need to figure out a way to handle cases were packed_simd is unavailable, but perhaps just making it an optional feature + duplicating the dequantize_and_idct_block_* functions would be enough?

Yes, we can do that. But ideally, I would have wanted even non-simd targets to benefit from this change, by using simulated simd, which is still faster than loops. Unfortunately, the API of ssimd seems to be out of date, and the crate seems to be unmaintained...

I have started to work on adapting ssimd to the latest packed-simd interface, so that we can use it with stable compilers here. The amount of work to do is moderate, since we use only a few simd types and operators.
https://github.com/lovasoa/ssimd

Final benchmark
Before this PR
$ RUSTFLAGS="-C llvm-args=--vectorize-slp -C target-cpu=native" cargo bench 'decode a 512x512 JPEG'

decode a 512x512 JPEG   time:   [2.6583 ms 2.6783 ms 2.6984 ms]

After this PR
all optimizations, packed_simd feature disabled
$ RUSTFLAGS="-C llvm-args=--vectorize-slp -C target-cpu=native" cargo bench 'decode a 512x512 JPEG'

decode a 512x512 JPEG   time:   [3.5087 ms 3.5182 ms 3.5281 ms]

~ 30% worse
all optimizations, packed_simd feature enabled
$ RUSTFLAGS="-C llvm-args=--vectorize-slp -C target-cpu=native" cargo +nightly bench --features="packed_simd" 'decode a 512x512 JPEG'

decode a 512x512 JPEG   time:   [2.0040 ms 2.0151 ms 2.0262 ms]

~ 20% better
So contrarily to what I thought, the benchmarks didn't improve, they worsened for the non-simd case.

Is there something you still wanted to do here? It's still marked as a draft and the comment alludes a partial regression.

Hi @HeroicKatora !
Yes, there is a performance regression for the non-simd version, so this PR should probably not be merged as-is. I contacted @lilith about this PR, but she told me her company doesn't need this anymore and thus isn't ready to fund this anymore, so I lost a little bit of the motivation needed to work on this.
I wanted to switch from packed_simd to simdeez, but then I found an inconsistency in simdeez, for which I proposed a PR. It took some time, but it is now merged, so there is no more blocker. Are you interested in working on this, @HeroicKatora ?

I see, and can totally feel how that would turn off some motivation. Yeah, the total of 20% performance is signifcant. Switching to a stable crate with 1.0 would be positive as well and if it helps avoid the regression even better!
I'm looking for what project to focus on next after apng is done, so maybe. The alternative is that I focus on [image-canvas] instead, still shepherding this if you think it has the potential.

Hi all!
I was wondering if there'd be some interest in picking this up, and, if not, if anybody here would mind me trying my hand at some handcrafted SIMD optimizations :) (probably with intrinsics for x86/sse and arm/neon, as I think portable SIMD is not stabilized yet)

Yes, please pick this up !


cli: add check for latest version of cli and control plane

As part of conduit check command, warn the user if they are
running an outdated version of the cli client or the control
plane components.
Fixes #314
Signed-off-by: Andy Hume<EMAIL_ADDRESS>

This is an initial version to get feedback on the approach.
The obvious question I have is about how I should approach requesting to version.json. In the spirit of getting something that works, I’ve just used net/http and defined a new struct to hold the response, but I suspect I should be mapping HTTP/JSON to grpc. So any steer on how you would make an ad-hoc request like this in Conduit would be good.

Thanks for the feedback. I've actioned most the stylistic points.

Do you think that this checker could also verify the version of the Conduit control planet installed in the cluster?

Not entirely sure I follow. versionStatusCheck.SelfCheck() returns a slice with cliIsUpToDate and controlPlaneIsUpToDate, so I think what you're asking is already done here?

Thanks - addressed those three points in latest push.
If you're keen and happy to merge, then please go ahead. I do have the beginnings of tests here which I'm happy to follow up with but it's not likely to be for another 24h or so. Let me know if there's any other small tweaks required here though, and I'll try and get them in.

@ahume great. I'll wait for the test then!
If you have any questions please don't hesitate to ask here or in the #conduit channel in the Linkerd slack, I'm always hanging out there

Tests added, mostly along happy paths. Happy to expand coverage where you want.


Add gif for character walking effect

It would be nice if the character walks as oppose to slide to the left and right. I would like to implement this if you think its a good idea @EdwardDunn . good game

@Aseged Yeah thats a great idea! Go ahead.

@EdwardDunn  I'm sorry I been quiet, I have been preoccupied lately. I will close the open issues assigned to me soon.

@Aseged Closed due to inactivity


chatterbot corpse install issue

carnage@Carnage:~/Documents/ChatterbotCent$  pip3 install chatterbot-corpus
Collecting chatterbot-corpus
Using cached https://files.pythonhosted.org/packages/ed/19/f8b41daf36fe4b0f43e283a820362ffdb2c1128600ab4ee187e84262fa4d/chatterbot_corpus-1.2.0-py2.py3-none-any.whl
Collecting PyYAML<4.0,>=3.12 (from chatterbot-corpus)
chatterbot 1.0.5 has requirement pyyaml<5.2,>=5.1, but you'll have pyyaml 3.13 which is incompatible.
Installing collected packages: PyYAML, chatterbot-corpus
Found existing installation: PyYAML 5.1.2
Uninstalling PyYAML-5.1.2:
Successfully uninstalled PyYAML-5.1.2
Successfully installed PyYAML-3.13 chatterbot-corpus-1.2.0

carnage@Carnage:~/Documents/ChatterbotCent$ python3 centaurbot1.py
Traceback (most recent call last):
File "centaurbot1.py", line 1, in
from chatterbot.trainers import ListTrainer

I was having a similar issue where this resulted in an incomplete install for chatterbot. I checked the chatterbot corpus repo for similar issues afterwards and it seems doing a clone of the current chatterbot-corpus repo instead of using a pip match somewhat resolved this issue.
I am still having an issue with my python file not being able to find the chatterbot module, but now when I try "python -m chatterbot --version", I get a version number.
https://github.com/gunthercox/chatterbot-corpus/issues/127

python -m chatterbot --version
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 163, in _run_module_as_main
mod_name, _Error)
File "/usr/lib/python2.7/runpy.py", line 111, in _get_module_details
import(mod_name)  # Do not catch exceptions initializing package
File
"/home/carnage/.local/lib/python2.7/site-packages/chatterbot/init.py",
line 4, in
from .chatterbot import ChatBot
File
"/home/carnage/.local/lib/python2.7/site-packages/chatterbot/chatterbot.py",
line 2, in
from chatterbot.storage import StorageAdapter
ImportError: No module named storage
i get this
when i run that command . I cant import no module in terminal only on
.pycharm
On Mon, Nov 4, 2019 at 2:19 AM Brandon Blank<EMAIL_ADDRESS>wrote:

I was having a similar issue where this resulted in an incomplete install
for chatterbot. I checked the chatterbot corpus repo for similar issues
afterwards and it seems doing a clone of the current chatterbot-corpus repo
instead of using a pip match somewhat resolved this issue.
I am still having an issue with my python file not being able to find the
chatterbot module, but now when I try "python -m chatterbot --version", I
get a version number.
gunthercox/chatterbot-corpus#127
https://github.com/gunthercox/chatterbot-corpus/issues/127
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gunthercox/ChatterBot/issues/1852?email_source=notifications&email_token=ANM2TOUUWCOJ3AIDIFVXJLLQR7LKNA5CNFSM4JIIV7E2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC6PZHI#issuecomment-549256349,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANM2TORTWPG233TFMJUIA2DQR7LKNANCNFSM4JIIV7EQ
.

--
Thank You
Elijio G Briceno :.

python3 -m chatterbot --version
/usr/local/bin/python3: No module named chatterbot i read somewhere that
chatter bot only work with python 3 and greater.
On Mon, Nov 4, 2019 at 3:15 PM Eli Ceno<EMAIL_ADDRESS>wrote:

python -m chatterbot --version
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 163, in _run_module_as_main
mod_name, _Error)
File "/usr/lib/python2.7/runpy.py", line 111, in _get_module_details
import(mod_name)  # Do not catch exceptions initializing package
File
"/home/carnage/.local/lib/python2.7/site-packages/chatterbot/init.py",
line 4, in
from .chatterbot import ChatBot
File
"/home/carnage/.local/lib/python2.7/site-packages/chatterbot/chatterbot.py",
line 2, in
from chatterbot.storage import StorageAdapter
ImportError: No module named storage
i get this
when i run that command . I cant import no module in terminal only on
.pycharm
On Mon, Nov 4, 2019 at 2:19 AM Brandon Blank<EMAIL_ADDRESS>wrote:

I was having a similar issue where this resulted in an incomplete install
for chatterbot. I checked the chatterbot corpus repo for similar issues
afterwards and it seems doing a clone of the current chatterbot-corpus repo
instead of using a pip match somewhat resolved this issue.
I am still having an issue with my python file not being able to find the
chatterbot module, but now when I try "python -m chatterbot --version", I
get a version number.
gunthercox/chatterbot-corpus#127
https://github.com/gunthercox/chatterbot-corpus/issues/127
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gunthercox/ChatterBot/issues/1852?email_source=notifications&email_token=ANM2TOUUWCOJ3AIDIFVXJLLQR7LKNA5CNFSM4JIIV7E2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC6PZHI#issuecomment-549256349,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANM2TORTWPG233TFMJUIA2DQR7LKNANCNFSM4JIIV7EQ
.

--
Thank You
Elijio G Briceno :.

--
Thank You
Elijio G Briceno :.

That would be correct. From https://github.com/gunthercox/ChatterBot/issues/1163 . You would need to go back to chatterbot 0.8.x to maintain Python 2.7 support.
After my post last night I was able to get it running on Python 3.7.5 with success. After getting a version number to display, the issue was actually resolved. The follow up issue I had was a problem with how I configured my setup. It was throwing an error in the IDE, but when I switched to the prompt for my Anaconda venv instance, it worked fine. I apologize for that since it probably added some confusion. I just started learning how to use Python a day ago.

I just started myself a week ago . Can you give more info about how you got
it to work? i read something about a env and had opened it in a terminal
which left a file  in the folder.
On Mon, Nov 4, 2019 at 4:38 PM Brandon Blank<EMAIL_ADDRESS>wrote:

That would be correct. From #1163
https://github.com/gunthercox/ChatterBot/issues/1163 . You would need
to go back to chatterbot 0.8.x to maintain Python 2.7 support.
After my post last night I was able to get it running on Python 3.7.5 with
success. After getting a version number to display, the issue was actually
resolved. The follow up issue I had was a problem with how I configured my
setup. It was throwing an error in the IDE, but when I switched to the
prompt for my Anaconda venv instance, it worked fine. I apologize for that
since it probably added some confusion. I just started learning how to use
Python a day ago.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gunthercox/ChatterBot/issues/1852?email_source=notifications&email_token=ANM2TOT42QHTAPDIAVBQ26TQSCP4TA5CNFSM4JIIV7E2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDA6XEQ#issuecomment-549579666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANM2TOSSCMDJENSUEFHWFX3QSCP4TANCNFSM4JIIV7EQ
.

--
Thank You
Elijio G Briceno :.

Just a heads up. I am using Windows 10, 64bit. Version 10.0.18362 Build 18362. Stating that just in case.
So I originally started with Python 3.8.0, but found blis to be causing an issue during the installation. So instead I installed Anaconda, and created a venv of Python 3.7.4.
From there I did a normal "pip install chatterbot". Then I upgraded to Python 3.7.5 as I forgot to do that before I installed chatterbot. Following that I tried to install the chatterbot corpus using a regular pip match without success. I got the same error you got. I looked around other issues and found that the corpus is a separate install, and did "python -m chatterbot --version" to confirm the program was successfully installed, but instead I got this error.

Traceback (most recent call last):
File "C:\Users\brand\Anaconda3\envs\py37\lib\runpy.py", line 183, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "C:\Users\brand\Anaconda3\envs\py37\lib\runpy.py", line 142, in _get_module_details
return get_module_details(pkg_main_name, error)
File "C:\Users\brand\Anaconda3\envs\py37\lib\runpy.py", line 109, in get_module_details
import(pkg_name)
File "C:\Users\brand\Anaconda3\envs\py37\lib\site-packages\chatterbot_init.py", line 4, in
from .chatterbot import ChatBot
File "C:\Users\brand\Anaconda3\envs\py37\lib\site-packages\chatterbot\chatterbot.py", line 2, in
from chatterbot.storage import StorageAdapter
File "C:\Users\brand\Anaconda3\envs\py37\lib\site-packages\chatterbot\storage_init.py", line 1, in
from chatterbot.storage.storage_adapter import StorageAdapter
File "C:\Users\brand\Anaconda3\envs\py37\lib\site-packages\chatterbot\storage\storage_adapter.py", line 3, in
from chatterbot.tagging import PosHypernymTagger
File "C:\Users\brand\Anaconda3\envs\py37\lib\site-packages\chatterbot\tagging.py", line 4, in
from chatterbot.tokenizers import get_sentence_tokenizer
File "C:\Users\brand\Anaconda3\envs\py37\lib\site-packages\chatterbot\tokenizers.py", line 4, in
from chatterbot.corpus import load_corpus, list_corpus_files
File "C:\Users\brand\Anaconda3\envs\py37\lib\site-packages\chatterbot\corpus.py", line 5, in
from chatterbot_corpus.corpus import DATA_DIRECTORY
ModuleNotFoundError: No module named 'chatterbot_corpus'

I then did some more searches and found gunthercox/chatterbot-corpus#127 and tried the suggestion in that thread. I used "pip install git+https://github.com/gunthercox/chatterbot-corpus.git#egg=chatterbot-corpus". After that it was successfully installed. To see if the module was still not found, I tried to get chatterbot's version number again, and got the current version: 1.0.5.
Finally I tried running an example file and it worked. I also enable access to a folder via my windows protection history so the files could be created to actually run the chatterbot file.
Sorry if it is kind of a long explanation. Just wanted to cover the steps I did to get it functioning.

@Carnage-Cloud @brandonjblank if your package is not installing properly try once deleting the pip folder as it extract cache file from there and results in incomplete installation once try deleting pip folder from
C:\users\soham\local\appdata\pip

You have to install chatterbot_corpus and not chatterbot-corpus, are differents
pip install chatterbot_corpus
or
pip3 install chatterbot_corpus


V1.3 Update Docs for Filter feature

Description:
This PR adds documentation for the new filter by name and filter by remarks functionality.
Key Changes:

Include UML Diagram for Filter Command
Update Developer Guide for Filter command (Implementation, Example usage, Alternatives considered)

Addresses (but not fix) #139 and #115 for filter command


[FEATURE REQUEST] Add nabar similar to blog page in strength.html

https://akshitagupta15june.github.io/Moksh/Blogs/Strength.html
add same logo as index.html and navbar should be similar as
https://akshitagupta15june.github.io/Moksh/Blogs/Blog.html

I would like to work on this issue. @akshitagupta15june

@akshitagupta15june Please assign this issue to me because I want to work on it

@akshitagupta15june mam can you please assign this to me. It'll be my first contribution in Moksh.

Hello @akshitagupta15june I would like to make the changes you stated above. Please, assign me the above work.
Thank You,
Sayantan Choudhury,
GSSOC'23 Contributor

@akshitagupta15june I would like to work on this issue. I request you to please assign this to me. Thank you ma'am.


[Runtime] Fix generation of .lnk files for static linking.

Use the new SWIFT_SDK_<sdk>_STATIC_LINKING_SUPPORTED and _STATIC_ONLY flags instead of hardcoding support for Linux and WASI.
Also, use the _LIB_SUBDIR variable rather than lowercasing the SDK.
rdar://123504757

Explanation: SwiftPM needs .lnk files when statically linking to determine which libraries it should link by default. These need updating to support the fully static SDK.
Original PR: #71843
Risk: Low. Only affects static linking, and should have the same behaviour as existing build script code for Linux and WASI.
Reviewed by: @MaxDesiatov
Resolves: rdar://123504757
Tests: There are some static linking tests in the repo already; the changes that affect the new static SDK are currently tested manually (by building some code with SwiftPM), but there will be CI at some point soon.

@swift-ci Please test


[MRG] Installation

Job d2l-zh/PR-560/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-560/

Job d2l-zh/PR-560/2 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-560/

Job d2l-zh/PR-560/3 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-560/

Job d2l-zh/PR-560/4 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-560/

Job d2l-zh/PR-560/5 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-560/

Job d2l-zh/PR-560/6 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-560/

Job d2l-zh/PR-560/7 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-560/

Job d2l-zh/PR-560/8 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-560/

Job d2l-zh/PR-560/9 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-560/

Job d2l-zh/PR-560/10 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-560/

Job d2l-zh/PR-560/11 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-560/

Closing due to conflict and branch rebase. You may re-open it later on the rebased branch. Thanks.


Handle throttled files in hosting upload

 Error while syncing (updating) css/coverpage.css
 Error: Request was throttled. Expected available in 1.0 second.

Maybe use https://github.com/jhurliman/node-rate-limiter/ or https://www.npmjs.com/package/request-rate-limiter ? Or some custom lighter solution?


Add makefile rule for running the REPL for spec tests

I noticed this seems to be missing. Our tests are obviously not using it, but I think it's nice to have for debugging and people wanting to try out our REPL.

Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock

I think the following rule only works for files that are literally *repl-spec.k, whereas this works for any -spec.k. It should also not be picked up as a test. The point isn't to run tests but to allow us or other people to easily run any spec in our test suite using the kore-repl.


The link is dead in https://hack.aragon.org/en/help.html

https://github.com/aragon/hack/blob/171edfc81eee122b8200eb75df5808fd6dda4ba1/website/pages/en/help.js#L26

Fixed in #29.


TryFrom<&str> for KeyCode for Deserialization

What problem does this solve or what need does it fill?
This will allow config parsers (ie. serde) to read user-given strings directly into bevy keycodes without having to copy the entire KeyCode enum into the game code.
Describe the solution would you like?
My current solution is to use a macro to map available keycodes into a list of [(&str, KeyCode]), then search on that.
https://github.com/John2143/bdo2/blob/e6a9a11d03db4c187db0ed07f3bdaf1e14d94f24/src/config_read.rs#L94-L120
The above version also provides the equivalent of a impl<const N: usize> TryFrom<[&str; N]> for [KeyCode; N], and  [&str; 1] -> KeyCode, but that would be gated by const-generics.
The solution of TryFrom would allow me to drop some of the serde code and remove the copy of KeyCode that I hold.
Describe the alternative(s) you've considered?
I'm not sure if this function exists already somewhere in the library, but I didn't see anything inside of keyboard.rs. I couldn't find any prior work by searching bevy keycode from string.
An alternative would be to provide a full blown impl Deserialze for KeyCode. This TryFrom is a requisite step that removes the most fragile part of my parsing right now. If the Deserialize impl is wanted, that could be a separate PR.
Additional context
https://github.com/John2143/bdo2/blob/e6a9a11d03db4c187db0ed07f3bdaf1e14d94f24/src/config_read.rs for a full implementation of Deserialize for KeyCode
Separate from #281 because that only adds From<u32> for KeyCode
May also be wanted on Mouse too, and perhaps a &str to Option<Result<KeyCode, Mouse>>

Interesting thought, I see value in it.

The main question is if a TryFrom<&str> makes sense on KeyCode (and Mouse). I can't implement it on my own crate because of the orphan rule + lack of enum definition, so it needs to be added into the bevy definition of KeyCode.
I can submit a PR with the proposed addition if you'd like. It might be clearer than me writing it out.

If added, I think the impl should go through FromStr. It would allow str::parse to be used without importing a trait.

I went to go try an implementation for it, but I found that #281, actually did solve what I was looking for. I thought that had only added the repr attribute.
The key is to add the optional serialize feature, which I couldn't find originally in the bevy docs. Serde basically provides the FromStr with that feature enabled.


Debug mode modifies the ONNX protobuf, changing behavior of forward pass

The behavior of the forward pass may change when debug=True because debug mode has the side effect of modifying the ONNX protobuf. This causes a problem in particular when a model has multiple outputs: the forward pass will only return the first output. The other output nodes get consumed and are not returned.

Is this still the case after your refactoring of the code?


Add a node.js project implementing queues using bull lib and redis

I would like to contribute adding a node.js project involving queues using bull (https://github.com/OptimalBits/bull) and redis

@rheimon1 sure you can proceed.

Can I create a node.js folder and put the project inside it?

@rheimon1 yes you can but keep the name as 'Node-JS Projects' .

I would love to contribute to this open source project!


Enable more tests fixed by sensor order guarantees

As @neykov mentions on #860 we've fixed tests attempted to be fixed in #641 described in https://issues.apache.org/jira/browse/BROOKLYN-525 .
This is the test-only subset of that closed PR.  Curious to see how it fares in jenkins.  Worth retesting this a couple times to make sure the items are fixed.

retest this please
first one aborted for no clear reason (tests included here passed):  https://builds.apache.org/job/brooklyn-server-pull-requests/2685/
2017-10-19 17:43:49,521 INFO  TESTNG PASSED CONFIGURATION: "Surefire test" - @BeforeMethod org.apache.brooklyn.test.framework.LoopOverGroupMembersTestCaseTest.setup() finished in 29 ms
2017-10-19 17:43:49,521 INFO  TESTNG INVOKING: "Surefire test" - org.apache.brooklyn.test.framework.LoopOverGroupMembersTestCaseTest.testNotTargetingGroup()
channel stopped
Adding one-line test results to commit status...
Setting status of bc0b392654624b3e36c411d1e555858a2e1cf1ff to FAILURE with url https://builds.apache.org/job/brooklyn-server-pull-requests/2685/ and message: 'FAILURE
 4586 tests run, 0 skipped, 0 failed.'
Using context: Jenkins: brooklyn-server-pull-request
Finished: ABORTED

retest this please

retest this please

retest this please
Test passed first time, and then failed with an unrelated test-failure the second time (captured below). Kicking it off a third time.
java.lang.AssertionError: failed succeeds-eventually, 21 attempts, 3000ms elapsed: AssertionError: entity=Application[kgg9to9f]; attribute=Sensor: service.state (org.apache.brooklyn.core.entity.lifecycle.Lifecycle) expected [stopped] but found [stopping]
	at org.apache.brooklyn.core.entity.lifecycle.ServiceStateLogicTest.assertAttributeEqualsEventually(ServiceStateLogicTest.java:339)
	at org.apache.brooklyn.core.entity.lifecycle.ServiceStateLogicTest.testStopsNicelyToo(ServiceStateLogicTest.java:143)
Caused by: java.lang.AssertionError: entity=Application[kgg9to9f]; attribute=Sensor: service.state (org.apache.brooklyn.core.entity.lifecycle.Lifecycle) expected [stopped] but found [stopping]
	at org.apache.brooklyn.core.entity.lifecycle.ServiceStateLogicTest.assertAttributeEqualsEventually(ServiceStateLogicTest.java:339)
	at org.apache.brooklyn.core.entity.lifecycle.ServiceStateLogicTest.testStopsNicelyToo(ServiceStateLogicTest.java:143)

need to fix org.apache.brooklyn.core.entity.lifecycle.ServiceStateLogicTest - this is happening a fair bit now on jenkins (never locally).  also org.apache.brooklyn.enricher.stock.UpdatingMapTest.testUpdateMapEmitsEventOnChange.
(adding the words "non-deterministic test failures" so we can search for them again)
passed here though so good to merge


Issues with Carbon Black Pluginv1.0.4

Plugin Setup UI conflicts with recommended value and data check

Fails to complete the setup, even with correct data [verified with postman]

Siva - can you please open a ticket with<EMAIL_ADDRESS>and we'll see what we can do to help you out. This looks like an issue with the API token credentials.
You need to set Create, Read, and Update for the following restful endpoints:
Custom Detections > Org.watchlist
Custom Detections > Org.feeds
FYI The Override reputation does nothing except enable you to create a reputation for a given plugin if one is not present in the raw feed (not sure if your mouse was there to tell us you suspected that or not).

Hi @drwillis07, it turned out to be API access issue. Choose Org.alerts.notes instead of Org.alerts.
Regarding "Override reputation", the tip says to use 0 for default but the webapp complains 0 is not valid value and only accepts 1-10.


Introduce a new ps command that displays containers

This PR, needed by #8, introduce a new ps command that list all the running or exited containers of the application. The main thing is that it also display the containers' DNS.
Here's a screenshot of the current output:

@richardmiller moved the table display logic.

:+1: Looks good to merge to me.


🧱 Steel price is 0

Describe the bug
Steel costs 0 zCoins
To Reproduce
Steps to reproduce the behavior:

Go to the materials mercenary
Try to trade with him

Expected behavior
Steel should have a worthy cost.
Screenshots


24Hrs clock

Please include a 24Hrs clock option as Non-US users never use a 12Hrs/AM-PM clock.

If you set your preference to 24 hour time at https://www.google.com/calendar/render?pli=1#settings-general_11, the time will appear in 24 hour format in this extension as well.


Node selector is not appear if peers-api.json includes "localhost"

@AnthonyLaw can you try to reproduce and report here?
Thanks @44uk :+1:

https://github.com/nemfoundation/nem2-explorer/blob/70b8a4a42b6c5cb3593045117eb543435b67bc76/src/helper.js#L54
Ohh, I found the issue, because in the validURL checker return 'http://localhost.com:3000'  is not valid.
will fix this in next PR

localhost.com is not valid ; but localhost:3000 should work


This branch has no code but now has a README.

i have mailed you a receipt for your 18 months of discord nitro

I have your


Can this project be set under Open Service Broker ?

I'm wondering if this project could be set under the umbrella of the Spring Cloud Open Service Broker because they are very close in what they do. Or am I wrong ?

Many thanks  for explaining! Then I will think how to make it more clear on the "overview". Like changing some wording in the App Broker Overview to highlight the difference between the 2. The wording are quite similar and I didn't catch it at first.
I'll create a PR so you can look what I'm proposing.
I'll close this issue.  Thank you again.


update some doc references [skip ci]

This is a small change, just as I was reading the docs I found some 404 links. This fixes the ones I found.
cc @MikeMcQuaid

Thanks for the PR @arthurnn. This is something I'm working on a bigger change for but it's not time critical because docs.brew.sh is considered the authoritative source.


[MDM-362] Fix log4j warnings printed while deploying "mdm-android-agent" webapp

Please refer the following log snippet for more details.[2015-07-17 17:14:27,847]  INFO {org.wso2.carbon.webapp.mgt.TomcatGenericWebappsDeployer} -  Deployed webapp: StandardEngine[Catalina].StandardHost[localhost].StandardContext[/mdm-android-agent].File[/Users/WSO2USER08/WSO2/git/cdm/test-17072015/pack/wso2mdm-2.0.0-SNAPSHOT/repository/deployment/server/webapps/mdm-android-agent.war]log4j:WARN No appenders could be found for logger(org.springframework.core.io.support.PathMatchingResourcePatternResolver).log4j:WARN Please initialize the log4j system properly.log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.Reference: https://wso2.org/jira/browse/MDM-362

This is no longer happening with "mdm-android-agent" webapp


Option to clear the done wish

Is your feature request related to a problem? Please describe.
Offering a feature where a clear button will be and after clicking that button the done wish will be cleared.
Describe the solution you'd like
A button will clear done wish

Also, I want to work on this feature if you assign me for that. Thanks a lot.

@skmirajbn bhai, have you started working on this? Otherwise, I want to work on this. @atapas


still "violation access"

Rasa version: 1.10.7
Rasa SDK version (if used & relevant): 1.10.7
Rasa X version (if used & relevant): 1.10.7
Python version: 3.7
Operating system (windows, osx, ...): windows
Issue:
In the latest rasa as above, I still get the "violation access" error. I have seen the related issues but they don't help. I also found that when my training data is small like 7KB, it works fine without the error, but when my training data is larger like 642KB, the error appears, the config files are exactly same. And the time the error appear is just when the program is trying to save the model in windows user's temp directory. So I doubt if there is anything to do with temp directory e.g. the model is too large to store in temp directory?
Error (including full traceback):
Part II: train segment classifier
now do training
num training samples: 17267
2020-07-29 12:47:34 INFO     rasa.nlu.model  - Finished training component.
Traceback (most recent call last):
  File "E:\PycharmProjects\rasa-1.10.7\rasa\train.py", line 453, in _train_nlu_async
    persist_nlu_training_data=persist_nlu_training_data,
  File "E:\PycharmProjects\rasa-1.10.7\rasa\train.py", line 482, in _train_nlu_with_validated_data
    persist_nlu_training_data=persist_nlu_training_data,
  File "E:\PycharmProjects\rasa-1.10.7\rasa\nlu\train.py", line 94, in train
    path, persistor, fixed_model_name, persist_nlu_training_data
  File "E:\PycharmProjects\rasa-1.10.7\rasa\nlu\model.py", line 231, in persist
    update = component.persist(file_name, dir_name)
  File "E:\PycharmProjects\rasa-1.10.7\rasa\nlu\extractors\mitie_entity_extractor.py", line 174, in persist
    self.ner.save_to_disk(entity_extractor_file, pure_model=True)
  File "D:\Anaconda3\envs\py37\lib\site-packages\mitie\mitie.py", line 271, in save_to_disk
    if (_f.mitie_save_named_entity_extractor_pure_model(filename, self.__obj) != 0):
OSError: exception: access violation reading 0x0000000000000030

Process finished with exit code 1

I tried to debug it, and it said named_entity_extractor doesn't have attribute __obj
Command or request that led to error:

Content of configuration file (config.yml) (if relevant):

Content of domain file (domain.yml) (if relevant):

Thanks for raising this issue, @Ghostvv will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

@chongtwo ,
This seems to be something specific to your environment.
can you post this question in the forum?
I will close the issue here, but please tag me in the forum issue, and I will try to help you find the cause of this error.

@ArjaanBuijk ,
I have post this question in the forum with topic "still 'violation access'" in Rasa Open Source category, but I don't find the right place to tag you, so I replied here. Hope you can help me solve the problem. Thanks in advance.


docs: Restructures quickstart

This commit restructures and rewrites the Zenith quickstart. It provides a hands-on experience of creating a Daggerized pipeline using modules from the Daggerverse, whilst also walking the user through a few important CLI sub-commands.
NOTE: This is marked as a draft as it uses the new dagger call syntax and should be published only after or with the 0.9.8 release.
Preview: https://deploy-preview-6262--devel-docs-dagger-io.netlify.app/zenith/quickstart

Awaiting your comments @vito @gerhard @helderco @sipsma

The linting failure is related to:

https://github.com/dagger/dagger/pull/6277

Also relevant (from Discord):

Why does make engine:lint complain on modules in docs/versioned_docs/version-zenith/ but not in core/integration/testdata/modules/go/, does anyone know (for example undefined: Optional)? I mean when you haven't dagger mod sync locally.

Thanks for this improvement!
Follow-ups: The quickstart should be multi-language

This is the next improvement planned :)

The linting failure is related to:

Delete leftover module files in zenith docs #6277

Also relevant (from Discord):

Why does make engine:lint complain on modules in docs/versioned_docs/version-zenith/ but not in core/integration/testdata/modules/go/, does anyone know (for example undefined: Optional)? I mean when you haven't dagger mod sync locally.

I don't know the answer to this but maybe @gerhard @matipan @marcosnils will have an idea?

Also relevant (from Discord):

got back there, but ftr: I believe this is because golangci-lint automatically skips anything under testdata by default. golangci-lint run --help shows that --skip-dirs-use-default is set to true by default and results in testdata being skipped (among some others)

Pushed a fix for the linting failure. @vikram-dagger I had to split out build/main-improved.go to a separate dir build-improved/main.go for it to work, but updated the docs too so think it should be good to go.
Agree w/ your suggestion in #6277 @helderco btw, we can automate a better way for all this.

Pushed a commit related to CLI install which I wanted to do for ages. Feel free to change what you disagree with @vikram-dagger.
Will continue reviewing this when I get another time block. Really want to get this merged either today or tomorrow.

Still haven't finished with this today, but it's first item on my list tomorrow.

Turning this into a draft as there will be some changes to the structure and content. Not closing it as there are specific improvements I want to ensure we don't lose, such as the changes to the CLI installation page.

Thanks for this improvement!
Follow-ups: The quickstart should be multi-language

This is now done.

Are we OK to resolve the conflict and merge this @vikram-dagger?

Are we OK to resolve the conflict and merge this @vikram-dagger?

I would like @shykes and others to review it first.

@vikram-dagger: I would like @shykes and others to review it first.

OK, not a problem, will not wait on this to release.

@shykes in your edit, i saw that you used both Dagger Functions and Dagger functions. Do we want to capitalize the word Functions?

@shykes in your edit, i saw that you used both Dagger Functions and Dagger functions. Do we want to capitalize the word Functions?

I think it should be capitalized when prefixed with "Dagger". Otherwise, not capitalized. WDYT?

I think it should be capitalized when prefixed with "Dagger". Otherwise, not capitalized. WDYT?

Yes, we can follow this rule. I guess similar to how we refer to Dagger Engine? What about modules - should we use "Dagger Modules"?


[TW#14794] When an additional task calls void esp_task_wdt_feed() only once it never triggers

When there is only one task in wdt_task_list, and that task only called esp_task_wdt_feed() once, the watchdog triggers but no task name is displayed.
When there is more than one task in wdt_task_list, and an additional task calls  esp_task_wdt_feed() once, no watchdog is triggered.
The reason turned out to be that when a task is added to the list in void esp_task_wdt_feed(), it initializes  newtask->fed_watchdog=true; This is not correct, it should be false. The reason is because it did not participate in the decision to feed.

@TomIowaSoft task watchdog timer has been overhauled, please refer to commit 9d63e1da4a233f8b3043157f58894fb9b2c2668d


[FEA] Reducing manual copy/paste between MONAI / MONAI Deploy / MONAI Label

Is your feature request related to a problem? Please describe.
Currently, manual work (copy/paste code) is required for a MONAI developer to create a MONAI Deploy App. That is, there is no common API between MONAI and MONAI Deploy (outside of importing MONAI utilities/libraries in a MONAI Deploy App). Due to this, an experienced MONAI developer (primarily focused on developing new algorithms/training paradigms) needs to learn the concepts/API in MONAI Deploy and copy/paste code in the appropriate places in a MONAI Deploy Operator and manually ensure that data transfer concepts (e.g. InferContext) in MONAI Deploy are correctly used.
Analogically, in the case of MONAI Label, an experienced MONAI developer must do much the same: familiarize themselves with the concepts of of MONAI Label (e.g InferTask, TrainTask, etc) and copy/paste their code in the appropriate sections to ensure that the MONAI Label App functions as expected.
Describe the solution you'd like
In addition to researchers "freestyling" training scripts using MONAI, they could also optionally use concepts of MONAI Label and Stream such as tasks (InferTask, TrainTask, etc.) and contexts (InputContext, OutputContext, ExecutionContext) to represent the conceptual task the use aims a certain branch of code to accomplish all the while using data transfer types that are portable across MONAI. The aim is ultimately to employ each of the execution branches for the application the user wants to create; this may still require manual adjustment but overall reduce the amount of manual copy/paste and reduce the learning curve across different MONAI products.
Describe alternatives you've considered
Learn each framework in isolation (MONAI, MONAI Label, MONAI Deploy) and copy/paste where appropriate.

Hi @MMelQin @dbericat ,
May I know why you need to copy / paste the code into MONAI Deploy? Can't we just install MONAI v0.7 package and call the transforms or other components?
Thanks in advance.

@Nic-Ma @aihsani
Sorry, that is a statement from Alvin. Maybe it is "copy/paste" in MONAI Label, but in Deploy App SDK logically and programmatically it NOT "cut and paste". I'd defer to Alvin to answer it.
The Deploy App SDK inference operator class has abstract Pre_transforms and Post_transforms methods, and then a model specific app needs to implement. For deploying the example trained models in MONAI repo, I did have to understand the transforms used (dictionary based), and then use those transforms in the Deploy app for the same model, adding SpacingD, not using Handler, tweaking SaveImageD etc. All in all, only the transforms Compose needs to be customized.
So, please see my last comment about self describing model, so that Deploy can generate code based on the metadata instead of referring to the Train app for the transforms used.
I do NOT agree with the statement that MOANI Deploy App SDK does "cut and paste" in general.

I was going to write something short but I've rambled a bit so I hope I'm not totally off base with what everyone else is thinking.
Looking at the MedNIST example notebook I can see the scope for discussion on duplication or copy-pasting. The deploy part of the example does use MONAI transforms in the MedNISTClassifierOperator definition which are similar to those in the training cells since they do the same initial regularization of data. The body of the compute method is pretty much a straight forward implementation of inference with a Pytorch model then stating which category the classifier chose for the given input. This code may be very similar to what is in one's training scripts (eg. the part doing validation inference), but requiring some non-obvious adjustment to work with Deploy's different environment.
So the pathway from training code to deploy code isn't straight forward, but we could introduce definitions wrapping common concepts shared between Core and Deploy if we can nail down the commonality in a way that isn't cumbersome and resolve how that dependency will work. Since Deploy shouldn't hard depend on Core this could take some work. For example having a common definition of doing inference with a network which is compatible with training and deployment, but which doesn't introduce constraints to use in either context. It'll be hard to avoid shoehorning the same concept in both places though.
One idea mentioned earlier here that I think important is a more semantically rich package for a network. As stated a Torchscript bundle isn't enough to use a network correctly, just enough to load the network correctly without code or implementation changes causing error. What we would benefit from is a mechanism to add more metadata to these bundles that covers the format of input and output, in terms of tensor shapes but also values that they should have, what the meaning of the output is such as which classes are being predicted or which tissues segmentation numbers correspond to, plain-language description of what the network is for, timestamp or version information, inference time hyperparameters, and other requirements or features. It's more than just being a self-describing network but including a lot of important scientific information as well as runtime relevant parameters. With this information we can on the deploy side define code, or generate code at runtime, which uses this information to wrap the network in the necessary infrastructure to operate.
For the MedNIST example this would be used to generate the code to preprocess the inputs such that it has a specific size and dimensions with scaled values, and be able to recognize the network as a classifier with a given set of labels and their meanings which can be returned as the output. With this the MedNISTClassifierOperator type wouldn't be necessary as there would be an implicit mechanism that can operate on any correctly augmented Torchscript bundle to do everything that it does. This generated code need not rely on MONAI Core either so we can still avoid that dependency, or on a potentially separate MONAI Transforms package.
Other aspects that are needed for deployment such as handling file IO (loading DICOMs, fixing orientation issues with Nifti, etc.) are common to Core, Deploy, and Label in very similar ways quite often so we may consider where a common solution would live and what it would look like. The example uses a PIL based operator for demonstration but in practice we'd want some unified way of representing image and other data in and out, with Core that's LoadImage to a large degree.
I'm not familiar enough with Labels's code to be certain but I feel that for Label there is a lot of scope to use Deploy more in place of existing code. I imagine duplication of concept between the two is down to being developed at different times by different teams.

I am very much ok to use sdk for monai-label.. and streamline the concepts for writing infer/train definitions/tasks.  but as a user i just have to one concern.  I have to understand extra concept here.. earlier i was using all monai components and defining the workflows, now i have to define additional operators and decide which one to define/use as operator vs core component for monai.. and then define my infer/train definition/workflows.
I am basically referring https://github.com/Project-MONAI/monai-deploy-app-sdk/blob/main/examples/apps/ai_spleen_seg_app/spleen_seg_operator.py
the current apps are for infer.. do we have any examples for training (single, multi gpu etc.. ) as well? or coming soon?  they will be some good reference materials for monai label..
I do agree on some points to have to those conversion utilities as part of core so that we can get more contributors also working on some corner cases related to dicom=>nifti or vice versa.  If I remember, we had discussed/proposed it to core team as well to own/introduce those dicom related utilities.


Credentials Provider returns session and jwt as null

I am attempting to implement authentication using the CredentialsProvider. However, when I access the JWT endpoint, both the session and JWT are returned as null, even though I have defined the callback functions. When I log the output in the console, I am receiving "null." Is there anything in particular that I am missing?
/api/auth/[...].ts
/* eslint-disable no-console */
import CredentialsProvider from '@auth/core/providers/credentials'
import GithubProvider from '@auth/core/providers/github'
import GoogleProvider from '@auth/core/providers/google'
import type { AuthConfig } from '@auth/core/types'
import { PrismaAdapter } from '@auth/prisma-adapter'
import { PrismaClient } from '@prisma/client'
import { NuxtAuthHandler } from '#auth'

const prisma = new PrismaClient()

const runtimeConfig = useRuntimeConfig()

export const authOptions: AuthConfig = {
  adapter: PrismaAdapter(prisma),
  providers: [
    CredentialsProvider({
      id: 'credentials',
      name: 'Credentials',
      credentials: {
        email: {
          label: 'email',
          type: 'text',
        },

        password: {
          label: 'password',
          type: 'password',
        },
      },
      // eslint-disable-next-line @typescript-eslint/ban-ts-comment
      // @ts-expect-error
      async authorize(credentials: { email: string; password: string }) {
        try {
          const user = await $fetch('/api/auth/login', {
            method: 'POST',
            body: {
              email: credentials.email,
              password: credentials.password,
            },
          })
          console.log('Response', user)
          return user
        }
        catch (err) {
          if (err instanceof Error) {
            createError({
              status: 500,
              message: err.message,
            })
          }
        }
      },
    }),
    GithubProvider({
      clientId: runtimeConfig.NUXT_GITHUB_CLIENT_ID,
      clientSecret: runtimeConfig.NUXT_GITHUB_CLIENT_SECRET,
    }),

    GoogleProvider({
      clientId: runtimeConfig.NUXT_GOOGLE_CLIENT_ID,
      clientSecret: runtimeConfig.NUXT_GOOGLE_CLIENT_SECRET,
    }),
  ],
  callbacks: {
    async signIn({ user, account, profile, email, credentials }) {
      console.log('user', user)
      return true
    },
    async jwt({ token, user, session }) {
      console.log('jwt callback', { token, user, session })
      return token
    },

    async session({ session, token, user }) {
      console.log('session callback', { token, user, session })
      return session
    },
  },
  // pages: {
  //   signIn: '/auth',
  // },
  session: { strategy: 'jwt' },
  jwt: {
    secret: runtimeConfig.NUXT_JWT_SECRET,
  },
  secret: '76543345678ytrertyujhfdcvbnbvc',
}

export default NuxtAuthHandler(authOptions, runtimeConfig)

Package.json
{
  "name": "nuxt-app",
  "private": true,
  "scripts": {
    "build": "nuxt build",
    "dev": "nuxt dev",
    "generate": "nuxt generate",
    "preview": "nuxt preview",
    "postinstall": "nuxt prepare"
  },
  "dependencies": {
    "@auth/core": "^0.12.0",
    "@auth/prisma-adapter": "^1.0.1",
    "@hebilicious/authjs-nuxt": "0.3.0-beta.2",
    "@prisma/client": "^5.1.1",
    "@vuelidate/core": "^2.0.3",
    "@vuelidate/validators": "^2.0.3",
    "bcrypt": "^5.1.1",
    "vue-router-better-scroller": "^0.0.0"
  },
  "devDependencies": {
    "@antfu/eslint-config": "^0.40.2",
    "@nuxt/devtools": "latest",
    "@nuxt/image": "1.0.0-rc.1",
    "@nuxthq/ui": "^2.7.0",
    "@nuxtjs/google-fonts": "^3.0.2",
    "@types/node": "^20.5.1",
    "eslint": "^8.47.0",
    "nuxt": "^3.6.5",
    "prisma": "^5.1.1"
  },
  "resolutions": {
    "h3": "1.8.0-rc.3",
    "nitropack": "npm:nitropack-edge@latest"
  }
}

Reproduction
https://stackblitz.com/edit/nuxt-authjs-reproduction

@selemondev Hi, I am not 100% sure what you're trying to achieve, but you don't need to define a callback to get the JWT, you can use getServerToken.
I'm getting a prisma error when running your stackblitz, so I'm not sure what exactly is your problem. I would recommend you start from the credentials playground add a jwt endpoint https://github.com/Hebilicious/authjs-nuxt/blob/main/playgrounds/github/server/api/jwt.ts, then add prisma in step by step to see if there's something wrong with it.

@Hebilicious I think this code is based on the sidebase NuxtHandler example. The Callbacks are used to add custom data to the JWT token, e.g. a role or some special Ids that are needed down the road in the backend code.

    async jwt({ token, user, session }) {
      console.log('jwt callback', { token, user, session })
      return token
    },

export const authOptions: AuthConfig = {
  secret: runtimeConfig.authJs.secret,
  providers: [
    CredentialsProvider({
      credentials: {
        username: { label: "Username", type: "text", placeholder: "admin" },
        password: { label: "Password", type: "password" }
      },
      async authorize(credentials) {
        if (credentials.username === "admin" && credentials.password === "admin")
          return { id: "1", name: "admin", email: "admin", role: "test" }
        return null
      }
    })
  ],
  callbacks: {
    async jwt({ token, user, session }) {
      console.log("jwt callback", { token, user, session })
      return token
    }
  }
}

Then something like that will work.


Fix shutdown executor tsan bug

Fixes #13767
Push needs to check for shutdown earlier. Otherwise the following events occur:

grpc_executor_set_threading(false) is called, which (under a lock) sets shutdown to true. Then (without a lock) calls gpr_cv_destroy(...)
concurrently, executor_push(...) is called, which (after grabbing the lock), calls gpr_cv_signal(...)

****************************************************************

libgrpc.so

     VM SIZE                                      FILE SIZE
 ++++++++++++++ GROWING                        ++++++++++++++
  +0.5%     +16 src/core/lib/iomgr/executor.cc     +16  +0.5%
      +1.2%     +16 executor_push                      +16  +1.2%

 -+-+-+-+-+-+-+ MIXED                          +-+-+-+-+-+-+-
  +0.0%     +16 [None]                             -80  -0.0%

  +0.0%     +32 TOTAL                              -64  -0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE                                      FILE SIZE
 ++++++++++++++ GROWING                        ++++++++++++++
  +0.5%     +16 src/core/lib/iomgr/executor.cc     +16  +0.5%
      +1.2%     +15 executor_push                      +15  +1.2%
      +1.2%      +1 [Unmapped]                          +1  +1.2%

 -+-+-+-+-+-+-+ MIXED                          +-+-+-+-+-+-+-
  +0.0%     +16 [None]                            -160  -0.0%

  +0.0%     +32 TOTAL                             -144  -0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

TSAN: #13122
Windows: #13149
C Dbg: #13132


GetUserId always return null

GetUserId always return null, even with authentication on. My guess is that it is looking for the claim: "http://schemas.xmlsoap.org/ws/2005/05/identity/claims/name" and should be "http://schemas.xmlsoap.org/ws/2005/05/identity/claims/nameidentifier".

It looks like is fixed in #233


feat: [IDP-2678]: Added the workflows quickstart tutorial

Description

Please describe your changes: Added the workflows quickstart tutorial

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[x] Code owner review.
[x] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://662639ce9d87110b620a28d3--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66275d9275565fa612d224f2--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6627780936d4cbbb8599cbda--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66279da03c7584108bc8a530--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66279e290d992d159e26339c--harness-developer.netlify.app


fluxnetworksdata file preventing server from lunching.

Minecraft version 1.16.5
Modpack: Valhelsia 3 version 3.3.2
FluxNetworks version 1.16.5-<IP_ADDRESS>
Happened once after that its been a routine, i stopped the server, came back next day, server freezes at "preparing spawn area 100%" never lunches.
Took me hours to figure to problem. from corrupted chunks to deleting every file by its self. i deleted the whole data folder in world and it worked. then started deleting every single .dat file from data folder until fluxnetworksdata.dat.
Now every time i stop the server i have to delete fluxnetworksdata.dat from the world folder to be able to lunch the server again.
Hope you can help.

So i fixed this problem for anyone in the future. Found out issue causing this was chunk loading feature by flux networks.
It only happens when the server tries to load a chunk that is far from where you left the server.
I just stopped using chunk loading and got a chunk loader mod (Chunk Loaders by SuperMartijn642).


feat(MDC-MIGRATION): Handle selectable MatChip

Feature Description
Legacy MatChip has a selectable attribute, but this has been moved to MatChipOption.
mdc-migration should change mat-chip items containing selectable attribute to mat-chip-option.
Ideally, <mat-chip [selectable]="false"> should just remove the attribute.
Use Case
Migration from Material 14 to 15 using MDC.

What is it currently being changed to for you? As far as I can tell from our unit tests, mat-chip is changed to mat-chip-option by default: https://github.com/angular/components/blob/main/src/material/schematics/ng-generate/mdc-migration/rules/components/chips/chips-template.spec.ts#L46
The only case where that won't happen is if there's an input element connected to the chip list.

My code looks more or less like this, and no change was made to it by migration:
<mat-chip-list>
  <ng-container *ngFor-"let c of criteria | keyvalue">
    <ng-container *ngIf="c.key === 'something'">
      <mat-chip *ngFor="let anItem of c.value" [selectable]="false">anItem</mat-chip>
    </ng-container>
  </ng-container>
</mat-chip-list>

I have another that looks like this that was left alone which may be what you said about inputs:
<mat-chip-list #chipList>
  <mat-chip *ngFor="let item of items">{{item}}</mat-chip>
  <input type="text" matInput [matChipInputFor]="chipList"
</mat-chip-list>

It was changed to mat-chip-grid during the standard 14 to 15 migration, but the mat-chip was left alone throughout.

Thanks for the clarification. The problem is that the schematic wasn't migrating template elements, e.g. the *ngFor in this case.


WARNING: pipecat-ai 0.0.29 does not provide the extra 'deepgram'

I am trying to install pipecat with this in the requirements.txt pipecat-ai[deepgram,openai]
But it returns the error:
WARNING: pipecat-ai 0.0.29 does not provide the extra 'deepgram'

Hi, thank you! The reason is because the current Deepgram TTS service uses HTTP connections so there's no need for any deepgram depdencies. However, PR #222 uses deepgram-sdk and it's added as a dependency.

This is now fixed in 0.0.30. Closing.


Nice Work!

Hello Miguel
and congrats on your beautiful theme! It's among the very few which works correctly with Golang. One thing though: Can we teak it a little? I mean something like to lower the background contrast and make it more dimmed, not so black?
Thank you very much!
Nick Christoforidis

Hey thanks for the feedback! We have a similar request in https://github.com/misolori/min-theme/issues/21 so I'll close this one in favor of that one. Thanks again for trying out the theme!


Containerize project : configure dockerfile and compose files

resolve #740
For user to build this project using docker they need to fulfil the prerequisites:
Prerequistes

Docker Desktop or docker should be installed
WSL - Windows Subsystem for Linux  (only for windows users)
should have X server installed in host machine
if not have then can download from command sudo apt-get install x11-xserver-utils.
to check its correct installation run xhost this should give output as

access control disabled, clients can connect from any host
LOCAL:
SI:localuser:wslg

demo image

References
https://github.com/cypress-io/cypress-documentation/issues/5816
--no-sandbox error

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you for setting this up! Since this is for a dev container, I think the .dockerignore doesn't need to be as restrictive, since the developer may care about the .md, config, and Yarn files. Otherwise it looks good to me!

Could you verify that this complies with the guidelines for the dev containers in the guide I linked (Developing inside a Container)? This one may also be helpful: Create a development container. I think you can skip setting a CMD to start on container run, since this is not necessary for developing inside a container. Also I realize the suggested changes I made left a bunch of new lines, could you clean those up?

Sure !

now I think it is good to go; now, the user can use the VS Code dev container feature for seamless development in the container with pre-install go , eslint and prettier extensions in it.

I don't think you need to build the app at all in the dockerfile, since it's fast enough. Also, because webpack-watch is blocking, the dockerfile never completes. I moved this to the "postStartCommand" in devcontainer.json, but I'm still unable to launch the app, as it's complaining about DBus not being started. I tried moving the service dbus start command to the postStartCommand too, but it's still not working. What host OS are you using to test this? I'm on macOS 14.
I'm concerned that if the devcontainer isn't reliable on all platforms or requires additional hoops to get working, it won't be that much better than just installing the deps directly. Aside from scripthaus, we don't have any crazy dependencies. We're also in the process of a pretty large rearchitecting and we're removing the scripthaus dependency as part of this, so our dependency story will get a lot simpler.

I don't think the host OS is the issue here. It's more likely an issue with the dbus service. I encountered a similar dbus error during my testing but resolved it by running service dbus start on the machine. You can refer to this link for more details. By the way, I'm using Windows as the host OS, so it's odd if the OS is indeed the problem. The rearchitecting plan sounds great! Simplifying the dependencies will definitely streamline the process.

FWIW, I did something similar as this PR - but, my goal was to build waveterm in a container - not run it in the container.
I purposely do not install multiple tool chains, project specific dependencies, etc. on localhost.
FYI, I am running on Fedora 41 and typically use VS Code (although I have been using emacs -nw again lately.
I concur that .devcontainer might be of dubious value, and there are several missing extensions that I typically use for web dev (vscode-tailwindcss, etc.). So I would probably just create my own.
In my case, I chose fedora:41 as the base OCI image because I wanted to be sure the resulting binary would run on my system.
That is not an issue when running in a container. If I was looking for containerization, I would prefer flatpak as an execution env because it is a more natural fit.
I suspect there may be a number of personal preferences out there.
But agree with @esimkowitz - thanks for putting in the effort. I am sure there are devs out there who can benefit.

Closing this PR for now, sorry we couldn't get it merged


Support data catalog concept?

Hi @roll! We will review this issue after the launch of the stable version. Thanks!