Task modules not working in ms teams

Sample information

Sample type: Samples
Sample language: nodejs
Sample name: 54. teams-task-module

Describe the bug
The task module is not opening on interaction with the button inside ms teams.
To Reproduce
Steps to reproduce the behavior:

Cloned the repo
Navigated to folder 54. teams-task-module
Did npm install
Updated .env file and manifest json with app ids
Created a zip and installed the app into ms teams.
Sent a message after installation and bot replied with hero card and adaptive card.
Tried pressing any one of the buttons in either of the above cards and nothing happens in desktop app.
Tried checking in web app for console errors and got 2 error messages. One interacting with the button for the first time got the error TaskModuleLaunchService: launchTaskModuleByInvoke failed: "could not find app definition for app e4ac1f5d-26e0-49e4-ad35-e9db5081035d" and in subsequent interactions this error Error: Extension already rendered, only one extension command request can be processed at a time.

Expected behavior
Task Module modal should work open and functions handleTeamsTaskModuleFetch and handleTeamsTaskModuleSubmit should get fired.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@harkiratsaluja, thank you for this information. I'm attempting a repro and will respond when I know more.

@harkiratsaluja, I was able to repro the issue. And, while a colleague and I tried every which way to get the sample to work or discover a workaround, we were simply unable to do so. Debugging every step has shown that all the appropriate values are being generated, and they match expected values as described in the task module docs regarding the TaskInfo object and Using Task Modules in Bots. I looked at values by debugging the app as well as network requests/responses.
It's worth noting that the code used for generating a task module in the sample was pulled almost line for line from the Teams JS docs/sample demonstrating the same. Lastly, we looked at the bot's Teams manifest and the Azure bot registration and app service. Nothing looked unusual though we did try altering some aspects to be sure, though doing so didn't alter the results.
As a comparison, we also ran the same dotnet sample and ended with the same results. At this point, I believe that either something is blocking the handleTeamsTaskModuleSubmit activity handler or how it functions / handles external resources (URLs, for example). If there was a change, then the docs have not been updated to reflect this.
@clearab, do you have any thoughts on this? This feels like it's an issue in the Teams SDK since this can be repro'd in two SDKs. If you agree, could you get this in front of Teams for their consideration?

I've escalated to Teams internally, seems like there is some issue propagating the app definition appropriately for some reason.

The Teams support group was able to repro this issue, and has raised a bug internally. I will follow up with more details as they are available.

Are there any updates on this, @clearab ?

It has been assigned to an engineer in Teams, no update on timeline for a fix yet.

We are seeing this issue randomly on few of the adaptive cards.

Hi, is there any update on this? This is blocking our release process.

Hi, is there any update on this? This is blocking our release process.

Hi, are there any updates on this? This is now impacting our customers randomly @clearab. It will really help if you can share some information on the same!.

@harkiratsaluja - We will check on the bug status and let you know the updates.

@harkiratsaluja - Issue is resolved. Cloned the repo.
Microsoft Teams Version <IP_ADDRESS>19 (64-bit)-E.
Could you please check it once and let us know?
Ref Doc-https://github.com/microsoft/BotBuilder-Samples/tree/main/samples/javascript_nodejs/54.teams-task-module

@harkiratsaluja -We will inform the internal team ,Let you know once we get any update.

@harkiratsaluja - Currently we don’t have ETA or Update on this thread.
We will update this thread once the bug is fixed.

@harkiratsaluja -Could you please provide teams details in which Ring you have tested?

@sayali-MSFT thank you for the reply. This issue is being faced by one of our customers. If you can share the exact details which are needed from our end, we can communicate them accordingly and get back to you with the same!

@harkiratsaluja -Please share the Microsoft teams version and please let us know testing environment which is Public Ring or developer preview?

@sayali-MSFT the Microsoft teams version is <IP_ADDRESS>61 and this is the public ring application that is available on ms teams at Rattle

@harkiratsaluja - There is latest update for teams. Could you please update and check with latest version? Are all the users on same teams version?
Thanks!

@sayali-MSFT looks like we are not yet able to see the latest version yet for update inside our ms teams. Is this released worldwide?

@harkiratsaluja -Yes .last update date 21-06-2022.

@harkiratsaluja - Ok. We will check it and let you know.

Ran into this issue while investigating https://github.com/microsoft/BotBuilder-Samples/issues/3778
The buttons seem to work, except for Custom Form & YouTube buttons where the HTML pages are not getting rendered.
Listed steps to repro here https://github.com/microsoft/BotBuilder-Samples/issues/3778#issuecomment-1163820067
Investigation so far has yet to yield the root cause of this issue.

@harkiratsaluja - Can you please check on the latest updates today and let us know whether you are still not able to see the latest version.

Ran into this issue while investigating #3778
The buttons seem to work, except for Custom Form & YouTube buttons where the HTML pages are not getting rendered. Listed steps to repro here #3778 (comment)
Investigation so far has yet to yield the root cause of this issue.

@ramfattah - Thanks for reporting this issue.
We will check it from our end and get back to you.

@ramfattah -We are unable to repro the scenario, It's working fine from our end.Could you please check it once with the updated version?
Microsoft Teams Version <IP_ADDRESS>05 (64-bit)-E
Sample-https://github.com/microsoft/BotBuilder-Samples/tree/main/samples/csharp_dotnetcore/54.teams-task-module
https://user-images.githubusercontent.com/93527375/176621184-25443533-5951-4312-a09d-baab786ef7f2.mp4

Thanks for the response @sayali-MSFT.
I've noticed sometimes the HTML pages works and sometimes it doesn't, the result seems to be inconsistent.
Looks like Teams updated today to version <IP_ADDRESS>67 (64-bit). It was last updated on 6/30/2022.
I can give it another try and report back.
Thanks.

Hi @sayali-MSFT @ChetanSharma-msft
Looks like I'm not able to repro this issue anymore.
Using Teams Version <IP_ADDRESS>71 (64-bit), the 54.teams-task-module sample ran successfully without any issues.
https://user-images.githubusercontent.com/38049078/177648822-03de3511-68cf-4122-bcd6-47b1e101b6e4.mp4
Thank you.

@harkiratsaluja - Could you please confirm if your issue has resolved or still looking for any help?

Closing due to inactivity.

@clearab We're facing this issue again, could you please re-open this issue?
Details:
Desktop Teams Versioning: <IP_ADDRESS>9 last updated on 2/10/23
Web Application: Both the desktop and web app freezes MS Teams on interacting the with Task module (buttons)


Form doesnt work with builder blocks and blockPreviews

Package
filament/filament
Package Version
v3.2.113
Laravel Version
v11.26.0
Livewire Version
v3.5.9
PHP Version
8.3.8
Problem description
Two issues:

If preview view has form element, create button doesn't work. Even tho the form is removed from html. (But create another button works)
If we add multiple blocks with forms, second forms are not even removed from html. (pic)

Expected behavior
Work? :D
Steps to reproduce
Clone repo
Composer install
Migrate
Add user
Login /admin
Go to admin/tests/create
Pick block
Reproduction repository (issue will be closed if this is not valid)
https://github.com/InfinityXTech/filament-dynamic-fields-issue/tree/builder-block-issue
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

Actions work if we use action instead of submit:
protected function getCreateFormAction(): Actions\Action
{
    return Actions\Action::make('create')
        ->label(__('filament-panels::resources/pages/create-record.form.actions.create.label'))
        ->action('create') // instead of submit('create')
        ->keyBindings(['mod+s']);
}


Fixed: Minor typo.

Hey guys, appreciate if you can review. Thanks!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@jaliyaudagedara  Thanks


catkin_make error

robot_state_’ was not declared in this scope.
**/home/james/catkin_ws/src/moveit_simple-indigo-devel/moveit_simple/src/moveit_simple.cpp: In member function ‘bool moveit_simple::Robot::isNearSingular(const std::vector&) const’:
/home/james/catkin_ws/src/moveit_simple-indigo-devel/moveit_simple/src/moveit_simple.cpp:802:5: error: ‘robot_state_’ was not declared in this scope
robot_state_->copyJointGroupPositions(joint_group_->getName(), local_joint_point);
^
/home/james/catkin_ws/src/moveit_simple-indigo-devel/moveit_simple/src/moveit_simple.cpp:804:3: error: ‘robot_state_’ was not declared in this scope
robot_state_->setJointGroupPositions(joint_group_, local_joint_point);
^
make[2]: *** [moveit_simple-indigo-devel/moveit_simple/CMakeFiles/moveit_simple.dir/src/moveit_simple.cpp.o] 错误 1
make[1]: *** [moveit_simple-indigo-devel/moveit_simple/CMakeFiles/moveit_simple.dir/all] 错误 2
make: *** [all] 错误 2
Invoking "make -j4 -l4" failed
**

Thanks for pointing this out.  It looks like this broke after the PR from @nsnitish.  CI also has the same error: https://travis-ci.org/shaun-edwards/moveit_simple/jobs/269031867
I will look into this, but if you can suggest a fix, that would be great.

This issue arises due to two PRs( #21 and #23 ) which are actually based on different original branches. It was meant to be modified. I have filed PR #25 which takes care of this.

This has been resolved by #25.  Please re-open if this is not the case.


Fix building with gcc 5

While building racon I noticed this:
src/graphmap/region_selection.cc:177:56: error: ‘floor’ is not a member of ‘std’
float min_allowed_bin_value = std::max(2.0f, (float) std::floor(parameters->min_bin_percent * max_bin_value));
Looks like include  is missing somewhere.

Hi,
Thank you for the report!
I now added the include for cmath.
Btw. You can build racon without running make tools. That one is useful for validating the tests.
Best regards,
Ivan.


pcre2-py: init at 0.4.0

Release: https://github.com/grtetrault/pcre2.py/releases/tag/v0.4.0

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Please set dontUseCmakeConfigure = true.

When using dontUseCmakeConfigure = true error message.
writing manifest file 'src/pcre2.egg-info/SOURCES.txt'
ERROR Missing dependencies:
cmake

@SuperSandro2000 a little bit of clean up needed or a better way to way to output results.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4336


Allow the session length middleware's redis key namespace to be configurable

At the moment, the session length middleware uses the message's transport_name to namespace the address-timestamp key-value pairs in redis. When we put the session length middleware on load balanced transports, the redis key used for start-of-session inbound messages is the actual transport name, not the exposed transport name rewritten by the load balancer (since the middleware sees the messages before the load balancer does).
One way around this is to allow the middleware's namespace to be configurable, and default to the message's transport_name.

Ready for review.

:+1: when Travis is happy.
The build failures looked unrelated so I restarted the two builds that failed.


Delta (B.1.617.2) Lineage VCF Request

Request for Delta (B.1.617.2) Lineage VCF
This lineage is described at CoVariants, outbreak.info and PHE Genomics variant definitions.

Delta (B.1.617.2)
The VCF for Delta (B.1.617.2) is attached.


Curriculum update needed on custom_properties.md

The Odin's file, custom_properties.md is updated. Please update the Kampus' file, checkout file here custom_properties.md
Latest commits:

Add Kevin Powell vid link and lint (#27175) (additions: 12, deletions: 12) on Jan 23 2024, 24:20 UTC

New commits have been made to the Odin's file. Please update the Kampus' file.
Latest commits:

Merge branch 'main' into custom-properties-fix-inaccurate-statement (additions: 1, deletions: 1) on Jan 28 2024, 23:02 UTC
fix: change "Learning Outcomes" to "Lesson overview" (additions: 8, deletions: 3) on Jan 16 2024, 06:55 UTC
fix: markdown linting (additions: 10, deletions: 10) on Jan 16 2024, 06:47 UTC
fix: inaccurate statement (additions: 1, deletions: 1) on Jan 16 2024, 06:47 UTC

New commits have been made to the Odin's file. Please update the Kampus' file.
Latest commits:

Update intermediate_html_css/intermediate_css_concepts/custom_properties.md (additions: 1, deletions: 1) on Feb 10 2024, 07:27 UTC
Remove words intermediate html css (additions: 3, deletions: 3) on Feb 10 2024, 07:27 UTC

New commits have been made to the Odin's file. Please update the Kampus' file.
Latest commits:

Merge branch 'main' into patch-25 (additions: 3, deletions: 3) on Feb 11 2024, 21:45 UTC
Update custom_properties.md (additions: 5, deletions: 5) on Feb 9 2024, 21:17 UTC

New commits have been made to the Odin's file. Please update the Kampus' file.
Latest commits:

feat: add assignment item (#27520) (additions: 3, deletions: 2) on Mar 2 2024, 15:26 UTC

New commits have been made to the Odin's file. Please update the Kampus' file.
Latest commits:

Set default content (#28173) (additions: 10, deletions: 2) on Jun 14 2024, 16:59 UTC


The CLI output should indicate what version of core is being installed if specified with --cv

When using --cv to indicate a specific version of core, the CLI should display what version is being installed to avoid any confusion.
flogo create --cv master -f myapp.json
Creating Flogo App: MyApp
installing core

Im confused by this..  isn't this redundant?

I guess not just specifically when you specify the version, just thinking it would be nice to display the version of core that is being installed.

I guess definitely makes sense for "latest"


Use wrapper for mongo client to protect poolboy

Use wrapper for mongo client to protect poolboy if the mongodb database
is not available.
I think we'd need to do the same for the postgres and maybe other drivers, but I thought I'd get the approach reviewed first. The wrapper I wrote I tried to keep general with the hope we can use it for other cases as well.
This is a first step toward fixing #475

Oh, I still need to figure out how the surrounding code handles the {error, not_connected} which can now be returned instead of a result...

@dergraf ready for another round of review...

We should do this for the clients that need it - I'll try and get this done this week.

[ ] mongodb
[ ] mysql
[ ] redis
[ ] postgres
[ ] memcached

Wrapped the postgresql driver. When postgres isn't there we get a log of crash logs which isn't pretty, but we can't really work around that as that happens within the driver.

I'll rebase after review approval @dergraf


Typo in overscanSlideBefore affecting TypeScript

In the virtualize component, the overscanSlideBefore property has a typo... in the latest NPM package, it's mis-typed as Slied
    return (
      <VirtualizeSwipeableViews
      index={this.getIndex()}
      onChangeIndex={(newIndex:any) => this.props.onRelativeIndexChanged(newIndex - start)}
      slideRenderer={this.slideRenderer}
      overscanSlideAfter={1}
      overscanSliedBefore={1}

It seems to only affect typescript... the types seem to think the property is misspelled, but using the misspelled version doesn't work (throws a console error and property doesn't get applied).
Workaround is to use any like follows...
    var additionalProps:any = {
      overscanSlideBefore: 1
    };

    return (
      <VirtualizeSwipeableViews
      index={this.getIndex()}
      onChangeIndex={(newIndex:any) => this.props.onRelativeIndexChanged(newIndex - start)}
      slideRenderer={this.slideRenderer}
      overscanSlideAfter={1}
      {...additionalProps}
      onWheel={this.onWheel}
      onScroll={this.onScroll}
      className={this.props.classes.swipeableViews}/>
    );

The type definitions aren't in this library, they're in DefinitelyTyped, where this issue was already fixed.


override_config decorator doesn't work for pytest classes

Describe the problem
When using the constance.test.pytest.override_config on a test class, pytest simply starts ignoring that test class without any errors.
Steps to reproduce
Write a test like this:
@override_config(SAMPLE_SETTING=10)
class TestSample:
    def test_sample(self):
        assert True is True

And run pytest on the file. It responded like this:
collecting ... collected 0 items

System configuration

Django version: 4.1.9
Python version: 3.11
Django-Constance version: 2.9.1

Looks like importing constance.test.pytest.override_config causes a problem, doesn't seem it's meant to be public api. If you import constance.test.override_config it works. Thus, there are two options:
from constance.test import override_config

@override_config(SAMPLE_SETTING=10)
class TestSample:
    def test_sample(self):
        assert True is True

Would give you Exception: Only subclasses of Django SimpleTestCase can be decorated with override_config (which is also in line with django's override_settings behaviour when decorating classes, see override_settings source
Alternatively:
from constance import config
from constance.test import override_config

@override_config(STRING_VALUE="foo")
def test_sample():
    assert config.STRING_VALUE == "bar"

Gives AssertionError: assert 'foo' == 'bar'


spike: partial workspace

spike of partial workspace migration

Test and dev dx is nicer than before as there is no build step, so changes are reflected instantly in the upstream.
bootstrap will generate types and workspaces will handle symlinking and hoisting so we can remove deps already in root package.json
I've also tried insomnia-url but plugins depend on it which creates some runtime errors I haven't found a workaround for yet.
insomnia-common seems like a good candidate too since plugins aren't aware of it.
doesn't play nice with existing lerna bootstrap :cry:  https://github.com/lerna/lerna/issues/1318

TODO

[ ] figure out how plugins can import insomnia-url as esm modules
[ ] figure out how to install old and new workspaces at the same time

not as straight forward as i'd hoped


Cell

How do I get columns based on XY coordinates

Will reogrid support this method  ?
GetPosByPoint(Point p)

No, currently there are no public methods that provide this feature. Since the cell's position can change due to scrolling, zooming, and freezing, coordinate translation is a complex process that can only be performed within ReoGrid.


jest (babel) fails if using ES6 imports

Environment
devDependencies:
"@haul-bundler/babel-preset-react-native": "^0.13.2",
"@haul-bundler/cli": "^0.13.1",
<EMAIL_ADDRESS>"^0.13.0",
"jest": "^24.9.0",

Node v12.4.0
Description
After changing babel.config.js preset @haul-bundler/babel-preset-react-native jest tests don't support import syntax anymore. If switching back to metro-react-native-babel-preset jest works again.
Trying to figure out what setting is triggering this, but not expert in babel configuration.
Getting errors such as
    import fetch, { Headers } from 'node-fetch';
           ^^^^^

    SyntaxError: Unexpected identifier

      at ScriptTransformer._transformAndBuildScript (node_modules/@jest/transform/build/ScriptTransformer.js:537:17)
      at ScriptTransformer.transform (node_modules/@jest/transform/build/ScriptTransformer.js:579:25)

@jounii
Add this to your Babel config:
overrides: [
  {
    test: /__tests__/,
    plugins: [['@babel/plugin-transform-modules-commonjs', { allowTopLevelThis: true }]]
  }
]

Thanks.  Your example pointed to right direction, however it was not enough because the jest is running and parsing all sort of files. Only common here was that it is running inside node.
So not sure if it is correct way, but this seems to work for now at least:
const isNode = () => typeof process === 'object'

    overrides: [
        {
            test: isNode,
            plugins: [
                ['@babel/plugin-transform-modules-commonjs', { allowTopLevelThis: true }]
            ]
        }
    ]

@jounii Looks a bit hacky, you can alternatively try this:
env: {
  test: {
    plugins: [['@babel/plugin-transform-modules-commonjs', { allowTopLevelThis: true }]]
  }
}

Thanks, that looks better solution and works too. Not familiar with babel config that well, but makes sense now how it works.


⚠ Bofrost.de has degraded performance

In dc613be, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1593 ms

Resolved: Bofrost.de performance has improved in b1f31a4 after 8 minutes.


[feature request] mongodb in default-provisioners

It would be great to have mongodb supported by default in the default-provisioners.
It could be directly used in there https://github.com/Humanitec-DemoOrg/aks-store-demo.

@mathieu-benoit what are the outputs we need here? just a connection string?


Github security warning for hoek

Bug Report or Feature Request (mark with an x)
- [X] bug report -> please search issues before submitting

Versions
Angular CLI: 6.1.1
Node: 10.7.0
OS: linux x64
Angular: 6.1.0
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
Package                           Version
@angular-devkit/architect         0.6.8
@angular-devkit/build-angular     0.6.8
@angular-devkit/build-optimizer   0.6.8
@angular-devkit/core              0.6.8
@angular-devkit/schematics        0.7.1
@angular/cdk                      6.4.1
@angular/cli                      6.1.1
@angular/material                 6.4.1
@ngtools/webpack                  6.0.8
@schematics/angular               0.7.1
@schematics/update                0.7.1
rxjs                              6.2.2
typescript                        2.7.2
webpack                           4.8.3
Repro steps

ng new my-app
push my-app to github

You can use yarn why to see why we have hoek

yarn why hoek

=> Found<EMAIL_ADDRESS>info Reasons this module exists
   - "@angular-devkit#build-angular#node-sass#node-gyp#request#hawk" depends on it

The log given by the failure

This links to: https://nvd.nist.gov/vuln/detail/CVE-2018-3728
Desired functionality
Remove or bump dependency on node-sass to remove dependency on reported vulnerability in hoek.

To fully rectify the issue for all users, this PR (https://github.com/nodejs/node-gyp/pull/1471) is required for node-gyp followed by a release and version bump in node-sass and finally a version bump on the CLIs end.

@clydin how about we fix this by moving to the new canonical sass compiler which is npmjs.com/sass

Sounds good.  Would we need to target 7.0 due to the potential for behavior differences?  This PR for the webpack loader would also be blocking (https://github.com/webpack-contrib/sass-loader/pull/573) unless we wanted to make our own.

This seems to fix it:
npm install<EMAIL_ADDRESS>


WiFi password entry fails on Safari 9.0.3

Ad-hoc Wifi based provisioning is replaced by BLE based bluenet daemon.


feat: docstrings - change from single quote to double quote

Is this a bug report or feature request? (choose one): feat

Python Version (output of python --version):
Development or Deployment Environment?:
Release Tag or Master:
Expected Behavior:
we need """ docstrings
What Actually Happened:
we are using ''' docstrings
How to Reproduce the Issue (as minimally as possible):
Any Additional Comments:

I can do this quickly after #62 is merged


Doesn't work with ES modules

Hello 👋🏻
As I can see #124 was implemented but I have some problems with it. When I try to import nexus-prisma it still tries to import cjs module so I get errors trying to run my app.
Any ideas? type is set to 'module' and nexus-prisma is imported using "import" keyword

EDIT: If I change import to
import nexusPrisma from 'nexus-prisma';

const {FuelStation} = nexusPrisma

then I get the following error:
(node:56355) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
/Volumes/Workspace/pets/cheapoil/core/node_modules/nexus-prisma/dist-esm/entrypoints/main.js:1
import { inspect } from 'util';
^^^^^^

SyntaxError: Cannot use import statement outside a module

Originally posted by @MikeYermolayev in https://github.com/prisma/nexus-prisma/discussions/141

If you share a repro repo it would help a lot.

@jasonkuhrt here you go https://github.com/MikeYermolayev/nexus-prisma-es6-repro
If iI run node src/index.js I receive the above mentioned issue

Looks like the problem is in wrong build for esm. Imports inside esm should contain extensions but as I can see they don't.
I'll try to look into this during this week (Nov 17- Nov 21)

Small update:
I can confirm that things start to work either when I change files extensions in dist-esm to .mjs or when I change "type" in package.json to "module" and set .js extension in imports.
So I suppose there are two possible solutions:

Use babel to generate dist-* folders with .cjs and .mjs extensions like in this article https://medium.com/dazn-tech/publishing-npm-packages-as-native-es-modules-41ffbc0a9dea
Change module type to ESM

@jasonkuhrt I can work on the solution but I'd like to know which of them is more preferable )

@MikeYermolayev can we change module type to ESM while still supporting CJS?
A "clean" solution here might be to wait for TS to support outputing ESM compliant files and supporting .exports package json spec.
TS supporting won't only mean less work for us here but a sign that the community is maturing toward that direction.
I'm not keen on taking on bespoke complexity for this if its a tiny minority group of users.
But I'm open to discussing more at this point.

The easiest way to fix this would be to just rename the file extension for the ESM build to .mjs.
AFAIK, this is not supported by tsc, but you could switch to https://github.com/evanw/esbuild, which would also improve build times. I have an example repo for a package that supports both CJS and ESM: https://github.com/p4sca1/cron-schedule
Alternatively, a simple build script (or npm package), which renames the file extension after building might be sufficient.

@P4sca1 I'll give esbuild and that repo a look. Is it truly a 100% drop in replacement? Would you be up for a PR?

Well, as I understand it's even doable with tsc alone. You will need to compile package with two different tsconfigs and fix type property in package.json in both dist folders. And the code should be written in esm manner.
But I like more the approach @P4sca1 suggested. Also tsup tool might be worth trying. Seems pretty easy in usage and it's powered by esbuild. example: https://antfu.me/posts/publish-esm-and-cjs#tsup

and fix type property in package.json in both dist folders
I don't want a solution that relies on multiple emitted package.json fields.

I am OK to work with Node 12+ package .exports. It should be enough.

Also tsup tool might be worth trying.

I looked briefly. It's a bundler. Why would we want to bundle ahead of time. To me bundling is a last step that should be handled by applications before shipping to production in toolchains.

Am accepting PRs for this.
Personally waiting for TS to support package exports. Considering it was in beta in 4.5 maybe it won't be to long for it to land... 4.6? We'll see...

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@next-major dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

ES modules is still not work. To use  @apollo/server , i have to change my type to module.
When i run my project, some error shown:
import { User } from 'nexus-prisma';
         ^^^^
SyntaxError: Named export 'User' not found. The requested module 'nexus-prisma' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from 'nexus-prisma';
const { User } = pkg;

And i make this change and restart, some error throw:
node_modules\.pnpm\nexus-prisma@1.0.4_63udwhd7sp4du2rzhags75opna\node_modules\nexus-prisma\dist-esm\entrypoints\main.js:1
import { inspect } from 'util';
^^^^^^

SyntaxError: Cannot use import statement outside a module

https://github.com/graphql-nexus/nexus-prisma/discussions/141

The transpiled esm files have .js extensions instead of .mjs. This breaks builds because .js is process as CJS while .mjs is processed as ESM. The output files should name them with the .mjs suffix.


Allow smart guesses without location

I see no reason smart guesses without location shouldn't be possible.
Let's think that through!

A big question I'm having: how many of our timeslots don't have locations? Maybe we can just make sure all of them do have locations? Except if we have no location updates for hours (or location is disabled). That's tricky.

For me, with location enabled, only half of my TS have location.

This will be part of the smart guess rewrite.


System.ArgumentException: 'Value does not fall within the expected range.' when setting TabbedThumbnail and ThumbnailToolBars.

Describe the bug
public Form_Main() {
    InitializeComponent();
    tabbedThumbnailManager = TaskbarManager.Instance.TabbedThumbnail;
    thumbnailToolBarManager = TaskbarManager.Instance.ThumbnailToolBars;
    #region //TabbedThumbnail
    tabbedThumbnail = new TabbedThumbnail(Handle, Handle);
    tabbedThumbnail.Title = this.Text;
    tabbedThumbnail.InvalidatePreview();
    tabbedThumbnail.SetWindowIcon(this.Icon!);
    tabbedThumbnailManager.AddThumbnailPreview(tabbedThumbnail);
    tabbedThumbnailManager.SetActiveTab(tabbedThumbnail);
    tabbedThumbnail.TabbedThumbnailClosed += TabbedThumbnail_TabbedThumbnailClosed;
    tabbedThumbnail.TabbedThumbnailActivated += TabbedThumbnail_TabbedThumbnailActivated;
    #endregion

    #region //ToolBarButton
    ToolBarButton_Previus = new ThumbnailToolBarButton(Icon.FromHandle(Properties.Resources.White_Previus.GetHicon()), "上一首");
    ToolBarButton_PlayPuase = new ThumbnailToolBarButton(Icon.FromHandle(Properties.Resources.Green_Play.GetHicon()), "播放");
    ToolBarButton_Next = new ThumbnailToolBarButton(Icon.FromHandle(Properties.Resources.White_Next.GetHicon()), "下一首");
    ToolBarButton_Previus.Click += ToolBarButton_Previus_Click;
    ToolBarButton_PlayPuase.Click += ToolBarButton_PlayPuase_Click;
    ToolBarButton_Next.Click += ToolBarButton_Next_Click;
    thumbnailToolBarManager.AddButtons(this.Handle, ToolBarButton_Previus, ToolBarButton_PlayPuase, ToolBarButton_Next);
    #endregion
    SetTabbedThumbnail("未開始播放");
    Form_Main_Resize(null, null);
}
[DllImport("User32.dll")]
public static extern int SetForegroundWindow(int hWnd);
private void TabbedThumbnail_TabbedThumbnailActivated(object? sender, TabbedThumbnailEventArgs e) {
    SetForegroundWindow(Handle.ToInt32());
    this.Activate();
    this.WindowState = FormWindowState.Maximized;
}
private void TabbedThumbnail_TabbedThumbnailClosed(object? sender, TabbedThumbnailClosedEventArgs e) {
    this.Close();
}
private void ToolBarButton_Next_Click(object? sender, ThumbnailButtonClickedEventArgs e) {
    NextButton.PerformClick();
}
private void ToolBarButton_PlayPuase_Click(object? sender, ThumbnailButtonClickedEventArgs e) {
    PlayPauseButton.PerformClick();
}
private void ToolBarButton_Previus_Click(object? sender, ThumbnailButtonClickedEventArgs e) {
    PreviusButton.PerformClick();
}
int WindowWidth;
int WindowHeight;
TabbedThumbnail tabbedThumbnail;
TabbedThumbnailManager tabbedThumbnailManager;
ThumbnailToolBarManager thumbnailToolBarManager;
ThumbnailToolBarButton ToolBarButton_Previus;
ThumbnailToolBarButton ToolBarButton_PlayPuase;
ThumbnailToolBarButton ToolBarButton_Next;
private void SetTabbedThumbnail(string text) {
    Bitmap bitmap1 = new Bitmap(WindowWidth, WindowHeight);
    using (Graphics bitmap1graphics = Graphics.FromImage(bitmap1)) {
        bitmap1graphics.Clear(Color.White);
        StringFormat stringFormat = new StringFormat();
        stringFormat.Alignment = StringAlignment.Center;
        stringFormat.LineAlignment = StringAlignment.Center;
        Font font = new Font("Microsoft JhengHei UI", (float)Math.Ceiling((float)(WindowWidth * 0.035)));
        SolidBrush brush = new SolidBrush(Color.Black);
        Rectangle rectangle = new Rectangle(10, 10, bitmap1.Width - 20, bitmap1.Height - 20);
        SolidBrush backgroundBrush = new SolidBrush(Color.White);
        bitmap1graphics.DrawString(text, font, brush, rectangle, stringFormat);
        bitmap1graphics.DrawRectangle(Pens.Black, Rectangle.Round(rectangle));
    }
    Bitmap bitmap2 = new Bitmap(WindowWidth, 100);
    using (Graphics bitmap2graphics = Graphics.FromImage(bitmap2)) {
        bitmap2graphics.Clear(Color.White);
        double p = (double)ProgressTrackBar.Value / (double)ProgressTrackBar.Maximum;
        bitmap2graphics.FillRectangle(Brushes.LimeGreen, 0, 0, (int)(bitmap2.Width * p), bitmap2.Height);
        bitmap2graphics.FillRectangle(Brushes.DarkGray, (int)(bitmap2.Width * p), 0, (int)(bitmap2.Width * (1 - p)), bitmap2.Height);
    }
    Bitmap mergedBitmap = new Bitmap(WindowWidth, WindowHeight + bitmap2.Height);
    using (Graphics mergedgraphics = Graphics.FromImage(mergedBitmap)) {
        mergedgraphics.DrawImage(bitmap1, new Point(0, 0));
        mergedgraphics.DrawImage(bitmap2, new Point(0, bitmap1.Height));
    }
    SetTabbedThumbnail(mergedBitmap);
}
private void SetTabbedThumbnail(Bitmap bitmap) {
    tabbedThumbnail.SetImage(bitmap);
}

this is the code I used to set TabbedThumbnail and ThumbnailToolBars, but this error appears:
System.ArgumentException: 'Value does not fall within the expected range.'
    If not, how can I make it work?
To Reproduce
In the following video.
Imgur: https://i.imgur.com/O4f3SM8.mp4
or
Streamable: https://streamable.com/4b5ins
Expected behavior
Show a custom thumbnail that I set.
Screenshots
Already on above.
Desktop (please complete the following information):

OS: Windows 11 22621.1928

Is anyone here?

Hi @SunnyLeu
Sorry for the delay, I just picked it up, as I've been working on other projects. Would you mind attaching a sample project so that I can test?

Hi @SunnyLeu
Sorry for the delay, I just picked it up, as I've been working on other projects. Would you mind attaching a sample project so that I can test?

Will......
I just tried the exactly same code all again, and it worked well done as before, but I didn't change any code related to Windows-API-CodePack, that's weird, the error just dissappeared......

Hi @SunnyLeu
Sorry for the delay, I just picked it up, as I've been working on other projects. Would you mind attaching a sample project so that I can test?

Will...... I just tried the exactly same code all again, and it worked well done as before, but I didn't change any code related to Windows-API-CodePack, that's weird, the error just dissappeared......

Had a look at the GIF again, and it doesn't show any values in the stack trace?

Hi @SunnyLeu
Sorry for the delay, I just picked it up, as I've been working on other projects. Would you mind attaching a sample project so that I can test?

Will...... I just tried the exactly same code all again, and it worked well done as before, but I didn't change any code related to Windows-API-CodePack, that's weird, the error just dissappeared......

Had a look at the GIF again, and it doesn't show any values in the stack trace?

No, there's no any values.

I have another question related to thumbnails, when I setted a thumbnail and move the cursor on it to try to preview the window, it will always shows the loading circle.

(The cursor position is at the word 'Pitch Perfect')

And after about 15 seconds, the loading circle will stopped and shows the apps icon.

Hi @SunnyLeu
Sorry for the delay, I just picked it up, as I've been working on other projects. Would you mind attaching a sample project so that I can test?

Sample_WinFormsApp1.zip
Here's a sample app.

Hi @SunnyLeu
Sorry for the delay, I just picked it up, as I've been working on other projects. Would you mind attaching a sample project so that I can test?

Sample_WinFormsApp1.zip Here's a sample app.

I can't replicate the bug, is it triggered when you press something? As for the loading circle, I think it's a OS thing, as I've seen it in other apps (not using this library).

@Wagnerp

Hi @SunnyLeu
Sorry for the delay, I just picked it up, as I've been working on other projects. Would you mind attaching a sample project so that I can test?

Will...... I just tried the exactly same code all again, and it worked well done as before, but I didn't change any code related to Windows-API-CodePack, that's weird, the error just dissappeared......

As I mentioned, the bug (System.ArgumentException) was disappeared for no reason, I don't know why, but it seems to be ok now.

@Wagnerp Ok, thanks for your help.


Preferences not saving

Not all preferences are being saved correctly. The recent files list is correct, but changing values in the preferences window does not persist between restarts.

Closing this an not reproducible. Future refactoring should improve the reliability of the settings.


fix(deps): :arrow_up: update dependency @tensorflow/tfjs to v4.13.0

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

@tensorflow/tfjs
4.12.0 -> 4.13.0

Release Notes

tensorflow/tfjs (@tensorflow/tfjs)
v4.13.0
Compare Source

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: package-lock.json
npm WARN cli npm v10.2.3 does not support Node.js v20.2.0. This version of npm supports the following node versions: `^18.17.0 || >=20.5.0`. You can find the latest version at https://nodejs.org/.
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@angular/animations
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>|| ^18.0.0" from<EMAIL_ADDRESS>npm ERR!   node_modules/@angular/material
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   2 more (@angular/platform-browser, @ngneat/spectator)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/@angular/core
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@angular/animations
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR!
npm ERR! For a full report see:
npm ERR! /tmp/renovate/cache/others/npm/_logs/2023-11-10T02_23_00_261Z-eresolve-report.txt

npm ERR!


website background

I don't love the brain logo background.  Seems busy and less awesome than a dark gray.  Too much happening?
I love the logo - just wondering if there is a way to simplify, with all the pictures/tiles happening too.

+1. I suggest we use the same background without the logo - this already
exists on the Google drive, I anticipated it's need :)
On Jun 21, 2015 12:07 AM, "William Gray"<EMAIL_ADDRESS>wrote:

I don't love the brain logo background. Seems busy and less awesome than a
dark gray. Too much happening?
I love the logo - just wondering if there is a way to simplify, with all
the pictures/tiles happening too.
—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/openconnectome.github.io/issues/22.

+1 keep it simpler. greg's background is perfect

@gkiar give me link to the background you like, and i'll try to include it.
alternately, you can try :)

https://drive.google.com/file/d/0B_dAuemRqBb9cFJRZ2hEbXd6MEU/edit?usp=docslist_api
:)
On Jun 21, 2015 12:27 AM, "joshua vogelstein"<EMAIL_ADDRESS>wrote:

@gkiar https://github.com/gkiar give me link to the background you
like, and i'll try to include it.
alternately, you can try :)
—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/openconnectome.github.io/issues/22#issuecomment-113862926
.

hmm.  somehow, i don't seem to be able to get the background to not be the logo.
@j6k4m8 little help?


Trigger a fit operation in response to a terminal-initialized event

Fixes #43

Released in v2.0.2


Added waste management through rl techniques

The project aims to develop a reinforcement learning (RL) agent to optimize waste collection in a simulated environment, minimizing overflow events and improving efficiency.
Environment and State Representation:
The state is represented by four features: Waste Level: Current waste level (0 to 1) Time of Day: A random value representing the time (0 to 24 hours) Weather Condition: A random value (0 to 1) indicating the weather Distance to Collection Point: A random value (0 to 10) representing the distance to the waste collection point.
Action Space:
The agent can choose between two actions: Wait (0): Do not collect waste. Collect Waste (1): Proceed with waste collection.
Reward Structure:
The reward system is designed to encourage efficient waste collection: +10 for timely collection when the waste level exceeds the threshold. -5 for premature collection when the waste level is below the threshold. -1 for each time step to penalize waiting.
Training Process:
The agent is trained over 100 episodes, where each episode simulates a series of time steps (up to 20) where the agent makes decisions based on the current state. The agent learns from experience using a replay memory and updates its policy through Q-learning.
Evaluation Metrics:
Performance is evaluated using: Average Reward per Episode: Measures the effectiveness of the agent's actions. Epsilon Decay: Tracks the exploration rate, indicating how the agent balances exploration vs. exploitation. Overflow Events: Counts occurrences when the waste level exceeds the maximum capacity as per previous updation.
Visualization:
The results are visualized using Matplotlib to plot: Average rewards per episode, showing the agent's learning progression and rewards gained on successfull execution and implementation of a specified condition Epsilon decay over episodes, illustrating the shift from exploration to exploitation. Overflow events per episode, highlighting improvements in waste management techniques

@Niketkumardheeryan

@Niketkumardheeryan Please review as GSSoC is going to end by 2 days

@Panchadip-128  add your dataset used details and add your name in .ipynb file , remove unnecessary files

@Niketkumardheeryan I have added the necessary  dataset informations and keys to be added. Please note that for a RL model like the waste management system using epsillon decay , typically no predefined dataset is required as the agent learns by interacting with a simulated environment.
Thanks and Regards


Closest subtree matching

This adds methods that make it possible to find the closest match of a sequence in a tree that represents different possible configurations of that sequence.

[ ] Conversation objects needs to be implemented

Closes #276

Master, any updates about this pr

@vkosuri Not currently. Before anything in this pull request can be made to work, ChatterBot's current concept of "Sessions" need to be replaced by database-backed "Conversations".

@gunthercox Master, could you please guide me how to do this replacement?

How to do this task

"Sessions" need to be replaced by database-backed "Conversations".

@vkosuri I will write up a guide as soon as I can. It is going to be a somewhat long process to make the switch because there is so much involved.


Fix the resources link

The old link (http://index-of.es/Java/Effective Java.pdf) is not available anymore. what a pity.

 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.

Thanks for reporting this! I've compiled many typos and fixes in a single PR https://github.com/Consensys/teku/pull/8408


permitted-uses and period should be overridable at runtime

Currently the properties quarkus.rate-limiter.buckets."bucket-id".limits[*].permitted-uses and quarkus.rate-limiter.buckets."bucket-id".limits[*].period are fixed at build-time which leads to a new release being necessary for every change.
For this reason, the properties should be overridable at runtime.

🎉


Dataview query not working for folders

Describe the bug
If the source of database data is selected as "query", and Dataview query is given as: FROM "folder_name", it breaks it.
Seeing the markdown file, i believe the error is because there is already a double quote for the whole TABLE query and another double quote within for the folder name is breaking it.
To Reproduce
Steps to reproduce the behavior:

Go to 'Open Database Settings' from the db file's menu
Select Source of database data is selected as "query"
Enter Dataview query as FROM "give_any_folder_name_here"
See error

Expected behavior
The query should take only the files from the folder.
Desktop (please complete the following information):

OS:  macOS BigSur
Version 11.6.7

Hi @pramodam ,
This problem was already fixed with 2.1.0 on https://github.com/RafaelGB/obsidian-db-folder/issues/233 . Currently is not an official version, you can download it as a beta with BRAT plugin.

Thank you for the quick reply. This is an amazing plugin. Very useful. Great work.
On Aug 8, 2022, 11:24 AM +0530, Rafael Gómez Bermejo @.***>, wrote:

Hi @pramodam ,
This problem was already fixed with 2.1.0 on #233 . Currently is not an official version, you can download it as a beta with BRAT plugin.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


using BaGet offline creates infinite loop without backoff for package icons

Describe the bug
When the machine BaGet is running on loses internet connection, loading the index page leads to an infinite loop of errors trying to load the package icons. Ideally, the attempts to load the icons should be tried with exponential backoff, not immediately after it failed.
To Reproduce
Steps to reproduce the behavior:

Set up BaGet, having at least one package present
Disconnect internet (can also be emulated by setting offline mode in browser dev tools after the initial page load, but that's a little bit difficult to time)
Load the page
See error in dev tools:
Request to https://www.nuget.org/Content/gallery/img/default-package-icon-256x256.png fails in endless loop.

Expected behavior
The immediate retry also leads to "flashing" of the icon area. So implementing exponential backoff would be appropriate for any external component.

Thanks for reporting this! This should be fixed by https://github.com/loic-sharma/BaGet/pull/435

This has been fixed by the latest release. Please let me know if you have any additional issues or feedback!

Confirmed to be fixed. Great work, Thanks!


[QUESTION] Why status 422 instead of 400 when invalid (header, query, etc.) parameters in the request?

Description
In the case of an invalid parameter in the request (e.g. header, query, etc.) fastapi returns a 422 status code with the following example body:
{
  "detail": [
    {
      "loc": [
        "header",
        "foo-bar"
      ],
      "msg": "field required",
      "type": "value_error.missing"
    }
  ]
}

Firstly, I was wondering what are the reasons for such choice instead of the most common 400 status code. It seems to me a bit strange, especially in the case where a required header is missing, as I would think this classifies as a syntactic error (from description of 422 versus 400 codes).
As such, is there any easy way to change default return http code on validation errors, while ideally keeping the same return body whose structure is pretty neat, instead of defining a custom schema as kind of described in issue #429 ?

from what I understand, following this answer https://github.com/tiangolo/fastapi/issues/553#issuecomment-533644801 retains the response structure while changing the status code, without defining an elaborate custom schema. It doesn't update the status codes in the automatically generated docs though...

The difference between 422 and 400 becomes a bit subjective.
The rationale is that the error is not about using the protocols and languages incorrectly (HTTP, JSON) nor about doing something otherwise not allowed or incorrect (like performing an operation without enough privileges), but about sending invalid contents, even though using the correct content format (JSON) through the correct channel (valid HTTP).
But I acknowledge that the election between 422 vs others is interpretable and subjective.
The second reason is that Flask-apispec with Marshmallow used 422 too, I thought that was a good idea.
And the other advantage is that you can then separate a whole class of errors, validation errors, that are created automatically by your FastAPI app. And handle them all in a consistent way in your client, without having to do some extra inspections of the contents to decide if it's a validation error or not.
And then you can raise 400 errors in your own code, and know that those 400 errors will be something specific that you raise, and handle them in a different, custom way in your client.
You can override the validation errors including the status code, here are the docs: https://fastapi.tiangolo.com/tutorial/handling-errors/#override-request-validation-exceptions

@tiangolo The only issue with this approach in the docs is that auto-generated documentation doesn't reflect the updated status code. The second issue, perhaps a more pedantic one, is that I also have to define the response structure again, when all I want to do is change the status code.
There's a very straightforward way to change the status code in webargs: https://webargs.readthedocs.io/en/latest/advanced.html#returning-http-400-responses
Is there any way we could have a way to define a custom status code for validation exceptions, and have that reflected in the generated docs, without defining and raising custom exceptions?

Thanks @tiangolo and @yusra-haider for your replies. Sound good.

@yusra-haider yep. Check the docs there, in the same link as above. You can add a custom exception handler without creating a custom exception: https://fastapi.tiangolo.com/tutorial/handling-errors/#use-the-requestvalidationerror-body
Thanks @yiannis-kt for reporting back and closing the issue :+1:

Hey @tiangolo , sorry for reviving this. But I'm looking for a way of changing the status code in the generated docs (I changed the validation_exception_handler so that it returns 400 and the body of the request)
According to your last message I understand there is a way of achieving this, but I'm not being able to understand how by reading the section you linked. Is it indeed possible?

Hey @tiangolo, I am also trying to reflect the updated status_code into the auto-doc without success, fastapi is really nice and I'm trying to convince a client to use it and auto-doc is a key argument !
@app.exception_handler(RequestValidationError)
async def validation_exception_handler(
    request: Request,
    exc: RequestValidationError
):
    return JSONResponse(
        status_code=status.HTTP_400_BAD_REQUEST,
        content=jsonable_encoder({"detail": exc.errors(), "body": exc.body}),
    )

Btw, thanks a lot for this framework and your dedication :) !

A discussion about a solution is happening here #1376

even though using the correct content format (JSON) through the correct channel (valid HTTP)

This is the problem, the format here is not simply JSON, but a rich OpenAPI schema. I see no reason why violating that schema, which is clearly the contract for interacting with the API, is not a client error. Of course one can argue this is subjective, but interpreting sending a string instead of a number as "the syntax of the request entity is correct" seems like a far fetch.
The reason that this is controversial to me is that 422 is a very nice (relatively) new response code to be able to use and distinguish from 400 (which always indicates there's a bug in the client) as meaning that the entity is in a state where it cannot currently be processed.

I don't mind the fact that FastAPI returns 422 when the request body contains syntactically valid JSON but fails pydantic validation. However, I think that it really should return 400 when the request body contains syntactically invalid JSON. This is what I came up with to achieve that behaviour:
from typing import Any, Sequence

from fastapi import FastAPI, Request, status
from fastapi.encoders import jsonable_encoder
from fastapi.exceptions import RequestValidationError
from fastapi.responses import JSONResponse

def is_error_dict_bad_request(errors: Sequence[Any]) -> bool:
    """Check if the given error indicates a malformed request."""
    if not len(errors) == 1:
        return False

    error_item = errors[0]

    if not isinstance(error_item, dict):
        return False

    if not isinstance(error_item.get("loc"), list):
        return False

    loc = error_item["loc"]

    if not 1 <= len(loc) <= 2:
        return False

    loc_item1 = loc[0]

    if loc_item1 != "body":
        return False

    loc_item2 = loc[1] if len(loc) > 1 else None

    if loc_item2:
        return False

    if not isinstance(error_item.get("msg"), str):
        return False

    msg = error_item["msg"]

    return (
        msg == "field required"
        or msg == "value is not a valid dict"
        or msg.startswith("Expecting value:")
    )

app = FastAPI()

@app.exception_handler(RequestValidationError)
async def request_validation_error_handler(
    request: Request, exc: RequestValidationError
) -> JSONResponse:
    errors = jsonable_encoder(exc.errors())
    status_code = (
        status.HTTP_400_BAD_REQUEST
        if is_error_dict_bad_request(errors)
        else status.HTTP_422_UNPROCESSABLE_ENTITY
    )

    return JSONResponse(
        status_code=status_code,
        content={"detail": errors},
    )

Then, it returns the appropriate status code for each use case.
Wrong content type in request header:
curl -X POST \
  -H "Content-Type: text/plain" \
  -d "{\"foo\": \"woo\"}" -v \
  http://localhost:8000/foo/
< HTTP/1.1 400 Bad Request
{
  "detail": [
    {
      "loc": [
        "body"
      ],
      "msg": "value is not a valid dict",
      "type": "type_error.dict"
    }
  ]
}

Missing request body:
curl -X POST \
  -H "Content-Type: application/json" -v \
  http://localhost:8000/foo/
< HTTP/1.1 400 Bad Request
{
  "detail": [
    {
      "loc": [
        "body"
      ],
      "msg": "field required",
      "type": "value_error.missing"
    }
  ]
}

Syntactically invalid JSON in request body:
curl -X POST \
  -H "Content-Type: application/json" \
  -d "foo" -v \
  http://localhost:8000/foo/
< HTTP/1.1 400 Bad Request
{
  "detail": [
    {
      "loc": [
        "body",
        0
      ],
      "msg": "Expecting value: line 1 column 1 (char 0)",
      "type": "value_error.jsondecode",
      "ctx": {
        "msg": "Expecting value",
        "doc": "foo",
        "pos": 0,
        "lineno": 1,
        "colno": 1
      }
    }
  ]
}

Syntactically valid JSON that fails pydantic validation in request body:
curl -X POST \
  -H "Content-Type: application/json" \
  -d "{\"foo\": \"woo\"}" -v \
  http://localhost:8000/foo/
< HTTP/1.1 422 Unprocessable Entity
{
  "detail": [
    {
      "loc": [
        "body",
        "moo"
      ],
      "msg": "field required",
      "type": "value_error.missing"
    }
  ]
}


⚠ Avatar API Endpoint has degraded performance

In 0447297, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1517 ms

Resolved: Avatar API Endpoint performance has improved in fb1f4ed.


API Docs Implementation

There's a lot here to review, and some important notes below, but the bulk of the API docs are implemented here.
What was done:

Finalize Doxygen and breathe implementation for C++ API doc generation.
Restructure api RST with both breathe and autodoc directives for C++ and Python tabs.
Adapted sphinxcontrib.prettyspecialmethods Sphinx extension to transform the Python doc naming to be more intuitive, and to play nicely with pybind11 generated signatures.
Wrote Python tool, based on pybind11_mkdoc, which leverages Doxygen XML instead of Clang to generate Python docstrings in a C++ header. As mentioned in previous discussions this will fail the build if it encounters a Doxygen XML element which it doesn't know how to render. I tried to make these failures clear by printing the offending XML and the resulting generated RST, and will note that the elements Doxygen uses are sometimes different on different platforms (which I learned while getting CI to pass).
Restructured Python bindings to include generated docstrings, and to forward declare iterators, enums, etc. used in Python classes, which allows generated docs to reference Python objects rather than C++ objects. Note that module attributes don't support __doc__ assignment in pybind11, and exception registration doesn't support __doc__ assignment in Python 2 bindings. For the attributes, which includes all the global constants, I put the docs directly in the RST, which is a bit of a maintenance annoyance since they are effectively duplicated, but it's limited to just that section. There may be ways around this in pybind11, but I could not find them as I dug through the pybind11 docs and source.
Remove automatic installation of documentation dependencies, but improve the detection and documentation around these dependencies.
Rework CMake to improve doc generation, including hints to rebuild when doc generation infrastructure (like the new Python tools, or Doxygen XML) change. My goal was to be able to build docs repeatedly and have the system only build what was changed, even if that was some upstream tooling which effects downstream docs in a subtle way. There may still be room for improvement here, but it is working fairly well in my tests.
Fixed some holes in the Python bindings from recent C++ changes.

TODO (in other PRs):

There are still quite a few Sphinx warnings to account for, but many are tied to the mixed doc style currently in the public C++ headers. I started to fix some of these, but decided to wait since I wasn't 100% sure of @doug-walker @hodoulp  and @BernardLefebvre 's vision there. Once those are restructured to adhere more strictly to Doxygen I can make refinements to the docstring extraction code as needed.
The Python docs are verbatim from the C++ right now, so code samples are wrong and unique bits of the API are undocumented. I (and others) can make smaller PRs to address these. I just wanted to get the mechanics in place, which you'll note if you build the docs are fairly good as-is.
I have not got these updates working in RTD yet. I will do that next, and we may want to wait on a merge until that point. Building the docs DOES require building OCIO, and there's no way around this without omitting the Python API docs. The build chain is such that the Python bindings MUST be built to have complete docs. The alternative to this would be to put the docs into stub Python modules, or directly in RST, which we can do, but was not my approach here. I would opt instead to leverage a CD pipeline if we have to, but will do some investigation into our options.

Thanks for having a look, and I highly encourage pulling this down and doing a local build to look at the HTML:
mkdir _build
cd _build
cmake ../. -DOCIO_BUILD_DOCS=ON
make -j<your thread count> docs
<your web browser> docs/build-html/index.html

Resolves #1148

@hodoulp , all of your feedback from #1160 has been addressed here as well.

Fixes #1148

Great. As 'time flies', I will have a look over the weekend to provide you feedback asap.

Thanks Michael -- I'll give it a spin this weekend too.

I've been investigating the RTD integration. We're good on the C++ end since RTD provides Doxygen in it's Docker images, but there's no support for building the Python bindings to utilize autodoc to build the Python API docs (without a matching Python binding binary, and thus an OCIO binary as well).
One option could be to stash those binaries somewhere and download them into the RTD VM as a sphinx build step, but that has some artifact storage requirements.
The solution I am leaning toward, and would like feedback on, is to store a complete set of API RST in the repo. By "complete" I mean a baked RST from both breathe and autodoc. That would mean that anytime the public headers, or Python bindings are modified, the PR would have to also submit an intermediate RST build for use by RTD (and others wanting to build the docs without building OCIO). To guarantee this generated RST and the OCIO source stay in sync, I would add a check to the CI which would build the RST and diff it with the same files in the repo, failing the build if they don't match. It's not an ideal solution, but would be manageable.
Certainly open to other ideas as well, like maintaining special CD-generated doc branches, but it' up for discussion. I'll put it on the TSC agenda, but feel free to discuss here if you have an opinion.

I started down this path for the original SIGGRAPH implementation, only to have it blow up in my face because of the sub-par quality of Autodoc -> RST generation (https://github.com/sphinx-contrib/restbuilder). Might still be worth having someone else validate my experience, but it produced terribly mangled RST that didn't look pretty or make sense in a lot of cases (esp. with Breathe sources)
Could be looked into of course, but adds to the growing mountain of custom solutions just to get docs out the door (which was a trend I tried to avoid, but feels inevitable at this point).

Thanks @scoopxyz . My approach to "baking" the RST was going to be to write a very lightweight extension which simply echo's the RST that breathe and autodoc generate themselves, with no other transformation. I found some hints at how to do this with very minimal code, but I have not yet tested the theory. In short, I would mokeypatch an autodoc method which appends RST lines during doc generation, which we would redirect to some output. My intent was to put this into some separate files and include them into the main API doc structure, as to not mess with the controlled portions of the docs.
As far as using custom solutions, I get the sense that other projects are just handwriting their Python docs in a similar way to what OCIO previously did. The downside to that approach is that we wouldn't be able to leverage pybind11's built-in type hinting, which produces excellent signatures for the Python docs without human input.

Heads up, this PR is going to be in flux for the next few days while I swap in Python API doc freezing functionality.

Ok, I think I'm done with this. I may push this to a test branch to do some RTD integration tests before we merge, but my simulated tests locally have worked so far. Here's a summary of how this all works:

If a contributor makes changes to any part of OCIO which affects the Python API docs (so, public headers, Python bindings, any documentation process code, etc.) they MUST do a local build with the new CMake option -DOCIO_BUILD_FROZEN_DOCS=ON, and add the modified RST files under docs/api/python/frozen to their PR.
The OCIO conf.py module has a switch that detects when docs are being built on GH Actions (CI env var == true) it will backup the frozen folder to a sibling backup folder on Sphinx init, and following Sphinx build completion will do a file-by-file comparison of the new frozen and the backup folders. If there are differences, the CI job will fail with an error explaining where the differences where found and with instructions on how to fix it.
The conf.py also has a switch that detects when it is being run on RTD, and in that case will itself run Doxygen to generate the XML needed by breathe prior to building the docs, and will also facilitate a CMake configure_file-like process (via Python) to handle substitutions in headers and docs source files that CMake would usually handle, but can't in this case. One potential plus to all of this is that if someone wants to just build OCIO docs, they can technically do so by running sphinx-build in the docs directory, and nothing more. Right now that only works when the READTHEDOCS env var == True, but it could be easily exposed another way if needed.
All of these required several custom Sphinx extensions tuned for our project specifically. They could be made more generic for improved sharing in the future, but that was not my main focus for now. It wouldn't be a huge amount of additional work to do that though, but in a future PR.
Reviewing all of this in GH is going to be a pain with ~400 modified files, but most of that is small, granular, RST files for the frozen docs support.

The CI failures here are from the frozen doc comparison check I mentioned. For example:
Extension error:
Frozen RST is out of date! Build OpenColorIO with CMake option '-DOCIO_BUILD_FROZEN_DOCS=ON' to update required frozen documentation source files in: /__w/OpenColorIO/OpenColorIO/docs/api/python/frozen

    Changed files: pyopencolorio_filetransform.rst, pyopencolorio_gpushaderdesc.rst, pyopencolorio_formatmetadata.rst, pyopencolorio_cpuprocessor.rst, pyopencolorio_setcomputehashfunction.rst, pyopencolorio_autostride.rst, pyopencolorio_config.rst, pyopencolorio_processor.rst, pyopencolorio_gradingrgbcurve.rst, pyopencolorio_gpushadercreator.rst, pyopencolorio_dynamicproperty.rst, pyopencolorio_gpuprocessor.rst

      Added files:

    Removed files:

I'm noticing the error varies a bit per platform, which could be problematic, but I'll troubleshoot until CI passes.

Now that CI is passing on all platforms, I'm starting RTD integration testing, using the newly created rtd_test branch.

Build succeeded on RTD: https://opencolorio.readthedocs.io/en/rtd_test/api/_index.html#api
This is ready for review.

One note to add to other explanations. When testing the frozen RST generation against the existing source in the repo, there were differences per platform. I implemented a difference check with an error tolerance which tests for a similarity ratio, and if it falls below that it fails CI. This does imply that the docs can vary slightly depending on who builds them, and that the comparison could miss small inconsistencies. An alternative approach would be to only do this check on Linux CI, and be stricter about it. Linux is the ideal choice there since RTD builds are done on Linux.  Up for discussion.


Template should create net6 version of Saturn

Current release of Saturn uses net6. Shouldn't this template create that version?

The new version on nuget should now use .NET 6 versions of saturn.cli and the saturn package.


adding type to support the Bson ObjectId and fix the int for goswagge…

Hello,
There are 2 changes here:

Support the Bson ObjectId type
And fix "unknown type: int".

Codecov Report

Merging #26 into master will decrease coverage by 0.49%.
The diff coverage is 50%.

@@            Coverage Diff            @@
##           master      #26     +/-   ##
=========================================
- Coverage   94.95%   94.45%   -0.5%
=========================================
  Files           4        4
  Lines         357      361      +4
=========================================
+ Hits          339      341      +2
- Misses         13       15      +2
  Partials        5        5

Impacted Files
Coverage Δ

property.go
56.52% <50%> (-1.38%)
:arrow_down:

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 8584ff3...e0db9de. Read the comment docs.

@jayhuang75 Sorry for late reply. LGTM.
Thank you for your contribution. 🎉 🎉 🎉 🎉 🎉 💯


serve-handler breaks relative paths in files named index.html

If you have an index.html in a subdirectory default behavior strips out the index.html part of the url which breaks relative paths. public/foo/index.html becomes public/foo.
This causes an issue when you have a relative script in public/foo/index.html referencing <script src="main.js"></script>. Instead of looking for public/foo/main.js default browser behavior is now to look for public/main.js.
Setting cleanUrls to true would fix this issue, except it causes the public/foo/index.html link to redirect to the directory listing for public/foo causing public/foo/index.html to be inaccessible.
If this is intended default behavior it should probably be documented, if not it would be great to have an option to serve index.html files.

Feel free to add trailingSlash to true. That'll fix this! 😊

thanks!

I've opened an issue here : https://github.com/zeit/serve/issues/535 who seems very similar, but even if i add trailingSlash to true, my problem stay the same, maybe somebody has a trick ? :)


LDC AST view reports no output

The AST pane reports <No output> for LDC as seen https://godbolt.org/g/SQRkEb
cc @JohanEngelen

Things work locally. Could this be because LDC outputs the AST file in the same directory (!) as the source file?  (cannot be specified currently, possibly need to change compiler behavior upstream)

I think I might have the solution. But first I need to know if the file is named example.d.cg or simply example.cg?

As I suspected
Mar 24 02:16:56 06b5a890b22d unified: warn: LDC AST file /tmp/compiler-explorer-compiler118224-56-1dx4lsj.ohjr/example.d.cg requested but it does not exist
Mar 24 02:16:56 06b5a890b22d unified: info:  - - [24/Mar/2018:01:16:56 +0000] "POST /api/compiler/ldc180/compile HTTP/1.0" 200 - "-" "Amazon CloudFront"

can be seen in the beta logs, meaning that this check https://github.com/mattgodbolt/compiler-explorer/blob/master/lib/compilers/ldc.js#L80 is failing, but afaik, it should be working! So weird!

Was a bug in the execution code! Fixed in f9fd86fc248ea3dc95bb440803fd41d990d3db12, sorry for all the confusion!


Inconsistent css in dev vs prod

Version
v2.2.0
Reproduction link
https://github.com/adi-zz/nuxt-css-dev-vs-prod
Steps to reproduce

git clone https://github.com/adi-zz/nuxt-css-dev-vs-prod
cd nuxt-css-dev-vs-prod
npm i

npm run dev
npm run build && npm run start

open http://localhost:3000/

What is expected ?
The background color of nuxt logo should be blue in both cases 3.1. and 3.2. (or at least red in both cases).
What is actually happening?
Background is blue in dev mode (3.1.), and red in prod mode (3.2.)
Additional comments?
My project was generated using npx create-nuxt-app test (choosing all defaults)

This bug report is available on Nuxt community (#c8047)

I'm also seeing this. For the time being I've pinned my version to v2.1.0 of nuxt and it works.
I suspect that this may have something to do with the following commit but have not tested enough to be 100% certain to say yes or no.
https://github.com/clarkdo/nuxt.js/commit/ec63cfc83529444189f47c16227d8e31842d4d32#diff-e08c5c9779ed38bdf6e44f502bb1e732

@spnc-omz Oh, so it worked with 2.1?

This gets even more weird. I think I had a dependency issue. Everything worked for me after I updated my postcss preset ebb to ~6.3.0

@spnc-omz which exact module are you referring to? I only see postcss-env-function in my node_modules folder

@AndrewBogdanovTSS 😸
I understand and I didn't mean that we must use scoped here.
Since the injection of css files is asynchronous, so the order of css files are not guaranteed, the root problem is more related to the way how we should organize css.
My point is that it's not a proper way to prioritize styles depending on the order of selector location in file.
If we want make a selector prior to another, we should use higher priority selector:
// component
<style>
.VueToNuxtLogo {
  background-color: red;
}
</style>

// index.vue
<style>
.index .VueToNuxtLogo {
  background-color: blue;
}
</style>

@clarkdo yeah, I also understand that, but you know, people been controlling css priority by defining specific order for years and  practically getting rid of this habit will mean rewriting thousands of already existing css selectors whereas just switching the file order will fix it in a finger snap (you don't even need to be a Thanos to do that)

I'm seeing this issue also.
Using the nuxt.config.js css option, I am seeing inconsistent CSS load order, which doesn't follow the order specified. in the CSS array
We are using 3rd party CSS libraries, of which we create an override style sheet, which needs to be loaded last, and sometimes it is being loaded first.

Wasn't this already been fixed? I had similar issue I logged here https://github.com/nuxt/nuxt.js/issues/4219#issuecomment-437545141
and now it's respecting an order for me

@tmorehouse did you try it with Nuxt 2.4.2? Is it working for you?

@AndrewBogdanovTSS
The issue appears to have been fixed.

Nice!

And it happening again for "nuxt": "^2.7.1"
In the inspector window, I see that styles have a different order.
The application was generated using npx create-nuxt-app test with vuetify ui.
And I see that _grid.styl appears after _lists.styl in dev and vice versa in prod.
There a lot of other scuffles as well.
Could this ticket be reopened? Or I need to create new one?

Yes, I'm having the same issue here with inconsistent styling between dev and prod environments.
Here's a link to a reproduction: https://codesandbox.io/s/examplenuxtvuetifycssorder-k4ob0

Expected: have card title and text on the same line because of CSS display: inline inside component (which works well in dev environment)
Result: card title and text are on 2 different lines because the third-party CSS display: flex appears after (only in prod environment)

I thought it might be an issue with vuetify-loader but they say it's not: https://github.com/vuetifyjs/vuetify-loader/issues/23
Can this issue be reopen?

I am experiencing the problem too.  Also using vuetify-loader

@manniL please, reopen! extractCSS is so buggy that I actually gave up on it, it was easier to reduce styles by 10 times than to fix all issues that keep happening with this property 🤦♂

Please see #4219

Styles are different - not using purgecss or any other plugin accept nuxt defaults

this issues happened to the css which used on the classname/id/element add by javascript, when you run npm run prod ,then nuxt will check which classname/id/element were used, and render these css only, different from npm run dev which nuxt will render all the css

i am using a combination of

Based on my experience, there  are 2 reasons that i know of that makes the css of the  built version not consistent with the dev version.

using  unscoped style


transformers model  address 'cl-tohoku/bert-base-japanese-v3'  now is not work

new address is 'tohoku-nlp/bert-base-japanese-v3'；need to change
https://github.com/myshell-ai/MeloTTS/blob/63869183f826c2109071fa81df9adf818942f296/melo/text/japanese_bert.py#L12
https://github.com/myshell-ai/MeloTTS/blob/63869183f826c2109071fa81df9adf818942f296/melo/text/japanese.py#L569

fixed. very much appreciated!


Rename operation to endpoint for RPC metrics

Part 1 of https://github.com/uber/jaeger-client-java/issues/172
After this, I'll work on abstracting out the metrics to not be http specific

Coverage remained the same at 85.7% when pulling 79bb02e0099b49c57edb74423567216a45fd9104 on rename_rpc_metrics into 39b0a4234a1c700332c2bba02be75a1948b6dca7 on master.


update image issues

Describe Changes
When updating an image and the image was the same as before two issues would occur:

last deployed by who and when did not get reported
the wait flag would cause an error because a certain value wasn't initialized properly.

Link to Issues
https://app.clickup.com/t/8655600/DUPLO-25663
https://app.clickup.com/t/8655600/DUPLO-25221
PR Review Checklist

[x] Thoroughly reviewed on local machine.
[ ] Have you added any tests
[x] Make sure to note changes in Changelog

Task linked: DUPLO-25221 Issue: Last Deployed By
and Last Deployed At doesn't get updated
Task linked: DUPLO-25663 Update image errors on wait when the same image


ngModel / Error

<ckeditor
    [(ngModel)]="article.description"
    [config]="{uiColor: '#99000'}"
    (change)="onChange($event)"
    (ready)="onReady($event)"
    (focus)="onFocus($event)"
   (blur)="onBlur($event)"
   debounce="500">
</ckeditor>

Error message : If ngModel is used within a form tag, either the name attribute must be set or the form
control must be defined as 'standalone' in ngModelOptions.
Any idea ?

as the error implies, you need to add a "name" property to your form control. ie: name="randomName"

@mwamufiya yeh that what i do and it's work fine and i forget to close issue and thank you for your answer


doc: fixing minor typos

there are two mentions to 'memecached' instead of 'memcached'.
Fix #3756

/bugzilla cc-qa


Draft PR for Snapshots

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to module CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

LRO recordings blocked on https://github.com/Azure/azure-sdk-for-go/pull/22102

Closing in favor of https://github.com/Azure/azure-sdk-for-go/pull/22118


send requestBody to mock function if it can handle it

The change is backward compatible

Can you name it request rather than reqBody? Thanks.

Can you name it request rather than reqBody? Thanks.

Yes, but the function got just the request.Body()


Math::is_nan(v.x) occures on ARVRCamera

Godot 3.1.1 Stable
Windows 10
Issue: Math::is_nan(v.x) thrown when playing around with the transformation for ARVRCamera
Steps to reproduce:

Just setup a normal VR Application
Create ARVRCamera as per normal VR Application:
-- it works just fine when no calculation has been involved ARVRCamera.
-- try attach one node to ARVRCamera as a child and do some transformation on the child node. the error will be catched as 'Math::is_nan(v.x) etc etc. (but still it is works just fine)
-- or else: without any node attached, try to call ARVRCamera and play with the transformation through the script somewhere anywhere, it will throw the same error again. (yes still, the scripts just works as expected)
-- for all these errors above, the codes or the nodes working great as expected, only that it throws an error... which the real matter is:-
-- this error will prevent me from 'exporting' as 'released' application

I wanted to give this issue a try for my first contribution to Godot but I can't reproduce the issue (on Linux).
Could you provide a sample scene where this error appears please?

Should we close this? Seems inactive, unreproduced, and like https://github.com/godotengine/godot/issues/32611 might have fixed it


Command to diff environment variables

I'm updating an app I work on that has tons of environment variables in the current deployed environments, some of them which have become unused over the time. It would be very useful for envied to provide a diff command which allowed to see the diff between the variables in say heroku and the manifest.

@Tonkpils could you provide more info? At first I thought you were suggesting a diff command to provide the current deployed environment config to compare against the Envfile before deploying to ensure all requirements are met. If that's the case that is something I've had to deal with recently and that is on my todo list for a new release. But since you mentioned about some being unused, are you wanting to find unused environment variables instead?

👋 @javierjulio unfortunately, I'm no longer at the company that required this use case. The use case you mention however is what I believe was what we needed at the time.

@Tonkpils no problem, thanks for following up! 👍🏻


Please consider a LICENCE clarification

Hello @cristinegulescu ,
reviewing the recursive JSON code as shown in https://www.youtube.com/watch?v=07qnWe9bf6k I noticed that
a) the code has a high potential for re-use and/or re-implementation
b) but the repo does not contain any specific lincence info.
To quote GitHub:

without a license, the default copyright laws apply, meaning that you retain all rights to your source code and no one may reproduce, distribute, or create derivative works from your work
https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/licensing-a-repository

(Which might be a perfectly acceptable intention, as I am happy that I can "fork and view" your work as per GitHub Terms of Service.)
Nevertheless, I would like to ask you to consider any clarification -- especially with the enterprise nature of RPA in mind.
Best regards,
Christian

Hello,
I think for me the logic of the license should be afl-3.0.  I use GITHUB to share the code from my Videos on Youtube. To be real I mean someone sees something in my movie and they have access also to the code from a movie.
The issue with this one probably is that I upload it as a zip, I don't know how to Upload as a project.
So please tell me where I have to go to set the license as Academic Free License v3.0
thanks,
Cristian

Done I upload like this
https://github.com/cristinegulescu/JSONtoDataTable
with MIT lincense

Oh, what a quick reply -- and even a very permissive licence!
The "challenge" of the zip file did not occur to me.
I was experimenting with your code to output a Dictionary (of String, Object), will explore this a bit, maybe fork -- and show you the result.


Update real-time-marketing-analytics.md

May 2023 release updates - should be published on May 2

Great, thanks @miltiadisv


Fix CVE-2022-28948

Drop logrus in favor of slog.
Bump testify and use in healthcheck
test to make it direct dependency.
Fixes #1932.

You cannot remove JSON marshal of the logs they are used by mint tests

I am not sure if I get this comment because JSON formatting of comments aren't changed but custom marshaler is removed since it's not needed (i.e. we use slog.JSONHandler now).

@harshavardhana  anything else is needed to get this in?


Add link to Guthub project in Error message

The error messages include the hint to open a bug report.

Please provide this when opening a bug report.

I think it would be be useful to add a link to this Github project to save the user searching for the project.

Just recognised some errors already include this link:
╭──────────────────────────────────────────────────────────────────────────────╮
│ ERROR                                                                        │
│                                                                              │
│ An error occurred while executing this command:                              │
│                                                                              │
│   Error: Filter flag has an invalid value                                    │
│                                                                              │
│ If you believe this to be a bug, please open an issue at                     │
│ https://github.com/mittwald/cli/issues/new                                   │
│ (https://github.com/mittwald/cli/issues/new).                              │
╰──────────────────────────────────────────────────────────────────────────────╯

But the formatting could be improved (there is no need to print the link twice) and some other errors (see #54) are missing the link.

Regarding the duplicate link: Is this some rendering variation of the <Link> component? AFAIK it should just render a clickable link into the terminal -- maybe this is some kind of fallback behaviour in case the terminal emulator doesn't support links?


Cannot add anything on kubernetes stack

I've added kubernetes env and tried to add a replication controller. It does not depend if i choose a catalog item or if i want to add something manually. The only thin i get is this error in the interface:
Unexpected token u in JSON at position 0
Rancher is running inside a VM with CentOS 7
Rancher is on different host than agent
Both hosts are inside a private network with vpn only connection from the outside
Both hosts can talk to each other

Useful
Info

Versions
Rancher v1.2.0-pre1 Cattle: v0.167.5 UI: v1.2.5

Access
Disabled

Orchestration
Kubernetes

Route
catalog-tab.launch

with kubectl get pods is see that there are no nodes but kubernetes is up and running if i take a look at the UI of rancher all is green.

so the kubernetes system stack is all green?
how many hosts are in your k8s env?
and can you share the image name of your kubernetes service in the system stack?

Yes the kubernetes system stack is all green.
I have two hosts at the moment but it also happens with one.
The kubernetes service has the image: rancher/k8s:v1.2.4-rancher9
Here are some screenshots.
http://screencast.com/t/I8NF0WrAsu
http://screencast.com/t/Sx6izhVz0c
http://screencast.com/t/EIuWRbhj
Im very into rancher and really want to keep it but i need to have a production ready setup soon. so im very thankful for any help because otherwise i need to switch to different solution.

I was able to solve the problem with some firewall rules to accept outgoing traffic to port 8091. In Documentation i did not found this port so it would be good to add this...
From my side you can close this issue.


chrome 69 does not delete Google cookies

I just came across this on Twitter. Vanilla chrome does not delete Google cookies when cookies are deleted. Is ungoogled-chromium affected by this?

This seems to be related to the "Sign-in to Chromium" function of the browser which is disabled.
Otherwise all Google cookies are deleted, at least the ones contained in the 'Cookies' file found in the browser profile.

If this is really the case, this issue can be closed. Is there a way to check this?

You run Chromium with a new temporary profile, login to various Google sites,
rm -rf /tmp/chromium-profile && chromium --user-data-dir=/tmp/chromium-profile
and inspect the cookie database with e.g. sqlitebrowser:
sqlitebrowser ~/.config/chromium/Default/Cookies

Trying the method from @xsmile, I can confirm that deleting cookies actually deletes all cookies on ungoogled-chromium-69.


crates publishing team

👋 Hi @frewsxcv you are the only "user-owner" of the topojson crate, so only you can edit that crate's owners on crates.io
Following up on https://github.com/georust/meta/pull/21, I created a new topojson team for publishing to crates.io with you as a maintainer (actually since @frewsxcv is an organization owner, GH won't let me mark him as a maintainer, but it'd be redundant permissions anyway 🤷).
Could you please update the crate owners? Specifically:

[ ] update topojson crate owners on crates.io:

cd topojson
# 💥 remove large publishing group
cargo owner --remove github:georust:core
# 🔐 add small publishing group
cargo owner --add github:georust:topojson-publishers

# you're currently the only user-owner of topojson. Consider adding another user-owner,
# that way if for some reason you become inaccessible we have someone else who can edit the owners.
# See https://github.com/rust-lang/crates.io/issues/2906
cargo owner --add <username>

# make sure everything looks good 👀
cargo owner --list

If there's anyone else you'd like to be able to publish the crates, feel free to add them to your publishing team, or as a user-owner. Consider that user-owners, as opposed to owners-via-team, can themselves edit the crate's owners, which could be desirable, or not, depending.
In particular, users who have been active in this repo in the past:

@mgax
@mthh
@pjsier

Are any of them interested in being added as publishers?
Let me know if you have any questions, or you can review https://github.com/georust/meta/pull/21.

I'm interested in being a publisher if that's alright

done


standalone提交任务运行报错

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
错误信息：
Caused by: java.lang.IllegalStateException: Expected BEGIN_OBJECT but was STRING at line 1 column 1 path $
at com.google.gson.stream.JsonReader.beginObject(JsonReader.java:385)
at com.google.gson.internal.bind.ReflectiveTypeAdapterFactory$Adapter.read(ReflectiveTypeAdapterFactory.java:213)
... 20 more
What you expected to happen
正常启动
How to reproduce
配置：
CREATE TABLE ops_config
(
id          INT,
config_key  VARCHAR,
type        VARCHAR
) WITH (
'connector' = 'mysql-x',
'url' = 'jdbc:mysql://<IP_ADDRESS>:3306/iot_db_ops',
'table-name' = 'ops_config',
'username' = '123',
'password' = '123'
);
CREATE TABLE ops_config_copy (
id INT,
config_key VARCHAR,
type VARCHAR
) WITH (
'connector' = 'stream-x',
'print' = 'true'
);
insert into ops_config_copy
select *
from ops_config;
启动：sh bin/chunjun-standalone.sh -job job/batch-temp.sql
启动日志：start command: /data/home/blysin/software/jdk1.8.0_381/bin/java -cp /home/blysin/software/chunjun/chunjun-dist/../lib/chunjun-clients.jar:/home/blysin/software/chunjun/chunjun-dist/../lib/* com.dtstack.chunjun.client.Launcher -job job/batch-temp.sql -mode standalone -jobType sync -chunjunDistDir /home/blysin/software/chunjun/chunjun-dist -flinkConfDir /data/home/blysin/software/flink/conf -flinkLibDir /data/home/blysin/software/flink/lib
Anything else
相同的bug为什么没处理直接关了？#89
Version
master
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

还有没有人维护啊？今天最新版本还是这个错！我是local模式


Feature/addons

PR o'clock
Description
Add support for eks addons recommended by was

VPC CNI
CoreDns
kube-proxy

The default behavior is to install the addons to the cluster.
Requires irsa to be enabled to deploy the VPC CNI addon
Checklist

[V] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

Need to add some fixes


SW-2179 Treat empty Mapbox token as absent

If your environment includes an empty TERRAWARE_MAPBOX_API_TOKEN environment
variable, the server shouldn't try to use that empty value to make Mapbox API
calls.

Current dependencies on/for this PR:

main

PR #792   👈

This comment was auto-generated by Graphite.


Added kanidm-rustlang server side implementation for webauthn

@harhitosw Add new line between sections please *)

@yackermann oops! can you check now ? I have added space in the recent commit.

Thanks @harhitosw !


Add Bitci HBAR/USD rate

Summary
Turkish exchange Bitci has an HBAR/USD pair
https://borsa.bitci.com/exchange/standart/HBAR_USD.
Possible resolution

Couldn't find an API that will give us price and volume.

This URL
https://api.bitci.com/api/ReturnTicker
is all coins but it does give HBAR price and volume
The HBAR specific URL
https://api.bitci.com/api/coin/HBAR_USD
isn't working

This URL
https://api.bitci.com/api/ReturnTicker
is all coins but it does give HBAR price and volume
The HBAR specific URL
https://api.bitci.com/api/coin/HBAR_USD
isn't working


add pt score chart PDF files

add /charts page route that lets user pick respective scoring chart for gender/age group

This issue has been resolved by #30


Added IAsyncDisposable support

Description
Added check against IAsyncDisposable in addition to IDisposable on shutdown
Purpose
This pull request is a:

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[X] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

LGTM 👍🏻