Fix Typescript 4.8 incompatibility

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/datorama/akita/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
Akita is incompatibility with the last Typescript version, 4.8, used on NX and Angular 14 by default.
Issue Number: 870
What is the new behavior?
Akita can be used on this latest release
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Akita should always be compatible with older versions
Other information
NX workspace had to be updated, it changes somes config file slightly.

Thanks for the contribution! Unfortunately we can't verify the commit author(s): laurent<EMAIL_ADDRESS>One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, sign the Salesforce.com Contributor License Agreement and this Pull Request will be revalidated.

It seems to be an error, I have accepted the CLA and the commits are referenced to my user (secondary e-mail though).
Should I give it another try or is it possible to re-run the check ?


🐞 Since v0.11.0 dagger version returns a warning when used in a git worktree

What is the issue?
As of v0.11.0, when running dagger version in a git worktree, it returns a warning:
❯ dagger version
dagger v0.11.0 (registry.dagger.io/engine) darwin/amd64
20:23:06 WRN failed to get repo HEAD err="reference not found"

This did not happen before:

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

❯ dagger version
dagger v0.10.3 (registry.dagger.io/engine) darwin/amd64

I do all my development in git worktrees and this is annoying. Just a small papercut, nothing major. Not sure if there is a workaround, but I would be happy to use it if there is one. cc @vito
Dagger version
dagger v0.11.0 (registry.dagger.io/engine) darwin/amd64

Duplicate of https://github.com/dagger/dagger/issues/7030


Lagging on scroll Macbook Air

I am seeing this in Chrome more noticeably there anywhere else. Any ideas what may be causing this? Have you seen this issue?
http://camwhite.site to see in action

@camwhite sorry for delay, but wasn't able to reproduce. What year of  manufacture of your Macbook Air?

I think you misunderstood the macbook CSS was giving me a ton of lag on scroll.. I'd really love to be able to go back to using this lib if you can find a fix.
It may be due to the fact that I had scaled it up pretty large..

Scaling should be done via font-size property, as described in Usage section.
Anyways, please drop somewhere a version that you had, so I can take a look.


Service worker in wrong directory

Short description of the problem:
Currently we copy the service worker from src/assets to www/assets. It should be copied to www.
Which Ionic Version? 2.0.0-rc.0

Not solved in RC0

yeah i’d just place it in /src directly, because its more “symmtrical” then. index+service-worker are copied into the root(www) the rest is either built etc. and all other stuffs is in assets. otherwise you’d not have the nice resolution of the file paths that you get now even in index.html
if ('serviceWorker' in navigator) {
        navigator.serviceWorker.register('service-worker.js')
          .then(() => console.log('service worker installed'))
          .catch(err => console.log('Error', err));
      }


Set audit 'Insert or Update or Delete' only

Set audit 'Insert or Update or Delete' only
Maybe create some table as AuditTable with fields
TableName: owner.TableName
Columns: if empty then all if list col1, col2 then audit only by list
Action: if empty then audit all actions if listed then only this actions

Hi,
sure you can add another table. But if you want more flexibility, you can edit the generated triggers at the table level or database level.
At the database level, you can modify how trigger work on new tables.
At the table level, you will find something similar to :
IF (SUBSTRING(COLUMNS_UPDATED(), @char, 1) & @mask) > 0
OR @Type IN ('I','D')
BEGIN
SELECT @sql = '
INSERT Audit ( Type,
…
All you have to do is to edit the condition (which determine if it has to add an entry in the Audit table).
So, if you want insert only :
IF @Type IN ('I')
…
If you want update only :
IF (SUBSTRING(COLUMNS_UPDATED(), @char, 1) & @mask) > 0
…
If you want delete only :
IF @Type IN (‘D’)
…
Or, any combination.
To filter columns, you can use @fieldname in the condition.
Please note, if you use different schema, it will not work out of the box.
Some work has to be done to support it.
I hope you find it useful ! :)

Maybe filter by Audit table trigger? All triggers will works usually but not all will be writes to Audit table ...

Hi,
Right now, there is no trigger on the Audit table.
Maybe it could be the solution to filter the inserted rows (on Audit table) and exclude some rows.
If you want to, you can submit a pull request.
Please note, I want to keep it backward compatible.
If you have to introduce some breaking changes, it would be great to create another script. (ex: version 2)
So I can keep the old version for applications which used it.
To support user name and computer name, there could be some options at the beginning of the script to specify which one are desired.
Thanks.


support for trailingSlash=false

Reference: https://docusaurus.io/docs/deployment#trailing-slashes
We use a hosting provider that requires this setting to be false. When false, the OpenAPI plugin does not show its sidebar. I can remove the config to fix the OpenAPI plugin, but that breaks the docs/ sidebar.

I think this might be fixed on the main branch. I'll be releasing 0.2.0 soon, which will have a few breaking changes, but it seems like the trailing slash issue is gone

I think this is fixed. Feel free to reopen if you're still experiencing this issue :)


Testnets: visible by default

Just a note, the users that have already visited the Console and whose initial app state was created will still have everything off by default.

Just a note, the users that have already visited the Console and whose initial app state was created will still have everything off by default.

yep, it will keep the state the user has set. Should we do something about that?

ideally yes, we should toggle to on for all users, even if they've visited before. is that a simple change?

Guess not, merging.


(PUP-8552) Create notdefaultfor blacklist for better management of default providers.

CLA signed by all contributors.

This PR isn't being picked up by tickets.puppetlabs.com, I am closing and recreating.


Add doc comments to public APIs

This repo currently has a lot of public APIs without any doc comments.
There's already an issue (#610) about adding doc comments for EventId.
There's also a pretty old PR (#529) to add doc comments to console logger types. Having doc comments to indicate things like the default value of ConsoleLoggerOptions.IncludeScopes could certainly be useful. There are other console logging types that don't have doc comments like IConsoleLoggerSettings, but I haven't created an exhaustive list.
None of the (extension) methods on ILoggingBuilder (i.e. AddFilter, AddProvider, ClearProviders, SetMinimuLevel) have doc comments.
I Think it would be particularly useful to note in the SetMinimumLevel doc comments that the default LogLevel is Information meaning that Trace and Debug logs will be ignored.  I think this should be noted on any API where a minimum level can be set.

I just learned that SetMinimumLoggingLevel gets overridden by what's configured in appsettings.json when using WebHost.CreateDefaultBuilder. I don't think this is obvious. While this is probably too detailed to go into a doc comment, I think this should be documented somewhere.

Just came here after googling ILoggingBuilder SetMinimumLevel - is this ticket really the only place on the entire internet that this API is discussed? :-)

Confused about how the API is intended to work - https://github.com/datalust/seq-extensions-logging/issues/22#issuecomment-324170847 - some docs would be very useful.


tests: lib: set integration_platforms

Set integration_platforms on these tests to just native_posix or are
relevant qemu platform.  This should be sufficient to make sure these
tests build and run.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Just to clarify: I'm not blocking as I don't know these tests, but I don't feel comfortable approving based on the above comment.

The word "align" in "heap_align" makes me nervous about dropping coverage for arches with different aligned access behaviors. A comment in the test file about heap chunks being returned low address to high makes me worried about not having this run on both big and little endian.

Agree. I would prefer to have that tested on all the arch.

it should be noted that those changes only apply for PRs, but will still run on all applicable boards in the daily run and when run locally.


spinner instead of followers list

The followers/following tabs never show on my Linux system.  I just get a spinner.  This was working previously.  Perhaps it was broken in a recent update, but I'm not sure.
log id: 1b480e443b3983538d7ada1c

this has resolved itself after upgrade to:
Version: 1.0.36-20171122003226.f68b10ef8


⚠ Minecraft has degraded performance

In 8e8071f, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1321 ms

Resolved: Minecraft performance has improved in 780b37b.


[Enhancement] Make the distribution balance threshold configurable

Why I'm doing:
The balance of distribution will affect the balance of the disk, and a parameter is needed to control the impact.
What I'm doing:
Introduce param tablet_sched_distribution_balance_threshold_ratio.
Doing distribution balance may break the disk size balance, but the maximum gap between disks cannot exceed tablet_sched_distribution_balance_threshold_ratio * tablet_sched_balance_load_score_threshold. If there are tablets in the cluster that are constantly balancing from A to B and B to A, reduce this value. If you want the tablet distribution to be more balanced, increase this value.
Fixes #SSU-1214
What type of PR is this:

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

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[x] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

@mergify rebase

@mergify backport branch-3.3


Depth reported by scene picker not accurate with orthographic camera

The problem is that the orthographic camera spawns the picking ray on a "safe" scene bounding sphere, instead of at the camera position. Fixing this may have other impacts which need to be investigated.

I would like to take a look at this bug. :)


-spec false makes change-deletection fail (ng g c product -is -it -spec false -c Default)

Bug Report or Feature Request (mark with an x)
- [X] bug report
- [ ] feature request

Versions
Angular CLI: 6.0.8
Node: 8.11.3
OS: linux x64
Angular: 6.0.6
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.6.8
@angular-devkit/build-angular     0.6.8
@angular-devkit/build-optimizer   0.6.8
@angular-devkit/core              0.6.8
@angular-devkit/schematics        0.6.8
@angular/cli                      6.0.8
@ngtools/webpack                  6.0.8
@schematics/angular               0.6.8
@schematics/update                0.6.8
rxjs                              6.2.1
typescript                        2.7.2
webpack                           4.8.3

The log given by the failure
ng g c product -is -it -spec false -c Default
Schematic input does not validate against the Schema: {"inlineStyle":true,"inlineTemplate":true,"i":[true,true],"prefix":"","e":true,"changeDetection":["false","Default"],"styleext":"css","spec":true,"flat":false,"skipImport":false,"export":false}
Errors:

  Data path ".changeDetection" should be string.`

and:
$ ng g c product -is -it -spec false
Schematic input does not validate against the Schema: {"inlineStyle":true,"inlineTemplate":true,"i":[true,true],"prefix":"","e":true,"changeDetection":"false","styleext":"css","spec":true,"flat":false,"skipImport":false,"export":false}
Errors:

  Data path ".changeDetection" should be equal to one of the allowed values.

Desired functionality
Component is created.

FYI, the aliases have changed with version 6, you can find the updated documentation here. But the general rule is that all of the aliases are single character following the POSIX standard so that they can be chained. For example for inline style and inline template, you would run ng g c foo -st where s is for inline style and t is for inline template.

Hmm.. thanks for the info.
In that sense, the error message is misleading. Better: Include info and refer to changes.


Edit the Contributor's page.

The page should display the cards of the owners of the project with their name and picture.

Please assign me this task to me.

Not just the owners but all the contributors that contributed to the website.

Please provide a general idea of what you would like to add to this page.
The page should display cards featuring the names and pictures of both project owners and all contributors who have contributed to the website.
Are you interested to work on this issue?


Investigate performance status of generated Python SDK

Investigate the SDK side performance such as REQUEST PER MINUTE, AVERAGE LATENCY.
Expect the result can be a base line for further performance improvement.

Where is this ask coming from?

Where is this ask coming from?

Hi @lmazuel , It's mentioned in Shanghai's Python scrum meeting since .Net has similar tasks: https://github.com/Azure/autorest.csharp/issues/2518

Let's not work on perf at this moment


TsMigration, Lib Phase 3: Syntax port of lib's test folders

Following the migration plan from #434, port the tests code from Javascript to Typescript.
Note that the tests can currently run in Javascript just fine because we're using ts-node. This is achieved by using require(ts-node/register) in truffle-config.js.

[ ] Port all the files to Ts.
[ ] Unify tsconfig.json and tsconfig.test.json
[ ] Remove ts-config TS_NODE_PROJECT environment variable from srcipts/test.sh

Will not implement


Cursor Pagination not returning expected results

Bug description
I'm using cursor-based pagination, and have run into a scenario where it's not returning the results I would expect.
How to reproduce
Here are two queries, one without pagination, and one with.
prisma.tasks.findMany({
  where: { child: { id: 136 } },
  orderBy: [{ confirmedAt: "desc" }, { id: "asc" }],
})

// Results:
[
  {
    id: 176,
    childId: 136,
    createdAt: 2020-09-02T17:58:31.048Z,
    updatedAt: 2020-09-02T17:58:31.048Z,
    completedAt: 2020-09-02T17:58:31.046Z,
    confirmedAt: null,
  },
  {
    id: 177,
    childId: 136,
    createdAt: 2020-09-02T17:58:31.058Z,
    updatedAt: 2020-09-02T17:58:31.058Z,
    completedAt: null,
    confirmedAt: null,
  },
  {
    id: 178,
    childId: 136,
    createdAt: 2020-09-02T17:58:31.067Z,
    updatedAt: 2020-09-02T17:58:31.067Z,
    completedAt: null,
    confirmedAt: null,
  },
  {
    id: 179,
    childId: 136,
    createdAt: 2020-09-02T17:58:31.075Z,
    updatedAt: 2020-09-02T17:58:31.075Z,
    completedAt: 2020-09-02T17:58:31.074Z,
    confirmedAt: null,
  }
]

prisma.tasks.findMany({
  where: { child: { id: 136 } },
  orderBy: [{ confirmedAt: "desc" }, { id: "asc" }],
  take: 2,
  cursor: { id: 177 },
  skip: 1,
})

// Results:
[]

Expected behavior
I would expect the paginated query to return that last two results.
[
  {
    id: 178,
    childId: 136,
    createdAt: 2020-09-02T17:58:31.067Z,
    updatedAt: 2020-09-02T17:58:31.067Z,
    completedAt: null,
    confirmedAt: null,
  },
  {
    id: 179,
    childId: 136,
    createdAt: 2020-09-02T17:58:31.075Z,
    updatedAt: 2020-09-02T17:58:31.075Z,
    completedAt: 2020-09-02T17:58:31.074Z,
    confirmedAt: null,
  }
]

Environment & setup

OS: macOS
Database: PostgreSQL
Node.js version: v12.18.1
Prisma version:

@prisma/cli          : 2.5.1
Current platform     : darwin
Query Engine         : query-engine c88925ce44a9b89b4351aec85ba6a28979d2658e (at node_modules/@prisma/cli/query-engine-darwin)
Migration Engine     : migration-engine-cli c88925ce44a9b89b4351aec85ba6a28979d2658e (at node_modules/@prisma/cli/migration-engine-darwin)
Introspection Engine : introspection-core c88925ce44a9b89b4351aec85ba6a28979d2658e (at node_modules/@prisma/cli/introspection-engine-darwin)
Format Binary        : prisma-fmt c88925ce44a9b89b4351aec85ba6a28979d2658e (at node_modules/@prisma/cli/prisma-fmt-darwin)
Studio               : 0.261.0

Hey @defrex
Can you please the task model's schema so that I can easily try this locally? Thanks!

Sure thing! I redacted a few of the fields from the example for easier reading, but their values shouldn't make any difference.
model Task {
  @@map(name: "task")

  id             Int             @default(autoincrement()) @id
  createdAt      DateTime        @default(now())
  updatedAt      DateTime        @default(now())
  amount         Int             @default(0)
  archivedAt     DateTime?
  childId        Int
  completedAt    DateTime?
  confirmedAt    DateTime?
  paidAt         DateTime?
  parentId       Int
  recurrenceId   Int?
  title          String
  validAt        DateTime?
  expiresAt      DateTime?
  child          User            @relation("task_childIdTouser", fields: [childId], references: [id])
  parent         User            @relation("task_parentIdTouser", fields: [parentId], references: [id])
  taskRecurrence TaskRecurrence? @relation(fields: [recurrenceId], references: [id])
}

Hey @defrex
I am unable to reproduce this. I got the expected result:

Can you please try this again with the latest release(2.6) ? Also, if this is still reproducible after the update, please share your Child model as well.

@pantharshit00 working on a repro with the latest version.
However, fwiw, in my example confirmedAt was null for all instances but has a value in all of yours. That may impact the outcome.

Confirmed the issue is still happening in 2.6.1. I also got 2 test regressions from upgrading to 2.6.1, but perhaps that's a topic for another issue.

@pantharshit00 here is a minimal reproduction.
https://github.com/defrex/prisma-page-repro

I'm having the same issue and have narrowed it down to my query having an orderBy item that is potentially null.  For example, I have products and vendors.  Third party vendors are an optional relation on products.
model Product {
  id                          String            @default(cuid()) @id
  sku                         String            @unique
  name                        String?
  price                       Int
  vendorId                    String?
  vendor                      Vendor?           @relation(fields: [vendorId], references: [id])
  createdAt                   DateTime          @default(now())
  updatedAt                   DateTime          @updatedAt
}

model Vendor {
  id           String         @default(cuid()) @id
  name         String
  products     Product[]
  createdAt    DateTime       @default(now())
  updatedAt    DateTime       @updatedAt
}

On our catalog page, we want to favor internal products over third party vendor products in the search results, so we've been sorting by vendorId with orderBy: [{ vendorId: 'desc' }].  This makes all products with a null vendorId show up first in the results.  However, as soon as I add a cursor arg to skip to the next page, it returns 0 results even though we have thousands in the db.  Full example query...
Confirm sanity...
// returns 10124
await prisma.product.count();

Get the first 10 results (works as expected with null vendorId's first)
await prisma.product.findMany({
  orderBy: [
    { vendorId: 'desc' },
    { createdAt: 'desc' }
  ],
  take: 10
});

Now when I try to fetch the next page, I get zero results even though there are still ~10k products left matching the query.
// returns 0 results
await prisma.product.findMany({
  orderBy: [
    { vendorId: 'desc' },
    { createdAt: 'desc' }
  ],
  take: 10,
  skip: 1,
  cursor: {
    id: '5cb0aac668e4040064000dc2' // id of tenth result above
  }
});

And here's the only clue I've come across... if I remove the { vendorId: 'desc' } from the orderBy array, pagination starts working as expected again.  So this works...
await prisma.product.findMany({
  orderBy: [
    { createdAt: 'desc' }
  ],
  take: 10,
  skip: 1,
  cursor: {
    id: '5cb0aac668e4040064000dc2' // id of tenth result above
  }
});

Also, if I add any other non-null field to the orderBy array, it works as expected.  So it appears to be something to do with the combination of sorting on a null field while also using cursor pagination.  For example, replacing the vendorId sort with price instead works...
// returns the next 10 results as expected
await prisma.product.findMany({
  orderBy: [
    { price: 'asc' },
    { createdAt: 'desc' }
  ],
  take: 10,
  skip: 1,
  cursor: {
    id: '5cb0aac668e4040064000dc2'
  }
});

I'm not sure where this got introduced, but it was some time in the last few releases because this was working on our production site a few weeks ago.  I'm assuming this started somewhere around when the support for multiple orderBy entries was added.
Any recommendations?

I can reproduce this now with @defrex's reproduction: https://github.com/defrex/prisma-page-repro
Thanks @jshimko for the write up as well. We will need to fix this.

Internal note: Pinned the issue to originate form incorrect SQL generation:
`Task`.`completedAt` = `order_cmp`.`completedAt`

The above will disregard NULL values because NULL != NULL. Issue fix surface is limited to this specific query, probably not too hard to fix.

Fix available from 2.8.0-dev.13 onwards (will be in the 2.8 release).


Taskbar empty class not working

Hi all,
I have an issue with the .empty class of #taskbar.
I have added
#taskbar.empty {
background:transparent;
}

to the style.css.

With the initial loading of waybar every works fine. Taskbar is hidden.
When I open an application the taskbar appears correctly.
But when I close all apps, the .empty class will not applied and the taskbar is still visible on waybar.

Any idea?
Thanks in advance.

I have tried this with a recent Waybar commit (b26ab1f982c06ce48e96f3e4da8cde034296eb1d). I'm starting waybar with GTK_DEBUG=interactive, and the taskbar module has the class empty when no window button is in the taskbar, and does not have the class empty when any window buttons are in the taskbar. Like I would expect it to behave.
empty:

not empty:

What exactly does it look like for you? Could it be the result of any other CSS, perhaps even from other elements?

Thanks for your feedback. I will test again and try to replicate your steps.

@RobertMueller2 I could integrate the GTK_DEBUG and here are my results.
When I initialize waybar the empty class is available:

When I open an application and close it again the empty class doesn't come back:

That works fine for me with latest git version of waybar (aka 0.10.4) and sway (fairly recent commit, 3f327b3db0c1fc6985c0ed3231e1bd6296584dad), and yes, I'm opening and closing a window as well.
What are you using?

I am using the latest aur with 0.10.3. This can be the reason.

Hm, there was a change in 1b1442e3ba8393edcd7f3a4b10ab212d35c3f523 which is not in 0.10.3. "fix: taskbar not applying empty class on empty" kinda sounds and also looks like it makes a difference here. :)


⚠ Bofrost.be has degraded performance

In 4118baa, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1275 ms

Resolved: Bofrost.be performance has improved in 1d67d20.


Parse error on named pipes?

The following target fails for me:
test named pipes:
  help: Testing named pipes
  formula: >
    grep -f  <(echo "foo\nbar") <(echo "bar\nbaz") | sort > foo.txt
  output:
    - foo.txt

Running the formula in a shell produces the expected output: a file foo.txt that contains the word bar. Sake chokes with the following error though:
Running target test named pipes
grep -f  <(echo "foo\nbar") <(echo "bar\nbaz") | sort > foo.txt
/bin/sh: -c: line 0: syntax error near unexpected token `('
/bin/sh: -c: line 0: `grep -f  <(echo "foo\nbar") <(echo "bar\nbaz") | sort > foo.txt'
Command failed to run

What could be causing Sake trouble parsing this command?

Is that Bash syntax, or POSIX shell-compliant?

Oh interesting. According to this source: http://wiki.bash-hackers.org/syntax/expansion/proc_subst, process substitution is supported by Bash but is not specified by POSIX and is disabled completely in Bash POSIX mode.

@justmytwospence Yup, that's it. Sake runs tasks in the POSIX shell.
Since #58 hasn't been implemented, you could do something like:
test named pipes:
  help: Testing named pipes
  formula: >
    bash 'grep -f  <(echo "foo\nbar") <(echo "bar\nbaz") | sort > foo.txt'
  output:
    - foo.txt

The master branch now has support for alternative shells.
You can now put
shell: bash

In your Sakefile and you'll get the behavior you want :)


Fepois: Add support for predict with fixed effects

Context
Currently, the predict method is disabled for Poisson Regression with fixed effects. Unfortunately, I cannot recall why... 😅
To do

Enable predict() for Poisson regression with fixed effects.
Clean up ValueError when trying to run Fepois.predict() on a fixed effect model in tests/test_errors.py.
Add unit tests to tests/test_vs_r::test_single_fit.

While I haven't completed implementation of all separation checks yet, I would like to pre-register my interest for working on this issue 🙂

More than happy to already assign this to you Leo! =)


Is it important to enable De-aliasing? how much deviation do you estimate?

We are conducting serious calibration tests. we wondering whether it is important to conduct such feature or not. tnks!

Kevin,
De-aliasing does not improve the accuracy. It only improves the unambiguous range of the system. For example, with 40MHz, a range of 3.75m and with 60MHz a range of 2.5m can be achieved. But with the combination, a range of 7.5m can be achieved (equivalent frequency of GCD(40,60)=20MHz). Please refer to the dealiasing section in the OPT9221 datasheet for more details.
Regards
Bharath


Optimize dependencies in android modules

Extract dependencies in :android-test and :regression-test modules to Dependencies.

Build failure
> Task :android-test:compileDebugAndroidTestKotlin FAILED
e: /Users/runner/work/okhttp/okhttp/android-test/src/androidTest/java/okhttp/android/test/OkHttpTest.kt: (23, 34): Unresolved reference: reflect
e: /Users/runner/work/okhttp/okhttp/android-test/src/androidTest/java/okhttp/android/test/OkHttpTest.kt: (107, 12): Unresolved reference: KotlinJsonAdapterFactory

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':android-test:generateDebugRFile'.
> Could not resolve all files for configuration ':android-test:debugCompileClasspath'.
   > Could not resolve androidx.annotation:annotation:1.1.0.
     Required by: project :android-test
      > Could not resolve androidx.annotation:annotation:1.1.0.
         > Could not get resource 'https://dl.bintray.com/kotlin/dokka/androidx/annotation/annotation/1.1.0/annotation-1.1.0.pom'.
            > Could not HEAD 'https://dl.bintray.com/kotlin/dokka/androidx/annotation/annotation/1.1.0/annotation-1.1.0.pom'. Received status code 502 from server: Bad Gateway


bugfix/IIA-1066: Pattern and Lidr windows are transparent and shouldn't be

Fixes: https://ikmdev.atlassian.net/issues/IIA-1066?jql=assignee%3D"557058%3A88aaf25e-8b39-4b66-ba0e-7f74b83d2ff2" AND resolution is empty

Thanks! Glad you like it! 🙂


style: fix bold

Docs Build status updates of commit 10051e3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sysinternals/downloads/psping.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[BUG]: WAL replay warnings suppression

Refs:  #1733
Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

Added a EmbeddingRecord flag (wal_replay) to suppress warnings in sqlite and local hnsw indices

Test plan
How are these changes tested?

[ ] Tests pass locally with pytest for python, yarn test for js

Documentation Changes
N/A
Colab to reproduce the issue in 0.4.23 (and prior) - https://colab.research.google.com/drive/1CtN0qGoMZoVfwQBnJE1MloLvvdv5KouE?usp=sharing

@atroyn, I've added a colab that reproduces the problem.
The gist is that we emit all entries from the WAL up to the index' MAX seqId, if there are duplicate entries (e.g. an item was added/removed multiple times) the respective index reports the duplicates as warnings.

Similar issue was reported by MemGPT - https://discord.com/channels/1073293645303795742/1205288940488359998

This is potentially made redundant by #2062

Close in favor of #2062


Update ci-theme.css

I'm okay with this change but you might want to save small tweaks like this and bundle them into a larger Pull Request, if there are more coming.

Handled by #84


Make persistent directory configurable.

As this is the base role where other roles (like silpion.etckeeper)
might depend on, I find it makes sense that this role is as configurable
as possible. Other people might not wanna use the complete role silpion stack.

v2 allowed to finaly implement this (like I wanted to do for 1.5 years now).


Feature/upgrade sensu

Updates to accommodate Sensu version 0.26. Updates to README and CHANGES. Update package structure for clarity.

LGTM


bin/tasks/commands_test.dart is flaky

The devicelab test bin/tasks/commands_test.dart is flaky. I had thought this was contention over a hard-coded service protocol port, but it's still failing after we switched to using any open port.
flutter drive --use-existing-app http://<IP_ADDRESS>:8100/ --keep-app-running --driver test_driver/commands_none_test.dart
drive:stdout: Using device Moto G 4.
drive:stdout: Will connect to already running application instance.
drive:stdout: 00:00 +0: (setUpAll)
drive:stderr: [info ] FlutterDriver: Connecting to Flutter application at http://<IP_ADDRESS>:8100/
drive:stderr: [trace] FlutterDriver: Looking for the isolate
drive:stderr: [trace] FlutterDriver: Isolate is not paused. Assuming application is ready.
drive:stderr: [info ] FlutterDriver: Connected to Flutter application.
drive:stdout: 00:00 +0: check that we are in normal mode
drive:stdout: 00:00 +0 -1: check that we are in normal mode [E]
drive:stdout:   Expected: 'log: paint'
drive:stdout:     Actual: ''
drive:stdout:      Which: is different. Both strings start the same, but the actual value is missing the following trailing characters: log: paint ...
drive:stdout:
drive:stdout:   package:test                               expect
drive:stdout:   test_driver/commands_none_test.dart 20:5   main.
drive:stdout:   ===== asynchronous gap ===========================
drive:stdout:   dart:async                                 _Completer.completeError
drive:stdout:   test_driver/commands_none_test.dart 22:3   main.
drive:stdout:   ===== asynchronous gap ===========================
drive:stdout:   dart:async                                 _asyncThenWrapperHelper
drive:stdout:   test_driver/commands_none_test.dart 19:53  main.

Yeah, I keep meaning to check out the problem.

I see all kinds of failures in the logs.

I cannot repro this locally.

Reassigning to Carlo since he's been working on this. @B3rn475 feel free to unassign if you aren't working on it.

It is confirmed that we are connecting to the wrong app during tests.
drive:stdout:   Expected: 'log: paint'
drive:stdout:     Actual: 'keyboard_resize'
drive:stdout:      Which: is different.
drive:stdout:             Expected: log: paint ...
drive:stdout:               Actual: keyboard_r ...

See https://flutter-dashboard.appspot.com/api/get-log?ownerKey=ahNzfmZsdXR0ZXItZGFzaGJvYXJkclgLEglDaGVja2xpc3QiOGZsdXR0ZXIvZmx1dHRlci81MDY2NzUwMDYzOTQ0YTI3ZDFjZjdkMzNlNmI0YmEwZWU0ODg1MTMxDAsSBFRhc2sYgICAgICAyAsM

It happens when the commands_test and integration_ui run on the same machine


System extensions example

It would be great to have some example how to enable system extensions like iscsi. I tried the extensions parameter in the node object but got a deprecation warning in talosctl.
What is the correct way to that?

Hi, @runningman84! Have you read the doc here? https://budimanjojo.github.io/talhelper/latest/guides/#adding-talos-extensions-and-kernel-arguments

I tried that example once but did not see any change in the generated machine configs but I will try to reproduce that...

sorry I used some old version of talhelper... the new one works!

@budimanjojo with reset and re apply the extensions were active. Unfortunately after a node shutdown and start the extensions were lost again. Do you have any idea how that can happen?

That's weird and I don't think it has anything to do with talhelper but rather the OS side problem. Maybe the log of the kernel might show you something.


Add warning that p4factory is being deprecated.

Redirect to p4lang/tutorials repository.
Point to INT demo in README, which is the only reason why p4factory
should still be used.
Move all other README instructions to separate USAGE.md file to
emphasize the warning in the README.

@herlenashavi please take a look at this

@antoninbas Looks good.


[NETBEANS-54] Module Review html.editor

Files to be solved centrally:
html.editor/src/org/netbeans/modules/html/editor/options/ui/formatSample.html
html.editor/src/org/netbeans/modules/html/editor/resources/HTMLExample
html.editor/src/org/netbeans/modules/html/editor/resources/XHTMLExample
html.editor/src/org/netbeans/modules/html/editor/todo.txt
Added license to html catalog files (i don't know if these files need to be licensed):
html.editor/src/org/netbeans/modules/html/editor/resources/DTDs/3_2/catalog
html.editor/src/org/netbeans/modules/html/editor/resources/DTDs/4_0/catalog
html.editor/src/org/netbeans/modules/html/editor/resources/DTDs/4_01/catalog
html.editor/src/org/netbeans/modules/html/editor/resources/DTDs/xhtml/catalog
(they are parsed in html.editor/src/org/netbeans/modules/html/editor/NbReaderProvider.java:98)
This module use external "library" html-4.01.zip for html help. It doesn't exist in maven public repositories. It is downloaded from https://www.w3.org/TR/html401/
Module should be updated to HTML5 https://www.w3.org/TR/html5/

My reading of NbReaderProvider is, that there can't be comments in the catalog files. At this point the catalog files are useless, as the corresponding DTDs are not present anymore. I raised NETBEANS-114 for this.
To get this on I would do this:

remove the license from the catalog files
add excludes to build.xml for rat:

<exclude name="html.editor/src/org/netbeans/modules/html/editor/options/ui/formatSample.html" /> <!--user visible file template-->
<exclude name="html.editor/src/org/netbeans/modules/html/editor/resources/HTMLExample" /> <!--user visible file template-->
<exclude name="html.editor/src/org/netbeans/modules/html/editor/resources/XHTMLExample" /> <!--user visible file template-->
<exclude name="html.editor/src/org/netbeans/modules/html/editor/todo.txt" /> <!--no degree of creativity-->
<exclude name="html.editor/src/org/netbeans/modules/html/editor/resources/DTDs/*/catalog" /> <!--no degree of creativity-->

For reference: I opened #240 that restores the removed documents mentioned above. Both PRs together should clear the html.editor module completely.

License from catalog files removed

Thank you - I added the excludes for the build.xml and merged this. html.editor is now clean.


Using the tool

Hello Friend.
I am not able to use the code in my angular project 1.x.
Just insert the angular library drag-and-drop-lists.js into my code?
I did this and copied the HTML and JavaScript codes, however, I did not succeed.
Would you help me?

You need insert library (drag-and-drop-lists.js ) and in your angular module inject dependency 'dndLists'
You could use this example http://plnkr.co/edit/OWwDCaYUjuN7fM5y5Btd?p=preview

Thanks for the help my friend.
It worked well here. Maybe I try to work out some transparency effect when dragging the blocks.
If I wanted, I can send a photo with the result obtained.


Issues about quantizing yolov8 using vitis-ai 2.5

Hello! When i want to quantize yolov8 model using vitis-ai 2.5,i encountered many problems:
1.when i start to quantize, the tool pytorch_nndct always generate a file named "autobackend.py" in "quant_result"directory.(I use yolov8's Autobackend class to load my trained v8 model),the file is as below:

Can anyone teach me how to read it and why it is generated?

Have anyone encounter the same issue as below?

I try to read its source code ,actually the problem occurs in the function "do_scan", the param res is a tensor but the source code shows that some  of the res have the attribute "values", it really confused me .

Thirsty for reply!!!

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."


about models/*.pt

could you please explain what kind of datasets are used to get models "linda_johnson.pt" and "multi_speaker.pt"?  Is "multi_speaker.pt" model corresponding to
 in the paper? And what's for "linda_johnson.pt" model?

linda_johnson.pt is trained using the open source dataset https://keithito.com/LJ-Speech-Dataset/
Yes, multi_speaker.pt is trained using our internal multispeaker dataset, and is expected to generalize across new speakers.

@wezteoh I tested the multi_speaker.pt model to synthesize new speakers' sounds and it did well. Have your team explored how long at least the training dataset of each speaker should be and how many speakers at least are needed to achieve such an effect? btw, for your internal multispeaker, are they all English speakers?

all english speakers, and we used about 10 hours each.

Thx! Besides when i run this repo with LJSpeech, i find the training is so slow, far from the reference time  in the demo webpage.(shown in the picture)

I stripped out the mel calculating module and fed the pre-processed mel directly into training process by dataloader, but it didn't accelerate the training. Could you give me some speeding-up advice?

I rewrote AudioDataset to directly feed processed audio data rather than processing wavs into audio data. The training speed is now close to the given one.

@MorganCZY so create a pull request

@MorganCZY can you share your code?


components, src: avoid cpu quota limitation contamination

Signed-off-by: BornChanger<EMAIL_ADDRESS>
What is changed and how it works?

Issue Number: Close #13084
What's Changed:

Sample build didn't check if cpu quota is from foreground or background.
So, if either cpu quota limit is set, cpu sampling would be conducted for both
foreground and background.  It's not expected and brought overhead.  We
need to distinguish the cpu quota limit sources.

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Manual test (add detailed scripts or steps below)
Debug to make sure no such contamination any more.

Release note
None

@tabokie @Connor1996 please have a review to this quick fix.

/cc @tabokie @Connor1996

/cc @sticnarf

/check-issue-triage-complete

/merge

/run-test retry=4

/merge

/test

/run-test retry=6

/run-test retry=9


fix(uiSrefActive): update the active classes when compiled

This PR makes sure the active classes are applied during the directive compilation phase so that they are available even after $stateChangeSuccess has already been fired.
This patch was originally developed for 1.0.0alpha0, but since the test for it didn't pass on master either, I've backported it.
Plunkr showing the issue: https://plnkr.co/edit/z3kdm41eyuXK052fsflH?p=preview

Looks good, thanks


Multiple input with same class

When i have many input with the same class ex: .datepicker only the first element work.
Did i miss something?
I edited my local PickMeUp.js for this work. Let me know if you want the change.

Yes, it only selects first element, this is how document.querySelector() works. If you need to apply it to many elements you don't need to edit source, you can just select all necessary elements and initialize PickMeUp on them individually like so:
var elements = document.querySelectorAll('.datepicker');
for (var i=0; i < elements.length; ++i) {
    pickmeup(elements[i], {...});
}

This is by design.

Thank you Nazar, but i fixed the source for this work.  That took me 5 minutes lol.
Cause i have different calendar with different init config.
Thank you for taking your time to answer me it's appreciate.
//Set config PickMeUp pluggin birthdate calendar
addEventListener('DOMContentLoaded', function () {
pickmeup('.birthdate',
{
first_day: 0,
default_date: false,
hide_on_select: true,
view: 'years',
mode: 'single',
min: '1920-01-01',
max: '2020-01-01',
date: '1975-01-01',
format: 'Y-m-d'
});
});
//Set config PickMeUp pluggin datepicker calendar
addEventListener('DOMContentLoaded', function () {
  pickmeup('.datepicker',
    {
      first_day: 0,
      default_date: false,
      hide_on_select: true,
      view: 'days',
      mode: 'single',
      format: 'Y-m-d'
    });
});

De : Nazar Mokrynskyi<EMAIL_ADDRESS>Envoyé : 9 février 2017 18:15
À : nazar-pc/PickMeUp
Cc : Buzzak; Author
Objet : Re: [nazar-pc/PickMeUp] Multiple input with same class (#163)
Yes, it only selects first element, this is how document.querySelector() works. If you need to apply it to many elements you don't need to edit source, you can just select all necessary elements and initialize PickMeUp on them individually like so:
var elements = document.querySelectorAll('.datepicker');
for (var i=0; i < elements.length; ++i) {
pickmeup(elements[i], {...});
}
This is by design.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/nazar-pc/PickMeUp/issues/163#issuecomment-278806294, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AQsACVAfHE85ScwCEA9uxTis0PR6qt06ks5ra54ZgaJpZM4L8lsH.

Hello, i have another problem,  maybe you could help me!
When i run the javascript on my local machine pickmeup working #1!
When it published on a windows server 2016 with iis 10 ,the datepicker work the first time perfectly and when i try to select another date while a date is already in the input the datepicker show a date into years 3000+ or a wrong date...
I have no more issue i can't debug the js cause he is minimized.
My initialization
var elements = document.querySelectorAll('.birthdate');
for (var i=0; i < elements.length; ++i) {
  pickmeup(elements[i],  {
    first_day: 0,
    default_date: false,
    hide_on_select: true,
    view: 'years',
    mode: 'single',
    min: new Date('1920-01-01'),
    max: new Date('2020-01-01'),
    date: new Date('1975-01-01'),
    format: 'Y-m-d'
  });
}

var elements = document.querySelectorAll('.datepicker');
for (var i = 0; i < elements.length; ++i) {
  pickmeup(elements[i], {
    first_day: 0,
    default_date: false,
    hide_on_select: true,
    view: 'days',
    mode: 'single',
    format: 'Y-m-d'
  });
}

thank you very much!

De : Nazar Mokrynskyi<EMAIL_ADDRESS>Envoyé : 9 février 2017 18:15
À : nazar-pc/PickMeUp
Cc : Buzzak; Author
Objet : Re: [nazar-pc/PickMeUp] Multiple input with same class (#163)
Yes, it only selects first element, this is how document.querySelector() works. If you need to apply it to many elements you don't need to edit source, you can just select all necessary elements and initialize PickMeUp on them individually like so:
var elements = document.querySelectorAll('.datepicker');
for (var i=0; i < elements.length; ++i) {
pickmeup(elements[i], {...});
}
This is by design.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/nazar-pc/PickMeUp/issues/163#issuecomment-278806294, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AQsACVAfHE85ScwCEA9uxTis0PR6qt06ks5ra54ZgaJpZM4L8lsH.


is it possible to use the module arm64 in ubuntu? since i updated my server this error happened

/home/ubuntu/AmebasTS/node_modules/@roamhq/wrtc/lib/binding.js:27
  throw new Error(Could not find wrtc binary on any of the paths: ${paths_to_try});
        ^
Error: Could not find wrtc binary on any of the paths: ../build-linux-arm64/wrtc.node,../build-linux-arm64/Debug/wrtc.node,../build-linux-arm64/Release/wrtc.node,@roamhq/wrtc-linux-arm64,./node_modules/<EMAIL_ADDRESS>    at Object.<anonymous> (/home/ubuntu/AmebasTS/node_modules/@roamhq/wrtc/lib/binding.js:27:9)
    at Module._compile (node:internal/modules/cjs/loader:1196:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1250:10)
    at Object.require.extensions.<computed> [as .js] (/home/ubuntu/AmebasTS/node_modules/ts-node/src/index.ts:1608:43)
    at Module.load (node:internal/modules/cjs/loader:1074:32)
    at Function.Module._load (node:internal/modules/cjs/loader:909:12)
    at Module.require (node:internal/modules/cjs/loader:1098:19)
    at require (node:internal/modules/cjs/helpers:108:18)
    at Object.<anonymous> (/home/ubuntu/AmebasTS/node_modules/@roamhq/wrtc/lib/index.js:23:5)
    at Module._compile (node:internal/modules/cjs/loader:1196:14)

ubuntu@instance-20220716-0418:~/AmebasTS$ uname -m
aarch64
ubuntu@instance-20220716-0418:~/AmebasTS$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 20.04.6 LTS
Release:        20.04
Codename:       focal

It should've prebuilds for linux-arm64, although you can try to build wrtc module from source.
https://github.com/WonderInventions/node-webrtc?tab=readme-ov-file#supported-platforms
https://github.com/WonderInventions/node-webrtc/blob/develop/docs/build-from-source.md

Also, you can try changing WebRTC module that haxball.js currently uses. I don't prefer them because they are deprecated, you can give it a try.
wrtc
@koush/wrtc


Runtime error when using bundle exec aws_lambda_ric from ENTRYPOINT

I'm installing the aws_lambda_ric using bundle along other gems  (s3) the application needs in the Dockerfile
I tried then this:
ENTRYPOINT ["bundle", "exec", "aws_lambda_ric"]
Lambda fails:
{
  "errorMessage": "RequestId: 62a1664f-58dc-48d7-8f80-0ac023a8fdd1 Error: Runtime exited with error: exit status 1",
  "errorType": "Runtime.ExitError"
}

and the full log:
**/usr/local/gems/bundler-2.3.22/lib/bundler/definition.rb:518:in `materialize': Could not find aws-sdk-s3-1.114.0, aws_lambda_ric-2.0.0, rspec-3.11.0, aws-sdk-core-3.152.0, aws-sdk-kms-1.58.0, aws-sigv4-1.5.1, rspec-core-3.11.0, rspec-expectations-3.11.1, rspec-mocks-3.11.1, aws-eventstream-1.2.0, aws-partitions-1.634.0, jmespath-1.6.1, rspec-support-3.11.1, diff-lcs-1.5.0 in locally installed gems (Bundler::GemNotFound)
from /usr/local/gems/bundler-2.3.22/lib/bundler/definition.rb:197:in `specs'
from /usr/local/gems/bundler-2.3.22/lib/bundler/definition.rb:254:in `specs_for'
from /usr/local/gems/bundler-2.3.22/lib/bundler/runtime.rb:18:in `setup'
from /usr/local/gems/bundler-2.3.22/lib/bundler.rb:162:in `setup'
from /usr/local/gems/bundler-2.3.22/lib/bundler/setup.rb:20:in `block in <top (required)>'
from /usr/local/gems/bundler-2.3.22/lib/bundler/ui/shell.rb:136:in `with_level'
from /usr/local/gems/bundler-2.3.22/lib/bundler/ui/shell.rb:88:in `silence'
from /usr/local/gems/bundler-2.3.22/lib/bundler/setup.rb:20:in `<top (required)>'
from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
/usr/local/gems/bundler-2.3.22/lib/bundler/definition.rb:518:in `materialize': Could not find aws-sdk-s3-1.114.0, aws_lambda_ric-2.0.0, rspec-3.11.0, aws-sdk-core-3.152.0, aws-sdk-kms-1.58.0, aws-sigv4-1.5.1, rspec-core-3.11.0, rspec-expectations-3.11.1, rspec-mocks-3.11.1, aws-eventstream-1.2.0, aws-partitions-1.634.0, jmespath-1.6.1, rspec-support-3.11.1, diff-lcs-1.5.0 in locally installed gems (Bundler::GemNotFound)
from /usr/local/gems/bundler-2.3.22/lib/bundler/definition.rb:197:in `specs'
from /usr/local/gems/bundler-2.3.22/lib/bundler/definition.rb:254:in `specs_for'
from /usr/local/gems/bundler-2.3.22/lib/bundler/runtime.rb:18:in `setup'
from /usr/local/gems/bundler-2.3.22/lib/bundler.rb:162:in `setup'
from /usr/local/gems/bundler-2.3.22/lib/bundler/setup.rb:20:in `block in <top (required)>'
from /usr/local/gems/bundler-2.3.22/lib/bundler/ui/shell.rb:136:in `with_level'
from /usr/local/gems/bundler-2.3.22/lib/bundler/ui/shell.rb:88:in `silence'
from /usr/local/gems/bundler-2.3.22/lib/bundler/setup.rb:20:in `<top (required)>'
from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
START RequestId: 62a1664f-58dc-48d7-8f80-0ac023a8fdd1 Version: $LATEST
END RequestId: 62a1664f-58dc-48d7-8f80-0ac023a8fdd1
REPORT RequestId: 62a1664f-58dc-48d7-8f80-0ac023a8fdd1	Duration: 517.48 ms	Billed Duration: 518 ms	Memory Size: 1024 MB	Max Memory Used: 24 MB
RequestId: 62a1664f-58dc-48d7-8f80-0ac023a8fdd1 Error: Runtime exited with error: exit status 1
Runtime.ExitError**

My Gemfile looks like this:
# frozen_string_literal: true

ruby '2.7.0'

source 'https://rubygems.org'

git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }

gem 'aws-sdk-s3'
gem 'aws_lambda_ric'

group :development, :test do
  gem 'rspec'
end

@cristianpaul0 it would be helpful if you show your Dockerfile too


Create train directory if not exists

Fixed #30 by adding check if dir exists

@pavelromashkin  Thanks for your contribution.


Missing currency

MRO seems to be missing.
I'm not sure if that's an effort to keep the library current or not but people will have records going back while MRO was still in use so need that to maintain integrity.
For whatever reason it's missing, it would be useful to have an additional column to indicate if the currency is still in use / circulation or not.

https://en.wikipedia.org/wiki/Mauritanian_ouguiya
See here that they replaced MRO with MRU.

I can't exactly remember now, but it seems that I've taken the active list from the wikipedia article.
We could add an active flag to the dataset and include the historical currencies.
If you add that in a PR I can take a look at it.

Active would make sense.
Mapping isn't necessarily so important, more of a thought. When a country changes one currency to another in the standard cases the previous is kaput rather than interchangeable with the previous.
Although mapping would be informative about the history of a country's currency transitions, I think think there are actually many use cases where it would be actionable.
I think the wiki will be from ISO. ISO can be a bit of a pain because it's behind a paywall and last time I considered paying for something (not too hard though) I wasn't immediately presented with a list containing any options that were obviously machine readable exports.
Using ISO as the limit for historical cut off would make sense. Otherwise adding Roman currency, etc would be too much (no one but a handful of historians or coin collectors will have computer records going that far and I'd assume if need be they'd establish their own standardised database for that).
https://github.com/moneyphp/iso-currencies/issues/8
This does come up sometimes in other places.
A search shows me some lists without MRO.
I wonder if this is a source problem, IE, by default ISO releases live currencies and people are using only that without bothering to get a historic list or not one available making the only way to take all release and merge them.
When I have some time I'll dig into what's actually on the ISO website.

A quick survey seems to suggest ISO is being a pain with the way they chain their currency standards (ISO 4217:YEAR) requiring an archaeologist to piece things together.


parser: fix some parse error for parenthese subqueries

Signed-off-by: lzmhhh123<EMAIL_ADDRESS>What problem does this PR solve?
close https://github.com/pingcap/parser/issues/1127
What is changed and how it works?
Check List
Tests

Unit test
Integration test

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change

Side effects

Possible performance regression
Increased code complexity

Related changes

Need to cherry-pick to the release branch

/rebuild

/rebuild

Please resolve the conflicts~

/merge

/run-cherry-picker

/run-cherry-picker


Too many repeated parameters are recommended after I update pylance！

Environment data
version: 1.47.3 (user setup)
commit: 91899dcef7b8110878ea59626991a18c8a6a1b3e
date: 2020-07-23T13:12:49.994Z
Electron: 7.3.2
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363
Pylance: 2020.8.0
To find your version, you can either:
Incredibly repeated parameters

This issue is similiar to #12503，now pylance has the similiar problem.

Thanks for the bug report. I've introduced a fix, and it will be in the next release of Pylance.
In case you're curious of the cause of the bug, Pylance was adding symbols from the current scope and all outer scopes (which is correct), but it wasn't properly detecting and dedup'ing symbol names that appeared in outer scopes.

Thanks for your reply.😁

This issue has been fixed in version 2020.8.1, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/master/CHANGELOG.md#202081-13-august-2020

This issue appears in pylance v2020.11.3-pre.1

@CatNofishing That's a different issue, actually; we probably don't want to be offering autoimports for things that are already named parameters, or at least ranking them differently. Would you mind opening a new issue with that screenshot so we can track it? This is a closed issue that was fixed a different way.


Make NTP config geographically diverse

As part of our audit of the AU family in the SSP, it came to light that we should be using geographically diverse NTP servers (AU-8(1) req.1). Using 0|1|2.pool.ntp.org, as I see in the repo, does not guarantee geographical diversity.
We should change our configs to guarantee geographical diversity to satisfy the SSP, perhaps by using the continental zones: http://www.pool.ntp.org/zone/@

https://github.com/18F/cg-deploy-bosh/pull/25 and again

Just waiting on a thumbs-up and merge so we can redeploy and test.

ssh into anything and dump out the CTP config.

We logged into a random box and deduced where ntp corrections come from (root's crontab runs ntpdate every 15 minutes, coming from here ). We observed the correct configuration in /var/vcap/bosh/etc/ntpserver.

paired with @datn and verified 🎉

This is something that should be put into 18F/cg-atlas#125 the Runbook issue. Something that can come out of this today would be to put @datn's comment into cloud.gov/docs under some Runbook heading in Contributions > Opersations > Runbook?. We're not sure on the name. But the
NTP settings aren't easily discoverable in the bosh documentation. @datn and I had to go through some source in cloudfoundry/bosh and within the vms to find out how it was being put into root's crontab.


SAKIII-4144 change newest text to most recent

https://jira.sakaiproject.org/browse/SAKIII-4144

Merged via squash


Rework hardware acceleration decoder selection

Bit to explain here...
tl;dr, I needed a way to force the use of a specific hwaccel/decoder pair.
This PR addresses two problems:

Improve decoder selection logic to only use decoders that are included in hwaccel_decoders.

Currently, the script will attempt to use a decoder that matches <input codec>_<hwaccel name>, provided it exists in ffmpeg -decoders. The first and largest issue with this approach is that this codec might not actually be supported by the underlying hardware. For example, if the input codec is av1 and you've provided cuda as a hwaccel, then you'll need Nvidia 30-series or later to run the av1_cuvid decoder, but ffmpeg doesn't know that.
Additionally, the script will currently append the first valid (exists in ffmpeg) hwaccel it finds.  In my case, even though my GPU didn't support av1, I'd like to use my CPU's iGPU to perform the decoding. This PR makes that possible.

Users might want to select a specific decoder that doesn't fit the <input codec>_<hwaccel name> mold. I added a new setting to override/manually control the process.

I wasn't aware of this, but when trying to solve my use-case I learned that there are internal ffmpeg codecs that support hardware acceleration, and there are external ffmpeg codecs specifically built for a single hardware platform. ffmpeg will implicitly use the internal codec unless you tell it otherwise.
i.e. you can run the implicit hevc decoder with -hwaccel cuda, or run -vcodec hevc_cuvid with -hwaccel cuda. I don't know too much about how these are maintained separately, but I read that sometimes there are differences in implementation that make one more efficient, so perhaps this will be useful to some.

Core problem: there is no good way to query ffmpeg to know if a given decoder is actually going to work with the provided hardware. The only one who knows whether it's going to work is the user.
Might be helpful to run through my situation to understand why this might be useful.
Hardware:

GPU: 1650 Super w/ Turing NVENC/NVDEC
CPU: i12400

Previously, I had
[Converter]
...
hwaccels = cuda
hwaccel-decoders = h264_cuvid, hevc_cuvid, mjpeg_cuvid, mpeg1_cuvid, mpeg2_cuvid, mpeg4_cuvid, vp8_cuvid, vp9_cuvid
hwdevices =
hwaccel-output-format = cuda:cuvid
...
[Video]
codec = hevc_nvenc, hevc, h265, x265, h264, x264
...

This worked well until I ran into an AV1 file, which failed transcoding
[av1 @ 0x55e6c68fdd80] Hardware is lacking required capabilities
[av1 @ 0x55e6c68fdd80] Failed setup for format cuda: hwaccel initialisation returned error.
[av1 @ 0x55e6c68fdd80] Your platform doesn't support hardware accelerated AV1 decoding.
[av1 @ 0x55e6c68fdd80] Failed to get pixel format.

My iGPU supports AV1 decoding, so I figured I could wrangle the settings into performing the decode with that. However, the following fails because the script attempts to use cuda.
hwaccels = cuda, vaapi

Even with the change to filter by hwaccel_decoders, the script then searches for <input codec>_<hwaccel name> and there is no av1_vaapi. The solution would be to use -hwaccel vaapi with -vcodec av1.

vcodec would actually be optional due to ffmpeg using it implicitly

Thus the second change.
If you have a cleaner solution I'm happy to work it out, but I think this works pretty well. I explored modifying the way hwaccel_decoders works, where we could detect a listed decoder that wasn't valid according to ffmpeg but corresponded to an internal decoder with hardware acceleration. Adding a new setting to manually specify hwaccel/decoder pairings seemed much cleaner and less confusing to users.

Reviewing this now
Question though, does using -hwaccel vaapi -vcodec av1 actually use your iGPU?
If your ffmpeg build doesn't have a vaapi decoder for av1 isn't it just falling back to software based on those options? Perhaps my understanding isn't correct but that was my assumption
Also, could you share the ffmpeg command generated for this output
[av1 @ 0x55e6c68fdd80] Hardware is lacking required capabilities
[av1 @ 0x55e6c68fdd80] Failed setup for format cuda: hwaccel initialisation returned error.
[av1 @ 0x55e6c68fdd80] Your platform doesn't support hardware accelerated AV1 decoding.
[av1 @ 0x55e6c68fdd80] Failed to get pixel format.

And additionally could you share the command generated with your fork?
Seems like I probably need to allow -hwaccel to be added if the decoder list is empty since some people still use that basic config option to get some generic hwaccel

Question though, does using -hwaccel vaapi -vcodec av1 actually use your iGPU?

It does, confirmed with intel-gpu-top. I assume it would fall back on software but I believe that's the best option.
Was about to grab you some ffmpeg outputs but it looks like something just changed with the sonarr-sma alpine container, and SMA_USE_REPO now installs a build of ffmpeg that doesn't have any hardware encoding support.

Had a bug with the startup script on sonarr-sma that I just fixed like 2 minutes ago, might want to just do a fresh pull and try again

Yeah upon reviewing the decoders, it looks like vaapi is the only one that doesn't respect the codec_hwaccel naming convention for the decoders (though it does for the encoders)
Could probably hard-code that exception

meh, I'm now an hour down a rabbit hole discovering why I have a half-finished attempt to put ffmpeg behind a thin http api so that we can abstract it from alpine-based images (maybe I'll revisit). My sonarr-sma image was in a weird state. I have no idea how I had a build of ffmpeg that supported qsv/vaapi/cuda/nvenc (I run a pull & up pretty regularly, maybe there was some weirdness in my docker stack left over from using the #build tag).
Regardless, off the top of my head I believe the script without any changes generated:
ffmpeg -hwaccel cuda -i av1_input.mkv -vcodec hevc_nvenc hevc_output.mkv

And now with the changes it generates
ffmpeg -hwaccel vaapi -vcodec av1 -i av1_input.mkv -vcodec hevc_nvenc hevc_output.mkv

Currently, the script will always set -hwaccel to the first entry in hwaccel that exists in ffmpeg. Definitely useful to be able to filter that selection by also requiring the decoder to exist in hwaccel_decoders. Hard-coding an exception to vaapi would definitely solve that issue, but I think there is still value in letting users force a vcodec (if they really wanted to use -hwaccel cuda -vcodec hevc instead of -hwaccel cuda -vcodec hevc_cuvid). Likely very rare that anyone would use it, but it allows for a lot of control/edge cases.

Thoughts on adding support for https://github.com/jellyfin/jellyfin-ffmpeg? Looks like a great source of pre-compiled ffmpeg binaries w/ hwaccel support.


LooksRare

The PR updates nft-marketplace schema for a bit. And implements LooksRare subgraph.
Deployment: check out https://okgraph.xyz/?q=0xbe1%2Fmessari-looksrare-ethereum

There are 2 piece of data that I am not sure how to extract:

listedTokens: LooksRare stores orders off-chain, therefore I cannot access tokens that are currently on-sale through the smart contract.
floorPriceETH: For the similar reason, there is no way to tell the floor price of tokens that are on-sale

Thoughts? @this-username-is-taken @mrbrianhobo

listedTokens: LooksRare stores orders off-chain, therefore I cannot access tokens that are currently on-sale through the smart contract.

since this is the case for the listedTokens field, i don't think we can extract this information with any marketplace so i would suggest removing this field. i forgot that orders are stored off-chain and originally thought that there might be an event emitted when listing an NFT (but that doesn't make sense).

floorPriceETH: For the similar reason, there is no way to tell the floor price of tokens that are on-sale

instead of floor price referring to tokens listed on-sale, i was thinking we could use this field for the floor price of sold tokens (fulfilled orders on-chain). i also agree with the sentiment that a historical all-time floor price for a collection doesn't make much sense, so i'd opt for it to be moved to the daily snapshot entity or renaming it something like floorPriceETH7d or floorPriceETH24h to make it clear what the current floor price of a collection is over a time period.


Django debug toolbar and raw() query

Hello,
I described my issue on http://stackoverflow.com/questions/7594529/django-debug-toolbar-and-raw-query
and was referred here. Possible bug?

I tried to add a  SQL similar to your post in StackOverflow but dint get any error with the latest DDT. Can you add a sample testcase or a views.py with the problem?


add missing unit tests

@cfkarakulak can you remind me where things landed with this? I think you wanted to make some changes but I can't remember if you wanted to hold on merging these until after your changes or not.


Where is the QuickPulseTelemetryModule in code?

Trying to figure out where to put in the code mentioned in the guide.

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

ID: e3307edb-b27c-a0b9-ff1e-610630c98c15
Version Independent ID: 20262b8a-31ff-4df9-dd05-859001f24eba
Content: Diagnose with Live Metrics Stream - Azure Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/live-stream.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@shyamal890 Thank you for your query, we will look into it and get back to you at the earliest.

No, I also have applicationinsights.config file in conjunction with this code in startup.cs

if you are using applicationinsights.config there is no need to make the additional code level changes you can just make the adjustment to the config file without needing to modify your code:
<Add Type="Microsoft.ApplicationInsights.Extensibility.PerfCounterCollector.QuickPulse.QuickPulseTelemetryModule, Microsoft.AI.PerfCounterCollector">
      <AuthenticationApiKey>YOUR-API-KEY-HERE</AuthenticationApiKey>
</Add>

One of our engineers also update the code example which make that less confusing in the future. But again if you are have a .NET Framework app and are using applicationinisghts.config it is easier to just make the single line change to the config file.

I am going to close-out this issue, but if further updates are needed in the docs just let us know.

#please-close


Handle unknown subjects in the retirement worker

This PR changes the retirement worker to ignore any unknown subject ids passed in. Previously this would raise and error and stop processing the list of subjects we had for retirement. Thus 1 incorrect subject_id could break the retirement list processing at any point.
I found this issue due to a failed (dead) job in sidekiq and some data debugging found 1 unknown subject id in the list.
linked HB error https://app.honeybadger.io/projects/40595/faults/38047692
Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

This looks good to me.
One question: would it be considered overkill if we add a test where subject_ids is nil or empty? (I'm assuming in those cases RetirementWorker would not be expected to perform). Totally fine if you think its an unnecessary.

One question: would it be considered overkill if we add a test where subject_ids is nil or empty? (I'm assuming in those cases RetirementWorker would not be expected to perform). Totally fine if you think its an unnecessary.

If it's nil or an empty array the first_or_create! would error and thus just skip any other work. So I think we're covered for behaviour but maybe we could add a spec.
Tbh this should only get called after Param checking events in the operation code etc which handles the nil / empty checks https://github.com/zooniverse/panoptes/blob/85af7c8f4afc9ca55632715648d5e39ef459064c/app/operations/workflows/retire_subjects.rb#L19

Ah got it. I forgot the context of when this is run (through the operation) and was thinking about if someone ran from the RetireSubjectWorker straight, which shouldn't be a concern since normal users shouldn't be able to do this anyway. I'm ok with not having the test.


UnregisterMenu

Hello, currently I am looking for a "UnregisterMenu" I make sure that I unregister all command before hand, but I have added this to CommandServiceBase (and the Interface as well)
        public virtual void UnregisterMenu(object menuTag, ICommandClient client)
        {
            MenuInfo info = GetMenuInfo(menuTag);
            if (info == null)
                return;

            UnregisterMenuInfo(info, client);
        }

        protected virtual void UnregisterMenuInfo(MenuInfo info, ICommandClient client)
        {
            info.Commands = 0;
            m_menus.Remove(info);
        }

Which seems to remove the menu properly, but for some reason when re-registering the menu it never shows up. I have no idea what I am missing and was hoping someone could enlighten me.

I also noticed that this isn't caused by my removal code at all. I disabled all of that code all together so it isn't referenced. Then if I leave the MenuInfo added, for example, with the information MenuText = "Test" CommandTag = "<random string>:Test" and I have confirmed that the random string before is different on the re-add. I have checked the CommandService.m_commands and the command gets removed and readded, but the menu bar stops updating. I don't know why.


Clear Screen

Scott is there a way there could be a " clear screen " button that would remove all data from the screen during a frequency change. I know that one could exit and restart but just wondering how hard it would be to implement.

Although not displayed on the spectrogram, the program is aware of the frequency of every WSPR spot. Perhaps I could have it only show spots in the same band as the dial frequency?

done, will be in next version


script execution is disabled on this system

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[ ] Search the existing issues.
[X] Refer to the FAQ.
[ ] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
If I execute this command: "notepad $PROFILE," the terminal opens my notepad, and I paste this line: "oh-my-posh init pwsh | Invoke-Expression." However, when I create a new tab, the terminal displays the following message: ". : Cannot load the file C:\Users\Roby3\Documents\WindowsPowerShell\Microsoft.PowerShell_profile.ps1 because script execution is disabled on this system. For more information, see about_Execution_Policies at https://go.microsoft.com/fwlink/?LinkID=135170.
At line character: 1:3"

Expected behavior
1

Actual behavior
1

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      5.1.22621.2506
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.22621.2506
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Visuals
No response

Ok I fix my problem


void -> Promise

Hey,
I want to use this library but I need to have promises, that's mandatory requirement for me, I'm going to use attachSync and I need to do awaits here.
I've already done development (https://github.com/jokio/ts-events), I'll need to re-write some tests and pull request will be ready, if its okey from you.
I was also thinking about having attachSync and attachAsync, when there will be attachSync with Promise, will still be necessary async version ?
P.S. thanks for creating this library

Hi @playerx thank you for reaching out. A couple of points here:

Promises are by definition a-synchronous (hence the async keyword). So having a promisified attachSync is a contradiction in itself.   Therefore you'd have to use attachAsync(), however then queue semantics become quite difficult and counterintuitive to implement.

What meaning would you need to give to a promisified event?  Would you want the sender to wait for all receivers to be called? If you do, how would this work with queued events?  And how would this work if multiple handlers returned an error? Will you aggregate the errors into one? What about their stack traces? Please note that you cannot use Promise.all() for this because this will leak unhandled exceptions if multiple promises fail.

You changed the post() method from sync to async method.  This is not backward compatible.  In keeping with terminology from other packages, you could make a new 'send' method next to the original 'post' method.  In various APIs, like the windows API, 'post' means 'fire and forget' while 'send' means 'wait for the result'.

Looking back at these remarks, I'd say the best route would be to either make a new type of event or a new NPM package to do what you want?

@rogierschouten  thanks for feedback. let me share my case scenario. I decided to implement ddd (domain driven design) with typescript and was looking for events library for mediator pattern. I liked ts-events because:

it has three different attach types and for my case scenario all of them has different use case, I'll share about it later
there was AnyEvent, you just post the event and only think how attach will work (sync, async) when you attach to it.

only problem I has was that when I wanted to had transaction with events and had async operations in events.
Now I think the solution for me would be, promisified attachAsync. But that needs post to be async also, and if we want AnyEvent, there are different api-s for syncPost and asyncPost, thats the problem I see here.
Maybe different package would be better here, but I liked this package very much and for me it would be great if we will find some solution and make it happen here.
thanks

here is the sample how I used it for one transaction:
https://github.com/altasoft/ddd-sample/blob/master/src/mediator/account.ts

I think thats what I need:
export class DomainEvent<TData>  {
	private actions: DomainEventAction<TData>[]
	private handleError = (action: DomainEventAction<TData>, catchError) => async (data: TData) => {
		try {
			await action(data);
		}
		catch (err) {
			if (catchError)
				console.warn(err);
			else
				throw err;
		}
	}

	async post(event): Promise<void> {
		await Promise.all(this.actions)
	}

	attach(action: () => Promise<void>) {
		this.actions.push(this.handleError(action, false));
	}

	attachSafe(action: () => Promise<void>) {
		this.actions.push(this.handleError(action, true));
	}
}

Previously I had:
export class DomainEvent<TData> extends AnyEvent<Event<TData>>{ }

This was a while ago. @playerx did you ever find a solution? I'm currently trying to use this package for a similar reason but i'm running into the same problem.

Hi, peoples that that needs promises support might want to check out https://evt.land

closing due to inactivity, please feel free to re-open and indeed follow suggestion from @garronej  above


Add support for all wf:functions

Hello,
Today we had a meeting about how to implement this functionality.
Notes are available
https://hackmd.io/T0YBhLtRSZCbx49GtMtNxQ
Thanks,

Today we discussed this and decided to change the objective from supporting all wf:functions to instead implementing a sample wf:function only. It will demonstrate how to access data from the Airflow database inside the function.
Apart from that, we'll describe it in the CONTRIBUTING documentation.
This way if anyone intends to implement the remaining wf:functions they'll have a tangible example and instructions on how to do so.


[Fix] run_robot_test.sh - OC login and return value

This fixes OC login on my local Fedora 38 machine. Without this change,
the strings contain also `"`.

Also missing `-` in front of the switch isn't allowed.

`yq` help:
  -e               set the exit status code based on the output;
  -r               output raw strings, not JSON texts;

```
$ yq --version
yq 3.2.3
```

Note - regarding the shellcheck, there is also this PR opened #936.

@tarukumar regarding our discussion whether to remove the e - if we remove it, then the script returns 0 even if the searched key isn't present in the provided variables file, see e.g.:
$ yq -r '.OCP_API_URLf' "ods_ci/test-variables.yml" ; echo $?
null
0
$ yq -er '.OCP_API_URLf' "ods_ci/test-variables.yml" ; echo $?
null
1

So, I guess we want to keep it there? Truth is that looking on the code, it won't stop execution even in case of failure. I can update that.

I added a commit with the check for the yq return value. Also rebased against master.

Rebased.
Also created #953.


Obtain credentials from environment - attempt # 2

This PR is mostly the same as #57, but just includes the changes to the authenticate function that (hopefully) will not be too complicated to discuss in a PR.  The design of the authenticate function is moduar so that the supporting functions can be used in different authentication flows, each of which is called from the primary authenticate:

        # Step 1: read in configuration file, if it exists
        if os.path.exists(self.config):
            config_data = self._authenticate_config(config_data)

         # Step 2:, get username/password from environment
        config_data = self._authenticate_environment(config_data)

         # Step 3: load into configuration!
        self._load_config(config_data)

The idea is that we should have a forward thinking mindset that other methods might be wanted, and it should be easy to add here.
I'm being very conservative because I spent a lot of time on the last PR, and most of the work now needs to be redone. This will part 1 of the revisions, and I won't proceed on the others until each is gone through! Thanks for your help and guidance with this work @rysteboe - if we both hang in there we can do this together! :)

okay I pushed some changes, but the functions for auth_ need discussion for the name, and there is also no testing set up! I'm literally needing to go to the airport right now but if you want to:

choose the name you like
clarify the indent
discuss preference for testing (e.g., travis, circle)

I can do these things when I'm back at keyboard. Gotta jet now, looking forward to more work on this!

Also, I don't know if you didn't have time to do it (understandable) or didn't notice the comment (even more understandable since Github doesn't work nicely with comments on deleted lines), but I made some notes on get_config_path that have to be addressed.

I just ran out of time, but wanted to get the changes I had done this far! Just boarding my plane now (typing on my phone)
I haven't manually tested yet - I will do this first too to save you some time in case there is a bug I missed.
Bon voyage! ✈

okay I've pushed new changes:

the two functions (for default config and data directory) are now separate
I added a starter script for testing the authentications. Feel free to scrap it, I started it before I saw you were already working on it!
still need to discuss function names

Overall looks good to me.  Do the function rename IMO and whatever tests you want to do.  Like I said, I'll do some manual tests before doing a release (although those will come after merging this so that I can pull it into our private repo easily).
wrt testing in general, I hope I'll be able to push the competitions tests this week.  If I don't have time to finish them, I'll at least try to push them up as a template.  Should note that I come from a pretty heavy Java background, so I make no promises that they will be particularly pythonic tests.  Will ping in #58 when that's done.

LGTM.  Let me know if you're ready to merge.

Ready! And as you mentioned, we can tweak the tests with the next PR.
 I can do another PR to get started on that with the latest version (after merging). I was doing something like:
        '''autheticate config is the first effort to get a username
           and key to authenticate to the Kaggle API. Since we can get the
           username and password from the environment, it's not required.

           Parameters
           ==========
           configuration: the Configuration object to save a username and
                          password, if defined
           quiet: add verbosity
        '''

I'll do the release tomorrow.  Having to balance API stuff against some other infrastructure work I have going on :)

okay! Good luck!

kind of necro, but is there a reason why paths to the kaggle.json file can't be declared manually in authenticate? Giving the authenticate method a kwarg path would handle this nicely.


add Clone/Copy to SslVersion

actually there are other enums missing these two derives. do you thin we should add them all?

Sure. Created a PR here: https://github.com/alexcrichton/curl-rust/pull/291


HDDS-7633. Compile error with Java 11: package com.sun.jmx.mbeanserver is not visible

What changes were proposed in this pull request?
Fix error when compiling Ozone for target Java version 11:
$ mvn -Djavac.version=11 -DskipTests clean package
...
[ERROR] hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/HddsDatanodeService.java:[74,19] package com.sun.jmx.mbeanserver is not visible
[ERROR]   (package com.sun.jmx.mbeanserver is declared in module java.management, which does not export it)

Change compile CI check so that it, in addition to using JDK of the specified version, also targets the same Java version.  This way we can catch such errors in CI.
https://issues.apache.org/jira/browse/HDDS-7633
How was this patch tested?
$ mvn -Djavac.version=11 -DskipTests clean package
...
[INFO] BUILD SUCCESS

https://github.com/adoroszlai/hadoop-ozone/actions/runs/3687864892/jobs/6242414957

cc @GeorgeJahad can you please take a look.

Thanks @kerneltime for the review.


Update sbt-scalafix to 0.9.32

Updates ch.epfl.scala:sbt-scalafix from 0.9.18 to 0.9.32.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala", artifactId = "sbt-scalafix" } ]

labels: sbt-plugin-update, early-semver-minor, semver-spec-patch

Superseded by #67.


etcd 0.4.6 crashes out of memory, then panics on startup with index out of range.

Using etcd 0.4.6 -- presumably fix is to move to 2.0?
Commandline: /usr/local/bin/etcd -peer-addr k8s-prod3-master:7001 -name k8s-prod3-master
[etcd] Jan 29 16:40:16.658 INFO      | k8s-prod3-master: snapshot of 10060 events at index 670889 completed
fatal error: runtime: out of memory

Then for each future startup:
[etcd] Jan 29 17:45:21.732 INFO      | Peer URLs in log:  / k8s-prod3-master ()
[etcd] Jan 29 17:45:21.735 INFO      | etcd server [name k8s-prod3-master, listen on <IP_ADDRESS>:4001, advertised url http://<IP_ADDRESS>:4001]
[etcd] Jan 29 17:45:21.736 INFO      | peer server [name k8s-prod3-master, listen on :7001, advertised url http://k8s-prod3-master:7001]
[etcd] Jan 29 17:45:21.736 INFO      | k8s-prod3-master starting in peer mode
[etcd] Jan 29 17:45:21.736 INFO      | k8s-prod3-master: state changed from 'initialized' to 'follower'.
[etcd] Jan 29 17:45:22.106 INFO      | k8s-prod3-master: state changed from 'follower' to 'candidate'.
[etcd] Jan 29 17:45:22.106 INFO      | k8s-prod3-master: state changed from 'candidate' to 'leader'.
[etcd] Jan 29 17:45:22.106 INFO      | k8s-prod3-master: leader changed from '' to 'k8s-prod3-master'.
panic: runtime error: index out of range

@alex-mohr Can you please check how large is the snapshot and log file? Thanks.

@xiang90 sorry, I don't have the snapshot or log file anymore.   I'll start a repro with a much larger VM (50GB ram instead of 4GB) and see where that ends up.  Took ~12 hours last time.

@alex-mohr Besides, do you have any scalability requirement in mind? How many keys do you want to save. What is avg size of keys?

@xiang90 I only had ~50 keys written at any given time.  The workload was create ~50 keys, wait for that batch to be processed, delete the keys, repeat.  Run with defaults, so 10k events between snapshots, though.

@alex-mohr What is the average size of keys?
etcd does put 10k events in the memory, and it is usually fine.

@alex-mohr I am closing this due to low activity. You might want to try to reproduce with etcd 2.0.x and report back with more information if there is still some problems.


Translate

I can translate to Portuguese Readme?

Sure you can :) Do you mean documentation http://planck.github.io/ ? Or just Readme.md at the root of this repo?

I can also try translating the documentation.
Do you have any instructions for passing me by?
@PinkaminaDianePie

@samverneck I never seen if anyone have translated README.md file. But as for documentation - some repos like to have multi-language docs. But with in this case if i'll upload new version to npm - i will need to update both English and Portuguese documentation at once, so if you can be interested in maintaining Portuguese - it will be OK, but if you want to just translate it once - it may be better to stay with only English version, because I don't know Portuguese and I can't maintain Portuguese docs alone. So if you really want to help - you can fork https://github.com/planck/planck.github.io and translate _posts and _pages directory on it. After that we can thought about how to merge it in single repo - i think we can create many versions of each file with prefixes (en.readme.md, pt.readme.md etc) and i'll create page/dialog for language selection. Also if you have any plans to use Planck in your own projects, email me at<EMAIL_ADDRESS>i have something to discuss.

let's keep in touch. We can translate yes the documentation I will join a galera here but for now I'm going to do that can be?
Are you able to understand my English?
@PinkaminaDianePie

@samverneck ok, as I said you can translate docs and i'll create page for language selection. You can ping me when you'll have something for merge.

Ok! Let's keep this contact daily then. @PinkaminaDianePie

@samverneck deal!

Ok! @PinkaminaDianePie

@samverneck So, how is it going?)

they are going! ;)

@PinkaminaDianePie Any questions I ask

@samverneck do you have any progress with translation?

@PinkaminaDianePie I'll post soon

@samverneck deal! ;)


[Feature Request] Add uddoktapay payment gateway on fossbilling

Add this gateway on fossbilling : https://github.com/UddoktaPay/whmcs-module https://github.com/UddoktaPay/raw-php

Sorry. This gateway is a fairly small regional one and we likely won't work on it.
You also can write the implementation yourself. Feel free to check the existing ones and ask for help in our Discord server if you're stuck.


Import without parameters

Describe the solution you'd like
It would be nice to be able to put imports without parameters in the plugin configuration.
Additional context

Please see the discussion in https://github.com/IanVS/prettier-plugin-sort-imports/issues/51 as to why we do not sort these kinds of imports.


[ Suggestion ] Auto claim daily check-in reward

i give suggestion to make an option "Auto claim" on "Daily reward" feature.
like this repository https://github.com/darkGrimoire/hoyolab-daily-bot
thank you

Thanks to suggest feature   x~x
Auto claim feature of Daily reward already plan to add on after version :)

Auto claim feature is added on Android version (v1.19.0)
Other platform will be added v1.19.x version later.

Will Auto claim be added for Honkai Star rail?

@Lii-Kun Honkai Star Rail Auto Claim feature will be updated :)

@Lii-Kun Honkai Star Rail Auto Claim feature is added on v1.21.0 release


chore: update sbt-multi-jvm

self evident

failure was repo.maven.apache.org being unreachable, re-running


[QUESTION] - shift_state (Parked, Driving,...)

At first: thank you sooo much @timdorr for this API!
I am trying to encounter the driving state of the tesla, but I am getting:
   "shift_state": null,
   "speed": null,

I am not getting something like: "Driving", "Parked", "Reverse", etc.
Does that mean, that null stands for Parked?
thanks :) 😊

Null tends to mean the car is parked and in some level if sleep state. Technically i think of it as “no data” but for the most part you only get into that state when parked for a bit.
Otherwise you’ll see “D”, “N”, “R”, “P”, and occasionally an empty string.

@davidhodge Thank you so much :) I was also thinking of that in null state is parking.


Rebuild Constraint Name Changes

Added pages preview:

https://deploy-preview-2102--prisma2-docs.netlify.app/docs/concepts/components/prisma-schema/names-in-underlying-database
https://deploy-preview-2102--prisma2-docs.netlify.app/docs/guides/upgrade-guides/upgrading-to-use-preview-features/enabling-named-constraints

Rich removed those when he tried to get the PR in a running state I think.

@molebox removed those when he tried to get the PR in a running state I think.

I did indeed remove them they confuse me, a tabs functionality is to provide more than 1 piece of information in an enclosed environment. I didn't see the point in having one tab. In most (if not all, i think) of the section where the tabs was removed, the text surrounding it indicates that its for relational databases, or is explicitly for mongodb

That's fair. But what is the connection to the namedConstraints preview feature?

That's fair. But what is the connection to the namedConstraints preview feature?

No direct connection apart from the tabs causing lots of issues with rendering the pages, I removed them as i went about fixing the issues left over from the merge up.
I of course understand what you're getting at here, this isnt related to this PR. But it was a product of refactoring so the pages would render.

Just important to know. If it bothers me during the review, I can quickly extract that change into its own PR - but I don't expect it will. This change is small enough.

Just important to know. If it bothers me during the review, I can quickly extract that change into its own PR - but I don't expect it will. This change is small enough.

Totally fair, tbh i didnt consider it an addition (or subtraction if you will) I was just trying to fix the pages. As i was doing it i wondered why we had these 1 tab sections.

Looks like we have many mention of Prisma "convention".
Maybe it would be nice to link somewhere so it's easy to find out what it is.
For example on this page it's not linked anywhere
https://deploy-preview-2102--prisma2-docs.netlify.app/docs/guides/upgrade-guides/upgrading-to-latest/upgrade-to-2290

For example on this page it's not linked anywhere
deploy-preview-2102--prisma2-docs.netlify.app/docs/guides/upgrade-guides/upgrading-to-latest/upgrade-to-2290

This page does not exist any more.
Most of the mentions of "Prisma convention" have comments open, that this should be de-emphasized and mention that people just want different things than the defaults often for various reasons.


Json Giving error

Tool version
3.1.0
Describe the bug
Terraform  giving error
To Reproduce
I am configured the components in naming tool according to company standards.
I am successfully able to run the API - Resource request name from the swagger tool using bellow Json
curl -X 'POST'
'https://test-app/api/ResourceNamingRequests/RequestName'
-H 'accept: /'
-H 'APIKey: ***********************************'
-H 'Content-Type: application/json'
-d '{
"resourceEnvironment": "t",
"resourceInstance":"03",
"resourceOrg": "m1",
"resourceProjAppSvc": "sap",
"resourceType": "vm",
"resourceid": 85,
"resourceunits/Depts": "db",
"customComponents": {
}
}'
Getting error when I run it through terraform .
Here is my terraform script where i am using data http block to generate the name using terraform
locals {
common_request_body = {
"resourceEnvironment": "t",
"resourceInstance":"01",
"resourceOrg": "m1",
"resourceProjAppSvc": "sap",
"resourceType": "vm",
"resourceid": 85,
"resourceunits/Depts": "db",
"customComponents": {
}
}
json_common = jsonencode(local.common_request_body)
}
data "http" "resource_naming_request" {
url = var.url
method = "POST"
request_headers = {
Content-Type = "application/json",
APIKey = var.APIKey
}
request_body = var.is_subscription ?  null  : local.json_common
}
response_content = "{"type":"https://tools.ietf.org/html/rfc7231#section-6.5.1\",\"title\":\"One or more validation errors occurred.","status":400,"traceId":"00-a6326ea594869925ab26254a47563f80-f4ba9b7b845000-00","errors":{"Name":["The Name field is required."],"DisplayName":["The DisplayName field is required."]}}"
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Installation Method
A description of the installation environment for the Azure Naming Tool.
Additional context
Add any other context about the problem here.

Hello Shiva,
Thank you for your feedback. In looking at the code it's not clear what the issue may be. The only model within the code with a ** Display Name ** field is the ResourceComponent.cs class. This model is used when creating a new Custom Component. I can't see in your Terraform script the actual API URL you are posting however, but regardless, this filed is not set manually when making a custom component anyway (the site will take the entered name and use that). THe error you have also shows the ** Name ** field is missing, so I am wondering if it is attempting to create a new custom component.  Additionally, the JSON contains a property (resourceunits/Depts) which doesn't exist in the system.
Can you please verify the URL you are using int he Terraform code?
Thank you!

Bryan

Closed due to inactivity.


Move RTK binaries into RTK binary directory

When building RTK against an ITK build directory, RTK binaries
go into the RTK binary directory.
When building RTK within ITK, RTK binaries are located in the ITK
binary directory.

Thanks!


Avoid failure on schedule metaparameter warning

Syntax checking was failing (exit code 1) when puppet raises a the following warning
schedule is a metaparam; this value will inherit to all contained resources in the

Applied the same approach as tag.

LGTM! Could you please rebase and squash your commits.

LGTM! Could you please rebase and squash your commits.

Great ! Done !

Test is already here.

Doh! Yes it is.. merging now :)


Include in postcss-preset-env ?

Should this polyfill be included in postcss-preset-env?
From what I can tell it seems relatively safe for users because it throws when something unsafe is encountered.

I think that would be reasonable, but it's really a decision for the postcss-preset-env owners.

I am one of the owners.
Let me rephrase.

I think this plugin would be a nice addition to postcss-preset-env and I would like to add it to postcss-preset-env. I don't want to add it unless you agree it's a good candidate.

Oh, I see! Yes, I think it would :smile:

Nice! 😄
I've drafted a PR : https://github.com/csstools/postcss-plugins/pull/1021
I will let you know when this ships 🙇


MLIR support

how to support mlir file about netron?

@victorygogogo can you provide some context. Are you trying to create a pull request to to add MLIR support? Can you share a sample model file, how it was created and how it is used.

mlir file content as flow:
module {
func.func @test_onnx_conv_simple_pattern(%arg0: tensor<5x3x32x32xf32>, %arg1: tensor<?x3x2x2xf32>) -> tensor<5x?x31x31xf32> {
%0 = "onnx.NoValue"() {value} : () -> none
%1 = "onnx.Conv"(%arg0, %arg1, %0) {auto_pad = "NOTSET", kernel_shape = [2, 2], pads = [0, 0, 0, 0]} : (tensor<5x3x32x32xf32>, tensor<?x3x2x2xf32>, none) -> tensor<5x?x31x31xf32>
return %1 : tensor<5x?x31x31xf32>
}
}
I am trying to find some tool to support mlir file to visualize as netron, if netron support is best.

@victorygogogo can you implement MLIR support and submit a pull request. Both an JavaScript parser and code to translate into objects for visualization can be added to mlir.js.

Duplicate of #1044