Form submits two times in Chrome when ajax validation is enabled

When ajax validation is enabled and I click submit button in Chrome - two records appears in DB. This occures in Chrome only.
On submit I see 1 request to the server in FF and 2 in Chrome.
This issue reproducing when you type in field and then QUICKLY click Submit button. When you do it slowly it works correctly (depends on server speed).
Probably form doesn't have time to receive respond from ajax validation or something else.
Also I see In Chrome it sends additional request on submit, but there is no such request in FF (maybe this is connected to the issue).
My form:
    <?php
    $form = ActiveForm::begin([
        'enableAjaxValidation' => true,
    ]); ?>

                <?= $form->field($model, 'title') ?>
                <?= $form->field($model, 'description')->textarea(['rows'=>6]) ?>

             <?= Html::submitButton(($model->isNewRecord? 'Post' : 'Update'),
                 ['class' => 'btn btn-primary', 'name' => 'update-button']) ?>

    <?php ActiveForm::end(); ?>

My Controller:
    public function actionCreateJob($id=null)
    {
        $model = new Post();

        if (Yii::$app->request->isAjax) {
            $model->load(Yii::$app->request->post());
            Yii::$app->response->format = Response::FORMAT_JSON;
            return ActiveForm::validate($model);
        }
        elseif ($model->load(Yii::$app->request->post()) && $model->save()) {
            Yii::$app->session->setFlash(
                'success',
                Yii::t('app', 'Post successfully added.')
            );
            return $this->redirect('/profile/edit');
        }

        return $this->render('create',['model'=>$model]);
    }

Hi @qiangxue
I catch same issue about multi times form submitting
It happens in Chrome when data is pre filled by browser. And these requests runs until the page is fully loaded
See image below.


Drop ReactBlessedTextComponent if really not useful anymore

I still have it lingering in the repo but it's completely unused for me.

Same for me now. It will probably disappear soon.

On a side note, if you drop the injection, does it cause any issue? Just wondering if I need to keep a noop class for the TextComponent.

I am currently not injecting it. But I sometimes got issues with text components rendered as my text because of it.
Looks like 0.14 still has an awful lot of references to the DOM.

Non reproducible anymore. I have to admin the renderer was in a bad state at that moment though.
The bug-triggerer looked like this
class Something extends Component {
  render() {
    const { ...children, props } = this.props;
    return (
      <SomethingElse {{ ...props }}>
        { children }
      </SomethingElse>
    );
  }
}

Notice how I confused { children, ...props } with { ...children, props }. I was using a keyed object as a child, therefore having the object values as children.

@yuchi, you confirm that I can drop the TextComponent?

Drop it like it’s hot…
Drop it like it’s hot…
Drop it like it’s hot…

Thanks for the laugh @yuchi! I am clearly hearing the soundtrack.

I just noticed that you’re using lodash to operate on children.
My I suggest you to switch to a less performant but guaranteed form that uses React.Children.* methods? (See mine)

:full_moon_with_face:

I'm sorry @jdalton, didn't mean to offend lodash! :)
Technically speaking react children are an opaque data structure that can be composed of arrays, objects, arrays of object of elements, single elements. Actually I don't know the whole spectrum and cases, that's why I use that react util.

I see your point about ReactChildren.map as it would probably be future-proof @yuchi, but I just cannot hurt @jdalton's feeling like that. Let me solve this soon :).

Lol :heart:


Add support for World of Warcraft .toc files

I'm less interested in the data format than I am fixing its widespread misclassification: searching for .toc files tosses up an impressive number of results: most of them WoW-related.
This misclassification sharply skews TeX's usage statistics, so it's worth addressing. As usual, I collated public .toc files from extension search results to test classification accuracy; the results are here. Since I've classified WoW's addon data as... well, data, it doesn't show up in the breakdown graph. Which I'm guessing is the expected behaviour, really.

Since I've classified WoW's addon data as... well, data, it doesn't show up in the breakdown graph. Which I'm guessing is the expected behaviour, really.

You can temporarily change its type to programming. Otherwise it won't show up in results and won't be counted in the overall pourcentage ;)

Heh, yeah, that occurred to me only after I'd scanned the files halfway. I felt that scanning the other half was probably going to be quicker than rebuilding the program.

Conflicts fixed.

How did you select the samples for tests? How did you distinguish WoW files from TeX files?

According to the format's documentation, WoW's metadata files provide tags to store auxiliary information to a consuming client. The tags are identified by lines beginning with "## ", with every other line being either a comment "# " or a filepath.
As for TeX, well, that's pretty easy. Files containing lines beginning with \contentsline, \defcounter, \beamer or \boolfalse are guaranteed to be TeX instead of WoW-metadata.
 The TeX fixtures were cobbled together using titles of the German Wikipedia page for Perl.

@arfon the grammar's repository is MIT-licensed. What do I have to do to get the build to succeed...?

Sorry @Alhadis - since you opened this PR we added an additional check for licenses. You need to do this too:
Download the license for the grammar: script/licensed. Be careful to only commit the file
for the new grammar, as this script may update licenses for other grammars as well.

Hahaha!  Looks like the current build failure is unrelated to this PR. I'm working on a fix.

@Alhadis would you mind cherry picking this commit into your PR? https://github.com/github/linguist/pull/2983/commits/eeae8d4839b92e89b1da77536ad4dd020421839c
It fixes the grammar failures.

Cherry picked, sir! =)

Awesome thanks for this! (And your patience!)

Haha, any time my friend. :D Always a pleasure to help.

Good to see this! Might be a bit late, but I should point out having "@no-lib-strip@" as part of the heuristic is a bit of a misnomer, it really has nothing to do with the toc format, they're tags used by a popular distribution site for packaging and are used in lua files as well. "^## Interface" would probably be the best test, as it's the only mandatory tag.

Well, Bayesian analysis handles most of the classification. It's when the classifier can't decide on a file that it consults the heuristics (AFAIU). The heuristics are there mainly to "point" the classifier in the right direction. To the best of my understanding, this is how it "learns" which results are wrong and which are accurate. @pchaigno may be able to offer a more informed explanation.
Anyway, since the heuristic is already based on lines that begin with ## , I felt a different-looking token would be more helpful to correctly classify WoW-ToC. Incidentally, a WoW that was still being misclassified as TeX was correctly identified once I'd added the @no-lib-strip@ part.

It's when the classifier can't decide on a file that it consults the heuristics (AFAIU).

Actually, it's the other way around :wink:
When the heuristic strategy fails to determine the language, the classifier is used (since it's the less reliable strategy). You can see the current order of strategies in linguist.rb.

Ah rats, whoops. Guess that makes a lot more sense from a performance standpoint too... thanks!

Anyway, since the heuristic is already based on lines that begin with ##, I felt a different-looking token would be more helpful to correctly classify WoW-ToC.

My point was that any WoW toc will have "## Interface: XXXXXX" so it's a better test, where as @no-lib-strip@ is pretty arbitrary.

Hrm, fair enough.
Either way, the heuristics are working perfectly; short of 3,000 files were correctly classified as WoW or TeX. Only four escaped correct classfication:

ChatKeys.toc
login.toc
nagrand.toc
test.toc

There was no hope for the last three to be correctly identified anyway, given their incomplete content.
ChatKeys is somewhat unusual though... it should be matching, but it isn't... @pchaigno, has Linguist had any difficulties with certain line-endings before? That one file is using old-style CR terminators (used on ancient Mac OS systems). Could that have something to do with it?

has Linguist had any difficulties with certain line-endings before?

Not that I know.

That one file is using old-style CR terminators (used on ancient Mac OS systems). Could that have something to do with it?

That's exactly what's happening. I tried on the same file with LF newlines and it worked fine.
I'm not sure how we should proceed...

I think this is worth opening a new issue for, quite frankly. :| If line-endings have the ability to affect language recognition, then this definitely isn't a trivial issue.

Agree :-)

Filed. =)


Deprecate Python 2 support

We plan to deprecate python 2.7 support.

Python 2 compatiblity code is removed in PyThaiNLP version 1.8


Feature/import from shlink

Closes #4
Closes #25

Codecov Report

Merging #26 (a7791e9) into main (0f3ace3) will increase coverage by 5.28%.
The diff coverage is 98.33%.

@@             Coverage Diff              @@
##               main      #26      +/-   ##
============================================
+ Coverage     92.59%   97.88%   +5.28%
- Complexity       83      131      +48
============================================
  Files            16       24       +8
  Lines           270      425     +155
============================================
+ Hits            250      416     +166
+ Misses           20        9      -11

Impacted Files
Coverage Δ
Complexity Δ

src/Sources/Bitly/BitlyApiProgressTracker.php
92.30% <ø> (ø)
11.00 <0.00> (ø)

src/Model/ImportedShlinkVisit.php
84.61% <84.61%> (ø)
5.00 <5.00> (?)

src/Sources/ShlinkApi/ShlinkApiImporter.php
98.36% <98.36%> (ø)
11.00 <11.00> (?)

src/Command/ImportCommand.php
100.00% <100.00%> (ø)
12.00 <5.00> (+1.00)

src/Http/InvalidRequestException.php
100.00% <100.00%> (ø)
5.00 <5.00> (?)

src/Http/RestApiConsumer.php
100.00% <100.00%> (ø)
4.00 <4.00> (?)

src/Model/ImportedShlinkUrl.php
100.00% <100.00%> (+63.63%)
11.00 <3.00> (+3.00)

src/Model/ImportedShlinkVisitLocation.php
100.00% <100.00%> (ø)
8.00 <8.00> (?)

src/Model/ImportedShortUrlMeta.php
100.00% <100.00%> (ø)
5.00 <5.00> (?)

src/Sources/Bitly/BitlyApiException.php
100.00% <100.00%> (ø)
1.00 <1.00> (ø)

... and 14 more

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 0f3ace3...a7791e9. Read the comment docs.


Ambassador Diagnostics Interface Reporting Newer Version Available When There Isn't

Go to the Diagnostics UI
Ambassador version 0.77.0
displays
INFO: Upgrade available! to Ambassador version 0.77.0

We are seeing this as well. From the logs:
2019-09-10 21:12:32 diagd 0.77.0 [P91TAmbassadorEventWatcher] INFO: Scout reports {"latest_version": "0.77.0", "application": "ambassador", "cached": false, "timestamp":<PHONE_NUMBER>.927258}
2019-09-10 21:12:32 diagd 0.77.0 [P91TAmbassadorEventWatcher] INFO: Scout notices: [{"level": "INFO", "message": "Upgrade available! to Ambassador version 0.77.0"}]```

I was facing the similar issue.


Problem with testinstall/strings.tst

brew test gap-system/homebrew-gap/gap fails with the following diffs:
testing: /usr/local/Cellar/gap/4.10.1/libexec/tst/testinstall/strings.tst
########> Diff in /usr/local/Cellar/gap/4.10.1/libexec/tst/testinstall/strings\
.tst:44
# Input is:
x:="\0xFF";
# Expected output:
"\377"
# But found:
""
########
########> Diff in /usr/local/Cellar/gap/4.10.1/libexec/tst/testinstall/strings\
.tst:48
# Input is:
x:="A string with \0xFF Hex stuff \0x42 in it";
# Expected output:
"A string with \377 Hex stuff B in it"
# But found:
"A string with ? Hex stuff B in it"
########
########> Diff in /usr/local/Cellar/gap/4.10.1/libexec/tst/testinstall/strings\
.tst:189
# Input is:
List("abcxyzABCXYZ\n\t !019?\377\000$", LowercaseChar);
# Expected output:
"abcxyzabcxyz\n\t !019?\377\000$"
# But found:
"abcxyzabcxyz\n\t !019??\000$"
########
########> Diff in /usr/local/Cellar/gap/4.10.1/libexec/tst/testinstall/strings\
.tst:191
# Input is:
List("abcxyzABCXYZ\n\t !019?\377\000$", UppercaseChar);
# Expected output:
"ABCXYZABCXYZ\n\t !019?\377\000$"
# But found:
"ABCXYZABCXYZ\n\t !019??\000$"
########
########> Diff in /usr/local/Cellar/gap/4.10.1/libexec/tst/testinstall/strings\
.tst:193
# Input is:
UppercaseString("abcxyzABCXYZ\n\t !019?\377\000$");
# Expected output:
"ABCXYZABCXYZ\n\t !019?\377\000$"
# But found:
"ABCXYZABCXYZ\n\t !019??\000$"
########
########> Diff in /usr/local/Cellar/gap/4.10.1/libexec/tst/testinstall/strings\
.tst:195
# Input is:
LowercaseString("abcxyzABCXYZ\n\t !019?\377\000$");
# Expected output:
"abcxyzabcxyz\n\t !019?\377\000$"
# But found:
"abcxyzabcxyz\n\t !019??\000$"
########

As said in https://github.com/Homebrew/homebrew-core/pull/36156, the same issue has been observed on SPARC Solaris 11 (https://github.com/gap-system/gap/issues/2194) and NixOS (https://github.com/gap-system/gap/issues/3152).
If I test GAP using
/usr/local/Cellar/gap/4.10.1/bin/gap -r -A /usr/local/Cellar/gap/4.10.1/libexec/tst/testinstall.g

without calling brew test, then the test passes without errors.
Since https://github.com/gap-system/gap/issues/2194 is marked as "wontfix", unless we figure out why brew test triggers this and which workaround to use, we can advise to the users to test it like http://www.gap-system.org/Download/index.html says, with
Read( Filename( DirectoriesLibrary("tst"), "testinstall.g" ));;

I gather it's either the locale settings, or something to do with curses/termcap, or both. See also https://github.com/gap-system/gap/issues/3426

Try starting GAP as follows
LC_CTYPE=en_US.UTF-8 ./gap

At least on OpenBSD this fixes it:
gap> "\377";
"\377"

instead of (with LC_CTYPE=C)
gap> "\377";
"�"

In fact, even on Linux,
$ LC_CTYPE=C gap
...
gap> "\377";
"�"

Thanks @dimpase - the key was to replace LC_ALL by LC_CTYPE. Now tests pass.


Doesn't work with go_router

Hello, go_router is very popular package, now maintained by the Flutter team.
Unfortunately, Phoenix doesn't work as expected with it, you can verify with the attached source code. Most likely it is not directly related to go_router, but the underlying Navigator 2.0 (with URL for navigation).
I left the print statements to understand what happens when I call the rebirth method. The app is rebuilt, stateful Detail page is rebuilt, unfortunately, the state of the Detail page is not recreated. The very same example with imperative navigator works just fine with Phoenix.
You can see the journey I see in the console:
# run app
MyApp build
Home build
# click Go to detail
Detail init
Detail build
Home build
# click the increment button twice
Detail build
Detail build
# click rebirth
MyApp build
Detail build
Home build
# no Detail init, the increment counter stays at 2, not 0

I would appreciate your advice on how this could be resolved. I'm using Flutter stable 3.3.5 and confirmed the same issue with the most recent, go_router 6.0.0 (most recent) and flutter_phoenix 1.1.0 (most recent).
import 'package:flutter/material.dart';
import 'package:flutter_phoenix/flutter_phoenix.dart';
import 'package:go_router/go_router.dart';

final _router = GoRouter(
  routes: [
    GoRoute(
      path: '/',
      builder: (context, state) => const Home(),
      routes: [
        GoRoute(
          path: 'detail',
          builder: (context, state) => const Detail(),
        ),
      ],
    ),
  ],
);

void main() {
  runApp(
    Phoenix(child: const MyApp()),
  );
}

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

  @override
  Widget build(BuildContext context) {
    debugPrint('MyApp build');
    return MaterialApp.router(
      routerConfig: _router,
    );
  }
}

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

  @override
  Widget build(BuildContext context) {
    debugPrint('Home build');
    return Scaffold(
      appBar: AppBar(title: const Text('Home')),
      body: Center(
        child: TextButton(
          onPressed: () => context.go('/detail'),
          child: const Text('Go to detail'),
        ),
      ),
    );
  }
}

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

  @override
  State<Detail> createState() => _DetailState();
}

class _DetailState extends State<Detail> {
  int _counter = 0;

  @override
  void initState() {
    debugPrint('Detail init');
    super.initState();
  }

  void _incrementCounter() {
    setState(() {
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    debugPrint('Detail build');
    return Scaffold(
      appBar: AppBar(title: const Text('Detail')),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Text('$_counter'),
            TextButton(
              onPressed: () => Phoenix.rebirth(context),
              child: const Text('rebirth'),
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        child: const Icon(Icons.add),
      ),
    );
  }
}

Kind regards

That's a pretty old issue, but du you still face the problem ?
We use the phoenix package on an internal project, while also using the go_router package, and we didn't encounter any specific behavior issue

I had the same issues and figured out that Phoenix doesn't work with go_router if you are not on the root screen. Anyway, if you do Navigator.of(context).pop(); before restarting with phoenix, I also got an error.
My solution was to add this line before Phoenix.rebirth(context);
context.go('/'); Phoenix.rebirth(context);


Deprecated BulkIndexingTimeWindowSeconds

Documentation for: https://github.com/mattermost/mattermost-server/pull/19841

Deprecated the BulkIndexingTimeWindowSeconds configuration setting for both Elasticsearch and Bleve in favour of BatchSize
Updated config settings documentation
Added BulkIndexingTimeWindowSeconds to the list of deprecated config settings
Added v6.7 upgrade caveats

@cwarnermm, would you be open to resolving the telemetry conflict?


Odd values for humidity

Version of home_assistant
Home Assistant 2023.10.5 / official docker image
Version of the custom_component
2.0.9
Describe the bug
since Oct. 24th the entity for humidity (Station 10763) is showing strange values (>100%):

Just added another station (P257), that is now also showing those strange values.
simple_dwd_weatherforecast is getting correct values. And Forecast data OrderedDict (see log) is also holding correct values. 🤔
(I could not find any updates etc. that have anything to do with the above date.)
Debug log
home-assistant_dwd_weather_2023-10-30.log
couldn't find a clue 🤷

my bad. 🤦
Found the error. Was experimenting with the python site libs to fix the irradiance issue

Ah, ok. Thanks for checking again :)


A way to remove the date?

Use case: User has selected a date, but want's to remove/clear the date from the input.
An X or cross icon that removes the date when clicked would be great. Happy to work on this if required.

I think this would be a good addition.
Happy to look at a PR for this.

Basic version working here, happy to workshop the icon or code.
https://github.com/gkiely/vuejs-datepicker

Great thanks!
I don't think you want to nullify currDate as this should always be present and serves as a pointer for which month to open the datepicker on. You shouldn't then need to check for it's existence in v-show either.
I like the svg - but looks like there are some errors in your css? As a side note stylus does not require semi colons between property and values.

Any updates on this fixture ? I can't figure out how to use gkiely's workaround

Hi @kwnovi sorry I haven't had a chance to look at @charliekassel feedback yet.
I'll take a look tomorrow, but looks like #81 might have fixed the issue.


Added Espanso and WinSCP

https://espanso.org/
Added Espanso, an open source text expander for multiple os

@KieranRobson Sorry, I don't know why GitHub closed this automatically. Did you close it?

Woops! I must've accidentally deleted the repo when I was cleaning up my repos, I'll refork it and submit a new pull request!


ML extension - View models styles / layout updates

Revised CSS styles and added modelBuilder components to the view models stage of the View and import models.
Figma comp

This PR revises styles per 3914

Pull Request Test Coverage Report for Build 332035960

19 of 19 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 39.684%

Totals

Change from base Build 331658025:
0.02%

Covered Lines:
20623

Relevant Lines:
47713

💛  - Coveralls


Adding test for mii end to end use case

Adding test for Mii end to end use case :
create unique namespace for operator
create unique namespace for domain
create service account for operator
install operator
verify operator pod is running
create image using model file
create domain using model in image
verify domain is created
verify admin and managed pods are started and ready
verify admin and managed services are running
delete domain
delete domain image
uninstall operator
delete service account
delete domain namespace
delete operator namespace
Adding Docker Primitive class.
Calling buildDockerImages.sh from pre-integration-test phase in pom.xml

@rjeberhard i think this is ready to merge


[Issue] siteConfig settings (linuxFxVersion/appCommandLine) aren't updated incrementally

[x] Make sure you've installed the latest version using instructions in the wiki

Output from azd version
azd version 0.3.0-beta.2 (commit 91a2c014887ebda63a0af2e623278f8ac40ce376)
Output from az version
{
  "azure-cli": "2.40.0",
  "azure-cli-core": "2.40.0",
  "azure-cli-telemetry": "1.0.8",
  "extensions": {
    "containerapp": "0.3.11",
    "containerapp-compose": "0.2.2"
  }
}

Describe the bug
When I do repeated azd deploy commands with changes to properties.siteConfig in my Microsoft.Web/sites resource, I don't see the changes made in the portal. I have to instead make the changes manually in the portal.
To Reproduce

Clone this repo: https://github.com/pamelafox/msdocs-django-postgresql-sample-app-azd
Run azd up
Change the appCommandLine to something else, run azd up againappCommandLine: 'python manage.py runserver'

See that the value hasn't changed in the Portal.

Expected behavior
I expected the configuration properties to change under "General settings" in the portal, but it remained unchanged.

We have been seeing app service issues over the last couple of days.  This could be related.

Just experienced this again when attempting to change configuration settings on a function app.
Bicep change:
https://github.com/pamelafox/regression-model-azure-demo/commit/691fc74a62a2f1b903abd11c03737d3aa1a1ef4e
I had to make the configuration changes manually in the Azure Portal.

Can you please see if you can get a repro of this without azd, just using az deployment?  We are also seeing this and trying to see determine if this is an az or azd issue or rp issue.

I just tried az deployment to change the linuxVersion and it seems to have succeeded.
Bicep diff:
--- a/infra/resources.bicep
+++ b/infra/resources.bicep
@@ -71,7 +71,7 @@ resource functionApp 'Microsoft.Web/sites@2020-06-01' = {
     serverFarmId: hostingPlan.id
     clientAffinityEnabled: false
     siteConfig: {
-      linuxFxVersion: 'Python|3.9'
+      linuxFxVersion: 'Python|3.8'
       appSettings: [
         {
            name: 'APPINSIGHTS_INSTRUMENTATIONKEY'

Logs:
% az deployment group create \
    --resource-group salary-model-rg \
    --template-file infra/resources.bicep \
    -c
Please provide string value for 'name' (? for help): salary-model
Please provide string value for 'resourceToken' (? for help): v5dvp62jp2mjg
Please provide object value for 'tags' (? for help): { 'azd-env-name': 'salary-model'}
Note: The result may contain false positive predictions (noise).
You can help us improve the accuracy of the result by opening an issue here: https://aka.ms/WhatIfIssues

Resource and property changes are indicated with these symbols:
  + Create
  ~ Modify
  = Nochange
  * Ignore
  x Noeffect

The deployment will update the following scope:

Scope: /subscriptions/32ea8a26-5b40-4838-b6cb-be5c89a57c16/resourceGroups/salary-model-rg

  ~ Microsoft.Insights/components/salary-model-v5dvp62jp2mjg-appinsights [2020-02-02-preview]
    + properties.Flow_Type:      "Bluefield"
    + properties.Request_Source: "rest"

  ~ Microsoft.Web/sites/salary-model-v5dvp62jp2mjg-function-app [2020-06-01]
    + properties.siteConfig.localMySqlEnabled:   false
    + properties.siteConfig.netFrameworkVersion: "v4.6"
    ~ properties.siteConfig.linuxFxVersion:      "Python|3.9" => "Python|3.8"

  = Microsoft.OperationalInsights/workspaces/salary-model-v5dvp62jp2mjg-logworkspace [2021-12-01-preview]
    x properties.features:

        searchVersion: 1

  = Microsoft.Storage/storageAccounts/salarymodelv5dvp6storage [2019-06-01]
  = Microsoft.Web/serverfarms/salary-model-v5dvp62jp2mjg-plan [2020-10-01]
  * microsoft.alertsmanagement/smartDetectorAlertRules/Failure Anomalies - salary-model-v5dvp62jp2mjg-appinsights

Resource changes: 2 to modify, 3 no change, 1 to ignore.
Are you sure you want to execute the deployment? (y/n): y

And then when I check in the Portal and clicked JSON view for the function app, I see this in the JSON:
            "metadata": null,
            "properties": [
                {
                    "name": "LinuxFxVersion",
                    "value": "Python|3.8"
                },
                {
                    "name": "WindowsFxVersion",
                    "value": null
                }
            ],
            "appSettings": null
        },
       ...

I'll try a configuration setting change next too.

I've now tried a change to the configuration app settings. That change also went through with az deployment, though what-if didn't catch it:
Bicep diff:
diff --git a/infra/resources.bicep b/infra/resources.bicep
index 6be5b89..36d3d48 100644
--- a/infra/resources.bicep
+++ b/infra/resources.bicep
@@ -93,14 +93,6 @@ resource functionApp 'Microsoft.Web/sites@2020-06-01' = {
           name: 'WEBSITE_CONTENTAZUREFILECONNECTIONSTRING'
           value: 'DefaultEndpointsProtocol=https;AccountName=${storageAccount.name};EndpointSuffix=${az.environment().suffixes.storage};AccountKey=${listKeys(storageAccount.id, storageAccount.apiVersion).keys[0].value}'
         }
-        {
-          name: 'ENABLE_ORYX_BUILD'
-          value: 'true'
-        }
-        {
-          name: 'SCM_DO_BUILD_DURING_DEPLOYMENT'
-          value: 'true'
-        }
       ]
     }
   }

az output:
The deployment will update the following scope:

Scope: /subscriptions/32ea8a26-5b40-4838-b6cb-be5c89a57c16/resourceGroups/salary-model-rg

  ~ Microsoft.Insights/components/salary-model-v5dvp62jp2mjg-appinsights [2020-02-02-preview]
    + properties.Flow_Type:      "Bluefield"
    + properties.Request_Source: "rest"

  ~ Microsoft.Web/sites/salary-model-v5dvp62jp2mjg-function-app [2020-06-01]
    + properties.siteConfig.localMySqlEnabled:   false
    + properties.siteConfig.netFrameworkVersion: "v4.6"

  = Microsoft.OperationalInsights/workspaces/salary-model-v5dvp62jp2mjg-logworkspace [2021-12-01-preview]
    x properties.features:

        searchVersion: 1

  = Microsoft.Storage/storageAccounts/salarymodelv5dvp6storage [2019-06-01]
  = Microsoft.Web/serverfarms/salary-model-v5dvp62jp2mjg-plan [2020-10-01]
  * microsoft.alertsmanagement/smartDetectorAlertRules/Failure Anomalies - salary-model-v5dvp62jp2mjg-appinsights

Resource changes: 2 to modify, 3 no change, 1 to ignore.

Are you sure you want to execute the deployment? (y/n): y

The settings no longer are displayed in the portal in Configuration > Application settings.

I just tried reverting that diff, and az deployment once again successfully brought back the app settings, despite what-if not detecting the change.
az deployment group create \
    --resource-group salary-model-rg \
    --template-file infra/resources.bicep \
    -c
Please provide string value for 'name' (? for help): salary-model
Please provide string value for 'resourceToken' (? for help): v5dvp62jp2mjg
Please provide object value for 'tags' (? for help): { 'azd-env-name': 'salary-model'}
Note: The result may contain false positive predictions (noise).
You can help us improve the accuracy of the result by opening an issue here: https://aka.ms/WhatIfIssues

Resource and property changes are indicated with these symbols:
  + Create
  ~ Modify
  = Nochange
  * Ignore
  x Noeffect

The deployment will update the following scope:

Scope: /subscriptions/32ea8a26-5b40-4838-b6cb-be5c89a57c16/resourceGroups/salary-model-rg

  ~ Microsoft.Insights/components/salary-model-v5dvp62jp2mjg-appinsights [2020-02-02-preview]
    + properties.Flow_Type:      "Bluefield"
    + properties.Request_Source: "rest"

  ~ Microsoft.Web/sites/salary-model-v5dvp62jp2mjg-function-app [2020-06-01]
    + properties.siteConfig.localMySqlEnabled:   false
    + properties.siteConfig.netFrameworkVersion: "v4.6"

  = Microsoft.OperationalInsights/workspaces/salary-model-v5dvp62jp2mjg-logworkspace [2021-12-01-preview]
    x properties.features:

        searchVersion: 1

  = Microsoft.Storage/storageAccounts/salarymodelv5dvp6storage [2019-06-01]
  = Microsoft.Web/serverfarms/salary-model-v5dvp62jp2mjg-plan [2020-10-01]
  * microsoft.alertsmanagement/smartDetectorAlertRules/Failure Anomalies - salary-model-v5dvp62jp2mjg-appinsights

Resource changes: 2 to modify, 3 no change, 1 to ignore.

Are you sure you want to execute the deployment? (y/n): y

The settings are now back to being displayed in the portal in Configuration > Application settings.

@jongio remember you had made fixes to our Bicep modules and the app service issues. We haven't seen this lately. Can we close this pointing to the Bicep PR fix?

@pamelafox - Are you able to repro this with latest azd?

I just attempted reproducing with latest azd by changing linuxFxVersion and then appCommandLine and both changes went through! I believe this can be closed.

Okay, Thank you.


Added control to toggle the display of version information

Addressed #72 by adding the SHOW_SHYNET_VERSION variable to TEMPLATE.env, shynet/settings.py and app.json.
Added comments to demonstrate usage. Uses the environment variable to manipulate the display of the version string in shynet\dashboard\templates\dashboard\includes\sidebar_footer.html as passed by shynet/dashboard/templatetags/helpers.py.

Closing due to failed Quality Review workflow.


[native] Update add-on strings with translations

📲 What
Updating strings with correct translations, with the exception of one string that has yet to be finalized.
👀 See

📋 QA
Change system language to any of the languages we support.
Open Kickstarter app, select your profile, and then select a backed project with add-ons.
Tap manage.
Tap the overflow menu and tap update pledge, for example.
There you should see that all of the strings are translated appropriately.

Pushing against master instead of the feature branch, my mistake.


Bumps in garden : ign-common5

See https://github.com/ignition-tooling/release-tools/issues/685

I noticed the timer test on this is quite flaky.

I noticed the timer test on this is quite flaky. It's not the first time it's randomly failed. Maybe we need to make it slightly more permissive?

Yeah I noticed the same, I think this should help:
diff --git a/src/Timer_TEST.cc b/src/Timer_TEST.cc
index 70628df..81232ec 100644
--- a/src/Timer_TEST.cc
+++ b/src/Timer_TEST.cc
@@ -36,6 +36,7 @@ TEST(Timer_TEST, Sequence)
   // Start the timer
   t.Start();
   EXPECT_TRUE(t.Running());
+  std::this_thread::sleep_for(std::chrono::milliseconds(100));
   EXPECT_LT(0.0, t.ElapsedTime().count());

   t.Stop();

Opened a separate PR for the timer test: https://github.com/ignitionrobotics/ign-common/pull/343


Issue 142: User management panel with dummy data

In response to comments on https://github.com/GMOD/Apollo3/issues/149

I've bulked out all the changeManager infrastructure for the ManageUsers panel. The async function onChangeTableCell is correctly triggering when changing the role dropdown value, and the params from onCellEditCommit contains the necessary information to apply the change, but for some reason I continue to see the message Uncaught (in promise) TypeError: changeManager is undefined in the browser console. I'm not getting any linting issues, and I've added it to ManageUsersProps etc in the same way as seen in AddAssembly.tsx, so I'm a bit stumped

You'll need to add it to the props when you do queueDialog, like here: https://github.com/GMOD/Apollo3/blob/98d7833832fd9d3a185e3c7c4f0c56694ac8d562/packages/jbrowse-plugin-apollo/src/index.ts#L139-L149

On another note, the requirement for an assemblyId in all Change class descendants feels a bit redundant with User model changes (as assemblies don't come into it). Is it worth making a parallel Change class that duplicates the functionality of the original Change class without that parameter, or should that parameter requirement be moved to a child of the Change class?

I've got a branch I'm working on where Changes are split up more so that not all of them need assemblyIds. For now you can just pass any string for assemblyId.

Will try to quickly hook up the delete button with a destroy request now.

Also, this

provides motivation to include the userId as a Change attribute at the same level as assemblyId currently is (or however it will be with your in-progress code)

I've brought the getUsers function into the ManageUsers component as a callback, to allow me to perform useEffect to initialise the user data.

I've rebased onto main and made necessary tweaks to ensure it's still working


Changed to quay.io/openshift-sre org

New org for openshift sre, swapping over this to push there.

@dak1n1 added prefix of "v" to version and tweaked the make clean goal.  :eyes:  again please?


same template for different datasets

Can we share a template across datasets? this would require instead of {col1} {col2} have something more generic that can eat any input column (and maybe some meta information of it like its name) for example to have {Sentences} or {Col_name}:{Sentence}.

We saw this need also in creme brulee tream.  They want to normalize all datasets of particular type (e.g. entailment to a standard format of "premise","hypothesis","entail"  and summary datasets to "text","summary") and then reuse the same templates accross all datasets.


Allow ttl to be set as string

Hello,
based on the documentation https://www.vaultproject.io/api/auth/approle/index.html#token_ttl
you can specify integer or string,
but you scheme restrict this only to integers:
https://github.com/kr1sp1n/node-vault/blob/c1d91b9973c0f02256112bed02cf8c4c7c5d5a31/src/commands.js#L1002
Could you please change that to string? Thank you.

@azachar feel free to PR this :)
Closing for the meantime
feature request ---> Parse TTL strings and allow string type for TTL


Added support for Cognito pre token generation with access token customization

Description of changes:
Added CognitoEventUserPoolsPreTokenGenV2 event context. Version 2 of the pre token generation trigger introduces the ability to customize the access token. See https://docs.aws.amazon.com/cognito/latest/developerguide/user-pool-lambda-pre-token-generation.html#cognito-user-pools-lambda-trigger-syntax-pre-token-generation
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

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

Comparison is base (30d7c7e) 72.93% compared to head (561c62a) 72.93%.

:exclamation: Current head 561c62a differs from pull request most recent head 849ad2a. Consider uploading reports for the commit 849ad2a to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #538   +/-   ##
=======================================
  Coverage   72.93%   72.93%
=======================================
  Files          26       26
  Lines        1456     1456
=======================================
  Hits         1062     1062
  Misses        322      322
  Partials       72       72

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


Add Github Action for testing on darwin

See https://github.com/Senzing/template-go/issues/76

Pull requests:

https://github.com/Senzing/explain/pull/22


Support for Lake Formation tag inheritance

We apply Lake Formation tags at the database level that are subsequently inherited by tables and columns. Currently dbt-athena attempts to remove all tags from tables, even those that are inherited, preventing our Lake Formation usage pattern from working.
The GetResourceLFTags API call returns full information sufficient to infer tag heredity for a given resource, so it would be possible for dbt-athena to not attempt to remove inherited tags this way. They could still be overridden when present at the table or column level in DBT metadata.
Alternatively a simple list of tags intended for inheritance only and not override could be supplied and ignored during the "remove all" phase of applying tags. This supports our use case where DBT principals don't even have permission to assign the tags in question.

@dacreify could you please describe your lakeformation + dbt usage pattern ? I'm just not sure if it's OK to have 2 points to manage lf tags: dbt and terraform for instance. Why don't you use only 1 tool for that ?
In my practice, I use only dbt for tags management and there are no issues with such approach. I'm wondering if having 2 management tools will lead to some mess.

@svdimchenko Sure I can elaborate. We use Terraform to provision our Glue databases for dbt-athena as well as DB-level tag associations for them, and we use DBT to manage table and column level tags. There's a particular tag we use for isolating data access into channels that's always applied at the DB level and should never be overridden. The principal that runs the dbt-athena project doesn't even have permission to associate this tag so that it's impossible to break the boundary between two channels by either forgetting to assign the tag or assigning it incorrectly. Without the change in the PR we therefore just get permission denied type errors from the Lake Formation API as when dbt-athena tries to remove the inherited tag.

I see. The question is if you put this inherited tag into the models config (it can be done in dbt_project.yml generally for all models or set by schema) you don't need to add inherited tags behaviour. And you won't face permissions error as these tags will be set on model's level. Did I miss smth ?

I have a similar setup to @dacreify - We do not want to use dbt to create a database and assign the main lf tags to the dbs, because we simply have too many, and we can simply do with some loops in terraform or cdk.
To work around what @dacreify, our analytics engineers (and dbt IAM role) is allowed to assign table and columns tags.
@svdimchenko but I believe that what is being raised here is really relevant, we cannot force people to use one single approach to handle lf tags.

@nicor88 I use this approach as well. I setup lf-tags for databases with terraform and lf-tags for table and columns with dbt.
And I agree with you and @dacreify this feature will bring us more flexibility for lf-tags management. I just was wondering if that will not lead to mess somehow.

I just was wondering if that will not lead to mess somehow.
@svdimchenko you have a good point, indeed, we need to be sure about the proposed approach - I didn't deep dive yet on what is proposed in the codebase.

The question is if you put this inherited tag into the models config (it can be done in dbt_project.yml generally for all models or set by schema) you don't need to add inherited tags behaviour.

My understanding is that the lf_tags_config config is not merged when applied at multiple levels but rather overridden. For example if I set my channel tag at the project level and then want to have further tags defined at the model level the latter will overwrite the former and my project-level tag will be lost. (This is why I made lf_inherited_tags a new top-level config instead of nested it inside lf_tags_config.)
Also our access control scheme requires that there's no way principals running DBT can override the database-level tag, so the permissions errors are actually as intended.
I tried to make my solution as narrow as possible to just support the case of wanting to inherit database-level tags that are never overridden by DBT metadata and spelled this out in the docs.


[BugFix] Fix fold str_to_date with %f failed

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

starrocks_be_unittest succeeded.

[FE PR Coverage Check]
:heart_eyes: pass : 7 / 7 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/util/DateUtils.java
4
4
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/operator/scalar/ConstantOperator.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rewrite/ScalarOperatorFunctions.java
2
2
100.00%
[]

starrocks_be_unittest succeeded.

[FE PR Coverage Check]
:heart_eyes: pass : 7 / 7 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/util/DateUtils.java
4
4
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/operator/scalar/ConstantOperator.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rewrite/ScalarOperatorFunctions.java
2
2
100.00%
[]

run starrocks_admit_test


Need documentation on variations of { } content for formatting strings.

Please consider documenting the different things that can go between the {} in formatting strings.
Examples are always great for this if they can be packed into the documentation.  Thanks !!

This documentation now exists here!


added Open States government links

Open states is a great open source team that keeps track of all US State legislations.

Hey @jschiarizzi sorry for the delay. Those seem cool but they need to link directly to the .json file like the others. I'll leave it open for now.

Oh, alright. I'll try to fix that later


Images not coming in pop-over track

src fields are blank. gotta fix track folding

are you working for the same already ?

I am not seeing it corrected after fixing the jsons

I will do this. You can work on the other views. I have to look at the JSON formats for v2 migration, and also for downloading of images and audios anyway.

cool :)

Thanks @championswimmer  Solved #338


Refactor codebase to async/await - spo util

Closes #5079
Sorry that it took so long, but I finally got some time again. Ready to pick up the contributions again!

Thank you for the PR @MathijsVerbeeck, we're glad you're back! We'll try to review it ASAP!

Could you resolve the merge conflicts, please?

Could you fix the merge conflicts please?

ouch @MathijsVerbeeck apparently we again have some conflicts. Sorry for that. May I kindly ask you to sort it out before we move forward?

No worries! Conflicts have been resolved!

Ready to merge 🚀 .
After the next minor release

merged manually 👏


Reactive examples and integration test

I am not as familiar with Reactive but an example in the docs and an Integration test would be great for someone to add as a good first issue.
https://quarkus.io/guides/resteasy-reactive#multipart

The problem with the reactive part is that the AntivirusEngine signature is not ready to implement the feature.
There is not a easy way to implement it without a breaking change.
Reactive signature example with Mutiny:
package io.quarkiverse.antivirus.runtime;

import java.io.InputStream;
import io.smallrye.mutiny.Uni;

/**
 * Service provider interface for virus scanning that might be used in file upload component for example when dealing with
 * untrusted files.
 */
public interface AntivirusEngine {

    /**
     * Indicate whether this {@link AntivirusEngineReactive} is enabled or not.
     *
     * @return true if enabled, false otherwise
     */
    boolean isEnabled();

    /**
     * Perform virus scan and throw exception if a virus has been detected.
     *
     * @param filename the name of the file to scan
     * @param inputStream the inputStream containing the file contents
     * @return the {@link AntivirusScanResult} containing the results
     */
    Uni<AntivirusScanResult> scan(final String filename, final InputStream inputStream);
}

To implement to reactive engine, the easy way is maybe to use Vertx. But we can also use CompletionStage.

@ggrebert PR is welcome.  This is still a BETA library only at 0.0.4 so I am not worried about breaking changes.


Dataset split problem in exchange rate

In actual operation, it was found that when the output length of exchange_rate data set was 720, the sample number of training set was 4496, the sample number of verification set was only 41, and the sample number of test set was 798. Is this data division not very reasonable (the number of verification sets was too small)? Because the size of the sample varies too much in this ratio.

Yes, the validation set is too small.
I think you can change the ratio of train and val dataset to balance model training and model validation. It is a possible way to  improve the model perfromance.


Feature request: Unified Diff

Would it be possible to provide the option to view unified diffs in addition to side-by-side?

What do you mean "unified diffs"?

https://en.wikipedia.org/wiki/Diff_utility#Unified_format

Isn't this currently supported? if there's a bug of parsing some diffs, it'll help if you can provide the case. Thanks!

The extension supports viewing patch files which are in the unified diff format, yes. But the view is changed to side-by-side from unified. The feature request is to enable coloured highlighting of the patch file, with not transformation. It's much clearer and easier to peruse when there's visual distinction between the lines that have been added/removed/modified are colour-coded when viewing the patch file.

oic, so just colorize the original file, right?

Yes, that's right.


Deleteing node_modules causes "You seem to not be depending on<EMAIL_ADDRESS>

- [x] feature request

ng -v
@angular/cli: 1.0.6
node: 6.10.3
os: win32 x64
@angular/common: error
@angular/compiler: error
@angular/core: error
@angular/forms: error
@angular/http: error
@angular/platform-browser: error
@angular/platform-browser-dynamic: error
@angular/router: error
@angular/cli: error
@angular/compiler-cli: error

Repro steps.

install npm
yarn global add<EMAIL_ADDRESS>or npm install -g<EMAIL_ADDRESS>(both produce the same)
ng new myproj
ng serve - works
check-in to your source-control all the project files except for node_modules directory
5.1. one can simply run move node_modules node_modules2 instead of steps 5+6
go to a different computer and run steps 1+2. then, fetch the project from the source-control.
ng b  -> produces: You seem to not be depending on "@angular/core". This is an error.
ng serve -> produces:

The "@angular/compiler-cli" package was not properly installed.
Error: The "@angular/compiler-cli" package was not properly installed.

Only yarn add or npm install will resolve this, which is somewhat unexpected (I am in the Angular CLI realm now, trying to build, why won't it solve my dependencies itself?)
Desired functionality.
I expected the angular-cli to successfully build the project and download all the dependencies.

@EekoS It's the desired behavior you must install the node modules before you run the project. It's not specific to @angular dependencies as there might be other dependencies needs to be resolved before running the project.

facing same issue. Any solution for these

@EekoS For current workaround I change version to 0.0.0 and it working fine. This may be a bug.

@shabbir-dhangot Is there a repo I can check?

Thank you for the reply, @sumitarora, I humbly don't see a reason why won't angular-cli simply call the package-builder (npm / yarn) to resolve dependencies when running ng build; but at least is it possible to write a more informative error?
There's no problem with angular's global installation (as it says in my step 7 to reproduce) - so the error should not say that there is IMHO.

@EekoS we have error already in the commands itself. Running ng b without node_modules shows this error node_modules appears empty, you may need to run \npm install``

nope, that's not what I get. see the original post...

I ran into this same issue deploying an app using ng-cli to Heroku. On the Heroku deploy, I was getting the You seem to not be depending on "@angular/core". message on ng build, which was very confusing, because it was right there in my package.json file. I'm glad I found this issue and was able to apply the workaround of making sure npm install was run before ng build. But, a more informative error message would be very helpful...

@sumitarora: like you, when I rename my node_modules directory and run ng build, I get the correct error message: node_modules appears empty, you may need to run npm install. But when I deploy to heroku, and the node_modules directory is empty, for some reason I'm getting the misleading message You seem to not be depending on "@angular/core". instead. I've tried to figure out what the difference is, but I'm unable to. Sorry, I realize that's not extremely helpful...

lo solucione con:
-> npm link
-> ng serve
y listo

npm link fixed mine. windows 10 here. latest angular cli as of now.

npm link
Helps to solve my problem.

solution:
->npm install
->ng serve


Documentation

The purpose of this branch is to allow me to click Sync Fork > Discard Commits on the base branch to pull down changes from upstream without having to resolve merge conflicts. After that action, this branch gets rebased onto the base branch and merged.
As of this writing, this branch only contains documentation changes that were rejected by the upstream author for valid reasons that I disagree with, but which I believe belong in our organization. If nothing else, I want these notes for myself. Using a wiki page or a document in the engineering repo may be a better solution, but they already existed here. When someone clicks on this repo in our organization, they will wonder how it works and why we use it. They should not have to go looking for this information.

Sorry, I meant to open this against my fork...not your repo. My bad.


K8SPXC-1142: Remove FLUSH PRIVILEGES

Test name
Status

cross-site-8-0
passed

haproxy-5-7
failed

haproxy-8-0
passed

init-deploy-8-0
failed

proxysql-sidecar-res-limits-8-0
failed

recreate-8-0
failed

restore-to-encrypted-cluster-8-0
failed

scaling-8-0
passed

scaling-proxysql-8-0
failed

scheduled-backup-8-0
failed

smart-update1-8-0
failed

storage-8-0
failed

tls-issue-self-8-0
failed

upgrade-haproxy-8-0
failed

We run 14 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1269/commits/fe5b615df70cad3129640fbcf8146de4d5b2e0c4
image: perconalab/percona-xtradb-cluster-operator:PR-1269-fe5b615d

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-8-0
passed

demand-backup-cloud-8-0
failed

demand-backup-encrypted-with-tls-8-0
failed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
failed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
failed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-8-0
failed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-8-0
passed

validation-hook-8-0
passed

We run 33 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1269/commits/f811a44c9877a76d9e28203faeffba23903347dd
image: perconalab/percona-xtradb-cluster-operator:PR-1269-f811a44c

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-8-0
passed

demand-backup-cloud-8-0
failed

demand-backup-encrypted-with-tls-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
failed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-8-0
passed

scaling-proxysql-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1269/commits/f811a44c9877a76d9e28203faeffba23903347dd
image: perconalab/percona-xtradb-cluster-operator:PR-1269-f811a44c

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
failed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-8-0
passed

scaling-proxysql-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1269/commits/f811a44c9877a76d9e28203faeffba23903347dd
image: perconalab/percona-xtradb-cluster-operator:PR-1269-f811a44c


Scaling / magnification on HiDPI

Can you please provide some kind of scaling / magnification of the whole window? Text and everything gets really, really small on 4k displays.
Here is for example comparison with standard font sizes in my terminal:

Or full size for getting the idea:

And here are some measurements:

I've tried several external ways how to increase the magnification of whole window on linux, but none of them have been successful.
Some way of just saying "scale this window at 2x the resolution" would be really useful. My eyes literally hurt when trying to work with GT at the moment.

Thanks for the report. On what operating system are you?

Linux Mint 20 Ulyana with Cinnamon.

We added and ability to scale up/down the UI independent from the scale factor provided by the OS:

In addition we added the How to scale the UI document which can be found on the home page:

Let us know if it improves the situation for you!

We added and ability to scale up/down the UI independent from the scale factor provided by the OS:

In addition we added the How to scale the UI document which can be found on the home page:

Let us know if it improves the situation for you!

Wow, thank you very much, this works wonderfully.

Wow, thank you very much, this works wonderfully.


Tabular: Refactored tabular_nn_model and tabular_nn_trial

Issue #, if available:
#266
Description of changes:
Major Changes:

Reworked TabularNN HPO to be consistent with other models, fixed various defects in TabularNN HPO.
Reworked TabularNN to minimize unnecessary variables and avoid alteration of original self.params during .fit(). Simplified code structure.

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

Job PR-338-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-338/1/index.html

Job PR-338-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-338/2/index.html


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/106841

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/108108

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/120055


sharedlibraries: Remove duplicated log wrapper

What does this PR do?
start is a private method that is being called only from InitWithLibsets In case we failed to start the manager we wrap the error twice with the same prefix of 'cannot start manager', once in the start method, and again in InitWithLibsets. Removed the wrapper in the start method, and allowing the caller wrap the error instead
Motivation
Better log messages
Describe how you validated your changes

Possible Drawbacks / Trade-offs
Additional Notes

/merge


Unification of for-loops with the same codegen, fixing of their short-circuit evaluation and usage of none with them

for .. in and for .. of loops now use the same codegen for their bytecode using 2 instructions ITER_CONDITION and ITER_END. ITER_CONDITION evaluates each iteration whether the condition of the loop was satisfied prematurely or whether the loop is still satisfiable, so essentially making sure that loops are properly short-circuited. ITER_END is used for final evaluation of the loop, whether it is satisfied or not, after the iterator got exhausted or the loop was short-circuited.
In order to be able to use single codegen for both iterators and string sets, new iterator type was instroduced - YR_ITERATOR_STRING_SET. When ITER_START_STRING_SET is executed, it is capable of consuming all strings pushed on the stack and construct YR_ITERATOR_STRING_SET object which can then be used as a regular iterator.
Currently, I'm not completely satisfied with the fact that the logic for short-circuiting and special handling is separate for OF, OF_FOUND_IN, OF_PERCENT and these for-loops. It would be the best if each of them used these new ITER_CONDITION and ITER_END instructions but I'm not sure whether someone else shares this sentiment with me.
Before I proceed with any further improvements of existing instructions, I wanted to know what would you tell me about this and if there's anything I should improve before this idea is brought even further.

Great PR!. My only comment is that it requires incrementing YR_ARENA_FILE_VERSION in arena.h, so that compiled rules generated by this version won't be loaded by older versions of YARA that won't recognize the new instructions.
I also like the idea of unifying OF, OF_FOUND_IN and OF_PERCENT with this for loops as much as possible.


Fechando o Pull Request.


FileNotFoundError

I ran into an issue when creating a new project, If I don't import any frames but I reopen the project I get the error
FileNotFoundError: "/app/Pose-app/.shared/data/test6/labeled-data"

closed by #53


Clarify PEAPS

PEAPS is mentioned in the diagram, but not in the text. Add it to the text with a brief explanation so it is clear how it matches the diagram.

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

ID: d1892de7-0902-054b-e430-8e23f714477b
Version Independent ID: 20e0ff84-5f29-f5e3-ddde-288e52059ad0
Content: Standard and targeted platform releases - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/fin-and-ops/get-started/public-preview-releases.md
Service: dynamics-ax-platform
GitHub Login: @kfend
Microsoft Alias: kfend

Bug 238418 was logged for tracking purposes.

Closing this issue since it's already fixed.


'groovyw * list' output condensed format

Contains
Addresses condensed output format for long lists mentioned in #3279
How to test
Run: groovyw module list
Long lists are presented like below. Currently output format depends only on list size (threshold is hardcoded to 50). I think it would be a good idea to allow user override it and explicte specify which format he wants to use.

Nice !
The non-condensed view could be forced using flags. I think that would be noce to have

Hi @digitalripperynr, I've added the flags and described them in the usage section.

Awesome ! I'll test them after my exams are over :)

Thanks again for working on the Groovyw Utility !
The new format is working nicely and I've got 2 suggestions :
1.
In the condensed view, the groups should be separated by spaces to make them look more clean.

I think the -default-list-format flag is not really required and could be removed.
Apart from these two things the changes work nicely and the flags are working properly.

Thank you so much for the work @mwesolowski ! And the review @digitalripperynr :-)
I tweaked the usage text while merging to shrink it a bit and take out the default option, but leaving it in as an undocumented feature in case it should ever prove useful for some reason.
The spaces turns out after a quick chat to be a display issue on mobile so that's good as-is
Merging and adding to credits, thanks again!


Fix incorrect event ordering

Resolves #985
This patch changes the way CallInfo structure is traversed when contract events are aggregated.
Before:

Emit current call events
Then recursively handle all inner calls

After:

Emit current call events while their orders are sequential (increase by one)
Otherwise try to find the continuation of the sequence in the next inner call
Repeat again until all events & inner calls are handled

Notes:

We trust blockifier that in every external/inner call events will be sorted and form a correct sequence

I will merge as soon as the CI is fixed


Added E2B to community servers

Description
Adds E2B MCP Server to community section
Server Details

Server: E2B MCP Server
Changes to: -

Motivation and Context
Run code in Claude
How Has This Been Tested?
With Claude.app, locally
Breaking Changes
None
Types of changes

[x] New MCP Server
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation update

Checklist

[x] I have read the MCP Protocol Documentation
[ ] My server follows MCP security best practices
[x] I have updated the server's README accordingly
[x] I have tested this with an LLM client
[ ] My code follows the repository's style guidelines
[ ] New and existing tests pass locally
[x] I have added appropriate error handling
[x] I have documented all environment variables and configuration options

Additional context

Ok, done so


Optional dynamic segments aren't allowed

I'm trying to do optional dynamic segments like this: /($foo)
This spits out an error saying the route prefix must come first.
Tried putting the $ first and got this for the output route:
<Route path="reports/:(type)/:id" file="routes/reports_.$(type).$id/_route.tsx" />

I don't know if it's intended but I was able to get it working with (:type), eg. reports.(:type).$id.tsx.

I don't know if it's intended but I was able to get it working with (:type), eg. reports.(:type).$id.tsx.

Seems like a happy accident. 😆

It's a bug in flat routes... it expects the segment to start with $. Optional segments change that, so should support ($foo). I'll try and get this fixed today.

Fixed in v0.5.5


Improve performance by ~60%

Description
This PR adds ~80 bytes to the final bundle by refactoring sources (not targets) to use arrays instead of LinkedList. But it makes it up by improving performance by 60-69% 😅
TL'DR I want benchmarks
Initial benchmarks:
Signals Local - Simple x 19,296,752 ops/sec ±0.83% x 15 Bytes/op (928725 runs sampled)
Signals Local - Unchanged Sources x 84,583 ops/sec ±0.38% x 12 Bytes/op (23931 runs sampled)
Signals Local - Conditional Sources x 161,475 ops/sec ±0.59% x 1.27 KB/op (44140 runs sampled)

Signals  Core - Simple x 18,081,537 ops/sec ±0.46% x 15 Bytes/op (917966 runs sampled)
Signals  Core - Unchanged Sources x 52,064 ops/sec ±0.29% x 18 Bytes/op (14879 runs sampled)
Signals  Core - Conditional Sources x 95,471 ops/sec ±0.41% x 7.75 KB/op (26329 runs sampled)

Measure in all benchmarks in this function:
function measure() {
	count.value++;
	double.value;
}

Each benchmark sample runs in a separate process and uses the measure method above:
// Simple
const count = signal(0);
const double = computed(() => count.value * 2);

// Unchanged sources
const s = [];
for (let i = 0; i < 1000; i++) {
	s[i] = signal(i);
}
const count = s[500];
const double = computed(() => {
	for (let i = 0; i < 1000; i++) {
		s[i].value;
	}
	return count.value * 2;
});

// Conditional sources
const s = [];
for (let i = 0; i < 1000; i++) {
	s[i] = signal(i);
}
const count = s[500];
const double = computed(() => {
	const mod = count.value % 10 === 0;
	const from = mod ? 0 : 601;
	const size = mod ? 500 : 1000;

	for (let i = from; i < size; i++) {
		s[i].value;
	}
	return count.value * 2;
});

The code is documented explaining every decision but here is some more info.
Why?
It's faster to iterate over arrays than it is to iterate over a LinkedList. Pretty often, sources of a "computed" or "effect" remain the same:
const c = computed(() => list.value.filter(stuff));

Based on this idea, I implemented sources as how hooks work in React/Preact. Every node uses a slot/index in the sources array. The execution roughly looks like this:
sources = [node1, node2, node3]

prepareSources -> for each node._version = -1, set index = 0

addDependency node1 -> [node1, node2, node3]
                          ↑ node1._version = 0, index += 1
addDependency node2 -> [node1, node2, node3]
                                 ↑ node2._version = 0, index += 1
addDependency node3 -> [node1, node2, node3]
                                        ↑ node3._version = 0, index += 1

cleanupSources -> for each node, rollback node

This is the best case scenario when sources do not change after re-computation so all there operations are fast.
An advantage of using arrays is that the JS engine allocates a capacity only when it's needed and not every time a new item is added. This means that, in this particular example, while the length is 3, the array has in fact more capacity for new items. This means, there is already enough capacity for new sources if the computed/effect has conditionals.
This opens the possibility to re-use already allocated capacity to push old dependencies - that need to be cleaned up - at the end of the array:
sources = [node1, node2, node3]

prepareSources -> for each node._version = -1, set index = 0

addDependency node1 -> [node1, node2, node3]
                          ↑ node1._version = 0, index += 1
addDependency node4 -> [node1, node4, node3, node2]
                                 ↑             ↑ node2 scheduled for clean up
                                 │
                                new node, index += 1
addDependency node3 -> [node1, node4, node3, node2]
                                        ↑ node3._version = 0, index += 1

cleanupSources -> for each node, rollback node
               -> for each node from index to length, unsubscribe
               -> set length of array to 3
               -> [node1, node4, node3]

Even though the length is set to 3, the internal capacity of the array doesn't shrink. In case of V8, the array doesn't shrink for small arrays or it shrinks if 50% of the capacity is unused for large arrays.
This greatly improves performance for most common cases when sources do not change but also improves performance for computed/effect with conditionals when the number of new sources + cleanup nodes doesn't exceed the internal already allocated capacity of the array or if the number of sources decreases by no more than 50% (for small arrays is makes no difference, capacity won't shrink)
Next
This is strictly speaking about V8 but from my understanding similar optimizations are made in JavaScriptCore and SpiderMonkey. The benchmarks were run in Node.js (V8) so I still need to benchmark how this looks like in other JS engines. 😅
Opening this PR for discussion meanwhile 😄
Edit: benchmarking with v8 and spidermonkey binaries:
v8
====================================
! Signals  Core - Simple x 22,009,361 ops/sec ±0.69% (5214020 runs sampled)
Signals  Core - Unchanged Sources x 60,536 ops/sec ±0.31% (17772 runs sampled)
Signals  Core - Conditional Sources x 91,436 ops/sec ±0.30% (26858 runs sampled)
Signals  Core - Computed + Effect x 58,906 ops/sec ±0.31% (17266 runs sampled)

Signals Local - Simple x 19,180,400 ops/sec ±0.37% (4539201 runs sampled)
! Signals Local - Unchanged Sources x 83,790 ops/sec ±0.28% (24567 runs sampled)
! Signals Local - Conditional Sources x 156,461 ops/sec ±0.28% (45920 runs sampled)
! Signals Local - Computed + Effect x 59,025 ops/sec ±0.31% (17344 runs sampled)

SpiderMonkey
====================================
! Signals  Core - Simple x 10,018,443 ops/sec ±0.12% (2692485 runs sampled)
Signals  Core - Unchanged Sources x 25,441 ops/sec ±0.33% (7479 runs sampled)
Signals  Core - Conditional Sources x 50,539 ops/sec ±0.30% (14892 runs sampled)
! Signals  Core - Computed + Effect x 20,169 ops/sec ±0.34% (5921 runs sampled)

Signals Local - Simple x 9,264,110 ops/sec ±0.11% (2479468 runs sampled)
! Signals Local - Unchanged Sources x 35,002 ops/sec ±0.31% (10272 runs sampled)
! Signals Local - Conditional Sources x 74,420 ops/sec ±0.26% (21852 runs sampled)
Signals Local - Computed + Effect x 15,839 ops/sec ±0.42% (4651 runs sampled)

"Computed + Effect" is the same as "Unchanged Sources" but has an effect that always reads the computed value.
Also, these benchmarks use performance.now (converted to ns) and not process.hrtime.bigint because neither v8 not SM have a process.hrtime so there is less accuracy. Overall, it seems the biggest improvement in performance is seen when using conditionals (different sources on every re-compute).

Closing in favor of #265
After fixing a couple of issues I haven't considered, the performance isn't that impressive anymore (at least to justify the +80 bytes) lol - PR #265 performs better and it's more stable across different use case than the changes in this PR e.g: the simple case benchmark performs slightly worst here and most benefits seem to be seen only with big arrays while PR #265 performance gains can be seen in simple cases as well as computed/effect with thousand sources.


Example SP Pipeline using multiple assets

Aim:
To create an example pipeline in a databricks notebook which does the following:

Combines a set of woodland assets into a single xarray DataArray/Dataset

Woodland assets can be found at the following location - /dbfs/mnt/lab/unrestricted/source_sds/l1_woodland

Collapse the assets into a single 1/0 binary array where 1 indicates the presence of any asset and 0 none.
Acquire the 'SHEET_ID' and 'PARCEL_ID' attributes for features in the RPA reference parcels dataset which intersect the combined asset array.

RPA reference parcels can be found at the following location - /dbfs/mnt/base/unrestricted/source_rpa_spatial_data_mart/dataset_rpa_reference_parcels/format_GPKG_rpa_reference_parcels/LATEST_rpa_reference_parcels

Dan's gpkg trick:
import pyogrio
import shutil
import tempfile

with tempfile.NamedTemporaryFile(suffix='.gpkg') as tmp:
    shutil.copy("/dbfs/mnt/base/unrestricted/source_rpa_spatial_data_mart/dataset_rpa_land_cover/format_GPKG_rpa_land_cover/LATEST_rpa_land_cover/land_cover.gpkg", tmp.name)
    data=pyogrio.read_dataframe(tmp.name, where="LAND_COVER_CLASS_CODE in ('110', '113', '118', '140')")

data['geometry'] = data['geometry'].to_wkb()

data=spark.createDataFrame(data)

(DataFrameWrapper(name='rpa_landcover', data=data, metadata=None)
 .index()
 .to_zarr(
<EMAIL_ADDRESS>     data_array_name="rpa_parcel_location_mask"))

After discussion with @jpd-defra i'm transferring this issue to sds-asset-recipes


Unit of temperature contains special character

When recently updating a project to common-definitions I was surprised to see that we use °C as the unit of temperature instead of Celsius.
I would advocate for using Celsius since it avoids using special characters.
Would love to hear your thoughts @IAMconsortium/common-definitions-coordination.

Agree, °C is difficult. We could use either K or Celsius

I don't like Celsius alone because I think degree should somehow be part of the unit, but strongly discourage using °. I would prefer K as it is clean, short and used by the MAGICC reporting.

One consideration is that this unit will be used as default in visualizations like the Scenario Explorer, so I'd keep something that is as intuitive as possible for a wide user group - so not K.
There is known issue in the ixmp package which made using °C a bit cumbersome, but this will be solved as we move to ixmp4. And unicode-problems should also be not a major concern in python/R/csv...
My personal preference here would be to only make changes when there is a strong case for it (which I don't quite see).

I remember having some issues opening IIASA csv data files in Excel if they were written by some Linux system before (the units showed as \u00B0C or one times even \u00C2\u00B0C). So that is some concern.
And we already moved away from US$2010 (AR6 unit) to USD_2010to avoid special character issues, and$` is even part of ASCII.

I remember having some issues opening IIASA csv data files in Excel if they were written by some Linux system before (the units showed as \u00B0C or one times even \u00C2\u00B0C).

This will be fixed with migrating to the ixmp4 infrastructure.

And we already moved away from US$2010 (AR6 unit) to USD_2010to avoid special character issues, and$` is even part of ASCII.

This was specifically changed to allow unit-conversion with the iam-units package (https://github.com/IAMconsortium/units), because the underlying pint package does not allow $. But I don't see why we would convert units regarding temperature.

This will be fixed with migrating to the ixmp4 infrastructure.

Might be, on your side, but we always had issues in our mappings of REMIND to project variables with it that needed fixing. Might be similar for other people either working with the data or preparing submissions. I would be really happy if it ° went away.


Compatibility problems with netty-socketio and socket.io

Netty-socketio not compatible with all versions after socket.io-1.4.8 .

咱也不知道 咱也不敢问


random crash

Issue description:
randomly crashing
Version (make sure you are on the latest version before reporting):
Forge: <IP_ADDRESS>35 272
Mekanism: 9..7
Other relevant version:
https://paste.dimdev.org/fomiyupewo.mccrash

Duplicate of #5392 and fixed by #5399


Feature: add the ability to optimize already existing ImageVector

I dont know if this is possible but i would like a feature like this to optimize your vectors. maybe reducing number of instructions in path or smth like this

Hello, will be covered with this huge task 🙂
https://github.com/ComposeGears/Valkyrie/issues/154


Dotzu + Alamofire

Hello,
I'm using Alamofire to handler my requests, and in your tutorial  you mentions it:
**/Your custom configuration
let configuration = URLSessionConfiguration.default
///Add the network logger on the configuration
Dotzu.sharedManager.addLogger(session: configuration)
//Use it:
//For Alamofire
let sessionManager = Alamofire.SessionManager(configuration: configuration)
//For URLSession
let session = URLSession(configuration: configuration)**
But I didn't use session in my requests, I make it directly ex:
Alamofire.request(url, method: .put, parameters..............
And it not log request on Doutzu, I think that it is because of session, How can I set a session in Alamofire manager??
Thx

you should create a session manager and use session manager instead of alamofire directly, dotzu need to observer the session configuration that you set.


Add Radix sort Algorithm

Detailed description
I want to add a radix sort algorithm in sorting in C++ programming language with step-by-step algorithm steps and code implementation.
Context
Radix sort is also a type of sorting algorithm in c++
Possible implementation
No response
Additional information
No response

@silky-modi22 please assign this issue for me

@silky-modi22 please can you assign me this issue ?

Hey @silky-modi22  Can you assign me this issue?

@silky-modi22 , you have already submitted a PR #2566 on this. Why are you raising this issue ?


Upload 0.0.4 to Cargo

Could you please go ahead and upload 0.0.4 to Cargo? Thanks.

Done. Thanks!


[Authy] Invalid base32 characters on incorrect backups password

Extension
https://www.raycast.com/guga4ka/authy
Description

Error:
Error: Invalid base32 characters

uu:OtpListItem.tsx:72:38
fu:search-otp.js:42:4668
Ir:index.js:5:5486
    at ray-list

Steps To Reproduce

Login to Authy
Use a correct user ID but an incorrect backups password
Complete the authorization steps and successfully setup the extension
Search for any token in your account

Current Behaviour
Something went wrong popup
Expected Behaviour
An error explaining that the password was incorrect

@raycastbot close this issue
duplication #5880


[docgen] Support for overview documentation.

This PR adds support for overview documentation to the documentation generator.
An overview is provided via a template file in markdown format. This markdown is subject to all the processing we do for Move documentation comments (automatic referencing, highlighting, and section organization). The template can also contain placeholders for generated documentation content:
> {{move-include NAME_OF_MODULE_OR_SCRIPT}}
> {{move-toc}}
> {{move-index}}

See a strawmann overview template for transaction scripts (rendered form) for more details.
The implementation also works for overviews for modules in the Libra framework, and a strawman template has been added there as well. Even without further documentation, the automatically genearted index of all scripts/modules is useful.
Technical details:

New default for documentation generation is to place specs together with implementations instead of a separate section. This is for now a flag-controlled experiment. Bugs needed to be fixed to make this unused feature work.
Boilerplate for script documentation has been removed, exploiting that they can contain only one function.
Removed the restriction that the spec-lang enviroment can contain only one script. This makes generation of script documentation more efficient because they can be all done in one run.

Motivation
Better documentation.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Adapted existing tests.
Related PRs
NA

This is amazing! Thank you so much for doing this.
Only question I have is whether or not we want included documents to be "expandable" instead of inlined? I can see arguments for both (and am fine with either!), but am interested in your and @sblackshear 's thoughts on this.

Good question. It seems nice to support both (e.g., {{doc}} could mean expandable and {{{doc}}} could mean inlined).

This is amazing! Thank you so much for doing this.
Only question I have is whether or not we want included documents to be "expandable" instead of inlined? I can see arguments for both (and am fine with either!), but am interested in your and @sblackshear 's thoughts on this.

Good question. It seems nice to support both (e.g., {{doc}} could mean expandable and {{{doc}}} could mean inlined).

One problem is that the collapsing doesn't seem to work on Github for navigation. So if you have something like a section within the collapsed part, and click on it in TOC, nothing seem to happen. We can investigate this further as a followup. Another option would be to include only a summary and the link for the full doc in a standalone file.

/land

One problem is that the collapsing doesn't seem to work on Github for navigation. So if you have something like a section within the collapsed part, and click on it in TOC, nothing seem to happen. We can investigate this further as a followup. Another option would be to include only a summary and the link for the full doc in a standalone file.

Sounds good to me and also a reasonable alternative for now. Happy to get this in as-is, and we can circle back later to see if there are any alternatives if we want to.


Cache Issue

While posting a new measurement i got this error "TypeError: Cannot read properties of undefined (reading 'length')"; It's caused by the measurementSchema.Pre function: it gets the feature from the cache but sometimes it's missing of the .items field.

Solved: Added "_thing", "_tag", "_device" at the end of the keys when saving data in the cache.


Permission denied

Cloning into '/backup/git'...
load pubkey "/backup/.ssh/id_rsa": Permission denied
Host key verification failed.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
Cloning into '/backup/git'...
load pubkey "/backup/.ssh/id_rsa": Permission denied
Host key verification failed.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

I have already mount the ssh secret but it seem the "backup" user have no permission to read /backup/.ssh

          volumes:
          - name: cache
            emptyDir: {}
          - name: sshkey
            secret:
             secretName: xxx
             defaultMode: 0420

Please make sure you have the correct access rights
and the repository exists.
Cloning into '~/git'...
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
@         WARNING: UNPROTECTED PRIVATE KEY FILE!          @
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
Permissions 0420 for '/root/.ssh/id_rsa' are too open.

Have you ever tested it on your own kubernetes cluster ?

I have:
    volumeMounts:
              - mountPath: /backup/
                name: cache
              - mountPath: /backup/.ssh
                name: sshkey
              - mountPath: /secrets
                name: gpgkey
            dnsPolicy: ClusterFirst
            nodeSelector:
              fs.evry.com/finods-group: aot
            restartPolicy: OnFailure
            schedulerName: default-scheduler
            securityContext: {}
            serviceAccount: kube-backup
            serviceAccountName: kube-backup
            terminationGracePeriodSeconds: 30
            volumes:
            - name: sshkey
              secret:
                defaultMode: 420
                secretName: kube-backup-ssh
            - name: gpgkey

which works fine

Finally, I run it as root. Then everything is fine.
See
my version
But I still wish there is a better solution .

        volumes:

Thank you for replying ! I got Permissions 0420 for '/root/.ssh/id_rsa' are too open. when I set defaultMode: 420 as I have already said.

I run as
    securityContext:
              runAsUser: 1000
              readOnlyRootFilesystem: true

and it works ootb.


ScratchJR on ubuntu require libgconf2-4

When I try to install scratchjr on Ubuntu 19.10 libgconf2-4 is required but impossible to find this package on ubuntu repos.
I successfully installed scratchjr by ignoring missing package, it works fine but I'm now blocked each time I try to install anything Aptitude and Apt-get ask me to uninstall ScratchJR to continue.
Is there any way to remove libgconf2-4 from the ScratchJR's dependencies.

Saw the same problem on Linux Mint 18.3.

Oh interesting.  Yikes.  So the underlying requirement is a library called Electron, which is basically a version of chrome.
I would search for electron and the specific error you are getting on your platform...
https://github.com/electron/electron/issues/1518

The solution working for some people is to install libgconf-2-4 instead of libgconf2-4 but on a fresh install of LUbuntu 19.10 it don't work. I tried this before posting my first message.
I will continue to search for a solution related to electro.

Hi
Did somebody have the debian/ files to build the package?


Camera parameter acquisition D435i

OK ，thank you read my problems！
I input command
roslaunch realsense2_camera rs_camera.launch
rostopic echo /camera/color/camera_info
and then I got some imformation as follows:
Camera.width: 640
Camera.height: 480
header:
seq: 1009
stamp:
secs:<PHONE_NUMBER>
nsecs:  37440538
frame_id: "camera_color_optical_frame"
height: 480
width: 640
distortion_model: "plumb_bob"
D: [0.0, 0.0, 0.0, 0.0, 0.0]
K: [607.324462890625, 0.0,<PHONE_NUMBER>300781, 0.0,<PHONE_NUMBER>890625,<PHONE_NUMBER>857422, 0.0, 0.0, 1.0]
R: [1.0, 0.0, 0.0, 0.0, 1.0, 0.0, 0.0, 0.0, 1.0]
P: [607.324462890625, 0.0,<PHONE_NUMBER>300781, 0.0, 0.0,<PHONE_NUMBER>890625,<PHONE_NUMBER>857422, 0.0, 0.0, 0.0, 1.0, 0.0]
binning_x: 0
binning_y: 0
roi:
x_offset: 0
y_offset: 0
height: 0
width: 0
do_rectify: False
Xdistorted=x(1+k1r^2+k2r^4+k3r^6)+2p1x*y+p2(r^2+2x^2)
and I need Camera.fx ; Camera.fy; Camera.cx; Camera.cy; Camera.k1; Camera.k2; Camera.p1; Camera.p2;Camera.width;
Camera.height  how can I get them.

Hello, a simple method for retrieving the intrinsics of the camera is to use the '-c' calibration information command of the librealsense rs-enumerate-devices tool:
rs-enumerate-devices -c
When this command is input into the Windows command prompt or the Ubuntu terminal then a long list of the detected camera's calibration information is generated in the terminal window.  You can then scroll upwards from the bottom of the list until you find the information relating to a particular stream profile (for example, shown below is the Depth 848x480 profile from a D435i).

Please note from the above image that the coefficients on the 400 Series cameras are set to zero for reasons described in the link below.
https://github.com/IntelRealSense/librealsense/issues/1430#issuecomment-375945916
These five distortion coefficients are k1, k2, p1, p2, k3.
More information about these intrinsic values can be found at the following documentation links:
https://intelrealsense.github.io/librealsense/doxygen/structrs2__intrinsics.html
https://github.com/IntelRealSense/librealsense/wiki/Projection-in-RealSense-SDK-2.0#intrinsic-camera-parameters
ppx and ppy are equivalent to the cx and cy values that you are seeking as both sets of terms describe the principal point.
Whilst rs-enumerate-devices provides an easy way to retrieve the calibration information, you can also do so with program scripting, as described in the discussion linked to below:
https://github.com/IntelRealSense/librealsense/issues/2930

When this command is input into the Windows command prompt or the Ubuntu terminal then a long list of the detected camera's calibration information is generated in the terminal window. You can then scroll upwards from the bottom of the list until you find the information relating to a particular stream profile (for example, shown below is the Depth 848x480 profile from a D435i).

Thank you very much for your help！


refactor: rename from go-ipfs to kubo

Closes #51

Triggered manual build: https://github.com/ipfs/npm-kubo/actions/runs/5904170330/job/16015683148


Adds new optional callback feature to flag toggle

I have added an optional callback that will be called upon completion of a feature toggle. The reason for this feature is that we have a business need to tie an integration into the completion of a toggle so that we can provide a notification system for flags being toggled.
Please let me know if there is anything I can do to clean this up further or any additional test cases you would like to see added!
I also noticed a deprecation warning regarding the usage of Redis.current so I updated that usage and the tests appear to all still pass. Happy to remove that portion if desired.
Thanks for a great Gem!

All of the tests pass locally, it looks like the CI is getting hung up on an SSH checkout issue. Something about an invalid public key. I'm not sure who all the maintainers are but @reneklacan I see your name at the top. Any ideas on this?
Thanks!

It has come to my attention that we actually can utilize the existing notify observers functionality to meet this need. Apologies!


Sigma Issue

Hi,
sigma: the variance used for the error polynomials.
It has a sigma param for the error variance, it's deviation right?
So why isn't there another param as the average value of the error variance? Is it because the average must always zero?
BR

The error is always centered around zero.


bug/Unable to parse xlsx files

Describe the bug
Using the unstructured library (via Langchain) and directly to read an excel file (.xlsx). Fails during the partition step.
To Reproduce
from unstructured.partition.xlsx import partition_xlsx
partition_xlsx("./new1.xlsx")
Expected behavior
Expected the file to be partitioned.
Screenshots
This is the file content. It is a simple xlsx file.

Environment Info
Am using unstructured-0.11.6, which is a transitive dependency from langchain.
Additional context
Error trace:

the same problem for me.

Closing as inactive and stale. Let us know if you're still having a problem and we'll take another look.


voiceConnection.PlayXYZ doesn't work on ubuntu 14.04

I've been trying to get my bot to play sound. On windows, the following code works fine but on ubuntu 14.04 with ffmpeg installed the bot just doesn't play anything.
"memes" : {
        name : "memes",
        usage : "!memes",
        command : function(bot, message) {
            bot.joinVoiceChannel(message.author.voiceChannel, function(error, voice) {
                voice.playFile("./res/meme.mp3", {"volume":1.0}, function(error, intent) {
                    if (error) console.log(error.stack);
                    intent.on("end", function() {
                        bot.leaveVoiceChannel();
                    });
                });
            });
        }
    },

the error variable is always null too in the above example.
the mp3 at ./res/meme.mp3 definitely exists and i've tried accessing it by direct path too.
I've also tried using libav-tools instead of ffmpeg too to no avail.
here is the info on the ffmpeg I have installed
Package: ffmpeg
Priority: optional
Section: video
Installed-Size: 106 MB
Maintainer: Doug McMahon<EMAIL_ADDRESS>Version: 7:3.0.0+git1~trusty
Depends: libasound2 (>= 1.0.16), libass5 (>= 0.13.0), libbz2-1.0, libc6 (>= 2.17), libfdk-aac0 (>= 0.1.1), libfreetype6 (>= 2.2.1), libgnutls26 (>= 2.12.17-0), libmp3lame0, libopencore-amrnb0, libopencore-amrwb0, libopus0 (>= 1.0.3), libpulse0 (>= 1:0.99.1), libsdl1.2debian (>= 1.2.11), libsoxr0 (>= 0.1.0), libspeex1 (>= 1.2~beta3-1), libva1 (>> 1.3.0~), libvdpau1 (>= 0.2), libvidstab1.0, libvorbis0a (>= 1.1.2), libvorbisenc2 (>= 1.1.2), libx11-6, libx264-148, libx265-79, libxcb-shape0, libxcb-shm0, libxcb-xfixes0, libxcb1, libxext6, libxv1, libxvidcore4 (>= 1.2.2), zlib1g (>= 1:<IP_ADDRESS>)
Download-Size: 31.7 MB
APT-Manual-Installed: yes
APT-Sources: http://ppa.launchpad.net/mc3man/trusty-media/ubuntu/ trusty/main amd64 Packages
Description: Multimedia player, server, encoder and transcoder

I had a similar/same issue with libav-tools on my Raspberry Pi 2; turns out that it doesn't support the -hide_banner parameter that the module puts in so it just errors out and ends the stream instantly.
Apparently -hide_banner is needed so it doesn't error out, but I removed it and it works just fine, even on a version of FFmpeg that does accept the parameter.

yea that's exactly what happens

If you want to test if this is your issue, go to lib/Voice/AudioEncoder.js and remove the parameter from the array on this line (for playFile) and see if it works then.

doesn't seem to be working :/

Update: This fixed it!!! thanks @zoton2 .
I'll leave this issue open so the devs can see it

seen it :+1:

@abalabahaha was the one who helped me figure out this was the issue so they deserve the credit really.


Added Clone trait to Payload

Added the trait Clone to Payload.
This pull request resolves issue #1474

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.266%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
53538

Relevant Lines:
54483

💛  - Coveralls


Fix openstack_crds target

This fixes a few things.

Removes prerequisites run with openstack_deploy_prep when NETWORK_ISOLATION=tue. We don't need to do them as they're already done with openstack_prep.
Also sets REGISTRY_AUTH_FILE as all users don't have permission to write to /run/containers. Else we get the error "reading JSON file "/run/containers/1001/auth.json": open /run/containers/1001/auth.json: permission denied".

@fao89 you added running network isolation pre-requ in  c05d142813f9b822baf2823b34aeb14ffe4eec10 . you see an issue removing it from openstack_deploy_prep?

@stuggi I was following the steps you provided in your PR to enable network isolation, so I'm the one who should be asking you if you see an issue removing it

so I'm the one who should be asking you if you see an issue removing it

Hmm... @dprince Is there an alternative? No one seems the intent of why it was added to openstack_predeploy and we're not ready to merge this fix for the current broken kuttl CI jobs across repos?

I used openstack_predeploy in the openstack_crds task as we need to checkout the openstack-operator repo and I didn't want to duplicate it.

That's ok. The issue is openstack_predeploy used to run NETWORK_ISOLATION=true targets as prereqs and now they all run with openstack_crds and fails for tests where the additional interfaces does not exist.

@stuggi I was following the steps you provided in your PR to enable network isolation, so I'm the one who should be asking you if you see an issue removing it

then it should be ok. if there is no other reason those steps installed the network components before we deployed the openstack operator https://github.com/openstack-k8s-operators/install_yamls/pull/127

Good to merge other than I think we should remove the ENV variable getting set in scripts/openstack-crds.sh.

Good to merge other than I think we should remove the ENV variable getting set in scripts/openstack-crds.sh.

OK, it does not work for me. But updated the PR to remove that ENV var as @dprince thinks it's some issue on my setup and it works for him without it.


Event Updater uses large block range

Cf. Kibana logs: https://logs.gnosisdev.com/goto/aa165cdbdba01c8750991118c26cba58

orderbook::event_updater: updating events in block range 8051255..=8056117

Note that the range is roughly 5000 blocks. We never actually tighten it to e.g. only the last 25 blocks. It's possible that this is causing some strain on our nodes (we have seen an increased amount of alerts in staging recently and I noticed rinkeby alerts popping up when I just restarted the rinkeby API after a bit of downtime as the nodes seem to have rescaled).
Same seems to be true for xDAI and mainnet.
I think this might be related to the fact that we only store the last_handled_block in storage if there is activity and thus in times of little activity accrue a very large range.

Ethcontract still cuts that range into multiple intervals but it is true that this is wasteful. The reason is what you found. A simple fix is to store in the struct the last current block we had. Will work on that.

Thanks for the quick fix. It does however still lead to a spike in load on cold start. Maybe we can create a "constants" table where we can persists the last queried block across restarts?

Good point and the solution makes sense to me too. Will still merge the other PR and then make an additional one.


Fix thread safety issue.

ProgressDrawTarget was unsafe impl Sync which hid a true threading
issue: multiple threads could call Sender::send simultaneously, which is
not permitted by the mpsc API.
Wrapping Sender in a Mutex not only fixes the bug but also permits us to
remove the unsafe impls. The same type is the reason for the unsafe impl
of MultiProgress; however, in that case, the call to Sender::clone is
protected by holding the write lock on MultiProgress::state.

That's fair but the entire point of ProgressBar is to be Send. Is it automatically send now?

Yes, per the test I just added (which is really compile-only; it's a no-op at test runtime).

(At a pure type level view, Mutex turns a Send into a Send+Sync. Incidentally, RwLock doesn't; it requires Sync already. That's the difference between them.)