Fixing download 304 tests again.

Another attempt (hopefully the last one) to fix the download 304 tests.
This time around we're just going to use the LastModifiedTime from the Upload Request and set that as the Last-Modified time for the download request.
That way we absolutely can't be sending a Last-Modified time that is before the blob was uploaded.
Turns out the Recording.UtcNow grabs the time from when the test started, not the current time at the time of calling that API.

/azp run net - storage - tests

API change check
API changes are not detected in this pull request.

Looks like the 304 tests aren't failing, I'm going to run the live tests one more time after I merge main as for a sanity check.

Unrelated: There's a flakey tests I saw that ran on this pipeline.
FindBlobsByTagAsync_AccountSas
https://github.com/Azure/azure-sdk-for-net/issues/33552
Made a github issue to address this test later

/check-enforcer override


fix: do not force mixin booting

This package is forcing the SelectRelated mixin to boot, thus other mixins won't use the correct table name. If we force the mixin to boot, all other mixins will use "select_related_mixins" table name and will have some problems with static attributes. This PR aims to fix this behavior.
All tests are green.

Hey @Melchyore,
shouldn't we use hasOwnProperty instead of this.booted? I've referred the official BaseModel code from lucid for this
if (this.hasOwnProperty('booted') && this.booted === true) {
    return
}

super.boot()

Hey @Melchyore,
shouldn't we use hasOwnProperty instead of this.booted? I've referred the official BaseModel code from lucid for this
if (this.hasOwnProperty('booted') && this.booted === true) {
    return
}

super.boot()

Hey!
Never had ant issue using it this way in many mixins, so I assume it's safe.

Hey, @Melchyore
I just tested out your code on the https://github.com/chirgjin/adonisjs-select-related-example repo with the following model:
import { DateTime } from 'luxon'
import {
  BaseModel,
  column,
  HasMany,
  hasMany,
  HasOne,
  hasOne,
  ModelAttributes,
} from '@ioc:Adonis/Lucid/Orm'
import { compose } from '@ioc:Adonis/Core/Helpers'
import { selectRelatedMixin } from '@ioc:Adonis/Addons/SelectRelated'
import { Profile, UserTodoList } from 'App/Models'
import type { NormalizeConstructor } from '@ioc:Adonis/Core/Helpers'

function myMixin<Base extends NormalizeConstructor<typeof BaseModel>>(base: Base) {
  class MyMixin extends base {
    public x: string
  }

  // Third party library called the boot. We can't control it
  MyMixin.boot()

  return MyMixin
}

export default class User extends compose(BaseModel, myMixin, selectRelatedMixin) {
  @column({ isPrimary: true })
  public id: number

  @column()
  public email: string

  @column()
  public name: string

  @column.dateTime({ autoCreate: true })
  public createdAt: DateTime

  @column.dateTime({ autoCreate: true, autoUpdate: true })
  public updatedAt: DateTime

  @hasOne(() => Profile)
  public profile: HasOne<typeof Profile>

  @hasMany(() => UserTodoList)
  public todoLists: HasMany<typeof UserTodoList>

  /**
   * Helper to convert a columnName to `<table_name>.<column_name>` format for use with select related
   */
  public static col(columnName: NonNullable<keyof ModelAttributes<User>>): string {
    return `${this.table}.${this.$getColumn(columnName)!.columnName}`
  }
}

When the current code is running (the one with explicit boot call), the table name for User model is users
When the code from your PR is running, the table name for User model is my_mixins
When the code with hasOwnProperty is running, the table name is users

Can you check on your end as well? Maybe I can add a test for this if you'd like

Hey, @Melchyore I just tested out your code on the https://github.com/chirgjin/adonisjs-select-related-example repo with the following model:
import { DateTime } from 'luxon'
import {
  BaseModel,
  column,
  HasMany,
  hasMany,
  HasOne,
  hasOne,
  ModelAttributes,
} from '@ioc:Adonis/Lucid/Orm'
import { compose } from '@ioc:Adonis/Core/Helpers'
import { selectRelatedMixin } from '@ioc:Adonis/Addons/SelectRelated'
import { Profile, UserTodoList } from 'App/Models'
import type { NormalizeConstructor } from '@ioc:Adonis/Core/Helpers'

function myMixin<Base extends NormalizeConstructor<typeof BaseModel>>(base: Base) {
  class MyMixin extends base {
    public x: string
  }

  // Third party library called the boot. We can't control it
  MyMixin.boot()

  return MyMixin
}

export default class User extends compose(BaseModel, myMixin, selectRelatedMixin) {
  @column({ isPrimary: true })
  public id: number

  @column()
  public email: string

  @column()
  public name: string

  @column.dateTime({ autoCreate: true })
  public createdAt: DateTime

  @column.dateTime({ autoCreate: true, autoUpdate: true })
  public updatedAt: DateTime

  @hasOne(() => Profile)
  public profile: HasOne<typeof Profile>

  @hasMany(() => UserTodoList)
  public todoLists: HasMany<typeof UserTodoList>

  /**
   * Helper to convert a columnName to `<table_name>.<column_name>` format for use with select related
   */
  public static col(columnName: NonNullable<keyof ModelAttributes<User>>): string {
    return `${this.table}.${this.$getColumn(columnName)!.columnName}`
  }
}

When the current code is running (the one with explicit boot call), the table name for User model is users
When the code from your PR is running, the table name for User model is my_mixins
When the code with hasOwnProperty is running, the table name is users

Can you check on your end as well? Maybe I can add a test for this if you'd like

Hey!
Couldn't reproduce the bug on my end, but I'll add the suggested changes. Can you check now, please?
Thank you!

Thanks @Melchyore! Published v1.2.1


Change the key of the GCP service account keyfile

Airflow version 2.4.2 connection generates a key_file with the key named "extra__google_cloud_platform__keyfile_dict" while the create_profile_vars_google_cloud_platform() function tries to access it with the keyword "keyfile_dict", which was used in older Airflow versions.

@mmenalla, tysm for this contribution! I'm going to go ahead and merge it and it'll be included on our next release (which is hopefully going out today).


Fix scala-cli version used in our build

Why not just disable the use of Scala CLI on Windows?

It is not only Windows it also fails on Linux (my machine) and macs (Ina's). It is caused by the problem with handling multiple -O options

A problem is that --cli-version makes us use the JVM version of Scala CLI, which is kind of slow.

problem will be fixed in next release, we do not want to have it merged


Verify slow starting http  - Help wanted

Hello,
I want to verify a docker container running logstash with dgoss. Unfortunately logstash needs approx 30s to start. During this time I cannot check its port (9600).
How can I add a delay and timeout to the http check?

Hey! I think the retry timeout and sleep options are what you're looking for.
Check the output of 'goss validate --help' for details or https://github.com/aelsabbahy/goss/blob/master/docs/manual.md#validate-v---validate-the-system
Hope that helps.

Hey @kaizen63 did this help you achieve what you wanted to?

Hello Tim,
yes it helped. Tx a  lot.


Make the icons in the side panel configurable and clickable

The launch icons don't work and can be made configurable and clickable.

Hi, I would like to work on this issue. Can you tell me more on how I should do this?

Sorry for getting back a little late on this one. If you're still interested in working on this issue, here is what I believe should be done:
The arrow icon as on the image in the issue can be made clickable to launch a web URL using an external package.
These URLs can be made customisable for the portfolio owner to specify.

Yes, I'm still interested in working on this issue. I hope you won't mind me taking a bit of time on this issue since I'm quite busy lately, but regardless I will give it a try.

Yes sure, please take your time.


allow configuring tooltip for interactive guideline

When defining valueFormatter for the tooltip in interactive guideline, than it needs to be done recursively, hence:
} else if (key === 'tooltip') {
  configure(chart[key], options[key], chartType);
}

btw, in order for this to work, https://github.com/novus/nvd3/issues/1247 also needs to be addressed in nvd3, as I've changed it here: https://github.com/grmanit/nvd3/commit/23be42942d8c6c06cd679d2a6cbb8e033846d7e4

Thanks! I'll update this a bit.


⚠ GreatPosterWall has degraded performance

In eaa0b45, GreatPosterWall ($GPW) experienced degraded performance:

HTTP code: 200
Response time: 1897 ms

Resolved: GreatPosterWall performance has improved in d9c2ef7 after 6 minutes.


[BUG] can't use ChatALL on mac

Describe the bug
I am unable to open ChatALL app
To Reproduce
I have Apple M1 and I install ChatALL-1.18.12-mac-arm64.dmg
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):
macOS Ventura 13.3.1 (a)
Additional context
Add any other context about the problem here.

Click "Show in Finder", find ChatALL, tap it with two fingers.

still happens

https://support.apple.com/guide/mac-help/apple-cant-check-app-for-malicious-software-mchleab3a043/mac


[FR]: Support mocking for StorageReference

Description
Because it looks like FirebaseStorage is an open class, I cannot subclass StorageReference (because init is not public) for mocking purposes. The compiler throws an error that I cannot call super.init() because it is not accessible.
The ideal solution would be to make the init() method in StorageReference public so I can mock the class.
Please let me know if there is a better workaround for mocking StorageReference / Firebase storage uploading data.
class MockStorageReference: StorageReference {
   override init() {
       super.init() // 'StorageReference' initializer is inaccessible due to 'internal' protection level error
   }
}

API Proposal
No response
Firebase Product(s)
Storage

Thanks for the FR and link.  The conclusion is convincing that we should do one or the other?
As a workaround, what about @testable import FirebaseStorage?

Thanks for the suggestion for the workaround! I wasn't aware of @testableannotation. I ended up creating a MockClient wrapper as a workaround. Thanks!


Serialization of Auth_OpenID_CheckIDRequest with "complex" Stores

We use serialization of Auth_OpenID_CheckIDRequest to store the original checkid request instance at the session when we can't reply with the proper identity immediately (user needs to log in to our IDP). If the Store associated with the server is something "more complex", the serialization does not work (we'll hit some unserializable objects sooner or later in the object graph).
I think this is due the $server property, set in the _CheckIDRequest's constructor.
In our server implementation, we do currently unset $server whenever the Request is serialized. In turn the $server is reset upon unserialization. I feel this a bit hacky - so just asking if there is any better way to do it? Or should the Auth_OpenID_Request code get refactored to better support storing it externally?

This repo is being archived. Closing issue.


Who should own createdby, createdon, modifiedby, modifiedon

Today the spec has these as read-only attributes, which means they're owned/managed by the server. This means a few things:

it requires some non-spec-defined mechanism by which the server determines "who" is touching the resource
if someone does an "import" type of operation all of the "by" and "on" fields will set to that one person and have the current time

Are these points ok? Should we make them mutable by default but then allow the server's model to make them read-only if that's the desired semantics?

Due to PR #104 this one can now be closed


Add flags to enable/disable the update of RTE folder contents

Complementary to https://github.com/Open-CMSIS-Pack/devtools/pull/432

Released in 1.4.0.


fix(webpack): remove usage of fallbackLoader and loader

Fixes #4434

Just to clarify; this is to fix a warning, correct? There's no need for an emergency release, right?

That's right @hansl, the apps work fine and the warning only appears once

Im still getting this issue and I deleted node_modules and redid npm install. Is this not on a release branch or something?

@HauntedSmores it's been merged into master, it will be available in the next release.

@beeman Thanks for the response!

@HauntedSmores this file is part of Angular CLI itself, it's not inside your app.
If you want to manually get rid of this error, the file should be located in<EMAIL_ADDRESS>

This is not only a warning, this error appears 4 times during compilation, and it makes webpack compilation fail...

any ETA to get this new release. Currently, my build is broken ..

@Beeman I know it is, I checked the path from your repo but I don't have styles.ts in the directory you just gave

I think CLI doesn't install individual ts files .. I can't find it either ...

It is also in the project :
<EMAIL_ADDRESS>

I get this error too when running "ng serve" - although my app still compiles and boots up. (see below). So it's more of an annoyance than something critical. But curious, is there a current resolution or do we just need to wait for the next update?

bash-3.2$ ng s
fallbackLoader option has been deprecated - replace with "fallback"
loader option has been deprecated - replace with "use"
fallbackLoader option has been deprecated - replace with "fallback"
loader option has been deprecated - replace with "use"
fallbackLoader option has been deprecated - replace with "fallback"
loader option has been deprecated - replace with "use"
fallbackLoader option has been deprecated - replace with "fallback"
loader option has been deprecated - replace with "use"
** NG Live Development Server is running on http://localhost:4200. **
Hash: 86877c66b943cc1f9090
Time: 37200ms
chunk    {0} main.bundle.js, main.bundle.map (main) 754 kB {3} [initial] [rendered]
chunk    {1} polyfills.bundle.js, polyfills.bundle.map (polyfills) 222 kB {4} [initial] [rendered]
chunk    {2} styles.bundle.js, styles.bundle.map (styles) 27 kB {4} [initial] [rendered]
chunk    {3} vendor.bundle.js, vendor.bundle.map (vendor) 5.05 MB [initial] [rendered]
chunk    {4} inline.bundle.js, inline.bundle.map (inline) 0 bytes [entry] [rendered]
webpack: Compiled successfully.

When will the next update be released? I also couldn't find style.ts in the suggested directory. Project does compile successfully but it would be nice to have it resolved asap.

Also looking for a release promptly with this fix. Thanks in advance!

For those who cannot find it in Angular CLI, the correct path is
node_modules/angular-cli/models/webpack-configs/styles.js

You should see this:
// load global css as css files
    if (globalStylePaths.length > 0) {
        rules.push.apply(rules, baseRules.map(function (_a) {
            var test = _a.test, loaders = _a.loaders;
            return ({
                include: globalStylePaths, test: test, loaders: ExtractTextPlugin.extract({
                    loader: [
                        // css-loader doesn't support webpack.LoaderOptionsPlugin properly,
                        // so we need to add options in its query
                        ("css-loader?" + JSON.stringify({ sourceMap: cssSourceMap }))
                    ].concat(commonLoaders, loaders),
                    fallbackLoader: 'style-loader',
                    // publicPath needed as a workaround https://github.com/angular/angular-cli/issues/4035
                    publicPath: ''
                })
            });
        }));
    }

Change the loader to use, and the fallbackLoader to fallback

See http://stackoverflow.com/questions/42076879/fallbackloader-option-has-been-deprecated-replace-with-fallback

@JanStureNielsen this has been fixed in the last release

The last release v1.0.0-beta.30 was released 10 days ago, and the patch was pushed 7 days ago... doesn't seem to be included in the last release.
https://github.com/MRHarrison/angular-cli/commit/f5624ab588a101fc3396aeb34962a8cb4761c7aa

The last release is beta 31 and has the fix. See the readme for the upgrading instructions.


Added libui

"Simple and portable (but not inflexible) GUI library in C that uses the native GUI technologies of each platform it supports."

@waruqi fixed the specified issues issues

@waruqi any updates on getting this merged?

it need gtk+3 on linux, you can install it to test this package in linux ci.
if it works, then we can remove it in ci.

https://github.com/xmake-io/xmake-repo/pull/1772


Torch with require'cunn'

Why I must run one of the  CUDA samples as root (for example the deviceQuery), then I can able to use CUDA based packages in Torch.
But when I  restart the computer, the CUDA will not be initialized, I must do that again to enable CUDA based packages in Torch.
@albanD

@zhunzhon use the google group for this kind of question not related to cutorch please.
The problem is with CUDA, this is broadly discussed on the internet: one part of the CUDA runtime require root to initialize.


https://beta.ionicframework.com/docs/api/config/Config

Bug Report
Ionic Info
Run ionic info from a terminal/cmd prompt and paste the output below.
[WARN] You are not in an Ionic project directory. Project context may be
       missing.

Ionic:

   ionic (Ionic CLI) : 4.0.3

System:

   NodeJS : v10.8.0
   npm    : 6.3.0
   OS     : macOS High Sierra

Describe the Bug
A clear and concise description of what the bug is.
Steps to Reproduce
https://beta.ionicframework.com/docs/api/config/Config
this url show:Unable to fetch api/config/Config:
Related Code
If you are able to illustrate the bug with an example, please provide a sample application via an online code collaborator such as StackBlitz, or GitHub.
Expected Behavior
Expected:Config docs is avaible
Additional Context
List any other information that is relevant to your issue. Stack traces, related issues, suggestions on how to fix, Stack Overflow links, forum links, screenshots, OS if applicable, etc.

Workaround: If you switch the doc to nightly instead of beta-1 it's ok

Perhaps better @ https://github.com/ionic-team/ionic-docs/


Fix broken path in exports.types property

I did mistake with path in this PR. This PR is about to fix it.

Thanks~


Forward port work present in branch wb/proposer-based-timestamps

Around 70% PBTS work was done on branch wb/proposer-based-timestamps (as sort of feature branch). However, that branch:

wasn't merged back on master with a merge commit
was rather "squash-merged" into a big PR: tendermint/tendermint#7605.

This issue forward-ports tendermint/tendermint#7605 on our new feature branch feature/pbts.

[x] #2018

Original PR: tendermint/tendermint#7605, this is a big "merge-squash" PR.

The original merge-squash PR contains the following smaller PRs (which have thus been forward-ported in this PR)

tendermint/tendermint#7170
tendermint/tendermint#7305
tendermint/tendermint#7300
tendermint/tendermint#7290
tendermint/tendermint#7307
tendermint/tendermint#7341
tendermint/tendermint#7373
tendermint/tendermint#7354
tendermint/tendermint#7376
tendermint/tendermint#7382
tendermint/tendermint#7418
tendermint/tendermint#7391
tendermint/tendermint#7413
tendermint/tendermint#7480
tendermint/tendermint#7415
tendermint/tendermint#7554
tendermint/tendermint#7563

This fw-port PR also includes two other PRs not included in tendermint/tendermint#7605 but needed by it

tendermint/spec#393 (protobufs, which were in a separate repo at the time)
tendermint/tendermint#8142

DoD

Forward-port tendermint/tendermint#7605 and all other needed PRs

Exclude unnedded refactoring not related to PBTS

Production code builds
Unit and e2e tests pass

Found out tendermint/tendermint#7541 was in a dangling branch. The changes actually landed in tendermint/tendermint#7711 (so, forward-porting the latter)

Closed by #2018


Missing LICENSE or COPYING

the license is missing, is this MIT or GPLv3 licensed?

:+1:
Yes, please add explicit license statement.

(Y)

:+1:

What license do you prefer?

Thanks.  Either GPL 3 or MIT would be great.
@AminaG, I'm not sure how familiar you are with software licenses and copyrights, but the copyright holder(s) can at anytime choose another license for upcoming versions.  What you cannot do is change the past, that is, as soon as you've released a version under a certain license you cannot the license for that version.  I'm saying this, because some people never get around to pick a license because they think they have to stick to it for the rest of life. The "only" thing you have to worry about who is holding the copyright and if you have contributors that have contributed enough to be considered copyright holders, you of course need to discuss with them.

Thanks for the info.
What is the most important main difference between GPL 3 MIT and Apache?
What do you prefer and why?

GitHub's choosealicense.com is a nice simple website which gives overview about which license to choose, depending on what you want.
If you want an advanced overview of open source license have a look at https://en.wikipedia.org/wiki/Comparison_of_free_and_open-source_software_licenses
I personally prefer GNU GPL v3. :)

Hi AminaG,
Thanks for this great tool!
We are working on an internal tool that needs screen capturing abilities and want to adapt your code for this purpose.
However, our legal department is asking under which license your code is released. Are you able to publish it under one of the standard open source licenses?
Adding such a license to your readme file would be great.

license/copying file is still missing, @doronpt at this point you may want to explore other options that have a GPL3 or LGPL or MIT license or ask your legal department what license they would assume (public domain may be a possibility but may present its own problems)
this is why it's still important to add a LICENSE or COPYING file to each and every project. I too would love to live in a world without software licenses where all code is assumed to be open source, but that's just not the world we live in unfortunately. Copyright laws care about you even if you don't care about them.

I choose MIT.

:+1:


[v5.0.0-beta.2] Old version of React is not happy with Storybook with React 16.8.1

Describe the bug
Storybook is totally crashed and printed out many theme related errors. Even though I removed all add-ons and stories, the problem persisted. The bug can be resolved by upgrade everything to React 16.8.1 or rollback storybook to v5.0.0.beta.1.
To Reproduce
Steps to reproduce the behavior:

Have a project running on React 16.7 or lower
Launch Storybook v5.0.0.beta.2

Expected behavior
No expected crush.
Screenshots
Some error was printed.
Warning: Failed prop type: The prop `viewMode` is marked as required in `<<anonymous>>`, but its value is `undefined`.
Uncaught TypeError: Cannot read property 'secondary' of undefined
Warning: Failed prop type: Invalid prop `notifications[0].id` of type `string` supplied to `NotificationList`, expected `number`.
Uncaught TypeError: Cannot read property 'size' of undefined
Uncaught TypeError: Cannot read property 'positive' of undefined
Uncaught TypeError: Cannot read property 'dark' of undefined
Uncaught TypeError: Cannot read property 'size' of undefined
Uncaught TypeError: Cannot read property 'preview' of undefined
The pseudo class ":first-child" is potentially unsafe when doing server-side rendering. Try changing it to "first-of-type"
Uncaught TypeError: Cannot read property 'secondary' of undefined
Uncaught TypeError: Cannot read property 'secondary' of undefined
Uncaught TypeError: Cannot read property 'size' of undefined
The above error occurred in the <Context.Consumer> component:
Uncaught TypeError: Cannot read property 'secondary' of undefined
The above error occurred in the <LocationProvider> component:
Uncaught TypeError: Cannot read property 'secondary' of undefined

System:

OS: MacOS Mojave
Device: MacBook Air 2013-mid
Browser: Chrome 72
Framework: React 16.3, React 16.7
Version: v5.0.0-beta.2

Additional context
Still have errors even though I upgraded my React to the latest. Here is the error stacks
vendors~main.cbfe59591d8b3f61e698.bundle.js:74002 Warning: Failed prop type: The prop `viewMode` is marked as required in `<<anonymous>>`, but its value is `undefined`.
    in Unknown (created by Manager)
    in Manager (created by Context.Consumer)
    in Location (created by QueryLocation)
    in QueryLocation (created by Root)
    in LocationProvider (created by Root)
    in HelmetProvider (created by Root)
    in Root
printWarning @ vendors~main.cbfe59591d8b3f61e698.bundle.js:74002
vendors~main.cbfe59591d8b3f61e698.bundle.js:74002 Warning: Failed prop type: Invalid prop `notifications[0].id` of type `string` supplied to `NotificationList`, expected `number`.
    in NotificationList (created by Sidebar)
    in Sidebar (created by Context.Consumer)
    in _default (created by Layout)
    in div (created by Context.Consumer)
    in Styled(div) (created by Nav)
    in Nav (created by Layout)
    in Layout (created by Context.Consumer)
    in WithTheme(Layout) (created by ResizeDetector)
    in ResizeDetector
    in div (created by Context.Consumer)
    in Styled(div)
    in Unknown
    in Unknown (created by ResizeDetector)
    in ResizeDetector
    in Unknown (created by Manager)
    in ThemeProvider (created by Manager)
    in Manager (created by Context.Consumer)
    in Location (created by QueryLocation)
    in QueryLocation (created by Root)
    in LocationProvider (created by Root)
    in HelmetProvider (created by Root)
    in Root
printWarning @ vendors~main.cbfe59591d8b3f61e698.bundle.js:74002
vendors~main.cbfe59591d8b3f61e698.bundle.js:82914 Warning: Unknown event handler property `onVisibilityChange`. It will be ignored.
    in div (created by Context.Consumer)
    in Styled(div) (created by InnerReference)
    in InnerReference (created by Context.Consumer)
    in Reference (created by TooltipTrigger)
    in Manager (created by TooltipTrigger)
    in TooltipTrigger (created by WithTooltip)
    in WithTooltip
    in div (created by Context.Consumer)
    in Styled(div)
    in Unknown (created by withState(Component))
    in withState(Component) (created by Context.Consumer)
    in Styled(withState(Component)) (created by Sidebar)
    in nav (created by Context.Consumer)
    in Styled(nav) (created by Sidebar)
    in Sidebar (created by Context.Consumer)
    in _default (created by Layout)
    in div (created by Context.Consumer)
    in Styled(div) (created by Nav)
    in Nav (created by Layout)
    in Layout (created by Context.Consumer)
    in WithTheme(Layout) (created by ResizeDetector)
    in ResizeDetector
    in div (created by Context.Consumer)
    in Styled(div)
    in Unknown
    in Unknown (created by ResizeDetector)
    in ResizeDetector
    in Unknown (created by Manager)
    in ThemeProvider (created by Manager)
    in Manager (created by Context.Consumer)
    in Location (created by QueryLocation)
    in QueryLocation (created by Root)
    in LocationProvider (created by Root)
    in HelmetProvider (created by Root)
    in Root
warningWithoutStack @ vendors~main.cbfe59591d8b3f61e698.bundle.js:82914
2vendors~main.cbfe59591d8b3f61e698.bundle.js:444 The pseudo class ":first-child" is potentially unsafe when doing server-side rendering. Try changing it to "first-of-type"
(anonymous) @ vendors~main.cbfe59591d8b3f61e698.bundle.js:444
client.js:92

FYI, I have reproduced this error on a boilerplate CRA1 project that I manually upgraded. It does not repro on a CRA2 project.

@domyen @ndelangen The theming is still half-baked. The themes define colorSecondary but the components still reference color.secondary. I'm assuming you guys have a plan for this--please let me know when things are consistent and I'll release a fix.

I'll take a look at the theming and write some docs tomorrow!

Still have errors even though I upgraded my React to the latest. Here is the error stacks

I can confirm these errors still appear with React 16.8.1 when using @storybook/polymer

Okay, things get worse.  If i upgrade to React 16.8.1, the same problem comes up again.  I guess the React user have to use the same version of storybook in V5 before this bug being fixed...

@leoyli et al can you check if two versions of @storybook/theming are installed?

Guessing this wasn't the case?! I only have these dependencies in my package.json:
    "@storybook/addon-a11y": "5.0.0-rc.1",
    "@storybook/addon-actions": "5.0.0-rc.1",
    "@storybook/addon-backgrounds": "5.0.0-rc.1",
    "@storybook/addon-info": "5.0.0-rc.1",
    "@storybook/addon-jest": "5.0.0-rc.1",
    "@storybook/addon-knobs": "5.0.0-rc.1",
    "@storybook/addon-options": "5.0.0-rc.1",
    "@storybook/addons": "5.0.0-rc.1",
    "@storybook/components": "5.0.0-rc.1",
    "@storybook/react": "5.0.0-rc.1",

I mean inside node modules?
Tom
On 17 Feb 2019, 2:15 PM +1100, Leo Y. Li<EMAIL_ADDRESS>wrote:

Guessing this wasn't the case?! I only have these dependencies in my package.json:
"@storybook/addon-a11y": "5.0.0-rc.1",
"@storybook/addon-actions": "5.0.0-rc.1",
"@storybook/addon-backgrounds": "5.0.0-rc.1",
"@storybook/addon-info": "5.0.0-rc.1",
"@storybook/addon-jest": "5.0.0-rc.1",
"@storybook/addon-knobs": "5.0.0-rc.1",
"@storybook/addon-options": "5.0.0-rc.1",
"@storybook/addons": "5.0.0-rc.1",
"@storybook/components": "5.0.0-rc.1",
"@storybook/react": "5.0.0-rc.1",
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@tmeasday Oh man! you are right!  I completely remove and reclone my entire project then install it, it then delights me again.

So I think the problem was what you mentioned in #5445 .

Yeah, looks it’s not great but I am not sure what else we can do about it :/
Tom
On 17 Feb 2019, 2:30 PM +1100, Leo Y. Li<EMAIL_ADDRESS>wrote:

So I think the problem was what you mentioned in #5445 .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Shall we close this? I'm not sure there is much we can do, beyond maybe #5445 or reporting a bug upstream to yarn/npm

Yes, I think then this issue should be closed...! Thank you!


Build fix & cayleygraph package path

The go mod tidy reported:
go: rye/evaldo imports
        github.com/cayleygraph/cayley/query/path: module github.com/cayleygraph/cayley@latest found (v0.7.7), but does not contain package github.com/cayleygraph/cayley/query/path

because at 0.7.7 release cotinains no path package in https://github.com/cayleygraph/cayley/tree/v0.7.7/query but it is in
https://github.com/cayleygraph/cayley/tree/v0.7.7/graph/path
It was moved few hours after 0.7.7 release in https://github.com/cayleygraph/cayley/commit/218e2bc199a8c9c1012383efd29f26a367804dcf

Thank you!


Allow choice for static vs dynamic qubit allocation

Current API only generates QIR with static qubit allocation. The API needs to make this configurable.

Fixed in #103


Some author names are associated with identifiers for their works.

There are a number of Concepts that have the wrong title (and probably ontologyType).  I believe that this is due to the use of an id corresponding to something by a given author, but just using the author's name for the title.
For example: http://id.loc.gov/authorities/names/n91074218 - which is "Freud, Sigmund, 1856-1939. Totem und Tabu. English", being used with the title "Freud, Sigmund, 1856-1939."
The tricky question is which of the two options the author meant.  Is the title correct and the identifier wrong, or the identifier correct and the title wrong?
For this reason, this problem can only be fixed at source (for existing data).  However, it would be ideal if we could prevent this problem occurring in the future.
Here is an aggregation that shows the extent of the problem.
GET catalogue-concepts/_search
{
  "size": 0,
  "query": {
    "bool": {"must":[{
    "term": {
      "authority": "lc-names"
    }},
        {"wildcard": {
      "identifier": "n*"
    }}]}
  },
  "aggs": {
    "common-labels": {
      "terms": {
        "size": 1000,
        "field": "label"
      }
    }
  }
}

https://wellcome.slack.com/archives/C02ANCYL90E/p1674039262532379

One aspect of this is that all of these instances are assigned the Person ontologyType, which means that however this gets fixed at source (by adding the extra details, or using the Person's actual id) those Person records in the id minter db will become redundant.
If the id minter were changed to ignore types, then those records would be able to be used for the corrected records.

This repetition doesn't look like the kind of error a human would make.  I wonder if this is imported data and whatever is generating it suffers from a copy-paste error, e.g.
if 'id':
  copy_stuff_from('0')
if 'title':
  copy_stuff_from('0') // whoops, should say   copy_stuff_from('t')

The same is true on the MARC text display:
100 1  Freud, Sigmund,|d1856-1939|0n  91074218 |0n  91074218

Notes on the process for finding these awry records.

Select a label from one of the buckets found by the aggregation in the description above
put it in this query to find which canonicalids it corresponds to.  The label query is a wildcard so I can stick a * on the end to also find some correctly labelled records

GET catalogue-concepts/_search
{
  "size": 100,
  "query": {
 "bool": {"must":[   { "wildcard": {
      "identifier": "n*"
    }},{ "wildcard": {
      "authority": "lc-names"
    }},{ "wildcard": {
      "label": "Linné, Carl von, 1707-1778"
    }}
    ]
  }
  }
}

From the list that that creates, hop on over to the catalogue pipeline DB and do a simple search for the id GET works-indexed-2023-01-19/_search?q=pnwmcx5x
From there, I can look for ids for the resulting Work, or the original source record, and look them up in the appropriate places: https://wellcomecollection.org/works/zyefuv3m or https://catalogue.wellcomelibrary.org/record=b1675525x

In the case of Linneus, in the example above, wildcarding his name to include entries with $t, shows that such concepts do exist, e.g.
{
        "_index": "catalogue-concepts",
        "_id": "lc-names:no98081877",
        "_score": 3,
        "_source": {
          "authority": "lc-names",
          "identifier": "no98081877",
          "label": "Linné, Carl von, 1707-1778. Systema naturae",
          "canonicalId": "hpy7qck3",
          "ontologyType": "Agent"
        }
      },```

This looks like an issue completely outside of our control.  It appears to exist in imported data from elsewhere.
Eventually, when the Concepts Embedder is up and running, the correct labels (from the identifiers) will be used in Works at the end of the catalogue pipeline, and composite concepts can probably be unpicked.


Rename Camlistore to Perkeep

It's time to finally rename Camlistore.
Camlistore was always supposed to be a temporary name, and it survived for way too long (over 7 years so far!).
Problems with the name Camlistore:

It's a nerdy acronym for Content-Addressable Multi-Layer Indexed Storage, which is nerdy. Nerdy was fine when the target was nerds, but as it gets increasing usable & user-friendly, I'd like a more approachable name.
I pronounce it Cam-lee-store, but most people say Camel-store.
People think it's written in OCaml. It's not.
It's three syllables.
Back before LetsEncrypt, companies wouldn't give us cheap/free certs because Camlistore contains the substring "store", so therefore it's used for commerce and we need an enhanced certificate blah blah. (No longer relevant, but amusing.)

After much thought and research, we've settled on the name Perkeep.
Perkeep lets you permanently keep things in your permanent keep.
I like that keep works as both a,

verb -- "have or retain possession of"
noun -- "the strongest or central tower of a castle, acting as a final refuge"

And "per" is a prefix for "through", which makes me think of data moving around, importers, exporters, etc. It's also short for "permanent".
I bought the relevant domain names and can't find existing users.
Things to update:

domain name, redirects
website
mascot (I'm thinking a derpy parakeet, "Keepy, the Perkeep Parakeet" maybe). I think @ashleymcnamara might still be interested in designing it for us.
docs
code structure, daemon/CLI names
github repo (second to last)
gerrit hostname (lowest priority, probably last)

Camlistore references will persist for quite some time, so we might as well partially embrace it with stuff like. "Perkeep, née Camlistore".
For daemon names, I'm thinking e.g.

server/perkeepd

For CLI tools:

cmd/pkput (camput)
cmd/pkget (camget)
cmd/pk (camtool)

cmd/pk could even get "get" and "put" subcommands that call the pkput/pkget binaries/code.
What else am I forgetting?
e.g. the name "perkeep" doesn't mean something offensive in some language I don't know, does it? Google Translate suggests we're safe there.
/cc @mpl @lindner @willnorris @tgulacsi @bslatkin

Well, crap, I never checked whether the Twitter account was free, and it's not: https://twitter.com/perkeep
Hasn't been used since 2014. I wonder if it's possible to get. Perhaps the owner will rename to something else.
Otherwise we need a new Twitter handle.

Small note: I'm not sure how it works in english, but if it's supposed to mimick the french word, "née" is a past participle that follows the subject, so it's "né" if the subject is male, and only "née" if it's female.

Whom do we know at Twitter? :)

"née" is primarily seen in english to express a woman's maiden name. So it is frequently (mis)used in this way.
I also have no problems with a piece of software being grammatically feminine. ;-)

@buro9, I filed a Twitter support request. @nickoneill told me:

Twitter says if a user hasn't logged in for a year, you can file a support request and get unused twitter names.

@bradfitz how about PerKeepr -- Permanent Keeper? The Twitter Handle is avaiable btw https://twitter.com/perkeepr

I don't want to do the Flickr -r thing. Also, I like "keep" as-is for being a real word. Sounds more professional. And "Perkeepr" makes me think of Trapper Keeper from my childhood.

Gotcha gotcha. Kuddos for the longevity and on Camlistore's graduation to PerKeep.

Among the other places to update:
https://opensource.google.com/projects/camlistore
Probably trivial for @bradfitz or @willnorris to do.

@bradfitz Keep is definitely a good name. The storage layer in Arvados is called Keep, and it is has some similar design elements to Camlistore.

Camlistore often shorted to Camli. Perkeep shorted is Perk. Familiarity and brevity breeds nicknames perkup, perkdown, perkpush, perkpull. All good. I was thinking about any related negative short names, only one is "perkwalk" (my bad). Came because many male celebrities currently getting in trouble.

@mpl, I've updated the nameserver records for perkeep.org to AWS and set the DNS A records to the same IP as camlistore.org. Should be in live everywhere in 12-24 hours. Want to make sure camweb does the right thing upon getting those Host/SNI values? (both "perkeep.org" and "www.perkeep.org" doing the redirect thing)
I think {www.,}camlistore.org should also redirect to https://perkeep.org/
Make sure you also update the acme autocert allowed hostnames.

ACK

https://camlistore-review.googlesource.com/c/camlistore/+/11986

Moved Camlistore to Perkeep on Wikipedia https://en.wikipedia.org/wiki/Perkeep added inline citations and updated talk page asking for removal of missing citations tag by reviewers.

Probably very low priority, but there's the app on Google Play - https://play.google.com/store/apps/details?id=org.camlistore - that needs a name change. It would be nice to update too (or temporarily remove?) as it's quite an old version and I think there have been significant bug fixes since it was last published.

https://camlistore-review.googlesource.com/#/c/camlistore/+/12046

@broomhall I can't update the official app as i don't have the key to sign it, but jsyk whenever  someone asks, I try to update the one at https://storage.googleapis.com/camlistore-release/android/app-debug.apk

https://camlistore-review.googlesource.com/c/camlistore/+/12066

https://camlistore-review.googlesource.com/#/c/camlistore/+/12106

Another one to the rename/migrate list: https://travis-ci.org/camlistore/camlistore

I'd suggest permastore.

@chriscroome, this thread isn't about voting on a name. The name has been chosen and required research for conflicts.

Hi all!
I'm not sure if anyone still uses this, but I'm happy to rename to whatever
https://github.com/dotMR/CamliClipper
Honestly haven't tested in awhile, assuming it still works OK

@bradfitz, in Seattle @rachelnabors does wonderful animations. Maybe a designer for Perkeep Parakeet (she even produces svg). https://github.com/rachelnabors

https://camlistore-review.googlesource.com/#/c/camlistore/+/12186

I love Perky the Perkeep Parakeet :3
On Fri, Dec 15, 2017 at 12:05 PM mpl<EMAIL_ADDRESS>wrote:

https://camlistore-review.googlesource.com/#/c/camlistore/+/12186
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/camlistore/camlistore/issues/981#issuecomment-352099268,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAObEu7RszsWECv-cF4L5322MBY_HiP1ks5tAtEIgaJpZM4Q270-
.

Twitter account is now https://twitter.com/PerkeepOrg (renamed, so it kept its followers)
And I created https://twitter.com/Camlistore with a note about the rename.

Someone with ops on #camlistore on freenode should create a #perkeep channel and add a topic to the old channel redirecting to the new one.  (optionally adding a bot to inform people of the new home)

I see you've created an empty perkeep org here on github, but FWIW, you can just rename the existing org (and repo) and everything will continue to just work: github even redirects command-line git pulls etc that still refer to the old location.

@purcell, actually somebody else created that for us and added me as an owner.

@bradfitz Cool. I think in that case you can just transfer the repo across to the other org, and the redirects will still work.

Isn't it easier to just treat that as a placeholder and rename this one when the time comes?

Isn't it easier to just treat that as a placeholder and rename this one when the time comes?

That's what I'd do, personally, but just saying that either would work.

https://camlistore-review.googlesource.com/c/camlistore/+/14606

https://camlistore-review.googlesource.com/#/c/camlistore/+/15518

https://camlistore-review.googlesource.com/#/c/camlistore/+/16746

https://perkeep-review.googlesource.com/c/perkeep/+/17466

Small update for the OSX client :) https://perkeep-review.googlesource.com/c/perkeep/+/18426

Should this issue be closed?

Rename looks set to me https://en.wikipedia.org/wiki/Perkeep

Looks like there are still quite a few references that can be cleaned up: https://github.com/perkeep/perkeep/search?q=camlistore
We'll never get it completely to zero, since some would be breaking changes, but I think it's worth keeping this issue open to track cleaning up these remaining references.


Creating a draggable component without header

Is it possible to create a draggable component without header?. It would also be preferable to remove the border resize option, is this feasible  with Golden-layout?
var layout = new GoldenLayout({
    settings:{
        hasHeaders: false,
    },
    dimensions: {
        borderWidth: 0,
    },
    content: [  ]
});

If the component doesn't have a header, how do you propose to drag it?

https://strml.github.io/react-grid-layout/examples/0-showcase.html

I'm also looking for some option to make component without header and have draggable opt in/out for it.

#251


EOL QuickSwap Gamma vaults which will not harvest

quick-gamma-usdc-dai-narrow
quick-gamma-usdc-weth-wide
quick-gamma-wbtc-weth-wide
quick-gamma-weth-usdt
quick-gamma-wmatic-quick
quick-gamma-wmatic-usdc-wide
quick-gamma-wmatic-weth-wide

Rewards:
pancake-ichi-usdt-wbnb
pancake-ichi-usdt-btcb
pancake-ichi-cake-usdt
sushi-base-gamma-weth-usdbc-narrow
sushi-gamma-weth-usdc-narrow
uniswap-gamma-btc.b-weth
sushi-gamma-magic-weth-narrow
aura-r-dai
moonwell-base-weth

Revive:
netswap-weth-m.usdc
stargate-metis-usdt

Revive: netswap-weth-m.usdc stargate-metis-usdt

also EOL
uniswap-gamma-gdai-dai
uniswap-gamma-gns-usdc-narrow
uniswap-gamma-gns-usdc-wide
uniswap-gamma-gns-weth-wide


Add observer book

It works but I don't know why there are ten pages, if somebody knows please feel free to make a pull request!

I don't see a reason to have a module for it, just give them a book in the observer nodule by using this exisdting utility https://github.com/twizmwazin/CardinalPGM/blob/master/src/main/java/in/twizmwaz/cardinal/util/ItemUtils.java#L28
Also there 10 pages because you created ten pages....

 Getting my statements mixed up!

I don't see why you need to create the other 10 pages for localization., just make one and use JDOM to iterate through all of the pages so it can be updated dynamically

@CaptainElliott It's what @twizmwazin wanted, unless if he has changed his mind!

@dentmaged I like Elliott's way

@twizmwazin Agreed, this method is too hard-coded.

@iEli2tyree011 @twizmwazin Okay! I'll redo it.

@dentmaged  This is the reason why you have 10 pages.
BOOK_PAGE_1("book.page1"), BOOK_PAGE_2("book.page2"), BOOK_PAGE_3("book.page3"), BOOK_PAGE_4("book.page4"), BOOK_PAGE_5("book.page5"), BOOK_PAGE_6("book.page6"),  BOOK_PAGE_7("book.page7"), BOOK_PAGE_8("book.page8"), BOOK_PAGE_9("book.page9"), BOOK_PAGE_10("book.page10"),
What you need to do is remove one per line until you let only BOOK_PAGE_1("book.page1")

@AngelCasillas01 If you read the code, you would see that it checks the page is null or "", but that doesn't work.

I think I know a better way to do this! Thanks anyway for all your help!


Dynamic Table Columns

Hey there,
I'm using this gem to generate docx files where I have a table that is generated based on a dataset. I can generate rows dynamically with no problem.
I'm using loops and conditionals using merge fields inside the table on the template.
But I also need to generate columns based on the dataset. From what I understand we don't actually have columns but cells inside rows. I tried to apply a conditional merge field just like I did for rows, but it did not work.
What's the best approach to accomplish this?
Thanks a lot!

@jppribeiro I'm not sure home complex your formatting is but I would probably use HTML insertion instead. Basically you'd need a "pre-processing" step where you generate the desired table in HTML and then use that in your context instead of trying to do it with loops.
Off the top of my head I don't know if it's possible to generate extra columns in a for loop since I don't remember the XML format for tables. Even if the XML is agreeable it might be hard to get your merge fields in the right places.

@stadelmanma That's exactly what I decided to do. I'm building the table structure with Nokogiri, converting it into html and using that in the context.
Thanks for answering so fast!

Glad you found a working solution. HTML insertion isn't perfect but the code I've written that uses sablon takes advantage of it extensively (actually, excessively).


Timer HVAC modes

Hi,
I currently have the option of turning my stove on and off using HVAC modes.
But when the timer is switched on, understandably nothing happens.
It would be great if the timer switch was available for the HVAC modes.


How could I get '../data/kitti/kitti_dbinfos_val.pkl'?

When I run GLENet Prediction, There is the error : No such file or directory: '../data/kitti/kitti_dbinfos_val.pkl'
How could I solve this problem?

Do you follow the instructions in the "Generate Label Uncertainty with GLEnet" part of README.md.
Please rename the provided files:
kitti_infos_train_wconf_v5.pkl -> kitti_infos_train.pkl
kitti_dbinfos_train_wconf_v5.pkl -> kitti_dbinfos_train.pkl

Do you follow the instructions in the "Generate Label Uncertainty with GLEnet" part of README.md?
Please rename the provided files: kitti_infos_train_wconf_v5.pkl -> kitti_infos_train.pkl kitti_dbinfos_train_wconf_v5.pkl -> kitti_dbinfos_train.pkl

Yes,I have followed the instructions,and I got :
kitti_infos_test.pkl
kitti_dbinfos_train.pkl
kitti_infos_train.pkl
kitti_infos_val.pkl
kitti_infos_trainval.pkl
But there isn't "kitti_dbinfos_val.pkl".
Is it necessary?
How could I generate this file?

Yes. If you want to test the performance of GLENet, you should generate the file.
But if you just want to generate label uncertainty for downstream detectors, you should follow the instructions in README.md and do not need that file.


Use the FullCalendar widget instead of the embedded Google widget

See also:

https://github.com/dlang/dlang.org/pull/2222#issuecomment-367156497
https://fullcalendar.io/

FullCalendar allows a lot of customization and also comes with a few themes:
https://fullcalendar.io/js/fullcalendar-3.8.2/demos/themes.html

CC @CyberShadow - are you okay with this too?
The only downside here is that a Google calendar API key needs to be exposed, but I restricted the domain to localhost, dlang.org and dlang.io which should make abuse harder. And if this ever becomes a problem we can easily revert back to the old widget.

For some reason some of the CDNs used on this page and dlang.org are timing out for me. Is it just me?

When it does load, the calendar I see is completely blank. I see some events currently on https://dlang.org/calendar , but here there's nothing.

Is it just me?

I think so. Does https://fullcalendar.io/js/fullcalendar-3.8.2/demos/themes.html work for you?

Yep, that works fine.

Interesting it works fine in my Chrome and Firefox:

What error do you get in your Firefox?

No error, it's just blank.


minor: update cranelift related dependencies

Which issue does this PR close?

Closes #2760 #2761 #2762 #2763
-- Clean the redundant tickets :(

Thank you @xudong963 !


Rack dependency loosened via upping the rack version

https://github.com/dtaniwaki/rack-dev-mark/pull/66
I merged a PR with the same change 🙇


running ycsb

while i m giving command ( mvn clean package ) i received the following error
T E S T S
Running com.yahoo.ycsb.db.accumulo.AccumuloTest
2018/09/11 13:31:19 WARN  org.apache.accumulo.core.client.impl.ServerClient  - There are no tablet servers: check that zookeeper and accumulo are running.
any knows how to solve that ?

What's your platform and what's your Maven version?
If you aren't customizing YCSB internals or adding a new data store binding then you shouldn't need to build things yourself. Download one of our convenience binary tarballs from the releases page instead.
https://github.com/brianfrankcooper/YCSB/releases/latest
i.e. the general purpose tarball for 0.15.0:
https://github.com/brianfrankcooper/YCSB/releases/download/0.15.0/ycsb-0.15.0.tar.gz

What is your OS @AhmadZia1991 ? If you're using Windows and wants to build from source it's not a good idea, I suggest switching to Ubuntu like I did.

Sir i am also using ubuntu i tried to run it without mvn package but still
i receiced the sam msg error of slf4g and i am not able to receive result.
On Sat 15 Sep, 2018, 11:10 AM Miko Chu<EMAIL_ADDRESS>wrote:

What is your OS @AhmadZia1991 https://github.com/AhmadZia1991 ? If
you're using Windows and wants to build from source it's not a good idea, I
suggest switching to Ubuntu like I did.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/brianfrankcooper/YCSB/issues/1229#issuecomment-421533076,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ak2GftmBWtbSll6Mc1yJfLPbTsdD6Qv9ks5ubJK-gaJpZM4Wjwr1
.

For the moment, here's the workaround:
mvn clean install -DskipTests


Update python-engineio

Pull Requests:

https://github.com/Senzing/docker-senzing-console/pull/29


feat: Implement CAST between struct types

Which issue does this PR close?

Closes https://github.com/apache/datafusion-comet/issues/815
Rationale for this change

We need support for casting between struct types to support reading structs from Parquet using DataFusion's ParquetExec.
What changes are included in this PR?

How are these changes tested?

@parthchandra @mbutrovich could you review?
We need more extensive tests for sure, but it will be easier to add those as part of the comet-parquet-exec feature branch.

We need more extensive tests for sure, but it will be easier to add those as part of the comet-parquet-exec feature branch.

Spark should have many Cast expression tests. As we pass Spark tests, it should be fine for general cases.


Excessive logging in Neo4jSession via ExecuteQueriesDelegate#validateQuery

After the update to Neo4j OGM 3.2.15 I discovered many log entries with level warn and the message "Cypher query contains keywords that indicate a writing query but OGM is going to use a read only transaction as requested, so the query might fail.".
In 3.2.15 the keyword call has been added to the list of WRITE_CYPHER_KEYWORDS and I use stored procedures to read data from the database (in read-only transaction).
Expected Behavior

Maybe there is a better way to handle this use case. In your tests (class QueryCapabilityTest) there is a comment that Bolt Driver 4.1 will fail with an exception if a write occurs in a read-only transaction. Maybe the check can be omitted if this driver is in use since it will fail with an exception anyway?
Another solution could be the change of the log level for this message to something like info or debug.
It feels wrong to set the log level for Neo4jSession to the level above warn.
Current Behavior

For each call of a stored-procedure in a read-only transaction a log message is created and the log file will grow rapidly.
Context

Without changing the log configuration the log file grows rapidly and I think that this is also a performance problem because a lot of log messages are created and written to disk.
Your Environment

Java version:
Neo4j version (The version of the database): 3.5.21
OGM Version (The version of this software): 3.2.15
OGM Transport used (One of Bolt, HTTP or embedded): Bolt
Neo4j Java Driver version (in case of Bolt transport): 4.1.1
Link to your project: ./.

Steps to Reproduce

Call a stored procedure in a read-only transaction

Stored procedures are kinda hard to treat correctly in that mapping scenario. I would recommend calling them through the bolt driver directly.
I agree with you on all aspects of the log level.
I would not introduce another check depending on which transport is used.

Yikes… I just realized that the warning messages goes through the session itself which just get's hold of the thread id. Good catch, @torstenkuhnhenne

A direct call is not so easy because it's SDN repository with a @Query annotation.
Yes, it's hard to treat this correct. I tought of reading the stored-procedure mode from neo4j (from the @Procedure annotation) but then you must parse the cypher query and extract the name of the stored procedure and nobody wants to do this ;-)


[BACKPORT][v1.3.x][IMPROVEMENT] Schedule volumes based on available disk space

https://github.com/longhorn/longhorn/issues/4185

Verified on v1.3.x-head (longhorn-manager a4b5f37) by the following steps:
(1) Create longhorn cluster with default disk on each node.
(2) Pick a random node A, attach an extra storage which has larger size than the default disk.
(3) Add this extra storage as an additional disk to the node A.
(4) Create and attach a longhorn volume, for the node A, the volume replica is scheduled to the additional disk instead of the default disk.
and by running the automated test case.


taxonomy diff/data generate: support taxonomy-base=empty

This allows users to specify that they want all taxonomy files in their repo to be included, where 'null' symbolically represents the empty tree before this repo was created.
See also instructlab/sdg#242
WIP because we need to update requirements.txt to require newer instructlab-sdg when we release a version with support for this newer semantic.
Checklist:

[x] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[x] Changelog updated with breaking and/or notable changes for the next minor release.
[x] Documentation has been updated, if necessary.
[x] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

LGTM

@Mergifyio backport release-v0.18


CacheMissException if __typename not in query

Describe the issue
Throws CacheMissException if __typename is not provided in the query:

Round trip cache re-read failed: cache.readQuery(request) returned null

To Reproduce
Steps to reproduce the behavior:
query changes($id: ID!) {
    task(id: $id) {
        changes {
            id
            name
        }
    }
}

working query:
query changes($id: ID!) {
    task(id: $id) {
        __typename
        changes {
            __typename
            id
            name
        }
    }
}

Expected behavior
Either no need to add __typename to the query or a more concise error message.
device / execution context
localhost, iOS and android simulator

@ninodinatale will need a few more details here –

are you using the gql helper?
do you know if the result.data from the server was null? Can you set a breakpoint in _resolveQueryOnNetwork to find out?

@micimize sure:

I am using Artemis for generating typed queries which uses gql internally, so yes, I guess?
The result.data was not null.

I was referring to the gql helper provided by  our libraries – Artemis doesn't add __typename automatically. Of course this is still an issue – the cache round-trip should work regardless as long as both write and read use the same AST. Will try and take a look soon

@ninodinatale I still need a sample of the data causing the failure to be able to tell what's going wrong. What code path and preconditions are present would also help – i.e. client.query, using the Query widget, and what FetchPolicy you're using

@ninodinatale I still need a sample of the data causing the failure to be able to tell what's going wrong. What code path and preconditions are present would also help – i.e. client.query, using the Query widget, and what FetchPolicy you're using

@ninodinatale after #795, this is one of the only issues I consider blocking the release of a stable v4, but I really need more info to figure out what the problem is.

@ninodinatale after #795, this is one of the only issues I consider blocking the release of a stable v4, but I really need more info to figure out what the problem is.

closing due to lack of response


About Training

Dear Authors,
Now I am running your code. I find the training process is very slow. Each epoch takes around 6 hours. The used GPU is 1080ti. The GPU memory of your code is around 2GB. Could you give me some advice? Thank you very much.

Is it possible that your features are stored in HDD? It is recommended to store it in SSD. I too faced a similar problem, and it is noted in the readme file.
The gpu part seems correct (assuming you are using GT5 setting).

I am closing this issue, feel free to re-open or create a new issue in case of any queries.


Allocation failed - process out of memory

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[ ] 2.8
[x] 3.0
[ ] self-compiled devel branch

On this operating system:

[x] DCOS on

[ ] AWS
[ ] Azure
[ ] own infrastructure

[x] Linux

[ ] Debian .deb
[x] Ubuntu .deb
[ ] SUSE   .rpm
[ ] RedHat .rpm
[ ] Fedora .rpm
[ ] Gentoo
[ ] docker - official docker library
[ ] other:

[ ] Windows, version:
[ ] MacOS, version:

this is an AQL-related issue:
[x] I'm using graph features
I'm issuing AQL via:

[x] web interface with this browser:  HTTP API
[ ] arangosh
[ ] this Driver:

When I run the traversal HTTP,
Arangod automatically killed with below message 'out of memory'.
But my machine has a fluent available memory.
2016-07-29T05:29:55Z [25683] INFO using endpoint 'http+tcp://<IP_ADDRESS>:8529' for non-encrypted requests
2016-07-29T05:29:55Z [25683] INFO Authentication is turned off
2016-07-29T05:29:57Z [25683] INFO ArangoDB (version 3.0.3 [linux]) is ready for business. Have fun!

<--- Last few GCs --->

  180014 ms: Mark-sweep 1381.2 (1435.0) -> 1381.2 (1435.0) MB, 1196.7 / 0 ms (+ 0.5 ms in 1 steps since start of marking, biggest step 0.5 ms) [allocation failure] [GC in old space requested].
  181096 ms: Mark-sweep 1381.2 (1435.0) -> 1381.2 (1435.0) MB, 1081.2 / 0 ms [allocation failure] [GC in old space requested].
  182143 ms: Mark-sweep 1381.2 (1435.0) -> 1381.2 (1435.0) MB, 1047.7 / 0 ms [last resort gc].
  183132 ms: Mark-sweep 1381.2 (1435.0) -> 1381.2 (1435.0) MB, 988.7 / 0 ms [last resort gc].

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x3e534343fcd9 <JS Object>
    1: clone [/usr/share/arangodb3/js/common/modules/@arangodb/graph/traversal.js:~70] [pc=0x38f99dfb4839] (this=0x392255c3f2b9 <JS Global Object>,obj=0x14ce368c3cd9 <String[8]: 12295287>)
    2: /* anonymous */ [/usr/share/arangodb3/js/common/modules/@arangodb/graph/traversal.js:85] [pc=0x38f99dffbbda] (this=0x392255c3f2b9 <JS Global Object>,k=0x392255c40869 <String[4]: _rev>)
    3: arguments...

#
# Fatal error in CALL_AND_RETRY_LAST
# Allocation failed - process out of memory
#

2016-07-29T05:27:23Z [25440] INFO using endpoint 'http+tcp://<IP_ADDRESS>:8529' for non-encrypted requests
2016-07-29T05:27:23Z [25440] INFO Authentication is turned off
2016-07-29T05:27:25Z [25440] INFO ArangoDB (version 3.0.3 [linux]) is ready for business. Have fun!

<--- Last few GCs --->

   49507 ms: Mark-sweep 1386.1 (1435.0) -> 1385.9 (1435.0) MB, 1157.5 / 0 ms [allocation failure] [GC in old space requested].
   50658 ms: Mark-sweep 1385.9 (1435.0) -> 1385.7 (1435.0) MB, 1151.6 / 0 ms [allocation failure] [GC in old space requested].
   51792 ms: Mark-sweep 1385.7 (1435.0) -> 1385.7 (1435.0) MB, 1133.9 / 0 ms [last resort gc].
   52899 ms: Mark-sweep 1385.7 (1435.0) -> 1385.7 (1435.0) MB, 1106.4 / 0 ms [last resort gc].

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x2c8b4733fcd9 <JS Object>
    1: /* anonymous */ [/usr/share/arangodb3/js/common/modules/@arangodb/graph/traversal.js:948] [pc=0x2e3a814fc87e] (this=0x2afa9642f519 <JS Global Object>,item=0x11a143658901 <an Object with map 0x194d0a7d07c9>)
    2: arguments adaptor frame: 3->1
    3: InnerArrayForEach(aka InnerArrayForEach) [native array.js:~946] [pc=0x2e3a8190687f] (this=0x2c8b47304139 <undefined>,br=0x1a74c4aba4e9 <JS ...

#
# Fatal error in CALL_AND_RETRY_LAST
# Allocation failed - process out of memory
#

Hi,
the problem is, that whatever you're currently doing you're using V8;
V8 has internal resource allocation mechanisms that sometimes fail long before the systems available memory is exhausted.
The last time we saw this was a user errnously feeding a collection into an AQL function like this:
  db._query("FOR x IN somecollection FILTER COUNT(somecollection) > 5 RETURN x")

This will for each iteration of a document of somecollection map the complete collection into V8 javascript for the function call to COUNT() - and if somecollection has some 100k documents it will abort with similar errors.
Next versions of ArangoDB may prohibit putting collections into function arguments to circumvent these sorts of errors.

@dothebart: COUNT(somecollection) is actually special and optimized away, even with {optimizer: {rules: ["-all"]} }. It shows as COLLECTION_COUNT(somecollection) in the explainer output and no warning like "collection ... used as expression operand" is issued. NOOPT(COUNT(...)) prevents this optimization and the warning is returned.

The original problem seemed to have occurred during a graph traversal and not AQL, as indicated by the following line from the stack trace:
[/usr/share/arangodb3/js/common/modules/@arangodb/graph/traversal.js:948]

The error is thrown when V8 runs out of memory during JavaScript execution. If it cannot get any more memory, it fails hard.
3.1 should mitigate this problem, so it may be worth trying it. Additionally it is possible to start ArangoDB with a special options that allows V8 to use more memory. This is described here.

@koorukuroo did you tried it with ArangoDB version 3.1?

Meanwhile ArangoDB 3.1.16 is available for download - Closing.


Add finite memory queue

This adds the possibility to create memory queues that do not block when the queue EOFs. This mimics the behavior before #13 that is expected by other projects.

Personally, I would consider use channels here or pass context with cancel to consumer (Next).
Because when you want to stop consuming it's kind of special case of EOF.
So either you can close the channel or cancel context.

I would also use a channel here. In fact the AMQP version uses a channel behind the scenes. The problem is that there's software already using this interface (rovers, borges and lookout).


SnapshotArray not thread safe - multiple race conditions

Issue details
The add method of SnapshotArray is not thread safe.
This causes issues when adding actors asynchronous to a stage while the rendering thread is iterating the actors.
Reproduction steps/code
A simple fuzzy test can reproduce the issue
public class TestSnapshotArray
{
	private static final int TICKS = 100000;

	@Test
	public void testSynchronized() throws InterruptedException
	{
		final SnapshotArray<Integer> array = new SnapshotArray<>();

		Thread t1 = new Thread(new BeginEndFuzzy<>(array));
		Thread t2 = new Thread(new AddRemoveFuzzy(array));
		t1.start();
		t2.start();
		t1.join();
		t2.join();
	}

	private static class AddRemoveFuzzy implements Runnable
	{
		private final SnapshotArray<Integer> array;

		private AddRemoveFuzzy(SnapshotArray<Integer> array)
		{
			this.array = array;
		}

		@Override
		public void run()
		{
			int tick = 0;
			while (tick < TICKS)
			{
				tick++;
				for (int X = 0; X < 50; X++)
					array.add(X);

				for (int X = 0; X < 50; X++)
					array.removeValue(X, true);
			}
		}
	}

	private static class BeginEndFuzzy<T> implements Runnable
	{
		private final SnapshotArray<T> array;

		private BeginEndFuzzy(SnapshotArray<T> array)
		{
			this.array = array;
		}

		@Override
		public void run()
		{
			int tick = 0;
			while (tick < TICKS)
			{
				tick++;

				array.begin();

				T[] items = array.begin();
				for (int i = 0, n = array.size; i < n; i++)
				{
					assertNotNull(items[i]);
				}
				array.end();
			}
		}
	}
}

Version of LibGDX and/or relevant dependencies
1.9.8 / 1.9.9
Stacktrace
Not required as test case provided.
The issue is that the size field is not thread safe therefore the read loop will access the null entries of the snapshot or even worse get an out of bounds exception
Please select the affected platforms

[X] Android
[X] iOS (robovm)
[X] iOS (MOE)
[X] HTML/GWT
[X] Windows
[X] Linux
[X] MacOS

libGDX is not thread safe, unless specified otherwise. This is for performance reasons and because thread safety is much more complicated than just throwing synchronized everywhere.
This is not a bug. You should add necessary locking yourself.

I know that libgdx is not thread safe but according to the docstring of SnapshotArray it is thread safe. Otherwise using that data structure wouldn't make sense I guess.

Where does it say that?

The very first sentence implies it:
https://libgdx.badlogicgames.com/ci/nightlies/docs/api/com/badlogic/gdx/utils/SnapshotArray.html
Also the sort methods are explicitly noted as not thread safe implying that the other methods are safe to use in parallel.
If that's not the case it would be cool to maybe add a short notice to it?

Nothing is thread safe unless explicitly said so. This does not emply it if you keep this in mind.
That said if you think the documentation could use more clarifications, feel free to add in a PR.

Okay, will do. Thanks for your fast reply!

The note about sort not being thread safe is actually an understatement - it is even less thread safe than other non-thread safe methods. To be more specific: you can have multiple SnapshotArrays on different threads and use them in parallel, as long as no two threads use the same array instance at the same time. This is the "standard" definition of thread safety. But sort is special as it uses a static field through Sort.instance, which means that even if two threads have different SnapshotArray instances, they must not sort at the same time.


fix Chinese translation error

Thank you for these improvements @jzoy. I have merged them in, to be part of the next release.


Replace archive/tar fork with use of archive/tar

Since Go 1.6, archive/tar has gained public support for custom PAX Records, so there's no need to maintain a fork of this library anymore.
We do end up with a fork of the PAX timestamp parsing/formatting functions, but they are much less likely to change.
Copying the PAX time-parsing/formatting headers, and use of modTime as a fallback for creationTime inspired by containerd's reimplementation of this functionality, but recreated by hand.

As it happens, this change will make it feasible to write sparse blocks to the stream in GNU tar PAX sparse format 1.0, which archive/tar will happily decompress for us. I'm not sure how valuable that would be though, particularly if gziping the tarballs anyway.

It looks like this fix (or further hacks on the forked archive/tar) is needed to fix support for files larger than 8g when importing a layer.

@ambarve @kevpar PTAL

Is there anything I can do to get this advanced, @ambarve, @kevpar? I know it says '37 files', but that's 34 deletions, one addition, and only two changed files, most of which was search-and-replace. I'd really like to land this, so we can start vendoring it into downstream users and resolve some of the referenced issues.

LGTM!


Avoid collections.sequence

This PR suppresses warning in Python3.7.
DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working```

jenkins, test this please.

Jenkins CI test (for commit f33a07a5c2b70fa299f46745c8a20296d7764cca) succeeded without errors!

LGTM!


Dutch translation

I'm working on a Dutch/Nederlands translation at https://github.com/JoostGadellaa/covid-19
Some help, especially with the images, would be great!
Edit: I will be starting with words.md, so if somebody else wants to help, the other parts would be a great place to start!

I made a start and included the script the Germans use to automatically update the Index from the Words.md file. However, I underestimated how much time I would have in the weekend, so it will be a while before I can continue. Feel free to clone my repo if you want to continue.


solved #6 social media handles opens up in new window

Please review my request, and merge it if it completes the task.

Please make changes for the social media accounts in the footer of ourteam.html as well @maahiunnati

Done @sejalvasan  mam


Background broaden cite trim

Changes incorporated to a satisfactory extent as part of the last co-working session, as indicated by @asmacdo


Prediction

Is there a script for the LayoutLM classifier to predict on unlabeled dataset as well as showing confidence in the prediction?

@sarahbacha were you able to find an answer to this?


Adding custom messages and services

Hello,
I have another issue. I'm using the utils in the CubeIDE. I would like to add custom messages and services to project. I added a package with a custom message type to the micro_ros_static_library_ide/library_generation/extra_packages directory. But it is not included, when I build the project. My project structure looks like this:

Where should I put the package with the message type, so that the build script recognizes it.
And secondly is it somehow possible to rebuild the micro-ros library, when custom packages have been added, without the apt install part, because this part of the building process takes very long.
Thank you for your time and your help.

The build script should copy the folders that you add to extra_sources and import the repos that you add to extra_packages.repos. Check here.
Could you please @Acuadros95 check this and let us know if something is not working correctly?

So I should add the the package to extra_sources and not to extra_packages? I'll try that.

Sorry my fault, extra_packages is where it should work. I'm going to edit my previous comment.

I have found the error. I did not copy the package to extra_packages, but only created a symbolic link. This is the reason why the content of the package is not found by the build script.
But can you tell me something about my second question. Do I have to rebuild the entire library every time I add or change packages? This takes quite a long time, mainly because of the apt downloads. Is there a way to speed up the build script so that it only looks for the new or changed packages and rebuilds incrementally?

Using this docker approach is not possible to speed up the build using the incremental build, sorry...
We will put this in our roadmap to improve the build time. Thanks for the feedback.
If you don't mind I'm closing since it is solved. Do not hesitate to reopen if you have further problems.


[QUESTION] Material2

Any plans on workaround for material beta-7/beta-8?

I have clone material design source and make it suite for universal, you can check it here check this https://github.com/nhutcorp/angular-webpack2-starter

material 2 beta 11 is out and now requires Angular 4.4.3 or greater...

@kodeine https://github.com/nhutcorp/angular-webpack2-starter has updated


feat: parse checkboxes

This pull request allow checkboxes values to be parsed and stored in an array as described in #20

Hi @elhmn, thanks for submitting this change. I do agree that the current output isn't perfect. From a quick glance it all make sense. I need some more time to review this in details since it's a breaking change. Therefore, please bear with me. Thanks in advance.

This has been finally landed in v3. Thanks for your patience with this release


Refresh of PyTorch Image Classification Multi-Node Distributed Data Parallel Training on CPU using Vertex Training with Custom Container Notebook

This PR contains a notebook that demonstrates how to create a distributed PyTorch training job using Vertex SDK and custom containers. This can help your training job scale to handle a large amount of data.

[x] Use the notebook template as a starting point.
[x] Follow the style and grammar rules outlined in the above notebook template.
[ ] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[x] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[x] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[x] This notebook has been added to the CODEOWNERS file under # Official Notebooks section, pointing to the author or the author's team.
[x] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

Notebook runs on Workbench, not Colab as it doesn't support custom containers yet

/gcbrun

This is failing tests because it is testing other unrelated notebooks:
get_started_vertex_experiments.ipynb                                  PASSED    00:03:46    https://console.cloud.google.com/cloud-build/builds/ff669a3e-9456-4f56-907a-557066a53aae?project=1012616486416
Step #4: sdk-metric-parameter-tracking-for-custom-jobs.ipynb                   PASSED    00:17:08    https://console.cloud.google.com/cloud-build/builds/c8bc125c-4f18-4c85-bbe2-59531005d6cf?project=1012616486416
Step #4: sdk-metric-parameter-tracking-for-locally-trained-models.ipynb        PASSED    00:04:51    https://console.cloud.google.com/cloud-build/builds/77557d10-7593-4f1c-a56c-cf65061f9710?project=1012616486416
Step #4: google_cloud_pipeline_components_model_upload_predict_evaluate.ipynb  PASSED    00:34:25    https://console.cloud.google.com/cloud-build/builds/17e904f3-3499-44c6-9447-591c81bedbb8?project=1012616486416
Step #4: pipelines_intro_kfp.ipynb                                             PASSED    00:06:25    https://console.cloud.google.com/cloud-build/builds/969521a9-df47-4092-bf82-cbc2c8cf519e?project=1012616486416
Step #4: get_started_bq_datasets.ipynb                                         FAILED    00:07:39    https://console.cloud.google.com/cloud-build/builds/4b60153e-c2e9-4265-94c0-3071bc6d5336?project=1012616486416
Step #4: get_started_vertex_datasets.ipynb                                     FAILED    00:03:32    https://console.cloud.google.com/cloud-build/builds/ffec1cad-f537-4a61-b32f-09b175938c22?project=1012616486416
Step #4: get_started_automl_training.ipynb                                     FAILED    02:29:05    https://console.cloud.google.com/cloud-build/builds/8f4be373-1281-4889-be88-bbb82b16699e?project=1012616486416
Step #4: get_started_bqml_training.ipynb                                       FAILED    00:12:54    https://console.cloud.google.com/cloud-build/builds/6b40b25e-a626-4f23-abcb-35def46332ad?project=1012616486416
Step #4: get_started_vertex_training.ipynb                                     FAILED    00:22:03    https://console.cloud.google.com/cloud-build/builds/0fa47cd5-da01-4e3e-8bbe-646af673c934?project=1012616486416
Step #4: get_started_with_cmek_training.ipynb                                  FAILED    00:02:45    https://console.cloud.google.com/cloud-build/builds/fa5592fd-a86a-4a08-a509-0e1be8acdb3a?project=1012616486416
Step #4: mlops_experimentation.ipynb                                           FAILED    00:02:54    https://console.cloud.google.com/cloud-build/builds/7c62b99c-c785-4e1a-a296-778460f9379c?project=1012616486416
Step #4: get_started_with_dataflow_pipeline_components.ipynb                   FAILED    00:02:48    https://console.cloud.google.com/cloud-build/builds/69ea9e1b-d3f7-492c-919d-b922c5803613?project=1012616486416
Step #4: get_started_with_vertex_private_endpoints.ipynb                       FAILED    00:06:13    https://console.cloud.google.com/cloud-build/builds/c34dede5-4839-4560-81cf-782e708044a4?project=1012616486416
Step #4: get_started_with_tf_serving.ipynb                                     FAILED    00:03:41    https://console.cloud.google.com/cloud-build/builds/a9b6db7b-3e4a-4fc0-90f1-0bd224c4bf8e?project=1012616486416
Step #4: sdk_automl_tabular_forecasting_batch.ipynb                            FAILED    01:33:21    https://console.cloud.google.com/cloud-build/builds/f660b309-63f4-4f81-95f5-65f5954858a3?project=1012616486416
Step #4: multi_node_ddp_gloo_vertex_training_with_custom_container.ipynb       FAILED    00:03:00    https://console.cloud.google.com/cloud-build/builds/82408d57-d2a9-48da-aef6-ba3da48470db?project=1012616486416
Step #4:

/gcbrun

Build error: File 'trainer/task.py' not found.

@ivanmkc tests not ran: Changed files did not match file filter

was passing tests before I fixed a couple of non-code titles. Now says internal error. Will ignore and merge.


A question

Is it possible to authorize a method inside a controller which is manually instantiated with Authorize attribute, was not instantiated within HTTP pipeline. Now it looks like this part of this logic is skipped somehow. How do I have to create an instance of controller with all properties properly instantiated to make the authorization/authentication work properly.
Reason: I'm trying to reuse MVC controller logic, which has been already written for communicating and make it work via web sockets, so I want to route messages to controller methods (some routinf class based on some rules), and I need then to authorize inside the controller ideally. Then covert result via some adapter to create message supported by custom protocol and sent it back via web sockets.

MVC does a lot more than just new up controllers. The piece you're missing here is the authorization filter that runs before each action if the Authorize attribute is present.
https://github.com/aspnet/Mvc/blob/rel/1.1.3/src/Microsoft.AspNetCore.Mvc.Core/Authorization/AuthorizeFilter.cs
In 2.0 (dev) more of this has been extracted to a common source in this repo.
https://github.com/aspnet/Mvc/blob/c27b07ef3f8b07dfe1d1de38a8d7bf2d6a9298f4/src/Microsoft.AspNetCore.Mvc.Core/Authorization/AuthorizeFilter.cs

No I don't think is that, I'm having all the necessary middleware registered using Identityserver4 (everything works fine for standard MVC scenario), the problem is when I created new instance of Controller manually (kind of hack) new SomeController() and that controller has decorated methods with Authorize attribute, then this attribute seems to be skipped or not evaluated.

Newing up a class and calling its methods does in .NET does not trigger any form of attribute execution. Attributes are just metadata. See that the AuthorizeAttribute contains no meaninful code to execute: https://github.com/aspnet/Security/blob/dev/src/Microsoft.AspNetCore.Authorization/AuthorizeAttribute.cs
It's MVC that adds the functionality to inspect for attributes and take actions based on their presence and parameters.

Per @Tratcher 's comment, it is not supported to manually instantiate MVC controllers. If you do, it's not MVC anymore - it's simply a class with methods on it.
To run MVC's filter attribute, such as for authorization, the controller must be created and run by MVC itself.


Provide pure floss build flavor

2.5.1 introduced com.google.android.gms:play-services-appindexing:8.1.0, which is a proprietary dependency. Could your provide a gradle build flavor that does not require this?

I removed dependency and updated the build flavor.I think it is good to go

Thanks, builds fine now.


Get a minimal backend server running

Get a minimal backend server setup running so that we can build off of it.

Tracked in #60

Branch is merged, so this issue is resolved.


leave percentage empty but editable in case of mede financiering

OP-1630

When I select Toezichthoudend I can edit the percentage but we shouldn't be able if I understand properly the task ?

woops i edited the comment instead of reply <_<
You should be able to edit mede financering and toezicht but not adviserend I think.


Create 94-2048.md

I wanted to add my contribution and noticed Isaac Sanchez already pull requested this file. It was missing some sections such as repository and live example, so I created this pull request to fix these and add my 2048 page.

I don't know if this will be useful to someone, but I guess here's the pull request mentioned by @MatthewBraun : #463


Using pull-request 422 base error - SOS :(

I have my org's main repo  and I forked it and brought it locally and then created a branch… no problems making changes locally and then committing to my own fork at github via the commmand line, but then when I want to issue a pull request from command line for my forked branch to my org's repo master I am getting errors on command line - every thing I have looked at online suggests that it should work - what am I missing?

git remote -v
origin   https://github.com/myfork/mine.git (fetch)
origin   https://github.com/myfork/mine.git (push)
mainrepo  https://github.com/mainrepo/orgrepo.git (fetch)
mainrepo  https://github.com/mainrepo/orgrepo.git (push)

I then run:
git pull-request –b mainrepo:master -h origin:mybranch
Here is what I get back:
Error creating pull request: Unprocessable Entity (HTTP 422)
Invalid value for "base"
I am knocking my head against the wall.
Please help.

I realized that the problem was in my upstream setting - I never used that term and should have once I did that and then explicitly called the main owner/repo:branch for base and then mynameasowner:mybranch the command above worked!


Natural Horizontal scrolling

Fixes: https://github.com/Kethku/neovide/issues/582,
Re-enables horizontal trackpad scrolling and swaps the scroll direction. This was actually the source of the "jarring" feeling I was experiencing in my original PR. I also raised the scroll threshold to -2.0 and 2.0 so its slightly less sensitive.
Tested on macOS and arch linux with MacBook trackpad, Magic Mouse, and a "conventional" scrolling razer mouse.

This only fixes the sensitivity issue (how much you need to move to do the initial scroll), neovim itself still scrolls too fast. As far as I understand theres no way around it, we would need to implement sub-pixel scrolling for that.


Add ERC11551155 bridges

:clipboard: Add associated issues, tickets, docs URL here.
Overview
Describe what your Pull Request is about in a few sentences.
Add ERC1155 bridges
It supports the L1 and L2 native tokens. Developers can deploy their tokens on either sides and move to the another side.
Changes
Describe your changes and implementation choices. More details make PRs easier to review.

Add core contracts
Add tests
Rename the bridge folder to ERC721Bridge to keep things clear

Testing
Describe how to test your new feature/bug fix and if possible, a step by step guide on how to demo this.
The unit tests and the integration tests are added

Codecov Report
Base: 46.03% // Head: 74.61% // Increases project coverage by +28.57% :tada:

Coverage data is based on head (7340f3c) compared to base (33f6a50).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop     #436       +/-   ##
============================================
+ Coverage    46.03%   74.61%   +28.57%
============================================
  Files          102       64       -38
  Lines         3971     1926     -2045
  Branches       792      321      -471
============================================
- Hits          1828     1437      -391
+ Misses        2143      489     -1654

Flag
Coverage Δ

boba-contracts
?

contracts
79.30% <ø> (ø)

core-utils
86.80% <ø> (ø)

data-transport-layer
31.17% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../contracts/contracts/ERC721Bridges/L1NFTBridge.sol

.../contracts/contracts/ERC721Bridges/L2NFTBridge.sol

...boba/contracts/contracts/libraries/SafeMathDAO.sol

.../contracts/contracts/lzTokenBridge/lzApp/LzLib.sol

...ges/boba/contracts/contracts/L2BillingContract.sol

...ckages/boba/contracts/contracts/ERC721Registry.sol

...contracts/DAO/governance/GovernorBravoDelegate.sol

...contracts/contracts/L1CrossDomainMessengerFast.sol

...ba/contracts/contracts/lzTokenBridge/EthBridge.sol

...acts/contracts/oracle/payment/AccessController.sol

... and 28 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Uprade to PS 0.12 ?

Hi,
many thanks for this great library that is very useful for our project.
Since the 0.12 upgrade we were wondering if there was any plan(s) to upgrade purescript-thermite.
There is a branch created for that purpose with my collaborator already:
https://github.com/sudhirvkumar/purescript-thermite/tree/migrate_0_12
But before going deeper, we wanted to know if there was any plan(s) or advice(s) for such target.
Many thanks in advance for your answer.

I'm not really maintaining this any more. I'd like to find another maintainer, but in the short term, if someone wants to make a PR to update things to 0.12 then I'll merge it and make a release.

@delanoe @paf31 I just submitted a PR: https://github.com/paf31/purescript-thermite/pull/102

@paf31 ping - I'd be happy to take ownership of this package if you'd be willing - my PR passes tests, and is orthogonal to the new purescript-react version.

@athanclark Yes, sorry for the massive delay. I've added you as collaborator on the repo. Please feel free to merge as you see fit. I'll try to answer questions if it's useful, but don't feel like you need to wait for me to reply for approval now, or anything like that.

Awesome, thank you so much!!

@delanoe Alright, sorry for the delay. Version v6.0.1 is stable for purs v0.12.x and react v16.x


Adding policyAssignment causes error code:InvalidTemplate

Describe the bug
When adding a policyAssignment for a built-In policyDefinition, the assignment gets deployed (Visible in Azure Portal), but the DevOps Push Pipeline throws the error:
'AzOps-Microsoft.Authorization_policyAssignments-Deploy-AKS-Polic' failed with error(s). Showing 1 out of 1 error(s). Status Message: Unable to process template language expressions for resource '/providers/Microsoft.Management/managementGroups/<managementGroup>/providers/Microsoft.Authorization/roleAssignments/2dba37e0-5e7a-53f2-b15c-f451f5c8a197' at line '116' and column '9'.'The language expression property 'details' doesn't exist, available properties are 'effect'.' (Code:InvalidTemplate)
The correlationId is: a7f82a5c-7b26-4c92-9b23-4fda3c71cc3e
So the issue regards to the roleAssignment generated in the deployment.
Steps to reproduce
1.Deploy a policyAssignment for the built-In roleDefinition "[Preview]: Azure Policy Add-on for Kubernetes service (AKS) should be installed and enabled on your clusters"
2.
Screenshots

Can you share how you made the assignment? If you did it using AzOps, pls share the JSON you used, as I suspect it contains an identity object - although not required for this policy since it is an audit policy

I fixed this 'issue' by using another Built-In policyDefinition 'Deploy Azure Policy Add-on to Azure Kubernetes Service clusters'. BTW I used for the both policyDefinitions the following JSON:
{     "$schema": "http://schema.management.azure.com/schemas/2015-01-01/deploymentParameters.json#",     "contentVersion": "<IP_ADDRESS>",     "parameters": {         "input": {             "value": {                 "Identity": {                     "type": "SystemAssigned"                 },                 "Location": "westeurope",                 "Name": "Deploy-AKS-Policy-AddOn",                 "Properties": {                     "Description": null,                     "DisplayName": "Deploy-AKS-Policy-AddOn",                     "NotScopes": null,                     "Parameters": {},                     "PolicyDefinitionId": "/providers/Microsoft.Authorization/policyDefinitions/a8eff44f-8c92-45c3-a3fb-9880802d67a7",                     "Scope": "/providers/Microsoft.Management/managementGroups/<managementGroup>"                 },                 "ResourceType": "Microsoft.Authorization/policyAssignments"             }         }     } }
Even when the issue is resolved, I don't understand why the deployment with the first Built-In policyDefinition didn't work.

For all policyDefinitions that are not using deployIfNotExist or modify, you don't need to carry the "identity" object in the parameter file.

Alright, thanks


Fix NumbaDeprecationWarning: Set JIT to use nopython mode by default

Fixes: https://github.com/slundberg/shap/issues/2909
This updates the Just-In-Time compiler decorators to explicitly avoid using "nopython" mode. Numba has recently been raising a DeprecationWarning as per:
https://numba.readthedocs.io/en/stable/reference/deprecation.html#deprecation-of-object-mode-fall-back-behaviour-when-using-jit
This change replaces @numba.jit with @numba.njit, which is equivalent to @numba.jit(nopython=True). This removes the deprecationwarning, but will explicitly cause an exception to be raised if the relevant functions cannot be jit-compiled without reverting to the slower "nopython" mode. So, might want to wait until the test suite is passing to ensure this change does not break anything.

Good to have you on board @connortann. Thank you for your contribution. Looks good. I agree, let's wait until the test suite is up and running.

I used pip install git+https://github.com/dsgibbons/shap and I still get the numba deprication warnings. Any idea why? I filter them with warnings.filterwarnings('ignore', message=".*The 'nopython' keyword.*").

The issue of numba warnings isn't fully fixed: this PR removed some warnings but inadvertently introduced other numba warnings that the nopython=False behavoiur is being deprecated entirely.
I'll make another PR shortly to address this..

@NegatedObjectIdentity the Numba Deprecation warnings relating to nopython mode are fixed now on this fork. 🎉
On a separate but related note, there are some other numba warnings relating to deprecated types; tracking that issue at #70 . Let us know if you encounter any other issues or warnings!

@connortann awesome! For me, the warnings are gone!

@connortann  Will you please update the conda package. The error is still there.

@0rky We haven't released anything on PyPI or conda yet. We will consider a release strategy soon.


Discriminated unions raise TypeError for unions of a single type

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.9.0
            pydantic compiled: False
                 install path: /home/tom/reinfer/env/lib/python3.8/site-packages/pydantic
               python version: 3.8.10 (default, Nov 26 2021, 20:14:08)  [GCC 9.3.0]
                     platform: Linux-5.11.0-44-generic-x86_64-with-glibc2.29
     optional deps. installed: ['dotenv', 'typing-extensions']

When a Union with only a single variant it tagged with a field as a discriminated union, a type error is thrown on defining the class. The below is a minimally reproducible example:
# repro.py
from typing import Literal, Union

from pydantic import BaseModel, Field

class DataAModel(BaseModel):
    kind: Literal["a"]

class RequestModel(BaseModel):
    data: Union[DataAModel] = Field(..., discriminator="kind")

produces
$ python repro.py
Traceback (most recent call last):
  File "repro.py", line 10, in <module>
    class RequestModel(BaseModel):
  File "/home/tom/r/env/lib/python3.8/site-packages/pydantic/main.py", line 204, in __new__
    fields[ann_name] = ModelField.infer(
  File "/home/tom/r/env/lib/python3.8/site-packages/pydantic/fields.py", line 488, in infer
    return cls(
  File "/home/tom/r/env/lib/python3.8/site-packages/pydantic/fields.py", line 419, in __init__
    self.prepare()
  File "/home/tom/r/env/lib/python3.8/site-packages/pydantic/fields.py", line 534, in prepare
    self._type_analysis()
  File "/home/tom/r/env/lib/python3.8/site-packages/pydantic/fields.py", line 605, in _type_analysis
    raise TypeError('`discriminator` can only be used with `Union` type')
TypeError: `discriminator` can only be used with `Union` type

The following code snippet works as intended. The only difference is I have introduced a second type in the discriminated union field. Note that simply repeating the existing type (such as Union[DataAModel, DataAModel]) still fails:
from typing import Literal, Union

from pydantic import BaseModel, Field

class DataAModel(BaseModel):
    kind: Literal["a"]

class DataBModel(BaseModel):
    kind: Literal["b"]

class RequestModel(BaseModel):
    data: Union[DataAModel, DataBModel] = Field(..., discriminator="kind")

After some poking, it looks like within _type_analysis, self.discriminator_key is set on the model DataAModel, rather than the parent field, Union[DataAModel]. I'm guessing there is a simplification step somewhere where singleton unions are simplified to their child type, which is no longer accurate?
Happy to submit a PR if you can point me further in the right approach.

Hi @tommilligan
The thing is we can't do anything about it because python changes Union[A] into A at interpretation time.
from typing import Union
class A: ...
assert Union[A] is A

So at runtime, pydantic has no way to know A was supposed to be a Union

Ah, that makes sense. I was about to start poking to see if the simplification was internal to pydantic or not, but if it's at a higher layer I'll give it up as a lost cause.
Would you accept a PR noting this as an edge case in the documentation/error message?
I suppose the workaround is to add a dummy second type into the union, or just to remove the discriminator until it is required.
For documentation forward compatibility, we were using openapi-schema-pydantic, which now collides with the discriminator Field property nicely. But that's not your problem! I'll file a bug over there now.

Documentation PRs are always welcome :)

Opened https://github.com/samuelcolvin/pydantic/pull/3639


Support for GPG keys in identities

Hi madx,
First, thank you very much for that simplistic yet awesome git plugin. I've been looking for such a solution since ages and you came with an incredibly efficient idea. I however thought it could benefit from GPG key integration for commit verification. So I added it and... here's my modest contribution if you're willing to integrate it to your project!
My modifications are:

Integration of GPG keys via git identity --define-gpg for those who would like to sign their commits
Documentation related to its usage + minor enhancements to the README
Additional conditions to avoid triggering things that could lead to errors/inconsistencies:

git identity <identity name> isn't triggered if the identity is not defined
git identity --define-gpg <identity name> <gpgkey> isn't triggered if the identity is not defined
git identity returns the current identity if it exists; otherwise a messages says no identity is selected

When run, git identity --define-gpg runs again the git identity <identity name> command to ensure the local repository is updated accordingly

Currently there's no support for removing/disabling a GPG key since I didn't quite see the use case there, but it could be implemented pretty easily.
Hope you'll find my update interesting, and thanks again to all contributors of this plugin!

1000 :+1: to this. It's natural addition, thanks for taking the time to publish this.
I can confirm this works as intended, but I haven't stressed tested it.

You are missing the new subcommand on the shell completions.
https://github.com/somini/git-identity/commit/2973a4b43e74773620fe412a13e3a58bbef299fa

Thanks and you're welcome. I thought it could be interesting for someone else so I shared. I'm glad you found it useful.
Oops, you're right. It seems I have forgotten to update the auto-complete scripts, thanks for your help!
I added 2973a4b43e74773620fe412a13e3a58bbef299fa to the pull request.

Hi everyone,
First, thank your for your kind words regarding this small project, and sorry for the late reply, I've been quite sick these last days :)
I'm very happy to include this feature in git-identity, thanks a lot for your contribution!