feat(ui): ic-icon component

Close #472

How about aligning with the behavior of @unocss/preset-icons? With the backgroundImage, we can easily use font-size to configure its width/height.

And why does this happen, is it because of treeshaking or should it be like this for offline first ?

And why does this happen, is it because of treeshaking or should it be like this for offline first ?

To comply with Chromium Manifest V3 requirements, we bundled all the ic icons(for legacy devtools plugin api) together instead of loading them online. Manifest V3 does not support loading external resources.

ahhh got it. you should have specified that the icon pack names are like: devtools-legacy :)

@webfansplz updated, can you review it? Thanks


fix(mongodb): cleaning mongodb entity identifier

Relevant information

in the definition, we should not add the targetName in the composite ID since the mongodb_cluster_name is already unique and added by the customer (this could be a problem since if the integration is restarted and a port assigned, the entity GUID changes)
we are not adding encodeIdentifierInGUID, which possibly causes the ID to be truncated at 50 chars. This would result in collisions for each ID longer than 50

Checklist

[x] I've read the guidelines and understand the acceptance criteria.
[x] The value of the attribute marked as identifier will be unique and valid.

@gsanchezgavier @carlossscastro


Theme Check Does Not Ignore Files Indicated

I have ignored files from theme-new folder, created when generating theme files, with .theme-check.yml:
  ...
  - theme-new/

The folder is also ignored in .gitignore, and .themekitignore file:
...theme-new/

Where .themekitignore is indicated as ignore listing in config.yml:
development:
   ...
  ignores:
  - .themekitignore

Current result:
When I go to "theme-new" and open a file with VS Code, Theme Check starts theme validation process.
Expected result:
Theme Check ignores files ignored by config.yml, .theme-check.yml or .gitignore and does not run theme check on them and whole project.

Are you missing a * perhaps?
https://github.com/Shopify/theme-check/blob/main/config/default.yml#L11-L12

Closing in favour of https://github.com/Shopify/theme-check/issues/579. It's an actual issue indeed.


!alog command not working

When someone tries using the command "!alog " the bot says "Insufficient permissions, missing: Embed Links"

Just saw a previous issue where a user had "Show website preview info from links pasted into chat." unchecked, but it is checked for me. Just fyi :-)

Alog, aswell as many other commands use embed messages for cleaner/nicer looking output. For those commands to work, the bot needs the permission "Embed Links". The !invite command will actually list every command that requires elevated permissions beyond the normal Read/Write/Read Message History.
Simply going into your guild settings, and creating a role for RuneInfo or editing an existing role with the additional permissions will allow the bot to use those commands.

Tyvm for fast response. Will relay the message and give it a shot.

http://prnt.sc/dvaysr this is a screenie from one of our admins. Any other ways to possibly fix?

Few things to check then... Check permission overwrites in the channel itself. If @everyone has deny on that channel, and there is no overwrite for the @bot role with an allow, then the bot wouldn't have permission.
Feel free to hop in https://discord.gg/TrDnKtQ for trouble shooting chat.

Since I haven't heard back from you in 9 days, I assume you have this issue sorted. Feel free to open again, or hop in my support guild if you need any more help.


Add Recipe Feature (and other fixes + cleanup)

Various Changes

refactoring

simplified forms
data hook extended (further simplified views and containers)
extracted logic to util modules

added new data hooks

added SimpleCard component

UI enhancements

updating the view on the home page to space better
enhanced up edit/view cards with better spacing and new borders
add recipe form added

cleanup

removed some comments
removed unused imports
demoImage extracted for easier removal later (will be adding image feature)

bugs

fixed the form rendering the backend type when empty
fixed some NREs
fixed carousel spacing
fixed API to return ID on adding a recipe

renders

renders


neovim: add 'configure' flag

so that we have the same options as in nixpkgs.
The configure options is a pain to use so I hope to improve it in nixpkgs at later date (to set $MYVIMRC, fix paths for neovim etc) but meanwhile we have to stick with this.

I've sent you a collaborator invite so that you can merge this yourself once you're happy with it. But please keep using PRs for larger changes and avoid creating merge commits for minor changes :-)

I've sent you a collaborator invite so that you can merge this yourself once you're happy with it. But please keep using PRs for larger changes and avoid creating merge commits for minor changes :-)

I dont get your point, isn't opening a PR for peer review a better practice than pushing straight to master ? even for minor changes ?
Thanks for the invitation.

Sorry, I should have been more clear. If you are fixing the spelling of some option description or something trivial like that then don't worry about creating a PR. For things larger than that you should always feel free to open a PR; basically if you are wondering whether the thing is large enough for a PR then make a PR :-)


Extensions incorrect typings generated (casing)

Bug description
defineExtension function types are targeting related types by incorrect casing, if the model name is lowercase
How to reproduce
https://github.com/Faithfinder/prisma-extenstions-lowercase-repro/blob/b664c7a2547c12926c3e56fe6c6bf2250143210d/prisma/built/index.d.ts#L877
Here's repro repo. Type Prisma.postGroupByArgs type doesn't exist (should be Prisma.PostGroupByArgs)
Expected behavior
No response
Prisma information

// This is your Prisma schema file,
// learn more about it in the docs: https://pris.ly/d/prisma-schema

generator client {
  provider        = "prisma-client-js"
  output          = "./built"
  previewFeatures = ["clientExtensions"]
}

datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

model User {
  id    Int     @id @default(autoincrement())
  email String  @unique
  name  String?
  posts post[]
}

model post {
  id        Int     @id @default(autoincrement())
  title     String
  content   String?
  published Boolean @default(false)
  author    User    @relation(fields: [authorId], references: [id])
  authorId  Int
}

Environment & setup

Prisma Version
4.9.0

Thanks for the report, I can confirm this.

This is a blocker for us using the the clientExtensions as well. Just wanted to drop a line in here in the hopes that this gets prioritized for the 4.10.0 release (or sooner!). Any chances of that happening?

I've been reviewing the versions in dev, currently I can confirm that the error still appears in the version "4.10.0-dev.81", it is a great limitation not to be able to use "clientExtensions"
This could make life easier.

I've been reviewing the versions in dev, currently I can confirm that the error still appears in the version "4.10.0-dev.81", it is a big limitation not to be able to use "clientExtensions"
This could make life easier.

I've been reviewing the versions in dev, currently I can confirm that the error still appears in the version "4.10.0-dev.81", it is a big limitation not to be able to use "clientExtensions"
This could make life easier.

Same. Completely stuck at 4.8.1 because of that

@tarekwiz same, I've tried the latest version (4.10.1) and the problem still be there, I hope they can make a priority this scenario.

I've tried the latest version (4.11.0) and the problem still be there.
@millsp

@millsp It's been almost 4 months and still not fixed yet..


[Bug] Golang docker builds seem not properly cached

Describe the bug
Consecutive builds should be fast, sometimes they are, other not.
Expected Behavior
Use cache properly to speed up. Probably, we need to cache by architecture (and not shared)
Steps To Reproduce
No response
Exceptions (if any)
No response
Version
No response

solved using cross compilation in docker file.


precompiled script plugin "is not supposed to be used along with regular Kotlin sources" in Gradle 8.1

If you checkout https://github.com/Vampire/setup-wsl/ and try to execute its help task with a recent 8.1 snapshot like 8.1-20230217231705+0000, the build fails with
w: Script 'dependency-updates-report-aggregation.gradle.kts' is not supposed to be used along with regular Kotlin sources, and will be ignored in the future versions by default. (Use -Xallow-any-scripts-in-source-roots command line option to opt-in for the old behavior.)
e: warnings found and -Werror specified

As gw --scan help cannot produce a build scan, here the build can of gw -p gradle/dependency-updates-report-aggregation/ --scan compileKotlin which fails the same way: https://scans.gradle.com/s/szbf6prug4mvo

Probably related to #23491 which is included in 8.1 but does not fix this here.

And if I comment out allWarningsAsErrors = true as "work-around", it even gets worse:
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:17:1 No script runtime was found in the classpath: class 'kotlin.script.templates.standard.ScriptTemplateWithArgs' not found. Please add kotlin-script-runtime.jar to the module dependencies.
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:35:1 Unresolved reference: plugins
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:36:5 Unresolved reference: ide: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:64:12 Unresolved reference: parent
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:64:30 Unresolved reference: parent
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:66:37 Unresolved reference: configurations
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:67:9 Unresolved reference: isCanBeConsumed
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:68:9 Unresolved reference: isCanBeResolved
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:69:9 Unresolved reference: attributes
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:70:13 Unresolved reference: attribute
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:70:43 Unresolved reference: objects
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:71:13 Unresolved reference: attribute
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:71:52 Unresolved reference: objects
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:76:10 Unresolved reference: includedBuilds
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:77:16 Unresolved reference: it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:79:5 Unresolved reference: dependencies
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:81:26 Unresolved reference: it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:82:14 Overload resolution ambiguity:
public inline fun <T> Iterable<TypeVariable(T)>.forEach(action: (TypeVariable(T)) -> Unit): Unit defined in kotlin.collections
public inline fun <K, V> Map<out TypeVariable(K), TypeVariable(V)>.forEach(action: (Map.Entry<TypeVariable(K), TypeVariable(V)>) -> Unit): Unit defined in kotlin.collections
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:82:52 Unresolved reference: it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:85:5 Unresolved reference: tasks
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:86:9 Unresolved reference: inputs
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:90:9 Unresolved reference: gradleReleaseChannel
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:91:9 Unresolved reference: checkConstraints
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:92:9 Unresolved reference: rejectVersionIf
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:94:35 Unresolved reference: objects
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:95:9 Unresolved reference: extensions
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:95:24 Unresolved reference: DomainObjectCollection
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:97:9 Unresolved reference: outputFormatter
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:97:27 Unresolved reference: closureOf
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:98:44 Overload resolution ambiguity:
public inline fun <T> Iterable<TypeVariable(T)>.forEach(action: (TypeVariable(T)) -> Unit): Unit defined in kotlin.collections
public inline fun <K, V> Map<out TypeVariable(K), TypeVariable(V)>.forEach(action: (Map.Entry<TypeVariable(K), TypeVariable(V)>) -> Unit): Unit defined in kotlin.collections
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:98:54 Cannot infer a type for this parameter. Please specify it explicitly.
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:101:61 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:103:17 Unresolved reference:current
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:104:17 Unresolved reference:outdated
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:105:17 Unresolved reference:exceeded
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:106:17 Unresolved reference:undeclared
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:107:17 Unresolved reference:unresolved
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:110:27 Unresolved reference:outdated
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:112:51 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:114:13 Unresolved reference:outdated
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:116:13 Unresolved reference:unresolved
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:117:18 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:118:30 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:118:65 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:119:29 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:122:26 Unresolved reference.None of the following candidates is applicable because of receiver type mismatch:
public val Result.withUpdatedCounts: Result defined in net.kautler.util in file ResultExtension.kt
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:124:31 Unresolved reference:project
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:124:40 Unresolved reference:revision
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:124:50 Unresolved reference:gradleReleaseChannel
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:128:67 Unresolved reference:revision
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:129:25 Overload resolution ambiguity:
public inline fun <T> Iterable<TypeVariable(T)>.forEach(action: (TypeVariable(T)) -> Unit): Unit defined in kotlin.collections
public inline fun <K, V> Map<out TypeVariable(K), TypeVariable(V)>.forEach(action: (Map.Entry<TypeVariable(K), TypeVariable(V)>) -> Unit): Unit defined in kotlin.collections
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:130:35 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:130:47 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:130:59 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:130:76 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:130:89 Unresolved reference:revision
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:131:21 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:131:57 Unresolved reference:it
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:135:24 Unresolved reference:current
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:139:24 Unresolved reference:current
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:145:36 Unresolved reference:configurations
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:146:9 Unresolved reference: isCanBeConsumed
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:147:9 Unresolved reference: isCanBeResolved
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:148:9 Unresolved reference: attributes
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:149:13 Unresolved reference:attribute
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:149:43 Unresolved reference:objects
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:150:13 Unresolved reference:attribute
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:150:52 Unresolved reference:objects
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:154:5 Unresolved reference: tasks
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:155:26 Unresolved reference:file
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:156:9 Unresolved reference: outputs
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:158:9 Unresolved reference: checkForGradleUpdate
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:159:9 Unresolved reference: checkConstraints
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:160:9 Unresolved reference: rejectVersionIf
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:162:9 Unresolved reference: outputFormatter
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:162:27 Unresolved reference:closureOf
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:164:26 Unresolved reference:parentFile
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:166:24 Cannot infer a type for this parameter. Please specify it explicitly.
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:168:59 Type mismatch: inferred type is Dependency_updates_report_aggregation_gradle but Result was expected
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:173:5 Unresolved reference: artifacts
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:174:9 Unresolved reference: add
e: file:///D:/Sourcecode/other/setup-wsl/gradle/dependency-updates-report-aggregation/src/main/kotlin/net/kautler/dependency-updates-report-aggregation.gradle.kts:174:43 Unresolved reference:tasks

https://scans.gradle.com/s/7gsf46e2cvzds

Hi Björn,
There are two places in your build where the freeCompilerArgs Kotlin compiler option is overwritten instead of appended to:

https://github.com/Vampire/setup-wsl/blob/master/gradle/build-logic/build-logic.gradle.kts#L48
https://github.com/Vampire/setup-wsl/blob/master/gradle/dependency-updates-report-aggregation/dependency-updates-report-aggregation.gradle.kts#L57

This fixes the problem reported in this issue.
But this uncover another issue with eager observation of freeCompilerArgs now that the Kotlin Gradle Plugin 1.8 actually uses Property API for this in compilerOptions {} instead of kotlinOptions {}.
Here's a diff of the changes that make the build work:

diff.patch
diff --git a/gradle/build-logic/build-logic.gradle.kts b/gradle/build-logic/build-logic.gradle.kts
index 4233ac2..e7c859e 100644
--- a/gradle/build-logic/build-logic.gradle.kts
+++ b/gradle/build-logic/build-logic.gradle.kts
@@ -38,14 +38,14 @@ dependencies {
 }

 tasks.withType<KotlinCompile>().configureEach {
-    kotlinOptions {
-        allWarningsAsErrors = true
+    compilerOptions {
+        allWarningsAsErrors.set(true)
     }
 }

 tasks.compileKotlin {
-    kotlinOptions {
-        freeCompilerArgs = listOf("-opt-in=kotlin.RequiresOptIn")
+    compilerOptions {
+        freeCompilerArgs.add("-opt-in=kotlin.RequiresOptIn")
     }
 }

diff --git a/gradle/dependency-updates-report-aggregation/dependency-updates-report-aggregation.gradle.kts b/gradle/dependency-updates-report-aggregation/dependency-updates-report-aggregation.gradle.kts
index a4f339d..0b0a42b 100644
--- a/gradle/dependency-updates-report-aggregation/dependency-updates-report-aggregation.gradle.kts
+++ b/gradle/dependency-updates-report-aggregation/dependency-updates-report-aggregation.gradle.kts
@@ -47,14 +47,14 @@ dependencyAnalysis {
 }

 tasks.withType<KotlinCompile>().configureEach {
-    kotlinOptions {
-        allWarningsAsErrors = true
+    compilerOptions {
+        allWarningsAsErrors.set(true)
     }
 }

 tasks.compileKotlin {
-    kotlinOptions {
-        freeCompilerArgs = listOf("-opt-in=kotlin.RequiresOptIn")
+    compilerOptions {
+        freeCompilerArgs.add("-opt-in=kotlin.RequiresOptIn")
     }
 }

Ah, yeah, thanks.
Totally forgot to switch to compilerOptions for KGP 1.8.
And the free compiler arg is actually obsolete in the meantime, the code that required it was gone already, so I deleted them.
I don't close this issue, as still a build that worked with 6.0 breaks with 6.1 in case you want to change something in that regard.
I wonder what changed that broke it.

We added compiler args, hence overriding them breaks.
This is worth adding to the upgrading guide, thanks for pointing that out!

Done in

https://github.com/gradle/gradle/pull/24003

Closing


How to set the "encoding" of a field when "code first" is used with "mysql"

database default encoding “utf-8”
There is a field that needs to be modified to encoding "utf-8mb4".
How to do it with "code first"？

@AndriySvyryd This looks related to an area you were looking at.

You should be able to do it with something like .HasColumnType("longtext CHARACTER SET utf-8mb4")


tests: timer_api: Fix duplicate timestamp update.

Remove duplicate "tdata.timestamp" update in duration_expire; this
value is already updated by k_uptime_delta.
Besides simply removing duplicate value update, this commit also
addresses the intermittent assertion failure that is caused by
updating "tdata.timestamp" at a later time than the actual execution
of the k_uptime_delta function.
Signed-off-by: Stephanos Ioannidis<EMAIL_ADDRESS>

The following is an example of intermittent assertion failure on nRF51, that is caused by this issue:
***** Booting Zephyr OS version 2.0.99 *****
Running test suite timer_api
===================================================================
starting test - test_timer_duration_period
PASS - test_timer_duration_period
===================================================================
starting test - test_timer_period_0

    Assertion failed at ../src/main.c:167: test_timer_period_0: tdata.expire_cnt == 1 is false

FAIL - test_timer_period_0
===================================================================
starting test - test_timer_expirefn_null
PASS - test_timer_expirefn_null
===================================================================
starting test - test_timer_periodicity
PASS - test_timer_periodicity
===================================================================
starting test - test_timer_status_get
PASS - test_timer_status_get
===================================================================
starting test - test_timer_status_get_anytime
PASS - test_timer_status_get_anytime
===================================================================
starting test - test_timer_status_sync

    Assertion failed at ../src/main.c:63: duration_expire: k_uptime_delta(&tdata.timestamp) >= 50 is false

ASSERTION FAIL [!(thread->base.user_options & ((1UL << (0))))] @ ZEPHYR_BASE/arch/arm/core/cortex_m/thread_abort.c:37
        essential thread aborted
[00:00:01.651,550] <err> os: r0/a1:  0x00000004  r1/a2:  0x00000025  r2/a3:  0x00006554
[00:00:01.661,773] <err> os: r3/a4:  0x00002349 r12/ip:  0x00000000 r14/lr:  0x00002351
[00:00:01.671,966] <err> os:  xpsr:  0x21000021
[00:00:01.678,344] <err> os: Faulting instruction address (r15/pc): 0x00005374
[00:00:01.687,622] <err> os: >>> ZEPHYR FATAL ERROR 4: Kernel panic
[00:00:01.695,953] <err> os: Current thread: 0x200002a0 (idle)
[00:00:01.703,765] <err> os: Halting system

After apply this fix, Assertion failed at ../src/main.c:63: duration_expire: k_uptime_delta(&tdata.timestamp) >= 50 is false completely goes away.
Meanwhile, Assertion failed at ../src/main.c:167: test_timer_period_0: tdata.expire_cnt == 1 is false is a whole other issue (which I haven't really analysed yet).

@pabigot I have pushed a new commit.
Thanks for the clarification on the second issue, by the way.

Sure. Though I do note that the call to k_uptime_get() at the end of the function has been removed, which is going to decrease the code coverage of this test case. Approving anyway because I know with 100% certainty there are other tests cases for that API.
In general I'm not a fan of "fixing" tests to be better code if they aren't actually causing failures. Bad code tends to find edge cases in APIs better, in my experience.

k_uptime_get is already called inside k_uptime_delta, which is called as part of the test.
Re-setting timestamp by calling k_uptime_get again after k_uptime_delta will, given the right conditions, cause the test to fail because the next call to k_uptime_delta will return a period slightly shorter than configured (as demonstrated on nRF51).


AddJVMConfigForIdleChannelTimerResolution

Represents the value used in ReaderIdleTimer and writerIdleTimer used in idlestatehandler.
Issue trying to solve:
Currently, the value we picked in SDK is optimized for normal/high load traffic, for very low traffic, customer sometimes will observe up to 100ms latency in pipelined stage. Lowering this value can reduce the frequency of the spike requests in very low traffic mode.

LGTM. Thanks @xinlian12.

thanks Mo for looking at this PR, but I just found out that we can just using the existing one "azure.cosmos.directTcp.defaultOptions", so this PR is not needed anymore


Updated shifter documentation

TXSxxx are now not recommended
added bigger warning about I2C shifters with an image

I can not preview the !!!warning box, not sure it displays correctly now.


Converting Mask-RCNN model trained on custom dataset to FP16 format gives error during inference

Hey there,
I tried to convert the model mentioned in #271 to FP16 format and run inference with that. Converting the model to FP16 seems to work but unfortunately the Inference Engine returns an error when it tries to load the model:
RuntimeError: Check 'element::Type::merge(element_type, element_type, node->get_input_element_type(i))' failed at core/src/op/util/elementwise_args.cpp:22:
While validating node 'v1::Multiply Multiply_1288 (StridedSlice_562[0]:f16{1000,4}, Mul_565/mul[0]:f32{1,4}) -> ()' with friendly_name 'Multiply_1288':
Argument element types are inconsistent.

There are numerous warnings when exporting the model to FP16 format, especially one which is related to the operation Mul_565 which is also mentioned in the error message of the Inference Engine:
[ WARNING ]  Changing Const node 'Mul_565/mul' data type from int64 to <class 'numpy.float32'> for Elementwise operation

I have applied all workarounds and fixes mentioned in #271.
For reference, I attach the full logs of the command line output from exporting the model and from running inference:
Export log: log_export.txt
Inference log: log_inference.txt
Thanks and best regards,
Jens

Hi! It's known issue that mo_pytorch converter works badly with --data_type=FP16 (see https://github.com/openvinotoolkit/openvino_contrib/pull/222). Do you need FP16 on GPU? Because for other hardware targets there is no difference for FP32 and FP16 (CPU converts automatically to FP32 and VPU converts to FP16).

Hey @dkurt,
ah, sorry... I haven't seen #222 before. I should have searched more extensively for existing tickets.
My goal is to run this model (and other models in the future) on a MyriadX in an OAK-D camera. As far as I understand the MyriadX documentation, the models work best when converted to FP16. Or have I misunderstood something here?

@jsiebert, you can use just FP32 model. Myriad plugin converts it to FP16 anyway.

Okay, understood! I will close this issue...
Thanks for your support 👍


QGAN.Py

define a Generator class, which generates quantum states from latent vectors.
define a Discriminator class, which distinguishes between real and fake quantum states.
The QGAN class combines the generator and discriminator to form the QGAN model.
The forward method of the QGAN class generates fake quantum states using the generator and then passes them through the discriminator to obtain the discriminator's output
fixes #264

Hello, thanks for your patience!
To be more specific for the examples, if you could create a new folder in the examples folder, add a README, parametrize your python file, and add a python notebook, that would complete the task. For more details, check out the other PR linked to the issue.
Let me know if you have any questions!

@AbdullahKazi500 Can you also address @rakeshy32’s comments? We’re happy to merge this, but it would be great if you could work with @rakeshy32 to update the example!


Rollbacks?

Hello,
We've recently been experience some issues with PermissionsEX where the file sometimes doesn't save/rollsback users after initiating a /pex reload command.
We have the following server:
http://www.soyoustart.com/us/offers/e3-ssd-3.xml // Centos 6
Multicraft Preview14 installed on the box
PermissionsEX Version:
V1.23.3
ChatEX Version:
V1.2.0
What does this occur?:
Almost everytime I /pex reload, initially it was not loading the data until I /pex reload'd twice in a row, now its rolling back data whenever I even /pex reload once, or even restart the server. It seems like PEX just isn't saving the data correctly.
Server IP:
mc.Imprisoned.me
What plugins do I have on the server?:
https://gyazo.com/f32c6c00f443c3cd8dbe5000f1b66378
There's a bit of custom plugins on the server, if you need any information on any of them feel free to ask.
Groups:
http://pastebin.com/5dgSQQEw
Users:
http://pastebin.com/61pAbZiZ
The Chmod of the server:
https://gyazo.com/72d8d80c7aaf959ff40c2365e6f23a06
Just in case you need this:
https://gist.github.com/anonymous/712a7b4401fcf4c6c217

This seems like it's happening because a reload is triggered before a save is complete. Unfortunately because PEX1.x isn't really designed to make async saving easy, this issue won't be resolved easily without finishing up PEX2


Support standard Ctrl+W keyboard shortcut for 'close tab' action

Is your feature request related to a problem? Please describe.
This is a usability issue. I frequently have a number of tabs open in Storage Explorer, e.g. for multiple blob containers, tables, queues, etc. The standard convention is to allow for the Ctrl+W keyboard shortcut to close the active tab, but this does not appear to be bound to the 'close tab' action in Storage Explorer. Instead, I have to use the mouse to close the tab. If I am heavily keyboard-focused (which is often the case when I am writing code and switching windows to work with Storage Explorer) this is quite frustrating.
Describe the solution you'd like
Please create a new keyboard shortcut to bind the Ctrl+W key combination to the 'close tab' action.
Describe alternatives you've considered
N/A
Additional context
This applies to the Windows build of Storage Explorer, and the feedback is current as of v1.15.1.

@johndowns, I just tried this and it looks like that is already the shortcut on macOS, and it's been that way since summer of 2018. You can see it in the File menu:

That shows it’s Command-W, not Control-W.

From: Matthew Rayermann<EMAIL_ADDRESS>Sent: Saturday, October 3, 2020 10:35:42 AM
To: microsoft/AzureStorageExplorer<EMAIL_ADDRESS>Cc: John Downs<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/AzureStorageExplorer] Support standard Ctrl+W keyboard shortcut for 'close tab' action (#2921)
@johndownshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjohndowns&data=02|01|john.downs%40microsoft.com|ed3339a4f99f4b32c0a308d8671b1d63|72f988bf86f141af91ab2d7cd011db47|1|0|637372713443885703&sdata=nGuFT4kVl8Rp9QWlvgP57YgY47yiznEuHHi1wZDm%2Fi4%3D&reserved=0, I just tried this and it looks like that is already the shortcut on macOS, and it's been that way since summer of 2018. You can see it in the File menu:
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAzureStorageExplorer%2Fissues%2F2921%23issuecomment-702968654&data=02|01|john.downs%40microsoft.com|ed3339a4f99f4b32c0a308d8671b1d63|72f988bf86f141af91ab2d7cd011db47|1|0|637372713443895659&sdata=RLPaf5RSe9lPki1typCTUCSJLjCK4h%2B%2F5EHFund1zhA%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABNENWJVIDOS2KO23PBAGHLSIZBS5ANCNFSM4M2CPV7Q&data=02|01|john.downs%40microsoft.com|ed3339a4f99f4b32c0a308d8671b1d63|72f988bf86f141af91ab2d7cd011db47|1|0|637372713443895659&sdata=rqPiJANblSnn1GlplJNFUeomHtYcGxz3sNr6JjI8wI4%3D&reserved=0.

@MRayermannMSFT You're correct - on macOS it works correctly (note this is Command-W rather than Ctrl+W though). This feedback is specifically about the Windows version. I have just verified this on v1.15.1 of the Windows build - on there, the 'Close Editor' menu item is linked to Ctrl+F4, which is not standard.

Ah, yes. My apologies.

New file menu on Windows:

Menus on mac still the same:

Awesome! Thank you @MRayermannMSFT, that is perfect.

Merged into main for 1.16


remove the edgedb dir, put everything in the project root

my fork was broken (no longer a fork) after this repo became public. I deleted my broken fork and re forked so I can't fix the conflicts. I'll fix and open a new PR.


enable deny for use of unwrap

Consider adding
# This will deny use of unwrap on Result and Option outside of tests
unwrap_used = { level = "deny",  allow-unwrap-in-tests = "true" }

to the lints.clippy section of cargo.toml to enable denying unwrap in our code.
There are some test helpers which are not explicitly tests and will now pass. Easy enough to allow unwrap on their mod definitions though to unblock. Similar for our main fn in build.rs which is easy to resolve.
There are still ~60 uses of unwrap to resolve though so its not job done and we should reach some consensus before doing more than this exploration I think.

related #9


Hangs or slows way down when packing > 1000 images

Thanks for this great library!
Unfortunately the library seems to become unusably slow once you go over 1000 images. I have a set of ~4000 images that I want to pack. I added some print statements in the library to see where the slowdown was, looks like its mostly in the _pack() call in MaxRectsPacker. Specifically, it gets stuck in the loop over the image_rect_list at around 1350 images.
Are there some settings that can be used to make this an approximate packing, so that it can move faster?

Thanks for using.
To make this an approximate packing, you can use TYPE_GUILLOTINE_PACK.
for example
packer = Packer.create(Packer.TYPE_GUILLOTINE_PACK , max_width=2048, max_height=2048, bg_color=0xffffff00)


Multi Juicer in an offline network

Multijuicer stops working within an offline network environment. The following steps were followed to install:
sudo apt install docker.io docker-compose -y
sudo snap install kubectl --classic
sudo curl https://raw.githubusercontent.com/helm/helm/main/scripts/get-helm-3 | bash
sudo curl -LO https://storage.googleapis.com/minikube/releases/latest/minikube-linux-amd64
sudo install minikube-linux-amd64 /usr/local/bin/minikube
minikube start
kubectl cluster-info
helm repo add multi-juicer https://iteratec.github.io/multi-juicer/
helm install multi-juicer multi-juicer/multi-juicer
kubectl get pods
wget https://raw.githubusercontent.com/iteratec/multi-juicer/main/guides/k8s/k8s-juice-service.yaml
kubectl apply -f k8s-juice-service.yaml
kubectl port-forward --address <IP_ADDRESS> service/juice-balancer 3000:3000

When the system is brought into the isolated network, started with kubectl port-forward --address <IP_ADDRESS> service/juice-balancer 3000:3000 and a client attempts to connect, here is the result:

Any ideas?

Hi!
Can you check if the pods are actually starting?
I think they might not be, as the default image pull policy is set to Always: https://github.com/iteratec/multi-juicer/blob/main/helm/multi-juicer/values.yaml#L5 and you don't seem to be overwriting it.
When you are running this without network access you'll have to preload the images beforehand and install MultiJuicer with setting the imagePullPolicy to Never or IfNotPresent.
helm upgrade --install multi-juicer multi-juicer/multi-juicer --set="imagePullPolicy=Never"

Other than that there shouldn't be anything blocking multi-juicer from running in a offline network setting, probably a issue with the cluster setup.


What distro should be support ? (After CentOS 8 EOL)

Related issue:

https://github.com/fluent-plugins-nursery/fluent-package-builder/issues/235

CentOS8 is EOLed already.

Rocky? Stream?

Rocky Linux 8 or Alma Linux 8 is better.
Maybe Rocky Linux 8 is more popular.


Error

Product and Version [VS/VSCode]:
OS Version [macOS/Windows/Linux]:
Live Share Extension Version:
Target Platform or Language [e.g. Node.js]:
What were you doing when you encountered the desync?
How often have you encountered desyncs?

Thanks for reporting this desync, @davidvanegas2. I'm going to close this as a duplicate of #2086. Please join the conversation there for updates.


custom --jp-editor-mirror-lsp-highlight-background-color value

Hello,
I don't understand how to set a custom color for "--jp-editor-mirror-lsp-highlight-background-color".
Thank you very much!

Are you developing a theme? Something like:
:root {
  /* highlight */
  --jp-editor-mirror-lsp-highlight-background-color: color-mix(
    in srgb,
    red 25%,
    transparent
  );

should work based on what is defined in:
https://github.com/jupyter-lsp/jupyterlab-lsp/blob/fdd3affdf07d8077c19433ef299e1d0fa777818e/packages/jupyterlab-lsp/style/variables/base.css#L1-L7
Of note, in JupyterLab 4.1 you will be able to include it in custom.css (see https://jupyterlab.readthedocs.io/en/latest/getting_started/changelog.html#v4-1)

Thanks a lot!
I understand now that it's not jupyterlab-lsp related at all, but I didn't know how themes worked in jupyter. This new option in version 4.1 is awesome. It works perfectly!
Thanks again and sorry for the inconvenience!

Fixed


Set up a custom domain requires prerequisite of a custom domain

[Enter feedback here]
very confused by this help page - I want to set up a custom domain using azure DNS - but the prerequisite is to already have a custom domain?  Am I supposed to purchase a custom domain elsewhere to use here, or what?

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

ID: 24ba4664-0090-280b-fde2-287c0a3d9593
Version Independent ID: 0eeb7bb3-5c9e-1e7b-110d-2d267e8ac165
Content: Set up a custom domain with Azure DNS in Azure Static Web Apps
Content Source: articles/static-web-apps/custom-domain-azure-dns.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@slacker298 Thanks for the feedback! We are currently investigating and will update you shortly.

@slacker298 Thanks for asking question! Yes, to set up a custom domain with Azure DNS in Azure Static Web Apps, you need to have a custom domain already registered with a domain registrar.
You can purchase a custom domain from a domain registrar such as GoDaddy, Namecheap, or Google Domains. Once you have purchased the domain, you can follow these steps to set up the custom domain with Azure DNS.
Let us know if further query or issue remains.

@slacker298 We will now proceed to close this thread. If there are further questions regarding this matter, please let us know.


Make info icon easier to read

Let's do this for "help" icon too! (Maybe similar to what you had in the original - sorry that I didn't have time to bring that over.)
We'll also want a max height (maybe calc(100vh - common.$header-height * 2)?) with overflow auto.


Expand quick info information for imports

TypeScript Version:  2.6.0-dev.201xxxxx
Code
foo.ts
export class Test<T extends object> {
}

bar.ts
import { Test } from './foo';

Test/* 1 */;

Expected behavior:
Quick info at 1 displays full type information like this:

Ideally with a prefix or something to indicate it's an import.
Actual behavior:
It only displays import Test on hover.
I debated opening this because Peek Definition exists, but that only works if your cursor is on the identifier. When you're scrolling through unfamiliar code, I think it'd be really nice to see fully resolved types as you hover over stuff without having to give up your cursor position, especially if you're jumping around different files. Even if it requires a little 'Loading...' tag first.
This seems like it should have a duplicate or two but I didn't find anything.

Duplicate of #5515?


Improve Application updates when write-back method is ArgoCD (#965)

cherry-pick from master to release-0.15 to improve application updates when write-back method is ArgoCD

I think is too big of a change for a patch release. Probably should release that with a 0.16.0?

Ah. YOLO :)


Use-after free in Op::connect

I was wondering why connecting to IPv4 addresses sometimes worked.. but IPv6 doesn't. Sometimes I got an errno 97 (address family not supported), sometimes it was just hanging... and adding a couple dbg! statements made it work?
All of this smells a lot like memory unsafety.. and it is!
Here, a Connect struct is created, moving a SockAddr into it:
https://github.com/tokio-rs/tokio-uring/blob/1c0a2bff4a5f8fdc34e91cac30b74305c3486e88/src/driver/connect.rs#L17-L30
Then, in submit_with, the data is moved into an Op struct, the callback is called to actually configure the SQE from the given data...
https://github.com/tokio-rs/tokio-uring/blob/1c0a2bff4a5f8fdc34e91cac30b74305c3486e88/src/driver/op.rs#L74-L78
And then op is dropped (and the data with it). We now have an op in the submission queue that refers to some userspace address that's been freed (somewhere on the stack, presumably). This probably works well in tests because immediately after attempting to connect, it sleeps. But we now have the kernel using some memory after we've freed it.
A potential fix would be to move the data into the Lifecycle::Submitted variant - but that of course raises the question of: should it be boxed? Probably not right? So there's only so much data we can stuff in there - maybe connect is one of the rare ops that needs something like this, so maybe it should itself contain an enum?
At any rate, that explains the random failures I've been getting.

Ah, the accept situation is a bit different since it's an out pointer. I see what holds the data now - the op isn't dropped, it's returned. That's why when I simply tried boxing the connect address, it also fixed it. It wasn't luck (that the heap memory location wasn't getting overwritten), it was that even though I was moving the Op future around, the location of the connect address remained stable.
So, Boxing the address would fix the connect thing for good, but I still feel like having the "future handle" hold associated data is the wrong model. If one were to drop the connect/listen future before the operation completes, the kernel would be reading from / writing to invalid memory - same lifetime concerns as buffers.

Is it the accept op or the connection op that you think is dropping the fd too early?
I don't know all the details of the latest Completable design but I hadn't had problems with the earlier one. I thought the code paths kept the fd in a SharedFd that only got dropped when the Op was dropped and the operation had completed per a cqe entry.

Oh, not the fd, the address?

Exactly. As things stand now, both accept and connect are unsafe:

accept cannot be dropped, or whenever the kernel does accept a socket it'll write the local_addr in a freed heap memory location
connect cannot be dropped (or kernel reads from a freed location) but it also cannot be moved, which moves the location of the SockAddr and has the kernel read garbage, too.

That model is flawed imho - both the inputs AND the outputs must be kept in the slab so they never move until the operation is successfully completed OR successfully cancelled. The future handle must be just that - a handle.

What version of kernel are you testing on? For sometime, the kernel driver folks have said the data only needs to be good
until the 'enter' call is made, not the whole time the operation is in flight.

What version of kernel are you testing on?

This is on:
❯ uname -r
5.15.0-48-generic

For sometime, the kernel driver folks have said the data only needs to be good
until the 'enter' call is made, not the whole time the operation is in flight.

Submissions are batched: if the queue isn't full, submit_with just does sq.push, which does not enter. (But you're right that this means the inputs can probably live right until enter). I literally started playing with io_uring today, so I'm still not sure about anything, but - isn't there also a mode where submitting ops doesn't require calling enter at all?

Confused by this. The kernel doesn't write to any of our data, it returns a result and some flags bits. Do you mean you think the kernel has kept the original SockAddr pointers during its accept? I didn't see that documented anywhere in their man pages but I've never tried looking at their kernel code.

That is what I think is happening, yes - we're giving the kernel a place to store the local_addr for the accepted connection:
https://github.com/tokio-rs/io-uring/blob/aae166577e307fd9f39d1d0401234b78b372ebb0/src/opcode.rs#L558-L581

Accept is fine because on drop the state gets dropped into the driver (Lifecycle::Ignored).

Accept is fine because on drop the state gets dropped into the driver (Lifecycle::Ignored).

That's true! That means this issue was closed by #126.


Release Arrow  12.0.0 (next release after 11.1.0)

This is a tracking ticket for the next arrow release after https://crates.io/crates/arrow/11.1.0 was released 2022-04-04
The next release from master contains some significant API changes (new prost/tonic, json decoder options, etc)
See full list here:
https://github.com/apache/arrow-rs/compare/11.1.0...master
cc @viirya @jhorstmann @tustvold

The PRs are coming fast and furious -- I plan to prepare changelog / release tomorrow (Friday) so if there are any issues that we want to get in (as in not wait another 2 weeks) please highlight them here and get them ready to merge asap

It would be really cool to get https://github.com/apache/arrow-rs/pull/1560 in, so that I can use it within DataFusion

It would be really cool to get https://github.com/apache/arrow-rs/pull/1560 in, so that I can use it within DataFusion

✔

Change log / version update is ready: https://github.com/apache/arrow-rs/pull/1569
I'll plan to try and create a release candidate later today to give people a chance to comment

But then we have the three days of voting for them to comment as well, so I don't really plan to leave the changelog up for any extended period of tiem

https://lists.apache.org/thread/j5d5m33ps56nqt79l7lqs4zmtrmgc9jq -- release approved and released
The release is available here:
https://dist.apache.org/repos/dist/release/arrow/arrow-rs-12.0.0
It has also been uploaded to crates.io:
https://crates.io/crates/arrow/12.0.0
https://crates.io/crates/arrow-flight/12.0.0
https://crates.io/crates/parquet/12.0.0
https://crates.io/crates/parquet-derive/12.0.0


Add support for h6

Fixes the issue brought up in Discord

Is there a reason that headings stop at h5 in all the types, and there is not h6? Using the markdown plugin the editor renders h6 fine and I'm still able to style them in the theme if I ignore the types

And closes #2355

Note: Doesn't add H4, H5, and H6 in the dropdown but ensures the playground supports the types h4, h5 and h6
Also, the icons used (as bootstrap only provides icons for H1, H2 and H3) are made by me (so no copyright issues :))

Hey @im-adithya, mind checking the failing tests please? A simple rebase may fix it

Done @zurfyx! :D


suggestion: add redstone wireless networks

just putting it out there because there currently isn't a wireless redstone mod afaik

Thanks for the suggestion but this isn't really in scope for the mod, it's a completely different feature implementation-wise


Collapse: Warning Failed context type: Minified exception occurred;

i am using the default example from here -> https://reactstrap.github.io/components/collapse/
and i get that warning ->
Warning: Failed context type: Minified exception occurred; use the non-minified dev environment for the full error message and additional helpful warnings. in e (created by t) in t (created by OverviewTab) in div (created by OverviewTab) in OverviewTab (created by LaborersTabs) in div (created by Pe) in Pe (created by LaborersTabs) in div (created by ke) in ke (created by LaborersTabs) in div (created by _) in _ (created by LaborersTabs) in div (created by t) in t (created by LaborersTabs) in div (created by LaborersTabs) in LaborersTabs
i am using reactstrap via cdn links

Closing this issue, feel free to re-open if you are still running into issues.


chore: bdd-nh-import-1585243334 to 0.0.1

chore: Promote bdd-nh-import-1585243334 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Multi-threading issue

I'm trying to use TensorBoardLogger in a multi-threaded environment. Every thread gets its own logger, writing to a different file. But I'm still getting occasional exceptions like the one below (ProtoBuf related ???).
Is there anything else that I should keep in mind?
TaskFailedException:
BoundsError: attempt to access 2×8 BitArray{2} at index [1:2, 9]
Stacktrace:
 [1] throw_boundserror(::BitArray{2}, ::Tuple{UnitRange{Int64},Int64}) at ./abstractarray.jl:537
 [2] checkbounds at ./abstractarray.jl:502 [inlined]
 [3] view at ./subarray.jl:155 [inlined]
 [4] maybeview at ./views.jl:133 [inlined]
 [5] dotview at ./broadcast.jl:1160 [inlined]
 [6] _unset_oneofs(::BitArray{2}, ::Array{Int64,1}, ::Int64) at /home/peter/.julia/packages/ProtoBuf/JY3fQ/src/codec.jl:612
 [7] _fillset(::TensorBoardLogger.Summary_Value, ::Symbol, ::Bool, ::Bool) at /home/peter/.julia/packages/ProtoBuf/JY3fQ/src/codec.jl:641
 [8] fillset at /home/peter/.julia/packages/ProtoBuf/JY3fQ/src/codec.jl:633 [inlined]
 [9] setproperty! at /home/peter/.julia/packages/ProtoBuf/JY3fQ/src/utils.jl:5 [inlined]
 [10] _protobuild(::TensorBoardLogger.Summary_Value, ::Base.Iterators.Pairs{Symbol,Any,Tuple{Symbol,Symbol},NamedTuple{(:tag, :simple_value),Tuple{String,Float32}}}) at /home/peter/.julia/packages/ProtoBuf/JY3fQ/src/utils.jl:61
 [11] Summary_Value at /home/peter/.julia/packages/TensorBoardLogger/gv4oF/src/protojl/summary_pb.jl:63 [inlined]
 [12] scalar_summary at /home/peter/.julia/packages/TensorBoardLogger/gv4oF/src/Loggers/LogValue.jl:19 [inlined]
 [13] log_metric(::TensorBoardLogger.TBLogger, ::Symbol, ::Float32, ::Int64, ::DateTime) at /home/peter/dev/Quanta.jl/test/multi.jl:5
 [14] macro expansion at /home/peter/dev/Quanta.jl/test/multi.jl:21 [inlined]
 [15] (::var"#29#threadsfor_fun#8"{UnitRange{Int64}})(::Bool) at ./threadingconstructs.jl:81
 [16] (::var"#29#threadsfor_fun#8"{UnitRange{Int64}})() at ./threadingconstructs.jl:48
wait at task.jl:267 [inlined]
threading_run(::Function) at threadingconstructs.jl:34
macro expansion at threadingconstructs.jl:93 [inlined]
test123() at multi.jl:15
top-level scope at multi.jl:27
include_string(::Function, ::Module, ::String, ::String) at loading.jl:1088

This code can reproduce the issue for anyone who is interested:
import TensorBoardLogger
using Dates

function log_metric(logger::TensorBoardLogger.TBLogger, name::Symbol, value::Real, step::Int, time::DateTime)
    summary = TensorBoardLogger.SummaryCollection(
        TensorBoardLogger.scalar_summary(string(name), value)
    )
    event = TensorBoardLogger.Event(wall_time=Dates.datetime2unix(time), summary=summary, step=step)
    TensorBoardLogger.write_event(logger.file, event)
end

function test123()
    Threads.@threads for i in 1:500
        logger = TensorBoardLogger.TBLogger("tmp/run_" * string(i))
        now = Dates.now()
        for step in 1:10_000
            log_metric(logger, :test, 12.0f0, step, now)
            sleep(0.001)
        end
    end
end

test123()

Looks protobuf related to me.
cc @tanmaykm

Based on the stacktrace indeed looks like that. And if it is a bug (and not expected behaviour, like tensorboardlogger cannot be used in a multithreaded env), feel free to close it and I can always re-open an issue against protobuf.

Yes. ProtoBuf.jl keeps some state in a common dict. It has not been made thread safe yet.

Good to know. I guess for the time being I can always serialize access to the logger.

HI @jbaron and thanks for the Bug Report.
Indeed I had never tested Tensorboard for multithreading. There would most probably also be issues when writing to file.
I'd leave this issue open for reference.
A more minimal MWE is
using TensorBoardLogger
function test123()
           Threads.@threads for i in 1:500
               for step in 1:10_000
                   TensorBoardLogger.Summary_Value(tag="ciao", simple_value=3.2)
                   sleep(0.001)
               end
           end
       end

test123()

@tanmaykm the error arises on _protobuild, but I fail to see where there is shared state in that codepath...
what common dicts are you talking about?

I was refering to these.

@tanmaykm from what I understand those are used to track what fields of a protobuf structure are set to a non-default non-null value? Probably we should move this discussion over to ProtoBuf.jl, but did you ever consider alternative strategies, such as moving those structures to a field of the struct itself, or something like this?

Yes indeed. I think moving that into the struct and having getproperty - setproperty would solve this. I have opened this issue (https://github.com/JuliaIO/ProtoBuf.jl/issues/140) on ProtoBuf.jl for this.

Closed by #80


Remove linux shared object files, provide dependancy instructions

Rather than shipping shared object files which may not be compatible with the system,
it seems more common to defer shared object files to system package management systems
such as yum or apt.
dependancies have been updated accordingly.
Closes #162

This still needs some thought - for instance we want to support managed-linux installs out of the box (i.e. not require non-root users to install dependencies).
Shipping the .so in 1.4 seemed to work OK, and specifying LD_LIBRARY_PATH does work for this install, but it would be better if the executable also looks in the expected location(s).
Perhaps we should try to use LD_RUN_PATH again looking in the $origin (i.e binary folder), then the lib dir? followed by the OS paths?

c030933 adds DT_RUNPATH values of $ORIGIN and $ORIGIN/../../../lib to the elf records.
This means that .so files can be placed in either the same directory as the binary (runtime only) or in the lib directory (runtime and/or compile time), supporting users on managed systems who must simply find the relevant .so files for their systems. The rc1.5/readme branch has been updated with some information on this (8fbd36d).
I now believe that this is ok to be merged, although we may wish to further expand the documentation to point to relevant .so files for some distributions (ubuntu, fedora, cent?)


Notification on Selection for Typeahead

Implement event notification for user selection of typeahead, so developer can capture selection and perform additional additional actions.
The AngularStrap project's typeahead, a wrapper for Bootstrap (vs. native AngularJS), fires a common event for all typeahead fields.
Example:

User enters an Employee ID.

Typeahead information gathered from $http request as field is filled.

User selects Employee ID from typeahead list, either by click or key stroke.
updater makes $http request for complete employee record and fills in remaining form elements.
User reviews fields and can update (e.g., update "Richard" to "Bob" in name field - or - job title is incorrect).

Additionally, typeahead is hooked up to name field so user can type in name instead of Employee ID.  Same process as above follows.
As a result, it would be very beneficial if each typeahead had its own updater callback instead of a single common event fired when any typeahead is updated.  Multiple typeaheads could exist on the same page.
Below is a jQuery implementation of the concept:
HTML:
<input type="text" pattern="[0-9]*" class="span6" id="textbox_EMPID" data-provide="typeahead" placeholder="8675309">

JavaScript:
$('#textbox_EMPID').typeahead({
  minLength: 3,
  source: function (query, process)
  {
    $.get(
      '/visitorLog/search',
      { b : query },
      function(data)
      {
        process(data);
      },
      "json"
    );
  },
  updater: function (item)
  {
    $.get(
      '/visitorLog/search',
      { b : item, emp : true },
      function(data)
      {
        jQuery.parseJSON(data).forEach(function(element, index, array)
        {
          // fills out the remaining form fields based on employee id search
          autoFieldComplete(element);
        });
      },
      "json"
    );

    return item;
  },
/*
  matcher: function (item) {
  },
  sorter: function (item) {
  },
  highlighter: function (item) {
  },
*/
});

Typeahead selection event could return the selected value or the selection object.  If selected object can be returned it opens up ability to query database only once, pulling a complete record with a single typeahead selection.
Basic extension of use case:

Manufacturing Parts Search.  On selection of part number from typeahead, the complete part record is queried and displayed.
Subcategory Selection.  The selection made in a "category" field determines what typeahead options are available in "subcategory" section, which would have to be retrieved after the initial selection (similar to an "editable combobox", allowing user to select existing or enter new value).

The notion is simply that an value from a typeahead field could one piece of a larger record, which could by queried on if the developer has access to the updater (to use Twitter Bootstrap terminology) event.

typeahead-on-select does work @hectormateosoblanca.  This worked for me.
http://stackoverflow.com/questions/16109364/angular-ui-bootstrap-typeahead-callback-on-selectmatch/17994624#17994624
@JobaDiniz $viewValue does not exist in the spec.  See:
https://github.com/angular-ui/bootstrap/blob/91ac17c9ed691a99647b66b3f464e3585398be19/src/typeahead/test/typeahead.spec.js#L372


blob uploading successfully but creting folder insted of image

Please see the above image. I have uploaded blob data but it is creating an empty folder every time. It should create an image file.

Hi, maybe you made the same mistake I did.
The sas you have to use is:
Config: UploadParams = {
sas: '?sv=2015-30-11&sws=b&sert=seco&rsp=rwl4ac&se=2120-02-31T44:52:54Z&st=2021-04-32T06:23:54Z&spr=https&sig=DFDFokreTERTGdgfijdigdg##!!',
storageAccount: '*****',
containerName: '*******'
};
and not the full address (like this: https://pluto.blob.core.windows.net/?sv=2015-30-11&sws=b&sert=seco&rsp=rwl4ac&se=2120-02-31T44:52:54Z&st=2021-04-32T06:23:54Z&spr=https&sig=DFDFokreTERTGdgfijdigdg##!!)
Hope this help,
bye
Ennio


[BUG][docs] Stop suggesting that calling building in the same job as publishing is okay with OIDC enabled

I just realized that https://github.com/hynek/build-and-inspect-python-package#usage shows that it's okay to call build tooling in a job that has id-token: write. This is something that we've been trying to discourage since the beginning of TP. I view it as insecure, as mentioned earlier @ https://github.com/hynek/build-and-inspect-python-package/issues/105#issuecomment-2136262019.
The fix is to change the example to use two jobs or add a big red warning mentioning the dangers. @woodruffw managed to fix this in GitHub's own docs, after months of waiting for reviews: https://docs.github.com/en/actions/security-for-github-actions/security-hardening-your-deployments/configuring-openid-connect-in-pypi#updating-your-github-actions-workflow / https://github.com/github/docs/pull/32146.

cc @jamesbraza


Cubeb mixer

Patches from @jyavenard Gecko Bug 1432779 and Bug 1431221

I fixed up CMakeLists.txt and a problem with use of assert(false) in cubeb_pulse.c.

Adapted from FFmpeg's libswresample, this mixer allows to convert from
any sane layout to any other and work directly on interleaved samples.

What does adapted mean?  Given the addition of Michael Niedermayer's copyright and the av_ naming of various parts of the code, I imagine this code retains the original license.  It looks like rematrix.c is LGPL, and with libcubeb being ISC, that is quite a significant change.  I don't think  we can  include this in Gecko without moving libcubeb into the lgpllibs library, either.

What does adapted mean? Given the addition of Michael Niedermayer's copyright and the av_ naming of various parts of the code, I imagine this code retains the original license. It looks like rematrix.c is LGPL, and with libcubeb being ISC, that is quite a significant change. I don't think we can include this in Gecko without moving libcubeb into the lgpllibs library, either.

I discussed this with @jyavenard on IRC.  The summary is that this code is the best available for our needs, so we'll deal with the licensing.  The current plan is to delineate the LGPL code clearly and provide the ability to build without it (leaving the ISC pue code having basically no remixing ability).  Within Gecko, we'll move libcubeb into lgpllibs.
Longer term (as part of the Rust rewrite work), we'll look at dropping this LGPL code and using libswresample directly for mixing, resampling, and other format conversions.  I'll file a separate issue to discuss and track that work.

This is now https://github.com/kinetiknz/cubeb/pull/426.


MetadataGenerator name collisions

From to-do in Meta/IdentifierFactory.cpp#L9
Reproduced with the KontaktSDK
In the metadata generator there are some static maps that contains hardcoded symbol names from the iOS SDK for which we know there is conflict in their JS name and must be renamed. Maybe there is better way to pass this map to the IdentifierGenerator. For third-party libraries the map may be parsed from file or something more flexible.
The most easy to consume way is if they can be generated from the decls, this won't involve manual setup. However it also involves some magic. Developers should be notified for such collisions and the way they are resolved. e.g. trace that: the TKTRegion interface collides with the TKTRegion protocol and the protocol is renamed to TKTRegionProtocol.

We have workaround for KontaktIO (0f67a2bb27c320e801fb7592bbcf39fe68729db0), but it is dirty solution. We just hardcode the names of the protocols which are in collision in our metadata generator. The solution is temporary until we find a better one.

@protocol MyName
+ (void)method;
@end

@protocol MyNameProtocol
+ (void)method1;
@end

@interface MyName : NSObject
@end

In this case the MyName protocol can't seem to be visible in swift

It seems that Swift has similar problems.
Case 1
char *timezone(int, int);
extern long timezone;

print(timezone) // Ambigous use of 'timezone'`

Case 2
@protocol MyName
+ (void)method;
@end

@protocol MyNameProtocol
+ (void)method1;
@end

@interface MyName : NSObject
@end

In this case the MyName protocol isn't visible in swift.

Sounds fine to me.

What do you think to report errors at runtime, instead of build time. Because we may sometimes fail the build, but if the user doesn't access this API, he wouldn't be able to do so.

How would we know at runtime that there has been a name collision at build time?

We could mark the name in the metadata that it is ambiguous or something.

With a flag, perhaps? Alright, I think this will work.

Those are some implementation details™. Lets clear the use case first :)

Fixed in #299


Add Person wizard should support continue by hiting the enter key

Because the wizard has some pages which do only have a select element (and no <input> elements) the default browser behaviour of using enter to send the form does not work.
I think we need to add that bahaviour with javascript for this wizard.

But be carefull on the last page where we have a field to enter the address. I should still be possible to enter multiple lines for an address.

Done in #192

Needs to be released.


Wrong index size usage for Ogre hardware index buffer when iterating through submeshes

Upon calling Ogre::HardwareIndexBuffer::lock(Ogre::HardwareBuffer::HBL_READ_ONLY), the pointer to the locked memory is casted to improper type in case of indices with size of 32 bits. This caused segmentation fault on Mac OS X when meshes (in this case 5 MB at least were used for the testing) are loaded in the scene. If you look at Ogre source file for HardwareIndexBuffer, notice that a size of unsigned int is used for 32-bit indices. If the casting is changed to be unsigned int here and here, the crash is gone.
Also, it could be worth mentioning that this appeared when I switched to using Tundra 64-bit version with Qt 4.7.1 linked against Cocoa framework (64-bit) and Ogre 1.7.3, as everyone is dropping Carbon support, and transitioning to Cocoa (Qt is now "fully Cocoa" by default).
I am noting this as a issue, because I do not know if it is safe and right to apply this on Windows yet - seems to be working fine there, maybe it has better memory management?

Should Cvetan apply this to rex repo or do you want to handle this in Ludos branch?

Looks good.


Consider changing artifactType cardinality to optional

this would be for compatibility with SPDX and perhaps simplify the spec.
Note the related issue in SPDX to add this as a required field (https://github.com/spdx/spdx-spec/issues/171) - one of these approaches (but certainly not both) is needed to make the SBOM and SPDX compatible in 3.0.

this comes from the CycloneDX model
the rationale for having a field and not to infer from relationships is for efficient processing of documents, e.g., to quickly discriminate between hardware, operating system, and software


Web Api error using RestSharp (.POST)

Im using RestSHarp for send data (Post) to serenity web api, and i have this problem. With the login that is a .POST i dont have problems.
Can anyone help me? Thanks.

This is the code of Restsharp that i use for send data (.Post)(RestSharp)
var request6 = new RestRequest(Method.POST);
request6.AddHeader("Content-Type", "application/octet-stream");
request6.Resource = "Services/Profesores/Profesores/Create";
request6.RequestFormat = DataFormat.Json;
string a = "{";
string b = ""Entity""; ;
string c = ":{"Nombre":"Lorenzo","Apellido":"Ferandez","Dni":123123123,"FechaNacimiento":null,"Sexo":"Masculino","Celular":null,"Domicilio":"SALTA SALTA SALTA","ActividadList":null,"Mail":"locosaltaloco@hotmail.com","Observaciones":null}";
string all = string.Concat(a, b, c);
request6.AddParameter("application/json", all);
request6.AddCookie(responseConex.Cookies[0].Name, responseConex.Cookies[0].Value);
IRestResponse response2 = restClient.Execute(request6);

did you first authenticate (to get authorizations) to perform requests ?

What you say is this?

Did get results with almost same code : (i simply added the header to specify content-type)
```
var restClient = new RestClient("http://localhost:59205/");
var request = new RestRequest(Method.POST);
request.Resource = "/Account/Login";
request.AddParameter("Username", "admin");
request.AddParameter("Password", "serenity");
restClient.CookieContainer = new CookieContainer();
  var response = restClient.Execute(request);

  request = new RestRequest("/services/Referential/RomCommonNetwork/List", Method.GET);
  request.AddHeader("Content-Type", "application/json; charset=utf-8");

  response = restClient.Execute(request);
  var content = response.Content; // raw content as string

Try something like this :      request = new RestRequest("/services/Referential/RomCommonNetwork/Create", Method.POST);
request.AddParameter(new Parameter() {Value = "{"Entity": {"TicketingReferenceId": 1,"CommonNetwork": 35,"TransportNetworkId": 4,"Label": "Reseau Commun Bidon"}}"});
request.AddHeader("Content-Type", "application/json; charset=utf-8");
  response = restClient.Execute(request);
  var content = response.Content; // raw content as string

To make it work simply send everything same time :
```
var json = "{"Entity": {"TicketingReferenceId": 1,"CommonNetwork": 36,"TransportNetworkId": 4,"Label": "Reseau Commun Bidon"}}";
request.AddParameter("application/json; charset=utf-8", json, ParameterType.RequestBody);

Thank a lot! But I still give the same error...

Did you try removing content type specifying 'application/octet-stream' ?

Error from response2:

Closing due to inactivity.


feat(RadialMenu): add dead zone to radial menu

feat(RadialMenu): add dead zone to radial menu
Allow the player to touch the centre of the touchpad without
immediately triggering OnHoverEnter event of a button.

Implements #1486

Only suggestions I have:

Change the tad naming to something more verbose as acronyms are generally not used anywhere else.
The TouchAngleDeflection constructor should go after the fields.
The TouchAngleDeflection constructor arguments shouldn't be acronyms. You can name them the same as the fields, you just have to prefix your field usage in the constructor with this. then. 😃

What should I change 'tad' to?

if tad is an acronym of touchAngleDeflection then that's fine. Unless you're using that variable name internally elsewhere, then you can just use something like givenTouchAngleDeflection as parameters in methods,

Yeah it's just an acronym of touchAngleDeflection. I don't use 'tad' except as a function argument name. I do use initialTad and currentTad in the radial menu controller - I'm not sure if those are acceptable?
I understand @bddckr's third point and will make that change tonight but I'm not sure on the second point (where to move the definition). The only fields in that file are inside the VRTK_RadialMenu class, but the TouchAndDeflection is defined outside of that class. Should it be moved and if so where to?

I've renamed 'tad' as I'm not sure where it would be acceptable.
I've moved the struct definition to between the using statement and the delegate definition. I saw other helper structs in this position in other files.
I haven't added any help comments to the struct - some other structs have them and some don't. If you'd like me to add one please let me know.

but I'm not sure on the second point (where to move the definition)

I'm talking about the TouchAngleDeflection struct. In that struct you currently have the constructor first, fields second. The existing codebase does it the other way round AFAIK.
It's all just nitpicking from me, so don't let that discourage you or something: It's a nicely done PR and a welcome addition. 😄

Ah I get what you mean now thanks, yep I'll fix that.

For some reason this PR now has 5 commits in it. I think you need to git squash it into 1 commit

There were the following issues with your Pull Request

Commit: 6d9b67bbf0ccb4b8ed94aaacfe4376c50dabfd7f

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: 641a031d333d6bef535bb397409a1cf2d8e5b032

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: a43e04718c226bb254d2e7304b2b44c2040b47b7

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: 599d3722a18e224f6bc3d0251597d19bc1dd6fe1

Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

Well I think I've broken the branch, I was trying to squash the commits and got something wrong which has added a load of other changes to this pull request. I may need to start again, if so I'll create a separate pull request as I think this one has had it.

There were the following issues with your Pull Request

Commit: 6d9b67bbf0ccb4b8ed94aaacfe4376c50dabfd7f

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: 641a031d333d6bef535bb397409a1cf2d8e5b032

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: a43e04718c226bb254d2e7304b2b44c2040b47b7

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: 599d3722a18e224f6bc3d0251597d19bc1dd6fe1

Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

I've created a new pull request which has a single commit: https://github.com/thestonefox/VRTK/pull/1509
@thestonefox, what would have been the correct way to squash the commits without having to introduce a new branch (and so a new pull request)? As far as I can tell, you can only squash while merging.

You can always squash commits using interactive rebase.

Thanks @bddckr. I thought git philosophy was that commits are sacrosanct (even editing the commit message is difficult), but I'm guessing that's only once the pull request is accepted so the code is in multiple repos?
Hopefully this is now a clean pull request: https://github.com/thestonefox/VRTK/pull/1510


Hide empty drive bays in the sidebar

What's the Problem?
In Explorer, drives like disc drives and floppy drives are only shown in the sidebar when there is something in them. In Files, empty drive bays are still shown in the sidebar.
Solution/Idea
When a drive bay is empty, Files should hide it from the sidebar (but it should still be visible on the Home screen).
Alternatives

Priorities
No response
Files Version
Version <IP_ADDRESS>
Windows Version
Version 21H1, Build 10.0.19043.1645, Windows Feature Experience Pack 120.2212.4170.0
Comments
No response

Can I work on this?

@the-garlic-os we made the decision not to auto hide empty drives but we're working on a way to see the drive size when hovering over the sidebar items. The goal of this change is to make it easier to see which drives are empty without having to open them.


Segmentation Fault

Software Versions

Python: 3.8.3
OS:  Windows 10
Kivy: 2.0.0
Kivy installation method: Anaconda

My Code
from kivy.app import App
from kivy.uix.floatlayout import FloatLayout
from kivy.lang import Builder

class RootWidget(FloatLayout):
    pass

class MainApp(App):
    def build(self):
        return RootWidget()

if __name__ == '__main__':
    MainApp().run()

Output
[INFO   ] [Logger      ] Record log in C:\Users\spame.kivy\logs\kivy_20-11-15_25.txt
[INFO   ] [deps        ] Successfully imported "kivy_deps.angle" 0.3.0
[INFO   ] [deps        ] Successfully imported "kivy_deps.glew" 0.3.0
[INFO   ] [deps        ] Successfully imported "kivy_deps.sdl2" 0.3.1
[INFO   ] [Kivy        ] Installed at "C:\Users\spame\anaconda3\lib\site-packages\kivy_init_.py"
[INFO   ] [Python      ] v3.8.3 (default, Jul  2 2020, 17:30:36) [MSC v.1916 64 bit (AMD64)]
[INFO   ] [Python      ] Interpreter at "C:\Users\spame\anaconda3\python.exe"
[INFO   ] [Factory     ] 186 symbols loaded
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_sdl2, img_pil (img_ffpyplayer ignored)
[INFO   ] [Window      ] Provider: sdl2
Segmentation fault
(base)

After that, it shows a dialog says python stop working....

Kivy 1.x.x doesn't support python 3.8.
Use recent version.
pip install --pre kivy==2.0.0rc4

Kivy 1.x.x doesn't support python 3.8.
Use recent version.
pip install --pre kivy==2.0.0rc4

I am sorry! my kivy version is 2.0.0 but it still not working. please help

please run with python main.py -c kivy:log_level:debug to have more info.

please run with python main.py -c kivy:log_level:debug to have more info.

Now it shows:
INFO   ] [Logger      ] Record log in C:\Users\spame.kivy\logs\kivy_20-11-15_5.txt
[INFO   ] [Kivy        ] Installed at "C:\Users\spame\anaconda3\lib\site-packages\kivy_init_.py"
[INFO   ] [Python      ] v3.8.3 (default, Jul  2 2020, 17:30:36) [MSC v.1916 64 bit (AMD64)]
[INFO   ] [Python      ] Interpreter at "C:\Users\spame\anaconda3\python.exe"
[INFO   ] [Factory     ] 186 symbols loaded
[DEBUG  ] [Cache       ] register <kv.lang> with limit=None, timeout=None
[DEBUG  ] [Cache       ] register <kv.image> with limit=None, timeout=60
[DEBUG  ] [Cache       ] register <kv.atlas> with limit=None, timeout=None
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_sdl2, img_pil (img_ffpyplayer ignored)
[DEBUG  ] [Cache       ] register <kv.texture> with limit=1000, timeout=60
[DEBUG  ] [Cache       ] register <kv.shader> with limit=1000, timeout=3600
[DEBUG  ] [App         ] Loading kv <.\main.kv>
[DEBUG  ] [App         ] kv <.\main.kv> not found
[INFO   ] [Window      ] Provider: sdl2

hm really weird, nothing in the log hints at a problem, maybe there is an issue with the anaconda build, try to install with a normal python installation?

The problem is that you're using anaconda, yet you're installing kivy using following the pip instructions, which is not compatible with anaconda.
Instead either use native python or for anaconda follow the conda instructions andinstall with conda install kivy -c conda-forge in a clean environment.


[1.9.4] ThermoElectric Boiler and Steam Turbine not showing informations

The ThermoElectric Boiler is not longer showing the Boil Rate and Max Boil, they're showing 0 when it should be 390.

The Steam Turbine doesn't show the right informations either, it's showing 45mb/t and 642 RF/t instead of 5k RF/t. It also looks like it's full of steam, when it's not.

It's producing power just fine but I don't know what caused it, and have no idea how to reproduce it, or make it behave normally again for that matter.

Broke a replaced a block, now the Boiler is fine but the turbine shows half the production.

I do have a theory about this, I think it's the same kind of bug as this one.
I added a tree farm nearby, and every time a tree grows or gets chopped down I see the particle effects as if the multiblock was re-forming.

  Let me know if you can prove otherwise.


Absolute input and model paths needed

Hi Nabin, I need to put the absolute paths in config/arguments.yml, otherwise I face the following error.
(c2s) [luod@gpu211-06 Cryo2Struct]$ python3 cryo2struct.py --density_map_name 34610

##############- Cryo2Struct -##############

Running with below configuration:
density_map_name : 34610
input_data_dir : input
amino_checkpoint : models/amino_acid_type.ckpt
atom_checkpoint : models/atom_type.ckpt
infer_run_on : cpu
infer_on_gpu : 0
threshold : 0.4
clustering_threshold : 2
use_sequence : full
std_lambda : 10

- This might take a bit. Time for a coffee break, maybe! -
Traceback (most recent call last):
  File "/ibex/scratch/projects/c2115/luod/Cryo2Struct/cryo2struct.py", line 131, in <module>
    main()
  File "/ibex/scratch/projects/c2115/luod/Cryo2Struct/cryo2struct.py", line 122, in main
    make_predictions(config_dict)
  File "/ibex/scratch/projects/c2115/luod/Cryo2Struct/cryo2struct.py", line 54, in make_predictions
    grid_division.create_subgrids(input_data_dir=config_dict['input_data_dir'], density_map_name=config_dict['density_map_name'])
  File "/ibex/scratch/projects/c2115/luod/Cryo2Struct/utils/grid_division.py", line 80, in create_subgrids
    protein = get_data(density_map_dir)
  File "/ibex/scratch/projects/c2115/luod/Cryo2Struct/utils/grid_division.py", line 41, in get_data
    os.chdir(density_map_dir)
FileNotFoundError: [Errno 2] No such file or directory: 'input/34610'

Hello RodenLuo,
Thank you for pointing this out.
This has been mentioned in the README file. Below is the screenshot if it.

Thanks, Nabin! I see that. The default in config/arguments.yml makes me think
input_data_dir : input
amino_checkpoint : models/amino_acid_type.ckpt
atom_checkpoint : models/atom_type.ckpt

is enough. But only after I changed them to the full path
input_data_dir : /ibex/scratch/projects/c2115/luod/Cryo2Struct/input
amino_checkpoint : /ibex/scratch/projects/c2115/luod/Cryo2Struct/models/amino_acid_type.ckpt
atom_checkpoint : /ibex/scratch/projects/c2115/luod/Cryo2Struct/models/atom_type.ckpt

it then worked.
A similar issue is when I run bash preprocess/run_data_preparation.bash input/.
I have to change
https://github.com/jianlin-cheng/Cryo2Struct/blob/f8b2f91eaab4b226fe4c5b2ded84ca9766b7cc19/preprocess/run_data_preparation.bash#L7-L8
to
 python3 preprocess/get_resample_map.py "$i"
 python3 preprocess/get_normalize_map.py "$i"

Hey @RodenLuo,
Thanks for finding that issue. You are right. I updated both the bash script and config file based on your findings. Thanks again.!


How to use GPT2 in local environment?

Hi, need I to load the gpt2 model in local environment and could I use play ground to run the code such as "argmax "Hello[WHO]" from "/Users/ximing/Desktop/gpt2""

If you want to use GPT 2 in the same process as the LMQL query runs in, you can just specify local:gpt2 as model in the from clause. Otherwise, you can also use lmql serve-model to run the model in the background. Make sure to also read the related chapter on the documentation: https://docs.lmql.ai/en/stable/language/models.html#transformers-models.
If you have custom weights to a local model, not available from the HuggingFace Hub, you can indeed also specify an absolute path, similar to how you can instantiate such models in HF with AutoModel.from_pretrained.

Hi, I have tried it, but it still gets error, my code is
import lmql
from lmql.tests.expr_test_utils import run_all_tests
from transformers import AutoModel
model = AutoModel.from_pretrained('/Users/ximing/Desktop/gpt2-medium')
def test_or():
@lmql.query
async def q():
'''lmql
argmax
"a b[OUTPUT]"
FROM
"AutoModel.from_pretrained('/Users/ximing/Desktop/gpt')"
WHERE
OUTPUT in ['a','b']
'''
result = lmql.main(q)[0]
print(result)
run_all_tests(globals())
could you give me a real example or how to fix it? thanks

You almost got it. Instead of providing "AutoModel.from_pretrained('/Users/ximing/Desktop/gpt')", just provide the path, i.e. "/Users/ximing/Desktop/gpt". Let me know if that works for you.


Flesh out API to cover edge cases and error states for POST

Post

URI
Payload
Expected Action
Return Payload
Return Headers
Meta Data

/api/v0/collections
none
returns error
error message
???
???

/api/v0/collections/{id}
none
returns error
error message
???
???

/api/v0/collections
collection
creates collection with collection
returns collection
???
???

/api/v0/collections/{id}
collection
returns error
error message
???
???

/api/v0/collections
record
creates record
returns record
???
???

/api/v0/collections/{id}
record
???
???
???
???

check json api compliance http://jsonapi.org/

type is required on POST.
fields should be in attributes:

Changing to disallow any post of multiple records at a time, and a extra error return, if an object is passed in that has a type that doesn't match the collection.

Most functionality working except the 409 error return and the tests which are mostly failing for an unknown reason.


Confusing handling of CLA agreement

I have submitted two PRs, and they have been flagged with cla-notrequired (they are both very simplistic fixes and additions). I have rechecked about whether they have been pulled in the meanwhile, and have noticed that the cla-notrequired tag has been removed on one of them, and it is pending, awaiting the signing of the CLA? The contributing guideline is setting a limit of 15 lines of code, my PRs are well below that. What has happened?
I am referring to PR #39.

I'm not sure what the exact specifications are for the CLA as I'm not a legal specialist, but for PR #46, GitHub lists 7 additions and 9 deletions, which (perhaps rather simplistically) means 16 lines were modified. #39 is an issue, not a PR, and #46 addresses the issue.
I noticed that in the PR you moved around some namespaces at the beginning of your file. I think not moving those would put you back under the 15 line limit.

Thank you for your take on this. I have modified my PR #46 to reduce the number of rearrangements and it is 12 changed lines now. Hope this helps in its acceptance.

Thank you for opening an issue against the Bot Framework Cognitive Services repo. As part of the Bot Framework v4 release, we’ve moved all the Luis and QnA work to platform SDK repo. We will continue to support and offer maintenance updates via this repo.
From now on, https://github.com/Microsoft/botframework-sdk repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the Bot Framework repo found here:
https://github.com/Microsoft/botframework-sdk/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


I'm looking for a way to import gerrit ACl's to Zoket

Hi Team,
I'm looking for a way to import gerrit ACL's to Zoket, Please let me know the steps to do it or any documentation that I can refer to mirror gerrit ACL's into Zoket.
Looking for a way to block the users at the repository level and at the branch level as well in Zoket.

"So, here is what you have to do

build a plugin in Gerrit that acts as a HTTP proxy. The plugin should enforce users to be logged in, and forward requests to a Zoekt instance, but also set a

X-Gerrit-AccountId:  1234
header.

Update the web server to extract the gerrit account ID from the HTTP request, and set it in the HTTP Context.

write a wrapper around zoekt.Searcher

https://github.com/google/zoekt/blob/b1730d8e0cc4aa35d6b2d593b83fef7cb0c81673/api.go#L263
that extracts the Gerrit user from the context. Then it should call back to the Gerrit instance, executing /check.access to verify if the user has access. If not, returon no results.
Then, when deploying, you create a firewall, so only the gerrit server can contact the zoekt server.
Overall, this is not an easy task. Good luck!"


Time Scale should align to beginning of time period

This can be reproduced with the current samples in master
Current Behavior
Samples show:

Apr 26 9PM, Apr 27 9AM, Apr 27 9PM, etc.
Apr 26 9PM, Apr 27 3AM, Apr 27 9AM, etc.

Expected Behavior
It would be nicer if these instead said:

Apr 26, 12PM, Apr 27, 12PM, Apr 28, etc.
Apr 26, 6AM, 12PM, 6PM, Apr 27, etc.

It's much easier to read the chart when it aligns on full days (or hours, weeks, months, etc.). E.g. in this case we should always try to have some labels at midnight as 9am and 9pm are weird times to align our labels to.
Also, much of the information is repetitive. You only need to print the major time period when you hit it and only need to print the minor time period in between. We should at least have an option that allows for this if we don't always do it (and I think setting it to true by default would be nice)
Steps to Reproduce (for bugs)

git clone git@github.com:chartjs/Chart.js.git
npm install
gulp build
Open samples/scales/time/combo.html and samples/scales/time/line-point-data.html

Examples
Here are a couple examples from other time series charts

I agree that that is probably a useful feature. I don't know if by making this change existing users would be unhappy. Do you think something like this should live behind a configuration setting?

For half of it. There are two things I asked for here:

Alignment on major time periods - I think this should always happen. I don't know how to describe the current behavior which seems to be random. I think aligned on time period always makes more sense than aligned randomly
Printing of labels for minor and major ticks differently. This one I was thinking probably makes sense to be an option

I can agree to part 1 being always on. I think it should actually be relatively straightforward because all I think needs to be done is taking away a few ifs.
https://github.com/chartjs/Chart.js/blob/master/src/scales/scale.time.js#L251
https://github.com/chartjs/Chart.js/blob/master/src/scales/scale.time.js#L306
and then making sure we round to the correct unit rather than the value of the round setting. The setting is used to round data points as well.
Part 2 is more complicated. Right now the axis only has the concept of tick marks. We'd need to introduce the concept of minor tick marks. I don't think it's actually too bad other than it requires duplicating a bunch of settings so they can be configured independently and having a second array of ticks (or storing the type of tick in the items in scale.ticks. I think the meat of the changes might be confined to just this loop https://github.com/chartjs/Chart.js/blob/master/src/core/core.scale.js#L550-L651

I don't think we actually need to concept of major/minor tick marks for part 2 in which case it wouldn't necessarily be that much more complicated. We already have the unit from determineUnit. So we just need to see if the time is at the beginning of that unit (or maybe the next larger or smaller unit instead. I haven't looked quite closely enough) and format a bit differently depending on whether it's the beginning of the unit or not.

That's actually a good idea. Should be relatively straightforward to do and changing the text might actually already be possible with the label callback. Changing the styling definitely requires some code changes

+1 on this idea! I like what I read.

I've documented how this should be implemented from a technical perspective in https://github.com/chartjs/Chart.js/issues/4612. I'm going to close this issue in favor of that one, which has a lot more technical detail and should be much more helpful to an implementor


Update aws-om-config.html.md.erb

need to add the pass role permissions back

Thank you, @skelly-pivotal